cognitedata / cognite-sdk-python

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

feat: adding load method to CogniteClient, GlobalConfig, ClientConfig, and CredentialProviders #1859

Closed nodegard closed 4 days ago

nodegard commented 1 month ago

Description

Please describe the change you have made.

Checklist:

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 90.77%. Comparing base (bdde64c) to head (b32bab9). Report is 1 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1859 +/- ## ========================================== + Coverage 90.64% 90.77% +0.13% ========================================== Files 126 126 Lines 19617 19685 +68 ========================================== + Hits 17781 17870 +89 + Misses 1836 1815 -21 ``` | [Files with missing lines](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1859?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/raw.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1859?src=pr&el=tree&filepath=cognite%2Fclient%2F_api%2Fraw.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvX2FwaS9yYXcucHk=) | `88.28% <ø> (-0.84%)` | :arrow_down: | | [cognite/client/\_cognite\_client.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1859?src=pr&el=tree&filepath=cognite%2Fclient%2F_cognite_client.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvX2NvZ25pdGVfY2xpZW50LnB5) | `93.40% <100.00%> (+0.30%)` | :arrow_up: | | [cognite/client/\_version.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1859?src=pr&el=tree&filepath=cognite%2Fclient%2F_version.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvX3ZlcnNpb24ucHk=) | `100.00% <100.00%> (ø)` | | | [cognite/client/config.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1859?src=pr&el=tree&filepath=cognite%2Fclient%2Fconfig.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvY29uZmlnLnB5) | `92.47% <100.00%> (+3.90%)` | :arrow_up: | | [cognite/client/credentials.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1859?src=pr&el=tree&filepath=cognite%2Fclient%2Fcredentials.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvY3JlZGVudGlhbHMucHk=) | `91.35% <100.00%> (+10.13%)` | :arrow_up: | | [cognite/client/data\_classes/\_base.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1859?src=pr&el=tree&filepath=cognite%2Fclient%2Fdata_classes%2F_base.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvZGF0YV9jbGFzc2VzL19iYXNlLnB5) | `92.78% <100.00%> (+0.16%)` | :arrow_up: | | [...ite/client/data\_classes/data\_modeling/instances.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1859?src=pr&el=tree&filepath=cognite%2Fclient%2Fdata_classes%2Fdata_modeling%2Finstances.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvZGF0YV9jbGFzc2VzL2RhdGFfbW9kZWxpbmcvaW5zdGFuY2VzLnB5) | `89.68% <ø> (ø)` | | | [cognite/client/data\_classes/datapoints.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1859?src=pr&el=tree&filepath=cognite%2Fclient%2Fdata_classes%2Fdatapoints.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvZGF0YV9jbGFzc2VzL2RhdGFwb2ludHMucHk=) | `91.82% <ø> (ø)` | | | [cognite/client/data\_classes/workflows.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1859?src=pr&el=tree&filepath=cognite%2Fclient%2Fdata_classes%2Fworkflows.py&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata#diff-Y29nbml0ZS9jbGllbnQvZGF0YV9jbGFzc2VzL3dvcmtmbG93cy5weQ==) | `95.15% <ø> (ø)` | | | [cognite/client/utils/\_auxiliary.py](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1859?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==) | `92.81% <100.00%> (+0.39%)` | :arrow_up: | ... and [1 file with indirect coverage changes](https://app.codecov.io/gh/cognitedata/cognite-sdk-python/pull/1859/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cognitedata)
haakonvt commented 4 days ago

Heads'up @gregertw #1892 , you may need to update load methods after this PR was merged 😄