sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: godot
Issue: Instantiating a scene spawns in all the wrong places
Labels: discussion,topic:rendering,topic:3d

### Godot version

4.1.2

### System information

Linux 64-bit (Garuda/Arch)

### Issue description

What I Do Want To Happen:
1. I use Marker3D to indicate where I want something to spawn in (enemies, water drops, etc).
2. I instantiate the appropriate scene and add it into the tree
3. I tell the new child where its proper position should be upon arrival
4. It goes to that spot and does its thing.

### Steps to reproduce

What Actually Happens:
1. I use Marker3D to indicate where I want something to spawn in
2. I instantiate the appropriate scene and add it into the tree
3. I tell the new child where its proper position should be upon arrival
4. The new child tells me to go fly a kite. If there are multiple Marker3D nodes, it may spawn at any one of them, or it may default to Vector3.ZERO.
5. I skip attempting to use the Marker3D position and instead use a specific Vector3() coordinates.
6. The child laughs and continues to do whatever it wants (sometimes the Vector3() coordinates will at least lock it onto a particular axis).

### Minimal reproduction project

Inside you will find two examples: cave_entrance and battle_map. This is a widespread issue, but I think these two maps show the issue the best. I apologize for the lack of player controls; somehow my normal PC file got "corrupted" while deleting everything, so I just made a cube stand-in.

cave_entrance: GameState singleton defines the starting position as "left" yet it spawns on the right (and not even at the "right" Marker3D). Similarly, the intermittent water drop spawns at Vector3.ZERO rather than the drop Marker3D.

battle_map: The PC spawns in at the correct location, but the enemies do not. If you change the x positions of the Marker3Ds (or on straight Vector3() coordinates, you will notice that they will either stack on top of each other or else spread out off the map.
[SparrowMin.zip](https://github.com/godotengine/godot/files/13065035/SparrowMin.zip)
Repository: flutter
Issue: [Proposal] - Let users query the files available in an AssetBundle
Labels: c: new feature,framework,a: assets,customer: crowd,c: proposal,P3,team-framework,triaged-framework

### Is there an existing issue for this?

- [X] I have searched the [existing issues](https://github.com/flutter/flutter/issues)
- [X] I have read the [guide to filing a bug](https://flutter.dev/docs/resources/bug-reports)

### Use case

Here's the motivating use-case. In Swift UI, an image can be displayed based on only the asset's name:

```swift
Image("logo")
```

Flutter requires full file path specification:

```dart
Image.asset("assets/images/logo.png");
```

I'm working on a community clone of Swift UI capabilities in Flutter and we'd like to make it possible for our users to similarly display images by name. Implementing such a widget requires the ability to search or index the asset bundle.

There's a supposed workaround where we could [parse the asset manifest ourselves](https://stackoverflow.com/questions/68862225/flutter-how-to-get-all-files-from-assets-folder-in-one-list), however, this workaround depends upon internal framework details that could change at any time.

### Proposal

I'd like for Flutter to add methods to [Asset Bundle](https://api.flutter.dev/flutter/services/AssetBundle-class.html) to make it possible to inspect the files available in the asset bundle.

There could be any number of APIs for this purpose, but some APIs might look like this:

```dart
DefaultAssetBundle.of(context).findAssetsByName("logo");

DefaultAssetBundle.of(context).findAssetsByNameThatMatch(myNameRegex);

DefaultAssetBundle.of(context).findAssetsByExtension("png");

DefaultAssetBundle.of(context).assets;
```

We can discuss the finer details of such APIs in the discussion section.

**Known Complication:**
Not all bundles use the filesystem. For example, Flutter has [NetworkAssetBundles](https://api.flutter.dev/flutter/services/NetworkAssetBundle-class.html). There may be times where its nice to use file-system and network bundles interchangeably, but as this use-case demonstrates, there are also times where we care whether we're working with app asset files or network assets.
Repository: opencv
Issue: why opencv change minAreaRect returned angle from -90~0 to 0~90
Labels: duplicate,category: documentation,incomplete

### Describe the doc issue

as aboved

### Fix suggestion

_No response_
Repository: TypeScript
Issue: Don't suggest private fields in contexts where they are out of scope
Labels: Suggestion,Experience Enhancement

### 🔎 Search Terms

private fields suggestions

### 🕗 Version & Regression Information

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

### ⏯ Playground Link

https://www.typescriptlang.org/play?#code/C4TwDgpgBAogHgQwLZgDYQJIDsCWwoC8UA3gFBQVQBmA9jQFxRYCuSARhAE4A0pAvgG5SpUJFiIU6ADI4A1tCLxkaaAB9xy9NjxDSAY1QIAzkY2ToZSlCPAEwHHqg4jS8wAoAbglTMIjBFggAJSMXj7QzmYqJORWlJwQwMycWFAAxLQ0TqlhvkJWfMJWNnYO1Jw0SG4QEioy8oyudXIQIVHoMXGUOFRQbk3oAHTOAxDVtdItQUGdXXEJSSlQNZoQ9RD5c4VzUAvJqVgQAO7tYyvm60GbFIWxFAkIACY0WKgg6ZmMLOxculZ6LxsnGYemANE4bmI1DoUD4jQmmFwwBmljmwAAFs5BhkYURMtdYfxhACsDZlgjCKdBlQKlVURQAPQMqAAFU471EOCwAHMoAAiKh8qABR7WCDQDF2KAYrgRUwIazMbnciA2HAvaGcD50fhXUhAA

### 💻 Code

```ts
type ExampleInit = {
    foo: number,
};

type ExampleLike = Example | ExampleInit;

class Example {
    static isExample(value: any): value is Example {
        return #foo in value;
    }

    static from(exampleLike: ExampleLike): Example {
        if (Example.isExample(exampleLike)) {
            return exampleLike;
        }
        return new Example(exampleLike);
    }

    readonly #foo: number;

    constructor({ foo }: ExampleInit) {
        this.#foo = foo;
    }
}

const example = Example.from({
    // Try typing "f" and see that there is a suggestion for #foo
});
```


### 🙁 Actual behavior

We get suggestions for private fields:

![Screenshot from 2023-10-23 15-45-37](https://github.com/microsoft/TypeScript/assets/8518303/98788e34-ce81-495a-ac9f-24fcd4c164ae)

This is particularly annoying as the private fields tend to be suggested first, so tab-completion tends to produce broken code.

### 🙂 Expected behavior

The private field should not be included as a suggestion when outside the class scope where the field is defined.

### Additional information about the issue

_No response_
Repository: next.js
Issue: When external library polyfills it breaks Next.js
Labels: bug

### Link to the code that reproduces this issue

https://github.com/luthfib/auth0-nextjs-repro/tree/fetch-issue

### To Reproduce

1. Run intstructions as outlined in https://github.com/luthfib/auth0-nextjs-repro/tree/fetch-issue
2. This will also require get env variables for auth0

### Current vs. Expected behavior

This bug is linked to https://github.com/vercel/next.js/issues/56969, however I was able to fix it by doing the following: 

https://github.com/luthfib/auth0-nextjs-repro/blob/fetch-issue/app/api/auth/%5Bauth0%5D/route.ts

I believe Next.js crashed before I included the following line because of this line the Next.js code... https://github.com/vercel/next.js/blob/6ed4fddf8a55f956ca6a714e081a384880d45e52/packages/next/src/server/future/route-modules/app-route/module.ts#L413

I think either Next.js needs to warn in the documentation that any library that polyfills fetch will break Next.js or the error if the library polyfills should be more clear. The library that polyfills in question is the following: https://github.com/Vermonster/fhir-kit-client/blob/3b36f30f29f5bbeac356feed0db6ce3be3eeb711/lib/http-client.js#L3. 

Spent quite a bit of time with a few engineers to figure out what was causing the issue, so some kind of warning would be helpful. 


### Verify canary release

- [X] I verified that the issue exists in the latest Next.js canary release

### Provide environment information

```bash
Operating System:
  Platform: darwin
  Arch: arm64
  Version: Darwin Kernel Version 22.6.0: Fri Sep 15 13:41:28 PDT 2023; root:xnu-8796.141.3.700.8~1/RELEASE_ARM64_T6000
Binaries:
  Node: 18.11.0
  npm: 8.19.2
  Yarn: N/A
  pnpm: 8.9.2
Relevant Packages:
  next: 13.5.6
  eslint-config-next: 13.5.5
  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

### Additional context

_No response_
Repository: next.js
Issue: Mobile swipe-navigation through history causes UI flickering
Labels: bug,Navigation,Pages Router

### Link to the code that reproduces this issue

https://codesandbox.io/p/sandbox/getserversideprops-ios-flickering-7plmvk

### To Reproduce

1. Start the application on mobile device
2. Click counter (`Link` component)
3. Swipe from left edge of screen to navigate to previous page

### Current vs. Expected behavior

Navigating through page history shouldn't cause UI to flicker and glitch

### Verify canary release

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

### Provide environment information

```bash
iOS 15.6
```


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

Not sure, Data fetching (gS(S)P, getInitialProps), Routing (next/router, next/navigation, next/link)

### Additional context

Example above uses `getServerSideProps`, but this issue also occurs using other rendering methods:
- https://github.com/vercel/next.js/issues/10465

Issue is confirmed present in:
- 13.5.7-canary.16
- 13.5.6
- 13.1.6
Repository: go
Issue: all: instruction alignment optimizations for assembly routines, good starter projects
Labels: Performance,help wanted,NeedsFix,compiler/runtime

Issue https://github.com/golang/go/issues/56474 added support for instruction alignment on the amd64 architecture. This is achieved with the PCALIGN assembly pseudo instruction, which inserts NOP's to align the next instruction to a given boundary.

Since this feature is pretty new on amd64, we didn't had much time to check which assembly routines in the runtime/libraries would benefit com instruction alignment. In most cases, the effect of instruction alignment is minimal, but on critical subroutines and critical innermost loops it can deliver a significant boost in performance.

Some examples:
 - [bytealg.Count](https://go-review.googlesource.com/c/go/+/516455)
 - [bytealg.Index](https://go-review.googlesource.com/c/go/+/516435)

There are multiple places were we might get interesting results using PCALIGN in amd64 assembly:
- `runtime` functions (memmove, memclr, etc)
- `internal/bytealg`
- `crypto/*`
- `*amd64*.s` hot loops/critical sections in assembly code

Generally a 16-byte alignment works fine, while 32-byte is better when aligning AVX2 instructions. Be careful when overusing it, routines may end up slower than before.

In order to verify if there are any speedups you can run the benchmarks for the affected package with a higher count, at least `-count=10`. Then use [benchstat](https://pkg.go.dev/golang.org/x/perf/cmd/benchstat) to compare the before/after results. I'd say anything higher than 3-5% consistently is worth submitting.

Code alignment is already supported on ppc64, arm64, loong64 and riscv64. Feel free to look into improvements for these architectures as well! You can rely on qemu-static to run the benchmarks after compiling the tests with `go test -c`.

Happy hacking!
Repository: kubernetes
Issue: Cleanup: validation errors which are conditional on another field
Labels: help wanted,sig/docs,sig/architecture,needs-triage

We have lots of validation which checks mutually-incompatible fields or dependent fields.  E.g. errors of the form "X: must not be set when Y is set" or "X: may only be set when Y is Z".  The error strings are all over the map.  I think we should encapsulate those into a few functions, like `field.Invalid` to make them easier and more consistent.

e.g. `field.IncompatibleWith(thisPath, otherPath)` or `field.DependsOnValue(thisPath, otherPath, someConstant)`

This does not eliminate the need to CHECK the fields, but it makes the errors better.

Someone can rack up a lot of commits by doing this carefully.  I'd do it as:
* 1 commit to add the functions and tests
* 1 commit per function or small-group of related functions converted to use these (and test cases!)

My WIP if someone wants to jump on this https://github.com/kubernetes/kubernetes/compare/master...thockin:kubernetes:validate_field_incompatible_errors?expand=1

I am sure that these are not sufficient for all cases, but will get a lot.

@BenTheElder @alexzielenski
Repository: godot
Issue: In tool script, providing constructor that takes typed array with untyped array produces 'Error constructing a GDScriptInstance.'
Labels: bug,topic:gdscript,topic:editor

### Godot version

v4.2.beta.custom_build [fd33c7b32], v4.1.2.stable.official [399c9dc39]

### System information

Godot v4.1.2.stable & v4.2.beta - Windows 10.0.19045 - Vulkan (Forward+) - dedicated NVIDIA GeForce GTX 1060 (NVIDIA; 31.0.15.3114) - Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz (12 Threads)

### Issue description

Passing an untyped array, ie. `Array` to a constructor that expects a typed array, ex. `Array[int]` in a tool script causes `Error constructing a GDScriptInstance.`. I don't know if it should accept it, but it should at least give a better error message.

Specific error messages:
```
 Error constructing a GDScriptInstance.
 res://something.gd:12 - Invalid type in function 'new' in base 'GDScript'. The array of argument 1 (Array) does not have the same element type as the expected typed array argument.
```

### Steps to reproduce

Load the project or reload the scene containing the tool script to see the error message.

### Minimal reproduction project

[Test.zip](https://github.com/godotengine/godot/files/13066937/Test.zip)
Repository: go
Issue: go/doc,x/pkgsite: `[Context.Err]` not linked in rendered doc comments
Labels: Documentation,NeedsInvestigation

[Variables section](https://pkg.go.dev/context#pkg-variables) has invalid links:

![Screenshot 2023-10-23 at 08-50-35 context package - context - Go Packages](https://github.com/golang/go/assets/585279/ba9d3813-a446-47c6-8c72-1cfb5610a45c)

See `[Context.Err]`.
Repository: stable-diffusion-webui
Issue: [Feature Request]: Latent Consistency model
Labels: enhancement

### Is there an existing issue for this?

- [X] I have searched the existing issues and checked the recent builds/commits

### What would your feature do ?

https://latent-consistency-models.github.io/

they distilled dreamshaperv7 into latent consistency model and it works with only 4 steps for sampling, but quality is somehow worse that default dreamshaper: https://huggingface.co/spaces/SimianLuo/Latent_Consistency_Model

### Proposed workflow

.

### Additional information

.
Repository: TypeScript
Issue: String completions on top-level properties are not consistent between JSX attributes and object properties
Labels: Domain: Completion Lists,Possible Improvement

### 🔎 Search Terms

string completions jsx overloads call function object properties

### 🕗 Version & Regression Information

- This is the behavior in every version I tried

### ⏯ Playground Link

https://www.typescriptlang.org/play?ts=5.3.0-dev.20231023#code/JYWwDg9gTgLgBAJQKYEMDG8BmUIjgIilQ3wG4BYAKCoBMk0AbFIuTAVwDsNgIO4YkAZxgAKZgHMAXART4AlNIBuEYDQqU6jZkladuvfkNETp+AEbylKtVSoDhI-PNJwA9K7gByFJ7gAfLzNPWw16JhZ2LhgePnsYACYxKCk4AG84MBwwU1k4AF8FOGVVdU1wnUj9WKNEkzSMrNMLfMLim2pKOMT0zIhsgnwWl3cvHxCy7V0omLgAFSMRXrBBaR7GmUGC6Q42BgZSsMnK6IN5hyWV+qWmzcKdvfUqAB4z+CWAXic3AD5hj29fAFPEEqEA

### 💻 Code

```ts
import React from "react";

declare function test(arg: "a"): void;
declare function test(arg: "b"): void;

test(""); // 'a' | 'b'

declare function test2(arg: { prop: "a" }): void;
declare function test2(arg: { prop: "b" }): void;

test2({ prop: "" }); // 'a'

declare function Test(props: { prop: "a" }): null;
declare function Test(props: { prop: "b" }): null;

<Test prop="" />; // 'a' | 'b'
```


### 🙁 Actual behavior

Completions list only `'a'` at `test2` call but they list both `'a'` and `'b'` at the `Test` JSX "call".

### 🙂 Expected behavior

Both of those represent the same thing - a function call. I'd expect the behavior to be consistent at both sites.

### Additional information about the issue

JSX case (and the first `test` call) goes through `getStringLiteralCompletionsFromSignature`. This gathers all potential overloaded candidates - so it's able to look for completions in all of them. The regular call to `test2` grabs the completions from `fromContextualType() || fromContextualType(ContextFlags.None)` so it can only list completions based on one of the overloads (the first one).

If we assume that the overload-capable solution is better then should it also be extended to handle deeper scenarios, like this?
```ts
declare function Test2(props: { prop: { deep: "a" } }): null;
declare function Test2(props: { prop: { deep: "b" } }): null;

<Test2 prop={{ deep: "" }} />; // 'a'
```
Repository: TypeScript
Issue: String completions are inconsistent between JSX attributes and JSX expressions
Labels: Domain: Completion Lists,Possible Improvement

### 🔎 Search Terms

string completions jsx attribute expression

### 🕗 Version & Regression Information

- This is the behavior in every version I tried

### ⏯ Playground Link

https://www.typescriptlang.org/play?ts=5.3.0-dev.20231023#code/JYWwDg9gTgLgBAJQKYEMDG8BmUIjgIilQ3wG4BYAKCoBMk0AbFIuTAVwDsNgIO4AVJAGcYACjA4wQgFxwA3nAkQws-CnxwAvgEpZHNgwYVKdRsyStO3XgOFilU2QoeqARhp16DRqlQA8giKKkgC8+BoA9AB8pHAREXAA5CiJcAA+Sa6J-oHwDiFy4ZpxMXEJydmUQA

### 💻 Code

```ts
import React from "react";

declare function Test(props: { prop: "a" }): null;
declare function Test(props: { prop: "b" }): null;

<Test prop="" />; // 'a' | 'b'
<Test prop={""} />; // 'a'
```


### 🙁 Actual behavior

Both sites provide different completions.

### 🙂 Expected behavior

Both of those sites represent the very same JS code and they should provide the very same completions.

### Additional information about the issue

_No response_
Repository: PowerToys
Issue: Move windows based on "Relative position" disables expand to multiple zones + minimize/maximize window hotkeys
Labels: Issue-Bug,Product-FancyZones,Needs-Triage

### Microsoft PowerToys version

0.74.1

### Installation method

PowerToys auto-update

### Running as admin

Yes

### Area(s) with issue?

FancyZones

### Steps to reproduce

Move windows based on "Relative position" imo is the most natural and intuitive option by far to have active, even on 1-row layouts.. like, it works perfectly. However, one thing I realized I really wanted was to also be able to minimize (especially) active windows, when I want to hide clutter.

The issue is that this apparently doesn't work when "Relative position" option is active. Switching to "Zone index" is something I really don't want to do, because it's much worse for basic grid snapping.

On top of minimize/maximize not working with "Relative position", the Win + Ctrl + Alt +[arrow keys] expand to multiple zones hotkey also gets disabled.

### ✔️ Expected Behavior

With "Move windows based on: Relative position" active, I expect minimize/maximize to still work, especially with a 1d layout.. or I expect an alternative hotkey to be offered, like with the "Switch between windows in the current zone" option.

### ❌ Actual Behavior

With "Move windows based on: Relative position" active, minimize current window is overriden, and can't be used.

### Other Software

_No response_
Repository: frp
Issue: nathole need a ip filter for zerotiler-like virtual network[Feature Request] 
Labels: proposal

### Describe the feature request

Like this issue: https://github.com/fatedier/frp/issues/3457, the zerotier ip has a low TTL.
So , the nathole will use zerotier for transmission. But the zerotier can be very slow.

I think add a config `nat_address_filter` like `192.168.0.0/24` which will remove these ips from  nathole.

To test that thought, I add several code in https://github.com/fatedier/frp/blob/dev/client/visitor/xtcp.go#L278:

```go

func (sv *XTCPVisitor) makeNatHole() {
	xl := xlog.FromContextSafe(sv.ctx)
	xl.Trace("makeNatHole start")
	if err := nathole.PreCheck(sv.ctx, sv.helper.MsgTransporter(), sv.cfg.ServerName, 5*time.Second); err != nil {
		xl.Warn("nathole precheck error: %v", err)
		return
	}

	xl.Trace("nathole prepare start")
	prepareResult, err := nathole.Prepare([]string{sv.clientCfg.NatHoleSTUNServer})
	if err != nil {
		xl.Warn("nathole prepare error: %v", err)
		return
	}
        // Simple Filter one IP
	var ipAddress = "172.16.1.70"
	var filteredAddrs []string
	for _, addr := range prepareResult.AssistedAddrs {
		addrParts := strings.Split(addr, ":")
		if addrParts[0] != ipAddress {
			filteredAddrs = append(filteredAddrs, addr)
		}
	}
	prepareResult.AssistedAddrs = filteredAddrs    
```

I test the code above, it will make frp go direct not through zerotier.


### Describe alternatives you've considered

 **The things I don't know is that if this is a good implement or not. Totally newbie for go.**

So if this is good and this config make sense, I will make a PR for this.
### Affected area

- [ ] Docs
- [ ] Installation
- [X] Performance and Scalability
- [ ] Security
- [ ] User Experience
- [ ] Test and Release
- [ ] Developer Infrastructure
- [ ] Client Plugin
- [ ] Server Plugin
- [ ] Extensions
- [ ] Others
Repository: pytorch
Issue: Precisely monitor the collective communication tasks
Labels: oncall: distributed,triaged,enhancement,oncall: profiler


<img width="1128" alt="elapsedTime" src="https://github.com/pytorch/pytorch/assets/54109317/5948680e-2735-4281-a4d2-04c36e15d020">
### 🚀 The feature, motivation and pitch

I have noticed that [https://github.com/pytorch/pytorch/pull/111072](https://github.com/pytorch/pytorch/pull/111072) has provided 'PG observability hooks' to obtain the time-consuming nature of collection communication. However, it seems that the CUDA event-based method for collecting communication time statistics is unable to capture the specific start and end times of the task. Additionally, if the task is not yet finished, users cannot obtain any information about it, including the start time and duration. Is it possible to use  ['cudaLaunchHostFunc'](https://docs.nvidia.com/cuda/cuda-runtime-api/group__CUDART__EXECUTION.html#group__CUDART__EXECUTION_1g05841eaa5f90f27124241baafb3e856f) to implement these new functionalities?

### Alternatives

_No response_

### Additional context

We aim to accurately monitor the collective communication operations during training.

cc @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @osalpekar @jiayisuse @H-Huang @kwen2501 @awgu @penguinwu @robieta @chaekit @aaronenyeshi @nbcsm @guotuofeng @guyang3532 @gaoteng-git @tiffzhaofb @dzhulgakov @davidberard98
Repository: pytorch
Issue: torch.jit.trace is not able to trace torch extension
Labels: oncall: jit

### 🚀 The feature, motivation and pitch

https://pytorch.org/tutorials/advanced/cpp_extension.html#integrating-a-c-cuda-operation-with-pytorch give a way to write torch extension, but those cpp_extension can not be traced into torchscript.

### Alternatives

_No response_

### Additional context

_No response_

cc @EikanWang @jgong5 @wenzhe-nrv @sanchitintel