RepRapFirmware 3.01beta1 released

dc42

@insertnamehere said in RepRapFirmware 3.01beta1 released:

I have a Duet Wifi 2 and I've been trying to access Initial Deviation in the Object Model.
If I execute the following line from the console:
echo "Initial Deviation", move.initialDeviation.deviation ^ "mm"
I get an error:
echo "Initial Deviation",move.initialDeviation.deviation ^ "mm"
Error: while executing command: unknown value initialDeviation.deviation

You've found a bug (the property table wasn't ordered alphabetically after I renamed a property, so the binary search failed). I've fixed it in the latest build at https://www.dropbox.com/sh/3azy1njy3ayjsbp/AACquxr2m00eV568RZg5QG5wa?dl=0. I also added a compile-time check to prevent this happening in future, and an extra property state.upTime.

carlosspr

I have updated and I can only use one tool.
On the tool configuration I have the following error:

M563 S"Yellow" P1 D1 H1 F0
Error: Tool creation: bad drive number

however on previous release this command was working fine.

kraegar

3.01 now appears to require endstops be defined on a duet2, where the 3.0 upgrade guide specifically says they do not. (That guide also says fans do not, but that change was covered in the 3.01 release notes)

dc42

@kraegar said in RepRapFirmware 3.01beta1 released:

3.01 now appears to require endstops be defined on a duet2, where the 3.0 upgrade guide specifically says they do not. (That guide also says fans do not, but that change was covered in the 3.01 release notes)

That's correct. I'll add it to the 3.01 release notes and check the 3.0 upgrade guide.

pipersw

@chas2706 said in RepRapFirmware 3.01beta1 released:

@dc42

I just upgraded to this on my Duet Ethernet and I thought my board suddenly broke.
The extruder fan stays permanently on, I cant move any axis because I get this:

Have you read the readme ?

If upgrading a Duet WiFi/Ethernet/Maestro from the 3.0 release, note that default fans are no longer created.
Unless your config.g file already used M950 to create the fans explicitly, add commands M950 F0 C"fan0", M950 F1 C"fan1" and M950 F2 C"fan2" to config.g before your M106 commands.
Likewise, default endstop switches are not set up, so you will need to set up X and Y endstops (and Z odf needed) explicity, using one M574 line for each, and specifying the port name. Example: M574 X1 S1 P"xstop".

dc42

@pipersw said in RepRapFirmware 3.01beta1 released:

Likewise, default endstop switches are not set up, so you will need to set up X and Y endstops (and Z odf needed) explicity, using one M574 line for each, and specifying the port name. Example: M574 X1 S1 P"xstop".

To be fair to @chas2706, I have only just added that bit!

kraegar

Got a macro in testing for iterating on bed leveling, and it's pretty amazing!

My initial round had two && in one line, which caused the duet to hard crash and reboot. I switched it to nested if statements, and that worked well. I want to tweak it a bit, but definitely working!

( was iterating until initialDeviation was < 0.01 AND deviation was within 0.002 of initialDeviation)

kraegar

Cleaned up the output to make it easier to follow. (also changed my leadscrew locations to make it take longer to converge for these tests, so I could troubleshoot easier)

dc42

@kraegar said in RepRapFirmware 3.01beta1 released:

My initial round had two && in one line, which caused the duet to hard crash and reboot.

Can you provide that version, so that I can fix it?

kraegar

This post is deleted!

kraegar

Here is the version that works well:

M561                         ; clear any existing bed transform
; If the printer hasn't been homed, home it
if !move.axes[0].homed || !move.axes[1].homed || !move.axes[2].homed
  G28
  
G1 Z7.5

;Run initial 4 point leveling routine
G30 P0 X15 Y45 Z-99999
G30 P1 X15 Y227.5 Z-99999
;G30 P2 X235 Y125 Z-99999 S3
G30 P2 X235 Y227.5 Z-99999
G30 P3 X235 Y45 Z-99999 S3



while true
  if iterations = 5
    abort "Too many auto calibration attempts"
  if move.initialDeviation.deviation < 0.01 
    if move.calibrationDeviation.deviation < move.initialDeviation.deviation + 0.005
      if move.calibrationDeviation.deviation > move.initialDeviation.deviation - 0.005
        break
  echo "Repeating calibration because initial deviation (" ^ move.initialDeviation.deviation ^ "mm) must be < 0.01"
  echo "and (" ^ move.calibrationDeviation.deviation ^ "mm) must be within 0.005 of initialDeviation"

  G30 P0 X15 Y45 Z-99999
  G30 P1 X15 Y227.5 Z-99999
  ;G30 P2 X235 Y125 Z-99999 S3
  G30 P2 X235 Y227.5 Z-99999
  G30 P3 X235 Y45 Z-99999 S3

echo "Auto calibration successful, deviation", move.calibrationDeviation.deviation ^ "mm"
echo "Auto calibration successful, initialDeviation", move.initialDeviation.deviation ^ "mm"  
    
G28 Z

G1 X0 Y0 F24000               ; move the head to the corner (optional)

And removing the nested if's to replace with && crash it

  if move.initialDeviation.deviation < 0.01 && move.calibrationDeviation.deviation < move.initialDeviation.deviation + 0.005 && move.calibrationDeviation.deviation > move.initialDeviation.deviation - 0.005
        break

Edit: I realize I could just have some major syntax issues, still figuring this out. But that's a hard crash if you do that

carlosspr

@carlosspr Solved. I was missing M584 drive mapping on config.g

dc42

@kraegar said in RepRapFirmware 3.01beta1 released:

And removing the nested if's to replace with && crash it

Thanks, I've reproduced the crash by adding that line to my own script. I will look into it. Even if you do make a syntax error, you should get a helpful error message.

PS - now I know that the new crash handler I added for uncaught exceptions works!

Last software reset at 2020-01-15 16:35, reason: Terminate called, spinning module GCodes, available RAM 9464 bytes (slot 2)

kraegar

Trying to test if move.initialDeviation.deviation is 0.000

if move.initialDeviation.deviation != 0
  echo "one " ^ move.initialDeviation.deviation

I've tried using 0.000 as well, with the same result. == 0.000 or == 0 doesn't match, even though it is.

(I'm trying to determine if leveling has been run at all yet, my only clear method for now is to see if initial and calirbated deviation and mean are all 0.000)

dc42

Just done some tests. There is a problem with equality between floats.

The other problem you had is because the GCode line was too long. I changed it to throw an exception when that happens, but the exception isn't caught.

This is all very new code, so I was expecting some bugs.

kraegar

Yep, I'm just playing, happy to just keep reporting. Variables will clean the code up a lot, but that's a big ask

kraegar

Once variables are implemented, if they're "system wide" and not constrained to the macro they're called in, I would set a "is_level = false" in config.g, and set to true once leveling is run. Negating the need for this method of checking.

But I can't resist playing

dc42

Thanks for your reports. These two issues are now fixed in the latest builds at https://www.dropbox.com/sh/3azy1njy3ayjsbp/AACquxr2m00eV568RZg5QG5wa?dl=0.

kraegar

Can I just say, HUGE thank you!
I've already made a script to tune PA, Temp, and Extrusion % (just call the macros via layer change) as well as the auto-leveling and a "pre-print" systems readiness check.

This is amazing!

kraegar

Float comparisons are now working, and the other errors gracefully:

Error: in file macro, line 20 column 1: GCode command too long

Thanks!