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

Add ability to configure model options from the GUI #90

Closed
wants to merge 3 commits into from

Conversation

dangunter
Copy link
Collaborator

Fixes #85

@dangunter dangunter marked this pull request as draft July 16, 2023 23:15
@ksbeattie ksbeattie added the Priority:Normal Normal Priority Issue or PR label Jul 20, 2023
@dangunter dangunter marked this pull request as ready for review July 22, 2023 00:16
@dangunter
Copy link
Collaborator Author

@MichaelPesce the UI parts of this are still not complete, but I did get "options" to show in a half-baked React component called FlowsheetOptions. Adding you as a reviewer and moving out of draft since I am out until Aug. 3rd and if you want to look at this before then and comment or help, please feel free. The main piece I think I will be most useful for is in the watertap repo, where the options are used to change flowsheet build/solve behavior & documentation is created to explain all this.

@MichaelPesce
Copy link
Collaborator

closing this because it was handled by #92

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Priority:Normal Normal Priority Issue or PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Expose model configuration options in GUI
3 participants