Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds a uid to the
Node
baseclass via theuuid
module. Refer Issue #123Current implementation utilizes
uuid4()
which creates ae.g.
UUID('d5181261-c06e-4f68-82ed-8f8d4592511b')
Two implementation questions arise:
UUID
type for loki's case a reasonable choice or would something simpler, i.e. a generator of incrementing integers, be enough?Comment
class was not calling theNode
post init function? As far as git blame goes there once was a call to the base class init_ but it was removed @mlange05 (d6fe4e1).Current tests are:
Node
types are collected with dummy constructor arguments and checked if behavior is as described above