Adding Test cases

Purpose
Port test cases implemented in test hackathon to master branch
Documentation
N/A
Certification
N/A
Automation tests

Unit tests
coverage-86%

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Fixed in #25


Analysis content not properly visible.

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The analysis part has a lighter background along with light font color making it difficult for users to read it.
Add ScreenShots
No response
What browsers are you seeing the problem on?
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC contributor
[X] I want to work on this issue
[X] I'm willing to provide further clarification or assistance if needed.

Can you please assign this task to me. @sanjay-kv

this is already done, if not attach a screenshot for reference


[php 7.0.*]Uninitialized string offset: 0 1. in /data/wwwroot/app/vendor/yiisoft/yii2/web/Request.php

What steps will reproduce the problem?
I just run xxx/index.php in php 7.0.1, how ever there is a notice error of Uninitialized string offset: 0
come from in /data/wwwroot/webstore/vendor/yiisoft/yii2/web/Request.php
line 726
if ($pathInfo[0] === '/') {
$pathInfo = substr($pathInfo, 1);
}
the reason may be that under php 7.0.* , $pathInfo is false and no string offset 0 is existed.
What is the expected result?
the correct index page
What do you get instead?
I just add the data type checking.
if (is_array($pathInfo)&&$pathInfo[0] === '/') {
$pathInfo = substr($pathInfo, 1);
}
Additional info

Q
A

Yii version
2.0.6

PHP version
php 7.0.3

Operating system
window&centos

It has been fixed in v2.0.7.

by the way，how about the intlException ?
vendor/yiisoft/yii2/i18n/MessageFormatter.php
try{
$formatter = new \MessageFormatter($language, $pattern);
}catch(\IntlException $e) {
$formatter= null;
}

I add a try catch for fixing it temporarily.


Update messaging.md

Added Azure Communication Services as an email sending option

Learn Build status updates of commit 45e293d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/aws-professional/messaging.md
:warning:Warning

Details

docs/aws-professional/messaging.md

Line 24, Column 133: [Warning: hard-coded-locale - See documentation] Link 'https://azure.microsoft.com/en-gb/products/communication-services/' contains locale code 'en-gb'. For localizability, remove 'en-gb' from links to most Microsoft sites.
Line 6, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'bogenera' is not a valid Microsoft alias.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Learn Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

@EdPrice-MSFT
Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Learn Build status updates of commit 28714de:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

docs/aws-professional/messaging.md
:bulb:Suggestion

Details

docs/aws-professional/messaging.md

Line 6, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'bogenera' is not a valid Microsoft alias.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

Learn Build status updates of commit b2357f3:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

docs/aws-professional/messaging.md
:bulb:Suggestion

Details

docs/aws-professional/messaging.md

Line 5, Column 12: [Suggestion: ms-author-invalid - See documentation] Invalid value for 'ms.author', 'architectures' is not a valid Microsoft alias.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

#Sign-Off
"architectures" is an alias.

@EdPrice-MSFT - Thank you for updating the author metadata as well!


BZ1208368: added testcase

Please, do not merge yet.

Duplicated call of view.showLoading() removed. I should be done.

http://github.com/droolsjbpm/kie-wb-common/commit/a1d02e646


Reachability test failed: no such host. But host is available

Hello! I am trying to setup kube-lego for my minio server.
After applying tls configuration for it, I see this messages in log of kube-lego:
time="2017-11-16T10:55:10Z" level=debug msg="testing reachability of http://s3.kubernetes.mydomain.com/.well-known/acme-challenge/_selftest" context=acme domain=s3.kubernetes.mydomain.com
time="2017-11-16T10:55:10Z" level=debug msg="error while authorizing: reachability test failed: Get http://s3.kubernetes.mydomain.com/.well-known/acme-challenge/_selftest: dial tcp: lookup s3.kubernetes.mydomain.com on <IP_ADDRESS>:53: no such host" context=acme domain=s3.kubernetes.mydomain.com

But if I try to curl it from master or another host, I get proper response.
curl http://s3.kubernetes.mydomain.com/.well-known/acme-challenge/_selftest
UtvITnmTxXWbIrGc

Without certs, service is accessible an works fine.
Where I need to dig to solve this problem?

I am having  the same problem. If you closed the issue, could you explain whether you did solve the issue and how?


Add GUI entry for ip_change_kill_states config parameter

The hidden parameter is present in pfsense since November 2014 but there was no GUI entry to enable/disable this parameter. This parameter is extremly helpful for unstable WAN connections using VOIP.
This is my first contribution to pfsense. I hope my commits are compliant to the pfsense coding style.

Wording corrected.


Unscramble TOPMed sample swaps

Notes from Google Doc:

Swap on loading for now, but re-write in MT for N+K callset
We need to track this in the meta table: use new IDs in s but keep extra annotation for old ID (for tracking purposes)

Added to the following PR https://github.com/broadinstitute/gnomad_meta/pull/1/files


Unable to install Twill 3 in a fresh Laravel 10 project

Description
Unable to install Twill 3 in a fresh Laravel 10 project.
I know it's an issue "down the dependency tree" but still this prevents to install/require twill.
With this issue I just want to make this known. I also try to look into it.
Steps to reproduce

Create a fresh Laravel 10 project: composer create-project laravel/laravel twill3 "^10.0"
Change working directory: cd twill3
Require Twill 3: composer require area17/twill "^3.0"

Expected result
Twill 3 be installed.
Actual result
Twill 3, due to requirement conflicts, is not installed:
❯ composer require area17/twill "^3.0"
./composer.json has been updated
Running composer update area17/twill
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - league/glide[2.0.0, ..., 2.2.0] require league/flysystem ^2.0 -> found league/flysystem[2.0.0, ..., 2.5.0] but the package is fixed to 3.14.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - league/glide[2.2.1, ..., 2.2.3] require psr/http-message ^1.0 -> found psr/http-message[1.0, 1.0.1, 1.1] but the package is fixed to 2.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - league/glide-symfony 1.0.0 requires symfony/http-foundation ^2.3 -> found symfony/http-foundation[v2.3.0, ..., v2.8.52] but the package is fixed to v6.2.8 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - league/glide-symfony[1.0.1, ..., 1.0.2] require symfony/http-foundation ^2.3|^3.0 -> found symfony/http-foundation[v2.3.0, ..., v2.8.52, v3.0.0, ..., v3.4.47] but the package is fixed to v6.2.8 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - league/glide-symfony 1.0.3 requires symfony/http-foundation ^2.3|^3.0|^4.0 -> found symfony/http-foundation[v2.3.0, ..., v2.8.52, v3.0.0, ..., v3.4.47, v4.0.0, ..., v4.4.49] but the package is fixed to v6.2.8 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - league/glide-symfony[1.0.4, ..., 1.1.1] require symfony/http-foundation ^2.3|^3.0|^4.0|^5.0 -> found symfony/http-foundation[v2.3.0, ..., v2.8.52, v3.0.0, ..., v3.4.47, v4.0.0, ..., v4.4.49, v5.0.0, ..., v5.4.22] but the package is fixed to v6.2.8 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - league/glide-laravel 1.0.0 requires league/glide-symfony ^1.0 -> satisfiable by league/glide-symfony[1.0.0, ..., 1.1.1].
    - area17/twill[3.0.0, ..., 3.0.1] require league/glide-laravel ^1.0 -> satisfiable by league/glide-laravel[1.0.0].
    - league/glide-symfony 1.1.0 requires league/glide ^2.0 -> satisfiable by league/glide[2.0.0, ..., 2.2.3].
    - Root composer.json requires area17/twill ^3.0 -> satisfiable by area17/twill[3.0.0, 3.0.1].

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

Versions

Twill version:  3.0.1 (^3.0)
Laravel version: 10.1.1 (^10.0)
PHP version: 8.2.5

FWIW, delete the composer.lock from the fresh Laravel 10 install before installing Twill 3 works, but obviously, we don't want to have to do that.
I just pushed a branch removing the league/glide-laravel dependency, but now we have another conflict:
% composer require area17/twill "dev-fix-laravel-10-install"
./composer.json has been updated
Running composer update area17/twill
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - google/auth[v1.10.0, ..., v1.17.0] require guzzlehttp/psr7 ^1.2 -> found guzzlehttp/psr7[1.2.0, ..., 1.9.1] but the package is fixed to 2.5.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - google/auth[v1.12.1, ..., v1.26.0] require psr/http-message ^1.0 -> found psr/http-message[1.0, 1.0.1, 1.1] but the package is fixed to 2.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - spatie/laravel-analytics[4.0.0, ..., 4.0.1] require laravel/framework ^7.0|^8.0 -> found laravel/framework[v7.0.0, ..., v7.30.6, v8.0.0, ..., v8.83.27] but it conflicts with your root composer.json require (^10.8).
    - spatie/laravel-analytics 4.1.0 requires laravel/framework ^9.0 -> found laravel/framework[v9.0.0, ..., v9.52.6] but it conflicts with your root composer.json require (^10.8).
    - spatie/laravel-analytics 4.1.1 requires google/apiclient ^2.12.6 -> satisfiable by google/apiclient[v2.12.6, v2.13.0, v2.13.1, v2.13.2].
    - area17/twill dev-fix-laravel-10-install requires spatie/laravel-analytics ^4.0 -> satisfiable by spatie/laravel-analytics[4.0.0, 4.0.1, 4.1.0, 4.1.1].
    - google/apiclient[v2.12.5, ..., v2.13.2] require google/auth ^1.10 -> satisfiable by google/auth[v1.10.0, ..., v1.26.0].
    - Root composer.json requires area17/twill dev-fix-laravel-10-install -> satisfiable by area17/twill[dev-fix-laravel-10-install].

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

We need this PR on Google's side to be merged.
The issue is due to https://github.com/spatie/laravel-analytics. Currently, if you try to install that package in a fresh Laravel 10 application, you get the same issue.
I may decide to make it an optional dependency if this is not solved quickly, since it is only used when the analytics module is enabled on the dashboard, and it is not by default.
You can test using composer require area17/twill "dev-fix-laravel-10-install"

Good news @ifox
https://github.com/googleapis/google-auth-library-php/pull/449 was merged.
Now the steps from the initial issue here work just fine.
composer show  | grep -E "laravel/framework|area17/twill"
area17/twill                        3.0.1     Twill is an open source CMS toolkit for Laravel that helps developers rapidly create a custom admin console that is intuitive, powerful and flexible.
laravel/framework                   v10.9.0   The Laravel Framework.

Bad news @ifox
it's broken again. This time because of league/flysystem-aws-s3-v3 / aws/aws-sdk-php.
composer require area17/twill "^3.0"

Info from https://repo.packagist.org: #StandWithUkraine
./composer.json has been updated
Running composer update area17/twill
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - aws/aws-sdk-php[3.132.4, ..., 3.173.19] require guzzlehttp/psr7 ^1.4.1 -> found guzzlehttp/psr7[1.4.1, ..., 1.9.1] but the package is fixed to 2.5.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - aws/aws-sdk-php[3.173.20, ..., 3.198.8] require guzzlehttp/psr7 ^1.7.0 -> found guzzlehttp/psr7[1.7.0, ..., 1.9.1] but the package is fixed to 2.5.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - aws/aws-sdk-php[3.199.0, ..., 3.271.8] require guzzlehttp/promises ^1.4.0 -> found guzzlehttp/promises[1.4.0, ..., 1.5.3] but the package is fixed to 2.0.0 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - area17/twill[3.0.0, ..., 3.0.2] require league/flysystem-aws-s3-v3 ^3.0 -> satisfiable by league/flysystem-aws-s3-v3[3.0.0, ..., 3.15.0].
    - league/flysystem-aws-s3-v3[3.12.1, ..., 3.15.0] require aws/aws-sdk-php ^3.220.0 -> satisfiable by aws/aws-sdk-php[3.220.0, ..., 3.271.8].
    - league/flysystem-aws-s3-v3[3.0.0, ..., 3.10.3] require aws/aws-sdk-php ^3.132.4 -> satisfiable by aws/aws-sdk-php[3.132.4, ..., 3.271.8].
    - Root composer.json requires area17/twill ^3.0 -> satisfiable by area17/twill[3.0.0, 3.0.1, 3.0.2].

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

IMHO AWS support should be optional anyways; Laravel does it too: for example https://laravel.com/docs/10.x/mail#ses-driver

Yeah, I noticed it yesterday. We will make most dependencies optional in the next major version. FWIW, the incompatibility is only present with an existing composer.lock file. It is somewhat sketchy from composer that there is no incompatibility when installing without a lock file. The result should be the same if done at the same time, I feel.

rm composer.lock did the trick 🙃


nixos/gitea: Move to lib.getExe

Description of changes
This changes the binary calls in the module from being hardcoded to using lib.getExe, which allows usage of alternative packages like pkgs.forgejo. All names and defaults for the module remain the same, which means that Forgejo is now a drop-in replacement for gitea. I have already been testing this in my personal git forge, and it has worked perfectly.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[x] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

That's been fixed. Tests now run, forgot to run them after updating.


Issue with the analytics example

Some JS are broken in there:

looks like a problem with https://github.com/manzt/anywidget - pinning an older version might fix the issue


Weights are not updating

Hi! My weights don't seem to be changing.
I printed out the grad for each of the layers and it comes back as none, the grad of the hidden layer is none as well.
In the code the only thing I changed in the reinforcement file was rl_loss -= (log_probs[0, top_i].cpu().detach().numpy()*discounted_reward) and then tried rl_loss -= (log_probs[0, top_i].item()*discounted_reward) due to an issue with cuda device = 0.
Would you know any reason why grad would be coming up as none for all of the layers? I believe this is why optimizer.step() is not working and the weights are not updating.

Hi @vmarar
I'd say this is expected behavior. Further in the code rl_loss is backpropagated by applying rl_loss.backward(). But your changes detach rl_loss from the computational graph and there is nothing to backpropagate. Is there any specific reason why you need to do this? What kind of issues/errors do you have with your cuda device?

Hi @vmarar
I'd say this is expected behavior. Further in the code rl_loss is backpropagated by applying rl_loss.backward(). But your changes detach rl_loss from the computational graph and there is nothing to backpropagate. Is there any specific reason why you need to do this? What kind of issues/errors do you have with your cuda device?

would this be the case using rl_loss -= (log_probs[0, top_i].item()*discounted_reward) as well? Would there still be a detachment?
The error is : can't convert cuda:0 device type tensor to numpy. Use Tensor.cpu() to copy the tensor to host memory first
I have currently have cuda visible devices set to 0. Is there a way to bypass this error?
Thank you!

would this be the case using rl_loss -= (log_probs[0, top_i].item()*discounted_reward) as well? Would there still be a detachment?
The error is : can't convert cuda:0 device type tensor to numpy. Use Tensor.cpu() to copy the tensor to host memory first
I have currently have cuda visible devices set to 0. Is there a way to bypass this error?
Thank you!

would this be the case using rl_loss -= (log_probs[0, top_i].item()*discounted_reward) as well? Would there still be a detachment?

Yes, this is exactly what .item() does. It returns just the value of a tensor.

The error is : can't convert cuda:0 device type tensor to numpy. Use Tensor.cpu() to copy the tensor to host memory first

Which line triggers this error? Could you please provide a traceback?

would this be the case using rl_loss -= (log_probs[0, top_i].item()*discounted_reward) as well? Would there still be a detachment?

Yes, this is exactly what .item() does. It returns just the value of a tensor.

The error is : can't convert cuda:0 device type tensor to numpy. Use Tensor.cpu() to copy the tensor to host memory first

Which line triggers this error? Could you please provide a traceback?

This is the traceback, it is directly from the rl_loss line.

This is the traceback, it is directly from the rl_loss line.

I tried converting from numpy to tensor and then pointing it towards cuda:0, no error but model weights are still not changing

I tried converting from numpy to tensor and then pointing it towards cuda:0, no error but model weights are still not changing

I tried converting from numpy to tensor and then pointing it towards cuda:0, no error but model weights are still not changing

Yes, that wouldn't help, because you are first detaching the loss and then creating a new node that is not connected to the existing computational graph. In this scenario, the loss is not backpropagated and that's why weights are not changing.
Let me investigate the issue. Could you please provide your environment details? The versions of PyTorch, NVIDIA driver, and what GPU you are using.

I tried converting from numpy to tensor and then pointing it towards cuda:0, no error but model weights are still not changing

Yes, that wouldn't help, because you are first detaching the loss and then creating a new node that is not connected to the existing computational graph. In this scenario, the loss is not backpropagated and that's why weights are not changing.
Let me investigate the issue. Could you please provide your environment details? The versions of PyTorch, NVIDIA driver, and what GPU you are using.

Here are the env details:
GPU : 'GeForce RTX 2080'
Version of pytorch : 1.6.0
NVIDIA-SMI 450.57       Driver Version: 450.57       CUDA Version: 11.0
Details:
I have a scoring script that I am using in lieu of a predictive model, it outputs a numpy array. Currently I am using output_final = torch.tensor(output, device='cuda:0') to convert to a tensor, which got rid of rl_loss error. Please let me know if there's any other details I can provide.
Thanks for your help!

Here are the env details:
GPU : 'GeForce RTX 2080'
Version of pytorch : 1.6.0
NVIDIA-SMI 450.57       Driver Version: 450.57       CUDA Version: 11.0
Details:
I have a scoring script that I am using in lieu of a predictive model, it outputs a numpy array. Currently I am using output_final = torch.tensor(output, device='cuda:0') to convert to a tensor, which got rid of rl_loss error. Please let me know if there's any other details I can provide.
Thanks for your help!

After trying some solutions, I found that after converting NumPy output to tensor output within my scoring script. AND addressing a warning of using a source tensor to create a copy in the rl_loss = torch.tensor(rl_loss, require_grad=True) line, I simply commented out this line, and weights seem to backpropagate.
Is there any reason it could be wrong to not create a tensor from the existing rl_loss tensor and backpropagate the grad from rl_loss = rl_loss/n_batch instead? It's working but I want to make sure that I am not backpropagating useless values.
Is it possible that by using a script in lieu of a predictive model , autograd only sees the output of the script which is in the form of a tensor? Therefore not leading to any breaks in the computational graph because it's only dealing with a script output?

After trying some solutions, I found that after converting NumPy output to tensor output within my scoring script. AND addressing a warning of using a source tensor to create a copy in the rl_loss = torch.tensor(rl_loss, require_grad=True) line, I simply commented out this line, and weights seem to backpropagate.
Is there any reason it could be wrong to not create a tensor from the existing rl_loss tensor and backpropagate the grad from rl_loss = rl_loss/n_batch instead? It's working but I want to make sure that I am not backpropagating useless values.
Is it possible that by using a script in lieu of a predictive model , autograd only sees the output of the script which is in the form of a tensor? Therefore not leading to any breaks in the computational graph because it's only dealing with a script output?

After trying some solutions, I found that after converting NumPy output to tensor output within my scoring script. AND addressing a warning of using a source tensor to create a copy in the rl_loss = torch.tensor(rl_loss, require_grad=True) line, I simply commented out this line, and weights seem to backpropagate.
Is there any reason it could be wrong to comment out that line and not create a tensor from the existing rl_loss tensor and backpropagate the grad from rl_loss = rl_loss/n_batch instead? It's working but I want to make sure that I am not backpropagating useless values.
Is it possible that by using a script in lieu of a predictive model , autograd only sees the output of the script which is in the form of a tensor? Therefore not leading to any breaks in the computational graph because it's only dealing with the output?

After trying some solutions, I found that after converting NumPy output to tensor output within my scoring script. AND addressing a warning of using a source tensor to create a copy in the rl_loss = torch.tensor(rl_loss, require_grad=True) line, I simply commented out this line, and weights seem to backpropagate.
Is there any reason it could be wrong to comment out that line and not create a tensor from the existing rl_loss tensor and backpropagate the grad from rl_loss = rl_loss/n_batch instead? It's working but I want to make sure that I am not backpropagating useless values.
Is it possible that by using a script in lieu of a predictive model , autograd only sees the output of the script which is in the form of a tensor? Therefore not leading to any breaks in the computational graph because it's only dealing with the output?


Fixing monaco snippets for fixed instances, enums, and block definitions

Fixes https://github.com/Microsoft/pxt-microbit/issues/925
This PR fixes three issues:

Enum arguments were always being replaced as 0 or null (now we grab the first declared member)

Fixed instances of types with qualified names exported from a namespace (e.g. input.Button rather than top-level Button) were generating weird snippets with the namespace showing up twice

The Monaco toolbox was ignoring default values for variable blocks. This was affecting a few of the changes I recently made in pxt-microbit for the image and game blocks

BTW I tested every snippet in the micro:bit toolbox and this should fix them all. We still have an issue where object types just get null for placeholder values but that's a separate feature.


Uploading from a URL

I'd like to suggest implementing uploads from a URL, to facilitate the rehosting of dumb memes among other things. Goploader is very neat, thanks!

Hey, thanks!
I've stopped working on goploader for a while now, but that could be a fun challenge. There would be a need to ensure goploader doesn't download the whole file if it's too big, otherwise it would be really easy to fill up disk space using massive files. I don't know if I'll implement that, but if someone makes a pull request regarding that issue I'll be sure to have a look at it and merge it ^^


Not running

Mcedit was running untill today. It opens a cmd window and close what i can do?

Can you post a screenshot of the command prompt?

it automatly close

i can not see it before it close

i taka a screenshot

now it puts me another text

It seems like one of your resource packs is broken..
For now, you can just remove your resource packs to use MCEdit but we'll try to fix it for next release..
If you'll find the resource pack that cause this crash, please send it to us

ok thanks

i find the resource pack that cause the problem it is the village sound pack from a map which name is The Abandoned Village V1.0

I think I solved this specific bug (Not sure) but for next release, the program would ignore broken packs and won't crash if it's broken


Enhance x-fhir-query to handle fhirpath expression

Is your feature request related to a problem? Please describe.
Enhance x-fhir-query to handle fhirpath expression in query as defined in spec https://build.fhir.org/ig/HL7/sdc/expressions.html#x-fhir-query-enhancements

@maimoonak and @f-odhiambo can you share some use cases and give us an indication of the urgency/priority of this issue?

@maimoonak If this is an important issue (see Jing's comment), is this one you are interested/able to pick up?
cc: @f-odhiambo @pld

As of current, we are not able to access the context that the x-fhir-query is in. In our use case, we need to be able to access the patient id in the query to only get tasks that the patient owns. For example
Task?code=http://snomed.info/sct|12345678&subject={{Patient.id}}
cc: @KhumboLihonga @Gental-Giant

@f-odhiambo We have the above use case

@jingtang10 thanks again for taking a look at this. One use case for this is where we want to get a list of active patients with the same organization id as the patient being observed. As per the documentation, I believe the questionnaire would look like this.

Hi @jingtang10 - another use case is for a scenario where you would want to use the information collected by a previous question in the questionnaire. For example we want to be able to select catchment areas based on selected district in the select district question in the questionnaire. When you select a certain district you would populate the drop down list of catchments areas that are part of that district. Below is the expression:

@KhumboLihonga  can you see if you can use enhanced x-fhir-queries now? The PRs merged should allow you to. When setting up the questionnaire fragment, use the setQuestionnaireLaunchContext builder function to pass in the resource you want to evaluate your expressions against

@sevenreup could you please take a look and confirm

@omarismail94 Tested the PR, and it works as intended.
The only thing that I think is missing is the ability to add more than one Resource to the launch context. So currently you can only add one Resource to be accessed in the enhanced x-fhir-queries but we have some use cases that will need access to multiple resources, for example, accessing both the Patient and the Location resource.

Okay, thanks @omarismail94 that will be appreciated

@sevenreup can you work off the changes I made in PR#2000 to see if that resolves your issue? When setting up the fragment, I changed the API to be setQuestionnaireLaunchContexts (pluralised). It expects an encoded list of all the resources you want to pass

@omarismail94 I managed to test out the PR, It works as intended.
Used a Patient and Encounter as launch contexts and managed to access them in the x-fhir-query.

@sevenreup  PR is merged and will be part of the next release of the SDC library. For now, you can fork/cherrypick


Can't login due to LOGIN_ERR_INVALID_TOKEN

Hi,
i tried the sample code can't connect with errorCode 5 which is LOGIN_ERR_INVALID_TOKEN
I/flutter ( 1734): Connection state changed: 2, reason: 1
I/flutter ( 1734): Login error: login failed errorCode:5
I/flutter ( 1734): Connection state changed: 1, reason: 3
So the token with null value won't work? in what condition null token will work? (i tried both on Live and Testing project). Or do i miss configuration at agora console?
Thank you

Hi,
Thanks a lot for your respond.
So you mean the token shouldn't be null? I wonder why it's nullable, is there any other usage with null?
Please let me know. Thanks


Develop

release 0.0

added basic folder structure
removed redundant messaging from read me

:+1:

:100:


Allow for one or more simultaneous touches for dragging and possibly rotating.

Is your feature request related to a problem? Please describe.
Apps and features that require 2 inputs are currently unusable. The simplest example: When editing in Google Photos and using any tool that draws like Magic Eraser requires 2 inputs to drag at the same time in order to move the zoomed view of the image.
Describe the solution you'd like
Hold a modifier key (Preferably LeftAlt) and drag just like the Pinch Zooming feature, This would create evenly spaced inputs that would follow the movements of the "Real" touch.
Using LALT+ is a possible solution for differentiating between needing a 2-finger drag and > 2-finger drag.
Implementing the 2-touch "rotate" that is featured on things like Google Maps could possibly be implemented using LCTRL+LALT which would place a virtual touch in the centre like Pinch zooming and orbiting your "Real" Touch around that centre point would simulate the gesture.
Maybe using LCTRL+LALT+Scroll would completely simulate the gesture at the centre point without requiring a manual touch at all?
Additional context
I'd happily try to take a crack at this myself but right now I'm just not experienced enough... Surly if The pinch zoom is possible, then using more than 1 input to drag will be childs-play to those who know their way around the code.

We'd be better off with trackpad support instead of trying to create more single input jank IMHO


https://community.chocolatey.org/packages/dbgl/ site timeout

Dear sir, madam,
It seems that my DBGL package's chocolatey page at https://community.chocolatey.org/packages/dbgl is unreachable, I'm getting " Error 524 Ray ID: 6be02b2adab07a42 • 2021-12-15 13:55:16 UTC A timeout occurred". Is something amiss?
Regards,
Ronald Blankendaal

Never mind, the problem seems to have resolved itself.

resolved


Adding support for native building on Windows using MSYS.

Use of "cygwin" build target in meson
If user uses meson package provided with python from MSYS repo (not
MINGW64) for configuration Python's sys.platform is set to "cygwin".
Meson declares to support Cygwin as a target and makes project
configuration frictionless under MSYS environment. This unfortunetly
means we have to check for "cygwin" target during meson configuration
when we check for "windows" targets, but makes configuration much
easier.

What does host_machine.system() say?
This looks like you're trying to create a Cygwin executable, which we don't really support. Using Cygwin to "cross-compile" to MinGW should be supported, though.

Sounds like a meson bug. Could you report to https://github.com/mesonbuild/meson/issues, saying that the MSYS/Cygwin-based tool reports that the target/host system to be "cygwin" when it's producing mingw ("windows") binaries?

Purely FYI: https://mesonbuild.com/Reference-tables.html#operating-system-names

Sounds like a meson bug. Could you report to https://github.com/mesonbuild/meson/issues, saying that the MSYS/Cygwin-based tool reports that the target/host system to be "cygwin" when it's producing mingw ("windows") binaries?

Not sure if this is a bug. I think they just use whatever sys.platform holds. Cygwin is a special case and meson is configured differently when cygwin is detected - that is my understanding of documentation.

I am going to do some minor refactor and additional testing. Will fix the issues this week and update PR.


.Net Core 2 OR .Net Standard 2?

Hello @scheshan
Last night the new version was released. So we should decide for next version.

Hi @OmidID
I've seen this great news. I will upgrade the project to use .NET Core 2.0 before this weekend. Then I will spend some days to add some front pages, such as "archives" and something else.

@scheshan Nice ;)


Add ability to FlushAll from cache

Add the ability to remove all from cache. Similar functionality to Redis FlushAll

What's the use case for this? I wouldn't expect most web services to expose some way for a consumer of an API to clear out an entire cache.

I believe the functionality I am really looking for is to remove all from a set. Updated ticket issue. I will take a look and see what needs to be done to add this.


Feat/trusted

Major new update.

Tests have passed for all the new services. Even the simple service now works. Documentation for these will be included in a separate PR later.


Added the Cologne Hackathon repository and Zenodo publication

Hi all,
I would like to add the NFDI4Bioimage - TA3-Hackathon - UoC-2023 (Cologne Hackathon) repository and the Zenodo publication of this repository. Topics covered OMERO-ARC interoperability, Neuroglancer and REMBI/Mapping.
(Also corrected one minor typo.)
Thanks!
Andrea

Fantastic, big thanks @andreaschrader !


Difference between power and power_calculated

What is the difference between the "power" and "power_calculated"  attributes? As much as I analyze it, I do not understand where the power value comes from, many times it is very different from the calculated power. Thanks

power value comes directly from Waveshare UPS HAT
and power_calculated is calculated based on voltage and current.
I've got this code directly from official SDK


Add annotations to databricks and trino connectors

Description

Checklist
- [ ] For UI or styling changes, I have added a screenshot or gif showing before & after

[x] I have added a changeset

Tested this out in example project getting the following error.
I suspect that it's the databricks token, which I have updated


[notebook] Support Paperspace as a potential ML Notebook platform

Due to issues with Kaggle (#348), it would also be wise to explore other providers such as Paperspace for training these models.

@RobotSail is this still relevant or can it be closed?

@nathan-weinberg Closing


update docker postgres to 12.5

Pre-Flight checklist

[x] Testing

[x] Changes have been manually tested

What are the relevant tickets? https://github.com/mitodl/ocw-studio/issues/455
What's this PR do?
Update local postgres to version 12.5 in db container only. Does not update concourse-db postgres.
How should this be manually tested?
Steps below assume OCW Studio docker containers are already running. Destroying the pg-11 database seemed like easiest upgrade path, so to keep your local data create a backup and restore from backup after update:
# Alert! Do not just dump these all into a shell script
#
# Putting all 8 steps into a shell script will not work b/c
#  - then 6 runs as soon as the container from 5 starts
#  - whereas 6 needs to run when db server has actually started
#
# Steps 1-5 could be run as a script, but do 6 and 7 manually.

# Assumes master-branch containers are already running

# 1. backup the old database for restoration after upgrade
docker-compose exec -T db pg_dumpall -h db -U postgres > dump.sql
# 2. stop all containers
docker-compose stop
# checkout this branch
git checkout update-pg
# 4. destroy the old database container
docker-compose rm -sv db
# 5. build+start the new database container
docker-compose up db -d
# 6. restore the database from backup
docker-compose exec -T -e PGPASSWORD=postgres db psql -h db -U postgres < dump.sql
# 7. start the remaining containers
docker-compose up
# 8 verify postgres 12 is in use:
docker-compose exec -e PGPASSWORD=postgres db psql -h db -U postgres

Try using ocw-studio and observe that it works as expected.
Notes

Again, this does not update concourse-db postgres
The original issue (https://github.com/mitodl/ocw-studio/issues/455) described 4 steps to upgrade. As far as I can tell, only steps 1 and 2 are currently unnecessary
I am fairly new to docker, so hopefully the above is a reasonable way to do this...

 Chris Chudzicki

So I tried this on my local database which had a full compliment of imported legacy courses.  The database import failed with a number of errors, too many to post in Github comments but the most common seemed to be invalid command \N.  It seems as if the imported content of the legacy courses is causing issues with the way the data is dumped / restored.  Stringified line breaks seem to be causing an issue and I'm not sure why or how we would resolve it.

@mitodl/devops do we need to upgrade the postgres attached to ocw-studio in production too? Is this the way we would do it?

@pdpinch The RC and prod instances of ocw-studio use RDS databases.  This PR simply upgrades the version of Postgres that the local Docker based setup uses and doesn't affect any of our Heroku setups.


Fix ARG scoping for Dockerfiles with multiple FROM

depends on #31352 (could rewrite it without it but there seems to be a consensus that #31352 will be merged)
fixes #31892
@dnephin

left some thoughts on the issue https://github.com/docker/docker/issues/31892#issuecomment-287475769

@thaJeztah Do you think I should rebase it without #31352 if you want to discuss that one more

@tonistiigi hm, perhaps it's best to keep the discussion separate (but let me know if it's troublesome to do 😅)

superseded by #31257


letters count as words

plugin doesn't work well with Arabic and Persian text; it counts letters as words.

Can you take a screenshot of your settings page?Regards,ben On 22 Jan 2023, at 10:53 PM, PSEUDO-SAPPHO @.***> wrote:
plugin doesn't work well with Arabic and Persian text; it counts letters as words.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you are subscribed to this thread.Message ID: @.***>

they are two words in " کلمه نخسین".

Try turning on this option in the plugin settings.


Why is Safari restricted from using the service?

It appears that Safari is functionally compatible with the website once the Safari-specific restrictions are removed. Is there a particular reason why the browser is prevented from using the service?
Safari 14.0.3

There seems to be a bug in the Webkit implementation of websockets preventing the message scaling algorithm to work properly, which results in upload measurements failing on some combinations of machine speed + connection speed. Further details: https://github.com/m-lab/ndt7-js/issues/19#issuecomment-808763993
There is also a different issue in the download measurement, so that it never reports more than ~500 Mb/s in our testing -- only on Safari.
While it would appear that the measurement is running just fine on Safari (and in most cases, i.e. < 500Mb/s, this is probably true), some percentage of measurements done on Safari will either fail while showing unreasonably high upload rate to the user (first bug), or under-report the download speed (second bug).
We decided to disable it until either we have a good workaround for these two issues, or they are fixed in Safari.


Blank pages after changing theme (DOMDocument not found in...functions:30)

Hi,
Fresh install on a test system and when I change the theme, the page goes blank (all white) and I can't do anything. The nginx logs then show errors, such as this:
thrown in /var/www/html/themes/parallax/functions.php on line 30" while reading response header from upstream, client: <IP_ADDRESS>, server: vdebby, request: "GET / HTTP/1.1", upstream: "fastcgi://unix:/run/php/php7.4-fpm.sock:", host: "vdebby"
2023/01/21 10:46:26 [error] 39457#39457: *19 FastCGI sent in stderr: "PHP message: PHP Fatal error:  Uncaught Error: Class 'DOMDocument' not found in /var/www/html/themes/parallax/functions.php:30
Stack trace:
#0 /var/www/html/index.php(1150): alterAdmin()
#1 /var/www/html/index.php(2337): Wcms->hook()
#2 /var/www/html/themes/parallax/theme.php(59): Wcms->settings()
#3 /var/www/html/index.php(1628): require_once('/var/www/html/t...')
#4 /var/www/html/index.php(183): Wcms->loadThemeAndFunctions()
#5 /var/www/html/index.php(16): Wcms->render()
#6 {main}
thrown in /var/www/html/themes/parallax/functions.php on line 30" while reading response header from upstream, client: <IP_ADDRESS>, server: vdebby, request: "GET /favicon.ico HTTP/1.1", upstream: "fastcgi://unix:/run/php/php7.4-fpm.sock:", host: "vdebby", referrer: "http://vdebby/"

Hello @Entheologian.
Looks like this theme requires the DOM extension for PHP. Depending on the system you use, this should be fairly easy to do:

https://www.php.net/manual/en/intro.dom.php
https://www.php.net/manual/en/class.domdocument.php

If you're stuck on blank pages and can't change the theme, you can open up the data/database.js folder and change the defaultTheme manually to another one so you can continue using WonderCMS without blank pages.

@Entheologian closing issue since this can be fixed my enabling the module mentioned above. If you need any help please feel free to continue commenting and we'll respond.
Closing issue for now.


What is the difference between this plugin and the cordova-plugin-push-notification?

Sorry for raising an issue while I actually have a question.
What is the difference between phonegap-plugin-push and cordova-plugin-push-notification ?
Previously our project was using cordova-plugin-push-notification, and I am currently to upgrading all plugins in our project to the latest version.. after upgrading it to the latest version, while searching the forums I accidentally found this plugin and wonder if we should move our project to this plugin instead. Is that plugin deprecated and replaced by this plugin? That plugin haven't been updating since one year ago.
I also tried to google this but couldn't find anything that answers my question.
Thanks!

@iori57 this plugin is the one that the PhoneGap team at Adobe officially supports. I can't really comment on cordova-plugin-push-notification as I don't know the maintainer. It looks like it was forked off of the now deprecated https://github.com/phonegap-build/PushPlugin.git repo. The main benefits of this plugin are a) it's being maintained and b) it has a consistent API.


[terra-layout] - Regenerated jest snapshots

Summary
The following pull request introduced a default z-index that caused the terra-layout snapshot tests to fail.
https://github.com/cerner/terra-core/pull/1955
Additional Details
Regenerated the jest snapshots to account for the z-index
Thanks for contributing to Terra.
@cerner/terra

Closing, the fix for this is already included in this PR: https://github.com/cerner/terra-framework/pull/320

Let's get this in master


Arducam OV2640 motion detector

Hi
I'm trying to get a OV2640 on an Arducam ESP8266 working with a PIR motion detector to take a picture when it detects movement. I have the motion detector trigger wire connected to GPIO12 and running the code below.
I use
attachInterrupt(12, intruder_detect, CHANGE);  in setup()
on pin 12 to trigger an intruder detection event which then captures a photo.
I get in my serial monitor (after pressing reset a couple of times). No pictures are ever captured
Code is a modified version of ArduCAM_ESP8266_V2_OV2640_Capture2SD along with interrupt for motion detector. Any ideas why it's not working?
2dN,4lMMØlb8óÿArduCAM Start! 85 SD Card detected! OV2640 detected. 2dO,4lMMØlb8óÿArduCAM Start!
Intruder detected
star Capture
¶Eô|vAM¬tè4C(ArduCAM Start!
85
SD Card detected!
OV2640 detected.
$lÍ+`,lløp$:òûArduCAM Start!
Intruder detected
star Capture
¿„FJ
ô@HàþArduCAM Start!
85
SD Card Error
OV2640 detected.
#include <ArduCAM.h>
#include <Wire.h>
#include <SPI.h>
#include <SD.h>
#include "memorysaver.h"
#if !(defined ESP8266 )
#error Please select the ArduCAM ESP8266 UNO board in the Tools/Board
#endif
//This demo can only work on OV2640_MINI_2MP or ARDUCAM_SHIELD_V2 platform.
#if !(defined (OV2640_MINI_2MP)||(defined (ARDUCAM_SHIELD_V2) && defined (OV2640_CAM)))
#error Please select the hardware platform and camera module in the ../libraries/ArduCAM/memorysaver.h file
#endif
// set GPIO16 as the slave select :
const int CS = 16;
//Version 2,set GPIO0 as the slave select :
const int SD_CS = 0;
ArduCAM myCAM(OV2640, CS);
int speakerOut = 12;//Piezo buzzer's positive terminal is connected to digital pin 9
byte names[] = {'c', 'd', 'e', 'f', 'g', 'a', 'b', 'C'};
int tones[] = {1915, 1700, 1519, 1432, 1275, 1136, 1014, 956};
byte melody[] = "2d2a1f2c2d2a2d2c2f2d2a2c2d2a1f2c2d2a2a2g2p8p8p8p";
// count length: 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0 1 2 3 4 5 6 7 8 9 0
//                                10                  20                  30
int count = 0;
int count2 = 0;
int count3 = 0;
int MAX_COUNT = 24;
int statePin = LOW;
const int motionSensorPin = 12;
void siren();
volatile byte intruder;
void myCAMSaveToSDFile(){
char str[8];
byte buf[256];
static int i = 0;
static int k = 0;
static int n = 0;
uint8_t temp, temp_last;
File file;
//Flush the FIFO
myCAM.flush_fifo();
//Clear the capture done flag
myCAM.clear_fifo_flag();
//Start capture
myCAM.start_capture();
Serial.println("star Capture");
while(!myCAM.get_bit(ARDUCHIP_TRIG , CAP_DONE_MASK));
Serial.println("Capture Done!");
//Construct a file name
k = k + 1;
itoa(k, str, 10);
strcat(str, ".jpg");
//Open the new file
file = SD.open(str, O_WRITE | O_CREAT | O_TRUNC);
if(! file){
Serial.println("open file faild");
return;
}
i = 0;
myCAM.CS_LOW();
myCAM.set_fifo_burst();
#if !(defined (ARDUCAM_SHIELD_V2) && defined (OV2640_CAM))
SPI.transfer(0xFF);
#endif
//Read JPEG data from FIFO
while ( (temp !=0xD9) | (temp_last !=0xFF)){
temp_last = temp;
temp = SPI.transfer(0x00);
//Write image data to buffer if not full
if( i < 256)
buf[i++] = temp;
else{
//Write 256 bytes image data to file
myCAM.CS_HIGH();
file.write(buf ,256);
i = 0;
buf[i++] = temp;
myCAM.CS_LOW();
myCAM.set_fifo_burst();
}
delay(0);
}
//Write the remain bytes in the buffer
if(i > 0){
myCAM.CS_HIGH();
file.write(buf,i);
}
//Close the file
file.close();
Serial.println("CAM Save Done!");
}
void setup()
{
attachInterrupt(12, intruder_detect, CHANGE);//Initialize the intterrupt pin for the motion sensor (Arduino digital pin 2)
uint8_t vid, pid;
uint8_t temp;
Wire.begin();
Serial.begin(9600);
Serial.println("ArduCAM Start!");
//Serial.begin(115200);
//set the CS as an output:
pinMode(CS,OUTPUT);
//initialize SPI:
SPI.begin();
SPI.setFrequency(4000000); //4MHZ
delay(1000);
//Check if the ArduCAM SPI bus is OK
myCAM.write_reg(ARDUCHIP_TEST1, 0x55);
temp = myCAM.read_reg(ARDUCHIP_TEST1);
Serial.println(temp);
if (temp != 0x55){
Serial.println("SPI1 interface Error!");
while(1);
}
//Initialize SD Card

if(!SD.begin(SD_CS)){
Serial.println("SD Card Error");
}
else
Serial.println("SD Card detected!");
//Check if the camera module type is OV2640
myCAM.wrSensorReg8_8(0xff, 0x01);
myCAM.rdSensorReg8_8(OV2640_CHIPID_HIGH, &vid);
myCAM.rdSensorReg8_8(OV2640_CHIPID_LOW, &pid);
if ((vid != 0x26 ) && (( !pid != 0x41 ) || ( pid != 0x42 )))
Serial.println("Can't find OV2640 module!");
else
Serial.println("OV2640 detected.");
myCAM.set_format(JPEG);
myCAM.InitCAM();
}
void loop()
{
}
void intruder_detect()//This function is called whenever an intruder is detected by the arduino
{
intruder++;
Serial.println("Intruder detected");
myCAMSaveToSDFile();
}

@rmcncal
Hi,
Your SD card is error,
The GPIO12 is the MISO signal.
Best regards,
ArduCAM support team.

Hi
The SD card is fine as it  captures photos when I run ArduCAM_ESP8266_V2_OV2640_Capture2SD
it's when I modify this sketch to use a PIR motion detector that it doesn't work, I have tried GPI02 & GPI016 pins without success, do you know which pin  I can use to process an output signal from a PIR motion detector? I use an interrupt to capture the output signal to take a photo.

@rmcncal
Hi,
If you use both SD card and camera module on our ArduCAM ESP8266 UNO board,
you will don't have other GPIO pin for PIR input.
Let us know if you need more help.
Regards,
ArduCAM support team.

Hello,
I am trying the same thing and have found 2 issues. One is the wifi ESP8266 which is not recognized when you type AT in the serial monitor at any baud rate. And 2. the PIR cannot be used if you want to use the SD card with this board.
So, do you recommend using the Arduino UNO R3 with a ESP8266-01 with the PIR and OV2640. If so, what is the pinout for this and what programs should be used?
Thanks


dependency check v5 - specify path for the DB with updateonly option

Hi
Is there any way specify the path where you want to store the database when you download the database with --updateonly?
thanks

Please consider reading https://jeremylong.github.io/DependencyCheck/dependency-check-cli/arguments.html
You are looking for --data [path].


OpenCL dynamic runtime support added

This pr adds a feature dynamic-runtime to load the OpenCL library in the runtime and avoid linkage errors during compilation stage (OpenCL.lib not found).
I have checked that there is no compilation warnings with both runtimes and with default and full features sets. I also have checked that opencl3 library can be compiled with my changes (there are only two warnings about unused Result).
Tests are passing using the command cargo test --all-features -- --test-threads=1.

@dmitry-zakablukov please can you change this PR to use the current develop branch?

@kenba , okay, but it will take some time.

@kenba , I have updated the pr, but I cannot check compilation with the current version of opencl-sys.
Can you please apply this patch to opencl-sys?
static.patch
In this patch all extern system functions are under the static feature.

@kenba , thanks. I have updated the pr, all tests and examples can be compiled without errors now.

Thanks @dmitry-zakablukov merged in now.


Need ISSUE_TEMPLATE

It'll be good to have an issue template for this project.

Also, I'll take this up and will raise a PR for the same.

Okay, thanks @utkarsh2102


PHP Fatal error:  Uncaught Exception: String could not be parsed as XML in /root/git/box-sfr-munin/script.php:137

Hello,
Thx for the share of this code. I try to use it without munin first by setting manually ip & password
function graph($graph) {
$ip = "myip";
$password = "mypassword";
But when I run the script.php I am facing this issue:
PHP Fatal error:  Uncaught Exception: String could not be parsed as XML in /data/box-sfr-munin/script.php:159
Stack trace:
#0 /data/box-sfr-munin/script.php(159): SimpleXMLElement->__construct()
#1 /data/box-sfr-munin/script.php(279): graph()
#2 {main}
thrown in /data/box-sfr-munin/script.php on line 159
Can you help me please ?
Teddy

You should print curl's output to check what the box is sending you.
Turn $ret = new SimpleXMLElement(curl_exec($ch)); into something like
$ret = curl_exec($ch);
echo $ret;
$ret = new $ret = SimpleXMLElement($ret);```


Don't alter require.paths.

Feature was removed in node 0.5.x.
This should get things working in the up and coming versions.

node v0.6.0 is released now. Can this be merged please and an npm version bump? It would be appreciated!

+1

+1

+3

@gf3 Might need to @mention you before you get these notifications...

+4

Sorry guys, I've been crazy busy lately, I'll get on these.

0.2.27 published


Prefer using react api for useProperty hook

Hey, @raveclassic!
Intro
I'm developing a chat. It means that I need to subscribe to properties in leaves (messages, etc.) so only necessary parts re-render.
I was testing performance and noticed that old version of useProperty lost updates (new works fine). This led me to some research. Results are below.
1 React-way to subscribe to mutable sources
Look like there are three versions of "how to subscribe to external mutable source" in React:

createSubscription + useSubscription
useMutableSource (was renamed to the below)
useSyncExternalStore (package)

It turns out that managing frp-ts like source is not that easy in React because you have to take into account SSR, Suspense and async rendering (coming in React 18).
My suggestion is to use useSyncExternalStore in frp-ts/react package so the React team handles all the details (I'll attach reading list at the end for a better context).
2 State modification during render
I know that this is an anti-pattern and it should not be done (react repo issue). But, now I wonder whether this still can happen in large codebases (sockets, event handlers, whatever).
React team introduces StrictEffects and this should catch the above. Until that lands, I wonder whether deferring state.set calls as done in most eliminates the problem completely.
I still need to wrap my head around this. Maybe this case will never happen in a real application. Maybe the only way to trigger this is to directly modify the state in a render function.
Two demos showcasing this:

frp-ts
MobX

3 Further read

Show
Packages:

use-subscription
create-subscription
use-sync-external-store

API Discussions:

use-mutable-source
use-sync-external-store
how to support strict effects

react-redux migration to useSES:

v8 release-notes
prep for React18
useMutableSource

mobx issues:

react requires immutability
supporting react concurrent

New version of the useProperty hook
I tested my component with the hook below and everything seems to work fine.
const bridge = <A,>(
  p: Property<A>,
): {
  subscribe: (onStoreChange: () => void) => () => void;
  getSnapshot: () => A;
} => ({
  getSnapshot: () => p.get(),
  subscribe: (onChange) => {
    const sub = p.subscribe({
      next: () => onChange(),
    });
    return () => sub.unsubscribe();
  },
});

const useProperty = <A,>(p: Property<A>): A => {
  const { getSnapshot, subscribe } = useMemo(() => bridge(p), [p]);
  return useSyncExternalStore(subscribe, getSnapshot);
};

Summary

Looks like useSyncExternalStore is the way to go for frp-ts React integration.
There is a potential issue with mid-render updates, maybe there is a smart way to prevent\fix that (not necessarily in frp-ts).
Maybe there is some works needed to prepare for React 18.

Hey @PalmZE, thanks for such a detailed issue!
So, correct me if I'm wrong, the issue is that during rendering of list items that reference list values in Properties by index, we can get out of sync with the data because this list item component is not unmount in time, right? This happens due to asynchronous nature of the renderer, right? Then I would expect such components reading the data by index to check whether the item is not undefined. That would be the correct behavior to work with changing lists and "stale" indexes. By "stale" I mean that they might get out of sync due to async rendering in React. That's a common problem actually and it's not related to setting the state in the render cycle. The wrong index, captured during rendering of the list, might reference non-existing item anyway, as the rendering most of the times is out of sync with the data (especially taking React's Concurrent Mode into account). So, in short, I would say it's the responsibility of the UI composition to write to/read from Atom/Property correctly.
Originally, @frp-ts/core was design to be fully synchronous at its heart, so although most and other libs implement asynchronous value delivery (microtasks for example), I'm not sure whether it's a good option to adopt similar concept here. At the end of the day, it's not the responsibility of Atom to think about how/when the value should be delivered to its consumers, but of the caller of Atom#set. In most cases, synchronous emission is enough, in some cases (like with React) it may break things but we should think about such cases outside of the core. For example, it's fine to tackle this in @frp-ts/react.
Ok, now speaking about @frp-ts/react, honestly I don't know wtf is use-subscription/``use-mutable-source` or any other fancy APIs they are going to bake into the core :D. But, seems until it's not in the core, we are free to ignore it. At the same time, I think it's fine to think about improving existing bindings to React if there are any issues with them.
Thanks for opening the issue and let's continue the discussion here.

So, in short, I would say it's the responsibility of the UI composition to write to/read from Atom/Property correctly.

I thought a bit more about the issue, and I agree.

At the same time, I think it's fine to think about improving existing bindings to React if there are any issues with them.

Existing bindings are working fine. This is an approach similar to other libraries (mobx, effector, etc).
I created this issue to discuss the possibility of using useSyncExternalStore since it simplifies the usage of atom-like sources. But, I agree that until this api is in the core (and React 18 is released) current version is safer.
I'll test useSyncExternalStore in my own project. Can let you know, if it works ok in the long run.

Conclusion: I have nothing to add, feel free to close this issue.

Thanks for the feedback @PalmZE. Closing.


Update GMSA credspec specification to refer to pod spec fields instead of annotations

Update GMSA credspec specification to refer to pod spec fields instead of annotations

/assign @VineethReddy02 @wk8

/assign @jaredbhatti

/sig windows

@kubernetes/sig-windows-bugs 👋 Thanks in advance for your review!

@wk8 did you intend to /approve or /lgtm as well?

/milestone 1.16

Indeed, thanks @sftim ! /lgtm

@wk8 👋 Thanks for the review! For future reference, the /lgtm only works if it's on a new line.

 Trying again :)
/lgtm

@ddebroy Please take a look at the review so that we can get this in soon.

I think this is ready from the tech review perspective. Need approval from owner: @jaredbhatti

/approve


mmCIF <-> PDB utility

It would be nice to have a simple way of converting a mmCIF file into the regular PDB file (and ideally the other way around too - but this might be more complicated)

This is something I've given a lot of thought to. My biggest concern is the only structures that do not have a PDB file available are those with too many residues or atoms or chains. If we had an internal representation for the data, sure, no biggie in parsing, manipulating everything, and then writing it in the right format. But we don't, sort of on purpose. Then there's the concern that the mmCIF format packs much more information, and that this information cross-references other parts of the file. So either we'd stick to dealing with coordinates and strip the mmCIF of most of the information, which is a bit of a waste, or we'd have to write a proper mmCIF parser.
What I can do is to have a one-way converter from mmCIF to PDB, with the caveat that it will fail if the structure is too large. This allows files in cif format to be manipulated as long as they are small enough and since we are losing information anyway with the PDB format. What do you think?

Probably a mmCIF to PDB utility would be nice. Not sure how long will the PDB keep providing the old PDB format... Can of course be done with other software like PyMol

We have such a converter in HADDOCK2.4 web server, this actually based on the FastMMCIFParser provided by biopython. Which also "supports" ihm_mmcif in the sense that it does not complain when he does not know some categories.
But this does not align well with the idea of not relying on external libraries... And I'm not sure about the behavior when there are too many atoms. Something to test.

My suggestion is to provide a simple converter that picks the _atom_site
entries of an mmCIF file and dumps ATOM/HETATM lines. If the file has over
9999 residues in a single chain or more than 99999 atoms, then it throws an
exception and quits. We already do some of these checks when renumbering
atoms/residues.
About FastMMCIFParser: I wrote that one, it basically parses the _atom_site
entries of the mmCIF file into a bunch of dictionaries, that's why it
probably never complains :)

Agree with the principle, just fetch the _atom_site and do some basic checks ;)
Was not completely sure it was you, good job!

Addressed by 35674d3. Can you check it produces decent things? It passed some of my tests, including going from a PDB to mmCIF and back to PDB.

Added tools, manually verified with PyMOL and OpenMM mmCIF files. Seems to work ... Added test suite as well.


add zh /docs/reference/glossary/static-pod.md

to address https://github.com/k8smeetup/website-tasks/issues/3434

/lgtm
/approve


SKBitmap.Decode (SkiaSharp 1.57.0) fails for some type of images

I'm using Montemagnos CrossMedia plugin to pick a photo from my camera roll.
Then I create a byte[] of the photo and use that byte[] to breate a SKBitmap.
var mf = await CrossMedia.Current.PickPhotoAsync();
byte[] bytes;
var s = new MemoryStream()
mf.GetStream().CopyTo(s);
mf.Dispose();
bytes = s.ToArray();
var bm = SKBitmap.Decode(bytes);
This worked perfect in version 1.56.x of SkiaSharp. (On both Android, UWP and iOS).
After updating to 1.57.0 this still works for iOS and UWP.
However, for some images, on Android, it fails with the dump shown below
This seems to be VERY dependent of witch photo I use. I took a photo with my Samsung S7. Worked fine!
Made the photo a bit brighter (in the phone) and saved a copy. The copy DOES NOT work.
Both photos work if I downgrade SkiaSharp to 1.56.x

Stacktrace:
at  <0xffffffff>
04-26 08:30:18.756 E/mono-rt (28205):   at (wrapper managed-to-native) SkiaSharp.SkiaApi.sk_codec_get_pixels (intptr,SkiaSharp.SKImageInfo&,intptr,intptr,SkiaSharp.SKCodecOptionsInternal&,intptr,int&) <0x00007>
04-26 08:30:18.756 E/mono-rt (28205):   at SkiaSharp.SKCodec.GetPixels (SkiaSharp.SKImageInfo,intptr,int,SkiaSharp.SKCodecOptions,intptr,int&) [0x00097] in :0
04-26 08:30:18.757 E/mono-rt (28205):   at SkiaSharp.SKCodec.GetPixels (SkiaSharp.SKImageInfo,intptr,int,SkiaSharp.SKCodecOptions,SkiaSharp.SKColorTable,int&) [0x00020] in :0
04-26 08:30:18.757 E/mono-rt (28205):   at SkiaSharp.SKCodec.GetPixels (SkiaSharp.SKImageInfo,intptr,SkiaSharp.SKColorTable,int&) [0x00012] in :0
04-26 08:30:18.757 E/mono-rt (28205):   at SkiaSharp.SKBitmap.Decode (SkiaSharp.SKCodec,SkiaSharp.SKImageInfo) [0x0003d] in :0
04-26 08:30:18.757 E/mono-rt (28205):   at SkiaSharp.SKBitmap.Decode (SkiaSharp.SKCodec) [0x00018] in :0
04-26 08:30:18.757 E/mono-rt (28205):   at SkiaSharp.SKBitmap.Decode (SkiaSharp.SKStream) [0x00026] in :0
04-26 08:30:18.757 E/mono-rt (28205):   at SkiaSharp.SKBitmap.Decode (byte[]) [0x00017] in :0
04-26 08:30:18.757 E/mono-rt (28205):   at Sketching.Renderer.BackgroundImageRenderer.ResizeBitmap

Are you able to attach a photo that breaks?

This photo worked in its original version. The version i uploaded is slightly brightend and doesn't work. Same height/width... Both photos works well in1.56.2

I have a simple test app, but it works on both Android and iOS. Are you able to share a sample app that fails for you?
I am using a Nexus 5, so it may be that. What is your device?
test.zip

No time to share a test-app right now. Will try to fix one soon! I'm having the problem on a Samsung S7 device.

I am closing this issue as it is fairly old. Please re-open if this is still not working for you.


use jena-stable-turtle for trig output

the latest version of jena-stable-turtle "sorts" trig files, we should use it when we export resources as trig files (like we do for ttls)

Done as of commit a57bf3a

@eroux
Can we close this one ?


License

Hi,
Which license this repository uses?
Thanks,
Omri.

Hi, sorry for the late reply. This repo adopts MIT LICNESE.
We have just added MIT LICENSE to the repository.

Hi, sorry for the late reply. This repo adopts MIT LICNESE.
We have just added MIT LICENSE to the repository.

Great, Thanks :)


Update dotnet.yml

Pull Request Test Coverage Report for Build 858482366

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

Totals

Change from base Build 857741308:
0.0%

Covered Lines:
1064

Relevant Lines:
1064

💛  - Coveralls


Skip duplicate pooled MIDI sources when transposing or adjusting velocity

SWS/FNG: Increase/Decrease selected MIDI items velocity by x
SWS/FNG: Transpose selected MIDI items...
Fixes #1406

:white_check_mark: Build sws 335-master completed (commit https://github.com/reaper-oss/sws/commit/db51f298e3 by @cfillion)

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Ubuntu1804, ARCH=armv7l

build/sws-<IP_ADDRESS>-Linux-armv7l-fng-midi-unique-pools-db51f298.tar.xz

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Ubuntu1804, ARCH=x86_64

build/sws-<IP_ADDRESS>-Linux-x86_64-fng-midi-unique-pools-db51f298.tar.xz

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Visual Studio 2019, ARCH=x86

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x86.pdb
build/sws-<IP_ADDRESS>-Windows-x86-fng-midi-unique-pools-db51f298.exe

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Visual Studio 2019, ARCH=x64

build/BuildUtils/SWS_Template.ReaperLangPack
build/BuildUtils/whatsnew.html
build/reaper_sws-x64.pdb
build/sws-<IP_ADDRESS>-Windows-x64-fng-midi-unique-pools-db51f298.exe

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Ubuntu1804, ARCH=i686

build/sws-<IP_ADDRESS>-Linux-i686-fng-midi-unique-pools-db51f298.tar.xz

Environment: APPVEYOR_BUILD_WORKER_IMAGE=Ubuntu1804, ARCH=aarch64

build/sws-<IP_ADDRESS>-Linux-aarch64-fng-midi-unique-pools-db51f298.tar.xz


Refactor scenarios and pipeline implementation

In this pull request, I am addressing issue #280.
I have opened this PR to receive your feedback on whether this is the best approach to solve the issue, as it will require changes to the inductiva-web-api as well.
I have made the following modifications:

Created a Commands class that should be inherited by each simulator. For example, I implemented the GROMACSCommands class, where specific arguments of the GROMACS commands were implemented (method_name).
Implemented the simulator.run_pipeline() method, which takes a list of Commands and the working directory as inputs to run the simulation.
Changed the api.run() argument parameters to accept a list of arguments instead of a dictionary. This way, we can use the api.run to run a single command or a list of them.
Added a method called simulate_pipeline to the Base class scenario.
Refactored the  protein solvation scenario to  inherit from the new scenario base class.

To complete this change and make it functional, I need to make alterations to the BaseExecuter on the backend side.
I still need to test this solution, but I wanted to get your feedback first.
Thank you :)

It is getting too big, I will stop making changes and wait for your feedback :)

Thanks a lot @cristianacarpinteiro, we are getting there! :)

Just a general comment on this thread: it seems that this is taking too long to converge and is raising a very large number of issues and comments. Next time let's make sure we break changes in smaller incremental pieces that can be merged subsequently. Also, if think get complicated, it may make sense to organize a design meeting to discuss things at high level before start implementing.
In any case: this is a good piece of work (it's just not organized in a more reasonable way)

Following today's discussion on the API meeting, I've decided to close this PR and open a new one, where I will implement the scenario using a JSON file template to list the commands necessary to run the simulation. I've opened issue #286, where I mention the need to implement a pipeline where the user can specify the commands he wants to run on the API.


Fix YAML spec for libyaml 0.2.1

Fixes #6283

@straight-shoota Suggestions for a good fix? :-)
There's nothing in YAML that can generate "...\n" at the end of the document other than a document. I really don't think we should put a lot more effort into this.

Take a look at the affected examples. There are very similar examples, which sometimes expect ...\n, sometimes not.
I'd suggest to implement assert_yaml_document accepting an argument expect_document_end which appends ...\n to the expected string if yaml_get_version is <0.2.1.

It's probably more correct to change the expected string based on the YAML version, so that's another solution. Of course, if later in libyaml 0.2.2 they revert that change because it was a mistake, our code will have to change again.

Actually, it would probably even be better to normalize the results from YAML::Builder itself. We shouldn't expect different output depending on libyaml version. I expect we will have support these different libyaml versions for quite some time.


fix: exception bug for debug mode

#54

SonarQube Code Analytics
Quality Gate failed

Additional information
The following metrics might not affect the Quality Gate status but improving them will improve your project code quality.
Issues
  0 Bugs
  0 Vulnerabilities
  0 Code Smells
Coverage and Duplications
 0.00% Coverage
 0.00% Duplication

:tada: This PR is included in version 1.2.1 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


adding properties to relationships

fixes #64
I just want to check with you guy if generating the relation identifier in cypher by prefixing the node name with rel_ is the right way to go, or if you prefer to use some kind of name generator like this one.
I think it might overlap if you have a model named whatever which is related to another model, AND a third model named rel_whatever, all involved in the same query.
If that's fine with you, I'll keep going :

[x] merge relationships properties while read the related models
[ ] ability to filter on relationships properties it looks like this can already be done with model.query
[x] add tests
[x] update README.md

I'm reasonably sure that there will be no overlap, if I remember correctly each name property on the save graph is unique. It would be best to add a test that tests that particular situation in any case.

Hello, this is ready to be reviewed :)

:+1:

:+1:

This looks great. Thanks for your contribution, and sorry it took so long to get around to reviewing it.

published as v0.10.0

sorry, meant v0.7.0

@jonpacker thanks, i'll get around and see what i can do with other issues.


ios14 oddness with 5.0.0

Bug Report
Problem
When camera.getPicture is invoked, a long-ish delay (5 sec) occurs before successfully
presenting the camera view.  In the xcode console I see these messages:
020-09-20 10:42:22.828828-0700 myApp[1069:319363] [reports] Main Thread Checker: UI API called on a background thread: -[UINavigationController setDelegate:]
PID: 1069, TID: 319363, Thread name: (none), Queue name: com.apple.root.default-qos, QoS: 0
Backtrace:
4   myApp                        0x0000000100411804 -[CDVCamera showCameraPicker:withOptions:] + 184
5   myApp                        0x0000000100410e74 __25-[CDVCamera takePicture:]_block_invoke_2 + 324
6   TCC                                 0x00000001cccb840c 503BE2BA-F75C-3D90-83A6-3A941D77C334 + 13324
7   TCC                                 0x00000001cccbc95c 503BE2BA-F75C-3D90-83A6-3A941D77C334 + 31068
8   libxpc.dylib                        0x00000001d479d8ec 71E5302F-09BE-3116-8D43-EC2D4C27BE10 + 104684
9   libxpc.dylib                        0x00000001d4791598 71E5302F-09BE-3116-8D43-EC2D4C27BE10 + 54680
10  libdispatch.dylib                   0x000000010080d678 _dispatch_client_callout3 + 20
11  libdispatch.dylib                   0x000000010082ab80 _dispatch_mach_msg_async_reply_invoke + 396
12  libdispatch.dylib                   0x0000000100821270 _dispatch_kevent_worker_thread + 1416
13  libsystem_pthread.dylib             0x00000001d476c940 _pthread_wqthread + 332
14  libsystem_pthread.dylib             0x00000001d477377c start_wqthread + 8
2020-09-20 10:42:23.103711-0700 myApp[1069:319150] [Camera] Failed to read exposureBiasesByMode dictionary: Error Domain=NSCocoaErrorDomain Code=4864 "*** -[NSKeyedUnarchiver _initForReadingFromData:error:throwLegacyExceptions:]: data is NULL" UserInfo={NSDebugDescription=*** -[NSKeyedUnarchiver _initForReadingFromData:error:throwLegacyExceptions:]: data is NULL}
2020-09-20 10:42:23.670110-0700 myApp[1069:319150] [Presentation] Attempt to present <CDVCameraPicker: 0x10187d800> on <MainViewController: 0x100b0cf80> (from <MainViewController: 0x100b0cf80>) whose view is not in the window hierarchy.
2020-09-20 10:42:33.640598-0700 myApp[1069:319150] [Camera] Failed to read exposureBiasesByMode dictionary: Error Domain=NSCocoaErrorDomain Code=4864 "*** -[NSKeyedUnarchiver _initForReadingFromData:error:throwLegacyExceptions:]: data is NULL" UserInfo={NSDebugDescription=*** -[NSKeyedUnarchiver _initForReadingFromData:error:throwLegacyExceptions:]: data is NULL}

What is expected to happen?
Click a button to start the camera, within 1sec the camera appears.
In plugin version 4.1.1, there is no lag, and the message doesn't appear.
What does actually happen?
Click a button to start the camera, 5 seconds go by, then the camera appears.
Information

Here's my invocation:
navigator.camera.getPicture(onSuccess, onFail, getCamOpts());
here are the camera options i'm using:
{
    quality: 95,
    destinationType: Camera.DestinationType.FILE_URI,
    sourceType: fromlib ? Camera.PictureSourceType.PHOTOLIBRARY :
                                  Camera.PictureSourceType.CAMERA,
    encodingType: Camera.EncodingType.JPEG,
    mediaType: Camera.MediaType.PICTURE,
    correctOrientation: true,
};

The stack reported above is associated with the sourceType of CAMERA
When sourceType is PHOTOLIBRARY, i also get a long delay and several stack traces like this:
Main Thread Checker: UI API called on a background thread: -[UIImagePickerController init]
PID: 1104, TID: 326131, Thread name: (none), Queue name: com.apple.root.default-qos, QoS: 0
Backtrace:
4   myApp                        0x0000000100db7040 +[CDVCameraPicker createFromPictureOptions:] + 112
5   myApp                       0x0000000100db17c0 -[CDVCamera showCameraPicker:withOptions:] + 116
6   myApp                        0x0000000100db0cf4 __25-[CDVCamera takePicture:]_block_invoke + 1024

Command or Code

navigator.camera.getPicture(onSuccess, onFail, getCamOpts()); // (above)
Environment, Platform, Device

IOS14 from macos catalina, xcode 12.0.0
Version information

IOS14 iPhone XR
cordova 10.0.0
cordova-plugin-camera: 5.0.0
cordova-ios: 6.1.1
xcode: 12.0.0
Checklist

[ x] I searched for existing GitHub issues
[ x] I updated all Cordova tooling to most recent version
[ x] I included all the necessary information above

I downgraded to 4.1.0 and appear to have this same issue, so this may be more an issue with iOS 14 than 5.0.0 enhancements.

One addition note: this lag is much-less an issue when running disconnected from xcode's debugger.  Probably lots of time spent synchronizing backtraces between debugger and app.

In the description you say that the issue is not present on 4.1.1, but 4.1.1 doesn't exist, then you say 4.1.0 also have the issue. Can you clarify?

Sorry - yes I was using 4.1.0 and thought that changing between 5.0 and 4.1 produced different behavior.  After more thorough nuking, etc, my current assessment is that it's not related to 4.1 or 5.0 (ie: i see the same behavior/backtraces).  The backtraces still appear to be problematic to me, and I don't currently have access to an iphone running a release prior to 14 in order to definitively state that this doesn't occur on all iOS devices, or what OS version the issue cropped up.

closing since the reporter disappeared and I can't reproduce on 5.0, it was probably a bad update from 4.1 that left the old files without the fix.


File picker tool doesn't have help

Rollover help is not showing up correctly in the file picker tool. Has to do with clearing the message when resetting focus from the action bar.

Fixed help for all of the UI except the action bar buttons.

Looks like displaying help is broken in the action buttons so creating a new issue.


Enabled the no-require-imports rule internally

As per the TODO list in tslint.json.

The PR queue is still pretty huge and this is low priority; closing for housekeeping.


Android Bluetooth mic does not work on voice call

We are using react-native-agora sdk v3.6.2 and our app is currently targeting sdk 31. on android 12 devices, whenever the user connects on voice call via bluetooth, the audio is captured through the device's mic and not through the bluetooth's mic. Please any help would be so helpful.

have you config the permission of bluetooth?

We have recently upgraded to v3.6.2 from v3.5.2 and face the same problem. This happens in Android only.
Our Android target sdk version is 30. Bluetooth permission is granted.
Audio is output through bluetooth headset, but captured from device's mic instead of bluetooth's mic.
Any help would be appreciated!

Same issue here, Microphone of Bluetooth headset not working anymore (as also mentionned in https://github.com/AgoraIO-Extensions/react-native-agora/issues/420#issuecomment-1339545433 from the other issue https://github.com/AgoraIO-Extensions/react-native-agora/issues/420).
@LichKing-2234 : Not sure whether this helps to find the root cause, but the issue only started happening on my Android 13 device (Google Pixel) after we had to change the targetSdkVersion from version 30 to version 31 (which we had to do for our release pipelines not supporting version 30 anymore)

@oelerpearler I have tested our example JoinChannelVideo with my android phone and AirPods pro, and it works fine.

@alekben Thanks a lot, I just wanted to post the very same here; I got help from the Agora Support, and it turned out that indeed, when targeting android SDK higher than 30 in the build.gradle, BLUETOOTH_CONNECT is required ... both in the AndroidManifest.xml as well as in the React Native App by getting permission granted from the user (as also described in the link that you have shared).
@LichKing-2234 I've already asked Agora Support to highlight this bit of information in the React Native SDK Docs, but maybe if you're close to the Documentation Team (or even contributing to it), you might also want to ensure that this information is available in the Agora Docs (less people bothering you in the future)


[ Security] [CVE-2023-0842] xml2js is vulnerable to prototype pollution

[ Security] [CVE-2023-0842] xml2js is vulnerable to prototype pollution
Can someone help fix this security issue? Thanks.
Refer to https://github.com/advisories/GHSA-776f-qx25-q3cc
Affected versions
<= 0.4.23
Description
xml2js version 0.4.23 allows an external attacker to edit or add new properties to an object. This is possible because the application does not properly validate incoming JSON keys, thus allowing the proto property to be edited.
References
https://nvd.nist.gov/vuln/detail/CVE-2023-0842
https://fluidattacks.com/advisories/myers/
https://github.com/Leonidas-from-XIV/node-xml2js/

@lukewang2018 Upgrade xml2js version to latest 0.5.0.

@MChuduk  Thanks a lot for your quick fixing!

@lukewang2018 You're welcome.

Due to the tags in GitHub, npm/yarn/GitHub might be confused as to what the latest non-beta release is.
Manually setting the version to 0.5.0 does resolve the security issue.
Please update the Tags in GitHub?

I tried to use version 0.5.0 in package.json and installed but still observing the security vulnerability
CVE-2023-0842. Is anyone else facing the same vulnerability issue in 0.5.0

Hi @sachindkagrawal18 I had the same problem, as of NPM cli 8.3 there is a new property called overrides that can do the trick while the new version is properly released.
Just add this snippet on your package.json
"overrides": {
  "officeparser": {
    "xml2js": "0.5.0"
  }
}

That worked

Bro dependabot was annoying me this whole time just because of this! haha

@Leonidas-from-XIV any chance of backporting the fix to 0.4? xml2js is pulled in from a deep transitive dependency, and getting every dependency to upgrade isn't gonna be quick.
I know I can force a newer version, but it's hard to say if that's safe without a changelog of some sorts. And regardless, forcing every consumer (14M weekly downloads) to do so seems a bit much.

@SimenB I don't think it's possible, as the fix is a backwards-incompatible change (#603), at least depending on how you use the object that gets returned (if you just use it as a data object . I first thought it could be mitigated in an easier way to stay in the 0.4 series, but this looks like a game of whack-a-mole.
If you have a PR that would fix the issue in a compatible way, I am perfectly happy to add another release to the 0.4 series.
But you can use overrides to force dependencies to use newer versions (as someone pointed out in one of the issues, can't find it anymore).


VS Code takes over .js file icon and type

On Windows install, VS Code is taking over .js and various .js flavor file extension icon and file type.  In Windows explorer, the user no longer sees the JavaScript icon and the detail file Type is "Visual Studio Code".
.php is also overridden.
doc feedback verbatim: "All my files show up with similar icon and similar file description type. It is hard to distinguish now which is the .php file and which is .css file. Even changing registry values doesn't help. It still opens with Code (that's OK) but the icons and file descriptions are very confusing. All my files now show "Visual Studio Code" and nothing else."

The user can still define any other application to be the default application to open those files. What happened in this story is that VS Code is the only application able to open those types of files and thus is automatically associated to them.
If we need to create icons and descriptions for every file type we are able to run, then that's another issue.


Add support for the Roundcube password plugin

We should create a simple driver for the Roundcube password changing plugin: https://github.com/roundcube/roundcubemail/tree/master/plugins/password
Using the API of Modoboa. Roundcube is a very popular webmail client. Doing this has the advantage of not having to have the database world-accessible.

FYI,
vboxadm also implemented an HTTP API on the server side and an roundcube plugin on client side to let users change their passwords (and some other things). Maybe that helps you ;-)
Server side:
https://github.com/dominikschulz/VBoxAdm/blob/master/lib/VBoxAdm/Controller/API.ipm
Client side:
https://github.com/dominikschulz/roundcube-plugin-vboxadm/blob/master/roundcube/plugins/vboxadm/vboxadm.php

So, the API part is done. Now, we need a roundcube plugin.


Update tests now that 3d plots work

Fixes #849
This PR updates our tests so that they expect 3d plots to work if matplotlib 3.3.3 is installed.
The plots also look a bit different than they used to:
Old:

New:

It would be nice if the legend was even more visible, but I think I'll leave it for now.

Oh wow, that’s been a long time coming!


Resource Store Cache Conflict

If two watchers watch the same resource, and one of them watches a smaller subset (namespace or list param scoped), and this watcher gets a Event::Restarted, it will empty the cache for that resource kind and objects will be missing until the other watcher restarts as well.

Moved to https://github.com/krator-rs/krator/issues/5


Bundle Files in single pass

Use webpack array configuration to create the bundles for es5 and es6 in a single build step

Coverage remained the same at 96.672% when pulling 1f8337bf44786a93ae85f764b66a7e8e27123833 on bundle-in-single-pass into 61c83efb8a84de293dd09a7025f87c53fd69a1fe on master.


Heatmap reported to not work on Chrome Version 71.0.3578.98

I have Chrome Version 71.0.3578.98 (Official Build) (64-bit) for Windows.
This problem also occurs on the Mac, where I have version 70.0.3538.110.
The heatmap feature of the protein family sorter is unusable. In modern
Chrome, the only way to enable Adobe flash is to click on the little
message that shows up. Since a custom display appears instead, the little
message does not show up, there is no way to click on it, and instead I am
directed to a help page with completely obsolete information. I have
attached a screen shot of the little message that appears on normal flash
web pages.

@ARWattam - I'm going to assume this is no longer an issue for the user and close the ticket.
I've just rewritten an updated version of the Flash Player tutorial that should be easier to follow. It covers modern versions of Chrome, Firefox, Safari and Edge.
https://github.com/PATRIC3/p3_docs/pull/83 is the update. It will go live on the docs website when it auto updates soon.


A reload or restart option could be great.

VSCode Version:1.2.1
OS Version:arch linux

when i updated some definition files or some path it does not show in vscode instantly so i have to close vscode and reopen .
If there is a reload option like just after installing extensions it would be great

Invoke command palette and type reload then you can reload JavaScript/TypeScript/Window :)

but a button on side pane would be great

@rebornix well i didint know that thanks :)

It's a very destructive action that is only really needed for developers of vscode or users that install an extension and dismiss the dialog. I'm going to close this off since I don't see this happening.
If you need to use it more frequently for some reason I recommend either adding a keybinding or accessing it via command palette (I use ctrl+shift+p, "rew", enter on Linux)

I had no idea about this command until I saw @Tyriar used it. A button on the side pane would be intuitive but we can't move all commands to it, @Vishal-Isharani right?
Maybe we can have something like Ubuntu hot key map when you press Windows button for a few seconds, this might be a good extension, what do you think @Tyriar

@rebornix I don't think it's a common case for users to need to reload outside of the extension install prompt. If they do you can always add a simple keybinding to make it easy:
{ "key": "alt+ctrl+r", "command": "workbench.action.reloadWindow" }

On adding it to the side panel, we need to be very careful when adding buttons to the UI, especially prominent places like on the activity bar. The UI is intentionally minimalistic in order to keep the UI lightweight, both in performance and perceived 'heaviness'. Doing something like this would be going against that design decision.

i got it thank you @Tyriar  and @rebornix


select inputs

This brings <select> input styling into Lit. There's no custom element here, it only exposes styles that consumers can apply to their own native <select>s inside their elements.
These changes do not yet bring it inline with the current design -- I have those changes ready and will add them in a follow-up PR. I just wanted to make these changes first so we could verify that they don't cause a visual diff change.
I've made a few modifications to the original Sass-based CSS:

I removed width: 100% on the <select>. In most cases, we just want its width to be as wide as the largest option. In cases where width is constrained, consumers need to put a width or max-width on it anyway.
I removed the CSS that was only applying things if :not([multiple]). Since consumers need to manually apply these styles to their own selects, they should just not do it for ones that support multiple selection.
I removed the CSS transition -- it wasn't doing anything.
The media query is gone. This was originally needed years ago to apply special styles for Chrome/Safari and Firefox and to prevent pre-IE10 from not working. Not needed anymore.
I've removed the font-weight styling on options. I couldn't find any browser combination where this did anything.
Placeholder, search-cancel-button, ms-clear styles are gone. These were only there because selects were using the same Sass mixins as other text-based inputs. Selects don't need these overrides.

@svanherk and I were chatting and there's another approach we could take, and that's to create a <d2l-input-select> custom element that simply applies these styles automatically to slotted child <select>s. We could even keep these styles and folks could choose which approach they want to use.
The advantage to a custom element would be that it could be used outside of LitElements, and it would automatically apply the styles and RTL would "just work". It could also pull off the text overflow gradient that's in the design that we can't currently do.
Disadvantage is that it's a kind-of useless custom element, and the <select>s would be excluded from <form> due to the shadow root.
Thoughts?

I'm a fan of just using native select if we can! Is there an easy way to make the styles available as both an ES6 module (for Lit) as well as just native CSS (for not-Lit)?

I'm not aware of a way to export as a CSS literal (which is a Lit thing only, currently anyway) but also expose things for use inside a normal HTML document.

Re the d2l-input-select idea and styling via ::slotted...

will the shadow boundary (between the select and the option elements in the slot) cause any problems (a11ywise)?
the biggest limitation of ::slotted() is that it only targets immediate children in the slot. So... are optgroup elements not supported, as they basically result in option elements one (or more) levels deeper?

The slot would be between <d2l-input-select> and the <select> element. Like <d2l-input-select> would only have <slot></slot> in its template, and would style via ::slotted(select) { ... }. Since we don't need to style the <option>s (that's the operating system's job), the limitation of immediate children is fine.

Ah ok...  I'd assumed we still had some option/optgroup styles.  It looks there was some previous style that was overriding font-weight for option elements.  I guess that's not needed anymore, and we're comfortable with it never being possible without larger changes?

Yeah there's a note about that in the PR description -- I couldn't find a scenario where it did anything. The operating systems have taken control of <option> rendering, and until the platform opens that back up, I think we're fine with that.

The only case that comes to mind is - we were somewhere trying to normalize the font-weight of optgroup (but I see no evidence that we were doing that, and nor does it seem to be even possible), and then that override (from the original code) was trying to "reset" font-weight for children of options within it. 🤷♂
select is such garbage.  For sake of comparison... default style of Chrome and FF (or more likely macOS vs Windows):

Alright, thanks for the discussion everyone. I'm going to merge this as-is, but I won't do a release just yet as I want to get the new design in (PR will follow in a few minutes) and then also bring the Sass styles over from BSI that the monolith will use so everything is centralized. Then I'll release this! 🚀


Add check to see if /json was changed

Following up to PR #528 , adding a check to the build process to see if any files were touched and fail if so.
Adding a /json here to test the build's new code and will revert before merging.

I don't understand travis...so yeah? 🤷

I'll take it 🍰


Installation failure

from @warshanks

So I tried to reinstall and I got this
pip3.6 install discurses
Collecting discurses
  Using cached discurses-0.3.0.tar.gz
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-swwsrz2u/discurses/setup.py", line 5, in <module>
        from pip.req import parse_requirements
    ModuleNotFoundError: No module named 'pip.req'

----------------------------------------

Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-install-swwsrz2u/discurses/

The concerned line should not appear in setup.py anyway.
Still, I have no problem while setting up discurses.

It's fixed by the way.

Using pip to install Discurses threw an error about the running Python version for me as well. After installing python3-pip and using pip3 to install Discurses the installation worked without error. Shouldn't the installation instructions (readme) be adjusted for this?


in c file color is wrong

Bug description
(this template is intended for bug reports, erase it when requesting features/enhancements)
Steps to reproduce the bug

go to piskelapp.com
import a png
export c file
get c file , extract hex color from c file, transfer c to json file
import xcode json file, to render pixel picture
every color is change , not same as png file

not only png is wrong , .piskel file is also wrong
Environment details

operating system: macos
browser (or offline application version):  chrome 131

Feel free to include a screenshot of the bug, a .piskel file of the sprite or anything else that can help us investigate.
original png

render

i find this problem because of the png is RGB Color, but c file is BGR Color


A way to change video on the fly?

hey nice wrapper for vue
I tried changing the data but it doesn't change the video, is there a way to change the video?
Thanks

Can you show me how you're trying to use the component? The channel, collection, and video parameters are all placed on watch.

yep no problem.
im using the compnent inside of another component
app.js
import VideoIndex from './components/videos/VideoIndex.vue';
import Twitch from 'vue-twitch-player'
Vue.component('video-index', VideoIndex);
Vue.component('twitch-player', Twitch);

VideoIndex.vue (essential snippets)
<twitch-player class="video-play" :video="first.provider_id"></twitch-player>

<div v-for="video in videos" @click="loadVideo($event, video)" class="col-12 col-sm-6 col-md-3">

        methods: {
            loadVideo(event, video) {
                console.log(video);
                this.first = video;
            }
        },

Hope this makes sense.
Kind Regards
MooseH

It looks like you've run into one of Vue's Change Detection Caveats. Namely, you can't assign an Object to a root data element. You'll need to do something like this:
loadVideo(event, video) {
  console.log(video);
  this.$set(this, 'first', video);
}