[occm] loadbalancer: add missing metric observations, improve logs

What this PR does / why we need it:
Add missing metrics observations to OpenStack API calls.
Fix unhandled errors from mc.ObserveRequest(err).
Add log lines for loadbalancer reconciliation.
Log the loadbalancer ID where available.
Which issue this PR fixes(if applicable):
fixes #
Special notes for reviewers:

Release note:

NONE

Sean Schneeweiss<EMAIL_ADDRESS>Daimler TSS GmbH, legal info/Impressum

/ok-to-test

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance

Please don't trigger the test until we fix the CI issue, thanks

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance

@seanschneeweiss I am currently trying to find out how to expose these metrics out of occm? Do you have idea how we can do that? Documentation says that metrics are available in https://localhost:10258/metrics. However, I would like to expose metrics port to be accessible for external prometheus.

@zetaab The documentation refers to a manual access for testing, if the exposing of the metrics work.

As a first step, I would manually check if your OCCM exposes metrics.

Create a port forward via kubectl to your OCCM pod

kubectl -n kube-system port-forward <insert-your-occm-pod-id-here> 10258

Now access the metrics endpoint in your local browser via https://localhost:10258

As a second step you have to setup Prometheus to scrape the endpoint. We use annotations on the OCCM DaemonSet spec. Prometheus will then pick up the endpoint.template:
  metadata:
    annotations:
      prometheus.io/port: "10258"
      prometheus.io/scheme: https
      prometheus.io/scrape: "true"

The configuration of Prometheus is out of scope here, but maybe these IBM docs help you. If you encounter OCCM specific problems please open a new issue.

@zetaab Sorry, I missed "external" Prometheus.
I guess you have already found a solution to it in #1400.

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance

/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.20

/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.19
/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.18

/retest

/test cloud-provider-openstack-acceptance-test-e2e-conformance-stable-branch-v1.20

/test cloud-provider-openstack-e2e-test-csi-cinder

/test cloud-provider-openstack-e2e-test-csi-cinder

/override openlab/cloud-provider-openstack-acceptance-test-csi-manila

/lgtm
All the changes look reasonble.

/cc @jichenjc @ramineni

/lgtm
/approve


Added two buildings and a background

The buildings are two different scenes that just how many players are on their areas. They do not contain any logic

There are merge conflicts between this request and another one so I'm going to merge them and make a new request.


Wait for master to come online

In docker compose version 3, the support for condition in depends_on configuration has been dropped. From docs

depends_on does not wait for db and redis to be “ready” before starting web - only until they have been started.

Moreover, it expects startup order to be controlled by the application.
By the change in this PR, membership manager will wait for master to come online, instead of exiting on psycopg2.OperationalError exception. This will allow the docker-compose file to be easily ported to version 3
My docker-compose.yml
version: "3"

services:
  citus_master:
    container_name: "${COMPOSE_PROJECT_NAME:-citus}_master"
    image: "citusdata/citus:8.3.1"
    labels: ["com.citusdata.role=Master"]
  citus_manager:
    container_name: "${COMPOSE_PROJECT_NAME:-citus}_manager"
    image: "pushowl/citus-membership-manager:0.2.0"
    volumes: ["/var/run/docker.sock:/var/run/docker.sock"]
    environment:
      - CITUS_HOST=citus_master
    depends_on:
      - citus_master
  citus_worker:
    image: "citusdata/citus:8.3.1"
    labels: ["com.citusdata.role=Worker"]
    depends_on:
      - citus_manager

Hi @a6kme , thanks for your contribution. There were several issues with docker-compose V3 that made me want to wait until they are resolved as well.
We were not able to block worker node services until the membership manager came online here, and my tests proved that the changes here are not really enough.
See the two PRs that I opened for reference:
https://github.com/citusdata/membership-manager/pull/9
https://github.com/citusdata/docker/pull/187

#9 is now merged that contains all the changes for Docker Compose V3

Hey thanks a lot @hanefi. Apologies for delayed response. Somehow I missed the notification for this repo. :)


[spi_device] Incoming address size cannot be validated

In flash/passthrough modes, the spi_device IP does not give any indication of how long the transaction actually was, so transactions using the wrong address mode may pass through. This might only be of interest for invalidating shortened Sector Erase transactions, where 3 bytes of address was provided when the address mode required 4 bytes of address.
This is tagged for a future release. It falls into the category of robustness in an environment with errors, but it is not critical for the current stepping.

just curious, wouldn't this get picked up through payload length or something?
ie, after we get the CSb rise, and software comes to look what was received and discover it's one byte short?

@tjaychen There is no payload for Sector Erase, though :)

Ah, thanks for the clarification.

Triaged for spi_device. Assigning to M2.5 with https://github.com/lowRISC/opentitan/labels/Priority%3AP2 because I think we should make sure the current HW behavior is suitable for production (discuss with product team), although this is "only" about robustness in an environment with errors.  If yes, defer back to FutureRelease.

I'm not super confident on this estimate and hence I have leaned towards pessimism. If it is not deferred, recommend to re-evaluate.

estimate 16
remaining 2023-03-23 16

This was actually fixed in #21120. I forgot that the address FIFO is written all at once, so there is no need to separately track the number of bytes written -- It will either be 0 or the number indicated by the address mode.


Patch to handle cross-validation when X is a sparse matrix

Hi,
Thanks for the awesome package.  Optunity, by default, does not handle the case when X is sparse, since it tries to figure out the shape of X by calling len(X).
I added a small patch where I changed calls to len(X) with X.shape[0] which is valid even in the case where X is a scipy sparse matrix.

From 1a49c9f965d5397471bc8105c73236e8af5edf1e Mon Sep 17 00:00:00 2001
From: Federico Vaggi<EMAIL_ADDRESS>Date: Wed, 24 Feb 2016 09:48:56 +0100
Subject: [PATCH] Changed len(x) to X.shape[0] handle case where X is a sparse
matrix

optunity/cross_validation.py |   10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/optunity/cross_validation.py b/optunity/cross_validation.py
index f94743e..beb9a13 100755
--- a/optunity/cross_validation.py
+++ b/optunity/cross_validation.py
@@ -234,7 +234,8 @@ def generate_folds(num_rows, num_folds=10, strata=None, clusters=None):
return folds
def mean(x):

return float(sum(x)) / len(x)

x_len = len(x)
return float(sum(x)) / x_len

def mean_and_list(x):
"""Returns a tuple, the first element is the mean of x and the second is x itself.
@@ -477,9 +478,12 @@ def cross_validated(x, num_folds=10, y=None, strata=None, folds=None, num_iter=1
AssertionError
 """

assert(num_folds <= len(x))
assert(y is None or len(y) == len(x))

x_len = x.shape[0]

assert(num_folds <= x_len)
assert(y is None or len(y) == x_len)
args = dict(locals())
args.pop('x_len', None)
def wrapper(f):
args['f'] = f
return cross_validated_callable(**args)
--
<IP_ADDRESS>

Thanks, I will check it out in the next few days!
2016-02-24 9:50 GMT+01:00 Federico Vaggi<EMAIL_ADDRESS>
From 1a49c9f965d5397471bc8105c73236e8af5edf1e Mon Sep 17 00:00:00 2001
From: Federico Vaggi<EMAIL_ADDRESS>Date: Wed, 24 Feb 2016 09:48:56 +0100
Subject: [PATCH] Changed len(x) to X.shape[0] handle case where X is a
sparse
matrix
optunity/cross_validation.py | 10 +++++++---
1 file changed, 7 insertions(+), 3 deletions(-)
diff --git a/optunity/cross_validation.py b/optunity/cross_validation.py
index f94743e..beb9a13 100755
--- a/optunity/cross_validation.py
+++ b/optunity/cross_validation.py
@@ -234,7 +234,8 @@ def generate_folds(num_rows, num_folds=10,
strata=None, clusters=None):
return folds
def mean(x):

return float(sum(x)) / len(x)
x_len = len(x)

return float(sum(x)) / x_len
def mean_and_list(x):
"""Returns a tuple, the first element is the mean of x and the second
is x itself.
@@ -477,9 +478,12 @@ def cross_validated(x, num_folds=10, y=None,
strata=None, folds=None, num_iter=1
AssertionError
"""

assert(num_folds <= len(x))
assert(y is None or len(y) == len(x))
x_len = x.shape[0] +
assert(num_folds <= x_len)
assert(y is None or len(y) == x_len) args = dict(locals())
args.pop('x_len', None) def wrapper(f): args['f'] = f return
cross_validated_callable(**args) -- <IP_ADDRESS>

—
Reply to this email directly or view it on GitHub
https://github.com/claesenm/optunity/issues/69#issuecomment-188143696.

Thanks for your suggested fix. I've incorporated this with some minor changes in master.


Patch feat name prop

Added name prop support for the Dropdown component. Since the emitted target is currently a <li> and not an input, I could not leverage the input name attribute.
Some minor propTypes re-ordering following #546 to fix eslint warnings.

Makes sense, thanks!


Handle LUIS Programmatic APIs v2.0

Refer https://dev.projectoxford.ai/docs/services/5890b47c39e2bb17b84a55ff/operations/5890b47c39e2bb052c5b9c2f

Already finished by 25a3e168f5a85d6b607e017f4f4765a16ad8276f


about embedding concat

layer1 ~ layer3 have different feature map size, so how to concate embedding from layer1 ~ layer3?
i can't find it from the paper

Hi,
Below is the reply from the original author, I hope this can help you.
The features extraction step is similar to Spade, (like in this (very slow) implementation https://github.com/byungjae89/SPADE-pytorch), and you can indeed up-sample the smallest features map to the size of largest feature maps to concatenate.
However, using up-sampling is very inefficient from a memory and time complexity point of view! You can just associate the corresponding activation without up-sampling
for example, if the first layer is 56x56 and the second is 28x28, the third layer is 14x14 like in a resnet.
the activation "pixels" of layer 1 in position (x1, y1) will be associated with the activation "pixels" of layer 2 (x1//2, y1//2) and (x1//4, y1//4) in layer 3.
For example (53,55) in layer 1 is concatenate with  (26,27) in layer 2 and (13, 13) in layer 3. (with index starting from 0, is it clear ?).

layer1 ~ layer3 have different feature map size, so how to concate embedding from layer1 ~ layer3?
i can't find it from the paper

The concatenation function is defined here:
https://github.com/xiahaifeng1995/PaDiM-Anomaly-Detection-Localization-master/blob/65adb9c461017e542b09b08042e21e606a8f8977/main.py#L286
You could also use torch.nn.functional.interpolate(input, scale_factor=2, mode='nearest') as you like.

@Omarelsaadany. you need to change your input image size, as with your current input image size there is an dimension error. 200724480/(20x256x50x50) is a float number and not an integer
The dimension "-1" is a PyTorch alias for "infer this dimension given the others have all been specified" (i.e. the quotient of the original product by the new product)


fix macos PR jobs on high-sierra workers

Looks like macos high sierra workers already have jq installed, so PR jobs are failing with:
Error: jq 1.5_3 is already installed
To upgrade to 1.6, run `brew upgrade jq`

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Merging early for fix broken PR tests.

Corrupt JSON data (indicates timeout or crash):
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.new: 10
    bm_call_create.BM_IsolatedFilter_ClientChannelFilter_NoOp_.counters.old: 10

[microbenchmarks] No significant performance differences


[Feature] :Unable to close chatbot

Description
There is no option for closing the opened chatbot which creates a bad user experience. @Rakesh9100 please assign this issue to me.
Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[ ] I am a IWOC'24 contributor.
[ ] I am a JWOC'24 contributor.
[ ] I am a SWOC'24 contributor.

Code of Conduct

[X] I agree to follow this project's "Code of Conduct".

Issue 621 is already open for this, pls create new issues @Garima-149


Refactor Azure refresh parser to make a single call to get network interfaces

At the moment the get_vm_nics method is making one request per VM (per NIC) to get NIC information. This PR replaces all that with a single .list call that gets all NIC's up front, and then uses Ruby code internally to compare .id strings as needed.
The id strings already contain the resource group, too, so there's no need to parse that out and compare them separately.
I would like to get this in before doing more work on security groups because I can use the list of NIC's to more easily retrieve security groups.

@bronaghs @blomquisg @bzwei Please review.

@djberg96. @blomquisg
LGTM :+1:

:+1:


HBASE-29035: Amount of region cached in the region metrics not update…

…d for a region immediately after it is flushed with cacheOnWrite turned on

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 31s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
hbaseanti
0m  1s

Patch does not have any anti-patterns.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m  3s

master passed

+1 :green_heart:
compile
2m 58s

master passed

+1 :green_heart:
checkstyle
0m 35s

master passed

+1 :green_heart:
spotbugs
1m 30s

master passed

+1 :green_heart:
spotless
0m 43s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 48s

the patch passed

+1 :green_heart:
compile
2m 58s

the patch passed

+1 :green_heart:
javac
2m 58s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

-0 :warning:
checkstyle
0m 34s
/results-checkstyle-hbase-server.txt
hbase-server: The patch generated 8 new + 0 unchanged - 0 fixed = 8 total (was 0)

+1 :green_heart:
spotbugs
1m 35s

the patch passed

+1 :green_heart:
hadoopcheck
10m 50s

Patch does not cause any errors with Hadoop 3.3.6 3.4.0.

-1 :x:
spotless
0m 35s

patch has 51 errors when running spotless:check, run spotless:apply to fix.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 40s

the patch passed

+1 :green_heart:
compile
0m 53s

the patch passed

+1 :green_heart:
javac
0m 53s

the patch passed

+1 :green_heart:
javadoc
0m 28s

the patch passed

+1 :green_heart:
shadedjars
4m 55s

patch has no errors when building our shaded downstream artifacts.

_ Other Tests _

+1 :green_heart:
unit
190m 46s

hbase-server in the patch passed.

213m 54s

Subsystem
Report/Notes

Docker
ClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6549/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6549

JIRA Issue
HBASE-29035

Optional Tests
javac javadoc unit compile shadedjars

uname
Linux ce67909c1328 5.4.0-200-generic #220-Ubuntu SMP Fri Sep 27 13:19:16 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / 39d4785303f7aa96507611b5ee737243025967b8

Default Java
Eclipse Adoptium-17.0.11+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6549/1/testReport/

Max. process+thread count
4745 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6549/1/console

versions
git=2.34.1 maven=3.9.8

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 30s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
hbaseanti
0m  0s

Patch does not have any anti-patterns.

_ master Compile Tests _

+1 :green_heart:
mvninstall
3m  0s

master passed

+1 :green_heart:
compile
2m 58s

master passed

+1 :green_heart:
checkstyle
0m 35s

master passed

+1 :green_heart:
spotbugs
1m 29s

master passed

+1 :green_heart:
spotless
0m 42s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 49s

the patch passed

+1 :green_heart:
compile
3m  3s

the patch passed

+1 :green_heart:
javac
3m  3s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 35s

the patch passed

+1 :green_heart:
spotbugs
1m 33s

the patch passed

+1 :green_heart:
hadoopcheck
10m 51s

Patch does not cause any errors with Hadoop 3.3.6 3.4.0.

+1 :green_heart:
spotless
0m 41s

patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m  9s

The patch does not generate ASF License warnings.

35m 43s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6549/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6549

JIRA Issue
HBASE-29035

Optional Tests
dupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless

uname
Linux 80c1713acd97 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / badf0728f65b18199ae0192c2d1237a94b355a30

Default Java
Eclipse Adoptium-17.0.11+9

Max. process+thread count
86 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6549/2/console

versions
git=2.34.1 maven=3.9.8 spotbugs=4.7.3

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Logfile
Comment

+0 :ok:
reexec
0m 31s

Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
detsecrets
0m  0s

detect-secrets was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
hbaseanti
0m  0s

Patch does not have any anti-patterns.

_ master Compile Tests _

+1 :green_heart:
mvninstall
2m 51s

master passed

+1 :green_heart:
compile
3m  7s

master passed

+1 :green_heart:
checkstyle
0m 35s

master passed

+1 :green_heart:
spotbugs
1m 28s

master passed

+1 :green_heart:
spotless
0m 42s

branch has no errors when running spotless:check.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 52s

the patch passed

+1 :green_heart:
compile
3m  0s

the patch passed

+1 :green_heart:
javac
3m  0s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
checkstyle
0m 35s

the patch passed

+1 :green_heart:
spotbugs
1m 35s

the patch passed

+1 :green_heart:
hadoopcheck
10m 46s

Patch does not cause any errors with Hadoop 3.3.6 3.4.0.

+1 :green_heart:
spotless
0m 42s

patch has no errors when running spotless:check.

_ Other Tests _

+1 :green_heart:
asflicense
0m  9s

The patch does not generate ASF License warnings.

35m 37s

Subsystem
Report/Notes

Docker
ClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6549/3/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/6549

JIRA Issue
HBASE-29035

Optional Tests
dupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless

uname
Linux d279c3a790a8 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
master / badf0728f65b18199ae0192c2d1237a94b355a30

Default Java
Eclipse Adoptium-17.0.11+9

Max. process+thread count
84 (vs. ulimit of 30000)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6549/3/console

versions
git=2.34.1 maven=3.9.8 spotbugs=4.7.3

Powered by
Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

retest


fix(streaming): correctly handle degree table not needed case

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
as title
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)
close #6093

rubber stamp
cc @yezizp2012 Could you please test whether the recovery test works on this branch?

Yes, I have test it locally and it works. 😃


feat: Add OrFailIf extension method

Hello there,
This pull request introduces a new extension method, OrFailIf, to the Result class.
I hope you don't mind that I went ahead and created this pull request directly. I felt a separate discussion wasn't necessary since this addition is minor, straightforward, and self-explanatory.
I often find myself needing to split Result.Merge() statements to provide more detailed error messages, particularly for null checks. To avoid repetitive code in every validation, I believe this feature offers a clean and effective solution to that problem.
Looking forward to your feedback!
Best regards,
Martin

We made multiple extension methods at my job to deal with failures. I can't legally share it, but you may want to add methods that take a predicate as a parameter, that acts on the value of a Result, so you can fail if the value isn't valid contextually.
You can also add other signature of this method to follow the existing formats of methods acting on Result and Task, like Bind.

That's exactly what I always found myself doing. Creating or copying the same extension methods from over and over again.
This time I'll try to contribute. 🚀


Provide good defaults

90% of the time I don't want to first do
using OptimizationOptimJL
solver = OptimizationOptimJL.LBFGS()
optim_options = (f_tol=1e-4, g_tol=1e-4, iterations=30, show_trace=false)

but simply do
using GeometryOptimization, AtomsBuilder, EmpiricalPotentials
sw = StillingerWeber()
at = rattle!( bulk(:Si), cubic=true) * 2, 0.1 )
minimize_energy!(at, sw)

This should work out of the box.

I agree. Definitely important to have good heuristics for defaults.


package: max open files limits are too low by default

/kind bug
Description
When podman package is built with defaults, the open files limits are taken from the host and can be tool low for some containers like memcached, etc.
Note that buildah and docker are built with 1048576 and we should probably change podman defaults to use this number as well.
Steps to reproduce the issue:

Deploy memcached container with MAXCONN set to 8192.

Describe the results you received:
Failure:
failed to set rlimit for open files. Try starting as root or requesting smaller maxconns value
Describe the results you expected:
No failure.
Additional information you deem important (e.g. issue happens only occasionally):
Output of podman version:
Version:       0.8.5-dev
Go Version:    go1.10.3
OS/Arch:       linux/amd64

Output of podman info:
host:
  Conmon:
    package: podman-<IP_ADDRESS>-1.gitf38eb4f.el7.x86_64
    path: /usr/libexec/podman/conmon
    version: 'conmon version 1.12.0-dev, commit: 74bba30adae4fdcacf887d35e4d6331318765028-dirty'
  MemFree: 210780160
  MemTotal:<PHONE_NUMBER>
  OCIRuntime:
    package: runc-1.0.0-37.rc5.dev.gitad0f525.el7.x86_64
    path: /usr/bin/runc
    version: 'runc version spec: 1.0.0'
  SwapFree:<PHONE_NUMBER>
  SwapTotal:<PHONE_NUMBER>
  arch: amd64
  cpus: 4
  hostname: undercloud75.localdomain
  kernel: 3.10.0-862.11.6.el7.x86_64
  os: linux
  uptime: 26h 37m 53s (Approximately 1.08 days)
insecure registries:
  registries: []
registries:
  registries:
  - docker.io
  - registry.centos.org
  - registry.access.redhat.com
store:
  ContainerStore:
    number: 79
  GraphDriverName: overlay
  GraphOptions:
  - overlay.override_kernel_check=true
  GraphRoot: /var/lib/containers/storage
  GraphStatus:
    Backing Filesystem: xfs
    Native Overlay Diff: "true"
    Supports d_type: "true"
  ImageStore:
    number: 39
  RunRoot: /var/run/containers/storage

Additional environment details (AWS, VirtualBox, physical, etc.):
Running everything in Virtualbox.
podman run fedora cat /proc/self/limits
Max open files            1024                 1024                 files

docker run fedora cat /proc/self/limits
Max open files            1048576              1048576              files

Opened #1355 to fix

Closed via #1355


.vue file includes decorators, compile error

when file includes decorator, compile error, maybe plugin should expose a bablePlugins props like @vitejs/plugin-vue2-jsx it did

import Vue from 'vue'
import { Component } from 'vue-decorator-property'

@Component
export default class Foo extends Vue {
    bar = 1
}


Bluetooth: csis: add RSI advertising callback

Makes it possible for an application to
handle CSIS RSI advertising by registering a
callback, which will disable the internal
CSIS advertising.
Also fixes registering callbacks in CSIS.
Signed-off-by: Lars Knudsen<EMAIL_ADDRESS>

As far as I can see, some tests seem to fail because the callbacks are registered and the following is now executed: https://github.com/zephyrproject-rtos/zephyr/blob/main/subsys/bluetooth/audio/csis.c#L306-L337
Investigating /cc @Thalley

We should also consider renaming the PRSI to just RSI

We should also consider renaming the PRSI to just RSI

agreed - now or later?

We should also consider renaming the PRSI to just RSI

agreed - now or later?

Up to you :) I'm fine with either

We should also consider renaming the PRSI to just RSI

agreed - now or later?

Up to you :) I'm fine with either

I can do some renaming ;)

We should also consider renaming the PRSI to just RSI

agreed - now or later?

Up to you :) I'm fine with either

I can do some renaming ;)

Pushing suggested changes including some renaming (all variants -> rsi)


Fetch from upstream integration with https://github.com/ianvs/prettier-plugin-sort-imports

Hey, can we include this commit also?
https://github.com/nderscore/prettier-plugin-nativewind/commit/f9b249b171003da870e6865fd2b54c226b501a97
It adds integration with https://github.com/ianvs/prettier-plugin-sort-imports

Synced with upstream to<EMAIL_ADDRESS>Released in<EMAIL_ADDRESS>:+1:


qt: bump qt5 to qt515 on darwin

Description of changes
It seems that we can bump qt5 to qt515 on darwin since qt515 build fine on x86_64 arch (https://hydra.nixos.org/jobset/nixpkgs/trunk#tabs-jobs, filter qt515.)
On aarch64 arch there is some issues with some libs (https://hydra.nixos.org/jobset/nixpkgs/nixpkgs-unstable-aarch64-darwin#tabs-jobs, filter qt515.), but since qt514 on aarch64 didn't build at all I don't see why we can't change qt514 to qt515.
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

See https://github.com/NixOS/nixpkgs/pull/184560


feat: re-enable plan cache for graphql handler

@fiam I'd like to ask you for 3 things before we switch from draft to open

[x] add testing capabilities to the testing framework for the GraphQL Endpoint
[x] add 2 basic tests for the GraphQL handler
[x] add another test with concurrency that ensures the gateway behaves correctly with concurrent requests

Motivation and Context
Re-enables a planning cache for the GraphQL handler.
Marketing Changelog
More and more users leverage the GraphQL Endpoint.
We wanted to make sure that using the Endpoint will scale, so we've implemented a planning cache with LRU capabilities, allowing the planning cache to grow until a certain point and then garbage collect less used plans to keep memory usage limited.
Fixes ENG-2088

@jensneuse Anything we should add in this PR?


add "code with" agreements

Pull Request on code with agreements
Link to the design document that prompted this PR
Issue #332
What are you trying to address
This is a "code with" playbook so it is important to add a topic on how CSE and Customer/Partner hold key roles like Tech Lead or Product Owner.
Description of new changes
Tried to find the best location for this and found that it would be to add a "Code With Agreements" section inside "team agreements".
How did you test it
used recommended linter for markdown.
Internal PR (@benjguin and @jcorioland) before submitting to the main repo.
Any relevant logs or outputs
N/A
Other information or known dependencies
N/A

Can you also link to the issue you are solving with this PR?

I created issue #332 and added the link in the description of this PR.

Hi @TessFerrandez, any idea where I could ask for a second reviewer, or who would be a good reviewer for this?
You approved, @jcorioland and I co-authored (and agreed), but the process requires a second reviewer.
Thanks for your help and valuable input so far.
:-) b

I would suggest this be reworded to be about working/team agreements that our customers can use on their own. The code-with practice is our internal engagement model and probably belongs elsewhere. I would remove the CSE specific terms and re-purpose this as advice for how a customer could establish a working agreement on their own.

I would suggest this be reworded to be about working/team agreements that our customers can use on their own. The code-with practice is our internal engagement model and probably belongs elsewhere. I would remove the CSE specific terms and re-purpose this as advice for how a customer could establish a working agreement on their own.

Just noticed we already have working agreements doc (thanks @fnocera for pointing that out). Given we already have working agreement doc. Maybe this content needs to be ported to our own internal wiki as things that are specific to our engagement model should live there, not in the playbook. The playbook is intended to be usable by anyone not just those who are working in an engagement with CSE.

Hi

I would suggest this be reworded to be about working/team agreements that our customers can use on their own. The code-with practice is our internal engagement model and probably belongs elsewhere. I would remove the CSE specific terms and re-purpose this as advice for how a customer could establish a working agreement on their own.

Just noticed we already have working agreements doc (thanks @fnocera for pointing that out). Given we already have working agreement doc. Maybe this content needs to be ported to our own internal wiki as things that are specific to our engagement model should live there, not in the playbook. The playbook is intended to be usable by anyone not just those who are working in an engagement with CSE.

What made me thing it would be useful in this repo include:

The URL of the repo is https://github.com/microsoft/code-with-engineering-playbook/
The repo already contains a page about CSE
I see 22 match results when looking for CSE in the repo content
the repo README.md title is CSE Code-With Customer/Partner Engineering Playbook

@benjguin Understand the confusion about what belongs where. I would suggest working with @fnocera internally to determine what content should remain and what should live in our own internal wiki. She can also help clarify what is intended to be here vs our internal wiki.

Adding @meulta who is the lead on the Agile front to help us decide on next steps.

Hello folks!
I agree that if we are talking about code-with in this repository, we need to explain what it means.
To your point @charleszipp one goal for this is to build something that can bring value to teams outside of CSE, and outside of Microsoft. I think most of it is pretty generic but some instances are not. As an example, the Engineering Feedback Section talks about the concept of feedback with some Microsoft/CSE specific terms and tools.
As you mentioned (if I understand correctly). We can use the same approach here. If we take the essence of what "coding with" means, it is something that a lot of other organizations, customers, SI, etc. are doing. You can be a dev team at a customer and have to code with another one. You can be an SI and be hired by a customer to code with them, and so on.
If we go down that path, we would add a short description of what Coding-with means at the top of the document so people know if they are in that case and if they should care, then we can replace "CSE" by "your team" and "Partner" by "the other team" or "the team you are working with" (or something shorter, probably).
@benjguin sorry I understand this is a lot of discussion for a simple thing, but I agree that we need to be careful about the goal of this repo and what is in it. Happy to hop on a call with you to chat about it.

I think that's a great idea - while it's beneficial to have the context on what CSE is (like the in the page about CSE), I like the idea of keeping that as more of a "learning about the authors and context of this repo" mentality, rather than something that needs to permeate into concepts that otherwise could be shared. If and where possible, generalizing our choices to not be CSE-specific will make this repo useful at a more general range. Code-with may be our motto, but it's not a unique concept.
I think it's a great idea to provide some context on what code-with means wherever it might be unclear, or a reader might be jumping right to a specific page without the context of the repo.

@meulta I agree, I think the most logical place for the definition of code with is as part of the working agreement vs a separate document/section of the playbook.

Since there have been no changes pushed to this PR since the comments were provided I am closing it. Of course the contributions would be much appreciated once they integrate the comments.

Since there have been no changes pushed to this PR since the comments were provided I am closing it. Of course the contributions would be much appreciated once they integrate the comments.


NDRS-894: handle downgrading the casper-node

Ref: https://casperlabs.atlassian.net/browse/NDRS-894
This PR updates the launcher to respect the casper-node exit code value of 102 meaning it requires a downgrade.
On running the node in either validator mode or migrate-data mode, if the node exits with 102 the launcher attempts to run the immediate previous installed version, and exits with an error if this fails.

bors merge


--dbPass should accept empty password

Hi, in my MySQL setup I have no password set for a user. But when I pass --dbPass "" it will still prompt for a password. If I just press enter and leave the password installation continue without problems. The problem is I want to use wireshell in a script without user interaction, so the password prompt is a trouble.

As the easy solution I see to ask for password only when no --dbPass options is passed, but not when the empty password is passed.

hi, I cannot find a possibility to determine whether an empty option has been passed or not.
$this->input contains the passed options array but this property is protected and I cannot access that protected property. All available methods check for a value or merge the options with the defaults:
/Symfony/Component/Console/Input/ArgvInput

getOptions()
Returns all the given options merged with the default values.

hasOption(string $name)
Returns true if an InputOption object exists by name.

hasParameterOption(string|array $values)
Returns true if the raw parameters (not parsed) contain a value.

getParameterOption(string|array $values, mixed $default = false)
Returns the value of a raw option (not parsed).

getOption(string $name)
Returns the option value for a given option name.

Hi, yes it looks like there is no suitable method for it. I made a little test and find out getOption returns NULL if the option is not present. Thus when the --dbPass option requires a value the empty password must be specified as an empty string. So you can just check if the password value is NULL.

I checked both cases and both returned Null, there wasn't any difference.
wireshell new foo --dbPass=""
wireshell new foo

var_dump($this->input->getOption('dbPass');
NULL

Really, I don't understand. It must behave differently for me. I'm on Windows, PHP 5.6.

I've just changed the line https://github.com/wireshell/wireshell/blob/master/src/Commands/Common/NewCommand.php#L167 to if (is_null($dbPass)) and it works as I need.

At the moment I'm not able to submit an empty string:
wireshell new foo --dbPass=""

[Symfony\Component\Console\Exception\RuntimeException]
The "--dbPass" option requires a value.

Therefore I changed the input option from required to optional:
-  ->addOption('dbPass', null, InputOption::VALUE_REQUIRED, 'Database password')
+  ->addOption('dbPass', null, InputOption::VALUE_OPTIONAL, 'Database password')

var_dump($input->getOption('dbPass'));
var_dump(is_null($input->getOption('dbPass')));

❯ wireshell new _foo --dbPass=""

NULL
bool(true)

❯ wireshell new _foo

NULL
bool(true)

I'm on Mac, PHP7, develop branch.

Ah I found the reason why you are getting another results. The parameter parsing behaves differently when you use --dbPass "" and --dbPass="". The first case is the one I've used. The second case is yours and will throw the exception as you mentioned. This looks like a bug in the Symfony's library.

Nope.
InputOption::VALUE_OPTIONAL
❯ wireshell new foo --dbPass ""

NULL
bool(true)

InputOption::VALUE_REQUIRED
❯ wireshell new _foo --dbPass ""

[Symfony\Component\Console\Exception\RuntimeException]
The "--dbPass" option requires a value.

Which version of symfony/console do you have installed?

^^

Maybe found the solution: https://github.com/symfony/symfony/issues/11572#issuecomment-197929086
$input->hasParameterOption('--dbPass'));

That's strange. I'm quite sure, I tested this yesterday. But now it seems to work!
❯ wireshell new foo --dbPass=""
bool(true)

❯ wireshell new _foo

bool(false)

I've tried it before as well, but I tried it this way: $input->hasParameterOption('dbPass') without --, but this don't work.

Will you update it? Or shall I make an PR?

I'll update it, but I'll add it to the develop branch. So it will be released with the next version.

OK fine 👍 . Will you update 0.6.0 version too, or will the new version support PW 2.x?

Added.
Unfortunately there won't be any 2.x support.

But could you make a hotfix for 0.6.0?

Actually I won't support older versions. Sorry.

OK, never mind. I made a patch myself.


Swift Package Manager support

Hi there 👋
We're using this component in our app but the iOS team are moving towards just using Swift Package Manager for managing dependencies.
So I've added a Package.swift file to this library via the initialisation steps and kept the same iOS support as the CocoaPod spec file and updated the README file.
Note. since it's Objective-C I've needed to mention where the public header files can be found in the Package.swift file and I had change the folder structure to remove the Demo project for some reason it produced module not found errors
I've tested integration here in a demo project https://github.com/madhikarma/BDKCollectionIndexViewSPMDemo and everything appears to be working.
Please take a look when you can and let me know what you think.
Thank you in advance

Thanks a ton for this, @madhikarma! I'll take a look hopefully today or tomorrow.

@madhikarma By moving the files around for SwiftPM support, you've broken CocoaPods support. Can you fix the .podspec so that it references the new paths?

Hi @kreeger sorry, I realised this after I authored the PR but looks like I forgot to add it to the PR description.
I will fix the Pod spec with the new path and test that the library can still be installed via CocoaPods as well as Swift Package Manager.
Q. Is there any other Pods related admin I need to do e.g. version bump? Pushing spec to the pods repo or is it just the path I need to change? (it's been a while since I authored a Pod)
Thank you for the other bits of code review / things to tidy up too
ETA: maybe sometime over this weekend

There's no need to do any version bumping or pushing to the spec repo! I can take care of that once this PR makes it in.


Add possibility to export sonarqube reports

Describe the solution you'd like
I'm pretty sure that sonarqube is used by almost every users that use kics.
For now, Sonarqube doesn't support the SARIF reports. May be kics could add a new output report style that follows sonarqube generic issue import format that will allow users to import kics analysis into sonarqube as any other static code analysis.
Describe alternatives you've considered

May be a complete Sonarqube plugin about SARIF reports could be considered
May be a simple way to transform SARIF reports in sonarqube generic issue import format could be useful and more easy to produce. You would just add another integration section in your docs.

Hello @LvffY
We will add a new report style that will be in json and be named sonarqube.
Regarding the structure, this is how it will look
{
    "issues": [
        {
            "engineId": "KICS v1.x.x",
            "ruleId": "<Query ID>",
            "severity": "<CRITICAL, MAJOR, MINOR, INFO>",
            "type": "<VULNERABILITY, CODE_SMELL>",
            "primaryLocation": {
                "message": "<Query Description>: <Query Expected Value>",
                "filePath": "<File path>",
                "textRange": {
                    "startLine": 1
                }
            },
            "secondaryLocations": [
                "<All Other Query Ocurrences>"
            ]
        }
    ]
}

Hello @joaoReigota1
That would be great !

thanks for adding this features, however, I am not able to import the report to SonarQube:

kics 1.7.0
SonarQube 10.0

kics scan -t Dockerfile -p Dockerfile -o . --report-formats sonarqube --output-name dockerfile-result
...
cat sonarqube-dockerfile-result.json
{
	"issues": [
		{
			"engineId": "KICS 1.7.0",
			"ruleId": "fd54f200-402c-4333-a5a4-36ef6709af2f",
			"severity": "CRITICAL",
			"type": "VULNERABILITY",
			"primaryLocation": {
				"message": "A user should be specified in the dockerfile, otherwise the image will run as root",
				"filePath": "Dockerfile",
				"textRange": {
					"startLine": 1
				}
			}
		},
		{
			"engineId": "KICS 1.7.0",
			"ruleId": "b03a748a-542d-44f4-bb86-9199ab4fd2d5",
			"severity": "MINOR",
			"type": "CODE_SMELL",
			"primaryLocation": {
				"message": "Ensure that HEALTHCHECK is being used. The HEALTHCHECK instruction tells Docker how to test a container to check that it is still working",
				"filePath": "Dockerfile",
				"textRange": {
					"startLine": 1
				}
			}
		}
	]
}

cat sonar-project.properties
sonar.projectKey=dockerfile-kics-sonarqube
sonar.sources=.
sonar.host.url=http://localhost:9000
sonar.token=sqp_XXX
sonar.externalIssuesReportPaths=sonarqube-dockerfile-result.json

sonar-scanner
sonar-scanner
INFO: Scanner configuration file: /opt/sonar-scanner/conf/sonar-scanner.properties
INFO: Project root configuration file: /dockerfile-kics-sonarqube/sonar-project.properties
INFO: SonarScanner <IP_ADDRESS>56
INFO: Java 11.0.19 Alpine (64-bit)
INFO: Linux 5.10.104-linuxkit amd64
INFO: User cache: /opt/sonar-scanner/.sonar/cache
INFO: Analyzing on SonarQube server <IP_ADDRESS>432
INFO: Default locale: "en_US", source code encoding: "UTF-8" (analysis is platform dependent)
INFO: Load global settings
INFO: Load global settings (done) | time=246ms
INFO: Server id: 147B411E-AYfYaESiZ-D8VVdPYgQ6
INFO: User cache: /opt/sonar-scanner/.sonar/cache
INFO: Load/download plugins
INFO: Load plugins index
INFO: Load plugins index (done) | time=146ms
INFO: Load/download plugins (done) | time=600ms
INFO: Process project properties
INFO: Process project properties (done) | time=40ms
INFO: Execute project builders
INFO: Execute project builders (done) | time=7ms
INFO: Project key: dockerfile-kics-sonarqube
INFO: Base dir: /dockerfile-kics-sonarqube
INFO: Working dir: /dockerfile-kics-sonarqube/.scannerwork
INFO: Load project settings for component key: 'dockerfile-kics-sonarqube'
INFO: Load project settings for component key: 'dockerfile-kics-sonarqube' (done) | time=35ms
WARN: SCM provider autodetection failed. Please use "sonar.scm.provider" to define SCM of your project, or disable the SCM Sensor in the project settings.
INFO: Load quality profiles
INFO: Load quality profiles (done) | time=97ms
INFO: Load active rules
INFO: Load active rules (done) | time=1981ms
INFO: Load analysis cache
INFO: Load analysis cache (404) | time=19ms
INFO: Load project repositories
INFO: Load project repositories (done) | time=31ms
INFO: Indexing files...
INFO: Project configuration:
INFO: 2 files indexed
INFO: Quality profile for json: Sonar way
INFO: ------------- Run sensors on module dockerfile-kics-sonarqube
INFO: Load metrics repository
INFO: Load metrics repository (done) | time=188ms
INFO: Sensor Import external issues report
INFO: Imported 0 issues in 0 files
INFO: External issues ignored for 1 unknown files, including: Dockerfile
INFO: Sensor Import external issues report (done) | time=64ms
INFO: Sensor JaCoCo XML Report Importer [jacoco]
INFO: 'sonar.coverage.jacoco.xmlReportPaths' is not defined. Using default locations: target/site/jacoco/jacoco.xml,target/site/jacoco-it/jacoco.xml,build/reports/jacoco/test/jacocoTestReport.xml
INFO: No report imported, no coverage information will be imported by JaCoCo XML Report Importer
INFO: Sensor JaCoCo XML Report Importer [jacoco] (done) | time=13ms
INFO: Sensor IaC CloudFormation Sensor [iac]
INFO: 0 source files to be analyzed
INFO: 0/0 source files have been analyzed
INFO: Sensor IaC CloudFormation Sensor [iac] (done) | time=116ms
INFO: Sensor IaC Kubernetes Sensor [iac]
INFO: 0 source files to be analyzed
INFO: 0/0 source files have been analyzed
INFO: Sensor IaC Kubernetes Sensor [iac] (done) | time=25ms
INFO: Sensor CSS Rules [javascript]
INFO: No CSS, PHP, HTML or VueJS files are found in the project. CSS analysis is skipped.
INFO: Sensor CSS Rules [javascript] (done) | time=3ms
INFO: Sensor C# Project Type Information [csharp]
INFO: Sensor C# Project Type Information [csharp] (done) | time=3ms
INFO: Sensor C# Analysis Log [csharp]
INFO: Sensor C# Analysis Log [csharp] (done) | time=50ms
INFO: Sensor C# Properties [csharp]
INFO: Sensor C# Properties [csharp] (done) | time=0ms
INFO: Sensor HTML [web]
INFO: Sensor HTML [web] (done) | time=10ms
INFO: Sensor TextAndSecretsSensor [text]
INFO: 1 source file to be analyzed
INFO: 1/1 source file has been analyzed
INFO: Sensor TextAndSecretsSensor [text] (done) | time=136ms
INFO: Sensor VB.NET Project Type Information [vbnet]
INFO: Sensor VB.NET Project Type Information [vbnet] (done) | time=4ms
INFO: Sensor VB.NET Analysis Log [vbnet]
INFO: Sensor VB.NET Analysis Log [vbnet] (done) | time=62ms
INFO: Sensor VB.NET Properties [vbnet]
INFO: Sensor VB.NET Properties [vbnet] (done) | time=1ms
INFO: Sensor IaC Docker Sensor [iac]
INFO: 0 source files to be analyzed
INFO: 0/0 source files have been analyzed
INFO: Sensor IaC Docker Sensor [iac] (done) | time=313ms
INFO: ------------- Run sensors on project
INFO: Sensor Analysis Warnings import [csharp]
INFO: Sensor Analysis Warnings import [csharp] (done) | time=4ms
INFO: Sensor Zero Coverage Sensor
INFO: Sensor Zero Coverage Sensor (done) | time=17ms
INFO: SCM Publisher No SCM system was detected. You can use the 'sonar.scm.provider' property to explicitly specify it.
INFO: CPD Executor Calculating CPD for 0 files
INFO: CPD Executor CPD calculation finished (done) | time=0ms
INFO: Analysis report generated in 319ms, dir size=124.7 kB
INFO: Analysis report compressed in 61ms, zip size=15.2 kB
INFO: Analysis report uploaded in 60ms
INFO: ANALYSIS SUCCESSFUL, you can find the results at: http://localhost:9000/dashboard?id=dockerfile-kics-sonarqube
INFO: Note that you will be able to access the updated dashboard once the server has processed the submitted analysis report
INFO: More about the report processing at http://localhost:9000/api/ce/task?id=AYfiakFzZ-D8VVdPYqPS
INFO: Analysis total time: 15.868 s
INFO: ------------------------------------------------------------------------
INFO: EXECUTION SUCCESS
INFO: ------------------------------------------------------------------------
INFO: Total time: 21.857s
INFO: Final Memory: 16M/60M
INFO: ------------------------------------------------------------------------

And nothing is uploaded to SonarQube 10, what am I missing? Is sonar.externalIssuesReportPaths the correct way to import the report?


Wait for test server to stop before continuing

On windows the Stop method will not wait for all OS resources to
be released. This will cause errors for the next test because
the port is already in use.
An error looks like this:
Administrator@WIN-MDJMHMVLENB MINGW64 ~/gowp/src/github.com/cloudfoundry/loggregator/src/metron (master)
$ ginkgo -r ./
[1442908548] EventWriter Suite - 2/2 specs •• SUCCESS! 0 PASS
[1442908548] NetworkReader Suite - 3/3 specs •panic: Failed to listen on port. listen udp4 <IP_ADDRESS>:3457: bind: Only                   one usage of each socket address (protocol/network address/port) is normally permitted.

goroutine 20 [running]:
github.com/cloudfoundry/gosteno.(*BaseLogger).Log(0xc082008d80, 0x857cd0, 0x5, 0x1, 0xc0822fa780, 0x98, 0x0)
...

@roxtar thx for the suggestion. I amended the commit.


Fix for pivoting empty datasets

Removes plugin crashes for time series plots with empty datasets. As in https://github.com/michelin/snowflake-grafana-datasource/issues/53

Thanks for you contribution.
I fixed this issue before the error was returned.
See #71


404 on clientresources/csp-app.js on Linux DXP

I've deployed to a Linux instance on DXP, when trying to load the Security page I get a 404 on the "clientresources/csp-app.js".

Seems like case sensitivity is causing an issue when hosted on linux?
.../EPiServer/Jhoose.Security/clientresources/csp-app.js  - Not Found
.../EPiServer/Jhoose.Security/ClientResources/csp-app.js - Capitalization on ClientResources works
Running locally on windows both resolve.

I have been doing some research about this.  The underlying FileProvider uses the underlying OS, and Linux is case-sensitive.
I will get a fix out for this over the next few days.
I think that this will catch a few people out.

A fix has been uploaded to the Optimizely nuget feed 1.1.2


Create point from device location

Make point from navigator location.

Thinking about this... That should be a 1hr job.
With the migration to the module (map, table, view) control structure the locate method is always available.
A method need to be added to this module to just return the lat lon of the location not toggle the display of the locator icon.
Then a new method needs to be added to draw. Whereas point asks for lat lon from map click the new method would ask for lat lon from locator and put the new location on map.
Done.

I added a locator draw method. https://github.com/GEOLYTIX/xyz/commit/d77ef07913171aff07555e28426a24daff22c1ea
In the edit block add locator:true. The 'Current Position' button will generate a point location on the current location as retrieved from the navigator API.


RenderPlanetarySky.drawStar Null Pointer Exception

Version of Advanced Rocketry
AdvancedRocketry-1.12.2-1.4.1-112-universal
Have you verified this is an issue in the latest unstable build
Y,N, N/A
Version of LibVulpes
LibVulpes-1.12.2-0.3.0-44-universal
Version of Minecraft
1.12.2
Crash report or log (if applicable)
https://pastebin.com/F75wrtGZ
Description of the problem
I'm not sure whats going on, but when I remove AR it works fine. If I load a local map it works, but if I join a server it freaks out.

Does this occur without sponge?  Does it occur with only AR installed?  If not, what's the minimum mod set required to replicate the issue?

I tried with only AR, it crashes. All other mods and sponge uninstalled. O.O

I tried with only AR, it crashes. All other mods and sponge uninstalled. O.O

Can you please Pastebin the crash report and post that here?
Keep in mind LibVulpes is required for AR.

Think I'm getting an identical issue.
https://pastebin.com/3dJL62ay

Translated translate.google.com
I have a similar error
Conducted tests on a clean server only from the AR and on the AR + Sponge
The problem occurs when using the sponge
On the main server, deleting the sponge also solved the problem.

This is the same mistake.
Client is crashing at server join #1415

Should be fixed by above commit


Implement Profile Screen

Description:
This issue involves implementing a simple profile screen with essential information.

Closed as fixed, please if you notice that the issue still exists open it again.


Correct path to CSS in readme

Hey, small documentation fix here. Thanks for making this!

Whoops!  Thanks for this.

No problem! Maybe it’s omgtoosoon to use this, I was wondering? In my project, I needed some extra CSS to get it working; specifically, I needed to set an explicit width and height and scale the background. I could make additional PRs to document that, or maybe this isn’t meant for public consumption yet and I got a supersecret advance preview in the Slack? haha

Hmm!  Let me look into that - they should default to 1em (and also I need to go look up what a rem is!).  The very first line of the generated CSS is:
.fa{display:inline-block;width:1em;height:1em;background-size:contain;}

As for being public-ready...  I don't know!  lol.  Having never put together something that people might actually want to use (outside of work), I don't really know how to deem it "ready."  There are several things I want to investigate to make it easier to use, but I feel like it ought to be nearly ready for actual use.  For what it's worth, I'm planning on rolling it into a site I maintain for a small non-profit.

oooh you know, I was having a CSS crisis where weird Javascript was inserted into the concatenated CSS. I have since fixed that and I get proper sizing when I remove my .fa-related rules! So disregard. Thanks. It does seem ready to use.

Sweet!  The biggest issue I have with widespread use is just the problem of coloring them however you want.  With Font-Awesome, it's easy: you set the color style because it's just a font.  Bit different here, though.


Reverts changes to default titlebar template and add a Window title to it

It is doing 2 things.

reverts #4035 Add custom titlebar to default Cs and Cpp WinUI template as there are changes coming later which will address some of the issues this PR was trying to solve, but better.
Adds window title in default template. As a result, every window will have a title by default, which would be name of the project (app).
Obviously, the user can choose to change it by using <Title> markdown in Window or Window.Title api, if they wish.

Solving https://task.ms/50309832

/azp run


Automate the release numbering

In GitLab by @arisilon on Oct 22, 2019, 16:16

In GitLab by @arisilon on Oct 30, 2019, 13:27
closed

In GitLab by @arisilon on Oct 29, 2019, 16:39
mentioned in commit 1a557c37b4425df26c48289c5f02599cc60ed017


Gemini exits following error messages in log

Issue description

[x] This issue is a regression.
[ ] It is unknown if this issue is a regression.

Looking at the Argus run, we can see that Gemini has exited before the end of the test:
2023-07-02 04:28:07.442: (GeminiStressEvent Severity.ERROR) period_type=end event_id=d1f2ede3-2301-4627-9342-109a98cadbd0 duration=47m50s: node=Node gemini-with-nemesis-3h-normal-maste-loader-node-3211fb00-1 [<IP_ADDRESS> | <IP_ADDRESS>] (seed: False) gemini_cmd=./gemini -d --duration 3h --warmup 30m -c 50 -m mixed -f --non-interactive --cql-features normal --max-mutation-retries 5 --max-mutation-retries-backoff 500ms --async-objects-stabilization-attempts 5 --async-objects-stabilization-backoff 500ms --replication-strategy "{'class': 'NetworkTopologyStrategy', 'replication_factor': '3'}" --oracle-replication-strategy "{'class': 'NetworkTopologyStrategy', 'replication_factor': '1'}" --test-cluster=<IP_ADDRESS>,<IP_ADDRESS>,<IP_ADDRESS> --outfile /gemini_result_edf4a8cd-ff67-4b3e-a2c2-635bc0f8b2cc.log --seed 80 --request-timeout 180s --connect-timeout 120s --oracle-cluster=<IP_ADDRESS>
result=Exit code: 1
Command output: ['{"L":"INFO","T":"2023-07-02T04:28:06.341Z","M":"test finished"}', 'gemini encountered errors, exiting with non zero status']
errors=['Command error: Error: gemini encountered errors, exiting with non zero status\n\n']

in this time frame, i can see in the logs 2 error messages, that are probably the reason for the Gemini to exit, although the logs of Scylla continued:
Jul 02 04:28:00 gemini-with-nemesis-3h-normal-maste-oracle-db-node-3211fb00-1 scylla[10299]:  [shard 1] compaction - [Compact ks1.table1_mv_0 d4371140-1890-11ee-8c9d-000000000001] Compacting [/var/lib/scylla/data/ks1/table1_mv_0-2f326790188a11ee8c9d000000000001/md-471-big-Data.db:level=0,/var/lib/scylla/data/ks1/table1_mv_0-2f326790188a11ee8c9d000000000001/md-473-big-Data.db:level=0]
Jul 02 04:28:06 gemini-with-nemesis-3h-normal-maste-oracle-db-node-3211fb00-1 scylla[10299]:  [shard 0] cql_server - exception while processing connection: seastar::nested_exception: std::system_error (error system:32, sendmsg: Broken pipe) (while cleaning up after std::system_error (error system:32, sendmsg: Broken pipe))
Jul 02 04:28:06 gemini-with-nemesis-3h-normal-maste-oracle-db-node-3211fb00-1 scylla[10299]:  [shard 1] cql_server - exception while processing connection: seastar::nested_exception: std::system_error (error system:32, sendmsg: Broken pipe) (while cleaning up after std::system_error (error system:32, sendmsg: Broken pipe))
Jul 02 04:28:06 gemini-with-nemesis-3h-normal-maste-oracle-db-node-3211fb00-1 scylla[10299]:  [shard 0] compaction - [Compact ks1.table1_col6_idx_index d34fe590-1890-11ee-bad0-000000000000] Compacted 2 sstables to [/var/lib/scylla/data/ks1/table1_col6_idx_index-2c31f8d0188a11eebad0000000000000/md-460-big-Data.db:level=0]. 71MB to 69MB (~98% of original) in 7104ms = 9MB/s. ~256 total partitions merged to 118.

Impact
Gemini exited, and later, failed to verify_results(), and test ended with error
How frequently does it reproduce?
i'm re-running the same job, using the same Gemini seed to confirm what happens, and will update
Installation details
Kernel Version: 5.15.0-1039-aws
Scylla version (or git commit hash): 5.4.0~dev-20230629.f6f974cdeb11 with build-id 7afc85749bdc68e7ee32eead35d51badd480c79f
Cluster size: 3 nodes (i3.large)
Scylla Nodes used in this run:

gemini-with-nemesis-3h-normal-maste-oracle-db-node-3211fb00-1 (<IP_ADDRESS> | <IP_ADDRESS>) (shards: 2)
gemini-with-nemesis-3h-normal-maste-db-node-3211fb00-3 (<IP_ADDRESS> | <IP_ADDRESS>) (shards: 2)
gemini-with-nemesis-3h-normal-maste-db-node-3211fb00-2 (<IP_ADDRESS> | <IP_ADDRESS>) (shards: 2)
gemini-with-nemesis-3h-normal-maste-db-node-3211fb00-1 (<IP_ADDRESS> | <IP_ADDRESS>) (shards: 2)

OS / Image: `` (aws: undefined_region)
Test: gemini-3h-with-nemesis-test
Test id: 3211fb00-4a70-4d4f-9232-1df186f67bf2
Test name: scylla-master/gemini-/gemini-3h-with-nemesis-test
Test config file(s):

gemini-3h-with-nemesis.yaml

Logs and commands

Restore Monitor Stack command: $ hydra investigate show-monitor 3211fb00-4a70-4d4f-9232-1df186f67bf2
Restore monitor on AWS instance using Jenkins job
Show all stored logs command: $ hydra investigate show-logs 3211fb00-4a70-4d4f-9232-1df186f67bf2

Logs:

db-cluster-3211fb00.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/3211fb00-4a70-4d4f-9232-1df186f67bf2/20230702_045901/db-cluster-3211fb00.tar.gz
sct-runner-events-3211fb00.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/3211fb00-4a70-4d4f-9232-1df186f67bf2/20230702_045901/sct-runner-events-3211fb00.tar.gz
sct-3211fb00.log.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/3211fb00-4a70-4d4f-9232-1df186f67bf2/20230702_045901/sct-3211fb00.log.tar.gz
monitor-set-3211fb00.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/3211fb00-4a70-4d4f-9232-1df186f67bf2/20230702_045901/monitor-set-3211fb00.tar.gz
loader-set-3211fb00.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/3211fb00-4a70-4d4f-9232-1df186f67bf2/20230702_045901/loader-set-3211fb00.tar.gz
parallel-timelines-report-3211fb00.tar.gz - https://cloudius-jenkins-test.s3.amazonaws.com/3211fb00-4a70-4d4f-9232-1df186f67bf2/20230702_045901/parallel-timelines-report-3211fb00.tar.gz

Jenkins job URL
Argus

reported https://github.com/scylladb/scylladb/issues/14474 to try to understand why we encountered this error in the oracle's logs

It failed due to the following reason:
{"L":"INFO","T":"2023-07-02T04:28:06.218Z","N":"work cycle.validation_job","M":"Retring failed validation stoped by reach of max attempts 5. Error: unable to load check data from the oracle store: system failed: gocql: no response received from cassandra within timeout period"}
Error detected: &joberror.JobError{Timestamp:time.Date(2023, time.July, 2, 4, 28, 6, 218489658, time.Local), Message:"Validation failed: unable to load check data from the oracle store: system failed: gocql: no response received from cassandra within timeout period", Query:"SELECT * FROM ks1.table1 WHERE col4=893580704621996895 AND col6=36 ALLOW FILTERING "}{"L":"INFO","T":"2023-07-02T04:28:06.218Z","N":"work cycle.validation_job","M":"ending validation loop"}

So basically request failed 5 times and gemini failed

@dkropachev , could you improve the log message in this case, by:

printing the max retry reach in a separated message
raise the log level of this message to ERROR?


[api] Print engine version and capabilities in debugEnvironment

Change-Id: If1c6f3b20b86b7cab1cafdbbb5ba5f976ec362c7
Description
Brief description of what this PR is about

If this change is a backward incompatible change, why must this change be made?
Interesting edge cases to note here

Codecov Report

Merging #1910 (cb1338c) into master (bb5073f) will decrease coverage by 2.19%.
The diff coverage is 67.10%.

@@             Coverage Diff              @@
##             master    #1910      +/-   ##
============================================
- Coverage     72.08%   69.89%   -2.20%
- Complexity     5126     5680     +554
============================================
  Files           473      543      +70
  Lines         21970    25071    +3101
  Branches       2351     2713     +362
============================================
+ Hits          15838    17523    +1685
- Misses         4925     6210    +1285
- Partials       1207     1338     +131

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

...nslator/InstanceSegmentationTranslatorFactory.java
14.28% <0.00%> (-3.90%)
:arrow_down:

.../cv/translator/SemanticSegmentationTranslator.java
0.00% <0.00%> (ø)

.../cv/translator/StyleTransferTranslatorFactory.java
40.00% <ø> (ø)

... and 444 more

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


Needless scroll bars for code example boxes

Both .highlight class and code tag has overflow: scroll;. They cause scrollbars to appear for code example boxes (actually double scrollbars).
See example here:
http://docs.phonegap.com/phonegap-build/configuring/
If you just remove this rules it will look fine... But maybe this is need elsewhere.
Oh, this looks the same both for Chrome and Firefox (at least on Windows).

Good call. I'll fix it.


Extra sorting action after resizing column.

Describe the bug
When using both useResizeColumns and useSortBy like Examples in Documentation, Drag 'resizer' will trigger Header click events and do sort every time. Bubbling?
Expected behavior
Drag 'resizer' to resize column, NO SORT.
Codesandbox!

You can solve this problem by taking different elements for header content on which you apply sort props and a sibling to it which applies resize props, something like https://codesandbox.io/s/tannerlinsleyreact-table-column-resizing-jel1b can help you.

☝ Well said. Apply the props to different elements.

I did not see resize working in this link: https://codesandbox.io/s/tannerlinsleyreact-table-column-resizing-jel1b

but it is working on my component. Thanks @gargroh

I just put the sort props inside a DIV tag (inside my TH tag), just like this:
<th {...column.getHeaderProps()}>
   <div
       {...column.getSortByToggleProps()}

 // My code and resizer div

   </div>
`</th>

and that worked for me


Bluetooth: controller: Fix assert on peer unknown rsp

Fixed an assert when peer responded with unknown rsp to
slave feature request when an existing another control
procedure was in progress.
This assert happened with a BT v4.0 peer implementation that
was performing a channel map update and local controller
initiated a slave feature request, receiving an unknown
response.
Signed-off-by: Vinayak Kariappa Chettimada<EMAIL_ADDRESS>

Hi,
I can confirm that this solved the Hard Fault I reported to the developers mailing list.
Kind regards,
Jehudi

@Laczen Thank you for the bug report and verification


try to unconditionally make $pandadir via .mkdir instead of &mkpath

mkdir on MoarVM already does the same thing as mkpath and does not fail in case of preexisting directories, so no need to check beforehand.
Fixes @mojca issues according to http://irclog.perlgeek.de/perl6/2015-12-30#i_11800996
Supersedes #264

Awesome, thank you!


President member pages are quite broken

The flag isn't working, a bunch of the grammar is weird, and the previously served lines link to other presidents.
Example: http://<IP_ADDRESS>/person/99902

But other than that, Mrs. Lincoln, how was the play?

Actually, it says lincoln served until after he was shot http://<IP_ADDRESS>/person/99884
Fixing this now.

If only they had some top rate actors like Mr. Booth.

@lukesonnet Your commit to fix people serving after death has an error; for people who have not died, Adam sets the died field to null (None in Python), and any numeric value is greater than that in a comparison. I fixed it in the member bio page, but just leaving this comment in case there's somewhere else that's using this data.


Update CMS schema

Update CMS.json Schema

Please squash your commits before so we can merge your PR :)

My bad, by resetting origin, it automatically closed the PR before i pushed my changes in one commit. I wanted to do it step by step to avoid mistake.


Native compliance phase

Fat finger here. Sorry

Paging @chef/docs-team for review on the documentation for these new resources

If this passes I vote merge. If we have additional tests here let's get an issue cut and merge those post 17.5


Update HpFeatures.astro

Improved spelling: Tunned should be Tuned

Thanks a lot!

Closed by #103
Thanks again!


Automatically set reviewed on file view

Related to scenario described here: #3513

I am in the process of reviewing a change to over 200 files. (yes, I know...)
I run VSCode with "workbench.editor.enablePreview": false (yes, I know.. I am a tab binger)

A "nice to have" would be an option to automatically mark as viewed when I open the file.  Maybe githubpullrequest.markAsViewedOnFileOpen, for this large checkin review a lot of workflow friction would be eliminated.
Thanks!

Hey @alexr00, is it alright if I implement this feature?

@joshuaobrien yes, I would accept a PR for this feature.

Hi @alexr00,
Seems like it's still open, I would like to take this up.

I've got a branch implementing some of this @umakantv, but I've been too busy to work on it. Let me know if it'd be helpful and I can upload it to my fork for you to take over

@joshuaobrien sure, let me know which branch, I would checkout and go through your changes, might be helpful.


Added api-log-level flag to log api calls

Description
Added api-log-level flag to give display the API call logs. It can be debug/info level.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: https://github.com/dapr/dapr/issues/4119
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[ ] Code compiles correctly
[ ] Created/updated tests
[ ] Extended the documentation

@amulyavarote once https://github.com/dapr/dapr/pull/4288 is merged, use that commit ref in the e2e test DAPR ref to override the latest version of DAPR in tests with the latest code.

Please add E2E tests exercising the api log level flag ... in tests/e2e/standalone ...


Make image info available through ResourceInfo

Description
The image info (layerCount, levelCount, sampleCount, extent, format, type)  was previously only available through ImageLayouts. This info is now also stored in ResourceInfo.
Storing this in ResourceInfo makes this info easily accessible from VkResourceRecords representing VkImages and VkImageViews. This simplifies upcoming changes to the image tracking and avoids the lock protecting the m_ImageLayouts map.
To simplify this duplication, a new struct ImageInfo is added here to hold the image info in both ResourceInfo and ImageLayouts.
This PR also updates ImgRefs to use the new ImageInfo class.

Thanks for catching the missing resInfo/sparse bindings change! This has been added.


Fix integration tests

Test plan

Green CI (at least 9/10 times 😅 )

Related to https://linear.app/sourcegraph/issue/CODY-1909/ci-tests-are-flaky#comment-da664388 (however this pr is about integration tests only that showed flakiness later)

oh irony...


Example from documentation doesn't work

I tried to run the example from https://authx.yezz.me/get-started/basic-usage/ and got an error:
INFO:     <IP_ADDRESS>:18932 - "GET /login?username=test&password=test HTTP/1.1" 200 OK
INFO:     <IP_ADDRESS>:18980 - "GET /protected HTTP/1.1" 500 Internal Server Error
ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\uvicorn\protocols\http\httptools_impl.py", line 409, in run_asgi
    result = await app(  # type: ignore[func-returns-value]
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\uvicorn\middleware\proxy_headers.py", line 60, in __call__
    return await self.app(scope, receive, send)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\fastapi\applications.py", line 1054, in __call__
    await super().__call__(scope, receive, send)
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\starlette\applications.py", line 113, in __call__
    await self.middleware_stack(scope, receive, send)
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\starlette\middleware\errors.py", line 187, in __call__
    raise exc
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\starlette\middleware\errors.py", line 165, in __call__
    await self.app(scope, receive, _send)
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\starlette\middleware\exceptions.py", line 62, in __call__
    await wrap_app_handling_exceptions(self.app, conn)(scope, receive, send)
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\starlette\_exception_handler.py", line 53, in wrapped_app
    raise exc
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\starlette\_exception_handler.py", line 42, in wrapped_app
    await app(scope, receive, sender)
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\starlette\routing.py", line 715, in __call__
    await self.middleware_stack(scope, receive, send)
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\starlette\routing.py", line 735, in app
    await route.handle(scope, receive, send)
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\starlette\routing.py", line 288, in handle
    await self.app(scope, receive, send)
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\starlette\routing.py", line 76, in app
    await wrap_app_handling_exceptions(app, request)(scope, receive, send)
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\starlette\_exception_handler.py", line 53, in wrapped_app
    raise exc
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\starlette\_exception_handler.py", line 42, in wrapped_app
    await app(scope, receive, sender)
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\starlette\routing.py", line 73, in app
    response = await f(request)
               ^^^^^^^^^^^^^^^^
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\fastapi\routing.py", line 291, in app
    solved_result = await solve_dependencies(
                    ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\fastapi\dependencies\utils.py", line 638, in solve_dependencies
    solved = await call(**solved_result.values)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\authx\main.py", line 619, in _auth_required
    return await self._auth_required(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\authx\main.py", line 351, in _auth_required
    request_token = await method(
                    ^^^^^^^^^^^^^
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\authx\main.py", line 308, in get_access_token_from_request
    return await self._get_token_from_request(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\authx\main.py", line 281, in _get_token_from_request
    return await _get_token_from_request(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\nawas\PycharmProjects\fastapi_playground\.venv\Lib\site-packages\authx\core.py", line 156, in _get_token_from_request
    raise MissingTokenError(*(str(err) for err in errors))
authx.exceptions.MissingTokenError: Missing 'Bearer' in 'Authorization' header.

I believe your second request is just curl -s http://<IP_ADDRESS>:8000/protected . You used this because it is under the Protected Routes - curl-without-jsonwebtoken section.
The error is on purpose, as it demonstrates an unauthorized request to a protected endpoint. If you want to see response to an authorized request, you should use the example under the tab "With a valid JsonWebToken".
This behaviour is actually explained under that very section:

In the curl requests above, a 401 HTTP Error is raised when the token is not valid. By default, AuthX triggers a 500 Internal Server Error HTTP Error. For the sake of simplicity, we won't delve into error handling in this section.

Example of a valid request (access token will be different in your case):
curl -s --oauth2-bearer eyJhbGciOiJIUzI1NiIsInR5cCI6...9vJcIWEE http://<IP_ADDRESS>:8000/protected

(I just tried it in local just to be sure that it still works. response: {"message":"Hello World"} )

Hello @John2013 Yes I'm working on a big refactoring for the documentation!
cc @gokhanmeteerturk in case you want to open a Pull request


Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
Analytics
Which platforms are affected?
iOS
Description
Added firebase to my new flutter project using firebase CLI and it all runs fine with remote config and core:
dependencies:
flutter:
sdk: flutter
firebase_core: ^3.4.0
firebase_remote_config: ^5.1.0
as soon as I add firebase_analytics it fails to build
dependencies:
flutter:
sdk: flutter
firebase_core: ^3.4.0
firebase_remote_config: ^5.1.0
firebase_analytics: ^11.3.0
void main() async {
WidgetsFlutterBinding.ensureInitialized();
await Firebase.initializeApp(
options: DefaultFirebaseOptions.currentPlatform,
);
runApp(const MainApp());
}
Error:
Failed to build iOS app
Error (Xcode): Undefined symbols:
Error (Xcode): Linker command failed with exit code 1 (use -v to see invocation)
Could not build the application for the simulator.
I've tried so many things, been through it all with different AI, recommendations on stackoverflow, nothing works. If I build with xcodebuild -verbose  it says:
fatal error: module
'firebase_analytics' not found
@import firebase_analytics;
I've tried this: https://stackoverflow.com/questions/66685348/flutter-fatal-error-module-firebase-analytics-not-found and also this:
Remove ios/Pods folder.
Copy podfile and keep it outside of your directory.
Remove podfile and podfile.lock
Remove pubspec.lock .
Run pod cache clean —all in terminal
Run flutter clean in terminal
Run flutter pub get in terminal
Copy the back up podfile, to newly regenerated podfile.
Run pod install in terminal
Build your project. Once successful, then archive.
Reproducing the issue

Install a new flutter project for iOS
Create ios project at firebase console
add dependencies   firebase_core: ^3.4.0  firebase_remote_config: ^5.1.0  firebase_analytics: ^11.3.0
Add the google info plist into the xcode project.
Run flutterfire in project's terminal to create the files required by firebase for flutter.
add pod 'Firebase/Analytics' into target 'Runner' in podfile, run pod install etc
Try to run or build app.

Firebase Core version
3.4.0
Flutter Version
3.24.2
Relevant Log Output
fatal error: module 'firebase_analytics' not found
@import firebase_analytics;
 ~~~~~~~^~~~~~~~~~~~~~~~~~
1 error generated.

Flutter dependencies
dependencies:
flutter:
sdk: flutter
firebase_core: ^3.4.0
firebase_remote_config: ^5.1.0
firebase_analytics: ^11.3.0
Additional context and comments
No response

What version of Xcode are you using?

Hi @hasen6 , I've reopened an older issue that is similar to this. I'll go ahead and close this one. 13277

What version of Xcode are you using?

15.0.1


🎮 Game Request
Pariah is set in a world where the destiny of humanity it's on the hands of one
man that was rejected by society, labeled as crazy and locked up into a
dungeon. Is the story of the last man in earth.
Point down the features
Controls: Arrows to move. 1, 2, 3 and 4 to fire elements
Select program in which you are contributing
GSSoC23
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @S-ishita !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @S-ishita ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


disable libcapng by default during build

disable libcapng by default to provide a way deal with inconsistencies in system packages installed in various linux distros

Please remove the --disable-libcapng at line 181.
Oh, I should have mentioned ... you need to sign your commits. (git commit -s). If you don't the DCO check can fail. I can override it this time, but you may find me less merciful in the future.

noted. (for next time)


SBT 0.11.2 Does Not Support Older Versions of Git?

Was trying to run an SBT build on an Amazon EC2 machine. The only version of Git that existed for this machine type was <IP_ADDRESS>-3.fc8 and it kept failing while trying to get from a git:// URL because it was trying to clone into an existing directory.
Is this as designed?

Quite likely a case of a very old git client. Don't remember exact version, but old git clients did have problem with git protocol over ssh.

I tried with a newer version prior to filing this issue and it worked, which is why I know this is the issue. However, nothing in the documentation indicates that an older version of git is not supported.
What is the oldest version of git supported by SBT 0.11.2?


Support Drag file to  HexFiend

Drag a file to HexFiend,then it can open the file.
It's inconvenient to use Open File or Open Path Command.

Are you dragging it into a Hex Fiend window? You can drag it onto the icon (in the Dock, or in Finder).

Yes, dragging into the window would be nice.
it's literally the only reason I use iHex over Hex Fiend.

Ok, seems reasonable.

This already works for me on 2.3.0. I can drag files onto the app icon in the Dock, and HF windows.
If it's not working, please provide more specifics, such as the exact type of file you're using.


Support for xml for test-authoring

XML test step client #9
In addition to CSV, and Excel support XML for test-authoring.

implemented in XML test step client #9


Add repography.com dashboard to README

Just for fun and statistics

https://repography.com/app/0/clap-rs/clap/recent-activity


calculate reg value in refresh_rate()

Calculate it arithmetically (base2 log, add/sub 5) instead of using the more memory hungry tables.

This was fixed in 1a49243732a13b386c1d7e81f27d21b00b64df98


ydl still aborts playlist download on a 403 error

When downloading a playlist from youtube if a 403 error happens (for example because that video is not available in yout country) the download is stopped.
It should continue or have an option to skip the item on error.
I though I already reported this like one or two years ago. But the problem is still there.
[download] Downloading video 13 of 100
[youtube] fPziOGGqEpw: Downloading webpage
WARNING: Requested formats are incompatible for merge and will be merged into mkv.
ERROR: unable to download video data: HTTP Error 403: Forbidden

and it stops there.

-i, --ignore-errors                  Continue on download errors, for
                                     example to skip unavailable videos in a
                                     playlist

Use the in-browser search feature on the readme, and don't skip the ticket template in these cases.

Another curious thing is that it happens every 13 elements.. hmmm.. maybe it happens because of something else.
You can try yourself:
youtube-dl -c -o '%(title)s.%(ext)s' "https://www.youtube.com/playlist?list=PLDIoUOhQQPlWvtxdeVTG3i7-SlSN0jfWj"

Sorry @rautamiekka , it's not only a problem of ignoring the error. but it is strange I get the error on that playlist exactly every 13 downloads.


Better error handling for presto

Issue: https://github.com/airbnb/superset/issues/1859
Before:

After:

@mistercrunch @bkyryliuk @ascott

LGTM


RegExp.lastIndex writeable can be set to false fixes:#238

RegExp instances have the property "lastIndex" which by default is: writeable but not configurable. However it should be possible with Object.defineProperty to change it to not writeable.
This would not previously work as RegExp::IsWriteable was hard coded to return true for the lastIndex property.
This fixes the issue by adding a boolean to track if lastIndex has been set to read only.
fixes:#238
Edit: this hasn't quite worked (it's my own test case for this issue that's failing) - I'll take another look; it passed offline on macOS - and as can be seen below it passes on some variants but not all within the CI.

Is there anything different that runtests.cmd does for each test that can't be replicated by calling ch manually?
Called from runtests.cmd my test case passes in the interpreted variant but fails in the dynopogo variant BUT with runtests.py it passes in both. Additionally simply copying the flags that runtests.cmd uses for dynopogo and calling ch with them the test case passes.
Flags for reference
-WERExceptionSupport -ExtendedErrorStackForTestHost -BaselineMode
-forceNative -off:simpleJit -bgJitDelay:0 -dynamicprofileinput:profile.dpl.UnnamedTest106
-args summary -endargs

There's probably also a corresponding set of test262 tests that this fix would cause to pass.

@rhuanjl The test runner prints out the actual command it uses to run the tests. There's nothing else fancy going there with the test runner scripts. If the dynamic profile file created by the interpreter pass doesn't exist then it will not be used and will run as if it were not there.

Wait, is this right?  I thought the point of configurable: false is that you can't change any of the attributes from what they currently are.

@fatcerberus Somewhat counter-intuitively, you can go from writable: true to writable: false on a non-configurable property.
https://tc39.github.io/ecma262/#sec-validateandapplypropertydescriptor Step 7.a.

Sorted a repo for the test fail - the test fails in dynopogo mode when there is a dynamic profile file. Without a dynamic profile it does not fail.
I guess that with the profile available the JIT attempts to add one or more optimisations that it doesn't without? And I suppose one of those optimisations somehow breaks this test case - possibly by optimising away the call the JavascriptRegExp::IsWritable? I don't know how to deal with this at the moment.

Moving the test case into a function and then calling it multiples in the script and running with:
-forceNative -off:simpleJit -bgJitDelay:0 -mic:1 is not sufficient to trigger the fail. (I've even gone up to calling the function from a loop with 10000 iterations.
I can only make this fail happen with a dynamic profile input file.

The problem there is the -forceNative flag, which overrides mic:1. mic:1 says to run in the interpreter once, collecting profile data, before compiling the function, but -forceNative says to compile right away without waiting to collect any profile data. I can see the problem with the following code, run with -mic:1 -off:simplejit. I doubt that this issue was introduced by your change; it seems that the function is running in sloppy mode in some cases.
const assert = {
    throws(x) { let err = false; try { x(); } catch (e) { err = true; } if (!err) throw new Error(); }
}

function t() {
    const re = /./;
    Object.defineProperty(re, "lastIndex", {writable : false});
    assert.throws(()=>{"use strict"; re.lastIndex = 10}, TypeError, "TypeError for writing to non-writeable property in strict mode");
}

t();
t();

@sigatrev it seems that inlining a callback is losing strictness; does that seem plausible to you?

With the profile data the strictness isn’t the only problem.
Running it non-strict the property actually gets overwritten even though it’s not meant to be writable - I was wondering if the JIT was using some different writeable check and not the RegularExpresdion::IsWriteable method which I updated.

Whilst my guess was right I think fixing this would:
a) be beyond me (I’ve tried to learn how your JIT works by reading the code but i’ve not got very far) AND
b) probably result in a performance regression
Considering how low priority a bug this must be (#238, open for well over a year) should I just close this PR?

I agree it's probably a performance hit, since there's a special function GenerateFastStFldForCustomProperty for this exact case (storing lastIndex on an object that IsLikelyRegExp). There are probably ways to limit the performance impact, but they're also probably complicated. I'd wait to hear from somebody on the JIT team in case there's a simple solution, but if not, I think the current consistently-wrong behavior is better than being sometimes right depending on whether a particular optimization is present.

We could also just disable the optimization and run through octane/sunspider/speedometer/regex nightlies to see what the difference is.

I suspect the cleanest solution would be to keep this optimization but add another step that looks something like this, before the MOVs.
cmp byte [object + offset(readOnlyLastIndex)], 0
je $helper

I talked with @Penguinwizzard offline and he said it's likely not worth the perf hit to make this spec-compliant.

Thanks for the help - sorry for the wasted effort. Perhaps the help wanted tag from #238 should be removed and a note added that this isn't to be done at the moment?


@kanaabe => [Venice] Routes use slug

Video index checks curation to find a matching section, otherwise returns the first section

Looks good!


I cannot use react navigator in the onRead function

What's happening?
I am trying to navigate the user to a different screen when onRead is successful. But throws an error saying TypeError: undefined is not an object (evaluating 'navigation.navigate')
How can it be reproduced?
Try to use react-navigation in the function of onRead to reproduce this error.
Build details?
I am using the latest version of react-native-qrcode-scanner on a physical device running android.
My code
import React from 'react';
import QRCodeScanner from 'react-native-qrcode-scanner';
import {RNCamera} from 'react-native-camera';
import {Text, View} from 'react-native';
import {globalStyles} from '../styles/global';
import {useDispatch} from 'react-redux';
import {startUsing} from '../redux/actions/deskActions';

const Start = ({navigation}) => {
  const dispatch = useDispatch();
  const onSuccess = e => {
    // dispatch(startUsing(e.data));
    console.log(e.data);
    navigation.navigate('Home');
    scanner.reactivate();
  };

  return (
    <View style={globalStyles.container}>
      <View style={{...globalStyles.center, marginVertical: 50}}>
        <Text style={{...globalStyles.titleText, marginBottom: 20}}>
          Scan the QR Code to start session
        </Text>
        <QRCodeScanner
          reactivate={true}
          reactivateTimeout={2000}
          showMarker={true}
          ref={node => {
            scanner = node;
          }}
          onRead={onSuccess}
          flashMode={RNCamera.Constants.FlashMode.auto}
          cameraStyle={{width: '80%', marginHorizontal: 20, height: '100%'}}
        />
      </View>
    </View>
  );
};

export default Start;

Technically this should work if your Start Screen is defined on the same navigation container with your Home Screen!
const AppNavigator = () => {
const Stack = createStackNavigator();
return (
<Stack.Navigator>
<Stack.Screen
name={'Home'}
component={HomeScreen}
options={{ headerShown: false }}
/>
<Stack.Screen
name={'Start'}
component={ScanScreen}
options={{ headerShown: false }}
/>
</Stack.Navigator>
);
};
you should have a configuration like that if you want to navigate to your Home screen
By the way if you have nested navigation, and you are using react navigation v5, you should do it like that navigation.navigate('Name of your navigation ', {screen : 'Home'})
Hope that help you!

This is what my navigation container looks like
import React from 'react';
import {createStackNavigator} from '@react-navigation/stack';
import {NavigationContainer} from '@react-navigation/native';
import Home from '../screens/Home';
import Stop from '../screens/Stop';
import Start from '../screens/Start';

const Stack = createStackNavigator();

const AuthStack = () => {
  return (
      <Stack.Navigator
        screenOptions={{
          headerShown: false,
        }}>
        <Stack.Screen name="Home" component={Home} />
        <Stack.Screen name="Stop" component={Stop} />
        <Stack.Screen name="Start" component={Start} />
      </Stack.Navigator>
  );
};

export default AuthStack;

I forgot to mention that you should add an arrow function to your onPress event
It should look like onPress = { () => onSucces() }
I reproduce that and it works perfectly! Try to add the arrow function on your OnPress event and let me know if it works!

I forgot to mention that you should add an arrow function to your onPress event
It should look like onRead = { () => onSucces() }
I reproduce that and it works perfectly! Try to add the arrow function on your onRead event and let me know if it works!

I tried this but this doesn't work too.

if you're user react navigation v5 you can try using hook
`
...
import { useNavigation } from '@react-navigation/native';
const navigation = useNavigation()
const Start = () => {
const dispatch = useDispatch();
const onSuccess = e => {
// dispatch(startUsing(e.data));
console.log(e.data);
navigation.navigate('Home');
scanner.reactivate();
};
...
`

if you're using react-navigation v5 you can try using hook

Yay! This is working as expected. Thanks a lot for helping. 😄

Glad that help you!
Happy Coding @manan2501 !


Should redirect to MainView if no request/gift is not found

Better have a 404 Not Found image returned from the back end if no such a request/gift can be found.


Added enhanced vpc routing

what

Just a simple addition to enable enhanced vpc routing. Disabled by default.

why

I needed enhanced routing for my redshift cluster

references

https://docs.aws.amazon.com/redshift/latest/mgmt/enhanced-vpc-routing.html

oops, you can do this via redshift


J2735 shared object library

Hi,
By using the Makefile in /examples/sample.source.J2735 I can get a libasncodec.a (static) library. Is there a way I can get a shared object library (.so)?
Thanks!
Joseph

@josephquzhou , there are static variables used in several places of skeleton files, e.g. #283, and function asn_bit_data_string() of asn_bit_data.c. So be cautioned that the built library might not be thread-safe. Also the generated C files shall be checked for it.
Of course if you use your built shared object library (.so) only within a single thread application. Thread-safety perhaps can be compromised.
If the aforementioned issues are not your concern, then :

Add -fPIC to CFLAGS of Makefile.
Add one line below $(AR) ..... of converter-example.mk.

$(ASN_LIBRARY): $(ASN_MODULE_SRCS:.c=.o)
	$(AR) rcs $@ $(ASN_MODULE_SRCS:.c=.o)
	$(CC) -shared -o $(ASN_LIBRARY:.a=.so) $@ $(ASN_MODULE_SRCS:.c=.o)

@brchiu,
After I did the two changes you recommended, the libasncodec.so was successfully generated. However the .so library couldn't be used for compiling. Is there anything else I should do?
Thanks!
Joseph

I am not sure your "the .so library couldn't be used for compiling" means.
I assume it is a result that compiler is not aware where this .so is.
If it is indeed the case, you can add -L<directory name> in your command line to instruct gcc adding library search path and add -l<library name without lib and .so> to instruct gcc this dynamic library name.
gcc -L/home/username/foo -Wall -o test main.c -lfoo
All these information can be easily found by searching internet using proper keywords which already exist in your questions. For example, the following link found :
https://www.cprogramming.com/tutorial/shared-libraries-linux-gcc.html


Extension adder not working

the extension adder does not work, please fix if you have time?

how tf did you make a issue

Idk I pressed a button labeled post issue or something like that

Use the discussions instead


Fix: flat sprite texture

keep w-ordinate for correct texture perspective

Nice fix, this is smart :)


Add image processor vilt

What does this PR do?
Add Vilt image processor

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.