sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: pytorch
Issue: Support @property decorator in TorchScript
Labels: triage review,oncall: jit

## 🚀 Feature
Currently, trying to use a `@property`-decorated method in TorchScript results in the following error:
```
(This attribute exists on the Python module, but it's an @property method. @property methods are not yet supported in TorchScript. Please file a feature request on Github):
```

It would be useful to support `@property` decorated methods in TorchScript rather than throwing the error.

## Motivation

<!-- Please outline the motivation for the proposal. Is your feature request related to a problem? e.g., I'm always frustrated when [...]. If this is related to another GitHub issue, please link here too -->
When implementing a `nn.Module`, it is useful to be able to distinguish between publicly-accessible instance variables and internal instance variables. Using the `@property` decorator is a useful way to distinguish between the two.

## Pitch

<!-- A clear and concise description of what you want to happen. -->
Support the `@property` decorator for methods called within TorchScript. For example:

```
import torch
from torch import nn

class Foo(nn.Module):
  def __init__(self, bar: int):
    super().__init__()
    self._bar = bar

  @property
  def bar(self) -> int:
    return self._bar

  def forward(self, input: torch.Tensor) -> torch.Tensor:
    return input + self.bar

foo = Foo(3)
torch.jit.script(foo)
```

Currently, running the above returns the following error:
```
RuntimeError: 
Module 'Foo' has no attribute 'bar' 
(This attribute exists on the Python module, but it's an @property method. @property methods are not yet supported in TorchScript. Please file a feature request on Github):
  File "<ipython-input-7-5327862aa311>", line 14
  def forward(self, input: torch.Tensor) -> torch.Tensor:
    return input + self.bar
                   ~~~~~~~~ <--- HERE
```

Adding support for `@property` methods would fix this error when calling `torch.jit.script`.

## Alternatives

<!-- A clear and concise description of any alternative solutions or features you've considered, if any. -->
Several alternatives are:
1. Avoid using the `@decorator` methods in any scripted methods, and use the internal variables (e.g. `self._bar` here) instead. However, this is poor practice when other classes / modules want to access the `@decorator` instance variable, since we are bypassing the `@decorator` public API.
2. Use instance variables instead of `@decorator` methods (e.g. `self.bar = bar` in the `__init__()` function above).

I am currently using alternative 1, but am running into issues where I need to access the internal instance variables, where I would like to use the `@decorator` public method.

## Additional context

<!-- Add any other context or screenshots about the feature request here. -->
N/A


cc @suo
Repository: three.js
Issue: VolumeShader is said to "not work very well with perspective yet"
Labels: Addons

https://github.com/mrdoob/three.js/blob/dev/examples/webgl2_materials_texture3d.html#L53 has a cautionary comment `// Create camera (The volume renderer does not work very well with perspective yet)`.

This seemed worth an issue.  Perhaps VolumeShader can be improved?  Perhaps we need a second shader, which works well with perspective?  Perhaps the caution is no longer true, can be removed to avoid confusion, and the example reverted to using perspective (to turn up any remaining problems)?

The example switched from PerspectiveCamera to OrthographicCamera in https://github.com/mrdoob/three.js/commit/5ab8129d92e249119f542a362657a1103c55f9df .  VolumeShader is https://github.com/mrdoob/three.js/blob/dev/examples/jsm/shaders/VolumeShader.js .
Thank you for your work.

@Mugen87 @takahirox @almarklein
Repository: pytorch
Issue: torch.addmv can't take as input tensors with different dtypes
Labels: triaged,module: type promotion

Is this expected?
```
>>> vec = torch.randn(3, dtype=torch.half)
>>> M = torch.randn(2, dtype=torch.float)
>>> mat = torch.randn(2, 3, dtype=torch.float)
>>> torch.addmv(M, mat, vec)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
RuntimeError: expected scalar type Float but found Half
```

Note: jax supports this behavior

>>> import jax.numpy as np
>>> a = np.array([[1, 0], [2j,3j]])
/data/users/chourdiaanjali/miniconda3/envs/myenv2/lib/python3.6/site-packages/jax/lib/xla_bridge.py:116: UserWarning: No GPU/TPU found, falling back to CPU.
  warnings.warn('No GPU/TPU found, falling back to CPU.')
>>> b = np.array([1, 2])
>>> np.matmul(a, b)
DeviceArray([1.+0.j, 0.+8.j], dtype=complex64)

cc. @nairbv
Repository: flutter
Issue: Cross-Compiling Engine for Linux doesn't build Unit Test Cases
Labels: a: tests,engine,platform-linux,P2,team-engine,triaged-engine

When cross-compiling Engine for Linux (86c0c54bef50b4e02e8f98bcd913e0293f82f7bf), relavent Unit Test Cases are not being generated.
Repository: svelte
Issue: Mutating spring store values sets the values instantly 
Labels: stale-bot,temp-stale

https://svelte.dev/repl/888343d074354aff99bde5bbee55b68d?version=3.22.2
Using the `$store.key = value` syntax with springs sets the values instantly when two keys are changed in the same frame
Should be fixed with #4742
Repository: TypeScript
Issue: Incremental type checking failure (regression)
Labels: Needs Investigation

**TypeScript Version:**  3.7.x-dev.20200505

<!-- Search terms you tried before logging this (so others can find this issue more easily) -->
**Search Terms:**

**Code**

1. clone https://github.com/falsandtru/typed-dom
2. npm i
3. open src/builder.ts (no error)
4. remove the closing parenthesis of `API(html, shadow)`
5. redo
6. error

```
Argument of type 'Factory<HTMLElementTagNameMap>' is not assignable to parameter of type 'Factory<ShadowHostElementTagNameMap>'.
  Property ''custom-tag'' is missing in type 'HTMLElementTagNameMap' but required in type 'ShadowHostElementTagNameMap'.ts(2345)
```

**Expected behavior:**
no error
**Actual behavior:**

**Playground Link:** <!-- A link to a TypeScript Playground "Share" link which demonstrates this behavior -->

**Related Issues:** <!-- Did you find other bugs that looked similar? -->
Repository: neovim
Issue: many char support for listchars's space.
Labels: enhancement,display


<!-- Before reporting: search existing issues and check the FAQ. -->

- `nvim --version`:0.4.3
- `vim -u DEFAULTS` (version: ) behaves differently? no
- Operating system/version: Winodow10
- Terminal name/version: 
- `$TERM`:

### Steps to reproduce using `nvim -u NORC`

```
nvim -u NORC

```

### Actual behaviour
  - I want to use some char to show listchars's space
    - this i try the char
      - `║`   <span style="color:green">ok</span>
      - `⿰`  <span style="color:red">no</span>
      - `●`   <span style="color:green">ok</span>
      - `¦`    <span style="color:green">ok</span>
      - `Ꝅ`   <span style="color:green;">ok</span>
      - `ɸ`   <span style="color:green">ok</span>
      - `▏`   <span style="color:green">ok</span>
      - `|`    <span style="color:red">no</span>
      - `\`   <span style="color:green">ok</span>
      ...

### Expected behaviour
  - i want to see to many char support for listchars's space.
Repository: go
Issue: cmd/compile: package-scope and function-scope types get the same name
Labels: NeedsInvestigation,compiler/runtime

This programs shows that we can get two different types with the same name.  This is permitted by the reflect package, but it seems unnecessarily confusing.  Should the compiler generate different names for these types?

```Go
package main

import (
	"fmt"
	"reflect"
)

type myint64 int64

func main() {
	var i1 myint64 = 100
	fmt.Printf("%T: %v\n", i1, i1)

	t1 := reflect.TypeOf(i1)

	type myint64 int32
	var i3 myint64 = 100
	fmt.Printf("%T: %v\n", i3, i3)

	t2 := reflect.TypeOf(i3)

	fmt.Printf("t1: %v %v\n", t1.Kind(), t1.String())
	fmt.Printf("t2: %v %v\n", t2.Kind(), t2.String())
	fmt.Println(t1 == t2)
}
```

Output:

```
main.myint64: 100
main.myint64: 100
t1: int64 main.myint64
t2: int32 main.myint64
false
```
Repository: youtube-dl
Issue: Request support for altcensored.com (same format as YT, it hosts deleted channels)
Labels: site-support-request

<!--

######################################################################
  WARNING!
  IGNORING THE FOLLOWING TEMPLATE WILL RESULT IN ISSUE CLOSED AS INCOMPLETE
######################################################################

-->


## Checklist

<!--
Carefully read and work through this check list in order to prevent the most common mistakes and misuse of youtube-dl:
- First of, make sure you are using the latest version of youtube-dl. Run `youtube-dl --version` and ensure your version is 2020.05.03. If it's not, see https://yt-dl.org/update on how to update. Issues with outdated version will be REJECTED.
- Make sure that all provided video/audio/playlist URLs (if any) are alive and playable in a browser.
- Make sure that site you are requesting is not dedicated to copyright infringement, see https://yt-dl.org/copyright-infringement. youtube-dl does not support such sites. In order for site support request to be accepted all provided example URLs should not violate any copyrights.
- Search the bugtracker for similar site support requests: http://yt-dl.org/search-issues. DO NOT post duplicates.
- Finally, put x into all relevant boxes (like this [x])
-->

- [x] I'm reporting a new site support request
- [x] I've verified that I'm running youtube-dl version **2020.05.03**
- [x] I've checked that all provided URLs are alive and playable in a browser
- [x] I've checked that none of provided URLs violate any copyrights
- [x] I've searched the bugtracker for similar site support requests including closed ones


## Example URLs

<!--
Provide all kinds of example URLs support for which should be included. Replace following example URLs by yours.
-->

- Single video: https://www.altcensored.com/watch?v=YP0-ibk2MBQ
- Single https://www.altcensored.com/watch?v=6pt8nriWCIQ
- Playlist: https://www.altcensored.com/channel/UCAhmDfQ1LfOYECmNNWgXJ7Q


## Description

<!--
Provide any additional information.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
-->

AltCensored hosts mirrors of controversial/deleted youtube channels. They mimic YT url formats.
Repository: create-react-app
Issue: Question: should we use `allowSyntheticDefaultImports:true` or `skipLibCheck:true` in every React project?
Labels: issue: proposal,needs triage

There are currently two ways of writing in the community:

In [TypeScript Documentation](https://www.typescriptlang.org/v2/docs/handbook/react-&-webpack.html#write-some-code) it write:

```ts
import * as React from "react";
```

and in CRA:

https://github.com/facebook/create-react-app/blob/9123aae7a3beaba8b245f73eb0180baa7b2f8411/packages/cra-template-typescript/template/src/index.tsx#L1

---

For users of a library, both usages are allowed. Because CRA has enabled `allowSyntheticDefaultImports` and `skipLibCheck`.

https://github.com/facebook/create-react-app/blob/30fc0bf5ed566d9b42194d56541d278013d7928c/packages/react-scripts/scripts/utils/verifyTypeScriptSetup.js#L104-L106

---

But for users of the library, the problem arises:

If I write a library using`import React from 'react';`.

Then downstream users must enable `allowSyntheticDefaultImports` or `skipLibCheck`, otherwise they will not be able to use this library in Typescript.

For users who configure webpack and tsconfig themselves, it is common to turn off `allowSyntheticDefaultImports` and `skipLibCheck`(default behavior).

---


Question: Which of the following two methods is more recommended?
* Force each project to force skipLibCheck. 
* Force each project to force enableSyntheticDefaultImports. 
* We should recommend the author of the library to use `import * as React from 'react';`, but not restrict library users to use `import * as React from 'react';` or `import React from 'react';`

Whichever method we choose, we should specify this behavior in the react / create-react-app / TypeScript documentation website, and we should have some more detailed suggestions for the react library authors and react library users.

---

related issues:
* https://github.com/facebook/create-react-app/issues/7586
* https://github.com/facebook/react/issues/11503

@ianschmitz @gaearon @RyanCavanaugh @DanielRosenwasser @orta
Repository: godot
Issue: Mouse Key is ignored when used as Shortcut in Button
Labels: bug,confirmed,topic:input,topic:gui

**Godot version:**
Godot 3.2.1

**OS/device including version:**
WIndows 10

**Issue description:**
Mouse Key that registered on input map is ignored when used as Button's Shortcut.

**Steps to reproduce:**
1. Add Mouse Button to Input Map, in this reproduction project I added it to ui_cancel
![InputMap](https://user-images.githubusercontent.com/8875839/81150360-f3c15480-8fa9-11ea-8b39-bc26006c1626.jpg)
2. Create a Button and Add an InputEventAction as its Shortcut and type the action that have the Mouse Button in step 1 (in this repro I used ui_cancel)
![Inspector](https://user-images.githubusercontent.com/8875839/81150806-8f52c500-8faa-11ea-8560-9cca4eb7165c.jpg)
3. Play the scene.
4. The Button will respond to all Keys in the ui_cancel Input Map except the mouse button.

**Minimal reproduction project:**
[Bug-MouseKeyIgnoredOnShortcut.zip](https://github.com/godotengine/godot/files/4585475/Bug-MouseKeyIgnoredOnShortcut.zip)
Repository: vscode
Issue: [semantic] proposals for new standard semantic token types 
Labels: feature-request,semantic-tokens

The new semantic token provider API comes with a list of standard token types and modifiers.
https://code.visualstudio.com/api/language-extensions/semantic-highlight-guide#semantic-token-classification

These type serve as a base across languages and by having all/most providers using theme will make easier to write theming rules across languages.

That said, semantic token providers are not forced to stick to the standard, but can add new types/modifiers, or extend existing types as seen in the doc.

This issue is to collect proposals for new types and modifiers. When making a suggestion, please add a description and a small code sample. If it exists, name the corresponding TextMate scope.

The standard token types should be be applicable across multiple languages and be useful for theming. We want to keep the set of standard tokens consistent  and coherent.


Proposed types:

| Identifier (extends)       | Description  | Ref  | Sample  | TextMate scope |
| ------------- |:-------------:| -----:| -----:| -----:|
| importKeyword (keyword)     |  keywords related to imports/includes | (1) | __import__ * from x | |
| modifierKeyword (keyword) | keywords describing a modifier | (1) | __private__ void foo(); | |
| docComment (comment) | documentation comments | #96712 | /** */ | |

Proposed modifiers:

| Identifier       | Description  | Ref  | Sample  | 
| ------------- |:-------------:| -----:| -----:| 
| unused     |  annotated all unused symbols | (2) | let __unusedVariable__; | 

References:
(1) https://github.com/microsoft/language-server-protocol/issues/968
(2) https://github.com/microsoft/vscode-languageserver-node/issues/604
Repository: godot
Issue: If you modify an exported script variable it shows it as modified even if you return the variable to its original value.
Labels: bug,topic:editor,usability

**Godot version:**
3.2.stable.official

**OS/device including version:**
Windows 10 - 1909

**Issue description:**
The exported variable will appear as if it is modified even if you manually set it to the original value it was exported as. 

**Steps to reproduce:**
Export a variable, modify it, and put it back to the original value. 

**Minimal reproduction project:**
N/A
![WRwZQBz 1](https://user-images.githubusercontent.com/34455502/81179540-29514800-8f80-11ea-91eb-7c13d9fcff24.png)

This is incredibly low priority but hopefully I'm still posting it in the correct location.
Repository: godot
Issue: Popupmenu 'items' property is an array that stores a weirdly complex pattern of objects.
Labels: discussion,topic:gui

**Godot version:**
3.2.stable.official

**OS/device including version:**
Windows 10 - 1909

**Issue description:**
If you want to reorder a PopupMenu's elements you need to modify an oddly complex array of objects manually. It seems strange that rather than being stored in a dictionary, these elements are just appended to the end of the array.

e.g. 

[Open, [Object:null], 0, False, False, 0, 0, Null, , False,
Save, [Object:null], 0, False, False, 0, 0, Null, , False,
Save As, [Object:null], 0, False, False, 0, 0, Null, , False]

**Steps to reproduce:**
Access the .items property of a PopupMenu and you will see what I mean by the ordering being weirdly complex.

**Minimal reproduction project:**
N/A

 **Suggestion:**

Perhaps document the items property and turn items into dictionaries that are stored in an array instead of appending each new item's properties onto the end of a single array.
Repository: terminal
Issue: Session text is invisible when Windows is set to "HDR mode".
Labels: Help Wanted,Area-Rendering,Issue-Bug,Product-Terminal,Priority-2

<!-- 
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨

I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING:
1. If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement.
2. If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement.
3. If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number).
4. If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement.
5. If I file something completely blank in the body, the core team may close my issue without further explanation or engagement.

All good? Then proceed!
-->

<!--
This bug tracker is monitored by Windows Terminal development team and other technical folks.

**Important: When reporting BSODs or security issues, DO NOT attach memory dumps, logs, or traces to Github issues**.
Instead, send dumps/traces to secure@microsoft.com, referencing this GitHub issue.

If this is an application crash, please also provide a Feedback Hub submission link so we can find your diagnostic data on the backend. Use the category "Apps > Windows Terminal (Preview)" and choose "Share My Feedback" after submission to get the link.

Please use this form and describe your issue, concisely but precisely, with as much detail as possible.

-->

# Environment

```none
Windows build number: 10.0.18363.0
Windows Terminal version (if applicable): 0.11.1191.0

Any other software?
```

# Steps to reproduce

Start Menu
Settings
System
Display
Play HDR games and apps - Set this to ON.

# Expected behavior

That text in the Windows Terminal would be visible as it is in "SDR mode".

# Actual behavior

All text regardless of session type (pwsh, Powershell, cmd, etc.) or ANSI codes is invisible (it has the same color as the terminal session background).
Repository: electron
Issue: Specify the resources path
Labels: enhancement :sparkles:

### 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
The `resources` directory is hard coded to be next the exec-path.
In some cases, we would like to locate it in a different dir.
For example, the main code could be in Program Files, but the resources in UserData,
so it would be easy to update the resources app.asar file without patching files in Program Files.

### Proposed Solution
Changing the code https://github.com/electron/electron/blob/979c291847f7aff333249dbe773b63f2b2059621/shell/common/node_bindings.cc#L257-L267 to be able to configure the resources dir during compilation-time or through flags/environment-variables that are passed to the executable.

### Alternatives Considered
Replacing the resources dir with a symlink.
Repository: flutter
Issue: WipError: WipError 4: {code: -32000, message: Cannot find context with specified id}
Labels: c: crash,tool,platform-web,P2,team-web,triaged-web

From `run -d chrome` from crash logging. No back trace =(

@jonahwilliams
Repository: PowerToys
Issue: [KBM] UI needs to respond to PT Settings Theme
Labels: Product-Keyboard Shortcut Manager,Area-User Interface,Priority-3

Currently UI only changes from light to dark mode in accordance with a users Windows settings.
Repository: flutter
Issue: plutil FormatException during `flutter create .`
Labels: c: crash,tool,P2,team-tool,triaged-tool

Seen in `master/1.18.0-9.0.pre.98`

We're possibly passing a flag the user's version of plutil doesn't like and not checking the exit code.

```
FormatException: Unexpected character (at character 1)
Usage: plutil -i|--infile FILE [-o|--outfile FILE] [-d|--debug]
^
  | at _ChunkedJsonParser.fail | (convert_patch.dart:1394 )
  | at _ChunkedJsonParser.parseNumber | (convert_patch.dart:1261 )
  | at _ChunkedJsonParser.parse | (convert_patch.dart:926 )
  | at _parseJson | (convert_patch.dart:31 )
  | at JsonDecoder.convert | (json.dart:505 )
  | at JsonCodec.decode | (json.dart:153 )
  | at PlistParser.parseFile | (plist_parser.dart:59 )
  | at PlistParser.getValueFromFile | (plist_parser.dart:77 )
  | at IosProject.productBundleIdentifier | (project.dart:424 )
  | at FlutterProject.organizationNames | (project.dart:105 )
  | at CreateCommand.runCommand | (create.dart:326 )
  | at FlutterCommand.verifyThenRunCommand | (flutter_command.dart:796 )
  | at _rootRunUnary | (zone.dart:1192 )
  | at _CustomZone.runUnary | (zone.dart:1085 )
  | at _FutureListener.handleValue | (future_impl.dart:141 )
  | at Future._propagateToListeners.handleValueCallback | (future_impl.dart:686 )
  | at Future._propagateToListeners | (future_impl.dart:715 )
  | at Future._completeWithValue | (future_impl.dart:526 )
  | at Future._asyncCompleteWithValue.<anonymous closure> | (future_impl.dart:560 )
  | at _rootRun | (zone.dart:1184 )
  | at _CustomZone.run | (zone.dart:1077 )
  | at _CustomZone.runGuarded | (zone.dart:979 )
  | at _CustomZone.bindCallbackGuarded.<anonymous closure> | (zone.dart:1019 )
  | at _microtaskLoop | (schedule_microtask.dart:43 )
  | at _startMicrotaskLoop | (schedule_microtask.dart:52 )
  | at _runPendingImmediateCallback | (isolate_patch.dart:118 )
  | at _RawReceivePortImpl._handleMessage | (isolate_patch.dart:169 )
```

@jmagman
Repository: flutter
Issue: [pigeon] print warning when `configurePigeon` overrides a commandline flag
Labels: c: new feature,package,team-ecosystem,p: pigeon,P3,triaged-ecosystem

cc @gaaclarke
Repository: go
Issue: x/build/cmd/gopherbot: instances of no reviewers being assigned to CLs
Labels: Builders,NeedsInvestigation

There are owners for this directory, so I'm surprised no one got auto-assigned until @hyangah added reviewers. Are there logs that I can take a look at? Happy to investigate myself, but I don't know how (or if I can) access gopherbot logs.