Tool/firmware tester#46
Open
dennisklappe wants to merge 3 commits into
Open
Conversation
Shared baud selector (localStorage-backed, default 250000) added to printer-control, printer-setup, lc-lb-measure, and calibrator. Tools were hardcoded to 115200 which silently broke any printer flashed with the firmware-builder default of 250000. Also fixes home() to use sendCommandAndWait with a 120s timeout instead of a fire-and-forget sendCommand + 2s sleep, which falsely errored "Position request timeout" on real homing operations.
The Firmware Builder ends at firmware.bin download — there's no easy way to confirm a flashed firmware matches the hardware. This tool fills that gap: after flashing, it walks the user through driver, endstop, stepper- direction, homing-direction, and extruder verification, then produces a corrected Firmware Builder JSON if anything is reversed. Wizard steps: 1. Connect (with optional Firmware Builder JSON upload) 2. Diagnostics — M115 / M122 / M119 / M105 sanity checks 3. Endstop walk-through — manual trigger of X, Y, Z, B, C; polled via M119 4. Stepper directions — small G91 moves with yes/no prompts per axis 5. Homing direction — G28 per axis, "moved toward endstop?" prompts 6. Extruder — direction test plus optional E-steps calibration 7. Results — per-test summary and corrected JSON for download Shared library additions: - printer-interface.js: queryEndstops, queryDriverStatus, queryFirmwareInfo parse M119, M122, M115 responses (none were parsed before) Firmware Builder hand-off: - exportConfig now also stashes the JSON in localStorage under rep5x_firmware_builder_config so the tester can auto-load it without the user re-uploading the file
dennisklappe
force-pushed
the
tool/firmware-tester
branch
from
0572f06 to
4a40b75
Compare
Deploying rep5x-tools with
|
| Latest commit: |
1d2051e
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://486e06a3.rep5x-tools.pages.dev |
| Branch Preview URL: | https://tool-firmware-tester.rep5x-tools.pages.dev |
Deploying rep5x with
|
| Latest commit: |
1d2051e
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://d80f5afb.rep5x.pages.dev |
| Branch Preview URL: | https://tool-firmware-tester.rep5x.pages.dev |
…ffle
Builds on top of the firmware-tester commit:
- Baud selector dropped into the Connect step, mirroring printer-control /
printer-setup / lc-lb-measure / calibrator. Loads shared/baud-selector.js
and mounts on the #baudSelector span.
- Em dashes removed throughout the firmware-tester tool (HTML strings,
JS comments, README, user-facing placeholders) per project style.
- Homing-direction step redesigned to be safe: no longer sends G28 (which
with a reversed home_dir keeps moving until the carriage hits the frame).
Instead asks the user where each linear endstop physically sits (MIN or
MAX) and compares to the firmware's xHomeDir / yHomeDir / zHomeDir from
the uploaded JSON. B is centred-home so MIN/MAX doesn't apply.
- Tools landing page reflowed:
* Workflow split 3 + 4 (firmware setup left, calibration & print right)
so the column-end positions align with phase boundaries.
* Printer Setup moved out of Setup & configuration into Calibration,
which is now a 3-tile / 3-column row (Printer Setup, LC/LB Measure,
Calibrator). Setup & configuration is back to a clean 2-tile row
(Firmware Builder + Firmware Tester).
dennisklappe
force-pushed
the
tool/firmware-tester
branch
from
4a40b75 to
1d2051e
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.