sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: rust
Issue: Wrong error message (or type system bug) for unsatisfied trait bound
Labels: C-enhancement,A-diagnostics,A-trait-system,T-compiler

Here's the [code](https://play.rust-lang.org/?gist=8295fe2b1cf5225ae082&version=nightly). Not sure how to get reduced example, sorry about that.
So, in general, the problem is that I have

``` rust
pub trait WhereType<'a>: ToSQL + CloneToTrait<'a> {}
```

and some implementations of it (32-37 lines). At 37 line I'm trying to implement this trait for struct which does not implement `ToSQL`. I would expect error message about that and instead I'm getting this:

``` rust
<anon>:37:1: 37:43 error: the trait `Numeric` is not implemented for the type `Subquery<'a>` [E0277]
<anon>:37 impl<'a> WhereType<'a> for Subquery<'a> {}
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
<anon>:37:1: 37:43 help: see the detailed explanation for E0277
<anon>:37:1: 37:43 note: required by `WhereType`
<anon>:37:1: 37:43 error: the trait `core::fmt::Display` is not implemented for the type `Subquery<'a>` [E0277]
<anon>:37 impl<'a> WhereType<'a> for Subquery<'a> {}
          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
<anon>:37:1: 37:43 help: see the detailed explanation for E0277
<anon>:37:1: 37:43 note: `Subquery<'a>` cannot be formatted with the default formatter; try using `:?` instead if you are using a format string
<anon>:37:1: 37:43 note: required by `WhereType`
error: aborting due to 2 previous errors
playpen: application terminated with error code 101
```

It looks like rustc is trying to satisfy this impl instead: 

``` rust
impl<'a, T: Numeric + ToSQL + Clone> WhereType<'a> for T {}
```

Not sure if it's a bug, but it's definitely wrong/misleading error message.
It's the same for stable/beta/nightly.
Repository: go
Issue: tour: confusion with pointer and methods.
Labels: NeedsInvestigation

Context: http://tour.golang.org/methods/3

I played around with the code and tried this:
.....
func (v Vertex) Scale(f float64) Vertex {
    v.X = v.X \* f
    v.Y = v.Y \* f
    return v
}
...
func main() {
    v := &Vertex{3, 4}
    fmt.Printf("Before scaling: %+v, Abs: %v\n", v, v.Abs())
    v = &v.Scale(5)
    fmt.Printf("After scaling: %+v, Abs: %v\n", v, v.Abs())
}

The compiler gives the error: prog.go:25: cannot take the address of v.Scale(5)
But if I write instead:
    w := v.Scale(5)
    v = &w

..., then the code works.
This was confusing for me! Can you clarify this?
Repository: rust
Issue: dropck doc oversights / revision
Labels: C-enhancement,A-destructors

The implementation of non-parametric dropck (#28861) included comments and doc that is missing a few things.
1. There was feedback from @Gankro on changes to the rustonomicon that should be further tweaks (see comment thread from #28861)
2. There should be a _huge_ warning (maybe even a lint or something) that if you put `#[unsafe_destructor_blind_to_params]` on a struct that is using native pointers (e.g. `struct Foo<T>( { ptr: *mut T }`) then you almost certainly need a `PhantomData` in there.

The old code before #28861 was in fact busted in this respect, as one can see by taking the test case from #29106 and running it on older versions of Rust.
Repository: rust
Issue: Dead code warning should provide some help for making the code public in library crates
Labels: C-enhancement,A-lints,A-diagnostics,T-compiler,L-dead_code

When compiling a library crate a dead code warning occurs it might be a sign that programmers actually wanted to make the code public. The warning message should tell them how to do that (e.g. `pub` keyword missing on `mod` or `fn`/`enum`/...).

Especially making the `mod` `pub` can easily be forgotten. (E.g. [this person](https://www.reddit.com/r/rust/comments/2dt2v6/why_does_rustc_complain_about_dead_code_when_im/) and me did.)
Repository: youtube-dl
Issue: Add support for tvfplay.com
Labels: site-support-request,account-needed

youtube-dl http://tvfplay.com/video/detail/477/TVF-Pitchers-S01E05-Where-Magic-Happens --verbose 
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['http://tvfplay.com/video/detail/477/TVF-Pitchers-S01E05-Where-Magic-Happens', '--verbose']
[debug] Encodings: locale 'UTF-8', fs 'UTF-8', out 'UTF-8', pref: 'UTF-8'
[debug] youtube-dl version 2014.02.17
[debug] Python version 2.7.6 - Linux-3.16.0-50-generic-x86_64-with-Ubuntu-14.04-trusty
[debug] Proxy map: {}
[generic] TVF-Pitchers-S01E05-Where-Magic-Happens: Requesting header
WARNING: Falling back on generic information extractor.
[generic] TVF-Pitchers-S01E05-Where-Magic-Happens: Downloading webpage
[generic] TVF-Pitchers-S01E05-Where-Magic-Happens: Extracting information
ERROR: Unsupported URL: http://tvfplay.com/video/detail/477/TVF-Pitchers-S01E05-Where-Magic-Happens; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 493, in extract_info
    ie_result = ie.extract(url)
  File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 158, in extract
    return self._real_extract(url)
  File "/usr/lib/python2.7/dist-packages/youtube_dl/extractor/generic.py", line 380, in _real_extract
    raise ExtractorError('Unsupported URL: %s' % url)
ExtractorError: Unsupported URL: http://tvfplay.com/video/detail/477/TVF-Pitchers-S01E05-Where-Magic-Happens; please report this issue on https://yt-dl.org/bug . Be sure to call youtube-dl with the --verbose flag and include its complete output. Make sure you are using the latest version; type  youtube-dl -U  to update.
Repository: TypeScript
Issue: Combining destructuring with parameter properties
Labels: Suggestion,In Discussion,Effort: Moderate

Today, we can take advantage of parameter properties to reduce the boilerplate, e.g:

``` typescript
class Person {
  constructor(public firstName: string, public lastName: number, public age: number) {

  }
}
```

Since 1.5, we can also use destructuring, e.g:

``` typescript
class Person {
  firstName: string;
  lastName: string;
  age: number;

  constructor({ firstName, lastName, age } : { firstName: string, lastName: string, age: number }) {
    this.firstName = firstName;
    this.lastName = lastName;
    this.age = age;
  }
}
```

I've tried in many ways to combine both features, but had no success. So:
1. Is it possible to combine them nowadays, and if yes, how?
2. If not, could it be an improvement to a future TypeScript version? E.g:

``` javascript
class Person {
  constructor(public { firstName, lastName, age } : { firstName: string, lastName: string, age: number }) {

  }
}

// the code above would possibly transpile to:
var Person = (function () {
    function Person(_a) {
        var firstName = _a.firstName, lastName = _a.lastName, age = _a.age;
        this.firstName = firstName;
        this.lastName = lastName;
        this.age = age;
    }
    return Person;
})();
```
Repository: youtube-dl
Issue: [cbsnews] Add support for playlists
Labels: request

It's been working fine for months now...  Here it is;

C:\Notify+>C:\Python27\Scripts\youtube-dl.exe --verbose -o r:+cbs-60.minutes.fl
v --ffmpeg-location=c:\python27\scripts --recode-video=mp4 http://www.cbsnews.co
m/latest/60-minutes/full-episodes/
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--verbose', u'-o', u'r:\+cbs-60.minutes.flv', u'-
-ffmpeg-location=c:\python27\scripts', u'--recode-video=mp4', u'http://www.cbs
news.com/latest/60-minutes/full-episodes/']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2015.10.18
[debug] Python version 2.7.8 - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg N-72086-g51f6455, ffprobe N-72086-g51f6455, rtmpdum
p 2.4
[debug] Proxy map: {}
[CBSNews] full-episodes: Downloading webpage
ERROR: Unable to extract video JSON info; please report this issue on https://yt
-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  t
o update. Be sure to call youtube-dl with the --verbose flag and include its com
plete output.
Traceback (most recent call last):
  File "youtube_dl\YoutubeDL.pyo", line 661, in extract_info
  File "youtube_dl\extractor\common.pyo", line 291, in extract
  File "youtube_dl\extractor\cbsnews.pyo", line 53, in _real_extract
  File "youtube_dl\extractor\common.pyo", line 594, in _html_search_regex
  File "youtube_dl\extractor\common.pyo", line 585, in _search_regex
RegexNotFoundError: Unable to extract video JSON info; please report this issue
on https://yt-dl.org/bug . Make sure you are using the latest version; type  you
tube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and in
clude its complete output.
Repository: go
Issue: tour: add a tour of text/template and html/template
Labels: NeedsInvestigation

The Go text/template and html/template are widely used and well-documented, but a tour would provide a better way of teaching the template language, starting from the basics and moving on to more advanced features.  I'd also like to see non-HTML uses of text/template called out, such as generating markdown or complex SQL queries.
Repository: opencv
Issue: VideoCapture::isOpened() test error
Labels: bug,priority: normal,category: videoio,affected: 3.4

modules/videoio/src/cap.cpp on line 600

``` .cpp
bool VideoCapture::isOpened() const
{
    return (!cap.empty() || !icap.empty());
}
```

Does the test shouldn't be ?

``` .cpp
return (!cap.empty() && !icap.empty());
```

I ran in a case where isOpened return true but the device is already in use and icap is empty but not cap
Repository: TypeScript
Issue: Heuristic/loose completions for the 'any' type
Labels: Suggestion,In Discussion

# Motivation

Today, if you have a value of type 'any', dotting off of the value and requesting members for completion will return nothing back. The original motivation for this was that we should never risk giving users incorrect completions.

This is great for a fully typed TypeScript project. However, for those migrating to TypeScript, as well as those who need to drop down to `any` fairly frequently, this can be slightly frustrating.

For those used to the JS editing experience in editors like Sublime where they just "got completions", the current behavior might be seen as unattractive.
# Proposal

We should consider adding a language service option for users to get "loose" completions akin to what we give in the Salsa language service. 

We can still keep the warning for each completion, but we should provide completions with a builder.
Repository: opencv
Issue: MacOSX: VideoWriter slows down after some point
Labels: bug,wontfix,priority: low,category: videoio

Performance of VideoWriter's write method gets very low after some point. In these two stackoverflow questions, problem is explained clearly:
- http://stackoverflow.com/questions/26695486/python-opencv-videowriter-slows-down-at-around-3000-write
- http://stackoverflow.com/questions/32163691/opencv-videowriter-become-slower-and-slower

What is reason of this specific problem? Is it not suitable to use VideoWriter class for writing long (longer than 3 minutes) videos?
Repository: kubernetes
Issue: NFS volume plugin should translate to service IP before making mount call
Labels: sig/storage,lifecycle/frozen

I'm trying to brush off the [nfs example](https://github.com/kubernetes/kubernetes/tree/master/examples/nfs) now that privileged is working by default, and one of the small warts in it is the web server pod needs a hardcoded IP or hostname in order to bring the pieces together. (And I'm running through it for GCE to make sure it works there.)

I think this is straightforward to fix with an API call during the mount, then we can just call system mount with the cluster IP.

cc @thockin @cjcullen @jsafrane
Repository: rust
Issue: Associated item projections across crate boundaries slightly broken
Labels: A-trait-system,P-medium,A-associated-items,T-lang,C-bug,T-types,A-coherence

``` rust
pub trait Foo<T> { }
pub trait Mirror {
        type Dual;
}

pub struct Eps;

pub struct A<T, U>(T, U);
impl<T, U: Mirror> Mirror for A<T, U> { type Dual = B<T, U::Dual>; }
pub struct B<T, U>(T, U);
impl<T, U: Mirror> Mirror for B<T, U> { type Dual = A<T, U::Dual>; }

impl Mirror for Eps { type Dual = Eps; }
```

```rs
extern crate whatever;
use whatever::*;

struct Dummy;

impl Foo<A<Eps, B<Eps, Eps>>> for Dummy {}

// doesn't work, causes "conflicting implementations for trait `whatever::Foo`"
// impl Foo<<A<Eps, B<Eps, Eps>> as Mirror>::Dual> for Dummy {}

// works just fine
// impl Foo<B<Eps, A<Eps, Eps>>> for Dummy {}
```

This despite the fact that `<A<Eps, B<Eps, Eps>> as Mirror>::Dual` and `B<Eps, A<Eps, Eps>>` are the same type. It works just fine when they're in the same crate.
Repository: opencv
Issue: Documentation: Update docs about connected component label images
Labels: priority: normal,feature,category: documentation,affected: 3.4

The docs should be updated to state that: given the same binary input image (inverted for `cv::distanceTransform()`), the labels in the images returned by all the following functions will all be consistent:
- `cv::distanceTransform()` (version with labels)
- `cv::connectedComponents()`
- `cv::connectedComponentsWithStats()`

See [SO Q&A](http://stackoverflow.com/questions/33256284/label-images-in-opencv) about this.
Repository: kubernetes
Issue: Testing doesn't sufficiently cover client/cluster version skew
Labels: area/test,priority/important-soon,area/kubectl,sig/api-machinery,area/test-infra,area/upgrade,lifecycle/frozen

As we're moving forward with the `v1.1` release, we're realizing we don't have sufficient test coverage for version-skew between clients and clusters.  Recently, we put in place upgrade tests:

```
  # kubernetes-upgrade-gke-1.0-1.1
  #
  # This suite:
  #
  #1. launches a cluster at release/latest-1.0,
  #2. upgrades the master to ci/latest-1.1
  #3. runs release/latest-1.0 e2es,
  #4. upgrades the rest of the cluster,
  #5. runs release/latest-1.0 e2es again, then
  #6. runs ci/latest-1.1 e2es and tears down the cluster.
```

When these tests run, the client version always matches the e2e version, so in `3.` and `5.` the client is a `1.0` client, in `6.` it's a `1.1` client.

Problems:
- [ ] We don't cover the reverse case, where we have a `1.1` client working a `1.0` cluster.
  - This is tough because we don't expect `1.1` tests to pass against a `1.0` cluster, so we'd, in theory, have to run `1.0` tests through a `1.1` client, which is tough.

Possible solutions:
- [x] Run `1.1` tests against a `1.0` cluster, and see what breaks (#16041);
- [ ] Figure out a way to link different versions of the tests and client library together;
- [ ] Figure out a way to run the test-command suite against a live, version-skewed cluster.

This is _not_ a `v1.1` blocker.

Please add other cases we don't catch, so we have a sense of where we need to go.
Repository: go
Issue: x/build/cmd/coordinator: limit number of TryBot reattempts
Labels: Builders,NeedsFix

I started http://farmer.golang.org/try?commit=ed6176f8 about 4.5 hours ago, and it appears to have attempted to start a Plan 9 GCE image every 6 minutes (i.e., ~45 times now).  While its determination is laudable, I think it could stop after maybe 3.

CC @bradfitz @adg
Repository: neovim
Issue: Feature: Disable jump list for buffer
Labels: enhancement

Would be useful to be able to disable the recording of jumps for a specific buffer. Vim doesn't have this, but several searches online indicate it would be useful for others than me.
Repository: opencv
Issue: VideoCapture problem with big videodata
Labels: bug,priority: normal,category: videoio,affected: 3.4

When try to load a video with using the whole UInt32 limit (over 4GByte uncompressed data). I get the error message "Frame offset points outside movi section." exactly 64 times.

The concerned line is in file "modules/videoio/src/cap_mjpeg_decoder.cpp". For me it looks like some header offsets overflow the uint32 max allowed videosize.

The videofile itself works fine on some (but not all) players.
Repository: opencv
Issue: GPU-hog doesn't work with win_stride smaller then (8, 8)
Labels: bug,priority: normal,category: samples,category: gpu/cuda (contrib)

If you execute the gpu-example-hog on a picture with win_stride (8, 8) and the other default parameters : 
- ./gpu-example-hog /lfs/psc/OpenCV_3_TestData/road.png --win_stride_width 8 --win_stride_height 8
  the output is : 
  ![gpu_hog_win_stride_8_8](https://cloud.githubusercontent.com/assets/11768321/10691617/b90faf24-798a-11e5-9285-2ef168af9a36.png)
  with win_stride (4, 4) : 
- ./gpu-example-hog /lfs/psc/OpenCV_3_TestData/road.png --win_stride_width 4 --win_stride_height 4
  the ouput is : 
  ![gpu_hog_win_stride_4_4](https://cloud.githubusercontent.com/assets/11768321/10691636/ea929764-798a-11e5-911e-05d912d9a9d5.png)

So if you pick a smaller win_stride then 8 the gpu-hog doesn't work but the FPS decrease because he has to calculate more.
Image I used for testing (road.png) : 
![road](https://cloud.githubusercontent.com/assets/11768321/10692353/47f6b8a8-7991-11e5-974c-76df1fa16323.png)
Repository: TypeScript
Issue: Stdlib typings for typed arrays (and specific errors) need brand fields to improve type safety
Labels: Bug,Help Wanted,Domain: lib.d.ts

All typed arrays are assignable in our present dts.

``` ts
let arr = new Uint8Array(20);
let arr2 = new Int16Array(20);
arr = arr2; // Valid.
```

The same is true of specific errors:

``` ts
let err = new SyntaxError();
let err2 = new RangeError();
err = err2; // Also valid
```

We should fix this, as both of these sets of types are sets whose members are meant to be distinguished by type rather than by structure. This is really important in the case of typed arrays, as the following happens:

``` ts
let arr = new Uint8Array(20);
let arr2 = new Int16Array(20);
arr = arr2; // Valid.
let sliced = arr.slice(0, 10); // Typed as a Uint8Array (really an Int16Array at runtime)
```
Repository: youtube-dl
Issue: Site Support Request: Supersport.com
Labels: site-support-request

```
Iliad:~ aaron$ youtube-dl --version
2015.10.24
```

```
Iliad:~ aaron$ youtube-dl -v -j http://www.supersport.com/cricket/sa-team/video/598304
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'-j', u'http://www.supersport.com/cricket/sa-team/video/598304']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.10.24
[debug] Python version 2.7.10 - Darwin-15.0.0-x86_64-i386-64bit
[debug] exe versions: none
[debug] Proxy map: {}
WARNING: Falling back on generic information extractor.
ERROR: Unsupported URL: http://www.supersport.com/cricket/sa-team/video/598304
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 1240, in _real_extract
    doc = parse_xml(webpage)
  File "/usr/local/bin/youtube-dl/youtube_dl/utils.py", line 1667, in parse_xml
    tree = xml.etree.ElementTree.XML(s.encode('utf-8'), **kwargs)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 1300, in XML
    parser.feed(text)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 1642, in feed
    self._raiseerror(v)
  File "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/xml/etree/ElementTree.py", line 1506, in _raiseerror
    raise err
ParseError: not well-formed (invalid token): line 30, column 24
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 661, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 291, in extract
    return self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 1838, in _real_extract
    raise UnsupportedError(url)
UnsupportedError: Unsupported URL: http://www.supersport.com/cricket/sa-team/video/598304
```
Repository: rust
Issue: #[macro_export] stops macro's documentation tests from working
Labels: T-rustdoc,A-macros,C-bug,A-doctests

Adding `#[macro_export]` to a macro stops it from working in documentation tests. Here's a few examples:

`````` rust
/// Test macro
///
/// #Examples
/// ```
/// test_macro!(TestStructure);
/// ```
#[macro_export]
macro_rules! test_macro {
    ($expression:ident) => (struct $expression);
}
``````

`````` rust
#[macro_export]
/// Test macro
///
/// #Examples
/// ```
/// test_macro!(TestStructure);
/// ```
macro_rules! test_macro {
    ($expression:ident) => (struct $expression);
}
``````

They both will yield output similar to this (using cargo):

```
running 0 tests

test result: ok. 0 passed; 0 failed; 0 ignored; 0 measured

   Doc-tests macro_doc

running 1 test
test test_macro!_0 ... FAILED

failures:

---- test_macro!_0 stdout ----
    <anon>:2:5: 2:15 error: macro undefined: 'test_macro!'
<anon>:2     test_macro!(TestStructure);
             ^~~~~~~~~~
error: aborting due to previous error


failures:
    test_macro!_0

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured

thread '<unnamed>' panicked at 'Some tests failed', ../src/libtest/lib.rs:252
```

Removing the export makes it work fine:

`````` rust
/// Test macro
///
/// #Examples
/// ```
/// test_macro!(TestStructure);
/// ```
macro_rules! test_macro {
    ($expression:ident) => (struct $expression);
}
``````
Repository: youtube-dl
Issue: Request: upload_year tag in output template
Labels: request

I'm trying to folder my videos by year. It would be very useful if I could do this natively, rather than managing it from my script.

This could be fixed by copying the relevant code for the upload_date tag, and returning the first four characters.

This would result in %(upload_year)s returning year in YYYY format.
Repository: electron
Issue: Expose the "Spelling and Grammar" related APIs 
Labels: enhancement :sparkles:,platform/macOS,component/menu

On OS X apart from the basic spell-checking it also provides a "Spelling and Grammar" menu that allows some more advanced operations:
<img width="666" alt="screen shot 2015-10-26 at 10 02 39 am" src="https://cloud.githubusercontent.com/assets/639601/10719716/bff72fd4-7bc8-11e5-9b97-980e9a75dd91.png">

Chrome browser has implemented most of the features, we should find a way to make Electron apps use it without any pain.
Repository: neovim
Issue: Repeated append is O(n²)
Labels: performance,bug-vim,normal-mode

Demonstration:
1. Open nvi (BSD nvi).
2. Type `1000000afnord` then ESC.
3. Screen fills up almost instantly.
4. Open neovim.
5. Tyle `1000000afnord` then ESC.
6. Wait... and wait... and wait...

Originally pointed out by [this rant](http://www.galexander.org/vim_sucks.html) (not mine).
Repository: go
Issue: runtime: lock_sema.go uses LIFO ordering for waking blocked Ms
Labels: compiler/runtime

In the lock_sema.go implementation of mutexes, lock handles contention by adding the current M to the front of the linked list of waiting Ms, and then unlock wakes up the M at the front of the list.  I.e., waiting Ms are awoken in LIFO order.

Is it worth changing unlock to awake the M at the _end_ of the list to effect a FIFO order?

For comparison, lock_futex.go leaves resolving contention to the OS.  On Linux, futex wakeup ordering doesn't seem to be guaranteed, but they do at least currently appear to use FIFO ordering (for threads with the same priority/nice level): http://lists.openwall.net/linux-kernel/2015/01/24/106