Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat(snomed): introduce NamespaceIdProvider to extract namespace values. #920

Merged
merged 6 commits into from
Oct 15, 2021

Conversation

cmark
Copy link
Member

@cmark cmark commented Oct 13, 2021

...from a collection of namespaceConceptIds

Introduce namespaceConceptId in SNOMED CT component create endpoints to
support creation of SNOMED CT content by specifying the
namespaceConceptId as opposed to the actual namespace value. The system
will be able to extract this information from the namespace concept and
use it to generate the new IDs with the correct namespace.

...from a collection of namespaceConceptIds

Introduce namespaceConceptId in SNOMED CT component create endpoints to
support creation of SNOMED CT content by specifying the
namespaceConceptId as opposed to the actual namespace value. The system
will be able to extract this information from the namespace concept and
use it to generate the new IDs with the correct namespace.
@cmark cmark requested review from apeteri and nagyo October 13, 2021 15:19
@cmark cmark self-assigned this Oct 13, 2021
cmark added 3 commits October 14, 2021 09:21
feature/support-namespace-concept-ids-in-create-api

Conflicts:
	snomed/com.b2international.snowowl.snomed.datastore/src/com/b2international/snowowl/snomed/core/SnomedPlugin.java
...id generation strategy, so it can be reused easily.
@codecov-commenter
Copy link

codecov-commenter commented Oct 14, 2021

Codecov Report

Merging #920 (5c731da) into 8.x (98bf761) will increase coverage by 0.55%.
The diff coverage is 52.23%.

Impacted file tree graph

@@             Coverage Diff              @@
##                8.x     #920      +/-   ##
============================================
+ Coverage     62.73%   63.28%   +0.55%     
+ Complexity    11684    11459     -225     
============================================
  Files          1727     1675      -52     
  Lines         55831    54314    -1517     
  Branches       5305     5077     -228     
============================================
- Hits          35023    34373     -650     
+ Misses        18586    17780     -806     
+ Partials       2222     2161      -61     
Impacted Files Coverage Δ
...l/snomed/core/rest/SnomedRf2ExportRestService.java 88.46% <ø> (ø)
...core/rest/domain/SnomedRf2ExportConfiguration.java 69.56% <ø> (ø)
...snowowl/snomed/core/domain/ConstantIdStrategy.java 57.14% <0.00%> (-9.53%) ⬇️
...snomed/core/domain/NamespaceConceptIdStrategy.java 0.00% <0.00%> (ø)
...nowowl/snomed/core/domain/NamespaceIdStrategy.java 66.66% <0.00%> (-13.34%) ⬇️
...e/request/SnomedComponentCreateRequestBuilder.java 85.71% <ø> (ø)
...owowl/snomed/datastore/request/SnomedRequests.java 89.47% <ø> (+9.19%) ⬆️
.../snowowl/snomed/reasoner/SnomedReasonerPlugin.java 94.11% <ø> (-0.33%) ⬇️
.../rest/domain/AbstractSnomedComponentRestInput.java 75.86% <12.50%> (-24.14%) ⬇️
...al/snowowl/snomed/datastore/request/IdRequest.java 85.95% <59.09%> (-6.21%) ⬇️
... and 15 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2f5d45a...5c731da. Read the comment docs.

@cmark cmark marked this pull request as ready for review October 14, 2021 07:53
Copy link
Member

@apeteri apeteri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

Copy link
Member

@nagyo nagyo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@cmark cmark merged commit 8f9daf6 into 8.x Oct 15, 2021
@cmark cmark deleted the feature/support-namespace-concept-ids-in-create-api branch October 15, 2021 08:13
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants