Setting default VPChannelID to invalid value

Changes default constructor value of VPChannelID to an invalid one. Default constructors is used (sometimes) as a dummy, like in the TrackVPOverlapMonitor and this let to wrong comparisons. One could add protection downstream, but this way is simpler and ensures these dummy values are not compared to or used as a possible true channelid 0 in the future.

Closes Rec#554 (closed)

---

Validated by

• Core Software
• RTA
• Simulation

added RTA Tracking bug fix labels

added 1 commit

• fb5afc14 - sets default VPChannelID to invalid one

Compare with previous version

changed target branch from master to 2024-patches

mentioned in issue Rec#554 (closed)

requested review from @hennessy

fyi @rmatev @cagapopo @wouter

changed the description

mentioned in merge request Rec!3865 (merged)

added VP label

Is this still needed now that Rec!3865 (merged) has been merged? I guess the default of 0 could potentially cause problems elsewhere?

added 5 commits

• fb5afc14...7b8a2155 - 4 commits from branch 2024-patches
• cf4fd738 - sets default VPChannelID to invalid one

Compare with previous version

added 1 commit

• 2ec334a0 - simplify

Compare with previous version

resolved all threads

added 1 commit

• 7168adc0 - minor fix

Compare with previous version

added Simulation label

marked the checklist item Core Software as completed

marked the checklist item RTA as completed

@gcorti @mexu is this okay for simulation?