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

New feature Outdoor Temperature for MiElHVAC #22345

Merged
merged 15 commits into from
Oct 23, 2024

Conversation

grzegorz914
Copy link
Contributor

Description:

  • Add Outdoor Temperature

Related issue (if applicable): fixes #

Checklist:

  • The pull request is done against the latest development branch
  • Only relevant files were touched
  • Only one feature/fix was added per PR and the code change compiles without warnings
  • The code change is tested and works with Tasmota core ESP8266 V.2.7.8
  • The code change is tested and works with Tasmota core ESP32 V.3.1.0.241015
  • I accept the CLA.

NOTE: The code change must pass CI tests. Your PR cannot be merged unless tests pass

@arendst
Copy link
Owner

arendst commented Oct 23, 2024

Is this it or are you planning to add even more "features"?

@arendst arendst merged commit 5200aca into arendst:development Oct 23, 2024
59 checks passed
@grzegorz914
Copy link
Contributor Author

I will add more, but one PR for feature.

@mamrai1
Copy link

mamrai1 commented Oct 27, 2024

Outdoor temperature is remote temperature feature?

@grzegorz914
Copy link
Contributor Author

grzegorz914 commented Oct 27, 2024

No, Remote Temperature is other thing.
Outdoor Temperature is Outside Temperature and is not supported by all devices.

@grzegorz914
Copy link
Contributor Author

Remote Temperature is a External Room Temperature Sensor Input.

@mamrai1
Copy link

mamrai1 commented Oct 27, 2024

And how do i publish the remote temperature with tasmota?

@grzegorz914
Copy link
Contributor Author

grzegorz914 commented Oct 27, 2024

Just like this: http://ip/cm?cmnd=HVACRemoteTemp 21

To working correct the temp need to be updated constantly.

@mamrai1
Copy link

mamrai1 commented Oct 27, 2024

Thanks. I tried and works but i thought if sending remote temp of 0 will revert back to internal sensor but it didn’t. Also waiting half an hour it didn’t revert to internal temp. I had to power cycle the unit

@grzegorz914
Copy link
Contributor Author

Are you try just stop sending remote temp?

@mamrai1
Copy link

mamrai1 commented Oct 27, 2024

Yes . It does not revert to internal temp (almost 30 minutes after)

@grzegorz914
Copy link
Contributor Author

Ok, please open new issue for this.

josef109 pushed a commit to josef109/Tasmota that referenced this pull request Nov 7, 2024
* Add prohibit function for MiElHVAC

Add Prohibit functions:
* Power
* Temperature
* Mode
 and all combinations of this functions
Updated VaneV names for better identify

* Fixed Compressor and Operation for MiElHVAC

Changed Widevane position name from ISEE to AUTO sam as in MELCLoud

* Revert "Fixed Compressor and Operation for MiElHVAC"

This reverts commit f0973c8.

* New feature for MiElHVAC

* Added Compressor map
* Added Operation Power in Watts
* Added Operation Energy in kWh
* Changed Widevane position name from ISEE to AUTO, displays sam as in
* Changed all map value to lover case MELCloud

* New feature for MiElHVAC

* Add device operation time in minutes

* New feature Outdoor Temperature for MiElHVAC

* Add Outdoor Temperature
josef109 pushed a commit to josef109/Tasmota that referenced this pull request Nov 10, 2024
* Add prohibit function for MiElHVAC

Add Prohibit functions:
* Power
* Temperature
* Mode
 and all combinations of this functions
Updated VaneV names for better identify

* Fixed Compressor and Operation for MiElHVAC

Changed Widevane position name from ISEE to AUTO sam as in MELCLoud

* Revert "Fixed Compressor and Operation for MiElHVAC"

This reverts commit f0973c8.

* New feature for MiElHVAC

* Added Compressor map
* Added Operation Power in Watts
* Added Operation Energy in kWh
* Changed Widevane position name from ISEE to AUTO, displays sam as in
* Changed all map value to lover case MELCloud

* New feature for MiElHVAC

* Add device operation time in minutes

* New feature Outdoor Temperature for MiElHVAC

* Add Outdoor Temperature
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants