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.

Sign up for GitHub

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

Jump to bottom

Planner: improve plan selection #18211

Merged
merged 10 commits into from
Jan 15, 2025
Merged

Planner: improve plan selection #18211

merged 10 commits into from
Jan 15, 2025
+69 −16

Conversation

naltatis
Copy link
Member

@naltatis naltatis commented Jan 13, 2025 •
edited
Loading

fixes #18212

  • 🔢 select the plan by "last possible start time" instead of "plan time"

TODO:

  • add tests
  • backend tests
  • check which files the diff belongs to (effective or plan)

@naltatis naltatis added the enhancement New feature or request label Jan 13, 2025
@naltatis naltatis marked this pull request as draft January 13, 2025 18:55
@naltatis naltatis marked this pull request as ready for review January 13, 2025 21:52
@andig
Copy link
Member

andig commented Jan 14, 2025

@naltatis Backendtest ergänzt- schau bitte mal ob vollständig. UI Test kann weg?

@naltatis naltatis merged commit 285b22e into master Jan 15, 2025
6 checks passed
@naltatis naltatis deleted the fix/improve_plan_selection branch January 15, 2025 14:40
@naltatis naltatis mentioned this pull request Jan 16, 2025
Closed
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andig andig andig left review comments

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Plan: improve next plan selection
2 participants
@naltatis @andig