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

Message when try to connect from another cliant #81

Open
asha-ndf opened this issue Nov 10, 2020 · 6 comments
Open

Message when try to connect from another cliant #81

asha-ndf opened this issue Nov 10, 2020 · 6 comments

Comments

@asha-ndf
Copy link

When I try to use xsqlite, this error error occurred.
If you want to connect to this kernel from an other client, you can use the ~~~.json file
But, I couldn't understand what should I do.

This message is whitten in (main.cpp line 65-67).
It seems that (main.cpp line 82-96) is same means, and easy to understand.

I think that it is better to replace (main.cpp line 65-67) by (main.cpp line 82-96).
I'm just beginner. I'm sorry if I have misunderstood.

@martinRenou
Copy link
Member

Thanks for opening an issue, indeed a message like the following could be added:

And then run for example: 
# jupyter console --existing ~~~.json

Note that if you want to use xeus-sqlite, the most common way of using it is to start it from the Jupyter Notebook or JupyterLab UI, selecting the SQlite kernel.

@asha-ndf
Copy link
Author

asha-ndf commented Nov 10, 2020

Yes. I want to use form other client, so I did
# jupyter notebook --ip=* --no-browser
My usage might be rare.

@marimeireles
Copy link
Member

Thanks for the tip @asha-ndf I can add it later or if you want to contribute it, you're also very welcome :)
And thanks @martinRenou

@martinRenou
Copy link
Member

It would be actually nicer if this code lived in xeus core, as it's duplicated for xeus-sqlite, xeus-python and xeus-cling right now. Maybe we could have a helper function or something that deals with all this.

This was referenced Nov 10, 2020
@asha-ndf
Copy link
Author

I wrote code, and sent a pull request. I don't have much knowledge about c++ .
Please validate it.

@martinRenou
Copy link
Member

Thanks a lot for your contribution, I just have a small change request on it though :)

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 a pull request may close this issue.

3 participants