sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: vue
Issue: Oddity with JS transition hooks used in combination with CSS
Labels: transition

### Version
2.5.17

### Reproduction link
[https://codesandbox.io/s/6x4k5vrrkn](https://codesandbox.io/s/6x4k5vrrkn)

### Steps to reproduce
Remove the [unused] `done` parameter from the `leave` callback signature in `SideSheet.vue`.

### What is expected?
The component to transition both on enter and leave.

![want](https://user-images.githubusercontent.com/22343/45375530-28221900-b5ed-11e8-90ca-f7eff6d20bc8.gif)

### What is actually happening?
The component enters immediately (without transitioning).

![get](https://user-images.githubusercontent.com/22343/45375595-5bfd3e80-b5ed-11e8-8894-7c839df72cb3.gif)

---
The `done` callback shouldn't be needed if the transition duration is implicit in CSS (as noted in the docs). However, by retaining the `done` parameter in the function signature it is unclear why it should "work" (since it's unused within the function definition itself).

<!-- generated by vue-issues. DO NOT REMOVE -->
Repository: vue
Issue: Race condition in transition-group
Labels: transition

### Version
2.5.17

### Reproduction link
[https://jsfiddle.net/nkovacs/Lskfredn/](https://jsfiddle.net/nkovacs/Lskfredn/)





### Steps to reproduce
1. Click the add button


### What is expected?
the animation should work properly, and animation classes should be cleaned up

### What is actually happening?
the enter animation doesn't work and the new item's element keeps the `list-enter-to` class forever

---
The style tag binding triggers a second rerender of the transition-group component between the transition-group setting `_enterCb` on the entering new child and `nextFrame` triggering its callback. `prevChildren` is updated to include the new item, and `update` calls the pending `_enterCb` callback. Then after that, `nextFrame` triggers, but because `_enterCb`  can only be called once, it won't be called, so the `enter-to` class will remain on the element.

This only happens if the transition-group has a move transition.

The bug also occurs if the elements are changed between `update` and `nextFrame`: https://jsfiddle.net/nkovacs/cnjso1h5/

<!-- generated by vue-issues. DO NOT REMOVE -->
Repository: go
Issue: cmd/compile: automatically stack-allocate small non-escaping slices of dynamic size
Labels: Performance,NeedsInvestigation,compiler/runtime

This commit:
https://github.com/golang/go/commit/95a11c7381e01fdaaf34e25b82db0632081ab74e

shows a real-world performance gain triggered by moving a small non-escaping slice to the stack. It is my understanding that the Go compiler always allocated the slice in the heap because the length was not known at compile time.

Would it make sense to attempt a similar code transformation for many/all non escaping slices? What would be the cons? Any suggestion on how to identify which slices could benefit from this transformation and which would possibly just create overhead?
Repository: pytorch
Issue: [JIT][tracer] Slicing shape is specialized to tensor rank
Labels: oncall: jit

Example:

```
import torch

def fill_row_zero(x):
    x = torch.cat((torch.rand(1, *x.shape[1:]), x[1:]), dim=0)
    return x

traced = torch.jit.trace(fill_row_zero, (torch.rand(3, 4),))
print(traced.graph)
traced(torch.rand(3, 4, 5))
```


```
graph(%0 : Float(3, 4)) {
  %4 : int = prim::Constant[value=1]()
  %5 : int = aten::size(%0, %4)
  %6 : Long() = prim::NumToTensor(%5)
  %7 : int = prim::TensorToNum(%6)
  %8 : int = prim::Constant[value=1]()
  %9 : int[] = prim::ListConstruct(%8, %7)
  %10 : int = prim::Constant[value=6]()
  %11 : int = prim::Constant[value=0]()
  %12 : int[] = prim::Constant[value=[0, -1]]()
  %13 : Float(1, 4) = aten::rand(%9, %10, %11, %12)
  %14 : int = prim::Constant[value=0]()
  %15 : int = prim::Constant[value=1]()
  %16 : int = prim::Constant[value=9223372036854775807]()
  %17 : int = prim::Constant[value=1]()
  %18 : Float(2, 4) = aten::slice(%0, %14, %15, %16, %17)
  %19 : Dynamic[] = prim::ListConstruct(%13, %18)
  %20 : int = prim::Constant[value=0]()
  %21 : Float(3, 4) = aten::cat(%19, %20)
  return (%21);
}
```

these `size()` calls we emit (e.g. `  %5 : int = aten::size(%0, %4)`) are specialized to the rank of the tensor we called `.shape` on
Repository: flutter
Issue: Search widget's text field might not have large enough tap area
Labels: framework,a: accessibility,P2,team-framework,triaged-framework

Currently at 44 height due to a the default edgeInsets of the no border decoration.  Android a11y scanner doesn't flag this for some reason, we should determine why.
Repository: pytorch
Issue: [CLEANUP] Context functions should return TypeExtendedInterface, not Type
Labels: triaged,better-engineering

See https://github.com/pytorch/pytorch/pull/11461 for information.

CC @ezyang.
Repository: go
Issue: cmd/go: do not cache tool output if tools print to stdout/stderr
Labels: Proposal,Proposal-Accepted,GoCommand

# **Update, Oct 7 2020**: see https://github.com/golang/go/issues/27628#issuecomment-702252564 for most recent proposal in this issue.

### What version of Go are you using (`go version`)?
tip (2e5c32518ce6facc507862f4156d4e6ac776754f), also Go 1.11

### Does this issue reproduce with the latest release?
Yes

### What operating system and processor architecture are you using (`go env`)?
darwin/amd64

### What did you do?
```
$ go build -toolexec=/usr/bin/time hello.go
# command-line-arguments
        0.01 real         0.00 user         0.00 sys
# command-line-arguments
        0.12 real         0.11 user         0.02 sys
$ go build hello.go 
# command-line-arguments
        0.01 real         0.00 user         0.00 sys
# command-line-arguments
        0.12 real         0.11 user         0.02 sys
$
```

### What did you expect to see?
The second invocation of `go build` doesn't have `-toolexec`, so it should not invoke the toolexec command (which I think it doesn't), nor reprint its output.

### What did you see instead?
toolexec output is reprinted.

In fact, I think it probably should not cache at all if `-toolexec` is specified, since the external command that toolexec invokes may do anything, and (intentionally) not reproducible.

cc @dr2chase
Repository: rust
Issue: Tracking Issue: Procedural Macro Diagnostics (RFC 1566)
Labels: A-diagnostics,T-lang,T-libs-api,B-unstable,C-tracking-issue,A-macros-1.2,Libs-Tracked,I-lang-radar

This is a tracking issue for diagnostics for procedural macros spawned off from https://github.com/rust-lang/rust/issues/38356.

## Overview

### Current Status

  * Implemented under `feature(proc_macro_diagnostic)`
  * In use by Rocket, Diesel, Maud

### Next Steps

- [x] https://github.com/rust-lang/rust/pull/44125
- [x] Implement introspection methods (https://github.com/rust-lang/rust/pull/52896)
- [x] Implement multi-span support (https://github.com/rust-lang/rust/pull/52896)
- [ ] Implement lint id for warnings (https://github.com/rust-lang/rust/pull/135432)
- [ ] Document thoroughly
- [ ] Stabilize

## Summary

The initial API was implemented in https://github.com/rust-lang/rust/pull/44125 and is being used by crates like Rocket and Diesel to emit user-friendly diagnostics. Apart from thorough documentation, I see two blockers for stabilization:

  1. **Multi-Span Support**
  
     At present, it is not possible to create/emit a diagnostic via `proc_macro` that points to more than one `Span`. The internal diagnostics API makes this possible, and we should expose this as well.

     The changes necessary to support this are fairly minor: a `Diagnostic` should encapsulate a `Vec<Span>` as opposed to a `Span`, and the `span_` methods should be made generic such that either a `Span` or a `Vec<Span>` (ideally also a `&[Vec]`) can be passed in. This makes it possible for a user to pass in an empty `Vec`, but this case can be handled as if no `Span` was explicitly set.

  2. **Lint-Associated Warnings**

     At present, if a `proc_macro` emits a warning, it is unconditional as it is not associated with a lint: the user can never silence the warning. I propose that we require proc-macro authors to associate every warning with a lint-level so that the consumer can turn it off.

     No API has been formally proposed for this feature. I informally proposed that we allow proc-macros to create lint-levels in an ad-hoc manner; this differs from what happens internally, where all lint-levels have to be known apriori. In code, such an API might look lIke:

     ```rust
     val.span.warning(lint!(unknown_media_type), "unknown media type");
     ```

     The `lint!` macro might check for uniqueness and generate a (hidden) structure for internal use. Alternatively, the proc-macro author could simply pass in a string: `"unknown_media_type"`.
Repository: pytorch
Issue: CrossEntropyLoss, ignore_index does not prevent back-prop if the logits are -inf
Labels: module: docs,module: nn,module: loss,triaged

## Issue description

When using CrossEntropyLoss, I assumed that as long as I ignore a target, its loss would not be calculated and will not get back propagated. Therefore I would pass logits = -float(inf) when I attempted to skip that target. 
However, even though the loss would be skipped; the loss.backward() would bring the infinity into my gradients.

I think this behavior is at least worth noticing in the document for CrossEntropyLoss. 

## Code example

```
import torch
loss = torch.nn.CrossEntropyLoss(ignore_index=-1)
input = torch.randn(3, 5, requires_grad=True)
target = torch.empty(3).long().fill_(-1)
logits = input - float('inf')
output = loss(logits, target)
output.backward()
input.grad
```

## System Info

Collecting environment information...
PyTorch version: 0.4.1
Is debug build: No
CUDA used to build PyTorch: 9.0.176

OS: Ubuntu 16.04.5 LTS
GCC version: (Ubuntu 5.4.0-6ubuntu1~16.04.10) 5.4.0 20160609
CMake version: version 3.5.1

Python version: 3.6
Is CUDA available: Yes
CUDA runtime version: Could not collect
GPU models and configuration:
GPU 0: GeForce GTX 1080 Ti
GPU 1: GeForce GTX 1060 6GB

Nvidia driver version: 396.44
cuDNN version: Probably one of the following:
/usr/lib/x86_64-linux-gnu/libcudnn.so.7.0.5
/usr/lib/x86_64-linux-gnu/libcudnn_static_v7.a

Versions of relevant libraries:
[pip] Could not collect
[conda] cuda90                    1.0                  h6433d27_0    pytorch
[conda] pytorch                   0.4.1           py36_cuda9.0.176_cudnn7.1.2_1    soumith
[conda] torchvision               0.1.9            py36h7584368_1    soumith


cc @brianjo @mruberry @albanD @jbschlosser
Repository: rust
Issue: Allow setting breakpoint when Err() is constructed in debug builds
Labels: C-feature-request

This would be really helpful for tracking down the source of errors. It seems the simplest way to allow this would be to emit a no-inline function that's used for setting a particular enum variant.
Repository: vscode
Issue: Use the modifier properties on mouse event instead of tracking keydown/keyup
Labels: help wanted,debt,editor-drag-and-drop

Extracted from our conversation at https://github.com/Microsoft/vscode/commit/e82498a544b88f5041ec7f8b531ab8ca6eb29eaf
Repository: go
Issue: x/text/message: package level docs about MatchLanguage are unclear
Labels: Documentation,NeedsInvestigation

Please answer these questions before submitting your issue. Thanks!


### What version of Go are you using (`go version`)?
go1.11 linux/amd64

### Does this issue reproduce with the latest release?
Yes.

### What operating system and processor architecture are you using (`go env`)?
linux/amd64

### What did you do?
I tried to use `message.MatchLanguage("nl")` to obtain `language.Dutch`, as per the example on [its site](https://godoc.org/golang.org/x/text/message).

See the following code for an example:
```
package main

import (
	"golang.org/x/text/message"
	"golang.org/x/text/language"
	"fmt"
)

func main() {
	nl := message.MatchLanguage("nl")
	fmt.Println(nl) // Prints "und", expected "nl"
	fmt.Println(language.Dutch) // Prints "nl" as expected
	p := message.NewPrinter(message.MatchLanguage("nl"))
	p.Printf("%.2f\n", 5000.00) // Prints "5,000.00", expected "5.000,00"

	p2 := message.NewPrinter(message.MatchLanguage("bn"))
	p2.Println(123456.78) // Prints "5,000.00", expected "১,২৩,৪৫৬.৭৮"
}
```

### What did you expect to see?
I expected to receive `language.Dutch`, so that when I called `p.Printf("%.2f\n", 5000.00)`, I would get "5.000,00", as Dutch uses "." for thousand separators and "," for decimal separators.

### What did you see instead?
Instead `message.MatchLanguage("nl")` returned the `und Tag` rather than the `Dutch Tag`.  And therefore when I called `p.Printf()` as per the example on the package's description, I got "5,000.00", which ironically matches the example in the description, but the example is wrong.

I also tried the third example in opening example, which also did not seem to produce the expected result.

It seems to me that `message.MatchLanguage()` either does not work or does not work as described.

Note:  Making a new `Printer` with `language.Dutch` works as expected.
Repository: You-Dont-Know-JS
Issue: Calling template literals "interpolated string literals" is misleading
Labels: for second edition

Chapter in question: https://github.com/getify/You-Dont-Know-JS/blob/master/es6%20%26%20beyond/ch2.md#template-literals

The suggestion seems to be that the template literals would be all about strings; there's only examples that result in strings, and phrases like "final string value" and "generating the string from the literal" are used in what should be a general context, but the tag functions can return any object, and there's obvious use cases where they'd return regexp objects, DOM nodes, etc.
Repository: flutter
Issue: Would like to measure/track Flutter's total download size
Labels: team,framework,P2,team-framework,triaged-framework

I've seen several claims that our total download is too large.  e.g.
https://twitter.com/FerventGeek/status/1038480155990261761

@gspencergoog do you know if we already track this as part of the bundle building?

CC @FerventGeek @mit-mit
Repository: pytorch
Issue: Request to import pytest in test/*.py
Labels: module: tests,triaged

Currently PyTorch uses the builtin `unittest` module for testing. Would it be possible to add a dependency on [pytest](https://docs.pytest.org/en/latest/) so developers can more easily write parametrized tests?

While working on [test/test_distributions.py](https://github.com/pytorch/pytorch/blob/master/test/test_distributions.py), @neerajprad, @alicanb, and @fritzo have found it challenging to write heavily parametrized tests. As a result, test coverage suffers. By contrast in Pyro we use heavily parametrized tests [using pytest](https://docs.pytest.org/en/latest/parametrize.html), and our Pyro tests seem to catch many bugs that aren't caught in PyTorch's own tests (#9917, #9521, #9977, #10241).

In particular, I'd like to be able to gather [xfailing tests](https://docs.pytest.org/en/documentation-restructure/how-to/skipping.html) in batch
```py
DISTRIBUTIONS = [Bernoulli, Beta, Cauchy, ..., Weibull]

@pytest.mark.parametrize('Dist', DISTRIBUTIONS)
def test_that_sometime_fails(Dist):
    dist = Dist(...)
    assert something(dist)
```
and then mark xfailing parameters
```py
DISTRIBUTIONS = [
    Bernoulli,
    Beta,
    pytest.param(Cauchy, marks=[pytest.mark.xfail(reason='schema not found for node')]),
    ...
    Weibull,
]
```
pytest makes it easy to collect xfailing tests in batch (`pytest -v --tb=no`) and to see the entire list of xfailing tests, and makes it easy to run xfailing tests to see which have started passing since the last time tests were run (`pytest -v`, or `pytest --runxfail`). In contrast, `unittest` fixtures typically parametrize via for loops and can report only a single failure on each run. (Apologies if I'm unaware of convenient functionality in `unittest`!)

I think allowing usage of pytest in test/*.py could help improve PyTorch test coverage.

cc @mruberry
Repository: opencv
Issue: VideoCapture bug with Acer Switch 5 tablet
Labels: priority: low,category: videoio,platform: winrt/uwp

##### System information (version)

- OpenCV = 3.4.1
- Operating System / Platform => Windows 64 Bit
- Compiler => Visual Studio 2015


##### Detailed description

Camera capture on Acer Switch 5 tablet appears to work, but if I try to manually focus the camera (using the onscreen menu) it actually just applies some kind of sharpen/blur, not focus.
The Windows Camera app allows a very close (macro) focus with the tablet camera, but OpenCV is unable to reproduce this.
It is as if the parameter OpenCV thinks is focus, is actually something completely different

##### Steps to reproduce
```
#include "opencv2/opencv.hpp"
#include <iostream>
 
using namespace std;
using namespace cv;
 
int main(){
 
  // Create a VideoCapture object and open the input file
  // If the input is the web camera, pass 0 instead of the video file name
  VideoCapture cap(0); 
    
  // Check if camera opened successfully
  if(!cap.isOpened()){
    cout << "Error opening video stream or file" << endl;
    return -1;
  }

cap.set(CV_CAP_PROP_SETTINGS,1); // To display the settings
// Now if the user tries to adjust the focus, something other than focus happens!
// If I leave it on auto-focus I can focus very close up so autofocus still works
// If I switch off autofocus and try to focus using the slider, I cannot focus 
// I suspect the parameter being changed by OpenCV is NOT focus, even though it thinks it is
// (Note. I do think the parameter being changed by the auto-focus on/off check box is correct - it switches off auto focus)
     
  while(1){
 
    Mat frame;
    // Capture frame-by-frame
    cap >> frame;
  
    // If the frame is empty, break immediately
    if (frame.empty())
      break;
 
    // Display the resulting frame
    imshow( "Frame", frame );
 
    // Press  ESC on keyboard to exit
    char c=(char)waitKey(25);
    if(c==27)
      break;
  }
  
  // When everything done, release the video capture object
  cap.release();
 
  // Closes all the frames
  destroyAllWindows();
     
  return 0;
}
 ```
Repository: rust
Issue: Confusing error message when wildcard importing two same-named traits
Labels: C-enhancement,A-diagnostics,A-trait-system,T-compiler

Discovered this when using `tokio-async-await` and `futures`. Both define `FutureExt` and `StreamExt` traits in their `preludes`. Both traits are meant to be used at the same time (from what I understand).

```rust
use tokio::prelude::*;  // contains StreamExt, FutureExt
use futures::prelude::*;  // contains StreamExt, FutureExt
```

After doing both this imports rust proceeds to act like _neither_ of the traits got imported and give error help on any method usage something like "trait not in scope; use `tokio::async_await::stream::StreamExt`".

Here is a Rust playground illustrating the bug: https://play.rust-lang.org/?gist=c8ebd2f79eca354ecb2dd6c828d73ad0&version=nightly&mode=debug&edition=2015

And here is a work-around for it (in nightly): https://play.rust-lang.org/?gist=f28928ef69a1985ee79d321cb58ae7e1&version=nightly&mode=debug&edition=2015

I think the way forward here is a good warning (or error) message when a duplicate trait item import happens between two wildcard imports.
Repository: bitcoin
Issue: Test coverage of our networking code
Labels: Tests

Our python functional testing framework is pretty limited for what kinds of p2p behaviors we can test.  Basically, we can currently only make manual connections between bitcoind nodes (using the `addnode` rpc), which are treated differently in our code than outbound peers selected using addrman.

While we do have some unit-testing coverage of some of the components (like addrman, and parts of net_processing), I don't believe we currently are able to test the overall logic of how bitcoind uses those components (I recall this coming up when working on #11560, as a specific example).

Anyway I am just mentioning this here as a potential project idea, as this is a material gap in our testing that I think would be valuable to work towards improving, and I wasn't sure how well known this is.
Repository: flutter
Issue: Document how to set SystemChrome brightness properly
Labels: framework,d: api docs,has reproducible steps,P2,found in release: 3.7,found in release: 3.8,team-framework,triaged-framework

## Steps to Reproduce
1. Create a simple app with an appbar
2. Try to set the color of the icons manually with `SystemChrome.setSystemUIOverlayStyle(SystemUiOverlayStyle.dark)`
3. The icon's colors don't change

[Here's an example app](https://pastebin.com/k00S4PWs)


[Here's an example video](https://cdn.discordapp.com/attachments/408312522521706496/489544643256516619/2018-09-12_23-15-07.mp4)

I believe the problem stems from [line 479-497 in app_bar.dart](https://github.com/flutter/flutter/blob/d927c9331005f81157fa39dff7b5dab415ad330b/packages/flutter/lib/src/material/app_bar.dart#L479)

## Logs
```
[√] Flutter (Channel beta, v0.7.3, on Microsoft Windows [Version 10.0.17134.228], locale en-BE)
    • Flutter version 0.7.3 at C:\tools\flutter
    • Framework revision 3b309bda07 (2 weeks ago), 2018-08-28 12:39:24 -0700
    • Engine revision af42b6dc95
    • Dart version 2.1.0-dev.1.0.flutter-ccb16f7282

[√] Android toolchain - develop for Android devices (Android SDK 27.0.3)
    • Android SDK at C:\Android\android-sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-27, build-tools 27.0.3
    • ANDROID_HOME = C:\Android\android-sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)
    • All Android licenses accepted.

[√] Android Studio (version 3.1)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 27.1.1
    • Dart plugin version 173.4700
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1024-b02)

[√] Connected devices (1 available)
    • Android SDK built for x86 64 • emulator-5554 • android-x64 • Android 7.0 (API 24) (emulator)

• No issues found!
```
Repository: pytorch
Issue: [feature request] Triangular Matrix Representation
Labels: feature,triaged

I tried searching the documentation, but besides sparse matrices (which in most cases would use _more_ space than a dense matrix), I didn't see any tensor types that would take advantage of the ability to save space with the knowledge that the tensor is triangular. This would also save time when performing `matmul()`, I'd imagine. For example, my use case is a symmetric matrix (a modification of the distance matrix (https://en.wikipedia.org/wiki/Distance_matrix), and I only need the upper half without the diagonal, so performing operations on the other elements would be worthless, and saving them would be a waste of space.

Can such a type be implemented? Apologies if I missed something and it already exists.
Repository: pytorch
Issue: Add min mode to embedding bags
Labels: module: nn,triaged,enhancement

It would be nice to add the min mode to embeddingBag.
Following [this paper](https://arxiv.org/pdf/1803.01400.pdf) it seems like it can trigger pretty good result. More globally adding the power-mean formula would be awesome.
Additionally, the current error `ValueError: mode has to be one of sum or mean` is not correct since it does allow `max`


cc @albanD @mruberry @jbschlosser