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

Add retransmission count to MRP TransmitEvent #37768

Merged
merged 3 commits into from
Feb 27, 2025

Conversation

tehampson
Copy link
Contributor

Testing

New additions to unit test are passing

Copy link

Review changes with  SemanticDiff

Copy link

github-actions bot commented Feb 25, 2025

PR #37768: Size comparison from ba98125 to 3807500

Full report (73 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
platform target config section ba98125 3807500 change % change
bl602 lighting-app bl602+mfd+littlefs+rpc FLASH 1096628 1096628 0 0.0
RAM 94842 94842 0 0.0
bl702 lighting-app bl702+eth FLASH 651606 651606 0 0.0
RAM 33509 33509 0 0.0
bl702+wifi FLASH 828878 828878 0 0.0
RAM 22233 22233 0 0.0
bl706+mfd+rpc+littlefs FLASH 1061274 1061274 0 0.0
RAM 32157 32157 0 0.0
bl702l contact-sensor-app bl702l+mfd+littlefs FLASH 892118 892118 0 0.0
RAM 26896 26896 0 0.0
lighting-app bl702l+mfd+littlefs FLASH 975014 975014 0 0.0
RAM 24644 24644 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 815300 815300 0 0.0
RAM 120256 120256 0 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 824220 824220 0 0.0
RAM 125352 125352 0 0.0
pump-app LP_EM_CC1354P10_6 FLASH 771080 771080 0 0.0
RAM 113724 113724 0 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 755364 755364 0 0.0
RAM 113932 113932 0 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 538842 538842 0 0.0
RAM 205112 205112 0 0.0
lock CC3235SF_LAUNCHXL FLASH 572998 572998 0 0.0
RAM 205360 205360 0 0.0
cyw30739 light CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 656429 656429 0 0.0
RAM 75324 75324 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 676289 676289 0 0.0
RAM 77964 77964 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 676289 676289 0 0.0
RAM 77964 77964 0 0.0
CYW930739M2EVB-02 unknown 2040 2040 0 0.0
FLASH 633213 633213 0 0.0
RAM 70392 70392 0 0.0
light-switch CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 616293 616293 0 0.0
RAM 71532 71532 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 635929 635929 0 0.0
RAM 74076 74076 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 635929 635929 0 0.0
RAM 74076 74076 0 0.0
lock CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 635789 635789 0 0.0
RAM 74540 74540 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 655513 655513 0 0.0
RAM 77084 77084 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 655513 655513 0 0.0
RAM 77084 77084 0 0.0
thermostat CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 612137 612137 0 0.0
RAM 68628 68628 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 631997 631997 0 0.0
RAM 71268 71268 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 631997 631997 0 0.0
RAM 71268 71268 0 0.0
efr32 lock-app BRD4187C FLASH 939536 939536 0 0.0
RAM 159920 159920 0 0.0
BRD4338a FLASH 732536 732536 0 0.0
RAM 234828 234828 0 0.0
window-app BRD4187C FLASH 1032000 1032000 0 0.0
RAM 128024 128024 0 0.0
esp32 all-clusters-app c3devkit DRAM 97312 97312 0 0.0
FLASH 1582004 1582004 0 0.0
IRAM 83820 83820 0 0.0
m5stack DRAM 116108 116108 0 0.0
FLASH 1550014 1550014 0 0.0
IRAM 117039 117039 0 0.0
linux air-purifier-app debug unknown 4720 4720 0 0.0
FLASH 2649887 2650189 302 0.0
RAM 111088 111088 0 0.0
all-clusters-app debug unknown 5528 5528 0 0.0
FLASH 5921360 5921662 302 0.0
RAM 513712 513712 0 0.0
all-clusters-minimal-app debug unknown 5424 5424 0 0.0
FLASH 5266394 5266696 302 0.0
RAM 221112 221112 0 0.0
bridge-app debug unknown 5440 5440 0 0.0
FLASH 4625036 4625338 302 0.0
RAM 199816 199816 0 0.0
camera-app debug unknown 5424 5424 0 0.0
FLASH 4671816 4672118 302 0.0
RAM 194592 194592 0 0.0
chip-tool debug unknown 6096 6096 0 0.0
FLASH 13302351 13302653 302 0.0
RAM 603392 603392 0 0.0
chip-tool-ipv6only arm64 unknown 21976 21976 0 0.0
FLASH 11495416 11495832 416 0.0
RAM 656112 656112 0 0.0
fabric-admin debug unknown 5784 5784 0 0.0
FLASH 11567249 11567551 302 0.0
RAM 603176 603176 0 0.0
fabric-bridge-app debug unknown 4688 4688 0 0.0
FLASH 4452254 4452556 302 0.0
RAM 187016 187016 0 0.0
fabric-sync debug unknown 4944 4944 0 0.0
FLASH 5569205 5569509 304 0.0
RAM 470400 470400 0 0.0
lighting-app debug+rpc+ui unknown 6152 6152 0 0.0
FLASH 5515489 5515793 304 0.0
RAM 203952 203952 0 0.0
lock-app debug unknown 5392 5392 0 0.0
FLASH 4688504 4688806 302 0.0
RAM 191144 191144 0 0.0
ota-provider-app debug unknown 4728 4728 0 0.0
FLASH 4310684 4310986 302 0.0
RAM 179832 179832 0 0.0
ota-requestor-app debug unknown 4680 4680 0 0.0
FLASH 4440810 4441112 302 0.0
RAM 184320 184320 0 0.0
shell debug unknown 4216 4216 0 0.0
FLASH 2949196 2949500 304 0.0
RAM 143672 143672 0 0.0
thermostat-no-ble arm64 unknown 9448 9448 0 0.0
FLASH 4138392 4138808 416 0.0
RAM 229016 229016 0 0.0
tv-app debug unknown 5720 5720 0 0.0
FLASH 5908117 5908421 304 0.0
RAM 593832 593832 0 0.0
tv-casting-app debug unknown 5296 5296 0 0.0
FLASH 11472829 11473133 304 0.0
RAM 718656 718656 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 907708 907708 0 0.0
RAM 142323 142323 0 0.0
nrf7002dk_nrf5340_cpuapp FLASH 901788 901788 0 0.0
RAM 124663 124663 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 846312 846312 0 0.0
RAM 141251 141251 0 0.0
nxp contact k32w0+release FLASH 585256 585256 0 0.0
RAM 70876 70876 0 0.0
mcxw71+release FLASH 600712 600712 0 0.0
RAM 63096 63096 0 0.0
light k32w0+release FLASH 611196 611196 0 0.0
RAM 70164 70164 0 0.0
k32w1+release FLASH 685680 685680 0 0.0
RAM 48584 48584 0 0.0
lock mcxw71+release FLASH 749536 749536 0 0.0
RAM 67500 67500 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 FLASH 1646188 1646188 0 0.0
RAM 211472 211472 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 FLASH 1552956 1552956 0 0.0
RAM 208288 208288 0 0.0
light cy8ckit_062s2_43012 FLASH 1438644 1438644 0 0.0
RAM 197040 197040 0 0.0
lock cy8ckit_062s2_43012 FLASH 1467516 1467516 0 0.0
RAM 224704 224704 0 0.0
qpg lighting-app qpg6105+debug FLASH 662188 662188 0 0.0
RAM 105116 105116 0 0.0
lock-app qpg6105+debug FLASH 620304 620304 0 0.0
RAM 99664 99664 0 0.0
stm32 light STM32WB5MM-DK FLASH 459832 459832 0 0.0
RAM 141472 141472 0 0.0
telink bridge-app tl7218x FLASH 665034 665034 0 0.0
RAM 90728 90728 0 0.0
contact-sensor-app tlsr9528a_retention FLASH 622078 622078 0 0.0
RAM 31488 31488 0 0.0
light-app-ota-shell-factory-data tl3218x FLASH 745218 745218 0 0.0
RAM 40396 40396 0 0.0
tl7218x FLASH 753780 753780 0 0.0
RAM 97540 97540 0 0.0
light-switch-app-ota-compress-lzma-factory-data tl7218x_retention FLASH 680850 680850 0 0.0
RAM 52192 52192 0 0.0
light-switch-app-ota-compress-lzma-shell-factory-data tlsr9528a FLASH 709412 709412 0 0.0
RAM 73400 73400 0 0.0
lighting-app-ota-factory-data tlsr9118bdk40d FLASH 600592 600592 0 0.0
RAM 138812 138812 0 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d FLASH 788820 788820 0 0.0
RAM 96388 96388 0 0.0
tizen all-clusters-app arm unknown 5076 5076 0 0.0
FLASH 1750852 1750852 0 0.0
RAM 93260 93260 0 0.0
chip-tool-ubsan arm unknown 11492 11492 0 0.0
FLASH 18983694 18983694 0 0.0
RAM 8306240 8306240 0 0.0

@andy31415
Copy link
Contributor

/gemini review

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

The changes look good overall. The addition of the retransmissionCount field to the TransmitEvent struct seems well-integrated and the unit tests have been updated accordingly. I have a minor suggestion regarding naming consistency in the test file.

Summary of Findings

  • Naming Consistency: Consider renaming forthTransmitEvent to fourthTransmitEvent in TestReliableMessageProtocol.cpp for better readability and consistency.

Assessment

The pull request introduces a new field retransmissionCount to the TransmitEvent struct, which provides valuable information for analyzing reliable message transmissions. The code changes are well-structured and include corresponding updates to the unit tests. Overall, the pull request is in good shape to be merged after addressing the minor suggestions provided below. Remember to have others review and approve this code before merging.

Copy link

github-actions bot commented Feb 26, 2025

PR #37768: Size comparison from ba98125 to 8becdc8

Increases above 0.2%:

platform target config section ba98125 8becdc8 change % change
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 538842 540646 1804 0.3
lock CC3235SF_LAUNCHXL FLASH 572998 574794 1796 0.3
cyw30739 light CYW30739B2-P5-EVK-01 FLASH 656429 658557 2128 0.3
CYW30739B2-P5-EVK-02 FLASH 676289 678417 2128 0.3
CYW30739B2-P5-EVK-03 FLASH 676289 678417 2128 0.3
CYW930739M2EVB-02 FLASH 633213 635341 2128 0.3
light-switch CYW30739B2-P5-EVK-01 FLASH 616293 618805 2512 0.4
CYW30739B2-P5-EVK-02 FLASH 635929 638441 2512 0.4
CYW30739B2-P5-EVK-03 FLASH 635929 638441 2512 0.4
lock CYW30739B2-P5-EVK-01 FLASH 635789 638293 2504 0.4
CYW30739B2-P5-EVK-02 FLASH 655513 658017 2504 0.4
CYW30739B2-P5-EVK-03 FLASH 655513 658017 2504 0.4
thermostat CYW30739B2-P5-EVK-01 FLASH 612137 614641 2504 0.4
CYW30739B2-P5-EVK-02 FLASH 631997 634501 2504 0.4
CYW30739B2-P5-EVK-03 FLASH 631997 634501 2504 0.4
esp32 all-clusters-app c3devkit DRAM 97312 98656 1344 1.4
FLASH 1582004 1589616 7612 0.5
m5stack DRAM 116108 117436 1328 1.1
FLASH 1550014 1556626 6612 0.4
linux all-clusters-app debug FLASH 5921360 5962884 41524 0.7
RAM 513712 514832 1120 0.2
all-clusters-minimal-app debug FLASH 5266394 5297596 31202 0.6
bridge-app debug FLASH 4625036 4649944 24908 0.5
shell debug FLASH 2949196 2980028 30832 1.0
RAM 143672 144344 672 0.5
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 907708 911760 4052 0.4
RAM 142323 142859 536 0.4
nrf7002dk_nrf5340_cpuapp RAM 124663 125195 532 0.4
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 846312 850424 4112 0.5
nxp contact k32w0+release FLASH 585256 587080 1824 0.3
light k32w0+release FLASH 611196 612796 1600 0.3
psoc6 all-clusters cy8ckit_062s2_43012 FLASH 1646188 1655684 9496 0.6
RAM 211472 212264 792 0.4
all-clusters-minimal cy8ckit_062s2_43012 FLASH 1552956 1562380 9424 0.6
qpg lock-app qpg6105+debug FLASH 620304 621944 1640 0.3
telink bridge-app tl7218x FLASH 665034 669192 4158 0.6
tizen all-clusters-app arm unknown 5076 5116 40 0.8
FLASH 1750852 1766136 15284 0.9
RAM 93260 93844 584 0.6
Full report (73 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
platform target config section ba98125 8becdc8 change % change
bl602 lighting-app bl602+mfd+littlefs+rpc FLASH 1096628 1096892 264 0.0
RAM 94842 94842 0 0.0
bl702 lighting-app bl702+eth FLASH 651606 651870 264 0.0
RAM 33509 33509 0 0.0
bl702+wifi FLASH 828878 829142 264 0.0
RAM 22233 22233 0 0.0
bl706+mfd+rpc+littlefs FLASH 1061274 1061538 264 0.0
RAM 32157 32157 0 0.0
bl702l contact-sensor-app bl702l+mfd+littlefs FLASH 892118 892382 264 0.0
RAM 26896 26896 0 0.0
lighting-app bl702l+mfd+littlefs FLASH 975014 975278 264 0.0
RAM 24644 24644 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 815300 817152 1852 0.2
RAM 120256 120272 16 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 824220 826072 1852 0.2
RAM 125352 125368 16 0.0
pump-app LP_EM_CC1354P10_6 FLASH 771080 772956 1876 0.2
RAM 113724 113740 16 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 755364 757240 1876 0.2
RAM 113932 113948 16 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 538842 540646 1804 0.3
RAM 205112 205128 16 0.0
lock CC3235SF_LAUNCHXL FLASH 572998 574794 1796 0.3
RAM 205360 205376 16 0.0
cyw30739 light CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 656429 658557 2128 0.3
RAM 75324 75412 88 0.1
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 676289 678417 2128 0.3
RAM 77964 78052 88 0.1
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 676289 678417 2128 0.3
RAM 77964 78052 88 0.1
CYW930739M2EVB-02 unknown 2040 2040 0 0.0
FLASH 633213 635341 2128 0.3
RAM 70392 70480 88 0.1
light-switch CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 616293 618805 2512 0.4
RAM 71532 71652 120 0.2
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 635929 638441 2512 0.4
RAM 74076 74196 120 0.2
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 635929 638441 2512 0.4
RAM 74076 74196 120 0.2
lock CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 635789 638293 2504 0.4
RAM 74540 74660 120 0.2
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 655513 658017 2504 0.4
RAM 77084 77204 120 0.2
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 655513 658017 2504 0.4
RAM 77084 77204 120 0.2
thermostat CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 612137 614641 2504 0.4
RAM 68628 68748 120 0.2
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 631997 634501 2504 0.4
RAM 71268 71388 120 0.2
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 631997 634501 2504 0.4
RAM 71268 71388 120 0.2
efr32 lock-app BRD4187C FLASH 939536 939672 136 0.0
RAM 159920 159920 0 0.0
BRD4338a FLASH 732536 732656 120 0.0
RAM 234828 234828 0 0.0
window-app BRD4187C FLASH 1032000 1032104 104 0.0
RAM 128024 128024 0 0.0
esp32 all-clusters-app c3devkit DRAM 97312 98656 1344 1.4
FLASH 1582004 1589616 7612 0.5
IRAM 83820 83820 0 0.0
m5stack DRAM 116108 117436 1328 1.1
FLASH 1550014 1556626 6612 0.4
IRAM 117039 117039 0 0.0
linux air-purifier-app debug unknown 4720 4728 8 0.2
FLASH 2649887 2651111 1224 0.0
RAM 111088 111088 0 0.0
all-clusters-app debug unknown 5528 5536 8 0.1
FLASH 5921360 5962884 41524 0.7
RAM 513712 514832 1120 0.2
all-clusters-minimal-app debug unknown 5424 5432 8 0.1
FLASH 5266394 5297596 31202 0.6
RAM 221112 221272 160 0.1
bridge-app debug unknown 5440 5448 8 0.1
FLASH 4625036 4649944 24908 0.5
RAM 199816 200144 328 0.2
camera-app debug unknown 5424 5432 8 0.1
FLASH 4671816 4673072 1256 0.0
RAM 194592 194592 0 0.0
chip-tool debug unknown 6096 6096 0 0.0
FLASH 13302351 13303567 1216 0.0
RAM 603392 603392 0 0.0
chip-tool-ipv6only arm64 unknown 21976 21976 0 0.0
FLASH 11495416 11496552 1136 0.0
RAM 656112 656112 0 0.0
fabric-admin debug unknown 5784 5784 0 0.0
FLASH 11567249 11568361 1112 0.0
RAM 603176 603176 0 0.0
fabric-bridge-app debug unknown 4688 4696 8 0.2
FLASH 4452254 4453510 1256 0.0
RAM 187016 187016 0 0.0
fabric-sync debug unknown 4944 4952 8 0.2
FLASH 5569205 5570501 1296 0.0
RAM 470400 470400 0 0.0
lighting-app debug+rpc+ui unknown 6152 6160 8 0.1
FLASH 5515489 5516785 1296 0.0
RAM 203952 203952 0 0.0
lock-app debug unknown 5392 5400 8 0.1
FLASH 4688504 4689760 1256 0.0
RAM 191144 191144 0 0.0
ota-provider-app debug unknown 4728 4736 8 0.2
FLASH 4310684 4311972 1288 0.0
RAM 179832 179832 0 0.0
ota-requestor-app debug unknown 4680 4688 8 0.2
FLASH 4440810 4442292 1482 0.0
RAM 184320 184320 0 0.0
shell debug unknown 4216 4216 0 0.0
FLASH 2949196 2980028 30832 1.0
RAM 143672 144344 672 0.5
thermostat-no-ble arm64 unknown 9448 9448 0 0.0
FLASH 4138392 4139576 1184 0.0
RAM 229016 229016 0 0.0
tv-app debug unknown 5720 5728 8 0.1
FLASH 5908117 5909349 1232 0.0
RAM 593832 593832 0 0.0
tv-casting-app debug unknown 5296 5304 8 0.2
FLASH 11472829 11474045 1216 0.0
RAM 718656 718656 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 907708 911760 4052 0.4
RAM 142323 142859 536 0.4
nrf7002dk_nrf5340_cpuapp FLASH 901788 902608 820 0.1
RAM 124663 125195 532 0.4
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 846312 850424 4112 0.5
RAM 141251 141271 20 0.0
nxp contact k32w0+release FLASH 585256 587080 1824 0.3
RAM 70876 70980 104 0.1
mcxw71+release FLASH 600712 601192 480 0.1
RAM 63096 63096 0 0.0
light k32w0+release FLASH 611196 612796 1600 0.3
RAM 70164 70268 104 0.1
k32w1+release FLASH 685680 685824 144 0.0
RAM 48584 48584 0 0.0
lock mcxw71+release FLASH 749536 750032 496 0.1
RAM 67500 67500 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 FLASH 1646188 1655684 9496 0.6
RAM 211472 212264 792 0.4
all-clusters-minimal cy8ckit_062s2_43012 FLASH 1552956 1562380 9424 0.6
RAM 208288 208560 272 0.1
light cy8ckit_062s2_43012 FLASH 1438644 1441180 2536 0.2
RAM 197040 197296 256 0.1
lock cy8ckit_062s2_43012 FLASH 1467516 1470068 2552 0.2
RAM 224704 224960 256 0.1
qpg lighting-app qpg6105+debug FLASH 662188 663468 1280 0.2
RAM 105116 105156 40 0.0
lock-app qpg6105+debug FLASH 620304 621944 1640 0.3
RAM 99664 99768 104 0.1
stm32 light STM32WB5MM-DK FLASH 459832 459840 8 0.0
RAM 141472 141472 0 0.0
telink bridge-app tl7218x FLASH 665034 669192 4158 0.6
RAM 90728 90752 24 0.0
contact-sensor-app tlsr9528a_retention FLASH 622078 622078 0 0.0
RAM 31488 31488 0 0.0
light-app-ota-shell-factory-data tl3218x FLASH 745218 745386 168 0.0
RAM 40396 40396 0 0.0
tl7218x FLASH 753780 753948 168 0.0
RAM 97540 97540 0 0.0
light-switch-app-ota-compress-lzma-factory-data tl7218x_retention FLASH 680850 681018 168 0.0
RAM 52192 52192 0 0.0
light-switch-app-ota-compress-lzma-shell-factory-data tlsr9528a FLASH 709412 709580 168 0.0
RAM 73400 73400 0 0.0
lighting-app-ota-factory-data tlsr9118bdk40d FLASH 600592 600760 168 0.0
RAM 138812 138812 0 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d FLASH 788820 788988 168 0.0
RAM 96388 96388 0 0.0
tizen all-clusters-app arm unknown 5076 5116 40 0.8
FLASH 1750852 1766136 15284 0.9
RAM 93260 93844 584 0.6
chip-tool-ubsan arm unknown 11492 11492 0 0.0
FLASH 18983694 18983958 264 0.0
RAM 8306240 8306328 88 0.0

@mergify mergify bot merged commit 4e7ce33 into project-chip:master Feb 27, 2025
70 checks passed
@@ -65,6 +67,9 @@ class ReliableMessageAnalyticsDelegate
// The outgoing message counter associated with the event. If there is no outgoing message counter
// this value will be 0.
uint32_t messageCounter = 0;
// If the eventType is kRetransmission, this value will be populated with the number of the
// retransmission attempt
Copy link
Contributor

@bzbarsky-apple bzbarsky-apple Feb 28, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This needs to document what "2" means: the first retransmission (i.e. the second packet sent), or the second retransmission (third packet sent).

gmarcosb pushed a commit to gmarcosb/connectedhomeip that referenced this pull request Mar 4, 2025
* Add retransmission count to MRP TransmitEvent

* Self review

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

Successfully merging this pull request may close these issues.

3 participants