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

Improve the default comparision value #12984

Open
wants to merge 2 commits into
base: Pharo13
Choose a base branch
from

Conversation

jecisc
Copy link
Member

@jecisc jecisc commented Mar 13, 2023

Do not use a random precision but use one that proved itself

This value is the default used by Polymath and comes from Numerical Recipes, 3rd edition, Section 5.7, which is pages 229-230

This is not for Pharo 11 but Pharo 12!

…ut to use one that proved itself

This value is the default used by Polymath and comes from Numerical Recipes, 3rd edition, Section 5.7, which is pages 229-230
**This is not for Pharo 11 but Pharo 12!**
@jecisc jecisc changed the base branch from Pharo11 to Pharo12 March 14, 2023 11:10
@jecisc jecisc added this to the 12.0.0 milestone Mar 14, 2023
@jecisc jecisc closed this Mar 20, 2023
@jecisc jecisc reopened this Mar 20, 2023
@jecisc
Copy link
Member Author

jecisc commented Mar 20, 2023

Bootstrap is timing out with this PR :/ I'll need to investigate why

@jecisc jecisc added the Status: Need more work The issue is nearly ready. Waiting some last bits. label Mar 20, 2023
@MarcusDenker MarcusDenker removed this from the 12.0.0 milestone Apr 25, 2024
@jecisc jecisc changed the base branch from Pharo12 to Pharo13 May 28, 2024 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Need more work The issue is nearly ready. Waiting some last bits.
Projects
Status: NeedsWork/Discussion
Development

Successfully merging this pull request may close these issues.

None yet

2 participants