Skip to content

Conversation

nhniina
Copy link
Collaborator

@nhniina nhniina commented Sep 17, 2024

This update removes unnecessary looping and individual parameter value queries.

Checklist before merging

  • Documentation is up-to-date
  • Unit tests have been added/updated accordingly
  • Code has been formatted according to SpineOpt's style
  • Unit tests pass

Copy link

codecov bot commented Sep 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.81%. Comparing base (5fc28a5) to head (cacf9a2).
Report is 50 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1093      +/-   ##
==========================================
- Coverage   86.82%   86.81%   -0.02%     
==========================================
  Files         143      143              
  Lines        4121     4117       -4     
==========================================
- Hits         3578     3574       -4     
  Misses        543      543              

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


🚨 Try these New Features:

@nhniina nhniina self-assigned this Sep 18, 2024
@nhniina nhniina requested a review from manuelma September 18, 2024 08:02
@nhniina nhniina removed the request for review from manuelma November 22, 2024 12:00
@nhniina nhniina merged commit cf90495 into master Nov 22, 2024
7 checks passed
@nhniina nhniina deleted the migration_update branch November 22, 2024 12:01
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.

1 participant