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

CompatHelper: bump compat for IntervalRootFinding to 0.6, (keep existing compat) #338

Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Nov 5, 2024

This pull request changes the compat entry for the IntervalRootFinding package from 0.5.9 to 0.5.9, 0.6.
This keeps the compat entries for earlier versions.

Note: I have not tested your package with this new compat entry.
It is your responsibility to make sure that your package tests pass before you merge this pull request.

@Datseris
Copy link
Member

This is actually breaking. Must change IntervalBox in the documentation as it no longer exists.

See https://github.com/JuliaIntervals/IntervalRootFinding.jl/releases/tag/v0.6.0

ds = DeterministicIteratedMap(henon_rule, zeros(2), [1.4, 0.3])
x = interval(-1.5, 1.5)
y = interval(-0.5, 0.5)
box = x × y
box = [x, y]
Copy link

Choose a reason for hiding this comment

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

The problem is here: the initial vector must have the same type as the output of the function.

Since henon_rule and henon_jacob use SArrays, this should be a SVector (and same for the two other testsets below).

Copy link
Member

Choose a reason for hiding this comment

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

thank you!!! That fixed it!!!

@Datseris Datseris merged commit c671fa9 into main Nov 16, 2024
2 checks passed
@Datseris Datseris deleted the compathelper/new_version/2024-11-05-12-40-29-974-02757858420 branch November 16, 2024 21:28
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