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

feat: add marimo app #101

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

feat: add marimo app #101

wants to merge 2 commits into from

Conversation

steven-murray
Copy link
Contributor

@steven-murray steven-murray commented Jan 22, 2024

Adds a little marimo app. With the app you can choose a built-in profile and look at a few sensitivity plots from the profile. We could work on this a bit in a few ways:

  1. Add ability to upload your own antpos file.
  2. Neaten up the plots a bit so you can see everything at once
  3. Host it somewhere

To run the app:

pip install marimo
marimo edit app/sense.py

Copy link

codecov bot commented Jan 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (d0178cd) 98.61% compared to head (a44e4fb) 98.61%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #101   +/-   ##
=======================================
  Coverage   98.61%   98.61%           
=======================================
  Files          15       15           
  Lines        1081     1081           
=======================================
  Hits         1066     1066           
  Misses         15       15           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mkolopanis
Copy link
Contributor

playing around a little, this is really cool. agreed though with your proposal that this could probably be done after JOSS

@steven-murray
Copy link
Contributor Author

As a counterpoint -- if this is easy enough to get going in the next day or so, it could be a neat little hook for the paper ("come and try out the app with your favorite 'scope at this link!")

@mkolopanis
Copy link
Contributor

that is a great point! unfortunately I don't think I personally have time to try to work on it within the week.

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.

2 participants