Skip to content

Commit 6e2e3db

Browse files
Apply suggestions from code review
Co-authored-by: Divya S Pillai <91891495+divipillai@users.noreply.github.com>
1 parent 827382a commit 6e2e3db

2 files changed

Lines changed: 6 additions & 4 deletions

File tree

doc/app/at_nrfcloud.rst

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -195,9 +195,10 @@ Syntax
195195
AT#XNRFCLOUDPOS=<cell_count>,<wifi_pos>[,<MAC 1>[,<RSSI 1>],<MAC 2>[,<RSSI 2>][,<MAC 3>[...]]]
196196

197197
* The ``<cell_count>`` parameter indicates the number of cells to include in the location request.
198-
The value range is ``0`` to ``15``. A good suggested value for cellular positioning is ``4``.
198+
The value range is ``0`` to ``15``.
199+
For cellular positioning, a recommended value is ``4``.
199200
``0`` means that no cellular network information will be included in the location request.
200-
The |SM| uses the ``AT%NCELLMEAS`` command to retrieve the cellular network information, and depending on the value of ``<cell_count>``, it may use the command multiple times.
201+
The |SM| uses the ``AT%NCELLMEAS`` command to retrieve the cellular network information, and depending on the value of ``<cell_count>``, the command might be executed multiple times.
201202

202203
.. note::
203204

doc/releases/migration_notes_v2.0.0.rst

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,9 @@ The following changes are mandatory to make your application work in the same wa
2727

2828
* Changed ``<cell_pos>`` parameter to ``<cell_count>``. The meaning changes from no cell positioning, single-cell or multi-cell to the number of cells to be included in the location request.
2929
``0`` means that cellular positioning is not requested at all.
30-
* The ``AT#XNRFCLOUDPOS`` command has been updated to use the ``AT%NCELLMEAS`` command internally so the host must not use it anymore.
31-
* ``#XNRFCLOUDPOS`` notification now includes the status of the location request. The syntax has changed from:
30+
* The ``AT#XNRFCLOUDPOS`` command has been updated to use the ``AT%NCELLMEAS`` command internally, so the host must not use it anymore.
31+
* ``#XNRFCLOUDPOS`` notification now includes the status of the location request.
32+
The syntax has changed from:
3233
::
3334
#XNRFCLOUDPOS: <error>
3435
#XNRFCLOUDPOS: <type>,<latitude>,<longitude>,<uncertainty>

0 commit comments

Comments
 (0)