sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: cmd/go: special case for chiselapp.com interferes with '.fossil' extension
Labels: NeedsInvestigation

### What version of Go are you using (`go version`)?
<pre>
$ go version
go version go1.12.1 linux/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
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/user/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/user/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/lib/go"
GOTMPDIR=""
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
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 -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build279953534=/tmp/go-build -gno-record-gcc-switches"
</pre></details>

### What did you do?
I tried to clone a fossil repo from chiselapp using the `.fossil` to test self-hosted fossil repos. It looks like the `chiselapp.com/user/kyle/repository/fossilgg` format is handled separately in [`cmd/go/internal/get/vcs.go`](https://github.com/golang/go/blob/master/src/cmd/go/internal/get/vcs.go#L1023).
```bash
$ go get chiselapp.com/user/kyle/repository/fossilgg.fossil
```

### What did you expect to see?
The `go get` succeeds and documentation can be listed via `go doc` for confirmation:
```bash
$ go doc -u -all -src fossilgg
package fossilgg.fossil // import "chiselapp.com/user/kyle/repository/fossilgg.fossil"


CONSTANTS

const greeting = "Hello from fossil!"

FUNCTIONS

func Hello() string {
        return greeting
}

```

### What did you see instead?
The `fossil clone` fails:
```
# cd .; fossil clone https://chiselapp.com/user/kyle/repository/fossilgg.fossil /home/user/go/src/chiselapp.com/user/kyle/repository/fossilgg.fossil/.fossil
redirect limit exceeded
server returned an error - clone aborted
package chiselapp.com/user/kyle/repository/fossilgg.fossil: exit status 1
```
This also leaves behind an empty `fossilgg.fossil` directory in `$GOPATH` that has the side effect of preventing future calls to `go get`:
```bash
$ go get chiselapp.com/user/kyle/repository/fossilgg.fossil
can't load package: package chiselapp.com/user/kyle/repository/fossilgg.fossil: no Go files in /home/user/go/src/chiselapp.com/user/kyle/repository/fossilgg.fossil
```
```
Repository: flutter
Issue: Scheduler.scheduleTask API has problems
Labels: framework,c: API break,c: proposal,P3,team-framework,triaged-framework

`SchedulerBinding.scheduleTask` enqueues a function to be executed in the next event loops and ensures that a frame is scheduled. There are only two usages I could find, in the widget_inspector and licenses in material.

Problems:
- Usage of Flow will definitely cause dart:developer to be retained in a production application (this is the case for licenses).
- PriorityQueue requires package:collection, or for us to fork and maintain one. Given that we don't even have examples of multiple tasks being scheduled, it seems unlikely that we're taking advantage of this structure.

### widget_inspector
```
 SchedulerBinding.instance.scheduleTask(
  selectionChangedCallback,
  Priority.touch,
 );
```

### licenses
```
  await SchedulerBinding.instance.scheduleTask<List<LicenseParagraph>>(
     () => license.paragraphs.toList(),
     Priority.animation,
     debugLabel: 'License',
    flow: flow,
    );
```

For reference the documentation of scheduleTask:
```
  /// Schedules the given `task` with the given `priority` and returns a
  /// [Future] that completes to the `task`'s eventual return value.
  ///
  /// The `debugLabel` and `flow` are used to report the task to the [Timeline],
  /// for use when profiling.
  ///
  /// ## Processing model
  ///
  /// Tasks will be executed between frames, in priority order,
  /// excluding tasks that are skipped by the current
  /// [schedulingStrategy]. Tasks should be short (as in, up to a
  /// millisecond), so as to not cause the regular frame callbacks to
  /// get delayed.
  ///
  /// If an animation is running, including, for instance, a [ProgressIndicator]
  /// indicating that there are pending tasks, then tasks with a priority below
  /// [Priority.animation] won't run (at least, not with the
  /// [defaultSchedulingStrategy]; this can be configured using
  /// [schedulingStrategy]).
```
### Mitigation
Update the license dialog to not use dart:developer. 

### Alternative: Soft deprecation?
- Change the type of Flow to Object and stop reporting it to the timeline.
- Replace PriorityQueue with FIFO or similar.
- Replace API with something like schedulePostFrameCallback, but that is guaranteed to schedule a frame?
Repository: TypeScript
Issue: refactoring for listing parameters from single line to column and back
Labels: Suggestion,Awaiting More Feedback

parameters
```ts
// before
function fn(a: A, b: B, c: C) { /*..*/ }

// after
function fn(
    a: A,
    b: B,
    c: C
) { /*..*/ }

// and back
function fn(a: A, b: B, c: C) { /*..*/ }
```

arguments
```ts
// before
return foldArray(toReversed(values), toListOf<T>(), (result, value) => add(result, value));

// after
return foldArray(
    toReversed(values),
    toListOf<T>(),
    (result, value) => add(result, value)
);
```
Repository: TypeScript
Issue: syntax to control over distributivness
Labels: Suggestion,In Discussion

from: https://github.com/Microsoft/TypeScript/issues/30569#issuecomment-476007534

so basically we need a way to say it clear and loud in the language whether we want:

```ts
Promise<A> | Promise<B>
```
or
```ts
Promise<A | B>
```
as a result of a type operation
Repository: react-native
Issue: JavaScript strings with NULL character are not handled properly
Labels: JavaScript,Bug,Never gets stale

## 🐛 Bug Report
JavaScript strings with NULL character are not handled properly

## To Reproduce
```jsx
<Text style={styles.welcome}>{'Hello \u0000 World'}</Text>
```
The text is cuted to Hello
It does not happen when Debug JS Remotely.

## Expected Behavior
Hello World

## Code Example
https://github.com/gaodeng/RN-NULL-character-ISSUE

## Environment
```
info 
  React Native Environment Info:
    System:
      OS: macOS 10.14.3
      CPU: (4) x64 Intel(R) Core(TM) i3-4130 CPU @ 3.40GHz
      Memory: 282.02 MB / 8.00 GB
      Shell: 3.2.57 - /bin/bash
    Binaries:
      Node: 10.15.1 - /usr/local/bin/node
      Yarn: 1.5.1 - /usr/local/bin/yarn
      npm: 6.8.0 - /usr/local/bin/npm
      Watchman: 4.7.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
      Android SDK:
        API Levels: 22, 23, 24, 25, 26, 27, 28
        Build Tools: 23.0.1, 25.0.0, 25.0.1, 25.0.2, 25.0.3, 26.0.0, 26.0.1, 26.0.2, 26.0.3, 27.0.2, 27.0.3, 28.0.0, 28.0.3
        System Images: android-18 | Google APIs Intel x86 Atom, android-22 | Google APIs Intel x86 Atom, android-23 | Google APIs Intel x86 Atom_64, android-24 | Intel x86 Atom_64, android-25 | Google APIs ARM EABI v7a, android-25 | Google APIs Intel x86 Atom_64, android-27 | Google APIs Intel x86 Atom, android-P | Google APIs Intel x86 Atom, android-P | Google Play Intel x86 Atom_64
    IDEs:
      Android Studio: 3.3 AI-182.5107.16.33.5314842
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: 16.8.3 => 16.8.3 
      react-native: 0.59.1 => 0.59.1 
    npmGlobalPackages:
      create-react-native-app: 1.0.0
      react-native-cli: 2.0.1
      react-native-create-library: 3.1.2
      react-native-git-upgrade: 0.2.7
      react-native-rename: 2.1.5
```
Repository: pytorch
Issue: Add layer-wise adaptive rate scaling (LARS) optimizer
Labels: feature,module: optimizer,triaged

## 🚀 Feature

Add a PyTorch implementation of layer-wise adaptive rate scaling (LARS) from the paper "[Large Batch Training of Convolutional Networks](https://arxiv.org/abs/1708.03888)" by You, Gitman, and Ginsburg. Namely, include the implementation found [here](https://github.com/noahgolmant/pytorch-lars) in the `torch.optim.optimizers` submodule.

## Motivation

LARS is one of the most popular optimization algorithms for large-batch training. This feature will expose the algorithm through the high-level optimizer interface. There are currently no other implementations of this algorithm available in widely used frameworks.

## Pitch

I have already written/tested a PyTorch optimizer for LARS in [this repo](https://github.com/noahgolmant/pytorch-lars). Changes would be minimal (I just have to make the imports relative at the top of the file).

## Alternatives

The only existing code for this is in caffe, and it looks like it is not exposed at a higher level within the solver framework. I also do not know whether or not that code has been tested.

## Additional context

Notably, I was unable to reproduce their results (namely reducing the generalization gap) on CIFAR-10 with ResNet. I would welcome additional tests to see if this implementation is able to replicate their performance on larger datasets like ImageNet.
Repository: godot
Issue: Complex model render incorrectly using GLES3 unless increasing blend shape max buffer size
Labels: enhancement,topic:rendering,documentation,topic:import,topic:3d

**Godot version:**
3.1 official

**OS/device including version:**
Windows 10, Vega 8 and Intel integrated, latest drivers

**Issue description:**
Imported model is not completly rendered when using GLES3, but it does when using GLES2, it also rendered correctly in versin 3.0.6

**Steps to reproduce:**
I'm linking a Git project because of model complexity project size is too big. I'm importing models using Fuse CC->Mixamo->Custom Script, 
[In this video](https://youtu.be/QYfEXgkOYnE) a few gaps are visible in the forehead and the neck areas.
[In this video](https://youtu.be/yzclSGAC4s4) the full left arm is missing.
The models have the same base body mesh, with 65 bones and about 80 blenshapes, second has more animations, meshes and materials (meshes were added trying to hide as much skin as possible, and skull polygons were removed too).

**Minimal reproduction project:**
[Git project](https://gitlab.com/DavidOC/GodotJam062018)
Repository: godot
Issue: Support offset in BoneAttachment
Labels: enhancement,topic:core

When adding a BoneAttachment to a Skeleton, the BoneAttachment's position is set to that of the bone it is attached to. This makes sense, but then if you explicitly alter the transform of the BoneAttachment in the editor, it always goes back to the bone's position at runtime. This is causing us issues, as the only bones we have in our character's hands are a) at the wrist, and b) at the fingers. This means that when we place a weapon as a child of the BoneAttachement, it is never in the appropriate position (it's eaier placed on the wrist, or on the fingers). Ideally, if the BoneAttachment is moved in editor, that offset would be taken into account when setting the BoneAttachment's position.
Repository: rust
Issue: Cannot implement trait on type alias involving another trait's associated type
Labels: A-type-system,A-associated-items,T-compiler,C-bug,T-types

When trying to implement `From` for nalgebra types, I discovered the following surprising behavior:

crate2/src/lib.rs
```rust
pub trait Foo {
    type Bar;
}

pub struct Baz;

impl Foo for Baz {
    type Bar = ();
}

pub struct Quux<T>(pub T);

pub type Corge = Quux<<Baz as Foo>::Bar>;
```

crate1/src/lib.rs
```rust
pub struct Grault;

// this doesn't compile
impl From<Grault> for crate2::Corge {
    fn from(x: Grault) -> Self {
        Self(())
    }
}

// but this seemingly equivalent impl works fine
impl From<Grault> for crate2::Quux<()> {
    fn from(x: Grault) -> Self {
        Self(())
    }
}
```

Error:
```
error[E0210]: type parameter `<crate2::Baz as crate2::Foo>::Bar` must be used as the type parameter for some local type (e.g. `MyStruct<<crate2::Baz as crate2::Foo>::Bar>`)
 --> src/lib.rs:3:1
  |
3 | impl From<Grault> for crate2::Corge {
  | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ type parameter `<crate2::Baz as crate2::Foo>::Bar` must be used as the type parameter for some local type
  |
  = note: only traits defined in the current crate can be implemented for a type parameter

error: aborting due to previous error

For more information about this error, try `rustc --explain E0210`.
```

The error occurs no matter which crate the type alias is in. In addition to failing at all here being almost certainly incorrect, the error message is strange; `Foo::Bar` is not a type parameter at all!
Repository: svelte
Issue: `bind:group` does not work with nested components
Labels: feature request

I'm trying to bind a store variable to group of checkboxes and it works till I move checkbox into a separate component, after that only one checkbox can be chosen at a time, here's an example from repl: https://gist.github.com/imbolc/e29205d6901d135c8c1bd8c3eec26d67
Repository: ant-design
Issue: 请为表格 (Table) 组件添加一个数据行 (tr) 分组的功能。
Labels: 💡 Feature Request,Inactive

- [ ] I have searched the [issues](https://github.com/ant-design/ant-design/issues) of this repository and believe that this is not a duplicate.

### What problem does this feature solve?
用表格展示数据的时候，希望能将具有关联的数据行分组展示，比如用边框线将多个数据行框起来作为一个整体来展示。

### What does the proposed API look like?
```js
const data = [
        [{}, {}], //分组1
        [{}, {}], //分组2
        [{}, {}], //分组3
        [{}, {}], //分组4
        //...
    ];
<Table dataSource={data} />
```

**如果可以的话，可以在原有的 `dataSource` 属性上扩展，要分组的数据行，由开发者自己分好。**


<!-- generated by ant-design-issue-helper. DO NOT REMOVE -->
Repository: electron
Issue: TitleBar/MenuBar Settings
Labels: enhancement :sparkles:

Good day
I like Electron because it has a good code implementation.
BrowserWindow has a lot of nice options for customization
[https://electronjs.org/docs/api/browser-window](url)

But I hope you will add some simple options for TitleBar customization
For some easy applications, change a color bar is more than enough. For more advanced applications I use index.html file with custom navbar settings and styles.

Proposed Solution
You have already added -titleBarStyle- String and it's really fine for me and my users on MacOS.
So I hope you will add some more options in near future.
My proposition is:
titleBarColor: String (optional) use HEXcolor (#80FFFFFF), // TitleBar color. It is really, what I wish for my users
titleBarHeight: Integer (optional) use values (50), // TitleBar height
titleBarTextColor: String (optional) use HEXcolor (#80FFFFFF), // Text color in TitleBar
titleBarElementsColor: String (optional) use HEXcolor (#80FFFFFF) // Some buttons elements color in TitleBar

With Great respect
A.W.
Repository: flutter
Issue: PaintingStyle should have a strokeAndFill enum value
Labels: framework,engine,d: api docs,P3,team-engine,triaged-engine

Hi, 

I know that this has been removed here https://github.com/flutter/engine/pull/3037
Is there any reason to remove this?
Or any other solution to avoid hitting the same pixels twice with a stroke draw and a fill draw?
Repository: scrcpy
Issue: can you make scrcpy work on ip stream .like play on live on any browser and media player
Labels: feature request
Repository: scrcpy
Issue: Hotkeys for record screen
Labels: feature request,record

To record a video I need to run the command:
```scrcpy -r file.mkv```
This method is inconvenient if the scrcpy is already running and I need to make a video. I will have to return to the console, close the program, enter the command. After recording the video, I will have to close the scrcpy and reopen it without parameters.

I would like to be able to manage the recording of video from the scrcpy, namely
1. Start recording
2. Pause recording
3. Stop recording

For this, I can use third-party software as OBS, but when I test mobile applications in the scrcpy, switching to between different windows is distracting.
Repository: vscode
Issue: [html] bracket matching in strings
Labels: bug,html

Issue Type: <b>Bug</b>

In HTML the minor sign inside of tags is sometimes recognised as opening a new tag.
For Example:
`<div data-ng-if="(something <= 0)" class="alert alert-warning">`

![image](https://user-images.githubusercontent.com/461708/54930193-bf5a1880-4f0e-11e9-8df6-932552bfb752.png)


VS Code version: Code 1.32.3 (a3db5be9b5c6ba46bb7555ec5d60178ecc2eaae4, 2019-03-14T23:43:35.476Z)
OS version: Windows_NT x64 10.0.16299

<details>
<summary>System Info</summary>

|Item|Value|
|---|---|
|CPUs|Intel(R) Core(TM) i5-3210M CPU @ 2.50GHz (4 x 2494)|
|GPU Status|2d_canvas: enabled<br>checker_imaging: disabled_off<br>flash_3d: enabled<br>flash_stage3d: enabled<br>flash_stage3d_baseline: enabled<br>gpu_compositing: enabled<br>multiple_raster_threads: enabled_on<br>native_gpu_memory_buffers: disabled_software<br>rasterization: unavailable_off<br>surface_synchronization: enabled_on<br>video_decode: enabled<br>webgl: enabled<br>webgl2: enabled|
|Memory (System)|7.94GB (3.81GB free)|
|Process Argv||
|Screen Reader|no|
|VM|0%|

</details><details><summary>Extensions (3)</summary>

Extension|Author (truncated)|Version
---|---|---
python|ms-|2019.2.5558
cpptools|ms-|0.22.1
vscode-spotify|shy|3.1.0

(2 theme extensions excluded)

</details>
<!-- generated by issue reporter -->
Repository: go
Issue: x/tools/cmd/goimports: prefer main module requirements
Labels: NeedsInvestigation,Tools

<!-- Please answer these questions before submitting your issue. Thanks! -->

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version go1.12.1 linux/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
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/myitcv/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/myitcv/gostuff"
GOPROXY=""
GORACE=""
GOROOT="/home/myitcv/gos"
GOTMPDIR=""
GOTOOLDIR="/home/myitcv/gos/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
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 -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build898038110=/tmp/go-build -gno-record-gcc-switches"
</pre></details>

### What did you do?

Ran [`testscript`](https://github.com/rogpeppe/go-internal/blob/master/cmd/testscript/README.md) on the following:

```
# install goimports
env GOMODPROXY=$GOPROXY
env GOPROXY=
go install golang.org/x/tools/cmd/goimports

# switch back to our local proxy
env GOPROXY=$GOMODPROXY

# "warm" the module (download) cache
go get gopkg.in/tomb.v1
go get gopkg.in/tomb.v2

# test goimports
cd mod
go mod tidy
exec goimports main.go
stdout '\Q"gopkg.in/tomb.v2"\E'

-- go.mod --
module goimports

require golang.org/x/tools v0.0.0-20190322203728-c1a832b0ad89

-- mod/go.mod --
module mod

require gopkg.in/tomb.v2 v2.0.0

-- mod/main.go --
package main

import (
	"fmt"
)

func main() {
	fmt.Println(tomb.Tomb)
}

-- .gomodproxy/gopkg.in_tomb.v1_v1.0.0/.mod --
module gopkg.in/tomb.v1

-- .gomodproxy/gopkg.in_tomb.v1_v1.0.0/.info --
{"Version":"v1.0.0","Time":"2018-10-22T18:45:39Z"}

-- .gomodproxy/gopkg.in_tomb.v1_v1.0.0/go.mod --
module gopkg.in/tomb.v1

-- .gomodproxy/gopkg.in_tomb.v1_v1.0.0/tomb.go --
package tomb

const Tomb = "A great package v1"

-- .gomodproxy/gopkg.in_tomb.v2_v2.0.0/.mod --
module gopkg.in/tomb.v2

-- .gomodproxy/gopkg.in_tomb.v2_v2.0.0/.info --
{"Version":"v2.0.0","Time":"2018-10-22T18:45:39Z"}

-- .gomodproxy/gopkg.in_tomb.v2_v2.0.0/go.mod --
module gopkg.in/tomb.v2

-- .gomodproxy/gopkg.in_tomb.v2_v2.0.0/tomb.go --
package tomb

const Tomb = "A great package v2"
```


### What did you expect to see?

A passing run.

### What did you see instead?

A failed run.

`goimports` should be "consulting" the main module for matches before dropping down to a module cache-based search. Here that would have resulted in `gopkg.in/tomb.v2` being correctly resolved, instead of `gopkg.in/tomb.v1`.

This will, I suspect, also massively improve the speed of `goimports` in a large majority of cases.

cc @heschik
Repository: opencv
Issue: New JasPer release fixing CVEs
Labels: category: imgcodecs

JasPer 2.0.16 got recently released.

I would like to see a transition to OpenJPEG as suggested in https://github.com/opencv/opencv/issues/5849 but in the meantime I think updating the sources for libjasper might be a good idea since it fixes two CVEs.

I would have created a PR instead but looking at the to me unfamilar directory structure at `opencv/3rdparty/libjasper` I'll rather let you handle it properly.
Repository: react
Issue: Memoized components should forward displayName
Labels: Type: Enhancement,Component: Shallow Renderer

**Do you want to request a *feature* or report a *bug*?**

I'd like to report a bug.

**What is the current behavior?**

First of all, thanks for the great work on fixing https://github.com/facebook/react/issues/14807. However there is still an issue with the current implementation.

`React.memo` does not forward displayName for tests. In snapshots, components display as `<Component />` and string assertions such as `.find('MyMemoizedComponent')` won't work.

**What is the expected behavior?**

`React.memo` should forward displayName for the test renderer.

**Which versions of React, and which browser / OS are affected by this issue? Did this work in previous versions of React?**

* React 16.8.5
* Jest 24.5.0
* enzyme 3.9.0
* enzyme-adapter-react-16 1.11.2

---

N.B. - Potentially related to https://github.com/facebook/react/issues/14319, but this is related to the more recent changes to support `memo` in the test renderer. Please close if needed, I'm quite new here!

I'd be happy to submit a PR if the issue is not too complex to look into :smile:
Repository: flutter
Issue: google_maps_flutter package - CameraPosition does not follow location updates
Labels: c: new feature,p: maps,package,c: proposal,team-ecosystem,P3,triaged-ecosystem

The GoogleMap widget already is getting the location and listening to location updates if it is instantiated with myLocationEnabled. I would like it to be able to do two additional things with this information: 
1. Instantiate the map with the CameraPosition set to the user's current location.
2. Update the CameraPosition when the user's location changes. As it is the blue dot showing the location moves, but the camera does not, letting the location marker go off screen. 

I have not found any way to do this without using another plugin that gets the user's initial position and then listens to updates in location. This is not ideal since the map widget is already listening for the location, it seems redundant to have two plugins listening for locations.
Repository: pytorch
Issue: improve jit error message for legacy constructor
Labels: module: docs,triaged

Reported by @jph00

> new_tensor is a legacy constructor and is not supported in the JIT

It'd be helpful to know in the error message what the non-legacy alternative is.