-
Notifications
You must be signed in to change notification settings - Fork 5
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have some minor commeths thoughts. But otherwise Im happy to merge.
Aside from the funding acknowledgement comment above, I think I resolved all the requested changes. @MansMeg |
Three minor additional questions/suggestions. |
I just took one that I found on google. try to find a better if you can and if there are none skip it. |
I mean, I downloaded their official logo pack and they're all transparent. If I get creative, I'll take it to photoshop or whatever. I guess most people use light mode anyway, for whatever reason. |
Then I think we are done! Im happy now! |
Adding dynamically updated summary stats and plots.
Scripts run as part of the release workflow.
A few points relating to quality control from the original issue are not included (not yet implemented), but can be added later.
closes #291