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

Make bisection test more expressive #156

Merged
merged 2 commits into from
Jun 26, 2020

Conversation

gwater
Copy link
Contributor

@gwater gwater commented Jun 25, 2020

see our discussion in #153 (comment)

@gwater gwater force-pushed the dev_bisection_test branch from db71201 to cbf17b0 Compare June 25, 2020 07:47
Copy link
Member

@dpsanders dpsanders left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

test/roots.jl Outdated
@@ -55,14 +55,19 @@ newtonlike_methods = [Newton, Krawczyk]
end


in_solution_set(point, solution_intervals) =
any(map(interval -> point in interval, solution_intervals))
Copy link
Member

Choose a reason for hiding this comment

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

I believe you remove the map and just write any(interval -> point in interval, solution_intervals), which should even be slightly more efficient.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, that's correct. I've adjusted the PR.

@dpsanders
Copy link
Member

Great, thanks!

@dpsanders dpsanders merged commit b5a76bb into JuliaIntervals:master Jun 26, 2020
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