-
Notifications
You must be signed in to change notification settings - Fork 70
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
feat: initial wasm/ts/js bindings #1079
Conversation
is |
Thats indeed the name Im going for but Im not a member of the |
I uploaded the current state to https://www.npmjs.com/package/@baszalmstra/rattler @trim21 does this help you? |
looks good. Some ideas: the |
Those should never be used, they are internal types. You should only need |
I think we also need There is no api to know if a version is a pre-release(only is-dev but not alpha/beta/rc) and there is no api to compare which version is newer |
ParseStrictness is missing from npm distribution |
I can add all of that when I find the time. But feel free to create PRs against this PR, or can we merge this already @wolfv ? |
Or should we add |
Happy to merge and iterate in PRs. I know that some other folks from @QuantStack are also potentially interested (in the solver functionality, particularly). |
What do you need this for? There are a number of higher. level methods to modify the version that can be used instead.
Conda does not define what a pre release is. There is "isDev" though |
Thanks, I didn't know that 😄 This is because when you upgrade your version with tools, normally you don't want to upgrade from "stable" (pep440 for example) to "unstable" version. If conda doesn't have "pre release" I have no more problems. |
This PR adds initial bindings for a WASM port of some functionality of rattler. This PR aims to add a framework, more bindings can be added later.
At this point only
Version
andVersionSpec
are exposed in a fashion that I think meets the standards of this library.Uses
wasm-pack
to create a WASM binary of Rust code including a javascript wrapper. This is then double wrapped in a typescript library that provides a more ergonomic API. Tests have also been added.Im not an expert in this area, feedback is welcome!
Fixes #1078.
If CI succeeds Ill do an initial upload of this package to npmjs.org. But we still need to find a good home for it on npmjs.org since rattler is already taken.