sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: rust
Issue: Unknown feature names are not reported if there are other errors
Labels: C-enhancement,A-diagnostics,T-compiler

I just added an impl that I thought should be valid now with https://github.com/rust-lang/rfcs/pull/2451, but I got a compiler error. So I found the RFC and copied the feature name `re_rebalancing_coherence` from it. However, I still got the compile error, so I spent a while trying to figure out if the impl was really valid. Eventually, I tried commenting out the impl, and then (thanks to #52644), the compiler told me the feature was unknown (it has been renamed to `re_rebalance_coherence`). The impl was valid using the correct feature name. I didn't get the error saying the feature name was invalid because of the error caused by the feature name being invalid.

I think the compiler should report unknown feature names even if there are other errors. For example: https://play.rust-lang.org/?version=nightly&mode=debug&edition=2018&gist=e46fde7bf4076220df3005d702df9cbb doesn't report the feature name is invalid. With box syntax, it suggests "help: add #![feature(box_syntax)] to the crate attributes to enable" but the coherence error didn't.

Feel free to close this issue if you don't think it is worth fixing.
Repository: TypeScript
Issue: Recursive definitions in mixins
Labels: Suggestion,Awaiting More Feedback

Its currently possible to create a circular references in classes just fine:
```ts
export class Class1 {
    another     : Class2     // compiles fine
}
export class Class2 {
    another     : Class1    // compiles fine
}
```
However, the same thing fails when using mixin-based classes:

```ts
export const SampleMixin1 = <T extends AnyConstructor<object>>(base : T) =>
class SampleMixin1 extends base {
    another             : SampleMixin2 // TS2502: 'another' is referenced directly or indirectly in its own type annotation
}
export type SampleMixin1 = Mixin<typeof SampleMixin1>


export const SampleMixin2 = <T extends AnyConstructor<object>>(base : T) =>
class SampleMixin2 extends base {
    another             : SampleMixin1 // TS2502: 'another' is referenced directly or indirectly in its own type annotation
}
export type SampleMixin2 = Mixin<typeof SampleMixin2>


// supporting declarations for mixin pattern
export type AnyFunction<A = any>      = (...input: any[]) => A
export type AnyConstructor<A = any>   = new (...input: any[]) => A
export type Mixin<T extends AnyFunction> = InstanceType<ReturnType<T>>
```

This makes things much more complicated, you need to introduce some dummy interfaces, etc, etc.

The workaround exists - to use different form of creating the type for the standalone mixin class - with interfaces:
```ts
export const SampleMixin3 = <T extends AnyConstructor<object>>(base : T) =>
class SampleMixin3 extends base {
    another             : SampleMixin4
}
export interface SampleMixin3 extends Mixin<typeof SampleMixin3> {}


export const SampleMixin4 = <T extends AnyConstructor<object>>(base : T) =>
class SampleMixin4 extends base {
    another             : SampleMixin3
}
export interface SampleMixin4 extends Mixin<typeof SampleMixin4> {}
```
But this notation seems to drive crazy the IDE (the language server under the hood?). It stop finding usages of properties, show many false type errors etc. Basically in such approach you are limited to old-good console `npx tsc` launch (which works well at least), but all the development aid facilities don't work.

I think mixin pattern should be supported first class. I'd even create some language construct for it.
Repository: go
Issue: net/rpc/jsonrpc: client implementation is not compatible with JSON-RPC 1.0 spec
Labels: help wanted,NeedsInvestigation

According to [JSON-RPC 1.0 specification](https://www.jsonrpc.org/specification_v1) the params property should be an array of object. The canonical example of calling remote methods with JSON-RPC 1.0 is to send the parameters as one dimensional JSON array:
`{"method": "concat", "params": ["foo", "bar"], "id": 123}`
The GO's client implementation will always wrap the params property in an additional array:
`{"method": "concat", "params": [["foo", "bar"]], "id": 123}`
This works fine as long as both ends use the GO's implementation and expect the actual params to be the first element of the params array. It becomes a problem if you have another server implementation of JSON-RPC 1.0 but want to use GO's client to call remote methods.

I have worked around this issue locally by fixing it in my [go fork](https://github.com/stefangluszek/go/commit/80d3eb1e1fd3c5f55dc0776bde7218c2e10650f6).
Repository: react-native
Issue: SectionList ListHeaderComponent gets rerendered when sections data changes
Labels: Ran Commands,Component: SectionList,Bug

## 🐛 Bug Report
This is a duplicate of:
https://github.com/facebook/react-native/issues/16824
https://github.com/facebook/react-native/issues/16823
https://github.com/facebook/react-native/issues/14249

But since those 3 were closed and they are missing a reproducible example of the bug I'm reopening.

## Description
If one uses a TextInput as a ListHeaderComponent when searching and when the returned sections are empty the TextInput loses focus.

If when searching the sections still have data - then the TextInput in the ListHeaderComponent doesn't lose focus.

## To Reproduce
https://snack.expo.io/H12h3BgHV

## Expected Behavior
I would expect for the TextInput not to lose focus

## Code Example
https://snack.expo.io/H12h3BgHV

## Environment
[skip envinfo]
RN 0.58

I've been trying to debug this with @pawelczerepak and what we noticed is that VirtualizedCellWrapper 
https://github.com/facebook/react-native/blob/master/Libraries/Lists/VirtualizedList.js#L1759
gets unmounted and mounted when sections is an empty array. And when this happens everything inside of it gets unmounted and mounted again. 
@sahrens - any ideas on what might be going on here?
Repository: flutter
Issue: Getting AnimationController from ExpansionTile
Labels: c: new feature,framework,f: material design,P2,team-design,triaged-design

<!-- Thank you for using Flutter!

     Please check out our documentation first:
      * https://flutter.io/
      * https://docs.flutter.io/

     If you can't find the answer there, please consider asking a question on
     the Stack Overflow Web site:
      * https://stackoverflow.com/questions/tagged/flutter?sort=frequent

     Please don't file a GitHub issue for support requests. GitHub issues are
     for tracking defects in the product. If you file a bug asking for help, we
     will consider this a request for a documentation update.

-->

Im trying to make an ExpansionTile with a custom trailing AnimatedIcon:
```dart
ExpansionTile(title: Text("Test"), trailing: AnimatedIcon(icon: AnimatedIcons.home_menu, progress: ???))
```
However I can't get the progress value from the ExpansionTile, because it's stored in a private variable https://github.com/flutter/flutter/blob/af3cdb33dab6092c47453b6cdcb790514caf8e6b/packages/flutter/lib/src/material/expansion_tile.dart#L91. Am I missing something or is it impossible for now to subscribe to that value?
Repository: electron
Issue: Unwanted Prevent Sleep behavior when using media in BrowserWindow
Labels: enhancement :sparkles:

**Is your feature request related to a problem? Please describe.**
When having media like a video in a `BrowserWindow`, the app prevents the system from going to sleep (on MacOS, have not tested Windows), much like chrome does. If that is not the desired behavior, there would be nice to have an API control wether that should happen or not

**Describe the solution you'd like**
An API that can enable/disable the "prevent from going to sleep" behavior

**Describe alternatives you've considered**
A workaround we have found for now is pausing the media when the `BrowserWindow` is blurred
Repository: flutter
Issue: Test output is hard to read
Labels: a: tests,c: new feature,team,tool,P2,team-tool,triaged-tool

The `compact` reporter isn't really compact if you have errors, info messages, or skips in your test run.  It's particularly difficult if there are multiple failures, or if you're using CI and the failures don't happen to be the very last tests that are run.

I have started work on addressing this: https://github.com/flutter/flutter/compare/master...dnfield:test_reporter but want to solicit feedback.  Some things I can think of that we want to acheive:

1. Any invocation of the test package use the new reporter (whether for packags/flutter, packages/flutter_tools, or wherever else in our repo).  My WIP change is only handling `flutter test` at the moment.
2. Right now I'm just wrapping the `json` reporter.  We could write our own reporter, but ultimately I'd like to see us able to use the data from our test runs in a more structured manner for analytics, which will probably require wrapping the JSON reporter anyway.

/cc @jonahwilliams
Repository: vscode
Issue: [html] [custom data] allow to give custom HTML tags/attributes a higher completion rank
Labels: feature-request,html

From https://github.com/Microsoft/vscode/issues/62976#issuecomment-462002568

Maybe similar to https://marketplace.visualstudio.com/items?itemName=VisualStudioExptTeam.VSIntelliCode#review-details, we should show custom tags/attributes on top of others (with a `*`).
Repository: opencv
Issue: Error on build for 3.4 with Visual Studio 2017
Labels: priority: low,category: build/install,platform: winrt/uwp

I am trying to build OpenCV 3.4 to use with UWP but I get errors saying method GetModuleHandleEx is not part of global namespace on file datafile.cpp

Error on build with Visual Studio 2017. 
core/src/utils/datafile.cpp  doesn't build for UWP (Windows Store 10.0 x64)
Error	C2039	'GetModuleHandleEx': no es un miembro de '`global namespace''	opencv_world
Error	C2065	'GET_MODULE_HANDLE_EX_FLAG_FROM_ADDRESS': identificador no declarado opencv_world
![opencvvs2017error](https://user-images.githubusercontent.com/17787020/52661967-d72e8d80-2ec8-11e9-8c33-f18595f6c529.PNG)

_Originally posted by @pablolq in https://github.com/opencv/opencv/issues/7143#issuecomment-462896650_
Repository: vscode
Issue: Warning for trailing spaces in folder names
Labels: feature-request,languages-diagnostics

Issue Type: <b>Feature Request</b>

My colleague and I had an insanely embarassing and unforgetable experience today at work due to the lack of a feature involving trailing spaces in folder names.

I was teaching my colleague how to use Redux with React Native when we got an error in the console. Something along the lines: "Could not resolve module reducers. Path ./reducers not found."

I had encountered this issue many times before with previous versions of Expo in which the Metro Bundler cache would not update correctly and it would not recognize the creation of a new folder. I believed the problem to be related to the cache and my colleague and I spent half a day researching the issue on Google and StackOverflow.

Eventually, we realized that my colleague had accidentally included a space at the end of the folder name as follows: "reducers ". Not in the import statement, however.

There should undoubtedly be a feature in VSCode that enables a warning message by default for trailing spaces in both folder and file names.

Though I will admit, the laugh we got out of this was well worth the loss of time.

VS Code version: Code - OSS 1.30.2 (61122f88f0bf01e2ac16bdb9e1bc4571755f5bd8, 2019-01-08T23:10:56.746Z)
OS version: Linux x64 4.20.6-arch1-1-ARCH


<!-- generated by issue reporter -->
Repository: TypeScript
Issue: Make automatic adding missing imports smarter
Labels: Suggestion,Awaiting More Feedback

### Scenario 1: Fuzzy Logic
I have a several files containing `import {connect} from "react-redux";`.  If I use the add missing imports context menu to add the missing import for `connect`, the system should realize "react-redux" is far more probable module than "net", "tls", and "http2", which are all recommended higher than "react-redux", but none of which are used anywhere in my project.

### Scenario 2: Respect node conventions
If I have a file `./myfolder/index.ts`, I can import the file with `import "./myfolder"`.  The system currently resolves missing imports from `./myfolder` as `./myfolder/index.ts`, when it should actually use `./myfolder`.
Repository: pytorch
Issue: Download speed issues with the pytorch conda channel
Labels: module: dependency bug,triaged

Download speed of pytorch/torch is too slow. I'm downloading the file around 80KB/s at 24MBit/s connection. I've searched the same file I've been downloading and found at a mirror which lets 1MB/s download speed, what is going on with pytorch? not enough bandwidth for science?

original link for pip install torchvision was "https://files.pythonhosted.org/packages/93/b3/672813e65ac7605bac14a2a8e40e1a18e03bf0ac588c6a799508ee66c289/torch-1.0.1.post2-cp27-cp27mu-manylinux1_x86_64.whl"

the link I'm downloading it now; https://pypi.lcsb.uni.lu/packages/93/b3/672813e65ac7605bac14a2a8e40e1a18e03bf0ac588c6a799508ee66c289/torch-1.0.1.post2-cp27-cp27mu-manylinux1_x86_64.whl

this is seriously weird.
Repository: rust
Issue: rustdoc should skip unresolvable private members 
Labels: T-rustdoc,A-visibility

When using the `doc_cfg` feature it's useful to run rustdoc on structures that aren't defined for the running platform.  However, compilation will fail if those structures have private members that also aren't defined on the running platform.  The only solution is to laboriously `#[cfg]` gate each private member, even if the struct itself is already gated.  This issue severely restricts the usefulness of `doc_cfg`.  Here's an example:
```rust
#[cfg(any(target_os = "freebsd", rustdoc))]
struct NewType {
    x: libc::something_freebsd_specific
}
```
rustdoc will fail unless you add another `#[cfg(target_os = "freebsd")]` attribute directly on `x`.  It should be possible for rustdoc to simply ignore `x`, since private members aren't normally documented anyway.

For a real-world example, see https://github.com/nix-rust/nix/pull/1019/files#diff-78f451c970a63ad3a93cab0d2670b05dR12 .  That PR only builds because of the extra `#[cfg()]` gate on the `libc::ucontext` variable, even though the entire module is already restricted to Linux.

doc_cfg tracking issue: https://github.com/rust-lang/rust/issues/43781
Repository: pytorch
Issue: Errors running distributed example
Labels: oncall: distributed,triaged

I am new to pytorch and distributed learning in general and I’m trying to go through this tutorial here: https://pytorch.org/tutorials/beginner/aws_distributed_training_tutorial.html. After setting everything up, when I run the 4 different python processes (2 on each machine) I always get the following error:

```
File “/home/ec2-user/anaconda3/envs/pytorch_p36/lib/python3.6/site-packages/torch/distributed/rendezvous.py”, line 95, in _tcp_rendezvous_handler
store = TCPStore(result.hostname, result.port, start_daemon)
RuntimeError: Address already in use
```

I feel like this is somehow related to the init_method being specified. I’m using the rank 0 machine ip and port for that value as specified in the tutorial. Nothing else is running on that port. Am I missing something about how to configure this properly?

I got around this by setting up EFS in AWS to avoid using the TCP init method, but then I ran into a second issue.

```
RuntimeError:
        An attempt has been made to start a new process before the
        current process has finished its bootstrapping phase.

        This probably means that you are not using fork to start your
        child processes and you have forgotten to use the proper idiom
        in the main module:

            if __name__ == '__main__':
                freeze_support()
                ...

        The "freeze_support()" line can be omitted if the program
        is not going to be frozen to produce an executable.
```

From what I found on this error it seems to be an issue that happens on Windows, but I'm running linux. I tried adding the suggested line anyways, but still get the same error.

Not sure what else to try at this point.
Repository: vue
Issue: 升级2.6以上版本后在部分安卓机上input使用v-model的同时有对数据格式化时光标异常
Labels: need repro,browser quirks

### Version
2.6.6

### Reproduction link
https://codepen.io/matf5/project/editor/ARGnKV


```
computed: {
    cardNoFormat: {
      get() {
        return this.cardNo.replace(/(\d{4})(?=\d)/g, '$1 ');
      },
      set(value) {
        this.cardNo = value;
      }
    }
}
```




### Steps to reproduce
例如输入输入62148时，数据会变为6214 8

### What is expected?
光标位于8后面

### What is actually happening?
有问题的安卓机如vivo，华为荣耀在部分浏览器内核下（如uc）光标会卡在8前面

<!-- generated by vue-issues. DO NOT REMOVE -->
Repository: kubernetes
Issue: Distribute debugging symbols for every binary in a Kubernetes release
Labels: kind/feature,help wanted,sig/release,lifecycle/frozen,needs-triage

**What would you like to be added**:
Distribute the debugging symbols generated when building every binary (e.g. kube-apiserver, kube-scheduler, kube-controller-manager, etc) in a Kubernetes release.

**Why is this needed**:
To allow debugging of official Kubernetes binaries "in situ," i.e., in a cluster that is already deployed. Currently, end users who want to debug "in situ" have to [build and distribute their own binaries](https://github.com/kubernetes/kubernetes/pull/70276#issuecomment-436831592). End users who choose to use (or are forced to use) official Kubernetes binaries are unable to debug "in situ."

Delve, the de facto go debugger, [added support in November 2018](https://github.com/go-delve/delve/commit/51c342c6b70ac8f75298ffc534e220da751c19d6) for reading debug symbols from external files. And GDB [has had support](https://sourceware.org/gdb/onlinedocs/gdb/Separate-Debug-Files.html) for longer.
Repository: terminal
Issue: AllocConsole failing with ERROR_GEN_FAILURE as a service in rare cases
Labels: Product-Conhost,Area-Server,Issue-Bug

We have a software package running on thousands of Windows machines from XP onward.  In 2 cases we have received a bug report that the users application fails to call AllocConsole with error code ERROR_GEN_FAILURE.  This has been seen once on a Windows Server 2008 system and the second on a Windows Server 2012 system.  In both cases the error was encountered while running as a Service.
The error message is "A device attached to the system is not functioning. (0x1f)"

Unfortunately we have not been able to reproduce this on any other systems.  It happens 100% of the time of the failing system however.

I was able to find any documentation that shows this as a possible error.
Any thoughts on this would be appreciated.
Repository: rust
Issue: Getter functions and indexing don't seem to follow the same lifetime rules
Labels: P-medium,T-compiler,A-NLL,NLL-complete

This is my first issue for the Rust project. Apologies up front if it is somehow unclear. I'd be happy to provide any additional information you need.

DESCRIPTION

The issue I'm running into is this. Consider the following code:
`collection.get_mut(collection.get(0).unwrap().index).unwrap().index = 1;`

The inner call to `get` causes an immutable borrow on `collection`. We then obtain a index from the return value, and use that for the outer call to `get_mut`. This causes a mutable borrow on `collection`.

The above code compiles just fine. My assumption is that because `index` implements `Copy`, NLL allow the immutable borrow on `collection` to be dropped as soon as we have a copy of `index`, so we can borrow it mutably again after that.

So far, so good. However, now consider the following code:
`collection[collection[0].index].index = 1`;

EXPECTED BEHAVIOR

This should be equivalent to the earlier code. In fact, I've implemented the `Index` trait to forward to `get(index).unwrap()`.

ACTUAL BEHAVIOR

This time, however, the compiler complains:
`error[E0502]: cannot borrow `collection` as immutable because it is also borrowed as mutable`


For more context, here is a playground link that illustrates the problem:
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=21d19fde2c8c58d4859bfdefa3b7720a
Repository: flutter
Issue: TextFormField doesn't support input shortcuts
Labels: a: text input,c: new feature,framework,P2,team-framework,triaged-framework

Some input software have some shortcut to help us quickly edit the input fields (like the screenshot below), but it seems the `TextFormField` (and maybe `TextField` and so on) doesn't support those behavour?
![image](https://user-images.githubusercontent.com/12682063/52709077-b6e4f880-2fc6-11e9-8992-eda3f502774e.png)


***
> flutter analyze
![image](https://user-images.githubusercontent.com/12682063/52163505-f46f9900-271d-11e9-8f1e-a26a1784fa58.png)

> flutter doctor -v
![image](https://user-images.githubusercontent.com/12682063/52163516-1406c180-271e-11e9-9a34-335ddaebd330.png)
Repository: opencv
Issue: Some Suggestions to OpenCV Team
Labels: RFC

##### System information (version)
<!-- Example
- OpenCV => 3.1
- Operating System / Platform => Windows 64 Bit
- Compiler => Visual Studio 2015
-->

- OpenCV => 4.0.1
- Operating System / Platform => Windows 64 Bit
- Compiler => Visual Studio 2017

**Suggestions to OpenCV Team:**
I want to suggest for some operation which is needed to be added in the feature releases.

1. Mat multiplication operation needs to support an optional mask parameter.
2. Mat division operation needs to support an optional mask parameter.
3. The normalize operation needs to support inverse of the current Min-Max operation, it means for example if we have an array like [0.1, 0.2, 0.3] then if we want to normalize the array in the range 0~255 it can give as [255, 128, 0], but now only gives [0, 128, 255]

These operations were what we needed, but was not implemented in OpenCV. And also when I searched, other people also ask these questions.
Thanks! Long live with OpenCV!
Repository: opencv
Issue: Updating installation instructions, adding package manager installation and macOS
Labels: category: documentation

https://docs.opencv.org/3.4.5/df/d65/tutorial_table_of_content_introduction.html

This link contains links to installation instructions for Linux, Windows and iOS, but not macOS.

A lot of macOS users will know that they can probably use the instructions for Linux (they can right? I'm not 100% sure) - but it's not safe to assume that they all will. Also, the Linux instructions do not mention that they can work on macOS, too (they only say`The following steps have been tested for Ubuntu 10.04 but should work with other distros as well.`). 

I don't have too much knowledge about OpenCV, but I feel like those instructions (for Linux, for example) should be updated. The fact that they mention testing on Ubuntu 10.04 is a hint towards that. They are also 100 lines long (if counting a few whitespace-only lines) and yet never mention installing via something like `apt-get opencv`. Is there a reason there isn't a package manager installation? I found instructions to install with `brew` on macOS, could that just be added there?
Repository: go
Issue: proposal: cmd/go/internal/lockedfile: add Append function
Labels: Proposal,Proposal-Hold

This is a proposal of adding `Append` function to [cmd/go/internal/lockedfile](https://tip.golang.org/pkg/cmd/go/internal/lockedfile/).  
`Append` will act as [Write](https://tip.golang.org/pkg/cmd/go/internal/lockedfile/#Write), except that it will not truncate a file if it already exists.