zlacker

[parent] [thread] 11 comments
1. gfosco+(OP)[view] [source] 2021-10-27 19:28:57
Likewise skeptical, but I have been super impressed with it. I just got in to the technical preview, and worked through a specific task I needed to do (involving mongoose, a mongo aggregate query, a few loops, some date functions) and started by adding a comment above each line. It helped a lot actually, felt like a collab.

I'll reproduce a generic example I sent a friend.

Prompt:

    const redis = require('redis');

    // Create a redis client
Copilot suggested:

    const client = redis.createClient();
I accepted it and moved 2 lines down, prompt:

    // Get the current count from redis
Copilot suggested a start to the operation:

    client.get('count', (err, count) => {
I accepted it, and Copilot continued:

      if (err) {
        console.log(err);
        return;
      }

      // If there is no count, set it to 0
      if (!count) {
        count = 0;
      }

      // Increment the count
      count++;

      // Set the new count in redis
      client.set('count', count);
  
      // Log the new count
      console.log(count);
    }
replies(3): >>aaaaaa+c2 >>foepys+Y2 >>matsem+bc
2. aaaaaa+c2[view] [source] 2021-10-27 19:38:09
>>gfosco+(OP)
Why does it increment the count?
replies(1): >>gfosco+D3
3. foepys+Y2[view] [source] 2021-10-27 19:41:29
>>gfosco+(OP)
Redis has the INCR command that does this in redis without the additional round-trips (and race conditions). It also sets the value to 0 if the key doesn't exist.

So, I actually consider this to be exactly the bad behavior that people accuse Copilot of.

replies(2): >>gfosco+i6 >>speedg+w7
◧◩
4. gfosco+D3[view] [source] [discussion] 2021-10-27 19:44:02
>>aaaaaa+c2
I assume that in its training, incrementing a counter in redis is common.
◧◩
5. gfosco+i6[view] [source] [discussion] 2021-10-27 19:56:41
>>foepys+Y2
This was just one of its suggestions, but you're right of course.. it's all based on the training data and idioms used there. If it doesn't weight more modern code higher, if it's not aware of new versions and methods, it isn't going to be super intelligent.. but it can still give you some ideas.
◧◩
6. speedg+w7[view] [source] [discussion] 2021-10-27 20:03:32
>>foepys+Y2
Yes it's not very shinning there. I would also throw the error instead of printing the error to the console and returning undefined.
replies(1): >>ponyou+9t
7. matsem+bc[view] [source] 2021-10-27 20:28:19
>>gfosco+(OP)

  // Create a redis client
Writing that probably takes a longer time than just doing it with the IDE help in jetbrains, though?

Press "r", press "." to autocomplete so it now says "redis.", then write "cC" and it suggests createClient (or write "create" or "client" if you're not sure what you're looking for). Now it says "redis.createClient()". Press ctrl+alt+v to extract a variable or write ".const" and press tab to apply the const live template. Ending up with your result in two seconds.

replies(1): >>anigbr+2f1
◧◩◪
8. ponyou+9t[view] [source] [discussion] 2021-10-27 22:12:08
>>speedg+w7
You are in a callback there. Goodbye your error!
replies(1): >>speedg+qq1
◧◩
9. anigbr+2f1[view] [source] [discussion] 2021-10-28 05:31:27
>>matsem+bc
The power of something like Copilot is in building out stuff you're not familiar with or don't have templates set up. It's probably not as helpful when you already have a clear idea of what you want to do and just need it rather than think about it.

+1 for the variable extraction thing, I've been using their IDE for ages and it never occurred to me to look for such a thing.

replies(1): >>matsem+Dp1
◧◩◪
10. matsem+Dp1[view] [source] [discussion] 2021-10-28 07:25:03
>>anigbr+2f1
I use it especially much when writing old school java. Instead of writing "MyClass myclass = new MyClass()", I just write "new MyClass()" and get the typing for free. Even better when you do longer expressions and don't want to think about the type up front. Like working with streams or so.
replies(1): >>anigbr+Cy1
◧◩◪◨
11. speedg+qq1[view] [source] [discussion] 2021-10-28 07:34:25
>>ponyou+9t
Oh right. I once made a layer on top of the redis client to use promises because callbacks are a pain to deal with.
◧◩◪◨
12. anigbr+Cy1[view] [source] [discussion] 2021-10-28 09:02:57
>>matsem+Dp1
Yeah this would be quite helpful for me as I tend to just experiment with things in the console (cleaning up messy datasets and the like) and then copy or rewrite into something more structured later. I feel like I'm only using about 20% of what Pycharm can do.
[go to top]