[yt-dev] RFC: Remove DD0010, DD0000 and tests/

Britton Smith brittonsmith at gmail.com
Wed Feb 5 13:14:45 PST 2014


+1


On Wed, Feb 5, 2014 at 4:13 PM, Nathan Goldbaum <nathan12343 at gmail.com>wrote:

> We use DD0010 in the plot window answer tests. Removing it won't break
> anyone's testing setup (unless they've se the test_data_dir to be the
> tests directory in the repo), it will just make the dataset harder to
> find for someone setting up the tests locally.
>
> That said, it could be easily switched out with another dataset or
> just put up on yt-project.org/data.
>
> +1 on removing the tests directory.
>
> On Wed, Feb 5, 2014 at 1:08 PM, Matthew Turk <matthewturk at gmail.com>
> wrote:
> > Hi all,
> >
> > My understanding is that we don't use the tests/ directory anymore,
> > ever.  It serves the purpose of having an example of writing your own
> > answer test runner, but we don't use it for that, either.
> > Additionally, we have the DD0010 and DD0000 datasets in there.  These
> > aren't *big*, but they are kind of pointless.
> >
> > Proposals:
> >
> > 1) Remove all *.py files from this directory, as we do not use them
> > and our tests are elsewhere
> > 2) Remove DD0010 since we have lots of examples on yt-project.org/data/.
> >
> > -Matt
> > _______________________________________________
> > yt-dev mailing list
> > yt-dev at lists.spacepope.org
> > http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.or
> _______________________________________________
> yt-dev mailing list
> yt-dev at lists.spacepope.org
> http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.spacepope.org/pipermail/yt-dev-spacepope.org/attachments/20140205/e31a2f81/attachment.htm>


More information about the yt-dev mailing list