sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: deno
Issue: Nondeterministic behavior when specifying `max-old-space-size`
Labels: perf,deno_core,needs investigation

I've noticed a weird problem when specifying `--max-old-space-size`. There are times when my test program fails with an OOM, but then works if I change the flag to something that logically shouldn't affect the outcome.

For example:

1. Run with `--max-old-space-size=32` *(fails)*
2. Run with `--max-old-space-size=128` *(succeeds)*
3. Run with `--max-old-space-size=32` again *(succeeds for some reason)*

The code I'm testing this out with is:

```typescript
Deno.test('handles large files', async () => {
  const sizeInMb = 64;
  const filename = await Deno.makeTempFile({ prefix: 'denotestfile' });
  try {
    const chunk = new Uint8Array(1024 * 1024);
    chunk.fill(170);
    for (let i = 0; i < sizeInMb; i++) {
      await Deno.writeFile(filename, chunk, { append: true });
    }
  } finally {
    await Deno.remove(filename);
  }
});

/*
Run via:
deno test -A this_test.ts --v8-flags=--max-old-space-size=32
*/
```

After step 3 above, I can get back into the error state by **increasing or decreasing** the program's `sizeInMb` value. The fact that even decreasing the file size causes the error to return is incredibly baffling to me.

Also, unless I'm missing something huge about the `Deno.writeFile` API, this program looks like it shouldn't be using much more than 1MB of memory anyways (the size of the chunk it builds, which gets written to disk many times). I'm confused about why that would ever fail with `--max-old-space-size=32` in the first place.
Repository: PowerToys
Issue: Move Window to Main Display
Labels: Idea-New PowerToy,Product-Window Manager,Needs-Triage

### Description of the new feature / enhancement

If you use multiple displays one handy feature is to have a shortcut to move the currently active window to the main display or simple cycle throw displays. 



### Scenario when this would be used?

Multiple displays, quickly re-arrange windows.
Having a problem with 1 display not working and you need to move a window from one to another.

### Supporting information

Currently using Dual Monitor Tool for this job.
Repository: neovim
Issue: nvim_open_win not entered when called from autocommand callback BufWritePost
Labels: bug-vim,needs:discussion,has:workaround,events

### Neovim version (nvim -v)

v0.8.0-dev-750-g68ec497d5

### Vim (not Nvim) behaves the same?

No (lua api)

### Operating system/version

macOS 12.0.1

### Terminal name/version

kitty 0.24.4

### $TERM environment variable

xterm-256color-italic

### Installation

build from repo

### How to reproduce the issue

Create a lua-file with this content. Then run it via `:source %` and save the buffer `:w`. 

```lua
local group = vim.api.nvim_create_augroup("windows-test", { clear = true })

vim.api.nvim_create_autocmd("BufWritePost", {
  group = group,
  callback = function()
    local buf = vim.api.nvim_create_buf(false, false)

    vim.api.nvim_buf_set_lines(buf, -1, -1, false, { "Hello there!" })
    vim.api.nvim_open_win(buf, true, {
      relative = "win",
      win = vim.api.nvim_get_current_win(),
      width = 30,
      height = 4,
      col = 2,
      row = 2
    })
  end
})
```

### Expected behavior

Using `true` as the second argument for `nvim_open_win`, I expect the new window to be entered. It works as expected when moving those lines outside the callback.

### Actual behavior

The cursor stays in the window and does not enter the floating window.
Repository: material-ui
Issue: [Dialog]: Not opened dialog renders when it's wrapped in Suspense component 
Labels: component: dialog

### Duplicates

- [X] I have searched the existing issues

### Latest version

- [X] I have tested the latest version

### Current behavior 😯

Hi there,

I have two related dialogs wrapped in React Suspense component and there is a component triggers suspense inside second dialog. Both dialogs are rendered in that case even if the `open` prop is `false`. 

Please see [Codesandbox example](https://codesandbox.io/s/sharp-driscoll-pp3nt8?file=/src/App.tsx) to see details. 
Thanks 

### Expected behavior 🤔

All dialogs with falsy `open` props should not be rendered 

### Steps to reproduce 🕹

_No response_

### Context 🔦

_No response_

### Your environment 🌎

<details>
  <summary><code>npx @mui/envinfo</code></summary>

```
  System:
    OS: macOS 11.6.4
  Binaries:
    Node: 18.0.0 - /usr/local/bin/node
    Yarn: 1.22.19 - ~/.yarn/bin/yarn
    npm: 8.6.0 - /usr/local/bin/npm
  Browsers:
    Chrome: 103.0.5060.134
    Edge: Not Found
    Firefox: 92.0
    Safari: 15.3
  npmPackages:
    @emotion/react: ^11.9.0 => 11.9.3 
    @emotion/styled: ^11.8.1 => 11.9.3 
    @mui/base:  5.0.0-alpha.91 
    @mui/icons-material: ^5.8.4 => 5.8.4 
    @mui/lab: ~5.0.0-alpha.91 => 5.0.0-alpha.92 
    @mui/material: ~5.9.1 => 5.9.2 
    @mui/private-theming:  5.9.1 
    @mui/styled-engine:  5.8.7 
    @mui/system:  5.9.2 
    @mui/types:  7.1.5 
    @mui/utils:  5.9.1 
    @types/react: ^18.0.9 => 18.0.15 
    react: ^18.1.0 => 18.2.0 
    react-dom: ^18.1.0 => 18.2.0 
    typescript: ^4.7.3 => 4.7.4 

```
</details>
Repository: pytorch
Issue: Deep copy models with `create_feature_extractor` produces different parameters
Labels: triage review,triaged,module: vision,oncall: fx

### 🐛 Describe the bug

When I define a model using the new feature `create_feature_extractor` and then deep copy the model, the parameters of the original model and the new models are different.

Here is an example:

```python
from torchvision.models.feature_extraction import create_feature_extractor
import torch.nn as nn
import torchvision
import copy

class cusResNet18(nn.Module):    
    def __init__(self, n_classes, pretrained = True):
        super(cusResNet18, self).__init__()
        self.resnet = torchvision.models.resnet18(pretrained=pretrained)
        self.resnet.fc = nn.Linear(512, n_classes)
        
        self.avgpool = self.resnet.avgpool
        
        self.returnkey_avg = 'avgpool'
        self.returnkey_fc = 'fc'
        self.body = create_feature_extractor(
            self.resnet, return_nodes={'avgpool': self.returnkey_avg, 'fc': self.returnkey_fc})

    def forward(self, x):
        outputs = self.body(x)
        return outputs[self.returnkey_fc], outputs[self.returnkey_avg].squeeze()
    
model = cusResNet18(n_classes=1)
copied_model = copy.deepcopy(model)

print(len(list(model.parameters())), len(list(copied_model.parameters())))
```
The output is ```62 124```.

If printing out the named_parameters, the differences are from the defined self.body.
```
print(dict(model.named_parameters()).keys(), dict(copied_model.named_parameters()).keys())
```

I'm wondering why there is a difference after using deepcopy and how can I deepcopy a model that has the create_feature_extractor feature inside?

### Versions


torch==1.10.2
torchvision==0.11.3

cc @fmassa @vfdev-5 @pmeier @ezyang @SherlockNoMad
Repository: pytorch
Issue: contiguous() not work for rank 1 length 1 tensor.
Labels: triaged,module: dlpack

### 🐛 Describe the bug

When I try `torch.tensor([1.+2.j, 3.+4.j]).real.contiguous().stride()`, it returns `(1,)` as expected.

But if the length is only 1, the stride is not 1 but 2, that is to say `torch.tensor([1.+2.j]).real.contiguous().stride()` gives `(2,)`

Although stride of length 1 tensor has no effect. But when I try to use mpi4py to bcast/allreduce tensor, it will throw an error because of wrong stride.

### Versions

Collecting environment information...
PyTorch version: 1.12.0
Is debug build: False
CUDA used to build PyTorch: None
ROCM used to build PyTorch: N/A

OS: Arch Linux (x86_64)
GCC version: (GCC) 12.1.0
Clang version: 14.0.6
CMake version: version 3.23.3
Libc version: glibc-2.35

Python version: 3.10.5 (main, Jun  6 2022, 18:49:26) [GCC 12.1.0] (64-bit runtime)
Python platform: Linux-5.18.12-arch1-1-x86_64-with-glibc2.35
Is CUDA available: False
CUDA runtime version: No CUDA
GPU models and configuration: No CUDA
Nvidia driver version: No CUDA
cuDNN version: No CUDA
HIP runtime version: N/A
MIOpen runtime version: N/A
Is XNNPACK available: True

Versions of relevant libraries:
[pip3] mypy==0.961
[pip3] mypy-extensions==0.4.3
[pip3] numpy==1.23.1
[pip3] torch==1.12.0
[conda] Could not collect

cc @ezyang @anjali411 @dylanbespalko @mruberry @Lezcano @nikitaved
Repository: flutter
Issue: [camera_web]: iOS Safari browser blob url is not playing the video if the duration is more than 60 seconds
Labels: platform-web,p: camera,package,has reproducible steps,P2,browser: safari-ios,found in release: 3.0,found in release: 3.1,team-web,triaged-web

When a video is recorded using camera plugin, a blob url is returned on web platform.
When you use this blob url to play the video either using the package video_player or using the HtmlElementView with Video tag on the web, it plays the video.

The issue arises when the recorded video duration is more than 60 seconds and the issue is on iOS Safari browser.

Steps to reproduce:
1. Build the example app for web using the canvaskit renderer. **_Note: I don't know if the issue is same with html renderer or not._**
2. Deploy the web build.
3. Run the deployed build on iOS device Safari browser.
4. Record a video of more than 60 seconds. 
5. See that the video_player will not be able to play the recorded video from the returned blob url.
6. I have added the size of the recorded video in the Snackbar after the recording is stopped. Check the below snippet.
7. When the video duration is more than 60 seconds, the size returned is 0 bytes.

I have already deployed the example app at https://deepak786.github.io/camera_issue/#/

In the build I have replaced the following code

```
void onStopButtonPressed() {
    stopVideoRecording().then((XFile? file) {
      if (mounted) {
        setState(() {});
      }
      if (file != null) {
        showInSnackBar('Video recorded to ${file.path}');
        videoFile = file;
        _startVideoPlayer();
      }
    });
  }
```

with

```
void onStopButtonPressed() {
    stopVideoRecording().then((XFile? file) async {
      if (mounted) {
        setState(() {});
      }
      if (file != null) {
        videoFile = file;
        String size = formatBytes(await file.length(), 2);
        showInSnackBar('Video recorded to ${file.path} having size $size');
        _startVideoPlayer();
      }
    });
  }

  /// Function to format the bytes in KB, MB, GB, etc.
  String formatBytes(int bytes, int decimals) {
    if (bytes <= 0) return "0 B";
    const suffixes = ["B", "KB", "MB", "GB", "TB", "PB", "EB", "ZB", "YB"];
    var i = (log(bytes) / log(1024)).floor();
    return '${(bytes / pow(1024, i)).toStringAsFixed(decimals)} ${suffixes[i]}';
  }
```


Note: If you record a video of small duration then a Widget is shown at the bottom right corner as a VideoPlayer. Thats the functionality of example app already.
Repository: vscode
Issue: Multiple comment decorations rendered when word wrap is turned on
Labels: bug,comments

<img width="184" alt="image" src="https://user-images.githubusercontent.com/876920/182255377-2219dbf7-8a0c-4a0b-8e6c-9044b81b937f.png">


When word wrap is on, we render comment icon for all wrapped lines. Ideally we should only render one on hover.
Repository: flutter
Issue: Flutter-web: add annotation/semantics to indicate button opens a pop-up (like `aria-haspopup`)
Labels: framework,engine,a: accessibility,platform-web,customer: google,P2,team-web,triaged-web

## Use case

We have a collection of menu and non-menu buttons in a custom on-screen keyboard. Each button opens a grid of options. In the process of making a custom menu button and testing it on Flutter web, we noticed that the button provides no indication that activating it pops up a menu. On native web, we would supply an `aria-haspopup` attribute (with value `"true"`, `"menu"`, `"list"`, `"grid"`, etc.) on the button and the screen reader would surface that to the end user (e.g., by vocalizing, e.g., "has popup" after the menu button label). There doesn't seem to be an equivalent annotation in Flutter.

1. None of [these Flutter UI semantics](https://api.flutter.dev/flutter/widgets/Semantics/Semantics.html) seem to apply.
2. Looking at the implementation of [Flutter UI's PopupMenuButton](https://github.com/flutter/flutter/blob/a76599adc4e21bdf31095ada32311cd4f3b05452/packages/flutter/lib/src/material/popup_menu.dart#L1005-L1274), I couldn't find any way to indicate that the button will open a pop-up when activated.

[popupmenubutton]: https://github.com/flutter/flutter/blob/a76599adc4e21bdf31095ada32311cd4f3b05452/packages/flutter/lib/src/material/popup_menu.dart#L1005-L1274

## Proposal

Please add some semantics analogous to the one on Web as typically for accessible UIs, it's considered an antipattern to add widget type or metadata ("has popup") to the label; instead, the screen reader does its own custom vocalization based on user verbosity settings.

cc: @goderbauer who suggested I file this here.
Repository: pytorch
Issue: FakeTensor Support For Pickling
Labels: triaged,module: fakeTensor

### 🐛 Describe the bug

See: https://github.com/pytorch/PiPPy/issues/298#issuecomment-1201790838

Needed for distributed use.

### Versions

master
Repository: flutter
Issue: Make go() asynchronous
Labels: package,c: proposal,P3,p: go_router,team-go_router,triaged-go_router

If we are making redirection asynchronous (https://github.com/flutter/flutter/issues/105808) we should also make go() and goNamed() asynchronous. We should also make push and pushNamed asynchronous, but it's not specified how redirection is handled for push methods.

Context: https://github.com/flutter/flutter/issues/105808#issuecomment-1153886855
Repository: go
Issue: runtime: occasional "exit status 0xc0000005" on windows-386-2012 builder
Labels: OS-Windows,NeedsInvestigation,compiler/runtime

```
#!watchflakes
post <- pkg == "runtime" && goos == "windows" && goarch == "386" && `exit status 0xc0000005`
```

Observed once in a TryBot run on [CL 420675](https://go-review.googlesource.com/c/go/+/420675/comments/a8acc56f_942c8a72):

```
exit status 0xc0000005
FAIL	runtime	18.548s
```

(https://storage.googleapis.com/go-build-log/1d5e997e/windows-386-2012_858cca10.log)

Searching around doesn't find this particular exit code, though there are similar ones like in issue #41450. That exit code seems to be a pretty generic "Application Error" failure, so given the rare occurrence maybe there's not much that can be done.

CC @golang/runtime.
Repository: vscode
Issue: Restarting a Debug Adapter doesn't reset the Visual Studio Interface
Labels: bug,debug

1) Create a debug adapter which supports restart by specifying `supportsRestartRequest`.
2) Stop at a breakpoint such that the Variables and Call Stack interfaces are populated
3) Press the Restart button in Visual Studio Code

Expected:
The Visual Studio Code interface is reset when it gets back the empty response from the Restart request message

Actual:
The Visual Studio Code interface remains stale after getting sent a response from the Restart request.
Repository: flutter
Issue: Implement mouse context menu on Android
Labels: a: text input,framework,c: proposal,P3,team-framework,triaged-framework

It turns out that Android has a different context menu (text selection menu) when using a mouse vs. when using tap gestures.  If you right click on a text field on native Android, this is what you get:

<img width="300" src="https://user-images.githubusercontent.com/389558/182268267-60b29102-c885-4912-ae35-cce6eb8d4876.png" />

Flutter doesn't currently implement this menu at all!  We should distinguish between mouse and tap gestures on Android and display the correct menu.

### Open questions
Is there a separate menu when using a mouse on iOS too?

Related: https://github.com/flutter/flutter/issues/106586
Repository: rust
Issue: Normalization can skip WF
Labels: I-ICE,P-medium,A-associated-items,I-unsound,C-bug,glacier,S-bug-has-test,T-types,fixed-by-next-solver

I tried this code: [[playground]](https://play.rust-lang.org/?version=nightly&mode=debug&edition=2021&gist=c5cd930a4dee00e88a93712e13ee57e0)

```rust
trait WellUnformed {
    type RequestNormalize;
}

impl<T: ?Sized> WellUnformed for T {
    type RequestNormalize = ();
}

const _: <[[[[[[u8]]]]]] as WellUnformed>::RequestNormalize = ();
const _: <Vec<str> as WellUnformed>::RequestNormalize = ();
```

I expected this to error; `[[[[[[u8]]]]]]` is clearly not a well-formed type, as
> error[E0277]: the size for values of type `[u8]` cannot be known at compilation time
> = help: the trait `Sized` is not implemented for `[u8]`
> = note: slice and array elements must have `Sized` type

`Vec<str>` is perhaps even more cursed.

Instead: it compiles.

This is especially relevant for blanket/builtin traits which have associated types, such as `marker::DiscriminantKind` and `ptr::Pointee`.

- Can cause ICE: https://github.com/rust-lang/rust/issues/100041#issuecomment-1586516771
- Believed to not be unsound in isolation, as the non-WF type never "fully" exists.

### Meta

1.64.0-nightly (2022-07-31 f9cba63746d0fff81625)

@rustbot label +A-associated-items

cc @eddyb who said this was
@rustbot label +I-unsound
Repository: create-react-app
Issue: Why is create-react-app still used by many? Very important
Labels: needs triage

## Disclaimer: It's not my intent to offend the maintainers of create-react-app, but tools are evolving and it's no doubt that people (who uses newer tools like nextjs or vite) will also ask this question.

### tl;dr
With numerous problems encountered by people in using create-react-app, why are there still many people who are using it? 
It doesn't even have features like hot module reloading and it forces a bunch of testing packages.
I think that cra isn't a viable option to be used by a bunch of new developers nowadays *unless you think that create-react-app is more convenient* (**which is actually not**).

## Here are 3 alternatives for cra:

### 1. Vite (Amateur to advanced)

So if a bunch of new devs read this issue, I would really recommend you switch to vite. Not only does vite have hot module replacement, it also simplifies making a react app (no testing libraries and such included in starter project).

To get started at using it, run one of the following command depending on what package manager you use:
```
# npm 6.x
npm create vite@latest my-react-app --template react

# npm 7+, extra double-dash is needed:
npm create vite@latest my-react-app -- --template react

# yarn
yarn create vite my-react-app --template react

# pnpm
pnpm create vite my-react-app --template react
```
Follow the instructions after running command and you have a react app with hmr enabled. [Official documentation](https://vitejs.dev/guide/)

For people who are building single page apps by default, this is a good option because as soon as your app grows in complexity, the rebundling of dependencies becomes longer, and therefore, decreased productivity.

### 2. Next.js (Level: Intermediate to advanced)

Aside from hot module reloading, nextjs supports a bunch of rendering patterns like server-side rendering, incremental static regeneration, static site generation, and also client-side rendering. 

The option to process everything on the server instead of the client gives the application search engine optimization

To get started, run:
```
npx create-next-app@latest
# or
yarn create next-app
# or
pnpm create next-app
```
And you have an application that can be indexed by google search and will be visible to many. More on [Next.js Official documentation](https://nextjs.org/docs)

### 3. Remix (Level: Intermediate to advanced)
Sometimes, project requirements are a lot bigger than what nextjs can offer. And for that, remix is the option.

Even though remix only supports client-side rendering and server-side rendering, remix has nested layouts. 

So instead of rendering the entire page when redirected with a link, remix only renders on part of the ui. This is great for dashboard-style applications.

To start a remix project, run:
`npx create-remix@latest`
And answer a bunch of questions to scaffold your app. more information [Remix documentation](https://remix.run/docs/en/v1)
Repository: godot
Issue: GraphEdit's child order changes after moving GraphNodes
Labels: documentation,topic:gui

### Godot version

3.4.3-stable_mono

### System information

Windows 10

### Issue description

After dragging a GraphNode it becomes the last child of a GraphEdit.
Might not strictly speaking be a bug, but it seems very arbitrary and messes with basic interation. Same goes for CLAYER and GraphEditFilter and other various "hidden" nodes (#7142) which aren't particularly well-documented.

### Steps to reproduce

A simple for loop iterating over the children in a GraphEdit script.
```
var children = GetChildren();
GD.Print(children);
for (int i = 0; i < children.Count; i++)
{
    if (children[i] is GraphNode){
        GD.Print(current_node.Name);
    }
}
```
Compare four initially created GraphNodes named 0 through 3
```
[CLAYER:[Control:1246], [GraphNode:1384], [GraphNode:1422], [GraphNode:1460], [GraphNode:1498], @@2:[GraphEditFilter:1245]]
0
1
2
3
```
And their order after moving nodes 3, 2, then 1
```
[[GraphNode:1384], CLAYER:[Control:1246], [GraphNode:1498], [GraphNode:1460], [GraphNode:1422], @@2:[GraphEditFilter:1245]]
0
3
2
1
```

### Minimal reproduction project

_No response_