WhereIsAI/github-issue-similarity huggingface dataset:
Issue 1:
React version: 17.0.2

Steps To Reproduce

Make nested array element like this. All element have unique, fixed key.

[Element0, [Element1, Element2]]

Change the element array.

[Element1, Element2]

Key of element 1 and 2 aren't changed, but the elements are re-mounted.

This is not a problem if element 1 and 2 aren't in nested array.

[Element0, Element1, Element2]

-> [Element1, Element2]

Link to code example: https://github.com/jwoo0122/nested-child

Clone this repo, open dev tools and see console. In App.js, I made some scenarios to test this behavior.

The current behavior

The element with same key always unmount, if they are in nested array.

The expected behavior

I have no idea whether this behavior is intended or kind of bug. But I expected that element have same key in same array (also in nested array) do not re-mount, just re-render. Please feedback if my expectation is not correct. Thank you.

Issue 2:
👉 Please follow one of these issue templates:

https://github.com/facebook/react/issues/new/choose

Note: to keep the backlog clean and actionable, issues may be immediately closed if they do not follow one of the above issue templates.
Issue 1:
An error message Uncaught TypeError: Cannot call method 'remove' of undefined in the console when trying to close a modal initialized without a backdrop.

jsfiddle

Issue 2:
With options { backdrop: false }, this.$backdrop.remove() is still being called which results in an exception as this.$backdrop is null.

hideModal: function () {
var that = this
this.$element.hide()
this.backdrop(function () {
that.removeBackdrop()
that.$element.trigger('hidden')
})
}

removeBackdrop: function () {
this.$backdrop.remove()
this.$backdrop = null
}
Issue 1:
Electron version: 1.3.3

Operating system: Windows

I'm reopening this issue (see #3494). @zcbenz closed it, citing that macOS doesn't support hiding the separator. However; it seems to be working on El Capitan (10.11.16).

On OSX, I have a menu that collapses as shown (when a user disables SSL):

On Windows, it looks like this:

The code responsible for this action looks like:

sslmenu.items.forEach((item) => {
item.visible = assets.CA.enabled
})

Clearly, changing the visibility of a separator on OSX works while Windows does not.

Issue 2:
OS X does this for us, but Windows and Linux does not. There's absolutely no reason to have leading and trailing separators and it's difficult to not introduce leading/trailing separators in complex context menus where multiple actors modify it. Especially when using the visible option to toggle items as the separator is not connected to that. I think Electron should handle this for us.

const electron = require('electron');

electron.app.on('ready', () => {
electron.Menu.setApplicationMenu(electron.Menu.buildFromTemplate([{
submenu: [{
type: 'separator'
}, {
label: 'Foo'
}, {
type: 'separator'
}]
}]));

(new electron.BrowserWindow())
.loadURL(`https://github.com`);
});

OS X

#### Windows

##

This includes cases like this, which is annoying to have to handle manually:

[ { label: 'Unicorn' },
{ type: 'separator' },
{ label: 'Cut', role: '', visible: false },
{ label: 'Copy', role: '', visible: false },
{ label: 'Paste', role: '', visible: false } ]

##
- Electron version: 1.2.1
- Operating system: OS X 10.11.5
Issue 1:
VSCode Version: 1.0

OS Version: Windows

In LICENSE file it says that VS Code is licensed under MIT. However, when installing it, you have to agree to a completely different license, which says that you can't not only "reverse engineer, decompile or disassemble the software" but also "share, publish, or lend the software". In my opinion it quite contradicts the MIT license:

Permission is hereby granted, free of charge, to any person obtaining a copy of this software and associated documentation

files (the "Software"), to deal in the Software without restriction, including without limitation the rights to use, copy,

modify, merge, publish, distribute, sublicense, and/or sell copies of the Software, and to permit persons to whom the Software

is furnished to do so, subject to the following conditions:

The above copyright notice and this permission notice shall be included in all copies or substantial portions of the Software.

THE SOFTWARE IS PROVIDED AS IS, WITHOUT WARRANTY OF ANY KIND, EXPRESS OR IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES

OF MERCHANTABILITY, FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE AUTHORS OR COPYRIGHT HOLDERS

BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM, OUT

OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE SOFTWARE.

Should it be like that or is it a mistake?

Issue 2:
Today we have an interface that describes the contract of telemetry (ITelemetryService) which is OK. The bad thing is that there many implementations of that interface (AbstractTS, MainTS, ElectronTS, AbstractRemoteTS, WorkerTS, ExtHostTS, NullTS, MockTS, TestTS) which have led to leakage and duplication of events. I believe this problem should be solved with less code and abstraction
Issue 1:
虽然是日志打印不对，但具体里面有没其他bug不清楚，麻烦检查下是否是版本号弄错了，非常感谢，2.5.7及之前的日志都是正确的。

Issue 2:
The Dubbo version report to registry is 2.0.1 after dubbo 2.5.9 and 2.5.10.

It should be the same as the dubbo version i think.
Issue 1:
On Android you can double tap a word in a text field to select it, but in a flutter TextField this doesn't seem to work. You need to long press to select it.

Steps to Reproduce

Add a new TextField

Write some words

Double tap a word

Issue 2:
Steps to Reproduce

Tap in a text field in a Flutter app.

Enter some text with spaces; e.g., foo bar baz.

Double-tap on a word.

On iOS, the double-tapped word should be selected. This behaviour is iOS-specific and does not apply on Android.

The double-tapped word should be selected.

Flutter Doctor

[✓] Flutter (on Mac OS X 10.12.4 16E195, channel master)

• Flutter at /Users/cbracken/src/flutter/flutter

• Framework revision 1b948cb (2 days ago), 2017-04-05 12:27:59 -0700

• Engine revision 5d9a642

• Tools Dart version 1.23.0-dev.11.5
Issue 1:
ES6 supports duplicate properties in object literals even in strict mode, but some (recent versions of) browsers still fail to parse strict mode code that has object literals with duplicate properties. Chrome 38 and Safari 8 are two browsers like this.

Currently, the following code:

var x = {o: 5, o: 6};

transpiles to

"use strict";

var x = { o: 5, o: 6 };

which entirely fails to parse in some browsers.

Could babel transpile code with duplicate properties in object literals as defining an object literal with no duplicate properties, and then assigning further properties, similar to how computed keys are handled? So that way

var x = {o: 5, o: 6};

and

var x = {o: 5, ["o"]: 6};

would both transpile to the same following thing, which works in all browsers:

"use strict";

function _defineProperty(obj, key, value) { if (key in obj) { Object.defineProperty(obj, key, { value: value, enumerable: true, configurable: true, writable: true }); } else { obj[key] = value; } return obj; }

var x = _defineProperty({ o: 5 }, "o", 6);

Issue 2:
The following ES6 code:

var x = {x: 1, x: 2};

generates the following code:

"use strict";

var x = { x: 1, x: 2 };

which will result in an error for ES5-compliant engines. I'm not sure what should be done here instead as duplicate keys are back to being valid in strict mode in ES6, but maybe something like this:

"use strict";

var x = { x: 1 };
x.x = 2;

This bug also occurs with JSX:

var x = <x x={1} x={2} />;

becomes

"use strict";

var x = React.createElement("x", { x: 1, x: 2 });
Issue 1:
The following code (as of commit 0012b50) gives an internal compiler error drop_ty_immediate: non-box ty.

use std::rt::rtio::RtioTimer;
use std::rt::io::Timer;

fn main() {
let maybe_timer = Timer::new();
maybe_timer.expect("Expected a timer").sleep(1);
}

Issue 2:
It seems like this was broken as a result of the recent change to make newtypes structs immediate in certain cases.

struct TestNode(@int);

fn mm(_x: &TestNode) {}

fn main() {
let x = &TestNode(@21);
mm(x);
}

% rustc foo.rs
error: internal compiler error: drop_ty_immediate: non-box ty
Issue 1:
When trying to run "superset db upgrade" I get an error message.

Expected results

I was expecting a successful message and program ready to use.

Actual results

sqlalchemy.exc.OperationalError: (sqlite3.OperationalError) duplicate column name: filter_type [SQL: ALTER TABLE row_level_security_filters ADD COLUMN filter_type varchar(255)]

Screenshots

How to reproduce the bug

Run 'sudo apt-get install build-essential libssl-dev libffi-dev python-dev python-pip libsasl2-dev libldap2-dev'

Run 'pip3 install virtualenv'

Run 'python3 -m venv venv'

Run '. venv/bin/activate'

Run 'pip3 install apache-superset'

Run 'pip3 install sqlalchemy==1.3.24' (had to downgrade existing version)

Run 'superset db upgrade'

'See error

Environment

Ubuntu 18.04 LTS;

superset version: 0.38.1

python version: 3.6.9

node.js version: 8.10.0

SQLAlchemy 1.3.24

Checklist

I have checked the superset logs for python stacktraces and included it here as text if there are any.

I have reproduced the issue with at least the latest released version of superset.

I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context

I was able to complete the following tasks:

i) install all required dependencies;

ii) install and start python virtual environment.

This is a virtual machine.

Issue 2:
We have a query that returns timeseries data with two columns:

date

count

I would like to simply graph the data as a line chart. However, when selecting the line chart, it requires that I choose a metric for the data, e.g. sum, average, etc

Since my data are already in the desired form, how can I just tell the chart to use the data as-is?

Note: I can also select the "sum" metric, since I am not performing any additional aggregation, but this seems a bit "kludgy".
Issue 1:
Challenge Accessing Objects Properties with Variables has an issue.

User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/56.0.2924.87 Safari/537.36.

Please describe how to reproduce this issue, and include links to screenshots if possible.

Code:

// Setup
var testObj = {
12: "Namath",
16: "Montana",
19: "Unitas"
};
// Only change code below this line;
var playerNumber; // Change this Line
var player = testObj; // Change this Line

If you use this code:

var playerNumber = 16;// Change this Line
testObj[playerNumber]
var player; // Change this Line

The challenge will pass.

...

...

However, the correct response is:

var playerNumber = 16;// Change this Line
var player = testObj[playerNumber]; // Change this Line

Issue 2:
Challenge Name

Accessing Objects Properties with Variables

Issue Description

https://www.freecodecamp.com/challenges/accessing-objects-properties-with-variables#?solution=%0A%2F%2F%20Setup%0Avar%20testObj%20%3D%20%7B%0A%20%2012%3A%20%22Namath%22%2C%0A%20%2016%3A%20%22Montana%22%2C%0A%20%2019%3A%20%22Unitas%22%0A%7D%3B%0A%0A%2F%2F%20Only%20change%20code%20below%20this%20line%3B%0A%0Avar%20playerNumber%20%3D%2016%3B%2F%2F%20Change%20this%20Line%0AtestObj%5BplayerNumber%5D%0Avar%20player%20%3D%20%22Montana%22%3B%20%20%20%2F%2F%20Change%20this%20Line%0A

Browser Information

Browser Name, Version: latest firefox browswer

Operating System: windows 10

Mobile, Desktop, or Tablet: desktop

Your Code

// Setup
var testObj = {
12: "Namath",
16: "Montana",
19: "Unitas"
};

// Only change code below this line;

var playerNumber = 16;// Change this Line
testObj[playerNumber]
var player = "Montana"; // Change this Line

Screenshot
Issue 1:
Feature request

Implement NodeJS v12+ internal package.json imports that only apply to import specifiers from within the package itself.

NodeJS documentation: https://nodejs.org/api/packages.html#packages_subpath_imports

What is the expected behavior?

Entries in the imports field must always start with # to ensure they are disambiguated from package specifiers.

For example, the imports field can be used to gain the benefits of conditional exports for internal modules:

// package.json
{
"imports": {
"#dep": {
"node": "dep-node-native",
"default": "./dep-polyfill.js"
}
},
"dependencies": {
"dep-node-native": "^1.0.0"
}
}

where import '#dep' does not get the resolution of the external package dep-node-native (including its exports in turn), and

instead gets the local file ./dep-polyfill.js relative to the package in other environments.

Unlike the "exports" field, the "imports" field permits mapping to external packages.

The resolution rules for the imports field are otherwise analogous to the exports field.

What is motivation or use case for adding/changing the behavior?

Maintaining interoperability with node package.json specifications.

This feature dis-incentive the use of subpath exports + self-referencing package for internal access. It offers a clean way to replace these import { X } from "../../../lib", (e.g: by import { X } from "#lib").

How should this be implemented in your opinion?

No idea, but looks like webpack subpath exports already implement most of the feature.

Are you willing to work on this yourself?

no

Issue 2:
Do you want to request a feature or report a bug?

Bug.

What is the current behavior?

When using any devtool strategy that uses eval, such as cheap-module-eval-source-map, the browser developer tools show two sets of files with quasi-identical names. The first set under the webpack:// origin contains original sources. The second set under webpack-internal:// origin contains transpiled sources.

Note that this has changed in webpack 4. In the previous version that webpack-internal:// files used numeric identifiers, so there was no risk to open them by accident and they did not show up when using Ctrl+P in Chrome.

If the current behavior is a bug, please provide the steps to reproduce.

git clone https://github.com/lorenzodallavecchia/webpack-bug-duplicatesource.git
cd webpack-bug-duplicatesource
npm install
npm start

What is the expected behavior?

I would expect to only see original sources (under webpack://) or at least give the "internal" sources a numeric identifier so that they do not show up when searching with Ctrl+P.

Please mention other relevant information such as the browser version, Node.js version, webpack version, and Operating System.

webpack 4

Node 6.11.0

Chrome 65

Windows 10 1709
Issue 1:
VSCode Version: Version 1.0.0-alpha Commit 6ea1266

OS Version: Ubuntu 14.04

Steps to Reproduce:

Compile https://gist.github.com/edumunoz/6e2ca90d4b5d3dbfa5340576e79ac113 with g++ -g -std=c++11 -pthread hello.cpp -o hello

Install the C++ debug adapter

Set a breakpoint in line 17 from the gist

Have the main thread expanded in the callstack viewlet

Expand another thread and click on a frame for which no source is available

The "Load More Stack Frames" button appears for the main thread

The "Load More Stack Frames" button is clicked, the callstack for the main thread gets duplicated (two main() in the screenshot)

Step 3

Step 5

Step 6

I understand if the debug adapter implements the "Load More Frames" feature, this will work. However the experience seems kind of weird if the debug adapter doesn't implement the feature (like C++ debug adapter as of now).

@isidorn @weinand

Issue 2:
VSCode Version: 0.10.11

OS Version: OS X 10.11.5

Steps to Reproduce:

Open a file and change its language.

Open the file in the git changes view.

The old version will be displayed in the default language for the file, ignoring the user selection.
Issue 1:
Currently multifield mapping support path parameter per property so if you need to have several fields mapped for a property they will either all have full path name or just_name

it is rather inconvenient when you want to have the property with say two secondary fields one with full name (because it only makes sense as a variant of primary field say not analyzed) and one with just_name because you want to have an all-like field to which many of your properties contribute.

consider an example (a part of a bigger json)

"category": {"code":"CTZ", "description":"My Description"}

code was indexed as multifield resulting in names

category.code

category.code.untouched

later I want to have my_all field to where I want to index category.code as well as other fields

I will add path:"just_name" to my mapping and another field my_all

that will immediately break my application because untouched will become a just_name mapping as well and all untouched from all my data elements will be rolled into it

My current workaround is to provide full name for untouched field (category.code.untouched) so it retains its full name. I am not sure it is intentional behavior but it seems to work (need to test it more)

But much cleaner approach would be to allow path per field in multifield mapping

Issue 2:
Hi,

I have indexed documents that have a duplicate field. During the search I want to display only the document that has the highest score among its duplicate class (the group of documents that have the same field value).

There has been a discussion about deduplication in Lucene mailing-list:

http://www.mail-archive.com/java-user@lucene.apache.org/msg21437.html

Would it be possible to implement the Duplicate filter which is in contrib:

https://svn.apache.org/repos/asf/lucene/dev/trunk/lucene/contrib/sandbox/src/test/org/apache/lucene/sandbox/queries/DuplicateFilterTest.java

So the request would look like:

{
"query": {
"term": { "name": "alexis" }
},
"filter": {
"duplicate": { "field": "name" }
}
}

Let me know if that makes sense...
Issue 1:
Migrated issue, originally created by florent breheret (@florentbr)

python 2.7.8, firebird 2.5.3, sqlalchemy 0.9.8, fdb 1.4.3

For a table with a check constraint, I noticed that this check constraint is missing in the reflected metadata.

The test case to reproduce:

import unittest
from sqlalchemy import *

class TestSuite(unittest.TestCase):

def setUp(self):
self.engine = create_engine('firebird+fdb://SYSDBA:masterkey@/c:/unitest.fdb')
self.connection = self.engine.connect()

def tearDown(self):
self.connection.execute("DROP TABLE TMP_TABLE")
self.connection.close()

def test_check_constraint_reflected(self):
table = Table('TMP_TABLE', MetaData(bind=self.connection),
Column('dt', Integer),
CheckConstraint('dt > 1')
)
table.create(self.engine)
reflected_constraints = MetaData(bind=self.connection, reflect=True).tables['tmp_table'].constraints
self.assertItemsEqual(
[c.__class__.__name__ for c in table.constraints],
[c.__class__.__name__ for c in reflected_constraints]
)

if __name__ == '__main__':
unittest.main()

Issue 2:
Migrated issue, originally created by Anonymous

I'm not sure about other DB backends, but postgres supports also unique/check constraints to be inspected on db. Any change to get this into SA?
Issue 1:
TypeScript Version: 1.9.0-dev.20160523-1.0

Code

// A self-contained demonstration of the problem follows...

class Foo {

identity<T>(x: T): T {
return x
}

compose<A, B, C>(f: (b: B) => C, g: (a: A) => B) : (a:A) => C {
return (a: A) => f(g(a))
}

constructor() {
let value = this.compose((a : number) => a, this.identity)
}
}

Expected behavior:

value should have type (a: number) => number

OR

it should issue an error as the tsconfig.json has "noImplicitAny" defined to true.

Actual behavior:

value has type (a: {}) => number

AND

no error from the compiler.

Issue 2:
I'm using TypeScript 1.6.3 in Visual Studio 2015. The es6-promise declaration I'm using contains:

declare class Promise<R> implements Thenable<R> {
constructor(callback: (resolve : (value?: R | Thenable<R>) => void, reject: (error?: any) => void) => void);
then<U>(onFulfilled?: (value: R) => U | Thenable<U>, onRejected?: (error: any) => U | Thenable<U>): Promise<U>;
}

The following works with an explicit typing of the Promise generic:

new Promise < { foo: number }>((resolve) => {
resolve({ foo: 1 });
}).then((value) => {
value.foo;
});

However, shouldn't type inference allow the following to work?

new Promise((resolve) => {
resolve({ foo: 1 });
}).then((value) => {
value.foo;
});

Instead, I am getting an error when accessing value.foo that foo does not exist on type '{}'. Why is TypeScript defaulting to the {} type?
Issue 1:
https://deno.land/std/manual.md#typescript-compiler-options

Issue 2:
Manual link https://deno.land/std/manual.md on the page https://deno.land/ forwards to https://github.com/denoland/deno/tree/master/docs.

I assume because there is no manual.md file in the docs folder.

As far as I remember a few days back the md files' content was shown under each other.
Issue 1:
I've upgraded caravel to 1313727 and on a sunburst slice i get this:

Traceback (most recent call last):
File "/home/rm/caraveltest/venv/lib/python3.4/site-packages/pandas/core/internals.py", line 3600, in set
loc = self.items.get_loc(item)
File "/home/rm/caraveltest/venv/lib/python3.4/site-packages/pandas/indexes/base.py", line 1985, in get_loc
return self._engine.get_loc(self._maybe_cast_indexer(key))
File "pandas/index.pyx", line 137, in pandas.index.IndexEngine.get_loc (pandas/index.c:4154)
File "pandas/index.pyx", line 159, in pandas.index.IndexEngine.get_loc (pandas/index.c:4018)
File "pandas/hashtable.pyx", line 675, in pandas.hashtable.PyObjectHashTable.get_item (pandas/hashtable.c:12368)
File "pandas/hashtable.pyx", line 683, in pandas.hashtable.PyObjectHashTable.get_item (pandas/hashtable.c:12322)
KeyError: 'm1'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
File "/home/rm/caraveltest/venv/lib/python3.4/site-packages/caravel/views.py", line 799, in explore
payload = obj.get_json()
File "/home/rm/caraveltest/venv/lib/python3.4/site-packages/caravel/viz.py", line 275, in get_json
'data': self.get_data(),
File "/home/rm/caraveltest/venv/lib/python3.4/site-packages/caravel/viz.py", line 1288, in get_data
ndf['m1'] = df[metric]
File "/home/rm/caraveltest/venv/lib/python3.4/site-packages/pandas/core/frame.py", line 2357, in __setitem__
self._set_item(key, value)
File "/home/rm/caraveltest/venv/lib/python3.4/site-packages/pandas/core/frame.py", line 2424, in _set_item
NDFrame._set_item(self, key, value)
File "/home/rm/caraveltest/venv/lib/python3.4/site-packages/pandas/core/generic.py", line 1465, in _set_item
self._data.set(key, value)
File "/home/rm/caraveltest/venv/lib/python3.4/site-packages/pandas/core/internals.py", line 3603, in set
self.insert(len(self.items), item, value)
File "/home/rm/caraveltest/venv/lib/python3.4/site-packages/pandas/core/internals.py", line 3704, in insert
placement=slice(loc, loc + 1))
File "/home/rm/caraveltest/venv/lib/python3.4/site-packages/pandas/core/internals.py", line 2619, in make_block
return klass(values, ndim=ndim, fastpath=fastpath, placement=placement)
File "/home/rm/caraveltest/venv/lib/python3.4/site-packages/pandas/core/internals.py", line 90, in __init__
len(self.mgr_locs)))
ValueError: Wrong number of items passed 2, placement implies 1

Issue 2:
Is your feature request related to a problem? Please describe.

We would like to have a feature that allow users to duplicate dashboard, and optionally duplicate (deep copy of ) charts associated with the dashboard.

Describe the solution you'd like

It would be great to have such feature as one of the items in the Action column in the dashboard list page (/dashboard/list/).

Describe alternatives you've considered

Also, it would be great to have such icon next to the report scheduling icon and before the 3 dots in the upper right corner of the single dashboard page, or put such icon in both dashboard list page and the single dashboard page.

Additional context

Add any other context or screenshots about the feature request here.