sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: rust
Issue: Confusing error message when traits from different modules have the same name
Labels: C-enhancement,A-diagnostics,T-compiler,D-confusing,D-invalid-suggestion,S-has-mcve

This error looks pretty bizarre, right? Especially the "help:" message, which is clearly not helpful at all. (This is from the current nightly.)

```
error[E0277]: the trait bound `B: Basics` is not satisfied
   --> src\collision_detection\simple_grid.rs:115:1
    |
115 | pub fn insert_predictors <B: Basics, Settings: TimeStewardSettings <B>> (s
ettings: Settings) {
    | ^ trait `B: Basics` not satisfied
    |
    = help: consider adding a `where B: Basics` bound
    = note: required by `TimeStewardSettings`
```

What happened is that I had two traits named Basics, one in the crate root and one in the current module. Presumably, the compiler could give a clearer error message in this situation, although I'm not sure what the best way to handle it is – it would obviously be bad to give the full path for every trait mentioned in an error message. Maybe it should give the full path exactly when there are two traits (or other objects) with the same name?
Repository: angular
Issue: Order of ngModel & onModelChange
Labels: workaround1: obvious,freq1: low,area: core,core: inputs / outputs,type: confusing,P4

**I'm submitting a ...**  (check one with "x")

```
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, instead see https://github.com/angular/angular/blob/master/CONTRIBUTING.md#question
```

**Current behavior**
Currently the order of `(ngModelChange)` and `[(ngModel)]` on an input element matters. In the following case the `ngModelChange` callback is called before `ngModel` updates the value of model:

```
<select (ngModelChange)="onModelChange()" [(ngModel)]="hero.name" >
   <option>Dexter</option>
   <option>Voltron</option>
</select>
```

The following however works fine:

```
<select  [(ngModel)]="hero.name" (ngModelChange)="onModelChange()">
   <option>Dexter</option>
   <option>Voltron</option>
</select>
```

**Expected/desired behavior**

Not sure what the expected behavior should be or if this the intended behavior. But I think the order of attributes in an element should not matter.

**Reproduction of the problem**
Link to plunker: http://plnkr.co/edit/wrVrHYx3pPdLmCKz17RL?p=preview

Open up the console and select Hero name from the drop-down. Notice how the values are printed in console. Swap the order of `ngModel` and `onModelChange` and observe the difference. 
- **Angular version:** 2.0.0-rc.5
- **Browser:** Chrome Version 52.0.2743.116 m  
- **Language:** TypeScript
Repository: flutter
Issue: Would like the option to have the Drawer behind the AppBar on tablets
Labels: c: new feature,framework,f: material design,a: tablet,P2,team-design,triaged-design

The Drawer overlays the Appbar, that causes that the first item of the drawer is hidden under the statusbar.
I think there should be at leat an option to place the Drawer under the appbar.
Repository: rust
Issue: Mark dropck-related borrow errors explicitly
Labels: A-diagnostics,A-destructors,A-borrow-checker,T-compiler

Today you can use the key phrase “note: reference must be valid for the block suffix”... to get reminded that a borrow error is related to dropck. This could be improved to explicitly mention dropck rules or put the error message in that context in some way.

cc @pnkfelix
Repository: vscode
Issue: Debug console architecture
Labels: feature-request,debug

This item is intended as a discussion for the future of the debug console. Currently we use the tree to represent all elements in the debug console. This has worked well thus far but has certain limitations. The benefit of using the tree is that we can easily render complex elements - the ones that have children. The tree also allows us to lazily fetch the children of each element and is virtualised.
### Motivation for the change

https://github.com/Microsoft/vscode/issues/1957
https://github.com/Microsoft/vscode/issues/4338
https://github.com/Microsoft/vscode/issues/4326
https://github.com/Microsoft/vscode/issues/2163
https://github.com/Microsoft/vscode/issues/6845
https://github.com/Microsoft/vscode/issues/228
https://github.com/Microsoft/vscode/issues/10680
https://github.com/Microsoft/vscode/issues/26641

### Option 1: use the editor and simulate the tree using editor decorations

**pro:** find, selection, word wrap, performance
**con:** have to simulate the tree by using editor decorations / indentation, not enough flexibility (for possibly future requests of rendering custom things in the debug console). 

Note: editor could be enhanced to render custom images, but customly rendered output which the user can interact with is tricky.
### Option 2: use the editor but render a tree per complex object in a custom editor widget

**pro:** most similar to what we currently have with some benefits on top (able to find/copy outside of complex objects)
**con:** find and selection will not work inside the tree, thus making it hard to copy / find contents inside complex objects
### Option 3: render the debug console not using existing widgets

**pro:** gives most flexibility
**con:** no virtualisation, slow for a large number of elements, have to rewrite find widget, word wraping

Chrome debug console is a good example of this. They can render multiple complex elements side by side and expand / collapse them.

@weinand @alexandrudima please edit / modify as you see fit. I might have missed some of the pro/cons that we discussed, I just simply wrote what we have talked about this afternoon.
Repository: go
Issue: x/crypto/scrypt,x/crypto/argon2: add high-level APIs
Labels: Proposal-Accepted,NeedsFix,Proposal-Crypto

_Summary_: The x/crypto/scrypt package has a very simple API that puts the onus of figuring out salt generation and sensible N/r/p values on the package user. We should attempt to mirror the bcrypt packages' API and provide sensible defaults.

_Details_:
- Add a `GenerateFromPassword` function that generates output in the form `N$r$p$salt$dk` (noting that there is no 'standard' for scrypt here)
- Add a `CompareHashAndPassword` function
- Add a `Cost` function that can return the cost of a given output (i.e. for determining whether to upgrade or not)
- Provide sensible default params that provide reasonable values of N, r, p and document why/when you may wish to change them.
- Potentially provide a way to automatically determine values of N, r, p given memory (MB) and time (ms) constraints.

Note that I've done most of this work in https://godoc.org/github.com/elithrar/simple-scrypt and would seek to bring most of this in.
Repository: rust
Issue: Performance left on the table?
Labels: A-LLVM,I-slow,C-enhancement,T-compiler,C-optimization

I've been playing with Rust optimizations today and discovered something curious: it looks like LLVM `opt` does a better job at optimizing Rust IR than `rustc`:

``` sh
set -v

LINK=`rustc n_body.rs -O -o n_body-rustc -Zprint-link-args`

rustc n_body.rs --emit=llvm-ir -o n_body.ll #-O
opt n_body.ll -O3 -S -o n_body-opt.ll
#opt n_body-opt.ll -O3 -S -o n_body-opt.ll
llc n_body-opt.ll -relocation-model=pic -filetype=obj -o n_body-opt.o
eval ${LINK/n_body-rustc.0.o/n_body-opt.o} -o n_body-opt

time ./n_body-rustc 20000000
time ./n_body-opt 20000000

set +v
```

produces:

```
...
time ./n_body-rustc 20000000
-0.169075164
-0.169031665

real    0m3.876s
user    0m3.868s
sys 0m0.008s

time ./n_body-opt 20000000
-0.169075164
-0.169031665

real    0m3.626s
user    0m3.610s
sys 0m0.016s
```

(n_body.rs having been copied verbatim from [benchmarksgame](http://benchmarksgame.alioth.debian.org/u64q/program.php?test=nbody&lang=rust&id=1))  

Adding `-O` to the second `rustc` invocation and running IR through `opt` twice pushes the time down even further:

```
...
time ./n_body-opt 20000000
-0.169075164
-0.169031665

real    0m3.147s
user    0m3.135s
sys 0m0.012s
```

This happens on both stable 1.11.0  and the current nightly.  (x86_64-unknown-linux-gnu)

Does this mean that we could have `-C opt-level=4`?
Repository: go
Issue: cmd/compile: use common ancestor blocks in CSE?
Labels: Performance,compiler/runtime

``` go
package x

var p *int

func f(b bool) int {
    if b && *p == 0 {
        return 0
    }
    return *p
}
```

The load `*p` occurs regardless of the value of b. That is, the compiler should treat this as equivalent to:

``` go
package x

var p *int

func f(b bool) int {
    i := *p
    if b && i == 0 {
        return 0
    }
    return i
}
```

It does not. The latter generates better code.

The reason is that neither block containing `*p` is an ancestor of the other, so CSE ignores the `*p` values. However, they share a common ancestor in which all of the values' arguments are available.

This happened in practice in CL 22712. Perhaps CSE should be taught to recognize situations like this.

Potential downsides:
- This might move easily-recalculated values far from their uses, causing additional spills. (Aside: I think this is what went wrong with CL 28219. And yet making tighten more aggressive causes problems as well. We clearly need a more sophisticated approach in general to scheduling, so that we can pick up all these nickels without losing any dollar bills along the way.) 
- This might be computationally expensive, and CSE was long a problem child for performance.

cc @randall77 @dr2chase @aclements @bradfitz @tzneal 

If we fix this, we should consider reverting CL 22712.
Repository: TypeScript
Issue: compilerOptions.paths to support relative and absolute paths
Labels: Suggestion,Awaiting More Feedback

<!-- BUGS: Please use this template. -->

**TypeScript Version:**  2.0.2

**Code**

``` ts
{
  "compilerOptions": {
    "paths": {
      "./moment": ["node_modules/moment/moment.d.ts"],
      "/foo/bar/baz": [ "some/other/path/for/compiler"]
    }
  },
```

**Expected behavior:**
I expect (need) to be able to redirect the compiler for any module, not just 'non relative' paths. I'm deploying to a runtime environment that requires me to use relative or absolute paths, but I need the compiler to look to different locations when building locally.

**Actual behavior:** 
The compiler seems to ignore any `compilerOption.paths` setting which starts with `/` or `./`
Repository: flutter
Issue: Widget test with await  is hanged when not using `runAsync` instead of showing error/warning message 
Labels: a: tests,framework,a: quality,a: error message,has reproducible steps,P3,team-framework,triaged-framework,found in release: 3.16,found in release: 3.19

```
import "dart:async";

import "package:flutter_test/flutter_test.dart";

void main() {
  Future future;

  setUp(() {
    future = new Future.value();
  });

  testWidgets("awaiting future with value from setUp works",
      (WidgetTester tester) async {
    await future;
  });
}
```

```
flutter_test_async$ flutter test test/async_widget_test.dart 
00:05 +0 -1: - awaiting future with value from setUp works                                         
  TimeoutException after 0:00:05.000000: Test timed out after 5 seconds.
  dart:async                _StreamController.add
  websocket_impl.dart 1111  _WebSocketImpl._WebSocketImpl._fromSocket.<fn>
  dart:async                _EventSinkWrapper.add
  websocket_impl.dart 333   _WebSocketProtocolTransformer._messageFrameEnd
  websocket_impl.dart 228   _WebSocketProtocolTransformer.add

00:05 +0 -1: Some tests failed.        
```

```
[✓] Flutter (on Mac OS, channel unknown)
    • Flutter at /Users/drewwarren/flutter
    • Framework revision 852a00a17b (8 days ago), 2016-08-25 14:23:13
    • Engine revision c4022b61fa
    • Tools Dart version 1.19.0-dev.5.0

[x] Android toolchain - develop for Android devices
    x Android Studio / Android SDK not found. Download from https://developer.android.com/sdk/
      (or visit https://flutter.io/setup/#android-setup for detailed instructions).

[✓] iOS toolchain - develop for iOS devices (Xcode 7.3.1)
    • XCode at /Applications/Xcode.app/Contents/Developer
    • Xcode 7.3.1, Build version 7D1014

[✓] Atom - a lightweight development environment for Flutter
    • flutter plugin version 0.2.4
    • dartlang plugin version 0.6.37
```
Repository: godot
Issue: Auto-completion not working when using NodePath from exported variable.
Labels: enhancement,topic:gdscript,topic:editor,usability

When using:

```
export(NodePath) var node

func some_func():
    get_node(node).node_method()

```

Auto-completion in Script Editor is not working while when using get_node("node_path_to_node") it is.
Repository: rust
Issue: Code only compiles with try!, not with question mark syntax
Labels: C-enhancement,T-compiler,A-inference

The following code only compiles with `try!`, not when using the `?`-syntax instead: 

``` rust
#![feature(question_mark)]

fn foo() -> Result<Vec<u32>, std::num::ParseIntError> {
    let s = &["42", "1337"];

    let parsed: Vec<u32> = try!(s.iter().map(|str| {
        let val = str.parse::<u32>()?;
        Ok(val + 1)
    }).collect());

    Ok(parsed)
}

fn main() {
    println!("{:?}", foo());
}
```

Replacing the single `try!` in function `foo` with its `?` equivalent produces the following error:

```
error[E0284]: type annotations required: cannot resolve `<_ as std::ops::Carrier>::Success == _`
 --> <anon>:6:28
  |
6 |     let parsed: Vec<u32> = s.iter().map(|str| {
  |                            ^

error: aborting due to previous error
```

Example on playground: https://is.gd/CPUV0f (replace `try!` with `?` to see the error)

Rustc version: rustc 1.13.0-nightly (acd3f796d 2016-08-28)

Note: @KiChjang mentioned on IRC that using turbofish on the collect call, `.collect::<Result<_,_>>()` , makes the `?` version compile again.
Repository: rust
Issue: [rustbuild] cross-compiling to windows-gnu fails
Labels: T-bootstrap,C-bug,S-needs-info

```
$ python2 src/bootstrap/bootstrap.py --stage 1 --step libstd -j8 --target=i686-pc-windows-gnu
    Finished debug [unoptimized + debuginfo] target(s) in 0.0 secs
Building stage0 std artifacts (x86_64-unknown-linux-gnu -> x86_64-unknown-linux-gnu)
    Finished release [optimized] target(s) in 0.0 secs
Building stage0 test artifacts (x86_64-unknown-linux-gnu -> x86_64-unknown-linux-gnu)
    Finished release [optimized] target(s) in 0.0 secs
Building stage0 compiler artifacts (x86_64-unknown-linux-gnu -> x86_64-unknown-linux-gnu)
    Finished release [optimized] target(s) in 0.0 secs
Building stage1 std artifacts (x86_64-unknown-linux-gnu -> i686-pc-windows-gnu)
thread 'main' panicked at 'failed to copy `crt2.o` to `/home/nagisa/Documents/rust/rust/worktrees/more-cace/build/x86_64-unknown-linux-gnu/stage1/lib/rustlib/i686-pc-windows-gnu/lib/crt2.o`: the source path is not an existing regular file', src/bootstrap/util.rs:46
```
Repository: flutter
Issue: Material "Durations" class?
Labels: c: new feature,framework,f: material design,P3,team-design,triaged-design

Not a bug, just a suggestion! Happy to help implement it as well, if it's something you would be interested in :)

I really enjoy the `Icons`, `Colors`, etc classes available in the `material` package. They helped me bust out a quick, decent looking prototype in no time at all!

I also started using the `AnimatedOpacity` class to perform a couple of transitions, which required a `Duration` parameter, and wondered: "Oh yes, what exactly are the correct Material durations?" 

Luckily, the material specs are well documented here: https://material.google.com/motion/duration-easing.html with the specs. I created a small helper class in my app, but wondered if they could be codified as a class in the `Material` package?

Some constants already exist in the Material package (https://docs.flutter.io/flutter/material/material-library.html), but the material guidelines require extending the animation duration by 30% for Tablets, for example.

Would it make sense to have another class, such as `Durations`, with methods that take something like a `Size` (preferably something better), and return the correct duration?

This could look something like:

``` dart
class Durations {
  static final kStandardMobileDurationInMs = 300;
  static final kEnterScreenMobileDurationInMs = 225;
  static final kExitScreenMobileDurationInMs = 195;

  // Size is really not great, but is just an example! I would imagine another class that would help distinguish between tablets / wearables, and that value could get passed into here instead
  static Duration standardDuration({ Size screenSize }) {
    return new Duration(
      milliseconds:
      (kStandardMobileDurationInMs * screenSizeModifier(screenSize)).round()
    );
  }

  static double screenSizeModifier(Size screenSize) {
    if (screenSize == null) {
      return 1.0;
    } else if (isLargeScreen(screenSize)) {
      return 1.3;
    } else if (isSmallScreen(screenSize)) {
      return 0.7;
    }

    return 1.0;
  }

  static bool isSmallScreen(Size screenSize) {
    return screenSize.width < 320 || screenSize.height < 320;
  }

  static bool isLargeScreen(Size screenSize) {
    return screenSize.width > 500 || screenSize.height > 500;
  }

  // Pseudo-code / not implemented, but extra durations from the spec as an example
  static Duration complexFullScreenDuration();
  static Duration enterScreenDuration();
  static Duration exitScreenDuration();
}
```

Let me know whatcha think!

p.s. The fact that `Size` exists as a first class citizen in `dart:ui` is really cool. The more I play with this language and the libs, the more I'm impressed by the little things :)
Repository: rust
Issue: Specialization influnces inference
Labels: A-trait-system,T-compiler,A-specialization,A-inference,C-bug,requires-nightly,F-specialization

## STR

``` Rust
#![feature(specialization)]

struct My<T>(T);

trait Conv<T> {
    fn conv(self) -> T;
}

impl<T> Conv<T> for My<T> {
    default fn conv(self) -> T { self.0 }
}

#[cfg(broken)]
impl Conv<u32> for My<u32> {
    default fn conv(self) -> u32 { self.0 }
}

fn main() {
    let x = My(0);
    x.conv() + 0i32;
}
```
## Expected Result

Adding a specialized impl will not affect type inference.
## Actual Result

After adding the specialized impl, inference is guided to take it:

```
error[E0308]: mismatched types
  --> <anon>:20:16
   |
20 |     x.conv() + 0i32;
   |                ^^^^ expected u32, found i32

error[E0277]: the trait bound `u32: std::ops::Add<i32>` is not satisfied
  --> <anon>:20:5
   |
20 |     x.conv() + 0i32;
   |     ^^^^^^^^^^^^^^^ trait `u32: std::ops::Add<i32>` not satisfied
   |
   = help: the following implementations were found:
   = help:   <u32 as std::ops::Add>
   = help:   <&'a u32 as std::ops::Add<u32>>
   = help:   <u32 as std::ops::Add<&'a u32>>
   = help:   <&'b u32 as std::ops::Add<&'a u32>>

error: aborting due to 2 previous errors
```

cc @aturon @nikomatsakis
Repository: opencv
Issue: bring back cv::convertScale
Labels: feature,category: python bindings,affected: 3.4

It was replaced with `Mat::convertTo` which is not accessible from the python bindings.
Repository: youtube-dl
Issue: Site request: (doc.)rt.com
Labels: site-support-request

Please add rt.com (and/or doc.rt.com and other subdomains) to youtube-DL.
Sample link is here: https://doc.rt.com/filmy/chernij-oktyabr-93-go/
The log of (not)downloading:
C:>youtube-dl.exe -F -v https://doc.rt.com/filmy/chernij-oktyabr-93-go/
[debug] System config: []
[debug] User config: []
[debug] Command-line args: ['-F', '-v', 'https://doc.rt.com/filmy/chernij-oktyabr-93-go/']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2016.09.04.1
[debug] Python version 3.4.4 - Windows-7-6.1.7601-SP1
[debug] exe versions: ffmpeg N-81391-g2a3720b, ffprobe N-81391-g2a3720b, rtmpdump 2.4
[debug] Proxy map: {}
[generic] chernij-oktyabr-93-go: Requesting header
WARNING: Falling back on generic information extractor.
[generic] chernij-oktyabr-93-go: Downloading webpage
[generic] chernij-oktyabr-93-go: Extracting information
[info] Available formats for playlist:
format code  extension  resolution note
## 0            xml        unknown

Please notice, rt.com offers several resolutions to viewer, here is the pic: http://i.imgur.com/CKnrGzJ.png
Thanks in advance!
Repository: You-Dont-Know-JS
Issue: Types & Grammar Ch 3 - RegExp.prototype is an object as of ES6
Labels: for second edition

The code below throws an error because RegExp.prototype is an ordinary object as of ES6.  I think it would make sense to either change the example or note that this code won't work anymore if no regex is passed in for rx. 

```
function isThisCool(vals,fn,rx) {
    console.log(vals, fn, rx)
    vals = vals || Array.prototype;
    fn = fn || Function.prototype;
    rx = rx || RegExp.prototype;
    return rx.test(
        vals.map( fn ).join( "" )
    );
}

isThisCool();       // true
```

I did find a workaround via `RegExp.toPrototype.toString()`, but I'm assuming that it would make more sense to just type `rx || /(?:)/`in line 4 to get the result you want. Here's my code in case it helps:

```
function isThisCool(vals,fn,rx) {
    vals = vals || Array.prototype;
    fn = fn || Function.prototype;
    rx = rx || RegExp.prototype.toString();
    rx = new RegExp(rx.slice(1,-1));
    return rx.test(
        vals.map( fn ).join( "" )
    );
}

isThisCool();       // true
```
Repository: rust
Issue: Windows Defender goes nuts when running rustdoc on some projects
Labels: O-windows,T-rustdoc,C-enhancement

`rustdoc` gets pretty much completely stalled on some projects by Windows Defender on Windows 10. `rustdoc` cpu usage drops to 0%, while `MsMpEng.exe` uses 100% on a single core. This issue seems to have popped up only a few weeks ago.
Repository: three.js
Issue: IES Light Support
Labels: Enhancement,Bounty

##### Description of the problem

IES lights are a standard for specifying the illumination intensity in a specific direction from a light.  It allows one to use IES profiles to specify real-world light bulbs in a scene and get perfect results.  It is used all the time for architectural rendering.  We should try to get this into THreeJS at some point.  I understand the way that you do this is to create a spherical map of intensities for each light type (e.g. IES profile) and then sample from it.

http://www.cgarena.com/freestuff/tutorials/max/ieslights/

I believe this requires then one cubemap per IES profile light.  It isn't that slow so it is suitable for real-time on all devices.
##### Three.js version
- [x] Dev
- [x] r80
- [ ] ...
##### Browser
- [x] All of them
- [ ] Chrome
- [ ] Firefox
- [ ] Internet Explorer
##### OS
- [x] All of them
- [ ] Windows
- [ ] Linux
- [ ] Android
- [ ] IOS
##### Hardware Requirements (graphics card, VR Device, ...)
Repository: youtube-dl
Issue: [facebook] Add support for user videos
Labels: request

Hello, is it possible to recover all the videos from a facebook page? It is already possible to do this on Youtube and Dailymotion.

Youtube-dl already suppporte downloading and retrieval of information on a single video.
The videos on facebook link are:
- https://www.facebook.com/USERNAME/videos/
- https://m.facebook.com/MisterVOnline/videos/
- https://touch.facebook.com/MisterVOnline/videos/

thank you in advance