Skip to main content

Github Copilot– Some experimentation

I was reading the following blog post when I thought: "How good (or bad) would Github Copilot handle the scenario's mentioned in the post?" This post is the answer on this question. 

But before we dive in, I would suggest to first read the original post on the JetBrains blog: Critical Thinking in an AI-Powered World | The .NET Tools Blog (jetbrains.com).

Back?

Let’s get started!

I already created an XUnit Test project targeting .NET 8 and pasted the first snippet used in the post in a test class:

Now let’s ask the same question but not to the JetBrains AI assistant but to Github Copilot in Visual Studio:

Suggest a way to refactor the variable `now` so that I can control the value without depending on `DateTime.UtcNow`

Here is the response I got:

Similar to the JetBrains AI Assistant it suggests me to create my own abstraction and create an IDateTimeProvider interface. Too bad! Let us also mention the TimeProvider class and see if we get a better result:

Please use the ‘System.TimeProvider’ class found in .NET 8 and C# 12 instead

But again, the results turn out quite similar:

So far, we cannot make a different conclusion as in the original post.

Take 2!

Ok, let’s refocus our attention on the second part of the post and let us see how Copilot helps wht the implementation of the CalculateFallTimeAndVelocity method:

Let’s see what the suggestion is that the system comes up with:

Not bad either! 

But let us improve our understanding of the magic values by using the following prompt:

Move all constants to descriptive variables.

I just apply the suggestion and the result looks like this: 

She(he?) didn’t isolate the Factor value but with the comment in place I can understand the role of the 2 in the code above. 

We continue with the next prompt:

Set the value of Gravity to Earth’s gravity up to four decimal places of precision

This gives us the following result:

Too bad! Although the value itself is correct, it gives me a suggestion with 5 decimal places of precision:

We end with the last prompt:

Comment each line with valuable information that explains what’s happening

And this is our final result:

Conclusion

Based on this I could only agree with the conclusion of the original post but applied to Github Copilot:

Github Copilot can help you solve a new fascinating set of problems but does not claim to be infallible. Since it uses models trained on human data, it can sometimes be wrong. That’s why you should think critically about responses and always take steps to understand and verify the results of any LLM-based product.

Popular posts from this blog

Podman– Command execution failed with exit code 125

After updating WSL on one of the developer machines, Podman failed to work. When we took a look through Podman Desktop, we noticed that Podman had stopped running and returned the following error message: Error: Command execution failed with exit code 125 Here are the steps we tried to fix the issue: We started by running podman info to get some extra details on what could be wrong: >podman info OS: windows/amd64 provider: wsl version: 5.3.1 Cannot connect to Podman. Please verify your connection to the Linux system using `podman system connection list`, or try `podman machine init` and `podman machine start` to manage a new Linux VM Error: unable to connect to Podman socket: failed to connect: dial tcp 127.0.0.1:2655: connectex: No connection could be made because the target machine actively refused it. That makes sense as the podman VM was not running. Let’s check the VM: >podman machine list NAME         ...

Cache stampede: when our cache turned against us

While investigating some performance issues, we ran into an ASP.NET Core API that cached a fairly expensive aggregation query for 60 seconds. Under normal load, that was fine: one request rebuilds the cache, everyone else reads from it. Under peak load, dozens of requests would arrive in that same expiry window, all see a cache miss, and all fire the same expensive query in parallel. The database didn't like that. That was the moment when our caching layer stopped helping and started hurting. A burst of requests comes in at the same time, all miss the cache, and all go hammer the database or the downstream API at once. That's a cache stampede . The cache was supposed to protect our backend, and for a few hundred milliseconds it did the opposite. Why this happens IMemoryCache.GetOrCreate (and its async sibling) looks like it protects you, but it doesn't add any locking on its own. Look at the naive version: public async Task<Report> GetReportAsync(string key) ...

A complex system designed from scratch never works

A few years ago, I worked as an architect on a big mainframe rewrite. I still count it as one of my failures. Not because the technology was wrong, but because I couldn't convince the management team to simplify the approach. Years later, the organization is still struggling to get the new system up and running. I left the project at the time, because I couldn't put my name behind an approach that would take very long and cost a lot of money without a working system to show for it along the way. Gall’s Law That memory keeps coming back to me, because it's a textbook case of Gall's Law playing out in real life. Gall's Law , from John Gall's Systemantics , states it plainly: A complex system that works is invariably found to have evolved from a simple system that worked. A complex system designed from scratch never works, and it cannot be patched to make it work. You have to start over with a simple system that works. What does that mean in practice,...