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

doc: MathConstants: add admonitions noting unknown rationality #55897

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nsajko
Copy link
Contributor

@nsajko nsajko commented Sep 26, 2024

Some of the Irrational constants are not known to be irrational, so it'd be good to note that in the doc string of each such constant to avoid being misleading.

@nsajko nsajko added docs This change adds or pertains to documentation maths Mathematical functions labels Sep 26, 2024
devmotion added a commit to JuliaMath/IrrationalConstants.jl that referenced this pull request Sep 27, 2024
* readme: note that some constants are not known to be irrational

xref JuliaLang/julia#55897

* rationality -> irrationality

Co-authored-by: David Widmann <[email protected]>

---------

Co-authored-by: David Widmann <[email protected]>
@nsajko
Copy link
Contributor Author

nsajko commented Oct 23, 2024

bump

@nsajko
Copy link
Contributor Author

nsajko commented Nov 10, 2024

ping

Some of the `Irrational` constants are not known to be irrational, so
it'd be good to note that in the doc string of each such constant to
avoid being misleading.
@nsajko nsajko force-pushed the doc_irrational_euler_catalan branch from dc6f851 to e5efa0b Compare November 23, 2024 08:18
@nsajko
Copy link
Contributor Author

nsajko commented Nov 23, 2024

bump

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs This change adds or pertains to documentation maths Mathematical functions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant