Skip to content
This repository has been archived by the owner on Jan 29, 2024. It is now read-only.

bbs_database add does not put the abstract in the sentences table #579

Open
jankrepl opened this issue Feb 14, 2022 · 0 comments · May be fixed by #593
Open

bbs_database add does not put the abstract in the sentences table #579

jankrepl opened this issue Feb 14, 2022 · 0 comments · May be fixed by #593
Labels
🐛 bug fix Something isn't working

Comments

@jankrepl
Copy link
Contributor

jankrepl commented Feb 14, 2022 •
edited
Loading

🐛 Bug description

Currently, the add command does not put the abstract in the sentences table. One problematic implication of this behavior is that adding pubmed articles will always result in an error. Or in other words, we don't extract any text from pubmed articles.

Search/src/bluesearch/entrypoint/database/add.py

Lines 175 to 176 in e2704e2

if not sentence_mappings:
raise RuntimeWarning(f"No sentence was extracted from '{parsed_path}'!")

@jankrepl jankrepl added the 🐛 bug fix Something isn't working label Feb 14, 2022
@EmilieDel EmilieDel linked a pull request Mar 7, 2022 that will close this issue
Draft
6 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
🐛 bug fix Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Integrate abstract into sentences table BlueBrain/Search
1 participant
@jankrepl