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

ведения проекта #68

Open
Raerten opened this issue Apr 22, 2022 · 23 comments
Open

ведения проекта #68

Raerten opened this issue Apr 22, 2022 · 23 comments

Comments

@Raerten
Copy link
Contributor

Raerten commented Apr 22, 2022

Большое пожелание - вести проект с правильной работой с гитом:

  • не удалять/создавать файл с кодом - теряется история, сильно затрудняет просмотр изменений, затрудняет ведение проекта несколькими людьми
  • заливать бинарники в релиз гитхаба, а не комитить в проет (сильно разрастается репозиторий)
  • завести отдельную ветку с разработкой (dev), не заливать обновления в мастер с одинаковым номером релиза
@alutov
Copy link
Owner

alutov commented Apr 22, 2022

Вот даже пока не знаю, что сказать. Не храню на гите ничего. И раньше не доверял, а сейчас тем более. Уже приходило письмо о том, что нахожусь в подсанкционном регионе. Уже что-то там ограничено, чем не пользуюсь. Так что рубануть доступ могут на раз. Есть и еще причина. Есть русские буквы в меню мультиварки. Их тупо вбил в ascii, и, если в винде стоит выбор отображение не юникода как русских букв, все срабатывает правильно. Но гит русские буквы корежит, и потом скачать исходник я уже не могу. Потому и заливаю его каждый раз. Ну и сильно гит не изучал на предмет совместной работы. Можно сделать вот как. Постараюсь привести исходник в нормальное состояние, а потом можно будет что-то делать.

@Raerten
Copy link
Contributor Author

Raerten commented Apr 22, 2022

посмотри мой пул реквест, вроде нормально получилось закомитить с русскими буквами

а локальная копия на непредвиденный случай должна быть в любом случае )

@alutov
Copy link
Owner

alutov commented Apr 22, 2022

Уже смотрел, пока сильно не вникал, изучу еще.

@Raerten
Copy link
Contributor Author

Raerten commented Apr 22, 2022

вообще у меня уже есть желание немного перелопатить исходники- попилить на несколько файлов, форматирование в порядок привести. и мне какжется что проверка на дефайн отключение экрана не всегда в нужных местах - у меня без него не собралось

@Raerten
Copy link
Contributor Author

Raerten commented Apr 22, 2022

и да, спасибо огромное, подключил на неделе 3 девайса, все работают

@Svet98
Copy link

Svet98 commented Apr 25, 2022

желательно и файл #readm описывать с версиями в изменениях.
Текущий трудно читаем, не понятно что в каком релизе было сделано.
Особенно касается когда используется частичный функционал, например без монитора.

П.С,
Ещё раз огромное спасибо за Ваш проект!
П.П.С.
на днях закончила интеграцию в MajorDoMo. Как и у всех, с временем беда)) как только закончу ролик с инструкцией, могу скинуть линк. Инструкцию делаю для форумчан. Сама люблю много вопросов задавать))

@bvp
Copy link
Contributor

bvp commented Jun 5, 2022

Тоже есть несколько предложений по ведению проекта:

  1. Использовать clang-format для форматирования кода - можно накидать конфиг в конфигураторе
  2. Разбиение на более мелкие файлы, а не одну "простыню" кода (хотя сам этим часто грешу, но стараюсь исправляться)
  3. Возможность сборки проекта с помощью Platformio - есть рабочий конфиг

@alutov
Copy link
Owner

alutov commented Jun 6, 2022

Первый пункт поизучаю. Что касается остального. Я вообще по си не специалист, мне больше нравится ассемблер. Там все просто, все метки и процедуры можно размещать как угодно, для этого есть 2 прохода. Здесь же нужно следить, чтобы вызов процедуры был позже ее декларирования. Разбирая похожий проект olehs на ардуино с кучей файлов и перекрестных ссылок, просто начинал путаться, что где и за чем. Потому стараюсь без надобности не делить файлы. Это нехорошо, конечно, надо исправляться.) Так вот, на ардуино проект так и не пошел, вероятно из-за кривизны ble библиотеки, хотя оперативку жрал хорошо. Перейдя на чистый esp-idf и написав недостающее с нуля, получил и больше оперативки, и лучшую стабильность. Хотя на доводку при работе с несколькими соединениями ушел еще год, есть там свои тонкости. С тех пор скептически отношусь и к ардуино, и к Platformio(это тоже по сути ардуино?), и к esphome. Последний, правда, переползает на esp-idf, но, говорят, не полностью. За Platformio ничего не знаю. Если в нем можно собрать в режиме чистого esp-idf и работает нормально, выложите, а я сделаю ссылочку.

@Svet98
Copy link

Svet98 commented Jun 7, 2022

Андрей, доброго времени!
сделала два видео по Вашему шлюзу.

  • настройка r4s шлюза, прошивка и прочее по минимальному, исключительно для интеграции через MQTT
    у меня как раз esp умерла
    https://www.youtube.com/watch?v=6_Xk5rTD8FI

  • и второе видео - сама интеграция в умный дом МажорДоМо. исходники на гитхабе.
    https://www.youtube.com/watch?v=qGivkpPkW98

@alutov
Copy link
Owner

alutov commented Jun 8, 2022

Лутов Андрей) Спасибо. Надо поправить описание как-нибудь, сделаю и ссылочки на эти ресурсы.)

@Svet98
Copy link

Svet98 commented Jun 8, 2022

Извините... Поправила. Спасибо)

@ggagnidze
Copy link

А может с веб-мордой что-нибудь сделаем? Выглядит не прям современно. Может помощь какая-то нужна с этим?

@alutov
Copy link
Owner

alutov commented Apr 16, 2023

Ну совершенствовать можно бесконечно). Может и нужна помощь, сам не особо вникал в html программирование. Но уже сейчас осталось где-то 11% и 9% свободного места для esp32 и esp32c3 с 4M флешкой. И не сильно хочется нагружать есп лишней работой.

@olgasafo
Copy link

Благодарю вас за такой проект.
Я подключала приводы AM43.
Все рботает, но ввиду видимо частых запросов со стороны гейта, батарея приводов разряжается очень быстро.
Можно ли попросить вывести настройку частоты запросов в интерфейс?
Или может есть какой-то иной способ увеличить жизнь батарей?
Так же не все настройки понятны, может быть будет время сделать короткий faq по веб интерфейсу?

@alutov
Copy link
Owner

alutov commented Sep 12, 2023

Интервал запроса шлюзом состояния привода штор три минуты. В папке jpg вариант с интервалом 10 минут. Уж не знаю, поможет ли. Вообще привод идет с солнечной батареей. Правда у меня она через окно почему-то аккумулятор практически не заряжает. Взял батарею тоже на 12 вольт, но побольше, вроде бы стало лучше.

@olgasafo
Copy link

olgasafo commented Sep 12, 2023

Благодарю, попробую. Но вы говорите о интервале 3 минуты. А я в сообщениях mqtt вижу данные о уровне сигнала передатчика привода с интервалом в 3-8 секунды. Это же опрос? Возможно в этом проблема быстрого расхода?

@alutov
Copy link
Owner

alutov commented Sep 12, 2023

Передача может быть и по инициативе привода. При изменении позиции штор или освещенности, или разряде батареи. У меня при заряде в интервале 96-100% привод постоянно шлет нотификационные сообщения. Потом вроде успокаивается.

@olgasafo
Copy link

В папке jpg вариант с интервалом 10 минут. Уж не знаю, поможет ли.

Двухнедельный эксперимент с прошивкой с 3х минутным запросом и 10 минутным показали абсолютно идентичный разряд. Так что дело не в опросе устойства, а видимо в непрерывном блютус соединении.
Приводов подключенных через r4sGate хватает на 14-17 дней на стоковой батареей (с износом в 3 года), в отличие от работы стационарно (около 30-40 дней).

Вообще привод идет с солнечной батареей.

Имхо вообще не работает никак... Или солнца мало, или батарея фыфел. В один из дней даже показалось, что разряд из за неее увеличился. :-)

Вот такая вот печальная статистика.

@alutov
Copy link
Owner

alutov commented Sep 24, 2023

Кто-то мне писал, что родная солнечная батарея заряжает аккумулятор, если ее ставить до оконного стекла, то есть на улице. Мне это никак не подходило, пришлось поставить батарею помощнее. Пока тестирую.

@Niorix
Copy link

Niorix commented Oct 8, 2023

Коли тут подобие чата образовалась залезу тоже. Пару лет назад был запрос на изменение домена на Water Heater, но был отклонен т.к. нет поддержки на уровне mqtt. С тех пор ничего не поменялось? https://www.home-assistant.io/integrations/water_heater.mqtt/ не оно?

@alutov
Copy link
Owner

alutov commented Oct 8, 2023

Вроде бы оно. Только я не помню, что там станет лучше, если переделать на water heater?
Upd:
Посмотрел детальнее, что-то не впечатляет. Доступные режимы работы: [“off”, “eco”, “electric”, “gas”, “heat_pump”, “high_demand”, “performance”]. В климате как-то все понятнее.

@Niorix
Copy link

Niorix commented Oct 9, 2023

Вроде бы оно. Только я не помню, что там станет лучше, если переделать на water heater?

До этого пользовался интеграцией SkyKettle. Но с версии 2023.9.x через ble_Proxy интеграция перестала корректно работать, а напрямую не домтает. Там не используется mqtt и чайник представляет из себя Water Heater. Из плюсов для себя заметил отсутствие "непонятных" для чайников режима охлаждения и авторежима. Есть режимы кипячения, подогрева, кипечения + последующего подогрева. Хотелось такого же...

Посмотрел детальнее, что-то не впечатляет.

Да уж, похоже лучше как есть оставить.

@alutov
Copy link
Owner

alutov commented Feb 23, 2024

Все же добавил water heater для чайников. Как оказалось, пока можно определять любые режимы, а не только определенный список, Вроде ошибок в логах нет.

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

7 participants