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

Is burn-train + metrics + wasm a goal? #2766

Open
AlexErrant opened this issue Feb 2, 2025 · 0 comments
Open

Is burn-train + metrics + wasm a goal? #2766

AlexErrant opened this issue Feb 2, 2025 · 0 comments

Comments

@AlexErrant
Copy link
Contributor

I believe that burn-train + wasm is a goal. Should the metrics feature work on wasm?

I'm maintaining a small fork of burn(-train) that runs on browser-wasm (ref) that uses functionality gated by metrics. In the process of upgrading to v0.16.0, I ran into two areas of interest. Note that both are related to wasm and not browser-wasm.

  1. async_channel calls recv_blocking here, which is not supported on wasm. The dependency on async-channel was added with the PR "Consistent sync/async handling, allow more functions to be async for wasm". There may be a future where it is supported, but there's currently no momentum. I deleted AsyncProcessor in my fork.

  2. metrics uses nvml-wrapper which doesn't work with wasm. If burn-train + metrics + wasm is a goal, should we gate nvml-wrapper behind a metrics-cuda flag?

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

No branches or pull requests

1 participant