-
Notifications
You must be signed in to change notification settings - Fork 53
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
Inconsistent repetition #3834
Labels
info-needed
Issue requires more information from poster
triage-needed
Issues needing to be assigned to the prospective feature owner
Comments
Were you using Copilot Edits, Chat, something else? Can you be a little more specific about what the request/response were? |
Thank you for your e-mail
After recent changes suggested by: GitHub CoPilot Chat, I was getting a 404 function on loading the page.
I submitted the relevant top section of the server.js file plus the top of the front end login.js file and asked what might be causing the error. In the response, Chat returned the exact same code I had originally presented in my question. Including this summary:
Summary
Ensure Endpoint Definition: Verify that the /api/getState endpoint is correctly defined in server.js.
Check Route Configuration: Ensure that the route configuration in server.js is correct and that the routes are properly set up to handle the /api/getState request.
Verify Client-Side Request: Ensure that the client-side code is making the correct request to the /api/getState endpoint.
By following these steps, you should be able to resolve the 404 Not Found error and ensure that the /api/getState endpoint is correctly handled by the server.
It did not notice that the problem was in the logic as the loadSate function was called in the opening DOMContentLoaded function, when no previous state had been saved. I deleted the line calling the loadState function and the error was resolved. The offending code had been previously presented by GitHub Chat.
It also includes the deprecated functions such as the deprecated document ready function instead of:
document.addEventListener('DOMContentLoaded', function() {
and it does this repeatedly, never learning from previous errors. It recently recommended that I load the npm package csurf, which is deprecated and has 2 vulnerabilities, instead of the current csrf package, which leads me to believe that its reference data for JavaScript is out of date, which is the disappointing for a paid service.
I'm trying to understand how much the in line GitHub CoPilot understands about the context of a fairly complex project like the one I am developing. It comes up with some very good relevant comment suggestions, which I frequently adopt. Lines of code are often very quickly completed, as I start to type, which is great, but I would like to know how much it can grasp the whole context of the project. It’s called GitHub CoPilot, so i wonder if it has access to my GitHub repository? Also, how much the in line CoPilot differs from the Chat in the sidebar, if at all. Some better information about its capabilities and limitations would be helpful.
…________________________________
From: Rob Lourens ***@***.***>
Sent: 13 January 2025 19:35
To: microsoft/vscode-copilot-release ***@***.***>
Cc: CharlieIoD ***@***.***>; Author ***@***.***>
Subject: Re: [microsoft/vscode-copilot-release] Inconsistent repetition (Issue #3834)
Were you using Copilot Edits, Chat, something else? Can you be a little more specific about what the request/response were?
—
Reply to this email directly, view it on GitHub<#3834 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AEWY2KFVDCV6DM4ZB33PGR32KQIO3AVCNFSM6AAAAABUYGCQESVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKOBYGAZDKNJUHE>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Thanks for the details. I'm going to close this under #2628 which covers these issues with it repeating the same response or not learning from the context |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
info-needed
Issue requires more information from poster
triage-needed
Issues needing to be assigned to the prospective feature owner
Type: Bug
GutHub Chat was asked to check a 714 line, front end, login.js file, to see if the state of the page was being saved the correct way in the right places. Instead of keeping the response to a simple yes or no then making specific changes, it returned the whole file. However, it got to line 508, then started the file again, in the middle of a dynamic html section of code, with the DOMContent loaded function as below:
tableRows += `
${order[i].ProdDesc}
£ ${(order[i].Price).toFixed(2)}
<td styledocument.addEventListener('DOMContentLoaded', function() {
const codeElement = document.getElementById('guestBarcode');
it then ran out at line 959 in the middle of a function.
This is not the first time GutHub Chat has done this.
Extension version: 0.23.2
VS Code version: Code 1.96.2 (fabdb6a30b49f79a7aba0f2ad9df9b399473380f, 2024-12-19T10:22:47.216Z)
OS version: Windows_NT x64 10.0.19045
Modes:
System Info
canvas_oop_rasterization: enabled_on
direct_rendering_display_compositor: disabled_off_ok
gpu_compositing: enabled
multiple_raster_threads: enabled_on
opengl: enabled_on
rasterization: enabled
raw_draw: disabled_off_ok
skia_graphite: disabled_off
video_decode: enabled
video_encode: unavailable_off
vulkan: disabled_off
webgl: enabled
webgl2: enabled
webgpu: enabled
webnn: disabled_off
A/B Experiments
The text was updated successfully, but these errors were encountered: