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

Revert "Change indices function in the fixed_om_costs function" #1132

Conversation

nnhjy
Copy link
Member

@nnhjy nnhjy commented Nov 4, 2024

Reverts #1123 to improve commit tracking

Copy link

codecov bot commented Nov 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.33%. Comparing base (7bc3fcf) to head (fc95832).
Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #1132   +/-   ##
=======================================
  Coverage   87.33%   87.33%           
=======================================
  Files         143      143           
  Lines        4137     4137           
=======================================
  Hits         3613     3613           
  Misses        524      524           

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

@nnhjy nnhjy merged commit c3fe4a3 into master Nov 4, 2024
7 checks passed
@nnhjy nnhjy deleted the revert-1123-1122-bug-function-fixed_om_costs-throws-and-error-when-using-representative-periods branch November 4, 2024 09:22
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