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

PER recording #197

Closed
SamirRachedi opened this issue May 20, 2021 · 7 comments
Closed

PER recording #197

SamirRachedi opened this issue May 20, 2021 · 7 comments

Comments

@SamirRachedi
Copy link

SamirRachedi commented May 20, 2021

hello @riebl,

How to record only Packet Error rate for a scenario (disable all other recordings)?
I tried adding this on my omnetpp.ini to disable all stats : .node[].wlan[*].**.statistic-recording = false

Then, I added these lines to record PER:

.node[].**.packetErrorRate.statistic-recording = true
.node[].**.packetErrorRate.result-recording-modes = all

But it's not working?

Thanks in advance.

@MadeInPierre
Copy link

MadeInPierre commented May 20, 2021

Hi @SamirRachedi,

I remember reading in OMNeT++'s documentation that the first line match in omnetpp.ini that corresponds to each NED parameter is selected, the other matches are ignored.

In this case, your two last lines are probably ignored. Maybe there is another way of disabling other statistics?

Best Regards

@SamirRachedi
Copy link
Author

Hi @MadeInPierre,

I thought of that too at first. So I tried adding the following lines to test:
.node[].middleware.CA.transmission.result-recording-modes = all
.node[].middleware.CA.reception.result-recording-modes = all

And these lines worked just fine (recording stats for CA transmissions and receptions). But not the PER stats. This is where I was lost, I don't know exactly what went wrong.

Thanks anyway.

@MadeInPierre
Copy link

If i'm not mistaken, the first of these two lines does not match the second since wlan[*] != middleware:
.node[].wlan[*].**.statistic-recording = false
.node[].middleware.CA.transmission.result-recording-modes = all

So the parameter .node[].middleware.CA.transmission.result-recording-modes only finds the second line and not the first.
I hope I'm correct, if not please ignore my message 😄

@SamirRachedi
Copy link
Author

No problem at all @MadeInPierre , it's always nice to exchange with others.
In the beginning, without all these lines, I noticed that Artery is recording lots of stats (I got a 60GB file !!!!). So I used: .node[].wlan[*].**.statistic-recording = false to disable all stats and it worked.
Than I added :
.node[].**.packetErrorRate.statistic-recording = true
.node[].**.packetErrorRate.result-recording-modes = all
to recorded PER. When I saw it's not woking I tried to add:
.node[].middleware.CA.transmission.result-recording-modes = all
.node[].middleware.CA.reception.result-recording-modes = all
Just to test (to be sure that moment is not ignoring other lines as you mentioned in your first reply) and it worked fine.

I guess you get my point. Thanks anyway.

@riebl
Copy link
Owner

riebl commented Jun 17, 2021

I guess you have managed to record PERs, do you @SamirRachedi? Do you think we should add some information about result recording to our documentation, i.e. beyond the information provided by OMNeT++ itself?

@SamirRachedi
Copy link
Author

Hello @riebl,

I managed to understand what I did wrong and I'm getting the PER now. Just Don't know how to get curves of PER vs Distance like in the figure below:

image

Appreciate any help. thanks a lot.

@riebl
Copy link
Owner

riebl commented Mar 12, 2023

I have added a task regarding "X-distance-plots" to #215 and thus close this issue now.

@riebl riebl closed this as completed Mar 12, 2023
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

No branches or pull requests

3 participants