WhereIsAI/github-issue-similarity huggingface dataset:
Issue 1:
Granularity.getIterable(Interval) returns an iterable of chunked intervals by the granularity. For example, DAY.getIterable(Intervals.of("2019-01-01/2019-01-03")) returns 2019-01-01/2019-01-02 and 2019-01-02/2019-01-03. This method is currently being used in indexing service (UniformGranularitySpec) as well as query processing (makeCursors() in QueryableIndexStorageAdapter and IncrementalIndexStorageAdapter).

But, the current behavior of this method is somewhat different from my expectation. The endTime of the last interval is not bound by the input interval. For example, DAY.getIterable(Intervals.of("2019-01-01T00:00:00/2019-01-03T12:00:00")) returns the below 3 intervals.

2019-01-01T00:00:00.000Z/2019-01-02T00:00:00.000Z
2019-01-02T00:00:00.000Z/2019-01-03T00:00:00.000Z
2019-01-03T00:00:00.000Z/2019-01-04T00:00:00.000Z

The last interval is 2019-01-03/2019-01-04 is not contained the input interval 2019-01-01T00:00:00/2019-01-03T12:00:00.

Sometimes, even the startTime of the first interval is not bound too. WEEK.getIterable(Intervals.of("2019-01-01/2019-01-11")) returns

2018-12-31T00:00:00.000Z/2019-01-07T00:00:00.000Z
2019-01-07T00:00:00.000Z/2019-01-14T00:00:00.000Z

This is because PeriodGranularity sets the start day to Monday.

I'm not sure this makes sense. The query result may contain timestamps of not specified in the query spec which can mislead users. This can be even worse when it comes to indexing because reindexing tasks can overshadow segments unexpectedly.

I think probably it's better to clip off the end time of the last interval (and maybe the start time of the first interval too), but not sure what was the initial intention.

Any thoughts?

Issue 2:
Description

Protobuf has a native timestamp type, defined here.

Motivation

Timestamps being passed around in protobuf bytes are using the native timestamp format and moving these to druid ingestion necessitates the duplication of that timestamp in a string.
Issue 1:
This would be a breaking change, so probably it will be rejected, but anyway:

The function call searchsorted(v, key, by=mytransform) applies mytransform to key, which isn't helpful. It would be more useful if the searchsorted function assumes that the key is already of the type equal to the output of mytransform. The common use for searchsorted is to search the sorted array v for a particular key that is a field or is computed from the fields of the array entries. Typically, the user would know the key but would not want to construct a dummy record whose type is eltype(v) just to hold that key.

Just to be clear, here is a trace of what I mean. I think the first invocation sequence (that gives the error) is more useful than the second.

julia> v = [(2,5), (3,9), (12,15)]
3-element Array{Tuple{Int64,Int64},1}:
(2, 5)
(3, 9)
(12, 15)

julia> searchsortedfirst(v, 7, by = a-> a[2])
ERROR: BoundsError
Stacktrace:
[1] getindex at .\number.jl:78 [inlined]
[2] #5 at .\REPL[4]:1 [inlined]
[3] lt at .\ordering.jl:51 [inlined]
[4] searchsortedfirst at .\sort.jl:178 [inlined]
[5] searchsortedfirst at .\sort.jl:291 [inlined]
[6] #searchsortedfirst#4 at .\sort.jl:293 [inlined]
[7] (::getfield(Base.Sort, Symbol("#kw##searchsortedfirst")))(::NamedTuple{(:by,),Tuple{getfield(Main, Symbol("##5#6"))}}, ::typeof(searchsortedfirst), ::Array{Tuple{Int64,Int64},1}, ::Int64) at .\none:0
[8] top-level scope at REPL[4]:1

julia> searchsortedfirst(v, (99999,7), by = a-> a[2])
2

Issue 2:
Consider this:

using Base.Dates: Second

immutable tsp # time series point
t :: DateTime
y :: Float64
end

t0 = now()
S = [tsp(t0+Second(10), 42), tsp(t0+Second(13), -1), tsp(t0+Second(17), 31)]

searchsortedfirst(S, t0+Second(12), by=x -> x.t)
searchsortedfirst(S, tsp(t0+Second(12), NaN), by=x -> x.t)

The first search call fails and the second is not ideal because one has to use a dummy value (NaN here) to fill in the parts of the item being searched that are not part of the key. I think that in searchsortedfirst(S, x, by=f) it would make more sense to apply f() to the items of S but not to the item x. If the searchsorted functions worked like that, then the current behavior is easily obtained by using searchsortedfirst(S, f(x), by=f). Inversion of f() is not necessarily as easy.
Issue 1:
Hello,

For now, when you add a lot of items to the statusbar via plugins, only first few items are displayed and the remaining items are hidden.

It would be great to have a dropdownlist which will show all those remaining items.

I hope such a feature could enhance vscode.

Thanks in advance for your comments.

Issue 2:
Implementing a block cursor is maybe not as straightforward as first seemed. When there are active selections and the cursor type is a block cursor, the column that the cursor is shown to occupy is supposed to be considered part of the selection.

STR

With a block cursor enabled, type "foo bar baz" in a new document

Using the mouse, try to select the entire word "foo"

Copy and paste it into some scratch space

Move the cursor over the "b" in "bar"

Using Shift+Right Arrow, select the entire word

Expected Results

Pasting in step 3 pastes the entire word "foo", and selecting the word "bar" with the keyboard in step 5 should leave the block cursor blinking over the "r", or alternatively, leave no visible trace of the cursor at all, while the selection is active.

Actual Results

The paste step in 3 only pastes "fo", and using the keyboard for selection in step 5 hints towards why: after selecting "bar", the block cursor is shown hovering over the space that follows instead of the "r" at the end of the word.
Issue 1:
Hi,

I know the Context is flagged as an experimental feature hence why it is missing from the Top-Level API but I find myself having to pass context to a legacy-compatible component.

ReactDOM.render(React.createElement(DialogComponent, {id: id, component: this.component, childProps: this.props}), document.getElementById(id));

React.createElement() does not allow for context setting and this is causing me to 'hack' a context in these cases by doing something like

componentWillMount: function() {
this.context = this.props.context;
},

in a mixin.

Is it something that could be looked into?

Thanks!!

Issue 2:
I happened to notice that React will throw Hooks can only be called inside the body of a function component if a sibling element calls ReactDOMServer.renderToStaticMarkup. I have a fiddle that shows this behavior (I think you need to manually open your console to see the error).

If you wrap the sibling element in a component or even just assign it a variable, this issue does not occur.

It's easy to get around and is a pretty specific bug, but why might this be happening?
Issue 1:
I have searched the issues of this repository and believe that this is not a duplicate.

I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.7.4

Operating System version: macos 10.13.6

Java version: 1.8

Steps to reproduce this issue

add unit test in ProtostuffObjectOutputTest.class and failed:

@Test
public void testListObject() throws IOException, ClassNotFoundException {
List<SerializablePerson> list = new ArrayList<SerializablePerson>();
list.add(new SerializablePerson());
list.add(new SerializablePerson());
list.add(new SerializablePerson());
SerializablePersonList personList = new SerializablePersonList(list);
this.protostuffObjectOutput.writeObject(personList);
this.flushToInput();

SerializablePersonList serializedTime = protostuffObjectInput.readObject(SerializablePersonList.class);
assertThat(serializedTime, is(personList));
}

private class SerializablePersonList implements Serializable {
private static final long serialVersionUID = 1L;

public List<SerializablePerson> personList;

public SerializablePersonList() {}

public SerializablePersonList(List<SerializablePerson> list) {
this.personList = list;
}

@Override
public boolean equals(Object obj) {
if (this == obj)
return true;
if (obj == null)
return false;
if (getClass() != obj.getClass())
return false;

SerializablePersonList list = (SerializablePersonList) obj;
if (list.personList == null && this.personList == null)
return true;
if (list.personList == null || this.personList == null)
return false;
if (list.personList.size() != this.personList.size())
return false;
for (int i =0; i < this.personList.size(); i++) {
if (!this.personList.get(i).equals(list.personList.get(i)))
return false;
}
return true;
}
}

This is one case, and I have found the reason and raised a pr to fix it, But there is another stackoverflow still exist.

Pls. provide [GitHub address] to reproduce this issue.

Expected Result

What do you expected from the above steps？

test success

Actual Result

What actually happens?

test failed

If there is an exception, please attach the exception trace:

Just put your stack trace here!

Issue 2:
I find that there are much duplicate code in the method afterPropertiesSet of ServiceBean and ReferenceBean. Can we move them to their common super class ?
Issue 1:
as showing in screenshot, Dropdown menu is not showing in IE 9

http://www.imageox.com/v/ddea3e2f20a0.png

IE version is 9.0.8112 64 bit on windows 7

Issue 2:
I'm having a problem which has only cropped up since upgrading from 2.1.0 to 2.1.1

A fixed-nav dropdown on IE7 and IE8 now doesn't seem to appear.

The root of the problem seems to be the overflow: visible !important rule in the respnosive stylesheet is not being read by IE7 and IE8 as it's within a media query they ignore. So the dropdown menu does not overflow the navbar and therefore barely appears.

Anyone else had / noticed this issue?
Issue 1:
Environment

Windows build number: Microsoft Windows [Version 10.0.20150.1000]
PowerToys version: v0.18.2
PowerToy module for which you are reporting the bug (if applicable): Keyboard Manager

Steps to reproduce

Remap Win (Left) to Alt (Left)

Remap Alt (Left) to Win (Left)

Press the newly remapped Win (Left) + R. Game Recording opens instead of the Run dialog.

Expected behavior

Win + R should open the Run dialog. It works fine if I deactivate the Keyboard Manager, but doesn't work correctlywith it enabled.

Actual behavior

Win + R is opening Game Recording instead of the Run dialog.

Issue 2:
📝 Provide a description of the new feature

there is no numpad on my laptop, so I think it would be helpful if I can set a shortcut to enable the keymap feature,

then I can remap jkluio789 to num 123456789

when I dont need numpad, I can press the shotcut to turn it off, but when I need it, I can easily press the shotcut to turn it on

further, if the keys can be grouped, and set shortcuts to enable/disable each group, it would be helpful to deal with different conditions

(sorry for my poor english)

If you'd like to see this feature implemented, add a 👍 reaction to this post.
Issue 1:
I'm in the process of getting MUI@next to work with next.js, and I just wanted to pass on my experience so far. Even though MUI uses JSS, it feels like MUI is getting involved in a lot of areas that are already handled by react-jss MUI now has its own SSR configuration, separate from jss and react-jss, and this, to me, feels like a possible mistake. Personally, I wish MUI just deferred SSR configuration to the JSS suite. If it did that, then a there would be a single point of SSR configuration for all use of JSS within an app, rather than separate MUI and non-MUI configurations. Also, react-jss now supports the the new JSS function properties, automatically generating both a static and dynamic stylesheet for the wrapped react component, and passing in the component props to all of the function properties. It seems like a decision was made to not use that library, and instead have a combination of jss-theme-reactor and MuiThemeProvider take over those responsibilities. This means that, as react-jss evolves, MUI will have to duplicate features to stay current.

I realize that a shortcoming with react-jss has been that it can only take in an object when injecting a stylesheet, when what we really want is for it to take in a function like theme => stylesheet, and this is probably the main perk of theme reactor. Now, with function properties, we can do:

injectSheet({
backgroundColor: ({theme}) => theme.backgroundColor,
color: ({theme}) => theme.color
})

It's more verbose, but it works. All that would be necessary would be to wrap the react-jss injectSheet HOC like:

export const withStyles = styleSheet => compose(
withTheme,
injectSheet(styleSheet)
)

If all of the MUI components then used that HOC, SSR config could be done just with the react-jss StyleRegistryProvider

If using function properties for theme-related styles is too much overhead, then perhaps the withStyles HOC could be reworked to both take in a theme => stylesheet function, populate an object with the theme data, and then delegate to injectSheet for the actual stylesheet injection. That way we would have the best of all worlds: a choice between static or dynamic theme-based styles.

Just my two cents. Thanks for the amazing library.

Jay

Issue 2:
Hello,

It's great to have code sandbox in docs. It'll save me a lot of time while prototyping.

I realized that, drawers(completely) and components with images are not working properly.

Avatar;

Drawer;

https://material-ui.com/demos/drawers/

https://codesandbox.io/s/zpro6nj7m

Gridlist;

https://codesandbox.io/s/r4mk39vkxq

etc.

Thank you for your effort.
Issue 1:
Bug report

with-apollo-and-redux-app Could not find "store"

Describe the bug

Run the example with-apollo-and-redux-app, when first get index page, data not loaded in ssr.

add console.log(error) in withApollo.js, here shows the error message:

Error { Invariant Violation: Could not find "store" in the context of "Connect(Index)". Either wrap the root component in a <Provider>, or pass a custom React context provider to <Provider> and the corresponding React context consumer to Connect(Index) in connect options.

To Reproduce

1, just follow the instructions in the repo

2, add console.log('Error', error) in withApollo.js,

3, yarn dev

4, go to localhost:3000 and you'll find error message as described above

Expected behavior

Data get loaded in ssr correctly

Screenshots

System information

OS: Ubuntu

Browser : Chrome latest

Version of Next.js: latest

Additional context

N/A

Issue 2:
Bug report

Missing prop on withData

Describe the bug

A clear and concise description of what the bug is.

To Reproduce

Steps to reproduce the behavior, please provide code snippets or a repository:

yarn dev

Watch the console

on canary branch

Expected behavior

To not have a warning.

A clear and concise description of what you expected to happen.

Screenshots

System information

Linux (Manjaro)

Chrome

Node [v11.15.0]

Additional context

Add any other context about the problem here.
Issue 1:
Q
A

Bug report?
yes

Feature request?
no

BC Break report?
no

RFC?
no

Symfony version
3.3

When we have:

embedded formB inside formA

validation on both forms defined in yamls

validation on formA -> formB include Valid constraint

eg. fieldA on formB has NotBlank with groups: "groupA" defined

we have hardcoded groups ['Default', 'groupA'] in validation_groups options on formB

And we want to validate formA (containing formB), the validation group is not working (NotBlank is not validated). What's odd, when we add same Valid constraint directly in constraints option on formA form builder - it's working as expected 😕

I'll provide a reproducible code in a while.

Issue 2:
Hi all,

I am not sure if this is a bug or if I am doing something wrong, but after 'composer update' I have sf 2.6. components and my cilex cli application is broken.

It seems like Symfony\Component\Console\Application is using deprecated helpers:

use Symfony\Component\Console\Helper\DialogHelper;
use Symfony\Component\Console\Helper\ProgressHelper;
use Symfony\Component\Console\Helper\TableHelper;

...

/**
* Gets the default helper set with the helpers that should always be available.
*
* @return HelperSet A HelperSet instance
*/
protected function getDefaultHelperSet()
{
return new HelperSet(array(
new FormatterHelper(),
new DialogHelper(),
new ProgressHelper(),
new TableHelper(),
new DebugFormatterHelper(),
new ProcessHelper(),
new QuestionHelper(),
));
}

Is it correct to use deprecated code in the component? If so....how to get rid of the errors?

TIA!
Issue 1:
Summary of the new feature/enhancement

Many window manager tools also include an option to split the taskbar, for example:

https://www.actualtools.com/multiplemonitors/features/multi_monitor_taskbar/

https://www.displayfusion.com/Features/Taskbar/

Proposed technical implementation details (optional)

A new section under FanzyZones configuration for "Taskbar" with checkboxes for:

Split taskbar between zones

Mirror taskbar buttons

Mirror Start button (if disabled only show start button in first zone)

Mirror Notification (if disabled only notification area in last zone)

Issue 2:
Summary of the new feature/enhancement

Similar to when using multiple monitors, where the icon of the app in the taskbar can be shown on the monitor which is displaying the app.

Proposed technical implementation details (optional)

Not sure what to put here.
Issue 1:
Hey there! First off: thanks a lot for putting this library online. The community will definitely benefit from having such an easy path to super-performant pages 👍

I was actually evaluating next.js for creating a docs for one of my github repos, but only later I realised that the next.js build process doesn't output any html pages which can be used for e.g. gh-pages. This would actually be a really nice option to have and I guess a common enough use case!

In case it's helpful, I'll link to this library which I successfully used to do something very similar.

Issue 2:
Looking at the overall project structure, this seems like a rather easy step on your roadmap. Maybe someone from the team can provide a good starting point for contributors?
Issue 1:
Not sure what the heck it's doing, but sometimes it ignores @" and sometimes it doesn't.

Attached is screen shot showing bogus highlighting and .cs source file used to create that screen shot (extension changed to .txt to enable upload).

Note that it highlights correct in both VS15 & VS13

VSCode Version: 1.1.0-insider

OS Version: Win7 & Win10

SharedConstants.cs.txt

Issue 2:
Not sure what the heck it's doing, but sometimes it ignores @" and sometimes it doesn't.

Attached is screen shot showing bogus highlighting and .cs source file used to create that screen shot (extension changed to .txt to enable upload).

Not sure if this belongs here or in VS Code. It's issue 6325 over there, duplicating it here by suggestion.

SharedConstants.cs.txt

Note that it highlights correct in both VS15 & VS13
Issue 1:
Upon opening VS Code with Wayland (tested with GNOME 3 and Sway),

~ > visual-studio-code
bash: cannot set terminal process group (-1): Inappropriate ioctl for device
bash: no job control in this shell
Xlib: extension "XFree86-VidModeExtension" missing on display ":0".

Issue 2:
Upon opening VS Code with Wayland (tested with GNOME 3 and Sway), I get a message saying Something has gone wrong..

~ > visual-studio-code
bash: cannot set terminal process group (-1): Inappropriate ioctl for device
bash: no job control in this shell
Xlib: extension "XFree86-VidModeExtension" missing on display ":0".

If I close the error window instead of logging out, VS Code starts up fine.
Issue 1:
When root logger hijacking is disabled, it feels rather inappropriate to change the log level of the root logger...

Issue 2:
Hi there,

I'm experiencing an issue similar to #2016, #1867, in that celery still hijacks the root logger despite setting CELERYD_HIJACK_ROOT_LOGGER to False (as read from http://celery.readthedocs.org/en/latest/configuration.html#celeryd-hijack-root-logger). I use celery + Django for several projects and except for this wart, it's fantastic. What I really, really, need is a way to configure celery so that it does not attempt to configure logging at all. It's quite frustrating to try and work out why our custom crafted logging configuration works as expected for our web servers, but is completely wiped out for our task code - only to realize that celery itself is blowing away our logging configuration.

In reading the previous issues related to this I seems that you are understandably reluctant to change the current behaviour of CELERYD_HIJACK_ROOT_LOGGER for compatibility reasons. Would you be open to a new setting? Something like CELERYD_CONFIGURE_LOGGING=False, that would cause celery to not attempt to configure logging at all. If you are open to it I'd be happy to contribute a PR with tests.

Thanks!
Issue 1:
I would appreciate if you make a comeback here.

Issue 2:
The npm page for @types/ssri, for example, incorrectly claims there is no README, even though when you install that package, there is a README. This is happening for all packages published by the Definitely Typed publisher to the @types/ scope since June 20.

Since Definitely Typed publishes a lot of packages, I was able to narrow the break down to the span between @types/react-vis@1.11.1:"2020-06-19T06:34:23.672Z" and @types/ssri@7.1.0: "2020-06-20T19:36:22.490Z". Somewhere between the 19th and 20th packages stopped showing their READMEs.

No code changes to our publisher happened in that time span, nor did a new version of the npm package get published.

And the couch packages are very similar:

http://registry.npmjs.org/@types/react-vis (good)

http://registry.npmjs.org/@types/ssri (bad)

Both jsons have a "readme" key with the readme contents, and a readmeFilename key that is just the empty string. When installed, both packages have near-identical READMEs. Even the order of the couch package keys is the same.

Steps to Reproduce

Recreate the package structure published by https://github.com/microsoft/DefinitelyTyped-tools: generate-packages.ts and package-publisher.ts are the interesting files. Let me know if this is needed and I can try to create a standalone repro. However, I'm mostly interested in having npm insiders

Look at the good-vs-bad couch packages for differences.

Check whether npm changed something related to README somewhere between June 19 and 20.

Expected Behavior

README should be visible on the npmjs.com page.

References

Could be a duplicate of #1221, except that it didn't start until much later, and we use the npm package to publish rather than the CLI.
Issue 1:
It would be nice to be able to drag a tab out of the window and have it open in a new window rather then having to open a new window then drag the tab into the new window.

I'm on Windows 8.1, not sure if this behavior is different on other OS.

Issue 2:
In many "tab" based products, you can grab a tab and "pop it out" to create a new window with that content. That would be awesome for atom when you want to windows side by side.
Issue 1:
Description

sklearn.utils.multiclass unique_labels() does not yet allow "multiclass-multioutput" input type for y .

See this comment in the code:

scikit-learn/sklearn/utils/multiclass.py

Line 53
in
3a48f0a

At the moment, we also don't allow "multiclass-multioutput" input type.

This type is used by multilabel classifiers such as sklearn.tree.DecisionTreeClassifier.

scikit-learn/sklearn/tree/tree.py

Line 763
in
4bba216

y : array-like, shape = [n_samples] or [n_samples, n_outputs]

So I propose to add support for this. At first glance, it doesn't appear to be difficult. Is there any reason not to to this?

Issue 2:
It looks to me like the decision trees use accuracy_score for their score but accuracy_score doesn't document that it's supporting multiclass-multioutput which the trees do.

I guess the y_true == y_pred works in this case, but it should be documented.

There's no list of scores supporting multiclass-multioutput in the docs, and that should be fixed, too.

Update:

Calling score in this case errors :-/