Added cell to create a project from a template #107
Conversation
Added cell to create a project from a template
| "A \"slug\" is the string of your Zooniverse username and your project name without the leading forward slash, for instance: \"username/project-name\". [Click here for more details](https://www.zooniverse.org/talk/18/967061?comment=1898157&page=1).\n" | ||
| "A \"slug\" is the string of your Zooniverse username and your project name without the leading forward slash, for instance: \"username/project-name\". [Click here for more details](https://www.zooniverse.org/talk/18/967061?comment=1898157&page=1).\n", | ||
| "\n", | ||
| "**If you have __not__ yet created a Zooniverse project do not provide a `slug_name` and run the subsequent cell**" |
There was a problem hiding this comment.
I recommend we change this to:
"The rubin.citsci package includes a method that creates a Zooniverse project from template. If you wish to use this feature, do not provide a slug_name and run the subsequent cell."
| "tags": [] | ||
| }, | ||
| "source": [ | ||
| "**Run the following cell if you would like to create a new Zooniverse project**" |
There was a problem hiding this comment.
I suggest changing this to:
"Run the following cell if you would like to create a new Zooniverse project from (the?) template"
There was a problem hiding this comment.
I changed it to Run the following cell if you would like to create a new Zooniverse project from the Vera Rubin template - let me know what you think!
| }, | ||
| "outputs": [], | ||
| "source": [ | ||
| "cit_sci_pipeline.project = cit_sci_pipeline.create_new_project_from_template()" |
There was a problem hiding this comment.
Something else to discuss is that people might still run this cell even if they do define a slugname. In that case, we would probably need to add some other controls, because do we want people to still be able to create a template project even if they separately defined the slug?
There was a problem hiding this comment.
We'll address this in a future PR about guardrails.
| }, | ||
| "outputs": [], | ||
| "source": [ | ||
| "cit_sci_pipeline.project = cit_sci_pipeline.create_new_project_from_template()" |
There was a problem hiding this comment.
My other thought here is that do we need to define the cit_sci_pipeline.project = part or should we just do _ = since we don't use it later?
There was a problem hiding this comment.
cit_sci_pipeline.project is used within the rubin.citsci package, but technically it's not necessary to declare it in the notebook itself since that already happens in the method - I'll remove the cit_sci_pipeline.project part of this line
Updated text for creating project/template
No longer using return val from create_new_project_from_template()
Added cell to create a project from a template to both
01and02notebooks. Also cleaned up some cells and wording regarding upgrading therubin.citscipackage.