search

luc-github / ESP3D-WEBUI

A Web UI for ESP8266 or ESP32 based boards connected to 3D printers / CNC
GNU General Public License v3.0
747 stars 305 forks source link

[FEATURE REQUEST]CNC Support of XYZABCUV and XYZUVW axis #268

Closed luc-github closed 1 year ago

luc-github commented 1 year ago

Per https://github.com/luc-github/ESP3D-WEBUI/issues/252#issuecomment-1251919463 starting discussion the letter axis will change in futur and grblHAL will support up to 8 axis - when currently webui only support 6 axis with letters XYZABC

per https://www.fictiv.com/articles/3-axis-to-12-axis-cnc-milling-machine-capabilities-compared and https://en.wikipedia.org/wiki/G-code

CNC are composed of translation and rotation axis

which are : X Y Z are translation axis other axis ABCUVW are supposed to be rotation axis

if axis are up to 6 XYZABC could be remaped as XYZUVW if more than 6 axis should be XYZABCUV

Each translation axis have possible probe and end stop pin Each rotation axis may not have probe support, neither end stop pin support, so they may not appear in ? Pn: report , only in position part, but support should be ready for it in WebUi

Note: when it is planned XYZUVW is not yet supported in grbHAL and there is a conflict in |PN:<signals> where signal W - Motor warning, new conflict with W axis

luc-github commented 1 year ago

test version of the support for UVW axis mapped on ABC axis if used for up to 6 axis, and support of 8 axis XYZABCUV code is here : https://github.com/luc-github/ESP3D-WEBUI/tree/newAxisDefinition it will be merged once constraints are clarified

refactoring the axis management allowed to decrease final package of grbHAL webui from 80,778 bytes to 80,760 bytes while improving the flexibility on axis number and now allow also support of any axis probing when before was limited to X Y Z

luc-github commented 1 year ago

PR merged - new management implemented

github-actions[bot] commented 1 year ago

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.