Skip to content

Add plastic wall for HYDRA#1335

Open
mduer wants to merge 1 commit intoR3BRootGroup:devfrom
mduer:hydra_pw
Open

Add plastic wall for HYDRA#1335
mduer wants to merge 1 commit intoR3BRootGroup:devfrom
mduer:hydra_pw

Conversation

@mduer
Copy link
Copy Markdown

@mduer mduer commented Apr 13, 2026

Add plastic wall for HYDRA


Checklist:

@mduer mduer force-pushed the hydra_pw branch 3 times, most recently from 2bcba23 to c73cd70 Compare April 14, 2026 11:58
@mduer
Copy link
Copy Markdown
Author

mduer commented Apr 14, 2026

Pass all checks

Copy link
Copy Markdown
Contributor

@jose-luis-rs jose-luis-rs left a comment

Choose a reason for hiding this comment

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

Thanks, please try to include the minor changes and rebase the dev branch again.

You can also add your name to the AUTHORS file

Comment thread pw/calibration/R3BPWCal2Hit.cxx Outdated
Comment thread pw/calibration/R3BPWCal2Hit.cxx Outdated
Comment thread pw/calibration/R3BPWCal2Hit.cxx Outdated
Comment thread pw/calibration/R3BPWCal2Hit.cxx Outdated
Comment thread pw/calibration/R3BPWCal2Hit.h Outdated
Comment thread pw/online/R3BPWOnlineSpectra.h
Comment thread pw/pars/R3BPWContFact.cxx Outdated
Comment thread r3bdata/pwData/R3BPWCalData.cxx Outdated
Comment thread r3bdata/pwData/R3BPWCalData.h Outdated
Comment thread r3bsource/pw/ext_h101_pw.h
@mduer mduer force-pushed the hydra_pw branch 3 times, most recently from 492452a to 39a9cb9 Compare April 22, 2026 11:10
@jose-luis-rs
Copy link
Copy Markdown
Contributor

After adding your name to the Author list, you must run the python scripts:
python3 update_zenodo.py
python3 update_codemeta.py
to generate the corresponding output files. Then git add "files"

Comment thread r3bdata/pwData/R3BPWWRData.cxx Outdated
@mduer
Copy link
Copy Markdown
Author

mduer commented Apr 22, 2026

After adding your name to the Author list, you must run the python scripts: python3 update_zenodo.py python3 update_codemeta.py to generate the corresponding output files. Then git add "files"

I get errors for both when running them with python3 xx.. in thr R3BRoot folder. Do I need to do something in addition?

Comment thread AUTHORS Outdated
#include "FairMultiLinkedData.h"
#include "TObject.h"

class R3BPWPreCalData : public FairMultiLinkedData
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Here you don't need FairMultiLinkedData, just
class R3BPWPreCalData : public TObject


#include "FairMultiLinkedData.h"

class R3BPWHitData : public FairMultiLinkedData
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Here you don't need FairMultiLinkedData, just
class R3BPWHitData : public TObject

[[nodiscard]] double GetTot0() const { return fTot0; }
[[nodiscard]] double GetTot1() const { return fTot1; }

protected:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

private: instead of protected:

inline void SetTime(double time) { fTime = time; }
inline void SetCharge(double Q) { fCharge = Q; }

protected:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

private: instead of protected:

[[nodiscard]] UShort_t GetEdge() const { return fEdge; }
[[nodiscard]] uint64_t GetEpoch() const { return fEpoch; }

protected:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

private: instead of protected:

[[nodiscard]] double GetTime() const { return fTime; }
[[nodiscard]] double GetTot() const { return fTot; }

protected:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

private: instead of protected:

#include "FairMultiLinkedData.h"
#include "TObject.h"

class R3BPWCalData : public FairMultiLinkedData
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Here you don't need FairMultiLinkedData, just
class R3BPWCalData : public TObject


#pragma once

#include "FairMultiLinkedData.h"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove #include "FairMultiLinkedData.h"

#pragma once

#include "FairMultiLinkedData.h"
#include "TObject.h"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

add also #include "Rtypes.h"

explicit R3BPWCalData(UShort_t DetId, UShort_t Bar, double Time0, double Time1, double Tot0, double Tot1);

/** Destructor **/
virtual ~R3BPWCalData() = default;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

~R3BPWCalData() override = default;

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.

2 participants