Every HybridOp should implement `map_wires`

[erick-xanadu]

Or alternatively, a base class should implement map_wires.

[dime10]

I do wonder, why should they implement this method (in what context does it come up or cause issues)?

[erick-xanadu]

Well, I had this test here, but it got deleted :sweat_smile: https://github.com/PennyLaneAI/catalyst/pull/555/files

The issue came from a template or transform that uses map_wires in operations. If HybridCtrl didn't implement it, then it would fail. I believe with the recent changes where HybridCtrl inherits from another control it might be solved, but I am not a 100% sure.

[dime10]

Ah thanks, that makes sense! I don't think it's fully solved yet.