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

Remove Deleted ConcentrationMeasurement Clusters #28053

Merged

Conversation

mhazley
Copy link
Contributor

@mhazley mhazley commented Jul 19, 2023

Description

Following discussion on https://github.com/CHIP-Specifications/connectedhomeip-spec/pull/7234, the previously provisional clusters will be removed from the Spec.

This amounts to:

  1. Ethylene,
  2. Ethylene Oxide,
  3. Hydrogen,
  4. Hydrogen Sulfide,
  5. Nitric Oxide,
  6. Oxygen,
  7. Sulfur Dioxide,
  8. Dissolved Oxygen,
  9. Bromate,
  10. Chloramines,
  11. Chlorine,
  12. Fecal coliform & E. Coli,
  13. Fluoride,
  14. Haloacetic Acids,
  15. Total Trihalomethanes,
  16. Total Coliform Bacteria,
  17. Turbidity,
  18. Copper,
  19. Lead,
  20. Manganese,
  21. Sulfate,
  22. Bromodichloromethane,
  23. Bromoform,
  24. Chlorodibromomethane,
  25. Chloroform
  26. Sodium

This PR removes them from the SDK.

Links

Fixes #27662

@mhazley mhazley added this to the 1.2 milestone Jul 19, 2023
@mhazley mhazley self-assigned this Jul 19, 2023
@p0fi p0fi changed the title [ConcentrationMeasurement] Remove Deleted Clusters Remove Deleted ConcentrationMeasurement Clusters Jul 19, 2023
@mergify mergify bot merged commit 02aec55 into project-chip:master Jul 20, 2023
erwinpan1 pushed a commit to erwinpan1/connectedhomeip that referenced this pull request Jul 21, 2023
* Remove unused clusters from XML

* Regenerating codegen

* Removing duplicate PICS

* Removing generated files that are no longer generated

* Remove clusters from ZAP's list

* Updated controller zap file

* Removed deleted clusters from import list

* Restyled by autopep8

* Restyled by isort

* Fixing formatting on python multi-line list

* Restyled by isort

* Fixing formatting on python multi-line list

---------

Co-authored-by: Restyled.io <[email protected]>
@mhazley mhazley deleted the remove-provisional-concentration-meas branch July 28, 2023 13:29
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.

Remove Previously Provisionally Aliased Concentration Measurement Clusters From the SDK
5 participants