More specific feedback when brew audit fails for bad URLs

Provide a detailed description of the proposed feature
When brew audit fails due to a specific field, printing out which field would be ideal. It may do this already for certain failures, but in the specific case of pulling a URL it was just telling me the URL returned a 404 and it wasn't clear that the 404 was related to the homepage field specifically.
What is the motivation for the feature?
I was creating a formula with a private GitHub repository as the homepage for a private tap and it was failing pulling a URL and because I was using the repository as the source for the formula it wasn't clear that it had to do with the home page.
$ brew audit cdayjr/private/private --new-formula
cdayjr/private/private:
  * The URL https://github.com/cdayjr/private is not reachable (HTTP status code 404)
Error: 1 problem in 1 formula detected

I was able to trace this down to the --online flag specifically by trying different flags, but it still wasn't totally clear that this had something to do with the homepage field.
How will the feature be relevant to at least 90% of Homebrew users?
It'd save a lot of mostly aimless troubleshooting to get feedback about what field is failing when brew audit fails.
What alternatives to the feature have been considered?
Additional documentation may help, it should be possible to update https://docs.brew.sh/Formula-Cookbook#fill-in-the-homepage to mention that the homepage must be publicly accessible. But ideally the feedback for failure should be in the tool when it fails.
Maybe detecting if it's a private repo before the homepage check would help in my specific use-case, but for other kinds of URLs that could fail I still think the feedback would be more helpful.

I was creating a formula with a private GitHub repository as the homepage for a private tap and it was failing pulling a URL and because I was using the repository as the source for the formula it wasn't clear that it had to do with the home page.

Good idea! We'd accept a PR to improve this messaging.

Fixed by https://github.com/Homebrew/brew/pull/11203


Add preliminary doc on calibration metrics

Explains different metrics and how to use them

I just noticed that the remaining unaddressed comments are exactly the comments hidden by github, so sending a heads-up that there is 12 comments left :-)

Oops.  I missed those.  There are too many comments, I am losing track!  I think I addressed them all now.


minimal test for bccache; fix #227

Same as https://github.com/mitsuhiko/jinja2/pull/240 ?

Looks like it.


Arithmetics operation update

@forman: I like the diff op, but I still disagree with the arithmetic op. We should really move to an op, where users can specify a normal expression which we either

eval without globals but with a safe locals environment, see cate.util.safe_eval()
and/or  use numexpr for boosting performance.

Idea: use new new_expression_op() function and register it as operation compute with expression as only input.
Problems:
This leads to a workflow step

with no other step inputs than expression, therefore it appears as it wouldn't have any dependencies to other steps. But actually, the expression may refer to any other step output, be referencing it by its resource name.
with a single return value of unknown data type, because an expression may evaluate to any Python data type.

The latter problem may be avoided by offering dedicated compute_<type> operations.
The first may be avoided by treating steps containing a compute operation (or one containing general Python expressions evaluated in the context of the workflow) as being dependent on all other steps. This way they would be re-invoked if any other step input changes.


Fix merging rights on main for this repositories

I think the merging rights on main should be updated for this repository?

⚠ Should be done by a MeiliSearch team member.
@curquiza @eskombro

I've protected the main:

only the integration team can merge (push on main) since we don't have bors on this repo. For the moment you are in the integration team @nicolasvienot 😉 so you can merge
the pylint check is required
1 reviewer is needed
if you push another commit the approval is dismissed.

If bors should be added, we should open another issue 🙂

@nicolasvienot We can add bors as soon as we add the tests :)


CONTACT address update

Want to update a contact and remove the address completely?  While not something that you will run into alot, it might be needed.  The SDK has a check that says if all fields are blank then do not include those in the function call, and so if you wanted to clear them, you cannot.  I would suggest that this type of functionality to deciding to exclude nodes due to being blank, shouldn't be done.

Hi @glickerman, if you are unhappy with the provided function classes you are encouraged to implement your own using the IFunction interface and/or AbstractFunction class.  We felt this was the most appropriate way to address everyone's individual needs instead of forcing you to use all of our own classes.
You could likely clone what exists for the contact classes and then override the lines here to compare AddressLine1 == null instead of !string.IsNullOrWhiteSpace(AddressLine1).
We can look at pushing down the WriteXmlMailAddress function into the CreateContact and UpdateContact classes themselves, but it will likely need to be in a new minor semantic version, like v2.1.


[BEAM-14118, #21639] Vendor gRPC 1.48.1

Linkage checker failures under 1.48.1:
Class org.eclipse.jetty.npn.NextProtoNego is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.ssl.JettyNpnSslEngine (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.eclipse.jetty.alpn.ALPN$ClientProvider is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.ssl.JettyAlpnSslEngine (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.eclipse.jetty.alpn.ALPN is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.ssl.JettyAlpnSslEngine (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.bouncycastle.asn1.x500.X500Name is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.ssl.util.BouncyCastleSelfSignedCertGenerator (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.bouncycastle.cert.jcajce.JcaX509v3CertificateBuilder is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.ssl.util.BouncyCastleSelfSignedCertGenerator (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.bouncycastle.operator.jcajce.JcaContentSignerBuilder is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.ssl.util.BouncyCastleSelfSignedCertGenerator (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.bouncycastle.cert.X509v3CertificateBuilder is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.ssl.util.BouncyCastleSelfSignedCertGenerator (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.bouncycastle.cert.jcajce.JcaX509CertificateConverter is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.ssl.util.BouncyCastleSelfSignedCertGenerator (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.bouncycastle.jce.provider.BouncyCastleProvider is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.ssl.util.BouncyCastleSelfSignedCertGenerator (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.eclipse.jetty.npn.NextProtoNego$ClientProvider is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.ssl.JettyNpnSslEngine (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.eclipse.jetty.npn.NextProtoNego$ServerProvider is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.ssl.JettyNpnSslEngine (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.eclipse.jetty.alpn.ALPN$ServerProvider is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.ssl.JettyAlpnSslEngine (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.jcraft.jzlib.JZlib$WrapperType is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.ZlibUtil (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.jcraft.jzlib.JZlib is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.ZlibUtil (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.jcraft.jzlib.Inflater is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.ZlibUtil (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.jcraft.jzlib.Deflater is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.ZlibUtil (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.aayushatharva.brotli4j.decoder.DecoderJNI$Status is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.BrotliDecoder (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.github.luben.zstd.Zstd is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.ZstdEncoder (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.aayushatharva.brotli4j.encoder.Encoder$Parameters is not found;
  referenced by 3 class files
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.StandardCompressionOptions (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.BrotliOptions (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.BrotliEncoder (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.aayushatharva.brotli4j.decoder.DecoderJNI$Wrapper is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.BrotliDecoder (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.aayushatharva.brotli4j.encoder.Encoder$Mode is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.BrotliOptions (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.JZlibEncoder is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.ZlibCodecFactory (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.JZlibDecoder is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.ZlibCodecFactory (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.aayushatharva.brotli4j.encoder.Encoders is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.handler.codec.compression.BrotliEncoder (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.slf4j.Logger is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.util.internal.logging.Slf4JLogger (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.slf4j.LoggerFactory is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.util.internal.logging.Slf4JLoggerFactory (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.slf4j.helpers.NOPLoggerFactory is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.util.internal.logging.Slf4JLoggerFactory (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.slf4j.spi.LocationAwareLogger is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.util.internal.logging.Slf4JLoggerFactory (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.apache.beam.vendor.grpc.v1p48p1.io.netty.util.internal.logging.LocationAwareSlf4JLogger is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.util.internal.logging.Slf4JLoggerFactory (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class reactor.blockhound.integration.BlockHoundIntegration is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.util.internal.Hidden (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown
Class reactor.blockhound.BlockHound$Builder is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p48p1.io.netty.util.internal.Hidden (org.apache.beam:beam-vendor-grpc-1_48_1:0.1-SNAPSHOT)
  Cause:
    Unknown

Linkage checker failures under 1.43.2:
Class org.eclipse.jetty.npn.NextProtoNego$ServerProvider is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.ssl.JettyNpnSslEngine (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.bouncycastle.asn1.x500.X500Name is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.ssl.util.BouncyCastleSelfSignedCertGenerator (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.bouncycastle.cert.jcajce.JcaX509v3CertificateBuilder is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.ssl.util.BouncyCastleSelfSignedCertGenerator (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.bouncycastle.operator.jcajce.JcaContentSignerBuilder is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.ssl.util.BouncyCastleSelfSignedCertGenerator (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.bouncycastle.cert.X509v3CertificateBuilder is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.ssl.util.BouncyCastleSelfSignedCertGenerator (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.bouncycastle.cert.jcajce.JcaX509CertificateConverter is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.ssl.util.BouncyCastleSelfSignedCertGenerator (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.bouncycastle.jce.provider.BouncyCastleProvider is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.ssl.util.BouncyCastleSelfSignedCertGenerator (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.eclipse.jetty.alpn.ALPN is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.ssl.JettyAlpnSslEngine (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.eclipse.jetty.npn.NextProtoNego is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.ssl.JettyNpnSslEngine (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.eclipse.jetty.alpn.ALPN$ClientProvider is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.ssl.JettyAlpnSslEngine (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.eclipse.jetty.alpn.ALPN$ServerProvider is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.ssl.JettyAlpnSslEngine (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.codec.compression.JZlibEncoder is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.codec.compression.ZlibCodecFactory (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.codec.compression.JZlibDecoder is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.codec.compression.ZlibCodecFactory (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.jcraft.jzlib.JZlib$WrapperType is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.codec.compression.ZlibUtil (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.jcraft.jzlib.JZlib is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.codec.compression.ZlibUtil (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.jcraft.jzlib.Inflater is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.codec.compression.ZlibUtil (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class com.jcraft.jzlib.Deflater is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.handler.codec.compression.ZlibUtil (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class reactor.blockhound.integration.BlockHoundIntegration is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.util.internal.Hidden (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class reactor.blockhound.BlockHound$Builder is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.util.internal.Hidden (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.slf4j.Logger is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.util.internal.logging.Slf4JLogger (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.slf4j.LoggerFactory is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.util.internal.logging.Slf4JLoggerFactory (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.slf4j.helpers.NOPLoggerFactory is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.util.internal.logging.Slf4JLoggerFactory (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.slf4j.spi.LocationAwareLogger is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.util.internal.logging.Slf4JLoggerFactory (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown
Class org.apache.beam.vendor.grpc.v1p43p2.io.netty.util.internal.logging.LocationAwareSlf4JLogger is not found;
  referenced by 1 class file
    org.apache.beam.vendor.grpc.v1p43p2.io.netty.util.internal.logging.Slf4JLoggerFactory (org.apache.beam:beam-vendor-grpc-1_43_2:0.1-SNAPSHOT)
  Cause:
    Unknown

The differences are in optional new compression algorithms zstd and brotli and NextProtoNego$ClientProvider which are safe to exclude.
Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @jigga @apilloud @emilymye

Run Java_Examples_Dataflow_Java11 PreCommit

Run Java PreCommit

LGTM - as a note, if you're also doing a BOM update, we should maybe revisit  https://github.com/apache/beam/pull/16672/ as well

Run Java PreCommit

LGTM - as a note, if you're also doing a BOM update, we should maybe revisit #16672 as well

No BOM update.

Last check succeeded but failed to update github ui: https://ci-beam.apache.org/job/beam_PreCommit_Java_Phrase/5475/console


[Mobile] "Cannot unlock while initializing dynamic indexing services: Check sessions/status/local"

Calling 'sessions/unlock/local' right after logging in gives me {"code": -139, "message": "Cannot unlock while initializing dynamic indexing services: Check sessions/status/local"} error. Trying again after about 10 minutes still gives this error.

This is an old issue related to old builds, has been resolved with new core.


Changed to DC

I dislike the use of D.C. and instead prefer DC. If you need authority, rely on the 18F Content Guide.

Your own personal style for DC spelling is authoritative enough for me!


python3Packages.xknx: 0.18.1 -> 0.18.2

Motivation for this change
Update to latest upstream release 0.18.2
Change log: https://github.com/XKNX/xknx/releases/tag/0.18.2

Related Home Assistant change: https://github.com/home-assistant/core/pull/50491
Target Home Assistant release: 2021.6.0
Target Home Assistant release: current, only dependency update no changes

Things done

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

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

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@ofBorg test home-assistant

Result of nixpkgs-review pr 122896 at 6bdea6c3 run on aarch64-linux 1

3 packages built successfully:

home-assistant
python38Packages.xknx
python39Packages.xknx


Validate our overall scope and goals

From our first in-person working session, we debated the following:

How do we address AI ethics / bias / safety / responsible?
Definition of in-scope technologies and adoption of standardised terminology for the group (outputs, training data, security of runtimes and SDLC, GRC)
What does “AI” mean: GenAI, LLMs, traditional ML
Agree what is in (and out of) scope for the group

Closed by https://github.com/finos/ai-readiness/pull/3


Upsert with has_many association

GORM Playground Link
https://github.com/go-gorm/playground/pull/447
Description
In this sample, I have a model Article with the field code that is indexed with a unique index. An article has many article names. So I have another model ArticleName with the foreign key to the article table article_id, a field name that is indexed with a unique index (the unique indices in this sample, are useful to perform upsert operations).
I want to ask if it is possible to upsert articles and article names as follow:
// Models

type Article struct {
	ID           uint64
	Code         string `gorm:"index:idx_code,unique"`
	ArticleNames []*ArticleName
}

type ArticleName struct {
	ID        uint64
	ArticleID uint64
	Name      string `gorm:"index:idx_name,unique"`
	Language  string
	Article   *Article
}

// Creation

articles := []*Article{
		{
			Code: "123",
			ArticleNames: []*ArticleName{
				{Name: "Jacket", Language: "en"},
				{Name: "Giacca", Language: "it"},
			},
		},
		{
			Code: "456",
			ArticleNames: []*ArticleName{
				{Name: "Shirt", Language: "en"},
				{Name: "Maglietta", Language: "it"},
			},
		},
	}

DB.Clauses(clause.OnConflict{UpdateAll: true}).
		Session(&gorm.Session{FullSaveAssociations: true}).
		Create(&articles)

// Update (case no changes)

articles = []*Article{
		{
			Code: "123",
			ArticleNames: []*ArticleName{
				{Name: "Jacket", Language: "en"},
				{Name: "Giacca", Language: "it"},
			},
		},
		{
			Code: "456",
			ArticleNames: []*ArticleName{
				{Name: "Shirt", Language: "en"},
				{Name: "Maglietta", Language: "it"},
			},
		},
	}

DB.Clauses(clause.OnConflict{UpdateAll: true}).
		Session(&gorm.Session{FullSaveAssociations: true}).
		Create(&articles)

// Update (case with changes)

articles = []*Article{
		{
			Code: "123",
			ArticleNames: []*ArticleName{
				{Name: "Jacket", Language: "en"},
				{Name: "Giacca", Language: "it"},
			},
		},
		{
			Code: "456bis",
			ArticleNames: []*ArticleName{
				{Name: "T-Shirt", Language: "en"},
				{Name: "Maglietta", Language: "it"},
			},
		},
	}

DB.Clauses(clause.OnConflict{UpdateAll: true}).
		Session(&gorm.Session{FullSaveAssociations: true}).
		Create(&articles)

Now the first DB.Create works and creates the 2 articles with names as expected. 🤟
The second DB.Create returns the following error:
Error 1452: Cannot add or update a child row: a foreign key constraint fails (`gorm`.`article_names`, CONSTRAINT `fk_articles_article_names` FOREIGN KEY (`article_id`) REFERENCES `articles` (`id`))
[5.139ms] [rows:0] INSERT INTO `article_names` (`article_id`,`name`,`language`) VALUES (0,'Jacket','en'),(0,'Giacca','it'),(0,'Shirt','en'),(0,'Maglietta','it') ON DUPLICATE KEY UPDATE `article_id`=VALUES(`article_id`),`name`=VALUES(`name`),`language`=VALUES(`language`)

As you can see, the article_id value is set to 0. How to avoid this?
Then the third DB.Create returns the following error:
Error 1452: Cannot add or update a child row: a foreign key constraint fails (`gorm`.`article_names`, CONSTRAINT `fk_articles_article_names` FOREIGN KEY (`article_id`) REFERENCES `articles` (`id`))
[8.834ms] [rows:0] INSERT INTO `article_names` (`article_id`,`name`,`language`) VALUES (5,'Jacket','en'),(5,'Giacca','it'),(6,'T-Shirt','en'),(6,'Maglietta','it') ON DUPLICATE KEY UPDATE `article_id`=VALUES(`article_id`),`name`=VALUES(`name`),`language`=VALUES(`language`)

In this case, the article_id are set to 4 and 5, but these articles do not exist.
Do you know how to pass the test I wrote in the linked playground?

+1

+1

+1

这个upsert只能根据主键来吧，你的操作对程序来说会新生成一个 主键id，但是由于你设置了唯一字段，无法成功插入，所以中间作为链接的表的外键值也不存在了


Upcoming Shows card not updating

I've been unable to get my Upcoming Shows card to update. It hasn't changed since the initial installation. I've tried reinstalling and rebooting but the same 3 shows persist after their air date/time.
Home Assistant Version: 2021.11.0
Trakt Integration: 0.1.1

Hi can you show me the trakt-integration yaml configuration and also the upcoming media card configuration.

configuration.yaml (indented correctly in yaml file)
trakt_tv:
language: en # Prefered language for movie/show title
sensors:
upcoming:
show:
days_to_fetch: 90 # How many days in the future you want to fetch
max_medias: 3 # How many medias you want to fetch
new_show:
days_to_fetch: 90
max_medias: 3
movie:
days_to_fetch: 90
max_medias: 3

upcoming media card (nothing indented in card yaml)
type: custom:upcoming-media-card
entity: sensor.trakt_upcoming_shows
title: Upcoming Shows
image_style: fanart
hide_empty: true
title_text: $title
line1_text: $episode
line2_text: $number
line3_text: $day, $date $time
line4_text: $empty
max: 10

Configurations seems to be fine, in developer tools, do you have the sensor trakt_upcoming_shows ?

sensor is there and populated. This morning it did update but it seems to be about a day behind on its update. It's Friday, and it shows Thursdays shows it but did remove Wednesday's shows. Not a big deal, at least its updating.

If the sensor is there, then it's not the fault of this integration. Can you compare with trakt if the sensor is too late, if it is too late then open a new ticket.

It appears to be late, as the "upcoming schedule" section on trakt does not match, one thing I noticed in the sensor is
airdate: '2021-11-05T00:00:00Z' (it shows 11-5 when actually its 11-4, perhaps a timezone issue?)  The date shows correctly in the upcoming media card as (Thursday 11-4 9:00PM). Timezone is set correctly in home assistant

Ok I will see what is going on

I saw nothing in the code, maybe the sensor was not updated yet. I close the issue for the moment.


qgrid is not working for a data from snowflake database

Environment

Operating System: Ubuntu

Python Version: 3.6

How did you install Qgrid: pip

Python packages:
jupyter==1.0.0
jupyter-client==5.2.2
jupyter-console==5.2.0
jupyter-contrib-core==0.3.3
jupyter-contrib-nbextensions==0.3.3
jupyter-core==4.4.0
jupyter-dashboards==0.7.0
jupyter-highlight-selected-word==0.1.0
jupyter-latex-envs==1.4.3
jupyter-nbextensions-configurator==0.4.0
jupyterhub==0.8.1
jupyternotify==0.1.15
qgrid==1.0.2
snowflake-connector-python==1.5.1
snowflake-sqlalchemy==1.1.0
SQLAlchemy==1.2.4
sqlalchemy-teradata==0.0.9.dev0
sqlparse==0.2.4

Jupyter lab packages (if applicable): $ jupyter labextension list

Description of Issue

What did you expect to happen? I execute a query and I expect qgrid to display the results.

What happened instead? It works when I run the query for the first time. When i run the same query, it runs forever. When I disable qgrid, there is no issues displaying results on notebook without qgrid. I see this issue when i run a query to get data from snowflake database. When i connect to any other database, qgrid works perfectly.

I am connecting database using SQL alchemy
Reproduction Steps

...
What steps have you taken to resolve this already?
...
Anything else?
...

Hmm not sure I understand, but are you saying that when you have qgrid enabled, it causes the execution of this database query to take forever? And then when you disable qgrid, the query succeeds in a normal amount of time?
It might be helpful if you could provide the code you're using to make the query and to construct the qgrid instance.  Thanks.


Natively get environment config to remove ember-get-config dependency

Fixes #297 and #298

v5.2.2
# Changelog

- Remove `ember-get-config` dependency #298
- Update documentation to not recommend using mixins #297
- Remove use of `ember-simple-auth` deprecated `data-adapter-mixin` #297
- Add support for `ember-cli-babel` v7

@jpadilla when you get a chance, can you please review?


Incorrect calculations

I have read the wiki about what to try first before creating a bug report.
Which custom card does give you issues?
power-wheel-card
Which version of HA do you use?
100.2
Which version of the card do you use?
0.0.15
Describe the bug
the calculation seems skewed: at the moment I'm not producing any energy as the sun is down.
Power flow to grid value from my inverter: 556w (energy drawn from the grid)
Power flow load value from my inverter: -556w (energy consumed)
Configuration of the card
- platform: template
  sensors:
    solar_power:
      friendly_name: 'Solar power production'
      unit_of_measurement: 'W'
      value_template: >-
        {% if is_state('sensor.power_photovoltaics_fronius_power_flow_0_http_10_20_20_58', 'unknown') %}
        0
        {% else %}
        {{ states("sensor.power_photovoltaics_fronius_power_flow_0_http_10_20_20_58" | float) | int }}
        {% endif %}
    power_consumption:
      friendly_name: 'Power Consumption'
      unit_of_measurement: 'W'
      value_template: >-
        {{  states("sensor.power_load_fronius_power_flow_0_http_10_20_20_58")  }}

    power_exported:
      friendly_name: 'Power Exported'
      unit_of_measurement: 'W'
      value_template: >-
        {{ states("sensor.power_grid_fronius_power_flow_0_http_10_20_20_58") }}

Expected behaviour
the arrow should point in the direction of energy flow and fix the calculations
Screenshots

Looks like you're feeding the card with two input sensors with same value. Can you share your card setup so I can check what parameters you are using? Look for - type: "custom:power-wheel-card" somewhere in your files and please share the code that's below it.

views:
  - cards:
      - grid_power_consumption_entity: sensor.power_load_fronius_power_flow_0_http_10_20_20_58
        grid_power_production_entity: sensor.power_grid_fronius_power_flow_0_http_10_20_20_58
        solar_power_entity: sensor.power_photovoltaics_fronius_power_flow_0_http_10_20_20_58
        type: 'custom:power-wheel-card'
    id: solar

https://photos.app.goo.gl/mXDFwpEng5YJtVZw6
This is how it shows on solarweb portal

Thx for the extra information.
Based on the picture you placed on google, I think your sensor.power_load_fronius_power_flow_0_http_10_20_20_58 is what matches the home symbol in the card. That one can be calculated and left out in the card config. You can also skip creating a template sensor for this one.
I expect your sensor.power_grid_fronius_power_flow_0_http_10_20_20_58 is a sensor that gives you the consumed-from-the-grid as wel as the produced-to-the-grid value, based on the sign of the value. You call this friendly_name: 'Power Exported' but a better name would be 'Power Grid'.
I'm not sure if you have to use production_is_positive: false, but you could check the sign of sensor.power_grid_fronius_power_flow_0_http_10_20_20_58 during the night. If it's negative, then you're already on the default value of production_is_positive which is true. Based on the direction of an arrow in your first image and the way you where feeding the card at that time, I would say, you're on the default already.
You'll end up with this:
- grid_power_entity: sensor.power_grid
  solar_power_entity: sensor.solar_power
  type: 'custom:power-wheel-card'

Hope to have helped you.


rectified the installation commands for various platform (Issue : #502)

Description
This PR fixes #
rectifying the installation commands for various platform #502
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

Why have you made changes to the Gemfile?

Why have you made changes to the Gemfile?

Actually I was facing problem with the version of ruby so had to make some changes there in order to make site run in my local system

Ohh okay, but you cannot commit those changes. Could you please revert the change back to what it was?

as @warunicorn19  said , Please do not commit your env specific code :-) Please revert, All other changes LGTM.

Ohh okay, but you cannot commit those changes. Could you please revert the change back to what it was?
Ohh okay, I think I have done that it's just a new line that has been added by mistake to gemfile, should I make a new PR in that case?

You could just change it back and make a commit to this PR itself, that should be fine

You could just change it back and make a commit to this PR itself, that should be fine

Yes,I have done that with a new commit and restored the gemfile to what it was previously. Does this work now?

@shahkushal38 thanks for this. Will you take a moment to remove the changes made to the gemfile and gemfile.lock?

Yes @leecalcote , I have done that in my last commit, can you please check whether the required changes have been made or else I redo it.

@shahkushal38 thanks for this. Will you take a moment to remove the changes made to the gemfile and gemfile.lock?

Yes @leecalcote , I have done that in my last commit, can you please check whether the required changes have been made or else I redo it.

Thank you. I still see both of these files included. Despite not having any material changes, they shouldn't be included at all. Thanks for having a look at this.


[Feature]: GlobalSetup after env

🚀 Feature Proposal
I was trying to setup a global beforeAll. I saw the globalSetup configuration option. However, I use jsdom as test environment and the setup involves with environment related mockups. I like the idea of setupAfterEnv which set up individual test suites after environment but I didn't find similar option for global setup.
Motivation
Global setup using test environment
Example
No response
Pitch
It is based on globalSetup configuration option

Will this be considered as feature addition?


Topointer 2

I made another revision, following your suggestion with the snprintf() buffer.
Additionally I figured, to complement toArrayBuffer(ptr, len) I could add an optional offset toPointer(arrayBuf[, offset]).
So I am proposing another small changest with only:

the changed toPointer(ab[, off])
a constant "pointerSize" which could simply be sizeof(void*).
a much simpler case in test.js: filling an ArrayBuffer with 0-terminated utf8, then toString(toPointer(buf)) it....

Any thoughts or suggestions from your side?

My specific use-case which suggested the toPointer() function was for conversion of a zero-terminated utf8 ArrayBuffer to a string:
ffi.toString(ffi.toPointer(buf))
Toyed around with some other possibilities the function opens, but it's too involved for test cases and of no real utility.
funky stuff in test.js
I'm currently investigating and experimenting in writing ArrayBuffer based interfaces to C structures of several system APIs (timeval, sockaddr_in, fd_set, socklen_t, winsize, termios).
For struct timeval its required to get the word-size of the platform, the sec & usec members change with it.
This led me to add the constant "ptrSize".
I had added also "argSize" (FFI_SIZEOF_ARG) and "littleEndian" (BYTE_ORDER) but didn't really need them:
There is already convenient machine byteorder versus network-byteorder handling through Uint16Array and DataView's setUint16(offset, value, littleEndian).
branch with API examples

My specific use-case which suggested the toPointer() function was for conversion of a zero-terminated utf8 ArrayBuffer to a string:
ffi.toString(ffi.toPointer(buf))
Toyed around with some other possibilities the function opens, but it's too involved for test cases and of no real utility.
funky stuff in test.js
I'm currently investigating and experimenting in writing ArrayBuffer based interfaces to C structures of several system APIs (timeval, sockaddr_in, fd_set, socklen_t, winsize, termios).
For struct timeval its required to get the word-size of the platform, the sec & usec members change with it.
This led me to add the constant "ptrSize".
I had added also "argSize" (FFI_SIZEOF_ARG) and "littleEndian" (BYTE_ORDER) but didn't really need them:
There is already convenient machine byteorder versus network-byteorder handling through Uint16Array and DataView's setUint16(offset, value, littleEndian).
branch with API examples

Background:
My specific use-case which suggested the toPointer() function was for conversion of a zero-terminated utf8 ArrayBuffer to a string:
ffi.toString(ffi.toPointer(buf))
Toyed around with some other possibilities the function opens, but it's too involved for test cases and of no real utility. funky stuff in test.js
I'm currently investigating and experimenting in writing ArrayBuffer based interfaces to C structures of several system APIs (timeval, sockaddr_in, fd_set, socklen_t, winsize, termios).
For struct timeval its required to get the word-size of the platform, the sec & usec members change with it.
This led me to add the constant "ptrSize".
I had added also "argSize" (FFI_SIZEOF_ARG) and "littleEndian" (BYTE_ORDER) but didn't really need them:
There is already convenient machine byteorder versus network-byteorder handling through Uint16Array and DataView's setUint16(offset, value, littleEndian).
branch with API examples

Roman
Thanks! The example is especially nice! (I can't actually publish what I use ffi for, because that is proprietary -- I make as
much open-source as possible).
It's nice to have people contribute.
Fred Weigel


Other repository with snippets

guys,  i have a repository that i made it public now, with snippets and others things from prado
could we merge it?
https://github.com/rspadim/Adv_Fin_ML/

I'm happy to look it over when I have some more free time during the next week or so. In the meantime I have already implemented a large portion of the snippets in the snippets.py script.


add raw response to handleError + additional API methods

We need the raw response (regToken etc...) when an error occured and some additional API methods for a project we are doing.
Is it possible to publish this to the public npm registry?

Hi.
I'll definitely get your commit added to the NPM registry, including the methods that I didn't add to the SDK.
Can you help me understand your use-case for including the raw JSON? The err object should essentially be a parsed version of the JSON at that point, including data on errors that is specific to a method (like validation errors).
There shouldn't be a difference between the JSON and the err object.

Hi Scott,
The reason we want access to the raw JSON is that the parsed error this lib returns doesn't contain all the data we want. I.e. Sometimes an errorCode ( != 0 ) is returned but the raw json contains a gigya user object (with UID, Signature, regToken). We want access to this userData regardless whether an error occured or not. Right now, in case of an errCode all we get is this:
 var e = new Error(json.errorDetails);
e.code = json.errorCode;
e.callId = json.callId;

The JSON returned from the gigya does contain a lot more information that is useful to us.
Like in the case of a errorCode 2006001 (Account Pending Registration) Gigya returns the user associated with this account. And we need this for our user login flow. Now all we get is a stripped down error object from this library.
Does that clarify our needs? Thanks!
{ UID: '61cb5a6c9cfa49c1aeae28b00c2b4a79',
  UIDSignature: 'YcH8KxiBYTFsUC8V48zgtJTmc3k=',
  signatureTimestamp: '1422433762',
  loginProvider: 'site',
  isRegistered: false,
  isActive: true,
  isLockedOut: false,
  isVerified: true,
  socialProviders: 'site',
  identities:
   [ { provider: 'site',
       providerUID: '61cb5a6c9cfa49c1aeae28b00c2b4a79',
       isLoginIdentity: true,
       email<EMAIL_ADDRESS>       allowsLogin: true,
       isExpiredSession: false,
       lastUpdated: '2015-01-27T13:16:25.803Z',
       lastUpdatedTimestamp:<PHONE_NUMBER>803,
       oldestDataUpdated: '2015-01-27T13:16:25.803Z',
       oldestDataUpdatedTimestamp:<PHONE_NUMBER>803 } ],
  created: '2015-01-27T13:16:25.771Z',
  createdTimestamp:<PHONE_NUMBER>771,
  lastLogin: '2015-01-28T08:29:22.122Z',
  lastLoginTimestamp:<PHONE_NUMBER>122,
  lastUpdated: '2015-01-27T13:16:25.803Z',
  lastUpdatedTimestamp:<PHONE_NUMBER>803,
  oldestDataUpdated: '2015-01-27T13:16:25.803Z',
  oldestDataUpdatedTimestamp:<PHONE_NUMBER>803,
  verified: '2015-01-27T13:17:56.821Z',
  verifiedTimestamp:<PHONE_NUMBER>821,
  regToken: 'RT1_stCAQRTU1RLBzY0MjUzMzEBAAAAAADAAQAAVM44ZkMYqJhdrBRFthCYaAKWyn0wRbM4ZIlpWA2qoaDigfnqoyi6rkjrpt2_24UnMQQIIoaUmnkehSiZTbLTZHCBXTGumrpWq0i4c154k2sy0Gf7fi--IT4g1ekeJJWMtKPt_2TXq4kl_fkQyXAdXc0ykZC1Zi0jX4Vxo3z0qMp8jaLdz3XqWX7hE_oNGFYXDN6ouzeS8PfvygiprhwxW3L7bj5Ym27rUemplrOTQHtdAd0C3TSctH5G5Rk92NBM5V1QRif4J6Pzp_gIBEBKxaw8mP6oXMQdrtQP5RZoN-9DCjPLorbcP77D2tP4YDY8W6TL1VlvE7YmSCKgGwJ4qKIxRzwh_wXMpVeYlr9v6oA4BZDv7s_z0FUYBm-ZQDGhH3_1Jtsev0fpB8YdpjkIk7EIZLqkyVju3NW3Xengb4XNp3x26j6ZkFz4CqpLWuwwZxz2am-vqKy1sIu1ibQj1wF4TWrXRqFjNa7HV8DduVcOIo3E49oRvmG-uh29kywDctN-wOxC-Q9k7xcvtI9zWImFcgBisGmAM9cyFdC6QRhVBSo5M9-SW1OD-v5FHGcXhDZnUgo4zB0m4bmyb2FSNBAAAAD7C-ZeWgpFTrB4PFD1TDR4',
  errorDetails: 'Missing required fields for registration: birthYear,country,gender,zip',
  errorMessage: 'Account Pending Registration',
  statusCode: 206,
  errorCode: 206001,
  statusReason: 'Partial Content',
  callId: '20e22af6bea04d28ace574b346243e77' }

I'll fix it so the error object includes everything. I'll also add the missing methods to the SDK.

Hi scott,
Any news on this issue?
Thanks,
Ilja

Merged. Thanks!

Thanks! I will remove our fork and depend on<EMAIL_ADDRESS>Kind regards,
I
On Thu, Jan 28, 2016 at 11:07 PM, scotthovestadt<EMAIL_ADDRESS>wrote:

Merged. Thanks!
—
Reply to this email directly or view it on GitHub
https://github.com/scotthovestadt/node-gigya/pull/2#issuecomment-176443111
.


Fix resolver workflow and update docs

End-user friendly description of the problem this fixes or functionality that this introduces

[ ] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

Fix resolver workflow and update docs

Give a summary of what the PR does, explaining any non-trivial design decisions
The LLM_MODEL was moved in this PR: https://github.com/All-Hands-AI/OpenHands/pull/5477/files
to be a variable. However, it did not have a default value and the docs were not updated to mention this is now a variable.

Link of any specific issues this addresses

To run this PR locally, use the following command:
docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:b29a4e1-nikolaik   --name openhands-app-b29a4e1   docker.all-hands.dev/all-hands-ai/openhands:b29a4e1

BTW I noticed that the PR I linked is essentially a breaking change since it removes LLM_MODEL from being taken from the secrets. We should be more careful about these kinds of changes. Depending on more changes down the line to the actual workflow, users may be forced to update their resolver.yaml files. Have you all thought about how to approach this? Perhaps in the examples, using a specific tag as opposed to main?
CC @neubig and @malhotra5

Yeah I'm not too sure about how to handle these changes. Usually, we can expect the reusable workflow to provide openhands resolver with secrets or variables. One strategy to mitigate this could be

Have user's set a SETTINGS_VERSION variable
On our workflow we could check against this version, and comment with error message requesting user to update their workflow
Whenever possible, set default values on our side


Example agent-pod YAML?

Might be nice to show an example agent-pod YAML; perhaps a tiny Python hello-world project that needs a build agent with both a Python CLI and a Docker CLI (for pushing to e.g., DockerHub)?

So I think for this, I will implement a Python library example first, then a docker example after as a separate issue, as docker introduces the concept of either using Docker-in-Docker or passing the docker socket from the physical host.
I will implement these examples in https://github.com/Awesomeware-ACME and make the documentation here reference them, as the example for GitHub Organization component already does.

So I think for this, I will implement a Python library example first, then a docker example after as a separate issue, as docker introduces the concept of either using Docker-in-Docker or passing the docker socket from the physical host.
I will implement these examples in https://github.com/Awesomeware-ACME and make the documentation here reference them, as the example for GitHub Organization component already does.

I will do the example in https://github.com/Awesomeware-ACME/pypi-module-jenkinsfile

I will do the example in https://github.com/Awesomeware-ACME/pypi-module-jenkinsfile


Issue with Grant-CsTenantDialPlan

** This seems to be the Microsoft Docs GitHub, but I can't seem to find the MicrosoftTeams PowerShell module github to log an actual issue. If this isn't the correct GitHub account, can you please point me to it.

When trying to add or remove a Tenant Dial Plan using Grant-CsTenantDialPlan to/from a resource account I'm hitting the following errors.
Other testing/notes

Tested under PowerShell v5.1 using an elevated PowerShell ISE on Windows 11 21H2 22000.258
Only applies to resource accounts (user accounts are OK - Except V2.6.1-Preview)
Tested on the following Microsoft Teams PowerShell module versions

V2.3.0 - Working OK
V2.5.1 - Error as below
V2.6.0 - Error as below
V2.6.1(Preview) - Error as below

Error for V2.5.1 & V2.6.0
When running the command Grant-CsTenantDialPlan -Identity "$UserUPN" -PolicyName $null you get the following error
Grant-CsTenantDialPlan : [ClientError] : User does not exist
At line:1 char:1
+ Grant-CsTenantDialPlan -Identity "$UserUPN" -PolicyName $null
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Grant-CsTenantDialPlan], Exception
    + FullyQualifiedErrorId : System.Exception,Microsoft.Teams.ConfigApi.Cmdlets.GrantCsTenantDialPlan

Error for V2.6.1-Preview
When running the command Grant-CsTenantDialPlan -Identity "$UserUPN" -PolicyName $null you get the following error
The term 'Grant-CsUserPolicy' is not recognized as the name of a cmdlet, function, script file, or operable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

However when running the command Get-Command -Module MicrosoftTeams, it's showing Grant-CsTenantDialPlan as there and available

@jantoney - thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome, let me know if it can be called out in the docs.
Please follow this link to contact support for business products:  https://docs.microsoft.com/office365/admin/contact-support-for-business-products.
Please keep us posted here on the resolution to feed whatever information you discover into the content.

@dariomws  Thank you very much for the contribution and sharing this explanation. @jantoney Hope this comment is helpful for you. Considering the nature of the issue, as advised, please open a service ticket in your tenant and follow with them for the resolution. We proceed to close it as here we are limited to documentation fixes and improvements only.
We will be very happy if you can share the outcome or resolution with us if you see documentation update is required. We will certainly update this documentation with that valuable information. Thanks again, for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Route [LaravelInstaller::environmentSave] not defined.

Hi, I have updated LaravelInstaller from 1.5 to 3.0 with composer.
When i go to http://localhost/site/install/ it shows installer , but when i click to Next button i receive this error :
ErrorException in UrlGenerator.php line 304:
Route [LaravelInstaller::environmentSave] not defined. (View: C:\xampp\htdocs\site\resources\views\vendor\installer\environment.blade.php)

How to fix it ?

same problem here

Remove your old resource/view/vendor/installer folder as well as config/installer.php file.
Then artisan vendor:publish to publish the required items


Meteor Todo App tests hanging with nightmare

Seems to work on another blank app with do nothing tests. But after using Meteor Todos app as a reference, some issues come up.
Server tests work fine, as expected.
Client app tests with --full-app worked once. Then on a second and subsequent runs it started hanging with no error after printing "client tests running". It's failing silently with no errors in 'debug'.
This command:
TEST_BROWSER_DRIVER=nightmare DEBUG=* meteor test --once  --driver-package dispatch:mocha

This is odd. Using Nightmare in a new blank meteor project works. Though adding and importing Nightmare to the Todos app (and other apps) causes this failure:
W20170217-14:22:16.891(-5)? (STDERR) Error: Cannot find module './actions'
W20170217-14:22:16.891(-5)? (STDERR)     at require (packages/modules-runtime.js:109:19)
W20170217-14:22:16.892(-5)? (STDERR)     at meteorInstall.node_modules.nightmare.lib.nightmare.js.options.queue.proc.stdout.pipe.on.proc.stderr.pipe.on.proc.on.instance.proc.on.instance (packages/modules.js:413:15)
W20170217-14:22:16.892(-5)? (STDERR)     at fileEvaluate (packages/modules-runtime.js:181:9)
W20170217-14:22:16.893(-5)? (STDERR)     at require (packages/modules-runtime.js:106:16)
W20170217-14:22:16.893(-5)? (STDERR)     at meteorInstall.server.testnightmare.js (server/testnightmare.js:1:17)
W20170217-14:22:16.893(-5)? (STDERR)     at fileEvaluate (packages/modules-runtime.js:181:9)
W20170217-14:22:16.894(-5)? (STDERR)     at require (packages/modules-runtime.js:106:16)
W20170217-14:22:16.895(-5)? (STDERR)     at /Users/LUCAS/Downloads/todos-master/.meteor/local/build/programs/server/app/app.js:1142:1
W20170217-14:22:16.896(-5)? (STDERR)     at /Users/LUCAS/Downloads/todos-master/.meteor/local/build/programs/server/boot.js:295:34
W20170217-14:22:16.896(-5)? (STDERR)     at Array.forEach (native)

Added here https://github.com/meteor/todos/pull/219

I'm not getting Electron[2464:70132] Cannot load Interface Builder file 'NSAlertPanel' which seems to be related to  "Too many open files" ( https://github.com/electron/electron/issues/4828 ).
The meteor bug tracking this is:
https://github.com/meteor/meteor/issues/6952

We can close, this fix worked for me: METEOR_WATCH_FORCE_POLLING=1


Observation space with FlattenWaypointEnv

I think shape should be (self.attitude_shape + self.context_length * self.target_shape,)
https://github.com/jjshoots/PyFlyt/blob/b985131be4ef69ff80d9a38fc4e7a1bf6763dbe5/PyFlyt/gym_envs/utils/flatten_waypoint_env.py#L35

Yep, you're right, would you be open to creating a PR for this? Otherwise I'll get to it later today. Thanks for the report!

The library has been fixed, installing 0.22.1 should work.


Added english tutorial for LOJ-1082 Array Queries

@rebornplusplus vaia after you approve, I will just add the Java code because there was an unpredictable verdict for the same code, check here.

@showrav-ansary you can add a PR following this one, if it gets merged. Also, I can't access your LightOJ submission. I get 500.


has no exported member 'MatTooltipModule'

Bug, feature request, or proposal:
error in documentation for the "tooltip" at:
https://material.angular.io/components/tooltip/api
What is the expected behavior?
import {MdTooltipModule} from '@angular/material';
What is the current behavior?
import {MatTooltipModule} from '@angular/material';

For this and your other issue (#7415), see https://github.com/angular/material2/issues/7313.
For beta.11,

all components have been exported currently as Both Md* and Mat*. However ultimately Mat* should be used.

thank you @willshowell for the comment
currently I am using the following dependencies yet my app did not compile with "Mat", so I thought it is a typo in documentation.
"dependencies": {
"@angular/animations": "^4.4.2",
"@angular/cdk": "^2.0.0-beta.10",
"@angular/common": "^4.2.4",
"@angular/compiler": "^4.2.4",
"@angular/core": "^4.2.4",
"@angular/forms": "^4.2.4",
"@angular/http": "^4.2.4",
"@angular/material": "^2.0.0-beta.10",
"@angular/platform-browser": "^4.2.4",
"@angular/platform-browser-dynamic": "^4.2.4",
"@angular/router": "^4.2.4",
"@ngx-translate/core": "^8.0.0",
"@ngx-translate/http-loader": "^2.0.0",
"core-js": "^2.4.1",
"hammerjs": "^2.0.8",
"rxjs": "^5.4.2",
"zone.js": "^0.8.14"
},

Yeah the docs only represent the latest version, which is 2.0.0-beta.11

See CHANGELOG for more information

@willshowell lol that is fast, thanks man. I will upgrade to 11

This is working as expected after the latest (beta.11) release


Request new module: cp-cp4ba-operator

Related user story
Provide the issue number for the user story related to this request #597
Type of module
gitops
If requesting a terraform module, what is the target cloud platform?
ibm
What is the purpose of the module?
Provision CP4BA operator
What is the category of the module?
business automation, integration

Please add @dineshchandrapandey as collaborator

Please add @dineshchandrapandey,  @saratrip , @dileeppaul   as collaborators

/displayName CP Business Automation operator

/softwareProvider ibm-cp


target size being ignored?

Or what is the proper way to use target size?

I honestly don't know.  That value is passed straight through to the Squoosh WebP WASM library.  I copied the list of available parameters from this document: https://developers.google.com/speed/webp/docs/api#advanced_encoding_api
I suspect not all of them are implemented in the WASM version 😞
I'm sorry.


Rename fails with "Bareword ... not allowed ..."

Was attempting to use the rename command and I'm getting the following.
Bareword "USA" not allowed while "strict subs" in use at (user-supplied code) line 3.

I'm not sure what is different, but it appears to be working now.


trans to .net core

See also: https://www.nuget.org/packages/NKnife.NLog.WPF

See also: https://github.com/dojo90/NLogViewer


Carla transformation matrix from roll,pitch,yaw

The actor.get_transform().rotation has roll,pitch and yaw values.
To get a rotation matrix, I need to understand the order in which these transforms were applied.
Is it roll,pitch,yaw or something else?
If i am correct, I can get transformation via-
trans=actor.get_transform().trans
rot=actor.get_transform().rotation
t=trans.x,trans.y,trans.z
r,p,y=rot.roll,rot.pitch,rot.yaw

trans_mat=np.eye(4)
trans_mat[:3,-1]=t
##Similary roll,pitch,yaw matrices
transformation_matrix= trans_mat@roll_mat@pitch_mat@yaw_mat #3x4 matrix

Could someone confirm?

I had the same problem and studied Carla's source code and then did some testing.
Studying the source code I found the formula Carla is using for the rotation matrix. Then I played around with sympy multipyling the yaw, pitch, and roll matrices in different order. What I found is that Carla performs roll, then negative pitch, then negative yaw. I verified this empirically.
Values for Carla
**

**
Values with sympy (please watch the minus signs!)


💡 [FEATURE]  contact section

Idea Contribution

[X] I have read all the feature request issues.
[X] I'm interested in working on this issue
[X] I'm part of GSSOC organization

Explain feature request
contact section is not created
i would like to work on contact section if you assign this to me under gssoc
Explain your solution
No response
Any alternative approaches/features
No response
Additional Context
No response

Hi, currently we're planning the to redesign the website so for now we are not entertaining more issues about the website since we're going to redesign it sooner. Maybe you can work on other issues, thanks


Update license headers and authors list

@GuillaumeGomez this one has slipped through somehow =)

Haha. And with this PR, we end the license headers and authors stuf. Very nice job !

:+1:


issue #11458:Triple stub support async mode

What is the purpose of the change
fix #11458

Codecov Report

Merging #11464 (b372a8a) into 3.1 (34b92b0) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                3.1   #11464      +/-   ##
============================================
- Coverage     64.67%   64.66%   -0.02%
  Complexity       14       14
============================================
  Files          1431     1431
  Lines         59969    59969
  Branches       8807     8807
============================================
- Hits          38786    38776      -10
- Misses        17111    17117       +6
- Partials       4072     4076       +4

Impacted Files
Coverage Δ

...e/dubbo/rpc/protocol/tri/transport/WriteQueue.java
64.28% <0.00%> (-9.53%)
:arrow_down:

...e/dubbo/remoting/transport/netty/NettyChannel.java
51.13% <0.00%> (-7.96%)
:arrow_down:

.../apache/dubbo/remoting/transport/AbstractPeer.java
67.39% <0.00%> (-4.35%)
:arrow_down:

.../dubbo/remoting/transport/netty4/NettyChannel.java
71.69% <0.00%> (-3.78%)
:arrow_down:

.../apache/dubbo/rpc/protocol/dubbo/DubboInvoker.java
67.79% <0.00%> (-3.39%)
:arrow_down:

.../src/main/java/org/apache/dubbo/rpc/RpcStatus.java
80.95% <0.00%> (-1.20%)
:arrow_down:

...he/dubbo/registry/zookeeper/ZookeeperRegistry.java
60.52% <0.00%> (-1.06%)
:arrow_down:

...java/org/apache/dubbo/remoting/api/Connection.java
78.33% <0.00%> (-0.84%)
:arrow_down:

.../dubbo/registry/integration/RegistryDirectory.java
45.58% <0.00%> (-0.71%)
:arrow_down:

...n/java/org/apache/dubbo/common/utils/NetUtils.java
62.67% <0.00%> (+0.27%)
:arrow_up:

... and 7 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


🔧 [Refactor] Admin/Database query performance optimization

🔧 Refactor ticket

The admin interface take a very long time to filter the users in the user search interface. This needs to be optimized since it affects the whole service.
Suggestion: Investigate if database indexes can mitigate the problem and define indexes for the database - not only on the user table, but all other(already refactored) tables aswell.

reopening this, since its still an issue


Plugin installation from sonobuoy-plugins repo

When doing plugins list/show/install it would be ideal to integration with the sonobuoy-plugins repo.
This would give plugins a method of discovery other than navigating to github manually and trying to figure out what plugins are available.
Instead, they could do things like
sonobuoy plugins list --registry
<get a list of plugins they could install from there>

sonobuoy plugins show foo --registry
<see the full details of the plugin before installation>

sonobuoy plugins install foo myfoo
<install>

sonobuoy plugins run -p myfoo

Also will recognize/coordinate with some organization of that repo. For example, allowing viewing/installing of all plugins but making it clear which ones are examples, community driven, vmware maintained, etc.

This has fallen off in priority; a nice to have right now but not the focus of my effort. If someone else wants to help out that'd be great.


[17.07] vndr libnetwork to bring in lock fix

vndr libnetwork to bring in fix:

docker/libnetwork/pull/1913 [backport 17.07] Overlay fix for IP reuse

comparison of changes from the libnetwork repo: https://github.com/docker/libnetwork/compare/230db20...4a242db

Known test failures:
$ ./check_failures.sh https://jenkins.dockerproject.org/job/docker-ce-pr/126/consoleText
FOR: https://jenkins.dockerproject.org/job/docker-ce-pr/126/consoleText
Passed (thought would fail):

Failed (should fail)
DockerSuite.TestRmiContainerImageNotFound
DockerSuite.TestRmiImageIDForceWithRunningContainersAndMultipleTags
DockerSuite.TestRunAttachDetachFromInvalidFlag
DockerSuite.TestRunWithNanoCPUs
DockerSuite.TestUpdateWithNanoCPUs
DockerSwarmSuite.TestServiceLogs
DockerSwarmSuite.TestSwarmNetworkPluginV2
DockerSwarmSuite.TestSwarmServicePsMultipleServiceIDs


Less strict integration tests

#skip-changelog
:scroll: Description

Only check if expected files were uploaded, without checking the count.
:bulb: Motivation and Context

:green_heart: How did you test it?
:pencil: Checklist

[x] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated the docs if needed
[ ] All tests passing
[x] No breaking changes

@vaind WDYT, is this enough for this kind of test? Same as before, but only checking if the file was set at least once.


Fixed navbar to add HOME tag

Description
A clear and concise description of what the PR does.

This PR does the following:

Updates

Related Issues

"Fixes #337 "

Screenshorts:
separate option for add listing

Changes
List the detailed changes made in this PR.

Fixed a bug in navbar. its check if user is admin or not if its admin only then its shows add listing option otherwise not.

Checklist
Make sure to check off all the items before submitting. Mark with [x] if done.

[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] I am working on this issue under GSSOC

@Soujanya2004 Requested to review PR..

@Meetpidev plz add necessary screenshots
other contributor is working on the bug actually

@Soujanya2004 Requested to review again...

@Soujanya2004 Please Review PR

@Meetpidev anyone can add their property once logged in

@Meetpidev anyone can add their property once logged in

But that is wrong.. we have to authorized feature in website.. we must defined that which type of user access which feature..

@Meetpidev anyone can add their property once logged in

If so then there there is no meaning of type of admin..

admin feature is to just modify any listing though he/she is not the owner of the property

admin feature is to just modify any listing though he/she is not the owner of the property

@Soujanya2004 okay i got you..what you are trying to say... but if someone want to add property then he/she must me first sigup as admin only then they can add... this is the better way.. by it we can authorized the features...

admin feature is to just modify any listing though he/she is not the owner of the property

@Soujanya2004 okay i got you..what you are trying to say... but if someone want to add property then he/she must me first sigup as admin only then they can add... this is the better way.. by it we can authorized the features...

no, anyone who logins will hv ability to add their property, they dont have to be admin, admin role is to maintain ethics in the website, just in case anyone tries to misuse website thats it

@Soujanya2004 Done.. i removed the admin condition... and added separate option for add properties... as i shown in screenshort...

@Meetpidev airbnb ur home and add ur listing does the same thing ryt

@Meetpidev airbnb ur home and add ur listing does the same thing ryt

@Soujanya2004  Its not same thing.. "airbnb ur home" and addlisting.. its not same..

@Meetpidev that is what i was trying to tell when you opened the issue

@Meetpidev airbnb ur home and add ur listing does the same thing ryt

@Soujanya2004 Its not same thing.. "airbnb ur home" and addlisting.. its not same..

how is it different can u explain

@Soujanya2004 but there should be separate option for to redirect to home and for add listing

@Meetpidev for redirecting to homepage there is 'explore' link and to add new listing there is 'airbnb ur home' link

@Soujanya2004 As i mentioned before thatm  that plane and ecplore is consider as a logo... when new user will come to it he think that its an logo.. they will find home tag to redirect homepage..

@Meetpidev fine?
you can just change the airbnb ur home button color to color used when its hovered
i will merge it

@Soujanya2004 As i mentioned before thatm that plane and ecplore is consider as a logo... when new user will come to it he think that its an logo.. they will find home tag to redirect homepage..

fine change explore to explore listing?

@Soujanya2004 i dont understand.. when we hover on that btn then btn must be change its backgroundColor.. right ??

@Meetpidev u will notice it on hovering the bg color changes to certain color, i want that color for the button, on hovering u can add box shadow or something

@Soujanya2004 As i mentioned before thatm that plane and ecplore is consider as a logo... when new user will come to it he think that its an logo.. they will find home tag to redirect homepage..

fine change explore to explore listing?

Its not good to change explore -> explore listing..  Because length of the string/character increased.. doesn fir on navbar and also doesnt suitable...

@Soujanya2004 As i mentioned before thatm that plane and ecplore is consider as a logo... when new user will come to it he think that its an logo.. they will find home tag to redirect homepage..

fine change explore to explore listing?

Its not good to change explore -> explore listing.. Because length of the string/character increased.. doesn fir on navbar and also doesnt suitable...

even i thought the same now

@Soujanya2004
I hope you understand why we need separate option for to redirect to home and for add listing..

@Soujanya2004 nono.. dont change...  i just changed the "airbnp ur home" -> Home , Add property -> add property form..
everything remains same..

just add clip of the demonstration

@Soujanya2004
Requested TO Review PR..
https://github.com/user-attachments/assets/2f3fdb3f-9898-4888-b070-23a27aa049f9

@Meetpidev as i said before can you remove add ur property link and 'airbnb ur home' , just replace 'Airbnb ur home' to Add your property'(functionality remains like original) ok, i will give you level2 no worries
rest all is fine

@Meetpidev i dont want you to change button redirecting, let it be same like first
instead of airbnb ur home change text to add ur property
i hope u get it

@Soujanya2004 ITs done .. Please Review Again...

@Soujanya2004 Requested to review again.. i changed code according to your requirements....

@Soujanya2004 When i was working on it, i faced issue that you wrote the code in all the css randomly...
i mean to say that if in html code if first tag is nav and othe classes you give that tag then all the style of it should be write first.. you should not do like .nav style write 1st then other styling and then again .nav styling...

@Soujanya2004
In style.css .. the footer style is always written in last as footer is placed last... and all the media quary should be written together....
there is very inconsistent style you wrote.. for the developer its hard to find that where the style of perticular component written ...
if you write code tag by tag like first header,nav,middle-section,footer,keyframes,mediaquary.. then its easy to find the style...

Uploading Recording 2024-11-03 122154.mp4…

https://github.com/user-attachments/assets/f5c78335-e8cd-41de-b0b2-2a01680354a4

@Soujanya2004 Can i add icons for login signup logout ?? as of now there is no icons displayed... like login (icon), signUp (icon) etc...

@Soujanya2004 When i was working on it, i faced issue that you wrote the code in style.css randomly... i mean to say that if in html code if first tag is nav and othe classes you give that tag then all the style of it should be write first.. you should not do like .nav style write 1st then other styling and then again .nav styling...

yeh i had arranges it well but all the contributors have changed the order

@Soujanya2004 Can i add icons for login signup logout and yourprofile ?? as of now there is no icons displayed as you can see in the video ... like login (icon), signUp (icon) etc...

sure sure you can raise separate issue i will assign u

since there were many changes and the contributor has made the requested changes many times i have labeled level2

@Meetpidev you can just run init->index.js file to load listings, since its empty now


Discussion on general level of PEP8 compliance.

Branched from #452 which is a more specific question at the suggestion @javdrher
I've personally always felt that extreme adherence to a coding standard could slow down development. I'm pretty happy with the compromise level we've existed at since inception which really has come from a common sense agreement between the main developers.
I recognise as the GPflow community grows and branches we need to keep these things under review.
Despite the fact the GPflow community is growing I still think there is a danger of distracting effort and attention from more importance questions about code content.
I don't think we should block PRs unless they have drastically the wrong coding style. There are after all automated tools we could periodically run if people wish.
The specific case of variable names mentioned by @javdrher in #452 is a good example. I'm totally happy for variable names to be chosen to match papers if that it was the contributors feel is most sensible.
To summarize, I think coding standards such as PEP8 can be helpful in a project of our size, but extreme adherence for its own sake it literally style over substance.
Thoughts?

Hello everyone!
The code style guides exist to simplify maintenance, get better readability, hence make your life easier as contributor and whole community as well. Different tools can relief development process, something like autopep8, pep8fy, YAPF, plugins to editors: python-pylint, python-tools and etc.
@javdrher gave extremely good example, but various authors can have divergent preferences for variable names, e.g. Xnew, xNew,  x_New, such flexibility makes code look like a mess. All of these name versions can be adapted as either xnew or x_new without loss of meaning. Most important part here is a context - why this variable was called in this way, if it is a replication of paper codestyle, where is a paper link and short description what it means. Here is an example from kernels.py:
    @AutoFlow((float_type, [None, None]))
    def compute_K_symm(self, X):
        return self.K(X)

    @AutoFlow((float_type, [None, None]), (float_type, [None, None]), (float_type,))
    def compute_eKxz(self, Z, Xmu, Xcov):
        return self.eKxz(Z, Xmu, Xcov)

    @AutoFlow((float_type, [None, None]), (float_type, [None, None]), (float_type, [None, None, None, None]))
    def compute_exKxz(self, Z, Xmu, Xcov):
        return self.exKxz(Z, Xmu, Xcov)

First of all, I think names are good, sometimes it does makes sense to ignore rules and write concise code (using abbreviations or whatever). But here it lacks a description of what's happening within compute_exKxz, compute_eKxz, compute_exKxz, and why autoflow has such parameters.
In any case, pylint has python directives, in fact it is just comments, with special labels for neglecting warning/error/info messages and it also has configuration file, where you can relax some rules and say that camelcase style is acceptable by GPflow. For example increasing line length from 80 to 100 is totally okay for me. If someone is going to use non-standard names, he/she could use directives to silence messages.

I think, it is important topic to discuss, so others are welcome to share their opinion.
@jameshensman, @icouckuy, @javdrher, @Bonnevie, and many more :)

I'm very much for the increasing the line length from 80 to 100. We don't all work on CGA terminals any more.

@awav @markvdw  can discuss this over lunch on Monday as well. Some more thoughts:
I think the decision to use an automated code tool as part of the build process would need to be taken very carefully. Of course, no automated code tool is perfect....
From a machine learning perspective any automated code tool is a probabilistic decision process, governed by probabilistic decision theory. To fully specify the problem we will need a loss and the loss that different stake holders in the code would use is not necessarily the same. For instance some people will care deeply that we have Xnew, xNew, x_New, in the code and some people will not care at all. Any decision that is made by the community will need to attempt to consider the range of views on this. We could also think about it in terms of developer time. Is the amount of developer time saved by using a given tool higher or lower? This will depend on the quality of the tool (how many false positives and negatives for a given user's definition of useful) and how well it is used.
For a more practical perspective, I've seen much vaunted automated code tools badly implemented before to the extent of drastically slowing down development. This has taught me to exercise a healthy scepticism on such things in general and where used insist they are implemented well,  before they become an established part of the workflow.
Final thought: I think it is important that whatever conclusion we come to we don't alienate potential contributors from outside the core group by adding unnecessarily to the barriers to entry. It is not a saving in developer time if we put off a large proportion of our potential developers. GPflow has always tried to be a very open community and the project has greatly benefited from this.
I'd encourage everyone who has an opinion on this to express it :)

I love your Bayesian analysis of the problem :) I'm not familiar with any automated code tools for PEP8. I use pycharm which has continuous checking of PEP8 violations which I try to adhere to as much  as possible.
In the end it depends on the goals of GPflow. First I thought it was best to comply with the paper notation as Matthews mentioned which makes it easier for newcomers to cross-check an approach with the corresponding paper and try to understand it. In short, keep only the code tying everything together in PEP8 and the algorithms keep the notation of the paper, most notably the predictive distributions and likelihood lowerbounds.
However, as far as I know the goal of GPflow was not to be a learning tool (in contrast to GPy). In addition, papers of extension to the same methodologies might even have significant notational differences. This leads me more towards trying to adhere to the PEP8 standard.
I do not care that much for it that I will fight over it though. In my opinion it is crucial to keep the contributing guidelines (contributing.md) as short as possible so newcomers can easily create new PRs. Code cleanup can then happen in the PR using common sense and/or a pep8 tool, as is done now.

I believe the analysis of @alexggmatthews is going to be confirmed empirically by the diversity in responses in this discussion. My opinion divert from Ivo's. I'm also using pycharms PEP8 and I fully agree most of the directives are plain common sense in terms of code readability and maintainability. Actually, my first programming steps were in PHP4: after one week of browsing examples online I understood the message how important some fundamental style requirements like spaces after commas and surrounding operators are. No discussion there.
For the naming aspect, I consider PEP8 useful recommendations but to be ignored when appropriate. In terms of package naming, classes and methods I can still follow to some level, but I think the constraints on variable naming are a very bad idea for two reasons:

even without caps, there are plenty of PEP8 legal ways to make naming a complete mess which means the need for common sense isn't ruled out.
sensible variable naming affects code readability a lot. Developers mastering the art of condensing the semantics of variables in a short name are exceptional (I'm horrible at it) but reading and maintaining their code is a lot easier. Therefor if a name is better of as a violation it should be. As a consequence I think reflecting common symbols in papers (this is of course not universal but X/Y/F/Z/q(.)/K ring bells) isn't a matter of being a learning tool. It is about code quality and readability.

A bit more practical: I know we can configure the policies of linters, or place pylint directives in the code to suppress linter warnings, but on the other hand I'm convinced contributors are not massively violating PEP8 up to the point it becomes confusing/a problem. Therefor I'm in doubt integrating automated quality checks will actually help the project (in its current state) forward but I'm open to be proven otherwise. I fully agree with Matthews though that there is a real threat of being worse of with slower development and a harder level to contribute. Occasionally (e.g., before a release), running a linter and resolving style problems (e.g. whitespaces) which have been overlooked like @awav did could be a middle ground?

Thank you for your contributions everyone. I sat down with @awav @marvdw and @jameshensman over lunch today to make a plan for this. We took into account the helpful contributions on this forum from those who aren't in the UK so thank you for taking the time to contribute.
The plan is:

To create, in consultation with the community, a style file for pylint.
To run pylint with this style file on pull requests. To start with this will only be as `advice' for reviewers and will not be a requirement to merge. Reviewers will still use their discretion on style which is what happens at the moment.
Periodically we will from time to time do PRs which solely cover style. To start with these will be reviewed like any other PR though it is likely we will relax this.
If after a trial period of time we feel that the continuous integration advice has been useful and reliable we will, in consultation with the GPflow community, make it a requirement of merging. This would save the need to do 3) . If it turns out that pylint can never be made reliable enough to be fully automatic we won't make it a requirement and will keep it as an advisory diagnostic.


Copy & pasting nodes into a child scene with "Editable Children" does not work correctly

Godot version
3.5.stable
System information
Linux. OpenGL ES 3.0. AMD Radeon RX 6700 XT
Issue description
I have a child scene with "Editable Children" and I can create new nodes within the child scene via the parent scene.
What does not work is pasting a node into the expanded child scene.
What looks to happen is that it thinks the pasted node is actually inherited from the child scene and doesn't actually create the copied node into the parent scene. I think this because it appears grey like the other inherited nodes. It also does not save. If you close and reopen the parent scene the pasted node is lost. It is also lost if you run the project.
Steps to reproduce
Copy the Light2D

Paste it into one of the inherited nodes

Minimal reproduction project
EditableChildrenCopyPasteMRP.zip

Same bug in Godot 4.0.2
(v4.0.2.stable.mono.official [7a0977ce2] )


about the #191

here is a possible fix for the problem that happens when you try a reinsert twice the same file.

Relates to #191 and #143

Thanks for the PR, can you fix your indentation for PHPCS please. Then I can test this.

hi, can't find the problem. It seems indentation is ok. May you give some help in finding the phpcs problem?

Hey @maxmasetti did you check the build on Travis?
https://travis-ci.org/josegonzalez/cakephp-upload/jobs/31120228#L71

hi,
yes I did:
FOUND 1 ERROR(S) AFFECTING 1 LINE(S)
but reading code line by line no space used for indenting found at all.
max

Il giorno 04/feb/2015, alle ore 15:45, David Yell<EMAIL_ADDRESS>ha scritto:
Hey @maxmasetti did you check the build on Travis?
https://travis-ci.org/josegonzalez/cakephp-upload/jobs/31120228#L71
—
Reply to this email directly or view it on GitHub.


Recorder does not capture POST request as submitted.

Expected behavior
When using the recorder and the localhost proxy to capture normal trafic flow, the POST request recorded and passed on should be unaltered:
-----------------------------41691790316285946593014481764
Content-Disposition: form-data; name="imagefile"; filename=""
Content-Type: application/octet-stream

-----------------------------41691790316285946593014481764

Actual behavior
An empty input field for a submission, in this case a file upload, is trimmed somewhat:
--awqbkPPRssLNd_sGDI4M8sD3AIl-MeCoVph
Content-Disposition: form-data; name="imagefile"

--awqbkPPRssLNd_sGDI4M8sD3AIl-MeCoVph

Which is interpreted as a normal parameter of content-type application/octet-stream in the resulting HTTP request generated by the recording:

Steps to reproduce the problem
Add a file input to a submision form, but submit withouth selecting a file:
<form action="submit.php" method="post" id="submitForm" enctype="multipart/form-data">
<input type="file" id="imagefile" name="imagefile" oninput="formUpdated();">
<input type="submit" value="Send to maintainer">
</form>

JMeter Version
5.6
Java Version
Java(TM) SE Runtime Environment (build 16.0.1+9-24)
OS Version
Microsoft Windows [Version 10.0.22621.1848]md

Could you please clarify if the behaviour is the same for 5.5 and 5.6 for you?

Upon revisiting, I have found that the bug was slightly different than I first interpreted.
I've Eddited/updated my initial post, but yes, this is also present in 5.5.

We are about to release 5.6.1, so if you could suggest a reproducer, we could probably integrate a fix as well if the case is not hard to resolve

The best reproducer i can think of (can't test int instantly):
Have a webpage/webform with the following HTML:

That should generate a request with something in the (Raw) form of:
-----------------------------41691790316285946593014481764
Content-Disposition: form-data; name="imagefile"; filename=""
Content-Type: application/octet-stream

-----------------------------41691790316285946593014481764

I'm not sure how this should be parsed, JMeter currently seems incapable of handling a file upload with an empty file path, and the issue here is that a POST with an input of an empty file path is exactly what isn't being replicated exactly.
(manually adding an entry to the HTTP Request's file upload with an empty File Path , Parameter Name "imagefile" and MIME Type "application/octet-stream" does not seem to generate an actual POST request upon running/validating)

JMeter uses filename parameter as both source for the data, and, well, filename it sends in the request header.
It might be nice to decouple the two, however, it is not that trivial as it would involve UI, documentation, and tests.
It might be a nice idea to capture file contents in proxy (e.g. save it side by side with jmx, or store it in the jmx as base64)
Pull requests are welcome.

However, if the only case here is sending empty file upload, then we could probably go for using blank filename as a sign of "file not selected"


php 8.4: Implicitly marking parameter $params as nullable is deprecated, the explicit nullable type must be used instead

When running the tests of ci3 with php8.4, I get these deprecation warnings:
PHP Deprecated:  CI_Encryption::encrypt(): Implicitly marking parameter $params as nullable is deprecated, the explicit nullable type must be used instead in .../system/libraries/Encryption.php on line 372
PHP Deprecated:  CI_Encryption::decrypt(): Implicitly marking parameter $params as nullable is deprecated, the explicit nullable type must be used instead in .../system/libraries/Encryption.php on line 507
.
Deprecated: CI_Encryption::encrypt(): Implicitly marking parameter $params as nullable is deprecated, the explicit nullable type must be used instead in .../system/libraries/Encryption.php on line 372

Deprecated: CI_Encryption::decrypt(): Implicitly marking parameter $params as nullable is deprecated, the explicit nullable type must be used instead in .../system/libraries/Encryption.php on line 507

Hello, I noticed that you are using PHP8.4.
Then you should often use different versions of PHP.
You can try using the Servbay tool, which supports PHP8.4beta, so you don’t have to worry about environmental pollution.
You can switch at any time


How to fix? Latest version... 15  ValueError: time data '2023-11-04T13:20:54.534Z' does not match format '%Y-%m-%dT%H:%M:%SZ'

How to fix? Latest version... 15
ValueError: time data '2023-11-04T13:20:54.534Z' does not match format '%Y-%m-%dT%H:%M:%SZ'

Where did you even get this. I removed the option to download v15, because it's dead for over half a year now.

Closing as yet another duplicate.


Add bin/spring, and update bin/bundle binstubs

I figured out why my tests weren't using Spring. Looks like it didn't work because the bin/spring command was missing. Then when I bundle binstubbed that, Spring complained that the bundle binstub itself was out of date and needed replacing.
@camallen I notice that you explicitly didn't add the bin/spring file in 8bc9b1d6. Can you still remember why not?
Review checklist

[ ] First, the most important one: is this PR small enough that you can actually review it? Feel free to just reject a branch if the changes are hard to review due to the length of the diff.
[ ] If there are any migrations, will they the previous version of the app work correctly after they've been run (e.g. the don't remove columns still known about by ActiveRecord).
[ ] If anything changed with regards to the public API, are those changes also documented in the apiary.apib file?
[ ] Are all the changes covered by tests? Think about any possible edge cases that might be left untested.

I think I was the only one using it at the time so i left the binstub out, to allow folks to opt in.
Though it looks like i checked in the modified binstubs for the other bin cmds in https://github.com/zooniverse/Panoptes/pull/1507/commits/cb83c57e4d3e41f26a1a25d0fc6dc5dbb461bbe2
If you haven't already, fancy running bundle exec spring binstub --all again to make sure this is all up to date?

Yeah I did that but the rest were up to date.
On Thu, Jul 12, 2018 at 2:54 PM, Campbell Allen<EMAIL_ADDRESS>wrote:

@camallen approved this pull request.
LGTM - be great to make sure all the bin stubs are up to date as well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zooniverse/Panoptes/pull/2839#pullrequestreview-136660901,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAABFVe8VdURf0kv8rT4W7nMvPegdCvhks5uF1UhgaJpZM4VM3cN
.

--

Marten


add directory on container because it cannot be created on runtime

It is still failing:
2021-09-10T15:12:47Z - ERROR    - MainProcess - trigger_scale: E: Failed to save config file: Failed to write file '/.config/ocm/ocm.json': open /.config/ocm/ocm.json: permission denied

Codecov Report

Merging #348 (313ff84) into master (01e6199) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #348   +/-   ##
=======================================
  Coverage   39.40%   39.40%
=======================================
  Files          15       15
  Lines         868      868
=======================================
  Hits          342      342
  Misses        526      526

Flag
Coverage Δ

gha
39.40% <ø> (ø)

python-3.6
39.40% <ø> (ø)

unit
39.40% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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 01e6199...313ff84. Read the comment docs.

Results for SNAFU CI Test

Test
Result
Runtime

Report\n>
FAIL
00:00:00


Separate setting CRUD management betwwen "basic setting" and "advance setting"

Is your feature request related to a problem? Please describe.
Need simplecity setting and don't confuse new user
Describe the solution you'd like
Just show basic setting first, and hide advance setting (model & controller override, etc) on a expansion panel
Describe alternatives you've considered
We can use expansion panel or anything for hide expansion panel, just choose the better one

Suggestion: this should be the basic settings while the others are advanced settings

This also should be all checked by default


Is this project still developed?

Is this project still developed?
On the main page, it is written:

Supported platforms
.NET Standard 1.4
Xamarin Android for API 19 through 24 (KitKat through Nougat)
Xamarin iOS for iOS versions 8.0 through 10.0
Xamarin.Forms (Android, iOS and UWP)
Universal Windows Platform
Other versions may work. We do not test them and thus cannot support them.

So what about newer versions of Android and iOS? What is the future of this project? I want to implement an offline sync functionality for my Xamarin.Forms app but I can't find any reasonable solutions.

Hi @Mikilll94,

So what about newer versions of Android and iOS?

They should work, please file an issue if you're facing with the issues.

What is the future of this project?

The answer is right in the readme:

Future of Azure Mobile Apps
Microsoft is committed to fully supporting Azure Mobile Apps, including support for the latest OS release, bug fixes, documentation improvements, and community PR reviews. Please note that the product team is not currently investing in any new feature work for Azure Mobile Apps. We highly appreciate community contributions to all areas of Azure Mobile Apps.

I'm closing this issue, but feel free to reopen/reply if you have more questions.

@MatkovIvan
Ok. So if newer versions of Android and iOS are working then you should update README file.
Because when someone reads:

Supported platforms
.NET Standard 1.4
Xamarin Android for API 19 through 24 (KitKat through Nougat)
Xamarin iOS for iOS versions 8.0 through 10.0
Xamarin.Forms (Android, iOS and UWP)
Universal Windows Platform
Other versions may work. We do not test them and thus cannot support them.

then they think that this project is no longer supported.


#wizard - add welcome message to wizard module

add welcome message to the wizard module.

New commit has been fix this issue.
if you find any bugs pls make new issue
Regards,
~Ryu


Update 2023.09.07-current_status.md

Updates to include suggested changes.

updated to include suggested changes.

This looks great, merging. Thanks!


borderRadius style prop

Hello!
I have an issue with the borderRadius prop. The svg style prop is not using it.
ex:
<Image
style={{borderRadius: 70}}
source={getCountryFlagIcon(this.props.country.toLocaleLowerCase())}
/>
It works when I apply the borderRadius to a parent View but it's having a weird behavior on android. On android, it's rendering the svg with no borderRadius for a second and then apply the style of the parent view. It's the same when I click on the svg, the style of the parent view disappear for a second.
Everything works fine on ios.
Any idea about that?
Thank you

Can you also try overflow: hidden  along with borderRadius, so that the WebView may get properly hidden? borderRadius sometimes used to just draw the radius but allowed the inner content to flow out, so maybe this would help.

Thank you for you answer, I will try it. But since that, I built it and tried with an android emulator, the svg are not appearing at all :-/


notify observer to complete async stream

notify observer to complete async stream.

If convert IObservable{T} to IAsyncEnumerable and use await foreach to enumerate stream, but observers never call OnCompleted, the thread will be permanently blocked.

see also #5006 .

Very cool, thanks for the contribution :)
Can you add one test case in the integration tests to test this behaviour?
You can add a test case to
https://github.com/ChilliCream/hotchocolate/blob/main/src/StrawberryShake/CodeGeneration/test/CodeGeneration.CSharp.Tests/Integration/TestGeneration.cs
that will generate the client.
And then just type out a test case like
https://github.com/ChilliCream/hotchocolate/blob/main/src/StrawberryShake/CodeGeneration/test/CodeGeneration.CSharp.Tests/Integration/StarWarsGetFriendsTest.cs

No problem, but I need some time.

In addition, this pull request can only let the client detect the subscription that ends normally. If the socket is closed due to server crash or network error, the client cannot receive the message of completing the subscription, so this notification will not be triggered. If you want to complete the subscription in this case, you may need to modify the interface definition. Because my interim solution needs to access non-public members through reflection to trigger the notification of subscription completion.

@PascalSenn I will patch this into 12.9 so let's keep it open for now... I will merge it on the weekend.

OK ... we now have a green main ... so lets see if this build is green.

/AzurePipelines run

/AzurePipelines run

/AzurePipelines run


Convert Reason record to TypeScript interface with the first letter being uppercased

This is a duplicate of https://github.com/cristianoc/genTypeScript/issues/4, but I guess since genFlow is the future I post it here again:
In TypeScript it's common that the first character of types and interfaces is upper-case.
Reason
type person = { name: string };

Ideally would be converted to
interface Person { name: string };

Why Interfaces? They offer more capabilities compared to a type declaration e.g. extending them.

In terms of trade-offs, those are the positives.
The negatives include:

need to remember to capitalize/uncapitalize the same type when accessed from Reason vs JS.
not every type can be made an interface. E.g. type t = int, so the price to pay is a bit of inconsistency.

Overall, I'm not sure.

Totally can see the capitalize/uncapitalize overhead, but on the other hand for any TypeScript user anything uncapitalize looks surprising.
Yeah the fact that you can represent an object as type or interface is one of the many things from my pov that make TypeScript not straight forward to use in the day to day.
My assumption is: If genType doesn't generate idiomatic TypeScript, it will be hard to convince TypeScript teams to adopt Reason.

Idiomatic is definitely high up on the priority list.
I'd just like to explore things a bit further first.
Interfaces can be extended. When an extended value is passed into Reason and later gets back into JS, it will change representation into Reason records and back to JS objects. This means, that any extra fields will get lost. This could be surprising as typically in JS one would pass the whole object, while crossing the JS/Reason interface means the value is copied.

One idea would be to only allow to export an interface from Js.t objects, as those don't need to be converted on the boundaries. The downside is that it'd require for the Reason author to explicitly annotate the functions where these types are used, but I think it could be worth it as objects seem to be closer in nature to Flow / TS interfaces than records.
Another (non mutually exclusive) idea is to make the interface generation opt-in with some special annotation like<EMAIL_ADDRESS>

See this test https://github.com/cristianoc/genType/pull/81 about how exporting types as interface would look like.
There are a few questions on how this could be deployed, including whether it should be per-project or even per-instance with a dedicated annotation<EMAIL_ADDRESS>In the test, the types are also capitalized, adding a "I" to the name. This is just one possible way. The "I" is a recommendation coming form the TS linter.
The test also applies to Flow, where one observation is that where interfaces are used, one loses the opportunity to emit exact object types {| ... |}.

Another observation is that if a type is capitalized when exported as an interface (and perhaps "I"-prefixed), then this becomes visible from JS. It means if a type is changed from object to something simpler, its name changes too.