Skip to content

Comments

Fix #219, define msgids via topicids#220

Merged
dzbaker merged 1 commit intonasa:mainfrom
jphickey:fix-219-topicids
Dec 12, 2023
Merged

Fix #219, define msgids via topicids#220
dzbaker merged 1 commit intonasa:mainfrom
jphickey:fix-219-topicids

Conversation

@jphickey
Copy link
Contributor

@jphickey jphickey commented Dec 7, 2023

Describe the contribution
The MsgID value is a conversion from TopicID

Fixes #219

Testing performed
Build and run tests

Expected behavior changes
None, topic IDs were chosen such that the resulting MsgIDs are the same when using default config

System(s) tested on
Debian

Additional context
Depends on nasa/cFE#2474

Contributor Info - All information REQUIRED for consideration of pull request
Joseph Hickey, Vantage Systems, Inc.

The MsgID value is a conversion from TopicID
@dzbaker dzbaker added the CCB:Approved Indicates code approval by CCB label Dec 7, 2023
dzbaker added a commit to nasa/cFS that referenced this pull request Dec 12, 2023
*Combines:*

to_lab v2.5.0-rc4+dev75
ci_lab v2.5.0-rc4+dev81
sample_app v1.3.0-rc4+dev69
sch_lab v2.5.0-rc4+dev75
cFE v7.0.0-rc4+dev434
osal v6.0.0-rc4+dev247

**Includes:**

*to_lab*
- nasa/to_lab#176

*ci_lab*
- nasa/ci_lab#165

*sample_app*
- nasa/sample_app#220

*sch_lab*
- nasa/sch_lab#156

*cFE*
- nasa/cFE#2472
- nasa/cFE#2411
- nasa/cFE#2474

*osal*
- nasa/osal#1430

Co-authored by: Joseph Hickey <[email protected]>
Co-authored by: Isaac Rowe <[email protected]>
Co-authored by: Avi Weiss <[email protected]>
@dzbaker dzbaker mentioned this pull request Dec 12, 2023
2 tasks
dzbaker added a commit to nasa/cFS that referenced this pull request Dec 12, 2023
*Combines:*

to_lab v2.5.0-rc4+dev75
ci_lab v2.5.0-rc4+dev81
sample_app v1.3.0-rc4+dev69
sch_lab v2.5.0-rc4+dev75
cFE v7.0.0-rc4+dev434
osal v6.0.0-rc4+dev247

**Includes:**

*to_lab*
- nasa/to_lab#176

*ci_lab*
- nasa/ci_lab#165

*sample_app*
- nasa/sample_app#220

*sch_lab*
- nasa/sch_lab#156

*cFE*
- nasa/cFE#2472
- nasa/cFE#2411
- nasa/cFE#2474

*osal*
- nasa/osal#1430

Co-authored by: Joseph Hickey <[email protected]>
Co-authored by: Isaac Rowe <[email protected]>
Co-authored by: Avi Weiss <[email protected]>
dzbaker added a commit to nasa/cFS that referenced this pull request Dec 12, 2023
*Combines:*

to_lab v2.5.0-rc4+dev75
ci_lab v2.5.0-rc4+dev81
sample_app v1.3.0-rc4+dev69
sch_lab v2.5.0-rc4+dev75
cFE v7.0.0-rc4+dev434
osal v6.0.0-rc4+dev247

**Includes:**

*to_lab*
- nasa/to_lab#176

*ci_lab*
- nasa/ci_lab#165

*sample_app*
- nasa/sample_app#220

*sch_lab*
- nasa/sch_lab#156

*cFE*
- nasa/cFE#2472
- nasa/cFE#2411
- nasa/cFE#2474

*osal*
- nasa/osal#1430

Co-authored by: Joseph Hickey <[email protected]>
Co-authored by: Isaac Rowe <[email protected]>
Co-authored by: Avi Weiss <[email protected]>
@dzbaker dzbaker merged commit 20ce593 into nasa:main Dec 12, 2023
dzbaker added a commit to nasa/cFS that referenced this pull request Dec 12, 2023
*Combines:*

to_lab v2.5.0-rc4+dev75
ci_lab v2.5.0-rc4+dev81
sample_app v1.3.0-rc4+dev69
sch_lab v2.5.0-rc4+dev75
cFE v7.0.0-rc4+dev434
osal v6.0.0-rc4+dev247

**Includes:**

*to_lab*
- nasa/to_lab#176

*ci_lab*
- nasa/ci_lab#165

*sample_app*
- nasa/sample_app#220

*sch_lab*
- nasa/sch_lab#156

*cFE*
- nasa/cFE#2472
- nasa/cFE#2411
- nasa/cFE#2474

*osal*
- nasa/osal#1430

Co-authored by: Joseph Hickey <[email protected]>
Co-authored by: Isaac Rowe <[email protected]>
Co-authored by: Avi Weiss <[email protected]>
@jphickey jphickey deleted the fix-219-topicids branch January 11, 2024 15:32
@dzbaker dzbaker added this to the v7.0.0 milestone Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CCB:Approved Indicates code approval by CCB

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Define and use Topic IDs instead of fixed Msg IDs

2 participants