adsabs / ADSIngestParser

Curation parser library
MIT License
0 stars 7 forks source link

jats 'affiliation' and 'relatedto' bugfixes #73

Closed mugdhapolimera closed 9 months ago

mugdhapolimera commented 10 months ago

Fix for poor affiliation parsing in some cases. See https://github.com/adsabs/ADSIngestParser/issues/54

Also bugfix for "relatedto" property -- it is now initialized as a list in jats.py

codecov-commenter commented 10 months ago

Codecov Report

Attention: 1 lines in your changes are missing coverage. Please review.

Comparison is base (7de9dde) 89.06% compared to head (e665e4e) 88.91%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #73 +/- ## ========================================== - Coverage 89.06% 88.91% -0.16% ========================================== Files 24 24 Lines 2589 2590 +1 ========================================== - Hits 2306 2303 -3 - Misses 283 287 +4 ``` | [Files](https://app.codecov.io/gh/adsabs/ADSIngestParser/pull/73?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=adsabs) | Coverage Δ | | |---|---|---| | [tests/test\_jats.py](https://app.codecov.io/gh/adsabs/ADSIngestParser/pull/73?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=adsabs#diff-dGVzdHMvdGVzdF9qYXRzLnB5) | `95.06% <ø> (ø)` | | | [adsingestp/parsers/jats.py](https://app.codecov.io/gh/adsabs/ADSIngestParser/pull/73?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=adsabs#diff-YWRzaW5nZXN0cC9wYXJzZXJzL2phdHMucHk=) | `92.02% <75.00%> (-0.52%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.