New issue
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
Fix rms closest bug #69
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What was the bug? That you could pass in a database with no overlap and no error would be raised?
The closest pathway didn't necessarily have any data for the follower, in which case nothing is actually infilled. |
Co-Authored-By: Zeb Nicholls <zebedee.nicholls@climate-energy-college.org>
4f57054
to
0bc1558
Compare
Co-Authored-By: Zeb Nicholls <zebedee.nicholls@climate-energy-college.org>
lgtm, will leave for @jkikstra to finalise |
Turns out @jkikstra is super-busy so maybe you can ok it @znicholls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm (one v minor suggestion) thanks @Rlamboll !
src/silicone/multiple_infillers/infill_all_required_emissions_for_openscm.py
Outdated
Show resolved
Hide resolved
…for_openscm.py Co-Authored-By: Zeb Nicholls <zebedee.nicholls@climate-energy-college.org>
Pull request
Please confirm that this pull request has done the following:
CHANGELOG.rst
added (waiting to avoid merge conflicts)Adding to CHANGELOG.rst
Please add a single line in the changelog notes similar to one of the following: