field parameters and extract_connected_sets

Hi,
Is there a way of passing field parameters to extract_connected_sets?
Currently,
print dd["NegEscapeVelocity"]
works fine but
contours = dd.extract_connected_sets("NegEscapeVelocity", 1, 30.0, maxv, log_space=False)
complains it can't find the set field variables.
Elizabeth

I wasn't able to reproduce your error. Is "NegEscapeVelocity" a derived field, and do you have the same problem if you test it out on "Density" ?
From G.S.
On Thu, Oct 20, 2011 at 1:32 AM, Elizabeth Tasker < tasker@astro1.sci.hokudai.ac.jp> wrote:
Hi,
Is there a way of passing field parameters to extract_connected_sets?
Currently,
print dd["NegEscapeVelocity"]
works fine but
contours = dd.extract_connected_sets("NegEscapeVelocity", 1, 30.0, maxv, log_space=False)
complains it can't find the set field variables.
Elizabeth _______________________________________________ yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org

Hi,
NegEscapeVelocity is a field I created that needs a field parameter. Density doesn't need any field parameters and works fine. I have the same problem if I try it with any field that needs a field parameter, for example:
contours = dd.extract_connected_sets("Radius", 1, 30.0, maxv, log_space=False)
fails because it needs:
center = data.get_field_parameter("center")
I can get around this by making the parameters I need global variables, I just wondered if there was a way to pass field variables to this routine?
Elizabeth
On Oct 20, 2011, at 6:05 PM, Geoffrey So wrote:
I wasn't able to reproduce your error. Is "NegEscapeVelocity" a derived field, and do you have the same problem if you test it out on "Density" ?
From G.S.
On Thu, Oct 20, 2011 at 1:32 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote: Hi,
Is there a way of passing field parameters to extract_connected_sets?
Currently,
print dd["NegEscapeVelocity"]
works fine but
contours = dd.extract_connected_sets("NegEscapeVelocity", 1, 30.0, maxv, log_space=False)
complains it can't find the set field variables.
Elizabeth _______________________________________________ yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org

Hi Elizabeth,
Looks like it was only retaining "center", not all the field parameters. I've pushed a fix in 5a39ef71743a that should fix the situation. Let me know if you run into any more problems!
-Matt
On Thu, Oct 20, 2011 at 4:32 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote:
Hi,
Is there a way of passing field parameters to extract_connected_sets?
Currently,
print dd["NegEscapeVelocity"]
works fine but
contours = dd.extract_connected_sets("NegEscapeVelocity", 1, 30.0, maxv, log_space=False)
complains it can't find the set field variables.
Elizabeth _______________________________________________ yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org

Hi Matt,
Still no luck, although possibly I'm updating incorrectly. When I run bin/yt update, I get:
yt module located at: /home/taskere/yt/src/yt-hg
The current version of the code is:
--- 8bcf892c82d2 (yt) tip ---
which isn't the # you list. Does this mean I don't have your fix and if so, how do I get it? :/
Elizabeth
On Oct 20, 2011, at 9:21 PM, Matthew Turk wrote:
Hi Elizabeth,
Looks like it was only retaining "center", not all the field parameters. I've pushed a fix in 5a39ef71743a that should fix the situation. Let me know if you run into any more problems!
-Matt
On Thu, Oct 20, 2011 at 4:32 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote:
Hi,
Is there a way of passing field parameters to extract_connected_sets?
Currently,
print dd["NegEscapeVelocity"]
works fine but
contours = dd.extract_connected_sets("NegEscapeVelocity", 1, 30.0, maxv, log_space=False)
complains it can't find the set field variables.
Elizabeth _______________________________________________ yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org

Hi Elizabeth,
On Fri, Oct 21, 2011 at 3:14 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote:
Hi Matt,
Still no luck, although possibly I'm updating incorrectly. When I run bin/yt update, I get:
yt module located at: /home/taskere/yt/src/yt-hg
The current version of the code is:
8bcf892c82d2 (yt) tip
which isn't the # you list. Does this mean I don't have your fix and if so, how do I get it? :/
That should have it. Can you run the test script I wrote, which works for me, and tell me if it passes? If it does not, then there is something going on in how you are generating the extraction:
http://paste.yt-project.org/show/1883/
Thanks,
-Matt
Elizabeth
On Oct 20, 2011, at 9:21 PM, Matthew Turk wrote:
Hi Elizabeth,
Looks like it was only retaining "center", not all the field parameters. I've pushed a fix in 5a39ef71743a that should fix the situation. Let me know if you run into any more problems!
-Matt
On Thu, Oct 20, 2011 at 4:32 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote:
Hi,
Is there a way of passing field parameters to extract_connected_sets?
Currently,
print dd["NegEscapeVelocity"]
works fine but
contours = dd.extract_connected_sets("NegEscapeVelocity", 1, 30.0, maxv, log_space=False)
complains it can't find the set field variables.
Elizabeth _______________________________________________ yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org

Elizabeth,
Recently (~2 weeks ago), I corrected a bug in the code with the "yt update" command, where previously running yt update pulled the changes but didn't actually rebuild the code. It should *now* work, but only if you've actually been able to rebuild to the current version. Just as a check, run: "yt instinfo -u" once (this function is identical to "yt update" but was fixed long ago to rebuild things correctly), and then see if your problems go away. Sorry about this, if I'm the one responsible for your woes.
Cameron
On 10/21/11 7:51 AM, Matthew Turk wrote:
Hi Elizabeth,
On Fri, Oct 21, 2011 at 3:14 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote:
Hi Matt,
Still no luck, although possibly I'm updating incorrectly. When I run bin/yt update, I get:
yt module located at: /home/taskere/yt/src/yt-hg
The current version of the code is:
8bcf892c82d2 (yt) tip
which isn't the # you list. Does this mean I don't have your fix and if so, how do I get it? :/
That should have it. Can you run the test script I wrote, which works for me, and tell me if it passes? If it does not, then there is something going on in how you are generating the extraction:
http://paste.yt-project.org/show/1883/
Thanks,
-Matt
Elizabeth
On Oct 20, 2011, at 9:21 PM, Matthew Turk wrote:
Hi Elizabeth,
Looks like it was only retaining "center", not all the field parameters. I've pushed a fix in 5a39ef71743a that should fix the situation. Let me know if you run into any more problems!
-Matt
On Thu, Oct 20, 2011 at 4:32 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote:
Hi,
Is there a way of passing field parameters to extract_connected_sets?
Currently,
print dd["NegEscapeVelocity"]
works fine but
contours = dd.extract_connected_sets("NegEscapeVelocity", 1, 30.0, maxv, log_space=False)
complains it can't find the set field variables.
Elizabeth _______________________________________________ yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org

Hi Cameron,
Thanks -- I checked this but it turned out to be fine. I've since managed to run and then break Matt's code, so I now don't think it's a version update issue :)
Elizabeth
On 2011-10-22, at 3:30 AM, Cameron Hummels wrote:
Elizabeth,
Recently (~2 weeks ago), I corrected a bug in the code with the "yt update" command, where previously running yt update pulled the changes but didn't actually rebuild the code. It should *now* work, but only if you've actually been able to rebuild to the current version. Just as a check, run: "yt instinfo -u" once (this function is identical to "yt update" but was fixed long ago to rebuild things correctly), and then see if your problems go away. Sorry about this, if I'm the one responsible for your woes.
Cameron
On 10/21/11 7:51 AM, Matthew Turk wrote:
Hi Elizabeth,
On Fri, Oct 21, 2011 at 3:14 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote:
Hi Matt,
Still no luck, although possibly I'm updating incorrectly. When I run bin/yt update, I get:
yt module located at: /home/taskere/yt/src/yt-hg
The current version of the code is:
8bcf892c82d2 (yt) tip
which isn't the # you list. Does this mean I don't have your fix and if so, how do I get it? :/
That should have it. Can you run the test script I wrote, which works for me, and tell me if it passes? If it does not, then there is something going on in how you are generating the extraction:
http://paste.yt-project.org/show/1883/
Thanks,
-Matt
Elizabeth
On Oct 20, 2011, at 9:21 PM, Matthew Turk wrote:
Hi Elizabeth,
Looks like it was only retaining "center", not all the field parameters. I've pushed a fix in 5a39ef71743a that should fix the situation. Let me know if you run into any more problems!
-Matt
On Thu, Oct 20, 2011 at 4:32 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote:
Hi,
Is there a way of passing field parameters to extract_connected_sets?
Currently,
print dd["NegEscapeVelocity"]
works fine but
contours = dd.extract_connected_sets("NegEscapeVelocity", 1, 30.0, maxv, log_space=False)
complains it can't find the set field variables.
Elizabeth _______________________________________________ yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org

Hi Elizabeth,
Should be fixed in 7da720c9a9ea.
-Matt
On Sun, Oct 23, 2011 at 6:10 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote:
Hi Cameron,
Thanks -- I checked this but it turned out to be fine. I've since managed to run and then break Matt's code, so I now don't think it's a version update issue :)
Elizabeth
On 2011-10-22, at 3:30 AM, Cameron Hummels wrote:
Elizabeth,
Recently (~2 weeks ago), I corrected a bug in the code with the "yt update" command, where previously running yt update pulled the changes but didn't actually rebuild the code. It should *now* work, but only if you've actually been able to rebuild to the current version. Just as a check, run: "yt instinfo -u" once (this function is identical to "yt update" but was fixed long ago to rebuild things correctly), and then see if your problems go away. Sorry about this, if I'm the one responsible for your woes.
Cameron
On 10/21/11 7:51 AM, Matthew Turk wrote:
Hi Elizabeth,
On Fri, Oct 21, 2011 at 3:14 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote:
Hi Matt,
Still no luck, although possibly I'm updating incorrectly. When I run bin/yt update, I get:
yt module located at: /home/taskere/yt/src/yt-hg
The current version of the code is:
8bcf892c82d2 (yt) tip
which isn't the # you list. Does this mean I don't have your fix and if so, how do I get it? :/
That should have it. Can you run the test script I wrote, which works for me, and tell me if it passes? If it does not, then there is something going on in how you are generating the extraction:
http://paste.yt-project.org/show/1883/
Thanks,
-Matt
Elizabeth
On Oct 20, 2011, at 9:21 PM, Matthew Turk wrote:
Hi Elizabeth,
Looks like it was only retaining "center", not all the field parameters. I've pushed a fix in 5a39ef71743a that should fix the situation. Let me know if you run into any more problems!
-Matt
On Thu, Oct 20, 2011 at 4:32 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote:
Hi,
Is there a way of passing field parameters to extract_connected_sets?
Currently,
print dd["NegEscapeVelocity"]
works fine but
contours = dd.extract_connected_sets("NegEscapeVelocity", 1, 30.0, maxv, log_space=False)
complains it can't find the set field variables.
Elizabeth _______________________________________________ yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org

Fixed indeed -- thank you!!
Elizabeth
On Oct 25, 2011, at 4:28 AM, Matthew Turk wrote:
Hi Elizabeth,
Should be fixed in 7da720c9a9ea.
-Matt
On Sun, Oct 23, 2011 at 6:10 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote:
Hi Cameron,
Thanks -- I checked this but it turned out to be fine. I've since managed to run and then break Matt's code, so I now don't think it's a version update issue :)
Elizabeth
On 2011-10-22, at 3:30 AM, Cameron Hummels wrote:
Elizabeth,
Recently (~2 weeks ago), I corrected a bug in the code with the "yt update" command, where previously running yt update pulled the changes but didn't actually rebuild the code. It should *now* work, but only if you've actually been able to rebuild to the current version. Just as a check, run: "yt instinfo -u" once (this function is identical to "yt update" but was fixed long ago to rebuild things correctly), and then see if your problems go away. Sorry about this, if I'm the one responsible for your woes.
Cameron
On 10/21/11 7:51 AM, Matthew Turk wrote:
Hi Elizabeth,
On Fri, Oct 21, 2011 at 3:14 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote:
Hi Matt,
Still no luck, although possibly I'm updating incorrectly. When I run bin/yt update, I get:
yt module located at: /home/taskere/yt/src/yt-hg
The current version of the code is:
8bcf892c82d2 (yt) tip
which isn't the # you list. Does this mean I don't have your fix and if so, how do I get it? :/
That should have it. Can you run the test script I wrote, which works for me, and tell me if it passes? If it does not, then there is something going on in how you are generating the extraction:
http://paste.yt-project.org/show/1883/
Thanks,
-Matt
Elizabeth
On Oct 20, 2011, at 9:21 PM, Matthew Turk wrote:
Hi Elizabeth,
Looks like it was only retaining "center", not all the field parameters. I've pushed a fix in 5a39ef71743a that should fix the situation. Let me know if you run into any more problems!
-Matt
On Thu, Oct 20, 2011 at 4:32 AM, Elizabeth Tasker tasker@astro1.sci.hokudai.ac.jp wrote: > Hi, > > Is there a way of passing field parameters to extract_connected_sets? > > Currently, > > print dd["NegEscapeVelocity"] > > works fine but > > contours = dd.extract_connected_sets("NegEscapeVelocity", 1, 30.0, maxv, log_space=False) > > complains it can't find the set field variables. > > Elizabeth > _______________________________________________ > yt-users mailing list > yt-users@lists.spacepope.org > http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org > _______________________________________________ yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org
yt-users mailing list yt-users@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-users-spacepope.org

Hi Matt,
That should have it. Can you run the test script I wrote, which works for me, and tell me if it passes? If it does not, then there is something going on in how you are generating the extraction:
Exactly as you've written it, yes it does work. However, if you try and create the connected sets with TestField, then it fails, i.e. using
sets = dd.extract_connected_sets("TestField", 1, (rho1+rho2)/2.0, rho2)
It can't find the field parameter 'hello'.
Elizabeth
participants (4)
-
Cameron Hummels
-
Elizabeth Tasker
-
Geoffrey So
-
Matthew Turk