Hi All, a couple questions: 1. With help from Matt I'm currently working on a treecode gravitational potential calculation using the Octree. Looking at FindBindingEnergy in data_point_utilities, periodicity is not considered there. I think that that should be fixed, and I'll include it in this new code. Do we agree? 2. A few weeks ago I raised the issue that yt/utilities/_amr_utils/ContourFinding.pyx yt/analysis_modules/level_sets/contour_finder.py are broken after changeset fd91f581b0e0. Should I put this into the bug tracker on Trac? Thanks! Stephen Skory stephenskory@yahoo.com http://stephenskory.com/ 510.621.3687 (google voice)
Periodicity should definitely be taken into account. No comment on issue 2.
On Tue, Mar 15, 2011 at 12:28 PM, Stephen Skory
Hi All,
a couple questions:
1. With help from Matt I'm currently working on a treecode gravitational potential calculation using the Octree. Looking at FindBindingEnergy in data_point_utilities, periodicity is not considered there. I think that that should be fixed, and I'll include it in this new code. Do we agree?
2. A few weeks ago I raised the issue that
yt/utilities/_amr_utils/ContourFinding.pyx yt/analysis_modules/level_sets/contour_finder.py
are broken after changeset fd91f581b0e0. Should I put this into the bug tracker on Trac?
Thanks!
Stephen Skory stephenskory@yahoo.com http://stephenskory.com/ 510.621.3687 (google voice) _______________________________________________ Yt-dev mailing list Yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
I agree that the binding energy should be aware of periodicity.
Care should be taken with corner cases that wrap around the
domain--I've seen contours like this, they might be problematic.
Also no comment on issue 2
d.
On Tue, Mar 15, 2011 at 9:31 AM, Britton Smith
Periodicity should definitely be taken into account. No comment on issue 2.
On Tue, Mar 15, 2011 at 12:28 PM, Stephen Skory
wrote: Hi All,
a couple questions:
1. With help from Matt I'm currently working on a treecode gravitational potential calculation using the Octree. Looking at FindBindingEnergy in data_point_utilities, periodicity is not considered there. I think that that should be fixed, and I'll include it in this new code. Do we agree?
2. A few weeks ago I raised the issue that
yt/utilities/_amr_utils/ContourFinding.pyx yt/analysis_modules/level_sets/contour_finder.py
are broken after changeset fd91f581b0e0. Should I put this into the bug tracker on Trac?
Thanks!
Stephen Skory stephenskory@yahoo.com http://stephenskory.com/ 510.621.3687 (google voice) _______________________________________________ Yt-dev mailing list Yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
_______________________________________________ Yt-dev mailing list Yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
-- Sent from my Stone Tablet and carried by my Pterodactyl.
For periodicity, please do not put the check inside the loop. Check
outside the loop if it looks like it wraps around the domain, and
recenter it so that it does not.
For the second issue, I almost have a working import script to go from
Trac to BitBucket. Please don't report it there.
And it might be easier if you figured out why it was broken ratehr
than waiting on me. The issue is in the .pyx, not the .py file, and
it is that the buffer of potential contour linkages is not long
enough. I thought it was, based on the loop structure that followed.
If you expand it to the correct length it will work.
-Matt
On Tue, Mar 15, 2011 at 9:36 AM, david collins
I agree that the binding energy should be aware of periodicity.
Care should be taken with corner cases that wrap around the domain--I've seen contours like this, they might be problematic.
Also no comment on issue 2
d.
On Tue, Mar 15, 2011 at 9:31 AM, Britton Smith
wrote: Periodicity should definitely be taken into account. No comment on issue 2.
On Tue, Mar 15, 2011 at 12:28 PM, Stephen Skory
wrote: Hi All,
a couple questions:
1. With help from Matt I'm currently working on a treecode gravitational potential calculation using the Octree. Looking at FindBindingEnergy in data_point_utilities, periodicity is not considered there. I think that that should be fixed, and I'll include it in this new code. Do we agree?
2. A few weeks ago I raised the issue that
yt/utilities/_amr_utils/ContourFinding.pyx yt/analysis_modules/level_sets/contour_finder.py
are broken after changeset fd91f581b0e0. Should I put this into the bug tracker on Trac?
Thanks!
Stephen Skory stephenskory@yahoo.com http://stephenskory.com/ 510.621.3687 (google voice) _______________________________________________ Yt-dev mailing list Yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
_______________________________________________ Yt-dev mailing list Yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
-- Sent from my Stone Tablet and carried by my Pterodactyl. _______________________________________________ Yt-dev mailing list Yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
participants (4)
-
Britton Smith
-
david collins
-
Matthew Turk
-
Stephen Skory