External comfort processing vastly improved and analytics implemented

[tg359]

NOTE: Depends on

BHoM/Versioning_Toolkit#207

Issues addressed by this PR

Closes #67
Closes #63
Closes #54
Closes #68

Many things changed. Most notably the inclusion of good analytics which don't add overhead to the processing methods.

Test files

Prior to testing you'll need to run the InstallPythonEnv_LBT method to set-up the Python environment. Afterwards you can compile the code which will move all the Python files where tehy need to go. After this you can also run the unit tests for the Python code by calling this command:

"C:\ProgramData\BHoM\Extensions\PythonEnvironments\LadybugTools_Toolkit\Scripts\python.exe" -m pytest "C:\ProgramData\BHoM\Extensions\PythonCode\LadybugTools_Toolkit\src\tests"

LBTToolkit_66.zip

Changelog

Additional comments

[tg359]

@BHoMBot check copyright-compliance

[bhombot-ci]

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

• check copyright-compliance

[tg359]

@jamesramsden-bh I'll wait until we have a look at this together before running any bot checks, in-case any changes need to be made beforehand.

[bhombot-ci]

@tg359 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 @FraserGreenroyd on Versioning_Toolkit

[bhombot-ci]

@tg359 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 @FraserGreenroyd on Versioning_Toolkit

[jamesramsden-bh]

Individual files not reviewed, but tested functionality as per PR description. GH script and unit tests all passing on my machine.

[jamesramsden-bh]

@BHoMBot check required

[bhombot-ci]

@jamesramsden-bh 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

[jamesramsden-bh]

@BHoMBot check null-handling

[bhombot-ci]

@jamesramsden-bh to confirm, the following actions are now queued:

• check null-handling

There are 12 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

There are 14 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

There are 15 requests in the queue ahead of you.

[tg359]

@BHoMBot check required

[bhombot-ci]

@tg359 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 null-handling has already been run previously and recorded as a successful check. This check has not been run again at this time.

[tg359]

@BHoMBot check copyright-compliance

[bhombot-ci]

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

• check copyright-compliance

[tg359]

@BHoMBot check ready-to-merge

[bhombot-ci]

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

• check ready-to-merge

[jamesramsden-bh]

Retested using provided GH script and unit tests, all passing.

[jamesramsden-bh]

@BHoMBot check ready-to-merge

[bhombot-ci]

@jamesramsden-bh 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

There are 4 requests in the queue ahead of you.