Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[crater] Add impl From<f16> for f32 #142723

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

Draft
beetrees wants to merge 1 commit into rust-lang:master
base: master
Choose a base branch
Loading
from beetrees:crater-impl-from-f16-for-f32

Conversation

Copy link
Contributor

@beetrees beetrees commented Jun 19, 2025
edited by rustbot
Loading

Crater run to see what the effects of adding impl From<f16> for f32 without changing the fallback (a.k.a. things that would be caught by the FCW in #139087). This needs a @craterbot check.

@rustbot rustbot added T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Jun 19, 2025
Copy link
Contributor

@bors2 try

Copy link

rust-bors bot commented Jun 19, 2025

⌛ Trying commit e4ad46e with merge 8d46be7...

To cancel the try build, run the command @bors2 try cancel.

rust-bors bot added a commit that referenced this pull request Jun 19, 2025
[crater] Add `impl From<f16> for f32`
Crater run to see what the effects of adding `impl From<f16> for f32` without changing the fallback (a.k.a. things that would be caught by the FCW in #139087). This needs a ``@craterbot` check`.
Copy link

rust-bors bot commented Jun 19, 2025

☀️ Try build successful (CI)
Build commit: 8d46be7 (8d46be77cd48fdbcced888d027457e2c9cf2aa16, parent: 2fcf1776b9ccef89993dfe40e9f5c4908e2d2d48)

Copy link
Contributor

@craterbot check

Copy link
Collaborator

👌 Experiment pr-142723 created and queued.
🤖 Automatically detected try build 8d46be7
🔍 You can check out the queue and this experiment's details.

i️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added the S-waiting-on-crater Status: Waiting on a crater run to be completed. label Jun 19, 2025
Copy link
Collaborator

🚧 Experiment pr-142723 is now running

i️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

Copy link
Collaborator

🎉 Experiment pr-142723 is completed!
📊 3143 regressed and 10 fixed (650587 total)
📰 Open the full report.

⚠️ If you notice any spurious failure please add them to the denylist!
i️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-crater Status: Waiting on a crater run to be completed. labels Jun 22, 2025
Copy link
Contributor

Various versions of epaint, glyph_brush, and taffy seem to account for the vast majority of the breakage. Whatever else we do, it'd probably be worth putting in PRs for those.

Copy link
Member

@traviscross I don't think we should submit any PRs to those crates until we have agreement that we're prepared to break passing e.g. 2.0 to a method taking Into<f32>. Because I think the likely response to such a PR is "wait, what?".

Copy link
Contributor

traviscross commented Jul 8, 2025
edited
Loading

We've speculatively submitted PRs before in these kind of cases. Such PRs said, essentially, that the code being changed was relying on something that had become an open question. But I'm happy with whatever the people pushing this forward want to do (or not do) in this regard.

Copy link
Contributor Author

beetrees commented Jul 8, 2025
edited
Loading

taffy is actually already fixed from when the impl was breifly added on nightly a year ago (DioxusLabs/taffy#643): all the regressions caused by that crate are from other crates using outdated 0.3.* and 0.4.* versions of it (the fix is in 0.3.19 and 0.4.3 and later).

(I plan to do a full write-up of the Crater run soon, I just haven't had time yet.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-libs Relevant to the library team, which will review and decide on the PR/issue.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /