WhereIsAI/github-issue-similarity huggingface dataset:
Issue 1:
I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Generated sizes include the type of unit, so it's not up to the interpreter to add missing information.

padding: {
paddingTop: {theme.spacing.unit}px,
paddingBottom: {theme.spacing.unit}px,
},

Current Behavior

Currently, a few calculated sizes in the code base are missing the type of unit. For example in https://github.com/mui-org/material-ui/blob/v1-beta/src/List/List.js

padding: {
paddingTop: theme.spacing.unit,
paddingBottom: theme.spacing.unit,
},

It might be our setup, but at production Firefox and Chrome are complaining about the missing unit type. This is correct as the inspector shows:

.c56 {
padding-top: 8;
padding-bottom: 8;
}

Steps to Reproduce (for bugs)

The issue is hard to track down as I am not a developer; however it does not occur at our development environments. This leads me to believe it's going wrong during npm build somewhere in our webpack stack.

Adding the unit types would however bypass the issue all together imho.

Might be related:

styled-components/styled-components#825

cssinjs/istf-spec#3

Context

Currently we encounter the issue (and found the culprit code) in a List, but there are more

Multiple development environments

Multiple prod environments

Your Environment

Latest Chrome and Firefox, zillion npm packages

├─┬ material-ui@1.0.0-beta.24

├─┬ material-ui-icons@1.0.0-beta.17

│ ├─┬ jss@9.4.0

│ ├─┬ jss-preset-default@4.0.1

│ │ ├── jss-camel-case@6.0.0

│ │ ├─┬ jss-compose@5.0.0

│ │ ├── jss-default-unit@8.0.2

│ │ ├── jss-expand@5.1.0

│ │ ├─┬ jss-extend@6.1.0

│ │ ├── jss-global@3.0.0

│ │ ├─┬ jss-nested@6.0.1

│ │ ├── jss-props-sort@6.0.0

│ │ ├─┬ jss-template@1.0.0

│ │ └─┬ jss-vendor-prefixer@7.0.0

│ ├─┬ react-jss@8.2.0

│ │ ├── jss@9.4.0 deduped

│ │ ├── jss-preset-default@4.0.1 deduped

Issue 2:
I noticed some text fields and components implemented using 'FormControl' were no longer in horizontal alignment after a recent package update to beta33.

I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Some of these controls are aligned using and on other pages they are aligned with a vanilla flex box row. All have margin set to 'none' and were previously in perfect horizontal alignment.

Current Behavior

Now, text fields are slightly higher whose type was set to 'tel'. Removing the type="tel" property fixes the alignment issue. There seems to be a '1.1875em' modification to the height that is calculated when type="tel". This appears to be linked to a change made in input.js involving class '.inputType'.

It may be linked to this commit: b706b42#diff-deaeb5c7e253bfa4f497fa8ace0b3607

This occurred on Chrome 64.0.3282.140 and the latest version of Safari ios.
Issue 1:
hello

I often use my keyboard to access menus of the editor,

I installed atom yesterday and like it, thank you!

But I doscovered this morning that I can't open menus on Windows 8.1 using the shortcuts Alt+H and Alt +F.

stangely the other menus work fine (find, edit, packages and selection)

Issue 2:
By default, alt-f is bound to editor:move-to-end-of-word and alt-h to editor:delete-to-beginning-of-word. These conflict with activating File and Help menus in Windows.

Suggestions:

Reserve alt-f and alt-h for menu bar access and change editor shortcuts.

Focus and activate menu bar via a single alt press so user can navigate through the menus using arrow keys (customary behavior in Windows).
Issue 1:
It is a common practice in scientific plottings that a barplot (or violinplot/swarmplot) has statistical annotations such as p-values marked with horizontal lines connecting the bars. There is already an implementation of this as a separate project:

https://github.com/webermarcolivier/statannot

but having such a feature incorporated into seaborn would be very handy. Is there any interest to pursue this? -Thanks

Issue 2:
It'd be nice to be able to run some sort of statistic (e.g. Mann-Whithney) to determine group differences in bar plots when hue is provided. When significant (p<0.05) an asterisks can be added above those groups.
Issue 1:
Original ticket http://projects.scipy.org/scipy/ticket/985 on 2009-08-06 by trac user rubert, assigned to unknown.

I assume the 1-D hilbert transform function in the signal module was intended to work like its Matlab equivalent. That is, if I hilbert transform an N-D array it will only do it along the zeroth dimension.

The hilbert transform was returning zeros for me when I attempted to use it on two-D arrays. This patch fixes that. The problem was with the arguments passed to ifft.

Issue 2:
Original ticket http://projects.scipy.org/scipy/ticket/1642 on 2012-04-14 by @rgommers, assigned to unknown.

This warning was showing up by default for a long time. The cause seems to be that the optional nxest/nyest parameters to surfit, which are determined in fitpack.pyf, are too small if kx=ky=1.

>>> x = [1,1,1,2,2,2,4,4,4]
>>> y = [1,2,3,1,2,3,1,2,3]
>>> z = array([0,7,8,3,4,7,1,3,4])
>>> lut = SmoothBivariateSpline(x, y, z, kx=1, ky=1, s=0)
/Users/rgommers/Code/scipy/scipy/interpolate/fitpack2.py:613: UserWarning:
The required storage space exceeds the available storage space: nxest
or nyest too small, or s too small.
The weighted least-squares spline corresponds to the current set of
knots.
warnings.warn(message)

This needs some investigation. I'll filter out the warning for now.

A likely related problem:

In [3]: %paste
x = [1,1,1,2,2,2,3,3,3]
y = [1,2,3,1,2,3,1,2,3]
z = [0,7,8,3,4,7,1,3,4]
s = 0.1
tx = [1+s,3-s]
ty = [1+s,3-s]
lut = LSQBivariateSpline(x,y,z,tx,ty,kx=1,ky=1)
## -- End pasted text --
/Users/rgommers/Code/scipy/scipy/interpolate/fitpack2.py:684: UserWarning:
The coefficients of the spline returned have been computed as the
minimal norm least-squares solution of a (numerically) rank deficient
system (deficiency=7). If deficiency is large, the results may be
inaccurate. Deficiency may strongly depend on the value of eps.
warnings.warn(message)
Issue 1:
removedFromOurArray = ourArray.shift(); // removedFromOurArray now equals "Stimpson" and ourArray now

Is missing the var keyword.

Issue 2:
Challenge Waypoint: Manipulate Arrays With shift has an issue.

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

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

My code:

var ourArray = ["Stimpson", "J", ["cat"]];

removedFromOurArray = ourArray.shift(); // removedFromOurArray now equals "Stimpson" and ourArray now equals ["J", ["cat"]].

var myArray = ["John", 23, ["dog", 3]];

// Only change code below this line.

var removedFromMyArray = myArray.shift();

// Only change code above this line.

(function(y, z){return 'myArray = ' + JSON.stringify(y) + ' & removedFromMyArray = ' + JSON.stringify(z);})(myArray, removedFromMyArray);
Issue 1:
The current implementation of GLTFExporter is caching textures as it processes the scene but other parts of three js (e.g. TextureLoader and ImageLoader) cache the image.

This creates problems when, for instance, you load various model files (via FBXLoader etc) that have references to the same image files and you try to export the resulting scene via GLTFExporter with embedImages = true and binary = true. Essentially, it packs duplicate image data into the glb, which bloats it considerably.

The problem seems to centre around the fact that TextureLoader creates a new texture (with a potentially cached Image) every time you call load() but GLTFExporter is basing it's image caching on the texture when you try to process the scene. You end up with a list of textures all referring to the same image but GLTFExporter will go through each texture and write out the corresponding cached image.

NOTE: I could solve this by hacking in something to patch/reformat all the textures in my scene to work with GLTFExporter but I think handling this in the exporter would be better and more user-friendly.

Three.js version

Dev

[x ] r92

...

Browser

All of them

Chrome

Firefox

Internet Explorer

OS

All of them

Windows

macOS

Linux

Android

iOS

Hardware Requirements (graphics card, VR Device, ...)

Issue 2:
I'm attempting to debug a shading issue that occurs on my meshes when animated via skeletal animation.

As far as I can tell, the mesh is being applied correctly, as are the skins. The only thing left that I could think of that would be influencing it is the normals. The mesh appears to be shading correctly in the bind pose, though it's difficult to tell for sure. Upon applying and updating the vertex normals helper, this was the result:

Is this the intended behavior? Should they be moving with the mesh during skeletal animations? If so, what could be causing them to remain static?

Thanks for your time.
Issue 1:
I have a project at the moment that uses nextjs embedded in an express server. I have a page called event.js and I route /event/:eventId to /event?eventId={id}.

My issue now is that if the user goes to /event without specifying an ID my page component is rendered anyway.

My suggestion is: Allow getInitialProps function to return null (or false or 404 or new Error("")), which will result in next intercepting this and rendering the 404 page.

I have searched the issues of this repository and believe that this is not a duplicate.

Current Setup

expressApp.get('/event/:eventId', (req, res) => {
const mapToUrl = '/event'
const query = {
...req.query,
eventId: req.params.eventId
}
nextApp.render(req, res, mapToUrl, query)
})

expressApp.get('*', nextHandler)

Issue 2:
I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Need to be able to update url without any navigation ie a history.replaceState

The docs say in https://github.com/zeit/next.js#imperatively:

Note: in order to programmatically change the route without triggering navigation and component-fetching, use props.url.push and props.url.replace within a component

Current Behavior

The call gives a deprecation warning and still triggers navigation

Steps to Reproduce (for bugs)

use props.url.replace within a component

notice that a navigation is triggered.

Context

I am trying to update the url without anything else changing. A way to do a history.replaceState in the next.js world. Calling history.replaceState steps outside the next.js router and then messes up

back navigation. Using {shallow:true} with Router.replace still calls componentWillReceiveProps.
Issue 1:
fit_generator() seems to keep consuming memory throughout an epoch if objects defined inside the generator class are not converted into a numpy array. RAM keeps on getting filled until an epoch ends. For large enough datasets, this leads to RAM completely filling up before a single epoch is completed.

Is it possible that the objects keep getting duplicated?

Python = 3.6.6

Keras = 2.2.4

Tensorflow = 1.12.0

Here is a sample script to reproduce the issue.

Issue 2:
A major use-case for the scikit-learn API is automating hyperparameter opt. With the current API we need to either:

Search through nb_epochs, duplicating most of the computation (essentially every nb_epoch except the largest is redundant).

Implement getting the best epoch ourselves, possibly using EarlyStopping or the like.

Would it be possible to just have KerasClassifier.fit() return the best epoch?
Issue 1:
julia> begin
module A
1
end
end
ERROR: syntax: module expression not at top level

Theoretically this should work, because module A is indeed in global scope? Would be helpful to get working for macros which operate on large blocks of global code

Issue 2:
Looks like a bug in eval:

_ _ _(_)_ | A fresh approach to technical computing
(_) | (_) (_) | Documentation: http://docs.julialang.org
_ _ _| |_ __ _ | Type "help()" to list help topics
| | | | | | |/ _` | |
| | |_| | | | (_| | | Version 0.2.0-514.r3d43
_/ |\__'_|_|_|\__'_| | Commit 3d43df9032 2013-03-16 06:59:07
|__/ |

julia> @eval begin
type T
x
end
T(sum([1,2]))
end
T(3)

julia> @eval begin
type T2
x
end
s = 0
for i in [1,2]
s += i
end
T2(s)
end
ERROR: error compiling anonymous: unsupported or misplaced expression composite_type in function anonymous
Issue 1:
Q
A

Bug report?
yes

Feature request?
no

BC Break report?
no

RFC?
no

Symfony version
4-beta2

When using .env in config, the casting is not work.

Invalid type for path "easy_admin.list.max_results". Expected int, but got string.

Issue 2:
Q
A

Bug report?
no

Feature request?
yes

BC Break report?
no

RFC?
no

Symfony version
3.2

Related to #20434, if you try to pull data such as CACHE from an environment variable and pass it to Twig, Twig's Bundle reads the type literally, as "false" or "true" - as it is passed from getenv.

How to Replicate

Add this environment variable:

CACHE false

In config.yml:

twig:
cache: %env(CACHE)%

This results in a cache being created under web/false/, which is certainly not the intended result.

Note that this has other unintended consequences throughout Symfony, where a boolean is expected but not validated.

Solutions?

A simple solution could be to use the env() library created by Laravel as it provides for type coercion out of the box.
Issue 1:
System information

OS Platform and Distribution: Linux 18.04

TensorFlow version 2.0-alpha

Python 3.6.8

Describe the current behavior

Trying to write any tensor as histogram summary:

/tmp/cpu-env/lib/python3.6/site-packages/tensorboard/plugins/histogram/summary_v2.py in _buckets(data, bucket_count)

89 if bucket_count is None:

90 bucket_count = DEFAULT_BUCKET_COUNT

---> 91 with tf.name_scope('buckets', values=[data, bucket_count]):

92 tf.debugging.assert_scalar(bucket_count)

93 tf.debugging.assert_type(bucket_count, tf.int32)

TypeError: init() got an unexpected keyword argument 'values'

Code to reproduce the issue

tf.summary.histogram('SampleKernel', tf.random.normal(shape=[3,3,3,32], dtype=tf.float32))

Issue 2:
TensorBoard version (from pip package, also printed out when running tensorboard)

-- TensorBoard 1.14.0a20190301

TensorFlow version if different from TensorBoard

-- TensorFlow-gpu 2.0-alpha

OS Platform and version (e.g., Linux Ubuntu 16.04)

-- Linux, Arch, Kernel 4.19

Python version (e.g. 2.7, 3.5)

-- Python 3.6.7

I have built a GAN with TF 2.0-alpha using tf.keras. I am using TensorBoard via train_writer = tf.summary.create_file_writer("log-train"). Adding scalars with tf.summary.scalar("loss g", loss_g, step=batch_id) and storing predicted images with tf.summary.image("g pred", prediction[0:3], step=batch_id) works without any problems.

However, when I try to add the generator's weights to TensorBoard with tf.summary.histogram it fails.

for layer in G_model.layers:
for weight in layer.weights:
tf.summary.histogram(weight.name, weight, step=batch_id)

G_model is a tf.keras model created with the functional API.

The exception seems to be TensorBoard-specific:

Traceback (most recent call last):
File "train.py", line 84, in <module>
tf.summary.histogram(weight.name, weight, step=batch_id)
File "(...)/python3.6/site-packages/tensorboard/plugins/histogram/summary_v2.py", line 73, in histogram
tensor = _buckets(data, bucket_count=buckets)
File "(...)/python3.6/site-packages/tensorboard/plugins/histogram/summary_v2.py", line 91, in _buckets
with tf.name_scope('buckets', values=[data, bucket_count]):
TypeError: __init__() got an unexpected keyword argument 'values'

If I simply delete , values=[data, bucket_count] in summary_v2.py (line 91) it works, weights are showing up correctly in TensorBoard then.

Thus, original code of summary_v2.py in line 91:

with tf.name_scope('buckets', values=[data, bucket_count]):

throws the exception posted above, while this code:

with tf.name_scope('buckets'):

is working properly.
Issue 1:
With r111, WebGlRenderer begins checking for a position buffer attribute and uses its count in deciding whether to invoke the program. This introduced an arbitrary new requirement that BufferGeometry have a 'position' attribute. I suggest that the old behavior be followed, in which BufferGeometry.setDrawRange() is respected.

WebGLRenderer.js:

var position = geometry.attributes.position;
if ( index !== null && index.count === 0 ) return;
if ( position === undefined || position.count === 0 ) return;

Issue 2:
Description of the problem

Rendering an InstancedBufferGeometry with no (used) attributes does not render.

This happens when a shader uses only (implicit) gl_InstanceID and/or gl_VertexID,

and user program establishes count with 'geom.maxInstancedCount = ...'

(It only applies to WebGL2 as earlier shaders do not support this.)

The internal value geometry._maxInstanceCount is never established, so

var instanceCount = Math.min( geometry.instanceCount, geometry._maxInstanceCount );

does not set a sensible value.

A workaround is to set geometry._maxInstanceCount in the user's program (eg to Infinity),

but this is obviously not the correct solution.

With earlier versions of three.js (eg 109) a program could provide a 'dummy' attribute not actually used in the shader to establish count; but 117 is clever enough to realize this dummy is not used, and therefore does not take account of it in the count calculation.

example to follow

Three.js version

Rel

r117

...

Browser

All of them

Chrome

Firefox

Internet Explorer

OS

All of them

Windows

macOS

Linux

Android

iOS

Hardware Requirements (graphics card, VR Device, ...)
Issue 1:
Shardingsphere 4.1.1 has the problem of duplicate and missing data in paging query？

Issue 2:
I'm using sharding-jdbc 3.0.0.

There's a complicated sql statements, and some big tables have several indexes, mysql will chose a better index according to its evalutation. But it'll not a best index, so we add use index statement to force mysql to use the index.

[bug details]

exception will be thrown if sql contains use index(index_name) in mysql.

2019-01-18 19:06:45 ERROR errorLogger:40 - Cannot find logic table name with logic index name: 'uk_e9fmb4uxwkpopbbfdbdleuj82'
Issue 1:
Glide Version: 4.3.1

Integration libraries: None

Device/Android Version: LG G4, 6.1

Issue details / Repro steps / Use case background:

Previously in Glide v3.7.0 the signature of the encode method in Encoder interface was:

boolean encode(T data, OutputStream os)

And in Glide v4.X it became:

public boolean encode(InputStream data, File file, Options options)

I understand that mostly, encoders will use File as a parameter that will be used for the encoding, but there's an exception when it comes to encryption encoding:

For instance, Conceal library uses cipher OutputStreams to write encrypted data.

In Glide v3.X My custom Encoder has created a ciphered OutputStream and then use the built-in BitmapEncoder to do the encoding work with the ciphered OutputStream it generated,

In Glide v4.X, BitmapEncoder accepts a File and not OutputStream as before, so my encoder cannot make use of the BitmapEncoder class to do the encoding work unfortunately, which means that my custom Encoder has to be responsible for the encryption + encoding process by itself, instead of just the encryption like in Glide v3.X.

I think that Encoder interface should accept also OutputStream and not just File objects to handle these kind of situations.

Here's an example usage in Glide 3.X:

OutputStream outputStream = crypto.getCipherOutputStream(new BufferedOutputStream(os), Entity.create("cache_entity"));
encoded = new BitmapEncoder().encode(data, outputStream);

Glide load line / GlideModule (if any) / list Adapter code (if any):

Glide.with...

Layout XML:

<FrameLayout xmlns:android="...

Stack trace / LogCat:

paste stack trace and/or log here

Issue 2:
My Project has encrypted files to get from diskcache, so use "DiskCache.get(Key key)", I can't read data from file.And I suggest we can change the return "File" with "InputStream".
Issue 1:
Dave Syer opened SPR-4855 and commented

Migrate Repeat and Retry from Spring Batch. They are almost completely self-contained now (on trunk), so it shouldn't be a tough job - just need to decide where to put them. Maybe the tests might have some batch dependencies, but they would be easily factored out. There is also a dependency on some utility stuff in org.springframework.batch.support which would need to be moved over or equivalents provided in core.

The only real work will be migrating the StatefulRetryOperationsInterceptor (very useful). It currently depends on ItemKeyGenerator and NewItemIdentifier, for good reasons, so equivalents will have to be provided. When this is refactored it might also be a good idea to revisit the whole stateful retry concept - it could be implemented as an extension of the RetryOperations interface instead of through a callback / policy pairing as it is now.

Affects: 2.5.4

Issue Links:

#13808 Consider moving Spring Batch Retry package into Spring Core ("is duplicated by")

1 votes, 3 watchers

Issue 2:
Sami Dalouche opened SPR-2578 and commented

According to the documentation (http://static.springframework.org/spring/docs/2.0.x/reference/transaction.html#transaction-declarative-annotations), Annotating interface "IFoo" using @Transactional is supposed to affect the implementations of implementations of "IFoo".

This is the case when using Interface-based proxying, but does not work when using CGLIB proxies.

This issue is easy to reproduce.. Just take any IFoo, set it @Transactional, create a class that implements IFoo, and use the following in the spring appContext.xml :

Affects: 2.0 RC3

Issue Links:

#7211 Semantics affected - tx:annotation-driven, proxy-target-class and @Transactional on interface ("duplicates")
Issue 1:
According to the documentation here: http://celery.readthedocs.org/en/latest/userguide/workers.html#process-signals

I should be able to send a SIGTERM to my running worker process and have it finish the task it is currently working on before it shuts down. However when I call sigterm the process exits immediately with the following traceback.

Traceback (most recent call last):
File "/edx/app/edxapp/venvs/edxapp/local/lib/python2.7/site-packages/billiard/pool.py", line 1171, in mark_as_worker_lost
human_status(exitcode)),
WorkerLostError: Worker exited prematurely: signal 15 (SIGTERM).

I'm using Celery 3.1.18 with django 1.4, is there something special that has to be done to integrate the signal handlers with django?

Also posted here on the mailing list: https://groups.google.com/forum/#!topic/celery-users/t8g5KvIvQZ8

Issue 2:
Checklist

I have checked the issues list

for similar or identical enhancement to an existing feature.

I have checked the pull requests list

for existing proposed enhancements.

I have checked the commit log

to find out if the if the same enhancement was already implemented in the

master branch.

I have included all related issues and possible duplicate issues in this issue

(If there are none, check this box anyway).

Related Issues and Possible Duplicates

Related Issues

None

Possible Duplicates

None

Brief Summary

The docs for celery.schedules.solar state that long and lat are ints. This triggers a warning for some linters and also the example on the page uses the function with floats:

lat (int): The latitude of the observer.
lon (int): The longitude of the observer.

from celery.schedules import solar

app.conf.beat_schedule = {
# Executes at sunset in Melbourne
'add-at-melbourne-sunset': {
'task': 'tasks.add',
'schedule': solar('sunset', -37.81753, 144.96715),
'args': (16, 16),
},
}

Design

Architectural Considerations

None

Proposed Behavior

Change the docstring to float

Proposed UI/UX

Diagrams

N/A

Alternatives

None