sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: go
Issue: x/net/http2: RoundTrip and/or CloseIdleConnections may leave connections in use after returning
Labels: NeedsInvestigation

In #43176, we observed that `x/net/http2.TestTransportGroupsPendingDials` sometimes fails on various platforms.

The test performs multiple concurrent calls to a `Transport`'s `RoundTrip` method, waits for them to return, closes the `Response` bodies, and then calls `CloseIdleConnections`. If all of those functions are synchronous — their signatures suggest that they should be, and nothing in their documentation states otherwise — then the `Transport`'s client connection pool should be empty as soon as that sequence is complete.

However, the connection pool is empirically <strong>not</strong> empty at that point, and to my knowledge never has been. The test has had a `retry` loop ever since it was added (in [CL 17134](https://golang.org/cl/17134)), and even then a single call to `CloseIdleConnections` before the retry loop is empirically not enough to ensure that it succeeds.

We should add more explicit tests for the expected invariants after `RoundTrip` and `CloseIdleConnections`, and/or update the documentation for those methods (and the corresponding methods and interfaces in the `net/http` package) to make those invariants clearer for end users, who may want to make use of any such invariants in their own tests.

(CC @neild @tombergan @bradfitz)
Repository: godot
Issue: Deleting a C# script and saving the scene saves the script as built-in
Labels: bug,topic:editor,topic:dotnet

### Godot version

v3.4.stable.mono.official [206ba70f4]

### System information

Pop OS 21.04

### Issue description

When a C# script is deleted from the FileSystem dock and the scene that had a reference to that script is open, saving causes the deleted script to be converted into a built-in script of that scene (this is not allowed in C#, built-in scripts are not supported).

Seems to be caused by `DependencyRemoveDialog` setting the CSharpScript path to an empty string in this line:

https://github.com/godotengine/godot/blob/46d384060ef20672e23b3c1ffe947ff7898ecf75/editor/dependency_editor.cpp#L483

### Steps to reproduce

1. Open a scene that contains a node with a C# script attached
2. Delete C# script file using the FileSystem dock
3. Save without previously closing the scene <kbd>Ctrl + S</kbd>
4. The deleted C# script is now a built-in script of the scene

### Minimal reproduction project

_No response_
Repository: excalidraw
Issue: When opening the mac native emoji picker, I'm then stuck in "grab" mode
Labels: bug

When pressing CMD + Control + Space (Mac emoji picker) then close it, I get stuck in grab mode:


https://user-images.githubusercontent.com/15836571/145077550-dcce64a3-4343-43ec-8d70-a2fc4eb33c4c.mov
Repository: create-react-app
Issue: When you use this plugin you must install `typescript`?
Labels: needs triage,issue: bug report

I posted the question in the discussion.  It's not clear if this is a bug or if I'm just doing it incorrect.
https://github.com/facebook/create-react-app/discussions/11704

### Describe the bug

I clone the create-react-app repo and ran the following commends:

```
npx create-react-app tester4 --scripts-version file:./create-react-app/packages/react-scripts --template file:./create-react-app/packages/cra-template-typescript

....

cd tester4
yarn start                                                                           
yarn run v1.22.17
$ react-scripts start
**When you use this plugin you must install `typescript`.**
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
```

I would expect it to work similar to when you just call `npx create-react-app tester4 --typescript`, but it's not.

Questions
1) isn't `npx create-react-app tester4 --typescript` the same as `npx create-react-app tester4 --scripts-version file:./create-react-app/packages/react-scripts --template file:./create-react-app/packages/cra-template-typescript`
2) When you see the message `When you use this plugin you must install `typescript`` what are you suppose to do.  Do you just install typescript in the custom --script-version.  Like in this case, isn't `-scripts-version file:./create-react-app/packages/react-scripts` the default anyways?

Thanks,
Derek
Repository: yt-dlp
Issue: [TikTok] Add more data to profile json
Labels: site-enhancement

### Checklist

- [X] I'm reporting a site feature request
- [X] I've verified that I'm running yt-dlp version **2021.12.01**. ([update instructions](https://github.com/yt-dlp/yt-dlp#update))
- [X] I've checked that all provided URLs are alive and playable in a browser
- [X] I've searched the [bugtracker](https://github.com/yt-dlp/yt-dlp/issues?q=) for similar issues including closed ones. DO NOT post duplicates
- [X] I've read the [guidelines for opening an issue](https://github.com/yt-dlp/yt-dlp/blob/master/CONTRIBUTING.md#opening-an-issue)
- [X] I've read about [sharing account credentials](https://github.com/yt-dlp/yt-dlp/blob/master/CONTRIBUTING.md#are-you-willing-to-share-account-details-if-needed) and I'm willing to share it if required

### Region

_No response_

### Example URLs

https://www.tiktok.com/@kyliejenner


### Description

When you download a TikTok user profile and use `--write-info-json` it write some info about the user in a json file.
Is it possible to add more data to it like follower count, following count, total likes, Bio and maybe also download the profile image?
Repository: rust
Issue: Tracking Issue for enabling `elided_lifetimes_in_paths` lints
Labels: A-lints,T-lang,E-help-wanted,C-tracking-issue,S-tracking-impl-incomplete,L-elided_lifetimes_in_paths

This issue tracks getting the currently-allow-by-default `elided_lifetimes_in_paths` lint <https://doc.rust-lang.org/rustc/lints/listing/allowed-by-default.html#elided-lifetimes-in-paths> into a place where it can be enabled by default.

### Context

(This was formerly in the in-band lifetimes tracking issue, #44524, but is being split out because it doesn't actually depend on *in-band* specifically now that `'_` is stable.)

There seems to be general agreement amongst lang that lifetimes that participate in elision should be visible.  That means things like
```diff
-fn foo<T>(x: &T) -> Foo<T>
+fn foo<T>(x: &T) -> Foo<'_, T>
```
and
```diff
-fn foo<T>(x: Foo<T>, y: Bar<T>) -> &T
+fn foo<T>(x: Foo<'_, T>, y: Bar<T>) -> &T
```

However, there's less agreement whether they're needed in other places.  For example, whether it's valuable to require showing the lifetime in 
```diff
 impl fmt::Debug for StrWrap {
-    fn fmt(&self, fmt: &mut fmt::Formatter) -> fmt::Result {
+    fn fmt(&self, fmt: &mut fmt::Formatter<'_>) -> fmt::Result {
         fmt.write_str(self.0)
     }
 }
```
A comment against needing that kind of `'_`: https://github.com/rust-lang/rust/issues/44524#issuecomment-414062464
And one saying it's helpful to show it: https://github.com/rust-lang/rust/issues/44524#issuecomment-414663773

(If anyone remembers where lang discussed this and has notes about it, please post or edit this comment!)

Perhaps one way to make progress is to start splitting up the lint to isolate the different cases?

### Related discussions

- https://github.com/rust-lang/rust/issues/131725

### Current status

- PR to split up `elided_lifetimes_in_paths` into two separate lints under the same lint group: https://github.com/rust-lang/rust/pull/120808

### Unresolved questions

- For #120808, what should the split-up lints be called, i.e. are "tied" vs "untied" terminology sufficiently clear?
Repository: pytorch
Issue: [Distributed Tests] Print stacktraces of all processes when one fails
Labels: oncall: distributed,better-engineering,Stale

## 🚀 Feature

To better debug stuck issues in our unittests, when one process times out or otherwise fails, it would be useful to print out the state of other processes to see where they are stuck/what they are doing. 

We currently already do this with `_get_timedout_process_traceback` in `common_distributed.py`, for timed-out processes but we should extend it to do so for other cases as well. 

As a result, it would better help us debug issues such as https://github.com/pytorch/pytorch/runs/4302443560?check_suite_focus=true, it would be super useful to see what process 1 was doing when the rest of the processes failed.

cc @pietern @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @osalpekar @jiayisuse @SciPioneer @H-Huang
Repository: flutter
Issue: Pigeon: using `dynamic` silently generates code that won't compile
Labels: package,team-ecosystem,p: pigeon,P3,triaged-ecosystem

As per the Pigeon [docs](https://pub.dev/packages/pigeon#feedback), I am tagging @gaaclarke to get help on this. Thanks in advance.

I have the following file, `pigeons/push.dart`:
```dart
import 'package:pigeon/pigeon.dart';

@HostApi()
abstract class MyApi {
  Map<String?, dynamic?> getMap();
}
```
Note, I concluded this is the right code based on my previous issue https://github.com/flutter/flutter/issues/94689 and @stuartmorgan's reply, even though I am confused because "`dynamic?` certainly _should_ not be required, since `dynamic` includes `null` already", and "what does an optional key type mean?".

When I run the code generation, it succeeds. However, the output of the code generation seems incorrect. One line in the generated code looks like this:
```dart
        return dynamic.decode(readValue(buffer)!);
```

![CleanShot 2021-12-07 at 21 53 44](https://user-images.githubusercontent.com/24711048/145111920-6b268218-0110-4b9d-bc4e-f2f4cf82ea89.png)

The error being:
> The method 'decode' isn't defined for the type 'Type'.

`dynamic` does not have a `decode` method.

The full class looks like:
```dart
class _MyApiCodec extends StandardMessageCodec {
  const _MyApiCodec();
  @override
  void writeValue(WriteBuffer buffer, Object? value) {
    if (value is dynamic) {
      buffer.putUint8(128);
      writeValue(buffer, value.encode());
    } else 
{
      super.writeValue(buffer, value);
    }
  }
  @override
  Object? readValueOfType(int type, ReadBuffer buffer) {
    switch (type) {
      case 128:       
        return dynamic.decode(readValue(buffer)!);
      
      default:      
        return super.readValueOfType(type, buffer);
      
    }
  }
}
```

Perhaps decoding should be skipped if the type is dynamic.
Repository: rust
Issue: Tracking Issue for `core::future::join!`
Labels: T-libs-api,C-tracking-issue

Feature gate: `#![feature(future_join)]`

This is a tracking issue for `core::future::join!`, which polls multiple futures concurrently and returns their outputs.

```rust
async fn run() {
    let (a, b) = join!(async { 0 }, async { 1 }).await;
}
```

### Public API

```rust
// core::future

pub macro join {
    ( $($fut:expr),* $(,)?) => {
        // ...
    }
}
```

### Steps / History

<!--
For larger features, more steps might be involved.
If the feature is changed later, please add those PRs here as well.
-->

- [x] Implementation: #91645
- [x] Minor improvements: #91721
- [ ] Final comment period (FCP)
- [ ] Stabilization PR

<!--
Once the feature has gone through a few release cycles and there are no
unresolved questions left, the feature might be ready for stabilization.

If this feature didn't go through the RFC process, a final comment period
(FCP) is always needed before stabilization. This works as follows:

A library API team member can kick off the stabilization process, at which point
the rfcbot will ask all the team members to verify they agree with
stabilization. Once enough members agree and there are no concerns, the final
comment period begins: this issue will be marked as such and will be listed
in the next This Week in Rust newsletter. If no blocking concerns are raised in
that period of 10 days, a stabilzation PR can be opened by anyone.
-->

### Unresolved Questions

<!--
Include any open questions that need to be answered before the feature can be
stabilised. If multiple (unrelated) big questions come up, it can be a good idea
to open a separate issue for each, to make it easier to keep track of the
discussions.

It's useful to link any relevant discussions and conclusions (whether on GitHub,
Zulip, or the internals forum) here.
-->

- Implicit .await? Tokio and futures currently do this.
- `await_all!` vs `join!`
- `join/and` method on `Future`: `a.and(b).await`
- Implement `IntoFuture` for N-arity tuples? `let (a, b, c) = (a, b, c).await`
Repository: create-react-app
Issue: Update sass-loader to 12.4.0 to support node-sass 7.0.0
Labels: needs triage,issue: bug report

### Describe the bug

Currently, an error is given when trying to use node-sass 7.0.0:
./src/index.scss (./node_modules/css-loader/dist/cjs.js??ref--5-oneOf-6-1!./node_modules/postcss-loader/src??postcss!./node_modules/resolve-url-loader??ref--5-oneOf-6-3!./node_modules/sass-loader/dist/cjs.js??ref--5-oneOf-6-4!./src/index.scss)
Node Sass version 7.0.0 is incompatible with ^4.0.0 || ^5.0.0 || ^6.0.0.

node-sass is still useful because sass (dart-sass) is slower

sass-loader was just updated to 12.4.0 today to support node-sass 7.0.0 (also a recent update).

It would be nice to have this! Thanks
Repository: pytorch
Issue: [subgraph_rewriter] Add match_filter for the matches
Labels: triaged,Stale,module: fx

## 🚀 Feature
Sometimes we may want to further filtering for the matches. like https://github.com/pytorch/pytorch/blob/master/torch/ao/quantization/fx/quantized_fusion_patterns_and_replacements.py#L23

we can add a match_filters argument to the replace_pattern function:

```
def replace_pattern(
        gm: GraphModule,
        pattern: Callable,
        replacement: Callable,
        is_match_filters: Optional[List[Callable]] = None) -> List[Match]:
   ...
```

feel free to take a look at https://github.com/pytorch/pytorch/blob/master/torch/ao/quantization/fx/subgraph_rewriter_FORKED_DO_NOT_USE.py for a possible implementation


cc @ezyang @SherlockNoMad
Repository: godot
Issue: Godot headless export does not include a png texture that is part of a shader
Labels: bug,topic:editor,confirmed,topic:import,topic:export

### Godot version

3.4 stable

### System information

windows, linux,html5

### Issue description

I have a reproduction project, that when exported via the editor it creates a .import asset for grid.ong and its visible in the exported binary.
however, when using the godot headless binary to export it misses that asset and the texture is missing.
The issue also happends in the github action in the project which uses the headless build.

I suspect its because its part of a shader.

I have made a repo, that reproduces the issue:

https://github.com/guysoft/godot_import_action_reproduce/issues/1

### Steps to reproduce

See: https://github.com/guysoft/godot_import_action_reproduce/issues/1

Use the github action or the command lised in the example project issue

### Minimal reproduction project

https://github.com/guysoft/godot_import_action_reproduce

Images and more description:
https://github.com/guysoft/godot_import_action_reproduce/issues/1
Pasting here so you can see too:
When using the godot editor and running/exporting the scene looks like this.
This is the expected result:
![expected](https://user-images.githubusercontent.com/325670/145123714-95ef3df4-bf63-4351-80ac-9bf11509efe8.png)


When using the gihub actions the grid.png is not imported and the scene looks like this.
This is the actual result.

How to reproduce in commandline:
```
Godot_v3.4-stable_linux_headless.64 --export Linux/X11 builds/LinuxX11/Energy-source-linux.x86_64 --verbose
```
(there is also a windows export that gives the same issue)

This also happens if you export this project using the godot headless script which this github action uses:
![actual](https://user-images.githubusercontent.com/325670/145123689-26f928bf-6121-4892-9b49-2cc2611bee55.png)


Binaries are available in the release page from the action:
https://github.com/guysoft/godot_import_action_reproduce/releases
Repository: godot
Issue: Duplicate Transposed should be renamed or merged with Duplicate
Labels: enhancement,topic:editor,usability

### Godot version

ff118d7

### System information

W10

### Issue description

I recently dug up this old issue: https://github.com/godotengine/godot-docs/issues/3967. It started from a user confused about Duplicate Transposed option. Then it was labeled as documentation issue and transferred, and finally closed, because the option was supposedly removed.

Except it wasn't, see my last comment. It still exists in the Edit menu, with a default shortcut and fully functional. So the original issue still stands. But instead of documenting an obscure option, I think it should be renamed to be more intuitive. Maybe to something like "Duplicate To Another Track".

Or maybe it could be merged with duplicate, which should just place the keys on the current track by default or on the selected track if one is selected and usable? (the operation fails for non-matching types)

### Steps to reproduce

1. Go to Animation tab
2. Open the Edit menu
3. Notice the Duplicate Transposed option
4. You are now automatically confused

### Minimal reproduction project

_No response_
Repository: pytorch
Issue: Make DDP + Zero have the same communication volume as regular DDP
Labels: oncall: distributed,module: ddp,Stale

## 🚀 Feature

Currently when training DDP + ZeRo, we still reduce all gradients but ZeRo will update on each rank only the local parameter shard and then broadcast the rest. As a result, DDP only needs to reduce the gradients corresponding to that shard. In addition, zero redundancy optimizer does N broadcasts to share parameters across ranks: https://fburl.com/3o00xa81, but a single allgather could be done instead to improve performance.

cc @pietern @mrshenli @pritamdamania87 @zhaojuanmao @satgera @rohan-varma @gqchen @aazzolini @osalpekar @jiayisuse @SciPioneer @H-Huang
Repository: terminal
Issue: IPv6 links can not be Ctrl+Clicked
Labels: Help Wanted,Area-TerminalControl,Product-Terminal,Issue-Task

### Windows Terminal version

1.12.2931.0

### Windows build number

10.0.22000.348

### Other Software

_No response_

### Steps to reproduce

1. Run any command that writes an IPv6 link in the terminal window, for example `echo http://[::1]/`
2. Hover over the link, or try to Ctrl+Click it to follow it

### Expected Behavior

Hovering over the link should show the underline and the tooltip about Ctrl+Clicking it, and Ctrl+Clicking it should follow the link like any other link.

### Actual Behavior

Nothing happens when hovering over the link or Ctrl+Clicking it.

![_](https://user-images.githubusercontent.com/2956956/145138785-78fb988a-36a7-4a1b-b661-2730f49282ba.gif)
Repository: PowerToys
Issue: Move Window directly to monitor a diff (useful in 3+ monitors) using shortcut
Labels: Idea-New PowerToy,Product-Window Manager

### Description of the new feature / enhancement

It allows you to move the currently active window to another monitor at once by using a shortcut key.

Requirements:
- You can assign a shortcut key.
- You can specify which shortcut to use when moving to which monitor.

### Scenario when this would be used?

I use a laptop monitor and two monitors, a total of 3 monitors, and it is not easy to move each window with the keyboard, which is always inconvenient in this window environment. I'm currently looking for such a shortcut, but haven't been able to find it yet. If it is PowerToys, I request this function because I think that this kind of function should be sufficient.

### Supporting information

_No response_
Repository: go
Issue: cmd/doc: constructor sometimes not associated with type
Labels: NeedsFix

Using `go1.17.1`

When I run:
```
$ go doc net/http.Request
```
I see (with some irrelevant details removed):
```
package http // import "net/http"

type Request struct {
    ...
}

func (r *Request) AddCookie(c *Cookie)
func (r *Request) BasicAuth() (username, password string, ok bool)
func (r *Request) Clone(ctx context.Context) *Request
func (r *Request) Context() context.Context
func (r *Request) Cookie(name string) (*Cookie, error)
func (r *Request) Cookies() []*Cookie
func (r *Request) FormFile(key string) (multipart.File, *multipart.FileHeader, error)
func (r *Request) FormValue(key string) string
func (r *Request) MultipartReader() (*multipart.Reader, error)
func (r *Request) ParseForm() error
func (r *Request) ParseMultipartForm(maxMemory int64) error
func (r *Request) PostFormValue(key string) string
func (r *Request) ProtoAtLeast(major, minor int) bool
func (r *Request) Referer() string
func (r *Request) SetBasicAuth(username, password string)
func (r *Request) UserAgent() string
func (r *Request) WithContext(ctx context.Context) *Request
func (r *Request) Write(w io.Writer) error
func (r *Request) WriteProxy(w io.Writer) error
```

This only lists the methods of `Request`, but does not show any of the constructor functions:
* [NewRequest](https://pkg.go.dev/net/http#NewRequest)
* [NewRequestWithContext](https://pkg.go.dev/net/http#NewRequestWithContext)
* [ReadRequest](https://pkg.go.dev/net/http#ReadRequest)

I would expect it to list the constructors as well, since it does so for other types (e.g., `go doc compress/gzip.Reader`).

Also, the fact that it does not list the constructors is [inconsistent with `go.pkg.dev`](https://pkg.go.dev/net/http#Request), which does group those three functions with the type.
Repository: ant-design
Issue: form.getFieldsValue(nameList, filterFunc), 第二个参数过滤函数是否受到第一个参数的限制？
Labels: Inactive

- [x] I have searched the [issues](https://github.com/ant-design/ant-design/issues) of this repository and believe that this is not a duplicate.

### Reproduction link
[![Edit on CodeSandbox](https://codesandbox.io/static/img/play-codesandbox.svg)](https://codesandbox.io/s/antd-form-demo-gvf1u?file=/form/index.tsx)

### Steps to reproduce
1.  填写user为111，填写password为222
2. 点击Password Form Item show Btn
3. 点击Submit Btn
4. 查看console

### What is expected?
form.getFieldsValue(nameList, filterFunc)，第二个参数里过滤函数与第一个参数的关系

### What is actually happening?
1. 只使用第一个参数传true，返回user和password内容
2. 第一个参数为true,第二个参数里的过滤函数只过滤现存（页面存在<Form.item>的）字段，返回user内容，这是符合预期的吗？(文档里没有介绍第二个参数)

| Environment | Info |
|---|---|
| antd | 4.8.6 |
| React | 17.0.1 |
| System | mac os |
| Browser | chrome  96.0.4664.55 |

---
就是想问一下第二个参数里过滤函数与第一个参数传true时的关系，现在项目了用到了这个特性（过滤函数只过滤页面存在<Form.item>的字段），想了解一下后续是否会修改此逻辑。


<!-- generated by ant-design-issue-helper. DO NOT REMOVE -->