Skip to content

9.2 kruskal’s algorithm 1#53

Merged
arademaker merged 15 commits intomainfrom
9.2-Kruskal’s-algorithm
Jul 10, 2025

Hidden character warning

The head ref may contain hidden characters: "9.2-Kruskal\u2019s-algorithm"
Merged

9.2 kruskal’s algorithm 1#53
arademaker merged 15 commits intomainfrom
9.2-Kruskal’s-algorithm

Conversation

@Darlan369
Copy link
Contributor

No description provided.

@Darlan369
Copy link
Contributor Author

Ainda não terminei, falta o Kruskal’s algorithm e tirar as implementações locais de funções de outros capitulos (faço isso para eu não ter que ficar viajando de aba em aba para entender o que está acontecendo), tudo parece estar funcionando bem.

@Darlan369
Copy link
Contributor Author

Ok, só falta o Kruskal’s algorithm

@arademaker
Copy link
Owner

implementações locais de funções de outros capitulos (faço isso para eu não ter que ficar viajando de aba em aba para entender o que está acontecendo)

isso não faz sentido. Funções de outros capítulos não devem ser redefinidas. O resultado interessante do reuso é exatamente termos um mapa de dependências como https://deepwiki.com/emap-taa-20251/fad.

Sim, parte da tarefa é entender as funções anteriormente definidas.

@Darlan369
Copy link
Contributor Author

Darlan369 commented Jul 8, 2025

Eu literalmente só copio e colo as funções não faço nenhuma mudança nelas, eu gosto de deixar as funções no arquivo que estou trabalhando por praticidade.
Eu não gosto de viajar aba a aba para ver uma função, ter que ficar dando scroll para ver uma função, mudando de aba, dando scroll para ver outra função, mudando de aba de novo e dando scroll mais uma vez, só para ver a mesma função que você tinha visto antes é muito irritante, também não gosto de dividir a tela com vários arquivos de uma vez, logo para mim é mais fácil deixar todas as funções em um arquivo enquanto eu ainda estou lidando com elas.

@Darlan369
Copy link
Contributor Author

Darlan369 commented Jul 8, 2025

Depois que eu termino uma dada função que usa uma função importado que eu copiei para ela ser uma função local no arquivo, eu apago a essa função local e importo ela, em outras palavras, não estou redefinindo nada

@Darlan369
Copy link
Contributor Author

No Update 1 o senhor pode ver que uso foldl1 e mindWith exatamente como eles são definidos no capitulo 7, eu só copiei eles no arquivo Chapter9 porque acho chato ter que ficar viajando em abas no Visual Code

@Darlan369
Copy link
Contributor Author

Ai eu só apago eles depois que usei, importo o Chapter7 e coloco Chapter7. na frente de todo foldl1 e mindWith chamado, como fiz no último commit

@Darlan369
Copy link
Contributor Author

Ok Professor @arademaker, acabei

open Chapter1 (wrap apply)
open Chapter5 (sortOn₃)

/- 9.1 Graphs and spanning trees -/
Copy link
Owner

Choose a reason for hiding this comment

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

aqui precisamos de um namespace para a seção namespace SpanningTrees

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Feito, está correto?

Copy link
Owner

Choose a reason for hiding this comment

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

Penso que os comandos open deveriam estar dentro do namespace da seção

Copy link
Owner

Choose a reason for hiding this comment

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

nao, deveriam estar dentro do namespace SpanningTrees

@Darlan369
Copy link
Contributor Author

Darlan369 commented Jul 9, 2025

Me enrolei aqui mas agora acho que foi, ACHO QUE PODE FAZER O MERGE

open Chapter1 (wrap apply)
open Chapter5 (sortOn₃)

/- 9.1 Graphs and spanning trees -/
Copy link
Owner

Choose a reason for hiding this comment

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

Penso que os comandos open deveriam estar dentro do namespace da seção

open Chapter1 (wrap apply)
open Chapter5 (sortOn₃)

/- 9.1 Graphs and spanning trees -/
Copy link
Owner

Choose a reason for hiding this comment

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

nao, deveriam estar dentro do namespace SpanningTrees

@arademaker
Copy link
Owner

o nome do seu branch contém caracteres UTF8... https://github.com/emap-taa-20251/fad/tree/9.2-Kruskal%E2%80%99s-algorithm

@arademaker arademaker merged commit 387730b into main Jul 10, 2025
2 checks passed
@arademaker arademaker deleted the 9.2-Kruskal’s-algorithm branch July 10, 2025 02:00
@arademaker
Copy link
Owner

@Darlan369, tive que resolver os conflitos localmente, seu branch não etava atualizado com o main...

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