sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: material-ui
Issue: [Tooltip] Type requires children as property
Labels: typescript,component: tooltip

### Duplicates

- [X] I have searched the existing issues

### Latest version

- [X] I have tested the latest version

### Current behavior 😯

with React.createElement as h, I write

`h(Tooltip, { title:"HTTPS" }, h(Lock) )`

and get an error from typescript that 'children' property is missing.
Then I can workaround it as

`h(Tooltip, { title:"HTTPS", children: h(Lock) } )`


### Expected behavior 🤔

I should be allowed to use the first (preferred) form

### Steps to reproduce 🕹

```
import { createElement as h } from "react"
import { Lock } from '@mui/icons-material'

//...

h(Tooltip, { title:"HTTPS" }, h(Lock) )

```

### Context 🔦

_No response_

### Your environment 🌎

<details>
  <summary>`npx @mui/envinfo`</summary>

```

  System:                                                   
    OS: Windows 10 10.0.19042                               
  Binaries:                                                 
    Node: 16.13.1 - C:\Program Files\nodejs\node.EXE        
    Yarn: 1.22.15 - C:\Program Files (x86)\Yarn\bin\yarn.CMD
    npm: 8.2.0 - C:\Program Files\nodejs\npm.CMD            
  Browsers:                                                 
    Chrome: Not Found                                       
    Edge: Spartan (44.19041.423.0), Chromium (97.0.1072.76) 
  npmPackages:                                              
    @emotion/react: ^11.7.1 => 11.7.1                       
    @emotion/styled: ^11.6.0 => 11.6.0
    @mui/base:  5.0.0-alpha.65
    @mui/icons-material: ^5.3.0 => 5.3.0
    @mui/lab: ^5.0.0-alpha.66 => 5.0.0-alpha.66
    @mui/material: ^5.3.0 => 5.3.0
    @mui/private-theming:  5.3.0
    @mui/styled-engine:  5.3.0
    @mui/styles: ^5.3.0 => 5.3.0
    @mui/system:  5.3.0
    @mui/types:  7.1.0
    @mui/utils:  5.3.0
    @mui/x-data-grid: ^5.3.0 => 5.3.0
    @types/react: ^17.0.38 => 17.0.38
    react: ^17.0.2 => 17.0.2
    react-dom: ^17.0.2 => 17.0.2
    typescript: ^4.5.5 => 4.5.5
```
</details>
Repository: vscode
Issue: Dark Editor Mouse Pointer - Again
Labels: under-discussion,editor-rendering

<!-- ⚠️⚠️ Do Not Delete This! bug_report_template ⚠️⚠️ -->
<!-- Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct/ -->
<!-- 🕮 Read our guide about submitting issues: https://github.com/microsoft/vscode/wiki/Submitting-Bugs-and-Suggestions -->
<!-- 🔎 Search existing issues to avoid creating duplicates. -->
<!-- 🧪 Test using the latest Insiders build to see if your issue has already been fixed: https://code.visualstudio.com/insiders/ -->
<!-- 💡 Instead of creating your report here, use 'Report Issue' from the 'Help' menu in VS Code to pre-fill useful information. -->
<!-- 🔧 Launch with `code --disable-extensions` to check. -->
Does this issue occur when all extensions are disabled?: Yes

<!-- 🪓 If you answered No above, use 'Help: Start Extension Bisect' from Command Palette to try to identify the cause. -->
<!-- 📣 Issues caused by an extension need to be reported directly to the extension publisher. The 'Help > Report Issue' dialog can assist with this. -->
- VS Code Version: 1.64.0
- OS Version: macOS Catalina 10.15.7 & Monterey 12.1 (Checked two different systems)

Steps to Reproduce:

1. Open VSCode
2. Select a dark theme
3. Observe text cursor

I opened an issue back in [May 2020](https://github.com/microsoft/vscode/issues/97550#issue-616187495) in regards to this "issue". This may also fit as a "feature request", as I am aware that this also may be more of a personal preference.

Before updating to 1.64.0 the mouse pointer appearance "inverted". (This screenshot is from X Code, but I can confirm that this was the style in VSCode before updating to 1.64.0)
<img width="137" alt="Screen Shot 2022-02-04 at 8 51 27 AM" src="https://user-images.githubusercontent.com/43049828/152555119-96fba9d0-ff3b-4f2f-94ee-2a1db66a1865.png">

After updating to 1.64.0 the cursor appears like with a black center and a white outline making it "thicker".
<img width="96" alt="Screen Shot 2022-02-04 at 8 48 57 AM" src="https://user-images.githubusercontent.com/43049828/152555129-1decca36-311e-4dc2-a699-5e19268e8cc6.png">

As mentioned above, this might be more of a personal preference. I like to use the text cursor for assistance when scanning code (as I imagine a lot of fellow coders do) and I find that the "light mode" cursor blocks text. But I can also understand the preference for the "thicker"/light mode cursor on a dark surface.

Would this be a possible setting to add to the settings? I believe it would also help avoid future "issues" regarding the mouse pointer.

Thanks for all your help and time.
Repository: PowerToys
Issue: Always On Top - monitor the flag and adjust accordingly
Labels: Idea-Enhancement,Product-Always On Top

### Description of the new feature / enhancement

I have no idea if this is even possible, ... if Windows actually provides any kind of hook/callback for being notified. But I was wondering/thinking if maybe AOT can be made aware of when other apps set/change the flag for a window, and add/remove the colored border / play the sound, as appropriate. 

I occasionally find windows being set to AOT accidentally by my pre-existing utility, and I don't realize it until I try to get to something behind the window. (Which I use to be able to set aot via mouse.)

If AOT could trap those changes and provide the same notifications as when it does it, that would be awesome! 
Assuming there is any hook/callback to monitor the flag. 

### Scenario when this would be used?

When toggling always-on-top via some other program.

### Supporting information

_No response_
Repository: pytorch
Issue: [JIT] Channels last optimization pass
Labels: oncall: jit

### 🚀 The feature, motivation and pitch

Channels last sometimes improves performance, on some ops, for some input shapes & dtypes - but not always.

We could add a pass that converts some or all of the tensors to channels last. Possibly a benchmarking pass could be added to determine whether channels_last is helpful for a given scenario, or even to determine which parts of the graph benefit from it. Probably this would be added into optimize_for_inference, not as a runtime pass.

* Unary - assume it doesn't matter
* Convolutions - check to see which behavior is better
* Conversions - measure the cost of conversion & whether or not the conversion happens in a place where it can be fused
* Other ops - e.g. batchnorm - need to check if it preserves channels last, & performance

Possibly added into optimize_for_inference (would need to add ability to add sample inputs to do the profiling on)

Want it to happen in AOTAutograd as well

cc @Chillee 

### Alternatives

_No response_

### Additional context

_No response_
Repository: flutter
Issue: Allow to center InteractiveViewer child widget
Labels: c: new feature,framework,c: proposal,P2,team-framework,triaged-framework

This should a parameter instead:

https://github.com/flutter/flutter/blob/bc6fd9e17daa803742dc78ef8c3001fde99d37b3/packages/flutter/lib/src/widgets/interactive_viewer.dart#L1097
Repository: flutter
Issue: `SwapBuffers` can be called multiple times in the same frame interval
Labels: engine,customer: money (g3),P3,team-engine,triaged-engine

Current behaviro is that, The rasterizer will queue a task to handle the next work item if there are more items in the pipeline: https://github.com/flutter/engine/blob/main/shell/common/rasterizer.cc#L213. The vsync timing drives requests for production of new frames that are ultimately fed into the rasterizer pipeline. There isn't any coordination between the rasterizer's frame consumption and the vsync timing.

Is there any good reason to allow that? If we are successful in rendering the second frame we spent useless cycles rendering the first frame, if not we are potentially eating away at the next frame's raster time. cc: @chinmaygarde 

Here is an example of one such instance:

![image](https://user-images.githubusercontent.com/1006071/152583511-85637e2d-bc1d-4451-b81d-c62b567fcf15.png)
Repository: pytorch
Issue: Add ZeroTensor fastpath for torch.addmm
Labels: triaged,ZeroTensor

see https://github.com/pytorch/pytorch/issues/69687

example PR: https://github.com/pytorch/pytorch/pull/71129

Benchmark the forward mode AD perf for your change!
Repository: pytorch
Issue: torch.distributed.new_group() should consolidate the type of rank list
Labels: oncall: distributed,module: c10d

### 🐛 Describe the bug

If we create a new PG using `torch.distributed.new_group(torch.tensor([0, 1, 2, 3]), ...)`, we will end up with ranks in tensor, not int.

For example, if we print out `_pg_group_ranks[group]` in line 277, it's `{tensor(0): 0, tensor(1): 1, tensor(2): 2, tensor(3): 3}`.

```
# From torch/distributed/distributed_c10d.py

 265 def _get_group_rank(group: ProcessGroup, rank):
 266     """
 267     Helper that gets a given group's local rank in the group from a given global
 268     rank.
 269     """
 270     if group is GroupMember.WORLD:
 271         raise RuntimeError(
 272             "group.WORLD does not have local rank to global " "rank mapping"
 273         )
 274     if group not in _pg_group_ranks:
 275         raise RuntimeError("The given group does not exist")
 276     try:
 277         group_rank = _pg_group_ranks[group][rank]
 278     except KeyError:
 279         raise RuntimeError(
 280             f"The global rank {rank} is not part of the group {group}"
 281         ) from None
 282     return group_rank
```

If we later use `tensor(0)` as rank in `broadcast()`, or any other combination that doesn't have native int on both side, we will get a very confusing exception `RuntimeError: The global rank 0 is not part of the group`.

### Versions

Python 3.9.10 + torch 1.10.2 on macOS

cc @pietern @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @osalpekar @jiayisuse @SciPioneer @H-Huang
Repository: pytorch
Issue: Add ZeroTensor fastpath for torch.addcmul
Labels: triaged,ZeroTensor

see https://github.com/pytorch/pytorch/issues/69687

example PR: https://github.com/pytorch/pytorch/pull/71129

Benchmark the forward mode AD perf for your change!
Repository: godot
Issue: Add a warning when saving an inherited scene, but then the inheritance is broken (especially due to changing node parent type)
Labels: enhancement,topic:editor,usability

### Godot version

v3.4.2.stable.official [45eaa2daf]

### System information

Windows 11, GLES3, RTX3060TI [511.23]

### Issue description

I'm aware that inheritance is broken when you change the type of the parent node on the inherited scene (also informed with the faded text on the scene tree is gone). You can only change the node parent type on import tab. However for me it is not that clear on the first time I'm trying Godot.

The issue that I'm facing is that when I have imported scene from gtlf, and inherit it to add a node (CollisionShape for example), I'm also changing the parent node to be a RigidBody, fully unaware at that time it broke the inheritance and when I save the scene, it basically write the vertex, texture, others data to the scene. For me this is unintended. So I think having a warning when saving broken inherited scene is good so others wont make a mistake that I made.

### Steps to reproduce

1. Import gltf asset using default config
2. Create inherited scene
3. Change scene root type on the inherited scene
4. Save scene

### Minimal reproduction project

_No response_
Repository: flutter
Issue: Timeouts are ignored in Android Studio tests
Labels: a: tests,framework,dependency: dart,has reproducible steps,P2,d: intellij,found in release: 2.10,team-framework,triaged-framework

## Steps to Reproduce

1. Test code:
```
@Timeout(const Duration(seconds: 2))
import 'package:flutter_test/flutter_test.dart';

void main() {
  test('timesout', () async {
    await Future.delayed(const Duration(seconds: 10), () {});
  }, timeout: const Timeout(Duration(seconds: 2)));
}
```
2. Test using Android Studio (Run 'tests in test.dart')
3. Observe that the test successfully finishes after 10s, ignoring the 2 second timeout.

Couple things to note: 

* This does not occur when running flutter tests on an entire directory. 
* A single file test has the `--start-paused` argument added to the command, while the directory tests do not.

## Version info

```
[✓] Flutter (Channel stable, 2.10.0, on macOS 12.0 21A344 darwin-arm, locale en)
    • Flutter version 2.10.0 at /Users/davidliu/flutter
    • Upstream repository git@github.com:flutter/flutter.git
    • Framework revision 5f105a6ca7 (2 days ago), 2022-02-01 14:15:42 -0800
    • Engine revision 776efd2034
    • Dart version 2.16.0
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at /Users/davidliu/Library/Android/sdk
    • Platform android-32, build-tools 32.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.2.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7249189)

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 97.0.4692.99

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!
```
Repository: PowerToys
Issue: Integrate KBM with Settings
Labels: Idea-Enhancement,Product-Keyboard Shortcut Manager

### Description of the new feature / enhancement

Changing the layout to have a "+" button on the side of "keys" and "shortcuts" labels to add the shortcut/key
showing the delete button beside the shortcut/key so we are able to get rid of the outer window as it make bad ease of access

### Scenario when this would be used?

no need to answer.

### Supporting information

_No response_
Repository: flutter
Issue: [cocoon] PushMessage does not decode data from base64
Labels: team-infra,P2,triaged-infra

This has lead to several places in the code where you need to know that your data needs to be base64 decoded first. As we want to make this as easy an API to use, we need to move this logic into the model, and update all existing subscriptions.
Repository: flutter
Issue: Flutter should expose text editing shortcut logic for reuse
Labels: team,framework,f: material design,c: proposal,team-design,triaged-design

Flutter has a ton of logic for handling keyboard shortcuts in the framework, mapping them in [default_text_editing_shortcuts.dart](https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/widgets/default_text_editing_shortcuts.dart) and actually handling them privately [in editable_text.dart](https://github.com/flutter/flutter/blob/4b82d47b012380ed952ab29e66033f62a3bcda84/packages/flutter/lib/src/widgets/editable_text.dart#L3105).  We should make this logic reusable by developers writing their own text editors.

## Why?

Previously, this logic was exposed in a class called `TextEditingActionTarget`, but that was removed in https://github.com/flutter/flutter/pull/90684.  It was brought up in the [comments](https://github.com/flutter/flutter/pull/90684#issuecomment-1029312785) after merge that this change had broken two major text editing projects.

Besides being impractical for developers to reimplement all of this logic on their own, doing so would quickly get out of sync with Flutter's behavior and cause strange discrepancies in text editing behavior between simple TextFields and the custom text editors.

## How?

The shortcuts and their behavior is already overridable by using Shortcuts and Actions widgets, but the problem is applying them to something other than an EditableText.  This was possible by mixing in `TextEditingActionTarget`, but it was brought up in the comments that [a composable solution may be preferred](https://github.com/flutter/flutter/pull/90684#issuecomment-1029697048) instead.

We should flesh out this idea and then expose it publicly.
Repository: flutter
Issue: [pigeon] Add correctly named convenience constructors for ObjC classes
Labels: package,team-ecosystem,p: pigeon,P2,triaged-ecosystem

(See https://github.com/flutter/plugins/pull/4726#discussion_r798193433 for background)

Currently pigeon makes data classes in ObjC with convenience constructors starting with `make`. E.g.:
```
@interface FLTTextureMessage : NSObject
+ (instancetype)makeWithTextureId:(nullable NSNumber *)textureId;
```

The ObjC naming convention for convenience constructors is to use the class name, since that's what's being returned. So for the above, the correct name would be `textureMessageWithTextureId:`. While we can't remove the `make...` versions without a breaking change, we can:
- Start generating the correct version in addition.
- Leave the `make` version as a forwarding method whole implementation is just to call the new method.
- Mark the `make` version as deprecated.
Repository: pytorch
Issue: Add ZeroTensor fastpath for torch.addbmm
Labels: triaged,ZeroTensor

see https://github.com/pytorch/pytorch/issues/69687

example PR: https://github.com/pytorch/pytorch/pull/71129

Benchmark the forward mode AD perf for your change!
Repository: pytorch
Issue: Add ZeroTensor fastpath for torch.addmv
Labels: triaged,ZeroTensor

see https://github.com/pytorch/pytorch/issues/69687

example PR: https://github.com/pytorch/pytorch/pull/71129

Benchmark the forward mode AD perf for your change!
Repository: pytorch
Issue: Add ZeroTensor fastpath for torch.baddbmm
Labels: triaged,ZeroTensor

see https://github.com/pytorch/pytorch/issues/69687

example PR: https://github.com/pytorch/pytorch/pull/71129

Benchmark the forward mode AD perf for your change!
Repository: pytorch
Issue: Add ZeroTensor fastpath for torch.bmm
Labels: triaged,ZeroTensor

see https://github.com/pytorch/pytorch/issues/69687

example PR: https://github.com/pytorch/pytorch/pull/71129

Benchmark the forward mode AD perf for your change!
Repository: go
Issue: x/pkgsite: inconsistent text render output in screentest
Labels: NeedsInvestigation,pkgsite

The testcase for "test empty jump to modal" in tests/screentest/testcases.txt occasionally fails because the font rendering on the Close button is inconsistent. [Here](https://github.com/puppeteer/puppeteer/issues/661) is a similar issue with puppeteer, it seems to be limited to linux platforms in both cases. There might be some combination of chrome flags we can add to the headless-shell command to disable font smoothing or antialiasing.
Repository: pytorch
Issue: Custom ProcessGroup Destructor Not Correctly Called in PT 1.10
Labels: oncall: distributed,module: c10d,module: ddp

### 🐛 Describe the bug

Hi PyTorch community,

I was trying to run the follow code with PT 1.10 + a custom process group backend. (I have substituted our own backend with intel-ccl(https://github.com/intel/torch-ccl) for reproducibility. We use the same binding logic and the issue is reproducible with both backends)

``` python
import torch
from torch.nn.parallel import DistributedDataParallel as DDP
import torch.distributed as dist
import torch.nn as nn
import torch.nn.functional as F
import torch_ccl
dist.init_process_group(backend="ccl")
class ToyModel(nn.Module):
    def __init__(self):
        super(ToyModel, self).__init__()
        self.net1 = nn.Linear(10, 10, bias=False)
        self.net2 = nn.Linear(10, 10)
    def forward(self, x):
        return self.net2(self.net1(x))
model = ToyModel()
ddp = torch.nn.parallel.DistributedDataParallel(
    model,
    find_unused_parameters=True)

inp = torch.randn(1, 10)
out = ddp(inp)
```

When `find_unused_parameters=True`, the destructor of ProcessGroupCCL was not correctly called. When `find_unused_parameters=False` there was no issue. This is a non-issue with intel-ccl since their ~ProcessGroupCCL() is empty. However, in our backend, we need to release certain resources/threads in the desctrctor or otherwise the python process will hang on exit.

This issue also does not exist in PT 1.9. Seems like some object life cycle management issue with PyTorch

Would really appreciate any insights and help!

### Versions

PT 1.10 and 1.10.2 
I built both tags from source 
intel-ccl is also built from source using this branch https://github.com/intel/torch-ccl/tree/ccl_torch1.10

cc @pietern @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @osalpekar @jiayisuse @SciPioneer @H-Huang
Repository: flutter
Issue: Add test harness for iOS to run engine benchmarks on device
Labels: engine,P3,team-engine,triaged-engine

In order to run benchmarks on device we need a signed app. There's currently one for the unit test framework, so we can do something similar for benchmarks as well.

The idea here is to vend a dylib from the flutter engine build that contains the benchmark tests that need to be run, and link to that from the iOS test app and call into the dylib's entrypoint.