Skip to content

Conversation

@mwestphal
Copy link
Member

@mwestphal mwestphal commented Dec 24, 2025

Describe your changes

  • Add scene::add(void* ptr, std::size_t size) API to read file from memory
  • Add supports in F3D to read from the input stream (std::cin) by reading the - file
  • Add testing logic and tests for all formats the support reading from memory, default conf and invalid
  • Add SDKTest for error code path
  • split gltf/glb reader into two different readers for the binary stream identification
  • Add python/c/java/wasm binding for the new add method and tests on python/c/java
  • Takes VTK_VERSION into account to enable supports streaming or not
  • Update doc about format support, - usage, plugins and more
  • Add stream support in --list-readers
  • Update VTK for stream features

Status:

  • VTKLegacy: OK : 9.6.X
  • VTKXMLVT*: OK : 9.6.X
  • VTKHDF: OK :
  • ExodusII: NO -> unsupported
  • NetCDF: NO -> issue
  • PLYReader: OK
  • STL: OK
  • DICOM: OK
  • Nrrd: NO -> unsupported
  • MetaImage: NO -> unsupported
  • TIFF: NO -> issue
  • QuakeMDL: OK
  • CityGML: OK
  • PTS: OK
  • STEP: OK
  • IGES: NO -> unsupported
  • BREP: OK (brep and bin.brep)
  • XBF: OK
  • Alembic: OK
  • OBJ: OK
  • GLB: OK (no mdl)
  • GLTF: OK (no external URI)
  • Draco: OK
  • 3DS: OK
  • VRML: NO
  • FBX: OK
  • COLLADA: OK
  • OFF: OK
  • DXF: OK
  • DirectX: OK
  • 3MF: OK
  • USD: NO -> issue
  • VDB: OK
  • Splat: OK
  • SPZ: OK

Issue ticket number and link if any

Checklist for finalizing the PR

  • I have performed a self-review of my code
  • I have added tests for new features and bugfixes
  • I have added documentation for new features
  • If it is a modifying the libf3d API, I have updated bindings
  • If it is a modifying the .github/workflows/versions.json, I have updated docker_timestamp

Continuous integration

Please write a comment to run CI, eg: \ci fast.
See here for more info.

@github-actions
Copy link

github-actions bot commented Dec 24, 2025

You are modifying versions.json, please update the docker timestamp as well, this will generate new docker images and caches needed for CI.

@mwestphal mwestphal changed the title Quick and dirty piped stream reading Add support for reading streams Dec 30, 2025
@mwestphal mwestphal marked this pull request as ready for review December 30, 2025 18:18
@codecov
Copy link

codecov bot commented Dec 30, 2025

Codecov Report

❌ Patch coverage is 98.83041% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 96.71%. Comparing base (0e7fbf5) to head (b117a10).
⚠️ Report is 13 commits behind head on master.

Files with missing lines Patch % Lines
application/F3DStarter.cxx 97.46% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2754      +/-   ##
==========================================
+ Coverage   96.66%   96.71%   +0.04%     
==========================================
  Files         142      152      +10     
  Lines       13011    13115     +104     
==========================================
+ Hits        12577    12684     +107     
+ Misses        434      431       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mwestphal mwestphal requested review from Meakk and snoyer December 31, 2025 11:53
@mwestphal mwestphal added this to the 3.5.0 milestone Dec 31, 2025
@mwestphal mwestphal requested a review from Meakk January 8, 2026 22:29
@mwestphal mwestphal merged commit 72a7fbf into f3d-app:master Jan 10, 2026
75 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants