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 small script to check provisional markings on spec #37709

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cecille
Copy link
Contributor

@cecille cecille commented Feb 21, 2025

Right now, this is just a manually run helper script so I can check the current state of master. This is clearly not sustainable, but I wanted to put this code up so folks could at least see what I was using to generate the P diff.

If the spec changes are accepted, I will open an issue on the spec repo to implement this in CI using alchemy.

Testing

manually run against the current ballot data model files in data_model/master and against a full ToT pull with all includes from this PR: https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/11281

Right now, this is just a manually run helper script so I can check
the current state of master. This is clearly not sustainable, but
I wanted to put this code up so folks could at least see what I was
using to generate the P diff.

If the spec changes are accepted, I will open an issue on the spec
repo to implement this in CI using alchemy.
Copy link

semanticdiff-com bot commented Feb 21, 2025

Review changes with  SemanticDiff

Changed Files
File Status
  scripts/spec_xml/check_provisional_markings_on_master.py  0% smaller
  src/python_testing/matter_testing_infrastructure/chip/testing/conformance.py  0% smaller

Copy link

PR #37709: Size comparison from 0d6b2fb to 755da59

Full report (3 builds for cc32xx, stm32)
platform target config section 0d6b2fb 755da59 change % change
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 538810 538810 0 0.0
RAM 205112 205112 0 0.0
lock CC3235SF_LAUNCHXL FLASH 572966 572966 0 0.0
RAM 205360 205360 0 0.0
stm32 light STM32WB5MM-DK FLASH 459800 459800 0 0.0
RAM 141472 141472 0 0.0

@andy31415
Copy link
Contributor

@cecille please add documentation on usage and expected output.

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.

2 participants