moose icon indicating copy to clipboard operation
moose copied to clipboard

Read command line arguments in file by line rather than by space

Open GiudGiud opened this issue 8 months ago • 1 comments

I'm fairly sure everyone was already using lines rather than spaces, but the way it was read was with spaces and would see vector parameters '1 2 3' as new cli_args for the next child app

refs #18596

@caolwec

GiudGiud avatar Jun 15 '25 13:06 GiudGiud

Job Documentation, step Docs: sync website on 2bee22c wanted to post the following:

View the site here

This comment will be updated on new commits.

moosebuild avatar Jun 15 '25 16:06 moosebuild

@joshuahansel this is for a WEC input

GiudGiud avatar Jul 31 '25 21:07 GiudGiud

where is my test

GiudGiud avatar Jul 31 '25 21:07 GiudGiud

nevermind. The test is the "multiline_file" test, which is now the only accepted way of doing files. "oneline_file" is removed

GiudGiud avatar Jul 31 '25 21:07 GiudGiud

Job Coverage, step Generate coverage on 2bee22c wanted to post the following:

Framework coverage

0aecda #30766 2bee22
Total Total +/- New
Rate 85.55% 85.55% +0.00% 100.00%
Hits 116046 116048 +2 3
Misses 19604 19603 -1 0

Diff coverage report

Full coverage report

Modules coverage

Coverage did not change

Full coverage reports

Reports

This comment will be updated on new commits.

moosebuild avatar Aug 01 '25 01:08 moosebuild

Sockeye doc fix PR is up.

joshuahansel avatar Aug 05 '25 12:08 joshuahansel

More correct than ever imo

(since you could do one line and split by apps)

GiudGiud avatar Aug 05 '25 13:08 GiudGiud