sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: pytorch
Issue: Overhead performance regression over time umbrella issue.
Labels: high priority,module: performance,module: internals,module: cuda,module: cpu,triaged,quansight-nack

This issue is meant to collect various performance-regression-over-time bug reports that aren't specific op regressions, that almost certainly overlap, but which we should track separately to make sure we cover all the cases.

To start:
https://github.com/pytorch/pytorch/issues/5388
https://github.com/pytorch/pytorch/issues/16717
https://github.com/pytorch/pytorch/issues/2560

cc @ezyang @gchanan @zou3519 @VitalyFedyunin @ngimel @mruberry
Repository: rust
Issue: Request: Format change for doc test cli output
Labels: T-rustdoc,C-enhancement,A-libtest,A-doctests

_Originally filed at https://github.com/rust-lang/cargo/issues/6927_

---

When running `cargo test` on a project with some doc tests I see the following output:

```
$ cargo test   
    Finished dev [unoptimized + debuginfo] target(s) in 0.06s
     Running target/debug/deps/tq_parser-9edae091aa940c1c

running 1 test
test tokenizer::tests::string_literal::simple ... ok

test result: ok. 1 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out

   Doc-tests tq-parser

running 3 tests
test src/tokenizer.rs - tokenizer::Tokenizer<'text>::new (line 158) ... ok
test src/tokenizer.rs - tokenizer::Tokenizer (line 36) ... ok
test src/tokenizer.rs - tokenizer::Token (line 122) ... ok

test result: ok. 3 passed; 0 failed; 0 ignored; 0 measured; 0 filtered out
```

This ticket is focusing on this portion:

```
   Doc-tests tq-parser

running 3 tests
test src/tokenizer.rs - tokenizer::Tokenizer<'text>::new (line 158) ... ok
test src/tokenizer.rs - tokenizer::Tokenizer (line 36) ... ok
test src/tokenizer.rs - tokenizer::Token (line 122) ... ok
```

I believe this output could be more useful, perhaps at the cost of being 'pretty', if the line numbers were joined to the source file path with a `:` to produce a clickable (in some environments) path.

For example, if the output were the following

```
   Doc-tests tq-parser

running 3 tests
test src/tokenizer.rs:158 - tokenizer::Tokenizer<'text>::new ... ok
test src/tokenizer.rs:36 - tokenizer::Tokenizer ... ok
test src/tokenizer.rs:122 - tokenizer::Token ... ok
```

I could cmd-click on `src/tokenizer.rs:158` and be taken, in my editor, directly to the doc test instead of having to open the file then navigate to the line.

This is more useful in the case of errors.

If I change the `tokenizer::Tokenizer<'text>::new` test to fail I get

```
test src/tokenizer.rs - tokenizer::Tokenizer<'text>::new (line 158) ... FAILED
```

Which could be

```
test src/tokenizer.rs:158 - tokenizer::Tokenizer<'text>::new ... FAILED
```
Repository: PowerToys
Issue: Add dock to desktop to any window
Labels: Idea-Enhancement,Product-Window Manager

Add the feature Dock to Desktop found in OneNote 2016 (keyboard shortcut Crtl + Alt + D)

Actually changes the window to a TaskBar allowing you to Maximize a second window to the screen.
Repository: go
Issue: x/playground: change vet support to happen in 1 HTTP request/response instead of 2
Labels: NeedsFix

Currently the playground makes 2 HTTP requests (/compile + /vet) instead of just 1.

(Noticed while working on #31944)

We need to do this first, otherwise the implementation of #31944 either gets nasty, or slow.

So let's just clean this up first.

Plan:
* keep old handler, to support older javascript clients
* add new "withvet=1" URL parameter to advertise to server that the client supports vet in one round trip
* return "vet" object in returned JSON
* have client skip /vet XHR if vet object key is returned

/cc @ysmolsky
Repository: godot
Issue: Local resource looses resource path
Labels: discussion,topic:core

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
<!-- Specify commit hash if non-official. -->
66baa3b

**OS/device including version:**
<!-- Specify GPU model and drivers if graphics-related. -->
Win 10

**Issue description:**
<!-- What happened, and what was expected. -->
attaching a resource to a node and then setting it to local_to_scene will set the resource_path to null

**Steps to reproduce:**
Create a Node with export var Resource and a Resource
attach Resource to node
print resource.get_path()
set Resource to local_to_scene
print resource.get_path()

**Minimal reproduction project:**
<!-- Recommended as it greatly speeds up debugging. Drag and drop a zip archive to upload it. -->
[TileTest.zip](https://github.com/godotengine/godot/files/3167637/TileTest.zip)
Repository: pytorch
Issue: Better documentation / molly-guards around use of multiprocessing with spawn in Jupyter/ipython notebooks
Labels: module: docs,triaged,enhancement

Apparently, it's really popular for users to use multiprocessing + spawn in ipython/Jupyter notebooks. For example, #17680; also, https://fb.workplace.com/groups/1405155842844877/permalink/2766399306720517/ (FB-only)

There is a loaded footgun that occurs when you combine these three ingredients: the spawn method will **run everything in your notebook top-level** (and you probably have live statements on your notebook top-level; that's why you're running in a notebook) before actually running the requested function in question. This is unexpected for users, who just expected to be directly dropped into the function in question, and leads to all sorts of fun (`mp.set_start_method` reports that it's already been called; deadlock when the child process starts trying to run the same thing again).

It would be *really good* to document this somewhere people are actually going to read it, and maybe add some runtime checks to detect if this situation is happening and help users do the right thing.

cc @colesbury @chandlerzuo
Repository: terminal
Issue: Job control and the Console API
Labels: Issue-Question,Product-Conhost,Area-Server

The Console API is presently lacking any notion of the foreground process/process group; that is, there is no equivalent to the UNIX tcsetpgrp() call, and indeed the API documentation notes that in the presence of multiple processes there is no guarantee which process will receive input read from the console.

This is really quite a serious bug — it prevents the use of any kind of job control from the shell, because the shell has no way to cause a process to stop if it tries to read from the attached console. Yes, it's possible that a shell could create pseudo consoles for every subprocess and attempt to manage this itself, but that doesn't work if the subprocesses themselves spawn subprocesses.

Please take a look at https://ftp.gnu.org/old-gnu/Manuals/glibc/html_node/Access-to-the-Terminal.html#Access%20to%20the%20Terminal which describes the UNIX behaviour; you *really* want to make sure that it's possible to do similar things in the Windows API. In particular, it must be possible to send a console control message to a process if it tries to read from a console for which it is not the foreground process or in the foreground process group; the default behaviour for the message, if unhandled, should be to suspend the process. It must also be possible, optionally, depending on the console configuration, to cause output to behave the same way.

I'd suggest adding something like

```c
BOOL ConsoleSetForegroundProcessGroup(HANDLE hConsole, HANDLE hProcessGroup);
HANDLE ConsoleGetForegroundProcessGroup(HANDLE hConsole);
```

Then I'd probably add an extra console mode flag to control the suspension of processes on output, maybe `ENABLE_SUSPEND_ON_OUTPUT`; you could also add `ENABLE_SUSPEND_ON_INPUT` and leave that off by default to maintain the present behaviour.

Finally, the console control events `CTRL_SUSPEND_FOR_INPUT` and `CTRL_SUSPEND_FOR_OUTPUT` need adding. It would also be a good idea to add, while you're about it, `CTRL_SUSPEND`, the default behaviour of which should be to suspend the process unless it's handled.
Repository: rust
Issue: `impl Trait` changes mutability requirements of closure
Labels: C-enhancement,A-diagnostics,T-compiler

This code behaves in a way that is not obvious to me:

```rust
fn main() {
    fn get_impl_trait() -> impl Default {}
    let impl_trait = get_impl_trait();

    let mut unit = ();

    let modify_unit = || {
        let _captured_impl_trait = impl_trait; // (1)
        unit = (); // (2)
    };

    modify_unit();
}
```

This code compiles without any warnings in stable Rust. But notice, the `modify_unit` closure is not declared `mut`, but it modifies the `unit` variable inside at location `(2)`.

However, when you comment out the line marked `(1)`, suddenly Rust complains that the closure is not mutable:

[Link to the code on the playground.](https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=7e64bdf75122c21de92d0d9195ca04b0)

```text
error[E0596]: cannot borrow `modify_unit` as mutable, as it is not declared as mutable
  --> src/main.rs:13:5
   |
7  |     let modify_unit = || {
   |         ----------- help: consider changing this to be mutable: `mut modify_unit`
...
13 |     modify_unit();
   |     ^^^^^^^^^^^ cannot borrow as mutable
```

What is the reason for this behavior?
Repository: flutter
Issue: [in_app_purchase] BillingResponse should be accessible
Labels: p: in_app_purchase,package,c: proposal,team-ecosystem,P3,triaged-ecosystem

The method `buyNonConsumable` of `GooglePlayConnection` calls `launchBillingFlow`, which returns `Future<BillingResponse>`. This response would be very useful to show the user why his/hers purchase was not successful.  I've seen that the equivalent `launchBillingFlow` for iOS would be `addPayment` of `AppStoreConnection`, which returns `Future<void>`. I believe this is the reason why buying methods return nothing. Makes sense. Still, I see no reason why `launchBillingFlow` could not throw an `Exception` containing `BillingResponse` if the enum value is not `BillingResponse.ok`.
Repository: terminal
Issue: Feature Request: Preview of open tabs in task bar
Labels: Issue-Feature,Help Wanted,Area-UserInterface,Product-Terminal

I would like to have each of my tabs show up in my task bar when I hover over the icon like Edge browser.
Repository: terminal
Issue: Suggestion: "One-click & snap" connect to bluetooth/serial devices and network hosts using QR Code and Code 128
Labels: Issue-Feature,Help Wanted,Area-Extensibility,Product-Terminal

IoT devices, network equipment, servers, ... can be accessed through serial over bluetooth, telnet, ssh, etc... but they can require quite a bit of information to configure the terminal properly to connect to them.
It is possible to put labels on them with their hostname or ip address, protocol to use, or for bluetooth or serial equipment, their bluetooth address, passkey, bits per second, data bits, parity, stop bits and flow control information...

Now, imagine if you could just print a QR Code or Code 128 (depending on what is easier to stick on the equipment) containing all that information, and you could just scan the label that is on the device to connect to it automatically.

For servers with a status screen but no keyboard attached, a simple `qrencode -t UTF8 "ssh://$(hostname)"` displayed on its status screen could make connecting to it much easier as well.

This could be a "Scan connection barcode" option in the [+][v] (new tab menu) of the Windows Terminal, using the Windows built-in barcode library to scan the sticker using the webcam and connecting to it in a single click and snap.

Typical scenario would be tech guy walking to a network switch, IoT endpoint, etc... and, using his laptop, connecting to it without having to setup the connection manually.
Future scenario would be when the Windows Terminal works on HoloLens 2, to be able to simply say "connect using terminal" to have a floating augmented-reality terminal window connecting to the device in front of the user.
Repository: TypeScript
Issue: when preserveConstEnums = true should allow use enum[key]
Labels: Suggestion,Awaiting More Feedback

<!-- 🚨 STOP 🚨 𝗦𝗧𝗢𝗣 🚨 𝑺𝑻𝑶𝑷 🚨

Half of all issues filed here are duplicates, answered in the FAQ, or not appropriate for the bug tracker. Even if you think you've found a *bug*, please read the FAQ first, especially the Common "Bugs" That Aren't Bugs section!

Please help us by doing the following steps before logging an issue:
  * Search: https://github.com/Microsoft/TypeScript/search?type=Issues
  * Read the FAQ: https://github.com/Microsoft/TypeScript/wiki/FAQ

Please fill in the *entire* template below.
-->

<!-- Please try to reproduce the issue with `typescript@next`. It may have already been fixed. -->
**TypeScript Version:**  3.4.0-dev.201xxxxx

<!-- Search terms you tried before logging this (so others can find this issue more easily) -->
**Search Terms:**

**Code**

preserveConstEnums = true

```ts
const enum EnumCacheName
{
	'toc_contents' = '.toc_contents.cache'
}

let { target } = yargs
	.option('target', {
		type: 'string',
	})
	.argv
;


let cache_file = path.join(ProjectConfig.cache_root, EnumCacheName[target as any]);
```

**Expected behavior:**

no error, because this code exists in .js

```js
var EnumCacheName;
(function (EnumCacheName) {
    EnumCacheName["toc_contents"] = ".toc_contents.cache";
})(EnumCacheName || (EnumCacheName = {}));
```

**Actual behavior:**

> Error: TS2476: A const enum member can only be accessed using a string literal.

**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: terminal
Issue: Feature request: theme preview / install UI
Labels: Issue-Feature,Area-UserInterface,Product-Terminal

You're probably already working on this, but it would be useful to have an issue to coordinate discussion.

@felixse's Fluent Terminal has a pretty lovely theme UI that gets it 90% right:

![image](https://user-images.githubusercontent.com/172594/57571662-d5616100-7408-11e9-9763-7c72ccca110d.png)

Some good things about it:

 - Previews of everything, rather than just the theme name
 - Users can click on the colors to edit (with a little photoshop-style editor)
 - It can import `.itermcolors` directly, rather than asking users to run `colortool`

One thing I'd add: the ability to just type in the name of [the many well known color schemes](https://iterm2colorschemes.com/), preview it, and have it installed. This saves having to download a color scheme, convert it, and put it in a directory and import it.
Repository: godot
Issue: Impractical behaviour when trying to enter a new line at the end of a collapsed section
Labels: enhancement,topic:editor,usability

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:** v3.1 stable
<!-- Specify commit hash if non-official. -->


**Issue description:** Unable to create new line in the script file when cursor is at the end of a collapsed function and also the last line of the script
<!-- What happened, and what was expected. -->


**Steps to reproduce:**
- write a function and collapse it
- make sure your function sits on the very last line in the editor
- move your cursor to the end of the function (between ":" and the dots ![collapsed_section_godot](https://user-images.githubusercontent.com/41997467/57572680-20d23a00-741e-11e9-966f-f73cfff28d90.png) indicating a collapsed section
- try hitting enter to add a new line

*EXPECTED*
new empty line of code is created after the function
*ACTUALL BEHAVIOUR*
a new line inside the function block is created
![godot_collapsed_issue](https://user-images.githubusercontent.com/41997467/57572514-a3a5c580-741b-11e9-9e13-d3f475d42a25.png)

*My recommendation for solving this*
Allow the user to navigate the cursor to a position after the three dots ![collapsed_section_godot](https://user-images.githubusercontent.com/41997467/57572618-27ac7d00-741d-11e9-90ff-8310576b364a.png)
but still on the same line of the script.
Another issue this would solve is when you hit the "backspace" key to delete a line right after a collapsed section the collapsed section becomes un-collapsed. I am not sure if this is a desired behavior.

*Update*
I also noticed that when trying to highlight a collapsed section ( which is also sitting at the end of a script) for copy & paste or move i was unable to highlight the collapsed code so i would end-up moving only the functions declaration or part of the code.
The same issue is present when i try to move a collapsed section by hitting control + direction. I would expect a collapsed section to move as a whole.

(I haven't found a related issue, feel free to correct me if i am wrong)
Repository: PowerToys
Issue: Xmouse (focus follows mouse)
Labels: Idea-New PowerToy,Product-Mouse Utilities

Please update xmouse!

I have a freeware tool that is serviceable, but it does not always play well with various pop-up UI elements, such as color palettes in tools like FileMaker, or the Windows system tray.
Repository: PowerToys
Issue: Send to X
Labels: Idea-New PowerToy,Product-File Explorer,Product-File Actions Menu

Send to X was the reason I installed PowerToys wherever I went for as long as it worked. It would be lovely to have this back again.
Repository: pytorch
Issue: CosineAnnealingLR has unexpected behavior with large step
Labels: module: optimizer,triaged

## 🐛 Bug

CosineAnnealingLR has unexpected behavior with large step. We use token count as the step for language modeling and this is often in the 1e9 range.

## To Reproduce
1. Construct torch.optim.lr_scheduler.CosineAnnealingLR with max_step=1e9
2. scheduler.step(x) for large x

Repro in this collab notebook
https://colab.research.google.com/drive/11htMBtisAHq7fQViW0inUMPAiA2J93t3

## Expected behavior

Behaves nicely with large numbers for step size

## Environment

 - PyTorch Version (e.g., 1.0): 1.1
 - OS (e.g., Linux): colab
 - Python version: 3.x
 - GPU models and configuration: None
Repository: angular
Issue: docs: please describe how to compile and load components in the browser
Labels: area: core,state: needs eng input,P4

<!--🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅

Oh hi there! 😄

To expedite issue processing please search open and closed issues before submitting a new one.
Existing issues often contain information about workarounds, resolution, or progress updates.

🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅🔅-->

# 📚 Docs or angular.io bug report

### Description

according to @alxhub there is a way to compile angular code and load it in a running Angular app, completely in the browser. i.e., JIT compilation+dynamic component loading (without entry components).
(see https://github.com/angular/angular/issues/30222)

I want to be able to "directly interface with the JIT compiler", but I could not find effective documentation.

Thank you
Repository: opencv
Issue: HEIF format support in OpenCV
Labels: feature,priority: low,category: imgcodecs,GSoC,effort: few weeks

This is a feature request, not a bug.

##### Detailed description

https://github.com/strukturag/libheif

I'd like to access HEIF files directly without conversion. Is it possible to add a build flag by using libheif?
Repository: PowerToys
Issue: Context Sensitive Emoji Menu
Labels: Idea-New PowerToy

It would be super nice if I type a sentence "I will see you at home" and press `WIN+.` to open the emoji menu then one of the top results is already if I would have searched for the last word that I typed.

Bonus points if I could go back anywhere in the sentence and do the same thing replacing things with emojis. Obviously, my most used emojis should still be near the top as well (different smiley's).

If you are wondering why someone would care so much about emoji? I live in Sweden and speak Swedish fluently but my native language is English. The Swedish language compared to the English language is quite poor in providing synonyms for words to account for different connotations and subtle nuances. So Swedes tend to use emoji's a lot to convey those emotions and nuances instead.

So even my business emails contain emojis 😁
Repository: PowerToys
Issue: Change the Z-index of notifications
Labels: Idea-New PowerToy

Notifications in Windows seem to be the highest z-index. However, sometimes those notifications don't have a close button just the arrow to perform an action. The only way to get rid of them is to select them and perform this awkward swipe...which for me always ends up with clicking the notification and opening the action.

The problem is that those notifications are show right where those background programs are running most of the time. So when I receive a notification and I want to then close the program that is running the menu to do so is hidden under the notification.

In short those menu's in the bottom right need a higher z-index if at all tweakable?
Repository: rust
Issue: improper_ctypes should not suggest reprs that lead to errors
Labels: C-enhancement,A-lints,A-FFI,T-compiler,L-improper_ctypes

Currently when the lint finds a repr(Rust) struct, enum, or union that is not FFI-safe, it generally suggests some repr attributes that could be applied to a struct/enum/union that would make it FFI-safe, which is nice. However, it bases these suggestions only on the kind of data type, without checking whether the attribute could actually be applied to the specific type.

For example, when compiling this code:

```rust
struct Rgb(u8, u8, u8);

extern "C" {
    fn get_color(name: *const u8) -> Rgb;
}
```

The compiler suggests `repr(transparent)` alongside `repr(C)`, but applying this suggestion will cause an error because the struct has multiple non-ZST fields.

Arguably the compiler should first check whether the suggestion "makes sense", and not suggest `repr(transparent)` in cases such as this one. Although it probably *should* continue to suggesting `repr(C)` even if that would then lead to another improper_ctypes about a *field* of the affected type (the user might want to repeatedly apply those suggestions to mark all necessary types as `repr(C)`).
Repository: rust
Issue: Box<dyn Error> does not impl Error
Labels: T-libs-api,C-bug

It seems reasonable to expect that `Box<dyn Error>` would implement `Error`, [but it does not](https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=36fba6624a6ac8bd886b76a4d838d7e5).

Unfortunately, extending the `impl<T: Error> Error for Box<T>` to also apply for `T: ?Sized` fails because then the `impl<'a, E: Error + 'a> From<E> for Box<dyn Error + 'a>` starts overlapping with `impl<T> From<T> for T`.
Repository: PowerToys
Issue: Configure windows borders size, scroll bars, ... in Windows
Labels: Idea-Enhancement,Product-Tweak UI Design

One of the main reasons that keeps me clinged to Windows 8.1 is the inability to set windows borders size in Windows 10.

Beside my personal (_and admittedly irrelevant_) idiosyncrasy, I'm strongly convinced that Windows 10 narrow borders can be a real issue for people with certain classes of motor and/or visual dysfunctions.

Please provide a PowerToy to make this aspect configurable.

Thanks!
Repository: pytorch
Issue: String in tensor
Labels: module: internals,triaged

## 🐛 Bug


`"whateverstring" in torch.tensor(1.)` returns True


![image](https://user-images.githubusercontent.com/8230801/57584985-5d805d00-7514-11e9-9303-d0e9f6d72257.png)

This bug happens in torch.1.1.0, I'm running pytorch with python 3.6.5, on windows linux subsystem. 

But it won't happen in 1.0.1post2, with Ubuntu 16.04 and python 3.6.8

![image](https://user-images.githubusercontent.com/8230801/57585000-9e787180-7514-11e9-8e03-c19320af7fcb.png)