Add Shell script linter to GitHub Actions

[] Do only one thing
[] Non breaking API changes
[] Tested

What did this pull request do?

Add a linter for shell script code
No production code change

User Case Description

Hello @ashishb
Thank you for your PR, but I think it should be fine to live w/o this linter, I don't expect any shell script will be changed...


Sketch 51.2 at Mac OS Mojave thumbnail bug

Sketch 51.2 at Mac OS Mojave thumbnail have a bug, OS will not render the thumbnails

what do you mean by thumbnail?

Blank space for Sketch file, unable to load preview. At the same time, the system will not generate all thumbnails including .sketch .png and .pdf.  Thumbnails refer to the "show icon preview".

Oh I see. We are working on fixing the quick look bug. I'm going to close this issue since it's not really about Sketch API.
Thanks for the report


OWIN ClaimsTransformation get called for each request

Using the current ClaimsTransformationMiddleware the Transform function is called for each request, including resources like scripts/content in a MVC5/Nancy app.
Before OWIN the claims transformation would happen only when creating the user identity.
Is there a way to hook the claims transformation to the authentication phase?

Hi Andrea, i'm running into a situation like you described. I want to renew the claims of the user after it's been authenticated?? I'm having this problem and i don't know which way to go. I want, for example, that my user's token lives for 3 hours but the claims (from a DB) get renewed every hour. Is this possible? Thanks!

It's called caching. You would implement that in your transformation logic.
Sent from my iPhone

On 25.03.2015, at 20:27, airomero<EMAIL_ADDRESS>wrote:
Hi Andrea, i'm running into a situation like you described. I want to renew the claims of the user after it's been authenticated?? I'm having this problem and i don't know which way to go. I want, for example, that my user's token lives for 3 hours but the claims (from a DB) get renewed every hour. Is this possible? Thanks!
—
Reply to this email directly or view it on GitHub.

Yes i understand that. My problem is to have different lifetime for the
authentication token and the claims. How do i do that? Also,  where is the
right place for doing the claims transformation since i need it to be
called multiple times (every hour). I want the claims transformation to run
on the authentication and then every hour.  Thanks!
El mar 25, 2015 5:22 PM, "Dominick Baier"<EMAIL_ADDRESS>escribió:

It's called caching. You would implement that in your transformation
logic.
Sent from my iPhone

On 25.03.2015, at 20:27, airomero<EMAIL_ADDRESS>wrote:
Hi Andrea, i'm running into a situation like you described. I want to
renew the claims of the user after it's been authenticated?? I'm having
this problem and i don't know which way to go. I want, for example, that my
user's token lives for 3 hours but the claims (from a DB) get renewed every
hour. Is this possible? Thanks!
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/IdentityModel/Thinktecture.IdentityModel/issues/33#issuecomment-86201926
.

Check the cache on every request - create the entry on the first, update every time an hour has passed by?

Dominick, if i understand, i could transform claims on login
(CookieAuthenticationOptions -> OnResponseSignIn), then i could check on
every request if the claims should be renewed from the DB, and with a
custom filter on my MVC actions i could check if i have the appropriate
claims to access that action in my controller. Is it possible to set a
expiration time on the cookie and force the user to sign in again? I've
googled it a lot and haven't found an answer that really works (i'm using
ws-federation).
2015-03-26 4:29 GMT-03:00 Dominick Baier<EMAIL_ADDRESS>
Check the cache on every request - create the entry on the first, update
every time an hour has passed by?
—
Reply to this email directly or view it on GitHub
https://github.com/IdentityModel/Thinktecture.IdentityModel/issues/33#issuecomment-86370267
.

Ah I see - you are using it with cookie middleware. Well in that case you need to issue a new cookie when you want to update claims. You can do that by constructing a new ClaimsIdentity and call SignIn on the Katana authentication manager.

I'm sorry to say this, but how do i do that? I construct the claims right
after the user logs in, inside the claims i create a claim with the
authentication time, then on every request i check if an hour has passed
since that authentication time. How could be done what you tell me? Can´t
it be done with some expiration time set in the cookie?? Thanks!
2015-03-27 4:14 GMT-03:00 Dominick Baier<EMAIL_ADDRESS>
Ah I see - you are using it with cookie middleware. Well in that case you
need to issue a new cookie when you want to update claims. You can do that
by constructing a new ClaimsIdentity and call SignIn on the Katana
authentication manager.
—
Reply to this email directly or view it on GitHub
https://github.com/IdentityModel/Thinktecture.IdentityModel/issues/33#issuecomment-86851770
.

Simply overwrite the cookie when the time has come. Use the AuthenticationManager.SignIn method on the OwinContext for that.

Thanks Dominick, i'll look into it!


DataVec build failure while using ND4J built with Jackson 2.8.7

I have built ND4J with Jackson version 2.8.7 on windows (with CUDA backend). The ND4J build with Jackson 2.8.7 completes successfully. When I try to build DataVec with the mentioned ND4J, build fails with error in datavec-api. A gist of the build log is shared at https://gist.github.com/ynpandey/2d66ed3a9394bc2c7c3056061636e3ba
Version Information

Deeplearning4j version: 0.8.1
platform information (OS, etc): Windows 7
CUDA version, if used: 8.0
NVIDIA driver version, if in use: NVIDIA graphics driver 376.51 (Graphics card: NVIDIA Quadro K5000)

@AlexDBlack Was there a reason to use different versions of Jackson (2.5.1 in ND4J and 2.8.7 in DataVec)?

Just to add, in the deeplearning4j dependencies, there are two different versions of Jackson defined.
<jackson.version>2.5.1</jackson.version> <spark.jackson.version>2.4.4</spark.jackson.version>
That makes it 3 different versions of Jackson 2.4.4, 2.5.1, and 2.8.7 in the DL4J stack.

Update: with the <jackson.version> defined as 2.5.1 in nd4j/pom.xml, 2.6.0 in nd4j/nd4j-parameter-server-parent/nd4j-parameter-server-client/pom.xml, and 2.8.7 in DataVec/pom.xml, the project datavec-api builds successfully.
I don't understand why having a consistent <jackson.version> 2.8.7 causes a build failure? Can someone please provide some information/help about this issue?

As far as I can tell, the only places jackson is directly used in DataVec are:
(a) in datavec-dataframe, which should be ok (it's essentially independent)
(b) in datavec-spark, which forces version 2.8.7 for some reason (we probably want to remove these dependency overrides)
Now, neither should have any impact at all on the shaded jackson code (and hence that error message).
It looks a lot like there was a Jackson API change between 2.5.1 and 2.8.7 that is leading to this:
[ERROR] /C:/dl4j-0.8.1/DataVec/datavec-api/src/main/java/org/datavec/api/transform/TransformProcess.java:[438,58] incomp
atible types: java.lang.Class<capture#1 of ?> cannot be converted to org.nd4j.shade.jackson.databind.JavaType

i.e., same method name, different return type.
Because of that API change, we'd probably need some (possibly only minor) changes in DataVec - and possibly ND4J/DL4J too.

@AlexDBlack I am in the process of integrating DL4J capabilities with Oryx framework. Latest version of Oryx 2.x uses Jackson version 2.8.7. For maintaining compatibility with Oryx APIs, Jackson 2.8.7 support is needed in DL4J stack.

Is this issue still viable?


Enable InfluxDB2 tag support

InfluxDB 2.x supports tagging which can be enabled on the iobroker.influxdb adapter as described here.
However, it seems the queries that ECharts uses are incompatible as no data can be retrieved, resulting in empty charts.
Could this be implemented?

ECharts is using getHistory calls, so please open this issue in InfluxDB adapter.
Please enable extended debug log for one example datapoint with a new 3.x Influxdb version(Beta repository) and also change the loglevel of the influxdb instance to debug. Attach that log from such a echarts request (getHistory)


Including the integration id

The model is not up to date and can hamper several developments.

Thanks for the report! The contents of this repository are auto-generated, so the changes in your PR would be overwritten next time we generate a release. I will file this issue internally so that it can be handled in a reusable way


Improve prognostic implicit edmf performance

This issue is for tracking the performance of the prognostic implicit edmf performance.

I looked at the nvtx report from #2947, and this is what I'm seeing:
Here is a full step! (labeled by ClimaTimeSteppers on the left), for the big picture view:

The 3 "large" cpu calls to remaining_tendency! / hyperdiffusion_tendency! are the ones that follow the implicit solve. They only appear large because many kernels are launched during the implicit solve, which pile up in the gpu queue, and there's a CUDA.synchronize call in hyperdiffusion_tendency!'s DSS call.
Let's zoom in to see who is responsible for these kernel launches. I'll click on the cpu functions (labeled ClimaAtmos), and the kernels (99.9% Kernels (named by NVTX) above will highlight:
dss!:

set_precomputed_quantities!

wfact!:

implicit_tendency!:

ldiv!:

The one after that is another call to set_precomputed_quantities!.
Summary
Some things are obvious and clear (for a single A100 gpu):

Most of the gpu time is spent in ldiv!.
Next comes set_precomputed_quantities!, not far behind is wfact!, then implicit_tendency!, then dss!
The single most expensive kernel is the call to buoyancy_gradients (in set_precomputed_quantities!), which takes about 8 ms

Perhaps fortunately, ldiv! basically calls MatrixFields.field_matrix_solve!(A.solver, x, A.matrix, b), so the majority of the performance improvements can be done in ClimaCore, hopefully without even requiring changes in ClimaAtmos.

Summary
Recap of total duration of gpu launched kernels:

dss! (2.7 ms)
set_precomputed_quantities! (22.7 ms)
wfact! (18.2 ms)
implicit_tendency! (7.4 ms)
ldiv! (107.4 ms)

Some things are obvious and clear (for a single A100 gpu):

Most of the gpu time is spent in ldiv!.
Next comes set_precomputed_quantities!, not far behind is wfact!, then implicit_tendency!, then dss!
The single most expensive kernel is the call to buoyancy_gradients (in set_precomputed_quantities!), which takes about 8 ms

Perhaps fortunately, ldiv! basically calls MatrixFields.field_matrix_solve!(A.solver, x, A.matrix, b), so the majority of the performance improvements can be done in ClimaCore, hopefully without even requiring changes in ClimaAtmos.


sample code for making external http calls  from extensions

Hi:
Do we have any tutorials , sample code on how to make external http calls from the extensions code.
For example i was able to add headers using the assemblyscript extension  code.  How can i make external http calls etc
Thanks

hi, you can see a c++ example here:
https://github.com/envoyproxy/envoy-wasm/blob/dfb104874b3babc7e676aaa0199f2926410c0671/test/extensions/filters/http/wasm/test_data/async_call_cpp.cc#L15

as for assembly script, there's a PR in progress that will also has an example it in

Thanks.  I progressed to some extent -  I am always ending up with a BAD_ARGUMENT HTTP call wasm return value --  I have pasted details of the assemblyscript code used and also the envoy wasm context.cc which is throwing this error. Unable to figure out what is missing.
*********  sample  **********
I have a standard istio bookinfo sample deployed.  Added a petstore pod additionally where i inject the below wasm extension .   It gets invoked and behaves as described.  So when i make a POST call to petstore addPets -  the filter gets invoked and within that I do a HttpCall to "details" service in the bookinfo sample (just to see if i can make a external http call)
Do i need to also add the cluster referenced in the httpcall to the envoyfilter as described here ? -- https://github.com/istio/istio/wiki/EnvoyFilter-Samples

Env = istio 1.5.1 , Kubernetes 1.17.3 , minikube
Using assemblyscript = 0.9.4 and solo-io / proxy-runtime 0.1.6

add_header istio wasm extension  using the webassemblyhub to publish and then deploy using wasme
`
class AddHeader extends Context {
onRequestBody(blen: size_t, estrm: bool): FilterDataStatusValues {
let headers : Headers = []
let pair : HeaderPair = new HeaderPair()
pair.key = String.UTF8.encode(":method");
pair.value = String.UTF8.encode("GET");
headers.push(pair);
pair = new HeaderPair()
pair.key = String.UTF8.encode(":path");
pair.value = String.UTF8.encode("/details/123");
headers.push(pair);

pair = new HeaderPair()
pair.key = String.UTF8.encode(":host"); < tried "Host"  also
pair.value = String.UTF8.encode("details:9080");
headers.push(pair);

pair = new HeaderPair()
pair.key = String.UTF8.encode(":authority");
pair.value = String.UTF8.encode("details.default.svc.cluster.local");  <-  tried "details" too
headers.push(pair);

// Always end up in a BAD_ARGUMENT = 2 return error
let resVal = root_context.httpCall("details", headers, new ArrayBuffer(0), [], 5000, this.context_id, authCB)
}

}
export function authCB(origin_context_id:u32, headers: u32, body_size: usize, trailers: u32): void {
log(LogLevelValues.debug, "Inside AUTH CB ");
}

Code which is giving  BAD_ARGUMENT
envoy-wasm/blob/master/source/extensions/common/wasm/context.cc
auto cluster_string = std::string(cluster);
if (clusterManager().get(cluster_string) == nullptr) {
return WasmResult::BadArgument;
}
Http::RequestMessagePtr message(
new Http::RequestMessageImpl(buildRequestHeaderMapFromPairs(request_headers)));
// Check that we were provided certain headers.
if (message->headers().Path() == nullptr || message->headers().Method() == nullptr ||
message->headers().Host() == nullptr) {
return WasmResult::BadArgument;
}
`

the first argument to httpCall is the cluster name, look at the enovy config dump to see the list of clusters

Thanks it worked.
Ran the http://<IP_ADDRESS>:15000/config_dump to get the actual cluster name.
It turns out to be "outbound|9080||details.default.svc.cluster.local".
So the above code  with method , path , authority and the above one for cluster.  I was able to make the httpCall and get the response.  Printed the response within the callback  "authCB" defined above

A few more questions --
I was able to get the response body via this in the callback function
let bytes =  get_buffer_bytes(BufferTypeValues.HttpCallResponseBody,0,body_size);
String.UTF8.decode(bytes);
The headers: u32 as defined in the callback function params -- how do i use that to get the headers content ?
On other observation which is just cosmetic is that it prints this
" wasm log add_header: onHttpCallResponse: Token 1 not found."  -
In /v0.1.6/assembly/runtime.ts - onHttpCallResponse the below statement happens even on a successful token existence
log(LogLevelValues.error, "onHttpCallResponse: Token " + token.toString() + " not found.");

good catch - i'll fix that. let me see if i can prepare an example

Thanks.
Essentially - once i make this external HttpCall -- I would want the data from the  call response header and body to be injected into the original request  via the stream_context.headers.request  for the upstream services to use.  It could be injected security headers etc.

I added a basic example that you should be able to modify to do this.  See examples/auth

@yuval-k can you please share  a link to the example? I can't find it

Thanks.  I followed the flow in the onRequestHeaders by making the external httpCall -  I am observing this  when i print the log times
For example I have a petstore app which prints the http headers for any  call  with the timestamp (including milliseconds).The wasme filter  is injected into the petstore app.

The sequence of log outputs based on timestamp -  wasme filters onRequestHeader , OnRequestBody   calls  followed with the petstore call and then the onResponseHeaders followed by the  original httpCall  -- callback function.
Whatever is add to the headers -- stream_context.header.request.add --  shows up in the petstore app dump headers -- any additions before the httpCall
What i want to do is the take the response output of the httpCall obtained via the callback functions --  and add them to the header of the original request
-- since the request already is in the petstore app before the httpCall call back functions is invoked -- how can i control this sequence -- the callback function invocation is async and happens after

Anything to wait for the httpCall callback before forwarding the request to the petstore app container ?

no need to wait; you can add the headers in the callback, after setEffectiveContext is called.
setEffectiveContext() restores the context to the request context.
then call stream_context.headers.request.add
and finally call continue_request

This is the sequence and  the issue i am facing

A. 2020-04-22 11:54:55.610 - Inside onRequestHeaders and HttpCall response done from that method
B. 2020/04/22 11:54:55.610927 -  Inside actual Petstore app iterating headers
C. 2020/04/22 11:54:55.612   -- Inside onResponseHeaders
D. 2020-04-22 11:54:55.653 - Inside HttpCall Callback Invocation
If you see long after the reponse (C) is sent  at 55.653 the Callback (D) gets invokes as if in the background.  The headers added in C is never seen in the Petstore app when we print the request headers.
Anything i add to stream_context before the httpCall is seen in the petstore app -- it is just that the changes in call back ate never seen.
// Code in onrequest headers
onRequestHeaders(a: u32) {
const root_context = this.root_context;
let resVal = root_context.httpCall("outbound|9080||details.default.svc.cluster.local", headers, new ArrayBuffer(0), [], 5000, this, authCB)
}
// Code in Callback
export function authCB(origin_context: Context, headers: u32, body_size: usize, trailers: u32): void {
let context = origin_context as AddHeader;
context.setEffectiveContext();
stream_context.headers.request.add("hello22AuthCB", "world!22AuthCB");  <-- never seen by petstore
// print stream_context.headers.http_callback.get_headers();
continue_request();
}

just tested you use case and you are right, with istio 1.5, the request headers are not available in the callback. this is already fixed in upstream envoy-wasm, just not in the version you are testing.
i created a pr with a working example here (test with istio/proxyv2:1.6.0-alpha.1): https://github.com/solo-io/proxy-runtime/pull/16
unfortunately in that envoy version some constants have changed, so it is not backwards compatible. as such, I will have to do some thinking prior to merging this

okay - let me change my istio version to  https://github.com/istio/istio/tree/1.6.0-alpha.1 and try to run the same. I will use the example you gave and report.

@sriramcms do note that you need to build with the updated runtime, because of the changes to the constants.
the easiest way to do so, is to git clone it, checkout the branch of the PR, and point to it in your package.json like so:
"@solo-io/proxy-runtime": "file:/PATH/TO/proxy-assemblyscript"

and run npm install again before building.

I built the  wasm filter this way
package.json
"dependencies": {
"@assemblyscript/loader": "^0.9.4",
"@solo-io/proxy-runtime": "file:./../"
},
$HOME/.wasme/bin/wasme build precompiled ./build/optimized.wasm -t webassemblyhub.io/sriramcm/demo-add-header:v0.24
INFO[0000] adding image to cache...                      filter file=./build/optimized.wasm tag="webassemblyhub.io/sriramcm/demo-add-header:v0.24"
INFO[0000] tagged image                                  digest="sha256:6726087078f75811efc9f86cc8fc1d9fca5dd72ba7967f22c8c782727be0badf" image="webassemblyhub.io/sriramcm/demo-add-header:v0.24"
When i apply to the petstore app -- i get this error
INFO[0000] cache rolebinding updated                     cache=wasme-cache.wasme image="quay.io/solo-io/wasme:0.0.19"
Error: image webassemblyhub.io/sriramcm/demo-add-header:v0.24 not supported by istio version 1.6.0-alpha.1

wasme is being conservative here; essentially in order to prevent mistakes, wasme white lists which api version is compatible with which istio version;
@ilackarms what would be the best course of action here?
probably extend wasme to support istio 1.6 beta1?

Can there be a flag for force apply and skip this check? Would be useful for dev and won't require adding any alpha/beta explicitly.

Instead of using wasme and webasseblyhub -- i create a configmap  with optimized.wasm  and then mounted that as a path via annotation in the petstore app
annotations:
sidecar.istio.io/userVolume: '[{"name":"wasmfilters-dir","configMap": {"name": "petstore-filter"}}]'
sidecar.istio.io/userVolumeMount: '[{"mountPath":"/var/local/lib/wasm-filters","name":"wasmfilters-dir"}]'
*************** here is the analysis
Same sequence of missing the headers set in the HttpCall callback as by that time the call already returns from petstore --  I am suspecting that the httpCall does not wait for the callback and instead immediately returns and the request proceeds to petstore
My intention is to use the response from the httpCall callback to augment the request headers and then allow it to go to petstore -- looks like i dont have control on this aspect -- any thoughts ?
****************************  Please check the timings of the call sequence**
OnrequestHeader ----
/wasm/context.cc:1098] wasm log myfilter add_header: Response of httpcall 0
2020-04-24T20:38:46.913891Z     error   envoy wasm      [external/envoy/source/extensions/common/wasm/context.cc:1098] wasm log myfilter
OnrequestBody ----
add_header: Inside onrequestBody 12 New false
2020-04-24T20:38:46.913919Z     error   envoy wasm      [external/envoy/source/extensions/common/wasm/context.cc:1098] wasm log myfilter add_header: Inside onrequestBody 11 {"id":3,"name":"Tiger","status":"available"}
2020-04-24T20:38:46.913946Z     error   envoy wasm      [external/envoy/source/extensions/common/wasm/context.cc:1098] wasm log myfilter add_header: Inside onrequestBody 12 New true
[2020-04-24T20:38:46.910Z] "POST /api/pets HTTP/1.1" 200 - "-" "-" 44 45 9 3 "-" "curl/7.58.0" "ac8a5c7d-ed56-91ec-ad8e-2eb0444d03d5" "petstore:8080" "<IP_ADDRESS>:8080" inbound|8080|http|petstore.default.svc.cluster.local <IP_ADDRESS>:32844 <IP_ADDRESS>:8080 <IP_ADDRESS>:51560 - default
Inside Actual Petstore App

2020/04/24 20:38:46.915568 Request: &http.Request{Method:"POST", URL:
2020/04/24 20:38:46.916233 Iterating Headers
2020/04/24 20:38:46.916262 Header field "User-Agent", Value ["curl/7.58.0"]
2020/04/24 20:38:46.916436 Header field "Accept", Value ["/"]
2020/04/24 20:38:46.916453 Header field "Content-Type", Value ["application/json"]
2020/04/24 20:38:46.916460 Header field "Content-Length", Value ["44"]
2020/04/24 20:38:46.916470 Header field "X-Request-Id", Value ["ac8a5c7d-ed56-91ec-ad8e-2eb0444d03d5"]
2020/04/24 20:38:46.916478 Header field "Hello22init", Value ["world22init"]
2020/04/24 20:38:46.916485 Header field "X-B3-Spanid", Value ["6cea313e9123ab86"]
2020/04/24 20:38:46.916492 Header field "X-Forwarded-Proto", Value ["http"]
2020/04/24 20:38:46.916499 Header field "Hello32init", Value ["world32init"]
2020/04/24 20:38:46.916507 Header field "X-B3-Traceid", Value ["1c7146e7b3573e8f6cea313e9123ab86"]
2020/04/24 20:38:46.916514 Header field "X-B3-Sampled", Value ["1"]
createPet
OnresponseHeaders ----
2020-04-24T20:38:46.917972Z     error   envoy wasm      [external/envoy/source/extensions/common/wasm/context.cc:1098] wasm log myfilter add_header: Inside onresponse headers
2020-04-24T20:38:47.173170Z     critical        envoy wasm      [external/envoy/source/extensions/common/wasm/context.cc:1101] wasm log add_header: Unable to set effective context: 2 with result: 2
OnrequestHeaders HttpCall callback ----
2020-04-24T20:38:47.173291Z     error   envoy wasm      [external/envoy/source/extensions/common/wasm/context.cc:1098] wasm log add_header: Inside AUTH CB MODIFIED 2222222
2020-04-24T20:38:47.173301Z     error   envoy wasm      [external/envoy/source/extensions/common/wasm/context.cc:1098] wasm log add_header: Inside AUTH CB Response Call Headers 11
2020-04-24T20:38:47.173333Z     error   envoy wasm      [external/envoy/source/extensions/common/wasm/context.cc:1098] wasm log add_header: :status
2020-04-24T20:38:47.173341Z     error   envoy wasm      [external/envoy/source/extensions/common/wasm/context.cc:1098] wasm log add_header: 200
For reference the code inside the httpCall callback sets the below headers  which you will find missing in the petstore -- headers dump
let context = origin_context as AddHeader;
//context.setEffectiveContext();
stream_context.headers.request.add("hello22AuthCB", "world!22AuthCB");

Thanks it worked.   Used the Istio 1.6 alpha 2 and applied envoyfilter directly without going through wasme.
***************  The sequence works perfectly
END OF HttpCall CALLBACK
2020-04-25T13:52:56.436886Z     error   envoy wasm      [external/envoy/source/extensions/common/wasm/context.cc:1098] wasm log myfilter add_header: callback continue request
INSIDE THE PETSTORE APP HEADERS ITERATION
2020/04/25 13:52:56.437989 Iterating Headers
2020/04/25 13:52:56.438037 Header field "Hello22init", Value ["world22init"]
2020/04/25 13:52:56.438050 Header field "Hello22authcb", Value ["world!22AuthCB"]
2020/04/25 13:52:56.438071 Header field "Content-Type", Value ["application/json"]
2020/04/25 13:52:56.438177 Header field "Added-Header", Value ["authorized"]
RESPONSE  HEADERS
2020-04-25T13:52:56.439660Z     error   envoy wasm      [external/envoy/source/extensions/common/wasm/context.cc:1098] wasm log myfilter add_header: Inside onresponse headers

Another request for a wasme deploy istio with --force flag.
I have verified that if I do wasme deploy istio on Istio 1.5.4, then upgrade Istio to 1.6.1, that the  webassemblyhub.io/$YOUR_USERNAME/add-header:v0.1 example still works on the running bookinfo pods.
It took me a bit longer to verify, but if I delete bookinfo and recreate it leaving the EnvoyFilter CRs untouched, and manually set the sidecar.istio.io/userVolume and sidecar.istio.io/userVolumeMount annotations, it all still works.
Adding Istio 1.6.x to the white list isn't enough.  I am also using daily builds with the image tag latest.

Work-around to force wasme to believe it is working with Istio 1.5.4 no matter what the version:
IMAGE=$(kubectl -n istio-system get deployment istiod -o jsonpath="{.spec.template.spec.containers[0].image}")
ALIAS=docker.io/$YOUR_USERNAME/pilot:1.5.4
docker pull $IMAGE
docker tag $IMAGE $ALIAS
docker push $ALIAS
kubectl -n istio-system patch deployment istiod --type=json --patch="[{\"op\": \"replace\", \"path\": \"/spec/template/spec/containers/0/image\", \"value\": "$ALIAS"}]"

It seems another work-around can be achieved by just not specifying any ABI version:
{
  "type": "envoy_proxy",
  "abiVersions": [
  ],
  "config": {
    "rootIds": [
      <filter-name>
    ]
  }
}

It seems in that case the ABI check is skipped, and you get a log like this:
time="2020-06-21T14:43:02Z" level=warning msg="no ABI Version found for image, skipping ABI version check"

For more details, this is the relevant code as far as I understand: https://github.com/solo-io/wasme/blob/master/pkg/deploy/istio/istio_provider.go#L132

Looks like the sample code provided by Yuval is working now. Separately the requested Istio force deploy feature has been implemented by https://github.com/solo-io/wasm/pull/179 and is available in wasme v0.0.27+, so I'm going to close this one out.


n

I solved my issue thank you my error

@Lawlet, glad you figured it out, would you mind closing this issue. Thanks!


[WIP] Introduce wasm-opt

Address #159
blocked by #194 ,wasm-opt and wasm-snip should be merged at same time.

Is this still blocked? Looks like #194 has been closed.

closing in favor of #625


Lxd doesn't have KVM images for focal arm64

In SQA testrun https://solutions.qa.canonical.com/testruns/testRun/66cfff7a-8dd3-43bf-9c2f-fd26071d4467, which is focal on arm64 with lxd latest/stable (5.5-37534be) fails because juju can't find arm64 images for focal kvm:
Machine  State    Address         Inst id              Series  AZ     Message
0        started  <IP_ADDRESS>  sqa-lab2-node-1-arm  focal   zone1  Deployed
0/lxd/0  started  <IP_ADDRESS>  juju-91e9d3-0-lxd-0  focal   zone1  Container started
1        started  <IP_ADDRESS>  sqa-lab2-node-2-arm  focal   zone2  Deployed
1/lxd/0  started  <IP_ADDRESS>  juju-91e9d3-1-lxd-0  focal   zone2  Container started
1/lxd/1  started  <IP_ADDRESS>  juju-91e9d3-1-lxd-1  focal   zone2  Container started
2        started  <IP_ADDRESS>  sqa-lab2-node-3-arm  focal   zone3  Deployed
2/lxd/0  started  <IP_ADDRESS>  juju-91e9d3-2-lxd-0  focal   zone3  Container started
2/lxd/1  started  <IP_ADDRESS>  juju-91e9d3-2-lxd-1  focal   zone3  Container started
3        started  <IP_ADDRESS>  sqa-lab2-node-4-arm  focal   zone1  Deployed
3/lxd/0  started  <IP_ADDRESS>  juju-91e9d3-3-lxd-0  focal   zone1  Container started
4        started  <IP_ADDRESS>  sqa-lab2-node-6-arm  focal   zone3  Deployed
4/lxd/0  started  <IP_ADDRESS>  juju-91e9d3-4-lxd-0  focal   zone3  Container started
5        started  <IP_ADDRESS>  sqa-lab2-node-7-arm  focal   zone1  Deployed
5/kvm/0  down                     pending              focal          no results for "arm64", "focal", "released", "uefi1.img"
5/lxd/0  started  <IP_ADDRESS>  juju-91e9d3-5-lxd-0  focal   zone1  Container started
5/lxd/1  started  <IP_ADDRESS>  juju-91e9d3-5-lxd-1  focal   zone1  Container started
6        started  <IP_ADDRESS>  sqa-lab2-node-8-arm  focal   zone2  Deployed
6/lxd/0  started  <IP_ADDRESS>  juju-91e9d3-6-lxd-0  focal   zone2  Container started
6/lxd/1  started  <IP_ADDRESS>  juju-91e9d3-6-lxd-1  focal   zone2  Container started
7        started  <IP_ADDRESS>  sqa-lab2-node-9-arm  focal   zone3  Deployed
7/kvm/0  down                     pending              focal          no results for "arm64", "focal", "released", "uefi1.img"
7/lxd/0  started  <IP_ADDRESS>  juju-91e9d3-7-lxd-0  focal   zone3  Container started
7/lxd/1  started  <IP_ADDRESS>  juju-91e9d3-7-lxd-1  focal   zone3  Container started
8        started  <IP_ADDRESS>  vault-1              focal   zone1  Deployed
9        started  <IP_ADDRESS>  vault-2              focal   zone2  Deployed
10       started  <IP_ADDRESS>  vault-3              focal   zone3  Deployed

I've been told that the kvm machines are handled through lxd, hence I'm are opening the bug here.
Logs and configs collected for this testrun can be found here:
https://oil-jenkins.canonical.com/artifacts/66cfff7a-8dd3-43bf-9c2f-fd26071d4467/index.html

Juju does not currently use LXD for virtual machines.
LXD does have access to focal arm64 images (we in fact use those ourselves daily):
stgraber@dakara:~$ lxc image info ubuntu:focal/arm64 --vm
Fingerprint: d1361bdb666ba432f6a5a1b79c6e89b05d2c4615ffaad48b989b63be3c5cda22
Size: 543.56MB
Architecture: aarch64
Type: virtual-machine
Public: yes
Timestamps:
    Created: 2022/09/20 00:00 UTC
    Uploaded: 2022/09/20 00:00 UTC
    Expires: 2025/04/23 00:00 UTC
    Last used: never
Properties:
    label: release
    serial: 20220920
    description: ubuntu 20.04 LTS arm64 (release) (20220920)
    type: disk1.img
    os: ubuntu
    release: focal
    version: 20.04
    architecture: arm64
Aliases:
    - 20.04/arm64
    - f/arm64
    - focal/arm64
Cached: no
Auto update: disabled
Profiles: []


Question to opencore and Linux and some other questions and issues

ok, i was trying around alot and checked my battery time. With normal use and multiple reboots a got operating time of a bit more of 4 hours, i think without rebooting so many times 5 hours could be realistic
so, now i can boot Windows with opencore, no more problems because your acpis have those if osi darwin string in it. my old own repo did not have, so i always crashed windows in acpi failure. this was the main cause why i switched to your repo and your acpis are much newer than my ones based on rehabman

my only problem now is booting linux Mint and opensuse with opencore. -it does not work, i get to the grub bootloader screen but it won´t boot. i think something is injected, maybe acpi tables where those darwin string is not inside???? did not find a solution, but internet shows that many people have the same problem. maybe you have a solution
in all kind of forums you can read that it never will be possible to get a working trackpoint, but since today i am not sure if this is true, because by loading voodoops2mouse it works, was a big surprise for me ;) -i think we should work on it
one last issue i have, after sleep the cardreader does not initialize anymore, the same like in my old repo
Thank you very much, now i think my efi is way more clean and  advanced like before

Hey, thank you for being so active! I look forward to work on this EFI with you!
I haven't tried booting Linux on this laptop before. Would be awesome to have it working. I'll look into it tomorrow and I'll let you know if I succeed - today is quite a busy day for me. If you find any solutions, feel free to make a commit and a pull request.
The trackpoint is an PS2 device. I think the kext breaks the touchpad, because it's a device that can work both on SMBus and PS2. On the VoodooRMI repo, it's recommended to not inject VoodooPS2Mouse. I think it would be possible to have both trackpoint and touchpad working, but VoodooPS2Mouse would have to ignore the touchpad, and only enable the trackpoint.
For the card reader, make sure you're using Sinetek-rtsx by cholonam. Adding boot args like -rtsx_mimic_linux and/or rtsx_sleep_wake_delay_ms=1000 may fix the card reader after sleep.

my only problem now is booting linux Mint and opensuse with opencore. -it does not work, i get to the grub bootloader screen but it won´t boot.

I’m having the same issue. I tried modifying all SSDTs and even disabling them with no success. Linux verbose doesn’t show anything because it can’t even load the kernel.
Pretty sure OpenCore is injecting some stuff into Linux making it unbootable.
I think the only way you can solve this is use rEFInd for booting Windows, macOS (OpenCore->macOS) and Linux.

Hi thank you for testing out, when I boot linux I directly boot via F9 key. there is a nice windows tool called Bootice, there you can.manage your bios boot entries very easy. so not a big deal, hope in future OC will support Linux.

with refind I had no success, it always says something like Volume icons and then a size, but did not try much with it.

Yeah, I think we can't do anything about it.
However, I got some more info on the trackpoint. I talked to the developer of VoodooRMI and tried to solve this. Unfortunately we couldn't do anything, because the problem is with VoodooPS2Controller, not VoodooRMI. This is quite complicated but it definitely can be fixed.
Also, does the card reader work on your laptop with those patches now? For my machine all I had to do is use -rtsx_mimic_linux boot arg to initialize it after wake from sleep.

Great! it works, now I completely switched to your efi made some changes for my external hardware and co. now after sleep SD Card is available. At the moment I work on other projects, I got some hp 6570b´s made some hardware repairs and Bios Unlocking. Followed this very interesting tutorial: https://www.eevblog.com/forum/general-computing/new-way-to-reset-laptop-bios-passwords-(probook-6570b-successful-unlock!!!)/

for some HP 6570b I installed BigSur and used the tool from Chris https://github.com/chris1111/HP-Probook-EliteBook-Package-Creator-OC
It's amazing after installing everything works fine and the config.plist is so clean, not even one patch to find, all inside dsdt. Great for such an old machine!

Good job! However, DSDT patching isn't actually better than using SSDTs. SSDTs are always less bloated. I use multiple different SSDTs in EFI for this laptop, so it's easy to find out what patches are required here. Keeping them all in a single SSDT or a patched DSDT would make it harder to reverse engineer ACPI.
I'm gonna close this issue, assuming all the issues here except booting Linux are solved. I think it's a problem on the OpenCore side but I'm gonna keep working on it.


Could not load file or assembly 'Mono.Posix, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=0738eb9f132ed756'

Hi,
When I got devices with the following code:
try
{
var devices = AdbClient.Instance.GetDevices();
foreach (var device in devices)
{
Console.WriteLine(device.Name);
}
}
catch (Exception e)
{
Console.WriteLine(e.StackTrace);
}
It throwing an exception likes this:
Exception thrown: 'System.TypeInitializationException' in SharpAdbClient.dll
at SharpAdbClient.AdbServer.get_Instance() location C:\projects\madb\SharpAdbClient\AdbServer.cs:line 113
at Dot42.AdbLib.DeviceMonitor.Run() location E:\WorkRelated\Projects\UTester-windows\UTestor\AdbLib\DeviceMonitor.cs:line 161
Step into: Stepping over property 'SharpAdbClient.AdbServer.get_Instance'.  To step into properties, go to Tools->Options->Debugging and uncheck 'Step over properties and operators (Managed only)'.
Exception thrown: 'System.TypeInitializationException' in SharpAdbClient.dll
Step into: Stepping over property 'System.Console.get_Out'.  To step into properties, go to Tools->Options->Debugging and uncheck 'Step over properties and operators (Managed only)'.
at SharpAdbClient.AdbServer.get_Instance() location C:\projects\madb\SharpAdbClient\AdbServer.cs:line 113
at Dot42.AdbLib.DeviceMonitor.Run() location E:\WorkRelated\Projects\UTester-windows\UTestor\AdbLib\DeviceMonitor.cs:line 161
How to solve this problem? Thanks!

Hi @ShuangpengPang,
Thanks for reporting this! We'll be removing the dependency on Mono.Posix soon so this issue should go away.
If this is currently blocking you, can you try to add a reference to the Mono.Posix NuGet package?

@qmfrederik : Thank you for your reply! This issue was solved When I adding "Mono.Posix.dll" to the folder: /bin/Debug. Thanks!

Looks like this is addes some hours ago. Could we Update the nuget-packed? I would love to reduce the dependency :)

I've released version 2.1.0-beta291 to NuGet. You'll need to enable prerelease versions on NuGet to see it.
Can you give this version of the package a try and let me know if it works for you?

It does not install on my .net 4.5.2 projekt:
"Das Paket "Microsoft.NETCore.Targets 1.0.1-rc2-24027" konnte nicht installiert werden.  Wenden Sie sich an den Paketersteller, um weitere Informationen zu erhalten."
(sry german error message :/)
But it simply said that "Microsoft.NETCore.Targets 1.0.1-rc2-24027" does not contains any assembly references for ".NETFramework,Version=v4.5.2"
And the nuget would like to install ALL this:

(Is all this needed?)

Sorry for that! I've pushed a new version to NuGet (2.1.0-beta292) which should fix that. Let me know if this works for you.

This installed just fine :)

@ShuangpengPang , haven't heard back from you but based on feedback from others, this should be working.
Feel free to reopen if you are still experiencing issues with this!


FPN

Paper
Implement Feature Pyramid Networks
TODOs

[ ] develop the model in a scalable way (using *Block syntax and nn modules)
[ ] draw the model on figma
[ ] write doc
[ ] test the model
[ ] get (if possible) the pretrained weights

There is also a follow-up version, where they add the segmentation branch that upsample all the features from the top-bottom branch to a fixed size (1/4 of the input) and then sum-up https://arxiv.org/pdf/1901.02446.pdf

Implemented both FPN and PFPN


Make namespace explicit

Needed for an internal change.

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

libgrpc.so

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

  [ = ]       0        0  [ = ]

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

libgrpc++.so

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

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

We found another workaround. Closing.

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 2,019,964      Total (=)      2,019,964

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
11,183,087      Total (<)     11,183,090

 No significant differences in binary sizes

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


[PDI-16715] Hadoop File Output/Input: Parameter support: If using par…

…ameters to define hdfs-path additional / is required in the path
@duarteteixeira @pamval

Build Completed
:x: This pull request has errors. They will need to be addressed before it can be accepted. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -Dsurefire.runOrder=alphabetical -B -fn -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/trans/steps/textfileoutput/TextFileOutputMeta.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes: <0.1% (click to expand)
org.pentaho.di.job.entries.waitforsql.JobEntryWaitForSQL

Branch Change: -0.7% :small_red_triangle_down:
Complexity Change: -0.9% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.constant.ConstantMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -1.7% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.dimensionlookup.DimensionLookupMeta

Branch Change: -1.2% :small_red_triangle_down:
Complexity Change: -0.7% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.insertupdate.InsertUpdateMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.ldapoutput.LDAPOutputMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -3.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.replacestring.ReplaceStringMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.textfileoutput.TextFileOutputMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.webservices.WebServiceMeta

Branch Change: -1.7% :small_red_triangle_down:
Complexity Change: -1.2% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.gpload.GPLoadMeta

Branch Change: -0.9% :small_red_triangle_down:
Complexity Change: -0.8% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -0.2% :small_red_triangle_down:

License header violations

:heavy_exclamation_mark: Copyright year is not the current year on engine/src/main/java/org/pentaho/di/trans/steps/textfileoutput/TextFileOutputMeta.java. Found 2017. Was expecting 2018

Updated copyright year.
@pamval @duarteteixeira

Build Completed
:fire: This pull request has some issues. It would be preferable to fix them in order for it to be just perfect. See below for more details. Some links are also available below for further assistance in addressing those issues.
Build Commands
mvn -Dsurefire.runOrder=alphabetical -B -fn -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd clean install && mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd site

Cleanup Commands
mvn -B -f 'pom.xml' -pl 'engine' -P '!assemblies' -amd build-helper:remove-project-artifact

Changed files
  engine/src/main/java/org/pentaho/di/trans/steps/textfileoutput/TextFileOutputMeta.java

Unit Test Coverage
These statistics help you identify how your changes have affected the coverage of the following files. If a file is not in this list, then its coverage was not affected by your changes. To get some help interpreting these metrics, please refer to Jacoco's documentation.

:warning: Coverage Changes: <0.1% (click to expand)
org.pentaho.di.core.injection.bean.BeanInjector

Branch Change: -0.8% :small_red_triangle_down:
Complexity Change: -1.3% :small_red_triangle_down:

org.pentaho.di.job.entries.evaluatetablecontent.JobEntryEvalTableContent

Branch Change: -1% :small_red_triangle_down:
Complexity Change: -1% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.job.entries.ftpsget.JobEntryFTPSGet

Branch Change: -0.5% :small_red_triangle_down:
Complexity Change: -0.6% :small_red_triangle_down:
Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.calculator.CalculatorMetaFunction

Branch Change: -8.7% :small_red_triangle_down:
Complexity Change: -3% :small_red_triangle_down:
Instruction Change: -0.7% :small_red_triangle_down:
Line Change: -0.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.datagrid.DataGridMeta

Branch Change: -2% :small_red_triangle_down:
Complexity Change: -1.7% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:

org.pentaho.di.trans.steps.memgroupby.MemoryGroupByMeta

Branch Change: -3.3% :small_red_triangle_down:
Complexity Change: -3.8% :small_red_triangle_down:
Instruction Change: -0.3% :small_red_triangle_down:
Line Change: -0.9% :small_red_triangle_down:

org.pentaho.di.trans.steps.pgbulkloader.PGBulkLoaderMeta

Branch Change: -1.8% :small_red_triangle_down:
Complexity Change: -0.9% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.3% :small_red_triangle_down:

org.pentaho.di.trans.steps.sort.SortRowsMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.trans.steps.textfileoutput.TextFileOutputMeta

Instruction Change: -0.1% :small_red_triangle_down:

org.pentaho.di.www.TransformationMap

Branch Change: -12.5% :small_red_triangle_down:
Complexity Change: -7.7% :small_red_triangle_down:
Instruction Change: -15.5% :small_red_triangle_down:
Line Change: -14.6% :small_red_triangle_down:

org.pentaho.di.ui.repository.pur.repositoryexplorer.model.UISecurityRole

Branch Change: -1.8% :small_red_triangle_down:
Complexity Change: -1.5% :small_red_triangle_down:
Instruction Change: -0.2% :small_red_triangle_down:
Line Change: -0.5% :small_red_triangle_down:

org.pentaho.di.trans.steps.salesforceupdate.SalesforceUpdateMeta

Instruction Change: -0.2% :small_red_triangle_down:


TNT is not ingiting when on

It's kind of if in XML has <fuse> and it has 4 secs, the tnt last 5 secs for destroy objects, appears like lag but i try it on a localhost server and it's the same problem

https://docs.oc.tc/modules/tnt

	Time till the TNT explodes once it is ignited.
Must be a value less than 4 seconds since the TNT entity vanishes after that.

Was an ex, when <fuse> is 1 same

A fuse of 1 second works fine for me


Wrote a Makefile to cross-compile on Linux

as the following c files cannot compile alone without errors,
added #include "types.h" and "screen.h" to frame.c
added #include "types.h" and "colors.h" to menu.c
Wrote a Makefile to cross-compile on Linux
The Makefile as now support the compile of snake.c that include all,
but commented there are generation of the single files in a canonical manner

please merge or reject the pull request


Organizer page

Jennifer - why aren't you on the organizers page?

I have another waffle ticket related (#1161), there are multiple people to add and I sent out emails twice asking for information from each person (title,organization,photo that they wanted included). I was waiting for a response so I could put everyone on there but I haven't heard back from anyone. I just forgot about myself while waiting I guess :)

Done


needsBodyBuffered() causes NullPointerException

I've got NulPointerException thrown when enabling needsBodyBufferred() with the following stack trace:
java.lang.NullPointerException
	at com.netflix.zuul.netty.filter.BaseZuulFilterRunner.initRunningFilterIndex(BaseZuulFilterRunner.java:88)
	at com.netflix.zuul.netty.filter.ZuulFilterChainRunner.filter(ZuulFilterChainRunner.java:52)
	at com.netflix.zuul.netty.filter.BaseZuulFilterRunner.invokeNextStage(BaseZuulFilterRunner.java:122)
	at com.netflix.zuul.netty.filter.ZuulEndPointRunner.filter(ZuulEndPointRunner.java:137)
	at com.netflix.zuul.netty.filter.ZuulEndPointRunner.filter(ZuulEndPointRunner.java:49)
	at com.netflix.zuul.netty.filter.BaseZuulFilterRunner.invokeNextStage(BaseZuulFilterRunner.java:112)
	at com.netflix.zuul.netty.filter.ZuulFilterChainRunner.filter(ZuulFilterChainRunner.java:115)
	at com.netflix.zuul.netty.filter.ZuulFilterChainHandler.channelRead(ZuulFilterChainHandler.java:84)

Ex:
public class GitHubEndpoint extends HttpAsyncEndpoint {

    @Override
    public Observable<HttpResponseMessage> applyAsync(HttpRequestMessage request) {

        return GitHubService.listRepos("danesavot");

    @Override
    public boolean needsBodyBuffered(HttpRequestMessage input) {
        return true;
    }
}

You can buffer a body on an Endpoint filter.

I'm having the exact same problem.
I needed an Async endpoint, which I think Zuul2 doesn't provide, and it needs to read the body of the request.
I also tried copying the workflow from Zuul's HttpSyncEndpoint, but they still cause the body to be unavailable to my endpoint.
@danesavot do you have any workaround that you implemented? As this issue is almost 2 years old, now.

Yeah, i think I have.
You need to create an InBoundFilter and override needsBodyBuffered() so that Zull will buffer the body for the Endpoint and avoid above NPE. Let me know if you need further clarification :)

Thank you @danesavot, it works like a charm.
Just to provide more context to Zuul's team of why this is needed: I have a framework that I made it use Observable to be able to use it in the filters. But some of the Proxy operations require the Proxy itself to access and persist the information, and I'd love to see it using the same framework.


fix storm-hbase pom parent version

This pull request was closed because it looked abandoned.  Please re-open if is not.


WIP: Generate HTML doc from Notebooks

Issue #551:
Description of changes:
I've tried to add a first draft for generating HTML docs out of the existing notebooks. However this approach has two major downsides:

It has some major prerequisites (nbsphinx, nbsphinx-link, IPython and pandoc)
There are many Sphinx warnings due to duplicate labels in all of the generated HTML files from the notebooks.

Nevertheless the generation works more or less (some CSS finetuning probably required) and the Notebooks are included in the generated docs. So maybe this state can server as a starting point for discussion.
PS: @igorborgest There were some leading whitespaces copy-paste "errors" in #562. These are also fixed in this PR but I can also create a separate one for that.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I did a checkout in your branch and built it locally, I loved it!

nbsphinx, nbsphinx-link, IPython are all python packages and will be under the developer requirements scope, so I don't worry with that.
pandoc might be a block to build on readthedocs, I'm not sure if we have it there or if there is someway to install it.
About copy-paste "errors", is up to you, we can wait and merge everything together if you prefer.

Also, readthedocs doesn't run our build script currently, so we will need to figure out a way to generate the link-files there.
Maybe calling the script from the Makefile??

The link files are now created with the help of a local Sphinx extension. However pandoc in combination with readthedocs seems to be a problem:
https://github.com/readthedocs/readthedocs.org/issues/4987

The link files are now created with the help of a local Sphinx extension.

Awesome!

However pandoc in combination with readthedocs seems to be a problem

I will try find out some solution 🤔

I've also tried to use some different libraries (m2r and mdToRst) for converting Markdown to rst but they all seem to fail (e.g. with images).

Maybe setting up the readthedocs environment with Anaconda could work: https://nbsphinx.readthedocs.io/en/0.8.1/usage.html?highlight=pandoc#Using-conda

Seems a good alternative. Could you add an equivalent file into your branch please?
I've created a specific branch to receive your PR. It will reflect here.

The current setup should work but is still missing adjustments to navigation, formatting, etc.
It seems like the readthedocs build is not triggered but I've tested it with a quick custom setup 🎉.

It will not trigger automatically from the PR, I will need to merge it into the target branch.

Can I merge it?
You can open a second one later with the tweaks.

Yes I think you can merge it to see if it actually works.

Awesome! It worked!
https://aws-data-wrangler.readthedocs.io/en/docs-stage/

Could you open another PR, please? We will point it to main now.

Yes I will create another one. Do you think we should point the links in the README and docstrings to the docs page or leave them pointing to GitHub?
My intuition would be to point the README links to GitHub (as many will read it there) and point the docstring links to the documentation. WDYT?

Yes I will create another one.

Thank you! 👏

My intuition would be to point the README links to GitHub (as many will read it there) and point the docstring links to the documentation.

Makes sense to me.
p.s. Don't forget to update the tutorials link in the readthedocs header.


diagonal window glass and pirate shuttle improvements

Added - Diagonal window glass (window, rwindow, plasma, rplasma, uranium, ruranium and shuttle) and also includes diagonal damage sprites.
Added - Diagonal grilles (includes diagonal electrified sprite).
Added - dark external shuttle airlock glass and standard  (to match the typical syndi shuttle walls).
Changed - syndicate airlock sprites to be closer to the typical syndicate shuttle walls (now grey instead of jet black).
Added - diagonal windows and grille are craftable.
Updated - Pirate shuttle with a new look and some new items listed below.
Updated - both the infiltrator and striker with dark external shuttle airlocks.
Added - ironclad II armor (not a hardsuit) intended to provide a bit of protection for pirates.
Added - pirate PDA and ID card (pirates have syndi access).
Changed - pirates now start with syndicate faction (this was the system change).
Pirate shuttle changes

Changed the shuttle walls to plastitanium (usual for syndis) and windows to plasma.
Changed the external shuttle airlocks to dark versions.
Added 2 suits of ironclad II armor (not hardsuits).
Added 2 improvised laser pistols.
Added 2 turret toolboxes (hit em with wrench to place).
Added 1 syndicate armored coat.
Changed airlocks to syndi airlocks (pirates now have syndi access).

Media

[x] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

i dont like how the glass is pink, the contrast is bad

Fantastic! Been missing diagonal windows for a while now. Only thing missing now is diagonal reinforced solid walls.

Diagonal windows are certainly good, and I'm very glad that you returned to supporting pirates.  But there are a couple, but:

This is the bridge of the ship (the cockpit, its style does not look appropriate).
Gateways... The black color looked much more aesthetically pleasing, and the plasma glass... In general, it doesn’t look bad.
Sorry for the harsh criticism, but I am more than sure that many people think the same.

Diagonal windows are certainly good, and I'm very glad that you returned to supporting pirates. But there are a couple, but:
1. This is the bridge of the ship (the cockpit, its style does not look appropriate).

2. Gateways... The black color looked much more aesthetically pleasing, and the plasma glass... In general, it doesn’t look bad.
   Sorry for the harsh criticism, but I am more than sure that many people think the same.

I darkened the syndi airlocks and i made the plasma glass more opaque.

And one small, impudent wish: would it be possible to make one room for holding prisoners on a pirate shuttle? Just keeping them in the captain's cabin or in storage is not very convenient)


FBOX.to

Source name
fbox.to
Source link
https://fbox.to
Source language
English
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a title with source name.
[X] I have checked that the extension does not already exist on the website extensions list or the app.
[X] I have checked that the extension does not already exist by searching the GitHub repository and verified it does not appear in the code base.
[X] I will fill out all of the requested information in this form.

It's a mirror of fmovies, which already exists as an extension.


Changing radius of RigidBody2D's shape ignores Area2D gravity

Godot version:
v3.2.beta1.official
OS/device including version:
Ubuntu 18.04.3 LTS
Issue description:
When changing the radius of a RigidBody2D's CircleShape2D, an Area2D's gravity is ignored.

(Apologies for low frame rate recording)
The yellow circle is the Area2D in the minimal reproduction project, the red circle is the RigidBody2D. Issue might be related to #33369 and #19271 as I also ran into those while running into this issue.
Also doesn't work when trying to use with non-gravity point Area2D.
Also doesn't work when using non CircleShape2D shapes for the Area2D.
Steps to reproduce:

Create a RigidBody2D node with a CircleShape2D for a shape that dynamically grows
Create an Area2D node with Gravity Point enabled and Space Override set to Replace
Let RigidBody2D grow inside of Area2D to observe effect

Minimal reproduction project:
minimal-reproduction-project.zip

Let me know if there's anything I can do to help!

Testing on linux mint 18.3.
Can reproduce in 3.2.beta1, 3.2.3.rc3.
If you connect the Area2D's body_entered and body_exited signals to functions that print "entered"/"exited", you'll see that a resize causes the player to exit, and resizes on successive frames keep it from re-entry.
First, you should really only interact with physics in a _physics_process function, not a _process function (but that doesn't make a practical difference here).
If you replace
$CollisionShape2D.shape.radius = size

with
$CollisionShape2D.shape.set_deferred("radius", size)

the problem is resolved.


Error in Popup

Describe the Error
The popup in blogs page is not aligned
Expected behavior
Alignment and remove animation from popup
Additional context

not required


Model inputs unloaded to memory

Model inputs are loaded in memory i.e if there is a 100 million documents collection.
For a model to run on it, we load all documents on memory before passing it to the model.

Please add more details: which methods/ classes are the primary offenders?


IMPORTANT: Some entities are patron OC's from Changed

Dark Latex Yufeng [Permission confirmed by BaiTian6641]
Latex Beifeng [Permission confirmed by Valurio9683]
Latex Blue Dragon [Permission confirmed by Valurio9683]
Latex Hypno Cat [Permission confirmed by BaiTian6641]
Latex Pink Wyvern [Permission confirmed by Valurio9683]
Latex Red Dragon
Latex Silver Fox
Latex Watermelon Cat
Latex Yuin [Permission confirmed by Valurio9683]

the leaf lizard is also not owned by DS

Dark Latex Yufeng now is OK
I contact with owner

Nice

Nice

You can remove the red panda, it's not OC

Issue is stale, we've had no issues with unconfirmed variants.


fix: make batch images a list

In sd-webui-comfyui we need images to be a list, so that each image in the list can have an arbitrary size, and so that the list size can change arbitrarily.
Note that len(tensor) is defined and is the same as tensor.shape[0]. It has been a thing for a long time:
https://github.com/pytorch/pytorch/blame/e3539a0e5497677a0f7ef29dc8278d249bbccb43/torch/_tensor.py#L907
See https://github.com/AUTOMATIC1111/stable-diffusion-webui/pull/11933 for details.

We're looking into other solutions, this may not be necessary after all.


Fix build error on Android

The size of struct could be different on each architecture.

@prideout @pixelflinger Is it possible to gurantee same type size on arm7 and arm64?

Oh we can just remove this assert, it's not very useful.


[Outlook] (mobile) Clarify add-in support for on-prem accounts

Fixes #3308.

Docs Build status updates of commit 4dfa4a8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/requirements-for-running-office-add-ins.md
:white_check_mark:Succeeded
View

docs/outlook/outlook-mobile-addins.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit babcdea:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/requirements-for-running-office-add-ins.md
:white_check_mark:Succeeded
View

docs/outlook/outlook-mobile-addins.md
:white_check_mark:Succeeded
View

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


Fastnlight: Watch +8801893252360

Fastnlight requests the watch of the watch_number +8801893252360. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

Approved by Jeff Schaller in Charcoal HQ


Docker Trick

I'm sure this is not the right solution, but I was able to hack the issue #164 by adding database credentials to an env.list file in the root directory, and modifying the scripts/*start.sh file to use those environment variables.
Reject this and give instructions or modify so the config.json file is read. :)
Signed-off-by: Sean Goggins<EMAIL_ADDRESS>

pipelined to be fixed in #166

closed as changes were integrated into another pr


Completed Activities still on list

The Activities list is full of completed objectives that still show as active. There is no way to reset or clear them

Details
none
Game Version
1.7.33
New Game
No
Reproduction Steps
Just play the game

Remains an issue in 1.7.36. I've had a few completed mission get stuck, they resolved themselves, but "Distilling Confidence"  seems permanent at this point.

Without specifics, this one cannot be investigated.


Duplicate entries in automation triggers/conditions/actions

Describe the bug
There are duplicate items in the automation trigger, condition and action dropdowns.
To Reproduce
Steps to reproduce the behavior:

Create a device automation
Select Philips Hue Play HDMI Sync Sox device
Open trigger, condition or action dropdowns
Duplicate actions listed

Expected behavior
No duplicate actions
Environment
Provide details about the versions you are using, which helps to reproduce and find the issue quicker.

Home Assistant Core version:  2021.12
Home Assistant operating environment (OS/Container/Supervised/Core):  OS
Huesyncbox integration version:  1.16
Last working version(if known):

Additional info

Solved in 2.0.0 by using standard HA entities


Huffman Coding

Description

Related Issue

This PR closes #5
Motivation and Context- Java Implementation of Huffman Coding using Greedy Approach

Screenshots (if any)
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

Hey, thanks for PR! Merged 👍

@SVijayB , as per the new rules of Hacktoberfest 2020, could you please label this pull request as "hacktoberfest-accepted"? Thank you.

@ASHMITA-DE it is also considered towards progress if the pull request is merged.


Any appetite to upgrade to python3?

Is there any appetite to switch to python3?  Has this already been looked into?

Why don't you do it? Python 2 is more common on systems imo.

not at this time.

Hey if I converted this to support python 2 + 3 is that something you would be willing to accept?


[IMP] Quartz scheduler was down in production for 15 mins

In our production env, i noticed that few times in the last couple of weeks, the quartz scheduler was down for almost 15 minutes. The scheduler triggers a cron job which runs every 20 seconds. Normally i can understand about the trigger misfire scenario but the scheduler to be completely down for a long time rings alarm bells for me. I checked Dynatrace for any issues at the VM level but couldn't find anything concerning. What can be the reasons and any help on debugging steps would be great!!

Hello team, i would really like some updates on this issue.  Following is the quartz config in PROD.
org.quartz.scheduler.instanceId=AUTO
org.quartz.dataSource.quartzDataSourceAOS.validateOnCheckout=true
org.quartz.scheduler.skipUpdateCheck=true
org.quartz.dataSource.quartzDataSourceAOS.maxConnections=5
org.quartz.jobStore.driverDelegateClass=org.quartz.impl.jdbcjobstore.MSSQLDelegate
org.quartz.jobStore.tablePrefix=QRTZ_
org.quartz.threadPool.class=org.quartz.simpl.SimpleThreadPool
org.quartz.threadPool.threadCount=5
org.quartz.dataSource.quartzDataSourceAOS.driver=com.microsoft.sqlserver.jdbc.SQLServerDriver
org.quartz.dataSource.quartzDataSourceAOS.URL=
org.quartz.jobStore.dataSource=quartzDataSourceAOS
org.quartz.scheduler.instanceName=AOSClient
org.quartz.dataSource.quartzDataSourceAOS.validationQuery=select CURRENT_USER;
org.quartz.dataSource.quartzDataSourceAOS.discardIdleConnectionsSeconds=10
org.quartz.jobStore.isClustered=true
org.quartz.dataSource.quartzDataSourceAOS.idleConnectionValidationSeconds=30
org.quartz.jobStore.clusterCheckinInterval=20000
org.quartz.scheduler.jobFactory.class=org.quartz.simpl.SimpleJobFactory
org.quartz.jobStore.class=org.quartz.impl.jdbcjobstore.JobStoreTX

This issue is very much relevant as currently it still happens in production and we have no clue about the reason for the same. We are just following a work around but the actual root cause needs to be found out.


Rewrite Puppeteer crawler to extend from Browser crawler

The new Puppeteer crawler should extend the Browser crawler and provide backward compatibility.
Expected breaking changes:

missing launchPuppeteerFunction

Added in v1.0.0

Added in v1.0.0


Thanks

Well done. Thanks for the contributions.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0193ed76-3eec-18ac-512f-45f8118dde34
Version Independent ID: 8e693833-3e45-2bb8-ddc2-d0c54936c3c1
Content: A New Feedback System Is Coming to docs.microsoft.com
Content Source: teamblog/a-new-feedback-system-is-coming-to-docs.md
Product: team-blog
GitHub Login: @EisenbergEffect
Microsoft Alias: EisenbergEffect

Thank you for the feedback!


add 24.05.82903 HF delegate

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: Add 24.05.82903 HF delegate
Jira/GitHub Issue numbers (if any):
Preview links/images (Internal contributors only):

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[x] Successful preview build.
[x] Code owner review.
[x] No merge conflicts.
[x] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://664634e3043bed0bced69441--harness-developer.netlify.app


Added note for Limitations

Added note about not sharing of monitors for the Limitations section.
Purpose of this pull request
This pull request (PR) ...
Issue number:

Select the type of change:

[x] Minor Changes - Typos, formatting, slight revisions
[ ] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

@angadrandhawa1 Could you please review this change?

Looks like this was merged in a different, new PR


Implement parse ruby in the lrc parser.

What's done in this PR:

Test and fix lrc parser with empty text.
Should pass the index for encode method.
Implement the logic for parsing the ruby.

Guess it's OK enough to make a release for testing.


XDY-011: Fixed implementation of sending logs

Changed log sending to every 5 minutes except when the minute strikes at 0
Changed condition from not bot.is_closed() to True for easier reading
Changed logic when showing channels depending on the environment

parfait!


Horizontal scroll doesn't work on Linux

The following code snippet (100 boxes that should scroll horizontally) doesn't respond to mouse/touchpad scrolling:
Window(onCloseRequest = ::exitApplication) {
    Row(modifier = Modifier.fillMaxSize().horizontalScroll(rememberScrollState())) {
        repeat(100) {
            Box(modifier = Modifier.height(40.dp).width(40.dp).padding(4.dp).background(Color.LightGray))
        }
    }
}

Vertical scrolling (a Column with .verticalScroll(rememberScrollState())) works correctly (it scrolls when using a scrollwheel/touchpad)
JVM: Temurin 17.0.4
Compose version: 1.2.0
Kotlin version: 1.7.10
OS info:

Just noticed that the scrollbars example works. The issue is only capturing mouse/touchpad events to scroll horizontally

The same  on macOS desktop app

I have the same problem :(

LazyRow also doesn't work correctly

In https://github.com/JetBrains/compose-jb/issues/2690 there was a small workaround described - use Shift with vertical scroll of wheel.
But we still need to support wheel left/right, if the rest of the system supports that.

Investigation.
Scroll isn't received by Swing (underlying framework):
import java.awt.Dimension
import javax.swing.JFrame
import javax.swing.SwingUtilities

fun main() {
    SwingUtilities.invokeLater {
        val j = JFrame()
        j.addMouseWheelListener {
            println("wheel")
        }
        j.size = Dimension(200, 200)
        j.isVisible = true
    }
}

Doesn't work on JBR 17, JDK 17. But it works in IDEA somehow. We should look at its sources.

If vertical scrolling works, surely the events are received, no?

When we scroll vertically - we receive events. When horizontally - we don't receive.

@igordmn I think we're talking about different things.
I think you're talking about physical horizontal scrolling, like a horizontal scroll-wheel?
I'm talking about vertical scroll events causing horizontal scrolling in the UI.
Not sure what the original bug reporter meant.

i'm wondering when this issue is going to fix

i'm wondering when this issue is going to fix

Based on other issues and priorities, we want to postpone fix of current issue.

@mbunderline76

i'm wondering when this issue is going to fix

This is a problem for me when using a trackpad, i.e. scrolling with two fingers left and right. I should not have to hold down shift for compose to receive horizontal scroll events.

I did an investigation and still can't receive horizontal scroll events (without pressing shift). I checked out intellij community source code, I even tried jetbrains runtime (21) but the scroll events are still not received! Also tried the --add-opens java.desktop/javax.swing.plaf.basic=ALL-UNNAMED but nothing works...
This is a huge problem for my app as well because it is 2d canvas based.
Do you have any updates on this @igordmn?
Thanks in advance

@igordmn same problem here, users (i include myself here) would expect scrolling without having to shift to work. Is there any workaround so that it works? The expected behaviour should be like the tabs row in intellj idea. Thanks to all the team btw!

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Refetch server data from scratch after server upgrade

Since zulip/zulip#18205, the server sends its new version and feature level in an event each time it restarts; and since #4707, we use that to keep our record of the server's version and feature level up to date.
This is good because it means that when we subsequently make a request to the server, if we have any logic that needs to look at the feature level to interpret the results of that request, we'll use the correct feature level in order to behave correctly.
There still may be edge cases, though, where that isn't quite enough. As I wrote on the chat thread that led to the mentioned changes: there can have been places where we did something relying on the old feature level, and have data sticking around from that that is no longer quite right for the new one. Also places where we have something in flight at the time of the restart -- like some request we decided to make, and prepared something based on the old version, and with the new version that's no longer right.
To be sure to put a bound on the effect of any such edge case, we should notice when a restart event indicates the server got upgraded (for which a change in zulip_feature_level should be enough), and then make sure to reload the data from scratch within some bounded period of time.
Right now we effectively get this automatically, because we already reload the server data from scratch on each app startup, plus whenever we've been offline for 10 minutes so that the queue was expired. Realistically that means we never go more than a day or so without reloading. So there's nothing we need to do about this now.
Once we're otherwise getting close to implementing long-lived event queues, though -- aka #3916 -- it'll start becoming possible for small data errors to persist indefinitely and to accumulate. This issue will be one of the things we should do to prevent such errors, before rolling long-lived event queues out broadly.

Thinking about how often this logic would cause a refetch, I see on the API changelog:
https://zulip.com/api/changelog
that there were about 35 new feature levels between Zulip Server 3.0 and 4.0. There was about a 10-month period between those releases, so that's roughly one new feature level per week on average. For people using Zulip Cloud (or another server that updates frequently from Git), that means a refetch probably every 1-2 weeks or so, or less often if deploys are less frequent.
That's often enough that it could reduce somewhat the benefit of #3916 -- though every 1-2 weeks is still a lot less often than the status quo, for anyone who uses the app frequently.
In principle, we don't actually have any need to refetch on every upgrade -- only on upgrades that change how the app itself would behave through some feature-detection logic. In other words, when the version and/or feature level moves past one of the values that we compare them against somewhere in the app's code. There's only a handful of those per year; so one strategy that would reduce the number of refetches here to a very low level would be to keep all such versions in a central list somewhere in the code, and only trigger a refetch on an upgrade that changes where the version falls vs. that list.

Also, when implementing this we should be sure to not just reload immediately, in order to avoid hitting the server with a thundering herd of reloading clients. Instead, following the API documentation at https://zulip.com/api/get-events#restart , we should spread the reload out randomly over a period of at least 10 minutes.


Add secure array type.

This change is

#434
#435  👈
dev

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @alonh5 and the rest of your teammates on  Graphite


Output arbitrary variable values along flow lines

The flow lines output on the Flow tab should allow the specification of an arbitrary (user-selected) list of variables sampled along the flow lines.

This one duplicates issue #2091 , thus closing it.


How to prevent property propagation?

I am using styled-components, and I pass color via component props, however, they get propagated to div element, and I'm getting an error of Unknown prop blinkColor on <div> tag.
Component:
// @flow

import React from 'react'
import { Segment } from 'semantic-ui-react'
import styled, { keyframes } from 'styled-components'
import { colors } from '../../constants'

type PropTypes = {
  blinkColor: string
}

const blink = blinkColor => (keyframes`
  from {
    background-color: ${blinkColor};
  }

  to {
    background-color: ${colors.shadow};
  }
`)

const ColoredSegment = styled(Segment)`
  animation: ${props => blink(props.blinkColor)} 0.7s !important;
  background-color: ${colors.shadow} !important;
`

export default (props: PropTypes) => <ColoredSegment {...props} />

Usage:
<ColoredSegment blinkColor={colors.error}>
  Header as="h3">{word}</Header>
</ColoredSegment>

I  don't want to manually remove properties, is there any other approach you can suggest?

You can ask usage questions in our Gitter room: https://gitter.im/Semantic-Org/Semantic-UI-React.  We reserve GitHub for bugs and features.
That said, your component should consume its own props and spread the rest.  Refer also to React's docs: https://facebook.github.io/react/warnings/unknown-prop.html.

You can ask usage questions in our Gitter room: https://gitter.im/Semantic-Org/Semantic-UI-React.  We reserve GitHub for bugs and features.
That said, your component should consume its own props and spread the rest.  Refer also to React's docs: https://facebook.github.io/react/warnings/unknown-prop.html.

You can use it like this
// wrap semantic component and don't pass blinkColor to semantic component
const SegmentContainer = ({ blinkColor, ...props }) => (<Segment {...props} />)

const ColoredSegment = styled(SegmentContainer)`
  animation: ${props => blink(props.blinkColor)} 0.7s !important;
  background-color: ${colors.shadow} !important;
`


Total Memory fix of Grafana Dashboard

Required information
https://grafana.com/grafana/dashboards/15726
Issue description
The grafana dashboard does not show the correct total memory in the overview.
Currently it sums the total memory over all containers. This is incorrect if there are multiple containers on the same host.
Instead of calculating the total by
sum(lxd_memory_MemTotal_bytes{job="$job",project="$project"})

it should only take one container per instance (which is a lxd host):
sum(topk by (instance) (1,lxd_memory_MemTotal_bytes{job="$job",project="$project"}))

that takes the maximum total memory for one container on each instance.
PS: Thanks for exposing the metrics and creating the dashboard. Is it possible to maintain the json in a git, so it is easier to create PRs.

It's even more tricky to get the lxd_memory_MemFree_bytes correct in an overview.

The dashboard assumes that all containers have memory limits set.
If you do not, then you indeed do need a change like the one you're suggesting.
However applying this change to the default dashboard would then completely break any deployment where instance or project memory limits are in place.

@simondeziel does that make sense? ^


I want get the Face Landmark、Hand Landmark and Body Landmark on MediaPipe Holistic

Hello, I'm developer with an iOS application,
Now I need Use MediaPipe Holistic To Tracking human body、face and hands, I try to Run the Example Demo and get the realtime pixelBuffer, But I can't get the Face Landmark、Hand Landmark and Body Landmark with the code.
I need your help now to get that landmark.
Before that, I found one that could be accessed Face Landmark Demo，get face landmark demo link
language：Swift/Objective-C
Xcode Version：14.0.1
Device：IPhone 12 Pro Max

Hello @Epoch2022-iOS,
We are upgrading the MediaPipe Legacy Solutions to new MediaPipe solutions However, the libraries, documentation, and source code for all the MediapPipe Legacy Solutions will continue to be available in our GitHub repository and through library distribution services, such as Maven and NPM.
You can continue to use those legacy solutions in your applications if you choose. Though, we would request you to check new MediaPipe solutions which can help you more easily build and customize ML solutions for your applications. These new solutions will provide a superset of capabilities available in the legacy solutions. Thank you


Add unit tests

This will add the unit test cases for:

main,go
iops.go
latency.go
throughput.go

Signed-off-by: Pradeepkumarbk<EMAIL_ADDRESS>

@Pradeepkumarbk Can you share the output of go test ./... -cover

@Akash4927
PASS
coverage: 31.4% of statements
ok  	github.com/weaveworks-plugins/scope-plugin	0.107s

@Pradeepkumarbk
Why this

PASS
coverage: 31.4% of statements
ok  	github.com/weaveworks-plugins/scope-plugin	0.107s

weaveworks-plugins  ?

@satyamz Thats my directory, I had cloned our repo there.
PASS
coverage: 31.4% of statements
ok  	github.com/openebs-plugin/scope-plugin	0.051s

changed.


Updated JamfUploaderTeamsNotifier to handle output from JamfPatchUploader

Haven't had a chance to test this (haven't gotten my test recipe to work) but I don't see any reason that it shouldn't work. Worth a review to make sure I'm using the right variables.

Maybe it's helpful to point you to the jamf-upload.sh script? This allows you to test processors without needing a recipe, just supply fake key values.
See my test script _tests/test.sh which already contains test runs. You'd just need to make sure you supply the webhook ID and any other appropriate generic values like JSS_URL in your prefs.

Added some tests and tested it, looks like it works. Thanks for the tip on the test scripts!

Great, thanks!
I'll copy it over to grahampugh-recipes tomorrow.


Return meaningful default values for text size and font for onSelectionChange (Editor)

After some research, this is not an issue.
Default value for font size and family are defined in the web page containing the editor. When these annotations returns null the app should take as value those in the page.


Refactor and use utils.h objects

There are a lot of objects not being used (which maybe should be used) and some sort of code duplication in utils.h. We should refactor and use these objects more consistently throughout the codebase.
It might make sense to split utils.h into two. One with all the lists for information and one with the structs.

I wouldn't mind having util functions, util data (data lists, enums) and util structures (structs, typedefs).
I saw somewhere else that it was recommended, to further split every util function implementation into it's own file.

I would normally split everything into its own file, it just doesn't seems like a common thing to do in c++ and it doesn't seems to have a good way to structure the many files which would be created.


Add pages for local churches

Should the metadata for local churches live in 11ty data files (cf. https://www.11ty.io/docs/data-global/)?
Is there a reason to create .md files for the individual church pages?

The base data for local churches should live in individual JSON files inside the global data directory (cf. https://www.11ty.io/docs/data-global/).
If we choose to keep the “about” information to introduce more information about local churches, then that should be what we put in the content area of a .md file or other such template.

#10 creates GeoJSON feature objects for each local church (cf. https://geojson.org/).
We still need to create page templates and .md files.

Update church naming for item.name

Add local service time in the church about section.

@czechju, how do you think the services JSON object should be formatted?

@czechju:

Update church naming for item.name

After thinking things over, it makes better sense to keep the file names hyphenated like slugs. They’re URL slugs, not hashtags. It’s easier to read and catch typos with the hyphens. And the hyphens make it possible to split the file-slug strings with JavaScript if we ever find ourselves wanting to do that (e.g., target all churches with “community” in their name).
The only reason we were considering eliminating the hyphens in the first place was to maintain dot notation in JavaScript chaining rather than mixing in Object[key] references. We still have to use Object[key] notation, but we would anyway in the templates to map over an array of all the churches or to target a particular local church. However, we should not have to hard code a specific church’s file slug in any of the templates.


Invalid_client error Error 401 on listEvents

This may be because I have confused the terminology. Here's my json file:
{
"type": "service_account",
"project_id": "slackchatbot-158302",
"private_key_id": "aaa1111111111aaa1111111',
"private_key": "-----BEGIN PRIVATE KEY-----\nMII0Khvqr0NetaXZBz6vPV\nzOmdPlvd4cFRuQiWTIvLyGfgbQ==\n-----END PRIVATE KEY-----\n",
"client_email"<EMAIL_ADDRESS>"client_id": "115464219574XXXXXXXXX",
"auth_uri": "https://accounts.google.com/o/oauth2/auth",
"token_uri": "https://accounts.google.com/o/oauth2/token",
"auth_provider_x509_cert_url": "https://www.googleapis.com/oauth2/v1/certs",
"client_x509_cert_url": "https://www.googleapis.com/robot/v1/metadata/x509/calendaraccess%40someacct.iam.gserviceaccount.com"
}
my settings.js file looks like this:
var fs = require('fs');
const KEYPATH='./SlackChatBot-aaa1111111.json';
var json = fs.readFileSync(KEYPATH, 'utf8');
var key = JSON.parse(json).private_key;
var SERVICE_ACCT_ID = 'aaa11111111ee10158c95f8d7577777xxx77x7x7x77x7x';
const CALENDAR_URL =<EMAIL_ADDRESS>var CALENDAR_ID = {
 'primary'<EMAIL_ADDRESS>};
module.exports.calendarId = CALENDAR_ID;
module.exports.serviceAcctId = SERVICE_ACCT_ID;
//module.exports.keyfile = KEYFILE;       //or if using json keys - module.exports.key = key;
module.exports.calendarUrl = CALENDAR_URL;
module.exports.key = key;
The SERVICE_ACCT_ID I retrieved from the credentials page under Service Account Credential the column ID, the CALENDAR_URL i got from the settings of the calendar itself under "Calendar Address" where it said "Calendar ID", this I duplicated in the CALENDAR_ID. But this would mean the SERVICE_ACCT_ID is the private_key_id from the JSON file, which seems redundant, but I'm not sure where there is another SERVICE_ACCT_ID to use.

NOTE: I will leave this here in case someone else runs across this. The SERVICE_ACCT_ID, is found by clicking "manage service accounts" from the main credentials page. On the page the "Service Account ID" is listed. That the one to use for SERVICE_ACCT_ID. in the .json file it is listed as "client_email"

CALENDAR_URL can be retrieved from Settings of the Calendar > Calendar Address and would look like https://calendar.google.com/calendar/.....
SERVICE_ACCT_ID is retrieved from the google developer console would look something like<EMAIL_ADDRESS>I've removed the CALENDAR_URL config in the instructions in new release, to avoid confusion.
Hope your error has been solved!


As a data owner I want an export format for submitting data to other portals eg NBN Atlas

related to #159
raises larger question about data flows etc

The route of data into other platforms needs to be considered e.g. trough DASSH and archiving, directly from JCDP, or by individual data owners/custodians...?


Parse out article info from parsely-page tags

See https://www.parse.ly/help/integration/ppage/. This isn't used as much now as the open graph protocol, but in the past it was used a lot more. As a result, this provides value to anyone using this library for parsing articles out of historical data e.g. the news crawl, common crawl, etc.

@dlrobertson do you have a live example of what this would be like in the wild? I would guess that it would be to find the `parsely-page' named meta tag and then serialize the json string (content) into a dictionary.

Thanks for the example. I will need to look into the common crawl I have never heard of it.

The common crawl (http://commoncrawl.org/) and the news crawl (http://commoncrawl.org/2016/10/news-dataset-available/) might be relevant to this project. There might be some users of this project that use it to parse out articles and article metadata from pages one of the common crawl datasets.


[e2e]: container-based schedule main default slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/8406461501
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container-based.schedule.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container-based.schedule.main.default.slsa3.yml
Trigger: schedule
Branch: main
Date: Sun Mar 24 03:06:05 UTC 2024

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/8414500720
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.container-based.schedule.main.default.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.container-based.schedule.main.default.slsa3.yml
Trigger: schedule
Branch: main
Date: Mon Mar 25 03:38:01 UTC 2024
Tests are passing now. Closing this issue.


[Bug] Uncaught Runtime error

Version
<EMAIL_ADDRESS>Reproduction link
https://demos.creative-tim.com/notus-nextjs?_ga=2.261760520.424340754.1610867537-881030562.1610431277
Operating System
Linux mint
Device
DELL PRECISION T5500
Browser & Version
Version 86.0.4240.75 (Official Build) (64-bit) Google chrome
Steps to reproduce
npm run dev ,
inspect element to view responsive index page
reload the mobile responsive page
nextjs throws you a uncaught runtime error
What is expected?
No runtime error
What is actually happening?
Unhandled Runtime Error
ChunkLoadError: Loading chunk 0 failed.
(error: http://localhost:3000/_next/static/chunks/0.js)
Source
../../client/next-dev.js (16:0) @ Object.eval
14 | // The runtimeChunk can't hot reload itself currently to correct it when adding pages using on-demand-entries
15 | // eslint-disable-next-line no-unused-expressions

16 | REPLACE_NOOP_IMPORT
17 |
18 | // Support EventSource on Internet Explorer 11
19 | if (!window.EventSource) {

Call Stack
eval
webpack-internal:///./node_modules/next/dist/client/next-dev.js (155:30)
Object../node_modules/next/dist/client/next-dev.js
file:///home/torch/Desktop/Fresh/notuspub/.next/static/chunks/main.js (737:1)
webpack_require
/_next/static/chunks/webpack.js (873:31)
checkDeferredModules
file:///home/torch/Desktop/Fresh/notuspub/.next/static/chunks/webpack.js (46:23)
webpackJsonpCallback
/_next/static/chunks/webpack.js (33:19)

file:///home/torch/Desktop/Fresh/notuspub/.next/static/chunks/webpack.js (1015:54)

/_next/static/chunks/webpack.js (1023:10)

Solution
Additional comments

Please make a video with the following:

Download the product
Unzip the product
Open terminal and cd into the project
Run node -v
Run npm -v
[Mac/Ubuntu/Linux] Run npm run install:clean
[Widnows] Run npm install
[Widnows] Run npm start
Showcase the issue

Best,
Manu


scanpy cannot be imported in my windows system

Hi,
I couldn't import scanpy due to an error: DLL load failed. I have checked pre-existing issues, but all of them seem to be an h5py issue. My error report seems different from them.
>>> import scanpy as sc
D:\Anaconda\lib\site-packages\dask\config.py:168: YAMLLoadWarning: calling yaml.load() without Loader=... is deprecated, as the default Loader is unsafe. Please read https://msg.pyyaml.org/load for full details.
  data = yaml.load(f.read()) or {}
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\yuhong\AppData\Roaming\Python\Python37\site-packages\scanpy\__init__.py", line 36, in <module>
    from . import tools as tl
  File "C:\Users\yuhong\AppData\Roaming\Python\Python37\site-packages\scanpy\tools\__init__.py", line 17, in <module>
    from ._sim import sim
  File "C:\Users\yuhong\AppData\Roaming\Python\Python37\site-packages\scanpy\tools\_sim.py", line 23, in <module>
    from .. import _utils, readwrite, logging as logg
  File "C:\Users\yuhong\AppData\Roaming\Python\Python37\site-packages\scanpy\readwrite.py", line 10, in <module>
    import tables
  File "C:\Users\yuhong\AppData\Roaming\Python\Python37\site-packages\tables\__init__.py", line 99, in <module>
    from .utilsextension import (
ImportError: DLL load failed: The specified procedure could not be found.

>>> print(sys.version)
3.7.3 (default, Mar 27 2019, 17:13:21) [MSC v.1915 64 bit (AMD64)]

Hi, if you use conda, try conda install pytables.
If you don't, try installing from the corresponding wheel here https://www.lfd.uci.edu/~gohlke/pythonlibs/#pytables

@Koncopd Hi, Thank you for the pointer. It seems to be a problem caused by pytables package. But I still couldn't import tables after installing and uninstalling pytables packages for many times.
(base) C:\Users\yuhong>python
Python 3.7.8 | packaged by conda-forge | (default, Jul 31 2020, 01:53:57) [MSC v.1916 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import tables
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\yuhong\AppData\Roaming\Python\Python37\site-packages\tables\__init__.py", line 99, in <module>
    from .utilsextension import (
ImportError: DLL load failed: The specified procedure could not be found.

You can also try creating a new environment and installing pytables there at first and try to import it.

pytables in pip is named tables, and scanpy import tables accordingly, so you have to separately install it using also pip, not conda. I suggest editing the installation guide.
Besides, my installing using conda install -c bioconda scanpy would always give conflicts with nvidia cuda versions, but changing the version makes nothing change except the conflict message. Is it a Windows feature?

I suspect it might be the same issue as in #454 , so a problem with building libraries for h5py (which pytables needs) since Windows misses a C++ compiler for it.


Admin Tool: fix batch size < 0 erro when peekmessage

Fixes #11300
Motivation
This pull request fix the error of X-Pulsar-batch-size < 0
Modifications

fix the logic of caculate X-Pulsar-batch-size  to make sure it's the right size of batch message
expose more attribute to Admin Tool command of getMessageById and examineMessage
add test code for these changes

Verifying this change
This change added tests in BrokerEntryMetadataE2ETest.testBatchMessage
Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (no)
The public API: (no)
The schema: (no)
The default values of configurations: (no)
The wire protocol: (no)
The rest endpoints: (no)
The admin cli options: (no)
Anything that affects deployment: (no)

Documentation

Does this pull request introduce a new feature? (no)

@BewareMyPower PATL


Reservoir sampling

Work In Progress:

Added code for reservoir sampling
Fix test error issues

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

pending two mypy errors

Going to approve, can clean except, etc. in subsequent PR. Base of intent is good though


@Press event not working on Firefox

When using the @press event on chrome everything works fine.
But when you use it on firefox it throw a : Uncaught ReferenceError: TouchEvent is not defined

The @click event works fine on both browser

I've tried with the new update and it still throw an error.


init.js

Hi,
Ui.R and server.R is doing well but where to specify the init.js file in project.
Voice recognition is not enabled but visualization is showing.
Please find the image below to showcase the code and location of java script file.

Description and screenshots of the locations are very much appreciated.
Cheers!!

I cannot reproduce your issue. Are you using the latest version of shiny? Your app looks very different from mine: https://github.com/yihui/shiny-apps/tree/master/voice I don't know what you changed. It seems you were trying to make it into an R package. If that is the case, you should not put the app under the R/ directory, but a certain directory under inst/ instead.

I think it should work if e.g. one changes hello.R to app.R und runs runApp("D:/Data_Sara/Data Analytics/R/MyPackage/R").
As far as i can see the app is run via the console and the code looks for "www/init.js" in the working directory `getwd(). So my best guess is getwd() is unequal to "D:/Data_Sara/Data Analytics/R/MyPackage/R" and that causes the error.

Try specifying the relative/absolute path to init.js
Ex: includeScript("./init.js")