Skip to content
This repository

namespace changes broke `foo.*bar*?` syntax #1076

fperez opened this Issue November 30, 2011 · 4 comments

3 participants

Fernando Perez Min RK Thomas Kluyver
Fernando Perez

On master we now have:

In [2]: sys.*pt*?
ValueError                                Traceback (most recent call last)
/home/fperez/usr/lib/python2.7/site-packages/IPython/core/oinspect.pyc in psearch(self, pattern, ns_table, ns_search, ignore_case, show_all)
    760             if name not in ns_table:
    761                 raise ValueError('invalid namespace <%s>. Valid names: %s' %
--> 762                                  (name,ns_table.keys()))
    764         #print 'type_pattern:',type_pattern # dbg

ValueError: invalid namespace <user>. Valid names: ['alias', 'user_global', 'builtin', 'user_local']

We should add a test for this form of call before fixing it, so that we're sure the bug doesn't come back.

Min RK

Should be a tiny fix, but should it be user_local or user_global?

Fernando Perez

user_local, I think. Sorry I didn't fix it right away, I caught it during the lecture today and just moved on :)

Thomas Kluyver takluyver referenced this issue from a commit in takluyver/ipython December 01, 2011
Thomas Kluyver Fix wildcard search for new namespace model.
Closes gh-1076
Thomas Kluyver

Both, I think. See PR #1081.

Fernando Perez fperez closed this December 01, 2011
Fernando Perez

Closed by #1081.

Brian E. Granger ellisonbg referenced this issue from a commit January 10, 2012
Commit has since been removed from the repository and is no longer available.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.