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.

Sign up for GitHub

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

Jump to bottom

Update get_command_from_result for ns-3.35+ #66

Closed
wants to merge 11 commits into from
Closed

Update get_command_from_result for ns-3.35+ #66

wants to merge 11 commits into from

Conversation

pagmatt
Copy link
Member

@pagmatt pagmatt commented Apr 18, 2023 •
edited

This PR:

  • Updates get_command_from_result to support both CMake and Waf versions of ns-3
  • Adds a test for get_command_from_result
  • Adds test cases for checking the runner for Waf-based versions of ns-3

@pagmatt
Copy link
Member Author

pagmatt commented Apr 21, 2023 •
edited

TODO: Extend get_command_from_result to test also debug commands
TODO: Test commands

pagmatt and others added 7 commits May 5, 2023 10:03
@pagmatt @Thecave3
Introduce support for custom callbacks, various fixes and improvements (
1aaa051 
#67)

* Draft base callback

* Add cb param in manager

* Draft base callback

* Add cb param in manager

* Support paraller runner

* Re-introduce support for parent cb init

* Merge changes in utils

* Small fixes

* fix ctrld by parent flag for callbacks

* Improve parameters support of the callbacks (#64)

---------

Co-authored-by: pagmatt <mattpagg@gmail.com>
Co-authored-by: Andrea Lacava <lacava.a@norhteastern.edu>

---------

Co-authored-by: Andrea Lacava <thecave003@gmail.com>
Co-authored-by: Andrea Lacava <lacava.a@norhteastern.edu>
@pagmatt
Use long-term averages for runner simulation ETAs (#70)
da63bc5
@pagmatt
Iniitial work on fixing get_command_from_result
9a6cc85
@pagmatt
improve ns3 command
b4ea9cf
@pagmatt
Add initial get_command_from_result test
03b5d41
@pagmatt
Test runner also with waf-based versions of ns-3
6ac0113
@pagmatt
Merge
8bbfc24
@pagmatt pagmatt closed this Feb 9, 2024
@pagmatt pagmatt deleted the fix-get-cmd-from-result branch February 9, 2024 15:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@pagmatt