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 Constraint Optimization Workload Allocation Stage #47

Merged
merged 30 commits into from
Oct 1, 2024
Merged

Conversation

asyms
Copy link
Contributor

@asyms asyms commented Sep 23, 2024

  • Add constraint optimization (CO) stage for the workload allocation
  • Add api call for CO
  • Update api call for genetic algorithm (GA)
  • Clean up main files keeping only two (one using GA and one using CO)
  • Introduce CostModelEvaluationLUT which replaces the structure of nested dictionaries to cache the hardware performance of different CNs on different cores
  • Various refactors of duplicate code

@RobinGeens
Copy link
Contributor

The restructuring work and file renaming is very nice! This makes it a lot more organized :)

@RobinGeens
Copy link
Contributor

I'm getting this error when running main_stream_co.py:

File "src/gurobipy/model.pxi", line 893, in gurobipy.Model.optimize
gurobipy.GurobiError: Model too large for size-limited license; visit https://gurobi.com/unrestricted for more information

Isn't this supposed to be detected by _sanity_check_gurobi_license ?

from networkx import DiGraph

from stream.workload.computation.computation_node import ComputationNode
from stream.workload.utils import prune_workload
Copy link
Contributor

Choose a reason for hiding this comment

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

This does not seem to exist anymore?



def convert_id(i: int, j: int) -> int:
k = 1000 * i + j
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this a magic number? :p

@asyms asyms merged commit 6a0e42f into master Oct 1, 2024
4 checks passed
@asyms asyms deleted the dev-co branch October 1, 2024 11:12
@asyms asyms mentioned this pull request Oct 1, 2024
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