sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: opencv
Issue: Create Non-Transparent OpenCL API for better GPU / Accelerators utilization
Labels: category: ocl,RFC

I am sorry in advance if this isn't the right place for feature-requests / suggestions. This is an important issue though.

OpenCV contains the most extensive OpenCL implementations for computer vision algorithms. Many of them contain very well written kernels.
 
Yet, every time I take a look inside the OpenCV 'Transparent API' implementation I am horrified by the lack of basic efficiency with regard to properly utilizing the GPU. 

For instance, OpenCV Canny performs at least 3 large and synchronous clBuffer allocations which totally stall the GPU pipeline and cannot be avoided by simply passing pre-allocated buffers for intermediate results.

Another example are algorithms which download results From GPU back to RAM even when it is not necessarily needed, thus stalling the GPU. 

These kind of redundant memory allocations and memory transfers really goes against the whole **asynchronous** GPU programming paradigm. Which strives for saturating the GPU with the maximum amount of computation work.
 
On many occasions, the OpenCV CPU implementation easily outperform the GPU implementation simply because of the stated reasons.

I understand the rational behind the transparent API, but it seems like all the wonderful OpenCL kernels written are simply wasted.

What is needed is a **non-transparent API**, one that allows the client to pass his/her own buffers, invokes OpenCL kernels and let the user decide if and when to perform memory transfers.

This paradigm is implemented by libraries such as CLFFT and CLBlast.
Repository: vscode
Issue: Support /mnt WSL links in windows not using Remote WSL
Labels: help wanted,feature-request,windows,WSL,terminal-links

<!-- ⚠️⚠️ Do Not Delete This! bug_report_template ⚠️⚠️ -->
<!-- Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct/ -->
<!-- Please search existing issues to avoid creating duplicates. -->
<!-- Also please test using the latest insiders build to make sure your issue has not already been fixed: https://code.visualstudio.com/insiders/ -->

<!-- Use Help > Report Issue to prefill these. -->
- VSCode Version: 1.44.2
- OS Version: Windows 10 1909

Using a Windows terminal for the integrated terminal system such as Powershell lets you jump to places in code logged by errors by holding ctrl and clicking on the directory. However, using WSL, you can't do this because your Windows filesystem is mounted so it shows `/mnt/` before the directory. I think this can be fixed with some Regex and I would appreciate if it were fixed because I find the code jumping convenient.
Repository: tensorflow
Issue: tf.io.gfile.listdir is inconsistent between GCS dir and local dir - adds trailing slashes
Labels: stat:awaiting tensorflower,type:bug,comp:ops,TF 2.7


**System information**
- TensorFlow version (use command below): v2.0.0-rc2-26-g64c3d38 2.0.0
- Python version: 3.6

**Describe the current behavior**

When listing the directory items for a GCS dir, listdir returns names with trailing slashes. 
When listing the directory items for a local dir, listdir returns names without trailing slashes. 

**Describe the expected behavior**

The behavior needs to be consistent.

**Standalone code to reproduce the issue**

```
tensorflow.io.gfile.listdir('gs://bucket/dir')
>>> ['eval/', 'train/']

tensorflow.io.gfile.listdir('..')
>>> ['eval', 'train']
```
Repository: youtube-dl
Issue: gigtv.com.au
Labels: site-support-request

<!--

######################################################################
  WARNING!
  IGNORING THE FOLLOWING TEMPLATE WILL RESULT IN ISSUE CLOSED AS INCOMPLETE
######################################################################

-->


## Checklist

<!--
Carefully read and work through this check list in order to prevent the most common mistakes and misuse of youtube-dl:
- First of, make sure you are using the latest version of youtube-dl. Run `youtube-dl --version` and ensure your version is 2020.05.03. If it's not, see https://yt-dl.org/update on how to update. Issues with outdated version will be REJECTED.
- Make sure that all provided video/audio/playlist URLs (if any) are alive and playable in a browser.
- Make sure that site you are requesting is not dedicated to copyright infringement, see https://yt-dl.org/copyright-infringement. youtube-dl does not support such sites. In order for site support request to be accepted all provided example URLs should not violate any copyrights.
- Search the bugtracker for similar site support requests: http://yt-dl.org/search-issues. DO NOT post duplicates.
- Finally, put x into all relevant boxes (like this [x])
-->

- [x] I'm reporting a new site support request
- [x] I've verified that I'm running youtube-dl version **2020.05.03**
- [x] I've checked that all provided URLs are alive and playable in a browser
- [x] I've checked that none of provided URLs violate any copyrights
- [x] I've searched the bugtracker for similar site support requests including closed ones


## Example URLs

<!--
Provide all kinds of example URLs support for which should be included. Replace following example URLs by yours.
-->

- Single video: https://webcast.gigtv.com.au/Mediasite/Showcase/vala2020/Presentation/46f03de9b50442788fc5fcc370534bee1d


## Description

Plays OK in Firefox, Safari, and Chrome on a Mac
Repository: rust
Issue: Better diagnostics when user forgets to wrap code in `main` function
Labels: C-enhancement,A-diagnostics,A-parser,T-compiler

```rust
// some_third_party_crate/lib.rs

mod some_third_party_crate {
    pub fn show() {}
}

//////////////////////////////////////////
//////////////////////////////////////////
//////////////////////////////////////////

// main.rs

some_third_party_crate::show();
```

https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=8fca04bfa35f22c6776578d6c9fd8fbc

<img width="608" alt="Screen Shot 2020-05-04 at 5 46 43 PM" src="https://user-images.githubusercontent.com/416575/81016859-3dd1fb00-8e2f-11ea-9abd-f84ba387d3d7.png">

We should suggest the user wrap their code in `fn main() { ... }`
Repository: kubernetes
Issue: DELETECOLLECTION doesn't always
Labels: kind/bug,sig/api-machinery,lifecycle/frozen

**What happened**:

Deleting a large collection times out (504) and leaks a goroutine that's busy deleting the collection. Calling multiple times in a row makes it worse. There's no indication that apiserver is currently deleting the collection.

**What you expected to happen**:

a) Change delete collection so that it deletes just the first page of items in the collection, rather than loading the whole collection into memory. Callers must call repeatedly to delete the whole collection.
b) Change delete collection to use a single-flight approach, so that if you call delete collection while one is in progress, you join it rather than start a new one. (This actually doesn't work because there's multiple apiservers.)

**Environment**:
- Kubernetes version (use `kubectl version`): 1.14.x (but I don't think it matters)

/sig api-machinery
Repository: react
Issue: Add devtools "Inspect React component" to right-click menu
Labels: Type: Feature Request,Component: Developer Tools

Noticed today that Vue's devtools have an awesome "Inspect Vue component" option when right-clicking on a webpage (added in https://github.com/vuejs/vue-devtools/pull/479).

The React devtools should have the same feature! It would be useful when debugging for sure.
Repository: go
Issue: net/http: drop support for any other TransferEncoding value than "chunked"
Labels: NeedsFix

As of RFC 7230, "identity" is not a thing anymore, and we are probably off-spec for any other value. nginx only supports "chunked" anyway. There is a lot of code that can be removed or simplified in the sending path. (The parsing path is already being made strict as security hardening.)

However, we document setting TransferEncoding to "identity" as the way to affect some automatic chunking behavior, so we should probably still catch it at a shallow level, and behave accordingly, even if we don't send it on the wire.
Repository: pytorch
Issue: Add Numpy-like "order" argument to reshape
Labels: feature,triaged,module: numpy,has workaround

## 🚀 Feature
<!-- A clear and concise description of the feature proposal -->
the reshape method should support Fortran-like order except for C-like order. e.g. wo could use a vecotr to fill a matrix in column order first, not just rows.

thx~~
## Motivation

<!-- Please outline the motivation for the proposal. Is your feature request related to a problem? e.g., I'm always frustrated when [...]. If this is related to another GitHub issue, please link here too -->

## Pitch

<!-- A clear and concise description of what you want to happen. -->

## Alternatives

<!-- A clear and concise description of any alternative solutions or features you've considered, if any. -->

## Additional context

<!-- Add any other context or screenshots about the feature request here. -->


cc @mruberry @rgommers @heitorschueroff
Repository: go
Issue: crypto/x509: stop looking at first root store
Labels: NeedsFix

For some reason, when looking for a CA root store on UNIX we stop at the first good file, still look at directories, and not stop at the first good directory. On most systems the file, if it exists, is a bundling of the roots in the directory, and the directories are for different systems, not for combining.

We should probably make the function exit sooner.
Repository: youtube-dl
Issue: Site support request for knowledge.ca
Labels: site-support-request

## Checklist

- [X] I'm reporting a new site support request
- [X] I've verified that I'm running youtube-dl version **2020.05.03**
- [X] I've checked that all provided URLs are alive and playable in a browser
- [X] I've checked that none of provided URLs violate any copyrights
- [X] I've searched the bugtracker for similar site support requests including closed ones

- Single video: https://www.knowledge.ca/program/canada-over-edge/s1/e1/best-nova-scotia

knowledge.ca has a bunch of exclusive content. The most awesome of which is the Canada Over The Edge series. HD video of Canada and all its beauty. The content is behind an account login.

Enjoy! And thanks! Love youtube-dl! :)
Repository: pytorch
Issue: DistributedDataParallel does not support Modules that take no inputs.
Labels: triaged,module: data parallel

## 🐛 Bug

If you wrap a module that takes no input in DDP and try to call the forward pass, it fails with an IndexError. These kind of modules are useful to store re-parameterizations (ex: 2D -> SO(3)) that you want to update consistently across DDP workers without having to manually synchronize gradients.

## To Reproduce

```python
import os

import torch as th
from torch.nn.parallel import DistributedDataParallel

class Holder(th.nn.Module):
    def __init__(self):
        super().__init__()
        self.x = th.nn.Parameter(th.zeros(3))
    def forward(self):
        return self.x

os.environ["MASTER_ADDR"] = "127.0.0.1"
os.environ["MASTER_PORT"] = "12345"
os.environ["WORLD_SIZE"] = "1"
th.distributed.init_process_group(
    backend="nccl",
    init_method="env://",
    world_size=1,
    rank=0,
)
holder = Holder().cuda()
holder = DistributedDataParallel(holder, device_ids=[0])
x = holder()
```

Result:
```
Traceback (most recent call last):
  File "test.py", line 25, in <module>
    x = holder()
  File ".../torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File ".../torch/nn/parallel/distributed.py", line 445, in forward
    output = self.module(*inputs[0], **kwargs[0])
IndexError: tuple index out of range
```

## Expected behavior

No exception is raised.

## Environment

 - PyTorch Version (e.g., 1.0): 1.5.0
 - OS (e.g., Linux): Linux
 - How you installed PyTorch (`conda`, `pip`, source): conda
 - Python version: 3.7
 - CUDA/cuDNN version: 10.1/7.6.5
 - GPU models and configuration: v100
Repository: kubernetes
Issue: CSI ephemeral volumes: idempotent NodeUnpublishVolume
Labels: kind/bug,sig/storage,lifecycle/frozen

<!-- Please use this template while reporting a bug and provide as much info as possible. Not doing so may result in your bug not being addressed in a timely manner. Thanks!

If the matter is security related, please disclose it privately via https://kubernetes.io/security/
-->


**What happened**:

During testing with PMEM-CSI, `NodeUnpublishVolume` was called twice by kubelet, presumably because the first call took too long (deleting volumes does take time). The second call blocked, waiting for the first to complete its work, and then returned `NOT_FOUND` because the first call had removed all information about the volume. That caused kubelet to keep calling `NodeUnpublishVolume` forever and the pod never got deleted.

**What you expected to happen**:

Returning `NOT_FOUND` is required by the CSI spec. The driver cannot know that the volume was ephemeral because it now longer has any information about it. The csi-driver-host-path does the same: https://github.com/kubernetes-csi/csi-driver-host-path/blob/941f7a8aa7b991c11293e806de80e6b6483d3d01/pkg/hostpath/nodeserver.go#L203-L206

Perhaps kubelet then needs to be changed and accept that `NOT_FOUND` indicates success? But doing that for persistent volumes would be a spec violation, too, so it has to be limited to ephemeral volumes.

/cc @jsafrane for suggestions...

**How to reproduce it (as minimally and precisely as possible)**:

We would have to artificially slow down `NodeUnpublishVolume` in a driver to trigger the error condition in kubelet.

/sig storage
Repository: go
Issue: crypto/tls: conflicting lock order in Handshake() and Read()
Labels: NeedsInvestigation

### What version of Go are you using (`go version`)?

1.14.2

### Does this issue reproduce with the latest release?

Yes.

### What operating system and processor architecture are you using (`go env`)?

GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/boqin/.cache/go-build"
GOENV="/home/boqin/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/boqin/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/data/suz305/go-workspace/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/data/suz305/go-workspace/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build519488021=/tmp/go-build -gno-record-gcc-switches"

### What did you do?

I analyzed the source code crypto/tls/conn.go and found a possible conflicting lock order which may lead to deadlock.
If `Conn.Handshake()` and `Conn.Read()` are executed concurrently in two threads, then a deadlock may occur.
In crypto/tls/conn.go
```
type Conn struct {
	...
	handshakeMutex sync.Mutex
	...

	// input/output
	in, out   halfConn
	
}
```
```
type halfConn struct {
	sync.Mutex

	...
}
```
1. `Conn.handshakeMutex.Lock()` -> `Conn.in.Lock()`
In `Handshake()`: `Conn.handshakeMutex.Lock()` is on L1344. `Conn.in.Lock()` is on L1354
https://github.com/golang/go/blob/9b189686a53d7fec7deb93d7521531157aa023cb/src/crypto/tls/conn.go#L1343-L1355

2. `Conn.in.Lock()` -> `Conn.handshakeMutex.Lock()`
In `Read()`: `c.in.Lock() ` is on L1243. `handlePostHandshakeMessage()` is on L1251.
https://github.com/golang/go/blob/9b189686a53d7fec7deb93d7521531157aa023cb/src/crypto/tls/conn.go#L1233-L1251
Then `handleRenegotiation()` is on L1178.
https://github.com/golang/go/blob/9b189686a53d7fec7deb93d7521531157aa023cb/src/crypto/tls/conn.go#L1176-L1179
 `Conn.handshakeMutex.Lock()` is on L1164
https://github.com/golang/go/blob/9b189686a53d7fec7deb93d7521531157aa023cb/src/crypto/tls/conn.go#L1164-L1172

### What did you expect to see?
--


### What did you see instead?
--
Repository: terminal
Issue: Display graphical images in terminal
Labels: Issue-Feature,Help Wanted,Area-TerminalControl,Product-Terminal

<!-- 
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨

I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING:
1. If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement.
2. If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement.
3. If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number).
4. If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement.
5. If I file something completely blank in the body, the core team may close my issue without further explanation or engagement.

All good? Then proceed!
-->

# Description of the new feature/enhancement
It would be nice to have the ability to display graphical images in the terminal by using the API provided by the Windows Terminal. Maybe something similar to https://sw.kovidgoyal.net/kitty/kittens/icat.html or https://www.iterm2.com/documentation-images.html

<!-- 
A clear and concise description of what the problem is that the new feature would solve.
Describe why and how a user would use this new functionality (if applicable).
-->
Repository: rust
Issue: Anchor names for some impls are not predictable
Labels: T-rustdoc,C-bug

I have [a struct with a generic type](https://gitlab.com/sequoia-pgp/sequoia/-/blob/master/openpgp/src/cert/bundle.rs#L112):

```
struct ComponentBundle<C> {
    component: C,
    ...
}
```

Then I have [implementations for different concrete types](https://gitlab.com/sequoia-pgp/sequoia/-/blob/master/openpgp/src/cert/bundle.rs#L628), e.g.,:

```
impl ComponentBundle<UserID> {
    ...
}
```

I want to link to the documentation for one of those implementations.  But their anchors are called ["#method.revocation_status-1"](https://docs.sequoia-pgp.org/sequoia_openpgp/cert/amalgamation/struct.ComponentAmalgamation.html#method.revocation_status-1), ["#method.revocation_status-2"](https://docs.sequoia-pgp.org/sequoia_openpgp/cert/amalgamation/struct.ComponentAmalgamation.html#method.revocation_status-2), etc., which is brittle.  It would be nice if the anchor names were more predictable.
Repository: godot
Issue: Creating a node and attaching a built-in script to it resets the names of built-in scripts with higher ID's (lower on the scene tree) than the newly created script/node.
Labels: bug,topic:editor

**Godot version:**
3.2.1

**OS/device including version:**
Windows 10 Education Version 1909 Build 10.0.18363

**Issue description:**
When a built-in script gets created and saved above other nodes (on the scene tree) that have built-in scripts (on a freshly opened project?) their built-in script names get reset and their ID's get bumped up by one.

**Steps to reproduce:**
1. Create a new scene.
2. Add a root node.
3. Add three nodes as children to the root node.
4. Attach a built-in script to the third child of the root node. Name the script in the resource panel on the right and save.
5. Attach a built-in script to the second child of the root node. Name the script in the resource panel on the right and save.
6. Save and exit the editor.
7. Reopen the editor and reopen the scene and reopen the scripts in the editor.
9. Attach a built-in script to the first child of the root node.
10. Hit save and observe the scripts you named earlier get reset to no names and their ID's get bumped by one.

**Minimal reproduction project:**
[script-bug.zip](https://github.com/godotengine/godot/files/4580056/script-bug.zip)
Repository: go
Issue: x/playground: exit code 2 is cached and returned for a valid program
Labels: NeedsInvestigation

### What version of Go are you using (`go version`)?

<pre>
$ go version
go1.14.2
</pre>

### Does this issue reproduce with the latest release?
Probably yes

### What operating system and processor architecture are you using (`go env`)?
`playground configurations`
### What did you do?

I changed the syntax of two programs(that are logically the same), by switching the order of the functions

- https://play.golang.org/p/ftthvh6ifYd
- https://play.golang.org/p/mn1PAI19xI6


### What did you expect to see?

I was expecting to see the same outputs

### What did you see instead?
The second program(from the above list) crashed
Repository: angular
Issue: Cannot parse slashes in unquoted attribute values
Labels: type: bug/fix,area: compiler,state: confirmed,P5,compiler: parser

<!--🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅

Oh hi there! 😄

To expedite issue processing please search open and closed issues before submitting a new one.
Existing issues often contain information about workarounds, resolution, or progress updates.

🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅-->


# 🐞 bug report

### Affected Package
<!-- Can you pin-point one or more @angular/* packages as the source of the bug? -->
<!-- ✍️edit: --> The issue is caused by package @angular/compiler.


### Is this a regression?

<!-- Did this behavior use to work in the previous version? -->
<!-- ✍️--> No.


### Description

Context: ikatyang/angular-html-parser#14, prettier/prettier#8210

According to the [HTML spec](https://html.spec.whatwg.org/#unquoted):

> The attribute name, followed by zero or more ASCII whitespace, followed by a single U+003D EQUALS SIGN character, followed by zero or more ASCII whitespace, followed by the attribute value, which, in addition to the requirements given above for attribute values, **must not contain any literal ASCII whitespace, any U+0022 QUOTATION MARK characters ("), U+0027 APOSTROPHE characters ('), U+003D EQUALS SIGN characters (=), U+003C LESS-THAN SIGN characters (<), U+003E GREATER-THAN SIGN characters (>), or U+0060 GRAVE ACCENT characters (`), and must not be the empty string.**

U+002F SLASH characters (/) are valid unquoted attribute values, but the parser treat them as end signals:

https://github.com/angular/angular/blob/42c58d9fe8e51931c465f779b3da64fe8551b8c4/packages/compiler/src/ml_parser/lexer.ts#L595-L599

https://github.com/angular/angular/blob/42c58d9fe8e51931c465f779b3da64fe8551b8c4/packages/compiler/src/ml_parser/lexer.ts#L761-L764

## 🔬 Minimal Reproduction
```js
const {parseTemplate} = require("@angular/compiler");
const ast = parseTemplate("<a href=https://angular.io/>Link</a>");
console.log(ast.errors); //=> Unexpected closing tag "a".
```

## 🔥 Exception or Error

_Unexpected closing tag "a". It may happen when the tag has already been closed by another tag. For more info see https://www.w3.org/TR/html5/syntax.html#closing-elements-that-have-implied-end-tags_
## 🌍  Your Environment

**Angular Version:**
```
@angular/compiler@9.1.4
```

**Anything else relevant?**
<!-- ✍️Is this a browser specific issue? If so, please specify the browser and version. -->
No.
<!-- ✍️Do any of these matter: operating system, IDE, package manager, HTTP server, ...? If so, please mention it below. -->