sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: godot
Issue: Replacing AnimationNodeStateMachine in AnimationNodeBlendTree during runtime breaks animation
Labels: documentation,needs testing,topic:animation

### Tested versions

Reproducible in Godot 4.3 dev 3, Godot v4.2.1.stable

### System information

Godot v4.2.1.stable - Windows 10.0.22621 - Vulkan (Forward+) - dedicated Radeon RX 590 Series (Advanced Micro Devices, Inc.; 31.0.21902.5) - 12th Gen Intel(R) Core(TM) i5-12400F (12 Threads)

### Issue description

I have such AnimationTree root_node:
![image](https://github.com/godotengine/godot/assets/32330527/9e7d6657-da4f-476d-8f47-1feb12f7a887)


I want to replace AnimationNodeStateMachine with another AnimationNodeStateMachine during runtime

I tried exporting my moveset
```GDScript
@onready var animator : AnimationTree = $AnimationTree
@export var moveset: AnimationNodeStateMachine
```
and then replace it and reconnect with my blend node but i get t-pose
```GDScript
var previous_position: Vector2 = animator.tree_root.get_node_position("moveset_1")
animator.tree_root.remove_node("moveset_1")
animator.tree_root.add_node("moveset_1", moveset, previous_position)
animator.tree_root.connect_node("Blend2", 1, "moveset_1")
```
also i tried to reconnect node, but it has the same result:
```GDScript
animator.tree_root.disconnect_node("Blend2", 1)
animator.tree_root.connect_node("Blend2", 1, "moveset_2")
```


### Steps to reproduce

attack with LMB
change moveset with "C"

If you connect moveset_2 with Blend2 before starting game, then this way animation will change without issues, but if you change moveset with "C", then animation will break

there are two ways in player's script, you can comment current and uncomment other one:
```GDScript
if Input.is_action_just_pressed("change_moveset"):
  #region type 1
  #var previous_position: Vector2 = animator.tree_root.get_node_position("moveset_1")
  #animator.tree_root.remove_node("moveset_1")
  #animator.tree_root.add_node("moveset_1", moveset, previous_position)
  #animator.tree_root.connect_node("Blend2", 1, "moveset_1")
  #endregion
  
  #region type 2
  animator.tree_root.disconnect_node("Blend2", 1)
  animator.tree_root.connect_node("Blend2", 1, "moveset_2")
  #endregion
```

### Minimal reproduction project (MRP)

[Animation Test.zip](https://github.com/godotengine/godot/files/14253880/Animation.Test.zip)
Repository: node
Issue: Expose `createErrDiff` for diffing objects
Labels: feature request

### What is the problem this feature will solve?

I like the general format of the `deepStrictEqual` diff comparisons, but it would be nice to get this functionality "natively" to print a diff instead of forcing it to error to see a diff.

### What is the feature you are proposing to solve the problem?

Expose `createErrDiff` from `assert` https://github.com/nodejs/node/blob/544cfc5ef151bca8d625fbccc581200a77b00bc0/lib/internal/assert/assertion_error.js#L79

### What alternatives have you considered?

Userland diff tool, but if the code is already there it would be nice to expose it
Repository: go
Issue: cmd/compile: avoid call to runtime.makeslice for zero-length types?
Labels: Performance,NeedsInvestigation,compiler/runtime

### Go version

Go 1.22

### What did you do?

`return make([]struct{}, 123)` compiles to a call to `runtime.makeslice`:

```
        LEAQ    type:struct {}(SB), AX
        MOVL    $123, BX
        MOVQ    BX, CX
        PCDATA  $1, $0
        NOP
        CALL    runtime.makeslice(SB)
        MOVL    $123, BX
        MOVQ    BX, CX
        ADDQ    $24, SP
        POPQ    BP
        RET
```

We see this show up in profiles (small, but non-zero) for calls to https://pkg.go.dev/tailscale.com/types/views#Slice.LenIter (which predates Go 1.22's range-over-int):

```go
// LenIter returns a slice the same length as the v.Len().
// The caller can then range over it to get the valid indexes.
// It does not allocate.
func (v Slice[T]) LenIter() []struct{} { return make([]struct{}, len(v.ж)) }
```

I realize that the compiler could omit those `makeslice` calls when the width of the type being made is zero (as there's no actual allocation).

The compiler already does that when the slice length is zero:

```
// Recognise make([]T, 0) and replace it with a pointer to the zerobase
(StaticLECall {callAux} _ (Const(64|32) [0]) (Const(64|32) [0]) mem)
        && isSameCall(callAux, "runtime.makeslice")
        => (MakeResult (Addr <v.Type.FieldType(0)> {ir.Syms.Zerobase} (SB)) mem)
```

/cc @golang/compiler @twitchyliquid64 @maisem
Repository: pytorch
Issue: Devcontainer for the nightly devenv
Labels: module: build,triaged

### 🚀 The feature, motivation and pitch

We need a light Devcontainer for the Nighlty workaround:
https://github.com/pytorch/pytorch/blob/main/CONTRIBUTING.md#nightly-checkout--pull

This is going to really lower the barrier for a occasional python only contribution.

See the CHAOSS (Linux Foundation) metric:
https://chaoss.community/kb/metric-newcomer-experience/


/cc @malfet @seemethere @shunting314 please mention a possible owner of this ticket

### Alternatives

_No response_

### Additional context

_No response_
Repository: rust
Issue: std::collections::BTreeMap docs could be improved by answering "When should I choose this over a HashMap"
Labels: A-collections,A-docs,T-libs

https://doc.rust-lang.org/std/collections/struct.BTreeMap.html
https://github.com/rust-lang/rust/blob/74c3f5a146860c94ff4d179fc3bfa34f879adf41/library/alloc/src/collections/btree/map.rs#L41-L169

The docs for `BTreeMap` discuss the performance of its implementation, but assume that you already know that a sorted map is the right fit for your use case. But there are also people visiting these docs who are not sure whether a sorted map is right for them.

I think one nice addition would be to try to answer the question "When should I choose a BTreeMap over a HashMap?"

Maybe something like this:

"You can use a BTreeMap for a map whose keys have an order. A BTreeMap has a few features based on this order that a HashMap does not: 1. You can iterate entries in key order. 2. You can use look up keys "between" entries in the map, for example you can use `map.range(10..).next()` or `map.range(..=10).next_back()` to get the closest entry after or before 10."

It would also be nice to say something about performance trade-offs between HashMaps and BTreeMaps.
Repository: godot
Issue: Releasing mouse button inputs like LMB seemingly halt InputEventMouseMotion events.
Labels: needs testing,topic:input

### Tested versions

- Reproducible in: 4.2.1.stable

### System information

Windows 10 - Godot v4.2.1.stable - Ryzen 5 3600x 6-Cores

### Issue description

Whenever you press and release something that would classify as `InputEventMouseButton` while gathering `InputEventMouseMotion` information, you can observe a little skip in the motion.

I wasn't able to confirm if this extends to _any_ button input, but for FPS/TPS games, having this issue can be a little irritating considering buttons like LMB are used for actions like shooting.

### Steps to reproduce

Move the camera with your mouse, and click the Left Mouse Button at different rates to observe different degrees of the dropped input.

It is subtle but it is there.

Tinker with the Max FPS or V-sync as you please.

### Minimal reproduction project (MRP)

[CameraBugMRP.zip](https://github.com/godotengine/godot/files/14255955/CameraBugMRP.zip)
Repository: next.js
Issue: The `with-opentelemetry` example doesn't send telemetry data
Labels: examples

### Verify canary release

- [X] I verified that the issue exists in the latest Next.js canary release

### Provide environment information

```bash
yarn run v1.22.19
$ /Users/yamada/work/samples/with-opentelemetry-app/node_modules/.bin/next info

Operating System:
  Platform: darwin
  Arch: arm64
  Version: Darwin Kernel Version 23.0.0: Fri Sep 15 14:42:57 PDT 2023; root:xnu-10002.1.13~1/RELEASE_ARM64_T8112
Binaries:
  Node: 18.17.0
  npm: 9.6.7
  Yarn: 1.22.19
  pnpm: N/A
Relevant Packages:
  next: 14.1.1-canary.51 // Latest available version is detected (14.1.1-canary.51).
  eslint-config-next: N/A
  react: 18.2.0
  react-dom: 18.2.0
  typescript: 4.7.4
Next.js Config:
  output: N/A


✨  Done in 1.72s.
```


### Which example does this report relate to?

with-opentelemetry

### What browser are you using? (if relevant)

Microsoft Edge for Business Version 121.0.2277.112 (Official build) (arm64)

### How are you deploying your application? (if relevant)

yarn dev

### Describe the Bug

The example doesn't send telemetry data.

This looks like a `@vercel/otel` bug.
I confirmed by adding `@vercel/otel` to my Next.js application without the example and it didn't send telemetry data as well.
https://nextjs.org/docs/app/building-your-application/optimizing/open-telemetry#using-vercelotel

I also confirmed that telemetry data is sent with manual configuration.
In this case default spans are sent.
https://nextjs.org/docs/app/building-your-application/optimizing/open-telemetry#manual-opentelemetry-configuration

### Expected Behavior

The example send telemetry data with `@vercel/otel`.

### To Reproduce

1. Launch an OTel collector using `opentelemetry-collector-dev-setup` as a preparation.
```
git clone https://github.com/vercel/opentelemetry-collector-dev-setup.git
cd opentelemetry-collector-dev-setup
docker-compose up -d
```

2. Create and launch an application using the example.
```
yarn create next-app --example with-opentelemetry with-opentelemetry-app
cd with-opentelemetry-app
yarn dev
```

3. Open localhost:3000 in a browser
4. Check Jaeger in the browser.
Repository: pytorch
Issue: Flight Recorder dump_entries() segfaults when used with coalesced operations
Labels: oncall: distributed

First observed when using batched_isend_irecv with send/recv P2pOps in PipelineParallel on `whc/pp` branch (torchtrain).  Isolated a minimal repro (https://github.com/pytorch/pytorch/pull/119757)

I suspect this issue applies to all collectives/point2point when the following conditions are true


1) We record the work into the flight recorder, which includes refs to cuda events
  (always true, more or less)

2) We don't call workEnqueue because coalescing is active or something
  (I don't understand this logic as well)
```
   if (!coalescing_state_ && capture_status == c10::cuda::CaptureStatus::None) {
    workEnqueue(work);
  } else {
    at::cuda::CUDAGraph::dec_pending_event_queries();
  }
```

My hypothesis is that we are losing the work object (it's getting destructed) in the path we don't put it in the workMetaList.  I would be surprised by this bc I assume something has to keep it alive for coalescing.

cc @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @osalpekar @jiayisuse @H-Huang @kwen2501 @awgu @penguinwu @fegin @XilunWu @wanchaol @fduwjj @wz337 @tianyu-l @yf225 @shuqiangzhang
Repository: PowerToys
Issue: PowerRename - Original Filename Length Limit
Labels: Issue-Bug,Product-PowerRename,Needs-Triage

### Microsoft PowerToys version

0.77.0

### Installation method

PowerToys auto-update

### Running as admin

No

### Area(s) with issue?

PowerRename

### Steps to reproduce

Attempt to rename file with long file name.
i.e.

- Hiring for Attitude꞉ A Revolutionary Approach to Recruiting and Selecting People with Both Tremendous Skills and Superb Attitude, 1e [B01NB12JWW] - 07 - Chapter 4꞉ Creating Brown Shorts Guidelines
- Hiring for Attitude꞉ A Revolutionary Approach to Recruiting and Selecting People with Both Tremendous Skills and Superb Attitude, 1e [B01NB12JWW] - 09 - Chapter 6꞉ Recruiting for Your Brown Shorts



### ✔️ Expected Behavior

Display original file name so it can be used in the "search for" function.

### ❌ Actual Behavior

They show in PowerRename as:
![image](https://github.com/microsoft/PowerToys/assets/105557996/36a00b78-401c-4993-98d5-60334ff9dbb1)

### Other Software

_No response_
Repository: rust
Issue: Check for "constants we match on must be PartialEq" misses lifetime-dependent impls
Labels: A-lifetimes,A-trait-system,T-compiler,C-bug,A-patterns,T-types

The following should trigger a warning that we can only match on PartialEq consts, but it doesn't: (example by @lcnr, then slightly tweaked)
```rust
use std::marker::PhantomData;

struct Inv<'a>(PhantomData<*mut &'a ()>);

// This type is only sometimes `PartialEq`.
impl PartialEq for Inv<'static> {
    fn eq(&self, _: &Inv<'static>) -> bool {
        true
    }
}

impl<'a> Inv<'a> {
    // The value `None` makes this have structural equality for any type `Self`.
    const NOT_STATIC: Option<Self> = None;
}

fn foo<'a>(x: Option<Inv<'a>>) {
    match x {
        Inv::<'a>::NOT_STATIC => (),
        Some(_) => panic!()
    }
    
    // Enabling the next line confirms that the type does
    // indeed not implement `PartialEq`.
    //x == Inv::<'a>::NOT_STATIC;
}

fn main() {
    foo(None)
}
```

Here is [another example](https://play.rust-lang.org/?version=nightly&mode=debug&edition=2021&gist=183a1d779e2b26eabec2dc43ee31a40f) of the same issue.

The problem is that we call `predicate_must_hold_modulo_regions` [here](https://github.com/rust-lang/rust/blob/9f58cf43c78e77b59479da2beaea0265f529280e/compiler/rustc_mir_build/src/thir/pattern/const_to_pat.rs#L257), but only borrowck knows the actual lifetimes and borrowck has no idea that this trait obligation even exists.

We should leave some sort of trace in MIR that there is a PartialEq obligations to ensure borrowck can check this with the right lifetimes.
Repository: stable-diffusion-webui
Issue: API is broken [Bug]: 
Labels: bug-report

### What happened?

I am using automatic web 1111 api,
for Text To Image and Image to Image 

I have GPU RTX A5000 24GB VRAM

When multiple request send to server
Sometime API is broken. It seems that the API requests are still being received and jobs are being run, however the responses back never happen and it times out.





### Steps to reproduce the problem

1. Send multiple request to server at time for long time 
2. Mean time server response getting timeout 
3. wait for minute and send again 
4. Now all request has same response timeout, while jobs continuously running on server   


### Console logs

```Shell
Job is continuous running on server  

2024-02-13T07:21:31.845235894Z 
Total progress:  55%|█████▌    | 11/20 [00:00<00:00, 11.58it/s]
2024-02-13T07:21:32.023977302Z 
Total progress:  65%|██████▌   | 13/20 [00:01<00:00, 11.44it/s]
2024-02-13T07:21:32.202504963Z 
Total progress:  75%|███████▌  | 15/20 [00:01<00:00, 11.36it/s]
2024-02-13T07:21:32.381845953Z 
Total progress:  85%|████████▌ | 17/20 [00:01<00:00, 11.31it/s]
2024-02-13T07:21:32.470545248Z 
  0%|          | 0/20 [00:00<?, ?it/s]
  5%|▌         | 1/20 [00:00<00:02,  8.01it/s]
 15%|█▌        | 3/20 [00:00<00:01,  9.89it/s]
 25%|██▌       | 5/20 [00:00<00:01, 10.49it/s]
 35%|███▌      | 7/20 [00:00<00:01, 10.70it/s]
 45%|████▌     | 9/20 [00:00<00:01, 10.86it/s]
 55%|█████▌    | 11/20 [00:01<00:00, 10.98it/s]
 65%|██████▌   | 13/20 [00:01<00:00, 11.03it/s]
 75%|███████▌  | 15/20 [00:01<00:00, 11.08it/s]
 85%|████████▌ | 17/20 [00:01<00:00, 11.12it/s]
 95%|█████████▌| 19/20 [00:01<00:00, 11.13it/s]
100%|██████████| 20/20 [00:01<00:00, 10.90it/s]
2024-02-13T07:21:33.004699776Z 
Total progress:  95%|█████████▌| 19/20 [00:01<00:00, 11.26it/s]
2024-02-13T07:21:33.004939829Z 
Total progress: 100%|██████████| 20/20 [00:02<00:00, 11.26it/s]
Total progress: 100%|██████████| 20/20 [00:02<00:00,  8.91it/s]
2024-02-13T07:21:33.149637668Z INFO:      - "POST /sdapi/v1/txt2img HTTP/1.1" 200 OK
2024-02-13T07:21:56.813383789Z 2024-02-13 07:21:56,812 - ControlNet - INFO - Loading model from cache: control_sd15_depth [fef5e48e]
2024-02-13T07:21:56.820560470Z 2024-02-13 07:21:56,820 - ControlNet - INFO - Loading preprocessor: depth_zoe
2024-02-13T07:21:56.820595653Z 2024-02-13 07:21:56,820 - ControlNet - INFO - preprocessor resolution = 512
```
Repository: fastapi
Issue: Use `RootModel` as query parameter
Labels: question

### Discussed in https://github.com/tiangolo/fastapi/discussions/11101

<div type='discussions-op-text'>

<sup>Originally posted by **WarpedPixel** February  6, 2024</sup>
### First Check

- [X] I added a very descriptive title here.
- [X] I used the GitHub search to find a similar question and didn't find it.
- [X] I searched the FastAPI documentation, with the integrated search.
- [X] I already searched in Google "How to X in FastAPI" and didn't find any information.
- [X] I already read and followed all the tutorial in the docs and didn't find an answer.
- [X] I already checked if it is not related to FastAPI but to [Pydantic](https://github.com/pydantic/pydantic).
- [X] I already checked if it is not related to FastAPI but to [Swagger UI](https://github.com/swagger-api/swagger-ui).
- [X] I already checked if it is not related to FastAPI but to [ReDoc](https://github.com/Redocly/redoc).

### Commit to Help

- [X] I commit to help with one of those options 👆

### Example Code

```python
# fail.py
# uvicorn fail:app --reload

from fastapi import FastAPI
from pydantic import RootModel

SimpleID = RootModel[str]

app = FastAPI()

@app.get("/test1")
async def t1(param1: SimpleID):     # BUG: expected to be query param, but it is body
    return {"message": f"Hello {param1}"}

# @app.get("/test2")
# async def t2(param1: Annotated[SimpleID, Query()]): # this workaround does NOT work
#     return {"message": f"Hello {param1}"}

@app.get("/test3")
async def t3(param1: SimpleID, param2: str):  # param2 is query param as expected, param1 is NOT
    return {"message": f"Hello {param1} {param2}"}
```


### Description

Simple types should be usable as query params. Pydantic provides `RootModel` (instead of `BaseModel`) precisely to implement types with rich validation based on native types like `list` or `str` (instead of the more typical object/dict). Types built with `RootModel` do not seem to work as query params, *even if it is functionally equivalent* to a simple type like `str`.
Furthermore, we cannot force them to work as query params with `Query()` annotations either.
**NOTE:** the example is overly simplified, my RootModel-derived types do much more in terms of validation.

### Operating System

macOS

### Operating System Details

_No response_

### FastAPI Version

0.108.0

### Pydantic Version

2.5.3

### Python Version

Python 3.11.4

### Additional Context

I am not sure this is possible, but I would like for FastAPI to understand RootModel derived types, if the root type is one of the types that can be used as query param, then it should work as such.
The workaround in my case is to use `str` parameters in my FastAPI calls then manually convert to the rich Pydantic type that does validation. Which kind of defeats the purpose of FastAPI rich type annotations.</div>
Repository: rust
Issue: Missed optimization opportunity with `.chars().count()`
Labels: I-slow,T-compiler,S-has-mcve,C-optimization

I'm not sure whether it's a missed optimization on LLVM's side or rustc's codegen preventing LLVM from applying an optimization, but this:
```rust
pub fn fun(str: &str) -> usize {
    str.chars().count() - str.chars().count()
}
```
... doesn't get reduced to `return 0;`, but actually calls `do_count_chars` twice (checked on release + nightly, x86_64 + aarch64):
```
playground::fun: # @playground::fun
# %bb.0:
	pushq	%r15
	pushq	%r14
	pushq	%r12
	pushq	%rbx
	pushq	%rax
	movq	%rsi, %r14
	movq	%rdi, %r15
	cmpq	$32, %rsi
	jae	.LBB0_1
# %bb.2:
	movq	core::str::count::char_count_general_case@GOTPCREL(%rip), %r12
	jmp	.LBB0_3

.LBB0_1:
	movq	core::str::count::do_count_chars@GOTPCREL(%rip), %r12

.LBB0_3:
	movq	%r15, %rdi
	movq	%r14, %rsi
	callq	*%r12
	movq	%rax, %rbx
	movq	%r15, %rdi
	movq	%r14, %rsi
	callq	*%r12
	subq	%rax, %rbx
	movq	%rbx, %rax
	addq	$8, %rsp
	popq	%rbx
	popq	%r12
	popq	%r14
	popq	%r15
	retq
                                        # -- End function
```

A more useful example could be:
```rust
pub fn padding(pad: usize, str: &str) -> usize {
    if pad > str.chars().count() {
        pad - str.chars().count()
    } else {
        0
    }
}
```

<!-- TRIAGEBOT_START -->

<!-- TRIAGEBOT_ASSIGN_START -->

<!-- TRIAGEBOT_ASSIGN_DATA_START$${"user":"DianQK"}$$TRIAGEBOT_ASSIGN_DATA_END -->

<!-- TRIAGEBOT_ASSIGN_END -->
<!-- TRIAGEBOT_END -->
Repository: vscode
Issue: [Feature] : Make scopes with different starting lines and same ending line in outline model appear in editor sticky scroll
Labels: feature-request,editor-sticky-scroll

Original issue is here https://github.com/microsoft/vscode/issues/204918

When the outline model features scopes that start on different lines and end on the same lines, only the scope with the highest starting line number is shown in sticky scroll, when the outline model is used to compute the sticky lines. 

This can be confusing, especially in CSS for example where often times, CSS selectors are on different lines and end their scopes end on the same line.
Repository: excalidraw
Issue: Allow to disable / overwrite keyboard shortcuts
Labels: enhancement

It would be great to allow developers to overwrite keyboard shortcuts or even disable them entirely.
Repository: vscode
Issue: Merge conflict editor: automatically "Accept Combination"
Labels: help wanted,feature-request,merge-editor

<!-- ⚠️⚠️ Do Not Delete This! feature_request_template ⚠️⚠️ -->
<!-- Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct/ -->
<!-- Please search existing issues to avoid creating duplicates. -->

<!-- Describe the feature you'd like. -->

Since I discovered the "accept combination" button several months ago, I've been using it consistently every time the option appears, and I've never encountered any issues with this approach.

I understand this could—in theory—produce an incorrect program, but even conflict-free merges can produce a non-working program, so I'm wondering if we could add a way to automatically accept combination so I don't have to manually click the button for each conflict (where the option is available)?

Related post on StackOverflow asking for the same thing: https://stackoverflow.com/questions/77038533/in-visual-studio-code-is-there-any-way-to-accept-combination-automatically

/cc @hediet