Error message of Response could has multi-lang ?

Hi,
First, It’s awesome.  This package is really help to me 👍
I notice the Core/GoogleReCaptchaV3Response has declare const var like this (maybe not only in here 😄 ) :
const MISSING_INPUT_ERROR = 'Missing input response.';
const ERROR_UNABLE_TO_VERIFY = 'Unable to verify.';
const ERROR_HOSTNAME = 'Hostname does not match.';
const ERROR_ACTION = 'Action does not match.';
const ERROR_SCORE_THRESHOLD = 'Score does not meet threshold.';
const ERROR_TIMEOUT = 'Timeout';
if have any chance, could you please support multi-lang message setting for those message ?
Thanks ^^

Sure, I can update it tonight for sure.

@chinlung I have added it in my local, I will try to merge the branch by tmr, I am missing some test cases because of safety. Sorry for the delay.

@chinlung I have added it in my local, I will try to merge the branch by tmr, I am missing some test cases because of safety. Sorry for the delay.

@RyanDaDeng It’s OK. Thanks for so quick to help that.  😸

@chinlung Please update the version to be ~2.2.19, and then run:
php artisan vendor:publish --provider="TimeHunter\LaravelGoogleReCaptchaV3\Providers\GoogleReCaptchaV3ServiceProvider" --tag=googlerecaptchav3.lang

A lang folder will be created under /resources/lang/vendor/GoogleReCaptchaV3/*

@RyanDaDeng
Work well !!  Thanks again 😄


Publish the app as a .NET Core Global Tool

Motivation

A .NET Core Global Tool is a special NuGet package that contains a console application. A Global Tool can be installed on your machine on a default location that is included in the PATH environment variable or on a custom location.

Having the generator installed as a global tool will allow for re-generating models as per https://github.com/Kentico/cloud-boilerplate-net/issues/31 .
Design guidelines
The goal of this task is to make changes necessary to publish the Kentico Cloud Code Generator for .NET as a .NET Core Global Tool via NuGet.
Resources

Official documentation
Howto blogpost
Example

I have looked at this and came across an issue while packing the NuGet package: The "GetRuntimeLibraries" task was not given a value for the required parameter "RuntimeIdentifier". I have tried setting the runtime argument for dotnet pack and also setting runtime identifiers to csproj, but it didn't help. The problem is related to the ILLink.Tasks package that we use, I have tried other versions of the package - didn't work, the only working solution I found to this is to remove the package. I have also searched for alternatives to ILLink.Tasks but found only tools that have to be run manually.

You can safely remove ILLink.Tasks. It's been replaced with Microsoft.DotNet.ILCompiler as described in #41. (You can pick that up as well, if you're interested :))

you can create more artifacts (publish it twice)..
quick tip: if you want to reduce the packaging time, you can use --no-build option


pdf加载失败

Warning: getHexString - ignoring invalid character. Failed to load PDF file
Warning: Indexing all PDF objects

加载失败：The API version "4.3.136" does not match the Worker version "2.4.456".

@FrankJingZhi

加载失败：The API version "4.3.136" does not match the Worker version "2.4.456".

Is this related to the original issue?

sorry, It nothing to do with. I resolve it, because I use more PDF plugins. removed redundant, it works.


Allow Proc#call to take splatted array

An example to clarify:
x = Proc(Int32, Int32, Int32).new { |x,y| x + y }
a = [1,2]
x.call(*a)

This give the error: argument to splat must be a tuple, not Array(Int32).
I think it severely limits the usefulness of lambdas if splatted arrays can't be passed in as arguments. Having to use tuples makes many use cases difficult (if not impossible).

Could you elaborate on how the compiler should guess the right types from an array? Let's take this example:
def add(a : Int32, b: Int32)
  a + b
end

def add(a : String, b : String)
  "#{a} #{b}"
end

a = [[1, 2], ["foo", "bar"]].sample
add(*a)

Doesn't the compiler know the type of the array (e.g. [] of String or [] of Int32)? Can't it use that?
In your example, of course, the sample causes a union type which isn't supported even if they are tuples. (splatting a union ({Int32, Int32} | {String, String}) is not yet supported).

The compiler knows the type of the array at compile-time, but not it's size.
What happens if you do this?
x = Proc(Int32, Int32, Int32).new { |x,y| x + y }
x.call(*ARGV.map(&.to_i))

One answer is that the compiler transforms the above to this:
x = Proc(Int32, Int32, Int32).new { |x,y| x + y }
temp = ARGV.map(&.to_i)
if temp.length == 2 && (first = temp[0]).is_a?(Int32) && (second = temp[1]).is_a?(Int32)
  x.call(first, second)
else
  raise "wrong number of arguments/incorrect types for proc call"
end

However, right now in the language there are no hidden raises when issuing method calls: they either compile and succeed, or don't compile at all. I mean, there's no "undefined method" nor "wrong number of arguments" errors at runtime, only at compile-time. We don't want to break that rule. So in this case you'll have to manually do the above checks.

What if it's explicit, like:
unpack_call myfunc, [1, 2, 3]

@trans so if we support it for methods/procs with uniform argument types, don't you think people will be even more disappointed about being unable to splat to def foo(a : String, b : Int32)?

Since arrays can only be of one type, it is clear why that is not possible.
However maybe it would be better to back up, and instead of worrying about Proc taking a splatted array, is it possible to allow tuples to be created from arrays? That would take care of this without changing the way Procs work.

@will has a trick for that, see #1466

Oh and foo = [] of String|Int32 is perfectly valid.

@jhass

Oh and foo = [] of String|Int32 is perfectly valid.

I realize that but I consider it one type -- a union type.

@will has a trick for that, see #1466

Unfortunately I don't understand this. Could you break it down into it's most essential form?

Can't you do something like http://play.crystal-lang.org/#/r/f99?

@kirbyfan64  yes but notice http://play.crystal-lang.org/#/r/f9w
@trans this is only useful for the cases where you really need it, so it will still involve some metaprogramming boilerplate, but http://carc.in/#/r/f9t

@jhass Thanks for the break down. It makes sense to me now. But it doesn't work in this case b/c the size has to be given explicitly -- it can't come from a variable.
Consider my use case, which is basically popping arguments off a stack and passing them to a Proc. The number of arguments to pop off and pass depends on the arity of the Proc. I do not know the arity of the Proc b/c it is provided by the library's user.
Now I could do something like:
a = a0.shift(aproc.arity)
case aproc.arity
when 0:
    aproc.call
when 1:
    aproc.call(*a2t(a,1))
when 2:
    aproc.call(*a2t(a,2))
...

And so on all the way up to some arbitrary maximum that I decide. But doesn't that seem rather silly to have to do that?

Hmm... I tried to use the "trick" and I am getting a strange error:
macro didn't expand to a valid program, it expanded to:

================================================================================
--------------------------------------------------------------------------------
  1.  String.as_cast(a[0]),  String.as_cast(a[1]),
--------------------------------------------------------------------------------
Syntax error in expanded macro: macro_74729888:1: unexpected token: EOF

 String.as_cast(a[0]),  String.as_cast(a[1]),
                                              ^

================================================================================

      { {% for i in (0..1) %} String.as_cast(a[{{i}}]), {% end %} }

Why EOF? That line is not the end of the program.

You missed the {% if true %} wrapper to enter the macro language so that the opening { and } of the tuple literal are considered part of the macro expansion and thus interpreted together with the elements.

I'm not sure if your library design is possible with Crystal, I'd certainly go as far and call it not idiomatic Crystal tbh. Did you consider ways to unify the signature of the procs? Like passing a value object similar to how one passes a request object to a HTTP server handler?

@7rans Can you provide a link to the code on github (if you have already
pushed it), or at least README.md file with your public API. We might be
able to come up with solution here together, once we understand what kind
of public API you want to provide to the user of a library. I'm pretty
sure, clever macro can do the trick.
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-14 21:18 GMT+02:00 Jonne Haß<EMAIL_ADDRESS>
I'm not sure if your library design is possible with Crystal, I'd
certainly go as far and call it not idiomatic Crystal tbh. Did you consider
ways to unify the signature of the procs? Like passing a value object
similar to how one passes a request object to a HTTP server handler?
—
Reply to this email directly or view it on GitHub
https://github.com/manastech/crystal/issues/1464#issuecomment-140181174.

@jhass It would be a shame if so b/c it would mean Crystal support for lambdas is very limited.
@waterlink https://github.com/trans/clik/blob/master/src/clik.cr
So I added the {% if true %} and LOL :laughing: this compiler is too damn smart!
splatting a union ({String, String, String, String} | {String, String, String} | {String, String} | {}) is not yet supported

        flag.call(*argt)

http://play.crystal-lang.org/#/r/fd0
It's pretty freaking hackish, since I can't figure out how to get the number of arguments a proc literal takes.

I already have almost working version ;) will publish once it compiles.
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-14 23:11 GMT+02:00 Ryan Gonzalez<EMAIL_ADDRESS>
http://play.crystal-lang.org/#/r/fd0
It's pretty freaking hackish, since I can't figure out how to get the
number of arguments a proc literal takes.
—
Reply to this email directly or view it on GitHub
https://github.com/manastech/crystal/issues/1464#issuecomment-140207059.

@waterlink Dang!
Another semi-working version: http://play.crystal-lang.org/#/r/fe5. It doesn't actually compile yet either...

GOT IT!!! http://play.crystal-lang.org/#/r/fep

Yeah, it uses blocks instead of procs, but that's arguably better for the intended use case. It basically constructs a temporary proc, so that this:
puts call_block(array) { |a, b| a+b }

Is compiled to:
->() {
    a = array[0]
    b = array[1]
    # Block body is inserted here
    a+b
}

:)

So, here you go: http://carc.in/#/r/fex
Quite a long one, almost the same public API that @7rans tries to achieve
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-14 23:44 GMT+02:00 Ryan Gonzalez<EMAIL_ADDRESS>
Yeah, it uses blocks instead of procs, but that's arguably better for the
intended use case. It basically constructs a temporary proc, so that this:
puts call_block(array) { |a, b| a+b }
Is compiled to:
->() {
a = array[0]
b = array[1]
# Block body is inserted here
a+b
}
:)
—
Reply to this email directly or view it on GitHub
https://github.com/manastech/crystal/issues/1464#issuecomment-140214206.

Cleaned up a bit and added comments: http://carc.in/#/r/fey
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-14 23:58 GMT+02:00 Alex Fedorov<EMAIL_ADDRESS>
So, here you go: http://carc.in/#/r/fex
Quite a long one, almost the same public API that @7rans tries to achieve
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-14 23:44 GMT+02:00 Ryan Gonzalez<EMAIL_ADDRESS>
Yeah, it uses blocks instead of procs, but that's arguably better for the
intended use case. It basically constructs a temporary proc, so that this:
puts call_block(array) { |a, b| a+b }
Is compiled to:
->() {
a = array[0]
b = array[1]
# Block body is inserted here
a+b
}
:)
—
Reply to this email directly or view it on GitHub
https://github.com/manastech/crystal/issues/1464#issuecomment-140214206
.

This has small problem, that current scope of blocks is in entirely
different module/class.
I am trying now to mitigate it by defining macro on the fly instead of
classes
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-15 0:11 GMT+02:00 Alex Fedorov<EMAIL_ADDRESS>
Cleaned up a bit and added comments: http://carc.in/#/r/fey
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-14 23:58 GMT+02:00 Alex Fedorov<EMAIL_ADDRESS>
So, here you go: http://carc.in/#/r/fex
Quite a long one, almost the same public API that @7rans tries to achieve
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-14 23:44 GMT+02:00 Ryan Gonzalez<EMAIL_ADDRESS>
Yeah, it uses blocks instead of procs, but that's arguably better for
the intended use case. It basically constructs a temporary proc, so that
this:
puts call_block(array) { |a, b| a+b }
Is compiled to:
->() {
a = array[0]
b = array[1]
# Block body is inserted here
a+b
}
:)
—
Reply to this email directly or view it on GitHub
https://github.com/manastech/crystal/issues/1464#issuecomment-140214206
.

Did it without changing scope: http://carc.in/#/r/ff3
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-15 0:13 GMT+02:00 Alex Fedorov<EMAIL_ADDRESS>
This has small problem, that current scope of blocks is in entirely
different module/class.
I am trying now to mitigate it by defining macro on the fly instead of
classes
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-15 0:11 GMT+02:00 Alex Fedorov<EMAIL_ADDRESS>
Cleaned up a bit and added comments: http://carc.in/#/r/fey
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-14 23:58 GMT+02:00 Alex Fedorov<EMAIL_ADDRESS>
So, here you go: http://carc.in/#/r/fex
Quite a long one, almost the same public API that @7rans tries to achieve
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-14 23:44 GMT+02:00 Ryan Gonzalez<EMAIL_ADDRESS>
Yeah, it uses blocks instead of procs, but that's arguably better for
the intended use case. It basically constructs a temporary proc, so that
this:
puts call_block(array) { |a, b| a+b }
Is compiled to:
->() {
a = array[0]
b = array[1]
# Block body is inserted here
a+b
}
:)
—
Reply to this email directly or view it on GitHub
https://github.com/manastech/crystal/issues/1464#issuecomment-140214206
.

Another interesting way to do that:
https://github.com/waterlink/clik-playground/commit/226e0f32e5b7cbab6630d6e716145fe11188af68#diff-ccc79beaefc85505f35940c3a2377411L92

this shows you even more how powerful AST macro can be ;)

--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-15 0:41 GMT+02:00 Alex Fedorov<EMAIL_ADDRESS>
Did it without changing scope: http://carc.in/#/r/ff3
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-15 0:13 GMT+02:00 Alex Fedorov<EMAIL_ADDRESS>
This has small problem, that current scope of blocks is in entirely
different module/class.
I am trying now to mitigate it by defining macro on the fly instead of
classes
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-15 0:11 GMT+02:00 Alex Fedorov<EMAIL_ADDRESS>
Cleaned up a bit and added comments: http://carc.in/#/r/fey
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-14 23:58 GMT+02:00 Alex Fedorov<EMAIL_ADDRESS>
So, here you go: http://carc.in/#/r/fex
Quite a long one, almost the same public API that @7rans tries to
achieve
--
Best Regards,
Alexey Fedorov,
Sr Ruby, Clojure, Crystal, Golang Developer,
Microservices Backend Engineer,
+49 15757 486 476
2015-09-14 23:44 GMT+02:00 Ryan Gonzalez<EMAIL_ADDRESS>
Yeah, it uses blocks instead of procs, but that's arguably better for
the intended use case. It basically constructs a temporary proc, so that
this:
puts call_block(array) { |a, b| a+b }
Is compiled to:
->() {
a = array[0]
b = array[1]
# Block body is inserted here
a+b
}
:)
—
Reply to this email directly or view it on GitHub
https://github.com/manastech/crystal/issues/1464#issuecomment-140214206
.

Those are very clever and full props for being able to devise such intricate macros, but to me it isn't really a solution to this issue. I feel the problem is more fundamental. If we can't freely pass parameters to lambdas then Crystal really has no support for lambdas, and this will greatly limit it DSL abilities.
@waterlink This macro warps my mind. Where is the ">>" ever mentioned in the code? It doesn't make sense to me that you can reference the receiver and the block without any mention of the method -- in such a case wouldn't any method name work? e.g. what prevents me from doing cli "-f --file" .wtfever{ |f| @@file = f }?

Nothing prevents you currently from doing that. Though you definitely can
add a check in the loop in the macro and raise error at compile time if
call.name is not >>
On Sep 17, 2015 4:41 AM, "7rans"<EMAIL_ADDRESS>wrote:

Those are very clever and full props for being able to devise such
intricate macros, but to me it isn't really a solution to this issue. I
feel the problem is more fundamental. If we can't freely pass parameters to
lambdas then Crystal really has no support for lambdas, and this will
greatly limit it DSL abilities.
@waterlink https://github.com/waterlink This macro warps my mind. Where
is the ">>" ever mentioned in the code? It doesn't make sense to me that
you can reference the receiver and the block without any mention of the
method -- in such a case wouldn't any method name work? e.g. what prevents
me from doing cli "-f --file" .wtfever{ |f| @@file = f }?
—
Reply to this email directly or view it on GitHub
https://github.com/manastech/crystal/issues/1464#issuecomment-140949660.

I think the conversation has gotten off track a bit about my particular use case. I think the general case is more important b/c I have a number of Ruby programs that I'd like to port to Crystal, but in many cases I can foresee it probably won't be possible because of this.
@asterite

However, right now in the language there are no hidden raises when issuing method calls: they either compile and succeed, or don't compile at all. I mean, there's no "undefined method" nor "wrong number of arguments" errors at runtime, only at compile-time. We don't want to break that rule. So in this case you'll have to manually do the above checks.

I think in the case of lambda's that rule might need to be bent. At the very least maybe there can be a separate class that has the additional flexibility?

I'll close this, we have no plans of supporting this.

:-1: Good luck Rails. This is just too limiting for too many of my Ruby project to consider porting.

@trans Almost all of those cases can be replaced by tuples/macros, though...

I don't think anyone (should) attempt truly porting Rails. Crystal is a different language, it has different limitations but also different (DSL) opportunities than Ruby.

I don't understand how the "can't splat array" would prevent a Rails port?
The big differences are: the metaprogramming that make porting the DSL complicated —but they can be replaced with more straightforward solutions— and of course the static typing that make some features a bit more convoluted (params in controller, types in model, etc).

@kirbyfan64 I tried writing a macro for splatting but got stuck. Your play link (http://play.crystal-lang.org/#/r/fep) helped me finally get it.
I'd like to suggest an improvement. In the current version, the macro may overwrite the values of local variables in the same scope, e.g.: https://play.crystal-lang.org/#/r/wfu
The improved version: https://play.crystal-lang.org/#/r/wfx


helm package removes multiple version of the same subchart.

Hi, i am using the feature to use the same dependency chart twice by using an alias. I do switch between those two by using tags.
Thats part of my chart.yaml:
dependencies:
  - name: vsa
    alias: transcoderExperimental
    version: 6.0.0-rc2
    repository: "https://artifacts.core.pke.de/artifactory/helm-stable"
    tags:
      - experimental
  - name: vsa
    alias: transcoderExperimental
    version: 5.3.1
    repository: "https://artifacts.core.pke.de/artifactory/helm-stable"
    tags:
      - production

if i do update the dependecies i get both version inside the charts directory. But if i execute an
helm package <chartname> the resulting .tgz does contain unpacked directorys. The problem is that both version contain the same directory so only one version resides that way.
I guess currently the only possibility is to pack it myself and upload it that way, correct?
Is this an known or wanted behavior?
Output of helm version:
version.BuildInfo{Version:"v3.4.2", GitCommit:"23dd3af5e19a02d4f4baa5b2f242645a1a3af629", GitTreeState:"clean", GoVersion:"go1.14.13"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.0", GitCommit:"9e991415386e4cf155a24b1da15becaa390438d8", GitTreeState:"clean", BuildDate:"2020-03-26T06:17:09Z", GoVersion:"go1.14", Compiler:"gc", Platform:"darwin/amd64"} Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.8", GitCommit:"9f2892aab98fe339f3bd70e3c470144299398ace", GitTreeState:"clean", BuildDate:"2020-08-13T16:04:18Z", GoVersion:"go1.13.15", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
on-prem vanilla kubernetes

We have the same issue with the newest version 3.5.2. We have the need to support two different versions of a library chart to support a migration path. The tar archive does contain the files like the Chart.yaml twice and the latest wins.
$ tar list -f parent-chart-0.0.0.tgz | grep -E "charts/.*/Chart"
parent-chart/charts/occonnect-common/Chart.yaml
parent-chart/charts/occonnect-common/Chart.yaml

version.BuildInfo{Version:"v3.5.2", GitCommit:"167aac70832d3a384f65f9745335e9fb40169dc2", GitTreeState:"dirty", GoVersion:"go1.15.7"}


chore(docs/halyard/service-settings): Added extensive documentation r…

…egarding what can be configured for a Kubernetes deployment via service settings

@ethanfrogers @ezimanyi - Documentation updated. I went ahead and did the other parts of the kubernetes service settings too while I was in there.

@dorbin - Sounds good. I merged in your changes and went back through and fixed the wording in places where I was using "Used to". 👍

Thanks, @clanesf !


Initial Testcontainers integration for Dapr

Description
This PR adds the testcontainers-dapr module to the Java SDK. The main reason to have this code here is to improve the Spring Boot integration, while providing a common way to start the Dapr Runtime for Java applications by using Testcontainers
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #1040
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

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

@artursouza @cicoyle I couldn't check why it is complaining about License Compliance.. can you help me out to check that?

@salaboy it is not a deal breaker, but it would be good to have the testcontainers-dapr pom.xml code formatted, right now it doesn't seem to be aligned with other POM files.

I removed Rest Assured, so one less dependency. But WireMock is mocking a webserver, so I think we cannot use Mockito for that.

@salaboy now that you have extracted all the inner classes from DaprContainer I think it make sense to prefix them with Dapr,  so Component becomes DaprComponent,Subscription -> DaprSubscription, etc. Otherwise the Component or Subscription is too generic.
Let me know your thoughts.

@salaboy now that you have extracted all the inner classes from DaprContainer I think it make sense to prefix them with Dapr, so Component becomes DaprComponent,Subscription -> DaprSubscription, etc. Otherwise the Component or Subscription is too generic.
Let me know your thoughts.

Hmm.. not so sure about this one.. that's why we have packages in Java to disambiguate what is the class for. Let me think about this.

I am not entirely sure why this is being reported.. or if it is useful. For this case, it makes no sense:
Error:  Error fetching link: /home/runner/work/java-sdk/java-sdk/sdk-workflows/target/javadoc-bundle-options. Ignored it.
Error:  Medium: io.dapr.testcontainers.DaprContainer doesn't override org.testcontainers.containers.GenericContainer.equals(Object) [io.dapr.testcontainers.DaprContainer] At DaprContainer.java:[line 1] EQ_DOESNT_OVERRIDE_EQUALS
Error:  Medium: io.dapr.testcontainers.DaprPlacementContainer doesn't override org.testcontainers.containers.GenericContainer.equals(Object) [io.dapr.testcontainers.DaprPlacementContainer] At DaprPlacementContainer.java:[line 1] EQ_DOESNT_OVERRIDE_EQUALS
Error:  Failed to execute goal com.github.spotbugs:spotbugs-maven-plugin:<IP_ADDRESS>:check (validate) on project testcontainers-dapr: failed with 2 bugs and 0 errors  -> [Help 1]

Refactored the script to update the version of artifacts. Also fixed to use "docker compose" instead of "docker-compose".
Approved. Will merge once tests pass.

@artursouza the docker compose upgrade is breaking all the tests.. because the outputs are completely different :(

@artursouza the docker compose upgrade is breaking all the tests.. because the outputs are completely different :(

@salaboy can you fix this? Using docker compose is required as docker-compose is being removed from GHA as it is deprecated already.


Removing dependencies on python inside virt-launcher and virt-handler

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:
/kind bug

/kind enhancement
What happened:
At this stage we don't need for production python inside the virt-launcher or virt-handler image. However, there is one e2e test which makes use of python: https://github.com/kubevirt/kubevirt/blob/main/tests/network/vmi_networking.go#L1052.
What you expected to happen:
Once this test does not rely on python anymore we can remove python completely from the images, reducing the image size and reducing the chance of being affected by CVEs (which would most of the time not really affect KubeVirt, but any scans would show that we have affected versions inside containers).
How to reproduce it (as minimally and precisely as possible):
The only test failing after removing python in #6177 was the mentioned test above.
Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version):
Kubernetes version (use kubectl version):
VM or VMI specifications:
Cloud provider or hardware configuration:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

/area networking
/cc @phoracek

/sig network

/assign @phoracek

/assign @rhrazdil


Insert timestamp in commit_txs db table

Fixes #1843.
The timestamp field cannot be null, but we forgot to include it when inserting into the commit_txs database table.

bors r+

bors r+

bors cancel

bors r+


Error initializing ViGem!

I'm using GloSC <IP_ADDRESS> 64-bit I get "Error initializing ViGem!" in the command window, plus a dialog box with the same error.
It executes and works fine after I click OK on the dialog box though. Is there a fix for this?

You have to manually update Vigem downloading the latest release from its repository, and install the library using relative .inf.

That worked! Thanks for taking the time to explain it.


Gitignored files containing '-' character are not hidden normally.

Bug description
When I was playing around with zig, I noticed that the zig-out directory, which I had gitignored, wasn't hidden. The only difference I could see was that it contained a dash. After some testing I could see that this bug could be reproduced universally, and holds for both files and directories. Note that the gutter signs work as intended, it's just that the file itself is still showing.
Pics:

Steps to reproduce

Open triptych in a git repository
Create a file tom-ato and directory pot-ato/
Add them to the .gitignore
Toggle triptych and <leader>. to see bug.

Setup

Operating system: MacOS Sonoma 14.5
Nvim version: 0.10.0
Triptych config

{
require("triptych").setup({
            mappings = {
                -- Everything below is buffer-local, meaning it will only apply to Triptych windows
                show_help = 'g?',
                jump_to_cwd = '.', -- Pressing again will toggle back
                nav_left = 'h',
                nav_right = 'l',   -- If target is a file, opens the file in-place
                open_hsplit = { '-' },
                open_vsplit = { '|' },
                open_tab = { '<C-t>' },
                cd = '<leader>cd',
                delete = 'd',
                add = 'a',
                copy = 'c',
                rename = 'r',
                cut = 'x',
                paste = 'p',
                quit = 'q',
                toggle_hidden = '<leader>.',
            },
            extension_mappings = {},
            options = {
                dirs_first = true,
                show_hidden = false,
                line_numbers = {
                    enabled = true,
                    relative = false,
                },
                file_icons = {
                    enabled = true,
                    directory_icon = '',
                    fallback_file_icon = ''
                },
                responsive_column_widths = {
                    -- Keys are breakpoints, values are column widths
                    -- A breakpoint means "when vim.o.columns >= x, use these column widths"
                    -- Columns widths must add up to 1 after rounding to 2 decimal places
                    -- Parent or child windows can be hidden by setting a width of 0
                    ['0'] = { 0, 0.5, 0.5 },
                    ['120'] = { 0.2, 0.3, 0.5 },
                    ['200'] = { 0.25, 0.25, 0.5 },
                },
                highlights = { -- Highlight groups to use. See `:highlight` or `:h highlight`
                    file_names = 'NONE',
                    directory_names = 'NONE',
                },
                syntax_highlighting = { -- Applies to file previews
                    enabled = true,
                    debounce_ms = 100,
                },
                backdrop = 60 -- Backdrop opacity. 0 is fully opaque, 100 is fully transparent (disables the feature)
            },
            git_signs = {
                enabled = true,
                signs = {
                    -- The value can be either a string or a table.
                    -- If a string, will be basic text. If a table, will be passed as the {dict} argument to vim.fn.sign_define
                    -- If you want to add color, you can specify a highlight group in the table.
                    add = { text = "+", texthl = "SignifySignAdd" },
                    modify = { text = "~", texthl = "SignifySignChange" },
                    rename = { text = "r", texthl = "SignifySignChange" },
                    untracked = { text = "?", texthl = "SignifySignDelete" },
                },
            },
            diagnostic_signs = {
                enabled = true,
            }
        })

        vim.keymap.set("n", "<leader>pv", vim.cmd.Triptych, { silent = true })
} -- paste your config here

I've merged a change which should fix it


Using duckdb's summarize table instead of individual calls

Replacing multiple duckdb queries with SUMMARIZE query

Thanks for investigating this @AdityaHegde. I would say that for simple datasets, the SUMMARIZE feature is actually unacceptably slow, taking a few seconds to do what we were already doing iteratively in the same amount of time or in less time. I think that introducing SUMMARIZE would regress the user experience a bit.

Closing this as it doesnt provide too much of an advantage.


Simplify make rpm

This is similar to what I did for osbuild-composer: https://github.com/osbuild/osbuild-composer/pull/280

We probably want to change the workflow to use the Makefile targets here as well, just as we did in osbuild-composer.


Write issue

Hi,
I'm looking for a solution, I would like to build a big filesystem in my QSPI flash memory, by littleFS. I would like to do more than a hundred files, which could contain different datas of devices.
My problem is that, when I open the file and write something, I have to close the file, so When I try to write it again, the previous data is not available (at least by function lfs_file_read), Is it resoluble somehow?

Hi, @ErDaN213.
You have to call lfs_file_sync() to flush internal LFS cache to flash. By the way, this function is calling automatically at lfs_file_close().


Fix/OpenSearch filter logic

Description
OpenSearch filter logic, check for both MetadataFilters object and MetadataFilters.filters length (MetadataFilters object can still be non-None even if the filters property is null)
Additional: revert the exclude embedding field from the response, let user pick by themself
Fixes # (issue)
New Package?
Did I fill in the tool.llamahub section in the pyproject.toml and provide a detailed README.md for my new integration or package?

[ ] Yes
[X] No

Version Bump?
Did I bump the version in the pyproject.toml file of the package I am updating? (Except for the llama-index-core package)

[X] Yes
[ ] No

Type of Change
Please delete options that are not relevant.

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[X] I stared at the code and made sure it makes sense

Suggested Checklist:

[X] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] I ran make format; make lint to appease the lint gods

Hi @logan-markewich
Is this supposed to be merged? As I don't have the permission to do so.


ockam_node testing methodology

The issue of testing came up in #2007 because it touches on a lot of core components. In general ockam_node is mostly tested through example code, not via a robust framework of tests that stress the system to a point where concurrency bugs occur.
We need to change that for the future to ensure that we don't break user code with changes to the core of the runtime.
For now this issue is just a scratchpad for ideas.  In the future we'll expand it with more concrete plans.

https://whileydave.com/2021/10/26/test-driving-the-rust-model-checker-rmc/
https://github.com/BurntSushi/quickcheck
https://sled.rs/simulation

Yeah, so 1000% this. I think this will become an issue for external use too, since if ockam is hard to test, code that uses it will likely become hard to test. It also could lead to a non-robust product, which would be... bad for obvious reasons.
I have a few ideas on how to make tests easier to write from the user side, which hopefully will make this a bit easier for us to do internally. That said, most of those would not be running in a fully realistic environment — they'd be within the same process probably. For now that might be fine for internal testing too, but long term we need some tooling that runs our code in multiple processes, using the real transports (even if just over loopback).

I think before stress-testing node, we could start with very basic tests not involving random inputs, high loads, etc.
Example: #2007 adds clusters which introduce rules to the order of shutdown of workers. It's pretty easy to create a basic test that would spawn regular worker, and create 2 clusters with few workers each. Then shutdown the node and check that the order in which workers were stopped is as expected. That would:

Verify that new feature is working
Show intended behaviour to those who are not familiar with the new feature (sometimes looking at test is much simpler than reading the documentation)
Simplifies PR review, since it's much simpler to verify that something is working from seen test pass in CI, than from reading the code


V3 TSV output corrupted from time to time

Specify CLI command
gdf_translate.exe --agent-file C:/tmp/a/GBS_myTalentWorld_login_UAT.zip --output-folder c:/tmp/a/out --source-lang en --target-lang pt-BR --cred-file C:/tmp/a/its-mytalentworld-login-pag-no-ab2c0018a18a.json --api-version v3
Error description
based on debug log it fails because v3 api produces this in TSV output:
0000001270     0x18e82cbd8a0 <to_translate>yes that't ok</to_translate> 0x18e82c relevanta0 <to_translate> sim, não está bem </to_translate>
i.e. 0x18e82cbd8a0  is somehow strangely corrupted and part of translation is put in between. Seems like some ML magic that did not go well.
We might need to redesign format of V3 TSV map and maybe come with original design where there was helper hashmap with seq number (put into TSV file) as key and rust reference as value (and do double lookup :( )
this happens only from time to time, running v3 translation with same input couple of times usually fixes the issue.
Google DialogFlow agent zip file
N/A - included in private repo only

resolved by release v0.1.3-beta


HA Databases for Mongo and Postgres

Check if all the DB are HA, currently we are using Mongo, redis and patroni, check the configurations before production ready

Done


[Azure][Track 2 SDK] Migrate Cosmos DB State Store

Migrate Cosmos DB State Store component to SDK github.com/Azure/azure-sdk-for-go/sdk/data/azcosmos

Requires the following to be completed

[ ] https://github.com/Azure/azure-sdk-for-go/issues/16219 (AAD support)
[ ] https://github.com/Azure/azure-sdk-for-go/issues/16220 (Query support)

Looks like the related issues have been addressed, we could try doing a POC in this iteration or next.

one of these fixed this #2032 #2033 #2035


fails to prettify django fixture with boolean value true or false

Trying to prettify a json fixture generated by django (from python manage.py dumpdata) will fail if the data contains boolean value.
Example that will fail:
[{ "model": "app1.model1", "pk": 1, "fields": { "name": "data1", "is_new": true, } }, { "model": "app1.model1", "pk": 2, "fields": { "name": "data2", "is_new": false, } }]

The issue here is the trailing commas rather than the boolean literals. This is not valid JSON and so the parser fails to read it and can not prettify it. linter-jsonlint is a good way to avoid issues like this as it will automatically highlight syntax issues in JSON for you.

Actually I made an error when stripping other data to make the sample.
But the error seems to be the encoding of the generated file and not related to pretty-json.


add Awesome Gists by @vsouza

Awesome Gists is a collection of amazing gists you must see.

Thanks, but you just made this and it doesn't really have a lot of useful content yet. Give it a few months and try again.

:cry: that's ok. thanks.


feat: BRC-20 tokens

This PR adds BRC-20 indexing support to the API.
To index operations, this code follows these indexer rules:
https://docs.bestinslot.xyz/indexer-rules
https://domo-2.gitbook.io/brc-20-experiment/
Additionally, this PR creates the following new endpoints:

/ordinals/brc-20/tokens
Return paginated token information (ticker, max, limit, supply)
/ordinals/brc-20/tokens/:ticker
Return token info for a single ticker (all details + total holders)
/ordinals/brc-20/tokens/:ticker/events
Return event history for a single ticker
/ordinals/brc-20/balances/:address
Return balance for an address (optionally filtered by 1 or more tickers) giving available balance, transferrable balance, overall balance

Fixes https://github.com/hirosystems/ordinals-api/issues/49

can't wait for this merge 🔥

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add an event similar to webcomponentsjs

Dispatch an event
This is tiny addition to the build script (affects only the plain js script), that will dispatch an event similar to webcomponentsjs polypill.
This might be handy if you want to have the logic that includes the polyfill in a file (conforming the strict CSP standard) but you also need to make sure the customElements.define() doesn't run prematurely.
This is my custom loader script:
(function(){
'use strict';

// Check if ES6 is ok
if(!Object.Reflect) {
	var newScr=document.createElement('script');newScr.src='//cdnjs.cloudflare.com/ajax/libs/es6-shim/0.35.3/es6-shim.js';document.head.appendChild(newScr);
}
// Check if custom elements are natively supported
if (!window.customElements) {
	var name = 'ce-loader.js',
	script = document.querySelector('script[src*="' + name + '"]'),
	newScript = document.createElement('script'),
	url = script.src.replace(name,'document-register-element.js');
	newScript.src=url;document.head.appendChild(newScript);
}else{
	document.addEventListener('DOMContentLoaded', function() {
		window.dispatchEvent(new CustomEvent('WebComponentsReady'));
	});
}
})();

This might be handy if you want to have the logic that includes the polyfill in a file (conforming the strict CSP standard)

I've used this file with and without CSP, I'm not sure I understand what CSP has to do with it.
Also, Google HTML Amp projects uses this module as dependency, and it has no issues.
Anything that is not standard won't land in this repository and relying on non-standard events is as problematic as not having the polyfill at all.
Be sure you have needed polyfills before you need to use standards, so one day you'll just get rid of them.
This PR would let developers lock behind an event nobody agreed on, an event stuck forever in your scripts, which is out of polyfill purpose.
I appreciate the fact you took time for this PR, but also please file a bug/request/question next time, that'd be easier/faster to be aligned for everyone.


How to export 3D model file with texture mapping, such as FBX model

How to export 3D model file with texture mapping, such as FBX model

I also want to know that how we can export 3d model with texture as result ?


copy-file doesn't work on SVG files

I thought I reported this already, but I can't find the issue.

Open https://github.com/sindresorhus/refined-github/blob/main/media/icon.svg
Click the "Copy" button
Nothing happens, there's no code visible, it should only appear when the raw code is visible

Basically a duplicate of https://github.com/sindresorhus/refined-github/issues/2957

You are talking about CSV

Yes, this is also a duplicate of https://github.com/sindresorhus/refined-github/issues/4369

We should find other rendered filetypes and make sure they all work the same way so we don't play wackamole

Fix: Revert the condition.
if there's code on the page
	renderButton()
else if view-markdown-source is applicable
	wait for its events
else
	return false


[FEATURE] Add signup/signin with the google authentication

Description

want to add google signup/signin facilities  which will increase overall user experience

Screenshots
No response
🥦 Browser
Google Chrome
Checklist ✅

[X] I checked and didn't find similar issue
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (blank for no)

Please donot raise duplicate issues, someone already raised this in #711
Closing this.


Unable to use find_package(OpenCV REQUIRED) and find_package(VTK REQUIRED) at the same time

Environment

OS: x64-linux
Compiler: gcc 9.3.0

To Reproduce
CMakeLists.txt
set(CMAKE_TOOLCHAIN_FILE $ENV{VCPKG_ROOT}/scripts/buildsystems/vcpkg.cmake)

cmake_minimum_required(VERSION 3.20)

project(example)

find_package(OpenCV REQUIRED)
find_package(VTK REQUIRED)

Simply run mkdir build & cd build & cmake .. shows
-- Found TIFF: /home/jasjuang/vcpkg/installed/x64-linux/lib/libtiff.a (found version "4.3.0")
-- Found TIFF: /home/jasjuang/vcpkg/installed/x64-linux/lib/libtiff.a;/home/jasjuang/vcpkg/installed/x64-linux/lib/liblzma.a;/home/jasjuang/vcpkg/installed/x64-linux/lib/libjpeg.a;/home/jasjuang/vcpkg/installed/x64-linux/lib/libz.a;m (found version "4.3.0")
-- Found LibXml2: /home/jasjuang/vcpkg/installed/x64-linux/lib/libxml2.a (found version "2.9.12")
CMake Error at /home/jasjuang/vcpkg/scripts/buildsystems/vcpkg.cmake:574 (_add_library):
  _add_library cannot create ALIAS target "hdf5::hdf5" because another target
  with the same name already exists.
Call Stack (most recent call first):
  /home/jasjuang/vcpkg/installed/x64-linux/share/vtk/patches/99/FindHDF5.cmake:8 (add_library)
  /home/jasjuang/vcpkg/installed/x64-linux/share/hdf5/vcpkg-cmake-wrapper.cmake:17 (_find_package)
  /home/jasjuang/vcpkg/scripts/buildsystems/vcpkg.cmake:738 (include)
  /home/jasjuang/vcpkg/installed/x64-linux/share/vtk/VTK-vtk-module-find-packages.cmake:303 (find_package)
  /home/jasjuang/vcpkg/installed/x64-linux/share/vtk/vtk-config.cmake:136 (include)
  /home/jasjuang/vcpkg/scripts/buildsystems/vcpkg.cmake:784 (_find_package)
  CMakeLists.txt:8 (find_package)

Expected behavior
OpenCV and VTK should be able to co-exist.

So port vtk provides its own  FindHDF5.cmake Find module (share/vtk/patches/99/FindHDF5.cmake), and this doesn't seem to properly work on repeated usage.
Note that share/hdf5/vcpkg-cmake-wrapper.cmake is written for CMake's FindHDF5.cmake.

Should be a port bug.
cc @cenit


Issues with sending multiple POST requests in a short timeframe

I ran into an issue that occurs when multiple POST requests are made to json-server within a short timeframe. This issue only seems to occur when the "--watch" flag is passed to the command line.
When two POST requests are sent within a short timeframe, only the body of the first request is added to the in-memory database. The database file is updated correctly. This means that a GET request run after the POST requests have completed will return incorrect data.
I've created a repository that reproduces the issue: https://github.com/tbroadley/json-server-multiple-posts-repro

I ran it with 100 of post() and got the corresponding 100 entry in db.json

I added the additional item before I run your script

Hmm, I can still reproduce it with json-server 0.9.2.
Output of node index.js:
{ id: 1 }
{ id: 2 }
[ { id: 1 } ]

Output of json-server --watch db.json:
  \{^_^}/ hi!

  Loading db.json
  Done

  Resources
  http://localhost:3000/items

  Home
  http://localhost:3000

  Type s + enter at any time to create a snapshot of the database
  Watching...

POST /items 201 20.557 ms - 13
POST /items 201 1.799 ms - 13
  db.json has changed, reloading...

  Loading db.json
  Done

  Resources
  http://localhost:3000/items

  Home
  http://localhost:3000

GET /items 200 4.912 ms - 23

Contents of db.json after running node index.js:
{
  "items": [
    {
      "id": 1
    },
    {
      "id": 2
    }
  ]
}

The GET request (the result of which is the last line of the output of node index.js) returns incorrect data.
I'm running these commands on Windows 10, via Cygwin.

Try to remove --watch flag. It may misunderstand updating json to  editing file by hand

Try to remove --watch flag. It may misunderstand updating json to editing file by hand

Yeah I can fix it..... Thank you.


PHOENIX-6284 : De-flake UpgradeIT#testConcurrentUpgradeThrowsUpgradeInProgressException

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 30s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

-1 :x:
test4tests
0m  0s
The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.

_ master Compile Tests _

+1 :green_heart:
mvninstall
14m 51s
master passed

+1 :green_heart:
compile
0m 54s
master passed

+1 :green_heart:
checkstyle
0m 26s
master passed

+1 :green_heart:
javadoc
0m 44s
master passed

+0 :ok:
spotbugs
2m 54s
phoenix-core in master has 973 extant spotbugs warnings.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
7m 36s
the patch passed

+1 :green_heart:
compile
0m 52s
the patch passed

+1 :green_heart:
javac
0m 52s
the patch passed

+1 :green_heart:
checkstyle
0m 27s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
javadoc
0m 44s
the patch passed

+1 :green_heart:
spotbugs
3m  5s
the patch passed

_ Other Tests _

-1 :x:
unit
109m 45s
phoenix-core in the patch failed.

+1 :green_heart:
asflicense
0m 29s
The patch does not generate ASF License warnings.

146m  3s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1043/2/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix/pull/1043

Optional Tests
dupname asflicense javac javadoc unit spotbugs hbaseanti checkstyle compile

uname
Linux fd9499e3cddb 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-personality.sh

git revision
master / d097857

Default Java
Private Build-1.8.0_242-8u242-b08-0ubuntu3~16.04-b08

unit
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1043/2/artifact/yetus-general-check/output/patch-unit-phoenix-core.txt

Test Results
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1043/2/testReport/

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

modules
C: phoenix-core U: phoenix-core

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/job/PR-1043/2/console

versions
git=2.7.4 maven=3.3.9 spotbugs=4.1.3

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

This message was automatically generated.

Thanks, @virajjasani


face detection using caffe

Related Issuse or bug

Face detection using deep learning with the help of caffee.

Fixes: #179

@smriti1313 great! merging your pr.


Adding VS Code folders

Reasons for making this change:
To add the folder and files that Unity generates for the VS Code IDE
Links to documentation supporting these rule changes:
Link to Unity Doc about VS Code

Why is that Unity related ? Seems related to https://github.com/github/gitignore/pull/3487 and VisualStudio.gitignore no ?

Because Unity creates the .vscode/setting.json to exclude the * .meta and other files from appearing in VS Code.  This isn't not related to Visual Studio at all, since VSCode is a different IDE than Visual Studio.

Bump. My team and I generally use VSCode for work in Unity rather than Visual Studio, and not having the .vscode folder ignored creates room for confusion when setting up new projects to work with git.


File extension for saveAs includes the codec name(s)

Description
After recording an audio/video using https://collab-project.github.io/videojs-record/demo/audio-video.html
and saving it using player.record().saveAs({'video': 'foo12'}) it presents the user with a filename of foo12.webm;codecs=vp8,opus. Before it didn't append the ;codecs=vp8,opus part.
Results
Expected
foo12.webm

Actual
foo12.webm;codecs=vp8,opus

Additional Information
Please include any additional information necessary here. Including the following:
versions
videojs
videojs-record 4.2.0, video.js 7.11.4
browsers
Google Chrome | 89.0.4389.72
OSes
macOS Big Sur

Turns out to be an issue in RecordRTC: https://github.com/muaz-khan/RecordRTC/issues/731


Consider Moving the document folder into the addon_config directory

Hi!
Historically it has been nice to be able to access the documents via the share folder but when having multiple addons that store there data there (Nextcloud does too for example) it's impossible to create a clean snapshot only containing all Paperless data.
If I for example want to create a daily backup of my Paperless documents but only a weekly one of Nextcloud this is currently impossible which is unfortunate as both contain huge amounts of data.
I think it would be good to consider moving the documents into the addon_config directory so a clean snapshot can be created.
Best,
Paul

Solved this by not backing up the share folder at all in snapshots and instead backing it up separately which is even cleaner and doesn't create huge snapshots.

Anyway it's intended that the backup save the paperless content and the share folder is also required to be able to mount remote storage.
It's more a Home Assistant issue about how it handle backups and storage mounting


Add docker-compose profile support

I would be happy if docker-compose profiles are supported in lazydocker.

bump

any update?

work around: use the environment COMPOSE_PROFILES

Love the amount of unknown feature that exist in the tools we use every day 🤯
While this is a fair workaround, it kind of defeats the 'lazy' aspect of this tool. I'd love to see it added to the x commands next to up/down project (and perhaps have a comma separated list of active profiles or even list all profiles with the active in bold with click support 😄 )
Either way, love this tool to death, and this addition would just make it that much nicer
Gustav

I've opened an MR to support --profile flag ^^


Cannot find -lhp on Ubuntu or other distributions

Describe the bug
Possibly pilot error, but failing to complete "Build from source" instructions.
Reproducing
Assuming an appropriate Ubuntu 19.10 installation:

Install opam (either via apt or from opam's repository using the make cold-install approach)
opam init
eval $(opam env)
opam switch create 4.09.0 4.09.0
eval $(opam env)
sudo apt-get install pkg-config libpcre3-dev
git clone https://github.com/comby-tools/comby
cd comby && opam install . --deps-only -y
eval $(opam env)
make

Observe error output ending in:
/usr/bin/ld: cannot find -lhp
collect2: error: ld returned 1 exit status
File "caml_startup", line 1:
Error: Error during linking
make: *** [Makefile:4: build] Error 1

See attached output below for more details.
Expected behavior
The make step completing without error.
Additional context
Output from make step should be attached.
make-output.txt
For the version of opam installed via apt:
$ opam --version
2.0.5

After some searching I'm guessing that -lhp refers to hack_parallel.  Does that seem right?
I also tried installation on an Arch-derivative distribution with similar results.

Hey--thanks for the report. I ran into this but haven't had a chance to find out what changed to cause it. But yes, basically the static library isn't being picked up (it's not on the path) despite doing eval $(opam env) on Ubuntu or others (AFAIK this doesn't happen on Mac). So this is something I still need to look into. For now, you can get around this by doing, e.g.:
eval $(opam env)
LIBRARY_PATH=$OPAM_SWITCH_PREFIX/lib/hack_parallel opam exec -- dune build

instead of make. I'll keep this issue open until I've pinned down a better workaround.

Thanks!
The suggested work-around did it.
It's working well :+1:

OK so it turns out that the static library is not being packaged in a way that dune (I think as of v2?) can't pick it up. https://github.com/rvantonder/hack-parallel/pull/14 will fix this.

Thanks for the heads up.  Good to hear that a fix may be coming :+1:

Should be fixed now when building from source, via 46bdbcf.

Unfortunately, I now get this:
$ make
dune build --profile dev
File "lib/language/omega_rule.ml", line 242, characters 4-11:
242 |   | Ok rule -> Or_error.return rule
          ^^^^^^^
Error: This pattern matches values of type ('a, 'b) Stdlib.result
       but a pattern was expected which matches values of type
         consume:Consume.t -> (expression list, string) Stdlib.result
File "lib/rewriter/rewrite_template.ml", line 22, characters 4-12:
22 |   | Ok label -> List.find_map label ~f:ident
         ^^^^^^^^
Error: This pattern matches values of type ('a, 'b) result
       but a pattern was expected which matches values of type
         consume:Consume.t -> (string option list, string) result
File "lib/matchers/omega.ml", line 333, characters 12-19:
333 |           | Error _ ->
                  ^^^^^^^
Error: This pattern matches values of type ('b, 'c) result
       but a pattern was expected which matches values of type
         consume:Consume.t -> (Omega.production * 'a, string) result
make: *** [Makefile:4: build] Error 1

I will file another issue.


How to use allKeyExists assertion?

Trying this
$data = ['one' => 1, 'two' => 2];

Assert::allKeyExists($data, ['one', 'two']);

//array_key_exists() expects parameter 2 to be array, integer given in Assert.php on line 891
//[internal function]: Webmozart\Assert\Assert::keyExists(1, Array)

and this
$data = ['one' => 1, 'two' => 2];

Assert::allKeyExists(['one', 'two'], $data);

//array_key_exists() expects parameter 2 to be array, string given in Assert.php on line 891
//[internal function]: Webmozart\Assert\Assert::keyExists('one', Array)

I'm one of dumbest man on Earth or this method has a bug?

allKeyExists works a bit different than you expect. The All in this case doesn't mean that all the keys you provide exists, it means that the key you provide exists on all the arrays.
e.g. this would pass
Webmozart\Assert\Assert::allKeyExists([['one '=> 1], [ 'one' => 2] ] ,'one');
and this would fail.
Webmozart\Assert\Assert::allKeyExists([['one '=> 1], [ 'two' => 2] ] ,'one');

@BackEndTea oh, i'm read the description (Assert that key exists in an array for all values.) for this method in beberlei/assert and have no question any more. Thanks!

Honestly, I had the same problem :sweat_smile: Is there other assertion for determining that all required keys are present and values are set?


For Vue.js

How can we use it in my VUE project?

I'm fix it.

Thx for sharing ..

This might help: https://github.com/khalby786/vue-utterances!


Update all snapshots.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2341/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/2358/
Test PASSed.


Review of cloud simulator terms

CMIP6 included output from ISCCP and MISR simulators. MODIS will be added in CMIP7. Discussion of standard names has revealed errors in the usage of standard names in CMIP6 which need to be corrected. The key issue is that the satellites do not see liquid water path and ice water path: rather, they see liquid water path of ice-topped cloud and liquid water path of liquid-topped clouds.

For CMIP6, the following MODIS variables where requested:
Emon.jpdftaureliqmodis
Emon.jpdftaureicemodis
Emon.clwmodis
Emon.cltmodis
Emon.climodis
Eday.jpdftaureliqmodis
Eday.jpdftaureicemodis

The following additional ones are proposed for CMIP7:
CFmon.clwvimodis
CFmon.cltmodis
CFmon.clmodisliquid_dup
CFmon.clmodisliquid
CFmon.clmodisice_dup
CFmon.clmodisice
CFmon.clmodis
CFmon.clivimodis

Q1: Are they really additional? For example, should CFmon.clmodis replace Emon.clmodis?
Q2: How do the variables proposed by Lisa relate to those proposed by Brandon? Is there duplication here?
For MISR we have a single variable, Emon.clmisr, requested on 16 altitudes and 7 optical depth categories.
Q3: From the CF discussion I gather that the altitude refers to the cloud top altitude and the optical depth refers to the optical depth of the cloud. I.e. is a a property of the cloud, not a coordinate used to describe locations within the cloud. Is this correct?

Q2: How do the variables proposed by Lisa relate to those proposed by Brandon? Is there duplication here?

The two variables (CFmon.clwvimodis and CFmon.clivimodis) we proposed are the only ones which are not about any cloud fraction but the total cloud and ice water path seen by MODIS. There is no overlap to already exisiting variables.


camera not working

It is not working with my camera on raspberry pi 4B, the camera I am using using the official pi camera,  do you know how to get it working with raspi camera?

Indeed, it will not work with the RaspiCam.
The 64-bit Ubuntu operating system doesn't support the official pi camera.
You have three options:

Use a simple webcam instead (see the end of this page https://qengineering.eu/install-ubuntu-18.04-on-raspberry-pi-4.html for details).
Use the regular 32-bit Raspbian OS. Your app will be somewhat slower.
Try to install the userland RaspiCam software under Ubuntu to get it working. (I never tried it, although some sites claim it is possible)

Hi I installed everything with Ubuntu 20.04 and edited the file /boot/firmware/config.txt added start_x=1 to the file and the camera is working for me. But the max fps I got was 6.7 not the 9+ fps you get, not a huge increase for me, previously I got about 4.5 FPS on the same h/w with Raspbian 32bit on raspberry pi 4B, 4GB RAM

Did you overclock your RPi?

When I add the line arm_freq=1925 to usercfg.txt the pi fails to boot and get stuck on an error could not get clk: -517

Same happens to me. Max frequency I got was 1825 MHz.

what FPS you got with that? It seems Ubuntu 64bit on raspi is a bit unstable for me. I guess the best way could have been to have 64 bit raspbian to achieve the faster speed.


Add support for reading .wux files

The tool "WudCompress" converts .wud files to .wux files, eliminating duplicate sectors:
https://gbatemp.net/threads/wii-u-image-wud-compression-tool.397901/
The download contains C source for reading .wux files (as well as .wud files, but that functionality is obviously already in wud2app).
It might be useful to have .wux read support in addition to the wudparts-like read ability that already exists.

A workaround for the time being is to just use WudCompress to decompress the WUX into a WUD file again and thereafter use wud2app.


Documentation update for PR #129 - POST file from memory buffer

Add examples for POSTing a memory buffer as a file upload, to explain how to use the changes in PR #129.

This is just what we needed, thanks @adam-nielsen for the PR!


use correct framework

the undocumented names (without period in dotnet 5+) can drop their support when move to double-digit (to avoid potential conflicts) learn.microsoft.com/en-us/dotnet/standard/frameworks#supported-target-frameworks

Thank you for the help, I'm new to C# so appreciate it :cat:


Text shadow (or color, or weight) different with AOS enabled

This is font without AOS enabled:

This is font with AOS enabled:

As you can see, font shadow is a bit different, not sure why.

Hi @crnkovic
Do you have working example maybe? If you'll provide me with codepen or smth I'll be able to investigate this issue. AOS styles shouldn't interfere with this. It's probably caused by transforms, is this problem replicable on different browsers or only this one you use?

Looks like a font-smoothing issue related to hardware acceleration.
Try setting a specific -webkit-font-smoothing in your CSS. This should solve the problem.

I think this might be the case. For now I'm closing this issue @crnkovic . Feel free to open this again when you prepare an example that would present this issue.


Decrease h1 font-size and make footer center-bottom

Edited it

Sorry for 2nd mistake. Edited it.

Thanks for your work!

Thanks for the merge ! :+1:


lock variable could be null in TabletServerLocks

lock could be null at https://github.com/apache/accumulo/blob/2.1/server/base/src/main/java/org/apache/accumulo/server/util/TabletServerLocks.java#L63

Found while working on #4861


feat: add string/base/distances/levenshtein

Resolves #835.
Description

What is the purpose of this pull request?

This pull request:

Implements the RFC proposal for implementing function to calculate the Levenshtein (edit) distance between two strings.

Related Issues

Does this pull request have any related issues?

This pull request:

resolves #835

Questions

Any questions for reviewers of this pull request?

No.
Other

Any other information relevant to this pull request? This may include screenshots, references, and/or implementation notes.

No.
Checklist

Please ensure the following tasks are completed before submitting this pull request.

[x] Read, understood, and followed the contributing guidelines.

@stdlib-js/reviewers

Thanks for the quick review! Had a minor query - post making these changes, should I do a squash and force push these changes or do you want me to retain the commit history?

@Planeshifter - I have force pushed with the requested changes. Could you please approve it if everything looks good? Thanks!

Package has landed!

Truly elated I could help! Thank you!


Can't Install: "Cannot find module 'fs-plus'"

Atom does not let me install this package. Installing it in "settings" gives me this error;
`npm WARN optional Skipping failed optional dependency /Remote-FTP/chokidar/fsevents:
npm WARN notsup Not compatible with your operating system or architecture<EMAIL_ADDRESS>npm ERR! Windows_NT 10.0.10586
npm ERR! argv "C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app\apm\bin\node.exe" "C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app\apm\node_modules\npm\bin\npm-cli.js" "--globalconfig" "C:\Users\alber\.atom\.apm\.apmrc" "--userconfig" "C:\Users\alber\.atom\.apmrc" "install" "C:\Users\alber\AppData\Local\Temp\d-116731-2448-wt05yr\package.tgz" "--runtime=electron" "--target=0.37.8" "--arch=ia32" "--global-style" "--msvs_version=2015"
npm ERR! node v4.4.5
npm ERR! npm  v3.10.5
npm ERR! file C:\Users\alber.atom.apm\inherits\2.0.1\package\package.json
npm ERR! code EJSONPARSE
npm ERR! Failed to parse json
npm ERR! Unexpected token '\u0000' at 1:1
npm ERR!
npm ERR! ^
npm ERR! File: C:\Users\alber.atom.apm\inherits\2.0.1\package\package.json
npm ERR! Failed to parse package.json data.
npm ERR! package.json must be actual JSON, not just JavaScript.
npm ERR!
npm ERR! This is not a bug in npm.
npm ERR! Tell the package author to fix their package.json file. JSON.parse
npm ERR! Please include the following file with any support request:
npm ERR!     C:\Users\alber\AppData\Local\Temp\apm-install-dir-116731-2448-8qnc6o\npm-debug.log`
^ There was no debug log (no apm-install-dir...) in my Temp folder, but I did find the "Atom Crashes" operation_log which is attached.
operation_log.txt
When I try to install "remote-edit" or "remote-sync", I get a similar error.
Dragging it manually in to the "packages" folder gives me the error below.
I have tried to reinstall Atom a few times, nothing seems to fix this.
Error: Cannot find module 'fs-plus'     at Module._resolveFilename (module.js:339:15)     at Function.Module._resolveFilename (C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app.asar\src\module-cache.js:383:52)     at Function.Module._load (module.js:290:25)     at Module.require (module.js:367:17)     at require (C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app.asar\src\native-compile-cache.js:50:27)     at Object.module.exports.activate (C:\Users\alber\.atom\packages\remote-ftp-master\lib\remote-ftp.js:40:9)     at Package.module.exports.Package.activateNow (C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app.asar\src\package.js:183:20)     at C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app.asar\src\package.js:156:32     at Package.module.exports.Package.measure (C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app.asar\src\package.js:92:15)     at C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app.asar\src\package.js:149:26     at Package.module.exports.Package.activate (C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app.asar\src\package.js:146:34)     at PackageManager.module.exports.PackageManager.activatePackage (C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app.asar\src\package-manager.js:538:21)     at C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app.asar\src\package-manager.js:519:29     at Config.module.exports.Config.transactAsync (C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app.asar\src\config.js:337:18)     at PackageManager.module.exports.PackageManager.activatePackages (C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app.asar\src\package-manager.js:514:19)     at PackageManager.module.exports.PackageManager.activate (C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app.asar\src\package-manager.js:497:46)     at C:\Users\alber\AppData\Local\atom\app-1.10.0\resources\app.asar\src\atom-environment.js:765:28
I use the latest version of Atom and your package.
I don't know if I have done something wrong here, but I've done everything I could think of to try and fix this problem.
Thank you in advance!

What version of Windows are you running?

@jimmaaay Windows 10

Have no idea what the first error really is. I guess you could try reinstalling atom or see if you can find the package.json C:\Users\alber.atom.apm\inherits\2.0.1\package\package.json. Looks as though there may be an error with the json there?
However you could try adding it in manually. First you would drag the remote ftp folder into packages, then go to that directory in powershell or cmd and run apm install. That should fix the issue you had when you tried doing it manually.


Prevent duplicate arguments from getting into a single collapser RequestBatch

If this is attempted, then 1 of 2 things can occur:
If request-caching is on: the response for the 2nd-nth instance of the argument is the same as the first
If off: the response for the 2nd-nth instance of the argument is an error
This fixes the failing unit test supplied in #1176

Isn't this kind of breaks collapser with global context? We don't use request context (because we don't have multiple requests for the same data within one request) and we don't use request caching because of this. At the same time, our service takes an input that is user-generated and may (and will) contain duplicates. As we want to minimize the impact on services down the flow we use global context for collapser here and that's where the issue appear since now a whole ton of errors fall through...

@edudar Thanks for the report, this case didn't come to mind during the fix above but seems like a valid one.  We actually ended up not using global-collapsing anywhere internally, as our use of request-level state makes it impossible.
Any suggestions on how this might be fixed for your case?  What happens if you turn request-caching on?

Thanks @mattrjacobs
Request-caching does nothing actually.
Incoming load is looks like enormous amount of request to the single resource (much simplified), thus, caching on a request level creates cache with single item within, and it is disposed when response is sent to the user.
We collapse all similar requests and produce a single call to downstream service instead of tons of them.
Other use-case is the counter service (update service) when request cashing can't be enabled from the logic perspective. It counts amount of similar requests. And we want to increment persistent counter one time with +100 instead of 100 times with +1. Request caching in this case produces only single +1 operation for all 100 duplicates during collapse interval.
Looks like the easy check for the collapser scope before throwing an exception should solve the issue. Global scope may and will have duplicated requests and there is no way to enable request caching. Request-scopes, on the other hand, will and probably should use caching, thus, it may be natural to not allow duplicates.
My opinion, is that the logic for handling duplicates should be in a final collapser implementation and it should decide what to do with duplicates, ignore, throw an error or aggregate them in a custom way, etc.


[MUGEN] Add checkpointing wrapper

Stack from ghstack (oldest at bottom):

-> #176
#175
#166

Codecov Report

Merging #176 (5b57b24) into gh/langong347/13/base (34ccb66) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@                    Coverage Diff                    @@
##           gh/langong347/13/base     #176      +/-   ##
=========================================================
+ Coverage                  92.80%   92.84%   +0.03%
=========================================================
  Files                         47       48       +1
  Lines                       2822     2837      +15
=========================================================
+ Hits                        2619     2634      +15
  Misses                       203      203

Impacted Files
Coverage Δ

...chmultimodal/modules/layers/transformer_decoder.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 34ccb66...5b57b24. Read the comment docs.

@langong347 has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.

@langong347 has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.


Feature Request: Catch job submission errors?

Not sure if this one is easy or not. This is definitely an end-user issue. I was just trying to submit a bunch of jobs with looper at the same time (>50) and 2 of the jobs werent actually getting submitted but looper was saying that they were getting submitted.
Because the output of looper is pretty heavy, I had a ton of trouble sifting through it to figure out what was going on. Turns out I was requesting an impossible combination of resources for those two jobs (because those input files happened to be bigger than the rest).
Those jobs were failing submission with sbatch errors. The sbatch errors were reported in the looper output but looper was saying that they had been submitted (which, technically speaking, they had been, but immediately failed). If looper could catch these errors and report them in the final line, that could be helpful.

makes perfect sense.
we should definitely be able to capture the return value of whatever command we execute (in this case sbatch and use that to modulate looper's submission summary.
good catch.

I remember a conversation about this sort of thing...the impossibility or peering into a user's cluster configuration, and the intractability of predicting or enumerating all of the possibilities. This seems like a great way to accomplish almost the same end result, in a fashion that's actually feasible :+1:

@rcorces This is now implemented in 0.7. thank @vreuter


Convenient to keep the db root ca in the project directory

Pull Request Description
We may want to keep the root ca file in the project directory but we don't want it under version control.

I think this makes sense. Should deal with any path issues.


⚠ Processors has degraded performance

In 8027465, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 51 ms

Resolved: Processors performance has improved in 50bd70b.


Zshrc lines not working

I have placed the required line (cat ~/.cache/wal/sequences &) in my .zshrc, but new terminals starts with the result of this cat (i.e., it prints the content of the file). The colors of urxvt are not changed. I would obviously like to have the reversed, colors changed and not line at the start of a new terminal.
How can I investigate this undesired behavior?

what is the content that gets print out into the terminal?

Oh, I have just solved the problem by answering your question! My cat utility is aliased to bat (https://github.com/sharkdp/bat), and if I use \cat .cache/wal/sequences (with a backslash to have the real cat), I get the expected behavior (ie, a theme change).
For reference, here is the content of the file:
4;0;#14140b4;1;#7487924;2;#7993A34;3;#8D92924;4;#99A1A44;5;#AFB1B04;6;#C1BCB44;7;#e1dbd24;8;#9d99934;9;#74│87924;10;#7993A34;11;#8D92924;12;#99A1A44;13;#AFB1B04;14;#C1BCB44;15;#e1dbd210;#e1dbd211;85]#14140b12;#e1db│d213;#e1dbd217;#e1dbd219;#14140b4;232;#14140b4;256;#e1dbd2708;[85]#14140b
Thank you!

Pywal is not changing at real-time, I have to do that manually. how can I solve this


Needs updating for pythondata* submodules

Hi, I had noticed that Litex had switched things around to start using pythondata*  submodules.   It seems the configuration here needs a bit of updating to match, or else, it should point to an older version of Litex.
When I follow the instructions, I encounter this:
Traceback (most recent call last):
  File "/home/tim/antmicro/zephyr-on-litex-vexriscv/third_party/litex/litex/__init__.py", line 10, in get_data_mod
    exec(imp, {}, l)
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'pythondata_cpu_vexriscv'

In third_party/litex, I am at a39660fa99b27b765e9115dcd3c77fde53db5906 (Sept 17).

Hi @tcal-x,
Thanks for reporting the issue!
I can confirm that two submodules are currently missing: pythondata-cpu-vexrisv and pythondata-software-compiler_rt.
I prepared a fix at: https://github.com/antmicro/zephyr-on-litex-vexriscv/tree/add_missing_repos and we are testing it on HW.

I guess this one is already fixed.


changehc indicator outputs no data for hhs, nation when data is present at lower geo levels

[Slack] context
In the changehc indicator, it seems to be possible for county/state/hrr data to pass minimum denominator thresholds when hhs/nation data does not. Our current hypothesis is that this is due to smoothing numerators and denominators separately and after geo aggregation: the smoother is being overly enthusiastic about curve fitting, such that some counties/state/hrrs may be concave-up when the majority of their siblings are concave-down, causing the overall curve for hhs or nation to fall below threshold.
There are several suspicious things going on here:

Smoothing after geo aggregation (though doctor-visits also does this)
Smoothing the numerator and denominator separately before computing the rate
Backfill causes the counts to be artificially low for the most recent period of data, and the smoother is probably not a good tool for modeling this effect
~Zero-filling (iirc the smoother expects NaNs, not zeros)~ (this turns out to be true zeros -- by definition CHNG does not report if the count is 0)
No corner-case tests, only completion tests
~how did this ever work~ (turns out pandas assigns NaNs for the remaining indices, which is fine)

To fix:

Write a test that fails when the relationship between having-data is not hierarchical
Consider smoothing ratios instead of numerators/denominators
Switch to using raw denominators for thresholding
Consider whether backfill is adding 2x smoothing
Consider reducing the bandwidth? though this would have other ramifications

Sample data

There are several suspicious things going on here:

Smoothing after geo aggregation (though doctor-visits also does this)

I think smoothing must be done after geo-aggregation, not before. do you agree?

I generally expect the following to be true for both smoothed and unsmoothed signals:
pa_state_df = covidcast.signal(...)
pa_counties_df = covidcast.signal(...)
censored_pa_counties_df = [...magic...]
sum(pa_counties_df) + sum(censored_pa_counties_df) == pa_state_df

which would imply smoothing before aggregation, no?

Hmm..  why not the following?

Aggregate unsmoothed, uncensored counts to all geo-levels.
From here on, treat each geo-level separately, for the purpose of censoring, smoothing, magic, etc.

just to verify I'm reading you right -- if you fetched data like this from our API, you would not find it suspicious?
signal: smoothed_count
region  value
hhs1    10
hhs2    10
hhs3    10
hhs4    10
hhs5    10
hhs6    10
hhs7    10
hhs8    10
hhs9    10
hhs10   10
usa     90

Perhaps, but the documentation should explain it.

Smoothing is done primarily due to small counts.  Even when it's done for other reasons (e.g. temporal fluctuations), the absolute magnitude of the counts is very relevant for determining the appropriate amount of smoothing.  Therefore, when geo-aggregating, we need to know the actual aggregated counts before smoothing.
Smoothing is data manipulation, which may or may not be desired by the end user (which is why we should give the user the choice of whether and how to do it);  Smoothing implies some modeling assumption.  In contrast, aggregation is straightforward and makes no assumptions.

The apparent inconsistency above does not seem problematic to me (it would be problematic if it held for the unsmoothed counts).  But perhaps there were other reasons to do smoothing before aggregation -- we should figure out who was involved in that decision, and ask them.

Okay, I'm happy with that.
I will note that we have tons of signals for which we do not publish the unsmoothed values (doctor-visits, hospital-admissions,  chng, the majority of the survey signals). In cases where we decided not to publish unsmoothed values because we wanted to conserve database space, we can and should switch to providing both smoothed and unsmoothed values once we have the API server infrastructure available for performing smoothing at query time. However, I suspect we decided not to publish unsmoothed values for doctor-visits for either data privacy or DUA reasons, which would prevent us from giving users that choice.

Agreed.
That check could be made at query-time and be source/signal-specific.  However, this introduces a vulnerability, where the restricted information is in the DB and is protected only by the integrity of the API code.  Which means that the DB itself is vulnerable if hacked.
An alternative might be to only store smoothed values in these cases.   Another alternative, assuming we shard the DB, is to specially designate and protect those shards which contain restricted information.


Skin loads fine on Museum but not demo site

https://skins.webamp.org/skin/863997d994c7a7ff2059e4a92a2a6357/trh_sakura kidou 123.wsz/

It looks like the file paths are parsed differently on the two platforms:
Winamp Skin Museum

Demo Site

I suspect that these are two different versions of JSZip

Fiddling with different versions of JSZip on https://codesandbox.io/s/sleepy-goldstine-4d6fc?file=/src/index.js I've been unable to trigger the demo site behavior.
I also noticed that often the JSZip.version runtime property is not quite right.


Single keybindings l,h,etc action doesn't occur until another key is pressed

When I am in the dired mode the commands l, h, j, or any single letter command just pauses and waits for another command to be input before performing the action. I can see this with sublime.log_commands(). Pressing l will do nothing until I press another key for example ctrl+n which I have set to scroll down one line. Then it will collapse the folder and go down one line at the same time.
I am running on MacOSX and using the sublemacs package for keybindings so not sure if this is conflicting but it shouldn't be as none of the bindings are the same. Is this the normal behavior or is something on my system conflicting with the bindings to cause the pause?

It is not normal, and I cannot reproduce it on Windows 8.1, even with sublemacspro installed.
Interesting, similar issue was reported recently in https://github.com/aziz/PlainTasks/issues/271 and @sdaza uses OSX too, it seems.
Try to disable sublemacspro by adding it into ignore list in main preferences, restart ST; if it is not the reason, try on clean profile.

I figured out Autospell is the culprit using FindKeyConflicts because of a keybinding conflict. I have just removed the package and it works fine. I also can't use the copy, move, and paste commands. The bindings from sublemacs takeover these in the dired windows. I think it overwrites the emacs bindings and the regular super bindings so it just does a regular copy, paste, and cut not the dired versions. Is there a way to tell it to overwrite the super copy, cut, and paste commands just in the Dired mode windows or context? Maybe I can just set the copy, cut and paste to different bindings and all should be fine.

Nevermind if I copy over the bindings for super into my user key bindings it works properly in the file windows with copy, cut, and paste. Cheers for the help

Er, slow typing, still worth to mention that ST takes keybinding files in certain order http://docs.sublimetext.info/en/latest/extensibility/packages.html#merging-and-order-of-precedence
I’m not sure what you mean by super, but if all work let’s close it.

Sorry only recently switched to mac so meant using the command key but super+c and super+v that you have set in the default key bindings. I just copied them to the keybindings for the user which I'd presume are read last. Yes it is closed.

:smile: too many contexts, you meant ⌘, I thought about super in Python
Speaking of Autospell, I think it can be fixed on their side, if it matters, suggest checking if view is editable (it is easy via ST API), and do not operate in read-only view, theoretically it should be possible.


GH-44795: [C++] Use arrow::util::span on arrow::util::bitmap_builders_utilities instead of std::vector

Rationale for this change
arrow::util::span (a backport of C++20 std::span) is more generally applicable than std::vector, so any public API currently accepting a vector const-ref argument should instead accept a span argument.
What changes are included in this PR?
arrow::util::BytesToBits accepts arrow::util::span  instead of std::vector
Are these changes tested?
Yes, existing C++ tests via CI
Are there any user-facing changes?
Yes,  from Result<std::shared_ptr<Buffer>> BytesToBits(const std::vector<uint8_t>&, MemoryPool* pool) to Result<std::shared_ptr<Buffer>> BytesToBits(util::span<const uint8_t> bytes, MemoryPool* pool)

GitHub Issue: #44795

@raulcd Yes, this is the kind of change I had in mind. See comments.


Update DG

Coverage remained the same at 90.504% when pulling f66e21ac16bdb98f013f3b078d667a588b4ff203 on jaivigneshvenugopal:DG_update into 420661063a3ca1208e35c440825ae8b22115970d on CS2103AUG2017-T17-B1:master.

Coverage remained the same at 90.504% when pulling f66e21ac16bdb98f013f3b078d667a588b4ff203 on jaivigneshvenugopal:DG_update into 420661063a3ca1208e35c440825ae8b22115970d on CS2103AUG2017-T17-B1:master.

Coverage remained the same at 90.504% when pulling b5d72c7c591c2590041e55fe2575b8598fb3e97e on jaivigneshvenugopal:DG_update into 420661063a3ca1208e35c440825ae8b22115970d on CS2103AUG2017-T17-B1:master.

Coverage decreased (-0.4%) to 90.144% when pulling 506831a1c40930b64633483d2f5c1ed845436996 on jaivigneshvenugopal:DG_update into 420661063a3ca1208e35c440825ae8b22115970d on CS2103AUG2017-T17-B1:master.

Coverage remained the same at 90.072% when pulling 518904f2bc1e6c2dcaa3db1375af80d99a409ff5 on jaivigneshvenugopal:DG_update into 02f09d267aec601073f9913618df49ac6dd90b89 on CS2103AUG2017-T17-B1:master.

Coverage remained the same at 90.072% when pulling 518904f2bc1e6c2dcaa3db1375af80d99a409ff5 on jaivigneshvenugopal:DG_update into 02f09d267aec601073f9913618df49ac6dd90b89 on CS2103AUG2017-T17-B1:master.

Coverage increased (+0.02%) to 90.094% when pulling c1796b7f015428a7ae83f7bf5397a3ef1859b0ca on jaivigneshvenugopal:DG_update into 970674fa0ea13ccc15e5fff8b7102a05ccf745ee on CS2103AUG2017-T17-B1:master.

Coverage increased (+0.02%) to 90.097% when pulling 5c95ac31a4b5449ab3fa58bb3db3298163dba622 on jaivigneshvenugopal:DG_update into 970674fa0ea13ccc15e5fff8b7102a05ccf745ee on CS2103AUG2017-T17-B1:master.


[FEAT] Add onComplete Callback

Issue
Sometime we need to add some handling logic whither the task successed or failed , causing code duplication
callApli().onSuccess {
         hideLoader()
      }.onFailure {
         hideLoader()
      }

To fix this now we can use onComplete callback
callApli().onComplete { res, e ->
         hideLoader()
      }

Now we can handle task completion regardless of the outcome

is there any side effect for calling onCompleteUI and onComplete together?


New XML element: potential_vulnerability

Output from nessus-6.3 contains a new element:
 New XML element detected: potential_vulnerability. Please report this at http://github.com/arxopia/risu/issues/new or via email to<EMAIL_ADDRESS>

Hi, do you have any context on the contents of the xml tag?
I will get this added to 1.7.5

still on my list of things to do, it's gnarly digging through that xml i need to make some smaller files to make it easy

Added in 1.7.5.


Add to official repository

Hello,
Will you want to add this plugin to official repository?

Thank you.


[docs] jss-to-tss migration advise to drop clsx in favor of cx

Hi everyone,
Another addition to the jss-to-tss migration guide.
I realised that most people migrating from v4 do not realise that clsx !== cx so I mention it.
Best regards,
https://deploy-preview-30527--material-ui.netlify.app/guides/migration-v4/#2-use-tss-react

I don't understand the diff. In https://deploy-preview-30527--material-ui.netlify.app/guides/migration-v4/#2-use-tss-react, it does the opposite:


angular-cli is not working

I'm opening this issue because:

[0] angular-cli is doing something I don't understand.

What's going wrong?
(after migrating into desired folder)when i ran -
ng new my-app
this is what i got -

even when i ran -
ng --version
i am getting the same error
supporting information:

npm -v prints:  5.6.0
node -v prints: v8.9.3
npm config get registry prints: https://registry.npmjs.org/
Windows, OS X/macOS, or Linux?: Linux
Network issues: none

What does  migrating into desired folder mean in your report? I cannot see this problem when running ng new myself.

"migrating into desired folder" means, going inside the folder in which i want to create my angular app using angular-cli

Closing as there hasn't been any activity for a while.


Eq 2266 dump submission failing on schemas with repeating groups s

What is the context of this PR?

A minor fix to dumping the submission, because an unset global was relied on. Adds a test for this case.
As part of 1, reduces (but doesn't entirely remove) use of g.schema throughout. This means passing the schema as an argument a lot more, particularly in the templating where this wasn't done before. This appears to be because until repeating groups were added, there was no need for knowledge of the schema in these places.
Due to the reduced use of g.schema, _answer_is_in_repeating_group has been removed from rules.py, which has required modification of a boat load of tests since this was mocked in many places. In general, I've tried to patch a higher level call - i.e. mock out evaluate_goto instead of the low level one. This is to try to isolate units more.
A couple of tests have been removed, either were they were testing another unit by proxy, and in one case there was an identical test under two names.

This PR has ended up being bigger than intended. Some of the unit tests in templating have changed more than they strictly had to, however since I needed to change all the call signatures anyway, I wanted to isolate the units more whist I was understanding what each test did.
How to review

Tests should all pass!
Running census_household.json or other surveys with repeating groups should allow dumping of the submission on /dump/submission. To ensure the fix works, test one is failing on master first (should return a 500 error)

I missed a few instances of _answer_is_in_repeating_group which appeared after rebasing, working through those now

@benhowes can you try and hold off with squashing until you have 2 ticks? it just makes re-reviews a lot easier as we can just re-review the changes and not have to look over all the files again.

@kierenbeckett @jonnyshaw89 since you reviewed, I added some tests and removed some unreachable code in this commit: https://github.com/ONSdigital/eq-survey-runner/pull/1694/commits/80920a80caf634bab0c81e98f366b22a02d2b7b9, can you have a look over to ensure you're happy with the changes?

👍 Happy


[Feature Request] Mouse Cursor toggle in the Launcher possible?

Hello!
I'm well aware of the mouse cursor toggle for the games, but I was wondering if it could be expanded to work in the launchers (MGS2 and 3) itself somehow?

This should be added in the next release.

Cheers and thanks for the incredible work you and everybody contributing have done!


Update link for MiRFleet

This PR updates the link pointing to the fleet adapter for MiRFleet after the package has been merged to main.

@xiyuoh this looks like the same as another PR that just got merged. Should we close/cancel this one?

 Will close it, thanks!


Support kotlin coroutines

I don't see any mentioning of kotlin coroutines adapter or something like that, does it exist? If it doesn't, it would be really great to have such integration

Unfortunately I'm not familiar with Kotlin.
Can you make some draft hot it should look like?
Are there any adapters for blocking feign client (aka OpenFeign)?

add suspend support please, or is there an alternate?


Fix Nplus directives definition

Proposed changes
In #100 we provided support files for ParseOptions.DirectiveSources. For some reason analyze_nplus_latest_directives.go was override to a legacy version with mismatch from NGX_CONF_TAKE4 to ngxConfTake2. This PR is to fix those directives(keyval and limit_req_zone). Thanks to @oliveromahony found this problem.
Checklist
Before creating a PR, run through this checklist and mark each as complete.

[x] I have read the CONTRIBUTING document
[x] If applicable, I have added tests that prove my fix is effective or that my feature works
[x] If applicable, I have checked that any relevant tests pass after adding my changes
[ ] I have updated any relevant documentation (README.md)

Approved, but let's think about how we might be able to test this kind of scenario to see if there are any more errors (and prevent them in the future).

Sure! I think such problems boil down to some small errors in generator leading to error in only one or two directives. Current UT unlikely to discover such problems. I'm thinking of adding a huge "standard map" which includes most of directives to test the latest OSS/Nplus matchFunc(kind of like what I was doing in POC).

@dengsh12 tests that will cover the generator tool will be useful when that lands. Maybe that's enough if there's enough coverage of the bitmasks.

Approved, but let's think about how we might be able to test this kind of scenario to see if there are any more errors (and prevent them in the future).

Sure! I think such problems boil down to some small errors in generator leading to error in only one or two directives. Current UT unlikely to discover such problems. I'm thinking of adding a huge "standard map" which includes most of directives to test the latest OSS/Nplus matchFunc(kind of like what I was doing in POC).

yet another case where a corpus of anonymized NGINX configs would be real handy

update: added ut from @oliveromahony's PR #104