-
Notifications
You must be signed in to change notification settings - Fork 6.8k
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: release: Add Zephyr 3.6 highlights #68994
doc: release: Add Zephyr 3.6 highlights #68994
Conversation
2731a9d
to
76da46a
Compare
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 a great start, thanks! Perhaps we could add links to the relevant documentation for each entry?
Yes I was thinking the same, we should certainly do that for sure. And if an item doesn't have a "good enough" link for us to use, now would probably be a good time to make sure there is one, as we want the highlights to be things people can easily learn more about. |
Love it! I am working on a section in the release notes for GNSS as well :) |
Eheh, excellent! If you're bored, I would love to have a demo of cellular + gnss + lvgl, i.e. get GPS coordinates, compute the corresponding openstreetmap tile URL, fetch it and display it on the screen :) |
@kartben wanna mention something about the keyboard matrix driver? It's just a driver in a sea of drivers but it'd be nice to mention since it it does not depend on anything in particular and could be used with any platform we support. (but also happy with not mentioning on the basis of: it's just a driver) |
Just read again :-) Guess the wording of that particular bullet item might be backwards or something, hence why you maybe missed it. Please suggest a better wording if you can think of one! |
@kartben I'm game helping you make the sample work! |
@kartben sorry somehow I missed it :-) plenty good thanks for the mention |
76da46a
to
768e665
Compare
This is great, thanks @kartben . Like the idea to add docs links. |
I think it's worth a mention, I can write something up tonight if that helps and post it here as a comment. |
That would be very helpful yes, and yes I agree it's totally worth mentioning. One challenge though is that it doesn't look like we would have something (think: doc page or code sample) to point people to? If you are to write something tonight, I'd rather this would be some blurb to maybe add to the sensor shell sample explaining how one may play with the streaming API, as this would make a huge difference in the "discoverability" of this feature, including for all the folks who might not even read the release notes and could easily miss it for it's not really documented (i.e searching "streaming" gives virtually no result) |
We were going to hold off on the docs until we had better rtio support which @teburd is currently working on. So we'll probably have a big pay for documenting and advertising going into 3.7 |
@kartben sorry for the delay, @teburd any changes that you'd like to make?
|
I agree this is the time frame when I'd really like to showcase and advertise these things more. The problem today with advertising this stuff is no one can use it yet, as its basically only supported on one board on one soc. That's exactly what I'm looking to improve on in the 3.7 timeframe. Ultimately having an open source user (e.g. cognipilot with @bperseghetti) will also help tremendously build up the case for it. It's been experimental, still is experimental, and needs to expand and be proven out maybe one more release cycle before the big tada moment. |
Absolutely fine by me! So we might not need more than the "Added new streaming APIs and implemented in the ICM42688 driver." bullet point that's already in the release notes then. |
@jori-nordic @jhedberg @Thalley @cvinayak @PavelVPV ^^ |
I don't have anything specific |
Could this line go into highlights? zephyr/doc/releases/release-notes-3.6.rst Line 105 in c33743e
|
768e665
to
ea0b649
Compare
added in the latest push |
@aescolar should we mention the UART you made + the fact that HCI H4 sample + driver combo is now tested in CI? |
Thanks @jori-nordic . It is mentioned at the end on the testing & samples section |
@kartben Can we please remove the DNM label and get this merged so we can finalize the release notes? |
This adds a high-level summary of the most notable highlights of Zephyr 3.6. Signed-off-by: Benjamin Cabé <[email protected]>
bbe89a3
ea0b649
to
bbe89a3
Compare
yes! Added LTO and fixed the issues pointed by @fabiobaltieri |
Taking a first stab at adding a high-level summary of the most notable highlights of Zephyr 3.6.
Maybe we can use this PR as a way to discuss & converge towards the final list, esp. as more release notes keep coming in (pretty sure -and hopeful!- there should be at least one entry related to Bluetooth in there)?
WDYT @henrikbrixandersen @MaureenHelm & others?
Edit: "as more release notes keep coming in "... and as I make progress on the announcement blog post, as I'm sure I will be spotting more, that at least to me will be worth highlighting :)