0xMiden / 0xMiden/miden-vm

Investigate removing `AceReplay` from `TraceGenerationContext`

未關閉
#2,255 1 則留言 0 個 reaction 已指派 1 人 已被 @huitseeker 認領 在 GitHub 檢視
processor
主要語言
Rust
星號
772
分支
352
平均合併
1 天 7 小時
30 天內合併 PR
84

描述

We should look into having the core trace fragment generators generate the `CircuitEvaluation` struct to pass to the `AceChiplet`, instead of having the fast processor include it in `TraceGenerationContext`. The main tradeoffs are size of `TraceGenerationContext` vs speed of `build_trace()` (i.e. what is the performance cost of generating the `AceChiplet` after the core trace, rather than in parallel? Compared to the size increase of `TraceGenerationContext` due to the inclusion of of `CircuitEvaluation`s?)

Context: https://github.com/0xMiden/miden-vm/pull/2188#discussion_r2399162111

貢獻指南

開啟貢獻指南

研究方向

Look at the `TraceGenerationContext` and `AceChiplet` in the codebase, likely in the processor or trace generation modules. Understand how `CircuitEvaluation` is currently generated and passed. Profile the performance of `build_trace()` with the current approach versus generating `CircuitEvaluation` after the core trace. The goal is to evaluate the trade-off between context size and generation speed.

由索引模型根據 Issue 內容生成。

評估

技術堆疊
rust
領域
backend, performance
Issue 類型
重構
難度
4/5
預估耗時
3-5 天
活躍度
冷清
描述清晰度
基本清楚
新手友好度
35/100

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。