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.

Sign up for GitHub

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

Jump to bottom

JaxSimulation properties to check adjoint run_time #1608

Merged
merged 1 commit into from
Apr 12, 2024
Merged

JaxSimulation properties to check adjoint run_time #1608

merged 1 commit into from
Apr 12, 2024

Conversation

tylerflex
Copy link
Collaborator

adds JaxSimulation.tmesh_adjoint and JaxSimulation.num_time_steps_adjoint for helping estimate.

Note: users can also currently see ._run_time_adjoint but it's a private method so not documented prominently.

@tylerflex tylerflex added the 2.7 will go into version 2.7.* label Apr 12, 2024
@tylerflex tylerflex linked an issue Apr 12, 2024 that may be closed by this pull request
Estimate adjoint simulation cost from forward simulation. #1607
Closed
@tylerflex tylerflex self-assigned this Apr 12, 2024
@tylerflex tylerflex mentioned this pull request Apr 12, 2024
Closed
@momchil-flex momchil-flex merged commit a4aea79 into pre/2.7 Apr 12, 2024
13 checks passed
@momchil-flex momchil-flex deleted the tyler/adjointadjoint_run_time_check branch April 12, 2024 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@momchil-flex momchil-flex Awaiting requested review from momchil-flex

Assignees

@tylerflex tylerflex

Labels
2.7 will go into version 2.7.*
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Estimate adjoint simulation cost from forward simulation.
2 participants
@tylerflex @momchil-flex