sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: flutter
Issue: Audit and clean up of GCP resource requirements
Labels: team-infra,P2,infra: device lab,triaged-infra

### Is there an existing issue for this?

- [X] I have searched the [existing infra issues](https://github.com/flutter/flutter/issues?q=is%3Aopen+is%3Aissue+label%3Ateam-infra)

### Type of Request

infra task

### Infrastructure Environment

Devicelab

### What is happening?

Our current spec on GCP bots (minimum CPU arch, etc.) seem to need an audit and cleanup (e.g. https://chrome-internal.googlesource.com/infradata/config/+/refs/heads/main/starlark/common/envs/flutterprj.star#43) .

This bug is to track cleanup/updating these values.

### Steps to reproduce

Step 1:
Step 2:
..
Step n:


### Expected results

I expect to see X when Y is finished.
Repository: neovim
Issue: autocmds with `once=true` can execute more than once
Labels: bug,api,has:plan,events

### Problem

The code below will execute the autocmd recursively.
This is of course a simplified repro. The one I encountered in the wild was not so obvious.

```lua
vim.api.nvim_create_autocmd("User", {
  once = true,
  pattern = "test",
  group = vim.api.nvim_create_augroup("test", { clear = true }),
  callback = function()
    vim.api.nvim_exec_autocmds("User", { pattern = "test" })
  end,
})
vim.api.nvim_exec_autocmds("User", { pattern = "test" })
```

### Steps to reproduce

run the repro with `nvim -u bug.lua`

### Expected behavior

the autocmd should only be ran once

### Neovim version (nvim -v)

nightly

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

na

### Operating system/version

linux

### Terminal name/version

kitty

### $TERM environment variable

xterm-256-kitty

### Installation

bob
Repository: pytorch
Issue: [dynamo]: `assert counter.frame_count == 1` is a bad practice when checking for no graph breaks
Labels: triaged,enhancement,oncall: pt2,module: dynamo

### Proposed Solution
Prefer `counters['graph_breaks']` or `fullgraph=True`/`nopython=True`

### 🐛 Describe the bug

First identified: https://github.com/pytorch/pytorch/pull/110683/files?diff=split&w=0#r1349139286

Bad uncaught graph breaks:

Graph break happens at beginning of function.
```python

from torch._dynamo.testing import CompileCounter
from torch._dynamo.utils import counters

counters.clear()
counter = CompileCounter()

def f(x):
    torch._dynamo.graph_break()
    x += 1
    return x

compiled_f = torch._dynamo.optimize(counter)(f)

x = torch.tensor([1])
compiled_f(x)

print("frame count", counter.frame_count) # 1
print("Graph breaks", len(counters["graph_break"])) # 1 
assert counter.frame_count - 1 == len(counters["graph_break"]) # fail
```

Graph break occurs at end of function:
```python
from torch._dynamo.testing import CompileCounter
from torch._dynamo.utils import counters

counters.clear()
counter = CompileCounter()

def f(x):
    x += 1
    return torch._dynamo.graph_break()

compiled_f = torch._dynamo.optimize(counter)(f)

x = torch.tensor([1])
compiled_f(x)

print("frame count", counter.frame_count) # 1
print("Graph breaks", len(counters["graph_break"])) # 1
assert counter.frame_count - 1 == len(counters["graph_break"]) # fail
```

Current usage:
1. https://github.com/search?q=repo%3Apytorch%2Fpytorch+assertequal+frame_count&type=code
2. https://github.com/search?q=repo%3Apytorch%2Fpytorch+assert+frame_count&type=code

CC: @voznesenskym @ezyang 

### Versions

main

cc @ezyang @anijain2305 @chauhang @penguinwu @voznesenskym @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @chenyang78 @kadeng @msaroufim @wconstab @bdhirsh @Xia-Weiwen @aakhundov
Repository: material-ui
Issue: [material-ui][Divider] Consistent spacing with text aligned left
Labels: component: divider,package: material-ui

### Duplicates

- [X] I have searched the existing issues

### Latest version

- [X] I have tested the latest version

### Summary 💡

When a horizontal Divider has text with `textAlign="left"`, the spacing before the text depends on its size. It would be nice if the spacing was consistent, possibly even configurable.

### Examples 🌈

Here are 3 dividers with text, note that the space before the text changes depending on its size:

https://codesandbox.io/s/modern-bird-ht8gst?file=/src/App.js


### Motivation 🔦

When you have a couple dividers on a page, it seems odd that their text is not aligned consistently.
Repository: flutter
Issue: Flutter packages socket_fail_test error 
Labels: f: integration_test,P2,team-framework,triaged-framework,from: manual-qa

### 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)

### Steps to reproduce

Install Flutter version 3.16.0-6.0.pre.2 (Note: this also fails in the latest stable version: 3.13.6)

Clone the Flutter packages repository: git clone [GitHub - flutter/packages: A collection of useful packages maintained by the Flutter team](https://github.com/flutter/packages)

Navigate to the packages/integration_test directory: cd packages/integration_test

Run the command flutter pub get

Run the command: flutter test test/socket_fail_test.dart >> socket_fail_test_logs.txt

### Expected results

Running all tests should succeed.

### Actual results

The following test case fails.

Logs:

```console
package:integration_test/integration_test.dart': Failed assertion: line 257 pos 14: 'info.serverUri != null': is not true.
  dart:core                                              _AssertionError._throwNew
  package:integration_test/integration_test.dart 257:14  IntegrationTestWidgetsFlutterBinding.enableTimeline
```

### Code sample

<details><summary>Code sample</summary>

```dart
[Paste your code here]
```

</details>


### Screenshots or Video

<details>
<summary>Screenshots / Video demonstration</summary>

[Upload media here]

</details>


### Logs

[socket_fail_test_logs.txt](https://github.com/flutter/flutter/files/12834257/socket_fail_test_logs.txt)




### Flutter Doctor output

<details><summary>Doctor output</summary>

```console
Flutter Doctor Output:

[✓] Flutter (Channel master, 3.16.0-6.0.pre.2, on macOS 14.0 23A344 darwin-arm64, locale en-UY)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.3)
[✓] IntelliJ IDEA Community Edition (version 2023.2.2)
[✓] VS Code (version 1.82.3)
[✓] Connected device (2 available)
```

</details>
Repository: go
Issue: expvar: default "memstats" key causes STW
Labels: Performance,NeedsInvestigation,compiler/runtime

### What version of Go are you using (`go version`)?

go1.21.2

### Does this issue reproduce with the latest release?

Yes

### What operating system and processor architecture are you using (`go env`)?

N/A

### What did you do?

Import `expvar`, consume all registered keys.

### What did you expect to see?

No application-wide impact.

### What did you see instead?

The "memstats" key calls the old `runtime.ReadMemStats` API, which involves an application-wide stop-the-world while it collects the most up-to-date possible data.

https://cs.opensource.google/go/go/+/refs/tags/go1.21.2:src/expvar/expvar.go;l=365

The `expvar` package doesn't guarantee the most up-to-date data. Users who need that (test code that wants to measure single allocations) would use `runtime.ReadMemStats` directly. We should be able to get a version of the data that's sufficient for `expvar`'s use without the app-wide STW; most of it is available in `runtime/metrics` already.

I intend to tackle this over the next several weeks.

CC @golang/runtime , and @mknyszek in particular
Repository: go
Issue: runtime: MemStats fields should document their runtime/metrics equivalents
Labels: Documentation,NeedsInvestigation,compiler/runtime

### What version of Go are you using (`go version`)?

go1.21.2

### Does this issue reproduce with the latest release?

Yes

### What operating system and processor architecture are you using (`go env`)?

N/A

### What did you do?

Considered updating a library to use the new `runtime/metrics` API, examined https://pkg.go.dev/runtime#MemStats docs to get the details on how to make the move.

### What did you expect to see?

References on each field describing which metric names to use with the `runtime/metrics` package to get the data without a call to `runtime.ReadMemStats` (and the resulting app-wide stop-the-world pause). A lot of these metrics are pretty subtle, and it takes very careful reading to make sure that the data from the new source will have the same meaning as the data from the old source.

### What did you see instead?

The docs for `runtime.MemStats` and `runtime/metrics` do not reference each other's fields.

CC @golang/runtime  , and @mknyszek in particular
Repository: flutter
Issue: [ios][engine][plugin]Make FlutterError compatible with Swift Error
Labels: platform-ios,engine,P2,a: plugins,team-ios,triaged-ios

### 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

This is so that iOS swift plugins can use `FlutterError` as Swift Error. 

Context: https://github.com/flutter/packages/pull/4753/files#r1349212583

### Proposal

The easiest way is to make `FlutterError` a subclass of `NSError`, so it auto conforms to Swift Error (we can't put a public extension of FlutterError since we don't have Swift code in the engine). 

For now the workaround is to put an empty extension in the plugin: 

```
extension FlutterError: Error {} 
```
Repository: godot
Issue: Color picker does not pick up color
Labels: bug,topic:editor,needs testing,topic:gui

### Godot version

v4.1.1.stable.official [bd6af8e0e]

### System information

Godot v4.1.1.stable - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1660 Ti (NVIDIA; 31.0.15.3623) - Intel(R) Core(TM) i5-9400F CPU @ 2.90GHz (6 Threads)

### Issue description

Picking up color with the color picker only works on surfaces that can't be selected. If it can be selected, it gets focused in the scene tree instead, closing the previous color picker dialog by loading the newly selected node's inspector information.

It seems to be connected with this [issue](https://github.com/godotengine/godot/issues/10249), maybe even just a special case - reporting it just in case it is not

### Steps to reproduce

Add colorful Sprite2D node
Add ColorRect node
Select ColorRect node
Open color dialog in Inspector tab
Press "Pick a color from the screen" button
Hover over Sprite2D node - Notice the color picker dialog updating the value as you move the cursor
Left click while cursor is over Sprite2D node

Expected behaviour:
The last active values are saved inside the ColorRect node's color value and the "Pick a color from the screen" mode is disabled

Actual behaviour:
The ColorRect node's color value does not update
The Sprite2D node is selected, filling the Inspector tab with it's values instead 

### Minimal reproduction project

N/A
Repository: tensorflow
Issue: tensorflow-intel binary packaging
Labels: stat:awaiting tensorflower,type:build/install,subtype:cpu-intel,TF2.14

### Issue type

Build/Install

### Have you reproduced the bug with TensorFlow Nightly?

Yes

### Source

binary

### TensorFlow version

2.13, 2.14

### Custom code

No

### OS platform and distribution

Windows x86_64

### Mobile device

_No response_

### Python version

_No response_

### Bazel version

_No response_

### GCC/compiler version

_No response_

### CUDA/cuDNN version

_No response_

### GPU model and memory

_No response_

### Current behavior?

`tensorflow-intel` is packaged with the TF native libraries stored as pyd files which wrap in some of the Python code. Other versions of tf (e.g. macOS and Linux) package native libraries in the default formats (e.g. `so` or `dylib`). In SIG-JVM/TensorFlow-Java we're planning on redoing our native builds so that we pull in pre-built binaries from the TF python release as that will make our builds much simpler and less error prone, but we can't re-use the Windows builds as the native binary depends on Python symbols. I reached out to SIG-BUILD and they told me to open an issue here. @mraunak

### Standalone code to reproduce the issue

```shell
Download the whl and unzip it.
```


### Relevant log output

_No response_
Repository: TypeScript
Issue: Adding an overload to .filter breaks specific inference with nested functions
Labels: Bug,Help Wanted

### 🔎 Search Terms

"filter inference overload", "filter boolean"

### 🕗 Version & Regression Information

- This is the behavior in every version I tried, and I reviewed the FAQ for entries about inferred types 


### ⏯ Playground Link

https://tsplay.dev/wQbrvN

### 💻 Code

```ts
type NonFalsy<T> = T extends false | 0 | "" | null | undefined | 0n
    ? never
    : T;

// Comment this out
interface Array<T> { filter(predicate: BooleanConstructor, thisArg?: any): NonFalsy<T>[]; }

const id = <T,>() => (t: T) => !!t;

['foo', 'bar'].filter(id())
//                    ^?
```


### 🙁 Actual behavior

Nested function inference works (type parameter is `string`) without the additional overload, and fails (type parameter is `unknown`) with the overload (even though the added overload isn't the one in use).

### 🙂 Expected behavior

Nested function inference should work regardless of whether an overload is added to .filter or not.

### Additional information about the issue

Adding the overload is described in https://github.com/microsoft/TypeScript/issues/50387 and made popular by `ts-reset`. 
It's possible the overload *order* may matter in this instance, but difficult to test when the main overloads are coming from lib definitions.
Repository: godot
Issue: Forward+ and Mobile Renderers Graphical Glitches on Intel Haswell IGP
Labels: platform:linuxbsd,topic:rendering,documentation

### Godot version

v4.2.dev.custom_build [bfd78bb91]

### System information

Godot v4.2.dev (bfd78bb91) - Zorin OS 16.3 - X11 - GLES3 (Compatibility) - Mesa DRI Intel(R) HD Graphics 4400 (HSW GT2) () - Intel(R) Core(TM) i5-4310U CPU @ 2.00GHz (4 Threads)

### Issue description

I was testing one of the official Godot 4 demos, specifically the [3D AI navigation project](https://github.com/godotengine/godot-demo-projects/tree/master/3d/navigation) on my Linux Laptop. And this is what I was greeted with **(flashing colours warning):**

With the Vulkan Renderer:
![Godot Vulkan Error](https://github.com/godotengine/godot/assets/63003002/6c74f6b4-497c-4379-ad79-908591b72c7e)

And the Mobile Renderer, is **so much worse**:
![Godot Vulkan Mobile Error](https://github.com/godotengine/godot/assets/63003002/7486a199-8193-4a4a-a4ef-267866139369)

Playing the game also brings the same graphical errors:
![Vulkan Mobile Playing game](https://github.com/godotengine/godot/assets/63003002/44155327-e825-491a-bdff-a8803069d6aa)

The compatibility mode is absolutely fine. Am I missing something? Or maybe (most likely) my laptop is just not compatible with Vulkan.

### Steps to reproduce

**Not recommended but:**
1. Open a 3d godot demo project, and make sure to use Vulkan and/or Mobile Renderers.
2. Wait for it to load.

### Minimal reproduction project

[3d-navigation.zip](https://github.com/godotengine/godot/files/12835065/3d-navigation.zip)
Repository: pytorch
Issue: Using ChainedScheduler with ReduceLROnPlateau leads to unexpected keyword argument error
Labels: module: optimizer,triaged,needs design,module: LrScheduler

### 🚀 The feature, motivation and pitch

I want to use ReduceLROnPlateau together with another schedule, say a warm-up or cosine schedule.
My understanding is that the suggested way to do that is through ``ChainedScheduler``.
However, when I try to call ``step`` with a validation loss, I get a ``TypeError``, since ``ChainedScheduler`` doesn't accept ``kwargs`` in ``step``.
I'm still pretty new to pytorch, so please let me know if I'm overlooking some other solution.

I don't see a super obvious solution. One would be to pass all ``kwargs`` to all steps in the chain, but that would mean all the schedulers would have to accept arbitrary ``kwargs`` which would hide errors. The other option that comes to mind would be to use inspect to figure out which parameters to pass to which scheduler. Or the ``step`` of ``ChainedScheduler`` could take a dict mapping from schedulers to parameters, but that seems more complicated than just calling all the schedules explicitly and not using ``ChainedScheduler``.

### Alternatives

_No response_

### Additional context

_No response_

cc @vincentqb @jbschlosser @albanD @janeyx99 @crcrpar
Repository: go
Issue: net: unexpected slowness with cgo resolving a domain that is manually added to /etc/hosts on Darwin
Labels: OS-Darwin,NeedsInvestigation

### What version of Go are you using (`go version`)?

<pre>
$ go version
go version go1.21.2 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>
GO111MODULE=''
GOARCH='arm64'
GOBIN=''
GOCACHE='/Users/travisbischel/Library/Caches/go-build'
GOENV='/Users/travisbischel/Library/Application Support/go/env'
GOEXE=''
GOEXPERIMENT=''
GOFLAGS=''
GOHOSTARCH='arm64'
GOHOSTOS='darwin'
GOINSECURE=''
GOMODCACHE='/Users/travisbischel/go/pkg/mod'
GONOPROXY='''
GONOSUMDB=''
GOOS='darwin'
GOPATH='/Users/travisbischel/go'
GOPRIVATE=''
GOPROXY='https://proxy.golang.org,direct'
GOROOT='/opt/homebrew/Cellar/go/1.21.2/libexec'
GOSUMDB='sum.golang.org'
GOTMPDIR=''
GOTOOLCHAIN='auto'
GOTOOLDIR='/opt/homebrew/Cellar/go/1.21.2/libexec/pkg/tool/darwin_arm64'
GOVCS=''
GOVERSION='go1.21.2'
GCCGO='gccgo'
AR='ar'
CC='cc'
CXX='c++'
CGO_ENABLED='1'
GOMOD='/Users/travisbischel/t2/go.mod'
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/vn/34y9tqdx6998z47x1dwbg_f40000gn/T/go-build2882954796=/tmp/go-build -gno-record-gcc-switches -fno-common'
</pre></details>

### What did you do?

Modify /etc/hosts to contain the following line:
```
93.184.216.34 foo.example.local
```

Then have a tiny Go module:
```
--- go.mod ---
module foo

go 1.21.0

--- main.go ---
package main

import (
	"fmt"
	"net"
)

func main() {
	c, err := net.Dial("tcp", "foo.example.local:80")
	fmt.Println(c, err)
	if err == nil {
		c.Close()
	}
}
```

Then:
```
time go run main.go
```

### What did you expect to see?
This to be quick.

### What did you see instead?
This is slow -- always >5s.
If you use GODEBUG=netdns=go, this is fast.

I was debugging the difference between how librdkafka uses getaddr info and how Go uses getaddrinfo.
librdkafka uses a single flag, `AI_ADDRCONFIG`. Go uses [this const](https://github.com/golang/go/blob/b72bbaebf9035c59806fd8073f2582e2d07764d5/src/net/cgo_darwin.go#L9) which, with the mask, ends up being `AI_CANONNAME`.

If I change [this line](https://github.com/golang/go/blob/8b6e0e6e8eb3a86ef1454a52a11bf75a077c56c5/src/net/cgo_unix.go#L153) to 0x400 to match AI_ADDRCONFIG, the cgo resolver is fast.

I don't really understand the ramifications of the 0x1407 mask, nor which flag should be used when -- but it is odd that _this_ specific single flag that is used forces getaddrinfo to always take 5s, whereas a different flag incantation is fast (and the pure Go resolver is also fast).

Please close this issue if this cannot be / wont be addressed, but I hope this is enough information to go on and to potentially fix _something_.
Repository: rust
Issue: Missing label direction is not properly handled when using at&t syntax, with segfault on windows.
Labels: I-crash,A-LLVM,A-inline-assembly,C-bug

I tried this code:

```rust
use std::arch::asm;

fn main() {
    println!("Hello, world!");

    unsafe {
        asm!(
            r#"
        jmp 9999
    9999:
            "#,
            options(nostack, att_syntax),
        );
    }
}


```

I expected to see this happen: An `invalid operand for instruction` error message, or something similar.

Instead, this happened: rustc segfaulted (windows). On Godbolt (Linux?), the code was accepted without error.
If `att_syntax` is removed, expected behaviour returns.

`rustc --version --verbose`:
```
rustc 1.73.0-nightly (03a119b0b 2023-08-07)
```
Repository: rust
Issue: Incorrect error about assignment to borrowed value when using destructuring and conditional returns
Labels: A-lifetimes,T-compiler,C-bug

I expect this code to compile:

```rust
struct Foo {
    blah: i32
}

fn some_fn(x: &mut Foo) -> &i32 {
    if x.blah == 2 {
        let Foo { blah } = x;
        
        if 333 == 444 {
            return blah;
        }
    }
    
    x.blah = 333;
    &x.blah
}
```

But I'm getting an E0506:

```
error[E0506]: cannot assign to `x.blah` because it is borrowed
  --> src/main.rs:14:5
   |
5  | fn some_fn(x: &mut Foo) -> &i32 {
   |               - let's call the lifetime of this reference `'1`
6  |     if x.blah == 2 {
7  |         let Foo { blah } = x;
   |                   ---- `x.blah` is borrowed here
...
10 |             return blah;
   |                    ---- returning this value requires that `x.blah` is borrowed for `'1`
...
14 |     x.blah = 333;
   |     ^^^^^^^^^^^^ `x.blah` is assigned to here but it was already borrowed

For more information about this error, try `rustc --explain E0506`.
error: could not compile `playground` (bin "playground") due to previous error
```

If the second level of `if` is commented out (lifting the return out into the first level of `if), it compiles fine. 

[Playground link](https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=b036f408b79716b02923b9ee8bb67f00)

rustc versions tried:

- local rustc 1.75.0-nightly (e0d7ed1f4 2023-10-01)
- stable from playground (1.73)
- nightly from playground (2023-10-05)
Repository: pytorch
Issue: torch.distributed.pipeline skip module throws assert error that portal.grad is not None
Labels: oncall: distributed

### 🐛 Describe the bug

I am exploring `torch.distributed.pipeline` module specifically the skip feature. My standard manual micro batch pipelines implemented using torch distributed rcp with manual micro batching seem to work okay, its when I convert it to `Pipe` with `skip` when I start to see the following error:

```
python3.8/site-packages/torch/distributed/pipeline/sync/skip/portal.py", line 228, in backward
    assert portal.grad is not None
AssertionError
```
I have tried a few toy problems to narrow it down and can confirm that skip works okay with `nn.Linear` layers. In this case I am using it in combination with `nn.Embedding` where I run into this issue. I have tried the same on both torch 2.0 and torch 2.1 and see the same error.

A mock code simulating the error can be found here for reproduction purposes (py3.8): 
https://gist.github.com/suneeta-mall/a7d1aa86732e67470758fa75db134a0b

### Versions

torch.__version__: '2.1.0+cu118'
Python 3.8.10 

cc @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @osalpekar @jiayisuse @H-Huang @kwen2501 @awgu @penguinwu