generated from golioth/reference-design-template
-
Notifications
You must be signed in to change notification settings - Fork 0
Merge/template v2.7.1 #17
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
Open
szczys
wants to merge
32
commits into
main
Choose a base branch
from
merge/template_v2.7.1
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+106
−650
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Fixes formatting to adhere to .clang-format. Signed-off-by: Daniel Mangum <[email protected]>
Fixes formatting to adhere to .clang-format. Signed-off-by: Daniel Mangum <[email protected]>
When using LTE, we may start fetching sensor readings before a connection is established with Golioth. Readings can be shown on a local display during this period, but attempting to send to Golioth will fail. This updates the sensor reading streaming, which is just a counter in this case, to only attempt to stream readings to Golioth when connected. This mirrors the behavior of the battery monitor. Signed-off-by: Daniel Mangum <[email protected]>
Adds a log message indicating the streaming data to Golioth was skipped due to no connection established. Signed-off-by: Daniel Mangum <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
The new version of upload artifact disallows repeat upload to the same directory. Instead, artifacts should be uploaded using unique names, then gathered into a single archive using the merge and pattern parameters. - Split board name into its own step - Use board name for build, artifact names, and upload archive name Signed-off-by: Mike Szczys <[email protected]>
This is based on NCS v2.8.0 - Update async callbacks: - Remove golioth_response param - Add golioth_status and golioth_coap_rsp_code params - Replace deprecated CONFIG_EVENTFD_MAX with CONFIG_ZVFS_OPEN_MAX - Replace deprecated CONFIG_POSIX_MAX_FDS with CONFIG_ZVFS_OPEN_MAX - Use CONFIG_MBEDTLS_LEGACY_CRYPTO_C to enable support for p-384 curves Signed-off-by: Mike Szczys <[email protected]>
Prevents a situation where the device blocks all other processes while trying to constantly read from the sensors. Signed-off-by: Mike Szczys <[email protected]>
see CHANGELOG for details Signed-off-by: Mike Szczys <[email protected]>
Add ecdsa to the pip install command. This package is needed since Golioth Firmware SDK v0.16.0 (probably part of NCS v2.8.0 but unconfirmed). Unfortunately the Zephyr requirements.txt does not include this package. Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
see CHANGELOG for details Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
This commit converts README.rst to README.md. No substantive changes have been made. Signed-off-by: Mike Szczys <[email protected]>
- add overview of functionality - add a supported hardware overview - move Golioth Features above Local set up - Separate Stream from State - Add Stream example JSON - Add LightDB State Example JSON - Clarify LightDB State functionality - update OTA instructions for Packages/Cohorts/Deployments - Move pipelines higher in the document - Reverence the pipelines section from the Stream section - Move hardware variants lower in the document - Simplify the hardware variants section - rename opening section - make all headings after the first subheadings - fix mention of LightDB Stream - properly style admonitions for GitHub - add section for additional sensors/components - remove hardware variations sections - add mention of use cases and link to projects site - add section for asking questions Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
Although all boards currently supported are using nRF9160, this repository is fully compatible with the nRF9151 when config/overlays are copied over to reflect that board name. Support this and other variants by changing the IFDEF statements to CONFIG_SOC_SERIES_NRF91X as suggested by user issue: khttps://github.com/golioth/reference-design-template/issues/120 Signed-off-by: Mike Szczys <[email protected]>
This prepares the battery monitor library to support other sources of voltage measurement. Signed-off-by: Mike Szczys <[email protected]>
Add support for reading the Elixir battery voltage and percentage using the MAX17262 fuel gauge. By default the library will now use the fuel gauge if a devicetree node for the MAX17262 is enabled. If not, the library will use a voltage divider that has the devicetree path `vbatt`. The library should only be used when one of the two above cases is present for a board. Signed-off-by: Mike Szczys <[email protected]>
Remove the battery monitor code from this repository into its own repository. Add battery monitor using west.yml. Signed-off-by: Mike Szczys <[email protected]>
- Itemize supported service with links back to relevant docs - Settings: Add link directly to Device Settings menu - RPC: explicitly reference the device view tab for RPC Signed-off-by: Mike Szczys <[email protected]>
The Golioth Aludel-Mini board has been replaced by the Golioth Aludel-Elixir and is no longer available. This commit removes support and CI testing for the Mini. Signed-off-by: Mike Szczys <[email protected]>
- silence warning when neither LED nor Ostentus is present - Use IF_ENABLED instead of #ifdef to wrap LET set/unset call Signed-off-by: Mike Szczys <[email protected]>
- Move MBEDTLS_SSL_IN_CONTENT_LEN and MBEDTLS_SSL_OUT_CONTENT_LEN to nrf9160_ns.conf as it is only used by NRF_SECURITY - The network info library only supports Nordic devices. Enable it in nrf9160_ns.conf and use Kconfig to conditionally build the function calls Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
Signed-off-by: Mike Szczys <[email protected]>
see CHANGELOG for details Signed-off-by: Mike Szczys <[email protected]>
Whitespace in project name is now forbidden by west, leading to the following error: > UnboundLocalError: cannot access local variable 'context' where it is not associated with a value Signed-off-by: Mike Szczys <[email protected]>
see CHANGELOG for details Signed-off-by: Mike Szczys <[email protected]>
release template_v2.7.1
15c8501
to
d701a2f
Compare
MarkoPura
approved these changes
May 15, 2025
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.
LGTM
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.