sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: cmd/compile: unnecessary bounds check in a loop passed as a closure
Labels: Performance,NeedsInvestigation,compiler/runtime

<!--
Please answer these questions before submitting your issue. Thanks!
For questions please use one of our forums: https://github.com/golang/go/wiki/Questions
-->

### What version of Go are you using (`go version`)?

<pre>
$ go version

go version go1.13.10 darwin/amd64
</pre>

### Does this issue reproduce with the latest release?

Yes, using `amd64 gc (tip)` on go.godbolt.org.

### 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/jordan/Library/Caches/go-build"
GOENV="/Users/jordan/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/jordan/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go@1.13/1.13.10_1/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go@1.13/1.13.10_1/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/kz/_525lwtd4f7cxsb_qzwp0mv00000gn/T/go-build573559598=/tmp/go-build -gno-record-gcc-switches -fno-common"

</pre></details>

### What did you do?

I loaded the following program into go.godbolt.org:

```
package foo

func run(f func ()) {
   f()
}

func foo(c []int64, n int, v int64)  {
   run(func() {
       _ = c[n-1]
       for i := 0; i < n; i++ {
           c[i] = v
       }
   })
}
```

https://go.godbolt.org/z/iPaz7f

### What did you expect to see?

I expected that the bounds check for `c[i]` would be eliminated because of the early bounds check triggered by `c[n-1]`, the way that it would if the inner loop wasn't sent as a closure to `run`.

This should be possible because `c` doesn't get assigned to within the closure.

### What did you see instead?

The compiler emits a bounds check for `c[i]`.
Repository: svelte
Issue: Store is not being updated synchronously / store value is incorrect
Labels: bug

**Describe the bug**
Store value is not up-to-date if using a `$` subscription inside a subscription function. 

**To Reproduce**
1. Open [this REPL](https://svelte.dev/repl/a1110f77ded84950b2706760713cdf51?version=3.23.2).
2. Open your browser console.
3. Click on "Increment" a few times.
4. Click on "Cause reset".
5. Check your console.

For a real life example look at [this REPL](https://svelte.dev/repl/62bcce156ab948b69a8dae228fd73d13?version=3.23.2).

![image](https://user-images.githubusercontent.com/6671521/85380947-2cab8d80-b546-11ea-816d-e38cb39281d2.png)

**Expected behavior**
Value should be in sync.

**Severity**
Blocking
Repository: pytorch
Issue: Add batched torch.combinations
Labels: triaged,enhancement,module: batching,function request

## 🚀 Feature
I would like `torch.combinations` to support a batch of vectors. So, the signature would change from: `torch.combinations(input, r=2, with_replacement=False)` to `torch.combinations(input, r=2, with_replacement=False, dim=0)`. 

For example, if the input tensor has shape `[n, d]` and `dim=0`, the resulting tensor would have shape `[n', d, r]`. The dimension with number of elements per combination (i.e. `r`) is added to the end. The value of  `n'` depends on `with_replacement` (e.g. would be n choose r in the case of `with_replacement=False`).

## Motivation
I have a tensor of n samples with d features each. I am interested in computing the Hodges-Lehmann estimate of the median on each feature (independently). I am not sure how to do this efficiently right now, but with the proposed addition, it would just be the following:

```python
matrix = ....  # n x d tensor
medians = torch.combinations(matrix).mean(-1).median(0).values
```

I'm sure there are many other use cases for this as well.

## Alternatives

If there is an efficient way to accomplish this with the current available methods, please let me know as it would make this feature request unnecessary.

The semantics on the default dimension and where the extra dimension is added could also changed based on convenience/efficiency concerns.
Repository: godot
Issue: Intensive CPU/GPU usage when displaying animated content in the editor scene
Labels: bug,topic:editor,confirmed,performance

___

***Bugsquad note:** This issue has been confirmed several times already. No need to confirm it further.*

___

<!-- Please search existing issues for potential duplicates before filing yours:	
https://github.com/godotengine/godot/issues?q=is%3Aissue	
-->

**Godot version:**
- 3.2.1.stable.official
- 3.2.2.rc3


**OS/device including version:**
<!-- Specify GPU model, drivers, and the backend (GLES2, GLES3, Vulkan) if graphics-related. -->
- OS: macOS Catalina 10.15.5
- Hardware:
  - macbook Pro 13' 2015 + Intel Iris Graphics 6100 + 8 GB RAM + i5 Dual Core
  - macbook Pro 13' 2018 + Intel Iris Plus Graphics 655 + 16 GB RAM + i5 Quad Core


**Issue description:**
When using AnimatedTexture(s) the editor seems to put a lot of load in the CPU, whose fans start to work intensively (they can become very loud in both macs).
As you can see in the following GIF, the CPU usage is around 20% and GPU ~35% even if the IDE is idle:

![Jun-22-2020 19-00-46](https://user-images.githubusercontent.com/122741/85317955-81f08c00-b4bf-11ea-80b6-f5c60db50285.gif)


I don't know if this is a CPU or GPU issue. You can also see that the Idle Wake-ups are higher than normal (I don't know if that's some indicator for this issue). Only when the node having the animations is hidden, the hardware load starts to normalize.

This is maybe related to https://github.com/godotengine/godot/issues/30115 , but I am not sure, that's why I created this ticket.

**Steps to reproduce:**

- Create a new project (2D, GLES3)
- Create a new AnimatedTexture with some frames, assigning images to them.
- Use it in a new TileSet resource (not sure if this is related or not), as single tile.
- Draw some tiles in a new TileMap node, using the animated tile you created.
- Wait some minutes with the IDE playing the animations of your animated tiles // or continue working with the IDE and the scene having the animations.


**This issue only affects the IDE. The game itself doesn't seem to have this problem when running.**

**Minimal reproduction project:**
<!-- A small Godot project which reproduces the issue. Drag and drop a zip archive to upload it. -->
[godot-issue-39758.zip](https://github.com/godotengine/godot/files/4814789/godot-issue-39758.zip)

PS: Would be nice and convenient to be able to stop playing animated textures (or animations in general) in the IDE. A button or a checkbox in the menu or in the IDE settings.
Repository: flutter
Issue: Web engine: audit `List<Foo?>` usages after NNBD syntax migration
Labels: engine,platform-web,a: null-safety,P2,team-web,triaged-web

Most element types in lists can be made non-null. However, the first pass (https://github.com/flutter/engine/pull/19172) left some of them nullable. We should follow-up and make them non-null where possible.
Repository: flutter
Issue: Switch frontend_server snapshot to Dart SDK vended version
Labels: P2,c: tech-debt,team-engine,triaged-engine

The flutter/engine additions to the frontend_server are entirely gone at this point: https://github.com/flutter/engine/blob/master/flutter_frontend_server/lib/server.dart

Now that the Dart SDK is building it, we could save some upload/download time and simplify the engine code base too.

- [x] The Flutter CLI should stop using the flutter_frontend_server from the Engine build.
- [ ] Remove internal usage: b/233000852, b/305121780
- [ ] Reland https://github.com/flutter/engine/pull/46863 and https://github.com/flutter/engine/pull/46842
- [ ] The Engine recipes should stop packaging the flutter_frontend_server (https://flutter-review.googlesource.com/c/recipes/+/30240)
- [ ] The flutter_frontend_server should be removed from the Engine.
Repository: rust
Issue: Support for ARMv8.3+ targets
Labels: A-LLVM,T-compiler,C-feature-request,C-tracking-issue,O-AArch64,O-apple

Currently, a blocker in a project of mine - [Crabapple](https://github.com/Crabapple-iOS/Crabapple) - is the lack of native arm64e support in Rust.

The `aarch64-apple-ios` target only emits arm64 (ARMv8) code, which will not work within an arm64e (ARMv8.3+) environment due to it's lack of awareness of Pointer Authentication, leading to segfaults when trying to access a signed pointer.

It is currently possible to work around this, by using a backtrace=false rustc, compiling with `--emit=llvm-ir -Clto`, and running the resulting IR through an arm64e-aware LLVM (such as [apple/llvm-project](https://github.com/apple/llvm-project)). Pointer authentication may be worked around by [linking to an FFI function that runs ptrauth_strip](https://github.com/Crabapple-iOS/librustsupport/blob/master/library.m#L19-L22).

This may require LLVM changes if done, and quite possibly a new target (`arm64e-apple-ios`, `aarch64-apple-ios-armv83`, etc)
Repository: go
Issue: x/tools/internal/stack: hyphenated identifiers not supported 
Labels: NeedsInvestigation,Tools

### Does this issue reproduce with the latest release?
Yes (verified by looking at the source code)

### What did you do?
Run the stack tool on a dump file created when a Go process got a SIGABRT.

### What did you expect to see?
A single summary to be printed for all the goroutine stack traces.

### What did you see instead?
Multiple summaries were displayed. This makes it hard to determine, for example, the callstack that is most common in the dump file.

The reason is that the "package" and "function" parts of the regexp in "reCall" in https://github.com/golang/tools/blob/master/internal/stack/parse.go do not include a hyphen. However, the compiler can use hyphens for internally generated function names as discussed in https://stackoverflow.com/questions/32925344/why-is-there-a-fm-suffix-when-getting-a-functions-name-in-go.
Repository: pytorch
Issue: New ProcessGroups created with dist.new_group may leak memory
Labels: oncall: distributed,module: memory usage,triaged,better-engineering

## 🐛  Bug
The script pasted below shows an example of how if we repeatedly create a group with `dist.new_group()` and then run `del` and `gc.collect()` in a loop, the memory usage of the process slowly increases which can eventually cause an OOM. There may be leaked resources that aren't properly cleaned up in the destruction of these process groups. This also seems to happen for both the NCCL and GLOO backends. I would expect that if we create and then delete a bunch of process groups, memory usage stays roughly constant.

## To Reproduce
Run the following script:
```
import os
import psutil
import torch.multiprocessing as mp
import torch.distributed as dist
import gc

def get_mb():
    process = psutil.Process(os.getpid())
    return process.memory_info().rss * 1e-6

def test_mem(rank):
    dist.init_process_group("gloo", rank=rank, world_size=2)
    mem = get_mb()
    if rank == 0:
        print(f"Rank {rank} mem usage: {mem}")
    for i in range(10000):
        group = dist.new_group()
        mb = get_mb()
        if rank == 0:
            print(f"Rank {rank} pid: {os.getpid()}: Mem usage mb: {mb}")
        del group
        gc.collect()

if __name__ == '__main__':
    os.environ["MASTER_ADDR"] = "localhost"
    os.environ["MASTER_PORT"] = "29500"
    mp.spawn(test_mem, args=(), nprocs=2, join=True)
```

One can also check the memory with `ps -aux`. 

## Expected behavior

The memory usage stays roughly constant.

## Environment
PyTorch master

cc @pietern @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @xush6528 @osalpekar @jiayisuse @agolynski
Repository: TypeScript
Issue: duplicate Object.defineProperty when code emit with re-export a rename
Labels: Bug

<!-- 🚨 STOP 🚨 STOP 🚨 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 the latest published version. It may have already been fixed.
For npm: `typescript@next`
This is also the 'Nightly' version in the playground: http://www.typescriptlang.org/play/?ts=Nightly
-->
**TypeScript Version:**  Version 4.0.0-dev.20200621

<!-- Search terms you tried before logging this (so others can find this issue more easily) -->
**Search Terms:**

**Code**

```ts

import { dfSumFloat } from 'random-extra';
export { dfSumFloat as create } from 'random-extra';

export function randomSumFloat(size: number, sum?: number, min?: number, max?: number)
{
	return dfSumFloat(size, sum, min, max)()
}

export default randomSumFloat

```

**Expected behavior:**

no `Object.defineProperty(exports, "create"` x2

**Actual behavior:**

```ts
"use strict";
Object.defineProperty(exports, "__esModule", { value: true });
exports.randomSumFloat = exports.create = void 0;
const random_extra_1 = require("random-extra");
Object.defineProperty(exports, "create", { enumerable: true, get: function () { return random_extra_1.dfSumFloat; } });
var random_extra_2 = require("random-extra");
Object.defineProperty(exports, "create", { enumerable: true, get: function () { return random_extra_2.dfSumFloat; } });
function randomSumFloat(size, sum, min, max) {
    return random_extra_1.dfSumFloat(size, sum, min, max)();
}
exports.randomSumFloat = randomSumFloat;
exports.default = randomSumFloat;
//# sourceMappingURL=index.js.map
```

**Playground Link:** <!-- A link to a TypeScript Playground "Share" link which demonstrates this behavior -->
https://www.typescriptlang.org/v2/en/play?target=6&module=1&ts=4.0.0-dev.20200621#code/JYWwDg9gTgLgBAbzgEwGYGUCuIBiAbCAQ3gF85UoIQ4ByKQgO2SoFoBTADxnpoG4AoTpFiIUGbPiLxCAZzgBjKG2Js4ZClVr0mrTt0J9+gjsPipMDeTGAQGcbcxBZcBYgAoZwAF5sAXHAZsACM2KAAaOBlsAH5-QJAQ8LgQYAZYgODQiJBCDnT4xIBKfgR+AEglGEwoOzRnSXdPHwiokGzU7NzCt2KSIyFoeGQ2VEJMPHgHKnrXGCA

**Related Issues:** <!-- Did you find other bugs that looked similar? -->
Repository: flutter
Issue: Slight improvement to pubspec version comments
Labels: tool,d: api docs,c: proposal,P3,team-tool,triaged-tool

In the pubspec file, the following comment is automatically generated:

```
# The following defines the version and build number for your application.
# A version number is three numbers separated by dots, like 1.2.43
# followed by an optional build number separated by a +.
# Both the version and the builder number may be overridden in flutter
# build by specifying --build-name and --build-number, respectively.
# In Android, build-name is used as versionName while build-number used as versionCode.
# Read more about Android versioning at https://developer.android.com/studio/publish/versioning
# In iOS, build-name is used as CFBundleShortVersionString while build-number used as CFBundleVersion.
# Read more about iOS versioning at
# https://developer.apple.com/library/archive/documentation/General/Reference/InfoPlistKeyReference/Articles/CoreFoundationKeys.html
version: 1.0.0+1
```

Some terminology in the explanation is difficult to track.

Twice near the beginning of the comment, the terms "version" and "build number" are mentioned twice.

Later it is stated that "In Android, build-name is used as versionName while build-number used as versionCode". 

This then begs the question as to whether the Flutter pubspec "version" has any relation to how the Android version is determined.

To answer that question, one must take "build-name" and realize that it corresponds to the aforementioned "--build-name" flag, and then the reader needs to further recognize that the "--build-name" flag corresponds to the aforementioned term "version". Then you need to repeat this logical chaining to go from "versionCode" to "build-number" to "--build-number" to "build number".

Considering the criticality of correctly marking the version information for a packaged app, it might be a good idea to find a more concise and unambiguous explanation for the "version" property in the pubspec.

One idea would be to include a shorter explanation, but then show a literal Android example and a literal iOS example within the comment for the purpose of illustrating the "version" property applied in practice. That might suffice for a complete understanding.
Repository: PowerToys
Issue: Remove Show window hack used to fix settings blank dialog
Labels: Product-Settings,External Dependency,Status-Blocked,Issue-Task

The code introduced in #4428 to fix #3384 should be removed once the fix is made in WPF Xaml Islands.
Repository: flutter
Issue: MaterialBanner updates
Labels: framework,a: animation,f: material design,a: fidelity,f: scrolling,a: quality,P3,team-design,triaged-design

Material Spec on Baners:
https://material.io/components/banners

The MaterialBanner needs some updates to match spec:
- [x] conditional elevation
- [x] enter and exit animation support
- [ ] Scroll behavior - SliverMaterialBanner?
  - [ ] should be like a SliverPersistentHeader when placed in a scroll view, first position
  - [ ] animation in and out should animate the rest of the contents in the scroll view

It should also be integrated with the `ScaffoldMessenger` to:
- [x] manage how many are presented (there should only be one at a time, similar to SnackBar),
- [x] control the animation in and out (show, hide, remove) 
  - [x] animate out from under the static AppBar
Repository: flutter
Issue: App does not react to theme changes when it is in the background
Labels: platform-ios,framework,f: material design,a: fidelity,has reproducible steps,P2,found in release: 2.10,found in release: 2.13,team-design,triaged-design

When switching to dark mode in the control centre, my flutter app does not switch immediately to dark mode unless I exit the control center.

Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 1.20.0-1.0.pre.132, on Mac OS X 10.15.5 19F101, locale en-GB)
 
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 11.5)
[✓] Android Studio (version 4.0)
[✓] Connected device (2 available)
Repository: flutter
Issue: Add package for SF Symbols 2
Labels: c: new feature,platform-ios,a: fidelity,f: cupertino,package,e: OS-version specific,P3,team-ios,triaged-ios

iOS 14 is coming with rounder SF Symbols 2 https://developer.apple.com/sf-symbols/

Once it's out, explore how to make it usable as a package that replaces cupertino_icons. 

Could deprecate https://github.com/flutter/flutter/issues/33916 and https://github.com/flutter/flutter/issues/16102
Repository: flutter
Issue: [ios][video_player] pip - picture in picture
Labels: c: new feature,platform-ios,p: video_player,package,c: proposal,a: layout,P3,team-ios,triaged-ios

## Use case

I have a customer asking for PIP when the app is playing a video and the user wants multi-tasking.

For Android I had to edit the plugin, and use this line of code:
`getActivity().enterPictureInPictureMode();`
https://developer.android.com/guide/topics/ui/picture-in-picture#java

But for iOS, it is a different story, iOS does not support PIP in UIWindow or UIView, it is supported only in a Custom Player.
We need a custom view for that.
https://developer.apple.com/documentation/avkit/adopting_picture_in_picture_in_a_custom_player
https://developer.apple.com/documentation/avfoundation/avplayerlayer

The video_player plugin is using Texture to show the video output.

## Proposal

I need a way to customize the UIView created by the engine for the texture and add AVPlayerLayer support to it.
Any suggestions on how can I **contribute** this feature to the plugin?
Repository: flutter
Issue: Allow `showDateRangePicker` to scroll horizontal or vertical
Labels: c: new feature,framework,f: material design,f: date/time picker,c: proposal,P3,team-design,triaged-design

<!-- Thank you for using Flutter!

     If you are looking for support, please check out our documentation
     or consider asking a question on Stack Overflow:
      * https://flutter.dev/
      * https://api.flutter.dev/
      * https://stackoverflow.com/questions/tagged/flutter?sort=frequent

     If you have found a bug or if our documentation doesn't have an answer
     to what you're looking for, then fill our the template below. Please read
     our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports
-->

## Use case

<!--
     Please tell us the problem you are running into that led to you wanting
     a new feature.

     Is your feature request related to a problem? Please give a clear and
     concise description of what the problem is.

     Describe alternative solutions you've considered. Is there a package
     on pub.dev/flutter that already solves this?
-->
The current Date range picker calendar only scrolls horizontal.

## Proposal

<!--
     Briefly but precisely describe what you would like Flutter to be able to do.

     Consider attaching images showing what you are imagining.

     Does this have to be provided by Flutter directly, or can it be provided
     by a package on pub.dev/flutter? If so, maybe consider implementing and
     publishing such a package rather than filing a bug.
-->

```dart
CustomScrollView(
  controller: _controller,
  center: sliverAfterKey,
  /// Add property here.
  scrollDirection: scrollDirection /// Either Axis.horizontal or Axis.vertical
),
```

Add a property of scrollDirection to the CustomScrollView & allow us to put scrollDirection in the `showDateRangePicker` widget

Edit: Reference Source Code
https://github.com/flutter/flutter/blob/b5ddf8559597be7f103a90a8567145a70857c7f2/packages/flutter/lib/src/material/pickers/calendar_date_range_picker.dart#L202
Repository: pytorch
Issue: Missing explanation in torch.utils.tensorboard.add_histogram()
Labels: module: docs,triaged,module: tensorboard

This function has an input parameter add_histogram(), what does it mean? There is no documentation about this parameter. 

Documentation link 
https://pytorch.org/docs/stable/tensorboard.html#torch.utils.tensorboard.writer.SummaryWriter.add_histogram

cc @jlin27