Bugzilla – Bug 2299
code review: UAN WOSS integration
Last modified: 2020-01-13 15:57:52 EST
Federico has been interested in integrating this with ns-3; additional reviewers are sought: https://codereview.appspot.com/14677043/
The ns-3 patches from this codereview are merged in changeset 13044:e0b228ec3c44. The remaining 'woss' code is intended for an ns-3 contributed module; it will be maintained at: https://github.com/MetalKnight/woss I'll leave this open until the ns-3 module is broadly available. Future woss module comments can be addressed at the above github issue tracker.
correct repo URL https://github.com/MetalKnight/woss-ns3
UAN modifications have not been properly merged in NS3 repo. This function https://codereview.appspot.com/14677043/patch/163001/173012 https://codereview.appspot.com/14677043/patch/163001/173011 is missing. I will provide a patch asap for Tom to merge
Created attachment 3122 [details] missing function has been implemented proposed patch attached. I haven't have the time to compile and test it yet. I will give you the final verdict as soon as possible.
(In reply to Federico Guerra from comment #4) > Created attachment 3122 [details] > missing function has been implemented > > proposed patch attached. > I haven't have the time to compile and test it yet. > I will give you the final verdict as soon as possible. OK, will wait for your final patch or confirmation.
Tom, patch is working you can merge it whenever you like. thanks Federico
Additional patch merged in commit 13689:f7c7da2c972f The remaining 'woss' code is intended for an ns-3 contributed module; it will be maintained at: https://github.com/MetalKnight/woss I'll leave this open in 'patch pending' state until the ns-3 module is broadly available. Future woss module comments can be addressed at the above github issue tracker.