Skip to content

fix(Statistics): Fixed export of functions#1263

Merged
BotellaA merged 3 commits intonextfrom
feat/smart_sampling
Apr 30, 2026
Merged

fix(Statistics): Fixed export of functions#1263
BotellaA merged 3 commits intonextfrom
feat/smart_sampling

Conversation

@MelchiorSchuh
Copy link
Copy Markdown
Member

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 22, 2026

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-tidy (v20.1.8) reports: 1 concern(s)
  • src/geode/mesh/helpers/mesh_statistics.cpp:38:20: warning: [readability-function-cognitive-complexity]

    function 'compute_surface_statistics' has cognitive complexity of 11 (threshold 10)

       38 |     MeshStatistics compute_surface_statistics(
          |                    ^
    /__w/OpenGeode/OpenGeode/src/geode/mesh/helpers/mesh_statistics.cpp:43:9: note: +1, including nesting penalty of 0, nesting level increased to 1
       43 |         for( const auto polygon_id : Range{ mesh.nb_polygons() } )
          |         ^
    /__w/OpenGeode/OpenGeode/src/geode/mesh/helpers/mesh_statistics.cpp:45:13: note: +2, including nesting penalty of 1, nesting level increased to 2
       45 |             for( const auto edge_id :
          |             ^
    /__w/OpenGeode/OpenGeode/src/geode/mesh/helpers/mesh_statistics.cpp:49:17: note: +3, including nesting penalty of 2, nesting level increased to 3
       49 |                 if( const auto adjacent =
          |                 ^
    /__w/OpenGeode/OpenGeode/src/geode/mesh/helpers/mesh_statistics.cpp:52:21: note: +4, including nesting penalty of 3, nesting level increased to 4
       52 |                     if( adjacent.value() < polygon_id )
          |                     ^
    /__w/OpenGeode/OpenGeode/src/geode/mesh/helpers/mesh_statistics.cpp:64:9: note: +1, including nesting penalty of 0, nesting level increased to 1
       64 |         if( nb_edges != 0 )
          |         ^

Have any feedback or feature suggestions? Share it here.

MelchiorSchuh and others added 2 commits April 28, 2026 15:36
Co-authored-by: Copilot <copilot@github.com>
@BotellaA BotellaA merged commit 27a0b29 into next Apr 30, 2026
19 checks passed
@BotellaA BotellaA deleted the feat/smart_sampling branch April 30, 2026 08:19
@BotellaA
Copy link
Copy Markdown
Member

🎉 This PR is included in version 16.5.1-rc.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@BotellaA
Copy link
Copy Markdown
Member

BotellaA commented May 1, 2026

🎉 This PR is included in version 16.5.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants