the variable is called ```taxa_summary_fp``` but it is used (e.g., on line 87) as an open file.  I would rename the variable ```taxa_summary```, and add a note about what the input type is to the docstring.  Note that in some languages the ```_fp``` suffix means "file pointer," but we generally use it to mean "file path"