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

Added case_sensitive_completion option to extension #1613

Merged
merged 3 commits into from
Jun 26, 2017

Conversation

jlstevens
Copy link
Contributor

This fixes a bug with recent IPython versions, namely the fact that completions are listed in a case-insensitive order (ignoring both the default behavior of dir and __dir__ as well as the semantic importance of case in Python identifiers).

As this monkey patches a particular function in IPython, the default of this parameter is False but you can enable it by adding a ~/.holoviews.rc:

import holoviews as hv
hv.extension.case_sensitive_completion=True

This is highly recommended as it allows you to index into Layout and Overlayas originally intended. Hopefully, this fix will make it's way into the next IPython release.

@jbednar
Copy link
Member

jbednar commented Jun 26, 2017

Looks good to me.

@jlstevens
Copy link
Contributor Author

jlstevens commented Jun 26, 2017

I've improved handling of the holoviews.rc file. You can now stick a 'holoviews.rc' at the top-level of the git repo and it will be used, instead of having to stick the file in the home directory (or using an environment variable).

@jlstevens
Copy link
Contributor Author

@jbednar @philippjfr I am open to suggestions for a better name than case_sensitive_completion. Otherwise ready for review.

@jbednar
Copy link
Member

jbednar commented Jun 26, 2017

Looks good to me. Ready to merge?

@philippjfr
Copy link
Member

Will merge once tests pass.

@philippjfr philippjfr merged commit e5a79b9 into master Jun 26, 2017
@jbednar jbednar deleted the sorted_completions_fix branch June 26, 2017 20:41
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants