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

Add read_variables for single row variables (no timestamp dependence) #1031

Merged
merged 2 commits into from
Dec 22, 2023

Conversation

rodrigomha
Copy link
Contributor

@rodrigomha rodrigomha commented Dec 19, 2023

This change allows to read results from variables that do not depend on time, for example shortage or surplus of storage systems at the end of the horizon.

closes #1022

Copy link
Contributor

Performance Results

Version Precompile Time
Main 6.824616946
This Branch 6.98535911
Version Build Time
Main-Build Time Precompile 62.784235268
Main-Build Time Postcompile 3.007246286
This Branch-Build Time Precompile 63.592352262
This Branch-Build Time Postcompile 2.809575034

Copy link

codecov bot commented Dec 19, 2023

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (bcc95fd) 79.81% compared to head (c5ef08d) 80.41%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1031      +/-   ##
==========================================
+ Coverage   79.81%   80.41%   +0.60%     
==========================================
  Files         116      116              
  Lines       12455    12423      -32     
==========================================
+ Hits         9941     9990      +49     
+ Misses       2514     2433      -81     
Flag Coverage Δ
unittests 80.41% <85.00%> (+0.60%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
src/operation/problem_results.jl 91.93% <88.88%> (-0.25%) ⬇️
src/operation/operation_model_interface.jl 77.73% <81.81%> (+0.09%) ⬆️

... and 21 files with indirect coverage changes

@jd-lara jd-lara merged commit bb936b8 into main Dec 22, 2023
9 checks passed
@jd-lara jd-lara deleted the rh/add_1d_results branch December 27, 2023 16:57
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.

Read results for 1D variables
3 participants