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

Fix docs build, add aux variable section #1248

Merged
merged 1 commit into from
Feb 17, 2025
Merged

Fix docs build, add aux variable section #1248

merged 1 commit into from
Feb 17, 2025

Conversation

GabrielKS
Copy link
Contributor

#1240 surfaced an issue in the get_decision_problem_results docstring that this fixes. The other reason the docs build was failing is that the new power flow-related auxiliary variables were not included; here I add a new section and put all the auxiliary variables there.

#1240 surfaced an
issue in the `get_decision_problem_results` docstring that this fixes.
The other reason the docs build was failing is that the new power
flow-related auxiliary variables were not included; here I add a new
section and put all the auxiliary variables there.
Copy link

codecov bot commented Feb 16, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.50%. Comparing base (6fc3590) to head (0f57a51).
Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1248      +/-   ##
==========================================
- Coverage   77.60%   77.50%   -0.11%     
==========================================
  Files         124      124              
  Lines       13840    13840              
==========================================
- Hits        10741    10727      -14     
- Misses       3099     3113      +14     
Flag Coverage Δ
unittests 77.50% <ø> (-0.11%) ⬇️

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

Files with missing lines Coverage Δ
src/simulation/simulation_results.jl 87.91% <ø> (ø)

... and 4 files with indirect coverage changes

Copy link
Contributor

Performance Results

Version Precompile Time
Main 6.00123817
This Branch 5.819014337
Version Build Time
Main-Build Time Precompile 67.993197298
Main-Build Time Postcompile 0.839412891
This Branch-Build Time Precompile 66.677214676
This Branch-Build Time Postcompile 0.824319032
Version Solve Time
Main-Solve Time Precompile 613.320477231
Main-Solve Time Postcompile 582.452325727
This Branch-Solve Time Precompile 406.898875876
This Branch-Solve Time Postcompile 366.819606851

@jd-lara jd-lara merged commit ea0bdc8 into main Feb 17, 2025
11 checks passed
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