ZigRazor / CXXGraph

Header-Only C++ Library for Graph Representation and Algorithms
https://zigrazor.github.io/CXXGraph/
Mozilla Public License 2.0
478 stars 116 forks source link

Algorithm to find Articulations points #235

Open ZigRazor opened 2 years ago

ZigRazor commented 2 years ago

Please add an algorithm to find articulation points of a graph

RachitGarg-12 commented 2 years ago

hey @ZigRazor please assign me this issue

ZigRazor commented 2 years ago

Yes! @RachitGarg-12

ZigRazor commented 2 years ago

@RachitGarg-12 are you working on it?

pradkrish commented 1 year ago

is this still available? I am happy to work on this.

ZigRazor commented 1 year ago

Yes, is still available, I assign it to you @pradkrish !

pradkrish commented 1 year ago

Thanks. There seem to be two different approaches. A simple approach where vertices are removed one by one and see if removal of a vertex causes disconnected graph and another approach being Tarjan's algorithm. Which one do you prefer?

It's also possible to have both approaches, one can be called articulation_points_simple() and another one called Tarjan().

ZigRazor commented 1 year ago

Taranto algorithm should be already implemented

pradkrish commented 1 year ago

Yes, Tarjan's can be used to find both Strongly connected components and articulation points. In that case, do you think it's best to first work on Tarjan's and then use that implementation for finding articulation points?

ZigRazor commented 1 year ago

Yes, but I think for completness the best way is implement both algorithms

pradkrish commented 1 year ago

Not sure what you mean by both here? :) Helpful if you can spell it out.

By both, did you mean both implementations for finding articulation points, that is, one using a simple method and another using Tarjans?

Or by both, did you mean using Tarjan's algorithm for finding Articulation Points and Strongly connected components?

ZigRazor commented 1 year ago

Both for me means implementation with simple method and implementation with tarjan algorithm

ZigRazor commented 1 year ago

@pradkrish any news?

ZigRazor commented 10 months ago

De-assigned due to inactivity