0xMiden/protocol

Optimize input note num inputs/assets memory layout

Aperta

#2256 aperta il 12 gen 2026

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (1 assegnatario)Rust (139 fork)github user discovery
good first issuekernels

Metriche repository

Star
 (123 stelle)
Metriche merge PR
 (Merge medio 5g 11h) (110 PR mergiate in 30 g)

Descrizione

After #2109 is closed (to avoid merge conflicts), we should be able to make a tiny PR that moves these two constants closer together:

https://github.com/0xMiden/miden-base/blob/eee0e8450b825857e7d56ffebfaa269378a61aca/crates/miden-protocol/src/transaction/kernel/memory.rs#L388-L389

It should be possible to do:

pub const INPUT_NOTE_NUM_INPUTS_OFFSET: MemoryOffset = 36;
pub const INPUT_NOTE_NUM_ASSETS_OFFSET: MemoryOffset = 37;
pub const INPUT_NOTE_ASSETS_OFFSET: MemoryOffset = 40

This applies to memory.masm in the same way.

Context: https://github.com/0xMiden/miden-base/pull/2249#discussion_r2679280006

Guida contributor

Direzione di ricerca
Verifica se la PR #2109 è stata unita, poi modifica le costanti in memory.rs e memory.masm come descritto nell'issue.
Tech stack
rust
Dominio
backendblockchain
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitRust
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75