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

Rename warmup_cc to warmup and then move this code into Home Assistant core. #4

Closed
foxy82 opened this issue Jan 5, 2020 · 6 comments

Comments

@foxy82
Copy link
Collaborator

foxy82 commented Jan 5, 2020

We need to split into 2 parts.

  1. The home assistant component - called warmup and added to Home Assistant repo - and removed from here.
  2. The third party warmup lib - which is this repo and name as per this issue: Rename the project? #1
@artmg
Copy link
Collaborator

artmg commented Jan 5, 2020

On 1, I was expecting to do a bit more development work on this using the custom component arrangement, until it was good and stable. If however you are confident that we're ready to move it into a branch of the main HA fork, then we can go ahead and do that already

@rct
Copy link
Collaborator

rct commented Jan 5, 2020

First, thanks to everyone who has been contributing to this.

I'd like to test and help if I can. However, at the moment, I'm not sure which repo/branch to start with and what the correct configuration is.

Given that old warmup components are broken. My suggestions are:

  1. Quick note covering the current state, ideally in the master/README to make it easier to find. Should include which repo/branch development is occurring in, where to put the files, what configuration is needed.
  2. Flesh out the minimum requirements for a release -- in progress in Initial warmup featureset to support #5
  3. What investigation and development would be required to make things stable enough so that once an HA release is done that it will be unlikely to break again soon?

@artmg
Copy link
Collaborator

artmg commented Jan 5, 2020

Hi @rct thanks for offering to help out here - I have updated the master/README of this repo ( https://github.com/ha-warmup/warmup ) to clarify this is the one users should clone to test and run. I have also updated the instructions for configuration.yaml

Hopefully once we have the component in the main HA tree then their release team would inform devs in advance of breaking changes affecting integrations, and if there are several people with access to update the code, we should not get stuck when someone is busy with real life

@rct
Copy link
Collaborator

rct commented Jan 5, 2020

@artmg - Thank you. I'm very happy to report my 4 thermostats are now showing in HA very nicely.

Note: there is a small error in the install steps warmup_cc vs warmup - fixed in #11.

@artmg
Copy link
Collaborator

artmg commented Jan 7, 2020

Many thanks for the PR, @rct
I have been collecting notes in the wiki about the different parts of this software. Hopefully your 3 points are now addressed. Feel free to expand on what's written there.

@foxy82 please take a look at what I wrote on the python library and let me know your views

artmg added a commit to artmg/warmup that referenced this issue Feb 4, 2020
as @alex-0103 mentioned in [ha-warmup#4 Dec 2019](alex-0103#4 (comment)) he was too busy, this README ONLY change mentions what was broken by core and gives a link to a forked project where issues have been fixed.
@artmg
Copy link
Collaborator

artmg commented Mar 20, 2023

The development approach for this component has settled on:

  • support use as HACS Custom Repository
  • leave python code copied within the custom_components folder until there is enough interest to warrant applying effort in creating a separate PyPI package

There is not currently enough development around this component to try and keep up with a fast-moving project like Home Assistant Code, but there should be enough interest in keeping the Home Assistant Community Store integration supported.

Please see CONTRIBUTING.md and the documentation wiki for more.

@artmg artmg closed this as completed Mar 20, 2023
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

No branches or pull requests

3 participants