blab / nextflu

Real-time tracking of influenza evolution
http://nextflu.org
GNU Affero General Public License v3.0
88 stars 19 forks source link

Clade frequency mouse over is broken #45

Closed trvrb closed 9 years ago

trvrb commented 9 years ago

Not sure when this arose, but clade frequencies are no longer showing on mouseover.

rneher commented 9 years ago

it says "Error: Source data is missing a component at (0,1)!" I have had this before. did the json change?

trvrb commented 9 years ago

That's probably it. I reran the pipeline yesterday and pushed the updated json.

trvrb commented 9 years ago

But I think this means the current pipeline in master is producing incomplete json.

rneher commented 9 years ago

bernoulli_freqeuncies.py writes into genotype_frequencies.json, auspice loads frequencies.json could that be the issue?

rneher commented 9 years ago

the problem is that the tree doesn't contain a field 'pivots'. this should be a property of the root node. but it isn't. tree_util writes it, if it is there and should call the debugger if it fails.

trvrb commented 9 years ago

Thanks. I'll look into this tomorrow.

rneher commented 9 years ago

the first thing to check would be whether tree_frequencies.json has the "pivots" at the root node. If it does, the field got lost in streamlining.

rneher commented 9 years ago

should be fixed now. json export was broken by mishandling the virus_count field.

trvrb commented 9 years ago

Fixed by @rneher in c762dfe19f380fe69e4ddcf808961d16391f2b14.