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

Relation bug #16

Merged
merged 4 commits into from
Apr 22, 2016
Merged

Relation bug #16

merged 4 commits into from
Apr 22, 2016

Conversation

gaede
Copy link
Contributor

@gaede gaede commented Apr 22, 2016

added a use case for storing two relations to the same class inside one collection

  • there is a bug that prevents the relations from being restored correctly:

    zitpcx16946:build gaede$ ./tests/write

start processing

processing event 0

particle 0 has daughters: 2 3 4 5 and parents:

particle 1 has daughters: 2 3 4 5 and parents:

particle 2 has daughters: 6 7 8 9 and parents: 0 1

particle 3 has daughters: 6 7 8 9 and parents: 0 1

particle 4 has daughters: and parents: 0 1

particle 5 has daughters: and parents: 0 1

particle 6 has daughters: and parents: 2 3

particle 7 has daughters: and parents: 2 3

particle 8 has daughters: and parents: 2 3

particle 9 has daughters: and parents: 2 3

zitpcx16946:build gaede$ ./tests/read

reading event 0

Referenced hit has an energy of 23

particle 0 daughters: 2 3 4 5 and parents:

particle 1 daughters: 2 3 4 5 and parents:

particle 2 daughters: 2 3 4 5 and parents: 0 1

particle 3 daughters: 2 3 4 5 and parents: 0 1

particle 4 daughters: and parents: 0 1

particle 5 daughters: and parents: 0 1

particle 6 daughters: and parents: 0 1

particle 7 daughters: and parents: 0 1

particle 8 daughters: and parents: 0 1

particle 9 daughters: and parents: 0 1

 - added ExampleMC with parent/daughter relations
 - added example code to write.cpp and read.cpp
  - for simplicity write/read only one and print the
    parent/daughter relationships to the screen
  - calling ./tests/write followed bu ./tests/read
    shows that the realtions are correctly set in
    the writer but are corrupt when read back from
    the file
@hegner hegner merged commit 3253cc6 into AIDASoft:master Apr 22, 2016
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