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

Evaluate multiple #182

Merged
merged 7 commits into from
Mar 6, 2024
Merged

Evaluate multiple #182

merged 7 commits into from
Mar 6, 2024

Conversation

hamishteagle
Copy link
Contributor

Context:
We occasionally need to run many sub-runs of evaluate in order to calculate sample variances in the metrics. In order to store this additional data and calculate the variances the database requires a new sub_run column.

Add sub-run column to database specification.

Adds capacity to extract sub-run env variable.

Copy link

codecov bot commented Mar 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.8%. Comparing base (88dac15) to head (83ab7b5).

Additional details and impacted files

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #182   +/-   ##
======================================
  Coverage    78.8%   78.8%           
======================================
  Files          11      11           
  Lines         968     971    +3     
  Branches      160     160           
======================================
+ Hits          763     766    +3     
  Misses        153     153           
  Partials       52      52           
Files Coverage Δ
src/insight/database/schema.py 100.0% <100.0%> (ø)
src/insight/metrics/base.py 72.5% <100.0%> (+0.3%) ⬆️

Copy link

sonarqubecloud bot commented Mar 5, 2024

@hamishteagle hamishteagle requested a review from marqueewinq March 6, 2024 10:40
@marqueewinq marqueewinq merged commit 4a7e731 into master Mar 6, 2024
5 checks passed
@hamishteagle hamishteagle deleted the evaluate-multiple branch March 6, 2024 10:43
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