Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Added documentation for create_gantt function #2543

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

Closed
wants to merge 3 commits into from
Closed

Added documentation for create_gantt function #2543

wants to merge 3 commits into from

Conversation

anuraghuram
Copy link

Please see suggested docstrings in lines 852-854.

@anuraghuram
Copy link
Author

Question: I see in the original issue documentation is also required for thedata (and description) parameters. Can you please clarify how datamight differ from the df parameter?

@emmanuelle
Copy link
Contributor

Hi @anuraghuram thank you for your PR! I think git treats this file as a completely new file because its path has changed, could you please move it (not git move it) to the existing path packages/python/plotly/plotly/figure_factory/_gantt.py and delete the file at the root of the repo in the index?

@anuraghuram
Copy link
Author

Hi @anuraghuram thank you for your PR! I think git treats this file as a completely new file because its path has changed, could you please move it (not git move it) to the existing path packages/python/plotly/plotly/figure_factory/_gantt.py and delete the file at the root of the repo in the index?

Hi @emmanuelle: new to github - can you please explain how to delete the file at the root?

@gvwilson
Copy link
Contributor

Hi - we are trying to tidy up the stale issues and PRs in Plotly's public repositories so that we can focus on things that are still important to our community. Since this one has been sitting for several years, I'm going to close it; if it is still a concern, please add a comment letting us know what recent version of our software you've checked it with so that I can reopen it and add it to our backlog. Thanks for your help - @gvwilson

@gvwilson gvwilson closed this Jun 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants