ref: Merge parameter transporter and resetter to parameter updater

[niermann999]

Merge the parameter transporter and parameter resetter to a single composite actor, with any actor that relies on the bound track parameters as their observers. This way, actors like the material interactor cannot be added to the actor chain without the parameter updater present and the parameter resetter cannot be omitted by accident either without a compiler error.

The observing actors are now only triggered if the new actor result type sets a notify flag, making checking for the navigation to be on surface necessary only in the parameter updater.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
7.2% Duplication on New Code

See analysis details on SonarQube Cloud

[niermann999]

Tested in acts-project/traccc#1244

[stephenswat]

Performance summary

Here is a summary of the performance effects of this PR:

Tabular

Benchmark	f86a37b		6a29182		Delta
	Throughput	CV	Throughput	CV
ODD (1 GeV, cov. on)	2.46 MHz	0.05%	2.55 MHz	0.07%	+3.71%
ODD (1 GeV, cov. off)	4.73 MHz	0.04%	4.94 MHz	0.06%	+4.33%
ODD (10 GeV, cov. on)	3.18 MHz	0.05%	3.29 MHz	0.07%	+3.42%
ODD (10 GeV, cov. off)	5.83 MHz	0.08%	6.12 MHz	0.06%	+4.95%
ODD (100 GeV, cov. on)	3.37 MHz	0.06%	3.48 MHz	0.05%	+3.27%
ODD (100 GeV, cov. off)	6.09 MHz	0.07%	6.39 MHz	0.03%	+4.90%
ODD ([1, 100) GeV, cov. on)	2.87 MHz	0.08%	3.00 MHz	0.10%	+4.81%
ODD ([1, 100) GeV, cov. off)	5.63 MHz	0.14%	5.83 MHz	0.04%	+3.56%
Toy detector ([1, 100) GeV, cov. on)	14.86 MHz	0.57%	14.97 MHz	0.33%	+0.75%
Toy detector ([1, 100) GeV, cov. off)	23.80 MHz	0.55%	24.28 MHz	0.78%	+2.04%
Wire chamber ([1, 100) GeV, cov. on)	11.40 MHz	0.02%	11.45 MHz	0.06%	+0.38%
Wire chamber ([1, 100) GeV, cov. off)	20.40 MHz	0.03%	20.84 MHz	0.03%	+2.20%

Note

This is an automated message produced upon the explicit request of a human being.

[stephenswat]

Happy to accept a small slowdown here if it simplifies the code.

[niermann999]

Happy to accept a small slowdown here if it simplifies the code.

I thought this is a throughput increase, or am I reading the results wrong?

[stephenswat]

I thought this is a throughput increase, or am I reading the results wrong?

You are correct I am dumb.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
6.8% Duplication on New Code

See analysis details on SonarQube Cloud