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

evan38109 · 27 Sept 2019, 23:27

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 · 27 Sept 2019, 23:42

@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 · 28 Sept 2019, 10:53

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 · 28 Sept 2019, 13:55

Thank you, @T3P3Tony and @dc42!

evan38109 · 26 Oct 2019, 17:52

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 · 26 Oct 2019, 22:43

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

Veti · 27 Oct 2019, 05:18

@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 · 27 Oct 2019, 06:18

@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 · 27 Oct 2019, 06:36

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

dc42 · 27 Oct 2019, 08:18

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 · 27 Oct 2019, 13:57

Awesome -- thanks, @dc42!

dc42 · 27 Oct 2019, 16:23

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 · 27 Oct 2019, 21:51

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.