sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: youtube-dl
Issue: Add muxing option; allow --embed-subs to automatically remux if needed
Labels: request

#### Problem

A video from Crunchyroll is downloaded with `--write-sub`, `--embed-subs` and `--merge-output-format mkv` enabled. The expected outcome is that the downloaded FLV file will be losslessly muxed with the subtitles into an MKV file. What actually happens is that the two files are downloaded as normal without merging; the message `[ffmpeg] Subtitles can only be embedded in mp4 or mkv files` is returned.
#### Possible Solution 1

Add an option to remux the video to a given container format without re-encoding. (i.e. `--mux mkv`, `--mux mp4` etc.)
#### Possible Solution 2

Change the behaviour of `--embed-subs` so that it automatically remuxes if the file format doesn’t allow embedded subtitles. Allow the container format (MKV, MP4) to be specified if possible.
#### Possible Solution 3

Change the behaviour of `--merge-output-format` to always remux regardless of whether or not a merge is required.

Verbose output follows.

```
$ youtube-dl --ignore-config -v -n --merge-output-format mkv --write-sub --sub-lang enUS --embed-subs "http://www.crunchyroll.com/symphogear/episode-1-awakening-heartbeat-685195"
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'--ignore-config', u'-v', u'-n', u'--merge-output-format', u'mkv', u'--write-sub', u'--sub-lang', u'enUS', u'--embed-subs', u'http://www.crunchyroll.com/symphogear/episode-1-awakening-heartbeat-685195']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2015.09.22
[debug] Python version 2.7.10 - Darwin-14.5.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 2.8, ffprobe 2.8, rtmpdump 2.4
[debug] Proxy map: {}
[Crunchyroll] Logging in
[Crunchyroll] 685195: Downloading webpage
[Crunchyroll] 685195: Downloading media info
[Crunchyroll] 685195: Downloading media info for 360p
[Crunchyroll] 685195: Downloading media info for 480p
[Crunchyroll] 685195: Downloading media info for 720p
[Crunchyroll] 685195: Downloading media info for 1080p
[Crunchyroll] 685195: Downloading subtitles for English (US)
[Crunchyroll] 685195: Downloading subtitles for Español
[Crunchyroll] 685195: Downloading subtitles for Español (España)
[Crunchyroll] 685195: Downloading subtitles for Français (France)
[Crunchyroll] 685195: Downloading subtitles for العربية
[info] Writing video subtitles to: Symphogear Episode 1 – Awakening Heartbeat-685195.enUS.ass
[debug] Invoking downloader on 'rtmpe://cp150757.edgefcs.net/ondemand/?auth=daEa.cxcIdqdYaucYc1ciaRc7b7c9dqdLd4-bwb6oB-dHa-nCGyptNDxuu&aifp=0009&slist=c20/s/ve1953099/video.mp4'
[download] Symphogear Episode 1 – Awakening Heartbeat-685195.flv has already been downloaded
[download] 100% of 541.18MiB
[ffmpeg] Subtitles can only be embedded in mp4 or mkv files
```
Repository: youtube-dl
Issue: Terminate download after command-line supplied duration
Labels: request

Looking through the documentation, I can't see a way to download a stream for a specific number of seconds.
This would be useful when downloading from audio services like tunein where the stream does not end at the end of the programme.
At the moment, I'm working around this by calling youtube-dl from a script and killing the background PID after a certain amount of time:

```
(/usr/local/bin/youtube-dl -q --no-part -o "${OUTFILE}" ${URL}) & DLPID=$!
sleep $DURATION
kill ${DLPID}
```

Having a --duration flag and terminating after this time would be quite useful for cases like this.
Repository: go
Issue: x/mobile: Fullscreen mode, app.SetFullscreen(fullscreen bool) ?
Labels: mobile

If it is okay to add this to the app package, i can send a CL implementing this to Android and IOS.
Repository: opencv
Issue: A quest for better docs for MatExpr and Mat
Labels: category: documentation,affected: 3.4,RFC

Hi devs,
After using Opencv for my project, I feel that we need a quest to improve docs of MatExpr. There is potential bugs if we do not use the library correctly. 
For example, MatExpr 

```
        // assuming matrix B of type CV_8U, I just want to normalize it into [0, 1]
        Mat A = B/255.0f;  // potiential bug
```

The results are completely different from what we want. The matrix `A` is still in CV_8U, so it will lose floating points value to convert to just _0_ or _1_. 
My opinion is that we should explicitly say that the type of resultant matrix is the same as with the most compatible operand matrix.
Repository: rust
Issue: Teach make-tidy to detect unused *.rs files
Labels: T-bootstrap,E-medium,C-feature-request

As an example, the file https://github.com/rust-lang/rust/blob/master/src/librustc_front/attr.rs is currently dead and can be removed, but before realizing it I spent several minutes trying to understand what's wrong with my text editor and why can't it find the definition of [`unlower_attribute`](https://github.com/rust-lang/rust/blob/master/src/librustc_front/attr.rs#L57).
Repository: go
Issue: go/doc: Examples not executable if pkg.name != basename(pkg.path)
Labels: help wanted,Tools

% export GOPATH=$(pwd)
% cat src/foo1/foo.go
package foo

func Foo() {
}

% cat src/foo1/foo_test.go
package foo_test

import "foo1" // defines foo

func ExampleFoo() {
        foo.Foo()
}

% godoc -play -http :9999 &
% open http://localhost:9999/pkg/foo1/#pkg-examples

The example is shown but not executable (grey not yellow); renaming foo1 to foo makes it executable, as does using an explicit (redundant) renaming import:
   import foo "foo1"

Seems like some code in godoc is assuming pkg.name == basename(pkg.path) instead of finding the actual package name.
Repository: javascript
Issue: Best practice for wrapping function arguments
Labels: question

Style guide should advise wrapping function arguments that go over maximum line length.

``` js
function someVeryLongFunctionName(someVeryLongArgumentName1, someVeryLongArgumentName2, someVeryLongArgumentName3) {
  // Function body
}
```

or:

``` js
function myFunction(someArg1, someArg2, someArg3, someArg4, someArg5, someArg6, someArg7, someArg8) {
  // Function body
}
```

Wrapped:

``` js
function myFunction(someArg1, someArg2, someArg3, someArg4,
                    someArg5, someArg6, someArg7, someArg8) {
  // Function body
}
```
Repository: rust
Issue: Range expressions: discrepancies between rustc and parser-lalr
Labels: A-grammar,P-medium,I-needs-decision,T-lang,C-bug

I'd guess that the last test case here is a `rustc` bug, but I don't know about the others.

``` rust
fn match_rangefull() {
    match .. { _ => () } // rustc compiles it, parser-lalr rejects
}
fn field_of_range() {
    123.. .start; // rustc rejects, parser-lalr accepts
}
fn eq_rangefull() {
    .. == ..; // rustc rejects, parser-lalr accepts
}
fn return_rangeto() -> std::ops::RangeTo<i32> {
    return ..5; // rustc compiles it, parser-lalr parses it as (return)..5
}
fn assign_range() {
    let x;
    x = 4..5; // rustc compiles it, parser-lalr parses it as (x = 4)..5
}
fn nonblock_expr_range() {
    // rustc compiles this, and I don't think it should.  I think parser-lalr
    // gets it right.
    //
    //    rustc parses it like:       let _one = (1..).start;
    //    parser-lalr parses it like: let _one = {1; ..}.start;
    //
    let _one = { {1} .. }.start;
}
fn main() {}
```

Also see https://github.com/rust-lang/rust/issues/25119.
Repository: youtube-dl
Issue: Default options (bestvideo+bestaudio) does not consider downloading a higher-resolution video
Labels: request

```
$ youtube-dl --get-format https://www.youtube.com/watch?v=X7pzGyfRx9A
WARNING: Could not download DASH manifest: HTTP Error 403: Forbidden
136 - 900x720 (DASH video)+141 - audio only (DASH audio)
```

```
$ youtube-dl -F https://www.youtube.com/watch?v=X7pzGyfRx9A
[youtube] X7pzGyfRx9A: Downloading webpage
[youtube] X7pzGyfRx9A: Downloading video info webpage
[youtube] X7pzGyfRx9A: Extracting video information
[youtube] X7pzGyfRx9A: Downloading DASH manifest
[youtube] X7pzGyfRx9A: Downloading js player new-en_US-vflJLt_ns
[youtube] X7pzGyfRx9A: Downloading DASH manifest
WARNING: Could not download DASH manifest: HTTP Error 403: Forbidden
[info] Available formats for X7pzGyfRx9A:
format code  extension  resolution note
171          webm       audio only DASH audio   94k , vorbis@128k (44100Hz), 7.20MiB
140          m4a        audio only DASH audio  129k , m4a_dash container, aac  @128k (44100Hz), 10.15MiB
141          m4a        audio only DASH audio  256k , m4a_dash container, aac  @256k (44100Hz), 20.16MiB
160          mp4        180x144    DASH video  114k , avc1.4d400c, 15fps, video only, 8.63MiB
242          webm       300x240    DASH video  148k , vp9, 1fps, video only, 6.92MiB
243          webm       450x360    DASH video  255k , vp9, 1fps, video only, 12.07MiB
133          mp4        300x240    DASH video  262k , avc1.4d400d, 15fps, video only, 19.29MiB
134          mp4        450x360    DASH video  403k , avc1.4d4015, 15fps, video only, 17.30MiB
244          webm       600x480    DASH video  467k , vp9, 1fps, video only, 22.22MiB
135          mp4        600x480    DASH video  698k , avc1.4d4016, 15fps, video only, 36.07MiB
247          webm       900x720    DASH video  786k , vp9, 1fps, video only, 40.29MiB
136          mp4        900x720    DASH video 1183k , avc1.4d401f, 15fps, video only, 71.93MiB
17           3gp        176x144    
36           3gp        320x240    
5            flv        400x240    
43           webm       640x360    
18           mp4        640x360    
22           mp4        1280x720   (best)
```

The format specification `mp4+bestaudio` downloads the higher quality combination:

```
$ youtube-dl --get-format --format mp4+bestaudio https://www.youtube.com/watch?v=X7pzGyfRx9A
WARNING: Could not download DASH manifest: HTTP Error 403: Forbidden
22 - 1280x720+141 - audio only (DASH audio)
```
Repository: go
Issue: x/build: add builders with different GODEBUG=cpu capabilities
Labels: Builders,NeedsFix,FeatureRequest,new-builder

Our builders for the most part only run on Haswell CPUs on Google Compute Engine.

A number of Go packages in the standard library and runtime use assembly to choose alternate paths depending on the capabilities of the processor.

I feel like we're not getting good test coverage, only testing the Haswell paths.

We do have a GOARCH=386 builder, though, which is at one extreme, but there are a number of steps in the middle untested.

Can we put an abstraction over CPUID etc somewhere (unexported) such that the builders can fake it before assembly code can?

/cc @ianlancetaylor @minux @adg
Repository: TypeScript
Issue: Give an error indicating that the triple slash has syntax error
Labels: Suggestion,Help Wanted

I think it will be very nice to give a syntax error when there is a syntax error in triple slash

For example

``` ts
/// <reference patj="file1.ts"/> => a syntax error that triple slash syntax
```
Repository: kubernetes
Issue: Write proposal for controller pod management: adoption, orphaning, ownership, etc. (aka controllers v2)
Labels: priority/important-soon,area/app-lifecycle,sig/api-machinery,sig/apps,lifecycle/frozen

Write a comprehensive proposal for how controllers should manage sets of pods. The main goal is to make controller APIs more usable and less error-prone.

We've discussed a number of changes:
- unique label generation: #12298, #34292
- controllerRef to manage overlap: https://github.com/kubernetes/kubernetes/issues/2210#issuecomment-135588169
- server-side cascading deletion via existence dependences: https://github.com/kubernetes/kubernetes/issues/1535#issuecomment-91413501
- GC prevention via finalizers: #3585
- deletion of non-conforming pods: https://github.com/kubernetes/kubernetes/blob/master/docs/design/daemon.md#cluster-mutations
- generation/observedGeneration: #7328
- status field containing selector query: #3676
- conditions: #14181
- aggregate status: #7483
- policy for naming generated resources: #15776
- reverse label lookup: #1348

We may want to split the following into separate issues:

Changes that would facilitate static work/role assignment:
- identity assignment: #260, #14188
- PVC replication: #12450

Long-standing idea to improve security and reusability around templates:
- separate template: #170

Reusability could also be addressed by:
- template proposal: #18215

Also need to make it easier to update existing pods:
- For labels, currently need to update pod template, then pods, then selector: https://github.com/kubernetes/kubernetes/blob/master/pkg/kubectl/rolling_updater.go#L616 . The loop to update pods after that wouldn't be necessary if it watched observedGeneration (#7328).
- Inline updates: #9043
- vertical auto-scaling
Repository: opencv
Issue: Shape Transformers and Interfaces - Thin Plate Spline Warping
Labels: bug,priority: normal,affected: 3.4,category: shape

Hey,
s.th. in this Shape Transformer Interface seems to be buggy. 
Maybe also see my former Question at the opencv forum http://answers.opencv.org/question/69384/shape-transformers-and-interfaces/

The following code will only produce a completly gray image. The error is reproducable by giving the last point the same  x/y value e.g. 2/2, but will work perfectly fine with 2/3.
But thats only one possibility to cause this error. I found other picutres with "real" landmark points which also cause this error but I couldn't figure out a pattern. (Let me know if you need the data)
I would be grateful for any help :)

```
int main(void)
{
cv::Mat sImg = cv::imread("D:\\Opencv3\\sources\\samples\\data\\graf1.png", cv::IMREAD_GRAYSCALE);
std::vector<cv::Point2f> points;
std::vector<cv::DMatch> good_matches;
cv::Mat tImg;

points.push_back(cv::Point(0, 0)); //Just to have a few points
points.push_back(cv::Point(1, 1));
points.push_back(cv::Point(2, 2)); // points.push_back(cv::Point(2, 3)) -> works fine

good_matches.push_back(cv::DMatch(0, 0, 0));
good_matches.push_back(cv::DMatch(1, 1, 0));
good_matches.push_back(cv::DMatch(2, 2, 0));

// Apply TPS
cv::Ptr<cv::ThinPlateSplineShapeTransformer> mytps = cv::createThinPlateSplineShapeTransformer(0);
mytps->estimateTransformation(points, points, good_matches); // Using same points nothing should change in tImg

imshow("sImg", sImg); // Just to see if I have a good picture

mytps->warpImage(sImg, tImg);

imshow("tImg", tImg); //Always completley gray ?

cv::waitKey(0);
}
```
Repository: opencv
Issue:  Bug: Dependencies between detector runs on LBP
Labels: bug,priority: normal,category: objdetect,affected: 3.4,category: ocl

There is some bug with cv::CascadeClassifier in case of lbpcascade_frontalface:
We have 3 photos (attached files arranged as 02463d131, 02463d139, 02463d142 respectively). Сonsistently run detector on them. For the last picture we get the wrong result. Remove, for example, the second photo. Now, the result for the third picture changed and became correct.
This observation is true only for lbp cascades. On haarcascade all is ok.
See attached files for more info.

Build info:
openCV 3.0.0, vc12, static

Code:

``` cpp
#include <opencv2\opencv.hpp>

int main() {

    std::string f1 = "C:/test/detection/imgs/02463d131.jpg";
    std::string f2 = "C:/test/detection/imgs/02463d139.jpg";
    std::string f3 = "C:/test/detection/imgs/02463d142.jpg";

    cv::Mat im1 = cv::imread(f1, CV_LOAD_IMAGE_GRAYSCALE);
    cv::Mat im2 = cv::imread(f2, CV_LOAD_IMAGE_GRAYSCALE);
    cv::Mat im3 = cv::imread(f3, CV_LOAD_IMAGE_GRAYSCALE);

    cv::CascadeClassifier faceCascade;
    bool isok = faceCascade.load("C:/test/detection/cv_bug/lbpcascade_frontalface.xml");
    //bool isok = faceCascade.load("C:/test/detection/cv_bug/haarcascade_frontalface_alt.xml");

    std::vector<cv::Rect> output1, output2, output3;
    faceCascade.detectMultiScale(im1, output1, 1.16, 1, 0, cv::Size(100, 100), cv::Size(0.99 * im1.cols, 0.99 * im1.cols));
    //faceCascade.detectMultiScale(im2, output2, 1.16, 1, 0, cv::Size(100, 100), cv::Size(0.99 * im2.cols, 0.99 * im2.cols));
    faceCascade.detectMultiScale(im3, output3, 1.16, 1, 0, cv::Size(100, 100), cv::Size(0.99 * im3.cols, 0.99 * im3.cols));

    std::cout << "02463d131.jpg: " << output1[0] << " Face detected: " << output1.size() << '\n';
    //std::cout << "02463d139.jpg: " << output2[0] << " Face detected: " << output2.size() << '\n';
    std::cout << "02463d142.jpg: " << output3[0] << " Face detected: " << output3.size() << '\n';

    std::cin.get();

    return 0;
}
```

![results](https://cloud.githubusercontent.com/assets/12760584/10251913/cb4b0822-693c-11e5-8069-73b80b4a62a9.png)

![02463d131](https://cloud.githubusercontent.com/assets/12760584/10251957/20319cca-693d-11e5-99db-577fe984ab96.jpg)
![02463d139](https://cloud.githubusercontent.com/assets/12760584/10251958/2182f970-693d-11e5-8789-77cb18740b11.jpg)
![02463d142](https://cloud.githubusercontent.com/assets/12760584/10251959/229b0e92-693d-11e5-8a2a-3f622835c761.jpg)
Repository: go
Issue: go/printer: unexpected handling of indented comment
Labels: NeedsInvestigation

The comment was indented one extra tab stop beyond where it should be by gofmt.
- Deleting one of the tabs and running gofmt again results in the proper alignment.
- Removing the statement below the comment and re-formatting will remove the extra tab stop.
- Changing the statement above the comment so the "func" line is on the same line as the assignment causes the extra tab to be removed as well.

Also unexpected, and the reason the bug was noticed: gofmt converts two spaces into two tabs here, which causes an unfortunate interaction with xxx (when not in tab mode) and probably other editors -- if you attempt to remove the extra indent and then re-run gofmt, you may be led to believe it's impossible to get the comment to line up as desired/expected.

```
package p

func f() {
    a =
        func(T1) T2 {
            return q
        }

        // Comment
    b =
        func(T1) T2 {
            return q
        }
}
```
Repository: neovim
Issue: RPC request (or other K_EVENT) causes redraw
Labels: needs:design,job-control,event-loop

I have noticed that the intro message is a bit too short to read. I was looking through the code to try and find where this timeout would be set, but came up empty handed. Can someone push me in the right direction so I can make my first Neovim contribution? :smile:
Repository: go
Issue: runtime: runtime-gdb_test mapvar failure on Fedora 20
Labels: compiler/runtime

I tried to build go from the sources. I followed the instruction from 

https://golang.org/doc/install/source

*\*  What version of Go are you using (go version)?

go version go1.4.2 linux/amd64

and 

go1.4

*\*  What operating system and processor architecture are you using?

fedora 20 (Linux  3.19.8-100.fc20.x86_64 )

Architecture:          x86_64
CPU op-mode(s):        32-bit, 64-bit
Byte Order:            Little Endian
CPU(s):                4
On-line CPU(s) list:   0-3
Thread(s) per core:    2
Core(s) per socket:    2
Socket(s):             1
NUMA node(s):          1
Vendor ID:             GenuineIntel
CPU family:            6
Model:                 42
Model name:            Intel(R) Core(TM) i5-2520M CPU @ 2.50GHz
Stepping:              7
CPU MHz:               2103.320
CPU max MHz:           3200.0000
CPU min MHz:           800.0000
BogoMIPS:              4988.73
Virtualization:        VT-x
L1d cache:             32K
L1i cache:             32K
L2 cache:              256K
L3 cache:              3072K
NUMA node0 CPU(s):     0-3

*\*  What did you do?

I did the steps as described by the installation-from-source webpage. 
- clone the go-language repository
- execute all.bash in subdir src.

*\*  What did you see?

The tests failed which means, the installation failed. The following steps
went seemingly ok
##### Building Go toolchain using/ home/msteffen/go1.4.
##### Building go_bootstrap for host, linux/amd64.
##### Building packages and commands for linux/amd64.

The following section of the installation failed, namely the testing
##### Testing packages.

in particular

--- FAIL: 
    runtime-gdb_test.go:42: gdb version 7.7
    runtime-gdb_test.go:146: print mapvar failed: $1 = map[string]string = {[<error reading variable: Cannot access memory at address 0xb5cc>] = <error reading variable: Cannot access memory at address 0xb5cc>, [<error reading variable: Cannot access memory at address 0x3>] = <error reading variable: Cannot access memory at address 0x3>}
FAIL
FAIL    runtime 41.748s

which resulted in

2015/10/03 14:54:11 Failed: exit status 1

*\* Go Versions:

   I made the above attempt for

```
 git checkout master
 git checkout go1.5
```

both gave the same error

The compilation of go1.5 was using  go1.4.2 (which is shipped with my fedora version) and I also tried to use go1.4 (where I downloaded the binaries and moved them to ~/go1.4). In
all combinations, the result was the same.

However, when I did

```
  git checkout go1.4
```

that one went through, i.e., the go1.4 version, I was able to build from the sources.
Repository: youtube-dl
Issue: Site support request: JizzHut.com
Labels: site-support-request

This website is glorious. It is also unsafe for work.
Repository: three.js
Issue: Specular color map?
Labels: Enhancement

Hi everyone.

Now the `Specular map` only controls the intensity of the specular highlights, it's will be great if we have `Specular color map` (http://wiki.polycount.com/wiki/Specular_color_map), one more step toward to the realistic material.

> Specular Color Map
> 
> A texture that controls the intensity/color of the specular highlights from real-time lights.
> 
> Usually a specular map simply controls the brightness of the highlights, at a per-pixel level. If the shader supports RGB the specular map can be used to colorize specular highlights, useful for surfaces that have more complex reflective properties like metals, beetle shells, etc.