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

Some problems with the ConfigManager #880

Closed
damianavila opened this issue Dec 17, 2015 · 4 comments · Fixed by #882
Closed

Some problems with the ConfigManager #880

damianavila opened this issue Dec 17, 2015 · 4 comments · Fixed by #882

Comments

@damianavila
Copy link
Member

Context: I have written a custom config manager but it is not working... looking for the issue, I just tried to setup the env path directly in the current ConfigManager with no luck:

If I change https://github.com/jupyter/notebook/blob/master/notebook/services/config/manager.py#L17
from
return os.path.join(jupyter_config_dir(), 'nbconfig')
to
return os.path.join(ENV_CONFIG_PATH[0], 'nbconfig')

and I still can not load the nbconfig from that conda env... ideas???

Even more, if I try the option the option
--ConfigManager.config_dir=path/to/env/etc/jupyter/nbconfig when I launch the notebook, it keeps loading nbconfig from the ./jupyter user space...

What other jupyter_config_dir-related things am I missing? Or it could be some problem upstream this? maybe traitlets?

pinging @takluyver @minrk who are probably interested, also @jdfreder who chat a little bit with me about this, although everyone else in invited 😉

@ellisonbg
Copy link
Contributor

With my proposal in #878 can you get by without a custom ConfigManager?

But the changes you are making should work fine.

@damianavila
Copy link
Member Author

yes @ellisonbg, I will comment in the issue soon...
This is just me playing with alternatives and I think I found and issue/bug because it is not working if I make the changes I suggested above... but I can not figure out why not yet 😉

@minrk
Copy link
Member

minrk commented Dec 18, 2015

The notebook app specifies that frontend config is always in the Juptyer config dir, so it's not really true that ConfigManager.config_dir is configurable, since it's not allowed to be set from config.

@damianavila
Copy link
Member Author

OK, so removing that line will fallback in the default at ConfigManager, making that config option to work... sound like a reasonable request for a PR, so here it is: #882 😉

@minrk minrk added this to the 4.1 milestone Dec 18, 2015
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 29, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants