sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: TypeScript
Issue: TSX related formatting bugs
Labels: Bug,Help Wanted,Domain: Formatter

**TypeScript Version:**  2.0 beta

**Code**

`|` is indicating cursor here.

``` ts
<Scene key="root"
| // cursor should be indented so that attribute definitions can be naturally indented
```

``` ts
<Scene key="root">
| // cursor should be indented, as cursor on unfinished array does
```

``` ts
<Scene key     =      "root" /> // formatting does not work
// PS: this is fixed by #10054
```

**Expected behavior:**

``` ts
<Scene key="root"
    |
```

``` ts
<Scene key="root">
    |
```

``` ts
<Scene key="root" />
// PS: this is fixed by #10054
```
Repository: angular
Issue: Problem when adding dynamically a component with g[test] selector in SVG
Labels: type: bug/fix,freq2: medium,area: core,state: confirmed,core: dynamic view creation,cross-cutting: SVG,P3,bug

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

[ x ] bug report
[ ] 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**

I try to add dynamically this component within SVG:

```
@Component({
  selector: 'g[test1]',
  template: `
    <svg:text x="10" y="15" fill="red">I love SVG!</svg:text>
  `
})
export class Test1Comp {
}
```

I use this approach:

```
this.resolver.resolveComponent(Test1Comp).then((factory:ComponentFactory<any>) => {
    this.cmpRef = this.viewContainerRef.createComponent(factory);
});
```

Whereas the corresponding content is added to the DOM, it appears that the host element (`g`) isn't a valid SVG element (perhaps the svg namespace isn't used):

```
<g test1> <!-- UnkownHTMLElement-->  
  <text>...</text> <!-- SVGTextElement -->
</g>
```

This prevents from displaying the SVG elements within the `g` one.

**Expected/desired behavior**

To be able to add dynamically components within SVG with selectors like `g[test1]`.

**Reproduction of the problem**

See this plunkr: http://plnkr.co/edit/6aLGnLNi0svLk4CVS9M2?p=preview.

**What is the expected behavior?**

I expect the added host element to be an `SVGElement` and not an `HTMLUnknownElement`.

**What is the motivation / use case for changing the behavior?**

**Please tell us about your environment:**
- **Angular version:** 2.0.0-rc.4
- **Browser:** [Chrome] 
- **Language:** [ TypeScript ]
Repository: godot
Issue: Improve tool script debugging
Labels: enhancement,topic:editor,usability

I am currently developping a GDScript editor plugin, and just like developping a game, I make mistakes and am getting countless errors, yay!
But the problem here is that I only get the error and the line where it occurred. I am currently spending a lot of time trying to figure out a bug happening everytime I use Ctrl+S that makes the editor hang for 30 seconds (it was 5 minutes when it first occurred!). Because I edit the tool in the editor, it is becoming a PITA to debug. Godot even froze at some point but not a clue what happened.

It would be a great improvement if we could get the call stack of tool script errors, because mine just tells `invalid index in get_value(x,y)`, which can happen in a lot of different ways, so I'm screwed to the point I have to print everywhere (making the test even longer!), erase stuff and wait for a while everytime just to find the culprit code path.

FYI: the bug was caused by https://github.com/godotengine/godot/issues/5984
Repository: rust
Issue: Test runner interacts badly with redirected child process
Labels: C-enhancement,A-libtest,A-process

(Moved here from rust-lang/cargo/2936 as it is believed to be a standard library bug.)

Consider the code at https://gitlab.com/BartMassey/ptyknot/tree/pipes-direct/misc/piperef-rs . The relevant portion is in piperef.rs:

```
        ...
       // Write "hello world" to stdout.                                       
        match write_mode {
            WriteMode::Macro => {println!("hello world");},
            WriteMode::C => {
                let buf = "hello world".as_bytes();
                let hello = std::ffi::CString::new(buf).unwrap();
                let bufptr = hello.as_ptr() as *const libc::c_void;
                check_cint!(libc::write(1, bufptr, buf.len()));
            }
        }
         ....
```

This code is run in a child process. When it is invoked by running it after `cargo build`, it works fine regardless of whether Macro or C mode is used. When it is invoked by `cargo test`, C mode works but Macro mode fails with an empty string. When it is invoked by `cargo test -- --nocapture`, both modes work again. Here's a transcript:

```
$ cargo build
   Compiling libc v0.2.14
   Compiling piperef v0.1.0 (file:///usr/local/src/ptyknot/misc/piperef-rs)
$ target/debug/piperef 
$ cargo test
   Compiling piperef v0.1.0 (file:///usr/local/src/ptyknot/misc/piperef-rs)
     Running target/debug/piperef-80efeb997239b2ac

running 2 tests
test write_macro_test ... FAILED$<2>
test write_c_test ... ok$<2>

failures:

---- write_macro_test stdout ----
    thread 'write_macro_test' panicked at 'assertion failed: `(left == right)` (left: `""`, right: `"hello world"`)', piperef.rs:60
note: Run with `RUST_BACKTRACE=1` for a backtrace.


failures:
    write_macro_test

test result: FAILED$<2>. 1 passed; 1 failed; 0 ignored; 0 measured

error: test failed
$ cargo test -- --nocapture
     Running target/debug/piperef-80efeb997239b2ac

running 2 tests
test write_macro_test ... ok
test write_c_test ... ok

test result: ok. 2 passed; 0 failed; 0 ignored; 0 measured

$ 
```

It looks like the test runner is fouling up the environment somehow, but I can't figure out how by looking at stuff.
Repository: bitcoin
Issue: Separate resource usage profiles
Labels: Feature,Resource usage

There has been talk before (and I'm surprised to not find an existing issue about it) of having different memory usage profiles. This could encompass defaults for:
- `-dbcache`
- `-par`
- `-maxsigcachesize`
- `-maxmempool`
- `-maxorphantx`
- `-maxconnections`
- `-maxreceivebuffer`
- `-maxsendbuffer`

For desktop systems, autodetecting could be used to determine optimal settings. For servers, it could be specified on the command line. I have heard about interest in a means for specifying "Just use all the memory I have".
Repository: react
Issue: Attach Stack at setState Calls in DEV Mode
Labels: Type: Enhancement,Component: Core Utilities,React Core Team

We have a queue for state transitions. There is a natural place for a `new Error()` stack frame to be stored there.

We could use this information to work our way back from errors to show information about which `setState` call was the source of the error.

Because of batching, it is not always possible to make the correct attribution. It could be one of several possible sources.

(Specifically I'd like to use this in Fiber when the starvation protection kicks in. We can show which state transition was kept getting deferred due to higher priority work.)
Repository: rust
Issue: type_id is not crate-independent with object associated types
Labels: A-trait-system,E-needs-test,T-compiler,C-bug

## Meta

```
$ rustc -V
rustc 1.12.0-dev (5556554e2 2016-07-31)
```
## STR

``` Rust
#![feature(core_intrinsics, rustc_private)]

#[cfg(arena_first)]
extern crate arena;
extern crate term;
#[cfg(not(arena_first))]
extern crate arena;

use std::intrinsics::type_id;

fn main() { unsafe {
    println!("same={:?} different={:?}",
        type_id::<arena::TypedArena<()>>(),
        type_id::<Iterator<Item=arena::TypedArena<()>>>()
    );
}}  
```

Any pair of crates, one of them with a struct, can be used instead of arena/term.
## Expected Result

The Type ID of `Iterator<Item=arena::TypedArena<()>>` should be the same when the cfg is toggled.
## Actual Result

The Type ID of `Iterator<Item=arena::TypedArena<()>>` differs when the cfg is toggled, but the id of `arena::TypedArena<()>` does not
Repository: rust
Issue: Verify that the AST does not have DUMMY_SP
Labels: A-debuginfo,C-enhancement,T-compiler,A-proc-macros

Currently, syntax extensions can generate code with `DUMMY_SP` in its AST. All compiler diagnostics from that code, and all stack backtraces generated by that code, are basically worthless.

That should not be possible. The lowered HIR should not have any DUMMY_SP.

cc @rust-lang/compiler
Repository: go
Issue: fmt: Scanf EOF error inconsistency
Labels: NeedsFix

Please answer these questions before submitting your issue. Thanks!

_What version of Go are you using (`go version`)?_

`go version devel +ff227b8 Thu Jul 21 01:04:22 2016 +0000 linux/amd64`
but older versions of Go are also affected.

_What operating system and processor architecture are you using (`go env`)?_

``` text
GOARCH="amd64"
GOBIN="/home/fjl/bin"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/fjl/"
GORACE=""
GOROOT="/usr/lib/go"
GOTOOLDIR="/usr/lib/go/pkg/tool/linux_amd64"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build153025883=/tmp/go-build -gno-record-gcc-switches"
CXX="g++"
CGO_ENABLED="1"
```

_What did you do?_

I use fmt.Sscanf to parse integers and big integers.

https://play.golang.org/p/aZVlPM7haY

_What did you expect to see?_

When there is not enough input Sscanf (and Fscanf, Scanf) should return io.ErrUnexpectedEOF

_What did you see instead?_

It returns io.ErrUnexpectedEOF for big.Int and io.EOF for uint.
Repository: TypeScript
Issue: Suggestion: output fewer errors for incorrectly implemented interface
Labels: Suggestion,Help Wanted,Effort: Difficult

**TypeScript Version:**  nightly

**Code**

``` ts
interface A {
    f(): void;
}
class B implements A {
    eff() {}
}

var b: B;
b.f();
b.f();
b.f();
```

**Expected behavior:**

```
src/a.ts(4,7): error TS2420: Class 'B' incorrectly implements interface 'A'.
  Property 'f' is missing in type 'B'.
```

**Actual behavior:** 

```
src/a.ts(4,7): error TS2420: Class 'B' incorrectly implements interface 'A'.
  Property 'f' is missing in type 'B'.
src/a.ts(9,3): error TS2339: Property 'f' does not exist on type 'B'.
src/a.ts(10,3): error TS2339: Property 'f' does not exist on type 'B'.
src/a.ts(11,3): error TS2339: Property 'f' does not exist on type 'B'.
```

There can be an overwhelming number of missing-property errors if a commonly-used class mis-implements an interface it declares.
Repository: rust
Issue: unsatisfyable HKL error message prints broken bound (`for<'b> 'b : `)
Labels: A-diagnostics,T-compiler,C-bug

the full error is

```
error[E0279]: the requirement `for<'b> 'b : ` is not satisfied (`expected bound lifetime parameter 'b, found concrete lifetime`)
 --> <anon>:2:5
  |
2 |     test::<FooS>(&mut 42);
  |     ^^^^^^^^^^^^
  |
  = note: required because of the requirements on the impl of `for<'b> Foo<'b>` for `FooS<'_>`
  = note: required by `test`
```

reproducible example on stable, beta and nightly:

``` rust
fn main() {
    test::<FooS>(&mut 42);
}

trait Foo<'a> {}

struct FooS<'a> {
    data: &'a mut u32,
}

impl<'a, 'b: 'a> Foo<'b> for FooS<'a> {}

fn test<'a, F>(data: &'a mut u32) where F: for<'b> Foo<'b> {}
```
Repository: opencv
Issue: loading 32 bit TIFF requires imread flags to be set
Labels: bug,category: imgcodecs

According to the documentation flag `IMREAD_ANYDEPTH` 

> If set, return 16-bit/32-bit image when the input has the corresponding depth, otherwise convert it to 8-bit

When reading 32bit TIFF image without any flags the image should load and get converted to 8bit. In practice however image is empty.
When flag `IMREAD_ANYDEPTH` is set the image loads correctly as a 32bit float.
Repository: TypeScript
Issue: Inconsistent DocComment merging for merged declarations
Labels: Bug,Help Wanted,VS Code Tracked

_From @unional on July 14, 2016 7:17_

Likely tsc error?
VSCode: 1.3.1
ts: 2.1.0-dev.20160712

``` ts
/**
 * some function
 */
function foo() {}

/**
 * some interface
 */
interface foo {}

/**
 * some namespace
 */
namespace foo {}
```

![image](https://cloud.githubusercontent.com/assets/3254987/16831061/bf2d9fb8-4957-11e6-96aa-3a812926abe0.png)

It works like this when it is variable:

``` ts
/**
 * some variable
 */
let foo: any;

/**
 * some interface
 */
interface foo {}

/**
 * some namespace
 */
namespace foo {}
```

![image](https://cloud.githubusercontent.com/assets/3254987/16831159/36418826-4958-11e6-8b98-0690f31767b9.png)

![image](https://cloud.githubusercontent.com/assets/3254987/16831083/da1d098a-4957-11e6-874c-51e4c0c4a364.png)

The highlighting is also wrong for interface, and interface is hidden when hovering variable and namespace.

_Copied from original issue: Microsoft/vscode#9254_
Repository: electron
Issue: Chrome Web Push support
Labels: enhancement :sparkles:,component/chrome-api

I came across [this previous issue](https://github.com/electron/electron/issues/3095), where it was said that Chrome Push would not be supported at the time since Google's private services are not included in Electron's content bundle. Now that Chrome 52 [supports a non-proprietary protocol](http://blog.chromium.org/2016/06/chrome-52-beta-css-containment-simpler.html), would Electron eventually support web push notifications?
Repository: rust
Issue: Tracking issue for future-incompatibility lint `patterns_in_fns_without_body`
Labels: A-lints,T-lang,T-compiler,C-future-incompatibility,C-tracking-issue

Code using patterns in parameters of foreign functions, function pointers or trait methods without body is not accepted by the compiler.

```rs
extern {
    fn f((a, b): (u8, u16)); // ERROR
}

type T = fn((a, b): (u8, u16)); // ERROR

trait Tr {
    fn f((a, b): (u8, u16)); // ERROR
}
```

Previously some simple patterns like `&ident`, `&&ident` or `mut ident` were allowed in these positions, now they aren't. Such patterns weren't properly type checked, e.g. `type A = fn(&arg: u8);` compiled despite `u8` not being a reference.
#### What are these errors for?

Full patterns don't make sense in functions without bodies, but simple identifiers may be useful for documenting argument purposes even if they aren't actually used - `type Callback = fn(useful_name: u8)`.
By restricting patterns in body-less function signatures to `ident: TYPE` we can make argument names optional and accept simply a `TYPE` in argument position (`type T = fn(u8)`) without introducing parsing ambiguities.
#### How to fix this warning/error

Remove `&` or `mut` from the pattern and make the function parameter a single identifier or `_`.
#### Current status
- [x] https://github.com/rust-lang/rust/pull/35015 made patterns in foreign functions and function pointers hard errors
- [x] https://github.com/rust-lang/rust/pull/37378 made patterns in trait methods without body warn-by-default lints
- [x] https://github.com/rust-lang/rust/pull/45775 made patterns in trait methods without body  hard errors, except for `mut IDENT`
- [x] https://github.com/rust-lang/rust/pull/65785 made patterns in trait methods without body deny-by-default lints
- [ ] ? made patterns in trait methods without body hard errors
- [x] https://github.com/rust-lang/rust/issues/78927 provide structured suggestions
Repository: flutter
Issue: determinate circular progress indicators have wrong animation
Labels: framework,f: material design,P2,team-design,triaged-design

See: https://material-design.storage.googleapis.com/publish/material_v_8/material_ext_publish/0B14F_FSUCc01N2kzc1hlaFR5WlU/061101_Circular_Sheet_xhdpi_005.webm
Repository: flutter
Issue: Improve transition from indeterminate to determinate circular progress indicators
Labels: framework,f: material design,a: quality,has reproducible steps,P2,found in release: 3.7,found in release: 3.9,team-design,triaged-design

We should transition cleanly. Right now we just switch modes with a sudden change.
Repository: go
Issue: x/net/http2: rename and export Transport's t1 field
Labels: NeedsFix

Please answer these questions before submitting your issue. Thanks!
1. What version of Go are you using (`go version`)?
   
   go version go1.7rc4 darwin/amd64
2. What operating system and processor architecture are you using (`go env`)?
   
   GOARCH="amd64"
   GOHOSTARCH="amd64"
   GOHOSTOS="darwin"
   GOOS="darwin"
3. What did you do?
   
   Directly use an `http2.Transport` to set its `AllowHTTP` field to true, to enable HTTP/2 over HTTP (and set DialTLS accordingly, as mentioned here: https://github.com/golang/go/issues/14141#issuecomment-219212895).
4. What did you expect to see?
   
   Have access to the same rich set of configuration options as are available on `http.Transport`.
5. What did you see instead?
   
   Many configuration options from `http.Transport` are not available on `http2.Transport` (e.g. controlling idle connections in the ClientConnPool).

As discussed on golang-dev mailing list (https://groups.google.com/forum/#!topic/golang-dev/HmqDsLpTywk), as it currently stands, if we have to directly use the `http2.Transport`, we have to give up on a number of configuration options that are present only on `http.Transport`. @bradfitz suggested to rename and export the `http2.Transport.t1` field to fix this.
Repository: youtube-dl
Issue: count of videos when giving playlist-start should number downloads starting from given number
Labels: request

### Make sure you are using the _latest_ version: run `youtube-dl --version` and ensure your version is _2016.08.01_. If it's not read [this FAQ entry](https://github.com/rg3/youtube-dl/blob/master/README.md#how-do-i-update-youtube-dl) and update. Issues with outdated version will be rejected.
- [x] I've **verified** and **I assure** that I'm running youtube-dl **2016.02.22**
### Before submitting an _issue_ make sure you have:
- [x] At least skimmed through [README](https://github.com/rg3/youtube-dl/blob/master/README.md) and **most notably** [FAQ](https://github.com/rg3/youtube-dl#faq) and [BUGS](https://github.com/rg3/youtube-dl#bugs) sections
- [x] [Searched](https://github.com/rg3/youtube-dl/search?type=Issues) the bugtracker for similar issues including closed ones
### What is the purpose of your _issue_?
- [x] Feature request (request for a new functionality)

---
### Description of your _issue_, suggested solution and other information

When I start a download from a particular --playlist-start value (say --playlist-start 15), the numbering shown is like "downloading 1 of 70".  It would help to have it also display the added value (like downloading 15 of 84, 16 of 84, etc.).  If the download breaks or has to be restarted from a point, we know what number to give next.
Repository: vscode
Issue: Git - Support git-bash ssh agent for Windows
Labels: help wanted,feature-request,git

cc @saggafarsyad

From #6202
Repository: javascript
Issue: The use of || operator in assignments
Labels: question

In the topic 15.7 this kind of assignment is favored over ternaries: `const foo = a || b;`. Actually, this pattern is used in a lot of the examples, but should it be discouraged since it can produce undesired behavior when the left operand is falsy? In the above example if `a` is falsy so `b` would be the result.

I know the developer should distinguish problematic cases, but one of the goals of the style guide is to avoid potential issues.
Repository: kubernetes
Issue: Bulk deletes are resource intensive on caches.
Labels: sig/scalability,area/apiserver,sig/api-machinery,lifecycle/frozen

We have a series of tests that fill a cluster, settle, then bulk delete using namespaces.  

This bulk delete is very memory intensive and expensive on the api-server.  The profile of the operation shows a large spike in memory, which is likely due to how we cache. 

![image](https://cloud.githubusercontent.com/assets/169553/17381654/98bceb7c-5992-11e6-8431-719c2379049c.png)

By contrast etcd profile looks like: 

![image](https://cloud.githubusercontent.com/assets/169553/17381688/c10a3e9a-5992-11e6-8426-32fdd325ce75.png)

If there are overlapping bulk operations on larger clusters, this could be very expensive. 
A good example would be bulk job creation and cleanup (e.g. builds). 

/cc @kubernetes/sig-scalability @smarterclayton @deads2k
Repository: kubernetes
Issue: Add support for custom stop signal to send to containers
Labels: area/kubelet,area/api,sig/node,kind/feature,lifecycle/frozen,area/pod-lifecycle

Docker added STOPSIGNAL to let container images change which signal is delivered to kill the container.  Should we support that?  It would be bad if an image expected SIGUSR1 and we sent SIGTERM.

They support ints or strings.  I think we should only support strings.
Repository: TypeScript
Issue: Formatter fails to format uglified files
Labels: Suggestion,Help Wanted,Domain: Formatter,VS Code Tracked

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

<!-- QUESTIONS: This is not a general support forum! Ask Qs at http://stackoverflow.com/questions/tagged/typescript -->

<!-- SUGGESTIONS: See https://github.com/Microsoft/TypeScript-wiki/blob/master/Writing-Good-Design-Proposals.md -->

**TypeScript Version:**  1.8.10

I use vscode from the Command Palette to format a javascript file uglified by uglify-js, but it failed. It not format code correctly, and only format partially.

**source file**
![2016-07-12 4 47 08](https://cloud.githubusercontent.com/assets/2631733/16760863/5b438ccc-4850-11e6-87de-4c2475b08d60.png)

**sublime jsFormat formated**
![2016-07-12 4 47 29](https://cloud.githubusercontent.com/assets/2631733/16760887/69fc490c-4850-11e6-9279-2b79e4f6c87f.png)

**vscode formated**
![2016-07-12 4 47 45](https://cloud.githubusercontent.com/assets/2631733/16760892/72414f4a-4850-11e6-9ce5-72e2633bb317.png)

You see, It seems like, vscode only add some spaces around brackets, letters, commas etc.

See https://github.com/Microsoft/vscode/issues/8914#issuecomment-231978337 and the discussion below for more details. The provided traces clearly show that the tsserver only returns white space (" ") changes. No insertion of new lines.
Repository: go
Issue: x/tools/cmd/goimports: can't find proto packages
Labels: NeedsInvestigation

Please answer these questions before submitting your issue. Thanks!
1. What version of Go are you using (`go version`)?

`go version go1.6.2 darwin/amd64`
1. What operating system and processor architecture are you using (`go env`)?

```
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/danielchatfield"
GORACE=""
GOROOT="/usr/local/Cellar/go/1.6.2/libexec"
GOTOOLDIR="/usr/local/Cellar/go/1.6.2/libexec/pkg/tool/darwin_amd64"
GO15VENDOREXPERIMENT="1"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fno-common"
CXX="clang++"
CGO_ENABLED="1"
```

Package `fooproto` in `github.com/danielchatfield/repo/foo/proto`

Running `goimports` on the following fails to find the import.

```
package main

func main() {
    _ := &fooproto.SomeStruct{}
}
```

The first commit that broke this is `https://github.com/golang/tools/commit/e047ae774b027ae958a270598c5ac2591e457afc`
Repository: kubernetes
Issue: Storage should not allow multiple users of a PVclaim, if the claim is RWO, even if the PV is RWX
Labels: priority/important-soon,area/kubelet,sig/storage,lifecycle/frozen

This is hard to validate, but it is a bug.  We need to clarify the meaning of a claim's mode.  Today it means "at least" but I think we also need to express "this pod demands exclusive access" vs "shared access is OK".  That is more of an app-level semantic than infrastructure.

Concretely, assume my app needs a RWO volume.  I get bound to an NFS share that is RWX (which is "greater than" RWO).  My app _does not_ tolerate multiple writers to the volume.  It is possible to schedule another pod against the claim, and have it succeed since the PV is RWX underneath.  Now my app is corrupted.

This pattern could maybe apply to RWO PVs, too, if the pods land on the same machine, but that is a more subtle discussion.