Mu2e / Production

Scripts and fcl for collaboration production procedures
Apache License 2.0
0 stars 33 forks source link

Update input file location to cvmfs #311

Closed brownd1978 closed 4 months ago

FNALbuild commented 4 months ago

Hi @brownd1978, You have proposed changes to files in these packages:

which require these tests: build.

@Mu2e/fnalbuild-users, @Mu2e/write have access to CI actions on main.

:hourglass: The following tests have been triggered for c2ca8be508f1030e33f8b3c053d1adcbc79cac5c: build (Build queue has 1 jobs)

About FNALbuild. Code review on Mu2e/Offline.

FNALbuild commented 4 months ago

:sunny: The build tests passed at c2ca8be508f1030e33f8b3c053d1adcbc79cac5c.

Test Result Details
test with :white_check_mark: Command did not list any other PRs to include
merge :white_check_mark: Merged c2ca8be508f1030e33f8b3c053d1adcbc79cac5c at 516610c672b48172cfee33d499dfe14653c1d5a6
build (prof) :white_check_mark: Log file. Build time: 11 min 13 sec
ceSimReco :white_check_mark: Log file.
g4test_03MT :white_check_mark: Log file.
transportOnly :white_check_mark: Log file.
POT :white_check_mark: Log file.
g4study :white_check_mark: Log file.
cosmicSimReco :white_check_mark: Log file.
cosmicOffSpill :white_check_mark: Log file.
ceSteps :white_check_mark: Log file.
ceDigi :white_check_mark: Log file.
muDauSteps :white_check_mark: Log file.
ceMix :white_check_mark: Log file.
rootOverlaps :white_check_mark: Log file.
g4surfaceCheck :white_check_mark: Log file.
FIXME, TODO :white_check_mark: TODO (0) FIXME (0) in 0 files
clang-tidy :white_check_mark: 0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at c2ca8be508f1030e33f8b3c053d1adcbc79cac5c after being merged into the base branch at 516610c672b48172cfee33d499dfe14653c1d5a6.

For more information, please check the job page here. Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.

kutschke commented 4 months ago

@gaponenko and @rlcee This PR adds a fcl script to make copies of a file. perhaps just copying a subset of events. Do we have an equivalent script somewhere else?

brownd1978 commented 4 months ago

I searched for a pre-existing script in Production and Offline and didn't find it. It's a trivial script, but I don't want to keep re-writing it if I need it again.

On Wed, Apr 24, 2024 at 4:00 PM Rob Kutschke @.***> wrote:

@.**** approved this pull request.

I don't know that we need CopyEvents.fcl - I have a vague memory that something like this already exists. If we make that decision we can remove it in a separate PR - I would like the cosmicOffSpill change to go in.

— Reply to this email directly, view it on GitHub https://github.com/Mu2e/Production/pull/311#pullrequestreview-2021092841, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABAH573WROTRVDXY5MJ76HLY7A2QZAVCNFSM6AAAAABGXXPO5GVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMRRGA4TEOBUGE . You are receiving this because you were mentioned.Message ID: @.***>

-- David Brown @.*** Office Phone (510) 486-7261 Lawrence Berkeley National Lab M/S 50R5008 (50-6026C) Berkeley, CA 94720

rlcee commented 4 months ago

@kutschke you are probably thinking of Production/JobConfig/common/artcat.fcl

kutschke commented 4 months ago

I searched for a pre-existing script in Production and Offline and didn't find it. It's a trivial script, but I don't want to keep re-writing it if I need it again.

As Ray points out, it does already exist and it's even in the same directory. I recommend we get rid of the new copy.