Security_Engine: Camera Field of View bug fix

[michal-pekacki]

NOTE: Depends on

Issues addressed by this PR

Closes #3265

• arc creation bug fixed
• skip obstacles if camera device is inside (don't return null as before)

Test files

Changelog

Additional comments

[michal-pekacki]

@BHoMBot check compliance

[bhombot-ci]

@michal-pekacki to confirm, the following actions are now queued:

• check code-compliance
• check documentation-compliance
• check project-compliance
• check branch-compliance
• check dataset-compliance
• check copyright-compliance

[michal-pekacki]

@BHoMBot check compliance

[bhombot-ci]

@michal-pekacki to confirm, the following actions are now queued:

• check code-compliance
• check documentation-compliance
• check project-compliance
• check branch-compliance
• check dataset-compliance
• check copyright-compliance

[michal-pekacki]

@BHoMBot check compliance

[bhombot-ci]

@michal-pekacki to confirm, the following actions are now queued:

• check code-compliance
• check documentation-compliance
• check project-compliance
• check branch-compliance
• check dataset-compliance
• check copyright-compliance

[pawelbaran]

Changes make sense in general, only one small question asked

[pawelbaran]

No concerns code-wise, happy to see this merged if the workflows benefit 👍

[michal-pekacki]

@BHoMBot check compliance

[bhombot-ci]

@michal-pekacki to confirm, the following actions are now queued:

• check code-compliance
• check documentation-compliance
• check project-compliance
• check branch-compliance
• check dataset-compliance
• check copyright-compliance

[michal-pekacki]

@BHoMBot check required

[bhombot-ci]

@michal-pekacki to confirm, the following actions are now queued:

• check code-compliance
• check documentation-compliance
• check project-compliance
• check core
• check null-handling
• check serialisation
• check versioning
• check installer

[bhombot-ci]

The check code-compliance has already been run previously and recorded as a successful check. This check has not been run again at this time.

[bhombot-ci]

The check documentation-compliance has already been run previously and recorded as a successful check. This check has not been run again at this time.

[bhombot-ci]

@michal-pekacki just to let you know, I have provided a check-versioning result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @michal-pekacki on Revit_Toolkit

[michal-pekacki]

@BHoMBot check unit-tests

[bhombot-ci]

@michal-pekacki to confirm, the following actions are now queued:

• check unit-tests

[bhombot-ci]

FAO: @FraserGreenroyd
@FraserGreenroyd is seeking dispensation on this Pull Request to skip a required check. Please can you provide authorisation for the check to be skipped, or provide assistance as appropriate.

The check they wish to have dispensation on is unit-tests.

If you are providing dispensation on this occasion, please reply with:

@BHoMBot this is a DevOps instruction. I am authorising dispensation to be granted on check ref. 21146284981

[FraserGreenroyd]

@BHoMBot this is a DevOps instruction. I am authorising dispensation to be granted on check ref. 21146284981

[bhombot-ci]

@FraserGreenroyd I have now provided a passing check on reference 21146284981 as requested.

[FraserGreenroyd]

@BHoMBot check ready-to-merge

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check ready-to-merge

[FraserGreenroyd]

@BHoMBot check ready-to-merge

[bhombot-ci]

@FraserGreenroyd to confirm, the following actions are now queued:

• check ready-to-merge