Simon Forbat

Results 7 comments of Simon Forbat

@dschult Thank you for adding the relevant label. Could you please tell me if there is anything I could do to get a review for this pull request?

@peijenburg Thank you for taking a look at this PR. This metric is not defined for any of the graph classes you mention. I will update the code and the...

@yoaocopy I updated the code as you suggested: Now only tree graphs are allowed.

@peijenburg I have now updated the code as you requested. Multigraphs and directed graphs are now excluded as the code raises an exception in those cases. The documentation is updated...

@peijenburg Thank you for your suggestions. I have now implemented both of them. Previously I did not know about the `is_tree` method.

@peijenburg Thank you! I have updated the documentation.

@dschult Thank you for your useful feedback. I have now implemented everything you suggested. In order to answer your questions: > We are building a large group of modules in...