-
Notifications
You must be signed in to change notification settings - Fork 42
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
Merge updates from 2023 CommNet lecture. #29
Conversation
platform/.gitignore
Outdated
@@ -157,3 +157,4 @@ cython_debug/ | |||
# Ignore generated config directories and files | |||
groups/ | |||
log.txt | |||
save_friday/* |
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.
This is not really needed. I could remove it if you want.
Update: I just realized some commits are still missing, I'll add them in a moment. |
Remaining stuff is committed. |
@KTrel I hope your deadline stress is (for the moment) over. Do you have a moment to maybe to a quick dry-run of the new project? "It works on my machine", but I'd be much more confident to merge if you could confirm that it works! |
Ping @KTrel |
For this iteration, we did a lot of small updates, mainly:
docker pause
anddocker unpause
.generate_configs.py
script was reworked to change topology sizes more easily.@KTrel , I tested the basic setup to make sure everything can be set up without issues, and it seems to work for me.
But I'd be thankful if you could try setting the default configuration up yourself and debugging a bit to make sure it is still easily usable.