sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: rust
Issue: Wrong error emitted with multiple traits in return position of trait function
Labels: A-diagnostics,T-compiler,C-bug,D-verbose,S-has-mcve

I tried this code:

```rust
pub struct Bar<T: Foo> {}

impl<T> Bar<T>
where
    T: Foo,
{
    pub fn dumb(&self) {}
}

pub trait Foo {
    fn qux(bar: &Bar<impl Foo>) -> impl Send + Sync;
}

pub struct Baz;

impl Foo for Baz {
    fn qux(bar: &Bar<impl Foo>) -> impl Send {
        bar.dumb();
    }
}

fn main() {}
```

I expected to see this happen: 
```Rust
error[E0392]: type parameter `T` is never used
 --> src/main.rs:1:16
  |
1 | pub struct Bar<T: Foo> {}
  |                ^ unused type parameter
  |
  = help: consider removing `T`, referring to it in a field, or using a marker such as `PhantomData`
```

Instead, this happened: 
```Rust
error[E0283]: type annotations needed
  --> src/main.rs:18:13
   |
18 |         bar.dumb();
   |             ^^^^ cannot infer type for type parameter `T`
   |
   = note: cannot satisfy `_: Foo`
   = help: the trait `Foo` is implemented for `Baz`
note: required by a bound in `Bar::<T>::dumb`
  --> src/main.rs:5:8
   |
5  |     T: Foo,
   |        ^^^ required by this bound in `Bar::<T>::dumb`
6  | {
7  |     pub fn dumb(&self) {}
   |            ---- required by a bound in this associated function

```
on nightly I get both errors but only the expected error (E0392) should appear. On stable I only get the latter (E0283).
E0283 is incorrect since the type of `bar` is annotated in the function signature.
If E0392 is fixed by adding PhantomData both errors disappear as expected.

### Meta
<!--
If you're using the stable version of the compiler, you should also check if the
bug also exists in the beta or nightly versions.
-->

`rustc --version --verbose` (stable):
```
rustc 1.76.0 (07dca489a 2024-02-04)
binary: rustc
commit-hash: 07dca489ac2d933c78d3c5158e3f43beefeb02ce
commit-date: 2024-02-04
host: x86_64-unknown-linux-gnu
release: 1.76.0
LLVM version: 17.0.6
```

`rustc --version --verbose` (nightly):
```
rustc 1.78.0-nightly (9c3ad802d 2024-03-07)
binary: rustc
commit-hash: 9c3ad802d9b9633d60d3a74668eb1be819212d34
commit-date: 2024-03-07
host: x86_64-unknown-linux-gnu
release: 1.78.0-nightly
LLVM version: 18.1.0
```
Repository: deno
Issue: Deno compile - Precompile Wasm
Labels: perf,suggestion,compile

If we can compile deno code to WASM, then we can transpile that WASM to native Assembly and eventually ELF..

This can optimize deno compiled executables' size and runtime performance...
Repository: PowerToys
Issue: [Keyboard Manager] Add support for cycling through apps if same keyboard shortcut is used for multiple apps
Labels: Idea-Enhancement,Product-Keyboard Shortcut Manager,Needs-Triage

### Description of the new feature / enhancement

Currently one  cannot assign the same shortcut key to two applications
![image](https://github.com/microsoft/PowerToys/assets/13977449/1088cac2-fa9b-45fc-89fa-0be7129c6448)

and one cannot assign the same shortcut key for the same application with different arguments to open multiple instances.
![image](https://github.com/microsoft/PowerToys/assets/13977449/4a675d43-19b1-413e-9c9b-3921f48552cb)

A very useful feature would be if you can assign different applications the exact same keyboard shortcut and when pressing it, it should cycle through the applications assigned to that keyboard shortcut i.e. Powertoys should remember which remap application was opened last  with that keyboard shortcuts and then the next time the keyboard shortcut  is pressed, should open the next application that has the  same keyboard shortcut assigned in the Remap Shortcut list.


### Scenario when this would be used?

Alt+tab is useful if switching between two full screen windows as it makes use of the last active window in order, allowing you to switch between multiple apps. However if you have a split screen  where you want to cycle through apps in the one screen  section and cycle through a different set of apps in the other screen section, then Alt+tab and Win+tab isn't so useful. Same applies if you have multiple monitors.

The feature I would find useful is to cycle through a predefined set of apps for a specific screen section or monitor with the same keyboard shortcut.

For example in the left side of my screen I cycle through my browser, Excel and MSTeams applications with assigned keyboard Alt+1. Then on the right side of my screen I cycle through my multiple note apps with Alt+2. This prevents me from losing focus when window switching with Alt+Tab or Windows by having to find the correct window

### Supporting information

There have previously been issues logged for Alt+Tab to cycle through a predefined set of applications instead of all windows applications. If one could setup a keyboard combination for the apps you wish to group e.g. Group 1 = Alt+1, Group 2=Alt+2 it will serve as a workaround to resolve those issues.

https://github.com/microsoft/PowerToys/issues/24737
https://github.com/microsoft/PowerToys/issues/13154
Repository: react-native
Issue: No haptic feedback when using RefreshControl with a tintColor prop
Labels: Issue: Author Provided Repro,Component: RefreshControl

### Description

On iOS, it's expected that the system gives small haptic feedback when using pull-to-refresh. This works by default when using `<RefreshControl>`. However, when passing a `tintColor` prop to `<RefreshControl>`, the haptics don't play.

I don't have another device to confirm, but I _feel_ like this may have just started recently, possibly with iOS 17.4.

### Steps to reproduce

Use a RefreshControl with a tintColor prop on a real iOS device and observe that the haptic feedback does not play. Comment out the tintColor prop, reload the app (fast refresh doesn't have an effect), and then observe the haptic feedback works.

### React Native Version

0.73.5

### Affected Platforms

Runtime - iOS

Device I'm testing on is an iPhone 15 Pro on iOS 17.4

### Output of `npx react-native info`

```text
System:
  OS: macOS 14.3.1
  CPU: (16) x64 Intel(R) Core(TM) i9-9980HK CPU @ 2.40GHz
  Memory: 2.40 GB / 16.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 19.6.0
    path: ~/.asdf/installs/nodejs/19.6.0/bin/node
  Yarn:
    version: 1.22.19
    path: /usr/local/bin/yarn
  npm:
    version: 9.4.0
    path: ~/.asdf/plugins/nodejs/shims/npm
  Watchman: Not Found
Managers:
  CocoaPods:
    version: 1.14.3
    path: /Users/marcshilling/.asdf/shims/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - visionOS 1.0
      - watchOS 10.2
  Android SDK: Not Found
IDEs:
  Android Studio: 2023.1 AI-231.9392.1.2311.11330709
  Xcode:
    version: 15.2/15C500b
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 11.0.18
    path: /usr/bin/javac
  Ruby:
    version: 2.7.6
    path: /Users/marcshilling/.asdf/shims/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.73.5
    wanted: 0.73.5
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false
```


### Stacktrace or Logs

```text
N/A
```


### Reproducer

https://github.com/marcshilling/react-native-refresh-control-haptics-bug

### Screenshots and Videos

_No response_
Repository: three.js
Issue: Allow controlling traverse flow
Labels: Suggestion

### Description

Right now there's no way to prevent `traverse` from traversing the whole tree (save for throwing an exception). However, often it would be useful to prevent processing the child nodes, or even rest of the hierarchy.

### Solution

Make traverse and the rest of the family respect special return values. The most important (in my opinion) would be the ability to ignore the children. For example:
```
// Somewhere in Three.js code. Could be replaced with an enum-style definition 
const IGNORE_CHILDREN = new Symbol('ignore_children');
const HALT_TRAVERSE = new Symbol('halt_traverse');

// I didn't check this for mistakes, but the idea should become clear
	traverse( callback ) {

		const controlFlowResult = callback( this );

                if (controlFlowResult === IGNORE_CHILDREN || controlFlowResult === HALT_TRAVERSE ) {

                      return controlFlowResult;

                } 

		const children = this.children;

		for ( let i = 0, l = children.length; i < l; i ++ ) {

			const childFlowResult = children[ i ].traverse( callback );

                        if (childFlowResult === HALT_TRAVERSE) {

                             return HALT_TRAVERSE;

                        }

		}

	}
```

Examples how a user might use it:

Find a specific node with some property. Then stop. 
```
// 
let oneSpecimenB;
myModel.traverse(node => {
           if (node.userData.isB) {
               oneSpecimentB = node;
                return HALT_TRAVERSE;
           }
});
```

Run code for all nodes except for all the descendants (and the node itself)  of the nodes we want to ignore
```
myModel.traverse(node => {
           if (node.userData.ignoreChildren) {
                // Optionally run operations here before returning
                return IGNORE_CHILDREN;
           }

          // Do operations for others
});
```


### Alternatives

For users, the alternatives are

-  Set an outside variable to a value from the callback to not do anything. For example
```
let ingoreChildren = false;

myModel.traverse(node => {
           if (node.userData.ignoreChildren) {
               ignoreChildren = true;
                return;
           }

          // Do operations for others
});
```

This will fail after all the descendants have been traversed and the execution returns to the siblings / parent.

- Use visibility in conjunction with `traverseVisible`. The user  would run the risk of messing up visibility and it's not convenient anyway, because it would force the user to traverse the hierarchy first to change it.

- `getObjectByProperty` only works for the direct properties of nodes, but not for `userData` for example. And since it's based on a simple equality comparison, it's generally less useful than a predicate based search.

- Implement your own `traverse`. The user would just duplicate the functionality of `traverse`

### Additional context

_No response_
Repository: godot
Issue: Code editor scrolls to show the caret when it's window loses focus
Labels: discussion,topic:editor,usability

### Tested versions

- occurs in v4.3.dev4.official [df78c0636]
- *does not* occur in v4.2.1.stable.official [b09f793f5]

### System information

linux

### Issue description

- when
  1. the focus leaves the window the code editor is in (either the full editor or the code editor in its own window), and
  2. the code editor is focused
- then the code editor scrolls to make the edition caret visible if it's outside the view



https://github.com/godotengine/godot/assets/71937306/bdde6001-e372-437e-8293-93a36c0a293c




### Steps to reproduce

1. get a long gdscript file
2. put the caret in a line and scroll it outside of the screen
3. give the focus to another os window
4. the code editor should scroll on its own

### Minimal reproduction project (MRP)

any
Repository: kubernetes
Issue: Support for Topology Aware Routing in winkernel proxier
Labels: area/kube-proxy,kind/feature,sig/windows,needs-triage

### What would you like to be added?

Topology Aware Routing was introduced in https://github.com/kubernetes/kubernetes/pull/99522 and at the time of introduction it was introduced for `iptable` and `ipvs` proxiers. The feature is in beta since 1.23 but support is missing for Windows worker nodes.

### Why is this needed?

Windows worker nodes are supported by various distributions and cloud providers. A lot of them already automatically set `topology.kubernetes.io/zone` based on the service provider's availability zone information for these nodes. if you have a multi-zone environment with Windows worker nodes, it is not possible to benefit from Topology Aware Routing, therefore all the benefits highlighted in https://github.com/kubernetes/enhancements/tree/master/keps/sig-network/2433-topology-aware-hints are missed. If the cluster has active Linux workloads in these zones, interacting with these services through Windows workload may actually turn out to be costly - both performance & bandwidth wise.

It looks like there may not be a technical blockers (HNS, for eg) to have this. Today all the endpoint registration logic in winkernel proxier resides in `syncProxyRules` where iTP, eTP policies are evaluated - this diverges from iptables, ipvs proxiers that have evolved to use `CategorizeEndpoints` from `topology.go` for this purpose, which also handles selection of endpoints matching node's topology zone. 

If there are technical reasons for this not being supported, the lack of support should be documented and tracked.
Repository: ollama
Issue: Does ollama support accelerated running on npu?
Labels: feature request

> The Intel Ultra 5 NPU is a hardware gas pedal dedicated to AI computing that boosts the performance and efficiency of AI applications.

Will ollama support using npu for acceleration? Or does it only call the cpu?
Repository: pytorch
Issue: Make unsupported error messages better in Dynamo
Labels: high priority,module: logging,triaged,months,oncall: pt2,module: dynamo

@iseeyuan and I were debugging exporting llava. We ran into the following error message:
```
  File "/Users/myuan/miniconda3/envs/executorch/lib/python3.10/site-packages/torch/_dynamo/exc.py", line 172, in unimplemented
    raise Unsupported(msg)
torch._dynamo.exc.Unsupported: call_function BuiltinVariable(iter) [UserDefinedObjectVariable(Image)] {}
from user code:
   File "/Users/myuan/src/executorch/examples/models/llava/model.py", line 29, in forward
    images_tensor = process_images(
  File "/Users/myuan/src/LLaVA/llava/mm_utils.py", line 170, in process_images
    for image in images:
```
What this actually meant was: "You tried to call iter(images), but images is of type PIL.Image so Dynamo didn't know how to handle it, thus leading to a graph break". This was an error in the model code (`images` probably should have been a `List[Image]`instead of an `Image`), but the actual error "torch._dynamo.exc.Unsupported: call_function BuiltinVariable(iter) [UserDefinedObjectVariable(Image)] {}" is very obtuse.

cc @ezyang @gchanan @kadeng @msaroufim @chauhang @penguinwu @voznesenskym @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @chenyang78 @amjames @rec @bdhirsh @anijain2305
Repository: angular
Issue: Support for setting Input Signals programmatically given a directive instance
Labels: area: core,core: inputs / outputs,cross-cutting: signals

### Which @angular/* package(s) are relevant/related to the feature request?

core

### Description

With Input signals they are read-only which forces any consumer of the component to have to set the Inputs through the template.   For some situations with a parent/child relationship (such as projected content) the parent needs to be able to programmatically set values of the Inputs as they don't have access to set the template bindings.

As a component library author there are often cases where we allow something like an `icon` component to be projected.  The `icon` component has an input that allows several sizes.  When the `icon` is used in a specific component in a projection slot that component can query through a `ContentChildren` query.  That component can further set the value of the `@Input` to "enforce" a given size.  This "parent" component can't set the value through the template bindings as it doesn't have template access to projected content, just component class instances.

Here's a non-working example of where a `parent` component allows projected `icon` components.  It is trying to query for `icon` content and then set the `size` input.  We've been able to accomplish in the past with `@Inputs` and the getter/setter pattern.  We'd love to be able to move to Input signals but this blocks us.

https://stackblitz.com/edit/stackblitz-starters-g5jzxx?file=src%2Fcomponents%2Fparent%2Fparent.component.ts



### Proposed solution

Some way for consumers to set the value of an Input programmatically.  I realize they are read-only but this breaks the notion of a component not knowing how it is consumed and used.  Consumers are allowed to set Inputs through the template OR through the component class instance with a reference.  

### Alternatives considered

We can continue to use a getter/setter syntax to fake Input signals.  Something like this:

```
@Input() get size(): number {
  return this._size;
}

set size(value: number) {
   this._size.set(value);
}
private _size = signal();

```

It would be nicer and easier to maintain if we can utilize Input signals programmatically. 


Model Inputs are also a potential option

I don't feel that the `Model` inputs are the solution here as that would mean all of our Inputs would have to be Model inputs because as component library authors we don't know how a consumer will access our Inputs (via template or programmatically).  Whereas Model inputs more feels like it's trying to solve ControlValueAccessor stuff and not necessarily meant for this.
Repository: kubernetes
Issue: Add integration tests for CEL logic around email verified
Labels: sig/auth,triage/accepted

Could you add [integrations tests](https://github.com/kubernetes/kubernetes/blob/master/test/integration/apiserver/oidc/oidc_test.go#L561) using `claims.email_verified` in
1. username.expression
2. extra[*].valueExpression
3. claimValidationRules[*].expression

_Originally posted by @aramase in https://github.com/kubernetes/kubernetes/pull/123737#discussion_r1516908029_
Repository: flutter
Issue: Add golden test review process to release cycle
Labels: c: proposal,P1,team-engine,triaged-engine

Currently, when we create release builds of the engine, the golden image test results are not checked (automated or manually).  Some of them are not even executed, only certain invocations of "run_tests.py" are used.

We'd like to have confidence that our cherry-picks are not causing some failure, so ideally the golden tests would be executed and checked.

The golden image results can not be naively uploaded to skia gold because skia gold has a consistent timeline that checks against one branch.  Comparing a golden from a release branch to the main branch would confuse skia gold and lead to situations where there is no correct course forward.

A potential solution would be to create new tests related to the release branch.  For example `aiks_foo_test.png` could become `aiks_foo_test.png.v12_34_5` for release `12.32.5`.  The tests would first be uploaded with zero cherry-picks and blanket approved.  Those images will then serve as the basis from which cherry-pick golden results will be compared.

cc @zanderso @matanlurey
Repository: ollama
Issue: Search on ollama.com/library is missing lots of models
Labels: bug,ollama.com

Current behavior:

Using @ehartford as an example since he's a prolific ollama model contributor:

https://ollama.com/search?q=ehartford&p=1

Shows his models:

<img width="1276" alt="Screenshot 2024-03-08 at 9 45 33 AM" src="https://github.com/ollama/ollama/assets/2172753/08b9dc80-5d94-4b86-82dd-37c0dddac326">


https://ollama.com/library?q=ehartford

Does not:

<img width="1257" alt="Screenshot 2024-03-08 at 9 45 52 AM" src="https://github.com/ollama/ollama/assets/2172753/6f1c248b-a93c-468a-a78c-24811a857731">


Expected behavior is that they would be the same.
Repository: pytorch
Issue: DISABLED test_bitwise_cuda_cuda_wrapper (__main__.TestCudaWrapper)
Labels: module: rocm,triaged,skipped

Platforms: rocm

This test was disabled because it is failing on main branch ([recent examples](https://torch-ci.com/failure?failureCaptures=%5B%22inductor%2Ftest_cuda_cpp_wrapper.py%3A%3ATestCudaWrapper%3A%3Atest_bitwise_cuda_cuda_wrapper%22%5D)).

cc @sunway513 @jithunnair-amd @pruthvistony @ROCmSupport @dllehr-amd @jataylo @hongxiayang
Repository: pytorch
Issue: Avoid shard 0 download on all ranks
Labels: oncall: distributed,triaged

### 🚀 The feature, motivation and pitch

When I save on N ranks and load on N ranks, I observe all ranks require the 0th shard. This seems to be because of optimizer.param_groups. Is this expected? Could elastic checkpointing avoid this?
```
2024-01-28 01:33:22,143: rank40[125][MainThread]: DEBUG: composer.utils.checkpoint: Loading plan item: ReadItem(type=<LoadItemType.BYTE_IO: 2>, dest_index=MetadataIndex(fqn='state.optimizers.DecoupledLionW.param_groups', offset=None, index=None), dest_offsets=torch.Size([0]), storage_index=MetadataIndex(fqn='state.optimizers.DecoupledLionW.param_groups', offset=None, index=None), storage_offsets=torch.Size([0]), lengths=torch.Size([0]))
```

@fegin or @wz337 maybe?

### Alternatives

_No response_

### Additional context

_No response_

cc @XilunWu @H-Huang @awgu @kwen2501 @wanchaol @fegin @fduwjj @wz337 @wconstab @d4l3k @c-p-i-o @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @osalpekar @jiayisuse @penguinwu @tianyu-l @yf225 @chauhang
Repository: flutter
Issue: Create an auto deploy workflow for discord tree-status-bot
Labels: team-infra,P3,triaged-infra

### Type of Request

infra task

### Infrastructure Environment

GCP, tree-status-bot

### What is happening?

The current code for discord tree status bot is not auto deployed. We need to create a cloud job to auto deploy changes from the source location https://github.com/flutter/cocoon/tree/main/dev/tree-status-bot to the cloud function hosting the bot.

### Steps to reproduce

Step 1:
Step 2:
..
Step n:


### Expected results

I expect to see X when Y is finished.
Repository: vscode
Issue: misalignment on border of markdown cell
Labels: bug,notebook-layout


## Environment data

-   VS Code version: 1.87.0, 1.87.1

## Expected behaviour

XXX

## Actual behaviour

![image](https://github.com/microsoft/vscode-jupyter/assets/46436648/7a010591-8e10-4a67-81e8-30b727620c15)
Repository: svelte
Issue: Auto propagate classes
Labels: css

### Describe the problem

An EXTREMELY common pattern when developing web apps is to give a component a class property and then apply that class to the top-level DOM element of that component. It allows the consumer of the component to customise it, resize it, place it in a different location on the screen etc. This is a much better solution than just wrapping it in a div and styling that because it allows much greater flexibility.

But you often also want to apply your own classes to the same top-level element within the component. This leads to having to write the same ugly boilerplate code in almost all of my components:

```svelte
<script lang="ts">
	let clazz: string = '';
	export { clazz as class };
</script>

<div class={`${clazz} foo bar ...`}>
    ..rest of my component
</div>
```

Sure, it's not that much. But, again, I am doing this on almost every component of my app.

### Describe the proposed solution

Make all components have a class prop by default which is automatically merged with the top level element of the component.

So something like this

```svelte
<!-- App.svelte -->
<MyComponent class="foo" />
```

```svelte
<!-- MyComponent.svelte -->
<div class="bar bazz"></div>
```

Would compile to something like this

```html
<div class="foo bar bazz"></div>
```

This simple feature would make svelte feel much more elegant and possibly save developers a bit of time too.

### Importance

would make my life easier