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

Observed issues with multiple IPs when the u32_t requests > 9 #5

Open
2BECKHAM2 opened this issue May 24, 2018 · 5 comments
Open

Observed issues with multiple IPs when the u32_t requests > 9 #5

2BECKHAM2 opened this issue May 24, 2018 · 5 comments

Comments

@2BECKHAM2
Copy link

Hi bluemurder,

While checking the ping with multiple IP I observed that when u32_t requests > 9, I face issues with Ping summary, seqno, Reply from IP. I have attached the file for your reference.

  1. LOG_PING_requests=9.txt - The results observed were OK as expected.

  2. LOG_PING_requests=10.txt - Observed issues with the printing of Ping summary. (Not available for all the IPs)

  3. LOG_PING_requests=50.txt - Observed issues with the printing of Ping summary and the icmp_sequence no.

  4. LOG_PING_requests=110.txt - Observed issues with the Reply from 192.168.31.1, printing of Ping summary, icmp_sequence no, and the number of pings initiated.

The above conditions were tested by changing the value of requests = XX in the Pinger.h.

LOG_PING_requests = 9.txt
LOG_PING_requests = 10.txt
LOG_PING_requests = 50.txt
LOG_PING_requests = 110.txt

Thanks in advance.

@bluemurder
Copy link
Owner

It is critic the delay between different ping calls. In your case it is possible that the onend callback is called when the next ping sequence is already started. Please send your main .ino file

@2BECKHAM2
Copy link
Author

Many thanks Bluemurder.

Find attached files.

  1. Ping.ino (The example file)

  2. PingModified.ino - where the IP, Information and number of ping count which are configurable.

I've tried delay(20000); in the above two files also, observed the same results.

Thanks again.

Ping.ino.txt
PingModified.ino.txt

@2BECKHAM2
Copy link
Author

Hi,

I've tried with Serial.begin(9600); also - Same results observed.

@2BECKHAM2
Copy link
Author

Thanks for the excellent library Blumurder.

Do you have any update on the issue raised.

Thanks in advance.

@bluemurder
Copy link
Owner

No updates yet.

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

2 participants