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

Problem: deep_space is not maintained #133

Open
thomas-nguy opened this issue Sep 26, 2022 · 3 comments
Open

Problem: deep_space is not maintained #133

thomas-nguy opened this issue Sep 26, 2022 · 3 comments
Assignees

Comments

@thomas-nguy
Copy link
Collaborator

thomas-nguy commented Sep 26, 2022

Currently the orchestrator code is using deep_space containing custom protobuf definition but it is lagging behind with cosmos-sdk and the official rust sdk (Cosmosrs)

Using Cosmrs directly can obviously work it’s just very verbose, though it may still be preferable to the very stale fork of deep_space we’re using. Tony Arcieri advises me frequently on Ocular and sees it as a development driver for cosmrs.

Lets give it a try and replace by Ocular which seems to be a wrapper of CosmosRs

https://github.com/PeggyJV/ocular/tree/v1.0.0
https://docs.rs/ocular/1.0.0-beta/ocular/index.html

@devashishdxt
Copy link
Collaborator

I was trying to tackle this but it looks like one of the dependencies of ocular, i.e., hyperx has an upper bound on dependency versions (specifically bytes). There's an already open issue for this on their repository dekellum/hyperx#39. Looks like it'll be difficult to move to ocular until this is resolved.

Any other ideas @tomtau @thomas-nguy?

@tomtau
Copy link

tomtau commented Oct 3, 2022

There's a PR to fix it on that repo, so perhaps for the moment, just use "replace" to use the fork instead of the crates.io version?

@thomas-nguy
Copy link
Collaborator Author

linked to this issue
PeggyJV#468

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

3 participants