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.

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

add file based distributed optimization loop #124

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

LGro
Copy link
Member

@LGro LGro commented Feb 6, 2024

For a distributed system where a file system is shared, this introduces an optimization loop setup where the optimizer always keeps the batch of proposals filled and clients can read proposed eval specs and report corresponding results.

@LGro LGro requested review from sfalkner and dynobo February 6, 2024 16:26
@LGro LGro force-pushed the feature/file-based-distributed-loop branch from bd53bb1 to 63dd0d7 Compare February 7, 2024 14:45
Copy link

github-actions bot commented Feb 7, 2024

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  blackboxopt
  __init__.py
  blackboxopt/optimization_loops
  file_based_distributed.py 82, 122-127, 194-199
Project Total  

This report was generated by python-coverage-comment-action

@LGro LGro force-pushed the feature/file-based-distributed-loop branch 2 times, most recently from d38329e to 400cdef Compare February 7, 2024 14:51
Signed-off-by: Grossberger Lukas (CR/AIR2.2) <Lukas.Grossberger@de.bosch.com>
@LGro LGro force-pushed the feature/file-based-distributed-loop branch from 400cdef to 4542471 Compare February 7, 2024 15:32
Copy link
Collaborator

@sfalkner sfalkner left a comment

Choose a reason for hiding this comment

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

Thanks @LGro for this addition. Except for one comment, this looks very good and useful :)

Signed-off-by: Grossberger Lukas (CR/AIR2.2) <Lukas.Grossberger@de.bosch.com>
@LGro LGro force-pushed the feature/file-based-distributed-loop branch from cc7b1c4 to dc56043 Compare February 8, 2024 10:04
Signed-off-by: Grossberger Lukas (CR/AIR2.2) <Lukas.Grossberger@de.bosch.com>
@LGro LGro force-pushed the feature/file-based-distributed-loop branch from 0a94557 to 1fcaef7 Compare February 8, 2024 10:09
Signed-off-by: Grossberger Lukas (CR/AIR2.2) <Lukas.Grossberger@de.bosch.com>
@LGro LGro force-pushed the feature/file-based-distributed-loop branch from 4d5fab6 to a4590cf Compare February 8, 2024 10:29
Signed-off-by: Grossberger Lukas (CR/AIR2.2) <Lukas.Grossberger@de.bosch.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants