Skip to content

Comments

Merge paper version to main#298

Merged
crangelsmith merged 385 commits intomainfrom
develop
Mar 4, 2024
Merged

Merge paper version to main#298
crangelsmith merged 385 commits intomainfrom
develop

Conversation

@crangelsmith
Copy link
Collaborator

We plan to do a big refactoring in the code, so we are merging the version used for the paper into main for reproducibility.

jolaem and others added 30 commits October 27, 2023 18:22
Capacity was hardcoded from April 2023 so had no effects
in any experiments post April 2023
settling to `python` instead of `python3`
adding `python -m pip install --upgrade pip` to instructions
marjanfamili and others added 26 commits February 2, 2024 16:45
…tes vis_all and give better control to the user, removed all path from personal pc from the notebook
…_post_process

Revert "Further post process"
…proc_notebook

State loading postproc notebook
…ed as png or pdf + adapt the vis.py + fix minor bugs
…ormat when functions are called in the postprocess notebook. + organising state files by date for restart flag
A patch to this branch which will allow overwriting the setting.vis_f…
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Collaborator

@marjanfamili marjanfamili left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All changes have been reviewed before

@crangelsmith crangelsmith merged commit e609837 into main Mar 4, 2024
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.

4 participants