Skip to content

Revert "Refactor thrust::complex as a struct derived from cuda::std::complex (#454)"#1286

Merged
jrhemstad merged 2 commits intoNVIDIA:branch/2.3.xfrom
miscco:revert_thrust_complex_changes
Jan 17, 2024
Merged

Revert "Refactor thrust::complex as a struct derived from cuda::std::complex (#454)"#1286
jrhemstad merged 2 commits intoNVIDIA:branch/2.3.xfrom
miscco:revert_thrust_complex_changes

Conversation

@miscco
Copy link
Contributor

@miscco miscco commented Jan 16, 2024

This is breaking pytorch so revert for 2.3.0

@miscco miscco requested review from a team as code owners January 16, 2024 18:21
@miscco miscco added the thrust For all items related to Thrust. label Jan 16, 2024
@jrhemstad jrhemstad enabled auto-merge (squash) January 16, 2024 22:32
@jrhemstad jrhemstad merged commit 0df3163 into NVIDIA:branch/2.3.x Jan 17, 2024
@leofang
Copy link
Member

leofang commented Jan 17, 2024

This is breaking pytorch so revert for 2.3.0

Q: Any chance there's a public reference on this issue?

@miscco miscco deleted the revert_thrust_complex_changes branch January 17, 2024 07:27
@miscco
Copy link
Contributor Author

miscco commented Jan 17, 2024

Q: Any chance there's a public reference on this issue?

Unfortunately not, this came up in internal QA, so there is only an internal bug (nvbug4445377)

@jrhemstad
Copy link
Collaborator

Q: Any chance there's a public reference on this issue?

@leofang The TL;DR is that Pytorch's unit tests exposed some numerical differences in thrust::complexs original implementation vs cuda::std::complex (e.g., #1272). It will require further analysis to determine the extent of the numerical differences and how to best address them.

miscco added a commit to miscco/cccl that referenced this pull request Feb 22, 2024
…d::complex` (NVIDIA#454)" (NVIDIA#1286)

We did not have any time to work on fixing the numerical issues with `cuda::std::complex` so punt on it for 2.4.x

Co-authored-by: Jake Hemstad <jhemstad@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

thrust For all items related to Thrust.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants