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

Use value to check for duplicate metadata property in the visualization graph #30

Merged
merged 1 commit into from
Oct 26, 2024

Conversation

kohlerca
Copy link
Collaborator

When building visualization graphs (with NetworkX), there is a check to avoid inserting ambiguous metadata in the nodes (if different objects exist with the same identifiers). However, when merging multiple graphs for visualization, the RDF description of the same data object may exist in both RDF files. Therefore, the graph visualization would fail because the check did not consider the attribute's value. This fixes the behavior by throwing an exception only when inserting the same attribute with different values in an existing node where that metadata attribute is already defined.

@kohlerca kohlerca merged commit 5d1e8ef into main Oct 26, 2024
6 checks passed
@kohlerca kohlerca added the bugfix Fix for an identified bug label Oct 26, 2024
@kohlerca kohlerca deleted the fix/node_duplicate_metadata branch October 26, 2024 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix Fix for an identified bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant