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

Wrong intended assignment of CovMatrix in createEDM4hepFile.py #355

Closed
peremato opened this issue Aug 30, 2024 · 3 comments · Fixed by #353
Closed

Wrong intended assignment of CovMatrix in createEDM4hepFile.py #355

peremato opened this issue Aug 30, 2024 · 3 comments · Fixed by #353

Comments

@peremato
Copy link

With the following line, the output file is written with 21 zeros instead of the intended value of cov6f.

state.CovMatrix = cov6f

I guess it should be state.setCovMatrix(cov6f)

@tmadlener
Copy link
Contributor

I think it should also work with state.covMatrix

@jmcarcell jmcarcell linked a pull request Sep 2, 2024 that will close this issue
@peremato
Copy link
Author

peremato commented Sep 2, 2024

Have a look at what is stored. If I remember correctly I was getting a covmatrix fill with zeros. See the TBrowser output:
Screenshot 2024-09-02 at 10 43 05

@jmcarcell
Copy link
Contributor

Yes, this will be fixed in #353

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 a pull request may close this issue.

3 participants