aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* heaters: Make heater.py an "extras" moduleKevin O'Connor2020-04-2516-45/+50
| | | | | | | The heater logic is an independent module that does not need to be treated as part of the "core" klipper code. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* heater: Move M105 command handling from gcode.py to heater.pyKevin O'Connor2020-04-255-49/+59
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* klippy: Add an is_shutdown() methodKevin O'Connor2020-04-251-3/+5
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* gcode_button: Simplify script executionKevin O'Connor2020-04-252-27/+12
| | | | | | | | | It's no longer necessary to queue gcode scripts - gcode.run_script() now does that automatically. Also, make release_gcode config optional. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* probe: Rename probe_prepare/finalize to probe_prepare/finishKevin O'Connor2020-04-252-3/+3
| | | | | | | | The probe_finalize() name is used for callbacks in both the PrinterProbe and ProbePointsHelper classes. Rename the PrinterProbe callback to avoid confusion. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* gcode: Rename respond() to respond_raw()Kevin O'Connor2020-04-248-31/+31
| | | | | | | | | | | Rename the method to make it more clear that it is a low-level call that should be rarely used. Also, change gcode_button.py, hall_filament_width_sensor.py, and tsl1401cl_filament_width_sensor.py to use respond_info() instead of respond_raw(). Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* gcode: Don't export respond_error()Kevin O'Connor2020-04-241-6/+6
| | | | | | | Don't export the respond_error() method as callers should raise a gcode.error() instead. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* virtual_sdcard: No need to call gcode.respond_error() on disk errorKevin O'Connor2020-04-241-2/+0
| | | | | | | Report low-level disk errors via the log and use the normal sdcard mechanism to report end of sdcard print to gcode users. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* gcode: Raise a gcode.error() from G20 commandKevin O'Connor2020-04-241-1/+1
| | | | | | | It's important to raise an error (instead of calling respond_error() ) so that a G20 will stop a virtual_sdcard print. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* respond: Simplify M118 handlingKevin O'Connor2020-04-241-16/+13
| | | | | | | | No need to check for '#original' in params - it's always there. Properly handle a lower case m118 command. Don't register a help text for the legacy M118 g-code command. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* z_tilt: Raise a gcode.error() instead of directly calling respond_error()Kevin O'Connor2020-04-242-11/+7
| | | | | | | Raising an error (instead of reporting an error) is important as only a raised error will stop a print from virtual_sdcard. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* gcode_arcs: Disable E moves in absolute extrude modeKevin O'Connor2020-04-232-7/+5
| | | | | | | | Commit 402110f6 didn't actually fix absolute extrude mode. For now, report an error if an extrude move is requested in absolute extrude mode. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* gcode_arcs: Minor cleanup of planArc()Kevin O'Connor2020-04-221-42/+27
| | | | | | | | Fix linear_travel optimization. Use "2*pi" instead of "radians(360)". Always emit at least one segment and always end at the final target coordinates. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* gcode_arcs: Fix Z moves and E movesKevin O'Connor2020-04-222-11/+18
| | | | | | | | Fix Z moves so that the movement is spaced out among the arc movements. Fix extrude moves when in absolute extrude mode. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* gcode_arcs: Simplify parameter parsingKevin O'Connor2020-04-222-87/+39
| | | | | | | | | | Use the normal gcode.get_float() mechanism for extracting parameters from the g-code command. Don't register descriptions for the G2/G3 commands as the convention is to only use descriptions for "extended g-code commands". Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* safe_z_home: Simplify check for axes to homeKevin O'Connor2020-04-221-5/+3
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* lpc176x: Add support for additional i2c busesKevin O'Connor2020-04-213-46/+66
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* neopixel: Make sure there is a minimum low duration after a long pulseKevin O'Connor2020-04-211-4/+3
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* rotary_delta: The get_status() method should take eventtime parameterKevin O'Connor2020-04-201-1/+1
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* config: Add tmc2130 example to generic-bigtreetech-skr-v1.4.cfgKevin O'Connor2020-04-201-4/+55
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* config: Remove obsolete description of menu_root in example-extras.cfgKevin O'Connor2020-04-191-5/+0
| | | | | | Reported by @ld3300. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* homing: Avoid divide by zero error on PROBE at minimum Z positionKevin O'Connor2020-04-191-0/+2
| | | | | | Reported by @proto3. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* docs: Note __test menu removed in Config_Changes.mdKevin O'Connor2020-04-181-0/+4
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* neopixel: Allow host software to configure neopixel timingKevin O'Connor2020-04-152-11/+23
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* neopixel: Use names for key timing valuesKevin O'Connor2020-04-151-12/+19
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* test: Fix printer-tronxy-p802e-2020.cfg and add to printers.testKevin O'Connor2020-04-142-11/+14
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* config: Create printer-tronxy-p802e (#2674)camatthew882020-04-141-0/+101
| | | Signed-off-by: Matthew Blankenbehler <mablankenbehler23@whitnall.com>
* bed_mesh: Fix typo (#2692)Piotr Usewicz2020-04-141-1/+1
| | | Signed-off-by: Piotr Usewicz <piotr@layer22.com>
* config: Use restart_method=cheetah in generic-fysetc-cheetah-v1.1.cfgKevin O'Connor2020-04-081-0/+1
| | | | | | Reported by @csutcliff. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* config: Change uart addresses in generic-fysetc-cheetah-v1.1.cfgKevin O'Connor2020-04-081-2/+2
| | | | | | Reported by @mihero and @Magoo0876. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* test: Add generic-ultimaker-ultimainboard-v2 to printers.testKevin O'Connor2020-04-081-0/+1
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* config: Add Ultimaker UltiMainboard v2 config. (#2696)Avery Todd2020-04-081-0/+123
| | | | | | | Adds config for the Ultimaker 2's UltiMainboard v2 board, with the correct pins assigned and current control for the A4988 motor drivers. Signed-off-by: Avery Todd <averyct@pm.me>
* Revert "bltouch: No need to pause the toolhead for PWM off commands"Kevin O'Connor2020-04-041-6/+4
| | | | | | | | | | This reverts commit 5c8d15bbee9b1c4c82e3a2ecb5c63ce37f4a8dcb. The lower_probe() code was already manually optimizing the timing of the pwm disable, and that conflicted with the optimization in commit 5c8d15bb. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* scripts: Update Docker file for current Klipper version (#2666)sillyfrog2020-04-021-3/+9
| | | | Signed-off-by: Trent Davis <tgh@sillyfrog.com>
* docs: Update BLTouch.md initial tests for users with a regular z endstopKevin O'Connor2020-04-021-10/+11
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* docs: Add BLTouch.md section describing clone config optionsKevin O'Connor2020-04-022-26/+61
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* docs: Add touch_mode sensor test to BLTouch.mdKevin O'Connor2020-04-021-13/+23
| | | | | | Describe how to test the sensor pin using the BLTouch touch_mode. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* config: Reword bltouch description in example-extras.cfgKevin O'Connor2020-04-021-18/+16
| | | | | | | | | | | | | | The pin_up_touch_mode_reports_triggered should be set to True on all genuine BLTouch. Expand the description for stow_on_each_sample and state the default. Reword the set_output_mode option to be more similar to other descriptions. It is no longer necessary to warn about clones being not supported. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* config: Fix reset/restart on Fysetc Cheetah v1.2Grigori Goronzy2020-04-021-0/+1
| | | | | | | | | Deassert RTS for this board and enable the new Cheetah-specific reset sequence. Fixes #2026. Signed-off-by: Grigori Goronzy <greg@chown.ath.cx>
* serial: Add Fysetc Cheetah board specific reset sequenceGrigori Goronzy2020-04-023-13/+56
| | | | | | | | | | | | | | | | | | Fysetc Cheetah v1.2 boards require a special sequence to reset reliably. This sequence works for me in all cases. Simpler sequences without double reset did not work correctly. This is likely because of a weird stateful circuitry for toggling the bootloader state. Cheetah boards use RTS to configure bootloader triggering. By default, pySerial sets RTS on connect, which unfortunately configures the board to start the bootloader on reset. Add a toggle for the RTS state to allow users to workaround. The RTS state is set before the serial connection is opened, so there are no glitches. Addresses #2026. Signed-off-by: Grigori Goronzy <greg@chown.ath.cx>
* bltouch: No need to pause the toolhead for PWM off commandsKevin O'Connor2020-04-021-4/+6
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* bltouch: Don't assume reset will do a pin_up in test_sensor()Kevin O'Connor2020-04-021-9/+16
| | | | | | | | | | | Some clones don't raise the pin on a reset and the ANTClabs BL-Touch sometimes doesn't raise the pin either. Rework the (infrequently called) sensor test code to always issue a pin_up command before the touch command. Also, perform a reset and retry if the sensor test fails. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* bltouch: Call sync_print_time() on multi_probe_end()Kevin O'Connor2020-04-021-0/+1
| | | | | | | Be sure to fully raise the probe prior to starting any future toolhead movements in the multi_probe_end() case. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* bltouch: Apply reset command for a full second on a raise_probe() failureKevin O'Connor2020-04-021-2/+1
| | | | | | | | If an error is found during the pin_up_not_triggered check, then apply the reset command for a full second (instead of just 100ms). This gives the bltouch more time to check its internal state. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* bltouch: Separate out pin_up_not_triggered case in raise_probe()Kevin O'Connor2020-04-021-6/+9
| | | | | | | Simplify raise_probe() by separating out the pin_up_not_triggered case. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* bltouch: Don't raise an error in verify_state()Kevin O'Connor2020-04-021-18/+17
| | | | | | | Return the triggered state from verify_state() and update the callers to raise the error (if needed). Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* config: Add example printer-alfawise-u30-2018.cfg configKevin O'Connor2020-03-302-0/+88
| | | | | | Reported by @elegos. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* docs: Update FAQ with TMC2208 "standalone" mode workaroundKevin O'Connor2020-03-293-150/+5
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* extruder: step_distance must be positive on SET_EXTRUDER_STEP_DISTANCEKevin O'Connor2020-03-281-1/+1
| | | | Signed-off-by: Kevin O'Connor <kevin@koconnor.net>
* stepper: No need to pass stepper_kinematics to set_step_dist()Kevin O'Connor2020-03-283-8/+11
| | | | | | | Also, no need to log in stepper.py as the update is already logged in extruder.py. Signed-off-by: Kevin O'Connor <kevin@koconnor.net>