Skip to content

Ignore trigger colliders in laser raycasts#33

Draft
Sean-Kenneth-Doherty wants to merge 1 commit into
Dunbaratu:masterfrom
Sean-Kenneth-Doherty:fix-ignore-trigger-colliders
Draft

Ignore trigger colliders in laser raycasts#33
Sean-Kenneth-Doherty wants to merge 1 commit into
Dunbaratu:masterfrom
Sean-Kenneth-Doherty:fix-ignore-trigger-colliders

Conversation

@Sean-Kenneth-Doherty
Copy link
Copy Markdown

Summary

  • Pass QueryTriggerInteraction.Ignore to the local Physics.RaycastAll call.
  • Prevent non-physical trigger volumes, such as scanner-arm range triggers, from being selected as laser hits.

Fixes #30.

Validation

  • xbuild src/LaserDist/LaserDist.csproj /p:Configuration=Release /p:ReferencePath="/home/sean/.local/share/Steam/steamapps/common/Kerbal Space Program/KSP_Data/Managed" /verbosity:minimal
  • jq empty LaserDist.version GameData/LaserDist/LaserDist.version LaserDist.netkan
  • brace-balance check for Parts/LaserDist01/part.cfg, Parts/LaserDist02/part.cfg, and Parts/LaserDist03/part.cfg
  • git diff --check

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.

To fix DLC scanner arm bug - ignore Colliders with triggers.

1 participant