P2: Cannot change photo download resolution - Roku Dev Team

P2: Cannot change photo download resolution, only SD option is present
https://roku.box.com/shared/static/1u32qvighsmtve9sltw6c2e4uxd13xmd.mp4
(Chris) This is by design. If only 720 TV, it will only give you SD. Need to confirm this is still teh case.

I've added a note:
screen.AddParagraph("You will only see options your Roku player supports.")
But this was not a bug.


Issues with Proposal E

As discussed on the working group Slack channel, I have some concerns with the current preferred proposal– Proposal E.
Things I'm concerned about in Proposal E (C1-3):

Adding another layer of object referencing (in addition to the existing OCI index/manifest list) to registries complicates the data model and GC.
Annotation filtering is an interesting feature (outside of just this work) but will be non-trivial for registries with large numbers of objects to add. I'd also be for doing this more generically rather than specifically for references.
The fallback of using tags as references will cause clutter on registries and require client side clean up as GCs currently preserve tagged items. Registries often have a repo tag limit (e.g.: ECR limits to 10k).

Questions I have (Q1-2):

How would one move an artifact and all its metadata from one registry to another?
What happens if the ecosystem does not broadly adopt the proposal and we're stuck in the fallback case?

For concerns (C1) and (C2), I think we need registry operators to chime in.
At a high level, I'm worried that requiring major changes to (properly) implement references will at best take a very long time to be adopted and at worst not be adopted broadly by the ecosystem.

Questions I have (Q1-2):

How would one move an artifact and all its metadata from one registry to another?

I'm doing this today with the tag based solutions in regclient. And extending that for the API support should be rather straightforward. For every manifest copied, there's a check to see if there's any tags referencing it that also need to be copied. It's a recursive process.

What happens if the ecosystem does not broadly adopt the proposal and we're stuck in the fallback case?

I'm hoping it's a bit carrot and stick where registries see both the value to supplying the feature to users, and where users insist on not doing this by tag, that motivates registries to eventually adopt. I am realistic in suspecting that many registries, especially self hosted ones, will use the fallback solution for many years to come.

Hey Chris, thanks for your comments! Safely rolling this out to registry implementations and clients has been a large and ongoing topic in the group even since before its inception.
To answer the questions:

How would one move an artifact and all its metadata from one registry to another?

As an example, let's say I'm moving registry.biz/my-user/my-app. While the registry only supports the fallback mode, a client recursively moving attachments would:

resolve that reference to its digest registry.biz/my-user/my-app@sha256:abcdef
list all tags for registry.biz/my-user/my-app and find any that match the form :sha256-abcdef.sbom (or :sha256-abcdef.0f0f0f.sbom using Prop E's form)
to find any attachments-on-attachments-etc, also resolve digests and list tags to find things attached to attachments, recursively until the tree is exhaustively discovered
copy all found images to the destination repository or registry

Under the non-fallback case, the tag list and filter would be replaced by a trip to the new /referrers API endpoint, which would return descriptors of attachments. Clients would still have to recursively call /referrers on attachments if they wanted to exhaustively discover the full tree.
Some versions of some proposals have proposed adding recursiveness to the /referrers API from the outset, but I think in general this hasn't made it to the latest versions of various proposals.
There's also been discussions about filtering /referrers so that you can discover only signatures, or only attestations, etc., but for the most part we've been trying to defer this to future proposals, since it becomes a pretty large topic.

What happens if the ecosystem does not broadly adopt the proposal and we're stuck in the fallback case?

To me, this is exactly why I think it's so important that we have a fallback case. Some proposals have basically required registries to adopt the new APIs in order to get this new behavior at all, which I think is just a non-starter for roughly the reasons you describe, around slow uptake.
If, N years from now, almost no registries have adopted the new APIs, then we're roughly in the same spot we're in today, with folks like cosign bolt on the tagging convention onto unchanged APIs. The benefit at least would be a bit more rigorous and thorough specification for the tagging convention, instead of just copying whatever cosign made up one day.
To @sudo-bmitch's point, as attaching things to things takes off, registry operators may find that they can avoid a lot of unnecessary tag list API calls by implementing the new APIs. Many registry instances will never update though, and that's why fallback is so important.

on copying across registries (Q1): This is less elegant than using an OCI index (Proposal D) that you can easily copy with existing tools. You could also write the OCI index out to disk using the OCI layout for air gapped use. This is another example of how fitting into the existing data model is beneficial.

We're seeing cosign use digest tags today, so existing tools that don't support this will end up stripping the signature off the image when copying it. That's not actually horrible to me, since the tooling doing the mirroring doesn't understand signatures, there's a good chance the tooling using the image on the mirror doesn't either. But if you want to copy an image with metadata, use a metadata aware image mirroring tool.
For the OCI layout, I've been pushing the other manifests in the layout with their digest tags, same as if I were copying between repositories that don't support the API.

But if you want to copy an image with metadata, use a metadata aware image mirroring tool.

That makes the proposed solution a lot less user-friendly, as not many user are aware of what that means, and they shouldn't have to be aware. Do folks agree that security-for-all would be a great goal?

That makes the proposed solution a lot less user-friendly, as not many user are aware of what that means, and they shouldn't have to be aware. In my understading primary use-cases are around signatures and other securoty-related metadata, not just some supplimentary metdata, in which case, I'd think that security-for-all would be a great goal. What do folks think?

It's hard to disagree with that sentiment, and I don't think anybody here would. 😄
Stuffing signatures into an index with the built image(s) would definitely work for signatures attached at build-time, and the built artifact could even be naively moved across registries using some* existing tools. But we still have the problem of appending more information to it, or updating information included in the index, without modifying the index's digest. That's fundamentally why we chose to explore moving attachments outside the referred-to thing. Once attachments are outside, tooling to move things has to expand to know about how to discover and move attachments.
*Notably, docker pull <src> && docker tag <src> <dst> && docker push <dst> still wouldn't naively work with this, unless the docker CLI learned to include attached signatures included in indexes when pulling/pushing. Today it would only pull the matching platform-specific image, and drop all the rest.

Closing since the outcome of these discussions resulted in Proposal F


Till

There are some health issues with the master branch.

Now the checks fail on any formatting problems 👍

Now the checks fail if clippy has anything at all to say 👍

should probably look at using a type for all these flags. Maybe a bitvector or something like that.


The polygon layer example in layer-browser needs some tweaking

The polygon layer in the example has the "extruded" and "wireframe" check box that doesn't work due to data provided. We should have some ways to disable or remove these two checkbox or at least explicitly tell our users not they are not effective.

Also, when clicking on the "extruded" box:

"extruded" and "wireframe" check box that doesn't work due to data provided. We should have some ways to disable or remove these two checkbox or at least explicitly tell our users not they are not effective

An option could be to fix the data, so that we can show the user the full capability of the layer.

Fixed in #424


Joint efforts?

Hi @asciidisco!
I was wondering if you wanted to join effort with @prigara, @trivikr and I with our tech-conferences organisation? Since it seems that we're trying to achieve the same goal, I thought that could be a good idea. 😄
Our goal is to have everything open source, crowd sourced and self maintained by the community. Our website is pretty well used so far and all the data are stored in JSON files.
Oh and I saw https://github.com/asciidisco/web-conferences-2018/issues/66 and people submit CFP end date along with the conferences which allows us to show them nicely => https://confs.tech/cfp/
Let me know what you think!

Hey @nimzco,
I´ve also seen your tweet about this, but I´ve been a bit busy in the past weeks.
So, sorry for the delay.
I´m not really sure if I will continue this list in 2019, mostly due to the fact that I´m a bit limited on spare time atm. & also, because I collected the initial data set as part of my research for conferences with open CFP's.
Nevertheless I would be happy to contribute to confs.tech, if I find the time.


TypeError: 'NoneType' not iterable

I apologize for creating what looks like an identical issue. This appears to be a different segment of code giving me the error.
See this screenshot:

I'm running into the same issue. Did you find a solution?


Failed to build nodejs app when binding the ApplicationInsights

What happened?
For nodejs app, during the detect pahse, the paketo-buildpacks/azure-application-insights didn't be detected

Build Configuration
when using legacy binding, the build failed.
Here is the build spec:

Here is the error logs:
Build reason(s): CONFIG
CONFIG:
- resources: {}
- source: {}
+ cnbBindings:
+ - metadataRef:
+     name: buildservices.default.builders.default.buildpackbindings.default.type.applicationinsights.metadata
+   name: 7default-7default-7default-1
+   secretRef:
+     name: buildservices.default.builders.default.buildpackbindings.default.type.applicationinsights.secret
+ resources:
+   limits:
+     cpu: "1"
+     memory: 1Gi
+   requests:
+     cpu: "0"
+     memory: "0"
+ source:
+   blob:
+     url: https://a83b1b7b0e514335b6fa70df.file.core.windows.net/2e688e5754b2413f9fee11449afbab42/resources/2022021704-f705c3f0-d14a-4fde-8452-c9eda50ed893?sv=2018-03-28&sr=f&sig=gQaEuVFT53m%2Feocd9ruCMsjV5CXXqYd%2BwEH1Hbf8rik%3D&se=2022-02-18T04%3A37%3A52Z&sp=r
Loading secrets for "acrda49c26bd8a84fd59.azurecr.io" from secret "docker-kpack"
Downloading a83b1b7b0e514335b6fa70df.file.core.windows.net/2e688e5754b2413f9fee11449afbab42/resources/2022021704-f705c3f0-d14a-4fde-8452-c9eda50ed893...
Successfully downloaded a83b1b7b0e514335b6fa70df.file.core.windows.net/2e688e5754b2413f9fee11449afbab42/resources/2022021704-f705c3f0-d14a-4fde-8452-c9eda50ed893 in path "/workspace"
Previous image with name "acrda49c26bd8a84fd59.azurecr.io/build-service-result-image-2e688e5754b2413f9fee11449afbab42-default-app1:result-1" not found
6 of 17 buildpacks participating
paketo-buildpacks/ca-certificates 2.4.1
tanzu-buildpacks/node-engine      0.6.0
tanzu-buildpacks/yarn             0.2.1
tanzu-buildpacks/yarn-install     0.2.1
tanzu-buildpacks/node-module-bom  0.1.1
tanzu-buildpacks/yarn-start       0.2.0

Paketo CA Certificates Buildpack 2.4.1
https://github.com/paketo-buildpacks/ca-certificates
Launch Helper: Contributing to layer
    Creating /layers/paketo-buildpacks_ca-certificates/helper/exec.d/ca-certificates-helper
Tanzu Node Engine Buildpack 0.6.0
Resolving Node Engine version
Candidate version sources (in priority order):
-> ""
 -> ""
Selected Node Engine version (using ): 14.17.6

Executing build process
Installing Node Engine 14.17.6
failure checking for dependency mappings: couldn't read binding type: open /platform/bindings/7default-7default-7default-1/type: no such file or directory
ERROR: failed to build: exit status 1
when using the servicebinding, the build succeeded but paketo-buildpacks/azure-application-insights didn't participate to the build.
Here is the build spec:

In both situations, the paketo-buildpacks/azure-application-insights didn't be detected.

What platform (pack, kpack, tekton buildpacks plugin, etc.) are you
using? Please include a version.
kpack0.5.0

What buildpacks are you using? Please include versions.
tanzu-buildpacks/java-azure 7.3.0
tanzu-buildpacks/nodejs 1.11.0

What builder are you using? If custom, can you provide the output from pack inspect-builder <builder>?

Can you provide a sample app or relevant configuration (buildpack.yml,
nginx.conf, etc.)?
https://github.com/digitalocean/sample-nodejs

Checklist

[ ] I have included log output.
[ ] The log output includes an error message.
[ ] I have included steps for reproduction.

@dmikusa-pivotal  Thanks for the quick reply. We follow the instructions for how to set up your K8s yaml to make this work here. Actually, Incarnation-p-lee is my colleague. When I changed the source to Java project in image.yaml, the other settings remain the same, the paketo-buildpacks/azure-application-insights can be detected. With the same binding settings, it can be detected in Java but not in Nodejs. It seems the detection logic didn't work in Nodejs

OK, thanks for trying that. I did a little more looking here.

The Paketo nodejs does not have Azure App Insights buildpack in it. That means it's not going to run unless you specifically include that buildpack.

I recently heard about this from another user: failure checking for dependency mappings: couldn't read binding type: open /platform/bindings/7default-7default-7default-1/type: no such file or directory. I think something changed recently on the Node.js side. I don't know though, as I'm not involved with that particular buildpack. Just a guess, but you might be able to work around this by removing your dependency mappings (I'm assuming that you have them). Maybe worth a shot.

#2 is outside of this buildpack though, so you'd probably be better served asking a question over on https://github.com/paketo-buildpacks/nodejs.

I'm going to close this out since it's been a little while. Feel free to reopen if you're still hitting issues.


Run-TestsInBCContainer fails with error  UnhandledException : Object reference not set to an instance of an object.

Describe the issue
Running standard test codeunit in Cronus database with Run-TestsInBCContainer fails with following error:
Codeunit 138696 Page Troubleshooting Test UnhandledException : Object reference not set to an instance of an object.
at , C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\ClientContext.ps1: line 117
at , : line 76
Current Interaction: Microsoft.Dynamics.Framework.UI.Client.Interactions.InvokeActionInteraction
Time spend: 3 seconds
at AwaitState, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\ClientContext.ps1: line 217
at InvokeInteraction, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\ClientContext.ps1: line 236
at InvokeAction, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\ClientContext.ps1: line 460
at Run-Tests, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\PsTestFunctions.ps1: line 809
at , : line 57
It works when executing the tests from the web client.
After the run-testsinbccontainer restarts the instance. There is Tenant Profile table locked for some time and web client cannot be used.
I also experienced this error with Codeunit 138697 Profile Import/Export Test. Same error: UnhandledException : Object reference not set to an instance of an object.
Scripts used to create container and cause the issue
import-module bccontainerhelper

New-BCContainer -containerName bc17-de-temp `
                -accept_eula -accept_outdated `
                -artifacturl "https://bcartifacts.azureedge.net/onprem/17.1.18256.18792/de" `
                -licenseFile "<path to license file>" `
                -updateHosts `
                -useBestContainerOS `
                -useGenericImage "mcr.microsoft.com/dynamicsnav:10.0.14393.3630-generic" `
                -includeTestToolkit `
                -auth UserPassword

Add the codeunit into the test tool manually from the web client
import-module bccontainerhelper

Run-TestsInBcContainer -containerName bc17-de-temp `
-companyName "Cronus AG" `
-credential (Get-Credential) `
-testSuite "DEFAULT" `
-testCodeunit 138696

Full output of scripts
{
    "container.logs":  [
                           "Using artifactUrl https://bcartifacts.azureedge.net/onprem/17.1.18256.18792/de",
                           "Using installer from C:\\Run\\150-new",
                           "Installing Business Central",
                           "Installing from artifacts",
                           "Starting Local SQL Server",
                           "Starting Internet Information Server",
                           "Copying Service Tier Files",
                           "Copying PowerShell Scripts",
                           "Copying dependencies",
                           "Copying ReportBuilder",
                           "Importing PowerShell Modules",
                           "Determining Database Collation from c:\\dl\\onprem\\17.1.18256.18792\\de\\database\\Demo Database NAV (17-0).bak",
                           "Changing Database Server Collation to Latin1_General_100_CI_AS",
                           "Restoring CRONUS Demo Database",
                           "Modifying Business Central Service Tier Config File for Docker",
                           "Creating Business Central Service Tier",
                           "Installing SIP crypto provider: \u0027C:\\Windows\\System32\\NavSip.dll\u0027",
                           "Copying Web Client Files",
                           "Copying Client Files",
                           "Copying ModernDev Files",
                           "Copying additional files",
                           "Copying ConfigurationPackages",
                           "Copying Test Assemblies",
                           "Copying Applications",
                           "Starting Business Central Service Tier",
                           "Importing license file",
                           "Stopping Business Central Service Tier",
                           "Installation took 151 seconds",
                           "Installation complete",
                           "Initializing...",
                           "Setting host.containerhelper.internal to <IP_ADDRESS> in container hosts file",
                           "Starting Container",
                           "Hostname is bc17-de-temp",
                           "PublicDnsName is bc17-de-temp",
                           "Using NavUserPassword Authentication",
                           "Creating Self Signed Certificate",
                           "Self Signed Certificate Thumbprint 2319BE8F2483026200D87FA5884E51CAEA20A493",
                           "Modifying Service Tier Config File with Instance Specific Settings",
                           "Starting Service Tier",
                           "Registering event sources",
                           "Creating DotNetCore Web Server Instance",
                           "Using license file \u0027c:\\run\\my\\license.flf\u0027",
                           "Import License",
                           "Creating http download site",
                           "Setting SA Password and enabling SA",
                           "Creating tasane as SQL User and add to sysadmin",
                           "Creating SUPER user",
                           "Container IP Address: <IP_ADDRESS>",
                           "Container Hostname  : bc17-de-temp",
                           "Container Dns Name  : bc17-de-temp",
                           "Web Client          : http://bc17-de-temp/BC/",
                           "Dev. Server         : http://bc17-de-temp",
                           "Dev. ServerInstance : BC",
                           "Setting bc17-de-temp to <IP_ADDRESS> in host hosts file",
                           "",
                           "Files:",
                           "http://bc17-de-temp:8080/ALLanguage.vsix",
                           "",
                           "WARNING: You are running a container which is 213 days old.",
                           "Microsoft recommends that you always run the latest version of our containers.",
                           "",
                           "Initialization took 39 seconds",
                           "Ready for connections!"
                       ],
    "docker.info":  [
                        "Client:",
                        " Debug Mode: false",
                        " Plugins:",
                        "  cluster: Manage Docker clusters (Docker Inc., v1.2.0)",
                        "",
                        "Server:",
                        " Containers: 7",
                        "  Running: 1",
                        "  Paused: 0",
                        "  Stopped: 6",
                        " Images: 5",
                        " Server Version: 19.03.5",
                        " Storage Driver: windowsfilter",
                        "  Windows: ",
                        " Logging Driver: json-file",
                        " Plugins:",
                        "  Volume: local",
                        "  Network: ics internal l2bridge l2tunnel nat null overlay private transparent",
                        "  Log: awslogs etwlogs fluentd gcplogs gelf json-file local logentries splunk syslog",
                        " Swarm: inactive",
                        " Default Isolation: process",
                        " Kernel Version: 10.0 14393 (14393.3564.amd64fre.rs1_release.200303-1942)",
                        " Operating System: Windows Server 2016 Datacenter Version 1607 (OS Build 14393.3564)",
                        " OSType: windows",
                        " Architecture: x86_64",
                        " CPUs: 4",
                        " Total Memory: 64GiB",
                        " Name: RDES-FIN-DEV02",
                        " ID: MIQJ:YOEV:FAXO:UHUF:FQHQ:7YQA:N6DY:GOLD:2MBU:3IVU:AJ6V:5YX3",
                        " Docker Root Dir: D:\\ProgramData\\Docker",
                        " Debug Mode: false",
                        " Registry: https://index.docker.io/v1/",
                        " Labels:",
                        " Experimental: false",
                        "  <IP_ADDRESS>/8",
                        " Live Restore Enabled: false",
                        ""
                    ],
    "container.labels":  {
                             "country":  "de",
                             "created":  "202008281422",
                             "eula":  "https://go.microsoft.com/fwlink/?linkid=861843",
                             "maintainer":  "Dynamics SMB",
                             "nav":  "",
                             "osversion":  "10.0.14393.3630",
                             "platform":  "17.0.18204.18738",
                             "tag":  "<IP_ADDRESS>",
                             "version":  "17.1.18256.18792"
                         },
    "container.ping":  [
                           "",
                           "Pinging bc17-de-temp [<IP_ADDRESS>] with 32 bytes of data:",
                           "Reply from <IP_ADDRESS>: bytes=32 time\u003c1ms TTL=128",
                           "Reply from <IP_ADDRESS>: bytes=32 time\u003c1ms TTL=128",
                           "Reply from <IP_ADDRESS>: bytes=32 time\u003c1ms TTL=128",
                           "Reply from <IP_ADDRESS>: bytes=32 time\u003c1ms TTL=128",
                           "",
                           "Ping statistics for <IP_ADDRESS>:",
                           "    Packets: Sent = 4, Received = 4, Lost = 0 (0% loss),",
                           "Approximate round trip times in milli-seconds:",
                           "    Minimum = 0ms, Maximum = 0ms, Average = 0ms"
                       ],
    "container.env":  [
                          "auth=NavUserPassword",
                          "ExitOnError=N",
                          "accept_eula=Y",
                          "accept_outdated=Y",
                          "passwordKeyFile=c:\\run\\my\\aes.key",
                          "artifactUrl=https://bcartifacts.azureedge.net/onprem/17.1.18256.18792/de",
                          "username=tasane",
                          "locale=de-DE",
                          "securePassword=\u003cspecified\u003e",
                          "databaseServer=",
                          "enableApiServices=Y",
                          "licenseFile=\u003cspecified\u003e",
                          "databaseInstance=",
                          "useSSL=N",
                          "removePasswordKeyFile=Y",
                          "COMPLUS_NGenProtectedProcess_FeatureEnabled=0"
                      ]
}

Additional context

It happens all the time
It did work once in the middle of december 2020 - i had a full test run, where this error did not pop up

I just tested with the latest version of onprem de using following artifacts url:
https://bcartifacts.azureedge.net/onprem/17.3.20469.20605/de
Same issue.

I just tested with the latest version of onprem de using following artifacts url:
https://bcartifacts.azureedge.net/onprem/17.3.20469.20605/de
Same issue.

bccontainerhelper version is 1.0.19

bccontainerhelper version is 1.0.19

The best way of running tests in 17 is to use run-tests with -extensionid - once for every app.
That should work.
Get-BcContainerAppInfo | % { run-tests -extensionid $_.appid }
or something like that.

The best way of running tests in 17 is to use run-tests with -extensionid - once for every app.
That should work.
Get-BcContainerAppInfo | % { run-tests -extensionid $_.appid }
or something like that.

What is the main difference between executing tests in this way compared to the old way?
Do we need to consider something special when developing our own tests apps to make it possible to execute the tests with the new way?

What is the main difference between executing tests in this way compared to the old way?
Do we need to consider something special when developing our own tests apps to make it possible to execute the tests with the new way?

No change to the tests.
you don't have to populate the test suite anymore
it is more stable - we know there are issues with the old way when running very large amounts of tests - it is just very hard to find and fix (especially when a working alternative exists)

No change to the tests.
you don't have to populate the test suite anymore
it is more stable - we know there are issues with the old way when running very large amounts of tests - it is just very hard to find and fix (especially when a working alternative exists)

Is there a way to filter on test codeunits with the new way? To pick some specific tests codeunits or exclude some.

Is there a way to filter on test codeunits with the new way? To pick some specific tests codeunits or exclude some.

You run tests one app at a time.
Disabledtests excludes tests you don't want to run.
You can find samples on howto on my blog.

You run tests one app at a time.
Disabledtests excludes tests you don't want to run.
You can find samples on howto on my blog.

i will test the new way and report back.

i will test the new way and report back.

I ran following command against the same container:
Get-BcContainerAppInfo -containerName bc17-de-temp | % {Run-TestsInBcContainer -containerName bc17-de-temp -extensionId $_.appid -credential $credentials }

and now it failed with following message:
Codeunit 138500 Common Demodata Failure (0.151 seconds)
    Testfunction ConfigTemplateCodeShouldStartWithTablePrefix Failure (0.016 seconds)
    Testfunction PaymentMethodsWithUseForInvoicing Failure (0.01 seconds)
UnhandledException : Object reference not set to an instance of an object.
at <ScriptBlock>, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\ClientContext.ps1: line 117
at <ScriptBlock>, <No file>: line 76
Current Interaction: Microsoft.Dynamics.Framework.UI.Client.Interactions.InvokeActionInteraction
Time spend: 11 seconds
at AwaitState, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\ClientContext.ps1: line 217
at InvokeInteraction, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\ClientContext.ps1: line 236
at InvokeAction, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\ClientContext.ps1: line 460
at Run-Tests, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\PsTestFunctions.ps1: line 447
at <ScriptBlock>, <No file>: line 57
ClientSession State is InError (Wait time 11 seconds)****

I ran following command against the same container:
Get-BcContainerAppInfo -containerName bc17-de-temp | % {Run-TestsInBcContainer -containerName bc17-de-temp -extensionId $_.appid -credential $credentials }

and now it failed with following message:
Codeunit 138500 Common Demodata Failure (0.151 seconds)
    Testfunction ConfigTemplateCodeShouldStartWithTablePrefix Failure (0.016 seconds)
    Testfunction PaymentMethodsWithUseForInvoicing Failure (0.01 seconds)
UnhandledException : Object reference not set to an instance of an object.
at <ScriptBlock>, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\ClientContext.ps1: line 117
at <ScriptBlock>, <No file>: line 76
Current Interaction: Microsoft.Dynamics.Framework.UI.Client.Interactions.InvokeActionInteraction
Time spend: 11 seconds
at AwaitState, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\ClientContext.ps1: line 217
at InvokeInteraction, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\ClientContext.ps1: line 236
at InvokeAction, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\ClientContext.ps1: line 460
at Run-Tests, C:\ProgramData\BcContainerHelper\Extensions\bc17-de-temp\PsTestTool\PsTestFunctions.ps1: line 447
at <ScriptBlock>, <No file>: line 57
ClientSession State is InError (Wait time 11 seconds)****

Could you paste in the full script end2end causing this issue?
thanks

Could you paste in the full script end2end causing this issue?
thanks

This time I used 17.3 onprem de container. Artifact url is in the following script I used to create the container:
import-module bccontainerhelper

New-BCContainer -containerName bc17-de-temp `
                -accept_eula -accept_outdated `
                -artifacturl "https://bcartifacts.azureedge.net/onprem/17.3.20469.20605/de" `
                -licenseFile "D:\Licenses\RIB Cosinus GmbH BC 17 11.12.2020.flf" `
                -updateHosts `
                -useBestContainerOS `
                -useGenericImage "mcr.microsoft.com/dynamicsnav:10.0.14393.3630-generic" `
                -includeTestToolkit `
                -auth UserPassword

Then I just ran the following commands:
$credentials=(Get-Credential)
Get-BcContainerAppInfo -containerName bc17-de-temp | % {Run-TestsInBcContainer -containerName bc17-de-temp -extensionId $_.appid -credential $credentials }

This time I used 17.3 onprem de container. Artifact url is in the following script I used to create the container:
import-module bccontainerhelper

New-BCContainer -containerName bc17-de-temp `
                -accept_eula -accept_outdated `
                -artifacturl "https://bcartifacts.azureedge.net/onprem/17.3.20469.20605/de" `
                -licenseFile "D:\Licenses\RIB Cosinus GmbH BC 17 11.12.2020.flf" `
                -updateHosts `
                -useBestContainerOS `
                -useGenericImage "mcr.microsoft.com/dynamicsnav:10.0.14393.3630-generic" `
                -includeTestToolkit `
                -auth UserPassword

Then I just ran the following commands:
$credentials=(Get-Credential)
Get-BcContainerAppInfo -containerName bc17-de-temp | % {Run-TestsInBcContainer -containerName bc17-de-temp -extensionId $_.appid -credential $credentials }

You should remove these parameters:
            -useBestContainerOS `
            -useGenericImage "mcr.microsoft.com/dynamicsnav:10.0.14393.3630-generic" `

Artifacts are always running best container os.
The generic image you point out is very old.
I don't know whether this is the solution, but it could be.
I will try to run a repro here.

You should remove these parameters:
            -useBestContainerOS `
            -useGenericImage "mcr.microsoft.com/dynamicsnav:10.0.14393.3630-generic" `

Artifacts are always running best container os.
The generic image you point out is very old.
I don't know whether this is the solution, but it could be.
I will try to run a repro here.

we need sql 2017. Can I enforce it somehow?

we need sql 2017. Can I enforce it somehow?

-useGenericImage "$(GetBestGenericImageName)-sql2017"

-useGenericImage "$(GetBestGenericImageName)-sql2017"

I tried this with de and get an error like you.
When running w1 it works.
Not sure whether this can be related to some specific tests in the german version but I will have to investigate that.

I tried this with de and get an error like you.
When running w1 it works.
Not sure whether this can be related to some specific tests in the german version but I will have to investigate that.

You can use:
$disabledTests = @'
[
{
"codeunitName": "Common Demodata",
"method" : "*"
}
]
'@
and add -disabledTests ($disabledTests | ConvertFrom-Json) to your run-tests, then it will disable all tests from that codeunit and the German tests can be run.
Maybe you can even find the method that is failing and disable that.
I do not know what the problem is, but I will not have the time to dive into the German tests and why they are failing here.

You can use:
$disabledTests = @'
[
{
"codeunitName": "Common Demodata",
"method" : "*"
}
]
'@
and add -disabledTests ($disabledTests | ConvertFrom-Json) to your run-tests, then it will disable all tests from that codeunit and the German tests can be run.
Maybe you can even find the method that is failing and disable that.
I do not know what the problem is, but I will not have the time to dive into the German tests and why they are failing here.

If anybody can figure out why this specific test codeunit isn't running in docker, I would be happy to take a PR or add a fix.
thanks

If anybody can figure out why this specific test codeunit isn't running in docker, I would be happy to take a PR or add a fix.
thanks

We have to deal with this (or somekind of similar) issue for quite a long time.
We figured out that our tests fail (with the same error as above) if we are posting an Item Journal (to get stock for our integration tests), but never could isolate the problem (gave up after one year...).
Never thought about our German environment.
We solved the problem by running the tests outside the container:
building up a ClientContext and running the Run-Tests routine.
At least the Get-TestsFromBCContainer function is working pretty well.


Add email notification functionality

@kburke
We have the need to send an email to the help desk when the "submit" button is pushed on Uploads or Datasets in the Ingest UI.  Currently these buttons are wired to the ingest-ui notify-slack endpoint.
Add an option to send an email via this endpoint, if the optional parameter send_to_email is included additionally send the same message that is sent to Slack via email to the address specified in send_to_email.  The other parameters needed to send the email:

MAIL_SERVER
MAIL_PORT
MAIL_USE_TLS
MAIL_USERNAME
MAIL_PASSWORD
MAIL_DEFAULT_SENDER
MAIL_DEBUG

should be included via in the standard app.cfg file.
The Slack notification functionality should remain the same (and always sent), with the new email notification functionality being optional. Additionally, change the name of the endpoint from notify-slack to notify, coordinate with @BirdMachine (Jes) to release a version of Ingest UI at the same time that supports the new endpoint name.

I have modified the response when the renamed endpoint is successful in both posting a Slack notification and sending an email to include summary for each, as below.
I'm not sure if this is relevant to @BirdMachine for UI revisions.
{
    "Email": {
        "email_recipient_list"<EMAIL_ADDRESS>        "message": "notification posting from Postman by KB at Fri Jun 23 2023 10:43:22 GMT-0400 (Eastern Daylight Time).",
        "user_email"<EMAIL_ADDRESS>        "user_name": "Karl Burke"
    },
    "Slack": {
        "channel": "data-testing-notifications",
        "message": "notification posting from Postman by KB at Fri Jun 23 2023 10:43:22 GMT-0400 (Eastern Daylight Time).",
        "user_email"<EMAIL_ADDRESS>        "user_name": "Karl Burke"
    }
}

See Comment on PR about using Dev for UI revisions, then coordinating with others on Test.


bower.json bundles angular and has it as a depedency

This commit https://github.com/driftyco/ionic/commit/bfd9f083b35542a09e655a64b33d5cd130a2ba2a altered bower.json so there is only one .js file. This means that angular is now loaded into the project twice making it complain and breaking dependency resolution.
It doesn't alter anything else, just needs reverted.

Greetings @john-banks!
My sensors indicate that you need to update your issue through our custom issue form. We are now requiring all issues to be submitted this way, to ensure that we have all of the information necessary to fix them as quickly as possible.
Click Here To Update Your Issue
I will have no choice but to close this issue if it is not resubmitted through the form. Please fill out the rest of the form, so that I may use my friendly robot powers to assist you.
Thank you!

Sounds like we should just remove angular as a dependency for our bower package, since we bundle a specific version in anyways?

That doesn't work as other angular modules you want to bring in will have angular as a dependency and force you to bring in another version anyway.
The dependency is already set at  "angular": "1.4.3" rather than  "angular": "~1.4.3" or  "angular": ">=1.4", meaning you are being forceful (perhaps overly so) about which version you want  used.

I don't see how the first commit would break it, since it would still be bringing in a non-versioned instance of Angular, right?

+1, had to revert to ionic 1.1 because of this

A few points for and against.
For: It's preferred to load all of Ionic as one file. Each Ionic version is tightly coupled with specific versions of Angular, and mixing versions is discouraged and unsupported. This forces that, and makes things more efficient at the same time. It may be necessary in the future to ship Ionic with a patched version of the Angular core (unlikely, but possible).
Against: Other libs and addons might require Angular too. This means Angular gets loaded twice.
I suggest we revert bfd9f083b35542a09e655a64b33d5cd130a2ba2a and maintain strict dependency settings. Any reason this is a bad idea?
A side note: We're moving away from bower in general towards NPM. Bower's still supported in v1, but v2 is all NPM.

It's preferred to load all of Ionic as one file. Each Ionic version is tightly coupled with specific versions of Angular, and mixing versions is discouraged and unsupported. This forces that, and makes things more efficient at the same time.

In that case the bundled file acts like something provided by CDN not as a bower component. As it stands currently https://github.com/driftyco/ionic/commit/bfd9f083b35542a09e655a64b33d5cd130a2ba2a will break other implementation (mine included). Because you specify a specific version, if you try and use another bower will warn you that ionic is on another and forces the developer to override. In that case your job is done and any mistakes are their own.
I'm interested in the move towards npm as I really dislike how over simplistic bower is yet npm seems intrinsically slow. npm behaves very differently in that dependencies are installed against themselves rather than with bower where they are installed into one directory and resolved into the agreed version. Not sure how that can be managed with this current situation.

Sounds like just reverting the commit is a solution for the short term. In the long term we are moving away from bower so I see no need to innovate on it

@john-banks regarding npm vs bower deps<EMAIL_ADDRESS>makes the dep tree as flat as possible, and nests only in case of conflicts.

I reverted commit c9bd3e8f6d940ba328cbe630f6f2bc3b3f77bbf2. Seeing as this commit was 25 days ago, I see no reason to innovate on our bower strategy that was working fine (not great, just fine). In the future we are killing off bower so let's keep it like this for now and get it right (no bundling angular) when we move fully to npm.


Remove Starter tile from default toolbar [#183062205]

👍
Is there a sample doc where this is turned on?

Yes, in the example curriculum.


How is it possible to localize "Close", "Clear" & "Apply" buttons and input string from..to

Hello! Thank you for your package. I found some functions in config, which can help me to work on international version.
So, here we are:

monthFormatter - months
predefinedRanges - ranges
weekdayFormatter - week days

What I miss:

action buttons at the bottom
text in input, for example: from 2020-11-12 to 2020-11-21 and I need DD/MM/YYYY - DD/MM/YYYY

Any suggestions?

https://github.com/allo-media/elm-daterange-picker/pull/16

Fixed in #16 and released under a new package namespace here https://package.elm-lang.org/packages/n1k0/elm-daterange-picker/latest/


Fix bug when using Credentials for initialization

When initializing the Nexmo instance using Rails Credentials, if you did not provide a path to a private_key than the application would fail on a nil error for the #exists? check, since there was no string provided. This PR fixes that issue and bumps up the version to 0.4.2.

@benhayehudi Can we add a test for this to prove that it's fixed?


#bootiny

Add command #bootiny such that upon passing the existing commander is booted and person proposing the #bootiny is given command.
the unseat3ed commander should have a boot message displayed reading: "You have been Mutinied and Booted by your team"
should follow similar structure to #mutiny, in that commander cannot see the vote and 50%+ yes to pass
consequences of a FAILED #bootiny are that person proposing #bootiny is booted to the lobby with the message:
"You have chosen poorly, the team disagrees"
Imported from trac ticket #124,  created by PsycHosis on 07-08-2010 at 01:58, last modified: 02-25-2012 at 15:33

bootiny should require a 2/3 majority!
Trac comment by lexaal on 07-08-2010 at 02:10

no squad bashing on trac, make an account and try to keep post factual -
anyways changing the ballot system, which all votes currently use, would be non trivial and would deserve it's own ticket.
Trac comment by Imago on 07-14-2010 at 09:01

r679, needs your testing
Trac comment by Imago on 07-08-2010 at 02:37

Removed from source code, producing to much drama.
Trac comment by pkk on 02-25-2012 at 15:33


Add files to solve NPM 10 lib/src strip problem

Since NPM 10, the lib/src dir is being stripped out. This doesn't happen on NPM 9.
This causes the following error:
Cannot find module './src/utils/buildXmlReport'
   Require stack:
   -<EMAIL_ADDRESS>   - /path/to/node_modules/jest-util/build/requireOrImportModule.js
   - /path/to/node_modules/jest-util/build/index.js
   - /path/to/node_modules/jest-config/build/getCacheDirectory.js
   - /path/to/node_modules/jest-config/build/Defaults.js
   - /path/to/node_modules/jest-config/build/normalize.js
   - /path/to/node_modules/jest-config/build/index.js

Apologies for the delays real life took over. Will be taking a look ASAP (within the week) and cutting a new release if everything working as intended

Thank you for your contribution! PR is merged and release 2.4.0 is now available containing the changes

Brilliant!


How about creating .gitignore when repo is init?

Thank you for your amazing work!
It would be great if .gitignore and README.md with an installation guide would be included in the initialized repo.
e.g.,
# .gitignore
.wrangler
node_modules

.gitignore should be available in the latest version.
Please try running
npx notehost@latest init


#55 Ignore usage in Interface builder doumentation

Thanks for this PR, I've had a cursory look, looks good in general. But I'd like to integrate more naturally into the existing sections text. I'll have a look at that soon and merge. Thanks again! 👍

I merged this with some commit renamed in 7b18a06b244c88701c270af5e1981838ca951ae0.


Add working steps in Skewness Calculator

Proposal
Working steps of the skewness calculator are not present. I would like to add them.

@Ananya-Misra  issue already exists


ORC-844: Improve hashCode Methods

What changes were proposed in this pull request?
Auto-generate some hashCode methods.
Why are the changes needed?
Better hash code values for better distributions; less code.
How was this patch tested?
No changes to functionality. Use existing unit tests.

Thank you, @wgtmac . Merged to main for Apache ORC 1.8.0.


Private attribute values are retrieved differently in instance or class methods

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Hi ! Thanks a lot for Pydantic
There is a difference in the way PrivateAttr is treated between instance methods and class methods.

instance methods access private attributes seamlessly. On the minimal example _float can be casted to float
class methods don't work with the same piece of code. _float is now retrieved as a PrivateAttr, and the value must be accessed through PrivateAttr.default

I wouldn't expect that behavior, but i must admit i don't know Pydantic that well
Example Code
from pydantic import BaseModel

class Model(BaseModel):
    _float = 2.1

    @classmethod
    def f(cls) -> None:
        float(cls._float)  # Failing

    def g(self) -> None:
        float(self._float)  # Passes

Model().g()  # Passes
Model.f()  # Failing

Python, Pydantic & OS Version
pydantic version: 2.9.2
        pydantic-core version: 2.23.4
          pydantic-core build: profile=release pgo=false
                 install path: C:\Users\XXX\.venv\Lib\site-packages\pydantic
               python version: 3.12.2 (tags/v3.12.2:6abddd9, Feb  6 2024, 21:26:36) [MSC v.1937 64 bit (AMD64)]
                     platform: Windows-11-10.0.22631-SP0
             related packages: typing_extensions-4.12.2
                       commit: unknown

https://docs.pydantic.dev/latest/concepts/models/#class-vars
https://docs.python.org/3/library/typing.html#typing.ClassVar

In [7]: from typing import ClassVar
   ...: from pydantic import BaseModel
   ...: class Model(BaseModel):
   ...:     _float:ClassVar[float] = 2.1
   ...:
   ...:     @classmethod
   ...:     def f(cls) -> float:
   ...:         return float(cls._float)  # Failing
   ...:
   ...:     def g(self) -> float:
   ...:         return(self._float)  # Passes
   ...:
   ...:
   ...: Model().g(), Model.f()
Out[7]: (2.1, 2.1)

Note that there's some sharp edges when trying to combine ClassVar with Field.

Oh great, wasn't aware of this one. Thanks a lot

Hi @ldouteau,
You can use a ClassVar annotation if you'd like to be able to access _float from the cls seamlessly:
from pydantic import BaseModel
from typing import ClassVar

class Model(BaseModel):
    _float: ClassVar = 2.1

    @classmethod
    def f(cls) -> None:
        float(cls._float)  # Failing

    def g(self) -> None:
        float(self._float)  # Passes

Model().g()  # Passes
Model.f()  # Failing

Going to close as resolved for now. Basically, it's inferred that _float is an instance attribute unless you specify otherwise.


Prevent ErrorException when value is a string

Hello,
When $value is a string, an ErrorException is thrown with the following message:
foreach() argument must be of type array|object, string given

This PR prevents that exception by checking if $value is an array, as expected.

Hi @scaytrase
I am thinking about using is_iterable instead of is_array. WDYT?

Sounds great

@scaytrase done :)
Thank you for the review!

Thanks @rogervila !


feat: replace openai with llmhub

Summary
This PR integrates LLMHub. This integration allows users to get started right away (1-click!), and automatically integrates all relevant LLM providers without requiring users to set up API keys with each. Additionally we provide various tools to make it easier to improve its accuracy/latency/costs (via tools for finetuning & prompt engineering).
A user would only need to do the following to get started:
pip3 install llmhub
# llmhub was globally installed as part of previous command.
llmhub auth

llmhub auth points users to a webpage where they need to sign in via GitHub. That's it!
Features
:massage: Allow users to use your app in 1-click.
:computer: Deploy in 1-click.
:haircut: Integrate ANY LLM in 3-lines with a standardised API.
:wrench: Monitor and improve your prompts.
:earth_americas: Discover new prompts easily, and showcase your work.
:muscle: Contribute to popular prompts, and improve your skills.
:dollar: Free credits while we're in beta
Some things to note

The current prompts are deployed to be an exact replica of existing code (models & parameters like temperature/frequency_penalty). Easy to edit.
We'd love to keep this LLM work public/open-source. Is that OK?

About Us
We're @vatsalaggarwal and @sidroopdaska. @sidroopdaska was the second engineering hire at the largest autonomous driving company in Europe, and I was an ML Scientist at Amazon for 4 years and researched deep generative models!

Hi, thank you for adding the changes. However I thought it would be optional to use LLMHub, the current version requires it. Could you edit this pull request and the one for https://github.com/tom-doerr/fix/pull/3 to make it optional? Otherwise it would break existing installations and make it incompatible with my other Codex projects that read the API keys from the same location.


chore: release 2.0.5

:robot: I have created a release *beep* *boop*
2.0.5 (2019-07-29)
Bug Fixes

deps: update dependency google-auth-library to v5 (#453) (755635c)

This PR was generated with Release Please.

Release is at https://github.com/googleapis/nodejs-common/releases/tag/v2.0.5

The release build has started, the log can be viewed here. :sunflower:

:egg: You hatched a release! The release build finished successfully! :purple_heart:


[Event Request] Codeunit 80 Sales Post - OnBeforeSetAmountsForBalancingEntry

Hi, would it be possible to modify the existing event OnBeforeSetAmountsForBalancingEntry to add a parameter, like this :
    local procedure SetAmountsForBalancingEntry(CustLedgEntry: Record "Cust. Ledger Entry"; TotalSalesLine2: Record "Sales Line"; TotalSalesLineLCY2: Record "Sales Line"; var GenJnlLine: Record "Gen. Journal Line")
    var
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeSetAmountsForBalancingEntry(CustLedgEntry, GenJnlLine, TotalSalesLineLCY2, IsHandled);
        if IsHandled then
            exit;

    [IntegrationEvent(false, false)]
    local procedure OnBeforeSetAmountsForBalancingEntry(var CustLedgEntry: Record "Cust. Ledger Entry"; var GenJnlLine: Record "Gen. Journal Line"; var TotalSalesLineLCY2: Record "Sales Line" ; var IsHandled: Boolean)
    begin
    end;

?
Thanks

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


Revert "pull: use homebrew remote if HOMEBREW_FORCE_HOMEBREW_ON_LINUX"

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

 See PR 7076 and https://github.com/Homebrew/brew/pull/7076#pullrequestreview-363749368 for context as to why the second commit.
The JENKINS_HOME envvar appears to be unset on the Linux Jenkins master that holds the BinTray credentials.
It's bed time, we're too tired to be fixing forward!

The JENKINS_HOME envvar appears to be unset on the Linux Jenkins master that holds the BinTray credentials.

We could always add another env if need be.

Oh, I saw it was already suggested on Slack - sorry, should have read there first!


Improve cross-account role handling

Correctly map additional_permissions to a role as this was missing
Insert extra roles to be assumed directly into policy document as a variable

At present this PR gets unhappy if no extra role ARNs are supplied. I'll look at making the statement block dynamic dependent on the presence of those extra roles.


The map in dungeons is not really good.

[ ] I have checked the known issues and searched Github to ensure this issue has not already been reported.
[ ] This is not a feature request, I know those belong on Discord/Github Discussions.

Description of the bug:
The map in dungeons is not really good.
Can you reproduce? If so please list the steps:

[ ] No, I cannot reproduce this bug

Expected behavior:
The blue square should be filling out only the room and not the whole square.
Screenshots or videos:

System Information:

OS: Windows 11
GFX Backend: DirectX
Game Version: 1.1
Rom Version: 1.0 USA

Thank you for the report. We already have an issue tracking the pause dungeon maps here https://github.com/HarbourMasters/2ship2harkinian/issues/409
I've added your image to it as an example of how it looks now. I'll be closing this now as a duplicate of it.


Lost entries

Hi,
Yesterday I deployed a new WikiJs through Heroku.
Everything was fine (4 entries so far, 4 users)
For some reason today when I entered again, all the entries from entries table were removed. Just that table. Users remain there, sessions remain there, and I am not sure what is bruteforces but I have 4 entries there.
Any idea why my entries table were removed not after 12 hours??

I just entered again with just 1 page created, and it's removed again. It seems each time heroku puts to sleep my server, the table entries get removed

I tried to deploy to another Heroku server. Realized from MongoDB that the entries are not removed. Here is what I found. If I check the DB on Heroku (mLab addon), entries are there.
However, when I log in, the intro Page ( welcome to the wiki...) comes up, and removes all my entries automatically!
Any idea why it removes my entries after login??

The free plan of Heroku deletes your instance after 30 mins of inactivity.
If you have the sync setup and you come back to the site after 30 mins inactivity, you need to wait a min or 2 for the sync to occurs and re-download the files, which is why you see the "Welcome to the wiki" page instead of your content.

Then, how is it possible that is just the table entries?
Users and Sessions still there

@NGPixel , hello!
I've got the same issue on Heroku free plan. I understand the workaround, however I don't understand how to enable git remote sync for wiki js. Is there an article on that somewhere?
Thanks

@beiota, That's, in fact, a really good question. I found it confusing as well.
@NGPixel If you explain us a little bit how to do that, I can try to write a small step-by-step guide for future users.

To enable git remote sync, you'll need a git repo somewhere -- like Github or Gitlab.
The Heroku environment variables -- which you can get to through Heroku > Your Wiki App > Settings, and then click on "Reveal Config Vars" -- need to be filled out with correct entries. Briefly, you'll need a user account on a git hosting service, and enter the username and password so that wikijs can push syncs to it.
I've copy/pasted the relevant entries below.
 "WIKI_GIT_URL": {
      "description": "URL of the remote git repository",
      "value": "https://github.com/Organization/Repo"
    },
    "WIKI_GIT_BRANCH": {
      "description": "Branch of the remote git repository to use. This branch must already exist.",
      "value": "master"
    },
    "WIKI_GIT_USERNAME": {
      "description": "Username to access the remote git repository",
      "value": "marty"
    },
    "WIKI_GIT_PASSWORD": {
      "description": "Password / PAT / Deploy Key to access the remote git repository",
      "value": "MartyMcFly88"
    },


How to scroll to the top of the page

Hi,
The last page of my fullpage slideshow is really long, so I use overflow scroll.
When the users goes back and then forward again I would like the long page to start from the top (and lot from the scrolling point where it was, is this possible, if so how?)
Example here: (see last page)
https://ebike.bikesquare.eu/baraggia/in-ebike-tra-risaie
Thanks
Massimo

See demo : https://river-lee.github.io/vue-fullpage/examples/scroll.html.
https://github.com/river-lee/vue-fullpage#options options.overflow = 'scroll'


code-splitting error with DatePicker

When using DatePicker / DatePickerInput, I get thins error:
[!] Error: Invalid value "iife" for option "output.format" - UMD and IIFE output formats are not supported for code-splitting builds.
It can be solved by adding
inlineDynamicImports: true
in rollup.config.js, section output.
I think that the solution creates a single bundle, but at least adding this in the docs would be very helpful for other people finding the same error.

Related: #778
Nice catch – we should document this, and add it to the Rollup example at the very least.


Fix connection problem

Hi @rockem , I encountered few errors with the previous code, fixed with the below changes

@Debora38 Hi. I'm adding tests (so it won't happen again) and will release a fixed version soon

@Debora38 This issue is now fix on either v1.2.2 or v1. Please update me if it still has any issue


More strict typing

Hi @alvaroserrrano, thank you for your contribution!
Good catch, the typo 👍  I've included it by cherry-picking the commit.
But I'll not merge the branch because of the second commit.
I think you don't use the project's config (Typescript and ESLint config at least).
Anyway, thanks for your time.


? Not found. Cant change title

Hey there i just upgraded realscoreboard to the newest update. Running 1.16.5. I cant seem to change the title of the scoreboard. Even with default config it shows up as “? Not found” was wondering if other had the same issues. I love using realscoreboard so if u could get back to me. Sorry for the minimal info im at work on my phone .

Fixed in the new version. Please check it out!

Let me know if it fixed your problem.

That fixed it ty :)


Color Format not changing

I am trying to change the color format but it is taking the default value.

Can you please supply some code of what you are trying to do. A codepen or jsfiddle is preferred

Thanx ruhley I solve this issue.


Question?

There's no plan for an editor??? :)

I always wanted an editor in Qor.  There was a basic map editor in before the rewrite so it wouldn't be too hard to put something like that back in.  My artist works in Blender primarily so I want to get that support finished first and of course get some better documentation so people can learn how to use the engine without fumbling around with the code. :)

I'm sorry if this was an unnecessary issue and needs to be closed... this is the only way I know to contact you as soon as I found it...
would love to try how this would go about... I really like to learn python someday... this might be it... and more making a game out of it... I wish you can build an editor... and I would like to see it so I can(hope) I can contribute to it the earliest time possible... :)

and what about the gui? :)

No problem.  I'm glad you're interested! :)  The project is still in early stages as it's mainly a way for me to learn engine programming, so usage in a project at this point wouldn't be recommended unless the project is fairly simple.  There's definitely still bugs.  Of course, if you're not afraid to tweak the engine (and wanting to learn some engine coding as well) it would make a good game base/example to start with.  The python bindings are also not complete.  I've been looking for better ways to autogenerate the bindings instead of writing them manually in PythonBindings.h, but I haven't had a need for it yet.
I think if you just want to toy around with something for learning purposes you'll want to use something more stable like Ogre for the time being.  That way if you have bugs you'll know that they're your own and not just my engine's. ;)
There's no full GUI integration but I do support Cairo/Pango or using 2D ortho meshes and textures if you wanted to roll your own or integrated CEGUI.  I also have a basic game menu system that can be hooked up to game settings (see my qorpse project's MenuState for an example) if just want something quick.
Btw, you can always email me at<EMAIL_ADDRESS>as well.   I'd be happy to answer more questions.

well... actually I'm more of a dev on top of something... like web dev... and game dev... not the backend/framework itself... that's where I don't excel in application but maybe in theory... like suggesting new features because I can't implement one and I don't because I want to focus on the previous topic I went about...
as for a game engine... (honestly) I don't want to toy around with engine where scripting takes most of the time... I'm more of a WYSIWYG(unless maybe for webdev) guy...
I do suggest you integrate a full GUI as most of the 3d engines I encounter(even when only for research purposes) doesn't have full GUI... they mostly rely on community built gui or a default gui with less customizability as the big guys has... so yeah... I suggest that and the editor... :)
oh and yeah... if I ever do suggest things... I always keep in mind that I suggest it considering the worst case scenario... like user irresponsibility I guess haha
(just so you know... :) )

Thanks for the suggestions!  I'm working on 2 more demos which I'll have up soon w/ some physics integration so be sure to follow me to see the progress.

yeah sure... I already followed this repo... I might be writing suggestions... not to be done as of now... but will be handy in the future... :)
and as they say... may this be a start of a good relationship... offers hand for a shakehand

I don't want... intended... or in any way express you to be offended...
 :D

Emailed you... :)

Emailed you again... :)

@flipcoder ~ Do you have a branding for this? I may want to put my designer skills into this one... :)

This is all I have so far, and it's just something I did myself in inkscape:
https://raw.githubusercontent.com/flipcoder/qor/master/bin/data/logo.png
If you wanna create one, go for it :)

@flipcoder ~ I've also sent you an email... a long one if you don't mind...
I have a question... what's the story you might want to tell with the logo... or the impression you might want to express... :)


fixing copy typo

Description
Please explain briefly the changes you made here.
Jira Ticket
Please add link to Jira ticket
Coding Standards
Code Style Guide: https://github.com/companieshouse/dissolution-web/wiki/Code-Style-Guide
Checklist

[ ] 3 Amigos
[ ] Acceptance Criteria met
[ ] Branch named {feature|hotfix|task}/{S4-*}
[ ] Commit messages are meaningful
[ ] Manually tested
[ ] All unit tests passing
[ ] WAVE accessibility testing tool ran on new or updated pages
[ ] All UI Tests Passing
[ ] Pulled latest master into feature branch
[ ] Sonar Analysis
[ ] Master branch on Rebel1 and CIDev CI/CD pipeline is green
[ ] Any Docker environment variables added are consistent with those on our upstream environments (Rebel1, CIDev etc)
[ ] If your pull request depends on any other, please link them in the description

Screenshots of new or updated views

CI: No IaC security warnings found


Develop the Lexer

The lexer will take the regex string and convert it into a list of tokens. Tokens can be characters, operators like * or |, and special symbols like ( and )
The terminals recognized by our lexer are
terminal ::  CHAR := Any printable ASCII character except the terminals below.
terminal ::  OR := |
terminal ::  STAR := *
terminal ::  PLUS := +
terminal ::  QUESTION := ?
terminal ::  LPAREN := (
terminal ::  RPAREN := )

[x] #2
[x] #3
[x] #4
[x] #17

Lexer has been developed, with the following PRs

#22
#26
#43 which is another set of PRs

#30
#36
#37
#38
#41
#42

#57


Improved less fix

previous version would skip "NameError" and potentially other errors
introduced support for the capture of the error message

Now the errorformat should be more robust for all the errors produced by lessc. Thanks for the previous clean up.

Pulled, cheers :)


Missing Alonzo genesis files

Updating documentation
Description of the change
When I was runnng Cardano node for the first time on my computer, the Alonzo genesis files where missing. I downloaded them and the node ran successfully.

#125

It is partially an issue with cardano-node CI/CD - which adds alonzo genesis for latest cardano-deployment , given latest is still marked to 1.27.0 for mainnet. I believe we need the change but as soon as new node release is marked.

@tweakch and @rdlrt are both right that cardano mainnet only supports Mary era for now and therefore we are not adding genesis files for Alonzo just yet. But as soon as it hits mainnet then expect major updates regarding this into the developer portal.


reference doc: top-level

https://kpt.dev/reference/
Just a high-level description of the command groups will suffice. Examples and demo are stale and should be deleted.

https://github.com/GoogleContainerTools/kpt/pull/1978
@frankfarzan Any other items should be covered by this issue ?


Llama2 70B Prefill MLP

Tracking Llama2 70B Prefill MLP here.
Goal: 32 users, 128 sequence length MLP performant prefill
Tasks:

Prefill MLP test cases
Prefill MLP functionality
Prefill MLP Perf

FYI @cglagovich @mikevin920

Added MLP functionality and rebased to main. Adding a merge request rn

The Prefill MLP has PCC of 0.988 and is waiting on #6556 to resolve the compatibility of ttlib.ops.primary.matmul to support fp32 accumulate so that PCC can get even higher.

Latest commit exposed that #6556 can be worked around with custom program config, but it also shows that fp32 accurate is too big to fit into L1

@cglagovichTT close?


First stage of fixing existing docs

Plan:

Run Kokoro with this commit. We should see output saying that it
would upload docs for 2 APIs, but it's just a dry run.
Revert the uploaddocs.sh change in another commit, so we can actually do it for
those 2 APIs, and wait for those APIs to be published.
Do it for the rest of the APIs.

Closing now this is fixed. Will leave the branch around for a while.


Lw/story/2891/uni lp icons

Title
New unique colorful circular icons are added for lp tokens
Changes 👷♀

- item 1
How to test 🧪
see if icons are as in figma design https://www.figma.com/file/EdkPKn73UBzSbkr1xKwo6c/App-(old)?node-id=9048%3A43454
Definition of done ✔

[ ] Acceptance criteria for each issue met
[ ] Unit tests written where needed and passing
[ ] End-to-end tests for happy path
[ ] Documentation updated
[ ] Project builds without errors
[ ] Non-functional requirements met
[ ] Code reviewed and functionality tested by the reviewer (locally, Heroku, etc.)
[ ] Feature verified and accepted by Product Owner
[ ] Project deployed to production environment

New icons


GUI mockup

User story
As a user I want to be able to decide what should be in the user application so a development of the UI can be made successfully with the features driving, plant status, driving speed, camera feed.
Acceptance criteria:

[x] Create a GUI mockup with the features specified.

Created a basic GUI, follow the link for the slide: https://docs.google.com/presentation/d/1SrHGW4eAhuL-4G-y3P0KRYM-HpgYRK7OkYgQqJxowog/edit#slide=id.g1244cf1277c_1_10


Fix Cripple Monster

Fixes #174

Has this been verified to be working?

I have verified it with just my machine and Cripple Monster was used after the fix.

Verified.


SLOC: make it work on directories

Updated readdirp compatibility
Problem
The current version of readdirp in the codebase is 3.3.0.
However, due to a series of breaking changes readdirp introduced during the transition from version 2.x to 3.x, current implementation does not function as expected when running on directories. However it still works when running on single files. In this PR i aim to restored the functionality.
Example:
sloc cli.js ../lib
/Users/domen/.nvm/versions/node/v20.10.0/lib/node_modules/sloc/node_modules/readdirp/index.js:264
    throw new TypeError('readdirp: root argument must be a string. Usage: readdirp(root, options)');
    ^

TypeError: readdirp: root argument must be a string. Usage: readdirp(root, options)
    at readdirp (/Users/domen/.nvm/versions/node/v20.10.0/lib/node_modules/sloc/node_modules/readdirp/index.js:264:11)
    at readDir (/Users/domen/.nvm/versions/node/v20.10.0/lib/node_modules/sloc/lib/cli.js:176:12)
    at /Users/domen/.nvm/versions/node/v20.10.0/lib/node_modules/sloc/lib/cli.js:194:16
    at FSReqCallback.oncomplete (node:fs:200:5)

Solution
In response to this issue, the code has been adapted to ensure compatibility with readdirp version 3.x. The changes align with the specifications outlined in the readdirp changelog.
✅   tests  80 passing (55ms)
✅   linter

✅  built and ran the generated cli to verify it now works as expected in comparison to not working before.

Thank you!!

still saying the same issue in https://github.com/Homebrew/homebrew-core/pull/158728


Having too many items causes lag spikes

Having too many items causes lag spikes and stuttering.

Details
Having too many items in your inventory will cause your equipped weapon to cause your character to lag spike and cause fps drops while you fire, reload, and aim your weapon. This is an oversight in the game's engine and while this issue most likely won't occur for most users, people who like to explore entire planets before having to jump back and forth to different shop keepers to sell your items or to store your items in your ship and your ingame home will find this issue to be annoying and potentially game breaking.
Game Version
<IP_ADDRESS>
New Game
Yes
Reproduction Steps
Have at least 300 items in your inventory and try to fire, aim, and reload your weapon and notice the fps drops, stuttering, and lag spikes.

Additional info: This is due to the game recalculating the stats of EVERY item whenever you equip something that has a trait due to being rare/epic/legendary.
Particularly noticeable with Helens Cutter and scanner as every time you activate the scanner you equip Helens Cutter that has a trait on it and thus the calculation happens.

Current fix is to declutter your inventory. I've basically removed most (if not all) Misc. items to storage (home or starship), sold all non-essential Aid items (just Med Packs and Panacea), kept only essential ammo and only keeping a fixed amount (around 300 for each weapon).


Add files via upload

Hi,
I couldn't see the results from the CHSH game program, so I have added the following lines
to the driver codes:
                    System.Console.WriteLine("\n\nPress Enter to continue...\n\n");

		System.Console.ReadLine();

Now I can see the results.

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

Looks like you pushed the wrong changes.
This PR only includes a Factolization.sln, for a project that is not even part of the repository.
I'm going to close this one, feel free to submit a new PR.


Feature Request

Hi
nice portal!
It would be nice if it is possible to check multiple items and restore them in one step (checkbox)

Implemented in version 3 of the portal (https://github.com/nielsengelen/vbo365-rest/releases/tag/v3.0).


fix: an issue where Vite requires a specific export method

otherwise Vite throws this error:

Thank you @mesqueeb! Happy new year!


Tick drive permission by default, omit openid permission?

Is there a way of defaulting to the Drive permission being selected?
When I set the scope to .appData it shows up when the user signs in, but it is not ticked by default.

Secondly, is it possible to not include openid permissions?
I'm not sure what the first line in the permissions is for.
When I look up the docs it seems to correspond to openid, which I'm not sure I need.
I am only interested in the user's name and email (and being able to store app data).
Before using drive_helper, I used GoogleSignIn(scopes: ['email', 'profile'])
Users are much happier, and likely to sign in, when we are only asking for the permissions we actually need.
Thank you for making this helpful package!

I figured out for the second question that I can pass in an arbitrary permissions array in the init:
driveHelper.signInAndInit([
        'email',
        'profile',
        'https://www.googleapis.com/auth/drive.appdata'
      ]);

Unfortunately this still shows the openid row, so I guess it is baked into the appdata one.

Regarding the drive permission not being ticked by default, it has nothing to do with this package so I will close the issue.
Just a bad oversight on Google's part IMO.

Just a bad oversight

I'm pretty sure it's a security feature to encourage users to read the permission properly, and not just click through the defaults.

Secondly, is it possible to not include openid permissions?

I honestly don't even remember much about this project. I am planning to update it and its dependencies once Dart 3 comes out, so I'll take a look into this then.

Thanks. I ended up reverting back to google_signin for main sign in, and then only using drive helper when I'm actually doing drive stuff.


Tut 130: Is there an alternative to use aws_iam_openid_connect_provider?

I have the following error because my policies do not allow me to use OpenIDConnectProvider.
Is there an alternative?
Error: creating IAM OIDC Provider: AccessDenied: User<EMAIL_ADDRESS>is not authorized to perform: iam:CreateOpenIDConnectProvider on resource: arn:aws:iam::XXXXXXXX:oidc-provider/oidc.eks.eu-central-1.amazonaws.com/id/XXXXX because no identity-based policy allows the iam:CreateOpenIDConnectProvider action
│       status code: 403, request id: XXXXX
│
│   with aws_iam_openid_connect_provider.eks,
│   on 8-iam-oidc.tf line 5, in resource "aws_iam_openid_connect_provider" "eks":
│    5: resource "aws_iam_openid_connect_provider" "eks" {

Sorry for the late reply. Yes, you can attach IAM policies directly to the nodes instead of using an OpenID Connect provider.


backend register size limit exception

Implements a test for the generation of a RegisterSizeError from the api when setting a register size which exceeds limits for the backend.
Description
In addition to the above test a couple of other minor bugs were addressed.

Motivation and Context
This exception would allow the user to catch this condition and change the register size accordingly.

How Has This Been Tested?
TestQuantumProgram.test_simulator_online_size

Screenshots (if appropriate):
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

This looks good to me.


Simple example to show usage of Last Operator

Again thanks for your contribution
Thanks
Amit Shekhar

Welcome @amitshekhariitbhu  , would be contributing more in future


Add functions for calculating max harmonics and eccentric transition

Source class no longer takes ecc_tol and max_harmonic arguments. Instead you pass gw_lum_tol and it uses that to calculate the rest!
In an ideal world, I would make Source.gw_lum_tol read-only so users could only change it with Source.update_gw_lum_tol() (and hence prevent the Source.max_harmonic() function and Source.ecc_tol from going out of sync)...but I couldn't work out how to do that and so am operating on a "let's hope our users use it the way they are supposed to!" haha.
I also:

added a verbose parameter so you can see whether your sources are stationary/circular/evolving/eccentric
fixed some tiny things in evol.py which were throwing warnings (divide by 0's and negative numbers to fractional powers etc)

I think the way that you signal to the user that a class attribute should be read only is to name is with a preceding underscore: self._gw_lum_tol (as per this stack overflow: https://stackoverflow.com/questions/10929004/how-to-restrict-setting-an-attribute-outside-of-constructor). I guess python does not want to fully restrict the setting of attributes ever, so this might be the way to go. I am not super worried about this though, so if you'd rather keep it as gw_lum_tol, let me know!

I think the way that you signal to the user that a class attribute should be read only is to name is with a preceding underscore: self._gw_lum_tol (as per this stack overflow: https://stackoverflow.com/questions/10929004/how-to-restrict-setting-an-attribute-outside-of-constructor). I guess python does not want to fully restrict the setting of attributes ever, so this might be the way to go. I am not super worried about this though, so if you'd rather keep it as gw_lum_tol, let me know!

Ah gotcha that's a good plan, I'll do that! :)

Ah gotcha that's a good plan, I'll do that! :)


feat: Add PermissionGuard and PermissionServices based on 'roles' claim

Initial implementation of a PermissionService/PermissionGuard/dhPermissionRequired-directive, based on Msal and 'roles' claim in access_token.
Tasks:

[x] Documentation.
[x] Unit Tests.

For information, we have added a feature flag disabling authorization for all environments, except u-002 and b-002.


What ist the root password in the latest official image for Microsoft SQL Server on Linux for Docker Engine.

In order to administer the installation I need root access for the 2019-CU4-ubuntu-16.04 Docker image, but I cannot find it. Trying did not work ...

The main problem is that there is no sudo which is need by mssql-conf.
Message: : ./mssql-conf: line 28: sudo: command not found

But why in the first place did you want to modify the container on a runtime? Container should be disposable, portable and not immutable, if you want a specific setting other than provided by released image then you might want to take a look at building your own container image with dockerfile customized to your needs.

I did want to use the Microsoft maintained SQL Server on Linux for Docker 2019-CU4-ubuntu-16.04 image. Obviously in this Image the SQL Srver Agent (needed for scheduled backups) is disabled and I wanted to switch it on. This did not work (at least) due to the lack of sudo (i.e. used in scripts like mssql-conf) and root password.
One benefit of docker is also the ease of use and so I did not want to invest the time in learning how to build own docker images.
May be I have to try a SQL Server installation into a docker Ubuntu container.

First off all maybe SQL Server Agent was turned off by design for pre-built Docker image for the sake of minimalism.
And yes you can do SQL Server installation on vanilla Ubuntu container, but keep in mind that container should designed to be disposable, portable and not immutable, prepare everything for it including persistent volume or storage. If you think of deploying the container once and for all, that could be less fit the case, instead install it on Linux machine itself.
But if what you need is a scheduled backup, you could do that by using sqlcmd from anywhere outside the running container and put it on cron.

The main problem is that there is no sudo which is need by mssql-conf.
Message: : ./mssql-conf: line 28: sudo: command not found

Fortunately, you don't need the root user to make a change in the mssql-conf file. There was another thread very similar to your issue, however, the solution will depend on what you want to do.
Are you looking to modify the mssql-conf file while the container is running (already started) or during a custom build process? I can guide you to make both changes but it will be easy to nail down the process to the specific requirement.
In case what you want is simply to have the SQL Agent started. This can be easily accompplish during the container creation using the MSSQL_AGENT_ENABLED environment variable. Here you have an example:
docker container run \
--env 'ACCEPT_EULA=Y' \
--env 'MSSQL_SA_PASSWORD=_SqLr0ck$_' \
--env 'MSSQL_AGENT_ENABLED=1' \
--detach mcr.microsoft.com/mssql/server:2019-CU4-ubuntu-18.04

Cheers,

Yes, I have set MSSQL_AGENT_ENABLED=true and according to the 2019 doc, it is preinstalled in the image. I also have enabled it via script, but the the context menu entry of the SQL Server Agent is still greyed out. The Maintenance Plan Wizard menu entry is also not existing.
I am wondering that this Microsoft maintained image is that poor.
I do not want to deploy anything, I just want to setup one installation for test and delelopment in an existing system without (more or less) touching that system.
Thx for your help.

@dbsmaster: the environment variables are correctly set. After looking into the log mssql files, it seems, that the agent canont connect to the server (no reason given). This would fit the observation, that in SSMS the context menu item to start the agent is greyed out.
FYI the container is running under qnap (current version).

@dbsmaster: the environment variables are correctly set. After looking into the log mssql files, it seems, that the agent canont connect to the server (no reason given). This would fit the observation, that in SSMS the context menu item to start the agent is greyed out.
FYI the container is running under qnap (current version).

Will try to replicate it from my side, I have not experienced a problem with the agent on any version but SQL 2019 CU2.

What’s the length of the hostname for the container? There’s an issue with SQL Agent on Linux cannot support hostnames linger than 15 characters.
Anthony
On May 3, 2020, at 11:41 AM, Carlos Robles<EMAIL_ADDRESS>wrote:

@dbsmaster: the environment variables are correctly set. After looking into the log mssql files, it seems, that the agent canont connect to the server (no reason given). This would fit the observation, that in SSMS the context menu item to start the agent is greyed out.
FYI the container is running under qnap (current version).
Will try to replicate it from my side, I have not experienced a problem with the agent on any version but SQL 2019 CU2.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/mssql-docker/issues/599#issuecomment-623139833, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AE4X5FNY57KRWH7KQAXMBKLRPWNCLANCNFSM4MWY25MA.

The length of hostname is 27. I will try a shorter one.

Even with a hostname of 5 letters still the same problem.
Extract of sqlagent.out:
2020-05-03 19:01:32 - ! [150] SQL Server does not accept the connection (error: 11001). Waiting for Sql Server to allow connections. Operation attempted was: Verify Connection On Start.
2020-05-03 19:01:32 - ! [000] Unable to connect to server '(local),1433'; SQLServerAgent cannot start
2020-05-03 19:01:32 - ! [103] SQLServerAgent could not be started (reason: Unable to connect to server '(local),1433'; SQLServerAgent cannot start)
2020-05-03 19:01:37 - ! [298] SQLServer Error: 11001, TCP Provider: No such host is known. [SQLSTATE 08001]
2020-05-03 19:01:37 - ! [165] ODBC Error: 0, Login timeout expired [SQLSTATE HYT00]
2020-05-03 19:01:37 - ! [298] SQLServer Error: 11001, A network-related or instance-specific error has occurred while establishing a connection to SQL Server. Server is not found or not accessible. Check if instance name is correct and if SQL Server is configured to allow remote connections. For more information see SQL Server Books Online. [SQLSTATE 08001]
2020-05-03 19:01:37 - ! [382] Logon to server '(local),1433' failed (DisableAgentXPs)
2020-05-03 19:01:37 - ? [102] SQLServerAgent service successfully stopped
2020-05-03 19:01:39 - ? [098] SQLServerAgent terminated  (normally)

Even with a hostname of 5 letters still the same problem.
Extract of sqlagent.out:
2020-05-03 19:01:32 - ! [150] SQL Server does not accept the connection (error: 11001). Waiting for Sql Server to allow connections. Operation attempted was: Verify Connection On Start.
2020-05-03 19:01:32 - ! [000] Unable to connect to server '(local),1433'; SQLServerAgent cannot start
2020-05-03 19:01:32 - ! [103] SQLServerAgent could not be started (reason: Unable to connect to server '(local),1433'; SQLServerAgent cannot start)
2020-05-03 19:01:37 - ! [298] SQLServer Error: 11001, TCP Provider: No such host is known. [SQLSTATE 08001]
2020-05-03 19:01:37 - ! [165] ODBC Error: 0, Login timeout expired [SQLSTATE HYT00]
2020-05-03 19:01:37 - ! [298] SQLServer Error: 11001, A network-related or instance-specific error has occurred while establishing a connection to SQL Server. Server is not found or not accessible. Check if instance name is correct and if SQL Server is configured to allow remote connections. For more information see SQL Server Books Online. [SQLSTATE 08001]
2020-05-03 19:01:37 - ! [382] Logon to server '(local),1433' failed (DisableAgentXPs)
2020-05-03 19:01:37 - ? [102] SQLServerAgent service successfully stopped
2020-05-03 19:01:39 - ? [098] SQLServerAgent terminated  (normally)

Can you share your Dockerfile / docker container run command? So, I can reproduce this error from my side?
I have been experimenting with the latest CU, so far I have not had any issues at all.
Cheers,

I am also looking for the root password, I want to install iputil to diagnose a connection failure, I can't do it without creating a docker file and hook it into docker-compose. it means for a simple diagnostic I need to change docker-compose and rehook it to the rest of the system while I could do it easily on the fly. when you are sharing the whole docker file, what is the point of setting a password for the root?

This allowed me to log in as root:
docker exec -u 0 -it mycontainer /bin/bash

Is it really that big of a deal to just share out the root password? I need to do some diagnostics on why the box isn't connecting the inability to do literally anything in the shell is ridiculous.

This allowed me to log in as root:
docker exec -u 0 -it mycontainer /bin/bash

legend 🚀


Added query of hipDeviceAttributeHdpMemFlushCntl and hipDeviceAttribu…

…teHdpRegFlushCntl

Abandon the original PR and resubmit this one to avoid issues due to code merged.

@yxsamliu Do you need these 2 new device prop fields in this PR to also be excluded from VDI path?

@yxsamliu Do you need these 2 new device prop fields in this PR to also be excluded from VDI path?

Not for now. These two fields should be excluded from VDI for now, like the above 3.


Remove leading zeros from displayed dates.

Closes #357.
Updates all instances of {entry_date format="%F %d, %Y"} to use {entry_date format="%F %j, %Y"}, which will remove the leading 0 from the day.
Ex: February 02, 2016 will become February 2, 2016.

@michellekondou @zeldman - this is ready for review.

@brettdewoody I merged your pull request Brett, thank you so much for contributing!


App must be running before you can run middleware on Docker

I have a Sanic-GraphQL app and I'm trying to run it into a Docker Container.
But i'm having this issue when my app receives any request:
Traceback (most recent call last):   File "/usr/local/lib/python3.7/site-packages/sanic/app.py", line 885, in handle_request     response = await self._run_request_middleware(request)   File "/usr/local/lib/python3.7/site-packages/spf/framework.py", line 533, in _run_request_middleware     "App must be running before you can run middleware!" AssertionError: App must be running before you can run middleware!

Here is my Dockerfile:

Here is my run.py:

Here is my routes.py:

I don't have this issue running outside from a Docker Container.
Thanks to advance!

Solved by installing all libs again from requirements.txt. It should warn that was a missing lib problem.

Solved by installing all libs again from requirements.txt. It should warn that was a missing lib problem.

I have the same issue here.
What do you mean "installing all libs again from requirements.txt"?
Can you paste your dockerfile please?


Breaking - remove endorser capability to write ledger transactions

WIP as I am still testing the changes.
I didn't remove the "request endorser to write transaction" flag.  In the author I hardcode to false.  In the endorser I raise an error if the author requests this function.  (Handles the case where the endorser is updated and the author isn't.)
See issue #2711
There is some work to do on problem reports, I added a separate Issue for this task.

Converting to draft, I'll hold onto this until @jamshale 's PR is merged (update endorse schema for anoncreds)


Allow User to Favourite or Unfavourite Sessions

This PR will add the feature to fav/unfav sessions. Favourites are part of the user not attendee model, because users are attending more than one event etc.
Implementation-wise it is kept simple and is missing other features like how many "favs" a certain session has etc. I leave this for later, when @artifactdev needs it.
The output of /api/sessions looks like this now:
[
    {
        "id": "acheing-a-internally-attached-limit",
        "url": "http://localhost:8000/api/sessions/acheing-a-internally-attached-limit/",
        "title": "Acheing A Internally Attached Limit",
        "description": "Amet modi numquam sed. Dolor dolore dolorem magnam dolore sit aliquam tempora. Etincidunt porro dolorem aliquam labore labore sit. Quiquia amet numquam numquam eius etincidunt velit dolorem. Tempora est neque dolore. Tempora velit sed consectetur dolore. Eius porro magnam amet dolorem etincidunt dolorem numquam. Quiquia neque quaerat neque quisquam non tempora. Consectetur non dolor magnam tempora etincidunt ut.",
        "speakers": [
            {
                "url": "http://localhost:8000/api/speakers/jana-vogel/",
                "name": "Jana Vogel",
                "image": ""
            }
        ],
        "event": "devdata.18",
        "actions": {
            "favourite": "http://localhost:8000/api/sessions/acheing-a-internally-attached-limit/favourite",
            "unfavourite": "http://localhost:8000/api/sessions/acheing-a-internally-attached-limit/unfavourite"
        }
    }
]

The URLs in the actions object have to be request with a POST request. Part of that request will be the authenticated user. At the moment we use the Basic/Session auth, which is sufficient for now.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
18 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.3%) to 99.055%

Files with Coverage Reduction
New Missed Lines
%

talk/api_views.py
18
59.62%

Totals

Change from base Build<PHONE_NUMBER>:
-0.3%

Covered Lines:
3803

Relevant Lines:
3841

💛  - Coveralls


Add finalizer to AzureBlobFileSystem to avoid having unclosed service

This should fix the Error print on AzureBlobFileSystem garbage collection
Fixes #129

Thanks @Darune !


Add subsections in service configuration

Adds subsections in the Service configuration section:

Extensions
Pipelines
Telemetry

I've been finding myself wanting to get a link to e.g. the "telemetry" section, but without a subsection, it's not possible. I believe these subsections make sense.

Hm not sure how I can get around this markdown lint error:
/home/runner/work/opentelemetry.io/opentelemetry.io/content/en/docs/collector/configuration.md:505: MD024/no-duplicate-heading/no-duplicate-header Multiple headings with the same content [Context: "### Extensions"]

"Extensions" seems like the natural name for this subsection, even though it is indeed a duplicate name.

🎉 thank you


[FLINK-26188] Refactor TestingHighAvailabilityServices to properly use the builder pattern.

https://issues.apache.org/jira/browse/FLINK-25985

FYI: I changed the title of this PR to point to FLINK-26188. The initial ticket FLINK-25985 was referring to e2e tests in the context of the JobResultStore which won't happen. This enables us to close FLINK-25985. This PR can continue to exist as part of the follow-up ticket FLINK-26188.


Add preferences for first parameter and arguments showing up on newlines

Fixes https://github.com/scala-ide/scalariform/issues/202 .
Sorry - this is a big, messy one. The underlying alignment code is complicated.
This doesn't change default behavior; that can come in a future PR.

This also fixes #158 .

Is this PR going to be merged? What is the road map for the 1.0?

Not being on the scala-ide team, I'm holding off on merging this until:

Jenkins configuration is fixed or confirmed ignorable for PR #225
PR #225 is merged
I resolve the merge conflicts here & fix any updated tests

After that, it's mostly getting an integration test running to validate bugs #142 and #153 . I have that partially working, but have been lazy getting it fully-working (and these PRs were blocking that as well).
After that, I'll need to figure out the scala-ide release process. :P

Thanks for the update @jkinkead
Just for information this basically THE bug/missing feature that Intellij Idea is managing well, as well as being a Standard Scala Style, and I guess many people would like to see in.
Do people agree with this?

Yes. This 100% will be in the final release. :)


Use one consistent way of adding BrainLocation properties in entities

After discussing different possibilities, it was agreed to add brainLocation properties in the following way:
use brainLocation property shape with BrainLocationShape as node in schemas:
{
"property":
[
"path": "nsg:brainLocation",
"node": "bbplocsh:BrainLocationShape"
]
}


IGColorPicker subview doesn't respond to gestures

I'm just following the basic code in the documentation and put this in my viewDidLoad:
// Setup colorPickerView
    colorPickerView = ColorPickerView(frame: CGRect(x: 0.0, y: 0.0, width: 200, height: 100))

    colorPickerView.delegate = self
    colorPickerView.layoutDelegate = self
    colorPickerView.style = .circle
    colorPickerView.selectionStyle = .check
    colorPickerView.isSelectedColorTappable = true
    colorPickerView.backgroundColor = .systemGray3

    colorPickerView.colors = [
    UIColor(red: 0.114, green: 0.169, blue: 0.325, alpha: 1.000),
    UIColor(red: 0.494, green: 0.141, blue: 0.325, alpha: 1.000),
    UIColor(red: 0.000, green: 0.529, blue: 0.318, alpha: 1.000),
    UIColor(red: 1.000, green: 0.000, blue: 0.302, alpha: 1.000),
    UIColor(red: 1.000, green: 0.639, blue: 0.000, alpha: 1.000),
    UIColor(red: 1.000, green: 0.925, blue: 0.153, alpha: 1.000),
    UIColor(red: 0.000, green: 0.898, blue: 0.212, alpha: 1.000),
    UIColor(red: 0.161, green: 0.682, blue: 1.000, alpha: 1.000),
    UIColor(red: 0.514, green: 0.463, blue: 0.612, alpha: 1.000),
    UIColor(red: 1.000, green: 0.467, blue: 0.659, alpha: 1.000)]

    jpTextView.addSubview(colorPickerView)

It looks as expected and appears above all my other views, but why can't I interact with it? I seem to be able to highlight text in the UITextView below this subview.

For some reason to make the circles respond, I need to hold down on them for about 3 seconds. Does anyone know why it's acting so odd? I want to just tap to select the color.

There is some way clicked ?


Update data automatically

I would like to automate the process of getting the data. Specially to have up-to-date prices.
It would be nice to know where did you download the decks, prices, data_types, rarity from. Right now I'm getting it from scryfall bulk data download, but I see some inconsistencies between that data and the one you used. For example the rarity is lower case instead of capitalized and the data_types include the sub-types while the one you provide doesn't. That makes me believe that isn't the data you used and I would have to process it so that it's the same. But I could also download it directly from the same place that you did.
Edit: never mind I've now seen the scraper is at ElLorans/GoldfishScrape. I wish I had seen that before scraping the penny dreadful decks from https://dreadrise.wizzerinus.tk/
Edit2: that scraper only gets the decklists, so I'm still curious about the rest of the data.

Sorry for taking 6 months to reply, I had received no notification. I have another private repo which I'll make public as soon as I can, but I use scryfall bulk data too.

Repo is now public: https://github.com/ElLorans/Best-MTG-Deck-GenerateData