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

doc: introduction: distinguishing NCS features #12219

Merged
merged 1 commit into from
Oct 9, 2023

Conversation

greg-fer
Copy link
Contributor

@greg-fer greg-fer commented Sep 1, 2023

Edited the introduction page to include distinguishing features similar to the section in Zephyr docs.
NCSDK-22322.


@greg-fer greg-fer added the DNM label Sep 1, 2023
@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval. labels Sep 1, 2023
@greg-fer greg-fer added the CI-disable Disable CI for this PR label Sep 1, 2023
@NordicBuilder
Copy link
Contributor

You can find the documentation preview for this PR at this link. It will be updated about 10 minutes after the documentation build succeeds.

Note: This comment is automatically posted by the Documentation Publishing GitHub Action.

@greg-fer greg-fer changed the title [WIP] doc: introduction: proposal 1 for main page rework doc: introduction: distinguishing NCS features Sep 28, 2023
@greg-fer greg-fer removed doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Sep 28, 2023
@greg-fer greg-fer marked this pull request as ready for review September 28, 2023 12:54
The |NCS| supports a wide range of connectivity technologies, such as Ant, Bluetooth® Low Energy, Bluetooth mesh, LTE-M/NB-IoT/GPS, Matter, Sidewalk, TCP/IP, Thread, and Wi-Fi®.

**Third-party integrations**
The |NCS| allows for using third-party and Nordic products within the SDK, such as AWS, nRFCloud, memfault, and more.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The |NCS| allows for using third-party and Nordic products within the SDK, such as AWS, nRFCloud, memfault, and more.
The |NCS| provides integrations with third-party and Nordic products within the SDK, such as AWS, nRFCloud, memfault, and more.

The |NCS| is optimized for pre-certified libraries, including SoftDevice, Matter, and Thread.

**Robust connectivity support**
The |NCS| supports a wide range of connectivity technologies, such as Ant, Bluetooth® Low Energy, Bluetooth mesh, LTE-M/NB-IoT/GPS, Matter, Sidewalk, TCP/IP, Thread, and Wi-Fi®.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"In addition to connectivity technologies provided by Zephyr [list] it supports [list]".

The |NCS| offers an extensible framework for building size-optimized software for memory-constrained devices as well as powerful and complex software for more advanced devices and applications.

**Pre-certified libraries**
The |NCS| is optimized for pre-certified libraries, including SoftDevice, Matter, and Thread.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Provided pre-certified and optimized libraries.

The |NCS| integrates the Zephyr™ real-time operating system (RTOS) and a wide range of complete applications and samples.

**Extensible framework**
The |NCS| offers an extensible framework for building size-optimized software for memory-constrained devices as well as powerful and complex software for more advanced devices and applications.
Copy link
Contributor

@wbober wbober Sep 29, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd remove this. Let's be concrete in this list.

The |NCS| includes middleware such as CoAP, MQTT, LwM2M, various libraries, hardware drivers, Trusted Firmware-M for security, and a secure bootloader (MCUboot).

**Based on Zephyr**
The |NCS| integrates the Zephyr™ real-time operating system (RTOS) and a wide range of complete applications and samples.
Copy link
Contributor

@wbober wbober Sep 29, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put as the first point. Refer to the distinguishing features of Zephyr.


The |NCS| supports :ref:`Microsoft Windows, Linux, and macOS <requirements>` for development.
**Advanced middleware and security**
The |NCS| includes middleware such as CoAP, MQTT, LwM2M, various libraries, hardware drivers, Trusted Firmware-M for security, and a secure bootloader (MCUboot).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In addition to middleware provided by Zephyr, adds extra libraries such as [list only these that are not available upstream].

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added some, but would appreciate a more thorough look at this.

@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval. labels Oct 2, 2023
@greg-fer greg-fer force-pushed the doc_new_intro_page_ver1 branch 2 times, most recently from 9a9ecb0 to e8f1ede Compare October 2, 2023 14:48
@greg-fer
Copy link
Contributor Author

greg-fer commented Oct 4, 2023

@wbober , I won't request merge until your sign-off on this one (when you're back).

@wbober wbober self-requested a review October 9, 2023 08:52
@greg-fer greg-fer removed DNM doc-required PR must not be merged without tech writer approval. changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. labels Oct 9, 2023

**Robust connectivity support**
The |NCS| supports a wide range of connectivity technologies.
In addition to connectivity technologies :ref:`provided by Zephyr <zephyr:connectivity>`, such as Bluetooth® Low Energy, IPv6, TCP/IP, UDP, LoRa and LoRaWAN, the |NCS| supports Ant, Bluetooth mesh, Find My, HomeKit, LTE-M/NB-IoT/GPS, Matter, Sidewalk, Thread, and Wi-Fi®, among others.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
In addition to connectivity technologies :ref:`provided by Zephyr <zephyr:connectivity>`, such as Bluetooth® Low Energy, IPv6, TCP/IP, UDP, LoRa and LoRaWAN, the |NCS| supports Ant, Bluetooth mesh, Find My, HomeKit, LTE-M/NB-IoT/GPS, Matter, Sidewalk, Thread, and Wi-Fi®, among others.
In addition to connectivity technologies :ref:`provided by Zephyr <zephyr:connectivity>`, such as Bluetooth® Low Energy, IPv6, TCP/IP, UDP, LoRa and LoRaWAN, the |NCS| supports ANT, Bluetooth mesh, Apple Find My, Apple HomeKit, LTE-M/NB-IoT/GPS, Matter, Amazon Sidewalk, Thread, and Wi-Fi®, among others.

In addition to that, the |NCS| adds extra libraries and drivers, such as CoAP, LwM2M, a secure bootloader (MCUboot), Mbed TLS, nRF 802.15.4 Radio Driver, nRF Security, nRF Profiler, nRF Remote procedure call libraries, and many more.

**Pre-certified libraries**
The |NCS| provides pre-certified and optimized libraries, including for SoftDevice, Matter, and Thread.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm pretty sure this syntax isn't entirely correct.

The |NCS| is out-of-tree ready and can be used for projects and applications of all sizes and levels of complexity.

**Third-party integrations**
The |NCS| provides integrations with third-party and Nordic products within the SDK, such as AWS, nRFCloud, memfault, and more.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The |NCS| provides integrations with third-party and Nordic products within the SDK, such as AWS, nRFCloud, memfault, and more.
The |NCS| provides integrations with third-party and Nordic products within the SDK, such as AWS, nRF Cloud, Memfault, and more.

@github-actions github-actions bot added changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. doc-required PR must not be merged without tech writer approval. labels Oct 9, 2023
Edited the introduction page to include distinguishing features
similar to the section in Zephyr docs.
NCSDK-22322.

Signed-off-by: Grzegorz Ferenc <[email protected]>
@rlubos rlubos merged commit a30d044 into nrfconnect:main Oct 9, 2023
12 checks passed
@greg-fer greg-fer deleted the doc_new_intro_page_ver1 branch October 9, 2023 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-entry-required Update changelog before merge. Remove label if entry is not needed or already added. CI-disable Disable CI for this PR doc-required PR must not be merged without tech writer approval.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants