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

Added Verner Methods #2274

Merged
merged 34 commits into from
Jul 9, 2024
Merged

Added Verner Methods #2274

merged 34 commits into from
Jul 9, 2024

Conversation

ParamThakkar123
Copy link
Contributor

Checklist

  • Appropriate tests were added
  • Any code changes were done in a way that does not break public API
  • All documentation related to code changes were updated
  • The new code follows the
    contributor guidelines, in particular the SciML Style Guide and
    COLPRAC.
  • Any new documentation only uses public API

Additional context

Add any other context about the problem here.

@ParamThakkar123 ParamThakkar123 changed the title Verner Added Verner Methods Jul 4, 2024
@ParamThakkar123 ParamThakkar123 mentioned this pull request Jul 4, 2024
5 tasks
@ParamThakkar123
Copy link
Contributor Author

@ChrisRackauckas Should I also shift Tsit, BS5, AutoVern methods to the OrdinaryDiffEqVerner lib ?? They are almost always defined together in some files

@ChrisRackauckas
Copy link
Member

Make an explicit RK group for those, though Tsit5 should be in its own because it's very popular so it should have the lowest dependency setup.

@ParamThakkar123
Copy link
Contributor Author

ParamThakkar123 commented Jul 5, 2024

@ChrisRackauckas Should I also shift Tsit, BS5, AutoVern methods to the OrdinaryDiffEqVerner lib ?? They are almost always defined together in some files

These methods are defined together in composite_algs.jl file. What should be done for that ??
Can you please take a look at that @ChrisRackauckas

@ChrisRackauckas
Copy link
Member

The default algorithm will need to move to an OrdinaryDiffEqDefault which depends on everything it uses in the default algorithm.

@ChrisRackauckas
Copy link
Member

The default solver needs to be its own thing. For now, just but the imports required for it before its definition.

@ParamThakkar123
Copy link
Contributor Author

ParamThakkar123 commented Jul 6, 2024

@ChrisRackauckas What changes must be made here ?? I will make those

@ParamThakkar123
Copy link
Contributor Author

@ChrisRackauckas I am getting a @fold error here. I saw that it's present in MacroTools. I tried to include it but it made no difference.

@ParamThakkar123
Copy link
Contributor Author

@ChrisRackauckas Meanwhile I am done with Rosenbrock methods.

@ParamThakkar123
Copy link
Contributor Author

@ChrisRackauckas There are some precision errors now. Due to which the tests are failing

alg_adaptive_order(alg::Feagin14) = 12
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't reformat all of the other ones. Your formatting here is incorrect. It's already corrected on master.

@ChrisRackauckas
Copy link
Member

For some reason CI didn't trigger, but that should be good to go with the last change.

@@ -410,7 +410,8 @@ export constructDormandPrince

export FunctionMap, Euler, Heun, Ralston, Midpoint, RK4, ExplicitRK, OwrenZen3, OwrenZen4,
OwrenZen5,
BS3, BS5, DP5, Tsit5, DP8, TanYam7, TsitPap8, CompositeAlgorithm, Anas5, RKO65, FRK65, PFRK87,
BS3, BS5, DP5, Tsit5, DP8, TanYam7, TsitPap8, CompositeAlgorithm, Anas5, RKO65,
FRK65, PFRK87,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are you sure you're on the right formatter version?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I am using this version of JuliaFormatter

[98e50ef6] JuliaFormatter v1.0.56

@ChrisRackauckas ChrisRackauckas merged commit 78053c1 into SciML:master Jul 9, 2024
30 of 34 checks passed
@ParamThakkar123
Copy link
Contributor Author

@ChrisRackauckas I started a PR for Rosenbrock Methods #2278

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