madara icon indicating copy to clipboard operation
madara copied to clipboard

feat: create `get_snos_input(block_id: BlockId) -> Resutl<SnosInput,_>` rpc route

Open tdelabro opened this issue 9 months ago • 2 comments

Feature Request

The prover will need to be able to data out of the chain for each block it want to prove. To allow this, we will add a custom RPC get_snos_input that return the Input datastruct expected by SNOS.

Describe Preferred Solution

Add the rpc route in crates/client/rpc-core/src/lib.rs to the MadaraRpcApi trait. Implement it in crate/client.

Related Code Here is the SnosInput struct https://github.com/keep-starknet-strange/snos/blob/01ed09975cb8b0cdb878f569fe2f8aac5c9e9a7a/src/io/input.rs#L18 Import it from this crate

tdelabro avatar May 06 '24 14:05 tdelabro

Hello, can i pick this one ? @tdelabro

0xSpyC avatar May 07 '24 22:05 0xSpyC

sure. It is not an easy one tho. Don't hesitate to ask for help on our telegram channel

tdelabro avatar May 13 '24 11:05 tdelabro

There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale. Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a 👍 Because this issue is marked as stale, it will be closed and locked in 7 days if no further activity occurs. Thank you for your contributions!

github-actions[bot] avatar Jun 13 '24 00:06 github-actions[bot]

@0xSpyC are you still working on this? It has been two weeks?

tdelabro avatar Jun 14 '24 08:06 tdelabro

Yes, I'm still on it, sorry for the delay, it took me a while to figure out how the commitment fact is computed, I'll do my best to deliver it quickly from now on. Is there anyone who has mastered the low-level state so they can tell me if my understanding is correct?

0xSpyC avatar Jun 14 '24 09:06 0xSpyC

Sure, push your code in a draft PR and add your question there

tdelabro avatar Jun 14 '24 09:06 tdelabro

Hey @0xSpyC

Are you still working on it ? If not I can take it up. As we are currently integrating the SNOS job in madara orchestrator we need this component.

ocdbytes avatar Jun 18 '24 18:06 ocdbytes

@0xSpyC I am assigning this to @ocdbytes, let me know in case you've made any progress.

apoorvsadana avatar Jun 19 '24 13:06 apoorvsadana

Hey @0xSpyC

Are you still working on it ? If not I can take it up. As we are currently integrating the SNOS job in madara orchestrator we need this component.

It's ok i understand

0xSpyC avatar Jun 20 '24 10:06 0xSpyC

There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale. Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a 👍 Because this issue is marked as stale, it will be closed and locked in 7 days if no further activity occurs. Thank you for your contributions!

github-actions[bot] avatar Jul 21 '24 00:07 github-actions[bot]

repository archived in favor of https://github.com/madara-alliance/madara

tdelabro avatar Aug 02 '24 18:08 tdelabro