Improve <diags>.<species>.variables = none

[ax3l]

Tested with Examples/Physics_applications/laser_acceleration/inputs_3d.

Even with

diag1.electrons.variables = none

we still write runtime integer attribute regionofinterest unconditionally

electrons.addIntegerAttributes = regionofinterest
electrons.addRealAttributes = initialenergy

(but interestingly not the real attribute initialenergy)

And the id of a particle is also written unconditionally.

This should be improved.

Follow-up to #4914

[ax3l]

Also, seen with Examples/Physics_applications/laser_acceleration/inputs_rz, filtering of positions in RZ does not really work.

The whole m_plot_flags logic should be factored out of ParticleDiag.cpp and either put directly into the backends or generalized with the r,z,theta -> x,y,z logic that is supported in (some?) of the backends.

This whole upfront parsing and bookkeeping in unrelated arrays like m_plot_flags is too fragile and error-prone, not dealing properly with runtime-added attributes, etc.

[ax3l]

Another test to to run: dumping the phi runtime attribute in ES examples (exists only in the pinned, filtered PC).

[RemiLehe]

Here is the PR that added the phi output: #4599

[ax3l]

Discussed: The implicit solver also adds runtime attributes that might become out of sync with the dump flags (even if we do not mean to dump the auxiliary positions&momenta we add).