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 errors in executing arcpy.sa.NbrAnnulus ArcGIS Pro #176

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

sdy623
Copy link

@sdy623 sdy623 commented Jan 29, 2024

No description provided.

@johngallo
Copy link
Member

@sdy623 Thank you so much for your contribution! One of our team tested this with the sample data a few months ago and it did not work properly. Last I heard he was going to try fixing it, but I'm assuming that has stalled. Please let me know if you would like me to pass on a more detailed report out.

@dkav
Copy link
Contributor

dkav commented May 17, 2024

I cannot test but the change looks correct. It is not the only neighborhood class update needed in this file. Defining neighborhood as a string and not as class goes back to ArcGIS 9.3. May be related to this Google Groups post.

@johngallo
Copy link
Member

Thanks @dkav ! Much appreciated. @sdy623 , dkav gives a good couple clues as to why it may have failed. (Note, the code tester ran it fine in ArcMap, but it failed in ArcPro using the similar but not exact parameters. He used the sample data in the toolbox I believe, and I can find the exact parameters if that helps. Pls let us know if you do not have access to ArcGIS Pro. Thanks again for your contribution!

@johngallo johngallo added bug Unexpected problem or unintended behavior barrier mapper Relating to Barrier Mapper tool labels May 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
barrier mapper Relating to Barrier Mapper tool bug Unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants