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

Rework of the template to better integrate the scenarios APIs #12

Merged
merged 19 commits into from
Jan 6, 2023

Conversation

AdrienLeGuillou
Copy link
Member

uses the netsim_scenarios functions defined here.

Reworks the template so that all the concepts are covered locally and all the necessary code can be tested before moving to the HPC.

This approach aims to make the code "run" locally and then assess it's correctness on the HPC. The intuition is that correctness needs many replications AND codes that runs. But debugging on the HPC is harder.

I am working on the Wiki at the same time (on the WIP branch, not visible online)

- move setup_packages to 00-setup_packages. It's the first step
- uses netsim_scenarios functions
- move 00-project_settings to utils-0_project_settings. it's not a step
- separate `netsim` tests (low level) from `netsim_scenarios` higher
  level
@AdrienLeGuillou AdrienLeGuillou marked this pull request as ready for review January 6, 2023 08:13
@AdrienLeGuillou AdrienLeGuillou merged commit 87190e5 into main Jan 6, 2023
@AdrienLeGuillou AdrienLeGuillou deleted the scenarios branch January 6, 2023 08:31
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