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

Updated optimize documentation for static_parameter_intervention definition #544

Merged
merged 2 commits into from
Mar 21, 2024

Conversation

anirban-chaudhuri
Copy link
Contributor

@anirban-chaudhuri anirban-chaudhuri commented Mar 21, 2024

No description provided.

@anirban-chaudhuri anirban-chaudhuri added awaiting review PR submitter awaiting code review from reviewer integration Tasks for integration with TA4 labels Mar 21, 2024
@anirban-chaudhuri anirban-chaudhuri self-assigned this Mar 21, 2024
@SamWitty SamWitty self-requested a review March 21, 2024 16:00
@SamWitty SamWitty merged commit fb0eec9 into main Mar 21, 2024
5 checks passed
@SamWitty SamWitty deleted the ac-opt_updatedoc branch March 21, 2024 17:18
anirban-chaudhuri added a commit that referenced this pull request Mar 21, 2024
* Add check if parameters have distributions (#543)

* add check for whether parameters have distribution

* add test without distributions

* lint

* add check to see if there are any params with distributions (#546)

* Updated optimize documentation for static_parameter_intervention definition (#544)

* Updated optimize documentation for static_parameter_intervention definition

* lint

---------

Co-authored-by: Sam Witty <[email protected]>

---------

Co-authored-by: Sam Witty <[email protected]>
SamWitty added a commit that referenced this pull request Mar 21, 2024
* update optimize documentation for definition of static_parameter_intervention

* Added warnings for optimize interface

* Linting

* Update optimize_interface.ipynb

* Add check if parameters have distributions (#543)

* add check for whether parameters have distribution

* add test without distributions

* lint

* add check to see if there are any params with distributions (#546)

* Updating from main (#548)

* Add check if parameters have distributions (#543)

* add check for whether parameters have distribution

* add test without distributions

* lint

* add check to see if there are any params with distributions (#546)

* Updated optimize documentation for static_parameter_intervention definition (#544)

* Updated optimize documentation for static_parameter_intervention definition

* lint

---------

Co-authored-by: Sam Witty <[email protected]>

---------

Co-authored-by: Sam Witty <[email protected]>

* update optimize documentation for definition of static_parameter_intervention

* Added warnings for optimize interface

* Linting

* Update optimize_interface.ipynb

* Revert "Update optimize_interface.ipynb"

This reverts commit 4427ff9.

* Revert "Linting"

This reverts commit 0afcb97.

* Revert "Added warnings for optimize interface"

This reverts commit 1644a75.

* Revert "update optimize documentation for definition of static_parameter_intervention"

This reverts commit 11cdaa9.

* Revert "Updating from main (#548)"

This reverts commit 3df403d.

* Added warnings for 'optimize' and rerun optimize

* Update interfaces.py

---------

Co-authored-by: Sam Witty <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting review PR submitter awaiting code review from reviewer integration Tasks for integration with TA4
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants