[FR] Input Shaping with I2S_STEPPER_STREAM

[ahmetcemturan]

Did you test the latest bugfix-2.1.x code?

Yes, and the problem still exists.

Bug Description

If Input shapıing is enabled, the printer will home x and then y axis. Then stops responding (does not home z).
Homing works fine if I send "M593 F0" before homing

Bug Timeline

It is existent in the 2.1.2 and the latest bugfix

Expected behavior

I expect it to home

Actual behavior

It stops responding during the homing routine.

Steps to Reproduce

enable input shaping and set it to a value >=1
Home the printer

Version of Marlin Firmware

2.1.2 or 2.1.bugfix

Printer model

Prusa clone (Bedslinger)

Electronics

Tinybee v3

Add-ons

Z-probe as Z-endstop

Bed Leveling

ABL Bilinear mesh

Your Slicer

Prusa Slicer

Host Software

Pronterface

Don't forget to include

• A ZIP file containing your Configuration.h and Configuration_adv.h.

Additional information & file uploads

>>> M593 F0

echo:DEBUG:ECHO,INFO,ERRORS,COMMUNICATION
echo:M105
echo:M105
echo:M105
echo:G28
echo:busy: processing
echo:busy: processing
echo:G92 E0
echo:M114
echo:M105
echo:M105
echo:M105
echo:M105
>>> M593 F22
SENDING:M593 F22
echo:M593 F22
echo:M105
echo:M105
echo:G28
echo:busy: processing
>>>>>Becomes Unresponsive....

will only respond after a reset.

Only other modification is That I have changed the ADC voltage to 2.585 as that gives correct values

Marlin.zip

[ahmetcemturan]

Some additional observations:

• when trying to compile Marlin with SHAPING_FREQ_ =0, I get an error. (not a big problem as it makes basically no sense)
• when I compile Marlin with IS enabled and then set F to 0 I am able to home and print.
• when I start the print and then enable IS it printed after I put a value for x then stopped after I put in a value for Y.
• Marlin with IS disabled but FT enabled behaved similar:
• I chose these values: M493 A39 B36 K0.06 P1 S11
• No homing possible
• After Restart and M493 S0, I was able to home and start the print but print stopped after I sent M493 A39 B36 K0.06 P1 S11
• Currently both disabled..

[tombrazier]

IS is not supported with ESP32. My naive guess is the FT is not either. We should add a sanity check.

ESP32 works around some of the core stepper logic. We really need a refactor of stepper.cpp that puts all the step generation in one place and prevents more than one step being generated per pass through the loop. Then ESP32 could be integrated with IS properly. (Not to mention baby stepping and really getting it working with LA.)

when trying to compile Marlin with SHAPING_FREQ_ =0, I get an error. (not a big problem as it makes basically no sense)

You could want to compile with SHAPING_FREQ_[XY]=0 so that IS defaults to disabled. That is possible if you define SHAPING_MIN_FREQ. There should be a sanity check error which tells you this.

[thisiskeithb]

IS is not supported with ESP32. My naive guess is the FT is not either.

I'll mark this as a feature request for I2S_STEPPER_STREAM instead.

We should add a sanity check.

Added:

• Input Shaping / FT Motion with I2S_STEPPER_STREAM ? #25632

[cbagwell]

when trying to compile Marlin with SHAPING_FREQ_ =0, I get an error. (not a big problem as it makes basically no sense)

You could want to compile with SHAPING_FREQ_[XY]=0 so that IS defaults to disabled. That is possible if you define SHAPING_MIN_FREQ. There should be a sanity check error which tells you this.

I created #25358 to apply the related AVR sanity check to everything.

[vfbank]

All hardware currently using the ESP32 chipset is experiencing the same issue. This is because the ESP32 chipset communicates through the I2S Stream. Unfortunately, most EPS32s can't handle Marlin's I2S stream properly.

Therefore, the following features are currently not implemented:

1.Incompatible with INPUT_SHAPING

2.Incompatible with Babystep

3.Incompatible with FT motion

4.Incompatible with Linear Advance

And there are currently issues with inaccurate temperature sensors and layer shift issues.

The ESP32 chipset has powerful processing performance, but unfortunately, only ARM or STM32 series are mainly developed in Marlin, so it's better not to use the ESP32 chipset until a capable person comes out. The reason why this board is inexpensive is because of the above reasons, and the manufacturer will no longer update after version 2.0.9.

(There were some translation errors, so I re-edited some phrases.)

[EvilGremlin]

@vfbank you weren't paying attention, LA and babystep are supported now