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

lightning: copy best model #659

Merged
merged 1 commit into from
Aug 14, 2023
Merged

lightning: copy best model #659

merged 1 commit into from
Aug 14, 2023

Conversation

dberenbaum
Copy link
Collaborator

Update per #649 (comment). @daavoo I think this is all we need for consistency with huggingface for now. WDYT?

@dberenbaum dberenbaum requested a review from daavoo August 14, 2023 15:53
@codecov-commenter
Copy link

Codecov Report

Patch coverage has no change and project coverage change: -0.03% ⚠️

Comparison is base (ee7968b) 88.33% compared to head (872b98d) 88.30%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #659      +/-   ##
==========================================
- Coverage   88.33%   88.30%   -0.03%     
==========================================
  Files          43       43              
  Lines        3138     3139       +1     
  Branches      279      279              
==========================================
  Hits         2772     2772              
- Misses        324      325       +1     
  Partials       42       42              
Files Changed Coverage Δ
src/dvclive/lightning.py 0.00% <ø> (ø)
tests/test_frameworks/test_lightning.py 6.06% <0.00%> (-0.04%) ⬇️

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@daavoo daavoo merged commit bdf39da into main Aug 14, 2023
10 checks passed
@daavoo daavoo deleted the lightning-copy-best branch August 14, 2023 17:04
@daavoo daavoo added the A: frameworks Area: ML Framework integration label Aug 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A: frameworks Area: ML Framework integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants