feat(enclosure): paint-quote drawing pack (parts table + masking plan) - #397
Open
tomassasovsky wants to merge 2 commits into
Open
feat(enclosure): paint-quote drawing pack (parts table + masking plan)#397tomassasovsky wants to merge 2 commits into
tomassasovsky wants to merge 2 commits into
Conversation
Powder coaters quote by m2, and nobody can eyeball the painted area off a flat pattern riddled with pedal slots and screen apertures. Adds vamp_paint_quote.pdf: a supplier-facing sheet with the parts table (qty, folded size, painted area both faces, per-material subtotals) plus masking pages. - MASK layer: annotation-only rings over the features that must stay bare -- the 3 transition PEM M4 threads, the 2 PEM threads per screen bracket, and a 20 mm bonding land around the M6 earth stud. Paint is an insulator, so a coated land under the ring terminal is a dead chassis ground. - Areas are derived from the CUT/VENT geometry of the generated DXFs and sizes from the STEPs, so the table cannot drift from the model the way a hand-typed one does. - vamp_post is broken out as a separate material line: it is 1.6 mm cold-rolled steel, not the aluminium shell, and takes a different pretreatment. - New vamp_pintura.zip pack: quote sheet + part PDFs, no DXFs (the coater cuts nothing, and a flat pattern only invites confusion). No geometry change; _check() still passes. Closes #396
tomassasovsky
added
stage:in-review
The screens get bonded to the shell, not clamped, so the brackets are never manufactured -- quoting paint on 8 of them inflates the m2 by 3.4%. The part stays in DXF_PARTS as the fallback if bonding is abandoned. Also narrows vamp_pintura.zip to the paint BOM instead of every DXF_PART: the adhesive overlay and the unused rear-panel variant were being sent to a coater who has no business painting either. Total: 1.4672 -> 1.4168 m2 per unit.
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.
Closes #396
Three powder coaters (Vieytes, Mora-Epox, Espocolor) were asked to quote the VAMP shell. Vieytes came back confirming the part fits their oven and that they quote by m² — which is the number the repo could not produce. Nobody can eyeball painted area off a flat pattern riddled with ten pedal slots and two screen apertures.
What this adds
out/vamp_paint_quote.pdf— a supplier-facing sheet (Spanish, that is who reads it):Notable details
MASKlayer carries the rings. It is excluded from extents and from the area maths the same wayNOTE/ENGRAVEare, and_fold_from_dxf.pyalready filters by layer so the 3D path ignores it.vamp_postis its own material line — 1.6 mm cold-rolled steel, not the aluminium shell, so it needs a different pretreatment and cannot be lumped into the same m².vamp_pintura.zip: quote sheet + part PDFs, deliberately no DXFs — the coater cuts nothing.Verification
vamp_enclosure.pyfull run: geometry assertions ALL PASS, every DXF/PDF/STEP regenerates.dart analyzereports 4 pre-existing errors that are a worktree artifact (the sameControlStateresolved through both the worktree and the main checkout path), unrelated to this change.Known gap
The faceplate row shows its flat size (850 × 407 × 2) because
build_step()exports the lid unfolded; note 7 on the sheet says so explicitly. The assembled envelope (850 × 423 × 100) is what governs oven fit and is in the header block.