It doesn't work with "foreign languages" keyboards

Hi,
great program, but unfortunately seems not working with a not-UK/US keyboard. In my case I have an Italian language keyboard and it happens that carnac catch keys as the keyboard was English. Not usable for me :-(

@solyarisoftware try this fork, it seems international layouts work for me.
https://github.com/bfritscher/carnac/releases

@bimtools-fpt-tungns8 It doesn't seem to work, at least not for Hebrew.
Carnac_2024-01-15_01-43-08.zip

@solyarisoftware You can try this, but sadly I can't find where I got it from:
https://github.com/Code52/carnac/issues/239#issuecomment-647620845

@solyarisoftware try my fork https://github.com/mugiseyebrows/carnac


apply is slow than pandas

version:
modin  0.14.0 and 0.14.1
python 3.9.7
win10 with i7-10875H, 16GB

code in Getting Started
start = time.time()

rounded_trip_distance_modin = big_modin_df["trip_distance"].apply(round)

end = time.time()
modin_duration = end - start
print("Time to apply with Modin: {} seconds".format(round(modin_duration, 3)))

print("Modin is {}x faster than pandas at `apply` on one column!".format(round(pandas_duration / modin_duration, 2)))

may cause crash and is slow than pandas (about 0.4 times of pandas )
MemoryError: Unable to allocate 130. MiB for an array with shape (8, 2137914) and data type float64

while thie is 2GB memory is free.

1 I have tested it in jupyter and spyder
3 the engine is ray
4 I watch the task view([Manager])

（6.2 GB is free）
However, in the procedure the free memory decrease dramaticly, it may comes to 200MB. is modin so memory expensive?

I wonder if this actually a duplicate of https://github.com/modin-project/modin/issues/4445

no


⚠ Bofrost.at has degraded performance

In 40d2ac7, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1140 ms

Resolved: Bofrost.at performance has improved in 0cb1011.


Should I need $scope.$apply() ?

I'm using a plugin (edgee:slingshot) and to upload files it has a client side method that uses a callback.  I'm familiar with how in vanilla angular, you'd have to call $scope.$apply() for the dom to update.  Using $scope.$apply() was the only way i could get the dom to refresh this.question on the screen.  Otherwise the change will not display in the dom and my ng-show and ng-hide tags in the html won't trigger.
    uploader.send(image, (error, downloadUrl) => {       if (error) {         // Log service detailed response.         console.error('Error uploading', uploader.xhr.response);         alert(error);       } else {         console.log('uploaded image to s3')         this.question.image = downloadUrl;         this.$scope.$apply();       }     });
Is this the best way to do things?  Am I missing a autorun method or something else possibly?

Yes you could do that, but there is a better way.
When you perform async operations, you can bind the context of the callback function with $reactiveContext.
For example
Accounts.forgotPassword({ email: this.email }, this.$bindToContext(err => {
    //  ... Here your callback logic ...
}));

In your case
 $reactive(this).attach($scope);

 uploader.send(image, this.$bindToContext( (error, downloadUrl) => {
 	if (error) {
	 // Log service detailed response.
	  console.error('Error uploading', uploader.xhr.response);
	  alert(error);
	 } else {
	  console.log('uploaded image to s3')
	   this.question.image = downloadUrl;

	   // You can delete this
	   // this.$scope.$apply();
	}
}));

You can read the docs here
http://www.angular-meteor.com/api/1.3.6/reactive-context

thanks for the great answer @santiq


Suggestion: Detail props

something similar like source would really make it easier to make forresty enviorments without as much performance loss https://developer.valvesoftware.com/wiki/Detail_props

This idea has been on the backburner for a while, and the idea would be to come up with a fake-random spawn generator for this so that the props don't change with a save/load. Once I make progress with such a concept, adding this shouldn't be a major hurdle.


Notification for Google Chat webhook

Hello,
Google chat webhook integration for notifications, like the one integrated into Teams and Slack, to notify admins of new "enrollments".
Can you add for google chat ?
function webHookMessage() {
if [[ $webhookURL == "chat.googleapis.com" ]]; then
...
Best regards

Thanks for the Feature Request, @stondelier.
This one falls under …

… finding available cycles to custom-code a feature we won’t use in our production environment is always challenging.
Feature Requests

I've unassigned myself; other SYM admins are invited to submit a feature-complete Pull Request.

I was planning to work on this but our instance doesn't have webhook or app integration enabled so I'm unable to test build and test this out

Thanks for the update, Andrew.

Reviewed on 2024-06-07-051726; help wanted.


Add .editorconfig

Added a simple .editorconfig file to make sure IDEs know to use the correct indentation for the project.

IDK which editors use this configuration file, but sounds like it is helpful, so thanks!


Small update README.md

Important indent added

@arslanbekov Thanks. Looks good to me based on https://kubernetes.io/docs/concepts/configuration/secret/#secret-files-permissions
/lgtm
cc: @GaneshSPatil


Main discussion

This is the main discussion thread for the repository. One of the main reasons it is here is to link the repo to the original issue (Microsoft/cppwinrt#256) in Microsoft/cppwinrt repo.

It looks like Microsoft has finally addressed this issue see here:


feat(acme): support acme listen host

support acme listen host
ref #978

I can confirm this works. Thanks


Windows:  Use XDG_CONFIG_HOME

I set ENV XDG_CONFIG_HOME in Windows.
But goneovim don't get XDG_CONFIG_HOME.
I want  "goneovim" folder set in .config folder.
It seems in config.go  line 162-175.
I should write code and send pull request.  Sorry, I tried and failed.
So, I send request.

@homaji
hi,
Since I did not know the prevalence of xdg_config on Windows, I restricted it to unix-like operating systems.
I have removed this limitation with latest.
Please try it on the following CI job artifacts.
https://github.com/akiyosi/goneovim/actions/runs/2482398740

@akiyosi
Thanks for your reply. Maybe, it's rare case to use xdg_config on windows.
I tried the binary in the link . It works!
read setting file from ~/.config/goneovim/setting.stoml .
There are 2 windows binaries. 1st one is no problem.
2nd one shows other window. Is it OK?

@homaji
Uh, sorry for the confusion.
The windows-2 artifacts are for my experimentation. Never mind.


Gnome 47 Persistent Remote Desktop session conflicts with SELinux

Describe the bug
After rebasing to bluefin-dx:41, attempting to access the persistent remote headless desktop session results in a black screen.
What did you expect to happen?
I expected to get a full remote session.
Output of rpm-ostree status
Deployments:
● ostree-image-signed:docker://ghcr.io/ublue-os/bluefin-dx:41
                   Digest: sha256:c78098caf0ceb463640afc8caead9b0a7b4862deabf131a95f8bd6f740eb63b1
                  Version: 41.20241028.0 (2024-10-28T04:44:51Z)
          LayeredPackages: thunar tumbler

  ostree-image-signed:docker://ghcr.io/ublue-os/bluefin-dx:latest
                   Digest: sha256:4072d44e95c70eadca4d754432baff0d58729c3f197676b64274944e59cdb1a6
                  Version: 40.20241011.0 (2024-10-12T22:01:46Z)
          LayeredPackages: thunar tumbler

Output of groups
No response
Extra information or context
This is an issue with SELinux. Changing SELinux to "permissive" solves this issue, but is obviously untenable.
A user on the Fedora discourse forums wrote an SELinux module that allows the session to work:
https://discussion.fedoraproject.org/t/gnome-remote-desktop-with-selinux-enforced/115832/8?replies_to_post_number=9
I have tested it and can confirm that it works like a dream. Would this be possible to add to the Bluefin/Bazzite-Gnome projects so that others don't encounter this issue?

This might be useful to add to main images even.
But it would be also good to notify Fedora folks about this.

It indeed is an issue for https://github.com/fedora-selinux/selinux-policy instead

I am experiencing the same issue on Bluefin-dx 40 gts. Only after setting setenforce 0 I can connect via RDP.
As I am actually testing some GNOME distros, I also installed Fedora Silverblue 41 yesterday and it worked out-of-the-box, so this seems to be Bluefin specific.


Version <IP_ADDRESS> fails to build with GHC 8.2.1 and cabal-install <IP_ADDRESS>

$ cabal install --allow-newer stack-1.5.0
Resolving dependencies...
Configuring store-<IP_ADDRESS>...
Building store-<IP_ADDRESS>...
Failed to install store-<IP_ADDRESS>
Build log ( /home/alex/.cabal/logs/ghc-8.2.1/store-<IP_ADDRESS>-EkbDeYEvc8E5BQvh9QLvPO.log ):
cabal: Entering directory '/tmp/cabal-tmp-32116/store-<IP_ADDRESS>'
Configuring store-<IP_ADDRESS>...
Preprocessing library for store-<IP_ADDRESS>..
Building library for store-<IP_ADDRESS>..
[ 1 of 11] Compiling Data.Store.Impl  ( src/Data/Store/Impl.hs, dist/build/Data/Store/Impl.o )
[ 2 of 11] Compiling Data.Store.Streaming.Internal ( src/Data/Store/Streaming/Internal.hs, dist/build/Data/Store/Streaming/Internal.o )
[ 3 of 11] Compiling Data.Store.TH.Internal ( src/Data/Store/TH/Internal.hs, dist/build/Data/Store/TH/Internal.o )
[ 4 of 11] Compiling Data.Store.TH    ( src/Data/Store/TH.hs, dist/build/Data/Store/TH.o )
[ 5 of 11] Compiling Data.Store.Internal ( src/Data/Store/Internal.hs, dist/build/Data/Store/Internal.o )

src/Data/Store/Internal.hs:110:1: error:
    Could not find module ‘Data.Typeable.Internal’
    it is a hidden module in the package ‘base-<IP_ADDRESS>’
    Use -v to see a list of the files searched for.
    |
110 | import           Data.Typeable.Internal (Typeable)
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
cabal: Leaving directory '/tmp/cabal-tmp-32116/store-<IP_ADDRESS>'
cabal: Error: some packages failed to install:
stack-1.5.0-AeEyMtn08gzB8Y5bVGRiVl depends on stack-1.5.0 which failed to
install.
store-<IP_ADDRESS>-EkbDeYEvc8E5BQvh9QLvPO failed during the building phase. The
exception was:
ExitFailure 1

Apologies. The latest version of the master branch builds correctly.


fix(clamav): Switch to official clamav image

I chose the _base version of the image because the previously used MailU version didn't include a database either.
Fixes #325
Fixes #432

@desaintmartin Did you have a chance to look at this? (Mentioning you since you seemed interested in #325)

That's nice, thanks for the contribution! Is there anything to know when changing to upstream?

@desaintmartin Not that I am aware of, no. As you can see, the UID of the clamav user in the image changes. But at least in our use case, everything else worked as expected.
Something we experienced with both MailU's and upstream's _base images - since they don't contain any database - were problems with the CDN's rate limit. Especially in combination with the HPA (which is enabled by default), the rate limit was quickly exhausted, causing the pods failing to start at all. We solved that by using a caching proxy (see #458 for the necessary changes to the chart). But maybe the HPA shouldn't be enabled by default, since frequently starting new pods facilitates early exhaustion of the rate limit.

It wasn't, thanks for noticing!
As I understand the docs, that option is only relevant when freshclam is started as root (not the case by default) and in that case the user given has to have write access to the database directory. Since (in the default security context) that directory is owned by group 101, the setting should probably default to user 100 as well (although it doesn't take effect in default config anyway)

I was worried because of #322   which implies there could be issues when a PVC is involved. They had to give the pod security context the fsGroup of 2000, I assume to match the value in freshclam
We're about to rollout an install using Flux to AKS and I was double checking issues and settings. This got me worried enough that we decided we were going to try and override the freshclam conf defaults to use a matching GID, but unclear if it's really going to be necessary.


Running with smplayer

I added smplayer to players list, but smplayer did not support adding external audio as argument. Only from GUI.
All videos except for videos with external audio play fine. If i specify audio as audio=>"AUDIO" video will stop working, and only audio will works. But smplayer can play youtube video just by adding youtube url, correctly set Title, and play video itself. Can we add smplayer support?
smplayer => {
arg => " -close-at-end -media-title TITLE",
cmd => "smplayer",
fs  => "-fullscreen",
srt => "-sub SUB",

Yes. We can use *URL* to pass the YouTube URL to SMPlayer.
Example:
smplayer => {
            cmd => "smplayer",
            fs => "-fullscreen",
            srt => "-sub *SUB*",
            arg => "-close-at-end -media-title *TITLE* *URL*",
},

Cool. Thanks.


container auto create - turn on and off

If Spark persists an objects in the container that is not exists, current code will automatically create the container. To make this happen, Stocator init method always checks if container exists and if not creates it automatically. All this happen in the init method of SwiftAPIClient.
In certain cases we would like to disable automatic container creation and skip the check if container exists during init method. This will save our cost of the HEAD Container operation during the init method.
There is need to introduce new property "fs.stocator.dataroot.autocreate" and make it true by default, when this property is not provided in the configuration. But if it's provided and the value is false, the init method should not check if container exists and skip automatic container creation.

@djalova Can you check it please?

@gilv In addition to the above, instead of doing the head at all, would it make sense to try the PUT of the object, catch the exception for the non-existing container and create the container, and then redo the object PUT?  The check first approach typically involves two calls but can be three.  The try and recover approach is typically one call but can be three.  The check first approach is simpler code.

@michaelfactor it depends on what is the 1st operation. If it's PUT - then your suggestion is fine. But it may be HEAD, or GET, etc.. so we will need to try-catch many operation in Stocator. It will be complicated change, since there is no single point when there is HEAD or GET.

@gilv Agreed completely -- this is a performance-simplicity tradeoff and the recovery on the HEAD or GET will be different for the PUT.  Not sure you need to autocreate on HEAD or GET of an object since if the container doesn't exist, obviously the object doesn't.  But this is obviously a performance optimization and we need to understand its value in practice before rushing to make the change.

@gilv I think all operations start out by calling FileSystem.exists() whether its a PUT or GET operation. In the case where auto create is off should we just throw a new exception when a container is not present?


Issue with k8s.io/docs/tasks/administer-cluster/kubelet-config-file/

This is a Bug Report

Problem:
This page does not tell me, nor point me to any page that tells me, what are all the fields (name, type, default value) in the kubelet configuration file.
Proposed Solution:
Document this.
Page to Update:
k8s.io/docs/tasks/administer-cluster/kubelet-config-file/

In the text there IS a pointer to the source code. That is the best thing we can get so far.

The subset of the Kubelet’s configuration that can be configured via a file is defined by the KubeletConfiguration struct here (v1beta1).

/kind feature

/language en

related: #21109

/triage accepted
/priority backlog
/remove-lifecycle rotten
Might be relevant to issue #23889

/remove-lifecycle rotten

This is resolved.


iPhone X display

On iPhone X, the notification view should be displayed a bit lower, otherwise it can't by fully seen.
Is there anyone having the same issue, or am i missing something here ? There's no one report about this issue, which seems strange to me.

@namanhams thanks for reporting this. What installation method do you use? I just checked out library, for me X looks OK:

@peterprokop : sorry for my late reply. I missed the notification email.
So i used the branch 'Swift-3.0'.
This is from Pod : pod 'SwiftOverlays', :git => 'https://github.com/peterprokop/SwiftOverlays.git', :branch => 'swift-3.0'
The function that i use is showAnnoyingNotificationOnTopOfStatusBar
I checked the source code (both branches), i couldn't find you use "safeAreaInsets" or "topLayoutGuide" anywhere, so wonder how you handle the top area in iPhone X.

@peterprokop : does you view extend under the status bar ?


Crashes when turning engine on/off -  Cannot read properties of undefined (reading 'vehicleInfoList')

Describe The Bug:
After setting up in home bridge, when turning the car on, the plugin crashes and reboots, but the start command does execute (verified in Kia app) and the state is properly updated after the plugin restarts.
To Reproduce:
Turn the car on or off
Expected behavior:
When turning the car on or off, the command is executed without crashing
Logs:

Environment:

Plugin Version: 1.2.0
Homebridge Version: v1.7.0
Node.js Version:  v18.19.0
Operating System: Raspbian

Could not replicate after restarting home bridge... may have been due to having the Kia app open resetting the session?


Scroll in Android

In Android, clicking on any feature (point, line, polygon) in the map to view the respective information, the scroll of the popup does not work. It is not possible to view all the information of the feature.
In Windows scroll works properly.

A workaround is here:
https://github.com/openlayers/ol3/issues/3253


Fix to #5043 - Query: Navigations: Using EF.Property with reference nav omits null compensation

Problem was that navigation expansion logic was only applied for MemberExpressions, but never for MethodCallExpression. Fix is to abstract that logic to work for both MemberExpressions and MethodCallExpression that happen to be EF.Property.
Also did some minor bugfixes and optimizations found along the way:

compensating for nullability differences in SqlTranslatingExpressionVisitor, when processing comparison,
preventing EntityEqualityRewritingVisitor to short-circuit on non-equality operations in case of nested binary expressions.

@anpete

@anpete  new version up

:shipit:


When running Istiod in a KNative environment the ports are multiplexed.

This fixes a bug where the port is labeled as http2, and KNative or
Istio upgrades the port to HTTP/2. Low risk for all other cases.
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[ ] Does not have any changes that may affect Istio users.

@costinm: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
3006736f960be5c6193b8ded6c94c1af52c173ef
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
3006736f960be5c6193b8ded6c94c1af52c173ef
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
3006736f960be5c6193b8ded6c94c1af52c173ef
link
/test release-notes_istio

lint_istio
3006736f960be5c6193b8ded6c94c1af52c173ef
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
3006736f960be5c6193b8ded6c94c1af52c173ef
link
/test release-notes_istio

lint_istio
3006736f960be5c6193b8ded6c94c1af52c173ef
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
3006736f960be5c6193b8ded6c94c1af52c173ef
link
/test release-notes_istio

lint_istio
3006736f960be5c6193b8ded6c94c1af52c173ef
link
/test lint_istio

gencheck_istio
3006736f960be5c6193b8ded6c94c1af52c173ef
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@costinm: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
3006736f960be5c6193b8ded6c94c1af52c173ef
link
/test release-notes_istio

lint_istio
3006736f960be5c6193b8ded6c94c1af52c173ef
link
/test lint_istio

gencheck_istio
3006736f960be5c6193b8ded6c94c1af52c173ef
link
/test gencheck_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


If view from right to left transtion is not working

check this repo , you will find the solution
https://github.com/Yasilis/SegmentedButton


patching heron for missing api/constants

shading the Kryo used by internal heron system
update deserialize(byte[]) to deserialize(ByteBuffer) in storm 1.0 api to comply with the storm api
add necessary constants and constructors

btw, it would be good if we can also resolve https://github.com/twitter/heron/issues/1626 in this PR...

👍

lgtm

@nlu90 - can you fix the CI and check the code in?


NSC, Error from monitorConnection,  RPC error code: PermissionDenied, Desc: no sufficient privileges

The setup is build on

cmd-nsc
cmd-nse-icmp-responder
nsc-kernel
nse-kernel

The platform is k8s 1.23, docker 20.10, on ubuntu 20.04, NSM 1.5.0
Things work as expected with the existing scripts -- NSC and NSE on their own, separate pods. (See related Issue #7051)
However, when we try to build NSC and NSE into the same container, we keep getting the NSC error:
[ERRO] [cmd:[/bin/nsc]] error from monitorConnection stream %!(EXTRA string=rpc error: code = PermissionDenied desc = no sufficient privileges)
Could not find any more information on the source of the error.
(It may not be related but it is very difficult for us to understand the whole spire/nsm integration, how things get worked into the containers, etc. We are really looking forward for some documentation on this.)
The Dockerfile that builds the node that has both the NSC and NSE is configured as:
FROM golang:1.18.2-buster as go
...
RUN tar xzvf spire-1.2.2-linux-x86_64-glibc.tar.gz -C /bin --strip=2 spire-1.2.2/bin/spire-server spire-1.2.2/bin/spire-agent

FROM go as nsc
...
RUN go build -o /nsc .

FROM go as nse
...
RUN go build -o /nse .

FROM ubuntu:18.04
...
COPY --from=nsc /nsc /bin/nsc
COPY --from=nse /nse /bin/nse
...
CMD ["/usr/bin/supervisord"]

The use case is to build topologies with nodes that have both NSC and NSE dynamic capabilities; many nodes with a single container; payload ETHERNET. Therefore, workarounds like sidecars, separate nsc/nse roles are not workable options.
Any help will be very much appreciated.
Thank you in advance.
nsc-rpc-auth-problem.txt
nsc.log
nse.log
kubectl-get-pods.txt
registry-k8s.log
forwarder-vpp.log
forwarder-sriov.log
nsmgr-nsmgr.log
nsm-test-setup.zip

In the provided test results, (unintentionally) both the nsc and the nse nodes provide the "icmp-responder" service (that was originating from the default Config). We corrected this after the logs were captured. Same result. Still get the error.
Also in the actual topology configuration there are no redundant service offers. Although each node provide and consume services, each service name is unique, based on the node id.

/cc @glazychev-art , @anastasia-malysheva May this be related to monitor OPA staff?

Thanks for the detailed information!
Most likely yes, it is related to OPA for monitoring. But it is not actually an error if you are not using init-container (cmd-nsc-init).
We probably need to rewrite the error message in order not to mislead anyone.

@glazychev-art Any idea what the root cause might be?  I'm not entirely sure why we would be seeing this, do you have a more specific idea?

@edwarnicke
cmd-nsc does Monitor connections before the Request. This is necessary to take the connection if there was cmd-nsc-init container before.
But as you know, you've implemented an open-policy for monitoring, and it is based on the spiffieID from the Request. But if we did not have an init container, then there were no Requests either. And an authorization error is returned.
But here the problem is different - as I see from the logs, there are many healing errors:
Aug 15 22:00:44.223 [WARN] [id:nsc-858c5dc57-2bf6l-0] [heal:eventLoop] [type:networkService] (7.1)         Data plane is down
Aug 15 22:00:44.223 [DEBU] [id:nsc-858c5dc57-2bf6l-0] [heal:eventLoop] [type:networkService] (7.2)         Reconnect with reselect

need to figure out why this is happening

That's an interesting scenario where we're trying to run nsc/nse together in the same container.
@bilgehan-erman

Do you have a diagram/scheme/proposal what do you finally get?
Did ping work  for your scenario?
I think that the problem with authz is related to dp healing. Could you re-test the setup with disabled dp heal https://github.com/networkservicemesh/cmd-nsc/blob/main/internal/config/config.go#L49? (means set env NSM_ LIVENESS_CHECK_ENABLED=false)

@denis-tingaikin
Unfortunately, LIVENESS did not seem to help:
nsc.log
nse.log

Did ping work for your scenario?

This the test scenario:

And this would be an example of building random topologies using these universal nodes:

@glazychev-art thank you very much for looking into this.
I'll try it out your suggestions and see how it goes.


[FEAT]: HTML Embeds - Please add a way to give the generated div a specific class name or ID?

What would you like to see?
When I use the html chat embedding feature, it works, but at some points we would like to manipulate the div position or visibility or existence of the div that is generated by the embed js script. It turns out that the div doesn’t have a class name or other identifying features. Was able to write a script to go a few layers deep and find one class named anything-LLM-chat, but it’s kind of a hacky thing to do. Would be nice to have a configuration for the outputted div. Thanks for the other configurations that are there. Just looking for one more :)

Which element? The toggle to open the window or the window itself?
The window has a tag:
https://github.com/Mintplex-Labs/anything-llm/blob/acbad2f2cf7c13e03e0a0dc1067041ba4c59930c/embed/src/App.jsx#L43

Was looking at the top level document body div it makes. I guess we could just do a remove for that selector

I think if anything we can add one to the floating button as well, that should be there as well

@atljoseph lets make that a new issue for feature request - it will get lost  in a closed comment thread

Moved to https://github.com/Mintplex-Labs/anything-llm/issues/1544


fixed small window under linux (x64)

Hiho,
i tried to start the current Ctrlr (binary) under a current Gentoo Linux. The program starts, but just show a very small quadratic segment of the upper left window which doesn't seem resizable - so Ctrlr seems not usable att on this platform.
Are there any sources for trying a self build?
If you need any system / environment details pls feel free to ask.
many thanks,
best regards,
Niels Dettenbach
http://www.syndicat.com

This should now be fixed in the source tree.


Introducing serde for benchmarking

Adding serde for benchmarking.
I was not able to integrate the benchmark with corrosion + cmake due to a lib/libunwind issue happening that I was not able to workaround when using corrosion.

I was not able to integrate the benchmark with corrosion + cmake due to a lib/libunwind issue happening that I was not able to workaround when using corrosion.

Let me have a look.

@FranciscoThiesen Is this PR still relevant?


[SERVER] message: window is not defined error on server-side rendering

I am getting the following error when setting the prerender config to true
[SERVER] message: window is not defined
[SERVER] stack: ReferenceError: window is not defined
    at new ResultContainer (eval at <anonymous> (/tmp/execjs20170520-8-hyuzsqjs:4353:2), <anonymous>:51:16)
    at eval (eval at <anonymous> (/tmp/execjs20170520-8-hyuzsqjs:2865:2), <anonymous>:295:18)
    at measureLifeCyclePerf (eval at <anonymous> (/tmp/execjs20170520-8-hyuzsqjs:2865:2), <anonymous>:75:12)
    at ReactCompositeComponent._constructComponentWithoutOwner (eval at <anonymous> (/tmp/execjs20170520-8-hyuzsqjs:2865:2), <anonymous>:294:16)
    at ReactCompositeComponent._constructComponent (eval at <anonymous> (/tmp/execjs20170520-8-hyuzsqjs:2865:2), <anonymous>:280:21)
    at ReactCompositeComponent.mountComponent (eval at <anonymous> (/tmp/execjs20170520-8-hyuzsqjs:2865:2), <anonymous>:188:21)
    at Object.ReactReconciler.mountComponent (eval at <anonymous> (/tmp/execjs20170520-8-hyuzsqjs:2505:2), <anonymous>:46:35)
    at eval (eval at <anonymous> (/tmp/execjs20170520-8-hyuzsqjs:3255:2), <anonymous>:45:36)
    at ReactServerRenderingTransaction.TransactionImpl.perform (eval at <anonymous> (/tmp/execjs20170520-8-hyuzsqjs:2559:2), <anonymous>:140:20)
    at renderToStringImpl (eval at <anonymous> (/tmp/execjs20170520-8-hyuzsqjs:3255:2), <anonymous>:43:24)

How can I fix this?

I am using react_on_rails 6 with mini_racer

Hi @chiraggshah you cannot use window for server rendering.
Please take a look at our Coaching Plan. We can give you some immediate help if you need it.


Escape YAML Where possible

Fixes #224
This is meant to help reduce issues where the linter is not able to run because it needs to parse the YAML due to a YAML rule being active, yet the YAML is invalid and can be fixed with a simple escape by single or double quotes. Some scenarios cannot be remedied, but hopefully this will catch most use cases.
Do note that using the try single line arrays option will not work properly if you nest a comma inside quotes since it will be treated as the delimiter despite being escaped.
Changes Made:

Removed some ghost code
Added a new rule for attempting to escape YAML special characters
Added tests to verify that the changes work as expected
Updated the linter rule running logic to make sure if the rule is active it gets run first and that only rules that are not disabled, run prior, or run after the regular rules get run in the for loop (this will hopefully save some time when linting hundreds to thousands of files at once).

The YAML changes run against several thousand of my one files looks good. I am not seeing any weird issues around it. It has however shown me that there may be an issue with a adding 2 spaces after content. I will take a closer look at it and see if I can fix it if it was not just me creating a bad YAML setup. It also seems that I may be able to reproduce #236 , but I will have to test it out a little more to be certain.


Extension not working, also downloads danbooru.csv everytime i relaunch the web-ui

Downloads the CSV file everytime I launch the automatic1111 ui

but the file is already in the extensions folder:

and also the extension does not work while enabled:

plz help TT

replaced the csv file in the tags folder with the one that is downloaded and it magically works!


ARROW-16000: [C++][Python] Dataset: Added transcoding function option to CSV scanner

WIP Adding an optional function that wraps all input streams with a user-supplied transcoding function.

The current state is that it works, but it relies on the workaround of adding an encoding parameter to pyarrow.dataset().
That needs to be dealt with before proceeding.

I pushed an alternative way of passing the encoding in https://github.com/apache/arrow/pull/13709/commits/22eff73806c8511288c16d87cbf64da26a7d941e. For the user it works the same way as in read_csv: it is a field in read_options. I store the value in CsvFragmentScanOptions, and then restore it.
How do you feel about this?

I store the value in CsvFragmentScanOptions, and then restore it.
How do you feel about this?

I like this approach if you can get it working.  Can you add this to the CsvFileFormat constructor?
        else :
            # default_fragment_scan_options is needed to add a transcoder
            self.default_fragment_scan_options = CsvFragmentScanOptions()
        if read_options is not None:
            self.default_fragment_scan_options.encoding = read_options.encoding

Is it an option to add encoding as a field to the C struct ReadOptions?

That seems undesirable.  The C++ csv reader doesn't have the field because it has no ability to handle encodings.  So I'm not sure we want to add a field that is completely ignored.

I got it working but it isn't pretty. I had to duplicate the python-specific encoding field to both CsvFileFormat and CsvFragmentScanOptions, and modify the getter function default_fragment_scan_options in FileFormat!
From https://github.com/apache/arrow/pull/13709/commits/4d9802b60480180e8deab6dd162a72d837cccdeb:
1 would report the default 'utf8' (forgetting the user's encoding choice), while the other would still properly report the requested encoding.
To the user it would be unclear which of these values would be eventually used by the transcoding.

This also seems pretty undesirable.
What about if we added an option to allow the user to specify a library to do the transcoding in C++, that we then dynamically link to? Then we can just add the field to the C struct and the Python transcoding in this PR would also still work nicely.

(4d819aa should be Removed encoding from CsvFragmentScanOptions.equals())

I think we're getting a bit far afield…Dynamic linking needs platform-specific code and usually we configure optional dependencies with build flags.
What if we add the C++-side field, have it error in C++ if not set to the default, and in python, we can reset the value to the default and configure the transcoder? That leaves us the path to upgrade and should avoid excessive python-side hacks. If we decide it's valuable to have built-in C++ side transcoding, then we have the option there already.
An alternative would be to have the Python wrappers for these structs no longer actually wrap the C++ structs, so that we aren't limited to the C++ fields. But that would lead to some code duplication/messiness as well.
I'm not sure we can avoid some messiness: the fundamental issue is that we have a Python-only field but are trying to directly wrap the C++ structs. That extra field needs to be mirrored somewhere. Either we do work to pass it around on the Python side or we give in and add it in C++.

That sums it up very nicely. Both alternatives are fine with me. I just pushed an update that aims to do what you suggest, which is adding an encoding field to the C++ struct. The CSV reader returns an Invalid error when the user has specified an encoding other than UTF-8 but the stream_transform_func is empty.
Is that the right error type or would an IOError be more suitable?
How do you feel about the name of the set_transcoder function in CsvFragmentScanOptions? Should I add a docstring to it? Should the stream_transform_func be added to the equals() function? (in that case I think I need to add a getter/setting for it too)

How do you feel about the name of the set_transcoder function in CsvFragmentScanOptions? Should I add a docstring to it?

It feels like it shouldn't be publicly accessible? Or else it should mirror the C++ side option name 1:1

Should the stream_transform_func be added to the equals() function? (in that case I think I need to add a getter/setting for it too)

I guess we can only get pointer equality, but yes

Somewhere in the CSV reader itself we should also validate the option

I tried this, but it doesn't work, because in that case we would need to re-set the field back to utf8 when adding a transcoder in python. Otherwise, the error is triggered even though we are transcoding to utf8. But then, we will again run into the issue where the ReadOptions object that the user created is changed:
>>> import pyarrow.dataset as ds
>>> import pyarrow.csv as csv
>>> ro =csv.ReadOptions(encoding='iso8859')
>>> fo = ds.CsvFileFormat(read_options=ro)
>>> dataset = ds.dataset("file.csv", format=fo)
>>> ro.encoding
'utf8'

This would be really strange if you ask me. And if we accept this strange behavior, we didn't need to add the encoding field in the first place.
So now, the field is basically ignored in the CSV reader, there only is the check in the dataset CSV reader that there must be a wrapping function set if the encoding is not utf8.

Ah, thanks for explaining.
Wonder if we should/could pass a copy of the ReadOptions then?

But it's not a big deal, I think so long as the field is clearly documented

@joosthooz Do you want reviewing at this point or are you looking to polish this PR first?

Thanks for checking in, @pitrou ! Most important is to choose which approach to take, to make that easier I opened https://github.com/apache/arrow/pull/13820 to compare against.
After that I need to check why some of the tests are failing (they seem unrelated) and maybe polish a bit and then I'll move it out of the draft state.

I would favor #13820, which pushes complexity into Python, over this one, which introduces a dummy option in C++ that has no effect.

Continuing here: https://github.com/apache/arrow/pull/13820


Add generic OCPP interface

Introduces the ocpp_generic interface to be shared by 1.6 and 2.0.1.
Implements this for OCPP 1.6

Requires:

https://github.com/EVerest/everest-utils/pull/79 (ev-cli version 0.2.4)

Since you require a newer version of ev-cli you should also update this in https://github.com/EVerest/everest-core/blob/5be1ef7d24f5eff6230ce5d8fc3078ae2b175def/cmake/ev-project-bootstrap.cmake#L8 (this already throws an error quite early in the build process, not just during compilation)


Typo in README

From my limited understanding of the plugin, you're not supposed to add multiple systems with the same name are you?

It's my spelling mistake, thank you for the correction.


[Help] How to remove the Backticks of inline-code

Hello I use the Nuxt Tailwind Module with Tailwind typography. The prose renders backticks, I wont want this.

I tried to set the following
module.exports = {
  darkMode: false, // or 'media' or 'class'
  prefix: 'tw-',
  mode: 'jit',
  theme: {
    extend: {
      typography: (theme) => ({
        DEFAULT: {
          css: {
            '.tw-prose code::before': {
              content: '""',
            },
            '.tw-prose code::after': {
              content: '""',
            },
          },
        },
      }),
    },
  },
  plugins: [require('@tailwindcss/typography')],
}

It does not have any effects... idk why. the nuxt config has no other tailwind config, but the module import

            'code::before': {
              content: '',
            },
            'code::after': {
              content: '',
            },

I think the problem is with single quotes.. got it to work. And there is no need to call the prose class


Cannot load the app if manifest.json is not present

Environment

Environment:

dev (rinkeby)

Steps to reproduce

Set up https://github.com/gnosis/safe-test-app
Remove manifest.json from the public folder
Try to access the app by pasting the URL directly to the address bar, like https://safe-team.dev.gnosisdev.com/app/rin:safeAddress/apps?appUrl=http://localhost:3000
The app doesn't load/crashes after some time

Expected result
You can load an app with manifest.json if the information is present on config service/pasting URL directly
Obtained result
The app doesn't load/crashes after some time
Screenshots

I overlooked https://github.com/gnosis/safe-react/issues/2959


[Proposal]FocusScopes should be escapable via tab-traversal

Use case
The FocusScope widget cannot be escaped purely via tab-traversal, which is relevant if the rendered UI contains nested FocusScope widgets. (This can happen if there are nested Navigator widgets, as each Navigator includes a FocusScope in its build method.) Clicking somewhere in the UI outside of the FocusScope resets the focus location and does allow a user to escape a FocusScope that does not account for the entire UI.
Proposal
Before cycling within a FocusScope, the Focus system could check to see if that FocusScope sits within an outer FocusScope, and if so, surrender Focus back to the wrapping FocusScope. However, this change would make the difference between a FocusScope and a FocusTraversalGroup somewhat unclear (at least, to me).
Alternatively, Navigators could cease to include a FocusScope within their build methods, but the implications of this change are much harder for me to anticipate.
Reproduction
import 'package:flutter/material.dart';

void main() => runApp(MyApp());

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'FocusScope Demo',
      home: FocusScope(
        child: Column(
          children: const <Widget>[
            FocusableWidget(),
            FocusScope(
              child: FocusableWidget(),
            ),
          ],
        ),
      ),
    );
  }
}

class FocusableWidget extends StatefulWidget {
  const FocusableWidget({Key? key}) : super(key: key);

  @override
  State<FocusableWidget> createState() => _FocusableWidgetState();
}

class _FocusableWidgetState extends State<FocusableWidget> {
  bool _isFocused = false;
  @override
  Widget build(BuildContext context) {
    return Focus(
      onFocusChange: (isFocused) => setState(() => _isFocused = isFocused),
      child: Container(
        color: _isFocused ? Colors.red : Colors.grey,
        height: 100,
        width: 100,
      ),
    );
  }
}

This is potentially related to #86972.

Leaving this issue open for further thoughts from the team.

I just chatted with @gspencergoog and we identified a few options to unblock this issue with nested navigators.

Before tab traversal cycles within a FocusScope, it could check to see if it is nested within another FocusScope, and, if so, release focus to that parent FocusScope.
Navigators could, by default, not add a FocusScope unless they are the root Navigator.
Navigators could expose a new parameter, shouldAddFocusScope, which would default to True. This would obviously allow nested navigator scenarios to instruct their Navigator widgets to omit the their FocusScopes.

cc @goderbauer

I think I prefer option 3,  since it's not a breaking change AFAICT. But I'm not sure what the default should be. If you are calling the Navigator constructor, you're either using the Router API and building the root Navigator (so shouldAddFocusScope should be true) or you're building an inner navigator (so shouldAddFocusScope should be false.)

I vote for 2, but remove the responsibility of Navigator adding the FocusScope. It will be WidgetsApp who creates the focusScope around the navigator entirely.

You're certainly most qualified to predict how hard it would be to land such a change, @chunhtai. fwiw, option 2 would also be my favorite if I magically knew it wouldn't cause a logjam of errors in Google apps. And in a scenario where we did decide that option 2 was too uncertain, then I'd be with @johnpryan and prefer option 3, but probably with a non-breaking default value of true.

3 is likely the least breaking way, but the api does not make sense to me, and therefor, will likely confused developer to use that incorrectly.
I think the best way to approach this is to go for 2 and see what breaks.

Option number 2 was implemented in https://github.com/flutter/flutter/pull/109702

closed as fixed


Need functionality to stop test suite when a test is failing

We need a way to stop a Jest test suite after first failing test. Our test suites are flows and test are steps dependent on the previous step. I know that this is not best practices for unit testing but our use case is different.
We are using Jest Circus so we are getting events about all test in our customized Jest environment class. We want a method to trigger stop of the test suite and mark the remain test as skiped or failed.
Motivation
This is the only missing functionality we are missing in Jest to run our end 2 end test suites.

Isn't this --bail?

https://jestjs.io/docs/en/24.x/configuration#bail-number--boolean
why it doesn't work?

The bail option is working only for test suites (aka describe) not for tests.
Any plan to fix this?

This jasmine-fail-fast hack works: https://github.com/facebook/jest/issues/2867#issuecomment-370624846

Any update on this without having to use the Jasmine fail fast hack?


Consider using standardized variables defined by schema.org

Consider using more standarized definitions for the start date and the end date (dob/dod,birth/death).
example
now:
dob: 2016-06-20
# dod: 2020-06-20 # overwrite lifespan
lifespan: 4

suggestion 1
date: 2016-06-20T10:00:00
#endDate: 2020-06-20T12:00:00
#duration: P4Y

Reference schema.org

duration
Duration
The duration of the item (movie, audio recording, event, etc.) in ISO 8601 date format.

Date  or                                 DateTime
The end date and time of the item (in ISO 8601 date format).

startDate
Date  or                                 DateTime
The start date and time of the item (in ISO 8601 date format).

--
--
--

duration Duration 	The duration of the item (movie, audio recording, event, etc.) in ISO 8601 date format.
endDate Date  or
DateTime 	The end date and time of the item (in ISO 8601 date format).
startDate Date  or
DateTime 	The start date and time of the item (in ISO 8601 date format).

Add alias for dob, lifespan, and dod as startDate, duration, and endDate in version v1.3.0 to make it more formal.

Thanks @hieuthi !


refactor(names): remove name aliases

Remove all name alises from last name change.
BREAKING CHANGE: Aliases from the last name change have been removed.  To avoid issues, upgrade to
that version (v1.9.1) first.

:tada: This PR is included in version 2.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix for issue "Duration missing as offAll() parameter" #250

https://github.com/jonataslaw/get/issues/250

Thanks for this PR.
LGTM

@all-contributors please add @kluverua for maintenance


Add variable to remove deprecation error on PHP 8.2

Fix deprecation error on PHP 8.2
[ Deprecated error ] Creation of dynamic property SageVariableData::$alreadyEscaped is deprecated ~ ROOT/vendor/php-sage/sage/inc/SageParser.php:280

that was an unused variable, thanks for pointing it out


Deprecate profiles/:id/unassigned_parameters

What does this PR (Pull Request) do?

[x] This PR deprecates the TO endpoint profiles/:id/unassigned_parameters

Which Traffic Control components are affected by this PR?

Documentation
Traffic Ops

What is the best way to verify this PR?
Pull changes, verify all builds and tests pass. Also verify the 1.1 endpoint returns a deprecation warning, and the 2.0 endpoint does not exist

The following criteria are ALL met by this PR

[x] This PR includes tests OR I have explained why tests are unnecessary
[x] This PR includes documentation OR I have explained why documentation is unnecessary
[x] This PR includes an update to CHANGELOG.md OR such an update is not necessary
[x] This PR includes any and all required license headers
[x] This PR ensures that database migration sequence is correct OR this PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

Additional Information

Not ready for review, waiting on some changes from #4452

Ready for review


Support KHR_materials_volume

The KHR_materials_volume GLTF extension allows for GLTF files to contain material data with information about volume thickness, refraction, and absorption.
Filament already supports all the required material features, but GLTFIO (and by extension the filament samples) don't currently support loading files with this GLTF extension.

KHR_materials_volume is supported. See https://github.com/google/filament/pull/4291

Oh apologies! It seems that the exporter I was using didn't correctly pack the data.


Nested relationship data missing in JWT token and req.user

Bug Report
Current Behavior
The JWT token created in the default login operation does not contain nested relationship data. This is usually okay, due to the passport JwtStrategy middleware fetching the user from DB (with nested data) on every request. The issue is static assets, where we still perform the DB query for the user, but the nested data is not populated.
The two issues I see with this are:

The read access control lacks nested relational data when run for static asset requests
Inefficient (redundant DB calls)

Expected Behavior
The data provided through req.user should contain the same data for document and static asset requests. Furthermore, I think it would be a good idea to retrieve the user state from the JWT token, instead of the DB.
Possible Solution
If the JWT token is populated correctly, we could use it's state in the JwtStrategy to populate req.user, which would solve the issue with static assets. Although, I see a few problems with the current login operation:
In order to populate the JWT token properly, the afterRead hook (which effectively is the function that populates the nested data into the user object) should be moved up before the assignment of the token. The req.user assignment also needs to be moved up then, so that the afterRead passes the user state to access control functions.
When the JWT contains all the data that a fresh DB query would return, we should be able to simply re-use it's state. The only downside I can see when doing this would be that a new JWT token would have to be generated (i.e by logging out and in again) in order for the state to be updated. So, changes in the JWT state would not be reflected immediately.
Steps to Reproduce
See tests in the PR: 1362
Detailed Description
Payload version: 1.1.21

Hey @dsod I know we've got a conversation going on Discord regarding populating req.user straight from the JWT. I am not opposed to this, and I think we should keep that conversation going. But for now, I've fixed this original issue and we can keep the conversation going either on Discord or on the PR that you've opened!

Thanks!


Download binary on npm postinstall hook

As was discussed in issue #16 the lefthook npm package contains all the binaries, this postinstall hook downloads only the appropriate binary for the host OS (or in case of CI: none).
Note: I submit my PR in accordance to https://github.com/Arkweid/lefthook/issues/16#issuecomment-517672355.
Note 2: I did not run any go/ruby command as this PR mainly changes the postinstall script in the .npm directory. If this is still necessary I will update my PR, just let me know what is missing. :v:

/cc @HellSquirrel
#16

Thanks! I will test it in a few days

Here how you can load unzip and execute lefthook binary on mac
const fs = require("fs");
const zlib = require("zlib");

const url =
  "https://github.com/Arkweid/lefthook/releases/download/v0.6.3/lefthook_0.6.3_MacOS_x86_64.gz";

const fileName = "lefthook-local";

https
  .get(url, res => {
    if (res.statusCode === 302) {
      const originalUrl = res.headers.location;
      console.log("redirect, redirect");

      https
        .get(originalUrl, res => {
          if (res.statusCode === 200) {
            res
              .pipe(zlib.createGunzip())
              .pipe(fs.createWriteStream(fileName))
              .on("finish", res =>
                fs.chmod(fileName, 755, err => {
                  if (err) {
                    console.log("error changing rights");
                  }

                  console.log("done");
                })
              );
          }
        })
        .on("error", e => {
          console.log("error after redirect!", e);
        });
    }
  })
  .on("error", e => {
    console.error(e);
  });

For Windows we should change the url variable to https://github.com/Arkweid/lefthook/releases/download/v0.6.3/lefthook_0.6.3_Windows_x86_64.gz

(I will have time (maybe today but definitely) on thursday to continue working on this PR. Sorry for the inconvenience. 🙇 )

Is it ready?

I finished with the PR for now (as in "I tried to implement the feature while considering all raised concerns").
I think someone should (or maybe also wanted to) test the current state, as I do not know how to test the postinstallation hook from an actual npm install/yarn add. Let me know, if the PR still needs work to be done.

I'm still going to test it :) I can get here in a week or two

There is a simple way to test postinstall

npm pack to get tgz file
Use link to this tar file in demo project npm install ../lefthook/lefthook.tgz

Still there? :)

Oh hey, yes. I actually overlooked the last message from Andrey, so I will check today/tomorrow on some systems whether it works ok, and then report back. ✌

I did now test my PR on mac and Windows 7 VM:
The downloading and calling of lefthook install -f seems to work alright. Unfortunately as my VM image is 32-bit, I couldn't verify that the complete process works.
It does for mac (if I change the index.js appropriately).

I also rebased the PR on the latest master branch, removed the [WIP] from the commit-in-question, and some further improvements to the postinstall script:

only allow 64bit downloads on windows (there are no 32bit executables in the releases tab),
remove the pre-build executables from the binary folder, and
consolidate the naming of the executable (rename the downloaded executable lefthook/lefthook.exe for MacOS/Linux resp. Windows), and call them by name/platform from index.js.

I'm not sure, but is it possible to point package.json's bin.lefthook property right into binary, instead of index.js, to eliminate node spawning for simply choose right binary?

Updated the PR again to fit snugly on the latest master.
@yumauri I am uncertain whether package.json#bin.lefthook can be the lefthook binary directly, as firstly the docs on that field give the impression that it must be a node script, and secondly when i attempted to do it npm (at least) complained about a missing script, although the binary existed. 🤷♂

Amendum: This PR works on the assumption, that binaries are always available for the most recent version as a gzipped package. That's why there is a commit aborting on 32bit Windows.
I understand that this is an added burden on the maintainer, as there are more and more diverse packages to be produced. The other solution would be to handle the zips as well, but require an additional (potentially heavy) node zip-library, which I tried to avoid until now.
Let me know what you think.

Github actions are publicly available ;) Theoretically it is possible to make an action to build and upload binaries somewhere on every push

Please, please, please do NOT merge this. Not everybody is working in an environment where all machines have direct internet connectivity!
I have customers in highly regulated environments where their working machines are isolated from the internet. Development is done with an internal package repository, acting as a proxy for upstream repositories. All package downloads come from this proxy.
The current situation, with the binaries included in the lefthook package, allow these customers to work with lefthook correctly. If you change it to downloading during the postinstall, their setups will break and they have no way to work around it as you hardcode the download URL.
Making the URL configurable also pushes a lot of chore onto the users as they have to copy the binaries, put them on an internal site and communicate the link. Quite error prone and not usage friendly.
To wrap up: can you please leave it as it is?

Yeap, I changed my mind. Let's close it.

Seems @ringods's concern was neglected in #188:

I have customers in highly regulated environments where their working machines are isolated from the internet. Development is done with an internal package repository, acting as a proxy for upstream repositories. All package downloads come from this proxy.

If the npm registry is proxied, I think this can be easily addressed by publishing the binaries to npm instead of using hosting them as a GitHub release.
For example, playwright does this with their browser flavors (eg. playwright-chromium).


Resiliency by Design – Capture Resiliency Features as part of CALM

Feature Request
Develop CALM to capture those aspects of resiliency that are decided or influenced by architecture designs choices.
Description of Problem:
Designing systems for resiliency is a complex endeavour.
While it is easy to find literature on resiliency techniques and considerations, there seems to be a lack of practical ways of effectively applying resiliency considerations to architecture designs.
Potential Solutions:
CALM offers the opportunity to capture and persist resiliency considerations as part of system architecture designs and subsequent implementations and can develop into more:
•	A structured, practical, and scalable guide for resiliency design.
•	Templated resiliency design options.
Leading to better resiliency capabilities:
•	Compare & contrast different resiliency design choices.
•	Development and identification of resiliency design patterns.
•	Improved resiliency measures.
•	Targeted resiliency testing.
Next Steps
Create a Framework to articulate Resiliency Requirements
Before a system can be declared "resilient", there needs to be an understanding of what the benchmark is - ideally expressed as a clear set of requirements that need to be met.
Here an outline of a potential framework to capture resiliency requirements:
Definitions

Resiliency - The ability of a system to maintain an "acceptable level of service" in the face of adverse conditions.
Acceptable level of service - The level of service agreed by stakeholders (for whom the system provides services to) that might exhibit some degradation compared to desired performance but is still accepted as performing its function (e.g. the service is not perceived to be fully stopping or halting, to suffer downtime or otherwise involve human intervention such as manual restore or recovery operations).

Scope

System
System's end of any relationships (communications, dependencies)
Architecture of the System
Resiliency
Recovery

Taxonomy

System Rating - custom defined - e.g. 1, 2, 3
Requirement Applicability - custom defined - e.g. Must Have, Optional
Requirement Type - custom defined - e.g. Policy, Implementation
Requirement - custom defined.

Resiliency Requirements Framework (Example)

System Rating

Requirement Type
Requirement
1
2
3

Policy
The system has a  clear (stakeholder agreed) defintion of the minimum acceptable level of   service.
Must Have
Must Have
Optional

Implementation
The acceptable level of service definition is expressed quantifiably in terms of availability, latency, performance and integrity requirements.
Must Have
Must Have
Optional

Policy
The system has a clear (stakeholder agreed) definition of RPO
Must Have
Must Have
Optional

Policy
The system has a clear (stakeholder agreed) definition of RTO.
Must Have
Must Have
Optional

Policy
The system is portable to run on different platforms and vendor services.
Must Have
Optional
Optional

Policy
The system maintains back-ups of all critical data points.

Must Have (X=2)
Must Have (X<8)
Must Have (X<400)

Propose a standard set of Resiliency Requirements Definitions
This working group could/should propose standard resiliency requirement definitions to pick and chose from.

@rocketstack-matt @develontopia , happy to contribute ideas to this domain. let me know if there are separate discussions scheduled.

@charleyalpha789 one area that might be of interest is looking at: https://github.com/finos/architecture-as-code/issues/426
Have you attended a CALM monthly meeting? This might be a good place to start/get an introduction to some of the other collaborators


Change db schema attribute display to lookup

According to metaschema changes https://github.com/metarhia/metaschema/pull/97 in version 0.0.19

Landed in https://github.com/metarhia/globalstorage/commit/bbb8b699b168efdb0f57fd0253a0d1dc9ef03a5b.


package name conflict

there's a conflict with package names at loading time when you install the standalone version of tensorboard from: https://github.com/dmlc/tensorboard
The use case is to not have to install tensorflow, since that's whole point of using pytorch. Any workaround for this? I'd would be more intuitive to rename the module to tensorboard-pytorch

Hi, the point of this package is to make use of tensorboard's feature, instead of getting rid of TF. I think most researcher have both pytorch and tensorflow in a single machine. :) Additionally, to use function such as add_embedding, you still need tensorflow installed at this time.
And there will be a more generic version in the future (supports Theano, numpy), so I would like to have a module name without a specific postfix. Please suggest if you have good idea.
A easy workaround of conflict will be https://stackoverflow.com/questions/27532112/how-to-handle-python-packages-with-conflicting-names, or rename the tensorboard folder to  tensorboard-pytorch folder and build by yourself.

Thanks for the answer. The first solution looks good, however the second one should be the way to go since overlapping package names in general is a bad practice. Besides, if you want to give support to other frameworks you will have to rename the package because it won't be pytorch specific anymore.


Incorrect Plugin Import Instructions

Thanks for this package, works great!
Just 1 update required for readme, we ran into an issue where it couldn't find your package after installing it. You need to add the @shelf/ prefix to the plugin name.
your instructions say:
plugins:
  - serverless-simplify-default-exec-role-plugin

should be:
plugins:
  - '@shelf/serverless-simplify-default-exec-role-plugin'

Thanks!!
Updated:


travis: Automatically publish binaries for tags

As part of our release process, we publish binaries of quilt for
various operating systems. Manually compiling and uploading the binaries
is very cumbersome, so we now have travis automatically update the
release when changes to a tag are made.

@Baisang they get published to the release page: https://github.com/quilt/quilt/releases. A new release is automatically created for each tag.


[SPARK-37340][UI] Display StageIds in Operators for SQL UI

What changes were proposed in this pull request?
Add explicit stageId to operator mapping in the Spark UI that is a more general version of https://issues.apache.org/jira/browse/SPARK-30209, where a stageId-> operator mapping is done with the following algorithm.

Read SparkGraph to get every Node's name and respective AccumulatorIDs.
Gets each stage's AccumulatorIDs.
Maps Operators to stages by checking for non-zero intersection of Step 1 and 2's AccumulatorIDs.
Connect SparkGraphNodes to respective StageIDs for rendering in SQL UI.
As a result, some operators without max metrics values will also have stageIds in the UI. In some cases, there is no operator->StageID mapping made because no stageIds have accumulatorIds that are a part of the Operator's accumulatorIds. URL links at the top to go to the succeeded jobs and completed stages that were executed as a part of the selected query are also provided.

Why are the changes needed?
Makes for easier and quicker debugging and navigation.

Does this PR introduce any user-facing change?
Yes, Succeeded Jobs: and Completed Stages:listed at the top of the UI, along with Stages: in some of the operators.

How was this patch tested?
Manual test locally in SQL UI.

Test build #145290 has started for PR 34622 at commit 3489292.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49760/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49760/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49760/

Test build #145293 has started for PR 34622 at commit 5734754.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49763/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/49763/

Test build #145290 has finished for PR 34622 at commit 3489292.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class StageAttempt(
case class GraphNodeToStages(

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/49763/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145290/

Test build #145293 has finished for PR 34622 at commit 5734754.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class StageAttempt(
case class GraphNodeToStages(

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/145293/

cc @tgravescs would this feature be of interest?

cc @sarutak and @gengliangwang FYI

yes, it would be nice to have the actual stagIds in the ui, I'll need to look closer at the logic though, which likely won't be til next week.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50214/

Kubernetes integration test status failure
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/50214/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/50214/

@tgravescs do you have time to take a quick look?

So a couple questions and concerns.

I'm not sure having the list of Completed Stages at the top of the page helps and I'm concerned that could be a very long list.  You can simply click on the job or go to the stages page to get there.  You could also make the Stages link in the operator box clickable.
what does this list for exchanges where the exchange crosses 2 stages?  Similar hash aggregates.
have you run this more than just a couple local jobs?  ie on large jobs or in production?

I'll try to get this built and try it out locally

Thanks for the feedback comments so far.

I originally added the list of completed stages for convenience on the SQL UI. I'll look into making the Stages link in the operator box clickable.
In this case, multiple stages will show up in the operator box. It would look like

I've run this in production at Workday.

sorry, I missed your response.
After looking some more the list of jobs can get very long as well.  I'm fine with leaving the list of stages as well.
Do you know how long the connectOperatorToStage takes for a query with larger graph?

sorry, I missed your response.
After looking some more the list of jobs can get very long as well. I'm fine with leaving the list of stages as well.
Do you know how long the connectOperatorToStage takes for a query with larger graph?

Unfortunately I don't know how long it takes for connectOperatorToStage with queries with a larger path. I haven't seen issues regarding to runtime here at least.

@tgravescs @martin-g should I create another pull request for this feature to try to get it merged? I'm unable to reopen the PR.

No, I don't know what sort of performance tests should be run for this feature.

This change is run in production workloads but we haven't noticed performance issues. At what scale did the performance issues come up and how were they detected in your case?


I can not reproduce the results

Hey, thanks for your excellent work!
I train TrackFormer on your default setting (load from pretrained CrowdHuaman checkpoint) on the joint set of CrowdHuman and MOT17, and i get result on MOT17 about 74.0 MOTA, but when i submit it to the motchallenge,i only get 72.7 MOTA, i changed the batch_size to 1,and keep other parameters unchanged.
Why is this happening?

Cause you changed the batch size to 1. :) A different batch size means you must find new optimal learning rates and training epochs.

Cause you changed the batch size to 1. :) A different batch size means you must find new optimal learning rates and training epochs.

What should I change the  learning rates and epochs to when I set the batch_size =1?

I do not know. You have to find new optimal LRs and epochs. A recommend starting point could be to half the learning rates just as you did with the batch size from 2 to 1. But there is no guarentee this will the same results. In fact, we tried working with batch_size=1 for a while but never achieved the same top performance as with batch_size=2.

Only the learning rates not the multiplicators (lr_linear_proj_mult ). The weight decay can remain as it is.
What appears different with different batch sizes? You mean why do you have to set different LRs for different batch sizes?

Only the learning rates not the multiplicators (lr_linear_proj_mult ). The weight decay can remain as it is.
What appears different with different batch sizes? You mean why do you have to set different LRs for different batch sizes?

Yes!
As I set the batch_size to 1, are there any other parameters other than the learning rate aforementioned that could have an impact on the results?

Explaining the relation between batch size and learning rate goes beyond the support of this repository. :)
You might have to adjust the number of epochs. but again you will most likely not get the same results easily. This requires some potentially expensive hyperparameter tuning.

I get it! Thanks for your enthusiastic answer anyway!


MLlib TODO items

[ ] Change Scala testName to match Python test names: “glm-regression” —> GLMRegressionTest
[ ] Make parameter names match across all tests. (num-examples, num-rows, etc.)
[ ] Refactor correlation tests so pearson/spearman is a parameter.
[ ] Better data generation in Python

Would be great to measure performance loss in pyspark vs scala for mllib models implemented in scala.

This can be done by running both sets of tests. (They use the same set of parameters in the config file.)  I've done it some, and the change in performance varies based on the particular algorithm.  For long training times, it does not matter.  For prediction, it varies some.  The Spark 1.2 release brought Python a  lot closer to Scala for prediction.

Any plans to test performance new ml API (Pipeline, Crossvalidation, GridSearch, etc.)?

I don't think we will for this release, but we will need to for the next one.  We've been focusing on the API for now, but I hope the API can be stabilized before long.


Updated SAP ERP environment variable doc to reflect new flow

Learn Build status updates of commit 43eeeab:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/sap-integration/env-variables-connection-refs.md
:white_check_mark:Succeeded

articles/sap-integration/media/env-refs/saperp-datasource-variable.png
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

#donotmerge This has not been released to production yet. I'm staging the documentation for when we do release.


Feature/function params and return

Included previous pull request

It may be changed in the future but now that's all.


Transfer Leadership When Leader Leaves Room.

Currently, a room is deleted if the leader leaves the room. This may not be the best experience if the leader is accidentally disconnected, or if the leader has to leave for some reason. The room should continue to exist with a new leader.

When leader leaves the room, make another arbitrary user the new room leader.
If room is empty, delete room.

Closing, moving all project tracking to JIRA.


bug: ion refresher not hiding on IOS

Bug Report
Ionic version:

[x] 5.x
Current behavior: The refresher is not hiding in IOS until I tap anywhere on the screen, it hides itself as normal behavior if you touch outside of the ionic list component. If you tap down from ones of the item in the ion list it doesn't hide.

Expected behavior: The refresher should hide after calling event.target.complete() function

Steps to reproduce: I run the refresher and when I finish loading the content I call event.target.complete() function. The bug reproduce if you tap down slowly, i notice that if i tap down fast it works. I can't reproduce it on a new sidemenu projects, here it works, incredible. It seems like the problem is due to list fetching from DB, with static list it works.

Other information:

Ionic info:

Ionic CLI                     : 6.9.1 (/usr/local/lib/node_modules/@ionic/cli)
Ionic Framework               : @ionic/angular 5.1.0
@angular-devkit/build-angular : 0.803.26
@angular-devkit/schematics    : 8.3.26
@angular/cli                  : 8.3.26
@ionic/angular-toolkit        : 2.2.0

Code reproduction:
page.component.html
<ion-header [translucent]="true">
    <ion-toolbar>
        <ion-buttons slot="start">
            <ion-menu-button></ion-menu-button>
        </ion-buttons>
        <ion-title>{{'interfaces.menu.notifiche' | translate}}</ion-title>
    </ion-toolbar>
</ion-header>
<ion-content [fullscreen]="true">
    <ion-refresher slot="fixed" (ionRefresh)="doRefresh($event)">
        <ion-refresher-content></ion-refresher-content>
    </ion-refresher>
    <ion-header collapse="condense">
        <ion-toolbar>
            <ion-title size="large">{{'interfaces.menu.notifiche' | translate}}</ion-title>
        </ion-toolbar>
    </ion-header>
    <ion-list>
        <ion-list-header lines="inset">
            <ion-label color="primary">{{'interfaces.menu.gestione' | translate}}</ion-label>
        </ion-list-header>
        <ion-item-sliding *ngFor="let notifica of this.listaNotifiche">
            <ion-item (click)="this.goToGestioneNotifica(notifica)">
                <ion-label>
                    <ion-text>

{{notifica.Title}}

                    </ion-text>
                    <ion-text color="medium">

{{notifica.Body}}

                    </ion-text>
                </ion-label>
            </ion-item>
            <ion-item-options side="end">
                <ion-item-option (click)="invia(notifica)">{{'interfaces.notifica.invia' | translate}}
                </ion-item-option>
                <ion-item-option (click)="cancella(notifica)" color="danger">{{'interfaces.notifica.cancella' | translate}}
                </ion-item-option>
            </ion-item-options>
        </ion-item-sliding>
    </ion-list>
</ion-content>

page.component.ts
doRefresh(event) {
   this.currentSubscribe = this.notificheService.getListaNotifiche().subscribe(res => {
   this.listaNotifiche = res.risultato;
   },
      error => {
        this.showError(error);
        event.target.complete();
      },
      () => {
        event.target.complete();
})}

Can you please provide this as a full reproduction? Code snippets are typically not enough to reproduce the issue. The easiest way is to add an Ionic app to a GitHub repo and include a link to it in this thread.

Hi Liam, I'm sorry but I can't play it with a repository on github due to corporate privacy policies. The problem can probably be caused by the speed of execution of the asynchronous request to my backend. In fact, using a timeout function (for example 3 seconds) everything always works.
Sorry again, I hope you can reproduce the bug the same even without my code and correct it.
Thanks.

Can you reproduce this issue in a blank Ionic starter application? That way you do not have to publish your private code.

In a blank ionic starter app using a dummies list of 50 elements added on the fly it alway works. Here for test the issue, i use the setTimeout function and wait 3 second before calling event.target.complete(), everythings works fine.

Thanks for the follow up. This issue seems specific to your project and its configuration. Unfortunately, without a reproduction I am unable to fix the issue.
I am going to close this issue for now. If you are able to create a code reproduction of the issue, feel free to reply here with a link to the repo, and I can reopen the issue. Thanks!

Ok Liam, here you can find repo for reproduction.
https://github.com/olaisen81/ionic-demo

Thanks! I was able to reproduce this behavior. This looks like another instance of https://bugs.webkit.org/show_bug.cgi?id=209968. I reported the issue to the WebKit team last month, but have not heard anything back since.
The problem is that when you update the DOM while rubber band scrolling (pulling from the top/bottom of the screen), the scroll position is reset. This also appears to interfere with the gesture that controls the refresher as the native touchend event is not called. This is why the refresher does not hide.
As a temporary workaround, you can add a setTimeout of 500ms in the doRefresh method:
setTimeout(() => {
  Storage.get({ key: "listTestObj" }).then(res => {
    this.listaNotifiche = JSON.parse(res.value);
    event.target.complete();
  })
}, 500);

While the workaround is not ideal, it should allow the refresher to function properly. I will follow up with the WebKit team and see if I can get any update.

Ok, i'll try this workaround, is not the optimal solution but for now it may be fine. I hope the Webkit team will help you correct it, thanks for the collaboration, I will stay tuned for updates


Unable to use hardware acceleration on Windows

This is log.
2023-07-22T14:57:03.018264Z  INFO apply_execution_providers: ort::execution_providers: CUDA execution provider registered successfully
2023-07-22T14:57:03.053798Z ERROR apply_execution_providers: ort::execution_providers: CUDA execution provider registration failed: D:\a\_work\1\s\onnxruntime\core\session\provider_bridge_ort.cc:1106 onnxruntime::ProviderLibrary::Get [ONNXRuntimeError] : 1 : FAIL : LoadLibrary failed with error 126 "" when trying to load "D:\Project\yolov8_onnx_rust\target\debug\onnxruntime_providers_cuda.dll"

2023-07-22T14:57:03.054196Z  INFO apply_execution_providers: ort::execution_providers: TensorRT execution provider registered successfully
2023-07-22T14:57:03.063023Z ERROR apply_execution_providers: ort::execution_providers: TensorRT execution provider registration failed: D:\a\_work\1\s\onnxruntime\core\session\provider_bridge_ort.cc:1106 onnxruntime::ProviderLibrary::Get [ONNXRuntimeError] : 1 : FAIL : LoadLibrary failed with error 126 "" when trying to load "D:\Project\yolov8_onnx_rust\target\debug\onnxruntime_providers_tensorrt.dll"

I use Windows 11 22621.1992 .
Uses a dll that is automatically downloaded by the programme.

Have you installed CUDA (>=11.6) and cuDNN (>=8.2.4)?
Also make sure to add the cuDNN libraries to PATH, or copy them from /bin to C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v11.8\bin

Ok now it works fine with 1.14, but 1.15 doesn't seem to work using cuda, tensorrt's features.

What CUDA version do you have installed? Are you getting the same LoadLibrary failed with error 126 error in 1.15?

Cuda 11.8
I deleted the compilation cache and now Cuda is fine, but Tensorrt prompts:
2023-07-22T15:59:19.752319Z WARN apply_execution_providers: ort::execution_providers: An error occurred when attempting to register `TensorrtExecutionProvider`: key/value cannot be empty

The TensorRT issue was fixed by #87 and is available in v1.15.4.

The TensorRT issue was fixed by #87 and is available in v1.15.4.

@decahedron1 I still can't use tensorrt provider , but CUDA is fine.
version = 1.15.4, default-features = false, features = ["load-dynamic", "cuda", "tensorrt", "directml", "half"]
apply_execution_providers: ort::execution_providers: An error occurred when attempting to register TensorrtExecutionProvider: D:\a_work\1\s\onnxruntime\core\session\provider_bridge_ort.cc:1131 onnxruntime::ProviderLibrary::Get [ONNXRuntimeError] : 1 : FAIL : LoadLibrary failed with error 126 "" when trying to load "G:\Environment\C\libort\onnxruntime_providers_tensorrt.dll"

@18930215187 you're missing some prerequisite for TensorRT, see the ONNX Runtime and NVIDIA docs.
https://onnxruntime.ai/docs/execution-providers/TensorRT-ExecutionProvider.html#install
https://docs.nvidia.com/deeplearning/tensorrt/install-guide/index.html

@18930215187 you're missing some prerequisite for TensorRT, see the ONNX Runtime and NVIDIA docs.
https://onnxruntime.ai/docs/execution-providers/TensorRT-ExecutionProvider.html#install https://docs.nvidia.com/deeplearning/tensorrt/install-guide/index.html

@decahedron1  Thanks for answer so fast. I checked my environment again and downloaded TensorRT for cuda 11.8, which helped me solve this problem. But another error occured like the following Information.
2023-09-24T17:10:01.506456Z  WARN apply_execution_providers: ort::execution_providers: An error occurred when attempting to registerTensorrtExecutionProvider: D:\a\_work\1\s\onnxruntime\core\providers\tensorrt\tensorrt_execution_provider_info.cc:94 onnxruntime::TensorrtExecutionProviderInfo::FromProviderOptions [ONNXRuntimeError] : 1 : FAIL : provider_options_utils.h:148 onnxruntime::ProviderOptionsParser::Parse Unknown provider option: "trt_force_timing_cache".2023-09-24T17:10:01.506456Z  WARN apply_execution_providers: ort::execution_providers: An error occurred when attempting to register TensorrtExecutionProvider: D:\a\_work\1\s\onnxruntime\core\providers\tensorrt\tensorrt_execution_provider_info.cc:94 onnxruntime::TensorrtExecutionProviderInfo::FromProviderOptions [ONNXRuntimeError] : 1 : FAIL : provider_options_utils.h:148 onnxruntime::ProviderOptionsParser::Parse Unknown provider option: "trt_force_timing_cache".


experimental/search poc

Great! I was thinking about to implement a POC like this on my free time!


Multiple WP on same host

You can't use this cookbook as a wrapper cookbook and it can't be used for multiple installs on the same host as it uses attributes to install.
There should be LWRPs to install wordpress site

I've put up a draft in the lib branch.  It is following the convention of most of the community cookbooks that are turning more into libraries.
Sample usage for installing two sites from tarball on the same machine:
# The following installs the latest version of WordPress to /var/www/wordpress
wordpress_install 'default' do
  action :create
end

# Here's a second installation to a different path that installs a different version
# of WordPress. Additionally the user/group settings are different since you
# may have different users that need permissions/access to different sites.
wordpress_install 'example2.com' do
  install_method 'tarball'
  install_path '/var/www/example2.com'
  version '3.9.3'
  owner 'www-data'
  group 'www-data'
  action :create
end

Another installation option is via unauthenticated git, inspired by #51.
# Defaults to master ref and syncs to /var/www/wordpress
wordpress_install 'default' do
  install_method 'git'
  action :create
end

wordpress_install 'example2.com' do
  install_method 'git'
  install_path '/var/www/example2.com'
  reference '3.9.3'
  owner 'www-data'
  group 'www-data'
  action :create
end

I've put in some basic tests as well. Once I change out how WordPress is installed in the wordpress::app, I'll merge it into master and cut a new release. This sounds like it should fit what you were looking for.

:+1:


Drop support for armv7hf sid base images

@petrosagg please review this PR

@nghiant2710 this has conflicts. Why are we dropping sid?

@petrosagg I will resolve it later but I think we can drop sid since it's quite old, not sure if any users're using it. Sometimes, its repository mirrors are not reachable and our build jobs fail for it

sid is not old at all. It is the unstable version of Debian and has all the latest packages

ok, I will close this PR


CredentialsFactory now buffers entity

Else, requests down the chain can't access the entity.

Coverage remained the same at 100.0% when pulling 61ad0f31a24bf41eb6e4000d66baed154e1c1bdb on feature/buffer_fix into 6061790f0c61b301063f82c57fc22f48f3d49ea3 on develop.


pulled out loadiconSprite function from inline script to its own TypeScript file

Fixes #9811
Fix summary

Pulled out the inline script loadiconSprite to it's own typescript file and made sure it had the correct types
Created a entrypoint file called icons.js that gets the currentScript of the icon-sprite and also gets the iconSprite DOM which is passed as arguments to the initIconSprite function.
Tested on the wagtail bakerydemo site in normal browser mode

Tested on incognito mode

Couldn't test on storybook because od=f storybook error

@Lovelyfin00 is this ready for a review or feedback?
Looks like one of the unit tests is failing

I have issues with testing because of the XMLHttprequest

Note: we could probably use Fetch here instead of XMLHttprequest


Enable backwards-compatible error code adjustments

E.g., for #957, I'd like the old codes to continue to work (at least for a few releases).

Probably also need to add some special-casing for noqa stuff.


Attributes applied to mobs not functioning properly?

dragon_stat_randomiser:
  icon: enchanted_book
  actions:
    cast:
    - class: CheckRequirements
      requirement:
        attribute:
          attribute: dragon_generated
          max: 0.5
      actions:
        - class: ModifyEntity
          mage:
            attributes:
              dragon_generated: 1
              health_dragon: ceil(rand(0,12))
              defence_dragon: ceil(rand(0,12))
              power_dragon: ceil(rand(0,12))
              speed_dragon: ceil(rand(0,12))
              agility_dragon: ceil(rand(0,12))
  parameters:
    bypass_undo: true
    target: other

the following spell is intended to apply a randomised number to these attributes on a Magic Mob (made as a workaround to my last issue) but it has some weird functionality-

Upon initial cast it works as intended, giving each stat a random value 0-12
However, this randomised stat then re-rolls and randomises 0-12 again after any reloads and '/magic load'
The randomised stat is set back to 0 when the server restarts

Is there a way to ensure the randomised stat is saved/will stay the same after the spell is first used?

its just how rand function behaves, i think solution is just to store "ceil(rand(0,12)" into a ModifyVariable action's variables and use them for your ModifyEntity action

its just how rand function behaves, i think solution is just to store "ceil(rand(0,12)" into a ModifyVariable action's variables and use them for your ModifyEntity action

No luck with this unfortunately :( I tried adding the following to the start of the spell:
    - class: ModifyVariable
      variable: hprand
      value: ceil(rand(0,12))

and calling it back later via
        - class: ModifyEntity
          mage:
            attributes:
              health_dragon: hprand

(varying with 'hprand' and '@hprand' and '$hprand') but it doesn't seem to work at all, and the stat remains at 0

What about using command action and /mage command instead of this?

It doesn't fix the problem obviously, I but I'm sure it'll work as a temporary solution

I'll give it a try now!

that's really strange, i need to test it myself...

This is working for me:
  actions:
    cast:
    - class: CustomProjectile
      actions:
        - class: ModifyProperties
          modify_target: "mage"
          modify:
          - property: attributes.ag
            value: rand(0,5)
        - class: Cast
          as_target: true
          spell: check
  parameters:
    target: other
    range: 32

check:
  actions:
    cast:
    - class: AreaOfEffect
      actions:
        - class: Message
          message_target: true
          message: "$ag -  dsadfasf"
  parameters:
    target: self
    target_self: false
    range: 5
    radius: 10
    y_radius: 10```

haven't tested that part, but i assume - yes

something here is definetly broken, these actions are not supposed to revert on restart - it's just breaks the whole purpose of them

Have you tried ModifyAttributes instead of ModifyProperties?
Sorry if I'm not totally following, I'm honestly a little hazy on the difference between the two, especially when using the latter to modify attributes :|

Have you tried ModifyAttributes instead of ModifyProperties?
Sorry if I'm not totally following, I'm honestly a little hazy on the difference between the two, especially when using the latter to modify attributes :|

Tried it now and it also does not work. I assume attributes itself are broken?

Also tried it on both magic and vanilla mobs - same result. So it's not a magic mob's template overwriting the data i guess

This is working for me:

I tried using this spell but it doesn't seem to work at all for me, it doesn't change the mob's stat and always prints '0'. The only things I changed from your original example was the attribute name-
test:
  actions:
    cast:
    - class: CustomProjectile
      actions:
        - class: ModifyProperties
          modify_target: "mage"
          modify:
          - property: attributes.health_dragon
            value: rand(0,5)
        - class: Cast
          as_target: true
          spell: check
  parameters:
    target: other
    range: 32

check:
  actions:
    cast:
    - class: AreaOfEffect
      actions:
        - class: Message
          message_target: true
          message: "$health_dragon"
  parameters:
    target: self
    target_self: false
    range: 5
    radius: 10
    y_radius: 10

Upon initial cast it works as intended, giving each stat a random value 0-12
However, this randomised stat then re-rolls and randomises 0-12 again after any reloads and '/magic load'
The randomised stat is set back to 0 when the server restarts

Is there a way to ensure the randomised stat is saved/will stay the same after the spell is first used?

This also occurs if using SpawnEntity instead. Still struggling to find a way to allow Magic Mobs to keep set-stats after restarts & to allow placeholders for some of these equations.
    - class: SpawnEntity
      type: monstrous_nightmare
      mage:
        attributes:
          speed_dragon: ceil(rand(0,12))

Just another update, I actually did manage to get ModifyAttributes to work on the mob, it sets the random stat without being changed on reloads and /magic load, but it does still reset upon server restart (as @PhvntxmX said in #1226)
    - class: ModifyAttribute
      bypass_undo: true
      attribute: health_dragon
      value: ceil(rand(0,12))

Update: from my testing so far, MagicMob attributes, edits, stats, etc, can be saved with save_non_player_mages: true set in config!

Well that makes sense!
Just keep in mind that is off by default for performance/storage reasons.
It's maybe not a huge deal, but this will cause a file (or db row, or whatever player data storage you're using) to get created for every magic mob you spawn.

Ahh I see, that makes sense. Could there be a way to differentiate/decide if particular mobs get their stats saved? I'm mostly just trying to make sure owned pets (or anything with the persist tag) don't get wiped. I assume this will be an issue as dragons are naturally spawning mobs that need to be tamed? (i.e. every wild dragon ever spawned will have a file generated)

Yeah that's kind of my concern .... are the naturally spawned ones persistent also, or do they not persist until tamed?
Either way it sounds like a special flag to save mobs would be nice.
Alternately I should really look into using the persistent data container on mobs for this kind of thing, I can't remember if magic does anything with that or not.

Naturally spawned dragons don't have the persistent tag until they're tamed, the interaction spell they have (like philbert's) applies the persist: true tag to them and thankfully this seems to stick/remain on server restarts, even if the save_non_player_mages is set to false. With this false, they seems they keep most vanilla attributes and tags even if they're set with magic, it was just things that are solely magic/magic integrated that got wiped (e.g. disguises, attributes)


Revisit and fix bulk payout's Partial - Paid logic

Currently we use partial logic to describe the state between bulk payouts. However it should be meant to be used as a state to track payments between chunks of a job.

Simplify bulk payout logic and transfer sequences from the smart contract level back to the reputation oracle.
Use Partial state always when the outstanding balance after bulk payout is more than 0, otherwise it's Paid

Revisited, works as intended. Another revisit needed when doing a pre-security audit.


Client font color will be broken after pipeline and findstr command are used in Windows CMD

Describe the bug

To Reproduce

Run az network private-endpoint list-types -l eastus2euap
Then run az network private-endpoint list-types -l eastus2euap | findstr vault

Environment summary
azure-cli dev branch 93a50bb
knack 0.6.3

@jiasli Please take a look.

Will be fixed by https://github.com/microsoft/knack/pull/181, https://github.com/Azure/azure-cli/pull/12604

Duplicate of #6080

Duplicate of #6080

This issue has been fixed. Please see https://github.com/Azure/azure-cli/issues/6080#issuecomment-763294594 for the solution.

This issue has been fixed. Please see https://github.com/Azure/azure-cli/issues/6080#issuecomment-763294594 for the solution.


Feature request: Shuffle and repeat option

Sorry for posting feature requests all the time 🙈 .
Would be nice to have that though.
I recently discovered another little GitHub project. Maybe you guys can make great stuff together :)
https://github.com/Ghosty141/MenuBarControls

+1