Change Filament Feature #2926

antonkopa · 2016-01-23T20:08:39Z

I decided to test out the "Change Filament" feature, and before it continues the print, it pulls the filament completely back out again.

When I commented out #define AUTO_FILAMENT_CHANGE, it doesn't have this issue.

The issue here I believe is the current position of the extruder is not being restored from the remembered position.

FILE: Configuration_adv.h

// Add support for experimental filament exchange support M600; requires display
#if ENABLED(ULTIPANEL)
#define FILAMENTCHANGEENABLE
  #if ENABLED(FILAMENTCHANGEENABLE)
    #define FILAMENTCHANGE_XPOS 3
    #define FILAMENTCHANGE_YPOS 3
    #define FILAMENTCHANGE_ZADD 10
    #define FILAMENTCHANGE_FIRSTRETRACT -2
    #define FILAMENTCHANGE_FINALRETRACT -100
    #define AUTO_FILAMENT_CHANGE                //This extrude filament until you press the button on LCD
    #define AUTO_FILAMENT_CHANGE_LENGTH 0.04    //Extrusion length on automatic extrusion loop
    #define AUTO_FILAMENT_CHANGE_FEEDRATE 300   //Extrusion feedrate (mm/min) on automatic extrusion loop
  #endif
#endif

Fix status LEDs update when reinitializing the display By using the regular update function during init. Fixing #2683

dx -> dsx dy -> dsy dz -> dsz de -> dse

anything is long, we don't need float there

st_set_position takes long ```void st_set_position(const long& x, const long& y, const long& z, const long& e)```

Make the output of M105 more similar to Repetier. Make the text-print pert of M105 an extra function to make it reusable. `print_heaterstates()` Use `print_heaterstates()` in M019, M190 and M303

For some reason that I cannot determine, using a sub-function causes the Tune sub-menu to act strangely, yet replacing the function call with its code content works perfectly.

//100k 0603 SMD Vishay NTCS0603E3104FXT (4.7k pullup) (calibrated for Makibox hot bed) Caveat Emptor —- I’m not sure that this really makes sense, but why keep someone from using it?

• Removing trailing whitespace • Adjusting indention for better readability • Removing duplicated entries

Instead of changing SDEXTRASLOW variable to a slower speed, the new SDULTRASLOW is used for rigidbot panel.

K8200: sample config back in sync

Fix crashes when the Z axis is moved via LCD by calling `plan_buffer_line` only when there is enough room in the planner buffer, to avoid endless recursion. A brief description about what went wrong is in #1166

Fix GCode handling of spaces between command letter and first digit

Fix board type for Hephestos: RAMPS 1.4

Fix "M145" output in M503 report

Example configuration for Kossel XL

raise_z_after_probing() was only called if the printer type was set to a DELTA or if the printer had a docking sled for the probe.

twice for a servo based probe.

Temporary workaround for stack overflow

Fix Z_RAISE_AFTER_PROBING for non DELTA printers

Code cleanup in gcode_T

Enable USE_WATCHDOG by default

Remove unused ALARM_PIN from all the Configuration.h's

Fix disable_all_heaters() for MAX6675

Remove unused ALARM_PIN

Fix status LEDs update when reinitializing the display

Correct raise_z_for_servo for Z offsets below the nozzle

thinkyhead · 2016-03-06T04:30:23Z

I will close this for now and we can continue to discuss this issue in a new topic around M600. Current topics of discussion are:

Integrating Josef Prusa's general implementation of M600
Adding keepalive and paused for input messages to keep hosts happy during long operations

See #3068 and #2933 for some background.

AnHardt and others added 30 commits December 3, 2015 00:38

Update README.md

e19fe40

Fix status LEDs update when reinitializing the display

8d1eeff

Fix status LEDs update when reinitializing the display By using the regular update function during init. Fixing #2683

Renaming delta speeds (PR2824)

0fbfb22

dx -> dsx dy -> dsy dz -> dsz de -> dse

The probe stowing coordinates have changed

6bfcb74

change dx, dy, dz and de from float to long

6d691ac

anything is long, we don't need float there

change nx, ny, nz and ne from float to long

78a347a

st_set_position takes long ```void st_set_position(const long& x, const long& y, const long& z, const long& e)```

Unglobalize no_wait_for_cooling

fcceb98

Host parseble output for M109, M190 and M303

57da1b8

Make the output of M105 more similar to Repetier. Make the text-print pert of M105 an extra function to make it reusable. `print_heaterstates()` Use `print_heaterstates()` in M019, M190 and M303

Merge ConformingHostMessages (PR#2822)

ae4acc4

Revert nozzle_bed_fan_menu_items

60d5658

For some reason that I cannot determine, using a sub-function causes the Tune sub-menu to act strangely, yet replacing the function call with its code content works perfectly.

Allow Thermistor #12 to be used on hotends also

4f314af

//100k 0603 SMD Vishay NTCS0603E3104FXT (4.7k pullup) (calibrated for Makibox hot bed) Caveat Emptor —- I’m not sure that this really makes sense, but why keep someone from using it?

Back port language translation

e2da2b4

Change link to U8glib source

e7b40bb

Clean up spacing and duplicate entries

0cebe85

• Removing trailing whitespace • Adjusting indention for better readability • Removing duplicated entries

Back port Cosmetic changes

0b42946

Backport cosmetic

1635613

Prepare Ramps for user redefinition

3b2973d

If unused, just don’t define it.

0fa3bac

Call the probe pin a probe pin and not a min endstop

58658c9

Improve planner & stepper PR #263

7a670e3

Residual unused variable — Backport

f6950ac

Fixing Rigidbot LCD Panel Support

5eeea2f

Cleaning up, getting rid of the fake encoder count variable.

83c4131

Adding SDULTRASLOW variable.

e6cc232

Instead of changing SDEXTRASLOW variable to a slower speed, the new SDULTRASLOW is used for rigidbot panel.

reusing millis()

a64bcd5

Getting rid of SDEXTRASLOW, SDSLOW variables.

2c9e041

Update Configuration_adv.h

71aafb7

Update Configuration_adv.h

056daf0

Update Configuration_adv.h

5ee6fe6

Update Configuration_adv.h

992fbb8

silvio-didonna and others added 24 commits February 25, 2016 15:52

Fix board type: RAMPS 1.4

eb58176

added kossel xl example configuration

68eae2b

Fix GCode handling of spaces between command letter and first digit

6a4f06d

Merge pull request #3018 from CONSULitAS/RCBugFix_K8200_backinsync

8dccd5a

K8200: sample config back in sync

Fix crashes when the Z axis is moved via LCD

fd9e94e

Fix crashes when the Z axis is moved via LCD by calling `plan_buffer_line` only when there is enough room in the planner buffer, to avoid endless recursion. A brief description about what went wrong is in #1166

minor cleanup in gcode_T

100271e

Merge pull request #3044 from thinkyhead/rc_gcode_spaces

d118e3f

Fix GCode handling of spaces between command letter and first digit

Fix "M145" output in M503 report

7253a19

Merge pull request #3031 from tnw513/RCBugFix

0bc6483

Fix board type for Hephestos: RAMPS 1.4

Merge pull request #3052 from thinkyhead/rc_fix_m145_eeprom_output

fd1243c

Fix "M145" output in M503 report

Merge pull request #3043 from oxivanisher/RCBugFix

252f4a7

Example configuration for Kossel XL

Z_RAISE_AFTER_PROBING value was being ignored by Marlin because

17cc2a9

raise_z_after_probing() was only called if the printer type was set to a DELTA or if the printer had a docking sled for the probe.

As pointed out by @Blue-Marlin the previous code would raise the Z axis

67e15aa

twice for a servo based probe.

Merge pull request #3050 from Blue-Marlin/fix-1166

a42fb35

Temporary workaround for stack overflow

Merge pull request #3057 from jbrazio/3034-z_raise_after_probing_fix

186629a

Fix Z_RAISE_AFTER_PROBING for non DELTA printers

Merge pull request #3051 from thinkyhead/rcbugfix_dual_axis_offset

222d13d

Code cleanup in gcode_T

Fix disable_all_heaters() for MAX6675

0631541

Merge pull request #2986 from thinkyhead/enable_watchdog

9eb4ace

Enable USE_WATCHDOG by default

Remove unused ALARM_PIN

a68228b

Remove unused ALARM_PIN from all the Configuration.h's

Merge pull request #3066 from Blue-Marlin/Fix-disable_all_heaters()

79778d4

Fix disable_all_heaters() for MAX6675

Merge pull request #3067 from Blue-Marlin/ALARM_PIN

547d35a

Remove unused ALARM_PIN

Merge pull request #2821 from MarlinFirmware/ReinitializeDisplay

04a0d45

Fix status LEDs update when reinitializing the display

correct raise_z_for_servo

0fb507a

Merge pull request #3089 from thinkyhead/rc_apply_2924

aa0b048

Correct raise_z_for_servo for Z offsets below the nozzle

thinkyhead closed this Mar 6, 2016

clexpert mentioned this pull request Apr 28, 2016

Auto Filament Change Reworked: finally working #3638

Closed

3 tasks

thinkyhead mentioned this pull request May 2, 2016

Rewritten FILAMENT_CHANGE_ENABLE feature #3662

Merged

3 tasks

jbrazio modified the milestone: 1.1.0 Jul 18, 2016

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Change Filament Feature #2926

Change Filament Feature #2926

antonkopa commented Jan 23, 2016

thinkyhead commented Mar 6, 2016

Change Filament Feature #2926

Change Filament Feature #2926

Conversation

antonkopa commented Jan 23, 2016

thinkyhead commented Mar 6, 2016