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

add Python 3.11 correction factor #10

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Conversation

EwoudK
Copy link

@EwoudK EwoudK commented Nov 15, 2023

No description provided.

@aldbr
Copy link
Collaborator

aldbr commented Nov 21, 2023

How did you get those results?
I am a bit curious as correction factors are different for Intel and AMD with python3.9, but not with python3.11.

@EwoudK
Copy link
Author

EwoudK commented Nov 21, 2023

The correction factors are not different for different cpu vendors in python 3.11 mainly because I have only done the analysis irrespective of cpu vendor. I am in the process of gathering more data to do an analysis for Intel and one for AMD.

Comment on lines +8 to +13
},
"3.11": {
"cpu_brand": {
"Intel": 0.912400392678881,
"AMD": 0.912400392678881
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

If I understand correctly, these factors aim at aligning the DB12 score, computed with python3.11, with the HEPScore. Am I right?

factors.json was originally introduced to align the original DB12 score, computed with python2, with the DB12 score computed with python3.9.

The questions are:

  • do we want to introduce this HEPScore scaling factor directly in the DB12 repo?

    • DB12 can also be used outside of the LHC context
    • The HEPSPEC06 scaling factor was set in LHCbDIRAC
  • do we still want to be aligned with the original DB12 score (python2)?

    • DB12 can be used outside of the LHC context again.
    • Regarding LHCb MC sim applications, the raw python3.9 DB12 score was more accurate than the original DB12 score on AMD CPUs (and a bit less accurate on Intel but not significantly...)
    • May be we need to compare the raw python3.11 DB12 score to the LHCb MC sim applications now.

Any opinion about that @fstagni?

@chrisburr
Copy link
Member

Shall we also look at Python 3.12 at the same time?

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.

3 participants