Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.12%. Comparing base (1393f5e) to head (a7a152d).

@@ Coverage Diff @@
## master #6518 +/- ##
=========================================
 Coverage 75.12% 75.12% 
 Complexity 5578 5578 
=========================================
 Files 686 686 
 Lines 19327 19327 
 Branches 3734 3734 
=========================================
 Hits 14520 14520 
 Misses 4245 4245 
 Partials 562 562 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@alxkm whats your opinion on these flowcharts? I'm honestly more a fan of animation-gifs :]

@DenizAltunkapan, I share the same view. Personally, I feel the flowcharts can be a bit confusing in this context, since the repository is about algorithm implementations. The animation GIFs seem more intuitive and informative to me as well.

@pranjalisr, Thank you for your contribution! We really appreciate the effort you’ve put into adding the flowcharts. However, since this repository is mainly focused on algorithm implementations, the flowcharts seem less suitable here. In this context, the GIFs seem to provide clearer and more useful information.

For future contributions, it might be a good idea to open an issue first, so we can discuss the proposed changes before adding them.

@alxkm , Thank you for your response! I can add animated GIFs as well and will remove the current flowchart. Should I move forward with the animated GIFs ?

@siriak, what are your thoughts on this suggestion?

I think that we don't need gifs or flowcharts in this repository because it's specifically for Java implementations. Flowcharts are generic and can be in pseudocode, not in this repository. In any case, they should also be in Mermaid or PlantUML to be editable by other users and be correct.

I agree with @siriak that flowcharts are not specific to Java and probably don’t belong directly in this repository.
However, I could imagine something like a separate repository under the TheAlgorithms organization – for example, Algorithms-Visualized.

That repository could collect visualizations (e.g., Mermaid flowcharts or PlantUML diagrams and GIFs) of common algorithms in a language-agnostic way. This way, learners who benefit from visual explanations would have a central place to look, while the language repos (like Java) remain focused purely on implementations.

@alxkm @siriak would you see value in such a repo or is this not worth it?

@DenizAltunkapan, I think it could be a good idea and worth discussing with the maintainers of TheAlgorithms. One thing to keep in mind is that different contributors might make diagrams in very different styles, which could be confusing. For example, one person might want to improve an existing diagram, but because of style differences it’s not clear what’s "better." Also, one explanation might work well for some learners, while another works better for others.

But again, this is more a question for the TheAlgorithms owners.

I agree with @alxkm in that it can be tricky to agree on what is a good visualization. If done in a separate repo, we can also add diagrams. But there are still questions before we can start accepting contributions. Like how to visualize them. If learners come to the repository, they will see just code, not visualization. Maybe include the diagrams on the website? It requires a discussion in Discord.

@pranjalisr
Thanks for your effort! As mentioned earlier in the PR discussion, GIFs or flowcharts aren’t wanted in this repository. Please refer to the messages above for details.

If you’re still interested in creating such diagrams, you’d need to contact the org owners and start a discussion on the TheAlgorithms Discord.
Thank you for your understanding.