Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Optimizer update #42

Open
wants to merge 24 commits into
base: dev
Choose a base branch
from
Open

Optimizer update #42

wants to merge 24 commits into from

Conversation

KaykyFidelis
Copy link
Collaborator

No description provided.

@KaykyFidelis KaykyFidelis requested a review from RuanGOA July 15, 2024 13:47
pserey and others added 14 commits July 19, 2024 15:54
@pserey
add: calculation rule to makefile
06bec45
@KaykyFidelis
ref: changing the function that create the optimizer allocation file
b5931b7
@KaykyFidelis
ref: removing some spaces and old guide lines
6448702
@KaykyFidelis
feat: adding a function to write the allocation file from the optimiz…
5b324e4 
…er directory
@KaykyFidelis
ref: changing the language of documentation to english
ad152a4
@KaykyFidelis
ref: changing the language of documentation to english and removing s…
4b02a67 
…paces
@KaykyFidelis
feat: creating the function to read allocation
f456117
@KaykyFidelis
feat: creating a function that write any type of file in a specific o…
eeb8ad4 
…utput directory
@KaykyFidelis
feat: integrating the optimizer output into the calculator input
cedbd8a
@KaykyFidelis
feat: putting the allocator into another module
fc7d4f8
@KaykyFidelis
docs: changing the description of the costplanner output files
f3724e9
@KaykyFidelis
test: optimizer allocator output tests
fe03a3a
@KaykyFidelis
build: adding the tests directory into the image
2150444
@KaykyFidelis
docs: adding the description of how to run the optimizer output test
21fcd1e
@caio-galvao caio-galvao self-requested a review August 30, 2024 14:52
caio-galvao
caio-galvao reviewed Sep 18, 2024
View reviewed changes
Makefile Show resolved Hide resolved
caio-galvao
caio-galvao requested changes Sep 18, 2024
View reviewed changes
Copy link
Collaborator

@caio-galvao caio-galvao left a comment

Choose a reason for hiding this comment

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

Encotrei problemas nos testes, que não estão executando, e outros dois problemas de design. Não revisei os Dockerfiles e o Makefile.

tests/test_aws_model.py Show resolved Hide resolved
tests/test_aws_model_cpp.py Show resolved Hide resolved
util/optimizer_util.py Show resolved Hide resolved
calculator/data_management.py Show resolved Hide resolved
@pserey pserey requested review from pserey and removed request for RuanGOA December 3, 2024 12:26
@pserey pserey requested a review from caio-galvao December 3, 2024 12:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pserey pserey Awaiting requested review from pserey

@caio-galvao caio-galvao Awaiting requested review from caio-galvao

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@KaykyFidelis @caio-galvao @pserey