make launch-kube halts, hangs

Following the steps mentioned in - https://github.com/pachyderm/pachyderm/blob/master/SETUP.md
After cloning the pachyderm repo from git,
the command - make launch-kube gets stuck;
On first run, it couldn't find image locally hence pulled it;
Status showed download complete
However it hangs there/snaps.  However if it can't get in contact it'll just sleep forever.
Install package 'kubernetes-client' to provide command 'kubectl'? [N/y] y

Waiting in queue...
Loading list of packages....
Waiting in queue...
Waiting for authentication...
Waiting in queue...
Downloading packages...
Requesting data...
Testing changes...
Installing packages...
NAME                   CLUSTER-IP   EXTERNAL-IP   PORT(S)    AGE
kubernetes             <IP_ADDRESS>             443/TCP    1h
NAME                   READY        STATUS        RESTARTS   AGE
k8s-etcd-<IP_ADDRESS>     1/1          Running       0          1h
k8s-master-<IP_ADDRESS>   4/4          Running       4          1h
k8s-proxy-<IP_ADDRESS>    1/1          Running       0          1h

[root@localhost pachyderm]# make launch-kube
etc/kube/start-kube-docker.sh
b7d67b55995f195f51761e8df6d1c42a605f9e1a3275f21507622343aa1ad516
[root@localhost pachyderm]# make launch
GOPATH/bin must be on your PATH to access these binaries:
GO15VENDOREXPERIMENT=1 go install ./src/server/cmd/pachctl ./src/server/cmd/pach-deploy
src/server/cmd/pachctl/main.go:5:2: cannot find package "go.pedge.io/env" in any of:
/usr/local/go/src/go.pedge.io/env (from $GOROOT)
/work/src/go.pedge.io/env (from $GOPATH)
Makefile:63: recipe for target 'install' failed
make: *** [install] Error 1
echo $PATH
/sbin:/bin:/usr/sbin:/usr/bin:/usr/local/go/bin

I downloaded go.pedge.io/env  in /usr/local/go/src/
go get go.pedge.io/env
Then when I try "make launch" in /pachyderm
Still the error exists
GOPATH/bin must be on your PATH to access these binaries:
GO15VENDOREXPERIMENT=1 go install ./src/server/cmd/pachctl ./src/server/cmd/pach-deploy
go install: no install location for directory /pachyderm/src/server/cmd/pachctl outside GOPATH
For more details see: go help gopath
go install: no install location for directory /pachyderm/src/server/cmd/pach-deploy outside GOPATH
For more details see: go help gopath
Makefile:63: recipe for target 'install' failed
make: *** [install] Error 1

echo $GOPATH
/work
Work structure is
/work/src/github.com/pachyderm/pachyderm
Thus according to
https://github.com/pachyderm/pachyderm/issues/422
Our repo expects to be located at $GOPATH/src/github.com/pachyderm.
Still the error persists

Hmm, so that error suggests that the repo is located outside of the $GOPATH but based on what you told me that doesn't seem to be the case. Would you mind trying this:
rm -rf /work/src
go get github.com/pachyderm/pachyderm

Then try the commands again, that should download the repo in the right spot and also download all of the dependencies.

GOPATH, GOROOT - environment variables seem to be set to null once I do go get github.com
Hence it gives me error that when I make install (it cant find it in the
[chai@localhost /]$ export GOROOT=$HOME/go
[chai@localhost /]$ export GOROOT=/usr/bin/go
[chai@localhost /]$ export PATH=$PATH:$GOROOT/bin
[chai@localhost /]$ export GOPATH=/work
[chai@localhost /]$ cd work/
[chai@localhost work]$ export GOROOT=/usr/local/go
[chai@localhost work]$ go get github.com/pachyderm/pachyderm
github.com/pachyderm/pachyderm
ar: u' modifier ignored since D' is the default (see `U')
[chai@localhost work]$ sudo systemctl start docker
[sudo] password for chai:
[chai@localhost work]$ sudo su
[root@localhost work]# ls
pkg  src
[root@localhost work]# cd src/github.com/pachyderm/pachyderm/
[root@localhost pachyderm]# make launch-kube
etc/kube/start-kube-docker.sh
6b2bcc4644e9a004b6a27b915fc6cb67de7a00f4a8e1f5116a6d1aad66cc056f
[root@localhost pachyderm]# make launch
GOPATH/bin must be on your PATH to access these binaries:
GO15VENDOREXPERIMENT=1 go install ./src/server/cmd/pachctl ./src/server/cmd/pach-deploy
src/server/cmd/pachctl/main.go:4:2: cannot find package "github.com/pachyderm/pachyderm/src/server/cmd/pachctl/cmd" in any of:
/usr/src/github.com/pachyderm/pachyderm/src/server/cmd/pachctl/cmd (from $GOROOT)
($GOPATH not set)
src/server/cmd/pachctl/main.go:5:2: cannot find package "go.pedge.io/env" in any of:
/usr/src/go.pedge.io/env (from $GOROOT)
($GOPATH not set)
src/server/cmd/pach-deploy/main.go:6:2: cannot find package "github.com/pachyderm/pachyderm/src/server/pkg/deploy/cmds" in any of:
/usr/src/github.com/pachyderm/pachyderm/src/server/pkg/deploy/cmds (from $GOROOT)
($GOPATH not set)
Makefile:63: recipe for target 'install' failed
make: *** [install] Error 1
[root@localhost pachyderm]# echo $GOPATH
[root@localhost pachyderm]# echo $GOROOT
[root@localhost pachyderm]#
When I set GOROOT to point to location where it is installed
[root@localhost pachyderm]# export GOROOT=/usr/local/go
GOPATH to /work (where I did go get pachyderm)
and try doing
make launch it says
[root@localhost pachyderm]# make launch
GOPATH/bin must be on your PATH to access these binaries:
GO15VENDOREXPERIMENT=1 go install ./src/server/cmd/pachctl ./src/server/cmd/pach-deploy
src/server/pfs/fuse/filesystem.go:15:2: cannot find package "bazil.org/fuse" in any of:
/usr/local/go/src/bazil.org/fuse (from $GOROOT)
/work/src/bazil.org/fuse (from $GOPATH)
src/server/pfs/fuse/filesystem.go:16:2: cannot find package "bazil.org/fuse/fs" in any of:
/usr/local/go/src/bazil.org/fuse/fs (from $GOROOT)
/work/src/bazil.org/fuse/fs (from $GOPATH)
src/server/pps/cmds/cmds.go:15:2: cannot find package "github.com/Jeffail/gabs" in any of:
/usr/local/go/src/github.com/Jeffail/gabs (from $GOROOT)
/work/src/github.com/Jeffail/gabs (from $GOPATH)
src/client/pkg/discovery/etcd_client.go:6:2: cannot find package "github.com/coreos/go-etcd/etcd" in any of:
/usr/local/go/src/github.com/coreos/go-etcd/etcd (from $GOROOT)
/work/src/github.com/coreos/go-etcd/etcd (from $GOPATH)
src/server/pfs/pretty/pretty.go:10:2: cannot find package "github.com/docker/go-units" in any of:
/usr/local/go/src/github.com/docker/go-units (from $GOROOT)
/work/src/github.com/docker/go-units (from $GOPATH)
src/server/pps/pretty/pretty.go:9:2: cannot find package "github.com/fatih/color" in any of:
/usr/local/go/src/github.com/fatih/color (from $GOROOT)
/work/src/github.com/fatih/color (from $GOPATH)
src/client/pfs/pfs.pb.go:60:8: cannot find package "github.com/gengo/grpc-gateway/third_party/googleapis/google/api" in any of:
/usr/local/go/src/github.com/gengo/grpc-gateway/third_party/googleapis/google/api (from $GOROOT)
/work/src/github.com/gengo/grpc-gateway/third_party/googleapis/google/api (from $GOPATH)
src/client/pkg/shard/sharder.go:12:2: cannot find package "github.com/golang/protobuf/jsonpb" in any of:
/usr/local/go/src/github.com/golang/protobuf/jsonpb (from $GOROOT)
/work/src/github.com/golang/protobuf/jsonpb (from $GOPATH)
src/client/pfs/pfs.pb.go:57:8: cannot find package "github.com/golang/protobuf/proto" in any of:
/usr/local/go/src/github.com/golang/protobuf/proto (from $GOROOT)
/work/src/github.com/golang/protobuf/proto (from $GOPATH)
src/client/pkg/uuid/uuid.go:6:2: cannot find package "github.com/satori/go.uuid" in any of:
/usr/local/go/src/github.com/satori/go.uuid (from $GOROOT)
/work/src/github.com/satori/go.uuid (from $GOPATH)
src/server/pkg/cmd/cmd.go:9:2: cannot find package "github.com/spf13/cobra" in any of:
/usr/local/go/src/github.com/spf13/cobra (from $GOROOT)
/work/src/github.com/spf13/cobra (from $GOPATH)
src/server/cmd/pachctl/main.go:5:2: cannot find package "go.pedge.io/env" in any of:
/usr/local/go/src/go.pedge.io/env (from $GOROOT)
/work/src/go.pedge.io/env (from $GOPATH)
src/server/pfs/fuse/mounter.go:11:2: cannot find package "go.pedge.io/lion" in any of:
/usr/local/go/src/go.pedge.io/lion (from $GOROOT)
/work/src/go.pedge.io/lion (from $GOPATH)
src/client/pkg/shard/sharder.go:14:2: cannot find package "go.pedge.io/lion/proto" in any of:
/usr/local/go/src/go.pedge.io/lion/proto (from $GOROOT)
/work/src/go.pedge.io/lion/proto (from $GOPATH)
src/client/pfs/pfs.pb.go:61:8: cannot find package "go.pedge.io/pb/go/google/protobuf" in any of:
/usr/local/go/src/go.pedge.io/pb/go/google/protobuf (from $GOROOT)
/work/src/go.pedge.io/pb/go/google/protobuf (from $GOPATH)
src/server/pfs/cmds/cmds.go:17:2: cannot find package "go.pedge.io/pkg/cobra" in any of:
/usr/local/go/src/go.pedge.io/pkg/cobra (from $GOROOT)
/work/src/go.pedge.io/pkg/cobra (from $GOPATH)
src/client/pfs.go:8:2: cannot find package "go.pedge.io/proto/stream" in any of:
/usr/local/go/src/go.pedge.io/proto/stream (from $GOROOT)
/work/src/go.pedge.io/proto/stream (from $GOPATH)
src/server/pfs/fuse/filesystem.go:21:2: cannot find package "go.pedge.io/proto/time" in any of:
/usr/local/go/src/go.pedge.io/proto/time (from $GOROOT)
/work/src/go.pedge.io/proto/time (from $GOPATH)
src/client/version/version.go:7:2: cannot find package "go.pedge.io/proto/version" in any of:
/usr/local/go/src/go.pedge.io/proto/version (from $GOROOT)
/work/src/go.pedge.io/proto/version (from $GOPATH)
src/client/pfs/pfs.pb.go:67:2: cannot find package "golang.org/x/net/context" in any of:
/usr/local/go/src/golang.org/x/net/context (from $GOROOT)
/work/src/golang.org/x/net/context (from $GOPATH)
src/client/pkg/grpcutil/dialer.go:4:2: cannot find package "google.golang.org/grpc" in any of:
/usr/local/go/src/google.golang.org/grpc (from $GOROOT)
/work/src/google.golang.org/grpc (from $GOPATH)
src/server/pfs/fuse/filesystem.go:24:2: cannot find package "google.golang.org/grpc/codes" in any of:
/usr/local/go/src/google.golang.org/grpc/codes (from $GOROOT)
/work/src/google.golang.org/grpc/codes (from $GOPATH)
src/server/pkg/obj/amazon_client.go:7:2: cannot find package "github.com/aws/aws-sdk-go/aws" in any of:
/usr/local/go/src/github.com/aws/aws-sdk-go/aws (from $GOROOT)
/work/src/github.com/aws/aws-sdk-go/aws (from $GOPATH)
src/server/pkg/obj/amazon_client.go:8:2: cannot find package "github.com/aws/aws-sdk-go/aws/credentials" in any of:
/usr/local/go/src/github.com/aws/aws-sdk-go/aws/credentials (from $GOROOT)
/work/src/github.com/aws/aws-sdk-go/aws/credentials (from $GOPATH)
src/server/pkg/obj/amazon_client.go:9:2: cannot find package "github.com/aws/aws-sdk-go/aws/session" in any of:
/usr/local/go/src/github.com/aws/aws-sdk-go/aws/session (from $GOROOT)
/work/src/github.com/aws/aws-sdk-go/aws/session (from $GOPATH)
src/server/pkg/obj/amazon_client.go:10:2: cannot find package "github.com/aws/aws-sdk-go/service/s3" in any of:
/usr/local/go/src/github.com/aws/aws-sdk-go/service/s3 (from $GOROOT)
/work/src/github.com/aws/aws-sdk-go/service/s3 (from $GOPATH)
src/server/pkg/obj/amazon_client.go:11:2: cannot find package "github.com/aws/aws-sdk-go/service/s3/s3Manager" in any of:
/usr/local/go/src/github.com/aws/aws-sdk-go/service/s3/s3Manager (from $GOROOT)
/work/src/github.com/aws/aws-sdk-go/service/s3/s3Manager (from $GOPATH)
src/server/pkg/obj/obj.go:7:2: cannot find package "github.com/cenkalti/backoff" in any of:
/usr/local/go/src/github.com/cenkalti/backoff (from $GOROOT)
/work/src/github.com/cenkalti/backoff (from $GOPATH)
src/server/pfs/server/obj_block_api_server.go:15:2: cannot find package "github.com/gogo/protobuf/proto" in any of:
/usr/local/go/src/github.com/gogo/protobuf/proto (from $GOROOT)
/work/src/github.com/gogo/protobuf/proto (from $GOPATH)
src/server/pkg/metrics/segment.go:4:2: cannot find package "github.com/segmentio/analytics-go" in any of:
/usr/local/go/src/github.com/segmentio/analytics-go (from $GOROOT)
/work/src/github.com/segmentio/analytics-go (from $GOPATH)
src/server/pkg/deploy/assets/assets.go:9:2: cannot find package "github.com/ugorji/go/codec" in any of:
/usr/local/go/src/github.com/ugorji/go/codec (from $GOROOT)
/work/src/github.com/ugorji/go/codec (from $GOPATH)
src/server/pfs/server/api_server.go:19:2: cannot find package "go.pedge.io/proto/rpclog" in any of:
/usr/local/go/src/go.pedge.io/proto/rpclog (from $GOROOT)
/work/src/go.pedge.io/proto/rpclog (from $GOPATH)
src/server/pkg/obj/google_client.go:7:2: cannot find package "golang.org/x/oauth2/google" in any of:
/usr/local/go/src/golang.org/x/oauth2/google (from $GOROOT)
/work/src/golang.org/x/oauth2/google (from $GOPATH)
src/server/pkg/obj/google_client.go:8:2: cannot find package "google.golang.org/cloud" in any of:
/usr/local/go/src/google.golang.org/cloud (from $GOROOT)
/work/src/google.golang.org/cloud (from $GOPATH)
src/server/pkg/obj/google_client.go:9:2: cannot find package "google.golang.org/cloud/storage" in any of:
/usr/local/go/src/google.golang.org/cloud/storage (from $GOROOT)
/work/src/google.golang.org/cloud/storage (from $GOPATH)
src/server/pfs/server/api_server.go:24:2: cannot find package "google.golang.org/grpc/metadata" in any of:
/usr/local/go/src/google.golang.org/grpc/metadata (from $GOROOT)
/work/src/google.golang.org/grpc/metadata (from $GOPATH)
src/server/pkg/metrics/external.go:6:2: cannot find package "k8s.io/kubernetes/pkg/api" in any of:
/usr/local/go/src/k8s.io/kubernetes/pkg/api (from $GOROOT)
/work/src/k8s.io/kubernetes/pkg/api (from $GOPATH)
src/server/pkg/deploy/assets/assets.go:11:2: cannot find package "k8s.io/kubernetes/pkg/api/resource" in any of:
/usr/local/go/src/k8s.io/kubernetes/pkg/api/resource (from $GOROOT)
/work/src/k8s.io/kubernetes/pkg/api/resource (from $GOPATH)
src/server/pkg/deploy/assets/assets.go:12:2: cannot find package "k8s.io/kubernetes/pkg/api/unversioned" in any of:
/usr/local/go/src/k8s.io/kubernetes/pkg/api/unversioned (from $GOROOT)
/work/src/k8s.io/kubernetes/pkg/api/unversioned (from $GOPATH)
src/server/pkg/deploy/assets/assets.go:13:2: cannot find package "k8s.io/kubernetes/pkg/apis/extensions" in any of:
/usr/local/go/src/k8s.io/kubernetes/pkg/apis/extensions (from $GOROOT)
/work/src/k8s.io/kubernetes/pkg/apis/extensions (from $GOPATH)
src/server/pkg/metrics/external.go:7:2: cannot find package "k8s.io/kubernetes/pkg/client/unversioned" in any of:
/usr/local/go/src/k8s.io/kubernetes/pkg/client/unversioned (from $GOROOT)
/work/src/k8s.io/kubernetes/pkg/client/unversioned (from $GOPATH)
Makefile:63: recipe for target 'install' failed
make: *** [install] Error 1
[root@localhost pachyderm]#

After go get of all the dependencies I got error in 4 dependencies related to k8s.io
make launch
GOPATH/bin must be on your PATH to access these binaries:
GO15VENDOREXPERIMENT=1 go install ./src/server/cmd/pachctl ./src/server/cmd/pach-deploy
src/server/pkg/metrics/external.go:6:2: no buildable Go source files in /usr/local/go/src/k8s.io/kubernetes/pkg/api
src/server/pkg/deploy/assets/assets.go:11:2: cannot find package "k8s.io/kubernetes/pkg/api/resource" in any of:
/usr/local/go/src/k8s.io/kubernetes/pkg/api/resource (from $GOROOT)
/work/src/k8s.io/kubernetes/pkg/api/resource (from $GOPATH)
src/server/pkg/deploy/assets/assets.go:12:2: cannot find package "k8s.io/kubernetes/pkg/api/unversioned" in any of:
/usr/local/go/src/k8s.io/kubernetes/pkg/api/unversioned (from $GOROOT)
/work/src/k8s.io/kubernetes/pkg/api/unversioned (from $GOPATH)
src/server/pkg/deploy/assets/assets.go:13:2: no buildable Go source files in /usr/local/go/src/k8s.io/kubernetes/pkg/apis/extensions
src/server/pkg/metrics/external.go:7:2: cannot find package "k8s.io/kubernetes/pkg/client/unversioned" in any of:
/usr/local/go/src/k8s.io/kubernetes/pkg/client/unversioned (from $GOROOT)
/work/src/k8s.io/kubernetes/pkg/client/unversioned (from $GOPATH)
Makefile:63: recipe for target 'install' failed
make: *** [install] Error 1

Hi there, sorry you're running into all these snags, but thanks for all the debugging diligence you're doing.
There seem to be two issues here:

Your go environment variables aren't set

Not sure why these are missing. It's standard practice to set these in your shell init script (e.g. ~/bash_profile). Are you setting them there?
You'll definitely need these set. You've identified that they're missing, which is great, but I want to make sure that these are always present, or you'll run into a lot of issues along the way.

You're missing all the dependencies for compiling the pachyderm source

What version of go are you using? (Please run go version and let us know what it reports). You should be using go 1.6
You errors around building suggest that the compiler isn't using the vendored copies of those dependencies from within our repo. You shouldn't have to manually grab each of them. The reason we vendor our dependencies is to lock them to specific versions that we know the pachyderm code is compatible with.

1.I used to export $PATH, $GOPATH, $GOROOT and set the environment variables. However now I have set them in shell init script as told (although in few internet articles export also seemed to work, maybe temporarily)
2.I have downloaded the tar file from https://golang.org/doc/install of latest version 1.6+
in usr/local I have the extracted tar
But
[root@localhost /]# go version
go version go1.4.2 gccgo (GCC) 5.3.1 20160406 (Red Hat 5.3.1-6) linux/amd64
I again tried with
[root@localhost /]# export PATH=$PATH:/usr/local/go/bin
[root@localhost /]# go version
go version go1.4.2 gccgo (GCC) 5.3.1 20160406 (Red Hat 5.3.1-6) linux/amd64

Issue was resolved
Problem
2 versions of go installed
1st GCC GO version 1.4 - already installed on my Fedora as a dev tool
2nd Go version 1.6
According to doc, 1.4 which was my default wasnt compatible with pachyderm and hence had to be removed, path changed and removed from Hash of the shell
Finally right go version and path
[root@localhost /]# go version
go version go1.6.2 linux/amd64
[root@localhost /]# which go
/usr/local/go/bin/go
Following steps followed

Get the git along with dependencies
[root@localhost work]# go get github.com/pachyderm/pachyderm

2.Starting docker
[root@localhost work]# sudo systemctl start docker

[root@localhost work]# cd ../workspace/
[root@localhost workspace]# ls
pkg  src
[root@localhost workspace]# cd src/github.com/pachyderm/pachyderm/
[root@localhost pachyderm]# make launch-kube
etc/kube/start-kube-docker.sh
62a213fadcd1cb4cc065414a020359259b7e0f617661c95460ddb3c53e663a37
Kubernete launched

Pachyderm deployed on kubernete
[root@localhost pachyderm]# make launch

GOPATH/bin must be on your PATH to access these binaries:
GO15VENDOREXPERIMENT=1 go install ./src/server/cmd/pachctl ./src/server/cmd/pach-deploy
kubectl  create -f etc/kube/pachyderm.json
serviceaccount "pachyderm" created
replicationcontroller "etcd" created
service "etcd" created
You have exposed your service on an external port on all nodes in your
cluster.  If you want to expose this service to the external internet, you may
need to set up firewall rules for the service port(s) (tcp:32080,tcp:32081,tcp:32082) to serve traffic.
See http://releases.k8s.io/release-1.2/docs/user-guide/services-firewalls.md for more details.
service "rethink" created
replicationcontroller "rethink" created
job "pachd-init" created
You have exposed your service on an external port on all nodes in your
cluster.  If you want to expose this service to the external internet, you may
need to set up firewall rules for the service port(s) (tcp:30650) to serve traffic.
See http://releases.k8s.io/release-1.2/docs/user-guide/services-firewalls.md for more details.
service "pachd" created
replicationcontroller "pachd" created
wait for the pachyderm to come up
until timeout 1s ./etc/kube/check_pachd_ready.sh; do sleep 1; done
Issue is - nothing is happening after this

Closing this as there isn't anything actionable left to be done.
One concrete change that came out of this is that we now check that kubectl is installed before we enter the wait loop. That should prevent the initial issue here.


Make retry policy internal

Reverting this to internal for now to allow more time for review.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Core


PoC of resizing the decoding buffer

In decodeLine and decodePolygon check the max buffer size to resize the static buffer if required.
Default values for line buffers are 4Mb and for polygon buffers to 8Mb


The system tray icon does not appear notion-app

System: Arch
desktop: KDE
The system tray icon does not appear with notion-app. This happened when I let notion start with the system and latte does not recognize it anymore.

@Emporioe I was unable to reproduce this. Is this still ocurring?


prestosql/presto removed from Dockerhub

The PrestoContainer is using prestosql/presto which does no longer exists on Dockerhub.
Not sure whats the proper substitution, as far as I've read Presto got renamed to Trino so it might be https://hub.docker.com/r/trinodb/trino, but then LogMessageWaitStrategy must be adjusted as the package name differs:
INFO    main    io.trino.server.Server    ======== SERVER STARTED ========

This was resolved in #3668 - Presto was deprecated and replace with TrinoContainer


Its pretty much a word guessing game but with a twist where an incorrect letter guess may change the visible letters location
How do you play your game?
A - move back a letter
D - Move forward to next letter
W - confirm letter selection
I - to restart game
L - to add custom words
Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x] The game name is not the same as the others from gallery

Image (If an image is used)

[ ] The image is in the /games/img directory.
[ ] The name of the image matches the name of your file.

Thanks for your PR!

Nice job with this game. I'm approving it, and you'll see it in the gallery this week- congrats!
If you're a teenager, you can request a free Sprig console at this link and join our online Slack community to chat with other people making Sprig games.

The sprig is great just missing the 2 spacers thanks On 3 Sept 2024 5:51 pm, Cheru Berhanu @.***> wrote:
#2276 was added to the merge queue.
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you modified the open/close state.Message ID: @.***>


App doesn't compile with generic destination on xcodebuild

I try to compile my app using this command
xcodebuild -scheme 'MyApp'   -sdk 'iphonesimulator' -destination 'generic/platform=iOS Simulator'
but it gives me error always
protocos/MyAppService.swift:5:17: external macro implementation type 'MockableMacro.MockableMacro' could not be found for macro 'Mockable()'
It will run fine if changed the destination into -destination 'platform=iOS Simulator,name=iPhone'
but that will prevent me from using the .app file it generates in the E2E testing i have.

I can repro this on an example project, will update you if I find anything.

Not sure why, but what I have found so far is that if you omit the -sdk 'iphonesimulator' flag, it compiles...
Are you sure you need to specify the base SDK explicitly?

I tried. but it crash when dragging the .app to simulator and try to run it. i think without sdk it takes the default which is probably the real device

@emadhegab I will close this for now due to inactivity, will reopen if needed.

Someting similar is happening to me in archiving in the CI:
** ARCHIVE FAILED **

The following build commands failed:
	ComputeTargetDependencyGraph

If I remove Mockable from all module dependencies, comment imports and macros, then it works. The command looks like:
xcodebuild -workspace MyProject.xcworkspace -scheme MyScheme -configuration MyConfig -destination 'generic/platform=iOS' -skipPackagePluginValidation clean archive

A minor finding:
If I leave my codebase like this:
#if canImport(Mockable)
import Mockable
#endif

#if canImport(Mockable)
@Mockable
#endif
protocol BlaBla {

And my Package.swift like this:
dependencies: [
        .package(path: "../PackageA"),
        .package(path: "../PackageB"),
        .package(path: "../PackageC"),
        .package(path: "../PackageD"),
        .package(url: "https://github.com/hmlongco/Factory", exact: "2.3.2"),
        .package(url: "https://github.com/Kolos65/Mockable", exact: "0.0.9"),
        .package(url: "https://github.com/realm/SwiftLint", exact: "0.55.1")
    ],
    targets: [
        .target(name: "MyPackage",
                dependencies: ["PackageA", "PackageB", "Factory", "PackageC", "PackageD"/*, .product(name: "Mockable", package: "Mockable")*/],
                path: "Sources",
                swiftSettings: [.define("MOCKING", .when(configuration: .debug))],
                plugins: [.plugin(name: "SwiftLintBuildToolPlugin", package: "SwiftLint")]),
        .testTarget(name: "MyPackageTests", dependencies: ["MyPackage", .product(name: "MockableTest", package: "Mockable")]),
    ]

Then it still works on CI, because the produces product is not included in the target, and at the same time is "easy" to activate it (just uncomment that part) without changing the actual codebase.
Still trying to figure out the cause of this issue, I'll use this as a temporary fix.

@carlossolana-aily Have you tried adding -skipMacroValidation to the xcodebuild command?

No, I didn't! Now I reverted everything and it works! Thank you very much! :D

Closing this as I don't see any library specific bug or issue here.


integrate polymorphic types with beacon-api-client

e.g. https://github.com/ralexstokes/beacon-api-client/blob/fc8a84c7f1e9fc24e49e5f86dd896d025a3d5241/src/api_client.rs#L558nn
this endpoint only works w/ phase0 currently
I think my preference is to support polymorphic types where possible in https://github.com/ralexstokes/ethereum-consensus and where needed add them here

update:
after #276, we just need to use the new types under ethereum_consensus::types::* and ensure the serde goes into the correct variant based on the fork metadata


Add the ability to change format per instance

There's only a class-specific gobal format available right now, however it would be great if there was a way to change format per instance or even per call to toFormat.

Okay, I have done what you suggested.
The formatting can now be from the format property of both constructor and instance, and/or from an object passed to the method itself.
See the README for examples.


Doesn't work if Alt+Spacebar Disable is disabled when using tools in Photoshop

Microsoft PowerToys version
v0.64.0
Installation method
Microsoft Store
Running as admin
Yes
Area(s) with issue?
Keyboard Manager
Steps to reproduce

✔ Expected Behavior
Hello! The shortcut I disabled (Alt+Spacebar) works fine elsewhere, but not stable in Photoshop, I hope this shortcut doesn't appear in the software anymore。
❌ Actual Behavior
Regarding the error in Photoshop, at present I found that there will be frequent failures in the Crop tool, which is also my usual operation behavior. I hope this error can be corrected, thank you.
Other Software
adobe photoshop (all series)

Are you still experiencing this issue? /needinfo


output filename

can you include a fuction like "setFileName" option instead only of doing random degits.
i managed to do file name through sessions.

can you include a fuction of speed?

Thanks for your interest in this personal project. I'm not planning to re-visit this project at this time, as I state in contributing:

This is a personal project. Contributions are not required. Anyone interested in developing this project are welcome to fork or clone for your own use.

Feel free to take a fork and amend as required, the project is MIT licensed.
The class can be amended to set the file name either in the constructor or a setFileName method. The code can be changed to see if this is set, if it isn't set use the original random file method. The other way is to take the random file which is returned and use PHP file functions to rename or move the file as required.
I hope this helps.

Closing issue


Add PHP runtime config option to enable serving static files

It's a common use case to allow PHP app serve static files with popular frameworks like Laravel. This should still be not enabled by default since some use cases specifically do not want to expose static files.

We shouldn't add more fields to the app.yaml. Rather, we should read it as a build environment variable.

Thanks @kennethye1 for your response. I have modified the PR to use environment variable instead of runtime config. PTAL, Thanks.

/gcbrun

Hi @kennethye1, thanks for reviewing and merging the PR! May I know when this will be rolled out to GAE?

I expect 2-3 weeks. You'll need specify the var in the build_env_variables in your app.yaml.

Can I check something with you @sonnysasaka and @kennethye1 please?
I was trying out your code snippets from this PR:
	{{if .ServesStaticFiles}}
	location / {
		try_files $uri /{{.FrontControllerScript}}$uri;
	}
	{{else}}

https://github.com/GoogleCloudPlatform/buildpacks/pull/386/files#diff-6ee4ff6f236ce9789b9d8f4037af3f7b606b3c86f73ccb02c55b659d4660cbd0R95
I assume {{.FrontControllerScript}} is usually public/index.php? It was in my case.
If that's true, it becomes try_files $uri /public/index.php and serves static files from the main application folder, NOT from the public folder as intended - am I mistaken here?

If using app engine, public is specified as the document_root in app.yaml: https://cloud.google.com/appengine/docs/flexible/php/configuring-your-app-with-app-yaml.
I think you can also make an app.yaml in your root directory to specify the document_root even though you are not using app engine, because this buildpacks reads the app.yaml if you specify GAE_APPLICATION_YAML_PATH=./app.yaml environment variable.

Thanks for the clarification @sonnysasaka ,
Looking at the buildpacks for Cloud Run version I was testing, it didn't seem to do that:
listen 8080 default_server;
listen [::]:8080 default_server;
server_name "";
root /workspace;

location / {
    try_files $uri /index.php$uri;
}

However, on closer inspection, it hasn't defaulted to the front controller being public/index.php and I can't see a way documented to specify this, outside of the usual way for App Engine (Flex or Standard).
I'll add it to the list of things to check up on with PHP on buildpacks for Cloud Run, as we noticed a couple of other things, like the available extensions & their versions don't match App Engine either.

Thanks @sonnysasaka ,
Something like this should be the easiest way to demo it:
buildpack-demo.zip
Built with pack build sample-php --builder=gcr.io/buildpacks/builder
Although I don't think this env var is available in the main builder yet, so when built, I ran:
docker run --entrypoint /bin/bash -it sample-php
Then in /layers/google.php.webconfig/webconfig/nginxserver.conf, you can see:
root /workspace;

rewrite ^/(.*)$ /index.php$uri;

which would be substituted for your try_files line with the newer builder with the env option, although this still shows the front controller not being in the right location, i.e. /index.php not /public/index.php.

By default frontcontroller is index.php.  Looks like we only allow overrides for the other configurations on App Engine Flex https://github.com/GoogleCloudPlatform/buildpacks/blob/7ea79c10cec45c2f8b0ac26afd11ef6fa403e229/cmd/php/webconfig/main.go#L76-L83
For this PR you need to run pack build img --env NGINX_SERVES_STATIC_FILES=true for the change to apply.

Thanks for the confirmation @kennethye1 ,
I guess this PR isn't really the right place to raise it, but all the same, being able to update the front controller will be pretty important for making the buildpack usable for Cloud Run, for a Laravel type setup.
Your link also confirms my suspicion that it isn't possible to update the php-fpm worker config either, it's hardcoded like this:
; Create child processes with a static policy.
pm = static

; The number of child processes to be created
pm.max_children = 2

On App Engine Standard, this is configurable via serve:

The entrypoint field uses the built-in serve command, which is a program within the PHP 7/8 runtimes that starts up the php-fpm implementation and a webserver in the background. This webserver routes all traffic to the provided PHP file using the front controller design pattern.
The serve command has two optional flags:
--workers=N: specifies the N number of php-fpm workers. If you don't set the --workers flag, the serve command guesses the number of workers based on the amount of memory available. For best results, set the values for the --workers flag on the serve command and the max_concurrent_requests element to be the same number.
--enable-dynamic-workers: specifies that you'd like the php-fpm workers to be spawned only as needed. The default is to use a static policy for the php-fpm workers. You can change the number maximum number of spawned workers by using the --workers=N flag. By default the maximum number of spawned workers defaults to that set by the static policy.
The optional flags must come before the front controller path:
entrypoint: serve --workers=2 --enable-dynamic-workers path/to/index.php

https://cloud.google.com/appengine/docs/standard/php-gen2/runtime#app_startup
As the documentation states, this is also quite important for Cloud Run, to be tuned in line with the maximum concurrent requests configured in the deployment.

I think document root should be overridable even outside app engine, so I sent out this PR: https://github.com/GoogleCloudPlatform/buildpacks/pull/390 @kennethye1
@iamacarpet in the meantime, if you want to unblock your project, you can use the hack to let this buildpack use app.yaml like this:
pack build sample-php --builder=gcr.io/gae-runtimes/buildpacks/google-gae-22/php/builder --env GAE_APPLICATION_YAML_PATH=./app.yaml --env GOOGLE_FLEX_APPLICATION=1

and in your app.yaml:
runtime_config:
  document_root: public

Tested this with your buildpack-demo.zip and verified it can serve the public directory.

for php-fpm configuration, it's already configurable by placing you php-fpm.conf in the root directory, tested it with this content:
pm.max_children = 4

Also @kennethye1, looks like NGINX_SERVES_STATIC_FILES is already rolled out to gcr.io/gae-runtimes/buildpacks/google-gae-22/php/builder, can I rely on this to be in app engine builder? If so we may mark https://github.com/GoogleCloudPlatform/buildpacks/issues/323 as closed as well.

@iamacarpet if you need to unblock your progress for now, this is the example to pretend to be app engine flex env: https://github.com/sonnysasaka/buildpack-demo/commit/24061851250730906e55e3e5429d4592e7a221e3
Configuring document root can be done with environment variable after https://github.com/GoogleCloudPlatform/buildpacks/pull/390

Thanks for all the help @sonnysasaka ,
Great solutions, I appreciate it - and yes, with php-fpm, I thought having a file in the root directory would completely replace the php-fpm config, but I can see it's just an extra include, which is helpful :).
Your PR for changing the document root with an environment variable is also super helpful, thank you.
I'm curious what you think about #392, if it's possibly a more developer friendly way forward?


[Docs] Docs for Heroku with examples of Procfile and env vars

Acceptance

[ ] Write proper docs for Heroku deploy (describe Procfile and ENV vars configuration)

The docs now contain more information about configuration using .env files. I'm not sure what a Procfile is (something Heroku specific?). If we want to add a "deploying on Heroku" guide to the docs, I'm all for that (but I don't have that knowledge). If we don't, I suggest we close this.
/cc @rufuspollock


Added a routine to make Viewport immutable before drawing canvas

I assume, that this solves #849.

Saw, that we use in some parts Copy() and in some parts Clone().


Wrong behavior of Observable type

Bug Report
Current Behavior
If I define Observable<never> I expect a TS error when I subscribe to the Observable.
But at the moment no related error.
Reproduction

link: https://stackblitz.com/edit/observable-never

let o: Observable<never> = new Observable((observer) => {});

// no TS error here
o.subscribe(() => {

});

Expected behavior
TS throws an error when I subscribe to Observable.
Environment

Runtime: Node v13.12.0, npm 6.14.4
RxJS version: 6.5.4

Not a bug. TypeScript knows nothing about the semantics of subscribe and there is no type conflict with the passed callback.
https://www.typescriptlang.org/docs/handbook/release-notes/typescript-2-0.html#the-never-type

@cartant
If this is not a bug when should I use Observable<never>? What advantages does such a type provide to programmer? What are the use cases for such a type?

@Webbrother Angular source code has plenty of use-cases:

https://github.com/angular/angular/blob/f8096d499324cf0961f092944bbaedd05364eea1/packages/service-worker/src/update.ts#L42-L43

https://github.com/angular/angular/blob/5eb7426216b1597cca3a7923cc714e59464fec89/packages/zone.js/test/rxjs/rxjs.never.spec.ts#L20

https://github.com/angular/angular/blob/65aaffcfe450b0e2149cf82b910db595909456e4/packages/service-worker/src/push.ts#L130

Closing for reasons outlined in comments above.


PP-6065 switch ChargeRefundsResource to use ChargeService

WHAT YOU DID

use ChargeService instead of ChargeDao

This is due to the new purge of charges change

I think our APIs default strategy is to go to Ledger for refunds, I don't think these resources need to be updated to factor in expunged charges. We'll have to review the behaviour of get one refund but this should end up going to Ledger anyway.


add new method for get information about Lock

This addition method provide snapshot about current lock state. who is owner, which application instance (connection id) and the last expire time point

@mrniko do you like the changes, or I can close the MR?
I ask it because has no experience with open source and I don't understood that should i resolve conflicts or no ;/

A lot of changes not related to code changes for this issues.
LockInfo should contain redissonId - id of Redisson instance which owns the lock.

@mrniko ok I will learn how to get this id and then I will improve the Mr or create a new one without so match changes


[openssl] legacy load failed on Windows Server 2019

Is your feature request related to a problem? Please describe.
cmd> vcpkg install openssl:x64-windows
legacy provider is installed as dynamic library in “vcpkg installed" directory on Windows，but static library on Linux。
Windows 2019 can not find legacy.dll path by default，I need to copy legacy.dll to target PC and set module search path
by function OSSL_PROVIDER_set_default_search_path in my source code before I use OSSL_PROVIDER_load.
vcpkg integrate with vs2022, when projects build finished, file "legacy.dll" didn't copy to target output directory, but only libssl and libcrypto.
Proposed solution
Could you please build openssl dynamic library  with static legacy provider library inside?  If not, could you please copy "legacy.dll” to target directory such as “[OutPutDir]/ossl-modules”
Describe alternatives you've considered
No response
Additional context
No response

legacy provider is installed as dynamic library in “vcpkg installed" directory on Windows，but static library on Linux。

I think it simply follows the triplet settings.

Could you please build openssl dynamic library with static legacy provider library inside?

Does OpenSSL officially provide this configuration (options)?

If not, could you please copy "legacy.dll” to target directory such as “[OutPutDir]/ossl-modules”

vcpkg has no general solution to copy modules aka plugins.


Enhance migration experience for cv_camera pkg users

Hi, first of all thanks for the quick response on my PR earlier.
I am not sure if possible but if you want to make migration towards your package easier for people that are coming from cv_camera pkg then it would be great to have the full parameter set available (http://wiki.ros.org/cv_camera#Parameters).
In general, having the frame rate and dimensions being exposed via parameters instead of needing a separate config file (as currently) might already be sufficient to catch 99% of the use cases from previous cv_camera pkg users. Question would be whether the parameter naming is consistent across different camera brands (I am new to this, so probably breaks here already). However, if possible that would be a great deal for anybody migrating as it would mean to be able to continue using the rest of an existing infrastructure by just having to switch the nodes that handle the cameras.
Anyways, was just an idea I got during migration.

There is lots of leeway on what GenICam features cameras support, with most of them not being required.
However if cameras do implement those features they should follow the SFNC (StandardFeatureNamingConvention).
For the frame rate there is the "recommended" AcquisitionFrameRate feature which probably most cameras implement, so this could be added as a dynamic reconfigure parameter.
The width/height you can usually not choose freely, so this might be a bit harder to map automatically...


Revoking Access to Client?

I'm trying to provide clients the ability to revoke the authorization token for the application so that they would have to go through the login process again. Is there a way to do so with this library? I can't find one.
For example, with Facebook we'd have to send an HTTP DELETE request to /PROFILE_ID/permissions.

That's entirely up to the provider you are authorizing against. This library only provides an abstraction of the standard OAuth2 spec, which does not include any standard for revoking tokens outside of normal expiration.

Bummer. Thanks for the response. Would be a nice addition to simple have a 'revoke()' method in the AbstractProvider so other providers can implement, though.

You could also use the getAuthenticatedRequest.
$request = $provider->getAuthenticatedRequest(
    'DELETE',
    '/PROFILE_ID/permissions',
    $accessToken
);

Since it's non-standard, I don't think a revoke() method on the AbstractProvider is necessary. However, you might propose that the league/oauth2-facebook provider add a revoke() method that does exactly what @stevenmaguire suggests. :-)


New android permission model

With all of these released [Android Marshmallow<EMAIL_ADDRESS>all official cordova plugins updated], we need to make sure we're up to date with the new permission model for android 6.0.
I'm not a plugin developer, nor am I sure that we even need to do any change, but better safe than sorry, so:
Does the following code require us to update our PushPlugin.java?
<uses-permission android:name="android.permission.ACCESS_NETWORK_STATE" />
<uses-permission android:name="android.permission.WAKE_LOCK" />
<uses-permission android:name="android.permission.VIBRATE"/>
<uses-permission android:name="com.google.android.c2dm.permission.RECEIVE" />
<permission android:name="$PACKAGE_NAME.permission.C2D_MESSAGE" android:protectionLevel="signature" />
<uses-permission android:name="$PACKAGE_NAME.permission.C2D_MESSAGE" />

Android permission documentation
<EMAIL_ADDRESS>cordova-android plugin documentation updated

@fredgalvao I believe that all of those permissions are NORMAL and do not require that we prompt the user. I have installed an app with push onto an Android M device and it ran fine however, I should double check it was targeting sdk 23.

Tested with Cordova Android 5.0.0 and there are no permission issues.


docs: add mcpeblocker as a contributor for docs

Codecov Report

Merging #256 (fdd8c17) into main (619ce94) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #256   +/-   ##
=======================================
  Coverage   37.24%   37.24%
=======================================
  Files          17       17
  Lines        4331     4331
  Branches      157      157
=======================================
  Hits         1613     1613
  Misses       2716     2716
  Partials        2        2

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Should the README mention system dependencies?

I think that in order to use this repository a user's system must have several things already installed. One example is a C compiler to compile treesitter parsers.
What level of documentation would be appropriate for the README regarding these system dependencies? On one end of the spectrum we could provide instruction for how to install the system dependencies on a bunch of different OSes. On the other end of the spectrum, we could just link to the docker file and say 'make sure your system has the things this dockerfile installs'. As with most things, the best answer is probably somewhere in the middle.

i think it's a gd idea to document system dependencies. We can start from ubuntu, arch, macos, windows. Then accepts PRs from the community to add other distros.

Agree! I had quite some issues getting "kickstart" to work on windows.
It might be helpful to at least make it clear upfront (tl;dr for the impatient) that certain things below are required, like:

C compiler tools are required, for eg, cmake, msvc build tools (or make, mingw)
other things that are somewhat required in the child package dependencies like ripgrep, fd
esp pwsh... this last one took me by surprise as I thought powershell was OK when in fact it wasn't, something I found out later through the following links:

https://github.com/williamboman/mason.nvim/pull/706
https://github.com/williamboman/mason.nvim/issues/764

Hi there,
is there a minimum version of neovim required? I have a Debian Buster version 0.3.4. Could not find any reference to what might be required. But I guess I have to install it and just try that out...
Thanks for the video, it was quite the teaser to explore some new packages.
Michael

Latest stable tagged Neovim at a minimum is the only guarantee, says so in the second paragraph in the readme: "Kickstart.nvim targets only the latest 'stable' and latest 'nightly' of Neovim. If you are experiencing issues, please make sure you have the latest versions."

Oh god, I am sorry, don't know how I missed that. Thanks for the pointer, I'll have to do some upgrading then, to try it out…

I started making a list of the System dependencies for Windows, (without listing every package, per TJ's guidance) and it basically boils down to a C Compiler and golang (if gopls is specified). I'm not sure how helpful having a list with these two items somewhere in the README would be. So, I'm going to close this issue.


[NEW ALGORITHM]  ADD Sweep Line Algorithm In Mathematics

Description:
The sweep line algorithm is a general computational geometry technique used to solve various problems involving points, line segments, or other geometric objects in 2D space. The idea is to conceptually "sweep" a line across the plane, from left to right (or top to bottom), processing events as the line intersects with objects.
Checklist:

Contributor in GSSoC-ext
Want to work on it

Additional Information:

you can assign me


Approval request is coming in multiple cards.

Saw approval request in 3 cards back to back.
https://github.com/user-attachments/assets/ca306bee-dba5-44a0-92be-3cd48147ef0c

IK is now discontinued. I think this issue is no longer relevant, but in general we should make a migration plan for users using the IK server.


Migration of python-kopf-s2i and python-kopf-s2i-example repositories

Type of repository request
migration of existing repository
Requested name for the repository
https://quay.io/repository/jkupferer/python-kopf-s2i -> redhat-cop/python-kopf-s2i
https://quay.io/repository/jkupferer/python-kopf-s2i-example -> redhat-cop/python-kopf-s2i-example
Repository Description
The python-kopf-s2i repository creates a source-to-image builder for the Python kopf operator framework.
https://kopf.readthedocs.io/en/latest/
The python-kopf-s2i-example repository is an example of how to use the source-to-image.
The s2i scripts have been tested and are compatible with odo, which should allow for a variety of IDE integrations for python operator development with kopf.
Admin Access
The babylon team.
Write Access
none
Read Access
none
Additional details
These repositories also will need quay.io integration. This can be requested in a separate issue...

Review request:
cc: @redhat-cop/container-cop-core

Hey @jkupferer I'm not seeing any source code included in this request. Either way, it looks to me like this would fit nicely in the [container-quickstarts] repo. We keep several other custom s2i images there.  That repo also already contains a secret config for pushing images to quay, so we then just need to create the image repositories.
If you want to go ahead and work on the PR for that repo, I will get the image repository.
I'm also a bit confused about the difference between python-kopf-s2i and python-kopf-s2i-example. Can you elaborate?

I can switch this to a PR to container-quickstarts, just need to figure out how to add that there.
python-kopf-s2i is the base s2i builder image while python-kopf-s2i-example is an example repository showing how to use python-kopf-s2i.

The PR has been merged to container-quickstarts, but it seems either the python-kopf-s2i repository does not exist or permissions are not configured for the robot account:
https://github.com/redhat-cop/containers-quickstarts/runs/691145587?check_suite_focus=true

All set on this! :D


Ensure seeds capacity for shoots is not exceeded

How to categorize this PR?
/area robustness
/kind enhancement
/priority normal
What this PR does / why we need it:
Ensures seeds capacity for shoots is not exceeded, as described in GEP-13 (see #3017).

Introduce capacity and allocatable resource lists in the seed's status, similarly to the Node status.
Initialise these fields in gardenlet seed controller based on the new resources.capacity and resources.reserved fields in the GardenletConfiguration.
Validate resources.capacity and resources.reserved.
During scheduling, filter out candidates that don't have available capacity for shoots.

Which issue(s) this PR fixes:
Fixes #2938
Special notes for your reviewer:
Resource requests by shoots and more advanced scheduling logic are not yet introduced in this PR. They will be introduced in separate PRs.
Release note:
The scheduler ensures that only candidate seeds with available capacity for shoots are considered during scheduling.

/cc @rfranzke @timebertt @mvladev @danielfoehrKn

/retest

/test

@rfranzke I addressed your comments, could you please take another look?

/cc @rfranzke I had to rebase to resolve conflicts.

@timebertt Ok, I will do it. I assume the ShootedSeed flags should be added to docs/usage/shooted_seed.md, however I couldn't find any appropriate document for fields in the Seed status? Same for the gardenlet configuration, except example/20-componentconfig-gardenlet.yaml, where an example for resources.capacity.shoots is already added.

I assume the ShootedSeed flags should be added to docs/usage/shooted_seed.md

Yes, that would be the right place.

however I couldn't find any appropriate document for fields in the Seed status?

Hmm, yes. Probably a good fit for the capacity topic is docs/concepts/scheduler.md.

@timebertt Done, see #3158


fix(aliaslookup): do the right thing with multiple inputs

I think there was some copy pasta here from another tool, this wasn't doing the right thing when invoked with a list of CVEs.

At least I managed to fix the submodule mess... 🤬


chore: update versions to angular rc3

There is no @angular/router-deprecated version 2.0.0-rc.3

@neilhem You are correct, there's also no references to it throughout the CLI, I removed it as a dependency altogether.

Eh I thought I had put forms in already. Can you add in<EMAIL_ADDRESS>


License.

All the produced code will be released under the MIT license, and will be distributed as an external
package in the distribution of Roassal.

https://github.com/ObjectProfile/HierarchicalVisualizations/blob/main/LICENSE


Change DOI link to https

It's recommended to use https for DOI links https://support.datacite.org/docs/datacite-doi-display-guidelines

👍 LGTM

Hi @tmorrell, many thanks for this PR. Please could you update the test fixtures as well so the tests still pass?

@hainesr I think I got them all. Could you approve the test workflow to check?

Coverage remained the same at 93.882% when pulling 459d8a77a16dc28b6a7e23c231187da04b37dc1c on tmorrell:patch-1 into 0a62411fea756a1bf3c6e759a899053b47a73b47 on citation-file-format:main.

Thanks @tmorrell!

Thank you!


fix(Accordion): start implementing css adjustments and native details

Fixes #2100 🥳
Deprecates #2176, #2190
Built on top of <u-details> for better accessibility on mobile
Removes level from AccordionHeader as this is not supported by native <details>
Removes onHeaderClick from AccordionHeader as this is identical to adding a onClick handler
JS-based animation can be removed and replaced by CSS when calc-size(auto) is fully supported 🚀
Fixes shrinking chevron on mobile/zoom
Fixes text-align in AccordionHeader on mobile/zoom
Follows: #2295
Also works in dir="rtl"

Question: It is now implemented so search-in-page only works when using defaultOpen, as a controlled open should not be affected by user interaction. Just checking - does this make sense to you guys as well? :)

Closes #2364

Design for new hover-colors: https://www.figma.com/proto/vpM9dqqQPHqU6ogfKp5tlr/branch/BoDyQWgF45QJjIAbJJM9sS/Designsystemet---Core-UI-Kit?page-id=13134%3A99159&node-id=37773-4265&node-type=FRAME&viewport=2401%2C777%2C0.19&t=bz3JYzZYqR9MkLp9-1&scaling=min-zoom&content-scaling=fixed&starting-point-node-id=37773%3A4265


Do you have interest in adding RIFE support (it would double the frame rate) ?

What
Rife is a tool that does a frame interpolation (create frames between two frames) to increase the frame rate.
By adding that It would convert the 24fps to 48fps or so.
I'm asking here because 24fps seems to be enough, if not, we could just add RIFE as an example about how to increase the FPS even more.
Why
To double the frame rate.
reference

I think the bottleneck of the current model is not frame rate, but rather stability. We hope to first address that and then look at these enhancements.
Anyway, thank you for the thoughtful suggestion.


Integrate trivial-system-loader and also list ocicl systems

Since adding external dependencies is a bad idea, maybe just integrating trivial-system-loader into micros is fine.  If there is a better way, please let me know.
I hope this is okay, I tried to incorporate the license but don't know if I did that correctly.
Furthermore, listing locally available systems is now also not dependent on quicklisp anymore and respects ocicl.
This would also close #11.

Thank you!

find-ocicl-systems ignores the user's global systems.  I think I should just add this feature to ocicl-runtime.  I'll do that and submit a lem patch once complete.


Update best-practices.md

Why make this change?

Improving the doc

What is this change?

Doc update

/azp run

/azp run

/azp run


added support for quart framework

Quart is async Flask-like API compatible framework.
Logging setup still needs some minor polishing, but it is "ready" to be merged.
Request logging works, other logging works as well.

@Darkless012 thanks for PR. It looks great. 👍
Just one more thing, could u pls add an example script into example folder and update readme accordingly.
After that i will do the build and release it to PyPi

Thanks for the remarks, I didn't rebased on current master though, will you be able to resolve README.md, or should I fix that?

i will take care of it and do a release

released : https://pypi.org/project/json-logging/0.0.9/
Thanks again for great contribution!


iOS Share function doesn't share image when using Whatsapp

This is code excerpt I use for sharing:
window.plugins.socialsharing.share("Kare'de geç beni geçebiliyorsan..", null, imageLink, Kare.shareLink);
When console.log imageLink:
/private/var/mobile/Containers/Data/Application/55ACD40F-D8E6-4863-88B4-4948B3879E54/tmp/myScreenShot.jpg
Email is working, it sends image as attachment. But WhatsApp only sends text with link, no image.
iOS 8.1.3

I have the same challenge. It works fine on Android Whatsapp and other IOS apps (Twitter, email, faebook, etc.)

i have the same challenge also , it's not working with outlook in IOS 9.2

i am also have the same problem.

Does any of you have a Mac and has tried opening Xcode and debugging the code? It would be of tremendous help if you did.

Hi Eddy,
I can try to help, can you give me specific pointer / piece of code I
should drill into ?
Rgds,
Albert
On Thu, Jan 21, 2016 at 4:00 AM, Eddy Verbruggen<EMAIL_ADDRESS>wrote:

Does any of you have a Mac and has tried opening Xcode and debugging the
code? It would be of tremendous help if you did.
—
Reply to this email directly or view it on GitHub
https://github.com/EddyVerbruggen/SocialSharing-PhoneGap-Plugin/issues/494#issuecomment-173343305
.

+1

I am facing this very problem right now, was there any resolution please?

I'm using the shareWithOptions method and I can't make it work either.
Only when selecting whatsapp on ios is when the image won't be shared. It used to work properly.

I had same issue. following are the logs. Hope this helps
2018-02-20 13:36:15.387886+0530 DemandFarm[1951:1479241] THREAD WARNING: ['Screenshot'] took '85.100342' ms. Plugin should use a background thread.
2018-02-20 13:36:15.543675+0530 DemandFarm[1951:1479241] [MC] Filtering mail sheet accounts for bundle ID: com.demandfarm.kam, source account management: 1
2018-02-20 13:36:15.545314+0530 DemandFarm[1951:1479241] [MC] Filtering mail sheet accounts for bundle ID: com.demandfarm.kam, source account management: 2
2018-02-20 13:36:27.931999+0530 DemandFarm[1951:1479241] [core] SLComposeViewController initWithExtension: <NSExtension: 0x100cf3cc0> {id = net.whatsapp.WhatsApp.ShareExtension} requestedServiceType: (null)
2018-02-20 13:36:27.933501+0530 DemandFarm[1951:1479241] [core] SLComposeViewController addExtensionItem: <NSExtensionItem: 0x1018e8a20> - userInfo: {
    NSExtensionItemAttachmentsKey =     (
        "<NSItemProvider: 0x1018e2ee0> {types = (\n    \"public.image\"\n)}",
        "<NSItemProvider: 0x106b15950> {types = (\n    \"public.plain-text\"\n)}"
    );
    NSExtensionItemAttributedContentTextKey = <7b5c7274 66315c61 6e73695c 616e7369 63706731 3235320a 7b5c666f 6e747462 6c5c6630 5c667377 6973735c 66636861 72736574 30204865 6c766574 6963613b 7d0a7b5c 636f6c6f 7274626c 3b5c7265 64323535 5c677265 656e3235 355c626c 75653235 353b7d0a 7b5c2a5c 65787061 6e646564 636f6c6f 7274626c 3b3b7d0a 5c706172 645c7478 3536305c 74783131 32305c74 78313638 305c7478 32323430 5c747832 3830305c 74783333 36305c74 78333932 305c7478 34343830 5c747835 3034305c 74783536 30305c74 78363136 305c7478 36373230 5c706172 6469726e 61747572 616c5c70 61727469 67687465 6e666163 746f7230 0a0a5c66 305c6673 3234205c 63663020 53686172 65642076 69612044 656d616e 64466172 6d204d6f 62696c65 20417070 2e20446f 776e6
2018-02-20 13:36:28.502465+0530 DemandFarm[1951:1479241] [core] viewWillAppear
2018-02-20 13:36:28.502614+0530 DemandFarm[1951:1479241] [core] SLComposeViewController including 1 explicit NSExtensionItems
2018-02-20 13:36:28.502701+0530 DemandFarm[1951:1479241] [core] SLComposeViewController about to instantiate remote view controller with array of 1 NSExtensionItems
2018-02-20 13:36:28.506016+0530 DemandFarm[1951:1479241] [core] viewDidAppear
2018-02-20 13:36:29.188169+0530 DemandFarm[1951:1479241] [core] SLComposeViewController finished instantiate remote view controller <SLRemoteComposeViewController: 0x10226aa00> error (null) extension request identifier B1FEE54C-203C-4D68-812E-DB3B6FBEC2FD
2018-02-20 13:36:29.188249+0530 DemandFarm[1951:1479241] [core] SLComposeViewController received remoteViewController of class SLRemoteComposeViewController, and setting its delegate to self
2018-02-20 13:36:29.188345+0530 DemandFarm[1951:1479241] [core] HOST: Successfully loaded remote SLComposeVC
2018-02-20 13:36:29.188419+0530 DemandFarm[1951:1479241] [core] [remoteViewController setupRemoteViewController]
2018-02-20 13:36:29.208426+0530 DemandFarm[1951:1479241] [core] Finished setting up remote view controller
2018-02-20 13:36:46.128971+0530 DemandFarm[1951:1479241] [core] SLComposeViewController completeWithResult: 1
2018-02-20 13:36:46.140029+0530 DemandFarm[1951:1479241] [core] SLComposeViewController skipping explicit dismiss because isBeingDismissed is already 1
2018-02-20 13:36:46.158361+0530 DemandFarm[1951:1479241] [core] SLComposeViewController dealloc <SLComposeViewController: 0x106b4b110>
2018-02-20 13:36:46.188314+0530 DemandFarm[1951:1479241] [core] SLRemoteComposeViewController: (this may be harmless) viewServiceDidTerminateWithError: Error Domain=_UIViewServiceErrorDomain Code=1 "(null)" UserInfo={Terminated=disconnect method}

Still facing the same issue right now on IOS 12, anyone found a solution to this?

I have the same issue. +1

For me I can click on Facebook or Twitter and the dialog opens then it crashes:
2019-06-28 11:16:56.913800-0400 App Name[7041:2527108] [core] SLRemoteComposeViewController: (this may be harmless) viewServiceDidTerminateWithError: Error Domain=_UIViewServiceInterfaceErrorDomain Code=3 "(null)" UserInfo={Message=Service Connection Interrupted}
2019-06-28 11:16:56.913843-0400 App Name[7041:2527108] viewServiceDidTerminateWithError:: Error Domain=_UIViewServiceInterfaceErrorDomain Code=3 "(null)" UserInfo={Message=Service Connection Interrupted}

Instagram/Email/Sms are working, I am also using shareWithOptions(). I can share to FB/Twitter with another picture but not any that I have taken with the camera plugin. I've tried base64, I've tried temp files, I've tried cache file, I've tried JPEG vs Png, I've cranked the quality all the way down and... Nothing. It just crashes. No clue what's going on because if I same the same picture to my camera roll it can share to FB/Twitter just fine.

I have the same problem, doing more tests, I could share images in WhatsApp sharing only the image and discarding test and url.
await this.socialSharing.shareWithOptions({ files });


[WIP] Fixed #24629 -- Allowed Func to be used as Transform

This is an experiment to see how a backwards compatible translation layer could help with the disconnect between Func and Transform.
The basic idea is that Funcs can return themselves wrapped in a Transform that proxies methods back to the Func. I'm not in love with this idea, but I think it might have a place if we decide we'd like to not break the Transform API so soon after it has been released.

Do we need all four of these to use a func as transform:

set lookup name
set transform bool
call transformer()
call register_lookup()

It seems the first two are redundant, likewise for the last two. Maybe we could do with just lookup name and register_lookup()? Though this reguires that magic happens inside register_lookup(), where we need to do automatically what transformer() does.

The lookup name is explicit, but could default to self.function.lower().
The transform bool is really only there to get around RegisterLookupMixin.get_transform which requires issubclass(x, Transform) which I've changed to hasattr in this patch. That's not really the way I want to go, but it was a quick way of verifying that the FuncTransform works.
The problem with register_lookup is that it doesn't know if it should be registering as a lookup or a transform. It inspects the type to figure out where to put it. So absent making Func inherit from Transform in some way, we're going to need another mechanism for determining whether or not a registered lookup is a Transform or Lookup.
I think we need to solve that problem no matter which way we go. If we could register_transform or register_lookup, and store those in two different dicts, I think everything else becomes much simpler.

ok cool thanks buddy I appreciate it all

Yes, register_transform seems like a good idea. I don't like deprecating the current behaviour too much but we don't have much choice.
Other than that, I think we should aim for "if an expression has lookup_name defined, then it is usable as transform".
Not this ticket's issue, but we likely need to solve an similar expression vs lookup issue for lookups. For lookups the use case is for example "order by StartsWith(field1, somevalue)".  The use cases are admittedly more limited, but likely still worth fixing.

OrderBy(StartsWith(field, value)) isn't valid is it? I can see the point you're trying to make though, though I'm struggling to find an example where a lookup/comparison would be useful outside a WHERE or WHERE-like situation.

I've added an alternative implementation of this (which I think has bugs due to missing relabeled_clone - what does this even do??, possibly also needs better output_field support) at https://github.com/django/django/pull/4748
The API is slightly different, instead of introducing a new wrapping class we explicitly create a new Transform subclass using MyFunc.as_transform. It also supports setting other Transform attributes like bilateral in that method, and following a functools.partial-like implementation idea to allow "freezing" other params in. See documented example in my patch.
I think this API is a bit more explicit and keeps the separation between Transform and Expression. I don't think making them the same object should be a goal - Func is much more expressive than Transform can be. We may be straying a bit into TMTOWDI problems here though.

Update: Yeah, you have a classmethod as well called transformer. as_transform() is similarly named to as_manager() which is why I used that. You're also not dynamically creating the class which could be better, not sure. I don't like the leaky approach of transform specific stuff living on the function class though.

@mjtamlyn I'm not attached to the implementation I have here. It was something rough to check the concept. I think I'd prefer to make Transforms and Lookups some kind of Expression subclass, and then make the Lookup specific APIs (process_xhs, self.xhs) internally use the Expression APIs (resolve_expression, get/set_source_expressions) or vis-versa, for some deprecation period.
If we make Transforms a subclass of Func (or some other Expression), then any changes we make to the Expression API will be automatically reflected in the Transforms and Lookups. Also, conceptually, they literally are parts of an expression. There's no reason they can't have the same ancestor other than backward compatibility issues. I think we should solve those problems rather than creating an interop layer between them.
Also, relabeled_clone updates the alias of the table when subqueries or multiple joins to the same table are involved. If USERS gets joined to twice, you might name one of the joins T1 and the other T2. Expressions referencing those joins SUBSTR(users.username..) now need to update to SUBSTR(T1.username...). That's the job of relabeled.


Allow advanced users to bring their own configmaps

So that we don't have to support every single config option available, add the option for the advanced end user to skip the chart managing configmaps and allow the user to provide their own.

Closing in favor of json config.


#27 added tests for the Protocol class

Automated testing for the Protocol class along with a few updates to the Protocol class itself

A class variable that starts with a single _ in python is a notation for saying "This variable should be private to this class and not accessed directly, bypassing and methods to modify it such as getters and setters". However, it is still possible for a user to adjust the variable directly.
Later versions of python introduced the double __ notation for saying the same thing, but actually preventing (or making it very hard at least) the variable from being edited directly so that users can only modify it through defined methods.


ci: avoid installing git on wolfi

It's already installed.
And also, we should move all our dependency installation out of here, and into previous steps, this is generally a bad practice in the first place.

Not sure how similar the wolfi module is vs just using
docker run -it --rm cgr.dev/chainguard/wolfi-base sh

/ # apk add go=1.22.3
fetch https://packages.wolfi.dev/os/aarch64/APKINDEX.tar.gz
ERROR: unable to select packages:
  go-1.22-1.22.4-r0:
    breaks: world[go=1.22.3]

if instead I do this and get go 1.22.4 and don't require 1.22.3, it completes.
/ # apk add go git
fetch https://packages.wolfi.dev/os/aarch64/APKINDEX.tar.gz
(1/36) Installing libbrotlicommon1 (1.1.0-r2)
(2/36) Installing libbrotlidec1 (1.1.0-r2)
(3/36) Installing libnghttp2-14 (1.62.1-r0)
(4/36) Installing libunistring (1.2-r2)
(5/36) Installing libidn2 (2.3.7-r2)
(6/36) Installing libpsl (0.21.5-r2)
(7/36) Installing libcurl-openssl4 (8.8.0-r0)
(8/36) Installing libexpat1 (2.6.2-r1)
(9/36) Installing libpcre2-8-0 (10.43-r1)
(10/36) Installing git (2.45.2-r0)
(11/36) Installing ncurses-terminfo-base (6.4_p20231125-r3)
(12/36) Installing ncurses (6.4_p20231125-r3)
(13/36) Installing bash (5.2.21-r4)
(14/36) Installing libgcc (13.2.0-r7)
(15/36) Installing libstdc++ (13.2.0-r7)
(16/36) Installing binutils-gold (2.42-r2)
(17/36) Installing binutils (2.42-r2)
(18/36) Installing libstdc++-dev (13.2.0-r7)
(19/36) Installing posix-cc-wrappers (1-r3)
(20/36) Installing libatomic (13.2.0-r7)
(21/36) Installing gmp (6.3.0-r2)
(22/36) Installing libgo (13.2.0-r7)
(23/36) Installing libgomp (13.2.0-r7)
(24/36) Installing isl (0.26-r3)
(25/36) Installing mpfr (4.2.1-r3)
(26/36) Installing mpc (1.3.1-r3)
(27/36) Installing gcc (13.2.0-r7)
(28/36) Installing libxcrypt-dev (4.4.36-r7)
(29/36) Installing linux-headers (6.6.29-r1)
(30/36) Installing nss-db (2.39-r6)
(31/36) Installing nss-hesiod (2.39-r6)
(32/36) Installing glibc-dev (2.39-r6)
(33/36) Installing make (4.4.1-r3)
(34/36) Installing pkgconf (2.2.0-r2)
(35/36) Installing build-base (1-r8)
(36/36) Installing go-1.22 (1.22.4-r0)
OK: 673 MiB in 50 packages

A couple questions about wolfi:

Does this just happen from time to time? Do dependencies just go missing? If so, that's a bit annoying.
I don't quite understand why installing git would cause conflicts for go.

A couple questions about wolfi:

Does this just happen from time to time? Do dependencies just go missing? If so, that's a bit annoying - that has the potential to just spontaneously break our CI even with no changes, I'd like to not do that if at all possible.
I don't quite understand why installing git would cause conflicts for go.

cc @amouat, we're trying to use Wolfi more in our CI, but running into some issues which this PR tries to fix.

I think that we should either:
* Move `test-publish` to the publish workflows

* Move publishing into the `sdk-*` workflows

The reason to have these separate at the moment is so that the test-publish can actually occur on PR runs - atm, the first sign that publishing could have gone wrong is during release (either on main, or from tags).
Some initial refactoring of the workflow/job structure is in https://github.com/dagger/dagger/pull/7483, I think this makes a lot of sense in that context.
I think ideally we should move publish into the SDK workflows. Then we use a conditional - if it was triggered by a PR, we do the dry-run, otherwise, we do the actual publish.

@shykes let me take a look and get back to you

Aha thanks @amouat :tada:
What's weird to me is that this issue only triggers when we try and call apk manually - when using an apko config file (created by https://github.com/vito/daggerverse/blob/main/apko/apko.bass, cc @vito), this seems to "work".


Update use-macros-in-a-custom-pallet.md

StorageMap code change
I got an error with T::BlockNumber that associated type BlockNumber not found for T
But with BlockNumberFor it compiled and run as expected.

seems these changes already made their way to master


Generate .cloupe file with 2µm bins from Visium HD data

Hello @LucaMarconato, thanks a lot for the quick reply!
I want to obtain a .cloupe file (it's produced as one of the outputs of the spaceranger count command)
at 2um resolution of spots. The file that spaceranger count produces by default has 8um spots. There
is a flag of the spaceranger count, called `--custom-bin-size` but it takes values from 10 to 100. I have
looked through a lot of 10x documentation but can't seem to find a way to do that. I think this would be
a common issue among the community too (this I say because it is a direct next step of
nuclei-segmentation task (https://www.10xgenomics.com/analysis-guides/segmentation-visium-hd),
which is what I am doing).
Hence, if anyone has any ideas or has been able to figure it out, could you please give some insights?

Thanks a lot!

Originally posted by @shashkat in https://github.com/scverse/spatialdata/issues/632#issuecomment-2221222053

Thanks a lot for creating this issue @LucaMarconato!
Currently, I am using napari-spatialdata, and I think it does a pretty good job. I just have to subset the sdata object to a smaller bounding box to have a smooth visualizing experience, but I am able to view the grouped 2um barcodes superimposed on the tissue.
Still, if anyone knows of a better way I would love to hear it. I can also give my inputs if anyone comes across this specific problem of viewing grouped 2um spots obtained using nuclei segmentation, superimposed on the H&E image using napari.

I just have to subset the sdata object to a smaller bounding box to have a smooth visualizing experience

Actually we implemented a strategy to have fast performance when viewing the full 2µm data, but we haven't "advertised it"  yet; you can find the details here: https://github.com/scverse/spatialdata/pull/578.
Happy to hear feedback on it 😊

Hey @LucaMarconato, thanks a lot for that suggestion! I tried it and it looks awesome.
I think at this point, the spot-pixels are colored in a binary way, according to presence or absence of obs/var. I guess one of my suggestions would be to allow for color gradients showing magnitude of value in obs/var.

Thanks for your feedback 😊 Please note that for various channels the output looks binary because most of the counts are 0 and 1. But I will double check the API ergonomics for using colormap for the rasterized image when the bin size is > 2µm.

I believe we can mark this issue as closed as creating a loupe browser converter is out of scope and the bin rasterization discussion has a solution.


Adds --yes option to auto-apply changes.

This addresses issue #38 and makes it possible to run dothub as a cron job or as part of a CI pipeline without having to use yes.
For the record: unit tests on a clean clone of the upstream repo failed for me, that is why I haven't been able to add more tests to validate my changes.

You should not have any the issue you had with git anymore after #55


Client should reconnect on failure

With backoff, obviously.

Decided that the best approach is to exit, systemd can restart on failure.


Error when building imglab

Hi,
I m on macos El Capitain,
when trying to build imglab i get this error:
[ 92%] Built target dlib
[ 93%] Building CXX object CMakeFiles/imglab.dir/src/main.cpp.o
In file included from /Users/thaiat/Documents/Phonegap/yoobic-poc/dlib/tools/imglab/src/main.cpp:2:
In file included from /Users/thaiat/Documents/Phonegap/yoobic-poc/dlib/dlib/../dlib/data_io.h:11:
In file included from /Users/thaiat/Documents/Phonegap/yoobic-poc/dlib/dlib/../dlib/data_io/load_image_dataset.h:9:
In file included from /Users/thaiat/Documents/Phonegap/yoobic-poc/dlib/dlib/../dlib/data_io/../image_io.h:14:
/Users/thaiat/Documents/Phonegap/yoobic-poc/dlib/dlib/../dlib/image_loader/load_image.h:14:10: fatal error:
      'gif_lib.h' file not found
#include <gif_lib.h>
         ^
1 error generated.
make[2]: *** [CMakeFiles/imglab.dir/src/main.cpp.o] Error 1
make[1]: *** [CMakeFiles/imglab.dir/all] Error 2
make: *** [all] Error 2

any idea what i m doing wrong ?
PS: i have giflib installed

I don't know what's happening.  CMake will only attempt to use gif if it
finds libgif, but apparently there is something wrong with your libgif
install.  Maybe it's missing gif_lib.h.

i removed giflib and redid cmake, everything is now fine
thks!!!


add eslint

I was doing slackapi/node-slack-sdk#805 and had this one hanging around in my git too.
again if you'd rather keep tslint LMK and ill close this PR.
otherwise, if you want rules shuffling around/changing just say and ill make the changes.
disallowing use of functions/variables before they are defined was the big one here as you often seem to define them at the end of a file, so i had to move some big blocks and make the test files a little uglier especially. i'd recommend you don't define things after using them and instead put them in a util file if anything.
also the required jsdoc in tests, i might drop and discard the docs i added there. guess nobody really needs jsdoc on little test harness functions

thanks for the PR! we recognize that moving to eslint is something we need to do, since tslint isn't going to be maintained any longer. this is going to take a little bit of time (as you can see some conflicts need to be resolved). right now, we're not ready to take this on ourselves, but we're happy to give feedback along the way if someone can help update this PR.

Hey @43081j,
Thanks for sending this in. We had an intern recently take a stab at the move to eslint. It just landed in https://github.com/slackapi/bolt-js/pull/529. Going to close this off, but thanks again for sending this in. It inspired us to make the move!


fix: :::warn messages

🚀 Deployed on https://668db7fc2d4ced03503eeef3--cozy-fox-0defec.netlify.app


Describe development-environment setup

Develop Local on Conda (e.g. useful with M1-Macbooks)
Develop Local/Remote Docker Devenvironment (e.g. useful when running on a remote server)

Docker Development Environment merged in #6


[Bug Report] pagination input box part of the browser, there is no jump problem

Element UI version
1.4.4
OS/Browsers version
win7/360安全浏览器8.1
Vue version
2.4.4
Reproduction Link
https://jsfiddle.net/c25mhsyc/12/
Steps to reproduce
初次生成页面时，点击第三或者第二页，然后选中输入框中页码替换页码为1（输入页码与第一次进入页面的初始页码相同）即可复现不跳转。
when the first time page appeared, click a another paganation button jump to the other page,then dubble click the  number input in the end of paganation or any other ways to select the page number in the input, enter the same  page number as the first time page created to replace the number in the input(not to delete the old number,use replace function to enter the page number),the scene will repeat
What is Expected?
paganation应该跳转至输入的页码页
paganation shuld jump to the number which i entered
What is actually happening?
paganation页码没有做任何跳转行为
paganation do nothing about the page jump

Cannot reproduce this.

Sorry for that mistake！ this is the reproduce gif

@Leopoldthecoder


[EGGO-48] Run eggo on a Cloudera Director provisioned cluster.

With these changes I was able to toast a dataset on a Cloudera Director cluster. Some of the changes conflict with running using the Spark EC2 scripts, so we should figure out if they can easily be made to coexist, or if we remove support for them.

ADAM is no longer compiled on the cluster. This is because there were Java 6/7 issues (both are installed and it was picking up the wrong one for part of the build). I think this is OK as long as we make it easier to supply an arbitrary ADAM tarball.
The memory used by ADAM had to be reduced to fit in the 4GB limit on the cluster. I chose 3GB, but we could go a bit higher.
I hit SPARK-4705 so had to turn off event logging.
Ideally we would not have to install anything on the worker nodes, but there's still a need for Python's ordereddict (since the cluster uses Python 2.6). We could remove that need by going back to the shell script for the streaming job.
Fabric wouldn't install on the cluster, but that's OK as it doesn't need to be installed (only needed on the client).
Installing eggo with Luigi as a required dependency causes us to hit http://stackoverflow.com/questions/28443041/python-daemon-2-0-5-wont-install-with-pip. If the dependency is removed then it works. Not sure the best way to handle this.

ADAM is no longer compiled on the cluster. This is because there were Java 6/7 issues (both are installed and it was picking up the wrong one for part of the build). I think this is OK as long as we make it easier to supply an arbitrary ADAM tarball.

Maybe support both.

The memory used by ADAM had to be reduced to fit in the 4GB limit on the cluster. I chose 3GB, but we could go a bit higher.

Is this because the instance types that are available with director are limited?

I hit SPARK-4705 so had to turn off event logging.

Can we use client mode instead?  Is this a bad idea?

Ideally we would not have to install anything on the worker nodes, but there's still a need for Python's ordereddict (since the cluster uses Python 2.6). We could remove that need by going back to the shell script for the streaming job.

Where does ordereddict come in again?  With the refactoring, in theory it comes in with ConfigParser, but you can specify that it should use a regular dict instead.  Anyway, there is a Python 2.6-compatible ordereddict impl that we could install as well.

Installing eggo with Luigi as a required dependency causes us to hit http://stackoverflow.com/questions/28443041/python-daemon-2-0-5-wont-install-with-pip. If the dependency is removed then it works. Not sure the best way to handle this.

If v2.0.3 does install fine with pip, we can install it before we install luigi.  Then, the luigi install would just pick up the python-daemon library without complaints.

Thanks for taking a look @laserson.  A few comments:

The memory limit is a Spark or YARN config limit, not due to the instance type. We can adjust that though.
Client mode would be fine.
ordereddict is used by Luigi for streaming (in config parsing IIRC). I would like to avoid having to install anything extra on worker nodes if possible since it's a source of failures for larger clusters.
I can try your suggestion for python-daemon to see if that helps.

This is an update that incorporates all the recent changes in configuration. The main changes are:

Use RHEL 6.5 (not 6.4) on new-generation EC2 instance types (d2, not m1)
ADAM is built on the cluster now and uses the default memory settings
Use yarn-client to avoid SPARK-4705 and SPARK-1921
Use conditional code for director and spark-ec2 so it runs on both.

I still get the python-daemon problem (I tried running pip install -U pip but it didn't seem to help). I think removing install_requires is OK as a workaround.
I have tested this on director and spark-ec2 and have successfully run the test-genotypes conversion on both. I think this is ready to be committed, so please take a look @laserson.

Looks great!

Manually merged in 2ce23ccbe3a1a61e0da2872262efdd30bd88b26c.  Thanks, @tomwhite!


Use fully-qualified Ansible module names where not used

Use fully-qualified Ansible module names where not used.

[X] Is unit test coverage still above 80%?
[X] Have you updated all applicable documentation?
[X] Have you followed the guidelines in our Contributing document?

@nick-stroud ready for another review. Based on manual testing, I anticipate that all 4 checks will pass.

Group 2 had a transient failure with Filestore API. I am re-running the check.

I meant to include a blueprint, although I'd wager you already have something similar:
# Copyright 2022 Google LLC
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
#      http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.

---
blueprint_name: simple

# Please review https://cloud.google.com/compute/docs/regions-zones
# for availability of A2 machine types
vars:
  project_id:  ## Set project id here
  deployment_name: ansible
  region: us-west4
  zone: us-west4-b

# Documentation for each of the modules used below can be found at
# https://github.com/GoogleCloudPlatform/hpc-toolkit/blob/main/modules/README.md

deployment_groups:
- group: primary
  modules:
  - id: network1
    source: modules/network/pre-existing-vpc
  - id: ansible-setup
    source: modules/scripts/startup-script
    settings:
      runners:
      - type: shell
        source: modules/startup-script/examples/install_ansible.sh
        destination: install_ansible.sh
  - id: simplevm
    source: modules/compute/vm-instance
    use:
    - network1
    - ansible-setup
    settings:
      machine_type: t2d-standard-4

This failure of group 3 appears to be a real timeout problem talking to Ansible Galaxy. I see enough evidence of it that I suggest we pause. I have a short-to-medium term version for this problem that bypasses a lot of these concerns anyhow.

Resolving all conversations and de-scoping PR to only fix the fully-qualified names of Ansible modules.

I have triggered all 4 checks just in case.


Default option for pull

Can a default option be added to pull only AC solutions from a given contest?

cf pull [ac] [<contest-id>] [<problem-id>]
so you just need to use
cf pull ac <contest-id>

cf pull [ac] [<contest-id>] [<problem-id>]
so you just need to use
cf pull ac <contest-id>

If you want to pull all contests' codes in Codeforces. You can write a simple script (bash, python, etc.) to iterate a number i from 1 to 2000 and execute cf pull ac $i

I don't want to put ac option as default.


3.2.5 rocksdb clean install - corrupted db

3.2.5 rocksdb on debian8 clean install
<IP_ADDRESS>:8529@_system> db._collections()
[
[ArangoCollection 603288, "_appbundles" (type document, status loaded)],
[ArangoCollection 60, "_apps" (type document, status loaded)],
[ArangoCollection 36, "_aqlfunctions" (type document, status loaded)],
[ArangoCollection 54, "_frontend" (type document, status unloaded)],
[ArangoCollection 2, "_graphs" (type document, status unloaded)],
[ArangoCollection 58, "_jobs" (type document, status loaded)],
[ArangoCollection 21, "_modules" (type document, status loaded)],
[ArangoCollection 56, "_queues" (type document, status loaded)],
[ArangoCollection 23, "_routing" (type document, status corrupted)],
[ArangoCollection 44, "_statistics" (type document, status loaded)],
[ArangoCollection 49, "_statistics15" (type document, status loaded)],
[ArangoCollection 38, "_statisticsRaw" (type document, status loaded)],
[ArangoCollection 6, "_users" (type document, status loaded)]
]
_routing is corrupted.
When I try to delete /var/lib/arangodb3 and restart arango to have everything recreated, same collection _routing is again corrupted.

After dpkg purge and new install it was again corrupted. I tried also older versions with same result. Then I found out that there is some arangod process hanging which was not killed during installation. After killing it, purge and install db is ok without corruption.

We have to fix that apt-get purge (or dpkg purge) does not seem to stop the arangodb3 service.

This has been fixed in 3.2.6, which is available for download now.

Thanks!


[SPARK-30984][SS]Add UI test for Structured Streaming UI

What changes were proposed in this pull request?

Add a UI test for Structured Streaming UI
Fix the unsafe usages of SimpleDateFormat by using a ThreadLocal shared object.
Use start to replace submission to be consistent with the API StreamingQuery.start().

Why are the changes needed?
Structured Streaming UI is missing UI tests.
Does this PR introduce any user-facing change?
No
How was this patch tested?
The new test.

cc @uncleGen @xuanyuanking @cloud-fan

Test build #119077 has started for PR 27732 at commit 1479502.

Merged build finished. Test PASSed.

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

Test build #119077 has finished for PR 27732 at commit 1479502.

This patch fails SparkR unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #119100 has started for PR 27732 at commit 1479502.

Merged build finished. Test PASSed.

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

Test build #119100 has finished for PR 27732 at commit 1479502.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

retest this please

Test build #119103 has started for PR 27732 at commit 1479502.

Merged build finished. Test PASSed.

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

Test build #119103 has finished for PR 27732 at commit 1479502.

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

Merged build finished. Test PASSed.

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

cc @xuanyuanking @gengliangwang @sarutak

thanks, merging to master/3.0!


Possibly misleading Dashboard info

While Forced-Closed the Dashboard continues to state that some of command circuits in the cluster remain open when in fact all circuits are closed.

Thanks for the report.  That does sound like a bug, assuming that you applied the forceClose to all members of the cluster, and not just a single one.
When I look into it, it could be that this is a problem in the Turbine aggregation.

I always interpreted this view in the following, rather important to me, way: In the parenthesis we see what the circuits should be, open or close, depending on the calculated metrics. This has nothing to do with is the forced status they finally have, due to to the forceXXX parameter. It is very important to know the calculated status regardless what is applied in the end. In most cases, the decision to force a status is based on this analysis and it should always be available. Else, if we do not care about the calculated status at all, we do not need to use forceClosed. We just disable the circuit breaker altogether.

@genshapiro / @pparth I think I lost track of this issue.  Is this still a problem?

Closing due to inactivity.  Please re-open if there should be further discussion


which dataset be used when runing "run_train_generator.sh"?

Can you give some instructions for using“train_generator.py”?What kind of data is used? NIR or RGB？

The training dataset is CASIA NIR-VIS 2.0, including both NIR and VIS images. We use paired NIR-VIS images to train our generator.

Refer to run_train_generator.sh, img_root is the path of your dataset, and train_list is the path of your image list. The example of the image list is

NIR/s2_NIR_10166_013.jpg 351 0
VIS/s2_VIS_10166_002.jpg 351 1
NIR/s1_NIR_00009_003.jpg 8 0
VIS/s1_VIS_00009_009.jpg 8 1

From left to right, each column is the image name, the identity label, and the domain label, resepctively.


reimplement viterbi decoding in tensorflow way

The original viterbi decoding in tensorflow.contrib.crf is implemented in numpy way and can only be used in testing. This update reimplement viterbi decoding in tensorflow style, can provide more flexibility during model building.

Can one of the admins verify this patch?

I signed it!

this implementation contain some import issues to be resolved.


Default menu links should check a real permission

https://github.com/simonw/datasette/blob/18a64fbb29271ce607937110bbdb55488c43f4e0/datasette/default_menu_links.py#L4-L6
This should check a named permission so that it can be customized by permission plugins.

Then this can make it available to root: https://github.com/simonw/datasette/blob/18a64fbb29271ce607937110bbdb55488c43f4e0/datasette/default_permissions.py

Permission idea: debug-menu

Document the new permission here: https://docs.datasette.io/en/stable/authentication.html#built-in-permissions

I mainly want this so I can add that debug menu to my Dogsheep.

Documentation: https://docs.datasette.io/en/latest/authentication.html#debug-menu


Fixed the dead band adjustment

Previously, I had made the dead band adjustment to increase the cool or decrease the hot set points the amount to ensure there was 1.5C/2.7F of dead band per the API documentation.
I was seeing issues which were spawning from rounding on the heat/cool points when at that dead band threshold.  I would try to bump up the cool set point by the delta, and get a "invalid cool set point" response from the Google API.
I've changed the code to now increase the cool set point by the dead band + heat point or decrease the heat set point by cool point - dead band to ensure there is at least the dead band of temperature difference.
The app will now properly round to 1-decimal place when setting the request back to Google to prevent larger fractional temperatures.

Hang tight on this PR.  Something is still off when trying to set a temperature that needs to move the dead band.  Need to log a bit more data....


Typos

A few small typo fixes after reading the first chapter.

Updated pull request after reading stub.md and auth.md.

Updated pull request after reading resolver.md.

This project seems to be dead, closing the PR.


Measurements when only Imager Pixel Spacing is present.

I have DX Images from a Canon System where only Imager Pixel Spacing is present.
So without mapping Imager Pixel Spacing to Pixel Spacing (which is what I have done at the moment) I can only measure in pixels.
So the question is: from what i can gather normally Imager Pixel Spacing should be used when present,
if it is not Pixel Spacing should be used, when both are present and are not the same than Imager Pixel Spacing.
Can some clear this up for me ?

There has been some discussion on this issue here:

https://github.com/cornerstonejs/cornerstoneWADOImageLoader/pull/261#issuecomment-499951878

I believe the consensus was:

Use pixelSpacing
Up to applications to optionally use imagerPixelSpacing, may wish to indicate/warn user

Personally, I have worked on viewers that go so far as to fallback to ultrasound regions as a fallback for pixelSpacing settings so that current cornerstone-tools continue to measure in mm. The issue is that it's non-standard, and should be opt-in product owner, and they should probably indicate that they're using a non-standard value.
We have some metadata changes (by the lovely @JamesAPetts) coming soon that centralize and simplify our metadata usage. This will make it easier for us, as an application, to deviate from the standard and make these decisions/fallbacks as we deem appropriate.
I'm speaking off the cuff, but that should be the gist.

Thank you, that helped especially the linked discussion. So in the end the best thing would be to warn the user in those cases if the measurements might not be accurate and have the the possibility to calibrate against known object in the image if available.


Flux dust automation does not work with IntegratedDynamics

Playing Enigmatica 6 with FluxNetworks-1.16.5-<IP_ADDRESS>
Automated flux dust production with secondary refined storage network (just takes redstone from chest and drops it in world) and IntegratedDynamics Player Simulator. Works fine on note block, keeps playing notes when redstone is dropped:

Doesn't work with obsidian - I think your integration does not recognises clicks somehow, as those were recognised fine by note block:

Using ftb-chunks but it does not work outside of those either.
Regards
Q

If I left click obsidian block myself flux dust will be produced

end of 2024, problem is still here

It requires a player or fake player click, and the crafting is not a block behavior, the obsidian itself is not clickable. Maybe IntegratedDynamics does not support this.

Actually it is working! (I am playing ATM9 - TTS)
You don't need any complex setups for this - only player simulator (+cable) and empty card!

...and setup actions with following:
Right click: false
Right hand: true
Continuous click: false
And its just works! I am dropping and collecting items with Modular Routers (dropper and vacuum modules), and placing/destroying obsidian/cobblestone with AE2 planes.
So I guess we can close this issue.


Exclude federal and international entities from CSA BCC functionality

🗣 Description
This PR modifies the CSA BCC functionality implemented in #101 so that CSAs are not sent reports/notifications for federal or international entities.

💭 Motivation and context
This change was requested by the CyHy team in #106.
Resolves #106.

🧪 Testing
Coming soon!

✅ Pre-approval checklist

[x] This PR has an informative and human-readable title.
[x] Changes are limited to a single goal - eschew scope creep!
[x] All relevant type-of-change labels have been added.
[x] I have read the CONTRIBUTING document.
[x] These code changes follow cisagov code standards.
[x] All relevant repo and/or project documentation has been updated
to reflect the changes in this PR.
[x] All new and existing tests pass.

✅ Pre-merge checklist

[ ] Finalize version.

✅ Post-merge checklist

[ ] Add a tag (but don't create a release).
[ ] Create a local Docker image: docker build -t cisagov/cyhy-mailer:1.7.0 --platform=linux/amd64 .
[ ] Push Docker image up to Docker Hub: docker push cisagov/cyhy-mailer:1.7.0
[ ] Deploy this change to Production.

Why not create a release?  It won't hurt anything, will it?

Why not create a release? It won't hurt anything, will it?

We don't have any releases yet in this repo (only tags), so I wasn't planning to break new ground.

Create a release.  Anytime we create a tag we should create a release to document the changes since the last release.  This repo doesn't have releases already just because it is old.