sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: nvm
Issue: installation of nvm with XDG variables sets doesn't seem to use a subdirectory
Labels: bugs,pull request wanted,installing nvm

Hey,

I tried to install nvm according to the README and my environment contains a few XDG variables, like

```
declare -x XDG_CACHE_HOME="$HOME/.cache"
declare -x XDG_CONFIG_HOME="$HOME/.config"
declare -x XDG_DATA_HOME="$HOME/.local/share"
```

The installation (using bash) took place inside of the `.config` directory - was used as root Git directory. Could it be possible that `/nvm` should not be part of the variable scoping - [bash parameter substitution](http://tldp.org/LDP/abs/html/parameter-substitution.html)?

https://github.com/creationix/nvm/blob/2410215b6a96c5a4376af8a41b6e2942b4b6cc2d/install.sh#L11

```
$ echo "${XDG_CONFIG_HOME}"
$HOME/.config

$ echo "${XDG_CONFIG_HOME/nvm}"                                                                                                        
$HOME/.config
                                                                                                                                                          
$ bash --version                                                                                                                       
GNU bash, version 4.4.12(1)-release (x86_64-pc-linux-gnu)                                                                                                                         
Copyright (C) 2016 Free Software Foundation, Inc.                                                                                                                                 
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>                                                                                                                                                                                                                                                                                       This is free software; you are free to change and redistribute it.                                                                                                                
There is NO WARRANTY, to the extent permitted by law.
```

Cheers
Repository: rust
Issue: Allow lints to declare that they're pure
Labels: C-enhancement,A-lints,T-compiler

Clippy and rustc both have a bunch of allow by default lints. Due to the way lint passes work, these have to be run even if never enabled, since lint passes may store state between `check_foo` calls.

Most lints don't do this, most lints are zero-sized.

It would be nice to allow LintPasses to declare that they are "pure", and have rustc omit allowed pure passes from the list of lint passes until enabled. This has some performance benefits but would also allow for incremental linting if we ever want that.

cc @Zoxc @oli-obk
Repository: react
Issue: Can an error boundary prevent React's error logging?
Labels: Type: Feature Request

I noticed [this unconditional `console.error`](https://github.com/facebook/react/blob/d0289c7e3a2dfc349dcce7f9eb3dee22464e97bd/packages/react-reconciler/src/ReactFiberErrorLogger.js#L86) which I'd like to prevent to keep the console clean from errors that are already "caught" in an error boundary.

Maybe a condition on `capturedError.errorBoundaryFound` could prevent this logging?
Repository: flutter
Issue: Explore making State loss more obvious in widgets tests
Labels: a: tests,framework,a: quality,c: proposal,P3,team-framework,triaged-framework

Make it easier for users to notice States are being disposed. 

One way is to make all State disposes need to be expected in widgets tests. Test: https://github.com/flutter/flutter/pull/29076

By blinding adding it in, 601 out of 3,804 (16%) of existing tests fail. 

By filtering some common states such as _AutomaticKeepAliveState, the failure rate is down to 497 (13%). Though maintaining a whitelist is unlikely the right solution. 

Perhaps modify pumpWidget such that only explicitly constructed widgets sent into pumpWidget have their associated States tracked.
Repository: flutter
Issue: Add a State create/dispose debug rainbow 
Labels: c: new feature,framework,a: debugging,customer: google,P3,team-framework,triaged-framework

Similar to #15550. Though rather than asserting for performance, this should help checking for functional correctness.
Repository: godot
Issue: TextureRegion doesn't update to new copy of AtlasTexture
Labels: bug,topic:editor,confirmed

<!-- 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. -->
master 86d6a72c9745bd15e3217d63b2cc5ba73fe4ad34
both GLES3 and GLES2

**OS/device including version:**
<!-- Specify GPU model and drivers if graphics-related. -->
Windows 10 64-bit
GTX 1060 6GB

**Issue description:**
<!-- What happened, and what was expected. -->
TextureRegion gets "disconnected from duplicate TextureRect's Atlas Texture when you make it unique"
It works fine with a Sprite.

**Steps to reproduce:**
- Create TextureRect and add Atlas Texture
- Select any image as atlas
- (Optional) Select any region
- Duplicate TextureRect
- Select TextureRect2 and click on AtlasTexture resource to open TextureRegion
- Make AtlasTexture unique
- Change region inside TextureRegion
- It doesn't update current region
To fix it:
- Click AtlasTexture resource to close TextureRegion and open it again (Doing it by clicking TextureRegion tab won't fix it)
Repository: rust
Issue: Resolve intra-doc-links on Self type implicitly
Labels: T-rustdoc,C-enhancement,A-intra-doc-links

With intra-doc-links, the following works:

```rust
/// [Self::bar]
pub struct Foo;

impl Foo {
    /// [Self::bar]
    fn bar() {}
}
```


but this does not:

```
/// [bar]
pub struct Foo;

impl Foo {
    /// [bar]
    fn bar() {}
}
```

For structs and impls we should try resolving things against `Self` anyway
Repository: pytorch
Issue: PyPy support
Labels: module: binaries,feature,triaged

## 🚀 Feature
Support pytorch from PyPy -- a fast, compliant alternative implementation of the Python language (http://pypy.org)

## Motivation

While pytorch itself probably won't benefit much from PyPy JIT, but often pytorch is used as a part of larger application, where using PyPy can have speed benefits, e.g. for evaluation, data generation, or other activities.

## Pitch

(A clear and concise description of what you want to happen). I can imagine several stages:

- PyPy support works by compiling from source
- PyPy is integrated into CI
- PyPy wheels are built and released

## Additional context

Right now it seems that PyPy support, at least for commonly used stuff, is not far from reality, for example this branch https://github.com/lopuhin/pytorch/tree/pypy3.6 works in PyPy 3.6 7.0.0 for a moderately complex application (didn't try running tests yet).

Here is a summary of fixes in that branch:

1. working around a compilation issue in the JIT: https://github.com/pytorch/pytorch/commit/0481da14052c09605b38940e08d8cae275c6a8cd - this is definitely a hack to get the compilation working, a proper implementation for PyPy needs to be in place.
2. **[merged]** `PySlice_Unpack` is not yet available in PyPy 3.6 -https://github.com/pytorch/pytorch/commit/752e204c90a782432eed77018e4fe3ad5fa4dbf2 - probably it's ok to merge this. With two above fixes, pytorch compiles on PyPy 3.6. PR https://github.com/pytorch/pytorch/pull/17836
3. **[merged in https://github.com/pybind/pybind11/pull/2146 ]** Applying https://github.com/pybind/pybind11/pull/1494 to pybind11 - https://github.com/pytorch/pytorch/commit/912b0b2a77b3ea0a5bac9fbe3ac5d577c86c6d7. Unfortunately, while the fix seems correct, and pybind11 has partial support and CI for PyPy, more work needs to be done to properly integrate it into pybind11, because other pybind11 tests are failing on a more recent PyPy, see https://github.com/pybind/pybind11/pull/1494 and https://github.com/pybind/pybind11/pull/1720
4. **[merged]** work around PyPy cpyext issue - https://github.com/pytorch/pytorch/commit/00941d8ae4e28bd03245a04c5096297c35a3128d - see https://bitbucket.org/pypy/pypy/issues/2968/segfault-calling-cpyext_tp_new_tuple (thanks @rlamy for the fix!), it seems that it is fine to merge this. PR https://github.com/pytorch/pytorch/pull/17837

Thanks to @xuhdev for an already merged PyPy compilation fix https://github.com/pytorch/pytorch/pull/11857

I submitted PRs for 2 and 4. If anyone can pick other items, that would be great.

While these are quite small fixes, still properly integrating them is not a small task, not speaking of setting up CI and supporting it.




cc @ezyang
Repository: opencv
Issue: cv2.imread error when read a jp2 image with 7 channel
Labels: priority: low,category: imgcodecs

- OpenCV => 4.0.1
- Operating System / Platform => Windows 10 64 Bit Python 3.6
- .pyd Compiler => Visual Studio 2015

Get a NoneType when read jp2 image with 7 channel. 
image url:http://www.usable-programming.com/zb_users/upload/2019/03/bavaria_north_7_channels.zip
Repository: vscode
Issue: default run task
Labels: feature-request,tasks

It would be nice if there was a default run task (build + execute), similar to the existing default build and test tasks.
Repository: go
Issue: x/image/font/sfnt: read more glyph metrics
Labels: NeedsDecision

Aligning text (left/right) requires access to more glyph metrics then AdvanceX. The side bearings (LSB/RSB) are required to know how much "whitespace" there is left/right of a glyph.

I'd like to propose a GlyphMetrics method which calculates and returns the LSB, RSB, Width and AdvanceX.

CL with working prototype is on its way.
Repository: godot
Issue: The Audio bottom panel goes under the taskbar on small displays (also SpriteFrames editor, shader editor, etc.)
Labels: bug,topic:editor,confirmed

**Godot version:**
v3.1.rc1

**OS/device including version:**
windows 10 x64

**Issue description:**
When click on "Audio" tab, the bottom side of godot goes under the taskbar, until toggle full screen and change to other tab (output, debugger, ..)

![image](https://user-images.githubusercontent.com/41632592/54072481-3fc41c80-4273-11e9-8c48-4ff82175714e.png)

![image](https://user-images.githubusercontent.com/41632592/54072489-52d6ec80-4273-11e9-9bff-1b919c40895c.png)
Repository: TypeScript
Issue: [BUG] Comments between export and import modules are removed
Labels: Bug,Domain: Comment Emit

<!-- 🚨 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.3.3333333333333333

<!-- Search terms you tried before logging this (so others can find this issue more easily) -->
**Search Terms:**
* Comments
* treeshaking
* __pure__

**Code**
https://www.typescriptlang.org/play/index.html#src=import%20%7B%20m%20%7D%20from%20'm'%3B%0D%0Aconst%20t1%20%3D%20%2F*%40__PURE__*%2F%20m.bind(m)%3B%20%2F%2Fkeep%0D%0A%2F%2Fhere%20is%20a%20bug%20-%3E%20%20%F0%9F%91%87%20removed%20%F0%9F%91%87%0D%0Aexport%20const%20test%20%3D%20%2F*%40__PURE__*%2F%20m.bind(m)%3B

```ts
import { m } from 'm';
const t1 = /*@__PURE__*/ m.bind(m); //keep
//here is a bug ->  👇 removed 👇
export const test = /*@__PURE__*/ m.bind(m);
```

**Related Issues:** <!-- Did you find other bugs that looked similar? -->
#28370
#13721
#28482
Repository: vue
Issue: Pass component instance as second argument in computed setters
Labels: feature request,has PR

### What problem does this feature solve?
Allow the usage of arrow functions  in copmuted setters
From #7688 

```js
computed: {
  value: {
    get: vm => vm.someValue,
    set: (val, vm) => vm.someValue = val
  }
}
```

### What does the proposed API look like?
```js
computed: {
  value: {
    get: vm => vm.someValue,
    set (val, vm) {
      this === vm // true
    }
  }
}
```

<!-- generated by vue-issues. DO NOT REMOVE -->
Repository: pytorch
Issue: Tensor::options() returns false for requires_grad when it is true
Labels: module: cpp-extensions,triaged

## 🐛 Bug ?

Hello,

I have very simple c++ function:
```cpp
#include <torch/extension.h>
#include <iostream>

void print_options(torch::Tensor x) {
    std::cout << x.options() << "\n";
}

PYBIND11_MODULE(TORCH_EXTENSION_NAME, m) {
  m.def("print_options", &print_options, "");
}

```
which I use via JIT compiling extensions functionality:
```python
import torch
from torch.utils import cpp_extension

cuda_module = cpp_extension.load(name="cpp_module", sources=["cpp_modue/my_fun.cpp"])
print_options = cuda_module.print_options

a = torch.zeros(2, 3)
a.requires_grad = True
print(a.requires_grad)

print_options(a)
```
And here is the output:

```
True
TensorOptions(dtype=float, device=cpu, layout=Strided, requires_grad=false)
```

Is this expected behaviour? Why does passing tensor to the c++ function change its options (requires_grad in particular)?

 - PyTorch Version is 1.0.1



cc @yf225 @glaringlee @zou3519
Repository: vue
Issue: Vue should not cause execution of content within <noscript> tags
Labels: has workaround

### Version
2.6.8

### Reproduction link
[https://jsfiddle.net/ncwuvekm/1/](https://jsfiddle.net/ncwuvekm/1/)

### Steps to reproduce
Check the "network" tab.

You can see that Vue causes a request to the content within the `<noscript>`

### What is expected?
Vue should not cause execution of `<noscript>`content

### What is actually happening?
Vue causes requests to elements within `<noscript>`

---
It's typical when websites are using Vue to supplement UI and a common pattern is to wrap the site with an `id="app"`

However, this introduces a problem when people are using `<noscript>` in various areas of the site that are not within Vue components as Vue will execute it regardless.

A use case example would be using a lazy loading library with a `<noscript>` fallback.

```html
<img src="thumbnail.jpg" data-src="hi-resolution.jpg" />
<noscript><img src="hi-resolution.jpg" /></noscript>
```

Vue will cause the `hi-resolution.jpg` image to download even though it's within the `<noscript>`

<!-- generated by vue-issues. DO NOT REMOVE -->

> For anybody experiencing this issue, a hack is to add `v-if="false"` to your `<noscript>` to prevent the element from rendering e.g.`<noscript v-if="false">`
Repository: flutter
Issue: Points of Path not Accessible
Labels: c: new feature,engine,P3,team-engine,triaged-engine

I am using the Path class from the painting.dart file to draw lines with a CustomPainter.

There is no way for me to access the contained points.

This is critical as the amount of points stored in the Path object can get huge (around 100k) which will lead to bad performance or I simply cannot work with these points.
As I cant access the points, I cannot do anything about it.
On the other hand storing these points to another list will unecessarily increase the used memory.

Please make a simple attribute or getter to access the Points stored to a Path object.

Thanks!
Repository: go
Issue: cmd/go: go mod why fails for replacement modules
Labels: NeedsInvestigation,modules

<pre>
$ go version
go version devel +ce7534ff06 Fri Mar 8 13:46:43 2019 +0000 linux/amd64
</pre>

When one module is replaced by another, `go mod why` does not acknowledge the replacement module.

Here's an example (https://godoc.org/github.com/rogpeppe/go-internal/cmd/testscript on the below file to reproduce):

It prints "main module does not need module `example.com/c`, but `example.com/c` is definitely in use as the replacement for `example.com/a` which isn't actually used at all.

```
# This command succeeds; Note: example.com/d is only
# depended on by example.com/c, but not by example.com/a, which is
# never used.
go mod why -m example.com/d

# This command succeeds, even though the program does not
# depend on example.com/a except as a replacement target.
go mod why -m example.com/a
! stdout 'main module does not need module'

# example.com is definitely used (and it's probably important
# the the user know that it is), but this command fails.
go mod why -m example.com/c
! stdout 'main module does not need module'

-- go.mod --
module m

go 1.11

require (
	example.com/a v1.0.0
	example.com/d v1.0.0 // indirect
)

replace example.com/a => example.com/c v1.0.0
-- go.sum --
example.com/c v1.0.0 h1:+JKa2qCailgQdye6M3nnfrb7q748qDLj2NNTEtN2DJs=
example.com/c v1.0.0/go.mod h1:NeOsx/KTizj35klXP3wYh3O0751aAtYrRoX+a6YAye8=
example.com/d v1.0.0 h1:+FTMPN+4iCiTwx9DTGWb788knewVgy4CGwf1SjyLpXY=
example.com/d v1.0.0/go.mod h1:jpRNKJ+rI4SFCFqRJlfe7G4saIJvHgJss1TcTCWmY18=
-- main.go --
package main

import _ "example.com/a"

func main() {
}
-- .gomodproxy/example.com_a_v1.0.0/.info --
{"Version":"v1.0.0","Time":"2018-10-22T18:45:39Z"}
-- .gomodproxy/example.com_a_v1.0.0/.mod --
module example.com/a
-- .gomodproxy/example.com_a_v1.0.0/a.go --
package a

import _ "example.com/b"
-- .gomodproxy/example.com_a_v1.0.0/go.mod --
module example.com/a

import _ "example.com/b"
-- .gomodproxy/example.com_b_v1.0.0/.info --
{"Version":"v1.0.0","Time":"2018-10-22T18:45:39Z"}
-- .gomodproxy/example.com_b_v1.0.0/.mod --
module example.com/b
-- .gomodproxy/example.com_b_v1.0.0/b.go --
package b
-- .gomodproxy/example.com_b_v1.0.0/go.mod --
module example.com/b
-- .gomodproxy/example.com_c_v1.0.0/.info --
{"Version":"v1.0.0","Time":"2018-10-22T18:45:39Z"}
-- .gomodproxy/example.com_c_v1.0.0/.mod --
module example.com/a
-- .gomodproxy/example.com_c_v1.0.0/a.go --
package a

import _ "example.com/d"
-- .gomodproxy/example.com_c_v1.0.0/go.mod --
module example.com/a

require example.com/d v1.0.0
-- .gomodproxy/example.com_d_v1.0.0/.info --
{"Version":"v1.0.0","Time":"2018-10-22T18:45:39Z"}
-- .gomodproxy/example.com_d_v1.0.0/.mod --
module example.com/d
-- .gomodproxy/example.com_d_v1.0.0/b.go --
package d
import _ "testing"
-- .gomodproxy/example.com_d_v1.0.0/go.mod --
module example.com/d
```
Repository: go
Issue: cmd/go: go mod why exit status should be non-zero if module is not used
Labels: NeedsFix,modules

<pre>
$ go version
go version devel +ce7534ff06 Fri Mar 8 13:46:43 2019 +0000 linux/amd64
</pre>

When `go mod why` is asked to show a module that isn't a dependency, it prints a "main module does not need" line, but it succeeds regardless.

Perhaps it should be changed so that the exit status reflects whether the dependency was actually found or not.
Repository: flutter
Issue: Gifs play faster than expected
Labels: engine,a: quality,a: images,has reproducible steps,P2,found in release: 3.3,found in release: 3.7,team-engine,triaged-engine

## Steps to Reproduce

```dart
Image.network('https://github.githubassets.com/images/spinners/octocat-spinner-128.gif')
```

Flutter version:

```sh
Flutter 1.3.9 • channel dev • https://github.com/flutter/flutter.git
Framework • revision f91df4abe1 (16 hours ago) • 2019-03-09 21:19:28 -0500
Engine • revision 4e54bc93ca
Tools • Dart 2.2.1 (build 2.2.1-dev.1.0 2fb6cd9f5f)
```

Possibly related to #24804
Repository: godot
Issue: macOS: Mission Control window focus issue
Labels: bug,platform:macos,topic:porting,confirmed

___

***Bugsquad note:** This issue has been confirmed several times already. No need to confirm it further.*

___

**Godot version:**
<!-- Specify commit hash if non-official. -->
Godot_v3.1-rc1_osx.64

**OS/device including version:**
<!-- Specify GPU model and drivers if graphics-related. -->
macOS Mojave 10.14.3

**Issue description:**
<!-- What happened, and what was expected. -->
NOTE: I'm not sure if this is a macOS issue or a Godot issue, but it only seems to happen with this application.

When using Mission Control to switch back to Godot, it sometimes switches to the application, but leaves focus on the previous application window. It does switch to the correct application as the Godot menu displays in the menu bar, but with the wrong application window focused. This requires me to either switch to Godot again or to hide the incorrectly focused window.

**Steps to reproduce:**

1. Open another application
2. Open Mission Control
3. Select the Godot application
Repository: angular
Issue: Why are angular elements swallowing error?
Labels: type: bug/fix,workaround1: obvious,freq1: low,area: core,state: confirmed,P4

I have the simplest possible angular element within an angular project. 

I throw error in component belonging to angular element as follows:

[dashboard-tile.component.ts:](https://github.com/goodmite/angular-element-starter/blob/master/src/app/dashboard/dashboard-tile/dashboard-tile.component.ts) (_referenced in [index.html](https://github.com/goodmite/angular-element-starter/blob/master/src/index.html) as  `<dashboard-tile a="100" b="50" c="25"></dashboard-tile>`_)
```
  ngOnInit() {
    debugger;
    throw "this is an error";
  }
```
But I see no error in chrome console.

Link to [video](https://youtu.be/EsDbn7k9JpQ).

**However**, if I start to use this component as a regular component, I immediately get an error on console. So this is likely an angular-element issue.

Link to github [repo ](https://github.com/goodmite/angular-element-starter)containg the code

Tested on both chrome and firefox and its reproducible so its not browser isssue.

**Other info:**

Angular CLI: 7.1.4
Node: 10.14.2
OS: win32 x64
Angular: 7.1.4
... animations, cli, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.11.4
@angular-devkit/build-angular     0.11.4
@angular-devkit/build-optimizer   0.11.4
@angular-devkit/build-webpack     0.11.4
@angular-devkit/core              7.1.4
@angular-devkit/schematics        7.1.4
@angular/elements                 7.2.8
@ngtools/webpack                  7.1.4
@schematics/angular               7.1.4
@schematics/update                0.11.4
rxjs                              6.3.3
typescript                        3.1.6
webpack                           4.23.1
Repository: go
Issue: x/tools/astutil: strange comment placement with AddNamedImport
Labels: NeedsInvestigation,Tools

Using `00c44ba9c14f88ffdd4fb5bfae57fe8dd6d6afb1`

Consider the following snippet:
```go
package main

import (
	"go/parser"
	"go/printer"
	"go/token"
	"os"

	"golang.org/x/tools/go/ast/astutil"
)

const source = `package comments//

// X comment
var X int
`

func main() {
	fset := token.NewFileSet()
	file, _ := parser.ParseFile(fset, "", source, parser.ParseComments)

	astutil.AddNamedImport(fset, file, "tar", "archive/tar")
	astutil.AddNamedImport(fset, file, "zip", "archive/zip")

	(&printer.Config{Mode: printer.TabIndent | printer.UseSpaces, Tabwidth: 8}).Fprint(os.Stdout, fset, file)
}
```

This currently outputs:
```go
package comments //

import (
	tar "archive/tar"
	zip "archive/zip"
) // X comment
var X int
```
Notice how the next comment group `X comment` is attached to the import block? It seems that this only occurs if an inline comment occurs on the package statement.

I expect this to output:
```go
package comments //

import (
	tar "archive/tar"
	zip "archive/zip"
)

// X comment
var X int
```