sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: create-react-app
Issue: Publish react-dev-utils/openBrowser as separate package?
Labels: issue: proposal

### Is this a bug report?

No.

---

First of all, thanks for this awesome project! Your work on low-configuration tooling for the React community is a huge help for those of all skill levels.

I would like to suggest the possibility of publishing `react-dev-utils/openBrowser` (and maybe other things) separately as low-dependency helper packages to avoid the issue caused by the locked dependencies of `react-dev-utils`.

Let me outline the use case with an example:

`mdx-deck` uses `react-dev-utils/openBrowser` to reliably open the deck in the browser and focus the tab:

https://github.com/jxnblk/mdx-deck/search?q=react-dev-utils&type=Code

This is a great use of the library, but it has the downside of creating a sub-dependency on the locked dependencies of `react-dev-utils`, which there are many of:

https://github.com/facebook/create-react-app/blob/57ef103440c24e41b0d7dc82b7ad7fc1dc817eca/packages/react-dev-utils/package.json#L53-L77

This, combined with the use of the common `^` minor version specifier for `react-dev-utils` and the relatively fast cadence of releases in `react-dev-utils` means that any incompatibilities between any versions of any of the locked dependencies can cause situations if used with other, older packages.

For example, using a Yarn workspace with package using a pre-2.0 version of mdx-deck along with another package using version `3.0.1` of `react-scripts`.

In my particular case, it resulted in a cryptic ["BrowserslistError: Unknown browser kaios"](https://stackoverflow.com/a/55872461/1268612), caused by the locked dependency on an older version of `browserslist` in `react-dev-utils`. 

I cannot downgrade `browserslist` via Yarn Resolutions reliably, because `create-react-app@3.0.1` depends on `react-dev-utils@^9.0.1`. And I cannot upgrade `react-dev-utils` because of the minor `^` selector in the old and current versions of `mdx-deck` (`react-dev-utils@^7.0.3` and `react-dev-utils@^8.0.0` respectively).

So I'm left with using a resolution that may break things more subtly if there's a different bug between `browserslist@4.4.1` (sub-dep from `mdx-deck@1.10.2` + `react-dev-utils@7.0.5`) and `browserslist@4.5.4` (sub-dep from `create-react-app@3.0.1` + `react-dev-utils@9.0.1`).

---

### Solution

If the module was published as a separate package, it could retain its locked dependency list but with far lower possibility for incompatibilities.

I wouldn't suggest this for every module in `react-dev-utils`, but it seems as if the `openBrowser` module is useful enough to warrant separate publishing.

In any case, thanks for listening!

### Alternative Solution

Strongly recommend that package authors do not depend on the modules from `react-dev-utils` or provide some different alternative for them.

Also, if you know of a third option that I haven't thought of, I'm all ears!
Repository: opencv
Issue: VideoWriter class does not support stream copy
Labels: priority: low,category: videoio

##### System information (version)
- OpenCV => 3.1 with bindings in Python 3.6
- Operating System / Platform =>Ubuntu 18.04 LTS
- Compiler => Visual Studio 2015
-->

##### Detailed description

The VideoWriter object does not have working stream copy functionality. Using a command line call to ffmpeg, for instance, one can read a video stream as input (akin to the VideoCapture object) and directly copy this stream into an output file without any decoding or re-encoding, which saves a lot of computation time. In OpenCV, if no codec is specified for the VideoWriter object, no file is output. Documentation suggests that entering a value of 0 into the fourcc codec code and a framerate of 0 when constructing the VideoWriter object should accomplish this but does not.

https://docs.opencv.org/3.4/dd/d9e/classcv_1_1VideoWriter.html
"If FFMPEG is enabled, using codec=0; fps=0; you can create an uncompressed (raw) video file."

The following code produces no output file.
    ```.py

        fourcc = 0

        fps = 0

        size =  1080,720

        out = VideoWriter("outfile.avi",outvid, fourcc, float(fps), size)    
    ```

-->
Repository: pytorch
Issue: @ignore annotation for user defined type
Labels: oncall: jit,triaged,jit-backlog

## 🚀 Feature
In user defined type, it would be nice to have a @ignore annotation similar to https://github.com/pytorch/pytorch/pull/16055

This allows the user to write python only methods that are only used in training / debugging (like `def  #visualize`)

cc @suo
Repository: youtube-dl
Issue: site support: mediagoblin, libreplanet
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 **2019.05.11**
- [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

- Playlist: https://media.libreplanet.org/u/libreplanet/tag/libreplanet-2019-video/

Single videos from this site work fine.

## Description

```
joshua@cocoa:/tmp$ youtube-dl -v https://media.libreplanet.org/u/libreplanet/tag/libreplanet-2019-video/
[debug] System config: []
[debug] User config: ['-x', '--no-mtime', '--audio-format', 'opus', '--audio-quality', '96k', '--ignore-errors', '--download-archive', '/home/joshua/.config/youtube-dl/downloaded.txt']
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://media.libreplanet.org/u/libreplanet/tag/libreplanet-2019-video/']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.05.11
[debug] Python version 3.7.3 (CPython) - Linux-5.0.0-13-generic-x86_64-with-Ubuntu-19.04-disco
[debug] exe versions: ffmpeg 4.1.3, ffprobe 4.1.3, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[generic] libreplanet-2019-video: Requesting header
WARNING: Falling back on generic information extractor.
[generic] libreplanet-2019-video: Downloading webpage
[generic] libreplanet-2019-video: Extracting information
ERROR: Unsupported URL: https://media.libreplanet.org/u/libreplanet/tag/libreplanet-2019-video/
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/dist-packages/youtube_dl/YoutubeDL.py", line 796, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/lib/python3.7/dist-packages/youtube_dl/extractor/common.py", line 529, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/lib/python3.7/dist-packages/youtube_dl/extractor/generic.py", line 3329, in _real_extract
    raise UnsupportedError(url)
youtube_dl.utils.UnsupportedError: Unsupported URL: https://media.libreplanet.org/u/libreplanet/tag/libreplanet-2019-video/

```
Repository: youtube-dl
Issue: Site support request: kvf.fo
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 2019.05.11. 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 **2019.05.11**
- [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://kvf.fo/live/1
- Single video: http://kvf.fo/vit/sjonvarp/2019/03/20/snipp-snapp-0


## Description

<!--
Provide any additional information.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
-->

More info about website itself here -> https://en.wikipedia.org/wiki/Kringvarp_Føroya
Content is streaming by MPEG-DASH. Generic extractor included in youtube-dl don't work and videos are downloadable when I get direct link to m3u8 file.
Repository: kubernetes
Issue: Simultaneous calls to add a finalizer and delete can race
Labels: kind/bug,priority/important-soon,sig/api-machinery,lifecycle/frozen

Store#Delete begins by looking up the existing object:

https://github.com/kubernetes/kubernetes/blob/8ae998ceb69ae83afe730795aea3bd44913ad868/staging/src/k8s.io/apiserver/pkg/registry/generic/registry/store.go#L865-L875

If that object has no pending finalizers, and that type does not do graceful deletion, and the incoming deletion options do not specify preconditions (uid or resourceVersion) or GC-related finalizers (like orphan or foreground deletion), then the object is deleted unconditionally:

https://github.com/kubernetes/kubernetes/blob/8ae998ceb69ae83afe730795aea3bd44913ad868/staging/src/k8s.io/apiserver/pkg/registry/generic/registry/store.go#L929-L932

In between the initial get and the unconditional delete, an update adding finalizers could have been made and succeeded.

/kind bug
/sig api-machinery
/priority important-soon
/cc @caesarxuchao
Repository: vscode
Issue: New revealProblems property in task presentation options is missing from the API
Labels: feature-request,api,tasks

A new `revealProblems` property is available in the `presentation` of tasks in `tasks.json` in current Insider's builds / the upcoming 1.34. However, this property [is absent from the `TaskPresentationOptions` API in `vscode.d.ts`](https://github.com/microsoft/vscode/blob/07ec42fb0070bd629c1a7fa8f17a8183d7ac4958/src/vs/vscode.d.ts#L4851-L4885), even though historically, new options were always added there. It also doesn't seem to be respected when you ignore the type declaration and set the property via the tasks API anyway.

Is this intentional, or simply an oversight?
Repository: flutter
Issue: Widgets show hover effect when clicked on a desktop embedder that has no hover support 
Labels: engine,a: desktop,P2,team-design,triaged-design

When running on a desktop embedder that has no hover support, a widget will incorrectly show hover effect when a mouse clicks inside it, which is dismissed when the mouse clicks out of it.

In the example below, the hover effects are the light-grey color and the tooltip.
![1](https://user-images.githubusercontent.com/1596656/57890421-79be1b80-77ec-11e9-9c10-7a8c52dd63c3.gif)

If hover is enabled onto the same embedder, the hover effect behaves as expected.

Environment: 
- gLinux
- Framework: 900875fd3ff2ec63cce2556a11ab5f86b02d47d6
- Embedder: https://github.com/google/flutter-desktop-embedding/pull/355

Related discussion: https://github.com/flutter/flutter/issues/32835
Repository: terminal
Issue: Encapsulate the background image settings into a JSON object
Labels: Area-Settings,Product-Terminal,Issue-Task

# Summary of the new feature/enhancement

As we're adding more settings to the JSON file, do we want to keep what's in each object flat or should we start introducing nested objects? I think having them be nested looks nicer and is easier to read (especially as the settings file gets really big). On the other hand, this might mean that we're forcing settings to look a certain way.

Consider this:
 - Today:
```
{
    guid: ...,
    useAcrylic: ...,
    colorScheme: ...,
    backgroundImage: ...,
    backgroundImageOpacity: ...,
    backgroundImageStretchMode: ...,
    backgroundImageSetting4: ...,
    backgroundImageSetting5: ...
}
```
 - Proposal:
```
{
    guid: ...,
    useAcrylic: ...,
    colorScheme: ...,
    backgroundImage: 
    {
        url: ...,
        opacity: ...,
        stretchMode: ...,
        setting4: ...,
        setting5: ...
    }
}
```

I think the proposal looks a lot nicer. I'm picking on backgroundImage from #853 but it definitely can apply to some other things like `fontProperties` (size, family, etc...) and `devEnvironment` (environmentVariables, startingDirectory, etc...).

Also, I think it's easier to see the benefit when there's a nasty looking settings file as follows:
```
{
    profileName: ...,
    backgroundImage: ...,
    guid: ...,
    backgroundImageOpacity: ...,
    useAcrylic: ...,
    backgroundImageStretchMode: ...,
    colorScheme: ...
}
```
Repository: pytorch
Issue: groupby function
Labels: feature,triaged

## 🚀 Feature
<!-- A clear and concise description of the feature proposal -->

## Motivation

Say for example you have a batch of data but want to take a different action (e.g. run a different model) based on the label index, or based on the length of the input, etc. You want to subdivide the batch according to that value, and in some cases you need to reconstruct the outputs back into the shape of the original tensor. What is needed is a function that groups the rows of the tensor based on the value of the groupby field.

I've ended up needing to implement this multiple times across projects. 

## Pitch

```
groups, values, inverse_map = T.groupby(F[, dim=0])
```

Basically T is an `Nx...` tensor and `F` is a size `N` tensor (assuming dim=0). This function returns a list `groups` of length equal to the number of unique values in F. Each group contains all the rows of `T` for that unique element. `values` is just a tensor containing the unique values corresponding to each group, and `inverse_map` is a list of tensors telling you where all the rows came from.

## Alternatives

This can be implemented in about 10 lines in pytorch with something like:
```
sorted_vals, order = F.sort(0)
delta = F[1:] - F[:-1]
cutpoints = delta.nonzero()[0].tolist()
res, inverse_map = [], [], torch.zeros(len(cutpoints))
for start, end, i in zip([0] + cutpoints, cutpoints + [len(F)], range(len(cutpoints) + 1)):
    res.append(T[order[start:end]])
    inverse_map.append(order[start:end])
    values[i] = F[start]

return res, values, inverse_map
```

This could be done faster and in a more standard way as part of Pytorch.

## Additional context

cc @colesbury @lerks
Repository: rust
Issue: Tracking issue for `HashSet` entry APIs
Labels: A-collections,T-libs-api,B-unstable,C-tracking-issue,Libs-Tracked

Feature gate: `#![feature(hash_set_entry)]`

This is a tracking issue for `Entry` and entry-like methods on `HashSet`.

### Public API

```rust
impl<T, S> HashSet<T, S>
where
    T: Eq + Hash,
    S: BuildHasher,
{
    pub fn get_or_insert(&mut self, value: T) -> &T {...}
    pub fn get_or_insert_with<Q: ?Sized, F>(&mut self, value: &Q, f: F) -> &T
    where
        T: Borrow<Q>,
        Q: Hash + Eq,
        F: FnOnce(&Q) -> T,
    {...}

    pub fn entry(&mut self, value: T) -> Entry<'_, T, S> {...}
}

pub enum Entry<'a, T, S> {
    Occupied(OccupiedEntry<'a, T, S>),
    Vacant(VacantEntry<'a, T, S>),
}
pub struct OccupiedEntry<'a, T, S> {...}
pub struct VacantEntry<'a, T, S> {...}

impl<T: fmt::Debug, S> fmt::Debug for Entry<'_, T, S> {...}
impl<T: fmt::Debug, S> fmt::Debug for OccupiedEntry<'_, T, S> {...}
impl<T: fmt::Debug, S> fmt::Debug for VacantEntry<'_, T, S> {...}

impl<'a, T, S> Entry<'a, T, S> {
    pub fn insert(self) -> OccupiedEntry<'a, T, S>
    where
        T: Hash,
        S: BuildHasher,
    {...}
    pub fn or_insert(self)
    where
        T: Hash,
        S: BuildHasher,
    {...}
    pub fn get(&self) -> &T {...}
}

impl<T, S> OccupiedEntry<'_, T, S> {
    pub fn get(&self) -> &T {...}
    pub fn remove(self) -> T {...}
}

impl<'a, T, S> VacantEntry<'a, T, S> {
    pub fn get(&self) -> &T {...}
    pub fn into_value(self) -> T {...}
    pub fn insert(self)
    where
        T: Hash,
        S: BuildHasher,
    {...}
}
```

The `get_or_insert[_with]` methods provide a simplification of the `Entry` API for `HashSet`, with
names chosen to match the similar methods on `Option`. The full `Entry` API mimics that
of `HashMap`, though without methods for the map value (which is just `()` in a set).

### Steps / History

- [x] Initial methods: #60894 
- [x] Fuller `Entry` API: #120077
- [ ] Final comment period (FCP)[^1]
- [ ] Stabilization PR

### Unresolved Questions

- None yet.

See also #133549 for `BTreeSet`.

[^1]: https://std-dev-guide.rust-lang.org/feature-lifecycle/stabilization.html
Repository: go
Issue: x/tools/godoc: remove replaceLinePrefixCommentsWithBlankLine in favor of better fix
Labels: NeedsInvestigation,Tools

Now that #7702 is resolved, it should be possible to remove the [`replaceLinePrefixCommentsWithBlankLine`](https://github.com/golang/tools/blob/1a8f2608bd819ae0c5aeb07edd6bae2fa2cebd71/godoc/parser.go#L55-L57) workaround with a better fix.

This issue is about tracking the progress on that (since all previous issues related to this were closed).

I've looked into this a bit and it's not clear to me if the fix belongs in `golang.org/x/tools/godoc` itself, or if perhaps it can be implemented in a lower level component: one of `go/doc`, `go/parser`, `go/scanner`, `go/ast`, etc. (The workaround would need to stay until it's possible to fully rely on the lower level fix in stdlib.)

I'm also not sure if it's expected behavior for `//line` comments to show up in `ast.CommentGroup` or not.

@griesemer Do you have suggestions on how to best deal with this issue, now that [`token.PositionFor`](https://godoc.org/go/token#File.PositionFor) API exists?

### History

Issue #5247, reported in 2013, was about `godoc` then not handling the following input properly:

```Go
package p

//line file:2
// G doc.
func G()
```

It was showing "line file:2 G doc." as the documentation of function `G`, rather than just "G doc." as it should've been. (If you were to change the line number in the comment to `//line file:10`, it wouldn't show up anymore.)

At that time, there was no API to obtain the raw source positions from a `token.Pos`, which was the main blocker and a long-standing TODO.

As a result, the godoc issue (#5247) was fixed via a workaround in https://github.com/golang/tools/commit/55ea53125770b1ee7bd5cdbd3794ccf2875831e with the comment:

```Go
// Temporary ad-hoc fix for issue 5247.
// TODO(gri) Remove this in favor of a better fix, eventually (see issue 7702).
replaceLinePrefixCommentsWithBlankLine(src)
```

Issue #7702 was filed about adding an API to access the original source lines. It has since been resolved via https://github.com/golang/go/commit/e66ff2b9080669373704914191abf2ee5f65eb75.
Repository: go
Issue: plugin: call dlclose if loading a plugin fails
Labels: NeedsInvestigation,compiler/runtime

<!-- Please answer these questions before submitting your issue. Thanks! -->

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version go1.12.5 linux/amd64
</pre>

### Does this issue reproduce with the latest release?
Yes

### What operating system and processor architecture are you using (`go env`)?

<details><summary><code>go env</code> Output</summary><br><pre>
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/wangzhizhao/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/wangzhizhao/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
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-build948496286=/tmp/go-build -gno-record-gcc-switches"
</pre></details>

### What did you do?
First I load an .so built with go1.12，while error occur，I load another .so built with go1.12.5，still got error：plugin was built with a different version of package errors。

If I only load the .so which built with go1.12.5, It works correctly.
<!--
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
-->

### What did you expect to see?
If load an .so built with different verison of go，while error occur，I load another .so built with the correct version of go，no error should occur，It should work correctly。

### What did you see instead?
Repository: rust
Issue: Android fails to link in libc-test 
Labels: A-linkage,O-android,T-compiler,C-bug

I was trying to fix and upgrade the Android build jobs in libc, and I've managed to get the x86_64 and i686 ones running, but the arm and aarch64 ones fail to link and I have no idea why (https://github.com/rust-lang/libc/pull/1344). These are the two errors that I get:

```
= note: /android/ndk-arm/bin/../lib/gcc/arm-linux-androideabi/4.9.x/../../../../arm-linux-androideabi/bin/ld: error: cannot open crtbegin_dynamic.o: No such file or directory
          /android/ndk-arm/bin/../lib/gcc/arm-linux-androideabi/4.9.x/../../../../arm-linux-androideabi/bin/ld: error: cannot open crtend_android.o: No such file or directory
          /android/ndk-arm/bin/../lib/gcc/arm-linux-androideabi/4.9.x/../../../../arm-linux-androideabi/bin/ld: error: cannot find -llog
          external/jemalloc/src/jemalloc.c:1422: error: undefined reference to 'pthread_atfork'
          external/jemalloc/src/jemalloc.c:1422: error: undefined reference to 'pthread_atfork'
          external/jemalloc/src/jemalloc.c:1422: error: undefined reference to 'pthread_atfork'
          external/jemalloc/src/jemalloc.c:1422: error: undefined reference to 'pthread_atfork'
          external/jemalloc/src/jemalloc.c:1358: error: undefined reference to 'atexit'
          clang80: error: linker command failed with exit code 1 (use -v to see invocation)
```

and

```
cannot find crtbegin_dynamic.o: No such file or directory
          clang80: error: linker command failed with exit code 1 (use -v to see invocation)
```          

I initially suspected that this might have something to do with the different API or NDK versions used by rust-lang/rust vs the ones in the PR which are newer, but the part of the test script that fails is the one that has `--no-default-features` in it, which means that `use_std` is disabled, so only `libcore` should be linked, so this shouldn't be an issue.

The Android build jobs have been failing for a couple of months already, and we have been adding linux APIs that might have been breaking Android because they are not properly scoped, so getting these up and running again is kind of important for Android support.
Repository: pytorch
Issue: Main page broadcasting (?) example image bug
Labels: module: docs,triaged

## 🐛 Bug

There is an image on the main page of this repo:
![image](https://user-images.githubusercontent.com/5480856/57926458-fc5dde00-78b3-11e9-819c-96119d99026e.png)

Looks like the numbers on the result tensor are wrong (and pretty unreadable btw).