Skip to content

Collider bit metadata#540

Merged
ChrisJChang merged 26 commits intomasterfrom
ColliderBit_metadata
Feb 2, 2026
Merged

Collider bit metadata#540
ChrisJChang merged 26 commits intomasterfrom
ColliderBit_metadata

Conversation

@ahye
Copy link
Member

@ahye ahye commented Jun 25, 2025

Added info files for all ColliderBit analysis and python scripts to extract information from these into JSON files to be used by HepData and the GAMBIT webpage.

@ahye ahye requested a review from ChrisJChang June 25, 2025 18:17
@ahye ahye self-assigned this Jun 25, 2025
Copy link
Collaborator

@ChrisJChang ChrisJChang left a comment

Choose a reason for hiding this comment

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

Looks all good to me.

@ChrisJChang
Copy link
Collaborator

I checked that this works after the fixes, and these generate sensible json files.

However, @ahye, I noticed that you also pushed some changes to some pythia backend files. Was this intended?

@ahye
Copy link
Member Author

ahye commented Jul 7, 2025

Hi @ChrisJChang,

The pythia files were the files we spent forever on Friday afternoon purging in Glasgow. I thought we got rid of them all in the last commit, and I can't see any of them in the overview of changed files. Which one(s) are still there?

@ChrisJChang
Copy link
Collaborator

@ahye ah, it looks like you did remove them. I must have been looking at the individual commits and seen they were changed (probably when you were changing them back).

@ChrisJChang
Copy link
Collaborator

Hi,

Just pinging this. I recall that this was being held off on being merged. Is this still the case, or can I merge this?

@ahye
Copy link
Member Author

ahye commented Feb 2, 2026

I think this is fine to be merged. @agbuckley might still want to fine-tune things, but we can keep this branch as a place to do more ColliderBit metadata work (I am thinking of adding some more validation stuff).

If you merge this @ChrisJChang, could you also merge master into this branch?

@ChrisJChang ChrisJChang merged commit 51e5e14 into master Feb 2, 2026
1 check passed
@agbuckley
Copy link
Collaborator

Hi @ChrisJChang @ahye , sorry for the late reply: I managed to leave the house without my phone today so GitHub 2FA locked me out (among a ton of other resulting problems!) Thanks for merging, no problems from my side as I think it's complete for now, and further developments can go in via this branch or a new one.

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