Skip to content

Conversation

@mhramani
Copy link
Contributor

No description provided.

@mhramani mhramani marked this pull request as draft May 16, 2025 22:47
@mhramani mhramani marked this pull request as ready for review May 28, 2025 20:14
@mhramani mhramani changed the title Added a class for job shop scheduling using NL Added NL formulation for Job-shop scheduling May 28, 2025
from model_data import JobShopData
from utils.utils import print_cqm_stats

sys.path.append("./src")

Choose a reason for hiding this comment

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

We shouldn't have to manipulate the path, this indicates we are missing some structure in the directories (__init__.py isn't present in places it should be)

return self._job_tasks

@property
def job_resources(self) -> dict:

Choose a reason for hiding this comment

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

missing a docstring here as well as get_ordered_tasks

Copy link
Collaborator

@k8culver k8culver left a comment

Choose a reason for hiding this comment

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

This errors while trying to run it in the interface because callback functions run_optimization_cqm and run_optimization_mip are calling run_shop_scheduler with parameters you've removed.

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.

3 participants