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

performance patch3 #691

Merged
merged 2 commits into from
May 26, 2021
Merged

performance patch3 #691

merged 2 commits into from
May 26, 2021

Conversation

jd-lara
Copy link
Member

@jd-lara jd-lara commented May 26, 2021

This makes some minor improvements in the speed of Standard PTDF. However, there are some underlying challenges still with ParameterJuMP JuliaStochOpt/ParameterJuMP.jl#85

@jd-lara jd-lara requested a review from daniel-thom May 26, 2021 04:44
@jd-lara jd-lara self-assigned this May 26, 2021
@codecov
Copy link

codecov bot commented May 26, 2021

Codecov Report

Merging #691 (9a159dc) into master (39c5a94) will increase coverage by 0.03%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #691      +/-   ##
==========================================
+ Coverage   84.49%   84.52%   +0.03%     
==========================================
  Files          87       87              
  Lines        7705     7710       +5     
==========================================
+ Hits         6510     6517       +7     
+ Misses       1195     1193       -2     
Flag Coverage Δ
unittests 84.52% <100.00%> (+0.03%) ⬆️

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

Impacted Files Coverage Δ
src/core/optimization_container.jl 78.24% <ø> (ø)
src/devices_models/devices/AC_branches.jl 95.91% <100.00%> (+0.46%) ⬆️
src/core/simulation.jl 88.64% <0.00%> (+0.32%) ⬆️

@jd-lara jd-lara merged commit ba73ac5 into master May 26, 2021
@jd-lara jd-lara deleted the jd/performance_patch3 branch May 26, 2021 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants