sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: vscode
Issue: [html] Formatting Breaks with Space Between Commented Lines in Script Tag
Labels: bug,help wanted,html

<!-- ⚠️⚠️ Do Not Delete This! bug_report_template ⚠️⚠️ -->
<!-- Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct/ -->
<!-- 🕮 Read our guide about submitting issues: https://github.com/microsoft/vscode/wiki/Submitting-Bugs-and-Suggestions -->
<!-- 🔎 Search existing issues to avoid creating duplicates. -->
<!-- 🧪 Test using the latest Insiders build to see if your issue has already been fixed: https://code.visualstudio.com/insiders/ -->
<!-- 💡 Instead of creating your report here, use 'Report Issue' from the 'Help' menu in VS Code to pre-fill useful information. -->
<!-- 🔧 Launch with `code --disable-extensions` to check. -->
Does this issue occur when all extensions are disabled?: Yes

<!-- 🪓 If you answered No above, use 'Help: Start Extension Bisect' from Command Palette to try to identify the cause. -->
<!-- 📣 Issues caused by an extension need to be reported directly to the extension publisher. The 'Help > Report Issue' dialog can assist with this. -->
- VS Code Version: Code 1.89.0 (https://github.com/microsoft/vscode/commit/b58957e67ee1e712cebf466b995adf4c5307b2bd, 2024-05-01T02:08:25.066Z)
- OS Version: Linux x64 6.5.0-18-generic

When commenting out sections in a script tag, 
the HTML formatting breaks 
if there is a line with at least 1 space in between 2 commented out lines.
( Highlighting the following code block may help clarify. )

```html
<script>
    //
 
    //
             </script>
```

Steps to Reproduce:

1. Copy and paste the code block shown above into a blank HTML file in vscode.
2. Format the HTML using the **Format Document** command ( keyboard shortcut is usually Ctrl + Shift + I ) .
3. Document will not format
4. Remove the single space ( not the line ) between the 2 commented out lines.
5. Format HTML using same method as Step 2.
6. Document should be formatted.
Repository: neovim
Issue: Escaped commas are not handled in options
Labels: bug,lua,options

### Problem

The function `convert_value_to_lua` in `runtime/lua/vim/_options.lua` does not consider escaped commas for arrays.

This makes it impossible to use e.g. `vim.opt.errorformat:append` with its default value together with `:make` as the value after appending is invalid. 

The same goes for maps and sets although my understanding is that they are not currently affected by this as they can't contain commas as values.

This issue has previously been mentioned [here](https://github.com/neovim/neovim/discussions/28151).

### Steps to reproduce

1. `:set errorformat=a\\,,ab\\,,a\\,b` (the values `a,`, `ab,` and `a,b`)
2. `:set errorformat`, inspect the result
3. `:lua =vim.opt.errorformat:get()`, notice how even escaped commas split array entries
4. `:lua vim.opt.errorformat:append("c")`
5. `:set errorformat`, notice how (in addition to the appended value) one comma have disappeared since 2., breaking the value

### Expected behavior

Escaped commas should not split the array.

### Neovim version (nvim -v)

0.10.0

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

-

### Operating system/version

Fedora 40

### Terminal name/version

-

### $TERM environment variable

-

### Installation

appimage
Repository: kubernetes
Issue: Parallel image pull should pull unique images only once
Labels: kind/bug,sig/node,triage/accepted

### What would you like to be added?

Creating large amount of pods that all use the same image causes the kubelet to start pull per each pod, when parallel pull is enabled. This causes the reigstry QPS to be used, slowing down other pods with other images and also wastes the bandwidth for downloading the same images multiple times.

If same image is being already downloaded in parallel, other pods should just wait for it, not starting new download and not using up registry QPS. 

### Why is this needed?

We have use case where we start large number (500+) pods at the same time where many pods gets scheduled on single node (100+), and most pods have same init container with the same image (given the image on the container is provided with full SHA digest).

This causes lot of bandwidth on the node to be used during the startup of those pods, and also some pods are delayed due to `ImagePullBackoff` caused by `pull QPS exceeded`.
Repository: PowerToys
Issue: Feature request: Remind Me Tool
Labels: Idea-New PowerToy,Needs-Triage

### Description of the new feature / enhancement

A new feature that allows us to set a reminder for a task/event/activity... When the timer is done, it shows a notification to the user.

Option to configure how the notification should appear:
- windows notification style (for normal reminders)
- popup message (for very important reminders) (optional)

Could be integrated into PowerToys Run where you can use a symbol like '!' to create the reminder; e.g. '!remindme 30m <description>' which will set a reminder in 30 minutes.

Or a dedicated UI where you can create a reminder with description, and the select duration or datetime of trigger.


### Scenario when this would be used?

when needed to be reminded of something in the near future.

### Supporting information

_No response_
Repository: angular
Issue: Directly Importing Template Dependencies in HTML Templates
Labels: area: core,core: directive matching

### Which @angular/* package(s) are relevant/related to the feature request?

compiler

### Description

Currently, template dependencies for standalone components are imported and configured within TypeScript files, even though their main application is within HTML templates. This establishes a gap between where the dependencies are defined and where they are implemented.

### Proposed solution

I propose a syntax that allows dependencies to be directly imported into the HTML template files. Taking inspiration from the proposed "let syntax" #15280 , the proposed syntax could be:

```html
@import {NgClass , NgStyle} from "@angular/common"
<div [NgClass]="something" ></div>
<!--  or -->
<div [ngStyle]="something" ></div>

@defer(..){
  @import {SomeComponet} from "something";
  <SomeComponet />
  <!--  or -->
  <some-componet />
}
```
Moreover, Imports could also be scoped to deferred blocks for lazy loading.

### Alternatives considered

None
Repository: transformers
Issue: Log multiple losses used along with the combined losses when a model returns a dictionary of losses.
Labels: trainer,Feature request

### Feature request

Able to log individual losses returned as dict.

### Motivation

I have multiple losses that are being added to form a combined loss. I want to log all these individual losses to observe the trend of the individual losses. SemanticSegmenterOutput accepts single loss at the moment and logs the loss in the SemanticSegmenterOutput.

### Your contribution

I have modified the Trainer class and SemanticSegmenter output as below but it is not working as expected. I have added a few print statements to check if the on_log part is being accessed or not but that code is not even being accessed.

    
class CustomTrainer(Trainer):
    def __init__(self, *args, **kwargs):
        super().__init__(*args, **kwargs)
        self.additional_losses = {}

    def training_step(self, model, inputs):
        outputs = model(**inputs)
        
        # Extract the additional losses from the model output
        self.additional_losses = outputs.additional_losses
        
        # Continue with usual training step process
        return super().training_step(model, inputs)

    def on_log(self, global_step, state, control, logs, **kwargs):
        loss_dict = logs.get("loss_dict", {})
        
        if 'wandb' in self.args.report_to:
            wandb.log({k: v for k, v in loss_dict.items()})
        
        if 'tensorboard' in self.args.report_to:
            tensorboard_writer = self.state.log_history.get('tensorboard')
            if tensorboard_writer is not None:
                tensorboard_writer.add_scalars('additional_losses', loss_dict, global_step)
        
        # Call the parent's on_log method
        super().on_log(global_step, state, control, logs, **kwargs)

    def on_train_end(self):
        tensorboard_writer = self.state.log_history.get('tensorboard')
        if tensorboard_writer is not None:
            tensorboard_writer.close()


@dataclass
class CustomSemanticSegmenterOutput(SemanticSegmenterOutput):
    additional_losses: Optional[Dict[str, torch.FloatTensor]] = None
Repository: ant-design
Issue: Head table title columns blink when use sticky and sorter with useEffect react 18 to set columns
Labels: Inactive,unconfirmed

### Reproduction link

[![Edit on StackBlitz](https://developer.stackblitz.com/img/open_in_stackblitz.svg)](https://stackblitz.com/edit/react-vyd7ax?file=demo.js)

### Steps to reproduce

1. Create antd sticky table with 0 columns
2. Click Button "Load Columns" to state load
3. useEffect trigger and set 40 columns with sorter=true

### What is expected?

Title in table head not blink, the columns's width be load true in the first view

### What is actually happening?

See title in table head "blink" like columns's width not set

| Environment | Info |
| --- | --- |
| antd | 5.17.4 |
| React | 18.3.1 |
| System | macOS 14.1.1 |
| Browser | Chorme |

---
You can see it more clear by Performance Record with Screenshot in Chrome
Image about title blink: 

<img width="1307" alt="Screenshot 2024-05-28 at 17 26 43" src="https://github.com/ant-design/ant-design/assets/16161343/38981e1b-8eb2-43dd-8c34-720565da8622">

<img width="512" alt="Screenshot 2024-05-28 at 17 19 20" src="https://github.com/ant-design/ant-design/assets/16161343/0a969f34-37d7-4fd5-a881-7350add08b33">


<!-- generated by ant-design-issue-helper. DO NOT REMOVE -->
Repository: pytorch
Issue: Fused AdamW maybe should accept lr_dict directly?
Labels: module: optimizer,triaged,needs research

### 🚀 The feature, motivation and pitch

Fused AdamW can accept tensor LR, and convert it to lr_dict internally, but sometimes not all lr lies in the same device, why not accept `dict[device, Tensor]` besides `float` and  `Tensor`?

### Alternatives

_No response_

### Additional context

_No response_

cc @vincentqb @jbschlosser @albanD @janeyx99 @crcrpar
Repository: TypeScript
Issue: Type a standard import with jsdoc only
Labels: Suggestion,Awaiting More Feedback,Domain: JSDoc,Domain: JavaScript

### 🔍 Search Terms

ecmascript module import jsdoc

### ✅ Viability Checklist

- [X] This wouldn't be a breaking change in existing TypeScript/JavaScript code
- [X] This wouldn't change the runtime behavior of existing JavaScript code
- [X] This could be implemented without emitting different JS based on the types of the expressions
- [X] This isn't a runtime feature (e.g. library functionality, non-ECMAScript syntax with JavaScript output, new syntax sugar for JS, etc.)
- [X] This isn't a request to add a new utility type: https://github.com/microsoft/TypeScript/wiki/No-New-Utility-Types
- [X] This feature would agree with the rest of our Design Goals: https://github.com/Microsoft/TypeScript/wiki/TypeScript-Design-Goals

### ⭐ Suggestion

Right now I can type an `require` call with 
```ts
/**
 * @type {typeof import("./accounts").userAccount}
 */
var x = require("./accounts").userAccount;
```
from https://www.typescriptlang.org/docs/handbook/jsdoc-supported-types.html#import-types

But when I my code uses an import statement
`import { shortNameFromImport } from "textmodule";`
I did not found a way to type that when the module resolution does found a hit.

`@type` in the line before can not be used because there is no `let` or `const` in this line.
```ts
/** @type {import('textmodule')} */
import { shortNameFromImport } from "textmodule";
```

And the [normal way](@type) to type a variable with parenthesis can not be used because it is not valid JS code:
```ts
import { /** @type {import('textmodule')} */ (shortNameFromImport ) } from "textmodule";
```

Both variants without parenthesis do not work...
```ts
import { /** @type {import('textmodule')} */ shortNameFromImport } from "textmodule";
```
```ts
import { shortNameFromImport /** @type {import('textmodule')} */ } from "textmodule";
```


### 📃 Motivating Example

With typescript code I can do some magic with `path` in the `tsconfig.json`
```json
"paths": {
  "textmodule": [ "./textmodule.ts" ]
},
```
But I wanted to have a JSDoc only solution.

### 💻 Use Cases

See above
Repository: rust
Issue: pre-push hook should only look at staged changes
Labels: T-bootstrap,C-bug,A-contributor-roadblock

if i have modified the repository locally, but not committed those changes, and i run `git push`, i expect it to lint on the changes that are *actually going to be pushed*, not the ones i have laying around.

instead, `src/etc/pre-push` lints on the working directory.

it would be nice if it checked if there are unstaged changes, and if so copied the working directory to a tempdir to run the checks there. here is a [hacky bash script](https://github.com/jyn514/dotfiles/commit/629a6617a10b4000ff75fb42654f8fe0e3ba6735) that does it for checks that work file-at-a-time; i don't *think* tidy ever looks at multiple files at once or tries to build code, but if it does it would need more work.

### Meta

HEAD is f989d2f62500df1696eb797d8800a705293b43b9

@rustbot label +A-bootstrap +A-contributor-roadblock
Repository: vscode
Issue: Notebook does not clear dirty indicator after saving
Labels: bug,notebook-format

Testing #213608

I followed to use the extension sample that is provided in the description and created a Jupyter notebook with this content:

```
{
 "cells": [
  {
   "cell_type": "code",
   "execution_count": null,
   "metadata": {},
   "outputs": [],
   "source": [
    "import \"xx\"\n"
   ]
  },
  {
   "cell_type": "code",
   "execution_count": null,
   "metadata": {},
   "outputs": [],
   "source": [
    "\n",
    "// Hello World\n"
   ]
  }
 ],
 "metadata": {
  "language_info": {
   "name": "python"
  }
 },
 "nbformat": 4,
 "nbformat_minor": 2
}

```

When I save, the dirty indicator never goes away.
Repository: vscode
Issue: Not being asked which save action to use
Labels: under-discussion,notebook-format,notebook-code-actions

Testing #213608

I have 2 but I am not being asked on save:

<img width="434" alt="image" src="https://github.com/microsoft/vscode/assets/900690/501dafb4-9ede-4499-85d7-9ed08c25f83e">
Repository: ui
Issue: [bug]: When FormDescription is not used there is an id still applied to aria-describedby causing Arc Toolkit errrors
Labels: bug

### Describe the bug

The FormDescription component can be used to add hint text to a form control. It will automatically have a generated id applied and is associated with the form control via aria-describedby. However, in cases where no hint text is required, and therefore the FormDescription component is not composed in, the id is still generated and added to the controls aria-describedby. I'm not sure how much of an issue this is (or if an easy solution exists), but it throws accessibility errors with Arc Toolkit (although Axe doesn't seem to error). Does having aria-describedby referencing non-existent elements really matter?

### Affected component/components

Form

### How to reproduce

1. Create a form example with no Form.Description component
2. Run Arc Toolkit to see the accessibility error relating to aria-describedby referencing non-existent id.

### Codesandbox/StackBlitz link

_No response_

### Logs

_No response_

### System Info

```bash
N/A
```


### Before submitting

- [X] I've made research efforts and searched the documentation
- [X] I've searched for existing issues
Repository: rust
Issue: rustdoc ignores `cargo` and linker settings when compiling doc tests
Labels: T-rustdoc,C-bug

When building doc tests, cargo settings like `--jobs` are ignored, and the `linker` specified in `config.toml` is also ignored.

However, by setting `rustdocflags` we can control the linker correctly. This should use the project-level linker by default.

### Context

These limitations make problems with slow / high memory doc test construction (see #75341) much harder to mitigate.
The approach proposed in #51228 may offer a way to fix this.

Initially discovered by Giooschi [on the Bevy Discord](https://discord.com/channels/691052431525675048/692572690833473578/1244966846218764319) while investigating https://github.com/bevyengine/bevy/issues/12207.

### Meta

```
rustc 1.78.0 (9b00956e5 2024-04-29)
binary: rustc
commit-hash: 9b00956e56009bab2aa15d7bff10916599e3d6d6
commit-date: 2024-04-29
host: x86_64-pc-windows-msvc
release: 1.78.0
LLVM version: 18.1.2
```
Repository: next.js
Issue: [Breaking] NextJs [14.2.3] Draft mode with static pages
Labels: bug,Navigation,Runtime

### Link to the code that reproduces this issue

https://github.com/AnasArafeh/NextJs-14.2.3-Draft-Mode-with-Static-pages

### To Reproduce

1. install packages ( you will probably need -legacy-peer-deps ) probably related to the react version
2. you can use both next start or next dev
3.  add  either this URL api/draft-mode?slug=home or api/draft-mode?slug=home/isr to the example : http://localhost:3000/api/draft-mode?slug=home
4. check the console.log in the terminal you will see that no search parameters being sent.

### Current vs. Expected behavior

Previously, when we are using draft mode all static games switch to be dynamic pages so we are able to use search parameters.

However, once we have upgraded to 14.2.3 and later, and issue happened where search parameters are being empty all the time using the exact same code.

** **Important** update also if you remove the **request.nextUrl.search** from the redirect then the **Static** pages will **not** switch into **Dynamic** 

**Static** page transform into **Dynamic** without having **access** to **search parameters** during **Draft mode**
![image](https://github.com/vercel/next.js/assets/68680829/ad529086-f779-4dc2-8876-67184eb61b95)

**Static** page stays as **Static** during **Draft mode**. (Also isEnabled from Draft Mode will also show as false even though it was enabled) 
![image](https://github.com/vercel/next.js/assets/68680829/53999d9d-f541-41f4-94bd-2c2bd4a2cd2a)





One more thing which is unrelated to this bug, I found another bug related to draft mode most likely, where you access draft mode given the steps above, when you try to kill the terminal using Ctrl + C, you get the following memory leak error: 

![image](https://github.com/vercel/next.js/assets/68680829/92b1db6f-3fe7-4e4f-85ff-35e8f746e6c6)


* you can enter the http://localhost:3000/api/draft-mode?slug=home in the browser or multiple times then try to close the terminal and you will have the error above. This is also starting happening after upgrading to the latest version


### Provide environment information

```bash
Operating System:
  Platform: win32
  Arch: x64
  Version: Windows 10 Enterprise
  Available memory (MB): 20323
  Available CPU cores: 8
Binaries:
  Node: 18.18.0
  npm: N/A
  Yarn: N/A
  pnpm: N/A
Relevant Packages:
  next: 14.3.0-canary.85 // Latest available version is detected (14.3.0-canary.85).
  eslint-config-next: N/A
  react: 19.0.0-rc-f994737d14-20240522
  react-dom: 19.0.0-rc-f994737d14-20240522
  typescript: 5.1.3
Next.js Config:
  output: N/A
```


### Which area(s) are affected? (Select all that apply)

Not sure, Navigation, Runtime

### Which stage(s) are affected? (Select all that apply)

next dev (local), next start (local)

### Additional context

I had NextJs version 14.0.3 while everything was working fine but when I upgraded to 14.2.3 I noticed this issue.
Repository: vscode
Issue: Markdown preview stops updating with unsaved content if dragged out to a new Window (and back again)
Labels: bug,markdown,webview

Not sure if this is expected or not - dragging a markdown preview out to its own window causes unsaved content to vanish and it stops updating.

Even dragging it back into the main window does not restore that behaviour. You need to close and re-open it.

https://github.com/microsoft/vscode/assets/1078012/1032d510-4989-4156-8a5f-f99f28b770a4

If this is expected, are the restrictions on multi-window interaction like this documented anywhere?
Repository: pytorch
Issue: I don’t know if it’s a problem with cuda or pytorch
Labels: needs reproduction,module: cudnn,module: cuda,triaged

### 🐛 Describe the bug

Dataset CS - Label range: [0, 1073741828]
Traceback (most recent call last):
File "segment.py", line 661, in <module>
main(args, get_dataset)
File "segment.py", line 608, in main
model = train(args, get_dataset, model, False) #Train
File "segment.py", line 279, in train
scaler.scale(loss_s).backward()
File "/home/amax/anaconda3/envs/yyl/lib/python3.8/site-packages/torch/_tensor.py", line 396, in backward
torch.autograd.backward(self, gradient, retain_graph, create_graph, inputs=inputs)
File "/home/amax/anaconda3/envs/yyl/lib/python3.8/site-packages/torch/autograd/__init__.py", line 173, in backward
Variable._execution_engine.run_backward( # Calls into the C++ engine to run the backward pass
RuntimeError: cuDNN error: CUDNN_STATUS_EXECUTION_FAILED
You can try to repro this exception using the following code snippet. If that doesn't trigger the error, please include your original repro script when reporting this issue.
```
import torch
torch.backends.cuda.matmul.allow_tf32 = False
torch.backends.cudnn.benchmark = False
torch.backends.cudnn.deterministic = False
torch.backends.cudnn.allow_tf32 = True
data = torch.randn([6, 128, 64, 128], dtype=torch.half, device='cuda', requires_grad=True)
net = torch.nn.Conv2d(128, 128, kernel_size=[1, 3], padding=[0, 2], stride=[1, 1], dilation=[1, 2], groups=1)
net = net.cuda().half()
out = net(data)
out.backward(torch.randn_like(out))
torch.cuda.synchronize()
```
ConvolutionParams
memory_format = Contiguous
data_type = CUDNN_DATA_HALF
padding = [0, 2, 0]
stride = [1, 1, 0]
dilation = [1, 2, 0]
groups = 1
deterministic = false
allow_tf32 = true
input: TensorDescriptor 0x7f9fed396640
type = CUDNN_DATA_HALF
nbDims = 4
dimA = 6, 128, 64, 128,
strideA = 1048576, 8192, 128, 1,
output: TensorDescriptor 0x7f9fed264960
type = CUDNN_DATA_HALF
nbDims = 4
dimA = 6, 128, 64, 128,
strideA = 1048576, 8192, 128, 1,
weight: FilterDescriptor 0x7f9fed35e5c0
type = CUDNN_DATA_HALF
tensor_format = CUDNN_TENSOR_NCHW
nbDims = 4
dimA = 128, 128, 1, 3,
Pointer addresses:
input: 0x7fa03c800000
output: 0x7fa0a5800000
weight: 0x7f9fe59e2800
Additional pointer addresses:
grad_output: 0x7fa0a5800000
grad_weight: 0x7f9fe59e2800
Backward filter algorithm: 1
../aten/src/ATen/native/cuda/NLLLoss2d.cu:103: nll_loss2d_forward_kernel: block: [17,0,0], thread: [962,0,0] Assertion t >= 0 && t < n_classes failed.

### Versions

PyTorch Version: 1.12.1+cu113
CUDA Version: 11.3
cuDNN Version: 8900

cc @csarofeen @ptrblck @xwang233 @msaroufim