Drop not used Vector3D in header and Update BTagPerformance.h

[akritkbehera]

PR description:

This PR performs minor cleanup and header corrections across a few packages.

• Dropped #include "DataFormats/Math/interface/Vector3D.h" from files in PhysicsTools/CandUtils/interface/{EventShapeVariables.h,Thrust.h}, as it’s no longer required.
• Updated PhysicsTools/PatExamples/interface/BTagPerformance.h to explicitly include

[cmsbuild]

cms-bot internal usage

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49104/46341

[smuzaffar]

@akritkbehera , looks like DataFormats/Math is needed as following files in PhysicsTools/CandUtils/interface have included DataFormats/Math/interface/Vector3D.h . Looks like DataFormats/Math/interface/Vector3D.h is actually not needed any we can cleanup the file in [a] and drop the include statement. This also means we do not need this change in BuildFile.xml here

[a]

EventShapeVariables.h:#include "DataFormats/Math/interface/Vector3D.h"
Thrust.h:#include "DataFormats/Math/interface/Vector3D.h"

[cmsbuild]

REMINDER @mandrenguyen, @ftenchini, @sextonkennedy: This PR was tested with cms-sw/cmsdist#10110, please check if they should be merged together

[cmsbuild]

A new Pull Request was created by @akritkbehera for master.

It involves the following packages:

• PhysicsTools/CandUtils (analysis)
• PhysicsTools/PatExamples (analysis)

@cmsbuild, @tvami can you please review it and eventually sign? Thanks.
@AlexDeMoor, @Ming-Yan, @Senphy, @andrzejnovak, @castaned, @pavlo-kashko, @philippgadow this is something you requested to watch as well.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49104/46367

[cmsbuild]

Pull request #49104 was updated. @cmsbuild, @tvami can you please check and sign again.

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49104/46368

[cmsbuild]

Pull request #49104 was updated. @cmsbuild, @tvami can you please check and sign again.

[akritkbehera]

Please test

[tvami]

@akritkbehera please make the PR title a little bit more descriptive, right now it seems like it applies to all cmssw-sw

[smuzaffar]

@akritkbehera , please squash these commits too

[cmsbuild]

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49104/46371

[cmsbuild]

Pull request #49104 was updated. @cmsbuild, @tvami can you please check and sign again.

[akritkbehera]

Please test

[cmsbuild]

+1

Size: This PR adds an extra 24KB to repository
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-7fcc5b/48568/summary.html
COMMIT: 7f06037
CMSSW: CMSSW_16_0_X_2025-10-09-1100/el8_amd64_gcc13
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/49104/48568/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

• You potentially added 3 lines to the logs
• Reco comparison results: 8 differences found in the comparisons
• DQMHistoTests: Total files compared: 51
• DQMHistoTests: Total histograms compared: 3940073
• DQMHistoTests: Total failures: 481
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3939572
• DQMHistoTests: Total skipped: 20
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 50 files compared)
• Checked 218 log files, 188 edm output root files, 51 DQM output files
• TriggerResults: no differences found

[tvami]

@akritkbehera I see you changed the commit name, that sounds good to me, but I meant to have the same for the PR title