bluemurder / esp8266-ping

Arduino library handling ping messages for the esp8266 platform
MIT License
33 stars 11 forks source link

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

Open 2BECKHAM2 opened 6 years ago

2BECKHAM2 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

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 commented 6 years ago

Hi,

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

2BECKHAM2 commented 6 years ago

Thanks for the excellent library Blumurder.

Do you have any update on the issue raised.

Thanks in advance.

bluemurder commented 6 years ago

No updates yet.