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
Inline completer displays ``` at the end of suggested code when using provider #686
Comments
Thank you for opening your first issue in this project! Engagement like this is essential for open source projects! 🤗 |
@krassowski Do you want to own this issue? 👀 |
It looks like the problem is two-fold:
I think (2) is easy to solve, I am happy to review a PR from new contributors here if anyone is interested. Ideally the PR would come with a new test case in |
@dlqqq @krassowski Thanks for bringing up this issue. I did a detailed analysis and provide my thoughts below. Installed Replicated this error on mac-os for the example above using both But it does not always give the same result (see the two examples below): Here we mimic the Clearly, adding special code to remove the ``` at the end as suggested here Further, the outcome of the transformer completion also depends on the Settings in JupyterLab. In the Settings The Issue can be reproduced when only the third box is checked as shown below. Checking any other configuration of the checkboxes generates other behavior, for example let’s check the second and third boxes and then we get the following examples: Checking the first and third boxes gives different behavior as well Checking all three boxes again alters behavior: A slightly different experience may arise depending on machine, environment, and code completion use cases. We may not want to fix jupyter-ai code for varying responses from the transformer used for JupyterLab completions. Also, the UX is altered with completions, so it is better to set all the three “Enabled” checkboxes off as default as different users may want completions versus not. We need to think/discuss the UX and generality of the code mods before going ahead. |
No, we do not want to special case anything in JupyterLab. I don't think that the LLMs should add any examples like on your second snapshot. Here is my point of view - there are two issues:
I do not understand how this relates to this issue. |
* Remove closing markdown identifiers (#686) * Remove whitespace after closing markdown identifier * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
As flagged in this Jupyterlab forum post; should be removed in post-processing.
The text was updated successfully, but these errors were encountered: