Indenting a block does not use tab size setting

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
My tab settings are to use 2 (the default) spaces:
"*":
  editor:
    atomicSoftTabs: false
    tabType: "soft"

When I select a block of text and press tab, it is indented by 4 spaces instead of 2. When I press shift+tab, it is unindented by 4 spaces instead of 2.
When no text is selected and I press tab (on an even character index), 2 spaces are inserted as expected.
Steps to Reproduce

Select some text
Press tab
Press shift+tab

Expected behavior:
Text should be indented/unindented by 2 spaces based on the setting.
Actual behavior:
Text is indented.unindented by 4 spaces, ignoring the setting.
Reproduces how often:
Always
Versions
Atom    : 1.57.0
Electron: 9.4.4
Chrome  : 83.0.4103.122
Node    : 12.14.1
Additional Information
None

I also noticed that the vertical indentation guidelines are only shown every 4 characters, rather than 2 as expected.

So... it turns out that there are all kinds of "packages" for different languages that may provide different defaults for tab size. If you set the tab size to the default value (2 at this time), Atom uses the default for the language.
This is counter intuitive: defaults should never override user-provided values.

If I do not set the tab size, I expect Atom to use the default value, even if this differs for various languages (Y THO?).
If I set the tab size, this should override any default. Even if I set it to the global default, it should override any language specific default.

I have updated the title to reflect the true nature of this issue.

Check it out on the atom/core modules


undefined reference to __rust_maybe_catch_panic

high there, I tried to upgrade from v1.1.0 to v.1.1.1  and get the following error
cp ./rust-sgx-sdk/compiler-rt/libcompiler-rt-patch.a ./lib
./lib/libenclave.a(substratee_worker_enclave-65e84a936368c742.substratee_worker_enclave.1358z1ty-cgu.14.rcgu.o): In function `sgx_tunittest::rsgx_unit_test':
substratee_worker_enclave.1358z1ty-cgu.14:(.text._ZN13sgx_tunittest14rsgx_unit_test17h7ba717c3629629daE+0x4b): undefined reference to `__rust_maybe_catch_panic'
collect2: error: ld returned 1 exit status

I assume this might be a rust version thing. With which rust version is v.1.1.1 confirmed to work?

sorry did not read the readme carefull enough. works with nightly-2020-03-12


Ordered and unordered list combined into a single list in the preview

Placing an unordered list right after an ordered list (or vice versa) leads to unexpected results:

I managed to fix it by upgrading marked (0.4.0 -> 0.6.1), which SimpleMDE relies on to render markdown previews.


Don't error when no changes are present

The default is not working properly https://github.com/getsentry/snuba/actions/runs/4982169145/jobs/8917438053?pr=4174. Let's debug w/ tests.

I believe this demonstrates the misbehavior we saw in snuba.

I don't think fail-on-empty should be an option

Done in 9805ada4e9c70789c06993c6e90472c7305174fd.

Is there an example of where it's used? Looks like snuba is using the default value for fail-on-empty and a quick search comes up with nothing
https://github.com/search?q=org%3Agetsentry+fail-on-empty&type=code

Done in 9805ada4e9c70789c06993c6e90472c7305174fd.

... and 24fb9f264aad66e860b38744ca0dd152e9f3f5c6. 😬

Is there an example of where it's used?

No, I only added it in #17 a few days ago, nobody is using it. As @asottile-sentry points out the important thing is preserving the existing default behavior (no-op when there are no changes). The option is YAGNI.


update documentation per Evo Pro

Fixes #633
This PR is ready for review.
Risk
This PR makes no API changes.
Testing Plan
n/a
Summary
Fix a documentation comment based on SDL Evolution #285.
Changelog
Bug Fixes

Fix some documentation

CLA

[x] I have signed the CLA

Codecov Report

Merging #645 into release/4.4.0-rc1 will not change coverage.
The diff coverage is n/a.

@@                 Coverage Diff                  @@
##             release/4.4.0-rc1     #645   +/-   ##
====================================================
  Coverage                41.27%   41.27%
  Complexity                2641     2641
====================================================
  Files                      360      360
  Lines                    15713    15713
  Branches                  1655     1655
====================================================
  Hits                      6486     6486
  Misses                    8940     8940
  Partials                   287      287

Impacted Files
Coverage Δ
Complexity Δ

...om/smartdevicelink/proxy/rpc/RadioControlData.java
100% <ø> (ø)
22 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Fix top padding for admins

broke it earlier today

Coverage remained the same at 39.445% when pulling 8bcfe7a1e1fda02fb00d3a79d8e045008ee0c4a0 on fix-admin-content-margin into 4f8eabb5fbccb87c8ee9272b98e9369b5f9077a3 on dev.


Container created from NAV2016 image via BCContainerHelper will not recognize DotNet 3.5 library

PLEASE DO NOT INCLUDE ANY PASSWORDS OR TOKENS IN YOUR ISSUE!!!
Describe the issue
NAV enviornment is NAV2016.  Trying to compile objects provided by ISV.  ISV includes a Add-in DLL that uses DotNet3.5.  Even after enabling DotNet3.5, the objects will not compile.  The "Types" listing in the Development Environment don't display the Types and a compile error "Could not load type 'EasyPost.Address.'EasyPost.Net35, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null''." is displayed.
We use Advania-Git scripts to create our environments
After creating the container with the script below, I attempted to enable DotNet3.5 support via the following commands:
##Enable Windows Update Service
Set-Service -Name wuauserv -StartupType "Manual"
##EnableDotNet
DISM.EXE /Online /Add-Capability /CapabilityName:NetFx3~~~~
See screenshots below
Scripts used to create container and cause the issue
if (![String]::IsNullOrEmpty($SetupParameters.dockerImage) -or ![String]::IsNullOrEmpty($SetupParameters.artifact)) {
    if ($BranchSettings.instanceName -eq "") {
        Write-Host "Starting Docker Image Development Container for Branch ..."
        $params = @{
            SetupParameters = $SetupParameters
            BranchSettings = $BranchSettings
        }
        if ($SetupParameters.dockerAdminPasswordId) {
            Import-Module RemoteManagement -DisableNameChecking | Out-Null
            $DockerAdmin = Get-NAVPasswordStateUser -PasswordId $SetupParameters.dockerAdminPasswordId
            Write-Host "Using stored authentication for Docker Container..."
            $params += @{
                AdminUsername = $DockerAdmin.UserName
                AdminPassword = $DockerAdmin.Password
            }
        }
        if ($SetupParameters.dockerMemoryLimit) {
            Write-Host "Using configured memory limit for Docker Container..."
            $params += @{
                MemoryLimit = $SetupParameters.DockerMemoryLimit
            }
        }
        if ($SetupParameters.dockerRestoreBackup) {
            $BackupFilePath = Get-NAVBackupFilePath -SetupParameters $SetupParameters
            if (Test-Path -Path $BackupFilePath) {
                Write-Host "Using backup file for Docker Container..."
                $params += @{
                    BackupFilePath = $BackupFilePath
                }
            }
        }
        if (![string]::IsNullOrEmpty($SetupParameters.LicenseFilePath)) {
            if (Test-Path $SetupParameters.LicenseFilePath) {
                    $params += @{
                        LicenseFilePath = $SetupParameters.LicenseFilePath
                    }
            }
        }
        Start-DockerContainer @params
        $BranchSettings = Get-BranchSettings -SetupParameters $SetupParameters
        & (Join-path $PSScriptRoot Verify-RepositorySetupInContainer.ps1)
    }
}

if ($BranchSettings.dockerContainerName -gt "") {
    ReStart-DockerContainer -BranchSettings $BranchSettings
} else {
    Load-InstanceAdminTools -SetupParameters $SetupParameters
    if ($BranchSettings.instanceName -eq "") {
        Write-Host "Requesting new NAV Environment for branch" $Setupparameters.projectName
        $BackupFilePath = Get-NAVBackupFilePath -SetupParameters $SetupParameters
        $ServerInstance = "NAV" + $($Setupparameters.navRelease) + "DEV" + (Get-EnvironmentNo)
        $params = @{
            BackupFilePath = $BackupFilePath
            DatabaseServer = $SetupParameters.defaultDatabaseServer
            DatabaseName = $ServerInstance
            DatabasePath = $SetupParameters.DatabasePath }
        if ($SetupParameters.defaultDatabaseInstance -ne "") { $params.DatabaseInstance = $SetupParameters.defaultDatabaseInstance }
        Write-Host "Restoring database..."
        Restore-NAVBackup @params
        Remove-Item -Path $BackupFilePath -Force -ErrorAction SilentlyContinue
        $BranchSettings.databaseInstance = $SetupParameters.defaultDatabaseInstance
        $BranchSettings.databaseServer = $SetupParameters.defaultDatabaseServer
        $BranchSettings.databaseName = $ServerInstance
        $BranchSettings.instanceName = ""
        Write-Host "Upgrading database..."
        Invoke-NAVDatabaseConversion -SetupParameters $SetupParameters -BranchSettings $BranchSettings
        Write-Host "Simplifying Database..."
        Set-NAVDatabaseToSimpleRecovery -DatabaseServer $SetupParameters.defaultDatabaseServer -DatabaseInstance $SetupParameters.defaultDatabaseInstance -DatabaseName $ServerInstance
        Write-Host "Compiling Service Objects..."
        Compile-NAVApplicationGITObject -SetupParameters $SetupParameters -BranchSettings $BranchSettings -Filter "Type=Table;Id=2000000004..2000000999" -SynchronizeSchemaChanges No
        Write-Host "Creating Service..."
        $DefaultInstanceSettings = Get-DefaultInstanceSettings -SetupParameters $SetupParameters -BranchSettings $BranchSettings
        $BranchSettings.clientServicesPort = $DefaultInstanceSettings.DocumentElement.appSettings.SelectSingleNode("add[@key='ClientServicesPort']").Attributes["value"].Value
        $BranchSettings.managementServicesPort = $DefaultInstanceSettings.DocumentElement.appSettings.SelectSingleNode("add[@key='ManagementServicesPort']").Attributes["value"].Value
        $params = @{
          ServerInstance = $ServerInstance
          DatabaseName = $BranchSettings.databaseName
          DatabaseServer = $BranchSettings.databaseServer
          ManagementServicesPort = $DefaultInstanceSettings.DocumentElement.appSettings.SelectSingleNode("add[@key='ManagementServicesPort']").Attributes["value"].Value
          ClientServicesPort = $DefaultInstanceSettings.DocumentElement.appSettings.SelectSingleNode("add[@key='ClientServicesPort']").Attributes["value"].Value
          SOAPServicesPort = $DefaultInstanceSettings.DocumentElement.appSettings.SelectSingleNode("add[@key='SOAPServicesPort']").Attributes["value"].Value
          ODataServicesPort = $DefaultInstanceSettings.DocumentElement.appSettings.SelectSingleNode("add[@key='ODataServicesPort']").Attributes["value"].Value
        }
        if ($BranchSettings.databaseInstance -ne "") { $params.DatabaseInstance = $BranchSettings.databaseInstance }
        New-NAVServerInstance @params -Force -ServiceAccount NetworkService -ErrorAction Stop
        Set-NAVServerConfiguration -ServerInstance $ServerInstance -KeyName ServicesDefaultTimeZone -KeyValue 'UTC'
        Set-NAVServerConfiguration -ServerInstance $ServerInstance -KeyName ServicesLanguage -KeyValue 'en-US'
        Set-NAVServerConfiguration -ServerInstance $ServerInstance -KeyName ODataServicesSSLEnabled -KeyValue $false
        Set-NAVServerConfiguration -ServerInstance $ServerInstance -KeyName ODataServicesEnabled -KeyValue $true
        Set-NAVServerConfiguration -ServerInstance $ServerInstance -KeyName SOAPServicesSSLEnabled -KeyValue $false
        Set-NAVServerConfiguration -ServerInstance $ServerInstance -KeyName SOAPServicesEnabled -KeyValue $true
        Set-NAVServerConfiguration -ServerInstance $ServerInstance -KeyName DataCacheSize -KeyValue 7
        Set-NAVServerConfiguration -ServerInstance $ServerInstance -KeyName CompileBusinessApplicationAtStartup -KeyValue $false
        Set-NAVServerConfiguration -ServerInstance $ServerInstance -KeyName PublicWebBaseUrl -KeyValue "http://$($env:COMPUTERNAME):$(Get-WebClientPort -MainVersion $SetupParameters.mainVersion)/${ServerInstance}/WebClient"
        if ($SetupParameters.developerService -eq $true) {
            Set-NAVServerConfiguration -ServerInstance $ServerInstance -KeyName DeveloperServicesEnabled -KeyValue $true
            Set-NAVServerConfiguration -ServerInstance $ServerInstance -KeyName DefaultClient -KeyValue 'Web'
            Set-NAVServerConfiguration -ServerInstance $ServerInstance -KeyName EnableSymbolLoadingAtServerStartup -KeyValue $true
            $BranchSettings.developerServicesPort = $DefaultInstanceSettings.DocumentElement.appSettings.SelectSingleNode("add[@key='DeveloperServicesPort']").Attributes["value"].Value
        } else {
            $BranchSettings.developerServicesPort = ""
        }
        $BranchSettings.instanceName = $ServerInstance
        Enable-DelayedStartForNAVService -BranchSettings $BranchSettings
        Enable-TcpPortSharingForNAVService -BranchSettings $BranchSettings
        Write-Host "Starting Service..."
        Set-NAVServerInstance -ServerInstance $ServerInstance -Start -Force
        Write-Host "Syncronizing Database..."
        Get-NAVServerInstance -ServerInstance $ServerInstance | Where-Object -Property State -EQ Running | Sync-NAVTenant -Mode ForceSync -Force
        Write-Host "Creating Web Server Instance..."
        New-NAVWebServerInstance -ClientServicesPort $BranchSettings.clientServicesPort -Server $env:COMPUTERNAME -ServerInstance $ServerInstance -WebServerInstance $ServerInstance
        Enable-NAVWebClientDesigner -BranchSettings $BranchSettings
        Enable-NAVWebClientPersonalization -BranchSettings $BranchSettings
        Update-BranchSettings -BranchSettings $BranchSettings
        Write-Host "Environment build completed..."
    } else {
        Write-Host "Environment already created..."
    }

    if (![string]::IsNullOrEmpty($SetupParameters.LicenseFilePath)) {
        if (Test-Path $SetupParameters.LicenseFilePath) {
            Update-NAVLicense -BranchSettings $BranchSettings -LicenseFilePath $SetupParameters.LicenseFilePath
        }
    }
    UnLoad-InstanceAdminTools

}

if (![string]::IsNullOrEmpty($Setupparameters.uidOffset)) {
    Write-Host "Set uidoffset in database $($BranchSettings.databaseName) to $($Setupparameters.uidOffset)"
    $command = 'UPDATE [dbo].[$ndo$dbproperty] SET [uidoffset] = ' + $Setupparameters.uidOffset
    $Result = Get-SQLCommandResult -Server (Get-DatabaseServer -BranchSettings $BranchSettings) -Database $BranchSettings.databaseName -Command $command -Username $SetupParameters.SqlUsername -Password $SetupParameters.SqlPassword
}

**Full output of scripts**

PS C:\AdvaniaGIT\Workspace\NAV2016_CCB> Start-AdvaniaGITAction -Repository "c:\AdvaniaGIT\Workspace\NAV2016_CCB" -ScriptName "Build-NavEnvironmentFromGIT.ps1" -Wait $false
Using License: Cryobank.flf on C:\AdvaniaGIT\License\Cryobank.flf
Starting Docker Image Development Container for Branch ...
Using backup file for Docker Container...
Connecting to repository navdocker.azurecr.io...
Preparing Docker Container for Dynamics NAV...
BcContainerHelper is version 2.0.4
BcContainerHelper is running as administrator
Host is Microsoft Windows 10 Enterprise - 1809
Docker Client Version is 20.10.2
Docker Server Version is 20.10.2
Fetching all docker images
ArtifactUrl and ImageName specified
Image custombuild:onprem-9.0.50357.0-na already exists
Using image custombuild:onprem-9.0.50357.0-na
Disabling Health Check (always report healthy)
Creating Container Cryobank
Version: 9.0.50357.0-na
Style: onprem
Multitenant: No
Platform: 9.0.50357.0
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.1697 (ltsc2019)
Host OS Version: 10.0.17763.1697 (1809)
Using process isolation
Using locale en-US
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Using license file C:\AdvaniaGIT\License\Cryobank.flf
Additional Parameters:
--volume "c:\AdvaniaGIT\Workspace\NAV2016_CCB:C:\GIT"
--volume "C:\AdvaniaGIT:C:\Host"
--env SqlTimeout=1200
--dns <IP_ADDRESS>
--restart always
--volume "c:\AdvaniaGIT\Workspace\NAV2016_CCB\Add-Ins:c:\run\Add-Ins"
--env bakfile="C:\Host\Log\d7555a61-f87d-4eaa-adf4-47ece16b57ce\NAVBackup.bak"
--env customNavSettings=EnableTaskScheduler=False
Files in C:\ProgramData\BcContainerHelper\Extensions\Cryobank\my:

AdditionalOutput.ps1
AdditionalSetup.ps1
license.flf
MainLoop.ps1
SetupVariables.ps1
updatehosts.ps1
Creating container Cryobank from image custombuild:onprem-9.0.50357.0-na
c5bc76cdd429c4bac151faf01de0371a91a7a2c8408bd90a48c396eacea79e3e
Waiting for container Cryobank to be ready
Initializing...
Setting host.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting gateway.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting kubernetes.docker.internal to <IP_ADDRESS> in container hosts file (copy from host hosts file)
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is Cryobank
PublicDnsName is Cryobank
Using Windows Authentication
Starting Local SQL Server
Starting Internet Information Server
Using Database .bak file 'C:\Host\Log\d7555a61-f87d-4eaa-adf4-47ece16b57ce\NAVBackup.bak'
Using database server localhost\SQLEXPRESS
Setting CompatibilityLevel for mydatabase on localhost\SQLEXPRESS
Modifying Service Tier Config File with Instance Specific Settings
Modifying Service Tier Config File with settings from environment variable
Creating EnableTaskScheduler and setting it to False
Starting Service Tier
Creating Web Site
Creating Web Server Instance
Creating ManagementServicesPort and setting it to 7045
Using license file 'c:\run\my\license.flf'
Import License
Creating http download site
Creating Windows user rons
Setting SA Password and enabling SA
Creating SUPER user
Container IP Address: <IP_ADDRESS>
Container Hostname  : Cryobank
Container Dns Name  : Cryobank
Web Client          : http://Cryobank/NAV/WebClient/
Setting Cryobank to <IP_ADDRESS> in host hosts file

Files:
WARNING: You are running a container which is 68 days old.
Microsoft recommends that you always run the latest version of our containers.
Container Total Physical Memory is 15.6Gb
Container Free Physical Memory is 5.6Gb
Initialization took 56 seconds
Ready for connections!
Reading CustomSettings.config from Cryobank
Container Cryobank successfully created
Use:
Get-BcContainerEventLog -containerName Cryobank to retrieve a snapshot of the event log from the container
Get-BcContainerDebugInfo -containerName Cryobank to get debug information about the container
Enter-BcContainer -containerName Cryobank to open a PowerShell prompt inside the container
Remove-BcContainer -containerName Cryobank to remove the container again
docker logs Cryobank to retrieve information about URL's again
Extending PSModulePath with C:\AdvaniaGIT\Scripts
Extending Computer wide PSModulePath with C:\AdvaniaGIT\Scripts
Importing the modules
Extending Path with C:\AdvaniaGIT\Tools
Extending Computer wide Path with C:\AdvaniaGIT\Tools
Updating BranchSettings.json...
Updating GITSettings.json...
Set uidoffset in database mydatabase to 50200
Docker Image on Cryobank is responding correctly...
...

**Screenshots**
If applicable, add screenshots to help explain your problem.
In Development Environment, codeunit objects which have the addon specified will not compile with the error message specified above.  The addon's Types are not displayed when "looked up" inside the variable definition
![image](https://user-images.githubusercontent.com/50466531/112026912-eb0b3b80-8af3-11eb-959b-8da0465dc8c7.png)

We also have a local NAV2016 environment.  When the same objects are inspected, the Addon's Types are properly displayed and the object compiles correctly:
![image](https://user-images.githubusercontent.com/50466531/112027176-2c035000-8af4-11eb-8095-457455e543bb.png)

**Additional context**
- does it happen all the time? Yes - any container built will display this error (tried on multiple machines)

- did it use to work? No - We are trying to install this ISV's addon for the first time.  ISV does not use Docker for NAV2016 development.

Not sure if the screenshots appeared in the submission above, so I am resubmitting here.
In NAV2016 Development Environment inside container, codeunit objects which have the addon specified will not compile with the error message specified above.  The addon's Types are not displayed when "looked up" inside the variable definition

We also have a local NAV2016 environment.  When the same objects are inspected, the Addon's Types are properly displayed and the object compiles correctly:

If you clone the nav-docker repository and open this file in PowerShell ISE:
https://github.com/microsoft/nav-docker/blob/master/generic/build.ps1
Then you can build your own generic image.
In the DOCKERFILE, you can add the dotnet35 binaries and run the installer (like SQLEXPRESS and the other things are installed)
Then you can use this generic image when you start your container.

Did you manage to make this work?

Hi Freddy,
Yes, we were able to make this work – I think that we were not adding the Dotnet Library to the server image within the container.  This script seems to have done the trick – our containername is “Cryobank”:
##Start Powershell within Docker
C:\WINDOWS\system32\CMD.EXE /C docker.exe exec -it Cryobank powershell -noexit c:\run\prompt.ps1
##Enable Windows Update Service
Set-Service -Name wuauserv -StartupType "Manual"
##EnableDotNet
DISM.EXE /Online /Add-Capability /CapabilityName:NetFx3~~~~
##Display enabled features
DISM.EXE /Online /Get-Capabilities
Thank you for the followup.
Ron
From: Freddy Kristiansen @.>
Sent: Sunday, June 6, 2021 11:33 PM
To: microsoft/navcontainerhelper @.>
Cc: Ron Saritzky @.>; Author @.>
Subject: Re: [microsoft/navcontainerhelper] Container created from NAV2016 image via BCContainerHelper will not recognize DotNet 3.5 library (#1824)
[EXTERNAL EMAIL]
Did you manage to make this work?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/navcontainerhelper/issues/1824#issuecomment-855628617, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AMBA5Y7YXKRU4UX5K356NMLTRRRYRANCNFSM4ZTPZEOA.
The information transmitted in this electronic communication is intended only for the person or entity to whom it is addressed and may contain confidential and/or privileged material. Any review, retransmission, dissemination or other use of or taking of any action in reliance upon this information by persons or entities, other than the intended recipient, is prohibited. If you received this information in error, please contact the sender and the privacy officer, and properly dispose of this information.

Thanks


Add Polish Translation

Added Polish (pl-PL) translation as part of #37

@piotrulos Thank you for your pull request, make sure to keep an eye on #37 to know when there are new strings to translate.


POST ob/orderfulfillment crypto listings support

Completes #876 in part...
POST ob/orderfulfillment

[x] When the order being fulfilled is for a crypto currency listing, the API should require a transactionId value as a string, which represents the ID of the transaction where the seller transferred the crypto currency to the buyer.
[x] When the order being fulfilled is for a crypto currency listing, if a trandactionId is not supplied, the server should return an error.
[x] Check to ensure that the wrong fulfillment type is not provided for fulfillment.

The transactionId could be represented as an object in the protobuff:
message CryptoCurrencyDelivery: {
  string transactionId
}

merged into cryptolistings branch.


Fix Daemon network issue.

Fix Daemon network issue.
The incorrect node name get used to verify cesSharedRoot is available.
Signed-off-by: Christoph Keil<EMAIL_ADDRESS>

Uses admin node name instead of daemon node name... looks good to me!
Is this only relevant for NFS? What about SMB / HDFS / Object?

Achim, the change is at the common section that also get used from other CES component as HDFS, OBJ, and SMB

Hi Christoph, I've since made some changes (https://github.com/IBM/ibm-spectrum-scale-install-infra/commit/0c67bd7ad170c1aff813dae3d9973255d0a93aca) to how this is supposed to work. scale_protocol_node_list now contains the inventory_hostname's of CES nodes. This can be translated into daemon nodename or admin nodename - e.g. in loops like so:
hostvars[item]['scale_admin_nodename']

See https://github.com/IBM/ibm-spectrum-scale-install-infra/pull/578 for details.

Do you think this PR is still relevant?
If so, can you please adjust for the new logic outlined above and re-submit against the nextgen branch?
Thanks much!


Update to README and other small changes

Changes
README.md
Big update to README.md, each of the options in checksit check have initial description with example of how to use, as in Issue #14.
The other three options for checksit have a basic description that can be flushed out as that functionality is increased.
requirements.txt
Addition of pandas requirement, as it is used in checksit summary, as mentioned in Issue #13.
checksit/generic.py
Within check_global_attrs, actually looks at global attributes within dct rather than just the top level, which was global attributes, variables and dimensions (I think). This meant that when checking a global attribute exists or its vocabs, the attribute was not being found due to the code looking in the wrong place.

Great stuff @joshua-hampton


show query duration for ExcecuteDax & ExcecuteCommand functions after the execution completes

Is your feature request related to a problem? Please describe.
No.
Describe the solution you'd like
Would be curious to have the ExecuteDax & ExecuteCommand functions return the query duration time somewhere on the screen (possibly the output window). Not sure how easy it is but just an idea :)

How about this:
var sw = new System.Diagnostics.Stopwatch();
sw.Start();
var result = EvaluateDax("1 + 1");
sw.Stop();
Info("DAX execution finished in: " + sw.ElapsedMilliseconds + " ms");
result.Output();

That works, thanks!


[Helm] Wrong version form chart

There is a wrong version (0.18.0) in the chart.yml:
https://github.com/kubernetes-sigs/descheduler/blob/d0fbebb77c79831137b1a14b5ceadb489c678859/charts/descheduler/Chart.yaml#L3
When we add the repo through helm, it shows only version 0.18.1:
$ helm repo add descheduler https://kubernetes-sigs.github.io/descheduler/
$ helm template descheduler/descheduler-helm-chart --version 0.18.0
Error: failed to download "descheduler/descheduler-helm-chart" (hint: running `helm repo update` may help)
$ helm search repo descheduler
NAME                              	CHART VERSION	APP VERSION	DESCRIPTION
descheduler/descheduler-helm-chart	0.18.1       	0.18.0     	Descheduler for Kubernetes is used to rebalance...

/kind bug

@vmdude thanks for opening this issue.
We attempted to release descheduler helm chart version 0.18.0, but our release automation failed. We fixed the glitch and were able to to release descheduler helm chart version 0.18.1 which was the first helm chart release. Version 0.18.1 can be found on the release-1.18 branch:
https://github.com/kubernetes-sigs/descheduler/blob/76ab9e765a7db36011b6be4e4e1c93018c0fb65e/charts/descheduler/Chart.yaml#L3
The master branch will be updated roughly sometime in August for the 0.19.0 release.
Based on this explanation are you good with closing this issue? Do you think there is some end user documentation that could be updated to make this more clear?
CC: @stevehipwell

I'm struggling to see where there is a bug? If you add the repo to helm, update it and run helm search repo descheduler you get the following result.
NAME                                    CHART VERSION   APP VERSION     DESCRIPTION
descheduler/descheduler-helm-chart      0.18.1          0.18.0          Descheduler for Kubernetes is used to rebalance...

The chart and app version's don't match but that's documented as expected behaviour, not having a 0.18.0 chart isn't ideal but it isn't an actual bug (and could happen again).

It's not a bug per-se, but your chart.yml file refered to a version that didn't exist in the helm repo (cause explained by @seanmalloy )
It's OK to close this issue.


Using low_cpu_mem_usage=True or a device_map requires Accelerate: pip install accelerate

hi, do you know why i see this:
Using low_cpu_mem_usage=True or a device_map requires Accelerate: pip install accelerate
workflow:
1.json

Same, using comfyui portable and installed it via manager.

Same, using comfyui portable and installed it via manager.

thanks. Which one should i donwload?

Same, using comfyui portable and installed it via manager.

thanks. Which one should i donwload? where should i go on manage? i have it also. should i download from github also?

i don't know dude, i'm saying i have the same problem as you. mine gives the same error

Added accelerate to the requirements, try re-installing now.

Added accelerate to the requirements, try re-installing now.

Thank you. it is solved very good.
i have one another question please.

i want to install this model.
i think this model is here:
https://huggingface.co/gokaygokay/Florence-2-Flux-Large/tree/main
i don't know how and where should i install it?
i thought if i click "Queue Prompt" cmd automatically download it but it didn't. i don't know how to add this model manually. thanks.

Added accelerate to the requirements, try re-installing now.

Thank you. it is solved very good.
i have one another question please.

i want to install this model.
i think this model is here:
https://huggingface.co/gokaygokay/Florence-2-Flux-Large/tree/main
i don't know how and where should i install it?
i thought if i click "Queue Prompt" cmd automatically download it but it didn't. i don't know how to add this model manually. thanks.

The auto download node is a different one, that one in your screenshot looks for models in the ComfyUI/models/LLM -folder.

Still getting the error

I am still getting the error

Have you solved the problem? I had the same problem

Have you solved the problem? I had the same problem

I changed accelerate to accelerate>=0.26.0 in requirements.txt and it installed it currectly.
it looks like this:
transformers>=4.39.0
matplotlib
timm
pillow >= 10.2.0
peft
accelerate>=0.26.0

Added accelerate to the requirements, try re-installing now.

Can you further explain which file should be modified


cmd/cgo/internal/testsanitizers,x/build: upstream clang does not build against zlib, so checking for ASAN symbolization is disabled during the ASAN tests

See https://github.com/golang/go/issues/65469#issuecomment-1930624462 for complete context.
For our LUCI infrastructure, we store an official clang toolchain release (https://releases.llvm.org/) in CIPD, and download it into our build environment on-demand. Previously, we just baked the toolchain into the image using an officially-maintained package for Debian (https://apt.llvm.org), but this made upgrading onerous enough that we were stayed on clang 7 (2018) for a long time.
Unfortunately, the official clang toolchain releases aren't built against zlib, and the Go toolchain compresses DWARF sections. When the ASAN runtime tries to symbolize the stack trace it takes, we get failures like:
        error: failed to decompress '.debug_aranges', zlib is not available
        error: failed to decompress '.debug_info', zlib is not available
        error: failed to decompress '.debug_abbrev', zlib is not available
        error: failed to decompress '.debug_line', zlib is not available
        error: failed to decompress '.debug_frame', zlib is not available
        error: failed to decompress '.debug_str', zlib is not available
        error: failed to decompress '.debug_loc', zlib is not available
        error: failed to decompress '.debug_ranges', zlib is not available
        error: failed to decompress '.debug_addr', zlib is not available
        error: failed to decompress '.debug_rnglists', zlib is not available
        error: failed to decompress '.debug_str_offsets', zlib is not available
        error: failed to decompress '.debug_line_str', zlib is not available

The alternatives here are not ideal. They require either maintaining more images or complicating our clang update process significantly. For now, we've decided to just not check for symbolized output for our ASAN tests when building with clang. We bake GCC into our images, and GCC can symbolize the output, so we still have coverage on our debug information.
This issue tracks either:

Waiting for clang to build against zlib in one of their releases, or
Deciding to bake clang into the image again.

Another benefit of keeping clang in CIPD is that it's easier to test multiple versions of clang, if we ever need to.


Lift MessagingService.minimum_version to 40 (CASSANDRA-18314)

all comments addressed.
rebased off trunk.
CI runs: https://app.circleci.com/pipelines/github/michaelsembwever/cassandra/201

I think another thing that can be slightly simplified now that we don't expect 3.0 nodes is QueryProcessor#useNewPreparedStatementBehaviour(), where we don't need QueryProcessor.NEW_PREPARED_STATEMENT_BEHAVIOUR_SINCE_30.

I think another thing that can be slightly simplified now that we don't expect 3.0 nodes is QueryProcessor

can we do that in a separate ticket, not related to MessagingService (see parent ticket)

I think another thing that can be slightly simplified now that we don't expect 3.0 nodes is QueryProcessor

can we do that in a separate ticket, not related to MessagingService (see parent ticket)

Yeah, it can be a separate ticket since the min cluster version doesn't come from MessagingService.

These methods are not used and I think can be removed:

FwdFrmSerializer#pre40DeserializeWithLength
Verb#toPre40Verb

Do we need to generate benchmarks for a 'pre40'?
MessageOutBench#serializePre40

            // if the other node is pre-4.0, it will respond only with its maxMessagingVersion
            if (maxMessagingVersion < VERSION_40 || handshakeMessagingVersion < VERSION_40)
                return new Accept(useMessagingVersion, maxMessagingVersion);

Should we return null for this case? This is the HandshakeProtocol class.

This is the HandshakeProtocol class.

changing that breaks HandshakeTest 🤷

 // if the other node is pre-4.0, it will respond only with its maxMessagingVersion
if (maxMessagingVersion < VERSION_40 || handshakeMessagingVersion < VERSION_40)
   return new Accept(useMessagingVersion, maxMessagingVersion);

Should we return null for this case? This is the HandshakeProtocol class.

changing that breaks HandshakeTest 🤷

Returning null there doesn't break HandshakeTest for me, at least locally. What failure are you finding?

Returning null there doesn't break HandshakeTest for me, at least locally. What failure are you finding?

On the outbound (encode) side, you get a npe.
Patch to deal with this (but still catch old versions so we can test ClosedChannelException is still correctly thrown) en route…

Patch to deal with this (but still catch old versions so we can test ClosedChannelException is still correctly thrown) en route…

Done.  See https://github.com/apache/cassandra/pull/2506/commits/168e986e1b6d4a7cef1f8ec582b223ead3f40a05
We don't test listening (outbound) on unsupported versions anymore, but preconditions are in place to prevent it from happening.

We don't test listening (outbound) on unsupported versions anymore, but preconditions are in place to prevent it from happening.

Thanks, the idea was exactly as you mentioned in the comment: close connections on a handshake if the node version is obsolete! :-)

Do we have a CI run for the last changes?

CI: https://app.circleci.com/pipelines/github/michaelsembwever/cassandra/207
failures known in trunk (and one timeout)

Only the following constant is used in this class LegacyLZ4Constants#HEADER_LENGTH, should we move it to the OutboundConnections? It's worth noting that since we've removed the FrameEncoderLegacyLZ4 and FrameDecoderLegacyLZ4 classes, it seems we can inline this constant as it is no longer required.

The FrameDecoder class the following links in java docs are broken:
 * 4. {@link FrameDecoderLegacy}:
          no compression; no integrity protection; turns unframed streams of legacy messages (< 4.0) into frames
 * 5. {@link FrameDecoderLegacyLZ4}
 *        LZ4 compression using standard LZ4 frame format; groups legacy messages (< 4.0) into frames

Should we remove the following configuration value as well? (maybe it is better to fix it in another issue).
Configuration: EncryptionOptions.ServerEncryptionOptions#legacy_ssl_storage_port_enabled
I know it is not marked as @Deprecated, but it seems the value released within the original commit here https://github.com/apache/cassandra/commit/310a48efcfdc0fb77fbd5fd6e734bf17667f617d and the ticket CASSANDRA-15066. The configuration value was previously called: enable_legacy_ssl_storage_port (here is the link to it https://github.com/apache/cassandra/commit/310a48efcfdc0fb77fbd5fd6e734bf17667f617d#diff-8c05296e94c174af68ee14e9c4a84982338aabf6d926a78608adf0f81775418bR240).
According to the following error message in InboundConnectionSettings class the SSL port was used in 3.0, right?
https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/net/InboundConnectionSettings.java#L161

Configuration: EncryptionOptions.ServerEncryptionOptions#legacy_ssl_storage_port_enabled

this is related to something else than the messaging service. a separate ticket under the epic CASSANDRA-18306 is warranted.

I have found a few if-else statements if (version >= MessagingService.VERSION_40) where the else part can be safely removed as this condition is always true from now on, right?

https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/db/ReadResponse.java#L343
https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/locator/InetAddressAndPort.java#L356
https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/locator/InetAddressAndPort.java#L431
https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/locator/InetAddressAndPort.java#L459
https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/locator/InetAddressAndPort.java#L475
https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/locator/InetAddressAndPort.java#L506
https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/schema/CompressionParams.java#L618

The ForwardingInfo class:

all of the if-else statements version >= VERSION_40 can be simplified, right?
according to the TODO statement 'in the next protocol version only serialize peers, message id will become redundant once 3.0 is out of the picture' we can get rid of messageIds field or am I missing something?

all comments addressed.

Sorry, the last one :-)
The RequestFailureReason class - the condition version < VERSION_40 can be simplified as well.

DropCompactStorage#validateCanDropCompactStorage
I think this can be removed as it is always false.
                if (MessagingService.instance().versions.knows(node) &&
                    MessagingService.instance().versions.getRaw(node) < MessagingService.VERSION_40)
                {
                    before4.add(node);
                    continue;
                }

ReadResponseTest#roundTripSerialization
Can remove the following:
            if (version < MessagingService.VERSION_40)
            {
                assertFalse(deser.mayIncludeRepairedDigest());
                // even though that means they should never be used, verify that the default values are present
                assertEquals(ByteBufferUtil.EMPTY_BYTE_BUFFER, deser.repairedDataDigest());
                assertTrue(deser.isRepairedDigestConclusive());
            }

For the Verifier class we can simplify the methods:

willProcessOnEventLoop
expiresAtNanos

The MessageGenerator class the methods can be simplified as well:

writeLength
readHeader

DropCompactStorage#validateCanDropCompactStorage

this is its own kettle of fish, separate ticket please.

comments addressed.

@michaelsembwever Thank you, re-checked the comments and sources - OK
The build with all checks passes locally.

CI

https://app.circleci.com/pipelines/github/michaelsembwever/cassandra/214

latest CI

https://app.circleci.com/pipelines/github/michaelsembwever/cassandra/215

all failures are not related.

CI results look good to me. There are failures for SSTablesSystemViewTest#testVirtualTableThroughIndexLifeCycle with tries that aren't reported, but they can be reproduced on trunk:

https://app.circleci.com/pipelines/github/adelapena/cassandra/3068/workflows/17760f03-9807-41cc-9cb3-a1fdf3e60f77/jobs/64069/tests

I have created CASSANDRA-18703 for that test failure.
As for the j17 workflow, I've started a few tests here: https://app.circleci.com/pipelines/github/adelapena/cassandra/3069/workflows/613644ed-b9d6-4d81-9ef3-ee9d13625fd5

Committed https://github.com/apache/cassandra/commit/4bbfd64fcd7ff208c8cc3833ffff00aea015d2a8


Implement responsive behavior

Once the feature described in #82 is implemented, the component should have a responsive behavior, by switching between horizontal and vertical layout, as needed.

It would be great that this feature can be enabled/disabled. Regarding how to to choose the orientation, maybe it should be based on the component's actual aspect ratio. If the component is wider than taller then the horizontal orientation should be automatically selected, and if it's taller than wider then the other way around.
Another enhancement would be to supply a callback so the user can decide when to select each orientation based on the size.


Add teams feature to the community theme

You need to update the core first before you update or install this package.

Added new login + register pages to this PR

Added dark mode to this PR.
You need to update the Core first (see PR https://github.com/concrete5/concrete5/pull/9373)

I believe we're still waiting on #9373 from the concrete5 project to be updated before I can merge this in.

Added showcase feature

Moved showcase code to https://github.com/concrete5/community.concretecms.com


Error

Build Ok for iOS 14.5
But cannot add new token to wallet
NSLocalizedDescription=The request timed out., NSErrorFailingURLStringKey=https://mainnet.infura.io/v3/da3717f25f824cc1baa32d812386d93f, NSErrorFailingURLKey=https://mainnet.infura.io/v3/da3717f25f824cc1baa32d812386d93f, _kCFStreamErrorDomainKey=4
On iOs 15. beta install live version download from Apple Store same error when try add token
after that app crash

On iOs 15. beta install live version download from Apple Store same error when try add token
after that app crash

@trustwalletvn Do you mean from AlphaWallet or your fork?
If you have the crash log, can you paste it here?

I'm used real phone with iOS 15 beta 5 and install live version
https://apps.apple.com/us/app/alphawallet/id1358230430#?platform=iphone
The app crash and cannot add new token
By the way please add change language setting in advanced UI
i'll translate for Vietnamese language

On iOs 15. beta install live version download from Apple Store same error when try add token
after that app crash

@trustwalletvn Do you mean from AlphaWallet or your fork?
If you have the crash log, can you paste it here?

Good morning. I volunteer to help with translation to Brazilian Portuguese
and Spanish.
On Wed, 25 Aug 2021 at 02:55 Trust Wallet Viet Nam @.***>
wrote:

I'm used real phone with iOS 15 beta 5 and install live version
https://apps.apple.com/us/app/alphawallet/id1358230430#?platform=iphone
The app crash and cannot add new token
By the way please add change language setting in advanced UI
i'll translate for Vietnamese language
https://camo.githubusercontent.com/aba4b9e07c52f37d84c910f2fdaff8f97c8157350d1449bcd6e79a054609e4fa/68747470733a2f2f7061737465626f6172642e636f2f4b687448435a572e706e67
On iOs 15. beta install live version download from Apple Store same error
when try add token
after that app crash
@trustwalletvn https://github.com/trustwalletvn Do you mean from
AlphaWallet or your fork?
If you have the crash log, can you paste it here?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/AlphaWallet/alpha-wallet-ios/issues/3094#issuecomment-905205080,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AD6BDAXZRT5QYSAJCBJ7GJDT6SAVPANCNFSM5CXKLUBA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

--
[image: photo]
Alexandre Carneiro
Gestão de TI e Homecare at  Multiclinic
A  Rua Pedro Firmeza 712, Cidade dos Funcionários
P  (85)3279-5059  <(85)3279-5059>
M  (85)98678-1071  <(85)98678-1071>
E  @.***  @.***>
W  https://homecare.multiclinic.com.br
https://homecare.multiclinic.com.br
http://facebook.com/multiclinic http://instagram.com/multiclinicprl
http://linkedin.com/in/multiclinic http://twitter.com/multiclinic
Do you really need to print this email?
Create your own email signature
https://www.wisestamp.com/signature-in-email?utm_source=promotion&utm_medium=signature&utm_campaign=create_your_own

Good morning. I volunteer to help with translation to Brazilian Portuguese and Spanish.
On Wed, 25 Aug 2021 at 02:55 Trust Wallet Viet Nam @.> wrote: I'm used real phone with iOS 15 beta 5 and install live version https://apps.apple.com/us/app/alphawallet/id1358230430#?platform=iphone The app crash and cannot add new token By the way please add change language setting in advanced UI i'll translate for Vietnamese language https://camo.githubusercontent.com/aba4b9e07c52f37d84c910f2fdaff8f97c8157350d1449bcd6e79a054609e4fa/68747470733a2f2f7061737465626f6172642e636f2f4b687448435a572e706e67 On iOs 15. beta install live version download from Apple Store same error when try add token after that app crash @trustwalletvn https://github.com/trustwalletvn Do you mean from AlphaWallet or your fork? If you have the crash log, can you paste it here? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#3094 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AD6BDAXZRT5QYSAJCBJ7GJDT6SAVPANCNFSM5CXKLUBA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .
-- [image: photo] Alexandre Carneiro Gestão de TI e Homecare at Multiclinic A Rua Pedro Firmeza 712, Cidade dos Funcionários P (85)3279-5059 <(85)3279-5059> M (85)98678-1071 <(85)98678-1071> E @. @.***> W https://homecare.multiclinic.com.br https://homecare.multiclinic.com.br http://facebook.com/multiclinic http://instagram.com/multiclinicprl http://linkedin.com/in/multiclinic http://twitter.com/multiclinic Do you really need to print this email? Create your own email signature https://www.wisestamp.com/signature-in-email?utm_source=promotion&utm_medium=signature&utm_campaign=create_your_own

If you like, you could do a Portuguese localisation for our Android app too :) You can take the 'strings.xml' file (we already have a values-es Spanish localisation, although it is incomplete) and create a values-pt directory with the localised strings.xml. Many of the strings will be identical to the iOS build.


Move maskprep_flat to plugins and rename

I find that the gdsfactory.geometry.maskprep_flat module is useful, but

it is not well distinguished from the maskprep module, making it confusing which to use
it has a bad name... beyond being ugly, it could be made untrue with certain enhancements to enable better hierarchical processing
it better fits the description of a "plugin" since it is almost 100% a wrapper around klayout functionality and fairly tangential to the gdsfactory core

I propose we move it to gplugins. I'm open to name suggestions.

moved it here
https://github.com/gdsfactory/gplugins/issues/32


exporting animation breaks views

I create my animation as usual but when I start the export the animated frames just disappear...
They are still there in sketch's layer list but not graphically here, and they don't appear in the animation either.
Sketch version : 41.2

Sounds that it could be the problem with groups like in this issue https://github.com/Creatide/AnimateMate/issues/8
This happens when you use groups. To avoid that you need to use "bounds" layer that is bigger than actual animated item. I mention that in this video if I remember right :) https://www.youtube.com/watch?v=bDm_c4wPBkc


does fonttools provide a "makeNameRecord" helper method?

# Maybe fonttools should provide us a helper method like this one...
# TODO: verify if fonttools doesn't really provide that and then
#       possibily contribute something equivalent to this upstream.
def makeNameRecord(text, nameID, platformID, platEncID, langID):
    """ Helper function to create a new NameRecord entry """
    name = NameRecord()
    name.nameID, name.platformID, name.platEncID, name.langID = (
        nameID, platformID, platEncID, langID)
    name.string = text.encode(name.getEncoding())
    return name

(issue #924)

Doesn’t setName() do this?
https://github.com/behdad/fonttools/blob/master/Lib/fontTools/ttLib/tables/_n_a_m_e.py#L122

Maybe. I'll take a look. Thanks for the tip.


Update SynAnno Documentation

Cross-reference #73
Description:
The existing SynAnno documentation needs refinement to reflect recent updates and changes made in the tool.
Expected Functionality:

Comprehensive overview of SynAnno's current features.
Detailed guide on cloud-volume integration and precomputed datasets.
Clear instructions on user-facing features and backend optimizations.

Proposed Solution:

Review and mark outdated sections of the current documentation.
Collaborate with recent contributors to capture all recent changes.
Introduce sections for new features and optimizations.
Ensure periodic documentation updates with significant code changes.

Addressed with PR #81


Generator Function

@sharmistharanit
I believe that we need to explain the generator function in the documentation. Can you please add it?

I do not think that I have to attach it documentation. It will be in the algorithm part. Right? @tirthbha

Yeah!! I just mean that we have to mention it in the documentation.

I do not think that I have to attach it documentation. It will be in the algorithm part. Right? @tirthbha

You have to mention a brief explanation about generator in the documentation.

I understood. I have done this. you can check now.


Bug: Alignment needs to be done check your feed and profile button

What happened?

Add screenshots
No response
What browsers are you seeing the problem on?
No response
Record

[X] I agree to follow this project's Code of Conduct
[X] I want to work on this issue

Hello @Sanchitbajaj02 please assign this issue to me under gssoc 24

Hi there, this is a duplicate issue so closing it, kindly refer to issue #305


Allow config path to be passed to Rasa X

Description of Problem:

Because users are able to pass custom config paths to rasa train, there should be CLI parity for the same.
Overview of the Solution:

Accept custom config path to override default config path, unless none is passed, then use default.
That, or allow for a config/ folder to be passed for all configuration-related files.
Examples (if relevant):

Mainly would help clean up some of the folder structure;  without using Rasa X, you can have something like this:
project_dir
├── actions
├── config
    ├── config.yml
    ├── credentials.yml
    └── endpoints.yml
├── data
    ├── nlu
    └── stories
├── logs
└── domain.yml

and pass the config file by passing --config config/config.yml
instead of having the config files clutter up the root directory.
Blockers (if relevant): N/A
Definition of Done:

[ ] Tests are added
[ ] config path handling is changed
[ ] Feature mentioned in the changelog

Thanks for submitting this feature request 🚀@wochinge will get back to you about it soon!✨

@ortsaCniveK Thanks for bringing this. I agree, that this could be useful 👍

Furthermore I would like to set a path to domain.yml and the models folder? Are there any plans to make this configurable too?

I think that's a good idea; there should be absolute parity between the rasa train command options, of which candidates are:
  -c CONFIG, --config CONFIG
                        The policy and NLU pipeline configuration of your bot.
                        (default: config.yml)
  -d DOMAIN, --domain DOMAIN
                        Domain specification (yml file). (default: domain.yml)
  --out OUT             Directory where your models should be stored.
                        (default: models)
  --augmentation AUGMENTATION
                        How much data augmentation to use during training.
                        (default: 50)
  --debug-plots         If enabled, will create plots showing checkpoints and
                        their connections between story blocks in a file
                        called `story_blocks_connections.html`. (default:
                        False)
  --dump-stories        If enabled, save flattened stories to a file.
                        (default: False)
  --fixed-model-name FIXED_MODEL_NAME
                        If set, the name of the model file/directory will be
                        set to the given name. (default: None)
  --persist-nlu-data    Persist the nlu training data in the saved model.
                        (default: False)
  --force               Force a model training even if the data has not
                        changed. (default: False)

Not sure if this is something we'd want to add to the CLI or just abstract into another config file, though.

The original request is now implemented:
-c CONFIG, --config CONFIG
                        The policy and NLU pipeline configuration of your bot.
                        (default: config.yml)

@wochinge , what do you think about implementation of the other command options:
-d DOMAIN, --domain DOMAIN
                        Domain specification (yml file). (default: domain.yml)
  --out OUT             Directory where your models should be stored.
                        (default: models)
  --augmentation AUGMENTATION
                        How much data augmentation to use during training.
                        (default: 50)
  --debug-plots         If enabled, will create plots showing checkpoints and
                        their connections between story blocks in a file
                        called `story_blocks_connections.html`. (default:
                        False)
  --dump-stories        If enabled, save flattened stories to a file.
                        (default: False)
  --fixed-model-name FIXED_MODEL_NAME
                        If set, the name of the model file/directory will be
                        set to the given name. (default: None)
  --persist-nlu-data    Persist the nlu training data in the saved model.
                        (default: False)
  --force               Force a model training even if the data has not
                        changed. (default: False)


Second to latest commit breaks discovery for editable installed projects

fe5fbe4e344fd1e822eee50d907ebd7aab031d54 breaks test discovery for projects installed with pip install -e .. I did not test yet if it also breaks without -e. If you want I can set up a repo with a minimal project for reproduction. I confirmed it works before installing, and also if I downgrade the plugin.
Here is the log, but it doesn't seem to contain anything interesting:
INFO | 2022-06-28T11:05:08Z+0200 | ...te/pack/packer/start/neotest/lua/neotest/client/init.lua:522 | Initialising client
DEBUG | 2022-06-28T11:05:08Z+0200 | .../pack/packer/start/neotest/lua/neotest/lib/file/find.lua:46 | Searching for files using command  { "fd", "--type", "f", ".", "/Users/tillb/Projects/debugging" }
DEBUG | 2022-06-28T11:05:08Z+0200 | .../pack/packer/start/neotest/lua/neotest/lib/file/find.lua:72 | Searching for files finished
DEBUG | 2022-06-28T11:05:08Z+0200 | ...k/packer/start/neotest/lua/neotest/client/state/init.lua:54 | New positions at ID /Users/tillb/Projects/debugging
INFO | 2022-06-28T11:05:08Z+0200 | .../packer/start/neotest/lua/neotest/client/events/init.lua:46 | Emitting discover_positions event
DEBUG | 2022-06-28T11:05:08Z+0200 | .../packer/start/neotest/lua/neotest/client/events/init.lua:48 | Calling listener diagnostic for event discover_positions
DEBUG | 2022-06-28T11:05:08Z+0200 | .../packer/start/neotest/lua/neotest/client/events/init.lua:48 | Calling listener status for event discover_positions
DEBUG | 2022-06-28T11:05:08Z+0200 | .../packer/start/neotest/lua/neotest/client/events/init.lua:48 | Calling listener summary for event discover_positions

I'm not seeing this when testing (just tried https://github.com/samuelcolvin/pydantic). Would be great if you can provide a repo :+1:

Ok so further investigating showed it wasn't whether the project is installed, or at least not anymore. The issue is now in what directory nvim is opened. It does not find tests when nvim is opened in the project directory (where setup.cfg etc. are), but it does work one level down in the source directory. For the following test repo, this means it does not work if nvim is open in test-nvim-dap/ but it does if nvim is opened in test-nvim-dap/debugging.
Test repo here: https://github.com/tbung/test-nvim-dap

I don't seem to be able to reproduce with that unfortunately. Can you provide logs again?

Here it is:

INFO | 2022-07-04T17:12:37Z+0200 | ...te/pack/packer/start/neotest/lua/neotest/client/init.lua:305 | Initialising client
DEBUG | 2022-07-04T17:12:37Z+0200 | .../pack/packer/start/neotest/lua/neotest/lib/file/find.lua:46 | Searching for files using command  { "fd", "--type", "f", ".", "/Users/tillb/Projects/test-nvim-dap" }
DEBUG | 2022-07-04T17:12:37Z+0200 | .../pack/packer/start/neotest/lua/neotest/lib/file/find.lua:72 | Searching for files finished
DEBUG | 2022-07-04T17:12:37Z+0200 | ...k/packer/start/neotest/lua/neotest/client/state/init.lua:54 | New positions at ID /Users/tillb/Projects/test-nvim-dap
INFO | 2022-07-04T17:12:37Z+0200 | .../packer/start/neotest/lua/neotest/client/events/init.lua:46 | Emitting discover_positions event
DEBUG | 2022-07-04T17:12:37Z+0200 | .../packer/start/neotest/lua/neotest/client/events/init.lua:48 | Calling listener summary for event discover_positions
DEBUG | 2022-07-04T17:12:37Z+0200 | .../packer/start/neotest/lua/neotest/client/events/init.lua:48 | Calling listener status for event discover_positions
DEBUG | 2022-07-04T17:12:37Z+0200 | .../packer/start/neotest/lua/neotest/client/events/init.lua:48 | Calling listener diagnostic for event discover_positions

I'll investigate if it is some weird interaction with another plugin, I also haven't updated nvim in a couple days, but just to be clear, this only happens if pwd is ~/Projects/test-nvim-dap, not if pwd is ~/Projects/test-nvim-dap/debugging. If you have any suggestions what I should test, please tell me.

 There should be a line saying initialisation finished.
Also can you paste your config for neotest/neotest-python?

Nope, that's all there is in the log file, I did not remove anything, just deleted the file, opened neovim, tried running a test, closed neovim and copied the log over.
Here's the config:
use({
  "rcarriga/neotest-python",
  -- commit = "de234d600e728f2350950355bb5c8d723ea7dff7",
})
use({
  "rcarriga/neotest",
  requires = {
    "nvim-lua/plenary.nvim",
    "nvim-treesitter/nvim-treesitter",
    "antoinemadec/FixCursorHold.nvim",
  },
  config = function()
    require("neotest").setup({
      output = {
        enabled = true,
        open_on_run = "yes",
      },
      adapters = {
        require("neotest-python")({
          runner = "pytest",
          dap = { justMyCode = false, console = "integratedTerminal" },
        }),
      },
    })
  end,
})

I also experience this. My config is the same as from @tbung .
Fixing the commit to https://github.com/nvim-neotest/neotest-python/commit/de234d600e728f2350950355bb5c8d723ea7dff7 does indeed resolve the issue.
The repo, I tested this with is https://github.com/Lightning-AI/lightning installed locally with PACKAGE_NAME=pytorch pip install -e .

Ah yep looks like it go stuck when poetry wasn't installed. I've pushed a fix, please try now

Can confirm, it works now! Thanks for debugging @justusschock and thanks for the quick fix @rcarriga!


react lifecycle events missing

"@ionic/react": "^0.0.5",
i cannot use lifecycle events such as 'onIonViewDidEnter' inside the IonRouterOutlet will this be added soon? Is there any hack - ie, listening to a dom event?

Fixes for this will be coming up in the next beta release


WIP:  add support for custom gravity

Gravity is not always along the Z-axis.
It is possible to define the direction of gravity in the project settings, and some games might require this.
This MR is a start to get this template to support the gravitational settings instead of assuming gravity will always be the Z-axis.
Still a work in progress and anyone is welcome to join in.

This is out of scope for a FPS Sample Project. Feel free to release your own with added features!


Publish New Release Soon?

Any chance a new release will be published soon with the 16 new commits since 1.0.0?

Closed with https://github.com/thefrontside/emberx-file-input/commit/a7ec862cabacf15a0fcdc206f4813aabeba90887
Sorry it took so long! I was going to cut a release a little while ago but travis CI started failing builds and I didn't have time to debug travis until now.


[Change Request] page 1190 "Create Payment" - procedure UpdateDimensions

Describe the request
Hello, can you please change the local procedure UpdateDimensions to protected procedure?
    protected procedure UpdateDimensions(var GenJnlLine: Record "Gen. Journal Line"; TempVendorPaymentBuffer: Record "Vendor Payment Buffer" temporary)
    var
        DimBuf: Record "Dimension Buffer";
        TempDimSetEntry: Record "Dimension Set Entry" temporary;
        DimVal: Record "Dimension Value";
        DimBufMgt: Codeunit "Dimension Buffer Management";
        DimMgt: Codeunit DimensionManagement;
        NewDimensionID: Integer;
        DimSetIDArr: array[10] of Integer;
    begin
        if GenJnlLine."Dimension Set ID" = 0 then begin
            NewDimensionID := GenJnlLine."Dimension Set ID";

            DimBuf.Reset();
            DimBuf.DeleteAll();

Additional context
This allow page extensions to modify the visibility of page elements after data updates.
Internal work item: AB#507945

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 18941.


clarification regarding this setting ' Anonymous users can join a meeting'

Hi,
Anonymous users in this context is referred to as a non-guest users within tenant, right? Which means even if you send a teams meeting invite directly to a users, (and if they aren't added as guest) they are seen as a anonymous user?
Regards

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

ID: 470478ed-3b4e-ced8-0e28-6cda545b8400
Version Independent ID: 95b4044b-3a47-a511-c726-b5ee75863441
Content: Manage meeting settings - Microsoft Teams
Content Source: Teams/meeting-settings-in-teams.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

@xNiklasJern Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

Hi @scanum ,
This isn't an issue more of general concern or question on the documentation? please reopen


fix(modal): allows user to insert custom close function based on close event location

Closes IBM/carbon-components-angular#1027
{{short description}}
Changelog
New

{{new thing}}

Changed

{{change thing}}

Removed

{{removed thing}}

Couple suggests, but otherwise looks good!

Interesting, I did not add the flattened styles... wonder where it came from 0.0

It looks like this might be a breaking change to some people?
Maybe we need to default to something or have an option (a default option perhaps) that does the old behavior?

I'm not sure how this changes any behaviour? If you don't do anything with the value passed in it will still always close the modal. We could make the trigger argument optional (trigger?:), since that may typecheck differently now, but otherwise there's be no actual behaviour changes
This is in AlertModal, not Modal if that clears things up?


No event data available when manually triggering validation event

My use case :
I have a custom built multi-step JS form where submission takes place only after the last step of the form has been filled in, and each step needs to be validated before one can move on to the next.

My code :
I created a validation callback like this :
var $form2 = jQuery('#form').parsley().subscribe('parsley:form:validate', function(formInstance) {
    // if one of these blocks is not failing do not prevent submission
    if (!formInstance.isValid()) {
        if (typeof (formInstance.submitEvent) !== typeof (undefined)) {
            // else stop form submission
            formInstance.submitEvent.preventDefault();
        }
    }
    return;
});

I'm manually triggering the validation event by clicking a link :
jQuery('.control').click(function(e) {
    if (!$form2.validate()) {
       e.preventDefault();
    }
});

My problem :
In my callback, formInstance.submitEvent is undefined.
In my callback, I do not have access the click event of my control.
I also do not seem to be able to pass theclick event of my control nor any other data to the validation callback (see also https://github.com/guillaumepotier/Parsley.js/issues/862).
Is there another way to pass whichever control was being pressed to my validation callback?

From our CONTRIBUTING guide:
Questions?
Please ask questions on StackOverflow and be sure to include the parsley.js tag. Please provide an example, starting for example from this jsfiddle

@marcandre:
I'm not entirely sure, but I think this is a missing feature.


Update developer documentation for the new build process

The new process is something along the lines of:

sudo npm install -g grunt-cli
npm install
grunt
grunt run

if it's that simple, there should be a build script or a make file.

I think we can at least roll the sudo npm install -g grunt-cli into the npm install step and have the grunt and grunt run steps become npm test and npm run steps, but we really can't make it much more simple than that. Plus, we already have a build script (Gruntfile.js).


Some updates to the cml script

Some minor updates with new options and use of pathlib.Path to make argument handling easier.

Also I removed the _{group_id} from the block name. After conferring with our collaborators at MIT we decided to rely on the block IDs for materal identification instead.

Group ID removal is an issue for Trelis 17.1 because of VOID or SOURCE (both common names) being protected names. May be an issue for newer versions of Cubit.
Everything looks good


Fix: Upgrade set-value version and tests

Updating set-value version from 3.0.0 to 4.1.0 since the v3 has a security vulnerability.
CVE-2021-23440

Sorry for the slow response, I’m just trying to get the bills paid at the moment, so I can carve out more time for open source. I’ll try to get to this over the weekend.
Sent from my iPhone

On Oct 7, 2021, at 10:41 AM, Spencer Cousino @.***> wrote:

@jonschlinkert any chance you'll be addressing this soon?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Hey any updates on this? This relates to security issues in React and React Native. Lots of projects depending on this. The work seems to already be done, it just needs merged and released.

Also nuxt projects,
nuxt > @nuxt/builder > @nuxt/webpack > webpack > watchpack > watchpack-chokidar2 > chokidar > anymatch > micromatch > extglob > expand-brackets > snapdragon > base > cache-base > union-value > set-value

This project needs some donations to speed things up. :)

This project needs some donations to speed things up. :) Ups. I'm to late

If this actually isn't being done because he's trying to force donations that is ridiculous and disgusting. The work is already done, and not by him. This is a security vulnerability, not a feature request. Do the right thing and get this done @jonschlinkert or let someone else take over. Don't hold the open source community for ransom for some freaking donations.

Whoa, hey guys, let's tone it down a bit. This isn't a good approach to achieve what we all want.

If this actually isn't being done because he's trying to force donations that is ridiculous and disgusting.

@scousino no one is doing anything of the kind. It's the opposite. Vytenis trolled me and is attempting to force me to address the issue by shaming me.  Here is his comment to me on LinkedIn, after I said "thank you" for a sponsorship:

Regardless of the issue or topic, using shaming tactics to get maintainers to do anything at all is completely unacceptable and IMO violates GitHub's Contributor Covenant, and is grounds for immediately banning from all of my projects. I have already banned Vytenis on GitHub, and I blocked him on LinkedIn. I don't need that negativity, and we certainly don't need this tone in the community.
It's worth considering that when you read a random troll's comment about someone and then assume the absolute worst, it says much more about you than it does about them.  I don't owe you an explanation, but I'm going to give you one anyway: I'm struggling at the moment. I'm a single father with two girls in elementary school. I take care of them 100% on my own, and I'm barely scraping by financially. I also barely have enough time to do what's necessary to live every day, much less contribute to open source. To be clear, I do contribute, and I will be contributing much more as I get my feet under me.   IMHO, it makes far more sense for us to be encouraging businesses to sponsor maintainers, than to be discouraging maintainers for wanting to do this full time. If we're picking battles, I'd focus on the later.
I never made sponsorships a "condition" of my contributions or anything of the sort, and I never would. I am, however, hoping with every fiber of my being that I can find a way to pay my bills and also contribute to open source. Even better if I can do it full time. Negative comments won't help me go faster, and it doesn't help with morale either.

@scousino I'm not going to delete your comment yet. I'll leave that to you to decide. I don't think it will reflect well on you or this discussion. If, however, your comment attracts additional negativity, I'll most likely delete it.  I've just created an issue for what's causing v4 to fail tests, if we dig deeper maybe we can find the cause and it get's resolve faster. :v:

Done. Thanks for the contributions and for your patience.

Thanks for the fix! Two suggestions for the future:

set better expectations on when things will be fixed
look for another project member to help handle urgent security fixes such as this one

Your welcome, and I’m sure you’ll do well when you get more experience. Good luck!
Sent from my iPhone

On Nov 6, 2021, at 8:46 PM, Cong @.***> wrote:

Thanks for the fix! Two suggestions for the future:
set better expectations on when things will be fixed
look for another project member to help handle urgent security fixes such as this one
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Hey @jonschlinkert! We are with you, keep it up, rest of the community is on a positive and supporting side!
Now we know what you're struggling with and all your contribution and opensource projects are valuable for the depending projects!
What may worth a thought is how can ease your weight from your shoulders if you split the responsibility of these maintenance work with some folks you trust, because projects like angular, nuxt, react-native couldn't provide the necessary security level what they promised to their users. Yes, on one hand, they shouldn't have promised it in the first place, but you, as the owner of these packages/projects also have the responsibility to be a "good" dependency.
Some say maintaining an OSS is pain in the neck because of the above, therefore they didn't even upload their projects to github, so I feel you, I know it can be hard. The key is to find the balance between appreciation and commitment, they walk hand-in-hand.


A set of ideas for fashion

I haven't fully studied the plugin yet, maybe some ideas have already been implemented
1 /deathlog player similar to https://www.curseforge.com/minecraft/mc-mods/deathlog A very necessary thing on the server, especially when players die due to mod bugs, you should not trust them in words
2 Improved invsee, some mods use their containers to store things, I believe you can implement support for a full view of the entire inventory as well as the ender of the player's chest
3 /createkit - the ability to customize the issuance of things by slots, it is more convenient than adding everything through the code. In addition, make a default set of Newbie which will be used to issue things to new players (1 visit to the server)
4 /chatcolor - sets the default color of the player text
5 /killall The command will kill specified types of mobs for example --monster, --animals
6 /maxplayer Changes maximum amount of players who can connect to server - and the permission to circumvent the restriction
7 /openbook Open book gui. Optionally you can provide exact file which needs to be loaded and shown.
8 /patrol Teleports to random players without repeating same player until all players have been visited.
9 change /day, /night let the cycle from night to day and vice versa pass smoothly, it will look beautiful.
10 /ride Ride target entity
11 /sit in addition, you can use the support of floor blocks on which the player can sit by right-clicking
https://proxy.spigotmc.org/673d00852dd28f20aee4d49175658424c20a873d?url=https%3A%2F%2Fi.imgur.com%2FRM9ZGHF.png
12 /lay

13 /playersit
https://proxy.spigotmc.org/bb7556c3a72e34e897dddfe818beb426c7c46706?url=https%3A%2F%2Fi.imgur.com%2FqhJNYoE.png
14 /unloadchunks Unloads chunks from server memory
15 /homes GUI player home
16 /freeze Suspends players actions
17 /actionbarmsg Sends action bar message to player

1 Already in the mod, try /deaths.
2 I suppose I could do this, but I don't really see the benefit.
3 Already in the mod.
4 Might do this in Chattix instead.
5 Already in the mod, try /wipeout.
6 I guess I could make a gamerule to bypass this limit, but I don't really see the use in a command to change this value in-game. Especially as it probably wouldn't save it to the disk anyway.
7 Don't get what use this has. Do you mean a command to edit signed books?
8 Don't really see the use in this.
9 Don't really see the use in this.
10 I guess I could do that.
11 You can already right click stairs to sit on them, but I'll leave it at that.
12 Don't really see the use in this.
13 Basically the same as 10
14 I could look into this, I guess. But chunks already get unloaded when they're not needed anyway. Doing it manually would probably just break stuff.
15 Don't really see the use in a GUI for homes
16 I could do this, but it's really hard/near impossible to properly integrate this with mods without breaking stuff.
17 I guess I could do that

I will not be implementing any of these ideas as either I wasn't intending to anyway or they are no longer relevant.


Understanding support vector machines

Brief Summary:
Support vector machines are set of supervised learning methods mainly used for classification problems though they can also be used for regression and outlier detection, they are very efficient in high dimensional spaces and are  memory efficient since they only use  a subset of the training points in the decision function
Key Takeaways:

Understand what support vector machines are.
Appreciating what support vector machines are  used for in machine learning.
Understand the matematical formulation behind support vector machines.
Be able to solve a real world problem using  support vector machines.

References:
N/A

good topic @mykolkiprono - approved

closing - can reopen a new issue with new topic 👍


Serialization for Immutable objects other than List/Map

Since Immutable already does .toJSON and .fromJS, and those functions support List/Map, I'm looking for a serialization library that can do more than just those. Maybe not Records, because I'm not sure how functions would be serializable, but Sets, and OrderedMaps come to mind as being really important.

The main advantage currently provided is the preservation of complex map keys - as this was the particular problem I had at the time of writing.
I think it is pretty easy to support the other built-in data types using similar code to List and Map - I'd take a PR for this. I'll try and add it myself if I have time and no-one else has done so.
Records would also be fairly straightforward, but the user would need to provide a lookup function to locate the constructor of given a record name

Support for Sets and OrderedMaps has been added in b323bc5bf90eabd71644d64cd2e4655409baa9e6 and published as v0.2.0

@glenjamin That's awesome, thanks for adding it :+1:


Include example to obtain the MSI token for a Java function/App

The document doesn't include documentation on what package/class to use for Java function. More specifically, the following package/class needs to be used in case of a Java function
https://azure.github.io/azure-sdk-for-java/com/microsoft/azure/credentials/AppServiceMSICredentials.html
Also there are some typos in the document. In the examples we are appending a slash in front of the MSI_ENDPOINT which already contains a trailing forward slash.

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

ID: f57d3177-ce9e-ebfa-03a7-45beb8ca3212
Version Independent ID: a59df23d-f3ac-9809-61d3-5741e31f020a
Content: Managed identities overview - Azure App Service
Content Source: articles/app-service/overview-managed-identity.md
Service: app-service
GitHub Login: @mattchenderson
Microsoft Alias: mahender

@kaushalp Thanks for the feedback! We are currently investigating and will update you shortly.

@kaushalp Thanks for bringing this to our attention. Your feedback has been shared with the content owner for further review.

thanks @SnehaAgrawal-MSFT
@mattchenderson There is a bug in the Java SDK which doesn't provide a way for the clientid parameter to be passed. I have raised an issue for this here: https://github.com/Azure/autorest-clientruntime-for-java/issues/609

Any updates on this?

Updated with info on Azure Identity client library for Java and links to code samples: https://github.com/MicrosoftDocs/azure-docs-pr/pull/186513.
#please-close


⚠ Punto de Fuga has degraded performance

In a5a24e8, Punto de Fuga (http://puntodefugapodcast.com) experienced degraded performance:

HTTP code: 200
Response time: 11659 ms

Resolved: Punto de Fuga performance has improved in 7bc612d.


feat(web-app): add dataviz pages

Closes #916
Add data viz collection pages
Changelog
New

Data viz pages

Changed

fix for aside component styles

Testing / reviewing
http://localhost:3000/data-visualization/axes-and-labels
http://localhost:3000/data-visualization/color-palettes
http://localhost:3000/data-visualization/chart-anatomy
http://localhost:3000/data-visualization/get-started
http://localhost:3000/data-visualization/legends
https://carbondesignsystem.com/data-visualization/dashboards/

@alisonjoseph I'm assuming this small bug should be opened up on the carbon repo, correct? vanilla -> Vanilla

@andreancardona yes that would be in the Carbon repo, however I'm not sure if vanilla should be capitalized, since its not a framework brand name like React, Angular.


fix theme

fix classes, use jsdelivr, remove usrbg, autocomplete background

Give me a bit to look over this, and then I'll merge.
I made this theme for a friend (who doesn't use it), which is why it wasn't updated, but I'll be more than happy to merge


Releases table in admin area

The releases table is now located in the admin area. The checkbox and date filters are removed. I added a details dialog with the info from the table but did not yet add more fields. The Import Button and the marking of duplicates are also not in the scope of this PR. I'll create separate cards for them. The ReleasesController was deleted as not needed anymore.

 [MetalDetector] Releases in admin area

Like we talked about: I reintroduced a controller for the releases page and a unit test for it.


fix: allowlist for source plugins with port forward

On the EDA Server side we try to open ports for port forwarding using the container engine (podman/k8s). We only allow a specific set of source plugins listed in the settings
SAFE_PLUGINS_FOR_PORT_FORWARD to be eligible for port forwarding. The default settings only supports the source plugin
ansible.eda.webhook
The user can augment this settings to include other source plugins
https://issues.redhat.com/browse/AAP-17416

@mkanoor For users using other webhook like plugins but not ansible.eda.webhook, their activations will fail after upgrading eda-server unless they modify the settings for SAFE_PLUGINS_FOR_PORT_FORWARD. Is this a breaking change that cannot be included in a minor release?
The default should include ansible.eda.alertmanager which is provided in our collection.


pom.xml:489-492: This limits are set according to current...

The puzzle 1067-ac9afd75 from #1067 has to be resolved:
https://github.com/yegor256/takes/blob/a9ec437a3b285b903aa0b8995d2f356855cc4db7/pom.xml#L489-L492
The puzzle was created by andreoss on 23-Oct-20.
Estimate: 30 minutes,  role: DEV.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The puzzle 1067-ac9afd75 has disappeared from the source code, that's why I closed this issue.

@0pdd the puzzle #1112 is still not solved.

@0pdd the puzzle #unknown is still not solved; solved: #1130.

@0pdd all 2 puzzles are solved here: #1130, #unknown.

@0pdd the only puzzle #1112 is solved here.


Update invoices and PDF download to include USD

Description
Requested by a customer for accounting purposes:
https://linode-internal.ideas.aha.io/ideas/MGR-I-260
Type of Change

Non breaking change ('update')

Note to Reviewers
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration.

I had to adjust the column widths a bit on the PDF for it to look ok.


Remove spurious 'make clean' steps which (now) delete our packages (see #1831)

I put this onto release manually to verify. It correctly released 0.13.0, so we are looking good.
Thanks @killerswan.


Anti forgery token is not generated when form does not have tag helpers inside

Create a form with no MVC tag helpers inside and anti-forgery token does not get generated.

@alvipeo the different behaviour actually relates to the attributes you've included in the <form> tag, not to tag helpers inside the element. The FormTagHelper does nothing (i.e. the <form> element just has HTML semantics) unless you include an asp- attribute. If all you want is the antiforgery token, add asp-antiforgery="true".

ok. here's my form:
<form asp-action="godown" role="form" method="post" class="appointment">

Still no hidden field in the source.

Hmm, that's odd. (I was also incorrect in my earlier comment: The no-op case occurs only if the action attribute is included in the <form> tag.)
Did you leave out the </form> end tag or self-close the element? Need a place to stick the <input type="hidden" .../> element.

ok, here's something weird - I had this issue https://github.com/aspnet/Home/issues/1144 and I fixed it and now I get the token.

@alvipeo glad you're all set. Closing this issue.


Fiware service returns path

This line returns the fiware service path but it needs to return the fiware service
https://github.com/RWTH-EBC/FiLiP/blob/57d72eb4c1db8f7863dfe0ca7ccf739b01473608/filip/clients/base_http_client.py#L110

return self.fiware_headers.service


chore: move to uupd as the main name

whoops, I should've done a global search and replace. Thanks so much for the PR!

no worries LOL


CONTRIBUTING.md: add sections on contributing code/documentation

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? 👉 N/A
[x] Have you successfully run brew tests with your changes locally?

This adds brief instructions on contributing code and improving command documentation/help. The latter section was inspired by https://github.com/Homebrew/brew/pull/270#issuecomment-221330769. Minor adjustsments are also made to the styling/wording of the only section that looks different from what we usually use in this file and its homebrew/core counterpart.

@UniqMartin Can you address the comments and get this 🚢d? Thanks!

Can you address the comments and get this 🚢d? Thanks!

I will as soon as possible. Sorry for letting this linger for way too long …

Gentle ping on this.

Passing on this for now but will review another PR with changes made. Thanks!


Conflicts with barrelsby (raises warning "Encountered an unknown block tag @file" for "@file Automatically generated by barrelsby" comment)

Search terms
barrelsby
Expected Behavior
No warnings.
Actual Behavior

Steps to reproduce the bug
An example of barrelsby generated file:
/**
 * @file Automatically generated by barrelsby.
 */

export * from "./ToolPing";
export * from "./ToolScoreboard";

I understand that it can also be considered as a missing feature in barrelsby (after all, those comments are pretty much useless, and they don't have a way to turn them off in barrelsby settings). But anyways, maybe typedoc can also do something with this @file?
Environment

Typedoc version: 0.27.5
TypeScript version: 5.2.2
Node.js version: v20.13.0
OS: MacOS, Linux

That helped, thanks!


hdkeychain: Update NewMaster to accept network.

This changes the NewMaster function to accept the network the generated extended master key is associated with.  This could previously be done by calling SetNet on the returned extended key, but that approach is more error prone since it is easy for a caller to forget to do it or never know they should to begin with.
Closes #65

Since the API is changing, should you drop SetNet, or is it still useful somehow?

It's still useful in some circumstances.

OK, and OK on the PR.

ACK

Reviewed 2 of 3 files at r1, 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from the review on Reviewable.io


Proposal to add wiki for translating rxjs to most equivalents

I think it would be nice to get a wiki going that has a table showing the most equivalents to rxjs commands

This is awesome! thanks so much :+1:


themes demo available in http and https breaks some functionalities

Hello,
themes are available with and without https
it seems to break some JS or CSS on demo websites (on the homepage page only):
http://themes.gohugo.io/theme/docdock/

live search (which rely on a lunr.js index file) broken
font awesome icon (css) broken

https://themes.gohugo.io/theme/docdock/

live search OK

font awesome OK

With Chrome Network debugger you can easily find the issue, the index page is http and all css and js links are https.

On my side (thanks to your advice) i use netlify to host the theme demo and use //docdock.netlify.com as base URL to make it work with http and https.

Problem solved ^^

Well, thinking about it, the issue described here isn't really fixed, as the demos are still available in http ... suggestions welcome.

HTTPS by default.

HTTPS by default.

OK, I meant "non-obvious suggestions welcomed".

Haha. Okay. Is it possible to add CORS headers?

If I'm reading the Font Awesome troubleshooting Wiki that's one of the suggestions for cross-origin requests: https://github.com/FortAwesome/Font-Awesome/wiki/Troubleshooting.

Alternatively, given the resources are being served from the same domain it may be possible to relax the same-origin restriction: https://html.spec.whatwg.org/multipage/origin.html#relaxing-the-same-origin-restriction

I tried to add a redirect with netlifies "_redirects" support, see if that works.

That did not work, looks like we have to do this on "DNS level". /cc @spf13

We can of course use "//" for the demo sites, of course. I will hack that in some time later.

if you publish your website with www.website.com the font won't work if a page is called with website.com (no www), and this is the case when people type your url (they usually won't add the www).
the opposite is also true, if you publish with website.com and then someone reach a page with www.website.com for some reason, the font won't work
you can fix this by adding
Header set Access-Control-Allow-Origin "*"
to your .htaccess

It's a little known fact going cross origin for sub-domains only can be performed using JS in the browser. Learned this one back in 2007: https://html.spec.whatwg.org/multipage/origin.html#relaxing-the-same-origin-restriction

Is this issue still present?

No

Thanks for the status update. I'll close this issue then.


Olivia Zoe: Watch pinpinterest.com

Olivia Zoe requests the watch of the watch_keyword pinpinterest\.com. See the Metasmoke search here and the Stack Exchange search here.

pinpinterest\.com has been seen in 0 true positives, 0 false positives, and 0 NAAs.

Coverage remained the same at 65.087% when pulling ad029370112d7ad69132582616c36520a807ef26 on auto-blacklist-1523619508.2430067 into 4451757b55a6991a64a0979bb1bb5c7fdb0c4dbf on master.

Reject, needs adjustment

Duplicates #2005


Adding support for Kurdish language

I  want to add  Kurdish language to my app ,but i have not see any parameter for change  months and days  names

this is something you need to add for your app, there are plenty of plugins for adding multi-language support for app

Hi @hussenIbrahim
To add a language, please check https://flutter.dev/docs/development/accessibility-and-localization/internationalization
This platform is not meant for assistance on personal code. Please see https://flutter.dev/community for resources and asking questions like this,
you may also get some help if you post it on Stack Overflow and if you need help with your code, please see https://www.reddit.com/r/flutterhelp/
Closing, as this isn't an issue with Flutter itself. If you disagree, please write in the comments and I will reopen it.
Thank you


#1165: Remove the concept of table suffixes

Resolves #1165

UAT tables (at least) must be removed before this PR is deployed


Use a checkmark icon instead of the envelope icon on the Package page

I forget where you suggested this @finestructure, but I liked the suggestion!

I've just been working on this and while I agreed with your suggestion at first, now I come to implement it, I don't 😂
So a checkmark on its doesn't fit with the weight of the rest of the icon, it's too sparse:

Which one is the odd one out? 😬
So the natural solution to that would be to stick it in a circle, or a square to make the weight of the icon match. But then I think that's too similar to the "stable" icon we have.
What do you think?

How about just reusing the stable one? Might be ok because it's black/dark grey?

I think using the same icon for two things is a little troublesome, logically.

Hold on, I have an idea.

True. Maybe just an outline circle?
Also I just noticed that the hour hands seem to be off centre. I think the tip is in the center, not the centre line.

The weighting might not look right with this, but it's different enough to be distinct from the stable icon. I'll give it a try in-place.

Also I just noticed that the hour hands seem to be off centre. I think the tip is in the center, not the centre line.

Fixed

I don't like the checkbox icon, it's just nowhere near "heavy" enough to match the rest.

This isn't bad, maybe without the check mark?

ChefsKiss.gif

That looks nice!


Make normalize optional in openeye.get_charges

Added an optional keyword argument, and a unit test that sets it to False.

Is there a specific way I can run (just the) openeye nosetests on my local computer?
This doesn't seem to work:
nosetests -vv --exe openmoltools/tests/test_openeye.py
nose.config: INFO: Ignoring files matching ['^\\.', '^_', '^setup\\.py$']
Failure: NameError (name 'openeye_exception_message' is not defined) ... ERROR

======================================================================
ERROR: Failure: NameError (name 'openeye_exception_message' is not defined)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/bas/miniconda3/envs/py2k/lib/python2.7/site-packages/nose/loader.py", line 418, in loadTestsFromName
    addr.filename, addr.module)
  File "/home/bas/miniconda3/envs/py2k/lib/python2.7/site-packages/nose/importer.py", line 47, in importFromPath
    return self.importFromDir(dir_path, fqname)
  File "/home/bas/miniconda3/envs/py2k/lib/python2.7/site-packages/nose/importer.py", line 94, in importFromDir
    mod = load_module(part_fqname, fh, filename, desc)
  File "/home/bas/Repos/openmoltools/openmoltools/tests/test_openeye.py", line 37, in <module>
    @skipIf(not HAVE_OE, "Cannot test openeye module without OpenEye tools.\n" + openeye_exception_message)
NameError: name 'openeye_exception_message' is not defined

----------------------------------------------------------------------
Ran 1 test in 0.001s

FAILED (errors=1)

Added a little fix to the nosetests. Maybe these were broken all along, but no one ever ran them because of travis skipping them?

Try nosetests -vv --nocapture --with-doctest openmoltools.tests.test_openeye

One fail, but not something caused by my PR.
FAIL: openmoltools.tests.test_openeye.test_butanol
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/bas/miniconda3/envs/py2k/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/bas/Repos/openmoltools/openmoltools/tests/test_openeye.py", line 61, in test_butanol
    assert m1.NumConfs() >= 2, "Butanol should have multiple conformers."
AssertionError: Butanol should have multiple conformers.

No clue what is causing the particular error, but this PR is good for merging as far as the added feature is concerned.

That failure is because the default behavior of get_charges() was changed in https://github.com/choderalab/openmoltools/pull/147 but the test was not changed appropriately.  I'm not sure how that happened.
Can you add the argument keep_confs=-1 to the call to get_charges() here?
https://github.com/bas-rustenburg/openmoltools/blob/master/openmoltools/tests/test_openeye.py#L59

Done.

Woohoo!  Thanks!

Yes, thanks! And thanks for catching the issue with the test. I probably
botched that when making a change before we had Travis testing the OpenEye
stuff, and apparently Kyle didn't notice when checking, either.
On Wed, Oct 28, 2015 at 3:39 PM, John Chodera<EMAIL_ADDRESS>wrote:

Woohoo! Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/choderalab/openmoltools/pull/159#issuecomment-152019016
.

--
David Mobley
Associate Professor
Department of Pharmaceutical Sciences
Department of Chemistry
3134B Natural Sciences I
University of California, Irvine
Irvine, CA 92697
<EMAIL_ADDRESS>work<PHONE_NUMBER>
cell<PHONE_NUMBER>


Remembers users' city selections

Fixes LD-159. Also fixes a crash for shows with missing cover_image properties. Also fixes the problem where the checkmark wasn't being displayed in the city picker.
#native_no_changes #skip_new_tests

Fails

:no_entry_sign:

Missing Test Files:

src/lib/Scenes/City/__tests__/CityPicker-tests.tsx

If these files are supposed to not exist, please update your PR body to include "#skip_new_tests".

  Generated by :no_entry_sign: dangerJS

Looks good!