BHoM / Python_Toolkit

GNU Lesser General Public License v3.0
3 stars 2 forks source link

Enable Analytics #87

Closed tg359 closed 1 year ago

tg359 commented 1 year ago

Issues addressed by this PR

Closes #65

Adds method usage capturing via method wrappers for BHoM analytics.

Test files

Call any "@analytics" decorated code in this or downstream toolkits to see metadata capture.

e.g.:

>>> "C:\ProgramData\BHoM\Extensions\PythonEnvironments\LadybugTools_Toolkit\Scripts\python.exe" -c "from ladybugtools_toolkit.helpers.cardinality import cardinality;print(cardinality(6))"
...
# see "C:\ProgramData\BHoM\Logs" for newly created log file

Changelog

Additional comments

FraserGreenroyd commented 1 year ago

@BHoMBot check installer

bhombot-ci[bot] commented 1 year ago
@FraserGreenroyd to confirm, the following actions are now queued: - check `installer` There are 8 requests in the queue ahead of you.
FraserGreenroyd commented 1 year ago

@BHoMBot check installer

bhombot-ci[bot] commented 1 year ago
@FraserGreenroyd to confirm, the following actions are now queued: - check `installer` There are 4 requests in the queue ahead of you.
bhombot-ci[bot] commented 1 year ago
@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 LadybugTools_Toolkit
FraserGreenroyd commented 1 year ago

@BHoMBot check required

bhombot-ci[bot] commented 1 year ago
@FraserGreenroyd 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 17 requests in the queue ahead of you.
bhombot-ci[bot] commented 1 year ago
The check `installer` has already been run previously and recorded as a successful check. This check has not been run again at this time.
FraserGreenroyd commented 1 year ago

@BHoMBot check copyright-compliance

bhombot-ci[bot] commented 1 year ago
@FraserGreenroyd to confirm, the following actions are now queued: - check `copyright-compliance`
FraserGreenroyd commented 1 year ago

@BHoMBot check ready-to-merge

bhombot-ci[bot] commented 1 year ago
@FraserGreenroyd to confirm, the following actions are now queued: - check `ready-to-merge` There are 1 requests in the queue ahead of you.
bhombot-ci[bot] commented 1 year ago
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 ready-to-merge. 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. `8475846325`
bhombot-ci[bot] commented 1 year ago
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 ready-to-merge. 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. `8475846325`
FraserGreenroyd commented 1 year ago

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

bhombot-ci[bot] commented 1 year ago
@FraserGreenroyd I have now provided a passing check on reference `8475846325` as requested.