search

LinkedEarth / PyleoTutorials

Jupyter-based, science-driven tutorials for using the LinkedEarth data-software Python ecosystem
http://linked.earth/PyleoTutorials
Apache License 2.0
13 stars 6 forks source link

Notebook shows PCA usage #38

Closed CommonClimate closed 1 year ago

CommonClimate commented 1 year ago

(will fail CI as the changes are not yet in the master branch)

review-notebook-app[bot] commented 1 year ago

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.

Powered by ReviewNB

CommonClimate commented 1 year ago

This review was less than helpful, as it made a bunch of scattershot comments on nearly every notebook but the one this branch created. Please keep the comments focused otherwise we can't make progress.

khider commented 1 year ago

Well, then tell me which notebook. Because I went by the differences shown in the NBReview and all of them had changes.

CommonClimate commented 1 year ago

Sorry, the PR is called "Notebook shows PCA usage", so I thought it implicit that it was the PCA notebook I wanted feedback on.