sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: kubernetes
Issue: PVC/PV conformance testing discussion
Labels: priority/backlog,sig/storage,kind/feature,sig/architecture,area/conformance,lifecycle/frozen

<!-- This form is for bug reports and feature requests ONLY!

If you're looking for help check [Stack Overflow](https://stackoverflow.com/questions/tagged/kubernetes) and the [troubleshooting guide](https://kubernetes.io/docs/tasks/debug-application-cluster/troubleshooting/).

If the matter is security related, please disclose it privately via https://kubernetes.io/security/.
-->

**Is this a BUG REPORT or FEATURE REQUEST?**:
@kubernetes/sig-storage-feature-requests 

As a follow up to the conformance testing discussion in https://github.com/kubernetes/features/issues/498, we had a meeting to brainstorm ideas around conformance testing of PVCs/PVs.  Some ideas we came up with:

* The core controller paths with PV controller (both static and dynamic provisioning), attach/detach, kubelet volume manager mount/unmount can be part of a core Kubernetes suite using a mock CSI plugin.  This would only test the Kubernetes control path and the PVC/PV lifecycle management.  Even something like default StorageClasses could be tested if we made the mock CSI driver a default StorageClass (and disabled the provider-specific default StorageClass if preinstalled)
* Data path tests (ie reading/writing), data persistence during disruption/errors (rescheduling pods on other nodes) requires a real volume plugin implementation and is a good candidate for a "persistent volumes profile" conformance suite.  The behavior here is very volume-plugin dependent, and not all providers may have a "default" volume plugin that they include.  So this may actually be more of a volume plugin conformance suite, instead of a provider suite, where each volume plugin can be certified to meet some standard behavior (ie attach/detach handling in error conditions, read only behavior, permission handling, etc).  Providers could potentially list which persistent volume plugins they include out of the box.

Some action items/questions:
* [sig-architecture] Is having a core conformance suite that tests the control path for volume handling with a mock plugin that doesn't actually run a real user experience useful?  It would indicate that your provider at least runs all the necessary controllers.
* [sig-architecture] Is having a profile conformance suite that requires providers to include support for at least one volume plugin out of the box useful?  This would map to user experience much more than the former.
* [sig-storage] For sure we should have a CSI plugin conformance standard (that is being handled separately outside of Kubernetes).  What about in-tree plugins?
* [@msau42] Once the above questions are answered, come up with a plan for:
    * what tests can go in each suite
    * what work needs to be done to modify existing tests, add new tests to the conformance suites
    * multi-release plan for adding those tests

cc @WilliamDenniss @bgrant0607 @AishSundar @pospispa @saad-ali
Repository: pytorch
Issue: Update tests to no longer spew debug info
Labels: module: tests,triaged,better-engineering

See recent CI, which spews the following:

23:49:53 test_anomaly_detect_nan (__main__.TestAutograd) ... No forward pass information available.
23:49:53 Enable detect anomaly during forward pass for more informations.
23:49:53 No forward pass information available.
23:49:53 Enable detect anomaly during forward pass for more informations.
23:49:53 test_autograd.py:2328: DeprecationWarning: Please use assertRaisesRegex instead.
23:49:53   with self.assertRaisesRegexp(RuntimeError, "Function 'MyFuncBackward' returned nan values in its 0th output."):
23:49:53 ok

cc @mruberry @VitalyFedyunin @walterddr
Repository: rust
Issue: [control flow analysis] Special treatment for `Err( )?`
Labels: A-type-system,T-lang,A-inference,C-feature-request,T-types

I don't know this issue should be `feature request` or `bug report`.

I have code like this:

```Rust
#[derive(Debug, Fail)]
#[fail(display = "Target can't open: {:?}", path)]
struct NotExists {
    path: PathBuf
}

#[derive(Debug, Fail)]
#[fail(display = "Target is not file: {:?}", path)]
struct NotFile {
    path: PathBuf
}

fn foo() -> Result<(), failure::Error> {
    let path = PathBuf::from("Rust/is/the/best/language");
    if path.exists() {
        Err(NotExists { path })?
    }
    if path.is_dir() {
        Err(NotFile { path })?
    }
    unimplemented!()
}
```

The compiler error is:

```
24 |         Err(NotExists { path })?
   |                         ---- value moved here
25 |     }
26 |     if path.is_dir() {
   |        ^^^^ value used here after move
``````

Obviously, `Err(foo)?` equals `return Err(foo.into());`, that means there should be no `use after moved` error. 
 
There are two other ways to achieve the goal:

```Rust
    if path.exists() {
        Err(NotExists { path })?
    } else if path.is_dir() {
        Err(NotFile { path })?
    } else {
        unimplemented!()
    }
```
or

```Rust
    if path.exists() {
        return Err(NotExists { path }.into());
    }
```

I think with control flow analysis, compiler should be able to pass my origin code(`Err(foo)?` version).
Repository: go
Issue: syscall, os: opening hidden file for write access on Windows gives ACCESS_DENIED_ERROR
Labels: OS-Windows,NeedsInvestigation,compiler/runtime

### What version of Go are you using (`go version`)?

go version go1.10.2 windows/amd64

also reproduces on go version go1.8.7 windows/386

### Does this issue reproduce with the latest release?

yes

### What operating system and processor architecture are you using (`go env`)?

set GOARCH=amd64
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOOS=windows

### What did you do?

I try to make a write access to an hidden text file on windows 10.

I made separate project to reproduce the issue here : https://github.com/FabienTregan/TestWritingToHiddenFilesInGo (with a bit more informations in the readme.md file)

### What did you expect to see?

Some text written in the file

### What did you see instead?

Received unexpected error:
                                        open C:\Users\[USER NAME]\AppData\Local\Temp\write_to_hidden_file_test_204946587: Access is denied.
Repository: vscode
Issue: [folding] unfold when pressing enter on last line
Labels: feature-request,editor-folding


![bug](https://user-images.githubusercontent.com/24494089/41499698-c931352c-7184-11e8-9e5a-9c620cb20474.gif)

Issue Type: <b>Bug</b>

Hi, 

After collapsing a Command (with code in) I can't enter after that.
In this example I can't enter between fds en the comment.

VS Code version: Code 1.21.1 (79b44aa704ce542d8ca4a3cc44cfca566e7720f1, 2018-03-14T14:47:13.351Z)
OS version: Windows_NT ia32 10.0.17134

<details>
<summary>System Info</summary>

|Item|Value|
|---|---|
|CPUs|Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz (8 x 4008)|
|Memory (System)|15.92GB (8.93GB free)|
|Process Argv|E:\ProgramFiles(x86)\Microsoft VS Code\Code.exe|
|Screen Reader|no|
|VM|0%|

</details><details><summary>Extensions (1)</summary>

Extension|Author (truncated)|Version
---|---|---
LiveServer|rit|5.0.0


</details>
Reproduces only with extensions
<!-- generated by issue reporter -->
Repository: TypeScript
Issue: Project compiles OK but hangs when --watch is added
Labels: Bug,Domain: Declaration Emit

**TypeScript Version:**  3.0.0-dev.20180616 and 2.9.2

**Search Terms:**  typescript watch --watch hangs stalls slow

Hi I have minimized one of my projects to an example source (which I have [attached](https://github.com/Microsoft/TypeScript/files/2108402/tsc-watch-hangs.zip) [1]) that: 
1. Compile fine when not using watch mode (takes about 5 seconds to compile on my machine)
2. Hangs the compiler when using watch mode (doesn't complete it's compile).

To compile the example:
1. download the [zip](https://github.com/Microsoft/TypeScript/files/2108402/tsc-watch-hangs.zip) and unzip it and enter that directory
2. run `yarn install` to add the project's deps.
3a. run `yarn tsc-ok` for the example where it does compile
3b. run `yarn tsc-bad` for the example where it doesn't

BTW: Is there some way for me to tell the compiler to dump out the source files it is processing as it processes them? It would be *extremely* helpful for tracking this down to the specific issue.

1: Source Code: [tsc-watch-hangs.zip](https://github.com/Microsoft/TypeScript/files/2108402/tsc-watch-hangs.zip)
Repository: flutter
Issue: Flutter Doctor reports the flutter version as v0.0.0-unknown if repository is cloned with depth 1.
Labels: c: new feature,tool,t: flutter doctor,has reproducible steps,P3,found in release: 2.6,team-tool,triaged-tool

On CI environments where the Flutter repository may be downloaded with `--depth=1` as an option to Git, `flutter doctor` says the installation is fine but won't actually build anything because the Flutter version it reports is `v0.0.0-unknown`. Builds will then fail on package constraint checks.

The workaround is to not specify the option but at the cost of longer checkouts.
Repository: flutter
Issue: CupertinoNavigationBar leading is too high
Labels: framework,a: fidelity,f: cupertino,has reproducible steps,P3,workaround available,team-design,triaged-design,found in release: 3.16,found in release: 3.19

For some reason the leading in CupertinoNavigationBar is too high. Any ideas?

<img width="315" alt="screen shot 2018-06-16 at 6 10 46 pm" src="https://user-images.githubusercontent.com/666539/41502812-b0308544-7190-11e8-9186-60ecad54af75.png">


```dart
class MyApp extends StatelessWidget {
  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return new MaterialApp(
      title: 'Flutter Demo',
      theme: new ThemeData(
        primarySwatch: Colors.blueGrey,
      ),
      home: CupertinoPageScaffold(
        navigationBar: new CupertinoNavigationBar(
      leading: Text("lead"),
      middle: Text("Title"),
      trailing: Text("Trailing")
    ),
        child: Center(child: Text("Test!")),

      )
    );
  }
}
```
Repository: TypeScript
Issue: [feat] Allow use of downlevelIteration on es2015 or greater
Labels: Suggestion,In Discussion

## Search Terms
downlevelIteration es2015 es6

## Suggestion
A new flag or way to enable `downlevelIteration` when targeting es2015 or greater
Maybe `forceDownlevelIteration` ?

## Use Cases
`for .. of` is slow: https://jsperf.com/for-vs-forof
This request comes from my recent work in Nodejs where you want to use higher targets for better performance from the runtime but have to replace `for .. of` with `for` due to it's inefficiencies.
Devs shouldn't be forced to write less ergonomic code for performance reasons when TypeScript could simply do it for you.

An example of having to refactor due to increasing the language target:
https://github.com/angular/angular/pull/24534

## Examples
N/A

## Checklist

My suggestion meets these guidelines:
* [x] This wouldn't be a breaking change in existing TypeScript / JavaScript code
* [x] This wouldn't change the runtime behavior of existing JavaScript code
* [x] This could be implemented without emitting different JS based on the types of the expressions
* [x] This isn't a runtime feature (e.g. new expression-level syntax)
Repository: opencv
Issue: resize: cannot specify both size and fx/fy
Labels: category: imgproc,RFC

##### System information (version)

- OpenCV => all
- Operating System / Platform => all
- Compiler => all

##### Detailed description

By reading the documentation it seems that it is possible to specify both parameters **dsize** for the destination size in pixels and **fx/fy** for a fine tuning of the scaling interpolation.
"..the size and type are derived from the src,dsize,fx, and fy"
"Either dsize or both fx and fy must be non-zero."

But looking at the code only one of the two parameters are considered. The other is always calculated.
If dsize is specified then fx/fy are calculated/overwritten. If fx/fy are specified then dsize is calculated/overwritten.
So if I want a fine tuning then I have to specify only fx/fy leaving dsize to zero.

##### Code in cv::resize

```.cpp
    if( dsize.area() == 0 )
    {
        dsize = ...;
    }
    else
    {
        inv_scale_x = (double)dsize.width/ssize.width;
        inv_scale_y = (double)dsize.height/ssize.height;
    }
```
##### Solution 1.

Update the documentation saying that only one of the two parameters should be specified, specifically fx/fy for fine tuning because slightly different values of fx/fy will produce different images even if the resulting image has the same size (as by design).

We could also make more clear in the documentation that by specifying only the destination size the caller doesn't care for a fine tuning of the resize call. If the caller specifies the fx/fy parameter then a fine tuning is required leading to different results even with identical destination size.

##### Solution 2.

Update the code so that it is possible to specify both values: dsize to set the exact size in pixels and fx/fy to specify a fine tuning of the resizing.
Repository: opencv
Issue: resize: fx/fy parameters near to 1.0 are ignored
Labels: category: imgproc,RFC

##### System information (version)

- OpenCV => all
- Operating System / Platform => all
- Compiler => all

##### Detailed description

By specificating the fx/fy parameters it is possible to fine tune the resizing. For example if I need to create multiple images for an animation with small increases in the resize. It works as designed except when the resulting image has the same size as the original image. And there is no workaround with the resize function.

We could also make more clear in the documentation that by specifying only the destination size the caller doesn't care for a fine tuning of the resize call. If the caller specifies the fx/fy parameter then a fine tuning is required leading to different results even with identical destination size.

##### Code in cv::resize

```.cpp
// as designed these two resize calls will create two different images: OK
// for both calls the resulting image has double size compared to original image.
resize(image, dest, size(), 2, 2);
resize(image, dest, size(), 2+0.499/image.cols, 2+0.499/image.rows);

// these two calls generate the same image: NOT OK.
// for both calls the resulting image has same size as the original image
resize(image, dest, size(), 1, 1);
resize(image, dest, size(), 1+0.499/image.cols, 1+0.499/image.rows);
```
##### Solution

Update the code so that the original image is returned only if the fine tune scale parameter is set to exactly 1. E.g. In resize replace the following code:
```.cpp
    if (dsize == ssize)
    {
        // Source and destination are of same size. Use simple copy.
        src.copyTo(dst);
        return;
    }
```

with something like
```.cpp
    if (inv_scale_x == 1. && inv_scale_y == 1.)
    {
        // Source and destination are of same size. Use simple copy.
        src.copyTo(dst);
        return;
    }
```
Repository: go
Issue: cmd/link: link failure caused by duplicated LDFLAGS passed to external linker
Labels: NeedsInvestigation,compiler/runtime

### What version of Go are you using (`go version`)?

go1.10.3

### Does this issue reproduce with the latest release?
Yes

### What operating system and processor architecture are you using (`go env`)?

macOS/amd64, doing cross compile for android/arm64

### What did you do?

export GOOS=android
export GOARCH=arm64
export CC=/path/to/androd/gcc
export CXX=/path/to/androd/g++
export CGO_ENABLED=1
export CGO_LDFLAGS="-Wl,--version-script=/the/version-script/export.txt"
go build -buildmode=c-shared -ldflags="-s -w -v" -v -o libxxx.so foo/bar

### What did you expect to see?

The shared library is generated without any error.

### What did you see instead?

In the output line

  0.19 host link: ...

"-Wl,--version-script=/the/version-script/export.txt" is repeated 4 time and there are 3 errors,

    anonymous version tag cannot be combined with other version tags

Note that if the version script contains a version name, the error message is 'duplicate version tag xxx'.
Repository: vscode
Issue: cursor up/down is confused by selection
Labels: feature-request,editor-commands

- VSCode Version: 1.24.0
- OS Version: Windows 7

Steps to Reproduce:

1. Create the following document
```
dorfus
hamnut
```
2. Place the cursor at the d and go right three. Then go down.  The cursor is at the n. Good.
3. Place the cursor at the end of dorfus and go left three. Then go down.  The cursor is at the n. Good.
4. Place the cursor at the d and hold down shift while going right three. Release shift.  Go down. The cursor is at the n. Good
5. Place the cursor at the end of dorfus and hold shift while going left three. Release shift. Go down. The cursor is at the end of hamnut? <--- BUG

I expect the cursor to be at n in every case

Other applications, for reference:
visual studio: good
wordpad: good
notepad: good
firefox: good
IE: good
chrome: mysteriously works today, but didn't when I wrote this bug
gedit: "broken" also, but on second thought, maybe it's sensible to some people. 

In any event, it's also senseless to some people.

Does this issue occur when all extensions are disabled?: Yes
Repository: rust
Issue: Code fails to link on macOS with incremental compilation
Labels: A-linkage,O-macos,C-enhancement,P-medium,T-compiler,A-incr-comp

The following code fails to link on macOS with incremental compilation. It works fine with incremental compilation turned off.

```rust
fn obj_alloc<T>(_p: T) -> *const [u8; 6] {
        struct Foo(*const [u8; 6]);
        unsafe impl Send for Foo {}
        unsafe impl Sync for Foo {}
        #[link_section="__TEXT,__objc_methname,cstring_literals"]
        static OBJC_METH_VAR_NAME_ : [u8; 6] = *b"alloc\0";

        #[link_section="__DATA,__objc_selrefs,literal_pointers,no_dead_strip"]
        static OBJC_SELECTOR_REFERENCES_: Foo = Foo(&OBJC_METH_VAR_NAME_);
        return OBJC_SELECTOR_REFERENCES_.0;
}

fn main() {
    let i: u32 = 0;
    obj_alloc(i);
}
```

The linker error is
```
  = note: Undefined symbols for architecture x86_64:
            "example::obj_alloc::OBJC_SELECTOR_REFERENCES_::ha196cc644aeae679", referenced from:
                example::obj_alloc::h54a034c209c8e4e3 in example-4a01b102e2898ce7.4jffkl93bkv60fxe.rcgu.o
          ld: symbol(s) not found for architecture x86_64
          clang: error: linker command failed with exit code 1 (use -v to see invocation)
```

The motivation for this weird code is getting native performance for objective-c message selectors. See https://github.com/SSheldon/rust-objc/issues/49
Repository: flutter
Issue: [Proposal] Allow to adjust hitSlop of GestureDetector
Labels: c: new feature,framework,f: gestures,c: proposal,P2,team-framework,triaged-framework

@Hixie already closed #14794 but I'm posting a new Issue because this is critical functionality

Hixie recommended using padding to adjust the hitbox of a `GestureDetector`. I tried to use padding but it merely produced whitespace around GestureDetector, or around its children.

I even tried to transform with hitTests up and then transform without hitTests down to try to hack my way to a solution to move the hitbox and that failed.

Now, back to GestureDetector without any hackiness!

First box is what Flutter does, second box is what I want for best functionality and user experience.

<img width="307" alt="screen shot 2018-06-17 at 9 01 52 pm" src="https://user-images.githubusercontent.com/666539/41514239-61e8f572-7273-11e8-8a89-d3082b63daa9.png">


in RN this would be achieved by typing `hitSlop={{top: 20, bottom:20}}`
Repository: youtube-dl
Issue: Adding boyztube.com and boyztube.xxx websites
Labels: nsfw

## Please follow the guide below

- You will be asked some questions and requested to provide some information, please read them **carefully** and answer honestly
- Put an `x` into all the boxes [ ] relevant to your *issue* (like this: `[x]`)
- Use the *Preview* tab to see what your issue will actually look like

---

### Make sure you are using the *latest* version: run `youtube-dl --version` and ensure your version is *2018.06.18*. If it's not, read [this FAQ entry](https://github.com/rg3/youtube-dl/blob/master/README.md#how-do-i-update-youtube-dl) and update. Issues with outdated version will be rejected.
- [x] I've **verified** and **I assure** that I'm running youtube-dl **2018.06.18**

### Before submitting an *issue* make sure you have:
- [x] At least skimmed through the [README](https://github.com/rg3/youtube-dl/blob/master/README.md), **most notably** the [FAQ](https://github.com/rg3/youtube-dl#faq) and [BUGS](https://github.com/rg3/youtube-dl#bugs) sections
- [x] [Searched](https://github.com/rg3/youtube-dl/search?type=Issues) the bugtracker for similar issues including closed ones
- [x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

### What is the purpose of your *issue*?
- [ ] Bug report (encountered problems with youtube-dl)
- [x] Site support request (request for adding support for a new site)
- [ ] Feature request (request for a new functionality)
- [ ] Question
- [ ] Other

---

### The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your *issue*

---

### If the purpose of this *issue* is a *site support request* please provide all kinds of example URLs support for which should be included (replace following example URLs by **yours**):
- Single video: http://www.boyztube.xxx/gay/now-this-is-a-true-cocksucker-clip/89882
- Single video: http://www.boyztube.com/gay-videos/watch/620553-straight_hustler.html
- Playlist: -

Note that **youtube-dl does not support sites dedicated to [copyright infringement](https://github.com/rg3/youtube-dl#can-you-add-support-for-this-anime-video-site-or-site-which-shows-current-movies-for-free)**. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

---

### Description of your *issue*, suggested solution and other information

Explanation of your *issue* in arbitrary form goes here. Please make sure the [description is worded well enough to be understood](https://github.com/rg3/youtube-dl#is-the-description-of-the-issue-itself-sufficient). Provide as much context and examples as possible.
If work on your *issue* requires account credentials please provide them or explain how one can obtain them.
Repository: go
Issue: cmd/vet: detect Get/Post http.Response assignment to "_", which is a memory leak
Labels: NeedsInvestigation,Analysis

I made a rookie mistake and introduced a memory leak into a codebase by failing to realize that you must close the `http.Response` body returned by `http.Post()` even if you don't need the response content.

I suspect I'm not only one who has made this mistake.

I have a diff, if the proposal is accepted.
Repository: pytorch
Issue: [Caffe2] How to build caffe2/mobile/ulp2/ulp_test?
Labels: caffe2

I'm trying to test out the kernel implementation of quantized convolution in `caffe2/mobile/ulp2df folder. Currently, I would like to build and run the ulp_test.cc. However, I don't know how to build the files in there. Can someone tell me how? Maybe some CMakeLists files. 

Thanks in advance

Cheers,