sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: x/tools/cmd/goimports: adds redundant alias to package paths containing hyphen/dash
Labels: NeedsInvestigation,Tools

<!--
Please answer these questions before submitting your issue. Thanks!
For questions please use one of our forums: https://github.com/golang/go/wiki/Questions
-->

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version go1.14.14 linux/amd64
</pre>

### Does this issue reproduce with the latest release?

Yes (compiled with go1.16.5 too)
<pre>
$ git log -n 1
commit 1225b6f53f675b0732bdf025af585b173de322ec (HEAD -> master, origin/master, origin/HEAD)
</pre>


### What operating system and processor architecture are you using (`go env`)?

<details><summary><code>go env</code> Output</summary><br><pre>
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/peregrine/.cache/go-build"
GOENV="/home/peregrine/.config/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/peregrine/go:/home/peregrine/core3:/home/peregrine/core3/src/plz-out/go:/home/peregrine/core3/src/plz-out/gen/third_party/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/opt/tm/tools/go/1.14.14/usr/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/opt/tm/tools/go/1.14.14/usr/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
AR="ar"
CC="cc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/peregrine/go/src/golang.org/x/tools/go.mod"
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-build730235714=/tmp/go-build -gno-record-gcc-switches"

</pre></details>

### What did you do?

<!--
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
-->
https://play.golang.org/p/8VOphLYHxto


### What did you expect to see?
No redundant alias


### What did you see instead?
Redundant alias added
Repository: youtube-dl
Issue: Site support request: voe.sx
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 2021.06.06. 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 **2021.06.06**
- [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://voe.sx/x63dioldjm1y
- Single video: https://voe.sx/e/x63dioldjm1y


## Description

<!--
Provide any additional information.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
-->
Some videos on VOE are restricted to embed view, but I didn't find any of that kind by search, that would comply with the beforementioned rules for links. Those urls can be transformed, by simply adding/removing the "/e" in the url which makes them embed. Some videos also are restricted to embed view and restricted to watch by region. That can be bypassed sometimes, when one calls the non-embed video URL once and calls the embed URL afterwards in the same tab. Videos for both special cases, I found using search term "site:voe.sx", but those videos where either NSFW or they violated copyrights, so if you can't implement those special cases, because of the lack of safely obtainable information, it wouldn't be neccessary.
Repository: vscode
Issue: Notebook suggestion: "Choose a different output mimetype" only available when different options exist
Labels: debt,polish,notebook-output

<!-- ⚠️⚠️ Do Not Delete This! feature_request_template ⚠️⚠️ -->
<!-- Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct/ -->
<!-- Please search existing issues to avoid creating duplicates. -->

<!-- Describe the feature you'd like. -->

Just a small suggestion from when I was looking at #125490. The "Choose a different output mimetype" option is available what looks like all the time, even when there is only a single mimetype in the output and a single available renderer for that output type. 

Example here:
![image](https://user-images.githubusercontent.com/812783/121051878-b2801300-c76e-11eb-8f38-d08a52214a4f.png)

![image](https://user-images.githubusercontent.com/812783/121052564-5964af00-c76f-11eb-9825-9c9c45dabbf5.png)


stdout output. When the option is clicked there is only one renderer and one mimetype. So there is no real option from the action. I think this might be nice in part that since there is also the "Clear Cell Outputs" command on the first output item the icon is always forced into the "..." overflow icon. Makes something like matplotlib output (which does text/plain before the image) have some odd iconography in the cell output gutter.

![image](https://user-images.githubusercontent.com/812783/121053360-1eaf4680-c770-11eb-9b92-42cafc4632c3.png)
Repository: flutter
Issue: [plugin_tools] auto-publishing does not need to be synchronized.
Labels: p: tooling,package,team-ecosystem,P2,triaged-ecosystem

The [`machine`](
https://github.com/flutter/plugins/blob/master/script/tool/lib/src/publish_check_command.dart#L37) flag in the `publish-check` command was created to ensure that auto-publishing workflow is synchronized: the newer ci jobs won't publish updates until the versions in older commits are published. 

We can remove this since publishing to pub does not need to be sequential as long as the versions are published on the correct github commit. 

CC @stuartmorgan, Also feel free to close this issue if you think this flag might be needed in some near future.
Repository: pytorch
Issue: [discussion] Make docs consistent for torch.nn Modules
Labels: module: docs,module: nn,triaged

Our documentation for the modules provided by `torch.nn` is inconsistent and in some cases is missing information (e.g. shapes for `forward()` inputs). Since good documentation is crucial for the usability of these modules, this issue proposes to define a consistent template for module docs with all information necessary to use each effectively. What "template" means here could range from simply a cultural convention backed up by dev docs to technically enforced or generated docs (e.g. using jinja templates, CI checks, etc.).

Concretely, I propose that each module doc page should include documentation for:
* All public module constructor args, including expected shapes for all tensors
* All public `forward()` method inputs, including expected shapes for all tensors
* All `forward()` method outputs, including shapes for all tensors
* All learnable parameters associated with the module, including their shapes
* Any caveats, gotchas, limitations, etc.

To streamline the usage of these docs by users, the documentation sections should be consistently named and ordered & shape information should follow a consistent notation.

Open questions:
* **How should the template be enforced?** The loosest thing to do is enforce it by convention- i.e. document it with a full example in the developer docs & use this as a reference when adding new modules or modifying existing module docs. Alternatively, it may be possible to have a stricter enforcement by technical means - jinja templates, CI checks, etc.
* How well should this match the operator documentation? At a high level, modules can be thought of as just operators with added state. A lot of the same information provided in operator docs would make sense for module `forward()` docs. Examples: whether broadcasting is supported, supported dtypes, etc.
* Often, modules are simply stateful wrappers around functions in `torch.nn.functional`. What is the best way to identify this relationship in the docs? Perhaps there should be a section pointing to docs for the wrapped functional form, if there is one. See https://github.com/pytorch/pytorch/issues/49992
* Anything missing above that should be in the docs?

cc @brianjo @mruberry @albanD @jbschlosser
Repository: flutter
Issue: Solving debugPrint truncation issues on "large" data blobs without whitespace (or similar).
Labels: c: new feature,tool,a: debugging,c: proposal,d: devtools,team-tool

Issue https://github.com/flutter/flutter/issues/22665  (print truncate) was closed because you  can use the workaround
of using `debugPrint(text, wrapWidth: 1024)` to prevent truncation of the text message.

But it's not a viable workaround in all situations as:

- It requires not a single word to be too long. But base64 blobs easily can
  become to long which means it still has a good chance to truncate your data
  and it's likely to happen in the places where it's less obvious.

- It still don't indicate that the output was truncated leading to general confusion
  for the developer if it's not clear from the output that it was truncated.

- Neither `print` nor `debugPrint` document that the output might be truncated. 

## Use case

Debugging larger data blobs in flutter applications.

## Proposal

I'm not sure what the best solution is but ideas include:

- Add a way to specify a `maxLineWidth` which will add lines brakes *in* words if necessary.

- Add some facility to `flutter run` which allows sending back data (json, string, etc.) in debug builds/runs without any size limitations,  this could be an additional tab in the DevTools.
Repository: vscode
Issue: iPadOS: first and last status bar item cannot be tapped on
Labels: bug,ios-ipados

Tap on the notification icon on the status bar on iPadOS doesn't work.
Repository: go
Issue: proposal: x/crypto/ssh: expose client/sever supported algorithms
Labels: Proposal,Proposal-Crypto

When working with a variety of SSH client/server software, it can be useful to expose the supported Key Exchange, MAC, and Ciphers for both client and server side. Primarily this allows someone to answer questions like, "if we remove support for X, will any of our clients fail?."

When looking into this, initially I thought we could expose this on the `ConnMetadata` interface (`*sshConn` implements this), and additionally extend `ssh.Client` with a new `GetConnMetadata() ConnMetadata` function which returns the underlying `*sshConn`.

This would involve exposing the values stored in the server/client `*kexInitMsg` on up, possibly through new fields on `*handshakeTransport` as `clientInitMsg *kexInitMsg` and `serverInitMsg *kexInitMsg`.

Is this something that would be appropriate for a PR or are other implementation options preferred here?
Repository: vscode
Issue: Git - Recommend to commit changed files before renaming/moving them
Labels: feature-request,git

Follow up from a discussion I had with @eamodio about https://github.com/microsoft/vscode/issues/62321 and [co](https://github.com/Microsoft/vscode/issues?q=is%3Aissue+sort%3Aupdated-desc+label%3Agit+mv+rename+is%3Aclosed).

Problem: Users misunderstand `git mv` and `Git: Rename` and expect magic to happen. Git's rename tracking works best when unchanged files are moved in one commit, not mixing in any changes.

Potential solutions:

- When a user tries to rename/move a version-controlled file that has uncommitted changes, VS Code could warn in the rename dialog to commit changes first.
- For the case of the Explorer, where files can be dragged/renamed inline, [contextual hints](https://github.com/microsoft/vscode/issues/120147) could maybe used.
Repository: create-react-app
Issue: React-App-Polyfill ignores browserlist
Labels: needs triage,issue: bug report

### Describe the bug

React-App-Polyfill seems to be ignoring the production browserlist from package.json and using some other browserlist instead.

### Did you try recovering your dependencies?

Yes

### Which terms did you search for in User Guide?

Read https://github.com/facebook/create-react-app/blob/master/packages/react-app-polyfill/README.md

### Environment

```
  current version of create-react-app: 4.0.3
  running from /Users/loren/.npm/_npx/11376/lib/node_modules/create-react-app

  System:
    OS: macOS 11.4
    CPU: (16) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
  Binaries:
    Node: 14.17.0 - ~/.nvm/versions/node/v14.17.0/bin/node
    Yarn: 1.22.10 - ~/.nvm/versions/node/v14.17.0/bin/yarn
    npm: 6.14.13 - ~/.nvm/versions/node/v14.17.0/bin/npm
  Browsers:
    Chrome: 91.0.4472.77
    Edge: Not Found
    Firefox: 88.0.1
    Safari: 14.1.1
  npmPackages:
    react: ^17.0.2 => 17.0.2 
    react-dom: ^17.0.2 => 17.0.2 
    react-scripts: 4.0.3 => 4.0.3 
  npmGlobalPackages:
    create-react-app: Not Found
```

### Steps to reproduce

1. `npx create-react-app react-polyfill`
2. `yarn add react-app-polyfill`
3. Edit `src/index.js` add `import 'react-app-polyfill/stable';` as the first line
4. `yarn build`
5. `npx source-map-explorer 'build/static/js/*.js'` 101 kb of core-js is included
6. Change `package.json` browserlist to
```
  "browserslist": {
    "production": [
      "last 1 chrome version"
    ],
    "development": [
      "last 1 chrome version",
      "last 1 firefox version",
      "last 1 safari version"
    ]
  }
```
7. `yarn build`
8. Build is the same size and still includes 101kb of core-js


### Expected behavior

Build should have only included `'web.immediate'` matching

```
console.log(require('core-js-compat')({ targets: "last 1 chrome version", version: '3', filter: /^(es|web)\./ }).targets);
// Output: { 'web.immediate': { chrome: '91' } }
```

### Actual behavior

Source Analysis:

![initialPackage](https://user-images.githubusercontent.com/1224505/121068239-0522f480-c79a-11eb-8f73-be7928d60a01.png)

### Reproducible demo

Created with steps above: https://github.com/loren138/react-polyfill-bug
Repository: flutter
Issue: [engine] args.gn appends not applying
Labels: engine,P2,team-engine,triaged-engine

Engine master branch tip of tree.

build/config/arm.gni line 56 of arm.gni

```
if (arm_float_abi == "") {
  arm_float_abi = "softfp"
}
```

The code in arm.gni executes before args.gn is read. Using the following segment ends up with compile_commands.json having the expected value:

```
# if (arm_float_abi == "") {
arm_float_abi = "hard"
# }
```

@chinmaygarde 
@aam
Repository: rust
Issue: `matches!` accepts unusable bindings in the pattern
Labels: A-diagnostics,T-compiler,C-bug,D-confusing,A-patterns,T-libs

<!--
Thank you for filing a bug report! 🐛 Please provide a short summary of the bug,
along with any information you feel relevant to replicating the bug.
-->

_Perhaps this is more a thing for clippy to detect, I'm not sure._

A co-worker of mine tried this code:

```rust
if !matches!(binding_to_uuid, binding_to_other_uuid) {
    // ...
}
```

I expected to see this happen:

Compiler should say that this is an obvious error, because there is no way to use the `binding_to_other_uuid` binding because it is internal to the match statement (and just shadows the actual `binding_to_other_uuid`).

Instead, this happened:

The `matches!` macro here expands to something that is never useful:

```rust
if !match binding_to_uuid {
    binding_to_other_uuid => true,
    _ => false,
} { ... }
```

Clippy suggests underscoring the `binding_to_other_uuid` because it's unused, but that's not super clear unless you are familiar with the macro expansion. Ideally providing variable bindings that are not used in a guard would be disallowed in `matches!` because they cannot otherwise be used.

### Meta
<!--
If you're using the stable version of the compiler, you should also check if the
bug also exists in the beta or nightly versions.
-->

`rustc --version --verbose`:
```
rustc 1.52.1 (9bc8c42bb 2021-05-09)
binary: rustc
commit-hash: 9bc8c42bb2f19e745a63f3445f1ac248fb015e53
commit-date: 2021-05-09
host: x86_64-unknown-linux-gnu
release: 1.52.1
LLVM version: 12.0.0
```
Repository: PowerToys
Issue: Handle CA2227 appropriately
Labels: Help Wanted,Good first issue,Area-Quality

### Provide a description of the new feature / enhancement

Few instances of CA2227 that have been inlined ignored.  We should handle this correctly.
Repository: pytorch
Issue: [clang-tidy] check for iteration over unordered data structures
Labels: triaged,better-engineering

Now that we are starting to set up some infrastructure for static analysis we should check for common pitfalls that devs have to keep in their heads currently, one of which is to not iterate over `std::unordered_map` or `std::unordered_set` and do real work

from https://fb.workplace.com/groups/pytorch.dev/permalink/856353394943142/ internally:

```cpp
struct Node {
    bool visited = false;
};

std::vector<Node*> toposort(std::unordered_map<Node*, std::vector<Node*>> adj_lists) {
    std::vector<Node*> result;
    std::function<void(Node*)> visit = [&](Node* node) {
        if (node->visited) {
            return;
        }
        node->visited = true;
        for (auto child : adj_lists[node]) {
            visit(child);
        }
        result.push_back(node);
    };
    for (auto it : adj_lists) {
        visit(it.first);
    }
    return std::vector<Node*>(result.rbegin(), result.rend());
}
```

can produce different results based on the iteration order if the map has nodes at the same level

```
# adj_lists:
a -> {c, d}
b -> {c, d}

# possible outputs:
{a, b, c, d}
{b, a, c, d}
```

as fixes the code can be restructured or use a listing of the keys (sorted by something, such as lexical or insertion time)

We should have a clang-tidy check for this in our custom build once it is up
Repository: excalidraw
Issue: Alt-duplicating an element after dragging it with Shift positions the original incorrectly
Labels: bug

steps:

1. start dragging an element
2. press shift in the middle
3. drag some more
4. press option/alt to duplicate
5. the original element tends to get positioned outside its original position

![excal_shift_duplicate_bug](https://user-images.githubusercontent.com/5153846/121080762-2573a300-c7dc-11eb-8f5e-abde148cffa6.gif)
Repository: godot
Issue: `AudioServer.get_time_to_next_mix()` can return negative values
Labels: bug,confirmed,topic:audio

**Godot version:**
3.3.2


**OS/device including version:**
Macbook Pro 2018 13 inch, Big Sur 11.3.1


**Issue description:**
AudioServer.get_time_to_next_mix() can return negative values. It seems that this happens if a relatively longer stream (eg 2 minutes long) was just set in the audio player.
Expectation: time to the *next* mix should always be non negative.


**Steps to reproduce:**
In the same call to _process, set the stream of an AudioStreamPlayer then print AudioServer.get_time_to_next_mix().


**Minimal reproduction project:**
[AudioServerBug.zip](https://github.com/godotengine/godot/files/6611828/AudioServerBug.zip)
Repository: scrcpy
Issue: Quality 
Labels: quality

Hey there, sorry to bother you with this. I was just wondering, is scrcpy suited to watching video playback from a mobile device or is it not really for that?

I tried to play a stream through a Live Sports app on my phone, so I can view it on my laptop screen using scrcpy. 
The stream is set to HD, though my device resolution is 720x1280. When watching back, it's not as smooth as it should be, but worse than that, the quality is a bit poor. It doesn't really look 720p to me. 
I've got a screenshot to help show this. 

https://i.postimg.cc/g07DLr9f/720.png

I have tried increasing the bitrate using `scrcpy -b20M` and tried changing the renderer to openGL. Could the issue just be on my end, or is my use case not what scrcpy is for?

Thanks!

Environment
 - OS: [Windows 10]
 - scrcpy version: [v1.8]
 - installation method: [Windows release]
 - device model: G8441
 - Android version: [Android 8]
Repository: pytorch
Issue: Build failed with TBB
Labels: module: build,triaged,module: tbb

## 🐛 Bug

I am trying to build PyTorch from source using TBB instead of OpenMP but I am getting the following errors every time:

```
    /usr/bin/ld: <PATH>/pytorch/build/lib/libtorch_cpu.so: undefined reference to `omp_get_max_threads'
    /usr/bin/ld: <PATH>/pytorch/build/lib/libtorch_cpu.so: undefined reference to `omp_get_num_threads'
    /usr/bin/ld: <PATH>/pytorch/build/lib/libtorch_cpu.so: undefined reference to `GOMP_barrier'
    /usr/bin/ld: <PATH>/pytorch/build/lib/libtorch_cpu.so: undefined reference to `GOMP_parallel'
    /usr/bin/ld: <PATH>/pytorch/build/lib/libtorch_cpu.so: undefined reference to `omp_in_parallel'
    /usr/bin/ld: <PATH>/pytorch/build/lib/libtorch_cpu.so: undefined reference to `omp_get_thread_num'
    collect2: error: ld returned 1 exit status
    make[2]: *** [caffe2/CMakeFiles/kernel_function_legacy_test.dir/build.make:109: bin/kernel_function_legacy_test] Error 1
```
You can find the full log file [HERE](https://drive.google.com/file/d/1YXSicCUcDm2jZyY8n0al8EdJ9TbychWO/view?usp=sharing):

## To Reproduce

Steps to reproduce the behavior:

1. `git clone https://github.com/pytorch/pytorch`
1. `cd pytorch`
1. `ATEN_THREADING=TBB BUILD_BINARY=1 USE_EIGEN_THREADPOOL=1 USE_CUDA=0 PARALLEL_BACKEND=NATIVE_TBB USE_OPENMP=0 USE_TBB=1 MKL_THREADING=TBB BLAS=MKL USE_MKLDNN=1 MKLDNN_THREADING=TBB BUILD_BINARY=1 python setup.py build --cmake 2>&1 | tee ~/output.txt
`
 - PyTorch Version (e.g., 1.0): 1.10.0a0+git1798ff0
 - OS (e.g., Linux): Manjaro
 - How you installed PyTorch (`conda`, `pip`, source): pip
 - Python version: 3.9.5
 - CUDA/cuDNN version: NO
 - GPU models and configuration: NO



cc @malfet @seemethere @walterddr
Repository: flutter
Issue: [pigeon] replace compilation tests with platform unit tests
Labels: package,team-ecosystem,p: pigeon,P2,triaged-ecosystem

Platform unit tests test compilation and runtime logic of generated code so they are a superset of compilation tests.

Historically we had compilation tests first, but there is no reason for them now that we have platform unit tests.