Skip to content
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

[NEAT-799] 🥹 Change behavior of neat.to.cdf.data_model(existing='update') #1012

Merged
merged 16 commits into from
Mar 5, 2025

Conversation

doctrino
Copy link
Collaborator

@doctrino doctrino commented Feb 26, 2025

Description

Please describe the change you have made.

Bump

  • Patch
  • Minor
  • Skip

Changelog

Changed

  • When running neat.to.cdf.data_models(existing='update') neat combines the data models/views/containers with the existing. For example, if you have a data model with 3 views and then you deploy a data model with 2 new views, the resulting model will have 5 views.

Copy link

github-actions bot commented Mar 3, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
15042 12052 80% 60% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
cognite/neat/_client/_api/data_modeling_loaders.py 78% 🟢
cognite/neat/_rules/exporters/_rules2dms.py 71% 🟢
cognite/neat/_session/_to.py 77% 🟢
TOTAL 75% 🟢

updated for commit: 48297c0 by action🐍

@doctrino doctrino changed the title [NEAT-799] neat.to.cdf.data_model exisitng=merge [NEAT-799] 🥹 Change behavior of neat.to.cdf.data_model(existing='update') Mar 3, 2025
@doctrino doctrino marked this pull request as ready for review March 3, 2025 13:13
@doctrino doctrino requested a review from a team as a code owner March 3, 2025 13:13
@doctrino doctrino enabled auto-merge (squash) March 4, 2025 15:28
@doctrino doctrino disabled auto-merge March 4, 2025 15:29
@doctrino doctrino merged commit 3af1cbf into main Mar 5, 2025
8 checks passed
@doctrino doctrino deleted the patch-option-to-cdf branch March 5, 2025 13:44
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