sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: net: add mechanism to wait for readability on a TCPConn
Labels: NeedsInvestigation

**EDIT**: this proposal has shifted. See https://github.com/golang/go/issues/15735#issuecomment-266574151 below.

Old:

The net/http package needs a way to wait for readability on a TCPConn without actually reading from it. (See #15224)

http://golang.org/cl/22031 added such a mechanism, making Read(0 bytes) do a wait for readability, followed by returning (0, nil). But maybe that is strange. Windows already works like that, though. (See new tests in that CL)

Reconsider this for Go 1.8.

Maybe we could add a new method to TCPConn instead, like WaitRead.
Repository: youtube-dl
Issue: Display instantaneous download speed instead of average speed
Labels: request

- [x] I've **verified** and **I assure** that I'm running youtube-dl **2016.05.16**
- [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_?
- [ ] 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)
- [ ] Question
- [ ] Other

---

I have an incredibly unstable internet connection. My download speed will often fluctuate from being in the low kbps decades to the 5-10 mbps range. From what I've been able to see from my cursory search of the codebase, youtube-dl calculates speed as an average over the whole download process:

from https://github.com/rg3/youtube-dl/blob/f9b1529af8aec98bffd42edb5be15e1ada791a20/youtube_dl/downloader/common.py

```
@staticmethod
    def calc_speed(start, now, bytes):
        dif = now - start
        if bytes == 0 or dif < 0.001:  # One millisecond
            return None
        return float(bytes) / dif
```

Perhaps you should add a method to return the speed over the past few seconds rather than the whole download, so that wierd internet slowdowns don't show me wildly different average speed than the current speed.
Repository: go
Issue: cmd/go: only rebuild dependent packages when export data has changed
Labels: ToolSpeed,NeedsFix,GoCommand

@mwhudson suggested in #15734:

> If the export data hasn't changed, can you skip the compilation of dependent packages?

Moving this idea here to be tracked separately.

My 2c: Interesting idea, seems worth pursuing. Not sure how often it happens that code changes don't impact export data (or how cheap that is to detect), but when it does happen, that could save a _bunch_ of computation.
Repository: go
Issue: testing: complain loudly during concurrent use of T.FatalX and T.SkipX
Labels: help wanted,NeedsFix

The `testing` package is explicit about the following:

> A test ends when its Test function returns or calls any of the methods FailNow, Fatal, Fatalf, SkipNow, Skip, or Skipf. Those methods, as well as the Parallel method, must be called only from the goroutine running the Test function.

However, this is easy to overlook and it is not immediately obvious that the code below is a bad test:

``` go
func TestFoo(t *testing.T) {
    go func() {
        t.Fatal("fatal") // Called from outside the Test function
    }()
    time.Sleep(1 * time.Second)
}
```

This currently outputs (what a user expects):

```
--- FAIL: TestFoo (1.00s)
    foo_test.go:10: fatal
```

However, since t.Fatal is not safe to call outside of the Test function, this is poor feedback since it makes the user think that the test is correctly written when it is actually racy. Instead it should complain loudly that this is wrong.

As I'm debugging poor tests, I've noticed that the calling of t.Fatal in a goroutine is actually a fairly common phenomenon.

Related: #15674 

/cc @mpvl @bradfitz
Repository: go
Issue: x/build: monitor all the parts of the build system 
Labels: Builders

The watcher is not pushing new commits to build.golang.org.

I see:

```
  949 ?        Ssl    8:51 docker daemon --host=fd:// --selinux-enabled
17215 ?        Ssl   15:55  \_ /usr/local/bin/watcher -role=watcher -watcher.repo=https://go.googlesource.com/go -watcher.dash=https://build.golang.org/ -watcher.poll=10s -watcher.http=127.0.0.1:21536 -watcher.mirror=htt
 5570 ?        Z      0:00      \_ [git-remote-http] <defunct>
 5608 ?        Z      0:00      \_ [git-remote-http] <defunct>
 5618 ?        Z      0:00      \_ [git-remote-http] <defunct>
16843 ?        S      1:34      \_ git push -f --mirror dest
18371 ?        S      0:00      |   \_ git-remote-https dest https://gopherbot:[redacted]@github.com/golang/go
18178 ?        S      0:00      \_ git push -f --mirror dest
18389 ?        S      0:00      |   \_ git-remote-https dest https://gopherbot:[redacted]@github.com/golang/mobile
18335 ?        S      0:00      \_ git push -f --mirror dest
18387 ?        R      0:00          \_ git-remote-https dest https://gopherbot:[redacted]@github.com/golang/oauth2
```

And http://farmer.golang.org/debug/watcher at the end says only:

```
2016/05/19 18:43:40 go: sending commit to dashboard: ---40hexomitted---[master]("build: unset GOBIN during build")
2016/05/19 19:29:24 exp: found new commit ---40hexomitted---("shiny/widget/flex: add HTML printing of tests")
2016/05/19 19:29:24 exp: updated branch head: "master"(Head: ---40hexomitted---[master]("shiny/widget/flex: add HTML printing of tests") LastSeen: ---40hexomitted---[master]("shiny/driver/x11driver: have textureImpl.draw use an opaque mask."))
2016/05/19 19:29:24 exp: sending commit to dashboard: ---40hexomitted---[master]("shiny/widget/flex: add HTML printing of tests")
2016/05/19 19:36:46 exp: found new commit ---40hexomitted---("shiny/widget/flex: basics of flex algorithm")
2016/05/19 19:36:46 exp: updated branch head: "master"(Head: ---40hexomitted---[master]("shiny/widget/flex: basics of flex algorithm") LastSeen: ---40hexomitted---[master]("shiny/widget/flex: add HTML printing of tests"))
2016/05/19 19:36:46 exp: sending commit to dashboard: ---40hexomitted---[master]("shiny/widget/flex: basics of flex algorithm")
2016/05/19 22:45:58 exp: found new commit ---40hexomitted---("shiny/driver/x11driver: tighten the textureImpl.draw dst rectangle.")
2016/05/19 22:45:58 exp: updated branch head: "master"(Head: ---40hexomitted---[master]("shiny/driver/x11driver: tighten the textureImpl.draw dst rectangle.") LastSeen: ---40hexomitted---[master]("shiny/widget/flex: basics of flex algorithm"))
2016/05/19 22:45:58 exp: sending commit to dashboard: ---40hexomitted---[master]("shiny/driver/x11driver: tighten the textureImpl.draw dst rectangle.")
```

Server time is:

```
$ date
Thu May 19 23:06:28 UTC 2016
```

So it's just the go repo that's hung?

But the git subprocess for the go repo keeps coming & going. It's not hung there.

We should have a debugging endpoint to get the watcher's goroutines too.

I'll kick it for now.

This also needs to be monitored.
Repository: flutter
Issue: test framework in live mode should relayout and repaint if you rotate device
Labels: a: tests,c: new feature,framework,P3,team-framework,triaged-framework
Repository: go
Issue: x/build/cmd/buildlet: detect and kill stray processes
Labels: Builders,NeedsFix

The buildlet, especially in reverse buildlet mode, needs to learn how to kill stray processes leftover from previous builds.

(This isn't relevant for VM and container-based builders, since the VM or container kill everything when they exit)

But on OS X and other weirder builders where we run the buildlet in reverse-dialing mode, I sometimes see stray processes. Like this:

Notice testgo spinning:

<img width="918" alt="screen shot 2016-05-20 at 2 32 25 pm" src="https://cloud.githubusercontent.com/assets/2621/15442767/f5e99af6-1e97-11e6-9e47-cba5a9477ed4.png">

(I think @ianlancetaylor fixed that bug, but still.... we should be tolerant of such bugs in the future)

This is a tracking bug to make the buildlet smarter: enumerate processes at start and end, and kill new things which didn't go away on their own, especially if they're owned or parented by us. Maybe scan the process list often during builds to learn parents, so we know if they were ours or not.

/cc @zombiezen @adg
Repository: youtube-dl
Issue: Support for http://traktrain.com/ ?
Labels: site-support-request

## Please follow the guide below
- You will be asked some questions and requested to provide some information, please read them **carefully** and answer honestly
- Put an `x` into all the boxes [ ] relevant to your _issue_ (like that [x])
- Use _Preview_ tab to see how your issue will actually look like

---
### Make sure you are using the _latest_ version: run `youtube-dl --version` and ensure your version is _2016.05.16_. 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.05.16**
### 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_?
- [ ] Bug report (encountered problems with youtube-dl)
- [x ] Site support request (request for adding support for a new site)
- [ ] Feature request (request for a new functionality)
- [ ] Question
- [ ] Other

I would like to request support for this website: http://traktrain.com/.
example:
http://traktrain.com/davidsanyamusic
Repository: go
Issue: go/types: provide better error message for errors in n:1 init assignment
Labels: NeedsFix

Given a.go:

```
package p

var a, b []int = f1() // error

func f1() (_ []int, _ int) {
    return
}
```

gotype reports:

```
$ gotype a.go
a.go:3:18: cannot use f1() (value of type int) as []int value in assignment
```

vs gc compiler:

```
$ go tool compile a.go
a.go:3: cannot assign int to b (type []int) in multiple assignment
```

The gc compiler's error message is much better.
Repository: go
Issue: cmd/compile: misleading error for multiple value function call in contexts that do accept multiple values
Labels: compiler/runtime

[playground](https://play.golang.org/p/kHimGTNfNc)

I can use a function's multiple returns as arguments to another function, but cannot use them as parameters in an initialization.

If it is intended, then the compiling error `multiple-value ... in single-value context` is misleading, because an array or struct initialization can indeed take multiple values.
Repository: youtube-dl
Issue: Is this possible to download? (flash player I believe)
Labels: site-support-request

## Please follow the guide below
- You will be asked some questions and requested to provide some information, please read them **carefully** and answer honestly
- Put an `x` into all the boxes [ ] relevant to your _issue_ (like that [x])
- Use _Preview_ tab to see how your issue will actually look like

---
### Make sure you are using the _latest_ version: run `youtube-dl --version` and ensure your version is _2016.05.21.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.
- [ ] I've **verified** and **I assure** that I'm running youtube-dl **2016.05.21.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_?
- [ ] Bug report (encountered problems with youtube-dl)
- [ x] Site support request (request for adding support for a new site)
- [ ] Feature request (request for a new functionality)
- [ ]x Question
- [ ] Other

---

Is it possible to download from websites like this: http://kswisha.com/ ?
Repository: youtube-dl
Issue: Improve Brightcove title extraction
Labels: request

For at least some videos using Brightcove, especially those without separate extractors, the 'wrong' title is extracted, e.g.

```
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.rollingstone.com/music/videos/norwegian-dj-cashmere-cat-goes-spartan-on-with-me-premiere-20131219']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.05.21.2
[debug] Git HEAD: e913537
[debug] Python version 2.7.11 - Darwin-15.3.0-x86_64-i386-64bit
[debug] exe versions: avconv 11.4, avprobe 11.4, ffmpeg 3.0.2, ffprobe 3.0.2, rtmpdump 2.4
[debug] Proxy map: {}
[generic] norwegian-dj-cashmere-cat-goes-spartan-on-with-me-premiere-20131219: Requesting header
WARNING: Falling back on generic information extractor.
[generic] norwegian-dj-cashmere-cat-goes-spartan-on-with-me-premiere-20131219: Downloading webpage
[generic] norwegian-dj-cashmere-cat-goes-spartan-on-with-me-premiere-20131219: Extracting information
[download] Downloading playlist: Norwegian DJ Cashmere Cat Goes Spartan on 'With Me' - Premiere
[generic] playlist Norwegian DJ Cashmere Cat Goes Spartan on 'With Me' - Premiere: Collected 1 video ids (downloading 1 of them)
[download] Downloading video 1 of 1
[brightcove:new] 4404576300001: Downloading webpage
[brightcove:new] 4404576300001: Downloading JSON metadata
[brightcove:new] 4404576300001: Downloading m3u8 information
[brightcove:new] 4404576300001: Downloading m3u8 information
[debug] Invoking downloader on u'https://secure.brightcove.com/services/mobile/streaming/index/rendition.m3u8?assetId=4404592834001&secure=true&pubId=3891678521001&videoId=4404576300001'
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 24
[download] Destination: Music Video-4404576300001.mp4
[download]   8.3% of ~23.41MiB at  2.24MiB/s ETA 00:25
```

It seems that the Brightcove extractor interprets the video as a playlist with a single video; the playlist title is the actual video title. Improving the Brightcove extractor so that it can better differentiate between playlists and single videos would solve the issue.
Repository: kubernetes
Issue: kubectl edit: error UX is confusing and inconsistent
Labels: priority/backlog,area/kubectl,kind/feature,help wanted,sig/cli,lifecycle/frozen

I edit a pod with some metadata such as:

```
# Please edit the object below. Lines beginning with a '#' will be ignored,
# and an empty file will abort the edit. If an error occurs while saving this file will be
# reopened with the relevant failures.
#
apiVersion: v1
kind: Pod
metadata:
  generateName: guestbook-
  labels:
    app: guestbook
```

Then I close the file and it pops back open with this message after I add `fun: true` label:

```
# Please edit the object below. Lines beginning with a '#' will be ignored,
# and an empty file will abort the edit. If an error occurs while saving this file will be
# reopened with the relevant failures.
#
# The edited file had a syntax error: unable to decode "edited-file": [pos 453]: json: expect char '"' but got char 't'
#
apiVersion: v1
kind: Pod
metadata:
  generateName: guestbook-
  labels:
    app: guestbook
    fun: true
```

Putting the error in the comment header is OK but I was very confused when this happened to me and didn't notice the errors at all. A few potential fixes:
1. Print out the errors and then ask the user to "press enter" to fix, then launch the $EDITOR again
2. Put the error message above the line that generated the error
3. Automatically quote true/false values before converting to JSON in fields like labels.

Given the three options 1 is probably easiest and would fix the issue well enough for now.

cc @janetkuo as we discussed this might be a duplicate but I can't find the other one.
Repository: vscode
Issue: Implement an overflow design for status bar entries
Labels: feature-request,ux,workbench-status

Hello,

For now, when you add a lot of items to the statusbar via plugins, only first few items are displayed and the remaining items are hidden.

It would be great to have a dropdownlist which will show all those remaining items.

I hope such a feature could enhance vscode.

Thanks in advance for your comments.
Repository: opencv
Issue: imreadmutli not clear and some suggestion
Labels: feature,category: imgcodecs,effort: few days

Good to know that OpenCV going to handle multiple tiff , but I still did understand imreadmulti documentation , if I have an image with unknown number of pages , who will direct me to use imread or imreadmulti , another issue imreadmulti read all pages in memory (verctor of MAT) , I have some books may reach to 1000 pages or more , loading all pages will consume most of device memory , I think it's better to be implemented as follow
1-Function to get number of pages .
2-If number of pages is one we can call imread 
3-If Number of pages larger than 1 we can call imreadmulti
4-Do not load all pages in memory , just open the required index.
5-If the user need all pages , he will use the pages counter obtained from point 1 and loop all pages.
Repository: opencv
Issue: MSER Connected Components not accessable from class
Labels: feature,category: features2d

### Please state the information for your system
- OpenCV version: 3.x
- Host OS: Linux (Ubuntu 16.04)
### In which part of the OpenCV library you got the issue?
- MSER feature detector
### Expected behaviour

I would like to have access to the connected components from MSER algorithm in order to implement text detection.
### Actual behaviour

There is not way to get the connected components
### Additional description

Hey, I want to implement [robust text detection in natural images](http://web.stanford.edu/~hchen2/papers/ICIP2011_RobustTextDetection.pdf), as shown in this [official matlab example](http://www.mathworks.com/help/vision/examples/automatically-detect-and-recognize-text-in-natural-images.html). But in order to do it you must have access to the connected components within the MSER algorithm.
Those are [computed by opencv here](https://github.com/Itseez/opencv/blob/09df1a286b62450f2b619ee1d4fec02000daa2e4/modules/features2d/src/mser.cpp#L368), but as you can see from the code they are not saved anywhere and there is no way to pull them out of the function. I am not sure how to pull this from the c++ library without breaking something. I would gladly make a pull request if someone points me in the right direction.
### Code example to reproduce the issue / Steps to reproduce the issue

Please try to give a full example which will compile as is.

python:

``` python
    img = cv2.imread(img)
    mser.setDelta(4)
    mser.setMinArea(200)
    mser.setMaxArea(8000)
    ngrey = cv2.cvtColor(origin, cv2.COLOR_BGR2GRAY)
    regions = mser.detectRegions(ngrey, None)
    #would like to have
    #regions, mser_comp = mser.detectRegionsWithConnectedComponents(img, None)

```

c++:

``` c++
Mat image = imread( img);
    vector<vector<Point>> contours;
    Ptr<MSER> mserExtractor  = MSER::create();
    vector<vector<Point>> contours;
   cvtColor( image, grey, CV_BGR2GRAY );
   vector<cv::Rect> mserBbox;
    mserExtractor->detectRegions(grey, contours, mserBbox);
```
Repository: godot
Issue: [TRACKER] Scene Inheritance related issues
Labels: bug,enhancement,topic:core,topic:editor,tracker

This is a tracker for issues related to Scene Inheritance.

```[tasklist]
- [x] #2223
- [x] #3897
- [x] #4759
- [x] #3904
- [x] #3945
- [x] #3395
- [x] #2946
- [x] #6106
- [x] #5798
- [x] #8103
- [x] #5997
- [x] #10147
- [x] #6212
- [ ] #7984
- [ ] https://github.com/godotengine/godot/issues/10156#issuecomment-320997658
- [x] #12373
- [x] #12990
- [x] #6169
- [x] #621
- [x] #4585
- [x] #13013
- [x] #481
```
Repository: TypeScript
Issue: Suggestion: NODE_PATH support in module resolution
Labels: Suggestion,Awaiting More Feedback

I'm working on a large project in which we'd like to use Typescript.  We use disparate directories for our JS source, relying on our NODE_PATH environment var to pull them all together.  However, Typescript does not use NODE_PATH in its module resolution.

I realize that our setup differs from the typical Node project, and may be considered an edge case.  However, if I were to submit a pull request that added NODE_PATH support, would it be approved?
Repository: go
Issue: x/mobile/exp/audio/al: Error when using `gomobile bind`: no current JVM
Labels: mobile

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

`go version go1.6.2 darwin/amd64`
## What operating system and processor architecture are you using (`go env`)?

```
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/hajimehoshi/go"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GO15VENDOREXPERIMENT="1"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fno-common"
CXX="clang++"
CGO_ENABLED="1"
```
## What did you do?

Calling `x/mobile/exp/audio/al` functions with `gomobile bind` causes error: `al: no current JVM`. It seems like `init` in `x/mobile/exp/audio/al` is called just before `setContext` in `x/mobile/bind/java/context_android.go` is called, which means `init` in `al` is called without the JVM context set.
## What did you expect to see?

No error.
Repository: go
Issue: cmd/compile: loads/constants not lifted out of loop
Labels: Performance,NeedsFix,compiler/runtime

```
var t [256]byte

func f(b *[16]byte) {
    for i, v := range b {
        b[i] = t[v]
    }
}
```

Compiles to:

```
    0x000f 00015 (tmp1.go:7)    MOVBLZX (AX), DX
    0x0012 00018 (tmp1.go:7)    LEAQ    "".t(SB), BX
    0x0019 00025 (tmp1.go:7)    MOVBLZX (BX)(DX*1), DX
    0x001d 00029 (tmp1.go:7)    MOVQ    "".b+8(FP), BX
    0x0022 00034 (tmp1.go:7)    MOVB    DL, (BX)(CX*1)
    0x0025 00037 (tmp1.go:6)    INCQ    AX
    0x0028 00040 (tmp1.go:6)    INCQ    CX
    0x002b 00043 (tmp1.go:6)    CMPQ    CX, $16
    0x002f 00047 (tmp1.go:6)    JLT $0, 15
```

Both the `LEAQ` and the `MOVQ "".b+8` could be done outside the loop.
Repository: kubernetes
Issue: Client interfaces should not expose internal rest clients
Labels: kind/cleanup,sig/api-machinery,lifecycle/frozen

https://github.com/kubernetes/kubernetes/pull/24220 added getters to expose internal RESTClient objects in Client interfaces. That tightly couples those interfaces to a particular implementation, which doesn't make sense (as evidenced by the fake clients returning nil, and anything calling those getters having to handle nils)

For example, this interface now couples all implementations to a RESTClient:

```
type CoreV1Interface interface {
	RESTClient() rest.Interface
	ComponentStatusesGetter
	ConfigMapsGetter
	EndpointsGetter
	EventsGetter
	LimitRangesGetter
	NamespacesGetter
	NodesGetter
	PersistentVolumesGetter
	PersistentVolumeClaimsGetter
	PodsGetter
	PodTemplatesGetter
	ReplicationControllersGetter
	ResourceQuotasGetter
	SecretsGetter
	ServicesGetter
	ServiceAccountsGetter
}
```

cc @krousey @smarterclayton @gmarek