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
CleanUp ROCK Methods and Memory Usage ROCK4 #741
CleanUp ROCK Methods and Memory Usage ROCK4 #741
Conversation
uᵢ₋₃ = u + a₄₁*uᵢ₋₁ | ||
u += B₁*uᵢ₋₁ | ||
integrator.opts.adaptive && (tmp = B̂₁*uᵢ₋₁) | ||
uᵢ₋₁ = u + (a₂₁ - B₁)*uᵢ₋₁ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ChrisRackauckas This is addition subtraction of the same thing but the only way to avoid it is to add another array. Also both a21
and B1
are constants so I think doing this should not be an issue.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah this should be fine.
Codecov Report
@@ Coverage Diff @@
## master #741 +/- ##
==========================================
+ Coverage 71.63% 71.64% +<.01%
==========================================
Files 89 89
Lines 29022 29022
==========================================
+ Hits 20791 20792 +1
+ Misses 8231 8230 -1
Continue to review full report at Codecov.
|
1 similar comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this a lot
@ChrisRackauckas
Reduces memory usage in
ROCK4
.Makes the code more readable and Implementation style now resembles the paper more for both
ROCK2
andROCK4
Also makes one small correction in
perform_step