Skip to content

Conversation

@evertoncolling
Copy link
Contributor

Description

Adds support for the Streams API in the Data Modeling module, enabling users to create, retrieve, list, and delete streams.

Checklist:

  • Tests added/updated.
  • Documentation updated. Documentation is generated from docstrings - these must be updated according to your change.
    If a new method has been added it should be referenced in cognite.rst in order to generate docs based on its docstring.
  • The PR title follows the Conventional Commit spec.

@codecov
Copy link

codecov bot commented Oct 24, 2025

Codecov Report

❌ Patch coverage is 98.24561% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.98%. Comparing base (f356839) to head (274b01b).
⚠️ Report is 6 commits behind head on master.

Files with missing lines Patch % Lines
cognite/client/_api/data_modeling/streams.py 96.77% 1 Missing ⚠️
...gnite/client/data_classes/data_modeling/streams.py 98.71% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           master    #2380    +/-   ##
========================================
  Coverage   90.97%   90.98%            
========================================
  Files         192      194     +2     
  Lines       26138    26256   +118     
========================================
+ Hits        23780    23889   +109     
- Misses       2358     2367     +9     
Files with missing lines Coverage Δ
cognite/client/_api/data_modeling/__init__.py 100.00% <100.00%> (ø)
cognite/client/_api_client.py 91.55% <ø> (ø)
...nite/client/data_classes/data_modeling/__init__.py 100.00% <100.00%> (ø)
cognite/client/testing.py 100.00% <100.00%> (ø)
cognite/client/_api/data_modeling/streams.py 96.77% <96.77%> (ø)
...gnite/client/data_classes/data_modeling/streams.py 98.71% <98.71%> (ø)

... and 10 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@haakonvt
Copy link
Contributor

I kindly ask we do this post v8 unless there is a very sudden need!

@evertoncolling
Copy link
Contributor Author

I kindly ask we do this post v8 unless there is a very sudden need!

No rush! But it would be nice to ship it in December or early January if possible 🙂

@haakonvt
Copy link
Contributor

I kindly ask we do this post v8 unless there is a very sudden need!

No rush! But it would be nice to ship it in December or early January if possible 🙂

You heard it here first, I promise v8 first half of November (or I'll eat my shoe)

@haakonvt
Copy link
Contributor

Btw, seen this? #2246

@evertoncolling
Copy link
Contributor Author

Btw, seen this? #2246

Yes, this one is based on that one, but focusing only on streams (plus making sure we keep single item operations on top of streams and changing the stream templates from enum to a simple string).

The idea is to get this one in, as it's simpler, and then have a second PR focused only on records.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants