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

Resolve an issue with empty test for training cache #84

Merged
merged 1 commit into from
Sep 7, 2023

Conversation

lhjohn
Copy link
Collaborator

@lhjohn lhjohn commented Sep 7, 2023

Resolves #79

@codecov
Copy link

codecov bot commented Sep 7, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (31ef832) 99.01% compared to head (99615b0) 99.01%.
Report is 1 commits behind head on develop.

❗ Current head 99615b0 differs from pull request most recent head 24e7110. Consider uploading reports for the commit 24e7110 to get more accurate results

Additional details and impacted files
@@           Coverage Diff            @@
##           develop      #84   +/-   ##
========================================
  Coverage    99.01%   99.01%           
========================================
  Files            9        9           
  Lines          408      408           
========================================
  Hits           404      404           
  Misses           4        4           

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

@egillax
Copy link
Collaborator

egillax commented Sep 7, 2023

Thanks! merging

@egillax egillax merged commit 85be689 into develop Sep 7, 2023
10 checks passed
@egillax egillax deleted the 79-empty-cache-test branch October 12, 2023 18:41
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 this pull request may close these issues.

2 participants