Project

General

Profile

Support #23780

Include hep_hpc in larsoft distribution

Added by Tingjun Yang 11 months ago. Updated 9 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
-
Start date:
12/18/2019
Due date:
% Done:

100%

Estimated time:
16.00 h
Spent time:
Experiment:
-
Co-Assignees:
Duration:

Description

Dear LArSoft Experts,

Would it be OK to include hep_hpc_toolkit in the larsoft distribution?

Thanks,
Tingjun


Update: Tingjun later confirmed that just including the hep_hpc package in the LArSoft distribution is sufficient at this point.

History

#1 Updated by Kyle Knoepfel 10 months ago

  • Estimated time set to 16.00 h
  • Assignee set to Kyle Knoepfel
  • Status changed from New to Assigned

This is a reasonable request.

#2 Updated by Tingjun Yang 9 months ago

LArSoft v08_41_00 depends on hdf5 v1_10_5.

However, the newest hep_hpc from scisoft:
https://scisoft.fnal.gov/scisoft/packages/hep_hpc/
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?

Thanks.

#3 Updated by Lynn Garren 9 months ago

The wirecell product added a dependency on hdf5. It also added a dependency on protobuf. I believe the toolkit needs to be updated.

#4 Updated by Brett Viren 9 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!

#5 Updated by Tingjun Yang 9 months ago

Would it be possible to make a new version of hep_hpc that depends on hdf5 v1_10_5? We just need e19:prof at the very least.

Thanks.

#6 Updated by Kyle Knoepfel 9 months ago

Yes, that is a reasonable request.

#7 Updated by Kyle Knoepfel 9 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.

#8 Updated by Tingjun Yang 9 months ago

Hi Kyle,

For now we just need hep_hpc package in the LArSoft distribution.

Thanks,
Tingjun

#9 Updated by Kyle Knoepfel 9 months ago

Okay, that's easily doable. Will talk with Lynn to get it in there.

#10 Updated by Kyle Knoepfel 9 months ago

  • % Done changed from 0 to 80
  • Status changed from Feedback to Assigned

#11 Updated by Kyle Knoepfel 9 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.

#12 Updated by Kyle Knoepfel 9 months ago

  • Status changed from Resolved to Closed

hep_hpc is included in the LArSoft v08_42_00 distribution.

Also available in: Atom PDF