sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: proposal: x/tools/go/packages: support a "dir=" query
Labels: Proposal

# Motivation

We can currently query `packages.Load` for a package import path with `pattern=string`, and a path to a Go file within a package of interest with `file=path/to/file.go`.  It would be very useful to be able to query for a package given its directory.

`go/build` can do this very easily with `ImportDir`, but `x/tools/go/packages` lacks the equivalent.

# Proposal

A new query "dir=path/to/package" matches the package or packages in the directory path/to/package. For example "dir=~/go/src/fmt" might return the packages "fmt" and "fmt [fmt.test]".

# Workaround

Use `filepath.Glob` to query for all `*.go` files in a given directory, pick the first one, then perform a `file=` query.  This has much more overhead due to the glob, and is very undesirable when performing a large recursive package import traversal.

Thanks for your consideration.
Repository: storybook
Issue: Helper functions for storybook components
Labels: question / support

Recently I needed to create a few helper functions that should be used just in storybook components. Checking the documentation, I didn't find anything specifically explaining about where these functions should be. I considered to add into the `globalType`, but definitely it doesn't seem the best for it.

So, my question is: where would be the best place to leave these helper functions that should be used just into storybook?
Also, maybe it'd be nice to have a `helper.js` within `.storybook/` folder where we could just export these helpers.
Repository: godot
Issue: Multiple same warning for 1 line script, need to place multiple same ignore
Labels: bug,topic:gdscript

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
<!-- Specify commit hash if using non-official build. -->
3.2.4 beta2 2800ab75ad

**OS/device including version:**
<!-- Specify GPU model, drivers, and the backend (GLES2, GLES3, Vulkan) if graphics-related. -->
not related

**Issue description:**
<!-- What happened, and what was expected. -->
Multiple same warning for 1 line script, need to place multiple same ignore _(as title says)_
It'd be better for 1 warning / ignore for 1 line if it is the same warning.

**Steps to reproduce:**
```gdscript
func _ready():
	var num := 9999
	var value := num / 60 / 60 / 24 # 3 INTEGER_DIVISION warning for this line
```
to ignore 3 warnings for this line, need to put 3 same ignore.
```gdscript
	var num := 9999
# warning-ignore:integer_division
# warning-ignore:integer_division
# warning-ignore:integer_division
	var value := num / 60 / 60 / 24
```

**Minimal reproduction project:**
<!-- A small Godot project which reproduces the issue. Drag and drop a zip archive to upload it. -->
Repository: pytorch
Issue: pin_memory=True in DataLoader converts a tuple to list automatically
Labels: module: dataloader,triaged

## 🐛 Bug

DataLoader with `pin_memory=True` converts the tuple returned from `Dataset` & `collate_fn` to a list automatically. This is because of this particular [line in the `pin_memory` implementation](https://github.com/pytorch/pytorch/blob/master/torch/utils/data/_utils/pin_memory.py#L55)  . The actual type of the return item shouldn't be changed.

```python
elif isinstance(data, container_abcs.Sequence):
    return [pin_memory(sample) for sample in data]
```


## To Reproduce

```python
class TestDataset(IterableDataset):
    def __iter__(self):
        for i in range(30):
            yield torch.rand(3, 10), torch.rand(3, 3)

def collate(batch):
    return batch[0]

dset = TestDataset()

dloader = DataLoader(
    dataset=dset,
    batch_size=1,
    collate_fn=collate,
    num_workers=0,
    pin_memory=True
)

iterator = iter(dloader)
batch = next(iterator)
print(type(batch))   # prints: < class 'list' >


dloader = DataLoader(
    dataset=dset,
    batch_size=1,
    collate_fn=collate,
    num_workers=0,
    pin_memory=False)

iterator = iter(dloader)
batch = next(iterator)
print(type(batch))   # prints: < class 'tuple' >
```

## Expected behavior

The first `print` statement above should print `<class 'tuple'>`

## Environment

PyTorch version: 1.7.0
Is debug build: True
CUDA used to build PyTorch: 10.2
ROCM used to build PyTorch: N/A

OS: Ubuntu 19.10 (x86_64)
GCC version: (Ubuntu 9.2.1-9ubuntu2) 9.2.1 20191008
Clang version: Could not collect
CMake version: version 3.18.2

Python version: 3.7 (64-bit runtime)
Is CUDA available: True
CUDA runtime version: Could not collect
GPU models and configuration: 
GPU 0: GeForce RTX 2080 Ti
GPU 1: GeForce RTX 2080 Ti
GPU 2: GeForce RTX 2080 Ti
GPU 3: GeForce RTX 2080 Ti

Nvidia driver version: 455.45.01
cuDNN version: Probably one of the following:
/usr/lib/x86_64-linux-gnu/libcudnn.so.7.6.5
/usr/lib/x86_64-linux-gnu/libcudnn.so.8.0.5
/usr/lib/x86_64-linux-gnu/libcudnn_adv_infer.so.8.0.5
/usr/lib/x86_64-linux-gnu/libcudnn_adv_train.so.8.0.5
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_infer.so.8.0.5
/usr/lib/x86_64-linux-gnu/libcudnn_cnn_train.so.8.0.5
/usr/lib/x86_64-linux-gnu/libcudnn_ops_infer.so.8.0.5
/usr/lib/x86_64-linux-gnu/libcudnn_ops_train.so.8.0.5
HIP runtime version: N/A
MIOpen runtime version: N/A

Versions of relevant libraries:
[pip3] numpy==1.19.4
[pip3] pytorch-lightning==1.0.5
[pip3] torch==1.7.0
[pip3] torchaudio==0.7.0
[conda] numpy                     1.19.4                   pypi_0    pypi
[conda] pytorch-lightning         1.0.5                    pypi_0    pypi
[conda] torch                     1.7.0                    pypi_0    pypi




cc @SsnL @VitalyFedyunin @ejguan
Repository: opencv
Issue: Paint will corrupt .tif-files
Labels: category: 3rdparty

##### System information (version)
- OpenCV => 4.2
- Operating System / Platform => Windows 10 64 Bit
- Python 3.8
##### Detailed description

I was spending the day trying to figure out, why opencv can't open several .tif files I've been creating saving with Paint.
Found out, that after I applied some manual adjustments on pictures using Paint and saving it, one is not able to reload it using imread().
Opening with scikit image first didn't succeed as well... Error was something like LZW <5 blabllabala... Sorry, i don't remember anymore =( After installing imagecodecs and resaving the picture via scikit got me rid of getting errors :)
Seems like Paint is corrupting some of the compression for tif files in a way that opencv is not able to load those pictures anymore...


##### Steps to reproduce

Load an image using cv.imread()
save that image, using cv.imwrite('blablbal.tif", img)
Open 'blablbal.tif" in Paint and save it again.
try to load that image again using  img = cv.imread('blablbal.tif") and confirm with smth. like print(img.shape)

######
I was actually not sure where to post that kind of issue, since I don't know if it's opencv related or just stupid me using Paint... Just wanted to post this behaviour somewhere :D
Hopefully someone out there can retrieve this information running into the same kind of error and won't have some painfull hours :)
Cheers
Repository: rust
Issue: Mismatched type for cyclic dev-dependency on same crate
Labels: A-resolve,T-compiler,C-bug

[Repository](https://github.com/kaichaosun/test-workspace-dependency) @kaichaosun

See above link for minimal viable example. There are two crates, one is test-utils, another is client. This is the client code with a dev-dependency on test-utils:

```rust
#[derive(Eq, PartialEq, Debug)]
pub struct Name {
	pub first_name: String,
	pub last_name: String,
}

pub fn full_name(first: String, last: String) -> Name {
	Name {
		first_name: first,
		last_name: last,
	}
}

mod test {
	use super::*;

	#[test]
	fn full_name_works() {
		assert_eq!(
			full_name(String::from("hello"), String::from("world")), 
			test_utils::helper()
		)
	}
}
```

This is test-utils code with a dependency on client:

``` rust
use client::Name;

pub fn helper() -> Name {
	Name {
		first_name: String::from("hello"),
		last_name: String::from("world"),
	}
}
```

Running `cargo test` (`cargo test +stable`) in the workspace shows this error:

```
error[E0308]: mismatched types
  --> client/src/lib.rs:19:3
   |
19 |           assert_eq!(
   |  _________^
20 | |             full_name(String::from("hello"), String::from("world")),
21 | |             test_utils::helper()
22 | |         )
   | |_________^ expected struct `Name`, found struct `client::Name`
```

I expect this code to correctly determine that the types are compatible, because they effectively reference the same crate.

### Meta

`rustc --version --verbose`:
```
rustc 1.49.0-nightly (5404efc28 2020-11-11)
binary: rustc
commit-hash: 5404efc28a0cddee103ef6396c48ea71ff9631c8
commit-date: 2020-11-11
host: x86_64-unknown-linux-gnu
release: 1.49.0-nightly
```

Other Issue: https://github.com/rust-lang/rust/issues/59305
Discussion on Rust forum: https://users.rust-lang.org/t/cyclic-dev-dependency-get-type-mismatch-error/48391/1
Repository: go
Issue: encoding/xml: encoding a start token with a namespace set in Name and Attr results in invalid XML
Labels: NeedsInvestigation

<!--
Please answer these questions before submitting your issue. Thanks!
For questions please use one of our forums: https://github.com/golang/go/wiki/Questions
-->

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version devel +6965b01ea248 20201124070706 linux/amd64
</pre>

### Does this issue reproduce with the latest release?

Yes

### What did you do?

https://play.golang.org/p/ZK8yhFoVwW9

```
	e := xml.NewEncoder(os.Stdout)
	e.EncodeToken(xml.StartElement{
		Name: xml.Name{Space: "space", Local: "local"},
		Attr: []xml.Attr{{Name: xml.Name{Local: "xmlns"}, Value: "spaceAttr"}},
	})
	e.Flush()
	// <local xmlns="space" xmlns="spaceAttr">
```



### What did you expect to see?

One "xmlns" attribute.

### What did you see instead?

Two "xmlns" attributes.

While the provided example is obviously very contrived this can happen in the real world (without a user doing something silly like manually adding a second xmlns attribute) in the situation where the user decodes some XML, makes modifications to the tokens, then encodes those tokens somewhere else (probably a fairly common use case). The decoder will add the "xmlns" attribute and fill out Name, then the encoder will duplicate it. The user would have to manually remove xmlns before encoding, or use `RawToken` (but this is not an option if the decoder is an arbitrary `xml.TokenReader` so this isn't always possible). Skipping xmlns attributes on marshal if `start.Name` is set is a simple patch that would fix this issue.
Repository: go
Issue: sum.golang.org: responding with a 410 fifteen minutes after publishing a new module version
Labels: Documentation,NeedsInvestigation,proxy.golang.org

```
$ go version
go version devel +f7a80af935 Mon Nov 23 05:47:51 2020 +0000 linux/amd64
$ go get -d github.com/multiformats/go-multicodec@v0.2.0
github.com/multiformats/go-multicodec@v0.2.0: verifying module: github.com/multiformats/go-multicodec@v0.2.0: reading https://sum.golang.org/lookup/github.com/multiformats/go-multicodec@v0.2.0: 410 Gone
	server response: not found: github.com/multiformats/go-multicodec@v0.2.0: invalid version: unknown revision v0.2.0
$ GONOSUMDB='*' go get -d github.com/multiformats/go-multicodec@v0.2.0
$ GOPROXY=direct GONOSUMDB='*' go get -v -d github.com/multiformats/go-multicodec@v0.2.0
```

At the time of running these commands, I pushed the new version via `git tag -a v0.2.0 && git push --tags` over fifteen minutes ago. Why am I still getting a 410?

Note how skipping sum.golang.org alone, or skipping both the sumdb and proxy, make the `go get` work. These commands were run outside of any module.

At first I assumed I was holding something wrong, but I've triple-checked everything. You can see the version at https://github.com/multiformats/go-multicodec/blob/v0.2.0/go.mod. I realise that the tag says "five hours ago", because that's when I tagged it - but I only pushed the tag fifteen minutes ago.
Repository: TypeScript
Issue: TS JSDoc visibility error
Labels: Bug

<!-- 🚨 STOP 🚨 STOP 🚨 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:**  4.1.2

<!-- Search terms you tried before logging this (so others can find this issue more easily) -->
**Search Terms:**
An explicit type annotation may unblock declaration emit
private name

**Code**

https://codesandbox.io/s/tsc-ts-docs-visibility-repro-f73gw?file=/src/index.js

```ts
// @filename: base.js
class Base {
    constructor() {}
}

const BaseFactory = () => {
    return new Base();
};

BaseFactory.Base = Base;

module.exports = BaseFactory;

// @filename: index.js
/** @typedef {import('./base')} BaseFactory */

/**
 *
 * @param {InstanceType<BaseFactory["Base"]>} base
 */
const test = (base) => {
    return base;
};
```

**Expected behavior:**
No error from the compiler

**Actual behavior:**
```bash
src/index.js:7:1 - error TS9006: Declaration emit for this file requires using private name 'Base' from module '"/sandbox/src/base"'. An explicit type annotation may unblock declaration emit.

7 const test = (base) => {
  ~~~~~


Found 1 error.

```


**Playground Link:** <!-- A link to a TypeScript Playground "Share" link which demonstrates this behavior -->
https://codesandbox.io/s/tsc-ts-docs-visibility-repro-f73gw?file=/src/index.js

**Related Issues:** <!-- Did you find other bugs that looked similar? -->
https://github.com/microsoft/TypeScript/issues/37832
https://github.com/microsoft/TypeScript/issues/41250
Repository: rust
Issue: Unfound macro diagnostic could detect a later definition and suggest moving it
Labels: C-enhancement,A-diagnostics,A-resolve,A-macros,T-compiler

I tried this code:

```rust
mod a {
    pub fn f() {
        hello!("from a::f"); 
    }
}

macro_rules! hello {
    ($x:expr) => {
        println!("Hello {}!", $x);
    }
}

fn main() {
    a::f();
    hello!("from main");
}
```

I expected to see this happen: 

"You have a use of an undefined macro. I found a macro definition with the same name later in the module hierarchy. I suggest you move the macro definition so that it appears before the `mod a` in the module hierarchy, like so: ..."

Instead, this happened: 

```
error: cannot find macro `hello` in this scope
 --> src/main.rs:3:9
  |
3 |         hello!("from a::f"); 
  |         ^^^^^
  |
  = help: have you added the `#[macro_use]` on the module/import?

error: aborting due to previous error
```

(where the help here does not actually address the problem.)

## Version info

Nightly channel

Build using the Nightly version: 1.50.0-nightly

(2020-11-23 d9a105fdd46c926ae606)

</p>
</details>
Repository: flutter
Issue: Flutter Web `Ctrl` +`Plus/Minus` keys don't disable browser zoom
Labels: engine,platform-windows,platform-web,has reproducible steps,P2,team-web,triaged-web,found in release: 3.19,found in release: 3.20

On the latest beta, Ctrl + Scroll just broke and Flutter no longer captures that shortcut correctly.

Individual onPointerSignal events are still captured. However, when used in conjunction with RawKeyboardListener for composing shortcuts, the browser takes control of the zoom instead of Flutter.

The previous version of beta supported capturing the shortcut.

Prior to the latest beta update, Flutter captured shortcut events. Now, Flutter no longer intercepts those events and key combinations no longer work.

## Steps to Reproduce
1. Setup a RawKeyboardListener to listen for Ctrl + [Any Key]
2. Prior to the latest Beta version, Ctrl + Zoom was captured by Flutter.
3. Now, Ctrl + Zoom activates the browser zoom instead.

## Example
```dart
 RawKeyboardListener(
      focusNode: FocusNode()..requestFocus(),
      onKey: (event) {
          if (event.logicalKey == LogicalKeyboardKey.controlLeft) {
          }
      },
)
```

## Logs
Flutter Doctor
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2) [√] Chrome - develop for the web [√] Android Studio (version 4.1.0) [√] VS Code (version 1.50.1) [√] Connected device (4 available)
Repository: TypeScript
Issue: Refactor Rename on string parameters for symbol named parameters
Labels: Suggestion,Awaiting More Feedback

<!-- 🚨 STOP 🚨 𝗦𝗧𝗢𝗣 🚨 𝑺𝑻𝑶𝑷 🚨

Half of all issues filed here are duplicates, answered in the FAQ, or not appropriate for the bug tracker.

Please help us by doing the following steps before logging an issue:
  * Search: https://github.com/Microsoft/TypeScript/search?type=Issues
  * Read the FAQ, especially the "Common Feature Requests" section: https://github.com/Microsoft/TypeScript/wiki/FAQ

-->

## Search Terms

Refactor Rename Symbol Named Parameters

## Suggestion

Refactor Rename on string parameters ([symbol named properties](https://www.typescriptlang.org/docs/handbook/release-notes/typescript-2-9.html))

## Use Cases

If you refactor rename a method every "reference" should be renamed.
Even the string parameter in e.g. jasmin.createSpyObj:


## Examples

This 

```
class MyService 
{
	public myMethod(n: number) {
		return n;
	}
}

const myServiceSpy = jasmine.createSpyObj<MyService>(['myMethod']);
myServiceSpy.myMethod.withArgs(999).and.returnValue(1);
```
should become

```
class MyService 
{
	public renamedMethod(n: number) {
		return n;
	}
}

const myServiceSpy = jasmine.createSpyObj<MyService>(['renamedMethod']);
myServiceSpy.renamedMethod.withArgs(999).and.returnValue(1);
```
after refactor rename.


## 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: SystemUiOverlayStyle animations
Labels: framework,a: animation,c: proposal,P3,team-framework,triaged-framework

### Use case

In some cases we might want to animate a system UI. Currently we have to write our own code to make it possible: save previous style, provide some global ticker, even implement `SystemUiOverlayStyleTween`. All these functionalities should be in the core framework, for my opinion, and I don't understand why they aren't yet, honestly.

### Proposal

My proposal is to add to framework something like [this](https://github.com/nt4f04uNd/nt4f04unds_widgets/blob/15e90cf13a3941a7de6cf3b7b04b78a1efc3a71d/lib/src/components/util/system_ui_animations.dart)
This is of course can't be used in the way it's written directly in the framework, just a working example that serves a purpose
Repository: TypeScript
Issue: JSDoc : support @interface and @implements (and more)
Labels: Suggestion,Awaiting More Feedback

I would like to re-open an issue: https://github.com/microsoft/TypeScript/issues/16142

As it is locked I am to open this new issue.

I personally do not like TypeScript as a language, I do however, like TypeScript as a tool for my JavaScript projects, hence why I would like more support for JSDoc.

**Code**

In a JavaScript file, with checkJs option :
```
/**
 * @interface Something
 */

/**
 * @function
 * @name Something#hello
 * @param {string} name
 */

/**
 * @implements {Something}
 */
const something = {};
```
**Actual behavior:**

No error.

**Expected behavior:**

An error should be raised. It's important in checkJs mode as JavaScript doesn't have native interfaces.
Repository: vscode
Issue: Change rulers on Editor via extension API
Labels: feature-request,api,editor-api

<!-- ⚠️⚠️ Do Not Delete This! feature_request_template ⚠️⚠️ -->
<!-- Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct/ -->
<!-- Please search existing issues to avoid creating duplicates. -->

<!-- Describe the feature you'd like. -->

The EditorConfig extension needs the ability to change the rulers on a particular editor if `max_line_length` is provided.

https://github.com/editorconfig/editorconfig-vscode/issues/272
Repository: transformers
Issue: Model Parallelism and Big Models
Labels: Model Parallel,WIP

# 🚀 Feature request

This is a discussion issue for training/fine-tuning very large transformer models. Recently, model parallelism was added for gpt2 and t5. The current implementation is for PyTorch only and requires manually modifying the model classes for each model. Possible routes (thanks to @stas00  for identifying these):
- `fairscale` to avoid individual model implementation
- `deepspeed` to possibly enable even larger models to be trained
Repository: rust
Issue: fs::copy: Cannot copy special/device files (ex: /dev/null) into a file
Labels: C-bug,T-libs,A-io,O-unix

I tried this code:

```rust
use std::fs;

fn main() {
    match fs::copy("/dev/null", "bar.txt") {
        Ok(_n) => (),
        Err(e) => {
            println!("{}", e);
            ()
        }
    }
}

```

_I expected to see this happen:_
I agree that the use case isn't super useful. But I would expect this to work and create an empty file.


_Instead, this happened:_

`the source path is not an existing regular file`

Don't hesitate to wontfix this bug...
Repository: godot
Issue: KinematicBody2D isn't moved by RemoteTransform2D if "sync to physics" is enabled, and the RemoteTransform2D doesn't update scale or rotation
Labels: bug,confirmed,topic:physics

<!-- Please search existing issues for potential duplicates before filing yours:
https://github.com/godotengine/godot/issues?q=is%3Aissue
-->

**Godot version:**
<!-- Specify commit hash if using non-official build. -->
v3.2.3.stable.official


**OS/device including version:**
<!-- Specify GPU model, drivers, and the backend (GLES2, GLES3, Vulkan) if graphics-related. -->
Fedora 33

**Issue description:**
<!-- What happened, and what was expected. -->
While using an AnimationPlayer to animate a RemoteTransform2D node whose remote path is set to a KinematicBody2D node, the node will not move if it has "sync to physics" enabled, and the RemoteTransform2D has rotation or scale updates disabled. If you disable "sync to physics" or enable rotation/scale updates, then the node will move correctly.

**Steps to reproduce:**
1. Create Node2D scene
2. Add a KinematicBody2D node as a child, with the "Sync to Physics" property enabled
3. Add a RemoteTransform2D node, with the remote path set to the KinematicBody2D node, and the "update_rotation" and/or "update_scale" properties disabled
4. Add an AnimationPlayer with an animation to modify the RemoteTransform2D position
5. Run the scene, and the KinematicBody2D will not move

**Minimal reproduction project:**
<!-- A small Godot project which reproduces the issue. Drag and drop a zip archive to upload it. -->
 
[Moving Platforms.zip](https://github.com/godotengine/godot/files/5592962/Moving.Platforms.zip)