sharjeelyunus/github-issues-dataset huggingface dataset:
Repository: flutter
Issue: Rename LICENSE to NOTICES and update code accordingly
Labels: tool,framework,engine,P2,team-engine,triaged-engine

We currently have a LICENSES file in the engine repo. Based on feedback from our legal team, we should (to make things clearer) rename that to NOTICES. This will involve updating the license script documentation to write to NOTICES instead of LICENSE.

We should also update the flutter tool to read from the NOTICES file of each package, if there is one, and only if there isn't fall back on the LICENSE file.

The file that we put into the application package should also be renamed NOTICES. The framework should be updated accordingly.
Repository: flutter
Issue: Update Tonic to better indicate error conditions.
Labels: team,engine,P2,team-engine,triaged-engine

Currently, Tonic has sub-optimal mechanisms of indicating errors in some of its functions. The convention followed (somewhat inconsistently) is [to log to standard error and then exit the process](https://github.com/fuchsia-mirror/topaz/blob/efc5ace130b83ed77accc67788ce28998d0b10fc/lib/tonic/file_loader/file_loader.cc#L176). This often causes unexpected failures that are hard to diagnose. For example, on iOS, logging to standard error does not redirect the logs to tooling that may potentially be listening for errors. Syslog must be used instead on that platform. Also, while calling `exit` on iOS does wind up calling `abort` (which the tooling identifies as process death), the same call is also used on the iOS simulator which does exit the process with the specified code.

We should rework error handling and log propagation in Tonic.
Repository: go
Issue: x/build/cmd/gerritbot: perform gofmt checks before pushing to Gerrit
Labels: help wanted,Builders,NeedsFix

In https://golang.org/cl/99337 a file using CRLF instead of LF line endings was added. When using git-codereview to send CLs, there is a gofmt check which prevents this. But it seems there is no corresponding check when sending CLs via Github PRs.

It would be nice to run the gofmt check also for PRs submitted via Github.

/cc @bradfitz @andybons
Repository: react-native
Issue: [SectionList][inverted] SectionList headers are sticky-at-the-top-footers if the list is inverted
Labels: Resolution: PR Submitted,Component: SectionList,Bug

- [x] I have reviewed the [documentation](https://facebook.github.io/react-native)
- [x] I have searched [existing issues](https://github.com/facebook/react-native/issues)
- [x] I am using the [latest React Native version](https://github.com/facebook/react-native/releases)

## Environment
Environment:
  OS: macOS High Sierra 10.13.1
  Node: 8.6.0
  Yarn: 1.5.1
  npm: 4.6.1
  Watchman: 4.9.0
  Xcode: Xcode 9.0.1 Build version 9A1004
  Android Studio: Not Found

Packages: (wanted => installed)
  react: 16.3.1 => 16.3.1
  react-native: 0.55.3 => 0.55.3

## Steps to Reproduce

Clone [this repository](https://github.com/terrysahaidak/ReactNative-SeactionList-Bug-Example) and run it via `react-native run-ios`.
Observe the section separators are in wrong (randomly?) places.

## Expected Behavior

The section list headers should be in the top of the section instead of the bottom like section footers do with an inverted list (but they aren't sticky).

#### Expected gif:
![kapture 2018-04-19 at 13 46 22](https://user-images.githubusercontent.com/7809008/38987367-1b25ac26-43d8-11e8-854b-05c4b1b64242.gif)


## Actual Behavior

The section headers are footers instead of headers. They are sticky because of https://github.com/facebook/react-native/pull/17762 but still footers, not headers.

Sample code:
```jsx
export default class App extends React.Component {
  render() {
    return (
      <View style={s.container}>
        <SectionList
          inverted
          sections={mock.sections}
          maxToRenderPerBatch={10}
          initialNumToRender={10}
          style={{ flex: 1 }}
          keyExtractor={(item) => item.messageId}
          renderSectionHeader={sectionProps => (
            <SectionHeader
              {...sectionProps}
            />
          )}
          renderItem={({ item }) => (
            <ListItem
              item={item}
            />
          )}
          ItemSeparatorComponent={ItemSeparator}
        />
        />
      </View>
    );
  }
}
```

#### Problem Gif:
![kapture 2018-04-19 at 13 43 42](https://user-images.githubusercontent.com/7809008/38987253-c12987e2-43d7-11e8-8baa-40d48e5743f6.gif)
Repository: pytorch
Issue: [Feature request] LayerNormLSTMCell and LayerNormLSTM
Labels: triaged,enhancement

It could be convenient to have `LayerNormLSTMCell` and `LayerNormLSTM` implemented in `torch.nn`.
Repository: flutter
Issue: Would like a tool to do symbolication (ideally, would like "flutter logs"/"flutter run" to do it automatically)
Labels: c: new feature,tool,engine,P2,team-engine,triaged-engine

Related to https://github.com/flutter/flutter/issues/1016

Would be really nice to have a tool to symbolicate, e.g. follow the steps of:
https://github.com/flutter/engine/wiki/Symbolicating-production-crash-stacks

Including from custom engine builds.  We hit an issue recently with customer:gold where they are building their own engine with cherry-picks, thus our normal symbolication instructions break down.  Would be nice if we could point them to a script to symbolicate.
Repository: go
Issue: x/build: add Gerrit plugin to give us REST endpoint for last-modified time of anything globally
Labels: Builders

Gerrit doesn't have pubsub, so we instead have a dummy Google account that subscribes to all possible email spam from Gerrit. That dummy Google account has an email address with a domain name whose MX record goes to an SMTP server we run (see https://pubsubhelper.golang.org/).

That lets us get realtime updates from Gerrit so gopherbot can react in realtime to changes in Gerrit.

But it only works for actions for which Gerrit generates an email.

Some actions on Gerrit, notably modifying hashtags on a CL, do NOT generate an email. This is good, because such email would be spammy. It is bad, though, in that gopherbot cannot react quickly.

It would be nice if maintner (https://maintner.golang.org/) could have a cheap way to poll Gerrit for changes. Currently no methods are cheap enough, which is why we do the SMTP server thing.

We resort to polling every 5 minutes or so. (whereby polling == `git ls-remotes`).

In particular, we want to know when the "meta" refs change (`refs/changes/98/103398/meta`).

I know of no Gerrit REST API to get that information cheaply. (and definitely no git API)

I don't mind whether this works at the project level or the server level. Project might be better, as we want to monitor all of go.googlesource.com/* but only part of code.googlesource.com.
Repository: pytorch
Issue: [caffe2] IfOp
Labels: caffe2

Does anybody know how to use the IfOp? Are there any examples?
Repository: pytorch
Issue: [caffe2] Training and inference
Labels: caffe2

Hi,

I have a train net and a validation net declared in the same workspace. It seems that the weights are shared as both nets are in the same workspace. Now, my question is: when applying:
            CAFFE_ENFORCE(workspace.RunNet(trainNet.predict().net().name()));
I'm expecting an update of my weights as I added some training operators.
But when I'm running:
                    CAFFE_ENFORCE(workspace.RunNet(validationNet.predict().net().name()));

Is this going to do a backprop? How to prevent the net from doing a weight update on the validation pass?
Repository: neovim
Issue: cannot change colors of existing :terminal with 'termguicolors'
Labels: enhancement,terminal

<!-- Before reporting: search existing issues and check the FAQ. -->

- `nvim --version`:
```
NVIM v0.2.2
Build type: Release
LuaJIT 2.0.5
Compilation: /usr/local/Homebrew/Library/Homebrew/shims/super/clang -Wconversion -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 -DNVIM_MSGPACK_HAS_FLOAT32 -DNVIM_UNIBI_HAS_VAR_FROM -DNDEBUG -DMIN_LOG_LEVEL=3 -Wall -Wextra -pedantic -Wno-unused-parameter -Wstrict-prototypes -std=gnu99 -Wimplicit-fallthrough -Wvla -fstack-protector-strong -fdiagnostics-color=auto -DINCLUDE_GENERATED_DECLARATIONS -I/tmp/neovim-20180209-92407-udkzoo/neovim-0.2.2/build/config -I/tmp/neovim-20180209-92407-udkzoo/neovim-0.2.2/src -I/usr/local/include -I/usr/local/include -I/usr/local/include -I/usr/local/include -I/usr/local/include -I/usr/local/include -I/usr/local/opt/gettext/include -I/usr/include -I/tmp/neovim-20180209-92407-udkzoo/neovim-0.2.2/build/src/nvim/auto -I/tmp/neovim-20180209-92407-udkzoo/neovim-0.2.2/build/include
Compiled by brew@HighSierra.local

Features: +acl +iconv +jemalloc +tui 
See ":help feature-compile"

   system vimrc file: "$VIM/sysinit.vim"
  fall-back for $VIM: "/usr/local/Cellar/neovim/0.2.2_1/share/nvim"

Run :checkhealth for more info
```
- Vim (version: ) behaves differently? `N/A`
- Operating system/version: macOS 10.13.4
- Terminal name/version: kitty
- `$TERM`: xterm-kitty

### Steps to reproduce using `nvim -u NORC`

```
nvim -u NORC --cmd "set termguicolors"
:terminal
printf '\x1b[31mRED\x1b[0m\n'
<C-\><C-n>:let g:terminal_color_1 = '#654321'
printf '\x1b[31mRED\x1b[0m\n'
```

### Actual and expected behaviour

Terminal colors don't change during terminal lifetime.

### Desired

Per [https://neovim.io/doc/user/nvim_terminal_emulator.html](https://neovim.io/doc/user/nvim_terminal_emulator.html), `g:terminal_color_#' is read only at terminal startup. As expected, attempting to change these variables doesn't cause any color change. Given the discussion at [#4696](https://github.com/neovim/neovim/issues/4696), changes to colors of the underlying terminal emulator also don't propagate into the neovim :terminal.

This makes drastic colorscheme changes impossible. I can use :colorscheme to change from a light background to a dark background neovim but have no way to update terminal colors accordingly, and usually do between day and nighttime. This can make the terminal either extremely unpleasant to use in one of the two cases, or slightly unpleasant to use all of the time.

Is there any way both to use termguicolors in most of neovim while also allowing the terminal to either reflect emulator colors or to have adjustable colors?
Repository: go
Issue: path/filepath: WalkFunc is called with a "file does not exist" error if a file is deleted between readdirnames and lstat (inside walk).
Labels: NeedsInvestigation

Please answer these questions before submitting your issue. Thanks!

### What version of Go are you using (`go version`)?
% go version
go version go1.10 linux/amd64

### Does this issue reproduce with the latest release?
Yes.

### What operating system and processor architecture are you using (`go env`)?
% go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/amistry/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/amistry/go-src"
GORACE=""
GOROOT="/home/amistry/go"
GOTMPDIR=""
GOTOOLDIR="/home/amistry/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 -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build740843021=/tmp/go-build -gno-record-gcc-switches"

### What did you do?
Call filepath.Walk on a directory where files are being constantly deleted (and added).

### What did you expect to see?
No errors.

### What did you see instead?
"file does not exist" errors. If a file doesn't exist, why are you telling me about it?

I guess this is more of a question of semantics. If a file a deleted between the directory listing and lstat, should the caller be told, or should this case be hidden? If a file is deleted before the readdirnames, then there's no error. If a file is added between readdirnames and lstat, it is hidden from the caller. I'd argue that hiding the error is a more consistent and expected.
Repository: pytorch
Issue: [Caffe2] TensorProtosDBInput AttributeError
Labels: caffe2

When I run the `lmdb_create_example.py --output_file ~/git/caffe2/build/caffe2/python/examples/test.lmdb`

I got the below error:

>>> Write database...
Inserted 0 rows
Inserted 16 rows
Inserted 32 rows
Inserted 48 rows
Inserted 64 rows
Inserted 80 rows
Inserted 96 rows
Inserted 112 rows
Checksum/write: 1744642
>>> Read database...
Traceback (most recent call last):
  File "/home/bill/git/caffe2/build/caffe2/python/examples/lmdb_create_example.py", line 107, in <module>
    main()
  File "/home/bill/git/caffe2/build/caffe2/python/examples/lmdb_create_example.py", line 103, in main
    read_db_with_caffe2(args.output_file, checksum)
  File "/home/bill/git/caffe2/build/caffe2/python/examples/lmdb_create_example.py", line 71, in read_db_with_caffe2
    db=db_file, db_type="lmdb")
  File "/home/bill/git/caffe2/build/caffe2/python/model_helper.py", line 436, in TensorProtosDBInput
    return helpers.db_input.db_input(
AttributeError: 'module' object has no attribute 'db_input'

However I check the /git/caffe2/build/caffe2/python/helpers/db_input.db_input is existed.
I'm new in Caffe2. 
Do anyone know how to fix this?



- PyTorch or Caffe2: Caffe2
- OS: Ubuntu 16.04
- PyTorch version:
- How you installed PyTorch (conda, pip, source): source
- Python version: 2.7
- CUDA/cuDNN version: 9.0
- GPU models and configuration: NVRM version: NVIDIA UNIX x86_64 Kernel Module  390.30  Wed Jan 31 22:08:49 PST 2018
- GCC version (if compiling from source): 5.4.0 20160609
- CMake version: 3.5.1
- Build command you used (if compiling from source):
- Versions of any other relevant libraries:

In addition, including the following information will also be very helpful for us to diagnose the problem:
- A script to reproduce the bug. Please try to provide as minimal of a test case as possible.
- Error messages and/or stack traces of the bug
- Context around what you are trying to do
Repository: go
Issue: gccgo: objcopy needs to be required by configure
Labels: NeedsFix

gcc 7.3.0

"objdump" is actually a required executable for building gccgo, as I understand it.
Therefore, it would be nice to flag it as such, in libgo/configure, rather than have the build bomb out later, mysteriously.

Oddly, there is already a line related to it, in configure.ac:

AC_CHECK_TOOL(OBJCOPY, objcopy, missing-objcopy)

but that does not seem to trigger a clear error to the user.

I'm no autoconf expert, but I'm guessing moding the above line as follows might do the trick:
AC_CHECK_TOOL(OBJCOPY, objcopy)
if test "x$OBJCOPY" = "x"; then
  AC_MSG_ERROR([objcopy from GNU binutils required for gccgo])
fi
Repository: three.js
Issue: Spatial Index and Occlusion culling introducing into core
Labels: Suggestion

# Spatial Index
Spatial index is necessary when dealing with large scenes, such large scenes are very common in games for example. 
### Motivation
If you want to do a raycast into the scene - currently you are stuck with a linear search, which is dominated by number of objects and polygons. A spatial index would enable a lot of internal optimizations, such as faster occlusion culling and sorting.

_Direct application in the renderer:_
# Occlusion culling
Good occlusion culling is required for good performance. Point above would help here. There are a lot of techniques that can be utilized further here.
 
born as a result of this discussion: #13807
Repository: three.js
Issue: Optimizations to Animation Engine
Labels: Suggestion

# Optimizations to animation engine
Currently animation engine chokes on some 500 bones being animated simultaneously, resulting in a very high CPU usage. 
### Motivation
It is not uncommon to see 3-5 characters at the same time with 500+ bones each in modern games, with current CPU demand such fidelity is not achievable, instead you have to compromise to about 15 bones per character in order to achieve decent performance.

born as a result of this discussion: #13807
Repository: three.js
Issue: Support of Compressed Textures in core
Labels: Suggestion

# Compressed Textures
Compressed textures as a first-class citizen, along with tools for on-line compression
### Motivation
Compressed textures offer a great amount of extra detail requiring only a little space, for applications with large textures and/or large number of textures, this draws a line between interactive frame-rate and a slide-show, this point becomes more relevant for lower-end GPUs, as they tend to have less RAM, being able to draw 2024 compressed textures instead of 512 uncompressed ones is a extremely important, as they take up potentially the same amount of GPU RAM. Compressed textures take less time to load and put less stress on browser, since decompression is not done by default (unlike PNG).

born as a result of this discussion: #13807
Repository: opencv
Issue: ORB detector crashed
Labels: bug,category: features2d,incomplete,needs reproducer

##### System information (version)
- OpenCV => 3.4.1 dev
- Operating System / Platform => Windows 64 Bit/32bit (check both)
- Compiler => Visual Studio 2017

##### Detailed description
// C++ code example
https://docs.opencv.org/3.4.1/dc/d16/tutorial_akaze_tracking.html

Can`t use ORB detector in demo and own application.
In all cases crash in the file "orb.cpp",  function: static void computeKeyPoints()

909:        std::swap(allKeypoints, newAllKeypoints);
910:   }  <-- here, after exit from cycle

Debugged application stoped with error message:
Windows has triggered a breakpoint in stack.exe.
This may be due to a corruption of the heap, which indicates a bug in stack.exe or any of the DLLs it has loaded.

Stoped here: file xmemory0
 #endif /* defined(_M_IX86) || defined(_M_X64) */
-->	::operator delete(_Ptr, _Bytes);
	}

Also impossible use AKAZE detector.
Detector working, if call function detectAndCompute(), and when application close, get error msg from:
             /* verify block type */
            _ASSERTE(_BLOCK_TYPE_IS_VALID(pHead->nBlockUse));  

If not call  detectAndCompute(), all ok.
Repository: opencv
Issue: imshow upscale displayed image
Labels: feature,category: highgui-gui,platform: win32

Using the latest OpenCV on Windows 10, any image I show or any options I use for creating the window (autosize, or normal with specified size) the resulted window displays a scaled version of the image. The scale seems to correspond to the general scale set in Windows display settings. Is there a way to disable that scale, at least when manually specifying the windows size? It is very important to preserve the original resolution while developing computer vision algorithms.
Repository: rust
Issue: PathBuf set_file_name and with_file_name need docs for input abspaths
Labels: C-enhancement,T-libs-api,A-io

Looking at:

https://doc.rust-lang.org/std/path/struct.PathBuf.html#method.with_file_name

and

https://doc.rust-lang.org/std/path/struct.PathBuf.html#method.set_file_name

I had thought, based on what was written there, that the code would extract the filename alone from the input, dropping the directory prefix.

But apparently what will actually happen is that it drops all the content from `self` and just turns it into a copy of the input.

(And if you give it a relative path with multiple components, then it effectively pushes all of the components onto `self`)


I don't actually *mind* the current behavior, and it is easy enough to workaround myself. (Basically, I think `set_file_name` and `with_file_name` are somewhat misleadingly misnamed...) But we should document it it, perhaps with concrete examples.

Here is an example of what I am getting at ([playpen](https://play.rust-lang.org/?gist=fc97d1c694aa12eb8e3586e0ca2bca31&version=stable)):

```rust
fn main() {
    use std::path::Path;

    println!("EXAMPLE 1");
    let path1 = Path::new("/dir1/file1.txt");
    let path2 = Path::new("/dir2/file2.txt");
    println!("path1: {}", path1.display());
    println!("path2: {}", path2.display());
    println!("path1.with_file_name(path2): {}", path1.with_file_name(path2).display());
    println!("Felix expected /dir1/file2.txt");
    
    println!("");
    println!("EXAMPLE 2");
    let path1 = Path::new("dir1/file1.txt");
    let path2 = Path::new("dir2/file2.txt");
    println!("path1: {}", path1.display());
    println!("path2: {}", path2.display());
    println!("path1.with_file_name(path2): {}", path1.with_file_name(path2).display());
    println!("Felix expected dir1/file2.txt");
}

```
Repository: vscode
Issue: Licensing unnecessarily prohibits usage in cloud services
Labels: under-discussion,license

I've been using a cloud service provider for remote work.  In their service they provide Atom as a text editor, but I've been using Code for about a year now and am loathe to switch over my development environment.

They've bumped on something in the licensing.  They cannot:
> share, publish, or lend the software, or provide it as a hosted solution for others to use, or transfer the software or this agreement to any third party.

After looking around at all the confusion over Code's license I can understand the reasons Microsoft has the dual licensing system.  (I don't agree, but I understand.)  I even built a copy of Code from source to see if that would be usable (verdict is still out there).  But simpler would be for Microsoft to strike this line from the software license.  Leave in the non-transferable bit and cut out the rest. This would allow cloud services to provide Code as the de-facto editor, which can only benefit you in the long run by increasing user count and brand awareness.

The thing is, I'm not sure why this is even in here.  In my opinion it fundamentally goes against the branding that Microsoft has given code, which is that it's free for everyone and usable everywhere.  It feels like a hangover from the commercial version of the VS IDE, or from Windows itself, some boilerplate that gets tacked on automatically.

The alternative is to have people build Code from source, which removes all MS branding and telemetry. I'd think this less desirable for you.

My other thought is that they could provide a pop-up that offers to install it for the user, something akin to how Code is now being distributed with Anaconda.  Anaconda's use of the official installer seems to bypass this clause in the license.  However, this is very much *not* the solution I want, because the last thing the world needs are more pop-ups. :)

Thanks for listening.