Fabian Brandt-Tumescheit

Results 82 comments of Fabian Brandt-Tumescheit

Makes sense. I added explanations for all ignored expressions. Also tweaked some expressions: - Removed leading/trailing underscores for template parameters. Is currently used in some places, but doesn't seem to...

Yes, it also enables the check. Maybe it is a good idea (like we did with `networkit-format`), to enable each check one-by-one in seperate PRs. This way, it would be...

Changed the scope of the PR and enabled the check for class-naming only (+ required changes in the code).

> Don't we need deprecations and aliases? @avdgrinten Good point. Most of the current changes concern private structs. Any advise whether they should also be deprecated?

Hi, thanks for reaching out. We recently merged PR #984 addressing this problem. We will either put this in a bug-fix release or in the next scheduled release in November....

Hi, thanks for reaching out. I could reproduce the error - it originates from the constructor. If you want indexed edges for the time being, create the graph without indexed...

Hi, thanks for the suggestion. We are currently working on a integration of several dynamic betweenness algorithms (see #624). Though OneNode for weighted graphs is not among them, this can...

@nahuelalmeira Did you find the time to make any progress so far? As an entry-point to begin with - your initial conclusion about BFS is correct. Basicallly a first step...

Closing this issue now. We merged a PR that improves the documentation and enables checking for multi edges when inserting edges. Feature will be part of the next release. You...

@avdgrinten Was this fix applied in the end? Can we close this issue?