search

reproducible-notebooks / ERDDAP_timeseries_explorer

Simple interactive ERDDAP time series explorer using Jupyter Widgets
MIT License
6 stars 4 forks source link

Finalize refactoring #37

Closed apkrelling closed 3 years ago

apkrelling commented 3 years ago

After this PR, the refactor of this notebook should be complete. @ocefpaf please let me know if there is anything else you would like me to do concerning refactoring.

review-notebook-app[bot] commented 3 years ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

ocefpaf commented 3 years ago

@apkrelling this is IMO the best we can do on the refactor. Let's tackle the GUI now and any extra changes required we can do while we implement the Panel interface.

apkrelling commented 3 years ago

@rsignell-usgs , this PR completes the second ("refactor notebook") phase of the work.

cc @ocefpaf