sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: electron
Issue: [Bug]: Missing user agent client hints
Labels: platform/windows,bug :beetle:,status/confirmed,has-repro-gist,27-x-y,28-x-y

### Preflight Checklist

- [X] I have read the [Contributing Guidelines](https://github.com/electron/electron/blob/main/CONTRIBUTING.md) for this project.
- [X] I agree to follow the [Code of Conduct](https://github.com/electron/electron/blob/main/CODE_OF_CONDUCT.md) that this project adheres to.
- [X] I have searched the [issue tracker](https://www.github.com/electron/electron/issues) for a bug report that matches the one I want to file, without success.

### Electron Version

19.0.6

### What operating system are you using?

Windows

### Operating System Version

Windows 10 version 21H2

### What arch are you using?

x64

### Last Known Working Electron version

_No response_

### Expected Behavior

Low entropy user agent client hints as:
sec-ch-ua
sec-ch-ua-mobile
sec-ch-ua-platform

Should be send on each request, also if not received header "Accept-CH" in the server response

High entropy user agent client hints as:
sec-ch-ua-arch
sec-ch-ua-full-version
sec-ch-ua-full-version-list
sec-ch-ua-platform-version
sec-ch-ua-bitness
sec-ch-ua-wow64

Aren't working at all
Electron never sending those, even after response with "Accept-CH" header

### Actual Behavior

After PR #34481  the low entropy hints are working from the second request (after receiving header "Accept-CH" in the server response)
If testing in Chrome Incognito, can see that those headers are always sent in Chrome versions 102/103

The high entropy hints aren't working at all, also after PR #34481 

I did test with new BrowserWindow/loadURL to:
https://www.bing.com/search?q=car

In newest Chrome versions the user agent is not include the full browser version
The server can find it only by high entropy hint sec-ch-ua-full-version or sec-ch-ua-full-version-list

Example of http request headers in Chrome 103.0.5060.53
...
Host: www.google.com
Connection: keep-alive
sec-ch-ua: ".Not/A)Brand";v="99", "Google Chrome";v="103", "Chromium";v="103"
sec-ch-ua-full-version-list: ".Not/A)Brand";v="99.0.0.0", "Google Chrome";v="103.0.5060.53", "Chromium";v="103.0.5060.53"
sec-ch-ua-mobile: ?0
User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/103.0.0.0 Safari/537.36
sec-ch-ua-arch: "x86"
sec-ch-ua-full-version: "103.0.5060.53"
sec-ch-ua-platform-version: "10.0.0"
sec-ch-ua-bitness: "64"
sec-ch-ua-wow64: ?0
sec-ch-ua-model: 
sec-ch-ua-platform: "Windows"
...


### Testcase Gist URL

https://gist.github.com/yellow1234/20370bd58dd1c0b5f1c385dcb0afe5ac

### Additional Information

_No response_
Repository: pytorch
Issue: Distributed Store `get` doesn't work well with `add`
Labels: high priority,triage review,oncall: distributed,module: docs,triaged,module: c10d

### 🐛 Describe the bug

While I was working on sharing distributed seed for DataLoader, internal users encountered a problem that the distributed process will hang when trying to `get` an integer counter from the distributed store (`PrefixStore`).

While I dived deeper on this problem, I found a comment here:
https://github.com/pytorch/pytorch/blob/590d3e5774110e4657dcaa6acdb387ef69e41b47/torch/distributed/distributed_c10d.py#L242-L246

Do we have a plan to remove the legacy store to make sure `get` is working with `add`? If not, should we add some notes to recommend users to use `add(key, 0)` to retrieve the value saved by `store.add`?
https://pytorch.org/docs/stable/distributed.html#torch.distributed.Store.add

### Versions

fblearner

cc @ezyang @gchanan @zou3519 @pietern @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @osalpekar @jiayisuse @SciPioneer @H-Huang @kwen2501 @svekars @holly1238
Repository: go
Issue: test: failures in `maplinear.go` with `iterdelete: too slow` on linux-mips*-rtrk
Labels: NeedsInvestigation,arch-mips

```
#!watchflakes
post <- pkg == "maplinear.go" && `iterdelete: too slow` && goarch ~ `mips`
```

```
##### ../test
# go run run.go -- maplinear.go
exit status 2
panic: iterdelete: too slow: 40000 inserts: 2.755283259s; 80000 inserts: 9.103015471s


goroutine 1 [running]:
panic({0xc7440, 0xc000010030})
	/tmp/gobuilder-mips64/go/src/runtime/panic.go:987 +0x4e0 fp=0xc00006ce88 sp=0xc00006cdd0 pc=0x50670
main.checkLinear({0xd8451, 0xa}, 0x9c4, 0xdfda8)
	/tmp/gobuilder-mips64/go/test/maplinear.go:59 +0x27c fp=0xc00006cf58 sp=0xc00006ce88 pc=0xb9dfc
main.main()
	/tmp/gobuilder-mips64/go/test/maplinear.go:158 +0x25c fp=0xc00006cf80 sp=0xc00006cf58 pc=0xbab4c
runtime.main()
	/tmp/gobuilder-mips64/go/src/runtime/proc.go:250 +0x308 fp=0xc00006cfd8 sp=0xc00006cf80 pc=0x54158
runtime.goexit()
	/tmp/gobuilder-mips64/go/src/runtime/asm_mips64x.s:617 +0x4 fp=0xc00006cfd8 sp=0xc00006cfd8 pc=0x88d7c
```

`greplogs -l -e FAIL\\s+maplinear\\.go`
[2022-06-21T17:32:57-530511b/linux-mips64-rtrk](https://build.golang.org/log/daf8ea1bc14be063d82e4214ca901fca1fb760f4)
[2020-02-21T20:59:36-bc5fc6d/linux-mips64-rtrk](https://build.golang.org/log/9446dad0ae42ddd7ec9fbf3af6d243da00746f4a)

Perhaps a good first step would be to double-check whether the test's timing expectations are reasonable for this builder?

(attn @golang/mips; CC @golang/runtime)
Repository: pytorch
Issue: [BE] Update ProcessGroupWrapper tests to test other collective message
Labels: oncall: distributed,triaged,better-engineering,module: c10d

### 🚀 The feature, motivation and pitch

Context of changes are in https://github.com/pytorch/pytorch/pull/79901.

Since the updated ProcessGroupWrapper message when `TORCH_DISTRIBUTED_DEBUG=DETAIL` is set will now print the op type, tensor shape, device type, etc. of the offending collective, we should also update the tests to check for these error messages. The tests in test_pg_wrapper.py currently only validate the error message for the current rank, but does not validate anything about the other rank's collective.

### Alternatives

_No response_

### Additional context

_No response_

cc @pietern @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @osalpekar @jiayisuse @SciPioneer @H-Huang @kwen2501
Repository: flutter
Issue: Runtime detection for html versus canvaskit renderer
Labels: engine,c: performance,dependency: dart,platform-web,P2,team-web,triaged-web

There are a number of optimizations we'd like to make to the framework that will only work with the canvaskit renderer. For now, we can turn these off with `kIsWeb`, but ideally we could runtime detect canvaskit and apply those optimizations there as well.

Specifically:

* Removal of opacity compositing
* Route transitions with toGpuImage

@yjbanov
Repository: godot
Issue: Incorrectly get an error when supplying null as a default value in ConfigFile.get_value() method
Labels: bug,topic:core

### Godot version

v3.4.4.stable.official [419e713a2]

### System information

Debian GNU/Linux, amdgpu, GLES3

### Issue description

Get the following error when using null as the default value, which is what is recommended in official examples.

	E 0:00:00.356   get_value: Couldn't find the given section "sound" and key "state", and no default was given.
  <C++ Error>   Condition "p_default.get_type() == Variant::NIL" is true. Returned: Variant()
  <C++ Source>  core/io/config_file.cpp:84 @ get_value()
  <Stack Trace> globals.gd:32 @ _ready()

**Quick Fix**: Changing null to "" makes the error go away.  This must be a bug.

### Steps to reproduce

	const config_path := "user://config.ini"
	var config := ConfigFile.new()

	config.load(config_path)
	if config.get_value("sound", "state", null) == null:
		config.set_value("sound", "state", 0)

### Minimal reproduction project

_No response_
Repository: pytorch
Issue: Modify _add_docstr to also set the correct module for the APIs
Labels: triaged,better-engineering,actionable,module: python frontend

Currently, many public APIs don't have their module set correctly to meet our requirements for public API (https://github.com/pytorch/pytorch/wiki/Public-API-definition-and-documentation). All the ops in `torch.linalg`, `torch.special`, `torch.sparse` etc currently use `_add_docstr` to correctly set the doc for all the functions in these modules. The proposal here is to modify `_add_docstr` to set the correct module. 

Once this is done, we can get rid of all the respective entries in this allowlist: https://github.com/pytorch/pytorch/blob/d67ce755ad1439b03d4abcc2d40496b20cfd570e/test/allowlist_for_publicAPI.json#L1786-L1827

cc. @albanD
Repository: go
Issue: runtime,cmd/go: `runtime: out of memory` in `runtime.sysMapOS` 
Labels: NeedsInvestigation,compiler/runtime

```
#!watchflakes
post <- `runtime: out of memory` && `^\s*runtime\.sysMapOS`
```

```
fatal error: runtime: out of memory

runtime stack:
runtime.throw({0xa25205?, 0x20000?})
	/tmp/workdir/go/src/runtime/panic.go:1047 +0x5d fp=0x243c4b248 sp=0x243c4b218 pc=0x43621d
runtime.sysMapOS(0xc001400000, 0xc000800000?)
	/tmp/workdir/go/src/runtime/mem_bsd.go:71 +0x11b fp=0x243c4b290 sp=0x243c4b248 pc=0x41975b
runtime.sysMap(0xe46c80?, 0x21b3a8000?, 0x42cb80?)
	/tmp/workdir/go/src/runtime/mem.go:142 +0x35 fp=0x243c4b2c0 sp=0x243c4b290 pc=0x419595
runtime.(*mheap).grow(0xe46c80, 0x6000292?)
	/tmp/workdir/go/src/runtime/mheap.go:1459 +0x23d fp=0x243c4b330 sp=0x243c4b2c0 pc=0x429c3d
runtime.(*mheap).allocSpan(0xe46c80, 0x6000292, 0x0, 0x1)
	/tmp/workdir/go/src/runtime/mheap.go:1191 +0x2e6 fp=0x243c4b3e0 sp=0x243c4b330 pc=0x4293c6
runtime.(*mheap).alloc.func1()
	/tmp/workdir/go/src/runtime/mheap.go:910 +0x65 fp=0x243c4b428 sp=0x243c4b3e0 pc=0x428d25
runtime.systemstack()
	/tmp/workdir/go/src/runtime/asm_amd64.s:492 +0x49 fp=0x243c4b430 sp=0x243c4b428 pc=0x464729
```

I notice that `sysMapOS` and `sysReserveOS` on BSD both invoke `mmap`, and `cmd/go` has recently started [using `syscall.Mmap`](https://cs.opensource.google/go/go/+/master:src/cmd/go/internal/mmap/mmap_unix.go;l=31;drc=06261062e9a1875338b04aaea3b6335d73b513a7) to access cached metadata during package loading.

I can think of the following possible explanations, but I'm not sure which (if any) is correct.
1. Perhaps a bug in `cmd/go` (perhaps on the indexing path) is causing it to use more memory than expected?
2. Perhaps a bug in the OpenBSD `mmap` implementation causes the `cmd/go` call to `syscall.Mmap` to steal an address that was supposed to be reserved for the runtime?
3. Perhaps a bug somewhere in the runtime causes it to drastically overshoot the heap target? (But I would expect the `syscall.Mmap` usage to instead cause it to undershoot, since the heap size used to compute `GOGC` will not include the Mmap'd data.)

This is another case where it would be really helpful to have some heap stats from the runtime when the process OOMs (#52546)...

(CC @golang/openbsd @golang/runtime)
Repository: flutter
Issue: `CupertinoSlider` with null `onChanged` doesn't display as disabled
Labels: framework,f: cupertino,P2,team-design,triaged-design

https://api.flutter.dev/flutter/cupertino/CupertinoSlider/onChanged.html claims that a `CupertinoSlider` with null `onChanged` will be displayed as disabled.  That doesn't seem to be the case visually, even though interaction is disabled.
Repository: storybook
Issue: Solidjs Support
Labels: feature request,needs triage

Is there any sort of plan to add support for Solidjs?
Repository: flutter
Issue: [Proposal] Support swipe gesture based navigation using the Navigator
Labels: c: new feature,framework,f: routes,would be a good package,c: proposal,P3,team-framework,triaged-framework

## Use case

Similar to how the CupertinoPageRoute can be popped using a swipe from the starting edge of the screen, it would be great if we could also add routes to the navigator's route history with a gesture controlled animation.

This behavior can be found for example in Safari (forward to next page in history: #76478), Slack mobile app (open last used chat from home), TikTok (opening profile from feed)

## Proposal

The difference between backwards and forward navigation is that in the case of backwards navigation, all routes are already in the route history and the current route only needs it's own animation controller to drive the relevant animations. However, in forward navigation, the relevant animation will be driven by the AnimationController of the new route which is not in the route history yet. Therefore, one must first add this new route to the navigator's route history but without triggering its transition animation and then access the route's AnimationController to pass to the previous route which then manually drives the animation using its gesture recognizer. The problem right now is that, to the best of my knowledge, the navigator does not allow routes to be added without already setting the route's controller.value to 1 and accessing the animation controller is also rather hacky. I think it would make sense to expose that step where `controller.foward()` or `controller.value = 1` is called at some point such that this behavior could *optionally* be altered.

The question then is where to best expose that animation controller. An idea would be to expose the controller inside the resolve function of the `TransitionDelegate`. By adding a function to the `RouteTransitionRecord` such as `markForAddManual()` returning the route's controller, that controller could then be provided to the previous route (and controller from other routes as well for complex transition logic), if it can handle the controller. (E.g. developers would extend `ModalRoute` to something like `GestureModalRoute`, which can take the controllers they need)

Please let me know your thoughts
Repository: rust
Issue: missing_copy_implementations warning appears on types which impl Iterator
Labels: A-lints,T-compiler,C-bug,T-libs,L-missing_copy_implementations

It has long been the policy of the standard library that Copy and Iterator on the same type is a footgun.

Therefore, the `missing_copy_implementations` lint should not fire on a type that impls Iterator.
Repository: pytorch
Issue: Support for learnable p Values in LPPOOL like Pool
Labels: module: nn,triaged,needs research

### 🚀 The feature, motivation and pitch

Make a LPPOOL layer that can learn p Values. The value for p = inf is max pooling and p = 1 is Average pooling. Different from LPPOOL where p = 1 is sum pooling.
The new LPPOOL has a p value for each feature map. So for each feature map a p Value can be learned.


### Alternatives

_No response_

### Additional context

I already tested this layer in my repo https://github.com/joerg-de/AdaptiveLearnableLPPool2d-test where I replaced the pooling layer of some well known Networks.
For shufflenet_v2_x0_5 Acc1 was around 1.9 % better and Acc5 was around 1.5 % better.
With 1024 additional Weights.

cc @albanD @mruberry @jbschlosser @walterddr @kshitij12345 @saketh-are
Repository: flutter
Issue: Tree red on "Windows framework_tests_widgets" failure
Labels: team,framework,P2,c: flake,team-framework,triaged-framework

Several failures in `flutter\packages\flutter\test\widgets\widget_inspector_test.dart`:

https://logs.chromium.org/logs/flutter/buildbucket/cr-buildbucket/8810172323157034049/+/u/run_test.dart_for_framework_tests_shard_and_subshard_widgets/test_stdout
Repository: TypeScript
Issue: Stack overflow in completion
Labels: Bug,Help Wanted

# Bug Report

<!--
  Please fill in each section completely. Thank you!
-->

### 🔎 Search Terms

completion, stack overflow

### 🕗 Version & Regression Information

2bc91a6b46af5071e002876545ce1badd924369b

### 💻 Code

Open file

https://github.com/microsoft/TypeScript/blob/2bc91a6b46af5071e002876545ce1badd924369b/tests/baselines/reference/binderBinaryExpressionStress.js#L12

Invoke completion after `0 + 1 + 2`

### 🙁 Actual behavior

```
RangeError: Maximum call stack size exceeded
    at visitNode (e:\\ts_gh\\built\\local\\tsserver.js:38721:31)
    at visitNode (e:\\ts_gh\\built\\local\\tsserver.js:30796:24)
    at forEachChild (e:\\ts_gh\\built\\local\\tsserver.js:31111:24)
    at visitNode (e:\\ts_gh\\built\\local\\tsserver.js:38735:17)
    at visitNode (e:\\ts_gh\\built\\local\\tsserver.js:30796:24)
    at forEachChild (e:\\ts_gh\\built\\local\\tsserver.js:31111:24)
```

### 🙂 Expected behavior

No exception
Repository: rust
Issue: [strict provenance] Provide a way to "create allocations at a fixed address"
Labels: T-lang,T-libs-api,WG-embedded,A-strict-provenance

This issue is part of the Strict Provenance Experiment - https://github.com/rust-lang/rust/issues/95228

On some platforms it makes sense to just take a hard-coded address, cast it to a pointer, and starting working with that pointer -- because there are external environment assumptions that say that certain things are provided at certain addresses.

This is perfectly fine for Rust as long as that memory is entirely *disjoint* from all the memory that Rust understands (`static` globals, stack allocations, heap allocations). Basically we can think of there being a single hard-coded provenance for "all the memory that is disjoint from the Abstract Machine", and that is the provenance we would like to use for these pointers created from hard-coded addresses. These restrictions make this operation a lot easier to specify than [`from_exposed_addr`](https://doc.rust-lang.org/nightly/std/ptr/fn.from_exposed_addr.html). (Remember: `from_exposed_addr` is *outside* of Strict Provenance. The goal of this issue is to provide a way to write such code while following Strict Provenance.)

In the spirit of the Strict Provenance APIs, that means we probably want a function that does this, and that we can attach suitable documentation to. There are some open questions for the syntax and semantics of that function:
- What should it be called? `make_alloc`, `assume_alloc`, `hard_coded_alloc`? I am leaning towards something with "alloc" because this function is basically like an allocator, except that you tell it at which address to allocate and you have to promise that that is Okay To Do.
- It should definitely take a `usize` for the address and return `*mut T`. Should it also take a size, saying how large this assumed allocation is?
- Should each invocation return a fresh provenance, or should they all return the same provenance? Probably the latter; using a fresh provenance comes with a bunch of restrictions that I doubt such low-level code will be happy about.
- (other things I have not thought of)

Cc @Lokathor who keeps mentioning this usecase every time I want to ban int2ptr casts. ;)
Tagging WG-embedded since that's where this kind of stuff mostly happens (AFAIK)
Repository: godot
Issue: Particles2D and CPUParticles2D with blend_mode Multiply blends particles incorrectly
Labels: bug,topic:2d,topic:particles

### Godot version

3.5-rc5

### System information

Windows 10. GLES3 and GLES2. GTX 1070 and Intel HD Graphics 5500.

### Issue description

Particles emitted by a Particles2D/CPUParticles2D node will blend incorrectly when a CanvasItemMaterial is added and blend mode is set to Multiply. This can be seen plainly when emiting many particles with a simple white texture.

Expectation:
![image](https://user-images.githubusercontent.com/46985106/176041949-d24c29f9-db8c-41ef-8c2b-0086d0fcd153.png)

Reality:
![image](https://user-images.githubusercontent.com/46985106/176041972-02787396-dbb8-43d0-8f0c-a4704f46df62.png)

### Steps to reproduce

1. Create a Godot project with a 2D scene.
2. Add a Particles2D/CPUParticles2D with a white texture.
3. Add a CanvasItemMaterial to the particles node and set blend_mode to Multiply.
4. Crank up the emitting property.
5. Behold a blue mass.

### Minimal reproduction project

[mrp_CPUParticles2D_blend_mode_multiply.zip](https://github.com/godotengine/godot/files/8995859/mrp_CPUParticles2D_blend_mode_multiply.zip)
Repository: godot
Issue: Texture2DArray lacks way of setting filter mode
Labels: enhancement,topic:rendering,documentation,topic:shaders

### Godot version

4.0 alpha 10

### System information

Win 10

### Issue description

I could not find a way to set the filter flags of a texture array. I created it using "Import As". There does not seem a way to create it using code (Texture Arrays have a .create() method in godot 3.x which has flags as an argument) and there is no documentation.

### Steps to reproduce

Import/"Import As"/Texture2DArray icon.png and try to change the filter mode.

### Minimal reproduction project

_No response_
Repository: godot
Issue: Vulkan: Separate near/far dof_blur_amount no longer available (unlike 3.x)
Labels: topic:rendering,regression,topic:3d

### Godot version

4.0.alpha10

### System information

Win 10/64 NVIDIA GeForce GTX 1060/PCIe/SSE2

### Issue description

Godot 4 has one setting for `dof_blur_amount`.
Godot 3 has two, one for near and one for far.
Since Godot isn't calculating real DOF, the near and far planes may need to be different values, so both should be provided.

Example 1:

I've written a physical camera for Out of the Ashes that calculates the real depth of field ranges based upon aperture and focal length of my lens. Then I position the DOF blur plane distances according to the math, set the transition value to the same as the distance, and simulate the amount of blur based upon math, a lot of experimentation, and my photographic experience.

Here you can see that the near plane has a strength of 0.12 and the far plane 0.062. Near distance 3.561, Far distance 4.619.

This is not possible in Godot 4, but should be.

![image](https://user-images.githubusercontent.com/632766/176052835-2c3fc84a-29ef-4064-8167-7f1f5252cf50.png)

Example 2:
At https://dofsimulator.net/en/ you can play with camera lenses and position of objects and see how the focus ranges are positioned. This demonstrates why having only one blur setting is physically incorrect.

Here is a 90mm lens at f/4, with the subject 20m away. You can see the background is nearly in focus in the sample image. This is because, as you see on the bottom, the far focus plane is very close to the infinity focus mark, where anything after that distance is in focus. You can see the hyperfocal distance is 34.91m, meaning if the subject is there or farther, the background will be perfectly in focus. However, that does NOT mean everything in front of the subject is also in focus! Imagine if someone's head or hand steps in front of the camera within 5m. Is it also going to be as nearly in focus as the background? No, it's going to be a very blurry mess.

![image](https://user-images.githubusercontent.com/632766/176054384-e3e1b100-43bf-48e4-9d80-96d364c3987e.png)

### Steps to reproduce

Built into the engine.

### Minimal reproduction project

_No response_