Possible TC scheduler bugfix
Closedmuellerr wants to merge 3 commits from
Reference in new issue
There is no content yet.
Delete Branch 'eive/fsfw:possible_tc_sched_fixes'
Deleting a branch is permanent. It CANNOT be undone. Continue?
I really think that logic is broken.
Possible TC schedulerto Possible TC scheduler bugfix 4 months ago
This is included in #735 so maybe not necessary.
As far as the logic goes, it is intended behavior that commands that are scheduled too close in the future are rejected.
This is an operational requirement as one should not upload scheduled TCs so close to execution time.
Thinking about it, I am not sure if it really belongs into the flight software, but I think the operational impact is minimal and would keep it to be backward compatible.
Also, agree on #735 fixing the integer underflow (unintentionally. While I recognized that line as suboptimal I actually did not realize the underflow).
Solved by #735