CosmiQ / CometTS

Comet Time Series Toolset for working with a time-series of remote sensing imagery and user defined polygons
Apache License 2.0
63 stars 16 forks source link

Improve notebooks #11

Closed rmsare closed 5 years ago

rmsare commented 5 years ago

The notebooks run smoothly, thank you for providing them. Here are some small changes that could make them much more user-friendly.

Each notebook should have a title and one sentence explaining what it does. Ideally some inline Markdown cells should tell the user what to change and what's going on.

Instead of a single script pasted in the first cell, I suggest you separate the cells logically so the user can step through the analysis. This might be: one cell for imports, one cell for user-defined variables not specified in a widget, one cell per widget definition, and a final "run" cell. Or, if you don't want to use cells, please add header comments as you did in CSV_Creator.ipynb.

See examples of clean notebooks and example scripts here and here.

Finally, please populate the widget fields in CometTS_Visualizer.ipynb with relative paths to your San Juan example. This will allow the user to immediately see what CometTS can do.

(JOSS review thread)

rmsare commented 5 years ago

The default example is now ready to run, and the notebooks have been improved