Open KushalGaywala opened 8 months ago
We could also clean the output of the MypyAnalyzer and provide the results or just leave them as it is
Sure, we could write a MypyResultsParser
class that extracts the problematic classes, functions and attributes. This could then be used in both the SignatureAnalyzer
and JediAnalyzer
.
We need to get the line and the column of the method causing the error
Getting the column could be tricky, maybe we could match the problematic call in the code with regex?
found that both were not able to find signatures for completion.create but while the integration process of both signature and jedi they were able to fetch signature for this method.
I'm not sure if I understand this correctly. Did they not parse this line of the mypy results or just couldn't find the package and therefore no signatures? This line
your completion:62: error: Argument "messages" to "create" of "Completions" has incompatible type ...
should already be parsed in the current version of the SignatureAnalyzer
:
# SignatureAnalyzer.py:353
patterns_attribute = {
"has_attribute": re.compile(r'to \"(.+?)\" of \"(.+?)\" has'),
...
Adding support for more mypy resultant errors would help in less feedback iterations for a completion.
I analyzed both
level_1__ft__mypy_signature_5_steps__1_choice.yaml
andlevel_1__ft__mypy_jedi_5_steps__1_choice.yaml
and found that both were not able to find signatures forcompletion.create
but while the integration process of bothsignature
andjedi
they were able to fetch signature for this method.Here
client.chat.completions.create
results in error for wrong type of parameter. Hence, the result from analyzers should be the following:We need to get the line and the column of the method causing the error, and then pass to
JediAnalyzer
to get a specific signature that would really help and integrate functionality intoSignatureAnalyzer
, and see if it helps significantly.We could also clean the output of the
MypyAnalyzer
and provide the results or just leave them as it is. What you guys think @psaegert @anacarsi . Currently I will start evaluation of theJediAnalyzer
without solving these error to have some results.Whole trace of Error: