Skip to content

Conversation

neha-sharma-geoai
Copy link
Collaborator

Changes made -

  • Updated the language of the notebook from start to end. Please read the language once, and kindly let me know if any changes are required.
  • Commented code cells to explain the code and added explanatory markdown cells wherever required.
  • Added cells to fetch, download, and unzip the training data.
  • Updated data item thumbnail.
  • Added test data and provided it as an item on ArcGIS Online.
  • Trained the model again with more epochs and provided it as an item on ArcGIS Online.
  • Added an overview of inference ways, and markdown cells with well-commented inference scripts.
  • Updated test outputs image and conclusion.

Kindly let me know if any more changes are required.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@neha-sharma-geoai neha-sharma-geoai changed the title Update "detecting-airplanes-on-satellite-imagery-using-deep-learning.… [Notebooks PR] Update "detecting-airplanes-on-satellite-imagery-using-deep-learning.… Sep 11, 2025
@neha-sharma-geoai neha-sharma-geoai added learn Issues, questions, and enhancements related to learn module Sample Notebook labels Sep 11, 2025
@neha-sharma-geoai
Copy link
Collaborator Author

This notebook is missing from the navigation pane of the ESRI developer website. @jyaistMap

Copy link
Collaborator

@jyaistMap jyaistMap left a comment

Choose a reason for hiding this comment

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

This notebook is missing from the navigation pane of the ESRI developer website. @jyaistMap

HI Neha -

All samples and guides need to be properly referenced in the appropriate directory in the Python API Developer Website repo. In the documentation directory, there are 2 subdirectories:

  • python - for the current release in the 2.4.x line of the API
  • python-2-3 - for the documentation in the 2.3.x line of the API

Within each of these subdirectories, there are yml files which control the appearance and functionality of the Table of Contents of the production website:

  • guide.navigation.yml
  • samples.navigation.yml

Please submit a PR in the https://devtopia.esri.com/ArcGISDevelopers/python-api-doc repo:

  • it should add 2 lines to the ../documentation/python/samples.navigation.yml file
    • It needs to placed under the proper Table of Contents heading within the overall Deep Learning heading

  • the notebook looks good when rendered locally

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
learn Issues, questions, and enhancements related to learn module Sample Notebook
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants