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

Add early linky (G1 pilote) #44

Merged
merged 3 commits into from
Nov 14, 2024
Merged

Add early linky (G1 pilote) #44

merged 3 commits into from
Nov 14, 2024

Conversation

jipem01
Copy link
Contributor

@jipem01 jipem01 commented Oct 29, 2024

Following my experience, it seams that there is early Linky (G1 pilote 90A, ADS type 67) with protocol deviation in standard mode. May also concern G1 pilote 60A, ADS type 68.
Not tested in historic mode.
Not tested in producer mode.

SINSTS is replaced by SINST1
SMAXSN is replaced by SMAXN
SMAXSN-1 is replaced by SMAXN-1

This PR add detection of early Linky models and a way to handle automatically protocol variation based on ADS code.

Solve issue #41 and #42

Add support for experimental/pilote ADS (65 to 68)
Handle protocol deviation for early ADS. Currently ADS 67 only is concerned. 68 could also be concerned.
Deviation :
SINSTS -> SINST1
SMAXSN -> SMAXN,
SAMXSN-1 -> SMAXMN-1
@hekmon
Copy link
Owner

hekmon commented Nov 14, 2024

Je l'ai adapté par rapport à #32 j'espère ne rien avoir cassé

@hekmon hekmon merged commit 684bf75 into hekmon:main Nov 14, 2024
2 checks passed
@hekmon
Copy link
Owner

hekmon commented Nov 14, 2024

Dites moi si tout fonctionne comme il faut:
https://github.com/hekmon/linkytic/releases/tag/v3.0.0-beta4

@tomleglaunec
Copy link
Contributor

Je l'ai adapté par rapport à #32 j'espère ne rien avoir cassé

Malheureusement il y a des doublons de capteurs qui dupliquent des ID des entités concernés par cette PR ! 💥

image

Rien de grave, je vais lancer une PR corrective

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