Skip to content

[prep CDF-24745] 🤵Canvas API #1676

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

Open
wants to merge 5 commits into
base: canvas-data-classes-part2
Choose a base branch
from

Conversation

doctrino
Copy link
Collaborator

@doctrino doctrino commented Jun 26, 2025

Description

Stacked on #1674 #1675

Adds a thin wrapper around the instance API for Canvas retrieval.

Motivation for adding them support migration of Canvas from asset-centric to data modeling.

Changelog

  • Patch
  • Minor
  • Skip

Copy link

github-actions bot commented Jun 26, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
20043 15745 79% 60% 🟢

New Files

File Coverage Status
cognite_toolkit/_cdf_tk/client/api/canvas.py 97% 🟢
TOTAL 97% 🟢

Modified Files

File Coverage Status
cognite_toolkit/_cdf_tk/client/_toolkit_client.py 85% 🟢
cognite_toolkit/_cdf_tk/client/data_classes/canvas.py 97% 🟢
cognite_toolkit/_cdf_tk/client/testing.py 100% 🟢
TOTAL 94% 🟢

updated for commit: 23496fc by action🐍

@doctrino doctrino changed the title refactor; canvas api [prep CDF-24745] 🤵Canvas API Jun 26, 2025
@doctrino doctrino marked this pull request as ready for review June 26, 2025 09:31
@doctrino doctrino requested a review from a team as a code owner June 26, 2025 09:31
@ronpal ronpal added the waiting-for-risk-review Waiting for a member of the risk review team to take an action label Jun 26, 2025
@doctrino doctrino force-pushed the canvas-data-classes-part2 branch from 1cdffe1 to f7bec1a Compare June 27, 2025 07:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
waiting-for-risk-review Waiting for a member of the risk review team to take an action
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants