adding dof manager to params and re-writing params as an equinox module #111

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

Open

cmhamel wants to merge 1 commit into main from dof-manager-as-param

Open

adding dof manager to params and re-writing params as an equinox module #111

adding dof manager to params and re-writing params as an equinox modu…

Codecov / codecov/patch succeeded Mar 3, 2025 in 1s

80.76% of diff hit (target 77.26%)

View this Pull Request on Codecov

80.76% of diff hit (target 77.26%)

Annotations

Check warning on line 52 in optimism/Objective.py

codecov / codecov/patch

optimism/Objective.py#L52

Added line #L52 was not covered by tests

Check warning on line 54 in optimism/Objective.py

codecov / codecov/patch

optimism/Objective.py#L54

Added line #L54 was not covered by tests

Check warning on line 60 in optimism/Objective.py

codecov / codecov/patch

optimism/Objective.py#L57-L60

Added lines #L57 - L60 were not covered by tests

Check warning on line 62 in optimism/Objective.py

codecov / codecov/patch

optimism/Objective.py#L62

Added line #L62 was not covered by tests

Check warning on line 82 in optimism/Objective.py

codecov / codecov/patch

optimism/Objective.py#L81-L82

Added lines #L81 - L82 were not covered by tests

Check warning on line 84 in optimism/Objective.py

codecov / codecov/patch

optimism/Objective.py#L84

Added line #L84 was not covered by tests

View more details on Codecov