sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: flutter
Issue: Hot reloading messaging is confusing
Labels: tool,t: hot reload,a: quality,P3,team-tool,triaged-tool

I ❤️ the new IDE "reload-on-save" feature(s), but I think the output emitted is pretty confusing:

> Launching lib/main.dart on iPhone SE in debug mode...
> Reloaded 0 of 466 libraries in 410ms.
> Reloaded 0 of 466 libraries in 303ms.

I ask myself the following questions when reading this output:

* Does it mean the reload _failed_?
* Does it mean that my save didn't require reloading any libraries?
* Why did it take 300ms to do nothing?

I think it might be more friendly to either say:

> "No libraries required reloading (0 of 466 in 303ms)."
> "Reload successful but no libraries reload (...)"

Upon further investigation, I managed to find out it was because I was editing files that were not yet imported from `lib/main.dart`. That seems unfortunate, because it means if I'm creating new files or editing files in `test/` I'm still reloading.

Other suggestions:

* Maybe don't include libraries that are dependencies? For example I only have "1" library.

## Steps to Reproduce

* Create a brand new repository `flutter create hello`
* Navigate to the new repository and run it (`cd hello`, `flutter run`)
* Add a new file (i.e. `lib/src/data.dart`) that is not imported yet
* Reload (in most IDEs, this is on save)

Originally moved from https://github.com/Dart-Code/Dart-Code/issues/450.

## Logs

> Launching lib/main.dart on iPhone SE in debug mode...
> Reloaded 0 of 466 libraries in 410ms.
> Reloaded 0 of 466 libraries in 303ms.
> Reloaded 0 of 466 libraries in 485ms.

## Flutter Doctor

```
[✓] Flutter (on Mac OS X 10.12.6 16G29, locale en-US, channel master)
    • Flutter at /Users/matan/git/flutter
    • Framework revision 167382480a (12 hours ago), 2017-11-23 10:24:16 +0100
    • Engine revision 93b2179597
    • Tools Dart version 1.25.0-dev.11.0

[✓] Android toolchain - develop for Android devices (Android SDK 25.0.2)
    • Android SDK at /Users/matan/Library/Android/sdk
    • Unable to locate Android NDK.

    • Unable to locate compiler in Android NDK.

    • Platform android-25, build-tools 25.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b05)

[✓] iOS toolchain - develop for iOS devices (Xcode 9.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.1, Build version 9B55
    • ios-deploy 1.9.2
    • CocoaPods version 1.3.1

[✓] Android Studio (version 2.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_112-release-b05)

[✓] IntelliJ IDEA Ultimate Edition (version 2017.2.5)
    • Flutter plugin version 17.0
    • Dart plugin version 172.4343.25

[✓] Connected devices
    • iPhone SE • 9F33391B-D2D7-4E44-95C4-9A912FDCD824 • ios • iOS 11.1 (simulator)
```
Repository: go
Issue: path/filepath: Clean definition is not idempotent 
Labels: Documentation,NeedsFix

Please answer these questions before submitting your issue. Thanks!


### What version of Go are you using (`go version`)?

version 1.9

### Does this issue reproduce with the latest release?

yes

### What operating system and processor architecture are you using (`go env`)?

windows/amd64

### Issue

The algorithm defined for the `Clean` function is faulty and, in addition, is not faithfully implemented.

In particular, the definition is faulty because the idempotent condition

    Clean(Clean(x)) == Clean(x)

could fail, if `Separator` is not `'/'`, as on Windows systems.

In fact, by the algorithm documentation, on Windows systems `'/'` would not be processed as `Separator`, until the final translation to `Separator`. Thus, applying `Clean` once or twice (with `'/'` translated to `Separator` at the end of the first cleaning step) could return different results, contrary to the idempotent condition.

In addition, the algorithm implementation slightly deviates from the documentation. In fact, every character that satisfies

    os.IsPathSeparator(c) == true

is processed as `Separator`, not only `Separator`, as stated in the algorithm definition. On Windows systems, this is true for both: `'\\'` and `'/'`.

Please note that, if `os.IsPathSeparator('/')` is true for all available implementations, `'/'` is always processed as `Separator`. Thus, idempotence can be magically preserved, despite the errors in the algorithm definition and implementation.

### Proposal

I propose to clarify the algorithm documentation and adapt the implementation as follows.

#### Documentation

1. change the first sentence to

> Clean returns the shortest path name equivalent to path by purely lexical processing. First, it replaces any occurrence of slash by Separator. Then, it applies the following rules iteratively until no further processing can be done:

2. replace `slash` with `Separator` in the sentence

> The returned path ends in a slash ...

3. remove the sentence

> Finally, any occurrences of slash are replaced by Separator.

#### Implementation

1. add the follwing statement at the first line

    path = FromSlash(path)

2. change any expression like `os.IsPathSeparator(x)` to

    x == Separator

2. change the last line to:

    return out.string()
Repository: go
Issue: fmt: "%%" placeholder accepts incorrect index values 
Labels: NeedsFix

What version of Go are you using (go version)?

go1.9.2

Does this issue reproduce with the latest release?

yes

What operating system and processor architecture are you using (go env)?

macOS Sierra 10.12.6, darwin/amd64

### What did you do?

Run the following code

```
package main

import (
	"fmt"
)

func main() {
	fmt.Printf("%[100]%")
	fmt.Printf("%[-100]%")
	fmt.Printf("%[not a mumber]%")
}

```

### What did you expect to see?

In all cases, errors about incorrect index is printed.

### What did you see instead?

In all cases, '%' is printed. This behaviour is unexpected and, at least, not documented here:
https://golang.org/pkg/fmt/
Repository: angular
Issue: Possibility to have a loop animation, using triggers. [Feature Request]
Labels: feature,area: animations,freq1: low,P4,feature: under consideration

Hello i`m trying to make a loop animation with full control using triggers, right now theres no easy way to make it, only a few possibles work arounds.

## I'm submitting a...
<!-- Check one of the following options with "x" -->
<pre><code>
[ ] Regression (a behavior that used to work and stopped working in a new release)
[ ] Bug report  <!-- Please search GitHub for a similar issue or PR before submitting -->
[X] Feature request
[ ] Documentation issue or request
[ ] Support request => Please do not submit support request here, instead see https://github.com/angular/angular/blob/master/CONTRIBUTING.md#question
</code></pre>

## Current behavior
Not possible to make a loop inside a trigger config, right now i have to check animation done to swap the trigger back and create the loop. 

edit: 
sharing the example of how i do now: 
https://angular-animations-demo.firebaseapp.com/loop-demo
source here: 
https://github.com/leonardopaiva/angular-animation-basic-project/tree/master/src/app/loop-demo


## Expected behavior
Something like a parameter to group, query or animate, setting loop true, or the number of repetitions, animate(300, boolLoopTrue) or animate(300, {boolRepetitions: 3}),  example:
<pre><code>
transition('default => standby', [
      group([
        query('.character', [
           animate(300, style({
            opacity: 0
          })),
          animate(300, style({
            opacity: 1
          }))
        ], boolLoopTrue)
      ])
    ])

</code></pre>


## What is the motivation / use case for changing the behavior?
I`m trying to make complex animations and need it to make things right, stop using work arounds.


## Environment
Angular 
<pre><code>
Angular version: 5.0.0
"@angular/animations": "^5.0.1"

</code></pre>
Repository: rust
Issue: Test coverage for double trailing commas is poor
Labels: C-cleanup,A-parser,E-needs-test

#### Theory

Every place in the grammar which supports trailing commas should be tested that it fails for double commas, lest Rust be locked into supporting it forever.  A particularly notable case of this is for `macro_rules!` macros, many of which must manually implement their own trailing comma support (leading to more chances for mistakes).

#### Reality

```sh
[ lampam @ 12:22:45 ] (master •2) ~/asd/clone/rust/src/test
$ ls **/*.rs -1 > files
$ python3
```

```python
>>> import re
>>> contents = [open(x.strip()).read() for x in open('files')]
>>> r = re.compile(',\\s*,\\s*[\\])}]', re.MULTILINE)
>>> [x for x in map(r.findall, contents) if x]
[]
```
Repository: pytorch
Issue: Considerable slowdown in Adam.step after a number of epochs with multiple losses
Labels: awaiting response (this tag is deprecated),needs reproduction,module: performance,module: optimizer,triaged

I have a model with multiple outputs and, therefore, multiple losses. When training I accumulate the losses using retain_graph. Something along the lines of:

```python
self.zero_grad()
for output_label, output in self(input, target).items():
    loss = self.loss(output, target[output_label])
    loss.backward(retain_graph=True)
self.optimizer.step()
```

where input, output and target are dictionaries with the respective data for the different inputs and losses.

I am using Adam for the optimization. 
I've noticed that after a number of epochs, the running time of an epoch goes suddenly up from 7sec to 34sec.
I also noticed a slowdown of CPU usage in my computer (I haven't test this yet on the GPU). Memory usage doesn't seem to increase.

I profiled the code and I saw this (output from cProfile):

Normal epoch:
```
       52    0.012    0.000    3.538    0.068 adam.py:30(step)
      624    0.646    0.001    0.646    0.001 {method 'addcdiv_' of 'torch._C.FloatTensorBase' objects}
```
Slow epoch:
```
       52    0.013    0.000   24.576    0.473 adam.py:30(step)
      624   21.469    0.034   21.469    0.034 {method 'addcdiv_' of 'torch._C.FloatTensorBase' objects}
```
I've tested with other adaptive losses like Adagrad, and there I can't see the issue.
It seems to be related to this line of code in Adam.step():
```
p.data.addcdiv_(-step_size, exp_avg, denom)
```
Any ideas about why this is happening? It seems like suddenly the size of the accumulated gradient explodes, but  I can't see why.

cc @vincentqb @VitalyFedyunin @ngimel
Repository: opencv
Issue: Open CV Segmentation Violation Java
Labels: priority: low,incomplete

##### System information (version)
Open CV 3.3.1
Linux ARM Debian
Oracle JDK 8 and Open JDK 8
##### Detailed description

Core dump when loading native Library, same as closed issue #10080 which I believe needs to be reopened.
I've investigated at length, confirm compiling with FFMPEG disabled, the problem doesn't exist. The problem appears to be when the libopenmpt library is loaded, at least the stack trace from gdb points there.

```
#0  0x5e21b8c4 in ?? () from /usr/lib/arm-linux-gnueabihf/libopenmpt.so.0
#1  0x76f15c5c in ?? () from /lib/ld-linux-armhf.so.3
```

##### Steps to reproduce

Compile with FFMEPG on ARM / Debian or Raspberry PI and load the native Open CV library.
Repository: TypeScript
Issue: tsc --watch should listen for stdin requests to transpile on demand
Labels: Suggestion,Needs Proposal

nodemon will listen for "rs" streaming to `process.stdin` and this allows the user to manually restart a process.

In the same way, there are times where it would be nice to manually restart an incremental build with `tsc --watch`, although it might not be much faster since it will rebuild everything, it should be a little smoother.
Repository: opencv
Issue: Increasing efficiency of the Keypoint and DMatch Python wrappers
Labels: feature,category: python bindings,category: features2d

##### Detailed description
Python wrapper for `Keypoint` and `DMatch` is totally inefficient in which we need to access each object separately in order to extract its attributes (center, radius, trainingIdx and queryIdx), rather than using Numpy arrays.

Is it possible to change that behavior to something similar to `connectedComponentsWithStats` wrapper of the `stats` variable?  That wrapper is a Numpy array, smoothly used in conjugation with an exported enum, which allows vectorized operations without any need for loops.
Repository: pytorch
Issue: [Feature Request] Implement "same" padding for convolution operations?
Labels: high priority,module: nn,module: convolution,triaged,enhancement,needs design

The implementation would be easy, but could help many people suffered from the headache of calculating how many padding they need.

cc @ezyang @gchanan @zou3519 @bdhirsh @jbschlosser @albanD @mruberry @walterddr
Repository: youtube-dl
Issue: Unsupported URL :- cricket.com.au
Labels: site-support-request

Cricket.com.au yields Unspported URL.  <br>
Here is the log. <br>
youtube-dl -v http://www.cricket.com.au/news/match-report/australia-england-first-ashes-test-day-three-gabba-live-stream-scores-highlights-scores-smith-marsh/2017-11-25 > log.txt
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--prefer-ffmpeg', u'-v', u'http://www.cricket.com.au/news/match-report/australia-england-first-ashes-test-day-three-gabba-live-stream-scores-highlights-scores-smith-marsh/2017-11-25']
[debug] Encodings: locale UTF-8, fs UTF-8, out None, pref UTF-8
[debug] youtube-dl version 2017.11.15
[debug] Python version 2.7.12 - Linux-4.4.0-101-generic-x86_64-with-Ubuntu-16.04-xenial
[debug] exe versions: avconv 10.7, avprobe 10.7, ffmpeg 3.4-1, ffprobe 3.4-1
[debug] Proxy map: {}
WARNING: Falling back on generic information extractor.
ERROR: Unsupported URL: http://www.cricket.com.au/news/match-report/australia-england-first-ashes-test-day-three-gabba-live-stream-scores-highlights-scores-smith-marsh/2017-11-25
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/generic.py", line 2170, in _real_extract
    doc = compat_etree_fromstring(webpage.encode('utf-8'))
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/compat.py", line 2539, in compat_etree_fromstring
    doc = _XML(text, parser=etree.XMLParser(target=_TreeBuilder(element_factory=_element_factory)))
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/compat.py", line 2528, in _XML
    parser.feed(text)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1653, in feed
    self._raiseerror(v)
  File "/usr/lib/python2.7/xml/etree/ElementTree.py", line 1517, in _raiseerror
    raise err
ParseError: not well-formed (invalid token): line 19, column 26
Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/YoutubeDL.py", line 784, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/common.py", line 437, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/lib/python2.7/dist-packages/youtube_dl/extractor/generic.py", line 3075, in _real_extract
    raise UnsupportedError(url)
UnsupportedError: Unsupported URL: http://www.cricket.com.au/news/match-report/australia-england-first-ashes-test-day-three-gabba-live-stream-scores-highlights-scores-smith-marsh/2017-11-25
Repository: rust
Issue: Adding serde_json as an unused external crate causes a compiler error in unrelated code
Labels: C-enhancement,A-diagnostics,T-compiler

This code compiles:
```rust
use std::cell::Cell;
use std::default::Default;

fn main() {
   let cell = Cell::new(0u64);
    cell.get() == Default::default();
}
```

This code does not:
```rust
extern crate serde_json;

use std::cell::Cell;
use std::default::Default;

fn main() {
   let cell = Cell::new(0u64);
    cell.get() == Default::default();
}
```
```
error[E0283]: type annotations required: cannot resolve `u64: std::cmp::PartialEq<_>`
 --> src/lib.rs:7:16
  |
7 |     cell.get() == Default::default();
  |                ^^

error: aborting due to previous error
```
Repository: pytorch
Issue: Wrap Cephes library for mathematical special functions
Labels: feature,triaged,module: numpy

The Cephes library provides many useful special functions and is used inside Scipy. The Deepmind folks wrapped Cephes for use in Torch ([site](http://deepmind.github.io/torch-cephes/), [code](https://github.com/deepmind/torch-cephes)).

Is if feasible to wrap Cephes for similar use in PyTorch?

**EDIT**:
Request for the Cephes functions should be directed to https://github.com/pytorch/pytorch/issues/50345 (Issue for tracking torch.special) as most of these functions are exposed in scipy via special module.

cc @mruberry @rgommers
Repository: svelte
Issue: await once
Labels: feature request,stale-bot,temp-stale

Follow-up to #952 / https://github.com/sveltejs/svelte/issues/654#issuecomment-345490875. It would modify the behaviour of `await` blocks such that you'd only see the 'pending' state once — thereafter, whenever a new promise value was set, the old one would be preserved until the promise resolved. An additional argument would be passed to the `then` block, allowing the UI to indicate that the currently displayed data was out of date:

```html
<!-- autocomplete suggestion list — we don't want to blow away
     the previous set of suggestions while we're waiting for
     the server to send us some new ones -->
{{#await once suggestions}}
  <span>loading...</span>
{{then value, pending}}
  <datalist id='suggestions' style='opacity: {{pending ? 0.5 : 1}}'>
    {{#each value as suggestion}}
      <option>{{suggestion}}</option>
    {{/each}}
  </datalist>

  {{#if pending}}
    <span>updating...</span>
  {{/if}}
{{catch err}}
  <span class='error'>could not get suggestions!</span>
{{/await}}
```
Repository: svelte
Issue: Async/streaming SSR renderer
Labels: feature request

Now that we have an `await` template primitive, it makes sense to have a streaming renderer:

```js
require('svelte/ssr/register');
const app = express();

const SomeRoute = require('./components/SomeRoute.html');

app.get('/some-route', (req, res) => {
  SomeRoute.renderToStream({
    foo: getPromiseSomehow(req.params.foo)
  }).pipe(res);
});
```

It would write all the markup to the stream until it encountered an `await` block (at which point it would await the promise, and render the `then` or `catch` block as appropriate) or a component (at which point it would pipe the result of `childComponent.renderToStream(...)` into the main stream).

We'd get `renderAsync` for free, just by buffering the stream:

```js
const markup = await MyComponent.renderAsync({...});
```

Doubtless this is slightly more complicated than I'm making it sound.
Repository: vscode
Issue: Git - Support git subtree
Labels: feature-request,git

- VSCode Version: Code - Insiders 1.19.0-insider (89b158e11cb1c3fe94a3876222478ed2d0549fc8, 2017-11-24T05:14:03.606Z)
- OS Version: Windows_NT x64 10.0.15063
- Extensions: 

Extension|Author (truncated)|Version
---|---|---
markdown-mermaid|bie|0.1.1
team|ms-|1.122.0
debugger-for-chrome|msj|3.5.0



---

Steps to Reproduce:

1. Ctrl-shift-P
2. type "Git:"

<!-- Launch with `code --disable-extensions` to check. -->
Reproduces without extensions: Yes

Git [git subtree](http://alistra.ghost.io/2014/11/30/git-subtree-a-better-alternative-to-git-submodule/) commands is currently missing in the git command list and wasn't included in the git sync button.

Subtree is great for maintaining a monorepo for large projects where there is a single source of truth for all the code. 

Currently this issue impacts developers experience as it is tedious to do manual sync of all the changes in the subtree with the original git repo of the project for the subtree. 

Proposal:
1. Add all the git subtree command (add, push, pull)
Repository: neovim
Issue: VimL heredoc (<<HERE) for user commands
Labels: enhancement,vimscript

Currently if one wants to emulate `:python` with user commands he will be stuck at implementing `<< EOF`: there is no way for user commands to have multiline input, except for not adding `-bar` and using `:execute 'MyCmd' "multiline\nstring"`. On the other hand it is impossible to just add `command -herestring` and expect it to work fine: not unless somebody wants to state that user commands must be defined not prior to its *usage*, but prior to its *appearance* (i.e. prior to the first function definition which happens to have that command inside) and can’t switch definition after that (or give up on having proper parser). Thus the suggestion: knowing that `python << EOF\n{multiline string}\nEOF` yields just the same result as `execute 'python' "{multiline string}"` I am suggesting to add `:here` command used like that:

    here MyCmd << EOF
    foo
    EOF

and

    if 1
      here! MyCmd << EOF
        First non-empty line appears as having zero indent when bang is used.
       And this appears as an error for it being less indented then the first one.
      EOF
    endif

. The only thing which `:here` command will do is calling `MyCmd` with multiline string in `<args>`, universally and without requiring any additional support in `MyCmd` definition.

Additionally `:here` may accept range and forward it to `MyCmd`. Also `:here!` is good for built-in `:python` itself.
Repository: react
Issue: RFC: Drop isAttributeNameSafe() check
Labels: Type: Enhancement,Type: Breaking Change,React Core Team

We currently validate DOM attributes on the client and ignore the ones with invalid names:

https://github.com/facebook/react/blob/0c164bb4851e78e5f789dd8619f17ffcfee0221f/packages/react-dom/src/client/DOMPropertyOperations.js#L202-L204

This check used to be important for safety when we did `innerHTML` rendering on the client side, but it's not anymore. If we just let it call `setAttribute`, the browser would throw on a bad attribute name.

This check used to run very infrequently (only for data attributes and custom elements), but now more attributes follow this code path (since any "simple" attributes with the same names are effectively treated as unknown attributes). So even though we cache the result, it seems unfortunate to do the work that the browser is already doing for us.

While this would be a breaking change (so it has to go in 17), I think we should just remove this check, and let the browser throw. This does make spreading props blindly a bit more dangerous, but we have a warning so it should be visible.
Repository: go
Issue: cmd/compile: implement powerpc 32-bit backend (ppc32)
Labels: FeatureRequest,compiler/runtime

Tracking bug for missing PowerPC 32-bit backend. People interested in this can put a 👍 reaction here so that we can track interest.