Skip to content
Permalink
Browse files

Do not modify the data attached to nodes in the tree during frequency…

… estimation

Fixes another bug with frequency estimation caused by a mismatch in the casing
of attributes attached to the nodes in the tree, weight attributes in the
weights JSON, and what the estimation code originally expected.
  • Loading branch information
huddlej committed Dec 17, 2019
1 parent 6e006b9 commit f7bff49fbfebc2ca345cc81a43258310457fbe3d
Showing with 1 addition and 1 deletion.
  1. +1 −1 augur/frequency_estimators.py
@@ -1166,7 +1166,7 @@ def estimate(self, tree):
# Find tips with the current weight attribute.
tips = [(tip.name, tip.attr["num_date"])
for tip in tree.get_terminals()
if tip.attr[self.weights_attribute].lower() == weight_key and self.tip_passes_filters(tip)]
if tip.attr[self.weights_attribute] == weight_key and self.tip_passes_filters(tip)]
frequencies.update(self.estimate_tip_frequencies_to_proportion(tips, proportion))
else:
tips = [(tip.name, tip.attr["num_date"])

0 comments on commit f7bff49

Please sign in to comment.
You can’t perform that action at this time.