sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: godot
Issue: Arbitrary incorrect error messages opening a .tscn with missing dependency
Labels: enhancement,topic:core,topic:editor,usability

### Godot version

4.2.dev5

### System information

Godot v4.2.dev (0ca854232) - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3090 (NVIDIA; 31.0.15.3713) - Intel(R) Core(TM) i9-9900K CPU @ 3.60GHz (16 Threads)

### Issue description

When a scene has missing dependencies, Godot will seemingly arbitrarily display one of several different confusing error message.

These three are the easy cases I found:

Error while loading? Technically true but doesn't explain why.
![image](https://github.com/godotengine/godot/assets/39946030/c0461093-0590-4ea1-81d7-c55a1b525651)

Moved or deleted. But it shows the wrong .tscn name, which certainly has not been moved or deleted.
![image](https://github.com/godotengine/godot/assets/39946030/4d0e9abc-7a1e-49b2-a2e5-dce5ba276c00)

Missing dependencies. Good error, in this case: You can fix with Edit Dependencies.
![image](https://github.com/godotengine/godot/assets/39946030/6fb1b8ac-c4d8-43c1-a4a4-702428a982e4)

And indeed, Edit Dependencies can be used to diagnose the problem
![image](https://github.com/godotengine/godot/assets/39946030/a3abe882-6ad4-474f-a923-2a4c06a5ed1c)

----

Then it gets worse. The bad_instanced_scene* scenes show the same error messages from the instanced scene but do not display the correct .tscn, so it is unclear where the error is. Further, Edit Dependencies shows no problem.


### Steps to reproduce

Open `missing_dependencies.tscn` - verify that a reasonably easy to understand "Missing dependencies" error is shown. This is the good case. Well, not the best case but at least it points the user in a good direction, though it would be better if it had a direct button to "Edit Dependencies"

Open `error_while_loading.tscn` - verify that an unhelpful generic error message is displayed. This will usually cause the user to read the console, which may lead them in the right direction. Not good, but not terrible.

Open `moved_or_deleted.tscn` - Verify that a misleading error message informs you that `moved_or_deleted.tscn` has been moved or deleted, which is untrue.

Finally, repeat the process with `bad_instanced_scene{1,2,3}.tscn` which instantiate the above three test scenes. Notice how they produce the error messages from their dependent scenes but with the incorrect filename. There is no way to diagnose the issue with Edit Dependencies either.

As a control, I also have `working_instanced_scene.tscn` to show what happens when there are no errors.

### Minimal reproduction project

[testmissing.zip](https://github.com/godotengine/godot/files/12782757/testmissing.zip)
Repository: rust
Issue: The ABI of float types can be changed by `-Ctarget-feature`
Labels: P-high,T-compiler,I-unsound,A-floating-point,A-ABI,A-target-feature,T-opsem

A function that returns an `f32`/`f64` is not ABI-compatible with other functions *that have the same signature* on i686 when certain target features differ. It looks like one can disable the `x87` feature or enable the `soft-float` and then it will use different ways of passing floating-point arguments.

This is *unsound* as code calling methods from the standard library would now use the wrong registers to return results. In other words, setting `-Ctarget-feature=-x87` or `-Ctarget-feature=+soft-float` can introduce UB unless the standard library is rebuilt with the same flags. We therefore should reject these flags, to avoid the UB. This issue tracks that problem, and transitioning it to a hard error.

(SIMD types have a similar problem, but we are dealing with that differently. See https://github.com/rust-lang/rust/issues/116558.)

Current status:
- It is a hard error to toggle some features with `#[target_feature]` (will be shipped in 1.84) -- not a breaking change since those features were not allowed in `#[target_feature]` before either, only the error message changed
- It is a warning to toggle them with `-Ctarget-feature` announcing that this will become a hard error in the future (will be shipped in 1.84)
- Not all relevant features are properly detected yet, not even on tier 1 targets
Repository: PowerToys
Issue: FancyZones for multiple monitors are often swapped when connecting with RDP
Labels: Issue-Bug,Needs-Triage

### Microsoft PowerToys version

0.73

### Installation method

Chocolatey

### Running as admin

Yes

### Area(s) with issue?

FancyZones

### Steps to reproduce

1. Install PowerToys on a machine with two monitors.
2. Connect to the machine using RDP with "Use all my monitors for the remote session" enabled.
3. Set specific zones for the two monitors using FancyZones.
4. Disconnect RDP (leave session running)
5. Reconnect RDP

### ✔️ Expected Behavior

Selected Zones from the previous session should remain.

### ❌ Actual Behavior

Often the zones of the two monitors are swapped and the zones set for Monitor 1 appear on Monitor 2 and vice-versa.

### Other Software

_No response_
Repository: rust
Issue: Heisenwarning `unused_braces` with `tracing`
Labels: A-lints,A-diagnostics,T-compiler,C-bug,D-invalid-suggestion,D-incorrect,L-unused_braces

### Code

```Rust
#[tracing::instrument]
fn foo() -> u8 { 42 }

#[tracing::instrument]
fn bar() -> u8 { 
    42
}
```


### Current output

```Shell
warning: unnecessary braces around block return value
 --> src/lib.rs:2:16
  |
2 | fn foo() -> u8 { 42 }
  |                ^^  ^^
  |
  = note: `#[warn(unused_braces)]` on by default
help: remove these braces
  |
2 - fn foo() -> u8 { 42 }
2 + fn foo() -> u8 42
  |
```


### Desired output

_No response_

### Rationale and extra context

I'd expect both `foo` and `bar` to either trigger the warning or not, since they differ only in formatting (in fact, `bar` is just a `rustfmt`-ed `foo`). Looking just at the suggestion, it seems that not warning is better (since this suggestion is not applicable), but this might be a bug in `tracing` and not in `rustc`.

### Other cases

_No response_

### Anything else?

Initially found on [URLO](https://users.rust-lang.org/t/probably-invalid-compiler-hint/100687).
Repository: terminal
Issue: Terminal installed on Windows Server 2022 with pre-installation package will not open
Labels: Needs-Repro,Issue-Bug,Area-UserInterface,Product-Terminal,Priority-2,Culprit-Centennial

### Windows Terminal version

1.18.2681.0

### Windows build number

20348.1970

### Other Software

Nothing special

### Steps to reproduce

Install Terminal:
Download and extract current pre-installation zip.
Install with Add-AppxProvisionedPackage -Online -PackagePath $Bundle -LicensePath $License. Command completes without error.
Terminal now shows shortcut in Start Menu and the installation exists in C:\Program Files\Windows\Apps
Clicking Start Menu shortcut does nothing. Attempting to run the executable wt.exe or WindowsTerminal.exe results in a permissions error.

Exact same behavior happens on a clean install of Windows Server vNext

### Expected Behavior

Windows Terminal opens.

### Actual Behavior

Terminal Shortcut does nothing, calling wt.exe or WindowsTerminal.exe results in permissions error.

Zip version of Windows Terminal will work.
Repository: go
Issue: x/text/language: Matcher returns Arabic with No confidence instead of falling back to first language
Labels: NeedsInvestigation

<!--
Please answer these questions before submitting your issue. Thanks!
-->

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version go1.21.0 darwin/arm64
</pre>

### Does this issue reproduce with the latest release?

Yes

### What operating system and processor architecture are you using (`go env`)?

<details><summary><code>go env</code> Output</summary><br><pre>
$ go env
GO111MODULE=''
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/idees/Library/Caches/go-build'
GOENV='/Users/idees/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/idees/redacted/go/pkg/mod'
GONOPROXY=''
GONOSUMDB=''
GOOS='darwin'
GOPATH='/Users/idees/redacted/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/opt/homebrew/Cellar/go/1.21.0/libexec'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/opt/homebrew/Cellar/go/1.21.0/libexec/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.21.0'
GCCGO='gccgo'
AR='ar'
CC='cc'
CXX='c++'
CGO_ENABLED='1'
GOMOD='redacted'
GOWORK=''
CGO_CFLAGS='-O2 -g'
CGO_CPPFLAGS=''
CGO_CXXFLAGS='-O2 -g'
CGO_FFLAGS='-O2 -g'
CGO_LDFLAGS='-O2 -g'
PKG_CONFIG='pkg-config'
GOGCCFLAGS='-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -ffile-prefix-map=/var/folders/l1/mykxjxqx4sggl2dq3144wcym0000gn/T/go-build2246364886=/tmp/go-build -gno-record-gcc-switches -fno-common'
</pre></details>

### What did you do?

I have a `language.Matcher` built with an array of languages. According to [the doc](https://pkg.go.dev/golang.org/x/text/language#NewMatcher) for `NewMatcher`, "The first element is used as the default value in case no match is found" so I expected `language.English` to be returned for a language that was not added to the matcher. For example, when I pass in the parsed `pa-PK`, I get `language.Arabic` with a confidence enum of `No`.

https://go.dev/play/p/qjnoDk9kXXM

### What did you expect to see?

I expected a match to `language.English`, the default language in the matcher.

### What did you see instead?

I get `language.Arabic` with a confidence enum of `No`.
Repository: go
Issue: crypto/tls: broken link in Config.KeyLogWriter docs 
Labels: Documentation,NeedsInvestigation

It looks like the docs for the NSS key_log_format have moved permanently, in early 2022: https://github.com/mdn/content/pull/12731

The link from `crypto/tls.Config.KeyLogWriter` points to a page that doesn't redirect to a new permanent home, and instead shows a "404 Not Found" message.

https://firefox-source-docs.mozilla.org/security/nss/index.html mentions `mozilla_projects_nss_key_log_format` in a section with the warning "References below this point are part of the deprecated documentation and will be ported in the future.", but does not include a link to the new home.

I'm not sure where the current home for that documentation is. It looks like the link in Go's documentation has been broken for a while, and that the owner of those external docs may not intend to repair it. Go's docs should point to a reputable source for the format specification.

### What version of Go are you using (`go version`)?

<pre>
$ go1.21 version
go version go1.21.1 darwin/arm64
</pre>

### Does this issue reproduce with the latest release?

Yes, and also in the development branch.

### What operating system and processor architecture are you using (`go env`)?

N/A

### What did you do?

Read docs for `crypto/tls.Config.KeyLogWriter`, tried to visit the reference at https://developer.mozilla.org/en-US/docs/Mozilla/Projects/NSS/Key_Log_Format .

```
$ go1.21 doc crypto/tls.Config.KeyLogWriter
package tls // import "crypto/tls"

type Config struct {
    // KeyLogWriter optionally specifies a destination for TLS master
    // secrets in NSS key log format that can be used to allow external
    // programs such as Wireshark to decrypt TLS connections. See
    // https://developer.mozilla.org/en-US/docs/Mozilla/Projects/NSS/Key_Log_Format.
    // Use of KeyLogWriter compromises security and should only be used for debugging.
    KeyLogWriter io.Writer

    // ... other fields elided ...
}
```

See also https://pkg.go.dev/crypto/tls@master#Config

And for what it's worth, https://docs.rs/rustls/latest/rustls/trait.KeyLog.html#required-methods which includes an identical link.

### What did you expect to see?

Documentation for the resulting key log entry format.

### What did you see instead?

A redirect to https://firefox-source-docs.mozilla.org/security/nss/legacy/key_log_format/index.html, resulting in:
```
404 Not Found
Code: NoSuchKey
Message: The specified key does not exist.
Key: security/nss/legacy/key_log_format/index.html
RequestId: 9E8NJMXRE2NNZGDF
HostId: Tb4D2Wi5Yh7dkbQonEOgbBePK/D5cmZ/UN1oOFQrBdtisJ3ybZQJVTOFSPMcyBWfFIgqYzXOdtI=
```
Repository: flutter
Issue: [Impeller] Optimization ideas for Contents::RenderToSnapshot
Labels: engine,P3,e: impeller,team-engine,triaged-engine

* If the contents are a SolidColorContents and the geometry is rectangular, then create a 1x1 texture with an appropriate scale on the CPU and avoid encoding a render pass/allocating a bigger texture.
* If the contents are a LinearGradient, and it has two stops at 0, 1, and it is rectangular geometry and the direction of the gradient is axis aligned, then create a 1x2 or 2x1 texture with the stop colors filled in.
* If the contents are a TiledTextureContents and the geometry is rectangular similar to TextureContents, pass through the texture as is.

Potentially more.

These are probably not world changing optimizations! but just some ideas for tinkering.
Repository: PowerToys
Issue: [QuickAccent] Adding of Bosnian Language special letters code repetition for languages of former Yugoslavia
Labels: Needs-Triage,Product-Quick Accent

### Description of the new feature / enhancement

I was about to add Bosnian language special letters to the QuickAccent, then I have noticed that Croatian and Serbian are already added what is great, but what I have noticed that they use completely identical function, GetDefaultLetterKeyHR and GetDefaultLetterKeySR are absolutely same code 2 times. 

Now if add Bosnian GetDefaultLetterKeyBH we will have 3 times identical code since all 3 languages use absolutely same special letters that are mapped on same keys, and adding additional identical function would just cause code repetition and clutter.

My proposal would be to refactor code and:
- Move all 3 functions next to each other, this would make future maintenance of code easier
- Extract general function GetDefaultLetterKeyYU or GetDefaultLetterKeyBCS and use it in all 3 national functions (Bosnian, Croatian, Serbian), thus reduce repetition of the code


### Scenario when this would be used?

Bosnian language would be added to the QuickAccent

### Supporting information

_No response_
Repository: godot
Issue: Pushing against window edge in MOUSE_MODE_CONFINED causes significant performance drop
Labels: bug,topic:porting,topic:input,performance

### Godot version

4.1.1

### System information

Windows 10

### Issue description

When mouse mode is set to confined using MOUSE_MODE_CONFINED and the mouse is "pushed" against the edge of the window, significant performance loss is noticed.


Empty project:
![image](https://github.com/godotengine/godot/assets/110487141/2f36369b-0dc5-40f0-bb1f-1e0dbcdff2fd)

My game:
![image](https://github.com/godotengine/godot/assets/110487141/0c609064-7bc5-4068-9bb9-3b55b75d9619)


### Steps to reproduce

Create a new project
Set VSYNC off
Attach code to a Node3D:
```
extends Node3D

func _ready():
	Input.set_mouse_mode(Input.MOUSE_MODE_CONFINED)
```
Run project
"Push" mouse against window edge

### Minimal reproduction project

[CameraBugTest.zip](https://github.com/godotengine/godot/files/12785059/CameraBugTest.zip)
Repository: godot
Issue: Error when using an explicit type in variable assignment, but := works fine
Labels: bug,discussion,topic:gdscript

### Godot version

v4.2.dev5.official [e3e2528ba]

### System information

Ubuntu 20.04 LTS

### Issue description

See title.

### Steps to reproduce

Create a script called `my_class.gd` with the following:
```
extends RefCounted

static func static_do_something():
  print("hello from MyClass.static_do_something")
  
func do_something():
  print("hello from MyClass.do_something()")
```

Create another script called `my_other_class.gd` in the same directory as `my_class.gd` with the following:
```
extends RefCounted

var MyClass = preload("my_class.gd")

func also_do_something():
  print("hello from MyOtherClass")
  MyClass.static_do_something() # ok

  var test1 = MyClass.new() # ok
  test1.do_something() # ok
  test1 = 5 # ok

  var test2 := MyClass.new() # ok
  test2.do_something() # ok
  # test2 = 5 # ok: Value of type "int" cannot be assigned to a variable of type "res://my_class.gd".

  var test3 : MyClass = MyClass.new() # "MyClass" is a variable but does not contain a type.
```

Notice that the last line prints an error even though `:=` used earlier was fine.

As a bonus, it would be great if the fix removes the same error from these too.  Otherwise, I can turn the below into a Godot proposal.
```
var test2 := MyClass.new()
test2 is MyClass: # "MyClass" is a variable but does not contain a type.
  print("test2 is MyClass)

func print_my_class(my_class : MyClass): # "MyClass" is a variable but does not contain a type.
  print(my_class)
```

Also, I know `class_name` could be a workaround. However considering `MyClass` is considered a type, `res://my_class.gd`, I suspect that the other ways of using the type ought to have no errors too.

### Minimal reproduction project

N/A

EDIT: slightly better description
Repository: flutter
Issue: [Impeller] Support indirect command encoding in the HAL.
Labels: c: new feature,P3,e: impeller,team-engine,triaged-engine
Repository: next.js
Issue: Refreshing the router within an error boundary doesn't actually refresh
Labels: bug,Navigation,linear: next

### Link to the code that reproduces this issue

https://github.com/goncy/error-boundary-repro

### To Reproduce

* Start the app 
* See error
* Click on reset
* Should get correct data but doesn't happen

Loom video: [https://www.loom.com/share/Visual-Studio-Code-routejs-error-boundary-repro-Visual-S[…]4451d81e1956cfd87d190?sid=2660c81f-4334-4ec2-87d4-5464f09a8c5e](https://www.loom.com/share/Visual-Studio-Code-routejs-error-boundary-repro-Visual-Studio-Code-15-September--6638dbdb7ff4451d81e1956cfd87d190?sid=2660c81f-4334-4ec2-87d4-5464f09a8c5e)

### Current vs. Expected behavior

Current: Reseting the router doesn't re execute the routes to get new data
Expected: It should

### 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: #1 SMP Fri Jan 27 02:56:13 UTC 2023
    Binaries:
      Node: 18.12.1
      npm: 8.19.2
      Yarn: 1.22.19
      pnpm: 8.7.1
    Relevant Packages:
      next: 13.5.4-canary.9
      eslint-config-next: 13.5.4-canary.9
      react: 18.2.0
      react-dom: 18.2.0
      typescript: 5.2.2
    Next.js Config:
      output: N/A
```


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

App Router, Routing (next/router, next/navigation, next/link)

### Additional context

In the example I tried it it was a co-located error.js for a parallel route, then I moved it to a page level error.js but still nothing

<sub>[NEXT-1670](https://linear.app/vercel/issue/NEXT-1670/refreshing-the-router-within-an-error-boundary-doesnt-actually-refresh)</sub>
Repository: ollama
Issue: Unsupported nVidia cards still cause Cuda Toolkit/Drivers to be installed
Labels: bug,nvidia

When installing on an older machine with an nVidia card which is not supported by cuda 11/12, the installer still forces you to install all of the nVidia stuff despite the GPU not being supported. Ideally this should skip installing that stuff and just use the CPU instead.
Repository: material-ui
Issue: [docs-infra] Additional white space left after expanding the code block
Labels: design,scope: docs-infra

### Steps to reproduce 🕹

This problem can be seen in any component page — for example, [Material UI's Button page](https://mui.com/material-ui/react-button/).

1. Go to a demo that has the "Expand code" button
2. Click on it to expand the code block
3. Then, click again on it to collapse the code block back
4. See how there's now an empty div creating additional white space

<img width="400" alt="Screen Shot 2023-10-02 at 16 01 08" src="https://github.com/mui/material-ui/assets/67129314/8636d93b-8896-43a6-ab61-77c410bb2b73">

### Expected behavior 🤔

The additional white space shouldn't be there — content spacing throughout the page should remain the same.
Repository: next.js
Issue: Localhost domains are not parsed on middlewares
Labels: bug,Runtime

### Link to the code that reproduces this issue

https://github.com/jackmerrill/nextjs-local-hostname-bug

### To Reproduce

1. Start application
2. Go to both `localhost:3000` and `test.localhost:3000`. 
3. Record the logged output in the Next.js console

### Current vs. Expected behavior

Current: Logs `localhost:3000 localhost`

Expected: Logs `test.localhost:3000 test.localhost`

The `NextRequest.nextUrl.host` and `NextRequest.nextUrl.hostname` should track subdomains on localhost (such as for testing on multi-tenant applications).

### 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: #1 SMP PREEMPT_DYNAMIC Sat, 23 Sep 2023 22:55:13 +0000
Binaries:
  Node: 20.8.0
  npm: 10.1.0
  Yarn: 1.22.19
  pnpm: N/A
Relevant Packages:
  next: 13.5.4-canary.9
  eslint-config-next: N/A
  react: 18.2.0
  react-dom: 18.2.0
  typescript: 5.1.3
Next.js Config:
  output: N/A
```


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

Middleware / Edge (API routes, runtime)

### Additional context

_No response_
Repository: tensorflow
Issue: `pip install tf-nightly[and-cuda]` fails for recent nightlies
Labels: stat:awaiting tensorflower,type:bug,comp:gpu,comp:gpu:tensorrt,TF2.14

### Issue type

Bug

### Have you reproduced the bug with TensorFlow Nightly?

Yes

### Source

source

### TensorFlow version

nightly

### Custom code

No

### OS platform and distribution

Google Colab

### Mobile device

_No response_

### Python version

3.10

### Bazel version

_No response_

### GCC/compiler version

_No response_

### CUDA/cuDNN version

_No response_

### GPU model and memory

_No response_

### Current behavior?

**The issue**

Attempting to run `pip install tf-nightly[and-cuda]` will download a ton of nightly candidates before installing one from mid-September (before tf bumped to cuda12).

Attempting to pin the more recent versions shows the error with recent nightlies.

```shell
pip install tf-nightly[and-cuda]==2.15.0.dev20231002
...
ERROR: Could not find a version that satisfies the requirement tensorrt-libs==8.6.1; extra == "and-cuda" (from tf-nightly[and-cuda]) (from versions: 9.0.0.post11.dev1, 9.0.0.post12.dev1, 9.0.1.post11.dev4, 9.0.1.post12.dev4)
ERROR: No matching distribution found for tensorrt-libs==8.6.1; extra == "and-cuda"
```

You can work around this with `pip install tf-nightly[and-cuda] --extra-index-url https://pypi.nvidia.com`.

**What should happen**
`pip install tf-nightly[and-cuda]` should not self conflict, and recent nighties should be installable via PyPI.

### Standalone code to reproduce the issue

https://colab.research.google.com/gist/mattdangerw/00acd58e43aabe7f80a74d595788bd86/tf-nightly-and-cuda.ipynb


### Relevant log output

_No response_