fprime icon indicating copy to clipboard operation
fprime copied to clipboard

Update Seqgen to support complex commanding

Open thomas-bc opened this issue 2 years ago • 3 comments

F´ Version
Affected Component


Feature Description

Update GDS Seqgen to support complex types commanding

thomas-bc avatar Jul 26 '23 18:07 thomas-bc

Did some initial investigation on this.

  1. This is going to be a substantial change as most of the seqgen file parser needs to be replaced
  2. The algorithm the UI uses has been ported to python here: https://github.com/LeStarch/fprime-gds/tree/update/complex-sequencing
  3. Need to deal with the time tokens:
  4. Could parse separately
  5. Could add a regex clause to support R and A time tokens
  6. Need to deal with ;

LeStarch avatar Oct 24 '23 17:10 LeStarch

@Joshua-Anderson and @timcanham given the desire to improve the sequencer, is improving the old sequencing tools still valuable? Or should I shelve this issue as it will be resolved with the revitalization effort?

LeStarch avatar Mar 06 '24 16:03 LeStarch

My suspicion is that the sequencer revitalization is still a ways off, and we would likely keep the current command sequencer around as an simpler option for folks who don't need the complexity of the full sequencer.

It would be a reasonable call to implement complex commanding in the current tinyseqgen compiler. That being said, I think it's probably a pretty low priority. Complex types seem to be less common in commanding than telemetry.

Joshua-Anderson avatar Mar 06 '24 16:03 Joshua-Anderson