Skip to content

Command-line tool to help debug NumpyDocString #146

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

Merged
merged 8 commits into from
Jun 6, 2018

Conversation

jnothman
Copy link
Member

No description provided.

@stefanv
Copy link
Contributor

stefanv commented Nov 15, 2017

Useful!

@jnothman
Copy link
Member Author

jnothman commented Nov 15, 2017 via email

@jnothman
Copy link
Member Author

jnothman commented Nov 15, 2017 via email

@jorisvandenbossche
Copy link
Contributor

Works nicely!

@rgommers
Copy link
Member

Nice!

PY2 is still unhappy.

@rgommers rgommers added this to the v0.9.0 milestone Apr 1, 2018
@rgommers
Copy link
Member

rgommers commented Apr 1, 2018

There's a problem here with submodules that are not imported by default into the main namespace. E.g.

python -m numpydoc scipy.LowLevelCallable

works, but

python -m numpydoc scipy.signal.lfilter

gives AttributeError: module 'scipy' has no attribute 'signal'.


parts = args.import_path.split('.')

for split_point in range(len(parts), 0, -1)[::-1]:
Copy link
Member Author

Choose a reason for hiding this comment

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

Of course. This does the reversal twice. Silly me.

@jnothman
Copy link
Member Author

jnothman commented Jun 6, 2018

Merging shouldn't hurt; we can iterate later if necessary.

@jnothman jnothman merged commit 6f911f6 into numpy:master Jun 6, 2018
@bsipocz
Copy link
Member

bsipocz commented Apr 24, 2019

This command line tool is brilliant, saved me lots of time by avoiding to build the full docs. Thank you.

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.

5 participants