sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: godot
Issue: Root path is counterintuitive; path debugging tools are lacking
Labels: topic:core,documentation

### Godot version

v3.5.2.stable.mono.official [170ba337a]

### System information

Windows 10

### Issue description

The path of the root node is counterintuitive; when a wrong path is used, the tools available are of little help in identifying and resoving the mistake.

In a normal path logic, a "/" would have been the path for a root. It makes no sense to specify "/root" unless there can also exist a "/nonroot" or other sibling nodes (but in that case a "/root" would not have been a _root_).

When a wrong absolute path with no "root" in it is used by mistake, no nodes can be accessed. Neither reading into the error message, nor visually inspecting the path, nor inspecting the remote scene in the editor reveals what the mistake is.

Ideally, if there is a root node it should be addresable by "/". Alternatively, if multiple nodes should coexist at "/", calling any one of them a "root" better be avoided as misleading. Copying path in the remote scene inspector should yield a complete absolute path, for any node there, including root. Checks in path handling code should identify when an impossible path such as "/" is used and report it as such, rather than simply reporting there is no node.

The issue is likely not version or OS specific.

### Steps to reproduce

Pass the root path "/" into GetNode()
`GetNode("/")`
observe no node returned, and no part of the error message explaining why there is no node at / .

Run any scene, switch to Remote tab in the editor and Copy Node Path on any non-root node. Observe the initial "/root" part missing in the copied path.
Copy Node Path on a root node. Observe it having a useless value of "." .

A valid complete absolute path can only be seen from `Node.GetPath()` .

### Minimal reproduction project

-
Repository: flutter
Issue: Add a parameter to OverflowBar to expand its children width if overflow
Labels: c: new feature,framework,c: proposal,P3,team-framework,triaged-framework

### Is there an existing issue for this?

- [X] I have searched the [existing issues](https://github.com/flutter/flutter/issues)
- [X] I have read the [guide to filing a bug](https://flutter.dev/docs/resources/bug-reports)

### Use case

I'm using `OverflowBar` to align two buttons horizontally if possible, one on the left, one on the right, with min width and space between.
When it overflows and the buttons goes vertical, I would like them to take full width instead of minimal.

### Proposal

Add a parameter to the widget `OverflowBar` to constraint its children to take max width if it overflows.

I achieved it by modifying the overflow vertical layout code (starting line 484) though I'm not sure its an efficient solution :
```
  if (actualWidth > constraints.maxWidth) {
      // Overflow vertical layout
      child =
          overflowDirection == VerticalDirection.down ? firstChild : lastChild;
      RenderBox? nextChild() => overflowDirection == VerticalDirection.down
          ? childAfter(child!)
          : childBefore(child!);
      double y = 0;
      while (child != null) {

---------------- ADDED THIS ---------------------------------------------------------------------------------------------
        child.layout(
          constraints.tighten(width: constraints.maxWidth),
          parentUsesSize: true,
        );
---------------- ADDED THIS ---------------------------------------------------------------------------------------------

        final _OverflowBarParentData childParentData =
            child.parentData! as _OverflowBarParentData;
        double x = 0;
        switch (overflowAlignment) {
          case OverflowBarAlignment.start:
            x = rtl ? constraints.maxWidth - child.size.width : 0;
          case OverflowBarAlignment.center:
            x = (constraints.maxWidth - child.size.width) / 2;
          case OverflowBarAlignment.end:
            x = rtl ? 0 : constraints.maxWidth - child.size.width;
        }
        childParentData.offset = Offset(x, y);
        y += child.size.height + overflowSpacing;
        child = nextChild();
      }
      size = constraints
          .constrain(Size(constraints.maxWidth, y - overflowSpacing));
    }
```
The `OverflowBar` if the content fits horizontally : 
![horizontal min width](https://github.com/flutter/flutter/assets/23617972/dfe79e5d-eb4c-4a4e-9a60-baac7c54c18b)

The `OverflowBar` if the content does not fit horizontally, make the children takes full width : 
![max width if vertical](https://github.com/flutter/flutter/assets/23617972/46c9d73d-2a13-40c0-86c3-3059d820fbc6)
Repository: vscode
Issue: Explore using multi diff editor for previewing workspace edit
Labels: feature-request,workspace-edit

We now have a diff editor that can show multiple (textual) diff in a single editor. We currently use this in source control contexts but it would be worth exploring to use this with previewing workspace edits, they are also across different files and from a single logical unit. This would also help with todays two step UI which shows a view which reveal individual diff editors
Repository: rust
Issue: std::mem::size_of::<Result<char,u16>>() says 8 when 4 should be feasible
Labels: C-enhancement,T-compiler,A-layout

I tried this code:

```rust
fn main() {
    println!("{}", std::mem::size_of::<Result<char,u16>>());
}
```

I expected to see this happen:

Expected it to print 4. Expected the `Ok` case to have the same representation as `char`. Expected the `Err` case to have a non-zero discriminant value in the upper 11 bits (upper 11 bits for `char` are always zero) and expected the `u16` to reside in the two least significant bytes. 

Instead, this happened:
Got 8, i.e. the compiler didn't make good use the top 11 bits of 32 bits that `u16` doesn't occupy at all and that are zero for `char`, since `char` only uses the lower 21 bits.

There should be enough available discriminant bits that `Option<Result<char,u16>>` could have size 4, too.

Use case: Surfacing the unpaired surrogate as `Err` when decoding UTF-16.

### Meta

Tried the current stable and nightly on https://play.rust-lang.org/?version=nightly&mode=release&edition=2021

It said:
Stable channel

Build using the Stable version: 1.74.0
Beta channel

Build using the Beta version: 1.75.0-beta.3

(2023-11-20 b66b7951b9b4258fc433)
Nightly channel

Build using the Nightly version: 1.76.0-nightly

(2023-11-26 6cf088810f66fff15d05)
Repository: godot
Issue: Godot 4 WebGL 100% CPU usage on Web export
Labels: bug,platform:web,topic:rendering,needs testing,performance

### Godot version

4.1.3

### System information

Godot v4.1.3.stable - macOS 12.5.1 - Vulkan (Compatibility) - Apple M1 Pro - Apple M1 Pro (10 Threads)

### Issue description

I have a pipeline where I export my Godot project into web environment using WebGL. At some point I noticed the built project started failing due to CPU going to 100% for no apparent reason. 

At some point Godot threw `Parent node is busy setting up children, add_node() failed. Consider using call_deferred("add_child", child) instead.` error but removing those calls and all the caches the problem still persisted. I eventually managed to reduce the problem into this https://github.com/TeemuKoivisto/godot-webgl-100-cpu-usage

There is now only one `ColorRect` left and if I remove it it starts to work but I have no idea what I'm doing wrong.

```
[gd_scene format=3 uid="uid://dmg02e8on1wyc"]

[node name="Main" type="Node"]

[node name="ColorRect" type="ColorRect" parent="."]
anchors_preset = 15
anchor_right = 1.0
anchor_bottom = 1.0
grow_horizontal = 2
grow_vertical = 2
color = Color(0.286275, 0.286275, 0.286275, 1)
```

### Steps to reproduce

https://github.com/TeemuKoivisto/godot-webgl-100-cpu-usage

### Minimal reproduction project

https://github.com/TeemuKoivisto/godot-webgl-100-cpu-usage
Repository: deno
Issue: Server side WebSocket should allow `1xxx` class close codes
Labels: suggestion,ext/websocket

Right now only 1000, and 3000-4999 are supported. This makes sense for clients, but for server side websockets (created from `Deno.upgradeWebsocket`) we should allow the full range of 1000-4999.
Repository: kubernetes
Issue: client-go: Add metrics into Informer
Labels: sig/scalability,sig/api-machinery,kind/feature,triage/accepted

### What would you like to be added?

The informer should expose metrics about queue/reflector/eventHandler.


### Why is this needed?

1. The informer lacks of metric,  it is hard to known how many item in its queue/store. Add metrics for queue/store, it will help developers to find the number of pending deltas.

2. The informer create a RingGrowing pendingNotifications for every eventHandler. This RingGrowing will grow, but never shrink.  An informer has some eventHandlers, it is hard to distinguish which pendingNotifications cost a lot memory. Add metrics for eventHandler pendingNotification, it will help developers to distinguish which eventHandler is slow.


related issue:
https://github.com/kubernetes/kubernetes/issues/121662
https://github.com/kubernetes/kubernetes/issues/117123
Repository: PowerToys
Issue: FancyZones experimental 'move newly created windows to current active monitor' shifts window with Greenshot
Labels: Issue-Bug,Product-FancyZones,Needs-Triage

### Microsoft PowerToys version

0.75.1

### Installation method

GitHub

### Running as admin

No

### Area(s) with issue?

FancyZones

### Steps to reproduce

Create zones on extended monitor (mine: 1 row x 3 columns)
Enable option 'move newly created windows to current active monitor (experimental).
Open excel in full screen on extended monitor
Use greenshot to capture part of excel spreadsheet


### ✔️ Expected Behavior

Capture the selected portion of the screen

### ❌ Actual Behavior

When activating greenshot, the excel spreadsheet on the extended monitor shifted down approximately one title bar.
The captured image did not include the correct area but was missing about one title bar worth of rows in the spreadsheet.
(I could not compensate for the shift.  Meaning, if I started my selection one row higher, I did not get expected screen capture, either)

### Other Software

Windows 10 Enterprise 10.0.19044 Build 19044
Microsoft® Excel® for Microsoft 365 MSO (Version 2309 Build 16.0.16827.20166) 64-bit 
Greenshot 1.2.10 Build 6 (64 bit)
Repository: rust
Issue: Cannot use unstable flags in RUSTFLAGS_BOOTSTRAP
Labels: T-bootstrap,C-discussion

I want to compile rustc with some unstable flags.

`RUSTFLAGS_BOOTSTRAP="-Zmeta-stats" x build compiler` errors with

```
error: failed to run `rustc` to learn about target-specific information

Caused by:
  process didn't exit successfully: `/home/gh-Nilstrieb/rust/build/aarch64-unknown-linux-gnu/stage0/bin/rustc - --crate-name ___ --print=file-names -Wrust_2018_idioms -Wunused_lifetimes -Zmeta-stats --crate-type bin --crate-type rlib --crate-type dylib --crate-type cdylib --crate-type staticlib --crate-type proc-macro --print=sysroot --print=split-debuginfo --print=crate-name --print=cfg` (exit status: 1)
  --- stderr
  error: the option `Z` is only accepted on the nightly compiler

  help: consider switching to a nightly toolchain: `rustup default nightly`

  note: selecting a toolchain with `+toolchain` arguments require a rustup proxy; see <https://rust-lang.github.io/rustup/concepts/index.html>

  note: for more information about Rust's stability policy, see <https://doc.rust-lang.org/book/appendix-07-nightly-rust.html#unstable-features>

  error: 1 nightly option were parsed
```

explicitly setting `RUSTC_BOOTSTRAP=1` doesn't work either. using `RUSTFLAGS=` directly also doesn't work
Repository: kubernetes
Issue: Exponential Backoff Retry for etcd3.GuaranteedUpdate
Labels: kind/feature,needs-triage,sig/etcd

### What would you like to be added?

etcd3.GuaranteedUpdate now supports exponential backoff retry for update failures.

### Why is this needed?

In scenarios where resource update frequencies are high, GuaranteedUpdate can result in numerous retries, unnecessarily burdening etcd. The introduction of an exponential backoff retry strategy in GuaranteedUpdate has reduced the number of conflict occurrences during update operations, thereby also decreasing the update load from apiserver on etcd.
Repository: rust
Issue: Tracking issue for `--env-set` flag
Labels: C-tracking-issue

Adds a new `--env-set` command line option on rustc. It allows to set environment variables value at compile-time to be used by `env!` and `option_env!` macros.

When retrieving and environment variable value, the one specified by `--env-set` will take precedence. For example if you want have `PATH=a` in your environment and pass `--env-set PATH=env`, then you will have:

```rust
assert_eq!(env!("PATH"), "env");
```

### Steps

- [x] Implement the [MCP](https://github.com/rust-lang/compiler-team/issues/653) -> Done in https://github.com/rust-lang/rust/pull/118368, in https://github.com/rust-lang/rust/pull/118830 and in https://github.com/rust-lang/rust/pull/119884
- [ ] Stabilization PR - https://github.com/rust-lang/rust/pull/119926

### Unresolved questions

- [ ] What is the desired behavior if the same `--env` variable is passed twice?
Repository: PowerToys
Issue: [Run: Unit converter] Add more types of unit convertion
Labels: Idea-Enhancement,Product-PowerToys Run,Needs-Triage,Run-Plugin

### Description of the new feature / enhancement

[UnitsNet](https://github.com/angularsen/UnitsNet) has many more unit conversations available to use. It would be great to have more possibilities with PowerToys Run. The one I'm missing at the moment is [VolumeFlow](https://github.com/angularsen/UnitsNet/blob/master/UnitsNet/GeneratedCode/Units/VolumeFlowUnit.g.cs)

### Scenario when this would be used?

Units sush as [VolumeFlow](https://github.com/angularsen/UnitsNet/blob/master/UnitsNet/GeneratedCode/Units/VolumeFlowUnit.g.cs) are generally used in conjunction with pressure units which are already part of Run.

### Supporting information

_No response_
Repository: transformers
Issue: Add flag for easily finetuning heads / linear probing to AutoModelforSequenceClassification
Labels: Feature request

### Feature request

Previous work has shown that last layer linear probing is cheaper and often generalizes better than normal finetuning (see [1](https://arxiv.org/pdf/2202.10054.pdf)). I imagine this could be implemented as a flag to AutoModelforSequenceClassification so that only the last layer classification head is trained. I believe this can be done manually by setting all the parameters except the last one to not track gradients, but a flag may be easier and encourage adoption. 

It may also be nice to have linear probing available at earlier layers (eg halfway through the model). This could be done through using the output_hidden_states flag during a forward pass. Mid layer linear probing can occasionally be more effective and is a widely used technique in the interpretability literature (see [2](https://direct.mit.edu/coli/article/48/1/207/107571/Probing-Classifiers-Promises-Shortcomings-and), [3](https://proceedings.neurips.cc/paper_files/paper/2019/file/159c1ffe5b61b41b3c4d8f4c2150f6c4-Paper.pdf), [4](https://arxiv.org/abs/2311.03658#:~:text=Informally%2C%20the%20%27linear%20representation%20hypothesis,directions%20in%20some%20representation%20space.), [5](https://arxiv.org/abs/2310.01405), and many others). 

Alternatively, if this were implemented generally for AutoModel (or maybe AutoModelforCausalLM?), it could use a wider variety of models. This feature could also be paired with an update that automatically allows models to be used for sequence classification by appending a final linear layer. 

### Motivation

Finetuning a head is extremely memory efficient and extremely fast (order of 1k parameters for most models, linear probes generally train in seconds, the main bottleneck will just be the forward pass) and oftentimes performs close to finetuning for classification tasks. It has also been shown to perform better OOD. 

### Your contribution

I can provide feedback and testing, have not looked deep enough to know how to fully implement this
Repository: next.js
Issue: Exporting a font gives a type error in Build mode 
Labels: bug,TypeScript

### Link to the code that reproduces this issue

https://github.com/Abdellah-Idrissi/font-error-repo

### To Reproduce

Just ``` npm run build ``` and you will face the error while building

![Screenshot 2023-11-27 160919](https://github.com/vercel/next.js/assets/106756822/f2c8793a-9b52-441f-b992-8b4f10c53e36)


### Current vs. Expected behavior

### Current behavior
I usually have 2 fonts in my apps and I always add an export to the second one so I can import it wherever I need it but since I updated to next14 and now next14.0.3 whenever I add an export to a font e.g
 ```export const inter = Inter({ subsets: ['latin'] }) ``` 
and I just run build without adding anything after it I get the type error in the pic above

### Expected behavior
adding an export the any font without having this type error in build mode


### 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 Pro
Binaries:
  Node: 20.9.0
  npm: N/A
  Yarn: N/A
  pnpm: N/A
Relevant Packages:
  next: 14.0.3
  eslint-config-next: 14.0.3
  react: 18.2.0
  react-dom: 18.2.0
  typescript: 5.3.2
Next.js Config:
  output: N/A
```


### Which area(s) are affected? (Select all that apply)

TypeScript (plugin, built-in types)

### Additional context

_No response_
Repository: ant-design
Issue: Unnecessary tiny horizontal scroll in virtualized tables when using browser scaling/zoom
Labels: Inactive,improvement

### Reproduction link

[![Edit on CodeSandbox](https://codesandbox.io/static/img/play-codesandbox.svg)](https://codesandbox.io/p/sandbox/basic-usage-antd-5-11-5-forked-h6sjfq?file=%2Fdemo.tsx%3A80%2C21)

### Steps to reproduce

- Open sandbox
- Adjust browser window size so no horizontal scroll is necessary to see table content
- See no horizontal scroll bar
  
![image](https://github.com/ant-design/ant-design/assets/16518144/e270b9bf-bd5a-4de6-a754-50d8f415be18)

- Adjust browser scaling/zoom (CTRL   Mouse Wheel Up/Down), for me it works if scaling is <=80%
- See horizontal scroll bar, even though table content should still fit horizontally it doesn't and can be scrolled a little bit

![image](https://github.com/ant-design/ant-design/assets/16518144/f65e8646-d21d-47f3-bad4-97352cd14deb)

### What is expected?

Not seeing horizontal scroll bar when table content fits into table width.

### What is actually happening?

Horizontal scroll bar appears, table can be scrolled a little bit horizontally.

| Environment | Info |
| --- | --- |
| antd | 5.11.5 |
| React | ^18.0.0 |
| System | Windows 11 |
| Browser | Microsoft Edge, Google Chrome |

<!-- generated by ant-design-issue-helper. DO NOT REMOVE -->
Repository: go
Issue: x/crypto/ssh: TestSSHCLIAuth fails on Windows
Labels: OS-Windows,NeedsFix

Not sure how I didn't notice this until today. See #60981 previously.

Example: http://ci.chromium.org/b/8763261739598890561:

`sign_and_send_pubkey: signing using rsa-sha2-512 SHA256:Anr3LjZK8YVpjrxu79myrW9Hrb/wpcMNpVvTq/RcBm8\r\nBad permissions. Try removing permissions for user: UNKNOWN\\\\UNKNOWN (S-1-15-2-2) on file C:/b/s/w/ir/x/t/TestSSHCLIAuth1586735692/001/rsa.`

Seems like a bug in the test so for now I'm going to skip.

cc @drakkan @golang/security @FiloSottile @rolandshoemaker
Repository: next.js
Issue: Sporadic timeouts in API method that seems to finish
Labels: bug,Runtime

### Link to the code that reproduces this issue

https://github.com/replayio/delta

### To Reproduce

The linked repository contains an app with an API for up-/downloading and comparing screenshots in test runs that is called in a GitHub CI workflow. Running that workflow will call the API dozens of times and some of those calls time out (but usually succeed when retried).
Furthermore there's a web app that uses the API to download the screenshots and that also times out sporadically.
Note that the app is deployed on Vercel.

### Current vs. Expected behavior

Current: sporadic API timeouts
Expected: no API timeouts

### Verify canary release

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

### Provide environment information

```bash
Operating System:
  Platform: linux
  Arch: x64
  Version: #99~20.04.1-Ubuntu SMP Thu Nov 2 15:16:47 UTC 2023
Binaries:
  Node: 18.18.2
  npm: 9.8.1
  Yarn: 1.22.18
  pnpm: 8.6.7
Relevant Packages:
  next: 14.0.3
  eslint-config-next: 13.0.7
  react: 0.0.0-experimental-ee8509801-20230117
  react-dom: 0.0.0-experimental-ee8509801-20230117
  typescript: 4.9.4
Next.js Config:
  output: N/A
```


### Which area(s) are affected? (Select all that apply)

Middleware / Edge (API routes, runtime)

### Additional context

Here's the log from one [`uploadSnapshotVariants.ts`](https://github.com/replayio/delta/blob/main/pages/api/uploadSnapshotVariants.ts) API request that timed out:
```
START RequestId: 3225c3dc-fe9a-4375-889a-a8d0e21eb78a Version: $LATEST
2023-11-25T19:44:36.592Z	3225c3dc-fe9a-4375-889a-a8d0e21eb78a	INFO	Finding most recent snapshots for branch \"main\" and run 2068320
2023-11-25T19:44:36.687Z	3225c3dc-fe9a-4375-889a-a8d0e21eb78a	INFO	Inserting Snapshot for Project replay
2023-11-25T19:44:36.742Z	3225c3dc-fe9a-4375-889a-a8d0e21eb78a	INFO	Inserting SnapshotVariant for Snapshot 463692
2023-11-25T19:44:36.783Z	3225c3dc-fe9a-4375-889a-a8d0e21eb78a	INFO	Inserting SnapshotVariant for Snapshot 463692
2023-11-25T19:44:36.835Z	3225c3dc-fe9a-4375-889a-a8d0e21eb78a	INFO	Done!
2023-11-25T19:45:36.363Z 3225c3dc-fe9a-4375-889a-a8d0e21eb78a Task timed out after 60.06 seconds

END RequestId: 3225c3dc-fe9a-4375-889a-a8d0e21eb78a
REPORT RequestId: 3225c3dc-fe9a-4375-889a-a8d0e21eb78a	Duration: 60061.61 ms	Billed Duration: 60000 ms	Memory Size: 1024 MB	Max Memory Used: 83 MB
```
Note that [this line](https://github.com/replayio/delta/blob/26c7121212e0c252c706ac1be2ece681e51d6cb9/pages/api/uploadSnapshotVariants.ts#L187) was apparently executed and the handler tries to send a response immediately after that but timed out instead.
Repository: vscode
Issue: Merge Editor: Current is not always HEAD for every merge conflict
Labels: bug,important,merge-conflict

https://github.com/microsoft/vscode/blob/e6b2fd82c18b264236d706abdacc842ab6d2babb/extensions/git/src/commands.ts#L527-L546

The content should be compared against the :1:/:2:/:3:

I think this happens when rebasing multiple (conflicting) commits.