Address in mailbox given does not comply with RFC 2822, 3.6.2.

When i try to send a newsletter i have this error :
Address in mailbox given<EMAIL_ADDRESS>] does not comply with RFC 2822, 3.6.2.
I don't understand what is wrong because this address realy exist...

its didnt help me

Its help me - before send message I do mysql_real_escape_string()

In a Symfony 2.6.4 application, a custom config parameter admin_email<EMAIL_ADDRESS>throws the above referenced error when the parameter is defined inside the bundle.  If the parameter is instead added to parameters.yml (and parameters.yml.dist) and no custom configuration appears in the bundle the error is NOT thrown.  The error is not dependent on validity or compliance of the e-mail address.  For example<EMAIL_ADDRESS>is not considered compliant.
The custom parameter was moved from parameters.yml to satisfy the Sensio Insight constraint that only default parameters from parameters.yml.dist are allowed.

For googlers, I had the same issue. The problem was the PCRE lib version used by the mod_php which dont want to compile the regex used by swiftmailer to check addresses. (there is a warning the the php logs).

Lou: which one did you use? Thanks!

@bastianmartin I did not change the pcre lib because I dont have the right on the server. I just made an ugly fork and removed the check. See LouTerrailloune/swiftmailer@d71ecbb35c33aaf8de4dfd5f2e1167cbe3383187 .

@LouTerrailloune Thank you! I'm trying to update PHP libs - maybe it was already fixed.
An interesting post: http://stackoverflow.com/questions/15963755/swift-rfccomplianceexception-on-valid-email-address-works-in-windows-not-in-ubu

Thanks - I've updated PHP to the latest DotDeb Build (5.6.10) and the issue is gone.

Why not change this code:
if (!preg_match('/^' . $this->getGrammar('addr-spec') . '$/D', $address))

With this:
if (0 === preg_match('/^' . $this->getGrammar('addr-spec') . '$/D', $address))

Then, any false will throw a classical error that will not break up the entire application.
Could also add a @ right before preg_match to be sure no error is thrown.

plz make sure there is no white space before or after the email address given

It's your role to use trim on the email address before sending it to Swiftmailer, I think

In my case, PHP was the culprit. After updating to 5.6.10 (without changing the existing PHP code) the issue was gone. However, I also think that Swiftmailer should use trim on the addresses.

In my case, it works after updating PHP from 5.5.25 to 5.5.28.

t

In my case, im using PHP 5.6.18, facing the same error message.

in my case it was a blank bcc

i used iconv() and trim() to fix this issue

trim() helps me too ^^)

same error here. Trim not helped me. This are the emails that throw the error:
<EMAIL_ADDRESS><EMAIL_ADDRESS>Really i don't want to send those emails, it's just a simulation, but this error break it.
any hint?

Hello! I can't to send any e-mail with addresses<EMAIL_ADDRESS>or<EMAIL_ADDRESS>and get the same error. Google supports this addresses. Why are you not support it?

Hello!
Seems like an old issue, but I don't know where to post this.
I'm using mailgun.com service, and there is a feature of batch sending emails, by defining receiver as %recipient% and adding header, containing recipients data.
It would be great, if there would be a way to redefine Swift_Mime_Grammar as DI and create interface for it?

Closing as the email validation is now done by an external library that you can configure (Swiftmailer 6+).

@fabpot how can you configure this external library? link to documentation or an example would be extremely helpful
see #999

Work for me just fine
$trim_ = trim($email);
$filter_var = filter_var($trim, FILTER_SANITIZE_EMAIL);
$iconv = iconv('ISO-8859-1','UTF-8//IGNORE', $filter_var);

Hello! I can't to send any e-mail with addresses<EMAIL_ADDRESS>or<EMAIL_ADDRESS>and get the same error. Google supports this addresses. Why are you not support it?

You should convert your пример.рф  domain into IDNA format like 'пример.рф'

i was also getting this problem when i was sending multiple emails at a time and there was email duplication passed in bcc , to , CC methods
$message->to('abc@gmail.com')->bcc(['abc@gmail.com','johndoe@gmail.com']);

Simply change :+1:
```
$email = new Swift_Message();
<EMAIL_ADDRESS>=> "Le ROBOT-gueulard(t) du (B25)"));
and
swiftmailer-5.x/lib/classes/Swift/Message.php at line 46 by
           ` array($this, 'Swift_Message::__construct'),`
instead of
           ` array($this, 'Swift_Mime_SimpleMessage::__construct'),`

For me, it was empty email address sent to setReplyTo(). Fixed by not calling this method if reply-to is absent. I think, the same would be with setFrom, setCc, setBcc.

In my case it was a blank bcc, too!

in my case it was a blank bcc
Your comment should be on top somewhere

This looks like a very old question. Recently, I've got an error like this because i have put the email address in the name argument and vice versa. Maybe this comment can help someone...


APOD

Astronomy Picture Of the Day
Rule Submission
Website: apod.nasa.gov

[X] I have made the rule as simple as possible ( K.I.S.S )
[X] I have run the rule myself for a period of time to spot any bugs

Thanks for your contribution


Refactored StorageManager's createBlock API.

This small PR refactors createBlock in StorageManager, as a part of replacing CatalogRelation with the read-only CatalogRelationSchema in InsertDestination.

I saw the word 'storage manager' so I reviewed the changes. LGTM

@cramja Then merge it. Thanks!


Mock SI Hardware

Requested by Barb. For testing and easier development.

[ ] Find where SI events are handled
[ ] Make new executable, with a window to specify what information to send to SI handler

[ ] Optionally, load configurations from file

GecoSI repository contains test classes that may be useful. They'd need to be migrated from rxtx to jSerialComm


Expose Get-DefaultSqlCredential function

I'm running the following script using the new function released Invoke-ScriptInNavContainer.
Invoke-ScriptInNavContainer -containerName $containerName -ScriptBlock {
Sync-NAVTenant -ServerInstance NAV -Mode Sync -Force
}
The build stops there for long time and from the Event Viewer seems the issue is related to the credentials not passed.
Looking in your code often is used this line before
$sqlCredential = Get-DefaultSqlCredential -containerName $containerName -sqlCredential $sqlCredential -doNotAskForCredential
I think Get-DefaultSqlCredential function should be exposed

Sync-NavTenant doesn't require Sql credentials - no reason for asking for that.
If you connect to the container Open-NavContainer (or Enter-NavContainer) and type
Sync-NavTenant -serverinstance NAV -mode sync -force
what happens? (it seems like you are missing a tenant parameter)
You can also try to run
Get-NAVTenant NAV | Sync-NavTenant -Mode ForceSync -Force

So, my preference is to use your new function because I do not need to import NAV Modules
I just run a test with this script
Invoke-ScriptInNavContainer -containerName $containerName -ScriptBlock {
Get-NAVTenant NAV | Sync-NavTenant -Mode Sync -Force
}
It took more tha 6 minutes. No errors and no info in the Event Viewer. Is there anyway how to see why is taking so long?
Am I correct saying that if I use Open-NavContainer or Enter-NavContainer i have to import NAV Modules?

No, both Open and Enter-NavContainer has imported the same modules as Invoke-Script.

I've just run a test using this script
Open-NavContainer $containerName
Get-NAVTenant NAV | Sync-NavTenant -Mode ForceSync -Force
and it failed "C:\agent_work\1\git\Powershell\SyncSchemaChanges.ps1 : The term 'Get-NAVTenant' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again."

Open-NavContainer opens a window - the Get-NavTenant needs to be executed in that window:

I am aware that you cannot use Open-NavContainer in scripting to execute your script - but you can test your script in there.
If it works there, it should also work in Invoke-ScriptInNavContainer.
This way you would have found the error with the first command easier.

No, I cannot do it, I'm working in a build script that is run thruough DevOps Agent in one of my servers.

You should work on your scripts locally or connected and then test then on devops agents (IMO)

Yep I'm doing, testing both side :-D
I will let you know what I will find.

Done, the Sync-NAVTenant took few seconds

Now the question is why is taking longer from the script using Invoke-ScriptInNavContainer

Quick Update:
I'm running the following command and not it is much faster
Invoke-ScriptInNavContainer -containerName $containerName -ScriptBlock {
Sync-NAVTenant -ServerInstance 'MicrosoftDynamicsNavServer$NAV' -Mode Sync -Force
}
The only difference is the ServerInstance name.
The error related to the credentials is still showing in the Event Viewer but it is during the creation (everythign works but I have 2 errors for each run)
Error 1
exec's CreateProcess() failed [exec=a2c9aba2eef1dc352885074088bf4397cf21acd57cd88a9fd6ff917f71441819 module=libcontainerd namespace=moby error=container deb0ecdd0e80dd06579d64671b12c757dada687d84b6a9a1c105aef579dcc17c encountered an error during CreateProcess: failure in a Windows system call: The user name or password is incorrect. (0x52e) extra info: {"CommandLine":"cmd.exe /C "ECHO <IP_ADDRESS>    host.docker.internal \u003e\u003e %systemroot%\system32\drivers\etc\hosts \u0026 ECHO <IP_ADDRESS>    gateway.docker.internal \u003e\u003e %systemroot%\system32\drivers\etc\hosts"","User":"Administrator","WorkingDirectory":"/","Environment":{"COMPLUS_NGenProtectedProcess_FeatureEnabled":"0","DatabaseName":"CRONUS","ExitOnError":"N","IsBcSandbox":"N","NAVDVDURL":"https://nav2016wswe0.blob.core.windows.net/dvd/13a89a5b-5a10-4bd0-a19a-1d424f4a9aa1","VSIXURL":"https://nav2016wswe0.blob.core.windows.net/dvd/al-0.12.16641.vsix","WebClient":"N","accept_eula":"Y","accept_outdated":"Y","auth":"NAVUserPassword","databaseInstance":"","databaseServer":"","enableSymbolLoading":"Y","httpSite":"N","licenseFile":"c:\run\my\license.flf","locale":"en-US","passwordKeyFile":"c:\run\my\aes.key","removePasswordKeyFile":"Y","securePassword":"76492d1116743f0423413b16050a5345MgB8AGgASgBoAEgARwBpAFQAVgBvAG8AVQBLAE8AWgBqAHYAYgBOAFoAcABDAGcAPQA9AHwANQBjAGIAYwA1ADIAZAAyADAAYwBjADgAMwA0ADkAYQAxADIANQA0ADYAMQBhAGIAOQBhADgAOAA4AGEAMABjAGYAMAA2AGEAOABlADEAOQA2AGMAMABlADgAOQBjAGUANABlADcAYgBjADgAOABlADMAYwAxADMANwA1ADkAMgA=","useSSL":"N","username":"admin"},"CreateStdInPipe":true,"CreateStdOutPipe":true,"CreateStdErrPipe":true,"ConsoleSize":[0,0]} container=deb0ecdd0e80dd06579d64671b12c757dada687d84b6a9a1c105aef579dcc17c]
Error 2
Error running exec a2c9aba2eef1dc352885074088bf4397cf21acd57cd88a9fd6ff917f71441819 in container: container deb0ecdd0e80dd06579d64671b12c757dada687d84b6a9a1c105aef579dcc17c encountered an error during CreateProcess: failure in a Windows system call: The user name or password is incorrect. (0x52e) extra info: {"CommandLine":"cmd.exe /C "ECHO <IP_ADDRESS>    host.docker.internal \u003e\u003e %systemroot%\system32\drivers\etc\hosts \u0026 ECHO <IP_ADDRESS>    gateway.docker.internal \u003e\u003e %systemroot%\system32\drivers\etc\hosts"","User":"Administrator","WorkingDirectory":"/","Environment":{"COMPLUS_NGenProtectedProcess_FeatureEnabled":"0","DatabaseName":"CRONUS","ExitOnError":"N","IsBcSandbox":"N","NAVDVDURL":"https://nav2016wswe0.blob.core.windows.net/dvd/13a89a5b-5a10-4bd0-a19a-1d424f4a9aa1","VSIXURL":"https://nav2016wswe0.blob.core.windows.net/dvd/al-0.12.16641.vsix","WebClient":"N","accept_eula":"Y","accept_outdated":"Y","auth":"NAVUserPassword","databaseInstance":"","databaseServer":"","enableSymbolLoading":"Y","httpSite":"N","licenseFile":"c:\run\my\license.flf","locale":"en-US","passwordKeyFile":"c:\run\my\aes.key","removePasswordKeyFile":"Y","securePassword":"76492d1116743f0423413b16050a5345MgB8AGgASgBoAEgARwBpAFQAVgBvAG8AVQBLAE8AWgBqAHYAYgBOAFoAcABDAGcAPQA9AHwANQBjAGIAYwA1ADIAZAAyADAAYwBjADgAMwA0ADkAYQAxADIANQA0ADYAMQBhAGIAOQBhADgAOAA4AGEAMABjAGYAMAA2AGEAOABlADEAOQA2AGMAMABlADgAOQBjAGUANABlADcAYgBjADgAOABlADMAYwAxADMANwA1ADkAMgA=","useSSL":"N","username":"admin"},"CreateStdInPipe":true,"CreateStdOutPipe":true,"CreateStdErrPipe":true,"ConsoleSize":[0,0]}

Last update for today:
The script is slow using the parameter -Mode Sync (more than 2 minutes) Same script using Force is fast (few seconds).

The right serverinstance in the container is NAV - the name you use in the latest blog post is the service name running inside the container.
The way to do sync is:
Get-NAVTenant NAV | Sync-NavTenant -Mode ForceSync -Force
Which will work for both single and multi-tenant instances.
I have no idea why Sync is slow and ForceSync is faster - but probably because it will test everything first - has nothing to do whether you are running containers or not I guess.
As for the two errors above - they seem to occur when running  CMD.exe /C ECHO something - I don't think this has anything to do with the Sync-NavTenant - nor do I think it has anything to do with the NAV/BC containers - it must be something happening on your setup.
Will close this issue


Make method less complex, add Code Climate badge

Hey there! I ran the ember-rails code through Code Climate when I had some spare time to do some OSS work, and the results are great! I made one small fix that it uncovered, to help make a method a bit more readable and to pull some code out that you may eventually want to test separately. I also added the Code Climate badge to your README. Thanks for the awesome lib!

@tricknotes Your comment has been addressed.

Thanks @mrb :dizzy:


Updated to 4.0 after the get method USES the image download progress.

 [imageView sd_setImageWithURL:imageUrl
                     placeholderImage:image
                              options:SDWebImageAvoidAutoSetImage | SDWebImageRefreshCached
                             progress:^(NSInteger receivedSize, NSInteger expectedSize, NSURL *targetURL) {

            NSNumber *rectivdNumber = [NSNumber numberWithInteger:receivedSize];
            CGFloat received = rectivdNumber.floatValue;

            NSNumber *expectedNumber = [NSNumber numberWithInteger:expectedSize];
            CGFloat expected = expectedNumber.floatValue;

            if (expectedSize != -1 && receivedSize != 0) {
                rotateScaleView.progressView.progress = received / expected;
            }
        } completed:^(UIImage * _Nullable image, NSError * _Nullable error, SDImageCacheType cacheType, NSURL * _Nullable imageURL) {
            if (image) {
                imageView.image = image;
            }
            if (error != NULL) {
                [rotateScaleView.progressView removeFromSuperview];
            }

        }];

Here get photo download progress will lead to program crashes.
   [imageView sd_setImageWithURL:imageUrl
                     placeholderImage:image
                              options:SDWebImageAvoidAutoSetImage | SDWebImageRefreshCached
                             progress:nil completed:^(UIImage * _Nullable image, NSError * _Nullable error, SDImageCacheType cacheType, NSURL * _Nullable imageURL) {
            if (image) {
                imageView.image = image;
            }
            if (error != NULL) {
                [rotateScaleView.progressView removeFromSuperview];
            }

        }];

Here the progress is set to nil, there is no problem.Excuse me what reason be?In previous versions have not found this problem.

What is the crash exactly

I use this on the simulator download image method is ok, but I'm really on board with this method will cause the program card dead flash back.
If I set SDWebImageDownloaderProgressBlock nil, there is no problem.

@LanYanQin can you reproduce this crash with our demo project? Could you create a demo project of your own where this crash happens all the time? It would help identify the issue.
Also, what's the stack trace for the crash?

@bpoplauschi  I create a new demo found unable to reproduce this issue.May be the cause of my own project.

@bpoplauschi encountered the same issue in my project.

@boycechang You also like this?I have a problem now used in the project, if you are creating a new DEMO, there is no problem.

@LanYanQin exactly the same.

@LanYanQin @bpoplauschi  This problem seems caused by Xcode project cache. Flow the steps bellow can fix this issue.

Clean Build.
Reset Simulator.
Restart Xcode.
Delete your DerivedData folder in ~/Library/Developer/Xcode/DerivedData.
Delete the contents (not the whole folder) of /var/folders.
Restart Computer.

@boycechang I solved the problem of your idea.

Great to hear. Closing based on the previous comments.


Dynamic registration of supported language modes

Description
As i see, you use constants for registration of supported modes. Now it looks incorrect and applicable for the concept of the cycle.
You can close this PR, because it's dispute about tastes 🌵
Changelog

Use 4 spaces for indentation in extension.ts as in other files
Dynamic registration of supported language modes instead of constants

@mrmlnc,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@mrmlnc Can you rebase and update this PR? There are some conflicts.

@ramya-rao-a, done 👍

Thanks!


⚠ Bofrost.at has degraded performance

In 3fde156, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1822 ms

Resolved: Bofrost.at performance has improved in dd9005e after 8 minutes.


Bump torch-mlir to 9938abf25e1e7526ca7f43a8c49e9078c14fc55c

Hi @ScottTodd, I'm trying to do the Torch-MLIR bump in IREE but there are 2 failing tests:
	 64 - iree/hal/drivers/hip/cts/hip_stream_driver_test (SEGFAULT)
	 74 - iree/hal/drivers/hip/cts/hip_graph_driver_test (SEGFAULT)

for the AMD GPU. I don't know how to fix this failure. Can you please shed some light on this?

@ScottTodd Can you please merge this PR for me? It seems I don't have commit access to the repo. Also, what's the criteria to get the commit access?

Also, what's the criteria to get the commit access?

https://iree.dev/developers/general/contributing/#obtaining-commit-access
Contributors can request to join https://github.com/orgs/iree-org/teams/iree-write once they are a member of iree-org. We don't have formal criteria, but it's usually based on a few factors:

Necessity - does some action require that level of access? E.g. testing infrastructure changes may require pushing branches to the shared repo.
Contribution history - does the contributor have enough history of PRs/commits that follow project guidelines

See also https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access

Also, what's the criteria to get the commit access?

https://iree.dev/developers/general/contributing/#obtaining-commit-access
Contributors can request to join https://github.com/orgs/iree-org/teams/iree-write once they are a member of iree-org. We don't have formal criteria, but it's usually based on a few factors:

Necessity - does some action require that level of access? E.g. testing infrastructure changes may require pushing branches to the shared repo.
Contribution history - does the contributor have enough history of PRs/commits that follow project guidelines

See also https://llvm.org/docs/DeveloperPolicy.html#obtaining-commit-access

Hi @ScottTodd, I'm already part of the IREE-Org, it's just that I need to be added to the iree-write team. I have the iree-triage access.


Feature/remove gtm update footer

What
This PR removes the GTM script from the markup and updates the footer to match with the current ONS one.
Updating the footer meant pulling in the new styles of Sixteens which required some additional changes. Most notably, the addition of a new _banner sass partial which contains the old styling of the beta banner. This is because the banner styling on the main ONS page has been updated to a new pattern.
How to review

Check that instances of GTM within the codebase are removed
Check that the page styling looks fine
Check that the footer matches the one on the main ONS site

Who can review
Anyone but me

lgtm, just checking it is right to use old palette blue here.

lgtm, just checking it is right to use old palette blue here.

Thanks for taking a look. Just FYI there isn't a specific requirement for the banner colour palette. I updated the version of Sixteens used by the developer site when I was updating the footer (so that the new typography could be included) - but it also made changes to the beta banner too (as on the main site the banner no longer has a blue background by default). So I initially made the call add in the new Scss file to ensure the beta banner could retain its old styling.
That said, I think using a new blue for the banner does make sense:
Old

New (using ocean-blue)

@armstrongb - if you want, I'm happy to update to use ocean-blue (or another colour if you think something else is preferable)?


nuget push getting 404 after upgrade

Describe the bug
After upgrading from 3.29. to 3.3.1 I am getting the following error from bamboo build task: Nuget push build artifacts
16-May-2023 15:14:50 | [consumer_0] Deploying artifact: https://artifactory.xxx.com/artifactory/duckcreek-nuget-local/develop/1.7.0-rc8/MedPro.Build.1.7.0-rc8.nupkg
16-May-2023 15:14:50 | Failed to upload file
16-May-2023 15:14:50 | [consumer_0] An exception occurred during execution:java.lang.RuntimeException: java.io.IOException: JFrog service failed. Received 404: <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
31-Dec-1969 19:00:00 |
31-Dec-1969 19:00:00 |
31-Dec-1969 19:00:00 | <meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1"/>
31-Dec-1969 19:00:00 | <title>404 - File or directory not found.</title>
31-Dec-1969 19:00:00 |
31-Dec-1969 19:00:00 |
31-Dec-1969 19:00:00 |
31-Dec-1969 19:00:00 |

Server Error

31-Dec-1969 19:00:00 |

31-Dec-1969 19:00:00 |
<fieldset>
31-Dec-1969 19:00:00 |
404 - File or directory not found.

31-Dec-1969 19:00:00 |
The resource you are looking for might have been removed, had its name changed, or is temporarily unavailable.

31-Dec-1969 19:00:00 | </fieldset>

31-Dec-1969 19:00:00 |

31-Dec-1969 19:00:00 |
31-Dec-1969 19:00:00 |
31-Dec-1969 19:00:00 |
31-Dec-1969 19:00:00 | at org.jfrog.build.extractor.clientConfiguration.util.spec.SpecDeploymentConsumer.consumerRun(SpecDeploymentConsumer.java:44)
31-Dec-1969 19:00:00 | at org.jfrog.build.extractor.producerConsumer.ConsumerRunnableBase.run(ConsumerRunnableBase.java:11)
31-Dec-1969 19:00:00 | at java.base/java.lang.Thread.run(Thread.java:829)
31-Dec-1969 19:00:00 | Caused by: java.io.IOException: JFrog service failed. Received 404: <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">

To Reproduce
Run Artifactory Nuget configuration set to push and run.
Expected behavior
Expect artifacts to be pushed.
Screenshots

Versions

Bamboo Artifactory plugin version: 3.3.1 (upgraded from 3.2.9 which was working)
Bamboo operating system: 8.2.8 (Data Center)
Artifactory Version: 7.35.2 (on-prem windows deployment)

Additional context
Add any other context about the problem here.

Additional Info:

Generic Deploy still worked on versions 3.2.10 and 3.3.0.  Broke with 3.3.1
Artifactory Nuget configuration (Push) - worked on 3.2.10 but got error listed above on 3.3.0

I have discovered the issue.  The Artifactory nuget push fails with a 404 if spaces are in the Build Name.  I changed to use the buildKey variable (no spaces) and it worked.  FYI this is also an issue with the jfrog cli when attempting to upload nuget artifacts with build info.


Format issue in the Add knowledge base information to your bot section

There is appears to be a formatting issue where the text renders as "| QnAKnowledgebaseId | The knowledge base ID that the QnA Maker portal generated for you. | | QnAAuthKey (QnAEndpointKey in Python) | The endpoint key that the QnA Maker portal generated for you. | | QnAEndpointHostName | The host URL that the QnA Maker portal generated. Use the complete URL, starting with https:// and ending with /qnamaker. The full URL string will look like "look like "https://< >.azure.net/qnamaker". |" within the "Add knowledge base information to your bot" section.

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

ID: 466e4935-993f-ed61-0446-c6eff7310572
Version Independent ID: 50fe83f8-c261-0bbe-5b8e-910ee27f17dc
Content: Azure Bot Service tutorial to have a bot answer questions - Bot Service - Bot Service
Content Source: articles/v4sdk/bot-builder-tutorial-add-qna.md
Service: bot-service
GitHub Login: @JonathanFingold
Microsoft Alias: kamrani

i'm having an issue exactly in this section..
i followed the instructions to add the knowledge base information in the .env file, but when i tested the bot locally it gives me this error:
"[onTurnError] unhandled error: TypeError: Cannot read property 'endsWith' of undefined"
also, if i comment this part in the code and just send an echo message (for example) it works well..
this are an example of the parameters i'm using:
QnAKnowledgebaseId:"xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"
QnAAuthKey:"xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"
QnAEndpointHostName:"https://xxxxxx.azurewebsites.net/qnamaker"
i'm wondering if this could be an issue in the KB data format

well, the issue was solved, probably it was a word mistype.. nothing to do with KB data format


Fixed random SQLite error code:522, 'not an error'

I've fixed an error that can affect some Simulator (mine for example) running unit-test.

2015-10-21 17:10:24.579 xctest[703:8687] CoreData: error: (522) I/O error for database at /Users//Library/Developer/CoreSimulator/Devices/5E7487DC-2883-45F6-8CC1-FB90A2EB73C1/data/Documents/Pokemon.sqlite.  SQLite error code:522, 'not an error'
fatal error: 'try!' expression unexpectedly raised an error: Error Domain=NSCocoaErrorDomain Code=522 "(null)" UserInfo={NSSQLiteErrorDomain=522, NSUnderlyingException=I/O error for database at /Users//Library/Developer/CoreSimulator/Devices/5E7487DC-2883-45F6-8CC1-FB90A2EB73C1/data/Documents/Pokemon.sqlite.  SQLite error code:522, 'not an error', NSFilePath=/Users/*/Library/Developer/CoreSimulator/Devices/5E7487DC-2883-45F6-8CC1-FB90A2EB73C1/data/Documents/Pokemon.sqlite}: file /Library/Caches/com.apple.xbs/Sources/swiftlang/swiftlang-700.0.59/src/swift/stdlib/public/core/ErrorType.swift, line 50

Running multiple tests the database seems to be corrupted, maybe caused by sqlite temporary files.
I've improved NSPersistentStore deletion in tearDown.
Thanks

hopefully this will let unit tests stop failing on my build server :dancers:

Same error?

mine was test manager service generic error (typical Xcode server error)... its integrating now... will let you know if it passes :D

IT PASSED!

GOOD!


SGR 1 and SGR 8 don't work when adjusting lightness of indistinguishable text

Windows Terminal version
1.12.2931.0
Windows build number
10.0.19041.1348
Other Software
No response
Steps to reproduce

In the Setting UI, go to the Profiles Defaults page and select the Appearance tab.
Make sure the option Automaticaly adjust lightness of indistinguishable text is enabled.
Open a bash shell.
Execute the following command: printf "\e[m DEFAULT \e[1m BRIGHT \e[8m CONCEALED \e[m\n"

Expected Behavior
I'd expect the text with the SGR 1 attribute to be brighter, and the text with SGR 8 attribute to be invisible.

Actual Behavior
All of the text looks exactly the same.

This wouldn't be that big a deal if it was an option that users specifically opted in to, but I realise now that it's actually on by default. Was that really intentional? Even ignoring these problems, I would have thought the performance overhead alone was enough of a reason to make it opt-in.

We ended up going with an option that pre-cooks nudged colors for pairs in the lower 16x16 color grid, instead of doing it per color pair on render (which I think you'd suggested -- so we didn't nudge direct RGB colors or anything from the xterm 256 palette above the base sixteen specified in the color scheme.) That significantly reduced the performance impact.

Yeah, I've seen the code - I've actually been trying to optimise it to some extent as part of another PR I was working on (which is how I discovered these issues). I haven't actually tested the hit on performance, but it just seems like there's still a lot code being executed there. I had actually hoped we could implement this with zero run-time overhead, but maybe that was wishful thinking.
I should also mention that there are probably other scenarios in which this option can break things - SGR 1 and SGR 8 are just the two most obvious examples. I'll follow up here with more examples if/when I find them.

So I can't seem to completely repro this - I can repro the issue with the concealed text showing up when it shouldn't (looking into that right now), but the bright text seems to be working as it should for me:

Here's another example of the kind of problem you can get with the color adjustment enabled.

Set your color scheme in the settings to Solarized Light.
Execute the following command in bash: printf "\e[m DEFAULT \e[7m REVERSED \e[m\n"
Using escape sequences, change the palette to Campbell.

Instead of getting the Campbell color scheme, you get a mix of the two, I think probably with the foreground colors still coming from the Solarized Light scheme.

If we could make the color adjustment algorithm only apply to the initial color scheme, then that would avoid problems like this (and also improve the performance).
Just spitballing here, but consider an algorithm that can calculate some kind of brightness value for each color in the color scheme, and sort them on a linear scale. If there is a minimum brightness difference that would make all colors reasonably distinguishable, it shouldn't be too hard to spread those values out until they're all visibly distinguishable. You then regenerate the color scheme with the adjusted brightness levels and you're done - no runtime modification required. Does that not seem feasible?


[Feature] Prevent use of respawn anchor in overworld / end

I like using the respawn anchor as a decoration block, but if you right click it by accident when it's charged it'll explode.
Would it be possible to disable interaction with the respawn anchor like the bed in the nether/end?

Yes, thats a good idea i'll implement that in the 1.1.0 Release


Use a Python version and project manager, UV.

https://medium.com/gitconnected/stop-making-your-python-projects-like-it-was-15-years-ago-125436b470a5

https://astral.sh/blog/uv


Headers are not parsed correctly in ASGI middleware

The ASGI middleware seems to get the raw headers from the scope which are all in byte form. This is used to get/start a transaction, which does not see the sentry-trace header as it checks for the string version of that header.
See this line for the faulty part:
https://github.com/getsentry/sentry-python/blob/master/sentry_sdk/integrations/asgi.py#L133
There is a method that parses the headers and turns it into a Dict[str, str] found here:
https://github.com/getsentry/sentry-python/blob/master/sentry_sdk/integrations/asgi.py#L228

Fixed in #984, thanks for that!

Fixed in #984, thanks for that!


Move counting of term frequencies to function relationships_to_closure_table() that makes closure table

Move counting of frequencies to function relationships_to_closure_table() that makes closure table, instead of doing it on the fly in every entity1/entity2 comparison in calculate_information_content_scores()

closing in favor of #42


Rails 4.2 compatability

I want to update my project from Rails 4.1 to Rails 4.2.
The globalize Readme states:

When using bundler put this in your Gemfile:
gem 'globalize', '~> 5.0.0'

To use the version of globalize for ActiveRecord 4.0 or 4.1, specify:
gem 'globalize', '~> 4.0.3'

So I also updated the globalize gem from 4 to 5. However when I do this I get a dependency error:
$ bundle update activeadmin-globalize globalize
Updating git://github.com/fabn/activeadmin-globalize.git
Fetching gem metadata from https://rails-assets.org/...
Fetching gem metadata from https://rubygems.org/.........
Fetching additional metadata from https://rubygems.org/..
Resolving dependencies...
Bundler could not find compatible versions for gem "globalize":
  In Gemfile:
    activeadmin-globalize (~> 1.0.0.pre) ruby depends on
      globalize (~> 4.0) ruby

    globalize (5.0.0)

Is there generally a way to get activeadmin-globalize to work with rails 4.2

Not yet. I will work on this on the next few days and I will use a travis build matrix in specs to ensure it's working on all released rails versions.

:+1:  Thanks for the heads up

Hi @fabn are there any progress in Rails 4.2 compatibility? I will happy to test a dev branch if you like.
Thank you

Status update on the Rails 4.2 support?

I relaxed dependencies to work with rails 4.2. Now you should be able to install this gem in rails 4.2 using the develop branch.

@fabn Is Rails 5 already on the roadmap?

@besi not yet. I'm pretty busy with work in this period and I don't know when I'll be able to handle that.
Meanwhile if you want to sketch a PR it would be very welcome 😉

Actually the develop branch seems to be working fine:

This is an excerpt from my Gemfile:
I'm using ruby 2.4.1.p111
gem 'rails', '~> 5.1.4'
gem 'activeadmin'
gem 'activeadmin-sortable'
gem 'globalize', git: 'https://github.com/globalize/globalize', branch: 'master'
gem 'activemodel-serializers-xml'
gem 'activeadmin-globalize', github: 'fabn/activeadmin-globalize', branch: 'develop'
gem 'globalize-versioning', github: 'besi/globalize-versioning', branch: 'patch-1'

Sidenote: I am using the globalize-versioning which isn't fully compatible yet. See this issue https://github.com/globalize/globalize-versioning/pull/14


FreeBSD: Xeon CPUs are not detected properly

cpuinfo built from source (HEAD) on FreeBSD 13.3.
Tested on 3 different hardware.

Desktop with i7-2700 - OK:

host-peter% ./build/local/cpuid-dump
CPUID 00000000: 0000000D-756E6547-6C65746E-49656E69 [GenuineIntel]
CPUID 00000001: 000206A7-03100800-1F9AE3BF-BFEBFBFF
CPUID 00000002: 76035A01-00F0B2FF-00000000-00CA0000
CPUID 00000003: 00000000-00000000-00000000-00000000
CPUID 00000004: 1C004121-01C0003F-0000003F-00000000 [SL 00]
CPUID 00000004: 1C004122-01C0003F-0000003F-00000000 [SL 01]
CPUID 00000004: 1C004143-01C0003F-000001FF-00000000 [SL 02]
CPUID 00000004: 1C03C163-03C0003F-00001FFF-00000006 [SL 03]
CPUID 00000005: 00000040-00000040-00000003-00001120
CPUID 00000006: 00000077-00000002-00000009-00000000
CPUID 00000007: 00000000-00000000-00000000-00000000 [SL 00]
CPUID 00000008: 00000000-00000000-00000000-00000000
CPUID 00000009: 00000000-00000000-00000000-00000000
CPUID 0000000A: 07300403-00000000-00000000-00000603
CPUID 0000000B: 00000001-00000002-00000100-00000003 [SL 00]
CPUID 0000000B: 00000004-00000008-00000201-00000003 [SL 01]
CPUID 0000000C: 00000000-00000000-00000000-00000000
CPUID 0000000D: 00000007-00000340-00000340-00000000
CPUID 80000000: 80000008-00000000-00000000-00000000
CPUID 80000001: 00000000-00000000-00000001-28100800
CPUID 80000002: 20202020-49202020-6C65746E-20295228 [       Intel(R) ]
CPUID 80000003: 65726F43-294D5428-2D376920-30303732 [Core(TM) i7-2700]
CPUID 80000004: 5043204B-20402055-30352E33-007A4847 [K CPU @ 3.50GHz]
CPUID 80000005: 00000000-00000000-00000000-00000000
CPUID 80000006: 00000000-00000000-01006040-00000000
CPUID 80000007: 00000000-00000000-00000000-00000100
CPUID 80000008: 00003024-00000000-00000000-00000000
host-peter% ./build/local/cpu-info
Packages:
	0: Intel Core i7-2700K
Microarchitectures:
	4x Sandy Bridge
Cores:
	0: 2 processors (0-1), Intel Sandy Bridge
	1: 2 processors (2-3), Intel Sandy Bridge
	2: 2 processors (4-5), Intel Sandy Bridge
	3: 2 processors (6-7), Intel Sandy Bridge
Logical processors:
	0: APIC ID 0x00000000
	1: APIC ID 0x00000001
	2: APIC ID 0x00000002
	3: APIC ID 0x00000003
	4: APIC ID 0x00000004
	5: APIC ID 0x00000005
	6: APIC ID 0x00000006
	7: APIC ID 0x00000007

Very old DELL server with Xeon - FAILED:

nashost# ./build/local/cpuid-dump
CPUID 00000000: 00000005-756E6547-6C65746E-49656E69 [GenuineIntel]
CPUID 00000001: 00000F41-00020800-0000641D-BFEBFBFF
CPUID 00000002: 605B5001-00000000-00000000-007C7040
CPUID 00000003: 00000000-00000000-00000000-00000000
CPUID 00000004: 00004121-01C0003F-0000001F-00000000 [SL 00]
CPUID 00000004: 00004143-01C0103F-000003FF-00000000 [SL 01]
CPUID 00000005: 00000040-00000040-00000000-00000000
CPUID 80000000: 80000008-00000000-00000000-00000000
CPUID 80000001: 00000000-00000000-00000000-20100800
CPUID 80000002: 20202020-20202020-20202020-20202020 [                ]
CPUID 80000003: 6E492020-286C6574-58202952-286E6F65 [  Intel(R) Xeon(]
CPUID 80000004: 20294D54-20555043-30322E33-007A4847 [TM) CPU 3.20GHz]
CPUID 80000005: 00000000-00000000-00000000-00000000
CPUID 80000006: 00000000-00000000-04006040-00000000
CPUID 80000007: 00000000-00000000-00000000-00000000
CPUID 80000008: 00003024-00000000-00000000-00000000
nashost# ./build/local/cpu-info
Error in cpuinfo: failed to detect topology
failed to initialize CPU information

DELL server with Xeon - FAILED:

srv1# ./build/local/cpuid-dump
CPUID 00000000: 0000000D-756E6547-6C65746E-49656E69 [GenuineIntel]
CPUID 00000001: 000206D7-0A200800-1FBEE3FF-BFEBFBFF
CPUID 00000002: 76035A01-00F0B0FF-00000000-00CA0000
CPUID 00000003: 00000000-00000000-00000000-00000000
CPUID 00000004: 3C004121-01C0003F-0000003F-00000000 [SL 00]
CPUID 00000004: 3C004122-01C0003F-0000003F-00000000 [SL 01]
CPUID 00000004: 3C004143-01C0003F-000001FF-00000000 [SL 02]
CPUID 00000004: 3C07C163-04C0003F-00002FFF-00000006 [SL 03]
CPUID 00000005: 00000040-00000040-00000003-00021120
CPUID 00000006: 00000077-00000002-00000001-00000000
CPUID 00000007: 00000000-00000000-00000000-00000000 [SL 00]
CPUID 00000008: 00000000-00000000-00000000-00000000
CPUID 00000009: 00000000-00000000-00000000-00000000
CPUID 0000000A: 07300803-00000000-00000000-00000603
CPUID 0000000B: 00000001-00000001-00000100-0000000A [SL 00]
CPUID 0000000B: 00000005-00000006-00000201-0000000A [SL 01]
CPUID 0000000C: 00000000-00000000-00000000-00000000
CPUID 0000000D: 00000007-00000340-00000340-00000000
CPUID 80000000: 80000008-00000000-00000000-00000000
CPUID 80000001: 00000000-00000000-00000001-2C100800
CPUID 80000002: 20202020-49202020-6C65746E-20295228 [       Intel(R) ]
CPUID 80000003: 6E6F6558-20295228-20555043-342D3545 [Xeon(R) CPU E5-4]
CPUID 80000004: 20373136-20402030-30392E32-007A4847 [617 0 @ 2.90GHz]
CPUID 80000005: 00000000-00000000-00000000-00000000
CPUID 80000006: 00000000-00000000-01006040-00000000
CPUID 80000007: 00000000-00000000-00000000-00000100
CPUID 80000008: 0000302E-00000000-00000000-00000000
srv1# ./build/local/cpu-info
Error in cpuinfo: failed to detect topology
failed to initialize CPU information

So, pytorch cannot be used correctly on the servers (only one CPU core is used).
On the desktop it uses only one core as well :(

@Peter2121 I only tested the code on FreeBSD 14.0.

@cyyever
host-peter is under 13.3-RELEASE (my PC where I built cpuinfo), srv1 is under 14.0-RELEASE.
I rebuilt cpuinfo locally on this server - no changes.

@Peter2121 What are the output of
sysctl kern.sched.topology_spec

on the failed hosts?

srv1# sysctl kern.sched.topology_spec
kern.sched.topology_spec: <groups>
 <group level="1" cache-level="0">
  <cpu count="12" mask="fff,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0">0, 1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11</cpu>
  <children>
   <group level="2" cache-level="3">
    <cpu count="6" mask="3f,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0">0, 1, 2, 3, 4, 5</cpu>
    <flags><flag name="NODE">NUMA node</flag></flags>
   </group>
   <group level="2" cache-level="3">
    <cpu count="6" mask="fc0,0,0,0,0,0,0,0,0,0,0,0,0,0,0,0">6, 7, 8, 9, 10, 11</cpu>
    <flags><flag name="NODE">NUMA node</flag></flags>
   </group>
  </children>
 </group>
</groups>

nashost# sysctl kern.sched.topology_spec
kern.sched.topology_spec: <groups>
 <group level="1" cache-level="0">
  <cpu count="4" mask="f,0,0,0">0, 1, 2, 3</cpu>
  <children>
   <group level="2" cache-level="2">
    <cpu count="2" mask="3,0,0,0">0, 1</cpu>
    <flags><flag name="THREAD">THREAD group</flag><flag name="SMT">SMT group</flag></flags>
   </group>
   <group level="2" cache-level="2">
    <cpu count="2" mask="c,0,0,0">2, 3</cpu>
    <flags><flag name="THREAD">THREAD group</flag><flag name="SMT">SMT group</flag></flags>
   </group>
  </children>
 </group>
</groups>

@Peter2121 Help check the fix in #249?

The patched version works correctly on srv1.
It does nos work on nashost:
illegal hardware instruction (core dumped)
It does not work on desktop anymore:
Error in cpuinfo: failed to parse topology_spec: <groups>
 <group level="1" cache-level="3">
  <cpu count="8" mask="ff,0,0,0">0, 1, 2, 3, 4, 5, 6, 7</cpu>
  <children>
   <group level="2" cache-level="2">
    <cpu count="2" mask="3,0,0,0">0, 1</cpu>
    <flags><flag name="THREAD">THREAD group</flag><flag name="SMT">SMT group</flag></flags>
   </group>
   <group level="2" cache-level="2">
    <cpu count="2" mask="c,0,0,0">2, 3</cpu>
    <flags><flag name="THREAD">THREAD group</flag><flag name="SMT">SMT group</flag></flags>
   </group>
   <group level="2" cache-level="2">
    <cpu count="2" mask="30,0,0,0">4, 5</cpu>
    <flags><flag name="THREAD">THREAD group</flag><flag name="SMT">SMT group</flag></flags>
   </group>
   <group level="2" cache-level="2">
    <cpu count="2" mask="c0,0,0,0">6, 7</cpu>
    <flags><flag name="THREAD">THREAD group</flag><flag name="SMT">SMT group</flag></flags>
   </group>
  </children>
 </group>
</groups>

Error in cpuinfo: failed to detect topology
failed to initialize CPU information

@Peter2121 Can you git pull and re-check? If you are familiar with valgrind, can you print the valgrind outputs for invocations leading to errors such as "illegal hardware instruction (core dumped)"?

@cyyever
I don't see any commits in master here, so if I revert #249 - I am up-to-date with the initial version.
Please, explain me what version do I need to test?

Test the PR mentioned in this discussion.

Anyway, for the patched version:
nashost# valgrind ./cpu-info
==464== Memcheck, a memory error detector
==464== Copyright (C) 2002-2022, and GNU GPL'd, by Julian Seward et al.
==464== Using Valgrind-3.22.0 and LibVEX; rerun with -h for copyright info
==464== Command: ./cpu-info
==464==
vex amd64->IR: unhandled instruction bytes: 0xC5 0xF8 0x57 0xC0 0xC5 0xFC 0x29 0x84 0x24 0x20
vex amd64->IR:   REX=0 REX.W=0 REX.R=0 REX.X=0 REX.B=0
vex amd64->IR:   VEX=0 VEX.L=0 VEX.nVVVV=0x0 ESC=NONE
vex amd64->IR:   PFX.66=0 PFX.F2=0 PFX.F3=0
==464== valgrind: Unrecognised instruction at address 0x205b76.
==464==    at 0x205B76: cpuinfo_x86_freebsd_init (src/x86/freebsd/init.c:71)
==464==    by 0x4873452: pthread_once (in /lib/libthr.so.3)
==464==    by 0x2058D6: cpuinfo_initialize (src/init.c:28)
==464==    by 0x204772: main (tools/cpu-info.c:291)
==464== Your program just tried to execute an instruction that Valgrind
==464== did not recognise.  There are two possible reasons for this.
==464== 1. Your program has a bug and erroneously jumped to a non-code
==464==    location.  If you are running Memcheck and you just saw a
==464==    warning about a bad jump, it's probably your program's fault.
==464== 2. The instruction is legitimate but Valgrind doesn't handle it,
==464==    i.e. it's Valgrind's fault.  If you think this is the case or
==464==    you are not sure, please let us know and we'll try to fix it.
==464== Either way, Valgrind will now raise a SIGILL signal which will
==464== probably kill your program.
==464==
==464== Process terminating with default action of signal 4 (SIGILL): dumping core
==464==  Illegal opcode at address 0x205B76
==464==    at 0x205B76: cpuinfo_x86_freebsd_init (src/x86/freebsd/init.c:71)
==464==    by 0x4873452: pthread_once (in /lib/libthr.so.3)
==464==    by 0x2058D6: cpuinfo_initialize (src/init.c:28)
==464==    by 0x204772: main (tools/cpu-info.c:291)
==464==
==464== HEAP SUMMARY:
==464==     in use at exit: 2,288 bytes in 6 blocks
==464==   total heap usage: 7 allocs, 1 frees, 2,802 bytes allocated
==464==
==464== LEAK SUMMARY:
==464==    definitely lost: 128 bytes in 2 blocks
==464==    indirectly lost: 0 bytes in 0 blocks
==464==      possibly lost: 0 bytes in 0 blocks
==464==    still reachable: 2,160 bytes in 4 blocks
==464==         suppressed: 0 bytes in 0 blocks
==464== Rerun with --leak-check=full to see details of leaked memory
==464==
==464== For lists of detected and suppressed errors, rerun with: -s
==464== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
zsh: illegal hardware instruction  valgrind ./cpu-info

Ah, I see that your PR was updated! :)
I'll repatch and recheck shortly...

Thank you! Help me check whether it still crashes

After new patch from the PR:
OK on srv1
OK on desktop
Still crashes on nashost:
nashost# valgrind ./cpu-info
==19756== Memcheck, a memory error detector
==19756== Copyright (C) 2002-2022, and GNU GPL'd, by Julian Seward et al.
==19756== Using Valgrind-3.22.0 and LibVEX; rerun with -h for copyright info
==19756== Command: ./cpu-info
==19756==
vex amd64->IR: unhandled instruction bytes: 0xC5 0xF8 0x57 0xC0 0xC5 0xFC 0x29 0x84 0x24 0x20
vex amd64->IR:   REX=0 REX.W=0 REX.R=0 REX.X=0 REX.B=0
vex amd64->IR:   VEX=0 VEX.L=0 VEX.nVVVV=0x0 ESC=NONE
vex amd64->IR:   PFX.66=0 PFX.F2=0 PFX.F3=0
==19756== valgrind: Unrecognised instruction at address 0x2064f3.
==19756==    at 0x2064F3: cpuinfo_x86_freebsd_init (src/x86/freebsd/init.c:71)
==19756==    by 0x4873452: pthread_once (in /lib/libthr.so.3)
==19756==    by 0x2062E2: cpuinfo_initialize (src/init.c:28)
==19756==    by 0x205292: main (tools/cpu-info.c:291)
==19756== Your program just tried to execute an instruction that Valgrind
==19756== did not recognise.  There are two possible reasons for this.
==19756== 1. Your program has a bug and erroneously jumped to a non-code
==19756==    location.  If you are running Memcheck and you just saw a
==19756==    warning about a bad jump, it's probably your program's fault.
==19756== 2. The instruction is legitimate but Valgrind doesn't handle it,
==19756==    i.e. it's Valgrind's fault.  If you think this is the case or
==19756==    you are not sure, please let us know and we'll try to fix it.
==19756== Either way, Valgrind will now raise a SIGILL signal which will
==19756== probably kill your program.
==19756==
==19756== Process terminating with default action of signal 4 (SIGILL): dumping core
==19756==  Illegal opcode at address 0x2064F3
==19756==    at 0x2064F3: cpuinfo_x86_freebsd_init (src/x86/freebsd/init.c:71)
==19756==    by 0x4873452: pthread_once (in /lib/libthr.so.3)
==19756==    by 0x2062E2: cpuinfo_initialize (src/init.c:28)
==19756==    by 0x205292: main (tools/cpu-info.c:291)
==19756==
==19756== HEAP SUMMARY:
==19756==     in use at exit: 2,288 bytes in 6 blocks
==19756==   total heap usage: 7 allocs, 1 frees, 2,802 bytes allocated
==19756==
==19756== LEAK SUMMARY:
==19756==    definitely lost: 128 bytes in 2 blocks
==19756==    indirectly lost: 0 bytes in 0 blocks
==19756==      possibly lost: 0 bytes in 0 blocks
==19756==    still reachable: 2,160 bytes in 4 blocks
==19756==         suppressed: 0 bytes in 0 blocks
==19756== Rerun with --leak-check=full to see details of leaked memory
==19756==
==19756== For lists of detected and suppressed errors, rerun with: -s
==19756== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)
zsh: illegal hardware instruction  valgrind ./cpu-info

IMHO, this is not very important as the server has is 10+ years old. But it would be nice to understand the reason of the crash ;)
I'll try to test on other physical CPUs if I find something with FreeBSD installed.

Tested on several recent servers, no crashes observed.

@Peter2121 The only failure seams like a bug of OS.


When using multiple displays, make layout shortcut key in FancyZones work on: (i) all displays (ii) display with active focus, or (iii) display with mouse cursor.

Description of the new feature / enhancement
I propose that FancyZones apply the layout shortcut key to all displays by default and give the user the option to have it apply to only one display. If the user chooses to have it apply to one display, then the user should be able to choose whether it applies to the display with the mouse pointer or the display with active focus.
Scenario when this would be used?
When I press the layout shortcut key in FancyZones while using multiple displays, the layout is applied to the monitor with the mouse pointer. This is problematic in two situations. The first occurs when I select an active display using alt+tab. The active display is often on a different display than the mouse pointer. After selecting the display using alt+tab, I press the FancyZones layout shortcut key (Win+Ctrl+Alt+[num]) with the expectation that it will apply the layout to the display with the active focus so I can then resize the active window. Instead, it applies the layout to the display with the mouse pointer. I must then move the mouse pointer to the correct display and press the layout shortcut key again.
The second situation occurs when I move a window from one display to another display using the Win+[arrow key] in FancyZones. I often need to change the layout on the display to which the active window is moved so I press the FancyZones layout shortcut key with the expectation that the layout will be applied to the display with the active window. Once again, the layout is applied to the display with the mouse cursor, which is not the one with the active window.
Please give us the option to have FancyZones apply the layout shortcut key to all displays, the display with the active focus, or the display with the mouse cursor.
I added a mockup of one way to implement this in the comments.
Supporting information
No response

I tried Fancy Zones this morning and when I pressed the layout shortcut keys for a layout it was applied to both of my monitors, which was great! I thought this suggestion had been implemented.
Unfortunately, it must have been a short-lived bug. After opening and closing Power Toys, it reverted to the old behavior of applying the layout associated with the layout shortcut key to only the monitor with the mouse pointer. Bummer.

Hi All,
I created the following mockups showing how this can be implemented.

It's been a couple of years. Any chance of this feature being implemented? It would be a great addition, particularly for those of us who move windows from one monitor to another and want the window to remain the same size.


Build Config: Verification

Is your feature request related to a problem? Please describe.
In order to create different versions of the app with different feature sets we need to be able to enable and disable the verification frontend.
This issue requires that #513 and #515 is done.
Describe the solution you'd like
Based on the verification feature flag from #517 we need to show the verification tab. We also need to adjust the permissions of the app like hinted in #430.
Right now the tab is shown or not shown based on the the dart entry point. This logic should be removed.
Describe alternatives you've considered
None
Additional context
None

Done in #517


Support extending base builder classes

This example for factories compiles fine:
interface BaseComponent {
  fun factory(): Factory

  interface Factory {
    fun create(@BindsInstance int: Int): BaseComponent
  }
}

@Subcomponent
interface SomeComponent : BaseComponent {
  override fun factory(): SomeComponent.Factory

  @Subcomponent.Factory
  interface Factory : BaseComponent.Factory {
    override fun create(@BindsInstance int: Int): SomeComponent
  }
}

Notice how I override all the base classes to the concrete types.
 The first I run into is:
[Dagger/MissingBinding] java.lang.CharSequence cannot be provided without an @Provides-annotated method.

After stopping to rely on the binding I see this error:
DaggerParentComponent.SomeComponentBuilder is not abstract and does not override abstract method charSequence(CharSequence) in Builder
  private final class SomeComponentBuilder implements SomeComponent.Builder {

Wrong Dagger repo.
https://github.com/google/dagger

Oh gosh...


Task: Migrate Helpers to Application Layer

Story: #1256
Description
Migrate Helpers to Application Layer

Since helpers are using Microsoft.Azure.Devices package, these classes has been dropped to Infrastructure


Formatter mess up if-else statements indention

Take this input as an example, by running snakefmt, the indent will be wrong.
rule cutadapt:
    input:
        "a.txt"
    output:
        "b.txt"
    run:
        if (
             "xxxxxxxxxxxxxxxxxxxxxx" not in wildcards.rn
               and "yyyyyyyyyyyyyyyyyyyyyyyyyy" not in wildcards.rn
           ):
            shell(
            """
            cutadapt \
                -m 30 \
                {input} \
                -o {output}
            """
            )
        else:
            shell(
            """
            cutadapt \
                {input} \
                -o {output}
            """
            )

New space will be added on each run.

I figure out that the cause is from the if statment, which is too long.

This is probably a result of #151 I suspect. I'll try and take a look at it this week sorry.

This is not related to #151 in the end. It seems a bit trickier. Leave it with me and I'll try get to the bottom of it soon.

This will be fixed in the next release


feat: powershell toolbox improvements

Improve PowerShell toolbox experience
Relates to #11
Description
Improve the UX experience of the new PowerShell toolbox.
Proposed Changes

Extend package list with various CLI utils

bat
eza
fd
fzf
gawk
jq
ripgrep
sed
zoxide

Fix missing PowerShell modules issue by installing PowerShell as a .NET Global tool

Add initial PowerShell profile with the following features:

Oh My Posh integration with mt theme
Zoxide integration
McFly integration via CTRL-r
PSReadline integration
PSFzf integration via CTRL-t and CTRL-f

Autocomplete for Azure CLI
Autocomplete for .NET (dotnet cli)
Update README

Notes
The initial launch of pwsh will take longer than usual since some utils need to be downloaded.

I think this is now ready for merge. Since the last review I made some adjustments.
I was particularly worried about copying the PowerShell profile to the user home since the user might already have a profile. If used with distrobox without a custom home that could have overridden their custom profile. Now I am correctly using the top level profile that will only get executed if the user does not already have any profiles.
Furthermore, keeping the same thought as above I changed the folder where the shell themes get installed to avoid polluting the user's home directory.
Lastly, I did some config updates to the history viewer to display more results out of the box.
It would be great if someone can test this to get some feedback but for me, it works quite well as an initial template that can be extended later on.


Ability to set auto generated password during installation and force reset during first login

Briefly describe the feature

As a paralus user, I would like to set an auto-generated password during installation thus avoid admin to reset the password after installation
As an admin, when I first login I would like to reset by password before logging in.
Applicable while creating new users via dashboard, cli

What problem does this feature solve? Please link any relevant documentation or Issues

Improves paralus usage for new users

Tasks to complete this feature:

Update Kratos library SDK in Paralus core (New Kratos library supports accepting password in account creation) [@akshay196 ]
Auto generate password in Paralus init script [@rustiever ]
Helm chart changes
Dashboard changes (Detect "admin" use is login for first time and ask to set password)

New PRs:

https://github.com/paralus/paralus/pull/124
https://github.com/paralus/dashboard/pull/152
https://github.com/paralus/helm-charts/pull/70

This is completed and merged. Available with Paralus helm version v0.2.1


CULane Data Set - Empty folders

I tried downloading the data set (from https://drive.google.com/drive/u/0/folders/1mSLgwVTiaUMAb4AVOWwlCD5JcWdrwpvu) and I downloaded driver_161_90frame.tar.gz but when I open it 4 folders are empty:
EDIT: I looked at the photos before and after the files and I think they were just deleted because its just a car sitting parked. Confused though because there's label images for all of those deleted photos
EDIT 2: Ok the two folders before and after are also missing images, I'm guessing it's just to censor something so this is just a PSA for anyone using the data set

@GreatGameDota Those images should be deleted deliberately when we built the dataset due to bad image quality, and are not in the training list.

@GreatGameDota Those images should be deleted deliberately when we built the dataset due to bad image quality, and are not in the training list.

Thanks for the reply! In the label file there are still images in these files even though there is no training image to train on.


[v2.0.6] bluebird warnings

My express application starts with this line  global.Promise = require("bluebird");. And when postcss-loader runs bluebird spews out this warning:
(node:13276) Warning: a promise was created in a handler at C:\Programming\HtmlProjects\Shadow\node_modules\postcss-loader\lib\index.js:173:16 but was not returned from it, see http://goo.gl/rRqMUw     at new Promise (C:\Programming\HtmlProjects\Shadow\node_modules\bluebird\js\release\promise.js:79:10)
The same warning is repeated about 15 times.

@czaarek99 what is version postcss-loader your use ? Because on https://github.com/postcss/postcss-loader/blob/master/lib/index.js#L173 line there is no promises which return something

I'm using version 2.0.6

@czaarek99 can your create minimum reproducible test repo?

What's the status here ? Maybe related to #269, but postcss-loader won't receive any changes for custom libs unrelated to itself, especially if an ECMA Standard exists

@michael-ciniawsky Yeah, that's for sure the same issue that I'm having.

I have the same problem，postcss-loader version is "^3.0.0"， how to solve it，thanks~


Test end without driver closing?

Is there a way to prevent the driver from closing when a test ends? If no such functionality exists, I would be happy to contribute!

What is the use case? Is it to reuse the driver to speed up tests?
All PR's are welcome


Some regressions in latest update

Expected Behavior
The previous update to comfy fixed the problem in every browser, including chrome browsers. Shortly later there was another update pushed, now stableswarm is very slow, unresponsive.  Not a total loss like yesterday. but, now it takes about 30-60 seconds for the workflow tab to become responsive. this happens everytime one refreshes  the page.
Waterfox logs:
unreachable code after return statement
[Reflect.js:897:21](http://localhost:7801/node_modules/reflect-metadata/Reflect.js)
unreachable code after return statement
[Reflect.js:1029:21](http://localhost:7801/node_modules/reflect-metadata/Reflect.js)
unreachable code after return statement
[Reflect.js:1288:21](http://localhost:7801/node_modules/reflect-metadata/Reflect.js)
unreachable code after return statement
[Reflect.js:1354:21](http://localhost:7801/node_modules/reflect-metadata/Reflect.js)
unreachable code after return statement
[Reflect.js:897:21](http://localhost:7801/node_modules/reflect-metadata/Reflect.js)
unreachable code after return statement
[Reflect.js:1029:21](http://localhost:7801/node_modules/reflect-metadata/Reflect.js)
ComfyUI Front-end version: 1.2.26 [index.html:4:11](http://localhost:7801/index.html?html-proxy&index=1.js)
XHRGET
[http://localhost:7801/ComfyBackendDirect/api/userdata/workflows/.index.json](http://localhost:7801/ComfyBackendDirect/api/userdata/workflows%2F.index.json)
[HTTP/1.1 404 Not Found 8911ms]

GET
http://localhost:7801/ComfyBackendDirect/extensions/dzNodes/dz_DZ_widgets.js

unreachable code after return statement
[Reflect.js:897:21](http://localhost:7801/node_modules/reflect-metadata/Reflect.js)
unreachable code after return statement
[Reflect.js:1029:21](http://localhost:7801/node_modules/reflect-metadata/Reflect.js)
unreachable code after return statement
[Reflect.js:1288:21](http://localhost:7801/node_modules/reflect-metadata/Reflect.js)
unreachable code after return statement
[Reflect.js:1354:21](http://localhost:7801/node_modules/reflect-metadata/Reflect.js)
unreachable code after return statement
[Reflect.js:1288:21](http://localhost:7801/node_modules/reflect-metadata/Reflect.js)
unreachable code after return statement
[Reflect.js:1354:21](http://localhost:7801/node_modules/reflect-metadata/Reflect.js)
Loading module from “http://localhost:7801/ComfyBackendDirect/extensions/dzNodes/dz_DZ_widgets.js” was blocked because of a disallowed MIME type (“text/plain”).
[ComfyBackendDirect](http://localhost:7801/ComfyBackendDirect/)
Loading failed for the module with source “http://localhost:7801/ComfyBackendDirect/extensions/dzNodes/dz_DZ_widgets.js”. [ComfyBackendDirect:20:68](http://localhost:7801/ComfyBackendDirect/)
Error loading extension /extensions/dzNodes/dz_debug.js TypeError: error loading dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/dzNodes/dz_debug.js [app.ts:1756:20](http://localhost:7801/src/scripts/app.ts)
Error loading extension /extensions/ComfyUI-Impact-Pack/impact-wildcard.js TypeError: refresh_btn2 is null
    <anonymous> http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-Impact-Pack/impact-wildcard.js:14
[app.ts:1756:20](http://localhost:7801/src/scripts/app.ts)
GET
http://localhost:7801/Error/404

Loading module from “http://localhost:7801/Error/404” was blocked because of a disallowed MIME type (“text/html”).
[ComfyBackendDirect](http://localhost:7801/ComfyBackendDirect/)
Loading failed for the module with source “http://localhost:7801/scripts/app.js”. [ComfyBackendDirect:20:68](http://localhost:7801/ComfyBackendDirect/)
Error loading extension /extensions/ComfyUI-Crystools/monitor.js TypeError: error loading dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-Crystools/monitor.js [app.ts:1756:20](http://localhost:7801/src/scripts/app.ts)
Error loading extension /extensions/ComfyUI-Crystools/progressBar.js TypeError: error loading dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-Crystools/progressBar.js [app.ts:1756:20](http://localhost:7801/src/scripts/app.ts)
Error loading extension /extensions/ComfyUI-Crystools/debugger.js TypeError: error loading dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-Crystools/debugger.js [app.ts:1756:20](http://localhost:7801/src/scripts/app.ts)
Error loading extension /extensions/ComfyUI-Crystools/extensions.js TypeError: error loading dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-Crystools/extensions.js [app.ts:1756:20](http://localhost:7801/src/scripts/app.ts)
GET
http://localhost:7801/Error/404

GET
http://localhost:7801/Error/404

Loading module from “http://localhost:7801/Error/404” was blocked because of a disallowed MIME type (“text/html”).
2 [ComfyBackendDirect](http://localhost:7801/ComfyBackendDirect/)
Loading failed for the module with source “http://localhost:7801/scripts/widgets.js”. [ComfyBackendDirect:20:68](http://localhost:7801/ComfyBackendDirect/)
Loading failed for the module with source “http://localhost:7801/scripts/api.js”. [ComfyBackendDirect:20:68](http://localhost:7801/ComfyBackendDirect/)
Error loading extension /extensions/ComfyUI-Crystools/liteGraph.js TypeError: error loading dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-Crystools/liteGraph.js [app.ts:1756:20](http://localhost:7801/src/scripts/app.ts)
Error loading extension /extensions/ComfyUI-Crystools/common.js TypeError: error loading dynamically imported module: http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-Crystools/common.js [app.ts:1756:20](http://localhost:7801/src/scripts/app.ts)
GET
http://localhost:7801/Error/404
[HTTP/1.1 404 Not Found 1ms]

Loading failed for the <script> with source “http://localhost:7801/kjweb_async/marked.min.js”. [ComfyBackendDirect:1:1](http://localhost:7801/ComfyBackendDirect/)
Object { status: false, message: "Failed to load the script /kjweb_async/marked.min.js" }
[help_popup.js:42:11](http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-KJNodes/js/help_popup.js)
GET
http://localhost:7801/Error/404
[HTTP/1.1 404 Not Found 1ms]

GET
http://localhost:7801/Error/404
[HTTP/1.1 404 Not Found 0ms]

GET
http://localhost:7801/Error/404
[HTTP/1.1 404 Not Found 1ms]

Loading failed for the <script> with source “http://localhost:7801/kjweb_async/purify.min.js”. [ComfyBackendDirect:1:1](http://localhost:7801/ComfyBackendDirect/)
Object { status: false, message: "Failed to load the script /kjweb_async/purify.min.js" }
[help_popup.js:45:11](http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-KJNodes/js/help_popup.js)
Loading failed for the <script> with source “http://localhost:7801/kjweb_async/svg-path-properties.min.js”. [ComfyBackendDirect:1:1](http://localhost:7801/ComfyBackendDirect/)
Object { status: false, message: "Failed to load the script /kjweb_async/svg-path-properties.min.js" }
[point_editor.js:76:11](http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-KJNodes/js/point_editor.js)
Loading failed for the <script> with source “http://localhost:7801/kjweb_async/protovis.min.js”. [ComfyBackendDirect:1:1](http://localhost:7801/ComfyBackendDirect/)
Object { status: false, message: "Failed to load the script /kjweb_async/protovis.min.js" }
[point_editor.js:79:11](http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-KJNodes/js/point_editor.js)
XHRGET
http://localhost:7801/ComfyBackendDirect/api/DZ/debug
[HTTP/1.1 404 Not Found 2ms]

Error calling extension 'DZ.widgets' method 'init'
Object { error: ReferenceError }

Object { extension: {…} }

Object { args: [] }
[app.ts:383:20](http://localhost:7801/src/scripts/app.ts)
Unhandled message: {"type": "crystools.monitor", "data": {"cpu_utilization": 26.8, "ram_total":<PHONE_NUMBER>2, "ram_used":<PHONE_NUMBER>8, "ram_used_percent": 46.1, "hdd_total":<PHONE_NUMBER>016, "hdd_used":<PHONE_NUMBER>792, "hdd_used_percent": 92.3, "device_type": "cuda", "gpus": [{"gpu_utilization": 34, "gpu_temperature": 58, "vram_total":<PHONE_NUMBER>8, "vram_used":<PHONE_NUMBER>, "vram_used_percent": 13.585631052652994}]}} Error: Unknown message type crystools.monitor
    createSocket_fn api.ts:226
    createSocket_fn api.ts:138
    init api.ts:240
    addApiUpdateHandlers_fn app.ts:1688
    setup app.ts:1874
    fulfilled index-D8Zp4vRl.js:55
    promise callback*step index-D8Zp4vRl.js:67
    fulfilled index-D8Zp4vRl.js:55
    promise callback*step index-D8Zp4vRl.js:67
    fulfilled index-D8Zp4vRl.js:55
    promise callback*step index-D8Zp4vRl.js:67
    __async index-D8Zp4vRl.js:68
    __async index-D8Zp4vRl.js:52
    setup app.ts:1854
    setup GraphCanvas.vue:73
    __async index-D8Zp4vRl.js:68
    __async index-D8Zp4vRl.js:52
    setup GraphCanvas.vue:58
    createHook runtime-core.esm-bundler.js:1808
    callWithErrorHandling runtime-core.esm-bundler.js:195
    callWithAsyncErrorHandling runtime-core.esm-bundler.js:202
    __weh runtime-core.esm-bundler.js:1788
    flushPostFlushCbs runtime-core.esm-bundler.js:379
    render2 runtime-core.esm-bundler.js:5805
    mount runtime-core.esm-bundler.js:3062
    mount runtime-dom.esm-bundler.js:1530
    <anonymous> main.ts:40
[api.ts:231:16](http://localhost:7801/src/scripts/api.ts)
Skipping invalid node definition: Stability Search and Replace. See debug log for more information. [api.ts:271:18](http://localhost:7801/src/scripts/api.ts)
Invalid ComfyNodeDef: {"input":{"required":{"image":["IMAGE"],"search_prompt":["STRING",{"multiline":true},"Search Prompt"],"prompt":["STRING",{"multiline":true}]},"optional":{"negative_prompt":["STRING",{"multiline":true}],"seed":["INT",{"default":0,"min":0,"max":4294967294}],"api_key_override":["STRING",{"multiline":false}],"output_format":[["png","webp","jpeg"]]}},"input_order":{"required":["image","search_prompt","prompt"],"optional":["negative_prompt","seed","api_key_override","output_format"]},"output":["IMAGE"],"output_is_list":[false],"output_name":["IMAGE"],"name":"Stability Search and Replace","display_name":"Stability Search and Replace","description":"","python_module":"custom_nodes.ComfyUI-SAI_API","category":"Stability","output_node":false}
Validation error: Array must contain at most 2 element(s) at "input.required.search_prompt" [api.ts:274:18](http://localhost:7801/src/scripts/api.ts)
XHRGET
[http://localhost:7801/ComfyBackendDirect/api/userdata/workflows/.index.json](http://localhost:7801/ComfyBackendDirect/api/userdata/workflows%2F.index.json)
[HTTP/1.1 404 Not Found 1ms]

[🐍 pysssss] Skipping LinkRenderMode as core extension found [linkRenderMode.js:9:12](http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-Custom-Scripts/js/linkRenderMode.js)
XHRPOST
http://localhost:7801/ComfyBackendDirect/api/DZ/debug
[HTTP/1.1 405 Method Not Allowed 4ms]

XHRHEAD
http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-Custom-Scripts/js/assets/favicon-active.user.ico
[HTTP/1.1 404 Not Found 4ms]

XHRGET
http://localhost:7801/ComfyBackendDirect/api/pysssss/autocomplete
[HTTP/1.1 404 Not Found 1ms]

XHRHEAD
http://localhost:7801/ComfyBackendDirect/extensions/ComfyUI-Custom-Scripts/js/assets/favicon.user.ico

Actual Behavior
Slow response time
Steps to Reproduce
Open program
Debug Logs
https://paste.denizenscript.com/View/125655
Other
No response

It's working as intended and loading fine for me in both Firefox and Chrome.
Your logs have errors from multiple different custom node packs, including dzNodes among others, there might be an error sourcing from one of those rather than Swarm or Comfy itself?

Its working almost perfectly fine in comfyui standalone, a 15 second delay opening the default workfly. but it takes 30-60 secondst to load in swarmui. many of swarmui ui's are broken and bugged out.

Mercury is in retrograde

Could be partly an upstream issue because comfyui is about 10-12 seconds slower loading than usual on its own. If i find anything else Ill post it and if you have any suggestions ill take a look, im off for not though.

I find upstream change to be far more likely.
There have only been two changes in Swarm's code that could possibly be relevant (considering a very recent revision worked fine):

1: I added tooltips to Swarm's comfy nodes, because the new comfy frontend supports those now
2: I tweaked the code to decide how to position the Swarm header on the comfy tab https://github.com/mcmonkeyprojects/SwarmUI/commit/6a8290196b3aed5b141173eb371da7037a91f63e which only changes anything if you have the comfy beta topbar enabled.

If you updated Swarm, but not comfy, when trying the swarm compat fix initially, you may have been hit by something inside https://github.com/comfyanonymous/ComfyUI/commit/b021cf67c7517aed914810d0f9dae3343d3305d4 which is a bulk pull to comfy of all the recent Comfy_Frontend code tweaks of the past few days.
If you updated both at the time, that leaves only custom nodes as potentially relevant

I have a batch file that auto git pulls all my ai git apps in one go, I ran a git pull outside of swarm using the update, it triggers the update batch file in your dir; then it does the same to comfyui's update.bat,
it said unpacking like 97 files or something, for swarmui. under D:\AI\Image\StableSwarmUI.git\objects*  as well as moving or modifying saved workflow jsons

Whatever that screenshot is, and what you claim was said by the thing you ran, are thoroughly not part of how anything in Swarm or Comfy natively works.

After a full night sleep the problem appears to be solved, i blame mercury


fixed SnackBarBehavior.floating when using it with bottomSheet

Description
This Pr fixes the Snackbar showing beneath a bottomSheet despite having SnackBarBehavior.floating
Related Issues
fixes: #63254
Tests
Added three test in snack_bar_test.dart
All test passed in snack_bar and as well as scaffold
Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[ ] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.

Sorry for the inconvenience, closing this now.


.local_repo/releases is cleaning when skip_bundle

always cleaning .local_repo/release directory.

thanks!

modified a bit https://github.com/sonots/capistrano-bundle_rsync/commit/5a3fa91fe3b638146e3e97ffed0791788752305f, should be essentially same.

released 0.4.7 !!

thanks!


[FancyZones Editor - Edit Custom layout]: 'Select a key that quickly apply the layout' combo box does not get activated/accessible via keyboard using ‘Enter/space’ key.

Microsoft PowerToys version
"0.35.0"
Running as admin

[ ] Yes

Area(s) with issue?
FancyZones Editor
Steps to reproduce
Test Environment:
OS: Windows 10 Build 21354.1
App version: 0.35.0
App name: PowerToys
Screen Reader: Narrator
Tool: Accessibility Insight for Windows
Repro Steps:

Launch 'PowerToys' application.
Application will get open with default list item 'General' selected.
Navigate to 'FancyZones' at the left side of the pane and activate it.
Now navigate to 'Launch layout editor button and activate it.
FancyZones editor window will get open.
Navigate to 'Edit layout' button of any layout (say Grid) under Custom and activate it.
Edit layout dialog gets open.
Navigate to 'Select a key that quickly apply the layout' combo box and press 'enter' key to activate it.
Observe the issue.

User Impact: Priority-0
If the content is not operated through a keyboard or keyboard interface, it would not be operable by people with no vision, Some people with hand tremors find using a mouse very difficult , as well as by people who must use alternate keyboards or input devices that act as keyboard emulators. Individuals with low vision would also find it difficult to control it from the keyboard.
MAS Reference:
https://microsoft.sharepoint.com/:w:/r/sites/accessibility/_layouts/15/Doc.aspx?sourcedoc={1F29ACF9-A3E7-414F-BD8D-7E3598D877BB}&file=MAS 2.1.1 – Keyboard.docx&wdLOR=c6FA267E7-81CB-46D6-A357-F39B5ABB492F&action=default&mobileredirect=true&cid=1e2d4435-a25f-4bd0-a407-38d21d38326e
MAS2.1.1_Custom-edit layout dialog_combo box not accessible via keyboard.zip
✔ Expected Behavior
'Select a key that quickly apply the layout' combo box should get activated/accessible via keyboard using ‘Enter/space’ key.
❌ Actual Behavior
'Select a key that quickly apply the layout' combo box does not get activated/accessible via keyboard using ‘Enter/space’ key.
Other Software
No response

Resolved with the release of v0.41.


--exclude-tags-regex doesn't work correct

Hi! I'd like to exclude minor versions from my changelog.
this is a flag what I use with --exclude-tags-regex ".*\..*\..*\..*"
but
Warning: unable to reject any tag, using regex nil in --exclude-tags option.
What can I do with it?

".*\..*\..*\..*" => How can this be a regex for git tags ?


Remove GOADA Decentralized exchange

I'm here to ask to remove GOADA from the decentralized exchange list.
While at one point it was running on testnet we had some problems and the team broke up.
GOADA is going to close and never see mainnet.

thanks for letting us know


Training module: ASL/LSQ video button

Prerequisites

[x] I've searched for existing issues

Describe the bug
The two sign language buttons are really confusing when next to each other - they look like duplicates.
Expected behavior
Reduce the space between first sign language button and the question. Put the second sign language video under the answers.

@cherylhjli I agree that the placement isn't the best. I sort of left it like this though because our conventions in the site for forms were to have the video under the text and before the options. However, in our other cases it is a single video. This is something worth thinking about in conjunction with #1546

@jobara as per our discussion, the conventions make sense, but unfortunately we have 2 videos instead of 1, which is what makes it confusing! I think this is a temporary solution until we can address it fully in #1546.


[Bug] Settings didn't save properly

Settings are not saved after completely exiting the application and restarting it.
"Dymatic part creation" can't save. Other settings might require multiple restarts (2-3 times) to be saved.
===
ab-download-manager v1.0.6
Window 10 21H2 x64

@FiorenMas Thank you. I fixed dynamic part creation setting not saved on app exit
But

Other settings might require multiple restarts (2-3 times) to be saved.

I don't face that problem can you tell me how can I reproduce this ?

Thank you for fixing the bug. The issue with other settings not saving is infrequent, and I haven't been able to consistently reproduce it. I will monitor this issue and report back with more details if it happens again.


Set PILOT_ENABLE_METADATA_EXCHANGE to false does not remove tcp MX filter in 1.12

Bug Description
We set PILOT_ENABLE_METADATA_EXCHANGE to false to disable metadata exchange filters. It has been working fine in 1.11.
We recently upgraded to 1.12 and found out that the flag is no longer removing TCP metadata exchange filter.
See a sample config dump
         "filter_chain_match": {
          "destination_port": 8080,
          "transport_protocol": "tls"
         },
         "filters": [
          {
           "name": "istio.metadata_exchange",
           "typed_config": {
            "@type": "type.googleapis.com/envoy.tcp.metadataexchange.config.MetadataExchange",
            "protocol": "istio-peer-exchange"
           }
          },

We confirmed that the env var is PILOT_ENABLE_METADATA_EXCHANGE:                  false in istiod pods.
Version
Works in Istio 1.11.4
Broke in Istio 1.12.6

Additional Information
No response

can you share the output of kubectl get envoyfilter -nistio-system?

@zirain - I can confirm that we don't have any EnvoyFilter that inserts "istio.metadata_exchange"

seems related to this https://github.com/istio/istio/pull/36756, need to verify.

We can. Are you running in to any issue because of this?

We found this during upgrade so did not have production impact. We have found a way to work around this - but it's appreciated if we can back-port the fix into v1.12!


feature: complete the funtion of runtime status

Signed-off-by: Starnop<EMAIL_ADDRESS>
Ⅰ. Describe what this PR did
complete the funtion of runtime status in cri.go
Ⅱ. Does this pull request fix one issue?

None
Ⅲ. Describe how you did it
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

LGTM


_get_full_indent undefined

Please rename _get_full_indent to get_full_indent; else code is throwing undefined _get_full_indent.

@adityacoder007 Please describe how to reproduce the error you are seeing.

Ah, I see.  https://github.com/beautify-web/js-beautify/blob/2f8e4f050200b85f3c3b9fb5a292823b52c618ac/js/src/html/beautifier.js#L452
The case where _beautifier is not defined is not covered by tests.


Custom Stage: adding PCs as signatures

Right now, we always add the PCs as invitees, readers and signatures of all CustomStage invitations.
There was an issue where the PC submitted the review revision of a review. This resulted in an error when invitation_edit_process.py tried to update all invitations and notes.
Two solutions:

Allow PCs to be signatures always in the CustomStage and fix invitation_edit_process.py. The only issue with this, is that if the field has readers "${5/signatures}",then the readers will include the PCs twice, since the PCs are the signatures.
Add PCs as Participants of the PC stage so we can decide when to add them as signatures and when not to.

@melisabok thoughts?

I do think PCs should be able to do anything, maybe in the CustomStage they are invitees, readers but not signatures. They should use the committee group id to sign the edit.
I would also fix the process function and get the number from the forum note, I wouldn't rely on the signatures id or invitation id.


Feature: watched status

Issue description
Provide an indication to the users if a TV-show/episode or movie has already been watched. Currently there is no way to see which movies/TV-show episodes have been watched (except when watching a show).
Things to consider

Profiles should be considered
When new episodes in a TV-show launch, the show should be re-added to continue watching and the watched status for the episodes should be: not watched.
A button to make an already watched show/episode/movie, unwatched again.
The watched status should also be visible for individual episodes in the shows info view.
Additionally a show could indicate how many unwatched episodes there are in the show. (picture below)

Example design
I'm not an UX designer but this is how it could look (please use a little imagination).
Unwatched episodes in show (from plex)

having a same question! thanks for the query.

This would be a really nice feature to have! +1

good feature :) i'll add it to the next release

Still Open:

When new episodes in a TV-show launch, the show should be re-added to continue watching and the watched status for the episodes should be: not watched.
The watched status should also be visible for individual episodes in the shows info view.
Additionally a show could indicate how many unwatched episodes there are in the show. (picture below)

Would it be possible to fix the issue of 'Mark Completed' not working for Tv shows?

In addition to the above a ‘mark watched’ button in addition to the ‘mark unwatched’ would maybe be a good idea as well.

v1.9.2 does have an 'unwatch' feature which I believe is supposed to be functionally the same as completed as this used to effectivly set the asset to unwatched. However v1.9.2 really breaks this with tv shows as they now appear on my watch list and you cannot remove them. Even if you delete the show an empty asset appears on your watchlist and cannot be removed. I tried to remove these directly from the database but have managed to corrupt the H2 files. Looking on how to move the db to mySQL..... :-(


Use Twisted for VLC communication

Known issues: there is a noticeable lag (~ 1 second) between the start of Syncplay MainWindow and the start of VLC.

It's slightly faster than that for me on Windows - there is only usually a notable delay if I'm running VLC for the first time.

Thanks for your work on this Alberto.

This may have broken VLC support for some, my friend coudn't use VLC with Syncplay v1.6.6 anymore. Syncplay would just simply not find the running VLC instance and it behaved as nothing was connected.
Downgrading to v1.6.5 fixed it

@smiba Thanks for bringing this to our attention. Please provide the following information on how to reproduce this issue:

What version of VLC was used.
What Operating System was used.
By what means was Syncplay installed.
What do you mean by: "Syncplay would just simply not find the running VLC instance and it behaved as nothing was connected."

@smiba Please see #365 for a discussion of this issue.

The issue raised by @smiba is hopefully resolved by the fix discussed in #365.


Fix for order of skew and scale in transform, plus a test

This is the fix for #3163

OK, what do I think about it: we should use SHEAR, and not SKEW. Make skew a property. Look at the code, its horrible! Game developers won't understand what is skew, while shear is just a rotation of axis X and axis Y

Oh, I forgot to say: it is a breaking change. I don't want to add "doSkewBeforeScale" flag, its obvious that it must be made that way.

I need to test against pixi-animate, will get back with results.


fix(client): fix VPLink attrs

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-1 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 40.382%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/client/src/components/VPLink.ts
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
675

Relevant Lines:
1716

💛  - Coveralls


RainFARM : Terzago et al. 2018

I wanted to ask you if you could add the version of RainFARM improved by Terzago et al.(https://doi.org/10.5194/nhess-18-2825-2018).
I would like to use it using Python. By the way, I am open to collaborate and help you to do it.

Hello @simonbeylat
I'm not familiar with the work of Terzago et al, can you please briefly summarize what did they improve with respect to the original RainFARM?
In any case, we would of course welcome your contribution! You can submit a draft of this new RainFARM version as a pull request following our developer guidelines. We will then support you during the review your code, by suggesting code changes and contributing with code to make sure that your contribution will be properly integrated in pysteps.
Before starting your implementation, it might be worth discussing some higher level details here. For example, should this become a separate method, an option in the existing RainFARM method, or simply replace some existing code? For this, I come back to my initial question: please try to describe in more detail what you'd like to do and how you propose to implement them.

Hello @dnerini,
Thank you for your reply.
Terzago has added 2 elements to RainFARM. The first is a smoothing operator at the end of the algorithm.
The second (and most important) is a weight applied at the fine scale of the field, before the last step.
These weights are calculated from the climatology. It allows RainFARM to reproduce the climatology of fine-scale precipitation.
The aim is to provide RainFARM with complex orographic information in order to reproduce a realistic fine-scale precipitation field.
I think we can follow what von Hardenberg (one of the authors of the RainFARM articles) has done in R and Julia and add an option to the current function. (https://github.com/jhardenberg/rainfarmr).

I like your proposition, @simonbeylat! If possible, I'd keep the two changes separate, hence work on two pull requests. The smoothing operator looks like a fairly straightforward addition, while the weighting is somewhat more involved, as it requires a climatological analysis. In this sense, I assume we would need to implement the methods to compute such weights based on a data archive?
Finally, I was  thinking we should get in touch with @jhardenberg, perhaps he's interested in contributing to a python version of his code within pysteps?

Great, it's a good idea to see if von Hardenberg is interested.
So I think we can do a first pull request for the smoothing operator where we create a little internal function to do that, and add a boolean parameter called Smoothing to the RainFARM function (downscale) and add this smoothing option to the main function.
A second pull request with a function to compute the weight that has as parameter the climatology we want to use, the output of this function would be the weights.
Then add a parameter called Weight to the RainFARM function with None as the default value, in case we don't want to use the weight, if that's the case you can just set this parameter to the output of the weight function.
Do you like it this way?
For the time component, it will be interesting to work on, but maybe we need to think a bit more about how we can implement it.
I assume that you used this environment to develop this package. Do you think I can use the xarray library, which I like, to read NetCDF files? Otherwise I would adapt

Hi folks, I have actually seen this development and it would be really cool to have a python implementation of the more recent developments. The Terzago et al. 2018 version actually differs from the original Rebora et al. 2006 version also in another detail: we introduced (actually already in D'Onofrio et al 2014 https://doi.org/10.1175/JHM-D-13-096.1 ) a 'spectral merging' feature which greatly increases the realism of the resulting fields. Indeed this occured at the cost of dropping the downscaling in time. Actually, I will give a look at the current implementation in pysteps and the come back with some comments.

The time downscaling was easy to implement in the original Rebora et al. 2006 by using 3D FFTs, but actually in D'Onofrio et al. 2014 we dropped it for climate downscaling when we introduced the spectral merging. Indeed the entire concept of downscaling in time made a lot sense at the weather event scale, but is less well defined for climate fields (should we do a fft in time over all the length of the simulation ? Only over pieces , which pieces? ). Also, as long as daily precipitation is considered, correlations in time are guaranteed by the conservation of the large-scale field by RainFARM (we do not expect significant subgrid correlations in time at the subdaily scales if we downscale for example from 50 to 1 km ....) so it is not needed.

Thank you for your reply @jhardenberg
In this case, it might be better to start by implementing this "spectral fusion" of D'Onofrio et al 2014. To follow the same historical development.

Very good @simonbeylat, I very much like your plan and look forward to review and contribute to your PRs!

I assume that you used this environment to develop this package.

For development you can use the dev environment. You should be able to follow these instructions, let me know if you encounter any problem.

Do you think I can use the xarray library, which I like, to read NetCDF files? Otherwise I would adapt

The introduction of xarray has been a long-standing issue, but as of today, we still haven't adapted pysteps (it's work in progress although somewhat stale unfortunately...) . So this means that we would prefer not to add xarray as a dependency (not for now), but this doesn't mean that you shouldn't use it for developing and testing your implementation locally. In the end, the methods should accept a numpy array, which you can always easily get from xarray, if you see what I mean.

Also (and apologies for the slow answer), hi @jhardenberg and thanks for your inputs! Very nice that you're interested in contributing to this effort.
It's very interesting what you mention about the downscaling in time, it sure does make a lot of sense. I wonder whether we should still try to implement that component for weather applications, though. Something to keep in mind...
@simonbeylat how do you plan to proceed?

Hello @dnerini,
I have some important things to finish (with deadline) for the next few weeks and then I'll have a few days off.
I think I'll focus fully on this in September, if that's okay for you ?

Absolutely, there is no hurry, just wanted to have an idea ;-)

Hi @dnerini
I am about to start the implementation, if I remember correctly we have 3 different implementations in 3 different pull requests (respecting what we said in July):

1) Spectral fusion from D'Onofrio et al 2014. Directly integrated with the main function.

2) The smoothing operator added with a boolean parameter called Smoothing to the RainFARM function (downscale) and a function that does it.

3) The weight function (which gives the weights using a climatology as input) and the weight parameter (where we can put the weights computed by the function).

Is it okey for you ?

Hi @simonbeylat, thank you again for the initiative, that is great.
If I may suggest I would indeed start from the smoothing operator, which looks like a quite easy change. A reference implementation can be found both in my R and in my julia implementations. Notice that in both cases the smoothing is implemented as a convolution using the trick of transforming it into a product in spectral space. A tricky thing to pay attention to is the treatment of missing values, as you will see in both cases.

Hey @simonbeylat very nice to hear from you ! The plan sounds very good, I look forward to see and discuss your pull requests!
Also, the suggestion from @jhardenberg sounds very reasonable, what do you think? Shall we start with the smoothing operator?
Final note, do not hesitate to draft a pull request early on with a preliminary version of your code: this often helps to get  feedback and get things right from the beginning (and avoid dead ends).

Hi @dnerini
I have created a first pull request with the smoothing operator.
I used @jhardenberg's R script to do it! I tested it with the same input and got the same result (with R and Python).
I'm waiting for your comments and starting the next pull request which should be the "spectral fusion of D'Onofrio et al 2014. Directly integrated with the main function. "

Hi @simonbeylat, I agree, the spectral fusion can be added as an option to the main downscaling function, and, as you mention it, such option will be set to false when downscaling in time.


user id's are not URL percent encoded

user id's are not URL encoded and results in some 404 errors when usernames contain symbols. Can be solved simply by using urllib.parser.quote but it seems this repository is no longer updated so I am posting this more as a public service announcement than anything else.

To review https://github.com/plamere/spotipy/pull/350/

Taking a look at the Spotify API itself, it states that the parameter to the users endpoint must be an userid, so I guess that the email-address will not work even with the encoding.
For the usernames I will try to make an pull request to fix this as soon as I found an userid that contains special characters to test with.

@Bressan09 thanks! Also note that there is https://github.com/plamere/spotipy/pull/350, what do you think of it? I am not sure myself in what scenario this is needed

Closing this as I don't think Spotify allows emails as possible usernames anymore


[Tree widget] More than one header button results in first button not showing up in dropdown

When using a ModelsTreeComponent or CategoriesTreeComponent that has more than one headerButton in the props (such as the default when the prop is not used), and then clicking the search bar for the component, the first button will not appear in the dropdown. If there is only one button, the button stays the same during the search bar being active. The issue appears to come from the HeaderButtons method/element in TreeHeader.tsx.

Not entirely sure what is being said here. Please attach screenshots and more details of whatever issue you are facing

Also, please specify the version of @itwin/tree-widget-react & @itwin/itwinui-react that you're using.

We are currently on 3.0.1 on @itwin/tree-widget-react and 3.15.4 on @itwin/itwinui-react. I've included some photos of the issue. When the search bar is not active, all buttons show up fine. However, when the search bar is activated, the dropdown menu only shows 5 of the 6 buttons.

I can't reproduce this in our test viewer and our e2e tests suggest it's working fine: https://github.com/iTwin/viewer-components-react/blob/master/packages/itwin/tree-widget/src/e2e-tests/ModelsTree.test.ts-snapshots/Models-tree-Density-default-header-buttons-overflow-1-chromium-linux.png.
Could you please try to set up a sandbox or something that we could use to reproduce it?

Sorry for the late response; I will need to look into getting a separate sandbox set up. I am not sure what the browser the issue was reported in was but I can replicate it in Edge

I'm getting 0 there. The @itwin/itwinui-react package received an overhaul for overflow handling in version 3.16.0 - can you see if it helps?

That seemed to do the trick! Thank you!

@aruniverse @grigasp this issue turns out to be unrelated to<EMAIL_ADDRESS>in fact, the issue was that our<EMAIL_ADDRESS>was using<EMAIL_ADDRESS>instead of 3.x
this happens because as we still have both itwinui-react v2 and v3 in our app and we set v3 as an alias dependency, when a transitive dep like tree-widget-react asks for a peer on itwinui-react it gets paired to the v2 version
this has been an ongoing issue since we know that itwinui-react should be a peer since it contains the ThemeProvider singleton but have no way to seemingly make it match with the v3 version (and if we flip the alias to v2 the inverse happens).
We do have a workaround though, which is to add some logic in the pnpmfile.cjs to remove the peer on itwinui-react and instead set it as a dependency for a set of packages we know we want to do this for. Doing this ensures the right version is used and we  also have a pnpm override to ensure only 1 version of itwinui-react v3 exists to prevent setting it as a dep instead of a peer from leading to multiple versions that would break the ThemeProvider use. T
This workaround simply wasn't yet matching to tree-widget-react because in v2 it sets itwinui-react as a dep instead of a peer and we didn't consider this when we first added v3. After updating the workaround to properly mend tree-widget-react this issue is no longer reproducible
btw the plan is (and we are nearly in a position to do this) to get all our deps to use v3 only then remove the alias make our app just install itwinui-react v3 and remove this pnpmfile.cjs workaround, but until then its worked well enough so long as we keep up with it
Sorry for not discovering this sooner and wasting your time!


GEOPY-1324: Improve import function

GEOPY-1324 - Improve import function

Looks like you already have a logger in the driver, so you probably want to instantiate in the init of the package and use logging.getLogger to re-use it. https://docs.python.org/3/library/logging.html#logging.getLogger