Skip to content

Refactoring the app into a library#9

Merged
ml-evs merged 42 commits into
MJCliffe:Livefrom
ml-evs:ml-evs/app_refactor_pass_1
Sep 21, 2023
Merged

Refactoring the app into a library#9
ml-evs merged 42 commits into
MJCliffe:Livefrom
ml-evs:ml-evs/app_refactor_pass_1

Conversation

@ml-evs
Copy link
Copy Markdown
Collaborator

@ml-evs ml-evs commented Aug 23, 2023

To-do:

  • refactor into several submodules
  • add docs for different modules
  • make plots and results usable from outside of app
  • tweak template layout s.t. plots and tables can still be screencapped
  • echem indicatrix
  • echem best chebyshev residual loop
  • compressibility plot bug
  • Add Jupyter notebook example

@ml-evs ml-evs force-pushed the ml-evs/app_refactor_pass_1 branch 5 times, most recently from dc73cc8 to 28ccf70 Compare August 30, 2023 10:01
@ml-evs ml-evs marked this pull request as ready for review September 18, 2023 17:46
Comment thread README.md Outdated
@ml-evs ml-evs force-pushed the ml-evs/app_refactor_pass_1 branch from d5b0cae to 51893b1 Compare September 20, 2023 15:32
@ml-evs ml-evs changed the title Refactoring the app Refactoring the app into a library Sep 20, 2023
@ml-evs ml-evs force-pushed the ml-evs/app_refactor_pass_1 branch 3 times, most recently from cc6d9b6 to ec32ce5 Compare September 20, 2023 15:50
@ml-evs ml-evs force-pushed the ml-evs/app_refactor_pass_1 branch from c146744 to b76bc36 Compare September 20, 2023 15:53
@ml-evs ml-evs requested a review from MJCliffe September 21, 2023 08:17
@ml-evs ml-evs merged commit cd7abaf into MJCliffe:Live Sep 21, 2023
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.

1 participant