How to execute modules of task 1 #14
Conversation
|
Oi @Wiltoon, obrigada por contribuir! As mudanças estão um pouco grandes, então vou precisar revisar esta contribuição por partes. Primeiro, por favor remova os arquivos de resposta do repositório. Eles são muito grandes para serem mantidos no git. Achei que o erro da region havia sido corrigido por esse PR, pode confirmar @fillipe-gsm? |
|
Entendi, vou retirar os arquivos de saida para reenviar |
|
Oi, @williamloggi, obrigado também pela contribuição! Eu tenho uma outra dúvida: no PR o diff avisa que alguns arquivos foram completamente reescritos, mas várias linhas deles são claramente idênticas: Isso me faz pensar que houve alguma troca de caracteres invisíveis como o EOL. E de fato, eu cheguei a ir à sua branch e rodar um Caso não seja o editor, você está usando o Windows para fazer as alterações? De acordo com a segunda resposta aqui, esse parece ser o caso. |
|
Sim estou usando o Windows, eu havia criado o fork porém eu já tinha os
arquvios modificados, então o que tinha feito foi substituir os arquivos
que modifiquei, as únicas alterações realmente estavam nas linhas que eu
havia mencionado, após realizar o diff --ignore-cr-at-eol quais
modificações sobraram?
Em seg., 26 de abr. de 2021 às 13:26, Fillipe Goulart <
***@***.***> escreveu:
… Oi, @williamloggi <https://github.com/williamloggi>, obrigado também pela
contribuição!
Eu tenho uma outra dúvida: no PR o diff avisa que alguns arquivos foram
completamente reescritos, mas várias linhas deles são claramente idênticas:
[image: image]
<https://user-images.githubusercontent.com/10814987/116124107-3d62ed80-a69a-11eb-898c-cacbb20a1d2e.png>
Isso me faz pensar que houve alguma troca de caracteres invisíveis como o
EOL. E de fato, eu cheguei a ir à sua branch e rodar um git diff
--ignore-cr-at-eol e o diff fica muito menor.
Caso não seja o editor, você está usando o Windows para fazer as
alterações? De acordo com a segunda resposta aqui
<https://stackoverflow.com/questions/1889559/git-diff-to-ignore-m>, esse
parece ser o caso.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#14 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADRKA5L5H526V363AKADHULTKWO4ZANCNFSM43SBS3GA>
.
--
Com atenção, Wilton Costa.
|
|
Wiltoon, eu gostei bastante da sua contribuição! Imagino que deva acontecer com outros devs. Eu sugiria fazer o PR direitinho para adicionar no main seu código :) Se precisar de ajuda estou a disposição. |
|
Wiltoon, eu não tenho palavras para te agradecer com o teu PR. O arquivo que você fez loggibud/v1/baselines/task1/HOW_TO_EXECUTE.md literalmente me tirou da estagnação total com o que esse projeto é e como ele funciona. Mais uma vez, obrigado! |
|
Olá, @gabisurita @fillipe-gsm eu tentei reenviar o código mas deem uma olhada pois meus arquivos do lkh_3.py estava já defasado com o de voces, o kmeans aggregation e o partition possuem pequenas mudanças incluindo o parametro "region", e possuí um README de como rodar os algoritmos da task1 que acredito que possa ajudar outras pessoas como @alexiascheffer @minterciso |
No arquivo kmeans_aggregation_ortools.py havia 2 erros:
line 25: from ..shared.ortools import solve_cvrp as ortools_solve, ORToolsParams, não havia o método solve_cvrp foi alterado para solve que era o padrão do run_task1
line 113: aggregated_instance: estava exigindo que a "region" fosse especificada no objeto CVRPInstance
No arquivo kmeans_partition_ortools.py
line 80: subinstances: estava exigindo que a "region" fosse especificada no objeto CVRPInstance
Foi criado um HOW_TO_EXECUTE é um demostrativo de como executar os exemplos da task1
Foi criado tbm um execution_task1.py na pasta loggibud, que na verdade é um script que gera a saída de todas as instancias que forem passadas, na main() é definido as pastas que é desejado percorrer