I think you’re right and in fact it should be gates_select instead of doubles_select. Thanks for pointing this out!
Would you like to open an issue in the PennyLane qml repo? This is where we host the code for the demos. You can also open a PR to make the fix if you want!
If you prefer that someone else does this there’s no problem, just let us know.
In this particular case the single excitations don’t contribute to lowering the ground state. I used qml.draw_mpl() to draw the circuit when using the full set of gates (gates_select). As you can see the single excitations all have 0.0 as the parameter, which is the same as not applying the gate at all. So in practice only a few gates contribute to the circuit.
So just to clarify, we should indeed use gates_select instead of doubles_select, but the reason why the energy doesn’t really change in this case is because for this particular case those single excitations end up having 0.0 as the parameter.
That’s great! Feel free to share the link to the issue and Pull Request here once you have them. And feel free to ask any questions you have along the way.