Skip to content

fixed broken formatting in docs - #184

Merged
lmeyerov merged 1 commit into
graphistry:masterfrom
khmariem:fix_broken_docs
Oct 14, 2020
Merged

lmeyerov merged 1 commit into
graphistry:masterfrom
khmariem:fix_broken_docs

Conversation

@khmariem

@khmariem khmariem commented Oct 14, 2020 •

Copy link
Copy Markdown
Contributor

The following pull request fixes #176

The formatting in the documentation was fixed by
correcting the indentation.
Sphinx does not allow any sort of alignment to be
executed, so for a correct display, the string
should be within the same indentation line of
the python code.

signed by: [email protected]

The formatting in the documentation was fixed by
correcting the indentation.
Sphinx does not allow any sort of alignment to be
executed, so for a correct display, the string
should be within the same indentation line of
the python code.

signed by: [email protected]
@lmeyerov
lmeyerov merged commit f7f3f86 into graphistry:master Oct 14, 2020
@lmeyerov

Copy link
Copy Markdown
Contributor

Excellent, thank you @khmariem !

lmeyerov added a commit that referenced this pull request Mar 13, 2022
fixed broken formatting in docs
pull Bot referenced this pull request in admariner/pygraphistry Mar 13, 2022
fixed broken formatting in docs
lmeyerov added a commit that referenced this pull request Aug 8, 2026
…ench

Perf magnitudes in a public repo double as an optimization roadmap. The
docstrings added with this change quoted build costs and break-even counts
outright; they now state the SHAPE of the constraint -- a grouped pass per type
column, one per label under label__X, spendable only by typed count shapes --
which is what a reviewer or a future maintainer needs to not delete the gate.
The numbers move to pyg-bench (#184), which is also the standing rule: claims
live in pins and receipts, not in prose.

Reasoning is deliberately NOT stripped: _dtype_kind still records that pandas'
is_integer_dtype raises on cudf ListDtype, because that is a correctness
constraint someone would otherwise 'simplify' away. Only the timings go.

Co-Authored-By: Claude Fable 5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01MF7uRZLKZaD6Q9FGWSmyXi
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.

[BUG] ReadTheDocs formatting broken

2 participants