Josh Izaac

Results 153 comments of Josh Izaac

Thanks for digging into this a bit further @say4n! When you are debugging, is the line where the crash is occurring at the bugfix? https://github.com/XanaduAI/strawberryfields/blob/4de1d4676660a55fe2fef296b365af00d62b6e8e/strawberryfields/backends/fockbackend/circuit.py#L767-L770

Thanks for catching this @nquesada 😬

I would propose `MeasureThreshold` - more explicit to the user, fits with the existing approach we already use for new measurements in SF. This allows the backends to implement this...

Hi @ziofil, this is actually intentional behaviour, but specific to the `Dgate`. With the displacement gate in particular, it is common to consider the alpha parameter in Cartesian form. However,...

Early on, we had a discussion internally on how much input validation we wanted to do in Strawberry Fields. We definitely did more input validation in places like the engine...

Also worth mentioning that while this is _unintuitive_, it is also expected behaviour; the backend API was designed such that after a measurement operation, the mode is always traced out...

Hi @felipeoyarce, nice! Feel free to open a WIP PR, and ask any questions as they arise :)

From previous discussions: the density matrix is computed because: * `state.quad_expectation` calls `self.reduced_dm()` * `self.reduced_dm()` calls `self.dm()` It may be possible to avoid the density matrix computation altogether, by being...

> I'm thinking there are two alternative, but still reasonable, solutions here: Thanks @co9olguy! It almost sounds like we should be working towards implementing _both_ solutions (they don't seem to...

@nquesada, what do you think the best solution to this is? If we change `rounding=12`, are we just delaying this bug (e.g., is it possible another edge case will fail...