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.

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

cFE cmd/tlm packet clean up #664

Open
dmknutsen opened this issue May 1, 2020 · 1 comment
Open

cFE cmd/tlm packet clean up #664

dmknutsen opened this issue May 1, 2020 · 1 comment

Comments

@dmknutsen
Copy link
Contributor

dmknutsen commented May 1, 2020

Is your feature request related to a problem? Please describe.
cFE telemetry/command packets could be organized such that variables are ordered in descending size to reduce overall length. This would make them easier to maintain and remove some of the alignment issues/reduce required padding.

There's also configurable length elements before spares, which doesn't make sense if the configuration changes.

Also should standardize spare/alignment names, many no longer make sense (why "4" in example):

uint8 ByteAlign4; /**< \cfetlmmnemonic \TBL_SPARE4
\brief Spare byte to maintain byte alignment */

Requester Info
Dan Knutsen
NASA/Goddard

@skliper skliper added this to the 7.0.0 milestone May 7, 2020
@skliper skliper removed this from the 7.0.0 milestone Jan 22, 2021
@skliper skliper changed the title cFE tlm packet clean up cFE cmd/tlm packet clean up Apr 20, 2021
@skliper skliper added the CFS-41 label Apr 20, 2021
@skliper
Copy link
Contributor

skliper commented Apr 29, 2021

Note cmd/tlm structure updates are currently deferred to delivery from data center and definition of generic data descriptions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants