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

Added chart scroll and 4x performance #12

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

dimecho
Copy link
Contributor

@dimecho dimecho commented Dec 16, 2021

Please test before merge.

@jsphuebner
Copy link
Owner

Hi Dima,
sorry I only got to look at this now. Will test the client side changes later. On the server side you moved enabling of fast uart to the setup routine. That only works if the STM32 boots faster than the ESP. Not sure. Also after a software update fastuart will not be enabled again.

@dimecho
Copy link
Contributor Author

dimecho commented Jan 8, 2022

OK fair point about ESP vs STM32 boot time competition. Maybe we can have a Sleep-Delay in ESP on startup? The idea was to keep "CMD" loop as simple and clean as possible, even if statements will slow it down. I had it down to about 10ms for 10 results (with complete path HTTP > ESP > STM32 > ESP > HTTP)

@jsphuebner
Copy link
Owner

How about handing over control to the client side? You have already started by adding the timeout parameter to the /baud resource. So now you could switch the UART between 115k and 920k with a parameter.
The web interface can then switch to fast uart mode in onload(). Also if it doesn't get reply it can try switching back to 115k

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

Successfully merging this pull request may close these issues.

2 participants