search

crypto-org-chain / gravity-bridge

A CosmosSDK application for moving assets on and off of EVM based, POW chains
Apache License 2.0
9 stars 17 forks source link

Problem: deep_space is not maintained #133

Open thomas-nguy opened 2 years ago

thomas-nguy commented 2 years ago

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 commented 2 years ago

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 https://github.com/dekellum/hyperx/issues/39. Looks like it'll be difficult to move to ocular until this is resolved.

Any other ideas @tomtau @thomas-nguy?

tomtau commented 2 years ago

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 commented 1 year ago

linked to this issue https://github.com/PeggyJV/gravity-bridge/issues/468