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

company-jedi vs jedi #21

Closed
hisnawi opened this issue Jun 29, 2017 · 4 comments
Closed

company-jedi vs jedi #21

hisnawi opened this issue Jun 29, 2017 · 4 comments
Labels
duplicate issue already reported

Comments

@hisnawi
Copy link

hisnawi commented Jun 29, 2017

So I first installed jedi and found it very useful and nice, it includes full function arguments.

Then I read that if I have company already then I should install company-jedi. I already use company/irony/clang for C++. So I deleted jedi and jedi-core and installed company-jedi instead. Unfortunately, all I see now with completion is function names and no full parameters like when jedi was installed. Am I missing something?

I am also pleased with eldoc integration with irony/company. But I can't seem to make it work with python. Is there special setup I need to include?

@nabajour
Copy link

It might be that the problem with not getting the full parameter list is not a company-jedi issue but a jedi-core issue.

I had the same, trying to use jedi.el instead showed me that the issue was present in there also.

Using manual function to show parameters (type a function like os.open and then run M-x jedi:get-in-function-call) got me a message like this:
error "\"AttributeError(\\\"'Definition' object has no attribute 'get_code'... blablabla...

It looks like its linked to these issues in jedi: #294 and #296

Applying the changes by hand to jediepcserver.py and reinstalling the jedi server fixed the issue here.

@vexx9
Copy link

vexx9 commented Jan 18, 2018

What's the status here? Why isn't PR being merged?

@SergiyKolesnikov
Copy link

company-jedi still requires jedi-core (which you deleted).

@gonewest818 gonewest818 added the duplicate issue already reported label Mar 24, 2020
@gonewest818
Copy link
Collaborator

duplicated by #24, closing this one and redirecting the discussion to the other ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate issue already reported
Development

No branches or pull requests

5 participants