uqrmaie1 / admixtools

https://uqrmaie1.github.io/admixtools
71 stars 14 forks source link

Error in satisfies_constraints(...) when using admix and event constraints with find_graphs #46

Open Tharsikav opened 11 months ago

Tharsikav commented 11 months ago

Hi, I've been getting the error below when I use both admix_constraints and event_constraints with find_graphs(). It typically appears after the constraints have been satisfied. Do you have any solutions to this?

Error in filter(., satisfies_constraints(g, nzf4, admixc, eventc)) : ℹ In argument: satisfies_constraints(g, nzf4, admixc, eventc). ℹ In row 3. Caused by error in mutate(): ℹ In argument: ok = satisfies_oneevent(...). ℹ In row 3. Caused by error in satisfies_oneevent(): ! satisfies_oneevent error

uqrmaie1 commented 11 months ago

Thanks for pointing that out! I found one edge case that could trigger this error (earlier1/later1 is a direct descendant of the root node, and earlier2/later2 is NA). That should be fixed now. If you still get this error, please let me know!

Tharsikav commented 11 months ago

Amazing, thanks so much! I'll check it out again, and get back to you if any issues.