Skip to content
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 privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update class dependency graph #126

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

Conversation

rafaelcitario
Copy link

No description provided.

Copy link
Member

@jeffque jeffque left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Obrigado pela contribuição.

Temos um repositório para gerar o gráfico de dependências, que infelizmente não gera uma imagem tão interessante e organizado quanto a imagem do upload. É interessante atualizar as coisas lá.

Em relação a imagem, a versão presente (dado hoje 2024-09-01) é um SVG, que permite melhor interação com o usuário do que o PNG que foi colocado como sugestão. Se for possível subir um SVG do grafo de dependências do curso seria mais adequado.

Outro ponto também é permitir que a imagem siga um versionamento dentro da UBL. Foi feito o apontamento para um asset solto.

Então, se for possível subir o asset como um SVG dentro dos repositórios da UBL, temos o que precisamos para aprovar o seu PR.

Se for possível também atualizar o código de geração da grade curricular dentro do https://github.com/Universidade-Livre/dependencias-aulas seria uma cereja de ouro no topo do bolo.

@rafaelcitario
Copy link
Author

Obrigado pela contribuição.

Temos um repositório para gerar o gráfico de dependências, que infelizmente não gera uma imagem tão interessante e organizado quanto a imagem do upload. É interessante atualizar as coisas lá.

Em relação a imagem, a versão presente (dado hoje 2024-09-01) é um SVG, que permite melhor interação com o usuário do que o PNG que foi colocado como sugestão. Se for possível subir um SVG do grafo de dependências do curso seria mais adequado.

Outro ponto também é permitir que a imagem siga um versionamento dentro da UBL. Foi feito o apontamento para um asset solto.

Então, se for possível subir o asset como um SVG dentro dos repositórios da UBL, temos o que precisamos para aprovar o seu PR.

Se for possível também atualizar o código de geração da grade curricular dentro do https://github.com/Universidade-Livre/dependencias-aulas seria uma cereja de ouro no topo do bolo.

Abri uma PR no repositório de gráficos de dependências com o gráfico em svg. Assim que aceito realizo a alteração na nova versão desta PR com a imagem no formato correto e versionada.

Sobre o código de geração de grade curricular, ainda não possuo os conhecimentos necessários para realizar alterações no código python no entanto vou estudar sobre e posso tentar refatorar aos poucos.

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