-
Notifications
You must be signed in to change notification settings - Fork 34
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
Next steps #72
Comments
@MabezDev @jessebraham @bjoernQ @JurajSadel @bugadani @Vollbrecht - Please forward to others who might be interested @lulf @Dirbaio - Topic might be interesting to you, given the existence of |
Currently the wifi trait is the primary API for esp-wifi ... we want to change that. If we do that probably very few of our users will need the wifi trait anymore. In bare-metal we don't implement any of the other traits currently. I'm not sure about the OTA trait - currently our users need to do things manually since there is no ready-made implementation. We have an (outdated) example somewhere how to do that. Once we implement such a thing, we might implement the OTA trait (or maybe not) Given embedded-svc shouldn't be the primary API for other crates it would only be useful when someone wants to write portable code but I'm not sure how realistic that is in reality. Maybe the traits like http,ws and mqtt might be useful for libraries but probably not wifi, eth and ota? I could imagine storage would also be something that could be useful in libraries but as you said maybe not the way it is currently. I have no idea how much effort it will be to maintain this in future. If we expect a lot of effort which is better spent on other things, I guess everyone could live with option A. Otherwise splitting it up into micro-crates sounds good since it's really a collection of very unrelated traits - but it will most likely increase the maintenance effort and give us nothing but making things look a bit more clean - so not sure if it's worth it. I agree that an API which relies on another crate ( TL;DR I don't have a strong opinion here. But given "It was not a smashing success" and regardless how much effort it is - there will be some effort - it might not be worth it long term. |
I think so far, the WiFi trait is the most useful, and also the most likely to be used across different vendors. I think it could quite easily be implemented for the rp2040 with WiFi for example. I guess the question is why it hasn't already. Are the traits too esp specific that implementing other targets is difficult (IMO no)? Are we not doing enough to promote the crate and no one other than esp-rs users know about it? Maybe there isn't a need for this abstraction layer? It would be very interesting to hear the opinions of someone from outside of the esp-rs ecosystem. Regarding the other traits, maybe we could take a similar approach to getting ehal to 1.0, strip it back completely to the "known" valuable traits, and incrementally add stuff when there is demand for it. Heapless probably needs to be removed from the API, unless we see a heapless 1.0 on the horizon, it's probably worth exploring what some of the traits look like without heapless. It might be possible to implement scanning with an iterator approach instead of the alloc/heapless way, for example. IMO embedded-svc is still useful, and in the same way, you can use ehal to switch from rp to esp or vice versa with relative ease you could conceivably do the same with embedded-svc for the networking part of an application. It could also provide a platform to create a generic captive portal crate that could work on anything that implemented at least the WiFi trait (there would also be some HTTP things involved here, but you get my drift, without this layer there will be X number of captive portal implementations). |
I don't think so. Some might be, but I've put so much effort to make - say - the HTTP threads as generic as possible - yet - as efficient as possible for embedded scenarios that I kind of doubt this. For Websockets, not sure it can get any simpler than what's in there already. For the MQTT client... I know the traits are implementable for the ESP IDF MQTT client (obviously) and for
I think so. It does not have enough visibility for sure.
That's my biggest concern indeed. Perhaps network layer 5 ("app layer") is simply not the place where we should put abstractions for the rust embedded world. Sure, it had been successful in other domains, but these are as far from Rust embedded as it gets. Here's my recent thinking: Say, the likes of edge-net, pico-serve and rust-mqtt become popular. (To use those on anything else than a hard-coded embassy-net dependency, they might need a rich, functioning So again, assuming the above become successful, why would I consider any alternatives when targeting Embedded? And how often would I switch between the alternatives? And are there any meaningful alternatives in the first place or are we solving a non-existent problem?? I mean:
Where I'm going is that at least for the Networking traits, perhaps we should standardize on layer 3/4 instead of the application layer? (It is another topic that this ain't going easy, either, and I've heard concerns that even on layer 3/4 we are trying to standardize the non-standardizable, which is another way to say it is too much work, I guess.)
Tell me about it. :) But... crickets.
Yes. Or taking
As per above, for the networking traits specifically not sure the app layer is the right place anymore. For example, you don't need app layer abstractions to write "a" captive portal plus some minimal HTTP server. You need network/transport layer. Abstracting the captive portal itself is app layer, but then again and as per above, do we really need this? (Also, not sure an already-useful captive portal implementation needs to interface with layer 2, unless you also want to automate this aspect of operating a captive portal in the captive portal crate itself of course.) To summarize - my feedback is probably not getting us anywhere, but wanted to share a bigger context as to why I'm raising the topic of what to do with the EDIT: rephrased and fixed the network layers numbering, as I managed to make a mess with the numbers. :) |
The above speech does not cover the |
I added this issue as an topic on today's |
Can you share a link to the mtg here? |
matrix link to the discussion: https://matrix.to/#/!BHcierreUuwCMxVqOf:matrix.org/$5RsNCpr-nWBcIMK1TfUUruuFFBmhrl4szVcI_q3XX3E?via=matrix.org&via=catircservices.org&via=tchncs.de |
Sorry it took a while to respond, but I've had to think about this for a while. I really appreciate the effort that has gone into
Now consider other programming language ecosystems, I've never seen a HTTP or MQTT 'interface type' in Java or Go, even though there are multiple clients in the wild. That being said, I think perhaps |
Appreciate the feedback! Comments inline.
100%
Indeed. What is concerning for me is that currently
I think folks are underestimating the roles of the factories here, where IMO these are kind of the core value of
Indeed. The higher you climb in the network stack, the less likely you'll see useful / used abstractions.
A nit: I tend to disagree w.r.t. the Servlet API. It is created precisely to allow one to create cross-platform and HTTP-server- agnostic HTTP handling code. Which is what
Let me check this.
A nit: the But yeah, the higher in the network stack, the less reusability, I guess.
So either we'll retire them, or perhaps better yet - let them wane for a few months (years?) and either the need arises, or we retire them by then for real! |
Status Quo
With the latest release,
embedded-svc
lost quite some weight. Following traits (did anybody know about them??) no longer there:asyncify
adaptors which are now merged intoesp-idf-svc
directlyWhat is still in here?
What the future looks like?
embedded-hal
is monolitic modulo the blocking-async splitheapless
from the APIs? Might result in less ergonomic traitsThe text was updated successfully, but these errors were encountered: