Skip to content

Conversation

@wawanbreton
Copy link
Contributor

@wawanbreton wawanbreton commented Apr 24, 2025

Add a method to parse 3D models defined in separate component files

CURA-12099

@github-actions
Copy link

github-actions bot commented Apr 24, 2025

Test Results

3 tests  ±0   3 ✅ ±0   0s ⏱️ ±0s
1 suites ±0   0 💤 ±0 
1 files   ±0   0 ❌ ±0 

Results for commit 031a70a. ± Comparison against base commit 28e0e4d.

♻️ This comment has been updated with latest results.

Copy link
Member

@rburema rburema left a comment

Choose a reason for hiding this comment

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

👍🏻

(I'll inform the other PR that we'd like to do it this way.)

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for exporting and importing 3MF 3D models by introducing a new method to parse external component files. The changes include:

  • Adding getComponentPath() and parseComponentData() methods in SceneNode.
  • Updating fillByXMLNode() to process external component nodes.
  • Bumping the dependency version in conandata.yml to 5.11.0-alpha.0.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
src/SceneNode.cpp New methods for component data handling and XML parsing
include/Savitar/SceneNode.h Header declarations for the new component handling methods
conandata.yml Version bump supporting the updated functionality

@HellAholic HellAholic merged commit c9fecec into main May 2, 2025
11 checks passed
@HellAholic HellAholic deleted the CURA-12099_export-and-import-to-bambu-3mf branch May 2, 2025 11:09
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