Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ENH: register sample data #163

Merged
merged 4 commits into from
Jul 5, 2019
Merged

ENH: register sample data #163

merged 4 commits into from
Jul 5, 2019

Conversation

lopezmt
Copy link
Contributor

@lopezmt lopezmt commented Jun 18, 2019

No description provided.

Modules/Scripted/Home/Home.py Outdated Show resolved Hide resolved
Modules/Scripted/Home/Home.py Outdated Show resolved Hide resolved
Modules/Scripted/Home/Home.py Outdated Show resolved Hide resolved
Modules/Scripted/Home/Home.py Outdated Show resolved Hide resolved
@jcfr
Copy link
Member

jcfr commented Jun 21, 2019

@lopezmt Here is an updated topic.

Next steps:

@jcfr jcfr mentioned this pull request Jun 21, 2019
8 tasks
lopezmt and others added 4 commits July 5, 2019 12:14
… modules

* Update CMakeLists enabling SampleData module
* Add SampleData as a Home module dependency
* Register sampleDataSources associating a customDownloader asking
  the user where to download the files.
…ator"

This is a follow-up of 531328b (ENH: Rename/re-categorize SlicerSALT modules)
@jcfr jcfr force-pushed the sampledata-registration branch from ea004cf to 69658b4 Compare July 5, 2019 21:43
@jcfr
Copy link
Member

jcfr commented Jul 5, 2019

@bpaniagua Every module for which we provides sample data will have a "SampleData" tab with corresponding button to download the data.

  • Data Importer

image

  • SPHARM-PDM Generator

image

@jcfr
Copy link
Member

jcfr commented Jul 5, 2019

I am moving forward with the integration. We will tweak in follow-up commits.

@jcfr jcfr merged commit 6617d8e into master Jul 5, 2019
@jcfr jcfr deleted the sampledata-registration branch July 5, 2019 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants