8000 Visualization for the fabric codebase by ivanmilevtues · Pull Request #2338 · fabric/fabric · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Visualization for the fabric codebase #2338

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and 8000 privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ivanmilevtues
Copy link

In this PR, I have generated a high-level diagram for the fabric project. You can see how it looks when Github renders is here: https://github.com/CodeBoarding/GeneratedOnBoardings/blob/main/fabric/on_boarding.md

Me and a friend are trying to make it easier for maintainers and developers to get to know new codebases. And we think visualization is the way to go, that is why we are working on these diagrams. We think that the best way to start is via getting a high-level knowledge first and then dig deeper (into each component). We generate these diagrams by combining Static analysis and LLMs and this way allowing for multple generations which we hope will keep the whole document always up-to-date (probably via workflow automation?).

I would love to hear your opinion on the matter, any feedback is more than welcome!

Full disclosure: we're trying to turn this into a startup, but we're still in a very early stage and figuring out what will actually be useful for people.

@davidjmemmett
Copy link
Contributor

I've had a very quick glance at what you've submitted & I'm not sure what you're trying to accomplish here. It looks like the typical rubbish we've all come to expect from LLM-based content generators; barely anything in these changes bares any semblance of reality.

From my perspective, these changes proposed don't serve any purpose other than to misinform and cause confusion.

@ivanmilevtues
Copy link
Author
ivanmilevtues commented Jun 7, 2025

Thank you for the answer @davidjmemmett!

The aim is to make a nice high-level diagram which represents the codebase, from your answer it seems like we are missing the point for your repository. Thank you so much for the input, I will take a close look into it and will try to adress the issues.

I wonder in general turms what do you think about a diagram driven documentation? Do you like the idea of having diagrams or you prefer better just a text document! Curious to catch your opinion on how you would go about on-boarding, as in the end of the day I don't want to spam people, but build something that people are happy to merge and use!

@davidjmemmett
Copy link
Contributor

In my opinion, you're wasting your time here.

If any such diagrams were ever required, they'd be crafted by developers who are familiar with the codebase, guided by our own experiences & using freely-available open-source tooling.

@ivanmilevtues
Copy link
Author

Thanks for sharing, I think we will continue working on this as we already got a merge request accepted in an atlassian repository: atlassian-api/atlassian-python-api#1543

This said I see your points and understand that it is not for everyone especially in the current state!

@davidjmemmett
Copy link
Contributor

I'm afraid you've misled yourself there too, that's a repository in a community organisation, which is not part of Atlassian.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0