sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: x/net/http2: data race on DebugGoroutines
Labels: Testing

This is an issue of http2 package-specific debug hook, so not critical.

```
go test -race golang.org/x/net/http2 -count 10

==================
WARNING: DATA RACE
Write at 0x000000705ee2 by goroutine 236:
  golang.org/x/net/http2.TestGoroutineLock()
      /swdev/src/golang.org/x/net/http2/gotrack_test.go:15 +0x82
  testing.tRunner()
      /go/src/testing/testing.go:610 +0xc9

Previous read at 0x000000705ee2 by goroutine 147:
  golang.org/x/net/http2.goroutineLock.check()
      /swdev/src/golang.org/x/net/http2/gotrack.go:32 +0x3e
  golang.org/x/net/http2.(*serverConn).stopShutdownTimer()
      /swdev/src/golang.org/x/net/http2/server.go:651 +0x58
  golang.org/x/net/http2.(*serverConn).serve()
      /swdev/src/golang.org/x/net/http2/server.go:722 +0x8c8
  golang.org/x/net/http2.(*Server).ServeConn()
      /swdev/src/golang.org/x/net/http2/server.go:339 +0xb70
  golang.org/x/net/http2.ConfigureServer.func1()
      /swdev/src/golang.org/x/net/http2/server.go:200 +0xe7
  net/http.(*conn).serve()
      /go/src/net/http/server.go:1493 +0x12ab

Goroutine 236 (running) created at:
  testing.(*T).Run()
      /go/src/testing/testing.go:646 +0x52f
  testing.RunTests.func1()
      /go/src/testing/testing.go:793 +0xb9
  testing.tRunner()
      /go/src/testing/testing.go:610 +0xc9
  testing.RunTests()
      /go/src/testing/testing.go:799 +0x4b5
  testing.(*M).Run()
      /go/src/testing/testing.go:743 +0x12f
  main.main()
      golang.org/x/net/http2/_test/_testmain.go:470 +0x1b4

Goroutine 147 (finished) created at:
  net/http.(*Server).Serve()
      /go/src/net/http/server.go:2240 +0x53e
  net/http/httptest.(*Server).goServe.func1()
      /go/src/net/http/httptest/server.go:235 +0xa2
==================
PASS
```
Repository: go
Issue: cmd/compile: escape analysis of interface calls to non-exported method names
Labels: compiler/runtime

Consider this Go source:

```
package p

import "fmt"

type A int
func (A) f(x *int) { fmt.Println("A", *x) }

type B int
func (B) f(x *int) { fmt.Println("B", *x) }
func (B) g() {}

type Fer interface { f(x *int); g() }

func H(f Fer) {
    i := 42
    f.f(&i)
}
```

At the f.f(&i) call site in H, we can't be sure exactly what method will be called.  However, since it's an unexported method name, it must be implemented by a method in the same package.  Consequently, if we know that all candidate methods in the package are non-escaping (such as A.f and B.f are), we can still be sure that f.f(&i) is a non-escaping call.

Note that we still have to consider A.f even though A itself doesn't implement Fer, because a downstream package could do something like:

```
package q

import "p"

type X struct { p.B }
type Y struct { p.A; X }

func Z() {
    p.H(Y{})
}
```

There are probably refinements that could avoid this though.

I happened to notice this because of package net's "exchange" function, which calls the unexported TCPConn.dnsRoundTrip or UDPConn.dnsRoundTrip methods.  I haven't investigated how common this situation is though.

/cc @dr2chase @alandonovan
Repository: neovim
Issue: :normal ex command should work in terminal mode
Labels: bug,terminal

- `nvim --version`: `NVIM v0.1.5-385-g999590b`
- Operating system/version: OS X /  10.11.5
- Terminal name/version: iTerm2 / Build 3.0.20160607-nightly
- `$TERM`: xterm-256color
- Actual behaviour:
  `execute "normal! gt"` have no effect when run from a terminal, if a terminal is active.
- Expected behaviour:
  `execute "normal! gt"`  should switch tabs
### Steps to reproduce using `nvim -u minimal_vimrc`
1. `nvim -u mini_vimrc`
2. `:edit foo<cr>`
3. `:tabedit bar<cr>`
4. `:sp | term<cr>`
5. `Ctrl-\ Ctrl\n :Tabs<cr>`
6. Select `Tab page 1` and press Enter
7. You are not moved to the first tab.
8. Now type `:messages<cr>` to see the command that was executed: `normal! 1gt`.

It works, If you do the same from another window that is not bound to a terminal buffer !!!!

Here is a [asciicast](https://asciinema.org/a/692o9i8u5e9g3al4qj8fz11mx).

Here is a mini_vimrc:

``` vim
call plug#begin('~/.config/nvim/plugged')

set rtp+=/usr/local/opt/fzf
Plug 'junegunn/fzf.vim'

function! s:tablist()
  redir => tabs
  silent tabs
  redir END
  return filter(split(tabs, '\n'), 'v:val =~ "Tab page"')
endfunction

function! s:tabopen(e)
  "Show what is being executed
  echomsg 'normal! '. matchstr(a:e, 'Tab page \zs[0-9]*\ze.*$').'gt'
  execute 'normal! '. matchstr(a:e, 'Tab page \zs[0-9]*\ze.*$').'gt'
endfunction

command Tabs call fzf#run({
      \   'source':  reverse(<sid>tablist()),
      \   'sink':    function('<sid>tabopen'),
      \   'options': '',
      \   'down':    len(<sid>tablist()) + 2
      \ })

call plug#end()
```
Repository: youtube-dl
Issue: hls process
Labels: bug

### Make sure you are using the _latest_ version: run `youtube-dl --version` and ensure your version is _2016.06.03_. 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.06.03**
### 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

---

Hi,

I've seen two problems with current hls download process:
1. With --hls-use-mpegts, ffmpeg must be used in all cases because native doesn't support it:
   At the moment, it only works if combination of protocol and hls-prefer options gives ffmpeg.
2. m3u8 fixup doesn't need to be done when ffmpeg is used:
   At the moment, it's also done with m3u8_native and --hls-prefer-ffmpeg.
Repository: rust
Issue: slice::sort_by_key has more restrictions than slice::sort_by
Labels: T-libs-api,C-bug

I expected that these invocations of `sort_by` and `sort_by_key` would be equivalent:

``` rust
struct Client(String);

impl Client {
    fn key(&self) -> &str {
        &self.0
    }
}

fn main() {
    let mut clients: Vec<Client> = vec![];

    // Error: cannot infer an appropriate lifetime for autoref due to conflicting requirements
    clients.sort_by_key(|c| c.key());

    // OK
    clients.sort_by(|a, b| a.key().cmp(&b.key()));
}
```

The [implementation of `sort_by_key`](https://github.com/rust-lang/rust/blob/d5759a3417fa395d439f4283825504dd4f78dc87/src/libcollections/slice.rs#L818-L822):

``` rust
pub fn sort_by_key<B, F>(&mut self, mut f: F)
    where F: FnMut(&T) -> B, B: Ord
{
    self.sort_by(|a, b| f(a).cmp(&f(b)))
}
```

An initial attempt at using HRTB didn't seem to pan out:

``` rust
pub fn sort_by_key<B, F>(&mut self, mut f: F)
    where for <'a> F: FnMut(&'a T) -> B + 'a,
          B: Ord
```
Repository: flutter
Issue: Should cross-fade the paginated data table header when it changes
Labels: c: new feature,framework,f: material design,P3,team-design,triaged-design

If you select something, we replace the header with a count of selected items. We should do that with an animation that cross-fades from the previous text, or something. Right now it feels a bit sudden.
Repository: opencv
Issue: Better to have an option for Python3 or Python2?
Labels: feature,priority: low,category: build/install

### Please state the information for your system
- OpenCV version: 3.x (git)
- Host OS: Linux (Ubuntu 16.04) 
- GCC 5.3.1 & CMake 3.5.1
### In which part of the OpenCV library you got the issue?

Python
### Expected behaviour

While configure, I can see

>  PYTHON2_EXECUTABLE               /usr/bin/python2.7  
>  PYTHON2_INCLUDE_DIR              /usr/include/python2.7  
>  PYTHON2_INCLUDE_DIR2  
>  PYTHON2_LIBRARY                  /usr/lib/x86_64-linux-gnu/libpython2.7.so  
>  PYTHON2_LIBRARY_DEBUG  
>  PYTHON2_NUMPY_INCLUDE_DIRS  
>  PYTHON2_PACKAGES_PATH            lib/python2.7/dist-packages  
>  PYTHON3_EXECUTABLE               /usr/bin/python3  
>  PYTHON3_INCLUDE_DIR              /usr/include/python3.5m  
>  PYTHON3_INCLUDE_DIR2  
>  PYTHON3_LIBRARY                  /usr/lib/x86_64-linux-gnu/libpython3.5m.so  
>  PYTHON3_LIBRARY_DEBUG  
>  PYTHON3_NUMPY_INCLUDE_DIRS       /usr/lib/python3/dist-packages/numpy/core/include  
>  PYTHON3_PACKAGES_PATH            lib/python3.5/dist-packages    

But, there is **NO** option for me to choose which version of python that I'm going to use for OpenCV build.
### Actual behaviour

I prefer using Python 3.5m, but finally, it always goes to Python 2.7 .

Any suggestions if I can specify which version of python I would like to use for compilation?

Cheers
Pei
Repository: youtube-dl
Issue: Feature Request - Remove Videos from YouTube Playlist after Download
Labels: request

### What is the purpose of your _issue_?
- [ ] Bug report (encountered problems with youtube-dl)
- [ ] Site support request (request for adding support for a new site)
- [X] Feature request (request for a new functionality)
- [X] Question
- [ ] Other

---
### Feature Request - Remove Videos from YouTube Playlist after Download

I would like to add a option to remove a video from a YouTube playlist after it is downloaded.
I believe this is possible as the YouTube API has a python section for this.
https://developers.google.com/youtube/1.0/developers_guide_python#adding-a-playlist

I have managed to add the option to the options list as "--remove-watched", but can't seem to call the method it points at.
I admit I am new to python but have coded in several other languages before so if someone could point me in the right direction I will do my best to figure it out.

Thanks,
Repository: youtube-dl
Issue: [eclassesbyravindra] Site support request
Labels: site-support-request,account-needed

C:\chirag\Videos\Gate Ravi Lectures\CN Problems>youtube-dl.exe http://eclassesby
ravindra.com/enrol/index.php?id=24/
[generic] index: Requesting header
[redirect] Following redirect to http://eclassesbyravindra.com/login/index.php
[generic] index: Requesting header
WARNING: Falling back on generic information extractor.
[generic] index: Downloading webpage
[generic] index: Extracting information
ERROR: Unsupported URL: http://eclassesbyravindra.com/login/index.php

It gets redirect to another mostly previous url in normal browsing and does not download the videos.
Repository: angular
Issue: Control Validators should be observable chains
Labels: feature,state: Needs Design,area: forms,feature: under consideration

- **I'm submitting a ...**
- [X] feature request

**Current behavior**
Validators are callback functions

**Expected/desired behavior**
Validators should be observables.
- **What is the motivation / use case for changing the behavior?**

Currently it is very counter intuitive to build debounced async validators (http://plnkr.co/edit/2NuNj1lBj2e6OdUJjFra?p=preview). It would be much cleaner to work with an observable chain that is subscribed once, instead of subscribed for every value change of a control.

It could look like this:

```
new Control('',validator$=>{
    return validator$.map(valid=>{
        valid.errors.minLen=valid.value.length>5?null,"Length must be at least 5";
        return valid;
    });
});
```
Repository: go
Issue: x/mobile/font: Default font fail on MacOS and iOS
Labels: mobile

Hello,
1. What version of Go are you using (`go version`)?
   I'm using Go version 1.6.2 and Gomobile version +c435d0b
2. What operating system and processor architecture are you using (`go env`)?
   I'm on MacOS (darwin / amd64)
3. What did you do?
   I'm trying to use the Default system font on MacOS and iOS (gomobile).
   The complete program can be found here: https://github.com/antoine-richard/gomobile-text
   See this line: https://github.com/antoine-richard/gomobile-text/blob/master/game.go#L47
4. What did you expect to see?
   I expect to see the Default font being used.
5. What did you see instead?
   I'm getting the following error: `freetype: invalid TrueType format: bad maxp length: 6` on both MacOS and iOS when using the Default font.
   The monospace font works on these OSes. 
   The default font works on Android.

I've found 2 issues mentioning the same kind of problem: https://github.com/golang/go/issues/10644 and https://github.com/golang/go/issues/10136

Also the test on default font fails on MacOS:

```
--- FAIL: TestLoadFonts (0.02s)
    font_test.go:41: default font: not a TTF: missing "glyf" table
FAIL
exit status 1
FAIL    golang.org/x/mobile/exp/font    0.035s
```

Is there a way to use the default font on Darwin?
Is there any known workaround?

Thank you
Antoine
Repository: go
Issue: x/net/http2: deal with half-closed remote in Transport, flaky TestTransportResPattern_c0h1d0t0, TestTransportResPattern_c1h2d0t0
Labels: Testing

See https://build.golang.org/log/6c16a5f66e28811897fa9ff2818873f56a9a128c and https://build.golang.org/log/5ea1f443a67a371b9acdebeee89cc802345ce5fb.

```
--- FAIL: TestTransportResPattern_c0h1d1t0 (0.00s)
    transport_test.go:664: client: RoundTrip: http2: stream closed
FAIL
FAIL    golang.org/x/net/http2  7.585s
```
Repository: vscode
Issue: Implement editor undo stack serialization
Labels: feature-request,editor-textbuffer,undo-redo

Related to #7169, #7535

Similarly to the view state, we need a way to serialize, and later deserialize, a model's undo stack.
Repository: TypeScript
Issue: Find all references for ambient module
Labels: Suggestion,Help Wanted,API

```
// declarations.d.ts
declare module "x" {
    export const x: number;
}

// index.ts
import {x} from "x";
```

Goto-declaration for "x" in index.ts works, but find-all-references for "x" in declarations.d.ts finds only itself.

In this case one can simply look through each of export in the module's body and find all of their references; but for shorthand ambient modules (`declare module "x";`) that won't work, as there is no module body.
Repository: youtube-dl
Issue: Request: Extract more (course) metadata for Lynda
Labels: request

Categories can be found as 'subjects.' Courses have their own description, along with other metadata such as 'view count' and 'creator,' which currently aren't extracted.
Repository: electron
Issue: macOS dynamic menu items (that change when the user holds down a modifier)
Labels: enhancement :sparkles:,platform/macOS,component/menu

> A dynamic menu item is a command that changes when the user presses a modifier key.

https://developer.apple.com/library/mac/documentation/UserExperience/Conceptual/OSXHIGuidelines/MenuChanging.html
- Operating system: OS X
Repository: TypeScript
Issue: Symlinks not resolved for `/// <reference path="..." />`
Labels: Bug

**TypeScript Version:** 

nightly (1.9.0-dev)

**Code**

``` shell
$ ls -alF
 a/
 project/
 symlink -> a/
$ cat a/one.ts 
import './zone';
$ cat a/zone.d.ts
declare type ZoneType = {a:string};
declare const Zone: ZoneType;
$ cat project/tsconfig.json
{
    "compilerOptions": {
        "module": "commonjs",
        "target": "es5",
        "noEmit": true,
        "traceResolution": true
    }
}
$ cat project/deps.d.ts
///<reference path="../symlink/zone.d.ts"/>
///<reference path="../symlink/one.ts"/>
```

**Expected behavior:**
There are two ways the program may import `zone.d.ts`, either by resolving symlinks to `../a/zone` or by using the path provided `../symlink/zone`. In order to treat these as the same file, TypeScript needs to use the same path internally.

**Actual behavior:** 
One case resolves the symlink and the other does not, causing the declarations in the file to conflict with themselves:

```
$ tsc -p project
======== Resolving module './zone' from '/usr/local/google/home/alexeagle/repro_pathmapping_zone/symlink/one.ts'. ========
Module resolution kind is not specified, using 'NodeJs'.
Loading module as file / folder, candidate module location '/usr/local/google/home/alexeagle/repro_pathmapping_zone/symlink/zone'.
File '/usr/local/google/home/alexeagle/repro_pathmapping_zone/symlink/zone.ts' does not exist.
File '/usr/local/google/home/alexeagle/repro_pathmapping_zone/symlink/zone.tsx' does not exist.
File '/usr/local/google/home/alexeagle/repro_pathmapping_zone/symlink/zone.d.ts' exist - use it as a name resolution result.
Resolving real path for '/usr/local/google/home/alexeagle/repro_pathmapping_zone/symlink/zone.d.ts', result '/usr/local/google/home/alexeagle/repro_pathmapping_zone/a/zone.d.ts'
======== Module name './zone' was successfully resolved to '/usr/local/google/home/alexeagle/repro_pathmapping_zone/a/zone.d.ts'. ========
a/zone.d.ts(1,14): error TS2300: Duplicate identifier 'ZoneType'.
a/zone.d.ts(2,15): error TS2451: Cannot redeclare block-scoped variable 'Zone'.
symlink/zone.d.ts(1,14): error TS2300: Duplicate identifier 'ZoneType'.
symlink/zone.d.ts(2,15): error TS2451: Cannot redeclare block-scoped variable 'Zone'.
```

with @vikerman
Repository: youtube-dl
Issue: Feature Request - Add to --match-filter keys one to identify the categorie of the YouTube Video.
Labels: request

### What is the purpose of your _issue_?
- [ ] Bug report (encountered problems with youtube-dl)
- [ ] Site support request (request for adding support for a new site)
- [x] Feature request (request for a new functionality)
- [x] Question
- [ ] Other
### Feature Request - Add to `--match-filter` keys one to identify the categorie of the YouTube Video.

For example only allow videos from the MUSIC category, I was thinking something like that:

`--match-filter "categorie = Music"`

or similar. Thanks for your work!
Repository: youtube-dl
Issue: Download only Audio bytes 
Labels: request

I want to download from YouTube the best Audio Quality possible.
Since the Audio Only files are at low bitrate (usually 128k), i have to download the whole Audio+Video (format 22 usually has the best audio), and then extract the audio.

But i don't want to download the entire video file to only get the audio.

I know it is possible to download only Audio bytes by parsing the format header and skip all video frames to save bandwidth.

For example 4K Video Downloader does this with YouTube to get the best Audio quality from a video.
It parses the MP4 header, and download only the audio bytes.

it is possible to implement this functionality?
Repository: opencv
Issue: crashes, when OutputArrayOfArrays is used with type inherited from cv::Point2f
Labels: bug,category: core

- OpenCV version: 3.1
- Host OS: Linux (Ubuntu 14.04)
- aruco
### Expected behaviour

 application detects markers and saves points information correctly
### Actual behaviour

 application crashes
### Code example to reproduce the issue / Steps to reproduce the issue

```
/* this works fine if points is of type: vector<vector<cv::Point2f>>; */
cv::Ptr<aruco::Dictionary> dict=aruco::getPredefinedDictionary(aruco::DICT_4X4_50);
aruco::detectMarkers(inputImage, dict, points, marIds);

let's create our own type for storing points:
class Vector2f : public cv::Point2f {
public:
    Vector2f() : cv::Point2f(){}
    Vector2f(float newX, float newY): cv::Point2f(newX, newY){}
    float length(); //implementation not meaningfull
}; 

now sample code:

/* this does not work if points is of type vector<vector<Vector2f>>; */
cv::Ptr<aruco::Dictionary> dict=aruco::getPredefinedDictionary(aruco::DICT_4X4_50);
aruco::detectMarkers(inputImage, dict, points, marIds);

```

Application crashes with exception. I used to work more with opencv some time ago I remember, that sometimes you are doing some low level metaprogramming stuff and hardcode things for certain types defined by you. This might be an issue.
Repository: rust
Issue: Lint against transmutes which could be `as` casts.
Labels: A-lints,C-feature-request

The relevant casts are at least those between pointers and integers, or between two pointers.
There's probably also some likelihood of same-type transmutes arising, which keep working after changing the types (of FFI signatures, for example) but could be removed because they're noops.

Clippy might be interested, if they don't already have this (cc @Manishearth).
Repository: TypeScript
Issue: Triple slash reference should include extension '.d.ts' or not?
Labels: Bug,Help Wanted,VS Code Tracked

_From @frogcjn on June 13, 2016 6:6_

`/// <reference path='../../../typings/index.d.ts' />`

or 

`/// <reference path='../../../typings/index' />`

?

Thanks!

tsc recognizes the file without .d.ts extension, but vscode's ts validation has a different behavior.
This makes a headache communication between typescript coder.

<img width="355" alt="2016-06-13 02 25 22" src="https://cloud.githubusercontent.com/assets/1777562/15998322/fa52bdb2-316f-11e6-84a8-6de7cc0316dc.png">

_Copied from original issue: Microsoft/vscode#7594_
Repository: kubernetes
Issue: kubectl run -it loses some lines of the log
Labels: kind/bug,priority/backlog,area/kubectl,sig/node,sig/cli,triage/accepted

When running a command via `kubectl run -it` that immediately prints something, we might lose some lines of the log due to a race of the execution of the container and the `kubectl attach` used by `kubectl run` to attach to the terminal (compare comment https://github.com/kubernetes/kubernetes/issues/16670#issuecomment-199017681).

As a workaround for running an interactive shell we have this ugly `Hit enter for command prompt` now. The actual root problem though is that `docker attach` does not replay the logs. It looks like `docker run` is the only way to actually synchronously run a command, get all the output and attach to the process.
Repository: TypeScript
Issue: Module augmentation with export list
Labels: Bug

**TypeScript Version:** 

1.8.10

**Code**

``` ts
// scale.ts
class Scale {
    weightOnEarth(mass) {}
}
export { Scale }
```

``` ts
// advancedScale.ts
import {Scale} from "./scale" ;

declare module "./scale" {
  interface Scale {
    weightOnMoon(mass);
  }
}
Scale.prototype.weightOnMoon = function (ee) {
}
```

``` ts
// consumer.ts
import { Scale } from "./scale";
import "./advancedScale";

let scale: Scale;
scale.weightOnMoon(10);  // ok
scale.weightOnEarth(10);
```

```
// tsconfig.json
{
  "compilerOptions": {
    "module": "commonjs",
    "target": "es5"
  }
}
```
# Bug 1

This code is from [this post](https://blogs.msdn.microsoft.com/typescript/2016/02/22/announcing-typescript-1-8-2/), with just a small change: in `scale.ts`, the syntax of export list is used `export { Scale }`.

The compiler returns errors:

> scale.ts(5,10): error TS2484: Export declaration conflicts with exported declaration of 'Scale'
> advancedScale.ts(9,1): error TS2304: Cannot find name 'Scale'.
> consumer.ts(7,7): error TS2339: Property 'weightOnEarth' does not exist on type 'Scale'.
# Bug 2

With the original code, in `consumer.ts`, the line `import "./advancedScale";` can be removed but the code still compiles.