Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

2023年04月27日 - Community Meeting #8062

mrengineer7777 started this conversation in General
Discussion options

Hello to all Arduino ESP32 Core Users 🚀

Meeting purpose,
we appreciate all of your contribution and help and we would like to improve your User experience on this Project. 🤩

As a part of this improvement we would like to uncover our internal Development to you, have clear and transparent planning and also listen to your requests, ideas and valuable Feedback.

To fulfill this, we would like to held Bimonthly Community Meeting where we can share main updates and also have an open discussion with you.


💥Tenth Community Meeting will take place on Thursday 27/04/2023 at 4 pm GMT+1 time zone. (1 hour length) We will use Google Meet.


We want to have some idea about how many of you would be interested in joining these Meetings so instead of sharing directly link to Online Meeting itself, it will work by invitation.

If you want to be added to upcoming Meeting and receive invitation please send an email to vojtech.bartoska@espressif.com.

If you already participated on any Community Meeting, you are automatically invited to upcoming Meeting and it's up to you if you join us again or not.


Agenda will be specified below in the comments: (feel free to add any topic or idea)

  1. Development updates & plans
  1. Community Topics
  1. Open Discussion

See you on the Meeting!

You must be logged in to vote

Replies: 14 comments 20 replies

Comment options

Hi @mrengineer7777, thanks for opening the thread, have edited the content to our usual description. I'm okay with changing the title. 👍

You must be logged in to vote
1 reply
Comment options

Invitation have been sent, feel free to add topics for discussion 💯

Comment options

Topic: Status of release 2.0.8

You must be logged in to vote
1 reply
Comment options

added to agenda

Comment options

Topic: Moving master to 3.0.0.
Currently PRs are targeting master, 2.x, or 3.0.0. It's confusing!

Discussion:
A reasonable expectation is that master should always compile and work correctly.
2.x branch has already diverged from master
3.x branch may not compile correctly and still needs a lot of library fixes

You must be logged in to vote
1 reply
Comment options

Also this will be explained

Comment options

Topic: Status of release 3.0.0

You must be logged in to vote
1 reply
Comment options

added to agenda

Comment options

Topic: issue labeling and cleanup

Overwhelming number of issues makes it difficult to find actual bugs.
I will suggest new labels and perhaps delete old ones.
Some issues are related to user code implementation and may not be bugs.
Lots of "for reference" issues. These should be added to documentation and closed out.

You must be logged in to vote
1 reply
Comment options

Added to agenda.

Update about "Type: For reference issues", we have started transfering those issues into the Documtation.

Some will be answer directly in related documentation page, like in this #8115.

Some will go into "FAQ section".

Comment options

Topic: PR cleanup

Need help here from Espressif employees! What would it take to implement or close out old PRs?

You must be logged in to vote
1 reply
Comment options

We did last Cleanup in Autumn 2022, we have left around 40 PRs. Some of them serve as a reminders, others need to be processed or investigated again. We will have another Clean-up session in May.

Let's put this at the bottom of our Agenda and if we have time, we can discuss this.

If there is any PR which anyone would like to move forward, don't hesitate to link it here, we can prioritize it.

Comment options

Topic: Communication

Please talk to us! We have several active developers in this community that know the ESP32 well enough to improve libraries, like me. However we don't necessarily know "why" things were done a certain way, so our PRs may not be acceptable for merging. Need timely feedback on PRs. Help us to help you.

A live comunications channel like Teams or Slack would be amazing. Even email would be acceptable if it doesn't get lost.
Gitter arduino-esp32 channel is used for general user issues.
Gitter esp32-arduino-devs (private) channel can be helpful. Notification of messages is poor and group chat nature means it gets ignored a lot.

You must be logged in to vote
4 replies
Comment options

Interesting idea, we are open to new suggestions, adding this to the Agenda.

Comment options

Interested in moving from Gitter to Discord, if applicable.

Comment options

mrengineer7777 Apr 26, 2023
Collaborator Author

-1 Vote for Discord. OK for gaming but not serious work.

Comment options

@mrengineer7777 I understand the sentiment - Discord is what we (adafruit) use for a fairly large community (and CircuitPython development), https://adafru.it/discord.

Do note that moving esp32-arduino-devs to Slack (free plan) has a 90-day message retention period.

Comment options

Topic: Automatic code formatting

As noted in #7975 , an automatic formatter would save time and make our code more readable.

You must be logged in to vote
1 reply
Comment options

added into agenda

Comment options

Topic: Clarify Licensing

As noted in #3121.
Licensing isn't a concern for hobbists.
Is it safe for commercial developers to use code from this repository?
What is the status of adding licensing info to headers?

You must be logged in to vote
0 replies
Comment options

You must be logged in to vote
1 reply
Comment options

mrengineer7777 Apr 17, 2023
Collaborator Author

Looking forward to hearing more about External Libraries testing

Comment options

Topic: Breaking Changes
#7942 May break existing projects
#7941 [Merged] Breaks external libraries: ESPASyncWebserver, TASMOTA, etc
#7566 [v2.0.6] PR changed partition structure. Upgrading firmware in field based on v2.0.5 to v2.0.6/7 causes issues due to changed partitions. Workaround: use a custom partition definition.

You must be logged in to vote
6 replies
Comment options

Must agree that changing all the partition tables is a significant breaking change and a bit of a puzzle. Adding Insights support should be an option only implemented when intenetionally selected by the developer.

An explanation would be good.

Comment options

mrengineer7777 Apr 21, 2023
Collaborator Author

I believe the breaking change in #7941 is the redefinition of #define F in WString.h. I would be tempted to revert the PR.

Comment options

Hi all, we'll cover #7941 and #7566 in updates about 2.0.8 release as those PRs have been already merged.

#7942 have been added as a seperate topic and we can shape it a bit as it's planned for 3.0.0 milestone. (there is still room for changes)

Comment options

@mrengineer7777 @VojtechBartoska Thank you for including #7941 in the agenda for tomorrow!

Comment options

Quick update, discussed approach for #7942 is shared in the PR, feel free to left your comments there.

Comment options

Topic: Documentation

Been using this framework for a few years now. The technical work by the maintainers and contributors is great. Biggest challenge initially and still now is the overall state of documentation.

A few points and suggestions.

1. Release Notes

A flat chronological list of all PR's included in a release is of little value.

Recommendations:

A. Group and summarize by library, changes in the release. A reference to the PR's pertaining to any given item in the summary could be included.

B. Include as a separate item, a list of all libraries and core changes (i.e. changing all partition files to include the Insights partition???), that have breaking changes. Then in the note for that library state what those breaking changes are. A great template is what is done by the Espressif-IDF team.

2. Official Documentation Pages

It needs to be much much much easier to allow people to contribute. Current method requires that every person who considers contributing to:

  • learn the Restructured Text (rst) markup language.
  • figure out an environment to create rst documents on your OS (macOS, Linux, Windows) for simple rst previews.
  • figure out how to get an environment up and running that replicates the look of the official pages.

All the above are barriers to contributing.

Recommendation:

Publish a "Getting Started" document that points to:

  • a simple guide to learning rst
  • recommended environments by platform for writing rst documents and previewing as they would be seen when finally published in the repository.

3. API Documentation

As the primary purpose of this repository is to allow Espressif to sell more devices, it must be seen as a priority to provide developers a clear way to use the framework.

Since "Arduino" is in the very name, it is logical that this repository would leverage the official Arduino documentation first. That is, each arduino-esp32 library should operate as its Arduino namesake. This means the API documentation for the arduino-esp32 version library only needs to detail enhancements (and differences necessitated by architectural differences in the SoC used by Arduino and the ESP32 variants) added by the arduino-esp32 library version.

After that, documentation would be added for each esp32-specific library.

Well, in an ideal world that would be the way it works.

Recommendation:

Given the challenge as per item 2. above, and the aversion programmers have to document their work, recommendation is use the @brief method of documenting the API in the header file of each library.

To start, make it a requirement that all PR's starting from date X must add an @brief statement for each function modified by that PR.

We used to say, "If it isn't documented, it doesn't exist." For example, there are a number of PR's that have added good functionality to a library but that addition seldom makes it's way to any document. So, after a period of time, that effort is essentially lost. Putting it into the @brief would at least allow the work to be noted and found in the future. If nothing else, it is a place to park the information until (or if) an section in the official docs pages is created.

While examples are great, they must be supported with good API documentation.

You must be logged in to vote
2 replies
Comment options

Thanks for topic proposal, have added it into the agenda.

Comment options

Comment options

Hi, agenda have been updated in the description.

You must be logged in to vote
0 replies
Comment options

I just figured out I forgot to share the slides, here we go.
10_Tenth_Community_meeting-Arduino-April_2023.pdf

You must be logged in to vote
0 replies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

AltStyle によって変換されたページ (->オリジナル) /