Skip to content

Conversation

@anabananana98
Copy link
Contributor

No description provided.

@alibuild
Copy link
Collaborator

a2bca95: approval required: 1 of @adriansev (Adrian Sevcenco), @chiarazampolli (Chiara Zampolli), @davidrohr (David Rohr), @germainma (Marie Germain), @jgrosseo (Jan Fiete Grosse-Oetringhaus), @ktf (Giulio Eulisse), @lcunquei (Leticia Cunqueiro Mendez), @loizides (Constantinos Loizides), @matplo (Mateusz Ploskon), @mfasDa (Markus Fasel), @pzhristov (Peter Hristov), @qgp (Jochen Klein), @raymondEhlers (Raymond James Ehlers), @sawenzel (Sandro Christian Wenzel), @shahor02 (Ruben Shahoyan), @vkucera (Vit Kucera)

Comment with +1 to approve and allow automatic merging,or with +test to run tests only. Please comment on the pull request: click here and comment at the bottom of the page.

std::vector<float> fEventTrack_pt; ///< event track pT
std::vector<float> fEventTrack_eta; ///< event track eta
std::vector<float> fEventTrack_phi; ///< event track phi
std::vector<float> fEventTrack_id; ///< event track id
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think you want to stream these? But if you do, you need to increment the ClassDef (in any case, you changed the type of the variables, so you'll need to bump the ClassDef anyway)

std::vector<float> fEventTrack_eta; ///< event track eta
std::vector<float> fEventTrack_phi; ///< event track phi
std::vector<float> fEventTrack_id; ///< event track id
Long64_t fEventId;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A quick search on this PR seems to indicate that you didn't implement this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants