sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: terminal
Issue: Only one part of unicode character get deleted on backspace in WSL/bash (again)
Labels: Area-Input,Issue-Bug,Area-TerminalControl,Product-Terminal,Priority-2

# Environment

Windows:

```
Platform ServicePack Version      VersionString
-------- ----------- -------      -------------
 Win32NT             10.0.19041.0 Microsoft Windows NT 10.0.19041.0
```

WSL2:

```
PS C:\Users\eugzol> bash -li
eugzol@DESKTOP-FAV7PTR:/mnt/c/Users/eugzol $ uname -a
Linux DESKTOP-FAV7PTR 4.19.128-microsoft-standard #1 SMP Tue Jun 23 12:58:10 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
```

# Steps to reproduce

```
eugzol@DESKTOP-FAV7PTR:~ $ cat test.sh
read user_input
echo $user_input | xxd
eugzol@DESKTOP-FAV7PTR:~ $ chmod +x test.sh
eugzol@DESKTOP-FAV7PTR:~ $ ./test.sh
а
00000000: d0d0 b00a                                ....
eugzol@DESKTOP-FAV7PTR:~ $ bash -li test.sh
а
00000000: d0d0 b00a                                ....
eugzol@DESKTOP-FAV7PTR:~ $ echo $LANG
ru_RU.UTF-8
```

When running `test_sh` script above I enter `а` (lowercase Cyrillic letter), then press backspace, then enter `а` again, then press Enter.

Hex representation of the resultant string, as shown above, is `d0 d0b0 0a`. `d0b0` is the correct representation of `а`. First `d0` is the first byte of the `а` which was entered before pressing backspace. Second byte was deleted. Last `0a` is the new line.

# Expected behavior

Hex representation of the resultant string is `d0b0 0a`. That is, all bytes of the `а` symbol is deleted when pressing backspace.

<!-- A description of what you're expecting, possibly containing screenshots or reference material. -->

# Actual behavior

<!-- What's actually happening? -->

Backspace deletes only the last byte of two-byte symbol.
Repository: pytorch
Issue: [proposal] Way to export/display the convolution algorithm found with cudnn.benchmark = True + allow to override cudnn algo choice with conv/matmul module/function algo/hint arguments
Labels: module: cudnn,feature,module: convolution,triaged

In the future it may be good to allow to export these choices to allow onnxruntime-gpu use exact same conv params and not have surprising slow-downs (https://github.com/microsoft/onnxruntime/issues/6744)

Full implementation of this feature may require some standard for exchanging these parameters between frameworks, maybe the ONNX format could include these in some metadata section

In the meanwhile, I'll check if [cudnn logging](https://docs.nvidia.com/deeplearning/cudnn/developer-guide/index.html#api-logging) is useful for displaying the decided conv params choice

cc @csarofeen @ptrblck @xwang233 @houseroad @spandantiwari @lara-hdr @BowenBao @neginraoof @SplitInfinity
Repository: electron
Issue: [Feature Request]: documentation on how to use a nightly
Labels: enhancement :sparkles:,documentation :notebook:

**Preflight Checklist**

* [X] I have read the [Contributing Guidelines](https://github.com/electron/electron/blob/master/CONTRIBUTING.md) for this project.
* [X] I agree to follow the [Code of Conduct](https://github.com/electron/electron/blob/master/CODE_OF_CONDUCT.md) that this project adheres to.
* [X] I have searched the issue tracker for a feature request that matches the one I want to file, without success.


**Problem Description**

While trying to confirm that a bug fix worked in a nightly build of Electron, I had a hard time figuring out how to install a nightly, and which errors were significant and which could be ignored.

The first time I installed a nightly, I got a ton of errors about not being able to write to .cache, but I let it run, and then was able to build and launch. The second time, electron-rebuild complained:
```
An unhandled error occurred inside electron-rebuild
Unable to find electron's version number, either install it or specify an explicit version
```

Both times it worked, but both times I did go down a rabbit hole since I didn't know if simply replacing `electron` with `electron-nightly` was the only step I should need to do.

**Proposed Solution**

I'd suggest adding a blurb to the documentation with information about the right method for installing a nightly build.

**Alternatives Considered**

The alternative is a lot of googling without much clear instruction to find, which is tractable, but inefficient since everyone needs to do it for themselves. If searching the electron repo for "nightly" yielded a clear paragraph from a docs page, that would be wonderful.

**Additional Information**

Thank you for your amazing work on Electron! ❤️🌈
Repository: kubernetes
Issue: Allow priority class values to be updated.
Labels: sig/scheduling,kind/feature,help wanted,lifecycle/stale,needs-triage

<!-- Please only use this template for submitting enhancement requests -->

#### What would you like to be added:
I wish to revisit the topic of priority classes being immutable. We made priority classes immutable so that once a priority class is created, we will not be able to update the name or value because of the following reasons:

```
-  May remove config portability: pod specs written for one cluster are no longer guaranteed to work on a different cluster if the same priority classes do not exist in the second cluster.
-  If quota is specified for existing priority classes (at the time of this writing, we don’t have this feature in Kubernetes), adding or deleting priority classes will require reconfiguration of quota allocations.
-  An existing pods may have an integer value of priority that does not reflect the current value of its PriorityClass.
```

While I agree with the first 2 points on changing priority class names, I feel that 3rd point of changing **priority class values** can be re-visited, considering we noticed a lot of people are anyways deleting and re-creating priority classes just to update the value and the updated priority class value is not reflected for existing pods.


#### Why is this needed:

While the deletion and re-creation seems like lesser of both the evils, our experience says that the deletion and re-creation of priority classes is more dangerous and error prone, considering people might delete another priority class, instead of doing just an update where an update can be rejected if there is a name mismatch or value update.

Ref: https://stupefied-goodall-e282f7.netlify.app/contributors/design-proposals/scheduling/pod-priority-api/#modifying-priority-classes


/sig scheduling
Repository: PowerToys
Issue: [FancyZones] shortcut to reapply current layout for windows that are maximized or resized
Labels: Idea-Enhancement,Product-FancyZones

## 📝 Provide a description of the new feature


* Say I have many windows 
* Some are snapped in the zones defined in FancyZones 
* Some are maximized 
* Some I have manually resized to a random (non-maximized, non facy zoned)   

Problem / Inconvenience:  
* Going through all the windows that are maximized and unmaximizing (restoring) them onto the zones from FancyZones is a quite time consuming process.   

* Proposed Solution:  
* add a keyboard shortcut that will have the effect of going through all the windows on this desktop and reapply every window to the last known zone (bring it to any of the zones if no previous zone exists).  

*  PS: This is currently doable by opening the zone editor and using the mouse to double click the desired layout. Of course a keyboard shortcut would make this like so much faster
Repository: rust
Issue: Add better diagnostic for const-generic `impl Foo for S<N>`
Labels: C-enhancement,A-diagnostics,A-resolve,T-compiler,A-suggestion-diagnostics,A-const-generics,D-papercut,D-terse

Adding better diagnostics when we resolve to a `res::Err` for N.

Original comment:
Ideally this would suggest "you meant to add a `const N` here and write this as `A<{ N }>`".

_Originally posted by @estebank in https://github.com/rust-lang/rust/pull/82055#discussion_r575739240_
Repository: godot
Issue: CSGPolygon path mesh has stair-stepping when far away from the world origin
Labels: topic:core,documentation,topic:3d

<!-- 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, mono, official


**OS/device including version:**
Windows 10
Ryzen 5 2600
RX 580 8GB, GLES3


**Issue description:**
Meshes from paths have stair stepping, so a zig-zag-behavior, when they have sharp curves, and go upwards. The farther the control points are from their origin point, the stronger the effect is.
![Screenshot 2021-02-18 181407](https://user-images.githubusercontent.com/20659579/108397469-48bc1780-7218-11eb-82a1-7d60d9c857ac.png)

**Steps to reproduce:**
Create a path
Add a CSG Polygon, using "Path" Mode as a child of the path
Create some points: create a curve (90° or sth like that), and the middle of the curve create a hill.
The path interval needs to be lower than 1. (best 0.1)
I am not sure, when exactly it occurs... maybe only with long paths?
You can see the issue in the sample project.

**Minimal reproduction project:**
<!-- A small Godot project which reproduces the issue. Drag and drop a zip archive to upload it. -->

[PathStairs.zip](https://github.com/godotengine/godot/files/6004736/PathStairs.zip)
Repository: electron
Issue: [Feature Request]: Add cssOrigin to RemoveInsertedCSS to allow the removal of user cssOrigin
Labels: enhancement :sparkles:,component/webcontents

**Preflight Checklist**

* [X] I have read the [Contributing Guidelines](https://github.com/electron/electron/blob/master/CONTRIBUTING.md) for this project.
* [X] I agree to follow the [Code of Conduct](https://github.com/electron/electron/blob/master/CODE_OF_CONDUCT.md) that this project adheres to.
* [X] I have searched the issue tracker for a feature request that matches the one I want to file, without success.


**Problem Description**

using `<webContents>.insertCSS(cssContentHere, { cssOrigin: 'user'}).then((key) => { ... });` with the cssOrigin set to **user** cannot be removed due to not providing the same cssOrigin value inside of `removeInsertedCSS` as it will default to **author**.

**Proposed Solution**

Add an additional Param to `removeInsertedCSS` to allow the use of cssOrigin similar to #19256/#19268 for `insertCSS`

**Alternatives Considered**

Create a separate function?

**Additional Information**

Electron Fiddle Gist: https://gist.github.com/d88a3fb3b36dbc6975ff685a180eff1b
as a Quick example of being able to reproduce this. Simple Change Line 22 from author to User.


https://user-images.githubusercontent.com/17199186/108412053-401c0f00-7221-11eb-96e9-6e6d36c6c479.mp4
Repository: TypeScript
Issue: Type narrowing discards template type on this-typed return value
Labels: Bug

# Bug Report

<!--
  Please fill in each section completely. Thank you!
-->

### 🔎 Search Terms

type narrowing template constraint
<!--
  What search terms did you use when trying to find an existing bug report?
  List them here so people in the future can find this one more easily.
-->

### 🕗 Version & Regression Information

Bug occurs in every version on the playground

### ⏯ Playground Link

[Playground link with relevant code](https://www.typescriptlang.org/play?ts=4.3.0-dev.20210218#code/IYIwzgLgTsDGEAJYBthjAgYge2wg3gFAIKiQzxLLYB2ApgBQCUAXAhABYCWYA3IQF9ChAGYBXGvC60qtOgFk6AHgAqCOgA8IdGgBMMObAB8AIsG0A+BhrYrWCNURJcRCawi41IwSXWyvDJgQoOggxKBoEDQA6FDlmfhIAeiSHDjowOiQ0CAxgEIR6WAywfIBPBBA6WGAxTIR-BA4ICAAHMBYUgHMuTjEQWOwAWyShrlgobDB-CCSVMta6AGUJrlbZnjAxDKSAZgAOXYAGAE5o4mDQ8Mj6AHcEM213NAfzOiCXlX4BIA)

### 💻 Code

```ts
abstract class Foo {
  abstract clone(): this;
}

function cloneMe<T extends Foo|Date>(x: T): T {
  if (x instanceof Foo) return x.clone();
  // These casts are necessary because of https://github.com/microsoft/TypeScript/issues/38309.
  return new Date(x as Date) as T;
}
```

### 🙁 Actual behavior

> Type 'Foo & Foo' is not assignable to type 'T'. 'Foo & Foo' is assignable to the constraint of type 'T', but 'T' could be instantiated with a different subtype of constraint 'Foo | Date'.

### 🙂 Expected behavior

No errors; just like the `instanceof` check narrows `x` to `Foo & T`, the `this` return type of `x.clone()` should also be `Foo & T` rather than degrading to `Foo & Foo`.
Repository: pytorch
Issue: Add autograd tests to verify correctness for R -> C cases
Labels: module: tests,triaged,module: complex,complex_autograd

We have been updating the derivative formula in `derivatives.yaml` to return correct gradients for complex functions, however we don't have many tests to verify the correctness of `R -> C` cases (i.e. the gradients returned are real valued). Some of the issues have been reported and fixed (e.g., Fix for `torch.cat` https://github.com/pytorch/pytorch/pull/51681) , however there are probably others that have gone undetected so far.
This is a tracker to systematically test the `R -> C` cases for complex functions in the new `OpInfo` based tests:

- [x] `torch.add`
- [x] `torch.sub`
- [x] `torch.mul`
- [ ] `torch.div`
- [ ] `torch.addcmul`
- [ ] `torch.addcdiv`
- [x] `torch.cat`

cc @ezyang @anjali411 @dylanbespalko @mruberry @VitalyFedyunin @walterddr
Repository: TypeScript
Issue: {} and { [K in never]: any } exhibit different simplification behavior
Labels: Bug

# Bug Report

Under some circumstances, TypeScript treats `{ [K in never]: any } & { foo?: number }` differently than `{} & { foo?: number }`.  However, this does not occur with `{ [K in never]: any } & { foo: number }`.

### 🔎 Search Terms

empty mapped object type, intersection with optional object, `K in never`

### 🕗 Version & Regression Information

Tested on `4.0.5`, `4.1.5`, `4.2.0-beta`, and the current `nightly`

### ⏯ Playground Link

[Playground link](https://www.typescriptlang.org/play#code/C4TwDgpgBAggThGAbJBlYcCWA7A5gZwBV8AeQgPigF4p8MdcoIAPYCbAE3ykKgH4oGAK7QAXFABmAQyT4IAbgBQoSDwh0AjNVgJkaeniKkA3pID2ZvuOxCAtgCMIcKAF9y8qJ6-efv7wHp-JmZIAGM2DnFpWQgAGigpcKEZKJk5ZXBoQnVgACZteEQUdCxDYhJTCQtrO0dnNw8-JuaoQOCwiNSY+MTgZKQu9JUsnIBmAt1igwJyulLGAB8oGwcndxaNnzaWDohIwTgReISklIORDNVsugAWCaL9eaMKqABtAGkoHGWIADcnAC64ik2BArigADIoJULFZlrUnK51tsQhBwntBnETn0zsJoFASABaQlQABiMAAkgAZS4jOgAVnuehKDGepg+X2wP3+cCBCVB4KhMLMNVW9XWrSCOzRnUkaSxvX6mMUQA)

### 💻 Code

```ts
type AreAllStringsTs<T> = string extends T ? true : false;
type Test1 = AreAllStringsTs<{ foo?: number }>;                         // expected: false, actual: false
type Test2 = AreAllStringsTs<{ foo: number }>;                          // expected: false, actual: false
type Test3 = AreAllStringsTs<string | number>;                          // expected: true,  actual: true
type Test4 = AreAllStringsTs<{ [K in never]: any } & { foo?: number }>; // expected: false, actual: true  <-- FAIL
type Test5 = AreAllStringsTs<{ [K in never]: any } & { foo: number }>;  // expected: false, actual: false
```

### 🙁 Actual behavior

`Test4` is `true`.

### 🙂 Expected behavior

`Test4` is `false`.

### Potential Workaround

If you have a mapped object type in which the type expression for the keys might reduce to `never`, add a conditional check to see if the keys set is empty.  For example, I encountered this when working with a type that would make all keys of an object that could take on the value `undefined` optional:

```ts
type Optionalize<T> =
    { [K in RequiredKeys<T>]: T[K] }
    & { [K in NonRequiredKeys<T>]?: T[K] };
```

Adding a check for `never` made the behavior consistent with what I was expecting when every field of the input type accepted `undefined`:

```ts
type Optionalize<T> =
    (RequiredKeys<T> extends never ? {} : { [K in RequiredKeys<T>]: T[K] })
    & { [K in NonRequiredKeys<T>]?: T[K]; };
```
Repository: go
Issue: x/website: filename in linux installation instructions is not updated if url to install page has ?download=
Labels: NeedsInvestigation

Hi,
On the release page for
https://golang.org/doc/install?download=go1.16.linux-amd64.tar.gz

The commands for installation are:
 
tar -C /usr/local -xzf go1.14.3.linux-amd64.tar.gz
Repository: pytorch
Issue: [nnc][perf] Performance decrease with CPU fusion on `freeze(script(pytorch_mobilenet_v3))`
Labels: module: performance,triaged,NNC

Here are the details of this perf decrease: https://github.com/pytorch/pytorch/issues/52264#issuecomment-779656537


cc @VitalyFedyunin @ngimel @gmagogsfm
Repository: rust
Issue: lint message paths are absolute in some cases
Labels: A-diagnostics,T-compiler

````
mkdir test
cd test
wget https://crates.io/api/v1/crates/crc/1.8.1/download
mv download crc-1.8.1.tar.gz
tar -xvf crc-1.8.1.tar.gz
cd crc-1.8.1
cargo clippy -- -Wclippy::pedantic
````

The crate seems to generate some code as part of its `build.rs` and that code is dumped somewhere into the target dir.
Now when we issue lint warnings about that code (that is inside the target dir), the path to the file is an absolute (canonicalized?) path and not a relative path from the crate root, as usual:

````
warning: long literal lacking separators
   --> /tmp/test/crc-1.8.1/target/debug/build/crc-a8ff9075a6d24ab1/out/crc64_constants.rs:509:5
    |
509 |     11317545863582056448,
    |     ^^^^^^^^^^^^^^^^^^^^ help: consider: `11_317_545_863_582_056_448`
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#unreadable_literal
````
Is that intentional?
Why is it not a relative path from the crate root like all the warnings (something like `target/debug/build/crc-a8ff9075a6d24ab1/out/crc64_constants.rs:509:5`)
Repository: PowerToys
Issue: PowerToys / FancyZones stop working
Labels: Area-Runner,Severity-High

My power toys were working normally and after update 0.31.2 the fancy zones stopped working. When trying to drag the windows with Shift nothing happens and when I click on the try icon it doesn't open anymore. Then I finish the process in the task manager and start again. It works for a short time and locks again. I uninstalled and deleted the folder in the appdata and went back to the previous version but it didn't work.
Repository: deno
Issue: Remote lock.json
Labels: cli,suggestion

For the purpose of being able to use deno as an alternative to `curl <url> | bash`, it would be useful to be able to specify `lock.json` files as a URL. This way you can run remote modules with integrity:

```sh
deno --allow-net --lock=https://domain.com/lock.json https://domain.com/mod.ts
```

The threat model here would be measurable to the integrity of both URLs. The benefit here is that given trust for the first URL, we can be confident that the dependencies for mod.ts have not been changed. Without this, we can only guarantee that the contents of mod.ts hasn't change given we trust the domain serving it's contents (the second URL).

Though not perfect, it should be stated, this threat model is a lot better than `curl <url> | bash`. :)
Repository: go
Issue: encoding/xml: document why round-trip stability is not guaranteed
Labels: Documentation,help wanted,NeedsInvestigation

It is extremely non-obvious why `encoding/xml` does not guarantee round-trip stability.  The package documentation should explain why that is the case, when this is a problem, and what users should do instead.
Repository: angular
Issue: Clarify documentation around svg fragments in templates
Labels: cross-cutting: SVG,P3,area: docs

# 📚 Docs or angular.io bug report

### Description

The Angular guide contains a section on how to use svg elements in templates, but this only deals with "complete" elements which include the base <svg> tag.

It is also useful to define components which render svg fragments within an existing parent svg element, which is not covered in the current docs. These only appear to work if the elements are prefixed with an (undefined?) "svg" namespace prefix.

Please update the docs to:
* Clarify if this is working as intended, or is just a by-product of the implementation which just happens to work.
* Include a similar example.

## 🔬 Minimal Reproduction

### Full Example
https://stackblitz.com/edit/angular-ivy-5schxp

### What's the affected URL?
https://angular.io/guide/svg-in-templates

### Reproduction Steps

```ts
// app.component.ts
interface SvgRect {
  x: number;
  y: number;
  text: string;
}
export class AppComponent  {
  rects: SvgRect[];
}
```
```html
<!-- app.component.html -->
<!-- create a separate g for each rect -->
<svg width="200" height="100">
  <g svgRect *ngFor='let rect of rects' [rect]="rect"></g>
</svg>
```
```ts
// svg-rect.component.ts
@Component({
  selector: '[svgRect]',
  template: `
      <rect
          [attr.x]="rect.x" [attr.y]="rect.y" width="80" height="30"
          stroke="black" stroke-width="1" fill="transparent"></rect>
      <text
          [attr.x]="rect.x + 40" [attr.y]="rect.y + 15"
          dominant-baseline="middle" text-anchor="middle">{{rect.text}}</text>
  `,
  styles: []
})
export class SvgRectComponent  {
  @Input() rect: SvgRect;
}
```
This template as written does not render anything, although inspecting the underlying html looks ok. Following the advice in [this SO issue](https://stackoverflow.com/questions/48453514/angular-svg-matching-component-with-svg-elements), adding the "svg" namespace prefix to the component template elements does render correctly:
```ts
  template: `
      <svg:rect
          [attr.x]="rect.x" [attr.y]="rect.y" width="80" height="30"
          stroke="black" stroke-width="1" fill="transparent"></rect>
      <svg:text
          [attr.x]="rect.x + 40" [attr.y]="rect.y + 15"
          dominant-baseline="middle" text-anchor="middle">{{rect.text}}</text>
  `,
```
Would also be good to clarify whether the selector should include the svg namespace if restricted to a specific element:
```ts
  selector: 'g[svgRect]',
  selector: 'svn:g[svgRect]',
```
Thanks
Repository: vscode
Issue: DND to download out of remote doesn't work
Labels: bug,upstream,file-explorer

- Drag a file from the explorer to Finder
- It tries to download, but the browser says "Couldn't download - No File"
![image](https://user-images.githubusercontent.com/323878/108445762-d1838500-7211-11eb-8c8d-d9da2f07d496.png)

- But right click -> download works fine
Repository: pytorch
Issue: Could you please increase the wheel compiled Linux (aarch64) GPU, thank you very much!
Labels: triaged,module: arm

## ❓ Questions and Help

### Please note that this issue tracker is not a help form and this issue will be closed.

We have a set of [listed resources available on the website](https://pytorch.org/resources). Our primary means of support is our discussion forum:

- [Discussion Forum](https://discuss.pytorch.org/)
Repository: go
Issue: time: js/wasm sets time.Local to a UTC offset instead of the actual timezone
Labels: NeedsInvestigation,arch-wasm,OS-JS

In `time/zoneinfo_js.go` the local timezone is determined by the UTC offset returned by the JS call: 

```js
(new Date()).getTimezoneOffset()
```

However `UTC offset != timezone` and this behaviour causes issues with displaying 'daylight savings' times. This behaviour is unexpected and undocumented and I would expect that time.Local is set to the JS environment's timezone (where available) instead of the current UTC offset.

The timezone can be determined in JS with the Internationalization API:

```js
Intl.DateTimeFormat().resolvedOptions().timeZone
```

As long as the tzdata package has been imported/embedded and the its init function runs first, adding the following code to the beginning of initLocal should (in theory) fix this issue.

```go
func initLocal() {
	if intl := js.Global().Get("Intl"); !intl.IsUndefined() {
		var err error
		tz := js.Global().Get("Intl").Call("DateTimeFormat").Call("resolvedOptions").Get("timeZone").String()
		Local, err = LoadLocation(tz)
		if err == nil {
		    return
		}
	}
	...
```