Fixes #310 by Edited the ConditionalMapping Class issue number 310#443
Open
pathi108 wants to merge 1 commit intodbpedia:masterfrom
pathi108:fixConditionalMappingIssue
Open
Fixes #310 by Edited the ConditionalMapping Class issue number 310#443pathi108 wants to merge 1 commit intodbpedia:masterfrom pathi108:fixConditionalMappingIssue
pathi108 wants to merge 1 commit intodbpedia:masterfrom
pathi108:fixConditionalMappingIssue
Conversation
-made tempory graph for each matching conditional mapping type -merged all the graphs with the default mapping and return the final graph
Member
|
Thanks @pathi108, can you explain in more detail what this PR changes? E.g. quads/graph with the previous code vs the your code? |
Author
In my code the I will check for all cases(line 3 and line 5) in the conditional mapping whether the tempelate node will match with the condition if it so it will extract data using that conditon in the mapping and create a tempry grap(line 7)h .then all these tempory graphs will be added to one graph as a sequence. (line 8).then if atleast one conditon maps it will create get the instance uri and extractdata using the default mapping and append them to the end of the graph and return the graph |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
-made tempory graph for each matching conditional mapping type
-merged all the graphs with the default mapping and return the final graph