sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: cmd/compile: review compiler-generated symbol naming scheme
Labels: help wanted,NeedsInvestigation,compiler/runtime

cmd/compile generates a lot of linker symbols for internal objects, and we've amassed a number of ad hoc naming schemes for them. I'm concerned that there might be collisions, particularly with user-defined symbols.

We should review and probably catalog the existing symbol, and try to find a more uniform naming scheme that we can ensure is collision-free.
Repository: kubernetes
Issue: Support adding an init process to containers
Labels: area/kubelet,sig/node,kind/feature,needs-triage

<!-- Please only use this template for submitting enhancement requests -->

**What would you like to be added**:
Add support for adding an init process (PID 1) to containers started by Kubernetes, across the supported container runtimes. This should be toggled by an appropriate flag on the `Container` spec.

A possible implementation is basically bind mounting a statically built init executable (e.g. `tini`) into the container and passing the containers entrypoint+cmd to it, which I think is what Docker's implementation of this does. Of course, the exact place to prepend the init command depends on the runtime, with Docker having its entrypoint/cmd separation and others possibly having different models.

**Why is this needed**:
Linux containers have a well known problem where due to the first process started in the container having PID 1, that process assumes the roles and responsibilities of `init`, and as such, has the default signal handling behavior disabled, unless they establish signal handles, and in addition have zombies reparented to them for reaping.

Well behaved Docker images take this into account and either: 
* Handle such details in the image's main process, that is, handle signals (depending on language/runtime/framework if this is needed), and reap zombies or otherwise don't create them by waiting appropriately. 
* Embed some minimal init into the image to handle this for them. Such as: `tini`, `dumb-init`, etc.

Sadly, there are plenty of badly behaving container images out there, and there are also plenty of use cases where requiring the user to add an init to the image is a problem. For example, systems where users submit jobs using their own custom images and commands, which might be less attentive to such issues.

To help with this, Docker added a flag that adds an init process backed by `tini` (Compiled as a vendored static executable named `docker-init) to its API. As far as I can tell, this is implemented by simply bind mounting this binary read only to the container and passing the command that would otherwise be executed as PID 1 to it.

This is of course Docker specific and for something like this to exist in Kubernetes it likely needs to be re-implemented by Kubernetes for all supported container runtimes. Though assuming you can bind mount such a binary in all runtimes, and can distribute it, it should be possible... I think...

Note that `tini`, specifically used by Docker, also supports some extra features that can be configured via environment variables that can sometimes be useful for certain kinds of containerizied workloads. Whether that's a good or a bad thing...

Of course, this entire thing is quite up for debate...
Repository: go
Issue: net: get npcap usable windows network device names 
Labels: OS-Windows,NeedsInvestigation

net.Interfaces(), net.InterfaceByName(), and net.InterfaceByIndex() return an Interface struct with a Name variable. On Linux, this will tend to be the name (eth0, wlan0, en0, etc) that can be passed to pcap functions/methods (eg. "github.com/google/gopacket/pcap".OpenLive() )

However, that isn't the case on Windows 10.

On my box net.InterfaceByIndex(5), for example returns a Interface.Name "Ethernet" and what I need for pcap.OpenLive() is "\Device\NPF_{13044533-0543-4AF5-9E3C-85EBBC7C04BB}"

gopacket and gopacket/pcap are really promising. ~as simple as scappy, and ~as fast as c, and I can write it once and run it on Windows and Linux. Except, not quite yet.
Repository: rust
Issue: rustdoc: document fields exposed by `Deref` implementations.
Labels: T-rustdoc,C-enhancement,A-rustdoc-ui

`rustdoc` provides a very useful "methods from `Deref<Target=...>`" section for types that implement the `Deref` trait, but this section only displays associated functions and never fields. It could be very useful if (public) fields exposed via `Deref` were also documented.
Repository: TypeScript
Issue: Incorrect or broken autocomplete suggestions for enum indexed interfaces
Labels: Bug,Domain: Completion Lists

<!-- 🚨 STOP 🚨 𝗦𝗧𝗢𝗣 🚨 𝑺𝑻𝑶𝑷 🚨

Half of all issues filed here are duplicates, answered in the FAQ, or not appropriate for the bug tracker. Even if you think you've found a *bug*, please read the FAQ first, especially the Common "Bugs" That Aren't Bugs section!

Please help us by doing the following steps before logging an issue:
  * Search: https://github.com/Microsoft/TypeScript/search?type=Issues
  * Read the FAQ: https://github.com/Microsoft/TypeScript/wiki/FAQ

Please fill in the *entire* template below.
-->

<!--
Please try to reproduce the issue with the latest published version. It may have already been fixed.
For npm: `typescript@next`
This is also the 'Nightly' version in the playground: http://www.typescriptlang.org/play/?ts=Nightly
-->
**TypeScript Version:**  3.6.3

<!-- Search terms you tried before logging this (so others can find this issue more easily) -->
**Search Terms:** intellisense suggestion hint enum

**Code**

In enum indexed interfaces ts suggestion fails (playground) or suggest incorrect informations (vscode).

```ts
enum IPiece {
    Id = "id",
    FirstName = "firstname",
    LastName = "lastname",
    Age = "age"
}

interface IPiecePerson {
    [IPiece.Id]: string | null;
    [IPiece.FirstName]: string | null;
    [IPiece.LastName]: string | null;
    [IPiece.Age]: number | null;
}

const o: Pick<IPiecePerson, IPiece.Age> = { [IPiece.Age]: 1 };
o.
```

**Expected behavior:**
```ts
// age
// or IPiece.Age
```

**Actual behavior:**
```ts
// IPiece enum
```

<img width="677" alt="Zrzut ekranu 2019-10-19 o 07 52 27" src="https://user-images.githubusercontent.com/1199350/67138560-9e0f9f80-f245-11e9-9100-f5579f733937.png">

**Playground Link:** [TypeScript Playground](http://www.typescriptlang.org/play/?ssl=16&ssc=3&pln=1&pc=1#code/KYOwrgtgBAkgCgS2AY2FA3gKCj2ATKAXigCIE8SAabXAMQQCcBnAFwDkBDCNYkgM0asQXYFRo4AMh1aduRUgBtpLYdzG4oAQQDmPUh10lMAX0yYEIFsAZ8OqWIhTA41pgHsQGcVADa8JKgAdDB4ALoAXFCsDBbaUAA+UOAKCgDc3n6OQfTM7CIRUSwxIHGJyWkZ-k6BUjL5kdGxCUlgKekamQHAgTrABeAQAEbWzeXpppjIHqxQbpGIyADWADxVqC7MHpQOXT26AHzy6L5r3b0FAIxQxulugUA)

**Related Issues:** https://github.com/microsoft/vscode/issues/83097
Repository: TypeScript
Issue: Allow static members in abstract classes to reference type parameters
Labels: Suggestion,Awaiting More Feedback

## Search Terms

abstract generic class static type parameter 

## Suggestion

It has been previously concluded in #24018 that referencing class type parameters in the static side is problematic unless that class is meant to be extended. Well, abstract classes _are_ meant to be extended, so it makes sense to allow type parameters to be used?

To quote @andy-ms in #24018:

> Without inheritance that wouldn't make much sense:
> ```ts
> class Super<T> {
>     static m(x: T): void;
> }
> Super.m(); // What's `T`?
> ```

This is a valid point. But with a solution for abstract static members from #34516, this could be refactored like so:

```ts
abstract class Super<T> {
    abstract static m(x: T): void;
}

class A extends Super<number> {
    static m(x) {
        console.log(x * 42);
    }
}

class B extends Super<string> {
    static m(x) {
        console.log(x + ' World!');
    }
}

A.m(2);
B.m('Hello');
```

## Use Cases

Pretty much everywhere that instance side types are related to static side types and where instance methods depend on static properties.

## Examples

As an example I'll give my personal use case. I have a class with a static `defaults` property and instances merge it with a constructor argument of the same type but partial. Then, the resulting object is stored in an instance property:

```ts
abstract class Base<T> {
    static defaults: T
    config: T

    constructor(options: Partial<T>) {
        this.config = Object.assign({}, (this.constructor as typeof Base).defaults, options);
    }
}

interface Options {
    a: string
    b: number
}

class A extends Base<Options> {
  static defaults = {
      a: 42,    // Type '42' is not assignable to type 'string'.
      b: 'oops' // Type '"oops"' is not assignable to type 'number'.
  };
}

let inst = new A({
    a: 'bar', // OK
    b: 'baz'  // Type '"baz"' is not assignable to type 'number'.
});

inst.config.a = 12; // Type '12' is not assignable to type 'string'.
```

## Checklist

My suggestion meets these guidelines:

* [x] This wouldn't be a breaking change in existing TypeScript/JavaScript code
* [x] This wouldn't change the runtime behavior of existing JavaScript code
* [x] This could be implemented without emitting different JS based on the types of the expressions
* [x] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, etc.)
* [x] This feature would agree with the rest of [TypeScript's Design Goals](https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals).
Repository: flutter
Issue: Cupertino Text Field Placeholder should expand to multiple lines if max lines is not set
Labels: a: text input,c: new feature,framework,f: material design,f: cupertino,a: quality,c: proposal,P3,team-text-input,triaged-text-input


I would expect that placeholder text automatically expands to multiple lines if number of lines for text field is not set - unlimited. It works if I put max lines ex. 2, but doesn't work if unlimited lines.

Tested on simulator iPhone X

## Steps to Reproduce


```
import 'dart:async';

import 'package:flutter/cupertino.dart';
import 'package:flutter/material.dart';

void main() {
  runApp(
    MyApp(),
  );
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: MyHomePage(),
    );
  }
}

class MyHomePage extends StatefulWidget {
  MyHomePage({Key key}) : super(key: key);

  @override
  _MyHomePageState createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  void initState() {
    super.initState();
  }

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: SizedBox(
          width: 100,
          height: 100,
          child: CupertinoTextField(
            placeholder: 'This is long placeholder and it should be in two lines',
            textAlign: TextAlign.center,
          //maxLines: 2,
          ),
        ),
      ),
    );
  }
}

```


## Logs

<!--
      Run your application with `flutter run --verbose` and attach all the
      log output below between the lines with the backticks. If there is an
      exception, please see if the error message includes enough information
      to explain how to solve the issue.
-->

```
```

<!--
     Run `flutter analyze` and attach any output of that command below.
     If there are any analysis errors, try resolving them before filing this issue.
-->

```
```

<!-- Finally, paste the output of running `flutter doctor -v` here. -->

```
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel beta, v1.10.7, on Mac OS X 10.14.6 18G87, locale en-US)
 
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Xcode - develop for iOS and macOS (Xcode 11.0)
[✓] Android Studio (version 3.5)
[!] IntelliJ IDEA Ultimate Edition (version 2018.1.2)
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
[✓] Connected device (1 available)

! Doctor found issues in 1 category.
```
Repository: rust
Issue: UnwindSafe docs are unclear
Labels: C-enhancement,T-libs-api,A-docs

After reading the current docs of `panic::UnwindSafe` I am left with the feeling that the explanation of the concept of "panic safety" and its auto trait has only been handwaved. The docs point to the RFC, which does a better job, but I still do not feel like I fully understood what "panic safety" is, what does `UnwindSafe` convey, and when is it ok to use `AssertUnwindSafe` or not. 

I feel like the docs could do a much better job at explaining all of this, and that some examples in the RFC would help, but I don't think that any documentation we currently have about this is enough.

This is a complex topic and I have no experience teaching it to others, but if somebody has done that, their feedback should probably be taken into account. 

I think the API docs are very important to get right because they are currently the only teaching materials that we have about this aspect of the language. The Rust book does not cover this either, and the nomicon [Exception safety](https://doc.rust-lang.org/nomicon/exception-safety.html?highlight=panic,safety#exception-safety) has very few words to say about this.

cc @steveklabnik @rust-lang/docs
Repository: godot
Issue: add_child command gives errors in script without extends command
Labels: enhancement,topic:gdscript,usability

Version : 3.2-alpha2

Windows 10

Create an empty project. Add a node to it of any type. Attach a script to the node that includes the add_child function, but don't have an "extends" line in the script.

e.g.
 func testfunc():
	var nodeTest    = Node.new()
	add_child(nodeTest)

The IDE will complain that "The method "add_child" isn't declared in the current class." but doesn't error that the "extends Node" line is missing.

I would suggest that either the IDE should error that an "extends" line is missing from the top of the script, or failing that, the error should be "The method "add_child" isn't declared in the current class - did you miss adding an extends line to your script?"
Repository: flutter
Issue: macOS AOT mode doesn't use bundle path
Labels: platform-mac,a: desktop,P2,team-macos,triaged-macos

I was trying to figure out why https://github.com/google/flutter-desktop-embedding/issues/599 was running the linked App.framework's AOT code instead of just failing outright, and realized that we're not actually doing anything in the macOS version of `FlutterDartProject` that would control the AOT code being used (i.e., the equivalent of the iOS `FlutterDartProject`'s `settings.application_library_path` code).

I believe this means it's impossible to run two different AOT compilations in the same app in an add-to-app scenario, for instance.

@chinmaygarde What's the right way to do the equivalent of `application_library_path` via the embedder API? `vm_snapshot_data`? (And if so, why is this currently working at all?)

/cc @jonahwilliams maybe you know since you wired this up.
Repository: youtube-dl
Issue: Option for Thumbnail Filename
Labels: request

<!--

######################################################################
  WARNING!
  IGNORING THE FOLLOWING TEMPLATE WILL RESULT IN ISSUE CLOSED AS INCOMPLETE
######################################################################

-->


## Checklist

<!--
Carefully read and work through this check list in order to prevent the most common mistakes and misuse of youtube-dl:
- First of, make sure you are using the latest version of youtube-dl. Run `youtube-dl --version` and ensure your version is 2019.10.22. If it's not, see https://yt-dl.org/update on how to update. Issues with outdated version will be REJECTED.
- Search the bugtracker for similar feature requests: http://yt-dl.org/search-issues. DO NOT post duplicates.
- Finally, put x into all relevant boxes (like this [x])
-->

- [x] I'm reporting a feature request
- [x] I've verified that I'm running youtube-dl version **2019.10.22**
- [x] I've searched the bugtracker for similar feature requests including closed ones


## Description

<!--
Provide an explanation of your issue in an arbitrary form. Please make sure the description is worded well enough to be understood, see https://github.com/ytdl-org/youtube-dl#is-the-description-of-the-issue-itself-sufficient. Provide any additional information, suggested solution and as much context and examples as possible.
-->

With the --write-thumbnail option, it would be great to have a Filenamen that differs from the video.
At the moment, you'd get `videoname.mkv` and `videoname.jpg` for the Video and Thumbnail. I'd like to have the option for something like `videoname.mkv` and `videoname-thumbnail.jpg`
Repository: pytorch
Issue: Get rid of libc10.so
Labels: module: build,triaged

I finally have a good reason to merge libc10.so into libtorch.so (and corresponding libc10_cuda.so into libtorch_cuda.so, etc.): I am trying to devirtualize access to AutogradMeta, but because TensorImpl lives in c10 and AutogradMeta lives in torch, I cannot do this as the destructor would have to cross a dynamic library boundary. By absorbing c10 into torch I will be able to do this.

Some dangers: putting c10 into libtorch might push Windows build over max library size. See https://github.com/pytorch/pytorch/issues/27215

Other alternatives I thought of:
* Move TensorImpl back to ATen/core
* Keep the virtual interface to AutogradMeta

Some alternatives that don't work
* Move AutogradMeta into c10. AutogradMeta must be declared after Tensor as it contains a field Tensor, and we cannot replace that field with an `intrusive_ptr<TensorImpl>` as the public API of tensor `x.grad()` returns a mutable `Tensor&` reference. (Though, it might be possible to fix this up by just moving TensorImpl out of c10)

cc @ezyang @gchanan @zou3519 @jerryzh168 @dzhulgakov @smessmer
Repository: rust
Issue: std::io::Stdout does not implement Clone
Labels: C-enhancement,T-libs-api,A-io

`std::io::Stdout` is currently not `Clone`, but there is no reason that I can see that it shouldn't be
Repository: pytorch
Issue: [JIT] __repr__ support for ScriptModules
Labels: oncall: jit,low priority,triaged,jit-backlog

It would be nice if we could preserve original module `__repr__`s while scripting. e.g.: 

```
import torch, io


class ScriptWithAttr(torch.nn.Module):
    def __init__(self):
        super().__init__()
        self.attr = 3

    def forward(self, x):
        return x + self.attr

    @torch.jit.export
    def __repr__(self):
        return 'foo ' + str(self.attr)


mod = ScriptWithAttr()
print(mod)

scripted = torch.jit.script(ScriptWithAttr())
print(scripted)
b = io.BytesIO()
torch.jit.save(scripted, b)
b.seek(0)
loaded = torch.jit.load(b)

print(loaded.__repr__())
```

Prints

```
foo 3
RecursiveScriptModule(original_name=ScriptWithAttr)
RecursiveScriptModule(original_name=ScriptWithAttr)
```

whereas it would be nice if it would print `foo 3` every time.

This has been confusing for users, e.g. https://github.com/pytorch/pytorch/issues/28331

cc @suo
Repository: pytorch
Issue: [jit] C++ Documentation
Labels: oncall: jit,triaged,jit-backlog

Things like `script::Module` have no user facing documentation

`IValue` also doesn't have any user facing docs (things like custom classes are hard to guess at)

cc @suo
Repository: pytorch
Issue: Unified management of thread local variables
Labels: high priority,module: performance,module: internals,feature,triaged,module: multithreading

We should add a centralized thread local handling (in c10).
This would have two main advantages:
- Make the code cleaner as all thread local elements can easily be identified and worked with.
- Allow improvement for example in `at::parallel_for` where it will be able to handle any Tensor operation within the loop.

(Setting high priority only to be discussed for triage)

cc @ezyang @gchanan @zou3519 @jerryzh168 @SsnL @albanD @gqchen @VitalyFedyunin @ngimel @mruberry
Repository: godot
Issue: BaseButton's button_mask does not affect focus
Labels: bug,confirmed,usability,topic:gui

**Godot version:**
Godot_v3.1.1-stable_win64

**Issue description:**
BaseButton's button_mask changes which mouse button is used to trigger "pressed" on a UI button. However, it does not change which mouse button makes the UI button grab the focus (that seems to be fixed as the left button).

Thus, if one sets button_mask = BUTTON_MASK_LEFT, left-clicking a UI button moves the focus (e.g. changes the button texture to "focused") and also triggers the function connected to the "pressed" signal, while right-clicking does nothing.

But, if one sets button_mask = BUTTON_MASK_RIGHT, right-clicking only triggers the function connected to the "pressed" signal, while left-clicking moves the focus without triggering anything.

I believe this would be confusing to players. I believe button_mask should affect focus grabbing in the same way it does button pressing.

BTW, is there some way to forego this button mask altogether and get the mouse button from InputMap["ui_accept"], as is done for the keyboard?
Repository: godot
Issue: Can't compare INF or NAN reliably when used on an int var
Labels: bug,discussion,topic:core

**Godot version:**
3.1.1.stable.official

**OS/device including version:**
Windows 7

**Issue description:**
If you assign an `int var` with `INF`, it gets initialized as the min `int` (-9223372036854775808) which is not useful as it is a negative value. If we are not meant to assign `INF` to an `int` then the docs should have a warning on it.

I haven't tested this using floats btw.

**Steps to reproduce:**
```
var int_inf : int = INF
var casted_int_inf = int(INF)
var as_inf = INF as int
var untyped_inf = INF

print(int_inf) # -9223372036854775808
print(casted_int_inf) # -9223372036854775808
print(as_inf) # -9223372036854775808
print(untyped_inf) # 1.#INF

print(1 > INF) # False. Ok
print(1 > int_inf) # True. Not ok.
print(1 > casted_int_inf) # True. Not ok.
print(1 > as_inf) # True. Not ok.
print(1 > untyped_inf) # False. Ok.

print(INF == INF) # True. Duh :P
print(INF == int_inf) # True. But what about if I wanted to check a negative INF ?
print(INF == casted_int_inf) # True. Same as above.
print(INF == as_inf) # True. Same as above.
print(INF == untyped_inf) # False. Huh?
```
Repository: pytorch
Issue: TestTorch.test_c10_layer_norm fails if you run it on a build of PyTorch with BUILD_CAFFE2_OPS=0
Labels: triaged,module: dispatch

Failure: 

```
======================================================================                                                      
ERROR: test_c10_layer_norm (__main__.TestTorch)                                                                             
----------------------------------------------------------------------                                                      
Traceback (most recent call last):                                                                                          
  File "test/test_torch.py", line 5580, in test_c10_layer_norm                                                              
    torch.ops._caffe2.LayerNorm(torch.tensor(X), torch.tensor(                                                              
  File "/data/users/ezyang/pytorch-tmp/torch/_ops.py", line 61, in __getattr__                                              
    op = torch._C._jit_get_operation(qualified_op_name)                                                                     
RuntimeError: No such operator _caffe2::LayerNorm                                                                           
                                                                       
```