text:
[WIP] Add support for v1alpha2 route rules

What this PR does / why we need it: Adds support for v1alpha2 route rules being discussed here: https://github.com/istio/api/pull/254
Which issue this PR fixes: contributes to https://github.com/istio/istio/issues/1925
Special notes for your reviewer: this is a work in progress PR. Right now we are looking for feedback on the general approach. We may break this PR into smaller pieces if necessary.
Release note:

Added support for v1alpha2 route rules.

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
f1d17b5e4a314e4611c2c6bf23a53d3b2c917475
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
9250c31145f53864fd5566aa1d5dfea2fc16a7e6
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
5d61b70891fd80052acaeaecd9b74d320f9eda28
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
74f55c0ac700116972b42e339ce1cd5435898bb8
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
370799966153127277292dbf03198356624e56e4
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
1207cea5494b5296f523fdec80f2234ba3bd3db9
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
168a13c40397e281c87cd409129ab483d47cd4c2
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
1b217d5fae11bba5b492821de5572c5cc1578eaf
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
00a5a8a09a90d4f30f2f2c9b7fa2217022f67cb0
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
757a9407ac3f29681c6aa19a4269dc60e14652e9
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
b049ff72d77f8c474081383cdda30d006a427081
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
1d396d9b5719b55950c9e536eb6f7b1181f9aac2
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
d128d37b557f0f6d344f1fdeb1cd2a6f09a1878c
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
2a949c5f118681df28ebff5b86521c3792ee4f28
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
b2b700de125bfb43e525ccc653ece3a5f3d3de6e
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
b011c73ffaa415bf7f4e17c3cccec1e1645fab59
link
/test istio-presubmit

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

@ijsnellf: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-presubmit.sh
ef0f11daf4977134f56c028058f7874eb24d5a79
link
/test istio-presubmit

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

authors: ijsnellf istio-testing


text:
Automator: update istio/api@release-1.5 dependency in istio/istio@release-1.5

Generated by Automator - 2020-02-17T10:24:45+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio_release-1.5
a92cf7316c4f06480cc5e82105046279a4a3e154
link
/test integ-pilot-k8s-tests_istio_release-1.5

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

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-k8s-tests_istio_release-1.5
a92cf7316c4f06480cc5e82105046279a4a3e154
link
/test integ-pilot-k8s-tests_istio_release-1.5

integ-security-k8s-tests_istio_release-1.5
a92cf7316c4f06480cc5e82105046279a4a3e154
link
/test integ-security-k8s-tests_istio_release-1.5

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

/retest

source: gharchive/pull-request

authors: dgn istio-testing

repo: istio/istio

license_type: permissive

license_source: github-api


text:
authz: add e2e tests for conditions

This PR adds e2e tests for all the conditions (e.g. source.ip, destination.port, connection.sni, etc.) in the authorization policy.

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

Test name
Commit
Details
Rerun command

lint_istio
7670eba64b6618e20729a50e6084fbf29ed3c660
link
/test lint_istio

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

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

Test name
Commit
Details
Rerun command

lint_istio
7670eba64b6618e20729a50e6084fbf29ed3c660
link
/test lint_istio

gencheck_istio
7670eba64b6618e20729a50e6084fbf29ed3c660
link
/test gencheck_istio

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

In response to a cherrypick label: new pull request created: #21444

authors: istio-testing yangminzhu


text:
istioctl verify-install: Colorize success/failure markers

Resolves https://github.com/istio/istio/issues/29336
The only output change is to colorize the little ✔ and ✘ markers.
Uses the same technique as istioctl analyze to see if it is safe to colorize.
I didn't test this on Windows.  Windows has some non-ANSI terminals.  Our progress bar uses github.com/mattn/go-colorable for those Windows terminals.  I didn't bring that it, because I don't have a Windows machine to test on and invoking it directly outside of a progress bar would mean changing go.mod.

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

Test name
Commit
Details
Rerun command

release-notes_istio
5225e6aaafacf2878fa07fadde2c078cb426c12a
link
/test release-notes_istio

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

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

Test name
Commit
Details
Rerun command

release-notes_istio
5225e6aaafacf2878fa07fadde2c078cb426c12a
link
/test release-notes_istio

unit-tests_istio
5225e6aaafacf2878fa07fadde2c078cb426c12a
link
/test unit-tests_istio

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

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

Test name
Commit
Details
Rerun command

release-notes_istio
5225e6aaafacf2878fa07fadde2c078cb426c12a
link
/test release-notes_istio

unit-tests_istio
5225e6aaafacf2878fa07fadde2c078cb426c12a
link
/test unit-tests_istio

integ-ipv6-k8s-tests_istio
5225e6aaafacf2878fa07fadde2c078cb426c12a
link
/test integ-ipv6-k8s-tests_istio

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

/retest

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

Test name
Commit
Details
Rerun command

unit-tests_istio
5225e6aaafacf2878fa07fadde2c078cb426c12a
link
/test unit-tests_istio

integ-ipv6-k8s-tests_istio
5225e6aaafacf2878fa07fadde2c078cb426c12a
link
/test integ-ipv6-k8s-tests_istio

release-notes_istio
5225e6aaafacf2878fa07fadde2c078cb426c12a
link
/test release-notes_istio

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

authors: esnible istio-testing


text:
Automator: update istio/client-go@release-1.12 dependency in istio/istio@release-1.12

Generated by Automator - 2021-11-15T19:56:54+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-security-multicluster-tests_istio_release-1.12
e621765d4d2e803ab33235f925540813271c07e9
link
true
/test integ-security-multicluster-tests_istio_release-1.12

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

/test integ-security-multicluster-tests_istio_release-1.12

authors: ericvn istio-testing


text:
docs: work with loki

Please provide a description of this PR:
Introduce a simple doc about how to make OpenTelmetry and Loki work together.
Loki use AGPL license, I'm not sure it's allowed.
This will be helpful for complex e2e tests, e.g. https://github.com/istio/istio.io/issues/12797.

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
10d1f456f70e2e9cb463f188981eb22cbeeaf9d3
link
true
/test lint

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

@howardjohn @lei-tang PTAL

kindly ping @howardjohn

authors: istio-testing zirain


text:
monitoring: migrate from OpenCensus to OpenTelemetry

~Blocked by https://github.com/open-telemetry/opentelemetry-go/pull/4306. I have a hold until that is ready; should be fairly quick~
For https://github.com/istio/istio/issues/44743
Fixes https://github.com/istio/istio/issues/44513
This PR migrates from OpenCensus to OpenTelemetry for Go metrics. This
doesn't impact proxy metrics, or non-metrics observability.
These changes are intended to be 100% transparent to end users.
The changes are mostly transparent from pkg/monitoring public
interface, with a few changes:

Register is no longer needed. Creating the metric registers it. This
makes no real difference in any of our usage
MustCreateLabel -> CreateLabel; it is infallable
No need to define the set of keys upfront with the metric.
RegisterIf -> WithEnabled
monitortest behaves slightly different; only metrics recorded after
its creation are counted. A few tests are updated to handle this
change.
Only one exporter can be defined; this only impacts tests, which are
updated to handle this.

There are a number of changes, of course, to the internals. Most of this
is a fairly mechanical translation. A few quirks:

There is no real Gauge like in OC. Instead, we can only do a callback.
So we need to handle a bit of this on our end; not a huge deal, and
looks like it will be added eventually.
Setting histogram buckets is a bit awkward currently. This will be
addressed eventually upstream.
monitortest previously was based on OC internals. This has been re-written to prometheus instead of OTEL. This is because prometheus is our primary usage, so its a bit more realistic.

~This results in a 1% increase in the binary size, which is due to importing both OC and Otel. This is addressed in https://github.com/istio/istio/pull/46010, which introduces a 1% decrease in binary size when combined with this PR.~

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

FYI: I test with opentelemetry-go offline, it's will add _total suffix fro counter, which maybe a regression
https://github.com/open-telemetry/opentelemetry-go/blob/4d473d105a5f59a58d153e7c492a4ea0ee027871/exporters/prometheus/exporter.go#L216

PR needs rebase.

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

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
a2e551c3eb25f7b0e38fe988c5e17cac8952a570
link
true
/test lint

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
a2e551c3eb25f7b0e38fe988c5e17cac8952a570
link
true
/test lint

unit-tests_istio
a2e551c3eb25f7b0e38fe988c5e17cac8952a570
link
true
/test unit-tests

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
a2e551c3eb25f7b0e38fe988c5e17cac8952a570
link
true
/test unit-tests

lint_istio
6ecac3bb4de7d4f88dffe72c534b3fce4f3d66af
link
true
/test lint

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

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
31fc60b32d87f0fcf8b5e324ba010ff9a1f713cd
link
true
/test lint

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

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
31fc60b32d87f0fcf8b5e324ba010ff9a1f713cd
link
true
/test lint

unit-tests_istio
31fc60b32d87f0fcf8b5e324ba010ff9a1f713cd
link
true
/test unit-tests

integ-ambient_istio
31fc60b32d87f0fcf8b5e324ba010ff9a1f713cd
link
true
/test integ-ambient

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

Looks very good.
One concern - not blocking this PR - is that I would prefer if labels are declared at metric creation, even if otel doesn't require it.

authors: costinm howardjohn istio-testing zirain


text:
fix ISTIO_DELTA_XDS in compatibility version 1.21 profile

Please provide a description of this PR:
Compatibility profile for v1.21 wouldn't disable delta XDS correctly.
proxyMetadata should be under meshConfig.defaultConfig in the meshConfig.
Current behavior:
      - name: PROXY_CONFIG
        value: |
          {"tracing":{"zipkin":{"address":"zipkin.istio-system:9411"}},"proxyMetadata":{"ISTIO_META_DNS_CAPTURE":"true"},"gatewayTopology":{"numTrustedProxies":2},"meshId":"my-mesh","holdApplicationUntilProxyStarts":true}

After patching:
      - name: PROXY_CONFIG
        value: |
          {"tracing":{"zipkin":{"address":"zipkin.istio-system:9411"}},"proxyMetadata":{"ISTIO_DELTA_XDS":"false","ISTIO_META_DNS_CAPTURE":"true"},"gatewayTopology":{"numTrustedProxies":2},"meshId":"my-mesh","holdApplicationUntilProxyStarts":true}

Hi @antoinedeschenes. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

/ok-to-test

@antoinedeschenes: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
9703c079d648458d8db15586efd7c9597a1e0169
link
true
/test release-notes

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

Should I fill a release note?

In this case its probably small enough to elide. Thanks for the fix!
/retest

/test integ-operator-controller_istio

@howardjohn: The specified target(s) for /test were not found.
The following commands are available to trigger required jobs:

/test bookinfo-build
/test gencheck
/test integ-ambient
/test integ-basic-arm64
/test integ-cni
/test integ-distroless
/test integ-ds
/test integ-helm
/test integ-ipv6
/test integ-pilot
/test integ-pilot-istiodremote
/test integ-pilot-istiodremote-mc
/test integ-pilot-multicluster
/test integ-security
/test integ-security-istiodremote
/test integ-security-multicluster
/test integ-telemetry
/test integ-telemetry-discovery
/test integ-telemetry-istiodremote
/test integ-telemetry-mc
/test lint
/test macbuildcheck
/test release-notes
/test release-test
/test unit-tests
/test unit-tests-arm64

The following commands are available to trigger optional jobs:

/test benchmark
/test experimental-tracing
/test integ-ambient-calico
/test integ-ambient-dual
/test integ-ambient-ipv6
/test integ-assertion

Use /test all to run the following jobs that were automatically triggered:

gencheck_istio
integ-ambient_istio
integ-basic-arm64_istio
integ-cni_istio
integ-distroless_istio
integ-ds_istio
integ-helm_istio
integ-ipv6_istio
integ-pilot-istiodremote-mc_istio
integ-pilot-istiodremote_istio
integ-pilot-multicluster_istio
integ-pilot_istio
integ-security-istiodremote_istio
integ-security-multicluster_istio
integ-security_istio
integ-telemetry-discovery_istio
integ-telemetry-istiodremote_istio
integ-telemetry-mc_istio
integ-telemetry_istio
lint_istio
macbuildcheck_istio
release-notes_istio
release-test_istio
unit-tests-arm64_istio
unit-tests_istio

In response to this:

/test integ-operator-controller_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

/cherry-pick release-1.23

/cherry-pick release-1.22

@dhawton: new pull request created: #52240

In response to this:

/cherry-pick release-1.23

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@dhawton: new pull request created: #52241

In response to this:

/cherry-pick release-1.22

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

authors: antoinedeschenes dhawton howardjohn istio-testing


text:
Add a development workflow for Mixer

Fixes (#1337)
This documentation adds a development workflow for Mixer.  With
this change, building custom Mixer binaries and testing them in
a local environment is now documented.
This is part of a larger set of changes to introduce a CentOS
implementation to Istio.  For that work, see:
https://github.com/istio/issues/issues/56
Release note:

This change is

Hi @sdake. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

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

/assign @geeknoid

This doesn't seem to be specific to mixer, can the generic part be integrated into
https://github.com/istio/istio/blob/master/devel/README.md rather than just here ?

lines 25-37 are unique to Mixer (PIlot has a different inconsistent build method).  That should probably be unified in a different PR.  The rest can go in the standard doc you mentioned.  I'll sort that out hopefully Thursday.
Suggestions for documenting lines 25-37?

thanks, how about putting mixer in front of it (and bonus, the current pilot way too)?
Note that istio/istio is already sort of mixer centric

Closing in favor of merging directly to https://github.com/istio/istio/issues/1060.

authors: istio-testing ldemailly sdake

repo: istio/mixer


text:
Stop checking quotas from the Check API method.

Checking quota must now be done using the Quota method.

This change is

Jenkins job mixer/presubmit passed

authors: geeknoid istio-testing


text:
DO NOT MERGE! Fast Forward stable to d3537aa388a287ab1658c4864d450b29fcea319f.

This PR will be merged automatically once checks are successful.

This change is

Jenkins job mixer/postsubmit passed


text:
sync chart.

Sync the chart from installer to cover some manifests diff found in run_mandiff.

@morvencao: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

operator-lint-dind
72eec914e10ddaee2e937264da08bee1d6a46464
link
/test operator-lint-dind

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

/retest

authors: istio-testing morvencao

repo: istio/operator

license_source: bigquery


text:
fix match rule discarded from ingress routes

This fixes #895
User issue: https://github.com/istio/issues/issues/38
This fix is made and tested against the release-0.1 branch. Corresponding fix needs to be made in master and the question is do we want to release this fix in a new alpha driver (0.1.x) or are we going to only fix it in 0.2.x? If we decide to put out another 0.1.x, we should also provide some kind of interim fix for the rbac issue that several people are complaining about (https://github.com/istio/istio/pull/333)

Jenkins job pilot/presubmit passed

Jenkins job pilot/presubmit passed

Jenkins job pilot/presubmit passed

authors: frankbu istio-testing

repo: istio/pilot


text:
Add a mechanism to propagate tunnel metadata to client/servers

When a user application accepts or initiates a connection, it should be able to somehow access metadata from ztunnel about the connection. At minimum, this would be the identity of the peer.
This can be used in applications to do app-specific logic based on identity, or even in a "sandwhich" case where we run ztunnel alongside Envoy

Is it something like metadata exchange?

So ztunnel would fetch metadata from control plane directly, then I do not understand the tunnel here,.

related to https://github.com/istio/istio/issues/42033

source: gharchive/issue

authors: howardjohn hzxuzhonghu linsun

repo: istio/ztunnel


text:
Update Live activity with Firebase cloud messaging FCM

The new FCM feature with live activity.
After setting the activityToken to the live_activity_token as present in the picture

The live activity didn't updated.
Note : the content-state contain keys that present in the use defaults
What should be included in the content-state & live_activity_token to make it work instead?

If we are talking about creating a LA:
The live_activity_token is the pushToStartTokenUpdates.
If I understand the code correctly @istornz added a section to monitor that token but its never transported to flutter.
The content state reflects the mutable state of the activity.

If we are talking bout updating a LA:
The live_activity_token is the token you received from the activityUpdateStream which provides the token after you called createActivity.
The content state is still the mutable content of your activity. In my case:
struct LiveActivitiesAppAttributes: ActivityAttributes, Identifiable {
    public typealias LiveDeliveryData = ContentState // don't forget to add this line, otherwise, live activity will not display it.

    public struct ContentState: Codable, Hashable {
        // make everything nullable to be able to retrieve initial
        // values before notification is sent to update
        let progress: Double?
        let eta: Int?
        let printState: String?
        let file: String?
    }

    var id = UUID()
}

This will be the progress, eta, printState and file properties.
I am using APNs directly, as FCM did not support LA while I developed my solution, I send a Map<String, String> with the field -> newValue  (e.g. progress : "1") mappings to the APNs

license: MIT


text:
Added changeable background.

dev
GitHub Project

Main GitHub Project ticket

Code reviewers

[ ] @mehalyna

Second Level Review

[ ] @PirogStanislav

Summary of issue
Need to change background picture every 5 sec.
Summary of change
Added a new package to change the background.
Testing approach
ToDo
CHECK LIST

[ ]  PR is reviewed manually again (to make sure you have 100% ready code)
[ ]  I've checked new feature as logged in and logged out user if needed
[ ]  PR meets all conventions

/AzurePipelines run dev-pipe

repo: ita-social-projects/EventsExpress


text:
[Change request] To hide the "Experts" section on the Main Page.

As a user I do not want to see the "Experts" section on the Main Page so that I will learn the authors information on the special page for this - Authors Page.
Description:
Main Page should contain:

"Important" section,
"Newest" section

Main Page should NOT contain:

"Experts" section.

Reason for Change:
Receiving design from client`s side.
Change request to US #13

It has been already implemented.

BA check 19/07/21
Not accepted.
There is no “Newest” section on the main page.

authors: EleonoraBudanova vitaliyhere

repo: ita-social-projects/dokazovi-be


text:
WebApp Registrazione Cancellazione per Decesso per Intestatario scheda solo

Data test 26/04/2017 - Ambiente test (dptest)
Effettuato cancellazione per decesso di soggetto intestatario scheda solo GRFMHL81E23L219F. Dopo aver completato l'operazione compare il messaggio "Operazione correttamente eseguita" e poi il warning "Attenzione:Aggiornare la famiglia del soggetto cancellato indicando un nuovo intestatario ed eventualmente i relativi rapporti di parentela (Codice: EN225)".
Siamo consapevoli che si tratti di un messaggio generico però almeno per i casi di intestatario scheda solo, non dovrebbe comparire.
Amprimo - Comune di Torino

Grazie della segnalazione. Apporteremo la modifica prima possibile

authors: annamariaGillone gcarbonin

repo: italia/anpr


text:
[webapp]: allegati del provvedimento di rifiuto

Ambiente
pre-produzione
Provenienza segnalazione
Software House
Riferimenti operatore/comune
Anna Maraspin, Sala Bolognese
Data del test (yyyy-mm-dd)
No response
Caso d'uso
No response
Descrivi il problema
Formato atto di nascita con genitori residenti in un altro Comune
QUesto Comune rifiuta l'atto e ci manda il provvedimento
Se vado in consultazione rifiuto e cerco di scaricare gli allegati mi compare il messaggio bloccante "Attenzione. L'utente loggato non ha accesso alla risorsa"
Questo messaggio è solo in ambiente di test? In produzione il Comune potrà scaricarsi gli allegati per capire meglio,eventualmente, il rifiuto? Nei servizi cooperativi sarà possibile?
Come riprodurre il problema
No response

La segnalazione è stata corretta e appena rilasciata con la versione 1.33.0

authors: annamaraspin gbuemi

repo: italia/ansc


text:
Richiesta verifica metadati Comune di Sona

Buongiorno,
Si richiede la verifica dei metadati per conto del Comune di Sona.
https://www.comune.sona.vr.it/c023083/spid-metadata.xml
Il servizio in oggetto è realizzato con eGovernment Halley già validato in precedenza per altri clienti.

Il Metadata è stato inoltrato verso gli IdP e da essi caricato.

authors: AntonioGiovanniSchiavone halleyveneto

repo: italia/spid


text:
Fix issue with logging FileHandler in unsupported random writes file system

In this PR, we are adding some env vars to control logging settings.
I needed it because the FileHandler causes an error with file systems that does not support random writes (e.g. dbfs).

SPARKMONITOR_LOG_LEVEL: Logging Level.

DEBUG, INFO (default value), WARNING, ERROR, CRITICAL

SPARKMONITOR_LOG_HANDLER: Use FileHandler or StreamHandler

"file" (default value) use FileHandler('sparkmonitor_kernelextension.log', mode='w')
"anything else" use StreamHandler(sys.stdout)

LGTM

authors: DougTrajano itsjafer

repo: itsjafer/jupyterlab-sparkmonitor


text:
Hotfix/infinite rolling time

I removed didset at DateComponent. So I set first initial date setting manually.
Multiple selection allow for hourTableView. HourTableView needed to set three cell selected.
Add a function that make select row which is hour % 24, function name is markSelectedForVisibleCells. UITableViewExtension.
Add exception phrase for DayCollectionView.

Thank you!

Hi, thanks for the pull request again! However, I have fixed the issues I addressed earlier (in a slightly different way, sorry for not pinging you), so I cannot merge this time.
There's still a minor issue with your fix though, like when you scroll from minute 02 to 01, the 01 cannot be highlighted (unlike when you scroll from 59 to 01). You may want to fix that for your source code; or you can work with my latest code instead.
Thanks for your help!

authors: itsmeichigo maybeiamme

repo: itsmeichigo/DateTimePicker

license: mit


text:
Once logged in, unable to login with another identity

Logged in as yvestech:

Selected sign out:

** Signed out successfully**:

** When trying to sign in, I get automatically signed in with the previous profile**:

The underlying IYO session remains acive (10 min cookie), so the user is still logged in. Related to #480

authors: LeeSmet yveskerwyn

repo: itsyouonline/identityserver


text:
Can't install MAGMA Server

When i run the Magma Server with this Command ("docker run -d -v /home/lib/leipziginblocks/magma_server:/data -e TZ=Europe/Berlin -e TYPE=MAGMA -p 25565:25565 -p 25575:25575 -e EULA=TRUE -e MEMORY=6G -e ENABLE_RCON=true -e RCON_PASSWORD=Password --name mc-server itzg/minecraft-server:java8
e1109db65d8eea04e8e6214bcd76bd3cfc08baef9a4e0aaa37f64c2984c80d77") it gave me this error
[init] Running as uid=1000 gid=1000 with /data as 'drwxrwxr-x   12 1000     1000          4096 Mar  4 09:10 /data'
[init] Resolved version given LATEST into 1.16.5
[init] Resolving type given MAGMA
curl: (22) The requested URL returned error: 500
[init] ERROR unable to download version 1.16.5 of Magma
[init]       Check https://magmafoundation.org/ for available versions
Why?
How can i solve it

If the Magma URL is returning a 500 status code then that's a problem on their end. Please report an issue with Magma.
FYI https://developer.mozilla.org/en-US/docs/Web/HTTP/Status/500

Their API server is still broken by confirming with curl:
> curl -v https://api.magmafoundation.org/api/resources/Magma/1.16.5/stable/latest/download
*   Trying 185.173.26.97:443...
* TCP_NODELAY set
* Connected to api.magmafoundation.org (185.173.26.97) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
*   CAfile: /etc/ssl/certs/ca-certificates.crt
  CApath: /etc/ssl/certs
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* TLSv1.3 (IN), TLS handshake, Server hello (2):
* TLSv1.2 (IN), TLS handshake, Certificate (11):
* TLSv1.2 (IN), TLS handshake, Server key exchange (12):
* TLSv1.2 (IN), TLS handshake, Server finished (14):
* TLSv1.2 (OUT), TLS handshake, Client key exchange (16):
* TLSv1.2 (OUT), TLS change cipher, Change cipher spec (1):
* TLSv1.2 (OUT), TLS handshake, Finished (20):
* TLSv1.2 (IN), TLS handshake, Finished (20):
* SSL connection using TLSv1.2 / ECDHE-RSA-AES128-GCM-SHA256
* ALPN, server accepted to use h2
* Server certificate:
*  subject: CN=api.magmafoundation.org
*  start date: Jan 13 14:10:48 2021 GMT
*  expire date: Apr 13 14:10:48 2021 GMT
*  subjectAltName: host "api.magmafoundation.org" matched cert's "api.magmafoundation.org"
*  issuer: C=US; O=Let's Encrypt; CN=R3
*  SSL certificate verify ok.
* Using HTTP2, server supports multi-use
* Connection state changed (HTTP/2 confirmed)
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* Using Stream ID: 1 (easy handle 0x55d44fac2670)
> GET /api/resources/Magma/1.16.5/stable/latest/download HTTP/2
> Host: api.magmafoundation.org
> user-agent: curl/7.68.0
> accept: */*
>
* Connection state changed (MAX_CONCURRENT_STREAMS == 128)!
< HTTP/2 500
< server: nginx/1.17.8
< date: Sat, 06 Mar 2021 02:49:09 GMT
< content-type: text/html; charset=utf-8
< content-length: 14
< x-powered-by: Express
< access-control-allow-origin: *
< etag: W/"e-I1dfNaUh6cJh0nSsTLKcLwWyYaw"
< vary: Accept-Encoding
< strict-transport-security: max-age=15724800; includeSubDomains
<
* Connection #0 to host api.magmafoundation.org left intact
Internal Error

Their API docs page also fails with 500:

Hey i'm one of the magma devs I will have a look what's wrong with the API
Our own website uses the API aswell and dousn't seem to have any issues

Hey i'm one of the magma devs I will have a look what's wrong with the API

Thanks. FWIW I just tried the curl above and I'm still getting a 500.

Our api has changed last week. The image has to be updated

@patchint You're already using java8 variant, so docker-compose pull will grab the latest.

authors: Johann-vln itzg kingdevnl

repo: itzg/docker-minecraft-server


text:
Parallel Execution of 32 cycles during authentication/ encryption: ~4.5x speed up 🌟

Previously

When initializing state of cipher, 32 consecutive cycles were executed in parallel
But during encrypting/ decrypting and authenticating associated data or plain text, 8 consecutive cycles were executed in parallel

Now

When ever possible 32 consecutive cycles are executed in parallel, generating 32 key stream bits, used for authentication & encryption/ decryption.
It should boost execution speed by ~3.5x ✅
When executing 32 consecutive cycles in parallel, both 128 -bit NFSR and LFSR are interpreted as uint32_t[4] for ease of bit/ byte indexing.
On x86_64 platform, if __BMI2__ CPU featue is available then efficient _pext_u32 intrinsic is used for extracting out even/ odd bits from uint32_t.
Indices which can possibly be computed in compile time are done such way.

With this, I'm seeing these improvements.

Latest benchmark results on ARM CPU systems, seeing performance improvement due to more efficient even, odd bit deinterleaving routine.


text:
Conversation parsing does not work

Error scraping post. Skipping this post. Index: 13. Error: waiting for selector `[data-qa="slack_kit_list"].c-virtual_list__scroll_container[role="list"][aria-label^="Thread"]` failed: timeout 30000ms exceeded.
Error scraping post. Skipping this post. Index: 35. Error: waiting for selector `[data-qa="slack_kit_list"].c-virtual_list__scroll_container[role="list"][aria-label^="Thread"]` failed: timeout 30000ms exceeded.
Error scraping post. Skipping this post. Index: 39. Error: waiting for selector `[data-qa="slack_kit_list"].c-virtual_list__scroll_container[role="list"][aria-label^="Thread"]` failed: timeout 30000ms exceeded.

That said I do not care about thread contents for now, so that's okay.
Maybe in the README the wording should indicate that ENV CONVERSATION_NAMES=[] will stop the thread parsing.
I had commented the line and so it was parsing threads by default
Thank you so much for this project !! 🎉

oh okay it does not work because aria-label is a translated value

Hi @williamdes, thank you for bringing this problem to my attention! I hadn't considered scraping threads would break if the Slack client used another language than English. Would temporarily changing the Slack client language to English for the scrape be a good solution for you?

Hi @williamdes, thank you for bringing this problem to my attention! I hadn't considered scraping threads would break if the Slack client used another language than English. Would temporarily changing the Slack client language to English for the scrape be a good solution for you?

I will have to try that but for now changing the strings in the code is an easier option ^^

That works too! I'll get around to improving the error message in case anyone runs into this again, then I'll close this issue.

That works too! I'll get around to improving the error message in case anyone runs into this again, then I'll close this issue.

Or make an ENV as a workaround ?

Finally, I decided to go with a clearer error message when the thread fails and a note in the README to change the language to English for the scrape. I prefer not to add too many config options to the .env file. Also, I don't want to open the project to support multiple languages at the moment. The parsing is tricky enough with one language to account for.

authors: iulspop williamdes

repo: iulspop/slack-web-scraper


text:
Plugin not accounting for Daylight Saving Time in the United States

Love this plugin, and am starting to use it more intensively because it's so helpful. But with the change to daylight saving time in the United States, the plugin does not properly behave. If I add a task to occur, say, between 13:00 and 14:00 today, it will show on the day planner at running from 1200 to 1300. I can work around this but it's not ideal. I hope this is helpful--I am not a programmer so let me know if this is insufficient information.

Hello, @TomBirkland! This is a known issue, and it happens only on the day when the time gets changed. If this happens on any other day, please re-open this issue. Closing it as a duplicate of #309 for now.

Thank you!

authors: TomBirkland ivan-lednev

repo: ivan-lednev/obsidian-day-planner


text:
Multi vendo centralize setup

Hi Sir, it would be much better if the system supports centralized setup. Thank you

ok i will consider this enhancement :-D, thank you


text:
[ReactiveCocoa] Adds reactive cocoa support

Adds reactive cocoa support. There is no demo project or test so I
can’t really test if it works as expected but it should since it almost
identical to my Argo mapping functions.
You will need to use --allow-warnings for the lint to pass for the
pod.

Merged :+1: will push new version soon

repo: ivanbruel/Moya-ObjectMapper


text:
Sourcemaps not being correctly mapped for preprocessed files.

Given TS files after being preprocessed by gulp-concat or anything equivalent, the sourcesContent on sourcemaps generated result to null and sources listed are not the original src files. Below is my current setup.
Example Setup:
└── src/
   ├── apple.ts
   └── banana.ts

gulp.task('default', function () {
  return gulp.src('src/**/*.ts')
    .pipe(sourcemaps.init())
    .pipe(concat('all.ts'))
    .pipe(typescript())
    .pipe(sourcemaps.write())
    .pipe(gulp.dest('build'));
});

When opening build/all.js using source-map-visualization it fails to load. Error: Source 'all.ts' missing.
Expected source map output:
{"version":3,"sources":["apple.ts","banana.ts"],"names":[],"mappings":"AAAA;IAAA;IAAA,CAAA;IAAA,YAAA;AAAA,CAAA,IAAA;ACAA;IAAA;IAAA,CAAA;IAAA,aAAA;AAAA,CAAA,IAAA","file":"all.js","sourcesContent":["class Apple {}","class Banana {}"]}

Actual source map output:
{"version":3,"sources":["all.ts"],"names":[],"mappings":"AAAA;IAAA;IAAa,CAAC;IAAD,YAAC;AAAD,CAAb,AAAc,IAAA;AACd;IAAA;IAAc,CAAC;IAAD,aAAC;AAAD,CAAd,AAAe,IAAA","file":"all.js","sourcesContent":[null]}

I've started debugging through the code looks like the issue is related to the file path being absolute and changing the file to a relative path seems to fix the issue.
Problem Code: output.ts@83
// sourceFile.fileNameOriginal = "C:\Users\Matt\Desktop\test\src\all.ts";
// sourceFile.gulp.base = "C:\Users\Matt\Desktop\test\src\";
generator.applySourceMap(consumer, sourceFile.fileNameOriginal, sourceFile.gulp.base);

Proposed Solution:
generator.applySourceMap(consumer, path.relative(sourceFile.gulp.base, sourceFile.fileNameOriginal), sourceFile.gulp.base);

Optional Solution:
sourceFile and sourceMapPath are optional parameters but involves reverting a code change from #6372612 commit (Fix issues with output paths).
generator.applySourceMap(consumer);

Any thoughts? Will this have any negative repercussion? All tests still pass after making the change.

Thanks for reporting and investigating. I think that your proposed fix is the desired fix, though I would need to do some testing. The optional solution does not work well, the issue is that all paths (involved with source maps) are relative, but relative to different directories. Can you create a pull request with your proposed solution? Then I'll do some testing with it.

Just created a pull request for my "Optional/Second Solution". The reason for not going with my "Proposed Solution" is because I wasn't able to test using absolute paths as the expected result would be confined to my dev environment location ("C:\Users\Matt...."). Now thinking about it I could of used something like gulp-rename to work around the issue maybe? :/
I'm still not sure if it would be a problem, could you possibly have a look over it let me know what issue it causes so I can get a better understating. Might even be worth us adding a test round these problem scenarios.

Thanks for the PR. The test case is a good addition to the test suite. I'm afraid that it might break some other scenario's, but I'll investigate that. I have my mind on other things now but I hope to do that this weekend. I'll try to construct a test case that demonstrates such a scenario.
Absolute paths should not be a problem since gulp-sourcemaps' documentation states that the paths in the sourcemap should be relative.

I can't use gulp-typescript until this issue is solved. I am reverting back to gulp-exec or something like that.

Fixed in #465

@ivogabe any idea of when this will make to a release?

@giggio I want to go through some issues today and hope to release it today too.

authors: giggio ivogabe techmatt101

repo: ivogabe/gulp-typescript


text:
라이브러리 구조 결정

라이브러리를 pip install해서 사용할 사람들이
from TextVault import KnapsackEncryptor 와 같이 사용할 수 있게 할 지
from TextVault.knapsack import KnapsackEncrytor 와 같이 사용하게 할 지
전자 예시
후자 예시

전자로 진행해도 후자로도 불러올 수 있어 전자로 진행했습니다.


text:
🛑 05Takagi is down

In 537465d, 05Takagi (https://lixingyong.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: 05Takagi is back up in d563872 after 14 minutes.

repo: iwyang/check


text:
🛑 05Takagi is down

In 52357a4, 05Takagi (https://lixingyong.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: 05Takagi is back up in 3e36177 after 9 minutes.


text:
🛑 N IXTANET is down

In fc27d81, N IXTANET ($N_IXTANET) was down:

HTTP code: 0
Response time: 0 ms

Resolved: N IXTANET is back up in fa0206c after 5 minutes.

repo: ixtanet/upptime


text:
Fix to Issue #377 - Added triggerListenersAfterEvaluate call in DefaultRuleEngine check

Added trigger triggerListenersAfterEvaluate after every rules evaluation in doCheck method. At the moment it was just called in fire method so, we were not able for example to remove a Fact after rules evaluation when using the check method.
Added related unit test in RuleListenerTest.

Hi @benas , any chance to get this request evaluated?

repo: j-easy/easy-rules


text:
Parsley 5 API Consolidation

Parsley 5 will provide an opportunity to address some of the pain points of the API once again. This list may evolve over time.
In addition to #172, #184, and #168:

Remove parsley.io._, and instead move the implicit class within object Parsley. This will go with a extends PlatformSpecific trick to allow for the io stuff to be enabled for jvm and native but not js but not require any imports. Rename parseFromFile to parse.
Flatten the Lexer hierarchy somewhat: the plain objects aren't really buying us much other than indirection: numeric, text etc can just go I think.
I want to rename attempt to atomic (after seeking feedback on this from other experts and users), since I think it makes much more sense as a name and clears up some misconceptions: this might even be done pre-5, but with the binary compatible change.
Consider making a Scala 3 parsley "prelude" that allows for import parsley.quickstart.* or something to cut down on the parsley.Parsley, Parsley.*, parsley.character.*, parsley.combinator.* import pattern.
Remove >>=, getOrElse, join: >>= is provided by parsley-cats anyway, and is not something to encourage (same with join); and getOrElse is a misleading name, so this is going.
Remove attemptChoice, I don't want to encourage that in the API.
Remove TokenSpan from the API, and change LexToken to use offset instead of position, wayyyyyy simpler.
traverse to be curried, with function as second set of brackets
Change semantics of .hide so that it totally suppresses the errors underneath: #193 and #198 have already laid the groundwork for this to happen, since label("") is no longer legal. Instead .hide should be its own combinator independent of label itself -- this is an outdated notion now. Additionally, clean up the various overloads: it should be a single label combinator requiring at least one label, all non-empty.
verifiedUnexpected -> verifiedExplain, with the argumentless variant remaining as is: new verifiedUnexpected variants to be introduced in 5.1?
genericbridges renamed to generic, we don't know what else might end up there, and the name is a bit nicer.
Americanise the Specialised error to Specialized
The LexToken can swap to the simpler definition from thesis, and TokenSpan can be removed.
many, some, and eof to move to Parsley -- they are definitely core, and I want to cut down on imports (this is in tandem with #141)
remove more and between, they just aren't useful.
Add the symbol label mapping into ErrorConfig.
Minor adjustments to register extension classes.

I also want to deal with the tab situation properly.
Here's a sketch of an implementation that might work better:

expose parsley.PositionConfig, with def startLine: Int, def startCol: Int, def tabCol(col: Int): Int.
Then, this could also expose a def updatePos(c: Char, pos: PosState): Unit (and one for codepoints, for #129?) with a default implementation using tabCol, checking for newline etc. This could be overriden by a user who wants the extra control over what characters do what.
PosState is an abstract class at the parsley level, but is sealed within parsley (abstract class PosState private [parsley]): this
way we control the sub-classes, and the user doesn't know what they are. This exposes def getCol, def setCol, def getLine, def setLine, methods: these are called by updatePos. The Context implements this itself.
To aid JIT, we need to keep the dispatches to updatePos hot. This may involve keeping a helper in Context that forwards to the config, which then bounces back. We absolutely want JIT to specialise this to whatever the user has come up with.

Hopefully, this design gives maximal control to the user, and minimal overhead to the internals. There will be a new implicit parameter added to parse, which is the PositionConfig.

If full result elimination is performed in dropped positions, we might actually want to consider removing the skip variants of the combinators, including skipMany, skipSome, skip, range_; but not forP, as this optimisation cannot recognise "dead registers".
Also, sequence should be given an extra argument, same with traverse. exactly should have an n > 1 invariant.

manyUntil and someUntil are unusual names, they should be called manyTill and someTill for consistency with all the other parsecs! (this change will affect parsley-cats

Rework the lexer config (and error config?) so that they use the forwards compatible case class mechanism. This way we can evolve it as we want, and if we ever reach a final state, we can unprivate the members and release it in full.

Restore the partial amend semantics of at least filters. Unclear yet whether we'd want them on verified and prevent yet. Note that the deprecated constructors in token.errors will be reinstated

Rework implicits packages/object into syntax? Gives us better "parity" with the typelevel style.

rename ifP, guard, when, and whileP to all use an S postfix

Change Integer, Real, String, Character to all have a Parsers postifx: it'll be better for clashes with Scala and important for gigaparsec.

Remove multiMap and singleMap in the Lexer config, they can just be merged into a mapping instead?

Make BasicFilter into object (and thus make FilterConfig contravariant?)

Add additional parameter to Ops et al

Simplify the error config hierarchy so that it actually documents properly: no more hidden classes with companion objects! (where possible)

Remove parsley.implicits.combinator and move parsley.extension as parsley.syntax.extension


text:
无法定位

2016年7月14日 PM1:07:17 UTC
Loading Xposed v54 (for Zygote)...
Running ROM 'KOT49H.I9500ZCUHNF1' with fingerprint 'samsung/ja3gzc/ja3g:4.4.2/KOT49H/I9500ZCUHNF1:user/release-keys'
Loading modules from /data/app/com.rong.xposed.fakelocation-1.apk
Loading class com.rong.xposed.fakelocation.XFakeLocation
[XFL initZygote] log status = false
2016年7月14日 PM1:11:45 UTC
Loading Xposed v54 (for Zygote)...
Running ROM 'KOT49H.I9500ZCUHNF1' with fingerprint 'samsung/ja3gzc/ja3g:4.4.2/KOT49H/I9500ZCUHNF1:user/release-keys'
Loading modules from /data/app/com.rong.xposed.fakelocation-1.apk
Loading class com.rong.xposed.fakelocation.XFakeLocation
[XFL initZygote] log status = false
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:271)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)
java.lang.NoSuchMethodError: android.location.GpsStatus#setStatus(java.lang.Integer,int[],float[],float[],float[],java.lang.Integer,java.lang.Integer,java.lang.Integer)#bestmatch
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:233)
at de.robv.android.xposed.XposedHelpers.findMethodBestMatch(XposedHelpers.java:284)
at de.robv.android.xposed.XposedHelpers.callMethod(XposedHelpers.java:947)
at com.rong.xposed.fakelocation.x.b.e.b(Unknown Source)
at com.rong.xposed.fakelocation.x.c.afterHookedMethod(Unknown Source)
at de.robv.android.xposed.XposedBridge.handleHookedMethod(XposedBridge.java:645)
at android.location.LocationManager.getGpsStatus(Native Method)
at com.nianticlabs.nia.location.LocationManagerProvider$2.onGpsStatusChanged(LocationManagerProvider.java:155)
at android.location.LocationManager$GpsStatusListenerTransport$1.handleMessage(LocationManager.java:1468)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:157)
at android.os.HandlerThread.run(HandlerThread.java:61)

Turn on GMS hook on advanced settings and hook google play service
http://www.coolapk.com/apk/com.rong.xposed.fakelocation


text:
Attempted to call an undefined method named "getEmail" of class "J4k\OAuth2\Client\Provider\User

Hi there,
I have an issue when using your client in a bundle knpuniversity/oauth2-client-bundle
I would like to know how can I retrieve user's email because my code doesn't work and I am getting an error  "Attempted to call an undefined method named "getEmail" of class J4k\OAuth2\Client\Provider\User"
Below is a snippet of code which I am using:
$client = $this->get('oauth2.registry')
->getClient('vkontakte');
$user= $client->fetchUser();
$user->getEmail();
From error message I see that method getEmail() is not exists.
Can you point me which method should I use or what I am doing wrong?

https://vk.com/dev/fields
Not entirely sure vk gives back an email address?

thank you, really no email address


text:
Better/earlier require for type.cr

I noticed in my own code, as well as in the example from #18, that require "type" is either missing in the appropriate place or is being done too late.
If we take the example from #18:
require "crystalizer/json"
require "crystalizer/yaml"
#require "../lib/crystalizer/src/type"

module Thing

  struct Point
    getter x : Int32
    getter y : String
    #getter data : Hash(String,String)

    def initialize(@x, @y, @data : Hash(String, String)) # Array(String) would occur a same type error
    end
  end

  hash = Hash(String, String).new
  hash["1"] = "1"
  hash["2"] = "2"
  point = Point.new 1, "a", hash

  {Crystalizer::YAML, Crystalizer::JSON}.each do |format|
    puts format
    string = format.serialize point # works fine
    puts string
    puts format.deserialize string, to: Point # got an error
  end

end

This results in:
 > 1 | def serialize(type : Crystalizer::Type)
                            ^----------------
Error: undefined constant Crystalizer::Type

One can manually require the file (as seen in the commented line at the top of example), but it would be ideal if it was happening automatically as expected.

Yes, I noticed this too. Already fixed in the just release v1.1.1 :smile:

license: ISC


text:
Fix incorrect assignment

Hi. I simple corrected incorrect assignment variable.

@neph1 I assigned you. Java wise this looks logical but since you know asset browser :)

It's assetpacks, not asset browser :)
But still. I don't see how this could be wrong.

authors: TheLivan neph1 tonihele

repo: jMonkeyEngine/sdk


text:
Weather error line 15

This is line 15:
print("%s, %i °%s" % (info, temp, unit_key))
I'm missing something here?
Thanks

Do you get an error from running it? I'm unsure as to what the question is here, sorry.

Yes I do have an error when I run it...
It show the path of the weather script then add the number 15.

Could you give us a little more information. What is the exact error? I'm assuming you are using the user contributed weather module, so can you give us the output of that script if you run it inside a terminal and not polybar.

I think this issue has been fixed on wiki.
I believe what caused the error is that if the returned HTTP request code is not OK,
then something else might be returned along the line.
See also: comparison of b74908 and the previous one.

@natebwangsut The code on the wiki has been changed but didn't yet fix the issue, it just spits out another message now if the HTTP status code is not 200. I have now changed it to output an error message if the request fails.
Closing this.

@patrick96 That just fixes the error if the code is not 200.
I guess it's safe to assume that the data would be in the expected format.
You can also do this
try:
    # HTTP CODE = OK
    if REQ.status_code == 200:
        CURRENT = REQ.json()["weather"][0]["description"].capitalize()
        TEMP = int(float(REQ.json()["main"]["temp"]))
        print("{}, {} °{}".format(CURRENT, TEMP, UNIT_KEY))
    else:
        print("Error: BAD HTTP STATUS CODE " + str(REQ.status_code))
except (ValueError, IOError):
    print("Error: Unable print the data")

Anyways, good job!

Yes we can do that too. Feel free to add this to the wiki page as well. Thanks!

authors: NBonaparte natebwangsut patrick96 picarda

repo: jaagr/polybar


text:
Feature/1089 transfer character check functionality to PSD

What's included?

[x] Update the answer options to the question "What is your Sex" in the character section (i.e. Male, Female, Prefer not to say).
[x] Ensure that the Character Checks answers are not editable after the questionnaire has been submitted.
[x] Validate that the start date is always before the end date in Professional details.
[x] Skip the "Professional details" page if the candidate has not stated any qualifications.

Who should test?
✅ Product owner
✅ Developers
✅ UTG
How to test?
Preview URL: https://jac-apply-develop--pr1121-feature-1089-transfe-0gthrom3.web.app
Use the following email addresses (password: 1qazZAQ!1qaz) to sign in to the apply side:

application-0001@jac-dummy-email.jac
application-0002@jac-dummy-email.jac
application-0003@jac-dummy-email.jac
application-0004@jac-dummy-email.jac
application-0005@jac-dummy-email.jac
application-0006@jac-dummy-email.jac
application-0007@jac-dummy-email.jac
application-0008@jac-dummy-email.jac

Steps:

Go to "Current applications" and click the button "Pre Selection Day Questionnaire" under exercise JAC00685.
Go to "Character Checks".
Check if the answer options to the question "What is your Sex" are Male, Female, and Prefer not to say.
Check if the start date is always before the end date in Professional details.
Check if you can not edit the character checks after the questionnaire is submitted.

Risk - how likely is this to impact other areas?
🟠 Medium risk - this does change code that is shared with other areas
Additional context
Include screen grabs, notes etc.

PREVIEW:DEVELOP
can be OFF, DEVELOP or STAGING

The "Professional details" page might be blank if the candidate does not state any qualifications in the application form. Maybe we could show some text to indicate candidates to proceed with the checks.

Yes please @HalcyonJAC please can you either include a message or, ideally, skip this page if it is not relevant/needed for the candidate?

The "Professional details" page might be blank if the candidate does not state any qualifications in the application form. Maybe we could show some text to indicate candidates to proceed with the checks.

Yes please @HalcyonJAC please can you either include a message or, ideally, skip this page if it is not relevant/needed for the candidate?

Otherwise all is looking good 👍

@warrensearle Thanks for the suggestion. I have made the changes to check if the qualifications exist or not. If it doesn't exist, it will skip the "Professional details" page.

authors: HalcyonJAC warrensearle

repo: jac-uk/apply


text:
add package argument for resources.files()

running bing-rewards in the terminal opens a browser window, however, exits prematurely since there happens to be something missing in the code itself.
note: bing-rewards installed using pipx.
$ bing-rewards
Doing 33 desktop searches
Press ESC to quit searching
Opening browser [1508]
Exception in thread Thread-1 (both):
Traceback (most recent call last):
  File "C:\Users\meina\AppData\Local\Programs\Python\Python311\Lib\threading.py", line 1045, in _bootstrap_inner
    self.run()
  File "C:\Users\meina\AppData\Local\Programs\Python\Python311\Lib\threading.py", line 982, in run
    self._target(*self._args, **self._kwargs)
  File "C:\Users\meina\AppData\Local\pipx\pipx\venvs\bing-rewards\Lib\site-packages\bing_rewards\__init__.py", line 213, in both
    desktop()
  File "C:\Users\meina\AppData\Local\pipx\pipx\venvs\bing-rewards\Lib\site-packages\bing_rewards\__init__.py", line 201, in desktop
    search(count, words_gen, options.desktop_agent, options)
  File "C:\Users\meina\AppData\Local\pipx\pipx\venvs\bing-rewards\Lib\site-packages\bing_rewards\__init__.py", line 149, in search
    query = next(words_gen)
            ^^^^^^^^^^^^^^^
  File "C:\Users\meina\AppData\Local\pipx\pipx\venvs\bing-rewards\Lib\site-packages\bing_rewards\__init__.py", line 53, in word_generator
    word_data = resources.files().joinpath('data', 'keywords.txt')
                ^^^^^^^^^^^^^^^^^
TypeError: files() missing 1 required positional argument: 'package'

for reference, what version of python are you using?

I'm using Python 3.11.9.

oops, I used a 3.12 feature. Thanks for the catch 👍

authors: jack-mil znarfm

repo: jack-mil/bing-rewards


text:
Right click options not showing

Neither of the right click options are showing when the plugin is activated and their respective check boxes are checked. I am also not using the transparent chat box plugin as seen in a previous issue. Help appreciated.

for some unexplained reason, having opaque chat breaks the right click functions. Simply swap to transparent chat in the interfaces section in the in-game OSRS settings.
If the author sees this, is there a reason for this interaction?

authors: LackOfCreativiT Nildorff


text:
add json dump in workflow run

This issue discusses feedback from @janash
Current Implementation
In a single workflow run...
s3task's results are passed to a save_results task, which goes directly to the database
Problem
Users are required to switch back to python or the web UI to view results.
A stand-alone s3task run will not have results saved anywhere.
Possible Solutions

have the final result of s3task.workup dumped to json in the run directory.
have the database object in save_results dumped to json the run directory.

1 --> fix would add code directly to s3task.run method.
2 --> would require passing directory to the save_results tasks. Local saving wouldn't be possible with stane-alone s3task runs.

I'm trying out solution 1 (s3task.workup) by writing user-friendly yaml files. This is addressed in #49

repo: jacksund/simmate


text:
Feature Request: Support For Rock Band drums

Hello Jackun,
I was wondering if you would be interested in adding support for a drums device type in this plugin for Rock Band drums. I have searched a while to see if any plugin supports drum input so I could play them on PCSX2 but have come up short. This leaves games like Rock Band with perfect emulation but unable to play all of its features since you cannot select to play drums without having a drum controller. It would be nice to have something similar to your Pad/Wheel device type except for drums where a Device API for DInput or Raw Input could be selected. I think that I wouldn't be the only one who would be interested in such a feature. In my search I have come across many others asking the same question. There might be other communities as well that might benefit from such a feature. The Guitar Hero community comes to mind off the top of my head.
Thank you for your time and consideration in this feature request.
Sincerely,
InlineData
P.S. Thank you for your wonderful support of Logitech USB Mic. It works great in Rock Band.

Can you dump its usb descriptors? I made a little utility, idk how well it works. On windows the drum controller probably needs to have WinUSB driver installed https://zadig.akeo.ie/
Or boot to linux live cd.

Thank you for getting back to me so quickly.
Unfortunately I don't own any PS2 Rock Band Drums specifically. I do however have the WII drums for Rock Band. I don't know how helpful they would be to you but I dumped the usb descriptors anyway just in case.
After converting the device drive from HIDUSB to WinUSB as recommended your utility provided me this output:
0) 04b4:0901
1) 8086:8c31
2) 1038:1378
3) 1bad:0005
4) 04b4:0510
5) 8087:8000
6) 0a5c:21e8
7) 8087:8008
8) 8086:8c2d
9) 8086:8c26
Select device, enter number: 3
Device descriptor: 1201000200000040ad1b0500001001020001
String descriptor 0:
        'Licensed by Nintendo of America '
        42034c006900630065006e0073006500640020006200790020004e0069006e00740065006e0064006f0020006f006600200041006d00650072006900630061002000
String descriptor 1:
        'Harmonix Drum Controller for Nintendo Wii'
        54034800610072006d006f006e006900780020004400720075006d00200043006f006e00740072006f006c006c0065007200200066006f00720020004e0069006e00740065006e0064006f002000570069006900
String descriptor 2:
        '╨ë'
        04030904
Config descriptor 1: 0902290001010080320904000002030000000921110100012289000705020340000a0705810340000a
bNumInterfaces: 1
bInterfaceNumber 0, bInterfaceClass 3
        HID report descriptor: 05010905a10115002501350045017501950d05091901290d81029503810105012507463b017504950165140939814265009501810126ff0046ff0009300931093209357508950481020600ff0920092109220923092409250926092709280929092a092b950c81020a21269508b1020a2126910226ff0346ff03092c092d092e092f751095048102c0

Press enter to exit

Very sorry once again that I don't own any PS2 Rock Band drums. I will keep an eye out for them and pick them up if I see something within my budget. Thanks again for all your help.
Sincerely,
InlineData

It would be better to get the descriptors off the PS2 drums and some USB traffic capture would be useful too as there's probably gonna be some vendor-defined data structures like the Wii's you posted. But there seem to be many DIY midi projects on the 'net, maybe someone got something useful posted already.

Oh okay. I thought it might be the case. I will try look around the net and check and see if someone already has some information on this already.
Yeah unfortunately the online market really jacks up the prices... I was going to check around my local used game stores and see if I have better luck. Usually they have better prices since they are motivated to get rid of them since they take up so much space. Lol yeah, if you were to get one of these drum sets would probably put your fanatec clubsport plans out the window for a bit. Of course I don't expect you to get one. I'm the person who requested the feature I should get it. It might just be a little while if that's okay. I originally decided to make this feature request gambling that the WII drums would be similar enough to get good info off of or the off chance you might already have a PS2 drum kit or someone will see this issue will provide the necessary info. I figured I can't win if I didn't play :P
Thank you I uninstalled the driver and it works normally.
I will get back to you if I find anything on the net or manage to get my hands on a reasonably priced PS2 drum set. Thanks again.

OK, seems the games just check the VID/PID luckily. I got it so far that Rock Band and Rock Band 2 detect the drum kit ( VID_0x12BA&PID_0x0210 Harmonix Drum Kit for PlayStation(R)3) but I have no idea about the report data format but passing random data to it, it keeps selecting/canceling in the menu  :smile:

Oh that's excellent!!!! Thanks for trying despite not having a drum kit! Glad that it recognizes it! And you found the drum pad/bass buttons? o.0 you are amazing!!!
P.S I will still keep on searching for any information and drums in my local area. Sorry I haven't had any luck so far.

Ok, so do the Wii drums work in windows as a controller? Like can you map the buttons in wheel configuration dialog?

Sorry I took so long.
Yes the Wii drums work in windows as a regular controller. I believe it is mapped as DInput. Before contacting you I actually tried to map the drums to a race wheel just to see if it would work with rock band lol 😆. Your plugin mapped the input fine but Rock Band wouldn't recognize the race wheel. I figured it was worth trying though just in case 😛
Yee it needs a usb hub. I actually have your plugin working with LilyPad to get the Microphone working with 2 guitars (I tested). This actually requires using some multitap settings from LilyPad. All together you need the microphone a dual shock 2 controller (for vocalist controls) 2 guitars and drum mapped to play with all players for this game... After this plugin was complete I was actually thinking on making a tutorial and putting it up onto YouTube to show people how to get a completely functioning version (all players supported) of Rock Band working on PCSX2 so other people can enjoy! It is so unfortunate that this game hasn't been ported to PC. I believe once this plugin is hopefully done PCSX2 will be the only emulator on windows to fully support Rock Band.
Anyway, I decided to include a screenshot from your plugin showing it working receiving input from my Wii drums...
DInput Summery from your plugin (just in case)
Wii Drum Layout:

Button on Wii Drums -> Button on your plugin
A: -1/0/0/394 -> CROSS / A
B: -1/0/0/395 -> CIRCLE / B
1: -1/0/0/393 -> SQUARE / X
2: -1/0/0/396 -> TRIANGLE / Y

+: -1/0/0/402 -> START
-: -1/0/0/401 -> SELECT

red drum pad: -1/0/0/395 -> L2
yellow drum pad: -1/0/0/396 -> L1 / L
blue drum pad: -1/0/0/393 -> R1 / R
green drum pad: -1/0/0/394 -> R2

bass drum pedal: -1/0/0/397 -> BRAKE

I noticed some of the buttons on the drums are the same as the pads on the drums. That is interesting. I did it twice to make sure I wasn't tapping a button accidentally. I guess that is just the way it is. Next time I get a chance I will try playing the game by pressing the buttons instead of hitting the drum pads see if it works lol.

Try this https://ci.appveyor.com/api/buildjobs/cwvc1oh36qtvnayr/artifacts/build%2FUSBqemu-wheel-0.7.4-2-2019-02-18-6818e5fb-Release.zip

I just tested it out. It works 100%!!! I think there is absolutely no problems with the way it functions. It also works with all the other instruments fine too!! Thank you so much!!! 😃

Unless I've broken something in the meanwhile, should be in 0.8.0

Hi there! Tried the "Rock Band Drum Kit" in PCSX2 2.0.2 using an XBox Guitar Hero Drum Kit and it works (AWESOME) with a small issue: the bass drum can be mapped with LeftShoulder, but there is no way to map the extra Orange Cymbal (RightShoulder) - the GH drum has an extra orange cymbal as compared to the RB drum kit.
Is there any way to provide support to the full GH drum kit?
Thanks in advance and congratulations for your amazing work with PCSX2... such a great emulator!

authors: InlineData isaacdlp jackun

repo: jackun/USBqemu-wheel

license: unlicense


text:
Have "Save State" and "Restore State" also function like Copy Component/Paste Component

Specifically, this should work if there are two different component types.

Dropping request for now. Please reopen if still desired.

repo: jacobdufault/fullinspector


text:
Small fix for faster TryIndirectTypeLookup

The function was doing really silly things if you had more than one
assembly in the project. This removed over 10MB of allocations on our
game.

Thanks!

authors: TiliSleepStealer jacobdufault

repo: jacobdufault/fullserializer


text:
Create NamedPipe with Everyone Access on Windows Platform

The IPC pipe created by windows service could not be accessed by other processes. It is due to the permission settings of the pipe.
The solution is passing proper security handle when creating pipe. The native method is called since .netstandard does not support it so far.
This modification only affect Windows platform.

@brightness007 thanks for you contribution but I cannot reproduce the issue that you are trying to resolve.

@brightness007 thanks for you contribution but I cannot reproduce the issue that you are trying to resolve.

I managed to reproduce the issue by creating a Windows service in .NET Core that uses namedpipes for IPC, then registering the app as a Windows Service.

Server HostBuilder

Requires Microsoft.Extensions.Hosting.WindowsServices NuGet package
IHostBuilder host = Host.CreateDefaultBuilder().
    ConfigureServices(services =>
    {
        services.AddScoped<ISomeContract, ISomeImplemention>();
    })
    .ConfigureIpcHost(builder =>
    {
        builder.AddNamedPipeEndpoint<ISomeContract>("SomeContractEndpoint");
    })
    .UseWindowsService(); // Makes the process usable as a Windows service

Setting up service and starting

In Powershell:
New-Service -Name "SomeService" -BinaryPathName "path/to/server/binary.exe"
Start-Service -Name "SomeService"

From this point forward, any client that tries to access the named pipe endpoint will crash with a System.UnauthorizedAccessException. Starting the server binary will work without issue.
An exception of type 'System.UnauthorizedAccessException' occurred in System.Private.CoreLib.dll but was not handled in user code: 'Access to the path is denied.'
   at System.IO.Pipes.NamedPipeClientStream.TryConnect(Int32 timeout, CancellationToken cancellationToken)
   at System.IO.Pipes.NamedPipeClientStream.ConnectInternal(Int32 timeout, CancellationToken cancellationToken, Int32 startTime)
   at System.IO.Pipes.NamedPipeClientStream.<>c__DisplayClass20_0.<ConnectAsync>b__0()
   at ...

I haven't tested yet if this PR resolves this issue on Windows.
However, the same issue exists if you run the same server executable without .UseWindowsService() built for Linux as root and try to access it from a lesser privileged user client application.

Sorry for reply so late.
The code looks OK to me.
The process is running with the current user when the binary is launched directly (by double click). No issue is found.
For Windows Service,  New-Service will create a service with "SYSTEM" user by default. On Windows platform, two processes running with different account could not talk to each other. That is why System.UnauthorizedAccessException is thrown.
.Net platform does not provide API to modify RPC security settings, even in .Net Core. My PR is aimed to solve this problem.
These security API are only available on Windows platform, and the RuntimeInformation.IsOSPlatform(OSPlatform.Windows) is checked. For  Linux, the security should be controlled by umask, which could be modified in code.
Sorry for my poor English.
Regards,
Peng ZHANG

From: InfinityGhost notifications@github.com
Sent: Friday, May 8, 2020 1:58 AM
To: jacqueskang/IpcServiceFramework IpcServiceFramework@noreply.github.com
Cc: brightness007 zhangpeng_vip@hotmail.com; Mention mention@noreply.github.com
Subject: Re: [jacqueskang/IpcServiceFramework] Create NamedPipe with Everyone Access on Windows Platform (#93)
@brightness007https://github.com/brightness007 thanks for you contribution but I cannot reproduce the issue that you are trying to resolve.
I managed to reproduce the issue by creating a Windows service in .NET Core that uses namedpipes for IPC, then registering the app as a Windows Service.

Server HostBuilder

Requires Microsoft.Extensions.Hosting.WindowsServices NuGet package
IHostBuilder host = Host.CreateDefaultBuilder().
ConfigureServices(services =>
{
services.AddScoped<ISomeContract, ISomeImplemention>();
})
.ConfigureIpcHost(builder =>
{
builder.AddNamedPipeEndpoint("SomeContractEndpoint");
})
.UseWindowsService(); // Makes the process usable as a Windows service

Setting up service and starting

In Powershell:
New-Service -Name "SomeService" -BinaryPathName "path/to/server/binary.exe"
Start-Service -Name "SomeService"
From this point forward, any client that tries to access the named pipe endpoint will crash with a System.UnauthorizedAccessException. Starting the server binary will work without issue.
An exception of type 'System.UnauthorizedAccessException' occurred in System.Private.CoreLib.dll but was not handled in user code: 'Access to the path is denied.'
at System.IO.Pipes.NamedPipeClientStream.TryConnect(Int32 timeout, CancellationToken cancellationToken)
at System.IO.Pipes.NamedPipeClientStream.ConnectInternal(Int32 timeout, CancellationToken cancellationToken, Int32 startTime)
at System.IO.Pipes.NamedPipeClientStream.<>c__DisplayClass20_0.b__0()
at ...
I haven't tested yet if this PR resolves this issue on Windows.
However, the same issue exists if you run the same server executable without .UseWindowsService() built for Linux as root and try to access it from a lesser privileged user client application.
―
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/jacqueskang/IpcServiceFramework/pull/93#issuecomment-625407972, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AEO46SYNLFWZTXGKTFL4URDRQLZDZANCNFSM4LRSOKMA.

Thank you @brightness007 for your contribution and sorry for the delay of my review.

authors: InfinityGhost brightness007 jacqueskang

repo: jacqueskang/IpcServiceFramework


text:
Update faq.html

Just an edit of the questions and answers.

Great ! I'm merging it

repo: jacquetc/skribisto-help-website


text:
Test helper clientGet: can't pass arguments because of conflict with token

The clientGet method is supposed to accept data as its second parameter.
But as soon as you pass parameters there, it overrides the the query string ?token that is used for JWT.
I'm guessing the fix would be use headers here.. or maybe refactor all test helpers to use headers

This might be fixed with 5.2