Fabian Brandt-Tumescheit
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?