Added conditional handling and logging of nil unstaged chunks

When the write function encounters a chunk that is defined as a Nil object, the call to synchronize fails. This failure causes an unexpected error that is caught and handled like a failure to connect, with a 30 second retry delay.
In an environment with a large amount of data, and this Nil scenario occurring frequently, it introduces significant processing delays since all processing activity stops during the 30 second window. The topic being queried continues to fill but no agents parse the data, leading to an ever-increasing backlog.

However, it does not seem possible.
https://github.com/fluent/fluentd/blob/78c91468d65da7cce0999fd94dbe61f7a35b606b/lib/fluent/plugin/buffer.rb#L356-L359

Tell that to my constant 100MB/sec data flow that throws this error frequently. :)
I'll see if I can forcefully reproduce it outside that environment.
Here's the stack trace, for clarification:
2023-09-15 17:33:32 +0000 [warn]: #0 emit transaction failed: error_class=NoMethodError error="undefined method `synchronize' for nil:NilClass" location="/opt/fluent/lib/ruby/gems/3.2.0/gems/fluentd-1.16.2/lib/fluent/plugin/buffer.rb:422:in `block in write'" tag="[redacted]"
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluentd-1.16.2/lib/fluent/plugin/buffer.rb:422:in `block in write'
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluentd-1.16.2/lib/fluent/plugin/buffer.rb:416:in `each'
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluentd-1.16.2/lib/fluent/plugin/buffer.rb:416:in `write'
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluentd-1.16.2/lib/fluent/plugin/output.rb:1095:in `block in handle_stream_simple'
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluentd-1.16.2/lib/fluent/plugin/output.rb:977:in `write_guard'
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluentd-1.16.2/lib/fluent/plugin/output.rb:1094:in `handle_stream_simple'
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluentd-1.16.2/lib/fluent/plugin/output.rb:967:in `execute_chunking'
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluentd-1.16.2/lib/fluent/plugin/output.rb:897:in `emit_buffered'
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluentd-1.16.2/lib/fluent/event_router.rb:115:in `emit_stream'
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluent-plugin-kafka-0.19.0/lib/fluent/plugin/in_rdkafka_group.rb:287:in `emit_events'
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluent-plugin-kafka-0.19.0/lib/fluent/plugin/in_rdkafka_group.rb:269:in `block in run'
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluent-plugin-kafka-0.19.0/lib/fluent/plugin/in_rdkafka_group.rb:214:in `each_batch'
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluent-plugin-kafka-0.19.0/lib/fluent/plugin/in_rdkafka_group.rb:229:in `run'
  2023-09-15 17:33:32 +0000 [warn]: #0 /opt/fluent/lib/ruby/gems/3.2.0/gems/fluentd-1.16.2/lib/fluent/plugin_helper/thread.rb:78:in `block in thread_create'

Sorry for the delay.
I think this fix would be reasonable, but I'm still checking the mechanism and impact of this bug.
I will merge this after it.

I think this fix would be reasonable, but I'm still checking the mechanism and impact of this bug.

I don't yet fully understand the role of unstaged_chunks...
At least, we can merge this to prevent the unexpected error.
It does not change the specification.
It only prevents this error.
So, I merge this now for the next release.

Seeing #4342, I understand now some mechanics of this bug.
When retrying write_step_by_step, the first staged chunk is added to operated_chunks duplicatedly.
So, unstaged_chunks[m].pop can be called more than once for the same metadata.
I believe this duplication of operated_chunks is another bug and we should fix it.
Anyway, these unstaged chunks will be enqueued immediately in the Buffer::write(),
so, for now, it is sufficient to avoid the error by this fix.


API Error Messages Suggestion

Currently the error messages in the API are hard to use in other software unless you just throw out the exact error response that comes back.
For example
Error occurred when starting proxy device: Error: Failed to receive fd from listener process

and
Error occurred when starting proxy device: Error: Failed to listen on <IP_ADDRESS>:80: listen tcp <IP_ADDRESS>:80: bind: address already in us

In order to use this API and give better responses to users, i would have to attempt to parse the error message string to find out what is going on.
I am wondering if the API can return a category error, and then a system error message if available, e.g ZFS cannot destroy data set because its busy, to make the API more friendly.
 {
    "type": "sync",
    "status": "Success",
    "status_code": 200,
    "operation": "",
    "error_code": 0,
    "error": "",
    "metadata": {
        "id": "4400cf9c-06a6-4202-8d07-b227540dc76a",
        "class": "task",
        "description": "Starting instance",
        "created_at": "2021-02-12T15:08:51.542477114Z",
        "updated_at": "2021-02-12T15:08:51.542477114Z",
        "status": "Failure",
        "status_code": 400,
        "resources": {
            "instances": [
                "/1.0/instances/ubuntu-apache"
            ]
        },
        "metadata": null,
        "may_cancel": false,
        "err": "Error occurred when starting proxy device: Error: Failed to listen on <IP_ADDRESS>:80: listen tcp <IP_ADDRESS>:80: bind: address already in use",
        "reason": "proxy-device-start-error",
        "message": "Failed to listen on <IP_ADDRESS>:80: listen tcp <IP_ADDRESS>:80: bind: address already in use"
        "location": "none"
    }
}

Initially marked as Maybe but unfortunately the answer really is a no in this case...
Most LXD functions just return a Go error which doesn't let you split the message and indicate categories like you'd want here.
This then gets turned into an ErrorResponse in the API handler.
So we could add context to tell you what API endpoint returned the error, but not much else.
In your example above, all we could tell you is /1.0/instances/NAME/state was the thing which failed, but you already know that based on the operation description.
What we do to make things easier to debug is extend the error struct every time it passes through as function, that's what caused Error occurred when starting proxy device to be prepended in this case, but that's just a string prefix, not something we can really turn into anything more structured.


Unable to install Wallet app on Mac

Hi @Grivante , regarding the issue Unable to install Wallet app on Mac #936, we can further discuss on the issue there. We would like to keep same issues in one thread. Thank you.


Implement Certificate Chain Secret and set CertificateAuthorityActivation default fields

Description of changes:

Input certificateChain field as a Secret
Output update certificateChain to a Secret
Set spec field "Status" to a default value

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/retest


Sometimes hangs when closing video on iPhone/iPod Touch

At least with iPhone 3Gs and iPod Touch 4, closing a video sometimes hangs the UI.

this versino of demo is obsolete, closing


Add your MoJ email address to your Github account

Your access to the ministryofjustice GitHub organisation will be revoked if you don't add a @digital.justice.gov.uk, @justice.gov.uk, @cica.gov.uk or @hmcts.net email address to your GitHub account. This is an automated process please do not change any settings, only make the email change in your settings area.

Hi Nick, I have added my @digital.justice & @hmcts.net emails to my github account.


Implement Code One S-type binary encoding

Currently the user must enter successive 5-bit characters using the ^NNN pattern.
We should perform the binary encoding of the given input, up to 18 digits -> 60 bits.

Fixed in release 2015-05-09.


Check Addresses early, report EC balance at boot

Checks addresses for staker early, so errors are caught quickly.

I have no idea how this change fails to build.


🌐 Add Chinese translation for docs/tutorial/dependencies/index.md

as title

Great, thank you @jaystone776! :cake: :coffee:
And thanks @waynerv and @Laineyzhang55 for the reviews! :bow:


Fix empty SSH username in ssh_gateway parameter

Signed-off-by: Baurzhan Konurbayev<EMAIL_ADDRESS>

Dear Chef guys,
When I am setting ssh_gateway for machine resource using chef-provisioning-fog
[:machine_options][:ssh_gateway] = 'USER@HOST'
Afterwards chef-provisioning's method 'gateway' receives ssh_gateway without USER right here:
gw_user, gw_host = options[:ssh_gateway].split('@')
Could you, please, consider this PR?
Thank you in advance!

I'll release this soon too. Sorry for the delay.


where is points3D_withlidar.txt

Thanks for your impressive work!
I downloaded the preprocessed data, but when I ran train.sh, I found that there was no points3D_withlidar.txt file. I want to ask if there is a replacement file or if I missed a step.

generate with scripts/shells/points_cloud_generate.sh

OK，I will try this script，Thanks！

generate with scripts/shells/points_cloud_generate.sh

it will gen points3d_withlidar.bin instead txt file, it's ok for training?

generate with scripts/shells/points_cloud_generate.sh

it will gen points3d_withlidar.bin instead txt file, it's ok for training?

yeah, I used it for training and got the correct result. You just need to change the file name in train.sh
data_root=$1
cuda_id=$2

mkdir -p output/

CUDA_VISIBLE_DEVICES=$cuda_id sgn-train street-gaussians-ns \
    --experiment_name street-gaussians-ns \
    --output_dir output/ \
    --vis viewer+wandb \
    --viewer.quit_on_train_completion True \
    colmap-data-parser-config \
    --data $data_root \
    --colmap_path colmap/sparse/0 \
    --load_3D_points True \
    --max_2D_matches_per_3D_point 0 \
    --undistort True \
    --segments-path segs \
    --filter_camera_id 1 2 3 4 5\
    --init_points_filename points3D_withlidar.bin         # this file name

I set --filter_camera_id to 1 2 3 4 5,I got this error:
[16:26:28] Caching / undistorting eval images                                                     sgn_datamanager.py:173
[16:26:29] Caching / undistorting train images                                                    sgn_datamanager.py:173
scripts/shells/train.sh：行 19: 82357 已杀死               CUDA_VISIBLE_DEVICES=$cuda_id sgn-train street-gaussians-ns --experiment_name street-gaussians-ns --output_dir output2/ --vis viewer+tensorboard --viewer.quit_on_train_completion True colmap-data-parser-config --data $data_root --colmap_path colmap/sparse/0 --load_3D_points True --max_2D_matches_per_3D_point 0 --undistort True --segments-path segs --filter_camera_id 1 2 3 4 5 --init_points_filename points3D_withlidar.bin
Can you tell me how to set to solve this?
Thanks

generate with scripts/shells/points_cloud_generate.sh

it will gen points3d_withlidar.bin instead txt file, it's ok for training?

yeah, I used it for training and got the correct result. You just need to change the file name in train.sh
data_root=$1
cuda_id=$2

mkdir -p output/

CUDA_VISIBLE_DEVICES=$cuda_id sgn-train street-gaussians-ns \
    --experiment_name street-gaussians-ns \
    --output_dir output/ \
    --vis viewer+wandb \
    --viewer.quit_on_train_completion True \
    colmap-data-parser-config \
    --data $data_root \
    --colmap_path colmap/sparse/0 \
    --load_3D_points True \
    --max_2D_matches_per_3D_point 0 \
    --undistort True \
    --segments-path segs \
    --filter_camera_id 1 2 3 4 5\
    --init_points_filename points3D_withlidar.bin         # this file name

I set --filter_camera_id to 1 2 3 4 5,I got this error: [16:26:28] Caching / undistorting eval images sgn_datamanager.py:173 [16:26:29] Caching / undistorting train images sgn_datamanager.py:173 scripts/shells/train.sh：行 19: 82357 已杀死 CUDA_VISIBLE_DEVICES=$cuda_id sgn-train street-gaussians-ns --experiment_name street-gaussians-ns --output_dir output2/ --vis viewer+tensorboard --viewer.quit_on_train_completion True colmap-data-parser-config --data $data_root --colmap_path colmap/sparse/0 --load_3D_points True --max_2D_matches_per_3D_point 0 --undistort True --segments-path segs --filter_camera_id 1 2 3 4 5 --init_points_filename points3D_withlidar.bin Can you tell me how to set to solve this? Thanks

Oh, sorry for my mistake. This parameter [-filter_camera_id 1 2 3 4 5] is used for choosing which cameras you are going to train with. You can set it according to your training data. For example, if you want to train with only one camera data in your training data including five cameras, just set [-filter_camera_id 1]. It seems that your error was caused by choosing non existing camera. Setting [-filter_camera_id 1] might help

generate with scripts/shells/points_cloud_generate.sh

it will gen points3d_withlidar.bin instead txt file, it's ok for training?

yeah, I used it for training and got the correct result. You just need to change the file name in train.sh
data_root=$1
cuda_id=$2

mkdir -p output/

CUDA_VISIBLE_DEVICES=$cuda_id sgn-train street-gaussians-ns \
    --experiment_name street-gaussians-ns \
    --output_dir output/ \
    --vis viewer+wandb \
    --viewer.quit_on_train_completion True \
    colmap-data-parser-config \
    --data $data_root \
    --colmap_path colmap/sparse/0 \
    --load_3D_points True \
    --max_2D_matches_per_3D_point 0 \
    --undistort True \
    --segments-path segs \
    --filter_camera_id 1 2 3 4 5\
    --init_points_filename points3D_withlidar.bin         # this file name

yes, same method and it works fine

generate with scripts/shells/points_cloud_generate.sh

it will gen points3d_withlidar.bin instead txt file, it's ok for training?

yeah, I used it for training and got the correct result. You just need to change the file name in train.sh
data_root=$1
cuda_id=$2

mkdir -p output/

CUDA_VISIBLE_DEVICES=$cuda_id sgn-train street-gaussians-ns \
    --experiment_name street-gaussians-ns \
    --output_dir output/ \
    --vis viewer+wandb \
    --viewer.quit_on_train_completion True \
    colmap-data-parser-config \
    --data $data_root \
    --colmap_path colmap/sparse/0 \
    --load_3D_points True \
    --max_2D_matches_per_3D_point 0 \
    --undistort True \
    --segments-path segs \
    --filter_camera_id 1 2 3 4 5\
    --init_points_filename points3D_withlidar.bin         # this file name

yes, same method and it works fine

I download preprocessed dataset from this repo which is<PHONE_NUMBER>2863604198_472_000_492_000,and cameras.txt in /colmap shows it has 5 cameras:
1 OPENCV 1920 1280 2044.3189893365634 2044.3189893365634<PHONE_NUMBER>401635<PHONE_NUMBER>955747 0.048519645404847 -0.31855151741237414 -0.0005279507169045765 -0.0010850539443264748
2 OPENCV 1920 1280 2064.096136816171 2064.096136816171<PHONE_NUMBER>820795<PHONE_NUMBER>405554 0.04536665392932437 -0.3481585691089555 0.0023209650920834515 0.000879426586508394
3 OPENCV 1920 1280 2047.353864208166 2047.353864208166<PHONE_NUMBER>600746<PHONE_NUMBER>214233 0.03923964629713441 -0.30937484864506715 0.0016984481880601527 -0.0009807100492894205
4 OPENCV 1920 886 2052.166978577554 2052.166978577554<PHONE_NUMBER>948891<PHONE_NUMBER>973026 0.04792858845797487 -0.3115144801129193 0.00044896409868372216 0.0006301442821205557
5 OPENCV 1920 886 2060.943963183803 2060.943963183803<PHONE_NUMBER>331334<PHONE_NUMBER>60424 0.04466498449403254 -0.33887981610387924 0.0015137815608006677 4.743970552947572e-05
So I think maybe is there any other settings I should set?

generate with scripts/shells/points_cloud_generate.sh

it will gen points3d_withlidar.bin instead txt file, it's ok for training?

yeah, I used it for training and got the correct result. You just need to change the file name in train.sh
data_root=$1
cuda_id=$2

mkdir -p output/

CUDA_VISIBLE_DEVICES=$cuda_id sgn-train street-gaussians-ns \
    --experiment_name street-gaussians-ns \
    --output_dir output/ \
    --vis viewer+wandb \
    --viewer.quit_on_train_completion True \
    colmap-data-parser-config \
    --data $data_root \
    --colmap_path colmap/sparse/0 \
    --load_3D_points True \
    --max_2D_matches_per_3D_point 0 \
    --undistort True \
    --segments-path segs \
    --filter_camera_id 1 2 3 4 5\
    --init_points_filename points3D_withlidar.bin         # this file name

yes, same method and it works fine

I download preprocessed dataset from this repo which is<PHONE_NUMBER>2863604198_472_000_492_000,and cameras.txt in /colmap shows it has 5 cameras: 1 OPENCV 1920 1280 2044.3189893365634 2044.3189893365634<PHONE_NUMBER>401635<PHONE_NUMBER>955747 0.048519645404847 -0.31855151741237414 -0.0005279507169045765 -0.0010850539443264748 2 OPENCV 1920 1280 2064.096136816171 2064.096136816171<PHONE_NUMBER>820795<PHONE_NUMBER>405554 0.04536665392932437 -0.3481585691089555 0.0023209650920834515 0.000879426586508394 3 OPENCV 1920 1280 2047.353864208166 2047.353864208166<PHONE_NUMBER>600746<PHONE_NUMBER>214233 0.03923964629713441 -0.30937484864506715 0.0016984481880601527 -0.0009807100492894205 4 OPENCV 1920 886 2052.166978577554 2052.166978577554<PHONE_NUMBER>948891<PHONE_NUMBER>973026 0.04792858845797487 -0.3115144801129193 0.00044896409868372216 0.0006301442821205557 5 OPENCV 1920 886 2060.943963183803 2060.943963183803<PHONE_NUMBER>331334<PHONE_NUMBER>60424 0.04466498449403254 -0.33887981610387924 0.0015137815608006677 4.743970552947572e-05 So I think maybe is there any other settings I should set?

Yeah, I used this dataset too. So we can eliminate error with dataset. Can you provide more log or more information about environment?

generate with scripts/shells/points_cloud_generate.sh

it will gen points3d_withlidar.bin instead txt file, it's ok for training?

yeah, I used it for training and got the correct result. You just need to change the file name in train.sh
data_root=$1
cuda_id=$2

mkdir -p output/

CUDA_VISIBLE_DEVICES=$cuda_id sgn-train street-gaussians-ns \
    --experiment_name street-gaussians-ns \
    --output_dir output/ \
    --vis viewer+wandb \
    --viewer.quit_on_train_completion True \
    colmap-data-parser-config \
    --data $data_root \
    --colmap_path colmap/sparse/0 \
    --load_3D_points True \
    --max_2D_matches_per_3D_point 0 \
    --undistort True \
    --segments-path segs \
    --filter_camera_id 1 2 3 4 5\
    --init_points_filename points3D_withlidar.bin         # this file name

yes, same method and it works fine

I download preprocessed dataset from this repo which is<PHONE_NUMBER>2863604198_472_000_492_000,and cameras.txt in /colmap shows it has 5 cameras: 1 OPENCV 1920 1280 2044.3189893365634 2044.3189893365634<PHONE_NUMBER>401635<PHONE_NUMBER>955747 0.048519645404847 -0.31855151741237414 -0.0005279507169045765 -0.0010850539443264748 2 OPENCV 1920 1280 2064.096136816171 2064.096136816171<PHONE_NUMBER>820795<PHONE_NUMBER>405554 0.04536665392932437 -0.3481585691089555 0.0023209650920834515 0.000879426586508394 3 OPENCV 1920 1280 2047.353864208166 2047.353864208166<PHONE_NUMBER>600746<PHONE_NUMBER>214233 0.03923964629713441 -0.30937484864506715 0.0016984481880601527 -0.0009807100492894205 4 OPENCV 1920 886 2052.166978577554 2052.166978577554<PHONE_NUMBER>948891<PHONE_NUMBER>973026 0.04792858845797487 -0.3115144801129193 0.00044896409868372216 0.0006301442821205557 5 OPENCV 1920 886 2060.943963183803 2060.943963183803<PHONE_NUMBER>331334<PHONE_NUMBER>60424 0.04466498449403254 -0.33887981610387924 0.0015137815608006677 4.743970552947572e-05 So I think maybe is there any other settings I should set?

Yeah, I used this dataset too. So we can eliminate error with dataset. Can you provide more log or more information about environment?

I found my error was caused by the following code snippet in sgn_datamanager.py:
if cache_images_device == "gpu":
for cache in undistorted_images:
cache["image"] = cache["image"].to(self.device)
if "mask" in cache:
cache["mask"] = cache["mask"].to(self.device)
elif cache_images_device == "cpu":
for cache in undistorted_images:
cache["image"] = cache["image"].pin_memory()
if "mask" in cache:
cache["mask"] = cache["mask"].pin_memory()
else:
assert_never(cache_images_device)
I comment code above,training process can be done,but render command 'ns-render camera-path --load-config xxx/config.yml --camera-path-filename xxx/camera_paths/2024-07-29-13-15-51.json --output-path renders/1044street-gaussians-ns/street_gaussians_ns/sgn_splatfacto_scene_graph.py", line 323, in get_outputs
assert camera.times is not None
AssertionError8102132863604198_472_000_492_000/2024-07-29-13-15-51.mp4' got the following error:
Do these two things have some relationship?

Not sure about the relationship. I noticed the memory cost was unusual. Have you tried to modify the configuration in sgn_config.py or sgn_splatfacto.py? less iteration or less initial gaussians may help. About the render command, I used the command generated by nerfstudio, and did not meet the problem. Here is one of my camera path of dataset<PHONE_NUMBER>2863604198 which worked for me.
2024-07-26-09-38-26.json

sgn_config.py

I user your json,it runs ok,but the mp4 contains nothing except black screen.Moreover,json I got from nerfstudio broswer is different from yours,it has no time info
2024-07-30-16-59-24.json

sgn_config.py

I user your json,it runs ok,but the mp4 contains nothing except black screen.Moreover,json I got from nerfstudio broswer is different from yours,it has no time info 2024-07-30-16-59-24.json What's the reason for this? Thanks

Oh, my camera path was generated from model trained by five cameras. If you use it on the model trained by cameras including no backward camera, you may get empty scenes.
For the timestamp in camera path,  because generated by nerfstudio, I guess it is related with your nerfstudio version. gsplat to 0.1.11 and nerfstudio to 1.1.0 worked for me.

sgn_config.py

I user your json,it runs ok,but the mp4 contains nothing except black screen.Moreover,json I got from nerfstudio broswer is different from yours,it has no time info 2024-07-30-16-59-24.json What's the reason for this? Thanks

Oh, my camera path was generated from model trained by five cameras. If you use it on the model trained by cameras including no backward camera, you may get empty scenes.
For the timestamp in camera path, because generated by nerfstudio, I guess it is related with your nerfstudio version. gsplat to 0.1.11 and nerfstudio to 1.1.0 worked for me.

Hi,do your render result looks good?I have the same dataset and camera 1-5 with you,but I got not so good render result,looks to https://github.com/LightwheelAI/street-gaussians-ns/issues/40

generate with scripts/shells/points_cloud_generate.sh

it will gen points3d_withlidar.bin instead txt file, it's ok for training?

The TXT version could be triggered by colmap command line given a BIN file


Running all Pytest tests no longer respects pytest.ini testpaths

Environment data

VS Code version: 1.59.1
Extension version (available under the Extensions sidebar): v2021.9.1246542782
OS and version: Linux x64 3.10.0-1160.41.1.el7.x86_64
Python version (& distribution if applicable, e.g. Anaconda): 3.9.0 (conda-forge)
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Relevant/affected Python packages and their versions: Pytest 6.2.5
Relevant/affected Python-related VS Code extensions and their versions: N/A
Value of the python.languageServer setting: Pylance

[NOTE: If you suspect that your issue is related to the Microsoft Python Language Server (python.languageServer: 'Microsoft'), please download our new language server Pylance from the VS Code marketplace to see if that fixes your issue]
Expected behaviour
Clicking the "run all tests" button in the testing sidebar (a.k.a. test explorer) should run pytest without specifying a path, respecting the testpaths setting from pytest.ini.
From looking at backups of previous Python logs before the recent update to the extension, this is what would get called
> python -m pytest --rootdir ~/myproject

(I've removed extraneous absolute paths and options)
Actual behaviour
It passes the root dir path to pytest. In the Python log:
> python -m pytest --rootdir ~/myproject ~/myproject

This is a problem because I can't run the tests successfully like this. I get ModuleNotFoundErrors as it tries to collect stuff it doesn't need.
Steps to reproduce:

Have a Python project with testpaths set in pytest.ini
Click the "run all tests" button

Logs

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

> ~/miniconda3/envs/<env-name>/bin/python -m pytest --rootdir <$HOME>/myproject --override-ini junit_family=xunit1 --junit-xml=/var/tmp/tmp-<rnd>.xml "-n 4" --disable-warnings -vv <$HOME>/myproject

@MetRonnie Can you share your workspace settings?

{
    "python.testing.pytestArgs": [
        "-n 4",
        "--disable-warnings",
        "-vv"
    ],
    "python.linting.flake8CategorySeverity.A": "Warning", // flake8-builtins
    "python.linting.flake8CategorySeverity.B": "Warning", // flake8-bugbear
    "python.linting.flake8CategorySeverity.C": "Warning", // flake8-comprehensions
    "python.linting.flake8CategorySeverity.M": "Warning", // flake8-mutable
    "python.linting.flake8CategorySeverity.N": "Warning", // flake8-broken-line
    "python.linting.flake8CategorySeverity.S": "Warning", // flake8-simplify
    "python.linting.flake8CategorySeverity.T": "Warning", // flake8-debugger
    "python.pythonPath": "<$HOME>/miniconda3/envs/<env-name>/bin/python",
}

@MetRonnie I will need the pytest.ini. Looks like the above info is not enough to repro.

The project is https://github.com/cylc/cylc-flow
pytest.ini: https://github.com/cylc/cylc-flow/blob/bf010514f8bd7bcce2529dc45a7c72d5709de417/pytest.ini#L17-L34

It seems to be working fine for me:

What if you invoke pytest on the command line with a path argument? This is what happens for me (same thing happens when clicking "run all tests" button in VSCode, except without the -x option it proceeds to run the tests and a lot of them fail):
$ cd ~/github/cylc-flow
$ pytest --disable-warnings -x .
================================== test session starts ==================================
platform linux -- Python 3.9.0, pytest-6.2.5, py-1.9.0, pluggy-0.13.1 -- ~/miniconda3/envs/cylc8/bin/python3.9
cachedir: .pytest_cache
rootdir: ~/github/cylc-flow, configfile: pytest.ini
plugins: forked-1.3.0, env-0.6.2, mock-3.4.0, xdist-2.1.0, cov-2.12.1, asyncio-0.15.1, anyio-3.3.0, tornasync-0.6.0.post2
[gw0] linux Python 3.9.0 cwd: ~/github/cylc-flow
[gw0] Python 3.9.0 | packaged by conda-forge | (default, Nov 26 2020, 07:57:39)  -- [GCC 9.3.0]
collecting 0 items / 1 error
======================================== ERRORS =========================================
_____ ERROR collecting tests/f/jinja2/08-local-lib-python/Jinja2Filters/qualify.py ______
tests/f/jinja2/08-local-lib-python/Jinja2Filters/qualify.py:19: in <module>
    import local_lookup
E   ModuleNotFoundError: No module named 'local_lookup'
================================ short test summary info ================================
ERROR tests/f/jinja2/08-local-lib-python/Jinja2Filters/qualify.py - ModuleNotFoundErro...
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! stopping after 1 failures !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
!!!!!!!!!!!!!!!!! xdist.dsession.Interrupted: stopping after 1 failures !!!!!!!!!!!!!!!!!
=================================== 1 error in 9.81s ====================================

I get the same ModuleNotFoundError if I disable xdist using -n0 --dist no

Ok. I am able to repo this now.

@MetRonnie Can you check if this build helps with this: https://github.com/microsoft/vscode-python/suites/3957288208/artifacts/99373426?
I tested it locally, I still need to run it through a bunch of other test scenarios, but let me know.

(Pinging my non-work account to remind myself to try it @binaryfunt)


Provision / release should immediately return with # of actions and time estimate

Long running actions timeout the frontend; provision and release should return quickly with the # of repos that will be changed (and a time estimate).

This was done at the start of that term. Works great! Does require that a tab be open though to allow complete processing, but also makes jobs easy to cancel and easy to resume.


unable to download video from hotstar

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.12.03. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2018.12.03

Before submitting an issue make sure you have:

[x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones
[x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

the provided link is playable in browser, but youtube-dl is not downloading due to RegexNotFoundError. Not running from any vpn, using jio network.
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', 'https://www.hotstar.com/tv/sarabhai-vs-sarabhai/s-36/meet-the-sarabhais/1000010418']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2018.12.03
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17134
[debug] exe versions: none
[debug] Proxy map: {}
[hotstar]<PHONE_NUMBER>: Downloading webpage
ERROR: Unable to extract app state; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpflqtkr62\build\youtube_dl\YoutubeDL.py", line 792, in extract_info
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpflqtkr62\build\youtube_dl\extractor\common.py", line 508, in extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpflqtkr62\build\youtube_dl\extractor\hotstar.py", line 80, in _real_extract
File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpflqtkr62\build\youtube_dl\extractor\common.py", line 983, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract app state; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.

youtube-dl https://www.hotstar.com/tv/sarabhai-vs-sarabhai/s-36/meet-the-sarabhais/1000010418
[hotstar]<PHONE_NUMBER>: Downloading webpage
[hotstar]<PHONE_NUMBER>: Downloading JSON metadata
[hotstar]<PHONE_NUMBER>: Downloading m3u8 information
[download] Destination: Meet the Sarabhais-1000010418.mp4
ffmpeg version n4.1 Copyright (c) 2000-2018 the FFmpeg developers
  built with gcc 8.2.1 (GCC) 20180831
  configuration: --prefix=/usr --disable-debug --disable-static --disable-stripping --enable-fontconfig --enable-gmp --enable-gnutls --enable-gpl --enable-ladspa --enable-libaom --enable-libass --enable-libbluray --enable-libdrm --enable-libfreetype --enable-libfribidi --enable-libgsm --enable-libiec61883 --enable-libjack --enable-libmodplug --enable-libmp3lame --enable-libopencore_amrnb --enable-libopencore_amrwb --enable-libopenjpeg --enable-libopus --enable-libpulse --enable-libsoxr --enable-libspeex --enable-libssh --enable-libtheora --enable-libv4l2 --enable-libvidstab --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxcb --enable-libxml2 --enable-libxvid --enable-nvdec --enable-nvenc --enable-omx --enable-shared --enable-version3
  libavutil      56. 22.100 / 56. 22.100
  libavcodec     58. 35.100 / 58. 35.100
  libavformat    58. 20.100 / 58. 20.100
  libavdevice    58.  5.100 / 58.  5.100
  libavfilter     7. 40.101 /  7. 40.101
  libswscale      5.  3.100 /  5.  3.100
  libswresample   3.  3.100 /  3.  3.100
  libpostproc    55.  3.100 / 55.  3.100
[hls,applehttp @ 0x564d07a4e0c0] Opening 'https://staragvod-vh.akamaihd.net/i/videos/plus/svss/01/1000010418_,16,54,106,180,400,800,1300,2000,3000,4500,_STAR.mp4.csmil/crypt.key?null=0&id=AgCLOxlXFwNGjWaJClyy8GIfwcbs3qUOXO%2fwrkuHx9iTKKPTAZfkwGINS%2fDHC9OYPnElecbf3TqKUA%3d%3d' for reading
[hls,applehttp @ 0x564d07a4e0c0] Opening 'crypto+https://staragvod-vh.akamaihd.net/i/videos/plus/svss/01/1000010418_,16,54,106,180,400,800,1300,2000,3000,4500,_STAR.mp4.csmil/segment1_7_av.ts?null=0&id=AgCLOxlXFwNGjWaJClyy8GIfwcbs3qUOXO%2fwrkuHx9iTKKPTAZfkwGINS%2fDHC9OYPnElecbf3TqKUA%3d%3d' for reading
[NULL @ 0x564d07ac6d80] non-existing SPS 0 referenced in buffering period
[NULL @ 0x564d07ac6d80] SPS unavailable in decode_picture_timing
[h264 @ 0x564d080aafc0] non-existing SPS 0 referenced in buffering period
[h264 @ 0x564d080aafc0] SPS unavailable in decode_picture_timing
Input #0, hls,applehttp, from 'https://staragvod-vh.akamaihd.net/i/videos/plus/svss/01/1000010418_,16,54,106,180,400,800,1300,2000,3000,4500,_STAR.mp4.csmil/index_7_av.m3u8?null=0&id=AgCLOxlXFwNGjWaJClyy8GIfwcbs3qUOXO%2fwrkuHx9iTKKPTAZfkwGINS%2fDHC9OYPnElecbf3TqKUA%3d%3d':
  Duration: 00:20:31.80, start: 0.100667, bitrate: 0 kb/s
  Program 0
    Metadata:
      variant_bitrate : 0
    Stream #0:0: Video: h264 (Main) ([27][0][0][0] / 0x001B), yuv420p, 1280x720 [SAR 1:1 DAR 16:9], 25 fps, 25 tbr, 90k tbn, 50 tbc
    Metadata:
      variant_bitrate : 0
    Stream #0:1: Audio: aac (LC) ([15][0][0][0] / 0x000F), 48000 Hz, stereo, fltp
    Metadata:
      variant_bitrate : 0
Output #0, mp4, to 'file:Meet the Sarabhais-1000010418.mp4.part':
  Metadata:
    encoder         : Lavf58.20.100
    Stream #0:0: Video: h264 (Main) (avc1 / 0x31637661), yuv420p, 1280x720 [SAR 1:1 DAR 16:9], q=2-31, 25 fps, 25 tbr, 90k tbn, 90k tbc
    Metadata:
      variant_bitrate : 0
    Stream #0:1: Audio: aac (LC) (mp4a / 0x6134706D), 48000 Hz, stereo, fltp
    Metadata:
      variant_bitrate : 0
Stream mapping:
  Stream #0:0 -> #0:0 (copy)
  Stream #0:1 -> #0:1 (copy)
Press [q] to stop, [?] for help
frame=   22 fps=4.9 q=-1.0 size=       0kB time=00:00:00.91 bitrate=   0.4kbits

post the output with --dump-pages option.

out.txt
This contains the output which you require.

I think the issue is back again !!
https://us.hotstar.com/tv/dance-/s-281/ranveer-sara-are-overwhelmed/1000225871
Connected to US ExpressVPN, still no use !!
With this url it's playing in chrome, but not even showing list of formats with -F !!
Neither with 09.12.2018 version nor with the latest i.e. 17.12.2018 !


Contributing to the repository

Hi! Just noticed the new repository when browsing the org for things I can contribute to. Is there any note or brief doc that I can read and get a general idea of how I can contribute to this repo? What's the best place to start? (I have noticed there is no Rust code yet, so I'm asking for after the few initial commits)
Thanks!

Thanks for your interest 😄 The general idea is to have Optimism specific types for alloy, just like we have Ethereum specific types currently:
https://github.com/alloy-rs/alloy/blob/9c2f0078d3e3475dc53dc387e69294e645614441/crates/network/src/ethereum/mod.rs#L10-L31
This includes consensus and rpc-types where there are differences
Currently I don't think there's really a great place to start until we have the foundations down

Thank you! Opened a PR related to this #2.


XeMemory

Let's create XeMemory, which will report the heap details.

@rultor release, tag is 1.2

@rultor release, tag is 1.2

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 1.2

@yegor256 Done! FYI, the full log is here (took me 10min)

Oops! Job gh:yegor256/takes#728 was not in scope


No such file or directory: NRFXLIB_DIR error while building for nRF SoC with cc310 HW

raveled by #813 as now CONFIG_BOOT_USE_NRF_CC310_BL=y by default.
https://github.com/JuulLabs-OSS/mcuboot/blob/master/boot/zephyr/CMakeLists.txt#L64-L69
this cause cmake build stage error:
/home/anpu/codedev/zephyrproject/zephyr/cmake/extensions.cmake:1579 (message):
No such file or directory: NRFXLIB_DIR:
'/home/anpu/codedev/zephyrproject/zephyr/../nrfxlib'
Call Stack (most recent call first):
CMakeLists.txt:66 (assert_exists)
/ext/nrf/README.md contains description, but this is buried

Proposed solution:

make cc310 disabled by default back -this will improve user experience
add help message to cmake script for help users who encountered missing nrfxlib error


Add active state to mobile navigation buttons

Before
After

Hey @donouwens, is that blue background (ROSE Wallet button) in dark theme correct? For me it does not match any other color used in a layout.

eslint/prettier formatting


Tutorial step 5 doesn't mention need to setup git credentials

git push --all google
generates:
fatal: remote error: Forbidden

gcloud init && git config --global credential.https://source.developers.google.com.helper gcloud.cmd
from the stackdriver debug codelab
as did creating the repo from the console
I'd like to think that the codelab will be updated rather than expecting people to trawl through the git issues


RCM API includes official repositories in pipelines

When running a compose through the RCM API, the official repositories for a distro are used in addition to the ones supplied by the API.
This is wrong: the semantics of the API is that only those repos that are provided are used. The resulting image might include packages from the wrong repos.

Agreed, this was not the intention. The distro package must be refactored to no longer make the official repositories implicit. As discussed, the official repository logic should be specific to the weldr package.

Except for distro refactoring, this sounds like something that could make use of a common transformation between the user request and fully defined image build. The transformation function would take care of argument validation (like arch and image type), depsolving (which can also fail). Each API could have its own tweaks to the function: weldr would add the repos, rcm would add implicit Koji targets.

Except for distro refactoring, this sounds like something that could make use of a common transformation between the user request and fully defined image build.

Yes, I think this makes sense (but is out of scope for this issue).


error: The sandbox is not in sync with the Podfile.lock. Run 'pod install' or update your CocoaPods installation.?

Hi I'm using PNChart to implement the bar graph and pie chart.
but I have an 3 issues.
please find the below issues to get an error.
diff: /../Podfile.lock: No such file or directory
diff: /Manifest.lock: No such file or directory
error: The sandbox is not in sync with the Podfile.lock. Run 'pod install' or update your CocoaPods installation.
can you please give me the solution.
Thanks in Advance,
Nandu K.

As README shows, this library is recommended install by Cocoapods.

Here are steps to make it work.

Go to PNChartDemo target -> Build Phases
Delete "Check Pods Manifest.lock"/"Embed Pods Frameworks"/"Copy Pods Resources"
Remove "libPods-PNChartDemo.a" in Link Binary With Libraries
Download the dependency library "UICountingLabel"
Import "UICountingLabel.h"/"UICountingLabel.m" to PNChartDemo Project manually
Import with "UICountingLabel.h" instead of <UICountingLabel/UICountingLabel.h> for classes that need it


Support bower in manageiq

After https://github.com/ManageIQ/manageiq/pull/7577, we'll need to have bower available in the appliance for ManageIQ (previously, it was only needed for SSUI).
Not quite sure what's needed here if anything, possibly https://github.com/ManageIQ/manageiq-appliance/pull/65 handles it .. but not sure, can anyone look at it please?

The only thing I can see is that maybe this comment should be changed: https://github.com/ManageIQ/manageiq-appliance-build/blob/ffe799032a1a2ab40ead63f4a6d3bcdba7e4209f/kickstarts/partials/packages/includes.ks.erb#L84-L85

Right.. https://github.com/ManageIQ/manageiq-appliance-build/blob/master/kickstarts/partials/post/bundler.ks.erb made me think that since we're doing something bundler-related, we may need to handle this as well, but.. should I do a PR for the comment thing?

might as well...it's pretty small :)

I think the changes to cfme-setup.sh in ManageIQ/manageiq-appliance#65 should handle it.

Aand here it is.. https://github.com/ManageIQ/manageiq-appliance-build/pull/118


Config: Enable multiple providers

Seemed like a good idea to support multiple different providers, and
possibly more than one simultaneously.

Coverage increased (+0.6%) to 94.262% when pulling e7f465edc1f996494e878b073041315dee9967e9 on config into 4f0cc1facbefe22535f21062cc73effe6981df58 on master.

Coverage increased (+0.6%) to 94.278% when pulling e93f7c2db7dafca64f139cbed9432f69a8ef03ff on config into 4f0cc1facbefe22535f21062cc73effe6981df58 on master.

Coverage increased (+0.6%) to 94.278% when pulling e93f7c2db7dafca64f139cbed9432f69a8ef03ff on config into 4f0cc1facbefe22535f21062cc73effe6981df58 on master.


Work in Progress: Builds styling

Not ready for merging!
There are still several things need doing before this is ready to merge, but I wanted to get the markup and styles in place.

[x] Replace placeholder markup with real markup based on build results.
[x] We should add a snapshot test now that this is becoming a real page.

Ok, I’ve wired this up and added the snapshot test. I’ll do the log links separately - good to merge!

Deploying to production as 1.4.2


Json.NET types not serialized properly

Objects of any type deriving from JToken are not being serialized with their values. All values are serialized as empty arrays. For example, an object with properties of types JValue, JArray, JObject fully populated with non-null values, will get serialized like this:
{
    "myJValue": [],
    "myJArray": [
        [],
        [],
        []
    ],
    "myJObject": {
        "a": [],
        "b": [],
        "c": {
            "x": [],
            "y": []
        }
    }
}

I posted a related question on Stack Overflow. Is this a bug? I'm aware you're using Json.NET under the hood and out of the box I know it doesn't have trouble serializing these types, so I'm wondering if it's something to do with your custom default serializer.

Hey @tmenier, I answered your question on Stackoverflow; let me know there if you need any further details 👍 I'm going to close this issue.

Thanks. With respect this being "expected behavior" (as you noted on SO), I think that depends a little on perspective. Regardless of any internal implementation details, when the values some dictionary-like thing (or object-like thing) are replaced by empty arrays, it's hard to see how the average user would expect that. I know it can't possibly know about every 3rd-party type that might be considered "dictionary-like", and it's probably just falling back on default behavior here, but an argument could be made that JToken/JObject are so ubiquitous in the .NET/JSON world that maybe they deserve special treatment (without installing a different package).

JToken implements IEnumerable<> though, so its not really a dictionary/object like thing as we'd might think of JToken.  JObject does implement IDictionary<string, JToken> so we do see the keys in the output as expected. So as far as known types it is behaving as expected.
One could argue that they are ubiquitous but unless these types ship as part of the BCL or default netstandard library I personally don't view these as something we need to support OOTB. We do support it however via an additional package which I think is the best default our users. It might not be your best default but for someone who would like to use JIL for their source serialization pulling in JSON.NET is not a good default either. The current setup serves both ends of the spectrum here.
Additional reasons for no longer depending on Json.NET by default are:

Your JSON.NET dependency is no longer tied to the whole client, you can now simply pick the NEST.JsonNetSerializer version that matches the JSON.NET dep you need.
Your JSON.NET configuration does not bleed into ours anymore when you pass a custom serializer.
We are free to internally switch away from JSON.NET and prebake all of our serialization routines.

Points well taken. "Knowing" about specific types from libraries it doesn't explicitly depend on does seem like something that shouldn't be expected, and exactly why NEST.JsonNetSerializer exists.
Regarding the interfaces that decedents of  JToken implement, (IEnumerable, IDictionary, etc.) NEST almost gets there without specific knowledge of Json.NET types. JValue might be the one specific case causing to to fall short. I'm reminded of something a little unorthodox I did in Flurl to try and infer if something is "dictionary-like" by looking for common property names (Key or  Name, Value). JObject has a Value property so something similar could work here. But again, what I did is a bit opinionated - what I call making the best attempt possible to infer intent, others might call a bad hack. :)


Mosaic Tiler

Mosaic Tiler will provide TMS endpoints into layers created by the Ingest
Inputs:

Set of layers to mosaic
Some ordering preference for mosaic (default: highest resolution goes on top
Resample method (default: bilinear)

Output:

Tile for z/x/y/ request

Critically this tiler will need to be able to mosaic layers that do not share the same maximum zoom level. For this it will need to find corresponding tile from the layer of lesser resolution and crop then resample it.
This component will not require spark as intent is to run it as light weight, stateless service that can be load balanced.
There may need to be considerations for caching as one tile of low resolution layer will be used for multiple tiles of high resolution layer. Incurring the naive IO cost in this case can be avoided.
It would be useful to assume that this component does not need to be tightly coupled to the spray route that will serve the tiles. Potentially the mosaic functionality can be re-used. Thus the API should provide for both "raw" tiles and colored tiles.
Questions

What coloring considerations need to be taken here?

Current use case is RGB imagery from Landsat
Likely use case is single band imagery

Does it make sense to start with the mosaic tiler? We'll need to have an endpoint that serves tiles only I think, without worrying about how mosaicing should work. Also, I'm not sure we have the mosaic UI done so that might make this task more difficult.
The color correction tile endpoints might be a better starting point as well and one that allows us to build up to mosaicing.
That's a good question about coloring considerations:

I suspect we'll want to allow people to define their own layer ordering eventually (to construct false color imagery to emphasize water, vegetation, land, etc.). If that makes things more difficult in the short-term then we should not do it and leave it as a separate task.
For single-band imagery most GIS tools I've used do grayscale so we'll likely want to leave that as an option

It probably makes sense to tackle as part of the first PR and back-scale if it becomes problematic, I wouldn't expect it to. This is something we did for phase 1 and even "put higher rez imagery on top" default rule or overall z-order is already useful without thinking through the UI and more complex specifications for a mosaic.
Good point about grey scale, as long as we're storing layer histograms we get that for free. I can see the color correction component also becoming nontrivial here. Possibly after the initial push here we need to start storing color-correction preferences in the datamodel.
Ideally we should still be able to encode everything into request to keep this stateless.

Got it. Yeah I tend to suggest we start from the simplest and work our way up. However, if you think a mosaic-tileserver isn't more difficult than a tile server just trying to serve a single layer then lets go for it.
I think the goal should definitely be stateless, so thinking about the URL and query parameters is a good idea.
For color correction, at least to start, that can be stateless until we have a better sense of how someone would want to store a set of color corrections (e.g. /layer/z/x/y?greenGamma=.09&minClip=.23 ...). I think the big (only?) use case we're talking about here is that someone would color correct one image and use that image as the source for histogram matching other images.

Thinking through this, having an endpoint that takes a scene instead of a scene deck and serves out tiles is more simple than one that takes a scene deck and layers images. We should probably start with the former so we can start seeing individual scenes that have been ingested, and avoid mosaicing logic. @notthatbreezy @echeipesh thoughts?

Sure, that'll at least allow us to look at wiring the ingest and the tiler at the same time. That by itself is helpful.

I'm on board with that as well, I think that's a good way to start.

@lossyrob and @echeipesh - should we create a new issue for the simple tile server task and color correction. Also, we should probably consult with @designmatty before too much more work is done on the mosaic tile endpoints so we have a good sense what support is necessary (he's working on the wireframe's this sprint). The color UI is already done so that might be an easier extension of a simple tile server.


Review2

Updates from Review  2. https://github.com/openjournals/joss-reviews/issues/5136

@feydan, @maryfnorris, or @shollingsworth. Is it alright if I bypass the merge protection for this? It's small and docs related but needed for finalizing our manuscript.


Use Mapping[str, Any] for localns parameter

As per https://github.com/python/typeshed/pull/12705.

Thanks!


I followed all the steps for setting up SQLite for Xamarin.Forms, and get this Exception when deploying the App in Android Phone via Xamarin Live.

Yes, i followed the instructions but it didn't help me with the issue. The problem was that i deployed the app to an Android Phone via Xamarin Live. When I connected the Phone via USB Cable, and than deployed it, it worked. And I didn't need to call the SetProvider method. Thank you anyway!


Add event to handle swipe to close

Add event to detect when a swipe gesture start to close the modal.
onClosed is triggered only once the closing animation is finished. onSwipeToClose allows to know when the closing animation is starting. It's useful in some case when you want to sync parallel animations.

I closed this because I've found this can crea


IOS - Webview opens but no contents inside (white screen)

Opened on an iPhone 6s but no content was displayed.

Hi, please provide some information when you open an issue. Otherwise we can't help you.
There're plenty of things that can make this, but if you don't post your iOS version, part of your code, extension version, and/or the things you already tried to do before asking for help, we can't help you.


chore: update image e2e test tag

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[x] N
[ ] Y

chore: update image e2e test tag

Pull Request Test Coverage Report for Build<PHONE_NUMBER>7
Details

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 15.591%

Totals

Change from base Build<PHONE_NUMBER>8:
0.0%

Covered Lines:
99

Relevant Lines:
635

💛  - Coveralls


Styleguide: Buttons (Simple)

Document buttons used within the app in a styleguide section
Includes:

default
primary
cancel
submit

Also:

define a Buttons.scss
define a Buttons.js in /styleguide/components/sections

Styleguide Index: #884
reference #877

The scss is going to be less complex (per button) if you create 3 (or 4 if you include cancel) React component buttons and associated scss files. Afterward, you should be able to get rid of every line of scss currently used to style any buttons anywhere in ~/../scss.
Requirements:

all buttons enter their changed state on hover and while clicked, but should return to their normal state immediately unmouseout (even if the button has not lost focus!)

this could mean dropping :focus or :active rules or both, not sure

default buttons are light gray with black text; hover turns the background gray
primary buttons are blue with white text; hover turns the background white, text blue, and border remains blue
submit buttons are not a special case in terms of styles, they should just be default
cancel buttons are blue link-like with no background or border; hover should only add text-underline

Would it make sense to call the buttons:

default buttons = Button
primary buttons = PrimaryButton
cancel button = LinkButton (or something?)

Looks funny to say DefaultButton all over the place and we have a lot of links that aren't technically cancellations of anything but otherwise share the behavior of CancelButton

How about the grayed-out .delete-button like on linodes/{label}/settings/advanced to delete a config?


[OpenCL] Warning: Use of non-host signal in host signal wait API (4.1.0)

Hello, after the update to 4.1.0 I am getting a lot of this warning when running opencl applications:
Warning: (!g_use_interrupt_wait || isIPC()) && "Use of non-host signal in host signal wait API." in virtual hsa_signal_value_t rocr::core::BusyWaitSignal::WaitRelaxed(hsa_signal_condition_t, hsa_signal_value_t, uint64_t, hsa_wait_state_t), <<private builds directory>>/hsa-rocr/src/ROCR-Runtime-rocm-4.1.0/src/core/runtime/default_signal.cpp:87
It looks like it is making reference to the directory it was built from.

Thanks @ivanmlerner for reaching out.
Can you please share your config like OS, Asic and all details of ROCm and also exact steps to reproduce the problem.
Thank you.

Hello, I am running Arch Linux with a Ryzen 3600X and an RX5500XT GPU. I know they are not supported, so if it has anything to do with that I am sorry for the trouble.
I am using amdgpu with rocm-opencl-runtime and rocm-smi installed from the AUR and packaged here.
I'll get more information and post them here.

Hi @ivanmlerner
I am not able to reproduce this here locally. I can not comment on arch linux, as we are not supporting it.
Thank you.


How to determine number of concurrent call supported per Logic App Standard plan Instance?

Since Logic App Standard plan instance is running on App Service plan, how to determine the max concurrent call supported e.g. triggers/second ( number of calls per second)? If user need high concurrency that exceed max concurrent connections on Logic App Standard plan, how to setup Logic App to meet the required concurrency? Is there any guidance available in this regard?

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 8d714050-c05a-cca9-ed86-fbd8ef223b4f
Version Independent ID: f02d2341-a0ad-19b1-edc1-d6f4f9cd3617
Content: Usage metering, billing, and pricing - Azure Logic Apps
Content Source: articles/logic-apps/logic-apps-pricing.md
Service: logic-apps
GitHub Login: @ecfan
Microsoft Alias: estfan

My confusion is because when I see Azure App Service plan limits, there is no details on inbound TCP/IP connections/second limit. App Service limit page has WebSocket connection limit, but I am looking for HTTPS inbound connections limit.

@Anand-Moghe Sorry for the late reply. The Trigger Concurrency settings should provide guidance: Runtime.Trigger.MaximumRunConcurrency sets to max concurrency and the Runtime.Trigger.MaximumWaitingRuns sets the max number of runs in wait after max concurrency exceeds.
Since we didn't determine any changes to this documentation upon reviewing this feedback, we will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Multiple Outputs Simultaneous

Since every Broadcast actor is associated with a Player Controller, I cannot send multiple views simultaneously. Instead you can only iterate over the same NDI source on the receiver side. Is there a way e to broadcast multiple camera views as separate NDI sources?

Hi,
try the NDI SDK Sample for Unreal Engine. You can define multiple Cameras/Sources in the scene. It seems not possible to create at runtime Camera/Sources.
It is possible to control the cameras as PTZ-Cameras with the Studio-Monitor WebInterface, if you implement the missing function in the sources.
T.

Thank you for your prompt response. There's no issue creating multiple camera sources the only thing is that at runtime only one stream (NDI Actor View point) is generated and available as NDI source. Only one NDI camera component can be instantiated when the player controller is loaded at runtime.  It should be possible I guess.

Well this is embarrassing, I forgot to apply the broadcast source to the broadcast component. It works perfectly.


devicelab: investigate down bots linux-14,24  mac-9,29

linux-14,24  mac-9,29 have been dead for a few days, investigate/recover them.

ssh is not working into these machines.

linux-25 is stuck on bios screen, looks like it could be SSD failure. Created https://github.com/flutter/flutter/issues/111216 to swap out.

mac-9 rebooted and its back.

win-3 was stuck on password needs to be changed screen. Reset password, its back up.

Same issue with linux-14 as linux-25:
Port2: BIWIN SSD
S.M.A.R.T. status bad, backup and replace.
created https://github.com/flutter/flutter/issues/111217 to track.

mac-29 is working properly.


Send alerts and notifications to K8s Slack

To automatically report issues, alerts should automatically send to slack. The release-ci-signal k8s slack channel can be used to publish alerts & notifications.
Slack webhooks: https://api.slack.com/messaging/webhooks

/kind feature

/assign @encodeflush

FYI: issue to request Slack channel|usergroup|bot|token|webhook

/assign @encodeflush


Show doi link prominently in record detail views

And don't display resource type and status on top right.

Implemented via 65b5911.


TypeError: this.pattern.map is not a function

Environment
Production
2.2.0
Steps to Reproduce
I have a webpack.config.js file with some plugins, the last one that runs is sentryWebpackPlugin. I am trying to upgrade our build process to have the new debug_ids sourcemaps. Getting the error while running with the following options
 sentryWebpackPlugin({
          // sentry-cli configuration
          authToken: config.get('SENTRY.AUTH_TOKEN'),
          org: config.get('SENTRY.ORG'),
          project: config.get('SENTRY.PROJECT'),
          release: packageJson.version,
          sourcemaps: {
            deleteFilesAfterUpload: true,
            assets: './extension/chrome/js/**',
            ignore: [
              'node_modules',
              'webpack.config.js',
              'scripts',
              'lib',
              'coverage',
            ],
          },
        }),

I removed the deleteFilesAfterUpload:true option and it ran fine.
Expected Result
Build process finishes without errors
Actual Result
.../develop/node_modules/@sentry/bundler-plugin-core/node_modules/glob/src/glob.ts:457
    const mms = this.pattern.map(p => new Minimatch(p, mmo))
                             ^
TypeError: this.pattern.map is not a function
    at new Glob (.../develop/node_modules/@sentry/bundler-plugin-core/node_modules/glob/src/glob.ts:457:30)
    at AsyncFunction.glob (.../develop/node_modules/@sentry/bundler-plugin-core/node_modules/glob/src/index.ts:120:10)
    at _callee2$ (.../develop/node_modules/@sentry/bundler-plugin-core/src/debug-id-upload.ts:122:41)
    at tryCatch (.../develop/node_modules/@sentry/bundler-plugin-core/dist/cjs/index.js:112:17)
    at Generator.<anonymous> (.../develop/node_modules/@sentry/bundler-plugin-core/dist/cjs/index.js:193:22)
    at Generator.next (.../develop/node_modules/@sentry/bundler-plugin-core/dist/cjs/index.js:137:21)
    at asyncGeneratorStep (.../develop/node_modules/@sentry/bundler-plugin-core/dist/cjs/index.js:374:24)
    at _next (.../develop/node_modules/@sentry/bundler-plugin-core/dist/cjs/index.js:393:9)
    at runMicrotasks (<anonymous>)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
error Command failed with exit code 1.

@lforst   I've got to say, in my opinion deleteFilesAfterUpload is a bad name. It really seems like it would be a boolean.

@lforst I've got to say, in my opinion deleteFilesAfterUpload is a bad name. It really seems like it would be a boolean.

@maccman suggestion?

@lforst something like filesToDeleteAfterUpload would suggest that it takes in an array of glob patterns / string

@loxator Sounds good to me! Thanks! https://github.com/getsentry/sentry-javascript-bundler-plugins/pull/313


Deprecate IContainerRuntimeOptions.flushMode

Description
Fixes AB#26497
Only the default value FlushMode.TurnBased is supported when calling ContainerRuntime.loadRuntime directly, so there's no need for consumers to pass this option in.  We'll remove the option altogether in 2.20.0.
Reviewer Guidance
Note that FlushMode.Immediate is still used in compatibilityConfiguration.ts, for 1.x compat mode.  #23272 adds an internal-only set of options which includes flushMode, to support that use.  But in the meantime we can finish removing it from the legacy-alpha API surface.

@jzaffiro -- want to review this changeset?  Similar to #23260

@tylerbutler thanks for the feedback, I took your suggestions.


Question: How to integrate real lexer

Hi there!
I'm writing a kind of framework, that allows you to generate lexer/scanner, parser and code highlighters from a kind of language definition / grammar file.
(Something like Xtext, ANTLR, but writtin in JavaScript and generating JavaScript code)
The lexer generator is already available and the generated lexer/scanner is working fine.
Getting the written code out of the CodeMirror editor instance is quite easy.
After the lexer has completed, I know where all my tokens start and end.
I have the following position information about each token:

start position (absoulte position in the input stream)
line number (number of line where the start of the token appeared)
line column (where exactly inside the line the token started)

If required, it should be possible to add more position information.
There is only one (hopefully little) thing missing: Highlighting tokens.
Is there something available, that would mark token A (start at position x, with length y, and z lines involved) as a keyword, terminal symbol etc. ?
Thanks!

Was also thinking about this. Seems a lot of effort is duplicated when writing CodeMirror rules and for an ANTLR grammar. Not sure how difficult this would be, but it would be nice if you could supply ANTLR lexer/parser rules with accompanying token names to CodeMirror. I guess the tradeoff would be, as you say, the performance hit of running the ANTLR code each time you need syntax highlighting.

Yes, running all of ANTLR doesn't seem like a great idea. It should be possible, and not terribly hard, to write a tool that converts declarative ANTLR token syntax into a lightweight CodeMirror mode, but I don't think anyone has done that so far.
(In CodeMirror 6, I'm trying to provide a more declarative syntax definition tool with lezer.)

If someone is interested in the integration of Codemirror 6 and Antlr4 grammar,
you can find an example on github page.
The main idea is here:
token: (stream, state) => {
        // getting tokens for the current stream.string, using antlr4 TokenStream
        const tokens = getTokensForText(stream.string);
        // getting the next token
        const nextToken = tokens.filter(t => t.startIndex >= stream.pos)[0];
        // matching the next token in current stream
        if (stream.match(nextToken.text)) {
            let valueClass = getStyleNameByTag(tags.keyword);

            switch (nextToken.type) {
                case ...:
                    valueClass = getStyleNameByTag(tags.string);
                    break;
               ...
                default:
                    valueClass = getStyleNameByTag(tags.keyword);
                    break;
             }

            return valueClass;
        } else {
            stream.next();
            return null;
        }
    }


[#768] feat(cli): Cli method for blacklist update

What changes were proposed in this pull request?
I created a command called uniffle cli to update blacklist, the following is the case of using the command.
Usage:
   Optional
     -a,--admin <arg>   This is an admin command that will print args.
     -c,--cli <arg>     This is an client cli command that will print args.
     -h,--help          Help for the Uniffle CLI.
     -rc,--refreshChecker    This is an admin command that will refresh access checker..
     -cc,--checkerClass <arg>     This is an client cli command that will print args.
     -s,--help <arg>          This is coordinator server host.
     -p,--port <arg>          This is coordinator server port.

Run the example-cli command

uniffle admin-cli -rc -cc org.apache.uniffle.test.AccessClusterTest$MockedAccessChecker -s localhost -p 12345
Why are the changes needed?
We use refreshChecker cli to update blacklist.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Unit test verification.

@smallzhongfeng @zuston Could you review this pr?

You can run mvn -B -fae checkstyle:check -Pspark3.

You can run mvn -B -fae test-compile spotbugs:check -Pspark3, too.

You can run mvn -B -fae test-compile spotbugs:check -Pspark3, too.

OK

Codecov Report

Merging #931 (d201d97) into master (fb8f4cf) will increase coverage by 1.74%.
The diff coverage is 21.05%.

@@             Coverage Diff              @@
##             master     #931      +/-   ##
============================================
+ Coverage     55.34%   57.09%   +1.74%
+ Complexity     2272     2191      -81
============================================
  Files           344      317      -27
  Lines         16982    14067    -2915
  Branches       1349     1290      -59
============================================
- Hits           9399     8031    -1368
+ Misses         7038     5603    -1435
+ Partials        545      433     -112

Impacted Files
Coverage Δ

...a/org/apache/uniffle/client/HttpClientFactory.java
0.00% <0.00%> (ø)

...java/org/apache/uniffle/client/RestClientConf.java
0.00% <0.00%> (ø)

...java/org/apache/uniffle/client/RestClientImpl.java
0.00% <0.00%> (ø)

...a/org/apache/uniffle/client/UniffleRestClient.java
0.00% <0.00%> (ø)

...uniffle/client/exception/UniffleRestException.java
0.00% <0.00%> (ø)

...nator/web/servlet/admin/RefreshCheckerServlet.java
15.78% <15.78%> (ø)

...c/main/java/org/apache/uniffle/cli/UniffleCLI.java
68.65% <76.66%> (+9.19%)
:arrow_up:

...main/java/org/apache/uniffle/api/AdminRestApi.java
100.00% <100.00%> (ø)

.../apache/uniffle/coordinator/CoordinatorServer.java
58.64% <100.00%> (+0.31%)
:arrow_up:

...rdinator/access/checker/AbstractAccessChecker.java
66.66% <100.00%> (-33.34%)
:arrow_down:

... and 35 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

BTW, the auth is not enabled in current implementation, especially for admin cli

BTW, the auth is not enabled in current implementation, especially for admin cli

I don't have any ideas, could you provide some hints or suggestions?

You should exclude the http depenecy for hadoop common.

 You should exclude the http depenecy for hadoop common.

OK.

BTW, the auth is not enabled in current implementation, especially for admin cli

@xianjingfeng I don't have any ideas, could you provide some hints or suggestions?Thank you very much.

BTW, the auth is not enabled in current implementation, especially for admin cli

@xianjingfeng I don't have any ideas, could you provide some hints or suggestions?Thank you very much.

I don't think we need to implement it in this pr, we can do this after #939

BTW, the auth is not enabled in current implementation, especially for admin cli

@xianjingfeng I don't have any ideas, could you provide some hints or suggestions?Thank you very much.

I don't think we need to implement it in this pr, we can do this after #939

I agree.

@smallzhongfeng @zuston @slfan1989 Could you review this pr?

@jerqi Sorry for the late reply.  I quickly look it and don't have any other comments. However, have we considered introducing an RPC service in the CLI?  In typical scenarios, we often need to use RPC calls in the CLI to request server-side operations rather than using HTTP methods.

@smallzhongfeng @zuston @slfan1989 Could you review this pr?

@jerqi Sorry for the late reply. I quickly look it and don't have any other comments. However, have we considered introducing an RPC service in the CLI? In typical scenarios, we often need to use RPC calls in the CLI to request server-side operations rather than using HTTP methods.

We already have some http restful api now. Our k8s operator will use these api, too. It's more convenient to use restful api than rpc calls. If we don't have another need, we won't add rpc calls.  It will increase maintenance cost.

We already have some http restful api now. Our k8s operator will use these api, too. It's more convenient to use restful api than rpc calls. If we don't have another need, we won't add rpc calls. It will increase maintenance cost.

@jerqi Thanks for the explanation! I have no other doubts.

@zuston , @xianjingfeng , @jerqi  and others, thank you for your guidance!


Google sign in - silentSignIn is failing (SIGN_IN_REQUIRED / INTERNAL_ERROR)

Describe your environment

Device: Multiples (Samsung, Google, Huawei, Xiaomi...)
OS version: Multiples (12, 11, 10, 9..)
SDK version: 33
[Android] Google Play Services version: 20.2.0

Describe the problem:
Steps to reproduce:
Couldn't reproduce.
Observed Results:
For some of our users, the silentSignIn method is failing with the error code SIGN_IN_REQUIRED or INTERNAL_ERROR during our auto-login process when starting the app (Splash screen), hence we are not able to send to our backend a valid token.
This past month we had 2k of SIGN_IN_REQUIRED and 3k of INTERNAL_ERROR non-fatals coming from our production users. So far we are not able to reproduce it, neither understand in which scenario it can happen.

Expected Results:
To always received the idToken, existing one if still valid or a new one if expired.
Relevant Code:
How we are trying to get a fresh token during the auto-login process :
val googleSignOptions: GoogleSignInOptions = Builder(GoogleSignInOptions.DEFAULT_SIGN_IN)
        .requestIdToken(serverClientId)
        .requestEmail()
        .build()
val googleSignInClient = GoogleSignIn.getClient(safeActivity, googleSignInOptions)
googleSignInClient.silentSignIn().addOnCompleteListener { task ->
    try {
        val account: GoogleSignInAccount? = task.getResult(ApiException::class.java)
        account?.idToken?.let { idToken ->
            // return success with the idToken
        }
    } catch (exception: ApiException) {
        // return the exception
    }
}

This is affecting badly the user experience as those users needs to sign-in again inside the app.
Can you share which workflow we should use to improve this experience for our end users ?
Thanks

@eloisance Did you find any solution for this? facing same issue.


My video screen is not smooth

I'm using rasipberry3B (1G memory) ,but i find my picture is not smooth can you tell me why

If you are talking about the framerate then it depends on how high your resolution is mainly, you could try to reduce it.

Hey i am glad you reply to me. But I have another question .When i using you code,my palm can't be detected.And I don't know what's the meaning for INC S DEC S or MAX VCan you tell me what i should change for these parameters.
------------------ 原始邮件 ------------------
发件人: "Raphael<EMAIL_ADDRESS>发送时间: 2019年9月5日(星期四) 晚上9:21
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [Shinao/SmartMirror] My video screen is not smooth (#11)
If you are talking about the framerate then it depends on how high your resolution is mainly, you could try to reduce it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

It's the HSV values of the camera capture (mainly used for checking brightness), check out the code if you need more infos.
Also unless you really need the gesture system I suggest you look to another SmartMirror repository, this one is becoming quite old and others are a lot more community oriented than my simple personal project.


feat (members): Use new API to add user to project instead of creating policy directly

ID-449
Problem statement:
Call an abstraction API instead of calling the policy creation API directly as it could be a security issue.
Solution:
Using V1Beta1CreatePolicyForProjectBody instead of frontierServiceCreatePolicy.
Note:
There are other occurrences of frontierServiceCreatePolicy API in the project which needs to be replaced with appropriate APIs.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

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 9.383%

Totals

Change from base Build<PHONE_NUMBER>0:
0.0%

Covered Lines:
10195

Relevant Lines:
108657

💛  - Coveralls


Feature/bl 138 mentor matching component

Description
We completely scraped out the accordion and made it functional to where you can assign a mentor to a mentee. We decided to go a different route and have a model pop up with a table with all the mentor's names, info, stack, notes, and a number of mentees assigned to that mentor. We would have loved to add more functionality to this but due to being limited with the back end, we had to work with what we got and have someone else finish the job when there is an improved back end.
Fixes # Made it so you can assign several mentors to a mentee all at once and make it look better visually
Loom Video
Please paste a link here of a quick loom video walking through what you did in your PR .
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

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 removed unnecessary comments/console logs from my code
[x] I have made corresponding changes to the documentation if necessary (optional)
[x] My changes generate no new warnings
[x] I have checked my code and corrected any misspellings
[x] No duplicate code left within changed files
[x] Size of pull request kept to a minimum
[x] Pull request description clearly describes changes made & motivations for said changes

just waiting on the loom video now

Nice work saving mentor matching data to state. We can now work on updating these changes to the back end. I don’t see any issues.


Install Issues on OpenShift

Hi there,
I'm running into some issues with installing (or deploying) hiredis on OpenShift. I can successfully install on Ubuntu 12.04 LTS, using the same node (v0.10.35) and npm (1.4.28) version. But the build on OpenShift (Red Hat Enterprise Linux Server release 6.6) gives the following error. Could anyone point me to what might be missing on OpenShift?
[freestyle@freestyle-ose fmemanager]$ npm install hiredis
/

<EMAIL_ADDRESS>install /home/freestyle/openshift/online/fmemanager/node_modules/hiredis
node-gyp rebuild

Traceback (most recent call last):
File "/opt/rh/v8314/root/usr/bin/gyp", line 15, in
import gyp
ImportError: No module named gyp
gyp ERR! configure error
gyp ERR! stack Error: gyp failed with exit code: 1
gyp ERR! stack     at ChildProcess.onCpExit (/opt/rh/nodejs010/root/usr/lib/node_modules/node-gyp/lib/configure.js:343:16)
gyp ERR! stack     at ChildProcess.emit (events.js:98:17)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (child_process.js:820:12)
gyp ERR! System Linux 2.6.32-504.23.4.el6.x86_64
gyp ERR! command "node" "/opt/rh/nodejs010/root/usr/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/freestyle/openshift/online/fmemanager/node_modules/hiredis
gyp ERR! node -v v0.10.35
gyp ERR! node-gyp -v v1.0.2
gyp ERR! not ok
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is most likely a problem with the hiredis package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls hiredis
npm ERR! There is likely additional logging output above.
npm ERR! System Linux 2.6.32-504.23.4.el6.x86_64
npm ERR! command "node" "/opt/rh/nodejs010/root/usr/bin/npm" "install" "hiredis"
npm ERR! cwd /home/freestyle/openshift/online/fmemanager
npm ERR! node -v v0.10.35
npm ERR! npm -v 1.4.28
npm ERR! code ELIFECYCLE
npm ERR! not ok code 0

File "/opt/rh/v8314/root/usr/bin/gyp", line 15, in
import gyp
ImportError: No module named gyp

It seems to invoke gyp in the mentioned path, which fails for some reason.
Normally node-gyp should invoke the gyp it comes with, but maybe because you have a global one installed, this fails.

Just an update. The problem is with the Node version on OpenShift. Here is my finding for anyone else that might be facing the same issues:

Installs, builds and runs fine on Ubuntu 12.02 (Node: v0.10.35)
Fails building on OpenShift Online (same Node version), therefore cannot deploy.
I used nvm to explicitly install Node v0.10.35 locally and it builds and runs fine on RedHat and Ubuntu, and deploys to OpenShift Online without an issue.


@Reference annotation ignore the @Method annotation

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

Environment

Dubbo version: 2.7.4-SNAPSHOT
Operating System version: MacOS
Java version: java11

Steps to reproduce this issue

Config reference like

    @Reference(id = "referenceId1" , methods = {@Method(name = METHOD, timeout = 7000, arguments = @Argument(index = 0))})
    private DemoService demoService;

debug the application, observe if the methods config works

Expected Result
the methods config works
Actual Result
the methods config doesn't work
If there is an exception, please attach the exception trace:
I find that it seems the code following generate the effect
package org.apache.dubbo.config.spring.util;

public class AnnotationUtils {

in the method public static Map<String, Object> getAttributes(Annotation annotation, PropertyResolver propertyResolver, boolean ignoreDefaultValue, String... ignoreAttributeNames)
            if (attributeValue.getClass().isAnnotation()) {
                continue;
            }
            if (attributeValue.getClass().isArray() && attributeValue.getClass().getComponentType().isAnnotation()) {
                continue;
            }

Should we delete these code ?

has remove it https://github.com/apache/dubbo/pull/4598

fixed


JS: All test assertion functions now shows the input by default

Apparently now every test assertion functions show the input by default, including Test.assertEquals and Test.assertSimilar.
This means that all the performance katas that tests between two massive objects/arrays will break because the amount of stdout output will terminate the process. The numbers are at least in the hundreds.

Related: https://github.com/Codewars/codewars-runner-cli/issues/599

What do you mean by "now"? cw-2 hasn't changed.
By the way, Node 10 will be added soon and it'll remove cw-2. To make the transition smooth, assertion methods under Test are still provided. So many of the existing contents will work without any change. If tests are not using describe/it, it needs to be updated in order to use Node 10. Node 8 will continue to work the same way.
The test framework will be Mocha under the hood with cw-2 style assertions. Users can now use assertion libraries like Chai. Assertion methods under Test are now just wrappers for Chai.
This will resolve most issues with cw-2.


Support parameter attributes inside function brackets

Add support for parameters annotations inside the () function brackets.
It isn't good practice but it works with psalm and phpstorm.

Thank you for the issue! Added @var support for promoted properties in 0.3.0.


Improve user experience.

Create needed Snowflake objects within Airflow DAG. Add cross-DAG dependencies.
Add _producer and _schemaURL to the OpenLineage event.

@sfc-gh-kbijon @Sheeri, would it be possible for either of you to review and merge the code? Thank you in advance for your help!

@sfc-gh-kbijon @Sheeri bumping this kindly

I don't have write access so it's not appropriate for me to review.


Feature Request: wysiwyg editor for issues and wiki

Gitea version (or commit ref): 1.5.1
Git version: All supported versions
Operating system: All supported platforms
Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[ ] Not relevant

Log gist:

Description
Please add some sort of markdown wysiwyg editor to wiki and issue input boxes.
Something like benweet/stackedit for example.
Screenshots

For wiki there already is one

@lafriks Wiki's editor is not WYSIWYG

I think it is already close enough to have it easy to use

It's not about ease but experience.
Editing and previewing simultaneously is a better experience than edititing then previewing.
However, some complex elements like tables, are much easier to edit with a WYSIWYG editor.
Less significantly, images and links are easier too.
By the way, inline code and checkboxes are missing from the current editor. I noticed that but other elements may be missing as well

Implementing new WYSIWYG editor in gitea is totally out of scope for this project. If there is any JavaScript library that is better than currently used we could look into it

I think it Maybe like typora ?

It's not about ease but experience.
Editing and previewing simultaneously is a better experience than edititing then previewing.

There is a problems with an live preview (Editing and previewing):

currently markdown is rendered by https://github.com/russross/blackfriday which is an internal go library
the live js implementation has to render the result exactly like the internal go variant, otherwise the user experience would be even worse.

the previewing button request gitea to render the text to html and show it back on website.
May you suggest some other CodeMirror theme or rules or something what should really be changed in edit mode. But I wouldn't like to see really colored markdown like here: https://codemirror.net/mode/markdown/.

I think it Maybe like typora ?

if i see it correctly 'typora' is standalone and no js library

I think this issue is more like #6975

the previewing button request gitea to render the text to html and show it back on website
How would that make live rendering impossible ?

I never said impossible, but

Let Go do the rendering:

this will trigger rendering with every keypress

may the update could be delayed to time interval, or X changed letters)

how do you integrate the returned html code into your text editor as it will return the whole text

A solution would be an additional container where the preview is shown. As it does not effect the editor at all.

Let JS do the rendering:

The JS render implementation has to do the same thing as the go code, otherwise it will render differently on live preview and when it is finished. This would be an even worse solution.

The codemirror (used by simpleMDEditor) currently does some simple highlighting.

However, some complex elements like tables, are much easier to edit with a WYSIWYG editor.

The current editor has a simple feature to add tables.
Are you looking for something like https://codepen.io/jasondavis/pen/eJBoeK ? Such things could be added to codemirror too. Or something else?

Less significantly, images and links are easier too.

You could simply paste links. Image pasting could be easily achieved too, if upload is working at all.

By the way, inline code

I don't know if it will make any sense to add inline code button if it could be done by simply adding ` around your code. Are there keyboards without this key?

and checkboxes are missing from the current editor.

* [ ] checkbox

There is a codemirror plugin called 'button' or 'buttons'. It allows to simply add some buttons and code there, by adding some lines to an array. May I will add some buttons later on to support

inline code
checkboxes

simpleMDE actually support side-by-side view -> which give live preview (and runs request by every keypress)
I will create an pr to anable this feature and i will delay the preview like mentioned above

this will trigger rendering with every keypress
may the update could be delayed to time interval, or X changed letters)

Smarter : we could trigger rendering when the user stops typing for 0.5s

how do you integrate the returned html code into your text editor as it will return the whole text
A solution would be an additional container where the preview is shown. As it does not effect the editor at all.

Even smarter : we could use contenteditable and only trigger rendering whenever the user creates a new block.

added an pull request to enable sideby side editor

Smarter : we could trigger rendering when the user stops typing for 0.5s

currently it will render on every 10th keystroke or with 1sec delay, i will reduce the delay to 500ms.

Even smarter : we could use contenteditable and only trigger rendering whenever the user creates a new block.

Codemirror will create elements on every line, and on every styled text element, so this could be quite often. (e.g. inserting an image by this: ![alias](image.png) will create 4 html elements. (and if i bind to that event, the renderer may get triggert 4 times))
Internal Codemirror does something like contenteditable on html elements. While updating an internal raw text state and tracks every keystroke, it will also update an rendered preview.

added PR for

inline code
checkboxes

too

@KaKi87 As you requested something for tables, i've written some jquery extension for it.
If you like it, i would create a PR to add this to simpleMDE/codemirror.
(First i will add this to wiki page, if all is done there, i will add all features to issues and normal markdown editor)
Running version could be found here:
https://jsfiddle.net/cherrg/kywdz0jc/
Features

create md tables of selected size
editable table
mark and edit tables
text alignment inside tables

parse and creates markdown tables from

markdown tables
html tables
csv
text based tables (copied by clipboard)

Images

Hi. What's the status of this issue? I really think this is a must-have feature if we want non-technical users to be able to report issues easily.

@Cherrg your commit (https://github.com/go-gitea/gitea/issues/5436#issuecomment-503757065) is worth a own issue ... i think
@callamaris gitea v1.11.0 will have wysiwyng editor for issue and wiki - can we close this issue?

@callamaris can you close this issue


Add Codium support to C# external editors

Selecting the VS Code option will now work if Codium is installed, picking it as the last option after attempting to run VS Code.
Not tested on any OS, so please test on macOS and Windows or Linux before merging.

This closes https://github.com/godotengine/godot-proposals/issues/6518.

Thanks!


Use proper minimum versions of cc and pkg-config

The version specifications for both cc and pkg-config are too low. Version 1.0.0 of cc, for example, is lacking the cflags_env() method [0] entirely and pkg-config 0.3.0 fails to compile on stable altogether, requiring nightly features and showing other compile errors. Bump both to the actual minimum supported versions.
[0] https://docs.rs/cc/1.0.78/cc/struct.Tool.html#method.cflags_env
Signed-off-by: Daniel Müller<EMAIL_ADDRESS>

^I.J (for I>0 or J>0) — equivalent to ^I.J.0

TIL.


This page is out of date

The information provided here is not accurate. The Communications Credits have been moved somewhere and are no longer able to be purchased as described above.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0d8f6008-136f-52a1-70c8-f721f1be0491
Version Independent ID: efaeb679-a1c9-ab98-9f0b-f253a19cecec
Content: Add funds and manage Communications Credits
Content Source: Teams/add-funds-and-manage-communications-credits.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

@dschuette01 @ Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@officedocsbot assign @scanum

@dschuette01 Thank you for pointing this out and helping improve MS Docs.  Are you suggesting retiring the whole article or just removing the section about the subscription page and communication credits?

@dschuette01 We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.

Why was this closed without updating the out of date documentation?


Extend Builtins with Steel null and is_null

As discussed today, Steel.Reference is not extracted by KreMLin, hence, the null and is_null functions that are not rewritten by FStar.Extraction.Kremlin.fs need to be provided as builtins to KreMLin.
This PR also marks the warning 2 as an error on the Steel tests to detect possible regressions.

Were you able to confirm the warning 2 is gone? If so, please remove steel_reference from the list here: https://github.com/FStarLang/kremlin/pull/223/files#diff-8a10cfa26a87bbd37d3c4409e456d459faa850823bca2c9220008a17a24d79e7L307 then go ahead and merge! Thanks for catching this!

The warning 2 is gone, but only if steel_reference is in the list of builtins so that Steel.Reference.null and is_null have a model.

As discussed on Slack, I removed steel_reference from the "hand_written" list since it's unneeded there.
Thanks!


feat(infra.ci): new tokens for dockerhub push and pull infraci

as per jenkins-infra/helpdesk#2837

Should the DOCKER_HUB_USERNAME variable be also removed from charts-secrets or is it used elsewhere?

It has been deleted in https://github.com/jenkins-infra/charts-secrets/commit/0f99e4e23eade88b9f7c2c613d93770eaf9b4dd0 :)


Java indexer: remove legacy documentation anchor nodes [phab:D1984]

Created by schroederc at 2017-10-20 21:53:19:
This is the Java component of #2437.

schroederc wrote at 2017-10-20 22:03:16:
Add regex end marker

zarko accepted this revision at 2017-10-20 22:04:58

schroederc closed via commit 7a222de15ba2387c74d46664f5fec0280d32c6ed at 2017-10-20 22:06:52


Rename port_type to protocol on rules

This will better map to what the field actually represents and improve UX for both the UI and upcoming REST API.

[ ] DB migration
[ ] UI field changes
[ ] Docs updates

refs #903

Closing since this will be changing with network policies.


TFRS - New Act Label Changes - IDIR Director Home Dashboard

Describe the task
Changing labels on the Home Dashboard for IDIR users with the Director/Deputy Director role.
Labels to change:
See attached document for description of label changes:  https://app.zenhub.com/files/80058415/3003733d-b143-4126-9541-9ba3b9c33598/download
Wireframe:
*

Note: wireframe does not include the one revision on the main nav bar to change 'Fuel Codes' label to 'Administration'. However, this change is listed in the 'labels to change' attached document.

Purpose
Changes labels and messaging to align with the language and framing used in the new Act.
Acceptance Criteria

[ ] first
[ ] second
[ ] third

Additional context

Add any other context about the task here.
Or here

Hey team! Please add your planning poker estimate with Zenhub @AlexZorkin @jig-patel @prv-proton


Type mismatch in generated externs

Closure complains in OL3-Cesium with the following message:
ERR! compile ol3/build/ol-externs.js:3727: ERROR - mismatch of the getSource property type and the type of the property it overrides from superclass ol.layer.Layer
ERR! compile original: function (this:ol.layer.Vector): (null|ol.source.Vector)
ERR! compile override: function (this:ol.layer.VectorTile): (null|ol.source.VectorTile)
ERR! compile ol.layer.VectorTile.prototype.getSource = function() {};

I see that ol.layer.VectorTile extends ol.layer.Vector.
However ol.source.VectorTile does not extend ol.source.Vector.

The same issue occurs in OL3-Google-Maps.

For information, we are no more relying on OpenLayers externs for OL3-Cesium.
Instead, we compile OL3 + OL3-Cesium code together.

@adube Can we close this?


Finished Server Test Code

73% lines covered in server package. Most lines in catch blocks cannot be tested.

Resolved #46


Thieving room (Raids)

Could a number be added above the Trough showing how many worms have been deposited? And maybe a timer showing when the next bar change happens?

We cant really count how many worms have been deposited as other players can deposit them too, and we cant determine bar change because of that too


Revisit of the representation of adjacently tagged enums tag

This PR is a follow on #2496.
This is a proof of concept to evaluate if such feature would be interesting.
I've run some tests with serde_json and the output is the same, but still allow some format to take advantage of variant_index during serialization.
If it looks promising I can continue to work on it

I did not realised that DeserializeSeed was imported only with feature = "std" so I had some trouble fixing the import.
Does something like this seams more appropriate ?

Is there a crate implementing a binary format that use variant_index to serialize enums and support adjacently tagged enums deserialization ?
I've looked into postcard but it does'nt support deserialize_identifier so adjacently tagged enums are not supported (maybe #2475 broke it?).
And I looked at ciborium but it serialize enums with the variant name.

Is there a crate implementing a binary format that use variant_index to serialize enums and support adjacently tagged enums deserialization ?

I don't know of any. But if there is, then yes, we would have heard about it breaking from #2475.

This PR is still marked as draft. Are there concrete changes planned (what are they?), or was this supposed to be ready for review?

@dtolnay I just forgot to open it for review, there is no changes planned

There is conflicts needed to be resolved, I'm gonna need some time to understand why thoses changes has been made and what can I do to not break anything.

All conflicts were related to the removal of try! in #2539

This PR has some breaking changes that results in failing tests in quick-xml (https://github.com/tafia/quick-xml/issues/630)

Since this is a breaking change, why was it released as a patch instead of a minor version?
v1.0.180
In my case, an enum
#[derive(Serialize, Deserialize, Debug)]
#serde(tag = "type", content = "data")]
enum Message {
  A(i32),
}

let m = Message::A(100);

was rmp_serded to
[130, 164, 116, 121, 112, 101, 161, 65, 164, 100, 97, 116, 97, 100]

which the JS msgpack package converts to JSON as
{
  "type": "A",
  "data": 100
}

with serde v1.0.180.
v1.0.181
Same enum and attributes, rmp_serde spits out
[130, 164, 116, 121, 112, 101, 129, 161, 65, 192, 164, 100, 97, 116, 97, 100]

and JS msgpack converts to
{
  "type": {
    "A": null
  },
  "data": 100
}

serde_json
What is further confusing this is that converting this to JSON with serde_json results in
{\"type\":\"A\",\"data\":100}

regardless of the serde version.
Can you please shed light on this?


documentation for polymorphic relationships

It will be good to add documentation for polymorphic relationships, especially for the json format - according to the specs there are 3 acceptable formats of polymorphic resource notation.

+1


19 | Remove Nth Node From End Of List | Java

Title - Remove Nth Node From End Of List
What will change - New Java code will be added
Type of Issue -
Please add/delete options that are not relevant.

[x] Adding New Code
[ ] Improving Code
[ ] Improving Documentation
[ ] Bug Fix

Programming Language
Please add/delete options that are not relevant.

[ ] Python
[ ] C++
[x] Java
[ ] SQL

Self Check

Ask for issue assignment and wait to get assigned to it before making Pull Request.
Add your file in the proper folder
Clean Code and Documentation for better readability
Add Title and Description of the program in the file

:star2: Star it :fork_and_knife:Fork it :handshake: Contribute to it!
Discord server ✉ - https://discord.gg/CXksGdn
Happy Coding,

@Khushaliketan Can you assign this to me? thank you!

Can you assign this to me

@rathovarun @IsuruAriyarathne I don't have write access in this repo. Besides, I am waiting for the maintainer to assign myself. Good luck! :)

@SSKale1 request to merge the pull request and add hacktoberfest-accepted tag, thanks!

please assign this task to me.


Added documentation for opc task.

Added documentation for opc task and edited the README.md file to point to the respective docs.
Signed off by @Senjuti256<EMAIL_ADDRESS>

@savitaashture

/approve
/lgtm