sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: terminal
Issue: Intermittent error saving properties in shortcut link
Labels: Product-Conhost,Issue-Bug,Area-Settings,Priority-3

# Environment

Windows build number: Version 10.0.18362.719
Windows Terminal version (if applicable): Commit b46d39306124363a4104afb74fa9b6657b1d17d0

# Steps to reproduce

1. Checkout and build a recent version of the _OpenConsole_ solution (I'm using an x64 release build in case that makes a difference).
2. Create a shortcut to the `OpenConsole.exe` that has just been built (in the `bin\x64\Release` directory).
3. Start the app from the shortcut.
4. Open the properties dialog.
5. Press <kbd>Enter</kbd> to save and close the dialog.
6. Repeat steps 4 and 5 a few dozen times.

# Expected behavior

You should be able to open and close the dialog without any problems.

# Actual behavior

After a couple of repeats, it'll eventually fail when closing the dialog with the error:

    Unable to modify the shortcut:
    C:\...\OpenConsole.exe - Shortcut.lnk.
    Check to make sure it has not been deleted or renamed.

I added a bunch of `OutputDebugString` logging in the code, and I tracked down the problem to the `s_GetLoadedShellLinkForShortcut` method when called from `s_SetLinkValues`. It tries to load the shortcut with mode `STGM_READWRITE | STGM_SHARE_EXCLUSIVE` and the `IPersistFile::Load` fails with error `0x80070020`, which is the `HRESULT` equivalent of `ERROR_SHARING_VIOLATION`.

This is not always easy to reproduce - sometimes I can repeat the process hundreds of times without an error, but other times it'll fail on the first or second attempt. I don't whether maybe it's just something wrong with my system.
Repository: deno
Issue: Detached processes in deno
Labels: feat

Currently node has the capability to run a child process in detached mode (`setsid()`), which is lacking in deno (?)

It seems the Tokio `Command` doesn't expose any specific functionality for this.
Is it possible for Deno to provide this functionality?

Also, this specific default (kill if handle is dropped from the scope) in rust std and tokio is overridden in Deno (`cli/ops/process.rs`). Is that related to detached mode? 
```rust
fn op_run(...) {
     ....
    // We want to kill child when it's closed
    c.kill_on_drop(true);
}
```


Is there a better place to ask such questions than cluttering the issues section?
Repository: flutter
Issue: Add support for MediaStore to  path_provider
Labels: c: new feature,customer: crowd,p: path_provider,package,c: proposal,team-ecosystem,P3,triaged-ecosystem

## Use case

I want to contribute files of various types (e.g. videos, images, generic files of yet to be defined types to Download) to MediaStore, specifially to MediaStore.downloads, since ExternalStorage seems to be deprecated starting Android SDK 29. 

## Proposal

The path_provider package only returns paths to the app's own paths, so files within these directories are not accessible through Android's file picker or file browser apps.
I'm not sure what the correct proposal is, but adding support to path_provider or flutter itself to contribute items to MediaStore or MediaStore.downloads to make them publically available would be quite important, I think.
(If this is already possible, I'd be very happy about any hints on how to do that :) )
Best regards
Christian
Repository: create-react-app
Issue: Reporter options for Jest
Labels: issue: proposal,needs triage

### Is your proposal related to a problem?

<!--
  Provide a clear and concise description of what the problem is.
  For example, "I'm always frustrated when..."
-->

CRA only supports overriding a few Jest options and `reporters` is not one of them. I know that I can add reporters via cli `--reporters=my-reporter`, the problem is that I would like to pass in some options to that, but reporter options are not available via cli (https://jestjs.io/docs/en/next/cli#--reporters).
 
I really don't want to eject my CRA setup, so i wish it was more flexible when it comes to Jest configurations.

### Describe the solution you'd like

<!--
  Provide a clear and concise description of what you want to happen.
-->

This is what I'd like to be able to do:

```
"reporters": [
  "default",
  ["./node_modules/jest-cucumber/dist/src/reporter", {
    "formatter": "json",
    "path": "./reports/test-report.json"
  }]
],
```

### Describe alternatives you've considered

<!--
  Let us know about other solutions you've tried or researched.
-->

I could use something like [craco](https://www.npmjs.com/package/@craco/craco), but it would be even nicer to have it (and other Jest configs) out of the box in CRA.

### Additional context

<!--
  Is there anything else you can add about the proposal?
  You might want to link to related issues here, if you haven't already.
-->
https://github.com/facebook/create-react-app/issues/2474
https://github.com/facebook/create-react-app/issues/6224
Repository: ant-design
Issue: Unable to demonstrate mobile adaptiveness / friendly capabilities of ant design on ant-design/components/table
Labels: Inactive,📱Mobile Device

- [ ] 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
[https://ant.design/components/table/](https://ant.design/components/table/)

### Steps to reproduce
1. Open https://ant.design/components/table/ in chrome browser, cache cleaned
2. Open dev tools
3. Switch to any phone (I reproduced with Iphone X, Pixel 2, Responsive). Works fine with iPad though or you can fix it switching Iphone X-> Responsive -> Iphone X, but it ruins the demo
4. The menu covers half of the screen even on scroll

### What is expected?
The layout is mobile-friendly (web is a key requirement, but sometimes users want to work from mobile devices)

### What is actually happening?
 The menu sticks to the top from the first opening, there are small other issues with the layout you can probably ignore at the step 1

| Environment | Info |
|---|---|
| antd | 4.2.3 |
| React | - |
| System | macos |
| Browser | Dev tools in Chrome 81.0.4044.138 |


<!-- generated by ant-design-issue-helper. DO NOT REMOVE -->
Repository: PowerToys
Issue: [FZ Editor] Colorful emoiji support for Zone names
Labels: Idea-Enhancement,FancyZones-Editor,Product-FancyZones,External Dependency-WinUI 3

0.17

Color >>> black & white. 
And those monochrome icons are just.....not very good-looking.
It would be really cool, if the win+; icons could show in full color.
Repository: pytorch
Issue: Investigate using -cospi(u) / sinpi(u) instead of tan(pi * (u - 0.5)) in transformation::cauchy
Labels: module: distributions,triaged,module: random

> The following code correctly implements the Cauchy quantile function. Again, for the sake of
> simplicity, it has been assumed that 0 < u < 1, a ∈ R and b > 0. Infinity should
> be returned for u = 0, 1.
> ```
> double cauchy_inv(double u, double a, double b)
> {
> double x;
> x = -cospi(u) / sinpi(u); /* x = tan(pi * (u - 0.5)) */
> return a + b * x;
> }
> ```
> Firstly, the catastrophic cancellation that would occur when a half is subtracted
> from u ≈ 1/2 is dispensed with, and secondly, functions are used that accurately
> compute cos(πx) and sin(πx) for x that are multiples of 1/2.1

as described in [Fast and accurate parallel computation of quantile functions for random number generation](https://discovery.ucl.ac.uk/id/eprint/1482128/1/Luu_thesis.pdf)

@peteroupc 

cc @vincentqb @fritzo @neerajprad @alicanb @vishwakftw @pbelevich
Repository: pytorch
Issue: Investigate exponential distribution improvements
Labels: module: distributions,triaged,module: random

described in [Reconditioning your quantile function by Keith Pedersen](https://arxiv.org/abs/1704.07949)

cc @vincentqb @fritzo @neerajprad @alicanb @vishwakftw @pbelevich @peteroupc
Repository: pytorch
Issue: Investigate using log1p instead of log in transformation functions(TransformationHelper.h)
Labels: module: distributions,triaged,module: random

Investigate performance and precision advantages of log1p

cc @vincentqb @fritzo @neerajprad @alicanb @vishwakftw @pbelevich @peteroupc
Repository: flutter
Issue: [google_maps_flutter] Get duration of animateCamera
Labels: c: new feature,p: maps,package,team-ecosystem,P3,triaged-ecosystem

Is there any way to know the duration of the animation created by`animateCamera()`? Since the Future completes at the start of the animation and not at the end (I don't understand why), if I want to perform some actions based on the new camera position I have to use a `Timer()` but so far I am using an arbitrary `Duration()` of 500ms to be sure that the animation is done.
Repository: opencv
Issue: Why the efficientnet no support OPENCL of Intel CPU？
Labels: optimization,priority: low,category: ocl,category: dnn

- OpenCV => 4.3
- Operating System / Platform => Win10 64 Bit
- Compiler => Visual Studio 2017
   -->

- OpenCV => :The efficientnet no support OPENCL of Intel CPU:
   -->
    ex::
    net.setPreferableTarget(cv::dnn::DNN_TARGET_OPENCL);
    What the tips means? (the efficientnet forward Inference is so slow)
    https://s1.ax1x.com/2020/05/17/YgrUts.jpg 

    BTW, The OPENCL also no support FP16（Intel CPU：Z8350 or N3150）?

    When The OPENCL adding FP8 support  for faster forward Inference ?

   Thanks.
   -->
Repository: flutter
Issue: [flutter pub outdated] doesn't expose the arguments of the equivalent pub command
Labels: c: new feature,tool,c: proposal,P3,team-tool,triaged-tool

The command `flutter pub outdated` has been added to the Flutter SDK with #53251, however it doesn't expose all the arguments of the original pub command:

```console
> flutter pub outdated -h
Analyze dependencies to find which ones can be upgraded.
This runs the "pub" tool in a Flutter context.

Usage: flutter pub outdated [<arguments...>]
-h, --help    Print this usage information.

Run "flutter help" to see global options.
```
```console
> pub outdated -h
Analyze your dependencies to find which ones can be upgraded.

Usage: pub outdated [options]
-h, --help                         Print this usage information.
    --[no-]color                   Whether to color the output. Defaults to color when connected to a terminal, and no-color otherwise.
    --json                         Outputs the results in a json formatted report
    --[no-]up-to-date              Include dependencies that are already at the latest version.
    --[no-]pre-releases            Include pre-releases when reporting latest version.
    --[no-]dev-dependencies        When true take dev-dependencies into account when resolving.
                                   (defaults to on)
    --[no-]dependency-overrides    Show resolutions with `dependency_override`
                                   (defaults to on)

Run "pub help" to see global options.
See https://dart.dev/tools/pub/cmd/pub-outdated for detailed documentation.
```

Example:

```console
> flutter pub outdated --json
Could not find an option named "json".


Run 'flutter -h' (or 'flutter <command> -h') for available flutter commands and options.
```

The problem here is that `pub outdated` cannot be run for Flutter projects:

```console
> pub outdated
Resolving...
Package doesn't exist (the Flutter SDK is not available).
```
Repository: pytorch
Issue: Can you add NMS,RoIAlign,RoIPool for libtorch?
Labels: triaged,module: vision

https://pytorch.org/docs/master/torchvision/ops.html

Can you add NMS,RoIAlign,RoIPool for libtorch?


cc @fmassa
Repository: vscode
Issue: Support Language, not just Editor Language in the keybindings "when" property.
Labels: feature-request,context-keys

<!-- ⚠️⚠️ 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. -->
Support Language, not just Editor Language in keybindings.json.

For example:

    {
        "key": "shift+alt+f",
        "command": "extension.formatSelectionAsHtml",
        "when": "editorTextFocus && editorHasSelection && langId == vue-html"
    },
Repository: flutter
Issue: Adding onChanged method in SearchDelegate class
Labels: c: new feature,framework,f: material design,c: proposal,team-design,triaged-design

<!-- 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
Adding an onChanged method for query in search delegate abstract class will be so much handy I guess. If I already have the list to be shown in the build results section then why should i use streams and stream builder as they are pretty heavy widgets to be used. Adding on query changed method can help us filter list and show in result as soon as query changes.
<!--
     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?
-->

## Proposal
Is there any workaround for what I am trying to point out above? 
Or flutter should give access to the textfield just like textformfield so that onchanged method can be used to show results.

<!--
     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.
-->
Repository: pytorch
Issue: Clarification for usage of negative loss with optim.lr_scheduler.ReduceLROnPlateau
Labels: module: docs,triaged

## 📚 Documentation

<!-- A clear and concise description of what content in https://pytorch.org/docs is an issue. If this has to do with the general https://pytorch.org website, please file an issue at https://github.com/pytorch/pytorch.github.io/issues/new/choose instead. If this has to do with https://pytorch.org/tutorials, please file an issue at https://github.com/pytorch/tutorials/issues/new -->

It would be nice to have explicit statement in the documentation that `torch.optim.lr_scheduler.ReduceLROnPlateau` with `mode=min` and `threshold_mode='rel'` expects loss to be `> 0`

This is the excerpt from documentation about `threshold_mode`:
```
threshold_mode (str): One of `rel`, `abs`. In `rel` mode,
dynamic_threshold = best * ( 1 + threshold ) in 'max'
mode or best * ( 1 - threshold ) in `min` mode.
In `abs` mode, dynamic_threshold = best + threshold in
`max` mode or best - threshold in `min` mode. Default: 'rel'.
```

And this is the corresponding excerpt from the code:
```python
def is_better(self, a, best):
    if self.mode == 'min' and self.threshold_mode == 'rel':
        rel_epsilon = 1. - self.threshold
        return a < best * rel_epsilon

    elif self.mode == 'min' and self.threshold_mode == 'abs':
        return a < best - self.threshold

    elif self.mode == 'max' and self.threshold_mode == 'rel':
        rel_epsilon = self.threshold + 1.
        return a > best * rel_epsilon

    else:  # mode == 'max' and epsilon_mode == 'abs':
        return a > best + self.threshold
```

I use `NegativeDiceLoss` in my experiments so the loss is negative and is expected to be minimized. The only correct mode of `ReduceLROnPlateau` I can use is `threshold_mode='abs'` because `threshold_mode='rel'` will not work for minimizing negative loss.

Imagine `mode='min', threshold_mode='rel', threshold=0.1`:
* If we use loss that is expected to be > 0 all the time and current loss value is `10` than dynamic threshold will be `(1 - 0.1) * 10 == 9`. And the check will be `a < 9`. So there will be a margin between previous loss value and the new one to ensure that model improved.
* If we use loss that is expected to be < 0 all the time (as in my case with `NegativeDiceLoss`) and current loss value is `-10` than dynamic threshold will be `(1 - 0.1) * -10 == -9`. And the check will be `a < -9`. __It does not correspond to logic of scheduler__ as it allows new loss value to be slightly worse than previous. As I understand the goal of the dynamic threshold is to guarantee that the loss improved __by some margin__ to deal with noise in loss calculations.

If my understanding is correct it is better to __explicitly state in documentation__ that for `threshold_mode='rel'` only positive loss functions are expexted. Probably negative loss functions are not common to use but sometimes they are required (as with dice score as loss function).
Please correct me if I'm wrong :)
Repository: rust
Issue: Splitting borrows through smart pointers has confusing diagnostics
Labels: C-enhancement,A-diagnostics,A-borrow-checker,T-compiler,D-confusing

```rust
use std::cell::RefCell;

#[derive(Default)]
struct NotCopy;

#[derive(Default)]
struct Split {
    one: Option<NotCopy>,
    two: NotCopy,
}

fn read(_: &NotCopy) {}

fn main() {
    let mut split = Split::default();
    
    // comment this:
    let split = &mut split;
    
    // uncomment this:
    // let split = RefCell::new(split);
    // let mut split = split.borrow_mut();

    if let Some(ref mut one) = split.one {
        read(&split.two);
        *one = NotCopy;
    }
}
```

([Playground](https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=f42ceb391fdaa9aad25aa7a387a67c88))


This compiles fine, but if you switch the commented out parts, you get:

```
error[E0502]: cannot borrow `split` as immutable because it is also borrowed as mutable
  --> src/main.rs:21:15
   |
20 |     if let Some(ref mut one) = split.one {
   |                                ----- mutable borrow occurs here
21 |         read(&split.two);
   |               ^^^^^ immutable borrow occurs here
22 |         *one = NotCopy;
   |         -------------- mutable borrow later used here
```

The reason this does not error in the original case is that Rust is aware of the semantics of `&mut T`'s `DerefMut` and can reason about splitting borrows, however it is _not_ aware of the semantics of `RefMut<T>`'s `DerefMut` and `&split.two` is able to read from `split.one` in a safely-written `DerefMut` impl.

The fix is to convert the `RefMut<T>` into an `&mut T`, via `&mut *split` before doing any of reads or writes.

One typically expects splitting mutable borrows to work in Rust, so it's weird when it doesn't. It would be nice if the compiler could detect when a situation is one of splitting borrows through a DerefMut impl and make the appropriate suggestion.
Repository: pytorch
Issue: Wheels not manylinux1 compliant
Labels: module: binaries,triaged

The current 1.5.0 manylinux1 release of torch links against `libcuda.so.1` which is against the specification.

According to the [manylinux1 policy](https://www.python.org/dev/peps/pep-0513/#the-manylinux1-policy) from PEP-513, manylinux1 wheels must only link against a limited set of external shared libraries.

Pytorch not following the spec leads to problems on build systems which rely on the correctness of the manylinux1 policy like for example installing wheels via Nix.

cc @ezyang @seemethere
Repository: excalidraw
Issue: Free draw with many points fails to filp
Labels: bug

It's probably because some of the point positions become too close to zero, and can't rescale properly.

![exdraw44](https://user-images.githubusercontent.com/490574/82153534-fc1b6880-98a2-11ea-8719-bf8c4542bf49.gif)
Repository: godot
Issue: Root viewport texture filtering only works in GLES2
Labels: enhancement,topic:rendering

**Godot version:** 3.2.stable

**OS/device including version:** Windows 10

**Issue description:** Enabling texture filtering on the root viewport is the only way for a consistent visual result with proper font hinting that works equally well with fullscreen/windowed mode. For this I'm using an AutoLoad (by the way I truly believe this feature should be available without scripting), but it's only working with GLES2. Switching to GLES3 the root viewport texture filtering is ignored. The flag is set, and no error or warning is shown, but the rendered texture is ugly and pixelated.

**Steps to reproduce:** Clone and open the included demo project. Will start in GLES2. Toggle the "Viewport filtering" CheckButton to see that when it's enabled the viewport texture is filtered. Now switch `rendering/quality/driver/driver_name` to GLES3 and observe that the filtering is no longer working.

**Minimal reproduction project:** https://github.com/sszigeti/GodotViewportFilteringDemo

[Further details on Reddit](https://www.reddit.com/r/godot/comments/fsw3n0/solved_i_finally_have_consistent_font_hinting_due/)
Repository: godot
Issue: Items remain highlighted or tooltips remain up when cursor leaves window
Labels: bug,topic:editor,topic:gui

**Godot version:**
3.2.1 stable

**OS/device including version:**
Fedora 32, x86_64

**Issue description:**
If I alt-tab to another app or move my mouse quickly out of the app window then any tooltip or highlighted control will remain in that state.

**Steps to reproduce:**
The best way to show this issue is with a video.

https://youtu.be/5WmqQeSy2SY

**Minimal reproduction project:**
You can easily reproduce the issue using the Godot editor.
Repository: opencv
Issue: Sample Python code not included in "Random generator and text with OpenCV" tutorial
Labels: category: documentation

The [Random generator and text with OpenCV](https://github.com/opencv/opencv/blob/master/doc/tutorials/imgproc/random_generator_and_text/random_generator_and_text.markdown) tutorial only contains C++ code but the program has already been translated to Python and is contained in the OpenCV samples [here](https://github.com/opencv/opencv/blob/master/samples/python/drawing.py). (Unfortunately I only noticed this about halfway through creating a translated version of my own!) On a related note, the tutorial also uses in-document code, instead of using references to the actual sample C++ file, and the link to the [C++ sample](https://github.com/opencv/opencv/blob/master/samples/cpp/tutorial_code/ImgProc/basic_drawing/Drawing_2.cpp) is broken. If so desired, I would be happy to submit a pull-request for:

- [ ] fixing the broken link to the C++ sample file
- [ ] revising the tutorial to use references to the C++ sample file
- [ ] relocating the Python sample file to the Python tutorial code directory
- [ ] referring to the Python sample file in the tutorial (with toggles for switching between the languages)