-
Notifications
You must be signed in to change notification settings - Fork 12
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
Avoiding phoSim indexing failures in Run 3 #312
Comments
Hmm. Do you know if phosim has been upgraded to use string or integer
indices, @TomGlanzman? Does anyone know whether there is an open ticket on
the phosim release bitbucket?
|
This doesn't seem to have been issued there: |
Good, thanks Jim. Rahul or Bryce, can you please write a phosim ticket and
ask for advice regarding a workaround? When you've made it, please post the
link to it here so we can follow along. Thanks!
|
drphilmarshall
changed the title
Indexing Failures for Run 3
Avoiding phoSim indexing failures in Run 3
Sep 21, 2016
I thought I saw that Bryce had already posted a link but : https://jira.lsstcorp.org/secure/RapidBoard.jspa?rapidView=12&projectKey=PHOSIM&view=detail&selectedIssue=PHOSIM-27 |
drphilmarshall
added
Epic: Run 3 phoSim Workflow
and removed
Epic: Run 3 phoSim Workflow
labels
Sep 30, 2016
This issue (specific to run 3) should be closed by #340 . However, I think we should track this general issue. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
@drphilmarshall
Is there a plan to avoid indexing failures due to phosim for run3 ? As a reminder in previous runs, we have found and reported that PhoSim tries to index objects by floats, and indices are messed up due to precision problems when these are large. I do not know if phosim was changed to avoid this problem. I forgot to mention this during the meeting.
A possibility could be checking out our own branch of phosim and modfiying the index to a string before writing it out. We can then tag this and use this tagged version. I will not have time to work on it immediately.
The text was updated successfully, but these errors were encountered: