Skip to content

Tracking issue for extra floating-point logarithm constants #50540

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
clarfonthey opened this issue May 8, 2018 · 15 comments · Fixed by #69249
Closed

Tracking issue for extra floating-point logarithm constants #50540

clarfonthey opened this issue May 8, 2018 · 15 comments · Fixed by #69249
Labels
B-unstable Blocker: Implemented in the nightly compiler and unstable. C-tracking-issue Category: An issue tracking the progress of sth. like the implementation of an RFC disposition-merge This issue / PR is in PFCP or FCP with a disposition to merge it. finished-final-comment-period The final comment period is finished for this PR / Issue. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.

Comments

@clarfonthey
Copy link
Contributor

Specifically, LOG10_2 and LOG2_10. PR: #50539

@XAMPPRocky XAMPPRocky added T-libs-api Relevant to the library API team, which will review and decide on the PR/issue. C-tracking-issue Category: An issue tracking the progress of sth. like the implementation of an RFC labels Oct 2, 2018
@clarfonthey
Copy link
Contributor Author

Has anyone used these constants at all and want to pitch in for stabilisation of these? These have landed for a while and I'm curious how many people are using them.

@kennytm
Copy link
Member

kennytm commented Dec 1, 2018

+1 for stabilization. (Tried to see if log2(10) exists and found that it is unstable 🙄.)

@clarfonthey
Copy link
Contributor Author

(Mind providing the specific use case? Curious to see if people are using this for anything other than scientific computing.)

@kennytm
Copy link
Member

kennytm commented Dec 1, 2018

@clarcharr to estimate the "−log2(probability)" of collision of two values chosen randomly from a set, which will be used to derive the probability of collision of N values (≈ 1 − e-pN2/2, p ≪ 1, N ≫ 1).

log2 is chosen instead of natural log because the set size is often 2n (e.g. u32), but sometimes it is drawn from a set of n decimal digits which would then involve n·log2(10).

@iliekturtles
Copy link
Contributor

One hartley is = log2(10) shannons (bits). Currently the constant is duplicated in uom.

@clarfonthey
Copy link
Contributor Author

I can't @ anyone from the libs team but do you think we'd be able to start an FCP for this?

@varkor
Copy link
Member

varkor commented Sep 28, 2019

cc @rust-lang/libs for a FCP.

@alexcrichton
Copy link
Member

@rfcbot fcp merge

I believe the stabilization includes core::{f32, f64}::{LOG2_10, LOG10_2}.

@rfcbot
Copy link
Collaborator

rfcbot commented Sep 30, 2019

Team member @alexcrichton has proposed to merge this. The next step is review by the rest of the tagged team members:

No concerns currently listed.

Once a majority of reviewers approve (and at most 2 approvals are outstanding), this will enter its final comment period. If you spot a major issue that hasn't been raised at any point in this process, please speak up!

See this document for info about what commands tagged team members can give me.

@rfcbot rfcbot added proposed-final-comment-period Proposed to merge/close by relevant subteam, see T-<team> label. Will enter FCP once signed off. disposition-merge This issue / PR is in PFCP or FCP with a disposition to merge it. labels Sep 30, 2019
@rfcbot rfcbot added final-comment-period In the final comment period and will be merged soon unless new substantive objections are raised. and removed proposed-final-comment-period Proposed to merge/close by relevant subteam, see T-<team> label. Will enter FCP once signed off. labels Nov 21, 2019
@rfcbot
Copy link
Collaborator

rfcbot commented Nov 21, 2019

🔔 This is now entering its final comment period, as per the review above. 🔔

@jonas-schievink jonas-schievink added the B-unstable Blocker: Implemented in the nightly compiler and unstable. label Nov 26, 2019
@rfcbot
Copy link
Collaborator

rfcbot commented Dec 1, 2019

The final comment period, with a disposition to merge, as per the review above, is now complete.

As the automated representative of the governance process, I would like to thank the author for their work and everyone else who contributed.

The RFC will be merged soon.

@rfcbot rfcbot added finished-final-comment-period The final comment period is finished for this PR / Issue. and removed final-comment-period In the final comment period and will be merged soon unless new substantive objections are raised. labels Dec 1, 2019
@LeSeulArtichaut
Copy link
Contributor

@alexcrichton Does this need a stabilization report/PR?

@clarfonthey
Copy link
Contributor Author

It needs a PR. Not sure if we should include TAU in that though; @m-ou-se, would you be willing to open a separate tracking issue for that, considering how that wasn't included in the FCP for this one?

@m-ou-se
Copy link
Member

m-ou-se commented Feb 16, 2020

@clarfon There is already #66770.

@clarfonthey
Copy link
Contributor Author

Oh, my bad. I for some reason noticed that this issue got linked in the PR and thought that there wasn't a separate tracking issue.

JohnTitor added a commit to JohnTitor/rust that referenced this issue Feb 18, 2020
…ts, r=sfackler

Stabilize {f32, f64}::{LOG2_10, LOG10_2}

Following the decision to stabilize `LOG2_10` and `LOG10_2` in rust-lang#50540 (comment).

Closes rust-lang#50540.
r? @sfackler
@bors bors closed this as completed in 20c4835 Feb 18, 2020
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
B-unstable Blocker: Implemented in the nightly compiler and unstable. C-tracking-issue Category: An issue tracking the progress of sth. like the implementation of an RFC disposition-merge This issue / PR is in PFCP or FCP with a disposition to merge it. finished-final-comment-period The final comment period is finished for this PR / Issue. T-libs-api Relevant to the library API team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

10 participants