-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated core model to version with required keywords and docstrings #1864
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1864 +/- ##
==========================================
- Coverage 91.06% 91.06% -0.01%
==========================================
Files 124 124
Lines 19245 19245
==========================================
- Hits 17526 17525 -1
- Misses 1719 1720 +1
|
scripts/generate_core_model.py
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is sweet!
CHANGELOG.md
Outdated
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
class CogniteDescribableNodeApply(CogniteDescribableProperties, TypedNodeApply): | ||
"""This represents the writing format of cognite describable node. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quite a lot of lower-cased cognite
:
class CogniteDescribableNodeApply(CogniteDescribableProperties, TypedNodeApply): | |
"""This represents the writing format of cognite describable node. | |
class CogniteDescribableNodeApply(CogniteDescribableProperties, TypedNodeApply): | |
"""This represents the writing format of Cognite describable node. |
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description
3D
correctly:object_3d
instead ofobject_3_d
Checklist:
If a new method has been added it should be referenced in cognite.rst in order to generate docs based on its docstring.