RichDecodedData: remove duplicate ID assert

Turns out, in real data this does happen (very rarely) presumably due to some low level corruption, due to bit flips in data (e.g. from radiation).

Allowing duplicate hits through is not an issue downstream in the reco, it will just use both copies blindly and thus make duplicate photons for the same hit. Always running a dupe check to remove them is not worth the CPU cost involved.

So just remove as its no longer completely impossible.

FYI @rmatev

added Decoding RICH labels

/ci-test

changed the description

added ci-test-triggered label

added RTA label

• [2023-08-31 16:14] Validation started with lhcb-master-mr#9095

resolved all threads

merged

mentioned in commit a6b9fa91

mentioned in issue Moore#659 (closed)

added backport RTA/2023.08.24 label

picked the changes into the branch cherry-pick-a6b9fa91 with commit 97d8afda

mentioned in commit 97d8afda

mentioned in merge request !4331 (merged)

mentioned in commit ca2d660d