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

analyzing_growth_factors_of_airbnb_properties_in_new_york_city.ipynb sample notebook now running and fixed #1744

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

Conversation

tarunkukreja003
Copy link
Collaborator

sample notebook fixed and corrected
resolves 9750

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

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

@jyaistMap
Copy link
Collaborator

@tarunkukreja003

the cells where the map widget appear are not correctly displaying. For guides and samples, please refer to the How to embed map images in your notebooks wiki for instructions and let me know if you have questions.

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.

Map widget cells need updated.

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