search

art-daq / artdaq_daqinterface

Other
0 stars 1 forks source link

DAQInterface bookkeeping should only include one copy of the host_map in destinations, as well as sources #39

Closed eflumerf closed 2 years ago

eflumerf commented 2 years ago

This issue has been migrated from https://cdcvs.fnal.gov/redmine/issues/22924 (FNAL account required) Originally created by @bieryAtFnal on 2019-07-11 20:12:08

As part of DFO testing at protoDUNE, I saw problems in which the DFO art process took a long time to start (30-50 seconds), and I believe that this is because of the very-large Config file for the DFO (148,000+ characters).

The bulk of the contents of that Config file was made up of repeated "host_map" entries in the "destinations" block. I don't believe that the multiple copies are needed.
We should confirm that (I've done initial confirmation tests at protoDUNE), and we should modify the daqinterface bookkeeping to only output one copy in the destinations block.

eflumerf commented 2 years ago

Comment by @jcfreeman2 on 2019-07-15 20:02:04

With commit c2eba590d564a219b81e673e590ca79b8ab5612d on the feature/issue22924_remove_host_map_entries branch, I essentially just made the same one-line change Kurt did to create his DFO-friendly DAQInterface on the ProtoDUNE cluster.

eflumerf commented 2 years ago

Comment by @bieryAtFnal on 2019-07-17 20:00:56

I've validated this code change via code inspection. It looks great to me.