Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update FccFastSimGeneration.md #140

Merged
merged 1 commit into from
Apr 24, 2024
Merged

Update FccFastSimGeneration.md #140

merged 1 commit into from
Apr 24, 2024

Conversation

pkoppenb
Copy link
Contributor

@pkoppenb pkoppenb commented Mar 5, 2024

k4run pythia.py -h is likely a typo. To get the line displayed you need to remove -h. In my case the piping into head causes a logging error (both at Nikhef and CERN).

`k4run pythia.py -h` is likely a typo. To get the line displayed you need to remove `-h`. In my case the piping into head causes a logging error (both at Nikhef and CERN).
@kjvbrt
Copy link
Contributor

kjvbrt commented Mar 22, 2024

Hi @pkoppenb,

the -h argument for k4run no longer shows the algorithm chain, instead one can use --dry-run.

Side note: ATM when k4run output is piped to head it will generage errors you observe. This PR key4hep/k4FWCore#186 attemps to fix the issue, but for the time being let's remove all uses of head.

@kjvbrt kjvbrt requested a review from gganis March 22, 2024 12:17
@pkoppenb
Copy link
Contributor Author

Thanks Juraj. Looks good.

For a bit of context: the tutorial was used with a bunch of 35 Nikhef PhD students. It went very well.

@kjvbrt kjvbrt merged commit 98aff9c into HEP-FCC:master Apr 24, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants