sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: flutter
Issue: [animations] PageTransitionSwitcher doesn't work well with IndexedStack
Labels: package,p: animations,team-ecosystem,P3,triaged-ecosystem

I want to use PageTransitionSwitcher with IndexedStack to display my tab pages.
If I don't set a **ValueKey** to **IndexedStack**, it doesn't animate but the IndexStack works well.
If **I do** set a **ValueKey**, it animates but IndexedStack rebuilds all the time and causes the tab page's state to be lost. Which defeats the purpose of using IndexedStack.

I did try to use **AutomaticKeepAliveClientMixin**, but still rebuilds the tabs, and I just want to avoid it anyways since IndexedStack keeps my code cleaner.

I believe there are many users that want to use this combination since it is a very common pattern and it shouldn't be so complicated.

What is the best solution?

Regards
Hiroshi
```
            PageTransitionSwitcher(
              duration: const Duration(milliseconds: 300),
              reverse: newTabIndex < tabIndexNotifier.previous,
              transitionBuilder: (
                Widget child,
                Animation<double> animation,
                Animation<double> secondaryAnimation,
              ) =>
                  SharedAxisTransition(
                child: child,
                animation: animation,
                secondaryAnimation: secondaryAnimation,
                transitionType: SharedAxisTransitionType.horizontal,
              ),
              child: IndexedStack(
                key: ValueKey(newTabIndex),
                index: newTabIndex,
                children: _pageWidgets,
              ),
            ),
```
Repository: node
Issue: WIP label vs convert to draft?
Labels: meta

Should we remove the WIP label in favor converting PR's to drafts? Update the collaborator guidelines maybe? Or what is the thoughts on this?
Repository: rust
Issue: std::process::Command should probably implement Display
Labels: T-libs-api,C-feature-request,A-process

At the moment, `Command` does not implement `Display`. However, it's often useful to display `Command` for the end user, the most common case being that the command failed, and you want to display an error message. If `cmd.to_string()` returned something like `cargo --metadata --manifest-path ./foo/Cargo.toml`, that would be sweet. 

It's also instructive to compare it with `Path`s. `Path` deliberately doesn't implement display directly, and the reason for that (I think) is not to make people thing about non-text path when displaing paths, but to make it harder to miss-use string formatting for path operations. IE, we don't want to people doing 
```
let path = PathBuf::from(format!("{}/{}", base, name))
```
, as that doesn't handle bag-of-bytes paths. 

For `Command`, I don't think `to_string()` can be missused in a similar manner, so adding `Display` seems like a clear win to me.
Repository: flutter
Issue: Changing into using Swift UI instead of Storyboard
Labels: c: new feature,platform-ios,tool,t: xcode,c: proposal,P3,team-ios,triaged-ios

<!-- 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 our the template below. Please read
     our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports
-->

## Use case

<!--
     Please tell us the problem you are running into that led to you wanting
     a new feature.

     Is your feature request related to a problem? Please give a clear and
     concise description of what the problem is.

     Describe alternative solutions you've considered. Is there a package
     on pub.dev/flutter that already solves this?
-->

This is not a problem in the meantime but could be in the near future. The problem is that Flutter uses `storyboard` instead of `Swift UI`. As I said not a real problem but could be in the future, better to consider it as a feature :).

## Proposal

<!--
     Briefly but precisely describe what you would like Flutter to be able to do.

     Consider attaching images showing what you are imagining.

     Does this have to be provided by Flutter directly, or can it be provided
     by a package on pub.dev/flutter? If so, maybe consider implementing and
     publishing such a package rather than filing a bug.
-->

The proposal is to switch from using `storyboard` to `Swift UI`. Short but to the point.
Repository: rust
Issue: cfg_accessible does not respect feature gating
Labels: T-lang,requires-nightly,F-cfg_accessible

(Since `cfg(version)`'s support for nightly is under discussion, I decided to check how `cfg(accessible)`'s support was going. I realise it is far from done, but since it's possible to write a testcase for this now I thought it would be worth having an issue to make sure it's not forgotten.)

```rust
#![feature(cfg_accessible)]

#[cfg_accessible(std::fmt::rt::v1)]
const _: () = { use std::fmt::rt::v1; };
```

This code ([playground](https://play.rust-lang.org/?version=nightly&mode=debug&edition=2018&gist=8ddc1c43d992b457d16dea7fd8d74ba4)) should cleanly compile as `std::fmt::rt::v1` is gated behind the inactive `fmt_internals` feature, and [`cfg(accessible)` should ignore disabled gated features](https://github.com/rust-lang/rfcs/blob/master/text/2523-cfg-path-version.md#feature-gating). Instead it errors out:

```
error[E0658]: use of unstable library feature 'fmt_internals': internal to format_args!
```

cc #64797
Repository: flutter
Issue: [VoiceOver] Confusing "Page -9223xxxxxx of 9223xxxxxx" when double tapping on the status bar time
Labels: platform-ios,framework,engine,a: accessibility,has reproducible steps,P2,found in release: 3.3,found in release: 3.7,team-ios,triaged-ios

When a flutter app is running with Voice Over on, selecting the iOS statusbar's time and double tapping causes Voice over to read some weird message.

The message is something like: "Page -9223xxxxxx of 9223xxxxxx". I've attached an image with the voice over text being read.

The code that reproduces it easily is:

```
import 'package:flutter/material.dart';
import 'package:flutter/rendering.dart';

void main() {
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: SemanticsCardHomePage(),
    );
  }
}

class SemanticsCardHomePage extends StatelessWidget {

  @override
  Widget build(BuildContext context) {

    SystemChrome.setSystemUIOverlayStyle(SystemUiOverlayStyle.dark);

    return Scaffold(
      body: Column(
        mainAxisAlignment: MainAxisAlignment.start,
        crossAxisAlignment: CrossAxisAlignment.stretch,
        children: <Widget>[
          Padding(
            padding: const EdgeInsets.all(24.0),
            child: Text("Hello"),
          ),
          Padding(
            padding: const EdgeInsets.all(24.0),
            child: Text("World"),
          ),
        ],
      ),
    );
  }
}
Repository: pytorch
Issue: Add helpers to save/load RPC-based models
Labels: feature,triaged,module: rpc

Currently, there is no convenient ways to save and load RPC models. Applications need to implement this in the user code, e.g., by using RPC to tell all participating works to trigger local save/load. Moreover, we intentionally disabled RRef pickling to avoid messing up reference count, which makes the save/load process even trickier. 


## Pitch

We should add a helper to save/load RPC-based models. 

### Save

Similar to distributed optimizer, the helper can potentially:

1. Take a model and recursively walk through all its attributes and extract all RRefs into a set 
2. Save all non-RRef attributes locally
3. For RRef attributes, use RPC to recursively trigger save on the remote side 
4. For `OwnerRRef` save its `local_value()` and remember all its `UserRRef`s. This also needs to be done recursively as the `local_value` can contain other RRefs. 

### Load

1. Load all local parts
2. Use loaded local parts to create `OwnerRRef`s. 
3. Use `OwnerRRef` to create all `UserRRef`s. Step 2 and 3 need to be done recursively, as `OwnerRRef.local_value` can contain other RRefs.




cc @pietern @mrshenli @pritamdamania87 @zhaojuanmao @satgera @gqchen @aazzolini @rohan-varma @xush6528 @jjlilley @osalpekar
Repository: TypeScript
Issue: TypeScript service: inconsystent behaviour between OpenExternalProject / CloseExternalProject commands on Windows
Labels: Bug

In WebStorm we use external projects for configs with non-standard names e.g. tsconfig.app.json.
The life-cycle is very simple:
1. Open an external project using 
```
{"sessionId":1588939608894,"seq":6,"type":"request","command":"openExternalProject","arguments":{"projectFileName":"C:\\BuildAgent\\temp\\buildTmp\\unitTest_useCustomConfig\\unitTest10801416240959877233\\subdir1\\tsconfig.app.json","rootFiles":[{"fileName":"C:\\BuildAgent\\temp\\buildTmp\\unitTest_useCustomConfig\\unitTest10801416240959877233\\subdir1\\tsconfig.app.json"}],"typeAcquisition":{"enable":false}}}
```
after work (when all files are closed) we close the external project:
```
{"sessionId":1588939608894,"seq":23,"type":"request","command":"closeExternalProject","arguments":{"projectFileName":"C:\\BuildAgent\\temp\\buildTmp\\unitTest_useCustomConfig\\unitTest10801416240959877233\\subdir1\\tsconfig.app.json"}}
```
It works fine for Linux / OS X but by some reason it doesn't close the project for Windows.

I've checked source code these open / close commands and I've found that the open command doesn't normalize the ``projectFileName``
e.g.
```
//src/server/editorServices.ts:3157
this.externalProjectToConfiguredProjectMap.set(proj.projectFileName, tsConfigFiles)
```
while the close command always use normalized name:
```
//src/server/editorServices.ts:2908
const fileName = toNormalizedPath(uncheckedFileName);
const configFiles = this.externalProjectToConfiguredProjectMap.get(fileName);
```

I believe it is a bug and the behavior must be similar for open / close.
Repository: go
Issue: x/build/cmd/gerritbot: should determine author from first commit rather not last
Labels: Builders,NeedsInvestigation,FeatureRequest,Community

When an imported PR has many commits, the Gerrit CL author is seemingly determined by taking the git author of the latest commit in the PR.

I think it may be better to take the git author of the first commit rather than last. That way, if an additional contributor adds more commits, the author of the CL would not change. The first commit seems like a more stable source for this information.

I noticed this in https://go-review.googlesource.com/c/go/+/216617/47..48 when I pushed a new merge commit to the end of https://github.com/golang/go/pull/36815/commits, and the author of the Gerrit CL changed.

/cc @golang/osp-team @griesemer

P.S. For bonus points, it can be considered to convert additional authors into "Co-authored-by" lines.
Repository: flutter
Issue: [gen_l10n] Exposing tool's options when using l10n.yaml to configure
Labels: tool,a: internationalization,c: proposal,P3,team-tool,triaged-tool

When running `gen_l10n` from the command line, there are descriptions for each option that are available to the user. When using `l10n.yaml` to configure and use the gen_l10n tool, we should still have a good way to share the list of available options with users, as well as descriptions for how to use the tool.
Repository: godot
Issue: Mono build StackOverflowException when opening project in editor
Labels: bug,topic:editor,topic:dotnet,crash

**Godot version:**
Godot_v3.2.1-stable_mono_win64.exe

**OS/device including version:**
Windows 8.1

**Issue description:**
Godot editor crashes 

**Steps to reproduce:**
Open a project to edit

**What happened**
I was editing C# script. Saved it and clicked on Build in Godot editor and it crashed. Tried to open my project but it closes every time while opening. Other projects are opening fine. 

Contents of projects log.txt file:

> Godot Engine v3.2.1.stable.mono.official - https://godotengine.org
> OpenGL ES 3.0 Renderer: GeForce 840M/PCIe/SSE2
>  
> Mono: Logfile is: C:\Users\Username\AppData\Roaming/Godot/mono/mono_logs/2020_05_08 19.08.42 (9340).txt
> **WARNING**: EditorSettings::_get - Property not found: mono/editor/editor_path_optional
>    At: editor/editor_settings.cpp:151:_get() - EditorSettings::_get - Property not found: mono/editor/editor_path_optional
> **ERROR**: System.StackOverflowException: The requested operation caused a stack overflow.
>    At: modules/mono/mono_gd/gd_mono_utils.cpp:357:debug_send_unhandled_exception_error() - System.StackOverflowException: The requested operation caused a stack overflow.
> **ERROR**: System.StackOverflowException: The requested operation caused a stack overflow.
>    At: modules/mono/mono_gd/gd_mono_utils.cpp:357:debug_send_unhandled_exception_error() - System.StackOverflowException: The requested operation caused a stack overflow.
> **ERROR**: System.StackOverflowException: The requested operation caused a stack overflow.
>    At: modules/mono/mono_gd/gd_mono_utils.cpp:357:debug_send_unhandled_exception_error() - System.StackOverflowException: The requested operation caused a stack overflow.

**Here is contents of [2020_05_08 19.08.42 (9340).txt](https://pastebin.com/xkaDVkqW) file.**
Repository: neovim
Issue: cursor shape in st
Labels: bug,tui,complexity:low

<!-- Before reporting: search existing issues and check the FAQ. -->

- `nvim --version`:
```
NVIM v0.4.3
Build type: Release
LuaJIT 2.1.0-beta3
Compilation: /usr/bin/cc -g -O2 -fdebug-prefix-map=/build/neovim-U5Ub0w/neovim-0.4.3=. -fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=1 -DDISABLE_LOG
-Wdate-time -D_FORTIFY_SOURCE=1 -O2 -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wshadow -Wconversion -Wmissing-prototypes -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fdiagnostics-color=always -DINCLUDE_GENERATED_DECLARATIONS -D_GNU_SOURCE -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -I/build/neovim-U5Ub0w/neovim-0.4.3/build/config -I/build/neovim-U5Ub0w/neovim-0.4.3/src -I/usr/include -I/usr/include/lua5.1 -I/build/neovim-U5Ub0w/neovim-0.4.3/build/src/nvim/auto -I/build/neovim-U5Ub0w/neovim-0.4.3/build/include
Скомпилирован  team+vim@tracker.debian.org

Features: +acl +iconv +tui
See ":help feature-compile"

            общесистемный файл vimrc: "$VIM/sysinit.vim"
          значение $VIM по умолчанию: "/usr/share/nvim"

Run :checkhealth for more info
```

- Operating system/version: `Ubuntu 18.04`
- Terminal name/version: `st 0.8.3`
- `$TERM`: `st-256color`

Terminfo entry is installed and has required capabilities.
Output of `infocmp -1x | grep "Se\|Ss"`:
```
        Se=\E[2 q,
        Ss=\E[%p1%d q,
```

`nvim -u NORC -V3` has this:
```
Extended string capabilities:
  Ms                        = <Esc>]52;%p1%s;%p2%s<C-G>
  Se                        = <Esc>[2 q
  Ss                        = <Esc>[%p1%d q
  rmxx                      = <Esc>[29m
  smxx                      = <Esc>[9m
  ext.get_bg                = <Esc>]11;?<C-G>
  Ss                        = <Esc>]50;CursorShape=%?%p1%{3}%<%t%{0}%e%p1%{5}%<%t%{2}%e%{1}%;%d;BlinkingCursorEnabled=%?%p1%{1}%<%t%{1}%e%p1%{1}%&%;%d<C-G>
  Se                        = <Esc>]50;<C-G>
  ext.resize_screen         = <Esc>[8;%p1%d;%p2%dt
  setrgbf                   = <Esc>[38;2;%p1%d;%p2%d;%p3%dm
  setrgbb                   = <Esc>[48;2;%p1%d;%p2%d;%p3%dm
  ext.save_title            = <Esc>[22;0;0t
  ext.restore_title         = <Esc>[23;0;0t
  ext.enable_lr_margin      = <Esc>[?69h
  ext.disable_lr_margin     = <Esc>[?69l
  ext.enable_bpaste         = <Esc>[?2004h
  ext.disable_bpaste        = <Esc>[?2004l
  ext.enable_focus          = <Esc>[?1004h
  ext.disable_focus         = <Esc>[?1004l
  ext.enable_mouse          = <Esc>[?1002h<Esc>[?1006h
  ext.disable_mouse         = <Esc>[?1002l<Esc>[?1006l
```
Notice how Se and Ss appear on the list twice. According to terminal log, neovim actually writes ESC]50 (the wrong one).

`tput Ss 6` works.

### Steps to reproduce using `nvim -u NORC`
```
nvim -u NORC
i
```

### Actual behaviour
Cursor remains in the block shape.

### Expected behaviour
Cursor takes the shape of a bar.
Repository: go
Issue: image/gif: "not enough image data" on gif that works in browser
Labels: NeedsInvestigation

There are images which browsers can render but which image/gif chokes on.

The following runs successfully for most gifs:
```
import (
    "fmt"
    "image/gif"
    "os"
)

const example = "sample.gif"

func main() {
    fmt.Println("attempting to read gif")

    file, err := os.Open(example)
    if err != nil {
        fmt.Println("Can't open this file:", err)
        return
    }

    img, err := gif.Decode(file)
    if err != nil {
        fmt.Println("Can't decode this as a gif:", err)
        return
    }

    _ = img.Bounds()

    fmt.Println("gif decoded")
}
```
But when trying to run it on a particular gif (below), this happens:

$ go run main.go
```
attempting to read gif
Can't decode this as a gif: gif: not enough image data
```
Here’s the offending gif
![girl](https://user-images.githubusercontent.com/12746710/81430635-5412ea80-9192-11ea-9a62-80208204bf45.gif)
Repository: flutter
Issue: [web] Can't place breakpoint to debug framework tests
Labels: tool,a: debugging,platform-web,P3,team-web,triaged-web

Repro:

flutter run -d chrome -t test/painting/image_provider_test.dart --start-paused

Able to put breakpoint in web engine but not in the test itself.
Repository: youtube-dl
Issue: Site support request for mixdrop.to
Labels: site-support-request

<!--

######################################################################
  WARNING!
  IGNORING THE FOLLOWING TEMPLATE WILL RESULT IN ISSUE CLOSED AS INCOMPLETE
######################################################################

-->


## Checklist

<!--
Carefully read and work through this check list in order to prevent the most common mistakes and misuse of youtube-dl:
- First of, make sure you are using the latest version of youtube-dl. Run `youtube-dl --version` and ensure your version is 2020.05.08. If it's not, see https://yt-dl.org/update on how to update. Issues with outdated version will be REJECTED.
- Make sure that all provided video/audio/playlist URLs (if any) are alive and playable in a browser.
- Make sure that site you are requesting is not dedicated to copyright infringement, see https://yt-dl.org/copyright-infringement. youtube-dl does not support such sites. In order for site support request to be accepted all provided example URLs should not violate any copyrights.
- Search the bugtracker for similar site support requests: http://yt-dl.org/search-issues. DO NOT post duplicates.
- Finally, put x into all relevant boxes (like this [x])
-->

- [X] I'm reporting a new site support request
- [X] I've verified that I'm running youtube-dl version **2020.05.08**
- [X] I've checked that all provided URLs are alive and playable in a browser
- [X] I've checked that none of provided URLs violate any copyrights
- [X] I've searched the bugtracker for similar site support requests including closed ones


## Example URLs

<!--
Provide all kinds of example URLs support for which should be included. Replace following example URLs by yours.
-->

- Single video: https://mixdrop.to/e/8418pr3et64g38o
- Single video: https://mixdrop.to/e/j9q1pn1wtm9vq3


## Description

<!--
Provide any additional information.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
-->

Add support for it please.
Repository: go
Issue: x/build/cmd/gopherbot: for repos not mirrored to GitHub, close issues listed under "Fixes" when CL is submitted (to an active branch)
Labels: Builders,NeedsFix,FeatureRequest


I recently sent a series of CLs (ending with https://golang.org/cl/232742) to fix a gollvm bug, https://github.com/golang/go/issues/38728.

Gopherbot noticed the CLs as they were put up, adding "mentioned" notices to the issue:

`  Change https://golang.org/cl/232742 mentions this issue: gollvm: add workaround for hard-wired cf-protection=branch
`

however when the CLs were merged, it didn't take any actions. In particular the bot didn't annotate the issue with the merge, and didn't auto-close the bug.

Wondering if there is something I did wrong with the CLs (I used the usual "Updates/Fixes" notation), or if there is something else gumming up the works?

Any pointers appreciated. Thanks.
Repository: pytorch
Issue: test_cpp_warnings_have_python_context_cpu fails under some build configurations
Labels: module: tests,triaged

On my local DEBUG build I have

```
======================================================================
FAIL: test_cpp_warnings_have_python_context_cpu (__main__.TestTorchDeviceTypeCPU)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/data/users/ezyang/pytorch-tmp/torch/testing/_internal/common_device_type.py", line 210, in instantiated_test
    return test(self, device_arg)
  File "test/test_torch.py", line 5943, in test_cpp_warnings_have_python_context
    self.assertTrue(re.search(s, str(warning.message)) is not None)
AssertionError: False is not true
```

This is because two warnings are emitted:

```
[<warnings.WarningMessage object at 0x7f437bebc780>, <warnings.WarningMessage object at 0x7f437c51d780>]
```

The first warning is:

```
pybind11-bound class 'torch.distributed.rpc.RRef' is using an old-style placement-new '__setstate__' which has been deprecated. See the upgrade guide in pybind11's docs. This message is only visible when compiled in debug mode.
```


cc @mruberry
Repository: godot
Issue: Container can't be shrunk past children when clip content is enabled
Labels: discussion,topic:gui

**Godot version:**
3.2.1

**OS/device including version:**
Windows 10 Pro Education 64 bit

**Issue description:**
Containers like VBoxContainer can't be shunk down to clip its content when clip content is enabled

this makes the clip content option useless on containers since there will never be anything to clip

**Steps to reproduce:**
- Add a VBoxContainer
- Add Two buttons to it
- Enable Clip content on the VBoxContainer
- Try to shrink the VBoxContainer to only show 1 button

**Minimal reproduction project:**
[Container Clip Bug.zip](https://github.com/godotengine/godot/files/4600926/Container.Clip.Bug.zip)
Repository: pytorch
Issue: TorchScript to support == None, != None
Labels: oncall: jit,triaged

## 🐛 Bug

We do support `is None`, `is not None` in TorchScript. We should also allow `== None` and `!= None` because unless the comparison operator is overwritten for the left hand side they behave equivalently.

Note that PEP8 recommends `is` version, but both work in python (https://docs.quantifiedcode.com/python-anti-patterns/readability/comparison_to_none.html)

## To Reproduce

```python
import torch
from typing import Optional

@torch.jit.script
def bar(x : Optional[torch.Tensor]):
  if x != None:
    x = torch.tensor([1,2,3])
  return x

bar(None)
```

produces

```
RuntimeError: 
Arguments for call are not valid.
The following variants are available:
  
  aten::ne.Tensor(Tensor self, Tensor other) -> (Tensor):
  Expected a value of type 'Tensor' for argument 'self' but instead found type 'Optional[Tensor]'.
...
```

## Expected behavior

Should have the same behavior as `is None`.

cc @suo