lingua-franca icon indicating copy to clipboard operation
lingua-franca copied to clipboard

Published 20 hours ago • verified publisher icon lf-lang

Static scheduler

Open lsk567 opened this issue 1 year ago • 7 comments

This is a PR for keeping track of the progress on a static scheduler.

Refer to the project board for more details. This PR contains all the items in the Done column.

lsk567 avatar Jun 08 '23 23:06 lsk567

@ChadliaJerad it looks like several commits from master got rebased and pushed to this branch and are now duplicated. Please use git merge to update a feature branch or rebase the feature branch on top of master (not the other way around). This way, we can keep a clean history.

cmnrd avatar Jun 15 '23 12:06 cmnrd

@ChadliaJerad it looks like several commits from master got rebased and pushed to this branch and are now duplicated. Please use git merge to update a future branch or rebase the future branch on top of master (not the other way around). This way, we can keep a clean history.

Yes! I was indeed wondering why are all these commits there.

ChadliaJerad avatar Jun 15 '23 14:06 ChadliaJerad

Thanks @cmnrd for catching this issue. I just cherry-picked the relevant commits and force-pushed to this branch. The history should be clean now.

lsk567 avatar Jun 16 '23 03:06 lsk567

Thanks for your detailed review, @oowekyala! I will work on the issues you pointed out and try to make the implementation more robust.

lsk567 avatar Jun 21 '23 15:06 lsk567

@ChadliaJerad it looks like several commits from master got rebased and pushed to this branch and are now duplicated. Please use git merge to update a feature branch or rebase the feature branch on top of master (not the other way around). This way, we can keep a clean history.

Looks like there is a similar problem with duplicate commits in https://github.com/lf-lang/lingua-franca/pull/1695.

lhstrh avatar Jun 25 '23 06:06 lhstrh

It seems like the Draft status prevents the CI tests from being run. To invoke CI, I will mark the PR as ready for review.

lsk567 avatar Jul 27 '23 14:07 lsk567

Just performed a huge rebase to this branch. Now this branch adopts the latest target property registration workflow. There is an unresolved problem where nested dictionaries in target properties do not seem to get loaded. Therefore, at commit 1e236c9, the MOCASIN scheduler cannot be used, and the EGS scheduler can only be invoked on the command line using --static-scheduler EGS.

Update: thanks to @lhstrh 's help, this is resolved.

lsk567 avatar Nov 16 '23 01:11 lsk567