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

added gui for static icf example #40

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

added gui for static icf example #40

wants to merge 2 commits into from

Conversation

shimwell
Copy link
Collaborator

Hi Remi

This PR adds a GUI interface to one of the examples. If you like it then it could be extended to include all the examples.

It is a very minimal PR and there is a ton of work that could be done to improve things. But let me know what you think.

Instructions for use are in the README.md

Screenshot from 2022-11-25 22-13-41

@RemDelaporteMathurin
Copy link
Owner

It looks grand! We may move to a Residence Time Method (RTM) since having both concentrations and volumes makes it very redundant...

I would wait a bit until we make a decision on this point

Awesome example of streamlit by the way!!

@shimwell
Copy link
Collaborator Author

@RemDelaporteMathurin if you could give me a nudge when the RTM move is done I can update this PR or made a new one.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants