Done, thanks for the input.<br><br><div class="gmail_quote">On Thu, Dec 9, 2010 at 6:10 PM, Matthew Turk <span dir="ltr"><<a href="mailto:matthewturk@gmail.com">matthewturk@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Actually, on reflection, you are correct: it should be only done on<br>
the root processor in any event.  There are no state changes.  If you<br>
use the root-only decorator to only have the routine<br>
print_key_parameters be run on root, that should be okay.<br>
<br>
Sorry, I don't know what I was thinking, you're right this should only<br>
be on root.<br>
<br>
-Matt<br>
<div><div></div><div class="h5"><br>
On Thu, Dec 9, 2010 at 2:58 PM, Matthew Turk <<a href="mailto:matthewturk@gmail.com">matthewturk@gmail.com</a>> wrote:<br>
> I think we shouldn't do it in that manner; I think you should instead<br>
> turn off logging on all but the root processor.  You can do this in<br>
> your script by importing ytcfg from yt.config and setting your warning<br>
> level to 50 on all the non-root processors.<br>
><br>
> -Matt<br>
><br>
> On Thu, Dec 9, 2010 at 2:55 PM, Britton Smith <<a href="mailto:brittonsmith@gmail.com">brittonsmith@gmail.com</a>> wrote:<br>
>> I'm playing around with inline analysis and just realized that the new print<br>
>> outs are done by every processor.  This could get ugly when running on 1000<br>
>> cores.  Is anyone opposed to me making the print out happen only on the root<br>
>> proc?<br>
>><br>
>> Britton<br>
>><br>
>> On Tue, Dec 7, 2010 at 3:37 PM, j s oishi <<a href="mailto:jsoishi@gmail.com">jsoishi@gmail.com</a>> wrote:<br>
>>><br>
>>> Hi Cam,<br>
>>><br>
>>> I don't think this should affect speed: first off, the parameter file<br>
>>> is usually only read once, and that is always a fast operation.<br>
>>> Slowing it down with a few console writes shouldn't be a rate limiter.<br>
>>> The print_stats() dives down the hierarchy, so for a dataset of any<br>
>>> consequence speedwise, that will be the rate limiting step.<br>
>>><br>
>>> j<br>
>>><br>
>>> On Tue, Dec 7, 2010 at 12:32 PM, Cameron Hummels<br>
>>> <<a href="mailto:chummels@astro.columbia.edu">chummels@astro.columbia.edu</a>> wrote:<br>
>>> > A little late, but +1.  As you note, I think we need to be careful we're<br>
>>> > not pushing *too* much information to the user (as then it all looks<br>
>>> > like noise), and also make sure these lookups don't slow down the<br>
>>> > process significantly.  It would be a shame to lose the speed of yt for<br>
>>> > this additional behavior.<br>
>>> ><br>
>>> > Cameron<br>
>>> ><br>
>>> > On 12/7/10 3:06 PM, Matthew Turk wrote:<br>
>>> >> Hi all,<br>
>>> >><br>
>>> >> Okay, it's pretty unanimous.  I'll make the changes and push 'em.<br>
>>> >>  Thanks!<br>
>>> >><br>
>>> >> -Matt<br>
>>> >><br>
>>> >> On Tue, Dec 7, 2010 at 12:04 PM, Sam Skillman <<a href="mailto:samskillman@gmail.com">samskillman@gmail.com</a>><br>
>>> >> wrote:<br>
>>> >>> +1<br>
>>> >>><br>
>>> >>> On Tue, Dec 7, 2010 at 11:29 AM, Britton Smith<br>
>>> >>> <<a href="mailto:brittonsmith@gmail.com">brittonsmith@gmail.com</a>><br>
>>> >>> wrote:<br>
>>> >>>> +1<br>
>>> >>>><br>
>>> >>>> On Tue, Dec 7, 2010 at 1:21 PM, j s oishi <<a href="mailto:jsoishi@gmail.com">jsoishi@gmail.com</a>> wrote:<br>
>>> >>>>> These kinds of simulation parameters are good to print. I'm a solid<br>
>>> >>>>><br>
>>> >>>>> +1<br>
>>> >>>>><br>
>>> >>>>> on both of these changes. I also love print_stats, and having more<br>
>>> >>>>> data there is a good thing.<br>
>>> >>>>><br>
>>> >>>>> j<br>
>>> >>>>><br>
>>> >>>>> On Tue, Dec 7, 2010 at 10:10 AM, Matthew Turk<br>
>>> >>>>> <<a href="mailto:matthewturk@gmail.com">matthewturk@gmail.com</a>><br>
>>> >>>>> wrote:<br>
>>> >>>>>> Hi all,<br>
>>> >>>>>><br>
>>> >>>>>> I would like to propose that we increase the verbosity of two<br>
>>> >>>>>> operations:<br>
>>> >>>>>><br>
>>> >>>>>> 1) When a parameter file is parsed, I would like several pieces of<br>
>>> >>>>>> information output via the "INFO" level of the logging handler.<br>
>>> >>>>>>  This<br>
>>> >>>>>> would be the current simulation time, the dimensions of the domain,<br>
>>> >>>>>> and any cosmological parameters.<br>
>>> >>>>>> 2) When print_stats is called in the hierarchy, I would like to<br>
>>> >>>>>> include all of the cosmological parameters.<br>
>>> >>>>>><br>
>>> >>>>>> The reason I'm bringing this up rather than just changing it is<br>
>>> >>>>>> that<br>
>>> >>>>>> in the past yt has (rightly) been accused of being too verbose (in<br>
>>> >>>>>> fact, it was one of the first things Jeff said to me the first time<br>
>>> >>>>>> we<br>
>>> >>>>>> met to talk about yt!) and I don't want to increase that<br>
>>> >>>>>> unnecessarily.  But, I feel strongly that as we move into more<br>
>>> >>>>>> codes<br>
>>> >>>>>> we need to be more upfront about what yt thinks of a given<br>
>>> >>>>>> simulation<br>
>>> >>>>>> output.  I think that this will also help ensure that errors are<br>
>>> >>>>>> avoided.<br>
>>> >>>>>><br>
>>> >>>>>> +1/-1?<br>
>>> >>>>>><br>
>>> >>>>>> -Matt<br>
>>> >>>>>> _______________________________________________<br>
>>> >>>>>> Yt-dev mailing list<br>
>>> >>>>>> <a href="mailto:Yt-dev@lists.spacepope.org">Yt-dev@lists.spacepope.org</a><br>
>>> >>>>>> <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
>>> >>>>>><br>
>>> >>>>> _______________________________________________<br>
>>> >>>>> Yt-dev mailing list<br>
>>> >>>>> <a href="mailto:Yt-dev@lists.spacepope.org">Yt-dev@lists.spacepope.org</a><br>
>>> >>>>> <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
>>> >>>><br>
>>> >>>> _______________________________________________<br>
>>> >>>> Yt-dev mailing list<br>
>>> >>>> <a href="mailto:Yt-dev@lists.spacepope.org">Yt-dev@lists.spacepope.org</a><br>
>>> >>>> <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
>>> >>>><br>
>>> >>><br>
>>> >>> _______________________________________________<br>
>>> >>> Yt-dev mailing list<br>
>>> >>> <a href="mailto:Yt-dev@lists.spacepope.org">Yt-dev@lists.spacepope.org</a><br>
>>> >>> <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
>>> >>><br>
>>> >>><br>
>>> >> _______________________________________________<br>
>>> >> Yt-dev mailing list<br>
>>> >> <a href="mailto:Yt-dev@lists.spacepope.org">Yt-dev@lists.spacepope.org</a><br>
>>> >> <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
>>> >><br>
>>> > _______________________________________________<br>
>>> > Yt-dev mailing list<br>
>>> > <a href="mailto:Yt-dev@lists.spacepope.org">Yt-dev@lists.spacepope.org</a><br>
>>> > <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
>>> ><br>
>>> _______________________________________________<br>
>>> Yt-dev mailing list<br>
>>> <a href="mailto:Yt-dev@lists.spacepope.org">Yt-dev@lists.spacepope.org</a><br>
>>> <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
>><br>
>><br>
>> _______________________________________________<br>
>> Yt-dev mailing list<br>
>> <a href="mailto:Yt-dev@lists.spacepope.org">Yt-dev@lists.spacepope.org</a><br>
>> <a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
>><br>
>><br>
><br>
_______________________________________________<br>
Yt-dev mailing list<br>
<a href="mailto:Yt-dev@lists.spacepope.org">Yt-dev@lists.spacepope.org</a><br>
<a href="http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org" target="_blank">http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org</a><br>
</div></div></blockquote></div><br>