sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: x/crypto/ssh: expose negotiated algorithms
Labels: Proposal,Proposal-Accepted,Proposal-Crypto

Folks running production servers are under opposing pressures, to both

1. stop supporting obsolete or vulnerable algorithms and/or key sizes.
2. keep serving customers and migrate them away from obsolete algorithms

they can only make informed decisions on what to do (reach out to customers, switch off algorithms etc.) if they have data which algorithms are actually in use.

For the TLS library,  I think this can be achieved by inspecting https://pkg.go.dev/crypto/tls#ConnectionState.

The SSH library doesn't expose this information, more in particular:

* Cipher, MAC, Compression in read/write directions
* host key algorithm
* key exchange algorithm.

this information is captured in `type algorithms`
https://cs.opensource.google/go/x/crypto/+/master:ssh/common.go;l=185;drc=6fad3dfc18918c2ac9c112e46b32473bd2e5e2f9

the proposal is to export `type algorithms`, and add

  func Algorithms() Algorithms

to the `ssh.Conn` type. 

This is technically an incompatible API  change. However, the ssh.Conn (as well as ssh.Channel) are misdesigned, and should have been `struct`s instead. They are not meant to be implemented by other package, so I think it's OK to add this method to the interface.
Repository: flutter
Issue: Accessibility DOM for `Tooltip` should use `title=`, not `aria-label`
Labels: engine,a: accessibility,platform-web,P2,customer: samehere,team-web,triaged-web

<!-- Thank you for using Flutter!

     If you are looking for support, please check out our documentation
     or consider asking a question on Stack Overflow:
      * https://flutter.dev/
      * https://api.flutter.dev/
      * https://stackoverflow.com/questions/tagged/flutter?sort=frequent

     If you have found a bug or if our documentation doesn't have an answer
     to what you're looking for, then fill out the template below. Please read
     our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports
-->

## Steps to Reproduce
Run this on a flutter-web local server
```dart
class Foo extends StatelessWidget {
  const Foo();

  @override
  Widget build(BuildContext context) {
   return Semantics(
      button: true,
      container: true,
      child: Semantics(
        tooltip: 'tooltip',
        child: Text('text label'),
      ),
    );
  }
}

void main() async {
  runApp(const MaterialApp(home: Material(child: Foo())));
  RendererBinding.instance.setSemanticsEnabled(true);
}
```

The generated accessibility DOM is:
```html
<flt-semantics id="flt-semantic-node-4"
               aria-label="tooltip
text label"
               role="button" tabindex="0"
               style="..."></flt-semantics>
```

**Expected results:** The tooltip should be [exposed using the `title=` attribute](https://developer.mozilla.org/en-US/docs/Web/HTML/Global_attributes/title).

**Actual results:** The tooltip here is merged into the `aria-label` of the button with a newline.
Repository: pytorch
Issue: Linking error with Libtorch
Labels: module: cpp,triaged,module: mkl

### 🐛 Describe the bug

Hello I'm trying to create an example-app case to run with libtorch as in the https://pytorch.org/cppdocs/installing.html, however I have a linking error.

### Error logs

LINK : fatal error LNK1104: cannot open file 'mkl_intel_ilp64.lib'

### Minified repro

_No response_

### Versions

libtorch 1.13.1+cpu 

cc @jbschlosser @ezyang @soumith @msaroufim @wconstab @ngimel @bdhirsh
Repository: godot
Issue: Unhelpful error message when interpolating on a NaN transform
Labels: enhancement,topic:core,confirmed

### Godot version

4.0.rc1 (custom build with double_precision)

### System information

Windows 10

### Issue description

In my project I'm interpolating moving mesh positions to their physics-body's location with the `get_physics_interpolation_fraction()` and `Transform3D::interpolate_with`. At some point I got this error message twice (this is taken from the provided example):

```
E 0:00:02:0117   InterpolationTest.gd:27 @ _ready(): Basis must be normalized in order to be casted to a Quaternion. Use get_rotation_quaternion() or call orthonormalized() if the Basis contains linearly independent vectors.
  <C++ Fehler>   Condition "!is_rotation()" is true. Returning: Quaternion()
  <C++ Source>   core\math\basis.cpp:702 @ Basis::get_quaternion()
  <Stacktrace>   InterpolationTest.gd:27 @ _ready()
```

followed by:

```
E 0:00:02:0127   RendererSceneCull::instance_set_transform: Condition "!v.is_finite()" is true.
  <C++ Source>   servers\rendering\renderer_scene_cull.cpp:906 @ RendererSceneCull::instance_set_transform()
```

`orthonormalized()` did not help, but in the end it turned out that I divided by zero at some point in my angles calculation and rotated the basis by `NaN`. This did not cause any issue (correctly?) but crashed when I tried to interpolate with a transform that resulted from that basis.

I feel like rotating the basis by `NaN` should have already resulted in an error or - if it is allowed to `move_and_slide` with a `NaN` transform - that the error message of `interpolate_with` should be more helpful here.

### Steps to reproduce

Run this scene to reproduce the issue

```gdscript
extends CharacterBody3D


var _mesh_instance := MeshInstance3D.new()
var _this_is_zero := 0.0

func _init() -> void:
    _mesh_instance.top_level = true
    add_child(_mesh_instance)


func _ready() -> void:
    # Set the rotation of this CharacterBody3D
    var new_basis := Basis()
    new_basis = new_basis.rotated(Vector3.UP, deg_to_rad(1 / _this_is_zero))
    # The basis now has NaN for all its values.
    transform.basis = new_basis
        
    # Move forward
    var old_transform = global_transform
    velocity = Vector3.FORWARD
    move_and_slide()
    var new_transform = global_transform
    
    # Interpolate the mesh half way to the physical position
    var current_transform = global_transform
    _mesh_instance.transform = old_transform.interpolate_with(new_transform, 0.5)

```

### Minimal reproduction project

N/A
Repository: PowerToys
Issue: Remapped Keyboard Shortcut Causing a Problem to Other Keys: Existing Issue
Labels: Issue-Bug,Product-Keyboard Shortcut Manager,Priority-1

### Microsoft PowerToys version

0.67.1

### Installation method

PowerToys auto-update

### Running as admin

Yes

### Area(s) with issue?

Keyboard Manager

### Steps to reproduce

I have defined "alt+C" and "alt+V" as remaps for "Ctrl+C" and "Ctrl+V". When I'm in a directory and want to housekeep some files using the defined map shortcuts, etc, you know, to copy some file, paste it in a topmost folder, etc, and then want to delete a folder using the "delete" button of the keyword, then it may not work.

### ✔️ Expected Behavior

I expect that I can smoothly work with both remapped shortcuts and the default keys together and unintermittently.

### ❌ Actual Behavior

The "delete" key (or probably some other keys) is not working once comes after using a remapped shortcut usage.

### Other Software

_No response_
Repository: vscode
Issue: supprt delete recent item in welcom page
Labels: feature-request,workbench-history,workbench-welcome,getting-started

<img width="1024" alt="image" src="https://user-images.githubusercontent.com/41773861/218817767-647bc2be-f71e-49c1-9232-54171014083b.png">

```
Version: 1.75.1 (Universal)
Commit: 441438abd1ac652551dbe4d408dfcec8a499b8bf
Date: 2023-02-08T21:34:59.000Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: 10.2.154.23-electron.0
OS: Darwin x64 22.3.0
Sandboxed: No
```
Repository: material-ui
Issue: [Checkbox] Slows down the render of the page
Labels: performance,component: checkbox

### Steps to reproduce 🕹

I have a long list of people (1000)
Each of them has a checkbox status attach to every user
I did notice when I use HTML checkbox instead of MUI one, the page renders about 3 to 5 times faster

The issue only occurs when I'm dealing with a long list of players, it does not affect a short list of players,
at least from the visible standpoint

Repository with live reproduction:
https://github.com/alexander-vashchuk/react-slow-render

### Current behavior 😯

Checkbox takes a while to update the page

### Expected behavior 🤔

Checkbox works immediately

### Context 🔦

**Demo:**
![animation](https://user-images.githubusercontent.com/86713661/218817450-0e876a54-5af7-4b8c-a821-feaea1dbc08d.gif)

I would like any help on this one, I might be doing something stupid from the React optimization standpoint, so I'd appreciate any advice

### Your environment 🌎

```
  System:
    OS: Windows 10 10.0.19044
  Binaries:
    Node: 16.15.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.19 - ~\.yarn\bin\yarn.CMD
    npm: 8.5.5 - C:\Program Files\nodejs\npm.CMD
    browser: Google Chrome
  Browsers:
    Chrome: Not Found
    Edge: Spartan (44.19041.1266.0), Chromium (110.0.1587.41)
  npmPackages:
    @emotion/react: ^11.10.5 => 11.10.5
    @emotion/styled: ^11.10.5 => 11.10.5
    @mui/base:  5.0.0-alpha.108
    @mui/core-downloads-tracker:  5.10.16
    @mui/material: ^5.10.16 => 5.10.16
    @mui/private-theming:  5.10.16
    @mui/styled-engine:  5.10.16
    @mui/system:  5.10.16
    @mui/types:  7.2.2
    @mui/utils:  5.10.16
    @types/react: 18.0.25 => 18.0.25
    react: 18.2.0 => 18.2.0
    react-dom: 18.2.0 => 18.2.0
    typescript: 4.9.3 => 4.9.3
```
Repository: pytorch
Issue: Allow Dynamo backends to use Inductor as fallback instead of eager mode
Labels: feature,triaged,oncall: pt2,module: inductor

### 🚀 The feature, motivation and pitch

I'm working on Dynamo oneDNN backend (being integrated at the mid-layer, like nvfuser or AITemplate) and would like to run models with mixture of Dynamo oneDNN backend + Inductor CPU backend. 

Based on oneDNN graph's programming model, we prefer to pass the full graph to oneDNN backend first, which decides its supported subgraphs, and run the rest of the graph with Inductor CPU afterward. 

This prototype PR [#90356] from @SherlockNoMad allows inductor to accepts opaque callable. This aligns with our design, where oneDNN backend will fuse subgraphs to call_module nodes and set callable for each fused node. The fused graph will be passed to Inductor CPU for compile and execute.

We'd like to check the plan of this feature and if the PR would be merged.   



cc @ezyang @msaroufim @wconstab @bdhirsh @anijain2305 @zou3519 @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @jiayisunx @peterbell10 @ipiszy @yf225 @chenyang78 @kadeng @muchulee8 @aakhundov @ColinPeppler @soumith @ngimel @yanboliang @Xia-Weiwen @desertfire @mlazos
Repository: vscode
Issue: Visual feedback when horizontal scroll "hides" columns
Labels: feature-request,editor-core

<!-- ⚠️⚠️ Do Not Delete This! feature_request_template ⚠️⚠️ -->
<!-- Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct/ -->
<!-- Please search existing issues to avoid creating duplicates. -->

<!-- Describe the feature you'd like. -->

It would be useful a shaded line at left of the text, when - and only than! - horizontal scroll is applied.

There are cases, when content can be wrongly interpreted when horizontally scrolled, and 
already exist a similar thing as suggested: when lines skipped at top there is a thin, shaded line.
Personally I'd prefer a little stronger/wider (or customizable) shade, but that line would help a lot too.

Attached image shows the existing "widget", and where I'm missing the line.

![HorizontalScroll](https://user-images.githubusercontent.com/1922838/218825652-d10f0eaf-2e97-4af1-ba76-cd5521c77c16.png)

Little clarification: this line would appear on full height, not just on some lines as the previous image.
Repository: godot
Issue: Function parameters accept type inference with null default value
Labels: bug,topic:gdscript

### Godot version

3.5.2-rc2

### System information

Manjaro Linux (6.0.2-2-MANJARO x86_64) Vulkan, GTX 2060 SUPER with proprietary drivers

### Issue description

Godot 3 throws an error when doing `var a := null` but fails to throw it when doing `func foo( a:= null)` and causes the function to accept args of any type and convert them to null.
This is fixed in Godot 4

### Steps to reproduce

Run a scene with the following code

```
extends Node

func foo ( a:= null):
	prints(a, typeof(a))

func _ready():
	foo(1)  # no error, prints Null
	foo("bar") # no error, prints Null
	foo([2,3,3]) # no error, prints Null
	foo(Sprite.new()) # no error, prints Null
```

This throws a parser error in Godot 4.0-rc1 but is allowed in Godot 3.5.2-rc2. Furthermore, you can pass arguments of any type and they will all be converted to null without any errors or warnings. 

### Minimal reproduction project

[InferredNullError.zip](https://github.com/godotengine/godot/files/10735812/InferredNullError.zip)
Repository: vscode
Issue: Git - Let me delete branches from branch picker
Labels: help wanted,feature-request,git

<!-- ⚠️⚠️ Do Not Delete This! feature_request_template ⚠️⚠️ -->
<!-- Please read our Rules of Conduct: https://opensource.microsoft.com/codeofconduct/ -->
<!-- Please search existing issues to avoid creating duplicates. -->

<!-- Describe the feature you'd like. -->
<img width="617" alt="image" src="https://user-images.githubusercontent.com/2644648/218830573-bfc6b5de-e827-40e5-a92c-f4bb70b59bba.png">

I accidentally made a `main'` branch and now it's there... I'd love to be able to delete from the picker using a 🗑️ icon.
Repository: flutter
Issue: [flutter_tools] allow filtering of adb logcat by "flutter" to be configured from CLI args
Labels: c: new feature,tool,c: proposal,P2,team-tool,triaged-tool

ADB logcat logs are very verbose, so we filter it by lines that contain the pattern `'flutter'`. We should make this configurable via a command-line flag, so that developers who want to debug the full logs can see this. This would also be leveraged by editor plugins, since--for example--Android Studio provides a UI for viewing full logcat logs in a natively launched application.

cc @jacob314
Repository: pytorch
Issue: Dynamo.export should support formatting tensor value within a string
Labels: feature,triaged,oncall: pt2,module: dynamo,internal ramp-up task

### 🚀 The feature, motivation and pitch

Code like this would fail to trace in dynamo.export

```
torch._assert(x.numel() > 0, f"My message, tensor value is: {x}")
```

Error message:
```
Unsupported: call_function BuiltinVariable(format) [ConstantVariable(str), TensorVariable()] {}
```

### Alternatives

N/A

### Additional context

It is desired because the code pattern is very common in modeling and there isn't really a work-around for its functionality.

cc @ezyang @msaroufim @wconstab @bdhirsh @anijain2305 @zou3519 @voznesenskym @penguinwu @EikanWang @jgong5 @Guobing-Chen @XiaobingSuper @zhuhaozhe @blzheng @wenzhe-nrv @jiayisunx @chenyang78 @aakhundov @kadeng @soumith @ngimel @yanboliang @Xia-Weiwen @desertfire
Repository: go
Issue: x/website: go.dev/dl was temporarily missing some port downloads
Labels: NeedsInvestigation,website

For one hour after the 1.19.6 / 1.20.1 release, [some ports, such as darwin-arm64, were missing](https://groups.google.com/g/golang-dev/c/G2APtTxT1HQ/m/W9vWNKHbBAAJ) from https://go.dev/dl/.

The likely cause is the [metadata cache](https://cs.opensource.google/go/x/website/+/master:internal/dl/dl.go;l=40;drc=62ac61ac47340da09c6091c6fb80e550677088d6), which may have updated at just the wrong moment as port packages were published, causing it to only see some ports, and then cache that partial view for 1hr until expiration.

cc @golang/release
Repository: go
Issue: net/http: multipart form parsing is hazardous and difficult to configure
Labels: NeedsInvestigation

`http.Request` helpfully parses `multipart/form-data` requests, not just explicitly in `req.ParseMultipartForm` but also implicitly in `req.FormFile`, `req.FormValue`, and `req.PostFormValue`.

Multipart form parsing uses `mime/multipart.Reader.ReadForm`, which has few limits on resource consumption:

* `ReadForm` takes a `maxMemory` parameter, but will allocate up to 10MiB of memory over and above this limit.
* `ReadForm` will write file form parts to disk, with no limit on the amount of disk consumed.

The defaults here are hazardous: A caller of `req.FormValue` may not intend to handle `multipart/form-data` forms at all, and will be surprised that the function can consume large amounts of memory and/or disk. The mechanisms for adjusting the defaults are inadequate: There is no simple way to disable multipart form parsing, to disable the use of disk temporary files, or to limit the memory consumption of `ParseMultipartForm` to less than 10MiB.

Compatibility may make it impossible to change the defaults, but at a minimum we should make it easy to configure reasonable limits.
Repository: opencv
Issue: Memory leak with std::async
Labels: bug,category: core,needs investigation

### System Information

OpenCV version: 4.5.4 and 4.7.0
Operating System / Platform: Ubuntu 22.04 (Jammy Jellyfish)
Compiler & compiler version: GCC 11.2.0 

### Detailed description

Hello, I've recently noticed a memory leak in our production software, inside our image processing pipeline, after switching to a parallel architecture based on ```std::async```.
Profiling through [Heaptrack](https://github.com/KDE/heaptrack), I can see that the leak comes from various functions in OpenCV, like ```cv::cvtColor``` and ```cv::forEach```, all of which call ```cv::parallel_for_``` and ```cv::TLSDataContainer::getData()```.
It seems that every async thread doesn't release its memory until the entire process is terminated.

![cv_mre_flame_graph](https://user-images.githubusercontent.com/42252688/218859676-687452b1-8bec-492e-b6f4-3e685845b963.png)

I have seen a few closed issues similar to this, but I don't understand if it's something that is supposed to have already been fixed, or if there is a more technical reason why async threads can't release memory at the end of their execution.

If that's the case, is there a patch/workaround to fix this, or is the only solution to abandon ```std::async``` and switch to a thread pool implementation ?

Thanks in advance.


### Steps to reproduce

```
#include <opencv2/opencv.hpp>
#include <future>

int main (int argc, char** argv)
{
    cv::Mat random = cv::Mat::zeros(cv::Size(100, 100), CV_8UC3);
    for (int i = 0; i < 1'000; i++)
    {
        cv::randu(random, 0, 255);
        std::async(std::launch::async, [&]()
            {
                cv::Mat result = cv::Mat::zeros(cv::Size(100, 100), CV_8UC1);
                cv::cvtColor(random, result, cv::COLOR_BGR2GRAY);
            }
        ).wait();
    }

    return 0;
}
```

Expected result: memory usage remains (near-)constant
Actual result: memory allocation grow with every async thread.
Results below obtained by running the above script, launching 1K and 1M async threads, respectively with a leak of 800KB and 800MB

1) 1'000 threads

![cv_mre_memory_nasync_1K](https://user-images.githubusercontent.com/42252688/218860875-490f46f6-fb55-4a44-9c4f-9f3e5328bd44.png)
![cv_mre_leak_report_nasync_1K](https://user-images.githubusercontent.com/42252688/218861196-d0d358a7-a4fa-430e-afbf-706b47ab8ba7.png)

2) 1'000'000 threads

![cv_mre_memory_nasync_1M](https://user-images.githubusercontent.com/42252688/218861283-4d56acee-a1c9-4c01-8fcf-32d00d9b54d4.png)
![cv_mre_leak_report_nasync_1M](https://user-images.githubusercontent.com/42252688/218861303-6b47a9ec-413d-44f0-8d0f-b1aaf515d6e3.png)


### Issue submission checklist

- [X] I report the issue, it's not a question
- [X] I checked the problem with documentation, FAQ, open issues, forum.opencv.org, Stack Overflow, etc and have not found any solution
- [X] I updated to the latest OpenCV version and the issue is still there
- [X] There is reproducer code and related data files (videos, images, onnx, etc)
Repository: flutter
Issue: CupertinoNavigationBar's "Back" button is not localized
Labels: framework,a: internationalization,f: cupertino,has reproducible steps,found in release: 3.7,found in release: 3.8,team-design,triaged-design

https://github.com/flutter/flutter/blob/480c54c370a651906e019f20e92d15673b6a4431/packages/flutter/lib/src/cupertino/nav_bar.dart#L1580-L1582
Repository: deno
Issue: NPM error: 'import', and 'export' cannot be used outside of module code at ...
Labels: bug,node compat

Hi.

I'm trying to use the [Kobalte](https://github.com/kobaltedev/kobalte) lib on a Deno project but I'm getting the next error:

```
error: 'import', and 'export' cannot be used outside of module code at file:///C:/Users/aw4ke/AppData/Local/deno/npm/registry.npmjs.org/@kobalte/core/0.6.0/dist/esm/index.js:1:1

  import { access, accessWith, getDocument, EventKey, getActiveElement, contains, ...

```

You can replicate the issue with the next code:

```typescript
import { Button } from 'npm:@kobalte/core';

console.log(typeof Button);
```

and ` deno run -A .\test.ts`

There is a similar opened issue (#17231) using another lib , but I have tried esm.sh, skypack.dev, etc... and it doesn't work either.

Any help is appreciated.
Repository: TypeScript
Issue: Promise constructor's return type doesn't unwrap nested promises
Labels: Needs Investigation

# Bug Report

### 🔎 Search Terms

promise constructor

### 🕗 Version & Regression Information


- This is the behavior in every version I tried, and I reviewed the FAQ for entries about Promises


### ⏯ Playground Link

<!--
  A link to a TypeScript Playground "Share" link which shows this behavior

  The TypeScript Workbench can be used for more complex setups, try
  https://www.typescriptlang.org/dev/bug-workbench/

  As a last resort, you can link to a repo, but these will be slower for us to investigate.
-->
[Playground link with relevant code](https://www.typescriptlang.org/play?ts=5.0.0-dev.20230210#code/MYewdgzgLgBGCm14BMAKAnEBbAlheMAvDABQBmArmMFDuCQJQwDeAUDBzOvFBemHHgB3GBmx54AHjG58ksBSwAjeOgB8aktwggANgDd4ARiJqW7TpYuWO2vYaMkEImRPmKV6rYnvwATKbmNsFcPgb+JACsDADc1jYAvgyxrPFJrEmMcawA9DkwAIbAoOjIOGAA5jBQIDAAKgDKADQwAAbaFLpQrTAQABYgncgwKoWimLLwLUoUsAWw6FS0WFMwOFAA5BCFNBQFuroAnmMKyqqsCEhoExIAdFB98GDeEJ2whGagkHrwt7ogFRIGzIOHQ0A2LQ6XRaUEOAAd4CAyKFXl1knE8iEsRwAHoAflSuXyQkeAgoEHKVVc+FudnCjBhj2q8IIeBgJW4NFYX2ggiu1PgACUwoYiKRKNRaPQmMxQrx+ONxDS6YYSALaSL4GqbsrNVFkgxMilLlAUALhTpwvdSS83oEeT8-gCgfgvsgISi3jCWUjPWiUpjsUHLPjCUA)

### 💻 Code

<!-- Please post the relevant code sample here as well-->
```ts
const nestedPromise = (function() {
    return new Promise<Promise<number>>(resolve1 => {
        
        resolve1(new Promise<number>(resolve2 => {
            resolve2(5);
        }));

    })
})();

// according to TS, `result` should be a Promise<number>, but at runtime, it's actually just a number
nestedPromise.then(result => console.log('first', result, typeof result));
//                 ^?


// when using Promise.resolve(), the type is correct
const nestedPromiseResolve = (function() { return Promise.resolve(Promise.resolve(Promise.resolve(5)))})();
nestedPromiseResolve.then(result => console.log('second', result, typeof result));
//                        ^?



```

### 🙁 Actual behavior

Return type of the promise constructor doesn't unwrap nested promises

### 🙂 Expected behavior

The return type of the promise constructor should match the runtime behavior


The current constructor type is `{ new <T>(executor: (resolve: (value: T | PromiseLike<T>) => void, reject: (reason?: any) => void) => void): Promise<T>; }`

Shouldn't the return type actually be `Promise<Awaited<T>>` like it is for `Promise.resolve()`? Is there something I'm missing here?