larrecodnn has optional dependencies on tensorflow and trtis_clients. It should be possible to build for either trtis_clients or tensorflow. However, all trtis_clients interfaces are found under larrecodnn/ImagePatternAlgs/Tensorflow. This means that tensorflow is required at all times.
We request that the trtis_clients interface be moved into its own independent subdirectory at same level as the Tensorflow subdirectory.
#1 Updated by Michael Wang about 2 months ago
Here is my proposal on how to restructure things. I will first show the current structure followed by the proposed structure. I didn't include every file, only those that are relevant or will be moved. Files that are not indicated will remain in whatever subdirectory they are currently in. It is assumed that if those subdirectories are moved, those files will move with them. Please let me know what you think.
================= CURRENT STRUCTURE: ================= larrecodnn | +-ImagePatternAlgs | +- job | +- scripts | +- Keras | +- Tensorflow | +- Modules | +- TF | +- PointIdAlg | +- PointIdAlgTools | | | +- IPointIdAlg.h | | | +- PointIdAlgKeras_tool.cc | | | +- PointIdAlgTf_tool.cc | | | +- PointIdAlgTrtis_tool.cc | +- WaveformRecogTools | +- IWaveformRecog.h | +- WaveformRecogTf_tool.cc | +- WaveformRecogTrtis_tool.cc ================== PROPOSED STRUCTURE: ================== larrecodnn | +-ImagePatternAlgs | +- job | +- scripts | +- Modules | +- ToolInterfaces | | | +- IPointIdAlg.h | | | +- IWaveformRecog.h | +- Keras | | | +- Tools | | | +- PointIdAlgKeras_tool.cc | +- Triton | | | +- Tools | | | +- PointIdAlgTrtis_tool.cc | | | +- WaveformRecogTrtis_tool.cc | +- Tensorflow | +- TF | +- PointIdAlg | +- Tools | +- PointIdAlgTf_tool.cc | +- WaveformRecogTf_tool.cc