string doesn't finish typing before next command about wifiduck HOT 12 CLOSED

Actually I just found some problems in the communication module.
I'll refactore the entire thing...

from wifiduck.

Ya. I just updated. I don't think it happens as much but if I run a script from the scripts section (without hitting the edit button) it still happens.

from wifiduck.

To recreate this issue, it only seems to happen sometimes after power on.

The easyest way to test it is to set the script to auto start and unplug and plug in the devce till you see it happen. should not take more than 10 tries

from wifiduck.

I've also encountered this issue. For some reason I've only really seen it when I am using "gui r"
Maybe that has something to do with it?

from wifiduck.

So I've been trying to recreate this issue but it just works for me.... every time 😅
On both my DIY and DSTIKE boards.

Does the issue still persists with the current codebase?

The only cause I can think of is a bug or glitch in the communication between ESP and Atmega. Seems like somehow the linebreak '\n' is not transmitted, received or parsed properly.

from wifiduck.

I'll test master again soon. But I'm pretty sure it will still be there. I'll update you on this soon

from wifiduck.

Well I have just updated and I cant reproduce this issue anymore. @Jilickpock do you still have this issue?

from wifiduck.

A week ago or so, I updated the communication module on both ends. That might have fixed it.
The issue was created afterwards, but maybe you had still the old code running?
Thats the only explaination I could come up with. I'm really just guessing here.

from wifiduck.

This seems to be a timing issue on the I2C bus.
I made a branch for the fix: https://github.com/spacehuhn/WiFiDuck/tree/com-refactor
But I will have to make some more changes before this can be merged.

from wifiduck.

Can you test the branch?
You will need to update both Atmega and ESP.
My quick tests show that everything is working, but at this point I feel very unsure about just merging it right away.

from wifiduck.

Tested it and it seems fine now

from wifiduck.

Merged :)

from wifiduck.