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

create CFE_TIME_PrintFmt(); #2316

Open
CDKnightNASA opened this issue May 1, 2023 · 1 comment · Fixed by #2356 · May be fixed by #2390
Open

create CFE_TIME_PrintFmt(); #2316

CDKnightNASA opened this issue May 1, 2023 · 1 comment · Fixed by #2356 · May be fixed by #2390

Comments

@CDKnightNASA
Copy link
Contributor

CDKnightNASA commented May 1, 2023

Is your feature request related to a problem? Please describe.
Currently the CFE_TIME_Print() function only produces YYYY-ddd HH:MM:SS.xxxx formatted timestamps. CFE_EVS now uses that for stdout messages, which adds quite a lot of text.

Describe the solution you'd like
I suggest the addition of a new CFE_TIME_PrintFmt() function, a-la strftime (preferably using strftime, if that is available on all our target platforms) so that applications can produce a time-only format, for example.

Additionally, I suggest a "T+" format for such things as EVS stdout output where the first report is the full timestamp for "T0" and all subsequent messages are the number of seconds (SS.xxxx) after the T0 epoch for even more storage efficiency and readability.

Once this capability is created, EVS should be modified to allow configuring the format.

Describe alternatives you've considered
Another option is to use something other than CFE_TIME_Print() in EVS_SendViaPorts().

Additional context
See the related PR #2295

Requester Info
Christopher.D.Knight@nasa.gov

@dzbaker
Copy link
Collaborator

dzbaker commented Jul 11, 2023

Issue is being re-opened to address in Equuleus-rc2.

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