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

Improved user messages and updated to new API for get_dependent_variable_shape #158

Merged

DominicDirkx merged 2 commits into develop from feature/updated_and_improved_dep_var_dictionary

Aug 5, 2024

Contributor

alopezrivera commented Aug 2, 2024

IMPORTANT API change: create_dependent_variable_dictionary now takes:

Dynamics simulator
Bodies

New API:

def create_dependent_variable_dictionary(
        dynamics_simulator: numerical_simulation.SingleArcSimulator,
        bodies: numerical_simulation.environment.SystemOfBodies
    ) -> DependentVariableDictionary:
    ...

Summary

Improved logs for users
Solved bug due to get_dependent_variable_shape API change

alopezrivera added 2 commits

August 2, 2024 17:46


          Improved user messages and updated to new API for get_dependent_varia…

b9aa321

…ble_shape


          Use bodies attribute of SingleArcSimulator to create dependent variab…

…le dictionary

DominicDirkx merged commit a25723c into develop

4 of 5 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet