sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: net/http/httputil: DumpRequestOut triggers clienttrace Get/GotConn
Labels: NeedsInvestigation

<!-- Please answer these questions before submitting your issue. Thanks! -->

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version go1.15.8 darwin/amd64
</pre>

### Does this issue reproduce with the latest release?

1.15.8 is the latest release.

### 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="/Users/dbrink/Library/Caches/go-build"
GOENV="/Users/dbrink/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/dbrink/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/dbrink/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/1.15.8/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.15.8/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
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 -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/gb/vk5wy1jd4msc49d1vpdwf_j80015jz/T/go-build050741674=/tmp/go-build -gno-record-gcc-switches -fno-common"
GOROOT/bin/go version: go version go1.15.8 darwin/amd64
GOROOT/bin/go tool compile -V: compile version go1.15.8
uname -v: Darwin Kernel Version 19.6.0: Tue Nov 10 00:10:30 PST 2020; root:xnu-6153.141.10~1/RELEASE_X86_64
ProductName:	Mac OS X
ProductVersion:	10.15.7
BuildVersion:	19H114
lldb --version: lldb-1200.0.44.2
Apple Swift version 5.3.2 (swiftlang-1200.0.45 clang-1200.0.32.28)
gdb --version: GNU gdb (GDB) 9.2
</pre></details>

### What did you do?

https://play.golang.org/p/_AAQ-0N9QI-

### What did you expect to see?

```
GET / HTTP/1.1
Host: www.foo.com
User-Agent: Go-http-client/1.1
Content-Length: 5
Accept-Encoding: gzip

hello
```

### What did you see instead?

```
GetConn: www.foo.com:80
GotConn: {0xc00000e360 false false 0s}
GET / HTTP/1.1
Host: www.foo.com
User-Agent: Go-http-client/1.1
Content-Length: 5
Accept-Encoding: gzip

hello
```
Repository: vscode
Issue: Source control: improve focus movement
Labels: bug,accessibility,scm

I did some improvements in the `ActionBar` focus which I think reflect positively in the SCM view. 
However there is still two corner cases which I think we could improve:

1. Focus onto an element in Changes view, press `tab` -> focus jump to inline actions (great)
2. Press `shit + tab` -> focus is moved to the Message box 🐛 

It would be cool if `tab` and `shift+tab` are always paired, thus `shift+tab` after a `tab` would bring you back to the original item.

One more corner case
1. Focus onto the first item in changes
2. Press `shift + tab` -> focus is moved in title bar actions 🐛 
3. I would expect the focus to move into the commit box

Overall I think the focus navigation in SCM works nice, thanks to this "top level" up and down navigation which allows to focus into something.

fyi @joaomoreno
Repository: go
Issue: io/fs: provide backwards-compatible migration path from ioutil.ReadFile to fs.ReadFile
Labels: NeedsInvestigation

Hi there, I'm enjoying the go1.16 RC, I'm trying to migrate some code from using ioutil.ReadFile to fs.ReadFile while staying backwards-compatible. I'm running into two things:

* It's not possible to make code that does something like `ioutil.ReadFile("./path/to/file.txt")` work with `os.DirFS(".")`, because it rejects the leading `./`
* There is no way to have absolute paths for the same reason, because 'Open should reject attempts to open names that do not satisfy ValidPath(name)' according to the fs.FS documentation

I can get around those things by doing something like this:

```
// osFS is an fs.FS implementation that just passes on to os.Open
type osFS struct{}

func (c osFS) Open(name string) (fs.File, error) {
	return os.Open(name)
}
```

Which is technically an invalid implementation, although it's not entirely clear why those restrictions are in place. It would be nice if something like the above `osFS` was built-in, so there's a clear migration path from `ioutil.ReadFile` to `fs.ReadFile`.

### What version of Go are you using (`go version`)?

<pre>
$ go version go1.16rc1 darwin/arm64
</pre>
Repository: godot
Issue: Resources: get_path() does not work anymore and the inspector show weird paths for resources.
Labels: bug,needs testing,topic:import

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
<!-- Specify commit hash if using non-official build. -->
master branch.

**OS/device including version:**
<!-- Specify GPU model, drivers, and the backend (GLES2, GLES3, Vulkan) if graphics-related. -->
Archlinux

**Issue description:**
<!-- What happened, and what was expected. -->
I am not sure this is a bug, but well, things are not behaving as before.

In my TileSet / TileMap editor rework, I wanted to display the name of a texture (from its file) in the list of atlases. I used to use get_path() on the texture, but it does not work anymore. The Resource get_path() function does not return anything.

Also, the inspector now displays a weird path for the texture, which seems to be the result of the get_import_path() function.
Something like that: res://.godot/imported/tileset.png-fc1b0642a848fef5fa6557a55d39c316.stex
It's not a big problem per say, but I don't know if this generated path is useful to be modified by the inspector.

The issue is likely caused by https://github.com/godotengine/godot/pull/45903, as the issue is quite recent.

**Steps to reproduce:**

Simply try to call get_path() on a resource (in my case a Texture2D).

**Minimal reproduction project:**
<!-- A small Godot project which reproduces the issue. Drag and drop a zip archive to upload it. -->
Ask if needed.
Repository: create-react-app
Issue: Serving a production build
Labels: issue: proposal,needs triage

I have opened this previously here, but it was closed: https://github.com/facebook/create-react-app/issues/6683

I'm not sure why, because [the solution provided](https://facebook.github.io/create-react-app/docs/deployment#static-server) is only a viable solution in a fraction of the use cases. There are the following limitations:

1. using `serve` bypasses react's proxy configuration and other things that may be set up in the dev environment, such as setupProxy.js, so this has to be set up manually. 
2. what about watching? Are we going to run a (slow) build from scratch each time a change is made?

I've hit bugs that are "only present in the production build", plenty of times on multiple projects. Usually it's a case of the project following bad practices or something unconventional - however having this working properly would save time, and help developers using CRA to investigate and fix these with just a simple command line parameter, such as `ng serve --prod` does.
Repository: godot
Issue: Pressing up/down in Tree search controls will move caret
Labels: bug,topic:editor,usability,topic:input

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
<!-- Specify commit hash if using non-official build. -->
3.2.4 beta

**Issue description:**
<!-- What happened, and what was expected. -->
So I noticed it just now, but when you press up/down to select items e.g. in new node dialog, the caret in the search box will move too
![oSE0FhssZC](https://user-images.githubusercontent.com/2223172/108058834-e23acc00-7054-11eb-9fc6-994534b6c44e.gif)
I noticed the same thing also in quick open dialog and other dialogs might be affected too.
Repository: flutter
Issue: [Google_Sign_In] signInSilently() allows to sign in successfully using suspended google workspace account and not throwing any exception.
Labels: p: google_sign_in,package,team-ecosystem,P3,triaged-ecosystem

Hey all,

I am trying to test if I am able to do google sign in silently using a suspended google workspace account and for some reason I am able to sign in successfully which I thought this shouldn't be the case. If I were to do a normal google sign in, google will throw an error stating that this account is disabled. How can I do google sign in silently yet receive the suspended status?

The flutter package that I am currently using is: google_sign_in 

Thank you.
Repository: terminal
Issue: Custom variables in settings.json for easier profile configuration
Labels: Issue-Feature,Area-Settings,Product-Terminal

<!-- 
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨

I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING:
1. If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement.
2. If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement.
3. If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number).
4. If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement.
5. If I file something completely blank in the body, the core team may close my issue without further explanation or engagement.

All good? Then proceed!
-->

# Description of the new feature/enhancement
Let one define custom variables in settings.json for use with profiles.
E.g. define myCustomVariable for later use within commandline to avoid adding same long path to SSH key file for every profile name.
Would also be great to reuse current profile parameters such as "name" to connect to this hostname by stating $profile.name with SSH as shown in example. 
<!-- 
A clear and concise description of what the problem is that the new feature would solve.
Describe why and how a user would use this new functionality (if applicable).
-->

# Proposed technical implementation details (optional)
```
"profiles":
{
    "defaults":
    {
        "myCustomVariable": "C:\\Users\\USERNAME\\SSH_KEY_FILE_PATH.key"
    },
    "list":
    [
        {
            "name": "HOSTNAME",
            "commandline": "ssh -i $myCustomVariable $profile.name", 
        },
```
<!-- 
A clear and concise description of what you want to happen.
-->
Repository: angular
Issue: Disable router location subscription
Labels: area: router,cross-cutting: custom elements,P4

# 🚀 feature request

### Relevant Package
@angular/router


### Description
We are wrapping an router supported angular component as a web component using createCustomElement().
This works perfectly fine right now except that when the component is unloaded, the router still listens to location changes and performs routing. I would expect to find something to turn this off somehow but there isn't.
I have tried the dispose() method but since the router instance seems to be kept alive, it will stop processing routes the second time I load the web component.

What I would like to do is to either stop the `locationSubscription` when my web component destroys or having `initialNavigation()` handle a second call, resetting the router.

Is there any way to do this?
Repository: godot
Issue: Batching don't work with CPUParticles2D and Particles2D
Labels: topic:rendering,documentation,topic:2d

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
Godot 3.2.4 rc 2


**OS/device including version:**
Windows 10


**Issue description:**
Can't batch multiple CPUParticles2D or Particles2D with the same material and texture at the same Node2d and each CPUParticles2D use new draw call.  I think its can be a bug and not a request for a new future, becaurse in doc page https://docs.godotengine.org/ru/stable/tutorials/optimization/batching.html in "default batches" is PA - particles.
Repository: go
Issue: encoding/asn1: possible inconsistency between asn1 tag and unmarshal parameter
Labels: NeedsInvestigation

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version go1.15.8 linux/amd64
</pre>

### Does this issue reproduce with the latest release?

Yes

### What operating system and processor architecture are you using (`go env`)?

Ubuntu 18.04.5 LTS/x86_64

### What did you do?

I stumbled upon this (possible) inconsistency between the asn1 tag embedded in a byte slice to be unmarshalled as asn1 and the parameter supplied to the unmarshaller. It seems that it's possible, for instance, to unmarshal the following byte array (specifying the "printable" parameter)

`[]byte{20 /* TagT61String */, 13, 72, 101, 108, 108, 111, 44, 32, /* 2 non-ASCII chars follow*/ 228, 184, 150, 231, 149, 140}`

while in fact it

1. is tagged as a T.61 string
2. contains non-ASCII characters.

More examples available [here](https://play.golang.org/p/30NeaSbSda-)

### What did you expect to see?

An error

### What did you see instead?

No error and a successful unmarshalling
Repository: pytorch
Issue: [FR} Implement Skellam distribution
Labels: module: distributions,triaged

## 🚀 Feature
Skellam distribution (difference of two independent Poissons) is fairly useful. 

It is a discrete distribution with two parameters as the rates of the Poissons.

It is easy to evaluate pdf. While cdf seems non-trivial, we can compute it by linking it to cdf of a non-central x2 (see Eqn (4) in On an Extension of the Connetion Between Poisson and chi-sq2 Distributions, N.L Johnson(1959)), and use existing libraries (e.g., cdflib). 

It is more involved to compute the gradient for the two parameters (rates) wrt cdf. But it might be doable by realizing that the non-central x2 cdf is a Marcun Q function and using its gradients detailed in On some properties of the Marcum Q function. 

### Note
As a POC, I have implemented a version that computes the Skellam CDF by porting cdflib functions to build custom c++ & cuda extensions, and can do backprop with CDF at zero, i.e., `P[ Skellam(mu1, mu2) <= 0 ]`. 


### Ref
On an Extension of the Connetion Between Poisson and chi-sq2 Distributions, N.L Johnson(1959)
On some properties of the Marcum Q function, Yu A. Brychkov (2012) 



cc @fritzo @neerajprad @alicanb @vishwakftw @nikitaved
Repository: node
Issue: Add on/off to EventSignal?
Labels: doc,events,eventtarget

Would if make sense to add on/off as aliases to addEventListener/removeEventListener?

I'm ending up writing quite a bit of code that checks for either since usually I like to implement api's to support both EventEmitter and AbortSignal. Having the aliases would make code much easier and less error prone.
Repository: bitcoin
Issue: Add Labels/Groups to Output Descriptors
Labels: Feature

[Output Descriptors](https://github.com/bitcoin/bitcoin/blob/master/doc/descriptors.md#multisig) do not contain labels.

Having labels for the descriptors could greatly enhance the UX of the wallets using these.

We could use syntax similar to this one to introduce labels. The characters `{}` are unused yet in the descriptors and the [Checksums](https://github.com/bitcoin/bitcoin/blob/master/doc/descriptors.md#checksums) section mentions that using these does not decrease the error detection ability of the currently used checksum.

```
sh(multi(2,{Alice}022f01e5e15cca351daff3843fb70f3c2f0a1bdd05e5af888a67784ef3e10a2a01,{Bob}03acd484e2f0c7f65309ad178a9f559abde09796974c57e714c35f110dfc27ccbe))
```

or

```
wsh(sortedmulti(1,{Alice}xpub661MyMwAqRbcFW31YEwpkMuc5THy2PSt5bDMsktWQcFF8syAmRUapSCGu8ED9W6oDMSgv6Zz8idoc4a6mr8BDzTJY47LJhkJ8UB7WEGuduB/1/0/*,{Bob}xpub69H7F5d8KSRgmmdJg2KhpAK8SR3DjMwAdkxj3ZuxV27CprR9LgpeyGmXUbC6wb7ERfvrnKZjXoUmmDznezpbZb7ap6r1D3tgFxHmwMkQTPH/0/0/*))
```

Related to https://github.com/nunchuk-io/bips/pull/1 where the need originated from.

Having labels also is also beneficial for scenarios where no multisig is used, because that allows the naming of exported descriptors for watch-only wallet, e.g.

```
pkh({Corporate Cold Wallet}[d34db33f/44'/0'/0']xpub6ERApfZwUNrhLCkDtcHTcxd75RbzS1ed54G1LkBUHQVHQKqhMkhgbmJbZRkrgZw4koxb5JaHWkY4ALHY2grBGRjaDMzQLcgJvLJuZZvRcEL/0/*)
```
Repository: rust
Issue: Make async futures `Unpin` if they don't borrow anything across `.await`s
Labels: C-feature-request,A-async-await,AsyncAwait-Triaged

Am I missing something or could rustc in principle make the anonymous (return) type of `async fn` and `async {}` blocks implement `Unpin` (or maybe it should be "not implement `!Unpin`") if the body does not contain any borrows across `.await` points? I couldn't find an existing issue about this.
Repository: flutter
Issue: making sense of SystemUiOverlay names
Labels: framework,f: material design,d: api docs,team-design,triaged-design

what does SystemUiOverlay part mean? isn't "Overlay" part redundant, wouldn't it better make sense if it was just SystemUi?

examples of possible renames

***enum*** SystemUiOverlay -> SystemUi
***class***  SystemUiOverlayStyle -> SystemUiStyle
***method*** setEnabledSystemUIOverlays -> setEnabledSystemUIs
***method*** setSystemUIOverlayStyle -> setSystemUIStyle
***method*** restoreSystemUIOverlays -> restoreSystemUIs

and by the way there's some inconsistency

in methods i listed UI is fully uppercased, whilst in class names it's only capitalised - Ui
if we follow style guidelines, they recommend to use camel case, but that's a tricky case imo
though still - it should be consistent
Repository: go
Issue: net/http/httputil: export function singleJoiningSlash
Labels: NeedsInvestigation

<!--
Please answer these questions before submitting your issue. Thanks!
For questions please use one of our forums: https://github.com/golang/go/wiki/Questions
-->

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version go1.16beta1 darwin/amd64
</pre>

### Does this issue reproduce with the latest release?

Yes, the function singleJoiningSlash is not exported.

https://github.com/golang/go/blob/1004a7cb31ae31d2ca0b54b507b996c12403d54c/src/net/http/httputil/reverseproxy.go#L102-L112


### What did you expect to see?

The function should be exported, since several open source projects have to copy it instead of using it directly, which is suboptimal if you already use net/http/httputil as dependency:

Openshift:
https://github.com/openshift/console/blob/master/pkg/proxy/proxy.go#L69

AWS Proxy:
https://github.com/acquia/aws-proxy/blob/master/proxy/reverseproxy.go

Chronograf:
https://github.com/influxdata/influxdb/blob/master/chronograf/server/proxy.go
Repository: pytorch
Issue: [JIT] __getitem__ on scripted ModuleDict returns unscripted module
Labels: oncall: jit,days

## 🐛 Bug
Using `__getitem__` to access a submodule stored inside a scripted `ModuleDict` instance (i.e. `module_dict["name"]) returns an unscripted module.

## To Reproduce
(Probably not minimal)
```
import torch
from typing import Dict, List

class Encoder(torch.nn.Module):
    def __init__(self):
        super().__init__()

    def _forward(self, x: int):
        return {x: torch.tensor(1)}

    def forward(self, x: int):
        out = self._forward(x)
        return out

class Fusion(torch.nn.Module):
    state: Dict[int, torch.Tensor]

    def __init__(self, encoders: List[Encoder]):
        super().__init__()
        self.encoders = torch.nn.ModuleList(encoders)
        self.state = {}

    def forward(self, x: int):
        if len(self.state) > 0:
            return self.state
        out = self.encoders[0](x)
        self.state = out
        return out


class Mod(torch.nn.Module):
    def __init__(self, fusion: Fusion):
        super().__init__()
        self.fusion = fusion

    def forward(self, x: int):
        return self.fusion(x)


class MtlMod(torch.nn.Module):
    def __init__(self, task_models: Dict[str, Mod]):
        super().__init__()
        self.task_models = torch.nn.ModuleDict(task_models)

    def forward(self, x: int):
        for task, model in self.task_models.items():
            model(x)

encoder = Encoder()
fusion = Fusion([encoder])
model = MtlMod({"t1": Mod(fusion), "t2": Mod(fusion)})
s = torch.jit.script(model)

a = s.task_models["t1"]
b = s.task_models.t1
print(type(a))
print(type(b))
```

## Actual output
```
<class '__main__.Mod'>
<class 'torch.jit._script.RecursiveScriptModule'>
```
## Expected output
```
<class 'torch.jit._script.RecursiveScriptModule'>
<class 'torch.jit._script.RecursiveScriptModule'>
```

cc @gmagogsfm
Repository: pytorch
Issue: Implementation of many complex functions is fast but inaccurate in libc++
Labels: module: numerical-stability,triaged,module: complex,enhancement,module: numpy

Follow up based on https://github.com/pytorch/pytorch/issues/47500 and https://github.com/pytorch/pytorch/issues/42952

libc++ uses template based implementation for many complex functions which is subject to numerical errors, see
implementation for acos for example:
https://github.com/llvm/llvm-project/blob/ce8f10d6cbe03c1a78a7938070654d363e25f27d/libcxx/include/complex#L1160
Whereas libm (on both MacOS and Linux) relies on slower but less prone to errors method described in
https://people.freebsd.org/~das/kahan86branch.pdf

**For most of the Linux distributions, libstdc++ is compiled to fall back to libc for c99 complex operations, however MacOS, Android and other platforms which do use libc++ are affected by this issue.** 

See 
https://github.com/bminor/glibc/blob/master/math/s_csqrt_template.c
https://github.com/bminor/glibc/blob/master/math/k_casinh_template.c
and 
https://opensource.apple.com/source/Libm/Libm-47.1/complex.c.auto.html
as an examples

Operators to be fixed:

- [x] sqrt
- [x] acos
- [ ] asin
- [ ] atan
- [ ] atan2
- [ ] acosh
- [ ] asinh
- [ ] atanh
- [ ] pow

cc @ezyang @anjali411 @dylanbespalko @mruberry @rgommers @heitorschueroff
Repository: PowerToys
Issue: [KBM] 2 finger zoom gesture causes false positive (verified on trackpad)
Labels: Issue-Bug,Product-Keyboard Shortcut Manager,Priority-3

Crutkas: do a 2 finger zoom motion with KBM's type dialog open, it has a false positive for control.

![image](https://user-images.githubusercontent.com/61511441/108102524-9bd08600-70c3-11eb-89eb-c72f04afe154.png)
双指缩放会被识别成“ctrl-left”，这会导致误触
Repository: go
Issue: crypto/x509: store stripped down trust anchors
Labels: NeedsInvestigation

Rather than storing full `x509.Certificate`s in the root pool we could instead store only the information that we require to accomplish chain building (SPKI, subject, name constraints), significantly reducing the memory footprint for the pool.

When returning certificate chains from Verify we'd need to construct a stand-in `x509.Certificate` which is only partially populated (we'd need to decide what fields to set, and what we can get away with setting based on a typical root profile, i.e. is it safe to consistently set the issuer to match the subject, even for roots where the issuer does not actually match?). Depending on what users are doing with certificates it is possible that this could break some things. As such we should try to land this change early in the 1.17 cycle so any significant breakages are signaled early.

This would also reduce binary sizes on iOS where we bundle roots, and also make things like #43958 somewhat more appealing from a binary bloat perspective.