Take codegen depth as an input to @model

Is your feature request related to a problem? Please describe.
There are lots of cases where a model is self-referential and the entities have a bi-directional relationship. Eg. posts with comments where each comment has a reference back to the post. When queries and mutations are generated for these models, the return objects can be unnecessarily nested.
Describe the solution you'd like
Be able to define optional codegen depth on a per-operation basis that will override the global default. The following is an example @model SDL where this behavior would be specified
directive @model(
  queries: ModelQueryMap,
  mutations: ModelMutationMap,
  subscriptions: ModelSubscriptionMap
) on OBJECT
input ModelMutationMap { create: String, update: String, delete: String, createDepth: Int, updateDepth: Int, deleteDepth: Int }
input ModelQueryMap { get: String, list: String, getDepth: Int, listDepth: Int }
input ModelSubscriptionMap {
  onCreate: [String]
  onUpdate: [String]
  onDelete: [String]
  level: ModelSubscriptionLevel
}
enum ModelSubscriptionLevel { off public on }

And then it could be used in the following way:
type Comment @model(queries: {getDepth: 4, listDepth: 5}) {
  id: ID
}

to generate get queries with a depth of 4 and list queries with a depth of 5.
Describe alternatives you've considered
Specify a reasonable codegen depth for the current project and create custom queries / mutations for any operations where this default doesn't work. This creates unnecessary work making sure these custom queries stay in sync with the rest of the project.
Another option that seems like it may solve most use cases is to ignore the "items" field in the depth count when generating queries for one-to-many relationships. Based on discussion in aws-amplify/amplify-cli#1013 this would generate code that is closer to developer expectations. However I would not say this is the same feature because there still may be cases where you have some models that are deeply nested and others that aren't.
Additional context
There have been some other related requests made aws-amplify/amplify-cli#1013, but they all seem to target trying to come up with more sensible defaults whereas this gives the developer total control to adjust depth for each operation. Another related issue is aws-amplify/amplify-cli#2977.

This would be so useful! There is no one-size-fits all w.r.t statement depth in most applications. Choosing a deep level and using auto-generated statements makes our app slow. We also tried hand-created queries, but maintaining them is a nightmare.

This would make my life so much easier. I currently generate graphql files of depth 2, 3, and 4 for different use cases in my application. These need to be regenerated every time I make a change to my schemas.
I wrote a gist that helps with this by running codegen configure before codegen, while also printing out the suggested configure settings before the prompt.

I wrote a gist that helps with this issue by running codegen configure before codegen, while also printing out the suggested configure settings before the prompt.
Fixing this Issue would make my life so much easier and require zero user input during the codegen. I currently generate graphql files of depth 2, 3, and 4 for different use cases in my application. These need to be regenerated every time I make a change to my schemas.

Would love to see this as well. Is there any idea if this would be picked up in the near future?


Allow imread(::Array, Images.ImageMagick) to read from array of bytes

I came across issue #279 since I head the same problem so I implemented @timholy's suggested fix.
The patch allows imread to accept an Array as well as String and IO - I'm not sure whether this is too general though, do you have any opinion on tightening it?

Maybe insist that it be a Vector{UInt8}, just so it seems more like a buffer?
Otherwise, this looks great!

Thanks!


possible memory leak when various functions return failure.

There are possible leak  in ntlm_compute_ntlm_v2_response(), when in error, the allocated SecBuffer are not freed.

@freerdp-bot test

@akallabeth looks like the CI host for vs2010 needs some love...

@hardening VS2010 doesn't like the int ret = -1; (as mentioned by @akallabeth)

@freerdp-bot test

@qunying please run clang-format on your changes (or copy the diff from the output of @freerdp-bot and apply) to enforce coding guidelines.

@freerdp-bot test


fix: migrate current auth to ts

The type of this PR is: TYPE

Description

The new onboarding flow lives currently on the react-native side of the world and gets stuff from the authModel. By enabling the new flag on users devices, they will be signed off since the authModel is not populated for them.

This PR does two things:

Migrate the nativeState to the authModel to make sure users do no get signed off when they get the new flag.
Populate the authModel already when the users log in to make sure both states are filled

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a changelog entry below or my changes do not require one.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Migrate nativeState to authModel - mounir

This PR contains the following changes:

Dev changes (Migrate nativeState to authModel - mounir)

  Generated by :no_entry_sign: dangerJS against 66927d6a5619990762dc1e1a0667610f6cd00fb4


Support development platforms other than Arch Linux

Candidates:

macOS
Ubuntu

I've been using Asahi Linux and it switched from Arch Linux to Fedora some months ago.
So I'm now using Fedora. I added instructions for Fedora.
5ceb008a558a103f6ab2e80b000f4edcca03695b.


A-10A resupply for gau is 1 bullet at a time

https://7cav.us/threads/4975/#post-27202

TODO: Check configs for A10A / A10C for ACE_Rearm


Incompatible declaration of Traits\HasSettingsField::settings() error on PHP 8

Hi. After updating to PHP 8.0, I get this exception:
Declaration of Glorand\Model\Settings\Traits\HasSettingsField::settings(): Glorand\Model\Settings\Contracts\SettingsManagerContract must be compatible with Glorand\Model\Settings\Traits\HasSettings::settings(): Glorand\Model\Settings\Managers\AbstractSettingsManager
at ...vendor\glorand\laravel-model-settings\src\Traits\HasSettingsField.php:37

@alin78hai I didn't started to test the package on PHP 8, but thx for the observation, probably is the moment to start...

Hey, any news on this? It blocks my project from upgrading to PHP 8 (however: just asking! Thanks for your efforts being put in this package - don't wanna pressure!).
Sebastian

@SebastianSchoeps thanks! I want to make a new release next week, probably monday.

@SebastianSchoeps, @alin78hai  we are PHP8 ready: #79

Thank you - great!


Service Tier doesn't exist / is not installed

PLEASE DO NOT INCLUDE ANY PASSWORDS OR TOKENS IN YOUR ISSUE!!!
Getting an error when trying to make a container using BcContainerHelper
$ContainerName = 'mod'
$Version = '18.1'
$Artifact = Get-BCArtifactUrl `
        -type OnPrem `
        -version $Version `
        -country 'dk' `
        -select Latest

New-BCContainer `
    -artifactUrl $Artifact `
    -containerName $ContainerName `
    -auth Windows `
    -updateHosts `
    -accept_eula `
    -accept_outdated

BcContainerHelper is version 2.0.16
BcContainerHelper is running as administrator
Host is Microsoft Windows 10 Pro - 21H1
Docker Client Version is 20.10.8
Docker Server Version is 20.10.8
Removing container mod
Removing mod from container hosts file
Removing mod-* from container hosts file
Removing C:\ProgramData\BcContainerHelper\Extensions\mod
Fetching all docker images
Fetching all docker volumes
INFO: Windows 10 21H1 images are not yet available, using 2004 as these are found to work better than 20H2 on 21H1
Using image mcr.microsoft.com/businesscentral:10.0.19041.1237
Creating Container mod
Style: onprem
Multitenant: No
Version: 18.1.24822.26330
Platform: 18.0.24800.26311
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.19041.1237 (2004)
Host OS Version: 10.0.19043.1266 (21H1)
WARNING: Host OS is 21H1 and Container OS is 2004, defaulting to process isolation. If you experience problems, add -isolation hyperv.
Using process isolation
Using locale da-DK
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Files in C:\ProgramData\BcContainerHelper\Extensions\mod\my:
- AdditionalOutput.ps1
- MainLoop.ps1
- SetupVariables.ps1
- updatehosts.ps1
Creating container mod from image mcr.microsoft.com/businesscentral:10.0.19041.1237
475731a44a422a29ce84466724d08913d34e24a9a23a79978e5cfd9f8edf4415
Waiting for container mod to be ready
Using artifactUrl https://bcartifacts.azureedge.net/onprem/18.1.24822.26330/dk
Using installer from C:\Run\150-new
Installing Business Central
Installing from artifacts
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
c:\dl\onprem\18.1.24822.26330\platform\ServiceTier\Program Files
c:\dl\onprem\18.1.24822.26330\platform\ServiceTier\System64Folder
Copying PowerShell Scripts
c:\dl\onprem\18.1.24822.26330\platform\WindowsPowerShellScripts\Cloud\NAVAdministration
c:\dl\onprem\18.1.24822.26330\platform\WindowsPowerShellScripts\WebSearch
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules
Determining Database Collation from c:\dl\onprem\18.1.24822.26330\dk\database\Demo Database NAV (18-0).bak
Initializing...
Setting host.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting gateway.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting kubernetes.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is mod
PublicDnsName is mod
Using Windows Authentication
Starting Local SQL Server
Starting Internet Information Server
Modifying Service Tier Config File with Instance Specific Settings
Service Tier doesn't exist / is not installed
at <ScriptBlock>, C:\Run\navstart.ps1: line 153
at <ScriptBlock>, C:\Run\start.ps1: line 363
at <ScriptBlock>, <No file>: line 1Error
Using artifactUrl https://bcartifacts.azureedge.net/onprem/18.1.24822.26330/dk
Using installer from C:\Run\150-new
Installing Business Central
Installing from artifacts
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
c:\dl\onprem\18.1.24822.26330\platform\ServiceTier\Program Files
c:\dl\onprem\18.1.24822.26330\platform\ServiceTier\System64Folder
Copying PowerShell Scripts
c:\dl\onprem\18.1.24822.26330\platform\WindowsPowerShellScripts\Cloud\NAVAdministration
c:\dl\onprem\18.1.24822.26330\platform\WindowsPowerShellScripts\WebSearch
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules
Determining Database Collation from c:\dl\onprem\18.1.24822.26330\dk\database\Demo Database NAV (18-0).bak
Initializing...
Setting host.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting gateway.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting kubernetes.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is mod
PublicDnsName is mod
Using Windows Authentication
Starting Local SQL Server
Starting Internet Information Server
Modifying Service Tier Config File with Instance Specific Settings
Service Tier doesn't exist / is not installed
at <ScriptBlock>, C:\Run\navstart.ps1: line 153
at <ScriptBlock>, C:\Run\start.ps1: line 363
at <ScriptBlock>, <No file>: line 1
Initialization of container mod failed
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.16\ContainerHandling\Wait-NavContainerReady.ps1:42 char:17
+ ...             throw "Initialization of container $containerName failed" ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Initialization of container mod failed:String) [], RuntimeException
    + FullyQualifiedErrorId : Initialization of container mod failed```

**Additional context**
It just started to day.
I have reinstalled docker.
I have installed all windows 10 updates

Have a look at #2086


Rename "unit tests" to "tests" in selection box when configuring tests

Steps to reproduce:
Run the command to Configure Tests and select pytest. This will show up:

Should be changed to "[...] containing the tests"

I have verified that this works on Linux using the insiders build from last night.


KAFKA-12943: update aggregating documentation

Regarding documentation glitch reported in KAFKA-12943, indeed implicit types are not automatically resolved unless explicitly defined in this case.
As confirmed in:
https://stackoverflow.com/questions/51040555/the-method-withvalueserde-in-the-type-materialized-is-not-applicable/51049472

@showuon @mjsax is there anything missing to merge this to trunk?

Thanks for the PR! Merged to trunk.


Response wrongs by Custom Json Response

Miss Cache response:

Hit cache response:

CustomJSONClass:

Close, i use default response json.


Basic installation with empty aws account

Hi all,
I'm trying to create vpn with this module. I created fresh aws account. But i'm getting error. Version is 0.2.3
Error: Error building docker image: Error response from daemon: Cannot locate specified Dockerfile: Dockerfile
I'm using this config:
instance_type = "t2.nano"  wg_group_name = "VPN"  vpc_id = "MY_VPC_ID"  wg_admin_email = "MY_EMAIL"
Do you have any idea?

Hi @oytunyuksel thank you for intresting in Wireguard module.
could you please provide full terraform definitions and steps to reproduce.

Hi @azatsafin,
Thank you for help in advance.
Actually, i'm using in Terragrunt. Terragrunt is running terraform in the background. The Terragrunt file is below:
`terraform {
source = "git::https://github.com/azatsafin/terraform-aws-wireguard.git//?ref=v0.2.3"
}
include {
path = find_in_parent_folders()
}
dependency "vpc" {
config_path = "../vpc"
}
inputs = {
instance_type                = "t2.nano"
wg_group_name                = "VPN"
vpc_id                       = dependency.vpc.outputs.vpc_id
wg_admin_email               = "MY_EMAIL"
}`
And Terragrunt apply's related outputs are below:
`
module.wg_manage_image.docker_registry_image.this will be created

resource "docker_registry_image" "this" {

id                   = (known after apply)

insecure_skip_verify = false

keep_remotely        = false

name                 = "324245454524.dkr.ecr.eu-central-1.amazonaws.com/wireguard-vpn-service-wg-manage:e1e420b522c92f7e2657973a4426fa74bc0ee8691cebff3f2b86ed953eb62hef"

sha256_digest        = (known after apply)

build {

context    = "./lambdas/wg-manage-iam:b89a4e002ffe8868c2ed2059be1d2f5c4a4ed8dd3ef104cf04d64aa160344432"
dockerfile = "Dockerfile"
}
}

Plan: 14 to add, 0 to change, 0 to destroy.
module.wg_manage_image.docker_registry_image.this: Creating...
╷
│ Warning: Argument is deprecated
│
│   with module.cognito_auth_redirect.aws_s3_bucket_object.lambda_package,
│   on .terraform/modules/cognito_auth_redirect/main.tf line 115, in resource "aws_s3_bucket_object" "lambda_package":
│  115:   bucket        = var.s3_bucket
│
│ Use the aws_s3_object resource instead
│
│ (and 9 more similar warnings elsewhere)
╵
╷
│ Error: Error building docker image: Error response from daemon: Cannot locate specified Dockerfile: Dockerfile
│
│   with module.wg_manage_image.docker_registry_image.this,
│   on .terraform/modules/wg_manage_image/modules/docker-build/main.tf line 22, in resource "docker_registry_image" "this":
│   22: resource "docker_registry_image" "this" {
│
╵
`

@oytunyuksel could you please check your setup from specified branch. I still not sure what option is the best path.cwd or path.module , because path.cwd could lead re-creation of resource if the path will be changed due to CD/CI systems or running the same state from different computers.

I checked and i think passed the Dockerfile finding step, but i get this error now:
Error: Error building docker image: 1: The command '/bin/sh -c yum install -y wireguard-dkms wireguard-tools' returned a non-zero code: 1
│
│   with module.wg_manage_image.docker_registry_image.this,
│   on .terraform/modules/wg_manage_image/modules/docker-build/main.tf line 22, in resource "docker_registry_image" "this":
│   22: resource "docker_registry_image" "this" {

It looks like related to some wireguard packages. Probably something wrong on Dockerfile.

Just checked full deployment from the scrutch with following terragrunt definition
terraform {
  source = "git::https://github.com/azatsafin/terraform-aws-wireguard.git?ref=align_submodule_docker_image_path"
}

inputs = {
  instance_type                = "t2.nano"
  wg_group_name                = "VPN"
  wg_admin_email               =<EMAIL_ADDRESS>}

All works fine.
But I have to add one notice, you have to use Nitro instance types, because I had hardcoded interface name in iptables settings.
Nitro instance type has network improvments in term of speed for the same price.

I'm still getting same error @azatsafin . Can it related with Apple M1? Did you test in M1 chip before? For my guess, maybe "wireguard-dkms" and "wireguard-tools" packages doesn't support M1.

Thanks a lot @azatsafin . I will try find a solution.


Make shapecast 3d node

https://github.com/godotengine/godot/pull/56470

Will close here and tack on github proposals.


"Wallet successfully recovered" is shown even when the node is not accessible

Describe the bug
When a user tries to recover a wallet, and the recovery fails due to a connection error to the server, the application incorrectly prints "wallet successfully recovered".
To Reproduce

Open the Epic GUI Wallet;
Click on "Recover";
Add a valid seed phrase from usernet/floonet wallet;
Click on "Finish enter seed phrase";
Select the wallet folder;
Fill the empty blanks (account, and the others);
Select "Mainnet" (or another net that is different from the one where the seed phrase was generated);
Click on "Recover";

Expected behavior
The application should report that the Epic server is not running/Node is not accessible.
Required data
OS: Windows
Epic server: master branch
Epic wallet: the same one that comes with the Epic GUI wallet project (available in "resources/bin/win")
Epic miner: master branch
Node version: v16.15.0
npm version: 8.5.5

You need no node to recover the wallet. If node is not accesible at this stage does not matter


[V2] Access is allowed from event dispatch thread only.

Version and Environment Details
Operation system: MacOS Mojava Version 10.14.4
IDE name and version:
PhpStorm 2019.2 EAP
Build #PS-192.4205.18, built on May 22, 2019
Plugin version: 2.1.0
Description
It spams me with errors all the time, but it seems to be working fine though.
I have created a gist wit the 3 error files so you can review them and debug through them
https://gist.github.com/olivernybroe/1a7f568bd0974513bdedd0508c900587

Ho Oliver.
Thanks for using the plugin.
I took a look at the gists. I should be able to reproduce and fix the issue on that basis.
Best regards,
Jim.

Same fatal error:
PhpStorm 2019.2 EAP
Build #PS-192.4205.18, built on May 22, 2019
PhpStorm EAP User
Runtime version: 11.0.3+12-b262.1 amd64
VM: OpenJDK 64-Bit Server VM by JetBrains s.r.o
Windows 8.1 6.3

Fixed in the upcoming 2.1.1 release.

@jimkyndemeyer Thank you for fixing this! Could you release the new version soon? My IDE is spammed with errors and your wonderful extension doesn't work anymore 😢

@dan-lee You can always just build it yourself and install the zip. Normally with intellij plugins you can just run the gradle command buildPlugin and then you get the zip.

I hope to release it by the end of the week. You can build it like Oliver said, but the plugin should also work in the stable non-EAP webstorm.

Testing with the EAP I found other new assertions introduced by https://github.com/JetBrains/intellij-community/commit/a0c101cd996f59b93d14f095b66b133892783cb9
This breaks the error annotator in the EAP, so please downgrade to the latest stable version if you encounter problems.
This is holding back the 2.1.1 release until I can work out how to get the editor instance without triggering the exception.

Fixed in the 2.1.1 release, pending JetBrains approval.

Amazing work! Thanks a lot, looking forward to the new release :)


Remove duplicate dependency

Ref: https://github.com/coveralls-clients/coveralls-python/blob/master/setup.py#L29

Coverage remained the same at 90.246% when pulling a9eb8d36541b90ec8aa61cb22ddcb8a41e24b4fa on joeirimpan:develop into b36fdc9c31618a4bc8313cc4ca071b56f7f0ca3b on sidecars:master.


Question: What is the best way to follow up with the Syntax crew on topics covered?

Recently I listened to a Syntax episode where questions from the Potluck form were covered. Personally I had some insights to share regarding one of the topics covered and so my question is...should I simply use the Potluck form to provide my long-form thoughts on topics covered from a previous Potluck content review episode? Or should I supply here in the website repo issues?
Simply want to reach out with feedback on show content but in a way that is helpful / encouraged. Thank you!

Hey @outwebbed – Potluck is one, or you could join our Discord which isn't well publicized (and admittedly, we're only occasionally active).


Problems downloading latest Jaybird (Firebird JDBC) driver due to dependency relocation

The latest version of Jaybird (Firebird JDBC driver) was released with new artifact coordinates on Maven (org.firebirdsql.jdbc:jaybird with a version per Java version). The old coordinates - for Java 8 org.firebirdsql.jdbc:jaybird-jdk18 - now have a POM with a relocation, see the pom.xml for org.firebirdsql.jdbc:jaybird-jdk18:4.0.0.java8:
<distributionManagement>
  <relocation>
    <artifactId>jaybird</artifactId>
  </relocation>
</distributionManagement>

When updating the driver to version 4.0.0.java8 in DBeaver, DBeaver doesn't follow the relocation specified in the POM, but tries to directly download the jar assuming the standard Maven conventions. This results in the error:
Driver file download failed.
Do you want to retry?

Reason:
Can't open 'https://repo1.maven.org/maven2/org/firebirdsql/jdbc/jaybird-jdk18/4.0.0.java8/jaybird-jdk18-4.0.0.java8.jar': Not Found

Ideally, DBeaver should follow the relocation specified in the pom.xml to org.firebirdsql.jdbc:jaybird:4.0.0.java8. However, if that is not possible, I would recommend that DBeaver switch to the new coordinates. As a complication, the new coordinates of Jaybird has older versions (3.0.x and older) with relocation to the 'old' coordinates.

I cannot reproduce the issue. The path is shown maven:/org.firebirdsql.jdbc:jaybird-jdk18:RELEASE for driver 4.0.0.java8 in DBeaver.
Have you tried:

Open Connection settings> Main> Edit Driver Settings
Press Download/Update btn
Select 4.0.0.java8 in Version dropdown menu.
Press Download?

If it doesn't help please tell your OS an DBeaver version.

When I reported the issues, I was using DBeaver 7.0.0. I have updated from DBeaver 7.0.0 to 7.0.1, and I also don't get an error anymore, but it also doesn't download the driver.
I'm using the 64 bit version on Windows 10

My starting point is (`org.firebirdsql.jdbc:jaybird-jdk18-3.0.8):

Then I click on Download/Update, and select version 4.0.0.java8

Click Download

The dialog closes, and no error appears anymore at this time, but attempts to test the connection result in error

Can't create driver instance
  Error creating driver 'Firebird' instance.
Most likely required jar files are missing.
You should configure jars in driver settings.

Reason: can't load driver class 'org.firebirdsql.jdbc.FBDriver'
  Error creating driver 'Firebird' instance.
Most likely required jar files are missing.
You should configure jars in driver settings.

Reason: can't load driver class 'org.firebirdsql.jdbc.FBDriver'
    org.firebirdsql.jdbc.FBDriver
    org.firebirdsql.jdbc.FBDriver

If I downgrade to DBeaver 7.0.0, then after step 3 I get error
Driver file download failed.
Do you want to retry?

Reason:
Can't open 'https://repo1.maven.org/maven2/org/firebirdsql/jdbc/jaybird-jdk18/4.0.0.java8/jaybird-jdk18-4.0.0.java8.jar': Not Found

I will update maven artifact configuration in the next version. Thanks for th report.
Also there was a bug in jar files resolution (fixed in #8326)

Verified.


Fix for exported resources with async storeconfigs [11415]

Hey, I have a problem with the following configuration:
thin_storeconfigs = true
async_storeconfigs = true

For example, when I tried to export nagios_host, puppetmaster lost host_name variable during pson export to stomp queue. Host_name variable is used as "namevar" for nagios_host resource and ignored during pson / json export. The patch removes "namevar" check during the export and fixes the problem.
Signed-off-by: Alexey Lapitsky<EMAIL_ADDRESS>

Hey.  Thanks for that - given that PSON transport of these is a fairly low level, and pervasive, part of the system I am going though and doing due diligence to make sure it won't break anything, and that I understand why we do this the way we do.
The code itself looks fine, and thanks for submitting the test.


Stuck during warmup in benchmark_latency.py

I'm trying to run the latency benchmark and am getting stuck. Am I using it correctly?
$ python benchmark_latency.py --model=meta-llama/Llama-2-7b-hf --tokenizer=hf-intern
al-testing/llama-tokenizer --n 100
Namespace(model='meta-llama/Llama-2-7b-hf', tokenizer='hf-internal-testing/llama-tokenizer', tensor_parallel_size=1, input_
len=32, output_len=128, batch_size=8, n=100, use_beam_search=False, num_iters=3, trust_remote_code=False)
INFO 08-03 21:56:59 llm_engine.py:70] Initializing an LLM engine with config: model='meta-llama/Llama-2-7b-hf', tokenizer='
hf-internal-testing/llama-tokenizer', tokenizer_mode=auto, trust_remote_code=False, dtype=torch.float16, use_dummy_weights=
False, download_dir=None, use_np_weights=False, tensor_parallel_size=1, seed=0)
INFO 08-03 21:56:59 tokenizer.py:29] For some LLaMA-based models, initializing the fast tokenizer may take a long time. To
eliminate the initialization time, consider using 'hf-internal-testing/llama-tokenizer' instead of the original tokenizer.
/mnt/ml/vllm/vllm/worker/worker.py:231: UserWarning: The torch.cuda.*DtypeTensor constructors are no longer recommended. It
's best to use methods such as torch.tensor(data, dtype=*, device='cuda') to create tensors. (Triggered internally at ../to
rch/csrc/tensor/python_tensor.cpp:83.)
  tokens_tensor = torch.cuda.LongTensor(input_tokens)
INFO 08-03 21:57:18 llm_engine.py:196] # GPU blocks: 1110, # CPU blocks: 512
SamplingParams(n=100, best_of=100, presence_penalty=0.0, frequency_penalty=0.0, temperature=1.0, top_p=1.0, top_k=-1, use_b
eam_search=False, stop=[], ignore_eos=True, max_tokens=128, logprobs=None)
Warming up...

^CTraceback (most recent call last):
  File "/mnt/ml/vllm/benchmarks/benchmark_latency.py", line 81, in <module>
    main(args)
  File "/mnt/ml/vllm/benchmarks/benchmark_latency.py", line 54, in main
    run_to_completion(profile=False)
  File "/mnt/ml/vllm/benchmarks/benchmark_latency.py", line 43, in run_to_completion
    llm.generate(prompt_token_ids=dummy_prompt_token_ids,
  File "/mnt/ml/vllm/vllm/entrypoints/llm.py", line 130, in generate
    return self._run_engine(use_tqdm)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/mnt/ml/vllm/vllm/entrypoints/llm.py", line 150, in _run_engine
    step_outputs = self.llm_engine.step()
                   ^^^^^^^^^^^^^^^^^^^^^^
  File "/mnt/ml/vllm/vllm/engine/llm_engine.py", line 300, in step
    seq_group_metadata_list, scheduler_outputs = self.scheduler.schedule()
                                                 ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/mnt/ml/vllm/vllm/core/scheduler.py", line 251, in schedule
    scheduler_outputs = self._schedule()
                        ^^^^^^^^^^^^^^^^
  File "/mnt/ml/vllm/vllm/core/scheduler.py", line 126, in _schedule
    prompt_limit = min(
                   ^^^^
KeyboardInterrupt

hm the tests also getting stuck so i think there must be an issue with my installation..
ytest -v tests/kernels/test_attention.py
=================================================== test session starts ===================================================
platform linux -- Python 3.11.4, pytest-7.4.0, pluggy-1.2.0 -- /home/tmm1/micromamba/envs/test/bin/python3.11
cachedir: .pytest_cache
rootdir: /mnt/ml/vllm
plugins: anyio-3.7.1
collected 2 items

tests/kernels/test_attention.py::test_single_query_cached_kv_attention ^C

i'm going to start over with py 3.10 and torch stable instead of 3.11 + nightly

Okay the test is not stuck, just very slow. It takes about 3 minutes to run.
I'm not sure what's normal here or how long latency benchmark warm up should take.

Aha, using --n 100 meant 100 samples and so it was taking forever.
Changing to --n 1 gives me a result much faster.

Feel free to submit another issue if you anything else goes wrong!


update and merge quickstart notebooks

updates optax101.ipynb notebook with README examples (same as #435, closes #229) and merges optax101.ipynb with quick_start.ipynb (closes #663)

Hello @amosyou,
And thanks again! As far as I can see, the optax-101 is just the README file put in notebook format.
@fabianp had made a pass on the README file recently to correct diverse typos. Could you sync with main and use the newest version of the README?
(examples of typos are broken links as we changed the docs structure, you may not want to redo that :) )

sounds good! also wanted to take note of #663, @fabianp did we come to a consensus on whether to merge the notebooks? if so, happy to include those changes here in addition to adjusting for the typos

yes, ideally we would merge both notebooks. What you propose sounds great @amosyou !

@fabianp I decided to include both examples in the notebook (ie. linear model from quick_start, mlp from optax-101). Coming from someone who's new to optax, I found the linear model example really useful and easy to understand, but the mlp example is good for understanding the params as a dict. Let me know what you think!

sounds perfect. Let me know when I should take a look at the diff 😄

yup feel free to take a look!

hey @amosyou , do you think you'll have cycles to implement the changes in the next couple of days? and if not, do you mind if I take over? I would really like to merge this before making any other changes to the documentation

yup! apologies for the delay, I can get the changes in today

thanks! And no rush, I just want this to be merged before I tackle other
doc improvements
On Tue, Jan 30, 2024 at 5:35 PM Amos You @.***> wrote:

yup! apologies for the delay, I can get the changes in today
—
Reply to this email directly, view it on GitHub
https://github.com/google-deepmind/optax/pull/726#issuecomment-1917441075,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACDZB5PMHEQYOLPZD5BARLYREOOZAVCNFSM6AAAAABCEK7PU6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJXGQ2DCMBXGU
.
You are receiving this because you were mentioned.Message ID:
@.***>

np! I had a couple comments under your code review, there might still be an error under one of the code cells. Also, I could either do it here or open a new issue to delete the other notebook in the gallery now that it's been combined

I've seen, no worries. I'll make a pass on it before merging :-)


ts: minor update for Provider.publicKey and AccountClient.createInstruction

Summary of Changes

Change type of Provider.publicKey to PublicKey | null so that it can be compatible with @solana/wallet-adapter-react's WalletContextState.publicKey
Change type of first param of AccountClient.createInstruction from Signer into PublicKey since Signer is never used for TransactionInstruction

This is a breaking change so it's not really a "minor update".

@acheroncrypto what do you think if we only update the first bullet?
PR is ready here - https://github.com/coral-xyz/anchor/pull/2838

Does this not work if you use useAnchorWallet instead of useWallet?

Does this not work if you use useAnchorWallet instead of useWallet?

useAnchorWallet is for wallet is connected only. We also need to read public onchain data w/o having wallet connected.
I think this is why Provider interface has optional publicKey.

Please give an example on what you want to do.

Please give an example on what you want to do.

For example I wanted to implment a model wherer it read & write data from and to onchain.
I can initialize Program instance using useWallet() which it implemented Provider interface.
Without having wallet connected, readMyDataByAddress() works but createMyData() not because provider.publicKey is null, or wallet is not connected.
class MyData {
  constructor(readonly program: Program) {}
  get provider(): Provider {
    if (!provider.publicKey) throw Error("Please connect your wallet");
    return this.program.provider;
  }
  async readMyDataByAddress(address: PublicKey): Promise<MyDataState> {
    return this.program.account.myData.fetch(address);
  }
  async createMyData(): Promise<TransactionSignature> {
    ...
    this.provider.signTransaction(tx);
    return this.provider.sendTransaction(tx);
  }
}

Thanks for the example! However, I generally side with not making breaking changes if the change doesn't bring massive advantages.
I understood your problem but it doesn't seem like it would be too hard to fix it:
const wallet = useAnchorWallet();
const provider = wallet
  ? new AnchorProvider(connection, wallet, opts)
  : { connection };
const program = new Program(idl, provider);

I'm not exactly sure how much breakage https://github.com/coral-xyz/anchor/pull/2838 causes, which is why I'm hesitant. I also prefer optional ? fields over explicit null in JS/TS in general.


⚠ Bofrost.es has degraded performance

In 6074fab, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1605 ms

Resolved: Bofrost.es performance has improved in c6e19c4 after 6 minutes.


⚠ Bofrost.si has degraded performance

In c85844b, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 5382 ms

Resolved: Bofrost.si performance has improved in 3dbabbc after 10 minutes.


[BEAM-11775]:Move org.apache.beam.runners.spark.aggregators.metrics.sink.InMemoryMetrics class out of test jar

Currently, org.apache.beam.runners.spark.aggregators.metrics.sink.InMemoryMetrics class is in test jar, but InMemoryMetrics is needed when doing unit tests and local runs for spark runner. So move it out of test jar to support unit tests and local runs.
Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

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

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @xinyuiscool

Run Java PreCommit

Run Java PreCommit

Run Java PreCommit

You'll need to check the couple build issues:
https://scans.gradle.com/s/azabfuw37q6os/console-log?task=:runners:spark:checkstyleMain
[ant:checkstyle] [ERROR] /home/jenkins/jenkins-slave/workspace/beam_PreCommit_Java_Phrase/src/runners/spark/src/main/java/org/apache/beam/runners/spark/aggregators/metrics/sink/InMemoryMetrics.java:1: Missing package-info.java file. [JavadocPackage]

and also you may need to run ./gradlew :runners:spark:spotbugsMain to see what other spotbugs issues may have been found.
fyi @MabelYC

@MabelYC - What is the next step on this PR?

@MabelYC - What is the next step on this PR? Should we keep this open?

@MabelYC @xinyuiscool any updates? : )

@MabelYC @xinyuiscool any updates? : )

Sorry for the late reply, just got my github account recovered. Will try to fix this next week.


Replace Task.Result with await

Fixes #35694

@Potapy4 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@Potapy4 - Thanks so much for making these fixes!
#sign-off


Excel output file empty

Hi Ben,
I hope all is well! My sumstats Excel output file is empty. I can only see the headers in the first row (the names of the statistics I wanted to output). Any lead on what the issue could be? Here is my line of code:
sumstats v101 using "$dhs_stats\stats.xlsx", replace stats(count mean sd max min)
Thanks a lot!

It now works!


Ansible Operator Should use status writer

per @hasbro17 comment on PR #473

Don't need to change this now but it might be worth looking into the Status client to see if there's a benefit to using that if we're just updating the status:
kubernetes-sigs/controller-runtime:pkg/client/unstructured_client.go@master#L136-L151
The default manager client comes with a status writer:
kubernetes-sigs/controller-runtime:pkg/client/split.go@master#L32

Moving this to blocked. We want the ansible operator to support k8s 1.10 which means that the status subresource is not on by default.

Closed by #787


Move repository to asciimath organization

This came already up before, but we didn't really make a decision…
I think we should move it to focus all repositories in one place!

I think we should finally tackle this issue =P

It's fine with me, if someone knows how to do it :)

I have the necessary rights but I was waiting for confirmation from @drlippman and @jipsen .

It's fine with me. (I hope this is unrelated to the AsciiMath inputjax for MathJax since the inclusion of AsciiMath in MathJax has been very useful :)

@jipsen thanks. Yes, it's just an organizational change on GitHub.

Done.

:+1:


Implementing automatic calling of commonProcess().

This change allows it to configure the component to automatically trigger commonProcess() in the beforeFilter. I've set the default to false by default but I think it should be in fact set to true because I don't see any bad side effects it could have. Also I've made it so that instead of just activating it you can specify an action it should be triggered on and pass both params to the commmonProcess().

Thanks :+1:


Global Asset Library

Been learning about Godot for about a week, and what seems like somewhat of a glaring issue to me is having to recreate basic assets for every new project. Or having to navigate to other projects, search through, pick out what you need (along with dependencies), and drag into the editor or copy and paste to your current project. Either way it's a hassle.
The first solution that comes to my mind is to have a global asset library you could drop all your materials, shaders, meshlibs, textures, etc into, and make them accessible from any project.
I think it's also important that, if something in the editor appears to be a single object, we have the ability to treat it like a single object everywhere, without having to worry about its parts or dependencies.

I believe you can achieve virtually the same effect by making your own folder (say, on your desktop), which you copy those sorts of things to. Then when you want to add them to a project, you just copy them to that project. This isn't a bad idea, mind you, just... not something I think you'd need the editor to be able to do.

How about just a way to link to folders outside of res:// ? I definitely see an advantage in not propagating copies of the same stuff.
I tried to put a symlink in res:// and while Godot is able to scan and import images from the linked folder, it doesn't show in the filesystem. Didn't test export, but there may be problems with that too. Maybe just making sure Godot plays nice with links would be enough for this feature proposal?

Symlinks don't work on windows very well.
Linking folders outside of res.. There was a brief design to rewritethe plugin system to include an entire project as an addon. The addon will include the entire library. This will avoid symlinks. Not sure if the entire design was completed.

A lot of this discussion has been going on in #19846 (that one is a hotpot of several different related discussions, so it's not quite as focused as the discussion going on here). Also relevant is this comment from #19506 (apparently there are plans to make the Asset Library host more individual classes in the future, when the API and plugin tooling is more flexible/powerful).

@willnationsdev Looks like you referenced wrong issue.

@KoBeWi Fixed, thanks.

Some info about git and handling symlinks on windows using a post checkout hook:
https://stackoverflow.com/questions/5917249/git-symlinks-in-windows

If you mean locally, then a folder does the trick.
If you mean being able to use other people assets, how is this different from AssetLib? Sure it's in need of serious TLC, but still.

try this -- its still a WIP but I use it for all my projects
https://gitlab.com/godot-stuff/gs-project-manager/wikis/home

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


docs(manifest.md): moved switch clauses to enhanced yaml syntax

It seems like the switch clause section has to be in the Enhanced YAML syntax section, not above.

LGTM! Thanks for the small fix!


Implement installation of RKE2 through direct download on vanilla OS images

Describe the solution you'd like:
The project is now designed to respond to a wider variety of use cases, including Air-gapped environments. Because of that, it supposes that:

Access to internet (more specifically Github.com and docker.io) is unavailable.
All pre-requisites for RKE2 are available on the OS image referenced in the infrastructure provider's MachineTemplate CR.

Why do you want this feature:
While the solution above is good enough and has the benefit to always work, it requires some involvement of the final user (selecting the right OS image in the MachineTemplate) and might in the long run be a limiter to the provider adoption. Therefore, it would be nice if the provider would also work with the most common use case:

Access to internet is available
User wants to use a vanilla OS image, like ubuntu cloudimg images or OpenSUSE Leap OpenStack images.

Anything else you would like to add:
This issue is to track a long term need and can be deprioritized in the beginning of the project.

Fixed in #60


feat(istream): implement OSC input stream

I've tested this with my iPhone running GryOSC.

The Mac OS build failed because libgrt.dylib. Could be fixed by the method used in #32.


notifications on ignored phrases

YO MOTHA SUCKA. I GOT SLURP AS A MUTED PHRASE, AND WHEN PEOPLE TAG ME WITH SLURP IT DONT SHOW IN CHAT, BUT IT SHOW IN NOTIF.
GAME BREAKING BUG

amongus


auto_name doesn't work with addition

Model:
@pm.model(auto_name=True)
def model():
    ....
    # The offending line is this one.
    nu = pm.Exponential(lam=1 / 29.0) + 1
    ...

Error message:
________________________________________________________________________ ERROR collecting pymc4/tests/test_hierarchical_comparison.py ________________________________________________________________________
pymc4/tests/test_hierarchical_comparison.py:41: in <module>
    model = model.configure()
pymc4/_model.py:64: in configure
    model._evaluate()
pymc4/_model.py:80: in _evaluate
    self._template._func(*args, **kwargs)
pymc4/tests/test_hierarchical_comparison.py:33: in model
    nu = pm.Exponential(lam=1 / 29.0) + 1
pymc4/random_variables/random_variable.py:134: in __init__
    raise ValueError("No name was set. Supply one via the name kwarg.")
E   ValueError: No name was set. Supply one via the name kwarg.
!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! Interrupted: 1 errors during collection !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
=================================================================================== 52 deselected, 1 error in 1.79 seconds ===================================================================================

What is the error message you got?

Updated with error message.

Actually this is OK, we could add this as a special case to the AST parsing but it's not meant to support every case and the error message is informative.

@twiecki do we have a plan for auto-naming? From @ericmjl's snippet it looks like we're going for an auto_name flag that uses the AST to parse (nice!). If that's, I can spend some cycles and powow on this issue.
I'm a bit confused at how the auto_name flag works, though: I don't actually see it in the codebase anywhere, except in example notebooks. Which .py file is the relevant one?

@eigenfoo That'd be great! Here is the code that walks the AST: https://github.com/pymc-devs/pymc4/blob/rugby_example/pymc4/ast_compiler.py#L117 and here is where it's used: https://github.com/pymc-devs/pymc4/blob/rugby_example/pymc4/_model.py#L30
Note that I think we should do things differently than I had originally implemented. I parsed and changed the AST to insert the name kwarg into the code. This has several downsides, one of them is that it's brittle and also that it doesn't work with @function compilation. Instead, it would be much simpler to just parse the AST to identify all yields, retrieve the rhs var names and store them in a list. Then when we do the interception in control-flow we change the default RV names to those from the stored list. Obviously this doesn't allow for dynamic graphs but that's a fine limitation. Some care needs to be taken care with model nesting but I think if we have that logic in the @model decorator we should be able to get a clean implementation.


infrastructure-playbooks: playbooks for BI partitioning on NVMe

Signed-off-by: Ali Maredia<EMAIL_ADDRESS>

There are a small handful of things I still need to do:

write up a README
remove makepartitions.sh and ansible-ize it
go over the logfile output
clean up vars/partition_vars.yaml
generalize the interface more to accommodate more devices
TBD

@andrewschoen @alfredodeza all reviews addressed, ready for another look.

@leseb could you expand how exactly do you envision this to be added to a scenario? For example, it wouldn't need a cluster, it isn't deploying Ceph, it can't be tested using the current conventions like testinfra. I don't see this getting updates in the near future at all, wouldn't some manual testing suffice for now?

@leseb @neha-ojha I'll take a quick look to see if I can get a test scenario added here. If I can get it added to the branch we can run it through jenkins even if it's not hooked into the PR testing.

thanks @andrewschoen

I've added some commits for a new testing scenario. I had to change a few things in the playbook to allow for it. The scenario is passing locally, but I was having a bit of a struggle getting it to work in the CI. I'll work on that a bit more later.

I was incorrect, the copy module does perform variable interpolation. I've included a commit to switch back to using it instead of template so we can avoid storing a template file in source.
I've rerun the scenario in the CI, you can see at the end of the log that the variables are expanded correctly: https://2.jenkins.ceph.com/job/ceph-ansible-scenario/309/console
Suggested cut and paste under "lvm_volumes:" in "group_vars/osds.yml" for configuring with osd_scenario=lvm
-----------------------------------------------------------------------------------------------------------
  - data: ceph-bucket-index-1
    data_vg: ceph-nvme-vg-sdb
    journal: ceph-journal-bucket-index-1-sdb
    journal_vg: ceph-nvme-vg-sdb
  - data: ceph-hdd-lv-sdc
    data_vg: ceph-hdd-vg-sdc
    journal: ceph-journal-sdc
    journal_vg: ceph-nvme-vg-sdb

@leseb by all capital letters you mean anything and everything? For example, is is common to use LV and PV vs. lv and pv. Since they are abbreviations (like SSD and HDD) it is usual to see them used in this way

@alimaredia yes, even abbreviations. Thanks.

@alfredodeza thanks!

jenkins test pipeline

jenkins test pipeline


Update dependency docker/cli to v20.10.18

Update DOCKER_VERSION from v20.10.12 to v20.10.18.
Fixes #1800

@duhow , could you please share actions-runner  image tag with this change.


Reduce memory use in transeq for both backends

Closes #40.

Anything else here @Nanoseb @pbartholomew08?

Nope, seems ready to go to me


Question: Recording .bag frame-by-frame

Required Info

Camera Model
D400

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win 10

Kernel Version (Linux Only)

Platform
PC

SDK Version
2.22.0

Language
python

Segment
{Robot/Smartphone/VR/AR/others }

Question:
I want to synchronize the streams from an intel D415 camera and a ZED camera. I know how to record a stream into a .bag file and play it back, but that does not guarantee that both cameras are synced. Any ideas?
The ZED API allows me to record frames into an .svo file in a "frame-by-frame" manner.
However, I don't know if it is possible to do the same for a .bag file.
One solution would be to use wait.for.frames and then retrieving the streams from the camera and saving them manually into .tif files, but I'm afraid that would not work for the desired framerate (25+).

Data from RealSense and non-RealSense cameras can be aligned using camera metadata.  the link below discusses alignment between a RealSense and a ZED camera.
https://github.com/IntelRealSense/librealsense/issues/2186

Thanks, I'll close this issue.


Why is Try delegate returning Exceptional?

Hi, I don’t get why Try delegate is returning Exceptional if it is Run()ks purpose to wrap the result of the lambda call. Invoking „raw” Try delegate can still throw exception but return type of Exceptinal suggests that ir will not happen or am I completly off here?

You're not completely off... ideally, one would like to "override" function invocation (what you call "raw" invocation). Since this is not possible, I defined Run(). "Raw" invocation of course completely defeats the purpose of Try, but again there's no way to prevent it.

so what do you think about making Try a struct wrapper around a Func<> disabling possibility to bypass Run()?

Yes, that would be possible, but it would introduce syntactic noise because you'd always have to instantiate the struct. For example, instead of
 Try<Uri> CreateUri(string uri) => () => new Uri(uri);

you'd have to write
 Try<Uri> CreateUri(string uri) => new Try(() => new Uri(uri));

This is because you cannot implicitly convert a lambda to a struct.


[v2] Allow direct input of popular credential fields in v2

from prior suggestion:

what if we kept shortcuts for the v1 fields (i.e., --username, --password, etc... are still available, they just auto-map to the inputs dict), and for custom types, you just have to pass in JSON?

Keep the simple use-cases simple. Let the complex things be possible.

this should support directly pointing it at the ssh key file ie --inputs'{"ssh_key_data": ~/.ssh/id_rsa}' rather than expecting the actual key.
This needs fixing asap.

Feel free to have a stab at it. The thinking here is to have options like --ssh-key-data=..., then you could accept files. The syntax pattern needs consideration, I think that older versions like 3.1.8 would consider ... to be a file.
This would require a translation layer to be added, and I think we have a good example with unified_job_template_options. This decorator could implement options ssh_key_data and inputs together, and combine them.
https://github.com/ansible/tower-cli/blob/master/tower_cli/utils/resource_decorators.py#L22
But this is different from the problem that $(cat ~/.ssh/id_rsa) won't really work - that's because the line breaks are removed. tower-cli could have a conditional to convert spaces into line breaks, but that kind of highly-automatic behavior can have surprising consequences.

Alan,can you explain why the user param is still required ? given I assume this is the user that ssh will login as ? there’s no other user associated with the cred in awx.
tower-cli credential create --name="SSH example" --user=$userval --inputs="$machine_cred_inputs" --credential-type=“Machine"

On 13 Dec 2017, at 15:29, Alan Rominger<EMAIL_ADDRESS>wrote:
Feel free to have a stab at it. The thinking here is to have options like --ssh-key-data=..., then you could accept files. The syntax pattern needs consideration, I think that older versions like 3.1.8 would consider ... to be a file.
This would require a translation layer to be added, and I think we have a good example with unified_job_template_options. This decorator could implement options ssh_key_data and inputs together, and combine them.
https://github.com/ansible/tower-cli/blob/master/tower_cli/utils/resource_decorators.py#L22 https://github.com/ansible/tower-cli/blob/master/tower_cli/utils/resource_decorators.py#L22
But this is different from the problem that $(cat ~/.ssh/id_rsa) won't really work - that's because the line breaks are removed. tower-cli could have a conditional to convert spaces into line breaks, but that kind of highly-automatic behavior can have surprising consequences.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub https://github.com/ansible/tower-cli/issues/356#issuecomment-351425279, or mute the thread https://github.com/notifications/unsubscribe-auth/ATZJnNFG5l2TcW3vxvJ1ei4htf2YncNXks5s_-1GgaJpZM4PcvN-.

I believe --user is for the owner of the credential in AWX. It's not required in the code I have here, that might have been changed, I don't remember.

Ok
Sent from my iPhone

On 13 Dec 2017, at 16:38, Alan Rominger<EMAIL_ADDRESS>wrote:
I believe --user is for the owner of the credential in AWX. It's not required in the code I have here, that might have been changed, I don't remember.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.


Azure Cosmos DB is tracked only if HTTP is used.

Modify documentation to say Azure Cosmos DB is tracked automatially only if http mode is used. TCP mode wont be captured by application insights.

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

ID: c4ded2ce-d8c3-7118-bec9-865f6225dbe0
Version Independent ID: a4089752-177c-866c-7237-3ac02c7750d9
Content: Dependency Tracking in Azure Application Insights
Content Source: articles/application-insights/app-insights-asp-net-dependencies.md
Service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@cijothomas Thank you for the feedback!  We will assign this to the document author so they can update the document accordingly.

@cijothomas can you confirm the terminology you want in the update to the doc? I have no expertise in CosmosDB but I am only seeing a "Gateway Mode" and a "Direct Mode". Gateway mode defaults to HTTPS, are you saying they need to use Gateway mode and they need swap from HTTPS to HTTP, or just that they need to use Gateway mode with HTTPS and that they can't use Direct Mode which can use HTTPS or TCP?

I haven't tried CosmosDB myself:( I think its best to just say that AI tracks dependencies only if HTTP/HTTPS is used and not for TCP, and perhaps a link to CosmosDB doc. (or just call out that Cosmos DB calls made using TCP is NOT tracked automatically by app insights)


Make it possible to add annotations or labels to OONI Run link lists

It would be nice if people creating OONI Run links could add an annotation to the measurements so that it's possible to search by them in the API and we can mine these lists to feed into the test-lists.
This was suggested by @pellaeon

Done months ago.
Ref:
https://github.com/ooni/probe-ios/blob/master/ooniprobe/Test/Test/AbstractTest.m#L60
https://github.com/ooni/probe-android/blob/master/app/src/main/java/org/openobservatory/ooniprobe/activity/OoniRunActivity.java#L195

Reopening as it might be needed for desktop

I guess we also need support for generating these sorts of links inside of the OONI Run web interface.
@lorenzoPrimi can you share an example OONI Run link with a custom annotation?

I added the code in the previous comment. We add a "ooni-run" annotation in every test that is run with OONI Run

Ah ok, that's good, yet this issue is about adding support for adding custom annotations via OONI Run links. So that a user would go to run.ooni.io and say "I want to add my custom annotation called 'my_custom_name: 'bar'" and then the app knows how to interpret that.


Fix String comparison in Java

In Java, we cannot use == to compare two strings, it will give out false as long as they're two different string instances.
Replace == with String.equals() and != with !equals().
(Sorry for annoying PRs, I just want to make it work)

WWW sorry I forget to debug Java version haha


[Python] Error while experimenting with Socket.io

I'm trying to get data in Python over the socket.io-connection.
This is my code:
import asyncio
import socketio
sio = socketio.AsyncClient()
@sio.event
async def connect():
    print('connection established')
    await get_data()
async def get_data():
    await sio.emit("login", {"username": "test", "password": "testtest"})
async def main():
    await sio.connect("http://localhost:3001/")
    await sio.wait()
if __name__ == "__main__":
    asyncio.run(main())

The problem: In the Server-log, I get this error:
Trace: TypeError: callback is not a function
    at Socket.<anonymous> (/app/server/server.js:222:17)
    at process.<anonymous> (/app/server/server.js:973:13)
    at process.emit (events.js:400:28)
    at processPromiseRejections (internal/process/promises.js:245:33)
    at processTicksAndRejections (internal/process/task_queues.js:96:32)
If you keep encountering errors, please report to https://github.com/louislam/uptime-kuma/issues

Info
Uptime Kuma Version: 1.6.0
Using Docker?: Yes
OS: Docker
Browser: -

Just follow Uptime-Kuma's frontend js code, you are missing the last parameter callback.
https://github.com/louislam/uptime-kuma/blob/31b4a5c33e872395b22c121bdb2fcab4189bccaa/src/mixins/socket.js#L205-L219
And just a remind, it is not an API, the call will be changed in the future.

Ok, thank you very much for your quick help!

I asked for help, because I made this: https://github.com/mawoka-myblock/uptime-kuma-static


Support Zeebe user task

What should we do?
<bpmn:userTask id="Activity_1kl1v6a">
  <bpmn:extensionElements>
    <zeebe:formDefinition externalReference="customFormKey" />
    <zeebe:userTask />
  </bpmn:extensionElements>
</bpmn:userTask>

zeebe-bpmn-moddle

[ ] Add extension: <zeebe:userTask> (in User Task)
[ ] Add externalReference to <zeebe:formDefinition>

properties panel

[ ] I can set implementation of a user task as either "job worker" or "Zeebe user task"

[ ] "Zeebe user task" is displayed if element contains <zeebe:userTask> extension element
[ ] "Job worker" is displayed otherwise

[ ] When I change the implementation to "Job worker", the <zeebe:userTask> extension is removed
[ ] When "Zeebe user task" is selected, only “Camunda form (linked)” and “Custom form key” for form linking are allowed. Embedded forms are disallowed.
[ ] In "Zeebe user task", set custom form key value to attribute “externalReference” instead of “formKey” (https://github.com/camunda/zeebe/issues/16006)

linting

[ ] Error is displayed if <zeebe:userTask> is used prior to Camunda 8.5

Why should we do it?
Part of efforts: User Task lifecycle & listeners in Zeebe
Epic: https://github.com/camunda/product-hub/issues/1823

I clarified the task listeners, and we are not implementing this feature yet (cf. https://github.com/camunda/product-hub/issues/1823#issuecomment-1918589148).

@barmac, we'll call the corresponding property of user task records in Zeebe externalFormReference to have enough context in a user task, looking at the attribute.
Should we align this here and also go for that name in modeling or do you think it's fine to have a slight difference here since the externalReference is part of the form definition in the model already?

@tmetzke As far as I know we treated a form key not starting with camunda-form as an external reference in the past. How did we decide to change course?

Hey @nikku, thanks for asking, we missed to publicly clarify this anywhere, I guess. As described in the (internal) technical proposal, there will be 2 options in the Zeebe user tasks: linked Camunda forms (via formId) or custom forms (via externalReference). As job-based user tasks are meant to be deprecated in the future, those will be the two options moving forward.
Why the change?
There has been quite some confusion around the formKey in Camunda 8 the past:

We more or less took this over from how it's done in Camunda 7. However, since Zeebe uses a key-based terminology instead of Camunda 7's id-based one for unique identification of entities (e.g. processInstanceKey instead of processInstanceId), this led to name clashes and misunderstandings. For example, all key-based attributes in a job record can be used as Long value-based references to other entities, while the formKey can't. This is merely a String value, describing how to find the related form.
The value of the formKey attribute is quite cumbersome for users to decode. The specific way the form is linked can only be obtained through dedicated String parsing and pattern matching, analyzing potential prefixes in the value to exist. @christian-konrad also expressed his preference of a clear separation of linked forms and custom forms in modeling (internally) over here.

As a result, we (internally) decided to go for formId and externalReference for user task forms, moving forward. We intend to highlight this in the public documentation as well.
Since we're still in development here and in alpha-status, I'm absolutely open to discussing this if you see any hard requirements to do this differently. Sorry for not involving the modeling team more in this move. With @christian-konrad also pushing for this, I figured this was an obvious move to make (technically from the Zeebe end but also product-wise).

I will also add this reasoning to the public issue in Zeebe.

Does this mean we'll now have 4 options for our users to choose?

Technically it's 5 different options spread among 2 groups:

Job worker:

Embedded form
Linked form (formId)
External form (formKey)

Zeebe user task

Linked form (formId)
External form (externalReference)

Thanks @tmetzke. This rationale makes absolute sense as we have the opportunity to re-think things.

Since each of the features in the upstream libraries are merged, this is now fixed upstream.

This will complete the story: https://github.com/bpmn-io/bpmn-js-properties-panel/issues/1032


Add Kerbal Launch Failure, Add Antares & Cygnus, Update Zero-Point Inline Fairings

Didn't realize I closed this (I suck at git)

Based on how 0point fairings distinctly doesn't work in Stock aero I think we should probably list it as a depends rather than a recommends. what does @NecroBones think about it?

Yes, that would be fine with me. We're trying to come up with a solution for stock aero, but we can always remove the dependency later if that ever happens.

Took another look and it might be worth hardcoding the author field for AntaresCygnus since there are 2 authors rather than the one we scrape from KerbalStuff. The other author is Kartoffelkuchen according to the forum thread :)

#rebuild

Damn it, didn't mean to push to that branch. Need to figure out how to fix/revert that.

Is https://github.com/plague006/NetKAN/commit/0ba3ea0b8b3522c003ca084f041d1276bd7457a7 identical to #1483 ? In that case I guess we can merge this and it wouldn't be a problem?

Yes, I accidentally pushed that commit to this branch (when it was meant for #1485). I then reverted 1 commit too far back (which recreated the lack of quotes) and then added the quotes back.
So in the end this isn't touching epicsuits, #1485 did that, this is for Launch Fails, Antares, and Fairings.


tvm/relay/std/prelude.rly is missing after TVM installation

prelude.rly is missing after TVM installation using setup.py:
/usr/local/lib/python3.7/dist-packages/tvm-0.6.dev0-py3.7-linux-x86_64.egg/tvm/relay/std/prelude.rly

dup of https://github.com/dmlc/tvm/issues/4225

Move to #4225


Fix warning for Dataloader if num_workers = cpu count = 1

What does this PR do?
This PR checks if the num_workers of dataloader should be more than the assigned.
Previously even if the available num_worker was 1 and the value in the dataloader was given 1, there was a warning stating to give num_worker=1, which is wrong.

Fixes #19223

Before submitting

Was this discussed/agreed via a GitHub issue? (not for typos and docs) - Yes
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary) - Not Necessary
Did you write any new necessary tests? (not for typos and docs) - No
[x] Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request? - Yes
Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors) - No

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

📚 Documentation preview 📚: https://pytorch-lightning--19224.org.readthedocs.build/en/19224/

@awaelchli @carmocca any update here?
I think one of the tests has been failed due to timeout.
I tried to restart it but I think I dont have the permission to do so.

@dipta007 We will merge the PR as soon as we fix the failing docs build. Thanks for your understanding.


Audit fix - Prioritize set_max_schema_model_bytes higher

https://github.com/LibertyDSNP/frequency/issues/756
Goal
The goal of this PR is to fix an issue found during the audit
Closes #756

Lgtm 👍 ..
Only comment if you want to make that 30,000 value a const otherwise good

I am trying to keep this focused on the audit.  I think we should revise that upon further review of benchmarking which is underway via another issue.


Update more Docs

Closes #633
Clean comments
Clean docs
rework the version gathering system

PR checklist

[X] PR is made against dev branch
[X] This comment contains a description of changes (with reason)
[X] Ensure the test suite passes (./scripts/test.sh -p docker -t ALL).
[x] CHANGELOG.md is updated
[X] README.md is updated

👍 👏

So now, where can we find some beginners to help us with this USAGE.md docs?

I can give it a go. But it would need some careful proofreading for factual errors.


Move perf pipelines to TME subscription

Move perf to TME
Flip default for resource deployment and removal to true
Remove unnecessary sub-config files

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

Tested one perf pipeline https://dev.azure.com/azure-sdk/internal/_build/results?buildId=4252347&view=results

/check-enforcer evaluate


midterm Imran Turganov

I forgot to save when I added deriving(Show, Eq, Ord)

I forgot to save when I added deriving(Show, Eq)

Build failed.


"Sâncrai" is listed in the drop-down menu

Description:
"Sâncrai" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Sâncrai" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sâncrai, Alba (Aiud)".


The images blink upon loading

Description:
Whenever I click a library or return to home, the images blink for a second. Also the main poster of the movies and shows do ot appear at all. I am currently using Jellyfin 10.8.1 on MS Edge.
To Reproduce
Just tried the monochromic preset.
@import url('https://cdn.jsdelivr.net/gh/CTalvio/Ultrachromic/presets/monochromic_preset.css');
Video
https://user-images.githubusercontent.com/65329453/176852371-bcb8dca5-13c4-4e75-b1c1-471e1561647d.mp4

same, however i also experienced this on jellyfin 10.8.0 and 10.7.7 on chrome

you can add:
.cardImageContainer {transition: none !important;}
below in the custom css section as a temporary fix until the css is updated :)

alright


preserve-movie-data

In this PR, I;
Preserved all data by saving collections in .json files.

Yes, notice that I didn't actually put that in review on the kanban board,
which was actually on purpose.
On Thu, Jan 26, 2023 at 6:38 PM Sammuel LEKPOR @.***>
wrote:

@.**** approved this pull request.
Hi, @blase147 https://github.com/blase147 !!!
There is no change to review as all of it looks added to the development
branch already! You can merge now 🎉
—
Reply to this email directly, view it on GitHub
https://github.com/SelmaNdapanda/Personal-Catalog/pull/61#pullrequestreview-1271497488,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXPDXQNB6GAZ7EXMGZPPULDWUKZANANCNFSM6AAAAAAUHUXPIQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_0b18f7c6-786f-4873-9ecd-0d04d6d0c5bf/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_0b18f7c6-786f-4873-9ecd-0d04d6d0c5bf.git

Docs Build status updates of commit d36d954:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-GitBranchDeletedOrForcePushed] Cannot sync git repo to specified commit because branch Release_Archive_master_2021-06-01-03-43-59 has been deleted or has been force pushed

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 Docs 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 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 docs.microsoft.com contributor guidesPost your question in the Docs support channel


working with classes

this is just something i have found hard to efigure out right now. but working with classes.
so ill keep my code seperate, even in different folders. but this does build, it even shows the console working. but nothing shows.
obviously doing something wrong. but just need some help
main.hx
class Main extends hxd.App {
	private var _gameState:GameState;
	override function init() {
		_gameState = new GameState();
	}

	override function update(dt:Float) {
		_gameState.update(dt);
	}

	static function main() {
		new Main();
	}
}

gamestate.hx
import h2d.Layers;

import char.PlayerController;

class GameState extends Layers {
	private var _player:PlayerController;
	public function new() {
		super();
		_player = new PlayerController();
	}

	public function update(dt:Float) {
		_player.update(dt);
	}
}

player.hx
package char;

import h2d.Bitmap;
import hxd.Key in Keys;

class PlayerController extends Bitmap {
	private var _image:Bitmap;
	private var _speed:Float = 9.0;
  public function new() {
  	super();
    var tile = h2d.Tile.fromColor(0xff00f0, 32, 32);
  	_image = new Bitmap(tile);
    _image.x = 400;
    _image.y = 400;
  }

  public function update(dt:Float) {
  	if (Keys.isDown(Keys.LEFT)) {
  		this._image.x -= _speed;
  	} else if (Keys.isDown(Keys.RIGHT)) {
  		this._image.x += _speed;
  	}
  }
}

You need to add _image to the scene (your game.s2d)

sorry i only understand some of that
so it would be in gameState.hx addChild(_player);?

To fix, you could do: new GameState(s2d), then pass the parent to your other classes.
always create Bitmap this way :  new Bitmap(parent)


Add Kim protocol support

Background
The Kim plugin is being developed as part of the GOAT SDK to provide integration with Kim Protocol, a decentralized exchange protocol. The plugin enables interaction with Kim's core contracts including swapping tokens, managing liquidity positions, and querying pool states.
What does this PR do?
This PR implements a new Kim Protocol plugin for the GOAT SDK that provides:

Core Protocol Integration

Swap functionality (exactInput/exactOutput, single/multi-hop)
Liquidity position management (mint, burn, increase, decrease, collect)
Pool state queries and interactions

Technical Components

Contract ABIs for Router, Factory, Pool, and Position Manager
Type definitions and Zod validation schemas
EVM wallet client integration
Mode network support

What kind of change is this?

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

Documentation changes needed?
My changes do not require a change to the project documentation.
Testing
Detailed testing steps

Initialize Kim Plugin with the appropriate contract addresses:

const kimPlugin = new KimPlugin({
  addresses: {
    swapRouter: "0x...",
    quoterV2: "0x...",
    factory: "0x...",
    positionManager: "0x..."
  }
});

Test basic token swaps:

\\ test exactInput swap
await kimPlugin.exactInput({
  path: {
    tokenIn: "0x...",
    tokenOut: "0x...",
    intermediateTokens: [],
    fees: [3000]
  },
  recipient: "0x...",
  deadline: Date.now() + 1800,
  amountIn: "1000000",
  amountOutMinimum: "900000"
});

Test liquidity position management:

// Test position creation
await kimPlugin.mint({
  token0: "0x...",
  token1: "0x...",
  tickLower: -1200,
  tickUpper: 1200,
  amount0Desired: "1000000",
  amount1Desired: "1000000",
  amount0Min: "900000",
  amount1Min: "900000",
  recipient: "0x...",
  deadline: Date.now() + 1800
});

For plugins

[ ] I have tested this change locally with key pair wallets
[ ] I have tested this change locally with hosted wallets (e.g. Crossmint Smart Wallets, etc.)

Discord username
sunosuporno

@sunosuporno build is failing and lint also but I'll take care of fixing this one since we might use it for the demo later today :)
For the other PRs:

Checkout the package.json file of other plugins and follow the same approach, version should start at 0.1.0, also see how the test script works.
Make sure you run pnpm lint before pushing (this is temporary since we will automate that for future PRs so you won't need to remember it)

Thank you for this!!


BREAKING CHANGE: xWebAdministration renamed to WebAdministrationDsc: Renamed all DSCResources, Examples, Modules and Tests where applicable

Pull Request (PR) description
xWebAdministration renamed to WebAdministrationDsc
This Pull Request (PR) fixes the following issues

Fixes #213
Fixes #369
Fixes #363

Task list

[x] Added an entry to the change log under the Unreleased section of the
file CHANGELOG.md. Entry should say what was changed and how that
affects users (if applicable), and reference the issue being resolved
(if applicable).
[x] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[x] Examples appropriately added/updated.
[x] Unit tests added/updated. See DSC Community Testing Guidelines.
[x] Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
[x] New/changed code adheres to DSC Community Style Guidelines.

This change is

I'm pushing new pipelines files in PR https://github.com/dsccommunity/xWebAdministration/pull/603. That PR passes integration tests. I will switch to the same build workers as in this PR inb another PR just to rule build workers out.

So I have switched to the new build workers and the pipeline stil works. I pushed a few files to get wiki documentation updated (README.md file in each resource folder), they need renaming too now.

If you rebase this PR now, let us see what fails and work from there, if something fails then now we can be certain that it is the code changes that we need to look into, not the build workers having issues. 🙂

@nickgw it seems this row fails in the integration tests:
https://github.com/dsccommunity/xWebAdministration/blob/df669ce662ff3e03ad1268498086204f34cb8717/tests/Integration/DSC_WebApplication.Integration.Tests.ps1#L102
I'm think, can it be that we run the integration tests to fast. Can we test adding Start-Sleep -Seconds 5 before running the command? If that doesn't work I suggest comment that test, the create an issue to track it and we move on. Let us in that case fix it in another PR. What do you think?

It is like the Get-TargetResource function throws an error or returns some unexpected value that Get-TargetResource does not like. 🤔

Now it fails with the previous error again. Looking at the tests thay run for xWeb I see the do run in a different order now... It might be something a previous test added that made the test, by luck, work previously. 🤔

I think the integration test need to be refactored, like you tried too. But we can do that in another PR. I happy to comment put the test and make an issue for it to fix, by someone, in another PR.

It is like the Get-TargetResource function throws an error or returns some unexpected value that Get-DscConfiguration does not like. 🤔

I'm creating an issue now. It baffles me that this worked in your PR, but is failing in mine.

I think the integration test need to be refactored, like you tried too. But we can do that in another PR. I happy to comment out the test and make an issue for it to be fixed, by someone, in another PR.

Created this for the failing integration test: https://github.com/dsccommunity/xWebAdministration/issues/606

Awesome work @nickgw! This PR is ready, I will rename this repo as soon as I have a chance.

Waiting for the API key to be update on the pipeline so we can merge this.


Python: Add experimental default client

Extracted from the Zenith prototype to merge in main as a standalone feature.
This change adds a default Dagger client, globally available, so there’s no more need to make a connection and pass the client around. Every method under client. is available in dagger.. This requires a new dagger.connection() (notice the lowercase name) which doesn’t return a client instance, but instead initializes a globally available client:
- async with dagger.Connection() as client:
+ async with dagger.connection():
-     secret = client.set_secret("token", os.getenv("API_TOKEN"))
+     secret = dagger.set_secret("token", os.getenv("API_TOKEN"))
      ...

It’s simpler to create reusable functions without passing client everywhere, and if they only use lazy methods, it’s now possible to use them even before having a connection established:
import anyio
import dagger

async def main():
    # This is lazy so there's no need to get a connection first
    # because we no longer depend on a client instance.
    ctr = base().with_exec(["python", "-V"])

    async with dagger.connection():
        # Only need a connection if we actually need to execute something.
        version = await ctr.stdout()

    # Connection is closed when leaving the connection scope, same as before.
    print(version)

def base():
    # Simpler reusability.
    return dagger.container().from_("python:3.11-alpine")

anyio.run(main)

There’s also a way to automatically establish the connection on first use (lazy) but the explicit version is better when debugging.
See reference docs.
Compatibility
There should be no impact to the user. This change is backwards compatible. The new default client is opt-in and marked experimental in the reference docs.

Should we also implement this logic on Go and Node SDK?

Should we also implement this logic on Go and Node SDK?

No need to do it right now. In Go it'll be available as part of Zenith and Node.js can do the same, I just like it so much that I backported it to be compatible and usable today (i.e., without Zenith). If there's demand for it we can do the same for the others, but there's other priorities and we'll want to shift usage to Zenith as soon as it's ready anyway.


Some Claim() transactions are failing

The owner of this address: 0x19992115F80c686F56c1365868060Cbea8AA7a41
has tried six times to redeem his SOV tokens by the method 0x4e71d92d, to the contract 0xE0f5BF8D0C58d9C8A078Db75a9d379E6cdF3149e, but all the atempts has failed: https://explorer.rsk.co/address/0x19992115f80c686f56c1365868060cbea8aa7a41
among the transactions some was tried with high gas limit: https://explorer.rsk.co/tx/0x8f7be9cd869531f03ff572ff4060640d93ceb0f3d1429b439c6cffb85d4196ee
with the same reult.
The address 0x19992115F80c686F56c1365868060Cbea8AA7a41 is among the list of SOV owners from Origin:
https://github.com/DistributedCollective/Sovryn-smart-contracts/blob/3d3265191e4056c80a46fffe69ae6cea481f827c/scripts/deployment/origin_claim_list.csv
(line 470)

Other addresses with the same known issue:
0x974a29d4b83273dd7e6c19ab267534902c75477a
0xb147bFdCCAf67A75e468ee7F99497b7D284c8ad6

The reason of the errors:
This user already had Vesting contract in VestingRegistry2 when tried to claim
VestingRegistry2.getVesting("0x19992115F80c686F56c1365868060Cbea8AA7a41") = "0x5E827ba538e240798E15b7D5642469Fa048B38A1"
OriginInvestorsClaim.claim() vestingContractAddress = vestingRegistry.getVesting(msg.sender);         require(vestingContractAddress == address(0), "OriginInvestorsClaim::withdraw: the claimer has an active vesting contract");
This user can claim SOV tokens now, because they will be transferred to his\her account directly without any Vesting contracts.


Second Admin loses access to the commissionee after Reboot

Problem

Run all-clusters-app in a shell
Commission the app using the chip-tool cli
Open the commissioning window (sample command below)

chip-tool pairing  open-commissioning-window <node-id> 1 600 1000 2280

Copy the manual code from the command above and paste it into the QRCode scanner screen on iOS CHIPTool.
Verify iOS CHIPTool was able to commission the accessory (try toggle or enumeration...)
Restart iOS CHIPTool.
Verify that control still works (toggle/enumeration) - FAIL

After restarting the iOS application, the accessory denies access control to the iOS CHIPTool controller.
[1646267292692] [59542:42704697] CHIP: [EM] Handling via exchange: 23826r, Delegate: 0x1033633c0
[1646267292692] [59542:42704697] CHIP: [DMG] InvokeRequestMessage =
[1646267292692] [59542:42704697] CHIP: [DMG] {
[1646267292692] [59542:42704697] CHIP: [DMG] 	suppressResponse = false,
[1646267292692] [59542:42704697] CHIP: [DMG] 	timedRequest = false,
[1646267292692] [59542:42704697] CHIP: [DMG] 	InvokeRequests =
[1646267292692] [59542:42704697] CHIP: [DMG] 	[
[1646267292692] [59542:42704697] CHIP: [DMG] 		CommandDataIB =
[1646267292692] [59542:42704697] CHIP: [DMG] 		{
[1646267292692] [59542:42704697] CHIP: [DMG] 			CommandPathIB =
[1646267292692] [59542:42704697] CHIP: [DMG] 			{
[1646267292692] [59542:42704697] CHIP: [DMG] 				EndpointId = 0x1,
[1646267292692] [59542:42704697] CHIP: [DMG] 				ClusterId = 0x6,
[1646267292692] [59542:42704697] CHIP: [DMG] 				CommandId = 0x2,
[1646267292692] [59542:42704697] CHIP: [DMG] 			},
[1646267292692] [59542:42704697] CHIP: [DMG]
[1646267292692] [59542:42704697] CHIP: [DMG] 			CommandData =
[1646267292692] [59542:42704697] CHIP: [DMG] 			{
[1646267292692] [59542:42704697] CHIP: [DMG] 			},
[1646267292692] [59542:42704697] CHIP: [DMG] 		},
[1646267292692] [59542:42704697] CHIP: [DMG]
[1646267292692] [59542:42704697] CHIP: [DMG] 	],
[1646267292692] [59542:42704697] CHIP: [DMG]
[1646267292692] [59542:42704697] CHIP: [DMG] 	InteractionModelRevision = 1
[1646267292692] [59542:42704697] CHIP: [DMG] },
[1646267292692] [59542:42704697] CHIP: [DMG] AccessControl: checking f=3 a=c s=0xC9920843139F706A t= c=0x0000_0006 e=1 p=o
[1646267292692] [59542:42704697] CHIP: [DMG] AccessControl: denied

This is seen with a single admin case as well:

Pair M5 board via iOS chiptool
Verify able to control M5 board eg. toggle light
Force close iOS chiptool
Relaunch iOS chiptool - toggling light fails with AccessControl: denied

This is a regression from https://github.com/project-chip/connectedhomeip/pull/15729
When we normally bring up the Matter stack on iOS chip-tool, we do this:
src/darwin/CHIPTool/CHIPTool/Framework Helpers/DefaultsUtils.m:        [controller startup:storage vendorId:0 nocSigner:nil];

where storage is a CHIPToolPersistentStorageDelegate instance.
The restart added in #15729, though, does:
src/darwin/CHIPTool/CHIPTool/View Controllers/QRCode/QRCodeViewController.m:    [self.chipController startup:nil vendorId:0 nocSigner:nil];

and then we end up using a NSUserDefaults as the storage.  The different storages can and do end up with different node ids.
So now the order of events is:

Start iOS chiptool.
Go into the "QRCode scanner" screen.  Brings up Matter stack, we're using the CHIPToolPersistentStorageDelegate as storage, have node id X.
Tap the camera icon.  This does the stack restart, comes up with the other storage, node id becomes Y.
We commission while we are node id Y and install the ACLs for that.
Restart the iOS chiptool.  Try to do something that brings up the Matter stack.  It comes up with node id X again, and we fail ACL checks.


Mariadb fails to start

The following error occurs when trying to run this image:
2016-05-06 05:45:58 7fb318858da8  InnoDB: Operating system error number 13 in a file operation.
InnoDB: The error means mysqld does not have the access rights to InnoDB: the directory.

If you chown the var/lib/mysql then it works fine:
$/> chown -R mysql:mysql /var/lib/mysql

This begs the question: should we make mysql run as app:app?  I think that it is not necessary.

Me and @tharna discussed about this on Wednesday and actually we need to use app:app only on php, nginx and possibly drush images/containers. Mysql, varnish and memcached can be run with the default user:group. Do you agree?

agreed on user groups

original issue still needs solving


Fix images' tag

Since the workflow of building docker images doesn't update tag latest, use a fixed tag name may be better...