sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: database/sql: think about a std error / informational messages
Labels: Thinking

Many databases support returning multiple errors, with each error containing a line number, text, and numeric code. I want to explore what it would take to define a standard error interface that could be used to unpack this information. A concrete use case is to 1000 line SQL block that errored, then show just the context of the portions where it errored. A concrete use case for getting the error code is when a user raises a user error, it is often useful to distinguish this from a SQL syntax error. You may wish to abort a query early within the SQL text and only in that condition expose the user to the underlying message, but in the case of generic SQL errors, hide or retry them.

In a similar way, Postgresql, MS SQL, and Oracle all support the concept of streaming messages while the query is running. This is less of an issue, but still might be useful. Fewer concrete uses for messages in SQL.
Repository: react
Issue: Define specific browser support guidelines
Labels: Component: DOM,Type: Discussion

As it stands the only real solid guideline we have for browser support is that we support IE9+. But there are so many other outdated browser versions that are unreasonable to support or worry about.

It would be useful if we had more specific guidelines on what browsers we should target. Does Safari on iOS 3 matter? FireFox 4? You get the idea.

If we had a well-documented range of browser version that we *know* should work it would make it a lot easier to run through our DOM fixtures in BrowserStack and know we're safe.

Maybe the internal core team can work internally with analytics to see what browsers Facebook needs to support and work forward from there?

cc @gaearon @nhunzaker
Repository: angular
Issue: AbstractControl.valueChanges event fires without respect on current disabled/enabled state
Labels: freq1: low,area: forms,type: confusing,forms: Controls API,P4


**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**
Every time when you call `.disable()` or `.enable()` method the `.valueChanges` observalbe triggers new value even the current state of input is already disable or enabled.

**Expected behavior**
It shouldn't trigger if there's no actual changes in value

**Minimal reproduction of the problem with instructions**
Here's the Plunk that illustrates the problem
https://plnkr.co/edit/kvVKi2?p=preview

Check the console and you'll see incredible amount of `changes` messages ended with Stack size error.

**What is the motivation / use case for changing the behavior?**
The Plunk illustrates trivial example of course. I came across during the making of bigger form where I need to listen form changes and disable some of controls based on it. Yes, you can listen individual controls, but this case doesn't work everywhere.  I believe that Angular need to be smart in that situations and doesn't trigger changes event unless something really changed.

**Please tell us about your environment:**
MacOS, npm

* **Angular version:** 
checked both in 2.4.8 and 4.0.1

* **Browser:** all
 
* **Language:** all
Repository: vscode
Issue: Allow project specific keybindings
Labels: feature-request,keybindings

same as the settings.json, I should be able to specify a local keybindings.json that overrides user keybindings and global keybindings. 

A prompt alerting the user to the override and asking which to use might be appropriate.
Repository: go
Issue: cmd/internal/obj/arm: should not put instruction encoding into Reloc.Add
Labels: NeedsFix

Please answer these questions before submitting your issue. Thanks!

### What version of Go are you using (`go version`)?

### What operating system and processor architecture are you using (`go env`)?

Whatever the trybots for linux-arm are using

### What did you do?

Built a sample file with -N -l then attempted to load it with cmd/internal/goobj:

https://go-review.googlesource.com/#/c/39230/

### What did you expect to see?

honest object files.

### What did you see instead?

https://storage.googleapis.com/go-build-log/e81c4c0b/linux-arm_e2ed47ce.log
Repository: flutter
Issue: Unify doctor checks and code that locates binary artifacts (executables, sdks)
Labels: tool,t: flutter doctor,a: first hour,P3,team-tool,triaged-tool

A lot of the tool presence/version checks we perform in `flutter doctor` would be useful to perform before steps that involve invocation of those tools.

Examples from the iOS side include checking Xcode and CocoaPods presence/version and presence of the Python six module before iOS build/run.

One option would be to factor out a tool class that has the ability to check presence/version of the tool and how to run it, then use this in the build step and in doctor.
Repository: go
Issue: cmd/compile: avoid copying if copied value isn't modified
Labels: Performance,compiler/runtime

Go 1.8 emits unnesessary value copyings in the following code (didn't test go tip yet, but it is likely it has the same behaviour):

```go
func sum(p *[1000][1000]int) int {
  s := 0
  a := *p // copy a million of ints
  for i := range a { // possible copy
      b := a[i] // copy a thousand of ints
      for j := range b { // possible copy
          s += b[j]
      }
  }
  return s
}
```

All these copyings could be substituted by pointers, which would result in direct memory access pointed by `p`. This looks like safe optimization, since both `a` and `b` values are read-only in this code.
This optimization also could save stack space used for storing `a` and `b` values.

Someone could argue this optimization can't be applied since the memory pointed by `p` could be modified by concurrently running goroutines, which will result in data races. But this code is already vulnerable to the same data races when compiled without the optimization, since value copying isn't atomic.
Repository: go
Issue: cmd/compile: share statictmp
Labels: Performance,compiler/runtime

Hello,

i have 2 file,
one ==> a go file with 10000 time repeat : `fmt.Print("...........\n")`
two ==> a c(gcc) file with 10000 time repeat : `printf("...........\n");`


size of output c is about 200kb.
size of output go is 50mb.(also long time need for build...)
Repository: opencv
Issue: Locality Preserving Projection (LPP)
Labels: feature,category: core

Their appears not to be a built in LPP functions like PCA or LDA!

Locality Preserving Projection (LPP) are described here: https://newtraell.cs.uchicago.edu/files/tr_authentic/TR-2002-09.pdf

My motivation behind this is to facilitate the writing of a Laplacianfaces implementation as described here: http://scholarpedia.org/article/Laplacianfaces in the contrib face module! https://github.com/opencv/opencv_contrib/tree/master/modules/face
Repository: vscode
Issue:  Line height and font size are too large by default
Labels: feature-request,editor-core,config

<!-- Do you have a question? Please ask it on http://stackoverflow.com/questions/tagged/vscode -->

- VSCode Version: 1.10.2
- OS Version: win10 x64
My monitor native resolution is 1920x1200

<br>

When opening any file in VSCode, it shows only **56 lines** at any time.
I feel that the current line height and font size are too much by default.

_For reference:_

```
Sublime Text 3: 71 lines 
 Notepad++ 6.9: 65 lines
      gVim 7.4: 66 lines 
  UltraEdit 23: 67 lines
```
_In contrast, Atom 1.15 displays even less: 49 lines_

<br>

By default VSCode has in its settings:

```
"editor.fontSize":  14,    // Controls the font size in pixels.
"editor.lineHeight": 0     // Controls the line height. Use 0 to compute the lineHeight from the fontSize.
```

If I set them to **13** and **15** respectively
then the visible lines area becomes on par with the other editors: 71 lines.

<br>

Also, _(while using the default `Dark+` theme
and the default setting `  "editor.fontFamily": "Consolas, 'Courier New', monospace"`)_
with my suggested settings, 13pt Consolas *I think* it is appears a bit boldish.
_Changing the value`"editor.fontWeight": "normal"` to anything below 400 doesn't make a difference. (related issue: #381)_
That's why I believe that, in conjunction with the above two suggested changes, 
it would be even better to switch to a slightly different font that would support setting `fontWeight` below 400 e.g. something like "Consolas Light" -I know that this font currently doesn't exit- )

<br>

**So, my suggestion is to change the default values of these settings to 13 and 15 respectively, 
plus lower the `fontWeight` value somehow**


-----
Here is VSCode with its default settings:
[screenshot 682](https://cloud.githubusercontent.com/assets/723651/24582811/7b7b0e24-1740-11e7-92d0-b488b89e6de9.png)



And here is comparison of VSCode with my suggested settings vs Sublime Text 3:

[screenshot 678](https://cloud.githubusercontent.com/assets/723651/24582818/a8b5cb2c-1740-11e7-95de-ee34554b7b72.png)
[screenshot 678b](https://cloud.githubusercontent.com/assets/723651/24582821/bfba6350-1740-11e7-8451-9aaec3773127.png)
Repository: neovim
Issue: 'guicursor' does not reflect guifg/gui highlights
Labels: bug,ui,complexity:low,options

- `nvim --version`: NVIM v0.2.0-1147-g58422f17d

- Vim (version: ) behaves differently? N/A
- Operating system/version: Archlinux 
- Terminal name/version: gnome-terminal
- `$TERM`: gnome-256color

### Actual behaviour

The cursor does not seem to pick up `guifg` and `gui` settings. Also the original cursor settings
do not seem to get restored after exiting Neovim.

I've set up a minimal `init.vim` that sets the cursor color to green and foreground to black as
well as `gui=reverse` all except the background color get ignored. The text remains white which is unfortunate of the cursor itself is.

When exiting Neovim the cursor color in the terminal suddenly has turned green as well. 
```
nvim -u NORC

set guicursor=a:block-blinkon100-Cursor/Cursor
hi Cursor guifg=black guibg=green gui=reverse

```
Repository: youtube-dl
Issue: [Feature Request] Change playback speed of downloaded files
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 *2017.04.02*. 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 **2017.04.02**

### 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

---



### Dear developers of youtube-dl,



In the first place, thank you for making this excellent tool. It has enabled me to listen to quality audio during my commutes.

I'd just like to ask if it is possible to include an option to save the video/audio in a faster speed, thus mimicking youtube's function of increasing the playback speed of their videos.

 I thought of helping with this, but I'm quite a beginner. I've been learning to code for only 3 months (with python 3), so I don't know if I could be of any help. If you think there is anything I can do, just let me know.
Repository: flutter
Issue: Add a Theme.merge that implements the logic of the main ThemeData constructor
Labels: c: new feature,framework,f: material design,customer: posse (eap),c: proposal,P2,team-design,triaged-design

"changing the accent color on an input on one screen is complicated" was feedback offered about our theming system.

Ian says on solution here would be to add a copyWith constructor onto theme?

Did I capture that correctly @Hixie?
Repository: go
Issue: all: test that all examples run on golang.org, i.e. the playground
Labels: Testing,help wanted,NeedsFix

Right now, examples are only run via `go test` by the trybots. That checks whether or not they run as tests, but it doesn't check if they run in the playground.

For instance, see #19823. That example depends on its own test file, `example_test.go`. That will be available if one runs `go test`, but not if one presses "Run" in https://golang.org/pkg/go/parser/#example_ParseFile which will run it in the playground.

There should be a mechanism to ensure that all of the examples run on the golang.org docs too. In other words, that they pass when executed in the playground.
Repository: pytorch
Issue: Batched sparse QR factorizations and solves with cusolver
Labels: module: sparse,feature,triaged,module: linear algebra,Stale

These should significantly speed up [qpth](https://github.com/locuslab/qpth) in some cases when sparse matrices can be used. Here is a [cusolver reference](http://docs.nvidia.com/cuda/cusolver/#cusolver-lt-t-gt-csrqrbatched) on these functions. This issue depends on the other core sparse functions being added (I think at least PR #1147, please add other dependencies here so I'll know when I can start) so I won't immediately start trying to add these. Also please post a reference if a good cusolver wrapper usage example gets added that we can use as a reference for wrapping these QR functions.

Following the naming conventions of the batched LU factorizations and solves of `btrifact` and `btrisolve`, I propose `bqrfact` and `bqrsolve` for the names and for now we can just implement them for sparse tensors.

\cc @nikitaved @pearu @cpuhrsch @jianyuh @mruberry @walterddr @IvanYashchuk @xwang233 @Lezcano @vincentqb @vishwakftw @SsnL @zkolter
Repository: angular
Issue: Platform Server - Attach cookies to HTTP requests
Labels: type: bug/fix,freq2: medium,area: server,area: common/http,P3

<!--
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**
<!-- Describe how the bug manifests. -->
Currently I see no option to attach cookies to a http request when the application runs on server platform. 
Using Universal + angular v2.x.x allowed setting 'Cookie' header property manually, in the current version I'm getting the following warning: 
`Refused to set unsafe header "Cookie"` ([node-xhr2](https://github.com/pwnall/node-xhr2) library logs this warning, as it disallows setting cookie header).

**Expected behavior**
It should be possible to pass cookies to a http request. 
[It was already asked in the roadmap](https://github.com/angular/angular/issues/13822#issuecomment-289734558) but no answer or suggestion was provided. 

**Minimal reproduction of the problem with instructions**
<!--
If the current behavior is a bug or you can illustrate your feature request better with an example, 
please provide the *STEPS TO REPRODUCE* and if possible a *MINIMAL DEMO* of the problem via
https://plnkr.co or similar (you can use this template as a starting point: http://plnkr.co/edit/tpl:AvJOMERrnz94ekVua0u5).
-->

**What is the motivation / use case for changing the behavior?**
<!-- Describe the motivation or the concrete use case -->
Without attaching cookies to a http request, the response on the server is different than the response on the client. It also makes the usage of transfer state dangerous, since the browser will not fire the http request and will use the response received from the server.

**Please tell us about your environment:**
<!-- Operating system, IDE, package manager, HTTP server, ... -->

* **Angular version:** 4.0.0
<!-- Check whether this is still an issue in the most recent Angular version -->

* **Browser:** [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]
<!-- All browsers where this could be reproduced -->
 
* **Language:**: TypeScript 2.2.2
Repository: opencv
Issue: Crash when calling imencod with exr from multiple threads
Labels: bug,priority: low,category: imgcodecs

<!--
If you have a question rather than reporting a bug please go to http://answers.opencv.org where you get much faster responses.
If you need further assistance please read [How To Contribute](https://github.com/opencv/opencv/wiki/How_to_contribute).

This is a template helping you to create an issue which can be processed as quickly as possible. This is the bug reporting section for the OpenCV library.
-->

##### System information (version)
<!-- Example
- OpenCV => 3.1
- Operating System / Platform => Windows 64 Bit
- Compiler => Visual Studio 2015
-->

- OpenCV => 3.1
- Operating System / Platform => Windows 64 Bit
- Compiler => Visual Studio 2013 (x86)

##### Detailed description

When calling imencode with ".exr" as the format from multiple threads there is a null reference access violation.
An example exception callstack:

 	ntdll.dll!_RtlEnterCriticalSection@4()	Unknown
	opencv_world310d.dll!IlmThread::Mutex::lock() Line 62	C++
 	opencv_world310d.dll!IlmThread::Lock::Lock(const IlmThread::Mutex & m, bool autoLock) Line 122	C++
 	opencv_world310d.dll!IlmThread::ThreadPool::numThreads() Line 343	C++
 	opencv_world310d.dll!Imf::globalThreadCount() Line 51	C++
 	opencv_world310d.dll!cv::ExrEncoder::write(const cv::Mat & img, const std::vector<int,std::allocator<int> > & __formal) Line 616	C++
 	opencv_world310d.dll!cv::imencode(const cv::String & ext, const cv::_InputArray & _image, std::vector<unsigned char,std::allocator<unsigned char> > & buf, const std::vector<int,std::allocator<int> > & params) Line 621	C++
 	test_opencv_vc12.exe!main::__l7::<lambda>() Line 20	C++
 	[External Code]	

I believe the issue is non atomic initialization of the local static variable in opencv-3.1.0\3rdparty\openexr\IlmThread\IlmThreadPool.cpp, line 443:

```.cpp
ThreadPool&
ThreadPool::globalThreadPool ()
{
    //
    // The global thread pool
    //

    static ThreadPool gThreadPool (0);

    return gThreadPool;
}
```

For VC12 (and other compilers) the magic static C++11 feature is not yet implemented, so the initialization of global thread pool is not done atomically. Suggest implementing a thread safe singleton there...

##### Steps to reproduce

<!-- to add code example fence it with triple backticks and optional file extension
    ```.cpp
    // C++ code example
    ```
 or attach as .txt or .zip file
-->
```.cpp
#include <thread>
#include <array>

#include "opencv/cv.hpp"


int main()
{
    const int nThreads = 10;
    std::array<std::thread, nThreads> threads;
    for (int i = 0; i < nThreads; ++i)
    {
        threads[i] = std::thread([]() 
        {
            for (int j = 0; j < 1000; ++j)
            {
                cv::Mat m(100, 200, CV_32FC1);
                cv::randn(m, cv::Scalar(23.4), cv::Scalar(4.2));
                std::vector<unsigned char> res;
                cv::imencode(".exr", m, res);
            }
        });
    }
    for (auto& t : threads)
    {
        t.join();
    }

}
```
Repository: go
Issue: x/mobile: reverse and swift
Labels: mobile

Please answer these questions before submitting your issue. Thanks!

### What version of Go are you using (`go version`)?
1.8


### What operating system and processor architecture are you using (`go env`)?

x-MacBook-Pro:gosync apple$ go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/apple/workspace/go"
GORACE=""
GOROOT="/usr/local/opt/go/libexec"
GOTOOLDIR="/usr/local/opt/go/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/wp/ff6sz9qs6g71jnm12nj2kbyw0000gp/T/go-build775985182=/tmp/go-build -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"
PKG_CONFIG="pkg-config"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"


### What did you do?

More a question. Gomobile reverse works well for Android, but i was wondering how to do it for swift.


### What did you expect to see?


### What did you see instead?
Repository: rust
Issue: LUB of a bivariant parameter is not fully general
Labels: I-needs-decision,T-compiler,C-bug

In https://github.com/rust-lang/rust/pull/40570, I removed bivariance somewhat aggressively. One area where we are not *quite* doing the right thing, it appears, is when computing the LUB/GLB of a bivariant type parameter. The code currently just picks one side, which seems clearly wrong, but I was [not able to craft a test case exploiting that](https://github.com/rust-lang/rust/pull/40570#issuecomment-288768167). Therefore, opening an issue to revisit later if we can get a concrete test case.

[This comment explains my preferred fix](https://github.com/rust-lang/rust/pull/40570#issuecomment-287307919):

> I think you are correct about the eventual result but I wouldn't expect it to be derived immediately from the LUB computation. Since the last parameter is considered "bivariant" (i.e., derived from constraints on the others), I'd expect that the last parameter simply gets a fresh variable, and that trait selection will constrain it (ultimately) to be 'c. In other words, the result from LUB itself would be Foo<fn() -> &'?0 u32, '?1> where 'a: '?0 and 'b: '?0, but enforcing the WF requirements would ultimately constraint '?1 = '?0.
>
> However, it occurs to me that I'm not entirely sure where these WF requirements would get enforced. Much of the code assumes that the resulting types from LUB operations etc are well-formed, and hence I'm not sure when we would add such an obligation. (One possibility is to have the LUB/GLB operation itself produce an obligation, at least if bivariance is involved, given that this is now a possibility.)

<!-- TRIAGEBOT_START -->

<!-- TRIAGEBOT_ASSIGN_START -->

<!-- TRIAGEBOT_ASSIGN_DATA_START$${"user":null}$$TRIAGEBOT_ASSIGN_DATA_END -->

<!-- TRIAGEBOT_ASSIGN_END -->
<!-- TRIAGEBOT_END -->