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

Publish internal temperature to Home Assistant #1243

Merged

Conversation

giedriuslt
Copy link
Contributor

Requested here #1188 and I also think it's useful to have even if not accurate

@openshwprojects
Copy link
Owner

Thank you!

@openshwprojects openshwprojects merged commit e609f37 into openshwprojects:main May 19, 2024
9 checks passed
Copy link

🎉 This PR is included in version 1.17.589 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

openshwprojects added a commit that referenced this pull request May 24, 2024
* Fix for Issue 808

Queueing sensor state changes while device is offline and publishing them as MQTT connection is established.

* minor typo fixes

* fix network data log (#1175)

* Add watchdog for LN882 (#1179)

* Fix index page performance for LN882H (#1180)

* Use ISO time everywhere (#1181)

* Fix windows build

* Tokenizer_GetArgFloatDefault

* support dpID mult for TuyaMCU

* add self test for TuyaMCU mult

* fix incorrect naming (cosmetic change only)

* update docs

* allow auto creation of tuya dpid storage

* add requestest flag for ac dir

* Add AHT10/AHT2X/AHT30 I2C driver (#1187)

* add aht sensor driver

* disable by default

* revert comment

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* Update README.md

* add self tests for addChangeHandler Power > 60 SetChannel 10 1234

* voltage change self tests

* Update selftest_energyMeter.c

* Update README.md

* Update README.md

* allow SM16703P driver compilation on windows (self test stub)

* per pixel led self test base

* WS2812 stub for simulator

* add reverse decoding of WS2812 pixels

* more ws self test

* add a link between simulator and per pixel drv

* adding deduplication of similar messages

* code cleanup

* Update OpenLN882H

* LN882H enable or disable sleep mode while using a PowerSave (#1186)

* add second state for PowerSave with LN882H to enable sleep mode

* delayed powersave - in sta connect callback - not tested

* Moved code for delayed application of changes to PowerSave settings

* Delaying only settings if WiFi is not connected

---------

Co-authored-by: Tester23 <[email protected]>

* better pi9xel self test, fix DDP define

* adding some comments, fixing typo

* declaring variables

* Set DNS hostname for ln882h (#1189)

* pixelanim (#1192)

* px

* it animates

* h

* bridge per pixel to LED driver

* link to led driver

* fire

* fixes

* tr

* pwm freqq (#1203)

* rename pwm freq command (#1204)

* pwm freqq

* rename

* Update OpenLN882H

* silence messages on each spi send in drv_spidma.c

* Update drv_pixelAnim.c

* hide CW bar fdr SM16703P

* pixelanim speed p1, reorganize

* add shooting star

* make LED basecolors public

* add command to change LED anim

* spoeed cmd

* Remove strftime to save space (#1210)

strftime takes a lot of space and we only use simple number printing for it. Should save 8kb

* Enable calendar events for bl602 (#1208)

* respect OBK_DISABLE_ALL_DRIVERS

* fix typo pointed by Wolfieee

* fix  buffer for pixelanim

* add pixe;lanim to prtoject

* add Motion channeltype

* self test for motion

* hass discovery test for illuminance

* try BL602 with ENABLE_EXPAND_CONSTANT	1 (#1212)

* add support for 7 segment driver HD2015E (#1214)

qqq

* second fix for channel type priority

* includfe HD2015 in prj

* Update drv_tm_gn_display_shared.c

* Update drv_tm_gn_display_shared.c

* Update drv_tm_gn_display_shared.c

* cosmetic rename

* Publish file (#1221)

* fix broken DSTime change at runtime

* publishfile

* ENABLE_LITTLEFS

* fix HASS discovery with unusually long name

* add test for HASS discovery with unusually long name

* docs

* docs

* Fix unknown method handling, lock on logging (#1224)

* extend doorsensor DSTime command

* add door sensor self test

* a bit better docs

* docs

* docs

* some more docs work

* optimized pins page? - part 1

* filter

* pins page is now down to 8kb (was 90kb, then 10kb)

* Implement LittleFS for LN882H - only first basic testing from WebApp … (#1191)

* Implement LittleFS for LN882H - only first basic testing from WebApp !!!!

- need to add littlefs-source to sdk/OpenLN882H/project/OpenBeken/CMakeLists.txt

Remarks:
I had to move the import part of LittleFS

in various cmd-source files to nearer to the top of the files

otherwise I had very strange (false!!!) compiler errors regarding
"conflicting types" for equal(!) typed prototypes and functions or even variables in .h and .c files

e.g.:

/OpenBK7231T_App/sdk/OpenLN882H/project/OpenBeken/app/src/cmnds/cmd_main.c:47:6: error: conflicting types for 'g_powersave'
   47 | bool g_powersave;
      |      ^~~~~~~~~~~
In file included from /OpenBK7231T_App/sdk/OpenLN882H/project/OpenBeken/app/src/cmnds/cmd_local.h:4,
                 from /OpenBK7231T_App/sdk/OpenLN882H/project/OpenBeken/app/src/cmnds/cmd_main.c:6:
/OpenBK7231T_App/sdk/OpenLN882H/project/OpenBeken/app/src/cmnds/cmd_public.h:33:13: note: previous declaration of 'g_powersave' was here
   33 | extern bool g_powersave;
      |             ^~~~~~~~~~~
/OpenBK7231T_App/sdk/OpenLN882H/project/OpenBeken/app/src/cmnds/cmd_main.c:47:6: error: conflicting types for 'g_powersave'
   47 | bool g_powersave;
      |      ^~~~~~~~~~~

Also I had to disable other drivers, or the OTA image would not be flashed (after rebbot old version was still present)
Is there a size limit to respect for OTA? the size was well below the shown OTA-size of 0xAA000 (696320k if I calculated correct)

* Moved "bool" from local int definition to stdbool.h
So I got rid of all those "conflicting types" errors.

Thanks to @giedriuslt for pointing this out!

This commit also includes a define of "ENABLE_TEST_COMMANDS" for the littlefs tests (lfs_test<n>).

* Restored all #defines to 1
Last image loaded without any problem

* Update cmd_main.c - temporarily removed one define to be able to sync to master

Changed 
#if (defined WINDOWS) || (defined PLATFORM_BEKEN) || (defined PLATFORM_BL602) || (defined PLATFORM_LN882H)
back to
#if (defined WINDOWS) || (defined PLATFORM_BEKEN) || (defined PLATFORM_BL602)

in src/cmnds/cmd_main.c

* Update cmd_main.c

Restored change in cmd_main.c after syncing

* Update user_main.c

Changed user_main.c to allow SVM_RunThreads() for LN882H with LittleFS

* Merged updated OpenLN882H

* Disable building test commands in obk_config.h

* Also "#define ENABLE_EXPAND_CONSTANT 1" to allow more advanced scripting

* Added LN882H to some more "#if defined" lines to allow scripting ...
Added BL602 in two lines missing this platform, too

---------

Co-authored-by: openshwprojects <[email protected]>

* door sensor clear self test

* allow bl602 to use constants in publishFloat ant similar (#1229)

It was under platform define, should be fixed to proper define

* better door sensor clear self test

* test command with spaces - ok?

* fix and selftest - backlog should not return non-zero value when commands list has empty command

* Stack size for BL is not enough to execute backlog commands (#1226)

Specifically "Save, Reset SVM and run file as script thread" crashes the chip due to stack overflow

* move more test commands to... cmd_test.c

move more test commands to... cmd_test.c

* correct BMP280.c startdriver example, add bmp280.h detected flash ID output, add BME680 detection (only) (#1230)

* testBMP

* correct startdriver example 8bitaddress

* add BME680 chip ID, add BME680 chip detection

* correct BME680_CHIP_ID

* Update obk_config.h

---------

Co-authored-by: openshwprojects <[email protected]>

* extend pin cfg page: (#1228)

introduce a function te get number of channels for an IO role
in JS code generate all elements, but hide and disable unused fields. This way the should not count for POST action

* Update workflow.yaml

* Update workflow.yaml

* few more self tests for command backlog

* Update workflow.yaml (#1234)

Deprecation notice: v1, v2, and v3 of the artifact actions
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "OpenBK7231T_App_1232_merge_36e61294cc71".
Please update your workflow to use v4 of the artifact actions.
Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

* Update workflow.yaml (#1235)

both upload and download artifact should be v4...

* Update workflow.yaml

* negation method self test

* Drawers (#1232)

* drawers draft

* test

* support post as well

* Update drv_drawers.c

* ambient

* add led_ambient

* foolproof compile

* disable drivers before merge

* allow disabling Tasmota JSON API (#1236)

* allow disabling Tasmota JSON API

* test enable

* Update README.md

* add smart LED drawers project information

* Fix possible wrong index /  shorten JS code (#1238)

* extend pin cfg page:
introduce a function te get number of channels for an IO role
in JS code generate all elements, but hide and disable unused fields. This way the should not count for POST action

* Fix for possible wrong index when pin can't be PWM
shorten code

---------

Co-authored-by: openshwprojects <[email protected]>

* refresh docs

* reenable tasmota json on simulator

* add a waitfor channel self test

* add bInv to TuyaMCU

* add bInv selftest

* add mult self test for tuyamcu

* cosmetic change only - rename to dpID

* selftest vcp constants

* self test - turn off script for a user - part 1

* Publish internal temperature to Home Assistant (#1243)

* allow auto wake up of DoorSensor

* Use functional define for constant expansion (#1244)

This fixes constants in publishFloat for LN882H and publishInt for BL602 and LN882H

* Update README.md

* Fix for Issue 808

Queueing sensor state changes while device is offline and publishing them as MQTT connection is established.

* minor typo fixes

* adding deduplication of similar messages

* code cleanup

* adding some comments, fixing typo

* typo fixes

---------

Co-authored-by: giedriuslt <[email protected]>
Co-authored-by: Tester23 <[email protected]>
Co-authored-by: NonPIayerCharacter <[email protected]>
Co-authored-by: openshwprojects <[email protected]>
Co-authored-by: MaxineMuster <[email protected]>
Co-authored-by: divadiow <[email protected]>
Co-authored-by: DeDaMrAzR <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants