-
Notifications
You must be signed in to change notification settings - Fork 72
perf: add perf nim libp2p #658
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
base: master
Are you sure you want to change the base?
Conversation
|
@vladopajic Here's the perf run: https://github.com/libp2p/test-plans/actions/runs/15475924179/job/43571160935 |
|
@p-shahi thanks man. if it passes i am thinking of changing version to v1.10 (since it uses nim-libp2p v1.10). |
|
@vladopajic I already changed it to match the versionsInput.json (without this change, the runner was trying to access a nonexistent v0.1 folder) Right now the run is failing for this reason (linked to the run in the above comment) |
|
@vladopajic I lifted this change: 7eb409d from your new branch. |
476a7a8 to
4b14d12
Compare
NP, done. Here's the run I triggered: https://github.com/libp2p/test-plans/actions/runs/15495806737/job/43632128611 |
|
thank you! i see, it failed again. why? why does it show i think server started, i'll add log for this. but client failed to start? |
|
@vladopajic it's because the I can move it and run it again |
|
@p-shahi hey i pushed one more commit. arguments have format edit: please wait; args parsing will not work with spaces this way, i'll need to change it. i'll let you know |
|
@p-shahi now it's fixed. tested args parsing locally. code should build and run now. could you try with latest code from same branch? :) |
29f41fe to
4b328dd
Compare
|
triggered run after adding this change 3b0a465 and 2bf1f1a https://github.com/libp2p/test-plans/actions/runs/15498770750/job/43641664863 @vladopajic still some failures Is the client reporting |
|
thanks man! iam very pleased that it is finally running. no, currently it's not reporting that. i wanted to make it run first (setting up). i'll add intermediate reports soon and report back once iam done. |
|
@vladopajic triggered run here: https://github.com/libp2p/test-plans/actions/runs/15539543975/job/43746772694 @vladopajic similar failure as before. Maybe a serialization issue? |
not really sure, when i run locally i get this output: nim-libp2p: go-libp2p: i have made #661 to help developers out in this situation, could you please check that one? |
…d-perf-nim-libp2p
|
@vladopajic I merged that PR into master and this branch, then triggered a run. Here's what the logs show: could be that the data isn't being written properly since the ResultValue is empty |
|
@p-shahi first successful run, good job for us!
yeah i have imagined that there is some empty line, that's being written like maybe last report should not be with line brake. but, thankfully with #661 we don't have to think about that anymore. questions:
|
|
also i have added one more commit 485ad5a. if with this commit workflow succeeds may we merge current pr? |
|
@vladopajic I added some more logging, and found that the intermediate values are reported but not the final. So the error is there. Does the isFinal block ever run? couldn't find where stats.isFinal is set Here's the run: https://github.com/libp2p/test-plans/actions/runs/15590060951/job/43906557856 Added another log line eb265dd to see whats going on https://github.com/libp2p/test-plans/actions/runs/15591661636/job/43911987318. Seems like isFinal is never hit: similarly, doesn't seem to report Final value for download benchmark But when uploading and downloading 1 byte i.e. connection establishment test, it seems to always report the final value |
|
@p-shahi hey, thanks! i think i know what's going on. i'am talking days off, but i should be able to push this update. |
|
gotcha, triggered here: https://github.com/libp2p/test-plans/actions/runs/15639940398/job/44064636827 Same issue on this run as well. |
hey, thank you! just did some investigation and it appears that nim perf is hitting timeout (20sec) - it would be cool if this message is actually shown in logs. |
|
question, intermediary report should include data since last report? |

No description provided.