Closed martinfleis closed 6 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 97.6%. Comparing base (
4037c70
) to head (399cf39
). Report is 6 commits behind head on main.
These two functions also quite significantly change the API compared to the original. @jGaboardi can you give it a thought whether it makes sense in this way? I think it does and it is more flexible than before but it is different.
I think it makes sense, cleaner and more efficient for sure.
of which 330 ms is assign_self_weight which will be used in other functions so I'm wondering if it is worth adding a keyword self_weighted=True to skip this. In practice it is nearly negligible difference though...
Solved in 0d897cc by cheaply (10ms) checking if the graph already has self-weights and assigning only if they do not. Which means that if you pass self-weighted graph, the new implementation takes about half the time compared to the original.
Gosh I spent way too much time on this...
mean_interbuilding_distance:
Given my first attempt took 5mins, I call this a victory.
building_adjacency:
of which 330 ms is
assign_self_weight
which will be used in other functions so I'm wondering if it is worth adding a keywordself_weighted=True
to skip this. In practice it is nearly negligible difference though...These two functions also quite significantly change the API compared to the original. @jGaboardi can you give it a thought whether it makes sense in this way? I think it does and it is more flexible than before but it is different.