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

#125 Create exception for empty figures and parse figure sub-points. #139

Merged
merged 3 commits into from
May 16, 2024

Conversation

nils-herrmann
Copy link
Contributor

Create exception for empty figures and parse figure sub-points. Further, create one test for every parsed field of a figure.

@nils-herrmann
Copy link
Contributor Author

The output for the reported article in the bug now looks like this:

[{'pmid': '36094679',
'pmc': '9539395',
'fig_caption': 'Aerosol delivery of sACE22.v2.4‐IgG1 alleviates lung injury and improves survival of SARS‐CoV‐2 gamma variant infected K18‐hACE2 transgenic mice \n\n',
'fig_id': 'emmm202216109-fig-0001',
'fig_label': 'Figure 1',
'fig_subpoints': [('A',
'K18‐hACE2 transgenic mice were inoculated with SARS‐CoV‐2 isolate /Japan/TY7‐503/2021 (gamma variant) at 1\u2009×\u2009104 PFU. sACE22.v2.4‐IgG1 (7.5\u2009ml at 8.3\u2009mg/ml in PBS) was delivered to the mice by a nebulizer in 25\u2009min at 12\u2009h, 48\u2009h, and 84\u2009h postinoculation. PBS was aerosol delivered as control.'),
('B, C',
'Survival (B) and weight loss (C). N\u2009=\u200910 mice for each group. The P‐value of the survival curve by the Gehan–Breslow–Wilcoxon test is shown. Error bars for mouse weight are centered on the mean and show SEM.'),
...},
...]

I noticed that the captions and sub-points contain unicode escape sequences (\u2009h, \u200910). Should we leave them?

tests/test_pubmed_oa_parser.py Outdated Show resolved Hide resolved
tests/test_pubmed_oa_parser.py Outdated Show resolved Hide resolved
pubmed_parser/pubmed_oa_parser.py Outdated Show resolved Hide resolved
pubmed_parser/pubmed_oa_parser.py Outdated Show resolved Hide resolved
@Michael-E-Rose Michael-E-Rose merged commit 0eb7114 into titipata:master May 16, 2024
1 of 2 checks passed
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.

2 participants