Daira-Emma Hopwood

Results 412 issues of Daira-Emma Hopwood

_Originally posted by @daira in https://github.com/zcash/librustzcash/pull/1496#discussion_r1722294295_: > Incidentally, why does the doc for `wallet::chain_tip_height` say > ```rust > /// Returns the minimum and maximum heights of blocks in the chain...

A-documentation

In #1489 we added a check for collisions between FVK components when adding accounts. As part of supporting Incoming accounts, we should: * Use a migration to add `orchard_ivk_item_cache`, `sapling_ivk_item_cache`,...

A-wallet

_Originally posted by @str4d in https://github.com/zcash/librustzcash/pull/1473#discussion_r1701063435_: > It turns out that the `GetTransaction` method added to `lightwalletd` is horribly broken, because it does not correctly propagate the output of the...

A-wallet

Use `--profile=dev` to disable this. (As well as the fact that much of our code essentially requires optimization to run at all reasonably, this ensures that we are testing the...

testing

### Describe the Feature Many users do not use screen readers but still find it useful to see alt text on videos. The current UI does not seem to have...

feature-request

> For the unwrap calls, i see that : > * `batch.rs` has 1 non-test unwrap > * `legacy.rs` has 1 non-test unwrap > * `lib.rs` has 3 non-test unwrap...

Currently "circuit" implies "std". I made a start in #460, but it's blocked by updating to `rand 0.9` (#463).

`rand 0.8` can't be `no_std`, which makes this block #459.

Currently failing. This is basically blocked on the update to `rand 0.9`.

Rust 1.81.0 started [aborting on uncaught panics in `extern "C"` functions](https://blog.rust-lang.org/2024/09/05/Rust-1.81.0.html#abort-on-uncaught-panics-in-extern-c-functions), which fixed the soundness hole that I think originally caused us to *have* to use `panic::catch_unwind` in all such...

rust_related
C-tech-debt