Skip to content

Model extension guide update #2251

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: next
Choose a base branch
from
Open

Model extension guide update #2251

wants to merge 1 commit into from

Conversation

neha-sharma-geoai
Copy link
Collaborator

Updated guide with updated scripts.

@neha-sharma-geoai neha-sharma-geoai added the learn Issues, questions, and enhancements related to learn module label Apr 4, 2025
@review-notebook-app 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
Copy link
Collaborator Author

Issue - https://github.com/ArcGIS/geosaurus/issues/12906

jyaistMap
jyaistMap requested changes Apr 7, 2025
View reviewed changes
Copy link
Collaborator

@jyaistMap jyaistMap left a comment •
edited
Loading

Choose a reason for hiding this comment

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

@neha-sharma-geoai @priyankatuteja @vikashranjan26

  • The new RT-DetrV2 Object Detector guide was merged into master with Added RTDetrV2guide #2250 so it was subsequently merged into next to sync those branches. Merging this full PR would be redundant. The guide is already available on production since it was merged into master, but there is no Table of Contents heading for it since no PR was made to the python-api-doc repo into the ../documentation/python/guide.navigation.yml file. Please submit a PR into the appropriate heading section so a heading is added to the documentation.
    image

    • Can you remove the guide file and the images from this PR so we can just merge updates to the using model extension guide?

  • Please submit a PR to the ../documentation/python/guide.navigation.yml into the appropriate section for the model extension guide. It is currently available on production but there is no Table of Contents heading for it.
    image

@neha-sharma-geoai neha-sharma-geoai added documentation Bugs and enhancement requests for doc (API ref and notebooks) Guide labels Apr 10, 2025
@neha-sharma-geoai
Copy link
Collaborator Author

neha-sharma-geoai commented Apr 15, 2025 •
edited
Loading

@jyaistMap
I have updated the PR to update only the required file. Pls check.
Is it ok to merge this PR with the next branch, or should it be merged with the master directly?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jyaistMap jyaistMap jyaistMap requested changes

@vikashranjan26 vikashranjan26 Awaiting requested review from vikashranjan26

@priyankatuteja priyankatuteja Awaiting requested review from priyankatuteja

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
documentation Bugs and enhancement requests for doc (API ref and notebooks) Guide learn Issues, questions, and enhancements related to learn module
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@neha-sharma-geoai @jyaistMap