xskat: enable darwin build

Motivation for this change
This enables the xskat build on darwin.
Things done

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

[ ] NixOS
[x] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build xskat

Failure on aarch64-linux (full log)
Attempted: xskat
Partial log (click to expand)
make Makefiles
make: Nothing to be done for 'Makefiles'.
make includes
make: Nothing to be done for 'includes'.
make depend
gccmakedep  --   -I/nix/store/n8jlkihi2da4mfaj113xqvqc7qvnkgnb-xorg-cf-files-1.0.6/include    -Dlinux -D__aarch64__ -D_POSIX_C_SOURCE=199309L                           -D_POSIX_SOURCE -D_XOPEN_SOURCE                                 -D_BSD_SOURCE -D_SVID_SOURCE                                 -D_LARGEFILE_SOURCE -D_FILE_OFFSET_BITS=64                                                                   -DFUNCPROTO=15 -DNARROWPROTO   -DDEFAULT_LANGUAGE=\"english\" -DDEFAULT_IRC_SERVER=\"irc.fu-berlin.de\"   -- skat.c ramsch.c null.c bitmaps.c xio.c xdial.c irc.c text.c
building
substitute(): ERROR: file 'makefile' does not exist
builder for '/nix/store/s6nw2kf93qqs94y76p9hf62nxiq6pmkq-xskat-4.0.drv' failed with exit code 1
error: build of '/nix/store/s6nw2kf93qqs94y76p9hf62nxiq6pmkq-xskat-4.0.drv' failed

Success on x86_64-darwin (full log)
Attempted: xskat
Partial log (click to expand)
+ mkdir -p /nix/store/8nkcz6ng7b6rgckl0jfd6darnm3nmrv1-xskat-4.0/man/man1
install -c -m 0444 xskat._man /nix/store/8nkcz6ng7b6rgckl0jfd6darnm3nmrv1-xskat-4.0/man/man1/xskat.1
install -c -m 0444 xskat-de._man /nix/store/8nkcz6ng7b6rgckl0jfd6darnm3nmrv1-xskat-4.0/man/man1/xskat-de.1
install.man in . done
post-installation fixup
moving /nix/store/8nkcz6ng7b6rgckl0jfd6darnm3nmrv1-xskat-4.0/man to /nix/store/8nkcz6ng7b6rgckl0jfd6darnm3nmrv1-xskat-4.0/share/man
gzipping man pages under /nix/store/8nkcz6ng7b6rgckl0jfd6darnm3nmrv1-xskat-4.0/share/man/
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/8nkcz6ng7b6rgckl0jfd6darnm3nmrv1-xskat-4.0/bin
patching script interpreter paths in /nix/store/8nkcz6ng7b6rgckl0jfd6darnm3nmrv1-xskat-4.0

@LnL7 The more general solution to the problem: The xorg.imake derivation features a command xmkmf, which generates these makefiles from xorgcffiles. This one in turn contains a file lib/X11/config/darwin.cf, hard coding:
# define CppCmd                 /usr/bin/cpp3
# define CppCmd                 /usr/bin/cpp
#define CcCmd                   /usr/bin/cc
#define CplusplusCmd            /usr/bin/c++

So if we patch the darwin.cf file, we can probably make a lot more builds using imake succeed on darwin.

Well. I think that's mostly it. The other programs don't seem overly interesting.

@GrahamcOfBorg build xorg.xorgcffiles xskat oneko Xaw3d gv xsok

Success on x86_64-linux (full log)
Attempted: xorg.xorgcffiles, xskat, oneko, Xaw3d, gv, xsok
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/jkhj630dc2kfqyafq931m38ma8pq1y07-gv-3.7.4/bin
patching script interpreter paths in /nix/store/jkhj630dc2kfqyafq931m38ma8pq1y07-gv-3.7.4
/nix/store/k41w8cv366mfqpys7xh7f79kh3sw18qa-xorg-cf-files-1.0.6
/nix/store/jkhj630dc2kfqyafq931m38ma8pq1y07-gv-3.7.4/bin/gv-update-userconfig: interpreter directive changed from " /usr/bin/perl" to "/nix/store/cxdmh98g0lvl1dyq304c1lq7f90dh01f-perl-5.24.3/bin/perl"
/nix/store/xpn78hkdslbwlzypyzvahkfxgwpmzh7d-xskat-4.0
checking for references to /build in /nix/store/jkhj630dc2kfqyafq931m38ma8pq1y07-gv-3.7.4...
/nix/store/4sks2689nv5l1n0b3snga8jmfy56d3m3-oneko-1.2.5
/nix/store/cgfc4m6grlqmxnrp5x9l7scy6ivlj0i7-Xaw3d-1.6.2
/nix/store/jkhj630dc2kfqyafq931m38ma8pq1y07-gv-3.7.4
/nix/store/8wrlr3fkxvcvlzrpx0gkljpgqw4qq9qp-xsok-1.02

Success on x86_64-darwin (full log)
Attempted: xorg.xorgcffiles, xskat, oneko, Xaw3d, gv, xsok
Partial log (click to expand)
strip is /nix/store/kdff2gim6417493yha769kh00n63lnrw-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/8406ry2rv0pzbv0840v7khmglwz3lrrc-gv-3.7.4/bin
patching script interpreter paths in /nix/store/8406ry2rv0pzbv0840v7khmglwz3lrrc-gv-3.7.4
/nix/store/8406ry2rv0pzbv0840v7khmglwz3lrrc-gv-3.7.4/bin/gv-update-userconfig: interpreter directive changed from " /usr/bin/perl" to "/nix/store/lizm7q36j5sf5524bw9gsykflv9i2l7z-perl-5.24.3/bin/perl"
/nix/store/0h83p7wskip2d39qzsfz4ddpn7i0yjx6-xorg-cf-files-1.0.6
/nix/store/zsd9jlhnj31affm2bzf0pkhjfdn75vnh-xskat-4.0
/nix/store/3pl3ksn76fvrwlfqcsf0cma7izg17s1k-oneko-1.2.5
/nix/store/hlv6ffrfyb4a5iaaz1g640k8srq430kp-Xaw3d-1.6.2
/nix/store/8406ry2rv0pzbv0840v7khmglwz3lrrc-gv-3.7.4
/nix/store/rz4brxmi8ks2qym7ah1bpvgryk2s3qsr-xsok-1.02

Success on aarch64-linux (full log)
Attempted: xorg.xorgcffiles, xskat, oneko, Xaw3d, gv, xsok
Partial log (click to expand)
stripping (with command strip and flags -S) in /nix/store/7gdrgsh111haj9aiavpca8n0r8h7vfsz-gv-3.7.4/bin
patching script interpreter paths in /nix/store/7gdrgsh111haj9aiavpca8n0r8h7vfsz-gv-3.7.4
/nix/store/7gdrgsh111haj9aiavpca8n0r8h7vfsz-gv-3.7.4/bin/gv-update-userconfig: interpreter directive changed from " /usr/bin/perl" to "/nix/store/3qhzffjvl8gbiabikiqdnz4jhh9v84nz-perl-5.24.3/bin/perl"
checking for references to /build in /nix/store/7gdrgsh111haj9aiavpca8n0r8h7vfsz-gv-3.7.4...
/nix/store/djn1mp567yh5mih7mr8sczk1pnr0di4b-xorg-cf-files-1.0.6
/nix/store/hk64bsg1lxlfrd68133kljbmg8w5k4pc-xskat-4.0
/nix/store/q107w5yb39n3cpryh31jlcw0qhz9n4m9-oneko-1.2.5
/nix/store/lnyj8qd3xq796d6y76d47xvazz1vqh61-Xaw3d-1.6.2
/nix/store/7gdrgsh111haj9aiavpca8n0r8h7vfsz-gv-3.7.4
/nix/store/3v1z89x6rb9hh1bch3pmkrabi0b526nd-xsok-1.02


Various memory leak fixes

removes events bound to win in datepicker and dropdown
popover and tooltip were incorrectly listening for "destroy" instead of "$destroy"
popover and tooltip elements have their listeners removed on destroy


Websocket with flask dont work with sentry

Hi,
I have a web socket using flask, gevent and SocketIO, I'm trying integrate Sentry but the websocket stops working
my websocker runner is like it:
monkey.patch_all()

if __name__ == '__main__':
    print ('Listening on http://<IP_ADDRESS>:%s and on port XXXX '
           '(flash policy server)' % WEBSOCKET_PORT)
    SocketIOServer(('', WEBSOCKET_PORT), app,
                   resource="socket.io").serve_forever()

and my sentry integration is similar to:
from raven.contrib.flask import Sentry

monkey.patch_all()
app = Flask(__name__)
app.debug = True

sentry = Sentry(app, dsn=RAVEN_WEBSOCKETS_DSN)

I tried use GEVENT TRANSPORT https://docs.sentry.io/clients/python/transports/#gevent but  still not working.

I'm not sure Sentry automatically registers for the SocketIO part of the app (it does for the Flask part).
I got it to with some manual setup:
from flask import current_app

app = create_app()

# Setup Sentry
if 'SENTRY_DSN_SOCKETIO' in app.config:
    app.sentry = Client(app.config['SENTRY_DSN_SOCKETIO'], transport=GeventedHTTPTransport)

socketio = SocketIO(app)

@socketio.on_error_default
def grasp_default_error_handler(e):
    # Report to Sentry
  current_app.sentry.captureException()

This is assuming you're using Flask-SocketIO.
I'm not sure it's currently possible to intercept exceptions in pure python-socketio (See: https://github.com/miguelgrinberg/python-socketio/pull/45#issuecomment-246673001)


fix: Combobox isSelected function

Hey, I noticed that the isSelected function is currently always returning false since we're comparing two non-primitive object types. I changed the function to use JSON.stringify() instead which fixes that. Now the checkmark also shows up in the preview.

Thanks!! We have a deep equal function that can be used instead

@TGlide great, I didn't know about that. I'll update it in a bit.

@TGlide updated 😄


Copy UUID impl

Use the crypto module implementation in uuid.ts in more places.

Adding Don and Peng as reviewers to check whether the crypto module can be used in the files listed in the diff.


PipelineTask Names Can Be Created That Violate TaskRun Naming Rules

When creating a Pipeline that has PipelineTasks, you can give each PipelineTask a name that differs from a TaskRef's name:
apiVersion: tekton.dev/v1alpha1
kind: Pipeline
metadata:
  name: sleep-pipeline
  labels:
    app: tekton-workshop
spec:
  tasks:
  - name: pipelineTask1
    taskRef:
      kind: Task
      name: sleep1s
  - name: pipelineTask2
    taskRef:
      kind: Task
      name: sleep10s

The PipelineTask names can include capital letters, such as in the example above. If someone does this, the TaskRuns associated with the PipelineRun for the Pipeline above will get the following error:
Warning  TaskRunCreationFailed  109s                pipeline-controller  Failed to create TaskRun "sleep-pipeline-run-pvs5c-pipelineTask1-6wjd6": TaskRun.tekton.dev "sleep-pipeline-run-pvs5c-pipelineTask1-6wjd6" is invalid: metadata.name: Invalid value: "sleep-pipeline-run-pvs5c-pipelineTask1-6wjd6": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')
  Warning  TaskRunCreationFailed  109s

This error above occurs since TaskRun names must be lowercase, but PipelineTasks are allowed to be capitalized.
This will actually cause the PipelineRun to continue running until it times out after an hour.
Expected Behavior
Similar to what is done for Tasks, the Pipeline above should be rejected since the PipelineTask names don't follow the rules of how TaskRun names are specified:
The Task "sleeP10s" is invalid: metadata.name: Invalid value: "sleeP10s": a DNS-1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and end with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')

Actual Behavior
The error above.
Steps to Reproduce the Problem

Create a Pipeline with PipelineTask names with capital letters
Try to run the Pipeline

/kind bug

/assign

/test build-me


Accelerate integration to Hezar in Trainer

In order to support distributed training and all its possible methods, we'd better integrate 🤗Accelerate into Hezar's Trainer.

The task is almost done in the migrate-to-accelerate branch but I couldn't manage to test some stuff since I don't have access to a multi-GPU machine so I'd be glad to have someone help us out on this one.


#1194: Enhance GETRANGE to support byte array

Enhances GETRANGE to support byte array.
PR Checklist

[x] All datatypes are supported and error handling must be consistent.
[x] updating unit tests to cover new enhancement
[ ] updating all integration test to test new enhancement
[ ] documentation updates with new enhancement

closes #1194

Hi, @apoorvyadav1111. To be reviewed, after #1089 gets merged.

PR #1269


Removal of eefriedman from the active reviewers list

r? @jdm

This change is

@bors-servo: r+

:pushpin: Commit 8f45bf8 has been approved by jdm

:hourglass: Testing commit 8f45bf8 with merge b799906...

:sunny: Test successful - travis


MQTT KEEPALIVE Value different than Ewon Default

https://github.com/hms-networks/sc-ewon-flexy-extensions-lib/blob/7e119f0b26257a3f522aa91c0f8bcb64af933afb/src/main/java/com/hms_networks/americas/sc/extensions/mqtt/MqttConstants.java#L31
Is there a reason this is different than the Ewon default?

https://developer.ewon.biz/system/files_force/rg-0006-01-en-basic-programming.pdf

@alexjhawk @oliver-walker-hms

Unless there is a reason for this I would argue for setting this to "60"

Its the value a user would expect as the default if they were following the documentation. They may be unaware the library is explicitly setting this value at startup as well.
It reduces data usage.


ENH Add FitNet style model compression

http://arxiv.org/abs/1412.6550

This seems non-trivial in general - will close for now but consider once finetuning, etc. is added.


Use multi-Node.js-version HID.node, and process.nextTick for callbacks

Here's my current version of the master branch of BlinkStick, with the changes I proposed in the forum:

Use multiple versions of HID.node to support Node.js versions 0.8.0 and up (except 0.11.0-0.11.15 because these are incompatible with Nan and thus HID.node cannot currently be built for them).
Use process.nextTick to call all callbacks. This prevents a Node.js crash because of stack exhaustion if a loop exists where the callback of a BlinkStick method calls another BlinkStick method, etc.

Tested with multiple Node.js versions on windows 8.1

Whilst trying to move some code from fedora to windows I came across the windows 8 issue, I can confirm this PR works on windows 8.1 with node 0.12.2 (64bit) :+1:


Token trading volume query changes

Checked Syntax, moved query from Testing to Final

@Stephen-Hilton @chilaraiSxt @geosxt Please review the change and merge


ng test losing typings after retrigger

OS?
Linux (Ubuntu 16.04)
Versions.
@angular/cli: 1.0.0-rc.1
node: 6.10.0
os: linux x64
@angular/common: 2.4.9
@angular/compiler: 2.4.9
@angular/core: 2.4.9
@angular/forms: 2.4.9
@angular/http: 2.4.9
@angular/platform-browser: 2.4.9
@angular/platform-browser-dynamic: 2.4.9
@angular/router: 3.4.9
@angular/cli: 1.0.0-rc.1
@angular/compiler-cli: 2.4.9
Repro steps.

ng new ngTest
cd ngTest
yarn
Create folder typings in src
Create file common.d.ts in src/typings with content:

declare module Common {
  export interface IAction {
    type: string;
  }
}

In app.component.ts add a constructor

constructor() {
  const action: Common.IAction = { type: 'test' };
}

Running ng test once works
Saving app.component.ts (retriggering ng test) fails to find type definition, but test succeeds

The log given by the failure.
ERROR in /home/tobias/projects/external/ngTest/src/app/app.component.ts (12,19): Cannot find namespace 'Common'.)
Chrome 56.0.2924 (Linux 0.0.0): Executed 3 of 3 SUCCESS (0.222 secs / 0.215 secs)

After fiddling some more with this I notice that the typing error actually stops the rebuild, and runs the test again with the previous build.
This make it look like the test succeeds even though the typing fails, but it's just showing a result from a previous build.
To be able to run tests that actually tests my current code, I need to use ng test --single-run true.

I'm sorry to say that it still doesn't work with the application I created for this issue. I updated to rc4 as shown here:
    _                      _                 ____ _     ___
   / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
  / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
 / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
/_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
               |___/
@angular/cli: 1.0.0-rc.4
node: 6.10.0
os: linux x64
@angular/common: 2.4.10
@angular/compiler: 2.4.10
@angular/core: 2.4.10
@angular/forms: 2.4.10
@angular/http: 2.4.10
@angular/platform-browser: 2.4.10
@angular/platform-browser-dynamic: 2.4.10
@angular/router: 3.4.10
@angular/cli: 1.0.0-rc.4
@angular/compiler-cli: 2.4.10

The change I see is that the test doesn't succeed, which is certainly an improvement. :)
I'm I doing something wrong by adding a d.ts file like this. ng serve and ng build works fine with this setup, just like before.

@bialad

Even though that change was merged a couple of days ago

@filipesilva The results are different now, but still not correct.
Even though it's a standard cli application, I created a repo to base the discussion on. I recreated the application from scratch with the CLI 1.0.0 and Anuglar 4.0.0 version, to be sure I'm not encountering any bugs because of mismatching version when updating.

Now when I run ng test the first time, everything work. As it did before.

Then changing the code in app.component.ts behaves as expected. For example changing 'test' to 1 fails. Great so far. :)

If I keep 'test' in AppComponent, and instead change the type property in common.d.ts from string to number, it successes. Not good.

If I keep the type change and stop and restart ng test it still succeeds, which it shouldn't...

Then changing 'test' i AppComponent to 1 succeeds (as it now should), BUT when changing 1 back to 'test' it falis (as it should).

This is very confusing to me, especially when the behavior isn't consistent. My guess is some sort of type caching not updating correctly...
Hope you could follow my steps here, and am able to reproduce this issue. Otherwise it must be something in my platform. The fact the I can now run my tests without single-run as long as I don't change my typings is i big step forward and a time saver.

I'm experiencing a similar issue as well. I'm on cli, 1.0.0-rc.2 but our ng version is at 2.4.9. Karma is 1.5, typescript is 2.1.6. Running a test the first time works as expected, make a simple change to kick off another test run and the build will randomly fail with some random typescript error, such as 'HTMLInputElement' not found, for what appears to be a problem with the typings get lost. This doesn't appear to be an issue that occurs every time, seems to be a bit random. I've been noticing this off and on for about a week. Not sure the root cause, but the above sounds like a match.

@bialad ah, that exist exists but it's a different one! https://github.com/angular/angular-cli/issues/5634 seems to be related to interfaces and probably all .d.ts files instead. We're tracking it there instead.
@boldwade unfortunately to fix this you really need to update to 1.0.0, that's when the fix landed.


Support headers via annotation

E.g.,
X-API-KEY: API_KEY
https://www.id123.io/knowledgebase/api-v2-introduction/
httpHeaders.set("APIToken", taxServiceConfig.getApiKey());

Merged to master.


[OSSM-1258] bump prometheus/client_golang version

Bump module prometheus/client_golang to v1.11.1. Tidy and update vendoring with new package versions.
Tested by:

built operator and deployed an SMCP on CRC, deployed bookinfo, ensured metrics were visible on grafana.
locally ran quay.io/maistra-dev/maistra-builder:2.3 make test, those tests passed on my end.

Show me some green tests prow :cold_sweat:

you were right, checked with Rob as well, closing as InstrumentHandlerCounter does not appear to be used in our code base or called in methods we use from dependencies.


Allow fomattedAddress in openContactForm android

fixes #454

Please check this and merge this.
I am copying the Java file every time after npm install.
It's very stressful.
Please merge this.
Thank you!

how does this compare to the iOS version? does iOS return the address?
btw, you can use npm link to change your dependencies locally.

You asked about the iOS, and I think in iOS version the formattedAddress is not needed.
Because in android phone there is only address field for showing contact. So we use formattedAdress in android.
And the ios, there are street, city, state, postalcode, country... so we don't need formattedAddress.

is this new field used in the example app? https://github.com/rt2zz/react-native-contacts/tree/master/example if not please add that as well.

I think there is no example of openContactForm in that example.
So I think it doesn't need to update example.

hmm, we should add an example of the openContactForm

I will pr if I have time.


⚠ Samsung has degraded performance

In a367d66, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2421 ms

Resolved: Samsung performance has improved in 74144b7.


Create witness comparison tool

ref https://github.com/paradigmxyz/reth/issues/10425
If we want to compare witness output from a "bad" node, from that of a "good" node, we need a way to easily compare witnesses.
This should take the output of debug_executionWitness (or the variant with preimages), and output a nice, human readable diff.

Prior art https://github.com/0xPolygonZero/zk_evm/tree/develop/mpt_trie/src/debug_tools


Unable to authenticate the request due to an error: [x509: certificate signed by unknown authority (possibly because of "crypto/rsa: verification error" while trying to verify candidate authority certificate "front-proxy-ca"), x509: certificate signed by unknown authority]

Clean kubernetes 1.13.4 setup, other operators work normally like rook/ceph.
Tried with default apiserver.ca and with the real one retrieved via kubectl config view --minify=true --flatten -o jsonpath='{.clusters[0].cluster.certificate-authority-data}' (base64 decoded for re-encoding...).
kubedb version: 0.11.0
install via ansible:
- name: install kubedb operator helm chart
  helm:
    host: localhost
    chart:
      name: kubedb
      version: 0.11.0
      source:
        type: repo
        location: https://charts.appscode.com/stable
    values:
      enableAnalytics: false
      apiserver.ca: "{{ ca.stdout | b64decode }}"
    state: present
    name: kubedb-operator
    namespace: kubedb
  environment:
    KUBECONFIG: "{{ playbook_dir }}/kubeconfig"
    TILLER_NAMESPACE: kube-system
- name: wait for kubedb crds
  pause:
    seconds: 5
- name: install kubedb catalog helm chart
  helm:
    host: localhost
    chart:
      name: kubedb-catalog
      version: 0.11.0
      source:
        type: repo
        location: https://charts.appscode.com/stable
    values:
      enableAnalytics: false
    state: present
    name: kubedb-catalog
    namespace: kubedb
  environment:
    KUBECONFIG: "{{ playbook_dir }}/kubeconfig"
    TILLER_NAMESPACE: kube-system

error message / log spammed:
I0430 07:09:55.791833       1 xray.go:232] testing ValidatingWebhook using an object with GVR = kubedb.com/v1alpha1, Resource=redises
E0430 07:09:55.794304       1 authentication.go:65] Unable to authenticate the request due to an error: [x509: certificate signed by unknown authority (possibly because of "crypto/rsa: verification error" while trying to verify candidate authority certificate "front-proxy-ca"), x509: certificate signed by unknown authority]

no creation of a postgres cluster is possible.
Didn't receive help in the slack channel so I guess it is a bug and not a configuration issue.

Moved forward with https://github.com/zalando/postgres-operator , in case nobody else is interested in solving this issue -> better close


Support for Custom-Header

Support for Adding Custom Header.

From ServiceClient

cc := kp.ClientConfig{
		BaseURL:    "BASE_URL",
		APIKey:     "API_KEY",
		InstanceID: "INSTANCE_ID",
		Headers: map[string]interface{}{
			"Custom-Header": "custom-value",
		},
	}

From ServiceCall (Per API)

Define Header just before the API Call and Set it nil when done with it.
client.Config.Headers = map[string]interface{}{
		"Custom-Header": "custom-value",
}

key, err := client.CreateKey(params)
	if err != nil {
		panic(err)
	}

client.Config.Headers = nil

Test Results

Integration Test :

go test -v -tags=integration ./...


ng-valid when form control is invalid

Using angular 18 and the latest version of this library, it happens that the formControl associated to the ngx-material-intl-tel-input component is invalid, but the ng-valid class is shown.

It should be fixed in the latest release(18.1.1). Let me know if it works.
Saludos!

Mañana pruebo, tengo instalada la 18.1.0, instalo la nueva y te digo si me ha funcionado. !Muchas gracias!

I have just to update the library and works fine, thanks a lot!!


All the novels are gone.

So, I woke up, and as the title suggested, all the novels are gone from my library, and I can't open any new ones. Even when I try to open one that’s already downloaded through the file manager, it won’t work. I also tried downloading new novels, but I can't open those either.

Never mind, it fixed itself. I’ll wait and see if it happens again or if I can find a way to reproduce it.


Plugin seems not triggered with serverless 1.33.0 (and higer)

Hi,
Since my update to sls 1.33.x few days ago, serverless-mocha-plugin doesn't work anymore.
While executing "sls invoke test", nothing happens. Literally nothing. If "--verbose" and "SLS_DEBUG=*" are used, the only output I get is:
...
Serverless: Load command invoke
Serverless: Load command invoke:test
Serverless: Invoke invoke:test

And that's it.
I tested with both 1.9.0 and 1.8.2 npm version of the plugin, and I'm using node version 11.2.0.
I tried using mocha through npm and all my tests are properly executed.
Guessing it had something to do with sls update, I downgraded it to 1.32.0 and the plugin worked again, properly.
I'll gladly provide any more information if needed, feel free to ask !
Thanks !

Same behaviour here, I had to downgrade to 1.32.0

still an issue in serverless 1.34, downgraded to serverless 1.32 to fix.

This should now be fixed in 1.9.1

Hi,
Thanks for the fix, sls invoke test is now working.
Unfortunately, there seems to be a regression with "sinon". Now I got this kind of error:
TypeError: Attempted to wrap fail which is already wrapped
      at checkWrappedMethod (node_modules/sinon/lib/sinon/util/core/wrap-method.js:39:21)
      at wrapMethod (node_modules/sinon/lib/sinon/util/core/wrap-method.js:87:13)
      at Function.spy (node_modules/sinon/lib/sinon/spy.js:46:16)
      at Sandbox.spy (node_modules/sinon/lib/sinon/sandbox.js:275:26)
      ...

I've tested with SLS 1.34.0 and 1.35.0 + Sinon 7.1.1 and above + serverless-mocha-plugin 1.9.1.
For the record, everything was working fine with SLS 1.32.0 + Sinon 7.1.1 + serverless-mocha-plugin 1.9.0.
I'm note quite sure what could cause this issue, but let me know if more tests or details are needed.
Thanks !


I can import radiomics, but no featureextractor

When I do this:
import radiomics.featureextractor as FEE
I get:ModuleNotFoundError: No module named 'radiomics.featureextractor'
But import radiomics only is successful. Could someone kindly help with this?

You might have installed radiomics package instead of pyradiomics.


Add Prysm

Adding Prysm Network chain.json and IBC

Removed the duplicated Prysm directory created outside of testnets directory, updated IBC.jsons to match that of comments/suggestions made. Updated Prysm description in the assetlist.json

IBC files amended

Removed initia, blumbus


Stabalise backend + remove mongoDB dependency from backend

https://trello.com/c/5VQjd4rT/81-look-into-removing-mongodb-as-dependency-for-types
Summary
This works to stabilise the backend endpoints and remove the MongoDB dependency from the frontend (which was done via the types package)
It also works to increase the security with the backend by doing runtime checks of the query.
This also adds testing for:

API Client
types
DB Client (partial)

TODO:

[x] Rebase off of sammy's testing branch
[x] Testing (including for new DB collection wrap)
[x] Move types to be dependency rather than devDependency
[x] Add runtime validation of the creation objects
[ ] Finish off endpoints marked as TODO
[ ] Update api client with new endpoints

@WilfSilver can you add coverage to these packages too?

I think due to the deadline today, I will postpone the finishing off endpoints and adding to the api client for another PR


Reload cell item with a new size

Is possible to change the size of an item dinamically? I've tried to reload a cell using reloadObjectAtIndex and changing the size inside the datasource but nothing happends.

I'm not sure if it's possible to have different cell sizes - i hardly think so (but I'm not the author).
If you need to change the size, reloadData is probably the way to go.

Yeah you're right, that's a mistake of mine :)
Okay it could be a to-do feature :P

I believe the goals of this grid project are different - allowing custom-sized cells is a whole different bag of pain.
You can use same-sized cells and use different size within - so that the absolute border to the next cell is the same.

There are actually 2 ways to approach this.  Assume ,for instance, that you want variable heights:

A GMGridViewLayoutVerticalVariableHeightStrategy that has no access to the datasource.

it merely caches cell sizes as it produces them, based on previously seen sizes.
this is much more difficult, and requires more modification in the core code.
A theme of the changes in core revolve around being able to handle unbounded ranges or sizes, and then checking continuously as new origins are asked for to notice when the range or contentSize is now determined.
I have hacked together some stuff to do this, but it still suffers from content scrolls that are fast enough to not have asked for all previous cells when asking for a position origin. (since it will not have cached those position frames)

A VariableHeightLayoutStrategy that is constructed with access to the datasource.  It can then maintain its own cache of cell sizes by asking the datasource directly.  Less change is needed to the core code for this approach.

the major theme here, which is also needed in approach 1, is to have the core code act on and pass along the actual cell sizes, instead of the dataSource "size" which in this case is more of a maxSize.

Also, for this type of strategy, it is probably best to convert layout strategies to return NSIndexedSet instead of NSRange, as you may have to say that 0 - 45 is in bounds, but yet only half of those cells may be in bounds depending on how the heights stack up.  Without this, your views and cache are bigger than they may need to be, which maybe isn't a big deal.
Note that i've only experimented with getting a layout and scrolling, not with any of the movement/editing/dragging/zooming etc...

http://i.imgur.com/l7rFv.jpg

Nice! One other way would be to have blocks in the layoutStrategy classes to compute the cellSizes. So the grid would set the block of the layout strategy and in it's implementation, call the datasource. The layout would just run the block to get the cellSize for each cell whenever needed; this way, the layout strategy doesn't know anything about the data source.
How are the cells aligned at the end of the scroll ?

I wasn't sure if you wanted the grid involved, so for now i just construct the strategy with a protocol that produces the sizes.
by aligned, do you mean visually at the bottom of the scroll?  however they stack up.  You can end up with a bunch of "jagged" white space.  although i have an idea for how to make it look better.


Reset query params to the initial params when un-mounting.

I encountered the same issue as https://github.com/storybookjs/addon-queryparams/issues/1 and ended up fixing it locally in our own decorator–no longer using this addon–but I might as well push it upstream to this addon.  Further basis in that issue and this comment.  Should fix https://github.com/storybookjs/addon-queryparams/issues/1.
When unmounting our Story, this takes the initialSearch (the initial location.search) and set location.search to the original value.
:warning: This does not maintain any query params added in the meantime, eg. if some other Storybook addon modifies the query parameters, it will be lost as initialSearch.current is from the very first render and never updated.
Quick video to visualize the Storybook file changes:
https://www.loom.com/share/bc19e586965f45a7b9b17b865b7e79ed

@kylorhall would you like maintainer right to this package?

Hey @ndelangen unfortunately my new role does not use Storybook very much and not this use-case for certain; I wouldn't have any need or ability to maintain this now.  I'm happy to direct someone from my prior organization if you were looking, see if they're keen—if they use this workaround I built or not.

Ok, I'll close this PR for now, until there's someone interested in this.


Dashr drug discovery

Issue for app: #[issue number here]
App pull request

[x] This is a new app
[x] I am improving an existing app (redesigns/code "makeovers")

About

Playground deployment URL (new version):
Current gallery app URL: (delete this line if inapplicable)
Python app repository link: (delete this line if you are working on a Python app)

Workflow

[x] I have created a branch in the appropriate monorepo, and the
elements necessary for successful deployment are in place.
[ ] If the app is a redesigned and/or restyled version of an
existing gallery app, I've summarized the changes requested in the
appropriate Streambed issue and confirm that they have been applied.
[ ] If the app is on the Dash Gallery portal, I have added a link to
the GitHub repository for the source code in the portal description.
[ ] If the app is a reimplementation of a Python gallery app for the
DashR gallery, the app in this PR mimics, as closely as possible,
the style and functionality of the existing app.=
[ ] I have removed all Google Analytics code from the app's
assets/ folder.

The pre-review review
I have addressed all of the following questions:

[ ] Does everything in my code serve some purpose? (I have removed
any dead and/or irrelevant code.)
[ ] Does everything in my code have a clear purpose? (My code is
readable and, where it isn't, it has been commented appropriately.)]
[ ] Am I reinventing the wheel? (I have used appropriate packages to
lessen the volume of code that needs to be maintained.)

As we discussed offline, you'll need to change
df <- read.csv('/dashr-drug-discovery/data/small_molecule_drugbank.csv', header = TRUE, sep = ",")

to
df <- read.csv('./data/small_molecule_drugbank.csv', header = TRUE, sep = ",")

since setwd already places us in the dashr-drug-discovery directory 🙂

You'll also need three more deployment-related files in your dashr-drug-discovery directory:
Aptfile

.buildpacks
init.R

@mleech1 OK, unfortunately, still not ready to merge; need to address a few warnings:
3D scatter, 2D scatter, 2D histogram on initial load of callback:
warning: `line.width` does not currently support multiple values. from NULL

On initial load of 2D histogram:
warning: `line.width` does not currently support multiple values. from NULL
warning: 'histogram2d' objects don't have these attributes: 'text'
Valid attributes include:
'type', 'visible', 'showlegend', 'legendgroup', 'opacity', 'name', 'uid', 'ids', 'customdata', 'selectedpoints', 'hoverinfo', 'hoverlabel', 'stream', 'transforms', 'uirevision', 'x', 'y', 'z', 'marker', 'histnorm', 'histfunc', 'nbinsx', 'xbins', 'nbinsy', 'ybins', 'autobinx', 'autobiny', 'xgap', 'ygap', 'zsmooth', 'zhoverformat', 'hovertemplate', 'zauto', 'zmin', 'zmax', 'zmid', 'colorscale', 'autocolorscale', 'reversescale', 'showscale', 'colorbar', 'xcalendar', 'ycalendar', 'xaxis', 'yaxis', 'idssrc', 'customdatasrc', 'hoverinfosrc', 'xsrc', 'ysrc', 'zsrc', 'hovertemplatesrc', 'key', 'set', 'frame', 'transforms', '_isNestedKey', '_isSimpleKey', '_isGraticule', '_bbox'
 from NULL
warning: 'scatter' objects don't have these attributes: 'colorscale', 'showscale'
Valid attributes include:
'type', 'visible', 'showlegend', 'legendgroup', 'opacity', 'name', 'uid', 'ids', 'customdata', 'selectedpoints', 'hoverinfo', 'hoverlabel', 'stream', 'transforms', 'uirevision', 'x', 'x0', 'dx', 'y', 'y0', 'dy', 'stackgroup', 'orientation', 'groupnorm', 'stackgaps', 'text', 'hovertext', 'mode', 'hoveron', 'hovertemplate', 'line', 'connectgaps', 'cliponaxis', 'fill', 'fillcolor', 'marker', 'selected', 'unselected', 'textposition', 'textfont', 'r', 't', 'error_x', 'error_y', 'xcalendar', 'ycalendar', 'xaxis', 'yaxis', 'idssrc', 'customdatasrc', 'hoverinfosrc', 'xsrc', 'ysrc', 'textsrc', 'hovertextsrc', 'hovertemplatesrc', 'textpositionsrc', 'rsrc', 'tsrc', 'key', 'set', 'frame', 'transforms', '_isNestedKey', '_isSimpleKey', '_isGraticule', '_bbox'
 from NULL

Related to hoverdata outside of window (as we discussed):
error: subscript out of bounds
request: <IP_ADDRESS> - ID_<IP_ADDRESS> [28/Jun/2019:11:51:49 -0400] "POST /_dash-update-component HTTP/1.1" 500 0 "http://<IP_ADDRESS>:8050/" "Mozilla/5.0 (Macintosh; Intel Mac OS X 10_13_4) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/11.1 Safari/605.1.15"

For the last one, I'll help sort out a workaround. I suspect that we could set a default that is non-null, or edit the callback to address this gracefully.

Awesome! All fixed now! 💃 💃 💃


feat: addComputed

#90 trial made me think that valtio's computed model is different from other libraries concepts.
proxyWithComputed works with object getters because it is the same proxy object.
We can't support addComputed and nested with the same way.
So, here we add a new attachComputed. Unfortunately, we can't infer typing nicely.
This should close #85 and close #86.
Also, I added overwriting check which resolves https://github.com/pmndrs/eslint-plugin-valtio/issues/5.

Should we name it addComputed?

addComputed would be better IMO

So this PR adds nested computeds and addComputed?

More precisely, I concluded that the ideal nested computed is not possible. You can get the similar result using addComputed (with the optional argument, we need to document this.)

I didn't saw a test demonstrating addComputed with get/set and nested, is the API different than proxyWithComputed then?

yes, different. Let me add some tests later.
And regretfully, I wasn't able to support set in addComputed. It could be with a different property like get for .foo and  set for .setFoo(), but then it doesn't have to be a util. At least, not for this one.

I see the optional third argument now. Why it can't be:
addComputed(
    state.math,
    {
      doubled: (snap) => computeDouble(snap.math.count),
    }
  )

btw if in the future we're able to support computed getter/setter with addComputed might as well remove proxyWithComputed to have only one way of having computeds in the state.

Oh, that was my desire as I wrote at first in #86.

doubled: (snap) => computeDouble(snap.math.count),

No way to access this snap. It has to be doubled: (snap) => computeDouble(snap.count),, which is not "nested".

Why a state property is not nested, does valtio stores it in a flattened path internally?

Not sure what's you are asking. it's just js objects. you don't even know if there's only one parent.
const obj1 = {}
const obj2 = {}
const child = {}
obj1.child = child
obj2.child = child

If you only get child, you don't know its parent.

That's what I thought, so I can't understand why you can't add the computed property to the reference as the first parameter

I guess you are confused with proxyObject and targetObject. The first argument is to create snapshot.

Still confused lol

Hm, not sure if I should merge this. Nobody is going to use it. I wish we could add this and drop proxyWithComputed but that's also a nice function with a cute hack.
Leaving it for a while..

Yeah, I feel like if you were able to drop the third parameter and have addComputed(state, obj_with_computeds) would be a cool replacement to proxyWithComputed

Didn't expect you say so. The third argument is for nesting and the big reason of developing addComputed over proxyWithComputed. addComputed doesn't support setter, which I'm personally fine, but it's one of the cute features in proxyWithComputed.

well, at least it's a optional argument :)


Fix dates and operation duration

When switching from moment to luxon, I’ve accidentally forgot to correctly change the formatting on the main page

Tasks

Change the “operating since” date to a common date format
Change the „already“ date to duration word format (eg “already 2 years, 5 months and 3 days”)

I can fix this one


Fixing broken links in FAGs

Added check for window to be present so that the build does not fail

Just realised that this pull request is a duplication of commit 7a28c49231fa6a880e1a611553e90bac8545f193 included in #7. Leaving this here in case it is of any use but otherwise feel free to reject it


[Feature Request] Song speedups

I think speedups would be pretty good for practicing skill with the easier songs, as some songs are really good for building up right hand finger independence but normally are too slow to where doesn't really help improve it but speeding those songs up would.

This may be possible, but it would require implementation of a time stretching library and loading the whole audio file for each stem into memory instead of streaming from disk


Modification of the current working directory can lead to unexpected behavior

Environment

Covalent version: 0.234.0rc0
Python version: 3.10.13
Operating system: Ubuntu

What is happening?
Covalent automatically changes the current process' working directory, which can lead to significant confusion for users who might not anticipate this behavior. For instance, if the user runs a @ct.electron that writes out a file to the filesystem but does so using a relative path, the files will not be where they think they are. In some cases, the user may not have full control of the path handling (e.g. if they are calling some external library).
How can we reproduce the issue?
import covalent as ct

@ct.lattice
@ct.electron
def test():
    from pathlib import Path

    return Path.cwd()

ct.dispatch(test)()

In my case, I got back
/home/rosen/software/miniconda/envs/covalent/lib/python3.10/site-packages/covalent_ui

even though I was not running from that directory.
What should happen?
The working directory of the Python process when run via the Dask or local executors should not be modified from that chosen by the user (or, at the very least, this details should be emphasized in the documentation).
Any suggestions?
No response

Apologies. This was a duplicate of the already-solved https://github.com/AgnostiqHQ/covalent/issues/1619.


fix(s3): Update aws-sdk to v1.11.717

Trying to assume an IAM role with the EKS pod identity webhook results in a stackoverflow error with Spinnaker v1.19.0 (Front 50 v0.22.0). The issue is described in https://github.com/aws/aws-sdk-java/issues/2136 and updating the aws-sdk should fix it.

I tested with a self-build image now, and with this change it works as expected.
To be able to access the credentials file in the injected volume a security context has to be set.
kubernetes:
  serviceAccountName: spinnaker-front50
  securityContext:
    fsGroup: 102

in ~.hal/default/service-settings/front50.yml

Closing this, so we can fix the root cause in kork (https://github.com/spinnaker/kork/pull/554).  Thank you for debugging this!


[flight-rpc][flight-jdbc][sql] - Dynamic parameters on columns that created in runtime not working

Describe the bug, including details regarding any error messages, version, and platform.
Here's an example of code that causes the issue:
# stmt = text("SELECT *, (CASE WHEN (`OID` IS NULL) THEN :x ELSE :y END) as Temp FROM `Dynamic.Schema.Dts`.`Table`")
# rows = con.execute(stmt, {'x': 1, 'y': 0}).fetchall()

I'm unable to select with this query because I create a column on fly in runtime, after it's created I would like to query it.
And it's seems impossible.
I've tested it with: 15.0.2 and it still has the same issue.
Thank you in advance for the help.
If anymore information is required please comment that.
Component(s)
FlightRPC, Java, Other

What exactly is the backend and what is the error? At first glance this is between you and your database vendor.


babashka: Add -H:PageSize=64K build option on aarch64

Description of changes
Babashka builds upstream with -H:PageSize=64K for aarch64. This PR updates things accordingly.
The change enables the package to be used with all aarch64 kernel page sizes (4K, 16K and 64K) with the downside of a bit of wasted memory.
Here is the PR from babashka that I based the changes on: https://github.com/babashka/babashka/pull/1606
Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux (4K and 16K kernel page size)
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

What are the implications of this change for usage in non-Asahi systems?

Thanks for the quick reply @thiagokokada. I updated the "Description of changes" to be more explicit on the reasons and effects of the change.
There should be no impact on non-Asahi. I also removed the mention to asahi as this issue applies to any non-4K kernels.
Also fixing the commits as requested.

Throwing my comment into the ring: I think having an explicit PageSize value set in a buildGraalvmNativeImage would be a good move, instead of relying on the default behavior of PageSize=0.

Throwing my comment into the ring: I think having an explicit PageSize value set in a buildGraalvmNativeImage would be a good move, instead of relying on the default behavior of PageSize=0.

Given everything that @diegobfernandez I would be fine having this set directly in buildGraalvmNativeImage too, however only in aarch64-linux systems because I think it is better to let macOS binaries be build with their default PageSize (that will be 64k in this case, since I imagine this is the the default in aarch64-darwin).

Got it. I will close this one and reference it in a new PR with the changes in buildGraalvmNativeImage. The changes are ready I just didn't had the chance to test all packages that rely on it (so far I tested babashka, clojure-lsp and cljfmt).


Changed values to real functions to get rid of warnings

Changed the values to functions to avoid casts

This should work for the cl2000. Can test on Monday.


[W07H03] simple behaviour tests for can-eat

Very basic tests for can eat behaviour

@jonasfeld Tag me, if this PR is ready to merge.

You can create a new PR for additional tests.


back to previous step in a conversation via a keyboard button in current step

I want to add a "back" button in beginning of keyboard button that is shown in each step of a conversation. when user touch that button Will be returned to exactly prev step conversation.
I added an else if block With the content you see that sets $notes['state'] value to prev step index:
//State machine
//Entrypoint of the machine state if given by the track
//Every time a step is achieved the track is updated
switch ($state) {
    case 0:
        if ($text === '') {
            $notes['state'] = 0;
            $this->conversation->update();

            $data['text']         = 'Type your name:';
            $data['reply_markup'] = Keyboard::remove(['selective' => true]);

            $result = Request::sendMessage($data);
            break;
        }

        $notes['name'] = $text;
        $text          = '';

    // no break
    case 1:
        if ($text === '') {
            $notes['state'] = 1;
            $this->conversation->update();

            $keyboard = new Keyboard(
                ['⬅ Back'],
                ['item1','item2','item3' ]
            );

            $keyboard
                ->setOneTimeKeyboard(true)
                ->setResizeKeyboard(true);

            $data['reply_markup'] = $keyboard;

            $result = Request::sendMessage($data);
            break;
        } else if ($text === '⬅ Back'){
            $notes['state']  =   0;
            $this->conversation->update();
            break;
        }

        $notes['surname'] = $text;
        $text             = '';

            // no break

Hi @ahmadbadpey, what is your question exactly? Does the code you have not work then?

@noplanman
no does not work and What I expect will not happen.is correct content of that else if block?

I recomment you to use inline buttons for this feature.
It more stable for do things like this.
So at "Back" button will be callback for function, which will update state of conversation and call first switch statement.

Code seems right. Please check in the database to see if the state is changed.
You aren't sending any new message in that else block, so the bot isn't sending anything back to the user.

@ahmadbadpey Any update here?


Support for receiving events in batches in processor client

This PR is to add support for receiving events in batches in event processor.

@conniey added code snippets and samples


Queue Enhancements

Firstly thanks BHJ for the work you've put into making a great Karaoke app.  Long story short I've had the opportunity to use the app a few times with small groups of friends.  I have some suggestions regards the queue that would certainly improve the usability for me and probably other people as well.
In my case I use the same room each time, more often than not there are items left in the queue from the last time.  So far as I can tell there is no ability to clear more than 1 item at a time from the queue, the only option is to delete the room and recreate it.  Ordinarily having to log out and back in again is a minor inconvenience, but if you use a smart tv as the player and have to do this via the tv's browser it's a major pain in the ass.  Surely there's justification for the admin to be able to better manage the queue, whether it be a clear all button, or a multi select and clear feature.
The other issue I've had is with monitoring the queue, a smart phone works great for people to be able to browse my library and pick the songs they want to sing, but not so much for keeping track of the queue, most phones lock themselves after a few seconds, so in order to check the queue you have to unlock it and hope you've left a tab open in the browser you can quickly re-open.  Most of my guests manage to close the tab/browser or navigate away from the page.  The only other indication for whose up next is the on screen notification, but that is at the beginning of the song which is too late if someone is not already where they need to be.  There are a few ways I think this could be improved, one way could be to have a separate /queue interface, like the player, so a separate display could be set up for the sole purpose of displaying the queue positions.  That may not be practical for most applications though, so perhaps having the queue show unobtrusively via the player down the side where it won't interfere with the lyrics, or even just briefly unobtrusively overlay via the player towards the end of a song, these could even be opt in features in the settings i.e enable queue overlay via player.

In my case I use the same room each time, more often than not there are items left in the queue from the last time. So far as I can tell there is no ability to clear more than 1 item at a time from the queue, the only option is to delete the room and recreate it.

Right, "rooms" are meant to be per-session partly so that the history is preserved, and could be used to determine popular songs, etc. It's best to set finished rooms to "closed" and create a new one if possible.

Ordinarily having to log out and back in again is a minor inconvenience, but if you use a smart tv as the player and have to do this via the tv's browser it's a major pain in the ass.

100%, changing rooms as an admin/player should be easier. I'm also thinking a "player" user that could stay logged in and be assigned to rooms would make sense.

a separate /queue interface, like the player, so a separate display could be set up for the sole purpose of displaying the queue positions

This would be super cool! Probably should happen at the API level so any app could query this stuff.

unobtrusively overlay via the player towards the end of a song

Agreed, now that in v1.0 the next-up singer gets "locked in", there could be an up next notification as a per-room setting. There's a lot of fun stuff that could be done with the player overlay; it's all very 1.0 currently :)
Great feedback, thanks!

Thanks for the awesome karaoke app, in the future it is better if you add a function to queue a newly downloaded song without scanning or refreshing my entire collection of songs. because if someone wants a song but not exist in my collection and I decided to download it from the internet and I need to scan again the entire existing collection before it will appear in the library search.

@jayaranah Thanks, and keep an eye on #27.


Fix typo in UIRequiredDeviceCapabilities for iOS

Summary
This PR changed the UIRequiredDeviceCapabilities value for iOS: https://github.com/mattermost/mattermost-mobile/pull/8011
However, after it was merged, Apple refused to accept the weekly beta build:  https://github.com/mattermost/mattermost-mobile/actions/runs/9646645994/job/26603731918#step:5:9750
Looking up some docs it seems the correct value expected by UIRequiredDeviceCapabilities is arm64: https://developer.apple.com/documentation/bundleresources/information_property_list/uirequireddevicecapabilities/
After this PR is merged, i'll trigger a new beta build for iOS cc @amyblais
Ticket Link
https://mattermost.atlassian.net/browse/CLD-7948
Release Note
NONE

Thanks for the fix


Add scratch

Build a Docker image of Linuxbrew from scratch.
Install Linuxbrew using only Linuxbrew bottles.

Requires this patch of linuxbrew/brew to work:
diff --git a/Library/Homebrew/requirements/glibc_requirement.rb b/Library/Homebrew/requirements/glibc_requirement.rb
index e52bf23..56c3d3c 100644
--- a/Library/Homebrew/requirements/glibc_requirement.rb
+++ b/Library/Homebrew/requirements/glibc_requirement.rb
@@ -12,6 +12,7 @@ class GlibcRequirement < Requirement
     end
     raise "Unable to locate the system's glibc" unless libc
     version = Utils.popen_read("#{libc} 2>&1")[/[Vv]ersion (\d\.\d+)/, 1]
+    version = Utils.popen_read("/lib64/ld-linux-x86-64.so.2 #{libc} 2>&1")[/[Vv]ersion (\d\.\d+)/, 1] unless version
     raise "Unable to determine the system's glibc version" unless version
     @system_version = version
   end

I'd suggest using /bin/bash as sn entry point.

/bin/sh is a symlink on this system to bash, but I vaguely recall bash starting in a compatibility mode if it's run as sh rather than bash. Does that ring any bells?

Rather than using the above patch to glibc_requirement, I'd like to fix it so that being unable to determine the host version of glibc is only a warning, not an error. Alpine Linux does not have glibc at all, so that fix is needed for Alpine Linux.
See https://github.com/Linuxbrew/docker/pull/25


Collision Sibling on Imported Scene

In my project, I have a node in my scene that is an instance of an automatic scene generated from an imported GLB that contains animations. I have the context menu option "Editable Children" enabled on this node so that I am able to select its AnimationPlayer for my AnimationTree state machine. In order to add collision detection to this node, I wanted to "Create Single Convex Collision Sibling" for its child mesh node.

However, because the scene is automatic, I am not able to begin editing this generated collision shape in any way, without actually instantiating the scene. Ideally I want to move it out of this part of the subtree and place it where my other collision shape node is, so that it can be used for the kinematic body. Any attempts to move or reparent the node result in an error message. It's not even possible to delete the node without hitting undo because of this!

Did you save the imported model to a scene? Godot cannot edit the exported files directly AFAIK, so you need to save first before being able to make any changes.

That much is understandable, and I was already aware of. However, is the creation of "Create Single Convex Collision Sibling" not an actual bug? Why is that option from the view able to modify the node, while other options from the context menu are not? Is it not confusing to the user for it to be allowed, and then not be able to reverse it unless it's still in the undo stack?

Just my 2nd day playing with Godot and right away having similar issues with the generated Collision Shapes being grayed-out for the imported glb assets. Saving the scene and then "Scene --> Reload Saved Scene" enables the Collision Shape node and it can then be properly edited, reparented or deleted.
To me it seems like a bug with the newly generated Collision Shape inheriting some non-editable property from the source sibling, which only gets cleared when the scene is being reloaded.

Just my 2nd day playing with Godot and right away having similar issues with the generated Collision Shapes being grayed-out for the imported glb assets. Saving the scene and then "Scene --> Reload Saved Scene" enables the Collision Shape node and it can then be properly edited, reparented or deleted.
To me it seems like a bug with the newly generated Collision Shape inheriting some non-editable property from the source sibling, which only gets cleared when the scene is being reloaded.

Can confirm, in Godot 3.4 this issue is more critical, since the created CollisionShape will be vanished after reloading the scene.

Also present in Godot 4.


Update to re_rav1d 0.1.3

What

Fixes https://github.com/rerun-io/rerun/issues/7735

Tested on @jleibs windows laptop.
Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[x] I have tested the web demo (if applicable):

Using examples from latest main build: rerun.io/viewer
Using full set of examples from nightly build: rerun.io/viewer

[x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG
[x] If applicable, add a new check to the release checklist!
[x] If have noted any breaking changes to the log API in CHANGELOG.md and the migration guide

PR Build Summary
Recent benchmark results
Wasm size tracking

To run all checks from main, comment on the PR with @rerun-bot full-check.

@rerun-bot full-check

might need to merge in main?


feat(client): add resolve timeout to HttpConnector

The recently-added connect timeout does not cover resolving hostnames, which
could also stall on an OS-level timeout if there are issues reaching the DNS
server.
As mentioned in #1972.

Ouch, serious rebase needed! I've done most of it but the pin stuff has me stumped. I'll try a bit longer.

Sorry for missing this! Though this reminds me, once Resolve is just a Service (#1903), a user could set a timeout on it individually:
use ServiceExt;

let resolver = GaiResolver::new().timeout(some_dur);
let connector = HttpConnector::with_resolver(resolver);

Ah cool, I was vaguely aware that some kind of custom resolver work was going on but I didn't realise you could do that.
I'll close this then but would you accept a PR to add my timeout to 0.12? I take it with 0.13 near that you won't be backporting the Service feature and we're not quite ready for 0.13 yet.
Sent from my Android device with K-9 Mail. Please excuse my brevity.


Add Carthage support

Please add project file and share it to support carthage

I know Carthage is widely used and we will get to more people if we implement it. However, it is not on our roadmap in short term because we do not use carthage.
You are welcome to submit a PR. It would be a very nice addition :)

Already https://github.com/jobandtalent/AnimatedTextInput/pull/65

Wow! I was going through all the issues before looking at PRs.
Thank you!! I will try to find sometime soon to review it


The format of the input dat

@yctung @aravindGear4  What is the format of the input data when SVM training and predicting?Thanks?Is there an example?Such as  XX.txt ?

The input is in LibSVM format.
please refer https://stackoverflow.com/questions/40436694/libsvm-data-preparation-excel-data-to-libsvm-format

Sorry for the late reply, I was in a trip out of US. @senthilki  is right, the format is the LibSVM format


[improvement] Common styled default figures

Problem statement.
There is a few of quite nice styled figures for archimate concepts with a concept type icon in the upper right corner. Using such figures make a nice impression on some stakeholders (like speaciality experts, CAD/CAM/CAE engenering experts and so on) , enabling more credit level to the solution presented.
At the same time such concepts as:

Device
Artifact

and more have either no such styled concept images, or an option to set a default figure at all.
Due this using views with figures having no common style for some stakeholders, creating a concern from them such as:
"A tool or a method being used could be a raw toy, and not applicable for design purposes".
By this credit level to design descisions and an architect overall could be impacted.
Concurrent issue
As I could see Archi development team has a ton of tasks to do, so creating common styled figures is not a priority.
Visible solution options

Add an  option to  "Set custom default figure" for every archimate concept to enable end user make their diagramm style by themselves.
Create a common style figure for every concept figure by a community member's pullrequest.
Add an interface options to customize figure per view (seems like it's already in progress looking for the roadpam)
Maybe other solutions (like social i think).

Best regards, Alex.

This is not practical at the moment because all the Figures and icons are created in code with a Draw2D Figure, and Edit Part and other supporting classes. See, for example, https://github.com/archimatetool/archi/tree/master/com.archimatetool.editor/src/com/archimatetool/editor/diagram/figures/elements
Our longer term proposal (on the roadmap) is to create a number of different figure types that can be used for each element together with something like FontAwesome for icons (Icons cannot be bitmaps because of scaling. Also, SVG is not possible.)

I'll try to make a shortterm workaround, then to add some figures that are missing.


Update README.md

README for Dart provides a solid overview of the language's purpose and how to get involved. However, there are a few areas where it could be improved or expanded to make it even more informative and user-friendly. For example, many developers looking at a README want to jump right into coding. Adding a "Quick Start" section with installation instructions and a simple "Hello World" example could make it more approachable. It would also be helpful to mention specific use cases for Dart beyond Flutter (e.g., backend development, CLI tools, web apps).The contribution section could be expanded with more detailed steps or a dedicated "How to Contribute" link.

Thanks for your contribution! Please replace this text with a description of what this PR is changing or adding and why, list any relevant issues, and review the contribution guidelines below.

[ ] I’ve reviewed the contributor guide and applied the relevant portions to this PR.

Contribution guidelines:

See our contributor guide for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Contributions to our repos should follow the Dart style guide and use dart format.

Note that this repository uses Gerrit for code reviews. Your pull request will be automatically converted into a Gerrit CL and a link to the CL written into this PR. The review will happen on Gerrit but you can also push additional commits to this PR to update the code review.

Please no LLM generated nonsense. Thanks.


[bug] can not disable webview2 mouse gesture use additionalBrowserArgs

Describe the bug
Microsoft updated webview2 to version 116.0.1938.54 that enable right mouse gesture featrue by default. here is the issue about this: https://github.com/MicrosoftEdge/WebView2Feedback/issues/3737.
According to that issue, it can be disabled by additional browser arguments:
--disable-features=msEdgeMouseGestureSupported,msEdgeMouseGestureDefaultEnabled --enable-features=kEdgeMouseGestureDisabledInCN.
I add these args to additionalBrowserArgs,but not work.

Reproduction
No response
Expected behavior
No response
Platform and versions
[✔] Environment
    - OS: Windows 10.0.22621 X64
    ✔ WebView2: 116.0.1938.54
    ✔ MSVC: Visual Studio Community 2022
    ✔ rustc: 1.68.2 (9eb3afe9e 2023-03-27)
    ✔ Cargo: 1.68.2 (6feb7c9cf 2023-03-26)
    ✔ rustup: 1.25.2 (17db695f1 2023-02-01)
    ✔ Rust toolchain: stable-x86_64-pc-windows-msvc (default)
    - node: 18.15.0
    - yarn: 1.22.19
    - npm: 9.5.0

[-] Packages
    - tauri [RUST]: 1.4.1
    - tauri-build [RUST]: 1.4.0
    - wry [RUST]: 0.24.3
    - tao [RUST]: 0.16.2
    - @tauri-apps/api [NPM]: 1.4.0
    - @tauri-apps/cli [NPM]: 1.4.0

[-] App
    - build-type: bundle
    - CSP: unset
    - distDir: ../dist
    - devPath: http://localhost:1421/
    - framework: Vue.js
    - bundler: Vite

Stack trace
No response
Additional context
No response

It looks like it will be disabled by default next week, so there is nothing we can do. Also if the flag doesn't work when provided in additionalBrowserArgs it is not our issue, so maybe report it to WebView2.

Yes, these args can not work on Edge browser too.
I have reported it to WebView2.


Support domain resolution to obtain the nameserver address

Currently, the address of the NameServer is resolved in the form of splitting ";".
In the k8s environment, we can deploy it in single deployment.
We need to support domain resolution to obtain the NameServer address.
目前对 NameServer 的地址是以分割 ";" 的形式进行解析的，在 k8s 环境中，由于 NameServer 水平对等无状态，我们可以采用 deployment 的形式部署，需要支持域名解析来获取地址。

This feature is pretty tricky... Java has a lot of related bugs, which may impact this feature negatively
To name a few
https://bugs.java.com/bugdatabase/view_bug.do?bug_id=6247501
https://bugs.java.com/bugdatabase/view_bug.do?bug_id=8274692

I make this feature optional and off by default, and emphasize the points needing attention in the config. Users need to understand its side effects when they enable a feature-gate like k8s.

@RongtongJin @odbozhou @dongeforever Do you have a better idea?

Though this is a very commonly used technique, shared by many popular products, say grpc, envoy. This feature is pretty tricky for Java applications because Java has a lot of related bugs, which may impact this feature negatively if JDK is not carefully chosen To name a few https://bugs.java.com/bugdatabase/view_bug.do?bug_id=6247501 https://bugs.java.com/bugdatabase/view_bug.do?bug_id=8274692

How did grpc-java solve this kind of problem?

How did grpc-java solve this kind of problem?

grpc use InetAddress.getAllByName(host) to get list


Add more detail to engine documentation

It previously wasn't clear why you would make something an engine, as this project reimplements part of Rails::Engine's functionality anyway.
The details were stolen from @ngan 's response in https://github.com/rubyatscale/packs-rails/discussions/14.
This still seems a little bit like bending over backwards to me; why isn't every pack an engine by default? Namespace isolation is just one of many engine features and could be off by default.

why isn't every pack an engine by default?

I like this question. Perhaps by default each pack should be an engine which is a great fit for a new code base. For an existing monolith, it's nice to do incremental modularization, so overwriting the default would be an option.

Hey guys, when I first wrote stimpack (now just packs) I added the option to make packs engines so that we were backwards compatible with our existing in-repo engines (aka components). I actually didn’t like that I had to do this which is why I didnt document the feature.
I think the reasoning was that engines are more powerful than what we actually needed (or even wanted). The hidden/metaprogrammed engine class was a bit weird to me as well, and didn’t work well with Sorbet. I actually wanted to remove the engine feature at some point.
Anywho, I’m not sure what my stance is nowadays because I haven’t thought much more about it. We are actively trying to remove our components at Gusto and move them into packs, which would make them less capable…but to me that’s a feature and not a bug.


Deploy the latest changes to heroku

https://mtaa-api.herokuapp.com/api/tanzania/Mbeya still raises internal server error
Can you please deploy the latest changes to heroku ?

I did, as soon as I merged the changes, I deployed too. So that's probably a bug still in the code.

Guess I have it works on my machine issue (Let me have a look )
What might be causing this ?

I'm not sure myself.
I'll take some time and check the whole thing out again.

Hi @AnoRebel, just tried it works just fine
Thank you


Workflow removed removing workflow instances newer than passed date

Required Information

[ ] AEM Version, including Service Packs, Cumulative Fix Packs, etc: AEM 6.3 / AEM 6.4__
[ ] ACS AEM Commons Version: 3.17_
[ ] Reproducible on Latest? yes

### Expected Behavior
Workflow remover should respect the 'Workflows older than' argument when deleting running workflow instances.
### Actual Behavior
Workflow instances newer than the passed date appear to get deleted.
### Steps to Reproduce

Start clean AEM 6.3 / 6.4
Install latest ACS Commons
Start a couple of workflows that will stay in running state - ie a couple of Request For Activation worklfows (against any pages or assets).
Confirm that the workflows are visible and showing as running in the workflow console
Open workflow remover and select to remove:
** Status: Running
** Workflows older than: any date before the date the workflows started above (ie 72 hours ago
Click 'Remove Workflows'
Refresh workflow console and notice that the workflows started above were removed, despite apparently not meeting the passed criteria.

Links
n/a

I, also get this bug.
I guess the problem is here
https://github.com/Adobe-Consulting-Services/acs-aem-commons/blob/acs-aem-commons-3.19.0/bundle/src/main/java/com/adobe/acs/commons/workflow/bulk/removal/impl/WorkflowInstanceRemoverImpl.java****
line 279
final Calendar startTime = properties.get(PN_STARTED_AT, Calendar.class);
PN_STARTED_AT has to be startTime

This should be fixed via: https://github.com/Adobe-Consulting-Services/acs-aem-commons/pull/1793


ubios-cert.sh setdefaultca requires initial first

Hello world,
In the course of troubleshooting #2 I discovered that if you want to specify a non-default CA,
you have to run sh /mnt/data/ubios-cert/ubios-cert.sh initial
before you can run sh /mnt/data/ubios-cert/ubios-cert.sh setdefaultca
Otherwise, it will error out thusly:
Error: error checking path "/mnt/data/ubios-cert/acme.sh": stat /mnt/data/ubios-cert/acme.sh: no such file or directory
I think it errors out like this because the setdefaultca switch doesn't install acme.sh, but requires acme.sh to be present so it can set a different value for DEFAULT_CA.
In the course of un/re-installing this script several times, I found that I had to

first run sh /mnt/data/ubios-cert/ubios-cert.sh initial
let it fail
then run sh /mnt/data/ubios-cert/ubios-cert.sh setdefaultca
then re-run sh /mnt/data/ubios-cert/ubios-cert.sh initial

to get it to utilize the DEFAULT_CA value from my /mnt/data/ubios-cert/ubios-cert.env file.
Not a big deal, but I thought I'd report it here. I'm not very good at scripting, but I wonder if the ubios-cert.sh script could check to see if acme.sh is installed first before proceeding. Seems like it would be good for the initial, setdefaultca, and cleanup switches to know about ahead of time.

Thanks for the report! Will have a look at it if I can include your solution in the script directly.

I just uninstalled and reinstalled everything.
I can confirm that these changes worked.
I was able to run sh /mnt/data/ubios-cert/ubios-cert.sh setdefaultca before running it with the initial switch without any issues.
Thanks for being so quick to fix this.

Thanks for your excellent debugging!
I stick with Let's Encrypt for the moment, all other providers might require additional configuration.


web api is not calling from jquery but it is called from postman

iam tying to call web api  by  using the juery ajax method but it was not showing anything . please let me how can i resolve the problem

Hi @Diwakarreddy15,
Can you provide some sample code as well as the versions of PhoneGap and the Developer App that you are using?

$.ajax({
url:'http://astro.preprod.co.in/api/values',
type:'GET',
data: empdeatails,
dataType: 'jsonp',
crossDomain: true,
success: function (data) {
console.log(data.msg);
},
error: function (xhr) {
alert(xhr.responseText);
}
});
iam using phonegap cli 5.3.6 version and am using developer app to test the felilds

I am able to run the jQuery Ajax method on my machine. Are you making sure to include jQuery within index.html @Diwakarreddy15

ya i have included jquery in index.html

can you provide some sample code to test my application @surajpindoria

$.ajax({
    url: 'http://astro.preprod.co.in/api/values',
    type: 'GET',
    dataType: 'json',
    success: function(result) {
        console.log(result);
    }
});

I found that I was having problems when trying to set dataType: 'jsonp', the jQuery docs talk a little bit about issues with jsonp and cross domain requests. Maybe this is what you are seeing @Diwakarreddy15


Bug

Bug Report
Current Behavior A clear and concise description of the behavior.
Input Code

REPL or Repo link if applicable:

your_code = "here"

Expected behavior/code A clear and concise description of what you expected to
happen (or code).
Environment

Python(s): [e.g. python3.8, python3.9, ...]

Possible Solution

Additional context/Screenshots Add any other context about the problem here. If
applicable, add screenshots to help explain.

hello

hi


[HW] Akash Sangami

Name: Akash Sangami
Email<EMAIL_ADDRESS>Linkedin Profile: https://www.linkedin.com/in/akash-sangami-b4bba580/
Attach the homework step I screenshots below:

Attach the homework step II screenshots below:

Hey Akash Sangami, great job! Congrats! Here is your badge! https://api.badgr.io/public/assertions/xPh-pV3iQnGQWuqMeoCduw


configure tinymce to restrict to styles we want to use on the site

There is some mezzanine documentation here http://mezzanine.jupo.org/docs/admin-customization.html#wysiwyg-editor
We can override with our own tinymce setup javascript; here's the default: https://github.com/stephenmcd/mezzanine/blob/e79fde8faa8817a95b7c0756ec9de7170f30c20b/mezzanine/core/static/mezzanine/js/tinymce_setup.js
And tinymce documentation https://www.tinymce.com/docs/configure/
I think we probably want to restrict the number of headers we allow; hoping we can add block elements/styles for the sidebar and name formatting on the home page.  We'll also need a block format for the notes in the zeplin mockup.  If possible/easy, we should revise the default image template so it actually generates what we want (no width/height; maybe figure with figcaption?).

I know I didn't mention it or call it out specifically, but please revise this to handle the sidebar formatting on the homepage, so that it can be edited by project team members without messing up the layout.
I'm hoping for a simple solution here - if it looks complicated, let's figure out a different way to handle it.

This looks good. Homepage formatting will be handled separately as part of #159


vscode intellisense for node_modules packages is truncated at deep level 2 in nodejs project

I have a javascript project with around 7 levels of depth de module.exports object.. something like
levels.l1.l2.l3.l4.l5.l6.l7 = 'some value'

When working in that codebase the intellisense works like a charm, but when installing the dependency from another project with npm the intellisense gets truncated at level 2. Something like
levels.l1.l2. ???

I found out about jsconfig.jsonfile at the root of the project with a compilerOptions.maxNodeModuleJSDepth with no luck.
this is an example of the file I am using
{
  "compilerOptions": {
    "maxNodeModuleJsDepth": 10
  },
  "include": [
      "node_modules/my-deep-package"
  ]
}

Is there something that I can do to get that intellisense for the package when is in the node_modules?
Thanks

@tomasruizr or @codycuellar Do you have a small example project you could share that demonstrates this issue?

Create a fresh NPM project, then extract this .tgz and npm link it, then npm link intellisense-jsdoc in your other new project. Everything should work correctly. Then from your new project, npm install /path/to/intellisense-jsdoc-1.0.0.tar.gz from the other project and it should stop working.
intellisense-jsdoc-1.0.0.tar.gz

We haven't seen any other reports of this - either this is fixed, or very very niche. Please open a new standalone issue if you're seeing this on recent builds. Thanks!


selectOnFocus does not work with autoFocus

Description
When using selectOnFocus with autoFocus, text is not being selected automatically on first mount.
Steps to reproduce

Open Expo Snack link - https://snack.expo.dev/@alexshukel/selectonfocus-bug
Text in TextInput is not selected automatically on mount

React Native Version
0.73.6
Affected Platforms
Runtime - Android
Output of npx react-native info
System:
  OS: Windows 11 10.0.22631
  CPU: (12) x64 Intel(R) Core(TM) i7-10850H CPU @ 2.70GHz
  Memory: 13.95 GB / 31.61 GB
Binaries:
  Node:
    version: 20.11.0
    path: C:\Program Files\nodejs\node.EXE
  Yarn:
    version: 1.22.21
    path: ~\AppData\Local\pnpm\yarn.CMD
  npm:
    version: 10.2.4
    path: C:\Program Files\nodejs\npm.CMD
  Watchman:
    version: 20231008.002904.0
    path: C:\soft\watchman-v2<IP_ADDRESS>-windows\bin\watchman.EXE
SDKs:
  Android SDK: Not Found
  Windows SDK: Not Found
IDEs:
  Android Studio: Not Found
  Visual Studio:
    - 17.8.34330.188 (Visual Studio Community 2022)
Languages:
  Java: javac 11
  Ruby: Not Found
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.73.6
    wanted: 0.73.6
  react-native-windows: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: Not found
  newArchEnabled: Not found

Stacktrace or Logs
Not seeing selected text in the input.

Reproducer
https://snack.expo.dev/@alexshukel/selectonfocus-bug
Screenshots and Videos
No response

Here i'd like to be assigned to this issue


consul: more efficient tag and service filtering

This is related to:

https://github.com/hashicorp/consul/pull/2549
https://github.com/prometheus/prometheus/pull/2028
https://github.com/prometheus/prometheus/issues/2285
https://github.com/prometheus/prometheus/pull/2066
https://github.com/prometheus/prometheus/issues/3452
https://github.com/prometheus/prometheus/issues/771
And some others

The official answer so far as been "use relabeling", but doing that ends up opening one TCP connection per service (it looks like all the services are being watched). This happens because if you don't specify a service name, and drop using relabeling shouldWatch()  will always return True and all services will be watched (which is currently very inefficient with consul, and clearly non-necessary).
According to https://www.consul.io/api/catalog.html#list-services, we still can only filter on dc and node-service. As discussed in https://github.com/hashicorp/consul/pull/2549 they would like to also add service-meta here, but that hasn't been done (and that was almost a year ago).
Currently the discovery/consul/consul.go simply watches the Catalog without any restrictions, but Services() returns both the service names and tags, so we can start to do some filtering here based on the tags (example in https://github.com/hashicorp/consul/blob/b3292d13fb8bbc8b14b2a1e2bbae29c6e105b8f4/command/catalog/list/services/catalog_list_services.go#L85)
catalog.Service() can take an also optional tag argument that could be used to do some filtering server-side, so we can use that too to get notified only when necessary.
I (or somebody else from my team) will try to get a PR ready next week as it is quite annoying to have thousands of connections for nothing, and relabeling isn't that obvious to do this.

For future reference, since one of my patch will include the ability to do stale reads, here is the impact on our cluster of not using stale reads (prometheus being one of the top issuer of non-stale read queries):

link


SSH Key pairs links should use the unique ID instead of the name

ISSUE TYPE

Bug Report

COMPONENT NAME

UI - ssh key pairs

CLOUDSTACK VERSION

4.17.0

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

I added my ssh key pair on multiple accounts / domains. They have the same name which causes the ui to only pick te first.
STEPS TO REPRODUCE

Add the same key under multiple accounts and try to click the last one. It will open the first entry.

EXPECTED RESULTS

I think the ui should use the ID instead of the name.

ACTUAL RESULTS

As far as I know, the return result of listSSHKeyPairs has 2 types: with id and without id. This makes the conversion to only id impossible. What do you think? @shwstppr cc @rohityadavcloud @Pearl1594

UUID for SSHKeyPair was added only in 4.16.0. https://github.com/apache/cloudstack/pull/5103/files#diff-57c8dd809042b11cd4a14741e2f797e968f23a9cc7bf7b875864397f32d68582R32-R35
Keypairs added earlier than that do not have UUID. This is why they are listed using the name.
@utchoang we may do something like the following but this would solve it only for keypair list. This won't work infocard as resource.keypair will only have the name
diff --git a/ui/src/components/view/ListView.vue b/ui/src/components/view/ListView.vue
index 292460541a..981152fe21 100644
--- a/ui/src/components/view/ListView.vue
+++ b/ui/src/components/view/ListView.vue
@@ -98,7 +98,7 @@

-
+
             <router-link :to="{ path: $route.path + '/' + record.id }">{{ text }}</router-link>
             <router-link :to="{ path: $route.path + '/' + record.id, query: { tab: 'comments' } }"><message-filled style="padding-left: 10px" size="small"/></router-link>

@@ -110,7 +110,7 @@
           <router-link :to="{ path: $route.path + '/' + record.name }" v-else>{{ $t(text.toLowerCase()) }}</router-link>

-          <router-link :to="{ path: $route.path + '/' + record.id }" v-if="record.id && $route.path !== '/ssh'">{{ text }}</router-link>
+          <router-link :to="{ path: $route.path + '/' + record.id }" v-if="record.id">{{ text }}</router-link>
           <router-link :to="{ path: $route.path + '/' + record.name }" v-else>{{ text }}</router-link>

diff --git a/ui/src/views/AutogenView.vue b/ui/src/views/AutogenView.vue
index ff75a6d269..e8397c0b2d 100644
--- a/ui/src/views/AutogenView.vue
+++ b/ui/src/views/AutogenView.vue
@@ -851,8 +851,11 @@ export default {
       if (this.$route.params && this.$route.params.id) {
         params.id = this.$route.params.id
         if (['listSSHKeyPairs'].includes(this.apiName)) {
-          delete params.id
-          params.name = this.$route.params.id
+          const regexExp = /^[0-9a-fA-F]{8}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{4}\b-[0-9a-fA-F]{12}$/gi
+          if (!regexExp.test(params.id)) {
+            delete params.id
+            params.name = this.$route.params.id
+          }
         }
         if (['listPublicIpAddresses'].includes(this.apiName)) {
           params.allocatedonly = false

@shwstppr should we raise a PR? For example, when name is missing use uuid in the list, and allow users to update/rename their keys?

@rohityadavcloud it won't be simple. It is not the name that is missing. SSH keypairs added before 4.16 don't have UUIDs and many API responses have the parameter keypair which returns the name of the keypair. As I mentioned, we can fix it in keypair list view in UI but when a keypair is navigated from the info card of another resource it may still use the name.

@bugoff issue you face should be fixed by #6616 if the keys were added after 4.16.0
Please reopen if you still face the issue


Wisdom

Link	comp4711/wisdom
Location	Lower right author image
Expected result	Isaac Asimov's quote
Routing Strategy	Regular expression routing
Required	Routing rule, leading to wise/bingo
How to Fix It	Remapped using regular expression (comp + digits / *)
Hint	application/controllers/Wise::bingo()

closes #8


Python compatibility clarifications

Python 3.5.2 turned out to be unsupported because there was an issue in the static typing library: https://github.com/python/typing/pull/308. The issue has been fixed in Python 3.5.3, so all newer versions function correctly. This is how you can reproduce the problem:
>>> import typing
>>> T = typing.TypeVar('T')
>>> G = typing.Callable[[], T]
>>> G[int]
typing.Callable[[], int]

The above is the expected behavior observable in Python 3.5.3+. In 3.5.2 you get this instead:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/usr/lib/python3.5/typing.py", line 815, in __getitem__
    raise TypeError("This Callable type is already parameterized.")
TypeError: This Callable type is already parameterized.

An alternative solution would be to remove the offending type annotations, which you can look at here:
https://github.com/UAVCAN/pydsdl/blob/9a0bd72b4fd298e96c57f77e3c9a48b6fe1ee70a/pydsdl/_expression.py#L563-L565
However, I would rather not do that because that would require us to plaster the code with linter suppression comments and doing so for the sake of an ancient version of the interpreter (which will be EOLed next year) doesn't seem particularly sensible.
Should we accept this change, or should we add workarounds instead and retain compatibility with all v3.5.x?

Pull Request Test Coverage Report for Build 232

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 225:
0.0%

Covered Lines:
2658

Relevant Lines:
2658

💛  - Coveralls

Sorry I didn't comment on this sooner. I'm okay with this stipulation. Nunavut uses python 3.5.7 as it's minimal support target (i.e. our python tox container uses 3.5.7).
Have you considered enabling tox for this project?
We should also discuss standardizing the UAVCAN python tools since nunavut uses a different set of them than pydsdl. This is fine for now but at some point we'll tire of supporting so many external dependencies.


Can't creat custom domain

Well, i had installed on my own IP and it work perfectly, but i want to creat custom domain point to it.
I had config the domain conf ( i use NGINX) like that:

server {
listen 80;

server_name play.anonymousvn.org;
root /home/play.anonymousvn.org/public_html;

location / {
    proxy_pass http://<IP_ADDRESS>:8080;
    proxy_http_version 1.1;
    proxy_set_header Upgrade $http_upgrade;
    proxy_set_header Connection 'upgrade';
    proxy_set_header Host $host;
    proxy_cache_bypass $http_upgrade;
}

}
and then t start the server with npm start, start the http-server. And then go to domain play.anonymousvn.org
It had shown the client look like, but when i click to visit chat rom, it didnt ask me for my username,
seem server.js didnt working, but when i go to ip <IP_ADDRESS>:8080 it still working perfectly.
Please help me

try to add this to your nginx.conf
server {
        listen 6060;
        location / {
            proxy_pass http://<IP_ADDRESS>:6060;
            proxy_http_version 1.1;
            proxy_set_header Upgrade $http_upgrade;
            proxy_set_header Connection $connection_upgrade;
        }
}

Can u help me via teamview?

the server port is 6060 and the client port is 8080 yesterday, but today it changed to 8081

Yesterday i typed npm start is okay, but today it get :

[root@vultr public_html]# npm start

<EMAIL_ADDRESS>start /home/play.anonymousvn.org/public_html
node server.js

Loaded config 'config.json'
Started server on <IP_ADDRESS>:6060
events.js:160
throw er; // Unhandled 'error' event
^
Error: listen EADDRINUSE <IP_ADDRESS>:6060
at Object.exports._errnoException (util.js:1012:11)
at exports._exceptionWithHostPort (util.js:1035:20)
at Server._listen2 (net.js:1252:14)
at listen (net.js:1288:10)
at net.js:1398:9
at _combinedTickCallback (internal/process/next_tick.js:77:11)
at process._tickCallback (internal/process/next_tick.js:98:9)
at Module.runMain (module.js:577:11)
at run (bootstrap_node.js:352:7)
at startup (bootstrap_node.js:144:9)
npm ERR! Linux 3.10.0-327.28.2.el7.x86_64
npm ERR! argv "/root/.bin/nodejs/bin/node" "/root/.bin/nodejs/bin/npm" "start"
npm ERR! node v6.3.1
npm ERR! npm  v3.10.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>start: node server.js
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script 'node server.js'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the hack.chat package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node server.js
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs hack.chat
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls hack.chat
npm ERR! There is likely additional logging output above.
npm ERR! Please include the following file with any support request:
npm ERR!     /home/play.anonymousvn.org/public_html/npm-debug.log
i still can go to play.anonymousvn.org but cant go to <IP_ADDRESS>:8080

Error: listen EADDRINUSE <IP_ADDRESS>:6060 means that the port is being used by another process.
So you'll have to either change the 6060 port to something else, or find out what process is using the port and terminate it.
( if you can't figure out what process is using it, restarting the server works :) )

Chances are, you've launched two instances of the server, and the second one is failing because it can't bind to port 6060 since the other one has taken it.


Attaching component with an application

As-salamu-alikum
Zazakallah khair for sowing interest to my issue.
I am new to the marionette as well as marionette.toolkit so my issue may be very stupid. Please pardon me if my issue irritate you .
Marionette.toolkit component idea is excellent. Is it possible to attach a component to an application anyway instead of a  view directly which we do by calling showView()?
Zazakallah khair for reading with passion.
Allah hafiz
Nadvi

var region = myApp.getRegion();
myComponent.showIn(region);

Zazakallah khair for answering my issue.
Allah-hafiz
Nadvi