Add Missing Class to Info Only Welcome Mat to Decrease Title Text Size

The title text for the DSR Info Only Welcome Mat should be 32px to align with SLDS (https://www.lightningdesignsystem.com/components/welcome-mat/#Info-only).
Adding slds-welcome-mat_info-only to <div class="slds-welcome-mat"> will make the title text 32px instead of 42px.

@kwilloughby-sf has red-lined/design audited the Welcome Mat. @aswinshenoy Please add this class based on the info-only variant.

Sure :) I will send PR by tonight.


Update Filesystem.php - sourcery

Summary by Sourcery
Enhancements:

Make the Config and PathNormalizer properties readonly in the Filesystem class.

@sourcery-ai

Hey @VicTest1234 - I've reviewed your changes - here's some feedback:
Overall Comments:

Please remove the unnecessary whitespace after normalizePath($location)

Here's what I looked at during the review

🟢 General issues: all looks good
🟢 Security: all looks good
🟢 Testing: all looks good
🟢 Complexity: all looks good
🟢 Documentation: all looks good

Sourcery is free for open source - if you like our reviews please consider sharing them ✨
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.

anything else?

@sourcery-ai any other recommendations?


Fix color-picker sliders

I guess these hacks are relics from HSV mode.

Thanks!

Raw mode was suposed work that way to be able to make colors with componnts greather than 1 and use them on Multiply operations
CC @reduz he explained it to me earlier

Ah right, so indeed only the alpha channel should be limited. So should it be changed back to how it was, with just the addition of the step of 1 for the non-raw mode?

the step of 0.1 is un purpose too, otherwhise you can only move from 1 on 1 making it less usefull for color operations

Thanks, I'll revert the change for now then. We can continue discussing if there's a need to actually change some of those lines or not.

Ah, I see. But it looks still like very broken design for me ;) The arrows doesn't
work reliably in the RGBA mode, the slides for that kind of raw mode are more than useless.
Why not just use line_edit to entry 4 arbitrary float values?

Nothing prevents you from using the spinboxes next to the sliders.
Maybe they could be replaced with the new spinslider, would give a lot more space for decimal digits

@MarianoGnu The spin boxes (and mouse scroll-wheel) doesn't work in the non-raw mode in the new inspector. Try it yourself. That was the reason I made this PR.

Sliders are kind of useless, since a range from 0 to 100 is very high, but as i said, this is not meant to pick a Color for drawing con a canvas, it's meant for color Operations (personally think 100 is too much, but dont have the experience to come to a conclusion)
Maybe step could be changed to logaritmic so it's easier to pick small values?

Sorry, I do not understand what you mean by "color operations", isn't it obvious that you can calculate with any float rgba values?
I think there is no reason to limit the raw input. I just changed the raw input UI into the 4 new user-friendly float input fields and used this (successfully) as input for further calculations in a shader.

Nothing you said so far implies that the float range must be limited to 0-10 or 0-100. Why shouldn't it be possible to use a regular float input and e.g. hand over a Color(-0.42, -0.42, -0.42, -1) to the shader and then do not multiply but add it to some other color?


Allow overwriting $SUPW by an environment variable

Thought this might be useful for the hashbang team as well: this change allows you to overwrite the generated SUPW by one provided as an environment variable, this is handy if your instances tend to rotate and you'd like to keep the password predictable.

Just a sec, feedback incoming.

@DeviaVir My feedback is in https://github.com/pebble/docker-mumble/pull/2  ;-)

LGTM.  I will merge once the Object::connect: No such slot MurmurDBus::userTextMessage issue is solved.

15:01:29       +DeviaVir │ KellerFuchs: running it locally, getting the error too but I can connect
15:01:34       +DeviaVir │ KellerFuchs: seems like that error means nothing
15:01:42       +DeviaVir │ KellerFuchs: so that ordeal can be ignored for this PR
15:01:55       +DeviaVir │ KellerFuchs: but thanks for your security contributions there, FTW.


How to: Use Arrays of Blocking Collections in a Pipeline

Ported from MSDN

Fixed formatting in #383.


fix: string representation containing double quotes or triple quotes

closes #79

Coverage increased (+0.006%) to 90.717% when pulling cbaf73b3694945a15b71907952faf7d0398fc41a on satra:fix/literal into 6217972a8a624cbb614d5c1fe0b46f02f66274de on trungdong:master.

Hi @satra,
Thank you very much for fixing this.


Cannot compile tutel kernels and got runtime error

I have installed tutel on my machine and have set up the related environment variables, such as the $CUDA_HOME and $CFLAGS.
However, when I try to run examples/hello_world.py, I got the following error:
[E custom_kernel.cpp:124] default_program(1): catastrophic error: cannot open source file "cuda_runtime.h"
1 catastrophic error detected in the compilation of "default_program".
Compilation terminated.
Failed to use NVRTC for JIT compilation in this Pytorch version, try another approach using CUDA compiler.. (To always disable NVRTC, please: export USE_NVRTC=0)
File "/private/home/hyhuang/.local/lib/python3.9/site-packages/tutel/impls/jit_compiler.py", line 26, in func
tutel_custom_kernel.invoke(inputs, ctx)
RuntimeError: (true) == (fp != nullptr)INTERNAL ASSERT FAILED at "/tmp/pip-req-build-pcbbciia/tutel/custom/custom_kernel.cpp":40, please report a bug to PyTorch. CHECK_EQ fails.
I am using PyTorch 1.10.1 + CUDA 11.3. Is there any other parameter I should fix to use tutel?

Your CUDA environment seems not be installed in the default location(e.g. /usr/local/cuda/include) can you print the value of CUDA_HOME. BTW, you can also try whether export USE_NVRTC=0 will help.

Your CUDA environment seems not be installed in the default location(e.g. /usr/local/cuda/include) can you print the value of CUDA_HOME. BTW, you can also try whether export USE_NVRTC=0 will help.

Thank you for your prompt reply! Yes, my CUDA environment is not installed in the default location because I'm using it on a cluster. Is there a parameter I can fix to ensure the compiler can find the correct CUDA? I will try to use  export USE_NVRTC=0
$ echo $CUDA_HOME
/public/apps/cuda/11.3

We just merge a PR that parse CUDA_HOME from environment variable. Can you try whether it works for you?

Thank you! The fix works and I think the CUDA_HOME can be found correctly. I can now successfully run the hello_world.py and hello_world_ddp.py under the examples folder without any error. However, when I tried to use it under the fairseq (the use case is here), I got the following two errors:

Compilation error: (seems like the CUDA compiler worked, otherwise there won't be the second error)

[W custom_kernel.cpp:158] nvrtc: error: unrecognized option --includ`��.�U found
� Failed to use NVRTC for JIT compilation in this Pytorch version, try another approach using CUDA compiler.. (To always disable NVRTC, please: export USE_NVRTC=0)

RuntimeError

 File "/private/home/hyhuang/.conda/envs/newnllb/lib/python3.9/site-packages/fairseq-1.0.0a0+b1b3eda-py3.9-linux-x86_64.egg/fairseq/modules/moe/top2gate.py", line 234, in top2gating
    locations1 = fused_cumsum_sub_one(mask1)

File "/private/home/hyhuang/.local/lib/python3.9/site-packages/tutel/jit_kernels/gating.py", line 22, in fast_cumsum_sub_one
    return torch.ops.tutel_ops.cumsum(data)
RuntimeError: (0) == (cuModuleLoadDataEx(&hMod, image.c_str(), sizeof(options) / sizeof(*options), options, values))INTERNAL ASSERT FAILED at "/tmp/pip-req-build-djl73tcc/tutel/custom/custom_kernel.cpp":214, please report a bug to PyTorch. CHECK_EQ fails.
    return torch.ops.tutel_ops.cumsum(data)

Would you be able to provide any suggestions between these two? I am so confused. This is the same environment I used to run the hello_world.py scripts.

You need to run unset USE_NVRTC since you may explicitly configure that variable before.

@hyhuang00 Can you help us to test whether the latest version (#170) still work for your environment? As we canceled the way to detect manual CUDA_HOME environment variable, but the new way should be compatible with different environment more robustly.

Sure, I'm happy to help. Let me try out the new version and I'll let you know if it works for me.

The new version works on my machine without any error. I installed the package via $ python3 -m pip install --user --upgrade git+https://github.com/microsoft/tutel@main

Thanks!


Windows 10 said the disk is not bootable

a.	Install and configure ZStack Enterprise in a VM in Virtual Box v6.1 (8 CPU, 8196Mb RAM, Enable PAE/NX, Enable Nested VT-x/AMD-V).
b.	Create a new image with Win 10 ISO. Set BIOS mode to UEFI.
c.	Create a new VM with said image.
d.	Boot with the Win10 ISO mounted.
e.	Win10 installation starts. But when it is time to create a partition:

If I choose Legacy as BIOS, then the guest won't boot at all:

How can I install a Win10 guest?

@kcwongigs
Thanks for your report.
First issue, it's the patch we applied to ovmf influenced the installation. If you are using ZStack with version > 4.3.6 and you do not want to reinstall a lower version. As a workaround, you can manully exit the windows installation and guest will go through UEFI boot again, make sure you press any key to make your guest boot from CD/DVD. Then the guest can be successfully installed.
But I recommand you to use LTS version based on 4.2.x，because this version still use old version ovmf.
Second issue, I'm not quite sure what happened to legacy boot, but I checked our test results, Win10 can be installed with Legacy. So maybe there is any difference with your image or nested virtualization issue. It will be helpful, if you can offer more details about your image and logs under /var/log/libvirt/qemu/your_vm_uuid.log.

I'm using ZStack 4.4.8 (ZStack-Cloud-x86_64-DVD-4.4.8-c76.iso).
Using UEFI, I triggered the issue again. Then I exited Windows installer. That brought me to a black screen and nothing happens. So I went to ZStack web page and rebooted the guest, and reopened the console. The DVD boots again, and got the same error again.
Win10.txt
8dbb51aa57d04a7591e182ea9371e44c.log.txt

I'm using ZStack 4.4.8 (ZStack-Cloud-x86_64-DVD-4.4.8-c76.iso).
Using UEFI, I triggered the issue again. Then I exited Windows installer. That brought me to a black screen and nothing happens. So I went to ZStack web page and rebooted the guest, and reopened the console. The DVD boots again, and got the same error again.
Win10.txt 8dbb51aa57d04a7591e182ea9371e44c.log.txt

Could you upgrade to the latest version (4.4.24) and then try again?

I upgraded to 4.4.24 and it still didn't work.
If I create a Win10 instance in Legacy mode, I get the same no display yet problem:

If I create a Win10 instance in UEFI mode, I pressed ENTER when it said press enter to boot with disc, but I still get dumped into UEFI interactive shell.

I deleted the VM instance and started from scratch.

Select Win10 UEFI image
Add CD Rom for VirtIO disc

Then upon the first launch of the console, I'm unable to press key to boot from CD fast enough, so I got dumped into UEFI shell and I got this map:

I rebooted (with UEFI shell's reset command), the press any key to boot from CD appeared and I pressed enter. Then I got this:

So I close the console tab and reopened it, then I got this:

Some of the items disappeared.

@kcwongigs
Sorry for getting back to you late.
In first case, if you boot windows with UEFI and see BLK0, that means your Win guest is not use right Boot Order, you can simply change it as following:
Vm Instances -> Your VM -> Actions -> System Configurations -> Set Boot Order -> Choose CD ROM

and reboot vm by:
Vm Instances -> Your VM -> Actions -> Reboot

Then, vm will boot from your ISO.
The second case, because default CD/DVD enter option only shows in a short time, if you not manually press any key to get into installation a UEFI shell will shows with your screenshot.
You can use UEFI shell as following:
FS0:
cd EFI
cd BOOT
BOOTX64.EFI

to continue your installation.
If your manually reboot guest inside. ZStack will detect if your boot from CD ROM and reboot inside guest means "installation finished" so Boot Order will be changed to HardDisk automaticly (which could be checked from details of VM).
So it's not recommand to manually reboot during installation phase or the result will be quite confused.


Fixing test neon renamemethod

@TomasVotruba it seems on latest CI run, the test indentation seems removed after services new line. This remove the indentation so the CI will be green.

@TomasVotruba the indentation change seems happen via internal nette's Nette\Neon\Node\LiteralNode, let's merge it for now to make test in CI green.

ok, closing per https://github.com/rectorphp/rector-nette/commit/707dfc1d1013008e18fa8481b218d52cb8deb42c


GSP-749: Unify Path Behavior

ref: https://github.com/beyondstorage/go-storage/issues/744

Let's start a quick discussion in room: https://matrix.to/#/#beyondstorage@gsp-749:matrix.org

Mostly LGTM, let's create a tracking issue for this GSP!


apds9930: Initial implementation

Signed-off-by: Lay, Kuan Loon<EMAIL_ADDRESS>

Thanks, but will have to wait for MRAA to merge IIO into master first.


Custom Marker for AR

Is it possible to create Custom Markers and use them for an Augmented Reality implementation?

Yes, we actually have a couple of examples based on marker tracking, you can find them here and here. They use the JSARToolkit and some custom Xflow operators to produce transformation matrices from the marker tracking information.
Those examples also show how to use a webcam as input for the marker detection process.


[SMALLFIX] Supply the variable name to Preconditions.checkNotNull in …

…BlockMasterClientServiceHandler.java

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16907/
Test PASSed.

Merged build finished. Test PASSed.


Recommend way to aggregate semantic code embeddings

What would be the recommend way to aggregate different semantic code embeddings from the same repository to represent the overall semantic of the repository? Currently I am averaging the UniXCoder embeddings and using cosine similarity score to evaluate the result, but I am not sure if it is the right way.
Thanks in advance!

If you lack supervised data to fine-tune UniXcoder, the averaging approach is appropriate.

If you lack supervised data to fine-tune UniXcoder, the averaging approach is appropriate.

Thank you!


New Version: Robware.RVTools version 4.3.2

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.4 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

...

/AzurePipelines run

@stephengillie - while you're at it... :)


"Controller - Installing" failing with "msg": "Unexpected templating type error occurred on during

Hi,
I'm getting error during the  "Controller - Installing" step with below error! playbook that calls role is triggered from ADO pipeline.
2021-09-14T15:44:10.8653790Z fatal: [SECNGXCTLSR1003]: FAILED! => {"msg": "Unexpected templating type error occurred on (timeout --foreground 10m ./install.sh  --accept-license  --tsdb-volume-type '{{ nginx_controller_tsdb_volume_type }}'  --smtp-host '{{ nginx_controller_smtp_host }}'  --smtp-port '{{ nginx_controller_smtp_port | default(25) }}'  --smtp-authentication '{{ nginx_controller_smtp_authentication }}'  --smtp-use-tls '{{ nginx_controller_smtp_use_tls }}'  --noreply-address '{{ nginx_controller_noreply_address }}'  --fqdn '{{ nginx_controller_fqdn }}'  --organization-name '{{ nginx_controller_organization_name }}'  --admin-firstname '{{ nginx_controller_admin_firstname }}'  --admin-lastname '{{ nginx_controller_admin_lastname }}'  --admin-email '{{ nginx_controller_admin_email }}'  --admin-password '{{ nginx_controller_admin_password }}'  {{ (nginx_controller_tsdb_volume_type == \"nfs\") | ternary(\"--tsdb-nfs-path '\" + nginx_controller_tsdb_nfs_path + \"'\",'') }}  {{ (nginx_controller_tsdb_volume_type == \"nfs\") | ternary(\"--tsdb-nfs-host '\" + nginx_controller_tsdb_nfs_host + \"'\",'') }}  {{ (nginx_controller_tsdb_volume_type == \"aws\") | ternary(\"--tsdb-aws-volume-id '\" + nginx_controller_tsdb_aws_volume_id + \"'\",'') }}  {{ (nginx_controller_configdb_volume_type != \"\" ) | ternary(\"--configdb-volume-type '\" + nginx_controller_configdb_volume_type + \"'\", '') }}  {{ (nginx_controller_configdb_volume_type == \"nfs\") | ternary(\"--configdb-nfs-path '\" + nginx_controller_configdb_nfs_path + \"'\",'') }}  {{ (nginx_controller_configdb_volume_type == \"nfs\") | ternary(\"--configdb-nfs-host '\" + nginx_controller_configdb_nfs_host + \"'\",'') }}  {{ (nginx_controller_configdb_volume_type == \"aws\") | ternary(\"--configdb-aws-volume-id '\" + nginx_controller_configdb_aws_volume_id + \"'\",'') }}  {{ ((nginx_controller_apigw_cert is defined and nginx_controller_apigw_cert | length > 0) and (nginx_controller_apigw_key is defined and nginx_controller_apigw_key | length > 0) ) | ternary(\"--apigw-cert '\" + nginx_controller_apigw_cert + \"'\",'') }}  {{ ((nginx_controller_apigw_cert is defined and nginx_controller_apigw_cert | length > 0) and (nginx_controller_apigw_key is defined and nginx_controller_apigw_key | length > 0) ) | ternary(\"--apigw-key '\" + nginx_controller_apigw_key + \"'\",'') }}  {{ (nginx_controller_smtp_authentication | bool) | ternary(\"--smtp-user '\" + nginx_controller_smtp_user + \"'\",'') }}  {{ (nginx_controller_smtp_authentication | bool) | ternary(\"--smtp-password '\" + nginx_controller_smtp_password + \"'\",'') }}  {{ (nginx_controller_self_signed_cert | bool) | ternary('--self-signed-cert','') }}  {{ (nginx_controller_overwrite_existing_configs | bool) | ternary('--overwrite-existing-configs','') }}  {{ (nginx_controller_auto_install_docker | bool) | ternary('--auto-install-docker','') }}  {{ ((nginx_controller_bundled_db | bool) and ( nginx_controller_version is version('3.8', operator='ge', strict=True ))) | ternary(\"--use-bundled-db\",\"--database-host '\" + nginx_controller_db_host + \"' --database-port '\" + nginx_controller_db_port + \"' --database-user '\" + nginx_controller_db_user + \"' --database-password '\" + nginx_controller_db_password + \"'\" ) }}  {{ ((nginx_controller_db_enable_ssl | bool) and ( not nginx_controller_bundled_db )) | ternary('--db-enable-ssl true','') }}  {{ ((nginx_controller_db_enable_ssl | bool) and (nginx_controller_db_client_cert | length > 0)) | ternary(\"--db-client-cert '\" + nginx_controller_db_client_cert + \"'\",'') }}  {{ ((nginx_controller_db_enable_ssl | bool) and (nginx_controller_db_client_key | length > 0)) | ternary(\"--db-client-key '\" + nginx_controller_db_client_key + \"'\",'') }}  {{ ((nginx_controller_db_enable_ssl | bool) and (nginx_controller_db_ca | length > 0)) | ternary(\"--db-ca '\" + nginx_controller_db_ca + \"'\",'') }}  {{ ((nginx_controller_version is version('3.5', operator='ge', strict=True)  ) ) | ternary('--non-interactive','') }}\n): must be str, not NoneType"}
I can't tell from error above if it's complaining about the last line or one of the other variables.
Any suggestions would be greatly appreciated.
Thanks,

Can you please provide the following?
To reproduce

Your playbook(s) including variables with any sensitive information redacted

Your environment:

Version of the NGINX Controller Installation role or specific commit
Version of Ansible
Version of Jinja2
Target deployment platform

Unfortunately Ansible repeats the entire command without vars.  So we don't know which var is missing and matches the error:  must be str, not NoneType

thank you Brian, went through the variables defined and found the nginx_controller_db_password was missing so added one directly to test and it's gotten past that error. I have now though stumbled onto another error.
"ERROR: Parameter --apigw-cert is invalid: File /etc/ssl/certs/star_azure_defra_cloud.crt does not exist or is not readable by current user.",
Have defined below in the variables.

nginx_controller_apigw_cert: /etc/ssl/certs/star_azure_defra_cloud.crt
nginx_controller_apigw_key: /etc/ssl/private/star_azure_defra_cloud.key

Should the crt and key be created as part of the install?

Not all variables are required.
For example;  the db password and user are tied to using an external PostgreSQL database server.  Not required.  You can use the internal-db flag to have the system support its own.  using local file storage if not production, and using a remote NFS backed volume if production.
the api gateway cert and key are specific to using your own cert and key for the interface of Controller - both GUI and API.
Again, not required.  You can also allow the system to self sign - and add a cert after install using helper.sh if you still require one.
The path you are providing for the cert and key assumes they are already stored in those places and the permissions are correct for the user that Ansible is using to execute the installer.  So, who is the owner of the files.  For example, you copied them over with root, but are executing the script as user ubuntu.

The user is not root when installing.  And installing as root is not supported by default.
The install needs to be executed by an administrator who is not root following best practices.

thanks again brian, apologies in advance if below question is not clear, new to ansible and so learning how it works as part of this install.
I've managed to sort out the above errors creating certificates in locations specified and sorted out permissions, though came unstuck as docker was not already installed so have add docker role to playbook.
I've tested the admin user and password and was able to connect to managed Azure PostgreSQL with pgAdmin and have specified same user@servername and password. However running the install it's failing with below error.
Thu Sep 16 20:24:23 UTC 2021   2. Checking required ports... OK Thu Sep 16 20:24:23 UTC 2021 Last step took 00m:00s   3. Attempting to detect your Operating System... Found bionic Thu Sep 16 20:24:23 UTC 2021 Last step took 00m:00s   4.  All found. Thu Sep 16 20:24:23 UTC 2021 Last step took 00m:00s   5. Checking Docker version...  Docker version 20.10.8, build 75249d8      We recommend setting native.cgroupdriver to systemd for Docker.      WARNING! Docker configuration does not seem to have log rotation enabled. We recommend enabling log rotation for docker containers.      For steps to enable log rotation follow this link: https://success.docker.com/article/how-to-setup-log-rotation-post-installation Loaded image: python:3.6-alpine Loaded image: postgres:12.5-alpine Loaded image: controller-infra/platform-mgr:2.28.24 Thu Sep 16 20:24:26 UTC 2021 Last step took 00m:03s   6. Config Database configuration  Could not connect to the database.
Using below versions btw.
controller-installer-3.15.0.tar.gz
remote controller  Ubuntu 18.04.5 LTS
Azure Database for PostgreSQL server PostgresSQL 9.6
ansible [core 2.11.4]
config file = /etc/ansible/ansible.cfg
configured module search path = ['/home/vmadmin/.ansible/plugins/modules', '/usr/share/ansible/plugins/modules']
ansible python module location = /usr/local/lib/python3.6/dist-packages/ansible
ansible collection location = /home/vmadmin/.ansible/collections:/usr/share/ansible/collections
executable location = /usr/local/bin/ansible
python version = 3.6.9 (default, Jan 26 2021, 15:33:00) [GCC 8.4.0]
jinja version = 2.10
libyaml = True

the key is this line: Config Database configuration Could not connect to the database.
This is the Controller server component attempting to use the provided credentials to connect to the remote PostgreSQL endpoint.
The failure could be the credentials or a block at the network level from a policy not being defined between the Controller machine and service endpoint.

hi Brian, thanks for all your help thus far we resolved the issue with connect to PostgreSQL by adding rule on the NSG where the controller resides to allow outbound connection to SQL over port 5432, however we then had issues trying to use SSL connecting to db not sure if this is in fact possible with a managed instance of postgreSQL that Azure provide.
I noted in the vars file the description below suggest certificate needs to be installed on both controller and db server but as we are using a managed postgreSQL instance we cannot install certificate on db server.
## Optional. Enable database ssl communications. Requires certificate installation at db server and controller. (true, false)
So we set  - nginx_controller_db_enable_ssl: false
This got us past this error;
  6. Config Database configuration
psql: error: SSL error: certificate verify failed
 Failed to log into the database. Check that the PostgreSQL username and password are correct and that it allows connections from this host.

However now it's failing much further into the install with below error, is this result of not using ssl connectivity which I thought was optional.
```

NGINX Controller database has been initialized.
system: unable to upsert accounts: pq: SSL connection is required. Please specify SSL options and retry.
Failed to create the admin account in NGINX Controller.
Detailed information can be found in installation logs at /var/log/nginx-controller/failure. Retry NGINX Controller installation or contact support, providing all logs.

Any help understanding above would be greatly appreciated.

I don't know about managed PostgreSQL, but if you install it yourself - you have to set the listener on the PostgreSQL side to accept connections, and the machines to accept connections from.
I have never use the PostgreSQL ssl setting in my labs myself.
And I also don't use a PostgreSQL managed service.
Also the user account used has some expectations.  The automation still has to meet all of these expectations: https://docs.nginx.com/nginx-controller/admin-guides/install/install-nginx-controller/

Ok thanks for your input there, be good to understand if using a managed PostgreSQL instance is supported solution.
I did find it kind of strange there is option to turn SSL off but then the role expects it to be on where it failed.
If we could see what command is being passed that throws up the below and understand why it's asking for SSL connection when turned off earlier might help us understand the issue.
system: unable to upsert accounts: pq: SSL connection is required. Please specify SSL options and retry.
I'm also speaking to colleague of yours Brain based in UK to align time zones, he's looking to try replicate the error.

Hello again, just to update managed to get past above error by disabling SSL on the managed PostgreSQL instance. So seems setting it to nginx_controller_db_enable_ssl: false it will try use SSL connection when it finds it's enabled on the PostgreSQL server itself.
Also looking at the MS doc it mentions downloading and using MS certificate to PostgreSQL instance.
https://docs.microsoft.com/en-us/azure/postgresql/concepts-ssl-connection-security#applications-that-require-certificate-verification-for-tls-connectivity
I'm wondering if we can change the below vars provided to the role nginx_controller_install to point to the certificate MS are suggest is needed to connect to PostgreSQL instance.

nginx_controller_db_client_cert: /etc/ssl/certs/star_azure_defra_cloud.crt
nginx_controller_db_client_key: /etc/ssl/private/star_azure_defra_cloud.key

To something like this;

nginx_controller_db_client_cert:  /etc/ssl/certs/Baltimore_CyberTrust_Root.pem

Would that work do you think from playbook perspective?


Why does selected row index changed when do sorting

Expected Behavior

All data is from browser side, Assuming  there are 3 rows data.
Example:
Row 1: Aaron   20
Row 2: Bob      25
Row 3: Danny  30

For example, If I tick only one checkbox(e.g. row 1) and then sort the rows in a different way, the selected checkbox should be bind with row (Aaron 20).

I‘m confused, the demo [https://codesandbox.io/s/wy2rl1nyzl](url) works fine, my code was the same with that, but doesn't work the same.

Current Behavior

The only one checked checkbox was bind with different row data each time

Steps to Reproduce (for bugs)

Your Environment

Tech
Version

Material-UI
3.1.0

MUI-datatables
^2.0.0-beta-39

React
16.5.2

browser
Chrome Version 70.0.3538.102

etc

const options = {             filterType: 'dropdown',             download: false,             print: false,             rowsPerPageOptions: [10, 20, 30],         };  return (             <MUIDataTable                 data={data}                 columns={columns}                 options={options}             />         )

@gregnb

Try the latest version let me know if this issue is still ongoing

Try the latest version let me know if this issue is still ongoing

Tried with ^2.0.0-beta-43 still doesn't work... Looks like the last version without this issue is ^2.0.0-beta-35.

Same issue with version "2.12.0"


Add pulling Sentinel-2 Imagery

One possibility is to use https://github.com/opendatacube/odc-stac to pull the Sentinel imagery into Xarray with STAC, which is how Planetary Computer stores the imagery. Should make it easy to put into the models.

This is done.


--MetadataMediator - Lighting Layout Configurations

Motivation and Context
This PR adds support for parsing lighting layout files, of the format found here into appropriately configured attributes files, which are to then be used to instantiate lights in scenes.
 The file is parsed so that each light entry is parsed into a LightInstanceAttributes object, which is aggregated into a single LightLayoutAttributes object.  This in turn is used to instance a lighting layout upon scene creation.

How Has This Been Tested

All existing c++ and python tests passed.  Tests were added to verify proper parsing of JSON.
Types of changes

[x] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

How will the standard handle GLBs or such that have lighting already defined inside of them?

How will the standard handle GLBs or such that have lighting already defined inside of them?

That will be handled by the loading of the assets.


Decouple fv3net docker images from fv3gfs-fortran

The prognostic run dockerfile depends on ESMF and MPI from images pushed
from the fv3gfs-fortran repo. This effectively coupled the build systems
of fv3net to fv3gfs-fortran, but this repo has different concerns.
fv3net emphasize modern tools to support experimental ML, while
fv3gfs-fortran empahsize stability in bit-for-bit checksums. As a
consequence, this coupling effectively pinned this repo to ubuntu
version supported by fv3gfs-fortran (18.04), preventing fv3net from
updating to a more recent OS and software like python 3.7 (See #1083).
Building esmf, mpi, and fms in the prognostic run dockerfile effectively
decouples these repos.
The checksums remain unaltered.

This passed tests locally. Proof:
$ git status
On branch decouple-docker-image
Your branch is up to date with 'origin/decouple-docker-image'.

nothing to commit, working tree clean
[I] noahb@noah-vm ~/w/V/fv3net (decouple-docker-image)
$ make build_image_prognostic_run 2> /dev/null && echo "succeeded"
tools/docker_build_cached.sh us.gcr.io/vcm-ml/prognostic_run:latest \
        -f docker/prognostic_run/Dockerfile -t us.gcr.io/vcm-ml/prognostic_run:bfdff9d8fb466b50f60bcab08ea15eb6e542feaa \
        --target prognostic-run .
tools/docker_build_cached.sh us.gcr.io/vcm-ml/prognostic_run:latest \
        -f docker/prognostic_run/Dockerfile -t us.gcr.io/vcm-ml/notebook:bfdff9d8fb466b50f60bcab08ea15eb6e542feaa \
        --target notebook .
succeeded

Could we similarly push build dependency images for ESMF, MPI, FMS, NCEP to the container registry to save time rather than requiring full-rebuild after a system prune?  Or have there been issues with this in the past?

@frodre Once this in master the ESMF etc steps will be cached. We already have remote caching enabled for prognostic_run image builds, but the cache is the image tagged with :latest. latest is still pointing to an image constructed with master, which is why this image required a long time to rebuild. When this is merged CI will push the docker image to this tag, and all the previous steps should be cached.
BTW I've tested this locally, by overwriting pushing an image to GCR and changing the CACHE_TAG here: https://github.com/VulcanClimateModeling/fv3net/blob/779ce7b70bf5e19c734defbc93667a496310eb3a/Makefile#L10


Fix Avro conversion of R4 resources with contained reources.

This is a regression in R4 only and was discovered when we enabled Bunsen's avro-conversion unit-tests for R4; in particular AvroConverterTestR4.testContainedResources.

The root cause of this issue does not seem to be related to contained resources but the way slices are handled; and it seems the bug is not specific to R4 but has been present for STU3 too. It seems the current code does not properly recognize slice definitions on non-extension fields. So for example in this sliced telecom example it will end up adding multiple telecom fields in Patient leading to an exception. The reason it was tirgerred now (and was mis-attributed to R4) is that the MedicationRequest resource of US-Core, has a slice definition on category, trigering the bug.


Evaluate low tech method of opening issues on repos

Overview
We need to have several pathways for people to update repos so that there is an available call to action for every audience member we reach
Action Items

[ ] Review method used during Brigade Congress 2020 and evaluate how it could be integrated into roll out plan

Resources/Instructions
Issue opened on a BetaNYC repo
Issue opened on a Code for Tulsa repo

integrate this into the tag generator.

integrate this into the tag generator.


Instarepl can't find library

InstaREPL shows an error when trying to require library.
core.clj
(require '[lanterna.screen :as s])

project.clj
(defproject untitled "0.1.0-SNAPSHOT"
  :description "FIXME: write description"
  :url "http://example.com/FIXME"
  :license {:name "Eclipse Public License"
            :url "http://www.eclipse.org/legal/epl-v10.html"}
  :dependencies [[org.clojure/clojure "1.8.0"]
                  [clojure-lanterna "0.9.4"]]
  :main ^:skip-aot untitled.core
  :target-path "target/%s"
  :profiles {:uberjar {:aot :all}})

The following error is shown
java.io.FileNotFoundException: Could not locate
lanterna/screen__init.class or lanterna/screen.clj on classpath.

The app builds and runs OK with lein repl and lein run

Yes this is a design issue with Nightcode's instaREPL. It doesn't have access to third party libraries. If you'd like this feature, please consider using Nightlight instead:
https://github.com/oakes/Nightlight
Refer to this issue for a previous discussion:
https://github.com/oakes/Nightcode/issues/171


Panner object parameter not supported

Object parameter not supported.  This functionality is consistent throughout the library.

Tone.js v13.8.5 *

Demo:
https://jsfiddle.net/timboie/mu28xgeq/

Thank you! closed by #472


Version change from 18.1.2 to 18.1.3

[!NOTE]
This PR depends on #259. Merge #259 first.

Changes

[x] Changed package version to 18.1.3.

Pending release references

#259

Regards.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.18%. Comparing base (9a70060) to head (0611495).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #260   +/-   ##
=======================================
  Coverage   91.18%   91.18%
=======================================
  Files         269      269
  Lines        6481     6481
  Branches     1313     1313
=======================================
  Hits         5910     5910
  Misses        464      464
  Partials      107      107

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Inform user about failure only on the last try

Signed-off-by: Frantisek Lachman<EMAIL_ADDRESS>Fixes: #1482

RELEASE NOTES BEGIN
Packit now correctly inform users about downstream errors only on the last try. (Previously, Packit informed for all tries even the last try succeded.)
RELEASE NOTES END

recheck

recheck

recheck


Training on the NeRF-DS dataset

I train SC-GS on the NeRF-DS dataset with this command:
python train_gui.py \
--source_path /home/gs/datasets/NeRF-DS/bell_novel_view \
--model_path outputs/bell_novel_view \
--deform_type node --node_num 512 --hyper_dim 8 \
--eval --gt_alpha_mask_as_scene_mask \
--local_frame --resolution 1 --W 800 --H 800

I can't get a good PSNR result. Is there something wrong?

The author said that the old version of the code given here (#18 (comment)) can reproduce the results of the paper with the following command , but the link is now inaccessible. --deform_type node --node_num 4096 --deform_lr_scale 1 --eval --resolution 1 --gt_alpha_mask_as_dynamic_mask --node_warm_up 500 --iterations_node_sampling 7500 --iterations_node_rendering 10000 --oneupSHdegree_step 5000 --warm_up 3000 --hyper_dim 8 --local_frame --pred_color --pred_opacity --dynamic_color_warm_up 20000 --deform_downsamp_strategy direct

Thank you very much! I'll try it!

The author said that the old version of the code given here (#18 (comment)) can reproduce the results of the paper with the following command , but the link is now inaccessible. --deform_type node --node_num 4096 --deform_lr_scale 1 --eval --resolution 1 --gt_alpha_mask_as_dynamic_mask --node_warm_up 500 --iterations_node_sampling 7500 --iterations_node_rendering 10000 --oneupSHdegree_step 5000 --warm_up 3000 --hyper_dim 8 --local_frame --pred_color --pred_opacity --dynamic_color_warm_up 20000 --deform_downsamp_strategy direct

Thank you very much! I'll try it!

please try to repleace def readNerfiesCameras(path, inter_valid=True): with def readNerfiesCameras(path, inter_valid=False):, in https://github.com/yihua7/SC-GS/blob/26cd57d09598b2f5d951029808a5ac9f0ff4f626/scene/dataset_readers.py#L517C5-L517C23
and make sure your data set file structure organized like this: dataset/NeRF/basin_novel_view/

Hi,
Happy to see the discussion.
I have updated the sharing link of the old version code: https://pan.baidu.com/s/1Ysa3ehDmeOX5EV6insK4dg?pwd=1234.
Hope the code helps as a reference.

Thanks for your time and patience, thank you very much!

Hi, I followed the training instructions you gave on the HooK dataset of D-NeRF (CUDA_VISIBLE_DEVICES=0 python train_gui.py --source_path data/d-nerf/hook --model_path outputs/hook -- deform_type node --node_num 512 --hyper_dim 8 --is_blender --eval --gt_alpha_mask_as_scene_mask --local_frame --resolution 2 --W 800 --H 800).
We get a result of 39.66, which cannot be inconsistent with the 39.87 inside the paper, can you tell me why this could be?  Thanks!

Hi,
If you strictly follow the readme instructions, the results won't vary too much. The two values are quite close and may differ from machines, seeds, and environments. If you are performing some comparison experiments, please feel free to fill in the results obtained on your devices.


Song overlapping with other players

When Google play is playing a song and i play an audio from the music player, it does not stop the google play, they are both playing at the same time.

Got same issue. Please add audio focus feature please.


installs and uses python3 and pip3 on frontproxy service example

installs and uses python3 and pip3 to address Issue #1597
uses ppa:deadsnakes/ppa
fork build passed

Could also incorporate the instructions on https://github.com/docker-library/python/blob/master/3.5/jessie/Dockerfile, though this might be overkill. 😄

Is it possible to add CI for this regression so we don't break again?

We used to have example verification as part of CI but it got commented out due to CI times being too long and flakey. IMO we can live with this situation until we get a better CI that runs faster.


Add HLSL-specific shader resource query

For HLSL shaders, this will allow querying shader registers

Fixed by https://github.com/DiligentGraphics/DiligentCore/commit/900dc223916d3590522b633f93914e8f5d6e1d6e and https://github.com/DiligentGraphics/DiligentCore/commit/07a7f4bd9e05a8c170d308af83063024bcdfaad4


How to get number of rows

just not using double (for ... range...)

Thanks for your issue. You can get number of rows like this:
rows := len(xlsx.GetRows("Sheet1"))

Thank you very much, get it!


Bump version for python 3.12

Updated python_requires to support Python 3.12. Current configuration means 0.0.3 is the latest release that will install with Python 3.12.
Given SemVer, I think it's reasonably safe to assume 3.13, 3.14, etc. won't introduce breaking changes. Therefore I've changed to allow any Python 3.x version greater than 3.7.
Also updated classifiers for current supported Python versions.

Hi @scottlimmer, have you actually tested this package with 3.11 and 3.12? I was actually wondering what was the author's reason to add the <3.11 requirement. Thanks.

@filippo82 I've been runnng my fork for more than a week with no issues.

@filippo82 I've been running my fork for more than a week with no issues.

That is great to hear! Thanks! I am on vacation right now but will test the PR when I return next week.
The reason for forcing Python version was an issue with a dependency with GLiNER. That may have been resolved now, it seems.
Thanks again!

Hi @scottlimmer thanks for the info and the good news! I will give it a try in a week or so 🤞🏻

Thanks, all! I just updated this for 0.0.9 -- sorry it took so long!


Broken scraper: opinions.united_states.state.kan(ctapp)

Looks like the site got shuffled around and the page that we previously scraped isn't there anymore:
HTTPError: 404 Client Error: Not Found for url: https://www.kscourts.org/Cases-and-Opinions/Date-of-Release-List/

I'm working on this now

published: https://www.kscourts.org/Cases-Opinions/Opinions/Published
unpublished: https://www.kscourts.org/Cases-Opinions/Opinions/Unpublished
while we are at it, we should add oralarg scraper:https://www.kscourts.org/About-the-Courts/Supreme-Court/Oral-Arguments/Oral-Argument-Archives

I've got a fix for this @mlissner but I need your input first, see this

fixed in commit: https://github.com/freelawproject/juriscraper/commit/f1b32a6959ef78eaa59dc3e2f8a1a09ba5f8813c


Snap: Register code as an editor in Debian with update-alternatives

Issue Type: Bug
This is a bug report regarding a feature that was implemented here: https://github.com/microsoft/vscode/issues/3541

Install Ubuntu 18.04 LTS
Open Snap Store aka "Ubuntu Software"
Search for VSCode
Click on VSCode
Click install
Wait for install to complete
Open terminal
run "update-alternatives --config editor"

Expected: VSCode appears as an option. See https://github.com/microsoft/vscode/issues/3541
Actual: VSCode does not appear as an option
VS Code version: Code 1.43.1 (fe22a9645b44368865c0ba92e2fb881ff1afce94, 2020-03-18T07:17:38.324Z)
OS version: Linux x64 5.3.0-42-generic snap

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-4770 CPU @ 3.40GHz (8 x 3829)

GPU Status
2d_canvas: unavailable_softwareflash_3d: disabled_softwareflash_stage3d: disabled_softwareflash_stage3d_baseline: disabled_softwaregpu_compositing: disabled_softwaremultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: disabled_offrasterization: disabled_softwareskia_renderer: disabled_off_okvideo_decode: disabled_softwareviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: unavailable_softwarewebgl2: unavailable_software

Load (avg)
1, 0, 0

Memory (System)
31.27GB (24.04GB free)

Process Argv
. --no-sandbox

Screen Reader
no

VM
0%

Extensions: none

@Tyriar Can you help out here, given you did the same for the deb?

You should just need to run this on post install:
https://github.com/microsoft/vscode/blob/7278373f3df5618df631ea6fe15d7e6c0626d7d5/resources/linux/debian/postinst.template#L10-L13
I'd check for the existence of update-alternatives to be safe.

I'm in a similar situation where I'm trying to add a terminal emulator snap as an alternative. It seems like it's not that easy given that:

update-alternatives requires a direct binary instead of a proxy like snap commands (code is a soft link to /usr/bin/snap)
you'd need to add hooks which needs to be run correctly on every distro (even on ones that doesn't have the update-alternatives kind of functionality)

See this thread.

Do this:
sudo update-alternatives --install /usr/bin/code editor /snap/bin/code 0
then this:
'sudo update-alternatives --set editor /usr/bin/code'
The official VSCode docs has it wrong: https://code.visualstudio.com/docs/setup/linux#_setting-vs-code-as-the-default-text-editor this does nothing on 20.04.


add [pt-br] /content/pt-br/docs/concepts/overview/working-with-objects/kubernetes-objects.md

Co-authored-by: Lorrany Oliveira<EMAIL_ADDRESS>Changes

Brazilian portuguese translation for the Understanding Kubernetes Objects page.

Related issues
https://github.com/kubernetes/website/issues/13939

Here is how the page is gonna look like:
https://deploy-preview-35646--kubernetes-io-main-staging.netlify.app/pt-br/docs/concepts/overview/working-with-objects/kubernetes-objects/
/approve
It looks good to me, just fix the POD/Pod thing and we are good to go :)
/hold

/lgtm
/label tide/merge-method-squash
/approve
Thanks

/hold cancel


DOC: mstats: add examples to kurtosis

Reference issue
Towards #7168
What does this implement/fix?
Add examples to mstats kurtosis
Additional information
[docs only]
First PR, tried to follow contributing guidelines, but apologies in advance if I've messed up!

Thanks for the pointers Tyler, think this should work now

Thanks @daniel-finnan, but given the responses so far in gh-22194, I don't think we'll be maintaining entirely separate examples for mstats functions and their stats counterparts. I'll adjust gh-7168 accordingly.


Linux: Setting WindowState="FullScreen" on Ubuntu results in window being shown behind launcher.

If one sets WindowState="FullScreen" in the main window XAML then when the program is run the window is shown behind the launcher on Ubuntu 20.04..
The window goes fullscreen just fine if you set it after the window is shown.

KDE: works fine
i3: works fine
It seems to be a GNOME bug that doesn't seem to respect the wm-spec:

A list of hints describing the window state. Atoms present in the list MUST be considered set, atoms not present in the list MUST be considered not set. The Window Manager SHOULD honor _NET_WM_STATE whenever a withdrawn window requests to be mapped. A Client wishing to change the state of a window MUST send a _NET_WM_STATE client message to the root window (see below).

We might try to send manual activation requests as a workaround, I guess.


ability to load custom tests from a specified path

maybe --extra-tests PATH?

maybe URL's as well?

--tests-url added in commit cf29db401b53d7b2526822ba25ab68edb6508cba.
helper flag --ignore-std-flags added in commit ac67159381c7e6be8dc123ea1ec3ab285f2e9fa7 for those who do not want to use built-in flags (and only their own)
file-path tests still need to be added.


Update app to recieve notifications while app is open (not just on startup)

Once push notifications are implemented this will likely be a simple byproduct. Since the app will be notified when a notif is received.

Duplicate issue, my bad


Convert global variables to local

Just a minor edit to move these (unnecessary) global variables into local scope.

Make sense, thanks.


install .tmux but nothing happen

Hello, I followed the readme file to complete the operation, but it did not work
my operation is:
git clone -b gh-243 https://github.com/gpakosz/.tmux.git
ln -s -f .tmux/.tmux.conf
cp .tmux/.tmux.conf.local .
my tmux version is:
tmux 2.1
and i have changed the branch to gh200-gh221 but it did not useful
i check my .tmux.conf.local file,it makes no error

i use Mobaxterm on windows to connect my sever

i know，because i have a session is running but tmux sever need you kill your all tmux session and then it will reload .tmux.conf.local


fix padding for arbitrary hop size

as title and a fix for issue #9 that I made.

Hey, thanks for your contribution! Sorry I didn't respond earlier.
Unfortunately, tests are failing with these changes. Can you have a look?


Remove xmlschema

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

35 of 35 (100.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.004%) to 90.625%

Files with Coverage Reduction
New Missed Lines
%

pygef/broxml.py
1
95.65%

Totals

Change from base Build<PHONE_NUMBER>:
-0.004%

Covered Lines:
1334

Relevant Lines:
1472

💛  - Coveralls


Install instructions are updated

AppMap for Node instructions have been updated to refer users to use appmap-node instead of @appland/appamp-agent-js.

:tada: This issue has been resolved in version 0.59.0 :tada:
The release is available on:

v0.59.0
GitHub release

Your semantic-release bot :package::rocket:


uib-tooltip-template error

Hi,
I am using ui-bootstrap 0.14.3 and angular 1.4.1.
I am trying to setup a tooltip using an ng-template:

and then in the same HTML file try to invoke it:

  test
  test

however, I'm seeing the following error in the console:
Error: Syntax error, unrecognized expression: test

I do see the tooltip dialog, however it is empty.

Please post a reproduction using Plunker.
Just a note, you should use angular 1.4.7.

Hey @wesleycho, sure I can do that.
Is there a base Plunker I can go off of?

In the docs page, click the Edit in Plunker link for any particular component.

Thanks, found the issue: jQuery 1.9.1
Reproducible Plunk
Fix with jQuery 1.11.3


Application doesn't start in debug mode with Python 3.10 and PyQT6 only

The problem doesn't happen when I use python 3.9.
Error message:
Traceback (most recent call last):
  File "<frozen importlib._bootstrap>", line 1027, in _find_and_load
  File "<frozen importlib._bootstrap>", line 1006, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 688, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 883, in exec_module
  File "<frozen importlib._bootstrap>", line 241, in _call_with_frames_removed
  File "/home/username/PycharmProjects/myproject/venv/lib/python3.10/site-packages/qasync/__init__.py", line 84, in <module>
    QtCore = importlib.import_module(QtModuleName + ".QtCore", package=QtModuleName)
  File "/usr/lib/python3.10/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
ModuleNotFoundError: No module named 'PyQt5.QtCore'

From the trace you posted, I see nothing pointing the the python version difference. Even though python 3.10 is not currently supported, feel free to create a separate issue with a reproducible example and I can look into it.


Alert: TargetDown in flux-system

Alert TargetDown firing in flux-system namespace
This is an automated issue created by the monitoring system. Please do not edit this message.
Alertmanager URL: https://alertmanager.ankhmorpork.thaum.xyz
Issue was last updated at 2023-03-07 14:40:19.861393323 +0000 UTC m=+1971.697355144.
Common Labels

alertname
TargetDown

cluster
ankhmorpork

job
flux-system/flux-system

namespace
flux-system

prometheus
monitoring/k8s

severity
warning

Common Annotations

description
33.33% of the flux-system/flux-system/ targets in flux-system namespace are down.

runbook_url
https://runbooks.prometheus-operator.dev/runbooks/general/targetdown

summary
One or more targets are unreachable.

Alerts

StartsAt
Links

2023-03-07 14:19:48.612 +0000 UTC
GeneratorURL

(DO NOT MODIFY: b6ed1bf387f2ced91ec3e0045f8943e99f05cb3f597b1e07a70bf48a0cbeea14 )

Spam. Increased threshold to 50% with https://github.com/thaum-xyz/ankhmorpork/commit/e16ee72ceb072d58917b542abef1d1a2e8d82fdc


Workflow can't be terminated because of the containerSet template that was referenced in the DAG task

Summary
The problem occurs when running workflow with DAG task that is referencing containerSet template.
If you will try to terminate the workflow at the moment when DAG nodes have been initialized (in Pending) but not started yet, then the workflow will stuck at the running phase.
I pressed "Terminate" button when nodes step-1 and step-2 were created and the result displayed below:

Diagnostics
The behavior is consistent and can be reproduced with the following workflow:
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  generateName: container-set-termination-demo-
spec:
  entrypoint: main
  templates:
    - name: main
      dag:
        tasks:
          - name: using-container-set-template
            template: problematic-container-set
    - name: problematic-container-set
      containerSet:
        containers:
          - name: step-1
            image: alpine
            command:
            - sh
            - -c
            - "sleep 10"
          - name: step-2
            image: alpine
            command:
            - sh
            - -c
            - "sleep 10"

I have tried to change entrypoint from main to problematic-container-set and then the bug is not reproducible. So the issue is specific to DAG templates.
After the failed termination the workflow has following state:
apiVersion: argoproj.io/v1alpha1
kind: Workflow
metadata:
  annotations:
    workflows.argoproj.io/pod-name-format: v1
  creationTimestamp: "2022-01-25T16:46:10Z"
  generateName: container-set-termination-demo-
  generation: 6
  labels:
    workflows.argoproj.io/phase: Running
    workflows.argoproj.io/resubmitted-from-workflow: container-set-termination-demo-rjx8r
  name: container-set-termination-demo-449lw
  namespace: argo
  resourceVersion: "45572"
  uid: cbbc9bb8-63c5-4ce1-abce-c8fe7192cdea
spec:
  activeDeadlineSeconds: 300
  arguments: {}
  entrypoint: main
  podSpecPatch: |
    terminationGracePeriodSeconds: 3
  shutdown: Terminate
  templates:
  - dag:
      tasks:
      - arguments: {}
        name: using-container-set-template
        template: problematic-container-set
    inputs: {}
    metadata: {}
    name: main
    outputs: {}
  - containerSet:
      containers:
      - command:
        - sh
        - -c
        - sleep 10
        image: alpine
        name: step-1
        resources: {}
      - command:
        - sh
        - -c
        - sleep 10
        image: alpine
        name: step-2
        resources: {}
    inputs: {}
    metadata: {}
    name: problematic-container-set
    outputs: {}
  ttlStrategy:
    secondsAfterCompletion: 600
status:
  artifactRepositoryRef:
    artifactRepository:
      archiveLogs: true
      s3:
        accessKeySecret:
          key: accesskey
          name: my-minio-cred
        bucket: my-bucket
        endpoint: minio:9000
        insecure: true
        secretKeySecret:
          key: secretkey
          name: my-minio-cred
    configMap: artifact-repositories
    key: default-v1
    namespace: argo
  conditions:
  - status: "False"
    type: PodRunning
  finishedAt: null
  nodes:
    container-set-termination-demo-449lw:
      children:
      - container-set-termination-demo-449lw-773192577
      displayName: container-set-termination-demo-449lw
      finishedAt: null
      id: container-set-termination-demo-449lw
      name: container-set-termination-demo-449lw
      phase: Running
      progress: 1/1
      startedAt: "2022-01-25T16:46:10Z"
      templateName: main
      templateScope: local/container-set-termination-demo-449lw
      type: DAG
    container-set-termination-demo-449lw-773192577:
      boundaryID: container-set-termination-demo-449lw
      children:
      - container-set-termination-demo-449lw-2800777053
      - container-set-termination-demo-449lw-2750444196
      displayName: using-container-set-template
      finishedAt: "2022-01-25T16:46:13Z"
      hostNodeName: k3d-argo-workflow-server-0
      id: container-set-termination-demo-449lw-773192577
      message: 'workflow shutdown with strategy:  Terminate'
      name: container-set-termination-demo-449lw.using-container-set-template
      phase: Failed
      progress: 1/1
      startedAt: "2022-01-25T16:46:10Z"
      templateName: problematic-container-set
      templateScope: local/container-set-termination-demo-449lw
      type: Pod
    container-set-termination-demo-449lw-2750444196:
      boundaryID: container-set-termination-demo-449lw-773192577
      displayName: step-2
      finishedAt: null
      id: container-set-termination-demo-449lw-2750444196
      name: container-set-termination-demo-449lw.using-container-set-template.step-2
      phase: Pending
      startedAt: "2022-01-25T16:46:10Z"
      templateName: problematic-container-set
      templateScope: local/container-set-termination-demo-449lw
      type: Container
    container-set-termination-demo-449lw-2800777053:
      boundaryID: container-set-termination-demo-449lw-773192577
      displayName: step-1
      finishedAt: null
      id: container-set-termination-demo-449lw-2800777053
      name: container-set-termination-demo-449lw.using-container-set-template.step-1
      phase: Pending
      startedAt: "2022-01-25T16:46:10Z"
      templateName: problematic-container-set
      templateScope: local/container-set-termination-demo-449lw
      type: Container
  phase: Running
  progress: 1/1
  startedAt: "2022-01-25T16:46:10Z"

What version of Argo Workflows are you running?

v3.2.4

What Kubernetes provider are you using?

k3d

What executor are you running? Docker/K8SAPI/Kubelet/PNS/Emissary

Emissary

Not sure about the root cause right now, but I would like to try and dig in the code in order to find the problematic place. I will update once I will find something.

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Could you please attach the workflow controller logs?
Could you please confirm if the issue persists if you replace the containerSet with container?

@alexec

Could you please attach the workflow controller logs?

container-set-termination-failure-logs.txt

Could you please confirm if the issue persists if you replace the containerSet with container?

I can't reproduce the same issue with container.

You do not need the DAG, but it shows the problem clear.
If you want for the containers to start, then the workflows terminates correctly. It is only when it is in pending that this is a problem.

@daniel-codefresh I think this is where the fix needs to be:
func (woc *wfOperationCtx) handleExecutionControlError(nodeID string, wfNodesLock *sync.RWMutex, errorMsg string) {
	wfNodesLock.Lock()
	node := woc.wf.Status.Nodes[nodeID]
	wfNodesLock.Unlock()
        // make sure that we mark pending child nodes as failed too
	woc.markNodePhase(node.Name, wfv1.NodeFailed, errorMsg)
	for _, nodeID := range node.Children {
		child := woc.wf.Status.Nodes[nodeID]
		woc.markNodePhase(child.Name, wfv1.NodeFailed, errorMsg)
	}
}

Would you like to submit a PR with tests?

@alexec yeah, thanks for the tip. I will try it and submit a PR if it solves the issue


how to run php-parse sdk

I downloaded php-sdk as a composer. How can start php-sdk?
How can I start a local server?

Hi! This module depend php-sdk parse.
Yes, you can run in local server


cocomac: Watch sanaigmore.com

cocomac requests the watch of the watch_keyword sanaigmore\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

sanaigmore\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Makyen in Charcoal HQ


After signaling SIGINT during npm run dev, console throws errors

Bug report
While trying for the first time nextjs got this error during development. Is this normal?
Describe the bug
After signaling SIGINT with control+c in the console during npm run dev the console throws some errors.
To Reproduce
Steps to reproduce the behavior:

Initialize npm
npm init -y
Install the core NextJS dependencies:
npm install --save next react react-dom
Add scripts to the package.json file:

"scripts": {
    "dev": "next",
    "build": "next build",
    "start": "next start"
}

Create pages/index.js file:

import React from "react";

export default () => {
  return (

Hello world

  );
}

Start npm run dev:

 DONE  Compiled successfully in 1801ms

> Ready on http://localhost:3000

Press control+c in the console (this is the buggy behavior):

^Cgot signal SIGINT, exiting
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>dev: `next`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>dev script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!      /Users/Felipe/.npm/_logs/2018-08-07T17_37_32_226Z-debug.log

Expected behavior
I think there shouldn't be any errors while gracefully exiting the app during development.
System information

OS: macOS

{
  "name": "test",
  "version": "1.0.0",
  "description": "",
  "main": "index.js",
  "scripts": {
    "test": "echo \"Error: no test specified\" && exit 1",
    "dev": "next",
    "build": "next build",
    "start": "next start"
  },
  "keywords": [],
  "author": "",
  "license": "ISC",
  "dependencies": {
    "next": "^6.1.1",
    "react": "^16.4.2",
    "react-dom": "^16.4.2"
  }
}

Additional context
debug.log output:
8 verbose lifecycle<EMAIL_ADDRESS>PATH: /usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/node-gyp-bin:/Users/Fe    lipe/Local_Projects/Aprender/nextjs/test/node_modules/.bin:/Users/Felipe/anaconda3/bin:/usr/local/sbin:/usr/local/opt/fzf/b    in:/Users/Felipe/google-cloud-sdk/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin
9 verbose lifecycle<EMAIL_ADDRESS>CWD: /Users/Felipe/Local_Projects/Aprender/nextjs/test
10 silly lifecycle<EMAIL_ADDRESS>Args: [ '-c', 'next' ]
11 silly lifecycle<EMAIL_ADDRESS>Returned: code: 1  signal: null
12 info lifecycle<EMAIL_ADDRESS>Failed to exec dev script
13 verbose stack Error<EMAIL_ADDRESS>dev: `next`
13 verbose stack Exit status 1
13 verbose stack     at EventEmitter.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/index.js:304:1    6)
13 verbose stack     at EventEmitter.emit (events.js:182:13)
13 verbose stack     at ChildProcess.<anonymous> (/usr/local/lib/node_modules/npm/node_modules/npm-lifecycle/lib/spawn.js:5    5:14)
13 verbose stack     at ChildProcess.emit (events.js:182:13)
13 verbose stack     at maybeClose (internal/child_process.js:961:16)
13 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:248:5)
14 verbose pkgid<EMAIL_ADDRESS>15 verbose cwd /Users/Felipe/Local_Projects/Aprender/nextjs/test
16 verbose Darwin 17.7.0
17 verbose argv "/usr/local/Cellar/node/10.7.0/bin/node" "/usr/local/bin/npm" "run" "dev"
18 verbose node v10.7.0
19 verbose npm  v6.3.0
20 error code ELIFECYCLE
21 error errno 1
22 error<EMAIL_ADDRESS>dev: `next`
22 error Exit status 1
23 error Failed at the<EMAIL_ADDRESS>dev script.
23 error This is probably not a problem with npm. There is likely additional logging output above.
24 verbose exit [ 1, true ]

Looked for any similar error but couldn't find any. If this is a duplicate please excuse me.

Nice catch. #4917 fixes it.

@timneutkens Thank you!


[Bug] Default threshold values [-500, 500] in the Threshold Tool [Segmentation Mode] not being initially applied correctly

Describe the Bug
In [Segmentation Mode] when you start a "new segmentation" and add a a segmentation via the "threshold tool" the paintbrush will only select a tiny region of the image (not represented by -500->500) anything at all until the default values are changed (even to the same values [-500,500].
On load :

After retyping 500:

Steps to Reproduce

Select any image set on OHIF Viewer demo (or any version up to 3.8.15) and open in segmentation mode
Click New Segmentation
Select Threshold Tool
Without changing the threshold draw on viewport selected when you created the new segment - only a couple voxels might be selected
Change the upper limit of the threshold to something else then 500 again
Contour again proper -500-> 500 range applied

The current behavior
The initial values for the threshold are not being applied correctly when segmenting the image.
The expected behavior
Default values should be applied when the user starts segmenting using  the threshold tool
OS
Windows 10
Node version
OHIF 3.7 demo
Browser
Chrome Version 119.0.6045.124 (Official Build) (64-bit)

We just release the OHIF 3.8, you can find more details here https://ohif.org/release-notes/3p8/
If you still encounter this issue in 3.8, please re-open this.


Using chrome driver

Does codeceptjs support alternate webdrivers. I tried starting selenium with the chrome driver. I'm on a mac (10.11). I downloaded the chromedriver, downloaded the selenium jar file.
Starting selenium like this:
java -Dwebdriver.chrome.driver="./chromedriver" -jar selenium-server-standalone-2.48.2.jar
Then my codecept.json file
{
  "tests": "./*_test.js",
  "timeout": 10000,
  "output": "./output",
  "helpers": {
    "WebDriverIO": {
      "url": "http://test.localhost",
      "browser": "chrome"
    }
  },
  "include": {},
  "mocha": {},
  "name": "Testing"
}

However when I run $ codeceptjs run it complains about firefox not being installed. I presume selenium by default ships with the firefox driver.
Sorry I am completely new to selenium.
--
Leon

@lvismer it's fixed in the latest commit, and should be working fine now.
https://github.com/Codeception/CodeceptJS/pull/4

@petehouston Thanks, that fixed it for me.

@lvismer cool to hear about that :+1:

I just published a new version with that PR. Please update to 0.1.6


Require at least one non-partition column for Hive tables

It's already fixed in 12247d5 by @raghavsethi.


feat: add benchmark

Codecov Report

Merging #12 (a6ed0e2) into main (f1c78f5) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head a6ed0e2 differs from pull request most recent head 18bee3a. Consider uploading reports for the commit 18bee3a to get more accurate results

@@             Coverage Diff             @@
##                main       #12   +/-   ##
===========================================
  Coverage     100.00%   100.00%
  Complexity        25        25
===========================================
  Files             10        10
  Lines             39        39
===========================================
  Hits              39        39

Flag
Coverage Δ

unittests
100.00% <ø> (ø)

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

Continue to review full report at Codecov.

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


Could I get help centering the contents of the modal?

I'm not very good at CSS. I'm trying to center a loader in the modal both vertically and horizontally but having quite a difficult time doing so. I'm using halogen as my modal and the code is below. I've tried various different combinations of CSS but failing to get both the vertical and  horizontal centering correct.
I'm not using flex, just bootstrap grids. If I could get some pointers would be greatly appreciated.
import { RingLoader } from 'halogen';
import React from 'react';
import ReactModal from 'react-modal';
import {connect} from 'react-redux';

class SpinnerModal extends React.Component {

  render() {

    return (

    <ReactModal
      isOpen={this.props.isLoading}
      contentLabel="Modal"
      style={{
        overlay : {
        },
        content : {
          position                   : 'absolute',
          top                        : '0px',
          left                       : '0px',
          right                      : '0px',
          bottom                     : '0px',
          border                     : 'none',
          background                 : '#fff',
          overflow                   : 'auto',
          WebkitOverflowScrolling    : 'touch',
          outline                    : 'none',
          backgroundColor            : 'rgba(200, 200, 200, 0.85)'
        }
      }}
      /*
       String className to be applied to the portal.
       See the `Styles` section for more details.
       */
      portalClassName="ReactModalPortal"
      /*
       String className to be applied to the overlay.
       See the `Styles` section for more details.
       */
      overlayClassName="ReactModal__Overlay"
      /*
       String className to be applied to the modal content.
       See the `Styles` section for more details.
       */
      className="ReactModal__Content"
      /*
       Boolean indicating if the appElement should be hidden
       */
      ariaHideApp={false}
      /*
       String indicating the role of the modal, allowing the 'dialog' role to be applied if desired.
       */
      role="dialog"
      /*
       Function that will be called to get the parent element that the modal will be attached to.
       */
      parentSelector={() => document.body}
    >
      <RingLoader color="#428bca" size="100px" margin="4em" styleName='loader-style'/>
    </ReactModal>

    );
  }
}

SpinnerModal.PropTypes = {
  isLoading: React.PropTypes.bool.isRequired
};

export default connect(state => state.app)(SpinnerModal);

Figured it out by using the following CSS styling.
.loader-style {
    position: absolute;
    top: 40%;
    left: 50%;
    margin-top: -50px;
    margin-left: -50px;
}


New version: Unity.Unity.2022 version 2022.3.31f1

Created by 🥟 Dumplings in workflow run #5502.
Logs
Updated: 2022.3.30f1 → 2022.3.31f1
Submitting WinGet manifests

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

Hi @SpecterShell,
This manifest removes Apps and Features entries that are present in previous PR versions. These entries should be added to this version, to maintain version matching, and prevent the 'upgrade always available' situation with this package.
(Automated response - build 901.)

@wingetbot run

This package installs and launches normally on a Windows 10 VM.

Hi @SpecterShell,
This manifest removes Apps and Features entries that are present in previous PR versions. These entries should be added to this version, to maintain version matching, and prevent the 'upgrade always available' situation with this package.
(Automated response - build 901.)

Hi @SpecterShell,
This manifest removes Apps and Features entries that are present in previous PR versions. These entries should be added to this version, to maintain version matching, and prevent the 'upgrade always available' situation with this package.
(Automated response - build 901.)


Fix consecutive uppercase characters in camelCase conversion

Consecutive uppercase characters were ignored, as (obviously) there was no lowercase character before it and did not match [a-z][A-Z]. The [A-Z]+ matches all characters and a for-loop is used to iterate over all characters.
Also added a testsuite to test the case-map functionality.
Fixes #3206

Waiting for the response on #3279. If that PR gets merged, I will close this one.

https://github.com/Polymer/polymer/pull/3208 resolves this issue also; @azakus will review both and choose the :trophy: winner :trophy:!

@azakus I did the performance test as created by @nazar-pc to compare the result. My base-case was consistenly twice as fast as nazar's, so that would be hugely dependent of the hardware.
before-1: 500.837ms
before-2: 508.146ms
after-1: 244.371ms
after-2: 382.750ms
after-nazar-1: 240.780ms
after-nazar-2: 221.723ms

Executing the test multiple times, nazar's implementation proved to be slightly faster. Therefore I will change this PR to just include the tests and then the implementation of nazar could be merged.

The CI build currently fails, as the original is not fixed. When #3279 is merged and this branch is rebased, the build will succeed.

@TimvdLippe Thanks for the perf test! LGTM

#3279 merged, merging this now.

Ha, I was a bit hasty. This won't get run automatically, but must be listed in test/runner.html. I'll do that in a different commit.

Oops yes you are correct. Sorry!


Update devtest-lab-faq.md

Remove unnecessary parenthesis.

Hi @alexandair, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
We will now validate the agreement and then real humans will evaluate your PR.
TTYL, AZPRBOT;

@alexandair : Thanks for your contribution to the Azure documentation! The author, @tomarcher, has been notified to review your proposed change.

@alexandair Thanks for your contribution. As you mention, the parens are completely necessary. However, they are there more for readability than function.

@TomArcher I don't understand how they increase the readability. Quite the opposite. They are not used in that way in any other article. If that's a good practice, everyone will use it. Please, remove them.

@tysonn #sign-off


Rename filters for user experience

Filters are named such as 'requestHeader' and 'requestMirror' which make sense internally in the package but can be confusing for a user using our CRDs.
More context: https://github.com/kubernetes-sigs/service-apis/pull/276#discussion_r485273444

@hbagdi are you still able to take this one?

Yes, apologies for the delay here.
I started making the patch for this but had second thoughts again.
In my thinking around this, the filters and their names are nouns and not verbs. Does it make sense to keep filters as nouns for our use case?

@hbagdi I submitted https://github.com/kubernetes-sigs/service-apis/pull/408 to fix this issue. Feel free to reassign yourself if you prefer your WIP PR over mine.
/assign


Anomaly Maps and Anomaly Scores

Hello @TArdelean,
I am trying to run this code as per the steps mentioned in the README on the carpet class of MVtec dataset. The anomaly maps looks good for the defective classes, however for the good class also the anomaly maps shows very strong highlighting. And how we can get the image level anomaly scores. Is it just the max(anomaly_maps)?

[Good class]

[Defective class(hole)]

Thanks.

Hi @rishabh-akridata,
Firstly, thanks for your interest in our work.
The reason the anomaly maps you show have large activations even for good image is because the maps are normalized for visualization; therefore all maps will have at least one pixel with maximum brightness. One way to get around this is to normalize the scores at level of the dataset instead of normalizing at the level of each image.
And to your second question: yes, the image-level anomaly score is usually computed as the maximum value in the anomaly map.
Best,
Timotei

Hi @TArdelean,
Is it possible to give the evaluation code for AUROCi and AUROCp calculation as well as mentioned in the paper? Because the current code only gives the NMI and clustering related metrics.

Thanks

Hi,
For the anomaly maps metrics, please refer to the evaluation code from the authors of the MVTecAD dataset: https://www.mvtec.com/company/research/datasets/mvtec-ad. This integration of the evaluation code might also prove useful: https://github.com/TArdelean/AnomalyLocalizationFCA.
Best regards,
Timotei

Thanks.


[cpp] Prototype compilation

For backends, it's much easier to just work in the mainline fork: https://github.com/sureshjoshi/pants/tree/cc-first-party-compilation

Using this repo right now for example code, and then pants_from_sources for the cc stuff.

See https://github.com/pantsbuild/pants/pull/16424


[ROS2] Backport removal of viewpoint object

Problem Description
Backport of https://github.com/ethz-asl/terrain-navigation/pull/29

@Jaeyoung-Lim there are some conflicts here because the code in the header has been moved to a cpp in the ros2 branch. I can make the updates and force push if you'd like.

@Jaeyoung-Lim - if you are ok with the changes in #48 I can squash them with the commits in this PR and force push (or do the rebase merge of #48 then squash after).

@srmainwaring Wow amazing! Thanks!


fix: adjust TARS scan period to one hour

Because the updates for push events work well, I think the period can be adjusted to be larger. This way there may be as little disturbance to everyone as possible. And it will save testing resources.

/lgtm

/merge


Create fake model with GB sizes

What needs to happen?
We need to create models of various sizes (1 GB, 5GB, 10 GB) to help with benchmarking RunInference on models of various sizes. These models don't have to be trained on anything, but be randomly initialized, fully-connected networks.
Subtask of issue https://github.com/apache/beam/issues/21435
Issue Priority
Priority: 2
Issue Component
Component: sdk-py-core

Planning to use off-the-shelf models.


Can't get rule34.us booru type.

Found model.ts to that booru but can't see it in booru types list, also can't choose automatically => so cant get any images from it.
Not sure that's bug or temporary disabled feature.

Compiled by myself, missing sites appeared but they still missing in latest 7.9.1 release


HOSTEDCP-1285: Consume kas Pod port by name

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/test unit

/lgtm

/test e2e-kubevirt-aws-ovn


Maybe a Bug in "List Edit" regarding Categories

This happens in the Web UI 0.7.1. Maybe a bug, maybe a misunderstanding on my side.

Go to a list
Type in a new item in the Search field and press ENTER to create it. Fill out all mandatory fields and realize that you do not have a fitting category for the item yet.
Save the Item.
Go to Categories.
Create a new Category.
Go back to the List, open the Item and assign the Category, Save the Item.
Click on the Pen right of the List to edit the list.
The Category will be at the bottom of the list in deativated state.

Although the category is deactivated, the item stays in the list, so at least you do not miss it when you go shopping ;-)
But I would expect the category to be activated, since it is used by an active item in the current list.
Bonus:
After finishing my List in the Web UI, open the Android App on my Phone. The app flickers a bit while it synchronizes the current state and I see all my Items, but the categories are not in the same order as in the Web UI. Switch to another list and back to the list I just edited and the sorting is OK. I suppose the items + sort order are synchronized correctly, but the sort order is only applied once you open the list again?

As for the initial issue, I have allowed for the creating of new units of measure and new categories directly from the Item creation/update screen so you don't have to do all those movements back and forth. Just hit the "+" button next to the UOM or Category drop-downs and a modal will open up allowing you to create a new one from there.
That being said, I don't currently have logic that when a category is added it becomes "active" on all lists. Even if the category is not "active" though, the item still shows up on the list if you have added it.  This would potentially only come into play if you had the setting "Add with same categories automatically".  In this case, let's say you created a category called "Organic Fruit", and maybe you had 2 different lists in a list group. At one of the stores, let's say they don't carry organic fruit. You could then uncheck that category at the store. In this case, if you added a new item "Organic Bananas" to the list, AND you had the setting to "Add with Same Categories automatically", it would add the organic bananas only to the 1 list and not the other.  All depends on how you shop/organize your lists.
I could certainly add some logic to automatically add any new categories to all available lists, just not sure that that is the common desired behavior either, and I am a bit loathe to continue to add new settings.  For instance, let's say you have two list groups - one for grocery stores and another for hardware stores. You might only have 5-6 categories at the hardware store that are relevant -- nuts/bolts, lawn supplies, etc. You might NOT want to add a new category "Organic Fruit" to that list automatically.
I'm also not seeing this behavior around sorting the lists or categories on Android differently, or needing a "refresh". Need to be able to replicate that in order to address.

The topic Category creation and active / inactive is solved for me after your explanation. That all makes sense to me and I do not think it requires another logic. More so you only might create a new category here or there when you start fresh and once it is tailored to your needs you might never touch it again. Thanks for sharing your thoughts behind the design process, that cleared all up for me.
That sorting not happening instantly might also have been a hiccup in the sync / my network connection or some caching issue outside of your application. It all looks good today. I will keep an eye on it and if I think I can reproduce it, I will let you know how. Regarding the emulator / simulator, I do not have one (yet). We have 2 Android phones (Mi 9 Lite and Poco F1), both with Lineage, so if something else comes up I make sure I test with both devices first.


fix testcafe show stoppers (close #148, close #149, close #150)

\cc @inikulin @AlexanderMoskovkin

\cc @VasilyStrelyaev check comments pls

lgtm

/r-

FPR

lgtm

lgtm

Lgtm


Better API Key access

To make updating easier, we should store the API token in a separate file which is untracked, so we don't have to keep adding/removing it while updating the github repo.

This has been fixed now.


Make Host#refresh_firewall_rules operate via SupportsFeatureMixin

This is only supported for Vmware Infra provider (see https://github.com/search?q=org%3AManageIQ+refresh_firewall_rules&type=code)

@miq-bot cross-repo-test #21026 including ManageIQ/manageiq-providers-vmware#697

@miq-bot cross-repo-test #21026 ManageIQ/manageiq-providers-vmware#697

@miq-bot cross-repo-test #21026, ManageIQ/manageiq-providers-vmware#697

@miq-bot cross-repo-test ManageIQ/manageiq-providers-vmware#697

@agrare cross-repo tests are green


can not work with NaughtyAttributes

NaughtyAttributes is a github repo ,and Editorbutton can not work will whit that plugin ,

and can not work with InspectorFoldoutGroup


Correct spelling of MinGW

John Scott wrote:

nothing.

I'll change it.
--
Facepalm statement #1: "I'm going to New York tomorrow, hopefully I have time
to visit the White House"
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \
\\  an exciting new programming language -- http://www.Zimbu.org        ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


When stack name is edited, there is no metadata changes being pushed resulting in DNS resolution for the service with the new stack name to fail.

Rancher versions: Build from v1.6
Steps to reproduce the problem:
Create a stack - hello3 with service - t1.
Edit stack name from hello3 to hello4.
Even after this ,  t1.hello4  does not resolve and t1.hello3 continues to resolve.
Once metadata reload gets triggered ,  t1.hello4  resolves as expected.

Tested with latest build from v1.6
Create a stack - hello3 with service - t1.
Edit stack name from hello3 to hello4.
t1.hello4 resolves and t1.hello3 does not resolve anymore.


Create animals table

In this milestone, we have:

Created the database (vet_clinic)
Created one table (animals)
Made the insertion of some animals in the "animals" table
Executed the following queries:

Find all animals whose name ends in "mon":

- List the name of all animals born between 2016 and 2019:

- List the name of all animals that are neutered and have less than 3 escape attempts:

- List the date of birth of all animals named either "Agumon" or "Pikachu":

- List name and escape attempts of animals that weigh more than 10.5kg:

- Find all animals that are neutered:

- Find all animals not named Gabumon:

- Find all animals with a weight between 10.4kg and 17.3kg (including the animals with weights that equals precisely 10.4kg or 17.3kg):

Hi @luis-pomare 😊,
While you made a great effort in this project, I cannot proceed to review your code.
Invalid Code Review Request 🛑 💀
You have submitted a project, that hasn't correct git-flow, you should open a PR from the feature branch to the development branch, not the main. Also, consider customizing the readme for this project and removing the placeholder text and making sure to include all sections in the readme and fill them with related info
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


Fix duelling loaders on the server

The issue is that on the server we have 2 competing loaders right now. The loader in the Realm and the loader in the SearchIndex. The loader in the Realm is actually never intended to be used operationally. Rather it serves as a "template" that the SearchIndex clones each time there is an indexing operation. The latest and greatest loader lives in the SearchIndex, however, the Realm does actually use it's own loader (which never gets updated from module writes) for serialization activities.
In this PR we rename the Realm.#loader so it's more clear that we don't operationally ever use it, and rather only use it as a means to clone SearchIndex loaders. As well as, the Realm.loader should be an alias to the SearchIndex.loader so that the file serialization logic uses the latest loader.
In terms of reproducing the bug CS-6354 locally, where there was a discrepancy between the browser's version of the card def and the server's version of the card def, it was a bit hit or miss for me. Initially I was able to reproduce this bug locally, however, occasionally I am not able to reproduce the bug locally. So I'm not 100% sure about that the Realm.loader is the true culprit here, there may be some async as well at play (?)--but as it is written it is definitely incorrect.
I'd like to get this into staging and then see where we stand in terms of reproducing CS-6354 (which i was able to reproduce pretty easily in staging)

i think i found another suspicious use of loaders. Recall that our indexing actually happens in an ember app that runs in fastboot. The entrypoint into this is the Card-Prerender component. We use a set of callbacks to wire together the SearchIndex's #fromScratch() and #incremental() indexing methods to this component's fromScratch() and incremental() methods, which kick off the indexing. As part of this, this component is using the LoaderService.loader to run card's whose serialized docs are placed in the index. This loader is never reset--that seems wrong and could result in stale card def's being used to run cards. We can make sure to reset the loader service (that runs in the ember app in fastboot) before we start any indexing operations. this should ensure that all modules in the loader at index time are fresh. I'm working thru test failures with this update now...

i think i found another suspicious use of loaders. Recall that our indexing actually happens in an ember app that runs in fastboot. The entrypoint into this is the Card-Prerender component. We use a set of callbacks to wire together the SearchIndex's #fromScratch() and #incremental() indexing methods to this component's fromScratch() and incremental() methods, which kick off the indexing. As part of this, this component is using the LoaderService.loader to run card's whose serialized docs are placed in the index. This loader is never reset--that seems wrong and could result in stale card def's being used to run cards in fastboot. We can make sure to reset the loader service (that runs in the ember app in fastboot) before we start any indexing operations. this should ensure that all modules in the loader at index time are fresh. I'm working thru test failures with this update now...

This was result on what we paired on previously but I left off and i hadn't continued.
I just published the branch https://github.com/cardstack/boxel/tree/add-loader-to-card-prerender and the main commit is 53d257c so you can see the corresponding changes for this

i think all this work has been superseded by the new test realm setup that Luke and Ed worked on.

Alright the test that I wrote reproduces the issue precisely as @lukemelia describes in the test and this PR fixes the broken test.


Infer ignore markers on iOS code

It's possible to have a special markers inside iOS to insert when a code snippet has not to be inspected by infer?
e.g.

@implementation MyClass
-(void)myMethod {
//inspected code
NSString *prettyVariable = @"Pretty variable";
###INFER-IGNORE
//uninspected code
NSNumber *nullNumber = nil;
NSArray *badArray = [NSArray arrayWithObjects:nullNumber, nil];
###INFER-IGNORE
//inspected code
NSString *anotherPrettyvariable = @"Another pretty variable";
}
@end

No, we don't have support for that at the moment unfortunately, but if you want to silence NULL_DEREFERENCE, you can add Nonnull annotations.


When using DrawerNavigator and connecting to redux the store gets cleared when the drawer opens

Today I was starting to put together navigation in a app I am beginning working on and wanted to use a drawer. In addition I want to keep the nav state in my redux store. After setting up what I think should be a working drawer navigation I noticed that when I slide open the drawer menu the reducer ends up returning null for its state which then exceptions out complaining that it can't find index on the navigation state object (which is true because the navigation state object is now null). I was able to put together a simple example here:
https://github.com/rmevans9/react-navigation-drawer-redux-test/blob/master/index.ios.js
If you run this and just slide out the drawer you will get a RSOD complaining about the index. I did a little bit of digging in the code and found that the null is being returned from here:
https://github.com/react-community/react-navigation/blob/master/src/routers/TabRouter.js#L158
I am still trying to learn the ins and outs of how React Navigation internals work so I am not sure if a null return is valid (or why it would be valid... typically if you don't want to change state then you return the current state). I am planning to dig more into this tomorrow but wanted to post this up here and see if anyone could either point out where I messed up in my example or if this is actually a bug in React Navigation.

So digging into why this issue shows in a redux setup but does not if using the internal state management I came across this bit of code:
dispatch = (action: NavigationAction) => {
    const { state } = this;
    if (!this._isStateful()) {
        return false;
    }
    const nav = Component.router.getStateForAction(action, state.nav);

    if (nav && nav !== state.nav) {
        if (console.group) {
            console.group('Navigation Dispatch: ');
            console.log('Action: ', action);
            console.log('New State: ', nav);
            console.log('Last State: ', state.nav);
            console.groupEnd();
        } else {
            console.log('Navigation Dispatch: ', { action, newState: nav, lastState: state.nav });
        }
        this.setState({ nav });
        return true;
    }

    return false;
};

from: https://github.com/react-community/react-navigation/blob/master/src/createNavigationContainer.js#L132
Looking at how it only updated the actual state if the getStateForAction returned a value I tried that in a redux reducer that basically just calls the getStateForAction method and only returns the updated state if a state is returned:
function navWorkaroundReducer(action, state) {
    const newState = {...}.getStateForAction(action, state);

    if (newState) {
        return newState;
    }

    return state;
}

And it worked. So the way I see it is there are two options:

Update the documentation for redux integration calling this out
Stop returning null in getStateForAction (which might break things when not using Redux...)

Thoughts?

What we discussed in past was returning an object like { state, hasChanged } or similar instead of returning null. Maybe we could do that.

That would work - doing that would allow us to setup the reducer to always just return {...}.getStateForAction(action, state).state which I think I like better then having to do the workaround I suggest above.

Just ran into this issue today, using 1.0.0-beta.3, has this been fixed? I'm gonna upgrade to 1.0.0-beta.5 and try it out

Sorry for the examples arguments being switched, wasn't thinking when I put it up... I have it in its own function in my code and I do the switching of the args when I call that function. But you are correct, it hasn't been fixed yet but this work around is working pretty well right now.

I simplified it even more:
return AppRoutes.router.getStateForAction(action, state) || state
You can reassign state if you're planning on modifying it, which I'm sure you are, since it's a reduced after all!
state = AppRoutes.router.getStateForAction(action, state) || state

Hi! Since this issue has been quiet for a while I'm going to go ahead and close it. If the issue persists in the latest version of React Navigation please re-open!

Same problem, 1.2 version. Redux store is reset when I open the drawer, crashes my app main page.

@cayodonatti I had the same issue, but the solution above worked for me.

Help out here, i discover that my state is mutating in when i use redux with react navigation. please help on this.
import Routes from './../Navigation/Routes'
import { NavigationActions } from 'react-navigation'

const initialState = Routes.router.getStateForAction(NavigationActions.init())

const navReducer = (state = initialState, action) => {
  const nextState = Routes.router.getStateForAction(action, {...state})
  return nextState || state
}
export default navReducer

hi there! if this is still a bug can you create a new issue that reproduces the bug on https://snack.expo.io? thanks!


Refactor: Copying images.json file on the base stack, only if the file exists

Summary

This PR places the code that copies the images.json file to the base stack, inside the if statement. That way we ensure that images.json is always available, avoiding any unexpected errors.
Practically this PR is a minor refactor of this PR: https://github.com/paketo-community/ubi-base-stack/pull/94
Use Cases

Checklist

[x] I have viewed, signed, and submitted the Contributor License Agreement.
[ ] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[x] I have reviewed the styleguide for guidance on my code quality.
[x] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@mhdawson


Enable Jaeger tracing, Tokio Console, file logging for wallet

Adds a couple backends for Tokio tracing, including Jaeger output support, a subscriber for Tokio console, as well as rotating file logging for the wallet CLI.  For now, configurable using env vars.
NOTE: I'm going to move the subscriber setup out of sui.rs and wallet.rs and to Mysten-infra

Nice one @velvia