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

Update platformio.ini | version adjustment #355

Merged
merged 8 commits into from
Oct 22, 2024

Conversation

HomeAutoUser
Copy link
Contributor

@HomeAutoUser HomeAutoUser commented Oct 14, 2024

  • hardware structure revised
    (better clarity for the user)
  • Espressif core update to v6.6
    (removal of fixed version information)
  • correction of spelling mistakes

issues 353

# hardware structure revised
# ESPcore update
# correction envNames
@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

# update espressif32 to v6.6.0
# revised envName
@sidey79
Copy link
Contributor

sidey79 commented Oct 14, 2024

Ich habe den Modellen extra ein "s" angehangen um die CC1101 Variante von der anderen "Simple oder Standard" unterscheidbar zu machen.

# revised documentation
# revised jobName
@HomeAutoUser
Copy link
Contributor Author

HomeAutoUser commented Oct 14, 2024

Ich habe den Modellen extra ein "s" angehangen um die CC1101 Variante von der anderen "Simple oder Standard" unterscheidbar zu machen.

Beziehst du das auf alle Kontroller?
Was definierst du als Simple bzw. Standard?

Edit: Nachdem ich mir deine Antwort noch einmal durchgelesen habe, denke ich zu verstehen was du meinst. Da würde ich denken, das es aber auch schnell zu einem Missverständnis kommen kann. Es gibt Bsp den ESP32s was einem NodeMCU https://docs.platformio.org/en/latest/boards/espressif32/nodemcu-32s.html entspricht. Wenn man diesen dann ohne CC1101 "beschriften" würde, so müsste dann das Kürzel ja ein doppeltes s sein. Vorraussetzung ich habe dich richtig verstanden.

@sidey79
Copy link
Contributor

sidey79 commented Oct 14, 2024

Wenn ich mich mal so richtig gut erinnern könnte.

Das Problem ist, wenn es eine Firmware nano328 und nano328cc1101 gibt, dann soll die nano328 ja dort installiert werden, wo kein cc1101 ist. Aber im Namen nano328cc1101 steckt nano328 auch.
Also müssen alle Firmware mit cc1101 ein cc in den Namen bekommen, quasi als Merkmal.
Und alle die keinen cc1101 haben, denen habe ich ein s als Merkmal spendiert.

Das ganze ist im Rahmen dieses commits passiert, in dem ich die Namen auch so angepasst habe, damit diese mit den erwarteten Namen im FHEM Modul übereinstimmen:

4d2be87

Irgend einen Grund muss das Kürzel gehabt haben.

@HomeAutoUser
Copy link
Contributor Author

Ich kann dich verstehen und auch das wir eine klare und eindeutige Zuordnung besitzen.
Aufgrund der Fortschreitung und Entwicklung des Projektes, denke ich, das die damaligen Namen teilweise ungünstig gewählt wurden.

Da stoße ich an den Punkt, das man alle Namen nach einem Schema haben sollte.
Derzeit gibt es auch einen nano328 und nanoCC1101 obwohl es nur ein nano wäre.

Kommen wir überhaupt noch zu einem klaren Schema bzw. Attributnamen oder ist der Zug abgefahren zwecks kompatiblität?

Ich füge mich dem ggf. auch und passe das Kleine "s" an in den Eventnamen.

# revised names to names from attribute  SIGNALduino Module
# remove some commented out content
@sidey79
Copy link
Contributor

sidey79 commented Oct 14, 2024

Da stoße ich an den Punkt, das man alle Namen nach einem Schema haben sollte. Derzeit gibt es auch einen nano328 und nanoCC1101 obwohl es nur ein nano wäre.

Kommen wir überhaupt noch zu einem klaren Schema bzw. Attributnamen oder ist der Zug abgefahren zwecks kompatiblität?

Ich glaube ein klares Schema kann nicht ohne Verluste erreicht werden, da der Name der Hardware in einem Attribut angegeben. wird und wurde. Darauf baut dann auch die Suche nach kompatibler Firmware auf.
Die Firmware und somit auch die Umgebungen müssen gleich sein. Auch groß/Kleinschreibung spielt eine Rolle :(

Hier stehen die Werte der Attribute.
https://github.com/RFD-FHEM/RFFHEM/blob/8bdecbcd0e0efd1e4fd17f8d59e8609f16bfc9e8/FHEM/00_SIGNALduino.pm#L329

@HomeAutoUser
Copy link
Contributor Author

Hallo @sidey79

Ich glaube ein klares Schema kann nicht ohne Verluste erreicht werden, da der Name der Hardware in einem Attribut angegeben. wird und wurde. Darauf baut dann auch die Suche nach kompatibler Firmware auf. Die Firmware und somit auch die Umgebungen müssen gleich sein. Auch groß/Kleinschreibung spielt eine Rolle :(

Hier stehen die Werte der Attribute. https://github.com/RFD-FHEM/RFFHEM/blob/8bdecbcd0e0efd1e4fd17f8d59e8609f16bfc9e8/FHEM/00_SIGNALduino.pm#L329

Das ganze Zusammenspiel kann ich mir denken und weiß wo alles zusammenspielt angefangen von Software Attribut bis hin zu Scripten oder einfach nur EventNamen beim Compiler.

Der jetzige IST-Stand des PRs basiert auf eine Gleichheit der Attribute im FHEM Modul welche dann ebenso in PlatformIO bezeichnet sind. Somit würden wir erstmal keine Verluste machen bzw. keine Inkompatiblität herstellen.
Wollen wir diesen so durchwinken als "derzeitigen Kompromiss ohne größere Veränderungen" ?

Die damalige Erkennungsmarke "s" oder "cc1101" sind vermutlich ungünstig gewählt. Das anzupassen auf die uC-Namen bringt mehr Anpassungen als gedacht. Man kann dies natürlich als Zukunftsgedanke belassen aber dann sollte man vorher das Schema definieren um es im FHEM Modul / uC Kompiler / AActions bzw. Scripte einheitlich zu haben.

Als Softwareüberlegung wäre vielleicht ein Schönheitsfehler, das wir die uC im Modul die Reihenfolge ...promini8cc1101,promini16cc1101,promini8s,promini16s,...
auf
...promini8s,promini8cc1101,promini16s,promini16cc1101,... anpassen. Somit ist das Anordnungsschema identisch wie beim esp32 / esp8266 ....

platformio.ini Outdated Show resolved Hide resolved
# back to fixed version
@sidey79 sidey79 assigned sidey79 and unassigned sidey79 Oct 19, 2024
@HomeAutoUser HomeAutoUser merged commit ade9949 into RFD-FHEM:master Oct 22, 2024
19 checks passed
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.

2 participants