Skip to content

feat(MedialBallsExtractor): mutex in ray tracing#1282

Merged
BotellaA merged 1 commit into
nextfrom
feat/MedialBallsExtractor
Jun 3, 2026
Merged

feat(MedialBallsExtractor): mutex in ray tracing#1282
BotellaA merged 1 commit into
nextfrom
feat/MedialBallsExtractor

Conversation

@MoulardM
Copy link
Copy Markdown
Contributor

@MoulardM MoulardM commented Jun 3, 2026

No description provided.

@MoulardM MoulardM requested a review from BotellaA June 3, 2026 08:25
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 3, 2026

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-tidy (v20.1.8) reports: 2 concern(s)
  • src/geode/model/helpers/ray_tracing.cpp:46:64: warning: [readability-identifier-naming]

    invalid case style for function 'directions_2D'

       46 |     const std::array< geode::Vector2D, NUMBER_2D_DIRECTIONS >& directions_2D()
          |                                                                ^~~~~~~~~~~~~
          |                                                                directions_2_d
  • src/geode/model/helpers/ray_tracing.cpp:64:64: warning: [readability-identifier-naming]

    invalid case style for function 'directions_3D'

       64 |     const std::array< geode::Vector3D, NUMBER_3D_DIRECTIONS >& directions_3D()
          |                                                                ^~~~~~~~~~~~~
          |                                                                directions_3_d

Have any feedback or feature suggestions? Share it here.

@BotellaA BotellaA merged commit 97403c2 into next Jun 3, 2026
23 checks passed
@BotellaA BotellaA deleted the feat/MedialBallsExtractor branch June 3, 2026 12:57
@BotellaA
Copy link
Copy Markdown
Member

BotellaA commented Jun 3, 2026

🎉 This PR is included in version 17.2.0-rc.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