Make execution parameters private

[alecandido]

Closes #1047

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 52.08%. Comparing base (6fbf53f) to head (c0a2e42).
Report is 8 commits behind head on 0.2.

Additional details and impacted files

@@            Coverage Diff             @@
##              0.2    #1052      +/-   ##
==========================================
- Coverage   52.10%   52.08%   -0.02%     
==========================================
  Files          63       63              
  Lines        2806     2803       -3     
==========================================
- Hits         1462     1460       -2     
+ Misses       1344     1343       -1     

Flag	Coverage Δ
unittests	52.08% <100.00%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[alecandido]

@stavros11 whenever you can, also take a look at this.

If you like the result, we could keep it and include in the release.

[stavros11]

Thanks @alecandido. It looks good and I believe we should have this in 0.2.

In c0a2e42 I updated some tests that were still using ExecutionParameters in platform.execute. Most were marked with qpu so are skipped and maybe we will even drop them at some point, but maybe still good to be consistent. Just found them by searching for ExecutionParameters.

[alecandido]

Thanks @stavros11 for the update.

I actually paid little attention to the qpu tests in general, and we may need to reconsider a bit the strategy concerning them, even in light of #1039 and similar. But this is something we should definitely do in 0.2.1 (or following).

I'm merging :)