sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: x/net/http2: support http2 proxy connections
Labels: FeatureRequest

Please answer these questions before submitting your issue. Thanks!


### What version of Go are you using (`go version`)?

```
go version go1.10.3 linux/amd64
```

### Does this issue reproduce with the latest release?

Yes

### What operating system and processor architecture are you using (`go env`)?

```
GOARCH="amd64"
GOBIN=""
GOCACHE="..."
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="..."
GORACE=""
GOROOT="/usr/lib/go"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
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-build612803013=/tmp/go-build -gno-record-gcc-switches"
```

### What did you do?

I have an HTTP/2 proxy (I tested a custom one and https://github.com/caddyserver/forwardproxy) and when trying to connect through it from a client using HTTP/2 I get an error. This does not happen if I don't use HTTP/2.

Here's my code:

```go
package main

import (
	"crypto/tls"
	"fmt"
	"io/ioutil"
	"net/http"
	"os"

	"golang.org/x/net/http2"
)

func main() {
	tr := &http.Transport{
		TLSClientConfig: &tls.Config{InsecureSkipVerify: true},
		Proxy:           http.ProxyFromEnvironment,
	}

	if err := http2.ConfigureTransport(tr); err != nil {
		fmt.Fprintf(os.Stderr, "error configuring transport: %v\n", err)
		os.Exit(1)
	}

	client := &http.Client{Transport: tr}
	rsp, err := client.Get("https://google.com")
	if err != nil {
		fmt.Fprintf(os.Stderr, "error getting: %v\n", err)
		os.Exit(1)
	}
	defer rsp.Body.Close()

	buf, err := ioutil.ReadAll(rsp.Body)
	if err != nil {
		fmt.Fprintf(os.Stderr, "error reading response: %v\n", err)
		os.Exit(1)
	}

	fmt.Printf("Response: %s\n", string(buf))
}
```

If I comment out the `http2.ConfigureTransport()` call (so it will use HTTP/1.1) it works fine.

### What did you expect to see?

A successful connection.

### What did you see instead?

On the proxy server:

```
2018/07/19 18:30:25 http2: server connection from 127.0.0.1:54850 on 0xc420174b60
2018/07/19 18:30:25 http2: server: error reading preface from client 127.0.0.1:54850: bogus greeting "CONNECT google.com:443 H"
```

On the client:

```
2018/07/19 18:30:25 http2: Transport failed to get client conn for google.com:443: http2: no cached connection was available
error getting: Get https://google.com: malformed HTTP response "\x00\x00\x18\x04\x00\x00\x00\x00\x00\x00\x05\x00\x10\x00\x00\x00\x03\x00\x00\x00\xfa\x00\x06\x00\x10\x01@\x00\x04\x00\x10\x00\x00"
```
Repository: rust
Issue: Sub-optimal performance of Iterator::max_by_key in some cases
Labels: I-slow,T-compiler

@pftbest noted that this test case:

```rust
    #[bench]
    fn bench_max_by_key(b: &mut Bencher) {
        struct Data { k: i32, z: i8 }
        let v: Vec<Data> = (0..10000).map(|i| Data { k: i, z: 48 }).collect();
        b.iter(|| v.iter().max_by_key(|x| x.k));
    }
```

is about 2–3 times slower than the same test run with this alternate implementation of `Iterator::max_by_key`:

```rust
    fn max_by_key2<B: Ord, F>(mut self, mut f: F) -> Option<Self::Item>
    where F: FnMut(&Self::Item) -> B {
        let mut max_el = self.next()?;
        let mut max_val = f(&max_el);
        for el in self {
            let val = f(&el);
            if val >= max_val {
                max_val = val;
                max_el = el;
            }
        }
        Some(max_el)
    }
```

However, if the field `z` is removed from `struct Data`, the libstd test case runs as fast as the alternate one.

[Complete example on playground](https://play.rust-lang.org/?gist=72627da67b11c06cc57fbffb3631f028&version=nightly&mode=debug&edition=2018)
Repository: pytorch
Issue: [JIT] Unify python -> SugaredValue construction + fix inlining of graphs with Tuple-typed inputs
Labels: oncall: jit

1. Remove stuff like this https://github.com/pytorch/pytorch/blob/master/torch/csrc/jit/script/init.cpp#L87 and move it into a unified python -> SugaredValue constructor (e.g. `create()`). This will require having flags that indicate that a value is e.g. constant or a submodule.

2. Fix disabled tests for annotated script functions and methods that are taking in Tuples. The simplest form is to get these graphs to be inlineable, first step is to relax the constraint in LowerTuples. 

cc @suo
Repository: flutter
Issue: No word-breaks for CJK locales
Labels: a: text input,engine,a: internationalization,a: china,has reproducible steps,P3,team-engine,triaged-engine,found in release: 3.19,found in release: 3.22

Internal: b/112618495

Flutter cannot properly detect word breaks in Chinese and Japanese text because the ICU configuration that we ship with Flutter doesn't include the dictionary to provide that functionality.

Chinese example:
Type 你好吗 into a text field. Long-pressing on either of the first two characters should select both characters as they form one word.

Japanese example:
Type 日本語学校 into a text field (means Japanese language school). Long-pressing on either of the last two characters should select them both to select the word school (学校).

Compare the result of the two experiments above between a native textfield and a Flutter text field to see the difference.

Other languages (e.g. English or German words) work fine and long-pressing a word selects the full word.

/cc @cbracken @xster @Hixie
Repository: vue-element-admin
Issue: adding Font-awesome
Labels: in plan

Hello,

Can i add font-awesome in our best powerfull template of vue-element-admin ?

Thank's for help,

Merci d'avance,
Repository: go
Issue: cmd/go: build: add -static flag
Labels: help wanted,Proposal,Proposal-Accepted,GoCommand

This is a proposal to add `-static` flag to `go build`.

Producing a static build with go already requires a non-trivial amount of flags passed to `go build`, which on Linux currently amounts to something like:

`-ldflags '-extldflags "-fno-PIC -static"' -buildmode pie -tags 'osusergo netgo static_build'` [1]

...and this magic string keeps growing.

It would be awesome to encapsulate this sacred knowledge internally, exposing it via a new `-static` flag for `go build` and friends.

In addition to the above benefit of hiding the complexity, `-static` can also bring us more niceties, such as:
* automatic `static` build tag third-party software can rely upon. Currently using `static_build` tag seems like a de-facto standard, used a lot, but it has to be explicitly defined like in [1] above.
* providing `--static` flag to `pkg-config` invocations (those initiated by `// #cgo pkg-config: lib` lines in the source code). It will solve another issue (linking against a proper set of libraries for both static and dynamic link cases) for which a somewhat verbose workaround is currently required (for example, see [ploop_link_static.go](https://github.com/kolyshkin/goploop/blob/master/ploop_link_static.go) and [ploop_link_dynamic.go](https://github.com/kolyshkin/goploop/blob/master/ploop_link_dynamic.go)).

### See also
* https://github.com/golang/go/issues/12058 (original proposal for adding conditional `--static` to `pkg-config`)
* https://github.com/golang/go/issues/24787 (initial version of this very proposal)
Repository: TypeScript
Issue: fourslash test breaks when setting "@lib"
Labels: Infrastructure

**TypeScript Version:**  master

**Code**

```ts
/// <reference path="fourslash.ts" />

// @lib: es2015

////function f(x: number[]) { return x.length; }

verify.noErrors();
```

**Expected behavior:**

No error.

**Actual behavior:**

`Error: At line 1, col 0: Found an error: /tests/cases/fourslash/quickInfoBar.ts@35: Property 'length' does not exist on type '{}'.`
Repository: rust
Issue: derive Debug incorrect assumption
Labels: C-enhancement,A-diagnostics,T-compiler

Hello,

I'm guessing there "might" be something wrong how the `Debug` trait is derived in case a type is owned but not actually needed for formatting the object. The following code fails because `B` is not necessarily `Debug` which is correct, however since `Foo` doesn't actually have an instance of `B` as a field, that shouldn't be needed, and `B::Item` is restricted to `Debug`:

```
#[derive(Debug)]
struct Foo<B: Bar>(B::Item);


trait Bar {
    type Item: Debug;
}

fn foo<B: Bar>(f: Foo<B>) {
    println!("{:?}", f);
}


struct ABC();

impl Bar for ABC {
    type Item = String;
}


fn main() {
    foo(Foo::<ABC>("a".into()));
}

```

```
error[E0277]: `B` doesn't implement `std::fmt::Debug`
  --> src/main.rs:65:22
   |
65 |     println!("{:?}", f);
   |                      ^ `B` cannot be formatted using `:?` because it doesn't implement `std::fmt::Debug`
   |
   = help: the trait `std::fmt::Debug` is not implemented for `B`
   = help: consider adding a `where B: std::fmt::Debug` bound
   = note: required because of the requirements on the impl of `std::fmt::Debug` for `middlewares::authentication::Foo<B>`
   = note: required by `std::fmt::Debug::fmt
```
Repository: go
Issue: cmd/compile: constant string -> []byte and []byte -> string conversions aren't constant folded
Labels: Performance,NeedsInvestigation,compiler/runtime

### What did you see?

Conversions like

```
string([]byte{...})
[]byte("...")
```

are not constant folded. 

### What did you expect?

Conversions of const literals should probably be constant folded because it significantly slows down
things like

```
for .... {
  p.Write([]byte(":=")
}
```

This is related to https://github.com/golang/go/issues/26264 but the scope of that issues should probably be the difference between `Write, WriteByte, WriteString` as some conversions not being constant folded is an issue on a different level that has little to do with the performance of these functions per se. 

I can't quite tell whether `[]rune(...)` is also not constant folded because I'm not sure what `type.[]int32` actually does but for `[]byte(...)` and `string(...)` there are calls to `runtime.stringtoslicebyte(SB)` and `runtime.slicebytetostring(SB)`. 


### What version of Go are you using (`go version`)?

 1.10.3
Repository: rust
Issue: Rustdoc: unify impl blocks where possible.
Labels: T-rustdoc

From #32631:
> We'll have to be sure to update rustdoc, however, to provide one unified view to the API surface area rather than multiple impl blocks.

I have run into a similar problem when using codegen (specifically, in [web-sys]), where currently each function/value is inside its own impl block, e.g.

```rust
impl Event {
    pub const CAPTURING_PHASE: u16 = 1;
}
impl Event {
    pub const AT_TARGET: u16 = 2;
}
impl Event {
    pub const BUBBLING_PHASE: u16 = 3;
}
impl Event {
    pub const ALT_MASK: i32 = 1;
}
```

Presently all these appear as separate impl blocks in rustdoc. It would be easier to read if they were in a single block.

[web-sys]: https://github.com/rustwasm/wasm-bindgen/tree/master/crates/web-sys
Repository: vue-element-admin
Issue: RTL Support
Labels: PR Welcome

Please add rtl support to project.

Tanks a lot
Repository: angular
Issue: browser back froward does not trigger router change detection on upgraded app 
Labels: type: bug/fix,freq1: low,area: router,area: upgrade,state: confirmed,P3

<!--
PLEASE HELP US PROCESS GITHUB ISSUES FASTER BY PROVIDING THE FOLLOWING INFORMATION.

ISSUES MISSING IMPORTANT INFORMATION MAY BE CLOSED WITHOUT INVESTIGATION.
-->

## I'm submitting a...
<!-- Check one of the following options with "x" -->
<pre><code>
[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report  <!-- Please search GitHub for a similar issue or PR before submitting -->
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead see https://github.com/angular/angular/blob/master/CONTRIBUTING.md#question
[ ] Other... Please describe:
</code></pre>

## Current behavior
<!-- Describe how the issue manifests. -->
browser back froward does not trigger router change detection on upgraded app 

## Expected behavior
<!-- Describe what the desired behavior would be. -->

browser back froward trigger router change detection on upgraded app !
## Minimal reproduction of the problem with instructions
<!--
For bug reports please provide the *STEPS TO REPRODUCE* and if possible a *MINIMAL DEMO* of the problem via
https://stackblitz.com or similar (you can use this template as a starting point: https://stackblitz.com/fork/angular-gitter).
-->
https://stackblitz.com/edit/ngupgraderouter
## What is the motivation / use case for changing the behavior?
<!-- Describe the motivation or the concrete use case. -->
:heart: 

## Environment

<pre><code>
Angular version: 6.0.7
<!-- Check whether this is still an issue in the most recent Angular version -->

Browser:
- [ ] Chrome (desktop) version XX
- [ ] Chrome (Android) version XX
- [x ] Chrome (iOS) version XX
- [x ] Firefox version XX
- [ ] Safari (desktop) version XX
- [ ] Safari (iOS) version XX
- [ ] IE version XX
- [ ] Edge version XX
 
For Tooling issues:
- Node version: XX  <!-- run `node --version` -->
- Platform:  <!-- Mac, Linux, Windows -->

Others:
<!-- Anything else relevant?  Operating system version, IDE, package manager, HTTP server, ... -->
</code></pre>
Repository: pytorch
Issue: Request: Pycuda interoperability
Labels: feature,triaged

For doing computations on tensors with `pycuda`, I can do this
for going from `Tensor` to `pycuda.gpuarray.GPUArray`:
```
def tensor_to_gpuarray(tensor):
    if not tensor.is_cuda:
         raise ValueError('Cannot convert CPU tensor to GPUArray (call `cuda()` on it)')
     else:
         return GPUArray(tensor.shape, dtype=torch_dtype_to_numpy(tensor.dtype), gpudata=tensor.data_ptr())
```
however, I'm stumped on how to convert back to a Tensor without copying the data. How can this be done?
Repository: go
Issue: x/tools/go/analysis/cmd/vet: add -cgoinclude check for #include of code in subdirectories
Labels: help wanted,NeedsFix,Analysis

The go build model is that all code for a package is in one directory.
The cache relies on this.
Vendoring tools rely on this.
Probably many other things rely on this.

One of the few ways a package can violate this rule is by using
#include in cgo source code to access code in nearby directories.
This appears to work, in that the package builds,
but it doesn't cache properly, doesn't vendor properly, and so on.

It seems reasonable for cmd/vet to notice such #includes and
complain about them, and we could enable that check as one
of the automatic ones run during go test.

Suggested in #26366.
Repository: flutter
Issue: Feature Request: Support for endDrawer on Scaffold when AppBar actions exist
Labels: c: new feature,framework,f: material design,c: proposal,P2,team-design,triaged-design

If the `actions` property exists on an `AppBar` object, then the `endDrawer` property on the parent `Scaffold` object is ignored. It would be nice to have support for both together, so that `endDrawer` menus could peacefully co-exist with `Scaffold` `action` properties.
Repository: flutter
Issue: Make flutter_driver and flutter_test APIs more similar
Labels: a: tests,c: new feature,tool,c: API break,t: flutter driver,P3,team-tool,triaged-tool

We have lots of minor unnecessary differences right now. We should implement as much of each API on the other, to make it easier to migrate tests back and forth.
Repository: kubernetes
Issue: GCE: ILB->ClusterIP orphans a few GCP resources
Labels: kind/bug,area/provider/gcp,lifecycle/frozen,sig/cloud-provider,needs-triage

/kind bug

**How to reproduce it (as minimally and precisely as possible)**:
1. Create a service of type ILB.
2. Edit the service 
    a. Change the type to ClusterIP
    b. Remove the port's nodeport

Observe the service controller calls the delete code for the external load balancer instead of the internal loadbalancer.

/sig gcp
/assign @bowei 
cc @freehan @MrHohn
Repository: go
Issue: x/tools/cmd/stringer: handle untyped constants with typed initial values
Labels: NeedsInvestigation,FeatureRequest,Tools

### What version of Go are you using (`go version`)?

1.10.2, 1.11beta2 (but "stringer" is the same version either way)

### Does this issue reproduce with the latest release?

probably ("stringer" is not really included)

### What operating system and processor architecture are you using (`go env`)?

linux/amd64

### What did you do?

Tried to run stringer on a program:
https://play.golang.org/p/y0bNVBnOyAi

### What did you expect to see?

I expected it to correctly discern that FooA and FooB were of type Foo, and FooAMask and FooBMask were of type FooMask. Looking at the code, stringer is looking for the declared type, not for an inferred type inferred from an explicitly-typed expression.

### What did you see instead?

$ go generate
stringer: no values defined for type Foo

This is surprisingly hard to fix in the code stringer is run against. You can do `FooA Foo = iota; FooAMask FoomMask = (1 << iota)`, but you can't do following lines and have them get the same types. You can't declare two things of different types on a line using commas. And if you don't declare them on the same line, it's possible for FooA and FooAMask to get out of sync. (Well, the first one in each list might be the same, but if you had two twenty-item lists, they would end up out of sync.)
Repository: pytorch
Issue: [caffe2]running problem,  No module named caffe2_pybind11_state_hip
Labels: caffe2

I follow the Detectron installation, and i succeed install caffe2
![1532156622 1](https://user-images.githubusercontent.com/35193543/43033153-b049446c-8cf7-11e8-816d-f67d6fca3ab0.jpg)
but i don't test the GPU support, so sad for this.

and i install cocoapi and detectron
when i finish this, and want to test detectron.
it appear this problem
![1532156829 1](https://user-images.githubusercontent.com/35193543/43033194-706aaf10-8cf8-11e8-80c1-8472a87520b8.jpg)


and however the more sorrowful is that 
![1532156930 1](https://user-images.githubusercontent.com/35193543/43033175-23f31dfc-8cf8-11e8-98f7-ea06fc897155.jpg)

and I want to know if my operation cause caffe2 error, i uninstall the python-pip when i install detectron.
and how to solve this problem, thank
Repository: go
Issue: x/text: p.Sprint outputs key + fallback
Labels: NeedsInvestigation

Please answer these questions before submitting your issue. Thanks!


### What version of Go are you using (`go version`)?
1.11b2

### Does this issue reproduce with the latest release?
yes

### What operating system and processor architecture are you using (`go env`)?
linux/amd64

### What did you do?
```
T.Sprint(message.Key("login.form.label.password", "Password"))
```
### What did you expect to see?
Password

### What did you see instead?
{login.form.label.password Password}

`T.Sprintf(message.Key("login.form.label.password", "Password"))` returns the expected output, however it's use is not necessary in many such cases, where there is no parameters for the string even the lint tool is showing warnings in such cases.
Repository: go
Issue: dl: golang.org/dl installer shouldn't create a new subdirectory of $HOME
Labels: Builders,NeedsInvestigation

@andybons sent an email to golang-dev yesterday, subject 'Go 1.11 Beta 2 is Released'.
The instructions included

    If you have Go installed already, the easiest way to try go1.11beta2
    is by installing it using the go command:
    $ go get https://golang.org/dl/go1.11beta2
    $ go1.11beta2 download

These fail with Arch Linux's go 2:1.10.3-1 package.

    $ go get https://golang.org/dl/go1.11beta2
    package https:/golang.org/dl/go1.11beta2: "https://" not allowed in import path
    $

Removing the `https://' worked.  It then matches the comment in https://go.googlesource.com/dl/+/master/go1.11beta2/main.go

Please consider warning that 'go1.11beta2 download' will install to
$HOME/sdk/go1.11beta2 as some folks, me included, won't appreciate new
top-level $HOME directories.  Altering $HOME for the run of 'go1.11beta2
download' avoided this on Unix, and it looks like similar is possible on other
platforms.

(Opened issue as requested by Andy in private email.)
Repository: opencv
Issue: Opencv 3.4.2 broke waitkey
Labels: bug,category: highgui-gui

<!--
If you have a question rather than reporting a bug please go to http://answers.opencv.org where you get much faster responses.
If you need further assistance please read [How To Contribute](https://github.com/opencv/opencv/wiki/How_to_contribute).

This is a template helping you to create an issue which can be processed as quickly as possible. This is the bug reporting section for the OpenCV library.
-->

##### System information (version)
<!-- Example
- OpenCV => 3.4.2
- Operating System / Platform => Antergos/X86_64
- Compiler => G++ 8.1.1
-->

- OpenCV => 3.4.2
- Operating System / Platform => Antergos/X86_64
- Compiler => G++ 8.1.1

##### Detailed description

Opencv 3.4.2 broke waitkey, producing SIGFPE randomly.
Opencv 3.4.1 does not.
P.S.
The PKGBUILD can be found here. 
https://git.archlinux.org/svntogit/packages.git/tree/trunk?h=packages/opencv

##### Steps to reproduce

<!-- to add code example fence it with triple backticks and optional file extension
    ```.cpp
    // C++ code example
    ```
 or attach as .txt or .zip file


-->
Build the following code with release mode will trigger the bug really quick on my PC.
Didn't try debug mode....
[bugtest2.zip](https://github.com/opencv/opencv/files/2216080/bugtest2.zip)
Repository: godot
Issue: Comment node in visual scripting does not properly update size
Labels: bug,confirmed,topic:visualscript

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
Godot 3.1 dev

**OS/device including version:**
Win 10 64 bit, AMD GPU


**Issue description:**
When changing size of the comment node in visual script canvas, the size in the inspector is not updated and if you change size on the canvas and then immediately chang it in the inspector, the size will snap back to previous value before the canvas size manipulation.

You need to change size of the comment node on the canvas, then deselect node, reselect it and the inspector size will be updated. Changing size via inspector then works as intended.

**Steps to reproduce:**
Place comment node in the canvas, change it's size using node corner drag gizmo and then change the value in the inspector size.
Repository: pytorch
Issue: Remove BatchNorm layers once the training is completed.
Labels: feature,module: nn,triaged,module: norms and normalization

In PyTorch, is there any way to remove the batchnorm layers by merging their parameters into their preceding convolutional layers?

ps: If you are familiar with the model deployment function in MatConvNet, you should know what I mean :)

cc @albanD @mruberry @jbschlosser