New RepRapFirmware 3.0 early beta

dc42

@wilriker said in New RepRapFirmware 3.0 early beta:

I tracked it down to Heater.cpp:CheckGood() where it says
return GetMode() == HeaterMode::fault && CheckProtection();
and I think the first comparison should be != or the heater is only good it if actually is in fault state.

Thanks, I've corrected the source.

dc42

@smoki3 said in New RepRapFirmware 3.0 early beta:

That change I now, as you can see in my script they already include G53 commands in the tfree.g
But the problem is:
Look at the last movement of my tfree.g: It move to G53 Y165 (no tool offset)
Then the script is finished. Then it starts with the G1 X76 in the tpre (still no tool offset)
What I then expect is that the printer is just moving form G53 X0 (from the tfree.g) to the G1 X76.
But it is also moving in the Y axis.
For me it sound like more that it is a bug of this change:

Please clarify. Are you saying that the G1 X76 command in tpre causes a move in the Y direction too? If so then this is certainly a bug. The coordinates stored in the tool change restore point should only be relevant at the end of the entire tool change sequence.

smoki3

@dc42 Yes the G1 X76 also do a move in the Y direction. Also the Z axis is moving.

It restores the coordinates where the nozzle was before. The reference point of the printer tries to reach the point where the nozzle was before.

The only way actually can avoid this behavior is to put also a G53 G1 Y165 into the tpre.g script. Then only a Z move is done (but also this movement shouldn't be done).

wilriker

@dc42 said in New RepRapFirmware 3.0 early beta:

Thanks, I've corrected the source.

Have you also seen my edit if the post regarding misleading message output?

smoki3

@dc42 I switched now back to 2.02.

Here is everything fine. Also 2.04 and 2.03 has this issue. I tried also the old v3 build from the dropbox. Here also everthing is fine.

I assume that it has something to do with the commits from May 25th.
I will later do a build an try it out

smoki3

I was trying to build a older build but it seems that it's necessary to build wifisocketserver. But I don't geht it to build in my mac

dc42

This is now on my list to investigate, and fix in 2.04 if I can confirm the problem.

@smoki3 said in New RepRapFirmware 3.0 early beta:

I was trying to build a older build but it seems that it's necessary to build wifisocketserver. But I don't get it to build in my mac

You don't need to build wifisocketserver, you just need the include directory in it to be present in the Eclipse workspace.

gtj0

@dc42 using the tip of the current RRF v3-dev...

Sent: M308 S0 P"bedtemp" Y"thermistor" T100000 B4138 C0 R4700
Sent: M308 S0
Sensor 0 type Thermistor using pin bedtemp, last error: sensor not ready, T:100000.0 B:4138.0 C:0.00e+0 R:4700.0 L:0 H:0

Sent: M308 S2 Y"thermistor" P"e1temp" A"HotEnd" T100000 B4725 C7.06e-8
Sent: M308 S2
Sensor 2 (HotEnd) type Thermistor using pin e1temp, last error: sensor not ready, T:100000.0 B:4725.0 C:7.06e-8 R:4700.0 L:0 H:0

Is there something wrong with this M308 syntax? All temps are 2000.0

smoki3

@dc42 said in New RepRapFirmware 3.0 early beta:

This is now on my list to investigate, and fix in 2.04 if I can confirm the problem.

@smoki3 said in New RepRapFirmware 3.0 early beta:

I was trying to build a older build but it seems that it's necessary to build wifisocketserver. But I don't get it to build in my mac

You don't need to build wifisocketserver, you just need the include directory in it to be present in the Eclipse workspace.

I always get this when I trying to compile from a older source. If I try the actual branch everything is fine

../src/Networking/ESP8266WiFi/WiFiInterface.cpp:1857:10: error: 'APIN_SPI_MISO' was not declared in this scope

make: *** [src/Networking/ESP8266WiFi/WiFiInterface.o] Error 1

wilriker

@smoki3 You have to make sure that all of the repositories are at about the same point in time to compile. Most of the times having all repos except RepRapFirmware at the latest commit works but every once in a while there are bigger changes across repos (as in the case you show where some constants had been renamed) that require you to revert the according repo (I think it is CoreNG in this case) to a commit prior to the breaking change.

smoki3

@dc42
I have done some further testing:

If I put not only "G1 X76" at the beginning of the tpre rather a "G1 X76 Y165" (Y165 same Y position as tfree) then just the X axis and Z axis is moving and Y axis stays as it should. Anyway the Z axis shouldnt move at this point.

So if you don't but every single axis into the first command of tfree it will restore the old position of the tool head

dc42

@wilriker said in New RepRapFirmware 3.0 early beta:

@smoki3 You have to make sure that all of the repositories are at about the same point in time to compile. Most of the times having all repos except RepRapFirmware at the latest commit works but every once in a while there are bigger changes across repos (as in the case you show where some constants had been renamed) that require you to revert the according repo (I think it is CoreNG in this case) to a commit prior to the breaking change.

That's right, the APIN_SPI_MISO message means that the RepRapFirmware project is not in sync with CoreNG.

dc42

@gtj0 said in New RepRapFirmware 3.0 early beta:

@dc42 using the tip of the current RRF v3-dev...

Sent: M308 S0 P"bedtemp" Y"thermistor" T100000 B4138 C0 R4700
Sent: M308 S0
Sensor 0 type Thermistor using pin bedtemp, last error: sensor not ready, T:100000.0 B:4138.0 C:0.00e+0 R:4700.0 L:0 H:0

Sent: M308 S2 Y"thermistor" P"e1temp" A"HotEnd" T100000 B4725 C7.06e-8
Sent: M308 S2
Sensor 2 (HotEnd) type Thermistor using pin e1temp, last error: sensor not ready, T:100000.0 B:4725.0 C:7.06e-8 R:4700.0 L:0 H:0

Is there something wrong with this M308 syntax? All temps are 2000.0

There are some reported bugs in the current v3-dev. In particular, you need a quarter second delay between creating a sensor using M308 and creating a heater that uses it. I also have a report that configuring thermistors using M308 doesn't work (at least on Duet 3), which I am about to investigate.

gtj0

@dc42 said in New RepRapFirmware 3.0 early beta:

There are some reported bugs in the current v3-dev. In particular, you need a quarter second delay between creating a sensor using M308 and creating a heater that uses it. I also have a report that configuring thermistors using M308 doesn't work (at least on Duet 3), which I am about to investigate.

No problem. Thanks!
If there's anything I can do to help, let me know. Specific bugs needing to be squashed, testing, etc.

dc42

I've just committed new source code to fix a number of RRF3 bugs, including issues with temperature readings.

gtj0

@dc42 said in New RepRapFirmware 3.0 early beta:

I've just committed new source code to fix a number of RRF3 bugs, including issues with temperature readings.

Yep, temps now read correctly. Thanks!

dc42

@smoki3 said in New RepRapFirmware 3.0 early beta:

@dc42
It was definitely not like this on 2.02 and the older 3.0.

And its a really wired behaviour. Because I use:

; tfree0.g
; called when tool 0 is freed
;
; generated by RepRapFirmware Configuration Tool on Wed Sep 19 2018 21:12:53 GMT+0200 (Mitteleuropäische Sommerzeit)
G91
G1 Z10 F7200
G90
G53 G1 X0 F25000 ;XPOS
G53 G1 Y190
G53 G1 Y195 F5000
M400
M98 P"/macros/Toolhead/1. Toolhead unlock"
G4 P320
G53 G1 Y207 F5000
M400
G53 G1 Y165 F25000
M400

; tpre1.g
; called before tool 0 is selected
;
; generated by RepRapFirmware Configuration Tool on Wed Sep 19 2018 21:12:53 GMT+0200 (Mitteleuropäische Sommerzeit)
G1 X76 F25000
M400
G1 Y192
M400
M98 P"/macros/Toolhead/1. Toolhead unlock"
G4 P400
G1 Y207 F5000
M116 P3
M400
M98 P"/macros/Toolhead/2. Toolhead lock"
G4 P260
G1 Y150 F15000

The I only expect that only a X movement is done while the start of the tpre1.g but it first moves in X and Y.

@smoki3, I am looking into this but I need your config.g file, also the toolhead unlock files that you call from the tool change files.

smoki3

@dc42 I will provide it until next week. I am on vacation

gtj0

@dc42 Homing seems to be broken at the moment.
If the axis is already homed and it's endstop is triggered, you can't run the homing sequence again without getting a Error: G0/G1: insufficient axes homed and the axis changing state to unhomed.

Example... Here's my CoreXY homex.g file...

G91               ; relative positioning
G1 H1 X-490 F3600 ; move quickly to X axis endstop and stop there (first pass)
G1 X5 F3600       ; go back a few mm
G1 H1 X-490 F360  ; move slowly to X axis endstop once more (second pass)
G90               ; absolute positioning

If the endstop isn't triggered the homing sequence runs. If it's already homed, I get the error.

Also, when it does work, it sometimes reports the tool at a small distance from 0. For instance, if I home X, sometimes the tool is reported at 0.000, sometimes, 0.105, sometimes 0.038, etc.

gtj0

Oh, here's another one. I'm not sure of this is related to the tool offset discussion but when RRF initializes, the Z axis height displays the offset from the heightmap so as you move the X or Y axes, the Z axis position display changes.