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

Add optimization constraints as a submodel in ClimateModel or ClimateModelParameters? #29

Open
hdrake opened this issue Aug 1, 2020 · 1 comment
Open

Add optimization constraints as a submodel in ClimateModel or ClimateModelParameters? #29

hdrake opened this issue Aug 1, 2020 · 1 comment
Assignees
@hdrake
Labels
enhancement New feature or request priority
Milestone
v1.0.0 release

Comments

@hdrake
Copy link
Collaborator

hdrake commented Aug 1, 2020

Would it be better to include the optimization parameters / constraints as a submodel in ClimateModel or ClimateModelParameters or should they remain as arguments in the optimize_controls! function?
@hdrake hdrake mentioned this issue Aug 25, 2020
Closed
@hdrake hdrake added the enhancement New feature or request label Aug 25, 2020
@hdrake hdrake added this to the v1.0 release milestone Sep 16, 2020
@hdrake hdrake modified the milestones: v2.0.0 release, v1.0.0 release Jan 17, 2021
@hdrake hdrake self-assigned this Jan 17, 2021
@hdrake
Copy link
Collaborator Author

hdrake commented Jan 17, 2021

Doesn't really make sense to make any progress on this until #44 is closed.

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

@hdrake hdrake

Labels
enhancement New feature or request priority
Projects
None yet
Milestone
v1.0.0 release
Development

No branches or pull requests
1 participant
@hdrake