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
Attempt to fix dictionary for edm::AssociationVector #5631
Attempt to fix dictionary for edm::AssociationVector #5631
Conversation
…ions The exception message which states that there is an incorrect put had the order of the C++ class type and the three strings used to identify a data product in a non-standard order.
Changed all the EDProducers to be stream types and added override declarations. Also VSimpleProducer and AVSimpleProducer to fill their containers with data which changes each event. This will be used in tests to guarantee that the content requested from an event is what was stored.
Added missing dictionaries needed for edm::AssociationVector Wrote an ioread rule for edm::AssociationVector but a bug in ROOT keeps it from being called.
Once the edm::AssociationVector dictionary problems were fixed, it was possible to use the class in the View test. Added a pyroot reading test for edm::AssociationVector. Unfortunately, a bug in ROOT prevents reading from working correctly. This unit test is commented out for now but will be activated once the ROOT bug is fixed.
A new Pull Request was created by @Dr15Jones (Chris Jones) for CMSSW_7_3_X. Attempt to fix dictionary for edm::AssociationVector It involves the following packages: DataFormats/FWLite @cmsbuild, @Dr15Jones, @ktf, @nclopezo can you please review it and eventually sign? Thanks. |
+1 |
Attempt to fix dictionary for edm::AssociationVector
Andrea Rizzi reported a problem reading edm::AssociationVectors from pyroot. The problem was a cache internal to edm::AssociationVector was not getting cleared on each event. An ioread rule was added for the class, however it appears there is a bug in ROOT which prevents this rule from being used.
This pull request sets up unit tests for edm::AssociationVector to prove it works properly in the full framework. There is also a commented out test for reading in pyroot which can be used to test any fixes in ROOT for the problem.