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

Implement a merge function in the Diagram class #53

Open
KGergo88 opened this issue Feb 13, 2021 · 0 comments
Open

Implement a merge function in the Diagram class #53

KGergo88 opened this issue Feb 13, 2021 · 0 comments
Labels
enhancement New, or improved functionality

Comments

@KGergo88
Copy link
Owner

Description

The CMDP changes introduced a feature that already stored diagrams can be now updated with new data. At the moment it is implemented in the simplest possible way that the old diagram object will be simply replaced by the new one. This is not an optimal solution and only the updates should be copied.

Todo

  • Define different merging rules (e.g. only the new points are added, old ones are reviewed as well, full copy)
  • Implement the feature
  • Add unit tests for the feature
@KGergo88 KGergo88 added the enhancement New, or improved functionality label Feb 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New, or improved functionality
Projects
None yet
Development

No branches or pull requests

1 participant