Structure_Engine: Clean up ToBeRemoved methods

[IsakNaslundBh]

trafficstars

NOTE: Depends on

Issues addressed by this PR

Closes #2736

Test files

Changelog

Additional comments

[IsakNaslundBh]

@BHoMBot check installer

Running to check more toolkits affected by the change

[bhombot-ci[bot]]

@IsakNaslundBh to confirm, the following checks are now queued:

• installer

[IsakNaslundBh]

@BHoMBot check installer

Running to check more toolkits affected by the change

[bhombot-ci[bot]]

@IsakNaslundBh to confirm, the following checks are now queued:

• installer

[IsakNaslundBh]

@BHoMBot check installer

[bhombot-ci[bot]]

@IsakNaslundBh to confirm, the following checks are now queued:

• installer

[IsakNaslundBh]

@BHoMBot check versioning

[bhombot-ci[bot]]

@IsakNaslundBh to confirm, the following checks are now queued:

• versioning

[IsakNaslundBh]

@BHoMBot check compliance

[bhombot-ci[bot]]

@IsakNaslundBh to confirm, the following checks are now queued:

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

There are 5 requests in the queue ahead of you.

[IsakNaslundBh]

@BHoMBot check required

[bhombot-ci[bot]]

@IsakNaslundBh to confirm, the following checks are now queued:

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

[bhombot-ci[bot]]

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[bot]]

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.

[FraserGreenroyd]

@IsakNaslundBh what's the sitrep on this PR then? 😄

[IsakNaslundBh]

@BHoMBot check versioning

[bhombot-ci[bot]]

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

• check versioning

[IsakNaslundBh]

@BHoMBot check versioning

[bhombot-ci[bot]]

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

• check versioning

[IsakNaslundBh]

@BHoMBot check compliance

[bhombot-ci[bot]]

@IsakNaslundBh 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

There are 7 requests in the queue ahead of you.

[IsakNaslundBh]

@BHoMBot check core @BHoMBot check null-handling @BHoMBot check serialisation @BHoMBot check unit-tests

[bhombot-ci[bot]]

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

• check core
• check null-handling
• check serialisation
• check unit-tests

There are 5 requests in the queue ahead of you.

[bhombot-ci[bot]]

@IsakNaslundBh just to let you know, I have provided a check-installer 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 @IsakNaslundBh on Robot_Toolkit

[IsakNaslundBh]

I Have reviewed the failing UnitTest, which is due to changes of now defaulting orientation of nodes to null rather than world XY.

This change can be deemed fine, why the UnitTest safely can be updated. WIll do this and push to this PR.

[IsakNaslundBh]

@BHoMBot check unit-tests

[bhombot-ci[bot]]

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

• check unit-tests

[IsakNaslundBh]

@BHoMBot check required

[bhombot-ci[bot]]

@IsakNaslundBh 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

There are 33 requests in the queue ahead of you.

[IsakNaslundBh]

@BHoMBot check copyright-compliance @BHoMBot check dataset-compliance