-
Notifications
You must be signed in to change notification settings - Fork 35
Merge private-beta version as initial release #162
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
Conversation
* add build-list to build needs * upload artifacts * yaml fix * use temp cfg file for parsing with yq * name artifacts like node name * add also firmware.bin as artifact * local custom components, use common sub-configs * added mic_streaming test * added mic live streaming
* added config for rev2 board and neopixel config. * separate configs for core rev 1 and 2 * fix: wrong reference in GH-action * fix: GH-action pass changed file list to build action * added led script skeleton for va phase change triggers * fix: added missing led/versioning package include in rev2 * fix: set led-pin via substitution for rev1/2
* condition firmware build on core and include file changes * fix: missing statement * test: changing core satellite file * test: second core file has changed * test: include file has changed
* gh: upload release assets
update workflow for debugging update workflow to ensure tags are fetched update workflow debug tests add run tree to check state fix name for upload artifact add tag to repository after successful build update release action update action trigger update workflow update needs and references fix branch name remove draft creation condense workflow. #major minor updates to tagging remove feature branch from workflow triggers add step for documentation dispatch update workflow dispatch uri
* merge satellite1 component from sandbox * fix: wrong pin setup call * adjusted satellite rev1,rev3 and udp_stream to use nabu_vk * bump GHA esphome version to 2024.9.2 * invert xmos_rst_pin * added a delay before disabling direct_flash_access_mode again. * fix: missed calling setup for satellite pins. * added basic pcm5122 component * test_xmos_flashing: changed xmos firmware-url to point to gh * hack: explicitly set pin40 to use GPIO matrix * hack: send spi dummy byte on setup to set clock high. * change firmware urls to use raw.githubusercontent.com instead of redirect * added firmware update button to satellite_va_core_rev3 * bump venv esphome version to 2024.9 * introduction of hat_sensors_r3 config * correct i2c address for dac * change temperature sensor to AHT10 variant * added a switch for enbling mww * swap i2s in/out pins. set number of leds to 24 * update xmos-port to status register assignment * correct bit to pin mapping for button binary sensors. * correct Mic Muted invertion * Increase SPI speed to 40Mhz * Commented out AHT10 & increased I2C to 400kHz * Add Status LED config * Add control over Media playback volume via buttons * change nabu-voice-kit repo to gnumpi fork. * disable tas2780 component for now * enabled debug version of tas2780 * fix: typo in header import * disable TAS and FUSB --------- Co-authored-by: alextrical <[email protected]>
Merging develop before working on release action
…ns-updates Feature/32 release actions updates
…ease-actions-updates Revert "Feature/32 release actions updates"
* Adding temp & humidity sensors - Including temp and humidity sensors - Also ability for customer to adjust sliders in ESPHome to offset their sensors if necessary
* Use the segments platform to have dedicated led ring component for user and system access
- Default is turned to off - Customer can uncomment the code to enable the sensor and it will just work.
- With proper SDKConfigs (thanks to Kevin @ Nabu for tips) we can now have BLE improv safely in our firmware while still having enough memory for optimal user-experience
…r main and staging (#158)
Set initial version to 0 and update release process for firmware builds
|
Very close! |
that1guy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@qnlbnsl Mainly I think we need updates to the satellite1.factory.yaml file with appropriate URLS before we merge this in.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where is this going to be hosted? Do we need some infra?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is/was intended for a developer to make his own XMOS firmware on his local machine available to the Satellite1 via http. As we are going to be able to embed the XMOS firmware we probably won't need this anymore. Do you agree?
| # Flashes Satellite1 with most recent XMOS firmware | ||
| #TODO: This needs to grab the latest firmware and not hardcoded version | ||
| #TODO: The LED ring does not show a visual status when flashing because we connect to the LEDs via XMOS and can't connect during flashing. Hrmmmmmm.... | ||
| - platform: template |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@gnumpi can we get your embedded XMOS solution in here? Or would you prefer we merge this PR first and add that later? Currently this code only updates the LEDs after the flashing is done and the animation continues until the user says the wake word or clicks the action button.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
actually the current code does NOT do any LEDs update while/after flashing. I didn't merged that part of the code as it is not working with this version and the upcoming version needs a different implementation to be able to update the LED status while flashing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As you already shipped devices with this current state of the firmware, we shouldn't change anything here.
We just keep that shipped version as our initial release and include new features via separate PRs.
| then: | ||
| - delay: 1s | ||
| # Enable stop wake word | ||
| # - lambda: id(stop).enable(); No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Created a ticket to enable this stop command. #167
config/satellite1.factory.yaml
Outdated
| name: None | ||
| id: update_http_request | ||
| # source: https://firmware.esphome.io/home-assistant-voice-pe/home-assistant-voice/manifest.json | ||
| source: https://raw.githubusercontent.com/FutureProofHomes/Documentation/refs/heads/simulating-latest-manifest-json-http-endpoint/manifests/latest/satellite1-esp32s3.manifest.json |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@qnlbnsl this is wrong URL
config/satellite1.factory.yaml
Outdated
|
|
||
| dashboard_import: | ||
| # package_import_url: github://esphome/home-assistant-voice-pe/home-assistant-voice.yaml | ||
| package_import_url: github://FutureProofHomes/Documentation/simulating-latest-manifest-json-http-endpoint/manifests/latest/satellite1.yaml |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@qnlbnsl Same here. Gotta find this URL.
| @@ -1,2 +1,2 @@ | |||
| esphome==2024.6 | |||
| esphome==2024.11.2 | |||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for catching this Mischa.
that1guy
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to make XMOS repo public, build the bin file, and update this URL to enable XMOS flashing again.
| - lambda: id(xmos_flashing_started) = true; | ||
| - script.execute: control_leds | ||
| - ota.satellite1.flash: | ||
| url: https://raw.githubusercontent.com/FutureProofHomes/Documentation/refs/heads/main/assets/firmware/xmos/${xmos_fw_version}/satellite1_firmware_fixed_delay.factory.bin |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
|
I will close this PR shortly in favor of this one: PR #166 |
* Enable fusb302b with optimized timing * set update interval of PD text sensor to never * DEBUGGING: adds printf for received and sent PD messages. * hardcoded PD version to 2.0; disabled debug printf
- Factory firmware to point to correct URLs endpoints ESPHome Dashboard, Updates, etc.
…or-launch Preparing factory firmware with appropriate URLs

No description provided.