Include hep_hpc in larsoft distribution
Dear LArSoft Experts,
Would it be OK to include hep_hpc_toolkit in the larsoft distribution?
Update: Tingjun later confirmed that just including the hep_hpc package in the LArSoft distribution is sufficient at this point.
#2 Updated by Tingjun Yang 12 months ago
LArSoft v08_41_00 depends on hdf5 v1_10_5.
However, the newest hep_hpc from scisoft:
v0_11_00 depends on hdf5 v1_10_3a. This causes a conflict for dune.
Would it be possible to make a new version of hep_hpc that depends on hdf5 v1_10_5?
#4 Updated by Brett Viren 12 months ago
Hi. There currently is no dependency of Wire-Cell Toolkit 0.13.x (latest release branch) on protobuf. The master branch had one for a while but adding that was a mistake (mine) and it was removed prior to the 0.13.x branching. If you still see some dependency on protobuf, please let me know where it comes in and we'll make sure to remove it.
Some off-topic background:
We are doing some new development that depends on protobuf (and some other new dependencies). This development is kept outside the WCT source repo but uses WCT's "wcb" builder which is why this dependency unwantedly "leaked" in. I'll bring up some issues about how best to handle this expanded dependency tree on the scisoft mailing list or somewhere as it's off-topic here. Just know that you needn't worry about protobuf for WCT right now!
#7 Updated by Kyle Knoepfel 12 months ago
- Status changed from Assigned to Feedback
Tingjun, is the request to just provide the
hep_hpc package (using HDF5 v1_10_5) with the LArSoft distribution, or the full
hep_hpc_toolkit as part of the LArSoft distribution? The toolkit right now supports Python 2, and it would take some effort to upgrade it to Python 3.
#11 Updated by Kyle Knoepfel 12 months ago
- % Done changed from 80 to 100
- Status changed from Assigned to Resolved
- Description updated (diff)
- Subject changed from Include hep_hpc_toolkit in larsoft distribution to Include hep_hpc in larsoft distribution
The source-code tarball for
hep_hpc v1_12_00 has been uploaded to SciSoft. This version requires C++17 (either the e19 or c7) qualifiers, and uses HDF5 v1_10_5. We will close this issue after the package has been included in the LArSoft distribution.