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

Return T if mode=TracedSetPath #810

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Conversation

glwagner
Copy link
Collaborator

No description provided.

glwagner and others added 3 commits February 25, 2025 16:34
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@wsmoses
Copy link
Member

wsmoses commented Feb 26, 2025

@avik-pal I don't want to distract you from mesh stuff, but can you potentially lend a hand here?

@avik-pal
Copy link
Collaborator

what is the issue here?

@wsmoses
Copy link
Member

wsmoses commented Feb 26, 2025

CI is sad.

Essentially the reason for this PR is because a data structure passed non-traced TwicePrecision{Float64} and StepRange into a traced if. TracedIf asks for tracednumbers, which are ilelgal for steprange [expected the inner elem to be an int].

This PR doesn't change the inner value types to change on a tracedtrack and related [since it should just update in place]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants