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

All spindles share the same settings - why? #35

Open
axg20202 opened this issue Nov 20, 2024 · 3 comments
Open

All spindles share the same settings - why? #35

axg20202 opened this issue Nov 20, 2024 · 3 comments

Comments

@axg20202
Copy link

I have 2 spindles, one PWM and the other is via ModBus VFD. For some reason, there is a single set of parameters (speed, frequency etc) which affects all spindles. This doesn't seem right. For example, my PWM spindle needs a speed settings of 0-1000, while my Modbus spindle needs 1200 to 24000 rpm. There really needs to be dedicated settings for each spindle.

@terjeio
Copy link
Contributor

terjeio commented Nov 20, 2024

the other is via ModBus VFD.

Which?

There really needs to be dedicated settings for each spindle.

Nope, RPM range should be read from the VFD, like this, and the VFD drivers that currently do not should be updated.
If settings were to be added for VFD drivers then the setting values has to be transferred to the VFD, and currently I do not have any to test against.

@axg20202
Copy link
Author

Ah, OK. So VFD/modbus spindles don't use the rpm setting in grblhal, only the PWM spindle? I could have sworn that changing those parameters affects the speed of my vfd spindle as well. My vfd is HY type connected over modbus. My other spindle is a laser that I believe requires 0 to 1000 rpm setting.

@terjeio
Copy link
Contributor

terjeio commented Nov 21, 2024

Which version are you running (build date in the $i output)?
If a relatively new version $spindlesh can be used to output details about the spindles, min/max RPM is output as the two last values for active spindles.
For VFDs the reported RPM range is taken from $30 and $31 initially, and changed to the actual values after reading min/max from the driver.

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

No branches or pull requests

2 participants