Skip to content

WIP: Batch API and Absolute pen position#85

Closed
techninja wants to merge 18 commits intomasterfrom
absolute-batch
Closed

WIP: Batch API and Absolute pen position#85
techninja wants to merge 18 commits intomasterfrom
absolute-batch

Conversation

@techninja
Copy link
Owner

Held back by the massive overhead in sending 100k+ ReSTful commands, I intend to wiping the slate clean on that for RoboPaint by introducing a brand new API endpoint: /v1/batch. This allows a simple JSON format to hold all data for each command as it would be sent individually, into one command, reading from a local/remote file, or even data in the POST body.

To do this requires removing one of the last remaining oddities of the original client system: the distance counter. This free's up the client to manage this during spooling and send & forget the rest of the commands, allowing for batch sends possibly 10x - 50x faster with the request overhead removed. To aid in the clientside requirement of taking this over, I'm going to add another oft wanted feature: absolute measurement pen positioning. Allowing for MM or IN absolute measurement X/Y positions.

This PR will remain WIP until I can make sure these changes mesh well and make sense with the evil-mad/robopaint#287 it's meant to be helping complete.

This may also require revamping the position update timing because of unknowns with the serial buffer sends.

@techninja
Copy link
Owner Author

RP release never happened, I'm going to go ahead and merge this into a new v3 branch and attempt to pull together all speculative work with a new rewrite to allow RP like functionality from CNCServer directly.

@techninja techninja closed this Jul 7, 2019
@techninja techninja deleted the absolute-batch branch July 7, 2019 22:50
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.

1 participant