sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: TypeScript
Issue: Reduce amount of code necessary to quickly format the diagnostics to a string
Labels: Suggestion,Awaiting More Feedback

## Search Terms

FormatDiagnosticsHost

## Suggestion

Often when someone is working with the TypeScript compiler API their type checker might have diagnostics. It would be useful for people to quickly check them by typing:

```ts
ts.formatDiagnostics(ts.getPreEmitDiagnostics(program))
````

Instead of:

```ts
ts.formatDiagnostics(ts.getPreEmitDiagnostics(program), {
    getCurrentDirectory: () => ts.sys.getCurrentDirectory(),
    getCanonicalFileName: f => f,
    getNewLine: () => "\n"
})
````

Possible solution is to make providing `FormatDiagnosticsHost` optional or add a default `FormatDiagnosticsHost` that people can use (or both).

Also for `ts.formatDiagnostic` and `ts.formatDiagnosticsWithColorAndContext`

## Use Cases

Easier for people to quickly check the diagnostics while they're developing with the compiler API.

## Checklist

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, etc.)
* [x] This feature would agree with the rest of [TypeScript's Design Goals](https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals).

## Related Issues

#29581
Repository: pytorch
Issue: Sorting in embedding_dense_backward_cuda takes very long time
Labels: module: performance,module: cuda,triaged,module: embedding

Sorting the indices in embedding_dense_backward_cuda takes very long time. In BERT training, the sorting part in embedding_dense_backward_cuda takes almost 99% of the embedding backward time, which slows down backward pass a lot. Can anyone explain why are the indices sorted here? Is there anyway to improve this? Thanks a lot!

cc @ngimel @VitalyFedyunin @mruberry
Repository: node
Issue: addons/openssl-binding/test very flaky on ARM
Labels: build,arm,flaky-test

***(Edit: see [comment below](https://github.com/nodejs/node/issues/30786#issuecomment-617575712) for recommendations if you are running into related problems on your ARMv7 platforms)***

arm-fanned has been offline for quite some time now since they've needed some heavy work, https://github.com/nodejs/build/issues/1840

but I'm bringing them back online now, and it seems that at least one thing has crept in while it was offline. addons/openssl-binding/test is flaky on the Pi 3's. So far I've only seen it fail there but it could be elsewhere too, there haven't been enough runs to be sure.

Error output (@guybedford's run just now) from https://ci.nodejs.org/job/node-test-binary-arm-12+/2928/

`Error in `out/Release/node': corrupted double-linked list: 0x045ed160`

/cc @addaleax and @danbev since e66a2acc4c looks like a possible candidate.
Repository: vscode
Issue: Proxy configuration doesn't allow URLs with port number AND pac files
Labels: feature-request,proxy

<!-- Please search existing issues to avoid creating duplicates. -->
<!-- Also please test using the latest insiders build to make sure your issue has not already been fixed: https://code.visualstudio.com/insiders/ -->

<!-- Use Help > Report Issue to prefill these. -->
- VSCode Version: 1.40.2 (user setup)
- OS Version: Windows_NT x64 10.0.18362

Steps to Reproduce:

1.` File` -> `Preferences` -> `Settings`-> `Application`-> `Proxy`
2. Under `Proxy` try entering URL `https://user:password@proxyurl.com:8080/file.pac`
3. Error message appears: `Value must match regex `a regex`.

![image](https://user-images.githubusercontent.com/4049421/70115911-2bd50c00-1687-11ea-807c-1a01c98e2a0d.png)
4. Both PAC file `https://user:password@proxyurl.com/file.pac` and port `https://user:password@proxyurl.com:8080` work independently, but not together.

<!-- Launch with `code --disable-extensions` to check. -->
Does this issue occur when all extensions are disabled?: Yes/No
Repository: pytorch
Issue: Method to broadcast parameters/buffers of DDP model
Labels: oncall: distributed,triaged,enhancement

## 🚀 Feature

Add DDP method to allow a user to broadcast parameters and/or buffers manually.

## Motivation

We only have the constructor kwarg `broadcast_buffers` at the moment (defaults to `True`, many folks set it to `False` for performance reasons). This flag toggles between never broadcasting buffers or always broadcasting buffers. There may be some schedule that is more optimal for the user (e.g. once per epoch) that is impossible to encode in DDP as is. Therefore, we should add a function to broadcast the buffers (and while we're at it, the parameters), so the user can call it whenever.

## Pitch

```python
def broadcast_parameters(self, root=0):
  # ...
def broadcast_buffers(self, root=0):
  # ... 
```

We'll need to rename the boolean member field and move existing functionality around.

## Additional context

The broadcast of model parameters that is done from the constructor can directly use the proposed function as well. The broadcast done today is a "coalesced" version, where first a subset of tensors is concatenated in a single buffer before running the broadcast, to make it more bandwidth-optimal.

cc @pietern @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @xush6528
Repository: create-react-app
Issue: Microfrontend support: Use credentials in dynamic imports from external sources
Labels: issue: proposal,needs triage

### Is your proposal related to a problem?

I have a Microfrontend which fetches components from different APIs. Currently I did this with React.lazy loading and for external sources I had to ignore webpack like so import( /*webpackIgnore: true*/ '${API.ORIGIN}/plugin'). Now what I need is to pass credentials for those dynamic imports.

### Describe the solution you'd like

I would like create-react-app to use-credentials on dynamic imports from external resources on default.

### Describe alternatives you've considered
I considered using react-app-rewired or eject create-react-app and edit the webpack config to use credentials on dynamic imports from other sources. Nevertheless I really would prefer create-react-app to support microfrontends and code-splitting across multiple projects.
Repository: rust
Issue: Licensing: It's unclear exactly what license text must be included in a Rust binary.
Labels: T-core,A-licensing

Previous context: <https://internals.rust-lang.org/t/does-the-standard-library-have-linking-exceptions-to-their-licensing/11377>

Now obviously if you have `[dependencies]` from crates.io then you have to follow each appropriate license.

But if you just make a "no dependency" program, you're still including all sorts of code from `std`/`alloc`/`core`/`libm`/`libc`/`compiler_builtins`/`hashbrown` and possibly others I'm not even thinking of.

* Can we please get a very clear, preferably single file, text to be included as the "attribution" for building a Rust program at all.
Repository: flutter
Issue: `Stack` relayout is too pessimistic with regard to `Positioned` children
Labels: framework,c: performance,c: rendering,P2,team-framework,triaged-framework

`RenderStack` lays out all its `Positioned` children with `parentUsesSize: true` even though it doesn't always use the children's sizes. When `overflow != Overflow.clip` and `Positioned.left`/`Positioned.bottom` aren't set, the child's size isn't used.

This causes worse performance when `Stack` is used to make, for example, a `Slider` or 2-dimensional color picker where thumb can be dragged around.

(I've resorted to drawing the thumb with a `CustomPaint` that doesn't change size in some cases, but that isn't always convenient.)
Repository: TypeScript
Issue: Please provide better feedback for issue 20357
Labels: Suggestion,Experience Enhancement

## Search Terms

disableSizeLimit, "Find All References"

## Suggestion

Issue #20357 is closed. That's great, what's not great is the poor error message feedback that the TS server provides. There is a project so telling the user there isn't one is confusing. The fix is not obvious and neither is the issue. Any chance of an improvement in the error text?

## Use Cases

## Examples

## Checklist

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, etc.)
* [ x] This feature would agree with the rest of [TypeScript's Design Goals](https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals).
Repository: godot
Issue: Inserting key in animation adds tracks for unselected nodes
Labels: bug,topic:editor

I'm using Godot 3.2.beta2.

I'm doing an animation, and I want to add keys, however, when I do so, extra tracks get added for nodes that are no longer selected. Note: I have three different animationplayers, each with a different set of nodes to animate.

I see there are a lot of issues logged for AnimationPlayer, but did not seem to find this issue.

In the image below, I have moved the handbonenode, using IKchain this means I need to key three bone nodes, but for some reason belly and back bone nodes also get added, even though they are not selected.

![3 2 bug](https://user-images.githubusercontent.com/50481103/70139604-1960cf80-1693-11ea-9163-3d037b814f29.png)
Repository: flutter
Issue: In a Stack, overflow is only applied if there is at least one Positioned element.
Labels: framework,engine,c: proposal,P2,team-engine,triaged-engine

In a Stack, overflow is only applied if there is at least one Positioned element.
If there isn't, overflow is ignored. That's kind of weird. Stack children may overflow
even if they are not positioned. So if you use overflow = Overflow.clip and you don't have
Positioned elements it won't clip. I believe the default should be Overflow.visible 
(since this is less expensive, right?) but if you make it Overflow.clip it should really clip 
no matter what.

**Otherwise, all this mess should be explained in the overflow parameter documentation.**

To make this a non-breaking change (or to be precise a less breaking change) you could
make the default overflow null, which means it keeps the current behavior, but if you
really define and Overflow it should honor it, no matter if positioned elements are present.
Repository: flutter
Issue: Standardized parameters for widgets
Labels: framework,f: material design,c: proposal,P3,team-design,triaged-design

### Problem

When setting a new WIdget, most of them have a standard value to look up for, but not all of them. There are widgets with default parameters at their constructor method, that makes it not look up to Themes and other kind of data configuration.

### Proposal
I hope for a standard way of development as well as slowly changing widgets with the current structure to the new one.

The new standard would be to make widgets follow a sequential check for values and avoid using it on their constructor unless there's no more parents to look up to.

```dart
Widget({double value}) {
    exampleSize = value ?? Theme.of(context).example.size ?? MaterialDesign.example.size
}
```

That way, setting Themes and possibly other kind of group settings would make life easier while not affecting current way of constructing.
Repository: flutter
Issue: [camera] allow different resolution for takePicture and imageStream
Labels: c: new feature,customer: crowd,p: camera,package,c: proposal,team-ecosystem,P3,triaged-ecosystem

## Use case

I am trying to use a live preview to detect certain points in real-time in an image stream and overlay them on the preview. This means that the resolution of the streaming images needs to be relatively low.
When the user hits "take photo", a best quality picture should be recorded. This should use any in-device processing etc available (eg HDR+, max native resolution etc) to create the best possible jpg on the file system.

Currently, if I set ResolutionPreset to max, the streaming images AND the takePicture will be affected. This means that I have to

- stop image stream
- stop and dispose the current controller
- handle the preview stopping visually
- create a new controller with high resolution
- execute takePicture
- stop and dispose the high-res controller
- create a new low-res controller
- start the preview again
- start image stream

This results in a bad user experience as it takes a long time, and convoluted code.

## Proposal

It would be good to be able to specify a resolution/quality to takePicture, startImageStream and the CameraPreview independently. As a first thing a simpler quality option to the takePicture function would already be ok. Or just change the default for takePicture to max or similar, as its not going to be used for real-time stuff anyway if its via the file system.
Repository: rust
Issue: Linux version of nonblocking TcpListener::accept differs from other platforms
Labels: O-linux,T-libs-api,C-bug

If `TcpListener` is in non-blocking mode then accepted streams inherit this property on all platforms except Linux, where `accept4` is used instead of `accept`.

This code (with commented out `stream.set_nonblocking(false)`)  unexpectedly works as expected on Linux only, because `stream` is in blocking mode.

```
 let listener = TcpListener::bind("0.0.0.0:3333").unwrap();
    listener.set_nonblocking(true).expect("set nonblocking");

    println!("Server listening on port 3333");
    loop {
        match listener.accept() {
            Ok((mut stream, _)) => {
                println!("New connection");
                //stream.set_nonblocking(false).expect("nonblock");
                stream
                    .set_read_timeout(Some(Duration::from_secs(10)))
                    .expect("set timeout");
                let mut data = [0 as u8; 2];
                match stream.read_exact(&mut data) {
                    Ok(_) => println!("Got message!"),
                    Err(e) => println!("Failed to receive data: {:?}", e),
                }
            }
            Err(_e) => {
                println!("Error: {:?}", e);
            }
        }
    }

```

`accept4` man page explains:
```
On Linux, the new socket returned by accept() does not inherit file status
flags such as O_NONBLOCK and O_ASYNC from the listening socket. 
This behavior differs from the canonical BSD sockets implementation.
Portable programs should not rely on inheritance or noninheritance
of file status flags and always explicitly set all required flags on the socket
returned from accept(). 
```

I agree that code above should also set non_blocking explicitly, at the same time it would be nice to behave the same way on all platoftms.

## Meta
rustc --version --verbose
rustc 1.37.0 (eae3437df 2019-08-13)
binary: rustc
commit-hash: eae3437dfe991621e8afdc82734f4a172d7ddf9b
commit-date: 2019-08-13
host: x86_64-unknown-linux-gnu
release: 1.37.0
LLVM version: 8.0

(but it applies to multiple versions)
Repository: go
Issue: x/mobile: Add support for callback blocks in ObjC reverse bindings with matching Go function signatures
Labels: help wanted,NeedsInvestigation,FeatureRequest,mobile

<!-- Please answer these questions before submitting your issue. Thanks! -->

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version go1.13.4 darwin/amd64
</pre>

### Does this issue reproduce with the latest release?

Yes

### What operating system and processor architecture are you using (`go env`)?

<details><summary><code>go env</code> Output</summary><br><pre>
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/dave/Library/Caches/go-build"
GOENV="/Users/dave/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GONOPROXY=""
GONOSUMDB="github.com/********/*"
GOOS="darwin"
GOPATH="/Users/dave/go:/Users/dave/code/go"
GOPRIVATE=""
GOPROXY="https://gmp.devclops.com"
GOROOT="/usr/local/Cellar/go/1.13.4/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/1.13.4/libexec/pkg/tool/darwin_amd64"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/4j/j8tb1q2s4kd3rdn0v_hzh_6m0000gn/T/go-build562373493=/tmp/go-build -gno-record-gcc-switches -fno-common"
</pre></details>

### What did you do?

Attempted to use a function pointer as a block parameter in ObjC reverse bindings

### What did you expect to see?

I hoped it would work

### What did you see instead?

Compilation error: `implicit type "void" not found`

Assume the function:

```
import (
	"ObjC/UserNotifications"
	"ObjC/UserNotifications/UNUserNotificationCenter"
)

func testBinding() {
	unc := UNUserNotificationCenter.CurrentNotificationCenter()
	handler := func(settings *UserNotifications.UNNotificationSettings) {
		// TODO
	}
	unc.GetNotificationSettingsWithCompletionHandler(handler)
}
```

Getting reference to the `UNUserNotificationCenter` singleton compiles and works. However, simply referencing the [`GetNotificationSettingsWithCompletionHandler`](https://developer.apple.com/documentation/usernotifications/unusernotificationcenter/1649524-getnotificationsettingswithcompl?language=objc) function causes a compiler error when trying to understand the method signature looking for a completion handler. 

```
GOMOBILE=/Users/dave/go/pkg/gomobile
WORK=/var/folders/4j/j8tb1q2s4kd3rdn0v_hzh_6m0000gn/T/gomobile-work-846542795
GOOS=darwin CGO_ENABLED=1 /Users/dave/go/bin/gobind -lang=go,objc -outdir=$WORK -tags=ios poc/poc
rm -r -f "$WORK"
gomobile: /Users/dave/go/bin/gobind -lang=go,objc -outdir=/var/folders/4j/j8tb1q2s4kd3rdn0v_hzh_6m0000gn/T/gomobile-work-846542795 -tags=ios poc/poc failed: exit status 1
2019/12/04 13:30:59 implicit type "void" not found
```
Repository: vscode
Issue: [themes] IconTheme doesn't work with spaces in names
Labels: bug,themes

Issue Type: <b>Bug</b>

1) have a file or folder that contains a space in its name, for ex 'Some stuff' folder
2) set up a custom icon theme and include a specific icon for folder/file of that name, for ex. :
```
"folderNames": {
        "Some stuff": "_special_icon"
    }

```
Result: icon is not applied to the 'Some stuff' folder.
Expected it to apply.

VS Code version: Code - Insiders 1.41.0-insider (7f1975a4fb29d0a788c21f17eb2aad056fec22d8, 2019-12-04T10:20:42.228Z)
OS version: Windows_NT x64 10.0.17763


<!-- generated by issue reporter -->
Repository: pytorch
Issue: Issues in linking libtorch c++ 
Labels: module: build,triaged

In file included from /usr/local/libtorch/include/c10/core/Device.h(5),
                 from /usr/local/libtorch/include/c10/core/Allocator.h(6),
                 from /usr/local/libtorch/include/ATen/ATen.h(3),
                 from /usr/local/libtorch/include/torch/csrc/api/include/torch/types.h(3),
                 from /usr/local/libtorch/include/torch/csrc/api/include/torch/data/dataloader_options.h(4),
                 from /usr/local/libtorch/include/torch/csrc/api/include/torch/data/dataloader/base.h(3),
                 from /usr/local/libtorch/include/torch/csrc/api/include/torch/data/dataloader/stateful.h(3),
                 from /usr/local/libtorch/include/torch/csrc/api/include/torch/data/dataloader.h(3),
                 from /usr/local/libtorch/include/torch/csrc/api/include/torch/data.h(3),
                 from /usr/local/libtorch/include/torch/csrc/api/include/torch/all.h(4),
                 from /usr/local/libtorch/include/torch/csrc/api/include/torch/torch.h(3),
                 from /home/sangeeta/ParMooN/ParMooN/2DPrograms/CD2D_ParMooN.C(27):
/usr/local/libtorch/include/c10/util/Exception.h(45): error: expected a declaration
    Error(
    ^
Getting this error. I checked thoroughly. Everything is declared already.
Repository: vscode
Issue: Should pasting newline with auto-indent move the cursor + inconsistencies when pasting newlines
Labels: typescript,javascript,polish,editor-autoindent,under-discussion,on-unit-test

Issue Type: <b>Bug</b>

- Have an indent block, eg:
```
() => {

}
```
- Paste something with a trailing newline
- Expected: The newline is indented and your cursor starts at the right indent position
```
() => {
	() => {
		a
	}
	<- Cursor is here
}
```
- Actual: The newline is indented but your cursor stays at the beginning of the line
```
() => {
	() => {
		a
	}
<- Cursor is here <indent after>	
}
```

Also when pasting content with a trailing newline, only pastes with 2+ lines (that isn't already correctly indented) end up indenting the last newline.
```
() => {
	() => {}
<- Cursor is here, no indent
}
```
```
() => {
	() => {

	}
<- Cursor is here <indent after>
}
```
```
() => {
	() => {

	}
<- Cursor is here, no indent (as pasted content already had correct indent)
}
```

VS Code version: Code 1.40.2 (f359dd69833dd8800b54d458f6d37ab7c78df520, 2019-11-25T14:54:45.096Z)
OS version: Windows_NT x64 10.0.19033
Repository: vscode
Issue: [themes] Minimal icon theme should use icon font
Labels: feature-request,icons-product,lcd-text-rendering

![image](https://user-images.githubusercontent.com/6561887/70174751-54123a00-168a-11ea-993a-b6e0b2e98c49.png)

Look at multiroot icon, file icons in explorer, file icons in search panel.
refs #85143
Repository: vscode
Issue: Search view checkboxes use greyscale aa
Labels: bug,lcd-text-rendering

![image](https://user-images.githubusercontent.com/6561887/70175118-f5998b80-168a-11ea-90e8-470a9a6eeda3.png)

not sure if this is equivalent to https://github.com/microsoft/vscode/issues/85589 since the text in the box is using proper SPAA

refs #85143
Repository: vscode
Issue: Call Hierarchy should not have nodes with chevrons without any children
Labels: under-discussion,callhierarchy

Issue Type: <b>Bug</b>

It feels strange (read: broken) to me that if there are no children of a node in the call hierarchy it still has a chevron, but no children when expanded. IMO, it should have either not have a chevron if possible (and not expensive), or add a dummy child node that says there are no callers/callees.

![image](https://user-images.githubusercontent.com/641685/70176623-c729a980-16a6-11ea-8ffe-9aa7ff1730b2.png)


Refs: #85924

VS Code version: Code - Insiders 1.41.0-insider (7f1975a4fb29d0a788c21f17eb2aad056fec22d8, 2019-12-04T10:20:42.228Z)
OS version: Windows_NT x64 10.0.19033

<details>
<summary>System Info</summary>

|Item|Value|
|---|---|
|CPUs|Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz (8 x 4008)|
|GPU Status|2d_canvas: enabled<br>flash_3d: enabled<br>flash_stage3d: enabled<br>flash_stage3d_baseline: enabled<br>gpu_compositing: enabled<br>metal: disabled_off<br>multiple_raster_threads: enabled_on<br>oop_rasterization: disabled_off<br>protected_video_decode: unavailable_off<br>rasterization: enabled<br>skia_renderer: disabled_off<br>surface_control: disabled_off<br>surface_synchronization: enabled_on<br>video_decode: enabled<br>viz_display_compositor: enabled_on<br>viz_hit_test_surface_layer: disabled_off<br>webgl: enabled<br>webgl2: enabled|
|Load (avg)|undefined|
|Memory (System)|31.93GB (17.91GB free)|
|Process Argv||
|Screen Reader|no|
|VM|0%|
</details>
<!-- generated by issue reporter -->