Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

3.0 updates #484

Closed
4 tasks done
dberenbaum opened this issue Mar 3, 2023 · 9 comments · Fixed by #684
Closed
4 tasks done

3.0 updates #484

dberenbaum opened this issue Mar 3, 2023 · 9 comments · Fixed by #684

Comments

@dberenbaum
Copy link
Collaborator

dberenbaum commented Mar 3, 2023

Tasks

Preview Give feedback
  1. A: frameworks refactoring
  2. A: studio
    daavoo

Post-release:

#221

@daavoo
Copy link
Contributor

daavoo commented Mar 14, 2023

Edited to add #499

@daavoo daavoo pinned this issue Mar 20, 2023
@daavoo
Copy link
Contributor

daavoo commented May 2, 2023

@dberenbaum we could do 3.0 to match DVC 3.0

@dberenbaum
Copy link
Collaborator Author

Do you think we can get all the callbacks moved to their frameworks? We probably can't count on it, but I'm open to trying.

@shcheklein
Copy link
Member

Do you think we can get all the callbacks moved to their frameworks? We probably can't count on it, but I'm open to trying.

My 2cs sound like a good to have vs something that improves the experience. I would personally try to prioritize more features, plot improvements, think what we can do to further simplify the onboarding (e.g. making it work w/o .dvc, next steps in moving to pipelines, etc). What else we can do to simplify things?

@dberenbaum
Copy link
Collaborator Author

@shcheklein I was just responding to @daavoo, so I'm not pushing for this, but the primary utility IMO is to get more exposure and position dvclive as an alternative on par with other loggers. Until we put it into those frameworks, it looks like something I'd be wary of using because it doesn't seem like it's well supported or taken seriously.

@shcheklein
Copy link
Member

@shcheklein I was just responding to @daavoo,

I totally get it, Dave! Read my comment is just like literally "my 2cs", not trying to move it one direction or another.

@dberenbaum
Copy link
Collaborator Author

Added item: Set report=none by default

I don't think the default html report is that useful these days.

@daavoo daavoo changed the title 3.0 checklist 3.0 release Aug 17, 2023
@daavoo
Copy link
Contributor

daavoo commented Aug 17, 2023

I think we should leave out the following:

Both are dragged from early design decisions and feel like it would be the "correct" way of doing things. However, I worry that we will add friction and/or potential issues for existing users regardless of any effort to make it smooth.

@dberenbaum
Copy link
Collaborator Author

I also think #221 should be an action item post-release, so the plan would be:

  1. 3.0 release
  2. Add callbacks to frameworks
  3. Deprecate internal callbacks in 4.0

@daavoo daavoo linked a pull request Aug 22, 2023 that will close this issue
@daavoo daavoo mentioned this issue Aug 22, 2023
@daavoo daavoo changed the title 3.0 release 3.0 tasks Aug 22, 2023
@daavoo daavoo changed the title 3.0 tasks 3.0 updates Aug 22, 2023
@daavoo daavoo unpinned this issue Sep 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants