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.

Sign up for GitHub

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

Jump to bottom

Remove use of HK parameter (LogEnabled) in EVS control logic #1153

Closed
skliper opened this issue Feb 4, 2021 · 0 comments · Fixed by #1178 or #1196
Closed

Remove use of HK parameter (LogEnabled) in EVS control logic #1153

skliper opened this issue Feb 4, 2021 · 0 comments · Fixed by #1178 or #1196
Assignees
@skliper
Labels
enhancement
Milestone
Caelum

Comments

@skliper
Copy link
Contributor

skliper commented Feb 4, 2021

Is your feature request related to a problem? Please describe.
CFE_EVS_GlobalData.EVS_TlmPkt.Payload.LogEnabled is used in logic, discouraged pattern (tlm data used for control)

Describe the solution you'd like
Simplify the logic

Describe alternatives you've considered
None

Additional context
#609, #1129

Requester Info
Jacob Hageman - NASA/GSFC
@skliper skliper added this to the 7.0.0 milestone Feb 4, 2021
@skliper skliper self-assigned this Feb 4, 2021
@skliper skliper mentioned this issue Feb 4, 2021
Merged
skliper added a commit to skliper/cFE that referenced this issue Feb 18, 2021
@skliper
Fix nasa#1153, Remove logic based on LogEnabled status
219a615
@skliper skliper mentioned this issue Feb 18, 2021
Merged
skliper added a commit to skliper/cFE that referenced this issue Mar 3, 2021
@skliper
Fix nasa#1153, Remove logic based on LogEnabled status
ab4f3fc
skliper added a commit to skliper/cFE that referenced this issue Mar 3, 2021
@skliper
Fix nasa#1153, Remove logic based on LogEnabled status
693cb5d
astrogeco added a commit that referenced this issue Mar 3, 2021
@astrogeco
Merge pull request #1178 from skliper/fix1153-logenabled_logic
aded3e4 
Fix #1153, Remove logic based on LogEnabled status
@astrogeco astrogeco mentioned this issue Mar 3, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@skliper skliper

Labels
enhancement
Projects
None yet
Milestone
Caelum
1 participant
@skliper