RepRapFirmware 3.01beta1 released

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!

dc42

Thanks for confirming. FWIW, maximum GCode line length excluding leading white space, line number and checksum (if present) and comments is currently 160 characters.

kraegar

We're going to need line numbers in the file editor in DWC now

4lathe

Can I use that new duet2combinedfirmware.bin on my maestro? The 3.01 was duetmaestrofirmware.bin.

DIY-O-Sphere

@4lathe
It's still DuetMaestroFirmware.bin

DIY-O-Sphere

@dc42
Hi David, maybe I have found a bug....
For testing I was back on 2.05 on a Maestro (DWC 2.06) and I was not able to get back directly to 3.01b.
Flashing 3.0 stable worked.

dc42

@DIY-O-Sphere said in RepRapFirmware 3.01beta1 released:

For testing I was back on 2.05 on a Maestro (DWC 2.06) and I was not able to get back directly to 3.01b.

Read the upgrade notes!

gtj0

@dc42 said in RepRapFirmware 3.01beta1 released:

Thanks for confirming. FWIW, maximum GCode line length excluding leading white space, line number and checksum (if present) and comments is currently 160 characters.

Are you sending them via text message???

DIY-O-Sphere

@dc42 said in RepRapFirmware 3.01beta1 released:

Read the upgrade notes!

Sorry....The one who can read is in advantage

dc42

@gtj0 said in RepRapFirmware 3.01beta1 released:

@dc42 said in RepRapFirmware 3.01beta1 released:

Thanks for confirming. FWIW, maximum GCode line length excluding leading white space, line number and checksum (if present) and comments is currently 160 characters.

Are you sending them via text message???

Hadn't you noticed your smartphone bills going up every time you print?

gtj0

@dc42 said in RepRapFirmware 3.01beta1 released:

@gtj0 said in RepRapFirmware 3.01beta1 released:

@dc42 said in RepRapFirmware 3.01beta1 released:

Thanks for confirming. FWIW, maximum GCode line length excluding leading white space, line number and checksum (if present) and comments is currently 160 characters.

Are you sending them via text message???

Hadn't you noticed your smartphone bills going up every time you print?

Ah that makes sense now. I though it was the cat texting his buddy across the street.

garyd9

@kraegar said in RepRapFirmware 3.01beta1 released:

Here is the version that works well:

I like this, but it seems unnatural to have to repeat the same G30 commands twice in the same script. This is more significant with a delta printer (that often has many more probe points.)

I'm wondering what alternatives might exist. I see there's no do/while (which would be my first thought), but I'm wondering if it would work to put the list of G30 gcodes in a different file (bed-probes.g)

@dc42, do the variables from the object model available survive across different scripts? For example, if I had file called "bed-ProbePoints.g" that contained something like:

G30 P0 Xn Yn 
...
G30 P18 X0 Y0 S6

And then my bed.g contained:

M561
G28
M98 P"bed-ProbePoints.g" 
while true
    if iterations > 5
        abort "something needs tightening..."
    if abs(move.initialDeviation.deviation - move.calibrationDeviation.deviation) <= 0.01
       break
    M98 P"bed-ProbePoints.g"

Would this work? Would the deviation values be updated across the scripts?

Thank you
Gary

kraegar

@garyd9 This was my very rough first iteration (and still is early, room for improvement, etc etc). Only posted in this thread due to the bugs I was noticing.

Anyway, see my post here where I did split the G30 segment out into a macro:
https://forum.duet3d.com/topic/13890/auto-tramming-a-railcore-using-conditional-gcode

garyd9

@kraegar That is great (and answers my question.)

thank you

insertnamehere

@dc42 said in RepRapFirmware 3.01beta1 released:

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.

Firstly, thanks for the quick response. I've tried the fixed firmware and it works great.

Another thing, I use an ATX type 24V supply controlled by the PS_ON pin.
After all axis are homed, if I turn of the 24V supply, the axis remain homed "Blue" on the DWC. If I turn on the ATX supply again I can move all axis as if they are homed. Shouldn't turning of the 24V supply automatically mark all axis as homed : false?

m409
{"key":"","flags":"","result":{"boards":[{"firmwareFileName":"Duet2CombinedFirmware.bin","firmwareVersion":"3.01-beta1+1","iapFileNameSD":"Duet2CombinedIAP.bin","mcuTemp":{"current":39.6,"max":39.9,"min":38.0},"name":"Duet 2 WiFi","shortName":"2WiFi","vIn":{"current":0.7,"max":24.4,"min":0.4}}],"heat":{"coldExtrudeTemperature":160.0,"coldRetractTemperature":90.0,"heaters":[{"current":22.0,"sensor":0,"state":"Off"},{"current":23.2,"sensor":1,"state":"Active"}],"sensors":[{"lastReading":22.0,"name":"","type":"Thermistor"},{"lastReading":23.2,"name":"","type":"Thermistor"}]},"job":{"file":{"filament":[],"firstLayerHeight":0.0,"generatedBy":"","height":0.0,"lastModified":"28115-10-29T03:41:51","layerHeight":0.0,"numLayers":0,"printTime":0,"simulatedTime":0,"size":0},"lastFileName":"","layer":0,"timesLeft":{"filament":0.0,"file":0.0,"layer":0.0}},"move":{"axes":[{"homed":true,"letter":"X","max":300.0,"min":0.0,"userPosition":70.000,"visible":true},{"homed":true,"letter":"Y","max":300.0,"min":0.0,"userPosition":265.000,"visible":true},{"homed":true,"letter":"Z","max":350.0,"min":0.0,"userPosition":4.213,"visible":true}],"calibrationDeviation":{"deviation":0.000,"mean":0.000},"currentMove":{"requestedSpeed":0.0,"topSpeed":0.0},"daa":{"enabled":false,"minimumAcceleration":10.0,"period":0.0},"idle":{"factor":0.7,"timeout":30000},"initialDeviation":{"deviation":0.004,"mean":0.014},"meshDeviation":{"deviation":0.032,"mean":0.099},"printingAcceleration":10000.0,"speedFactor":100.0,"travelAcceleration":10000.0},"network":{"interfaces":[{"actualIP":"192.168.1.142","firmwareVersion":null,"gateway":"0.0.0.0","subnet":"0.255.255.255","type":"wifi"}]},"state":{"currentTool":0,"machineMode":"FFF","status":"Off","upTime":193}}}