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

$translate operation implementation for remote terminology #3552

Merged
merged 11 commits into from
Apr 25, 2022

Conversation

lilamikalson-smilecdr
Copy link
Contributor

@lilamikalson-smilecdr lilamikalson-smilecdr commented Apr 19, 2022

$translate operation implementation for remote terminology

fixes #3442

@codecov
Copy link

codecov bot commented Apr 19, 2022

Codecov Report

Merging #3552 (8a04fc7) into master (c29bb46) will increase coverage by 0.04%.
The diff coverage is 92.63%.

@@             Coverage Diff              @@
##             master    #3552      +/-   ##
============================================
+ Coverage     83.00%   83.05%   +0.04%     
- Complexity    21378    21457      +79     
============================================
  Files          1424     1426       +2     
  Lines         77032    77325     +293     
  Branches      11384    11419      +35     
============================================
+ Hits          63942    64219     +277     
- Misses         8705     8717      +12     
- Partials       4385     4389       +4     
Impacted Files Coverage Δ
...a/uhn/fhir/jpa/term/TermConceptMappingSvcImpl.java 91.77% <66.66%> (-0.44%) ⬇️
...a/uhn/fhir/context/support/IValidationSupport.java 86.14% <83.33%> (-0.42%) ⬇️
...ort/RemoteTerminologyServiceValidationSupport.java 84.87% <94.52%> (+3.22%) ⬆️
.../jpa/dao/dstu3/FhirResourceDaoConceptMapDstu3.java 85.71% <100.00%> (+5.71%) ⬆️
...n/fhir/jpa/dao/r4/FhirResourceDaoConceptMapR4.java 100.00% <100.00%> (+8.33%) ⬆️
...n/fhir/jpa/dao/r5/FhirResourceDaoConceptMapR5.java 75.00% <100.00%> (+5.76%) ⬆️
...tor/ResponseTerminologyTranslationInterceptor.java 96.92% <100.00%> (+1.92%) ⬆️
.../ca/uhn/fhir/jpa/api/model/TranslationRequest.java 96.70% <100.00%> (+12.32%) ⬆️
...h/processor/GoldenResourceAnnotatingProcessor.java 76.74% <0.00%> (-4.84%) ⬇️
...r/UserRequestRetryVersionConflictsInterceptor.java 83.33% <0.00%> (-4.17%) ⬇️
... and 32 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 c29bb46...8a04fc7. Read the comment docs.

@lilamikalson-smilecdr lilamikalson-smilecdr merged commit 68c8523 into master Apr 25, 2022
@lilamikalson-smilecdr lilamikalson-smilecdr deleted the 3442-translate-op-remote-terminology branch April 25, 2022 19:37
michaelabuckley added a commit that referenced this pull request Apr 27, 2022
* master:
  Add New Index On HFJ_RESOURCE For $reindex Operation (#3536)
  $translate operation implementation for remote terminology (#3552)
  3558 bulk export premature deletion (#3562)
  3524 export poll status returning the wrong partition in the url (#3526)
  3547 Interceptor hook for client-assigned IDs (#3559)
  Support forward references in Group Bulk Export (#3558)
  Freetext search query builder consolidation (#3556)
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.

Provide RemoteTerminologyServiceValidationSupport Implementation For Operation $translate
2 participants