Code review for new features in RavenPlot.set_zlim
Hi all, Adam Ginsburg, a grad student here at CU who does not work with yt, was kind enough to add some nice features to set_zlim in RavenPlot, which are the plots held in the PlotCollection. These features add some flexibility to adding ticks to the colorbars on images. Since he's not a regular yt user, I've submitted a code review on his behalf. Please, check it out and leave comments here, if you have any: http://review.enzotools.org/r/15/ Normally, an email would be automatically sent, but there was a small error. Matt is fixing that now. Britton
Hi Britton, I'm still working on the email bug, but I think this looks good to me. I've clicked on "Ship It!" :-) Thanks to both you and Adam! -Matt On Fri, Jul 10, 2009 at 10:30 AM, Britton Smith<brittonsmith@gmail.com> wrote:
Hi all,
Adam Ginsburg, a grad student here at CU who does not work with yt, was kind enough to add some nice features to set_zlim in RavenPlot, which are the plots held in the PlotCollection. These features add some flexibility to adding ticks to the colorbars on images.
Since he's not a regular yt user, I've submitted a code review on his behalf. Please, check it out and leave comments here, if you have any: http://review.enzotools.org/r/15/
Normally, an email would be automatically sent, but there was a small error. Matt is fixing that now.
Britton
_______________________________________________ Yt-dev mailing list Yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
participants (2)
-
Britton Smith
-
Matthew Turk