Unique Error with Image Retraining Tutorial

I'm using the image retraining tutorial (here) and keep getting a very odd error. Previously, retraining, then using the graph in a slightly modified classify.py script was possible, but as of two days ago, I get the following error when the graph is loaded by the expression _ = tf.import_graph_def(graph_def, name=''):
InvalidArgumentError: NodeDef mentions attr 'data_format' not in Op<name=Conv2D; signature=input:T, filter:T -> output:T; attr=T:type,allowed=[DT_FLOAT, DT_DOUBLE]; attr=strides:list(int); attr=use_cudnn_on_gpu:bool,default=true; attr=padding:string,allowed=["SAME", "VALID"]>; NodeDef: conv/Conv2D = Conv2D[T=DT_FLOAT, data_format="NHWC", padding="VALID", strides=[1, 2, 2, 1], use_cudnn_on_gpu=true, _device="/job:localhost/replica:0/task:0/cpu:0"](Mul, conv/conv2d_params) 	 [[Node: conv/Conv2D = Conv2D[T=DT_FLOAT, data_format="NHWC", padding="VALID", strides=[1, 2, 2, 1], use_cudnn_on_gpu=true, _device="/job:localhost/replica:0/task:0/cpu:0"](Mul, conv/conv2d_params)]] Caused by op u'conv/Conv2D', defined at:
I looked into the API and saw that tf.nn.conv2d has an attribute, data_format, that always defaults to None. Is there some update to Tensorflow that has caused this to be changed? Has anyone encountered an error like this, or could provide some clue as to where to look to solve it?

This sounds like a versioning issue. Does upgrading to a nightly version of TensorFlow solve it? Assigning to @shlens in case he knows of a change to the model that could have caused this.

I figured it was a versioning issue, but I was under the impression that my versions were matched. I'll try updating to a nightly. Can this be done on OSX thru pip, or do I have to install via source in Docker?

Found what seems to be the offending change.
https://github.com/tensorflow/tensorflow/commit/ee5326c99ad6e01b3b62dedb954923e11b513a83
in conv2d, there is no longer a data format operation.
@shlens , do you think this may be the cause of my error?

I am not aware of any changes in this code base. @petewarden but might be able to comment as well.

I think this change might be more likely to have caused the error you're seeing: https://github.com/tensorflow/tensorflow/commit/01a6f5e504d9299395888a786e52c589c16af529
If the graph for the model that you're retraining was regenerated after that change, and you're using TensorFlow version 0.7.1, then I'd expect to see that message. I'm not sure of the provenance of the trained model graphs though. @oweingrod: Can you confirm how you obtained the graph (i.e. which download link)? @shlens or @petewarden: Would you know if the graph had changed since the 0.7.1 release?
Here are links to the Mac OS X nightly PIP packages: Python 2 / Python 3

@mrry thanks for the prompt response! I'm using the bazel built inception graph retraining as described in the tutorial I linked to in my OP (the one recently written, I believe, by @petewarden). I'm assuming that it is the case that there is a discrepancy between the version of tensorflow used by bazel to build the inception graph for retraining, and my (likely slightly more recent) build that I am using to classify?

@shlens: did you regenerate the graph at the same time that you regenerated the checkpoint?  If so, you'll probably need to replace the graph with the old version.

I'm assuming that it is the case that there is a discrepancy between the version of tensorflow used by bazel to build the inception graph for retraining, and my (likely slightly more recent) build that I am using to classify?

I believe from @mrry and @shlens comments that you should only see the message if the version used for retraining was more recent than the version that you're calling in classify_image.py. Looking at the code, the data_format attribute was added to Conv2d back on February 19th:
https://github.com/tensorflow/tensorflow/commit/898d26a1cef46c0848188e22ba4b435846649b6d
I believe the other commit mentioned is a red herring, because it only affects pooling ops. So you'd have to be running code from before February 19th in classify_image.py to run into this error I believe. Can you try printing tf.version when you run your modified classify_image.py script?

@petewarden I can't seem to find tf.version in the API, and when I attempt to use it in python, as print tf.version() or print tf.version, I get an error. Am I going about this in the wrong way?
Again, thank you so much for your prompt responses!

@martinwicke can you help on the version printing?

@petewarden I discovered how to print. It's tf.version, not tf.version.
Anywho, I'm on 0.7.1 in classify.py. Is the example running on a different version?

I have a feeling the example was regenerated by accident using a newer version of TF that introduced new attributes.  If we used the old graph but a new checkpoint, I think things would work.

@vrv I think you're right. How can I go about making sure that my tensorflow versions are correctly matched? They seem to both be on 0.7.1, but maybe the graph (before retraining the final layer) is based on 0.7.0?

Also, having a newer issue recently where I cannot access the "final_result:0" tensor after the graph has been retrained. TF tells me that it is not available or does not exist, but it is specified as "final_result" in the retrain.py file. Any ideas?

I bet the new graph was built on a version newer than 0.7.1.  My guess if you installed one of the pip nightlies off the main github page, you might have better luck, but we should really fix the graph.

@vrv got it. So i'll update tensorflow to the absolute newest nightly, then retrain, then evaluate again on the absolute newest nightly. Fingers crossed!

@vrv Looks like updating to the newest version has solved this issue—it no longer crashes on Graph creation. However, it looks like something is up with the graph creation, because I'm still unable to access "final_result:0", which is supposed to be the final inference layer. Any ideas?

@shlens/@petewarden: deferring to you at this point, I've exhausted my knowledge of retrain.py :)

@oweingrod What's the exact error that you see when you run classify_image.py?

@petewarden, problem has been solved. I accidentally deleted the _ = tf.import_graph(graph, name='') instead of commenting it. This resulted in me not being able to access the final result tensor. Thank you all so much for your help!

Great to hear, let us know what you end up building!

@petewarden or @shlens Is there some way one of you could generate a graph that's compatible with the current release and upload it? I predict other people will run into the same issue, and telling people that they have to use the nightly isn't great....

The problem was that the image_retraining example (which loads in an old graph and writes out a new one) was run with a nightly build, and then later classify_image.py was run on that new graph with an older build of TensorFlow. So I don't believe this requires any changes to the graph. I'll stop by your desk to make sure I'm making sense!

It does make sense. Thanks Pete!

@petewarden
I run into the same problem on my computer. Do you mind to give any suggestion? I checked my tensorflow by pip show tensorflow:
Metadata-Version: 2.0
Name: tensorflow
Version: 0.8.0
Summary: TensorFlow helps the tensors flow
Home-page: http://tensorflow.org/
Author: Google Inc.
Author-email<EMAIL_ADDRESS>Installer: pip
License: Apache 2.0
Location: /home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages
Requires: six, protobuf, wheel, numpy
Classifiers:
Development Status :: 4 - Beta
Intended Audience :: Developers
Intended Audience :: Education
Intended Audience :: Science/Research
License :: OSI Approved :: Apache Software License
Programming Language :: Python :: 2.7
Topic :: Scientific/Engineering :: Mathematics
Topic :: Software Development :: Libraries :: Python Modules
Topic :: Software Development :: Libraries
Entry-points:
[console_scripts]
tensorboard = tensorflow.tensorboard.tensorboard:main
I added print tf.version in my retrain_test.py code. It print out:
0.8.0
Below is the copy of my terminal:
(tensorflow)xu@xu-ThinkCentre-M72e:~/tensorflow/tensorflow/models/image/imagenet(master) $ python retrain_test.py /home/xu/Robotimages/Robot1/w/1.jpg
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcublas.so locally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcudnn.so locally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcufft.so locally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcuda.so.1 locally
I tensorflow/stream_executor/dso_loader.cc:105] successfully opened CUDA library libcurand.so locally
0.8.0
I tensorflow/stream_executor/cuda/cuda_gpu_executor.cc:900] successful NUMA node read from SysFS had negative value (-1), but there must be at least one NUMA node, so returning NUMA node zero
I tensorflow/core/common_runtime/gpu/gpu_init.cc:102] Found device 0 with properties:
name: GeForce GTX 750 Ti
major: 5 minor: 0 memoryClockRate (GHz) 1.2545
pciBusID 0000:01:00.0
Total memory: 2.00GiB
Free memory: 1.77GiB
I tensorflow/core/common_runtime/gpu/gpu_init.cc:126] DMA: 0
I tensorflow/core/common_runtime/gpu/gpu_init.cc:136] 0:   Y
I tensorflow/core/common_runtime/gpu/gpu_device.cc:755] Creating TensorFlow device (/gpu:0) -> (device: 0, name: GeForce GTX 750 Ti, pci bus id: 0000:01:00.0)
W tensorflow/core/kernels/batch_norm_op.cc:36] Op is deprecated. It will cease to work in GraphDef version 9. Use tf.nn.batch_normalization().
W tensorflow/core/kernels/batch_norm_op.cc:36] Op is deprecated. It will cease to work in GraphDef version 9. Use tf.nn.batch_normalization().
W tensorflow/core/kernels/batch_norm_op.cc:36] Op is deprecated. It will cease to work in GraphDef version 9. Use tf.nn.batch_normalization().
E tensorflow/core/common_runtime/executor.cc:332] Executor failed to create kernel. Invalid argument: NodeDef mentions attr 'T' not in Op<name=MaxPool; signature=input:float -> output:float; attr=ksize:list(int),min=4; attr=strides:list(int),min=4; attr=padding:string,allowed=["SAME", "VALID"]; attr=data_format:string,default="NHWC",allowed=["NHWC", "NCHW"]>; NodeDef: pool = MaxPoolT=DT_FLOAT, data_format="NHWC", ksize=[1, 3, 3, 1], padding="VALID", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/gpu:0"
[[Node: pool = MaxPoolT=DT_FLOAT, data_format="NHWC", ksize=[1, 3, 3, 1], padding="VALID", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/gpu:0"]]
Traceback (most recent call last):
File "retrain_test.py", line 25, in
{'DecodeJpeg/contents:0': image_data})
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 340, in run
run_metadata_ptr)
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 564, in _run
feed_dict_string, options, run_metadata)
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 637, in _do_run
target_list, options, run_metadata)
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 659, in _do_call
e.code)
tensorflow.python.framework.errors.InvalidArgumentError: NodeDef mentions attr 'T' not in Op<name=MaxPool; signature=input:float -> output:float; attr=ksize:list(int),min=4; attr=strides:list(int),min=4; attr=padding:string,allowed=["SAME", "VALID"]; attr=data_format:string,default="NHWC",allowed=["NHWC", "NCHW"]>; NodeDef: pool = MaxPoolT=DT_FLOAT, data_format="NHWC", ksize=[1, 3, 3, 1], padding="VALID", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/gpu:0"
[[Node: pool = MaxPoolT=DT_FLOAT, data_format="NHWC", ksize=[1, 3, 3, 1], padding="VALID", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/gpu:0"]]
Caused by op u'pool', defined at:
File "retrain_test.py", line 18, in
_ = tf.import_graph_def(graph_def, name='')
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/framework/importer.py", line 240, in import_graph_def
op_def=op_def)
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2154, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1154, in init
self._traceback = _extract_stack()

I am having the same problem. I see that the proposed solution was to use nightly build.
I created a model using v0.9.0 (which is built by pulling master branch today) and I am trying to use it in  a script which uses v0.9.0 (installed by pip).
The python source is here https://github.com/eldor4do/Tensorflow-Examples/blob/master/retraining-example.py
and full stack trace here :

W tensorflow/core/framework/op_def_util.cc:332] Op BatchNormWithGlobalNormalization is deprecated. It will cease to work in GraphDef version 9. Use tf.nn.batch_normalization().
E tensorflow/core/common_runtime/executor.cc:334] Executor failed to create kernel. Invalid argument: NodeDef mentions attr 'T' not in Op output:float; attr=ksize:list(int),min=4; attr=strides:list(int),min=4; attr=padding:string,allowed=["SAME", "VALID"]; attr=data_format:string,default="NHWC",allowed=["NHWC", "NCHW"]>; NodeDef: pool = MaxPool[T=DT_FLOAT, data_format="NHWC", ksize=[1, 3, 3, 1], padding="VALID", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/cpu:0"](pool/control_dependency)
	 [[Node: pool = MaxPool[T=DT_FLOAT, data_format="NHWC", ksize=[1, 3, 3, 1], padding="VALID", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/cpu:0"](pool/control_dependency)]]
Traceback (most recent call last):
  File "classify.py", line 51, in
    run_inference_on_image()
  File "classify.py", line 34, in run_inference_on_image
    {'DecodeJpeg/contents:0': image_data})
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 372, in run
    run_metadata_ptr)
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 636, in _run
    feed_dict_string, options, run_metadata)
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 708, in _do_run
    target_list, options, run_metadata)
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 728, in _do_call
    raise type(e)(node_def, op, message)
tensorflow.python.framework.errors.InvalidArgumentError: NodeDef mentions attr 'T' not in Op output:float; attr=ksize:list(int),min=4; attr=strides:list(int),min=4; attr=padding:string,allowed=["SAME", "VALID"]; attr=data_format:string,default="NHWC",allowed=["NHWC", "NCHW"]>; NodeDef: pool = MaxPool[T=DT_FLOAT, data_format="NHWC", ksize=[1, 3, 3, 1], padding="VALID", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/cpu:0"](pool/control_dependency)
	 [[Node: pool = MaxPool[T=DT_FLOAT, data_format="NHWC", ksize=[1, 3, 3, 1], padding="VALID", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/cpu:0"](pool/control_dependency)]]
Caused by op u'pool', defined at:
  File "classify.py", line 51, in
    run_inference_on_image()
  File "classify.py", line 28, in run_inference_on_image
    create_graph()
  File "classify.py", line 15, in create_graph
    _ = tf.import_graph_def(graph_def, name='')
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/framework/importer.py", line 274, in import_graph_def
    op_def=op_def)
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2260, in create_op
    original_op=self._default_original_op, op_def=op_def)
  File "/usr/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1230, in __init__
    self._traceback = _extract_stack()

CC @petewarden @mrry Please help!

Hi @thammegowda , Can I ask, how would I install it on ubuntu? thanks alot for your help! :)

@Waffleboy Try using the same version for creating the model as well as for using the model (i.e. retrain.py and classify.py).
(Here are the scripts am I using to retrain & classify - https://github.com/thammegowda/notes/tree/master/deeplearning/imagerec, these are working fine with v0.9.0rc0 - installed using pip on OS X )

@thammegowda I tried to use your classify.py as below: it gives me the same error:
python classify.py --in=/home/xu/Desktop/Robot/Data/a/0.jpg --model=/home/xu/Desktop/Robot/retrained_graph.pb --labels=/home/xu/Desktop/Robot/retrained_labels.txt
W tensorflow/core/framework/op_def_util.cc:332] Op BatchNormWithGlobalNormalization is deprecated. It will cease to work in GraphDef version 9. Use tf.nn.batch_normalization().
E tensorflow/core/common_runtime/executor.cc:334] Executor failed to create kernel. Invalid argument: NodeDef mentions attr 'T' not in Op<name=MaxPool; signature=input:float -> output:float; attr=ksize:list(int),min=4; attr=strides:list(int),min=4; attr=padding:string,allowed=["SAME", "VALID"]; attr=data_format:string,default="NHWC",allowed=["NHWC", "NCHW"]>; NodeDef: pool = MaxPoolT=DT_FLOAT, data_format="NHWC", ksize=[1, 3, 3, 1], padding="VALID", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/cpu:0"
[[Node: pool = MaxPoolT=DT_FLOAT, data_format="NHWC", ksize=[1, 3, 3, 1], padding="VALID", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/cpu:0"]]
Traceback (most recent call last):
File "classify.py", line 81, in
run_inference(images=images, out_file=args['out'], labels=labels, model_file=args['model'])
File "classify.py", line 35, in run_inference
{'DecodeJpeg/contents:0': image_data})
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 372, in run
run_metadata_ptr)
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 636, in _run
feed_dict_string, options, run_metadata)
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 708, in _do_run
target_list, options, run_metadata)
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 728, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors.InvalidArgumentError: NodeDef mentions attr 'T' not in Op<name=MaxPool; signature=input:float -> output:float; attr=ksize:list(int),min=4; attr=strides:list(int),min=4; attr=padding:string,allowed=["SAME", "VALID"]; attr=data_format:string,default="NHWC",allowed=["NHWC", "NCHW"]>; NodeDef: pool = MaxPoolT=DT_FLOAT, data_format="NHWC", ksize=[1, 3, 3, 1], padding="VALID", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/cpu:0"
[[Node: pool = MaxPoolT=DT_FLOAT, data_format="NHWC", ksize=[1, 3, 3, 1], padding="VALID", strides=[1, 2, 2, 1], _device="/job:localhost/replica:0/task:0/cpu:0"]]
Caused by op u'pool', defined at:
File "classify.py", line 81, in
run_inference(images=images, out_file=args['out'], labels=labels, model_file=args['model'])
File "classify.py", line 20, in run_inference
create_graph(model_file)
File "classify.py", line 13, in create_graph
_ = tf.import_graph_def(graph_def, name='')
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/framework/importer.py", line 274, in import_graph_def
op_def=op_def)
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2260, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1230, in init
self._traceback = _extract_stack()

@thammegowda  I trained and tested them on my linux with anaconda installed tensorflow 0.9.0.

@thammegowda  Ok, I reinstalled tensorflow 0.9.0rc0. It still shows the same error.
(tensorflow)xu@xu-Lenovo-Yoga-2-Pro:~$ pip show tensorflow
Metadata-Version: 2.0
Name: tensorflow
Version: 0.9.0rc0
Summary: TensorFlow helps the tensors flow
Home-page: http://tensorflow.org/
Author: Google Inc.
Author-email<EMAIL_ADDRESS>Installer: pip
License: Apache 2.0
Location: /home/xu/anaconda2/envs/tensorflow/lib/python2.7/site-packages
Requires: numpy, six, protobuf, wheel
Classifiers:
Development Status :: 4 - Beta
Intended Audience :: Developers
Intended Audience :: Education
Intended Audience :: Science/Research
License :: OSI Approved :: Apache Software License
Programming Language :: Python :: 2.7
Topic :: Scientific/Engineering :: Mathematics
Topic :: Software Development :: Libraries :: Python Modules
Topic :: Software Development :: Libraries
Entry-points:
[console_scripts]
tensorboard = tensorflow.tensorboard.tensorboard:main

@thammegowda Ok, it works now. I was using example files from tensorflow 0.9.0 from github. I updated it to 0.9.0rc0, now it works. Thank you.

@zxzhijia I am glad you found a way to get around with this issue! Good luck


Make the cache size field immutable

How to categorize this PR?

/kind enhancement
What this PR does / why we need it:
Currently we are using StatefulSet volumeClaimTemplate to provision the PVC. However the corresponding StatefulSet field that controls the PVC size is immutable. It is not possible to resize/enlarge the underlying PVC through the StatefulSet spec. See https://github.com/kubernetes/enhancements/issues/661 This commit makes the cache size field immutable until we decide how to implement the resize.
GRM currently does not try to update the volumeClaimTemplates of a StatefulSet as the field is immutable. See https://github.com/gardener/gardener/blob/292970d299b4a5b113c148fc23488b11542c8dd5/pkg/resourcemanager/controller/managedresource/merger.go#L284-L287. Hence, updating the cache's size via the Shoot spec gives the impression that the update went through and was successful but actually the update does not happen at all.
Which issue(s) this PR fixes:
Part of #3
Special notes for your reviewer:
N/A
Release note:

NONE

/test pull-gardener-extension-registry-cache-e2e-kind

/assign


feature: make retry behavior user-configurable

Motivation and Context

Users have various reasons that they need to configure retry behavior but it's currently difficult to do so. This PR addresses that unmet need.
Description

Adds config providers to retrieve retry behavior configuration from a user's profile or environment
Adds ability to configure retry behavior in code

Testing

I wrote and ran tests based the tests for pre-existing providers
Checklist

[x] I have updated CHANGELOG.md
[x] I have updated aws/SDK_CHANGELOG.md if applicable

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

ah I missed that legacy was handled by InvalidRetryMode I think this is
all good
On Fri, Oct 8, 2021 at 10:54 AM Zelda Hessler @.***>
wrote:

@.**** commented on this pull request.
In rust-runtime/smithy-types/src/retry.rs
https://github.com/awslabs/smithy-rs/pull/741#discussion_r725082840:

   set_by: String,

},
+}

+impl Display for RetryConfigErr {

fn fmt(&self, f: &mut Formatter<'_>) -> std::fmt::Result {

   use RetryConfigErr::*;

   match self {

       InvalidRetryMode { set_by, source } => {

           write!(f, "invalid configuration set by {}: {}", set_by, source)

       }

       MaxAttemptsMustNotBeZero { set_by } => {

           write!(f, "invalid configuration set by {}: It is invalid to set max attempts to 0. Unset it or set it to an integer greater than or equal to one.", set_by)

       }

       AdaptiveModeIsNotSupported { set_by } => {

           write!(f, "invalid configuration set by {}: Setting retry mode to 'adaptive' is not yet supported. Unset it or set it to a supported mode.", set_by)

I was covering legacy with the InvalidRetryMode(RetryModeError) error
where it would come up as invalid. I added the special case for adaptive
because we do intend to support it, just not yet. Would you like me to do
either of:

create a special case for legacy. it's a not-uncommon AWS settings
for other SDKs and people should get a more helpful message
remove the special case for adaptive and just say that anything
other than standard is invalid
Another option I hadn't considered

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/smithy-rs/pull/741#discussion_r725082840, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AADYKZ22LGYINKS7ATFRPZLUF4A2LANCNFSM5FME6UWQ
.


bind admin roles in Kueue

What type of PR is this?
/kind feature

What this PR does / why we need it:
binding admin roles in Kueue
Which issue(s) this PR fixes:

Fixes # https://issues.redhat.com/browse/RHOAIENG-2047
Special notes for your reviewer:
Does this PR introduce a user-facing change?

To test this task, you should find the rhods-admins group in your cluster, inside this group, in the RoleBindings tab, paste the content of binding_admin_roles.yaml from this PR. From https://console.redhat.com/openshift/ create a new user with a dedicated-admins group, then create identity providers for this user. Login into the cluster with new creds. Check if this user can create
ClusterQueue (an error should appear as this user is not a member of rhods-admins group yet). After that, inside the cluster, add a new user to rhods-admins and try to create the ClusterQueue again (it should be possible to create)

Thank you!


Services page is missing information

The services page only shows databases in it, currently. It should show other information related to the service, such as ingestion schedule, JDBC connection information etc.

This got closed by PR #235


Allow overriding of cache file paths

I'm using nix package manager and I want to use separate cache for different neovim instances. The problem right now is I manage plugins with nix. All plugin paths are immutable. So impatient.nvim isn't working correctly. Because it implements a module resolution cache. In my case after upgrading all new plugins will be in a new directory. But the previous directory isn't removed immediately. It's immutable and it stays until I clean previous generations of a nix profile. So impatient.nvim checks that there is module in cache, checks that file is valid and it's still the same. My idea is that I will use a separate cache for every immutable plugin pack (for example I'll add a hash to cache file path). This way after upgrading there will be a new cache. But I'll still be able to use already running instance of neovim form previous generation with an old cache. I already tested it by temporary changing vim.env.XDG_CACHE_HOME before requiring impatient but I'd like a more simple approach. Also it's not required, but maybe create a directory if it doesn't exist?

If a cache lookup fails then it is invalidated and re-cached. What's wrong with this?

If a cache lookup fails then it is invalidated and re-cached. What's wrong with this?

Nothing wrong with this. But in my case cache lookup never fails.
Maybe an example will help? I have plugins in e.g. /nix/store/rlz8rs0si9mdsiqvw8ywgp6hs09y6aad-nvim-data-dir directory. After I upgraded something my new plugin pack will be in, say, /nix/store/0hj82c991xc8kvrjdcbjlp79xyfx1xnh-nvim-data-dir. After I require something impatient.nvim will lookup a module, say, telescope. It founds it in a cache, then checks that file is not changed. But the problem is, old plugin pack /nix/store/rlz8rs0si9mdsiqvw8ywgp6hs09y6aad-nvim-data-dir isn't deleted, so impatient.nvim thinks that it can return a cached copy, but it's a stale copy from old plugin pack.

In that case, why can't you just run :LuaCacheClear via an autocmd each time you update your plugins?
I don't want to add any complicated logic to cover niche cases when a simple workaround exists.
The only thing I can think of in order to invalidate the module-path-cache would be to run a check against runtimepath, however runtimepath is dynamic, so I'm not quite sure what kind of check to do. Perhaps when we require a module we check for an entry in runtimepath where the module would load from, if there isn't one, then invalidate?

In that case, why can't you just run :LuaCacheClear via an autocmd each time you update your plugins?

I update my plugins not within neovim, but still I probably can save somewhere a path to plugin pack. And check it during startup, then :LuaCacheClear if a path is different from last time. But the reason why I don't want to do that, is that my old copy of neovim will probably be invalid (I often update plugins, but I don't restart my current instance of neovim right away and I want it to work with a cache from old plugin pack). If two instances of neovim with different plugin packs are running at the same time it can result to random errors.

I don't want to add any complicated logic to cover niche cases when a simple workaround exists.

Fair enough.

Perhaps when we require a module we check for an entry in runtimepath where the module would load from, if there isn't one, then invalidate?

It could work but with the same problem with two instances of neovim.
I can close this. I think I can just apply a patch to impatient.nvim during nix build process with required changes.


Allow set params from actions

use case
class UsersViewAction extends \yii\rest\ViewAction
{
    /**
     * @inheritdoc
     */
    public function run($id = null, $expand = 'profile')
    {
        if (is_null($id)) {
            $id = \Yii::$app->user->id;
        }

        return parent::run($id);
    }
}

and we have clean url like this /api/v1/user/

I don't really get it. what is the code you would write now and how does it change with this feature?

@cebe
to include default values for actions. With this changes we can get extra fields without changing url:
GET /api/users/
instead of
GET /api/users/?expand=profile

default params for URLs should be added in the UrlManager, not at action level like this. Also adding this to Serializer and not making it available in other parts of the framework is inconsistent. I can not accept this PR in this way. If you want to discuss the problem further, please open an issue. Thanks for your contribution!


WireGuard fails to connect over GPRS

Hi! First of all thank you for your code. It's been easy to understand with your example. Anyway, it works when I am connected to a wifi but it isn't when I use a GSM connection over my board (Lily T-SIM7000G). I have tried your code in the same device, with wifi and GSM and, always fails with the GSM.
Can you shed some light?

I can confirm this.
But after some retries it was able to connect properly via a wifi hotspot on my android phone or my local internet connection wifi.
After changing the connection informations and uploading the code to the ESP32 the connection will not working (connection to WIFI yes, to WG no)
after making no changes and reuploading the ESP restarts and than WG starts working again.
This happens when I switch to LTE or to Local Wifi...
Another reproduction was not possible for my.

Hi! In other GitHub post someone (https://github.com/ciniml/WireGuard-ESP32-Arduino/issues/34 ) said that the issue may be with how TinyGPSPlus provides the interface and I found this, but couldn’t made any advance https://github.com/trombik/esp_wireguard/issues/39#issuecomment-2153254489


General Projects on Python

Can i contribute in this repository with some general projects in Python

Yes, you can
But make sure you make a different folder/directory before making any contribution to this repository.

I'm adding plant disease detection project kindly merge  in PR

can i merge a project made using django framework of python?

Hello @gantavyamalviya ,I want to contribute to this repo by adding "Face mask detection" project. Please assign it to me

hey hi im new to this github,,,will any one of u please help me understanding or how to use these platform. any video suggestions  for learning github


iconsole wont load

Issue Report Checklist

[x] Searched the issues page for similar reports
[x] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[x] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[x] Tried basic troubleshooting (if a bug/error)

[x] Restarted Spyder
[x] Reset preferences with spyder --reset
[x] Reinstalled the latest version of Anaconda
[x] Tried the other applicable steps from the Troubleshooting Guide

[x] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
I upgraded spyder to 4, and when it threw the error i downgraded but with no success. I have since preformed a new clean install of anaconda(twice), and currently have spyder3.3 installed but the error remains. I am on a linux(mint) system. 3.1 had been working fine. ALSO, I updated qtconsole, but i get a 'kernel died, restarting..'when launching the console. when killing the process the error message is the same.
What steps reproduce the problem?

launch spyder3, either through anaconda-navigator or from the cmd line
launch qtconsole

What is the expected output? What do you see instead?
Expect console to open
error message
Paste Traceback/Error Below (if applicable)

An error ocurred while starting the kernel
Traceback (most recent call last):
File "/home/chris/anaconda3/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/chris/anaconda3/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/console/main.py", line 11, in
start.main()
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/console/start.py", line 272, in main
import_spydercustomize()
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/console/start.py", line 32, in import_spydercustomize
import spydercustomize
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/customize/spydercustomize.py", line 25, in
from IPython.core.getipython import get_ipython
File "/home/chris/anaconda3/lib/python3.7/site‑packages/IPython/init.py", line 54, in
from .core.application import Application
File "/home/chris/anaconda3/lib/python3.7/site‑packages/IPython/core/application.py", line 23, in
from traitlets.config.application import Application, catch_config_error
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/init.py", line 6, in
from .application import *
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/application.py", line 19, in
from traitlets.config.configurable import Configurable, SingletonConfigurable
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/configurable.py", line 12, in
from .loader import Config, LazyConfigValue, _is_section_key
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/loader.py", line 16, in
from ipython_genutils.path import filefind
File "/home/chris/anaconda3/lib/python3.7/site‑packages/ipython_genutils/path.py", line 13, in
import random
File "/home/chris/anaconda3/lib/python3.7/random.py", line 42, in
from math import log as _log, exp as _exp, pi as _pi, e as _e, ceil as _ceil
ImportError: cannot import name 'log' from 'math' (/home/chris/math.py)
Traceback (most recent call last):
File "/home/chris/anaconda3/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/chris/anaconda3/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/console/main.py", line 11, in
start.main()
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/console/start.py", line 272, in main
import_spydercustomize()
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/console/start.py", line 32, in import_spydercustomize
import spydercustomize
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/customize/spydercustomize.py", line 25, in
from IPython.core.getipython import get_ipython
File "/home/chris/anaconda3/lib/python3.7/site‑packages/IPython/init.py", line 54, in
from .core.application import Application
File "/home/chris/anaconda3/lib/python3.7/site‑packages/IPython/core/application.py", line 23, in
from traitlets.config.application import Application, catch_config_error
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/init.py", line 6, in
from .application import *
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/application.py", line 19, in
from traitlets.config.configurable import Configurable, SingletonConfigurable
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/configurable.py", line 12, in
from .loader import Config, LazyConfigValue, _is_section_key
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/loader.py", line 16, in
from ipython_genutils.path import filefind
File "/home/chris/anaconda3/lib/python3.7/site‑packages/ipython_genutils/path.py", line 13, in
import random
File "/home/chris/anaconda3/lib/python3.7/random.py", line 42, in
from math import log as _log, exp as _exp, pi as _pi, e as _e, ceil as _ceil
ImportError: cannot import name 'log' from 'math' (/home/chris/math.py)
Traceback (most recent call last):
File "/home/chris/anaconda3/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/chris/anaconda3/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/console/main.py", line 11, in
start.main()
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/console/start.py", line 272, in main
import_spydercustomize()
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/console/start.py", line 32, in import_spydercustomize
import spydercustomize
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/customize/spydercustomize.py", line 25, in
from IPython.core.getipython import get_ipython
File "/home/chris/anaconda3/lib/python3.7/site‑packages/IPython/init.py", line 54, in
from .core.application import Application
File "/home/chris/anaconda3/lib/python3.7/site‑packages/IPython/core/application.py", line 23, in
from traitlets.config.application import Application, catch_config_error
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/init.py", line 6, in
from .application import *
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/application.py", line 19, in
from traitlets.config.configurable import Configurable, SingletonConfigurable
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/configurable.py", line 12, in
from .loader import Config, LazyConfigValue, _is_section_key
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/loader.py", line 16, in
from ipython_genutils.path import filefind
File "/home/chris/anaconda3/lib/python3.7/site‑packages/ipython_genutils/path.py", line 13, in
import random
File "/home/chris/anaconda3/lib/python3.7/random.py", line 42, in
from math import log as _log, exp as _exp, pi as _pi, e as _e, ceil as _ceil
ImportError: cannot import name 'log' from 'math' (/home/chris/math.py)
Traceback (most recent call last):
File "/home/chris/anaconda3/lib/python3.7/runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "/home/chris/anaconda3/lib/python3.7/runpy.py", line 85, in _run_code
exec(code, run_globals)
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/console/main.py", line 11, in
start.main()
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/console/start.py", line 272, in main
import_spydercustomize()
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/console/start.py", line 32, in import_spydercustomize
import spydercustomize
File "/home/chris/anaconda3/lib/python3.7/site‑packages/spyder_kernels/customize/spydercustomize.py", line 25, in
from IPython.core.getipython import get_ipython
File "/home/chris/anaconda3/lib/python3.7/site‑packages/IPython/init.py", line 54, in
from .core.application import Application
File "/home/chris/anaconda3/lib/python3.7/site‑packages/IPython/core/application.py", line 23, in
from traitlets.config.application import Application, catch_config_error
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/init.py", line 6, in
from .application import *
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/application.py", line 19, in
from traitlets.config.configurable import Configurable, SingletonConfigurable
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/configurable.py", line 12, in
from .loader import Config, LazyConfigValue, _is_section_key
File "/home/chris/anaconda3/lib/python3.7/site‑packages/traitlets/config/loader.py", line 16, in
from ipython_genutils.path import filefind
File "/home/chris/anaconda3/lib/python3.7/site‑packages/ipython_genutils/path.py", line 13, in
import random
File "/home/chris/anaconda3/lib/python3.7/random.py", line 42, in
from math import log as _log, exp as _exp, pi as _pi, e as _e, ceil as _ceil
ImportError: cannot import name 'log' from 'math' (/home/chris/math.py)

Versions

Spyder version: 3.3
Python version: 3.7
Qt version: qt-5.9.6
PyQt version: pyqt-5.9.2
Operating System name/version: linux mint

Dependencies

PASTE DEPENDENCIES HERE
```chris@chris-MS-7823 ~ $ apt-cache depends spyder3
spyder3
  Depends: <python3:any>
    python3:i386
    python3
  Depends: python3-spyderlib
  Breaks: python3-spyderlib
  Replaces: python3-spyderlib

ImportError: cannot import name 'log' from 'math' (/home/chris/math.py)

This is the problem: you have a file called math.py in your home and that creates a conflict with the math library that comes with Python.
Simply renaming or removing that file will fix the error.

embarrassingly simple. This worked. wont make that mistake again. thank you


IGV viewer still displays "all" view on many amplicons

Using the prod version of AmpRepo I still get the 'all' view on many amplicons. E.g., 59M_OVARY amplicon2 gives the all view. Clicking through a few samples seemed to give either
a) single locus view (correct)
b) multiple highlighted regions for a single chromosome (not wrong, but perhaps not what we wanted?)
c) the "all" view (bug)
SW579_THYROID is an example that has amplicons with all three views above.
I couldn't figure out what properties seemed to trigger each of these three cases, but perhaps samples with multiple intervals on a chromosome were more likely to give b).

Hi Jens,
I found the issue. When the IGV tracks are being created, it will query the click data to find the chromosome number it's on. From the last fix, I added a new condition that if the ref-genome is NOT grch37, then concatenate 'chr' to the front of the chromosome number. The JS script is in a for loop and it will concatenate 'chr' to the chromosome number every time a chromosome number is found. I've changed it so it will only concatenate 'chr' to the chromosome number when the actual amplicon is found, and the fix seems to be working correctly for all reference genomes.
The fix is on branch issue_66_fix, and this issue can be closed once the PR is approved.
Edwin


Gateway - Failed to yarn start when passhrase is numerical (property must be of type string)

Gateway - Failed to yarn --start when passhrase is numerical
Steps:

git clone -b development https://github.com/hummingbot/hummingbot.git development
install, compile and start the bot
for gateway generate-certs use numbers as passphrase (1234)
open another terminal and clone gateway git clone -b development https://github.com/hummingbot/gateway gateway
cd gateway && yarn && yarn build
./gateway-setup.sh
yarn start  --passphrase=1234

Actual:
Bot returns error and Failed to start
error |   Failed to start the server with https. Confirm that the SSL certificate files exist and are correct. Error: TypeError [ERR_INVALID_ARG_TYPE]: The "options.passphrase" property must be of type string. Received type number (1234)
Expected:
Bot is able to start when passphrase is numbers

FYI: Issue was previously fixed here for gateway in hummingbot repo:  https://github.com/hummingbot/hummingbot/pull/6067

Fixed by #23


added 'position_ms' to the options in the play function

Coverage remained the same at 89.091% when pulling c6c015cb7c42710b39438b69dd57e0aa135f4060 on DoctorFishy:hotfix/play_position_ms into 37a78656198185776ca825b37e2de4a31c2ae5de on thelinmichael:master.

Coverage remained the same at 89.091% when pulling c6c015cb7c42710b39438b69dd57e0aa135f4060 on DoctorFishy:hotfix/play_position_ms into 37a78656198185776ca825b37e2de4a31c2ae5de on thelinmichael:master.

Closing since this is a duplicate of a merged PR.


Tab line rendering seems odd

The tab lines seem not to be in equal distance of each other. Also, one line is broken apart. Finally, there are no spaces or tabs in the last row, so there should not be any tab lines either.
This is v1.2.0 on OSX 10.11

Duplicate of #9369, please subscribe there for updates.


EmailTable size isn't set to contents

When using EmailTable the html table is expanding to the full size of the window, not autosizing to the content. Happens in -AttachSelf file, Outlook, and GMail
Example Code:
$processes = get-process | Select Name, Id -first 5
Email -AttachSelf {
    EmailHeader {
         EMAILSETTINGS
    }
    EmailBody {
        EmailTextBox {
            STRING CONTENT
        }
        EmailText -LineBreak
        EmailText -LineBreak
        EmailTextBox {
            MORE CONTENT
            ""
        }
        EmailTable -Table $processes -HideFooter
    }

}

Added -AutoSize to New-HTMLTable/EmailTable. This makes it fit the content rather than using 100% width for the table.


[Extension] Add CheckMetadataHash signed extension support

Related: https://github.com/polkadot-fellows/runtimes/pull/337

Version test: http://<IP_ADDRESS>:8900/s/D25rp9qsmSWjQpC
Validation passed

Install
Upgrade version
Perform transactions with Rococo

Extension: https://rococo.subscan.io/extrinsic/0x151f0eee711cbc36c408bcf60e5dd33432317219d80fc10f30fd228782edeb44
dApp: Polkadot js app

Re-check all action:

Transfer/ XCM/Send NFT/ Buy token/Receive/Crowdloans
Swap
Earning
History
Manage account/ network/ NFT
WalletConnect/ Connect dApp
Settings
Reset account/Erase all

Version test: http://<IP_ADDRESS>:8900/s/D25rp9qsmSWjQpC
Validation passed

Install
Upgrade version
Perform transactions with Rococo

Extension: https://rococo.subscan.io/extrinsic/0x151f0eee711cbc36c408bcf60e5dd33432317219d80fc10f30fd228782edeb44
dApp: Polkadot js app

Re-check all action:

Transfer/ XCM/Send NFT/ Buy token/Receive/Crowdloans
Swap
Earning
History
Manage account/ network/ NFT
WalletConnect/ Connect dApp
Settings
Reset account/Erase all


XglcdFont's glitched rendering

Hi, using your library, and a spun off version specifically for the CYD variant i have (2.4 capacitive touch with USB C). all the display draw functions work great...except XglcdFont usage. I do not know if the XglcdFont class returns faulty objects or if the draw_text and draw_letter functions are the issue.
It does not matter if I use glcd fonts found online or ones created with the tool, the rendering is always messed up (like a jumbled random image)
There seems to be some interpretation fault, and being new to microcontroller coding (my experience is with creating programs and scripts automating SolidWorks, a CAD program) I have zero idea how to  proceed because I cannot be certain if the error is with me or with the library method.

A spun off version? Can you give me more information to debug this please? More in depth hardware information, debugging logs, a copy of the font, photos etc


OCM-13124 | fix: Make route53 role arn usable for classic oproles

the route53 role arn was mistakenly put as HCP only for create/operatorroles - removed this validation

/retest-required

/retest-required

/retest-required

/retest-required

/retest-required

/retest-required

/cherry-pick release_1.2.49


setup Kanban board

Project Link
https://github.com/melashu/Space-Travelers-Hub
Group Member
We are 2 members in this group

@Masuduzzaman811 and @melashu

Hello Team👋,
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉 🎉 🎉 🎉 🎉
Cheers and Happy coding!👏👏👏

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Certain images will report exception when generating greyscale_cam

Hello.
I encountered a issue with the generation of greyscale_cam (located in core.py). In short, some images will report exception "'int' object has no attribute 'backward'", while others won't. I found out that the problem is caused by the line that generates greyscale_cam.
The line originally looks like grayscale_cam = self.method(tensor, [self.target])
Even after changing the line to grayscale_cam = self.method(tensor.requires_grad_(), [self.target]) the problem still persists.
For records, I am using YOLOv8n.pt downloaded from Ultralytics and ISIC2019 dataset. The exception occurs when I try to use any method but "EigenCAM".
Hopefully a solution can be found as soon as possible.

Thanks for raising the issue.
We have been caught up in a few exams for the past week. We will try to replicate the issue and resolve it at the earliest.

@tvc5586 can you share the any eaxct image in ISIC2019 dataset as when i treidon randon=ms ample its w working fine.


Sync doesnt want to upload.

When trying to upload via SYNC , I get an error popup window and the SYNC
doesnt seem to want to do its job.
Yes, I have already set up a GIST as I followed the instructions.
VS-Code Version: 1.32.3 (system setup)
Date: 2019-03-14T23:43:35.476Z
Electron: 3.1.6
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 6.1.7601
.

whats the error logged in console ?

for some odd reason, there is nothing in console.

The console is available on Help Menu > Toogle Developer tools
are shown in the image.

I have the same problem, and the error I get in the console is:

In my vscode setttings I have:
"http.proxyStrictSSL": false,
And I'm on Windows 10 behind a corporate proxy.

Notice that back in summer when I installed the extension, I managed to upload fine:
"lastUpload": "2018-08-27T09:37:00.854Z",

I can't upload settings because I have the following error:

https://github.com/shanalikhan/code-settings-sync/issues/799#issuecomment-473869577 is due to certificates problem. see the link here
https://code.visualstudio.com/docs/setup/network#_ssl-certificates

back in summer when I installed the extension, I managed to upload

Yes as Settings Sync was completely disabling the SSL certificates issue which was reported and fixed : https://github.com/shanalikhan/code-settings-sync/issues/776
Check the comments there and see if any options help

#799 (comment) is due to certificates problem. see the link here
https://code.visualstudio.com/docs/setup/network#_ssl-certificates

back in summer when I installed the extension, I managed to upload

Yes as Settings Sync was completely disabling the SSL certificates issue which was reported and fixed : #776
Check the comments there and see if any options help

Thank you for your answer and the links you provided. The only thing that worked for me finally was to set up a Windows environment variable NODE_TLS_REJECT_UNAUTHORIZED=0.
I realize this is a security risk, but when you are behind an MS corporate proxy like ours, you're not left with many choices.

I would like to add a small note that my sister figured out.
If you have any errors in the settings.json file after editing that file or something.
Sync will not upload or work until there are no errors in the Settings.json file of VS Code.
just a heads up, something she bumped into recently.

Yes, settings.json should be valid JSON


Image error, and won't start running.

im using the v.0.7.3 version
Describe the bug
the bot is loading the game, and joining the game. after it checks my inventory, it gives me this error
opencv(4.5.5) d:\a\opencv-python\opencv-python\opencv\modules\imgproc\src\color.cpp:182: error: (-215:assertion failed) !_src.empty() in function 'cv::cvtcolor'
how do i fix this?
i have tried to run the d2bug f10 and f9. nothing worked.

Fix your file permissions/run botty as an admin/make sure your disk isn't broken.

Please follow the template. I can't debug this.


Set BUILD_SHARED_LIBS=true and BUILD_STATIC_LIBS=false in cmake setup hook

Issue description
For consistency with --disable-static and --enable-shared configure flags.

See https://github.com/NixOS/nixpkgs/pull/56391#discussion_r264198732 for rationale.


runReader

Alias of apply.
https://hackage.haskell.org/package/transformers-<IP_ADDRESS>/docs/Control-Monad-Trans-Reader.html#v:runReader

Is this what you'd expect?
export const runReader: <R, A>(r: R) => (reader: Reader<R, A>) => A =
  (r) => (reader) =>
    reader(r)

If I were to write it just like the Haskell implementation I think it would likely be a bit annoying to use in a pipe e.g with my proposed solution you could
type Env = { env: string }
declare const myReader: RTE.ReaderTaskEither<Env, Error, string>

const getString: TE.TaskEither<Error, string> = pipe(
  myReader,
  runReader({env: 'hello'})
)

But if I were to reverse the order of r and reader to be closer to the Haskell's signature you'd end up having to do something like
type Env = { env: string }
declare const myReader: RTE.ReaderTaskEither<Env, Error, string>

const getString: TE.TaskEither<Error, string> = pipe(
  myReader,
  _ => runReader(_)({env: 'hello'})
)

I don't really know much about of Haskell so I'm sorry if I understood the signature incorrectly.
Also, wouldn't it also make sense to introduce a runReaderTask function that would runrunReader as well as Task.execute? 😄

That first runReader is it, although r => reader => reader(r) could be replaced with apply. The purpose is to provide a centralised way to run a reader that hides the implementation detail that it's just a function.
Yep, per-monad transformer variants make sense. I wouldn't run execute though, that's really for the very edges of the application. Example:
declare const runReaderTask: <R>(r: R) => <A>(m: ReaderTask<R, A>) => Task<A>

That could probably also be just apply.


Autofix for Style/StructInheritance

Adds an autofix for Style/StructInheritance.
This is my first attempt at writing developing on Rubocop so I'm open to any suggestions.  In particular I know that the autofix method is currently violating Metrics/AbcSize and Metrics/MethodLength but I'm unsure of the best way to fix this.  Should I pull out some of the fixes in to private methods in the checker?  I think that would address at least some of those violations.
I'm also sure there's a better way to do the autofix itself.  Removing the class keyword and changing the < operator to an = is simple enough, but I seem to sometimes be losing the do of a block that is passed in.  Maybe there's a different part of the AST I should be operating on.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[x] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

Sorry if this is an ill-formed question, but is there a different mechanism I could use for implementing the autocorrect?  What I mean is, rather than manipulating strings with ranges of source locations and flipping the < character to a = character, can I construct a completely new set of AST nodes that will generate the autocorrected code?
I think that might be clearer, though I'm not sure how the original source's formatting would be preserved.

Hi @bubaflub, sorry for the late response. Struct.new (like any other method) can take one block only. For Struct.new it can be used to extend the class being defined (by defining methods for the struct, for example).
irb(main):003:0> s = Struct.new(:foo) { def bar; end }
=> #<Class:0x0055c07d8f04b0>
irb(main):004:0> s.instance_methods(false)
=> [:foo, :bar, :foo=]

In the example above, class Person is then a subclass of that class that was constructed by Struct.new.
irb(main):005:0> class Person < s
irb(main):006:1>   def baz; end
irb(main):007:1>   end
=> :baz
irb(main):008:0> Person.instance_methods(false)
=> [:baz]
irb(main):009:0> Person.superclass
=> #<Class:0x0055c07d8f04b0>
irb(main):010:0> Person.superclass.instance_methods(false)
=> [:foo, :bar, :foo=]

The example above was more a pathological one. I cannot imagine someone actually writing such code. But it would be good to guard against such cases and to not attempt autocorrection if Struct.new already has a block.
As to the mechanics of performing auto-corrections: Unfortunately, to my knowledge, in current RuboCop there is no way of transforming the AST in order to rewrite the code. At the moment, it is all low-level string operations on the source text, with all the potential problems that entails. :no_mouth:

Hi @bubaflub, sorry for the late response.

No worries!  Thanks for taking time to respond and helping me better understand Rubocop and the Ruby language.

The example above was more a pathological one... But it would be good to guard against such cases...

Fully agree!  Who knows what code lurks out there and I'd rather have an autocorrect not fix something that it might have been able to fix than to an autocorrect change code incorrectly.
I'll rebase this branch, fix up the commit, and add a few more tests.

I think this should do it for the auto-correct -- we now ignore the autocorrect if the Struct itself has a block.
Should I have updated config/default.yml to specify AutoCorrect: false or SafeAutoCorrect: false?

Should I have updated config/default.yml to specify AutoCorrect: false or SafeAutoCorrect: false?

SafeAutoCorrect: false

SafeAutoCorrect: false

Added that config, rebased off of latest master, updated the CHANGELOG, regenerated the docs, and pushed.

@bubaflub And I totally missed your message back then. Mind going over the rebase process one more time?

@koic @dvandersluis Would one of you be interested in wrapping up this PR?

@bbatsov the majority of this PR was implemented separately in other PRs, such as #8111.
The only thing missing from this PR is a fix for this case:
class Person < Struct.new(:first_name, :last_name) do def baz; end end
  def age
    42
   end
end

Which I'm not sure we need to even handle? It doesn't look like code that someone would actually write haha.

Roger that! Thanks for looking into this! 🙇♂


Why get-models api after splitting the column does not return the column type with some value even click on "Guess cell type" while splitting the column

Describe the bug
I have doubt before splitting the column the get models api return various parameter in which one parameter is type to know the data type of the column which is fine but after while splitting the column there is an option Guess the cell type"even checking that option hoping that in get models response after splitting the new column will also return the type parameter with some value (string/number/so on) but it return the type as blank value.
To Reproduce
Steps to reproduce the behavior:

Upload the data set
Click on create project
After creating the project check the get-models api response particularly the type parameter
Split any column remember to checked Guess cell type option and click on ok
After splitting check the get-models response particularly the type parameter

Expected behavior
As before splitting i am getting the type parameter with some value which predict the data type of the column even after splitting the column (clicking on Guess cell type while splitting) the response should return type with some value but right now it is returning with blank value.
Screenshots
Before splitting the column the type parameter

After splitting (while splitting i have checked Guess cell type) the column the type parameter

OpenRefine (please complete the following information):

OpenRefine 3.0 Beta

Even i am adding video link for better understanding.

Apologies for this getting ignored for so long. I'm guessing that you either found your answers or lost interest, but if you still need help let us know.

The type parameter in column metadata is unused, it was introduced for Data Package support but without any way to make anything useful out of it.


[BugFix] Account finalization cost to scheduler

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #16432
Problem Summary(Required) ：

When a large query is finalized, especially when it is cancelled and resource group is enabled, the other small query will wait a long time.
The reason is that, finalization cost of driver isn't accounted in driver_queue. For exmaple, there is a group rg1 executing a large query and finalizing and a group rg2 executing a small query like select 1. If rg1.vruntime < rg2.vruntime, then all the drivers of this large query will be finalized before executing select 1 in `rg2.
Therefore, account the finalization cost time to driver_queue.
Test case
rg1 executes large queries and rg2 executes select 1.
# would fail by OOM
mysqlslap --concurrency=8  \
           --iterations=1  \
           --number-of-queries="100000"      \
           --query="tpcds/query04.sql"   \
           --delimiter=";" -h <host>  -P <port> -u rg1 -p123456  --create-schema=tpcds_100g

mysqlslap --concurrency=1  \
           --iterations=100  \
           --number-of-queries="1"      \
           --query="select 1 from dual"   \
           --delimiter=";" -h <host>  -P <port> -u rg2 -p123456  --create-schema=tpcds_100g

Before this PR, enabling resource group costs 20s and disabling resource group costs 4s.
After this PR, enabling and disabling costs 0.3s.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 2.5
[x] 2.4
[x] 2.3
[ ] 2.2

run starrocks_admit_test

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.3


Optimize create uneval tree via memoization

Description
For every evaluation, uneval tree is computed on the main thread and sent to the worker. In moderately complex apps, this step takes between 40 to 300ms. Most of the computations in this step are rework because none of the widgets change, and the only change is meta properties.
This PR separates the part dependent on widget and metaprops, and memoizes the expensive part dependent on widget
Fixes #12966
Type of change

Non breaking change which improves performance

How Has This Been Tested?

Profiled and verified the performance improvements
Compared the output of the changes with the old one.

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:red_circle: Total coverage has decreased
// Code coverage diff between base branch:release and head branch: feat/optimized-memoize-create-uneval-tree

Status
File
% Stmts
% Branch
% Funcs
% Lines

:red_circle:
total
56.55 (-0.04)
38.38 (-0.01)
35.84 (0)
56.8 (-0.04)

:red_circle:
app/client/src/entities/DataTree/dataTreeFactory.ts
37.74 (-8.6)
100 (0)
50 (0)
36.54 (-8.46)

:red_circle:
app/client/src/entities/DataTree/dataTreeWidget.ts
60.68 (-37.43)
41.67 (-41.66)
57.14 (-42.86)
59.8 (-38.07)

:green_circle:
app/client/src/entities/Widget/utils.ts
92.04 (0.15)
89.13 (0)
100 (0)
91.96 (0.22)

:green_circle:
app/client/src/utils/WorkerUtil.ts
89.76 (0)
72.55 (1.96)
100 (0)
93.33 (0)

:red_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.71 (-0.23)
40.83 (-0.84)
36.21 (0)
56.74 (-0.25)

:green_circle:
app/client/src/widgets/MultiSelectWidget/component/index.tsx
19.44 (0.26)
0 (0)
0 (0)
20.59 (0.3)

/ok-to-test sha=683a514

/ok-to-test sha=4b80e25

/ok-to-test sha=bd271ab

/ok-to-test sha=7c4e456

/ok-to-test sha=c3f6562

/ok-to-test sha=121111d

/ok-to-test sha=86dbd23

/ok-to-test sha=86dbd23

/ok-to-test sha=e53bfb0

/ok-to-test sha=e53bfb0

/ok-to-test sha=08c2da2

/ok-to-test sha=08c2da2

/ok-to-test sha=20523c2

/ok-to-test sha=20523c2

/ok-to-test sha=f5d0524

/ok-to-test sha=d1b9a06

/ok-to-test sha=90e8289

/ok-to-test sha=21fb4a0

/ok-to-test sha=cee2807

/ok-to-test sha=bfb4e9a


Fix TestRuntimeHandler logging

Resolving issue #5032:

An empty --runtime-handler flag results in testing against the default runtime so modified and removed logging to clarify.
Added a TODO for future work on potentially adding test(s) for multiple runtimes at the same time.

/cc @mikebrow

/ok-to-test

@chrisfregly would you squash commits? The change is small. Thanks

Commits have been squashed. Thanks for reviewing


refactor: remove the global package in dfdaemon

Signed-off-by: inoc603<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
Remove the global package in dfdaemon to make testing (which I haven't done yet) easier.
This PR is larger than I would like, since the global package is referenced in so many places.
Notable changes:

remove dfdaemon/global and cmd/dfdaemon/app/options
remove dfdaemon/initializer and move some of its functions to cmd/dfdaemon/app
remove cli params "rule" and "trust-hosts"
use github.com/spf13/viper for config management and make most of the cli parameters configurable from config file

Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
Everything should work as before. Additionally these fields from the cli parameters are now configurable through config file, thanks to github.com/spf13/viper
port: 65002
hostIp: <IP_ADDRESS>
certpem: ""
keypem: ""
node: []
localrepo: /root/.small-dragonfly/dfdaemon/data
dfpath: /root/src/github.com/inoc603/Dragonfly/bin/linux_amd64/dfget
ratelimit: 50M
urlfilter: Signature&Expires&OSSAccessKeyId
callsystem: com_ops_dragonfly
notbs: true
verbose: true
maxprocs: 4

Ⅴ. Special notes for reviews

Codecov Report

Merging #581 into master will decrease coverage by 0.74%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #581      +/-   ##
==========================================
- Coverage   47.36%   46.62%   -0.75%
==========================================
  Files          97       96       -1
  Lines        5578     5577       -1
==========================================
- Hits         2642     2600      -42
- Misses       2718     2760      +42
+ Partials      218      217       -1

Impacted Files
Coverage Δ

dfdaemon/config/config.go
0% <0%> (ø)
:arrow_up:

dfdaemon/proxy/proxy.go
8.54% <0%> (-1.52%)
:arrow_down:

supernode/daemon/mgr/scheduler/manager.go
25.19% <0%> (-0.79%)
:arrow_down:

dfget/core/api/supernode_api.go
100% <0%> (ø)
:arrow_up:

...t/core/downloader/p2p_downloader/p2p_downloader.go
0% <0%> (ø)
:arrow_up:

dfget/core/helper/test_helper.go
0% <0%> (ø)
:arrow_up:

...get/core/downloader/p2p_downloader/power_client.go
52.23% <0%> (+1.12%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 82daa6e...54ba2b4. Read the comment docs.

Dfdaemon will get errors when the /etc/dragonfly/dfdaemon.yml is not exist. Should we add use the default config instead of return errors directly?

Dfdaemon will get errors when the /etc/dragonfly/dfdaemon.yml is not exist. Should we add use the default config instead of return errors directly?

I noticed that as well. I think you're right, we should ignore the error that config file is not found, but only for the default config file /etc/dragonfly/dfdaemon.yml. If user specified a config file using --config, we should return the error.

@Starnop I've made some changes to ignore the not found error for default config file, and added tests  for it. Please review.

LGTM. And could you please update related docs ? THX. @inoc603

LGTM. And could you please update related docs ? THX. @inoc603

Sure. Would you please open a new issue for the documentation? The changes on https and proxies can be updated together.

LGTM. And could you please update related docs ? THX. @inoc603

Sure. Would you please open a new issue for the documentation? The changes on https and proxies can be updated together. FAQ, quick_start, and user_guide all need some update, I think it's going to be quite some work.
@Starnop @lowzj Can we merge this first? So I can work on tests and docs for dfdaemon

I will submit an issue ASAP. 😄
And as for this PR, I hope that @lowzj will have a review again. After a comment LGTM from him ,and then I will merge it. I also hope that this PR can be merged ASAP.

Rebased


Add Reverb option for positional audio [Enhancement]

In game engines, filters and convolvers are used for environment-responsive sound. This is used to make sound feel more immersive (for instance, muffling speech behind a wall vs being in the same room with the speaker). To implement this, a developer would need to be able to set rules on the filter or convolver based on the geometry of the scene. Examples of this:

When position of camera is in some area mapped by a box (i.e. a room), set a low pass filter cutoff frequency on an audio emitter in the same box high. When the camera leaves the room, set the frequency low.

Add a Convolver node in series with the Panner in PositionalAudio and have them both connect to the common output. The distance may cut off the volume of a raw sound quickly, but the falloff for the Convolver should be much more subtle, giving a much more immersive sound. This way you can model environments anywhere from small rooms to giant subway tunnels along with the position of the source of sound.

Hi
I want to add some audio effects down the road but i want to include them as "nodes" and have gui to add and connect these nodes.
This should be easy to implement with web audio since the API already uses this "node" structure.
Regarding zones (rooms), the zone change should already be easy to implement using the physics engine.
Right now im trying to focus all my attention on the animation component of the editor. But if want to feel free to create a PR with some of these effects. There are still some problems with audio in the engine, i have to manage better when to dispose my buffers to avoid getting error messages.
Thanks a lot for the suggestion i really appreciate it.


Editing SSL keys should show some confirmation that keys already exist

keys are displayed when you choose to paste keys


Adapt to brand-new Idobata™

:+1:

Thanks a lot!
I merged and now checking another points to allow using brand-new Idobata.


Implement logic for PLZ test runs

Subtasks:

[X] Figure out resources conversion.
[X] Unpacking test run data fully, including pre-signed URL thing.
[X] The script is expected to be downloaded by URL. Consider options on how best to pass the script to the test run.
[X] PLZ test run is a test run with cloud output but it is already created; so TestRunID field is set. That implies that CloudTestRun & CloudTestRunCreated should be set to True. Does such a test need other changes in logic? Confirm.
[X] Add sending state info to the backend (on errors?) - awaiting API
[X] Polling state info from the backend and stopping test in case of abort.

Update: the last part (events) was added in PR https://github.com/grafana/k6-operator/pull/283


Explore RWKV-LM

How can we use RWKV-LMM to implement "infinite" context lengths?
From https://github.com/BlinkDL/RWKV-LM:

RWKV is an RNN with transformer-level LLM performance. It can be directly trained like a GPT (parallelizable). So it's combining the best of RNN and transformer - great performance, fast inference, saves VRAM, fast training, "infinite" ctx_len, and free sentence embedding.

Additional reading:
From Deep to Long Learning?
Hyena Hierarchy: Towards Larger Convolutional Language Models

RWKV-LM can be used throughout the project to:

summarize texts associated with the screenshots
better suggest prompts (like google's autofill)
better understand prompts from the user
in the future: handle translations

Implementation:

can get "infinite" context lengths through iterative refinement

states would be the recording up until that point and the token would be the next screenshot

Limitation:

this process can be computationally expensive

As Dian mentioned above, RWKV-LM could be helpful for us since it could analyze large amounts of past user actions to better predict future actions. It could also learn how to do more complex tasks that the user themselves don't know how to do, as it could learn from longer texts like instruction manuals. These benefits would also help with efficiency and speed up the automation.
As for implementation, we could consider combining RWKV-LM with another tool that could help with the graphical side of things. For example, we could have RWKV_LM understand the task we wish to execute and lay out the steps needed, then using something like OCR or MiniGPT-4, we could handle how to execute the steps on the GUI.


Improve the recoverability of DoFn

What would you like to happen?
During a recent Dataflow streaming job, an ongoing DoFn experienced a period of frequent 429 error responses from an overloaded Elasticsearch sink (using ElasticsearchIO). During this time, it was observed that autoscaling occurred due to increased backlog, but workers did not re-establish their connections once the sink was back online, causing the pipeline to remain frozen.
Steps should be taken to improve the resiliency of DoFn to these sorts of disruptions and ensure that they resolve themselves without the pipeline needing to be restarted manually.
Issue Priority
Priority: 2 (default / most feature requests should be filed as P2)
Issue Components

[ ] Component: Python SDK
[X] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

To add more context, the connections are created in the Setup method, that are ran once per DoFn
https://github.com/apache/beam/blob/master/sdks/java/io/elasticsearch/src/main/java/org/apache/beam/sdk/io/elasticsearch/ElasticsearchIO.java#L2417
If the connection breaks, it won't be restated until a new DoFn instance is created (due to new workers or, as in this case, a Dataflow Runner V2 update)

It's unclear to me how this could be improved, moving the connection creation to startBundle would mean many more connections and lower performance.
The only approach I can think of is to check fi the connection   is alive in startBundle, and if it's not, recreate it

thanks for reporting @InigoSJ / @BjornPrime 😊  The approach to alter when/where connections are created is a good idea so long as, as was mentioned, connections are not created unnecessarily or in an uncontrolled manner.
@InigoSJ Based on what was observed, do you happen if there is a way to tell programmatically that a connection is alive? Maybe within the code to handle bad statuses from ES (429's) would be a good spot to re-create connections?

Just as side note, I have never used this IO besides this Git Issue, so I am definitely not an authority here.
I think the idea I mention above should work, check if the connection is alive in StartBundle() and if it's not recreate. I would do something like this:
ElasticIO {
Connection connection

@SetUp() {
connection = CreateConnection()
}

@StartBundle() {
if (!connection.IsAlive()) {
    connection = CreateConnection
}
}

@ProcessElement {
....
}

CreateConnection {
     Same code as now
}

}

Again, I don't know the intricacies of the IO, so this may not be possible, but any other idea I can come up with seems to have more downsides that the current approach


PHOENIX-6288 Minicluster startup problems on Jenkins

increase minicluster master startup timeout to 200 seconds
log minicluster startup time

Nice one. Default value of hbase.master.init.timeout.localHBaseCluster is also 200s.

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 22s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 1 new or modified test files.

_ master Compile Tests _

+1 :green_heart:
mvninstall
14m 14s
master passed

+1 :green_heart:
compile
0m 55s
master passed

+1 :green_heart:
checkstyle
0m 26s
master passed

+1 :green_heart:
javadoc
0m 46s
master passed

+0 :ok:
spotbugs
3m 36s
phoenix-core in master has 973 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
10m 50s
the patch passed

+1 :green_heart:
compile
1m  3s
the patch passed

+1 :green_heart:
javac
1m  3s
the patch passed

+1 :green_heart:
checkstyle
0m 28s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 53s
the patch passed

+1 :green_heart:
spotbugs
4m  5s
the patch passed

_ Other Tests _

-1 :x:
unit
248m 25s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 28s
The patch does not generate ASF License warnings.

290m 19s

Reason
Tests

Failed junit tests
phoenix.end2end.RegexpReplaceFunctionIT

phoenix.tx.TransactionIT

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1045/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/1045

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux 8fe6c67c2630 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / 3891050

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1045/1/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1045/1/testReport/

Max. process+thread count
6165 (vs. ulimit of 30000)

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1045/1/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Doc divisible assets

Small example with divisible assets
This was branched from #158. The only file actually changed was docs/usage.rst

Current coverage is 100% (diff: 100%)

Merging #166 into master will not change coverage

@@           master   #166   diff @@
====================================
  Files           9      9
  Lines         191    191
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
  Hits          191    191
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update 9f9c8ea...45fa5b6


Update AABB algorithms

Resolves #163

Codecov Report

Merging #164 into master will increase coverage by 0.02%.
The diff coverage is 86.48%.

@@            Coverage Diff             @@
##           master     #164      +/-   ##
==========================================
+ Coverage   80.68%   80.70%   +0.02%
==========================================
  Files         100      100
  Lines        3345     3354       +9
  Branches      280      282       +2
==========================================
+ Hits         2699     2707       +8
- Misses        415      416       +1
  Partials      231      231

Flag
Coverage Δ

#unittests
80.70% <86.48%> (+0.02%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/shape/aabb.ts
76.66% <60.00%> (-8.63%)
:arrow_down:

src/standard/collision/algorithm/aabb_algorithm.ts
97.22% <96.29%> (+5.91%)
:arrow_up:


Wallets stateful widget

The Wallets slice should become a Stateful widget. It receives an address list and a TIKI SDK as parameters.
The widget shows the current TIKI SDK address in the card. When tapped it navigates to another screen, showing the Wallet List. It has the methods for creating and switching the wallet and sends it back to home with the Navigator.pop method.


Issue with Citeseer DataSet Shape

Hi there,
There is a discrepancy between the dimensions of data matrices for Citeseer: adjacency matrix is (3327, 3327), while features and labels have 3312 rows which is the number of nodes.
Based on the data source at http://www.cs.umd.edu/~sen/lbc-proj/LBC.html there must be 3312 nodes. Why adjacency matrix has more row/columns than Graph node size in this repo?
Thanks!

Hi there,
There is a discrepancy between the dimensions of data matrices for Citeseer: adjacency matrix is (3327, 3327), while features and labels have 3312 rows which is the number of nodes.
Based on the data source at http://www.cs.umd.edu/~sen/lbc-proj/LBC.html there must be 3312 nodes. Why adjacency matrix has more row/columns than Graph node size in this repo?
Thanks!

Hi there,
I got the same issue, have you any response for it?
Would be great if you share it.
Cheers

Same problem . Please  let me know if anyone solved this .

Sorry for late reply - I did not get any response and I ended up using the data set from other sources which matches the description.


Need example for BLU motion triggering switch without cloud

Is your feature request related to a problem? Please describe.
Need to replace Shelly cloud scene solution with script because deployment does not have Internet.
Describe the solution you'd like
Provide same functionality without cloud, that scene allows to do with cloud.
Describe alternatives you've considered
Existing scripts in https://github.com/ALLTERCO/shelly-script-examples, e.g.: ble-shelly-motion.js do not suffice.
Additional context

Need for script to trigger gen2 switch on based on motion, and off based on nomotion.
Need for script to support time-range when triggering is effective.
I tried to understand how to add time condition to ble-shelly-motion.js,
but and got frustrated by js complexity of Sys.GetStatus() to use callback only (not a js expert).
Need for script to be reliable in case of BLE messages being lost.
If nomotion BLE message is lost, switch will stay on for a long time. Unacceptable.
Could add code to switch off after some timeout, but need to be js expert to do that.
And if switching off after some timeout, then BLE motion would need to periodically send motion events
But only when there is motion, not when there is not (save battery).
but it is unclear to me how to set up BLE motion to periodically send motion event when there is ongoing motion.
and then how to program script on switch to reset nomotion timeout event.

ok. i found ble-shelly-motion.js and managed to enhance it to what i need. Attached below.
Maybe you want to use it.
ble-shelly-motion-enhanced.js.txt


rfcs/008 - Best UX for long-running, blocking operations

It was discussed offline that it would be best to a have good message setting the expectation for how long iasql_commit or iasql_install can last for the time being and continue to monitor the issue and collect more data about the issue and how to best move forward https://github.com/iasql/iasql/pull/2311


bump-cask-pr: replace --write with --write-only in help text

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

--write is deprecated, so it should say --write-only instead.

Thanks again @FnControlOption!
FYI: I think your committing email address needs added to your GitHub account if you don't want all your PRs to show up as from a "first-time contributor" 😁


Use correct media type for config blob in schema2 manifest

The schema2 manifest builder fills in this part of the manifest based on
the descriptor it gets back from BlobIngester's Put method. It passes
the correct media type to Put, but Put ends up replacing this value with
application/octet-stream in its return value.
This commit works around the issue in the manifest builder. Arguably Put
should not be changing the media type in its return value, but this
commit is a targeted fix to keep it very low-risk for possible inclusion
in Docker 1.11.
Fixes #1621 (but maybe we should open a separate issue for the media
type behavior in the distribution client, and the unnecessary stat).

LGTM

LGTM

Current coverage is 51.22%

Merging #1622 into master will decrease coverage by -6.70% as of 2de3f1a

@@            master   #1622   diff @@
======================================
  Files          122     122
  Stmts        10682   10683     +1
  Branches       839     747    -92
  Methods          0       0
======================================
- Hit           6188    5472   -716
+ Partial        839     747    -92
- Missed        3655    4464   +809

Review entire Coverage Diff as of 2de3f1a

Uncovered Suggestions

+0.30% via .../driver/s3-aws/s3.go#732...763
+0.28% via .../driver/s3-aws/s3.go#268...297
+0.21% via .../driver/s3-aws/s3.go#884...906
See 7 more...

Powered by Codecov. Updated on successful CI builds.


[Bug] echarts.time.format get wrong time.

Version
5.2.2
Link to Minimal Reproduction
https://codesandbox.io/s/echarts-time-bug-mxphv
Steps to Reproduce
Just look at the result logged in the console.
Current Behavior
console.log(echarts.time.format("2021-12-25T00:00:00Z", "yyyy-MM-dd", true) );
// yyyy-MM-dd

Expected Behavior
console.log(echarts.time.format("2021-12-25T00:00:00Z", "yyyy-MM-dd", true) );
// 2021-12-25

Environment
- OS: macOS Big Sur
- Browser: Chrome Version 96.0.4664.110 (Official Build) (x86_64)
- Framework<EMAIL_ADDRESS>
Any additional comments?
when I use echarts heatmap calendar examples: Simple Calendar

See #12859. You need to wrap the format with {}.

@plainheart  LGTM closing it...


Base Struct parameter "daysAtCritical" not initialised

type Base struct {
	id             string
	hp             int
	floor          int
	agentType      int
	inbox          *list.List
	tower          *Tower
	mx             sync.RWMutex
	logger         log.Entry
	hasEaten       bool
	daysAtCritical int
}

func NewBaseAgent(world world.World, agentType int, agentHP int, agentFloor int, id string) (*Base, error) {
	if world == nil {
		return nil, errors.New("agent needs a world defined to operate")
	}
	tower, ok := world.(*Tower)
	if !ok {
		return nil, errors.New("agent needs a tower world to operate")
	}
	logger := log.WithFields(log.Fields{"agent_id": id, "agent_type": agentType, "reporter": "agent"})
	return &Base{
		id:        id,
		hp:        agentHP,
		floor:     agentFloor,
		agentType: agentType,
		tower:     tower,
		inbox:     list.New(),
		logger:    *logger,
		hasEaten:  false,
	}, nil
}

Tbf, this is handled in hpDecay, but might be nice for posterity

Int are initialized with 0 in go, so it is not necessarily needed to initialize it specifically in my opinion


Download mnist dataset from Yann LeCun website.

I am trying to create a script that downloads all the datasets used in this repository. However, LSTM electricity consumption and google stock dataset do not seem easy to find online.
The LSTM datasets seem similar to the one from the Kaggle website (maybe part of it) https://www.kaggle.com/uciml/electric-power-consumption-data-set.
https://www.kaggle.com/uciml/electric-power-consumption-data-set.
Since Kaggle does not allow us to download the dataset unless if we have an account, the script will not be able to download the dataset.
Is it possible to move a copy to mlpack website? (I do not know the bandwidth limit). We can compress well these data since they are all text, this will reduce the size of the files.
If this is the case, we can delete all the Kaggle website and allow the user to download the dataset if required.
Signed-off-by: Omar Shrit<EMAIL_ADDRESS>

Great idea!

Is it possible to move a copy to mlpack website? (I do not know the bandwidth limit). We can compress well these data since they are all text, this will reduce the size of the files.

Yes it is possible, It was discussed on the video meet that we could e-mail @rcurtin the datasets so that they could be uploaded to https://www.mlpack.org/datasets/ . I think more info can be found here.

Could you update the readme to reflect the addition of this step?

@kartikdutt18 I agree. I remember it from video chat, but I was not able not to find any traces for the website.
Now, I can continue downloading and extracting other datasets.

@birm Agreed. I will update the readme as well.

Hey @shrit, Do you think adding arguments to download either all or selected datasets would be better? I think that way user can download only the datasets that he / she requires.

@kartikdutt18 I agree, that would be much easier faster and save bandwidth.

Nice, really useful to add this.  But do you think maybe it might be better to implement it in Python so that Windows users have an easier time?  It seems there are a lot of Windows users, so we should probably find some way to be able to serve their needs too. :)  Alternately, we can probably provide a Windows shell script or something, but it seems easier just to use Python libraries to do the same thing.  Let me know what you think. :+1:

Ok, Python is a good idea since I do not use windows.
I will follow the same logic by allowing to download dataset separately, but I do not know If I keep the same dataset or update it with a new one? Because in all cases they are not on mlpack website yet.

It's up to you.  I can store them (in preprocessed form too) on mlpack.org if you like, just let me know what files you want and I'll add them to mlpack.org/datasets/. :+1:

Ok, I've added the datasets to the website:

https://www.mlpack.org/datasets/examples/electricity-usage.csv
https://www.mlpack.org/datasets/examples/Google2016-2019.csv
and then we can get the MNIST data directly from the source.

@zoq @rcurtin @birm This one is ready for review, no hurry of course.
Thanks.

@zoq tell me what do you think, I think it is ready to get merged

@zoq I have moved the script into a tools directory


Add Glenpool High School (Glenpool, OK) to domains

@wright1703 There are several files in the request, along with file.zip. It must not be there. Please remove file.zip and make pull request again.  I have to close this request without merging. Thank you for your understanding.