averbis / averbis-python-api

Conveniently access the REST API of Averbis products using Python
Apache License 2.0
12 stars 4 forks source link

Feature/103 add update textanalysis result #104

Closed UWinch closed 2 years ago

UWinch commented 2 years ago

What is in this PR

How to test Example usage:


new_process = collection.create_process(NEW_PROCESS_NAME)

process.add_text_analysis_result_to_document(cas, DOCUMENT_NAME)
process.update_text_analysis_result_for_document(update_cas, DOCUMENT_NAME)
codecov-commenter commented 2 years ago

Codecov Report

Merging #104 (118b91d) into dev (a8c907e) will increase coverage by 0.49%. The diff coverage is 98.36%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #104      +/-   ##
==========================================
+ Coverage   91.41%   91.91%   +0.49%     
==========================================
  Files          16       16              
  Lines        1863     1978     +115     
==========================================
+ Hits         1703     1818     +115     
  Misses        160      160              
Impacted Files Coverage Δ
averbis/core/_rest_client.py 86.58% <97.22%> (+1.02%) :arrow_up:
tests/test_document_collection.py 100.00% <100.00%> (ø)
tests/test_process.py 100.00% <100.00%> (ø)

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 a8c907e...118b91d. Read the comment docs.

UWinch commented 2 years ago

@reckart do you want to have another look? I opened https://github.com/averbis/averbis-python-api/issues/105 for the remaining remarks.

reckart commented 2 years ago

@UWinch there is a space missing in the commit messages between Issue and #103 which lets GitHub fail to link the commits to the issue.