sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: cmd/compile: redundant loads in strconv not eliminated
Labels: Performance,NeedsInvestigation,FeatureRequest,compiler/runtime

<!--
Please answer these questions before submitting your issue. Thanks!
-->

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version go1.20.3 linux/amd64
</pre>

### Does this issue reproduce with the latest release?

Yes.

### What operating system and processor architecture are you using (`go env`)?

<details><summary><code>go env</code> Output</summary><br><pre>
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/czhong4/.cache/go-build"
GOENV="/home/czhong4/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/czhong4/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/czhong4/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/home/czhong4/gocode/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/home/czhong4/gocode/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.20.3"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/czhong4/gocode/go/src/go.mod"
GOWORK=""
CGO_CFLAGS="-O2 -g"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -Wl,--no-gc-sections -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build822587075=/tmp/go-build -gno-record-gcc-switches"
</pre></details>

### What did you do?
I ran a redundant loads detecting tool on function strconv.FormatFloat().
<!--
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on go.dev/play is best.
-->



### What did you expect to see?
Redundant loads can be optimized by the compiler.


### What did you see instead?
1. In function rightShift() in package strconv, 
```
...
for ; n>>k == 0; r++ {
        if r >= a.nd {
...
```
`a.nd` is a loop invariant. But the compiler failed to put it in a register, resulting in `a.nd` being loaded from memory repeatedly in each iteration.  

2. In function Assign() in package strconv,
```
...
a.nd = 0
for n--; n >= 0; n-- {
	a.d[a.nd] = buf[n]
	a.nd++
}
...
```
`a.nd` is loaded from memory twice in a single iteration.
Repository: vscode
Issue: some keys of the Keyboard does not works in Visual Studio 
Labels: bug,keybindings

Hello Everyone I installed visual studion code in vmware workstation 10 
but some keyboard keys does not work like ( > & < ) threre fore I cant put html code manually 
May you help me please ? 

Regards
Repository: flutter
Issue: Selection cursor over WidgetSpan
Labels: framework,a: desktop,a: mouse,has reproducible steps,P2,found in release: 3.7,found in release: 3.9,team-text-input,triaged-text-input

## Steps to Reproduce

Cursor displaying as selection over `WidgetSpan` even if wrap `child` into `SelectionContainer.disabled`.

If wrap `WidgetSpan.child` into `MouseRegion`, small area with selection cursor still will be present. 

**Expected results:**

Cursor displaying as selection only over text.

**Actual results:** 

Cursor displaying as selection over WidgetSpan.

<details>
<summary>Code sample</summary>

```dart

import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const SelectionArea(
          child: MyHomePage(title: 'Flutter Demo Home Page')),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  int _counter = 0;

  void _incrementCounter() {
    setState(() {
      _counter++;
    });
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Column(
          mainAxisAlignment: MainAxisAlignment.center,
          children: <Widget>[
            const Text.rich(
              TextSpan(
                  text: 'You have pushed the button this many times:',
                  children: [
                    WidgetSpan(
                      // child: MouseRegion(
                      //   cursor: SystemMouseCursors.basic,
                        child: SelectionContainer.disabled(
                          child: SizedBox(
                            width: 50,
                            height: 20,
                          ),
                        ),
                      //),
                    )
                  ]),
            ),
            Text(
              '$_counter',
              style: Theme.of(context).textTheme.headlineMedium,
            ),
          ],
        ),
      ),
      floatingActionButton: FloatingActionButton(
        onPressed: _incrementCounter,
        tooltip: 'Increment',
        child: const Icon(Icons.add),
      ),
    );
  }
}


```

</details>

<details>
<summary>Video with SelectionContainer.disabled</summary>


https://user-images.githubusercontent.com/71345244/230584426-c2f5c345-c925-45ed-b6d7-e35a4883e622.mp4



</details>

</details>


<details>
<summary>Video with MouseRegion</summary>


https://user-images.githubusercontent.com/71345244/230584434-8baf9bcf-092e-439a-ae33-5eeaa70a143d.mp4



</details>
Repository: pytorch
Issue: Cannot use AT_CUDA_DRIVER_CHECK from user code
Labels: module: build,module: cpp-extensions,module: internals,module: cuda,triaged

### 🐛 Describe the bug

In my C++ extension that links against PyTorch I cannot use the AT_CUDA_DRIVER_CHECK macro to check the return code of a call to the CUDA driver API, because that macro makes use of the at::cuda::NVRTC struct which is defined in the ATen/cuda/nvrtc_stub/ATenNVRTC.h header, which doesn't get installed/shipped by PyTorch!

### Versions

I'm using PyTorch 2.0.0 installed from the official conda channels (build py3.9_cuda11.7_cudnn8.5.0_0).

cc @malfet @seemethere @zou3519 @ezyang @bhosmer @smessmer @ljk53 @bdhirsh @ngimel
Repository: storybook
Issue: [Bug]: Play function throws RangeError testing an emotion styled component
Labels: bug,compatibility with other tools,addon: interactions,sev:S2

### Describe the bug

Adding a `play` function, even an empty one, to a Story using a component created with `@emotion/styled` throws the following error:
```
RangeError: Maximum call stack size exceeded
```

### To Reproduce

`Component.tsx`
```tsx
import styled from '@emotion/styled'

export const Component = styled.span`
  background-color: red;
`
```

`Component.stories.tsx`
```tsx
import type { Meta, StoryObj } from '@storybook/react'
import { within } from '@storybook/testing-library'

import { Component } from './Component'

const meta: Meta<typeof Component> = {
  title: 'Component',
  component: Component,
}

type Story = StoryObj<typeof Component>

export const Simple: Story = {
  args: {},
  play: async ({ canvasElement, step }) => {
    const canvas = within(canvasElement)
    await step('Should...', async () => {})
  }
}

export default meta
```

### System

```shell
System:
    OS: macOS 13.2
    CPU: (10) arm64 Apple M1 Pro
  Binaries:
    Node: 16.16.0 - ~/.nvm/versions/node/v16.16.0/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v16.16.0/bin/yarn
    npm: 9.3.0 - ~/.nvm/versions/node/v16.16.0/bin/npm
  Browsers:
    Chrome: 112.0.5615.49
    Safari: 16.3
```


### Additional context

_No response_
Repository: stable-diffusion-webui
Issue: [Bug]: Generated Info not showing up
Labels: bug-report

### Is there an existing issue for this?

- [X] I have searched the existing issues and checked the recent builds/commits

### What happened?

The latest updates bugged out the generated info element on the webui while or after generating, it stays always blank.

![image](https://user-images.githubusercontent.com/25234371/230590108-50416710-4b71-4510-a4cb-09b336ece2c7.png)
(example screenshot, this information is not showing on my end after the update)


### Steps to reproduce the problem

n/a

### What should have happened?

Displaying the information as intended.

### Commit where the problem happens

latest

### What platforms do you use to access the UI ?

Windows

### What browsers do you use to access the UI ?

Mozilla Firefox

### Command Line Arguments

```Shell
`--xformers --opt-sub-quad-attention --medvram`
```


### List of extensions

- Remove Background Extension (makes the background transparent)
- Default extensions that came enabled

### Console logs

```Shell
No logs or console logs shows this.
```


### Additional information

_No response_
Repository: stable-diffusion-webui
Issue: [Feature Request]: API response with image filename/url or token
Labels: enhancement

### Is there an existing issue for this?

- [X] I have searched the existing issues and checked the recent builds/commits

### What would your feature do ?

When sending a request with parameters like requested by #3946 and completed by #8287 `"send_images": false, "save_images": true`, i would like the response to include the saved images folder- and filenames and/or their static URL.

Most likely a generated uuid (a token referencing the folder and file) would be the best way to request a specific image via api-route to respect auth restrictions.

Cuz the image files are stored by folders (today's date) and a prefix (#####) followed by the seed, the existing textinfo (including only the seed to identify the files) isn't enough, to accomplish the requested function nor enough to extract the needed information at this point in time.

### Proposed workflow

The frontend can send a payload to /sdapi/v1/txt2img or wherever, which can optionally contain an additional key. Something like `"get_image_token": true/false", "get_image_url": true/false"`.

If set to `true`, the API response includes the requestet information, to request the image afterwards via extra route / url.

#### 1. Request the Image via
`/sdapi/v1/txt2img`
> response includes image _folder- and filename_ or _token_

#### 2. Request image via client by _folder- and filename_ or _token_
```
/sdapi/v1/getImage/{folder}/{filename}
/sdapi/v1/getImage/{token}
```
> Response includes the image as base64 or a URL pointing to the image (only usefull without auth or on public folders)

### Additional information

This will be useful to generate images once, store them locally (on the server) and make them accessible via API by multiple clients without transmitting the image data itself between clients. 

Especially in scenarios where the generated images should be shared via a network protocol (multiuser applications), only a string containing (the url or the token) must be transmitted, rather the whole image data itself. Each client then will be responsible to download the image by itself.
Repository: PowerToys
Issue: Calculator in PowerToys Run regardless of localization
Labels: Needs-Triage,Needs-Team-Response

### Description of the new feature / enhancement

I propose to perform calculations in PowerToys Run regardless of localization, that is, as a fractional separator use as equal sign "." as well as the sign ",". Because very often it happens that users do not watch the sign, and calculations occur incorrectly. For example, if you type in "0,1/0,000001" the result will be "1" which is incorrect. If you type in "0.1/0.000001", the result will be correct "100000". This problems is very actual for some regions where as a fractional separator can be used both of «.» and «,». Please fix this situation (you can just add the button "take as separator "." and "," in the settings). I hope you will see this appeal and wish further development of the project!

### Scenario when this would be used?

This is relevant when using mathematical calculations in PowerToys Run.

### Supporting information

_No response_
Repository: next.js
Issue: Next (SWC) fails to build dependencies with with dynamic imports.
Labels: bug,SWC

### Verify canary release

- [X] I verified that the issue exists in the latest Next.js canary release

### Provide environment information

```bash
Operating System:
      Platform: darwin
      Arch: arm64
      Version: Darwin Kernel Version 22.3.0: Mon Jan 30 20:38:37 PST 2023; root:xnu-8792.81.3~2/RELEASE_ARM64_T6000
    Binaries:
      Node: 18.15.0
      npm: 9.5.0
      Yarn: 1.22.19
      pnpm: 7.27.0
    Relevant packages:
      next: 13.3.1-canary.0
      eslint-config-next: 13.3.0
      react: 18.2.0
      react-dom: 18.2.0
```


### Which area(s) of Next.js are affected? (leave empty if unsure)

SWC transpilation

### Link to the code that reproduces this issue

https://codesandbox.io/p/sandbox/ancient-bush-iffhbs?file=%2F.gitignore

### To Reproduce

Import a package with dynamic import, for example: 

```ts
import admzip from "adm-zip";
```

Which includes the following code:
```js
exports.require = function () {
    if (typeof process === "object" && process.versions && process.versions["electron"]) {
        try {
            const originalFs = require("original-fs");
            if (Object.keys(originalFs).length > 0) {
                return originalFs;
            }
        } catch (e) {}
    }
    return require("fs");
};
```

### Describe the Bug

Compilation fails, i.e.

```
Module not found: Can't resolve 'original-fs' in '/project/sandbox/node_modules/adm-zip/util'

Import trace for requested module:
./node_modules/adm-zip/util/fileSystem.js
./node_modules/adm-zip/util/fattr.js
./node_modules/adm-zip/util/index.js
./node_modules/adm-zip/adm-zip.js
./app/cant-compile/route.js
```

### Expected Behavior

Build succeeds.

The SWC's [ignoreDynamic](https://swc.rs/docs/configuration/modules#ignoredynamic) option might have helped here.

### Which browser are you using? (if relevant)

_No response_

### How are you deploying your application? (if relevant)

Vercel
Repository: PowerToys
Issue: Custom/customize/select Icon request
Labels: Idea-Enhancement,Area-User Interface,Needs-Triage

### Description of the new feature / enhancement

It would be awesome to be able to pick from a few tray icon look options. Maybe a black and white one, or gray-ish dark? or maybe a windows main color based? Or even a custom one? It would be awesome cause i wanna have it permanent on my task bar without ruining my aestethic

### Scenario when this would be used?

self explanatory

### Supporting information

_No response_
Repository: PowerToys
Issue: When the keyboard is not in use, the keyboard lock prevents false touches.
Labels: Idea-New PowerToy,Needs-Triage

### Description of the new feature / enhancement

When we are not using the keyboard, we can choose to turn it off.

### Scenario when this would be used?

You may need to clean your keyboard, write paper in the keyboard location (watch MOOCs), put your stuff on the keyboard, or even prevent naughty kids from sneaking around your computer to the point of disturbing your work. I think there are many other scenarios where I think this feature is important.

### Supporting information

Google and a lot of people need it.
https://www.google.com/search?q=windows+keyboard+locked&source=hp&ei=FQ0wZI3pJI_xkPIPrsyKiAY&iflsig=AOEireoAAAAAZDAbJZajAHkvhKAlwaE_sDJtDuVPMgiG&oq=windows+keyb&gs_lcp=Cgdnd3Mtd2l6EAEYCDIFCAAQgAQyBQgAEIAEMgUIABCABDIFCAAQgAQyBQgAEIAEMgUIABCABDIFCAAQgAQyBQgAEIAEMgUIABCABDIFCAAQgAQ6CwguEIAEEMcBENEDOg4ILhCABBDHARDRAxDUAjoHCAAQgAQQDDoFCC4QgARQAFidQWCzgAFoAHAAeACAAdgEiAG2KpIBBzMtNS40LjOYAQCgAQE&sclient=gws-wiz
Repository: godot
Issue: Error in set_multiplayer after changing node's path
Labels: bug,needs testing,topic:multiplayer

### Godot version

v4.0.2.stable.official [7a0977ce2]

### System information

Ubuntu 22.04

### Issue description

I have a Server and multiple clients in the same application.
Each one have a CustomMultiplayer so they can have different paths.

When I reparent the client, I update the path with set_multiplayer(multiplayer_api, new_path).
It works fine, but an error is thrown.

`
E 0:00:07:0220   Client.gd:13 @ update_mp(): Condition "root_path != p_config.operator NodePath()" is true. Returning: ERR_INVALID_PARAMETER
  <C++ Source>   modules/multiplayer/scene_multiplayer.cpp:597 @ object_configuration_remove()
  <Stack Trace>  Client.gd:13 @ update_mp()
                 main.gd:17 @ _process()
`

Since everything works as expected, is this error relevant? Is it a bug? Or is there any underlying problem I should care about?

### Steps to reproduce

Launch the project
Every 3 seconds an rpc is called
if you press space, the client goes fullscreen, and an error is thrown (not the first time, but always starting from second time)

### Minimal reproduction project

[set_multiplayer_bug.zip](https://github.com/godotengine/godot/files/11178996/set_multiplayer_bug.zip)
Repository: next.js
Issue: Dev server stops responding when using intercept route
Labels: Parallel & Intercepting Routes

### Verify canary release

- [X] I verified that the issue exists in the latest Next.js canary release

### Provide environment information

```bash
$ next info

    Operating System:
      Platform: darwin
      Arch: arm64
      Version: Darwin Kernel Version 21.6.0: Wed Aug 10 14:28:23 PDT 2022; root:xnu-8020.141.5~2/RELEASE_ARM64_T6000
    Binaries:
      Node: 18.12.1
      npm: 8.19.2
      Yarn: 1.22.19
      pnpm: 8.1.1
    Relevant packages:
      next: 13.3.1-canary.0
      eslint-config-next: N/A
      react: 18.2.0
      react-dom: 18.2.0
```


### Which area(s) of Next.js are affected? (leave empty if unsure)

App directory (appDir: true)

### Link to the code that reproduces this issue

https://github.com/sandervspl/intercept-route-test

### To Reproduce

There are two ways to reproduce this

- Try to run the dev server and go to http://localhost:3000/ or http://localhost:3000/foo/bar/baz. Nothing will happen (both client-side and the dev server)
- Try to build the project

### Describe the Bug

The dev server stops working in this project. Building the project also results in an error

```
TypeError: Missing parameter name at 12
    at lexer ...
```

### Expected Behavior

You should be able to see the page or at least an error page.

### Which browser are you using? (if relevant)

_No response_

### How are you deploying your application? (if relevant)

_No response_
Repository: TypeScript
Issue: Generic arguments gets casted as "unknown" when used as function argument
Labels: Needs More Info

**Typescript version :** 5.0.3 (But found on all versions I tested)

**Search terms :** `generic type as function argument unknown`

### ⏯ Playground Link

[Playground link with relevant code](https://www.typescriptlang.org/play?target=99#code/MYewdgzgLgBBwAsCmBbAhjAvDARgVwEsAbAEwGVFU0AKAbwCgY4oQAnJALhgG1GmZaMANZIAnlwDkAMwJJSARgkAaGADc0RPJxjUAlFgB8PCVCTQJAXRhoIMUJFgBfJXyaCR4mBIQhlajVpceoZePhLWtvbQzq4CwmKSptAAGn7qmtrBmEYAzDCOfBYuTDJyJBBcvPxxsUwekh6KKrUwJGhQaFyCpQqVJmZQlio+kmEqSVDJXHnOMAD0czAAKggEtgDubEIQKm0dMGsHYFJIrOwkdmzswFBEoi17aADqBFA+eFAAgqwA5kH62R03VkvWME0sEUuDmGIEk4UcuhUCxgTy2thsMHWciIKnYEDwRFgjxgPyQUFsBGOp3OD3az1eCG+fx0AAc0Kw0CgIACjGyOVz5osAHIgWCbVhCSk-GBSNgwPmcnYwKCiFlIGAgKTy9mKg4QMAmElkqBSo4nM5IEgtZEAeTAytWtggauABBkwDsNiQKgVAsOKrVFwxAEkqawyAh2ZayCx2AAROkAHjwYCEYBA6zABliBSYFnoCPoRakKZuBHAuEIpAoyHQib4SxrVBgSAAHqYwOUYAAxNgoJt1mNsJAAaTEKiH8bpBmKMEno7ELfbSE7TqgrCls-nCf2bY7XYASkhQKwSInoBuwD8VCm0xms1gYKHzRGo+RY0gd2h69VG5R0NwABE0DDoB+ZMNmBjUPAtadMsA5oPoDBMOwUB4Kw9owc2GJ-rBBZFgG6q9qw-b-t+fDzmOohLvua6XteFEfl+9BGNgyHMMOXTxJ4lFiAA3P4GT-CEt7ppm+TcBYfF8D05RcN8HKiIm7F1AkzD0dJ1SPFw250pp-CPC8bwgB8TLCYCukdPpTCGQyZmsjqXI6UxdI8nOLlWXwjgGNJjjSUW9CEU+YavuclnfvONErl27hqReUoCekgQ6G5on3hJFisQIfDcCORzucO3BgHgKA4KcVgYiOQEeGBFjyesaCvJaiZHmhGFLKqSCJlVgFJUgYEGD5BZAA)

### 💻 Code

Here are the two main generics involved in my case :

```ts
type FormSchema<
  StoreKey extends string,
  StoreData extends Record<string, unknown>
> = {
  store: { key: StoreKey; value: () => unknown }[];
  fields: Array<{
    key: string;
    data: StoreData;
    dataWithoutArg: () => StoreData;
    dataWithArg: (params: StoreData) => StoreData;
  }>;
};

type InferSharedStoreData<Store extends { key: string; value: () => unknown }[]> = {
  [K in Store[number] as K["key"]]: Awaited<ReturnType<K["value"]>>;
}
```

And here's the actual function that casts these : 

```ts
function buildSchema<
  TSchema extends FormSchema<StoreKey, StoreData>,
  StoreKey extends string,
  StoreData extends Record<string, unknown> = InferSharedStoreData<
    TSchema["store"]
  >
>(schema: TSchema) {
  return schema as TSchema
}
```

### 🙁 Actual behavior

The "params" argument of the "dataWithArg" sub-property of "fields" is being typed as InferSharedStoreData<unknown>, which means the "store" property type of the schema passed as argument of buildSchema is unavailable in the context of the function argument.

the StoreData type is being properly inferred for every other property it's assigned to, except on the case where it is used as a function argument.

### 🙂 Expected behavior

The function argument "params" should be inferred as StoreData, which itself is `InferSharedStoreData<schema['store']>`. A complete example is set-up on the playground link above.
Repository: puppeteer
Issue: [Feature]: Open page tab on specific profile with puppeteer.connect existing session.
Labels: feature

### Feature description

I know which is possible with launch, but I dont know about connect.
Example trying to chrome:
```js
import puppeteer from 'puppeteer';
(async () => {
  const browser = await puppeteer.connect(...);
//open page to currentProfile
  const pageDefault = await browser.newPage();
//open page to Profile 1
  const pageProfile_1 = ????;
//open page to Profile 2
  const pageProfile_2 = ????;

  await browser.close();
})();
```
Repository: deno
Issue: Different error code given vs Node for `fs.readFile` when path is a directory on Windows
Labels: bug,windows,node compat,node API

We're giving ENOENT and Node.js is giving EISDIR:

```
> deno
Deno 1.32.3
exit using ctrl+d, ctrl+c, or close()
REPL is running with all permissions allowed.
To specify permissions, run `deno repl` with allow flags.
> import fs from "node:fs"
undefined
> fs.readFileSync("cli/tests/")
Uncaught NotFound: The system cannot find the path specified. (os error 3): readfile 'cli/tests/'
    at Object.readFileSync (ext:deno_fs/30_fs.js:679:14)
    at Object.readFileSync (ext:deno_node/_fs/_fs_readFile.ts:35:23)
    at <anonymous>:2:4
> _error.code
"ENOENT"
> close()
V:\deno [main ↑478 +0 ~1 -0 !]
> node
Welcome to Node.js v18.12.1.
Type ".help" for more information.
> fs.readFileSync("cli/tests/")
Uncaught Error: EISDIR: illegal operation on a directory, read
    at Object.readSync (node:fs:748:3)
    at tryReadSync (node:fs:448:20)
    at Object.readFileSync (node:fs:494:19) {
  errno: -4068,
  syscall: 'read',
  code: 'EISDIR'
}
```

I believe this unit test here is incorrect and should be `EISDIR` for both windows and linux, but I'm unsure:

https://github.com/denoland/deno/blob/5d9172467eee8cdceefa944199459ddd410f7388/cli/tests/unit/read_file_test.ts#L173-L186

I discovered this when running the unit tests in Node for [node_shims](https://github.com/denoland/node_shims).

Related: #17958 (cc @kamilogorek), #18309