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

The scalable computations #77

Merged
merged 17 commits into from
Feb 5, 2024
Merged

The scalable computations #77

merged 17 commits into from
Feb 5, 2024

Conversation

StannisMod
Copy link
Collaborator

@StannisMod StannisMod commented Jan 31, 2024

Introduced ComputationalBackend - an interface to the computational-bound operations.
It can be implemented in different ways because of different ways to do such computations.

Now implemented two backends:

  • DefaultComputationalBackend that runs computations on plain Python
  • MultiprocessingComputationalBackend that uses pathos.multiprocessing to parallelize execution

Additional:

  • created singletone SAMPO that stores reference to the backend and logger
  • now we have centralized log handler
  • slightly reworked Genetic's scheduler_builder

sampo/scheduler/genetic/base.py Show resolved Hide resolved
sampo/scheduler/genetic/schedule_builder.py Outdated Show resolved Hide resolved
sampo/scheduler/genetic/operators.py Show resolved Hide resolved
sampo/scheduler/genetic/operators.py Outdated Show resolved Hide resolved
sampo/scheduler/genetic/operators.py Outdated Show resolved Hide resolved
sampo/scheduler/genetic/schedule_builder.py Show resolved Hide resolved
sampo/scheduler/genetic/utils.py Outdated Show resolved Hide resolved
Timotshak
Timotshak previously approved these changes Feb 2, 2024
@StannisMod StannisMod added the enhancement New feature or request label Feb 2, 2024
@StannisMod StannisMod closed this Feb 5, 2024
@StannisMod StannisMod reopened this Feb 5, 2024
@vanoha01 vanoha01 self-requested a review February 5, 2024 09:49
Copy link
Collaborator

@vanoha01 vanoha01 left a comment

Choose a reason for hiding this comment

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

Approved

@StannisMod StannisMod merged commit 5775ac9 into main Feb 5, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants