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

run compile jobs with platformio environment settings #278

Merged
merged 8 commits into from
Jun 9, 2023

Conversation

sidey79
Copy link
Contributor

@sidey79 sidey79 commented Apr 23, 2023

Updated hardware attribute:
RFD-FHEM/RFFHEM#1172

run compile jobs with platformio environment files
@sidey79 sidey79 force-pushed the build_via_platformio branch 6 times, most recently from fd3791c to 4729a10 Compare April 23, 2023 11:58
@sidey79 sidey79 marked this pull request as ready for review April 23, 2023 13:36
@sidey79 sidey79 force-pushed the build_via_platformio branch 20 times, most recently from a4c6494 to d4837d8 Compare April 23, 2023 22:04
@RFD-FHEM RFD-FHEM deleted a comment from github-actions bot Apr 23, 2023
@RFD-FHEM RFD-FHEM deleted a comment from github-actions bot Apr 23, 2023
@elektron-bbs
Copy link
Contributor

elektron-bbs commented Apr 26, 2023

Mhmm, bis auf die bei allen Dateinamen fehlenden Versionsnummern sehe ich keinen Unterschied:

SIGNALDuino_nano328_3.5.0.hex
SIGNALDuino_nanocc1101_3.5.0.hex
SIGNALduino_nano328.hex
SIGNALduino_nanoCC1101.hex

Für den "promini" müssten wir die Attributliste anpassen. Da steht bisher nur eine Variante zur Auswahl.

Das kleine oder große "d" bei "...duino" spielt ja wahrscheinlich keine Rolle?

Wir haben noch ein Hardware-Attribut "miniculCC1101" im FHEM. Das ist aber wahrscheinlich auch nur ein Arduino Nano?

Versions Erstellung für Tags korrigiert
@sidey79
Copy link
Contributor Author

sidey79 commented Apr 27, 2023

Mhmm, bis auf die bei allen Dateinamen fehlenden Versionsnummern sehe ich keinen Unterschied:

SIGNALDuino_nano328_3.5.0.hex
SIGNALDuino_nanocc1101_3.5.0.hex
SIGNALduino_nano328.hex
SIGNALduino_nanoCC1101.hex

Tia, manchmal sind last minute Änderungen nicht problemlos.
Schau ich mir an, für die getaggte Version habe ich den Fehler schon korrigiert, mal schauen wieso es Pull Requests nicht klappt.

Für den "promini" müssten wir die Attributliste anpassen. Da steht bisher nur eine Variante zur Auswahl.
Das kleine oder große "d" bei "...duino" spielt ja wahrscheinlich keine Rolle?

Wir haben noch ein Hardware-Attribut "miniculCC1101" im FHEM. Das ist aber wahrscheinlich auch nur ein Arduino Nano?

Die bisher erstellten Versionen Minicul und promini sind für einen auf 8Mhz Laufenden atmega328 compiliert.
https://github.com/damianmelson/miniCUL-433MHz

Irgendwo gab es auch mal was mit 16 MHz.

@elektron-bbs
Copy link
Contributor

Eine Firmware für den miniCUL sehe ich hier https://github.com/RFD-FHEM/SIGNALDuino/releases/tag/untagged-825c87b5d9aaccc683a9 aber nicht.
Laut Stromlaufplan vom miniCUL ist dort ein Quarz mit 8 MHz bestückt. Außerdem läuft er mit 3,3 Volt. Da macht eine 16-MHz-Variante wenig Sinn.

In der platformio.ini sind die Varianten für den promini @ 16 MHz eigentlich schon drin:

[env:pro_promini_16MHz_CC1101]
; Arduino Pro or Pro Mini - Atmel ATmega328 running at 16MHz
; RAM:   [====      ]  45.0% (used 921 bytes from 2048 bytes)
; Flash: [========  ]  80.8% (used 24820 bytes from 30720 bytes)
platform = atmelavr
board = pro16MHzatmega328
monitor_speed = 57600
framework = arduino
monitor_port = ${env.monitor_port}
upload_port = ${env.upload_port}
build_flags=-D OTHER_BOARD_WITH_CC1101=1 

[env:pro_promini_16MHz]
; Arduino Pro or Pro Mini - Atmel ATmega328 running at 16MHz
; RAM:   [====      ]  42.1% (used 862 bytes from 2048 bytes)
; Flash: [=======   ]  70.2% (used 21554 bytes from 30720 bytes)
platform = atmelavr
board = pro8MHzatmega328
monitor_speed = 57600
framework = arduino
monitor_port = ${env.monitor_port}
upload_port = ${env.upload_port}
build_flags=

@sidey79
Copy link
Contributor Author

sidey79 commented Apr 27, 2023

Eine Firmware für den miniCUL sehe ich hier https://github.com/RFD-FHEM/SIGNALDuino/releases/tag/untagged-825c87b5d9aaccc683a9 aber nicht. Laut Stromlaufplan vom miniCUL ist dort ein Quarz mit 8 MHz bestückt. Außerdem läuft er mit 3,3 Volt. Da macht eine 16-MHz-Variante wenig Sinn.

Also die promini cc1101 8 MHz Version läuft wohl auch auf dem miniCUL nehme ich an.

Die 16 MHz Variante läuft halt auf jedem proMini der mit 16 MHz läuft.
Irgendwer muss da ja Mal Bedarf gehabt haben.

@HomeAutoUser
Copy link
Contributor

Also die promini cc1101 8 MHz Version läuft wohl auch auf dem miniCUL nehme ich an.

Die 16 MHz Variante läuft halt auf jedem proMini der mit 16 MHz läuft. Irgendwer muss da ja Mal Bedarf gehabt haben.

ProMini mit 8Mhz könnte ich gewesen sein.

@sidey79 sidey79 force-pushed the build_via_platformio branch from 28294b8 to c3ecc03 Compare April 28, 2023 22:46
name archive same as firmware file
@sidey79 sidey79 force-pushed the build_via_platformio branch from c3ecc03 to b5d8593 Compare April 28, 2023 22:50
@sidey79
Copy link
Contributor Author

sidey79 commented Apr 28, 2023

Die Versionnummer sind nun in dem Dateinamen, sowie auch das Datum wenn es kein Release ist.

Wie sollen wir nun mit promini / nanocul verfahren=

@HomeAutoUser
Copy link
Contributor

Wie sollen wir nun mit promini / nanocul verfahren=

Ich lese nur via Handy mit derzeit aber denke das „Problem“ erkannt zu haben.

Wäre es nicht Sinnvoll jeweils die Möglichen Varianten zu generieren? Wenn ich einen ProMini mit 8Mhz betreibe und es diesen auch mit 16Mhz gibt? Wenn die Hardware / PIN identisch sind, könnte man sich die einer Version sparen aber sollte dem User in der Commandref geben einen Hinweis diesbezüglich zu lesen ;)

@sidey79
Copy link
Contributor Author

sidey79 commented Apr 29, 2023

Wäre es nicht Sinnvoll jeweils die Möglichen Varianten zu generieren? Wenn ich einen ProMini mit 8Mhz betreibe und es diesen auch mit 16Mhz gibt? Wenn die Hardware / PIN identisch sind, könnte man sich die einer Version sparen aber sollte dem User in der Commandref geben einen Hinweis diesbezüglich zu lesen ;)

Die Firmware mit 8Mhz nützt nichts für den mit 16 MHz. :(
Derzeit wird ja promini 8Mhz und 16Mhz erzeugt.
Die Frage ist ja eher, wie wir mit den geänderten Hardware Bezeichnungen umgehen.
Früher haben wir eine als miniCul deklariert. Jetzt nennt sich das ganze pro_mini_8Mhz.

@sidey79
Copy link
Contributor Author

sidey79 commented Apr 29, 2023

Hab noch mal geschaut, für den miniCUL braucht es eine andere PIN Definition.
Dann generieren wir diese Version Zusätzlich zum ProMini 8Mhz und 16Mhz.

@elektron-bbs
Copy link
Contributor

Bisher haben wir folgende Hardware-Attribute im SIGNALduino-Modul (ohne ESPxxx, MAPLExxx und radino):

  • nano328
  • nanoCC1101
  • miniculCC1101
  • promini

Den promini müssten wir ersetzen durch folgende Varianten:

  • promini8mhz
  • promini16mhz
  • promini8mhzCC1101
  • promini16mhzCC1101

- Added  env for minicul
@sidey79
Copy link
Contributor Author

sidey79 commented Apr 29, 2023

@elektron-bbs
@HomeAutoUser

Ich habe die Hardware Attribute überarbeitet und somit auch die Namen der Firmwaredateien

@sidey79
Copy link
Contributor Author

sidey79 commented Apr 30, 2023

@elektron-bbs

'd' habe ich durch 's' ersetzt und ESP in Kleinbuchstaben geändert

@sidey79
Copy link
Contributor Author

sidey79 commented May 18, 2023

@elektron-bbs
@HomeAutoUser

Passt es jetzt so aus eurer Sicht?

@elektron-bbs
Copy link
Contributor

Tja, wahrscheinlich ja, aber das sehen wir erst, wenn du Release 3.5.1 offiziell veröffentlichst.

@sidey79
Copy link
Contributor Author

sidey79 commented May 18, 2023

Tja, wahrscheinlich ja, aber das sehen wir erst, wenn du Release 3.5.1 offiziell veröffentlichst.

Tja, wahrscheinlich ja, aber das sehen wir erst, wenn du Release 3.5.1 offiziell veröffentlichst.

Okay, ich hab die 3.5.1 als prerelease veröffentlicht :)

pin platformio to 6.1.6, because 6.1.7 is incompatible with env names
@sidey79
Copy link
Contributor Author

sidey79 commented May 20, 2023

@elektron-bbs

Bei dem Job mit Release 3.5.1 hatte ich einen Denkfehler. Das lief ja nicht auf dem angepassten Stand, von daher sehen wir hier wirklich erst mehr, wenn wir es zusammengeführt haben

@sidey79 sidey79 merged commit 4d2be87 into master Jun 9, 2023
@sidey79 sidey79 deleted the build_via_platformio branch June 9, 2023 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants