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

ref: update detray to v0.88.1 #841

Merged
merged 1 commit into from
Feb 17, 2025

Conversation

niermann999
Copy link
Contributor

@niermann999 niermann999 commented Feb 4, 2025

Update to the new detray version. I also made sure that the traccc custom scalar type is handed through to detray, so that it will be built in double precision when required. Apart from that, I removed a detray thrust related cmake variable, which doesn't exist anymore.
The Kalman Fit actor state was switched to use the non-owning vecmem::device_vector type always, since it just needs a view on the track states returned from the CKF and the detray detector no longer provides a host/device vector type publicly.

@niermann999 niermann999 added bug Something isn't working feature New feature or request labels Feb 4, 2025
@niermann999 niermann999 marked this pull request as ready for review February 5, 2025 10:51
@niermann999 niermann999 force-pushed the ref-update-detray branch 11 times, most recently from ef634a9 to 02b3b41 Compare February 7, 2025 14:38
@niermann999 niermann999 force-pushed the ref-update-detray branch 3 times, most recently from a3eeb14 to 2bfaa59 Compare February 13, 2025 16:00
@niermann999 niermann999 changed the title ref: update detray to v0.88.0 ref: update detray to v0.88.1 Feb 13, 2025
@niermann999 niermann999 force-pushed the ref-update-detray branch 5 times, most recently from 068a9a8 to d730e0b Compare February 14, 2025 13:29
@niermann999 niermann999 force-pushed the ref-update-detray branch 3 times, most recently from 8bc5f92 to c70646f Compare February 17, 2025 08:54
@niermann999 niermann999 merged commit 8c061f0 into acts-project:main Feb 17, 2025
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants