graph-learning-ita / airnet-holding-ml

Graph Machine Learning in Holding Flights Airspace Prediction
Apache License 2.0
0 stars 0 forks source link

Add docs/tcc and update seminary #5

Closed JorgeLuizFranco closed 5 days ago

ryukinix commented 1 week ago

Legal, Jorge! Vejo que tem alguns arquivos que precisa ser removidos, não sei bem como eles foram adicionados, eram pra ter sido ignorados pelo gitignore. Devo ter me esquecido de algo.

De toda maneira, vou dar uma olhada mais profunda agora nesse domingo.

Em sáb., 16 de nov. de 2024 16:36, JorgeLuizFranco @.***> escreveu:

@JorgeLuizFranco https://github.com/JorgeLuizFranco requested your review on: #5 https://github.com/graph-learning-ita/airnet-holding-ml/pull/5 Tcc.

— Reply to this email directly, view it on GitHub https://github.com/graph-learning-ita/airnet-holding-ml/pull/5#event-15322206040, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB2J57W7GRAIQ77M5THKESD2A6NENAVCNFSM6AAAAABR5D3P6KVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJVGMZDEMRQGYYDIMA . You are receiving this because your review was requested.Message ID: <graph-learning-ita/airnet-holding-ml/pull/5/issue_event/15322206040@ github.com>

ryukinix commented 1 week ago

@JorgeLuizFranco Você criou sua branch a partir da master? Tem modificações sem sentido aqui que eram da outra branch e foram feito squash durante o merge. Tem commits que nem deveria estar aqui, por exemplo, todos esses:

image

Pra piorar, não incluiu as modificações que foram importantes para o merge: image

Ou seja, todas as modificações que revisei no MR anterior, foram ignoradas, e re-introduzidas novamente! Você poderia considerar em recriar um novo pull-request a partida master (atualizada) e apenas adicionar os arquivos que você mexeu?

ryukinix commented 6 days ago

Corrigi @JorgeLuizFranco a questão dos commits apontado nesse comentário: https://github.com/graph-learning-ita/airnet-holding-ml/pull/5#issuecomment-2481400422

Por favor, para evitar que isso ocorra novamente:

Peço que revise as alterações, tente compilar novamente os documentos, pois eu nunca consegui compilar o documento tcc. O make seminary funciona corretamente, mas o make tcc não.