Fwd: [yt_analysis/yt] Change time series objects to *not* instantiate pfs by default (pull request #143)
Hi all, A couple people (Nathan and Britton) have mentioned memory problems with time series data because the StaticOutput objects sometimes hold on to hierarchies. I've taken a pass at preventing this from happening by keeping the static outputs from being instantiated until requested, and allowing the references to be passed rather than retained. Anyway, it works for me and I've checked that the references go to zero. If somebody else that was having memory issues could take a gander that'd be great. Here's my sample script; if you run before&after the numbers it prints out at the end should be "something" and "0". http://paste.yt-project.org/show/2302/ -Matt ---------- Forwarded message ---------- From: Matthew Turk <pullrequests-noreply@bitbucket.org> Date: Sun, Apr 22, 2012 at 10:52 AM Subject: [yt_analysis/yt] Change time series objects to *not* instantiate pfs by default (pull request #143) To: yt@enzotools.org A new pull request has been opened by Matthew Turk. MatthewTurk/yt has changes to be pulled into yt_analysis/yt. https://bitbucket.org/yt_analysis/yt/pull-request/143/change-time-series-obj... Title: Change time series objects to *not* instantiate pfs by default This change, as described in the commit message, will prevent time series data from initially instantiating any parameter file objects. By keeping the references to pfs temporary, this will prevent hierarchies from sticking around during the course of analysis, which should improve overall memory usage. Changes to be pulled: -- This is an issue notification from bitbucket.org. You are receiving this either because you are the participating in a pull request, or you are following it.
participants (1)
-
Matthew Turk