create container with already existing Multi tenant backup.

Freddy,
Can you point the way here for me?
i have a multi tenant database that i want to be restored in the container.
i studied the new-bccontainer script,
it has the multitenant flag..
and i checked that later on in the code there is a search on app.bak  (as application backup) (in combination with -bakfolder) and all the other backups are the tenants backups. (which probebly defines the tenant name as well?)
So i copy the folder T1702-Template which contains app.bak and T1702.bak to the workspace of the agent (devops agent)
and i pass this folder to the new-bccontainer (in my case) F:\DevOps\MSBUILD15-1_work\1\T1702-Template
but it will fail like below.
Is this way even possible? is multitenant with bakfolder intendend for this purpose?
how could i make this work?
i could not find allot of docu around this.

(the issue below is because the ArtifactURL is made empty after creating the image:

and then lateron

will not be true so it moves to the else...
but i guess ist more of a wrong using of the parameters i guess?
-- container generation ---
BcContainerHelper version 2.0.7
Actual imagename is  bcimage-Multitenant:onprem-17.4.21491.21531-nl
MemoryLimit set to 10G
SQL MemoryLimit set to 30%
use traefik
Public DNS is MSBUILD15.MyDomain.nl
License found at F:\DevOps\MSBUILD15-1_work\1\s\Licenses\18.flf
Multi-tenant -> use backupfolder
backupfolder from : \MyDomain\share\MSFBUILDS\ContainerizedTestenBackups\T1702-Template
VERBOSE: Performing the operation "Copy Directory" on target "Item:
\MyDomain\share\MSFBUILDS\ContainerizedTestenBackups\T1702-Template Destination:
F:\DevOps\MSBUILD15-1_work\1\T1702-Template".
VERBOSE: Performing the operation "Create Directory" on target "Destination:
F:\DevOps\MSBUILD15-1_work\1\T1702-Template".
backupfolder agent F:\DevOps\MSBUILD15-1_work\1\T1702-Template
BcContainerHelper is version 2.0.7
BcContainerHelper is running as administrator
Host is Microsoft Windows Server 2019 Datacenter - ltsc2019
Docker Client Version is 19.03.14
Docker Server Version is 19.03.14
Fetching all docker images
Fetching all docker volumes
ArtifactUrl and ImageName specified
Image bcimage-multitenant:onprem-17.4.21491.21531-nl already exists
Enabling SSL as otherwise all clients will see mixed HTTP / HTTPS request, which will cause problems e.g. on the mobile and modern windows clients
Using image bcimage-multitenant:onprem-17.4.21491.21531-nl
PublicDnsName is MSBUILD15.MyDomain.nl
Get-Item : Cannot find path 'F:\DevOps\MSBUILD15-1_work\1\s\True\ServiceTier\program files\Microsoft Dynamics NAV' bec
ause it does not exist.
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.7\ContainerHandling\New-NavContainer.ps1:886 char:2
8

... vversion = (Get-Item -Path "$dvdPath\ServiceTier\program files\Micros ...

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

CategoryInfo          : ObjectNotFound: (F:\DevOps\MSBUI...ft Dynamics NAV:String) [Get-Item], ItemNotFoundExcep
tion
FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.GetItemCommand

New-BcContainer has a parameter called -bakFolder
First time you run a container like this:
New-BcContainer `
    -containerName $containerName `
    -accept_eula `
    -auth $auth `
    -Credential $credential `
    -artifactUrl $artifactUrl `
    -licenseFile $licenseFile `
    -bakFolder "c:\programdata\bccontainerhelper\mybakfolder"

It will populate the folder with .bak files for the databases in the container.
Next time you run the exact same command it will reuse the database backups in this folder (and be faster)
Description:
 .Parameter bakFolder
  A folder in which a backup of the database(s) will be placed after the container has been created and initialized
  If the folder already exists, then the database(s) in this folder will be restored and used.

I don't think this can be used for New-BcImage though, that is probably not supported, but if you place your databases in this folder and maybe set databaseName to the name of the app database, then you should get a container with your databases.

thnx ill try this out next week

Did you make this work?

Freddy, the priority on the job had changed last weeks, Next week i will be on it for the whole week again, i will let you know ok?

Freddy, it wont start, i get the same message as posted in the first post.

somehow it takes a turn in the New-BcContainer where it walks against this error.
New-BcContainer `
    -accept_eula `
    -accept_outdated `
    -containerName $ContainerName `
    -artifactUrl $ArtifactURL `
    -auth $Auth  `
    -credential $credential  `
    -licensefile $LicensePath `
    -imageName $imageName `
    -updateHosts `
    -doNotExportObjectsToText `
    -TimeZoneId "W. Europe Standard Time" `
    -memoryLimit $MemoryLimit `
    -sqlMemoryLimit $MemoryLimitSQL `
    -useTraefik
    -PublicDnsName $PublicDNS `
    -bakFolder $backupFolder `
    -multitenant
    -sqlTimeout 600

Instead of posting screenshots, I would appreciate if you post the text marked with the "insert Code" attribute in the toolbar.

Yes Sir,
`"C:\windows\System32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command ". 'F:\DevOps\MSBUILD15-1_work_temp\dc57f9c1-1716-4f9a-9225-6d0fd6fe71a6.ps1'"
BcContainerHelper version 2.0.11
Actual imagename is  bcimage-Multitenant:onprem-18.0.23013.23795-nl
MemoryLimit set to 12G
SQL MemoryLimit set to 30%
use traefik
Public DNS is MSBUILD15.mysolution.nl
License found at F:\DevOps\MSBUILD15-1_work\1\s\Licenses\18.flf
Multi-tenant -> use backupfolder
backupfolder from : \msfile01\share\MSFBUILDS\ContainerizedTestenBackups\T1702-Template
backupfolder agent C:\ProgramData\BCContainerHelper\T1702-Template
BcContainerHelper is version 2.0.11
BcContainerHelper is running as administrator
Host is Microsoft Windows Server 2019 Datacenter - ltsc2019
Docker Client Version is 19.03.14
Docker Server Version is 19.03.14
Fetching all docker images
Fetching all docker volumes
ArtifactUrl and ImageName specified
Image bcimage-multitenant:onprem-18.0.23013.23795-nlwith_db already exists
Enabling SSL as otherwise all clients will see mixed HTTP / HTTPS request, which will cause problems e.g. on the mobile and modern windows clients
Using image bcimage-multitenant:onprem-18.0.23013.23795-nlwith_db
PublicDnsName is MSBUILD15.mysolution.nl
Get-Item : Cannot find path 'F:\DevOps\MSBUILD15-1_work\1\s\True\ServiceTier\program files\Microsoft Dynamics NAV' bec
ause it does not exist.
At C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\ContainerHandling\New-NavContainer.ps1:903 char:
28

... vversion = (Get-Item -Path "$dvdPath\ServiceTier\program files\Micros ...

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

CategoryInfo          : ObjectNotFound: (F:\DevOps\MSBUI...ft Dynamics NAV:String) [Get-Item], ItemNotFoundExcep
tion
FullyQualifiedErrorId : PathNotFound,Microsoft.PowerShell.Commands.GetItemCommand

##[error]PowerShell exited with code '1'.
Finishing: Create Build Container
`

And if you remove the imageName parameter to avoid the image creation alltogether?

Every run takes a long time on container creation.
"C:\windows\System32\WindowsPowerShell\v1.0\powershell.exe" -NoLogo -NoProfile -NonInteractive -ExecutionPolicy Unrestricted -Command ". 'F:\DevOps\MSBUILD15-1_work_temp\d269a066-30b0-4063-bf3e-04c509a3648c.ps1'"
BcContainerHelper version 2.0.11
Actual imagename is  bcimage-Multitenant:onprem-18.0.23013.23795-nl
MemoryLimit set to 12G
SQL MemoryLimit set to 30%
use traefik
Public DNS is MSBUILD15.MyDomain.nl
License found at F:\DevOps\MSBUILD15-1_work\1\s\Licenses\18.flf
Multi-tenant -> use backupfolder
backupfolder from : \msfile01\share\MSFBUILDS\ContainerizedTestenBackups\T1702-Template
VERBOSE: Performing the operation "Copy Directory" on target "Item:
\msfile01\share\MSFBUILDS\ContainerizedTestenBackups\T1702-Template Destination:
C:\ProgramData\BCContainerHelper\T1702-Template".
VERBOSE: Performing the operation "Create Directory" on target "Destination:
C:\ProgramData\BCContainerHelper\T1702-Template".
VERBOSE: Performing the operation "Copy File" on target "Item:
\msfile01\share\MSFBUILDS\ContainerizedTestenBackups\T1702-Template\app.bak Destination:
C:\ProgramData\BCContainerHelper\T1702-Template\app.bak".
VERBOSE: Performing the operation "Copy File" on target "Item:
\msfile01\share\MSFBUILDS\ContainerizedTestenBackups\T1702-Template\T1702.bak Destination:
C:\ProgramData\BCContainerHelper\T1702-Template\T1702.bak".
backupfolder agent C:\ProgramData\BCContainerHelper\T1702-Template
BcContainerHelper is version 2.0.11
BcContainerHelper is running as administrator
Host is Microsoft Windows Server 2019 Datacenter - ltsc2019
Docker Client Version is 19.03.14
Docker Server Version is 19.03.14
Fetching all docker images
Fetching all docker volumes
Enabling SSL as otherwise all clients will see mixed HTTP / HTTPS request, which will cause problems e.g. on the mobile and modern windows clients
Using image mcr.microsoft.com/businesscentral:10.0.17763.1817
PublicDnsName is MSBUILD15.MyDomain.nl
Creating Container MSBUILD151
Version: 18.0.23013.23795-nl
Style: onprem
Multitenant: Yes
Platform: 18.0.22893.23773
Generic Tag: <IP_ADDRESS>
Container OS Version: 10.0.17763.1817 (ltsc2019)
Host OS Version: 10.0.17763.1817 (ltsc2019)
Using process isolation
Using locale nl-NL
Adding special CheckHealth.ps1 to enable Traefik support
Disabling the standard eventlog dump to container log every 2 seconds (use -dumpEventLog to enable)
Additional Parameters:
-e webserverinstance=MSBUILD151
-e publicdnsname=MSBUILD15.MyDomain.nl
-l "traefik.protocol=https"
-l "traefik.soap.frontend.rule=PathPrefix:/MSBUILD151soap;ReplacePathRegex: ^/MSBUILD151soap(.) /BC$1"
-l "traefik.soap.port=7047"
-l "traefik.rest.frontend.rule=PathPrefix:/MSBUILD151rest;ReplacePathRegex: ^/MSBUILD151rest(.) /BC$1"
-l "traefik.rest.port=7048"
-l "traefik.dev.frontend.rule=PathPrefix:/MSBUILD151dev;ReplacePathRegex: ^/MSBUILD151dev(.) /BC$1"
-l "traefik.dev.port=7049"
-l "traefik.snap.frontend.rule=PathPrefix:/MSBUILD151snap;ReplacePathRegex: ^/MSBUILD151snap(.) /BC$1"
-l "traefik.snap.port=7083"
-l "traefik.dl.frontend.rule=PathPrefixStrip:/MSBUILD151dl"
-l "traefik.dl.port=8080"
-l "traefik.dl.protocol=http"
-l "traefik.enable=true"
-l "traefik.frontend.entryPoints=https"
--env customNavSettings=PublicODataBaseUrl=https://MSBUILD15.MyDomain.nl/MSBUILD151rest/odata,PublicSOAPBaseUrl=https://MSBUILD15.MyDomain.nl/MSBUILD151soap/ws,PublicWebBaseUrl=https://MSBUILD15.MyDomain.nl/MSBUILD151
Files in C:\ProgramData\BcContainerHelper\Extensions\MSBUILD151\my:

AdditionalOutput.ps1
CheckHealth.ps1
MainLoop.ps1
SetupVariables.ps1
updatehosts.ps1
Creating container MSBUILD151 from image mcr.microsoft.com/businesscentral:10.0.17763.1817
811d2c749f869192f725ad49db00fea983e517517d95d68931d48bcdfac60ed9
Waiting for container MSBUILD151 to be ready
Using artifactUrl https://bcartifacts.azureedge.net/onprem/18.0.23013.23795/nl
Using installer from C:\Run\150-new
Installing Business Central
Installing from artifacts
Starting Local SQL Server
WARNING: Waiting for service 'SQL Server (SQLEXPRESS) (MSSQL$SQLEXPRESS)' to
start...
Starting Internet Information Server
Copying Service Tier Files
c:\dl\onprem\18.0.23013.23795\platform\ServiceTier\Program Files
c:\dl\onprem\18.0.23013.23795\platform\ServiceTier\System64Folder
Copying PowerShell Scripts
c:\dl\onprem\18.0.23013.23795\platform\WindowsPowerShellScripts\Cloud\NAVAdministration
c:\dl\onprem\18.0.23013.23795\platform\WindowsPowerShellScripts\WebSearch
Copying dependencies
Copying ReportBuilder
Importing PowerShell Modules
Determining Database Collation from c:\dl\onprem\18.0.23013.23795\nl\database\Demo Database NAV (18-0).bak
Changing Database Server Collation to Latin1_General_100_CI_AS
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:\dl\onprem\18.0.23013.23795\platform\WebClient\Microsoft Dynamics NAV
Copying Client Files
c:\dl\onprem\18.0.23013.23795\platform\LegacyDlls\program files\Microsoft Dynamics NAV
c:\dl\onprem\18.0.23013.23795\platform\LegacyDlls\program files\Microsoft Dynamics NAV
c:\dl\onprem\18.0.23013.23795\platform\LegacyDlls\systemFolder
Copying ModernDev Files
c:\dl\onprem\18.0.23013.23795\platform
c:\dl\onprem\18.0.23013.23795\platform\ModernDev\program files\Microsoft Dynamics NAV
Copying additional files
Copying ConfigurationPackages
C:\dl\onprem\18.0.23013.23795\nl\ConfigurationPackages
Copying Test Assemblies
C:\dl\onprem\18.0.23013.23795\platform\Test Assemblies
Copying Applications
C:\dl\onprem\18.0.23013.23795\nl\Applications
Starting Business Central Service Tier
Importing license file
Stopping Business Central Service Tier
Installation took 432 seconds
Installation complete
Initializing...
Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file
Starting Container
Hostname is MSBUILD151
PublicDnsName is MSBUILD15.MyDomain.nl
Using NavUserPassword Authentication
Creating Self Signed Certificate
Self Signed Certificate Thumbprint 98874D2E1A4A2D830F3A172C7D1754DEEF4E29D4
Modifying Service Tier Config File with Instance Specific Settings
Modifying Service Tier Config File with settings from environment variable
Setting PublicODataBaseUrl to https://MSBUILD15.MyDomain.nl/MSBUILD151rest/odata
Setting PublicSOAPBaseUrl to https://MSBUILD15.MyDomain.nl/MSBUILD151soap/ws
Setting PublicWebBaseUrl to https://MSBUILD15.MyDomain.nl/MSBUILD151
Starting Service Tier
Registering event sources
Creating DotNetCore Web Server Instance
Creating http download site
Setting SA Password and enabling SA
Creating Build as SQL User and add to sysadmin
Creating SUPER user
Container IP Address: <IP_ADDRESS>
Container Hostname  : MSBUILD151
Container Dns Name  : MSBUILD15.MyDomain.nl
Web Client          : https://MSBUILD15.MyDomain.nl/MSBUILD151/
Dev. Server         : https://MSBUILD15.MyDomain.nl
Dev. ServerInstance : BC
Setting MSBUILD151 to <IP_ADDRESS> in host hosts file

Files:
http://MSBUILD15.MyDomain.nl:8080/ALLanguage.vsix
http://MSBUILD15.MyDomain.nl:8080/certificate.cer
Container Total Physical Memory is 8.0Gb
Container Free Physical Memory is 2.5Gb
Initialization took 144 seconds
Ready for connections!
Reading CustomSettings.config from MSBUILD151
Set SQL Server memory limit to 3686 MB
Set TimeZone in Container to W. Europe Standard Time
Creating Desktop Shortcuts for MSBUILD151
WARNING: The new settings value will not take effect until you stop and restart the service.
Removing Database CRONUS from localhost\SQLEXPRESS
Restoring C:\ProgramData\BcContainerHelper\T1702-Template\app.bak to CRONUS
Restoring C:\ProgramData\BcContainerHelper\T1702-Template\T1702.bak to T1702
The database restore operation failed due to the following error returned by SQL Server:
Execution Timeout Expired.  The timeout period elapsed prior to completion of the operation or the server is not responding.
at Restore, : line 25
at , : line 54
at , : line 53
at Invoke-ScriptInBcContainer, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\ContainerHandling\Invoke-ScriptInNavContainer.ps1: line 44
at Restore-DatabasesInBcContainer, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\Bacpac\Restore-DatabasesInNavContainer.ps1: line 70
at New-BcContainer, C:\Program Files\WindowsPowerShell\Modules\BcContainerHelper\2.0.11\ContainerHandling\New-NavContainer.ps1: line 1828
at , F:\DevOps\MSBUILD15-1_work\1\s\Build_tools\BuildScripts\Create-BC-Container.ps1: line 192
at , F:\DevOps\MSBUILD15-1_work_temp\d269a066-30b0-4063-bf3e-04c509a3648c.ps1: line 3
at , : line 1
The database restore operation failed due to the following error returned by SQL Server:
i set the sql timeout to 1200 but i think this restore does not pickup the parameter of the new-bccontainer?

Freddy, I have a workaround,
i just make a normal container based on a image, and then i just set some settings on the NST, restore with bakfolder and then dismount and mount and all is fine.


Make crossbeam no std

This makes the crossbeam-queue crate no_std/alloc and fixes no_std in crossbeam as a whole (I couldn't get it working without the fix).

This still has a bug apparently - will fix soon. Don't merge yet.

I couldn't actually get crossbeam to work in no_std even with the following setup:
[dependencies.crossbeam]
path = "../../crossbeam"
default-features = false
features = ["nightly"]

I've restructured how no std works (necessary for no std lazy_static). I don't know if this is correct or desired. It might count as breaking too. I've pushed it now so it can be reviewed (and also in the meanwhile used in flower, because we couldn't figure out how to get it to compile on no std without these changes).

I think commit 2639822 was fine, we just needed to add more tests to CI. :)
Are you sure that the setup you mentioned doesn't work?
[dependencies.crossbeam]
version = "0.7"
default-features = false
features = ["nightly"]

Is there perhaps a way for me to reproduce the problem?

@stjepang -- it will work/fail silently on platforms with std, as the dependencies will just link it in -- including the subcrates. However, on platforms where std is not present at all (like, in my case, for Flower's kernel's custom target) it will fail, as std does not exist. See this stackexchange question for a bit more background.

@Restioson So if I understood correctly, it seems like Crossbeam is properly setting #![no_std] for itself, but Crossbeam's dependencies are not, therefore pulling std in?
In that case perhaps we should fix those dependencies or conditionally disable them if the std feature is not enabled? Do you know which crates are at fault here?

I think it's mostly the subcrates. In the commit I pushed I have fixed it and verified that it works (we are currently using the fork in flower on local repo).

The current crossbeam's CI does not check for no_std environment, so we need to add some checks to CI and make some corrections, but I don't think a big change like this PR is necessary.

@taiki-e if you can figure out a way to get crossbeam to work truly without std without these kinds of changes I will be impressed lol. Currently the fact that it isn't using std isn't propagated to the subcrates nor its dependencies.

After some discussion (in that PR) closing in favour of #368.


Updating script to accept poetry 1.x

Currently the poetry-setup.py script only detects an installation of poetry if it is in version 0.12.x. However, poetry has been in version 1.x for a while now. This pull request updates the script accordingly.

@cfarrend I think the only reason why this script still exists is because it is mentioned in the README  as part of the installation instructions (https://github.com/qctrl/python-open-controls#from-source). That said, I'd be happy to just update the README and delete the script

This script may have been created when Poetry was still in its infancy. I think referring to the official installation instructions will be more useful https://python-poetry.org/docs/#installation as it contains instructions Linux, OSX and Windows
So yeah I'd say removing the script and suggesting the official install docs may be better for collaborators

Sounds good, I've removed the file and removed the mentions of it from the documentation.


Typo in prop wrapper function

In ShallowWrapper.js lines 210-214:
exports.prop = function(key) {
  return function(wrp) {
    return wpr.prop(key)
  }
}

You wrote wpr instead of wrp.

Thanks!


Replaced boost::format with standard functions

This PR implements the third alternative of replacing boost::format as discussed in #143 and #142.

Thank you!


chore: cmake cleanup

Working on some CMake cleanups. Second comment is automated formatting, so look at the changes from the first comment only if you want to see what has changed.

chore: cleanup cmake
style: cmake format

The first commit are all manual, intentional edits. The second commit should be all automated formatting. I am somewhat hoping this is being tested; cibuildwheel would explicitly test the built wheel and make sure it also passes tests in a fresh environment; my old scripts do not do that.


v1.0.1.mod not exsit

go<EMAIL_ADDRESS>reading<EMAIL_ADDRESS>404 Not Found.
please help fix it ~

<EMAIL_ADDRESS>,I can find it, but v1.0.1 not ~

Seems like the tagging isn't correct:

go get and go mod are looking for vX.X.X tags. (at least according to https://go.dev/blog/publishing-go-modules)


Adding a way to listen for json events over http.

This lets telegraph or other system send events in.

I think this is ready to look at. Sorry for the big refactor.
Does 3 things:

Moves kentik specific code out of the main kkc.go file into its own kentik.go section. Possibly later move this further out into an input module?

Creates a http input. This lets telegraf 's http output for example send input into ktranslate.

Creates a kflow format. This finally will allow sending from any intput to kentik itself. Right now, this handles arbitrary fields in a new way which kentik itself will not know how to handle. First a mapping flow is sent, with a list of all the custom field ids used. Then the flows themselves are sent over -- yes, this is basically ipfix ... Seemed good enough for now, what do you think?


Add Windows Web Account Manager (WAM) Brokered Authentication Support

This is a tracking work item for creating/finishing the new Azure.Identity.BrokeredAuthentication client library (separate package from the existing Azure Identity client library).
Related:

https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/identity/Azure.Identity.BrokeredAuthentication/README.md

Dependencies:

MSAL WAM is supported (Beta) in .NET and Python today
MSAL WAM support (Beta) for Java and Javascript is not yet available but is expected to be available in time for this scheduled development milestone.

Link to dependency - https://github.com/AzureAD/microsoft-authentication-library-for-js/issues/4346

@mpodwysocki Let's default to a silent auth experience that doesn't show the account picker UX if a user is already logged in. See https://github.com/Azure/azure-sdk-for-net/issues/36178


Remove quicli from dependencies

quicli is not being used and as it's a fairly heavy dependency there's really no reason to keep it.

Thanks for catching this. I'd honestly forgotten that it was once being used in the past.


[unittests] Ignore googletest DWARF directories

What's in this pull request?

The Swift build system does not generate DWARF symbols in the unittests build directory. For build systems that do, however, files would be generated with the test suffix "Tests". These need to be excluded because otherwise lit.formats.GoogleTest tries to execute them. See http://reviews.llvm.org/D18647 for details.
/cc @ddunbar
Resolved bug number: (SR-)

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

/cc @gribozavr

@swift-ci Please test and merge

Hmm, looks like the Linux test failed. Something to do with SwiftPM and package-tests (I believe this is a known issue on master at the moment). The Swift test suite, on the other hand, appears to have completed successfully. Anyone mind if I merge this before the SwiftPM failure is fixed?

Go for it, that test failure is being addressed independently.

Great, thanks!! :bow:


hot-reload/incremental build JSON language files

breaking change:
path option is deleted and users should load language JSON files using
gatsby-source-filesystem then load it using GraphQL query.
fix #4

@jngbng can we display a warning that the path config is deprecated in the onCreateNode handler? Also, wouldn't a breaking change warrant incrementing the major version, as the semantic versioning guidelines suggest? Otherwise, users might unintentionally update to the breaking change.

@jngbng can we display a warning that the path config is deprecated in the onCreateNode handler? Also, wouldn't a breaking change warrant incrementing the major version, as the semantic versioning guidelines suggest? Otherwise, users might unintentionally update to the breaking change.

@jdtzmn  Thanks for good suggestion. The maintainer will pick the version when releasing to NPM as this project manages version using release-it tool. He/she should edit README.md in my PR, though.
Displaying a warning is a good idea! But onCreateNode handler seems not a good place to print a warning as it is called frequently and will cause  excessive warnings. (That may be what we want, thought.) Since I am new to Gatsby plugin api, I have no idea of the best place. Another option would be post install hook of this package. (like core-js) Any suggestions are welcomed.

@jdtzmn  Thanks for good suggestion. The maintainer will pick the version when releasing to NPM as this project manages version using release-it tool. He/she should edit README.md in my PR, though.
Displaying a warning is a good idea! But onCreateNode handler seems not a good place to print a warning as it is called frequently and will cause  excessive warnings. (That may be what we want, thought.) Since I am new to Gatsby plugin api, I have no idea of the best place. Another option would be post install hook of this package. (like core-js) Any suggestions are welcomed.

@jngbng Makes sense. I was reviewing the PR and I saw the section in the README.md and figured I should mention it.
I found the following issue that addresses where to display deprecation warnings:
https://github.com/gatsbyjs/gatsby/issues/6974
They recommend using the onPreBootstrap handler.

@jngbng Makes sense. I was reviewing the PR and I saw the section in the README.md and figured I should mention it.
I found the following issue that addresses where to display deprecation warnings:
https://github.com/gatsbyjs/gatsby/issues/6974
They recommend using the onPreBootstrap handler.

@jdtzmn Thanks for your suggestion. Now plugin prints an error message when deprecated "path" option still exists in plugin option.

@jdtzmn Thanks for your suggestion. Now plugin prints an error message when deprecated "path" option still exists in plugin option.

@jngbng Love it! Looks great. I think it's ready for @jimmyn to review.

@jngbng Love it! Looks great. I think it's ready for @jimmyn to review.

I will check it asap, thanks for your contribution

I will check it asap, thanks for your contribution

If we are moving back to graphql we should also support loading namespaces separately. This is actually what I wanted to implement next and I need it myself. If it is a big page with lots of translation it does not make sense to load those translations on other pages but at the same time, you can have common translations for header, for example.
Is it going to work with a query like this:
query($language: String!) {
    locales: allLocale(filter: {lng: {eq: $language}, ns: {regex: "/(common|about)/"}}) {
      edges {
        node {
          ns
          data
          lng
        }
      }
    }
  }

Ideally, we should simplify it somehow or at least document
Another thing is that the language variable is inconsistent across the code
We now have language, lang, lng. I get that lng is shorter but I'd prefer consistency

If we are moving back to graphql we should also support loading namespaces separately. This is actually what I wanted to implement next and I need it myself. If it is a big page with lots of translation it does not make sense to load those translations on other pages but at the same time, you can have common translations for header, for example.
Is it going to work with a query like this:
query($language: String!) {
    locales: allLocale(filter: {lng: {eq: $language}, ns: {regex: "/(common|about)/"}}) {
      edges {
        node {
          ns
          data
          lng
        }
      }
    }
  }

Ideally, we should simplify it somehow or at least document
Another thing is that the language variable is inconsistent across the code
We now have language, lang, lng. I get that lng is shorter but I'd prefer consistency

@jimmyn Yes, it works as you mentioned. I added sections for it. https://github.com/microapps/gatsby-plugin-react-i18next/pull/43/commits/c40fcb15b2add0e5b6fdd1a8d451e4f80f6b14cb
Also changed lang, lng to language.

@jimmyn Yes, it works as you mentioned. I added sections for it. https://github.com/microapps/gatsby-plugin-react-i18next/pull/43/commits/c40fcb15b2add0e5b6fdd1a8d451e4f80f6b14cb
Also changed lang, lng to language.


Semivariance optimization

Describe the bug
The current implementation of the semicovariance is to compute the standard covariance matrix after setting returns below the benchmark to zero for the individual assets. Applying standard mean-variance optimization to such a matrix does, in general, not correspond to the actual objective of obtaining an efficient portfolio in the mean-semivariance space. It can also be shown that a matrix S for which semivariance_portfolio = x'Sx does not exist in general. Intuitively, this can be seen as the current behavior sets returns to zero in time periods where the asset is below the benchmark, whereas the portfolio semivariance sets the portfolio returns to zero where the portfolio is below the benchmark.
A detailed description can be found in Computation of mean-semivariance efficient sets by the Critical Line Algorithm, Markowitz, Todd, Xu, and Yamane (1993).
One approach to deal with this is to use an implementation of the Critical Line Algorithm, which can handle semivariances as described in the mentioned paper (I am working on that and plan to add a PR here once ready). An easier, if computationally more expensive, approach is to use an augmented problem that splits up the portfolios' returns in above and below benchmark parts. This has the drawback that for n assets and T periods, the number of optimization variables increases from n to n + 2T. A CVXR implementation of this approach can be found here.

@phschiele Thanks for reaching out!
I've changed the label to "enhancement" from "bug" because currently PyPortfolioOpt uses the implementation of Estrada, so it is one approach (if not the best approach) to find a portfolio that maximises return for a given semivariance.
But I'd love to improve PyPortfolioOpt's current implementation and greatly look forward to a PR :)

Fixed in v1.3.0

Fixed in v1.3.0


set priorityClass x509 helm chart

Recently we started to use x509 helm chart.
At the time it didn't have prirotiyClass in it, https://github.com/enix/x509-certificate-exporter/pull/91
The new helm chart containing the change is currently in beta.
Waiting for the new feature to get released.

Fixed by a4b0a66.


Return Parser<T> instead of Optional<T>

Aeson returns a Parser type, not a Maybe type. Generalizing away from Optional might help us support other return formats in the future (like Result)

addressed in #72


[WIP] Simplify the build, use .syso rather than manual go tool invocations

Go recently restricted the ability to use go:cgo_* directives outside of cgo-generated code so the current build no longer works.
I've modified the build to use the syso trick which greatly simplifies the tool invocations. Everything builds successfully and all the symbols are found (tested on Debian GNU/Linux buster) but I get a segfault as soon as I run the program.
So I guess rustgo.s needs to be updated, but my assembly hacking skills are lacking here. Since you've been through this process before, I thought you might be able to fix this up quicker on top of what I've already done.
For a start, changing MOVQ scalar_base_mult(SB), AX \n CALL AX to CALL scalar_base_mult(SB) results in a "simpler" segfault error but I have no idea if it's a "better" error.

Looks like go is not performing relocations when linking the syso into the binary, objdump -rS of the syso file has expected lines like:
0000000000000000 <scalar_base_mult>:
#[no_mangle]
pub extern fn scalar_base_mult(dst: &mut [u8; 32], k: &[u8; 32]) {
   0:   55                      push   %rbp
[..]
  4a:   e8 00 00 00 00          callq  4f <scalar_base_mult+0x4f>
                        4b: R_X86_64_PLT32      _ZN16curve25519_dalek5curve13ExtendedPoint16compress_edwards17h445d29c3c5f4a1baE-0x4
[..]
0000000000000000 <_ZN16curve25519_dalek5curve13ExtendedPoint16compress_edwards17h445d29c3c5f4a1baE>:
    pub fn compress_edwards(&self) -> CompressedEdwardsY {
   0:   55                      push   %rbp

but the binary ./ed25519-dalek-rustgo only has:
0000000000401000 <scalar_base_mult>:
  401000:       55                      push   %rbp
[..]
  401052:       e8 00 00 00 00          callq  401057 <scalar_base_mult+0x57>
  401057:       c5 f8 10 00             vmovups (%rax),%xmm0

which is obviously invalid, yet the compress_edwards function still exists later in that file.
Not sure if this is fixable by passing extra flags to go build...

Looks like this is the same issue mentioned near the end of your original blog post, but unfortunately cgo_import_dynamic is not allowed these days... :(

Might be fixed by https://go-review.googlesource.com/c/go/+/94675


Deserializing std::map

What is the issue you have?
When trying to deserialize a std::map which has a int/Enum as a key I'm getting compilation error.
Error	C2166	l-value specifies const object

When i try to deserialize a std::map with a key as a string for example, it compiles fine.

Please describe the steps to reproduce the issue. Can you provide a small but working code example?
Serialize a std::map having int/enum as a key. Try to deserialize it.

json.at(_materialTypeByFaceSelectionJson).get<map<MaterialTypeEnum, FaceSelectionEnum>>();

What is the expected behavior?
It should compile.

And what is the actual behavior instead?
It doesn't.

Which compiler and operating system are you using? Is it a supported compiler?
I'm using windows 10 and Visual Studio 2017.

Did you use a released version of the library or the version from the develop branch?
I'm using released version.

There is an open PR to provide this feature: #1089
Could you try this commit and tell me if it fixes your issue?

@nlohmann: since I need this feature, I am wondering if the develop branch is reliable to use it (unless 3.1.3 is coming).

I hope to release 3.1.3 before the end of July.

If one of the map pairs (e.g., std::pair<int64_t, double>) has std::nan, it becomes a null in the written json.
If I read it back as a map, I get: [json.exception.type_error.302] type must be number, but is null.
What is the suggested approach in such a case? Should I iterate through each pair? If yes, how?

@nlohmann do you prefer that I move the described issue in a stand-alone ticket?

Yes, please.


Remote Site Settting issue

Hi,
I am getting the following error when I click the Create Remote Site Setting.  Also in the error dialog section the last bullet point states the following...
Press the Create Remote Site Setting button to perform this step or refer to the post install step below to perform this manually.
Not sure where to go!
ERROR after clicking he button....
Cannot create a new component with the namespace: . Only components in the same namespace as the organization can be created through the API

You can create a remote site setting using these steps:

Go to Security Controls -> Remote Site Settings
Create a new remote site setting with the 'remote site url' being the url part before "/apex/custommetadatarecorduploader" when you load the Custom Metadata Loader Tab. It will look something like "https://NAMESPACE.INSTANCE.visual.force.com".

Can you let me know what is special about your org where you are seeing this error? Did you package this data loader? Is the data loader's namespace different from the org's namespace?

That did the trick - thank you for your help.

Hi @haripriyamurthy :
I am also getting following error:
Error:
Cannot create a new component with the namespace: wubs. Only components in the same namespace as the organization can be created through the API.
I have deployed using workbench and I created remote site setting also as per your post with the 'remote site url' same as of my sandbox domain url.
Could you please help me here if I am missing anything here. Thanks.

@Nikhil-93 When do you see this error? Creating a remote site setting manually should solve the problem above.

@haripriyamurthy : It's fixed now. Actually issue was that there were some letters in uppercase in domain url and browser automatically converts uppercase letters into lowercase. For eg. browse was redirecting at "https://abcdev.cs40.my.salesforce.com" and when I checked for actual endpoint on workbench, it was showing "https://ABCDev.cs40.my.salesforce.com". Once I updated actual endpoint, it worked good for me.
Thanks.

Thanks for the details. I know this case is closed. But is there Remote Site Name we need to provide. or we can name whatever we choose to?

It should follow the convention for remote site settings in org and that check if your vf page uses the same name you provided.


level=error msg="Handler for GET /v1.17/containers/4921160fdc99796b2a1cd133b23bff6a9e77bfd7971f537a2cdfd7b9fbe35e6b/logs returned error: Failed to get logging factory: logger: no log driver named 'none' is registered"

Our log collection container uses the docker logs API through /var/run/docker.sock to collect container logs.

  So I configured the container I don't want logs from to use the syslog driver instead, and now I get this:
Jan  5 19:22:55 ip-172-20-176-161 docker[1117]: time="2017-01-05T19:22:55.894198023Z" level=error msg="Handler for GET /v1.17/containers/a8ba3caf03a2d033a78b53aa32d5de2adc6461208496c422ff76bf2b0e9aad51/logs returned error: configured logging reader does not support reading"

@gswallow Not sure what you are expecting to happen here.
We do catch the "none" error earlier now, but you'll still get an error when you tried to read logs on none.
Likewise with log drivers that don't support reading.
Currently the only drivers that do support reads are jsonfile and journald.
This is generally because other protocols, e.g. syslog, do not have a standard way to retrieve logs.

My intent here was to separate the ECS agent container's logs from everything else.  The ECS agent container logs to stdout, which gets collected by Docker in /var/lib/container/X/log, and it logs to a file mapped as a volume: /var/log/ecs/ecs-agent.log*.  I added --log-driver=none for that container in its systemd file.
Docker shouldn't log errors to journald when it finds a container that has been explicitly told not to log?

The error in this case is the user requested something that is impossible -- e.g. To consume logs on a container that doesn't support this.
I agree we probably shouldn't log errors which are user errors. Today we log every non-successful request as an error, though.

Looks like your settings attempt to get logs for all containers (allContainers: true), probably need to narrow that down to only attempt getting logs from containers that have logging enabled

If I understand the docker remote API docs correctly, one (in this case, Sumologic) would need to get a list of the containers first by GETting /containers, and then for each container ID or name in the list, GET /containers//logs, optimally with a timestamp.
With docker remote API v1.19 or earlier, the only way to filter the first call (get all containers) is by exit code, status, or label.  I suppose Sumologic could get the list of containers and then filter out containers whose tags include "ignore-my-logs," or something.  But the filtering logic would need to be written by the Sumologic guys.

Yes, I think something like that would be needed. I'm not familiar with Sumologic, so don't know what options they offer.
Another approach is to not "poll" the docker remote API to get those logs; if you want to centralise logging, use a logging driver that sends all logs to a central logging collector, https://docs.docker.com/engine/admin/logging/overview/

I agree we probably shouldn't log errors which are user errors. Today we log every non-successful request as an error, though.

I'm not sure what's best here; is "silently" ignoring the problem the best solution?

I would agree about what's best.  It should be obvious to the docker engine that I configured a container with --log-driver=none.
What is your policy on backporting changes to older docker releases?  We're sort of stuck on version 1.11.x because AWS ECS and Empire recommend/require it, but the fix, mentioned by @cpuguy83, above, applies to Docker 1.12.4 onwards.  My solution now is just to discard AWS ECS logs lower than "crit" priority.  The ECS agent is very chatty and its logs are less than helpful, but telling the ECS agent to only log critical messages may bite me in the end.

It should be obvious to the docker engine that I configured a container with --log-driver=none.

Well, it is, so it outputs an error if someone still tries to obtain logs through the API

What is your policy on backporting changes to older docker releases

We maintain only the "current" release; no backporting.

That's unfortunate.  Unrelated changes in the API mean I can't upgrade.
Hopefully you'll take that step towards product maturiity soon.
On Mon, Jan 9, 2017 at 1:02 PM, Sebastiaan van Stijn <
<EMAIL_ADDRESS>wrote:

It should be obvious to the docker engine that I configured a container
with --log-driver=none.
Well, it is, so it outputs an error if someone still tries to obtain logs
through the API
What is your policy on backporting changes to older docker releases
We maintain only the "current" release; no backporting.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/docker/docker/issues/29915#issuecomment-271357214,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEB7qDutuLVscT06BPaW1rG2CL27MqR2ks5rQnZOgaJpZM4LcACP
.

Unrelated changes in the API mean I can't upgrade.

Could you add more information about this? The API is versioned, so docker 1.12 or 1.13 still supports the same API version as 1.11 uses (API version 1.23); https://docs.docker.com/engine/reference/api/docker_remote_api/

That's good to know.  I know that I'm being affected by this compatibility issue: https://github.com/remind101/empire/pull/915, which was fixed after the release of Empire 0.10.1.  Because the Remind team switched their default backend scheduler, I'll need to test the migration to Empire 0.11.0 before we can adopt it in production.
I think I'll just wait until both Empire and the ECS agent recommend a newer version of Docker.

Ah, yes, the /containers/<id>/copy endpoint was deprecated in docker 1.8, and does not work with API versions 1.24 and up; docs/deprecated.md#containersid-or-namecopy-endpoint (https://github.com/docker/docker/pull/22149). It should still work on API 1.23 and under though. If that's not the case, please open a bug report


Symfony version requirement

You should set symfony version to at least 2.5 since Symfony\Component\Console\Question\Question is not available in 2.4 and it's required to generate the gulpfile

Thanks for reporting this. I just fixed the issue. Once I create a new tag you'll be able to get the fix by doing composer update regularjack/frontend-bundle. I want to get some other fixes out before creating the tag.

Thanks for reporting this. I just fixed the issue. Once I create a new tag you'll be able to get the fix by doing composer update regularjack/frontend-bundle. I want to get some other fixes out before creating the tag.

No problem, I just upgraded my project from 2.4 to 2.5 and everything went fine ;)

No problem, I just upgraded my project from 2.4 to 2.5 and everything went fine ;)

@regularjack The issue with that solution is that people using Symfony with the Standard Edition from 2.3 won't get the update at all. A better solution would be to only use the Question class when available.

@regularjack The issue with that solution is that people using Symfony with the Standard Edition from 2.3 won't get the update at all. A better solution would be to only use the Question class when available.

@xabbuh You are indeed correct. I naively assumed symfony-standard allowed upgrading to minor versions, e.g. it required symfony/console ~2.3 but it's clearly not the case.
In fact there is no reason why Symfony 2.3 projects should not be able to use symfony/console 2.5 because 2.5 is backwards compatible with 2.3 but since symfony-standard does not allow this another solution is indeed necessary.
I'm looking into the simplest way to fix this, suggestions are welcome.


Graceful handling of Bluetooth access disable

First of all I want say HUGE thanks to everyone involved in this library. Pleasure to use: clean, easy to use and reliable. Outstanding work!
I'm trying to figure out is it possible to handle error gracefully in one of the edge-cases, when access to the Bluetooth is disabled when app is running. Currently iOS crashes with log entries:
bluetoothd	TCC changed, lets figure out if one of our sessions needs to be removed
bluetoothd	BluetoothTCC changed
bluetoothd	App <private> bluetooth authorization changed
bluetoothd	Push AWD stats for {my.bundle.id}: triggerLoc: 1, prevPermission: 0, currPermission: 1, appSDKVer: d0400
bluetoothd	bundle ID's not allowed to use bluetooth <private>
tccd	Creating service facility connection with <private>
tccd	[FBSSystemAppProxy:0x10f28ac90] Service facility connection invalidated
tccd	Terminating {my.bundle.id}[13123] because access to the kTCCServiceBluetoothAlways service changed.
backboardd	Connection removed: IOHIDEventSystemConnection uuid:A8B343CF-8E77-408E-83CC-A0CD2BC5DA66 pid:13123 process:{app.name} type:Passive entitlements:0x0 caller:BackBoardServices: <redacted> + 384 attributes:{
    HighFrequency = 1;
    bundleID = "{my.bundle.id}";
    pid = 13123;
} inactive:0 events:96 mask:0x800
SpringBoard	Workspace connection invalidated for <FBWorkspaceServer: 0x283d62940>
SpringBoard	[application<{my.bundle.id}>:13123] Now flagged as pending exit for reason: workspace client connection invalidated
ackboardd	HIDAnalytics Unregister Send event com.apple.hid.queueUsage
accountsd	"<private> (<private>) received"
bluetoothd	Closed XPC connection to session "{my.bundle.id}-central-13123-677"
bluetoothd	Unregistering session "{my.bundle.id}-central-13123-677"
bluetoothd	bundle ID's not allowed to use bluetooth <private>

Message is very clear and logical. But maybe it is possible to handle this in the app somehow to prevent "crash"?
Steps to reproduce

Connect to the BLE device
Navigate into Settings -> Your App
Disable Access to the Bluetooth

Expected behavior
App handles event and  stops using bluetooth
Actual behavior
App crashes
Configuration
Version of the Plugin: 2.1.1
Platform: iOS 13.3.1
Device: iPhone Xs

Hi @asarapas,
I'm the friendly issue checker.
Thanks for using the issue template :star2:
I appreciate it very much. I'm sure, the maintainers of this repository will answer, soon.


i18n(zh-cn): Update prefetch.mdx

Description (required)
Update prefetch.mdx

Related issues & labels (optional)

#8246

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

zh-cn
guides/prefetch.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Error with the ninja building "sparse_attn"

In
python 3.6.7
pytorch 1.5.0
torchvision 0.6.0
cuda 10.2
gcc 5.4.0
ubuntu 16.04.5 LTS
Using /tmp/torch_extensions as PyTorch extensions root...
Emitting ninja build file /tmp/torch_extensions/sparse_attn/build.ninja...
Building extension module sparse_attn...
Allowing ninja to set a default number of workers... (overridable by setting the environment variable MAX_JOBS=N)
[1/2] c++ -MMD -MF utils.o.d -DTORCH_EXTENSION_NAME=sparse_attn -DTORCH_API_INCLUDE_EXTENSION_H -isystem /usr/local/lib/python3.6/dist-packages/torch/include -isystem /usr/local/lib/python3.6/dist-packages/torch/include/torch/csrc/api/include -isystem /usr/local/lib/python3.6/dist-packages/torch/include/TH -isystem /usr/local/lib/python3.6/dist-packages/torch/include/THC -isystem /usr/include/python3.6m -D_GLIBCXX_USE_CXX11_ABI=0 -fPIC -std=c++14 -O2 -fopenmp -c /usr/local/lib/python3.6/dist-packages/deepspeed/ops/csrc/sparse_attention/utils.cpp -o utils.o
FAILED: utils.o
c++ -MMD -MF utils.o.d -DTORCH_EXTENSION_NAME=sparse_attn -DTORCH_API_INCLUDE_EXTENSION_H -isystem /usr/local/lib/python3.6/dist-packages/torch/include -isystem /usr/local/lib/python3.6/dist-packages/torch/include/torch/csrc/api/include -isystem /usr/local/lib/python3.6/dist-packages/torch/include/TH -isystem /usr/local/lib/python3.6/dist-packages/torch/include/THC -isystem /usr/include/python3.6m -D_GLIBCXX_USE_CXX11_ABI=0 -fPIC -std=c++14 -O2 -fopenmp -c /usr/local/lib/python3.6/dist-packages/deepspeed/ops/csrc/sparse_attention/utils.cpp -o utils.o
/usr/local/lib/python3.6/dist-packages/deepspeed/ops/csrc/sparse_attention/utils.cpp: In function 'void segment_blocks(at::Tensor, at::Tensor, at::Tensor, int, ret_t&)':
/usr/local/lib/python3.6/dist-packages/deepspeed/ops/csrc/sparse_attention/utils.cpp:87:71: error: converting to 'std::vector<std::tuple<int, at::Tensor> >::value_type {aka std::tuple<int, at::Tensor>}' from initializer list would use explicit constructor 'constexpr std::tuple<_T1, _T2>::tuple(_U1&&, _U2&&) [with _U1 = int&; _U2 = at::Tensor; <template-parameter-2-3> = void; _T1 = int; _T2 = at::Tensor]'
     if (!to_cat.empty()) ret.push_back({max_width, torch::cat(to_cat)});
                                                                       ^
/usr/local/lib/python3.6/dist-packages/deepspeed/ops/csrc/sparse_attention/utils.cpp: In function 'ret_t sdd_segment(at::Tensor, int)':
/usr/local/lib/python3.6/dist-packages/deepspeed/ops/csrc/sparse_attention/utils.cpp:110:90: warning: narrowing conversion of 'H' from 'size_t {aka long unsigned int}' to 'long int' inside { } [-Wnarrowing]
     torch::Tensor scratch = torch::empty({H, layout.sum().item<int>(), 4}, layout.dtype());
                                                                                ^
/usr/local/lib/python3.6/dist-packages/deepspeed/ops/csrc/sparse_attention/utils.cpp:110:90: warning: narrowing conversion of 'H' from 'size_t {aka long unsigned int}' to 'long int' inside { } [-Wnarrowing]
ninja: build stopped: subcommand failed.

and
ERROR [01/26 16:37:15 fastreid.engine.train_loop]: Exception during training:
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/torch/utils/cpp_extension.py", line 1400, in _run_ninja_build
    check=True)
  File "/usr/lib/python3.6/subprocess.py", line 418, in run
    output=stdout, stderr=stderr)
subprocess.CalledProcessError: Command '['ninja', '-v']' returned non-zero exit status 1.

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "./fastreid/engine/train_loop.py", line 121, in train
    self.run_step()
  File "./fastreid/engine/train_loop.py", line 200, in run_step
    outputs = self.model(data)
  File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/torch/nn/parallel/data_parallel.py", line 153, in forward
    return self.module(*inputs[0], **kwargs[0])
  File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "./fastreid/modeling/meta_arch/baseline.py", line 58, in forward
    features = self.backbone(images)  # (bs, 2048, 16, 8)
  File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "./fastreid/modeling/backbones/sparse_transformer.py", line 415, in forward
    x = self.forward_features(x)
  File "./fastreid/modeling/backbones/sparse_transformer.py", line 408, in forward_features
    x = blk(x)
  File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "./fastreid/modeling/backbones/sparse_transformer.py", line 257, in forward
    x = x + self.drop_path(self.attn(self.norm1(x)))
  File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "./fastreid/modeling/backbones/sparse_transformer.py", line 235, in forward
    x = self.sparse_self_attn(q, k, v).transpose(1, 2).reshape(B, N, C)
  File "/usr/local/lib/python3.6/dist-packages/torch/nn/modules/module.py", line 550, in __call__
    result = self.forward(*input, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/deepspeed/ops/sparse_attention/sparse_self_attention.py", line 152, in forward
    attn_output_weights = sparse_dot_sdd_nt(query, key)
  File "/usr/local/lib/python3.6/dist-packages/deepspeed/ops/sparse_attention/matmul.py", line 712, in __call__
    db_lut, db_num_locks, db_width, db_packs = self.make_lut(a.dtype, a.device)
  File "/usr/local/lib/python3.6/dist-packages/deepspeed/ops/sparse_attention/matmul.py", line 634, in make_lut
    c_lut, c_num_locks, c_width, c_packs = _sparse_matmul.make_sdd_lut(layout, block, dtype, device)
  File "/usr/local/lib/python3.6/dist-packages/deepspeed/ops/sparse_attention/matmul.py", line 99, in make_sdd_lut
    _sparse_matmul._load_utils()
  File "/usr/local/lib/python3.6/dist-packages/deepspeed/ops/sparse_attention/matmul.py", line 94, in _load_utils
    _sparse_matmul.cpp_utils = SparseAttnBuilder().load()
  File "/usr/local/lib/python3.6/dist-packages/deepspeed/ops/op_builder/builder.py", line 180, in load
    return self.jit_load(verbose)
  File "/usr/local/lib/python3.6/dist-packages/deepspeed/ops/op_builder/builder.py", line 216, in jit_load
    verbose=verbose)
  File "/usr/local/lib/python3.6/dist-packages/torch/utils/cpp_extension.py", line 898, in load
    is_python_module)
  File "/usr/local/lib/python3.6/dist-packages/torch/utils/cpp_extension.py", line 1086, in _jit_compile
    with_cuda=with_cuda)
  File "/usr/local/lib/python3.6/dist-packages/torch/utils/cpp_extension.py", line 1186, in _write_ninja_file_and_build_library
    error_prefix="Error building extension '{}'".format(name))
  File "/usr/local/lib/python3.6/dist-packages/torch/utils/cpp_extension.py", line 1413, in _run_ninja_build
    raise RuntimeError(message)
RuntimeError: Error building extension 'sparse_attn'

Can you try pre-compiling the op instead of using ninja? you can do this by re-installing with DS_BUILD_SPARSE_ATTN=1 pip install . from within the source directory (or replace . with deepspeed to get from pypi).

is gcc version too lower?

is gcc version too lower?

Thanks, It worked  atfer I updated gcc and other components.

Can you try pre-compiling the op instead of using ninja? you can do this by re-installing with DS_BUILD_SPARSE_ATTN=1 pip install . from within the source directory (or replace . with deepspeed to get from pypi).

Thanks, It worked  atfer I updated gcc and other components.

is gcc version too lower?

Thanks, It worked atfer I updated gcc and other components.

Thanks，I  update the gcc g++ from 5.4 to 7.5 ,then I success!
sudo apt-get install gcc-6 g++-6 cd /usr/bin sudo rm gcc sudo ln -s gcc-6 g++ sudo rm g++ sudo ln -s g++-6 g++from

Still get this error on CentOS-7, with gcc g++ of version 8.3.1.


Remove meta_mixin references

[Current coverage][cc-pull] is 94.39%

Merging [#238][cc-pull] into [develop][cc-base-branch] will not change coverage

@@            develop       #238   diff @@
==========================================
  Files            19         19
  Lines          1070       1070
  Methods           0          0
  Messages          0          0
  Branches        124        124
==========================================
  Hits           1010       1010
  Misses           27         27
  Partials         33         33

Powered by Codecov. Last updated by [4024a86...335a12b][cc-compare]
[cc-base-branch]: https://codecov.io/gh/nephila/djangocms-blog/branch/develop?src=pr
[cc-compare]: https://codecov.io/gh/nephila/djangocms-blog/compare/4024a860dddedb656be51af4ff935cec95081aa4...335a12bdd554296186d10e24cf7f92112877e74e
[cc-pull]: https://codecov.io/gh/nephila/djangocms-blog/pull/238?src=pr

Coverage remained the same at 97.477% when pulling 335a12bdd554296186d10e24cf7f92112877e74e on feature/meta_mixin_cleanup into 4024a860dddedb656be51af4ff935cec95081aa4 on develop.


Added warning on accidental exit

This PR fixes the issue: https://github.com/odk-x/tool-suite-X/issues/129

I got your point, @wbrunette. I've committed the changes. So, can you please review it?


_rest_info: get_rest_info was passed an invalid transform.

Ever since updating to Godot 4.3 (and updating Godot Jolt accordingly), whenever my player is instantiated, I'm getting spammed by this error:
E 0:00:24:0752   _rest_info: get_rest_info was passed an invalid transform. A scale of (1.000000, 1.200000, 1.000000) is not supported by Godot Jolt for this shape/body. The scale will instead be treated as (1.066667, 1.066667, 1.066667).
  <C++ Source>   src\spaces\jolt_physics_direct_space_state_3d.cpp:435 @ _rest_info()

This error never happened before updating, and I can't see where it could come from or how I could fix it.

Are you using a ShapeCast3D node somewhere perhaps? If that was scaled (maybe by some parent node) you would be seeing an error like that.
I realize the error message isn't ideal, but the context from where the error is emitted is very limited unfortunately. I've changed it to at least mention the possibility of ShapeCast3D, but haven't been able to cut a new release with that included yet.

Thank you for your help. I do use ShapeCast3D nodes in my weapons, but after trying with all weapons completely removed, the error still happens.
Is it possible that using PhysicsDirectSpaceState3D.intersect_shape() in a script could lead to this error?


Using the transport seems to break changing the log level

It seems that using this transport prevents me from changing the pino default log level.
For example:

import pino from 'pino';

const logger = pino({
  transport: {
    targets: [
      {
        target: 'pino-pretty',
        options: { colorize: true },
      },
      {
        target: 'pino-opentelemetry-transport',
        options: {
          resourceAttributes: {
            'service.name': 'whatever',
          },
       },
     },
   ]
 }
});
logger.level = "trace";
logger.trace("trace");
logger.debug("debug");
logger.info("info");

Gives the output:
[20:51:12.705] INFO (2543437): info
Whereas this:

import pino from 'pino';

const logger = pino({
  transport: {
    targets: [
      {
        target: 'pino-pretty',
        options: { colorize: true },
      },
    /*  {
        target: 'pino-opentelemetry-transport',
        options: {
          resourceAttributes: {
            'service.name': 'whatever',
          },
       },
     }, */
   ]
 }
});
logger.level = "trace";
logger.trace("trace");
logger.debug("debug");
logger.info("info");

Gives the output:
[20:54:06.302] TRACE (2543472): trace
[20:54:06.303] DEBUG (2543472): debug
[20:54:06.303] INFO (2543472): info

And my otel collector only ever receives the info log when the transport enabled.
Am I missing something obvious?
Perhaps worth mentioning I am writing this in typescript and running via ts-node. Perhaps that's an issue?

Hello @TreeOfLearning, sorry for the wait.
I was able to reproduce your issue in plain JS so ts-node is not the issue here.
The problem is visible only if both pino-pretty and pino-opentelemetry-transport are set as transports. If only one of those is set, all log levels are visible in the logs as expected.
You will have to explicitly set the log level for each transform for this to work the way you expect it to:
const logger = pino({
  transport: {
    targets: [
      {
        target: 'pino-pretty',
        options: { colorize: true },
        level: 'trace'
      },
      {
        target: 'pino-opentelemetry-transport',
        level: 'trace',
        options: {
          resourceAttributes: {
            'service.name': 'whatever'
          }
        }
      }
    ]
  }
})
logger.level = 'trace'
logger.trace('trace')
logger.debug('debug')
logger.info('info')

Or like this:
const logger = pino({
  transport: {
    targets: [
      {
        target: 'pino-pretty',
        options: { colorize: true },
        level: 'trace'
      },
      {
        target: 'pino-opentelemetry-transport',
        level: 'trace',
        options: {
          resourceAttributes: {
            'service.name': 'whatever'
          }
        }
      }
    ]
  },
  level: 'trace'
})
// logger.level = 'trace'
logger.trace('trace')
logger.debug('debug')
logger.info('info')

This is not specific to pino-opentelemetry-transport though,
You will see the same behaviour in similar combinations of different transports ... for example:
run
const pino = require('pino')

const logger = pino({
  transport: {
    targets: [
      {
        target: 'pino-pretty',
        options: { colorize: true },
      },
      {
        target: 'pino/file',
        options: { destination: 1 }, // this writes to STDOUT,
      }
    ]
  }
})
logger.level = 'trace'
logger.trace('trace')
logger.debug('debug')
logger.info('info')

compare it to:
const pino = require('pino')

const logger = pino({
  transport: {
    targets: [
      {
        target: 'pino-pretty',
        options: { colorize: true },
        level: 'trace'
      },
      {
        target: 'pino/file',
        options: { destination: 1 }, // this writes to STDOUT,
        level: 'trace'
      }
    ]
  }
})
logger.level = 'trace'
logger.trace('trace')
logger.debug('debug')
logger.info('info')


Update to Rails version <IP_ADDRESS>

This upgrades rails to version <IP_ADDRESS> with recent security fixes.
Requires use of the rails_4_2_11_1 branch of anemone.
See issue #342 for a related issue.

I merged the protocols-test-export PR into master, and updated this PR to merge into master.

There doesn't seem to be many changes here, just the qualifier on the anemone gem. I guess there is kind of an issue there in that the old versions of aquarium will need to use the old version of anemone, so it needs to stay there.


Qualification tool - Handle cancelled jobs and stages better and don't skip the app

Fixes https://github.com/NVIDIA/spark-rapids-tools/issues/1032
I ran an event log through the qualification tool and it got labelled as not applicable because it had failed stages. Those failed stages though were cancelled by AQE runs.
We should take this into account in the qual tool.
The reasons in task show up as: Stage cancelled...
The stage failure reason shows: Job 243 cancelled
tool output:
24/05/23 10:00:26 WARN QualificationEventProcessor: SQL execution id 47 had failures, skipping
24/05/23 10:00:26 WARN QualificationEventProcessor: SQL execution id 125 had failures, skipping
This PR fixes that by looking for cancelled in the failure messages ignores those as failures.
I tested on customer event log and this is working. Need to put that event log into our integration tests.

they should be separate. when I looked briefly at the profiling tool, I know its outputting failed jobs to files.  We still want to do that as that is how Spark is showing them.  I didn't look at all the rollups though to see where it they are affected.  Again a separate issue which I don't think is as important.


improve stability of ctx.add_basemap()

Fixes #247
Changed:

made sure all HTTPErrors are captured and retried
included the last error/reason message when max_retries is reached

The code in this contribution catches all HTTPErrors and therefore makes it impossible that the code continue with None in the arrays list. This used to cause varying errors related to None, depending on whether the first in the arrays list was None or not, or any of the others. By capturing all HTTPErrors, the code is redirected to retrying instead of that the _retryer() function returns None. After max_retries is reached, the user now gets a informative error message. The format of the message is based on what request.raise_for_status() would return.
The current code proves useful, these were the error messages during the manual tests:

HTTPError: Connection reset by peer too many times. Last message was: 502 Error: Bad Gateway for url: https://a.tile.openstreetmap.fr/hot/10/812/458.png
500 Error (forgot to copy this one when it occurred

@martinfleis could you review this PR?

I see some of the tests are failing because of os.wait() already taken care of in https://github.com/geopandas/contextily/pull/245. But that PR also has failing tests related to this PR. If you would merge #245 I can update the branch if required, but I leave it to you to decide about the way forward of course.

I've merged #245, you can try updating this one

The branch/fork was synced, but I think you have to approve the test workflow to be kicked off again.

I believe that the remaining failures are not related.

Ok, in that case please go ahead with merging the PR. Thanks for your consideration. I would like to update the minimal contextily version in my pyproject.toml once this fix is released. Is it already know when there will be a new release or not yet?

I'll cut the patch release today.


:bug: fix(bundler): Fix foundry detection bug

Description
Fix bug with foundry toml loading
Testing
Tested manually in vite app
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

Current dependencies on/for this PR:

main

PR #276   👈

This comment was auto-generated by Graphite.


Rework routing

Changes

changed it so the page path cant have a / at the beginning or the end
added sorting to GroupsSections
added groups routing
changed PageBuilder so it requires the id and not the path

Everything seems to work as intended!
we need to keep in mind that 404 pages will also include the navbar and footer now..


Pjax support

Hi, I'm trying to use javascript to reload the comments instead of reloading all the page, I thought that using pjax whould be the simplest way, but I can't think of a way of reloading the widget with the new comment once i have sent the data. Does something come to mind to you to be able to use ajax for this comment system?

Hi, a pjax submit feature has been added in the release 1.0.1. Please update your package via composer.

Le fuuuuuu-! I was ending a version of the pjax to share with you XD adding edit comments and rbac. Thanks for your hard work man! Would you be interested in comparing versions?

yes, of course. Give us link of your repository.

Here it is: https://github.com/Patroklo/yii2-comments-1
Made a lot of changes, probably you wont like a lot of them, but seemed useful at the moment. In the pjax and edition approach I have decide to use a less javascript based solution, since pjax does all the heavy lifting of the work.
Also added changes into the migration, you should add a double index in the entity, entityId columns, will fasten a lot your queries.
Thanks!


Proposer can de-register

Requirements
CHECK - original text says "A proposer can de-register from an optimist": clarify if this is only optimist/network
Implementation notes

FYI Replaces nf3.deregisterProposer

No longer valid


Use pixi for building geoarrow.proj wheels including proj

Ok so it looks like I'm able to build for linux and mac with pixi, just not windows yet

So I think half of the issue with Windows is that many packages don't include their pkg-config metadata files for windows. So just running
pkg-config --libs --cflags proj "proj >= 9.2.0"

doesn't work without polyfilling a few .pc files.
The other half is a path issue. So even when manually calling pkg-config works, it isn't found when called from rust.


flannel: set up route for the entire flannel network

Without it, packets leave via default route iface
and come back via interface added by flannel plugin.
If rp_filter=1, the packets are dropped.

@steveeJ PTAL

Approach and code look good on first sight. Allow me some time on the weekend to do a flannel set up for a full verification.

I was able to reproduce the situation and verify that your PR allows communication with other subnets in the flannel network.
LGTM


Bug: Reminder-cron Not Working Properly

The reminder-cron command is not properly functioning with the current implementation, outputting some sort of JSON statement containing a bool for success rather than a confirmation message for the reminder. (Edit: A more detailed bug report will be submitted later)

The functionality for this is somewhat lacking and the gif shown in the ReadMe does not actually reflect its actual functionality. Additionally, the test for this functionality is fabricated and does not do anything. I'm going to remove this for the time being, both for the sake of time and the lack of implementation by the previous team.

Closing for unique situation, may add this for future scope for future teams


Add the ability to configure exporters system ca bundle

[x] I have read Pelorus Contributing guidelines

Linked Issues
N/A
Description
Some people need to trust internal CAs emitting TLS certificates for internal only services.
Adding the internal CAs to the container system CA bundle avoids certificate validation errors in the exporters.
This PR adds the ability to override the exporters system CA bundle with a ConfigMap.
The ConfigMap must contain the ca-bundle.crt key populated with PEM formatted certificates to be trusted.
The ConfigMap will be mounted into the exporter pods updating the system CA bundle located in /etc/pki/tls/certs/ca-bundle.crt.
Testing Instructions

Create the ConfigMap with the CA bundle:
apiVersion: v1
kind: ConfigMap
metadata:
  labels:
    config.openshift.io/inject-trusted-cabundle: "true"
  name: cluster-ca-bundle

Create a values file with the custom_ca_configmap configuration:
instances:
  - app_name: deploytime-exporter
    exporter_type: deploytime
    custom_ca_configmap: cluster-ca-bundle
  - app_name: committime-exporter
    exporter_type: committime
    custom_ca_configmap: cluster-ca-bundle
    extraEnv:
    - name: PROVIDER
      value: containerimage

Deploy the exporters helm chart using the values file just created.

Test images available! 🧪🚀 To test operator with them, run
operator-sdk run bundle \
quay.io/pelorus/rc-pelorus-operator-bundle:vpr1141-23e90f7 \
--namespace test-pelorus-operator

To clean up environment afterwards, run
operator-sdk cleanup pelorus-operator --namespace test-pelorus-operator

Tested successfully following this instructions.


docker build command doesn't run

Bug 💥
running docker build -t dd:<your_tag> -f docker/pytorch-cpu-jupyter fails.
removing -f flag yields the following error:
 ERROR: Directory '.[source-pt]' is not installable. Neither 'setup.py' nor 'pyproject.toml' found.

Expected behavior  🧮
docker container starts as expected.
Screenshots 🖼

Desktop (please complete the following information, if any other than the one in the install requirements):

OS: windows11
docker: Docker version 20.10.14, build a224086
python: 3.11.2

Thanks.
Can you try instead
docker build -t dd:<your-tag> -f docker/pytorch-cpu-jupyter/Dockerfile .

This works on my local machine (Ubuntu 20.04) and creates an image that can be run with the following commands.

Doc fixed with 5fb23557f07d30c241a9b1fda7f42cb0f59f05cb

Thanks that fixed the issue!


extend timeouts on http operations

This change addresses https://github.com/pivotal-cf/pivnet-resource/issues/129
We have some operations against pivnet that consistently take a couple of minutes, so failing after 1 minute of waiting causes blocking issues in our pipelines.
CC @angelachin

Thank you for contributing to the go-pivnet API library!

Have you made this pull request to the develop branch?
The develop branch does not appear to exist.

We ran the units.

@julian-hj Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@julian-hj Thank you for signing the Contributor License Agreement!

Regarding our comment on https://github.com/pivotal-cf/pivnet-resource/issues/129#issuecomment-662079709 would you be interested in iterating on this PR?

Regarding our comment on pivotal-cf/pivnet-resource#129 (comment) would you be interested in iterating on this PR?

...Maybe? I am not sure whether we will have bandwidth or not, but we would defintely like to help. In the near term, is there some reason that a 10 minute timeout is problematic? We haven't seen any use cases on our (RelEng) end that get worse with a longer timeout, but maybe y'all have some cases that are different from ours.


update tooling, some fixes in types

fixes #10757

What kind of change does this PR introduce?
types

Did you add tests for your changes?
linting

Does this PR introduce a breaking change?
no, a bugfix

What needs to be documented once your changes are merged?
migration guide should contain hints how to migrate from @types/webpack to webpack types

@sokra did you also fix the Entry /EntryObject type mentioned in https://github.com/webpack/webpack/issues/10757
I'm not seeing changes in this diff

3. EntryObject is wrong
type Entry =
	| string
	| (() => string | EntryObject | [string, string] | Promise<EntryStatic>)
	| EntryObject
	| [string, string];
declare interface EntryObject {
	[index: string]: string | [string, string] | EntryDescription;
}

but doc says here :  https://webpack.js.org/configuration/entry-context/#dependencies
module.exports = {
  //...
  entry: {
    app: { import: './app.js', dependOn: 'react-vendors' },
    'react-vendors': ['react', 'react-dom', 'prop-types']
  }
};

which triggers a typescript error


[MINOR]: Fix usage of SchemaRegistryProvider with Row source

Change Logs
Added check for SchemaRegistryProvider with row source. For more context, see https://github.com/apache/hudi/issues/7064.
Impact
No. Users will be able to use SchemaRegistryProvider with row source like CsvDFSSource.
Risk level (write none, low medium or high below)
None
Documentation Update
none.
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

@pratyakshsharma @bvaradar in which version of hudi this change will come? I don't see this bug in release notes of 0.13.0 and 0.12.2


key does not exist

I have noticed that in bad network environment, I can see this error
2017-09-12T18:37:25+08:00 server /usr/local/bin/confd[29789]: ERROR template: tem.tmpl:1:21: executing "tem.tmpl" at <getv "mykey">: error calling getv: key does not exist: mykey

Of course this key exists.
And after debugging, resource.go - process - setVars of Watch process, t.storeClient.GetValues doesn't return error and return null map of vars. I guess that it is caused by timeout or other something.
Then it causes that createStageFile fails because that store has no any keys, so error above occurs.

And, because now confd use 2016-Spetember version of etcdv3 client, so maybe updating it can fix this and some other problems.

This should be fixed in this week by author.


Automated cherry pick of #9706: feat(region): multilingual timer description

Cherry pick of #9706 on release/3.7.
#9706: feat(region): multilingual timer description

Codecov Report

Merging #9707 (0699ba0) into release/3.7 (56d583e) will increase coverage by 5.96%.
The diff coverage is 0.00%.

@@               Coverage Diff                @@
##           release/3.7    #9707       +/-   ##
================================================
+ Coverage         6.00%   11.96%    +5.96%
================================================
  Files              845      231      -614
  Lines           131245    25750   -105495
================================================
- Hits              7879     3082     -4797
+ Misses          122665    22357   -100308
+ Partials           701      311      -390

Flag
Coverage Δ

aFlag
11.96% <0.00%> (+5.96%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/apis/list.go
0.00% <ø> (ø)

pkg/compute/guestdrivers/apsara.go
13.63% <0.00%> (-0.65%)
:arrow_down:

pkg/compute/guestdrivers/managedvirtual.go
0.00% <0.00%> (ø)

pkg/compute/guestdrivers/utils.go
0.00% <0.00%> (ø)

pkg/multicloud/esxi/host.go
0.77% <ø> (+<0.01%)
:arrow_up:

pkg/multicloud/esxi/image.go
0.00% <ø> (ø)

pkg/multicloud/esxi/manager.go
0.61% <0.00%> (-0.01%)
:arrow_down:

pkg/multicloud/esxi/snapshot.go
0.00% <ø> (ø)

pkg/multicloud/esxi/storage.go
0.00% <ø> (ø)

pkg/multicloud/esxi/storagecache.go
0.00% <ø> (ø)

... and 555 more

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 de64687...0699ba0. Read the comment docs.


No method for superclass constructor for a given class

There currently doesn't seem to be a way to call the super constructor of a given class without using .__class.__parent.__init. This means that trying to extend a class beyond one level of inheritance segfaults (from a stack overflow) at class.super(self), since self is the same in all constructors:
local BaseClass = class("BaseClass")
function BaseClass:new()

end

local SecondClass = class("DerivedClass", BaseClass)
function SecondClass:new()
  class.super(self) -- Instantiating SecondClass() works fine; class.super(self) refers to BaseClass
end

local ThirdClass = class("DoubleDerivedClass", SecondClass)
function ThirdClass:new()
  class.super(self) -- Instantiating ThirdClass segfaults, because `class.super(self)` in both ThirdClass and SecondClass refers to SecondClass:new() (`self` in both cases is the same)
end

This problem applies to MoonScript classes inherited from inherited Lua classes, too. I was able to quickly fix it in Muun by exposing parent.__init to base under the field super, which enables the more explicit
function SecondClass:new()
  SecondClass.super(self)
end

This way seems to work better, but without the syntactic sugar of super, we have to use SecondClass.__class.__parent.__init(self) which is...less than ideal, to say the least.
I don't know enough about how MoonScript's OOP works to ensure that adding this method won't break something down the line, so it'd be great to have a second opinion on this.

Since super is resolved at compile time in MoonScript, there is no clean solution that wouldn't introduce some new field into the classes that isn't there in MoonScript. I'm hesitant to do that. That's why this poor excuse for super exists.
I also see that the current implementation is basically useless, so I guess this additional field is absolutely required. But if it has to be added, then as a full implementation of MoonScript's super, with the same semantics - not just as an alias for the superclass constructor as it is now.

I've pushed branch super-test with a new implementation of super.
Your test code would change to this:
local BaseClass = class("BaseClass")
function BaseClass:new()

end

local SecondClass = class("DerivedClass", BaseClass)
function SecondClass:new()
  SecondClass.__super(self)
end

local ThirdClass = class("DoubleDerivedClass", SecondClass)
function ThirdClass:new()
  ThirdClass.__super(self)
end

This implementation has more complete semantics:
ThirdClass.__super.foo(self) -- calls `foo` in `SecondClass`

But it's not equivalent to MoonScript:
assert(ThirdClass.__super == SecondClass) -- this will fail

The old muun.super function does not exist anymore.
If you wanna test this and let me know if it works for you, that'd be great.

It's working for me--I think as long as you put a note letting people know that it's not semantically identical so much as the resulting classes are compatible the differences should be fine. My use case is just a LOVE-based UI toolkit I wrote in Lua that I'm extending and using in a MoonScript program, and it's working perfectly for that. Thanks!


Change wrong words korean README

Description

I naturally changed the awkward parts.
Thanks for pretty projects. 👍

@wooooooak can you take a look at these changes and approve, or leave comments otherwise?

@techinpark @mfix22 cooool. Thanks for better idea~!

@wooooooak this looks good to you? 👍 👎? 🙂

@mfix22 thanks for merge!
can you add me README.md in Contributors section?


Aptly CLI alongside the Api

Can you add the flag -no-lock to the aptly-api so that the CLI can be used alongside the api?

Now that this functionality has been merged in to master I'll close this issue. Thank you for the suggestion.


import uf2 random number generator

@pelikhan can you import the uf2 random number generator as a react component so i can embed it in the spec page?
https://github.com/microsoft/uf2/blob/master/patcher/index.html

See https://github.com/microsoft/jacdac/commit/fa941fbc701935d206290af153c60e4804c89cfe

Thanks Peli!


More packaging metadata & CircleCI publishing

Updates the csproj files for both distributable projects to have more accurate packaging metadata and updates the CircleCI config to setup nightly builds and also create a draft Github release & publish to nuget.org when tagging the repo.

@robbkidd and I added the user - thanks for the reminder @vreynolds 👍

@robbkidd and I added the user - thanks for the reminder @vreynolds 👍


01 - Create Initial HTML

This project will be carried out in a series of steps. First, get a basic structure into the html file. After I have done this will the css and js files, I will start to update all of them for a final version.

Done!


Modal and Drawer doesn't show or show delayed when open modal in Baidu map

Steps to reproduce
The modal doesn't show or show delayed when I click the marker in Baidu map.
I have a marker in Baidu map, it contains a function that to switch variable 'isVisible' from false to true to open a modal. I checked the inspector that isVisible of Modal is already became true, but modal doesn't show or show delayed.
I tried nz-modal and modal component both, but both not work.
The modal shows only if I resize the window of browser.
HTML:
<button nz-button [nzType]="'primary'" (click)="showModal(markers[1])">My OK</button>

<baidu-map [options]="opts" >
 <marker *ngFor="let marker of markers"
         (clicked)="showModal(marker)"
         (loaded)="setAnimation($event)"
         [point]="marker.point" [options]="marker.options"></marker>
 <!--导航控件-->
 <control type="navigation" [options]="controlOpts"></control>
 <!--比例尺-->
 <control type="scale" [options]="scaleOpts"></control>
 <!--地图类型-->
 <control type="maptype" [options]="mapTypeOpts"></control>
 <control type="geolocation" [options]="geolocationOpts"></control>
</baidu-map>

TS:
 public showModal(marker) {
   console.log(marker);
   this.text = marker['point']['lng'];
   // const theDiv = document.querySelector('.ant-layout-content') as HTMLElement;
   const modal = this.modal.create({
     nzTitle: 'Modal Title',
     nzContent: ModalComponent,
     nzWidth:300,
     nzZIndex:1200,
     nzAutofocus: 'ok',
     // nzGetContainer: () => theDiv,
     nzComponentParams: {
       modalSelector: 'map',
       marker: marker
     },
     nzFooter: null
   });

   modal.afterOpen.subscribe(() => console.log('[afterOpen] emitted!'));
   // Return a result when closed
   modal.afterClose.subscribe(result => console.log('[afterClose] The result is:', result));
 }

new:
我已经改成component方式打开modal了，还是出现一样的问题, 不过它会出现遮罩，然后在页面的最左边出现一个白色的大约50*50的正方形,如下图所示。这个问题只会出现点击在百度地图里的元素才会出现，我在百度地图之外的地方设置一个button，打开modal就完全没有问题，而且我试了Drawer也会出现同样的问题，但是Message就可以正常打开

What is expected?
Modal shows when I click marker.
What is actually happening?
Modal doesn't show or show delayed.

Environment
Info

ng-zorro-antd
9.3.0

Browser
Chrome85.0.4183 and Safari

marker(clicked) 没被 zone 处理到，手动在 ngZone.run 里面调用.
https://stackblitz.com/edit/angular-ivy-aycda1


Error: The API deployment for this resource does not exist.

I completed running the azd up command without any errors and looks like my app is deployed successfully. However, I keep getting the following error (It's been a while since I completed the deployment, but I still get the error.) Any idea what causes this error?
Error: The API deployment for this resource does not exist. If you created the deployment within the last 5 minutes, please wait a moment and try again.

azd version?
azd version 1.0.2 (commit 145e046b1ea9394bd4e1b1d539eb32e860d692fb)
PS. I came across this issue too, but I can't still figure out how to solve the problem.

This is probably related to:
#388

@fakoe Yup! Problem solved.

I am using existing OpenAI service. Can you tell me how to fix the error.

If you are getting an error like this, it is probably due to the deployment ID not being specified correctly. I recommend enabling debug logging (see the FAQ in the README about logging) which will show you what URL OpenAI tries to reach, and then you can compare that URL to what the Azure studio shows.


mssql: 'Invalid column name' error, on valid column name; still executes query

I'm using LogMode to see the T-SQL that gets generated. The line that gets executed in orm.go:71 is this:
db.Where("UserID = ?", 2).First(&task)
After executing, it prints this error:
[35m(C:/Projects/tmp/go-sample/orm.go:71) mssql: Invalid column name 'UserID'.]
Right afterwards, it still executes the correct query:
[35m(C:/Projects/tmp/go-sample/orm.go:71) SELECT * FROM "tasks"  WHERE "tasks"."deleted_at" IS NULL AND ((UserID = '2')) ORDER BY "tasks"."id" ASC OFFSET 0 ROWS FETCH NEXT 1 ROWS ONLY]

I tried this out as well, it seems the printed error is simply the result from the executed query. As in "This error came from this query".
If I run the generated query manually I get the same error. The confusion here probably comes from the print order.

Does this works? db.Where("user_id = ?", 2).First(&task)


If a check fails, and fix runs and displays to the console, it's not obvious what is running

I am starting a check for aws sso login. I borrowed logic for checking if a session is expired, and then signin in from code I've used before. I've noticed that check doesn't typically show the output on the console, and was kinda surprised when I got the output for fix on the console. It is actually what I wanted, because there is output the user needs to attend to 😁 I am more creating is to see if this is the desired behavior or not.
I did notice that you get that output before it's clear  what the output is from.
Here's the implementation:
#!/usr/bin/env bash

subcommand=$1
if [ -z "$subcommand" ]; then
  echo "USAGE: $0 [check|fix]"
  exit 1
fi

aws_path=$(which aws 2>/dev/null)

function check() {
  echo -n "Checking for aws on your PATH... "

  if [[ -z "$aws_path" ]]; then
    echo "Couldn't find aws on the PATH"
    exit 1
  else
    echo "OK ($aws_path)"
  fi

  echo -n "Checking SSO session... ";
  if ! aws sts get-caller-identity >/dev/null 2>&1;  then
    echo "PROBLEM (SSO session invalid or expired)"
    exit 1
  else
    echo "OK"
  fi
}

function fix() {
  if [[ -z "$aws_path" ]]; then
    echo "Installing awscli..."
    brew install awscli
  fi

  if ! aws sts get-caller-identity >/dev/null 2>&1;  then
    echo "Signing in... keep an eye out for the browser window"
    aws sso login
  fi
}

case "$subcommand" in
  fix) fix; check ;;
  check) check ;;
  *)
    echo "Don't know how to '$subcommand'"
    exit 1
    ;;
esac

And the YAML:
apiVersion: scope.github.com/v1alpha
kind: ScopeDoctorCheck
metadata:
  name: 00-aws-sso
spec:
  check:
    target: aws-sso.sh check
  fix:
    target: aws-sso.sh fix
  description: AWS SSO
  help: You need to login to AWS SSO

And here's the output:
❯ scope doctor run --only 00-aws-sso
 INFO More detailed logs at /tmp/scope/scope-root-20240129-JH8m.log
Signing in... keep an eye out for the browser window
Attempting to automatically open the SSO authorization page in your default browser.
If the browser does not open or you wish to use a different device to authorize this request, open the following URL:

https://device.sso.us-west-1.amazonaws.com/

Then enter the code:

REDACTED
Successfully logged into Start URL: redacted
Checking for aws on your PATH... OK (/opt/homebrew/bin/aws)
Checking SSO session... OK
 INFO Check `00-aws-sso` failed. Fix ran successfully.

I'm up for showing the output. My initial thought was, there woudln't be anything that a user would want to see from a check, so that's why it's hidden, but happy to switch it up.

+1 I think it would be good to display some output from the checks.


(Question) Escaping nested input

Hello again 🙃
I've been struggling quite a bit while writing a parser that can handle the following:

Expressions inside single quotes (easy part).
Nested expressions are allowed, but it requires the inner quotes to be escaped (tricky part).

I've been trying to address the second point with something like expr.nested_in(just('\\').ignore_then("'").or(none_of("'")).repeated().collect().padded_by(just("'"))), but then the problem is that expr takes a &str, but collect will only allow me to map the escaped input to a String.
Mainly I'm looking for something like slice, but that it respects the escaping I performed. Is there a clean way to do this?
Sorry for the newbie question, I've just been stuck on this one for a while 😅

It seems like you might be using the wrong combinator here. nested_in is specifically for inputs that have nesting (for example, Rust-style token trees). See the nested.rs example.
It's a bit hard to tell what behaviour you're looking for from the information you provided alone unfortunately. Could you give me a few examples of cases you want to be able to handle?

@zesterer thanks for the reply! Okay so I just realized I used single quotes in the description when I was actually thinking about backticks. I shouldn't write GitHub issues at 1am lol.
The full story is that I'm trying to write a lexer for bash, using these rules for token recognition. I'm struggling with command expansions of the form `command`, since nested expansions are allowed but the inner backticks are escaped, and so my inner delimiters for the recursion are different from the outer ones.

So this seems to be a context-sensitive rule in bash. You'll probably want to pass down context about the delimiter that should be used using the then_with_ctx and configure combinators. Take a look at indent.rs for an example of how to use these combinators to introduce context sensitivity. In indent.rs, the context is 'the amount of indentation that corresponds to the current block', whereas in your case the context is 'the delimiter that should be used for representing command substitutions'.

@zesterer Got it, thanks for the clues! I'll close this now since it's not a bug and I don't want to clutter your issues.


When downloading curves, both blue and green curve should be written to file

If no flexibility option is selected, the green curve can be taken to be identical to the blue one.

If this is easy, it would be nice to have soonish because it helps to test whether flexibility-based measures are calculated correctly.


Clicking on an image while previewing an NImage image in NPopover will close abnormally

TuSimple/naive-ui version (版本)
2.34.4
Vue version (Vue 版本)
3.3.4
Browser and its version (浏览器及其版本)
Chrome(114.0.5735.133)
System and its version (系统及其版本)
MacOS(13.2.1 (22D68))
Node version (Node 版本)
16.17.0
Reappearance link (重现链接)
https://codesandbox.io/s/suspicious-ben-574g4c
Reappearance steps (重现步骤)

Click the button and wait for the image to load
Click on the image and wait for the image preview to appear
Click on the picture to enlarge or reduce operations, the picture will be closed abnormally

Expected results (期望的结果)
In popover, click the image to enlarge the operation can still preview without closing
Actual results (实际的结果)
Clicking on the image to enlarge in popover will turn off the image preview
Remarks (补充说明)

You need customize the display of n-popover according to your own business logic:
https://codesandbox.io/s/affectionate-alex-w3zcxq?file=/src/App.vue


Consider enabling strict roslyn setting

See: https://github.com/dotnet/corefx/pull/31457
cc @natemcmaster @davidfowl @Eilon

@pakrym - is the main place of value in Kestrel? If so, can you go ahead and run with strict temporarily just to see if there are any issues? (We'd need to fix the issues anyway if/when we enable this globally.)

We use structs all over, there’s probably code comparing a StringValues to null elsewhere.

OK. We can still preempt a lot of the fallout by fixing up areas where we know it'll be a disaster.


Codelens is displayed in the top left of the editor

VSCode Version: 1.7.1
OS Version: Windows 10

Steps to Reproduce:

Open any file that has a codelens
The codelens is on the top left of the editor.

Dev tools error:
Cannot read property 'range' of undefined: TypeError: Cannot read property 'range' of undefined
located at .vscode\extensions\chrmarti.regex-0.0.6\out\src\extension.js:26:79

This issue was moved to chrmarti/vscode-regex#7


Missing subscribeKeys(store, keys, cb) method ?

We have both store.subscribe(cb) and store.listen(cb) to decide if we want to call the listeners immediately on subscribe or not.
So why not having the same with subscribeKeys(store, keys, cb) along listenKeys(store, keys, cb) ?
import { listenKeys, type StoreValue } from 'nanostores';

export function subscribeKeys<
  SomeStore extends { get: () => StoreValue<SomeStore>; setKey: (key: any, value: any) => void }
>(
  $store: SomeStore,
  propsToListen: SomeStore extends { setKey: (key: infer Key, value: never) => unknown } ? readonly Key[] : never,
  callback: (_props: StoreValue<SomeStore>) => void
) {
  listenKeys($store, propsToListen, callback);
  callback($store.get());
}

I'm using it in prod and it's working pretty good !

Do you have a real use case for that? If yes, we can add.

The use case is similar to using the already implemented store.subscribe(cb) method.
It's to trigger the callback immediately on subscribe.
I'm using it in vanillaJS. For instance to init my JS classes with values from the store.
Exemple :
export default class MyExample {
  valueA = 0;

  constructor() {
    subscribeKeys(webgl, ['valueA'], this.handlePropsChanged);
  }

  handlePropsChanged = ({ valueA }) => {
    if (valueA != this.valueA) {
      // Animate with the new values directly instantiation and all the other times
      gsap.to(this, { valueA, valueB, duration: 1 });
    }
  }
}

If it's a use case you don't want to support, maybe for consistency store.subscribe(cb) should be removed ?

Send PR with a function to next branch

This was released with 0.11.0


Tensorflow.data module not recognized after vscode update

Environment data

VS Code version: Version: 1.62.0
Commit: b3318bc0524af3d74034b8bb8a64df0ccf35549a
Date: 2021-11-03T15:18:44.904Z
Electron: 13.5.1
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Linux x64 5.10.70-1-MANJARO snap
Jupyter Extension version (available under the Extensions sidebar): v2021.10.1001414422
Python Extension version (available under the Extensions sidebar): v2021.11.1422169775
OS (Windows | Mac | Linux distro) and version: Manjaro Linux
Python and/or Anaconda version: 3.9
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): conda
Jupyter server running: Local

Problem
I updated my vscode a couple of minutes ago and this feature was working: the problem I am going to describe is cause by the update:
The autocompletion is not working for the following module:

Logs

[FG] completion at XXX/test.py:0:18 ...
[FG]   parsing: XXX/test.py (1ms)
[FG]   binding: XXX/test.py (0ms)
[FG] completion at XXX/test.py:0:18 [found 0 items] (1ms)

in the examples i see its
import tensorflow as tf
tf.data.AUTOTUNE

which version of tensorflow r you using?

which version of tensorflow r you using?

The statement I used in my screenshot is a valid one and is executed normally, the problem comes from Pylance.
I am using TensorFlow 2.7.0.
Thanks for looking into this!

Pylance is able to recognize when import is done https://github.com/microsoft/pylance-release/issues/2050#issuecomment-963426222 way but when import is made using from tensorflow.data import AUTOTUNE pylance isn't able to recognize for me as well. But I assume this issue is there even before the latest vs code update and I also think  this is the same in other editors like PyCharm as well.

@vigneshgarrapally I am now *also* having a problem with autocompletion using Nvidia Dali:

This problem seems to hit a wide variety of libraries, I have other examples if you want

@vigneshgarrapally @bschnurr I am now *also* having a problem with autocompletion using Nvidia Dali:

This problem seems to hit a wide variety of libraries, I have other examples if you want

You can file a bug with tensorflow, but I dont think you are supposed to use it that way.
this also works
from tensorflow._api.v2.data import AUTOTUNE


Hello From IRAN

What the program do ?
Print Hello Hacktoberfest , IRAN

In what programming language it is written?
Golang

This pull request contains conflicts. Please fix it and reopen