sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: vscode
Issue: Triggering actions.find causes active match to lose its highlight.
Labels: bug,ux,editor-find

- VSCode Version: Code 1.7.1 (02611b40b24c9df2726ad8b33f5ef5f67ac30b44, 2016-11-03T13:53:14.822Z)
- OS Version: Windows_NT ia32 10.0.14393

Steps to Reproduce:

1. In any file, highlight the text to find.
2. Trigger the `actions.find` command with the keyboard (`Ctrl+F`).
3. Observe that the originally highlighted text is not highlighted in any way that distinguishes it as the active match or target for replacement.

Alternate Steps to Reproduce:

1. In any file, highlight the text to find.
2. Trigger the `editor.action.startFindReplaceAction` command with the keyboard (`Ctrl+H`).
3. Observe that the originally highlighted text is not highlighted in any way that distinguishes it as the active match or target for replacement.

Example of both Steps and Alternate Steps shown in the following gif. In the gif, you can see that whenever the Find/Find+Replace box is shown, the originally highlighted text is no longer marked as the active match, even though it is the active match.

This is a major problem because there is no way to confirm what the very first replacement will be.

![vscode-find-replace](https://cloud.githubusercontent.com/assets/623736/20039981/9d36c112-a41c-11e6-9b6c-7c8151d255c0.gif)
Repository: nvm
Issue: npm is only available in interactive shell
Labels: installing nvm: profile detection

It seems that nvm modifies `~/.bashrc` to load npm and node. This means that non interactive shells that don't run `bashrc` won't have it. For instance I was running a script that calls npm from cron and it was failing because npm was unknown.
Repository: rust
Issue: misleading error message for trait usage from external scope
Labels: C-enhancement,A-diagnostics,T-compiler

Trying to implement a trait for a type and then running a unit test produces the following misleading error message.

I tried this code:
```rust
use std::vec::Vec;
mod rlinq {
    trait RLINQ {
        fn is_where(&self);
    }

    impl RLINQ for Vec<String> {
        fn is_where(&self) {
            println!("{:?}", "Vec now implements where");
        }
    }
}
#[cfg(test)]
mod tests {
    use rlinq::RLINQ;
    #[test]
    fn is_where_test() {
        let x: Vec<String> = Vec::new();
        x.is_where();
    }
}
```

I expected to see this happen: Error letting me know trait was private and unable to be used in current scope

Instead, this happened: 
```
help: items from traits can only be used if the trait is in scope; the following trait is implemented but not in scope, perhaps add a `use` for it:
   = help: candidate #1: `use rlinq::RLINQ`
```
## Meta

`rustc --version --verbose`:
```
rustc 1.14.0-nightly (7c69b0d5a 2016-11-01)
binary: rustc
commit-hash: 7c69b0d5ae8c404c4847fa214126ebd7c0246277
commit-date: 2016-11-01
host: x86_64-unknown-linux-gnu
release: 1.14.0-nightly
LLVM version: 3.9
```
Backtrace: N/A
Repository: vscode
Issue: Option to render proper scrollbar
Labels: feature-request,ux

The current VS Code scrollbars (both vertical and horizontal) are OK, but I would prefer:
- The scrollbars not to disappear when the mouse stops moving
- The scrollbars to be a bit thicker, thus easier to click
- An up and a down arrow at the top and bottom (or left and right of the horizontal one) to scroll one line per click
- The scrollbar to jump "towards" where you click on empty space either side of the scroll handle, instead of the handle immediately jumping to where the mouse cursor is

In short, they should be like the Visual Studio 2015 scrollbars.  Maybe a setting could be introduced to render this more "traditional" type of scrollbar.
Repository: flutter
Issue: Document how to do animations at the render object layer
Labels: framework,a: animation,d: api docs,c: proposal,P3,team-framework,triaged-framework

(From Mikkel:)

> It may be helpful to document the preferred approach to animations [at the RenderObject/RenderBox layer].
>
> My current understanding is that when animations (i.e. the lerp values) can be described adequately at the Widget level, that's where animation should be implemented, by arranging for setState on some StatefulWidget to be called at each animation tick.
>
> But sometimes you seem to need to transform between the layouts calculated for your start and end widgets instead. The two cases we have seen involve size-dependent layouts containing animation-relevant information not present at the widget level. That moves animation code into the rendering layer, and it would be nice to have a standard way of accomplishing that.
Repository: go
Issue: cmd/link: generate manifest in EXE files
Labels: OS-Windows,NeedsFix,early-in-cycle

On Windows 10 and later, apps can opt into long path support by embedding an XML manifest:

https://msdn.microsoft.com/en-us/library/windows/desktop/aa365247(v=vs.85).aspx

We should automatically embed this manifest when linking EXE files. For prior art, see this third-party tool for generating a .syso file containing a manifest:

https://github.com/josephspurrier/goversioninfo/commit/f8c5d36fe414d924ff060440f22bf7103278b8b0#diff-88fef3681df95cd75f2ac89251404081
Repository: vscode
Issue: Remember undo history on hot-exit
Labels: feature-request,workbench-hot-exit,undo-redo

This will make restarts/crashes more and more seamless.
Repository: youtube-dl
Issue: [edge-apps.net] Site support request (used in ameracatv and TN.com.ar)
Labels: site-support-request

There some tv channels like this http://prepublish.f.qaotic.net/epa/ngrp:americatvlive-100056_all/chunklist_b596000.m3u8 , youtube dl send this messege: HTTP error 403 acces denied.Please find some way to download.Thanks
Repository: go
Issue: cmd/compile: implement duffcopy on ppc64x
Labels: Performance,NeedsDecision,compiler/runtime

implement duffcopy on ppc64x.

See TODO in code and stale pending CL https://go-review.googlesource.com/c/9684/ which could be resurrected.

/cc @minux @laboger
Repository: flutter
Issue: When InkWell has a GlobalKey and changes position, splash should not stop
Labels: framework,a: quality,f: gestures,P2,has partial patch,team-design,triaged-design
Repository: rust
Issue: demangled characters like { make debugging annoying
Labels: A-debuginfo,C-enhancement,T-compiler

(in gdb at least)

I think I complained to @tromey about this already and he proposed a solution, but I'm finding this more and more annoying when debugging that the '{' was chosen demangle to, for closures and impls, e.g.:

`_ZN4clap4args3arg3Arg5short28_$u7b$$u7b$closure$u7d$$u7d$17h7c4595ce99cae01bE`

demangles to:

`clap::args::arg::Arg::short::{{closure}}::h7c4595ce99cae01b`

as it breaks gdb's symbol auto-complete, it's hard to set a breakpoint, print the type, etc.

As I mentioned, I think @tromey suggested how to fix this with certain gdb expressions, or some work around, but I've already forgotten and maybe we should just make it demangle to something different ? :angel:
Repository: angular
Issue: Touch and change event is not propagated to ngModelGroup and ngForm if using separate component
Labels: workaround1: obvious,freq1: low,area: forms,type: use-case,P4

<!--
IF YOU DON'T FILL OUT THE FOLLOWING INFORMATION WE MIGHT CLOSE YOUR ISSUE WITHOUT INVESTIGATING
-->

**I'm submitting a ...**  (check one with "x")
```
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see https://github.com/angular/angular/blob/master/CONTRIBUTING.md#question
```

**Current behavior**
Only form control is marked as touched and changed

**Expected behavior**
The complete hierarchy should be informed

**Minimal reproduction of the problem with instructions**
http://plnkr.co/edit/2Bj5M2zwOG3vXTauZ9GP?p=preview

**What is the motivation / use case for changing the behavior?**
<!-- Describe the motivation or the concrete use case -->

**Please tell us about your environment:**
<!-- Operating system, IDE, package manager, HTTP server, ... -->

* **Angular version:** 2.1.2

* **Browser:** Chrome 53
 
* **Language:** TypeScript 2
Repository: rust
Issue: support `default impl` for specialization
Labels: C-enhancement,A-trait-system,T-compiler,A-specialization,E-medium,F-specialization

rust-lang/rfcs#1210 included support for `default impl`, which declared a set of defaults that other impls can inherit (if they correspond to a subset of the default impl's types). This was intended as a replacement/improvement of the existing default methods that appear in trait bodies (it was also originally called `partial impl`).  Unfortunately, this feature is not yet implemented!

Some things to be sure of:

- [ ] a `default impl` need not include all items from the trait
- [ ] a `default impl` alone does not mean that a type implements the trait
- [x] all items in a `default impl` are (implicitly) `default` and hence specializable

cc https://github.com/rust-lang/rust/issues/31844
Repository: go
Issue: time: optimize time.Time.Sub and time.Since
Labels: Performance,help wanted

According to a profiling data from a large number of servers at Google, `time.Time.Sub` is within the top 150 functions by CPU time. The current implementation of `Sub` is not inlineable. The logic for [`Sub` is not particularly complicated](https://github.com/golang/go/blob/2c5bce1cfa242f27ffece3d30e8b851e3e923be2/src/time/time.go#L631-L642) and I believe it can be made to be inlined with some love.
Repository: go
Issue: net/url: optimize unescape and escape
Labels: Performance,help wanted

According to a profiling data from a large number of servers at Google, URL related functions are among the top 150 functions by CPU time. The simplest optimizations I see are around using LUTs in `shouldEscape`, `isHex`, and `unHex`.
Repository: go
Issue: x/review/git-codereview: handle many-parent merges in hooks
Labels: NeedsInvestigation

When running the git-codereview hooks on a commit that is (soon to be a child of) a merge of three branches, I get:

```bash
$ git commit
git branch -a --contains 3c49c114ba9e7292b342230a0b238d4684157d2b fce0592992d48cbd5e7ca44724f3f14ee25c0f05
error: malformed object name 3c49c114ba9e7292b342230a0b238d4684157d2b fce0592992d48cbd5e7ca44724f3f14ee25c0f05

...
git-codereview: exit status 129
```

Low priority to fix, but an annoyance.
Repository: go
Issue: cmd/compile: elide branches that must be taken
Labels: Performance,compiler/runtime

Consider the following two benchmarks:
```go
var sink int
var A, B, C = 1, 2, 3

func BenchmarkA(b *testing.B) {
	for i := 0; i < b.N; i++ {
		var m int
		if A != 0 && B+1 <= C {
			m = 0
		} else {
			panic("never called")
		}
		sink = m
	}
}

func BenchmarkB(b *testing.B) {
	for i := 0; i < b.N; i++ {
		var m int
		if A != 0 && B+1 <= C {
			m = 0
		} else {
			m = -1
		}
		if m == -1 {
			panic("never called")
		}
		sink = m
	}
}
```
On Go 1.8, version A is 25% ~~slower~~ faster than version B. The reason for this is because the compiler actually sets `m = -1` and then does the subsequent conditional check `m == -1`. The compiler should be able to prove that this unnecessary.

Code that looks like this commonly occurs when a smaller inlineable helper is used for a quick path and then a conditional is used to check for a fallback. For example:
```go
m := tryMethod()
if m != -1 {
	m = slowMethod()
}
```

This came up in http://golang.org/cl/32921.

/cc @bradfitz @randall77
Repository: TypeScript
Issue: [tsserver] CompileOnSaveEmitFile should returns emited files (js, map, d.ts) path
Labels: Suggestion,API,Awaiting More Feedback

I'm consumming "CompileOnSaveEmitFile " with tsserver inside Eclipse. It works great and improve a lot of performance (before I called tsc when for each save of file and performance was not very good).

In Eclipse, when a file is created outside Eclipse (like tsserver generates js, map files), the files doesn't appear in the Navigator files of the Eclipse. We must do a refresh at hand. 

It should be very cool if CompileOnSaveEmitFile could returns the file path for each generated files. With this feature I could refresh those files inside Eclipse.
Repository: go
Issue: hash/adler32: optimize on 64-bit architectures
Labels: Performance,NeedsFix

### What version of Go are you using (`go version`)?
go version go1.7 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/dev/gocode"
GORACE=""
GOROOT="/opt/local/lib/go"
GOTOOLDIR="/opt/local/lib/go/pkg/tool/darwin_amd64"
CC="/usr/bin/clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"

### What did you do?
$ cd /opt/local/lib/go/src/hash/adler32
$ go test -bench .
BenchmarkAdler32KB-8   	 3000000	       519 ns/op	1971.24 MB/s
PASS

### What did you expect to see?
64bit version written in C easily delivers >4GB/s on this machine

### Solution
Use 64bit arithmetics via unsafe.Pointer casting.
NOTE: usage of C style indexing instead of slices in the inner loop also gives a small ~5-10% boost.

### Result
~1.8x improvement if implemented with 64bit arithmetics.
$ go test -bench .
BenchmarkAdler32KB-8   	 5000000	       289 ns/op	3531.85 MB/s
PASS
Repository: youtube-dl
Issue: please support NSFW site stream-mydirtyhobby.biz
Labels: site-support-request,nsfw

```
> youtube-dl http://stream-mydirtyhobby.biz/view/destroyed89-riskantes-pinkeln/i7l7T
[generic] i7l7T: Requesting header
WARNING: Falling back on generic information extractor.
[generic] i7l7T: Downloading webpage
[generic] i7l7T: Extracting information
ERROR: Unsupported URL: http://stream-mydirtyhobby.biz/view/destroyed89-riskantes-pinkeln/i7l7T

```

example URL http://stream-mydirtyhobby.biz/view/destroyed89-riskantes-pinkeln/i7l7T

thanks
Repository: youtube-dl
Issue: Unable to download orlandosentinel.com's embedded Flash brightcove videos.
Labels: site-support-request

Example:

$ youtube-dl -v http://www.orlandosentinel.com/features/kayla-obrien/91839398-132.html
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-v', 'http://www.orlandosentinel.com/features/kayla-obrien/91839398-132.html']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2016.11.08.1
[debug] Python version 3.4.2 - Linux-3.16.0-4-amd64-x86_64-with-debian-8.6
[debug] exe versions: ffmpeg 3.1.4-1, ffprobe 3.1.4-1, rtmpdump 2.4
[debug] Proxy map: {}
[generic] 91839398-132: Requesting header
WARNING: Falling back on generic information extractor.
[generic] 91839398-132: Downloading webpage
[generic] 91839398-132: Extracting information
ERROR: Unsupported URL: http://www.orlandosentinel.com/features/kayla-obrien/91839398-132.html
Traceback (most recent call last):
  File "/home/ant/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 694, in extract_info
    ie_result = ie.extract(url)
  File "/home/ant/bin/youtube-dl/youtube_dl/extractor/common.py", line 357, in extract
    return self._real_extract(url)
  File "/home/ant/bin/youtube-dl/youtube_dl/extractor/generic.py", line 2433, in _real_extract
    raise UnsupportedError(url)
youtube_dl.utils.UnsupportedError: Unsupported URL: http://www.orlandosentinel.com/features/kayla-obrien/91839398-132.html


Thank you in advance. :)
Repository: flutter
Issue: Having to manage both ends of an AnimationController -> CurvedAnimation chain is a pain
Labels: framework,a: animation,a: quality,c: proposal,P3,team-framework,triaged-framework

Maybe we should have an object that wraps the two so you can add/remove listeners, forward/reverse, set value, and dispose all at the same time with one object.
Repository: opencv
Issue: Different outcomes when using StereoBM and cuda::StereoBM with same parameters.
Labels: bug,category: calib3d,category: gpu/cuda (contrib)

##### System information (version)
- OpenCV => 3.1 (head)
- Windows 10 32+64 Bit
- Compiler => Visual Studio 2013

##### Detailed description

Im currently testing various stereo reconstruction implementations of OpenCV and i get different results for when using the OpenCL version of StereoBM and the cuda version StereoBM. Both should implement the same algorithm, right? Shouldnt they then output the same results given the same input parameters?

For instance when setting the textureThreshold, both implementations seem to use it differently.

The documentation of StereoMatching algorithms totally lack in any information about the parameters and their intended function. In Version 2.4 those parameters are described better.


##### Steps to reproduce

int numberOfDisparities = 256;
int textureThreshold = 10;
int blockSize = 21;


cv::Ptr<cv::StereoBM> bm = cv::StereoBM::create();
bm->setNumDisparities(256);
bm->setTextureThreshold(textureThreshold);
bm->setBlockSize(blockSize);

cv::Ptr<cv::cuda::StereoBM> bmCUDA = cv::cuda::createStereoBM(numberOfDisparities, blockSize);
bmCUDA->setTextureThreshold(textureThreshold);
Repository: You-Dont-Know-JS
Issue: You Don't Know JS: Async & Performance Chapter 4: Generators Thunks
Labels: for second edition

It's slightly off topic, but, when discussing 'Thunks' as a concept, and speaking about 'thunkory' functions might it be useful to note that this is essentially the same concept as partial function application?
Not like, a paragraph or anything, but just a short blurb to point out that these terms are more or less the same thing.
It might help folks connect the dots in the future when researching other topics, and it's a pretty useful concept for organizing all kinds of code, not just for organizing asynchronous code.
Repository: TypeScript
Issue: Suggestion: Allow Types in JS Files
Labels: Suggestion,Awaiting More Feedback

I have a large Angular2 project written in JavaScript, in which we use [transform-flow-strip-types](https://babeljs.io/docs/plugins/transform-flow-strip-types/) in order to allow Angular2 dependency injection to work from JavaScript.

This is currently an error in a JavaScript file compiled with `tsc`:
```
class MyClass {
  prop:string;                   //err: 'types' can only be used in a .ts file
  constructor(x: string) {       //err: 'types' can only be used in a .ts file
  }

  someMethod():string {        //err: 'types' can only be used in a .ts file
  }
}
```

I would like an `allowTypesInJsFiles` compiler option which would allow us to compile JavaScript files which contain types. All of the above errors would be valid if the flag is set. This would allow us to use Angular2 DI from JavaScript files.

I have a [branch](https://github.com/Microsoft/TypeScript/commit/57a69d880251088d550500dcb018fc4e53895916) with a simple implementation which I have verified to work.

Is this a change which could be accepted? If so I will add tests and sign the CLA.

I think it would be very useful for people migrating large JavaScript projects to TypeScript.
Repository: youtube-dl
Issue: Use OAuth tokens instead of .netrc where possible
Labels: 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.11.08.1*. 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.11.08.1**

### 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)
- [ ] Site support request (request for adding support for a new site)
- [x] Feature request (request for a new functionality)
- [ ] Question
- [ ] Other

---

### The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your *issue*



### If the purpose of this *issue* is a *site support request* please provide all kinds of example URLs support for which should be included (replace following example URLs by **yours**):
- Single video: https://www.youtube.com/watch?v=BaW_jenozKc
- Single video: https://youtu.be/BaW_jenozKc
- Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

---

Currently, the only method to automatically authenticate youtube-dl requests is with a cleartext .netrc file. This is obviously extremely insecure, and I think it would be much better if youtube-dl requested OAuth tokens from supported sites instead.

Alternatively, allow a command to be run before and after accessing .netrc, which would allow the user to encrypt and decrypt the file as they see fit.
Repository: rust
Issue: Overflow for trait implemented recursively on references
Labels: A-type-system,T-lang,T-compiler,C-bug,T-types

Consider this snippet [[playpen](https://play.rust-lang.org/?gist=6c8dc5378a2b2c3401061024e8937b3f&version=nightly&backtrace=0)]:

```rust
trait Foo {}

impl<'a> Foo for &'a usize {}

// Ok when this impl is commented-out.
impl<'a, T> Foo for &'a Vec<T> where &'a T: Foo {}

fn foo<T>(_: T) where for<'a> &'a T: Foo {}

fn main() {
    foo(0);
}
```

```
error[E0275]: overflow evaluating the requirement `_: std::marker::Sized`
  --> <anon>:11:5
   |
11 |     foo(0);
   |     ^^^
   |
   = note: consider adding a `#![recursion_limit="128"]` attribute to your crate
   = note: required because of the requirements on the impl of `for<'a> Foo` for `&'a std::vec::Vec<_>`
   = note: required because of the requirements on the impl of `for<'a> Foo` for `&'a std::vec::Vec<std::vec::Vec<_>>`
   = note: required because of the requirements on the impl of `for<'a> Foo` for `&'a std::vec::Vec<std::vec::Vec<std::vec::Vec<_>>>`
   [ and so on... ]
   = note: required by `foo`
```

There is no issue when trait implementation for `&'a Vec<T>` is commented-out, even though `Vec` isn't actually used anywhere when calling `foo` method.