sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: flutter
Issue: WebViewPlatform calls implementation's build more than once with the same set of arguments.  Also, there's no dispose to clean up resources of a web view.
Labels: customer: fuchsia,p: webview,package,team-ecosystem,P2,triaged-ecosystem

We're seeing on Fuchsia that using WebViewPlatform requires us to make cls like this to both allow us to dispose of the webview:
https://fuchsia-review.googlesource.com/c/topaz/+/301291
and to prevent us from calling the webview created callback multiple times:
https://fuchsia-review.googlesource.com/c/topaz/+/302155

While somewhat easy for an implementation to work around, I believe based on the API description an implementor of WebViewPlatform should expect 'build' to be called only once with the same parameters.  It should be up to flutter web view's own state to ensure it hangs on to created webviews instead of calling build with the same parameters again.

I believe some example code that may trigger the build to be called multple times with same parameters is something like this:

``` dart
ValueNotifier<bool> toggle = ValueNotifer<bool>(false);

Timer.repeated(Duration(milliseconds:100),(_){toggle.value = !toggle.value;});

build(BuildContext context) {
return ValueListenableBuilder(valueListenable:toggle, builder:(_,toggleValue,__)=> toggleValue ? WebView(...) : Container(child:WebView(...));
}
```

We're putting undue burden on the implementations of WebViewPlatform on knowing when the same webview should be returned or a different one.
Repository: storybook
Issue: Addon-docs: Use MDX shortcodes
Labels: feature request,addon: docs,mdx

MDX 1.0 introduced "shortcodes" which allow you to [use certain elements without needing to import them explicitly](https://mdxjs.com/blog/shortcodes).

Should we use this for `<Meta>`, `<Story>`, `<Preview>`, `<Source>`, and `<Props>` so you can use them in `*.stories.mdx` without having to import them every time?

- [ ] Update compiler
- [ ] Update codemods
- [ ] Update docs

Pros: much more convenient
Cons: makes the MDX less portable / discoverable
Repository: rust
Issue: Document that str, slices, (more?) can't safely straddle allocation boundaries
Labels: C-enhancement,T-lang,A-docs

Slicing, indexing, and other safe operations on slices and strings pervasively use `<*T>::offset` and APIs built on top of it. These have the requirement that

> Both the starting and resulting pointer must be either in bounds or one byte past the end of the same allocated object.

So if one allocates two pieces of memory and after proper checking miraculously finds they are directly adjacent, one can't safely construct a slice/str/etc. that spans both of these allocations. At least, one can't do very many things with the result it without causing UB from crossing the boundary between the allocations.

I couldn't find anything documenting this. It should be noted on the unsafe constructors (`from_raw_parts` etc.) at minimum. These already link to `offset`'s documentation but only refer to its "no larger than isize::MAX" requirement, with no mention that the other requirements are also relevant. 

cc https://github.com/oberien/str-concat/issues/8
cc @rust-lang/wg-unsafe-code-guidelines 

(Similar issues apply to references-to-arrays and field accesses in aggregates, but this is due to the compiler's codegen for language primitives rather than due to standard library code, so it should go into the UCG and I believe we're more or less covering that already.)
Repository: neovim
Issue: tests: flaky(?): :edit term://* runs TermOpen early enough to set buffer-local 'scrollback'
Labels: test

This was seen on CI:
```
[  ERROR   ] 1 error, listed below:
[  ERROR   ] ...neovim/neovim/test/functional/terminal/edit_spec.lua @ 34: :edit term://* runs TermOpen early enough to set buffer-local 'scrollback'
test/functional/ui/screen.lua:567: Row 1 did not match.
Expected:
  |*96: foobar          |
  |                    |
  |^[Process exited 0]  |
  |                    |
Actual:
  |*26: foobar          |
  |                    |
  |^[Process exited 0]  |
  |                    |
```

I've thought this might just be flaky, but it is odd that there is "Process exited 0" already, but not the whole output is visible.

Code/Test: https://github.com/neovim/neovim/blob/353b3852fd03168585d868c5c7580e2f0599cb19/test/functional/terminal/edit_spec.lua#L34-L67

_Originally posted by @blueyed in https://github.com/neovim/neovim/pull/10528#issuecomment-512563901_
Repository: TypeScript
Issue: Using && with a string produces union type with empty string
Labels: Bug

<!-- Please try to reproduce the issue with `typescript@next`. It may have already been fixed. -->
The below code is a super simplified version of a real world scenario I just ran into. When trying to assign a value only if some other condition is true and otherwise returning `undefined`, TypeScript produces a strange union type that includes empty string `''`.
The linked Playground contains a slightly larger example that gets closer to demonstrating my real world use case.

I would argue that in cases where the compiler can clearly know that something is truthy, it should evaluate to the type of the value on the right hand side of `&&`, even with `strictNullChecks` off.

**TypeScript Version:**  3.5.1 (With `strictNullChecks` off)

<!-- Search terms you tried before logging this (so others can find this issue more easily) -->
**Search Terms:** `logical and empty string`, `string && empty`

**Code**

```ts
const x = true && 'a'
```

**Expected behavior:**
Type of `x` is `'a'`

**Actual behavior:**
Type of `x` is `'' | 'a'`

**Playground Link:** <!-- A link to a TypeScript Playground "Share" link which demonstrates this behavior -->
https://www.typescriptlang.org/play/?strictNullChecks=false#code/C4TwDgpgBAysBOBLAdgcwKrMQe2VAvFAOQCGRUAPsQEZECwAUIwGYCuyAxsDnqhMHFbNmACgCUALlgIUGLLkpR2AEwjMUEZVADejKPqgdcAZ2BQANthKqthBKwgBuPQfj9W8PAFkSwABYAdPAkyMrYALbiUAD8FlY2UABkicRkUFIqahrKzgwAvkA

**Related Issues:** <!-- Did you find other bugs that looked similar? -->
Couldn't find any 🕵🏼‍♂️
Repository: rust
Issue: "Foo is ambiguous" only when deriving Debug
Labels: A-resolve,T-compiler,C-bug

[playpen](https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=903c3c6207c05d529bd58cc739350d65)

This may be related to #62768.

Given this code:
```rust
pub use Foo::*;
#[derive(Debug)]
pub enum Foo {
    Foo(i32),
}
```
The compiler errors, saying that the `Foo` in `use Foo::*` may refer both to the enum (type-namespace) `Foo` and the yet-to-be-imported variant (value-namespace) `Foo::Foo`:

```rust
error[E0659]: `Foo` is ambiguous (glob import vs macro-expanded name in the same module during import/macro resolution)
 --> src/lib.rs:1:9
  |
1 | pub use Foo::*;
  |         ^^^ ambiguous name
  |
note: `Foo` could refer to the enum defined here
 --> src/lib.rs:3:1
  |
3 | / pub enum Foo {
4 | |     Foo(i32),
5 | | }
  | |_^
note: `Foo` could also refer to the variant imported here
 --> src/lib.rs:1:9
  |
1 | pub use Foo::*;
  |         ^^^^^^
  = help: consider adding an explicit import of `Foo` to disambiguate
```

However, if I remove the `#[derive(Debug)]`, then the code compiles just fine as expected ([playpen](https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=a80dbcca330ff235b52204d29358261c)).
Repository: flutter
Issue: Expose attributes of widgets to Flutter Driver
Labels: c: new feature,tool,t: flutter driver,P3,team-tool,triaged-tool

## Use case
When retrieving a SerializableFinder, it would be helpful if we could find out if a button is enabled/disabled by accessing the button's attributes.
There are software requirements for example where a button can only be enabled for clicking if text fields are filled out.

## Proposal
Add a function in FlutterDriver or SerializableFinder where we can retrieve the attributes in a form of map or other data structure you think is better for this. We see these attributes in Flutter Inspector and it would be help us in our tests if we could make use of these attributes.
Repository: scrcpy
Issue: Lens correction
Labels: feature request

Is there a way to apply lens correction for streams coming from Oculus Quest / Go?  With a recorded MP4 video file I can use ffmpeg like this to fix up lens distortion:

`ffmpeg -i input.mp4 -vf 'lenscorrection=k2=0.1:k1=-0.4' -r 24 output.mp4`

Is it possible for the output of scrcpy to be corrected in realtime?
Repository: godot
Issue: [.WebGL-053FF6D0]RENDER WARNING: there is no texture bound to the unit 30
Labels: bug,platform:web,topic:rendering,confirmed,topic:3d

___

***Bugsquad note:** This issue has been confirmed several times already. No need to confirm it further.*

___

**Godot version:** 3.1.1
<!-- Specify commit hash if non-official. -->


**OS/device including version:** Windows 10 - Chrome
<!-- Specify GPU model and drivers if graphics-related. -->
Firefox doesn't suffer from this.

**Issue description:**
<!-- What happened, and what was expected. -->
Everytime I launch any project in Chrome, I get 254 of the error in the title and in some case it's `unit 31` instead of `unit 30` until it stops with
```
WebGL: too many errors, no more errors will be reported to the console for this context.
```

It's enough to slow down my 3D project with physics running right from the start. Although a 2D project would still get this error.

_Workaround_: Your game should have a menu scene so when the player got through it, WebGL has already stopped reporting 254 errors.

**Steps to reproduce:** Export to HTML5 and run it on Chrome.

**Minimal reproduction project:** An empty project would suffice.
<!-- Recommended as it greatly speeds up debugging. Drag and drop a zip archive to upload it. -->
Repository: flutter
Issue: flutter drive --trace-skia
Labels: c: new feature,tool,c: performance,t: flutter driver,P3,team-tool

when we flutter run an app can add --trace-skia to get timeline with skia functon.
But flutter drive can not add --trace-skia, so can not get time line with skia funcion.
Repository: kubernetes
Issue: Migrate all uses of leader-election to use Lease API
Labels: kind/cleanup,sig/scalability,help wanted,priority/important-longterm,lifecycle/frozen

Currently, Kubernetes components (scheduler, kcm, ...) are using leader election that is based on either Endpoints or ConfigMap objects.
Given that both of these are watched by different components, this is generating a lot of unnecessary load.

We should migrate all leader-election to use Lease API (that was designed exactly for this case).

The tricky part is that in order to do that safely, I think the only reasonable way of doing that would be to:
- in the first phase switch components to:
  1. acquire lock on the current object (endpoints or configmap)
  2. acquire lock on the new lease object
  3. only the proceed with its regular functionality
  [ loosing any of those two, should result in panicing and restarting the component]
- in the second phase (release after) remove point 1

@kubernetes/sig-scalability-bugs
Repository: flutter
Issue: DropdownMenuItem RTL isn't working
Labels: framework,f: material design,a: internationalization,has reproducible steps,P2,found in release: 3.3,found in release: 3.7,team-design,triaged-design

in this sample code i can't set right to left texts on dropdown menus

```dart
  List<DropdownMenuItem<SessionsEntity>> buildDropdownMenuItems(List sessions) {
    List<DropdownMenuItem<SessionsEntity>> items = List();
    for (SessionsEntity session in sessions) {
      items.add(
        DropdownMenuItem(
          value: session,
          child: Directionality(
              textDirection: TextDirection.rtl,
              child: Text(session.sessionName,
                  textAlign: TextAlign.right,
                  textDirection: TextDirection.rtl,
                  style: Theme.of(context).textTheme.caption.copyWith(
                  color: Colors.black,
                  fontFamily: 'ShabnamLight'
              ))),
        ),
      );
    }
    return items;
  }

```
all of items are LEFT to RIGHT
Repository: flutter
Issue: Different Appbar with same bottomNavigatioBar without nesting Scaffolds
Labels: c: new feature,framework,f: material design,c: proposal,P3,team-design,triaged-design

If you want to have a bottom navigation bar(CircularNotchedRectangle) with a nice floatingActionButton on all your app sections(pages) but different appBar the only solution is to use nested Scaffolds. 
Based on some observations and comments (https://github.com/flutter/flutter/issues/23106#issue-370255150) nesting scaffolds is not so good strategy.

The comment (https://github.com/flutter/flutter/issues/23106#issuecomment-452689090) didn't get the attention that deserved. 
Here is my situation, I have Scaffold without appBar with bottom navigation bar and the body is a TabController.

Main Page(HomePage)
```
Scaffold(
      floatingActionButtonLocation: FloatingActionButtonLocation.centerDocked,
      floatingActionButton: FloatingActionButton(...),
      bottomNavigationBar: BottomAppBar(...),
      body: TabBarView(
        physics: NeverScrollableScrollPhysics(),
        controller: _tabController,
        children: _tabList,
      ),
```
Every section use (fancy appBar if needed):
```
@override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text("Settings"),
      ),
      body:....
```
I think in iOS the bottom navigationBar is a controller of its own. On Android, BottomAppBar in MainActivity (???). Maybe Scaffold is doing too many things. 
A widget that wraps bottomNavigationBar should be something usefull and from there you cand have independent Scaffolds.
Repository: vscode
Issue: [css][html] Publish language servers modules on npm
Labels: feature-request,css-less-scss

We'd like to reuse more easily language servers from VSCode (HTML, CSS, JSon...), On npm.js we can see the related "languageservice" which provide some part of the logic, but those don't talk LSP.
Could the language server modules be published on npm.js for easier reuse? Or merged with the languageservice modules directly?
Repository: pytorch
Issue: BatchNorm1d fails on first run through GPU
Labels: module: nn,module: cuda,triaged

## 🐛 Bug

BatchNorm1D results in
```
RuntimeError: cuDNN error: CUDNN_STATUS_EXECUTION_FAILED
```
on first run through. Afterwords it works as intended. Note that this does not happen on CPU.

## To Reproduce

```
import torch
a = torch.rand((2, 32, 200), requires_grad=True).cuda()
batchnorm = torch.nn.BatchNorm1d(32).cuda()
batchnorm(a)
```
At which point the error will occur. Running
```
batchnorm(a)
```
again outputs a tensor, resulting in desired behavior.

## Expected behavior

BatchNorm1d working the first time around.

## Environment
```
PyTorch version: 1.1.0
Is debug build: No
CUDA used to build PyTorch: 9.0.176

OS: Ubuntu 18.04.2 LTS
GCC version: (Ubuntu 7.4.0-1ubuntu1~18.04.1) 7.4.0
CMake version: version 3.11.4

Python version: 3.7
Is CUDA available: Yes
CUDA runtime version: Could not collect
GPU models and configuration: GPU 0: GeForce RTX 2070
Nvidia driver version: 410.93
cuDNN version: /usr/local/cuda-9.0/lib64/libcudnn.so.7.0.5

Versions of relevant libraries:
[pip3] msgpack-numpy==0.4.3.1
[pip3] numpy==1.16.4
[conda] torch                     1.1.0                    pypi_0    pypi
[conda] torchvision               0.3.0                    pypi_0    pypi
```
Repository: go
Issue: os/signal: TestNohup flaky
Labels: Testing,NeedsInvestigation,compiler/runtime

Seen on the `solaris-amd64-oraclerel` builder in https://build.golang.org/log/d5185d101eddc0f5a05103f11b74caf24421bf46:
```
--- FAIL: TestNohup (1.25s)
    signal_test.go:329: ran test with -send_uncaught_sighup=1 and it succeeded: expected failure.
        Output:
        PASS
FAIL
FAIL	os/signal	3.575s
```

See previously #8682, #5526.

CC @ianlancetaylor
Repository: bitcoin
Issue: Ensure we have sufficient transaction-relay peers
Labels: Brainstorming,P2P

Currently, we have no protections in place if our outbound peers are running in `-blocksonly` mode.  In practice, we rely on our outbound peers to be our best source of announced transactions, but if all our outbound peers were `-blocksonly`, we would make no effort to disconnect any to find an alternative peer to receive transactions from.

We discover at connection time (in the VERSION message) whether our peer will relay transactions on our link.  Perhaps we should tolerate some number of `-blocksonly` peers as outbounds, and once we hit that threshold we should disconnect new blocksonly outbounds?

I suppose another issue could be if it's worth having logic that tries to determine whether a peer is silently withholding transactions from us...  We have logic for blocks, where we try to find a new outbound peer if we haven't received a new block in a while, but I don't know if there's a reasonable way to do this for transactions.  My guess is that this is not a very pressing concern, but perhaps we could do something that just tried making new outbound connections if we aren't receiving any transactions at all?
Repository: flutter
Issue: [tool]`--wrap-column` flag has no effect
Labels: tool,a: quality,has reproducible steps,P2,found in release: 3.0,found in release: 3.1,team-tool,triaged-tool

Related: https://github.com/flutter/flutter/issues/23074.

In the following, there is no wrapping of the option description.

```
> flutter help --wrap-column=80 --verbose | grep column
    --wrap-column              Sets the output wrap column. If not set, uses the width of the terminal. No wrapping occurs if not writing to a terminal. Use --no-wrap to turn off wrapping when connected to a terminal.
```

I was expecting option descriptions to be wrapped nicely to the specified column. So something like the following.

```
    --wrap-column              Sets the output wrap column. If not set, uses 
                               the width of the terminal. No wrapping occurs 
                               if not writing to a terminal. Use --no-wrap to 
                               turn off wrapping when connected to a terminal.
```

Nor does `flutter help --wrap --wrap-column=80 --verbose | less` work.

My environment is as follows.

```
> flutter --version
Flutter 1.7.8+hotfix.3 • channel stable • https://github.com/flutter/flutter.git
Framework • revision b712a172f9 (9 days ago) • 2019-07-09 13:14:38 -0700
Engine • revision 54ad777fd2
Tools • Dart 2.4.0
```

I am using bash in Terminal.app.
Repository: pytorch
Issue: Add automatic tuning flags to utils.data.dataloader
Labels: feature,module: dataloader,low priority,triaged

## 🚀 Feature
Adding automatic tuning flags for `batch_size` and `num_workers` to `torch.utils.data.dataloader`

## Motivation

We'd like to help users to get good performance (throughput) in their data loading jobs. This will allow users to focus on defining the logic for the model inference.

Today, users have to tune batch_size and num_workers manually when they hit performance issues. Tuning requires some expert knowledge of PyTorch and systems design. Even with this knowledge, users may spend hours or days benchmarking, profiling, identifying bottlenecks, and tweaking parameters to improve performance.

This feature is especially relevant to IO bound pipelines, commonly found in inference workloads, where images are not cached in memory and must be loaded from disk. Optimal `batch_size` and `num_workers` configs can improve an imgs/sec metric by as much as 10x.

## Pitch

Just as Tensorflow has introduced the `tf.data.experimental.AUTOTUNE` feature for `batch_size`, `prefetch_size`, and `num_parallel_calls` in datasets, PyTorch should include a similar feature for torch.utils.data.dataloader.

## Alternatives

If autotuning flags are not introduced, users can guess at the optimal configs, or hyperparameter tune these configs before running full training / inference jobs on big data.

In practice, setting `batch_size` to be the largest possible without running into OOM errors works well, as does setting `num_workers` equal to the number of cores on the machine. However, these configs are not always optimal, and may not take advantage of features like hyperthreading.

Hyperparameter tuning can also work to discover optimal parameters, but lacks information about buffer sizes and has to deal with more noise and variance in imgs/sec result metrics. Additionally, extensive engineering is required to enable online tuning; otherwise, valid inference results may be discarded. 

## Additional context

My Questions
* Are there any features like this in the PyTorch Pipeline?
* Would this be useful to other folks in the contributor or user community of PyTorch?
* Has anyone thought about autotuning before and/or have ideas/stubs of implementation?
Repository: react
Issue: onBeforeInput fires after browser updates the DOM for special characters like "中" or 😣on Firefox and Edge
Labels: Component: DOM,Type: Needs Investigation

<!--
  Note: if the issue is about documentation or the website, please file it at:
  https://github.com/reactjs/reactjs.org/issues/new
-->

**Do you want to request a *feature* or report a *bug*?**
Bug

**What is the current behavior?**
onBeforeInput fires after browser updates the DOM for special characters like "中" or 😣on Firefox and Edge
https://codesandbox.io/s/modest-franklin-muirj  
NOTE: pasting it would not trigger the bug, you have to type it in. You can use control-command-space to open the emoji keyboard on mac

**What is the expected behavior?**
DOM should not update before onBeforeInput fires

**Which versions of React, and which browser / OS are affected by this issue? Did this work in previous versions of React?**
React: 16.3.1
Firefox: 68.0.1 
Mac: 10.14.5
Repository: pytorch
Issue: ConcatDataset returns different error messages setting out of range plus index and minus index.
Labels: module: docs,low priority,triaged

## 🐛 Bug

ConcatDataset returns different error messages setting out of range plus index and minus index. 

## To Reproduce
```
>>> import torch
>>> from torch.utils import data
>>> x=data.ConcatDataset((range(10),range(10)))
>>> x[-100]
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/mueda/code/python/rc/doc/.venv/lib/python3.7/site-packages/torch/utils/data/dataset.py", line 78, in __getitem__
    raise ValueError("absolute value of index should not exceed dataset length")
ValueError: absolute value of index should not exceed dataset length
>>> x[100]
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/mueda/code/python/rc/doc/.venv/lib/python3.7/site-packages/torch/utils/data/dataset.py", line 85, in __getitem__
    return self.datasets[dataset_idx][sample_idx]
IndexError: list index out of range
```
## Expected behavior
I think it's better that x[100] and x[-100] have the same error message.

My way to solve this is to chage [these lines](https://github.com/pytorch/pytorch/blob/master/torch/utils/data/dataset.py#L197-L200) like this.
```
        if idx < 0:
            idx = len(self) + idx
        if not (0 <= idx < len(self)):
            raise ValueError("absolute value of index should not exceed dataset length")
```

## Environment
 - PyTorch Version (e.g., 1.0): 1.1.0
 - OS (e.g., Linux): Ubuntu 18.04
 - How you installed PyTorch (`conda`, `pip`, source): pip
 - Python version: Python 3.7.3
 - CUDA/cuDNN version: None
 - GPU models and configuration: None