Skip to content

write multiple shapes to one STEP file - #226

Merged
bschwind merged 2 commits into
bschwind:mainfrom
torusJKL:step-write-all
Jun 9, 2026
Merged

write multiple shapes to one STEP file#226
bschwind merged 2 commits into
bschwind:mainfrom
torusJKL:step-write-all

Conversation

@torusJKL

@torusJKL torusJKL commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

OCCT can write multiple shapes as individual items to a single step file.
The current write_step method only accepts a single shape.

This change includes:

  • Add write_all_step to transfers multiple shapes to a single step writer
  • Refactor write_step to delegate to write_all_step
  • Add StepWriteNoShapes error for empty input (OCCT does not write empty step files and silently fails)
  • Add StepWriteTransferFailed error to distinguish transfer failures

torusJKL added 2 commits June 7, 2026 16:03
- Add write_all_step to transfers multiple shapes to a single step writer
- Refactor write_step to delegate to write_all_step
- Add StepWriteNoShapes error for empty input
- Add StepWriteTransferFailed error to distinguish transfer failures

@bschwind bschwind left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM, thanks!

@bschwind
bschwind merged commit fc6b069 into bschwind:main Jun 9, 2026
4 checks passed
@torusJKL

torusJKL commented Jun 9, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for merging!

@torusJKL
torusJKL deleted the step-write-all branch June 9, 2026 15:13
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.

2 participants