Skip to content

Conversation

@neha-sharma-geoai
Copy link
Collaborator

✅ Enhancements Made to the Notebook & Workflow

The notebook and supporting workflow have been upgraded for clarity, reproducibility, and ease of use.
Below is a detailed breakdown of all improvements, organized by category.

  • Notebook Language and Documentation Improvements
    • Revised the language throughout the notebook for consistency, readability, and conceptual understanding.
    • Enhanced introduction to the notebook.
    • Expanded explanations across the workflow.
    • Added/edited markdown cells to separate conceptual sections, output analysis, and procedural steps.
    • Reorganized/added a few cells for better flow.
  • Inserted comments within code blocks to explain:
    • Purpose of each cell
    • Syntaxes
    • Parameter roles
    • Dependencies and assumptions
    • Expected outputs
  • All 8 input and intermediary datasets used in the notebook were identified, collected, and published to the api_data_owner ArcGIS Online account. Item IDs in the notebook were updated to reference the newly published items. Added cells for fetching the items in the notebook.
  • Added thumbnails and descriptions for each item on the ArcGIS Online portal.
  • Modified script to automatically replace hardcoded file paths containing employee username with a generic placeholder <USER_HOME>. Ensures the notebook can be safely shared publicly or across teams without exposing personal directory structures.
  • The trained model from extended epochs was missing. Re-trained the Mask R-CNN instance segmentation model at high epochs. Exported and published this trained model as a new item on ArcGIS Online. Added a cell to load the model directly from ArcGIS Online
  • Explained the custom toolbox methodology and added steps to add and run a custom toolbox with an embedded Python Script in ArcGIS Pro
  • Ensured that it is a run-all notebook.

Datasets -

Please let me know if any other change is required.
Renaming of the notebook is required, which will be done in a separate PR.
Currently, the custom toolbox is not generating the expected 3D results; an issue has been created for it.

@neha-sharma-geoai neha-sharma-geoai self-assigned this Nov 17, 2025
@neha-sharma-geoai neha-sharma-geoai added learn Issues, questions, and enhancements related to learn module Sample Notebook labels Nov 17, 2025
@review-notebook-app
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 reconstructing_3D_buildings_from_aerial_lidar_with_deep_learning.ipynb [Notebooks PR] Update reconstructing_3D_buildings_from_aerial_lidar_with_deep_learning.ipynb Nov 17, 2025
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