Skip to content

[mapping] BeamProjectionDifferenceMultiMapping: new feature and refactoring#212

Open
EulalieCoevoet wants to merge 4 commits intosofa-framework:masterfrom
EulalieCoevoet:pr_beamprojectiondifference
Open

[mapping] BeamProjectionDifferenceMultiMapping: new feature and refactoring#212
EulalieCoevoet wants to merge 4 commits intosofa-framework:masterfrom
EulalieCoevoet:pr_beamprojectiondifference

Conversation

@EulalieCoevoet
Copy link
Collaborator

In this PR:

  • Outputs the projections
  • Add a filter: when updating the projection at each time step, use this parameter to filter small changes. If the projection is on the same edge as the previous step, and the weight (in [0, 1]) is smaller than the value of 'filter', skip.
  • Refactoring

@EulalieCoevoet EulalieCoevoet added the enhancement New feature or request label Jan 21, 2026
@EulalieCoevoet EulalieCoevoet marked this pull request as ready for review February 10, 2026 09:16
@EulalieCoevoet EulalieCoevoet added the pr: status to review To notify reviewers to review this pull-request label Feb 10, 2026
, d_updateProjectionOrientation(initData(&d_updateProjectionOrientation, false, "updateProjectionOrientation", "Update the projection on the beam at each time step even when direction[0]=1."))
, d_draw(initData(&d_draw, "draw", "Draw projection points and directions"))
, d_drawSize(initData(&d_drawSize, Real(3), "drawSize", ""))
, d_projections(initData(&d_projections, "projections", ""))
Copy link
Contributor

Choose a reason for hiding this comment

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

help text is missing

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I've added "Computed projection points".

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

Labels

enhancement New feature or request pr: status to review To notify reviewers to review this pull-request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants