Optimizer update #42
Optimizer update #42
Conversation
There was a problem hiding this comment.
Não tenho conhecimento para revisar esse arquivo.
There was a problem hiding this comment.
Não tenho conhecimento para revisar esse arquivo.
There was a problem hiding this comment.
Não tenho conhecimento para revisar esse arquivo.
There was a problem hiding this comment.
Os testes não estão executando, sendo necessário criar o diretório tests/test_data. Mas, mesmo com esse diretório, alguns testes ainda falham.
There was a problem hiding this comment.
Mesmo problema de test_aws_model.py.
There was a problem hiding this comment.
Não seria melhor colocar o tipo de mercado de reservas como uma constante? Seria semelhante a RES_DURATION.
There was a problem hiding this comment.
Apesar de não ter havido mudanças nesse PR em relação à isso, acredito que este arquivo está fazendo duas coisas ao mesmo tempo: ler/escrever em arquivos e fazer a alocação da demanda. O ideal seria colocar a alocação em um arquivo separado.
No description provided.