We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
In working on the new windows bug discovered in #353, I'm digging into bulk_statistics.py/get_statistics_from_mask. In looking at this line:
bulk_statistics.py/get_statistics_from_mask
tobac/tobac/utils/bulk_statistics.py
Line 201 in 5f096e3
it appears that there should be another loop for field? I'm not that familiar with this code though, so tagging @JuliaKukulies .
The text was updated successfully, but these errors were encountered:
Good catch! It'll be using the last field array from the loop at L185. Should be a quick fix to update to a generator, I'll do that now
field
Sorry, something went wrong.
Good catch @freemansw1 and thank you so much for taking care of this, @w-k-jones !
Closing with #368 being merged
Successfully merging a pull request may close this issue.
In working on the new windows bug discovered in #353, I'm digging into
bulk_statistics.py/get_statistics_from_mask
. In looking at this line:tobac/tobac/utils/bulk_statistics.py
Line 201 in 5f096e3
it appears that there should be another loop for field? I'm not that familiar with this code though, so tagging @JuliaKukulies .
The text was updated successfully, but these errors were encountered: