Closed stichbury closed 1 year ago
I'm working on writing down further enhancements to the spaceflights tutorial, as well as various bits I'm finding about the CLI, hence I added this issue to the current sprint. However, other tasks have higher priority, and as such implementing those enhancements will likely need to be done at a later stage.
Some notes below. My north star is the 4-quadrant framework Diátaxis.
(Disclaimer: All statements below are my opinion and can be debated)
First of all, the website contains 2 calls to action that sound very similar: "Get started" and "Take the Tutorial". The two overlap a bit, because they both cover basic commands like kedro new
, starters, kedro run
and so on, but the spaceflights tutorial is of course much more comprehensive, and the "Get started" section contains lots of explanations that somehow get in the way. Wondering if this landing could be simplified. The "Get started" section is not in scope for this issue though, so I'll skip to the spaceflights tutorial.
I'm going to assume that the target audience for the tutorial are
In general, I think the tutorial contains too many explanations that are a bit distracting, and many optional parts that made me think a bit too much and could perhaps belong to a how-to guide. Now, some specific things:
git
part suggests Gitflow, but then the Flatiron blog post that comes right before doesn't mention it. Gitflow is a quite heavy workflow and most beginners will either pick something simple for their personal projects or adopt whatever their team is using, so I think we should be less opinionated about it. Also, many people using GUIs for git (like VS Code, Gitkraken and such) so maybe we don't even need to include a micro crash course on git.kedro lint
, kedro build-docs
and so on. We should replace or remove those.credentials.yml
ends up being tracked in version control or not, we could clarify that a bit.name: str = None
should be name: str | None = None
or name: Optional[str] = None
.__default__
pipeline is mentioned, it's not explained - I only understood it at the very end. If we don't want to explain it at that point, we could as well remove it.kedro viz
for the first time. We could be more purposeful about it..egg
files from Kedro at #2273.Just to note that #2273 is now purely about any changes to the framework to stop producing eggs. This ticket will take care of any associated docs changes.
One more: the default project name for the spaceflights
starter is Spaceflights
, and this might cause confusion (see https://github.com/kedro-org/kedro/pull/2364).
This comment records work done and work to do.
Reordered pages to keep hands-on separate from concepts
Renamed sections of docs to make the on-boarding more linear, particularly in the get started phase
Removed TL;DR summary page in the get started section and added those sections where appropriate
Revised the copy in the page about getting started to push the iris example out of the limelight
Removed the section about Git in the tutorial
Removed the option to copy/paste code — this is now an express tutorial without any actions required from the reader (except thought)
Removed mention of "command line" in docs and kept a consistent "terminal" or "terminal window" throughout
Added pipenv
and venv
back to installation/setup guide (alongside conda) (@astrojuanlu please can you review?)
Reviewed all the notes and work out if they're all needed. Likewise all the explanatory text that may "get in the way"
Made the workflow more explicit (we assume the reader isn't using a notebook, so should say so)
Removed deprecated commands like Kedro lint, Kedro build-reqs
Checked all the file paths
Made the "What you've learned" and "where to go next?" sections more explicit
Renamed throughout from "Kedro Tutorial" to "Spaceflights" to correspond with default name for starter project
Removed mention of eggs 🥚
Clarify whether credentials.yml is tracked in version control
There are a few extra tweaks we can make to the spaceflights tutorial following onboarding by @amandakys and @astrojuanlu and this ticket captures those. I'll expand them more fully in due course, but want to record the basics here.
From @amandakys
From @astrojuanlu Quite a lot to think about, but one thing I definitely need to fix immediately:
<I'll record the rest here shortly>