-
Notifications
You must be signed in to change notification settings - Fork 3
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
xray/Shotset: out of memory? #7
Comments
Ah good point, this is a feature that should probably be added. Once the https://github.com/tjlane/thor/blob/master/src/python/xray.py#L1728 This isn't too hard of a fix, though. If you want to take a shot at it, feel free to create a new branch and open a pull request. If you don't feel up to it, I'll fix it (eventually). Here's an example of the kind of code you want, using pytables: https://github.com/tjlane/thor/blob/master/src/python/xray.py#L3528 As a short term solution, just break your simulation into smaller chunks and recombine them later. I noticed that I still need to implement the Shotset add function, so I can open a separate issue to take care of that sooner rather than later. Final note: the https://github.com/tjlane/thor/blob/master/src/python/xray.py#L3600 That's useful for splitting up simulations across many machines & then recombining them. |
Simulation of 20000 shots using the polar detector works. Thanks for mentioning the Rings class! |
Unfortunately, after running the Rings.simulate() with 20000 shots I get this:
|
Citing from the api docs:
"The key power/functionality of Shotset is that it provides a layer of
abstraction over the intensity data on disk. Specifically, it provides many
powerful capabilities for analyzing data sets that are larger than can
fit in memory."
However, when I want to create 20000 shots via simulate() then I get:
With 5000 shots it works without complains...
So the promise does not stand?
The text was updated successfully, but these errors were encountered: