sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: godot
Issue: 3D KinematicBody is not correctly detecting collision with PlaneShape
Labels: bug,confirmed,topic:physics

**Operating system or device - Godot version:**
Windows 10 64bit - Godot 2.1 RC1

**Issue description** (what happened, and what was expected):
The KinematicBody `is_colliding()` function does not return true when the body is colliding with an object that has a PlaneShape collision shape. It does still get stopped by the object, but doesn't report the collision.

**Steps to reproduce:**
Create a StaticBody with a PlaneShape collision. Create a KinematicBody with a simple BoxShape collision, and add a script to it to move into the StaticBody. Print `is_colliding()`
Repository: go
Issue: encoding/xml: panics with Field index out of range when XMLName is in embedded struct
Labels: NeedsInvestigation,early-in-cycle

Please answer these questions before submitting your issue. Thanks!
1. What version of Go are you using (`go version`)?

`go version devel +67f799c Tue Jul 26 00:18:42 2016 +0000 linux/amd64`

But it also affects the version of 1.6 on the playground at the moment.
1. What operating system and processor architecture are you using (`go env`)?

```
GOARCH="amd64"
GOBIN="/home/sam/bin"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/sam"
GORACE=""
GOROOT="/home/sam/src/go"
GOTOOLDIR="/home/sam/src/go/pkg/tool/linux_amd64"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build800221808=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
```
1. What did you do?

When attempting to unmarshal XML into a struct that is composed of another struct that has an XMLName field that has a field number greater than or equal to the number of fields in the original struct, `finfo.Value` panics because reflect tries to get the value of the field in the parent struct at the location of `XMLName` in the child struct (which has a field index that is out of bounds when applied to the parent struct).

For example, this contrived example panics:

``` go
package main

import (
    "encoding/xml"
)

type IQ struct {
    Type    string   `xml:"type,attr"`
    XMLName xml.Name `xml:"iq"`
}

func main() {
    resp := struct {
        IQ
    }{}
    xml.Unmarshal([]byte(`<iq/>`), &resp)
}
```

([Playground](https://play.golang.org/p/XY2iCSSduO))

but if we move the fields around:

``` go
type IQ struct {
    XMLName xml.Name `xml:"iq"`
    Type    string   `xml:"type,attr"`
}
```

it works ([Playground](https://play.golang.org/p/EUMfU7TFyi))
Repository: go
Issue: x/net/http2: Server doesn't respect client's flow control
Labels: NeedsFix

Go's http2.Server doesn't respect the client's flow control.

It accounts for it partly, and enforces the other direction (that the client can't send too much to the server), but does not ever deduct its own credit when sending DATA frames, and doesn't wait for flow control when doing so.

In the process of fixing #16481 I had expected a test like this to hang:

``` go
        body := strings.Repeat("a", 16<<10)
        st := newServerTester(t, func(w http.ResponseWriter, r *http.Request) {
                println(r.RemoteAddr)
                // Write out a 16KB body which the client won't ever read.                                                                                                   
                io.WriteString(w, body)
        }, optOnlyServer)
        defer st.Close()

        tr := &Transport{TLSClientConfig: tlsConfigInsecure}
        defer tr.CloseIdleConnections()

        // Verify we can do this over 4 times without hanging:                                                                                                               
        for i := 0; i < 10; i++ {
                println(i)
                req, _ := http.NewRequest("GET", st.ts.URL, nil)
                res, err := tr.RoundTrip(req)
                if err != nil {
                        t.Fatal(err)
                }
                res.Body.Close()
                println(i, "done")
        }
```

But it didn't.

Add a test like that back once the http2.Server respects flow control.
Repository: opencv
Issue: OpenCV Stereo SGBM release/debug results differ
Labels: bug,category: calib3d,affected: 3.4

<!--
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)
- OpenCV => 3.1
- Operating System / Platform => Windows 7 64 Bit
- Compiler => Visual Studio 2015
- CPU => Intel Core i7 3770 (MMX, SSE, SSE2 ..., AVX)
##### Detailed description

We observe different disparity maps calculated using 64 bit release and debug versions of the same example based on the SGBM stereo algorithm.

Thanks for your help!!

Silvano
##### Steps to reproduce

Stereo pipeline configuration is described in the provided example.

[opencv_sgbm_rel_dbg.zip](https://github.com/opencv/opencv/files/384020/opencv_sgbm_rel_dbg.zip)
Repository: opencv
Issue: crash at  cv::TiffDecoder::readData(cv::Mat&) 
Labels: bug,affected: 2.4,category: imgcodecs

based on opencv-2.4.10. my code was:

```
#include <opencv2/opencv.hpp>
#include <iostream>
#include <sys/time.h>

using namespace std;
using namespace cv;

int main (int argc, char **argv)
{
    cv::Mat img = cv::imread (argv[1], -1);
    cout << img.channels () << endl;
    return 0;
}
```

the input image was  http://img4y4.oss-cn-hangzhou.aliyuncs.com/imgs/201607/5790dc3534394.jpg .
the program crashed and core dumped.
```Program terminated with signal 11, Segmentation fault.
#0  0x00007f1fd15ab8dd in cv::TiffDecoder::readData(cv::Mat&) () from /usr/local/lib/libopencv_highgui.so.2.4```
Repository: youtube-dl
Issue: Embedded Videos Not Supported/Detected + Possible Solution
Labels: request

### Make sure you are using the _latest_ version: run `youtube-dl --version` and ensure your version is _2016.07.26.2_. If it's not read [this FAQ entry](https://github.com/rg3/youtube-dl/blob/master/README.md#how-do-i-update-youtube-dl) and update. Issues with outdated version will be rejected.
- [x] I've **verified** and **I assure** that I'm running youtube-dl **2016.07.26.2**
### Before submitting an _issue_ make sure you have:
- [x] At least skimmed through [README](https://github.com/rg3/youtube-dl/blob/master/README.md) and **most notably** [FAQ](https://github.com/rg3/youtube-dl#faq) and [BUGS](https://github.com/rg3/youtube-dl#bugs) sections
- [x] [Searched](https://github.com/rg3/youtube-dl/search?type=Issues) the bugtracker for similar issues including closed ones
### What is the purpose of your _issue_?
- [x] Bug report (encountered problems with youtube-dl)
- [ ] Site support request (request for adding support for a new site)
- [ ] Feature request (request for a new functionality)
- [ ] Question
- [ ] Other

---
## A bit of background

I'm an avid youtube-dl user and every once in a while I try to download a video.
It sometimes happens that the video is embedded from another site (similar to YouTube), but youtube-dl doesn't detect it.  This makes me have to dig through the source code to find the code for the embedded player and enter that into youtube-dl and this gets a bit annoying after 100 videos.  

Luckily pretty much all embedded YouTube players are supported, but others are not.
So my question is, why can't we detect those as well?  There is already a list of the supported sites so wouldn't it only be a matter of checking those domains in the source code and seeing if they're found in an attribute?
### So what the hell can we do?

One way is to manually find and add support for every single embed that youtube-dl currently supports - the other is to automate it like I'm going to describe below:

So let's say that I enter this page: http://unsupported-embed.com/video?id=123 and youtube-dl doesn't support it.
Youtube-dl would then find all the attributes containing URLs (e.g <iframe src="**http://free-video-embeds.com**"></iframe>), add them to an array, loop through the array and check in the supported site list to find the embed.  Once and if the embedded player is found it runs a new query for that URL to finally get the video.

Now this could be a bit heavy to run in default so we it could simply be added as a super useful parameter like `--extensive-embed-search`.
## Currently Supported Embed

[YouTube](http://www.webdevout.net/test?0XP&raw)
[Vimeo](http://www.webdevout.net/test?0XN&raw)
## Currently Unsupported Embed

[Twitter](https://dev.twitter.com/web/embedded-video)
[Yahoo](https://www.yahoo.com/tv/39-agents-of-s-h-i-1430049850908726.html)
[LiveLeak](http://www.webdevout.net/test?0XS&raw)
[Ebaum's World](http://www.webdevout.net/test?0XR)
[USANetwork](http://www.usanetwork.com/mrrobot/videos/rami-malek-talks-mr-robot-season20)
**...and many more**
Repository: TypeScript
Issue: Private/protected properties on siblings in union types
Labels: Suggestion,Help Wanted,Domain: Error Messages

**TypeScript Version:**  1.8.9

**Code**

``` ts
class Example {
    protected width = 0;
    constructor(sibling: Example | HTMLCanvasElement) {
        console.log(sibling.width);
    }
}
```

**Expected behavior:** No errors.

**Actual behavior:** Error: `Property 'width' does not exist on type 'Example | HTMLCanvasElement'. Line 4, column 31`

The reason this error occurs is that the `width` property is `protected` on `Example` but `public` on `HTMLCanvasElement`. However, the error is confusing for two reasons:
1. The `width` property is available on both component properties in this situation. If the `sibling` parameter had a type of either `Example` or `HTMLCanvasElement`, the code would compile.
2. The error message doesn't explain why the property does not exist on the type. Understanding the error requires the knowledge that properties are only available on union types if they have the same privacy level, even if those properties would be accessible in the component types in the active context.

I am not sure if there are situations in which the desired behavior is a compilation error when accessing a potentially protected property on a union type if that property is accessible on all component types. I can't think of any, and so I think the ideal situation would be that the compiler does not throw an error here. However, I think it would be an adequate solution to improve the error message.

(I apologize if this has been reported/discussed before. I searched but couldn't find anything exactly along these lines.)
Repository: godot
Issue: negative scale changed into rotation after a spatial node is translated using the 3d viewport gizmo
Labels: enhancement,topic:editor,usability,documentation

**Operating system or device - Godot version:**

2.1rc1 official win64
2.1beta official win64
2.0.4.1 stable official win64
2.0.2stable official win64

**Issue description** (what happened, and what was expected):

When you set a negative scale for a Spatial node and move it using the 3d viewport gizmo, the inspector set the scale positive and change the Rotation to -180° on some axis.

**Steps to reproduce:**

Create a new scene with a TestCube.

Set its scale to (-2, 1, 1) using the Inspector.

Translate it to (3, 0, 0) using the 3d viewport gizmo.

Look at the Inspector.
- Scale became ( 2, 1, 1 )
- Rotation became ( -0, 0, -180 )

On the other hand, if you set the Translation using the Inspector, there is no such behavior.

**Link to minimal example project** (optional but very welcome):
Repository: flutter
Issue: Material components: Ability to long-press a menu icon to open menu, follow finger
Labels: c: new feature,framework,f: material design,from: review,P2,team-design,triaged-design

A user should be able to long-press the three-dot menu icon in the upper right, and have the menu open. Then, the user should be able to move their finger down to an item and have the selection indicator follow their finger. All without lifting their finger.

This behavior also applies to drop-down buttons, as seen in Buttons, Dropdown.
Repository: flutter
Issue: Once a drawer is transitioning out, don't let it grab taps on its components
Labels: framework,f: material design,from: review,P2,team-design,triaged-design

A design review asked us for the following enhancement:

Once a user triggers a transition to remove the left drawer from the screen, that drawer's components (e.g. checkboxes on the drawer) should no longer grab any inputs or taps. The reasoning: once you've requested the drawer to be removed, you no longer want to interact with its components.
Repository: go
Issue: cmd/compile: print better error message when assigning to keywords
Labels: compiler/runtime

I saw a new programmer struggle with the following:

``` go
func main() {
    map := "f"
    fmt.Println(map)
}
```

which is an invalid program since `map` is a keyword.

However, the error being printed is currently:

```
prog.go:6: syntax error: unexpected :=, expecting [
prog.go:7: syntax error: unexpected ), expecting [
prog.go:8: syntax error: unexpected }, expecting ]
```

It should really print a more helpful error message:

```
prog.go:6: identifier cannot be a keyword: map
```

/cc @griesemer
Repository: go
Issue: reflect: NamedOf
Labels: NeedsFix,FeatureRequest,compiler/runtime

Consider adding a function to the reflect package for creating a new named type with a method set:

```
func NamedOf(t Type, name string, methods []Method) Type
```

(Broken out from #4146.)
Repository: go
Issue: x/crypto/ssh/agent: Support ssh.Signer for AddedKey
Labels: NeedsInvestigation

## 1. What version of Go are you using (`go version`)?

go1.7rc3
## 3. What did you do?

I want to use a `crypto.Signer` as a private key for an SSH Agent. This would allow using keys that are stored on an HSM or have other reasons that a full `*rsa.PrivateKey` is not available.  Creating an `ssh.Signer` from a `crypto.Signer` is provided by the [ssh.NewSignerFromSigner](https://godoc.org/golang.org/x/crypto/ssh#NewSignerFromSigner) function, but because of the `agent.Agent` interface, they are not possible to add to the SSH Agent.

This is likely a change needed for #16304.
## 4. What did you expect to see?

https://godoc.org/golang.org/x/crypto/ssh/agent#AddedKey

The `agent.AddedKey` struct is used in the `agent.Agent` interface, through the `Add()` method.

`AddedKey` asks for a Private Key (using `interface{}`), and optionally an SSH Certificate `*ssh.Certificate`. 

The `keyring` implementation of Add() then converts these into an [`ssh.Signer` interface](https://godoc.org/golang.org/x/crypto/ssh#Signer) : https://github.com/golang/crypto/blob/bc89c496413265e715159bdc8478ee9a92fdc265/ssh/agent/keyring.go#L127-L152

I believe any other implementations of the `agent.Agent` interface also follow this pattern, because the other methods of the interface require `ssh.Signer` as a return value.

Happy to submit a CL if there is agreement on how to best do this.
Repository: go
Issue: cmd/trace: execution tracer is essentially not documented
Labels: Documentation,NeedsInvestigation,compiler/runtime

I was able to find only 3 short mentions:
- [Command trace](https://golang.org/cmd/trace/)
- [Package trace](https://golang.org/pkg/runtime/trace/)
- [Go 1.5 changelog](https://golang.org/doc/go1.5#trace_command)

They are not very helpful. Compare this to how race detector ([article](https://golang.org/doc/articles/race_detector.html), [blog post](https://blog.golang.org/race-detector)) or pprof ([blog post](https://blog.golang.org/profiling-go-programs)) is documented. People on Reddit even think that [trace tool flopped](https://www.reddit.com/r/golang/comments/3isl6n/is_there_a_go_15_trace_tool_tutorial/).
#11605 mentions future blog post, but it never materialised.
Repository: rust
Issue: Rust should allow specifying a custom .def file
Labels: A-linkage,O-windows,T-compiler,C-feature-request

… on Windows. Especially for `cdylib`s.
Repository: opencv
Issue:  camera_calibration app fails fisheye calibration
Labels: bug,category: calib3d,affected: 3.4

Note I've accidentaly posted this as [issue #742](https://github.com/opencv/opencv_contrib/issues/742) in opencv_contrib. I've already closed it. Sorry for the noise.
##### System information
- OpenCV => 3.1 https://github.com/opencv/opencv/commit/8cdd91cd236f88915eb34658527f511f8a321e86 
- Operating System / Platform => Ubuntu Linux 14.04.4 x64
- Compiler => `gcc (Ubuntu/Linaro 4.7.3-12ubuntu1) 4.7.3`
##### Detailed description

opencv offers a fisheye camera model. Currently the only in-tree sample application that allows camera calibration for these cameras is [samples/cpp/tutorial_code/calib3d/camera_calibration/camera_calibration.cpp](https://github.com/opencv/opencv/blob/master/samples/cpp/tutorial_code/calib3d/camera_calibration/camera_calibration.cpp).

Although it has a (undocumented) configuration option for the fisheye model, calibration fails to return sensible
results with at least one data-set (see below). 

`cv::fisheye` is still in active development and subject to [some,](http://answers.opencv.org/question/87904/camera_calibrationcpp-fisheye-calibration-aborted/) [headache](http://answers.opencv.org/question/80255/unknow-exception-when-using-fisheyecalibrate-function/), and issues #5389

With the recent interest in VR, AR, the use if `cv::fisheye`, will increase. As camera_calibration.cpp is 
part of the tutorials, it is the go-to place for newcomers. Having a working reference example for fisheye
calibration in-tree would be beneficial for both users, and development.
##### Steps to reproduce
- build opencv with samples (`cmake -DBUILD_EXAMPLES=1`)
- download this example [data-set](https://github.com/derlunz/opencv-fisheye-calibration-set)
- call `BUILD_PATH/bin/cpp-tutorial-camera_calibration config-fisheye.xml`
- examine the undistorted image view at the end of the calibration process
- examine the resulting file `out_camera_fisheye_data.xml`
- the resulting camera parameters are clearly bogus, as can be seen, at the end of the calibration process.
  - lens distortion parameters returned by `cv::fisheye::calibrate` are all 0.0
  - estimated camera parameters look sensible 
  - undistorted images view at the end of the calibration process does not show a correctly undistorted images.
Repository: rust
Issue: Make LLVM better at using XMM registers to perform structure moves
Labels: A-LLVM,I-slow,C-enhancement,A-codegen,O-x86_64,I-compiletime,T-compiler,E-help-wanted,C-optimization,O-x86_32

Here's a small snippet of Servo code from block flow fragmentation:

```
servo[0x100bf974e] <+6638>:  mov    r10b, byte ptr [rbp - 0x729]
servo[0x100bf9755] <+6645>:  mov    r9, qword ptr [rbp - 0x728]
servo[0x100bf975c] <+6652>:  mov    rax, qword ptr [rbp - 0x720]
servo[0x100bf9763] <+6659>:  mov    rdi, qword ptr [rbp - 0x718]
servo[0x100bf976a] <+6666>:  mov    r8, qword ptr [rbp - 0x6c8]
servo[0x100bf9771] <+6673>:  mov    rdx, qword ptr [rbp - 0x710]
servo[0x100bf9778] <+6680>:  mov    qword ptr [rbp - 0x710], rdx
servo[0x100bf977f] <+6687>:  mov    qword ptr [r13 + 0x78], r8
servo[0x100bf9783] <+6691>:  mov    qword ptr [r13 + 0x80], rdi
servo[0x100bf978a] <+6698>:  mov    qword ptr [r13 + 0x88], rax
servo[0x100bf9791] <+6705>:  mov    qword ptr [r13 + 0x90], r9
servo[0x100bf9798] <+6712>:  mov    byte ptr [r13 + 0x98], r10b
servo[0x100bf979f] <+6719>:  mov    al, byte ptr [rbp - 0x34a]
servo[0x100bf97a5] <+6725>:  mov    byte ptr [r13 + 0x9f], al
servo[0x100bf97ac] <+6732>:  mov    ax, word ptr [rbp - 0x34c]
servo[0x100bf97b3] <+6739>:  mov    word ptr [r13 + 0x9d], ax
servo[0x100bf97bb] <+6747>:  mov    eax, dword ptr [rbp - 0x350]
servo[0x100bf97c1] <+6753>:  mov    dword ptr [r13 + 0x99], eax
servo[0x100bf97c8] <+6760>:  mov    eax, dword ptr [r13 + 0x15c]
servo[0x100bf97cf] <+6767>:  test   ah, 0x6
servo[0x100bf97d2] <+6770>:  je     0x100bf9949               ; <+7145>
```

I see this all over the place. It should be using XMM registers instead. This is bad because: (a) it clogs up the instruction stream; (b) it's an inefficient way to perform structure moves; (c) it kills tons of registers, resulting in spills elsewhere (notice rax, rdx, rdi, r8, r9, and r10 are all dead for no good reason); (d) it puts pressure on the register allocator, making compile times worse.

Is there some way to get LLVM to emit the right thing here?

cc @eddyb @brson
Repository: TypeScript
Issue: Protected nested classes
Labels: Suggestion,Awaiting More Feedback

**TypeScript Version:**  1.8.9

**Code**

``` ts
class Outer {
    protected static Inner = class {}
    private myInner: Outer.Inner;
}
```

**Expected behavior:** No errors.

**Actual behavior:** Compiler error: `Cannot find namespace 'Outer'. Line 3, Column 24`

**Code**

``` ts
class Outer {
    protected static Inner = class {}
    private myInner: Outer.Inner;
}
namespace Outer {}
```

**Expected behavior:** No errors.

**Actual behavior:** Compiler error: `Module 'Outer' has no exported member 'Inner'. Line 3, Column 32`

**Code**

``` ts
export class Outer {
    protected static Inner = class {};
}

export class Child extends Outer {
    static temp = class extends Outer.Inner {}
}
```

**Expected behavior:** No errors.

**Actual behavior:** Compiler error: `Property 'Inner' is protected and only accessible within class 'Outer' and its subclasses. Line 6, Column 35`

---

The problem here is that there is no way to define a protected nested class in a way that makes it possible to use in a type declaration. The third example shows why I want to use this: I want a nested class that can only be extended by classes that extend the parent class.
Repository: rust
Issue: Tracking issue for promoting `!` to a type (RFC 1216)
Labels: A-type-system,B-RFC-approved,T-lang,B-unstable,C-tracking-issue,finished-final-comment-period,F-never_type,Libs-Tracked,S-tracking-design-concerns,T-types

Tracking issue for rust-lang/rfcs#1216, which promotes `!` to a type.

### About tracking issues

Tracking issues are used to record the overall progress of implementation.  They are also used as hubs connecting to other relevant issues, e.g., bugs or open design questions.  A tracking issue is however *not* meant for large scale discussion, questions, or bug reports about a feature.  Instead, open a dedicated issue for the specific matter and add the relevant feature gate label.

## Pending issues to resolve

- [ ] Regressions from fallback to `()` https://github.com/rust-lang/rust/issues/67225
- [ ] How to handle fallback of diverging expressions https://github.com/rust-lang/rust/issues/65992

## Interesting events and links

- Initial PR #35162 
- Stabilization PR https://github.com/rust-lang/rust/pull/65355
- Reverting stabilization https://github.com/rust-lang/rust/pull/67224 due to regressions https://github.com/rust-lang/rust/issues/66757
Repository: You-Dont-Know-JS
Issue: This & prototype inheritance - Chapter 1 - Grammar issue
Labels: for second edition

> JavaScript's this mechanism isn't actually that advanced, but developers often paraphrase that quote in their own mind by inserting "complex" or "confusing", and there's no question that without lack of clear understanding, this can seem downright magical in your confusion.

Towards the end of the sentence the part that says, "without lack of clear understanding,..." is I think a sort of double negative that to me seems to mean WITH a clear understanding of JS, which wouldn't make sense. 

I think it should be either, "..without a clear understanding", or "..a lack of clear understanding can make this seem.."