Skip to content

Conversation

@BongHwi
Copy link
Contributor

@BongHwi BongHwi commented Mar 14, 2024

From Pythia 8.303, The PDG code of Lambda(1520) has been updated to 102134 instead of 3124

@github-actions
Copy link

REQUEST FOR PRODUCTION RELEASES:
To request your PR to be included in production software, please add the corresponding labels called "async-" to your PR. Add the labels directly (if you have the permissions) or add a comment of the form (note that labels are separated by a ",")

+async-label <label1>, <label2>, !<label3> ...

This will add <label1> and <label2> and removes <label3>.

The following labels are available
async-2022-pp-apass4
async-2023-pbpb-apass
async-2023-pp-apass1
async-data
async-mc
async-2022-pp-apass6

@benedikt-voelkel
Copy link
Contributor

Hej @BongHwi
good call! Thanks for spotting. In fact we use Pythia8@8.304 since months or even years now when building SW with --defaults o2 as far as I can see.

@benedikt-voelkel
Copy link
Contributor

@BongHwi once merged, should this be ported to the code we use for PbPb MC productions?

@BongHwi
Copy link
Contributor Author

BongHwi commented Mar 15, 2024

@benedikt-voelkel Yes, we'd like to apply this to O2-4365.

@benedikt-voelkel
Copy link
Contributor

The ticket you linked seems to be a different one or I don't see the connection to that ticket 🤔

@BongHwi
Copy link
Contributor Author

BongHwi commented Mar 15, 2024

Oops.. yes. it was O2-4306

@benedikt-voelkel
Copy link
Contributor

ok, so it is for pp 2022 apass4. I added the label. In addition, I put PbPb 2023 because since it is generator functionality, it wouldn't hurt to have it there as well. If your disagree, please remove the label for that.

@benedikt-voelkel benedikt-voelkel merged commit cd5b970 into AliceO2Group:master Mar 15, 2024
@BongHwi
Copy link
Contributor Author

BongHwi commented Mar 15, 2024

Thanks a lot! We'll prepare the Pb-Pb production as well once we finish the pp production, so I guess it's an appropriate tag.

@BongHwi
Copy link
Contributor Author

BongHwi commented Mar 19, 2024

Sorry @benedikt-voelkel , we found an issue with running the Pythia with this new PDG code.
Therefore, we'd like to revert this commit until we solve the issue.

In detail, the Pythia tells this error when we try to produce the event:

{"etaMax":1.2,"etaMin":-1.2,"genDecayed":true,"n":1,"pdg":102134,"ptMax":20,"ptMin":0.0}
===> Unknown particle requested with PDG 102134, mass set to 0
[01:03:19][FATAL] Could not find mass for mPdg 102134
libc++abi: terminating due to uncaught exception of type fair::FatalException

So it couldn't find the 102134 even though it was an 8.3 version of Pythia.
The definition of the particle is on the Pythia particle list: https://pythia.org/manuals/pythia8308/ParticleData.html

@benedikt-voelkel
Copy link
Contributor

@BongHwi The error does not seem to come from Pythia but ultimately from the fact that we do not have that PDG in our table.

It fails after running /MC/config/PWGLF/pythia8/generator_pythia8_longlived.C where it looks up the particle in TDatabasePDG.

So, I think, you should be able to fix it by adding that to

@BongHwi
Copy link
Contributor Author

BongHwi commented Mar 19, 2024

Great, thanks for the suggestion, I'll try to update these files too.

@BongHwi
Copy link
Contributor Author

BongHwi commented Mar 19, 2024

Here I tried to update the O2 database.

BongHwi/AliceO2@f6aa24b

But it still shows an another issue as below:

[15:42:22][INFO] importParticles 0
[15:42:22][INFO] Printing particles that are appended
[15:42:22][INFO] Particle 0 is a 2224 with status -1610094078 and px = -14.2135 py = -8.90426 pz = 19.1223
[15:42:22][INFO] Particle 1 is a 2212 with status -1610566143 and px = -12.8402 py = -7.8437 pz = 17.0538
[15:42:22][INFO] Particle 2 is a 211 with status -1610566143 and px = -1.37331 py = -1.06056 pz = 2.0685
[15:42:22][INFO] Event generation took 0s and produced 3 primaries 
[15:42:22][INFO] GENEVENT CHANGING STATE TO SERVING
[15:42:22][INFO] ASSIGNED PIPE HANDLE 27
[15:42:22][INFO] INITTASK CHANGING STATE TO SERVING
[15:42:22][STATE] INITIALIZING TASK ---> READY
[15:42:22][STATE] READY ---> RUNNING
[15:42:22][INFO] fair::mq::Device running...
[15:42:23][INFO] INFO REQUEST RECEIVED
[15:42:23][INFO] Received config request
[15:42:23][INFO] config reply send 
[15:42:23][INFO] INFO REQUEST RECEIVED
[15:42:23][INFO] Received config request
[15:42:23][INFO] config reply send 
[15:42:30][INFO] Received device shutdown request (signal 15).
[15:42:30][INFO] Waiting for graceful device shutdown. Hit Ctrl-C again to abort immediately.
[15:42:30][ERROR] Some error/interrupt occurred on socket during receive
[15:42:30][INFO] CONDRUN CHANGING STATE TO STOPPED
[15:42:30][INFO] Waiting info thread
[15:42:30][STATE] RUNNING ---> READY
[15:42:30][STATE] READY ---> RESETTING TASK
[15:42:30][STATE] RESETTING TASK ---> DEVICE READY
[15:42:30][STATE] DEVICE READY ---> RESETTING DEVICE
[15:42:30][STATE] RESETTING DEVICE ---> IDLE
[15:42:30][STATE] IDLE ---> EXITING
[15:42:30][STATE] Exiting FairMQ state machine

Especially,

[15:42:30][ERROR] Some error/interrupt occurred on socket during receive

This error is coming from this line: https://github.com/AliceO2Group/AliceO2/blob/dev/run/O2PrimaryServerDevice.h#L438

If this issue is not trivial to solve, we would like to go back and produce the mc based on the old configuration to boost up the speed.

@benedikt-voelkel
Copy link
Contributor

Mh, is that reproducible?
It looks a bit odd that now the message received seems to be somewhat corrupt. Are you including any other updates in your setup besides this PR and the changes in O2 for the TDatabasePDG?

Otherwise, for now we should probably revert and investigate in more detail

@BongHwi
Copy link
Contributor Author

BongHwi commented Mar 19, 2024

Yes, I tested twice and the results were the same.
Now I'm trying to perform the same test on another system since I don't trust my MacOS env.

I'll update the result as soon as I get it.

@BongHwi
Copy link
Contributor Author

BongHwi commented Mar 20, 2024

I tested it on another system (Linux ubuntu 18.04) and.. it looks like it's working well...
So I have to say that it was an issue with the MacOS system.

I'll open the PR for the O2 update.

@benedikt-voelkel
Copy link
Contributor

benedikt-voelkel commented Mar 20, 2024

ok, thanks for checking. Good and somewhat bad news at the same time.
But let's focus on the production systems for now, which are Linux.
It might be worth to follow up on that, I will put something on my list. Honestly, I haven't run simulation on a Mac in a long time so I would need to see.

Once I get to it and if there is an issue, we will follow up in a ticket.

@BongHwi
Copy link
Contributor Author

BongHwi commented Mar 20, 2024

Thanks! And, of course, I have to rebuild the O2 system on my MacOS from scratch for a fair comparison.
Since we'd like to hurry the production for the upcoming approval campaign, I'd leave it as it is for the moment.

I attached the link to the PR for the O2.
AliceO2Group/AliceO2#12896

@BongHwi
Copy link
Contributor Author

BongHwi commented Apr 9, 2024

+async-label async-2022-pp-apass6-2023-PbPb-apass2

@BongHwi BongHwi deleted the ResoPDGCodeUpdate branch April 9, 2024 13:38
noferini pushed a commit that referenced this pull request Apr 12, 2024
From Pythia 8.303, The PDG code of Lambda(1520) has been updated to 102134 instead of 3124
noferini pushed a commit that referenced this pull request Apr 18, 2024
From Pythia 8.303, The PDG code of Lambda(1520) has been updated to 102134 instead of 3124
noferini pushed a commit that referenced this pull request Apr 19, 2024
From Pythia 8.303, The PDG code of Lambda(1520) has been updated to 102134 instead of 3124
benedikt-voelkel pushed a commit to benedikt-voelkel/O2DPG that referenced this pull request May 14, 2024
From Pythia 8.303, The PDG code of Lambda(1520) has been updated to 102134 instead of 3124
benedikt-voelkel pushed a commit that referenced this pull request May 14, 2024
From Pythia 8.303, The PDG code of Lambda(1520) has been updated to 102134 instead of 3124

(cherry picked from commit cd5b970)
benedikt-voelkel pushed a commit that referenced this pull request May 16, 2024
From Pythia 8.303, The PDG code of Lambda(1520) has been updated to 102134 instead of 3124

(cherry picked from commit cd5b970)
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