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

Fix python imports when built without SIO support #490

Merged
merged 3 commits into from
Sep 22, 2023

Conversation

tmadlener
Copy link
Collaborator

BEGINRELEASENOTES

ENDRELEASENOTES

@kjvbrt do you want to give this a go?

@kjvbrt
Copy link
Contributor

kjvbrt commented Sep 21, 2023

Works for AlmaLinux9 now :)

16:53:49 [jsmiesko@fcc-ironic-03 podio (fix-py-import-no-sio =)]$ podio-dump output_fullCalo_SimAndDigi.root 
/home/jsmiesko/podio/install/python/podio/EventStore.py:4: FutureWarning: The EventStore based I/O model is deprecated and will be removed. Switch to the Frame based model.
  warnings.warn("The EventStore based I/O model is deprecated and will be removed. Switch to the Frame based model.",
input file: output_fullCalo_SimAndDigi.root

datamodel model definitions stored in this file: edm4hep

Frame categories in this file:
Name                 Entries   
-------------------------------
metadata             1         
events               2         
configuration_metadata 1         

#################################### events 0 ####################################
Collections:
Name                       ValueType                  Size  ID
-------------------------  -----------------------  ------  --------
CaloClusters               edm4hep::Cluster              1  706db4eb
CorrectedCaloClusters      edm4hep::Cluster              1  38fde31e
ECalBarrelCells            edm4hep::CalorimeterHit      68  269139f6
ECalBarrelPositionedCells  edm4hep::CalorimeterHit      68  65cb8aec
ECalEndcapCells            edm4hep::CalorimeterHit       0  55d4e559
GenParticles               edm4hep::MCParticle           1  0bcf5f90

Parameters:
Name    Type    Elements
------  ------  ----------

@tmadlener
Copy link
Collaborator Author

Thanks for checking.

@tmadlener tmadlener merged commit 590e3f2 into AIDASoft:master Sep 22, 2023
17 checks passed
Ananya2003Gupta pushed a commit to Ananya2003Gupta/podio that referenced this pull request Sep 26, 2023
* Do not unconditionally try to import the sio bindings

* Make regex a bit less error prone

* Avoid noisy but harmless ROOT message during podio builds
@tmadlener tmadlener deleted the fix-py-import-no-sio branch November 8, 2023 12:13
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.

New python import structure does not work if SIO backend is not built
2 participants