Duet 3: "Attempt to move motors when VIN is not in range" @ 30V

Veti

@evan38109 said in Duet 3: "Attempt to move motors when VIN is not in range" @ 30V:

Is this just a bug, or can the Duet 3 not run at 30V? Or am I doing something else wrong?

i would say its a bug. from the documentation
https://duet3d.dozuki.com/Wiki/Duet_3_Mainboard_6HC_Hardware_Overview
VIN in the range 12V-32V must be provided to the Duet.

pipersw

I have the same bug with Duet2Wifi@24V, when I do the first movement of a stepper (homing). Does not seen after.

evan38109

I tried with a lower-voltage power supply this evening and the steppers move just fine. I'm now reasonably confident that this is a bug.

(...or maybe I'm missing a voltage setting somewhere? There's always a chance this is my fault. )

Is there a preferred method to file a bug report, or is this kind of forum post sufficient?

T3P3Tony

@evan38109 said in Duet 3: "Attempt to move motors when VIN is not in range" @ 30V:

Is there a preferred method to file a bug report, or is this kind of forum post sufficient?

In this case comments here will be seen by @dc42 so no need to file a bug report on GitHub.

dc42

It's a bug. That code should be active for the Duet WiFi/Ethernet build but not for the Duet 3 build of RRF. I will fix it in the next beta.

evan38109

Thank you, @T3P3Tony and @dc42!

evan38109

I've updated to the latest firmware version, and this issue persists.

If it helps, the web app is reporting these versions:

Board: Duet 3 version v0.6 (MB6HC)
DSF Version: 1.1.0.5
Firmware: RepRapFirmware for Duet 3 v0.6
3.0beta11 (2019-10-21b1)

I don't want to be a jerk, but...any ETA? It's been a month now.

dc42

Are you certain that it isn't fixed? I put the fix in the source code well before beta 11.

Veti

@dc42 the current v3-dev still has

https://github.com/dc42/RepRapFirmware/blob/v3-dev/src/Platform.cpp

constexpr uint16_t driverOverVoltageAdcReading = PowerVoltageToAdcReading(29.0); // voltages above this cause driver shutdown

wilriker

@Veti said in Duet 3: "Attempt to move motors when VIN is not in range" @ 30V:

@dc42 the current v3-dev still has

https://github.com/dc42/RepRapFirmware/blob/v3-dev/src/Platform.cpp

constexpr uint16_t driverOverVoltageAdcReading = PowerVoltageToAdcReading(29.0); // voltages above this cause driver shutdown

This constexpr needs to stay for Duet 2 (and possibly other boards as well) - it's the place where it is used that seems to fail for Duet 3 to do the right thing.

Veti

but should there not be a constexpr for the duet3 over voltage where it shuts down?

dc42

I am sorry, there were two places where maximum VIN was enforced, and I only changed one of them. New firmware build coming soon.

evan38109

Awesome -- thanks, @dc42!

dc42

You can try the new firmware at https://www.dropbox.com/sh/3azy1njy3ayjsbp/AACquxr2m00eV568RZg5QG5wa?dl=0 if you like. This isn't an official beta, but it's running OK on my test setup.

evan38109

Just tried the firmware from dropbox, and it works like a charm. Thanks for taking the time to put this out on a weekend!

Also -- and I'm sure you get this a lot -- but I can't get over how easy the firmware update was. Kudos.