RFC: Remove DD0010, DD0000 and tests/
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
+1
On Wed, Feb 5, 2014 at 2:08 PM, Matthew Turk matthewturk@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:
- 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@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
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@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:
- 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@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.or
+1
On Wed, Feb 5, 2014 at 4:13 PM, Nathan Goldbaum nathan12343@gmail.comwrote:
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@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:
- 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@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.or
yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
+!
On Wed, Feb 5, 2014 at 4:14 PM, Britton Smith brittonsmith@gmail.comwrote:
+1
On Wed, Feb 5, 2014 at 4:13 PM, Nathan Goldbaum nathan12343@gmail.comwrote:
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@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:
- 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@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.or
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
+1
On Thu, Feb 6, 2014 at 9:52 AM, j s oishi jsoishi@gmail.com wrote:
+!
On Wed, Feb 5, 2014 at 4:14 PM, Britton Smith brittonsmith@gmail.comwrote:
+1
On Wed, Feb 5, 2014 at 4:13 PM, Nathan Goldbaum nathan12343@gmail.comwrote:
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@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:
- 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@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.or
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
yt-dev mailing list yt-dev@lists.spacepope.org http://lists.spacepope.org/listinfo.cgi/yt-dev-spacepope.org
participants (6)
-
Britton Smith -
Cameron Hummels -
j s oishi -
Matthew Turk -
Nathan Goldbaum -
Sam Skillman