BHoM / Grasshopper_UI

Tools for Grasshopper
GNU Lesser General Public License v3.0
15 stars 4 forks source link

Add ability to get unique Canvas IDs for analytics #664

Closed FraserGreenroyd closed 1 year ago

FraserGreenroyd commented 1 year ago

NOTE: Depends on

https://github.com/BHoM/BHoM_UI/pull/429

Issues addressed by this PR

Closes #

Test files

Changelog

Additional comments

When trying to do project analytics per canvas, the doc.DocumentID wasn't unique between canvases, so doing a new canvas or loading in a saved script would give the same DocumentID. Instead, there is a unique RuntimeID available instead, but that is just a ulong rather than a more unique GUID, and resets each time you load Grasshopper. Therefore, the decision has been made to use that to generate unique IDs per canvas for the analytics, rather than the DocumentID.

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 1 requests in the queue ahead of you.
FraserGreenroyd commented 1 year ago

@BHoMBot check versioning

bhombot-ci[bot] commented 1 year ago
@FraserGreenroyd to confirm, the following actions are now queued: - check `versioning` There are 9 requests in the queue ahead of you.
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 12 requests in the queue ahead of you.
bhombot-ci[bot] commented 1 year ago
The check `versioning` has already been run previously and recorded as a successful check. This check has not been run again at this time.
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 ready-to-merge @BHoMBot check copyright-compliance

bhombot-ci[bot] commented 1 year ago
@FraserGreenroyd to confirm, the following actions are now queued: - check `ready-to-merge` - check `copyright-compliance` There are 27 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 copyright-compliance. 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. `8987860273`
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 documentation-compliance. 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. `8987115997`
FraserGreenroyd commented 1 year ago

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

FraserGreenroyd commented 1 year ago

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

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