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

[ENH] PubMed - add language to corpus #927

Merged
merged 2 commits into from
Feb 6, 2023

Conversation

PrimozGodec
Copy link
Collaborator

@PrimozGodec PrimozGodec commented Jan 11, 2023

Derived from #874

Description of changes

Add language to the corpus. Always add English since PubMed only index the English version of articles.

Includes
  • Code changes
  • Tests
  • Documentation

@PrimozGodec PrimozGodec marked this pull request as draft January 11, 2023 08:58
@PrimozGodec PrimozGodec marked this pull request as ready for review January 17, 2023 10:11
@codecov-commenter
Copy link

Codecov Report

Merging #927 (1b5c4ea) into master (25952ea) will increase coverage by 0.01%.
The diff coverage is n/a.

❗ Current head 1b5c4ea differs from pull request most recent head 40304a2. Consider uploading reports for the commit 40304a2 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #927      +/-   ##
==========================================
+ Coverage   77.53%   77.55%   +0.01%     
==========================================
  Files          86       86              
  Lines       12261    12261              
  Branches     1608     1608              
==========================================
+ Hits         9507     9509       +2     
+ Misses       2455     2454       -1     
+ Partials      299      298       -1     

@PrimozGodec PrimozGodec changed the title PubMed - add language to corpus [ENH] PubMed - add language to corpus Jan 17, 2023
@VesnaT VesnaT merged commit c827343 into biolab:master Feb 6, 2023
@PrimozGodec PrimozGodec deleted the lagdetect-pubmed branch February 6, 2023 07:53
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.

3 participants