Relocate algos from DataFormats/CaloRecHit, clean up for coding rules

[jpata]

The Patatrack PRs #31704 and #32144 introduced calo-specific maths functionality depending on Eigen into DataFormats/CaloRecHit.

It was pointed out in #32144 (comment) and elsewhere that if it's just algorithms and not actual data formats, it might suit better in e.g. RecoLocalCalo/CommonAlgos or CommonTools/RecoAlgos. In addition, there is ongoing discussion in #31735 to remove the Eigen dependency from DataFormats.

Since the Patatrack PRs are still in flux, the proposal would be to address this post-Patatrack-integration, once it becomes more clear what methods and dataformats are actually brought in and are common between the PRs and between the CPU & GPU codes.

In addition, there are coding rules violations which slipped through (single character variable names, uppercase variables), these should be addressed.

[cmsbuild]

A new Issue was created by @jpata Joosep Pata.

@Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

[jpata]

assign reconstruction, heterogeneous

[cmsbuild]

New categories assigned: heterogeneous,reconstruction

@slava77,@perrotta,@makortel,@jpata,@fwyzard you have been requested to review this Pull request/Issue and eventually sign? Thanks