sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: storybook
Issue: Build issue with yarn 2 when not in git or svn context
Labels: bug,compatibility with other tools,has workaround,core,yarn / npm

**Describe the bug**
I found that a local build of storybook worked fine, but the same thing refused to build in Docker. After much hair-pulling, I discovered that it did work if the compilation occurred in the context of a git repo. The .git directory was quite reasonably being excluded from the Docker image.

In `getProjectRoot()`, the root is found by looking for a .git, .svn or node_modules directory, with process.cwd() as as fallback. However, in a Docker environment, there will be no .git or .svn and in yarn 2, there is no node_modules. It appears yarn 2 is presenting the cwd as a zip file.

This results in the following in the generated webpack:

```
module: {
  rules: [
    {
      test: /\.(mjs|tsx?|jsx?)$/,
      use: [ redacted ],
      include: [
        'D:\\react\a\\.yarn\\$virtual\\@storybook-core-virtual-0a602ea853\\0\\cache\\@storybook-core-npm-6.1.18-f25c70fd19-ac341dcf78.zip\\'
```

With .git present, it's
```
include: [ 'D:\\react\\a' ]
```

**To Reproduce**
Build in yarn 2 after deleting or renaming .git

**Expected behavior**
Build is successful

**System**
Repeatable on Windows/Ubuntu on WSL/node:current-alpine Docker image
Repository: flutter
Issue: Linux "flutter/generated_plugin_registrant.{h,cc}" are checked into source control, building generates "// clang-format off"
Labels: tool,platform-linux,a: desktop,P2,team-linux,triaged-linux

I vaguely remember having a conversation with @stuartmorgan about why this file isn't in the ephemeral directory but I can't recall the details.

Either it should be generated in the ephemeral directory so it's not checked into source control, or if it is supposed to be checked into source control, all the `generated_plugin_registrant` files checked in the Flutter repo should be updated to have `// clang-format off` so we don't get a dirty working copy every time we build an example or integration test app.
https://github.com/flutter/flutter/pull/76662

```
diff --git a/dev/integration_tests/flutter_gallery/linux/flutter/generated_plugin_registrant.cc b/dev/integration_tests/flutter_gallery/linux/flutter/generated_plugin_registrant.cc
index 026851fa2f..f6f23bfe97 100644
--- a/dev/integration_tests/flutter_gallery/linux/flutter/generated_plugin_registrant.cc
+++ b/dev/integration_tests/flutter_gallery/linux/flutter/generated_plugin_registrant.cc
@@ -2,6 +2,8 @@
 //  Generated file. Do not edit.
 //
 
+// clang-format off
+
 #include "generated_plugin_registrant.h"
 
 #include <url_launcher_linux/url_launcher_plugin.h>
```
Repository: TypeScript
Issue: API: add function to resolve global symbol
Labels: Suggestion,In Discussion

# Suggestion

<!--
  Please fill in each section completely. Thank you!
-->

## 🔍 Search Terms

<!--
  💡 Did you know? TypeScript has over 2,000 open suggestions!
  🔎 Please search thoroughly before logging new feature requests as most common ideas already have a proposal in progress.
  The "Common Feature Requests" section of the FAQ lists many popular requests: https://github.com/Microsoft/TypeScript/wiki/FAQ#common-feature-requests

  Replace the text below:
-->

symbol, constructor, well known, global

## ✅ Viability Checklist

<!--
   Suggestions that don't meet all these criteria are very, very unlikely to be accepted.
   We always recommend reviewing the TypeScript design goals before investing time writing
   a proposal for ideas outside the scope of the project.
-->
My suggestion meets these guidelines:

* [x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
* [x] This wouldn't change the runtime behavior of existing JavaScript code
* [x] This could be implemented without emitting different JS based on the types of the expressions
* [x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
* [x] This feature would agree with the rest of [TypeScript's Design Goals](https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals).


## ⭐ Suggestion

Expose `getGlobalSymbol` on `TypeChecker`

## 📃 Motivating Example

TypeScript applies some special handling to global declarations, e.g. Array, Object, Function, ReadonlyArray,
The recent change to well known Symbols adds yet another special case for the global `Symbol` value symbol.

Currently there's no reliable way to obtain the symbol of a global declaration.

## 💻 Use Cases

To find out whether a type is iterable, I need to look for a `__@iterable` property. Previously that worked well with a hard coded string. Now I need to look up the global `Symbol` constructor type, get the type of its property `iterator` and use the `escapedName` for the lookup on the type in question. The first step is actually the tricky one, because there's no API for it.

There are a few workarounds I can think of:

* the hacky way: `checker.getSymbolsInScope('' as any, ts.SymbolFlags.Value).filter(s => s.escapedName === 'Symbol')`
* the not so obvious way:
  * find a property whose name starts with `__@` since those are symbol names
  * its `valueDeclaration` must be a `NamedDeclaration` with a `ComputedPropertyName`
  * get the type of the computed property name to obtain a reference to the `(unique) symbol` type
  * `getApparentType` to obtain a reference to the global `Symbol` type
  * its `symbol` should hopefully always refer to the global `Symbol` constructor
Repository: go
Issue: net/http: ReadResponse rejects duplicate `Transfer-Encoding: chunked` headers
Labels: NeedsInvestigation

Since Go 1.15 (commit d5734d4f2dd), http.ReadResponse (and therefore http.Transport.RoundTrip) returns an error if the response has more than one Transfer-Encoding header. 

This makes it impossible to download content from certain sites. For example, attempting to download from https://sis.cat.com/ returns the error `too many transfer encodings: ["chunked" "chunked"]`. (https://play.golang.org/p/3rJb7ClBQ_U)

If I understand the discussion around the change correctly, there are security issues related to multiple Transfer-Encoding headers on a request. But this is on a response. 

Could we be a little less strict for responses? Or allow multiple Transfer-Encoding headers if they are all "chunked"?
Repository: pytorch
Issue: [nnc] Grouped convolutions
Labels: triaged,NNC

Speed up grouped convolutions in NNC.  Examples, with current pytorch performance (measured on skylake-avx512, single-threaded)

conv2d(c_out, c_in, kernel, stride, pad, dilation, groups) | input | gflops
-- | -- | --
(144, 144, (3, 3), (2, 2), (1, 1), (1, 1), 144, False, 'zeros') | (32, 144, 56, 56) | 1.1
(48, 48, (3, 3), (2, 2), (1, 1), (1, 1), 48, False, 'zeros') | (32, 48, 112, 112) | 1.22
(96, 96, (3, 3), (2, 2), (1, 1), (1, 1), 96, False, 'zeros') | (32, 96, 112, 112) | 1.29
(192, 192, (3, 3), (2, 2), (1, 1), (1, 1), 192, False, 'zeros') | (32, 192, 28, 28) | 1.35







cc @gmagogsfm
Repository: flutter
Issue: [Proposal] Add SetCustomParameters to GoogleSignIn
Labels: c: new feature,p: google_sign_in,package,c: proposal,team-ecosystem,P3,triaged-ecosystem

I need to set custom parameters, such as 'max_age' and 'prompt', similarly to the [GoogleAuthProvider](https://firebase.google.com/docs/reference/js/firebase.auth.GoogleAuthProvider) using the [GoogleSignIn](https://pub.dev/documentation/google_sign_in/latest/) package.  It seems that only adding scopes is possible. Is setting custom parameters being worked on?
Repository: pytorch
Issue: [nnc] Optimize tiny convolutions
Labels: oncall: jit,NNC

Small convolutions don't perform particularly well, possibly due to overhead?  Speed these up with NNC.  (Perf is single-thread skylake).

Examples:
conv2d(c_out, c_in, kernel, stride, pad, dilation, groups) | input | gflops
-- | -- | --
(32, 16, (1, 1), (1, 1), (0, 0), (1, 1), 1, False, 'zeros') | (32, 32, 112, 112) | 7.6
(8, 4, (3, 3), (1, 1), (1, 1), (1, 1), 1, True, 'zeros') | (64, 8, 64, 64) | 9.15





cc @gmagogsfm
Repository: pytorch
Issue: [nnc] Optimize 1x1 convolutions
Labels: oncall: jit,NNC

Many 1x1 convolutions are well below roofline perf (single-thread, skylake), e.g.:


conv2d(c_out, c_in, kernel, stride, pad, dilation, groups) | input | gflops
-- | -- | --
(64, 256, (1, 1), (1, 1), (0, 0), (1, 1), 1, False, 'zeros') | (32, 64, 56, 56) | 16.76
(96, 128, (1, 1), (1, 1), (0, 0), (1, 1), 1, False, 'zeros') | (32, 96, 56, 56) | 18.58
(40, 120, (1, 1), (1, 1), (0, 0), (1, 1), 1, False, 'zeros') | (1, 40, 14, 14) | 18.82



cc @gmagogsfm
Repository: pytorch
Issue: [nnc] Placeholder::load braced init list fails to deduce
Labels: oncall: jit,NNC

E.g., with `Placeholder P`, `P.load({n, c, h, 0})` won't compile, since it can't deduce that it needs to cast the literal 0 to an ExprHandle.  `store` gets around this by explicitly using `std::vector<ExprHandle>` as the arg type.  We should either do that for load, or get rid of the braced version and just use the variadic template version.

cc @gmagogsfm
Repository: flutter
Issue: Local engine usage not available to android "option B" add-to-app
Labels: platform-android,tool,a: existing-apps,P2,team-android,triaged-android

**description:**
If you follow "Option B" instructions here https://flutter.dev/docs/development/add-to-app/android/project-setup there isn't a way to use a local engine.

**attempted workaround:**
In theory someone should be able to apply the correct `-P` arguments to the gradlew invocation to make it work.  In practice I couldn't get that to work.  I looked at the output from `flutter --local-engine-src-path ~/dev/engine/src --local-engine=android_debug_unopt  -v` but found some of the arguments were referencing directories that only exist temporarily so I couldn't use them in android studio.

**workaround:**
~~Switch to "Option A" defined in the aforementioned document.~~  edit: This didn't appear to work for me either.

**expected behavior:**
On iOS the settings about what engine to use are cached in the module you just run with the local engine, then hit the play button from xcode and it will launch with the local engine.  We should emulate that behavior on Android.

cc @xster @blasten
Repository: pytorch
Issue: distributed/optim/test_zero_redundancy_optimizer uses unittest.main() instead of common_utils.run_tests()
Labels: oncall: distributed

## 🐛 Bug

See also #50448

Current commit 7b54a8fc2318296aaab3cc91e4544050471987f5

cc @pietern @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @osalpekar @jiayisuse @agolynski @SciPioneer @H-Huang @mrzzd @cbalioglu @mruberry @walterddr @malfet
Repository: kubernetes
Issue: Pod probes lead to blind SSRF from the node
Labels: kind/bug,sig/network,lifecycle/frozen,sig/security,triage/accepted

Note: I disclosed this private and was asked to open in public

#### What happened:

A pod probe can be configured to send readiness probes to arbitrary destinations, including localhost on the node. This is similar to CVE-2020-8555.

Kubernetes Version:
v0.20.0 (Kind) and v1.19.6 (GKE) were tested. I suspect its an issue on all versions

Steps To Reproduce:

```
apiVersion: v1
kind: Pod
metadata:
  name: test
spec:
  containers:
  - image: docker.io/kennethreitz/httpbin
    name: httpbin
    readinessProbe:
      httpGet:
        host: 169.254.169.254
        httpHeaders:
        - name: Metadata-Flavor
          value: Google
        path: /computeMetadata/v1/
        port: 80
        scheme: HTTP
```

This shows an example of accessing GCE metadata.

Other possible targets are localhost endpoints, such as 127.0.0.1:2381/metrics (etcd metrics), or external targets (httpbin.org, for example).

Note that because this is coming from the node, we can also access internal targets (pods) running on the same node, **bypassing NetworkPolicy restrictions**.

In generally, the attack only allows sending GET requests, and does not allow viewing the data in the response.

The status code of a response is, however, returned in pod events: "HTTP probe failed with statuscode: .... This could be used to extract some degree of private information about the node.

Additionally, access to kubelet logs with verbosity >= 4 will log the full body, which would make this much work severe.

Other possible risks are that these external calls are now coming from the k8s node which may have differing firewall/routing rules than a pod would.

#### What you expected to happen:

I don't know. Its certainly a tricky problem since its a part of the API and hard to change.
Repository: flutter
Issue: [web] Modifier keys get stuck if released before other keys
Labels: team,engine,platform-web,P2,c: tech-debt,team: skip-test,team-web,triaged-web

On the web, if the following sequence of keys is received:

```
1. down(ShiftLeft)
2. down(KeyA)
3. up(ShiftLeft)
4. up(KeyA)
```
### Expected behavior:
`ShiftLeft` should be removed from `pressedKeys` in 3.

### Actual behavior:
`ShiftLeft` remains in `pressedKeys` until `KeyA` is released in 4.
Repository: neovim
Issue: docs/api: clarify multi-byte indexing conventions
Labels: documentation

<!-- Before reporting: search existing issues and check the FAQ. -->

- `nvim --version`: NVIM v0.5.0-dev+1102-g595f6e4d6
- Operating system/version: MacOS
- Terminal name/version: iTerm
- `$TERM`: zsh

### Steps to reproduce using `nvim -u NORC`
Insert this text into a buffer:
```
“So, Hermione, when you are going to look for your parents in Australia?”
```
Then:
```
:lua vim.api.nvim_buf_set_extmark(0, vim.api.nvim_create_namespace('test'), 0, 5, {end_col=13, hl_group='Error'})
```

### Actual behaviour
<img width="552" alt="Screenshot 2021-02-24 at 23 54 04" src="https://user-images.githubusercontent.com/7904185/109082293-95fa2680-76fb-11eb-9d6c-648228be9f65.png">
### Expected behaviour
<img width="540" alt="Screenshot 2021-02-24 at 23 54 40" src="https://user-images.githubusercontent.com/7904185/109082334-ac07e700-76fb-11eb-8b60-29714068f061.png">
Repository: pytorch
Issue:  Add format indexed argument support in JIT
Labels: oncall: jit

## Issue description
Currently TorchScript only supports basic format usage {}. Add indexed arguments support.




cc @gmagogsfm
Repository: go
Issue: x/tools/go/gcexportdata: Write incorrectly handles unexported identifiers that are reexported
Labels: NeedsFix,Tools

I don't have a standalone repro for this yet, but I'm seeing roughly this issue internally:

```
package a
type T struct { x int }
```

```
package b
import "a"
type T a.T
```

```
package c
import "a"
import "b"
var _ = (*b.T)((*a.T)(nil))
```

This code is valid and accepted by cmd/compile. But typechecking c with go/types will fail if: packages a and b are typechecked with go/loader; their export data is written out with gcexportdata.Write, and then read back in again with gcexportdata.Read; and finally used to typecheck package c.

I seem to remember having to recentlyish fix some import/export issues around selector identifiers like this in the compiler. Those fixes probably just need to be carried over to gcexportdata.
Repository: pytorch
Issue: Create CI test checker to ensure # of test ran is the same as # of test reported in xmlrunner
Labels: module: ci,triaged,enhancement

See #52788 and also #50448

This is not the first instance of test misusing unittest.main(). We need a way to ensure all newly added tests are covered in reporting. Also test stats are not uploaded to S3 if no XML report was generated (see #49190 and #50171)

cc @ezyang @seemethere @malfet @walterddr @pytorch/pytorch-dev-infra
Repository: rust
Issue: write! generate longer assembly compared to format!
Labels: I-slow,I-heavy,T-libs,A-fmt

```rust
use std::fmt::Write;

let mut s = String::new();
write!(s, "{}", 1);
```
seemed to be generate more assembly compared to:
```rust
let mut s = String::new();
let v = format!("{}", 1);
s.push_str(&v);
```

For the above example, https://rust.godbolt.org/z/KPdra8

A larger example available at https://rust.godbolt.org/z/TcsbMK
Repository: youtube-dl
Issue: nzherald.co.nz
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.02.22. 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.02.22**
- [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://www.nzherald.co.nz/sport/americas-cup-2021-beyond-the-cup-tracks-team-new-zealand-amid-top-speed-rumours/E4GRJHXVOAP4KWZTW53YDNNPPA/
- Single video: https://www.nzherald.co.nz/nz/politics/covid-19-coronavirus-one-final-papatoetoe-high-student-to-be-found-as-1000-kmart-shoppers-deemed-close-contacts-told-to-isolate/CHFLKC3BI2VA3DCUMZT73SPXGA/


## Description

<!--
Provide any additional information.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
-->

Both example videos play OK on Firefox, but youtube-dl gives an "Unsupported URL" error.
Repository: flutter
Issue: Get widget location offset is different within/without MethodChannel
Labels: framework,f: material design,a: existing-apps,has reproducible steps,found in release: 2.10,found in release: 2.13,team-design,triaged-design

Generally, we can use GlobalKey and RenderBox.localToGlobal API to get widget's location:

```dart
RenderBox renderBox = _globalKey.currentContext.findRenderObject();
Offset textOffset = renderBox.localToGlobal(Offset.zero);
```

Every thing works just fine EXCEPT you're not using flutter MethodChannel!

By adding MethodChannel, the offset calculated by above API is different!

Code to reproduce the issue: https://github.com/KenChoi1992/flutterlocation
Repository: rust
Issue: Suggested resolution to "unconstrained generic constant" does not make sense for types other than usize
Labels: A-diagnostics,T-compiler,A-suggestion-diagnostics,F-generic_const_exprs

Based on a [StackOverflow question](https://stackoverflow.com/q/66361365/3402566).

Given the following code: ([Playground](https://play.rust-lang.org/?version=nightly&mode=debug&edition=2018&gist=664f18be85840e14c179ae3e21963461))

```rust
#![feature(const_generics)]
#![feature(const_evaluatable_checked)]

use std::ops::Mul;

pub struct Foo <const bar: i64> {
    value: f64,
}

impl<const baz: i64, const quux: i64> Mul<Foo<quux>> for Foo<baz> {
    type Output = Foo<{baz + quux}>;

    fn mul(self, rhs: Foo<quux>) -> Self::Output {
        Self::Output {
            value: self.value * rhs.value,
        }
    }
}

fn main() {
    let a: Foo<3> = Foo { value: 2.0 };
    let b: Foo<4> = Foo { value: 9.0 };
    let c = a * b;
    println!("{}", c.value);
}
```

The current output is:

```
error: unconstrained generic constant
  --> src/main.rs:11:5
   |
11 |     type Output = Foo<{baz + quux}>;
   |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
   |
help: try adding a `where` bound using this expression: `where [u8; {baz + quux}]: Sized`
```

This suggestion fails to compile, because `baz` and `quux` (and their sum) have type `i64`, while the context of a slice length requires `usize`. Ideally, it should instead suggest `where Foo<{baz + quux}>: Sized`, which does compile.

Strictly speaking, pulling out the context of the constant expression's use and checking if it implements `Sized` will not work if the type is not supposed to implement `Sized`; perhaps `PhantomData<Foo<{baz + quux}>>: Sized` would be an alternative to suggest in that case. Overall, the pattern does seem a little hacky, working around the fact that there is no way to specify a constraint that a type exists.
Repository: godot
Issue: Child class variable bug
Labels: bug,topic:gdscript

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
3.2.3-stable
<!-- Specify commit hash if using non-official build. -->


**OS/device including version:**
<!-- Specify GPU model, drivers, and the backend (GLES2, GLES3, Vulkan) if graphics-related. -->


**Issue description:**
Child class variable is null during parents class _init
but is not null when the child class have _init on its functions, even if its empty
<!-- What happened, and what was expected. -->


**Steps to reproduce:**
```gdscript
class ParentClass
   func _init() -> void:
      add_child(overwrite_this_func())
       
   func overwrite_this_func() -> Button:
       var button := Button.new()
       # Some random stuffs
       return button


class ChildClass

extends ParentClass

var pressed := false

# If this function is not commented, the pressed wont be null
#func _init() -> void : pass

func overwrite_this_func() -> Button:
   var button := Button.new()

    # pressed var will be null and cause error, but not null, if _init function is present on this child class
    button.pressed = pressed
    return button
```
**Minimal reproduction project:**
<!-- A small Godot project which reproduces the issue. Drag and drop a zip archive to upload it. -->
Repository: vscode
Issue: Issue reporter stops me from posting issues if my auth session is invalid
Labels: bug,issue-reporter

Issue Type: <b>Bug</b>

1. Sign in to github
1. Open VS Code's issue reporter
1. Back in main VS Code window, sign out of Github and then back in again 
1. Try posting an issue with the old issue reporter window

**Bug**
Nothing happens

In the dev tools I see:

```
Failed to load resource: the server responded with a status of 403 ()
```

VS Code version: Code - Insiders 1.54.0-insider (c47da720f351455681978a48edc4d9a0d095ea0e, 2021-02-24T11:23:03.943Z)
OS version: Windows_NT x64 10.0.19041

<details>
<summary>System Info</summary>

|Item|Value|
|---|---|
|CPUs|Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz (4 x 2400)|
|GPU Status|2d_canvas: enabled<br>gpu_compositing: enabled<br>multiple_raster_threads: enabled_on<br>oop_rasterization: unavailable_off<br>opengl: enabled_on<br>protected_video_decode: unavailable_off<br>rasterization: unavailable_off<br>skia_renderer: enabled_on<br>video_decode: enabled<br>vulkan: disabled_off<br>webgl: enabled<br>webgl2: enabled|
|Load (avg)|undefined|
|Memory (System)|7.04GB (1.66GB free)|
|Process Argv|--crash-reporter-id 98ab83f8-a588-4ded-9863-d25098038154 --crash-reporter-id 98ab83f8-a588-4ded-9863-d25098038154 --crash-reporter-id 98ab83f8-a588-4ded-9863-d25098038154|
|Screen Reader|no|
|VM|100%|
</details>
<!-- generated by issue reporter -->