sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: pytorch
Issue: detectron2_fcos_r_50_fpn and other models have enough graph breaks that we end up with multiple cache entries on module blocks
Labels: triaged,module: dynamic shapes

### 🐛 Describe the bug

If you run detectron2_fcos_r_50_fpn with automatic_dynamic_shapes, you'll notice that there are dynamic shapes recompilations, which aren't really true dynamic shape recompilations:

```
[2023-06-15 05:43:07,747] torch.fx.experimental.symbolic_shapes: [INFO] 12.0: create_env
[2023-06-15 05:43:08,412] torch.fx.experimental.symbolic_shapes: [INFO] 12.0: produce_guards
[2023-06-15 05:43:08,423] torch.fx.experimental.symbolic_shapes: [INFO] 12.1: create_env
[2023-06-15 05:43:08,425] torch._dynamo.variables.builder: [WARNING] automatic dynamic L['input'] size(1) 256 != 64
[2023-06-15 05:43:08,425] torch.fx.experimental.symbolic_shapes: [INFO] 12.1: create_symbol s0 = 256 for L['input'].size()[1]
[2023-06-15 05:43:08,525] torch.fx.experimental.symbolic_shapes: [INFO] 12.1: eval Eq(s0, 256) [guard added] at home/ezyang/local/b/pytorch-env/lib/python3.10/site-packages/detectron2/layers/wrappers.py:106 in forward (_subclasses/fake_tensor.py:612 in conv)
[2023-06-15 05:43:09,075] torch.fx.experimental.symbolic_shapes: [INFO] 12.1: produce_guards
```

Instead, they're due to the architecture of detectron2_fcos_r_50_fpn, which stacks a number of similar blocks together but with differing parameters. For Dynamo's intents and purposes, these are all using the same code block, and so each block with differing configuration all gets chucked in the same code cache.

I think if our policy is that we want to specialize on every unique parameter size (this seems reasonable to me), it would be best if we could arrange for our compiled code to NOT live on all the same code object.

@anijain2305 also mentioned to me that he noticed this could contribute to guard overhead, although ISTR that in the end it wasn't clear how much this actually mattered in the end.

### Versions

main
Repository: next.js
Issue: next/font/local preloads woff fonts
Labels: bug,Font (next/font)

### Verify canary release

- [X] I verified that the issue exists in the latest Next.js canary release

### Provide environment information

```bash
Operating System:
      Platform: darwin
      Arch: x64
      Version: Darwin Kernel Version 22.5.0: Mon Apr 24 20:51:50 PDT 2023; root:xnu-8796.121.2~5/RELEASE_X86_64
    Binaries:
      Node: 18.16.0
      npm: 9.5.1
      Yarn: N/A
      pnpm: N/A
    Relevant packages:
      next: 13.4.5
      eslint-config-next: 13.4.4
      react: 18.2.0
      react-dom: 18.2.0
      typescript: 5.1.3
```


### Which area(s) of Next.js are affected? (leave empty if unsure)

Font optimization (next/font)

### Link to the code that reproduces this issue or a replay of the bug

codesandbox doesn't work anymore, sorry

### To Reproduce

```js
const CircularStd = localFont({
  src: [
    {
      path: '../public/fonts/legacy/CircularStd-Book.woff2',
      weight: '400',
      style: 'normal',
    },
    {
      path: '../public/fonts/legacy/CircularStd-Book.woff', // This should never get preloaded
      weight: '400',
      style: 'normal',
    },
  ]
})
```

### Describe the Bug

When using `next/font/local` to declare both woff2 and woff sources of a font, all of them get added to the preloading routine.

However, browsers that don't support woff2, don't support preloading anyway, so there's no need to preload woff files.
It's also causing a lot of extra (unnecessary) network requests.

### Expected Behavior

`.woff` files to never be preloaded.

### Which browser are you using? (if relevant)

_No response_

### How are you deploying your application? (if relevant)

_No response_
Repository: neovim
Issue: gf (<cfile>) should handle "file:" scheme
Labels: enhancement,complexity:low,jumps-navigation

### Problem

Neovim sometimes inclues a prefix to the filename for **gf** command when it shouldn't. It seems to happen only with an absolute path and not with a relative path.

<file1.txt>
file:/abs/path/to/tmp.txt

Action:
set isfname-=:
type "gf" when inside the file path

Result:
nvim tries to open "file:/abs/path/to/tmp.txt" instead of "/abs/path/to/tmp.txt"

### Steps to reproduce

<file1.txt>
file:/abs/path/to/tmp.txt

Action:
nvim --clean
set isfname-=:
type "gf" when inside the file path


### Expected behavior

nvim should open the file path without the leading "file:"

### Neovim version (nvim -v)

0.9.1 release

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

no, vim 9.0.1452

### Operating system/version

Red Hat Enterprise Linux Server release 7.9

### Terminal name/version

gnome-terminal

### $TERM environment variable

gnome-terminal

### Installation

appimage
Repository: angular
Issue: Should LocationStrategy.onPopState filter with APP_BASE_HREF ?
Labels: area: common

### Which @angular/* package(s) are relevant/related to the feature request?

router

### Description

_Context_
We are using web components to create independant micro frontends (we don't use module federation - no negociations).

All apps are Angulars and we are using hash strategy urls.

We have the root app on `mydomain.com/#/root-app` and it has sub routes on its own. Its APP_BASE_HREF is `/`.

Some routes redirect to a specific component that loads a web component from distance (load the JS bundle, inject it in head and insert in HTML the correct HTML tag to display the web component app).

So we have a child app on `mydomain.com/#/root-app/child-app`. Its APP_BASE_HREF is `/root-app/child-app`.

_The problem_
I go on the child-app route `mydomain.com/#/root-app/child-app`, it properly loads the child-app, I can navigate in it, no problems. If I'm navigating through Angular's router no problems, I can go in my root-app, the child-app's router does not react.

But, when I'm in my child-app, if I'm modifying the browser url to point to another route of my root-app, the child-app's router is triggered, and cause errors (redirection to a custom made 404 page, and/or errors in console logs).

I've found nothing in Angular 15 code that filters popState.

### Proposed solution

I'm wondering if Angular should do the filtering (like done in my alternative solution), or provide a clean way to active/define it.

### Alternatives considered

So in my child-app, I came up with this solution, to filter the `LocationStrategy.onPopState` before sending the event to the router.

```typescript
// there are  // @ts-ignore because I'm accessing private properties
@Injectable()
export class CustomHashLocationStrategy extends HashLocationStrategy {

    constructor(
        platformLocation: PlatformLocation,
        @Optional() @Inject(APP_BASE_HREF) baseHref?: string) {
        super(platformLocation, baseHref);
    }

    override onPopState(fn: LocationChangeListener): void {
        const filteredFn = (event: any) => {
            // we only execute the callback if the browser url contains the this._baseHref
            // @ts-ignore
            if (location.hash.indexOf(this._baseHref) !== -1) {
                fn(event);
            }
        };
        super.onPopState(filteredFn);
    }
}

NgModule({
    ...
    providers: [
        {provide: APP_BASE_HREF, useValue: environment.baseUrl},
        {provide: LocationStrategy, useClass: CustomHashLocationStrategy},
        ...
    ]
})

```
Repository: godot
Issue: High baseline energy impact on iOS when a Control node is present on screen (even if empty)
Labels: bug,platform:ios,topic:rendering,confirmed,performance

### Godot version

v4.1.beta1.official [828ec2c5d]

### System information

iPhone 13 Pro Max - iOS 16.3.1

### Issue description

Running the project on my iPhone, Xcode Debug shows a high energy consumption on Godot the moment there is one GUI control, such as a Panel, even if it's empty and no code is running. The GPU goes insane.

### Without PanelContainer:

<img width="1220" alt="Screenshot 2023-06-15 at 4 42 38 PM" src="https://github.com/godotengine/godot/assets/905216/950967be-772d-4d11-80ae-9b8b9332377b">

### With PanelContainer:

<img width="1216" alt="Screenshot 2023-06-15 at 4 41 49 PM" src="https://github.com/godotengine/godot/assets/905216/bdb25610-7b3f-446c-94a5-95c6939425ac">

It sounds excessive that just adding a panel apparently shoots up the GPU usage to 80% (mind it's relative) and energy consumption to High. Unless this is an expected overhead and isn't making actually running the game any slower.

### Steps to reproduce

Setup a structure like this:

<img width="355" alt="Screenshot 2023-06-15 at 4 30 59 PM" src="https://github.com/godotengine/godot/assets/905216/b4e24f7d-210e-4148-832c-3a6f190335bd">

Remove "PanelContainer" for the consumption to go from High to Low.

### Minimal reproduction project

N/A
Repository: godot
Issue: Black artifacts when vrs mode non-disabled and window resized
Labels: bug,topic:rendering,confirmed,topic:3d

### Godot version

v4.1.beta2.official [a2575cba4] (Happens in Beta 1 as well)

### System information

Godot v4.1.beta2 - Windows 10.0.22621 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 4090 (NVIDIA; 31.0.15.3623) - AMD Ryzen 9 7950X 16-Core Processor (32 Threads)

### Issue description

When window is resized and `rendering/vrs/mode` is set to either Texture or XR and no texture is provided, this happens:

![kuva](https://github.com/godotengine/godot/assets/40064911/3e150a56-59cc-4f39-963a-fc467c5e84e1)

Cool effect but I'd rather have it optional.

### Steps to reproduce

^

### Minimal reproduction project

[black_flicker.zip](https://github.com/godotengine/godot/files/11759636/black_flicker.zip)
Repository: node
Issue: V8 Inspector is not available in --require in `node --test`
Labels: inspector,test_runner

### Version

v20.3.0

### Platform

Darwin MacBook-Pro-2.local 22.5.0 Darwin Kernel Version 22.5.0: Mon Apr 24 20:51:50 PDT 2023; root:xnu-8796.121.2~5/RELEASE_X86_64 x86_64

### Subsystem

test_runner

### What steps will reproduce the bug?

1. Have a file **start-inspect.js** containing:

```js
const inspector = require("node:inspector");
inspector.open();
```

2. Have some other `file.test.js`, which can be empty.
3. Run `NODE_OPTIONS="--require ./start-inspect.js" node --test ./hello.test.js`

### How often does it reproduce? Is there a required condition?

100%

### What is the expected behavior? Why is that the expected behavior?

The inspector should open as usual, as if `--test` was not given in the CLI

### What do you see instead?

```
$ NODE_OPTIONS="--require ./start-inspect.js" node --test ./hello.test.js

node:inspector:181
  open(port, host);
  ^
This Environment was initialized without a V8::Inspector
(Use `node --trace-uncaught ...` to show where the exception was thrown)

Node.js v20.3.0
```

### Additional information

This does not happen if `--test` is omitted from the command. This causes issues with the VS/Code debugger which uses this mechanism to initiate debugging. I think Jetbrains does the same thing, so Webstorm and friends probably have the same issue, though I have not tested it.

Also, if I try to open the inspector in the test file itself, it works fine, so it seems like there might be some initialization ordering that's different for `--test`.

Originally reported on https://github.com/microsoft/vscode-js-debug/issues/1727
Repository: PowerToys
Issue: Edit PDF file properties
Labels: Idea-Enhancement,Idea-New PowerToy,Product-File Explorer,Needs-Triage,Product-File Actions Menu

## Description of the new feature / enhancement

I would like to suggest that File Explorer need to have the feature to edit PDF file properties such as authors, year, tags etc., in order to make it possible to sort (or group) a huge collection of PDF files based on these properties.

This can be done via a pdf property handler under the File Explorer Add-ons of PowerToys.
You already have preview and thumbnails handlers.

#25650 

### Scenario when this would be used?

For example college students or PhD or companies will be more productive with this feature.

For example a Business can sort files based on clients or other companies. Thank you, I believe that my suggestion will be considered and planned to be executed.

### Supporting information

_No response_
Repository: next.js
Issue: Warning: Prop `className` did not match. Server:
Labels: bug

### Verify canary release

- [X] I verified that the issue exists in the latest Next.js canary release

### Provide environment information

```bash
Operating System:
      Platform: win32
      Arch: x64
      Version: Windows 10 Home Single Language
    Binaries:
      Node: 18.16.0
      npm: N/A
      Yarn: N/A
      pnpm: 8.4.0
    Relevant packages:
      next: 13.4.6-canary.7
      eslint-config-next: 13.4.5
      react: 18.2.0
      react-dom: 18.2.0
      typescript: 5.1.3
```


### Which area(s) of Next.js are affected? (leave empty if unsure)

_No response_

### Link to the code that reproduces this issue or a replay of the bug

https://github.com/4nur4g/nextapppnpm

### To Reproduce

Just run the development server by, I'm running it using: `pnpm run dev`, the issue remains even if I use the `--turbo` flag. 

![image](https://github.com/vercel/next.js/assets/33851499/e79febc5-765a-4452-8b38-e9655741d769)

For reproducing from scratch: 

- pnpm create next-app

√ What is your project named? ... nextapppnpm
√ Would you like to use TypeScript with this project? ... No / Yes
√ Would you like to use ESLint with this project? ... No / Yes
√ Would you like to use Tailwind CSS with this project? ... No / Yes
√ Would you like to use `src/` directory with this project? ... No / Yes
√ Use App Router (recommended)? ... No / Yes
√ Would you like to customize the default import alias? ... No / Yes

Keep on entering for every prompt, i.e. default values. 

- pnpm install
- pnpm run dev

Check the log window. You'll find the warning. 

### Describe the Bug

When I start the development server, it logs the warning, even when I don't change the app created by create next app. 

![image](https://github.com/vercel/next.js/assets/33851499/c78022bb-bd51-4b9b-a795-9dc2919250ea)


### Expected Behavior

There should be no such warning, especially when the app wasn't touched after it was created using create next app. 

### Which browser are you using? (if relevant)

Microsoft Edge 

### How are you deploying your application? (if relevant)

_No response_
Repository: opencv
Issue: Unable to decode QR Codes with particular payloads
Labels: bug,category: objdetect,confirmed

### System Information

OpenCV python version: 4.7.0.72
Operating System: Ubuntu 22.04
Python version: 3.10.6

.venv dump (`pip freeze`) - also contains other QR detection libraries which were tested:
```
joblib==1.2.0
numpy==1.24.3
opencv-python==4.7.0.72
py4j==0.10.9.7
PyBoof==0.41
pyzbar==0.1.9
pyzxing==1.0.2
segno==1.5.2
six==1.16.0
transforms3d==0.4.1
```

### Detailed description

I was testing the performance of various QR code detection libraries using "perfect" version 1-H QR codes generated by `segno`, and I noticed that some of these codes could not be detected. In particular, the following values failed to be detected when testing exhaustively in the range $[0, 10^6]$:
```
9133, 24901, 27002, 28211, 30270, 33094, 36997, 37379, 51851, 62533, 63856, 70558, 71054, 74397, 76142, 79599, 91536
```
This issue was not exclusive to OpenCV, and other tested libraries such as `BoofCV` also had issues with some payloads (although for different values), which are confirmed as a bug due to false positives in the finder pattern (which may or may not be the same issue in this case) - see lessthanoptimal/PyBoof#23. Please note that there is also an identified bug in `segno` in which the padding is malformed for 4-digit payloads - see heuer/segno#123, although this does not seem to affect OpenCV as I'm assuming you ignore the padding bits (which seems to be a strategy employed by many detector libraries).

### Steps to reproduce

```python
import cv2
import segno

TEST_FILE = 'qr_test.png'

qr_opencv = cv2.QRCodeDetector()
bad_detections = []

for i in range(0, 100000):
    qrcode = segno.make(i, version=1)
    assert qrcode.error == 'H'
    qrcode.save(TEST_FILE, scale = 5)

    img = cv2.imread(TEST_FILE)
    img = cv2.cvtColor(img, cv2.COLOR_BGR2GRAY)
    img = cv2.resize(img, (88, 88))

    detected, points, straight_qrcode = qr_opencv.detectAndDecode(img)
    if not detected:
        print(f"Unable to detect {i}")
        bad_detections.append(i)
    else:
        assert str(detected) == str(i), f"Mismatch between detected value {detected} and input {x}"

print(f"Unable to detect the following data payloads:\n{bad_detections}")
```

### Issue submission checklist

- [X] I report the issue, it's not a question
- [X] I checked the problem with documentation, FAQ, open issues, forum.opencv.org, Stack Overflow, etc and have not found any solution
- [X] I updated to the latest OpenCV version and the issue is still there
- [X] There is reproducer code and related data files (videos, images, onnx, etc)
Repository: next.js
Issue: Link component "href" and "as" not respected when loading server component
Labels: bug

### Verify canary release

- [X] I verified that the issue exists in the latest Next.js canary release

### Provide environment information


I am (rightly or wrongly) passing some user-selected IDs from local state as a query parameter, so that the server component that renders the page containing favourited items can do so on the server side (instead of filtering them out on the client side).

My approach is this:
```bash
        <Link
          href={`/favourites?ids=${favorites.join(",")}`}
          as={`/favourites`}
          className="text-yellow font-normal text-md ml-4"
        >
          Favourites
        </Link>
```

What I'd like to happen is for the user to see `/favourites/` in the address bar, while the query param of favourited item IDs is passed, and the server component can grab them as search params.

However, in practice, the URL changes to `/favourites/` but the IDs aren't passed.

It only works if I remove the "as", which then reveals the ids query parameter, which is undesirable.

Is this simply not possible when using server components?
```


### Which area(s) of Next.js are affected? (leave empty if unsure)

_No response_

### Link to the code that reproduces this issue or a replay of the bug

https://gist.github.com/joneslloyd/76add7a12b28fe023f552e051ebae1e0

### To Reproduce

Make a Link component with href and as attributes.

If query params are passed in the href, they won't be received on the server (presumably they're never sent, as the "as" attribute is taking precedence?)

### Describe the Bug

The query params aren't passed to the server component as `searchParams`

### Expected Behavior

The server component receives the params as `searchParams`, while the address bar does not reveal them

### Which browser are you using? (if relevant)

_No response_

### How are you deploying your application? (if relevant)

_No response_
Repository: vscode
Issue: Resize window area is 1 pixel in top right of screen
Labels: bug,upstream,windows,electron,titlebar,upstream-issue-linked

I've noticed I naturally resize windows in the top right almost always, it's very difficult in VS Code as the hit area is a single pixel on Windows with the custom title bar.

![image](https://github.com/microsoft/vscode/assets/2193314/0e2b3511-db0f-40dc-9d58-f77ce3e72d05)
Repository: PowerToys
Issue: Remember location to include command line arguments for specific executables
Labels: Needs-Triage

### Description of the new feature / enhancement

For some applications the actual program or a specific variation of the program being run depends on the command line arguments. Would it be possible to list the programs and the salient arguments for which the zone is remembered separately from the base executable?

### Scenario when this would be used?

1. Java where the -jar {jarname} pair or the first argument determines the program. 
2. Firefox profiles
3. TextPad workspaces which I'd like in specific zones.

### Supporting information

I imagine it is possible for a list similar to the exclusion list to give an executable and subset of either specific arguments (with glob matching), or simply required strings (which would only need to be present somewhere in the command line) which would have a separate slot. For example, I could list:

With Globs:

````
javaw.exe -jar **/TimeKeeper.jar
javaw.exe -jar **/TestBench.jar
TextPad.exe "$devdoc.tws"
firefox.exe -P Developer
firefox.exe -P Private
````

Simple Strings:

````
javaw.exe TimeKeeper
javaw.exe TestBench
TextPad.exe $devdoc.tws
firefox.exe Developer
firefox.exe Private
````
Repository: pytorch
Issue: Exported model with dropout incorrectly applies dropout during eval
Labels: module: nn,triaged,module: correctness (silent),oncall: pt2,oncall: export

Normally calling `model.eval()` on a model with dropout changes the behavior of the dropout ops into a noop. However, this is not true if model is already exported, since the aten dropout pattern remains in the graph. Using the resulting model for inference led to incorrect computation, since we continue to apply dropout to the inputs during eval, which is unexpected. This affects QAT for the PT2 flow, where all transformations are done on an exported model.

Simple repro:
```
import torch
import torch._dynamo

class MyModel(torch.nn.Module):
    def __init__(self):
        super().__init__()
        self.dropout = torch.nn.Dropout(p=0.5)
    def forward(self, x):
        return self.dropout(x)

model = MyModel()
model, _ = torch._dynamo.export(model, torch.randn(1), aten_graph=True)
model.eval()
print(model)
```

This prints:
```
def forward(self, x):
    arg0, = fx_pytree.tree_flatten_spec(([x], {}), self._in_spec)
    empty_like_default = torch.ops.aten.empty_like.default(arg0, memory_format = torch.contiguous_format)
    bernoulli__float = torch.ops.aten.bernoulli_.float(empty_like_default);  empty_like_default = None
    div__scalar = torch.ops.aten.div_.Scalar(bernoulli__float, 0.5);  bernoulli__float = None
    mul_tensor = torch.ops.aten.mul.Tensor(arg0, div__scalar);  arg0 = div__scalar = None
    return pytree.tree_unflatten([mul_tensor], self._out_spec)
```

But it's supposed to print:
```
def forward(self, x):
    arg0, = fx_pytree.tree_flatten_spec(([x], {}), self._in_spec)
    return pytree.tree_unflatten([arg0], self._out_spec)
```

cc @albanD @mruberry @jbschlosser @walterddr @mikaylagawarecki @ezyang @msaroufim @wconstab @bdhirsh @anijain2305 @zou3519