Don't relocate scoped style tags

1. Open a page that is using <style scoped="scoped">..</style> with
modpagespeed enabled (link below)
2. Open a page that is using <style scoped="scoped">..</style> with
modpagespeed disabled (link below)
3. Compare.

Due to the fact that style scoped applies its styles to and only it's children,
the layout of the page is broken as modpagespeed is moving all style tags to
the <head>..</head> section.

Solution: Do not relocate style scoped tags

Version: X-Mod-Pagespeed: <IP_ADDRESS>-4009

Fedora 20
Server: Apache/2.4.9 (Fedora) OpenSSL/1.0.1e-fips mod_nss/2.4.6 NSS/3.15.3
Basic ECC PHP/5.5.14 mod_wsgi/3.5 Python/2.7.5 mod_perl/2.0.9-dev Perl/v5.18.2

Compare these urls:
https://www.highdefinition.ch/kendo/listview/index.php
https://www.highdefinition.ch/kendo/listview/index.php?ModPagespeed=off

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 3 Aug 2014 at 12:31

I think this is coming up again with WebComponents.  They use scoped css, but without setting the scoped attribute: http://www.html5rocks.com/en/tutorials/webcomponents/shadowdom/
It looks like maybe <style> inside <template> should be treated like scoped?

Talking to Polymer people, I think the problem is that our parser isn't <template>-aware at all.  We should probably skip over <template> blocks entirely?  I should read the spec for it and see what's ok for us to do.

My feeling is we should parse it, but be wary of it in much the same way as
we are wary of noscript blocks in the current system.
On Tue, Aug 2, 2016 at 3:24 PM, Jeff Kaufman<EMAIL_ADDRESS>wrote:

Talking to Polymer people, I think the problem is that our parser isn't

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pagespeed/mod_pagespeed/issues/965#issuecomment-237015153,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAmZSNrs2TWuSBW8OLUSEtP1v52tibkuks5qb5l2gaJpZM4Ja7By
.

One problem with templates from our perspective is that they don't do anything in the form that pagespeed will see them, and then they are arbitrarily modified by js before they are actually on the page.  So we may resolve urls incorrectly, or a url may be present that's intended to be rewritten to point somewhere else.  Minification should be safe, though.

Template spec: https://html.spec.whatwg.org/multipage/scripting.html#the-template-element


Feature: add domain and dto suffix support

Please make sure the below checklist is followed for Pull Requests.

[x] Travis tests are green
[x] Tests are added where necessary
[ ] Documentation is added/updated where necessary
[x] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

PR for https://github.com/jhipster/generator-jhipster/issues/8647

According to this conversation : https://github.com/jhipster/generator-jhipster/pull/8649#pullrequestreview-168506204
I'll make the adjustment to replace 'domain suffix' by 'entity suffix'.


queue?

I'm wondering about the pros and cons of renaming stack with queue or some such?
The reasoning is that this function is used to build an experiment (i.e. at the user layer), whereas stack is really describing the implementation.

Other ideas:  procedure, design (nouns), stage, build (verbs)

It's been stewing in the back of my mind a bit and I like the idea of build.
tl.build(
tl.create(...),
cl.ramp(...),
...
)

I do not think that the implementation has much to do with stack, at least I didn't have the implementation in mind when I coined the name.
I'm not sure about build, in fact, create also "builds" in itself.
Stack has the connotation for me of something layered, which is what I meant.


Clean up tree naming

Rename nodeUtils to treeUtils because "node" could mean either "TreeItem" or "node.js"
Rename azureFunctionsExplorer to azFuncTree because it's shorter and more consistent with contextValue names (which all start with azFunc)

Well, treeUtils also sounds like things related to the tree view itself. Is treeItemUtils too verbose?

I think it's totally fine for this file to have utils related to both tree items and the tree view itself


Set initial token to head when all contained Event Handling Components of a Streaming Event Processor disallow replays

Feature Description
The current default for Sagas backed by a StreamingEventProcessor (SEP) is to set the initial TrackingToken to the head of the stream in the absence of configuration customization.
Although helpful, we can set a more reasonable approach concerning where a SEP should start processing based on any Event Handling Component (EHC) it deals with.
Namely, when all EHCs disallow replay behavior, it would be reasonable to default the TrackingToken to the head of the stream (i.o. the tail of the stream).
Current Behaviour
Axon Framework's current behavior around initial TrackingTokens for Sagas backed by StreamingEventProcessors is as follows:

When there is no configuration customization, the TrackingToken defaults to the head of the stream.
When there is configuration customization, the framework does nothing.

Wanted Behaviour
Set the initialTrackingToken to the head of the stream when all EHCs of a Streaming Event Processor disallow replays.
And, in the absence of other user customization, of course.
Possible Workarounds
None.

Closing this issue since it's a duplicate.


⚠ Bofrost.nl has degraded performance

In 7e78d7b, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 1965 ms

Resolved: Bofrost.nl performance has improved in 57ceb32.


Configure SCSS-Lint and Hound

Having both config files allows you to run SCSS-Lint locally via the command line, as well as tweak Hound’s default config to align with the purpose of Bitters

:+1:


Update to avoid errors executing npm cache clear

New versions of node throws an error with npm cache clear. Use verify instead

I don't think we need verify here? I'll check to see why node is throwing an error.


2.0.0-beta2 - constraintSet.applyTo() stops working after orientation change

In a fragment that uses constraintSet.applyTo() to animate layout changes, rotating phone (HTC M8 One) for orientation change or tapping edit text box to reveal keyboard results in constraintSet.applyTo() no longer working, leaving the layout locked in its current state. Minimizing the app and returning or navigating away from the fragment and returning fixes the issue. Reverting to 2.0.0-beta1 removes the issues entirely.

I'm having the same problem as explained above. Reverting to 2.0.0-beta1 also removed the issue.

Same with samsung z fold 3 at version 2.1.3 !


Check git tag matches galaxy.yml

Refuse to publish when version in galaxy.yml doesn't match current git tag. This is to keep versions aligned. It is also possible to use manifest.json instead of galaxy.yml, but I don't we use this in any of our collections.

Need to check ubuntu-latest has yq

Need to check ubuntu-latest has yq

Think this: https://github.com/actions/runner-images/blob/main/images/linux/scripts/installers/yq.sh means that it is

I noticed we are using this as a manually triggered workflow: https://github.com/stackhpc/stackhpc-release-train/blob/49316c8e3d9abdc4c66105cadfaf2e48446083e6/ansible/roles/source-repo-sync/templates/publish-collection.jinja#L7 so this approach breaks that. Possibly only do the check if triggered by tag push?

I noticed we are using this as a manually triggered workflow: https://github.com/stackhpc/stackhpc-release-train/blob/49316c8e3d9abdc4c66105cadfaf2e48446083e6/ansible/roles/source-repo-sync/templates/publish-collection.jinja#L7 so this approach breaks that. Possibly only do the check if triggered by tag push?

Good catch. Although you can either do a manual dispatch from a branch or a tag, and arguably it should only be done from a tag and we should fail otherwise?

I noticed we are using this as a manually triggered workflow: https://github.com/stackhpc/stackhpc-release-train/blob/49316c8e3d9abdc4c66105cadfaf2e48446083e6/ansible/roles/source-repo-sync/templates/publish-collection.jinja#L7 so this approach breaks that. Possibly only do the check if triggered by tag push?

Good catch. Although you can either do a manual dispatch from a branch or a tag, and arguably it should only be done from a tag and we should fail otherwise?

Added a check for triggering on a tag. Seems to work: https://github.com/jovial/ansible-collection-linux/actions/runs/5655939960/job/15322019650#step:4:5


refactor: dimension

Description:
Refactor UnitValue:

Use Dimension (spec https://www.w3.org/TR/css-values-4/#dimensions)
Use Ident in unit, we don't need extra AST type, because (from spec):

When written literally, a dimension is a number immediately followed by a unit identifier, which is an identifier.

Yes
Related issue (if exists):
No

WIP

@kdy1 Done :+1:


ASP.NET Core 2.1, quite random 404 - NotFound ?

After updating to ASP.NET Core 2.1, random 404 - NotFound
EDIT: It seems I did not properly upgrade to NET Core 2.1.
After I upgraded our companies site to ASP.NET Core 2.1 (via VisualStudio Nuget update packages), we get some strange-random behaviour.
Sometimes (it seems quite randomly) the call to the route (it doesn't matter which route, it happens on all of them returns 404 - NotFound.
This behaviour can easily be seen inside IIS logs. Random 404 lines.
Say I call (when testing) any route for the first time. And I get 404 -> the next call and all subsequent calls succeeds.

General

Windows Server 2016
NET Core hosted in IIS: No Managed Code, Integrated pipeline

@natemcmaster @muratg

@sabiland can you produce more detailed logging? There isn't enough information here for us to identify the issue.
You can elevate the logging level to "Debug" by changing your appsettings.Production.json file.
{
    "Logging": {
        "LogLevel": { "Default": "Debug" }
    }
}

You will also need to configure IIS to put these logs somewhere writable. See https://docs.microsoft.com/en-us/aspnet/core/host-and-deploy/aspnet-core-module?view=aspnetcore-2.1#log-creation-and-redirection for how to capture more logs.

Ok I have found the issue - the problem was with my csproj settings.
After I removed all Microsoft.* PackageReferences and added only this one:
<PackageReference Include="Microsoft.AspNetCore.App" />
Everything started working as expected! :)

Glad you found a fix.

I am facing the same issue after upgrading to 3.1
there is nothing in stdout logs and aspnetcore-debug logs are below.
[aspnetcorev2.dll] Initializing logs for 'C:\Program Files\IIS\Asp.Net Core Module\V2\aspnetcorev2.dll'. Process Id: 10500.. File Version: 13.1.19350.1. Description: IIS ASP.NET Core Module V2. Commit: e276c8174b8bfdeb70efceafa81c75f8badbc8db. [aspnetcorev2.dll] Resolving hostfxr parameters for application: 'dotnet' arguments: '.\MyApp.dll' path: 'C:\inetpub\wwwroot\TestApp\' [aspnetcorev2.dll] Known dotnet.exe location: '' [aspnetcorev2.dll] Process path 'dotnet.exe' is dotnet, treating application as portable [aspnetcorev2.dll] Resolving absolute path to dotnet.exe from 'dotnet.exe' [aspnetcorev2.dll] Invoking where.exe to find dotnet.exe [aspnetcorev2.dll] where.exe invocation returned: 'C:\Program Files\dotnet\dotnet.exe ' [aspnetcorev2.dll] Current process bitness type detected as isX64=1 [aspnetcorev2.dll] Processing entry 'C:\Program Files\dotnet\dotnet.exe' [aspnetcorev2.dll] Binary type 6 [aspnetcorev2.dll] Found dotnet.exe via where.exe invocation at 'C:\Program Files\dotnet\dotnet.exe' [aspnetcorev2.dll] Resolving absolute path to hostfxr.dll from 'C:\Program Files\dotnet\dotnet.exe' [aspnetcorev2.dll] hostfxr.dll located at 'C:\Program Files\dotnet\host\fxr\3.1.1\hostfxr.dll' [aspnetcorev2.dll] Converted argument '.\MyApp.dll' to 'C:\inetpub\wwwroot\TestApp\.\MyApp.dll' [aspnetcorev2.dll] Parsed hostfxr options: dotnet location: 'C:\Program Files\dotnet\dotnet.exe' hostfxr path: 'C:\Program Files\dotnet\host\fxr\3.1.1\hostfxr.dll' arguments: [aspnetcorev2.dll] Argument[0] = 'C:\Program Files\dotnet\dotnet.exe' [aspnetcorev2.dll] Argument[1] = 'C:\inetpub\wwwroot\TestApp\.\MyApp.dll' [aspnetcorev2.dll] Loading hostfxr from location C:\Program Files\dotnet\host\fxr\3.1.1\hostfxr.dll [aspnetcorev2.dll] Canceling standard stream pipe reader [aspnetcorev2.dll] Loading request handler:  'C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App\3.1.1\aspnetcorev2_inprocess.dll' [aspnetcorev2.dll] Creating handler application [aspnetcorev2_inprocess.dll] Initializing logs for 'C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App\3.1.1\aspnetcorev2_inprocess.dll'. Process Id: 10500.. File Version: 13.1.19350.1. Description: IIS ASP.NET Core Module V2 Request Handler. Commit: e276c8174b8bfdeb70efceafa81c75f8badbc8db. [aspnetcorev2_inprocess.dll] Waiting for initialization [aspnetcorev2_inprocess.dll] Starting in-process worker thread [aspnetcorev2_inprocess.dll] Resolving hostfxr parameters for application: 'dotnet' arguments: '.\MyApp.dll' path: 'C:\inetpub\wwwroot\TestApp\' [aspnetcorev2_inprocess.dll] Known dotnet.exe location: 'C:\Program Files\dotnet\dotnet.exe' [aspnetcorev2_inprocess.dll] Process path 'dotnet.exe' is dotnet, treating application as portable [aspnetcorev2_inprocess.dll] Resolving absolute path to hostfxr.dll from 'C:\Program Files\dotnet\dotnet.exe' [aspnetcorev2_inprocess.dll] hostfxr.dll located at 'C:\Program Files\dotnet\host\fxr\3.1.1\hostfxr.dll' [aspnetcorev2_inprocess.dll] Converted argument '.\MyApp.dll' to 'C:\inetpub\wwwroot\TestApp\.\MyApp.dll' [aspnetcorev2_inprocess.dll] Parsed hostfxr options: dotnet location: 'C:\Program Files\dotnet\dotnet.exe' hostfxr path: 'C:\Program Files\dotnet\host\fxr\3.1.1\hostfxr.dll' arguments: [aspnetcorev2_inprocess.dll] Argument[0] = 'C:\Program Files\dotnet\dotnet.exe' [aspnetcorev2_inprocess.dll] Argument[1] = 'C:\inetpub\wwwroot\TestApp\.\MyApp.dll' [aspnetcorev2_inprocess.dll] Setting environment variable ASPNETCORE_IIS_HTTPAUTH=anonymous; [aspnetcorev2_inprocess.dll] Setting environment variable ASPNETCORE_IIS_PHYSICAL_PATH=C:\inetpub\wwwroot\TestApp\ [aspnetcorev2_inprocess.dll] Loading hostfxr from location C:\Program Files\dotnet\host\fxr\3.1.1\hostfxr.dll [aspnetcorev2_inprocess.dll] Initial Dll directory: '', current directory: 'c:\windows\system32\inetsrv' [aspnetcorev2_inprocess.dll] Setting dll directory to c:\windows\system32\inetsrv [aspnetcorev2_inprocess.dll] Setting current directory to C:\inetpub\wwwroot\TestApp\ [aspnetcorev2_inprocess.dll] In-process callbacks set [aspnetcorev2_inprocess.dll] Event Log: 'Application 'C:\inetpub\wwwroot\TestApp\' started successfully.'  End Event Log Message. [aspnetcorev2_inprocess.dll] Starting app_offline monitoring in application 'C:\inetpub\wwwroot\TestApp\' [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Starting file watcher thread [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0 [aspnetcorev2_inprocess.dll] Adding request. Total Request Count 1 [aspnetcorev2_inprocess.dll] Removing Request 0


Unable to Setup Quasar RAT

Hi there, I'm new to RATs and I'm not really sure how I should install this, I watched a hackersploit video but that shows the files from 2017 and the current version doesn't have those .bat files and I don't know how to install quasar now, can someone guide me on what I should do?

To compile it yourself, download Visual Studio 2019 (I think you need to check the .NET desktop development workload), download the Quasar source code, open the Quasar.sln file, and click Build > Build solution. Wait for it to compile and you can use Quasar.

To compile it yourself, download Visual Studio 2019 (I think you need to check the .NET desktop development workload), download the Quasar source code, open the Quasar.sln file, and click Build > Build solution. Wait for it to compile and you can use Quasar.

To compile it yourself, download Visual Studio 2019 with .NET Desktop Development. Download this repository, open the Quasar.sln (once Visual Studio 2019 has finished installing), and click Build > Build Solution. Once you've done that, you can open the compiled program.
You could also just download the binaries without downloading Visual Studio 2019 here.
I meant to answer this three hours ago but my browser crashed and I forgot.

Also make sure you compile in Release mode not Debug.

where do you find the finished compiled program?

After you compile the project you will use the "Server.exe" to setup the "Client.exe". Then use the newly created Client.exe you saved to Desktop/Or where ever you want to run on the system to control. Test using LoopBack method using localhost/<IP_ADDRESS>

This is a bug tracker, for actual software bugs. Please refer to the guide for setup instructions: https://github.com/quasar/Quasar/wiki/Getting-Started


App URL with a279f postfix (app-a279f.domain.com)

Every time I push an app, the app url is created with a postfix. eg: app-a279f.domain.com
Epinio version: 1.1.0
Environment: AWS EKS
Ingress: nginx
Install epinio on EKS to reproduce this behavior.

Hi @riqueps , the suffix is and md5 hash of the namespace and was added here: https://github.com/epinio/epinio/commit/7c265a578#diff-31f0dac3ead01690242c1b3b77aecaedbe6e597da094216d65f95da3d093cc4aR28 as part of this Pull Request (also check the relevant issue in the PR description). The suffix ensures:

route uniqueness for applications of the same name which live in a different workspace so that users of different namespace can't use the same routes intentionally or non-intentionally
no user can try to "hijack" the default epinio API url (again intentionally or non-intentionally)

In a real world scenario, the automatic route provided by Epinio would be used as a CNAME to point an actual domain to the application or a custom route would be used for the same purpose.
What I'm not sure about, is why we didn't put the namespace "raw" in the route instead of a hash. That would make the url more readable and easier to remember. Maybe there was some limitation in what characters can be used in a url or something? @enrichman or @andreas-kupries ?

@jimmykarily I don't think there was a particular reason, the only collision that I can think about is the one about distinguishing the app name from the namespace: foo-bar-baz.
At the beginning of the development the suffix was just a random string, something like other providers or kubernetes do.

Maybe we can do something like: appname.namespace.systemdomain (separated with dots). With system domain being a wildcard one, this could work.

One thing that I didn't understand why is this start happening now?
Because I have others epinio installations (on eks and k3d) and all of them the route is created with the app name (eg: appname.domain.com).
Is there a way to force always create the route with the app name, and just put a postfix if the route name isn't available?

This change was introduced in 1.1.0. Are you sure the other installation are on that version? You need to push a new app for this to happen, old apps won't get updated (e.g. apps deployed when still on 1.0.0).
The default application chart changed here: https://github.com/epinio/helm-charts/commit/cbb88ca for version 1.1.0.
You can always set the routes manually with the --route flag of epinio push.
We could maybe add the suffix only when there is a conflict but there is more validation happening to avoid conflicts with existing ingresses and the whole logic would become even more complicated, while the end result would also be less consistent. What do you think about my suggestion with the subdomain above @riqueps ?

@jimmykarily I just realized that my old installation was at 1.0.0.
That would be great!!

This change was introduced in 1.1.0. Are you sure the other installation are on that version? You need to push a new app for this to happen, old apps won't get updated (e.g. apps deployed when still on 1.0.0).
The default application chart changed here: epinio/helm-charts@cbb88ca for version 1.1.0.
You can always set the routes manually with the --route flag of epinio push. We could maybe add the suffix only when there is a conflict but there is more validation happening to avoid conflicts with existing ingresses and the whole logic would become even more complicated, while the end result would also be less consistent. What do you think about my suggestion with the subdomain above @riqueps ?

So it is a feature NOT a bug ;-). Kidding aside, we found this issue surprising as we have a use case that we need to control our URI. eg:

ui app: uiapp.system.domain
ui-backend: uiapp-api.system.domain

Because it is easy to use epinio CLI, we using to for our deployment... with this "feature" our app broke because the new URI becomes

uiapp-a279f.system.domain
uiapp-api-a279f.system.domain

I think --route will work for us as it is just a simple change in our deployment script. But i wonder if there is a simpler way to alert the developer/operator. We have this issue also in CloudFoundry. Why dont we stop the push if we know there is a conflict in the URL. I feel like we are designing for what is a rare occurrence with a slight more complicated solution (which is a postfix in the URI)
Just my 2cents

I am currently modifying my automation script.. very easy fix.
By accident, i try to push an app with --route defined and an existing app already holding the URI and this is the error that i got
🚢  About to push an application with the given setup
Manifest: <<Defaults>>
Name: mani
Source Origin: /Use...ai-ui/build
AppChart:
Target Namespace: workspace
Environment 'BP_WEB_SERVER': nginx
Environment 'BP_WEB_SERVER_ENABLE_PUSH_STATE': true
Environment 'BP_WEB_SERVER_FORCE_HTTPS': true
Environment 'BP_WEB_SERVER_ROOT': .
Routes:
1: mani.orbitnao.beta.com

⚠  Hit Enter to continue or Ctrl+C to abort (deployment will continue automatically in 5 seconds)

Create the application resource ...

❌  error pushing app to server: Bad Request: route 'mani.orbitnao.beta.com' already exists

If so, it seems like it is already working as the way i was suggesting... Then why would need the fix for a random string behind the app again ? Cos nobody try to "hijack" the default epinio API url because it should fail like above (unless epinio is NOT an epinio by definition?)

Maybe we can do something like: appname.namespace.systemdomain (separated with dots). With system domain being a wildcard one, this could work.

This will not work properly unless you do special magic to request cert from letsencrypt for another wildcard domain for *.namespace.systemdomain. Knowing user/operator can create alot of namespace... this can be problematic. Need special magic to get certmanager to request cert.
Not sure if you want to do this...

If so, it seems like it is already working as the way i was suggesting... Then why would need the fix for a random string behind the app again ? Cos nobody try to "hijack" the default epinio API url because it should fail like above (unless epinio is NOT an epinio by definition?)

History. The check to make it work like you are suggesting is something which was very recently added. The random string suffix was likely added much earlier.
... Searching ... Huh. I have to correct myself here. The PR https://github.com/epinio/epinio/pull/1640 added both the check and the suffix.

And re-reading comments and review comments I see https://github.com/epinio/epinio/pull/1640#discussion_r919774755 where I commented on the suffix, although only regarding the technical side of the change. I apparently did not question the need.

And re-reading comments and review comments I see #1640 (comment) where I commented on the suffix, although only regarding the technical side of the change. I apparently did not question the need.

Hi @andreas-kupries, not sure what is the resolution here ? I would advocate to remove the need to add a random string post-fix unless it is an deliberate action eg: --random-route (actually, similar to CF)
One of major reason is that most of us will use cli for automation... if the route is taken... then error out with appropriate  message is actually good.
Just my thoughts

Maybe we can do something like: appname.namespace.systemdomain (separated with dots). With system domain being a wildcard one, this could work.

This will not work properly unless you do special magic to request cert from letsencrypt for another wildcard domain for *.namespace.systemdomain. Knowing user/operator can create alot of namespace... this can be problematic. Need special magic to get certmanager to request cert.
Not sure if you want to do this...

This is already the case. For every application, if a matching certificate doesn't already exist, Epinio will try to create a new certificate with the configured issuer. It's only the route that will be different.

PR #1710 undos the hash postfix of the default route.


[msal-common] Enable Strict null checks

Current Configuration:
        "strict": true,
        "strictNullChecks": true,
        "strictFunctionTypes": true,
        "strictPropertyInitialization": false,
        "noImplicitThis": true,
        "noImplicitAny": true,
        "alwaysStrict": true,
        "noUnusedLocals": true,
        "noUnusedParameters": false,
        "noFallthroughCasesInSwitch": true,

Coverage remained the same at 79.29% when pulling 8b779e1ec5e316a5e709ea2ef0b290ddc8a18545 on strict-null-checks into b9e06ede2ca404df3a18872b5a77fe0168a5e2a4 on dev.

In what situations is this value null?

@jviney @josmithua I believe this is because in code, the value set for account is initially set to null and then dynamically filled in after processing the response.

In what situations is this value null?

@jviney @josmithua I believe this is because in code, the value set for account is initially set to null and then dynamically filled in after processing the response.

So it might still be null if it fails to process the response? Or will an exception be thrown? If the account property will never be null when returning from the function then I think the interface should reflect that

So it might still be null if it fails to process the response? Or will an exception be thrown? If the account property will never be null when returning from the function then I think the interface should reflect that


fix(client): not includes never and change generics

Closes https://github.com/honojs/hono/issues/3122
The author should do the following, if applicable

[ ] Add tests
[x] Run tests
[x] bun run format:fix && bun run lint:fix to format the code
[ ] Add TSDoc/JSDoc to document the code

hi @yusukebe
this pr contains the following two changes.
1.  Not includes never
This issue is involved only when the middleware is chained by .use.
When we mark * with middleware, the Schema value contains never.
{
  "*": never
} & {...}

This will also be suggested by hono client as a route.
This is a serious UX concern.
However, this PR does not solve the problem of overriding the handler's schema in the middleware path when defining middleware and handler in the same path.
(Instead of using & when creating Schema, I can suggest creating and using the type mergeSchema.
I have already built this type.)
2. Change Generics of createHandler/createMiddleware
Previously, Path was marked as any by default, so if you did not explicitly mark Path as any, it was marked as never when any was on Schema.
So I marked at ""

It is working now.

Maybe the two changes should be in different PRs.
If desired I will do so.

hi @yusukebe
This PR was half resolved by https://github.com/honojs/hono/pull/3128
but I think "1" is not resolved

@EdamAme-x
That is another issue.

ok @yusukebe
I'll create new issue

@EdamAme-x
Don't do that. I'll do it.


Add Temp Logo To Repo

Made a temporary logo to the repo header. I'll make a proper one at a later time. But this'll be an identifier for this repo in the meanwhile.
Just a simple idea for now .. literally dreaming prompt to image.

There's also a color variant but I'm not too happy with it at the moment. Because this repo is still in development, I felt the black and white logo will indicate that too. Once we're in full flow, I'll make a new one with more thought and time put into it.

@blessedcoolant , I was thinking that we're reaching a point where we could have the "proper" logo  you talked about in the original PR. Are you still up for working on it? I'd also like to see a favicon for the web interface.

@blessedcoolant , I was thinking that we're reaching a point where we could have the "proper" logo you talked about in the original PR. Are you still up for working on it? I'd also like to see a favicon for the web interface.

I'm a little swamped with work for a week or so but I'll try to see if I can work something out.


Matplotlib widget not rendering in Notebook output

Environment data

VS Code version: VS Code Insiders
Jupyter Extension version (available under the Extensions sidebar): Latest Pre-release version
Python Extension version (available under the Extensions sidebar): 3.8
OS (Windows | Mac | Linux distro) and version: WSL
Python and/or Anaconda version: Python
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): VIrtual Env
Jupyter server running: Local

Expected behaviour
Widget renders, and is completely visible.

Exact same code in VS Code stable (same version of Python & Jupyter)

@rebornix looks like we'll be shipping something in VSCode insiders, that will break widgets (break = not rendering in a usable manner)

@DonJayamanne Do you see this in an otherwise empty jupyter notebook with that cell? I just tried but could not reproduce the problem

Do you see this in an otherwise empty jupyter notebook with that cell?

Yes, thats what I've done

Any errors?

I can't get it to work at all, in environments where I'm sure I've used this stuff before: ModuleNotFoundError: No module named 'ipympl'

You'd need to install that python -m pip install ipympl or in the cell type %pip install ipympl

FYI - just tested outside WSL and that works

Yes, confirmed works outside WSL, but in WSL it doesn't work in VS Code insiders even in different workspaces in WSL (different folder names).
But exact same notebook works in WSL when using VS Code stable.

Works in ssh. Can you check for errors in devtools, or look at the DOM? Maybe something related to the webview in a remote window.
It looks like it managed to render something though, so I'm a little confused, like if the webview was failing to load scripts it would be worse

Let me try to see if I can repro

Works for me in WSL

Errors when opening this notebook (however i get this same error in Stable as well)

No errors after running the cell,

I manually adjusted the height of the widget container and this is what I can see
I think the whole widget didn't get rendered as the container was too small (or some other issue)

Works for me in WSL

Weird, fails on insiders for me and works on stable, consistently.

@rchiodo this is my version of VS Code insiders, please could you compare against yours
Version: 1.65.0-insider (user setup)
Commit: 3112460dc48ce7e557ea9feeaae04b912164b48b
Date: 2022-02-24T05:15:20.989Z
Electron: 16.0.9
Chromium: 96.0.4664.174
Node.js: 16.9.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22000

Version: 1.65.0-insider (user setup)
Commit: 3112460dc48ce7e557ea9feeaae04b912164b48b
Date: 2022-02-24T05:15:20.989Z
Electron: 16.0.9
Chromium: 96.0.4664.174
Node.js: 16.9.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19044
Yeah same

Maybe the script we download for the widget is out of date? Mine asked to download.

Yes you are right, weird, a new version must have been uploaded in the past day (the scripts are in the extesnsion folder, which gets blown away with each new version of insiders).
Closing this issue.
Thanks,


Unintended Database Write in Radzen Inline Edit due to Multi-Row Editing

Describe the bug
The Radzen Inline Edit functionality currently allows users to edit rows without triggering the necessary save action. When editing one row (withouth implicitly saving) and subsequently clicking the edit button for another row, the system temporarily saves the first row's state, even though it is not explicitly saved by the user. Upon saving the second row, both the edited values of the first and second rows are written to the database, leading to unintended database writes. This behavior allows for a potential bypass of logic, as both values are sent to the database instead of only the intended row.
The issue was initially identified in our own blazor server project, and it can be fully reproduced using the inline edit example provided on the Radzen website.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://blazor.radzen.com/datagrid-inline-edit?theme=material'
Click on 'Edit Source'
Paste the following code RadzenBug- InlineEdit UpdateRecord eventhough no save.txt
This is the same example code as the base inline edit with a removal of the add/remove methods since we only want to update records and not add or delete them.
Click on 'Run' (It now should load the example from the example tab. incase it doesnt manually click on the 'Example' tab in the header)
Modify the value in the first row (e.g., Ship Name) but intentionally refrain from clicking the 'Save' button.
Without saving the changes in the first row, click the "Edit" button for a different row in the datagrid (e.g., the second row).
Modify the value in the second row and proceed to click the 'Save' button.
Observe that both the edited values from the first and second rows are written to the database, even though the changes in the first row were not explicitly saved.
Rerun the code again without refreshing the page to observe that the first row that wasn't explicitly saved also persists in the database data, even though we didn't explicitly save it. (Note that refreshing the example page wont work since it would bring the code back to its original unedited state)

Note: The observed behavior in the example is consistent with our own application, which utilizes a MSSQL database.
Expected behavior
The expected behavior is that when editing a row using Radzen Inline Edit, changes made to a row should only be saved to the database if the user explicitly clicks the 'Save' button. In the described scenario, modifying the value in the first row should not result in the system saving those changes unless the user actively clicks 'Save' for that specific row. Subsequently editing another row and clicking 'Save' should only save changes for the edited row, not for any other rows that were previously edited but not explicitly saved. The behavior should align with the intended actions of the user, preventing unintended database writes for rows that were not explicitly confirmed for update.
Screenshots
Below, you will find the first four screenshots, each corresponding to the steps taken to reproduce the bug. Screenshots from point 5 till 8 depict the reproduction of the bug in our own application.

Original state before editing.

Edit of the first row without implicitly saving it.

Edit of the second row and saving of the new value using the save button.

New state after saving the second row.

Upon rerunning the code, it becomes evident that the first row retains the edited value, despite not being explicitly saved. This persistent behavior is also consistently replicated in our own Blazor server application.

Original state of bools

Changing all bools except last to true withouth implicitly saving the first 5 rows.

New state after only saving the last row and reloading the data from our database

Desktop (please complete the following information):

OS: [Windows]
Browser [Chrome, Safari, Edge]
Version Radzen example site [Latests]
Version Own project [Radzen 4.21.3]

Hey @Shinigami9000,
This is just an example that can be adapted to your scenario in the way you need - not something part of the component itself.


Need a general database API generation module

We have a number of different objects that we keep generating essentially the same database API boilerplate for each time a new object is invented. This is a problem because it's inefficient and error prone. We should come up with a solution to this problem.
Note that this problem or some variant of it is very traditional database/API problem where the solutions have ranged from introducing a different set of difficult problems to solve or causing even more headaches at worst. For example, we do not want to introduce or use an ORM system. The problems with these are myriad -- first and foremost that eliminate any semblance of fine grained control and hide the levers to work with your data in undesirable ways.
We could try and unify our database storage and create an abstraction layer that ends up treating everything as a giant graph (or dataset of graphs) of information, but this is also difficult to manage.
Perhaps the simplest thing we can do at this time is create a module that will auto-generate the basic APIs  (and permissions, etc.) we want and allow overrides in individual modules as needed.
We have similar data access problems on frontends as well -- but we may have more flexibility with finding a solution there (it's unclear at this time).
cc: @msporny, @mattcollier, @davidlehn

Is this type type of thing you do not want to use?  https://loopback.io/

@mattcollier, to be clear -- we're not talking about REST APIs in this issue, but backend JS APIs.

Also, we're not looking at a unify everything, it all works the same, type solution either because those have been found wanting every time they have been tried. They fall under the "just create a different, new set of difficult problems to solve/eliminate fine grained control" set of solutions.

I think we want to preserve a modular approach to this problem, but just make it so we don't have to hand code so much of those modules. Going farther than that, at least now, is probably too risky given past failed attempts at doing so.

Just looking over looback.io for the first time now, but it does have the model layer which includes both REST and node APIs to access data http://loopback.io/doc/en/lb3/Defining-models.html

@mattcollier, yes, but it's essentially the same sort of top-down, ORM + lots of abstraction approach that other frameworks provide. The basic design is to say "There are only four things you are ever going to with anything in your system, so just tell us what you want to store and we'll give you your four buttons." These work great for small, static, simple systems or for large static systems with very simplistic interactions, but tend to generate more pain than they're worth otherwise.

Closing this due to inactivity.


UTF8 Normalization

System OS
Docker
Python Version
3.11 (CPython)
Install Source
Other
Install version / commit hash
Docker
Expected Behavior vs Actual Behavior
Duplicate files should download just once, not twice - suspect a UTF8 normalization issue, for example I have the following two files twice:
Nikolauss, John O'Callaghan - Bláithín.m4a
Nikolauss, John O'Callaghan - Bláithín.m4a
Steps to reproduce - Ensure to include actual links!
Download playlist, receive duplicate files in directory.
Traceback
No error, just a normal download.

Other details
Wondering if there's a way to normalize filenames on save?

Still an issue on my end…


Why is Kap asking for permission to access local servers?

When I open Kap, or press that crop button, Kap is opening a dialog (5 times) to login to local servers.
This is most likely as a result of favouriting those servers in Finder.
Platform: MacOS 10.12.1
Kap Version: 1.0.0 (<IP_ADDRESS>)
Steps to reproduce: Open Kap with Servers currently disconnected in Finder Sidebar Section – "Favourites"
Current behaviour: Kap opens permission dialog (5 times) to access favourited servers not currently connected.
Expected behaviour: Don't ask to access personal server, or ask permission for unnecessary network locations.
Workaround: N/A (Unable to use app)
Screenshot:

That doesn't sound like much fun, I'm sorry! Have you had this issue in other applications? Is Kap located on the same HD as your OS, or on an external disk?

Nup, Kap is in /Applications/, and not on an external disk.

It's really weird, it's not something we're purposefully asking for. Do you have this issue in other Electron apps, Atom, for example?

It's happened with one other electron app before, wasn't Atom though.
The issue has disappeared. Worth investigating if someone else has the issue.

If the issue was also present in another Electron app I'm guessing the issue is upstream.
I'm on mobile right now so I can't check myself, but are there any issues describing this behaviour on the Electron repo?

This started happening recently on one of our editing machines as well except it throws the following error: "There was a problem connecting to the server "xxxxx". The server may not exist or it is unavailable at this time. Check the server name or IP address, check your network connection, and try again"
This only happens when you click the Crop button which then doesn't allow you to ever record.

@t-ack3 @sifex Please test the latest Kap beta: https://kap-beta-updates.now.sh A lot has changed since v1.


Bumps in ionic : ci_matching_branch/bump_ionic_gz-physics8

See https://github.com/gazebo-tooling/release-tools/issues/1027

@osrf-jenkins run tests please


⚠ Bofrost.si has degraded performance

In 9d18d54, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 3385 ms

Resolved: Bofrost.si performance has improved in 031a1b3.


Problem with offline receive

Create an invoice in Misty Breez
Scan with Breez Mobile
Force close Misty Breez
Approve the payment on Breez Mobile

The payment starts to execute but it is stuck pending in Breez mobile until I open Misty again
When I open Misty the payment shows as sent in Breez mobile then becomes pending on Misty and is completed in about a minute.

Force closing an app behaves differently from manufacturer to manufacturer. It can prevent allocating resources to the application to run in background to process the payment. Do you experience the same issue when application is in the background?

Same behavior when in the background

Please share logs to help us investigate the notification plugin.

Receiver logs Misty:
l-breez.logs (8).zip
Sender logs Breez mobile:
breez.logs (38).zip

Fixed with

#190


Replace Trip type with Sector type

Sector is the correct name for this type according to kiwicom glossary

rename the React components to be tripsector

Will do it next PR


Flex: modify gap to support row/column values

Summary
This PR modifies Flex's gap prop from 0..12 to 0..12 | {| row: 0..12, column: 0..12 |}. This allows the user to specify spacing for the both the row and column directions, which is not currently possible.
Why?
When I first implemented Flex, I frankly misunderstood the spec and implemented it incorrectly. I didn't realize that providing a single value for gap is actually shorthand for row-gap and column-gap, not just a single value. So that implementation only worked with a single row or single column of items. If the items wrapped, there was no way to specify the spacing between rows, leading to squished layouts and forcing the user to use Box instead (and implement the children spacing manually 😞).
The original implementation is fine when wrapping isn't an issue:

…but is no bueno when the items wrap:

The new implementation allows for the vertical spacing to let this layout breathe:

Codemod
yarn codemod --parser=flow -t=packages/gestalt-codemods/64.0.0/flex-gap-expansion.js relative/path/to/your/code

A NOTE ABOUT THE CODEMOD:
From now on, gap={3} will be equivalent to gap={{ row: 3, column: 3 }}. This matches the spec, so it's the best choice going forward. But that's not how the original implementation worked, so we actually to codemod something like <Flex direction="row" gap={3}> to <Flex direction="row" gap={{ row: 3, column: 0 }}>. This is ugly and unfortunate, but we can't assume that adding the missing spacing won't break layouts. In the future, most Flex usages will probably just use the shorthand, but for now we need to replicate the current behavior. Blame 2020 Ryan for that. 😬

@dangerismycat curious, did we also consider using rowGap and columnGap as props instead of passing in an object to gap? Seems like it would be more consistent with what we do with e.g. paddingX and marginTop

@dangerismycat curious, did we also consider using rowGap and columnGap as props instead of passing in an object to gap? Seems like it would be more consistent with what we do with e.g. paddingX and marginTop

Not really — that would allow for the weird state of <Flex gap={2} rowGap={3} columnGap={4}>, which isn't great. I'm not a fan of the padding/margin props either but also not eager to go through the pain of refactoring them 😅  For gap at least, I wanted to guide users to a Pit of Success rather than allow nonsensical states.


Revert "XFAIL Kickstarter and ReactiveSwift targets affected by SR-13…

…548"
With https://github.com/apple/swift/pull/33946 and https://github.com/apple/swift/pull/33955 applied, I no longer see any !hasValidInsertionPoint assertion failures when building the source compat suite locally.

@swift-ci Please Test Source Compatibility

@swift-ci Please Test Source Compatibility

@swift-ci test


Adding a fourth li seems to break my example

I've inherited the maintenance of a site that uses query-scrollbox that has three li items.
I added a fourth by copying one of them, and changed the eq to eq(3)
but it does not behave well.
I'm sure I missed something easy.
Any clues would be greatly appreciated.

I don't know what do you mean by 'changed the eq'? Maybe this action is not required. Could you give any example for that?

I've since discovered that there are two instances that need to be addressed: the script and the list item itself. I just overlooked that.  thanks for checking


require environment pause before deletion

Currently an environment can be deleted at any time. It seems to me that since the deletion is non-atomic the orchestrator might start misbehaving. I think it would be better to only allow delete while the environment is paused. It should not be allowed to unpause an environment when a delete is in progress. The same for deleting an active model version.

Conclusion from planning meeting:

[ ] delete first halts the environment
[x] clear just stops the agents (already implemented)
[ ] environment can not be resumed while deleting
[ ] reject deletion of active model version

This ticket should also makes the necessary changes to pytest-inmanta-lsm. Currently halting and clearing of an environment is configurable using config options. With this change in-place some of these combinations might not be possible anymore, when running against a sufficiently new version of inmanta-core


Implement a light CLI to retrieve rainfall-related data

Title says it all...
Not a user-friendly feature, but could be nice for retrieving some data fastly.
For instance: python cli.py rainfall:average 1970 1999

Potential libs for that:

https://typer.tiangolo.com/
https://click.palletsprojects.com/en/8.1.x/


Add template janitor sanity check / enhancements

Codecov Report

Merging #9 (1639013) into main (e513bb8) will decrease coverage by 3.70%.
The diff coverage is n/a.

:exclamation: Current head 1639013 differs from pull request most recent head dc1021c. Consider uploading reports for the commit dc1021c to get more accurate results

@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
- Coverage   77.77%   74.07%   -3.71%
==========================================
  Files           3        3
  Lines          36       27       -9
  Branches       19        0      -19
==========================================
- Hits           28       20       -8
  Misses          7        7
+ Partials        1        0       -1

Flag
Coverage Δ

Linux
?

Windows
74.07% <ø> (-5.93%)
:arrow_down:

macOS
?

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

Impacted Files
Coverage Δ

src/main.cpp
22.22% <0.00%> (-20.64%)
:arrow_down:

test/tests.cpp
100.00% <0.00%> (ø)

test/constexpr_tests.cpp
100.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

Adding remaining tests for #3

@lefticus not sure if you are working on the feature to turn a project into a template using the github cli. If so i wanted to make you aware that i see a potential race condition between the janitor and the creator.
Either both have to be aware of the special meaning of the suffix or the janitor has to call the other action after turning the project into a template.

@LtdSauce I wasn't going to do the _template feature right now (not sure if it makes sense or not, honestly)
What I have attempted (and seems to be working)

Test to make sure project is still configurable
automatic update of template-names in the case that the project is a template, but the name of the project has changed.

That is:

You can fork, and it will change the names in the main README to match the names of the new template-name
you can "use template" and it will do everything it was already doing

This should help with maintenance when creating new forks

@LtdSauce I think this is ready now, your review would be greatly appreciated.

@LtdSauce I wasn't going to do the _template feature right now (not sure if it makes sense or not, honestly)

I guess forking and renaming it afterwards is fine. The suffix "hack" would make it a little bit easier to fork as the renaming step is directly done.
I'll try to have a look at your changes tomorrow.

I guess forking and then renaming it afterwards is fine. The suffix "hack" would make it a little bit easier to fork as the renaming step is directly done.

With what I did, it should track the name changes whenever they happen. So it will track the forked owner name, then if you rename the fork, will track the new project name... if I did it right.

I guess forking and then renaming it afterwards is fine. The suffix "hack" would make it a little bit easier to fork as the renaming step is directly done.

With what I did, it should track the name changes whenever they happen. So it will track the forked owner name, then if you rename the fork, will track the new project name... if I did it right.

Hmm... tracking the project name seems reasonable. But i see one caveat with the owner. Contributing to the template by forking and then opening a PR will always have an unwanted diff because the org/owner will be replaced in every fork. Maybe the renaming should only happen when the project name changes?

Hmm... tracking the project name seems reasonable. But i see one caveat with the owner. Contributing to the template by forking and then opening a PR will always have an unwanted diff because the org/owner will be replaced in every fork. Maybe the renaming should only happen when the project name changes?

That's a valid point ... but in a normal fork the badges will always be broken otherwise.
Hmm..
The check to commit is here, so it's easy to adjust for: https://github.com/cpp-best-practices/cpp_boilerplate_project/blob/template-janitor-testing/.github/workflows/template-creator.yml#L78

On the other side... After the PR gets merged the create workflow will fix the badges again...
But i think it is okay for a contribution fork to have the badges point to the root repo. At least i only look at the github actions directly triggered by my commits. Futhermore, the forked project has to be created at codecov and lgtm seperately. Which i do not think a regular contributor will do, don't you think?

@LtdSauce I believe I have addressed all of your comments.

@LtdSauce I believe I have addressed all of your comments.

@lefticus you did. But my review was incomplete :/

I meant i missed something. I finished my review. And now i think everything is covered :)

@LtdSauce I had a busy week last week and will be digging into this today.

I just noticed that the bug is because of an actual bug in the README.md file. I think fixing the README will fix the underlying issue, and I'll start with that.
I avoided the suggestion you make because it breaks other things I wanted to preserve in the readme, such as the link over to cpp_starter_project

@LtdSauce I added automated testing with test names (which I kind of regret now, because it took so long) and tested it all here: https://github.com/ColoradoPlusPlus/test_boilerplate_project/pull/1/commits/0b34d3b9559e4adeee209b04b6a21fc3cafdf4db
It appears to be working in all paths now.

considering the size of this diff, it has to be the most vetted and discussed PR ever between just two people. Thanks @LtdSauce


Add subtitle support?

I want to see something like song lyrics to play when the voice is played.

Could you better describe what you're imagining? Potentially mock up a visual?

Absolutely great suggestion! I would also love to have subtitles

Could you better describe what you're imagining? Potentially mock up a visual? The read text is actually already held in the soundData table, so its just a matter of displaying it.

I would suggest displaying it like regular subtitles in a movie. 1 or maximum 2 sentences at once. With option to activate or deactivate them.

Also the current playback box is already set up to be very similar how the addon Immersion does its text, that would be my first choice on how to display them. It might even come as a side product of implementing #37 with all the text splitting and syncing, just requires adding the ability to have VoiceOver do it for those of us who can't (1.12.1) or don't want to use Immersion.

Yeah, it would be like the talking head frame like Retail.

This is a huge thing to take on and do correctly. It would add value, but I dont have the bandwidth to take it on anytime soon.
There was some talk a while back in the discord about generating offsets of the spoken text from the audio files using that to sync to the shown text. The small addon window that shows can't fit the entire quest into it at the same time. If you're looking to see the static text in its entirety, you could open the quest log and follow it yourself.
Maybe someone could make a proof of concept where the text isnt synced to the audio, but is based on text length.

This issue is related to #37. They're both valid approaches for creating subtitles.


Issue 51021: WebSocket to add window focus listener

Rationale
https://www.labkey.org/home/Developer/issues/issues-details.view?issueId=51021
We previously added code in the apps so that on browser window focus we do a whoAmI check via WebSocket. checkForExpiredSession() so that we can reliably show the session timeout / logout modal to the user. This PR moves that approach to the WebSocket.js file directly (note that I will remove the app specific code in develop).
Changes

add window focus listener to help identify session timeout and user logout more consistently session timeout and user logout more consistently

Is the problem that the web socket connection gets cut off when the tab/window doesn't have focus, so we don't get the notification that the session is done?

Is the problem that the web socket connection gets cut off when the tab/window doesn't have focus, so we don't get the notification that the session is done?

I'm not really sure. The previous iteration of this WebSocket.js code didn't seem to have a problem with getting notified of the session timeout event when the tab/browser was not in focus. Not sure if the change was related to embedded tomcat, browser versions, etc. but this approach of checking for whoAmI on tab focus is something we've been doing on the app side React code for awhile now.


Activated Snapshot does not Appear under "Active Snapshot"

 Issue by demory
Wednesday May 02, 2018 at 17:38 GMT
Originally opened as https://github.com/catalogueglobal/datatools-ui/issues/102

After clicking "Make Active" for a snapshot, the data is loaded into the editor but the "Active Snapshot" panel remains empty:

:tada: This issue has been resolved in version 4.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


std.uni unittest examples

this PR removes a lot of comment examples and replaces them with unittest examples

What's going on with the autotester failures?? They seem to be linker errors?

I have no idea, when I'm demangle the undefined reference correctly, it doesn't anything about std.uni

Get rid of "cute" commit messages please. And no white-space only commits, of course

@DmitryOlshansky I'm currently using the autotester to figure out why it fails linking. When I fixed that. I will squash all commits into one.

And I salute the brave - narrowing down imports in std.uni is something I tried in the past only to see obscure forward reference bugs. The reason is that uni sits deep in Phobos and even std.format uses uni, which is in turn used by std.typecons.Tuple and... you see where it goes.

Autotester is passing now. Squash?

I made the failing unittest a doc example again. I can't get it to link on the autotester.
It guess it is ready to merge

NVM - I see them outside the diff.

Auto-merge toggled on

thanks


Fix I18n lookup for enum values in nested select fields

This PR accomplishes the same as #1227, albeit with a more focused approach.
When constructing the key for looking up the localized enum value, we now use ActiveModel::Name.i18n_key instead of object_name, which returned something like user[posts_attributes][0] in case of nested forms.
I've added a test that reproduces the exact scenario that triggered the fixed bug (#1151).
Bonus: I've added a commit to update Formtastic version in Gemfile.lock (it was still 4.0.0.rc1).
Fixes #1151
Supersedes #1227

@mikz @justinfrench Any plans to review/merge this in the near future?

Thanks!
Will there be a new release anytime soon? I think that there have been some interesting fixes since 4.0.0 that may be worth a patch release.

@Fs00 indeed, but as we are not doing many there is always some element of fear included :)
I'd recommend pointing bundler to the git repo in the meantime.

Oh well, less changes, less fear! 😄 I think that waiting will only make things more risky and difficult.
Anyway, I cannot point Bundler to the Git repo, since I use this gem in an enterprise environment where dependencies can't be pulled from the "outside world". I'll wait patiently 🤷♂


Application could not be started, or no application associated with the specified file.

Describe the bug
Fusion360.exe does not exist in any installs, only FusionLauncher.exe does.
I have ran the installer countless times but this issue never resolves.
To Reproduce
Steps to reproduce the behavior:

Install
The launcher doesn't work in the first place, so you must change "LAUNCHER_RUN_FUSION360" to "LAUNCHER="$(find $HOME/.fusion360/wineprefixes/default -name Fusion360.exe -printf "%T+ %p\n" | sort -r 2>&1 | head -n 1 | sed -r 's/.+0000000000 (.+)/\1/')" && WINEPREFIX="$HOME/.fusion360/wineprefixes/default" wine "$LAUNCHER""
Run
Errors out

Expected behavior
Run Fusion360.exe
Desktop (please complete the following information):

Version number of Autodesk Fusion 360: 2.0.13168
AMD ATI FirePro W5100, AMDGPU-pro
Arch Linux, Linux 5.18.2
DXVK and OpenGL

It is likely happening because of installation getting killed before it is completed as the 3 minute time may not be enough for your PC. The installer needs start and stop due to #232
Try this :
This will basically make the installation script wait for 10 minute instead of 3 minute.

Before executing the installation script, we need to edit it. So only run the starting part
mkdir -p "$HOME/.fusion360/bin" && cd "$HOME/.fusion360/bin" && wget -N https://raw.githubusercontent.com/cryinkfly/Autodesk-Fusion-360-for-Linux/main/files/builds/stable-branch/bin/install.sh && chmod +x install.sh
Open the install script in gedit
gedit ./install.sh
Edit the line 380 to from sleep 3m to sleep 10m

  function SP_FUSION360_INSTALL_STOP_1 {
      sleep 10m
      kill -9 $(ps aux |grep -i '\.exe' |awk '{print $2}'|tr '\n' ' ')
  }

Now execute installation script


Modify the cluster-registry repo to enable checked-in BUILD files

This should complete the work necessary for #24 and unblock vendoring of this repo.
Only the last two commits (starting with "Modify the cluster-registry repo to enable checked-in BUILD files." are meant to be reviewed here). See #58 and #61 for the reviews of the other commits.
/assign @madhusudancs @pmorie @ericchiang
/cc @font

/lgtm


Speed up pagination for pit_crew

The default number of models that fuel returns is 20. This PR makes the number of models returned per page 5x larger, meaning fewer calls are made, and pagination completes much faster.
Setting the per_page to higher than 100 causes fuel to reject the arg and drop you back down to 10 models per page.
This mostly impacts people building from scratch, since the greatest impact is on initial building of the model cache.

PS: I don't know why DCO is failing even though I have a signoff.

Looks good! It seems the DCO expected @google.com but got @gmail.com instead, but I think that's OK.


428 document all dependencies in a single doc in eo phi normalizerdata for each eo version

Closes #428

PR-Codex overview
This PR updates file paths and function names, replaces hardcoded values with variables, and adds a new function to write dependencies in Markdown format.
Detailed summary

Updated file paths and function names
Replaced hardcoded values with variables
Added write_dependencies_markdown_for_eo_version function
Added write_dependencies_markdown function
Updated dependencies.md files in eo-phi-normalizer/data/0.37.0 with detailed program dependencies.

The following files were skipped due to too many changes: eo-phi-normalizer/data/0.37.0/dependencies.md, eo-phi-normalizer/data/0.38.0/dependencies.md, eo-phi-normalizer/data/0.38.4/dependencies.md, eo-phi-normalizer/data/0.36.0/dependencies.md

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@fizruk Thanks for the review! You've earned +37 points for this: +25 as a basis; +7 for 75 hits-of-code; +5 for 0 comments.  Your running balance is +70.


Update to LLVM 28b27c1b10ae8d1f5b4fb9df691e8cf0da9be3f6

This PR updates onnx-mlir to LLVM commit 28b27c1b10ae8d1f5b4fb9df691e8cf0da9be3f6.
This is the same LLVM commit level that torch-mlir is currently at.

Thanks @mikeessen! LGTM

@philass Thanks for the review!

@jenkins-droid  test this please

Jenkins Linux s390x Build #13480 [push] Updates to move to LLVM ... started at 16:32

Jenkins Linux amd64 Build #13453 [push] Updates to move to LLVM ... started at 15:32

Jenkins Linux ppc64le Build #12474 [push] Updates to move to LLVM ... started at 16:41

Jenkins Linux amd64 Build #13453 [push] Updates to move to LLVM ... passed after 2 hr 2 min

Jenkins Linux s390x Build #13480 [push] Updates to move to LLVM ... passed after 2 hr 29 min

Jenkins Linux ppc64le Build #12474 [push] Updates to move to LLVM ... passed after 3 hr 8 min


How to get sub-workflow failed exception.

Hello,
As your example in #17 , TaskError is raised because activity failed.
I try to implement subtask retry for sub-subtask failed, but I got SuspendTask, not TaskError.
Do you have any suggestion to implement subtask retry ?
Is it possible to get TaskError from sub-subtask failed ?
Thank you.
jinkuen

Hello,
Yes, subworkflows should raise TaskError exceptions just like activities. Make sure you're not wrapping the subworkflow call itself in try-except but only the result() call:
Don't do this:
try:
    r = self.subworkflow(...)  # this can raise SuspendTask
except TaskError:
    pass

Do this:
r = self.subworkflow(...)
try:
    x = r.result()
except TaskError:
    pass


fix: better stories for popover to eliminate styling confusion

The existing popover stories introduced some styling confusion as the "customized" example looked too much like the example that uses the standard <PopoverHeader />/<PopoverBody /> combination.
This makes an explicit, unambiguous customized example, and leave the standard ones fairly plain.

https://trello.com/c/5cDnH4iS


[DOC] core dev onboarding and offboarding instructions

We should write a list of actions required to onboard core developers, and a process. E.g., that actions that are not the onboardee's ought to be carried out by the core developer who has proposed the new core developer, directly or indirectly (pulling in an admin).
Some onboarding steps that I can think of, below.
To do by an admin or core dev:

invite new core developer to #core-developers slack channel
give them core developer role on discord
give maintainer (not admin) access to this repo
add to core dev team here: https://github.com/orgs/sktime/teams/core-developers/members

To do by new core dev:

PR to team.rst, to add themselves

Offboarding should be the opposite of the above, I guess, with the slight modification that on team.rst the name moves to "former/inactive".

@khrapovs, you have onboarded recently. Is there anything the above list is missing?

FYI @aiwalter, @GuzalBulatova, @TonyBagnall, anything missing?


The name is very LOW!

Description of the issue:
Imagine this:
I create a project named PENIS. do you english people feel offended?

Steps to reproduce:
step 1: find a chinese man/woman
step 2: say 'jib' to he/she in face
step 3: Just waiting to be slap on face
Environment:
Any Chinese-speaking community
Additional Information:
insult
humiliate

兄die。你怕不是个杠精？？
人家英文单词jib在您这儿就成了ji'ba了？
恕在下直言，您这行为挺丢人的！！

While we appreciate your concerns with our choice of name, it is probably going to remain Jib.  If it is a large concern, perhaps you could use "java image builder" or "J-I-B", when talking about this product.

挺会挑刺，晚上来我这吃鱼吧。
真的是丢中国开发者的人！

你这个...怕不是杠精....丢人到国外来了..... -_-|| spring boot 还简写成sb呢...


Fix bugs preventing use in Firefox

Event Fix
Firefox throws an error whenever a user clicks the "Sign Up" button that reads "ReferenceError: event is not found." This is fixed by removing a reference to the undefined "event" in SignUpFormContainer. This doesn't throw an error in Chrome or IE because both have a global symbol for "event".  The SignUpForm container already handles preventing event defaults anyway.
UI Fix
Firefox causes an unwanted line break in the menu as shown in the image below. This is due to the right floated elements in the menu. A simple fix is to switch the menu elements so that the floated element comes first followed by the elements still within the flow of the page.

Good catch @baldwinjj !


Pass through timeupdate, seeked, and seeking events

This PR will use a CustomEvent for proxied events so it can attach a proxied flag on the event itself. Then we can detect if the event was proxied in the preventEvent and 'seeking' listeners to detect where the original event came from. Using this logic, we can safely passthrough timeupdate, seeking, and seeked events from the <audio> node to the consumer.
Please let me know what I can do to make this PR mergeable with your build-process.
Thanks!
Relates to #8, #57

Thank you! But as mentioned in #8:

I could also proxy them (and timeupdate) from audio element, but then what do I do with silent videos?

Given that the audio's seeking is triggered by exactly this point, I think the solution would be to just work around that to allow those events through.
timeupdate is already there and uses this technique. video[ಠevent] = true is how I allow an event through. That @event symbol for timeupdate is specified here and new symbols should be created to allow more events.
As for the built file, no need to include it :)

This doesn't work as well as I'd hoped. I'm closing it out.


popeye: init at 0.8.10

Motivation for this change
Add popeye, a Kubernetes cluster resource sanitizer at version 0.8.10. A big thanks to @mtrsk and the brazilian telegram group of NixOS for the help.
NOTE: I did not find a good way to pass the version information, so for now, they are locked directly in the package definition based on the information from the last release and its makefile, I also deactivated the tests, as it gave some sandbox-related errors and the expectation of a kubernetes environment. Perhaps in the future it will be useful to enable some of them.
Things done
Added package definition at pkgs/applications/networking/cluster/popeye/default.nix and added a reference line to pkgs/top-level/all-packages.nix below k9s.

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

@GrahamcOfBorg eval


linux_samus_4_11: init at 4.11.1

This patch contains an expression for linux-samus kernel [1].
I've been using it for a few days and had no major problems so far.
Specific lines in configuration.nix the one might want to add are the following:
services.xserver.wacom.enable = true;
services.xserver.synaptics.enable = true;

boot.initrd.kernelModules = [ "i915" ];

boot.extraModprobeConfig = ''
  options snd_soc_sst_bdw_rt5677_mach index=0
  options snd-hda-intel index=1
'';

The sound can be controlled by the following instructions (without -D pulse as in original instruction):
amixer -q sset Master toggle
amixer -q sset Master 5%-
amixer -q sset Master 5%+

I have a problem with suspension: the led doesn't change it's colors and the touchscreen stops working after wake-up.
mxt-app doesn't seem to help here, so I turned it off:
services.logind.extraConfig = ''
  HandleLidSwitch = ignore
''

PS. I don't think we should build it in hydra, since there is not so many users and the build process takes about one hour on this laptop.
PPS. And again, many thanks to @abbradar for assistance in this pr!
[1] https://github.com/raphael/linux-samus

Any reason not to include this pr? I think it would be nice to have some how-to on how to actually install NixOS with the kernel on the Pixel

I upgraded the kernel to 4.12. I only could tested it in a vm due lack of the original hardware.

You might want to add your nixos snippets to https://github.com/NixOS/nixos-hardware/


Invinsible Armor And Character Photon Shader With ShoulderSurfing

Description
My armos and character are invisible, and mostly look like a bug.
Steps to Reproduce
Downloaded The Mod
Ran It With Photon Shader
Minecraft version
1.20.10
Game Logs
https://mclo.gs/G2idqCK
Additional Information
I really hope you can fix this, Photon shader is really good with Patrix texture pack and your ShoulderSurfing mod, it really brings the perfect vibe for the modpack.

All the armors I used Enchanted Iron Helmet, Enchanted Iron Plate, Chainmail Leggings, Chainmail Boots

Duplicate of #219 and #199. Please refer to the FAQ section or this comment for a workaround.


Problem with Cuda and setup.py installation

Hi. Thank you for this great work. I have problem with this part of the installation:
PTv1 & PTv2 or precise eval
cd libs/pointops
usual
python setup.py install
and i get this error:
"No CUDA runtime is found, using CUDA_HOME='/usr/local/cuda'
running install
/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/cmd.py:66: SetuptoolsDeprecationWarning: setup.py install is deprecated.
!!
    ********************************************************************************
    Please avoid running ``setup.py`` directly.
    Instead, use pypa/build, pypa/installer or other
    standards-based tools.

    See https://blog.ganssle.io/articles/2021/10/setup-py-deprecated.html for details.
    ********************************************************************************

!!
self.initialize_options()
/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/cmd.py:66: EasyInstallDeprecationWarning: easy_install command is deprecated.
!!
    ********************************************************************************
    Please avoid running ``setup.py`` and ``easy_install``.
    Instead, use pypa/build, pypa/installer or other
    standards-based tools.

    See https://github.com/pypa/setuptools/issues/917 for details.
    ********************************************************************************

!!
self.initialize_options()
running bdist_egg
running egg_info
writing pointops.egg-info/PKG-INFO
writing dependency_links to pointops.egg-info/dependency_links.txt
writing requirements to pointops.egg-info/requires.txt
writing top-level names to pointops.egg-info/top_level.txt
reading manifest file 'pointops.egg-info/SOURCES.txt'
writing manifest file 'pointops.egg-info/SOURCES.txt'
installing library code to build/bdist.linux-x86_64/egg
running install_lib
running build_py
running build_ext
Traceback (most recent call last):
File "setup.py", line 19, in
setup(
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/init.py", line 108, in setup
return distutils.core.setup(**attrs)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/core.py", line 184, in setup
return run_commands(dist)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/core.py", line 200, in run_commands
dist.run_commands()
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/dist.py", line 970, in run_commands
self.run_command(cmd)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/dist.py", line 945, in run_command
super().run_command(command)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/dist.py", line 989, in run_command
cmd_obj.run()
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/command/install.py", line 92, in run
self.do_egg_install()
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/command/install.py", line 144, in do_egg_install
self.run_command('bdist_egg')
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/cmd.py", line 316, in run_command
self.distribution.run_command(command)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/dist.py", line 945, in run_command
super().run_command(command)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/dist.py", line 989, in run_command
cmd_obj.run()
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/command/bdist_egg.py", line 167, in run
cmd = self.call_command('install_lib', warn_dir=False)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/command/bdist_egg.py", line 153, in call_command
self.run_command(cmdname)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/cmd.py", line 316, in run_command
self.distribution.run_command(command)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/dist.py", line 945, in run_command
super().run_command(command)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/dist.py", line 989, in run_command
cmd_obj.run()
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/command/install_lib.py", line 13, in run
self.build()
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/command/install_lib.py", line 110, in build
self.run_command('build_ext')
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/cmd.py", line 316, in run_command
self.distribution.run_command(command)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/dist.py", line 945, in run_command
super().run_command(command)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/dist.py", line 989, in run_command
cmd_obj.run()
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/command/build_ext.py", line 93, in run
_build_ext.run(self)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/setuptools/_distutils/command/build_ext.py", line 359, in run
self.build_extensions()
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 434, in build_extensions
self._check_cuda_version(compiler_name, compiler_version)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/torch/utils/cpp_extension.py", line 808, in _check_cuda_version
torch_cuda_version = packaging.version.parse(torch.version.cuda)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/packaging/version.py", line 56, in parse
return Version(version)
File "/home/h4g/anaconda3/envs/pointcept/lib/python3.8/site-packages/packaging/version.py", line 200, in init
match = self._regex.search(version)
TypeError: expected string or bytes-like object"
Any advice is appreciated

Hi, no Cuda environment is installed (or exported) in your local environment.

how to fix this?

Currently, the most simple way to solve the problem is to install a conda environment with the following environment.yml
name: pointcept-torch2.3.1-cu12.1
channels:
  - pyg
  - pytorch
  - nvidia/label/cuda-12.1.1
  - nvidia
  - bioconda
  - conda-forge
  - defaults
dependencies:
  - python=3.10
  - pip
  - cuda
  - conda-forge::cudnn
  - gcc=12.1
  - gxx=12.1
  - pytorch=2.3.1
  - torchvision=0.18.1
  - torchaudio=2.3.1
  - pytorch-cuda=12.1
  - ninja
  - google-sparsehash
  - h5py
  - pyyaml
  - tensorboard
  - tensorboardx
  - yapf
  - addict
  - einops
  - scipy
  - plyfile
  - termcolor
  - timm
  - ftfy
  - regex
  - tqdm
  - matplotlib
  - black
  - open3d
  - pytorch-cluster
  - pytorch-scatter
  - pytorch-sparse
  - pip:
    - torch_geometric
    - spconv-cu120
    - git+https://github.com/octree-nn/ocnn-pytorch.git
    - git+https://github.com/openai/CLIP.git
    - git+https://github.com/Dao-AILab/flash-attention.git
    - ./libs/pointops
    - ./libs/pointgroup_ops


Update setting-up-your-phone-system.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/2549
Fixed link.

@nenonix @JohanFreelancer9 Copy editing is needed for this PR. Thanks.


Strange default flags on AllocationCreateInfo

The default trait implementation of AllocationCreateInfo adds DEVICE_LOCAL in the list of required flags of an allocation. This makes it impossible to create a CPU visible buffer using the usage parameter without also overriding required_flags.

I've fixed this in https://github.com/gwihlidal/vk-mem-rs/pull/4

Thanks, will merge and release


Add XAML Hot reload - change only

So in normal Visual Studio it's possible to have "change" only hot reload. Essentialy what it does I think it hot reloads one component on the page not just reloads the page.
For example: If the page is long and there is something at the and you hot reload on VS it doesn't scroll up, it just changes one thing only. On DotNet.Meteor it scrolls up probably because the whole page is reloaded.
Would it be possible to have this here?

Hi, @ziomek64 ! Unfortunately, .NET Meteor uses a full page reload because it can't detect an exact element that needs to be reloaded.

Hi, @ziomek64 ! Unfortunately, .NET Meteor uses a full page reload because it can't detect an exact element that needs to be reloaded.

Hmm, how does Microsoft doing that in Visual Studio 2022? Is this proprietary stuff?
What about their official VS code extension? Can't make it run tbh so couldn't test

Holy, actually official VS Code extension from MS has change only hot reload. Is the extension open source? Couldn't find the repo

To be honest, I do not know how it works in VS2022 or VSCode. It is not an open source. ILSpy won't even help me here, since there are a lot of DLL files and code there.

That's sad, because it really is a neat feature which makes UI development really quick. Perhaps label it as enhancement for further investigation in the future


TvLazyRow/TvLazyColumn scrolling issue

https://github.com/android/tv-samples/assets/43243372/fe1d34ed-6f94-46f9-ba0d-42ffefff2cff
Upon scrolling horizontally using TvLazyRow or vertically using TvLazyColumn, the items won’t snap into place immediately. There’s always an animation before snapping the items into place. This animation shows itself as a jumping action or a laggy behavior. Is there a way to deactivate this animation? Or, can the animation and snapping of items into place occur smoothly?

IssueTracker bug: https://issuetracker.google.com/issues/285746919

@dogancankilic I believe the animation performance is improved with the latest commit, and it shows less laggy behavior. Let me close this issue for now. Feel free to re-open when you find the screen shows laggy behavior.


Use Exificient to support EXI format for infosets

This is primarily an example of how to use EXI with Daffodil, although
an additional class was added to support the output of an EXI infoset.
This class may not be necessary with some refactoring and could instead
be replaced by passing in an EXI aware content handler, similar to how
we use an EXI aware XMLReader to support unparsing from an EXI infoset

I've made some cleanup changes for removing SAX from the CLI tool, but I didn't address the following from Mike B:
Ex: there should be no code in DataProcessor associated with particular SAX handlers, and no base classes or traits one must derive from in order to use our SAX API. I think the requirements were not clear, and SAX is for some reason fairly confusing. But there are things we need only for unit testing that are in the primary libraries.
Ex: Class DaffodilParseXMLReader and Class DaffodilUnparseContentHandler are defined as if they are part of our API, when they should just be under src/test.
DaffodilParseXMLReader and DaffodilUnparseContentHandler to me seem like they might be necessary as I believe they are responsible for translating the various SAX events into something that Daffodil understands.  I feel like these, or something similar, are necessary in order for someone to use Daffodil with SAX.  I feel like it shouldn't be required of someone using the API to have to define something for Daffodil to be able understand SAX events.
I do think the API could be smoothed out in this regard, where the DaffodilParseXMLReader and DaffodilUnparseContentHandler aren't really exposed by the API.  For example, when parsing we are currently happy to accept a users ContentHandler, but then the user needs to call DFDL.DataProcessor.newXMLReaderInstance to get the current processors XMLReader, and then set that XMLReader's ContentHandler to the one they are trying to pass into Daffodil.  To me this feels pretty clunky and could be hidden, but I haven't looked at this long enough to come up with a cleaner solution.

Binary data test files need to be added to the rat-check exclusions.


Add help for pkeyopt values for the genpkey commandline app.

Checklist

[x] documentation is added or updated
[x] tests are added or updated

Example output for
openssl genpkey -algorithm DHX -help

Usage: genpkey [options]

General options:
 -help               Display this summary
 -engine val         Use engine, possibly a hardware device
 -paramfile infile   Parameters file
 -algorithm val      The public key algorithm
 -verbose            Output status while generating keys
 -quiet              Do not output status while generating keys
 -pkeyopt val        Set the public key algorithm option as opt:value
 -config infile      Load a configuration file (this may load modules)

Output options:
 -out outfile        Output file
 -outform PEM|DER    output format (DER or PEM)
 -pass val           Output file pass phrase source
 -genparam           Generate parameters, not key
 -text               Print the in text
 -*                  Cipher to use to encrypt the key

Provider options:
 -provider-path val  Provider load path (must be before 'provider' argument if required)
 -provider val       Provider to load (can be specified multiple times)
 -propquery val      Property query used when fetching algorithms
Order of options may be important!  See the documentation.

pkeyopt values are
    type:string
    group:string
    priv_len:int
    pbits:uint
    qbits:uint
    digest:string
    properties:string
    gindex:int
    hexseed:string
    pcounter:int
    hindex:int

You do know, I hope, that you're kinda sorta reinventing the wheel?  Have a look at apps/include/app_params.h and apps/lib/app_params.c.
Of course, the output from those functions is bit more verbose, but I suspect it wouldn't be too hard to add a verbosity parameter.
Also, I would expect that this sort of help will find its way to all the other commands that use these sorts of opts too...  not asking for it in this PR, but I'd really like it if that was the intent, at least.

https://www.google.com/search?q=reinventing+the+wheel&client=firefox-b-e&sxsrf=AJOqlzWirg2zqy7PI3zZdrknVrMGFBlfuA:1673422900600&source=lnms&tbm=isch&sa=X&ved=2ahUKEwjIlLKMgr_8AhUMwjgGHUSVClcQ_AUoAXoECAIQAw&biw=1378&bih=619&dpr=2#imgrc=ZcomoXmbW4dgdM&imgdii=Ju5m33rWyILzZM

You do know, I hope, that you're kinda sorta reinventing the wheel?.

This is NOT trying to print out every possible param value like the list does.. Only ones that are actually usable by the command (e.g. OSSL_PARAM_UTF8_PTR is not something you can set from here).

Also, I would expect that this sort of help will find its way to all the other commands that use these sorts of opts too... not asking for it in this PR, but I'd really like it if that was the intent, at least.

Is there really that many of them? This particular one is the one I keep on looking up all the time.

Also, I would expect that this sort of help will find its way to all the other commands that use these sorts of opts too... not asking for it in this PR, but I'd really like it if that was the intent, at least.

Is there really that many of them? This particular one is the one I keep on looking up all the time.

I haven't looked thoroughly, but I suspect that all pkey commands (openssl pkeyutl, openssl pkeyparam, openssl pkey) do, and I'm pretty sure that the newer operation commands (openssl mac, openssl kdf) do.  Also, for example openssl dgst has things like -sigopt and -macopt, although I'm unsure how relevant that is.

Having the command I want to run produce a list is useful IMO.
Knowing that list can also produce such (but in a less convenient verbosity) isn't a totally natural step.  It ought to be but it isn't.

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

@hlandau ?

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

As @hlandau removed their stale review, this has passed the 24 hour threshold.

Merged to master. Thank you for your contribution!


security: fix CVE-2023-45288

This is a PRIVATE issue for CVE-2023-45288, tracked in http://b/319262343.
/cc @golang/security and @golang/release

@gopherbot please open backport issues.

Duplicate of #65051


Add Wasm bindings for CoreDocument

Description of change
Create wasm bindings for

CoreDocument
Service -> CoreService
CoreDIDURL
VerificationMethod -> CoreVerificationMethod

Links to any relevant issues
Type of change
Add an x to the boxes that are relevant to your changes.

[ ] Bug fix (a non-breaking change which fixes an issue)
[x] Enhancement (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Fix

This looks good to me, but I think there should also be a second review.


Minor fix to incorporate integration into QuTiP

Fixed import call to account for QuTiP integration.

Codecov Report

Merging #65 (04855ac) into master (50c32f1) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #65   +/-   ##
=======================================
  Coverage   93.33%   93.33%
=======================================
  Files           5        5
  Lines        1291     1291
=======================================
  Hits         1205     1205
  Misses         86       86

Continue to review full report at Codecov.

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


[SPARK-8111] [SparkR] SparkR shell should display Spark logo and version banner on startup.

spark version is taken from the environment variable SPARK_VERSION

Can one of the admins verify this patch?

nice! @shivar

ok to test. @aloknsingh Can you post a screenshot of what the logo looks like?

Merged build triggered.

Merged build started.

Test build #35501 has started for   PR 6944 at commit acd5b85.

Test build #35501 has finished for   PR 6944 at commit acd5b85.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Thanks @aloknsingh for the change. I had some minor comments inline. Also to reply to your question from the JIRA I'd say we can just keep the ASCII art as Spark right now.

Merged build triggered.

Merged build started.

Test build #35564 has started for   PR 6944 at commit ed607bd.

@sivaram: I modified the code as per your suggestions.
@andrewor14: Here is the screenshot.

Test build #35564 has finished for   PR 6944 at commit ed607bd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Jenkins. Please retest.

@aloknsingh Actually the tests passed if you look at the Spark QA message (above the jenkins one) -- the jenkins plugin has an issue where it reports failures for some builds.

retest this please

Merged build triggered.

Merged build started.

Test build #35565 has started for   PR 6944 at commit ed607bd.

LGTM. Will merge once tests pass

Hm, this should have just run the R tests.

Yeah I have no idea why that changed -- @JoshRosen did the limited unit test running thing change again ?

It looks like the pull request builder somehow thinks that the sql module changed.

hmm maybe it compared this PR to a different cached git commit ? I've seen that produce warnings for new classes added etc. before

There might have been a race if a PR that changed SQL was committed between the time that the PRB was triggered with a merge commit and the time that we fetched origin/master to do a comparison for changed files.  We can probably fix this by using a different base / SHA to compute the diff.

Test build #35565 has finished for   PR 6944 at commit ed607bd.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.


Create an SAP Fiori App Using SAP Business Application Studio

Tutorial URL: https://developers.sap.com/tutorials/appstudio-fioriapps-create.html
Please specify the step you are referring to
Step 1: Create new SAP Fiori project
(8. )For Consume Services, select the following, and click Next.
In the above mentioned step,
after i select system and source, it is asking me to enter user name and password for the backend system, and when i enter it says not authorized. Can you please guide me on this.
what is the user name and password for backend system.
where can i find and change it.
i entered as it was in the destination - ES5 Gateway user and password.

Hi,
The user/pass that should be part of the destination configuration is the one that is used to access the ES5 system. You can try accessing the ES5 service directly using this link. You should get a popup asking for your ES5 credentials. If you are able to access the ES5 service, then use the same credentials in the ES5 destination configuration. If you are not able, your user might be blocked and you need to reset it (follow this tutorial).
Regards - Raz

Hi Raz,
It worked!!
Thank you so much for the quick response.
I followed the tutorial you mentioned to reset the password and i was able to create my first SAP Fiori Project.
Many thanks..
Regards