# Should we include download of SampleData not available for commercial use?

**URL:** <https://discourse.slicer.org/t/should-we-include-download-of-sampledata-not-available-for-commercial-use/14717>\
**Category:** Development\
**Created:** [November 20, 2020, 8:41pm UTC](https://discourse.slicer.org/t/should-we-include-download-of-sampledata-not-available-for-commercial-use/14717 "2020-11-20T20:41:32Z")\
**Posts on this page:** 4\
**Page:** 1

<div class="post-metadata">

**Author:** ![jcfr](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/jcfr/32/17825_2.png) [@jcfr](https://discourse.slicer.org/u/jcfr)\
**Post date:** [November 20, 2020, 8:41pm UTC](https://discourse.slicer.org/t/should-we-include-download-of-sampledata-not-available-for-commercial-use/14717/1 "2020-11-20T20:41:32Z")

</div>

In both Slicer or extensions, datasets with specific restrictions can be distributed.

For example, the `CTA abdomen (Panoramix)` dataset coming from [Osirix DICOM image library](http://www.osirix-viewer.com/resources/dicom-image-library/) is available for download through the SampleData module interface but it is exclusively available for research and teaching. It is not possible, to redistribute or sell it, or use it for commercial purposes.

While the `Acknowledgement` section is explicit about it. See screenshot below, I suggest we update the SampleData module to (1) display a warning when used click on the download button or (2) remove datasets that are not freely available for any purpose.

Thoughts ?

![image](https://us1.discourse-cdn.com/flex002/uploads/slicer/original/3X/6/1/61ec9cc424604c7fb71df9e8a0087b79a9075545.png)

![image](https://us1.discourse-cdn.com/flex002/uploads/slicer/original/3X/0/8/08f4d48e85944c4d9055a4c004743277ca617755.png)

---

<div class="post-metadata">

**Author:** ![pieper](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/pieper/32/8_2.png) [@pieper](https://discourse.slicer.org/u/pieper)\
**Post date:** [November 20, 2020, 8:43pm UTC](https://discourse.slicer.org/t/should-we-include-download-of-sampledata-not-available-for-commercial-use/14717/2 "2020-11-20T20:43:24Z")

</div>

Maybe better (easier) to just remove it? We can also get sample data from TCIA, probably very similar but with fewer restrictions.

---

<div class="post-metadata">

**Author:** ![jcfr](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/jcfr/32/17825_2.png) [@jcfr](https://discourse.slicer.org/u/jcfr)\
**Post date:** [November 20, 2020, 9:13pm UTC](https://discourse.slicer.org/t/should-we-include-download-of-sampledata-not-available-for-commercial-use/14717/3 "2020-11-20T21:13:39Z")

</div>

Here is a pull request: [https://github.com/Slicer/Slicer/pull/5311](https://github.com/Slicer/Slicer/pull/5311)

Should we also remove these datasets:

- `Panoramix.mrb` from [SlicerDataStore](https://github.com/Slicer/SlicerDataStore/releases/tag/SHA256)
- `Panoramix-cropped.nrrd` from [SlicerTestingData](https://github.com/Slicer/SlicerTestingData/releases)

---

<div class="post-metadata">

**Author:** ![pieper](https://sea2.discourse-cdn.com/flex002/user_avatar/discourse.slicer.org/pieper/32/8_2.png) [@pieper](https://discourse.slicer.org/u/pieper)\
**Post date:** [November 20, 2020, 10:01pm UTC](https://discourse.slicer.org/t/should-we-include-download-of-sampledata-not-available-for-commercial-use/14717/4 "2020-11-20T22:01:00Z")

</div>

It looks like that volume is [used in a couple tests](https://github.com/Slicer/Slicer/search?q=CTAAbdomenPanoramix) but it should pretty easy to swap in a different volume and update the tests. One of the tests depends on the geometry of this volume, but we could just swap in dummy pixel data into the same header if we didn’t want to update the hard coded geometry data in the test.
