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

R bindings? #3

Open
multimeric opened this issue Feb 24, 2023 · 6 comments
Open

R bindings? #3

multimeric opened this issue Feb 24, 2023 · 6 comments

Comments

@multimeric
Copy link

Hi, this project looks super valuable. Are you interested in the contribution of R bindings via extendr (which I am a contributor to).

@kaizhang
Copy link
Owner

@multimeric I'm not familiar with writing R bindings. This is outside my expertise.

@multimeric
Copy link
Author

I'm offering to write them myself, I'm just asking if you would be happy to have them in this repo or if they would have to live elsewhere.

@kaizhang
Copy link
Owner

kaizhang commented Feb 27, 2023

@multimeric Thanks for doing this! In that case, I think R bindings should live in its own repo. I don't have a preference for where it should be. But please note that the Rust APIs are subject to heavy changes as they are not mature.

@multimeric
Copy link
Author

Ah okay. Would you advise waiting for the APIs to stabilize, or perhaps focusing on a more stable subset of functions?

@kaizhang
Copy link
Owner

kaizhang commented Mar 9, 2023

The APIs listed here are fairly stable: https://github.com/kaizhang/anndata-rs/blob/main/anndata/src/traits.rs. If you want to start, then sticking with these APIs is probably a good idea.

@multimeric
Copy link
Author

Thanks, that's super helpful. I might hold off until layers are implemented (#4 (comment)), because they're important for my use case.

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

2 participants