This PR addresses #11. It fixes the issue where for the three boolean properties pubmed_evidence, omim_evidence, refInOMIM, instead of false, Article nodes will have null.
It consists of two changed files:
fix_evidence_property.py fixes this issue on all existing Article nodes in the database, by setting all instances where the property is null to false. After this script is run, pubmed_evidence, omim_evidence, and refInOMIM is guaranteed to be defined on all Article nodes in the database, and will have value of either true or false (non-null).
update-neo4j.py has been fixed to properly define these properties when creating Article nodes.
Again, I would like to test this on the full database if possible before merging to main. The update-neo4j.py script has already been tested.
This PR addresses #11. It fixes the issue where for the three boolean properties
pubmed_evidence
,omim_evidence
,refInOMIM
, instead offalse
,Article
nodes will havenull
.It consists of two changed files:
fix_evidence_property.py
fixes this issue on all existingArticle
nodes in the database, by setting all instances where the property isnull
tofalse
. After this script is run,pubmed_evidence
,omim_evidence
, andrefInOMIM
is guaranteed to be defined on allArticle
nodes in the database, and will have value of eithertrue
orfalse
(non-null).update-neo4j.py
has been fixed to properly define these properties when creatingArticle
nodes.Again, I would like to test this on the full database if possible before merging to main. The
update-neo4j.py
script has already been tested.