sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: pytorch
Issue: torch Summary writer does not display torchvision.io.read_video output
Labels: module: tensorboard,oncall: visualization

## Issue
I am trying to utilize the torch tensorboard [`add_video`](https://pytorch.org/docs/stable/tensorboard.html#torch.utils.tensorboard.writer.SummaryWriter.add_video) functionality to view my `.mp4` video generated from `matplotlib` figures. I first convert the `.mp4` into a `torch.Tensor` via [`torchvision.read_video`](https://pytorch.org/docs/stable/torchvision/io.html#torchvision.io.read_video), then write it to tensorboard without errors:
```
......
    vframes, aframes, info = read_video(vid, pts_unit="sec")
    vframes = vframes[None] # add batch dimension
    vframes = vframes.permute(0, 1, 4, 2, 3) # (N, T, H, W, C) -> (N ,T , C, H, W)
    writer = self._get_writer(training)
    writer.add_video(tag, vframes, step, fps, walltime)
```
but upon viewing the Tensorboard it is just empty. I inspect my tensor and ensure that they are as required by the `add_video` module:
```
>>>  from torchvision.io import read_video
>>> vframes, aframes, info =  read_video(vid, pts_unit="sec")
>>> vframes
        [[[255, 255, 255,  ..., 255, 255, 255],
           [255, 255, 255,  ..., 255, 255, 255],
           [255, 255, 255,  ..., 255, 255, 255],
           ...,
           [255, 255, 255,  ..., 255, 255, 255],
           [255, 255, 255,  ..., 255, 255, 255],
           [255, 255, 255,  ..., 255, 255, 255]],

          [[255, 255, 255,  ..., 255, 255, 255],
           [255, 255, 255,  ..., 255, 255, 255],
           [255, 255, 255,  ..., 255, 255, 255],
           ...,
           [255, 255, 255,  ..., 255, 255, 255],
           [255, 255, 255,  ..., 255, 255, 255],
           [255, 255, 255,  ..., 255, 255, 255]],

          [[255, 255, 255,  ..., 255, 255, 255],
           [255, 255, 255,  ..., 255, 255, 255],
           [255, 255, 255,  ..., 255, 255, 255],
           ...,
           [255, 255, 255,  ..., 255, 255, 255],
           [255, 255, 255,  ..., 255, 255, 255],
           [255, 255, 255,  ..., 255, 255, 255]]]]], dtype=torch.uint8)
>>> vframes.sum()
tensor(20125152368)
```

Is there an issue with conversion, or perhaps something else I am not aware of?
Repository: rust
Issue: Android: Instant not advancing while the screen is off
Labels: O-android,T-libs-api,C-bug,T-libs,A-time

I tried this code on Android:

```rust
let now = Instant::now();
// Wait some hours
now.elapsed().as_secs()
```

I expected to see this happen: Get some hours as a result

Instead, this happened: Got only minutes because the time while the process is sleeping / the app is "dozing" is not counted.

Nice explanation from  mbrubeck (https://users.rust-lang.org/t/std-now-with-android/41774):
`Instant::now()` on Android (and other Linux platforms) is currently implemented using `libc::clock_gettime(CLOCK_MONOTONIC)` . Apparently this clock does not advance while the CPU is in deep sleep modes ("Doze"). This is also how Android's [ `SystemClock.uptimeMillis` ](https://developer.android.com/reference/android/os/SystemClock#uptimeMillis()) method works.

Android also offers a [ `SystemClock.elapsedRealTime` ](https://developer.android.com/reference/android/os/SystemClock#elapsedRealtime()) method, which seems to be implemented using an [Android-specific ioctl ](https://android.googlesource.com/platform/frameworks/native/+/jb-dev/libs/utils/SystemClock.cpp#121). Perhaps it would be possible to migrate the Rust standard library to use this call on Android, if there are no trade-offs in precision.
Repository: godot
Issue: Inconsistent behavior in get_rect() between nodes.
Labels: discussion

**Godot version:**
3.2.1 stable

**Issue description:**
The `get_rect()` method available on a few different types, including Control, Sprite, and Image, do not behave the same.

- `Control.get_rect()`: Returns the _global_ position and size of the object.
- `Sprite.get_rect()`: Returns the _local_ position of the upper right corner relative to its pivot (if a sprite is 64x64 and the pivot is centered, the upper left origin is -32,-32 regardless of its position on the canvas). This does not change if the sprite is resized on any axis resulting in weird coordinates being returned like -13.2566666667,25.7 if local mouse position is polled.
- `Image.get_rect(Rect2 rect)`: Returns a copy of the image from the specified `rect` argument.

I feel this is an inconsistency in the language API that should be smoothed out to avoid confusion with expected behavior by appropriately renaming methods and/or making them behave the same everywhere they are implemented.

An expected and sensible behavior is that `get_rect()` on all types would simply return the global coordinates of the object from its upper left corner and its bounding box size. This not only simplifies this but will also simplify getting/calculating the mouse position within the objects.
Repository: godot
Issue: Creating a texture3D distorts the values (unwanted filtering?)
Labels: topic:core

**Godot version:**
3.2.1.stable.mono

**OS/device including version:**
Windows 10

**Issue description**:
When creating a Texture3D in code, the texture data is distorted. An entered byte value of 1 becomes 13 when read back for example. When doing the same procedure on an ImageTexture with the same setup, this problem does not occur.

**Steps to reproduce:**
Create a Texture3d in code with Image.Format_r8 and flags = 0. 
Create an Image from a poolbytearray containg values between 0 and 255. 
Set the Image as one of the Texture3D's layer data. 
Retrieve the data from the same layer and read the bytearray.

**Minimal reproduction project:**
GD-script, uploaded as .txt. Attach sript to a node and run. 
[Node.txt](https://github.com/godotengine/godot/files/4572974/Node.txt)

*Bugsquad edit: Complete minimal reproduction project for easier testing: [test_texture_3d.zip](https://github.com/godotengine/godot/files/4573269/test_texture_3d.zip)*
Repository: excalidraw
Issue: Tabs in Excalidraw 
Labels: enhancement

Usually I need to work on multiple diagrams at the same time. Tabs should be a good way to work on different workspaces at the same time.
Repository: pytorch
Issue: Documentation of _CtxMethodMixin: must be tensors?
Labels: module: docs,module: autograd,triaged

The documentation for _CtxMethodMixin.save_for_backward seems to imply that it accepts only tensor arguments:

https://github.com/pytorch/pytorch/blob/843c0230f2928aad61a6940688da3a6cd6c4cd57/torch/autograd/function.py#L13

... as does the function name of `saved_tensors`. https://github.com/pytorch/pytorch/blob/843c0230f2928aad61a6940688da3a6cd6c4cd57/torch/autograd/function.py#L386 .   
and this documentation:
https://github.com/pytorch/pytorch/blob/843c0230f2928aad61a6940688da3a6cd6c4cd57/torch/autograd/function.py#L161

But the documentation here:
https://github.com/pytorch/tutorials/blob/f557ee0addc24900929bafd4ceff2fc4c3f47072/beginner_source/examples_autograd/two_layer_net_custom_function.py#L26
indicates that they can be tensors or other types.

I attempted to read the code (which isn't easy as it involves metaclasses) and as far as I can tell they may be tensors or other types.  I think the documentation should be made consistent on this point.

cc @ezyang @SsnL @albanD @zou3519 @gqchen
Repository: rust
Issue: WebAssembly ABI mismatch between clang and rust
Labels: A-FFI,T-compiler,O-wasm,C-bug,A-ABI

When I compile this C code to `wasm32-wasi`:

```c
typedef struct Vector {
    int a;
    int b;
} Vector;

extern int extract_a(Vector v) {
    return v.a;
}
```

And this Rust code to `wasm32-wasi`:

```rust
#[repr(C)]
struct Vector {
    a: i32,
    b: i32,
}

extern "C" {
    fn extract_a(v: Vector) -> i32;
}

fn main() {
    unsafe {
        extract_a(Vector { a: 5, b: 4 });
    }
}
```

And try link them together, I'm getting linking errors:

```
  = note: lld: error: function signature mismatch: extract_a
          >>> defined as (i32, i32) -> i32 in /home/pierre/Projets/wasm-abi-test/target/wasm32-wasi/debug/deps/wasm_abi_test.67xpw7l331r9o5x.rcgu.o
          >>> defined as (i32) -> i32 in /home/pierre/Projets/wasm-abi-test/target/wasm32-wasi/debug/build/wasm-abi-test-9c49ce7f6c5ca031/out/libfoo.a(test.o)
```

It seems that, according to clang, passing a struct by value should inline the fields, while for Rust it should be passed by pointer.

I uploaded an example project here: https://github.com/tomaka/wasm-abi-test
It can be built with something like:
> AR_wasm32_wasi=/path/to/wasi-sdk-10.0/bin/ar CC_wasm32_wasi=/path/to/wasi-sdk-10.0/bin/clang cargo run --target=wasm32-wasi

Rust version: rustc 1.44.0-nightly (38114ff16 2020-03-21)
Repository: PowerToys
Issue: [FancyZones] Allow linking of windows in a layout together to bring them all to the front when one is pressed.
Labels: Idea-Enhancement,Product-FancyZones

# Summary of the new feature/enhancement

I think it would be incredibly useful if windows in a Zone were able to be linked together such that if you brought one to the front it would bring all of them to the front at the same time. I understand that you could have multiple windows in the same section on a zone so there'd need to be some way to say which windows are linked together. This could perhaps be a separate toy all together.

The use case for this is that if a monitor is used for multiple things, some windows may cover part or all of a set of other windows. If two or more windows have been snapped to a zone it's likely they should all be visible at the same time so needing to click each window one by one to bring them all forward again can be tedious. 

I currently don't have any suggestions for how this would look from a UI/UX perspective.
Repository: flutter
Issue: Material Component widget request: multi-select search field with auto-suggestion
Labels: c: new feature,framework,f: material design,c: proposal,P3,team-text-input,triaged-text-input

## Use case
[RecipeRadar](https://www.reciperadar.com/) currently provides the user with the ability to enter multiple ingredients and/or kitchen equipment items that they would like to include or exclude when searching for recipes.

This functionality is built using a server-side auto-suggestion API that responds to the user's query text with suggested valid ingredient and equipment identifiers (and their corresponding human-readable names).

While the user-facing application does not use `flutter`, we are evaluating it as a possible next-generation application framework, particularly as [support for Progressive Web Apps](https://flutter.dev/web) as a target platform continues to develop.

One detractor that we have encountered is that `flutter` does not yet support a multi-selection auto-suggest search input.

We are currently using [select2's multi-select functionality](https://select2.org/getting-started/basic-usage#multi-select-boxes-pillbox) and have discovered existing native client-side functionality in [Android's MultiAutoCompleteTextView](https://developer.android.com/reference/android/widget/MultiAutoCompleteTextView).

## Proposal
A [Material Component widget](https://flutter.dev/docs/development/ui/widgets/material) that [implements](https://github.com/flutter/flutter/tree/0549ab239009e6f76f95881364e935cd345a4da7/packages/flutter/test/material) support for multi-valued text input, with support for auto-suggestion.

- [Android's MultiAutoCompleteTextView](https://developer.android.com/reference/android/widget/MultiAutoCompleteTextView) may provide a useful reference for design, accessibility and implementation concerns
- Support for population of values from a server-side API endpoint is required; we and likely other customers would be greatly aided during system migration if the component would support client-side transformation of the data returned by the endpoint
- User-selectde items will require both a user-accessible text description, and also a service-defined ID value (similar to the distinction between an HTML [option](https://developer.mozilla.org/en-US/docs/Web/HTML/Element/option) field's innerHTML and its `value` attribute)
- Support for purely client-side autosuggestion and tokenization may be valuable for other customers (although it is not currently a requirement that we have)

## Relates to
- https://github.com/flutter/flutter/issues/17119
- https://github.com/flutter/flutter/issues/23827
Repository: deno
Issue: deno install should bundle before installation
Labels: feat,cli

`deno install` is a utility that allows to install any module as an executable by creating shell script that invokes actual module.

Currently it works by caching all dependencies of the module in `$DENO_DIR` and adding entry point local URL to the shell script. 

Huge caveat in this approach is that if user reloads dependencies installed script might break (especially for untagged imports...). 
To mitigate this problem the script should be bundled into a single file during installation. 

This way installed script can be made into standalone utilities that do not share cache with other development projects and are completely standalone.

CC @ry
Repository: pytorch
Issue: CMake Documentation Issue
Labels: module: docs,triaged

It isn't mentioned in PyTorch docs about the Set()
Also, the Cmake file isn't quite complete and needs to be updated. Maybe new users don't know about it.
```
set(CMAKE_PREFIX_PATH "libtorch/share/cmake/Torch")
```
[here](https://pytorch.org/tutorials/advanced/cpp_frontend.html) as well as [here](https://pytorch.org/tutorials/advanced/cpp_export.html)

The CMake file needs to be clear in the documentation. It is causing trouble for many people.
#12449 #21976 #806 #12449

It's not a bug. But a doc improvement issue.
Repository: PowerToys
Issue: Fine-tune multi-monitor positioning tool
Labels: Help Wanted,Idea-New PowerToy,Product-Display management

# Summary of the new feature/enhancement

<!-- 
A clear and concise description of what the problem is that the new feature would solve.
Describe why and how a user would use this new functionality (if applicable).
-->

The current way to position monitors is via Display Settings, but this does really bad snapping as one can see in the following example for a Dell XPS 4k laptop monitor and 2 Dell 1080p external monitors:

![image](https://user-images.githubusercontent.com/85504/80983340-5d175b00-8e24-11ea-83e2-3c7cf3ff718b.png)

(for the record, I've actually raised a Feedback Hub issue on this problem: https://aka.ms/AA6rwp7)

# Proposed technical implementation details (optional)

<!-- 
A clear and concise description of what you want to happen.
-->

Ideally, I would like this screen with a more powerful interface, that would allow me to even specify coordinates on the monitors positioning.

DisplayFusion does provide a tool to do this, but it is a paid utility to fix a Windows problem, and I believe this should be natively provided (or via PowerToys)
Repository: rust
Issue: Incorrect recommended import for renamed dependencies
Labels: A-diagnostics,T-compiler,C-bug,D-papercut,D-invalid-suggestion

This requires two crates: `a` and `b`.
`a/src/lib.rs`:
```rust
pub struct A;
pub trait Foo {
    fn foo(&self);
}
impl Foo for A {
    fn foo(&self) {}
}
```
`b/Cargo.toml`:
```
[package]
name = "b"
version = "0.1.0"

[dependencies]
c = { path = "../a", package = "a" }
```
`b/src/lib.rs`
```rust
fn b() {
    c::A.foo();
}
```

When compiling `b` we get the following error:
```
error[E0599]: no method named `foo` found for struct `a::A` in the current scope
 --> src/lib.rs:2:10
  |
2 |     c::A.foo();
  |          ^^^ method not found in `a::A`
  | 
 ::: /home/programming/f/a/src/lib.rs:3:8
  |
3 |     fn foo(&self);
  |        ---
  |        |
  |        the method is available for `std::boxed::Box<a::A>` here
  |        the method is available for `std::sync::Arc<a::A>` here
  |        the method is available for `std::rc::Rc<a::A>` here
  |
  = help: items from traits can only be used if the trait is in scope
help: the following trait is implemented but not in scope; perhaps add a `use` for it:
  |
1 | use a::Foo;
  |
```

Adding `use a::Foo` is not correct, as we have to use `c::Foo`.
Repository: flutter
Issue: Adding span with Url throws assertion in RichText
Labels: framework,a: typography,found in release: 1.17,P2,team-framework,triaged-framework

In RichText when we add span with `Url` it throws this assertion.
![image](https://user-images.githubusercontent.com/2319867/80993655-4a832e80-8df0-11ea-88b5-e3a08c00af88.png)

## Steps to Reproduce

```
getParsedMessage() {
    if (message.body == null || message.body.isEmpty) return Container();
    List<TextSpan> spans = [];
    var urlPattern =
        r"(https?:\/\/(?:www\.|(?!www))[a-zA-Z0-9][a-zA-Z0-9-]+[a-zA-Z0-9]\.[^\s]{2,}|www\.[a-zA-Z0-9][a-zA-Z0-9-]+[a-zA-Z0-9]\.[^\s]{2,}|https?:\/\/(?:www\.|(?!www))[a-zA-Z0-9]\.[^\s]{2,}|www\.[a-zA-Z0-9]\.[^\s]{2,})";

    String tmp = message.body;
    Iterable<Match> matches = RegExp(urlPattern).allMatches(message.body);
    matches.forEach((m) {
      String match = m.group(0);
      int index = tmp.indexOf(match);

      if (index > 0) {
        spans.add(
            TextSpan(text: tmp.substring(0, index), style: normal14blackish));
      }
      spans.add(TextSpan(
          text: match,
          style: normal14blackish.copyWith(color: Colors.blue),
          recognizer: MultiTapGestureRecognizer()
            ..onTap = (int) async {
              if (await canLaunch(match)) launch(match);
            }));
      tmp = tmp.substring(index).replaceFirst(match, '');
    });
    spans.add(TextSpan(text: tmp, style: normal14blackish));

    return RichText(text: TextSpan(children: spans));
  }
```

## Logs
Found in 1.17.0-3.4.pre.
Repository: flutter
Issue: [Web] _CaretMetrics are incorrect for TextSpans
Labels: a: tests,team,framework,a: typography,platform-web,e: web_html,has reproducible steps,P2,found in release: 3.3,found in release: 3.6,team-web,triaged-web

Following up on three skipped tests in `text_painter_test.dart`. The offset and height of the **_CaretMetrics** are incorrect for these tests on Web.

Test name:  _TextPainter caret emoji test_
https://github.com/flutter/flutter/blob/c32d1387ab9a6cba7a4f9849dce1d6df63f5a173/packages/flutter/test/painting/text_painter_test.dart#L124

```md
02:01 +245 ~14 -1: /tmp/flutter sdk/packages/flutter/test/painting/text_painter_test.dart: TextPainter caret emoji test [E]                                                                            
  Expected: <42>
    Actual: <14>
  package:dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 216:49      throw_
  package:test_api                                                                          expect$
  package:flutter_test/src/widget_tester.dart 364:3                                         expect$
  text_painter_test.dart 80:5       
```

---

Test name:  _TextPainter caret center space test_
https://github.com/flutter/flutter/blob/c32d1387ab9a6cba7a4f9849dce1d6df63f5a173/packages/flutter/test/painting/text_painter_test.dart#L146

```md
02:01 +245 ~14 -2: /tmp/flutter sdk/packages/flutter/test/painting/text_painter_test.dart: TextPainter caret center space test [E]                                                                     
  Expected: <21>
    Actual: <0>
  package:dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 216:49      throw_
  package:test_api                                                                          expect$
  package:flutter_test/src/widget_tester.dart 364:3                                         expect$
  text_painter_test.dart 135:5                                                              <fn>
```

---

Text name:  _TextPainter caret height and line height_
https://github.com/flutter/flutter/blob/c32d1387ab9a6cba7a4f9849dce1d6df63f5a173/packages/flutter/test/painting/text_painter_test.dart#L835

```md
02:02 +253 ~16 -4: /tmp/flutter sdk/packages/flutter/test/painting/text_painter_test.dart: TextPainter caret height and line height [E]                                                                
  Expected: <50>
    Actual: <14>
  package:dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 216:49      throw_
  package:test_api                                                                          expect$
  package:flutter_test/src/widget_tester.dart 364:3                                         expect$
  text_painter_test.dart 833:5                                                              <fn>
```
Repository: react
Issue: Bug: devtools "rendered at" for child components uses value of top component
Labels: Status: Unconfirmed,Component: Developer Tools,React Core Team

React version: 17.0.2
devtools version: 4.18.0-f58bbcf9a

## Steps To Reproduce

1. Record profiling session
2. Goto flamegraph
3. Click on a "bar" that's not the top-most

Link to code example: https://codesandbox.io/s/vibrant-thunder-bm3so

## The current behavior

"rendered at" uses the value from the root component

![devtools-profiler-rendered-at](https://i.ibb.co/pPfBCHM/devtools-profiler-rendered-at.png)




## The expected behavior

it should use the same value that is displayed in the tooltip. Otherwise I don't see much value in displaying it since I could just read the value directly from the root. I thought it was intended for very dense flamegraphs e.g.
![Screenshot from 2020-05-04 19-53-53](https://user-images.githubusercontent.com/12292047/80996992-02402c80-8e41-11ea-9396-f3777c01cf4d.png)
Repository: pytorch
Issue: Guard Gloo and TensorPipe related code in RPC with #ifdef
Labels: module: bootcamp,triaged,module: rpc,module: tensorpipe

As we have compile flags like `USE_GLOO` and `USE_TENSORPIPE` that can disable Gloo and TensorPipe independently when `USE_DISTRIBUTED=1`, RPC code (e.g., in `rpc/init.cpp`) should not expect `process_group_agent` and `tensor_pipe_agent` are always available. They should be guarded by `#ifdef`.

cc @pietern @mrshenli @pritamdamania87 @zhaojuanmao @satgera @gqchen @aazzolini @rohan-varma @xush6528 @jjlilley @osalpekar @jiayisuse @lw @beauby
Repository: go
Issue: runtime: nmspinning's position in  struct schedt 
Labels: NeedsInvestigation,compiler/runtime

i think the nmspinning's position in  struct schedt in runtime should be writen in the "m" part instead of "p" part

current is
```
	midle        muintptr // idle m's waiting for work
	nmidle       int32    // number of idle m's waiting for work
	nmidlelocked int32    // number of locked m's waiting for work
	mnext        int64    // number of m's that have been created and next M ID
	maxmcount    int32    // maximum number of m's allowed (or die)
	nmsys        int32    // number of system m's not counted for deadlock
	nmfreed      int64    // cumulative number of freed m's

	ngsys uint32 // number of system goroutines; updated atomically

	pidle      puintptr // idle p's
	npidle     uint32
	nmspinning uint32 // See "Worker thread parking/unparking" comment in proc.go.

```
i think it should like this:
```
	midle        muintptr // idle m's waiting for work
	nmidle       int32    // number of idle m's waiting for work
	nmidlelocked int32    // number of locked m's waiting for work
	mnext        int64    // number of m's that have been created and next M ID
	maxmcount    int32    // maximum number of m's allowed (or die)
	nmsys        int32    // number of system m's not counted for deadlock
	nmfreed      int64    // cumulative number of freed m's
	nmspinning uint32 // See "Worker thread parking/unparking" comment in proc.go.

	ngsys uint32 // number of system goroutines; updated atomically

	pidle      puintptr // idle p's
	npidle     uint32
```