single-row tables have poor spacing in header

this is a recent regression from a layout optimization that removed padding-right from table headers, assuming the sort SVGs were always there. but we don't use isSortable for single-row tables.

nvm, on a dev branch


Add cbmc tests for xProcessReceivedTCPPacket_IPV6

Description

Test Steps
CBMC test
python prepare.py
make

Checklist:

[x] I have tested my changes. No regression in existing tests.
[x] I have modified and/or added unit-tests to cover the code changes in this Pull Request.

Related Issue

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Closing due to re-creating PR #904.


GDScript(2.0) Documentation generation system

Suppression of my initial pr(https://github.com/godotengine/godot/pull/39359). Refactored with GDScript 2.0

Bugsquad edit: This closes https://github.com/godotengine/godot-proposals/issues/993.

Good job. If it's not too difficult, would be nice highlight the comment, like happens with syntax """ Comment """.

@AndreaCatania IMO, making the comments/documentations less visible from the source is easy to read the code,
here is a side by side comparison

Usually the documenting code is marked with a different colour. Probably that yellow is wrong and too strong for its purpose.

Perhaps a dark purple or dark green color will work. They are not used.

@vnen completed!

Hello @ThakeeNathees, very sorry for the delay on this and because of that now it has many conflicts. Are you willing to fix those? We can do a final review then (so far the code looks mostly good to be merged, just the conflicts and minor details are in the way). If you don't want to I can do it. I think it will be easier for you because you're more familiar with your code. I can help if you have doubts about something.

@vnen Thanks, I'll fix those conflicts soon

There are some style issues to fix: https://github.com/godotengine/godot/pull/41095/checks?check_run_id=1470115199
Maybe you can also amend the commit message for the second commit to be a bit more explicit, e.g. "Refactor DocData into core and editor (DocTools) parts".

Thanks hugely for this amazing feature, and for following up on belated reviews after GSoC :)

Will this feature ever be available in 3.x?

@ithinkandicode That's unlikely, as this feature is built on top of the changes in master which are not compatible. Nothing is impossible, but that would require dedicated 3.x GDScript maintainers, and there is currently nobody interested in doing that work.

I am noticing the tooltips don't update until restarting in 4 beta 11. Is this intended behaviour or am I missing an option to tell them to regenerate?

That's a known issue / expected behavior for any change that impacts an already open inspector for the node you're editing. If you click another node and then back to this one, it should refresh.

@akien-mga Thanks for the reply, that was helpful. Have a good one!

The tooltips don't seem to generate if the variable is within an export category

The tooltips don't seem to generate if the variable is within an export category

Yeah can confirm this happening for me as well.

Not sure why this is receiving comments lately so I locked to avoid cluttering the discussion. If you have a problem with the feature then open an issue about it since comments in merged pull requests tend to be forgotten.


Improve documentation on generated registers and fields

Previously, the descriptions for fields were only listed on the docs for the accessor methods on the R and W structs.  This adds the descriptions to the field accessor structs so they appear on the register module page, too.

r? @reitermarkus
(rust_highfive has picked a reviewer for you, use r? to override)

bors r+

bors retry


Use Run-AlPipeline cmdlet to compile Test automation extension

Hello,
I tried to create a pipeline to build (compile and generate the app file) for a test automation BC extension.
No compilation when launch the cmdlet locally on the server (For other extension it works correctly).
Context:
Docker on windows server 2019.

At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.12\ContainerHand
ling\New-NavImage.ps1:445 char:25
+ ...             Copy-Item -path $licenseFilePath -Destination "$dbPath\CR ...
+                 ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : ObjectNotFound: (C:\Users\pedro.pereraaguila\Model:St
   ring) [Copy-Item], ItemNotFoundException
    + FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.CopyItemCo
   mmand

Additional context

does it happen all the time? Yes.
did it use to work? Yes.

Thanks in advance for your help,

File structure in the server

It looks like the error is when trying to copy the license file, but I cannot see that you specify a license file???

-appfolders should be the relative path of your apps (relative to basefolder)
-testfolders should be the relative path of your test apps (relative to basefolder)
imageName is "VE_<IP_ADDRESS>10615.app" - I cannot understand how this value is set (you do not set ImageName in your parameter set) ????

Hello Freddy,
Thank you for your answer.
No, I didn't specify a license, I thought that with BC18 the license was no longer needed for the test toolkits.
I tried again with our license and got a new error. What is strange is that it works with the main extension, not with the test automation extension.
 Toolkit       False
CopySymbolsFromContainer    True
enableCodeCop               False
enableAppSourceCop          False
enableUICop                 False
enablePerTenantExtensionCop False
escapeFromCops              False
useDefaultAppSourceRuleSet  False
rulesetFile
azureDevOps                 False
License file                Specified
CodeSignCertPfxFile         Not specified
TestResultsFile             C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test
-automation\TestResults.xml
TestResultsFormat           JUnit
AdditionalCountries
PackagesFolder
OutputFolder
BuildArtifactFolder
CreateRuntimePackages       True
AppBuild                    0
AppRevision                 0
Install Apps

C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model\mymainextension.app
Install Test Apps

None
Previous Apps

None
Application folders

C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation
Test application folders

C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation\test

/ |              | | ()                             | |      ()
| |     _ __ ___  __ | | _ _ __   __ _    ___ ___  _ __ | | __ _ _ _ __   ___ _ __
| |    | '__/ _ /  | __| | '_ \ / _ |  / _/ _ | ' | / ` | | ' \ / _ \ '|
| || | |  __/ (| | || | | | | (| | | (__ () | | | | |_ (| | | | | |  __/ |
_||  ___|_,|_||| ||_, |  __/|| ||__,||| ||___||
/ |
|_/

BcContainerHelper is version 2.0.12
BcContainerHelper is running as administrator
Host is Microsoft Windows Server 2019 Standard - ltsc2019
Docker Client Version is 20.10.4
Docker Server Version is 20.10.4
Fetching all docker images
Fetching all docker volumes
ArtifactUrl and ImageName specified
Building image model:onprem-18.1.24822.26330-w1 based on mcr.microsoft.com/businesscentral:10.0.17763
.1999 with https://bcartifacts.azureedge.net/onprem/18.1.24822.26330/w1
Pulling latest image mcr.microsoft.com/businesscentral:10.0.17763.1999
10.0.17763.1999: Pulling from businesscentral
Digest: sha256:330f2f492453232dc1dfd6a1d0ca16174e4372d4f1bfb7211d4eb503ce112897
Status: Image is up to date for mcr.microsoft.com/businesscentral:10.0.17763.1999
mcr.microsoft.com/businesscentral:10.0.17763.1999
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.1999 (ltsc2019)
Host OS Version: 10.0.17763.1999 (ltsc2019)
Using process isolation
Using license file C:\Users\Documents\mylicense.flf
Files in c:\bcartifacts.cache\eyjef0kl.ryl\my:
Copying Platform Artifacts
Copying Database
Copying Licensefile
c:\bcartifacts.cache\eyjef0kl.ryl
Sending build context to Docker daemon  1.505GB
Step 1/6 : FROM mcr.microsoft.com/businesscentral:10.0.17763.1999
---> 9ab098ce53ac
Step 2/6 : ENV DatabaseServer=localhost DatabaseInstance=SQLEXPRESS DatabaseName=CRONUS IsBcSandbox=N
artifactUrl=https://bcartifacts.azureedge.net/onprem/18.1.24822.26330/w1 filesOnly=False
---> Using cache
---> 309810bce5f2
Step 3/6 : COPY my /run/
---> Using cache
---> 8e01f2bc0f9b
Step 4/6 : COPY NAVDVD /NAVDVD/
---> 4bd75559a42b
Step 5/6 : RUN \Run\start.ps1 -installOnly
---> Running in 7180cf219332
Using installer from C:\Run\150-new
Installing Business Central
Installing from DVD
Starting Local SQL Server
Starting Internet Information Server
Copying Service Tier Files
C:\NAVDVD\ServiceTier\Program Files
C:\NAVDVD\ServiceTier\System64Folder
Copying PowerShell Scripts
C:\NAVDVD\WindowsPowerShellScripts\Cloud\NAVAdministration
C:\NAVDVD\WindowsPowerShellScripts\WebSearch
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules
Determining Database Collation
Restoring CRONUS Demo Database
Setting CompatibilityLevel for CRONUS on localhost\SQLEXPRESS
Modifying Business Central Service Tier Config File for Docker
Creating Business Central Service Tier
Installing SIP crypto provider: 'C:\Windows\System32\NavSip.dll'
Copying Web Client Files
C:\NAVDVD\WebClient\Microsoft Dynamics NAV
Copying Client Files
C:\NAVDVD\LegacyDlls\program files\Microsoft Dynamics NAV
C:\NAVDVD\LegacyDlls\program files\Microsoft Dynamics NAV
C:\NAVDVD\LegacyDlls\systemFolder
Copying ModernDev Files
C:\NAVDVD
C:\NAVDVD\ModernDev\program files\Microsoft Dynamics NAV
Copying additional files
Copying ConfigurationPackages
C:\NAVDVD\ConfigurationPackages
Copying Test Assemblies
C:\NAVDVD\Test Assemblies
Copying Applications
C:\NAVDVD\Applications
Starting Business Central Service Tier
Importing CRONUS license file
Stopping Business Central Service Tier
Installation took 127 seconds
Installation complete
Removing intermediate container 7180cf219332
---> 12d4b51bafab
Step 6/6 : LABEL legal="http://go.microsoft.com/fwlink/?LinkId=837447"       created="202106212040"
nav=""       cu=""       country="w1"       version="18.1.24822.26330"       platform="18.0.2480
0.26311"
---> Running in 3fcf473689fe
Removing intermediate container 3fcf473689fe
---> d43aea406479
Successfully built d43aea406479
Successfully tagged model:onprem-18.1.24822.26330-w1
Building image took 394 seconds
Using image model:onprem-18.1.24822.26330-w1
Creating Container bc
Version: 18.1.24822.26330-w1
Style: onprem
Multitenant: No
Platform: 18.0.24800.26311
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.1999 (ltsc2019)
Host OS Version: 10.0.17763.1999 (ltsc2019)
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:\Users\mylicense.flf
Additional Parameters:
--volume "C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation:c:\sou
rces"
--volume "C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder:c:\shared"
--env customNavSettings=EnableTaskScheduler=False
Files in C:\ProgramData\BcContainerHelper\Extensions\bc\my:

AdditionalOutput.ps1
license.flf
MainLoop.ps1
SetupVariables.ps1
updatehosts.ps1
Creating container bc from image model:onprem-18.1.24822.26330-w1
f59f76c4c0f10883982aed8e2c53ff48c6e2b97329e8e8dfd7992d0e4eb67ce4
Waiting for container bc to be ready
Initializing...
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is bc
PublicDnsName is bc
Using NavUserPassword Authentication
Starting Local SQL Server
WARNING: Waiting for service 'SQL Server (SQLEXPRESS) (MSSQL$SQLEXPRESS)' to
start...
Starting Internet Information Server
Creating Self Signed Certificate
Self Signed Certificate Thumbprint 256C9E00100E7A42CDAE8122ED01245CBDE899E8
Modifying Service Tier Config File with Instance Specific Settings
Modifying Service Tier Config File with settings from environment variable
Setting EnableTaskScheduler to False
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Using license file 'c:\run\my\license.flf'
Import License
Creating http download site
Setting SA Password and enabling SA
Creating admin as SQL User and add to sysadmin
Creating SUPER user
Container IP Address: <IP_ADDRESS>
Container Hostname  : bc
Container Dns Name  : bc
Web Client          : http://bc/BC/
Dev. Server         : http://bc
Dev. ServerInstance : BC
Setting bc to <IP_ADDRESS> in host hosts file

Files:
http://bc:8080/ALLanguage.vsix
Container Total Physical Memory is 32.0Gb
Container Free Physical Memory is 20.6Gb
Initialization took 186 seconds
Ready for connections!
Reading CustomSettings.config from bc
Creating Desktop Shortcuts for bc
Container bc successfully created
Use:
Get-BcContainerEventLog -containerName bc to retrieve a snapshot of the event log from the container
Get-BcContainerDebugInfo -containerName bc to get debug information about the container
Enter-BcContainer -containerName bc to open a PowerShell prompt inside the container
Remove-BcContainer -containerName bc to remove the container again
docker logs bc to retrieve information about URL's again
Creating Tenant VE_<IP_ADDRESS>10615.app on bc
The Container is not setup for multitenancy
at , : line 6
at Invoke-ScriptInBcContainer, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.12\Co
ntainerHandling\Invoke-ScriptInNavContainer.ps1: line 44
at New-BcContainerTenant, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.12\TenantH
andling\New-NavContainerTenant.ps1: line 46
at , C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.12\AppHandling\Run
-AlPipeline.ps1: line 663
at , C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.12\AppHandling\Run
-AlPipeline.ps1: line 596
at , C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.12\AppHandling\Run
-AlPipeline.ps1: line 580
at Run-AlPipeline, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.12\AppHandling\Ru
n-AlPipeline.ps1: line 547
at , : line 1

|  __ \                         ()              / |          | |      ()
| |) | _ __ ___   _____   ___ _ __   __ _  | |     ___  _ __ | | __ _ _ _ __   ___ _ __
|  _  // _ \ '  _ \ / _ \ \ / / | '_ \ / _ | | |    / _ | ' | / ` | | ' \ / _ \ '|
| | \ \  / | | | | | () \ V /| | | | | (| | | |__ () | | | | | (| | | | | |  __/ |
||  __|| || |_|_/ _/ ||| ||_, |  ____/|| ||__,||| ||___||
/ |
|/
Removing container bc
Removing bc from container hosts file
Removing bc-* from container hosts file
Removing C:\ProgramData\BcContainerHelper\Extensions\bc
Removing container took 3 seconds
The Container is not setup for multitenancy
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.12\ContainerHandling\Invoke-Scri
ptInNavContainer.ps1:45 char:13

        throw $_.Exception.Message

        ~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : OperationStopped: (The Container i...or multitenancy:String) [], Runt
imeException
FullyQualifiedErrorId : The Container is not setup for multitenancy

Path extension structure:
@.***D766F1.F69F3760]
De : Freddy Kristiansen @.>
Envoyé : lundi 21 juin 2021 10:51
À : microsoft/navcontainerhelper @.>
Cc : PERERA AGUILA Pedro @.>; Author @.>
Objet : Re: [microsoft/navcontainerhelper] Use Run-AlPipeline cmdlet to compile Test automation extension (#1974)
It looks like the error is when trying to copy the license file, but I cannot see that you specify a license file???

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fnavcontainerhelper%2Fissues%2F1974%23issuecomment-864854860&data=04|01|pedro.pereraaguila%40vinci-energies.com|3bdac871614b4d9e32b908d93491aa40|cae7d06108f340dd80c33c0b8889224a|0|0|637598622494725493|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=h8AT5HLFss1fCK5c7v5q0oWSlLR9Cj%2BoEVF6afHolW4%3D&reserved=0, or unsubscribehttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQNMUDYHBW4QKXHE6OFWYPDTT34ONANCNFSM47A4G2CQ&data=04|01|pedro.pereraaguila%40vinci-energies.com|3bdac871614b4d9e32b908d93491aa40|cae7d06108f340dd80c33c0b8889224a|0|0|637598622494735448|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=S5bfKduL8%2BSTdG%2BzpEm2ghxFwhdaryP32wM2fXT1ww8%3D&reserved=0.

Hello,
I have a doubt, the dependent extension, on which the test automation extension was created, must first be present in a container, the app file can be loaded, installed on the fly with the run-alpipeline command.
Thanks again,
De : Freddy Kristiansen @.>
Envoyé : lundi 21 juin 2021 10:51
À : microsoft/navcontainerhelper @.>
Cc : PERERA AGUILA Pedro @.>; Author @.>
Objet : Re: [microsoft/navcontainerhelper] Use Run-AlPipeline cmdlet to compile Test automation extension (#1974)
It looks like the error is when trying to copy the license file, but I cannot see that you specify a license file???

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fnavcontainerhelper%2Fissues%2F1974%23issuecomment-864854860&data=04|01|pedro.pereraaguila%40vinci-energies.com|3bdac871614b4d9e32b908d93491aa40|cae7d06108f340dd80c33c0b8889224a|0|0|637598622494725493|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=h8AT5HLFss1fCK5c7v5q0oWSlLR9Cj%2BoEVF6afHolW4%3D&reserved=0, or unsubscribehttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQNMUDYHBW4QKXHE6OFWYPDTT34ONANCNFSM47A4G2CQ&data=04|01|pedro.pereraaguila%40vinci-energies.com|3bdac871614b4d9e32b908d93491aa40|cae7d06108f340dd80c33c0b8889224a|0|0|637598622494735448|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|1000&sdata=S5bfKduL8%2BSTdG%2BzpEm2ghxFwhdaryP32wM2fXT1ww8%3D&reserved=0.

Please do not respond to this via email, please edit the response on github, it is very hard to read the responses send in by email.
There is something wrong/strange in the script.
Example - in the output, it says:
ImageName: model
This should only be possible if the command is run with -imageName "model"
And now it tries to create a tenant called VE_<IP_ADDRESS>10615.app - I cannot see why/how this should happen - that name is not normal.
Please check that all parameters are using quotes (like in the licensefile in your last example) around the values you are sending.
Beside that,
-appfolders should be the relative path of your apps (relative to basefolder)
-testfolders should be the relative path of your test apps (relative to basefolder)

Thanks Freddy,
I put the quotes for all strings, and relaunch the cmdlet,  not the previous error, but no compilation.
 Toolkit       False
CopySymbolsFromContainer    True
enableCodeCop               False
enableAppSourceCop          False
enableUICop                 False
enablePerTenantExtensionCop False
escapeFromCops              False
useDefaultAppSourceRuleSet  False
rulesetFile
azureDevOps                 False
License file                Specified
CodeSignCertPfxFile         Not specified
TestResultsFile             C:\GitLab-Runner\builds\kMddQHnW\0\vesi\erp\naviquartz\bc-cor
e-model-test-automation\TestResults.xml
TestResultsFormat           JUnit
AdditionalCountries
PackagesFolder
OutputFolder
BuildArtifactFolder
CreateRuntimePackages       True
AppBuild                    0
AppRevision                 0
Install Apps

C:\GitLab-Runner\builds\kMddQHnW\0\vesi\erp\naviquartz\VESI_Core_Model_VE_<IP_ADDRESS>106
15.app
Install Test Apps

None
Previous Apps

None
Application folders

C:\GitLab-Runner\builds\kMddQHnW\0\vesi\erp\naviquartz\bc-core-model-test-automation
Test application folders

C:\GitLab-Runner\builds\kMddQHnW\0\vesi\erp\naviquartz\bc-core-model-test-automation\te
st

/ |              | | ()                             | |      ()
| |     _ __ ___  __ | | _ _ __   __ _    ___ ___  _ __ | | __ _ _ _ __   ___ _ __
| |    | '__/ _ /  | __| | '_ \ / _ |  / _/ _ | ' | / ` | | ' \ / _ \ '|
| || | |  __/ (| | || | | | | (| | | (__ () | | | | |_ (| | | | | |  __/ |
_||  ___|_,|_||| ||_, |  __/|| ||__,||| ||___||
/ |
|_/

BcContainerHelper is version 2.0.12
BcContainerHelper is running as administrator
Host is Microsoft Windows Server 2019 Standard - ltsc2019
Docker Client Version is 20.10.4
Docker Server Version is 20.10.4
Fetching all docker images
Fetching all docker volumes
ArtifactUrl and ImageName specified
Image my:onprem-18.1.24822.26330-w1 already exists
Using image my:onprem-18.1.24822.26330-w1
Creating Container bc
Version: 18.1.24822.26330-w1
Style: onprem
Multitenant: No
Platform: 18.0.24800.26311
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.1999 (ltsc2019)
Host OS Version: 10.0.17763.1999 (ltsc2019)
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:\Users\pedro.pereraaguila\Documents\VINCI ENERGIES_7435776-BC18 2021
0510.flf
Additional Parameters:
--volume "C:\GitLab-Runner\builds\kMddQHnW\0\vesi\erp\naviquartz\bc-core-model-test-autom
ation:c:\sources"
--volume "C:\GitLab-Runner\builds\kMddQHnW\0\vesi\erp\naviquartz:c:\shared"
--env customNavSettings=EnableTaskScheduler=False
Files in C:\ProgramData\BcContainerHelper\Extensions\bc\my:

AdditionalOutput.ps1
license.flf
MainLoop.ps1
SetupVariables.ps1
updatehosts.ps1
Creating container bc from image my:onprem-18.1.24822.26330-w1
a1dc694e0ab60541866b8e985a77717352ce1da802c971ec3ffa0966db412052
Waiting for container bc to be ready
Initializing...
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is bc
PublicDnsName is bc
Using NavUserPassword Authentication
Starting Local SQL Server
Starting Internet Information Server
Creating Self Signed Certificate
Self Signed Certificate Thumbprint 7AFC59AB4825A2867FA99492F6CF0820259B5C01
Modifying Service Tier Config File with Instance Specific Settings
Modifying Service Tier Config File with settings from environment variable
Setting EnableTaskScheduler to False
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Using license file 'c:\run\my\license.flf'
Import License
Creating http download site
Setting SA Password and enabling SA
Creating admin as SQL User and add to sysadmin
Creating SUPER user
Container IP Address: <IP_ADDRESS>
Container Hostname  : bc
Container Dns Name  : bc
Web Client          : http://bc/BC/
Dev. Server         : http://bc
Dev. ServerInstance : BC
Setting bc to <IP_ADDRESS> in host hosts file

Files:
http://bc:8080/ALLanguage.vsix
Container Total Physical Memory is 32.0Gb
Container Free Physical Memory is 18.0Gb
Initialization took 185 seconds
Ready for connections!
 Verify that the
property exists.
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.12\AppHandling\Sort-
AppFoldersByDependencies.ps1:103 char:13

$apps | Where-Object { $_.Name -eq "Application" } | ForEach-Obje ...

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Where-Object], PropertyNotFoundExcepti
on
FullyQualifiedErrorId : PropertyNotFoundStrict,Microsoft.PowerShell.Commands.Wher
eObjectCommand_

-appfolders should be the relative path of your apps (relative to basefolder)
-testfolders should be the relative path of your test apps (relative to basefolder)
In your case, appfolders is the same as basefolder. I assume this is wrong (there is no app.json in basefolder path)

Unless I'm mistaken, yes, there is an app.json file in the base folder:

the same for the path dependency, all the path are relative to the basefolder:
PS C:\Users> Run-AlPipeline -appFolders "C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation"
-artifact "https://bcartifacts.azureedge.net/onprem/18.1.24822.26330/w1" -baseFolder "C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation"
-containerName "bc" -installApps "C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation\myextension.app"
-installTestFramework -installTestLibraries
-installTestRunner -licenseFile "C:\Users\pedro.pereraaguila\Documents\mylicense.flf"
-outputFolder "C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation" -pipelineName "test"
-reUseContainer -sharedFolder "C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\"
-useDevEndpoint -createRuntimePackages
-CopySymbolsFromContainer

|  __ \                             | |
| |) |_ _ _ __ __ _ _ __ ___   | | ___ _ __ ___
|  /  | '__/ _ | ' ` _ \ / _ \ / _ \ '/ __|
| |  | (| | | | (| | | | | | |  _/ |  _/ |  _
||   _,||  _,|| || ||_|____||  |___/
Pipeline name               test
Container name              bc
Image name                  my
ArtifactUrl                 https://bcartifacts.azureedge.net/onprem/18.1.24822.26330/w1
SasToken                    Not Specified
BcAuthContext               Not Specified
Environment
ReUseContainer              True
KeepContainer               False
Credential                  admin/Kece1352
MemoryLimit                 8G
Enable Task Scheduler       False
Assign Premium Plan         False
Install Test Runner         True
Install Test Framework      True
Install Test Libraries      True
Install Perf. Toolkit       False
CopySymbolsFromContainer    True
enableCodeCop               False
enableAppSourceCop          False
enableUICop                 False
enablePerTenantExtensionCop False
escapeFromCops              False
useDefaultAppSourceRuleSet  False
rulesetFile
azureDevOps                 False
License file                Specified
CodeSignCertPfxFile         Not specified
TestResultsFile             C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation\TestResult
s.xml
TestResultsFormat           JUnit
AdditionalCountries
PackagesFolder
OutputFolder
BuildArtifactFolder
CreateRuntimePackages       True
AppBuild                    0
AppRevision                 0
Install Apps

C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation\myextension.app
Install Test Apps

None
Previous Apps

None
Application folders

C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation
Test application folders

C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation\test

/ |              | | ()                             | |      ()
| |     _ __ ___  __ | | _ _ __   __ _    ___ ___  _ __ | | __ _ _ _ __   ___ _ __
| |    | '__/ _ /  | __| | '_ \ / _ |  / _/ _ | ' | / ` | | ' \ / _ \ '|
| || | |  __/ (| | || | | | | (| | | (__ () | | | | |_ (| | | | | |  __/ |
_||  ___|_,|_||| ||_, |  __/|| ||__,||| ||___||
/ |
|_/

BcContainerHelper is version 2.0.12
BcContainerHelper is running as administrator
Host is Microsoft Windows Server 2019 Standard - ltsc2019
Docker Client Version is 20.10.4
Docker Server Version is 20.10.4
Fetching all docker images
Fetching all docker volumes
ArtifactUrl and ImageName specified
Image my:onprem-18.1.24822.26330-w1 already exists
Using image my:onprem-18.1.24822.26330-w1
Creating Container bc
Version: 18.1.24822.26330-w1
Style: onprem
Multitenant: No
Platform: 18.0.24800.26311
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.1999 (ltsc2019)
Host OS Version: 10.0.17763.1999 (ltsc2019)
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:\Users\pedro.pereraaguila\Documents\VINCI ENERGIES_7435776-BC18 20210510.flf
Additional Parameters:
--volume "C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation:c:\sources"
--volume "C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder:c:\shared"
--env customNavSettings=EnableTaskScheduler=False
Files in C:\ProgramData\BcContainerHelper\Extensions\bc\my:

AdditionalOutput.ps1
license.flf
MainLoop.ps1
SetupVariables.ps1
updatehosts.ps1
Creating container bc from image my:onprem-18.1.24822.26330-w1
8a0fe2ea454a966fa01fc9ea1551ba7285b9e0f3354c91bbeb86fba36c3ed8ec
Waiting for container bc to be ready
Initializing...
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is bc
PublicDnsName is bc
Using NavUserPassword Authentication
Starting Local SQL Server
Starting Internet Information Server
Creating Self Signed Certificate
Self Signed Certificate Thumbprint 48AA9C63266F875B380BE5ADF89541A190BFADB5
Modifying Service Tier Config File with Instance Specific Settings
Modifying Service Tier Config File with settings from environment variable
Setting EnableTaskScheduler to False
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Using license file 'c:\run\my\license.flf'
Import License
Creating http download site
Setting SA Password and enabling SA
Creating admin as SQL User and add to sysadmin
Creating SUPER user
Container IP Address: <IP_ADDRESS>
Container Hostname  : bc
Container Dns Name  : bc
Web Client          : http://bc/BC/
Dev. Server         : http://bc
Dev. ServerInstance : BC
Setting bc to <IP_ADDRESS> in host hosts file

Files:
http://bc:8080/ALLanguage.vsix
Container Total Physical Memory is 32.0Gb
Container Free Physical Memory is 16.3Gb
Initialization took 138 seconds
Ready for connections!
 Verify that the property exists.
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.12\AppHandling\Sort-AppFoldersByDependencies.ps1:103
char:13

$apps | Where-Object { $_.Name -eq "Application" } | ForEach-Obje ...

        ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

CategoryInfo          : NotSpecified: (:) [Where-Object], PropertyNotFoundException
FullyQualifiedErrorId : PropertyNotFoundStrict,Microsoft.PowerShell.Commands.WhereObjectCommand

Sorry about the delay.
The appfolder is not relative to the basefolder - it is the same as the basefolder.
-appFolders "C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation" `
-baseFolder "C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder\bc-core-model-test-automation" `

It should be
-baseFolder "C:\GitLab-Runner\builds\kMddQHnW\0\folder\erp\folder" `
-appFolders "bc-core-model-test-automation" `

The app you have in installApps - is that a 3rd party dependency?
Feel free to .zip your folder together and share a secure url to me by email and I will try to run the pipeline to compile that and give you the script needed. Email is freddyk at microsoft dot com.

Also Run-AlPipeline is supposed to build both the app and the test app and also run the tests.
Appfolders should be apps,
TestFolders should be test apps
then everything should work.
Please send me enough info to repro your problem if I am to reopen this issue.


change source url

change 'https://raw.githubusercontent.com/brather1ng/RePoE/master/data/gem_tooltips.min.json'
to 'https://raw.githubusercontent.com/brather1ng/RePoE/master/RePoE/data/gem_tooltips.min.json' in data.js
Currently doesn't work unless you make that change

Fixed.
Thanks for pointing the bug out, and also hunting down the new URL.
Make sure to refresh with a clear cache, to avoid github storing the old js script.


Delete history without printing whole history by history -r

Is it possible to config hstr to not print out whole history by history -r after deleting history?
I found it quite annoying because it shows little information (it is a super long list) and it flushed out all my scrollback which may be useful in some situtation

Did you find a way to do this?
@dvorka, after deleting an entry and doing "ctrl+c", history -r runs, printing the whole history. Any way to avoid that?

Perhaps I can make it configurable. The purpose of history -r is to get history items from shell sessions in other terminals to the current one. I agree that it might be subjective and some users don't need it. WDYT?

@dvorka how about history -r > /dev/null? seems can keep the functionality and do not flush the screen

I also suspected another weird behavior that is related to history deletion. When an item from history is deleted (and deletion is confirmed with 'y') then selecting another item from history appends string history -r to it. So for example when (after deletion) I select by TAB line git log it actually constructs on command line string git loghistory -r.

Note, that it behave same for 2.0.0 (tested on Ubuntu) and 2.2.0 (tested in NixOS). And by the way: this is most useful utility I've found ever. Magic :-)

@pkral78 Oh, your are right - it's reproducible! I created https://github.com/dvorka/hstr/issues/386. Thank you for the report!

And by the way: this is most useful utility I've found ever. Magic :-)
My pleasure!

@dvorka #386 is actually not fixed, see https://github.com/dvorka/hstr/issues/386#issuecomment-1103557204

Bump. I made #457 to fix this.


[sysbox-fs, ipc, mgr, runc] Update submodules

Signed-off-by: Rodny Molina<EMAIL_ADDRESS>

Skipping CI as it just ran a few mins ago for equivalent changes.


TypeError: register() takes 3 positional arguments but 4 were given

When running the main I'm facing this TypeError.The source of the error is while registering the sockets in blueprint
app.register_blueprint(html, url_prefix=r'/')
sockets.register_blueprint(ws, url_prefix=r'/')

app.register_blueprint seems to be working fine.

same issue here

got past the issue with this change, from:
blueprint.register(self, options, first_registration)
to:
blueprint.register(self, options)
not sure if that will break anytime though, as my application is completely new and I've never used this package before

you can run:
pip install --upgrade 'flask==1.1.2'


[Bug] Sink Connector not respecting client.conf brokerServiceUrl

Search before asking

[X] I searched in the issues and found nothing similar.

Version
OS: Win10 (Dockerhost), RHEL 8.8 (Container)
Pulsar Container Version: "Latest" (as of right now, this points to 3.1.0, running in the container)
Minimal reproduce step
https://github.com/Mortom123/kotlin-io-example/  (thanks to @shibd)
Follow the README.md:
    docker-compose -f broker-compose.yml up -d
    mvn clean package
    docker-compose -f sink-compose.yml build
    docker-compose -f sink-compose.yml up

What did you expect to see?
I expect to see no error logs and to succesfully boot the sink instance in a remote localrun setup. (Separate containers for broker and function worker)
What did you see instead?
kotlin-io-example-sink-1  | 2023-08-29T13:15:20,180+0000 [pulsar-client-io-1-4] WARN  org.apache.pulsar.client.impl.ConnectionPool - Failed to open connection to localhost/<unresolved>:6650 : io.netty.channel.AbstractChannel$AnnotatedConnectException: finishConnect(..) failed: Connection refused: localhost/
<IP_ADDRESS>:6650

The client.conf sets:
webServiceUrl=http://broker:8080/
brokerServiceUrl=pulsar://broker:6650/

which are resolved through the docker-compose network, however the localrun command still tries to setup a connection to localhost.
Anything else?
The whole pulsar setup is containerized and ran using docker.
It is important to use localrun because the sink itself will run in container and requires some native C-Libraries to bind against (not shown here), and I don't want to clutter the pulsar broker container with that.
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

@shibd @RobertIndie  would u guys mind to take a look at this?

@Mortom123 You should use ./pulsar-admin sinks localrun --broker-service-url xxx.

@shibd thanks for the advice.
I tried:
root@fe41c5f74637:/pulsar/bin# ./pulsar-admin sinks localrun --broker-service-url pulsar://broker:6650 --tenant public --namespace default --sink-type /pulsar/bin/*.nar --name sinkTest --sink-config-file /pulsar/bin/sink-config.yml --inputs test
2023-09-05T13:25:53,387+0000 [main] INFO  org.apache.pulsar.functions.utils.io.ConnectorUtils - Searching for connectors in /pulsar/connectors
2023-09-05T13:25:53,396+0000 [main] WARN  org.apache.pulsar.functions.utils.io.ConnectorUtils - Connectors archive directory not found
2023-09-05T13:25:53,410+0000 [main] INFO  org.apache.pulsar.common.nar.NarUnpacker - Created directory /tmp/pulsar_localrunner_nars_342497436883571291/pulsar-nar/kotlin-io-example-0.0.1.nar-unpacked
2023-09-05T13:25:55,241+0000 [main] INFO  org.apache.pulsar.common.nar.NarUnpacker - Extracting /pulsar/bin/kotlin-io-example-0.0.1.nar to /tmp/pulsar_localrunner_nars_342497436883571291/pulsar-nar/kotlin-io-example-0.0.1.nar-unpacked/F1JRTek3XV6vbSMOdIsP6w
2023-09-05T13:25:55,699+0000 [main] INFO  org.apache.pulsar.functions.runtime.RuntimeSpawner - public/default/sinkTest-0 RuntimeSpawner starting function
2023-09-05T13:25:55,705+0000 [main] INFO  org.apache.pulsar.common.nar.FileUtils - Jar file /pulsar/bin/kotlin-io-example-0.0.1.nar contains META-INF/bundled-dependencies, it may be a NAR file
2023-09-05T13:25:55,706+0000 [main] INFO  org.apache.pulsar.functions.runtime.thread.ThreadRuntime - Trying Loading file as NAR file: /pulsar/bin/kotlin-io-example-0.0.1.nar
2023-09-05T13:25:57,339+0000 [main] INFO  org.apache.pulsar.functions.runtime.thread.ThreadRuntime - Initialize function class loader for function sinkTest at function cache manager, functionClassLoader: org.apache.pulsar.common.nar.NarClassLoader[/tmp/pulsar_localrunner_nars_342497436883571291/pulsar-nar/k
otlin-io-example-0.0.1.nar-unpacked/F1JRTek3XV6vbSMOdIsP6w]
2023-09-05T13:25:57,344+0000 [main] INFO  org.apache.pulsar.functions.runtime.thread.ThreadRuntime - ThreadContainer starting function with instanceId 0 functionId f25b56f8-701f-4c18-abd5-63b5a0ab4e3f namespace default
HELLO
2023-09-05T13:25:57,592+0000 [pulsar-client-io-1-3] INFO  org.apache.pulsar.client.impl.ConnectionPool - [[id: 0xf684190d, L:/<IP_ADDRESS>:36932 - R:broker/<IP_ADDRESS>:6650]] Connected to server
2023-09-05T13:25:57,774+0000 [pulsar-client-io-1-7] WARN  org.apache.pulsar.client.impl.ConnectionPool - Failed to open connection to <IP_ADDRESS>/<unresolved>:6650 : io.netty.channel.AbstractChannel$AnnotatedConnectException: finishConnect(..) failed: Connection refused: /<IP_ADDRESS>:6650

It is able to connect to the broker:
2023-09-05T13:25:57,592+0000 [pulsar-client-io-1-3] INFO  org.apache.pulsar.client.impl.ConnectionPool - [[id: 0xf684190d, L:/<IP_ADDRESS>:36932 - R:broker/<IP_ADDRESS>:6650]] Connected to server
but still fails when trying to connect to localhost? Any idea what I am missing.
How would I handle the described scenario with multiple brokers? Can --broker-service-url occur multiple times or take a list?

oh, In your case. Even if you configure the correct broker address in time, it will not work.
Because the broker lookup returns a result of <IP_ADDRESS>.
So you get an error:
2023-09-05T13:25:57,774+0000 [pulsar-client-io-1-7] WARN  org.apache.pulsar.client.impl.ConnectionPool - Failed to open connection to <IP_ADDRESS>/<unresolved>:6650 : io.netty.channel.AbstractChannel$AnnotatedConnectException: finishConnect(..) failed: Connection refused: /<IP_ADDRESS>:6650

Maybe you also need to set advertisedAddress on the broker side.
https://github.com/apache/pulsar/blob/10ffbeb4dfd6328238088ec4c5909dc14391d99c/conf/standalone.conf#L46-L48

Hi, please remove advertisedListeners in your broker configuration: https://github.com/Mortom123/kotlin-io-example/blob/ca73141b8cb257a782e1443cce12010d9fe35f17/broker-compose.yml#L82
From the doc: https://github.com/apache/pulsar/blob/0d25e9ac3eb96221d0527162b6731174bbf39c0d/pulsar-broker-common/src/main/java/org/apache/pulsar/broker/ServiceConfiguration.java#L224-L230
We should not use advertisedListeners with advertisedAddress. The Default value of advertisedListeners is absent, which means using advertisedAddress and brokerServicePort.
And when you run the connector, specify the broker URL using --broker-service-url just like @shibd mentioned.
This works for me now:
sink_1  | 2023-09-11T08:07:31,244+0000 [pulsar-client-io-1-3] INFO  org.apache.pulsar.client.impl.ConsumerImpl - [test][public/default/sinkTest] Subscribing to topic on cnx [id: 0x94901f74, L:/<IP_ADDRESS>:46098 - R:broker/<IP_ADDRESS>:6650], consumerId 0
sink_1  | 2023-09-11T08:07:31,330+0000 [pulsar-client-io-1-3] INFO  org.apache.pulsar.client.impl.ConsumerImpl - [test][public/default/sinkTest] Subscribed to topic on broker/<IP_ADDRESS>:6650 -- consumer: 0

@RobertIndie - thanks a lot! This proposed solution works like a charm.

I tried some more and have a question you might be able to answer @RobertIndie
Right now my sink depends on being in the same network as the broker, hence the Dockerfile declares an external network.
I additionally set the sinks broker-service-url to pulsar://broker:6650.
If I understand the notion of advertisedListeners correctly the configuration:
- internalListenerName=internal
- advertisedListeners=internal:pulsar://broker:6650,localhost:pulsar://localhost:6650,docker:pulsar://host.docker.internal:6650

should enable, depending on listener_name, to connect either via name, via localhost or via host.docker.internal to the broker.
How would I create a sink that uses a listener that is not the internal listener? E.g. by letting it use the docker listener_name.
I cannot find anything referencing a listener_name on https://pulsar.apache.org/reference/#/3.0.x/pulsar-admin/sinks.

How would I create a sink that uses a listener that is not the internal listener? E.g. by letting it use the docker listener_name.

We don't have that setting and seems like we couldn't do that. I think it's worth raising a feature request.


Fix conda-list warning

Also: add myself as maintainer

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@CJ-Wright I can confirm that this change makes the warning go away.

LGTM, will merge once appveyor passes


Added comment

@vedpatil611 this is a very interesting pr, I read through all the changes you have committed and believe it would significantly improve the application.


do-not-merge: Test kata-deploy helm changes

Hi @fidencio !
Your change is correct here so it's indeed passing the tests (unless the image you built has wrong code, which I doubt).
I ran the operator tests 4x on same environment using quay.io/fidencio/kata-deploy:debug-uninstall-operator, they passed 100% of time. It's working beautifully \o/

I don't want to be overly optimistic here but previously I never got over 50 deploy/cleanup iterations but with your image it's on the iteration 51 and still running...

Actually it's 193 and still going so @fidencio either it's just different timeouts or the kata-containers change indeed improved the stability and I'm looking forward for it to be merged.


Update rustler version in Cargo.toml

Sorry, I wanted to push this PR to my fork as a newer version of rustler is needed in order to compile on Erlang/OTP 24


Avoid double-wal

Is your feature request related to a problem? Please describe.
In the RocksDB example, as well as numerous places in the documentation and discussions, it appears that apply_to_state_machine() is meant to be atomic and durable. Unfortunately, for some store implementations, persisting the whole data structure to disk could be quite costly. An alternative approach is shown in the RocksDB example, where rocks writes it's wal (but not the actual data) to disk, and this is better, but doesn't this result in a double-wal? Raft has it's own log (i.e. append_to_log()), then the application has to have it's own wal in order to complete apply_to_state_machine() quickly and durably.
Describe the solution you'd like
Instead of nested wals, would it be possible to allow the application to:

not persist anything during apply_to_state_machine() and keep changes in memory
the app can periodically flush to disk (data + last_applied_index) based on it's own cron job
if the app crashes, it replays the raft logs since last_applied_index (on it's own - without being told by openraft)
now that the state machine is rebuilt, the app re-joins the cluster
Openraft is unaware that apply_to_state_machine() doesn't immediately persist, but this is fine because the app state machine is still restored prior to resuming anything raft related.

Forgive me, I may be misunderstanding how this is supposed to work, but it appears #437 is effectively asking the same thing? I saw another issue (#208) out there about pipelining requests to apply_to_state_machine() to help alleviate this issue, but perhaps the solution above is either better or at least complimentary?
Describe alternatives you've considered

Creating an app wal in addition to flushing log entries to disk (WAL for both append_to_log and apply_to_state_machine)
Flushing full state to disk in apply_to_state_machine() and having a really slow application.

You are right. An application only needs one WAL. Raft is just a distributed WAL.

not persist anything during apply_to_state_machine() and keep changes in memory
the app can periodically flush to disk (data + last_applied_index) based on it's own cron job

It's OK not to persist the state, except you may want to persist the last_applied_index.
If you do not persist any state at all, when a node restarts, just rebuild state machine from the last snapshot, and wait for openraft to inform RaftStorage to re-commit and re-apply other logs.
Or last_applied_index should be persisted before apply_to_state_machine() returns. And when a node starts, rebuild state machine from last snapshot, then re-apply logs up to last_applied_index.

Forgive me, I may be misunderstanding how this is supposed to work, but it appears #437 is effectively asking the same thing?

Yes. it's meant to get rid of the expensive persisting state operation.
#208 is another thing about state machine. It's about batch-applying logs, and won't conflict with what you want to do.

Thank you @drmingdrmer for taking the time to provide this detailed description.
I did have one remaining question regarding:

just rebuild state machine from the last snapshot, and wait for openraft to inform RaftStorage to re-commit and re-apply other logs.

Assuming we haven't flushed either the state or last_applied_index the  during the last apply_to_state_machine(), when the node crashes and restarts, do we need to do any special initialization logic? Or does openraft call last_applied_state() and as long as we return the correct results, openraft will do all the necessary initialization (re-applying logs to the state machine) for us?

n/m, I think you answered the above question here: https://github.com/datafuselabs/openraft/discussions/616#discussioncomment-4444773

Whether to apply these log has to be left to openraft to decide

ty!! :smile:


minimize the amount of universe annotations in category_theory

TO CONTRIBUTORS:
Make sure you have:

[x] reviewed and applied the coding style: coding, naming
[x] for tactics:

[x] added or adapted documentation in tactics.md
[x] write an example of use of the new feature in tactics.lean

[x] make sure definitions and lemmas are put in the right files
[x] make sure definitions and lemmas are not redundant

For reviewers: code review check list

@semorrison and @rwbarton : Does this look like a useful improvement to you guys?

If this isn't fragile, then I think it's a good cleanup. But it feels pretty fragile to me, in the sense that I'm not sure if this use of the order of universe declarations is an intentional feature.

I don't believe this is any more fragile than it was before and I'm confident it's less fragile than it was. Ideally, you'd never have to specify which universe you're talking about, it would always be inferred from the arguments of your functions. In the case of some definitions like category, you have to specify some of the universes. The order of the universe declaration tells us in what order they have to be specified. If you're only using category (as opposed to writing a similar definition), the order of your universe declaration doesn't matter.
If you're writing a definition where the universes must be specified when you use it, you can declare the universes in any order you like but you would improve the user experience dramatically if you choose it in a way to minimize the number of universes that must be specified.

Note: I changed the order in all the files not so much by necessity so much as out laziness. By doing it this way, I didn't have to hunt down which definitions require the user to specify universes. They all got adapted automatically.

This looks like a good idea, I think. I didn't realize that specifying only some of the universe parameters works like this.

Note: I changed the order in all the files not so much by necessity so much as out laziness. By doing it this way, I didn't have to hunt down which definitions require the user to specify universes. They all got adapted automatically.

It makes sense to do this consistently, as it's not always obvious which definitions will need their universe parameters specified.

Thanks, this is a very helpful change!
On Fri, Dec 28, 2018, 9:07 AM Reid Barton<EMAIL_ADDRESS>wrote:

This looks like a good idea, I think. I didn't realize that specifying
only some of the universe parameters works like this.
Note: I changed the order in all the files not so much by necessity so
much as out laziness. By doing it this way, I didn't have to hunt down
which definitions require the user to specify universes. They all got
adapted automatically.
It makes sense to do this consistently, as it's not always obvious which
definitions will need their universe parameters specified.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/leanprover/mathlib/pull/552#issuecomment-450239518,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAdLBBaEH9ANnNHqBuH-VzA8eLW0lDbgks5u9UScgaJpZM4Zg-LF
.


Please, update your android example

Hi, I test your ncnn android example with my custom model with legacy weight and it's working fine. Now I train model with new weight and convert to ncnn for deploy on android . The result is app always force quit from app.

Hey, @FateScript I upload my android project for you, I think it may help you. (link)[https://github.com/RTae/YOLOX-android]

Hey, @FateScript I upload my android project for you, I think it may help you. link

Many thx~ I will update later.

@FateScript I found the solution, your code is working fine, but you need to update your document about how to convert from onnx to ncnn. I get this solution from this issue


trial command

make the trial command not be role based and make it store users for that trial per league they specify, so it pings the users themselves so we can have one lab channel instead of three
have to make a typereader for the trialname enum or think of a different approach of handling the trial name so it's not text based

This has been changed, as with the announcement of PS4, we have changed the discord design. No longer going to happen.


Link to 'Remote debugging and SSH in App Service on Linux' goes to blog home

The link 'https://aka.ms/linux-debug' ('Remote debugging and SSH in App Service on Linux') does not lead to a page explaining the topic. Seems disappeared??

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

ID: b421fa01-1f39-28a5-78c2-63dd5700c8c5
Version Independent ID: 10ed9c91-16e0-d371-fcfc-d6d5a7f984b2
Content: App Service on Linux FAQ - Azure
Content Source: articles/app-service/containers/app-service-linux-faq.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

@jhgbrt, Thanks for the question! We are taking a look into this and will get back to you soon.

@jhgbrt, We have made the requested changes and it should be public within the next 24 hours. Thanks again for reporting this, it's much appreciated.


Fix IODMA interface width selection

In some cases, this bug caused an erroneous intfWidth setting of the input IODMA, resulting in a significant performance regression for memory-bound accelerators.

Thanks Felix!


[QUERY] ServiceBusProcessorClient lock expiry

Query/Question
Hello! I'm wanting to create a lock expiry monitor for the ServiceBusProcessorClient in PEEK_LOCK mode. I've done a bit of research and was hoping you could point me in the right direction:

On failed lock refresh (due to network failure for example, within maxAutoLockRenewDuration) and when maxAutoLockRenewDuration has been exhausted, processError is not invoked. This makes sense as processMessage is still running / possibly can't be interrupted. Is there another way to hook into these events?
Within our processMessage handler, we could check message.message.lockedUntil, which is updated on lock refresh. This only works for non-session processors however, when using sessions messages are locked until +10000-01-01T00:00:00Z (which I'm assuming is because the lock is held for the entire session, even past sessionIdleTimeout). Is there a way of retrieving the actual lockedUntil value?

Why is this not a Bug or a feature Request?
I'm curious about functionality which may already exist.
Setup (please complete the following information if applicable):

Library/Libraries: com.azure:azure-messaging-servicebus:7.17.2

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X] Query Added
[X] Setup information Added

Hello @, you are correct that for session scenarios, each message does not include the lock timeout, as the lock is scoped to the session.
I'm not exactly sure what the monitoring you mentioned targets, but if it helps, the library emits telemetry observability purposes - for sessions, there is session lock renew spans and spans for message operations like receiving, completing, or abandoning messages.

Hello @rileymichael, you are correct that for session scenarios, each message does not include the lock timeout, as the lock is scoped to the session.
I'm not exactly sure what the monitoring you mentioned targets, but if it helps, the library emits telemetry observability purposes - for sessions, there is session lock renew spans and spans for message operations like receiving, completing, or abandoning messages.

Thanks for the reply @anuchandy. To be more clear, we want to run some logic in the event a message lock expires before handling has completed (the monitor was just a simple usecase example). We've looked at comparing the current time with the lock expiration time, but as you've confirmed that's not always held by the message. Is it possible to get the session expiration?


fix: pass globals through signature resolution

fixes #74
thanks @lucyleeow

fix coming up in v0.1.9


Decode ChannelType causes DataIntegrityError

Hi @dr-orlovsky,
I tried to decode a ChannelType using lightning_deserialize from message open channel by clightning, but it occurred an error:
Err(DataIntegrityError("can't unmarshall bolt LNP2P message. Details: IO error"))

According to the BOLT-2 spec, this occurs because some situations are not necessary channel_type information in the message.
You can test this by adding the bellow test in bolt2.rs, in lnp-core repo:
#[cfg(test)]
mod test {
    use lightning_encoding::LightningDecode;

    use crate::bolt::Messages;

    #[test]
    fn real_clightning_open_channel() {
        // Real open_channel message sent by clightning
        let msg_recv = [
            0, 32, 6, 34, 110, 70, 17, 26, 11, 89, 202, 175, 18, 96, 67, 235,
            91, 191, 40, 195, 79, 58, 94, 51, 42, 31, 199, 178, 183, 60, 241,
            136, 145, 15, 22, 145, 1, 71, 149, 179, 87, 248, 190, 135, 14, 128,
            6, 231, 127, 76, 45, 137, 175, 179, 41, 39, 181, 150, 135, 218, 40,
            68, 235, 9, 140, 89, 0, 0, 0, 0, 0, 1, 134, 160, 0, 0, 0, 0, 0, 0,
            0, 0, 0, 0, 0, 0, 0, 0, 2, 34, 255, 255, 255, 255, 255, 255, 255,
            255, 0, 0, 0, 0, 0, 0, 3, 232, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,
            253, 0, 6, 1, 227, 2, 163, 76, 91, 209, 179, 37, 241, 217, 227,
            190, 56, 5, 236, 7, 160, 138, 88, 114, 175, 48, 21, 46, 14, 153,
            146, 158, 126, 225, 12, 111, 131, 10, 2, 225, 239, 131, 243, 222,
            240, 239, 98, 190, 230, 192, 188, 131, 126, 244, 155, 76, 222, 5,
            101, 185, 160, 228, 83, 23, 84, 249, 194, 79, 163, 177, 186, 3,
            204, 49, 206, 253, 45, 131, 197, 237, 230, 119, 210, 47, 23, 109,
            85, 213, 115, 0, 250, 124, 98, 66, 91, 111, 57, 112, 126, 135, 216,
            176, 85, 129, 3, 187, 13, 129, 200, 13, 124, 92, 181, 198, 96, 147,
            178, 18, 171, 20, 138, 192, 51, 37, 199, 166, 220, 71, 26, 233,
            232, 54, 78, 2, 245, 15, 101, 3, 156, 36, 195, 109, 14, 233, 15,
            49, 144, 81, 57, 24, 18, 235, 59, 209, 131, 185, 165, 48, 1, 223,
            28, 207, 119, 153, 246, 147, 12, 2, 72, 208, 2, 68, 86, 229, 216,
            34, 4, 234, 220, 238, 164, 21, 1, 132, 103, 87, 228, 53, 93, 84,
            217, 197, 134, 141, 149, 32, 34, 25, 136, 118, 190, 172, 126, 1, 0,
            0, 1, 2, 16, 0,
        ];
        let message = Messages::lightning_deserialize(&msg_recv);
        println!("{:?}", message);
        assert_eq!(true, message.is_ok());
    }
}

Expected Result:
Ok(OpenChannel(OpenChannel { chain_hash: Slice32(06226e46111a0b59caaf126043eb5bbf28c34f3a5e332a1fc7b2b73cf188910f), temporary_channel_id: TempChannelId(Slice32(1691014795b357f8be870e8006e77f4c2d89afb32927b59687da2844eb098c59)), funding_satoshis: 100000, push_msat: 0, dust_limit_satoshis: 546, max_htlc_value_in_flight_msat:<PHONE_NUMBER>3709551615, channel_reserve_satoshis: 1000, htlc_minimum_msat: 0, feerate_per_kw: 253, to_self_delay: 6, max_accepted_htlcs: 483, funding_pubkey: PublicKey(0a836f0ce17e9e92990e2e1530af72588aa007ec0538bee3d9f125b3d15b4ca322a82c74f042f684a14c20d465a1ae310c0fce6ade479d472354d4509500efd6), revocation_basepoint: PublicKey(bab1a34fc2f9541753e4a0b96505de4c9bf47e83bcc0e6be62eff0def383efe1dc2bfcba80ce52711b4fc49dacfed03588975784846285ad4ee75b75868f05d3), payment_point: PublicKey(8155b0d8877e70396f5b42627cfa0073d5556d172fd277e6edc5832dfdce31ccdfb17ff537910b3363e522b120b2595ca29babee106b4d9ae4e0c80440a05529), delayed_payment_basepoint: PublicKey(650ff5024e36e8e91a47dca6c72533c08a14ab12b29360c6b55c7c0dc8810dbb29ac75d2fcb3583059ee0267743df95e4b53ecea2437501990fc65713dd56ddf), htlc_basepoint: PublicKey(d048020c93f69977cf1cdf0130a5b983d13beb1218395190310fe90e6dc3249c5152ba613988416e44fdbcddc4988ce1472422e26f889026dfb87038c28949bf), first_per_commitment_point: PublicKey(7eacbe7688192220958d86c5d9545d35e45767840115a4eedcea0422d8e55644ca3e53ec75512a6b9b1fd5d0e33beaee1ea8ae65485789a25876b372c5ec9620), channel_flags: 1, shutdown_scriptpubkey: Some(PubkeyScript(Script())), channel_type: Some(Basic), unknown_tlvs: Stream({}) }))

PS: I bypassed this issue changing this code to:
impl LightningDecode for usize {
    fn lightning_decode<D: Read>(mut d: D) -> Result<Self, Error> {
        let mut buf = [0u8; 2];
        let _ = d.read_exact(&mut buf);
        Ok(u16::from_be_bytes(buf) as usize)
    }
}

And made other change in ChannelType (lnp-core repo):

I known this not correct approach, but I couldn't solve this problem.
I hope this help.
Thanks

So basically the issue is the fact that FlagsVec can be absent and that is not indicated anyhow, right?

So basically the issue is the fact that FlagsVec can be absent and that is not indicated anyhow, right?

So so, the situation occurs in others parts too.
I was thinking of modifying the ln encode/decode so that it detects when the buffer EOF and ends the operation instead of return a Error. But I'm not sure if this is correct approach.
What's your opnion?

For me it looks like the whole Lightning encode traits need a redesign. But I need a bit more time to figure out the best way forward - maybe we can avoid such dramatic changes.

For me it looks like the whole Lightning encode traits need a redesign. But I need a bit more time to figure out the best way forward - maybe we can avoid such dramatic changes.

Sure.
For now, I will open a PR with some "workarounds" and minor issues.
I have a goal to finish the LN review in end of mounth.

Ok, I figured out the source of the issue and a fix: #5
The issue was caused by a TLV with 0 bytes, which must be handled as a default FlagVec. The approach you proposed in the comments is invalid leading to failing parsing length values in all lightning-encoded data inproperly.


duplicate page / and /index

The sidebar currently links to /configuration/index but it should link to ``/configuration/`.
/configuration/index should never be used.

Yeah, I think I'll filter /index out of there at core. Good point.


Update  Docker to reuse NEXUS_HOME variable

Hi @slachiewicz we actually already have this in progress with changes @mcculls is making for the 3.1 release, I'd rather close and wait for those since we have another couple changes. You can see the changes in the nexus-3.1-layout-changes


Support for loop labels

loop1:
for (let i = 0; i < 5; i++) {
  if (i === 1) {
    continue loop1;
  }
  str = str + i;
}

super old request here, but I think this should be easy to implement
if i % 2 == 0 then goto continue end   print(i)   ::continue::


Improve build

Add NuGet.Config.
Use local package repository for all builds.
Tidy SupportLibraries.proj; and add "Clean" target
Fix XML comment.

Go for it 🚢


Update Model Builder Doc

Update Model Builder doc to be more up-to-date.
Fixes #24825

@luisquintanilla I believe this is ready for review.

Closing and reopening to trigger build


Updated styling for ArgDescriptor component in workspaces/tests #146

Title: Modified "workspaces/docs/src/components/ArgDescriptor/styles.module.css"
Description:
I've made some changes to the mentioned file to fit the ask.
Changes Made:

Added bottom margin of 10px to .arg-description--name and .arg-description--type
Added bottom margin, border bottom and padding bottom as mentioned for .description

Please review and let me know if there are any further changes or suggestions. I'm open to feedback and willing to make additional updates as needed. Thanks for considering this pull request!

Committed a few more changes:

Ignore Storybook in workspaces/calendar-widgets/.npmignore
Remove Default FavIcon from Docs

Hi @Subham5401 how awesome, thank you for taking on so many issues! 💯 Moving forward, i'd kindly request we keep each PR consolidated to 1 issue so we can avoid scope-creeping in merges. In this case, these are all relatively small changes with little chance of regressions so there are no worries. 🙂
I left a few comments, please let me know if you have any questions! Awesome work btw!

These changes are looking great @Subham5401 ! I really appreciate you taking the time contribute to the project. You've covered a-lot of ground with this one. 💯 I'm going to merge these changes into the main branch now.

I'm grateful for the opportunity. Thank you @9mbs !


Add support for qt5 and python 3.6

This implies switching to my next version of enaml. (I am currently pushing quite hard to get it merged). Tests will fails till the new packages are updated.

Delayed till I fix some issues in enaml

Reopening now that enaml is fixed and packages rebuilt for 3.6

Codecov Report

Merging #96 into master will decrease coverage by -0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #96      +/-   ##
==========================================
- Coverage   98.27%   98.25%   -0.02%
==========================================
  Files         155      155
  Lines       12317    12335      +18
==========================================
+ Hits        12104    12120      +16
- Misses        213      215       +2

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 ba2bc6f...9ba80dc. Read the comment docs.

Now that I fixed the tests does anybody wants to review this @Ecpy/owners


Feature request: declarative YAML client as an alternative to Helm CLI

As I see it, the Helm CLI is pretty fat as there are quite many commands and the workflow falls on the imperative side in terms of managing charts and releases.
I think it would be great to have a declarative Helm client with these features:

YAML manifest file(s) for managing releases, charts, values.yaml override values and anything else already implemented in Helm CLI
Addition of helm apply to the CLI or direct kubectl apply integration

The goal here is to bring Kubernetes closer to "everything as code" type of workflow.
Ideas? 🤔

So it sounds like there are two existing scenarios that would solve this problem:

https://github.com/kubernetes/helm/issues/3089 where others have written proof of concepts
You could pipe the output of helm template to kubectl apply, though that means you lose out on helm commands like helm list.

Do either of those seem to fit your use case?

Yeah, https://github.com/kubernetes/helm/issues/3089 definitely fits.
Closing this in favor of https://github.com/kubernetes/helm/issues/3089.


Page behind a session leads to error on Lighthouse report

Describe the bug
Even though the payload.html are properly created in the reports folder (no error there, the page loaded properly), all lighthouse.json files contains this:
  "fetchTime": "2023-05-12T00:53:53.678Z",
  "gatherMode": "navigation",
  "runtimeError": {
    "code": "ERRORED_DOCUMENT_REQUEST",
    "message": "Lighthouse was unable to reliably load the page you requested. Make sure you are testing the correct URL and that the server is properly responding to all requests. (Status code: 401)"
  },
  "runWarnings": [
    "Lighthouse was unable to reliably load the page you requested. Make sure you are testing the correct URL and that the server is properly responding to all requests. (Status code: 401)"
  ],

Reproduction
No response
System / Nuxt Info
System:
    OS: Linux 5.15 Debian GNU/Linux 11 (bullseye) 11 (bullseye)
    CPU: (6) x64 AMD Ryzen 7 5800X3D 8-Core Processor
    Memory: 14.22 GB / 15.62 GB
    Container: Yes
    Shell: 5.8 - /usr/bin/zsh
  Binaries:
    Node: 18.16.0
    Yarn: 1.22.19
    npm: 9.5.1

Hi @pocesar, thanks for the issue.
I think this may be related to the site you're scanning. You'll need to provide it for me to investigate further.

Hm interesting, I've pushed up a change that may fix it in 0.7.2 (sets the cookies in the hook).


Difference to code-server option?

What is the difference to the code-server option is already present in Nixos?

It didn't exist when I started this project and I did not know of it before this issue was made. From what I gather it is pretty much the same, but targeting vscodium (like how chromium differs from chrome). Never used vscodium myself, not sure if it is a full drop-in replacement, so it might be that the code-server option already present in NixOS doesn't work with normal vscode.

My impression is that code-server runs VS Code in your browser, while this repo is enabling a local VS Code installation to connect to a remote NixOS server via SSH.


running on armel error: LLVM ERROR: Symbol not found: __sync_fetch_and_add_4

From a recent build of 2020.6.30a0 on Debian
https://buildd.debian.org/status/fetch.php?pkg=python-cogent&arch=armel&ver=2020.6.30a0%2Bdfsg-1&stamp=1606735454&raw=0
   dh_auto_test -a -O--buildsystem=pybuild
I: pybuild pybuild:284: cp -r /<<PKGBUILDDIR>>/tests/data /<<PKGBUILDDIR>>/.pybuild/cpython3_3.8_cogent3/build
I: pybuild base:232: cd /<<PKGBUILDDIR>>/.pybuild/cpython3_3.8_cogent3/build; python3.8 -m unittest discover -v
LLVM ERROR: Symbol not found: __sync_fetch_and_add_4
E: pybuild pybuild:353: test: plugin distutils failed with: exit code=1: cd /<<PKGBUILDDIR>>/.pybuild/cpython3_3.8_cogent3/build; python3.8 -m unittest discover -v
dh_auto_test: error: pybuild --test -i python{version} -p "3.8 3.9" returned exit code 13

Running the tests on the mipsel architecture we get a similar error in the same circumstance
LLVM ERROR: Symbol not found: __floatdidf

https://buildd.debian.org/status/fetch.php?pkg=python-cogent&arch=mipsel&ver=2020.6.30a0%2Bdfsg-1&stamp=1606735937&raw=0

Closing this as from version 2020.12 we now use numba and any compilation issues derive from that.

Closing this as from version 2020.12 we now use numba and any compilation issues derive from that.

Hello, we still get these errors with cogent3 2020.12.21a; can this issue be re-opened?
https://buildd.debian.org/status/fetch.php?pkg=python-cogent&arch=armel&ver=2020.12.21a%2Bdfsg-3&stamp=1612781156&raw=0
https://buildd.debian.org/status/fetch.php?pkg=python-cogent&arch=mipsel&ver=2020.12.21a%2Bdfsg-3&stamp=1612782448&raw=0

Looking at the error, it's arising during the build of documentation process:
sphinx-build -b html -d _build/doctrees   . _build/html
Running Sphinx v3.4.3
making output directory... done
WARNING: html_static_path entry '_static' does not exist
[autosummary] generating autosummary for: api/__init__/cogent3.__init__.available_apps.rst, api/__init__/cogent3.__init__.available_codes.rst, api/__init__/cogent3.__init__.available_models.rst, api/__init__/cogent3.__init__.available_moltypes.rst, api/__init__/cogent3.__init__.get_code.rst, api/__init__/cogent3.__init__.get_moltype.rst, api/__init__/cogent3.__init__.load_aligned_seqs.rst, api/__init__/cogent3.__init__.load_table.rst, api/__init__/cogent3.__init__.load_tree.rst, api/__init__/cogent3.__init__.load_unaligned_seqs.rst, ..., examples/scope_model_params_on_trees.rst, examples/seq_features.rst, examples/simple.rst, examples/simulate_alignment.rst, examples/testing_multi_loci.rst, general.rst, index.rst, install.rst, licenses.rst, pycogent.rst
'generic' is not a recognized processor for this target (ignoring processor)
'generic' is not a recognized processor for this target (ignoring processor)
'generic' is not a recognized processor for this target (ignoring processor)
generic' is not a recognized processor for this target (ignoring processor)
...
LLVM ERROR: Symbol not found: __floatdidf
make[2]: *** [Makefile:40: html] Error 1
make[2]: Leaving directory '/<<PKGBUILDDIR>>/doc'

Can you turn off the documentation build? All the docs are available at cogent3.org

I should also point out that the main code for building the documentation is hosted in a separate repo.
I retain the actual documentation in this repo for ease of maintenance. The Sphinx Makefile is likewise retained in the doc directory for maintenance, but it's not the production version.
So my recommendation is to skip the documentation build in your packaging, link to cogent3.org instead.

Okay, I tried building without the docs for armel and mipsel, but that segfaulted under qemu. I'll try again on real hardware.
For s390x the build segfaults (on real hardware) while running the tests: https://buildd.debian.org/status/fetch.php?pkg=python-cogent&arch=s390x&ver=2020.12.21a%2Bdfsg-3&stamp=1612781283&raw=0
Likewise for mips64el (also on real hardware) https://buildd.debian.org/status/fetch.php?pkg=python-cogent&arch=mips64el&ver=2020.12.21a%2Bdfsg-3&stamp=1612857835&raw=0

armel and mipsel tests also fail on real hardware
armel:
I: pybuild pybuild:284: cp -r /home/crusoe/python-cogent-2020.12.21a+dfsg/tests/data /home/crusoe/python-cogent-2020.12.21a+dfsg/.pybuild/cpython3_3.9_cogent3/build
I: pybuild base:232: cd /home/crusoe/python-cogent-2020.12.21a+dfsg/.pybuild/cpython3_3.9_cogent3/build; python3.9 -m pytest -k 'not test_pickleable_member_roundtrip and not
 test_pickleable_roundtrip'
============================================================================ test session starts ============================================================================
platform linux -- Python 3.9.1+, pytest-6.0.2, py-1.10.0, pluggy-0.13.0
rootdir: /home/crusoe/python-cogent-2020.12.21a+dfsg
collecting 0 items                                                                                E
: pybuild pybuild:353: test: plugin distutils failed with: exit code=1: cd /home/crusoe/python-cogent-2020.12.21a+dfsg/.pybuild/cpython3_3.9_cogent3/build; python3.9 -m pyte
st -k 'not test_pickleable_member_roundtrip and not test_pickleable_roundtrip'
dh_auto_test: error: pybuild --test -i python{version} -p 3.9 returned exit code 13

mipsel build with NUMBA_DEVELOPER_MODE=1 and -s added to the pytest invocation
I: pybuild base:232: cd /home/crusoe/python-cogent-2020.12.21a+dfsg/.pybuild/cpython3_3.9_cogent3/build; python3.9 -m pytest -k 'not test_pickleable_member_roundtrip and not test_pickleable_roundtrip' -s
============================================================================ test session starts ============================================================================
platform linux -- Python 3.9.1+, pytest-6.0.2, py-1.10.0, pluggy-0.13.0
rootdir: /home/crusoe/python-cogent-2020.12.21a+dfsg
collecting 0 items                                                                                'generic' is not a recognized processor for this target (ignoring processor)
'generic' is not a recognized processor for this target (ignoring processor)
'generic' is not a recognized processor for this target (ignoring processor)
'generic' is not a recognized processor for this target (ignoring processor)
'generic' is not a recognized processor for this target (ignoring processor)
'generic' is not a recognized processor for this target (ignoring processor)
'generic' is not a recognized processor for this target (ignoring processor)
'generic' is not a recognized processor for this target (ignoring processor)
LLVM ERROR: Symbol not found: __floatdidf

Attached is a mipsel (real hardware) log with NUMBA_DEBUG=1
python-cogent-2020.12.21a+dfsg_mipsel.log

A couple of remarks:

both the Debian build logs show it's still building the docs.
segfault is more likely to come from one of our dependencies (like numba, numpy)
this is build off the pypi release. Can you assess whether the develop branch of the git repo gives the same issue?

Please note I'm absolutely smashed for time and am unlikely to be free for over a week, so responses will be slow!

* both the Debian build logs show it's still building the docs.

Yes, we only skip building the docs on platforms that have problems.

* segfault is more likely to come from one of our dependencies (like numba, numpy)

Okay. Assistance in narrowing this down so we can give the right group a good big report will be appreciated

* this is build off the pypi release. Can you assess whether the develop branch of the git repo gives the same issue?

Sure, I'll try a snapshot. Would you like a PR to add additional architectures to your CI?

Closing as we can't support platforms for which we don't have access.


Balena shutdown

Hi,
I've just completed my first Balena project, the Balenasound raspberry pi build. It works great, but I am not sure how to remotely shut my pi down without just turning it off at the wall. Help please

Go to balena then login then click on the blue device name then click on the arrow and then click shutdown.

@AlexProgrammerDE thanks for replying here! 👍 As mentioned above, the shutdown option is on the dashboard in the device screen under the dropdown.


Don't close Commands View on losing Focus

Closing Commands View on losing focus is annoying, particularly when installing or searching for extensions, and commands that are not instantaneous.
Please stop the commands view from closing itself when the window loses focus.

As designed, however we also see the issue that extensions should go into a more stable view and not quick open.

I see this as future solution to the extensions issue: https://github.com/Microsoft/vscode/issues/637
However, while I understand its "as designed", I really see no good reason why it is really designed that way. Is there a reason why the command window has to be closed when the window loses focus?
I see this as something that seems to be inherited from Atom. The "ext install" is just one scenario that highlights this problem. But often switching between windows, and losing the command window is extremely frustrating. Just changing this design decision will not just enable the multi-taskers to work with ease, but also provide a temporary mitigation to the "ext install" problem until the new UI is designed (which I surmise will take quite a while looking at the state of things).
Could this be reconsidered?

https://github.com/Microsoft/vscode/issues/444

This is coming for August via #444.


In the translation result, the "left" quotation mark is missing.

import deepl

target_language = 'ZH'

translator = deepl.Translator('my---key')
content = ”Transcript ‘In Trust’ Episode 11: The Return to Osage County“

result = translator.translate_text(content , target_lang="ZH")
print(result.text)

The result is：信任》第 11 集实录：重返奥沙郡
the result from OpenAI’api is fine：剧本“信任”第11集：回到奥萨奇县
here is another content
import deepl

target_language = 'ZH'

translator = deepl.Translator('my---key')
content = "In Trust: A Return to Osage County as Killers of the Flower Moon Debuts"

result = translator.translate_text(content , target_lang="ZH")
print(result.text)

the DeepL‘s return is：信任：花月杀手》首映，重返奥萨奇县 （the "left" quotation mark is missing）
while the OpenAI’s return is somehow OK：在《信任：回到奥萨奇县》中，花月杀手首次亮相（the "left" quotation mark is there，the‘’ mark can get process later）

I had a similar issue with the right single straight quotation mark, https://github.com/DeepLcom/deepl-python/issues/86


storage: skip merging when no remote storage configured

Prometheus is hard-coded to use a fanout storage between TSDB and a remote storage which by default is empty.
This change detects the empty storage and skips merging between result sets, which would make Select() sort results.
Bottom line: we skip a sort unless there really is some remote storage configured.

I am not familiar with NewMergeQuerier but shouldn't it do something special if after all filtering len(primaries) == 1 && len(secondaries) == 0? Sounds like it should return primaries[0].

Currently that optimization is done in Select().
You’re right it could move to the constructor, but I would see that as a separate change.

/prombench main

Big win: 10% CPU reduction, 30% faster instant queries.

(Arguably the benchmark queries are extreme, YMMV)


Maven error

When do mvn clean install with the source code of this plugin, it says me that PreciusStones arent find
What should i do?

ok, i found its repository. now, why massivecraft is needed? i cant find it the repository for this plugin lol

Answered via discord.


some nydus images are larger than OCI image by about 30%

Displayed by nerdtcl images
These images are converted by acceld
# nerdctl images

REPOSITORY                                   TAG                IMAGE ID        CREATED           PLATFORM       SIZE         BLOB SIZE
xxxxxxxxxxxxxxxxxx.xxxxxxxx/rrrrrr/java      latest             f841a2abd042    7 seconds ago     linux/amd64    662.7 MiB    235.9 MiB
xxxxxxxxxxxxxxxxxx.xxxxxxxx/rrrrrr/java      latest-nydus-v5    00da0fb797d0    30 seconds ago    linux/amd64    0.0 B        314.9 MiB
xxxxxxxxxxxxxxxxxx.xxxxxxxx/rrrrrr/java      latest-nydus-v6    af716a2aa975    27 seconds ago    linux/amd64    0.0 B        314.7 MiB
xxxxxxxxxxxxxxxxxx.xxxxxxxx/rrrrrr/python    3.7                44fd0dc34f00    21 minutes ago    linux/amd64    933.2 MiB    331.0 MiB
xxxxxxxxxxxxxxxxxx.xxxxxxxx/rrrrrr/python    3.7-nydus-v5       c718004ac488    15 hours ago      linux/amd64    0.0 B        438.7 MiB
xxxxxxxxxxxxxxxxxx.xxxxxxxx/rrrrrr/python    3.7-nydus-v6       045764b7b03e    21 minutes ago    linux/amd64    0.0 B        438.3 MiB

who cares? If anyone cares, they should use zstd or whatever algorithm instead of lz4.

Customers have to pay for storage space like Aliyun OSS bucket, larger images cost more fees. 30% more fee can hardly be ignored. No matter what compression algorithm is preferred or defaulted, we'd better give customers advice. If we already figure out a better and more suitable compression algorithm, turning to that one might be better.

nope, I don't think there is a silver bullet of this. Zstd actually makes much worse if running with internal network.

We should enable support of zstd in v2.2

We'd better investigate why it's so much bigger. eStargz uses gzip compression and the chunk size defaults to 4MB, the eStargz java image and OCI image are close in size.

Looks like zstd is the right way, I tested (using acceld) a java image with lz4 and zstd compression:
ociv1 with gzip: ~234MB
nydus with zstd: ~232MB
nydus with lz4 (default level): ~317MB

Looks like zstd is the right way, I tested (using acceld) a java image with lz4 and zstd compression:
ociv1 with gzip: ~234MB nydus with zstd: ~232MB nydus with lz4 (default level): ~317MB

Good to hear about it. Thanks

When it's network bandwidth limited, zstd seems like the right solution.

The default compression algorithm has been switched to zstd, could you please try again?


TF Flag, for mode is raising exception --mode="train"

python run_summarization.py --mode="train" --data_path="../misc/finished_files/chunked/train_*" --vocab_path="../misc/finished_files/vocab" --log_root="log/" --exp_name="first_test"
/home/ubuntu/anaconda3/lib/python3.6/site-packages/h5py/__init__.py:36: FutureWarning: Conversion of the second argument of issubdtype from `float` to `np.floating` is deprecated. In future, it will be treated as `np.float64 == np.dtype(float).type`.
  from ._conv import register_converters as _register_converters
/home/ubuntu/anaconda3/lib/python3.6/site-packages/matplotlib/__init__.py:1067: UserWarning: Duplicate key in file "/home/ubuntu/.config/matplotlib/matplotlibrc", line #2
  (fname, cnt))
/home/ubuntu/anaconda3/lib/python3.6/site-packages/matplotlib/__init__.py:1067: UserWarning: Duplicate key in file "/home/ubuntu/.config/matplotlib/matplotlibrc", line #3
  (fname, cnt))
Tensorflow Version  1.8.0
INFO:tensorflow:Starting seq2seq_attention in train mode...
Warning : Incorrectly formated line in vocab file : 0800 555 111 356

Warning : Incorrectly formated line in vocab file : 1800 333 000 139

Warning : Incorrectly formated line in vocab file : 2 1/2 124

Warning : Incorrectly formated line in vocab file : 3 1/2 86

max_size of vocab was specified as 50000; we now have 50000 words. Stopping reading.
Finished constructing vocabulary of 50000 total words. Last word added: farina
Traceback (most recent call last):
  File "run_summarization.py", line 305, in <module>
    tf.app.run()
  File "/home/ubuntu/anaconda3/lib/python3.6/site-packages/tensorflow/python/platform/app.py", line 126, in run
    _sys.exit(main(argv))
  File "run_summarization.py", line 301, in main
    raise ValueError("The 'mode' flag must be one of train/eval/decode")
ValueError: The 'mode' flag must be one of train/eval/decode

I tried manually passing inside the run() with argv.
I have tried passing it as command and string ie --mode=train and --mode="train"
Using
Python 3.6
Tensorlfow 1.8

Have you tried to print the type of the value you pass ?

Try tensorflow version under 1.4.0. The hps API has changed around version 1.4~1.5, and this repository's code doesn't work after that.

You need to see mode.value if you are running TF version > 1.4.

@Adityagrao
Here you go. This code is executable with Tensorflow>=1.5.0.
https://github.com/seungwonpark/pointer-generator
@abisee
Would you mind listing this repository to "About this code" section of README.md, like:

Python 3, Tensorflow>=1.5.0 version: see @seungwonpark's fork

Thanks in advance.


Please add voting system

Hi please add voting system for actual played song and after vote add to up rating to most plays

???

@24dubstep I'll definitely look into it as a feature for the future. It may start off as an API-only feature, where you can submit votes for songs that are played.

We're migrating our feature requests over to FeatureUpvote, which allows users to vote on what they would like to see the most, and helps clear our GitHub issues page up for bugs and error reports.
Your enhancement request has been migrated over to FeatureUpvote and can be found here:
https://azuracast.featureupvote.com/suggestions/38777/add-the-ability-for-listeners-to-vote-on-songs


selection rectangle position stays/renders in drag and drop & drop-ask is not provided with target

drop-ask /home/wael/.profile

https://user-images.githubusercontent.com/47404953/212342650-0e6f14ab-e121-410a-b2c9-afe4f7bc930e.mp4

I could not reproduce the error here, but I found the lines of code that could be causing it for you.
I think it should be fixed now.


fenics: use normal pytest

Description of changes
Seems to test fine with the normal pytest so this should remove another python2 bit.
NB: The noise at the top is due to a sort.

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Please also move the directory under python packages.
@ofborg python310Packages.fenics

This is a semi-automatic executed nixpkgs-review with nixpkgs-review-checks extension. It is checked by a human on a best effort basis and does not build all packages (e.g. lumo, tensorflow or pytorch).
Result of nixpkgs-review pr 170266 run on x86_64-linux 1

2 packages built:

python310Packages.fenics
python39Packages.fenics


添加JustList: {qt4.json}

JustList URL: https://raw.githubusercontent.com/wangqqqqq/tokenlist/master/qt4.json
JustList 名称: qt4
JustList 提交者的网站或主页:无

Sorry, your issue will be closed as you did not submit your information in the correct format.


Ensure polybar builds without optional dependencies

The main goal of this was to make sure polybar can still build even when everything optional is turned off. I have added a new travis build configuration to do that. This new build would have caught the build failure caused by building without xrm.
This also fixes the underlying cause of the xrm build failure (missing include)
While working on this, I have also noticed that the xcb-xkb is optional, if we make the xkeyboard module optional
Fixes #1264

Bump


'Invalid or incomplete response' when using Mistral API

I have spent lot of time to understanding the Mistral API within the CrewAI framework, specifically for 'mistral-medium' and 'mistral-small'. Although I am able to execute the code smoothly, I am encountering an issue: it returns the error 'Invalid or incomplete response. Parsing LLM output produced both a final answer and a parse-able action.'
To address this, I have attempted the following solutions, but without success:
Updating the versions of the relevant packages: langchain to 0.1.1, langchain_mistralai to 0.0.3, and crewai to 0.1.32.
Adjusting the parameters such as temperature, max_tokens, and top_p across various ranges.
Does anyone have a solution for this problem?"
Thanks

I've wrote email to Mistral AI and the reply is they don't support Stop parameter at this moment. They're still working on it.

Also having an issue with this.

can you try the new RC and tell me if it works better?
pip install crewai==0.17.0rc2
João Moura
@joaomdmoura http://twitter.com/joaomdmoura
Em sáb., 2 de mar. de 2024 às 17:05, Joaquin Dominguez <
@.***> escreveu:

Also having an issue with this.
—
Reply to this email directly, view it on GitHub
https://github.com/joaomdmoura/crewAI/issues/155#issuecomment-1974895013,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFC3N5E6FJJ4O3KKVNNW7TYWIWHPAVCNFSM6AAAAABCASIGXCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZUHA4TKMBRGM
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I just tried it with the new rc you specified and still getting same error:


Error message

NHS design system guidance
NHS design system discussion
Related
GOV.UK Design System guidance
GOV.UK Design System discussion
Examples on the NHS App

Examples

Journeys used on

Request repeat prescription
Prescriptions

Overview
The content of error messages is not consistent throughout the app. The sentence structure used is not always consistent, for example, ‘Select X’ vs ‘You need to select X’. Additionally, the content is not always specific as per the guidelines, for example, ‘You need to select yes or no’.
Objective
Ensuring the content of our error messages is clear, specific and consistent, provides better support to users and reduces the cognitive load in understanding what has happened. Making our error messages consistent also will bring consistency across the app and with other NHS services.
Activities

[ ] Review all error messages + document inconsistencies
[ ] Update all error messages


Fix the build

Codecov Report

Merging #42 (134e0c3) into master (6b79f5a) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #42   +/-   ##
=======================================
  Coverage   93.17%   93.17%
=======================================
  Files           1        1
  Lines         953      953
=======================================
  Hits          888      888
  Misses         65       65

Impacted Files
Coverage Δ

source/argparse.d
93.17% <100.00%> (ø)

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 6b79f5a...134e0c3. Read the comment docs.