live() method value does not reset in infinite mode

Hi !
Thanks for your very nice plugin. I've been through a lot of plugins like these before finally finding yours, which is very light and gives all the options I need.
I have noticed the live() method does not return the correct value in infinite mode after the first loop.
Say you have 4 slides looping indefinitely, the live() method should return (in my opinion) :
0, 1, 2, 3, 0, 1, 2, 3, 0, 1, 2, 3...
But instead it returns :
0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10...
Is this a wanted behavior ?
Thanks for your help!

Hi!
Thanks for your finding - this is definitely not the wanted behavior.
Your assumption of the order (0, 1, 2, 3, 0, 1, 2, 3, 0, 1, 2, 3...) is correct.

@gterras I've created a quick JSBin and it seems to output the correct values. Can you please share a bit more about your use case when does the error happen? Thanks!

Hi @zoltantothcom,
Thank you for your answer. I have boiled down the case to the option dots : false, just change this in your demo and the bug will show up.
I suppose this code is responsible  :
function adjustCurrent(val) {
    current += val;

    if (options.dots) { // If dots:false, current is perpetually incremented
        currentDot();
    }
}

Thank you and have a nice day !

Thank you for taking the time to report the issue! Now it's fixed.

Nice work, thank you !


Add Java Record support

Co-authored with @buchananwill
I'm opening this to do review, add tests, and make some changes.
Resolves #51

Cool, I think my next pull request will cover all this. Do you want me to add tests within that PR? I would expect to test:

schema is generated correctly to match an equivalent declared as a class
any annotations currently supported are picked up from the constructor params same as class field declarations

My current implementation does these.

@buchananwill I don't mind adding some tests here. I have some free time today so I'm going to go through and work on a few of these. I think the most useful thing at this stage is if you can figure out a way to write tests for the Gradle plugin? I don't know how to do that

@buchananwill I've cleaned this up and added tests that match a POJO test. Please let me know what you think. I can release this today unless you have any concerns that I should address

Cool, I've had quick read through the TypeDescriptor class and test and
that all looks good. I'll be at a birthday party for the rest of the day
(mine, plus my sister in laws' who have birthdays this week also!) so
probably won't get a chance to do any work till tomorrow.
I'm happy for you to release though if it meets your satisfaction. 👍
On Sat, 28 Sept 2024, 17:26 Ivan Greene, @.***> wrote:

@buchananwill https://github.com/buchananwill I've cleaned this up and
added tests that match a POJO test. Please let me know what you think. I
can release this today unless you have any concerns that I should address
—
Reply to this email directly, view it on GitHub
https://github.com/ivangreene/java-to-zod/pull/52#issuecomment-2380730136,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBL2M6OXQ7FQUEVF4Q5KKDDZY3KEJAVCNFSM6AAAAABPAWAUROVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGOBQG4ZTAMJTGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

@buchananwill I merged the code, but the release was botched due to something changing with the Maven central repository. I'll look into that later

This has been released in version 0.6.1


ElMo not working

Hello there,
Thank you for making TensorFlow Hub, it is an awesome tool to simplify the process. Recently I tied to use ELMO in TensorFlow Hub. However, I did the procedures online:
elmo = hub.Module("https://tfhub.dev/google/elmo/1", trainable=True)
This gives me
INFO:tensorflow:Using /tmp/tfhub_modules to cache modules.
INFO:tensorflow:Initialize variable module/aggregation/scaling:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with aggregation/scaling
INFO:tensorflow:Initialize variable module/aggregation/weights:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with aggregation/weights
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_0:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/W_cnn_0
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_1:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/W_cnn_1
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_2:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/W_cnn_2
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_3:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/W_cnn_3
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_4:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/W_cnn_4
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_5:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/W_cnn_5
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_6:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/W_cnn_6
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_0:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/b_cnn_0
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_1:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/b_cnn_1
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_2:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/b_cnn_2
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_3:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/b_cnn_3
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_4:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/b_cnn_4
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_5:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/b_cnn_5
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_6:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN/b_cnn_6
INFO:tensorflow:Initialize variable module/bilm/CNN_high_0/W_carry:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN_high_0/W_carry
INFO:tensorflow:Initialize variable module/bilm/CNN_high_0/W_transform:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN_high_0/W_transform
INFO:tensorflow:Initialize variable module/bilm/CNN_high_0/b_carry:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN_high_0/b_carry
INFO:tensorflow:Initialize variable module/bilm/CNN_high_0/b_transform:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN_high_0/b_transform
INFO:tensorflow:Initialize variable module/bilm/CNN_high_1/W_carry:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN_high_1/W_carry
INFO:tensorflow:Initialize variable module/bilm/CNN_high_1/W_transform:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN_high_1/W_transform
INFO:tensorflow:Initialize variable module/bilm/CNN_high_1/b_carry:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN_high_1/b_carry
INFO:tensorflow:Initialize variable module/bilm/CNN_high_1/b_transform:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN_high_1/b_transform
INFO:tensorflow:Initialize variable module/bilm/CNN_proj/W_proj:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN_proj/W_proj
INFO:tensorflow:Initialize variable module/bilm/CNN_proj/b_proj:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/CNN_proj/b_proj
INFO:tensorflow:Initialize variable module/bilm/RNN_0/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/bias:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/RNN_0/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/bias
INFO:tensorflow:Initialize variable module/bilm/RNN_0/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/kernel:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/RNN_0/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/kernel
INFO:tensorflow:Initialize variable module/bilm/RNN_0/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/projection/kernel:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/RNN_0/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/projection/kernel
INFO:tensorflow:Initialize variable module/bilm/RNN_0/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/bias:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/RNN_0/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/bias
INFO:tensorflow:Initialize variable module/bilm/RNN_0/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/RNN_0/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel
INFO:tensorflow:Initialize variable module/bilm/RNN_0/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/projection/kernel:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/RNN_0/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/projection/kernel
INFO:tensorflow:Initialize variable module/bilm/RNN_1/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/bias:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/RNN_1/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/bias
INFO:tensorflow:Initialize variable module/bilm/RNN_1/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/kernel:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/RNN_1/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/kernel
INFO:tensorflow:Initialize variable module/bilm/RNN_1/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/projection/kernel:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/RNN_1/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/projection/kernel
INFO:tensorflow:Initialize variable module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/bias:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/bias
INFO:tensorflow:Initialize variable module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel
INFO:tensorflow:Initialize variable module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/projection/kernel:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/projection/kernel
INFO:tensorflow:Initialize variable module/bilm/char_embed:0 from checkpoint /tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables with bilm/char_embed
Then I tried this
`
embeddings = elmo(
["the cat is on the mat", "dogs are in the fog"],
signature="default",
as_dict=True)["elmo"]
sess = tf.Session()
sess.run(elmo)
`
Then I got this error:
tensorflow.python.framework.errors_impl.FailedPreconditionError: Error while reading resource variable module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel from Container: localhost. This could mean that the variable was uninitialized. Not found: Container localhost does not exist. (Could not find resource: localhost/module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel)
[[Node: module_apply_default/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel/Read/ReadVariableOp = ReadVariableOpdtype=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:CPU:0"]]
[[Node: module_apply_default/bilm/add_1/ReadVariableOp/_177 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device_incarnation=1, tensor_name="edge_562_module_apply_default/bilm/add_1/ReadVariableOp", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"]]
Caused by op u'module_apply_default/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel/Read/ReadVariableOp', defined at:
File "", line 4, in
File "/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow_hub/module.py", line 203, in call
name=name)
File "/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow_hub/native_module.py", line 447, in create_apply_graph
restore_collections_predicate=(lambda key: key in import_collections))
File "/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1955, in import_meta_graph
**kwargs)
File "/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/framework/meta_graph.py", line 743, in import_scoped_meta_graph
producer_op_list=producer_op_list)
File "/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/util/deprecation.py", line 432, in new_func
return func(*args, **kwargs)
File "/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/framework/importer.py", line 513, in import_graph_def
_ProcessNewOps(graph)
File "/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/framework/importer.py", line 303, in _ProcessNewOps
for new_op in graph._add_new_tf_operations(compute_devices=False):  # pylint: disable=protected-access
File "/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 3540, in _add_new_tf_operations
for c_op in c_api_util.new_tf_operations(self)
File "/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 3428, in _create_op_from_tf_operation
ret = Operation(c_op, self)
File "/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1718, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
FailedPreconditionError (see above for traceback): Error while reading resource variable module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel from Container: localhost. This could mean that the variable was uninitialized. Not found: Container localhost does not exist. (Could not find resource: localhost/module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel)
[[Node: module_apply_default/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel/Read/ReadVariableOp = ReadVariableOpdtype=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:CPU:0"]]
[[Node: module_apply_default/bilm/add_1/ReadVariableOp/_177 = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device_incarnation=1, tensor_name="edge_562_module_apply_default/bilm/add_1/ReadVariableOp", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"]]
I tried on 3 ubuntu machines, cuda is 9.0 and cudnn 7.0. The issue is consistent.
Any help is highly appreciated! Thank you very much!

Please try using https://tfhub.dev/google/elmo/2 (the version 2 of the
module) this version https://www.tensorflow.org/hub/modules/google/elmo/2
fixes the issue around making which variables are marked as trainable for
trainable=True mode that you are using.
On Fri, Jun 22, 2018 at 8:32 PM linnan wang<EMAIL_ADDRESS>wrote:

Hello there,
Thank you for making TensorFlow Hub, it is an awesome tool to simplify the
process. Recently I tied to use ELMO in TensorFlow Hub. However, I did the
procedures online:
elmo = hub.Module("https://tfhub.dev/google/elmo/1", trainable=True)
This gives me
INFO:tensorflow:Using /tmp/tfhub_modules to cache modules.
INFO:tensorflow:Initialize variable module/aggregation/scaling:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with aggregation/scaling
INFO:tensorflow:Initialize variable module/aggregation/weights:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with aggregation/weights
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_0:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/W_cnn_0
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_1:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/W_cnn_1
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_2:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/W_cnn_2
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_3:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/W_cnn_3
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_4:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/W_cnn_4
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_5:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/W_cnn_5
INFO:tensorflow:Initialize variable module/bilm/CNN/W_cnn_6:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/W_cnn_6
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_0:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/b_cnn_0
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_1:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/b_cnn_1
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_2:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/b_cnn_2
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_3:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/b_cnn_3
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_4:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/b_cnn_4
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_5:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/b_cnn_5
INFO:tensorflow:Initialize variable module/bilm/CNN/b_cnn_6:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN/b_cnn_6
INFO:tensorflow:Initialize variable module/bilm/CNN_high_0/W_carry:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN_high_0/W_carry
INFO:tensorflow:Initialize variable module/bilm/CNN_high_0/W_transform:0
from checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN_high_0/W_transform
INFO:tensorflow:Initialize variable module/bilm/CNN_high_0/b_carry:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN_high_0/b_carry
INFO:tensorflow:Initialize variable module/bilm/CNN_high_0/b_transform:0
from checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN_high_0/b_transform
INFO:tensorflow:Initialize variable module/bilm/CNN_high_1/W_carry:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN_high_1/W_carry
INFO:tensorflow:Initialize variable module/bilm/CNN_high_1/W_transform:0
from checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN_high_1/W_transform
INFO:tensorflow:Initialize variable module/bilm/CNN_high_1/b_carry:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN_high_1/b_carry
INFO:tensorflow:Initialize variable module/bilm/CNN_high_1/b_transform:0
from checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN_high_1/b_transform
INFO:tensorflow:Initialize variable module/bilm/CNN_proj/W_proj:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN_proj/W_proj
INFO:tensorflow:Initialize variable module/bilm/CNN_proj/b_proj:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/CNN_proj/b_proj
INFO:tensorflow:Initialize variable
module/bilm/RNN_0/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/bias:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/RNN_0/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/bias
INFO:tensorflow:Initialize variable
module/bilm/RNN_0/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/kernel:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/RNN_0/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/kernel
INFO:tensorflow:Initialize variable
module/bilm/RNN_0/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/projection/kernel:0
from checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/RNN_0/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/projection/kernel
INFO:tensorflow:Initialize variable
module/bilm/RNN_0/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/bias:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/RNN_0/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/bias
INFO:tensorflow:Initialize variable
module/bilm/RNN_0/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/RNN_0/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel
INFO:tensorflow:Initialize variable
module/bilm/RNN_0/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/projection/kernel:0
from checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/RNN_0/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/projection/kernel
INFO:tensorflow:Initialize variable
module/bilm/RNN_1/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/bias:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/RNN_1/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/bias
INFO:tensorflow:Initialize variable
module/bilm/RNN_1/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/kernel:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/RNN_1/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/kernel
INFO:tensorflow:Initialize variable
module/bilm/RNN_1/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/projection/kernel:0
from checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/RNN_1/RNN/MultiRNNCell/Cell0/rnn/lstm_cell/projection/kernel
INFO:tensorflow:Initialize variable
module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/bias:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/bias
INFO:tensorflow:Initialize variable
module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel
INFO:tensorflow:Initialize variable
module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/projection/kernel:0
from checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/projection/kernel
INFO:tensorflow:Initialize variable module/bilm/char_embed:0 from
checkpoint
/tmp/tfhub_modules/0eadd30f548cbfd3065119697ded6bb381668285/variables/variables
with bilm/char_embed
Then I tried this
embeddings = elmo( ["the cat is on the mat", "dogs are in the fog"],
signature="default", as_dict=True)["elmo"] sess = tf.Session()
sess.run(elmo)
Then I got this error:
tensorflow.python.framework.errors_impl.FailedPreconditionError: Error
while reading resource variable
module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel from
Container: localhost. This could mean that the variable was uninitialized.
Not found: Container localhost does not exist. (Could not find resource:
localhost/module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel)
[[Node:
module_apply_default/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel/Read/ReadVariableOp
= ReadVariableOpdtype=DT_FLOAT,
_device="/job:localhost/replica:0/task:0/device:CPU:0"
http://module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel]]
[[Node: module_apply_default/bilm/add_1/ReadVariableOp/_177 = _Recvclient_terminated=false,
recv_device="/job:localhost/replica:0/task:0/device:GPU:0",
send_device="/job:localhost/replica:0/task:0/device:CPU:0",
send_device_incarnation=1,
tensor_name="edge_562_module_apply_default/bilm/add_1/ReadVariableOp",
tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"
]]
Caused by op
u'module_apply_default/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel/Read/ReadVariableOp',
defined at:
File "", line 4, in
File
"/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow_hub/module.py",
line 203, in call
name=name)
File
"/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow_hub/native_module.py",
line 447, in create_apply_graph
restore_collections_predicate=(lambda key: key in import_collections))
File
"/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/training/saver.py",
line 1955, in import_meta_graph
**kwargs)
File
"/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/framework/meta_graph.py",
line 743, in import_scoped_meta_graph
producer_op_list=producer_op_list)
File
"/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/util/deprecation.py",
line 432, in new_func
return func(*args, **kwargs)
File
"/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/framework/importer.py",
line 513, in import_graph_def
_ProcessNewOps(graph)
File
"/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/framework/importer.py",
line 303, in _ProcessNewOps
for new_op in graph._add_new_tf_operations(compute_devices=False): #
pylint: disable=protected-access
File
"/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py",
line 3540, in _add_new_tf_operations
for c_op in c_api_util.new_tf_operations(self)
File
"/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py",
line 3428, in _create_op_from_tf_operation
ret = Operation(c_op, self)
File
"/data/lwang53/tensorflow-p2/local/lib/python2.7/site-packages/tensorflow/python/framework/ops.py",
line 1718, in init
self._traceback = self._graph._extract_stack() # pylint:
disable=protected-access
FailedPreconditionError (see above for traceback): Error while reading
resource variable
module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel from
Container: localhost. This could mean that the variable was uninitialized.
Not found: Container localhost does not exist. (Could not find resource:
localhost/module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel)
[[Node:
module_apply_default/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel/Read/ReadVariableOp
= ReadVariableOpdtype=DT_FLOAT,
_device="/job:localhost/replica:0/task:0/device:CPU:0"
http://module/bilm/RNN_1/RNN/MultiRNNCell/Cell1/rnn/lstm_cell/kernel]]
[[Node: module_apply_default/bilm/add_1/ReadVariableOp/_177 = _Recvclient_terminated=false,
recv_device="/job:localhost/replica:0/task:0/device:GPU:0",
send_device="/job:localhost/replica:0/task:0/device:CPU:0",
send_device_incarnation=1,
tensor_name="edge_562_module_apply_default/bilm/add_1/ReadVariableOp",
tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:GPU:0"
]]
Any help is highly appreciated! Thank you very much!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/tensorflow/hub/issues/88, or mute the thread
https://github.com/notifications/unsubscribe-auth/AbunTJIm14FC4phJCVsVfgrYjmzikPHUks5t_TgkgaJpZM4U0Pie
.

thank you, just figured it out.


Improve code quality

Codecov Report

Merging #310 (4806f50) into master (996775c) will increase coverage by 0.20%.
The diff coverage is 82.05%.

@@             Coverage Diff              @@
##             master     #310      +/-   ##
============================================
+ Coverage     74.21%   74.41%   +0.20%
- Complexity      771      772       +1
============================================
  Files           111      111
  Lines          2474     2486      +12
  Branches        314      309       -5
============================================
+ Hits           1836     1850      +14
- Misses          517      519       +2
+ Partials        121      117       -4

Impacted Files
Coverage Δ

.../java/seedu/address/logic/commands/AddCommand.java
100.00% <ø> (ø)

...va/seedu/address/logic/commands/DeleteCommand.java
100.00% <ø> (ø)

...eedu/address/logic/commands/DeleteTaskCommand.java
100.00% <ø> (ø)

...java/seedu/address/logic/commands/EditCommand.java
86.25% <ø> (ø)

.../address/logic/commands/GenerateEmailsCommand.java
77.77% <ø> (ø)

...java/seedu/address/logic/commands/HelpCommand.java
100.00% <ø> (ø)

.../seedu/address/logic/commands/ListTaskCommand.java
75.00% <ø> (ø)

...edu/address/logic/parser/AddTaskCommandParser.java
85.45% <ø> (ø)

...ress/logic/parser/GenerateEmailsCommandParser.java
100.00% <ø> (ø)

src/main/java/seedu/address/model/Model.java
75.00% <ø> (ø)

... and 20 more

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 996775c...4806f50. Read the comment docs.


luci - show raw counters / human-readable counters button not displaying correctly (bootstrap)

Running latest release,  r16219 with luci-ssl and bootstrap theme (I only have bootstrap, so can't verify other themes). Noticing under Status -> Firewall, the show raw counters / human-readable counters button seems to not display correctly.
Verified on Chrome and Firefox:
Chrome:

FireFox:

Just did a test compile with luci-theme-openwrt-2020, and for me the Status -> Firewall page just doesn't show anything. Tested also on FF and Edge Chromium.


Use provided CAs for oidc client connections

Issue: https://github.com/rancher/rancher/issues/43217

Problem

When connecting Rancher to an auth provider such as keycloak, there were a few issues.

The oidc client does not use certificates provided via tls-ca or tls-ca-additional secrets.
The client certificate provided in the UI must also contain the specific CA that signed the server cert for the auth provider.

To verify this, you can

Create a CA and an Intermediate CA
Sign a server cert with the Intermediate CA
Sign a client cert with either CA
Use that server cert with something like Keycloak
Configure a X509 browser flow with Keycloak for PKI auth
Install Rancher using privateCA: true
Connect Rancher with Keycloak using the client cert and key

In this scenario, connecting Rancher to Keycloak for auth delegation will fail unless you also provide the Intermediate CA as well as the client certificate in the "Certificate" field of the UI. Note that it has to be the intermediate CA and (in my case) does not with with the full PEM chain.
Solution

Use the x509.SystemCertPool() to pick up on the existing truststore within the pod, and only use the client certificate in the http client config. Also use the http.DefaultTransport to pick up on things like http.ProxyFromEnvironment to make sure the client honors settings from proxy: ... and noProxy: ...
Testing
Built and confirmed against a Keycloak instance with a multi CA chain.
Engineering Testing
Manual Testing

Automated Testing

Test types added/modified:

Unit
Integration (Go Framework)
Integration (v2prov Framework)
Validation (Go Framework)
Other - Explain: EXPLAIN
None
REMOVE NOT APPLICABLE BULLET POINTS ABOVE

If "None" - Reason: EXPLAIN THE REASON

If "None" - GH Issue/PR: LINK TO GH ISSUE/PR TO ADD TESTS

Summary: TODO
QA Testing Considerations

Regressions Considerations

TODO
Existing / newly added automated tests that provide evidence there are no regressions:

TODO

Anything I can do to help move this along? This bug is kind of a big blocker for us and preventing us from using Rancher across our environments where the CA chains can have dozens of CAs.

I will ask the engineers again to take a look.

@ethanchowell Would you mind rebasing your PR to the latest release/v2.8 branch? Otherwise I don't believe the CI will pass.

Merged, thanks again @ethanchowell for your contribution!


Children and Member tabs show up regardless of edit permission

In fixing https://github.com/jhu-idc/idc-isle-dc/issues/35, it sort of creates another issue, because of https://github.com/jhu-idc/idc-isle-dc/issues/107.
The "Children" and "Media" tabs each have a button ("Add child", "Add media") to add new nodes or media.  The buttons show up regardless of whether or not a user has permission to add Media or Children to that collection. The logic for these tabs is in Islandora module and we may want to override how they work, so they don't show up if the user doesn't have edit access to the parent object.
Because of https://github.com/jhu-idc/idc-isle-dc/issues/107, it makes it easier for a user who shouldn't be able to to add children to a collection.  (The access terms field works just fine, btw, as it filters out options the user doesn't have access to.  In other words, the user cannot give the new node any access terms they do not have access to).

Note that this is due to the logic for the tabs being in the Islandora module and the logic for access controls being in Workbench Access.  It's just that they are disjoint and can be reconciled.

This is admin-facing, may be candidate to defer to after launch, or not fix at all

Per Aaron's comment, we wont fix this for now.


Only wrap jobs that inherit ActiveJob::Base

Hi,
We have a lot of jobs in our resque schedule that are not ActiveJob::Base descendants, just plain old resque jobs pre-dating active job.
We want to support activejobs moving forward in our resque schedule, but are not interested to update the existing jobs.
This changes active_scheduler to not assume every job in the schedule is an active job, at the marginal expense of looking up class objects (.constantize) when wrapping the config, which is fine for us.

Coverage remained the same at 100.0% when pulling e56ae9008724dd957870808eac89cc84f3f84114 on ximus:only-wrap-active-jobs into 0d6993a395248898eee7366c3a2fd3d13429de75 on JustinAiken:master.

Cool, useful idea for people transition!
Released as 0.1.0


Click the ‘here’ link in the readme to directly fork this repository.

Issue Title

Click the ‘here’ link in the readme to directly fork this repository.
Type

[ ] Big
[ ] Feature Request
[ ] Info
[ ] Bug
[x] Documentation
[ ] Other (please specify):

Description
When I click on the ‘here’ link, it redirects me to this repository. Perhaps it would be better to navigate to the forked page instead.
Screenshots / Code Snippets (if applicable)

Your PR is merged @Nauxscript Thanks for the subtle change. 😉


new theme

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[ ] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

To do list:

[ ] add the before after with to the code element. (see master layout.css)
[ ] change the font of the guides to Lato to increase readability for larger articles.
[ ] make sure the steel blue shades are equally away from one another
[ ] align the search page with the theme( change background and font)
[ ] adjust the search box size
[ ] Made the display of the challenges change text color and links and etc according to theme(builders.js)
[ ] change the placeholder text of the search to "search more than 10,000 lessons"
[ ] adjust the navigation bar and center logo in all times
[ ] adjust nav bar in mobile
[ ] make all gray svg same color and find a color that has decent contrast in dark and light theme
[ ] adjust the d3 svg in landing page to confirm to theme color
[ ] change the companies logos from the landing page to an svg that could change color based on theme
[ ] adjust the accepted privacy policy button and convey intention better
[ ] search for instances of color and see if they could be replaced with color variables
[ ] visit pages modals individually to check for inconsistencies.

Bump.


Add Mintel-specific changes

Update readme and license
Update Alpine image
Add build hook and vcs args
Add changelog

Sorry, wrong repo!

Sorry, wrong repo!


Signed CLA but the Status Is Not Updated Even After the Recheck

URL to the linked Repository or Organization
https://github.com/gardener/terraformer/
Steps to reproduce the problem
I've signed the CLA, but in the PR, it still shows the CLA is not signed. I tried retriggering the check with the URL
https://cla-assistant.io/check/gardener/terraformer/?pullRequest=114, but no help.
What is the expected result?
The status gets updated.
What happens instead?
The CLA status is not updated
Any other information? (attach screenshot if possible)

Hi @xiaofenhappy, thanks for opening the issue.
The problem is actually that in your PR the PR author and commit author are two different GitHub users (PR: @xiaofenhappy, commit: @molecule-z) and for CLA-Assistant to mark a PR as signed both need to accept the CLA.
The commit author is determined based on the used email address (you can see it in the raw patch) and this email address is not associated with your GitHub account and thus needs a separate signature.
To solve this there are two ways:

You can adjust the commit email address and force push the branch to update the author to your own email address. (Details on how to fix this at why-are-my-commits-linked-to-the-wrong-user#commits-are-linked-to-another-user)
- or -
Login to @molecule-z and sign the CLA with that account as well.

@Shegox Yep, that's the reason.
It works, thanks.


@azure/cosmos, client.getDatabaseAccount statusCode undefined

When I use client.getDatabaseAccount, and the statusCode is undefined, how should I interpret that? The operation partially failed, or. something else?

@dfberry can you help me with the code snippet which you used? It can help me in reproducing this problem.

Hey @dfberry ,At the moment, we are not returning any status code, resulting in an undefined outcome for all scenarios. However, we plan to address this issue in our upcoming releases.


[BUG] Unable to ignore the top-level/root directory when dir_names="true"

Is there an existing issue for this?

[X] I have searched the existing issues

Does this issue exist in the latest version?

[X] I'm using the latest release

Describe the bug?
When attempting to detect unique directories which have changed from a commit, the Github Action results in the following error if the top-level directory (represented by .) is specified as a directory to ignore using the ignore_files input property:
/usr/bin/git rev-parse --show-toplevel
/home/runner/work/containers/containers
/usr/bin/git diff --diff-filter=D --name-only ccecf3fe3c741e028264707bd98319b7065bd91f..09a3feaed99eae19565cbc92d72fdc363ddb9ebd
Error: No paths found using the specified patterns

To Reproduce

Set the following step in a workflow:

      - name: Identify changed application subdirectories
        id: changed-dirs
        uses: tj-actions/changed-files@v35
        with:
          dir_names: "true"
          dir_names_max_depth: "1"
          files_ignore: .github,.
          files_ignore_separator: ","

Modify a file at the top-level/root of the repository and trigger the following workflow.
Observe that the workflow fails at the above step.

What OS are you seeing the problem on?
ubuntu-latest or ubuntu-20.04
Expected behavior?
The expected behavior is that the . current directory entry is ignored and not returned in the action's output when set to ignore.
Relevant log output
Run tj-actions/changed-files@v35
  with:
    dir_names: true
    dir_names_max_depth: 1
    files_ignore: .github,.
    files_ignore_separator: ,
    separator:
    include_all_old_new_renamed_files: false
    old_new_separator: ,
    old_new_files_separator:
    files_separator:

    path: .
    quotepath: true
    json: false
    json_raw_format: false
    fetch_depth: 50
    since_last_remote_commit: false
    write_output_files: false
    output_dir: .github/outputs
    match_directories: true
Run # "Calculating the previous and current SHA..."
  # "Calculating the previous and current SHA..."
  bash $GITHUB_ACTION_PATH/diff-sha.sh
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
  env:
    GITHUB_SERVER_URL: https://github.com
    GITHUB_REPOSITORY: 0x0I/containers
    GITHUB_REF: refs/pull/7/merge
    GITHUB_SHA: 09a3feaed99eae19565cbc92d72fdc363ddb9ebd
    GITHUB_WORKSPACE: /home/runner/work/containers/containers
    GITHUB_EVENT_HEAD_REPO_FORK: false
    GITHUB_EVENT_PULL_REQUEST_NUMBER: 7
    GITHUB_EVENT_PULL_REQUEST_BASE_REF: main
    GITHUB_EVENT_PULL_REQUEST_HEAD_REF: test_ignore_toplevel_dir
    GITHUB_EVENT_PULL_REQUEST_BASE_SHA: ccecf3fe3c741e028264707bd98319b7065bd91f
    GITHUB_EVENT_PULL_REQUEST_HEAD_SHA: 287b5789a3d0f0e6beb475f8c38b8671ab4563ba
    GITHUB_EVENT_PULL_REQUEST_COMMITS: 1
    GITHUB_EVENT_BEFORE:
    GITHUB_EVENT_FORCED:
    GITHUB_REFNAME: 7/merge
    INPUT_SHA:
    INPUT_BASE_SHA:
    INPUT_SINCE:
    INPUT_UNTIL:
    INPUT_PATH: .
    INPUT_FETCH_DEPTH: 50
    INPUT_SINCE_LAST_REMOTE_COMMIT: false
changed-files-diff-sha
Run tj-actions/glob@v16
/usr/bin/git rev-parse --show-toplevel
/home/runner/work/containers/containers
/usr/bin/git diff --diff-filter=D --name-only ccecf3fe3c741e028264707bd98319b7065bd91f..09a3feaed99eae19565cbc92d72fdc363ddb9ebd
Error: No paths found using the specified patterns

Anything else?
The current use-case is if there is a monorepo of sorts with which I only want my CI workflows to execute against subdirs which share a similar structure and represent various related apps/services. For example, each subdir has a Dockerfile (corresponding to the app its meant to encompass) whereas the top-level root dir won't. If i modify a file at the top level (e.g. the repo's main README), then my logic will fail without ignoring the root since the root won't and isn't really expected to be tested against or contain this expected structure.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

@0x0I I believe this didn't work because the files_ignore input requires each pattern to be separated by a newline character
Example
      - name: Identify changed application subdirectories
        id: changed-dirs
        uses: tj-actions/changed-files@v35
        with:
          dir_names: "true"
          dir_names_max_depth: "1"
          files_ignore: |
              .github
              .
          files_ignore_separator: ","

But this setup wouldn't solve the problem you highlighted since the glob patterns are parsed by a separate action which returns absolute paths and the dir name output of . is generated after matching the patterns to the changed files.
I've added a PR that would work for you use case i.e You'll now have a new input that should exclude . entirely from the dir name output.

@0x0I I believe this didn't work because the files_ignore input requires each pattern to be separated by a newline character
Example
      - name: Identify changed application subdirectories
        id: changed-dirs
        uses: tj-actions/changed-files@v35
        with:
          dir_names: "true"
          dir_names_max_depth: "1"
          files_ignore: |
              .github
              .
          files_ignore_separator: ","

But this setup wouldn't solve the problem you highlighted since the glob patterns are parsed by a separate action which returns absolute paths and the dir name output of . is generated after matching the patterns to the changed files.
I've added a PR that would work for you use case i.e You'll now have a new input that should exclude . entirely from the dir name output.

Hey @jackton1, ah I thought that the files_ignore_separator would change the default of newline to the character that is specified. In the case mentioned, this would be ,.
Anyway, that's awesome - thanks! Will check it out.

Oh you are correct I must have missed that in your earlier post files_ignore_separator would solve that.

@0x0I This should now be available in the latest release.
Example
      - name: Identify changed application subdirectories
        id: changed-dirs
        uses: tj-actions/changed-files@v35
        with:
          dir_names: "true"
          dir_names_max_depth: "1"
          files_ignore: ".github"
          dir_names_exclude_root: "true"


Controlling nginx and other Modules

This issue tracker is only for issues related to EasyEngine. Please use http://community.rtcamp.com/c/easyengine for support questions.
If you feel the issue is an EasyEngine core specific issue, please attach the output of the following commands.
System Information
- [ ] lsb_release -a
Distributor ID: Ubuntu
Description:    Ubuntu 16.04.5 LTS
Release:        16.04
Codename:       xenial

- [ ] ee cli version
EE 4.0.0-beta.6

- [ ] ee cli info
+-------------------+------------------------------------------------------------------------------------+
| OS                | Linux 4.15.0-29-generic #31~16.04.1-Ubuntu SMP Wed Jul 18 08:54:04 UTC 2018 x86_64 |
| Shell             | /bin/bash                                                                          |
| PHP binary        | /usr/bin/php7.2                                                                    |
| PHP version       | 7.2.9-1+ubuntu16.04.1+deb.sury.org+1                                               |
| php.ini used      | /etc/php/7.2/cli/php.ini                                                           |
| EE root dir       | phar://ee.phar                                                                     |
| EE vendor dir     | phar://ee.phar/vendor                                                              |
| EE phar path      | /root                                                                              |
| EE packages dir   |                                                                                |
| EE global config  | /opt/easyengine/config.yml                                                         |
| EE project config |                                                                                |
| EE version        | 4.0.0-beta.6                                                                       |
+-------------------+------------------------------------------------------------------------------------+

Unlike the stable version, I see the 4.0.x version is completely re-written, can you tell me how do we control the PHP version, generating MySQL database config or a database for each site and putting nginx with it? I see no stack option in it.
Also, when I added a site "ee site create mysite.com --type=php" what I'm getting is a 503 error.

@sagargulati Sorry at the time you asked the question, we didn't have much documentation around v4. Now we have documented much of it. Have a look at our release blog post and have a look at the documentation section linked there. I think it would answer most of your questions.
I'm closing this issue for now. In case you still have more questions, you can ask them in our community forum. We use GitHub issue tracker only for tracking issues or feature request of EasyEngine. General questions like this are better suited in the community forum.


Java API review becomes invalid if someone uploads non sources.jar

User uploads non sources.jar to create a revision and language processor process this file. But Java parser returns incorrect revision details and this causes review to be not accessible.
Java language parser should skip processing package if that file is not supported instead of making the whole review invalid. We have got many requests recently to fix review after partner team uploads incorrect jar.

@JonathanGiles I'll take this one


WIP moving mysql types.rs file to shared area

cc @sgrif @weiznich
#1882

@weiznich
Is this PR still worth working on? If so, I'm going to take over this.

@weiznich
Is this PR still worth working on? If so, I'm going to take over this.

I'm not currently working on this, no. You can take over if you'd like. I ran into some hangups trying to make diesel/src/mysql/types/mod.rs be generic over both backends to avoid duplication, but someone with a bit more rust experience might know the correct approach.

@TaKO8Ki In principle this is something worth working on, but this is a quite large change and therefore quite a lot of work. Ideally the implementation of a mariadb backend would be done by someone that is interested in maintaining that backend.

@weiznich Ok. I understand.


Adding changelog and changelog verifier

Signed-off-by: Harsha Vamsi Kalluri<EMAIL_ADDRESS>Description

Introduces changelog and changelog verifier
Updates text around admins and maintainers

Issues Resolved
List any issues this PR will resolve, e.g. Closes [...].
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #89 (4d75097) into main (18d50b4) will decrease coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
- Coverage   42.09%   41.98%   -0.12%
==========================================
  Files          49       49
  Lines       28666    28666
==========================================
- Hits        12067    12035      -32
- Misses      16599    16631      +32

Flag
Coverage Δ

integration
41.98% <ø> (-0.12%)
:arrow_down:

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

Impacted Files
Coverage Δ

opensearch/src/http/transport.rs
42.35% <0.00%> (-7.46%)
:arrow_down:

yaml_test_runner/tests/common/client.rs
87.75% <0.00%> (-6.13%)
:arrow_down:

yaml_test_runner/src/skip.rs
92.50% <0.00%> (-2.50%)
:arrow_down:

opensearch/src/http/response.rs
16.18% <0.00%> (-2.32%)
:arrow_down:

opensearch/src/client.rs
29.40% <0.00%> (-0.25%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[3.3] Update hessian-lite to 4.0.0 & Set hessian serialization back

What is the purpose of the change
Brief changelog
Verifying this change

Checklist

[x] Make sure there is a GitHub_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GitHub issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Check if is necessary to patch to Dubbo 3 if you are work on Dubbo 2.7
[ ] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[ ] Add some description to dubbo-website project if you are requesting to add a feature.
[ ] GitHub Actions works fine on your own branch.
[ ] If this contribution is large, please follow the Software Donation Guide.

@CrazyHZM Please help to take a look at the native part.


Issue with Url-analysis before file writing

Hi, Leon. I have an issue with converting post urls in static files when the permalinks are setted without slash at the end of post/page slug. In src/Crawler I found possible cause of this bug: https://github.com/leonstafford/wp2static/blob/develop/src/Crawler.php#L150 i.e without end-slash Crawler's code recognizes such post link as file without extension. In old version(6.x) of plugin this logic contains php function pathinfo() which performs this task more accurately.

Thanks for reporting @ekvasnyi! Sorry for my delayed response.
So, you're after something more like https://github.com/leonstafford/static-html-output/blob/e02b9b39afb9e3bb03cce7a7d61f5ef900b3c42d/src/SiteCrawler.php#L458 ?
I'm still not sure if I'd try to add support for that permalink structure, as I could see it adding a bit more complexity to the plugin in various places. I have a feeling you're correct though...
Could you please try to convince me more and add more information, such as example input HTML and how it looks when processed, so I can have a better idea of the issue?


Feature requeset: MySQL like multicraft

The title says, the mysql system like multicraft where you can create a sql database for your server things.

please dont put your suggestions in the issues section. if you have a feature idea please post it in the appropriate location on the puffer-panel forum. i suggest posting it under general discussion.
forum link:
https://community.pufferpanel.com/category/2/general-discussion


Sampler reduction modes

Would be great to support samplers that compute min/max instead of the weighted average.
See also - https://github.com/gpuweb/gpuweb/pull/1283

This doesn't need to be in hal-0.7, postponing.

This doesn't need to be in hal-0.7, postponing.


How to install kernel in VS code

Environment data

VS Code version: XXX
Jupyter Extension version (available under the Extensions sidebar): XXX
Python Extension version (available under the Extensions sidebar): XXX
OS (Windows | Mac | Linux distro) and version: XXX
Python and/or Anaconda version: XXX
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): XXX
Jupyter server running: Local | Remote | N/A

Expected behaviour
XXX
Actual behaviour
XXX
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

XXX

Logs

Output for Jupyter in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Jupyter)

XXX

If you are working on Python , then you do not need to install a Python kernel all you need is

Install Jupyter extension
Install Python extension
Install a Python runtime (preferably conda or the like)
Go into the kernel picker and select a Python environment
You will be prompted to install some dependencies, click ok
You can now run code against these Python environments

If you are working on Non-Python code

Install the kernel based on the documentation of the corresponding kernel you are interested in, see here for more information https://docs.jupyter.org/en/latest/install/kernels.html. See docs for installing Julia, R kernels
Install the Jupyter extension
Go into the kernle picker and select the corresponding kernel, such as Julia or R or other
Run the code against this kernel

Hope this was helpful.
The latest version of VS Code insiders and pre-release version of teh Jupyter extension has an improved kernel picker with more links to documentation explaining the above.

@AftabHaider680 as a suggestion questions such as these are perfect candidates for the Discussions sections, as those would help others with similar questions as well.


Update styles.css

Just a small fix on style of the Footer select element to be inligned with the rest of the footer.

And by the way in the home page when you change the language to 'de' and the page refreshes, the select element is still showing by default 'en'.
To go back to to 'en' you have to click 'de' (that makes no change) and then again on 'en'.

We switched to styled-components, so the styles.css files no longer exist! Would you mind resubmitting this change against an up-to-date dev branch? Thanks!
Also, for that second issue we have #1181

@mxstbr , any critical reason to change style.css to styled-components ?


Add favicon.(ico|png|svg) as fallback for site's avatar on fediverse

What
I learned that ActivityPub (AP) plugin uses site icon to retrieve and show user avatar on fediverse. As discussed with @pfefferle on Mastodon, I'd be happy to have a fallback to standard favicon, since I don't use site icon in my WordPress instance and, for this reason, right now its fediverse profile has a WP.com icon as avatar (@feed@manualdousuario.net).
Why
I can't tell if WP site icon is widely used to justify this decision from AP plugin, but I can tell that favicon in *.png and specially in *.svg has way better quality and smaller footprint.
Here's an evidence/example (mine, in the red squircle, is a small *.svg file with transparency):

How
No response

Would be interesting if Mastodon supports .svg or .ico as Avatars?!?

Good catch, @pfefferle! Mastodon only supports *.png, *.gif and *.jpg for avatars.
In this case, an alternative would be fallback to apple-touch-icon, which is mostly (always?) a *.png image.

But how do I get these? I think parsing the HTML code is not really efficient, so I either need to know the plugin you use or you should use the site_icon feature ;)

It's a standard link rel= on header. For example:
<link rel="apple-touch-icon" href="/apple-touch-icon.png?v=4">

I resist using site_icon because it generates a lot of redundant images. Right now, my favicon structure is this:
<link rel="icon" href="/favicon.ico?v=4" sizes="any"> <!-- Pretty basic, a general fallback for older browsers -->
<link rel="icon" href="/favicon.svg?v=3"> <!-- Main favicon, high quality and small footprint -->
<link rel="apple-touch-icon" href="/apple-touch-icon.png?v=4"> <!-- For Apple devices. There are other uses I'm not aware of -->
<link rel="icon" href="/favicon-192x192.png" sizes="192x192"/> <!-- For Android devices -->

That is not the question. The question is what generates the codes or have you added it by hand (directly in your Themes HTML)?

Oh, sorry about that. I added by hand.

Then it is not possible to do it automatically, because I think parsing the HTML is not a performant thing.
So in this case you have to use some custom code and a filter to change the icon.

I understand. Can you or someone help me with custom code, @pfefferle? Maybe we could add it to AP's documentation. Although I recognize this is an edge case, maybe there are more weird people like me out there 😄🥲

For the Blog User it is something like:
add_filter( 'activitypub_activity_blog_user_object_array', function ( $array ) {
    $array['icon']['url'] = '{absolute path to your image}';
} );

for an author it is
add_filter( 'activitypub_activity_user_object_array', function ( $array ) {
    $array['icon']['url'] = '{absolute path to your image}';
} );

Awesome! I'll try right now. Thanks, @pfefferle

I tried the first snippet and, so far, it didn't work. Maybe Mastodon has an aggressive cache? It's been 3 hours. I'll wait until tomorrow (~24h) and if nothing changes, I'll let you know.

The code seem to have broken something... your profile page is not accessible atm. 🫣

Oops. Weird, I can see it on mastodon.social, including posts published after I inserted the code.
Just in case, I just removed the code from functions.php.

now it works again! the infos on mastodon are cached, so they will last for quite some time even if your site might not be accessible any more. can you share the exact code you used, so that I can have a look?

Sure! Here it is:
add_filter( 'activitypub_activity_blog_user_object_array', function ( $array ) {
    $array['icon']['url'] = 'https://manualdousuario.net/apple-touch-icon.png';
} );

oops, forgot to return the array!
add_filter( 'activitypub_activity_blog_user_object_array', function ( $array ) {
    $array['icon']['url'] = 'https://manualdousuario.net/apple-touch-icon.png';

    return $array;
} );

Code added. Can you reach my instance?

yup, works 👍
to get the cache cleared, it helps when you publish some posts :)

I will close this for now, feel free to still comment or re-open.

It worked, @pfefferle!

Thank you! 😊


once is not an option for removeEventListener

https://github.com/WebReflection/dom4/blob/1fec339bec701430287fc41a34d480998e6b4a47/src/event-target.js#L133
Is there a specific reason once was added to rEL for the feature test? The docs on MDN don't list this as a valid option.
I would expect the line to be rewritten as rEL('_', increment);, since there might be other browsers that will actually throw an error at this point, even though aEL('_', increment, { once: true }); functions properly.

no browser will throw an error ever, they eventually ignore the third argument. on the other side, if you added/removed a listener via true or false, how do you know which of the two you should remove?
here I am just using a symmetric signature, and it worked well from IE8+, and this module is old, changing this might, instead, cause issues, while keeping it won't throw ever (because new browsers don't get patched while old browsers never threw a thing).

no browser will throw an error ever, they eventually ignore the third argument. on the other side, if you added/removed a listener via true or false, how do you know which of the two you should remove?
here I am just using a symmetric signature, and it worked well from IE8+, and this module is old, changing this might, instead, cause issues, while keeping it won't throw ever (because new browsers don't get patched while old browsers never threw a thing).


Support running Conftest before running terraform plan

Feature Overview
SSIA.
Why is the feature needed?
Currently, tfaction supports testing plan files by Conftest, but some tests should be run before running terraform plan.
Example Code
No response
Note

https://zenn.dev/shunsuke_suzuki/scraps/53d7f5e954f90d
https://engineering.mercari.com/en/blog/entry/20230706-bucket-full-of-secrets-terraform-exfiltration/

tfaction-root.yaml:
conftest:
  hcl_policy: policy/hcl

We want to support more flexible syntax.
Users may want to run Conftest on other files too.
conftest:
  - policy: policy/state
    state: true
  - policy: policy/hcl
    tf: true
  - policy: policy/hcl-combine
    combine: true
    patterns:
      - "*.tf"
      - "*.tf.json"
  - policy: policy/yaml
    patterns:
      - "*.yaml"
      - "*.yml"

tfaction-root.yaml
conftest:
  policies:
    - policy: policy/state
      state: true
    - policy: policy/combine/state
      state: true
      combine: true
      data: data/data.yaml
    - policy: policy/hcl
      tf: true
    - policy: policy/combine/hcl
      tf: true
      combine: true
    - policy: policy/yaml
      patterns:
        - "*.yaml"
        - "*.yml"
        - "!main.yaml"

tfaction.yaml
conftest:
  disable-all: true
  policies:
    - policy: policy/state
      state: true

https://github.com/suzuki-shunsuke/tfaction/pull/1893


[Data Sync] Adapt engine to run on Jenkins

Adapt the engine created so it can run on a Jenkins server.

This issue is blocked due to a couple of reasons:

We still do not have proper access to Jenkins environment;
The testing database is not yet properly set (final tables created) and there is no data available to test the sync process;
We just found out that other teams that were supposedly running their automated data extraction on Jenkins are actually running the process on an Emerald cluster on OpenShift using Docker;
It seems to be a better approach to run it on OpenShift instead of Jenkins, further investigation needs to be addressed.

Some updates so far:

Already got access to Jenkins Cron (link at: https://apps.nrs.gov.bc.ca/int/cron/me/my-views/view/all/job/SPAR/)

Jenkins Cron is under Windows OS, so, all tests done in the Unix-like environments must be "translated" to be executed in Windows (i.e. "bat scripts instead sh/bash")

To start a spike in a OpenShift we will need access to use a dev/stage environment to set up a plan to deploy a docker containerized data sync engine.

This issue is blocked by:

This error happens when we tried to deploy a Data Sync engine instance running in Jenkins Cron server.
We would need some help from the Jenkins Admins to allow github BCGOV repository to be accessed by git scm on Jenkins cron.

Issue with proxy access solved using the following parameters in the Jenkins Config:

Documentation can be found at: https://apps.nrs.gov.bc.ca/int/confluence/display/FSADT2/Jenkins+configuration

PR opened at: https://github.com/bcgov/nr-spar-data-sync/pull/18


Re-add line that has been removed accidentially

Sorry about this :/ Should have tried it first...


How many connections will be created if there are 10M peers?

Will each peer connect to all the 10M peers?
If there is a limit of max connection for each peer, how to handle so many peers?

No, each peer would most likely not be able to make so many connections. The limit most likely would be set by each individual OS.


Same id in @Results annotation in methods selectOne and selectMany

MyBatis version
not for sure, I'm reporting a bug of MyBatis generator.
MyBatis generator version
1.4.0
Database vendor and version
MySql Ver 8.0.21 for Win64 on x86_64
Test case or example project
N/A
Steps to reproduce

Config my own generatorConfig.xml
Use the default TargetRuntime - MyBatis3DynamicSql
Define table property with parameters like following:

Expected result
The generated code can be built successfully
Actual result
Build reports error - "Result Maps collection already contains value for yourpackage.mapper.MyTableMapper.MyTableResult
Cause analysis
In the config file, the property of table's parameters, like enableSelectByExample or selectByExampleQueryId, may cause the error. After I remove those parameters, the generated mapper class is built well.
In the generator offficial documet, it says those parameters should be ignored when the target runtime is mybatis3dynamicsql. But it seems not like that, please help to check it out.

Steps to reproduce

Config my own generatorConfig.xml
Use the default TargetRuntime - MyBatis3DynamicSql
Define table property with parameters like following: enableCountByExample="false" enableUpdateByExample="false" enableDeleteByExample="false" enableSelectByExample="false" selectByExampleQueryId="false"


Macros Handlers are not being imported

Hi,
I'm using ReactiveMongo and is a nice piece of software.
But right now I'm having a weird error with the macros.
I'm trying to use this code but doesn't compile and I can't understand why.
package org.realsoftwarematters.wallboard.model

import reactivemongo.bson._
import reactivemongo.bson.{Macros => MongoMacros}

object MongoFunctions {

  import org.realsoftwarematters.wallboard.model.MongoModelImplicits._

  val driver = new MongoDriver
  val connection = driver.connection(List("localhost"))
  val db = connection.db("reporter")
  val collection = db.collection[BSONCollection]("teams")

  def getTeams: Future[List[Team]] = {

    collection.find(BSONDocument()).cursor[Team].collect[List]()
  }

  def getSimpleTeams: Future[List[SimpleTeam]] = {
    implicit val simpleTeamFormat = Macros.handler[SimpleTeam]
    implicit val sprintViewformat = Macros.handler[SprintView]
    implicit val rapidViewformat = Macros.handler[RapidView]
    implicit val sprintVelocityformat = Macros.handler[SprintVelocity]
    implicit val sprintCarryOversformat = Macros.handler[SprintCarryOvers]
    implicit val teamFormat = Macros.handler[Team]
    collection.find(BSONDocument()).cursor[SimpleTeam].collect[List]()
  }

}

object MongoModelImplicits {

  implicit val simpleTeamFormat = MongoMacros.handler[SimpleTeam]
  implicit val sprintViewformat = MongoMacros.handler[SprintView]
  implicit val rapidViewformat = MongoMacros.handler[RapidView]
  implicit val sprintVelocityformat = MongoMacros.handler[SprintVelocity]
  implicit val sprintCarryOversformat = MongoMacros.handler[SprintCarryOvers]
  implicit val teamFormat:BSONHandler[BSONDocument, Team] = MongoMacros.handler[Team]

}

Im my example I have and object with functions to get Teams and SimpleTeams, in the function getSimpleTeams the method compiles without problems as I have included in it the declaration of the Handlers but in the function getTeams doesn't compile given the following error:
Compilation error[could not find implicit value for parameter reader: reactivemongo.bson.BSONDocumentReader[org.realsoftwarematters.wallboard.model.MongoModel.Team]]

Can anyone please help me ?
Thanks in advance

Hi, please ask such question on the user MailingList. This issue tracker is available for confirmed issue. Best regards.
P.S: Check you import the recommanded play.modules.reactivemongo.json._ as indicated in the documentation.

Done!
Thanks


Discussing the API for request variables eg. whodunnit, controller_info

I originally wanted to just have controller_info be able to take a block like whodunnit, and realized that this would be a good opportunity to remove some of the duplication between the two, and allow arbitrary settings to accept a block.  Let me know what you think.  I'm not attached to the idea of the private config_handler, and can put that logic back into the methods if you think it would be better.

I spent some more time looking through the code base.  I wonder if you think it would be better to have a
PaperTrail::RequestStoreManager that handles all interactions with the request store.  It could be a private class that is delegated to by the public PaperTrail API.  Basically wherever we use paper_trail_store could use this class.  It might not be necessary yet, but just a thought.

I originally wanted to just have controller_info be able to take a block like whodunnit,  and realized that this would be a good opportunity to remove some of  the duplication between the two, and allow arbitrary settings to accept a  block.

I like this idea. Let's discuss it. Our request_store has the following keys:

key
versions table
api

:request_enabled_for_controller
no
public

:"enabled_for_#{model}"
no
public

:whodunnit
yes
public

:controller_info
yes
public

:transaction_id
yes
private

Some keys are data, to be saved in the versions table, others are configuration. Most are public, but :transaction_id is private.  The only thing they have in common is that they affect the current request.

I wonder if you think it would be better to have a
PaperTrail::RequestStoreManager that handles all interactions with the request store.

I like this idea too. It could simply be called PaperTrail::Request.

It could be a private class that is delegated to by the public PaperTrail API.

Yes, delegation will be necessary to prevent breaking changes. I'm picturing an API like this:
PaperTrail.request.enabled?
PaperTrail.request.whodunnit = 'Alex'
PaperTrail.request = PaperTrail::Request.new(
  whodunnit = 'Alex',
  controller_info = {}
}
PaperTrail.request(whodunnit: 'Alex', controller_info: {}) do
  # ...
end

This would be the preferred API and the old methods could eventually be deprecated. What do you think?
Initial code review, just trivial stuff for now:

Please don't change version_number.rb. I'll change it when I do the release.
Keep the changelog entry small, a single sentence if possible. The docs should go in various places in the readme.

Good ideas -- I'll update this PR with the discussed changes hopefully within the next week and we can take it from there.

@jaredbeck I decided to go ahead and take a first pass at this to get your thoughts before polishing the API and adding specs.
I did not change any existing specs yet, except to get rid of deprecation warnings.
If you like this approach, I'll use the request_key_handler or something like it on every method and clean up the new class's internal implementation, as well as make PaperTrail.request(options) do; ... end; functional.

All tests are passing -- by the way it looks like there is a bit of flakiness in the test suite.

Addressed your comments:

[x] Using plain ruby
[x] Left the deprecation as a comment rather than as an actual deprecation in the code
[x] Removed unnecessary require

Let me know if you think the general approach is good and I'll go ahead and move the tests over from paper_trail_spec to request (and have the ones in paper_trail_spec simply test that we are delegating to request.

Alex, I think I'm going to open a new PR with what I'd like to see, based on your work here (preserving your authorship). I think that'll be easier than trying to specify, in detail here, the API I'm imagining.

Sounds good!


Ability to override specific widget attr

django-allauth has placeholders in form field widget attrs. We get unwanted behavior when render this forms with django-material form tag.
What Is simple solution for this?

currently you can override whole widget rendering with
{% part form.email_field control %}
        <input ...>
 {% endpart %}

Some idea for implementation
{% attrs from field.widget.attrs %}
    {% part field attr.type.value %}{{ field.widget.input_type }}{% endpart %}
    {% part field attr.class.value append %}{% if bound_field.errors %}invalid {% endif %}"{% endpart %}
    {% part field attr.class.value append %}{{ field.widget.attrs.class }}{% endpart %}
    {% part field attr.name.value %}{{ bound_field.html_name }}{% endpart %}
    {% part field attr.id.value %}id_{{ bound_field.html_name }}{% endpart %}
    {% part field attr.value.value %}{{ bound_field.value }}{% endpart %}
{% endattrs %}

I override material/fields/django_input.html to realize this.
2016-02-05 8:23 GMT+03:00 Mikhail Podgurskiy<EMAIL_ADDRESS>
currently you can override whole widget rendering with
{% part form.email_field control %}
<input ...>
{% endpart %}
—
Reply to this email directly or view it on GitHub
https://github.com/viewflow/django-material/issues/73#issuecomment-180203605
.

That's definitely wrong approach.


Display Banner Photos

Once the CSV is parsed and stored, we want to display the photos in our series endpoint that we currently have. doesn't have to be pretty yet, we just want to see if it will work.

Banner photos displayed


error: Argument 1 to "mul" of "Series" has incompatible type "timedelta64" [arg-type]

Describe the bug
Series.mul has incompatible signature with Series.__mul__
To Reproduce
import pandas as pd
import numpy as np

s = pd.Series([1, 2, 3])
x = s * np.timedelta64(1, "s")  # ✔
y = s.mul(np.timedelta64(1, "s") )  # ✘ [arg-type]

Please complete the following information:

OS: Ubuntu 22.04
python version 3.10
mypy 1.4.1
pandas                        2.0.3
pandas-stubs                  <IP_ADDRESS>605

Any kind of multiplication of a series with Timedeltas induce a TimedeltaSeries. I have updated mul, __mul__, rmul and __rmul__ to be coherent.
I have quickly tested + / // & | and they all raise an error when dealing with Series and Timedeltas. So I wonder if I should add a bunch of Never or keep as is managing only *. @Dr-Irv if you could advise ?

I have quickly tested + / // & | and they all raise an error when dealing with Series and Timedeltas. So I wonder if I should add a bunch of Never or keep as is managing only *. @Dr-Irv could you advise ?

Let's leave the scope as is.  The challenge here is that you could get a Series that contains timedeltas, but from a typing perspective, it is Series[Any], but addition is allowed at runtime.
So make a PR that just aligns the arguments for things like SOMEOP and __SOMEOP__ , where SOMEOP is mul, add, etc., with appropriate tests.  I'm pretty sure that the arguments we use for the __SOMEOP__ versions are well tested so if you just copy those declarations over to SOMEOP and add tests for the SOMEOP versions, you'll be in good shape.
Separately, if you want to experiment with using Never to prevent operations, you could do a separate PR, and as long as none of the existing tests fail, it's something we could consider.
Thanks for your contributions!

Series[int] or Series[float] works fine of course for * but fails for +
>>> s2 = pd.Series([1, 2, 3])
>>> s2 * np.timedelta64(1, "s")
0   0 days 00:00:01
1   0 days 00:00:02
2   0 days 00:00:03
dtype: timedelta64[s]
>>> s2 + np.timedelta64(1, "s")
(...)
TypeError: Concatenation operation is not implemented for NumPy arrays, use np.concatenate() instead. Please do not rely on this error; it may not be given on all Python implementations.

however with Series[object] (as we mix int and timedelta):
>>> s = pd.Series([1, 2, 3, np.timedelta64(1, "s")])
>>> s
0            1
1            2
2            3
3    1 seconds
>>> s * np.timedelta64(1, "s")
(...)
numpy.core._exceptions._UFuncBinaryResolutionError: ufunc 'multiply' cannot use operands with types dtype('O') and dtype('<m8[s]')
>>> s + np.timedelta64(1, "s")
(...)
TypeError: unsupported operand type(s) for +: 'Timedelta' and 'int'
>>> s[0:2] * np.timedelta64(1, "s")
(...)
numpy.core._exceptions._UFuncBinaryResolutionError: ufunc 'multiply' cannot use operands with types dtype('O') and dtype('<m8[s]')

+ and * are failling. (you can not multiply to timedeltas anyway)
I can not find any case where Series + Timedeltas are allowed. So I'll go for Never.
I will PR what I have done for * so far and give a try with Never all the other operators in a separate branch.


pyarrow integration broken as of 0.15.0

Dask is currently passing preserve_index to pyarrow. This is deprecated as of pyarrow 0.13.0 (has no effect anymore) and is removed with pyarrow 0.15.0.

@rjzamora is this something you could look into ?

@crepererum - I see that Dask is passing a preserve_index argument to pa.Table.from_pandas, but I'm not finding it anywhere else - Am I missing something?
If you are refering to the from_pandas case, can you link documentation stating that preserve_index is deprecated? (I don't see anything in 0.15.0 documentation)

Oh bollocks, I was using an old dask version locally and was probably also looking at the wrong dask code when trying to figure out the root cause. 🤦♂  Sorry for that, works on latest.

No worries.  It's reassuring that you're out there testing things and
comfortable raising issues.
On Thu, Oct 17, 2019 at 3:10 AM Marco Neumann<EMAIL_ADDRESS>wrote:

Oh bollocks, I was using an old dask version locally and was probably also
looking at the wrong dask code when trying to figure out the root cause.
🤦♂ Sorry for that, works on latest.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/issues/5498?email_source=notifications&email_token=AACKZTBIPIMM2MD4ES5HDYDQPAMWHA5CNFSM4JBNTI22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBPG2AA#issuecomment-543059200,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACKZTDJC6OE7KNSOHAV5WDQPAMWHANCNFSM4JBNTI2Q
.

Thanks for answering @rjzamora


Very high CPU usage

Windows Version
Microsoft Windows [Version 10.0.22631.3447]
WSL Version
<IP_ADDRESS>
Are you using WSL 1 or WSL 2?

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 22.04
Other Software
No response
Repro Steps
have no idea what triggered this issue, but i built kernel linux-msft-wsl-<IP_ADDRESS> with webcam enabled, and this happened while i'm coding with vscode.
Expected Behavior
no ridiculous CPU usage
Actual Behavior

Diagnostic Logs
No response

just web development, no CPU-intensive task running.

/logs

Hi, I just sent the log via email.


Stop bundling an unused copy of Guava

#220 by @alecharp reset the bundled Guava to that included in Jenkins core—11—but this is just misleading as the plugin did not specify maskClasses nor pluginFirstClassLoader to begin with, so it was already using the copy from core.
https://www.jenkins.io/doc/developer/plugin-development/dependencies-and-class-loading/#pluginfirstclassloader-and-its-discontents

In develop branch,
$ (mvn -Pquick-build clean package && jar tf target/google-compute-engine.hpi) | fgrep guava
[WARNING] Bundling transitive dependency jsr305-1.3.9.jar (via guava)
[INFO] Bundling direct dependency guava-11.0.1.jar
[INFO] Excluding com.google.guava:guava:jar:11.0.1 from the shaded jar.
WEB-INF/lib/guava-11.0.1.jar

whereas in the PR branch,
$ (mvn -Pquick-build clean package && jar tf target/google-compute-engine.hpi) | fgrep guava
[INFO] Excluding com.google.guava:guava:jar:11.0.1 from the shaded jar.

Ping @donmccasland

FYI the log message in this case is from https://github.com/jenkinsci/maven-hpi-plugin/pull/130. @Vlatombe also had a series of PRs which corrected various cases of transitive dependencies being bundled for nonsensical reasons, though in this case the dependency was expressed directly. Not the first time we have found some JARs sitting in a plugin for no good reason. :grin:

hey!  thanks for this.

Hello, can maintainer cut a release for this please ?


T

https://github.com/web-platform-tests/wpt/blob/master/fetch/redirect-navigate/resources/destination.html

Nothing actionable here, so closing this.


enquiry about stronger detector

@akhilpm Hi! thanks for sharing the great work! I'm wondering you have tried using Cascaded RCNN for the detector to achieve a stronger performance? and I note there were some prior works like FOCUS-AND-DETECT and AdaZoom that perform very good on Visdrone, have you compared to them and could you please give some comments?

Hello @twangnh, no I used Faster RCNN in the two-stage family and FCOS in the one-stage family. Code is also published for the same. But if you want to use Cascaded RCNN, I think it will be easy to adapt my code just the way detectron2 inherits the "GeneralizedRCNN" and make the "CascadedRCNN".
I remember coming across FOCUS-AND-DETECT, but it was using a ResNeXt101 backbone and no code is available to reproduce to a comparable backbone. It also uses deformable conv whereas we used the standard one. AdaZoom is new to me. It is around 1.8 points better than my recent FCOS train where I found the crops by running the crop labeling algorithm on the detection predictions instead of directly taking the crop predictions. This is slightly slower than the way described in the paper, but I got an mAP around 34.4. I would expect a bit more improvement if you use high resolution features "P2" of the FPN.
But in general, the fundamental problem we cited in the paper remains valid for these two you listed here as well. Adding external networks, complicated training, etc. We find the practitioners are still using uniform cropping or variants of it like SAHI. This we assume is because of their plug-and-play nature and no significant change in the standard detector training. So we aimed to bring the same simplicity with our proposed approach.


qs Config customization

We have the ability to customize the qs config used for stringifying but not for parsing.
Add the ability to customize for both and optionally add the ability to set a default qs config to use for all qs operations.

Add the ability to customize for both and optionally add the ability to set a default qs config to use for all qs operations.

This would be quite a nice DX to have. We can have introduce some getters and setters for the qs options. Now that makes me wonder about how initializeWithQueryString and getQueryString would behave when additional options are send after setting some qs config.
We can,
Option A. Override qs options when such parameters are present (making the behaviour consistent with how getQueryString behaves now)
Option B. Deep merge both qs options in such a way that the one that got passed in wins when there are conflicting keys
Option C. Replace qs options with the most recently passed options.
I am in favour of Option A, but what do you think?

Yep, I prefer option A. If a user wants to pass different options they can use the getter to get the internal options, control how they are merged themselves, then pass in the preferred options as an override. Or they can just use the getters and setters to update the internal options as needed.

Fixed in version 2.2.0


Scrambled text above protein page context cartoon

https://www.beta.patricbrc.org/view/Feature/PATRIC.866909.3.AHHF01000082.CDS.58964.61276.rev

@olsonanl, I can not reproduce it.

I think this is an old version before we implement multiple tracks to avoid overlap.

seems to be fixed.


Update servers.js

New Steam servers.

Where is this list from?

From the official API.

SteamKit2 is considering reintroducing a bootstrapped list (https://github.com/SteamRE/SteamKit/pull/597#issuecomment-428130452). If they do, I'll use their list. Otherwise I think I'll get rid of servers.js and use cm0.steampowered.com (which works most of the time) by default and try to make it clearer that one should really handle the 'servers' event (as if the code in example.js weren't enough).

Thank you for your contribution. I chose a different approach in https://github.com/seishun/node-steam/commit/8f0f68bc244bc02fd38c585bbe956b9546c562a0. Instead of hard-coding a server list, it is downloaded automatically when installing from git and is (will be) bundled only in the npm package.


Combined machine->host bugs

hostname needs to be creatable but not editable
name should not be required
Hosts in some states (provisioning, possibly others?) can't be deleted

Note: This is preventing swarm testing as labels are not able to be set onto hosts for swarm deployment as the name is required.

With latest build from master, I am not able to add Digital Ocean machine. User is presented with - "Name" is required  error message even after name is provided.

Tested with latest build from master:
1.Able to add Digital Ocean machine successfully.
2.Able to edit host and add labels successfully.
3. Not allowed to edit host name , but name is editable.
I am still having problems when trying to delete a host when it is provisioning state.
I see it get deleted ( or atleast removed from the host view) for sometime. But it reappears again in  host list view in "registering" state.
When listing host using API , the deleted host is not listed. But it is still listed in machine list API in "active" state,

Tested with rancher-server version - v1.2.0-pre4-rc5
When deleting a host that is  still in provisioning state, host gets to "removing" state and then disappears from the UI.
Machine entry associated with this host continues to remain in DB in "removing" state for a while and then gets to "removed" state.


undefined method '+' down in net/http.rb when using Request.execute

I've got a problem I've tried to diagnose, but I'm drawing a blank, and thinking somebody here might be able to help. I'm trying to make a REST call to an API that requires SSL and a few custom headers. This was working fine until this morning and now for whatever reason refuses to work and I'm not sure what has changed.
When my code tries to do this:
RestClient::Request.execute(method: :get, url: "#{@url}/#{remote_method}?username=#{@login}&#{params}", headers: {:password => @password, :id => @id}, verify_ssl: false)
If it matters, @url is set to a string that is "https://server.i.need.to.keep.private.com:443/rest/v1" and can confirm the other variables in that string are all non-nil.
I get this stack trace:
error: undefined method `+' for nil:NilClass
/home/p7r/.rvm/rubies/ruby-2.3.1/lib/ruby/2.3.0/net/http.rb:1561:in `addr_port': undefined method `+' for nil:NilClass (NoMethodError)
        from /home/p7r/.rvm/rubies/ruby-2.3.1/lib/ruby/2.3.0/net/http.rb:1494:in `begin_transport'
        from /home/p7r/.rvm/rubies/ruby-2.3.1/lib/ruby/2.3.0/net/http.rb:1433:in `transport_request'
        from /home/p7r/.rvm/rubies/ruby-2.3.1/lib/ruby/2.3.0/net/http.rb:1407:in `request'
        from /home/p7r/.rvm/gems/ruby-2.3.1/gems/rest-client-1.8.0/lib/restclient/request.rb:270:in `net_http_do_request'
        from /home/p7r/.rvm/gems/ruby-2.3.1/gems/rest-client-1.8.0/lib/restclient/request.rb:418:in `block in transmit'
        from /home/p7r/.rvm/rubies/ruby-2.3.1/lib/ruby/2.3.0/net/http.rb:853:in `start'
        from /home/p7r/.rvm/gems/ruby-2.3.1/gems/rest-client-1.8.0/lib/restclient/request.rb:413:in `transmit'
        from /home/p7r/.rvm/gems/ruby-2.3.1/gems/rest-client-1.8.0/lib/restclient/request.rb:176:in `execute'
        from /home/p7r/.rvm/gems/ruby-2.3.1/gems/rest-client-1.8.0/lib/restclient/request.rb:41:in `execute'
        [... snip ... ]

My Gemfile.lock shows rest-client as follows:
    rest-client (1.8.0)
      http-cookie (>= 1.0.2, < 2.0)
      mime-types (>= 1.16, < 3.0)
      netrc (~> 0.7)

What really confuses me is that this was working just fine until this morning and then stopped working. I've tried different versions of Ruby, but no dice.

Is there any more detail I can provide to help debug this?

Based on that backtrace it sounds like you are somehow passing an empty hostname in the URL. It's a bit hard to debug if you need to keep that secret, but I'd look into what the rendered URL actually is after the string interpolation.

Closing, but feel free to reopen with more info.


How to pass secret on POST /services/create ?

I'm using Docker v1.3, I want to pass secret on POST /services/create remote API.
In docker command line, it will be equal to docker service create --secret login-password  ubuntu.
However documentation on the website is only showing for v1.2.
So how to pass secret on  POST /services/create remote API?

The API documentation for 1.13 has not been published yet, because 1.13 is not released yet. See https://github.com/docker/docker/tree/v1.13.0-rc7/api#updating-the-api-documentation on how to view the API docs (which is not generated using swagger)
However, the easiest way to learn how the API calls look is to start the daemon (dockerd) in debug mode, and run the command with the CLI. API requests are visible in the daemon logs (or printed to stdout/stderr if you manually run the daemon in the foreground).
Please keep in mind that the GitHub issue tracker is not intended as a general support forum,
but for reporting bugs and feature requests. For other type of questions, consider using one of;

https://forums.docker.com
the Docker community Slack channel
StackOverflow

I'm going to close this issue because this is not a bug, but feel free to continue the conversation 👍


Added Coursera & Leetcode

Hey @diegotf30, would you be able to answer @tupaschoal's questions? We cannot in good faith merge this PR until they are addressed.

Hi! Sorry I got caught up on school projects and forgot about this, I have made my comments on the respective changes

@diegotf30 Your changes look good, but your branch does not seem to be in a good state to be merged... This is partially due to the fact that your master branch did not have updates from this repo's master branch incorporated before you continued working on it since previous PRs.
I've done the work of reworking your commits into neater commits (one per site, including gulp run per commit) and pushed that branch to nightfirecat/justdelete.me as the "diego-master" branch if you'd like to use it to update your repository. (it seems you have declined to grant access for us to push to your repository)
You should be able to force-update your master branch from this one with the following commands:
git remote add nfc git@github.com:nightfirecat/justdelete.me.git  # add my repository as the "nfc" remote
git fetch nfc                                                     # fetch my repository's branches
git branch new-master nfc/diego-master                            # create a new local "new-master" branch from my repository's "diego-master" branch
git checkout new-master                                           # set HEAD to the "new-master" branch
git branch -D master                                              # delete your local copy of "master"
git checkout -b master                                            # create a local "master" branch from HEAD (which is your local "new-master" branch), and move HEAD to that new branch
git branch -D new-master                                          # remove your local "new-master" branch--no longer needed, since local "master" is now a copy of it
git branch -u origin/master                                       # set the new local "master" branch's upstream to your repository's master branch
git push -f                                                       # force-sync your repository's branch to be identical to your local "master" branch
git remote remove nfc                                             # remove my repository from your tracked remotes; it's no longer needed

Weird, I used git pull before working the branch and there were no updates, anywho, thanks for the detailed tutorial and help provided! :)

Your master branch's upstream is probably not pointing to this repository; you'd need to do git fetch && git merge --ff-only jdm/master (assuming this repository's remote is named jdm)


Fix the error on getship KCSAPI

Not always reproducible? At least that happened when an I-168 is received.

Closing this bug as I haven't encountered this in the last few months.


set standard id without spaces

https://code.visualstudio.com/api/language-extensions/language-server-extension-guide#logging-support-for-language-server
logging support from vscode-languageclient uses
langId.trace.server = verbose

but as vala-vscode uses LanguageClient constructor that defines both id and name, id has spaces and capital letters and probably that confuses the config. None of this works:
"Vala Language Server3.trace.server": "verbose",
"Vala\\ Language\\ Server3.trace.server": "verbose",

with this change:
"vala-language-server.trace.server": "verbose"

works and allow to see all communications

We can document this and add to extension config if needed


Allow enter key on login to try to login

When accessing on web page on browser on a computer, enter key should trigger the login

Closed in 1.1.6


Implement decode_len on storages by decl_storage

parity-scale-codec does provide the feature of decoding only the length an element (the lenght of a vec for instance). storage with such element should provide this feature on its element.
This is a placement issue for reference from the code.

I am more or less doing the same here: #3071 but the PR has taken a bit too long. I suppose if either of us can add Basti's recommendation on having a noDefault  marker trait, then it should be good to go.

yes maybe but at some point I wanted them to be implemented same as append in StorageValue just after a where condition.
Actually I think we can implement them for both map and linkedmap so it is fine behind where condition.

actually done in https://github.com/paritytech/substrate/pull/3344


add basic conan recipe

Added a basic conan recipe file. This both is a complete file but also WIP depending on what people need. Its been a while since I used conan so there likely exists obvious adjustsments that should be done.
To install into conan cache, run
conan create .

in doctest repo.
In the consumers conan recipe file add
build_requires = "doctest/2.3.8"
generators = "cmake_paths"

I prefer using cmake_paths generator as is fully decouples conan from cmake with the exception of the inclusion of the conan_paths.cmake file that appends to CMAKE_MODULE_PATH and CMAKE_PREFIX_PATH.
and in the cmake files of the consumer add
if(EXISTS ${CMAKE_BINARY_DIR}/conan_paths.cmake)
    include(${CMAKE_BINARY_DIR}/conan_paths.cmake)
endif()
find_package(doctest)
include (doctest)

Then link against target doctest::doctest as normal.

That is very much a valid point.
Not sure why the conan center recipe does not use the installation procedure provided by your library, as it does not copy the doctestConfig.cmake file, so using find_package is not possible.
I should maybe just try to do a PR against that file instead.

As I am also entirely unfamiliar with conan I would't be able to maintain it either. I have updated the conan links (e3e2127) in the docs (recipe is 2 minor versions behind) and would of course appreciate any PRs sent their way!