Skip to content

relax the full profile fp16 tan accuracy requirement to 2.5 ULP#1387

Merged
bashbaug merged 2 commits intoKhronosGroup:mainfrom
bashbaug:relax-fp16-tan-ulp
Jul 1, 2025
Merged

relax the full profile fp16 tan accuracy requirement to 2.5 ULP#1387
bashbaug merged 2 commits intoKhronosGroup:mainfrom
bashbaug:relax-fp16-tan-ulp

Conversation

@bashbaug
Copy link
Copy Markdown
Contributor

@bashbaug bashbaug commented Jun 5, 2025

fixes #1303

The embedded profile accuracy requirement is unchanged and remains 3 ULP.

The embedded profile accuracy requirement is unchanged and remains
3 ULP.
@bashbaug bashbaug changed the title relax the full profile fp16 tan accuracy requirement to 2.25 ULP relax the full profile fp16 tan accuracy requirement to 2.5 ULP Jun 24, 2025
@bashbaug
Copy link
Copy Markdown
Contributor Author

I further relaxed the fp16 "tan" accuracy requirement to 2.5 ULP, as we discussed.

If we decide that we prefer 2.25 ULP instead, we can simply drop the last commit (and re-rename the PR).

@bashbaug bashbaug added the needs-cts-coverage The CTS needs to be extended label Jul 1, 2025
@bashbaug
Copy link
Copy Markdown
Contributor Author

bashbaug commented Jul 1, 2025

Merging as discussed in the July 1st teleconference.

@bashbaug bashbaug merged commit b61ab44 into KhronosGroup:main Jul 1, 2025
2 checks passed
@bashbaug bashbaug deleted the relax-fp16-tan-ulp branch July 1, 2025 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-cts-coverage The CTS needs to be extended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Relax the required accuracy of tan(half)?

2 participants