sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: net/http: TestClientTimeout* failures with "missing timeout substring" on Windows
Labels: OS-Windows,NeedsInvestigation

```
#!watchflakes
post <- goos == "windows" && pkg == "net/http" && test ~ `^TestClientTimeout.*` && `missing timeout substring`
```

[2020-12-09T01:34:53-ae9b442/windows-arm-zx2c4](https://build.golang.org/log/7a1edb0354b793d5c2604ab2c4d1582096f9585c)
[2020-12-07T17:19:41-7f9a2bc/windows-arm-zx2c4](https://build.golang.org/log/b5cc317d726914082ad7cc11ab715fd4b4f8773e)
[2020-12-03T02:35:36-da54dfb/windows-arm-zx2c4](https://build.golang.org/log/04ef88039ac999db82b14622604e6b0f1161fddc)
[2020-12-02T20:17:57-2d0258d/windows-arm-zx2c4](https://build.golang.org/log/58d45ddbad19fa20b6abbb2253e59610b4b8a5ec)

```
--- FAIL: TestClientTimeout_h1 (0.20s)
    client_test.go:1277: error string = "context deadline exceeded"; missing timeout substring
--- FAIL: TestClientTimeout_h2 (0.20s)
    client_test.go:1277: error string = "context deadline exceeded"; missing timeout substring
FAIL
FAIL	net/http	43.328s
```

CC @zx2c4 @bradfitz @neild @empijei
Repository: rust
Issue: diagnostics: suggest Result instead of Option if inner arguments would match
Labels: A-diagnostics,P-low,T-compiler,A-result-option

````rust
pub struct Error {}

pub fn want_result() -> Option<(), Error> {
     unimplemented!()
}
````

rust fails to compile this:
````
error[E0107]: wrong number of type arguments: expected 1, found 2
 --> src/lib.rs:3:36
  |
3 | pub fn want_result() -> Option<(), Error> {
  |                                    ^^^^^ unexpected type argument

error: aborting due to previous error
````

Since `<x,y>` matches the `Result` signature, it would be super neat if rustc could suggest something like `Did you mean to use  Result<(), Error> instead of Option<(), Error> here?`
Repository: rust
Issue: rustc span format seems broken: file:line:line:column
Labels: A-diagnostics,T-compiler,C-bug

repro:
````
git clone https://github.com/matthiaskrgr/cargo-cache
cd cargo-cache
git checkout rustfmt_bug
cargo fmt
````

````
error[internal]: left behind trailing whitespace
   --> /home/matthias/vcs/github/cargo-cache/src/library.rs:147:147:0
    |
147 |
    | ^^^^
    |

warning: rustfmt has failed to format. See previous 1 errors.
````

This `"147:147:0` seems super suspicious, why is printing the column twice in the span location??
rustfmt gets its span information from rustc afaik thus I opened this ticket against rustc.

`rustfmt 1.4.29-nightly (70ce182 2020-12-04)`
````
rustc 1.50.0-nightly (f0f68778f 2020-12-09)
binary: rustc
commit-hash: f0f68778f798d6d34649745b41770829b17ba5b8
commit-date: 2020-12-09
host: x86_64-unknown-linux-gnu
release: 1.50.0-nightly
````
Repository: flutter
Issue: Bug in Material text selection menu button layout when using narrow buttons
Labels: a: text input,framework,f: material design,P2,team-design,triaged-design

In the case of text selection menu buttons that are narrower than the overflow button, it's possible that there is underutilized space in the text selection menu.

Currently, the algorithm lays out as many buttons as can fit until they overflow, then checks if removing the overflow button could allow everything to fit:

```
|------- width -------|
[o] [button 1] [button 2]
```
```
|------- width -------|
[button 1] [button 2]
```

Removing the overflow button [o] allowed both buttons to fit without needing an overflow menu at all.

However, if the overflow button is wider than an overflowing button, this algorithm will not work perfectly:

```
|------- width -------|
[overflow] [b1] [b2] [b3] [b4]
```

The algorithm will not remove the overflow button here because it assumes that the overflow button is narrower than b4, which is not correct.

This is unlikely to happen in reality because the overflow button is hardcoded to a small IconButton, but with the customization allowed in  https://github.com/flutter/flutter/pull/69428, it's possible that users could run into this bug with a custom menu.

https://github.com/flutter/flutter/blob/36bed2b4c3c2cd9e812e6f8b2beca983ad07eea2/packages/flutter/lib/src/material/text_selection.dart#L483-L487
Repository: PowerToys
Issue: [Feature Request] Character map picker
Labels: Idea-New PowerToy

I often find myself going into the Windows Character map to find a character to use for the software that I design and write. I have used other tools such as Universal Character Map which I do find is much clearer to use in comparing to the Windows Character map where the characters displayed are very small. 

It definitely  would be nice to see a Character Map picker similar to the functionalities found in the Emoji picker and the Color picker, where if I am inside Visual Studio writing XAML I'd like to be able to quickly launch the character map picker, find a character to use and insert the selected character into the Visual Studio editor or any focused text field. 

Some ideas:

- Shortcut keys to launch the picker, not sure what we could use though?

- The ability to select a font family with Segoe MDL2 Assets possibly being the default font family

- Clicking a character will automatically insert the character into a focused text field

- We should possibly somehow show options to be able to insert the clicked character as either a character, hex, XAML, or json with a way of making one as the default option preferred by the developer
Repository: TypeScript
Issue: Strange error: string is not a string type 
Labels: Bug,Help Wanted,Effort: Moderate,Domain: Error Messages

**TypeScript Version:**  4.1.2

**Search Terms:** downlevelIteration, string, spread

**Code**

```ts
function test(name: string) {
    for (const _char of [...name]) {
    }
}
```

Run tsc on just this ts file with no extra flags (I couldn't find the right combination of playground checkboxes to simulate this.

**Expected behavior:** Assuming the code is wrong, the error message should probably not say that string is not a string type.

**Actual behavior:**
```
error TS2569: Type 'string' is not an array type or a string type. Use compiler option '--downlevelIteration' to allow iterating of 
iterators.
```
Repository: pytorch
Issue: Investigate `torch.linalg.norm` performance
Labels: module: performance,triaged,module: linear algebra

In issue #47833, @mthrok posted some measurements of `torch.linalg.norm`. It looks like it should be possible to improve performance. I will look into what may be causing bottlenecks.

cc @jianyuh @nikitaved @pearu @mruberry @heitorschueroff @walterddr @VitalyFedyunin @ngimel
Repository: pytorch
Issue: [NNC] Some ops have type promotion logic which adds extra casts & does compute in different dtype than eager
Labels: module: bootcamp,triaged,NNC

## 🐛 Bug

Many NNC type promotion goes through a common path which [promotes inputs according to the highest input type](https://github.com/pytorch/pytorch/blob/master/torch/csrc/jit/tensorexpr/kernel.cpp#L725), and then after compute, casts the output to whichever output type was recorded when the op was run in eager. 

Because we have the output type specified, all NNC ops will give the correct dtype. However, the compute may be done in a different dtype than eager, and there may be extraneous casts added. 

In the (worst) case of something like, 
```
x = torch.ones([4,  4], dtype=torch.int16)
torch.addcmul(x, x, x, value=1)
```

We will cast all three tensor inputs to int32, do the addcmul, then cast the operation back. As opposed to eager, which will just cast `value=1` to int16. 

The advantage of the current approach is that we are guaranteed to have the same output dtype. I think ops could gradually be migrated over to have the correct casting behavior.

Most ops do promote to the highest dtype, so this isn't **that** big of an issue, but it does come up.

TODO: 

- [ ] torch.addcmul (add your name here to claim)




<!-- A clear and concise description of what the bug is. -->

## To Reproduce

1. Comment out the [output type casting](https://github.com/pytorch/pytorch/blob/master/torch/csrc/jit/tensorexpr/kernel.cpp#L730)  of the various `comput{number}Operands` 

2. Run `python test/test_jit_fuser_te..py`
Repository: scrcpy
Issue: Snap: adb server version (39) doesn't match this client (41)
Labels: snap

 - [x] I have read the [FAQ](https://github.com/Genymobile/scrcpy/blob/master/FAQ.md).
 - [x] I have searched in existing [issues](https://github.com/Genymobile/scrcpy/issues).

**Environment**
 - OS: Ubuntu 18.04
 - scrcpy version: 1.16
 - installation method: snap
 - device model: Pixel 4a
 - Android version: 11

**Describe the bug**
When I run `scrcpy`:
```
adb server version (39) doesn't match this client (41); killing...
* daemon started successfully
connected to 192.168.50.18:5555
eater $INFO: scrcpy 1.16 <https://github.com/Genymobile/scrcpy>                                                                                                      
adb server version (41) doesn't match this client (39); killing...
* daemon started successfully
adb: error: failed to get feature set: no devices/emulators found
ERROR: "adb push" returned with value 1
```
I think because:
```
$ /snap/scrcpy/current/usr/bin/adb --version                                                                   
Android Debug Bridge version 1.0.39
Version 1:8.1.0+r23-5~18.04
Installed as /snap/scrcpy/256/usr/lib/android-sdk/platform-tools/adb
```
Repository: pytorch
Issue: torch.utils.data.DistributedSampler allow uneven inputs
Labels: module: dataloader,triaged,enhancement,module: data

## 🚀 Feature

Right now, `torch.utils.data.DistributedSampler` has two modes of dealing with datasets where `len(dataset) % num_replicas != 0`: if `drop_last=False`, then we "pad" the dataset with extra samples, or if `drop_last=True`, we drop the last batch of samples.

I think we should also have an option to just *have* uneven samples across the different replicas -- now that https://github.com/pytorch/pytorch/issues/38174 is merged in, the user should be able to deal with this case themselves via the `join` APi.



cc @SsnL @VitalyFedyunin @ejguan @pietern @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @osalpekar @jiayisuse @agolynski @SciPioneer @H-Huang @mrzzd
Repository: vscode
Issue: Ports panel tab keeps moving around
Labels: bug,workbench-views

It seems like the tab for ports keeps moving around in the panel. I keep dragging it to the far left, but it doesn't stay. Then just now in the same session, I saw it move from the far left to here

![image](https://user-images.githubusercontent.com/323878/101822072-aa0b9000-3add-11eb-8910-f67d87b204f9.png)
Repository: go
Issue: encoding/json: UnmarshalTypeError.Error message seems insensible
Labels: NeedsInvestigation

[Consider this snippet](https://play.golang.org/p/lKh1SHb-9VT):
```go
type GoStruct1 struct {
	AlphaField GoStruct2 `json:"alpha_field"`
}
type GoStruct2 struct {
	BravoField []GoStruct3 `json:"bravo_field"`
}
type GoStruct3 struct {
	StringField string `json:"string_field"`
}

func main() {
	fmt.Println(json.Unmarshal([]byte(`{"alpha_field":{"bravo_field":[{"string_field": 3.14159}]}}`), new(GoStruct1)))
}
```

On Go1.15, this prints:
```
json: cannot unmarshal number into Go struct field GoStruct3.alpha_field.bravo_field.string_field of type string
```
Notice that it combines two different namespaces where it is:
```
{{GoTypeName}}.{{JSONPath}}
```
and that the message says that it is a "Go struct field" when it isn't.
Also, it uses `GoStruct3`, when it makes little sense.
At minimum, it should at least used `GoStruct1`.

\cc @mvdan
Repository: TypeScript
Issue: String literal rename doesn't work with template literals
Labels: Bug,Domain: Refactorings

**typescript next 4.2.0-dev20201210**

There are other cases when string literal rename refactor (https://github.com/microsoft/TypeScript/pull/39298) deosn't work

**Example:**

```ts
type Foo = 'donald' | 'joe' // allow to rename any of these values

const foo: Foo = 'joe'

// using string literal:

const temp: `hello ${Foo}` = `hello donald`
```

After literals rename:

**Actual**
```ts
type Foo = 'donald duck' | 'pizza joe' // allow me to rename any of these values

// just declaration

const foo: Foo = 'pizza joe ' // this works

// using string literal

const temp: `hello ${Foo}` = `hello donald` // this doesn't work
```

**Expected:**
```ts
// using string literal

const temp: `hello ${Foo}` = `hello donald duck` // should rename here too
```

Related:
https://github.com/microsoft/TypeScript/issues/41489
https://github.com/microsoft/TypeScript/issues/41923
Repository: TypeScript
Issue: String literal rename doesn't work in switch/case statment
Labels: Bug,Domain: Refactorings

**typescript next 4.2.0-dev20201210**

There are other cases when string literal rename refactor (https://github.com/microsoft/TypeScript/pull/39298) deosn't work

**Example:**

```ts
type Foo = 'donald' | 'joe' // allow to rename any of these values

// basic usage

const foo: Foo = 'joe '

// using string literal:

switch (foo) {
   case 'donald': ...
   case 'joe': ...
}
```

After literals rename:

**Actual**
```ts
type Foo = 'donald duck' | 'pizza joe' // allow me to rename any of these values

// basic usage

const foo: Foo = 'pizza joe ' // this works

// rename doesn't work

switch (foo) {
   case 'donald': ...
   case 'joe': ...
}
```

**Expected:**
```ts
// using string literal

// should rename here too

switch (foo) {
   case 'donald duck': ...
   case 'pizza joe': ...
}
```

Related:
https://github.com/microsoft/TypeScript/issues/41489
https://github.com/microsoft/TypeScript/issues/41922
Repository: flutter
Issue: Consider migrating to a variant of vector_math that has a smaller API surface area
Labels: framework,dependency: dart,P3,team-framework,triaged-framework

For example, we don't seem to need https://master-api.flutter.dev/flutter/vector_math/Colors-class.html but it is currently in our API docs. We may also be able to make a simpler API surface, e.g. `Matrix4.leftTranslate` has an argument that is `dynamic`.

The question is can we do this without breaking everyone... I expect many people are importing vector_math themselves, so this may be a non-starter.

Not a high priority. cc @dnfield @yjbanov @goderbauer
Repository: flutter
Issue: dart:html appears in our API docs in a way that confuses matters
Labels: framework,dependency: dart,d: api docs,P3,team-framework,triaged-framework

dart:html appears in our API docs in a way that confuses matters. For example, searching for the "Text" widget brings you here:
https://master-api.flutter.dev/flutter/dart-html/Text-class.html

I'm not sure what the solution is; arguably, dart:html is an important library for us to include in our API docs. Maybe it should just be ordered differently? Or in a different color or something?

cc @yjbanov @jcollins-g @gspencergoog
See also https://github.com/dart-lang/dartdoc/issues/2457
Repository: flutter
Issue: Error running 'gclient sync' in flutter engine folder: failed to resolve flutter_internal/android/sdk/licenses@latest (line 28): prefix "flutter_internal/android/sdk/licenses" doesn't exist or "user:andrewjohncoutts@gmail.com" is not allowed to see it, run `cipd auth-login` to login or relogin
Labels: engine,P2,team-engine,triaged-engine

After updating to the latest stable `1.22.5` and attempting to `gclient sync` with engine branch `flutter-1.22-candidate.12`, I'm getting the below error:

```
➜  engine gclient sync -D
Syncing projects: 100% (105/105), done.                                                                                        

________ running 'cipd ensure -log-level error -root /Users/andrewcoutts/Projects/engine -ensure-file /var/folders/dv/xkfz56ls6fldcm8nc612gxg80000gn/T/tmps18wuun_.ensure' in '.'
Errors:
  failed to resolve flutter_internal/android/sdk/licenses@latest (line 28): prefix "flutter_internal/android/sdk/licenses" doesn't exist or "user:andrewjohncoutts@gmail.com" is not allowed to see it, run `cipd auth-login` to login or relogin
Error: Command 'cipd ensure -log-level error -root /Users/andrewcoutts/Projects/engine -ensure-file /var/folders/dv/xkfz56ls6fldcm8nc612gxg80000gn/T/tmps18wuun_.ensure' returned non-zero exit status 1
```

Attempting to authenticate using a token from `cipd auth-login` does not fix it either.

Here is my `.gclient` file (unchanged from before):
```gclient
solutions = [
  {
    "managed": False,
    "name": "src/flutter",
    "url": "git@github.com:bottlepay/engine.git",
    "custom_deps": {},
    "deps_file": "DEPS",
    "safesync_url": "",
  },
]
```
Repository: create-react-app
Issue: Proxy server from package.json not working
Labels: needs triage,issue: bug report

### Describe the bug

Same issue as in: https://github.com/facebook/create-react-app/issues/6031

Proxy is not working when I add `"proxy": "http://localhost:8080",` into `package.json`.

Only when you follow manual proxy config at https://create-react-app.dev/docs/proxying-api-requests-in-development/#configuring-the-proxy-manually everything works fine.

React-Scripts version: 4.0.1
Env: Win 10 + Node 14
React env: development
Repository: go
Issue: mime: RFC 2045 vs 2616 confusion in FormatMediaType
Labels: NeedsInvestigation

So ... this is like the definition of pedantic, but:

The list of characters that can't be in tokens in RFC 2045 [MIME] is (https://tools.ietf.org/html/rfc2045 section 5.1):

`()<>@,;:\"/[]?=`

as implemented in https://github.com/golang/go/blob/master/src/mime/grammar.go .

The list of characters that can't be in tokens in RFC 2616 [HTTP/1.1] is (https://tools.ietf.org/html/rfc2616 section 2.2):

`()<>@,;:\"/[]?={}`

Note the extra 2 characters. This impacts `mime.FormatMediaType` which is advertised as usable for either RFC in that it might try to stick a curly brace into a string which is otherwise considered a token.

My particular use-case for this is

    mime.FormatMediaType("attachment", map[string]string{"filename": filename}))

which is used as the value of the `Content-Disposition` header. So for example

`attachment; filename=foo`

is valid, while

`attachment; filename=foo{`

is not -- that one would require quoting, at least per the spec. See sample code in https://play.golang.org/p/TbkEuMijPPb .

I have no idea if any web browsers care about this. I happened upon this when trying to figure out how filename characters need to be escaped properly before finding this function in the go standard library and then comparing it to the spec.
Repository: rust
Issue: Long argsfile doesn't work properly on Windows
Labels: O-windows,C-bug

Example: https://github.com/KapJI/rust_example
Commands to repro: https://github.com/KapJI/rust_example/blob/main/test.sh
Example CI job: https://github.com/KapJI/rust_example/runs/1534666156

With shorter argsfile it compiled correctly but longer one failed with
```
error: cannot determine resolution for the import
 --> b/lib.rs:1:5
  |
1 | use a::A;
  |     ^^^^
  |
  = note: import resolution is stuck, try simplifying other imports
```
which is very obscure and not really relevant.

Note that I added many duplicated lines in argsfiles to demonstrate that it starts to fail after some threshold, actual content is not important. In the real project we have different lines for different dependencies.

If you comment
```rust
pub trait SerdeWith<SerdeType> {
    fn deserialize<'a, D: ::serde::Deserializer<'a>>(
        deserializer: D,
    ) -> Result<Self, D::Error>
    where
        Self: Sized;
}
```
in `a/lib.rs` it will work in both cases. But this trait is actually not used in this example.

At some point I also got internal compiler error, not sure if it's relevant:
```
thread 'rustc' panicked at 'index out of bounds: the len is 236 but the index is 236', compiler\rustc_codegen_ssa\src\back\symbol_export.rs:292:13
```
Repository: vscode
Issue: Can't not open files or dictionary by right click -> open in code
Labels: bug,help wanted,windows,workbench-os-integration

<!-- ⚠️⚠️ Do Not Delete This! bug_report_template ⚠️⚠️ -->
<!-- Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct/ -->
<!-- Please search existing issues to avoid creating duplicates. -->
<!-- Also please test using the latest insiders build to make sure your issue has not already been fixed: https://code.visualstudio.com/insiders/ -->

<!-- Use Help > Report Issue to prefill these. -->
- VSCode Version: 1.51.1(system setup
- OS Version: Windows_NT x64 10.0.19042

Steps: watch in video
https://drive.google.com/file/d/1oMNbewjwI_k6fWhybatZ7rTJEk9Nji4U/view?usp=sharing


<!-- Launch with `code --disable-extensions` to check. -->
Does this issue occur when all extensions are disabled?: Yes/No
Repository: ant-design
Issue: 表头分组和固定列搜索定位问题
Labels: help wanted,Inactive

Reproduction link

https://codepen.io/miyatis/pen/BaLQRXj?editors=0010

Steps to reproduce

ctrl f搜索test5

What is expected?

定位到搜索的表头

What is actually happening?

无反应

Environment	Info
antd	4.9.1
React	16
System	win
Browser	谷歌
Repository: godot
Issue: Normal Maps with large height gradients are not getting applied correctly
Labels: bug,topic:rendering

**Godot version:**
Godot 3.2.3 and 3.2.4-beta3

**OS/device including version:**
Windows 10, Nvidia GTX 960 GLES2 GLES3

**Issue description:**
The normal file in question is labeled Asteroid_DeepNormal.png as that has some height gradients. I also have an Asteroid_SoftNormal.png file that I made with softer gradients. The issue is I want some large height gradients for my normal maps to give an effect of deeper indents in my asteroids. I could see this have a use for other game objects as well.

The issue exists for GLES3 and GLES2 for both Sprite, Sprite3D, and MeshInstance. Parts of the sprite get shaded incorrectly when using normal maps that have large ranges in values. The issue does diminish when using the 3D methods and setting the normal scale to 0.5 or less. I have confirmed my normal map files are valid to work by testing on the Unity game engine.

Note about Godot Master branch: this issue is only for GLES2 and GLES3 when using a MeshInstance/Sprite3D as Sprite2D does not have a normal map option when making the issue using Godot Master branch with Volkan. I also can't seem to test the GLES2 on the master branch at this moment.

**Steps to reproduce:**
3DScene and 2DScene files are provided and should be runnable. It does seem that if the normal scale is set to 0.5 most of the issue is no longer visible.

**Minimal reproduction project:**
[TestNormals.zip](https://github.com/godotengine/godot/files/5676896/TestNormals.zip)
Repository: pytorch
Issue: Bug in profiler on CI machines when profiling NCCL distributed calls
Labels: oncall: distributed,triaged,module: c10d

## 🐛 Bug

As mentioned in https://github.com/pytorch/pytorch/pull/48947, the profiler can sometimes give inaccurate results on distributed collective ops quite rarely. This seems to happen only very rarely on CI machines and nowhere else, but it is still worth looking into. 

## To Reproduce

Steps to reproduce the behavior:

1. ssh into multigpu machine
1. disable all tests: sed -i 's/def test_/def xtest_/g' /opt/conda/lib/python3.6/site-packages/torch/testing/_internal/distributed/distributed_test.py, then manually pick a test such as `test_reduce_sum_cuda`
1.Run the test for a while until it fails: `while [ $? -eq 0 ]; do python test/run_test.py --verbose -i distributed/test_distributed_fork; done` (can take several tries to fail)


cc @pietern @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @osalpekar @jiayisuse @agolynski @SciPioneer @H-Huang @mrzzd