Fixed several bugs and simplified non-critical objects

[tg359]

Issues addressed by this PR

Closes #154
Closes #155
Closes #156
Closes #157

Test files

Changelog

Additional comments

[tg359]

@BHoMBot check compliance

[bhombot-ci]

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

[tg359]

@BHoMBot check core

[bhombot-ci]

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

• check core

There are 7 requests in the queue ahead of you.

[tg359]

@BHoMBot check versioning

[bhombot-ci]

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

• check versioning

There are 9 requests in the queue ahead of you.

[tg359]

@BHoMBot check null-handling

[bhombot-ci]

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

• check null-handling

[tg359]

@BHoMBot check unit-tests

[bhombot-ci]

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

• check unit-tests

[tg359]

@BHoMBot check serialisation

[bhombot-ci]

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

• check serialisation

There are 1 requests in the queue ahead of you.

[Tom-Kingstone]

After a quick code review, and test in jupyter, the bugs set out in the issues are fixed. I've got a comment for future reference, but am happy for this to be merged as it is now

[FraserGreenroyd]

@BHoMBot check compliance
@BHoMBot check required

[bhombot-ci]

@FraserGreenroyd 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
• check code-compliance
• check documentation-compliance
• check project-compliance
• check core
• check null-handling
• check serialisation
• check versioning
• check installer

[FraserGreenroyd]

@BHoMBot this is a DevOps instruction. I am requesting neutral checks on: unit-tests

[bhombot-ci]

@FraserGreenroyd I have provided neutral checks to the checks requested. These checks will need to be run properly to obtain full results.

[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 installer.

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. 19445745530

[FraserGreenroyd]

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

[bhombot-ci]

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

[FraserGreenroyd]

@BHoMBot check ready-to-merge

[bhombot-ci]

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

• check ready-to-merge