techninja / cncserver

A RESTful API server for driving serial based CNC devices
133 stars 39 forks source link

Running on Windows smells like trouble: Possible serial drain issue #88

Closed techninja closed 4 years ago

techninja commented 7 years ago

It always seems at conventions and conferences while running exhaustive demos is when I find bugs. I have just today confirmed through a very specific test that something about windows 10 is causing a very bad problem: Missed commands, or simply, missed offsets.

Release RoboPaint running CNCServer stock:

I suspect this is because of bad code on the side of node-serialport for windows to verify serial drain. Their current release is v5, and we're on v2. Switching to v5+ may not be such a simple thing as the API has likely changed considerably... though I have to admit we're not using it much, or at least very much for code.

techninja commented 7 years ago

Apparently this was a known problem and fixed in some recent version released in January. Tested on windows this is now working!...or rather, not breaking in that way anymore. Though the "sudden stop" is still a problem. Will be looking into what this is and why soon. Initial experiments with Express 4 have yielded terrible breaking changes with no chance to test for improvements. 😖

techninja commented 7 years ago

"Sudden stop" confirmed as dire and reproducible issue in development branch, recorded as EmergingTechnologyAdvisors/node-serialport#1188, though I doubt this will get traction without more details.

techninja commented 4 years ago

Managed in v3...