Paul

Results 94 comments of Paul

Reminder to update the same changes on lightning kokkos to the lightning repo (e.g. https://github.com/PennyLaneAI/pennylane-lightning/pull/819)

If we were to raise an error, we can say something like ``` catalyst.cond() must take in callables, but got f{instantiated_gate}. See [documentation]. ```

@jzaia18 and I ran the benchmark suite at https://github.com/PennyLaneAI/catalyst/pull/1652/commits/55722b6b1e0ac2ad8cc02ebbf9397522ad899c54 We saw that runtime score was not impacted (yay), but there is an increase in memory usage. Since we still generate...

After investigations with @mehrdad2m , we think the memory increase is a false positive. The memory increase is a flat ~3MB across all workload levels (from `light` to `heavy`; above...

At https://github.com/PennyLaneAI/catalyst/pull/1652/commits/ab00ce1489df94031055ce6522863c8bd13b4a9a, wheels CI all succeed except mac x86. This is because jax dropped mac x86 from 0.5.0 onwards https://github.com/jax-ml/jax/blob/main/CHANGELOG.md#jax-050-jan-17-2025

I have removed the wheels build requirement. We have checked at a previous commit and now it is known that mac x86 will fail.

Thanks to everyone involved, for the work and for the review 🥳 🚀 🎈 ! Also a shoutout to @mudit2812 for the many helpful insights during. I have done a...

It's not a segfault, but a ``` Assertion failed: (inserted.second && "expected region to contain uniquely named symbol operations") ``` Maybe something wrong during `renameToUnique` (or whatever it was called)?...

Versions: ``` pennylane 0.43.0 pennylane_lightning 0.43.0 pennylane_lightning_kokkos 0.43.0 ```