Skip to content

fatal error for invalid configuration#1054

Merged
jgrosseo merged 2 commits into
AliceO2Group:masterfrom
jgrosseo:master
Jul 27, 2022
Merged

fatal error for invalid configuration#1054
jgrosseo merged 2 commits into
AliceO2Group:masterfrom
jgrosseo:master

Conversation

@jgrosseo

Copy link
Copy Markdown
Contributor

No description provided.

@jgrosseo jgrosseo requested review from a team, alibuild, iarsene and ktf as code owners July 26, 2022 12:58
@jgrosseo

Copy link
Copy Markdown
Contributor Author

@victor-gonzalez What do you think?
In addition the centrality table has both process functions (run2 and run3) off by default. The disadvantage is one cannot run it at all to get a dpl-config.json to then edit it. I would suggest to enable one of them by default, as we do for the multiplicity task. What do you think?

@victor-gonzalez

Copy link
Copy Markdown
Collaborator

@jgrosseo I was just running a training session at the PAG about it (you probably got the input from Maja)
I was a bit surprised because the dpt-config.json file was actually written although the workflow was hung
Anyway, I agree, one of the processes should be enabled by default (Run 2)
Regarding the solution, as these are core service tasks, probably it is needed for orienting the local user

@jgrosseo

Copy link
Copy Markdown
Contributor Author

in my case the file is written but has an empty config entry for centrality-table. It is not the case for you?

Is this PR ok for you?

@victor-gonzalez

Copy link
Copy Markdown
Collaborator

In the case of Malgorzata, which was the one which reported it, surprisingly, she got a complete centrality-table entry in the json file. Actually we used it for creating a configuration file which fixed the issue

The PR is OK for me

@jgrosseo jgrosseo enabled auto-merge (squash) July 27, 2022 07:46

@alibuild alibuild left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approving on behalf of @jgrosseo.

@jgrosseo jgrosseo merged commit b873ac7 into AliceO2Group:master Jul 27, 2022
lbariogl pushed a commit to CF-tutorials/O2Physics that referenced this pull request Jul 28, 2022
* fatal error for invalid configuration

* switch processRun2 on by default
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants