Add a device compat flag to allow Basic Set value mapping

Is your feature request related to a problem? Please describe.
In short, since zwave-js v7.0.1, the Vision Security ZG8101 Tilt Sensor is no longer functional at all in HA (and in zwavejs2mqtt does not function the same way).
In more detail, this device doesn't work the way it used to prior versions: the Binary Sensor CC value never changes. I am sure this is due to PR #2152, as the device sends a Basic Set 0 when closed, and Basic Set 255 when open. Those previously mapped to the Binary Sensor CC (device generic type Binary Sensor), resulting in false/true values. With the referenced PR, the Sets have no effect.
I presume this device does not follow the Z-Wave specification, from what I see a Basic Set command is not allowed to be mapped to a Binary Sensor CC. However, the past behavior worked perfectly, both in ZWave-JS and OZW. The binary sensor value is very easy to use in Home Assistant without any workarounds.
Describe the solution you'd like
A device compat flag called treatBasicSetAsReport, or something similar, that enables the previous behavior of mapped values for Basic Set commands. For this device, the Basic Set would map to Binary Sensor CC. Coincidentally, this is something OZW supports. Its default behavior is to not map Basic Sets, only Reports, and this compat flag allows the desired behavior. So looking for something very similar.
Describe alternatives you've considered
I could use the Basic value in HA, however there appears to be some new bug (related?) in the zwave-js-server and it doesn't even expose the Basic currentValue value anymore (it did previously). So I can't even use that right now.
If that did work, I'd still have to make a template sensor to bring back the previous behavior. Not the end of the world, but it would be more convenient to have the binary sensor working by default. I imagine most users expect this device to work this way, as it always has across different software.
This device also supports Notification V2 values, but they are buggy as well (surprise). The values are permanently set to non-zero values (tilt open sets the same value as tilt closed), and will auto-idle to 0 after 5 mins (by zwave-js). For a garage door tilt sensor this behavior is useless, the state will nearly always be wrong.
Additional context
The docs still indicate that basic mapping is the default: https://zwave-js.github.io/node-zwave-js/#/config-files/file-format?id=disablebasicmapping. I assume this must only apply to Basic Reports now. Would it be helpful to update the docs and be more specific (Basic Report) due to the change in behavior with 7.0.1?
Cache files and log (including re-interview) for completeness. The Tilt Sensor is Node 23.
zwavejs2mqtt-store.zip

In addition to the Nortek WAPIRZ-1 device mentioned by @kevingeorge, this issue seems to apply to the Nortek device WADWAZ-1 door/window sensor, and most likely the Vision and Monoprice variants of the same device.

The fix works perfectly, thank you! No re-interview was necessary, for anyone else needing this.

kpine- I am not clear what the fix is/was.  Is there something I need to do?

If the device really needs the flag (I'd need the device manual, device id and a log to confirm), the flag needs to be added to that device's configuration file.

If the device really needs the flag (I'd need the device manual, device id and a log to confirm), the flag needs to be added to that device's configuration file.

Thanks- I don't think I edit the device's configuration file in Home Assistant using Z-Wave js.  It may be something they add later.

I can do it but I need the stuff I asked for above as verification

If the device really needs the flag (I'd need the device manual, device id and a log to confirm), the flag needs to be added to that device's configuration file.
I can provide a log, device id and manuals, i'm using just the standard zwave-js integration in HA.  Can you let me know which log you need?

The one node-zwave-js writes. Should start with a large ascii art of "Z-WAVE"

My apologies it looks like I spoke too soon on the logs point; I'm having a hard time figuring out how to get the whole log file out of zwave-js in HA supervised.
The devices are: 0x014f/wadwaz-1.json and 0x014f/wapirz-1.json
I've included the manuals.
WADWAZ-1 WAPIRZ-1 manuals.zip

I guess I'll need that log and your cache file after all. One of your manuals includes this table, which looks like the device is supposed to send Notification CC Reports:

Best if you ask the HA community how you can get these - I don't know.

Here's the log.  Node 036 is the WADWAZ-1 and Node 007 is the WAPIRZ-1
zwavejs_359.log

And I think this is the other file you were asking for.
e4440334.zip


Automation for periodic verification of user access

Describe the issue
It is a security best practice to periodically verify user access to systems and ensure that all users that have access are users that should have access. Automation to contact Product Owners (PO) and verify the list of users that have access to their environments will increase our security.
Additional context

Would require integration with the SSO system
The SSO system is about to be replaced with the same kind but configured in different ways.  In particular, the user fields might be changed.

Definition of done

Automation that sends and email to all POs with the list of users for each project set that they own
Email should request that POs notify the CPF team with any updates required to the user access on their project sets.

No longer relevant since we are migrating to Azure OIDC only.


DBZ-2047: Add 'flush table' alias to MySQL grammar

According to MySQL docs - FLUSH TABLE is an alias for FLUSH TABLES. This patch adds that alias together with some test-cases.

Rebased and applied. Thanks, @andersenleo!
Could you also create an issue + ideally PR for updating the upstream grammar in the Antlr project, just like this one from an earlier fix we did to the grammar: https://github.com/antlr/grammars-v4/pull/1627. We have our own copy for build reasons mostly, but try to keep both in sync. Thanks again!


Typo

Additional members might include interface data and pointers to additional structures or routines that anther driver can call.

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

ID: 7adb0e0b-0ab1-94b6-c643-286348588433
Version Independent ID: b02d2738-9f80-b91b-b362-aa47b4757eac
Content: Using Driver-Defined Interfaces - Windows drivers
Content Source: windows-driver-docs-pr/wdf/using-driver-defined-interfaces.md
Product: windows-hardware
Technology: wdf
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Thanks @mcspohrer appreciate the feedback.  I just pushed a fix, should be live in a few minutes.
In the future note you can open a PR if you discover more issues like this. To get started just click Edit from the page on the Docs site, then Edit again from the GitHub page.


Update docs - "+500" is a valid number format

Money::USD('+500') is parsed into Money with no problems, despite the documentation stating otherwise.
Version: 3.1.3

Closed, in favour of #525 .


Mc 550 agencies matching endpoint

Fixes

https://github.com/Police-Data-Accessibility-Project/data-sources-app/issues/550#issuecomment-2531936683

Description

Sets up /match/agency
LIBTYFI: Some refactoring of test

Testing

Run tests in tests directory, and confirm all function as expected
Inspect API and confirm presence of expected changes.

Performance

New endpoint, whose middleware consists of one database call, but otherwise has no network requests aside from overhead typical to all endpoints
Additional test overhead

Docs

API documentation added

Breaking Changes

No breaking changes

@maxachis for a foundational new endpoint like this, it may be best to comment a few example API calls and their results to illustrate that the original issue requirements are resolved. I can see from the docs that this is about what we'd expect, but a few examples go a long way!

@maxachis for a foundational new endpoint like this, it may be best to comment a few example API calls and their results to illustrate that the original issue requirements are resolved. I can see from the docs that this is about what we'd expect, but a few examples go a long way!

Got it! Will note for the future!


Directory exclude on the local filesystem/ SMB share connection for v2 Music Assistant

What version of Music Assistant has the issue?
2023.3.6
The problem
Hi,
Could you please add directory exclusion for local files in the library?
I'm using QNAP NAS Samba share for my local files, but this share also includes @Recently-Snaphot and @Recycle directories.
This is problematic, because Music assistant library finds music file and playlist duplicates in the  @Recently-Snaphot folder.

It would be great if we could exclude chosen directories from scanning.
How to reproduce
Install Music Assistant v2 beta addon
Add QNAP NAS share with snapshots enabled as a Samba share library.
Relevant log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2023.3.6
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Alternative

are these folders in the root level or also within sublevels ?

@Recycle happens to be also in the @Recently-Snaphot directory, but as a rule it only exists just in a root SMB share

This will be fixed in beta 16


[WIP] Post Video Renderer

What does this PR?
Added iframe support in Renderhtml by adding @native-html/iframe-plugin .
Added webview in anchor renderer to render markdown-video-link in webview instead of in a separate modal.
Issue number
Trello: explore iframe native render plugin
Screenshots

@feruzm
Current and new video playback system comparison....
Current system
https://user-images.githubusercontent.com/6298342/150485977-e75a0ce4-efbe-4964-ab17-e50027c22dd8.mov
New System in progress
https://user-images.githubusercontent.com/6298342/150485924-3415490a-7b63-4897-98c5-5ffb716c6dc8.mov

@aliseyalvi please test and try to perfect the play back of this post, it's a perfect example of a post content containing iframes....
https://ecency.com/hive-172868/@vaketo/-----xgo102be5k8xjugq0drdwpsg4rjvu0gk

@feruzm Current and new video playback system comparison....
Current system https://user-images.githubusercontent.com/6298342/150485977-e75a0ce4-efbe-4964-ab17-e50027c22dd8.mov
New System in progress https://user-images.githubusercontent.com/6298342/150485924-3415490a-7b63-4897-98c5-5ffb716c6dc8.mov

I like new system 👏

@noumantahir

Added iframe support. Replaced iframe with custom iframe renderer using webview.  https://github.com/ecency/ecency-mobile/pull/2154/commits/68a94dd2c13541ab5c11b77430c98f66fc35f9d5
Wrapped source with video tag if  source is video file which fixes the autoplay issue.
changed webview source for supporting all type of videos like Youtube, vimeo etc. https://github.com/ecency/ecency-mobile/pull/2154/commits/8b4e2ea013c661cb4f03d881e255eb3fdadfd670
Some Youtube videos are showing not available error. Might be due to privacy issue or could be other bug.

@aliseyalvi after assessing your solution, it seems we should be able to easily repurpose videoPlayerSheet.tsx to achieve our goal instead of rewriting webviews.
We can rename VideoPlayerSheet to simple VideoPlayer and call it with videoUrl for regular videos and with youtube id in case of youtube and that should also fix the youtube issue you are facing.
Another suggestion from my side would be to skip comments from using new video playback system as they can overload the system.

@noumantahir created videoPlayer component which handles youtube and other videos as well. Iframe renderer has kept as it is because it handles source differently. review the changes https://github.com/ecency/ecency-mobile/pull/2154/commits/9c3f12e06e4b52dac0c3328382f12dbfc6f828bc

Updates
Unified Video Player Component
Using Sheet Style player for comments and inline videos
Disabled autoplay youtube for post videos
Simplified anchorRenderer
@aliseyalvi please go through changes and test if I missed anything that was intentional
@feruzm you may review as well

@feruzm this PR is awaiting your approval before merging


Sub-workflow for pangenome analysis

Description of feature
Create sub-workflow for pangenome analysis using the following nf-core modules

[ ] roary
[ ] pirate
[ ] panaroo

The following modules require writing nextflow modules before integration

[ ] peppan

#40 is required for closing this issue

Manually added in module for PEPPAN from #40 to make merging easier

Completed by #55


10.0.0-alpha.2 - React Developer Tools not active

I use Preact in two of my private projects together with Parcel.js as module bundler. In the last stable version I just had to import 'preact/devtools' in my index.js where the first render function from Preact will be called and then I could use to React Developer Tools too see my current context. Since Preact X this import is no longer available and I had to remove it. I just read in the release notes that the developer tools git a complete rewrite and the support for the Developer Tools was improved. But in both of my projects, the  Developer tools are not active and when I click on the Developer Tools icon, I get only "This page doesn’t appear to be using React."
So my question is what do I have todo to make the Developer tools usable in my project? Maybe it should work out of the box with Preact X but then I would like to know how I could find out, why it's not active in my projects.
The package.json where all the additional config for is made can be found here: https://github.com/janbaer/mdwiki/tree/feature/migration_to_preactx. But thanks to parcel.js there's not so much to configure additionally.

Hi @janbaer
Just import preact/debug where you previously included preact/devtools and the react devtools will start working again.

Thanks @Kanaye, works great. It should be mentioned in the release notes.

I'll reopen this until we've added it to the notes 👍

Back home again and I added a section about that to our release notes :+1: Thanks again for spotting this :)


Add support for Azure blob storage

Signed-off-by: Prasad Ghangal<EMAIL_ADDRESS>

Test result:

Let's create a job to test this before pushing upstream.

@PrasadG193 - regarding @julio-lopez 's comment. We can add an integration test under go/src/test/integration/kopia that tests this.

@PrasadG193 - regarding @julio-lopez 's comment. We can add an integration test under go/src/test/integration/kopia that tests this.

Actually - scratch that - we should create a job that runs the kopia tests.

Made changes to create azure container if does not exists while executing azure tests

Tests:
$ go test -v ./repo/blob/azure/ -count=1
=== RUN   TestAzureStorage
--- PASS: TestAzureStorage (30.01s)
=== RUN   TestAzureStorageInvalidBlob
--- PASS: TestAzureStorageInvalidBlob (0.52s)
=== RUN   TestAzureStorageInvalidContainer
--- PASS: TestAzureStorageInvalidContainer (0.25s)
=== RUN   TestAzureStorageInvalidCreds
--- PASS: TestAzureStorageInvalidCreds (0.00s)
PASS
ok      github.com/kopia/kopia/repo/blob/azure  30.782s

superseded by upstream PR.

Merged upstream


Kotlin 1.4 EAP SAMs

Expected Behavior
Ktlint should not reject SAMs in Kotlin 1.4:
fun interface Extractor<T> {     fun extract(rs: ResultSet): T }
Observed Behavior
Using Kotlin 1.4 EAP, when KTlink encounters a valid  "SAM" (see above) it yields:
Not a valid Kotlin file (expecting function name or receiver type) (cannot be auto-corrected)
Your Environment

Version of ktlint used:
Tool Version: 0.36.0

Name and version (or code for custom task) of integration used (Gradle plugin, Maven plugin)
Gradle Plugin  "org.jlleitschuh.gradle.ktlint" "9.2.1"

Version of Gradle used (if applicable):
Gradle 6.3

Operating System and version:
OS X

Link to your project (if it's a public repository):
https://github.com/nwillc/funkjdbc/tree/feature/kotlin-1.4

We are now publishing ktlint snapshots that are built with the latest Kotlin-preview version, please check how to try it out here

@romtsn I'm curious why this issue is closed because I'm still seeing it on the latest version (0.41.0).

@byencho Just checked on the latest (0.41.0) - the fix is there indeed.

So here's some sample code I'm checking against:
package com.example

fun interface KtLintTest {
    fun testMethod()
}

Using:

ktlint: "0.36.0"
"org.jlleitschuh.gradle:ktlint-gradle:9.2.1"

I get the following:

Task :smartkit:ktlintMainSourceSetCheck
.../src/main/java/com/example/KtLintTest.kt:6:4: Not a valid Kotlin file (expecting function name or receiver type) (cannot be auto-corrected) ()

Which is the same as this original issue. When I upgrade to

ktlint: "0.41.0"
"org.jlleitschuh.gradle:ktlint-gradle:10.1.0"

I get this instead:

A failure occurred while executing org.jlleitschuh.gradle.ktlint.worker.KtLintWorkAction
KtLint failed to parse file: .../src/main/java/com/example/KtLintTest.kt

If I make that a normal interface (instead of a functional one with "fun") then the issue goes away.
So I'm not sure what's going on. Maybe its a bad interaction with some other dependency, or something wrong with that particular plugin.

@byencho the issue you are experiencing is most likely caused not by the ktlint itself.
We use ktlint just by running the sh ./scripts/ktlint.sh command. So it looks like it is the org.jlleitschuh.gradle:ktlint-gradle that causes the issue for you.

@vitkhudenko Yeah it was unfortunately. There was a version hardcoded in the local configuration of the plugin for some reason. Updating that fixed the issue. Thanks for reminding me about this because now I've actually got it all working 👍


Add regex option to filter events to move

Add regex option to filter events to move

would still move from month x to next

But could add related option: target date

closing - already have black, white lists!


VS Code problems language

I have Win 10 language set to Czech (I live there) and VS Code to English. But the errors (Problems panel) still show in Czech. How do I change it to English?

/causedByExtension
Problems come from extensions. VS Code simply displays what it receives.


Emails Blocked By Server

I'm not sure if this is where I can get some help on this but I'm pretty stuck at the moment.
I've setup my DKIM, SPF and DMARC records but for some reason, the only server accepting my emails are Gmail servers.
When I send to any Microsoft Live accounts (Outlook, Hotmail, Live), it says that it has been sent and accepted but nothing shows up on the other end. Not even in Junk Mail.
However, unlike any emails sent from gmail which return a 250 response, and says: 250 2.0.0 OK<PHONE_NUMBER> y193si5439120pgd.41 - gsmtp, anything sent to a Microsoft Live account says 250<EMAIL_ADDRESS>Queued mail for delivery
When I send to any Apple servers, it gets rejected with the error:
553 5.1.8 Domain of sender address does not exist
There are A records pointing to my server so I don't see what would cause that.
I've done the mail-tester.com test and got a 10/10.
I'm wondering if it's because I'm using AWS EC2 and it's being blocked by providers?

Domain of sender address does not exist is probably the biggest clue.
What is the sending domain?

@mattmartincouk it's rallystarter.org

Can you output the full headers from a message that is getting through?

received
from web-ui (ip-119-228-239-173.router.siliconvalleywifi.com [<IP_ADDRESS>]) by Postal with HTTP; Sat, 13 May 2017 23:40:43 +0000
date
Sat, 13 May 2017 23:40:43 +0000
from
<EMAIL_ADDRESS>to
<EMAIL_ADDRESS>message-id
<EMAIL_ADDRESS>subject
Test Message at May 13, 2017 23:40
mime-version
1.0
content-type
text/plain; charset=UTF-8
content-transfer-encoding
7bit
dkim-signature
v=1; a=rsa-sha256; c=relaxed/relaxed; d=rallystarter.org; s=postal-dd8GOi; t=1494718843; bh=mvuu3b9qEfdaZCSaVxLvRSqKbMB5YN4qxRCRw5FtvuE=; h=date:from:to:message-id:subject; b=FNp1+6pSICA9PiJmfTRjsQB7RpCcAqHodsqdW9+ZmwOSTiaHpg9H6MAKg9Wf9//hC0wYGubq7cbz8cdTA8IPXJU51FKHCXfekSeYLrQj/yuD7fULaUaHiDLjHtT3uRJWGT2ghXl4JzXqS8czma7ucauT8VC50o3G9dz4DgKh08A=
x-postal-msgid
ewaLiV6BRQ1B

Is that the headers from the GUI or the receiving end? Need them from the receiving end from any of the mails that got through.

That was from GUI.
Delivered-To<EMAIL_ADDRESS>Received: by <IP_ADDRESS> with SMTP id t18csp555756qvc;
        Sat, 13 May 2017 01:14:37 -0700 (PDT)
X-Received: by <IP_ADDRESS> with SMTP id w1mr8659523pfd.128.1494663277395;
        Sat, 13 May 2017 01:14:37 -0700 (PDT)
ARC-Seal: i=1; a=rsa-sha256; t=1494663277; cv=none;
        d=google.com; s=arc-20160816;
        b=y0nt+bX/YottYwdX4Qy5TAjFZg6oh9iqmkIPvu0DIvmTzGNkZQizJ+QFbHqVv6vOLE
         rLviAasy7IU22gsJfbHFNkQc4B3ak7vw2Pic4hisVA6+my9/X6RkP4x3qGFVQQPVMZxi
         87Oqd1ZxikVMWrvETd8qsABO3asepcDKukLaBooZBmZKJMJTeqgDHVGydi0FnI7VGAtg
         vry1GhoOuTFO9cwrQx0p4c6lxWVuayDyLfx95G/bDEoWyR27wl/pXvJR7EfvmlN/LvCv
         rDLpdNfk6arap7SEUyKNumnNIDQORJ76XfwB8Kkg+eMrHXZHFTd65fkkNzl7r+hrM+Xk
         lF4w==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816;
        h=content-transfer-encoding:mime-version:subject:message-id:to:from
         :date:dkim-signature:resent-sender:arc-authentication-results;
        bh=mvuu3b9qEfdaZCSaVxLvRSqKbMB5YN4qxRCRw5FtvuE=;
        b=f89I5vsMmwKnKmKB9v1K2svgEhwHsTHwIAt8niOGCpJvX2vO7Tg4EBlZYnWR298Rgc
         yeHCzffpX2So1HNRNenHlOjwtprYvKB6ycdyEchMem+GYHJLfMF4WWY0aE0d55baOvVH
         P+jgjuiXA8/UJ/DTNfzBnQRe1Ijbds8ue1QbIBd001U+HyrxMs09VJONtW+ddLAegoUp
         EnBDmDD3qS73N46P0m3vsjngb8la12Dx0agn90bk2OZFEL+TVe2I1E41edNEBQQDOHPN
         mTiBhFk1e6cXjKKG7yeMpLQ+G/vqwnmbU94WGgsCA/WjAvqaQS+wWu6yDMGk++EAwAGs
         T3zg==
ARC-Authentication-Results: i=1; mx.google.com;
       dkim=pass<EMAIL_ADDRESS>       spf=pass (google.com: domain of<EMAIL_ADDRESS>designates <IP_ADDRESS> as permitted sender<EMAIL_ADDRESS>       dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=rallystarter.org
Return-Path<EMAIL_ADDRESS>Received: from smpt.rallystarter.org (postal.rallystarter.org. [<IP_ADDRESS>])
        by mx.google.com with ESMTPS id 59si5392509pld.76.2<IP_ADDRESS>.14.36
        for<EMAIL_ADDRESS>        (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);
        Sat, 13 May 2017 01:14:37 -0700 (PDT)
Received-SPF: pass (google.com: domain of<EMAIL_ADDRESS>designates <IP_ADDRESS> as permitted sender) client-ip=<IP_ADDRESS>;
Authentication-Results: mx.google.com;
       dkim=pass<EMAIL_ADDRESS>       spf=pass (google.com: domain of<EMAIL_ADDRESS>designates <IP_ADDRESS> as permitted sender<EMAIL_ADDRESS>       dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=rallystarter.org
Resent-Sender<EMAIL_ADDRESS>DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=rallystarter.org; s=postal-dd8GOi; t=1494663276; bh=mvuu3b9qEfdaZCSaVxLvRSqKbMB5YN4qxRCRw5FtvuE=; h=date:from:to:message-id:subject; b=jf8gfk53h6mEHWPsaeoj5DZt0F+u2fVck6h30cRkv5tifvqhGmkp4rEGG+plNph282bz/z/M05RLddio7HcybJ78v//0kpiVHj7pL2ctJaOUJ7P4RMQfiaDZ3Z/PJRnOVZs5FmZnhF+D3FyVb1S74V0U3zRi5/EzoSgTOxmyYgc=
X-Postal-MsgID: HSVe62eD57Rv
Received: from web-ui (c-67-169-16-68.hsd1.ca.comcast.net [<IP_ADDRESS>]) by Postal with HTTP; Sat, 13 May 2017 08:14:36 +0000
Date: Sat, 13 May 2017 08:14:36 +0000
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>Subject: Test Message at May 13, 2017 08:14
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit

This is a message to test the delivery of messages through Postal.

DNS is your issue.
psrp.rallystarter.org points to rp.smtp.rallystarter.org
rp.smtp.rallystarter.org does not exist
Review: https://github.com/atech/postal/wiki/Domains-&-DNS-Configuration

Thank you for that! Looks like that fixed the issue with Apple.com, but I guess Microsoft Live accounts are still rejecting my emails unless I add the sender as a contact.

Maybe - could be propagation too.
If sorted, would close this issue as it is more config than software.

Some providers don't like new IPs regardless of how well DNS is configured. They do eventually sort themselves out. You might be able to help by configuring feedback loops with them.

Hi there .
I have almost same issue  and stack .
Please help me .
I have configured my DKIM, SPF and DMARC records correctly.
I think that i know what cause the problem but i don't know how to resolve it .
Thank for future advice .
There are A records pointing to my server so using telnet everything look fine .
Gmail headers of email sanded from postal :
Delivered-To<EMAIL_ADDRESS>Received: by 2002:ab0:1548:0:0:0:0:0 with SMTP id p8csp625817uae;
Wed, 3 Feb 2021 15:13:59 -0800 (PST)
X-Google-Smtp-Source: ABdhPJw5xROdFM+MG/MuUs035NkBNRGvoIvQSU+AmGVjEf1MyIlmzfsEfI0wEQABpAEJBO12j+Uz
X-Received: by 2002:a17:906:e106:: with SMTP id gj6mr5304785ejb.337.1612394039050;
Wed, 03 Feb 2021 15:13:59 -0800 (PST)
ARC-Seal: i=1; a=rsa-sha256; t=1612394039; cv=none;
d=google.com; s=arc-20160816;
b=ielo/RLHla/QlMbDEqOgCPygaXyPsyl31lJvtYHdrbGuXBqeecPgFX2Cg+l8/2XPRj
zKKQu2BJkgI06FGq+nDNg+5v7bWE3pTEsDI/L25uTKFsr9Jrcl++E6+GW4UrJQLvAxSy
+0Orki9P/xUls2HqaGOU36QsN96UQYkz6AWrfZzN+5E8t7Ntgg0UuvHEc5MSpIMO1rpb
Sj/x/Yh3wY4WWl/TPd2XfI1HXv6IvT3OBpLCvtG0P6Zmi+qdHkDeDhb6vk4rNUMvcw76
nluNSO9VSjv83rbFnM/gq3swBsG7c9KW+U6poDdK+DJzYUpRg9CGs4Lced4MUpryA//S
FyNA==
ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816;
h=content-transfer-encoding:mime-version:subject:message-id:to:from
:date:dkim-signature:resent-sender;
bh=RwML7bZTV0QEdoeJq45vt7A1976/m7ol7MMBjBCLcIM=;
b=LheGMxSOSarkVySKY1DwpRtfzeamis3j1+Xd5Xw3mmjqoXlXfzIngxrhAxS9MWNBxR
IfHIt9TL/kFqSjv+lJ7JQkM1i7F8jvpTllFMxVpjhIWdBPUu1r54V3NQgQL47TyJDMVR
NFrjuo6AFpJrEW2RDHZyIn5IPmEyBdq30gqfzBqoiuWH8ofTit0r4a0BYCzwWMy81J/J
p7JcozKgHSKy4xfI9nNfUtr8uI+WRP3CtHdC6tqSiOzQ0syd4L93+vOBp6Q4t1+k76YG
OzgO1itpTsjeFr7ZsmU3L3eiCKhWiTa+k+ZsQvtlsic5eZ025l/Bz+e0dq1J06168WGR
5YNA==
ARC-Authentication-Results: i=1; mx.google.com;
dkim=neutral (body hash did not verify<EMAIL_ADDRESS>header.s=postal-Adzx8X header.b=hycIOfA1;
spf=neutral (google.com: 84.228.160.xxx is neither permitted nor denied by best guess record for domain of<EMAIL_ADDRESS><EMAIL_ADDRESS>Return-Path<EMAIL_ADDRESS>Received: from mail.xmaxfr.com (IGLD-84-228-160-102.inter.net.il. [84.228.160.xxx])
by mx.google.com with ESMTPS id z17si2320314edl.91.2<IP_ADDRESS>.13.58
for<EMAIL_ADDRESS>(version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);
Wed, 03 Feb 2021 15:13:59 -0800 (PST)
Received-SPF: neutral (google.com: 84.228.160.xxx is neither permitted nor denied by best guess record for domain of<EMAIL_ADDRESS>client-ip=84.228.160.xxx;
Authentication-Results: mx.google.com;
dkim=neutral (body hash did not verify<EMAIL_ADDRESS>header.s=postal-Adzx8X header.b=hycIOfA1;
spf=neutral (google.com: 84.228.160.xxx is neither permitted nor denied by best guess record for domain of<EMAIL_ADDRESS><EMAIL_ADDRESS>Resent-Sender<EMAIL_ADDRESS>DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=xxxmaxfr.com; s=postal-Adzx8X; t=1612394037; bh=+ESn9ALTRi6ARpK6byMSwGuOWq1ov3AMtZD/HY4DxAE=; h=date:from:to:message-id:subject:mime-version:content-type:content-transfer-encoding; b=hycIOfA1O90MRzwriR90c+BM9V+A1GCKLCI7/JpAoB9M2V1yXGZzqsKp3zcQeqhULKutPJusZ4Ge0sdtPKSnFlJHRSZBmCoxTl9chWkASvGn1rNzGiNFwfauKJxScZx+Rg+he67U8tBR6U6DsPt8CpiJnZkfaItqjI79PzDHTEs=
X-Postal-MsgID: jh7r7E4B379S
Received: from web-ui (proxyserver.home-test-email-validator_infrastructure [<IP_ADDRESS>]) by Postal with HTTP; Wed, 03 Feb 2021 23:13:57 +0000
Date: Wed, 03 Feb 2021 23:13:57 +0000
From<EMAIL_ADDRESS>To<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>Subject: Test Message at February 03, 2021 23:13
Mime-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 7bit
This line come from my revers proxy docker .
Received: from web-ui (proxyserver.home-test-email-validator_infrastructure [<IP_ADDRESS>]) by Postal with HTTP; Wed, 03 Feb 2021
But i use smtp as localhost 25 of postal inside docker, and map port  to host machine like this.
I really do not understand how is it work.
postal:
    #image: catdeployed/postal:alpine
    build:
        context: postal/
        dockerfile: Dockerfile
    container_name: postal
    privileged: true
    command: run
    ports:
      - <IP_ADDRESS>:25:25
    depends_on:
      - "mysql"
      - "rabbitmq"
    volumes:
      - ./postal/data/persistent_assets:/storage

      - ./postal/data/config/lets_encrypt.pem:/opt/postal/config/lets_encrypt.pem
      - ./postal/data/config/signing.key:/opt/postal/config/signing.key
      #- ./postal/data/postal_assets:/opt/postal/public/assets
      - ./postal/src/templates/:/templates
    environment:
      - MYSQL_ROOT_PASSWORD=${MYSQL_ROOT_PASSWORD}
      - MYSQL_DATABASE=postal
      - RABBITMQ_DEFAULT_USER=postal
      - RABBITMQ_DEFAULT_PASS=changeme
      - RABBITMQ_DEFAULT_VHOST=postal
      - POSTAL_FNAME=Sia
      - POSTAL_LNAME=Cat
      - POSTAL_PASSWORD=StrongPassword
      -<EMAIL_ADDRESS>    healthcheck:
      test: curl -sS --resolve http://postal.home.test:5000:<IP_ADDRESS> http://postal.home.test:5000 || exit 1
      interval: 5s
      timeout: 10s
      retries: 3
    networks:
        infrastructure:
            aliases:
                - postal.home.test

proxyserver:
    container_name: proxyserver
    build:
        context: ./proxyserver/
        dockerfile: Dockerfile
    #volumes:
        #- ./proxyserver/config/certs:/etc/ssl/certs
        #- ./proxyserver/config/private:/etc/ssl/private
    ports:
        - "80:80"
        - "443:443"
        - "42:22"
    networks:
        - infrastructure
    tty: true

    depends_on:
        - postal

networks:
infrastructure:
driver: bridge
i have mounted this files  because postal fail after reboot on error about cant find /opt/postal/config/lets_encrypt.pem key .
volumes:
- ./postal/data/config/lets_encrypt.pem:/opt/postal/config/lets_encrypt.pem
- ./postal/data/config/signing.key:/opt/postal/config/signing.key
SMTP Banner Check  Reverse DNS does not match SMTP Banner
SMTP TLS | Warning - Does not support TLS.
  SMTP Transaction Time | 8.564 seconds - Not good! on Transaction Time
  SMTP Reverse DNS Mismatch  OK - 84.228.160.XXX resolves to IGLD-84-228-160-XXX.inter.net.il
  SMTP Valid Hostname  OK - Reverse DNS is a valid Hostname
  SMTP Connection Time  0.617 seconds - Good on Connection time
  SMTP Open Relay  OK - Not an open relay.
Connecting to 84.228.160.XXX
220 mail.xmaxfr.com ESMTP Postal/BNVK2P [464 ms]
EHLO keeper-us-east-1c.mxtoolbox.com
250-My capabilities are
250 AUTH CRAM-MD5 PLAIN LOGIN [726 ms]
MAIL<EMAIL_ADDRESS>250 OK [260 ms]
RCPT<EMAIL_ADDRESS>530 Authentication required [6813 ms]
LookupServer 9390ms

@maks200179 please open a new issue rather than commenting on long-closed issues


Update checkpoints up until block 510000

This adds two new checkpoints: for block 500000 and block 510000.

Hey
Pls help me Binance to coinbase
My lost money my fund not receive. I requested you..
https://omniexplorer.info/tx/1e3346ef5f0e77f71c59b55c89e04bf201a824d6c2f831b4e0d5fdc54c642ae7

Hi @Amin1214, please see:
https://github.com/OmniLayer/omnicore/wiki/I-sent-tokens-to-an-incompatible-wallet-such-as-Coinbase-or-GDAX.-Can-you-reverse-the-transaction%3F


Task:24310 - free text search field added in form modeler properties …

…and on save form popup

merged into new PR


Wire magnet "prop filter" not working

Whatever it's supposed to do, it's not doing it.
Pasting the model of a prop does not exclude it nor solely target it.

Magnet has been rewritten, the propfilter seems to be implemented (from what I've seen in the source) however it's still not working (I have checked wire_magnet_propfilter, it was "models/sprops/geometry/sphere_18.mdl", however the magnet still targeted all props)


assetmgr: convert on load

to improve workflow with content creation we can convert assets to engine format
upon loading. eg: allow the user to choose a png file in an asset chooser - and generate the dds file
on the fly. Store the dds file into the datablock cache so it is only done once. At some point we will
have a datablock cache manifest and a method for archiving datablock caches for deployment.
Allow for placement of the converter into the tool so one is not forced to include conversion code in a deployable project. After datablock cache archival is complete - deployed projects will not need the source assets.

working for textures.
meshes and animations still in progress

working for meshes
https://github.com/tweakoz/orkid/commit/ddecea33ea8b138055971e290ee922d44a6b33cb
..
https://github.com/tweakoz/orkid/commit/028fc4214525c7ec686937080c73ee7b8adc3919


create a javascript function to check if a string is palindrome or not

learning and coding in javascript will take some time to accomplish.


[bitnami/kubewatch] Add global imagePullSecrets to overwrite any other existing one

What this PR does / why we need it:
This PR allows using a global parameter to indicate the array of secret names to use as Image Registry Secret.
Use:
$ kubectl create secret XXXX ...
$ helm install bitnami/apache --name apache --set global.imagePullSecrets=XXXX ...

Specify a secret globally for a specific registry, in this way, we are able to modify the registry and the secret for pulling images for all Charts globally without having to specify this per image
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[X] DCO signed
[X] Chart Version bumped
[X] Variables are documented in the README.md

/ok-to-test
/lgtm


filesDownload and no fileBlob or fileBinary

I'm using Cordova + AngularJS. When I try to read the filesDownload response value I've got the metadata available except the fact the fileBlob and fileBinary fields are undefined. Finally I solved this by using filesGetTemporaryLink() and downloading the link with some cordvoa plugin, but I would prefer to use the API method.

Unfortunately, the SDK doesn't explicitly support Angular/Cordova, so I'm afraid I don't have a good solution to offer. For reference though, it sounds like this is essentially the same issue as #82, so you may want to try the workarounds I mentioned in #82. I'll send this along as a request for explicit Angular/Cordova support though.


Report coverage in a PR comment

When a PR is pushed, this will generate the coverage report, and
post a comment back to the PR with the coverage total.
Signed-off-by: Brad P. Crochet<EMAIL_ADDRESS>

Can't go cross-fork. Looking for a workaround.


Can't use "#" to find all results by symbol?

VSCode Version: Code - Insiders 1.18.0-insider (2ab10d50163f008ada9a4aa215de8c884857ce99, 2017-11-01T05:19:21.302Z)
OS Version: Windows_NT x64 6.1.7601
Extensions:

Extension
Author (truncated)
Version

vscode-eclipse-keybindings
alp
0.9.1

vscode-markdownlint
Dav
0.12.0

jshint
dba
0.10.15

java-debug
DSn
0.0.2

EditorConfig
Edi
0.11.1

windows-explorer-context-menu
ele
2.0.0

tortoise-svn
fan
0.1.1

php-debug
fel
1.12.1

php-intellisense
fel
2.0.1

php-pack
fel
1.0.1

composer
ika
0.7.1

vscode-todo-parser
min
1.9.1

team
ms-
1.122.0

php-docblocker
nei
1.3.2

material-icon-theme
PKi
3.1.0

(4 theme extensions excluded)

Steps to Reproduce:

Typing CTRL+P, and typing "#symbol" to search file results which contain symbol.
Then the editor shows just one result for me.

For example, I searched "#jsonOutput", then return this result.(see the picture below)

For actually, I have many files contain the symbol "jsonOutput", this symbol is a function name in PHP class.(see the picture below)

But in SublimeText Editor, I can type CTRL+SHIFT+R to search by "jsonOutput", then editor shows the result "jsonOutput".
Immediately follows above, typing RETURN on keyboard, then editor shows many results which contained "jsonOutput".
(see the picture below)

So, I mean VS-code exits a small small bug at symbol searching. It occurs in multi-projects or multi-files.
Thank you for resolving it  :)   :+1:

Reproduces without extensions: Yes/No

Please file this issue on the repo for the PHP extension you're using, this support comes from the extension.

@roblourens
I confirmed this bug is caused PHP extension.
However, how to find all results contain the symbol I need?
Thx!


Maria DB connector driver support

Describe the improvement
Provide the ability to use the Maria DB connector to connect to Maria DB instead of using the MySQL connector
Additional context
Possible place to change
https://github.com/wso2/carbon-kernel/blob/4.6.x/core/org.wso2.carbon.utils/src/main/java/org/wso2/carbon/utils/dbcreator/DatabaseCreator.java#L241

Done with following PRs
wso2/carbon-kernel#2820
wso2/carbon-identity-framework#3194
wso2/carbon-consent-management#178
wso2-extensions/identity-fetch-remote#46
wso2-extensions/identity-inbound-auth-oauth#1483
wso2-extensions/identity-oauth-uma#75


CODEC-240 Percent-Encoding codec proposal

Coverage decreased (-0.08%) to 92.718% when pulling cdde0085266245dacbe66ed42dab3c4604069012 on sermojohn:CODEC-240 into 847a6d07793a198d28819731e5166f9ed3845164 on apache:trunk.

Coverage decreased (-0.08%) to 92.718% when pulling cdde0085266245dacbe66ed42dab3c4604069012 on sermojohn:CODEC-240 into 847a6d07793a198d28819731e5166f9ed3845164 on apache:trunk.

Coverage decreased (-0.04%) to 92.759% when pulling 5da3e7eeb5ee7ecab8eaa0d381ddcb7937bd93a2 on sermojohn:CODEC-240 into f23f600acdd5b0b8d47a079761f2f8d71e541c7d on apache:trunk.

I refactored the solution a bit, in order to improve the performance and support the space character to plus conversion as well. So feature-wise PercentCodec and PercentEscaper (from google-http-client) are equal. Except that I do not provide a fixed set of characters for the different URL encoding scenarios.
Regarding performace, I benchmarked both solutions using JMH beanchmark library and it seems that my solution is a bit faster (10-20% depending on the input).
I had to do some optimizations while comparing with PercentEscaper performance, but I did not copy the byte indexing method, so I keep using BitSet for storing the special characters to always encode instead of using byte[], which as far as I can see will cause high memory utilization.


can't create external table

just followed all steps mentions in READ.ME file . but im getting below error while creating extranal table.
hive> create table wasbtable2 (name varchar(255)) row format delimited fields terminated by ',' stored as textfile location<EMAIL_ADDRESS>FAILED: Execution Error, return code 1 from org.apache.hadoop.hive.ql.exec.DDLTask. MetaException(message:Got exception: org.apache.hadoop.fs.azure.AzureException com.microsoft.azure.storage.StorageException: The server encountered an unknown failure: ) hive>
i updated azure AZURE_STORAGE_ACCOUNT_NAME=, AZURE_STORAGE_ACCOUNT_KEY=
in  env.conf  file

@arsenvlad

Does the blob listOfStockSymbol.txt already exist in that location? Can you try a different location value for your table with a not yet existing file e.g<EMAIL_ADDRESS>to see if it gets created in the storage account?

Does the blob listOfStockSymbol.txt already exist in that location?
yes, it exists.
Can you try a different location value for your table with a not yet existing file
still throwing the same error.

I just tested on my end and it works.
Are you providing the storage account name and key in the env.conf file or env.conf.private file?
By default, docker-compose.yml is loading environment variables from env.conf.private file which is not checked into Git repro. If you are using env.conf file instead, you will need to update docker-compose.yml to point to that file instead since otherwise the required environment variables are not getting set.
Also, when using create table syntax, the location that you specify must NOT be existent yet. hive will automatically create a file and "folder" in the container for the location you specify. From your 2nd error, it looks like hive tried to create a folder with name "listOfStockSymbol.txt" but it was already there. You should create an empty table using a non-yet existing name like this:
<EMAIL_ADDRESS>Take a look at the video recording on the top of README.md file where I step through the demo to see if  it maybe can help you spot the difference between what I am doing and what you have.
Please let me know if you find the difference to add it to readme.
Thanks!


Implement the Transfer command for the Audio module

Description
A step towards achieving #6. The Transfer command is used to change the ownerAddress of an Audio NFT token. This command should

Change the ownerAddress of the Audio NFT token in the AudioSubstore.
Remove the corresponding key from the previous owner's audio.audios array.
Add the key to the new owner's audio.audios array.

Params Schema
The parameters schema looks like:
transferCommandParamsSchema = {
  $id: 'audio/transfer-command',
  title: 'TransferAsset transaction asset for audio module',
  type: 'object',
  required: ['audioID', 'recipientAddress'],
  properties: {
    audioID: {
      type: 'bytes',
      fieldNumber: 1,
    },
    recipientAddress: {
      type: 'bytes',
      fieldNumber: 2,
    },
  },
};

Resolved by #26


TrueNAS Scale compatibility?

Does this driver work with SCALE as well as CORE?

I kicked the tires on TrueNAS SCALE. The lights come on if you remove the "hardware_handler" row from the TrueNAS stanza in /etc/multipath.conf. I've raised a PR to have this removed from the stanza. I've elaborated a bit more in the PR.
If the PR gets merged I'll perform some more testing in the next release. It will most likely be towards the end of the year.

Looks the changes were merged. Good to go now?

@ishioni unfortunately not. We need to wait for a new build to get published. There's an imminent beta drop happening and a production released scheduled for early next year. I might start testing with the beta builds. If you're eager to get started, you can easily change the multipath.conf file manually on your worker nodes when the CSI driver has been deployed.

Nah, that's ok, I'll wait :) I'll give this a spin once i get my new k3os based cluster up anyway
On a completely separate note, would you mind if I submitted a PR integrating the creation of the api secret and Storageclasses as part of values.yaml in the chart? I'm guessing you had a reason not shipping that in the first place?

The problem with StorageClass objects is that they're immutable from Helm (or at least was, when we started working on this). We had functionality to provide StorageClass and backend Secrets in the chart at launch but it caused more angst than it was worth in the end, hence we added those steps as post-install steps.
Also, k3os is not something we know if the CSI driver works with yet. Let me explore that.

Please ignore the k3os part. I've abandoned it in lieu of a k3s+ubuntu setup. Waiting for a version which supports scale, and i'll be happy :)

Please give v2.1.0 a whirl and create a new issue if TrueNAS SCALE doesn't work out for you. TrueNAS SCALE 22.02 RC1 passes full CSI e2e tests in my lab with this release of the CSP.
As a side note. I've verified the HPE CSI Driver for Kubernetes works with K3s, not K3os yet though.

I just switched over, and it works great :)


Change long to short array syntax

I copied the example and recognized that my IDE are not happy about it :)

Duplicates #49


Remove assets dir

Removed the assets directory
fixes https://github.com/fabric8-services/fabric8-wit/issues/1759
There are references to the assets directory in Makefile[0] and main.go[1] file.
Should these too be removed?
[0] - https://github.com/fabric8-services/fabric8-wit/blob/master/Makefile#L254
[1] - https://github.com/fabric8-services/fabric8-wit/blob/master/main.go#L372

Codecov Report

Merging #1785 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1785      +/-   ##
==========================================
- Coverage   57.51%   57.49%   -0.02%
==========================================
  Files         129      129
  Lines       15260    15260
==========================================
- Hits         8777     8774       -3
- Misses       5782     5783       +1
- Partials      701      703       +2

Impacted Files
Coverage Δ

controller/workitem.go
80.38% <0%> (-0.58%)
:arrow_down:

Continue to review full report at Codecov.

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

@xcoulon @kwk @baijum Please review.

@jarifibrahim thank you for this PR. I'm not aware of having used that at all. But be aware that not everything that has "asset" in this name should be removed. We use bindata in other places to pack things with the binary and access it through an asset function in go code.

@kwk Yes. go-bindata should not be removed but go-bindata-assetfs can be removed. go-bindata-assetfs is being used to serve index.html and client.js files in the asset directory.

@jarifibrahim Any movement on this?

@baijum @aslakknutsen Updated the PR

@jarifibrahim snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1785-3

@jarifibrahim snapshot fabric8-wit image is available for testing.  docker pull docker.io/fabric8/fabric8-wit:SNAPSHOT-PR-1785-4


Fix thread leak in worker API

This fixes an issue where worker API invocations could eagerly expand the thread pool way beyond what's necessary to process the work submitted.  This PR does two things:

Only expand the thread pool when there is incomplete work associated with this build operation (for instance, when nested work has been submitted) and there is work waiting to be processed.
Moves the work queue up to build session scope (from project scope) so that all projects use the same queue.

+1 on some comments, lgtm

Thanks for the feedback, guys.  I've addressed the comments and I also added some test coverage for checking the thread pool size when running nested operations as well (with a few minor functionality changes).  Unless you have further comments, I'll plan on merging once the latest changes pass CI.


Either blocks broke of listed counts towards stage

Where else have you looked for answers to this question?
Looked through closed issues and on forums without much luck.
Share your question with as much detail as possible?
I'm wondering if it's possible or will be to list multiple blocks in a stage to be broken, and have one amount they all count to.
For example. A lumberjack quest could require you to cut 64 of any log in the game since they're now separate id's
If applicable, which class or method is this in reference to?
Not too sure sorry,

Thanks for the question. Please see #789


Inconsistent usage of "Centcom" and "Centcomm"

Description
Inconsistent usage of "Centcom" and "Centcomm". Which one is true one? Capitalization-wise as well? grep -iR "centcom" reveals both being in use. Seeing:

Centcom
Centcomm
CentCom
CentComm

Reproduction
Run grep on repo.

means central command, whichever you want to use i guess

Several SS13 servers actually lint to make sure it's always the correct spelling "CentCom" as part of their CI testing.

Anyone wants me to make massive PR replacing everything to CentCom?

Looks good to me, feel free to take it :)

The CDN is called centcomm
There's still 47 references in the codebase to centcomm.

I think we should just revert and go for 2 m's.

...or get new domain with one m, plus rename surviving things I did not touch in order to avoid breaking maps, configs and other PRs.

Getting a new domain is just silly. Revert the pr and then use CentComm, as that's clearly what is intended (see: url).
 CentComm with two "m"s forever.

Centcomm map seems the last usage of this (that github search can find). Looks like the previous changes were merged so Centcom it is?
Only other references are in workflows, Github\Content.Shared\CCVar\CCVars.cs and ShuttleSystem.EmergencyShuttle.cs

Do we want to include changing the changelog?

We're still seemingly having issues with which name is correct on this new pr related to the issue. Maints should probably state a correct name in stone and revert the other prs if need be...

https://github.com/space-wizards/space-station-14/pull/21108#issuecomment-1771962208

I'm new here and looking for my first task, so I might not know what I am talking about. But, shouldn't this be closed now? It looks like the issues was handled and merged.

I'm new here and looking for my first task, so I might not know what I am talking about. But, shouldn't this be closed now? It looks like the issues was handled and merged.

I think the pr you're refering to only partially addressed it?


Update Mandragora.toml

Description
All previous genesis validators should name their PRs "Update {validator_alias}.toml for tesntet-15" and provide links to previous PRs merged.
https://github.com/anoma/namada-testnets/pull/629
https://github.com/anoma/namada-testnets/pull/1045
https://github.com/anoma/namada-testnets/pull/1886
If this is an UPDATE for a previous genesis validator
Checks are made against your net_address. If this has changed since the previous testnet, make sure you provide links of previous prs merged from your previous git username.
Thanks in advance!
Checklist before merging

[x] Only one toml is added in this PR
[x] The file being added is indeed a .toml file
[x] The toml being added is to the correct folder, and only to the correct folder
[x] The eth_hot_key and eth_cold_key are present
[x] The email, discord, elements telegram, and twitter fields are present and valid

Successfully validated transactions.toml file for commit hash 3fbc888


Create a time script

Create a shell script which can determine the run time of a program. Integrate this shell script into the vim workflow mentioned in #3.

Can instead just use the standard time command.


Can't seem to run ANYTHING

I've downloaded multiple 'versions' (stable and otherwise) and it gives me errors all over the place! Depending on what I run it is all around "no module..."
The Modules so far are 'future', 'past', and 'malmo'
I haven't downloaded the "source" and I've gone through the requirements multiple times, and it appears to all be setup correctly..
Windows 10/Python 3.7.1
Any help welcome!

Add to the "no module..." list: MalmoPython

The supported Python version is Python 3.6 on Windows. You could try the following pip install:
pip install -i https://test.pypi.org/simple/ malmo
And follow instructions for the pip installable package https://github.com/Microsoft/malmo/blob/master/scripts/python-wheel/README.md

New python wheel and binary release have now been uploaded for Windows / Python 3.7. Please give these a try.


Spleen_segmentation_3d.ipynb bug

Describe the bug
After training the segmentation model, the evaluate code occur error
---------------------------------------------------------------------------
RuntimeError                              Traceback (most recent call last)
File ~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/transform.py:141, in apply_transform(transform, data, map_items, unpack_items, log_stats, lazy, overrides)
    [140](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/transform.py:140)         return [_apply_transform(transform, item, unpack_items, lazy, overrides, log_stats) for item in data]
--> [141](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/transform.py:141)     return _apply_transform(transform, data, unpack_items, lazy, overrides, log_stats)
    [142](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/transform.py:142) except Exception as e:
    [143](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/transform.py:143)     # if in debug mode, don't swallow exception so that the breakpoint
    [144](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/transform.py:144)     # appears where the exception was raised.

File ~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/transform.py:98, in _apply_transform(transform, data, unpack_parameters, lazy, overrides, logger_name)
     [96](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/transform.py:96)     return transform(*data, lazy=lazy) if isinstance(transform, LazyTrait) else transform(*data)
---> [98](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/transform.py:98) return transform(data, lazy=lazy) if isinstance(transform, LazyTrait) else transform(data)

File ~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/spatial/dictionary.py:527, in Spacingd.inverse(self, data)
    [526](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/spatial/dictionary.py:526) for key in self.key_iterator(d):
--> [527](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/spatial/dictionary.py:527)     d[key] = self.spacing_transform.inverse(cast(torch.Tensor, d[key]))
    [528](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/spatial/dictionary.py:528) return d

File ~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/spatial/array.py:543, in Spacing.inverse(self, data)
    [542](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/spatial/array.py:542) def inverse(self, data: torch.Tensor) -> torch.Tensor:
--> [543](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/spatial/array.py:543)     return self.sp_resample.inverse(data)

File ~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/spatial/array.py:248, in SpatialResample.inverse(self, data)
    [246](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/spatial/array.py:246) with self.trace_transform(False):
    [247](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/spatial/array.py:247)     # we can't use `self.__call__` in case a child class calls this inverse.
...
    [169](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/transform.py:169)     else:
    [170](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/transform.py:170)         _log_stats(data=data)
--> [171](https://vscode-remote+ssh-002dremote-002bhk.vscode-resource.vscode-cdn.net/data2/xiaoyi/MICCAI/~/anaconda3/envs/ldm/lib/python3.8/site-packages/monai/transforms/transform.py:171) raise RuntimeError(f"applying transform {transform}") from e

RuntimeError: applying transform <monai.transforms.post.dictionary.Invertd object at 0x7f7faf0f23a0
```>

It seems like the post_transforms have problem when dealing with val_outputs

**Environment (please complete the following information):**
 - OS ubuntu 22.04
 - Python version 3.8.19
 - MONAI version 1.4.dev2427
 - CUDA/cuDNN version 11.3
 - GPU models and configuration 3060

Hi @MaybeRichard, I tried running the notebook but couldn’t reproduce the issue. Could you check if it persists with the latest release version of MONAI? Thanks!

sorry, I tried it again in python mode rather than jupyter, it worked now. I'm not sure what caused this problem.


Add emqx_mountpoint, emqx_session test suites, and fix bug in emqx_sm test suite

There are two commits I write


Add another stub for Entitlement for preventing to load from app folder in migration

https://github.com/ManageIQ/manageiq/issues/6739
this change is allowing to use https://github.com/testdouble/good-migrations
and prevents to load Entitlement model from app folder
@jrafanie @Fryguy @chrisarcand

Good find @lpichler :+1:

Wait the reason you added a second entitlement in this order is because
Rails auto loads the class via the association as soon as the model is
defined? Is that right?
On Tuesday, April 5, 2016, Joe Rafaniello<EMAIL_ADDRESS>wrote:

Good find @lpichler https://github.com/lpichler [image: :+1:]
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/ManageIQ/manageiq/pull/7729#issuecomment-206003131

--
Chris Arcand
@chrisarcand

@chrisarcand yes exaclty but this is valid when you are using definition with class
has_one :entitlement, :class_name => RemoveMiqUserRoleFromMiqGroups::Entitlement

but when you use it in quotes
has_one :entitlement, :class_name => 'RemoveMiqUserRoleFromMiqGroups::Entitlement'

it seems that is not loading the class 'RemoveMiqUserRoleFromMiqGroups::Entitlement' for this case even when I deleted model entitlement.rb => good_migration gem was silent and be db:migrate ran succesfully
(I am not sure what are exact differences in these cases when sql is generated or how impact it has, if any)
@jrafanie @Fryguy
so should I rather use it in quotas ? (then we can avoid multiple definition of classes in migrations for these cases)

I don't think Entitlement should be defined twice like that.  I would think that all :class_name should be strings as that's how we do it in other migrations, and don't have a problem.

Ooooh I see. My fault entirely. Yes we should be using strings.


[WIP] Feature: review code output options

The option to output devai review code in the existing markdown (remain default) format along with json and in a table. This change adds -o with the options markdown, json, table.
To achieve this the prompts have also been externalised to their own individual files (including ones that define the required output for json and markdown).
The default is markdown to allow for backwards compatibility.
$ devai review code -c ../sample-app/src/main/java

Response from Model: ## Code Review: Bank of Anthos - BalanceReader

**Overall:** The BalanceReader code demonstrates a good understanding of Spring Boot, caching, and security principles. However, there are some areas for potential improvement in terms of error handling, thread management, and code style.

**Transaction.java**

* **No Issues Found:** This class is well-structured and straightforward.

**LedgerReader.java**

* **Thread Management (Best Practices):** The `backgroundThread` is managed within the `LedgerReader` class. While this works, it might be more robust to delegate thread management to a dedicated `ExecutorService` for better control over thread lifecycle and potential thread pool management in the future.
   * **Suggestion:** Consider using `ScheduledExecutorService` to manage the polling task, providing a more structured way to schedule and control the thread.

* **Error Handling (Robustness):** The `pollTransactions` method silently catches `ResourceAccessException` and `DataAccessResourceFailureException`. While the intention might be to retry, consider logging the exceptions with more details (e.g., stack trace) or adding retry logic with exponential backoff for transient errors.

* **Magic Number (Code Style):** Using `-1` as `STARTING_TRANSACTION_ID` could be made more expressive.
   * **Suggestion:** Define a constant like `private static final long NO_TRANSACTIONS = -1L;` to improve readability.

**BalanceCache.java**

* **No Major Issues Found:** The use of Guava's `LoadingCache` is appropriate, and the configuration appears reasonable.

**JWTVerifierGenerator.java**

* **Key Loading (Security/Best Practices):** The public key is loaded from the file system on each request. In a production environment, consider caching the loaded key for performance reasons.
    * **Suggestion:** Load the public key once during application startup and store it in memory for subsequent use.  You might use a library like `jasypt-spring-boot` for secure configuration loading if needed.

**BalanceReaderApplication.java**

* **Environment Variable Check (Robustness):** The environment variable check during startup is good practice. However, consider using a more structured approach for configuration validation, perhaps using Spring Boot's `@ConfigurationProperties` and `@Validated` annotations to leverage Spring's validation framework.

**BalanceReaderController.java**

* **Bearer Token Handling (Code Style):**  The manual parsing of the `bearerToken` could be simplified. Spring Security provides utilities for handling bearer tokens more cleanly.
   * **Suggestion:** If you're open to using Spring Security, leverage its `BearerTokenAuthenticationFilter` or related components for token extraction and validation.

* **Error Responses (Best Practices):** Returning string literals like `"not authorized"` and `"cache error"` might not be ideal for all clients. Consider returning more structured error responses, possibly with error codes, to make it easier for clients to handle different error scenarios.

**General Observations:**

* **Logging (Maintainability):** The logging is generally good. However, consider adding more detailed log messages in critical areas like exception handling to aid in debugging.
* **Testing (Best Practices):** While not directly part of the review, ensure you have sufficient unit and integration tests covering the logic, especially error handling and edge cases.

Let me know if you'd like a more in-depth analysis of a specific aspect or have any particular concerns.

$ devai review code -c ../sample-app/src/main/java -o json

[
    {
        "class": "BalanceReaderController",
        "method": "getBalance",
        "issue": "Potential Cache Miss Handling",
        "severity": "Medium",
        "description": "The `getBalance` method retrieves the balance from the cache using `cache.get(accountId)`. While the `LoadingCache` should ideally handle cache misses, it's worth considering explicitly handling cases where `cache.get()` might throw an exception (e.g., due to a backend data store issue). This could involve logging the error and returning an appropriate error response to the client, preventing potential application crashes."
    },
    {
        "class": "LedgerReader",
        "method": "startWithCallback",
        "issue": "Infinite Loop in Background Thread",
        "severity": "Critical",
        "description": "The background thread in `startWithCallback` has a potential issue. If an exception occurs within the `try` block (e.g., `InterruptedException`, `ResourceAccessException`, etc.), the `alive` flag might not be set to `false`. This could result in an infinite loop where the thread keeps running but doesn't process transactions, potentially leading to resource exhaustion. It's recommended to ensure that `alive` is set to `false` in all exception scenarios to gracefully stop the background thread."
    },
    {
        "class": "LedgerReader",
        "method": "startWithCallback",
        "issue": "Resource Management: Thread Interruption",
        "severity": "Medium",
        "description": "In the `startWithCallback` method, the background thread doesn't handle thread interruption cleanly. If the thread is interrupted while sleeping (e.g., during application shutdown), the `InterruptedException` is caught and logged, but the thread continues running. It's best practice to honor thread interruption and exit gracefully. Consider re-throwing the `InterruptedException` or setting a flag to indicate that the thread should terminate."
    },
    {
        "class": "BalanceReaderController",
        "method": "getBalance",
        "issue": "Exception Handling and Logging",
        "severity": "Low",
        "description": "In the `getBalance` method, the catch blocks for `JWTVerificationException` and `ExecutionException/UncheckedExecutionException` log the error but return generic error messages to the client. While this is acceptable for security reasons (not revealing too much information about the error), consider providing more informative logging, including the exception message or stack trace, to aid in debugging potential issues."
    },
    {
        "class": "LedgerReader",
        "method": "pollTransactions",
        "issue": "Potential Logging Overhead",
        "severity": "Low",
        "description": "The `pollTransactions` method logs \"Polling Transactions\" in debug mode. If this method is called frequently, it might lead to excessive logging, especially if the debug level is enabled in production. Consider logging at a less verbose level (e.g., trace or info) or logging only when necessary, such as when a new transaction is found."
    }
]

$ devai review code -c ../sample-app/src/main/java -o table

┏━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┳━━━━━━━━━━┓
┃ class                   ┃ method           ┃ question                             ┃ answer                                                                                ┃ severity ┃
┡━━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━╇━━━━━━━━━━┩
│ General                 │ N/A              │ Overall Impression                   │ The code is well-structured and demonstrates good use of caching and background processing.  However, there are opportunities to improve  │ Medium   │
│                         │                  │                                      │ error handling and logging.                                                                                │          │
│ LedgerReader            │ run              │ Handling Remote Database Out of Sync │ When the remote database is out of sync, the application stops processing transactions. Would it be more resilient to implement a retry   │ High     │
│                         │                  │                                      │ mechanism or a more sophisticated synchronization strategy instead of completely halting the processing?                                  │          │
│ BalanceReaderController │ getBalance       │ JWT Verification Exception Handling  │ The code catches JWTVerificationException but returns a generic 'not authorized' message for different verification failures. Could we    │ Medium   │
│                         │                  │                                      │ provide more specific error messages (e.g., expired token, invalid signature) to aid in debugging and client-side error handling?         │          │
│ BalanceReaderController │ getBalance       │ Cache Error Handling                 │ Similar to the JWT verification, the cache error handling returns a generic message.  It might be beneficial to log the specific          │ Medium   │
│                         │                  │                                      │ exception and provide a more informative error message or potentially attempt a fallback mechanism.                                       │          │
│ LedgerReader            │ pollTransactions │ Logging Transaction Details          │ Currently, the logging in 'pollTransactions' only indicates the polling action.  Would it be helpful to log the transaction details being │ Low      │
│                         │                  │                                      │ processed for debugging and auditing purposes?                                                                                │          │
│ BalanceCache            │ load             │ Cache Loading Logging                │ The log message 'Cache loaded from db' in the 'load' method might benefit from including the accountId for which the cache is being       │ Low      │
│                         │                  │                                      │ loaded.  This could help in tracing cache behavior during debugging.                                                                      │          │
└─────────────────────────┴──────────────────┴──────────────────────────────────────┴───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────────┴──────────┘

replaced with #35


Improved tokenizer function to handle extra spaces

This pull request does the following:-

Improved tokenizer function
Added tests for tokenizer function
Closes #84

I think some CI script is broken 🤔 because it is failing to start itself.


[Python] Should RecordBatchStreamReader/Writer be AsyncIterable?

Filing this issue after a discussion today with @xhochy about how to implement streaming pyarrow http services. I had attempted to use both Flask and aiohttp's streaming interfaces because they seemed familiar, but no dice. I have no idea how hard this would be to add – supporting all the asynciterable primitives in JS was non-trivial.
Reporter: Paul Taylor / @trxcllnt
Note: This issue was originally created as ARROW-4283. Please see the migration documentation for further details.

Any progress on this?


feat: chained printing method

Hey @tr3v3r - made a bit of progress today towards the chained printing method, looking for any feedback on this draft. I'll be looking at the image tomorrow

@zchryst Grate job! Thanks a lot!
Are you planning to add the same feature for Android?
p.s. I'll review today / tomorrow the draft

@tr3v3r Sorry, I'm afraid our project is only using iOS and due to time constraints, I won't be able to commit to the android side.
I'm hoping that this can at least lay the groundwork for someone else to do that part?

Alrighty - I've got the image coming through as well, now converting to a proper PR 🥳

@zchryst Thanks. I've left several comments. Could you please take a look and then we can merge it into the main branch?
I'll provide the same functionality for Android as well on my own.

Thanks @tr3v3r - I'll try fix that up towards the end of the week

@tr3v3r - apologies for the delay, I've added the stub, updated the docs, and added the promise wrapper - seems to all be working rather well, though I do not have an android device to test with I'm afraid

merged. I'll make a short check and will release a new version soon

@zchryst hi! I've just tested your solution regarding printing image and it seems that it doesn't work ;( ( I'm tested on my Epson printer TM-T20II ). For some reason, the base64 image is not processed correctly.
On the other hand, I'll found another solution to put images into iOS/Android assets folder ( on native side ) and print image from the giving source just using asset name like:

ios

assets

image.png

.image('image', 200)

So for now I'll keep your method in the code and rename it to imageFromBase64 to use in chain printing and add another method 'imageFromAssets' to print from source I've mentioned above.
It would be nice if could investigate what the problem and fix if it possible. Thanks


"Failed: There was an unexpected error -- try again in a few seconds" while using createPaymentMethod.

Describe the bug
I'm using a CardField component to pay for services and using createPaymentMethod to add it to stripe and 2-3 months later it doesn't work and I get this strange error: "Failed There was an unexpected error -- try again in a few seconds" without any error code.
To Reproduce
This is the method I'm calling, while using CardField:
	const { paymentMethod, error } = await createPaymentMethod({ 			paymentMethodType: 'Card', 		});
Carfield component:
    <CardField
         postalCodeEnabled={false}
         autofocus
         onCardChange={(cardInfo) => {
            setCardDetails(cardInfo);
         }}
          onFocus={(focusedField) => console.log('focusField', focusedField)}
     />

I'm using the newest version of stripe-react-native: 23.01
Expected behavior
I  expect to have returned paymentMethod by createPaymentMethod function so I can pass it to confirmPayment function.
Additional context
It used to work and also there's no other issue with Stripe, the payment goes through with pre-saved card.

Is this happening on Android or iOS? Is cardInfo.complete true?

Is this happening on Android or iOS? Is cardInfo.complete true?

It's happening on iOS, cardInfo.complete true ? Where it says I should add it?
Also I have an issue with pre-saved card not going through too now, which all of a sudden needs "paymentMethod" to be added to confirmPayment function while I pass the paymentMethodID from presaved credit card to it.

Is this happening on Android or iOS? Is cardInfo.complete true?

It's happening on iOS, cardInfo.complete true ? Where it says I should add it? Also I have an issue with pre-saved card not going through too now, which all of a sudden needs "paymentMethod" to be added to confirmPayment function while I pass the paymentMethodID from presaved credit card to it.

Had a similar issue in the https://github.com/flutter-stripe/flutter_stripe library. Not sure how this translates in the react plugin, but a possible fix may be to initialize the Stripe instance without a stripeAccountId and only init with parameters like publishableKey and merchantIdentifier etc..

@Karniej can you let me know if you're passing in a stripeAccountId to the StripeProvider or initStripe call?

I found out that the issue was made by our backend not returning the correct values, preventing stripe from working properly. Closing :)

@Karniej  Still Facing The Problem, working fine on ios emulator but giving exact the same error on real device.

For me fix was to change publishableKey prop to live key because in other places I have set live version but here I had it as test key:


WidgetPart should contain a 'CssClass' property

On a site containing various widgets it would be awesome - especially for sitebuilders - to be able to set one or more CSS classes to the widget containers from the dashboard instead of doing it from an overridden shape. Now it is possible to do it using the Name property but it is unique and only one class name is allowed. It makes sense because this property is not only for styling.
All in all, that would be great to see a fully displayed WidgetPart property value (ie. without removing spaces or dashes) in the class attribute of a widget container.

You can do this today by attaching the TextField for example to the WidgetPart. That way all widgets will have that field.

Yes, but for that you'd need a shape override or a wrapper that will actually use that value as CSS classes. It's not that it's hard to do, but this is a missing feature when you have a lot of widgets.

Do you have any idea how often this feature is missed? Sometimes, or most of the times?

If you have a site with more than 3 widgets and a non-trivial design then all the time.

I sort of like the idea - we have it for elements after all, so why not for widgets too - but it seems arbitrary to do it only for widgets. Why not, in that case, take it all the way and allow entering CSS classes on content items as well?

But I must say I have never missed it either, and I have built sites with more than 3 widgets and non-trivial design. :) I find that in the majority of cases (put conservatively) a widget does not share any traits with other widgets that would make me want to apply styling to them as a group. Widgets are normally only defined (and styled) by virtue of A) being a widget and B) being a specific widget, both of which we already have classes for.

Projector Layouts also can have CSS classes and an ID specified.

We are accepting this suggestion for next release. However as @sfmskywalker mentioned you can configure widgets to support that, and the Vandelay Indutries module has this feature too.


Support for UserDefinedValue can be fetched from configured json DataSource

Created support for UsedDefinedValue to be fetched from a Json or http endpoint along with option to apply filter on the Datasource. This requirement is needed for me, when I tried creating berserker configurations in the form of a workflow for a Rest service having dependent rest Endpoints. And I believe that this is a requirement that most berserker users like.
Method Usage:
userDefinedValue("DataSourceName","JsonPathQuery","refreshIntervalInSeconds","regExpressionToFetchExpectedStringFromFetchedField")
DataSourceName: Can be a name of JSON file available in classpath/ A Rest endpoint that return a JSON response
JsonPathQuery: JSON path query to extract a Filed from specified Data Source. For semantics, please refer to https://github.com/json-path/JsonPath
refreshIntervalInSeconds: To refresh the data at configured intervals if the source support dynamic data retrieval.
regExpressionToFetchExpectedStringFromFetchedField: To support the case where the requirement is not complete field and only specific part.
example definitions:
Sample JSON:
{
"firstName": "John",
"lastName" : "doe",
"age"      : 26,
"address"  : {
"streetAddress": "naist street",
"city"         : "Nara",
"postalCode"   : "630-0192"
},
"phoneNumbers": [
{
"type"  : "iPhone",
"number": "0123-4567-8888"
},
{
"type"  : "home",
"number": "0123-4567-8910"
}
]
}
userDefinedValue("resources","$.phoneNumbers[].type","100"," ")
userDefinedValue("http:/url/resources","$.phoneNumbers[].type","100"," ")
above Value Definition will evaluate to Values
"iPhone",
"home"

@vajda Yes I need HTTP (JSON) DataSource for Berserker the same way as CSV DataSource exists within Ranger project

@hemanthkilari
The feature request you made is reasonable and something we already had in our mid-term plan, but unfortunately haven't had enough
I hope your proposed implementation solves your problem and that you can build Berserker version with this ranger feature included and use it to solve your problem.
However, proposed implementation does not reflect the approach to solution we want to have and we cannot accept your pull request as such. We will be able to implement it in next 2-3 months due current lack of time. Or if you are interested, maybe we can guide your proposed solution to a point where we both are satisfied with the outcome. What do you think about it?

@vajda Let us discuss your approach. We will take decision based on outcome.

@hemanthkilari
Ok, first of all name userDefinedValue is misleading, since this will handle only documents returning JSON. I'd suggest to call it fetchJson or getJson or something in that fashion.
Let's discuss parameters:

DataSourceName, I'd call it just source, classpath/http endpoint semantics is pretty good idea
JsonPathQuery, it would return subtree of JSON response
refreshIntervalInSeconds, although I understand necessity of this performance wise, this is something that I would not introduce on this level. I'd rather have some additional wrapper DSL type to handle caching of values in case value retrieval time is large enough. Something like:
cached(fetchJson('http://x.y.z', '$.a.b'))
regExpressionToFetchExpectedStringFromFetchedField This is also something that should be solved through additional transformer value type, similar to third param as it can benefit other value types as well (csv for example)
As this will return subtree of JSON which effectively will be a map, get can be used for further extraction.

I'd like that we come up with common agreement on this conceptual level first, then we can discuss implementation approach and details.

@vajda I have no issues with your suggestions. The presented code was written for our need. I do not mind in changing it to satisfy Ranger specification. Provide me other details to proceed with implementation.

@hemanthkilari
Well, change the name of function to fetchJson and value class to FetchJsonValue, remove the third and fourth parameter. Semantics for remaining two parameters (the first and the second) can be as you already proposed.
There should not be e.printStackTrace(); in the code. In all the cases where checked exception needs to be caught, rethrow RuntimeException like:
catch (Exception e) {
            throw e instanceof RuntimeException ? (RuntimeException) e : new RuntimeException(e);
        }

Besides ValueExpressionParser which represents YAML API for Ranger, there is also BuilderMethods class which represents Java API. This class needs to provide method for creation of FetchJsonValue.
Also tests should be added and documentation files updated (yaml-configuration.md and java-api.md)
I think that's all. When you make these changes, I'll do a code review on the pull request.
Once again, thanks for your interest in Ranger, your help is very much appreciated and welcome!


[#184281154] Fix rebuilds can cause state corruption

What
The pivotal storey provides the detail.
The create-bosh-concourse pipeline has been changed such that Terraform is now configured with its own backend and is no longer reliant on the pipeline providing and saving its state via resources.
I have also used the aws cli to pull down a file from an s3 bucket to obtain the current bosh manifest state.
How to review
1 - Review the code.
2 - Run the branch through a dev env (I've tested this branch on dev01).
3 - After the run, check the size of the files (bosh.tfstate, bosh-manifest-state-(region).json, concourse.tfstate, bootstrap-cyber.tfstate, vpc.tfstate) in the appropriate bucket for the env e.g. for dev01 the bucket would be gds-paas-dev01-state
Who can review
Not me.

🚨⚠ Please do not merge this pull request via the GitHub UI ⚠🚨

This PR has the changes from https://github.com/alphagov/paas-bootstrap/pull/568. If that's needed here, it'd be helpful to have them at least as a different commit so that it's easier to review the changes for this PR

Ooh yes, I hadn't thought of the impact on the reviewer.

Need to provide separate commits


Media library

Chat widget
Demo:

Sped up:

Conversations
Demo:

Screenshot:

Slack
Demo:

Getting started
Screenshot:

Integrations
Screenshot:

Playground

Twilio integration docs
Create a Twilio account


Update database.json

The key of your module should not use dashes (-) in its name. You should use underscores (_) instead.

@aaronwlee You can still get this merged if you update the key to be attain_graphqL. The repository name can stay as is.


Add an option for specifying the port for npm

Currently UseReactDevelopmentServer and UseAngularCliServer randomly select a port for starting the npm script. However in a few cases (for example when caching the webpack configuration) it would be useful to be able to specify the port being used.
Would it therefore make sense to add an optional port parameter to those two functions?

Another example is when JS app is using 3rd party services with CORS protection enabled. In my case, it's QlikSense server with Access-Allow-Origin with a particular port set up for development. It would be nice if we could provide port value, e.g. using SpaOptions


docs(hierarchicalMenu): showParentLevel is boolean

see #2505, thanks @Giovanni-Mattucci

uh that snapshot failure isn't mine @iam4x


Correction

Added a missing step in line no. 80. Please review

Docs Build status updates of commit ed2c93f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/create-a-bot-for-teams.md
:white_check_mark:Succeeded
View

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.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Thank you, @Rajeshwari-v!


docs: Update useConnect example with client param

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR updates the useConnect.ts file to incorporate a client configuration when connecting a wallet, enhancing the connection process with the createThirdwebClient function.
Detailed summary

Added an import statement for createThirdwebClient from thirdweb.
Created a client instance using createThirdwebClient with a clientId.
Modified the wallet.connect() method to include the client parameter.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#5550  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


test: ensure apt and python c extensions work properly

Repost of this PR which was closed: https://github.com/railwayapp/nixpacks/pull/1103

I ran into a tricky problem:

I have a py project
The project has to execute a system command, notably apt in this case
stdenv.cc.cc.lib is required for numpy to work, but it breaks apt (and, more importantly, I'm assuming many other system commands) with the following error:

0.110 nix-env: /nix/store/02mqs1by2vab9yzw0qc4j7463w78p3ps-glibc-2.37-8/lib/libc.so.6: version `GLIBC_2.38' not found (required by nix-env)

This error can be fixed by reverting the mutated LD_LIBRARY_PATH from:
LD_LIBRARY_PATH=/nix/store/cq3kvw7kwz2n9c13fg6qaaj23mkm116l-gcc-12.3.0-lib/lib:/nix/store/wzm0753byf1app0xrsa97hjnfk9m1zs0-zlib-1.3/lib:/usr/lib

to:
LD_LIBRARY_PATH=/usr/lib

Did some research:

https://github.com/mcdonc/.nixconfig/blob/e7885ad18b7980f221e59a21c91b8eb02795b541/videos/pydev/script.rst
https://github.com/nix-community/poetry2nix
https://discourse.nixos.org/t/nixos-with-poetry-installed-pandas-libstdc-so-6-cannot-open-shared-object-file/8442/8

I'm new to nix. It seems like creating a bespoke shell env with LD modified is the way to go, but that wouldn't actually solve this issue since
LD_LIBRARY_PATH would be inherited by the py process and therefore passed to the shell (unless the modified LD only applies to package installation, but I'm confident that would work either.
In any case, I'm curious what folks think here. I added a failing test to work off of, but unsure of the best solution.
Docker container success assertion
In investigating this, I found that the docker container execution is not asserted to be successful. This feels like a good change
to make for all containers to avoid swallowing otherwise important errors.

The problem seems to be a mismatch between the glibc version installed in Ubuntu and used by the system packages and the glibc version installed and used by the Python Nix package

Yup, this is definitely the issue.

I'm not super comfortable globally modifying the LD_LIBRARY_PATH variable to not include any of the Nix packages

Agreed. That would cause issues in other ways.

If possible I think we should update the Ubuntu glibc version or have an configuration option to reset the LD_LIBRARY_PATH variable.

Yeah, having a shell script bundled to do this wouldn't be terrible. However, there's got to be a better way. I don't know nix enough to understand what the canonical way to handle this, but I imagine this is something that other nix users have run into before.

Friendly reminder about this pull request! Let me know if there's anything else needed here.

Yeah, I'm not really sure what the best solution here is. My limited knowledge of nixpkgs is working against me. Will fix the lining and see if that gets us closer.

Friendly reminder on this pull request! Let me know if there are any updates or additional changes needed.

Yes sorry @iloveitaly, haven't forgot about this. Just unsure of the best solution currently and haven't had that much time to explore solutions.

Yeah, I'm unclear as well. Is there anyone tied to the project that has deeper nix experience that could weigh in? I imagine this problem has come up quite a bit in the past.

Hmm could we dynamically set the LD_LIBRARY_PATH based on the providers?
You'd almost need like, a hash map, and then each provider could either create an entry, or blank an entry, during a phase. And then you'd join it all into the LD_LIBRARY_PATH
Or maybe have an install phase run as a "pre-phase" execution?
Like in theory you'd solve this by doing all your apt stuff, modifying LD_LIBRARY_PATH, and then doing "pip install" right?

Hmm could we dynamically set the LD_LIBRARY_PATH based on the providers?

We already do this with the nixLibs build plan config https://nixpacks.com/docs/configuration/file#nix-libraries
But the problem in the failing code example provided is that both numpy and apt are used.

I wonder if we could build a base image of debian where the glibc version matches the glibc version in Nix

What are the use cases for requiring apt in Python? This Nix way would be to install everything with Nix packages and then the glibc versions would all be compatible. The problem you are trying to mix non-Nix with Nix packages together.
One option would be for us to install Python not using Nix at all (and with something like asdf)

I like this idea of using mise or asdf to install python. It would be really interesting to use mise across all providers.
I ran into this specifically when launching an interactive she'll and debugging a container issue locally. However, I ran into this issue on tools beyond apt which means that users could run into this when shelling out to a CLI binary.
I like the map-based modification, although I don't have a strong idea of what impacts that would have on the rest of the system. I just don't have enough incentive right now to learn nix.


(JSON) Properly format cell JSON objects

If the option to export cells wrapped with { } as JSON objects is active, the cell's data does not get formatted before exporting. It should be automatically formatted like other JSON values, and indented in a similar manner to the rest of a JSON export.

This is now supported as of v40. The JSON object is validated to make sure it is correctly formatted, and will simply be added as an empty object ( { } ) if invalid.


Soft fullscreen removes  element so document.body.style causes null deref

Testing on Chrome, press F11 to enter fullscreen, F11 again to exit, then this exception is thrown:
Uncaught TypeError: Cannot read property 'style' of null
at HTMLDocument.restoreOldStyle (craft.js:9026)
because emscripten's soft fullscreen seems to remove all other elements in the html document including <body>, so document.body is null!

Attempted in https://github.com/satoshinm/NetCraft/pull/48 to workaround this by doing my own "windowed full" mode, but it stretches the canvas undesirably, so using soft fullscreen in emscripten is desirable but this is probably a bug in its implementation (who removes body from html? bad idea?).

Didn't see this before, but could this be useful? https://github.com/kripken/emscripten/blob/master/tests/test_glfw_fullscreen.c - not using glfwSetWindowMonitor added in https://github.com/satoshinm/NetCraft/issues/9 since emscripten doesn't have it (glfw 3.2+), but rather emscripten's fullscreen and a glfw window size callback

Starting from test_glfw_fullscreen, writing a minimal example, am able to successfully toggle between fullscreen and "fullwindow" (emscripten soft fullscreen), whipped up a demo here: test_fullscreen_fullwindow_toggle (source)  - press F11 to toggle. So there may be some error on my end causing this body problem.
test_fullscreen_fullwindow_toggle has at least one important difference: listens for window size changes with glfwSetWindowSizeCallback, not .canvasResizedCallback. NetCraft is using the latter, as on_canvassize_changed(), to set glfwSetWindowSize(). test_fullscreen_fullwindow_toggle only is using a solid gray background currently, so I can't see the stretching/scaling effects, but it does correctly fill the entire screen and window.
test_fullscreen_fullwindow_toggle also is able to toggle fullscreen using F11 through glfwSetKeyCallback, unlike in NetCraft where I thought I had to register a separate fullscreen_key_callback from emscripten_set_keydown_callback to be considered a "user action" for the purposes of calling the fullscreen APIs as a result of user action, but it works either way. This means it ought to be possible to simplify and remove the emscripten_set_keydown_callback special case for F11 from NetCraft.


Unable to pull rancher/mirrored-pause as PodSandBox from a secured registry

Environmental Info:
K3s Version:
1.22.2+k3s2
Node(s) CPU architecture, OS, and Version:
Linux k3d-devcluster-server-0 3.10.0-1160.45.1.el7.x86_64 #1 SMP Fri Sep 24 10:17:16 UTC 2021 x86_64 GNU/Linux
Cluster Configuration:
1 Agent, 1 Server setup with k3d
Describe the bug:
I have a secured mirror registry reachable from the cluster. The Server has only access to the company network and no access the the public internet. When I start the cluster and connect to it, no pods can be initialized because it failed to pull the rancher/mirrored-pause:3.1 image, while setting up the pod:
FailedCreatePodSandBox:  failed to pull and unpack
 image "docker.io/rancher/mirrored-pause:3.1":
 failed to resolve reference
"docker.io/rancher/mirrored-pause:3.1": failed to authorize:
 failed to fetch anonymous token: unexpected status: 401 Unauthorized

I believe to correct setting up the cluster with k3d. I will print you the configs:
How the registires.yaml ist configured: /etc/rancher/k3s/registries.yaml -
mirrors:
  docker.io:
    endpoint:
    - https://docker-public.docker.XYZ.com
configs:
  docker.io:
    auth:
      username: USERNAME
      password: PASSWORD
      auth: BASICAUTH
      identity_token: ""
    tls:
      ca_file: ""
      cert_file: ""
      key_file: ""
      insecure_skip_verify: true

I've also checked, if i can pull the images with containred from the server. I works fine:
ctr --debug i pull -u USERNAME:PASSWORD docker-public.docker.XYZ.com/rancher/mirrored-pause:3.1
I've checkd the contaiered config for k3s /var/lib/rancher/k3s/agent/etc/containerd/config.toml
[plugins.opt]
  path = "/var/lib/rancher/k3s/agent/containerd"

[plugins.cri]
  stream_server_address = "<IP_ADDRESS>"
  stream_server_port = "10010"
  enable_selinux = false
  sandbox_image = "rancher/mirrored-pause:3.1"

[plugins.cri.containerd]
  snapshotter = "overlayfs"
  disable_snapshot_annotations = true

[plugins.cri.cni]
  bin_dir = "/bin"
  conf_dir = "/var/lib/rancher/k3s/agent/etc/cni/net.d"

[plugins.cri.containerd.runtimes.runc]
  runtime_type = "io.containerd.runc.v2"

[plugins.cri.registry.mirrors]

[plugins.cri.registry.mirrors."docker.io"]
  endpoint = ["https://docker-public.docker.XYZ.com/"]

[plugins.cri.registry.configs."docker.io".auth]
  username = "USERNAME"
  password = "PASSWORD"
  auth = "BASICAUTH"

[plugins.cri.registry.configs."docker.io".tls]
  insecure_skip_verify = true

Expected behavior:
authenicate to the registry and sucessfully pull the rancher/mirrored-pause:3.1 from the mirrored registry
Actual behavior:
FailedCreatePodSandBox - 401 Unauthorized
Additional context / logs:
FailedCreatePodSandBox:  failed to pull and unpack
image "docker.io/rancher/mirrored-pause:3.1":
failed to resolve reference
"docker.io/rancher/mirrored-pause:3.1": failed to authorize:
failed to fetch anonymous token: unexpected status: 401 Unauthorized
Backporting

[ ] Needs backporting to older releases

mirrors determines endpoints for a registry
configs determines credentials and tls for an endpoint

You've asked to use docker-public.docker.XYZ.com as your local registry mirror, but haven't configured any credentials for it; you've only configured credentials for docker.io. If you want to provide credentials for your internal registry, you should specify it in the configs section:
mirrors:
  docker.io:
    endpoint:
    - https://docker-public.docker.XYZ.com
configs:
  docker-public.docker.XYZ.com:
    auth:
      username: USERNAME
      password: PASSWORD
      auth: BASICAUTH
      identity_token: ""
    tls:
      ca_file: ""
      cert_file: ""
      key_file: ""
      insecure_skip_verify: true


chore: Udate OTel and prometheus components

This change does the following:

Updates OTel components
Removes the untyped metrics hack, and replace it with the support
upstream (using metric metadata instead of an additional attribute per
sample)
Confgenerator updates
Several test updates

This change brings our fork much closer to upstream OTel and Prometheus.
This should be pretty helpful when unforking.

Related library change: https://github.com/GoogleCloudPlatform/prometheus/tree/feature/shutdownscrape_v0.54.1


Use process.resourcesPath so that macOS can find the resource directory

Currently, resourcesPath in index.js looks directly for 'resources/app' to decide whether or not to use the relative directory, or resources/app/. However, for macOS, 'resources/app'  will not exist in the packaged app, as the structure is instead '/Contents/Resources/app'. As a result, macOS then uses the path './', which then causes the resources folder to not be found.
Using the electron builtin process.resourcesPath ensures we check for the correct resources path regardless of OS.
I have tested and confirmed this builds and runs on macOS, though someone should probably confirm and verify this doesn't break anything on other platforms.

I have since confirmed that the packaged versions still run on Windows and Linux with this change.

Ah, thank you


Date docs jky t5

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


check for applied adjustments in spree way

Please Make the changes for 3-1-stable as well.

Done. Please check #17 .