Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #79 vector zonal stats html page #80

Merged
merged 1 commit into from
Jul 6, 2022

Conversation

butchtm
Copy link
Collaborator

@butchtm butchtm commented Jul 6, 2022

Fix #79

  • cause: possible bug in nbdev_update_nbs so that the _all_ code in the notebook (which gets turned into a #nbdev_comment _all_ in the code) is not converted back into a cell with the original _all_ statement.
  • reminder to file issue in nbdev

@butchtm butchtm requested a review from jtmiclat July 6, 2022 02:39
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@github-actions
Copy link
Contributor

github-actions bot commented Jul 6, 2022

Visit the preview URL for this PR (updated for commit 9f36133):

https://geowrangler--pr80-fix-add-back-missing-g20pigbv.web.app

(expires Wed, 13 Jul 2022 02:42:36 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

@jtmiclat jtmiclat merged commit fed3e3a into master Jul 6, 2022
@jtmiclat jtmiclat deleted the fix/add-back-missing-internal-api branch July 6, 2022 02:59
@tm-kah-alforja tm-kah-alforja mentioned this pull request Jul 21, 2022
12 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Fix 02_vector_zonal_stats.html
2 participants