Make Impact.impact_at_reg support all zero impacts#773
Conversation
timschmi95
left a comment
There was a problem hiding this comment.
The code looks good to me. I'm wondering if there is a use case to calculate imp_at_reg if all the values are zero? If not, one could also throw a different error (or warning) saying that all impacts are zero if self.imp_mat.nnz == 0 .
|
When calibrating impact functions (see #692), one usually aggregates impacts in space, and One has to distinguish two cases.
|
|
Ah yes, calibration is a good example where all impacts can be zero. And I guess if user has 0 impacts and calculates their 0 impacts for each region it doesn't neccesarily require a warning, although it's not a very interesting calcuculation^^ |
|
Thanks for the review and discussion, @timschmi95! |
Changes proposed in this PR:
PR Author Checklist
develop)PR Reviewer Checklist