Skip to content

Conversation

@lmoehlenbrock
Copy link
Contributor

No description provided.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@@ -0,0 +1,1563 @@
{
Copy link
Contributor

@ovalle15 ovalle15 Feb 2, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove the api key, I also recommend to delete this key from your account.


Reply via ReviewNB

@@ -0,0 +1,1563 @@
{
Copy link
Contributor

@ovalle15 ovalle15 Feb 2, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you also update the name for polygon_geo --> polygon_geo_2 in the NDJSON annotation ?


Reply via ReviewNB

@@ -0,0 +1,1563 @@
{
Copy link
Contributor

@ovalle15 ovalle15 Feb 2, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line #3.    from PIL import Image

Also remove imports here, since we are already importing them at the top of the doc


Reply via ReviewNB

@@ -0,0 +1,1563 @@
{
Copy link
Contributor

@ovalle15 ovalle15 Feb 2, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line #8.    import gdal

Remove this since we are not using it


Reply via ReviewNB

@ovalle15 ovalle15 self-requested a review February 6, 2023 18:01
Copy link
Contributor

@ovalle15 ovalle15 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

@ovalle15 ovalle15 merged commit e2a8b34 into develop Feb 6, 2023
@ovalle15 ovalle15 deleted the lmoehlenbrock-patch-2 branch February 6, 2023 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants