train priblem

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):Distributor ID:
CentOS
Description:    CentOS Linux release 7.2.1511 (Core)
Release:        7.2.1511
TensorFlow installed from (source or binary):pip
TensorFlow version (use command below):1.5.0
Python version: Python 2.7.5
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:no
GPU model and memory:no
Exact command to reproduce:

ps
2018-03-06 11:13:09.816655: I tensorflow/core/distributed_runtime/rpc/grpc_server_lib.cc:324] Started server with target: grpc://localhost:2222
2018-03-06 11:14:56.902071: W tensorflow/core/framework/op_kernel.cc:1198] Not found: checkpoint/model.ckpt-1_temp_9e8396c2f6d04e91ad3c3fbc194a9f70; No such file or directory
2018-03-06 11:14:56.902071: W tensorflow/core/framework/op_kernel.cc:1198] Not found: checkpoint/model.ckpt-1_temp_9e8396c2f6d04e91ad3c3fbc194a9f70; No such file or directory

work:
2018-03-06 11:10:24.342583: I tensorflow/core/distributed_runtime/master_session.cc:1017] Start master session 592fd1b6a4409f07 with config: gpu_options { allow_growth: true } allow_soft_placement: true
Exception in thread QueueRunnerThread-dummy_queue-sync_token_q_EnqueueMany:
Traceback (most recent call last):
  File "/usr/lib64/python2.7/threading.py", line 812, in __bootstrap_inner
    self.run()
  File "/usr/lib64/python2.7/threading.py", line 765, in run
    self.__target(*self.__args, **self.__kwargs)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/queue_runner_impl.py", line 266, in _run
    coord.request_stop(e)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/coordinator.py", line 211, in request_stop
    six.reraise(*sys.exc_info())
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/queue_runner_impl.py", line 250, in _run
    enqueue_callable()
  File "/usr/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 1251, in _single_operation_run
    self._session, None, {}, [], target_list, status, None)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/framework/errors_impl.py", line 473, in __exit__
    c_api.TF_GetCode(self.status.status))
CancelledError: Step was cancelled by an explicit call to `Session::Close()`.

Traceback (most recent call last):
  File "leave.py", line 204, in <module>
    tf.app.run()
  File "/usr/lib/python2.7/site-packages/tensorflow/python/platform/app.py", line 124, in run
    _sys.exit(main(argv))
  File "leave.py", line 195, in main
    rnn_model.global_step], feed_dict={keep_prob: FLAGS.dropout_keep_prob})
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/monitored_session.py", line 539, in run
    run_metadata=run_metadata)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/monitored_session.py", line 1013, in run
    run_metadata=run_metadata)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/monitored_session.py", line 1104, in run
    raise six.reraise(*original_exc_info)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/monitored_session.py", line 1089, in run
    return self._sess.run(*args, **kwargs)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/monitored_session.py", line 1169, in run
    run_metadata=run_metadata))
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/basic_session_run_hooks.py", line 452, in after_run
    self._save(run_context.session, global_step)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/basic_session_run_hooks.py", line 468, in _save
    self._get_saver().save(session, self._save_path, global_step=step)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1614, in save
    raise exc
tensorflow.python.framework.errors_impl.NotFoundError: checkpoint/model.ckpt-1_temp_5d8d532077d241a38f2d79ac2ffeb9a7; No such file or directory
         [[Node: save/SaveV2 = SaveV2[dtypes=[DT_INT64], _device="/job:ps/replica:0/task:0/device:CPU:0"](save/ShardedFilename, save/SaveV2/tensor_names, save/SaveV2/shape_and_slices, global_step)]]
         [[Node: save/Identity_S445 = _HostRecv[client_terminated=false, recv_device="/job:worker/replica:0/task:0/device:CPU:0", send_device="/job:ps/replica:0/task:0/device:CPU:0", send_device_incarnation=-8763115780137487660, tensor_name="edge_111_save/Identity", tensor_type=DT_STRING, _device="/job:worker/replica:0/task:0/device:CPU:0"]()]]

Caused by op u'save/SaveV2', defined at:
  File "leave.py", line 204, in <module>
    tf.app.run()
  File "/usr/lib/python2.7/site-packages/tensorflow/python/platform/app.py", line 124, in run
    _sys.exit(main(argv))
  File "leave.py", line 184, in main
    config=set_config_proto()) as mon_sess:
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/monitored_session.py", line 380, in MonitoredTrainingSession
    stop_grace_period_secs=stop_grace_period_secs)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/monitored_session.py", line 787, in __init__
    stop_grace_period_secs=stop_grace_period_secs)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/monitored_session.py", line 511, in __init__
    self._sess = _RecoverableSession(self._coordinated_creator)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/monitored_session.py", line 972, in __init__
    _WrappedSession.__init__(self, self._create_session())
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/monitored_session.py", line 977, in _create_session
    return self._sess_creator.create_session()
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/monitored_session.py", line 668, in create_session
    self.tf_sess = self._session_creator.create_session()
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/monitored_session.py", line 431, in create_session
    self._scaffold.finalize()
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/monitored_session.py", line 210, in finalize
    self._saver = training_saver._get_saver_or_default()  # pylint: disable=protected-access
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 821, in _get_saver_or_default
    saver = Saver(sharded=True, allow_empty=True)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1239, in __init__
    self.build()
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1248, in build
    self._build(self._filename, build_save=True, build_restore=True)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1284, in _build
    build_save=build_save, build_restore=build_restore)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 756, in _build_internal
    save_tensor = self._AddShardedSaveOps(filename_tensor, per_device)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 382, in _AddShardedSaveOps
    return self._AddShardedSaveOpsForV2(filename_tensor, per_device)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 356, in _AddShardedSaveOpsForV2
    sharded_saves.append(self._AddSaveOps(sharded_filename, saveables))
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 297, in _AddSaveOps
    save = self.save_op(filename_tensor, saveables)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 240, in save_op
    tensors)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/ops/gen_io_ops.py", line 1174, in save_v2
    shape_and_slices=shape_and_slices, tensors=tensors, name=name)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
    op_def=op_def)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 3160, in create_op
    op_def=op_def)
  File "/usr/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1625, in __init__
    self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access

NotFoundError (see above for traceback): checkpoint/model.ckpt-1_temp_5d8d532077d241a38f2d79ac2ffeb9a7; No such file or directory
         [[Node: save/SaveV2 = SaveV2[dtypes=[DT_INT64], _device="/job:ps/replica:0/task:0/device:CPU:0"](save/ShardedFilename, save/SaveV2/tensor_names, save/SaveV2/shape_and_slices, global_step)]]
         [[Node: save/Identity_S445 = _HostRecv[client_terminated=false, recv_device="/job:worker/replica:0/task:0/device:CPU:0", send_device="/job:ps/replica:0/task:0/device:CPU:0", send_device_incarnation=-8763115780137487660, tensor_name="edge_111_save/Identity", tensor_type=DT_STRING, _device="/job:worker/replica:0/task:0/device:CPU:0"]()]]

   with tf.train.MonitoredTrainingSession(master=server.target,
                                                   is_chief=is_chief,
                                                   checkpoint_dir="checkpoint/",
                                                   hooks=hooks,
                                                   save_checkpoint_secs=60,
                                                   config=set_config_proto()) as mon_sess:

The development environment for the MAC can run. On the server, if the amount of data is relatively small, it can run, and the data volume is too large to fail.

Even if all the data is cleared, the rerun is a failure.

have you solved this problem? @cyqluowang

From the error message NotFoundError (see above for traceback): checkpoint/model.ckpt-1_temp_5d8d532077d241a38f2d79ac2ffeb9a7; No such file or directory it seems that the workers are having trouble writing to the checkpoint directory.
It is possible that the parent directory where checkpoints are to be written is either not accessible or is being removed while the training is in process?

Closing since I interpreted your last comment to imply that my hunch was correct. Please feel free to reopen if I was mistaken. Thanks.

i got a same error when doing sync training in distribution environment.
Sys: Linux 14.04
TF: 1.6.0
Exception in thread QueueRunnerThread-dummy_queue-sync_token_q_EnqueueMany: Traceback (most recent call last):   File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner     self.run()   File "/usr/lib/python2.7/threading.py", line 754, in run     self.__target(*self.__args, **self.__kwargs)   File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/queue_runner_impl.py", line 268, in _run     coord.request_stop(e)   File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/coordinator.py", line 213, in request_stop     six.reraise(*sys.exc_info())   File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/training/queue_runner_impl.py", line 252, in _run     enqueue_callable()   File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/client/session.py", line 1259, in _single_operation_run     None)   File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/framework/errors_impl.py", line 516, in __exit__     c_api.TF_GetCode(self.status.status)) CancelledError: Step was cancelled by an explicit call to Session::Close().
and part of my codes is:
`class _LoggerHook(tf.train.SessionRunHook):
def begin(self):
self._step = 0
self._start_time = time.time()
self._total_loss = 0
            def before_run(self, run_context):
                self._step += 1
                return tf.train.SessionRunArgs(loss)

            def after_run(self, run_context, run_values):
                loss_value = run_values.results
                self._total_loss += loss_value
                if self._step % FLAGS.log_frequency == 0:
                    current_time = time.time()
                    duration = current_time - self._start_time
                    self._start_time = current_time
                    if self._step==0:
                        avg_loss = loss_value
                    else:
                        avg_loss = self._total_loss/self._step
                    eg_per_sec = FLAGS.log_frequency * FLAGS.batch_size / duration
                    sec_per_batch = float(duration / FLAGS.log_frequency)
                    print('%s: training step %d cur loss = %.4f avg loss = %.4f (%.1f images/sec %.3f sec/batch)'
                          % (datetime.now(), self._step, loss_value, avg_loss, eg_per_sec, sec_per_batch))

        all_hooks=[tf.train.NanTensorHook(loss), tf.train.StopAtStepHook(last_step=FLAGS.max_steps), _LoggerHook()]
        if FLAGS.issync:
            all_hooks.append(sync_replicas_hook)
        if FLAGS.debug:
            all_hooks.append(tfdbg.LocalCLIDebugHook(ui_type='curses'))
        if FLAGS.finetune:
            print('Finetune from %s'%FLAGS.finetune)
            saver = tf.train.Saver()
        config = tf.ConfigProto(log_device_placement=FLAGS.log_device_placement)
        config.gpu_options.allow_growth=True
        with tf.train.MonitoredTrainingSession(
                master=server.target,
                is_chief=(FLAGS.task_index == 0),
                checkpoint_dir=FLAGS.model_dir,
                hooks=all_hooks,
                config=config,
                save_summaries_steps=100,
                save_summaries_secs=None,
                log_step_count_steps=None) as sess:
            if FLAGS.finetune:
                print('Load Pretrained model')
                ckpt = tf.train.get_checkpoint_state(FLAGS.finetune)
                if ckpt and ckpt.model_checkpoint_path:
                    saver.restore(sess, ckpt.model_checkpoint_path)
                print('-------------------------')
            while not sess.should_stop():
                sess.run(train_op)`

Got the same error log of @HandsomeHans  in TensorFlow 1.8.0.


Mup Setup failed with 302 and 403 Forbidden error when Installing PhantomJS

Here's the error message:
[IPAddress] - Installing PhantomJS
[IPAddress] x Installing PhantomJS: FAILED
-----------------------------------STDERR-----------------------------------
ing response... 302 FOUND
Location: https://bbuseruploads.s3.amazonaws.com/ariya/phantomjs/downloads/phantomjs-1.9.8-linux-x86_64.tar.bz2?Signature=fEXVDWJCqI8YUw6fQ4FocBrm%2FXo%3D&Expires=1458683393&AWSAccessKeyId=AKIAIWY5XSVPZPDQYRQQ&versionId=null&response-content-disposition=attachment%3B%20filename%3D%22phantomjs-1.9.8-linux-x86_64.tar.bz2%22 [following]
--2016-03-22 17:55:04--  https://bbuseruploads.s3.amazonaws.com/ariya/phantomjs/downloads/phantomjs-1.9.8-linux-x86_64.tar.bz2?Signature=fEXVDWJCqI8YUw6fQ4FocBrm%2FXo%3D&Expires=1458683393&AWSAccessKeyId=AKIAIWY5XSVPZPDQYRQQ&versionId=null&response-content-disposition=attachment%3B%20filename%3D%22phantomjs-1.9.8-linux-x86_64.tar.bz2%22
Resolving bbuseruploads.s3.amazonaws.com (bbuseruploads.s3.amazonaws.com)... <IP_ADDRESS>
Connecting to bbuseruploads.s3.amazonaws.com (bbuseruploads.s3.amazonaws.com)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 403 Forbidden
2016-03-22 17:55:04 ERROR 403: Forbidden.

When I browse to the URL: https://bbuseruploads.s3.amazonaws.com/ariya/phantomjs/downloads/phantomjs-1.9.8-linux-x86_64.tar.bz2?Signature=fEXVDWJCqI8YUw6fQ4FocBrm%2FXo%3D&Expires=1458683393&AWSAccessKeyId=AKIAIWY5XSVPZPDQYRQQ&versionId=null&response-content-disposition=attachment%3B filename%3D"phantomjs-1.9.8-linux-x86_64.tar.bz2"
I got AccessDenied response.

AccessDenied
Request has expired
2016-03-22T21:49:53Z
2016-03-22T22:06:12Z
9834BD444F97E752

9w1+9A66YDzuQfUd259qoY8JXFFTT9UZyBxf9qKwaD5S23G08gSiOphUU3pNIrq5RBf++6car0I=

Is this simply because the bits for PhantomJS hosted on Amazon S3 is now simply expired and needs to re-activate or point to a new valid URL?
Thank you in advance for your help.

Changed to use MUPX and it works like a charm. You guys are awesome!! Thanks for an amazing tools!


dak.unzip

Now that Uproot 5 (and thus uproot.dask) is officially released, I'm going to start making issues for these NotImplemented errors when I run into them:
    @borrow_docstring(ak.unzip)
    def unzip(array, highlevel=True, behavior=None):
>       raise DaskAwkwardNotImplemented("TODO")
E       dask_awkward.utils.DaskAwkwardNotImplemented: TODO
E
E       If you would like this unsupported call to be supported by
E       dask-awkward please open an issue at:
E       https://github.com/dask-contrib/dask-awkward.

The use case here is of course to be able to invert the zip() operation, which for me is mainly important for being able to access the individual members of tuples created by combinations() or cartesian()

I started working on this one and while I see a few hacky ways to deal with a tuple of arrays being returned, I couldn't really decide a nice way to do it given the current infrastructure.
@douglasdavis I think the best way to do this is to maintain the appearance of the awkward interface which return tuple(ak.Array), but it wasn't really clear to me how to make an efficient graph for this using dask.HighLevelGraph as there isn't any immediate documentation on indexing into returned products. I guess the evolution of the meta would also have to understand how to interpret the tuple.


Suspected bug in pull requests (getting an Error: 500)

I'm experimenting with Gogs' pull-requests, and I can't make them work. Our existing workflow doesn't use pull requests, but we've got a team that wants to try them. On my development server, I tried to make a couple of accounts and experiment with pull requests.
I made a new repo called less in my main LDAP-authenticated account, and then added it as a remote master and pushed into it. That was all fine. I was also able to fork the repo onto a local-user account that I created (which also works fine).
When I try to generate a PR, however, Gogs reports a 500 error. Looking at the log file, I see the following error: [...routers/repo/pull.go:679 CompareAndPullRequestPost()] [E] NewPullRequest: newIssue: GetRepositoryByID: repository does not exist [id: 13, uid: 0, name: ]
I verified that both repos exist on the filesystem, and I also tried deleting both repos and redoing the experiment with new repos. I also tried rebooting the server.
My setup:

Gogs version: v0.9.46
Git version: v2.7.4
Operating system: Ubuntu 16.04
Database: SQLite
Can't replicate it on try.gogs.io.

Duplicate: #3291


Error starting Plugin

What issue do you have? Please be as thorough and explicit as possible.
Hi, I have an issue related with this message when I start the homebridge.
“[eWeLink]************ cannot load home bridge-ewelink***********
[eWeLink] invalid value “ undefined” for header “host”
I have had eWeLink-Max but I uninstalled before use your plugin
Thanks Nico
...
Details of your setup.
Do you use Homebridge or HOOBS?
HomeBridge
...
Which version of Homebridge/HOOBS do you have?
V1.1.6
...
Which version of this plugin (homebridge-ewelink) do you have?
V3.9.0
...
Please paste any relevant logs below. It helps if you can turn debug and debugReqRes in the package settings for more thorough logging.

If you are posting an error then it is helpful for me to also see the previous few lines as this can show the cause of the error.

...

Hi Nico,
Please could you turn on "Debug Logging" and "Request & Response Logging" via the plugin settings, restart Homebridge, and copy and paste the logs from Homebridge - the error itself and the lines around it too.
This would be much more helpful to me.
Thanks

Of corse:
************* Cannot load homebridge-ewelink *************
[10/31/2020, 14:04:42] [eWeLink] TypeError [ERR_HTTP_INVALID_HEADER_VALUE]: Invalid value "undefined" for header "Host"
at ClientRequest.setHeader (_http_outgoing.js:529:3)
at new ClientRequest (_http_client.js:243:14)
at Object.request (https.js:316:10)
at RedirectableRequest._performRequest (/usr/local/lib/node_modules/homebridge-ewelink/node_modules/follow-redirects/index.js:241:24)
at new RedirectableRequest (/usr/local/lib/node_modules/homebridge-ewelink/node_modules/follow-redirects/index.js:60:8)
at Object.wrappedProtocol.request (/usr/local/lib/node_modules/homebridge-ewelink/node_modules/follow-redirects/index.js:437:14)
at dispatchHttpRequest (/usr/local/lib/node_modules/homebridge-ewelink/node_modules/axios/lib/adapters/http.js:179:25)
at new Promise ()
at httpAdapter (/usr/local/lib/node_modules/homebridge-ewelink/node_modules/axios/lib/adapters/http.js:21:10)
at dispatchRequest (/usr/local/lib/node_modules/homebridge-ewelink/node_modules/axios/lib/core/dispatchRequest.js:52:10) {
code: 'ERR_HTTP_INVALID_HEADER_VALUE'
}
This is the configuration (excluded personal info):
{
"name": "eWeLink",
"countryCode": "34",
"username": "",
"password": "",
"debug": true,
"debugReqRes": true,
"disableHTTPRefresh": false,
"disableEveLogging": false,
"inUsePowerThreshold": 0,
"hideTHSwitch": false,
"lowBattThreshold": 25,
"sensorTimeDifference": 120,
"hideZBLDPress": false,
"groups": [
{
"operationTime": 100
}
],
"bridgeSensors": [
{
"sensorTimeLength": 60
}
],
"platform": "eWeLink"
}

Could you post the logs from the beginning of restarting Homebridge up to this error?

[HB Supervisor] Restarting Homebridge...
[10/31/2020, 14:04:28] [HB Supervisor] Starting Homebridge with extra flags: -I
[10/31/2020, 14:04:28] [HB Supervisor] Started Homebridge v1.1.6 with PID: 11916
[10/31/2020, 14:04:31] Loaded config.json with 0 accessories and 3 platforms.
[10/31/2020, 14:04:31] ---
[10/31/2020, 14:04:34] Loaded plugin<EMAIL_ADDRESS>[10/31/2020, 14:04:34] Registering platform 'homebridge-config-ui-x.config'
[10/31/2020, 14:04:34] ---
[10/31/2020, 14:04:39] Loaded plugin<EMAIL_ADDRESS>[10/31/2020, 14:04:39] Registering platform 'homebridge-ewelink.eWeLink'
[10/31/2020, 14:04:39] ---
[10/31/2020, 14:04:40] Loaded plugin<EMAIL_ADDRESS>[10/31/2020, 14:04:40] Registering platform<EMAIL_ADDRESS>[10/31/2020, 14:04:40] ---
[10/31/2020, 14:04:40] Loading 3 platforms...
[10/31/2020, 14:04:40] [Config] Initializing config platform...
[10/31/2020, 14:04:40] [Config] Running in Service Mode
[10/31/2020, 14:04:40] [TuyaWebPlatform] Initializing TuyaWebPlatform platform...
[10/31/2020, 14:04:40] [eWeLink] Initializing eWeLink platform...
[10/31/2020, 14:04:40] [TuyaWebPlatform] Loading accessory from cache: Cartel APP
[10/31/2020, 14:04:40] [eWeLink] Plugin has finished initialising. Syncing with eWeLink.
[10/31/2020, 14:04:40] [eWeLink] Sending HTTP getHost() request. This text is yellow for clarity.
{
"appid": "oeVkj2lYFGnJu5XUtWisfW4utiN4u9Mq",
"country_code": "+34",
"nonce": "0fmaqr6a",
"ts":<PHONE_NUMBER>,
"version": 8
}
Setup Payload:
X-HM://0023SWMMJ9B5D
Enter this code with your HomeKit app on your iOS device to pair with Homebridge:
#################
[10/31/2020, 14:04:40] Homebridge is running on port 51284.
[10/31/2020, 14:04:40] [TuyaWebPlatform] Missing required id property on device type overwrite, received:
{
"device_type": "form"
}
[10/31/2020, 14:04:40] [TuyaWebPlatform] Existing Accessory found [Name: Cartel APP] [Tuya ID: bf704dc4d9829e2bbf0doe_1] [HomeBridge ID: 7d9c640a-daab-4842-9162-9c5b59b590cf]
[10/31/2020, 14:04:40] [TuyaWebPlatform] [Cartel APP] Characteristic.On - Enabled
[10/31/2020, 14:04:40] [TuyaWebPlatform] Enable cloud polling with interval 605s
[10/31/2020, 14:04:42] [eWeLink] Sending HTTP login() request. This text is yellow for clarity.
{
"countryCode": "+34",
"password": "hidden",
"email": "hidden"
}

For some reason you aren't being provided with a http host. I will have to dig a bit deeper, could I message you on the Homebridge Discord? My username there is bwp91

Ok I’m in.

I need to post something?

I’ve already sent you a direct message on there!

Closing this issue now as @iNaiks it is working for you now?

I think yes. I will test in the next days


Flipswitch buttons on rtl

@r0ms Flipswitch buttons doesn't while the RTL mode is activated.

Thanks Antoine for all of you reviews and evething else !
I saw that you already merged my buttons branch but I willl have a look on it on monday !
Thanks again and I guess from all the others Front Dev @ Viadeo, they will enjoy that they can use these new facay sexy buttons !

Can it be fixed so we can release a new version of Tetra UI?

@r0ms we need to upgrade tetra in order to use the new alt-buttons. Can you try to fix it?


Add theme: Floating Status Bar

Add theme: Floating Status Bar
This PR adds a new theme to the theme library.
Theme Details

Name: Floating Status Bar
Description: Mod for Zen Browser that detaches the status bar from the bottom left corner of the browser window.
Homepage: https://github.com/AmirhBeigi/zen-floating-statusbar/
Author: @AmirhBeigi

Great!


KAFKA-5246: Remove backdoor that allows any client to produce to internal topics

removing unused AdminUtils.AdminClientId, as its a security hole.
@ijuma @guozhangwang would appreciate a review please =)
I state the contribution is my own work and I license the work to the project under the project's open source license.

Thanks for the PR. Not sure I would call this a security hole. In a secure cluster, ACLs would be used to determine who can write to internal topics, right?
Having said that, if there is no good use case for allowing this, it may be a good idea to disallow it. It is technically a breaking change to the protocol, which means a KIP is required. @junrao, do you know why this was allowed in the first place?

@ijuma thanks for taking a look. Seems a shame that a three line code change to remove an undocumented backdoor requires a KIP.  Are we sure we want the overhead of going down that route?  @junrao any background and opinions much appreciated!

@ijuma  @junrao friendly bump...

@datalorax : Thanks for the patch.
This is added in https://issues.apache.org/jira/browse/KAFKA-1580 with the intention of allowing the admin to manually fix the data in internal topics like __consumer_offsets. It seems that in those cases, it's better to let the admin issue commitOffsetRequest instead of producing to the internal topics directly.
So, I agree that it seems it's better to remove this logic. @jjkoshy : Since you suggested this change, what do you think?

@junrao I actually think it helps to leave open the option of directly producing to internal topics. While it is true that a user could send an OffsetCommitRequest, we have in the past also used the facility to emit tombstones. It also allows manually emitting group state (should you wish to do that) into the topic. So in short, IMO I don't think this backdoor should be removed.

Hummm... OK. Given that this can be locked down using ACLs, I guess there is no harm in leaving this in. Let's close this then. Thanks all.


Disconnect from GSM network, turn off modem

1. SDK version（SDK 版本）
{
V2.112
}

2. In what kind of operation problems appear, and how to reproduce the problem ?（什么样的操作步骤问题会出现，是否是稳定复现，如何复现问题？）
{
Is there any way of disconnecting/unregistering from the GSM network and stop searching? And turn off the modem while still keeping the main processor running? PM_SleepMode(true) does not disconnect from the network, as mentioned here #79
}

Here is the API for the Network.
You can simply search in Include folder for more API functionality.
https://github.com/Ai-Thinker-Open/GPRS_C_SDK/blob/master/include/api_network.h

Ahh thank you!
I cannot get it to turn back ON?

@medida This works for me:
Network_Register((uint8_t[]){0,0,0,0,0,0}, NETWORK_REGISTER_MODE_MANUAL);
However, there is a bug (again) where if you connect to GPRS, then deregister from the network and register again, GPRS will no longer be able to connect and might crash the module. Only restarting the module fixes it PM_Restart();.

thanks - i'll give that ago.
I really want to be able to put the device to sleep - turn off the modem / gps to save as much power as possible... but if i have to reboot i would have to save any settings and re-read them... therfore taking more time.

The lowest current consumption I can get the module down to is 1.6mA @ 4V (the datasheet says 1.46mA, so not too far off). To get any lower you will have to shutdown the module, which for me gives ~240uA (datasheet says 216uA), which is still kinda high, and you'll loose settings and things, and it can't automatically turn itself back on.
If want super low power down current you will need a microcontroller that can completely remove power from the module by controlling a P-MOSFET and communicate over I2C or something to store and retrieve settings.
An external microcontroller is a good idea anyway, as the SDK is super buggy and likely to crash or get stuck, so you'll need the controller to make sure the module is still ok and to reset it when it dies.

I just found that setting GPIO_PIN31, GPIO_PIN32, GPIO_PIN33 and GPIO_PIN34 to OUTPUT LOW reduces current by around 0.4mA, so now the lowest current draw while the module is on is 1.2mA.

I have found i can reconnect the network by calling the following code.
For some reason - if i try to connect with the wrong APN - then the correct one - it works!
Network_PDP_Context_t context = {
                .apn        ="somethingwrongtomakeitfail",
                .userName   = "",
                .userPasswd = ""
            };
            Network_StartActive(context);

            Network_PDP_Context_t context2 = {
                .apn        ="mobiledata",
                .userName   = "",
                .userPasswd = ""
            };

            Network_StartActive(context2);
           //   Network_SetFlightMode(false); // worked when here
            OS_Sleep(3000);

Thanks for this @medida! I gave it a quick go and it seems to work!
General flow of things is:

Connect to GPRS with correct APN settings (Network_StartActive(context))
Wait for connection (API_EVENT_ID_NETWORK_ACTIVATED)
Disconnect from GPRS (Network_Deactive(1)) (I'm not sure what the context ID is supposed to be, I just put it in a 0 - 10 loop and ID 1 is usually the one that works)
Wait for disconnect (API_EVENT_ID_NETWORK_DEACTIVED)
Connect to GPRS with incorrect APN settings (Network_StartActive(contextBad))
Wait for connection failure (API_EVENT_ID_NETWORK_ACTIVATE_FAILED)
Connect to GPRS with correct APN settings (Network_StartActive(context))
Wait for connection (API_EVENT_ID_NETWORK_ACTIVATED)
Done!

I just found that setting GPIO_PIN31, GPIO_PIN32, GPIO_PIN33 and GPIO_PIN34 to OUTPUT LOW reduces current by around 0.4mA, so now the lowest current draw while the module is on is 1.2mA.

thanks for this info,
the lowest current after:

removing gps antenna and close gps
go to flightmode
set cpu to 32khz
close uart1

is 4mA
what did you do to get that 1.2mA !

I have no idea an more - i abandoned the product sometime ago as it was not working as i wanted and the project it was inteneded for stopped as well.


CVAT will not let me upload any type of annotation

My actions before raising this issue

[x] Read/searched the docs
[x] Searched past issues

I have tried multiple different formats and I keep getting an error telling me to check the browser console every time I try to upload annotations.
Expected Behaviour

For testing purposes, I want it so that I can export annotations from CVAT, delete those annotations, then import them back.
Current Behaviour

CVAT will not let me upload.
Possible Solution

Steps to Reproduce (for bugs)

Annotate 12 files in CVAT
Export the annotations
Delete annotations in the CVAT UI
Import exported annotations back into task

Context

Your Environment

Git hash commit (git log -1):
Docker version docker version (e.g. Docker 17.0.05):
Are you using Docker Swarm or Kubernetes?
Operating System and version (e.g. Linux, Windows, MacOS):
Code example or link to GitHub repo or gist to reproduce problem:
Other diagnostic information / logs:
Logs from `cvat` container

Next steps
You may join our Gitter channel for community support.

Hi, I am not sure the issue is still relevant.
So, have you tried to do logout/login?
If it is not the first time you use CVAT, was you able to dump annotations before?
Could you please send us logs from cvat container? (docker logs cvat)

I will close the issue for now.
Do not hesitate to reopen if you have any updates.

@bsekachev could you help solve this #5653?


Pin chartpress to 1.0.3

A new version of chartpress was released and now our chart publishing job thinks that it does not need to publish anything. More details in https://github.com/henchbot/mybinder.org-upgrades/issues/7#issuecomment-766126450
Let's merge this and see if this brings the chart publishing back to life. If not we know it wasn't the chartpress release, if yes we can use the old version of chartpress until there is a fix.
It might make sense to keep the chartpress version pinned for a "long time" and rely on explicit upgrades of it so that we don't have to investigate why our machinery has started failing. In particular as there were no problems with the old chartpress version from a BinderHub point of view.

@betatim the issue is the chartpress.yaml config of binderhub isn't correct I think, it did not consider a change of the webpack.config file to be relevant in order to trigger a version bump.
I'd like to have this unpinned still because if there is a time when we want a new feature, we suddenly must consider why this was pinned etc. And at this point, it was a configuration error unrelated to the version bump afaik.

@betatim the issue is the chartpress.yaml config of binderhub isn't correct I think, it did not consider a change of the webpack.config file to be relevant in order to trigger a version bump.
I'd like to have this unpinned still because if there is a time when we want a new feature, we suddenly must consider why this was pinned etc. And at this point, it was a configuration error unrelated to the version bump afaik.

If #1250 fixes it then that is good and we don't need to pin chartpress.

If #1250 fixes it then that is good and we don't need to pin chartpress.

#1250 fixed it

#1250 fixed it


warp-terminal: 0.2<IP_ADDRESS>.02.stable_04 -> 0.2<IP_ADDRESS>.02.stable_00

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for warp-terminal is: Rust-based terminal
meta.homepage for warp-terminal is: https://www.warp.dev
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2<IP_ADDRESS>.02.stable_00 with grep in /nix/store/prd9l2qdzg5m2swdzxchh4rlaa7bz1y4-warp-terminal-0.2<IP_ADDRESS>.02.stable_00
found 0.2<IP_ADDRESS>.02.stable_00 in filename of file in /nix/store/prd9l2qdzg5m2swdzxchh4rlaa7bz1y4-warp-terminal-0.2<IP_ADDRESS>.02.stable_00

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

warp-terminal

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/prd9l2qdzg5m2swdzxchh4rlaa7bz1y4-warp-terminal-0.2<IP_ADDRESS>.02.stable_00 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A warp-terminal https://github.com/r-ryantm/nixpkgs/archive/51265569687ef6af8e82f6af608afee5942277b5.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/51265569687ef6af8e82f6af608afee5942277b5#warp-terminal

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/prd9l2qdzg5m2swdzxchh4rlaa7bz1y4-warp-terminal-0.2<IP_ADDRESS>.02.stable_00
ls -la /nix/store/prd9l2qdzg5m2swdzxchh4rlaa7bz1y4-warp-terminal-0.2<IP_ADDRESS>.02.stable_00/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

warp-terminal

Maintainer pings
cc @emilytrau @imadnyc @DontEatOreo @johnrtitor for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 337690 run on aarch64-darwin 1

1 package built:

warp-terminal

@NixOS/nixpkgs-merge-bot merge


New version: MLJScikitLearnInterface v0.3.0

Registering package: MLJScikitLearnInterface
Repository: https://github.com/JuliaAI/MLJScikitLearnInterface.jl
Created by: @ablaom
Version: v0.3.0
Commit: 075403ee06254aad114a0d39374940b07a9014ef
Reviewed by: @ablaom
Reference: https://github.com/JuliaAI/MLJScikitLearnInterface.jl/issues/7#issuecomment-1447500757
Description: MLJ Interface for ScikitLearn.jl
Release notes:

Major update to support Python's scikitlearn 1.2 (#50) @OkonSamuel

[noblock]
Please close. We want to delay the breaking release.

Please do not merge.

CI/CI 1.7 has hung, which prevented auto merge from proceeding. Fortunately for us, we prefer to close this PR, as we want to amend our breaking release.
Can we simply try again with the amended release?
@DilumAluthge

Thank you @DilumAluthge


Teams policy 5.1 needs to be implemented in the Rego code

💡 Summary
As part of recent Teams SCB updates, policy MS.TEAMS.5.1v1 has been updated The earlier policy was checking for the agency to   allow all apps published by Microsoft. In the new Teams SCB's, this has been updated to only allow installation of Microsoft Apps approved by the agency. This issue is to update the code to be consistent with the new policy.
Motivation and context
This is to make the Rego code align with the current SCB policy.
Implementation notes
Update the rego code for existing MS.TEAMS.5.1v1 to now check the Global (org-wide default) policy to be set to "Allow Specific apps and block all others". The following more restrictive options should also produce a "PASS" results, "Block all apps"
Acceptance criteria

[ ] The ScubaGear report for Teams should have new language aligning to new MS.TEAMS.5.1v1 policy.
[ ] MS.TEAMS.5.1v1 should produce a pass for the following criteria:

[ ] Microsoft apps permissions set to "Allow Specific apps and block all others" and "Block all apps"

done as #588 is merged


Highlighter no longer working in 4.0.0.alpha.32

https://playground.babylonjs.com/index.html#AWGQWM
expected that it should highlight!
:)

cc @TrevorDev cause it is related to the latest multiview changes:
Effect.ts line 654:
// Add multiview setup to top of file when defined
var hasMultiviewExtension = this.defines.indexOf("#define MULTIVIEW\n") !== -1;

We ll fix it ASAP thanks for reporting

Actually the fixes required are:

[ ] Fix the bug when there is no multiview ( I will do it as it breaks the highlightlayer)
[ ] Check if highlightlayer and glowlayer to see if they can work in multiview mode


Add managed reactor-core and reactor-test to catalog

When switching microstream to the latest build for micronaut 4.0.0, it raised an issue
https://github.com/micronaut-projects/micronaut-microstream/pull/144
We could pull reactor-core from the core-bom, but as it didn't contain reactor-test we had to jump through hoops.
In https://github.com/micronaut-projects/micronaut-reactor/pull/248 we swiched to using the reactor bom instead of managed dependencies.
But as the reactor-bom hard-codes versions instead of setting them as properties we need to specify the version of reactor-core and reactor-test that we want to include.
The downside here is we will need to keep them in sync whenever the BOM version gets updated

@melix would we be better just adding these as a managed-reactor version and two managed libraries in the catalog itself?  Then they're both together which might make keeping in sync easier?

@melix would we be better just adding these as a managed-reactor version and two managed libraries in the catalog itself? Then they're both together which might make keeping in sync easier?

Yes it probably makes sense to do this. An alternative would be to add logic to parse the reactor BOM file and infer the version from there. Doable but I'm not sure I want to go that route.

@melix done, ptal 👍

Urgh...3.5.0 has some breaking changes

mmm, well this is an interesting question. core actually needs reactor. There are many places where we use implementation libs.managed.reactor. Which means that it should be a managed dependency...... of core! If we change core to use this BOM instead, it would be semantically more correct, but it raises the question of why core has dependencies on reactor! More problematic, it would create a circular dependency.
So maybe, after all, we should have the managed version of reactor in core as it is today, and add the missing test dependency in core (with a comment so that we understand why it's not used in core directly).

Done in https://github.com/micronaut-projects/micronaut-core/pull/8470


Use of private key to encrypt

It seems this library uses the private key to encrypt data. Shouldn't this be happening with the public key or am I missing something?

I agree that this is unusual. any thoughts?

@munsanje one workaround is for the holder of the private key to simply publish encryptions of 0 and 1 (and identify them as such). since these are the only plaintexts, they can be re-used by external parties.


Documentation

Will need documentation to show how to use the library

Docs have been added to the readme file. Addressed in 3f77e098ee97099fa5467ce75cc30354e22ed0a8


[FIX] Added fix for ml input buttons

This PR disables the default button styles for .ml-btn and replaces it with a simple darken on hover.
Original:

New:

New on hover:

It looks more obvious in practice than the screenshot suggests :)

Also added support for variable length language codes. This is really simply done by removing the fixed with of the element so it can scale as needed


Backend

Dotnet 8.0 api

Api operations handled.
Postgres database used.

Api operations handled.
Postgres database used.


Why not Wire BotAPI?

Why would you run this bot as a user and not as a bot?

We changed it to use @wireapp/bot-api.


[ Bug ] Docker image on Dockerhub not updated for 4 months

The docker image on Dockerhub hasn't been updated for 4 months.
Given that the latest release is ~3 weeks old, there may be an issue with the github action.

Indeed https://github.com/excalidraw/excalidraw/actions/runs/7117050046/job/19376898412#step:4:251

Does anyone know whether the docker image can now be used with several participants at the same time? The official site still says "At the moment, self-hosting your own instance doesn't support sharing or collaboration features".

Any update on this since? It's now almost 7 months since the image was updated: https://hub.docker.com/r/excalidraw/excalidraw/tags
Even though the last release was 3 days ago: https://github.com/excalidraw/excalidraw/releases

Any update on this? It's now almost 7 months since the image was updated: https://hub.docker.com/r/excalidraw/excalidraw/tags
Even though the last release was 3 days ago: https://github.com/excalidraw/excalidraw/releases

Waiting for someone to test out the docker build in https://github.com/excalidraw/excalidraw/pull/7430 and then we should be good to merge

Can confirm the build https://excalidraw-git-fork-agebhar1-feature-gh7403-f-1f6410-excalidraw.vercel.app/

Any update on this? The image on Docker Hub is now 8 months behind since last update.


Discussions

How would you feel about enabling Discussions on this repo? I have some questions that I think would benefit from being in a place where anyone can read them and see the answers, instead of in private conversation or an issue that would be closed eventually

I think this is a wonderful idea, so I've gone ahead and enabled discussions for this repo.


Fix "error":"case_clause" using latest=true and an old revision with conflicting children as rev

Closes #3362.

Currently built on SM86 and Erlang23 fixes, but can be merged after.


Adding SSL support for OVN

do we really need to upgrade vendor?

I think it would be good one time thing since there are couple of changes in latest vendor codes. I can submit that as a seperate patch. @hzhou8 are you fine with vendor edit or want to upgrade vendor?

@noah8713 the vendor socketplane/libovsdb hasn't been updated for 2 years. We were update to date since day 1 and even have bug fixes on top of that. The files you added were deleted on purpose by @sudhi-vm 's commit (https://github.com/eBay/go-ovn/commit/340f8e3d3c26e15381cdd369ee9a0c02b374be02). If you think that is a mistake please confirm with @sudhi-vm and fix in separate commit but don't mix with this one.

Yes, you can make changes in vendor, but one commit for one change. Please don't mix with anything unrelated.

@hzhou8 as discussed,   as maintenance of upstream repo socketplane is unknown, I have edited the vendor files directly for ssl changes. PTAL.

May be fork it and use directly from ebay?
ср, 13 мар. 2019 г., 20:27 aginwala<EMAIL_ADDRESS>
@hzhou8 https://github.com/hzhou8 as discussed, as maintenance of
upstream repo socketplane is unknown, I have edited the vendor files
directly for ssl changes. PTAL.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/eBay/go-ovn/pull/17#issuecomment-472525047, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAdYG9oHv0UBerPHPANnjRI2EOZwbfkSks5vWTUWgaJpZM4brpw4
.

LGTM


Engineering Inspiration skill increases from knowledge are not taken into account in Simulation Mode

Not sure if this affects all professions or just Engineering, but I noticed in the latest version (7.6.0) at least Simulation Mode is only showing base inspiration skill.

As you can see in the screenshot on this recipe I gain 100 skill from inspiration but CraftSim has my Inspiration skill at just 66.7.
Same thing here where I gain 80 skill from inspiration but CraftSim has me at 53.3:

I have 20% bonus skill from Optimized Efficiency 40/40:

And I have 30% bonus skill from Novelties 30/30:

Thank you for maintaining this addon!

thx for the report, I recently change the spec data parsing ^^ so this might have affected this.. I will take a look :)

Thanks for the update!
With 7.7 the 20% bonus skill from Optimize Efficiency is taken into account but the 1-30% bonus skill from Novelties is still ignored.

Thanks for the update!
With 7.7 the 20% bonus skill from Optimized Efficiency is taken into account but the 1-30% bonus skill from Novelties is still ignored.

Should be fixed with the updated Engineering spec data


Add an is_resizable trait?

It'd be nice if there was an abstract array interface specifying whether an array is resizable in-place.

Agreed.

Is it worth distighishing this into:
isgrowable for things that can increase size (like Vector)
and
isresizable that can just shuffle the size around into different dimensions?
Or maybe it is a trait with multiple values:

Growable() > Resizable > FixedSize ?

Really i feel like we want to answer questions about:

hasmethod(resize!, ...)
hasmethod(push!, ...)  (and maybe its flavours pushfirst!)
hasmethod(deleteat, ...) (and maybe its flavors pop!, popfirst, filter!)
maybe hasmethod(reshape,...) ?

If hasmethod is a compile-time "trait", then do we still need a trait for this?

Yes, to group them. And to name them. But anyway Jameson says my plan is impossible.

I created something similar for this here https://github.com/Tokazama/StaticRanges.jl/blob/f8dac65496c10ea0ec0fbf748006af105f77f62e/src/staticness.jl#L1.
It separates things into dynamic, static, fixed. Where a Vector is dynamic, a typical range is fixed, and SVector is static. Would this be a good solution to this?

an isresizable trait function is all that's needed.

FYI, BangBang internally now has implements(f!, x) and possible(f!, args...) to query the capability of containers/objects.  ATM only implements(setindex!, x), implements(push!, x),  and implements(setproperty!, x) are defined.  They are like hasmethod but manually implemented (like @oxinabox mentioned https://github.com/JuliaDiffEq/ArrayInterface.jl/issues/22#issuecomment-518005555).  I also need to consider "element" types so that's why there is possible function that can do:
julia> possible(push!, Int[], 1)
true

julia> possible(push!, Int[], 0.5)
false

At some point, I'd like to pull this out of BangBang.jl as a separate library but I also need to deal with non-array containers and objects.  So, I don't see ArrayInterface.jl as a good target.  It would be great if the part of ArrayInterface.jl that is not array-specific can be factored out.

Is this compile time thing or is it all checked at run time?
Sorry if you document it elsewhere but I only read what you linked.

It's all type-level so everything should be done at compile time.

If I understand your code correctly you're using a union internally to group mutable and immutable containers. Wouldn't we want to use traits to avoid having to redefine the union type for every new type that needs this behavior?

The Union is just an implementation detail to minimize code duplication. implements is just yet another trait system. You can add anything afterwards. That's how static arrays are treated.

Okay, maybe I misunderstood the suggestion. Is your suggestion that instead of defining is_resizeable
to have define something like:
implements(::typeof(resize!), ::Type{<:Array}) = true

I'm just mentioning that I explored the API @oxinabox mentioned above https://github.com/JuliaDiffEq/ArrayInterface.jl/issues/22#issuecomment-518005555.  (I'm also suggesting to create a new package (upstream to ArrayInterface.jl) that can handle arbitrary containers and objects, not just arrays. But that's not the main topic here.)

Perhaps I should open a separate issue for this, but it would also be useful to have grow_first, grow_last, shrink_first, shrink_last. However, I don't know any of the magic behind reserving chunks of memory that this would require for arrays.


HBASE-23635 Reduced number of Checkstyle violations in hbase-mapreduce

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

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

_ Prechecks _

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

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

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

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

_ master Compile Tests _

+1 :green_heart:
mvninstall
5m 57s
master passed

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

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

+1 :green_heart:
shadedjars
5m  1s
branch has no errors when building our shaded downstream artifacts.

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

+0 :ok:
spotbugs
0m 51s
Used deprecated FindBugs config; considering switching to SpotBugs.

+1 :green_heart:
findbugs
0m 50s
master passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
5m 27s
the patch passed

+1 :green_heart:
compile
0m 26s
the patch passed

+1 :green_heart:
javac
0m 26s
the patch passed

+1 :green_heart:
checkstyle
0m 19s
hbase-mapreduce: The patch generated 0 new + 0 unchanged - 35 fixed = 0 total (was 35)

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

+1 :green_heart:
shadedjars
5m  5s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
hadoopcheck
17m 18s
Patch does not cause any errors with Hadoop 2.8.5 2.9.2 or 3.1.2.

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

+1 :green_heart:
findbugs
0m 56s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
16m 46s
hbase-mapreduce in the patch passed.

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

67m 46s

Subsystem
Report/Notes

Docker
Client=19.03.5 Server=19.03.5 base: https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-981/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/981

Optional Tests
dupname asflicense javac javadoc unit spotbugs findbugs shadedjars hadoopcheck hbaseanti checkstyle compile

uname
Linux 53360e04e69d 4.15.0-66-generic #75-Ubuntu SMP Tue Oct 1 05:24:09 UTC 2019 x86_64 GNU/Linux

Build tool
maven

Personality
/home/jenkins/jenkins-slave/workspace/HBase-PreCommit-GitHub-PR_PR-981/out/precommit/personality/provided.sh

git revision
master / 85ecdd10a9

Default Java
1.8.0_181

Test Results
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-981/1/testReport/

Max. process+thread count
5262 (vs. ulimit of 10000)

modules
C: hbase-mapreduce U: hbase-mapreduce

Console output
https://builds.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-981/1/console

versions
git=2.11.0 maven=2018-06-17T18:33:14Z) findbugs=3.1.11

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

This message was automatically generated.


Correct display of BulkImport duration in monitor

The formatting for monitor display accepts milliseconds. This
fix corrects issue where seconds (duration / 1000) were used,
causing the duration displayed by the monitor to format and display
incorrectly.

@EdColeman A few helpful things for using GitHub:

There is no need to create an issue and a PR if you are creating them at the same time. GitHub PRs are their own issue. Sometimes it's useful to create an issue for future work.
It is not necessary to close a PR and create a new one to change the branch. You can click the "Edit" button and select a different branch to do the PR against from the drop-down.
When you use the form "ACCUMULO-NNN", it automatically links against the JIRA issue with that ID, and that JIRA issue (such as https://issues.apache.org/jira/browse/ACCUMULO-551) may be completely unrelated. GitHub issues are mentioned using the pound syntax: #551

Do you know if a similar issue affects the 2.0 monitor?

The code for 2.0 has changed such that I do not see the same issue - but I'll need to dig deeper to find the actual display code - a quick look and it seems that milliseconds are being passed around.


Resolve risk - albeit in a test

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 95.166%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6653

Relevant Lines:
6772

💛  - Coveralls


mopidy-iris: 3.31.1 -> 3.31.2

Motivation for this change
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Failure on x86_64-linux (full log)
Attempted: mopidy-iris
Partial log (click to expand)
patching script interpreter paths in /nix/store/mbf920qmnj6njqgnk9m74np0pjmbisg5-Mopidy-Iris-3.31.2
/nix/store/mbf920qmnj6njqgnk9m74np0pjmbisg5-Mopidy-Iris-3.31.2/lib/python2.7/site-packages/mopidy_iris/system.sh: interpreter directive changed from "/bin/bash" to "/nix/store/yal8k8rgbaj8p9vi08q1kh21ls0ndq95-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/mbf920qmnj6njqgnk9m74np0pjmbisg5-Mopidy-Iris-3.31.2...
Found duplicated packages in closure for dependency 'tornado':
  tornado 4.5.3 (/nix/store/kmfa8k4pdslpqrwhjdv7m09ixfkrryr5-python2.7-tornado-4.5.3/lib/python2.7/site-packages)
  tornado 5.1 (/nix/store/865yhm74nfj4cr7kpzfqjgzpsbg3c720-python2.7-tornado-5.1/lib/python2.7/site-packages)

Package duplicates found in closure, see above. Usually this happens if two packages depend on different version of the same dependency.
builder for '/nix/store/sjj3kwkbrgp0iyvgy8g0j4c7iw7iadpi-Mopidy-Iris-3.31.2.drv' failed with exit code 1
error: build of '/nix/store/sjj3kwkbrgp0iyvgy8g0j4c7iw7iadpi-Mopidy-Iris-3.31.2.drv' failed

Failure on aarch64-linux (full log)
Attempted: mopidy-iris
Partial log (click to expand)
patching script interpreter paths in /nix/store/rsrr2vjbsbp4lfd4i3rz6fkw9nd2cxan-Mopidy-Iris-3.31.2
/nix/store/rsrr2vjbsbp4lfd4i3rz6fkw9nd2cxan-Mopidy-Iris-3.31.2/lib/python2.7/site-packages/mopidy_iris/system.sh: interpreter directive changed from "/bin/bash" to "/nix/store/bpspdmsl0yys24gs70flsvcw3wcnl7lx-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/rsrr2vjbsbp4lfd4i3rz6fkw9nd2cxan-Mopidy-Iris-3.31.2...
Found duplicated packages in closure for dependency 'tornado':
  tornado 4.5.3 (/nix/store/128iihgrjqlqgjday6kq3hhbn4a9lhax-python2.7-tornado-4.5.3/lib/python2.7/site-packages)
  tornado 5.1 (/nix/store/6nf980qymn2dhmzk5w3xgxfgxliwl849-python2.7-tornado-5.1/lib/python2.7/site-packages)

Package duplicates found in closure, see above. Usually this happens if two packages depend on different version of the same dependency.
builder for '/nix/store/3cv5whwp3dwm7j37c8iwxy3hcbbhvwms-Mopidy-Iris-3.31.2.drv' failed with exit code 1
error: build of '/nix/store/3cv5whwp3dwm7j37c8iwxy3hcbbhvwms-Mopidy-Iris-3.31.2.drv' failed

Found duplicated packages in closure for dependency 'tornado':
  tornado 4.5.3 (/nix/store/kmfa8k4pdslpqrwhjdv7m09ixfkrryr5-python2.7-tornado-4.5.3/lib/python2.7/site-packages)
  tornado 5.1 (/nix/store/865yhm74nfj4cr7kpzfqjgzpsbg3c720-python2.7-tornado-5.1/lib/python2.7/site-packages)

Package duplicates found in closure, see above. Usually this happens if two packages depend on different version of the same dependency.

Mopidy is using tornado_4 instead of tornado so we just need to change it here.

Fixed it

@GrahamcOfBorg build mopidy-iris

Success on x86_64-linux (full log)
Attempted: mopidy-iris
Partial log (click to expand)
/build/Mopidy-Iris-3.31.2
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/24b4f3gby6bfjiyl6ilxhifqzmwflz7d-Mopidy-Iris-3.31.2
strip is /nix/store/n4hb93w6j076xcjw5pm09rdmc09s075b-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/24b4f3gby6bfjiyl6ilxhifqzmwflz7d-Mopidy-Iris-3.31.2/lib
patching script interpreter paths in /nix/store/24b4f3gby6bfjiyl6ilxhifqzmwflz7d-Mopidy-Iris-3.31.2
/nix/store/24b4f3gby6bfjiyl6ilxhifqzmwflz7d-Mopidy-Iris-3.31.2/lib/python2.7/site-packages/mopidy_iris/system.sh: interpreter directive changed from "/bin/bash" to "/nix/store/yal8k8rgbaj8p9vi08q1kh21ls0ndq95-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/24b4f3gby6bfjiyl6ilxhifqzmwflz7d-Mopidy-Iris-3.31.2...
/nix/store/24b4f3gby6bfjiyl6ilxhifqzmwflz7d-Mopidy-Iris-3.31.2

Success on aarch64-linux (full log)
Attempted: mopidy-iris
Partial log (click to expand)
/build/Mopidy-Iris-3.31.2
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/sgyh1nnvdl5xwfpn8ig8is7vv4c5d6mm-Mopidy-Iris-3.31.2
strip is /nix/store/qjrnv0qw44bw1hc23zhfh26xd1c25dfs-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/sgyh1nnvdl5xwfpn8ig8is7vv4c5d6mm-Mopidy-Iris-3.31.2/lib
patching script interpreter paths in /nix/store/sgyh1nnvdl5xwfpn8ig8is7vv4c5d6mm-Mopidy-Iris-3.31.2
/nix/store/sgyh1nnvdl5xwfpn8ig8is7vv4c5d6mm-Mopidy-Iris-3.31.2/lib/python2.7/site-packages/mopidy_iris/system.sh: interpreter directive changed from "/bin/bash" to "/nix/store/bpspdmsl0yys24gs70flsvcw3wcnl7lx-bash-4.4-p23/bin/bash"
checking for references to /build in /nix/store/sgyh1nnvdl5xwfpn8ig8is7vv4c5d6mm-Mopidy-Iris-3.31.2...
/nix/store/sgyh1nnvdl5xwfpn8ig8is7vv4c5d6mm-Mopidy-Iris-3.31.2


[CWS][BACKPORT] do not try to open non regular file

(cherry picked from commit 3769e51cd5840630d0023d1fa07a75ba7473d300)

What does this PR do?
stat before opening the file to ensure that the file is a regular file, otherwise it can block the open syscall and generate a deadlock.

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/merge


ImageCircle does not render as a circle in release mode for UWP.

Bug Information
When compiled under release conditions, our UWP program does not render the ImageCircle control as a circle.  Rather, it appears as a square image.  It appears correctly under debug conditions, as well as in Android.
Version Number of Plugin:
v<IP_ADDRESS>-beta
Device Tested On:
Windows 10 with vs2017, Lumia 950XL (latest versions of OS)
Simulator Tested On:
Mobile Emulator 10.0.15063.0
Version of VS:
VS Pro 2017 15.4.4
Version of Xamarin:
<IP_ADDRESS>635
XAML
    <Image x:Name="imgBusy" Grid.Row="1" Source="activity.png" HorizontalOptions="Center" VerticalOptions="Center" WidthRequest="100" HeightRequest="100"/>
    <controls:CircleImage x:Name="imgAudio" Grid.Row="1" HorizontalOptions="Center" VerticalOptions="Center">
        <controls:CircleImage.WidthRequest>
            <OnPlatform x:TypeArguments="x:Double"
                iOS="75"
                Android="75"
                WinPhone="75"/>
        </controls:CircleImage.WidthRequest>
        <controls:CircleImage.HeightRequest>
            <OnPlatform x:TypeArguments="x:Double"
                iOS="75"
                Android="75"
                WinPhone="75"/>
        </controls:CircleImage.HeightRequest>
    </controls:CircleImage>

``

You must add
var rendererAssemblies = new[]
{
    typeof(ImageCircleRenderer).GetTypeInfo().Assembly
};
Xamarin.Forms.Forms.Init(e, rendererAssemblies);

That got it.  Not sure how I missed that at the bottom.  Thanks a million.


Update publishing plugin docs: include env vars.

Fixes #1246
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

💅 @ov7a this could also target release?


Idea: Have pelican invoke Docker

Watching @aowen-uwmad's tutorial during HTC24, one idea came to me.  Origins are currently started with the following command:
docker run --rm -it \
    -p 8444:8444 -p 8443:8443 \
    -v $(pwd)/config/issuer.jwk:/etc/pelican/issuer.jwk \
    -v $(pwd)/config/issuer-pub.jwks:/etc/pelican/issuer-pub.jwks \
    -v /etc/hostcert.pem:/etc/hostcert.pem \
    -v /etc/hostkey.pem:/etc/hostkey.pem \
    -v $(pwd)/config/pelican.yaml:/etc/pelican/pelican.yaml \
    -v $(pwd)/data:/data \
    hub.opensciencegrid.org/pelican_platform/origin:v7.9.2 \
    serve -p 8444

That's long and ugly enough that we ended up writing a small shell script (start-origin.sh in this repo) to avoid having users type this out.
Pelican, in principle, could parse its configuration file to determine the ports, the certificates, the data mounts, etc.  We could have:
pelican origin serve --docker

be a shortcut for the above CLI invocation.

Ideally the above Docker command would just mount /etc or /etc/pelican to shorten things up. But Pelican (or maybe more accurately XRootD) creates root-owned files/directories that cannot be removed by the user once the Docker container is exited. If things need to be cleaned up or reset, the user needs to either have sudo or (not tested) needs to run another Docker container with the mounts in order to remove those files.
I think this is a good idea, but will need to be careful about permissions for folks who have Docker access but not root access.

Yeah - I think that's surmountable.  One could invoke the container as the current user, then bind mount all the mutable directories (run, logs) to corresponding directories inside the user's home.

Going to close this for now -- it's a good idea but, unless we find a strong motivator for it, probably not on the current workplan's radar.


test: Avoid pgrep -q, not available with GNU pgrep

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

[x] Have you successfully run brew tests with your changes locally?

macOS pgrep has -q but Linux does not: https://linux.die.net/man/1/pgrep
From macOS: -q Do not write anything to standard output.
pgrep >/dev/null would be more portable.

Thanks @sjackman!


Some illustrations don't stretch completely to the borders

Description
change fill mode to aspect fill
Link to Jira
(https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-8482)

@pascal-brause  I tested on xr and iPhone 13 pro max and they look ok


Feature request: Show banners defined by ConsoleNotification CR also in CLI

Hi,
I love the ConsoleNotification feature where custom banners are presented to the users (can do a 'We'll do maintenance on this cluster in two weeks' stuff).
Would it be possible to also show a banner after logging in to a console on the terminal with the CLI oc or kubectl ? It would be useful to notify our console hackers about maintanance tasks :-)
> oc login ...

Message: The cluster okd4.c1.dev will get planned maintenance in two weeks.

>

Greetings,
Josef

Hi Josef,
could I ask you to open an issue against www.github.com/openshift/oc for this?
Thank you !

Hi Christian,
thanks for your feedback. Is done.
Greetings,
Josef

Thank you @jomeier!
xref: https://github.com/openshift/oc/issues/344


Error on startup of first example

510  git clone https://github.com/gluon-framework/examples.git
511  ll
512  cd examples
513  ll
514  cd gluworld
515  npm install
516  node .
517  h
dean@pop-os ~/.../examples/gluworld (main) $ node .
[Gluon] starting browser...
file:///home/dean/projects/gatsby/LIBRARY/gluon/examples/gluworld/node_modules/@gluon-framework/gluon/src/index.js:198
const [ browserPath, browserName ] = await findBrowserPath(forceBrowser, forceEngine);
^
TypeError: (intermediate value) is not iterable
at startBrowser (file:///home/dean/projects/gatsby/LIBRARY/gluon/examples/gluworld/node_modules/@gluon-framework/gluon/src/index.js:198:40)
at async Module.open (file:///home/dean/projects/gatsby/LIBRARY/gluon/examples/gluworld/node_modules/@gluon-framework/gluon/src/index.js:255:19)
at async file:///home/dean/projects/gatsby/LIBRARY/gluon/examples/gluworld/index.js:17:3
Node.js v18.12.1
dean@pop-os ~/.../examples/gluworld (main) $
Happy to help any way I can.

What browsers do you have installed on your system?

chrome and firefox

I'm using !Pop_os so it's a flatpak install

I can run it from the command line with this command:
flatpak run com.google.Chrome

of course, I can also use
/usr/bin/flatpak run com.google.Chrome


MMCA-4738 | Amend Duty Deferment Confirmation Page

Hey @Neozxc according to the ticket shouldn't we not add this to the translation files

We'll send an email within 48 hours to<EMAIL_ADDRESS>when your request is processed.

Hi @wg-hmrc this translation already lives on the main branch as i've merged main into this branch. In this branch i've only updated cf.historic.document.request.confirmation.body-text2.DutyDefermentStatement & cf.historic.document.request.confirmation.DutyDefermentStatement.link-text which are todo's on my ticket.


[DO NOT SUBMIT] gRPC streaming in speech

Initial draft, for discussion.

This is now obsolete - closing and deleting.


ATM_CARD_VALIDATOR Program in Java

Added A program in java to Validate a variety of identification numbers, such as credit card and debit card numbers, IMEI numbers etc.

Kindly look into it and requesting you to kindly merge it also.


Add the ability to set locale environment variables

Allow setting locale environment variables $LC_ALL and $LANG from prezto runcom.

Hi Prezto team. Is there a particular reason why this trivial contribution has not been merged yet?

In general LC_ALL should not be set because it overrides everything. Setting LANG should be enough... And if that's the case, it seems like that could just go in the runcoms directly

As it has been multiple months I don't remember exactly now, but I had to explicitly set LC_ALL as well to avoid Python locale error: unsupported locale setting on my setup. We can remove it from the condition and perhaps add a zstyle to set LC_* separately, probably somewhat like this:
zstyle ':prezto:environment:language' all 'en_US.UTF-8'
zstyle -s ':prezto: environment:language' all 'LC_ALL'

What do you think?

Yeah, the language settings can be confusing.
The locale command can be used to debug how the system interprets the environment variables (though some languages may not respect all these). My guess is that the language for your system was set to something that didn't exist. There's a ton of background here if you're interested in details.
Also, I played around with it a bit - python's getlocale isn't as useful as I'd like because it doesn't respect the LANG variable and just directly checks LC_ALL (by default). It looks like getdefaultlocale is more useful (if you're writing your own programs) as it will properly respect the locale variables.
There's also some background info in those docs which state "According to POSIX, a program which has not called setlocale(LC_ALL, '') runs using the portable 'C' locale. Calling setlocale(LC_ALL, '') lets it use the default locale as defined by the LANG variable. Since we do not want to interfere with the current locale setting we thus emulate the behavior in the way described above.
Thanks for submitting this, but I'd still prefer to leave this out (at least the portion relating to LC_ALL because it overrides other language settings)... and this seems like it would be better solved by simply setting these in the runcoms directly if you need them.

@belak's comments make sense to me, shall we close this?
@N4M3Z what do you think? If you are aware of more facts re: language settings, I and the other maintainers would be happy to hear them and reconsider, aside from @belak I suspect not many of us know very much about these...

I don't understand what's the problem with having LANG  and LC_ALL optionally setable by the runcom using the zstyle convention. We can simply remove the default:
zstyle ':prezto:environment' language 'en_US.UTF-8'

and then everything being optional, I don't see the harm.

Setting LC_ALL isn't a great practice to get into because it overrides a ton of locale variables, even the default environment in which programs start. Many people use this because it allows you to force the locale to a specific setting, but it's more of a blunt instrument. This can be what's intended, but as many people assume a project like prezto will support best practices, I'd rather not provide a built-in way to override it.
Also, any user can set these variables in their zshrc or zprofile. Setting it using zstyle adds additional code (and this is additional code which needs to be maintained) without making the system more flexible or convenient.
I'm open to other suggestions if there's a strong reason that can be backed up with documentation and/or reasoning, or if another maintainer disagrees with me here, but my current opinion is that while this definitely can solve some issues, this generally fits better when set manually by the user.

As you wish, I'll just use my fork.

Out of curiosity, why do you view this as better than adding the export calls directly in zpreztorc or your zshrc?

It's just a preference. In my current setup I configure everything through zstyle, I've expanded the options to suit my needs where prezto hasn't implemented it.

FWIW, you don't need a fork to have this accessible. I'm with @belak on this.


Square Root Calculator is added

Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes:  #2176

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I have made this from my own
[ ] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

@Sulagna-Dutta-Roy please check the PR. I have added the manifest file


[nodejs,angular] stop adding local node_modules/.bin to path

Why

We were having issues where our local node_modules bin had a "which" binary that did not behave well.
It isn't typical for local node_modules/.bin to be on the path, if you want something on the path, you typically use npm install --global, which is still added to the PATH after this change.

What changed

Remove $REPL_HOME/node_modules/.bin from PATH

Test plan

Check template tester still passes
Check PATH does not contain that path in nodejs Repls

Rollout
Describe any procedures or requirements needed to roll this out safely (or check the box below)

[ ] This is fully backward and forward compatible

I don't get how "This is fully backward and forward compatible" though. Existing Repls might have scripts that rely on local NPM binaries being in the PATH, and this would break them. And even if it only applies to new Repls, the same could happen for GitHub imports, right? You could try to import someone's old repo that had such a script and it would break.
I might be wrong though, I'm still trying to wrap my head around Nix.

Yes, you're right, this is a breaking change. I updated the checkmark.
https://docs.npmjs.com/cli/v10/configuring-npm/folders says

Any bin files are symlinked to ./node_modules/.bin/, so that they may be found by npm scripts when necessary.

we're pretty sure it is not conventional to put this dir on the user's PATH and we have evidence of this breaking at least one user, so we're going to move forward with it.
People can mitigate this themselves by adding an env section to their .replit if they need this:
[env]
PATH="$PATH:$REPL_HOME/node_modules/.bin"


chore: remove unused variable

This removes a variable that wasn't being used (warnings were silenced because name was _all).

thread 'main' panicked at 'Cannot parse version.: Error("unexpected character '+' after major version number")', src/dfx/src/config/mod.rs:10:51

Don't think that's me

unexpected character '+' after major version number

AAARRGH not that again 🤦 that's definitively not you. let me see what I can do


First client request fails

The first time a client makes a request to the server, the server consistently errors,
Initializing hand3d
WARNING:tensorflow:From /home/nate/Documents/ASLive/webserver/hand3d/utils/general.py:221: calling argmax (from tensorflow.python.ops.math_ops) with dimension is deprecated and will be removed in a future version.
Instructions for updating:
Use the `axis` argument instead
WARNING:tensorflow:From /home/nate/Documents/ASLive/webserver/hand3d/utils/general.py:253: sparse_to_dense (from tensorflow.python.ops.sparse_ops) is deprecated and will be removed in a future version.
Instructions for updating:
Create a `tf.sparse.SparseTensor` and use `tf.sparse.to_dense` instead.
WebSocket DISCONNECT / [<IP_ADDRESS>:43440]
Loaded 32 variables from ./hand3d/weights/handsegnet-rhd.pickle
Loaded 102 variables from ./hand3d/weights/posenet3d-rhd-stb-slr-finetuned.pickle

Fixed with new conv2d model on conv2d branch


Update the BIDS validator page to match the new validator

https://bids-website.readthedocs.io/en/latest/tools/validator.html

done in #572


Validate Input to Include User Selections

Validate Input to Include User Selections
User Story
AS AN employee with access to sensitive data
I WANT to randomly generate a password that meets certain criteria
SO THAT I can create a strong password that provides greater security
Acceptance Criteria
GIVEN I need a new, secure password
WHEN I answer each prompt
THEN my input should be validated and at least one character type should be selected

A password is generated using the generatePassword function that includes all criteria selected by the user.


Add The Spice House to README's list of places

@theotherzach I notice it doesn't use the "card" display on mobile... were there issues, or just trying to save that space?


Change base currency

@kwunyeung provide the foreign exchange api

@calvinkei the forex API is from https://www.alphavantage.co/ . We did paid for premium subscription but it seems they don't have that option anymore. I guess the free API is enough for our usage.

Tested.


[test]  check PR 576415 for multi logs

Choose a PR Template
Switch to "Preview" on this description then select one of the choices below.
Click here to open a PR for a Data Plane API.
Click here to open a PR for a Control Plane (ARM) API.

/azp run

/azp run


Testing with $stateChangeStart

How does one test with $stateChangeStart or other emitted events?
I have the following code, which in essence checks that the user is logged in, if not, redirect to the app.login state
app.run(function ($rootScope, $state, AuthenticationService) {

  $rootScope.AuthenticationService = AuthenticationService
  $rootScope.isLoggedIn = AuthenticationService.getIsLoggedIn

  if (!$rootScope.isLoggedIn()) {
    $state.go('app.login')
  }
  // Catch all errors on state change
  $rootScope.$on('$stateChangeError', function (event, toState, toParams, fromState, fromParams, error) {
    $state.go('app.home')
  })

  // Sets up the role requirements per state
  $rootScope.$on('$stateChangeStart', function (event, toState) {
    if (AuthenticationService.getIsLoggedIn()) {
    } else {
      if (toState && toState.name !== 'app.login') {
        $state.go('app.login')
      }
    }
  })
})

The test I am trying to achieve:
'use strict'

describe('Controller', function () {

  var $scope
    , $state
    , $rootScope
    , AuthenticationService
    , $controller

  beforeEach(module('replanApp'))

  beforeEach(inject(function ($injector) {
    $state = $injector.get('$state')
    $rootScope = $injector.get('$rootScope')
    AuthenticationService = $injector.get('AuthenticationService')
    $scope = $rootScope.$new()
    $controller = $injector.get('$controller')
  }))

  describe('Initializers', function () {
    it('should redirect to /login if the user is not logged in', function () {
      $state.go('app.admin.index')
      $rootScope.$apply()
      assert.notOk(AuthenticationService.getIsLoggedIn())
      assert.equal($state.current.name, 'app.login')
    })
  })
})

It basically should go to a state, then $rootScope.$on('$stateChangeStart', fn(){}) should have see that the user is not logged in, and transit them to the app.login state.
But I am getting AssertionError: expected 'app.admin.index' to equal 'app.login'
How would one achieve testing with $stateChangeStart and other events?

https://gist.github.com/eddiemonge/f6a58169c2846731a1eb


'count' difference between SQLs using group by time or not

System info: [Include InfluxDB version, operating system name, and other relevant details]
influxdb: 1.5.4
os: CentOS Linux release 7.2.1511 (Core)
kernel:3.10.0-327.36.3.el7.x86_64 #1 SMP Mon Oct 24 16:09:20 UTC 2016 x86_64 x86_64 x86_64 GNU/Linux
Steps to reproduce:
SELECT count("http_version") FROM "three_days"."production_nginx_access_log" WHERE time >= now() - 1s
name: production_nginx_access_log
time                count
----                -----
1531104466490448482 12463
SELECT count("http_version") FROM "three_days"."production_nginx_access_log" WHERE time >= now() - 1s group by time(100ms) fill(0);
name: production_nginx_access_log
time                count
----                -----
1531104479900000000 29
1531104480000000000 264
1531104480100000000 285
1531104480200000000 277
1531104480300000000 290
1531104480400000000 297
1531104480500000000 284
1531104480600000000 275
1531104480700000000 293
1531104480800000000 290
1531104480900000000 246

Expected behavior: [What you expected to happen]
What you can see that the difference between the two sqls above is using group by or not.
What I epected is that the first count is nearly the same as the sum of second output.
But it turns out that the two are too far away.

You can not assume that you are accessing the same data if you use now(). The ungrouped timestamp<PHONE_NUMBER>490448482 does not overlap with any of the grouped ones. Data may change between your queries.


added author in package.json

Describe the changes
Added author to package.json as library was giving error because in react-native-video.podspec we have
Line 11 : s.author         = package['author'] and author was not available in package.json
Attached screenshot for the same

No, we need to remove the author from the podspec.

I tried but it game me error, that's why I added

In that case please change it to Community Contributors

What will be the authors in that case, if you can specify that would be great :)

I've made made changes, if you can check @hueniverse

Any update on this, when can we Merge @hueniverse


Fix mac os build

Description
The PR contains:

fixes of Mac OS build and wallet creation;
corrections of JS/TS code style for wallet-rpc service;
introduces eslint for the project;
adds more precise logging of xcash-wallet-rpc and HTTP request / responses between the electron app and xcash-wallet-rpc;
updates dependencies;

In the future PRs it's better to apply formatting for the files where changes are made.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Run npm run package-mac;
Create a new wallet and see something similar to this:

Test Configuration:

Firmware version: Mac OS BigSur (version 11.2 (20D64))
Hardware: MacBook Pro (13-inch, 2019, Four Thunderbolt 3 ports)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings

~- [ ] I have added tests that prove my fix is effective or that my feature works~
~- [ ] New and existing unit tests pass locally with my changes~

[x] Any dependent changes have been merged and published in downstream modules

Agreed offline with @zachhildreth that's it's ok to be merged


patch release v3.0.1

This pull request prepares the following release:

Repository
Branch
Update
Change

fluent-vue-loader
releases/v3.0.1
patch
3.0.0 → 3.0.1

Release Summary
This is going to be published with the following command:
npm publish --tag latest
Merging Instructions
When merging this pull request, you need to Squash and merge and make sure that the title starts with patch release v3.0.1.

See details
After that, a commit patch release v3.0.1 will be added and you or your CI can run shipjs trigger to trigger the release based on the commit.

This pull request is automatically generated by Ship.js.

Codecov Report

Merging #7 (e91adb3) into main (3e3e320) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main        #7   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            2         2
  Lines           30        30
  Branches         9         9
=========================================
  Hits            30        30

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 3e3e320...e91adb3. Read the comment docs.


Make changes to python backend to run on inf2 and trn1 devices

Add setup script changes to use the updated runtime for inf2 and trn1
Incoming:

container script changes
generate model script changes

Separate inf2/inf1 scripts will be merged and supported via cmdline to specify which device to generate model for

Pending:

Doc updates
Updates to qa folder Dockerfile to allow for testing model generation script

Hi @nskool, thanks for working on the inferentia improvement, I notice that you are also making changes with respect to inf1, I wonder if you have tested the change fixes inf1 deployment as well? There was report regarding the inf1 scripts are outdated.

Hi @nskool , there is change on inf1 setup merged recently (https://github.com/triton-inference-server/python_backend/pull/236), just an reminder that you may need to resolve conflicts regarding to that, and feel free to let us know that whether the inf1 setup needs to be improved.

@GuanLuo can you help review

Looks like there is conflict in inferentia/scripts/setup.sh, can you look into resolving it?

Looks like there is conflict in inferentia/scripts/setup.sh, can you look into resolving it?

Thanks, resolved - it was the part dependent on conda/python stub, I don't think it's needed if pip is being used - it works as expected. Let me know if you think otherwise.

LGTM

@nskool I have made some changes to fix the script format and actually remove TensorBoard dependency for inf1 (possibly can do the same with inf2), do you mind to apply it to your branch? I think at least for TF1, the TensorBoard dependency needs to be remove as it seems to be installing different protobuf version (both with and without pinning version) which will cause issue when using TF at later stage. https://github.com/triton-inference-server/python_backend/commit/6c70acd0a7e42cbf6c848ebfeb5a50b6c4198035

@nskool I have made some changes to fix the script format and actually remove TensorBoard dependency for inf1 (possibly can do the same with inf2), do you mind to apply it to your branch? I think at least for TF1, the TensorBoard dependency needs to be remove as it seems to be installing different protobuf version (both with and without pinning version) which will cause issue when using TF at later stage. 6c70acd

@GuanLuo That makes sense to me, thanks, I've updated the PR

@nskool The test passed, let me know if the PR is ready to be merged

@nskool The test passed, let me know if the PR is ready to be merged

Thanks @GuanLuo, yes, The PR can be merged.


[BEAM-8006] Add retracting to windowing strategy translation.

Follow up work of BEAM-7965.
Note that this PR hasn't exposed retracting mode to users, in other words, users cannot specify retracting mode after this PR in Java SDK.
To really expose retracting mode for users, it will require a change at Window.java, which will happen much later.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Choose reviewer(s) and mention them in a comment (R: @username).
[x] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Run Java PreCommit

Run Java PreCommit


Include trust relationship when snapshotting the accounts

Describe the issue
Currently the scripts that create the snapshot for an account (https://github.com/BCDevOps/aws-sea-guardrail-scanner) list all the roles available to an account. However, the account only can assume the roles. Technically speaking if the AssumeRolePolicyDocument includes an statement similar to
"Principal": {
"Federated": "arn:aws:iam::900136227765:saml-provider/BCGovKeyCloak-tmhl5tvs"
},
That means the user that has logged into AWS using KeyCloak can assume the role.
Currently the script is not capturing this information, so if an account is granted more roles the script will not detect it.
This ticket is to add the code to indicate the roles for which the account has a trust relationship
Additional context
This new piece of information may detect if permissions (through changing the roles an account can assume) have been changed.
Definition of done
Add the code to capture this information to the script that snapshots a single account
Add the code to capture this information to the script that snapshots a multiple accounts
Add the code to the comparison script that compares two snapshots
Technical details:
Probably use  list-roles and detect the ones where the principal is the federated user. list-roles output can ve uses as input for get-role --role-name, if Access Denied, there is no trust relationship

aws iam list-roles | jq '.Roles[] | .AssumeRolePolicyDocument.Statement[] | select(.Principal.AWS=="arn:aws:iam::940346833507:root")'


Included jQuery in form-docs

Well, I was using it with forms and the fields were not working as I expected with Materialize. Took me hours to realize that this needed jQuery and it was not included in the documentation.
I hope this would save time to other people.

Thank you!

Thanks to you for making this awesome thing :) :+1:


Introducing assessments and assessmentsMetadata types

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[x] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-ruby

Automation for azure-sdk-for-python

Automation for azure-sdk-for-js

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

Can one of the admins verify this patch?

@fearthecowboy I'm not sure what this modeler failure means, can you please elaborate?

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/5844
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/5844/commits/685d360c59a1447e19d7275abd0e17386d923c23

@jhendrixMSFT all you comments were resolved, can you please review again?

@chlahav for SDK this looks good, still need ARM to sign off.  Have these APIs been deployed?  We can't merge until they're live.

@jhendrixMSFT these APIs are not exposed since we need ARM review to open them in the manifest.
after the ARM review we will expose the endpoint in the manifest, validate the quality and remove the DoNotMerge label so the PR will be completed

startbuild

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/6006
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/6006/commits/eb2620dced9ceb794949ff760addc9a2de412241

The following error needs to be fixed.
Swagger document contains two or more x-ms-enum extensions with the same name 'ReportedSeverity' and different values: Informational,Low,Medium,High vs. Compute,Network,Data,IdentityAndAccess,IoT

@jhendrixMSFT fixed

startbuild

Automation for azure-sdk-for-net
A PR has been created for you:
https://github.com/azure/azure-sdk-for-net/pull/6207
.NET SDK Commits:
https://github.com/adxsdknet/azure-sdk-for-net/pull/6207/commits/3a36eb3246ce051906ca1bfadcbc3a979e6f058f

@majastrz the requested changes were pushed, can you please review?

Now that https://github.com/Azure/azure-rest-api-specs/pull/5929 has been merged into master can you please merge master into your fork to pick up those changes?

@chlahav hello!  What's the status of this PR?

@chlahav hello! What's the status of this PR?

we are still working on validating the API,
it is still not ready to public preview yet so we don't want to merge it
we will update this PR right beforehand to save you guys unnecessary reviews

@chlahav is this ready yet?  also note there are several merge conflicts that need to be resolved.

@chlahav can we close the PR for now since you're not ready for merge, and the PR is over 4 months, there're lots of conflict with latest code base already.

Closing due to inactivity.


create VLAN based network

to create a VLAN based tenant network set vlan_provider_network is true and
set the below config in extra heat config

NeutronBridgeMappings=openshift:br-openshift,datacentre:br-ex
NeutronNetworkVLANRanges=openshift:500:1000,datacentre:1:500

fixes #327

@venkataanil @masco Do we not need a new bridge mapping for he openshift_vlan_interface?

Yes

@venkataanil @masco Do we not need a new bridge mapping for he openshift_vlan_interface?

Yes

@masco your PR is incomplete. It is not defining needed THT params i.e
NeutronNetworkType: vlan
network_vlan_ranges=openshift:500:1000,datacentre:1:500

@masco your PR is incomplete. It is not defining needed THT params i.e
NeutronNetworkType: vlan
network_vlan_ranges=openshift:500:1000,datacentre:1:500

@masco @venkataanil Is this working now and is the PR final or, are more changes expected?

@masco @venkataanil Is this working now and is the PR final or, are more changes expected?

@masco @venkataanil Is this working now and is the PR final or, are more changes expected?

@smalleni we are facing some issues while creating an instance. working on it and will update the PR after fixing the issue.

@masco @venkataanil Is this working now and is the PR final or, are more changes expected?

@smalleni we are facing some issues while creating an instance. working on it and will update the PR after fixing the issue.


Hive Boost not working for heating or hot water since recent updates

I have used both of these before in scripts and automations but they do not work now. For example:
'1582797368073':
alias: Heating - Boost
sequence:

data:
temperature: 20.5
time_period: '01:30:00'
entity_id: climate.heating
service: hive.boost_heating

This is the log entry when the script is run:
Error executing script script.1582797368073. Invalid data for call_service at pos 1: value should be a string for dictionary value @ data['entity_id']

Hi there @coedeva, could you please use the issue template provided when creating an issue? There is all kinds of information missing in your report now, which is valuable and needed to resolve your issue.

Hi,
Please could you send me a link to the issue reporting template as I can’t find it.
Regards,
Mark Nott
+44 7894 843955
From: Franck Nijhof<EMAIL_ADDRESS>Sent: 27 February 2020 10:56
To: home-assistant/home-assistant<EMAIL_ADDRESS>Cc: coedeva<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [home-assistant/home-assistant] Hive Boost not working for heating or hot water since recent updates (#32252)
Hi there @coedeva https://github.com/coedeva , could you please use the issue template provided when creating an issue? There is all kinds of information missing in your report now, which is valuable and needed to resolve your issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/home-assistant/home-assistant/issues/32252?email_source=notifications&email_token=ANR45ZGXTZXQSLALFR7YZZTRE6L3LA5CNFSM4K4X2KJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEND5BJQ#issuecomment-591909030 , or unsubscribe https://github.com/notifications/unsubscribe-auth/ANR45ZF3Z4RBU2II4KR47XLRE6L3LANCNFSM4K4X2KJA .  https://github.com/notifications/beacon/ANR45ZBMHBSNTNC7IO4CQDDRE6L3LA5CNFSM4K4X2KJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEND5BJQ.gif

Please fill out the issue template, you can find it by clicking on New Issue and then selecting Report a bug with Home Assistant. Copy and paste the provided template into this already created issue, click preview to take a look at the empty template and then fill out as much as you can. Any additional information can be helpful!
Thanks for understanding :)


golang.org/x/net/icmp

Thanks for the package.
Not sure which came first, but is there any reason why you wouldn't use golang.org/x/net/icmp instead of your own icmp.go?

I can't remember for sure but when I wrote this, there wasn't enough functionality in the package but now it has so I changed to use it instead of my own. Thanks for your comment!

Can this issue now be closed?


Fix Connected wallet tag color

Should be green when wallet is connected

Hello @peterjah
It sounds like your query would be best dealt with by the support team.
I'll have to refer you to the official support live chat with the ticket ID WAL07638
Please see the link below to our dedicated support line:
Wallet Network DApps Support Ticket Requests.
Click on the live chat icon at the bottom corner of the page to initiate chat with a live support agent.


Component data is not send to child components (1.0.0-beta.4)

Components are not sending its data to the props requested by child components.
Or i'm missing something...
https://jsfiddle.net/x8k3v6rt/1/

Did you read the guide? http://rc.vuejs.org/guide/components.html#Props
You "send" props  in the template,

My apologies.
I have been looking over that for the last half hour.

see working example.
https://jsfiddle.net/x8k3v6rt/2/