arcgis-python-api icon indicating copy to clipboard operation
arcgis-python-api copied to clipboard

`fighting_california_forest_fires_using_spatial_analysis.ipynb` corrected

Open tarunkukreja003 opened this issue 1 year ago • 4 comments
trafficstars

fighting_california_forest_fires_using_spatial_analysis.ipynb corrected, resolves https://github.com/ArcGIS/geosaurus/issues/9821

environment details: cloned the latest Pro environment arcgis version - 2.2.0.1 Pro version - 3.2.1 python version - 3.9.18

please install puremagic in your environment before testing this sample notebook - pip install puremagic

tarunkukreja003 avatar Dec 15 '23 17:12 tarunkukreja003

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

View / edit / reply to this conversation on ReviewNB

ManushiM commented on 2024-01-31T03:49:55Z ----------------------------------------------------------------

Since item.share() is deprecated in 2.3.0, can we please replace this?


tarunkukreja003 commented on 2024-01-31T22:25:57Z ----------------------------------------------------------------

since the current publicly released version is 2.2.0.2, and since the sample notebook will be released to the public generally the next day after the PR is merged into the arcgis-python-api repo, the item.sharing.sharing_level will fail, since it will be supported from 2.3.0. As soon as the 2.3.0 version is released we can return to this notebook and make this change 

View / edit / reply to this conversation on ReviewNB

ManushiM commented on 2024-01-31T03:49:56Z ----------------------------------------------------------------

same comment as above


since the current publicly released version is 2.2.0.2, and since the sample notebook will be released to the public generally the next day after the PR is merged into the arcgis-python-api repo, the item.sharing.sharing_level will fail, since it will be supported from 2.3.0. As soon as the 2.3.0 version is released we can return to this notebook and make this change or we can create a new issue for this for 2.3.0 


View entire conversation on ReviewNB

tarunkukreja003 avatar Jan 31 '24 22:01 tarunkukreja003