<div dir="ltr">On Sat, Apr 28, 2018 at 10:46 PM Ralf Gommers <<a href="mailto:ralf.gommers@gmail.com">ralf.gommers@gmail.com</a>> wrote:<br>><br>> On Mon, Apr 2, 2018 at 11:50 AM, Warren Weckesser <<a href="mailto:warren.weckesser@gmail.com">warren.weckesser@gmail.com</a>> wrote:<br><br>>> (c) We actually *use* the dataset in one of *our* docstrings or tutorials.  I don't think our datasets package should become a repository of interesting scientific data with no connection to the scipy code.  Its purpose should be to enrich our documentation.  (Note that by this criterion, the recently added ECG signal would not qualify!)<br>><br>> I'd add the criterion that we should *only* use any dataset in the docs. Hence there are zero internal imports, and the whole datasets submodule can then very simply be stripped for space-constrained usage scenarios. (in those cases a separate package would help even)<br><br>I believe that one of the motivations for adding the ECG dataset was to make some of the scipy.signal unit tests more realistic. Is that something you'd like to forbid? On the one hand, if you're strapped for space, you probably want to remove the test suites as well. On the other hand, you do want to be able to test your stripped installation!<br><br>--<br>Robert Kern</div>