sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: pytorch
Issue: Bilinear interpolation behavior inconsistent with TF, CoreML and Caffe
Labels: triaged,module: interpolation

## Issue description

Trying to compare and transfer models between Caffe, TF and Pytorch found difference in output of bilinear interpolations between all. Caffe is using depthwise transposed convolutions instead of straightforward resize, so it's easy to reimplement both in TF and Pytorch.
However, there is difference between output for TF and Pytorch with `align_corners=False`, which is default for both.

## Code example

```Python
img = cv2.resize(cv2.imread('./lenna.png')[:, :, ::-1], (256, 256))
img = img.reshape(1, 256, 256, 3).astype('float32') / 255.
img = tf.convert_to_tensor(img)
output_size = [512, 512]
output = tf.image.resize_bilinear(img, output_size, align_corners=True)
with tf.Session() as sess:
        values = sess.run([output])
out_tf = values[0].astype('float32')[0]

img = img.reshape(1, 256, 256, 3).transpose(0, 3, 1, 2).astype('float32') / 255.
out_pt = nn.functional.interpolate(torch.from_numpy(nimg), 
                                   scale_factor=2, 
                                   mode='bilinear', 
                                   align_corners=True)
out_pt = out_pt.data.numpy().transpose(0, 2, 3, 1)[0]

print(np.max(np.abs(out_pt - out_tf)))
# output 5.6624413e-06
``` 
But
```Python
img = cv2.resize(cv2.imread('./lenna.png')[:, :, ::-1], (256, 256))
img = img.reshape(1, 256, 256, 3).astype('float32') / 255.
img = tf.convert_to_tensor(img)
output_size = [512, 512]
output = tf.image.resize_bilinear(img, output_size, align_corners=False)
with tf.Session() as sess:
        values = sess.run([output])
out_tf = values[0].astype('float32')[0]

img = img.reshape(1, 256, 256, 3).transpose(0, 3, 1, 2).astype('float32') / 255.
out_pt = nn.functional.interpolate(torch.from_numpy(nimg), 
                                   scale_factor=2, 
                                   mode='bilinear', 
                                   align_corners=False)
out_pt = out_pt.data.numpy().transpose(0, 2, 3, 1)[0]

print(np.max(np.abs(out_pt - out_tf)))
# output 0.22745097
``` 

Output diff * 10:
![image](https://user-images.githubusercontent.com/301147/44241919-c4d2d180-a17b-11e8-81eb-9e44c5741d42.png)

Output of CoreML is consistent with TF, so it seems that there is a bug with implementation of bilinear interpolation with `align_corners=False` in Pytorch.

Diff is reproducible both on cpu and cuda with cudnn 7.1, cuda 9.1.
Repository: opencv
Issue: app link error with iOS framework 
Labels: category: build/install,platform: ios/osx,needs investigation

##### System information (version)
- OpenCV => 3.4.2
- Operating System / Platform => Mac OSX 10.13.6 (17G65)
- Compiler => XCode Version 9.4.1 (9F2000)

##### Detailed description

![image](https://user-images.githubusercontent.com/7456629/44246549-5b9d9d80-a211-11e8-865d-472f6568db1e.png)
![image](https://user-images.githubusercontent.com/7456629/44246627-b7682680-a211-11e8-98c8-8ecfe8beb4b3.png)
Repository: pytorch
Issue: Request for better memory management
Labels: feature,module: memory usage,triaged

This issue is mainly about how to recover from an `out of memory` exception, previously posted in [forum](https://discuss.pytorch.org/t/whats-the-best-way-to-handle-exception-cuda-runtime-error-2-out-of-memory/11891).

Till now, it's not always possible to recover from OOM exception whether during train or inference. This will pose huge risk when you use pytorch in large-scale training or deployment.

Please pay attention to this problem, I really like pytorch anyway.
Repository: rust
Issue: Unrelated error report for trait bound checking
Labels: C-enhancement,A-diagnostics,T-compiler

Given the following code:
```rust
use nalgebra::{Dynamic, MatrixMN};
use std::fmt::Debug;

trait CheckedType : Copy + Clone + Debug + Ord {}

struct Foo<T: CheckedType> {
    matrix: MatrixMN<T, Dynamic, Dynamic>,
}
```

The compilers reports:
```
error[E0310]: the parameter type `T` may not live long enough
 --> src/main.rs:9:5
  |
8 | struct Foo<T: CheckedType> {
  |            -- help: consider adding an explicit lifetime bound `T: 'static`...
9 |     matrix: MatrixMN<T, Dynamic, Dynamic>,
  |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  |
note: ...so that the type `T` will meet its required lifetime bounds
 --> src/main.rs:9:5
  |
9 |     matrix: MatrixMN<T, Dynamic, Dynamic>,
  |     ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
```

Checking the definition of `nalgebra::MatrixMN`, it seems the trait bound needs to include `Scalar`, so if I add `nalgebra::Scalar` to the bound of `CheckedType`, then the code passes compile.

The error report for the original code seems to be completely unrelated to the actual fix. I suspect this is a compiler bug.
Repository: react
Issue: onChange doesn't fire if input re-renders due to a setState() in a non-React capture phase listener
Labels: Type: Bug,Component: DOM,React Core Team

Extracting from https://github.com/facebook/react/issues/12643.

This issue has always been in React. I can reproduce it up to React 0.11. However **it's probably extremely rare in practice and isn't worth fixing**. I'm just filing this for posterity.

Here is a minimal example.

```js
class App extends React.Component {
  state = {value: ''}
  handleChange = (e) => {
    this.setState({
      value: e.target.value
    });
  }
  componentDidMount() {
    document.addEventListener(
      "input",
      () => {
        // COMMENT OUT THIS LINE TO FIX:
        this.setState({});
      },
      true
    );
  }
  render() {
    return (
      <div>
        <input
          value={this.state.value}
          onChange={this.handleChange}
        />
      </div>
    );
  }
}

ReactDOM.render(<App />, document.getElementById("container"));
```

Typing doesn't work — unless I comment out that `setState` call in the capture phase listener.

Say the input is empty and we're typing `a`.

What happens here is that `setState({})` in the capture phase non-React listener runs first. When re-rendering due to that first empty `setState({})`, input props still contain the old value (`""`) while the DOM node's value is new (`"a"`). They're not equal, so we'll set the DOM node value to `""` (according to the props) and remember `""` as the current value.

<img width="549" alt="screen shot 2018-08-17 at 1 08 42 am" src="https://user-images.githubusercontent.com/810438/44241204-4b0e0880-a1ba-11e8-847d-bf9ca43eb954.png">

Then, `ChangeEventPlugin` tries to decide whether to emit a change event. It asks the tracker whether the value has changed. The tracker compares the presumably "new" `node.value` (it's `""` — we've just set it earlier!) with the `lastValue` it has stored (also `""` — and also just updated). No changes!

<img width="505" alt="screen shot 2018-08-17 at 1 10 59 am" src="https://user-images.githubusercontent.com/810438/44241293-e0110180-a1ba-11e8-9c5a-b0d808f745cd.png">


Our `"a"` update is lost. We never get the change event, and never actually get a chance to set the correct state.
Repository: electron
Issue: Split session 'Media' permission to 'Camera' and 'Microphone'
Labels: enhancement :sparkles:

**Is your feature request related to a problem? Please describe.**
In the actual Electron Session permission request handler, the 'media' permission englobes both microphone and camera.

**Describe the solution you'd like**
I suggest that it should be splited into microphone and camera, in order to work like Chrome, Firefox and other major browsers.

It will also bring a more detailed option for a specific set of applications, that only require access to either microphone or the webcam of the user.
Repository: rust
Issue: Query system cycle errors should be extendable with notes
Labels: C-enhancement,A-diagnostics,T-compiler

Relevant PR: https://github.com/rust-lang/rust/pull/53316
Relevant Issue: #52985 

The [query system](https://rust-lang-nursery.github.io/rustc-guide/query.html) automatically detects and emits a cycle error if a cycle occurs when dependency nodes are added to the query DAG. This error is extensible with a custom main message defined as below to help human readability,
https://github.com/rust-lang/rust/blob/a385095f9a6d4d068102b6c72fbdc86ac2667e51/src/librustc/ty/query/config.rs#L93
but is otherwise closed for modification outside of the query::plumbing module:
https://github.com/rust-lang/rust/blob/b2397437530eecef72a1524a7e0a4b42034fa360/src/librustc/ty/query/plumbing.rs#L248

It would be nice to have a mechanism in addition that allows custom notes and suggestions to be added to these errors to help illustrate why a cycle occurred, not just where. It may be possible to expose the `DiagnosticBuilder` or provide wrappers for methods like `span_suggestion()` and `span_note()`
Repository: go
Issue: x/net/idna: Display returns invalid label for r4---sn-a5uuxaxjvh-gpm6.googlevideo.com.
Labels: NeedsInvestigation

### What version of Go are you using (`go version`)?
```
go version go1.10.3 linux/amd64 via docker golang:1.10.3-alpine
(also go version go1.10.3 darwin/amd64)
```

### Does this issue reproduce with the latest release?
```
Yes
```

### What operating system and processor architecture are you using (`go env`)?
```
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/go"
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build623899479=/tmp/go-build -gno-record-gcc-switches"
```

### What did you do?

Transforming domain name to human-readable form with "golang.org/x/net/idna".

```golang
package main

import (
	"fmt"
	"golang.org/x/net/idna"
)

func main() {
	_, e := idna.ToUnicode("r4---sn-a5uuxaxjvh-gpm6.googlevideo.com")
	fmt.Println(e)
	_, e = idna.Display.ToUnicode("r4---sn-a5uuxaxjvh-gpm6.googlevideo.com")
	fmt.Println(e)
}
```

### What did you expect to see?

I expect to see no error.
```
<nil>
<nil>
```

### What did you see instead?

But `idna.Display` does not agree to accept the host name as a valid one.
```
<nil>
idna: invalid label "r4---sn-a5uuxaxjvh-gpm6"
```

`.ToASCII` has the same property.
Repository: vscode
Issue: [npm] hover should show relevant latest version
Labels: feature-request,json

In a `package.json`, when you hover the version for a dependency it will show the "[Latest version](https://github.com/Microsoft/vscode/blob/9a03a86c0a54a24c355bd950ddad91a0e74de6dd/extensions/npm/src/features/packageJSONContribution.ts#L303)". I think it would be useful to (also) show the _relevant_ latest version, which might differ if using a version prefix like `^` or `~`.
Repository: flutter
Issue: Expose the tri-state checkbox "indeterminate" state in the semantics data
Labels: framework,a: accessibility,platform-fuchsia,c: proposal,P2,team-framework,triaged-framework

Right now we treat "indeterminate" and "unchecked" as the same. This works for Android and iOS since their accessibility APIs don't support tristate checkboxes, but hopefully Fuchsia's API will support tristate checkboxes so we should expose the data there at least.
Repository: awesome-mac
Issue: Recategorize Art/Protoyping/Modeling Software
Labels: help wanted,organization

It should really be better organized, and the categories we have now are insufficient.

Maybe add a `3D Modeling` category?
Also, maybe rename `Screenshot Tools` to `Screencapturing Software`?

- [x] I have checked for other similar issues
- [x] I have explained why this change is important
Repository: flutter
Issue: Box shadow doesn't have flexible features.
Labels: c: new feature,framework,P2,team-framework,triaged-framework

I love flutter. I have been using it for a year. I'm working on a client project. And the demand is We need Box-shadow for only 3 sides. How can I achieve this? If it would have been like `BoxShadow.only` as in the case of `EdgeInsted.only`, that would be much better. So that as a Developer, I can't say "No" to my Client. And that's what Flutter is For. I don't know where to suggest this. So I'm reporting here.
![37932941_1016917551815921_3708467126361653248_n](https://user-images.githubusercontent.com/24828445/44294926-bb04b780-a2bc-11e8-9f46-c24b45d734c4.png)
Repository: flutter
Issue: [web] Drawer menu button is not reachable by screen readers
Labels: framework,f: material design,a: accessibility,platform-web,has reproducible steps,P2,found in release: 3.3,found in release: 3.6,team-web,triaged-web
Repository: godot
Issue: How to Scroll via ScrollContainer in android or touch screen devices
Labels: bug,platform:android,platform:macos,confirmed,usability,topic:input,topic:gui

Godot version: 3.1.dev
commit 15ee6b7

OS/device including version: MacOSX High Sierra, Android

Issue description:How to scroll via ScrollContainer. It doesnt scroll in android or touch screen or via dragging

Steps to reproduce:

1. Add Scroll Container in the scene
2. Add VBoxContainer as child of scrollContainer
3. Add Panel with min. size with a sprite inside

Note: Scrolling via mouse wheel works or dragging the scrollBar but if its in android It should be able to scroll via dragging the content right? 

Thanks for the response in advance
Repository: TypeScript
Issue: Add an option to force source maps to use absolute paths?
Labels: Suggestion,In Discussion,Add a Flag

Previous related issue: https://github.com/Microsoft/TypeScript/issues/23180

It seems VS2017 is incompatible with relative paths, when debugging a UWP app now tries to load source maps from the app URI and the host does not support it.

```
'WWAHost.exe' (Script): Loaded 'Script Code (MSAppHost/3.0)'. 
SourceMap ms-appx://6b826582-e42b-45b9-b4e6-dd210285d94b/js/require-config.js.map read failed: The URI prefix is not recognized..
```

Hard-coding the full path in `tsconfig.json` is not a good idea here, I hope we have an option to behave as 2.9.2 so that we can get the auto-resolved full path.

Edit: My tsconfig.json that works in TS2.9 but not in TS3.0:

```json
{
  "compilerOptions": {
    "strict": true,
    "noEmitOnError": true,
    "removeComments": false,
    "sourceMap": true,
    "target": "es2017",
    "outDir": "js/",
    "mapRoot": "js/",
    "sourceRoot": "sources/",
    "emitBOM": true,
    "module": "amd",
    "moduleResolution": "node"
  },
  "exclude": [
    "node_modules",
    "wwwroot"
  ]
}
```

My failed trial to fix this: (removed `mapRoot` and changed `sourceRoot`)

```json
{
  "compilerOptions": {
    "strict": true,
    "noEmitOnError": true,
    "removeComments": false,
    "sourceMap": true,
    "target": "es2017",
    "outDir": "js/",
    "sourceRoot": "../sources/",
    "emitBOM": true,
    "module": "amd",
    "moduleResolution": "node"
  },
  "exclude": [
    "node_modules",
    "wwwroot"
  ]
}
```
Repository: TypeScript
Issue: Proposal: `export as namespace` for UMD module output
Labels: Suggestion,In Discussion

## Current problem

TypeScript supports UMD output but does not support exporting as a global namespace.

## Syntax

  _NamespaceExportDeclaration:_
   `export` `as` `namespace` _IdentifierPath_

## Behavior

```ts
export var x = 0;
export function y() {}
export default {};
export as namespace My.Custom.Namespace;

// emits:

(function (global, factory) {
  if (typeof module === "object" && typeof module.exports === "object") {
    var v = factory(require, exports);
    if (v !== undefined) module.exports = v;
  }
  else if (typeof define === "function" && define.amd) {
    define(["require", "exports"], factory);
  }
  else {
    global.My = global.My || {};
    global.My.Custom = global.My.Custom || {};
    global.My.Custom.Namespace = global.My.Custom.Namespace || {};
    var exports = global.My.Custom.Namespace;
    factory(global.require, exports);
  }
})(this, function (require, exports) {
  "use strict";
  exports.__esModule = true;
  exports.x = 0;
  function y() { }
  exports.y = y;
  exports["default"] = {};
});
```

## Note

* This proposal basically follows Babel behavior.
* Importing any module without a module loader will throw in this proposal. A further extension may use global namespaces as Babel does.
* Babel overwrites on the existing namespace whereas this proposal extends the existing one, as TS `namespace` does.
* Rollup has a special behavior where `export default X` works like CommonJS `module.exports = X` whereas this proposal does not.

Prior arts: [Babel exactGlobals](https://babeljs.io/docs/en/babel-plugin-transform-es2015-modules-umd/#more-flexible-semantics-with-exactglobals-true), [Webpack multi part library](https://github.com/webpack/webpack/tree/v4.16.5/examples/multi-part-library), [Rollup `output.name` option with namespace support](https://rollupjs.org/guide/en#core-functionality)

## See also

#8436
#10907
#20990
Repository: godot
Issue: Allow SkeletonIK to ignore target's rotation
Labels: enhancement,topic:core

**Godot version:**
`master` / c93888ae

**OS/device including version:**
Manjaro Linux 17.1

**Issue description:**
Currently, `SkeletonIK` will try to match the bone hierarchy to a given target's transform which is not always desired.

For example, IK can be used to make a character stretch its hand towards an item when picking it up. And in this case, the arm could be twisted in an unnatural manner, if the character is positioned in a 'wrong' direction from the item.

As an item should be picked up from any direction, it would be much easier if the IK system allowed matching a bone chain to its target's position, without affecting rotation of its constiuent parts.

(For those who need such a feature now: you can change `SkeletonIK.Target` in each frame, so that it matches rotation with the assigned target.)
Repository: flutter
Issue: FlutterDriver locator and fluent-style improvements
Labels: a: tests,c: new feature,framework,t: flutter driver,customer: crowd,P2,team-framework,triaged-framework

To be more like Selenium-WebDriver, FlutterDriver would be better if it had a mechanism to find lists of matches for a given locator.

```java
// WebDriver's Java API:
Element element = webDriver.findElement(locator); // singular match
List<WebElement> elements = webDriver.findElement(locator); // multiple matches, returned as list
String secondElemText = elements.get(1).getText();
```

FlutterDriver only has mechanisms to target single matches to locators, right now. 

To be honest, FlutterDriver's API feels more like Selenium-RC from 2004 (I co-created that) rather than the superior Selenium2 API (WebDriver) that came a few years later.
Repository: go
Issue: crypto/tls: fix pseudo-constant mitigation for lucky 13
Labels: NeedsInvestigation

As detailed in the paper "Pseudo Constant Time Implementations of TLS
Are Only Pseudo Secure"
 https://eprint.iacr.org/2018/747
Repository: youtube-dl
Issue: HRTi Extractor not working
Labels: account-needed

## 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.08.04*. 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.08.04**

### 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*?
- [x] 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 HRTi extractor is not working anymore. HRTi has changed the web site. Can anyone help with writing a new extractor?

Single Video: https://hrti.hrt.hr/videostore/moviedetails?referenceId=41616AF4-5113-4&refer=videostore%7Cmovies&customCatalogueReferenceId=PUTOPISI&player=True&heading=PUTOPISI
Repository: rust
Issue: Tracking issue for RFC 2504, "Fix the Error trait"
Labels: B-RFC-approved,T-libs-api,C-tracking-issue,A-error-handling,Libs-Tracked,PG-error-handling

This is a tracking issue for the RFC "Fix the Error trait" (rust-lang/rfcs#2504).

**Steps:**

- [x] Implement the RFC (cc @rust-lang/libs)
     - [X] `source` method #53533
     - [x] Backtrace API #64154
- [x] ~~Implement proof of concept showing stabilizing `backtrace` method won't prevent moving `Error` into `core` later https://github.com/rust-lang/rust/pull/77384~~
- [ ] Fix std::backtrace::Backtrace's fmt representations
     - ~~Precision flag support #65280~~ this is non-blocking for stabilization
     - [ ] Differences with `panic!` backtraces #71706
- [ ] Adjust documentation ([see instructions on forge][doc-guide])
- [ ] Stabilization PR ([see instructions on forge][stabilization-guide])

[stabilization-guide]: https://forge.rust-lang.org/stabilization-guide.html
[doc-guide]: https://forge.rust-lang.org/stabilization-guide.html#updating-documentation

**Unresolved questions:**

- [x] The choice to implement nullability internal to backtrace may prove to be a mistake: during the period when backtrace APIs are only available on nightly, we will gain more experience and possible change backtrace's constructors to return an `Option<Backtrace>` instead.

**Current status:**

https://github.com/rust-lang/rust/issues/53487#issuecomment-726326510
Repository: rust
Issue: Error message for E0453 is missing details when source is out of current crate
Labels: C-enhancement,A-diagnostics,T-compiler

From <https://travis-ci.org/open-i18n/rust-unic/jobs/417843357>:

```rust
   Compiling unic-ucd-version v0.7.0 (file:///home/travis/build/open-i18n/rust-unic/unic/ucd/version)
     Running `rustc --crate-name unic_ucd_version unic/ucd/version/src/lib.rs --emit=dep-info,link -C debuginfo=2 --test -C metadata=0f052cf9e92ebfb5 -C extra-filename=-0f052cf9e92ebfb5 --out-dir /home/travis/build/open-i18n/rust-unic/target/debug/deps -C incremental=/home/travis/build/open-i18n/rust-unic/target/debug/incremental -L dependency=/home/travis/build/open-i18n/rust-unic/target/debug/deps --extern unic_common=/home/travis/build/open-i18n/rust-unic/target/debug/deps/libunic_common-4592f220db252de4.rlib`
error[E0453]: allow(unused) overruled by outer forbid(unused)

error: aborting due to previous error

For more information about this error, try `rustc --explain E0453`.
error: Could not compile `unic-ucd-version`.
```

Apparently `allow(unused)` mentioned here is from a file external to the crate being compiled. (There's no allow-unused rule in the `rust-unic` repository.)

IMHO, the error is missing any information on where it is originated (the `allow(unused)` file or crate), and only displays the name of the crate being compiled.

Can we improve this error message in any way?