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 a tasks_full_details attribute to State for scheduling notebook #211

Merged
merged 1 commit into from
Sep 30, 2022

Conversation

nhuet
Copy link
Contributor

@nhuet nhuet commented Sep 26, 2022

After the code optimization made on scheduling algorithms in commit b76e15a, the attribute tasks_details does not keep track of all tasks anymore. Thus:

  • we introduce a new attribute tasks_full_details which holds also information for non-completed tasks and is actually a property reconstructed on-the-fly via rebuild_all_tasks_dict
  • we add a dummy task for each task id in rebuild_all_tasks_dict output so that we can call a task detail even before starting any task as required in the scheduling notebook

After the code optimization made on scheduling algorithms
in commit b76e15a, the attribute `tasks_details` does not keep track of
all tasks anymore. Thus:

- we introduce a new attribute `tasks_full_details` which holds also
  information for non-completed tasks and is actually a property
  reconstructed on-the-fly via `rebuild_all_tasks_dict`
- we add a dummy task for each task id in `rebuild_all_tasks_dict`
  output so that we can call a task detail even before starting any task
  as required in the scheduling notebook
Copy link
Contributor

@galleon galleon left a comment

Choose a reason for hiding this comment

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

LGTM

@galleon galleon merged commit bf56097 into airbus:master Sep 30, 2022
@nhuet nhuet deleted the nht/tasks_full_details branch October 6, 2022 12:21
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