Alexander Saites

Results 12 issues of Alexander Saites

I recently migrated to a monorepo, but when [updating the `knope.toml` file](https://knope.tech/recipes/convert-to-monorepo/#step-2-update-knopetoml), I missed the `s` in `packages`, resulting in something like this: ```toml [package.old-package] versioned_files = ["old-package/Cargo.toml", "Cargo.lock"] changelog...

An old `mypy` bug causes `stubtest` to reject the ordering of `@property` attributes generated by `pyo3_stub_gen`. [They've fix that](https://github.com/Jij-Inc/pyo3-stub-gen/issues/289), but it's not yet released. When it is, we should upgrade...

While merging `quil-py` into `quil-rs`, I came upon [an old `TODO`](https://github.com/rigetti/quil-rs/blob/9db8cb4cfe75e67a4fa572e96e86ae8e59d0bdfc/quil-py/src/program/mod.rs#L180) without an issue, which I've moved into `quil-rs` as so: ``` // TODO: (9b9690d4 Marquess Valdez 2023-06-01 14:10:25 -0700)...

The conversions to/from these Owned types and their non-Owned counterparts involves a lot of cloning, and they're the types exposed by the Python bindings. At present, this is just a...

This is the current implementation of `__add__` for a `Program`: ```rust fn __add__(&self, rhs: Self) -> Self { self.clone() + rhs } ``` Obviously this requires cloning both `self` and...

This code panics: ```python from quil.instructions import Gate, Qubit Gate("X", (), (Qubit.from_fixed(0),), ()).to_unitary_mut(0) ``` Running that with a debug build gives something like: ``` attempt to subtract with overflow ```...

bug

The documentation for `Expression`s states often: > Note that when comparing Quil expressions, any embedded NaNs are treated as *equal* to other NaNs, not unequal, in contravention of the IEEE...

bug

As with #452, `ProgramError` is huge (almost half a kilobyte) and used in the `Result` type for several functions. Moreover, there are large differences in the sizes of variants, so...

The `TypeError` enum is huge, mainly because `Instruction` is huge (~280 bytes), and so the `TypeResult = TypeError` is likewise huge, yet the `T` in question is almost always the...

The `qvm-app` accepts an `rng-seed` parameter to seed the QVM's PRNG. The documentation states that it should be greater than 0, but any value in $\\{ 2^{32+n}, n>0 \\} =...