Skip to content

Revert filament_diameter and min_nozzle_diameter to mm - #154

Merged
CZDanol merged 2 commits into
OpenPrintTag:mainfrom
ljanecek:revert-diameter-v2
Sep 25, 2026
Merged

CZDanol merged 2 commits into
OpenPrintTag:mainfrom
ljanecek:revert-diameter-v2

Conversation

@ljanecek

@ljanecek ljanecek commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Restore keys 30 (filament_diameter) and 33 (min_nozzle_diameter) as number in mm. Mark keys 61 and 62 (the former _v2 µm fields) as deprecated so they are never reused.

This reverts the field changes from #149 (04ddc46). The _v2 fields were published in the spec docs, so the changelog now records their removal.

Also fixes the TPU default in material_type_enum.yaml, which used the old min_nozzle_diameter name with a µm value (400 → 0.4).

Restore keys 30 (filament_diameter) and 33 (min_nozzle_diameter) as number in mm. Mark keys 61 and 62 (the former _v2 µm fields) as deprecated so they are never reused.
ondratu
ondratu previously approved these changes Sep 25, 2026
Comment thread docs_src/changelog.md Outdated
## Not yet released
1. `filament_diameter` (mm) replaced with `filament_diameter_v2` (µm) to align with the OpenPrintTag architecture
1. `min_nozzle_diameter` (mm) replaced with `min_nozzle_diameter_v2` (µm) to align with the OpenPrintTag architecture
1. `filament_diameter_v2` and `min_nozzle_diameter_v2` (µm) removed, `filament_diameter` and `min_nozzle_diameter` (mm) restored

@CZDanol CZDanol Sep 25, 2026 •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You've removed the "filament_diameter_ removal info, so "restored" does not make much sense now.

The entry said the mm fields were restored, but their removal is no longer listed, so it read as undoing a change that is not in the changelog.
@CZDanol
CZDanol merged commit 86c0b75 into OpenPrintTag:main Sep 25, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants