yatisht / usher

Ultrafast Sample Placement on Existing Trees
MIT License
121 stars 40 forks source link

On dev server: `Warning/Error(s): Stack overflow in jsonWritePush` #321

Closed corneliusroemer closed 1 year ago

corneliusroemer commented 1 year ago

Uploading 249 sequences to the dev/staging server, not production, i.e. dev.usher.bio, with 5000 sequences for context using full tree with GISAID sequences, I got the following error:

Warning/Error(s):
Stack overflow in jsonWritePush

image

At around 14:11 UTC, maybe you can find something in the logs.

The sequences were queried from GISAID via: China in open search field AA: Spike_N460K

image

@AngieHinrichs

AngieHinrichs commented 1 year ago

Doh! Another old "we'll never get that deep" assumption in library code blown away by SARS-CoV-2 data. (I suspect it's the bat-CoV sequences making a long branch from the root of the tree that lead to the extra depth in this case -- downsampling goes all the way back to root.) Fixed on the test site although it will take another couple weeks to propagate to the main site.