search

cognitedata / cognite-sdk-python

Cognite Python SDK
https://cognite-sdk-python.readthedocs-hosted.com/
Apache License 2.0
76 stars 27 forks source link

Nitpick, docfixes and missing filters [part 1] #1840

Closed haakonvt closed 2 months ago

haakonvt commented 2 months ago

Not adding a release before part 2 (or 3), hence leaving the changelog empty.

Can be reviewed commit-by-commit

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 70.00000% with 12 lines in your changes missing coverage. Please review.

Project coverage is 92.46%. Comparing base (2aa4995) to head (735480e). Report is 3 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1840 +/- ## ========================================== - Coverage 92.46% 92.46% -0.01% ========================================== Files 124 124 Lines 18445 18440 -5 ========================================== - Hits 17056 17051 -5 Misses 1389 1389 ``` | [Files](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1840?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata) | Coverage Δ | | |---|---|---| | [cognite/client/\_api/data\_modeling/instances.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1840?src=pr&el=tree&filepath=cognite%2Fclient%2F_api%2Fdata_modeling%2Finstances.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvX2FwaS9kYXRhX21vZGVsaW5nL2luc3RhbmNlcy5weQ==) | `87.62% <ø> (ø)` | | | [...ient/data\_classes/data\_modeling/typed\_instances.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1840?src=pr&el=tree&filepath=cognite%2Fclient%2Fdata_classes%2Fdata_modeling%2Ftyped_instances.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvZGF0YV9jbGFzc2VzL2RhdGFfbW9kZWxpbmcvdHlwZWRfaW5zdGFuY2VzLnB5) | `92.00% <ø> (ø)` | | | [cognite/client/data\_classes/time\_series.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1840?src=pr&el=tree&filepath=cognite%2Fclient%2Fdata_classes%2Ftime_series.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvZGF0YV9jbGFzc2VzL3RpbWVfc2VyaWVzLnB5) | `96.00% <ø> (+0.06%)` | :arrow_up: | | [cognite/client/utils/\_auxiliary.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1840?src=pr&el=tree&filepath=cognite%2Fclient%2Futils%2F_auxiliary.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvdXRpbHMvX2F1eGlsaWFyeS5weQ==) | `91.91% <100.00%> (+0.12%)` | :arrow_up: | | [cognite/client/utils/\_identifier.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1840?src=pr&el=tree&filepath=cognite%2Fclient%2Futils%2F_identifier.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvdXRpbHMvX2lkZW50aWZpZXIucHk=) | `90.45% <100.00%> (ø)` | | | [cognite/client/\_api/datapoints\_subscriptions.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1840?src=pr&el=tree&filepath=cognite%2Fclient%2F_api%2Fdatapoints_subscriptions.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvX2FwaS9kYXRhcG9pbnRzX3N1YnNjcmlwdGlvbnMucHk=) | `95.65% <0.00%> (ø)` | | | [cognite/client/\_api/extractionpipelines.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1840?src=pr&el=tree&filepath=cognite%2Fclient%2F_api%2Fextractionpipelines.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvX2FwaS9leHRyYWN0aW9ucGlwZWxpbmVzLnB5) | `97.70% <0.00%> (ø)` | | | [cognite/client/\_api/transformations/\_\_init\_\_.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1840?src=pr&el=tree&filepath=cognite%2Fclient%2F_api%2Ftransformations%2F__init__.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvX2FwaS90cmFuc2Zvcm1hdGlvbnMvX19pbml0X18ucHk=) | `77.14% <50.00%> (+5.34%)` | :arrow_up: | | [...gnite/client/\_api/transformations/notifications.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1840?src=pr&el=tree&filepath=cognite%2Fclient%2F_api%2Ftransformations%2Fnotifications.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvX2FwaS90cmFuc2Zvcm1hdGlvbnMvbm90aWZpY2F0aW9ucy5weQ==) | `90.62% <0.00%> (ø)` | | | [cognite/client/\_api/transformations/schedules.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1840?src=pr&el=tree&filepath=cognite%2Fclient%2F_api%2Ftransformations%2Fschedules.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvX2FwaS90cmFuc2Zvcm1hdGlvbnMvc2NoZWR1bGVzLnB5) | `96.00% <0.00%> (ø)` | | | ... and [3 more](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1840?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata) | | ... and [5 files with indirect coverage changes](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1840/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata)