-
Notifications
You must be signed in to change notification settings - Fork 1
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
Optimizer update #42
base: dev
Are you sure you want to change the base?
Optimizer update #42
Changes from 8 commits
0ea6852
9ef27a3
9c27e82
91efc80
6593652
5b09433
bcde612
06bec45
e27aafc
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,3 +5,4 @@ aws_model.log | |
result_cost.csv | ||
build/*.o | ||
build/*.elf | ||
.pytest_cache |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,22 +1,31 @@ | ||
FROM awsome-savings:latest | ||
FROM registry-git.lsd.ufcg.edu.br/pedro.serey/awsome-savings:or-tools | ||
|
||
COPY ./requirements.txt . | ||
# Optimizer build | ||
WORKDIR /calculation/optimizer/ | ||
|
||
RUN pip3 install -r requirements.txt | ||
COPY optimizer/cpp/*.cpp optimizer/cpp/*.h Makefile util/*.sh ./ | ||
|
||
RUN mkdir -p /calculation/optimizer && \ | ||
cp -r /optimizer/* /calculation/optimizer | ||
RUN mkdir -p ./build && \ | ||
make docker-compile && \ | ||
chmod +x ./run_optimization.sh && \ | ||
chmod +x ./collect-cpu-usage.sh && \ | ||
chmod +x ./collect-memory-usage.sh | ||
|
||
COPY costplanner_cli.py /calculation/ | ||
COPY services/ /calculation/services/ | ||
COPY example_input/ /calculation/example_input/ | ||
COPY data/ /calculation/data/ | ||
COPY *.cpp *.h Makefile /util/*.sh /calculation/optimizer/ | ||
VOLUME ["/optimizer-files"] | ||
VOLUME ["/optimizer-logs"] | ||
|
||
WORKDIR /calculation/optimizer | ||
# Calculator build | ||
WORKDIR /calculation | ||
|
||
RUN make compile | ||
COPY ./requirements.txt ./ | ||
|
||
WORKDIR /calculation | ||
RUN pip3 install -r requirements.txt | ||
|
||
COPY costplanner_cli.py ./ | ||
COPY calculator/ ./calculator/ | ||
COPY util/optimizer_util.py ./util/ | ||
COPY data/ ./data/ | ||
|
||
ENV LD_LIBRARY_PATH=/usr/local/lib:$LD_LIBRARY_PATH | ||
|
||
CMD ["/bin/sh", "-c", "echo AWSome-Savings!"] | ||
CMD ["/bin/sh", "-c", "echo AWSome-Savings!"] |
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Não tenho conhecimento para revisar esse arquivo. |
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Não tenho conhecimento para revisar esse arquivo. |
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 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. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Não tenho conhecimento para revisar esse arquivo.