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

rewrite Haier HSU07-HEA03 #1247

Merged
merged 3 commits into from
Aug 25, 2020
Merged

Conversation

siriuslzx
Copy link
Collaborator

@siriuslzx siriuslzx commented Aug 23, 2020

change Haier HSU07-HEA03 by using bit fields, and change position of bits which Fan speed uses in remote_state, and update ir_Haier_test

Ref: #1246

Copy link
Owner

@crankyoldgit crankyoldgit left a comment

Choose a reason for hiding this comment

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

Looks good so far. I'll take another look in the morning.

src/ir_Haier.h Outdated Show resolved Hide resolved
src/ir_Haier.cpp Outdated Show resolved Hide resolved
src/ir_Haier.cpp Outdated Show resolved Hide resolved
src/ir_Haier.h Outdated
Comment on lines 295 to 296
// static uint16_t getTime(const uint8_t ptr[]);
// static void setTime(uint8_t ptr[], const uint16_t nr_mins);
Copy link
Owner

Choose a reason for hiding this comment

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

Rather than commenting them out, remove them.

src/ir_Haier.cpp Outdated Show resolved Hide resolved
Copy link
Owner

@crankyoldgit crankyoldgit left a comment

Choose a reason for hiding this comment

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

As usual, LGTM, just a handful of nitpick/style comments.
Update/resolve all of those and we can merge away.

src/ir_Haier.cpp Outdated Show resolved Hide resolved
src/ir_Haier.cpp Outdated Show resolved Hide resolved
@crankyoldgit crankyoldgit self-requested a review August 25, 2020 12:23
Copy link
Owner

@crankyoldgit crankyoldgit left a comment

Choose a reason for hiding this comment

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

LGTM. Again, thanks for all the effort. It's really great!

@crankyoldgit crankyoldgit merged commit ba2d5ca into crankyoldgit:master Aug 25, 2020
crankyoldgit added a commit that referenced this pull request Aug 31, 2020
_v2.7.10 (20200831)_

**[BREAKING CHANGES]**
- move SPIFFS to LittleFS for ESP8266 (#1182 #1226)
- Daikin176: Change & increase operating mode values. (#1233 #1235)

**[Bug Fixes]**
- TOSHIBA_AC: not turning off when using `IRac` class. (#1250 #1251)
- Haier: change position of Fan speed bits. (#1246 #1247)

**[Features]**
- Voltas: Add detailed support for Voltas A/Cs (#1238 #1248)
- Add support for Metz protocol. (#1241 #1242)
- Basic support for Voltas A/C protocol (#1238 #1243)
- Add low level bit formatting sanity checks. (#1232)

**[Misc]**
- Rewrite Airwell by using bit fields (#1254)
- Rewrite Haier YRW02 using bit fields (#1253)
- rewrite Haier HSU07-HEA03 (#1246 #1247)
- rewrite ir_Gree & ir_Midea by using bit field (#1240)
- Incorrect usage of `assert()` (#1244 #1245 #1232)
- rewrite Gree (#1210)
@crankyoldgit crankyoldgit mentioned this pull request Aug 31, 2020
crankyoldgit added a commit that referenced this pull request Aug 31, 2020
## v2.7.10 release
_v2.7.10 (20200831)_

**[BREAKING CHANGES]**
- move SPIFFS to LittleFS for ESP8266 (#1182 #1226)
- Daikin176: Change & increase operating mode values. (#1233 #1235)

**[Bug Fixes]**
- TOSHIBA_AC: not turning off when using `IRac` class. (#1250 #1251)
- Haier: change position of Fan speed bits. (#1246 #1247)

**[Features]**
- Voltas: Add detailed support for Voltas A/Cs (#1238 #1248)
- Add support for Metz protocol. (#1241 #1242)
- Basic support for Voltas A/C protocol (#1238 #1243)
- Add low level bit formatting sanity checks. (#1232)

**[Misc]**
- Rewrite Airwell by using bit fields (#1254)
- Rewrite Haier YRW02 using bit fields (#1253)
- rewrite Haier HSU07-HEA03 (#1246 #1247)
- rewrite ir_Gree & ir_Midea by using bit field (#1240)
- Incorrect usage of `assert()` (#1244 #1245 #1232)
- rewrite Gree (#1210)
@crankyoldgit
Copy link
Owner

The code changes mention have now been included in the newly released v2.7.10 of the library.

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