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

Github pages for graphs #379

Merged
merged 10 commits into from
Aug 6, 2021
Merged

Conversation

az85252
Copy link
Contributor

@az85252 az85252 commented Aug 3, 2021

How should I say that we need dataprofiler[reports] to be installed for requirements?

@az85252 az85252 changed the title Github pages for graph Github pages for graphs Aug 3, 2021
@JGSweets JGSweets added Documentation Improvements or additions to documentation Low Priority Cosmetic improvement or minor bug labels Aug 3, 2021
Copy link
Contributor

@AnhTruong AnhTruong left a comment

Choose a reason for hiding this comment

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

lgtm

.. code-block:: python


data = pd.Series([1, 2, 3], dtype=str)
Copy link
Contributor

Choose a reason for hiding this comment

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

specify the imports here too so user can just copy paste

@JGSweets
Copy link
Contributor

JGSweets commented Aug 6, 2021

FYI, the images will have to be uploaded in a separate PR.

@JGSweets JGSweets merged commit 128145f into capitalone:gh-pages Aug 6, 2021
@JGSweets JGSweets mentioned this pull request Aug 9, 2021
JGSweets pushed a commit that referenced this pull request Oct 5, 2022
* added files for reports and graphs folder in gh pages

* added files for how to plot histograms in gh pages

* fixed syntax for graphs.rst

* fixed syntax for graphs.rst

* fixed syntax for graphs.rst

* added images for graphs.rst

* fixed syntax for graphs.rst

* fixed syntax for graphs.rst
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation Improvements or additions to documentation Low Priority Cosmetic improvement or minor bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants