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

time::every subscription improvements / information #352

Open
azriel91 opened this issue May 19, 2020 · 0 comments
Open

time::every subscription improvements / information #352

azriel91 opened this issue May 19, 2020 · 0 comments
Labels
feature New feature or request improvement An internal improvement

Comments

@azriel91
Copy link
Contributor

azriel91 commented May 19, 2020

Heya, happened to see that the time module is disabled on WASM. Have recently worked in this space, and learnt:

  • it's cheap to swap std::time::Instant for instant::Instant with no cost on native, and it works on WASM.
  • async-std::stream::interval works on WASM, but performance is much lower natively compared to governor, and the "unstable" feature brings in the rest of async-std, which may be a bit heavy for some users.
  • That said, async-std "just works" on both native and WASM when using instant::Instant.
  • governor doesn't yet support WASM, but that's pending Feature/34/support wasm boinkor-net/governor#35

I haven't tried tokio's interval implementation, so can't compare that.

I'm looking at using iced, and so when I have the chance I may implement this, though would be waiting for governor to have support first.

@hecrj hecrj added feature New feature or request improvement An internal improvement labels May 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request improvement An internal improvement
Projects
None yet
Development

No branches or pull requests

2 participants