scanner-research / scanner

Efficient video analysis at scale
https://scanner-research.github.io/
Apache License 2.0
620 stars 108 forks source link

Prototype of new scheduler #218

Open swjz opened 6 years ago

swjz commented 6 years ago

Hi Alex,

scanner/engine/scheduler.h contains my prototype of the new scheduler. Please take a look and let me know if you have any comments.

fpoms commented 5 years ago

Goal: Merge the experimental scheduler in this PR into master.

We would like to merge this branch into master so that it's development can occur in parallel and in sync with improvements to the systems. By merging, it will make it possible to test both variants of the scheduler side by side for performance and correctness, and enable use cases that might benefit from the nascent experimental scheduler. The current approach for enabling both schedulers side-by-side is to insert a flag when starting the master that tells it which mode to start in: the existing scheduler or the new experimental one.

Minimum requirements:

I'm likely missing some TODOs. Let me know and I'll add them.

swjz commented 5 years ago

I've finished these minimum requirements:

When new_scheduler is True, the new scheduler should behave exactly the same as what I specified in my doc. Can you also test it from your side and let me know if any problem occurs?

fpoms commented 5 years ago

Awesome! I'll do a review pass now.