sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: x/net/http2: error message is unclear when connecting to an http 1.x server
Labels: NeedsInvestigation

<!-- Please answer these questions before submitting your issue. Thanks! -->

### What version of Go are you using (`go version`)?

1.13

### Does this issue reproduce with the latest release?

yes

### What operating system and processor architecture are you using (`go env`)?
GOARCH="amd64"

### What did you do?

https://play.studygolang.com/p/_1kMO-E_FEq
I use an http2 client try to connect an http client.

I expect to see an protocol error or somethig that can tell me the destination is not a valiad http2 service.

### What did you see instead?
I get an "unexpected EOF" error.
I think we can make the error more clearly.


According to  the rfc, the last segment of [https://httpwg.org/specs/rfc7540.html#rfc.section.3.5]

> Clients and servers MUST treat an invalid connection preface as a connection error (Section 5.4.1) of type PROTOCOL_ERROR. 

It seems we should return PROTOCOL_ERROR
Repository: ant-design
Issue: Layout.Sider has no ability to set breakpoint from custom string
Labels: Inactive

- [x] I have searched the [issues](https://github.com/ant-design/ant-design/issues) of this repository and believe that this is not a duplicate.

### What problem does this feature solve?
If I want to make custom onBreakpoint event with size, which not included in default breakpoints {
  xs: '480px',
  sm: '576px',
  md: '768px',
  lg: '992px',
  xl: '1200px',
  xxl: '1600px',
}

### What does the proposed API look like?
    breakpoint?: 'xs' | 'sm' | 'md' | 'lg' | 'xl' | 'xxl'; add custom string or, may be, number here


<!-- generated by ant-design-issue-helper. DO NOT REMOVE -->
Repository: pytorch
Issue: Wrong substitution of aten::to 
Labels: oncall: jit,triaged,oncall: mobile

## 🐛 Bug

The bug takes place when TorchScript scripting is used from eager Python code. ATen::to in resulting TorchScript code has one extra parameter `None`. Then `C++` cannot deal with it
## To Reproduce

This comes from an article on discuss:

https://discuss.pytorch.org/t/jit-mobile-wrong-substitution-of-aten-to/60846

<!-- If you have a code sample, error messages, stack traces, please provide it here as well -->

## Expected behavior

Substitution doesn't contain extra `None`

## Environment

 - PyTorch Version (e.g., 1.3.1 or Nightly):
 - OS (scripted on Linux then tested on Android):
 - How you installed PyTorch (`conda`):
 - Python version 3.7:



cc @suo
Repository: opencv
Issue: Documentation for cmake-gui build flags
Labels: category: documentation

Not a bug per se, but may as well be, given the amount of confusion resulting:
No one person seems to know, much less understand, the bewildering proliferation of options afforded by cmake/cmake-gui.  Especially advanced options.

One of the single greatest benefits to users of OpenCV could be the ability to build OpenCV according to a coherent plan, and not having to comb through countless google searches for keywords that should really be documented somewhere.  The build process turns into a lengthy trial-and-error, apparently even for some of the experienced veterans of the OpenCV forum.

This seems like it would be a very worthwhile effort.  I have noted many good programmers who build OpenCV with templates found on various websites, only to find that essential features are missing later.  This certainly seems within reach, and worthwhile.
Repository: neovim
Issue: Neovim allows setting invalid `language` without encoding
Labels: bug,platform:macos,localization,encoding,complexity:low,has:plan

I noticed an annoying behavior pasting german text from nvim to other apps. All umlauts appear to become two characters.  At first I expected my terminal or terminal emulator to be at fault, but by chance I opened regular vim and noticed it behaves as expected.

- `nvim --version`:
```
NVIM v0.4.3
Build type: Release
LuaJIT 2.0.5
```
- `vim -u DEFAULTS` (version: 8.1) behaves differently? 
  yes
- Operating system/version:
MacOS 10.15.1

- Terminal name/version: Terminal.app, iTerm
- `$TERM`: xterm-256color
- encoding and fileencoding are UTF-8

### Steps to reproduce using `nvim -u NORC`

```
nvim -u NORC
```
insert text: Müller (on english mac keyboards ü is typed `<M-u> <u>`, I suppose you could also open a text file containing the string)
```
iMüller    
```
Copy to System clipboard
```
<Esc>
"+yy
```

### Actual behaviour

Pasting into e.g. notepad results: `M√ºller`

### Expected behaviour

Pasting into e.g. notepad results: `Müller`.
Vim 8.1 has this behavior.
Repository: three.js
Issue: CompressedTextures: Add support for serialization/deserialization.
Labels: Enhancement

##### Description of the problem

When calling `toJson()` on an object that contains a CompressedTexture, an error gets thrown:
```
Uncaught TypeError: Failed to execute 'drawImage' on 'CanvasRenderingContext2D': The provided value is not of type '(CSSImageValue or HTMLImageElement or SVGImageElement or HTMLVideoElement or HTMLCanvasElement or ImageBitmap or OffscreenCanvas)'
    at Object.getDataURL (js.js?v=000:4184)
    at CompressedTexture.toJSON (js.js?v=000:4398)
    at MeshStandardMaterial.toJSON (js.js?v=000:10171)
    at serialize (js.js?v=000:7295)
    at Mesh.toJSON (js.js?v=000:7349)
    at Object3D.toJSON (js.js?v=000:7363)
    at Scene.toJSON (js.js?v=000:7363)
    at Scene.toJSON (js.js?v=000:7507)
```
My specific use case is that I load textures in the basis universal format, and I call `toJSON()` to export my entire scene to a file in order to debug it in the threejs editor.

I don't think it's easy to convert a compressed texture to a data url. But I'll try to make a pull request that at leasts gives a warning and exports it without the texture. Instead of throwing an error.

##### Three.js version

r110

##### Browser

Chrome

##### OS

MacOS
Repository: flutter
Issue: Add TextOverflow  "ellipsisStart" , "ellipsisMiddle"  and "ellipsisEnd" 
Labels: c: new feature,framework,a: typography,customer: crowd,customer: google,c: proposal,P1,team-framework,triaged-framework,:hourglass_flowing_sand:

<!-- Thank you for using Flutter!

     If you are looking for support, please check out our documentation
     or consider asking a question on Stack Overflow:
      * https://flutter.dev/
      * https://api.flutter.dev/
      * https://stackoverflow.com/questions/tagged/flutter?sort=frequent

     If you have found a bug or if our documentation doesn't have an answer
     to what you're looking for, then fill our the template below. Please read
     our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports
-->

## Use case

When displaying file-paths, I'd prefer to show the file-name and clip the file-path.

In this example, it's not possible to see the filenames:

```dart
Text(
  '/root/sdcard/storage/camera/2019-11-20 12:12:11 a-file-with-a-long-path.png',
  overflow: TextOverflow.ellipsis,
)
```

<img width="410" alt="Screenshot 2019-11-21 at 10 47 04" src="https://user-images.githubusercontent.com/46879/69331201-43a29e00-0c4c-11ea-833e-9fa115b88926.png">

## Proposal

Allow TextOverflow to clip text on the leading edge rather than the trailing edge.


<!--
     Briefly but precisely describe what you would like Flutter to be able to do.

     Consider attaching images showing what you are imagining.

     Does this have to be provided by Flutter directly, or can it be provided
     by a package on pub.dev/flutter? If so, maybe consider implementing and
     publishing such a package rather than filing a bug.
-->

Internal bug - b/180975303
Repository: node
Issue: fs.copyFile does not actually do a full copy
Labels: fs,libuv,macos

<!--
Thank you for reporting a possible bug in Node.js.

Please fill in as much of the template below as you can.

Version: output of `node -v`
Platform: output of `uname -a` (UNIX), or version and 32 or 64-bit (Windows)
Subsystem: if known, please specify the affected core module name

If possible, please provide code that demonstrates the problem, keeping it as
simple and free of external dependencies as you can.
-->

* **Version**: 13.1.0
* **Platform**: MacOS
* **Subsystem**: fs

<!-- Please provide more details below this comment. -->

copyFile on MacOS does not correctly copy files. A simple example

Here's a .jpg downloaded with Chrome

```
$ ls -l -@     
total 6792
-rw-r--r--@ 1 gregg  staff  3476085 Nov 21 20:28 original.jpg
	com.apple.macl	     72 
	com.apple.metadata:_kMDItemUserTags	     42 
	com.apple.metadata:kMDItemWhereFroms	    190 
	com.apple.quarantine	     57 
```

copy with it `cp`

```
$ cp original.jpg copied-with-cp.jpg
$ls -l -@                          
total 13584
-rw-r--r--@ 1 gregg  staff  3476085 Nov 21 20:48 copied-with-cp.jpg
	com.apple.macl	     72 
	com.apple.metadata:_kMDItemUserTags	     42 
	com.apple.metadata:kMDItemWhereFroms	    190 
	com.apple.quarantine	     57 
-rw-r--r--@ 1 gregg  staff  3476085 Nov 21 20:28 original.jpg
	com.apple.macl	     72 
	com.apple.metadata:_kMDItemUserTags	     42 
	com.apple.metadata:kMDItemWhereFroms	    190 
	com.apple.quarantine	     57 
```

now copy it with fs.copyFileSync

```
$ node
Welcome to Node.js v13.1.0.
Type ".help" for more information.
> const fs = require('fs')
undefined
> fs.copyFileSync('original.jpg', 'copied-with-fs-copyFile.jpg');
undefined
> 
$ ls -l -@
total 20376
-rw-r--r--@ 1 gregg  staff  3476085 Nov 21 20:48 copied-with-cp.jpg
	com.apple.macl	     72 
	com.apple.metadata:_kMDItemUserTags	     42 
	com.apple.metadata:kMDItemWhereFroms	    190 
	com.apple.quarantine	     57 
-rw-r--r--  1 gregg  staff  3476085 Nov 21 20:49 copied-with-fs-copyFile.jpg
-rw-r--r--@ 1 gregg  staff  3476085 Nov 21 20:28 original.jpg
	com.apple.macl	     72 
	com.apple.metadata:_kMDItemUserTags	     42 
	com.apple.metadata:kMDItemWhereFroms	    190 
	com.apple.quarantine	     57 
```

notice all the metadata is missing.

I believe like [windows calls an OS level copy function](https://github.com/nodejs/node/blob/7d433a936bce26707f7a10bb56b121fe0dfb0a66/deps/uv/src/win/fs.c#L1958) in MacOS [`copyItem`](https://developer.apple.com/documentation/foundation/filemanager/1412957-copyitem) needs to be called.
Repository: go
Issue: os: TestReadWriteDeadlineRace failure on linux-amd64-clang builder
Labels: NeedsInvestigation

There was a single failure of `TestReadWriteDeadlineRace` on the `linux-amd64-clang` builder at head:
```
--- FAIL: TestReadWriteDeadlineRace (0.00s)
    timeout_test.go:498: Read returned non-timeout error read |0: not pollable
FAIL
FAIL	os	0.638s
```

I've only seen this failure once, but recording it here in case it becomes a pattern.

[2019-11-20T22:38:34-39a9cb4/linux-amd64-clang](https://build.golang.org/log/704e7ab7a8d45ba1e50da8784b9846a86adad9e3)

CC @ianlancetaylor
Repository: godot
Issue: RID Objects are always "Invalid RID"
Labels: enhancement,topic:editor,confirmed,usability

**Godot version:**
Beta 1
**OS/device including version:**
Windows 10 1903
**Issue description:**
No matter which function i use, the RID object always gives `Invalid RID`.
**Steps to reproduce:**
Try any of these:
`var ps_rid = get_viewport().get_world().get_space() # ps_rid is Invalid RID`
`var vs_rid = get_viewport().get_world().get_canvas() # vs_rid is Invalid RID`
```
var rigidBody = RigidBody2D.new()
var rb = rigidBody.instance()
add_child(rb)
print(rb.get_rid()) # prints Invalid RID
```

![image](https://user-images.githubusercontent.com/1850856/69358795-1ed01a00-0c90-11ea-94af-c51ca5d259cc.png)
![image](https://user-images.githubusercontent.com/1850856/69358816-2b547280-0c90-11ea-82d5-59fc8196ef0c.png)
![image](https://user-images.githubusercontent.com/1850856/69358839-34454400-0c90-11ea-8874-8eb08b92cc81.png)
Repository: go
Issue: os: ExpandEnv: Missing curly bracket at end of string substitutes var name whether or not env var is set
Labels: Documentation,help wanted,NeedsFix

<!-- Please answer these questions before submitting your issue. Thanks! -->

### What version of Go are you using (`go version`)?

<pre>
go version go1.12.9 linux/amd64
</pre>
and
<pre>
Go Playground (1.13.4)
</pre>


### Does this issue reproduce with the latest release?
Yes. (Go Playground)


### What operating system and processor architecture are you using (`go env`)?

<details><summary><code>go env</code> Output</summary><br><pre>
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/ceineke/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/ceineke/.gvm/pkgsets/go1.12.9/global"
GOPROXY=""
GORACE=""
GOROOT="/home/ceineke/.gvm/gos/go1.12.9"
GOTMPDIR=""
GOTOOLDIR="/home/ceineke/.gvm/gos/go1.12.9/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build732428512=/tmp/go-build -gno-record-gcc-switches"
</pre>
and
<pre>
Go Playground
</pre></details>

### What did you do?
https://play.golang.org/p/GSfBKiZqj2L

### What did you expect to see?
Either
```
Hello, ${name
Hello, ${name
```
or
```
Hello, ${name
Hello, foo
```

### What did you see instead?
```
Hello, name
Hello, name
```
Repository: terminal
Issue: Add support for `tmux` Control Mode
Labels: Issue-Feature,Help Wanted,Area-Interop,Area-UserInterface,Area-Extensibility,Product-Terminal

From discussion in #532, #1000:

> If we're looking at tmux for things like splits, why not implement [Tmux Control Mode](https://medium.com/@gveloper/using-iterm2s-built-in-integration-with-tmux-d5d0ef55ec30) like iTerm on MacOS. I switch back and forth between my Macs (with iTerm2) and my Surface Laptop with WSL but WSL2 combined with a _really_ usable terminal (comparable to iTerm2) would really make me consider leaving the Apple camp.
> 

> In #532, from @zadjii-msft:
> 
> > I did not know that Tmux Control Mode even existed. That sounds really awesome, I'm totally on board with trying to make that work somehow.
> 
> Was there any further thought put into this? I love tmux control mode - it's literally the only thing keeping me tethered to MacOS at work (since only iTerm supports it, and iTerm is mac-only). If it were implemented, I'd trash my Mac and request a Windows laptop immediately.
> 
> I'd love to see it listed as a post-1.0 milestone :)

We should do this.


Docs link: https://github.com/tmux/tmux/wiki/Control-Mode
Repository: go
Issue: x/build/cmd/gopherbot: make decision on whether anyone can add/remove release-blocker without specifying reason
Labels: Builders,NeedsDecision

gopherbot removed the release-blocker label in https://github.com/golang/go/issues/35482#event-2820741690.

Need to investigate and understand why it happened.
Repository: pytorch
Issue: CUDA masked_select uses way too much memory
Labels: module: performance,module: cuda,triaged

```py
In [1]: x = torch.randn(20000, 20000,device='cuda')

In [2]: torch.cuda.max_memory_allocated()
Out[2]: 1600126976

In [3]: m = ~torch.eye(20000, dtype=torch.bool, device='cuda')

In [4]: torch.cuda.max_memory_allocated()
Out[4]: 2401239040

In [5]: x[m]
Out[5]:
tensor([ 0.6415,  0.2951, -0.7630,  ...,  1.0940,  0.3103,  0.3631],
       device='cuda:0')

In [6]: torch.cuda.max_memory_allocated()
Out[6]: 10001317888

In [7]: torch.__version__
Out[7]: '1.4.0a0+91c6d2e'
```

Titan XP
CUDA: 10.1

Why does it allocate about 4.75x of the indexed tensor, rather than just 1x? 

cc @ngimel @VitalyFedyunin @mruberry
Repository: react
Issue: SuspenseList revealOrder="forwards" has issues when Suspense is nested in another element
Labels: Type: Needs Investigation,Component: Suspense,Component: Concurrent Features

**Do you want to request a *feature* or report a *bug*?**

SuspenseList is currently experimental, so it's neither. But I think that this should be fixed before it's stable.

**What is the current behavior?**

![Screen Recording 2019-11-21 at 12 39 56 PM](https://user-images.githubusercontent.com/1500684/69371223-a6039a80-0c5c-11ea-9fd4-848a3f1672fa.gif)

Note the 1000ms at the bottom (which should show last based on the `revealOrder`) shows at 2000ms for the nested version.

The only difference between those two sets of SuspenseLists is that one has nested the two bottom suspense elements in a `div`. Something like this:

```javascript
function App() {
  return (
    <SuspenseList revealOrder="forwards">
      <Suspense fallback={<div>2000ms loading</div>}>
        <A />
      </Suspense>
      <div> {/* nesting */}
        <Suspense fallback={<div>3000ms loading</div>}>
          <B />
        </Suspense>
        <Suspense fallback={<div>1000ms loading</div>}>
          <C />
        </Suspense>
      </div> {/* nesting */}
    </SuspenseList>
  )
}
```

https://codesandbox.io/s/suspenselist-issues-sdqzb

**What is the expected behavior?**

I would expect them to behave exactly the same. Nesting things is often necessary for styling reasons.

**Which versions of React, and which browser / OS are affected by this issue? Did this work in previous versions of React?**

I'm experiencing this on `react@0.0.0-experimental-b53ea6ca0` and `react-dom@0.0.0-experimental-b53ea6ca0`. I don't believe this ever worked in previous versions of React.
Repository: pytorch
Issue: Add scripts for comprehensive benchmark TensorIterator
Labels: module: performance,feature,triaged

## 🚀 Feature

Proposed in: https://github.com/pytorch/pytorch/pull/29743

TensorIterator is performance-critical but complicated. Changing some parts of it might
cause hard-to-realize regression on the other parts. It would be very convenient to have a script that benchmark all the case we could imagine:

**Memory layout**: trivial 1D, (contiguous dim, non-contiguous dims)
**Problem size**: small, medium, large
**Type of computation**: unary ops, binary ops, compare ops, reduction
**Data type**: all dtypes with/without promotion
**Inplace**: True, False
**Device**: CPU, CUDA

## The designed usage of the script could be:

**Step 1**:
Install a PyTorch build of the master branch, and run
```
python main.py benchmark baseline.json
```

**Step 2**:
Go to your branch, build install and run
```
python main.py benchmark new.json
```

**Step 3**:
Run the following command to get the report:
```
python main.py compare baseline.json new.json report.html
```



cc: @ngimel @VitalyFedyunin 
cc: @csarofeen 

cc @VitalyFedyunin @ngimel @mruberry
Repository: go
Issue: x/build/cmd/gopherbot: add/remove label functionality can be confusing if original comment is deleted
Labels: Builders,NeedsInvestigation,Community

`gopherbot` allows anyone to add or remove labels, as described at https://golang.org/wiki/gopherbot. Normally, it's possible to see who made the label add/remove change because of the comment requesting it. However, if that comment is deleted, it looks like gopherbot did it on its own, and it's not easy to find out why:

![image](https://user-images.githubusercontent.com/1924134/69372489-cbe66a80-0c6f-11ea-8242-4a9ae0708e5a.png)

maintner tracks all events and can be helpful in situations like this, but this issue is about finding a better way.

This happened recently, see https://github.com/golang/go/issues/35482#event-2820741690 and #35755.
Repository: go
Issue: proposal: crypto/tls: add support for exported authenticators
Labels: Proposal,Proposal-Hold,Proposal-Crypto

Proposal to implement support for exported authenticators in `crypto/tls` as described in [draft-ietf-tls-exported-authenticator-10](https://tools.ietf.org/html/draft-ietf-tls-exported-authenticator-10). I'm happy to write a CL for this.

cc @grittygrease @wbl
Repository: rust
Issue: Disambiguate between `()` as type and as value in const generic arguments
Labels: T-compiler,A-const-generics

```rust
#![feature(const_generics)]

trait Trait<const U: ()> {}

impl Trait<()> for () {} // error (expected const argument, found type argument)
```
I wonder if we can resolve the unit value correctly here (though maybe it'll require a small hack), because this is quite unfortunate.

<!-- TRIAGEBOT_START -->

<!-- TRIAGEBOT_ASSIGN_START -->

<!-- TRIAGEBOT_ASSIGN_DATA_START$${"user":null}$$TRIAGEBOT_ASSIGN_DATA_END -->

<!-- TRIAGEBOT_ASSIGN_END -->
<!-- TRIAGEBOT_END -->
Repository: flutter
Issue: [web] Reduce expensive CSS matchers
Labels: engine,c: performance,platform-web,P3,team-web,triaged-web

Some examples include:
* `flt-scene p`
* `flt-ruler-host p`
* `flt-glass-pane *`
Repository: svelte
Issue: Keyed each does not preserve element (re-create) when reordered
Labels: feature request,temp-stale

**Describe the bug**

I try to render a list of input element using keyed each block so that when enter key is pressed when an input is focused it will be swapped with the next input element. Likewise when shift+enter is pressed it will be swapped with the previous input element. Forward swapping works as expected and input keeps the focus. But backwards swap (shift+enter) causes the previous input to be re-created and the focus is lost.

I share a sample app written with both Svelte and React. React version work as expected.

**Codesandbox Repo**
[svelte demo](https://codesandbox.io/s/svelte-move-focus-i49mm)
[react demo](https://codesandbox.io/s/react-move-focus-piqis)

**Expected behavior**
Keyed list should respect the keys and matching elements should be reordered by getting detached and attached to preserve element instance.

**Information about your Svelte project:**
Svelte 3
Repository: pytorch
Issue: pairwise_dist eps argument is confusing
Labels: module: numerical-stability,module: bc-breaking,module: docs,triaged

The doc says that `eps` helps preventing division by zero (without mentioning how).  The actual implementation is 
https://github.com/pytorch/pytorch/blob/0fdbb762d1a9e6b9c262760b6b67bcb3d0814407/aten/src/ATen/native/Distance.cpp#L15-L17

However, `norm(zero_vector)` is perfectly stable and gives correct subgradient. So unless `pairwise_dist` output is used as a denominator, there is no DBZ or any instability. And how to use `pairwise_dist` output should be in users' control, unless we want to automatically add `eps` to every op that can output zeros.

Proposal: remove `eps` argument.

cc @ezyang @SsnL @gchanan
Repository: flutter
Issue: Generated files are created for platforms that are not configured
Labels: tool,P2,team-tool,triaged-tool

## Steps to Reproduce

1. On Linux machine, remove the `ios` and `android` directory to remove support for those platforms
1. `flutter run` a desktop app

ios and android directories are recreated with flutter_export_environment.sh, Generated.xcconfig, GeneratedPluginRegistrant.java, and local.properties.