jackflem1
(Jackflem1)
December 8, 2022, 6:52pm
322
where is the code stored in HA if I want to change it? Or how can I get the update?
jeeftor
(Jeef)
December 8, 2022, 9:26pm
323
Submitted a patch
home-assistant:dev
← jeeftor:intellifire_challenge_patch
opened 09:24PM - 08 Dec 22 UTC
## Proposed change
<!--
Describe the big picture of your changes here to com… municate to the
maintainers why we should accept this pull request. If it fixes a bug
or resolves a feature request, be sure to link to that issue in the
additional information section.
-->
Version bump of backing library - to increase timeout on challenge request.
Here is the change:
https://github.com/jeeftor/intellifire4py/compare/2.2.1...2.2.2
Addressing this comment:
<img width="778" alt="image" src="https://user-images.githubusercontent.com/6491743/206569639-686667f2-6e71-4008-b764-24f707476d12.png">
## Type of change
<!--
What type of change does your PR introduce to Home Assistant?
NOTE: Please, check only 1! box!
If your PR requires multiple boxes to be checked, you'll most likely need to
split it into multiple PRs. This makes things easier and faster to code review.
-->
- [x] Dependency upgrade
- [x] Bugfix (non-breaking change which fixes an issue)
- [ ] New integration (thank you!)
- [ ] New feature (which adds functionality to an existing integration)
- [ ] Deprecation (breaking change to happen in the future)
- [ ] Breaking change (fix/feature causing existing functionality to break)
- [ ] Code quality improvements to existing code or addition of tests
## Additional information
- This PR fixes or closes issue: fixes #
- This PR is related to issue:
- Link to documentation pull request:
## Checklist
<!--
Put an `x` in the boxes that apply. You can also fill these out after
creating the PR. If you're unsure about any of them, don't hesitate to ask.
We're here to help! This is simply a reminder of what we are going to look
for before merging your code.
-->
- [x] The code change is tested and works locally.
- [x] Local tests pass. **Your PR cannot be merged unless tests pass**
- [x] There is no commented out code in this PR.
- [x] I have followed the [development checklist][dev-checklist]
- [x] The code has been formatted using Black (`black --fast homeassistant tests`)
- [x] Tests have been added to verify that the new code works.
If user exposed functionality or configuration variables are added/changed:
- [ ] Documentation added/updated for [www.home-assistant.io][docs-repository]
If the code communicates with devices, web services, or third-party tools:
- [x] The [manifest file][manifest-docs] has all fields filled out correctly.
Updated and included derived files by running: `python3 -m script.hassfest`.
- [x] New or updated dependencies have been added to `requirements_all.txt`.
Updated by running `python3 -m script.gen_requirements_all`.
- [x] For the updated dependencies - a link to the changelog, or at minimum a diff between library versions is added to the PR description.
- [x] Untested files have been added to `.coveragerc`.
<!--
This project is very active and we have a high turnover of pull requests.
Unfortunately, the number of incoming pull requests is higher than what our
reviewers can review and merge so there is a long backlog of pull requests
waiting for review. You can help here!
By reviewing another pull request, you will help raise the code quality of
that pull request and the final review will be faster. This way the general
pace of pull request reviews will go up and your wait time will go down.
When picking a pull request to review, try to choose one that hasn't yet
been reviewed.
Thanks for helping out!
-->
To help with the load of incoming pull requests:
- [x] I have reviewed two other [open pull requests][prs] in this repository.
[prs]: https://github.com/home-assistant/core/pulls?q=is%3Aopen+is%3Apr+-author%3A%40me+-draft%3Atrue+-label%3Awaiting-for-upstream+sort%3Acreated-desc+review%3Anone+-status%3Afailure
<!--
Thank you for contributing <3
Below, some useful links you could explore:
-->
[dev-checklist]: https://developers.home-assistant.io/docs/en/development_checklist.html
[manifest-docs]: https://developers.home-assistant.io/docs/en/creating_integration_manifest.html
[quality-scale]: https://developers.home-assistant.io/docs/en/next/integration_quality_scale_index.html
[docs-repository]: https://github.com/home-assistant/home-assistant.io
@GarrettKranz - that’s cool that you work for HHT → any chance of getting HHT to join: Introducing the Works with Home Assistant program - Home Assistant - the downside is perhaps ya’ll would have to either help maintain the lib or take it over directly … but still … it would be a nice marketing move to get a works with home assistant for the very small nerd community
1 Like
Definitely something I’d consider looking at. Thanks for bringing that up!
jeeftor
(Jeef)
December 8, 2022, 11:31pm
325
Pr was accepted - so next point release should have the 6 second update
jackflem1
(Jackflem1)
December 9, 2022, 12:18am
326
Hopefully this will correct the local access hangs or at least reduce when it happens. When it working it’s great. Hope HHT will get involved in the project, jeeflor has done a fantastic job creating and getting it to this point, but would be nice to get HHT to help fine tune it.
jackflem1
(Jackflem1)
December 9, 2022, 12:59pm
327
Just installed 2022.12.1 which has the timeout change in it.
jeeftor
(Jeef)
December 9, 2022, 2:04pm
328
It seems like the color of my error sensors has changed with the latest release:
Old
December Release
aesterling
(Aaron Esterling)
January 2, 2023, 10:10pm
329
Does this indicate the HTT server is currently offline? I can’t login to the Intellifire app, iftapi.net , etc.
jackflem1
(Jackflem1)
January 2, 2023, 11:50pm
330
My Intellifire app currently shows Error your appliance is currently offline. But local control via HA is working right now.
2 Likes
jeeftor
(Jeef)
January 5, 2023, 4:29am
331
Is anybody still using the official Alexa integration?
I currently am - because - i donno i never turned it off - also it does “cloud control”. Sometimes I get the following:
“Alexa turn on the fireplace”
“Fireplace is not responding … BEEP” and it turns on.
If that doesn’t work I’ll say “Alexa Flame On” and it calls my HA automation…
Eventually I’ll dump alexa but i like to compare the two of them at least.
tomlynch
(Tom Lynch)
January 5, 2023, 8:06pm
332
I was using it but disabled it because of the “not responding” messages. I now use Alexa through HA and it works great.
wnelson
(Will Nelson)
January 6, 2023, 5:52pm
333
Tangential - how many of you have gone through the process of replacing the ECM or the wifi module? Mine really won’t stay up more than a day or two, sometimes only hours…
aesterling
(Aaron Esterling)
January 6, 2023, 5:54pm
334
Wifi module, yes. The replacement one has been much more reliable than the original, but still has occasional issues.
1 Like
jeeftor
(Jeef)
January 9, 2023, 10:30pm
335
So my intellifire module is a TURD and just isnt working lately - but aside from that the lights just got accepted to core…
1 Like
jeeftor
(Jeef)
January 9, 2023, 10:30pm
336
I’m in the same situation as of the last week or so
jackflem1
(Jackflem1)
January 9, 2023, 10:52pm
337
Mine has been working ok, a hang up once in a while. They sent me an updated ECM module back in October that was suppose to reduce or eliminate the hangs. I think it has made a difference and reduced the times it hangs up.
1 Like
wnelson
(Will Nelson)
January 9, 2023, 11:26pm
338
Well i should probably try to get an updated ECM or WFM. In the meantime, my plan is to open the thing up to access the wall plug and put a zwave plug in there. Maybe some automation that power cycles if your integration reports the module is offline for an hour or more…
1 Like
Sorry to hear that. Are you showing any error code being reported while it is not working?
wnelson
(Will Nelson)
January 12, 2023, 6:50am
340
For me, almost always 3269….
jeeftor
(Jeef)
January 12, 2023, 1:27pm
341
Its continually an ECM Offline error…
{
"name": "",
"serial": "9CE2E834CE109D849CBB15CDDBAFF381",
"temperature": 18,
"battery": 0,
"pilot": 1,
"light": 0,
"height": 4,
"fanspeed": 0,
"hot": 0,
"power": 0,
"thermostat": 0,
"setpoint": 0,
"timer": 0,
"timeremaining": 0,
"prepurge": 0,
"feature_light": 0,
"feature_thermostat": 1,
"power_vent": 0,
"feature_fan": 1,
"errors": [
3269
],
"fw_version": "0x00030200",
"fw_ver_str": "0.3.2+hw2",
"downtime": 0,
"uptime": 382,
"connection_quality": 957745,
"ecm_latency": 42,
"ipv4_address": "192.168.1.65"
}
Like a LOT of errors
1 Like