-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Import/tlm packetizer #878
Conversation
@check-spelling-bot ReportUnrecognized words, please review:
Previously acknowledged words that are now absentFIXME lxrTo accept these unrecognized words as correct (and remove the previously acknowledged and now absent words), run the following commands... in a clone of the git@github.com:timcanham/fprime.git repository
|
This pull request introduces 13 alerts when merging b47ef5d into d3fa31c - view on LGTM.com new alerts:
|
@check-spelling-bot ReportUnrecognized words, please review:
Previously acknowledged words that are now absentFIXME lxrTo accept these unrecognized words as correct (and remove the previously acknowledged and now absent words), run the following commands... in a clone of the git@github.com:timcanham/fprime.git repository
|
@check-spelling-bot ReportUnrecognized words, please review:
Previously acknowledged words that are now absentFIXME lxrTo accept these unrecognized words as correct (and remove the previously acknowledged and now absent words), run the following commands... in a clone of the git@github.com:timcanham/fprime.git repository
|
Note: I'll work through the lgtm warnings when I submit the final script. |
This pull request introduces 13 alerts when merging cfe5980 into d3fa31c - view on LGTM.com new alerts:
|
@check-spelling-bot ReportUnrecognized words, please review:
Previously acknowledged words that are now absentFIXME lxrTo accept these unrecognized words as correct (and remove the previously acknowledged and now absent words), run the following commands... in a clone of the git@github.com:timcanham/fprime.git repository
|
This pull request introduces 13 alerts when merging f4448fd into d3fa31c - view on LGTM.com new alerts:
|
@check-spelling-bot ReportUnrecognized words, please review:
Previously acknowledged words that are now absentFIXME lxrTo accept these unrecognized words as correct (and remove the previously acknowledged and now absent words), run the following commands... in a clone of the git@github.com:timcanham/fprime.git repository
|
This pull request introduces 5 alerts when merging 6c5bfc0 into d3fa31c - view on LGTM.com new alerts:
|
1 similar comment
This pull request introduces 5 alerts when merging 6c5bfc0 into d3fa31c - view on LGTM.com new alerts:
|
@check-spelling-bot ReportUnrecognized words, please review:
Previously acknowledged words that are now absentFIXME lxrTo accept these unrecognized words as correct (and remove the previously acknowledged and now absent words), run the following commands... in a clone of the git@github.com:timcanham/fprime.git repository
|
This pull request introduces 5 alerts when merging 9d728b4 into e7ba8aa - view on LGTM.com new alerts:
|
@check-spelling-bot ReportUnrecognized words, please review:
Previously acknowledged words that are now absentFIXME lxrTo accept these unrecognized words as correct (and remove the previously acknowledged and now absent words), run the following commands... in a clone of the git@github.com:timcanham/fprime.git repository
|
This pull request introduces 5 alerts when merging 865517a into e7ba8aa - view on LGTM.com new alerts:
|
This pull request introduces 5 alerts when merging 5b683d6 into e7ba8aa - view on LGTM.com new alerts:
|
This pull request introduces 5 alerts when merging 3bf09f8 into e7ba8aa - view on LGTM.com new alerts:
|
Svc/TlmPacketizer
Change Description
This PR contains the
Svc/TlmPacketizer
component and updates to the autocoder scripts to support generating packet C++ code. It is not the final delivery; there are some dependency issues to sort out, and the packetizer script isn't fully complete. This is meant to deliver the component and scripts to avoid diverging too much.Rationale
This is a useful alternative to
Svc/TlmChan
if fixed packets make more sense.Testing/Review Recommendations
The
Ref
application is tested to show it runs normally. You can exercise theSET_LEVEL
command with a0
argument and see theLevelSet
evr andSendLevel
channel.Future Work
The dependency issues discovered during development need to be fixed, and the full end-to-end testing would need to be completed.