-
Notifications
You must be signed in to change notification settings - Fork 25
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
Pycorn does not recognize injection points properly #12
Comments
The injections points are getting correctly written in the .csv that I can generate via PyCorn but the -t option does not properly recognize them! |
Yes, please provide the res file, that will make it easier to debug. |
The file is zipped because GitHub does not allow the direct upload of .res files. Thank you, Ahmed! |
The correct positions should be: 0.0, 0.0, 2.39, 5.22, 8.2, 8.2, and 11.16. |
OK - I will take a look.. once I have time ;-) |
Cool! This would make things a lot easier. Thank you. |
I had a look with a HEX-viewer. Injection points are at (absolute mL): The module mode gets it wrong as well if you do:
gives:
But they are correctly read out from the file by the pycorn class: which returns: So when doing full-extraction meta1_read is only called twice (for 0.0 and 2.46).. need to investigate further.. |
Okay. Sounds good. I looked in your source but couldn't really find the problem. I also barely know how to run scripts... |
In def inject_det in the pycorn.py you only call two variables (for x,y) could that be a problem? |
This should be fixed with the latest dev-version. If you don't want to wait for me updating the PyPI package then just replace current pycorn.py with the one from dev-branch ( dcb350c ). Thanks for reporting :-) |
Works like a charm. Thank you very much for your hyper fast fix! |
Merged & Uploaded to PyPI. |
Hi,
I have a unicorn file with multiple injections points. However Pycorn is not able to properly recognize the injection points. In fact, it does recognize two injection points. However, these are not at the correct volume position and there are actually more than just two injection points. Any ideas on how to fix that?
I'm happy to provide the .res where the injections are not properly recognized.
The text was updated successfully, but these errors were encountered: