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

Ncsdk 4043 release notes #1854

Merged
merged 2 commits into from
Feb 14, 2020
Merged

Conversation

ru-fu
Copy link

@ru-fu ru-fu commented Feb 5, 2020

Please check and comment with additions and changes.

PDF printout:
nRF Connect SDK v1.2.0 Release Notes — nRF Connect SDK 1.1.pdf

Check here once all input for your team (Change log and Known issues) has been provided:

  • Code owners
  • Hussar - Protocol
  • Hussar - LowLevel
  • IoT
  • Aurora
  • Crypto
  • Thingy:91
  • Desktop
  • Dragoon
  • Test
  • Project management

Approve when all content for your team is included correctly.

@ru-fu ru-fu added the DNM label Feb 5, 2020
@ru-fu ru-fu added this to the 1.2.0 milestone Feb 5, 2020
@ru-fu
Copy link
Author

ru-fu commented Feb 5, 2020

@MonikaLieLarsen

doc/release-notes-1.2.0.rst Outdated Show resolved Hide resolved
doc/release-notes-1.2.0.rst Outdated Show resolved Hide resolved
doc/release-notes-1.2.0.rst Outdated Show resolved Hide resolved
@thst-nordic
Copy link
Contributor

master is ready for rebase to fix documentation build.

doc/release-notes-1.2.0.rst Outdated Show resolved Hide resolved
doc/release-notes-1.2.0.rst Outdated Show resolved Hide resolved
doc/release-notes-1.2.0.rst Outdated Show resolved Hide resolved
doc/release-notes-1.2.0.rst Outdated Show resolved Hide resolved
doc/release-notes-1.2.0.rst Outdated Show resolved Hide resolved
Other NFC samples start with "NFC:". Update to be consistent.

Signed-off-by: Ruth Fuchss <ruth.fuchss@nordicsemi.no>
Copy link
Contributor

@sigvartmh sigvartmh left a comment

Choose a reason for hiding this comment

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

DFU target , FOTA Download and AWS FOTA release notes looks good.

@kapi-no
Copy link
Contributor

kapi-no commented Feb 12, 2020

In case you want to see the full diff between the current master and v1.1.0 release, here is a link to it:
v1.1.0...master


* :ref:`asset_tracker`:

* Added high/low threshold configuration command handling for sensor data. ??? what? Added handling for commands that configure the threshold? Added command handling for threshold configurations?
Copy link
Contributor

Choose a reason for hiding this comment

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

Thresholds can be set so that only sensor data below or above certain thresholds is sent.
@jayteemo Is the support for this in the cloud UI in place? Could you add examples of commands to set thresholds, intervals and send AT commands somewhere publicly available?

Copy link
Author

Choose a reason for hiding this comment

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

Updated to "Added functionality to configure high/low thresholds for sensor data, so that only data below/above the threshold is sent to the cloud." - correct, @jtguggedal ?

Copy link
Contributor

Choose a reason for hiding this comment

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

@ru-fu that is correct.
Example JSON command:

{
  "appId": "TEMP",
  "messageType": "CFG_SET",
  "data": {
    "enable": true,
    "thresh_lo": 20,
    "thresh_hi": 50
  }
} 

The above JSON can be sent to the device through the device's Terminal card on nrfcloud.com.
The device would then only report temperatures less than 20 or greater than 50.

Copy link
Contributor

Choose a reason for hiding this comment

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

example AT command sent from the Terminal card on nrfcloud.com:
{"appId":"MODEM", "messageType":"CMD", "data":"AT+CFUN?\r\n"}
AT response is then displayed in the Terminal card.

What is the desired public place for such examples?

Copy link
Contributor

@oyvindronningstad oyvindronningstad left a comment

Choose a reason for hiding this comment

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

bootloader/fw_info/fprotect OK

doc/release-notes-1.2.0.rst Outdated Show resolved Hide resolved
doc/release-notes-1.2.0.rst Show resolved Hide resolved
@jhn-nordic jhn-nordic self-requested a review February 14, 2020 07:54
doc/release-notes-1.2.0.rst Outdated Show resolved Hide resolved
As a workaround, insert a delay (``k_sleep()``) of a few seconds after the ``printf`` on line 294 in :file:`main.c`.
* Deprecation warning: The nrf_inbuilt_key API in the :ref:`nrfxlib:bsdlib` will be removed in a future release.
A replacement library that wraps the AT commands for ``AT%CMNG`` will be available in the |NCS|.
* The :ref:`asset_tracker` sample might show up to 2.5 mA current consumption in idle mode with ``CONFIG_POWER_OPTIMIZATION_ENABLE=y``.
Copy link
Contributor

Choose a reason for hiding this comment

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

@jorgenmk is this still valid?

doc/release-notes-1.2.0.rst Outdated Show resolved Hide resolved
doc/release-notes-1.2.0.rst Outdated Show resolved Hide resolved
Add the release notes for the new release.

Signed-off-by: Ruth Fuchss <ruth.fuchss@nordicsemi.no>
@ru-fu ru-fu removed the DNM label Feb 14, 2020
@carlescufi
Copy link
Contributor

@ru-fu we are still missing hussar protocol and PMT?

@ru-fu
Copy link
Author

ru-fu commented Feb 14, 2020

@ru-fu we are still missing hussar protocol and PMT?

No, I think we're good to merge. :)

@carlescufi carlescufi merged commit 9f80e17 into nrfconnect:master Feb 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.