sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: pytorch
Issue: Mis-annotated return for `F._no_grad_embedding_renorm_` (also JIT related)
Labels: oncall: jit,module: typing,triaged,bug

### 🐛 Describe the bug

This is a sub-task of #103761.

This is not user-facing as this only involves functions prefixed with an underscore.

`_no_grad_embedding_renorm_` has mismatched typing on the return value:
https://github.com/pytorch/pytorch/blob/918fe519a0f9bdb2dd213eaaffbcd420b7391280/torch/nn/functional.py#L2123-L2124

1. This function does not return anything at all. This is modified on purpose in #18684. (PS: the return value is not used anyway.)
2. The return now got annotated as a `Tuple[Tensor, Tensor]` in #78560. I don't know why it's changed in this way. @samdow Can you please explain this as the author of that PR?

---

Other occurrences of `_no_grad_embedding_renorm_`: (I'm not quite sure whether these are related, nor which is correct)
https://github.com/pytorch/pytorch/blob/918fe519a0f9bdb2dd213eaaffbcd420b7391280/torch/csrc/api/include/torch/nn/functional/embedding.h#L15-L22
https://github.com/pytorch/pytorch/blob/918fe519a0f9bdb2dd213eaaffbcd420b7391280/torch/csrc/jit/runtime/register_special_ops.cpp#L330
https://github.com/pytorch/pytorch/blob/918fe519a0f9bdb2dd213eaaffbcd420b7391280/torch/csrc/jit/runtime/serialized_shape_function_registry.cpp#L3196
https://github.com/pytorch/pytorch/blob/918fe519a0f9bdb2dd213eaaffbcd420b7391280/torch/jit/_shape_functions.py#L1109

### Versions

master

cc @EikanWang @jgong5 @wenzhe-nrv @sanchitintel @ezyang @malfet @rgommers @xuzhao9 @gramster
Repository: electron
Issue: [Feature Request]: support import in asar archives
Labels: enhancement :sparkles:,component/asar

### Preflight Checklist

- [X] I have read the [Contributing Guidelines](https://github.com/electron/electron/blob/main/CONTRIBUTING.md) for this project.
- [X] I agree to follow the [Code of Conduct](https://github.com/electron/electron/blob/main/CODE_OF_CONDUCT.md) that this project adheres to.
- [X] I have searched the [issue tracker](https://www.github.com/electron/electron/issues) for a feature request that matches the one I want to file, without success.

### Problem Description

I'm trying to pack an next.js app into electron. When loading runtime config there's an error 

![image](https://github.com/electron/electron/assets/4531670/05d697d7-9b27-4df6-815a-d025b76e5d97)

I build the electron binary with asar packing. "Require" works fine with absolute paths but "import" does not.

The complaining code https://github.com/vercel/next.js/blob/67853aa8cdc649cc35a6b7efc033bf262ff4ba87/packages/next/src/server/config.ts#L734

### Proposed Solution

Support "import" in electron node bindings.

### Alternatives Considered

- monkey patch next.js load config --> Too many "import" calls in the code base. You can never finish
- transform using webpack & babel --> Theoretically doable. But too much modules to transform in node_modules and requires a lot of path hacking
- unpack asar --> need to manually change all require paths from 'app.asar' to 'app.asar.unpacked', and it's needlessly trivial I never made it work.

### Additional Information

_No response_
Repository: next.js
Issue: White screen during build when using useSearchParams
Labels: Navigation

### 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: arm64
  Version: Darwin Kernel Version 22.5.0: Mon Apr 24 20:52:24 PDT 2023; root:xnu-8796.121.2~5/RELEASE_ARM64_T6000
Binaries:
  Node: 18.16.0
  npm: 9.5.1
  Yarn: 1.22.19
  pnpm: N/A
Relevant packages:
  next: 13.4.7-canary.1
  eslint-config-next: 13.4.6
  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)

App directory (appDir: true), Routing (next/router, next/navigation, next/link)

### Link to the code that reproduces this issue or a replay of the bug

https://github.com/Ne-Ne/test-project

### To Reproduce

- Create a NextJs project using the app directory structure.
- Create a test component using:

```typescript
"use client";

import { usePathname, useRouter, useSearchParams } from "next/navigation";

export async function Test() {
  const router = useRouter();
  const pathname = usePathname();
  const searchParams = useSearchParams();

  return <h1>Test.tsx</h1>;
}
```

- Import into the root `page.tsx`

### Describe the Bug

When trying to use the new API `useSearchParams` during dev it works fine.

During build all I get is the white screen, nothing is returned on the `<Body>` node.

I have tried this on both the latest stable and canary, both yield the same white screen.

Canary:

https://test-project-ne-ne.vercel.app (latest build)
https://test-project-lxf4s0qq1-ne-ne.vercel.app (unique deployment)

Stable:

https://test-project-q6sxazc8s-ne-ne.vercel.app (unique deployment)



### Expected Behavior

The page renders both nodes and matches the development behaviour.

### Which browser are you using? (if relevant)

Brave, Chrome, Safari, Firefox

### How are you deploying your application? (if relevant)

Vercel
Repository: godot
Issue: PhysicalBone2D does not work
Labels: topic:physics,needs testing

### Godot version

v4.0.3.stable.official [5222a99f5]

### System information

Windows 10 - Godot v4.0.3.stable - Vulkan (Forward+) - NVIDIA GeForce GTX 1650 (nvidia, 31.0.15.1694) - Intel Core i5-10400F CPU @ 2.90GHz

### Issue description

i make a 2D ragdoll with 4.0.3, PhysicalBone2D doesn't work, but RigidBody2D work, how to use PhysicalBone2D?

in this video, left is PhysicalBone2D, right is RigidBody2D:

https://github.com/godotengine/godot/assets/67264008/1fd8fe7b-95f7-4300-854b-4c07bf2045ea


### Steps to reproduce

1. download the project.
2. run main_scene.
3. left is PhysicalBone2D, right is RigidBody2D.

### Minimal reproduction project

[PhysicalBone2D in 4.0.3.zip](https://github.com/godotengine/godot/files/11779281/PhysicalBone2D.in.4.0.3.zip)
Repository: d3
Issue: Add d3-hexbin to the default bundle?
Labels: enhancement

Seems like a reasonable add.
Repository: d3
Issue: Add d3-tile to the default bundle?
Labels: enhancement

Seems like a reasonable add.
Repository: pytorch
Issue: Support `Sequence` type in JIT
Labels: oncall: jit,module: typing,triaged,enhancement

### 🚀 The feature, motivation and pitch

This is a sub-task of #103798.

We have functions annotated to accept `List[int]` but actually get `torch.Size` passed in, e.g.:
https://github.com/pytorch/pytorch/blob/918fe519a0f9bdb2dd213eaaffbcd420b7391280/torch/nn/functional.py#L2435
https://github.com/pytorch/pytorch/blob/918fe519a0f9bdb2dd213eaaffbcd420b7391280/torch/nn/functional.py#L2481

However, static type checkers like mypy does not like it because `torch.Size`, which is a subtype of `Tuple[int, ...]`, is not compatible with `List[int]`. This conflicts with JIT, which does not understand `Tuple[int, ...]` without a fixed length.

There are many cases of this problem, all involving functions taking in tensor sizes or similar (e.g., `Tensor.size` or `kernel_size`), where we only use them as `collections.abc.Sequence`.

Therefore, I propose to annotate them as `Sequence[int]` and ask JIT to interpret `Sequence[int]` as `List[int]`.

By annotating the sizes as `Sequence[int]`, we can i) make mypy happy because `List[int]` and `Tuple[int, ...]` (including `torch.Size`) are both `Sequence[int]`, and ii) borrow the JIT infrastructure for `ListType` which does not care about the length, bypassing the tuple length problem.

### Alternatives

1. _Adjust the implementation of `TupleType` in JIT to support `Tuple[int, ...]`_. This can be the best solution, but I don't know how to do it. Also, I assume it's not easy because otherwise it would already be implemented.
2. _Implement `SequenceType`, which differs from `ListType`_. This is also a good option, maybe second to the above, because it's safer than reusing the `ListType` -- some operations supported by `List` are not by `Sequence`, although JIT seems not to be using them.
3. _Add dozens of `type:ignore`s to suppress all related mypy warnings_. The good point is: the affected functions are for internal use (with `_` prefix) and will not affect public APIs (i.e., the users will not see any problem). However, this can leave the code error-prone.

### Additional context

Since we mark them as `List[int]` for now, even with `Tuple`s/`Size`s, directly reusing `List` for `Sequence` will not cause any problems. We actually keep how everything works under the hood, just switching to a different name that can satisfy mypy.

cc @EikanWang @jgong5 @wenzhe-nrv @sanchitintel @ezyang @malfet @rgommers @xuzhao9 @gramster
Repository: stable-diffusion-webui
Issue: [Bug]: Can no longer generate same exact images upgrading from 1.3.0 to 1.3.1-1.3.2
Labels: bug-report

### Is there an existing issue for this?

- [X] I have searched the existing issues and checked the recent builds/commits

### What happened?

I have upgraded from AUTOMATIC1111 from 1.3.0 to 1.3.2. when I try to now generate the same images from 1.3.0 with the exact same seed, prompt and everything the images no longer generate the same.
I have tried to generate with various images from version 1.3.0 but the result is the same, I can't generate the same image.

### Steps to reproduce the problem

1. insert 1.3.0 image in png info on automatic1111 v1.3.2
2. send to txt2img
3. apply same checkpoint/model
4. generate image


### What should have happened?

Exact same image is generated in the inserted image in png info on Stable diffusion v1.3.2

### Commit where the problem happens

https://github.com/AUTOMATIC1111/stable-diffusion-webui/commit/baf6946e06249c5af9851c60171692c44ef633e0

### What Python version are you running on ?

Python 3.10.x

### What platforms do you use to access the UI ?

Windows

### What device are you running WebUI on?

Nvidia GPUs (RTX 20 above)

### What browsers do you use to access the UI ?

Mozilla Firefox

### Command Line Arguments

```Shell
No
```


### List of extensions

Stable-Diffusion-Webui-Civitai-Helper
sd-dynamic-prompts

### Console logs

```Shell
Normal
```


### Additional information

_No response_
Repository: terminal
Issue: CONTROL+C on close instead of CONTROL_CLOSE_EVENT
Labels: Product-Conhost,Area-Server,Issue-Task,Priority-3

<!-- 
🚨🚨🚨🚨🚨🚨🚨🚨🚨🚨

I ACKNOWLEDGE THE FOLLOWING BEFORE PROCEEDING:
1. If I delete this entire template and go my own path, the core team may close my issue without further explanation or engagement.
2. If I list multiple bugs/concerns in this one issue, the core team may close my issue without further explanation or engagement.
3. If I write an issue that has many duplicates, the core team may close my issue without further explanation or engagement (and without necessarily spending time to find the exact duplicate ID number).
4. If I leave the title incomplete when filing the issue, the core team may close my issue without further explanation or engagement.
5. If I file something completely blank in the body, the core team may close my issue without further explanation or engagement.

All good? Then proceed!
-->

# Description of the new feature/enhancement

<!-- 
on closing the x button send a control+c signal to all processes by default instead of terminating them.
Definition: the x button is described as both the x button for windows terminal itself and every tab
-->
on closing the x button send a control+c signal to all processes by default instead of terminating them.
Definition: the x button is described as both the x button for windows terminal itself and every tab

# Proposed technical implementation details (optional)

<!-- 
there are three ways to implement this maybe make them all options
- Send Control+C signal and do nothing if processes don't respond. If nothing is done then the user clicks the X button a second time send a SIGTERM(CONTROL_CLOSE_EVENT) that gives the application 5s to respond before closing
- Send Control+C signal and do nothing for 5-8s, After that prompt the user if they are sure they want to kill all processes inside before closing like macOs but not immediate as we want CLI programs to try and save on regular shutdown
- Add an option to always use Control_CLOSE_EVENT (SIGTERM) on X button for those users that don't want the prompt and don't care about data loss / file corruption
-->
there are three ways to implement this maybe make them all options
- Send Control+C signal and do nothing if processes don't respond. If nothing is done then the user clicks the X button a second time send a SIGTERM(CONTROL_CLOSE_EVENT) that gives the application 5s to respond before closing
- Send Control+C signal and do nothing for 5-8s, After that prompt the user if they are sure they want to kill all processes inside before closing like macOs but not immediate as we want CLI programs to try and save on regular shutdown
- Add an option to always use Control_CLOSE_EVENT (SIGTERM) on X button for those users that don't want the prompt and don't care about data loss / file corruption
Repository: angular
Issue: Incorrect type inference for FormRecord
Labels: area: forms

### Which @angular/* package(s) are the source of the bug?

forms

### Is this a regression?

No

### Description

`FormRecord` is sometimes incorrectly inferred to be a `FormGroup`.

This is apparently caused by:
- `FormRecord` class actually extends `FormGroup`
-  The way [`ɵElement`](https://github.com/angular/angular/blob/9588b11e5b556fdc64e075e97d8f707f29cddcdf/packages/forms/src/form_builder.ts#L64) is implemented (first checks for instance of `FormGroup`, before `FormRecord`)

### Please provide a link to a minimal reproduction of the bug

This appeared as a TS error while working on #50750 (which adds a new method on `FormRecord`)
https://github.com/EmmanuelRoux/angular/tree/formrecord-clear

Revealed by some Angular's own tests, such as [this one](https://github.com/angular/angular/blob/9588b11e5b556fdc64e075e97d8f707f29cddcdf/packages/forms/test/typed_integration_spec.ts#L1401) (extract below):
```ts
        const c = fb.array([fb.record({bar: 'foo'})]);
        {
          type ControlsType = Array<FormRecord<FormControl<string|null>>>;
          let t: ControlsType = c.controls; // <-- incorrectly inferred as controls of `FormGroup` instead of `FormRecord`
          let t1 = c.controls;
          t1 = null as unknown as ControlsType;
        }
```

### Please provide the exception or error you saw

_No response_

### Please provide the environment you discovered this bug in (run `ng version`)

```true
Working on PR #50750 which adds a new method on `FormRecord` (but not on `FormGroup`)

Angular CLI: 16.0.0-rc.4
Node: 20.3.0 (Unsupported)
Package Manager: yarn 1.22.19
OS: darwin arm64

Angular: undefined
... 

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1600.0-rc.4
@angular-devkit/build-angular   16.0.0-rc.4
@angular-devkit/core            16.0.0-rc.4
@angular-devkit/schematics      16.0.0-rc.4
@angular/build-tooling          0.0.0-5f06c4774df908ed69e1441f4ec63b898acf0c68
@angular/cdk                    16.0.0-rc.2
@angular/cli                    16.0.0-rc.4
@angular/material               16.0.0-rc.2
@angular/ng-dev                 <error>
@bazel/bazelisk                 1.12.1
@bazel/buildifier               6.1.2
@bazel/concatjs                 5.8.1
@bazel/esbuild                  5.8.1
@bazel/ibazel                   v0.16.2
@bazel/jasmine                  5.8.1
@bazel/protractor               5.8.1
@bazel/rollup                   5.8.1
@bazel/runfiles                 5.8.1
@bazel/terser                   5.8.1
@bazel/worker                   5.8.1
@schematics/angular             16.0.0-rc.4
rxjs                            6.6.7
typescript                      5.1.3
```


### Anything else?

_No response_
Repository: PowerToys
Issue: [Feature Request] Implement Activity/Productivity profiles to allow open and closing of apps (foreground & background) when users switch between gaming and studying/productivity
Labels: Idea-New PowerToy

### Description of the new feature / enhancement

There would be a Gaming Profile and a Productivity Profile, and perhaps additional custom profiles that the user could make for other miscellaneous needs, so that the user can open the app collection they need when they're gaming, and exit those same programs (especially the background ones) them when studying. There would be toggles as well for specific apps to close (if they are open) if a specific profile is launched. Gaming laptops could gain significantly more battery life when not gaming, without having to consciously micromanage which apps are open and which ones need to close (especially background apps), since the PC will manage that for me.

My hope is that this feature becomes built into Windows.

### Scenario when this would be used?

I have apps such as Medal and Blitz installed, that automatically open when my laptop starts up, and consume a considerable amount of battery life. They are specifically for gaming, so I do not need them open when studying instead. With the profile switcher, I can have Medal and Blitz automatically launch if I launch the Gaming Profile, and add toggles to the Productivity Profile so that if I ever launch the Productivity Profile, it'll look specifically for both Medal and Blitz and end tasks for both programs.

### Supporting information

If background apps meant specifically for gaming stayed open all the time, they would consume an unnecessary amount of battery life. Micromanaging background apps (finding which ones to open and which ones to close when switching between gaming and studying) can be unnecessarily complicated and consume more time than needed. Implementing profiles significantly cuts down both the amount of time consumed micromanaging apps as well as the battery life that would otherwise be uselessly consumed for apps that aren't being used. This implementation would allow gaming laptops to provide more battery life for their users, when they've usually had a reputation for not having good battery life.
Repository: PowerToys
Issue: Remember previous entered remote computer names in Mouse Without Borders
Labels: Idea-Enhancement,Needs-Triage,Product-Mouse Without Borders

### Microsoft PowerToys version

0.70.1

### Installation method

PowerToys auto-update

### Running as admin

No

### Area(s) with issue?

Mouse Without Borders

### Steps to reproduce

Remember previous entered remote computer names in Mouse Without Borders. Why should I need to enter it again and again.

### ✔️ Expected Behavior

The previously entered computer names to connect MWB to are remembered.

### ❌ Actual Behavior

The previously entered computer names to connect MWB to are not remembered.

### Other Software

_No response_
Repository: PowerToys
Issue: Support seamless dragging of windows across machines in Mouse Without Borders
Labels: Idea-Enhancement,Needs-Triage,Product-Mouse Without Borders

### Description of the new feature / enhancement

Mouse Without Borders is already very useful.

Support seamless dragging of windows across machines in Mouse Without Borders and then it will be really great because sometime the monitor in front of your gaze is the best monitor and with this the best monitor will be usable with all computers. Behind the scenes VCN or RDP could be used.

And with that the utility name can be changed to Desktop Without Borders.

### Scenario when this would be used?

It will be really great because sometime the (main) monitor in front of your gaze is the best monitor and with this the best monitor will be usable with all computers. 

### Supporting information

_No response_
Repository: neovim
Issue: LSP config (settings) interface
Labels: enhancement,lsp,architecture

# Problem

Discussion in https://github.com/neovim/neovim/pull/23984#issuecomment-1586364409 and other LSP work indicates we need a full story for configuring LSP clients...

1. globally ("default")
    - per-filetype
3. dynamically (on-demand)
    - for all clients attached to current buffer
    - for specific client?
5. per-buffer (this is low-priority, avoid much complexity to support this)


https://github.com/neovim/neovim/pull/22598#issuecomment-1463367606



# Expected behavior

- `vim.lsp.config` : ~~#18506~~ #31031
    - potential blockers:
        - should we adopt `workspaceFolder` instead of "root markers"
        - "multi-server" conflicts with per-server configs.
            - how far can we get with the "obvious" approach of per-server config?

## Rejected ideas

- [Flags](https://github.com/neovim/neovim/pull/23984#issuecomment-1587818178) in `vim.lsp.start()`, e.g. `start({ ..., formatting = false, semantic_tokens = false, inlay_hints = true, ... })`
    - Why: LSP has [tons of configuration](https://github.com/neoclide/coc.nvim/blob/2bd87f66c7048e6c7c3a3d067be5c5e6e0036d33/data/schema.json), so `start()` flags are not sufficient.
- `vim.lsp.start({capabilities={...}})`
    - Why: un-declaring client capabilities is probably too hacky?
    - Related: https://github.com/neovim/neovim/discussions/24058
Repository: godot
Issue: Buggy behavior with native windows in GNU/Linux
Labels: bug,platform:linuxbsd,topic:porting,topic:gui

### Godot version

4.1.dev [a83eb16]

### System information

Kubuntu GNU/Linux 23.04

### Issue description

This has been tested on both Wayland and X11.

- `Window`s will always stay on top of parent windows, even if `exclusive` and `always_on_top` is disabled.
- `exclusive` will work even if `transient` is disabled.
- `transient` windows wont be hidden with their parents, and despite what the documentation says, they can be made fullscreen.

### Steps to reproduce

Download the project below, and play it. You can change some stuff easily in other to test the bugs above.

### Minimal reproduction project

[WindowBugs.zip](https://github.com/godotengine/godot/files/11780436/WindowBugs.zip)
Repository: stable-diffusion-webui
Issue: [Feature Request]: Settings | Images filename pattern for generation type
Labels: enhancement

### Is there an existing issue for this?

- [X] I have searched the existing issues and checked the recent builds/commits

### What would your feature do ?

Request to add an option to include the type of generation as filename pattern.  ie:  t2i and i2i for text to image and image to image, and maybe inp for inpaint.

### Proposed workflow

Go to Settings
Go to Saving images/grids
In images filename pattern
  option for generation type filename pattern

### Additional information

_No response_
Repository: PowerToys
Issue: Match Lock/Shut Down/Screen Saver/Log out
Labels: Idea-Enhancement,Needs-Triage,Product-Mouse Without Borders

### Description of the new feature / enhancement

When using Mouse Without Borders, there is already a feature that keeps the screensaver of connected computers from engaging while another machine is actively being used. Could also have 3 new features to allow the machines to all Lock at the same time when Win + L is pressed (or selected from any machines start menu), Screen Saver start and end at the same time across all machines when none of them are being used, and when shutting down or logging out of one machine, it shuts down or logs out of all machines.

### Scenario when this would be used?

This feature would come in to play many times:

When stepping away from a computer, the screen savers could engage all at the same time.
When locking a computer, a user wouldn't have to lock each and every machine, same with shut down and log out.

### Supporting information

_No response_