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

resolves issue#9773, fixed using_and_updating_GIS_content.ipynb #1738

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

tarunkukreja003
Copy link
Collaborator

@tarunkukreja003 tarunkukreja003 commented Jan 9, 2024

resolves #9773 , fixed using_and_updating_GIS_content.ipynb,
-> data is now available
-> fixed the error there is no attribute called baseMap for WebMap['baseMap'] with WebMap.basemap

environment used:
-> arcgis version - 2.2.0.1
-> arcgis pro - 3.2.1
-> python - 3.9.18

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link

review-notebook-app bot commented Jan 31, 2024

View / edit / reply to this conversation on ReviewNB

ManushiM commented on 2024-01-31T03:43:30Z
----------------------------------------------------------------

Replace the second sentence with - ArcGIS includes several classes to make use of this content, publish new items and update them when needed.


tarunkukreja003 commented on 2024-01-31T21:22:06Z
----------------------------------------------------------------

done

Copy link

review-notebook-app bot commented Jan 31, 2024

View / edit / reply to this conversation on ReviewNB

ManushiM commented on 2024-01-31T03:43:31Z
----------------------------------------------------------------

The publishing steps are similar to what is described earlier in the data preparation section

Let us query the operationalLayers section of the web scene to understand what the layers look like.


tarunkukreja003 commented on 2024-01-31T21:22:15Z
----------------------------------------------------------------

done

Copy link

review-notebook-app bot commented Jan 31, 2024

View / edit / reply to this conversation on ReviewNB

ManushiM commented on 2024-01-31T03:43:31Z
----------------------------------------------------------------

Now let us explore what the baseMap dictionary of the web scene looks like.


tarunkukreja003 commented on 2024-01-31T21:24:00Z
----------------------------------------------------------------

done

Copy link
Collaborator Author

done


View entire conversation on ReviewNB

Copy link
Collaborator Author

done


View entire conversation on ReviewNB

Copy link
Collaborator Author

done


View entire conversation on ReviewNB

@jyaistMap
Copy link
Collaborator

@tarunkukreja003

  • the cells where maps are supposed to appear are blank when previewing the site with a local build. We should correct these before merging this PR:

image

image

image

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.

We need to review this guide on how to draw the map widgets.

@tarunkukreja003
Copy link
Collaborator Author

maps now being displayed

Copy link
Collaborator

@ManushiM ManushiM left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants