-
-
Notifications
You must be signed in to change notification settings - Fork 84
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
DynamicalODE Changes #1067
DynamicalODE Changes #1067
Conversation
No that's backwards in compat. |
…into DynamicalODE
@ChrisRackauckas any changes needed further ?? |
|
|
I think this is already present here : https://github.com/SciML/SciMLBenchmarks.jl/blob/master/benchmarks/DynamicalODE/single_pendulums.jmd Where can I make an update if this needs a fix ?? |
The point of the project is to get that benchmark set working on the latest versions 😅 . That's in the benchmark script of the folder. |
Oh, Got it 😅 |
But my question was why did we get a undef error if that's already defined there ?? |
Previous bad bump maybe? |
@ChrisRackauckas How can I see the errors that have occurred ??
Like here |
@ParamThakkar123 Check the README of this repo: https://github.com/SciML/SciMLBenchmarks.jl?tab=readme-ov-file#inspecting-benchmark-results, and the built benchmarks can be checked in the corresponding md files |
Seems the errors are caused by PyPlot.jl You may consider figuring out what's wrong with PyPlot.jl here or change to Plots.jl(recommend) |
The errors says that the figure function is not defined. Though it's there in PyPlot |
You can try to update the compat and the Manifest of the packages in this benchmark to the latest version, many of them are outdated |
It would probably be best to just update the plots to use Plots.jl |
Noted. I will make the changes |
@@ -2345,4 +2345,4 @@ version = "3.5.0+0" | |||
deps = ["Artifacts", "JLLWrappers", "Libdl", "Pkg", "Wayland_jll", "Wayland_protocols_jll", "Xorg_libxcb_jll", "Xorg_xkeyboard_config_jll"] | |||
git-tree-sha1 = "9c304562909ab2bab0262639bd4f444d7bc2be37" | |||
uuid = "d8fb68d0-12a3-5cfd-a85a-d49703b185fd" | |||
version = "1.4.1+1" | |||
version = "1.4.1+1" |
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.
this manifest bump is not correct!
The updates to the plots are correct, but the manifest was not bumped so this isn't actually testing the newer versions. Can you bump the manifest? Otherwise we need to revert. |
Yes @ChrisRackauckas , I will create a new pr by today which bumps the manifest with the latest versions for both DynamicalODE and BayesianInference. |
Checklist
contributor guidelines, in particular the SciML Style Guide and
COLPRAC.
Additional context
Add any other context about the problem here.