logging: sketch of an updated package

I'm planning on rewriting the existing Go client for the Stackdriver logging API. In addition to using the latest API version (v2beta1), I'd like to build the implementation on top of the Google API Code Generator library and expose more of the API's functionality.
The current Go client exports a Client type that includes connection and project information, as well as the log name. I'd like to bring the client more in line with the other languages by having a Client type responsible for networking and auth, and a separate Logger that holds the log name and other fields, like resource and common labels.
(This change alone requires that we wrap the Code Generator library instead of using it directly or augmenting it, because I don't think a tool could come up with the Client/Logger split from the API alone.)
The Go client logs asynchronously. Calls to the main Log method add log entries to a queue and return immediately. I'd like to keep that behavior, but redo the implementation on top of the generated code, exploiting gax's support for retries. I'd also like to find a way to surface errors from the API calls; they are currently swallowed.
The Go client will continue to provide a way to get a standard log.Logger. I'll remove support to get an io.Writer, since that interface is too low-level. We'll tackle how best to support 3rd-party logging on a case-by-case basis. For example, logrus, a popular package, supports a hook interface that takes a log entry (not an io.Writer). We can provide a logrus hook in another package.
I don't have much to say about the retrieval part of the API. I don't think having a nice veneer for it is as important as for the logging itself. At this point, I have no plans to do anything more than directly wrap the generated implementation.
Comments appreciated.

Stackdriver has a logging API too?

by having a Client class responsible for networking and auth

Btw, we don't use the word "class" in Go. There are "types".

exploiting gax's support for retries

What is gax?

This change alone requires that we wrap the Code Generator library instead of using it directly or augmenting it

What's the difference between wrapping it and using it directly?

Stackdriver has a logging API too?

I believe that is the full name of the (one and only) logging API: https://cloud.google.com/logging.

Btw, we don't use the word "class" in Go. There are "types".

Fixed.

What is gax?

A  gRPC helper library used by the Google API Code Generator.

What's the difference between wrapping it and using it directly?

That was a bad choice of words. By using it directly I mean having the generated client library be the official preferred client for the API; not hand-writing any code at all. (You didn't ask, but by augmenting it I mean dropping hand-written code in the same package, so some of the surface would be generated and some wouldn't.)

Landed as preview/logging.


Phase trajectories for beta=1 section

Phase trajectories for beta=1 section looks like for beta=0.999 ( buckets are still observable for ~100 MeV energies). Probably, it is related to division by zero when 1-beta appears in some denominator.

Also, fix the wrong interpolation of the cell parameters for beta=1 (and probably >1)

Now the Er and Hth equations have 1-1/beta in the denominator. If beta=1 (for the wave it's a typical situation), the equation will produce 1/0 error. However, this is multiplied by I1(0), which should produce r/2 or so. However, this simple solution didn't work, so a temporary solution is to substitute beta=1 to 1000 TeV. This problem should be solved completely after rewriting the equations for gamma instead of beta.

Now fixed, given beta=1 ~ 1 TeV


Traffic Portal trims "space" from the end of the text area

I'm submitting a ...

bug report

Traffic Control components affected ...

Traffic Portal

Current behavior:
When entering text in the "value" box in the parameter section. If you add spaces at end (because it was required for my application), they get trimmed when saving.
Expected behavior:
Text should stay as entered by the user
Minimal reproduction of the problem with instructions:
See current behavior. Simple create a new parameter and add a space at the end. It will be removed when it will be saved.

iirc, we did this on purpose because we had issues in production with a blank space at the end of the line in one of the configs.
As much as its a pain, it might be best to use some sort of string to indicate a blank space?  Something like space

We could also highlight trailing spaces to make it more obvious what's going to happen

what application requires trailing spaces @smalenfant ?


Automated cherry pick of #7904

Cherry pick of #7904 on cloud.
/cc  @hahmadia

Will try to auto merge this PR once all tests and checks are passing. This might take up to an hour.

/update-branch

/update-branch


Need a nudge to help make an anti autofill

Hi again,
I learnt of a method where you can stop queue 30 seconds or so before the estimated time and requeue and this heavily reduces chance secondary and auto fill. I was wanting to add this to your program but as I've said before I'm not too familiar with c# and I'm finding it a little difficult to find what I'm looking for to find: "time since queuing" and the "estimated queue time". I know I'm looking for something within Page10.xaml.cs. I was just looking for a nudge in the right direction to find the private fields or methods would give me what I'm looking for.

after i was also wanting to have a crack to revamp it to be more like this btw:

When semesters over I'm also going to try to make it something like this:

does this sound good to you?

Hello, i think this be easier to chat about this via discord, you can add me j.s.l


feat: add argentX download url of edge

@janek26 / @dhruvkelawala  could u pls review?

Any updates on this? Our users on edge browser are having isses, and this PR will fix the problem.

Any updates on this? Our users on edge browser are having isses, and this PR will fix the problem.

hey, are you on the argent team? I asked @janek26 or @dhruvkelawala to approve this PR 🙏

thanks

thanks guys!


Using mk functions

I noticed in https://github.com/nix-community/home-manager/issues/2249 that dconf2nix generates output in a "raw" dconf format, for example, the uint32 value 0 is emitted as a Nix string "uint32 0". This is not really the intended way to construct GVariant values in the HM library. The recommended way to emit values is to use the various mk functions as described in the manual (scroll down to where hm.types.gvariant starts).
For example, instead of having "uint32 0" one would use lib.hm.gvariant.mkUint32 0. Using the functions should be considerably more robust.
Btw, very neat project! Good job 🙂

Btw, you can see the functions in action in the gvariant test case.

Thanks a lot @rycee ! Didn't know about those mk functions, will get it don soon :)

Thanks to both of you for taking care of this! It works perfectly now.


Edge/Server Caching (Full-page, 1P & 3P, subquery) in Oxygen

At launch, there will not be a full-page cache built into Hydrogen.

you can still set cache control headers on full-page responses (e.g. loader's json ), which will affect cache in user's browsers, but not at the Edge/Server.
if we want to add support for this, we'd need to update remix-oxygen to implement a full built-in cache mechanism (due to CF Workers not respecting these headers — this may potentially be resolved but needs validation)

In general, we should not be adding caching abstractions for Hydrogen, but instead this should be done at the platform level (Oxygen & Cloudflare), or at the framework layer (Remix).
Options for us to resolve as a fast-follow to GA launch:

Guide to implementing your own full-page cache
Oxygen platform feature
Remix feature

Requirements:

Need to support edge/server caching — we should be able to cache both first party and third party query/responses — should most likely be a part of the Oxygen platform, but will discuss with Remix as well in case there is alignment there.

How did server caching work in Hydrogen 1 with Oxygen?
@benjaminsehl  I am pretty sure the server caching was working in H1 using Oxygen and Cloudflare.
Was CF worker respecting cache-control back then but not in H2?
Or, was there a proper server-caching set up in Oxygen?  If so, why can't H2 re-use the same mechanism?
Thanks!

How did server caching work in Hydrogen 1 with Oxygen?
@benjaminsehl  I am pretty sure the server caching was working in H1 using Oxygen and Cloudflare.
Was CF worker respecting cache-control back then but not in H2?
Or, was there a proper server-caching set up in Oxygen?  If so, why can't H2 re-use the same mechanism?
Thanks!

@steve-jiang we hand-rolled our own full-page cache mechanism into Hydrogen v1. We did this because we controlled the entire request/response lifecycle as a custom meta-framework.
We decided not to implement this behavior in Hydrogen 2 as:

It's based on Remix, and we have less control over the request/response lifecycle
It's not a concern of commerce tooling, which is what Hydrogen is now hyper-focused on. Instead, it should be implemented either at the hosting layer (Oxygen, Vercel, etc) or at the framework level (Remix).

Would it be hard to turn that on for Oxygen by default?

We're investigating this right now :)

Sub-request's cache-header also doesn't seem to be respected even though it's content-type is application/json. Is that also intentional?

Not sure I follow 🤔 Sub-request caching works today via storefront.query, and you can roll your own with the cache instance available in Oxygen. We're also working on potentially documenting an example for caching 3p sub-requests, or providing an abstraction at the Oxygen layer.

@steve-jiang gotcha, right — the issue is that Cloudflare runs workers at the edge, meaning the typical CDN respecting of headers does not apply. This is what we're reckoning with right now in our Oxygen exploration. If we add support to Oxygen to respect these headers, the example you shared will work once again 👍

I'm going to close this issue in favor of other https://github.com/Shopify/hydrogen/discussions/640


Add -- as marker for end of options

This allows for passing positional arguments that start with a -.
local cli = require 'cliargs'
cli:add_argument('VALUE1', 'first integer')
cli:add_argument('VALUE2', 'second integer')
cli:add_option('--verbose', 'display verbose output')
local args = cli:parse()
local val1 = tonumber(args.VALUE1)
local val2 = tonumber(args.VALUE2)
print(string.format("%d + %d = %d", val1, val2, val1+val2))

$ lua myapp.lua --verbose -- -1 5
-1 + 5 = 4

Specifying a -- stops parsing options and starts parsing positional arguments.

just to get this right; all this does is enabling positional arguments that start with - or -- right?
What's missing imo in the help text being adjusted to display this option. Somewhere in the print_help() method.

How would such a message look like? Do we have an example to look at?
The only command I can currently think of is ruby's gem install, here's the relevant snippet from its help listing that highlights the -- usage:
# $ gem install --help
Usage: gem install GEMNAME [GEMNAME ...] [options] -- --build-flags [options]

  Options:
        --platform PLATFORM          Specify the platform of gem to install
    -v, --version VERSION            Specify version of gem to install
        --[no-]prerelease            Allow prerelease versions of a gem
                                     to be installed. (Only for listed gems)
# ...

You can also look at git checkout as an example:
$ git help checkout
NAME
       git-checkout - Checkout a branch or paths to the working tree

SYNOPSIS
       git checkout [-q] [-f] [-m] [<branch>]
       git checkout [-q] [-f] [-m] --detach [<branch>]
       git checkout [-q] [-f] [-m] [--detach] <commit>
       git checkout [-q] [-f] [-m] [[-b|-B|--orphan] <new_branch>] [<start_point>]
       git checkout [-f|--ours|--theirs|-m|--conflict=<style>] [<tree-ish>] [--] <paths>...
       git checkout [-p|--patch] [<tree-ish>] [--] [<paths>...]

so maybe something like:
Usage: myapp [options] -- [args]

  Options:
      ...

Ok, that looks fine to me, but do you think we should have an explicit switch to accept that? Or should we always append this to the output?
Also, we will have to revisit the readme to make this clear; when I get options passed after the --, how do I process them? (from the caller's perspective, not cliargs, that they have to use _G["arg"] which will now contain only the remaining options)
I can do the readme update and the switch/hook if needed.

I was missing the [] around the --. So it should look more like
Usage: myapp [options] [--] [args]

  Options:
      ...

I think we should always add the [--] to the help output.
There cannot be any options after the --, that's the whole point of --, it by definition marks the end of options and only positional arguments can be accepted after the --.  This way positional arguments starting with a - can be accepted.

Ok. I'm gonna see if there are any regressions, especially around the splat argument; if we do something like /file required_arg splat_arg1 splat_arg2 "splat_arg -- 3".
I'll make the adjustments we discussed and update this today.

Also, -- cannot occur after a positional argument, otherwise it is treated as just another argument. The -- marks the end of options and the beginning of positional arguments, if it is encountered after the start of a positional argument, it should be treated as just another positional argument.
So in your above example
/file required_arg splat_arg1 splat_arg2 "splat_arg -- 3"

the first positional argument is required_arg, so the -- near the end should just be another positional argument, it should not be skipped.

Just rebased on the latest master branch.

Yes, that's what I meant: that -- might appear inside some argument should not be mistaken for the delimiter. I think we're on the same page.
What we want is:
./script required_arg splat_arg1 "splat_arg that contains -- inside of it" -- -x5 -y10
Would give us:

required_arg = "required_arg"
splat[1] = "splat_arg1"
splat[2] = "splat_arg that contains -- inside of it"
_G["arg"] now contains "-x5 -y10" for the caller to process

In your example you were missing the second --

required_arg = "required_arg"
splat[1] = "splat_arg1"
splat[2] = "splat_arg that contains -- inside of it"
_G["arg"] now contains "-- -x5 -y10", but cli:parse would continue to parse these and add them to the subsequent arguments
splat[3] = "--"
splat[4] = "-x5"
splat[5] = "-y10"

./script -f --option val -- required_arg splat_arg1 "splat_arg that contains -- inside of it" -- -x5 -y10

We would get:

opt["f"] = true
opt["option"] = val
required_arg = "required_arg"
splat[1] = "splat_arg1"
splat[2] = "splat_arg that contains -- inside of it"
splat[3] = "--"
splat[4] = "-x5"
splat[5] = "-y10"

Yes, my example was with your patch in, just to clear things up. Will work on this after the short-key fix patch.

Merged, thank you.


Fix the way training data is generated in rasa test nlu -P

Fixes #6120
The command used to exclude percentages from the previous partition
of the training dataset instead of excluding percentages from the
whole training dataset
Proposed changes:

This PR will fix the bug in excluding parts of the training dataset in comparison mode.

Status (please check what you already did):

[ ] added some tests for the functionality
[ ] updated the documentation
[ ] updated the changelog (please check changelog for instructions)
[ ] reformat files using black (please check Readme for instructions)

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

Thanks for submitting a pull request 🚀 @b-quachtran will take a look at it as soon as possible ✨

Hi @b-quachtran
Thanks for the review.
I have updated the file.
I couldn't really build Rasa from source yet so I failed to run black before creating the PR.
I have used the package as an external package (without building rasa) and seems like this did the job.


Can not handle 'Uncaught Error'

Hi,
Here is my custom module:
//segment_splicer.js
var config = require('./../config');
var utilities = require('./../utilities');
var ffmpeg = require('fluent-ffmpeg');

module.exports = {
	splice: function(raw_ad_time, crop) {
		if (!raw_ad_time || !crop) throw new Error("!!!!!!!!!! Missing argument");
		console.log("@@@@@ LAST SEGMENT IS BEING SPLITTED.");
		var segment_time = utilities.ten_seconds(raw_ad_time);
		var last_segment_path = config.akamai_user_base + 'segment' + (segment_time + 1) + "_" + config.default_bitrate + "_av-p.ts?sd=10&rebase=on";
		var command = ffmpeg(last_segment_path)
			.on('start', function(commandLine) {
				console.log('@@@@@ COMMAND: ' + commandLine);
			})
			.seekInput('0.000')
			.outputOptions(['-c copy', '-map_metadata 0:s'])
			.duration(crop)
			.on('error', function(err, stdout, stderr) {
			throw new Error('@@@@@ VIDEO COULD NOT BE PROCESSED: ' + err.message);
				console.log('@@@@@ VIDEO COULD NOT BE PROCESSED: ' + err.message);
			})
			.output('public/' + 'segment' + (segment_time + 1) + "_" + config.default_bitrate + "_av-p.ts").run();
	}
}

and here is my test:
//test/segment_splicer.js
var chai = require('chai');
var chaiAsPromised = require('chai-as-promised');
chai.use(chaiAsPromised);
var expect = require('chai').expect;
var Q = require('q');

var segment_splicer = require('../lib/segment_splicer');

describe('Segment Splicer', function() {
	it('should not work', function(done) {
		var promise = Q.promise(function(resolve, reject) {
			setTimeout(function() {
				segment_splicer.splice(1111111, "a");
			})
		})
		return expect(promise).to.eventually.throw(Error).notify(done);
	});
});

And here is what I get after running this test:
g2 [timing1●●] mocha -g 'Segment Splicer'

  Segment Splicer
@@@@@ LAST SEGMENT IS BEING SPLITTED.
@@@@@ COMMAND: ffmpeg -ss 0.000 -i http://URL/segment1111112_360_av-p.ts?sd=10&rebase=on -y -c copy -map_metadata 0:s -t a public/segment1111112_360_av-p.ts
    1) should not work

  0 passing (773ms)
  1 failing

  1) Segment Splicer should not work:
     Uncaught Error: @@@@@ VIDEO COULD NOT BE PROCESSED: ffmpeg exited with code 1: http://URL/segment1111112_360_av-p.ts?sd=10&rebase=on: Input/output error

      at FfmpegCommand.<anonymous> (/Users/scaryguy/node/g2/lib/segment_splicer.js:19:10)

What am I doing wrong?

You appear to not be using promises at all in segment_splicer.splice. Even if you were, you then wrap it in a setTimeout, which is not promise-based.
I think this question might be better for StackOverflow, since it's more about basic promise usage than about Chai as Promised.


[WIP] docs: Add developers section to docs

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Commit Message: docs: Add developers section to docs
Additional Description:
This is currently very WIP, and the focus is mostly on figuring out a rough structure for dev docs atm...
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue] Fix #13431
[Optional Deprecated:]

/retest

@mattklein123 this is a really first pass attempt to thrash out a rough structure for dev docs - with one eye on what im aware of in terms of existing docs, and the other on how we might best structure this from  pov of developers getting started
rendered docs (no copy etc atm) are here:
https://storage.googleapis.com/envoy-pr/13745/docs/dev/index.html

bump

bump


astraea-974 Implement ReassignmentHandler post request

[JSON][WEB] Implement ReassignmentHandler post request by Jackson
ref #974

麻煩rebase一下


feat: automatically update dependent functions on layer version deploy

Description of changes

added  multi env changes for function
Introduces new prompt for function to select layer latest version

Issue #, if available

Description of how you validated changes

added unit tests for the same

Checklist

[x] PR description included
[x] yarn test passes
[x] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #7140 (1d36884) into layers-rework (6de2c26) will not change coverage.
The diff coverage is n/a.

@@              Coverage Diff               @@
##           layers-rework    #7140   +/-   ##
==============================================
  Coverage          56.39%   56.39%
==============================================
  Files                448      448
  Lines              21886    21886
  Branches            4384     4384
==============================================
  Hits               12343    12343
  Misses              8764     8764
  Partials             779      779

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 6de2c26...1d36884. Read the comment docs.


PWN-5388 - terms update

Jira Ticket
https://p2pvalidator.atlassian.net/browse/PWN-5388
Description of Work
p2p_privacy_policy.pdf & p2p_terms_of_service.pdf files updated + Terms of Use -> Terms of Service

@kabramovp2p I can't see where you get the pdf files which you uploaded.
If I remember correctly we don't have any places right now where we show the button which opens terms of service

@kabramovp2p I can't see where you get the pdf files which you uploaded. If I remember correctly we don't have any places right now where we show the button which opens terms of service and looks like we need to add it

textViewTermsAndPolicy.apply {
                text = SpanUtils.buildTermsAndPolicyText(
                    context = requireContext(),
                    onTermsClick = { presenter.onTermsClick() },
                    onPolicyClick = { presenter.onPolicyClick() }
                )
                movementMethod = LinkMovementMethod.getInstance()
            }

NewOnboardingFragment


custom jwt token header name

When the request comes in via the Google Identity Aware Proxy (IAP), the IAP adds the jwt token using the header name x-goog-iap-jwt-assertion. Does OIDC-Guard support reading the jwt token from a custom header instead of the default Authorization header?

Version 2.0.0 supports this:
  JWT:
    # Sets Authorization header name, if not set "Authorization" will be used
    authorizationHeader: ""

Thanks and I will give it a try!
Is the "bearer" in the auth header value required? Normally jwt header looks like "Authorization: bearer xxxx", but Google IAP sends "x-goog-iap-jwt-assertion: xxxx" without the "bearer" word. If it is mandatory, can it make it configurable?

It does not append Bearer at the moment, I will make a new feature for this.

Added this feature in, https://github.com/IvanJosipovic/OIDC-Guard/pull/34


Units passed to jsPDF, seems to have no effect

Describe the bug
When using the html-to-pdf-options prop, the jsPDF units options seems to have no and always assumes pt.
To Reproduce
    const htmlToPdfOptions = {
      jsPDF: {
        unit: "mm", //this seems to not do anything
        format: [86, 54], // this is passed, but assumes units in `pt`
        orientation: "landscape",
      },
    };

Package Version
1.8.0

After looking at the source it seems this is an issue with jsPdf, and not this library as you're simply passing the config.

Alright cool, I'll be closing this now.


Always prefer the post-table for TrueType fonts with (0, x) cmap-tables (issue 13433)

While I don't know if this is necessarily the "correct" solution, it does fix issue 13433 without breaking any of the existing reference-tests.

/botio test

/botio-windows browsertest


Deploy endorsers for all (most) ledgers

Current endorser deployments only support CanDy dev/test/prod.
We need to support other ledgers:

[x] BCovrin dev/test/prod

Related ticket: https://github.com/bcgov/DITP-DevOps/issues/61

[x] CANdy dev/test/prod
[x] Sovrin StagingNet/MainNet

Related ticket; https://github.com/bcgov/DITP-DevOps/issues/28

Necessary for https://github.com/bcgov/traction/issues/408

Setup of endorsers for BCovrin legers is underway

Setup of endorsers for BCovrin legers is complete. Refer to https://github.com/bcgov/DITP-DevOps/issues/61 for details.

Setup of the endorsers for the Sovrin networks is underway.  Refer to https://github.com/bcgov/DITP-DevOps/issues/28 for details.

The portion of this task locking https://github.com/bcgov/traction/issues/408 has been completed; https://github.com/bcgov/DITP-DevOps/issues/61

This is complete.  Tasks have been created to track the related work of registering the issuer agents with the endorsers.


Non-Public Members only on IPublishingProfile (via GetPublishingProfile)

With Microsoft.Azure.Management.AppService.Fluent version 1.0.0-beta50
https://github.com/Azure/azure-sdk-for-net/blob/Fluent/src/ResourceManagement/AppService/Microsoft.Azure.Management.AppService.Fluent/Domain/IPublishingProfile.cs
I can access those via Reflection, yes, but aren't those intended to be public?
Further observations:
Why is WebDeploy information the publishing profile missing?
Is this intentionally not also available as Async method?

Hi @christophwille, thanks for reporting this. This is a mistake. We will fix them in this month's release.
Which WebDeploy support are you looking for? The file upload in Kudu?

Hi @christophwille
According to https://docs.microsoft.com/en-us/dotnet/articles/csharp/programming-guide/classes-and-structs/access-modifiers, interface members are always public.


no limping installed on macOS

BOSSA 1.9.1. MacOS Mojave: Crash with the following message:
Library not loaded: /usr/local/opt/libpng/lib/libpng16.16.dylib
Referenced from: /Applications/BOSSA.app/Contents/MacOS/bossa
Reason: image not found

To fix the problem a simple:
$ brew install libpng

Fixes things
bossa-crash-2019-08-29.txt
.

BTW: BOSSA 1.8 worked fine until I selected the "wrong" com port. Once selected, the app crashed. On restarting the app, you get an immediate crash. In both cases, there was no crash dialog -- the app just died. Upgrading to 1.9.1 then started fine.
This is not a critical issue for me (yet). Currently there is no new firmware for my product so I don't really need the tool (yet).


Structure changes

-make saves folder
-change character folder to hold only template folder

template folder will have sheet templates based on which system it belongs to

Hey, I'd like to work on this

Hey, I'd like to work on this

hey, do you have discord or something?
well just general info: create a branch, do the issue and commit -> push -> pull request, dont know how much experience you have I myself am junior and doing this while working so not much time and this project is made more like long run and for fun, just so you know.

I have discord yes
On Mon, Apr 17, 2023 at 5:55 PM ChattyTurtle @.***>
wrote:

Hey, I'd like to work on this
hey, do you have discord or something?
well just general info: create a branch, do the issue and commit -> push
-> pull request, dont know how much experience you have I myself am junior
and doing this while working so not much time and this project is made more
like long run and for fun, just so you know.
—
Reply to this email directly, view it on GitHub
https://github.com/ElectronicMouse/game/issues/10#issuecomment-1511741531,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXVPPE4BP4RD66HS7Z5CNFLXBVYZPANCNFSM6AAAAAAXBEX6S4
.
You are receiving this because you were assigned.Message ID:
@.***>


Application crash only when AppBundled on the Google Play Store

OS Platform and Distribution
Android 12, 13 and 14
Compiler version
No response
Programming Language and version
Java
Installed using virtualenv? pip? Conda?(if python)
No response
MediaPipe version
No response
Bazel version
No response
XCode and Tulsi versions (if iOS)
No response
Android SDK and NDK versions (if android)
No response
Android AAR (if android)
Yes
OpenCV version (if running on desktop)
No response
Describe the problem
AppBundle app crash for Android API Level > 31
Complete Logs
I'm developing a flutter application embedding a native plugin with Mediapipe using selfie_segmentation example.
The app works well in APK and in debug mode, but as soon as I create an AppBundle for the Google Play Store I've an error that made the app crash (for compileSdkVersion  > 31).
You can try the app here: https://play.google.com/store/apps/details?id=com.karaocine.app
It's compiled with compileSdkVersion 33 and targetSdkVersion 33.

Looking forward to hearing from you!
Thanks.
P.S: I opened another issue here: https://github.com/google/mediapipe/issues/5208 which is probably not related but it is worth mentioning it.

Hi @UnlimitedX,
Could you kindly share the specific steps you are following as per the documentation? Currently, based on the issue descriptions, it appears that you are utilizing MediaPipe with Flutter, a scenario that we have not tested yet.
Unfortunately, we are unable to provide support for this at the moment. However, we would like to inform you that we are actively working on implementing official support for Flutter, and development is currently in progress. You can monitor the progress by checking the official updates here but for now we can not provide you any timeline for this. if this is the scenario you are encountering, there may be limited actions we can take to address the issue at this time.
Thank you!!

Hi @kuaashish
Thanks for the prompt reply!
I've developed my own Java plugin for flutter, but there is nothing special handled by Flutter for Mediapipe, only the permissions of the app and the generation of the surfaceTexture.
Otherwise I browsed Mediapipe examples available online to implemented the code. It is working great when using the debug mode and when compiling in APK using Flutter (except the error here that happens sometimes). I will try to update to the latest version of Mediapipe and Flutter to check if the problem could be fixed.
The FrameProcessor is initialized like this :

And we inject the background video here using the VideoInput class here:

Just to follow - are you able to use our new Image Segmenter? The Selfie Segmentation API that you are using is no longer being actively developed.
https://developers.google.com/mediapipe/solutions/vision/image_segmenter
As a side note, most of MediaPipe is also deployed as a pre-compiled binary. Any C++ code we distribute is already compiled with release flags and all optimizations enabled.


feat(system-contracts): remove unecessary methods from L2BaseToken.sol

What ❔
Remove Name, Symbol and Decimal from L2BaseToken.sol
Why ❔
Multiple issues with those methods :
1- For hyperchains that does not use ETH as base token, the values returned by those methods are wrong so it should not be hardcoded. I couldnt find a good way to make them variable without heavy changes on the initialisation process
2- L2BaseToken IS not an ERC20 as it does not implement IERC20 interface (no approval or allowance) but it looks like it which confuse some explorers
One example is on https://era.zksync.network/
The value of ETH is counted twices as it is also recorded as an ERC20
https://era.zksync.network/address/0xd0c511bdbf219e3d5793e47b9ad559c9f69b3404
We should remove all confusion and make "symbol" and "name" agnostics as it is done on Ethereum
Checklist

[x] PR title corresponds to the body of PR (we generate changelog entries from PRs).
[x] Tests for the changes have been added / updated.
[x] Documentation comments have been added / updated.

@vladbochok any additional info you need ?

@vladbochok any advices on how to update the SystemContractsHashes file properly?

@vladbochok any advices on how to update the SystemContractsHashes file properly?

@thomas-nguy you should be able to merge in dev, take any of the changes (this will be overwritten), and then run yarn sc build && yarn sc calculate-hashes:fix from the root folder

@koloz193  Thanks, I have rebased and updated the file
I am ending having a lot of differences, perhaps the file in the dev branch is outdated?

Hey @thomas-nguy, we should be good to go with this PR. Let's wait for the CI and then I would merge.

@vladbochok ?


[Concurrency] Disable applying the -fswift-async-fp flag for Xcode builds until someone figures out the root cause

Since people are still complaining about this on the forum, disable it for the Xcode builds for now.
@davezarzycki, let me know what you think.

Hi @buttaface -- I don't have any opinion about the change in and of itself other than it's good to be defensive about passing compiler flags. That being said, I did a clean build and it doesn't break my minimal Linux setup, so that's good.


Update to Spring Boot 2.1.5.

Update Gradle to version 5

EnvironmentTestUtils.addEnvironment has been deprecated in favour of TestPropertyValues

With Spring Boot 2.1 the bean overriding has been disabled by default. Seems like we have been relying on this feature with at least the RestClientSpecification bean, so I needed to enable it with configuration property "allow-bean-definition-overriding" since I'm not sure whether that was done on purpose or not. https://github.com/spring-projects/spring-boot/wiki/Spring-Boot-2.1-Release-Notes

There was a breaking change in Lombok v1.16.20 where by default Lombok does not add the @ConstructorProperties annotation to constructors anymore. This breaks with the immutable bean deserialization, so we need to configure Lombok to generate this annotation on constructors.

HttpServerErrorException has now a hierarchy: https://github.com/spring-projects/spring-framework/issues/19967

Something has changed in the Jackson-Spring library, not exactly sure what, but I ran into a failing tests case testOptionalCollectionResource_found. Basically the test was failing cause Jackson didn't know how to deserialize the Hateoas type inside the Optional. Don't ask why or how, but for some reason the objectMapper which was used for the deserialization of the response did not contain the Jackson2HalModule. So I basically "fixed" it by configuring the test class objectMapper to contain all needed modules. I'd love to know what really broke the test case, but it got too complicated for me to follow the deserialization configuration. So if you have a better solution you come across, I'd love to try that.

Hi @polysantiago! Have you had change to look at my PR? Thanks :)

@mmorsky I have pushed a new branch 2.x which includes upgrading to Gradle 5.5.1. You can resolve the conflicts and then we merge.


Refresh page

If anybody besides me is using reset and wondering why secret doesn't appear, then please, refresh the secret page. It will appear only then as it turns out.
Spent too much time with this...
P.S. Might be good to add this as an instruction under reset command :)

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

ID: 364ccdfb-268e-08f0-a357-94a19c5213fd
Version Independent ID: d2c1a19f-bfbd-b273-77e2-9fdb596181b5
Content: az ad app credential
Content Source: src/azure-cli/azure/cli/command_modules/role/_help.py
Service: active-directory
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

hi @arrownj could you ps have a look? thanks

Hi @wonderplayer , @rloutlaw, do you mean the secret is to displayed in the output of az ad app credential reset ? If yes, there is actually the password property in the output, below is an example:

PS D:\code\cli\azure-cli> az ad app credential reset --id 448488a7-becf-4d79-aa9e-8262f8******
{
"appId": "448488a7-becf-4d79-aa9e-8262f8******",
"name": "448488a7-becf-4d79-aa9e-8262f8******",
"password": "P!CC;jw?F]p{;${I5P4n[T`[ge******",
"tenant": "54826b22-38d6-4fb2-bad9-b7b93a******"
}

If it's not this case, could you please help provide us the exact commands you are using ? Thanks very much.

Hi @arrownj, I meant in the interface. Command output is fine.
I was creating secrets and then moving between blades, but what I really needed is to refresh the page to see my newly created secret.

Hi @arrownj Xiaojian Xu FTE, I meant in the interface. Command output is fine.
I was creating secrets and then moving between blades, but what I really needed is to refresh the page to see my newly created secret.

I'm not sure I understand your problem exactly. Are all these done with CLI commands? And what is 'refresh the page` ? Could you please help give a reproduce steps ? Thanks

So, you can create a secret in Azure portal at http://portal.azure.com. There in the interface I see that no secrets are created for my app registration when I execute command. But, when I refresh the browser page, I see that it is created.

Do you mean you create a secret via CLI but it's not shown in portal unless you refresh the browser ? Could you please provide the exact reproduce steps? What's the actual and expected behavior you have for CLI ? Thanks very much.

Hi @wonderplayer , any feedback ?

Hi @wonderplayer , will close this issue first as lacking of further feedback. You can reopen it or submit a new one if you still have problems with it. Thanks


docs: update ci badge

Codecov Report

Merging #203 into master will not change coverage.
The diff coverage is n/a.


Refutation cases

Is there any syntax for refutation cases? Or is that planned?

I don't have a specific need for it at the moment. It's only come up for me while experimenting.
I partially asked just to make sure I'm not missing something. However, it seems like it would be a relatively simple syntax to implement. (Perhaps a naïve assumption on my part?)


Added ignoreDevices property to @TargetDevices annotation

📃 Description
Added ignoreDevices property to @TargetDevices annotation
🤔 Motivation and Context
Sometimes, You need to run the test on all devices except some.
🧪 How Has This Been Tested?
Manually
⚒ Does the PR contain new tests or refactored old ones?
No
📷 Screenshots (if appropriate)
Not needed
📦 Types of changes

[x] New feature (non-breaking change which adds functionality)
[ ] Enhancement to a current functionality
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Bug fix (non-breaking change which fixes an issue. Please add issue link here too).

✅ Checklist

[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] This PR contains new tests that cover the new code.
[ ] This PR refactor previous tests to cover the new code.
[ ] This PR is part of a set of PRs that build a bigger feature. If so, please, add here links to previously merged or open PRs.

@ancirja-m can you please show an example of its usage? e.g.
@Test
fun foo () {
...
}

Not now, I'll add it in the future user stories. I'll add some android tests samples for each annotation and use cases.
I wanted to add it somewhere in the SDK, but I didn't find a good place for this ignoreDevices case.


Stylesheet suggestion - light grey

Perhaps lighten it slightly to e.g. #BBBBBB to improve legibility while still retaining the white/grey combination

@jofster for which parts of the website? Can you explain a bit please?

for the ICO homepage - the "body" copy is in a grey around which there were some comments about legibility over the map background images. If the grey is lightened it could resolve this

Yeah we fixed that by changing a few things :wink:, we now have ways better backgrounds!


Fix CI

CI started failing today due to chrome version issues (again). https://app.circleci.com/pipelines/github/pulibrary/dpul/1705/workflows/97010416-5f25-4527-8cae-97454703b932/jobs/6729

fixed via #1476


⚠ Site ANTS has degraded performance

In 3151aa3, Site ANTS (https://ants.gouv.fr) experienced degraded performance:

HTTP code: 200
Response time: 5730 ms

Resolved: Site ANTS performance has improved in 5504fd4.


use multi json instead of json

it seems weblogic (though not tomcat,jetty, etc) has some issues with the json(-java) gem. The only working workaround we found is using jrjackson for jruby and another (Currently oj) for pure ruby platform. The multi_json gem wraps around multiple json parsers and uses what is available.

closing, as I believe the json dependency should just go all together. JSON is part of the ruby kernel as of ruby 2.0 (see http://ruby-doc.org/stdlib-2.0.0/libdoc/json/rdoc/JSON.html)


merging annotation layers needs to be fixed

due to the commit modifying Feature2DHandler merging annotation layers does not work instead all of the layers appear to disappear.

Pretty sure I resolved this a while back when fixing the mini annotation layer. But lmk if it's still an issue.


groestlcoin_hash: init at 1.0.1

Motivation for this change
groestlcoin_hash: init at 1.0.1
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

When running nixpkgs-review pr 156497:
error: hash mismatch in fixed-output derivation '/nix/store/xspchm5hfm592nvrlv7hlyjz61xi525a-groestlcoin_hash-1.0.1.tar.gz.drv':
         specified: sha256-qVkjviwsKaDMBrf9VwwCzzz1/xPcUdpYd0eAeckiAqY=
            got:    sha256-Nkco8ZA0rJaT9Mx4NIcptMvzd4h0BsRn2+gomdesirg=

When running nixpkgs-review pr 156497:
error: hash mismatch in fixed-output derivation '/nix/store/xspchm5hfm592nvrlv7hlyjz61xi525a-groestlcoin_hash-1.0.1.tar.gz.drv':
         specified: sha256-qVkjviwsKaDMBrf9VwwCzzz1/xPcUdpYd0eAeckiAqY=
            got:    sha256-Nkco8ZA0rJaT9Mx4NIcptMvzd4h0BsRn2+gomdesirg=

Its asking for the SRI hash it seems. After reading on https://nixos.wiki/wiki/Nix_Hash im unsure what is the correct syntax:
sha256 = "sha256-Nkco8ZA0rJaT9Mx4NIcptMvzd4h0BsRn2+gomdesirg=";
or
hash = "sha256-Nkco8ZA0rJaT9Mx4NIcptMvzd4h0BsRn2+gomdesirg=";

Also, could you add pythonImportsCheck? Missed that before

Done


Use regex for topLevelImportPaths option

In the 2.x.x version of babel-plugin-styled-components, topLevelImportPath requires a regex, so we escape the literal here.

I don't think this is a good idea. I'd rather apply the anchors in the plugin than run around adding them to callers of the plugin.
If that PR gets merged, resulting in plugin 3.0, then the dep here could change to ^1.12.0 || ^3
If for some reason we go with this approach instead, then beware the anchors are backwards so it will never match. I don't know if this is somehow passing tests, but if so, it needs a new test.

I'm closing this for now, I'm not sure how this macro is working at all, and I've run out of time to debug


Please stick to C++11 in Dart VM for now

Google-internal host (linux) toolchain is not on C++14 or C++17 yet. They explicitly told us we can build Dart VM that way but it is not supported.
We currently do build Flutter engine using these unsupported flags but the risk is mitigated by the fact that Android and iOS toolchain is on C++14. For Flutter, only flutter_tester needs to be built on the host toolchain which makes the risk somewhat manageable.
I am opening this issue so we can discuss whether going to C++14 or 17 is a must for Dart VM and whether it is worth the risk of the VM being broken in Google with no real help from toolchain owners.

cc/ @a-siva , @mraleph

@sjindel-google updated our build configuration to use C++11 instead of C++14 (see https://dart-review.googlesource.com/c/sdk/+/85202)
We can't use C++14 anyway because Flutter builds 32-bit host binaries with C++11 due to buildroot limitations.

We can't use C++14 anyway because Flutter builds 32-bit host binaries with C++11 due to buildroot limitations.

This has been resolved and Flutter now builds with C++14 everywhere: https://github.com/flutter/buildroot/commit/50e07b839afd81dab05e7d77168dbd1a553fc13b
@mehmetf Do you know if Google supports C++14 internally yet? In https://dart-review.googlesource.com/c/sdk/+/101740 we're discussing utilizing std::unique_ptr (C++11) within the Dart VM, and using std::make_unique (C++14) came up. Seems like this is worth revisiting.

Looks like C++11 compatibility is still required by Google for the foreseeable future. :( Googlers can find details at go/cxx17.

Update: Google now supports C++17 (Googlers, see go/towt/167).
Also, Flutter uses C++17 on all supported platforms (at least as far as I can tell).

Not entirely :-/
Unfortunately the Cast toolchain which customer: dream uses is still on C++11 as far as I can tell. Will investigate more.

[sad react]

It seems like the Cast toolchain issue is resolved now, and C++17 is available everywhere we're building Flutter.
Are there any other Dart customers still keeping Dart on C++11?


fix kill-others concurrently option

Otherwise if a job dies, the others doesn't

Thanks for the contribution, but this is intentional. --kill-others-on-fail is the default (only option currently) because I don't want to wait for my build to finish if my tests fail (for example).


Broken links to IPEA gpkg files

All links to download geographic data from IPEA is broken returning 404 (Not Found).
Hence, the generation pipeline described in instance generation README.md finish unsuccessfully. The generate.py script fails to find the *.gpkg files in the data_raw directory, raising following error:
fiona._err.CPLE_OpenFailedError: ./loggibud/loggibud/v1/instance_generation/../../../data_raw/33.gpkg: No such
file or directory

The (broken) links used in download.sh script are:

https://www.ipea.gov.br/geobr/data_gpkg/census_tract/2010/33.gpkg
https://www.ipea.gov.br/geobr/data_gpkg/census_tract/2010/53.gpkg
https://www.ipea.gov.br/geobr/data_gpkg/census_tract/2010/15.gpkg

Thanks for the report. I believe IPEA changed the path internally. I'll investigate this further.

I'm also experiencing this issue, will this be updated soon?

By checking the metadata from IPEA (available at this CSV), I believe that the new paths are:
https://www.ipea.gov.br/geobr/data_gpkg/census_tract/2010/33census_tract_2010.gpkg
https://www.ipea.gov.br/geobr/data_gpkg/census_tract/2010/53census_tract_2010.gpkg
https://www.ipea.gov.br/geobr/data_gpkg/census_tract/2010/15census_tract_2010.gpkg
But I couldn't validate if anything else changed, and the rest of the pipeline still works.

@gabisurita I used these links to manually download the files (using curl) and placed them into the data_raw directory. The instances were generated successfully.
Therefore, I believe that replace the links in "download.sh" fix this issue. I can test this and open a PR if all OK.

Solved by #28 .

We should also fix the README links, but I'll keep this closed since the script is already working.


Write custom serialisation for experience replay

Although it is now possible to save/load experience replay memories (https://github.com/Kaixhin/PlaNet/issues/3), naively using torch.save fails with large memories. Dealing with this would require custom serialisation code.

The way I have been getting around this is saving each part of the memory as .npz
D = np.empty((10 ** 5, 3, 64, 64), dtype=np.uint8)

# previous method
torch.save(D, "torch_save.pth")

# npz method
np.savez_compressed("numpy_save.npz", data=D)

# size: 1.2 MB
x = np.load("numpy_save.npz")
print(x["data"].shape)

# size: 1.23 GB
y = torch.load("torch_save.pth")
print(y.shape)

As you can see, the size is reduced by an order of magnitude. If you like, I can provide a pull request that adds this save & load functionality to the memory class.

The memory savings are far less when the arrays are full:
# make sure array is full
D = np.random.rand(10 ** 4, 3, 64, 64)

# torch method
start_time = time.time()
torch.save(D, "torch_save.pth")
print("Torch save time: {:.2f}".format(time.time() - start_time))

# numpy method
start_time = time.time()
np.savez_compressed("numpy_save.npz", data=D)
print("Numpy save time: {:.2f}".format(time.time() - start_time))

# torch method (1.4 GB)
start_time = time.time()
x = torch.load("torch_save.pth")
print("Torch load time: {:.2f}".format(time.time() - start_time))

# numpy method (0.9 GB)
start_time = time.time()
x = np.load("numpy_save.npz")
print("Numpy load time: {:.2f}".format(time.time() - start_time))

Output:
Torch save time: 10.19
Numpy save time: 38.53
Torch load time: 6.40
Numpy load time: 0.05

So you get a reasonable memory decrease, but a large increase in the time taken to write.
Any further suggestions most welcome, as this makes it hard to do runs across sessions.

I suspect one solution might be to use memory-mapped numpy arrays? Never looked into them though.

Oh nice, this absolutely seems viable. (10**6, 3, 64, 64) is 12 GB on the hard-drive but reading and writing are mostly instant. As a minimal working example:
class Buffer(object):
    def __init__(self, buffer_size=10 ** 6):
        self.buffer_size = buffer_size
        self.obs_shape = (buffer_size, 3, 64, 64)
        self.obs_path = "obs.dat"
        self.idx = 0

    def create_new_file(self):
        obs_f = np.memmap(
            self.obs_path, dtype=np.uint8, mode="w+", shape=self.obs_shape
        )
        del obs_f

    def add(self, obs):
        obs_f = np.memmap(
            self.obs_path, dtype=np.uint8, mode="w+", shape=self.obs_shape
        )
        obs_f[self.idx] = obs
        del obs_f
        self.idx += 1

    def sample(self, idxs):
        obs_f = np.memmap(self.obs_path, dtype=np.uint8, mode="r", shape=self.obs_shape)
        return obs_f[idxs]

if __name__ == "__main__":
    buffer = Buffer()
    for i in range(10 ** 6):
        obs = (np.random.rand(3, 64, 64) * 255).astype(np.uint8)
        buffer.add(obs)
    idxs = np.random.randint(0, 10 ** 4, size=100)
    data = buffer.sample(idxs)

I'll get round to integrating it with your buffer at some point - if you think it'd be worthwhile.

Nice! I guess it'd be good to have this as a sort of helper class, so that the serialisation of the buffer can include other bits, like the current index.

I think I follow.
I was imagining that the buffer was serialised in a standard fashion and contained metadata about things like .dat file locations, index, data shapes, etc. The problem with memmap is you need both the data type and array shape whenever you invoke it - these can't be serialised into the .dat file itself.
Unless I've misunderstood your suggestion.

I was just talking about https://github.com/Kaixhin/PlaNet/blob/master/memory.py#L15-L18 , but yes, anything that needs to be stored to recover the whole ExperienceReplay class.

Yeah, that's how I have it set up - you can serialise the Replay class, and as long as the .dat files have not changed location, it works just fine (and has no computational overhead, as far as I can tell):
https://github.com/alec-tschantz/planet/blob/master/planet/training/_buffer.py
I can make this consistent with your repo if you'd accept a pull request

At a quick glance looks good, but please put in a PR so I can have a proper review. Thanks!


sandbox does not send SMS for some reason

cant test because it seems that the sandbox does not sent SMS verifications for 2FA.
https://sandbox.transferwise.tech/public-keys/create/upload

Two factor authentication (2FA) code for sandbox login is 111111.


Add support for initializing certificates (user certificates)

We add a reactor that will run an orchestration that will be executed
during the first boot of the services.
This new orchestration will make sure the CA generates both a private
and a public CA certificates, and depending on the service one or the
other will be used. E.g. salt-api uses the private CA, while velum will
use the public CA.
In the future, this salt-states can be modified, so the public CA is
a CA provided by the customer.
Fixes: bsc#1043589
Depends on:

https://github.com/kubic-project/velum/pull/210
https://github.com/kubic-project/caasp-devenv/pull/19
https://github.com/kubic-project/caasp-container-manifests/pull/64
https://github.com/kubic-project/e2e-tests/pull/65

Closing this one for now, will revisit it later.


async/await issues

Explanation

Doesn't work

     const result = await db.executeSql(`SELECT * from name_actions`);

returns undefined

Doesn't work

     db.transaction(async tx => {
         const res = await tx.executeSql('SELECT * from name_actions', []);
         console.log(res);
     });

returns undefined

Works

     db.transaction(tx => {
         tx.executeSql('SELECT * from name_actions', [], (tx, res) => {
             console.log(res);
         });
     });

returns expected ResultSet
Your Environment
Workspace

Typescript
Android 10.0 (Google Pixel 3)

Dependencies
  "dependencies": {
        "@react-native-community/masked-view": "^0.1.6",
        "@react-navigation/native": "^5.0.0",
        "@react-navigation/stack": "^5.0.0",
        "@types/react-native-sqlite-storage": "^3.3.2",
        "react": "16.9.0",
        "react-native": "0.61.5",
        "react-native-card-stack-swiper": "^1.1.9",
        "react-native-gesture-handler": "^1.5.6",
        "react-native-reanimated": "^1.7.0",
        "react-native-safe-area-context": "^0.7.2",
        "react-native-screens": "^2.0.0-beta.2",
        "react-native-sqlite-storage": "^4.1.0",
        "react-native-vector-icons": "^6.6.0",
        "styled-components": "^5.0.1"
    },
    "devDependencies": {
        "@types/styled-components": "^4.4.2",
        "@babel/core": "^7.6.2",
        "@babel/runtime": "^7.6.2",
        "@react-native-community/eslint-config": "^0.0.5",
        "@types/jest": "^24.0.24",
        "@types/react-native": "^0.60.25",
        "@types/react-test-renderer": "16.9.1",
        "@typescript-eslint/eslint-plugin": "^2.12.0",
        "@typescript-eslint/parser": "^2.12.0",
        "babel-jest": "^24.9.0",
        "eslint": "^6.8.0",
        "prettier": "^1.19.1",
        "jest": "^24.9.0",
        "metro-react-native-babel-preset": "^0.56.0",
        "react-test-renderer": "16.9.0",
        "typescript": "^3.7.3"
    },

My current HACK solution to get it working
async function query<T>(query: string, args: any[] = []): Promise<T[]> {
    const db = await getDB();
    return new Promise<T[]>((resolve, reject) => {
        db.transaction(tx => {
            tx.executeSql(
                query,
                args,
                (_tx, res) => {
                    const rows = res.rows;
                    const items: T[] = [];
                    for (let i = 0; i < rows.length; i++) {
                        const item = rows.item(i);
                        items.push({ ...item });
                    }
                    resolve(items);
                },
                (_tx, error) => {
                    reject(error);
                },
            );
        });
    });
}

Have you enabled promises?
SQLite.enablePromise(true);

Ehh could mention this in README.md had no idea this existed. As I understand it was a pattern a long time ago


ShadowEffect

Description of Change
Adding Shadows support for iOS and Android.
As I know .NET MAUI will support shadows from the box, but it's a good idea to put a shadow effect into XCT while we support XF.
Bugs Fixed

Fixes #461

API Changes
public class ShadowEffect : RoutingEffect
{
	Color Color;
	double Opacity;
	double Radius;
	double OffsetX;
	double OffsetY;
}

Behavioral Changes
None
PR Checklist

[ ] Has tests (if omitted, state reason in description)
[ ] Has samples (if omitted, state reason in description)
[ ] Rebased on top of main at time of PR
[ ] Changes adhere to coding standard

[ ] Updated documentation

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here

A new build for this PR has been completed. As a result a NuGet has been created which allows you to verify this fix or try out the new feature! Download here

@AndreiMisiukevich Do we merge the PR with support for Android and iOS or should we wait to include Windows/macOS support in this same PR?

@AndreiMisiukevich Do we merge the PR with support for Android and iOS or should we wait to include Windows/macOS support in this same PR?

@AndreiMisiukevich Do we merge the PR with support for Android and iOS or should we wait to include Windows/macOS support in this same PR?

macOS is already supported :)
As for UWP, I would be happy if somebody else will help with that (as the separate PR)

@AndreiMisiukevich Do we merge the PR with support for Android and iOS or should we wait to include Windows/macOS support in this same PR?

macOS is already supported :)
As for UWP, I would be happy if somebody else will help with that (as the separate PR)


fix: notification denomination hotfix

combined with: https://github.com/arconnectio/ArConnect/pull/262


Update jline-reader, jline-terminal, ... to 3.26.3

About this PR
📦 Updates

org.jline:jline-reader
org.jline:jline-terminal
org.jline:jline-terminal-jna

from 3.14.1 to 3.26.3
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.jline" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.jline" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #1559.


ObtainJSONWebToken does not return refresh token

ObtainJSONWebToken mutation  does not return refresh token even though it is in the schema
and the graphene version had it.
Example:

In my schema:

Solution:
set
JWT_LONG_RUNNING_REFRESH_TOKEN": True

This library has very weird way of doing things

I have changed this in f3a0afc38d4fe1977834d94de5a89e53be2ce6b1. Now refresh_expires_in is returned only if refresh tokens are enabled in settings.

Thanks for letting me know, Nice to hear from you from a time to time :upside_down_face:

For other developers wandering here in order to get refreshToken, you should add this section into your settings.py file:
GRAPHQL_JWT = {
    "JWT_LONG_RUNNING_REFRESH_TOKEN": True,
}

That is correct, if you set JWT_ALLOW_REFRESH to True (which is ON by default), you can use access tokens as single-use refresh tokens.

... but in order to make it useful, you also have to force the module to invalidate expired token with settings like this:
GRAPHQL_JWT = {
    "JWT_VERIFY_EXPIRATION": True,
    "JWT_LONG_RUNNING_REFRESH_TOKEN": True,
}


get cue temp from remote through URI

When it's going to get cue templates of capabilities, use templateURI as higher priority than template strings in definition file.
BTW, I will complete tests after this PR containing test data is merged.

you could use existing file here to do test https://github.com/oam-dev/kubevela/blob/master/vela-templates/web-service.cue @captainroy-hy

you could use existing file here to do test https://github.com/oam-dev/kubevela/blob/master/vela-templates/web-service.cue @captainroy-hy

Sure, got it.


[TEAM6] Trust

Summary
Added variables for trust and means of keeping track of neighbours. Will continue with implementing trust in parallel with messages in future
Additional Information
Changed initialisation of array in config:
memory{} looks like an empty struct of type memory. make(memory, 0) clarifies that it's a slice (and calls runtime.makeslice)
neighbours struct stores the conceived IDs of the above and below neighbours for use in the trust map

Good To Go


Topical organization of folders and files

@skwp what do you think of a topical approach like Holman's dotfiles for supporting different languages, plugins, and so on:
https://github.com/holman/dotfiles
Cheers

Hope you don't mind my chiming in 😄 . It is an interesting approach, but I can easily this abstraction starting to leak when you bring a framework like prezto into the mix. On top of that, and in my specific case, I wouldn't have enough "material" per topic anyway, so I'd end up with a long list of topics with minimal content in each. Hence makes more sense to have zsh/java-aliases.zsh than java/aliases.zsh.

Of course I don't mind! All brains welcome 😉
What do you mean or how to you see this leaking happening? My idea is not to make this modules ("topics") toggleable in runtime but rather in installation time. So, for example, if I disable "ruby" it wouldn't be installed in my system and in order to install it later on I'd have to reinstall yadr...
As to your second point, I agree: we all use just a small portion of the repo features, we all have just a few topics. But that's precisely one of the pain points in the repo:

It causes a lot of friction to add new features or supported languages or vim plugins since all users of the repo would be getting it and getting their system bloated with stuff they don't use
It makes contributing back to this repo very unlikely and harder. For exempla, if I'm a Go developer, then I have to fork the repo and forked it will have to stay forever. Then eventually I might have conflicts in my fork which are impossible to cleanly rebase against the upstream (my case now). So if we had this easy way of adding and removing topics, we could easily implement an ignore or skip list for installation time ("only install ruby", "only install ruby, javascript and go", etc etc)

prezto for example comes with multiple optional modules installed under its own root dir. I guess you could specify which modules to load in your own topic dirs and the installation script would take care of the rest.

It causes a lot of friction to add new features or supported languages or vim plugins since all users of the repo would be getting it and getting their system bloated with stuff they don't use

Good point.

For exempla, if I'm a Go developer, then I have to fork the repo and forked it will have to stay forever.

Dotfiles Are Meant to Be Forked so this is only inevitable and ultimately a good thing imho.

So if we had this easy way of adding and removing topics, we could easily implement an ignore or skip list for installation time

Would each topic be a git submodule?

Hey there.
Regarding dotfiles being meant to be forked... Maybe this came to be precisely due the difficulty of having an upstream repo easier to customize. Then indeed we can't run away from forking. Again, I'm not against that, but I think there's a huge middle ground where people fork because of only a couple of lines that differs from upstream, if that much.
And the downside being he looses the changes made to upstream while upstream also looses on any improvements that fork could have made...
As for each topic being a submodule, I didn't even think of that as of now. "Whatever works" if you ask me. Worst case scenario we could just leave the files in there (no submodules). It would be just a few extra small text files so no big deal IMHO.

I'm not optimistic on module support, one of the reasons I started yadr vs using something like janus  was as repo maintainer you end up maintaining a complex module system instead of a simple set of dotfiles. We already have sort of modular stuff going on under the hood with prezto and etc and I think we'll end up with a very hard to reason about system. The whole idea of yadr was to be "opinionated" meaning don't allow so much customization, just have something that works, which means the philosophy should be more fork oriented - fork and fix whatever you want. We're not really making such huge changes to yadr now so it's relatively stable.
But I am open to other opinions if you think modular system can be done simply. I just don't want to end up holding the ball for supporting this down the line and having people frustrated that the module system isn't compatible with some newfangled thing X.

I agree with you. I also don't want to increase maintainability complexity.
I'd only go for smt modular if was pretty simple and straightforward.
My main motivations were:

I was trying to think creatively to ways of improving the fork > contribute back workflow. Most of the time when we fork we never contribute back to repos like this...

The before and after approach (for both zsh and vim) work fine but only to add or modify configurations. If I wanted to remove (not to download or use at all) a specific vim plugin, I couldn't. I'd have to fork the repo and then, after while, due to so many conflicts, one would stop checking up (and contributing to) upstream (point 1. above)...

So..up to you if you want to pursue a modular system. I definitely don't have time to work on this right now but if you have smething simple that will let people opt out of certain plugins that might be nice


Scaffold index operators and included (covered) index columns

Closes #407
/cc @khellang

@austindrenski thanks for reviewing so quickly! Pushed a commit fixing the nits, and see my comments on the larger issues of versioning.


feat: torch cache-able dataset, with sampling support

Pytorch Dataset

Work with PyTorch pipelines
Local filesystem backed cache
Faster pseudo random samplings.
Support project projection and filterings.

how useful the sampling functions are given they do nothing to randomize the order in which the dataset is iterated over, and ML users seem to care a lot about randomization.

This can be done with shuffle in torch.dataset.DataLoader or tf.data.pipeline.


Add horizontal stacked bar chart to Batch Job view

When viewing a batch job, instead of showing:
Active Slices: 840
Failed Slices: 23
Output Slices: 353632
Queued Slices: 24767

Display a horizontal stacked bar chart that consists of only one bar. The stacks on the bar chart are each from left to right:

Queued
Active
Failed
Completed

Use the same colors as each of the above states already in RJMC.

I'm working on it!

I've found that
index.html.erb is used to show All, Queued, Running, Completed, Paused, Failed, Aborted job list.
_status.html.erb is used to show the detail if any job (All, Queued, Running, Completed, Paused, Failed, Aborted job).

Hello everybody! I haven't updated this issue until now. I'm looking for good resources for javascript in rails. I have this https://guides.rubyonrails.org/working_with_javascript_in_rails.html. Do you know other resources for it? I appreciate your advice!

I'm still not able to show the Pie graphic.https://codepen.io/k3no/pen/dXAQOp?cf_chl_jschl_tk=72b1cccc0a56a3f0cd93b3b45807255d653ed394-1589573195-0-AW3z7-fdQ6jsgg84X1WznCWU92eOA5_YnoR57uVvIA8JQ4O1HjTLIlfo_WoDEw3xheJsvohdSksgMFIYEYScshSytJQyx5ao4pS9AScCPXYr33lpXFa_HC75CujhP3VIsTCihX9GZMLuPyb6VYDzNs_tdOunrrwHDJ_Z9Qgxd9W9aqZHRM9xZIB0lmLKuQwbphnBNwn5anSgUNjh09tA47b4_F24ATetw1LD48v-bqOwhyswUTVzCFYkbzECrZpxo04CZS_1NpKABwBu0qaqDGLZpBm31JWqxAzUAY9GugEp9ZEK2oDRzsPbGp8VzYV2CgN1KAQM_QdVuaBwwkxXpb_Vt63B2yyw5TiTwtRbLafo


Contact Page - Contact EmailId / Phone Number must not repeat

Contact Page - Contact Email Id / Phone Number must not repeat

this makes editing current Contact with same email lot more difficult

user cannot save contact untill user correct data, inthis case remove duplicate email/phone

We do not need any change here. we need to just check at the time of invite for mobile app. plz call me so that i can explain

NO NEED OF DEVELOPMENT GO WITH EXISTING SOLUTION ONLY PROVIDE EMAIL INVITE AS WELL.

UNLESS WE HAVE GOOD USE CASE WE DO NOT NEED ANY DEVELOPMENT

still No Patch for Existing Data


Add support for Rally issue tracker

http://www.rallydev.com

We've moved over to a new system for notifications and we have an internal ticket for this. The good news is it shouldn't take us much longer to get this shipped so it should be coming soon!
Closing this issue as this repo is being retired.


Website not loading on Browsers

Describe the bug
The site is down
To Reproduce
Steps to reproduce the behavior:

Go to 'https://www.covid19india.org.'
See blank page
Console error

Expected behavior
Should load the website
Screenshots

Desktop (please complete the following information):

OS: MacOS
Browser : Both Chrome and Safari
Version: Chrome : 81.0.4044.138, Safari : Version 13.1

Smartphone (please complete the following information):
Working on mobile

Seems to be fixed


Dump, Restore performance issue

During dump process, everything goes slower !
Also during restore process, the dump/restore page shows Loading at some point, not much important but it's worth to have a  look at it.

Currently, it will stay as beta till first release. ONHOLD


fixed Gradle typo

Gradle appears to have been mistyped as "cradle" on the title of the lab for module 19

Thanks Akilien, appreciate the correction.


[BUG] Hydra misleads users when it doesn't have write permissions to a folder

Describe the bug

Issue #1059 kind of mentions this, but it doesn't go into detail and give solutions.

When starting a download to a folder which Hydra doesn't have permissions to access, (such as Program Files) instead of throwing an error and telling the user to pick another location, Hydra shows that the download has 0 seeds (and shows the number of peers inaccurately, fluctuating between 4 and 1 and sometimes going up to 7 or 9, when the torrent has in reality 45+ peers) and keeps on increasing the completion time and showing a very small download speed, misleading the user into thinking that the torrent in question has no seeders. (this was why it took me a while to discover why my Hydra was no longer downloading anything as before, Hydra was always launched as admin, so I set the download path to Program Files, which only programs with admin rights can write to)
No permissions

With permissions

Steps to Reproduce

Start a new download
Set the download location to a folder which Hydra doesn't have access to (for example Program Files if Hydra isn't running as admin)
Watch above behaviour happen and the download not downloading
Cancel the download
Start another download but this time to a location which Hydra has access to (the user's Downloads folder for example)
Watch the download downloading normally

Expected behavior
When starting a new download, Hydra should check the download location and throw an error if it can't access it.
The error should tell the user to change the download location and try again, or to try running Hydra as an administrator.
Additional information and data
logs.txt - Recreated the bug and the "normal" behaviour, but these logs should probably not be needed anyways.
Operating System
Windows 11 Pro
Hydra Version
3.0.1
Before opening this Issue

[X] I have searched the issues of this repository and believe that this is not a duplicate.
[X] I am aware that Hydra team does not offer any support or help regarding the downloaded games.

Thanks for the report. We noticed this while running tests for 3.0. Unfortunately, we couldn't solve it in time.


Update the STC membership

With the completion of the STC 2023 voting https://blog.dapr.io/posts/2023/11/30/dapr-steering-and-technical-committee-2023-elections-results/
Need to update the STC membership here https://github.com/dapr/community/blob/master/steering-and-technical-committee-charter.md

Merged


Adds Galaxy Zoo 3D Segmentation Data

Adding segmentation maps for MaNGA galaxies annotated by humans - collected by Galaxy Zoo 3D and published in Masters et al. 2022.  There are a total of 30k samples - most without bar of spiral features.
HuggingFace Dataset Format is outlined in the updated DESIGN.md file.
Each example contains:
The dataset structure is organized with the following features:
Metadata:

total_classifications: The total number of Galaxy Zoo classifications made for this target. Note this is not the number of segementations but the number of citizen scientists who voted on if the galaxy is a spiral or barred galaxy.
healpix: The healpix value in which the subject lies.
ra: Right ascension of the target in degrees.
dec: Declination of the target in degrees.
object_id: The MaNGA ID (unique to each galaxy, not unique to each plate IFU).

Image data:

image: The RGB false color image of the target (MaNGA) galaxy. Data format: [H,W,C] = [512,512,3] 16-bit integer.
scale: The angular extent of a pixel in the image in degrees.
segmentation: The segmentation map data. Each key refers to an ordered list of four elements each referring to a specific class.

class: The class of the segmentation map. One of center, star, spiral, bar
vote_fraction: The fraction of total_classifications that were positive towards this class for a class of spiral or bar (e.g. 20/50 total classifications state the subject is barred -> 0.4). -1.0 flags that the class is either center or star as there are no votes for these classes. -0.5 flags if the total_classifications are 0 for the subject.
array: The segmentation map itself. Values range between 0 and 15, with the value at a given pixel indicating the number of users who included that region in their annotation for the respective class. I.e. a value of 5.0 indicates 5 of the 15 users marked segmentation masks which included that pixel. Data format: [H,W,C] = [512,512] 8-bit integer.

Note: Compare the orientation and scale to legacy survey data.

[ ] @oliu-io reported that the images are not aligned with the legacysurvey images, apparently there is flip, we should fix that.


Bind bomb to jid

There is a little trick with bomb command. If i set up bomb to, for example, refeaime and after that he changes his nick to whatever else — bomb failed to kick.

Added in 094c05edc3106945741eb4614215096155b514db


Removing SLDS source from Static Resource Sources

@cpolcyn looks like the build org for this build has an expired password.

I swapped the org and retried the build, and it passed. Should be good to go!

Thanks!


Design | Ensure footer component and subcomponents are up to date

To Do:

[x] Make sure subcomponents for all 4 bands fit in the new grid system for desktop (internal assets)
[x] Create Mobile variants for all 4 bands and make sure they fit in the new grid system for mobile (internal assets)
[x] Publish changes so they can be accessed by main figma file (internal assets)
[x] Set up mobile variants for the main footer components (main)
[x] Make sure this component aligns properly with Footer component on large and small screens (main)

Header + Footer Branch for review
Footer work:
https://www.figma.com/file/mh2maMG2NBtk41k1O1UGHV/branch/UunlAOOFqYCYbwG4G0ALcA/Canadian-Digital-Service
---GC-Design-System?type=design&node-id=16355-3009&mode=design&t=n5723dHCNqcRmRVa-0

Subcomponent work
https://www.figma.com/file/7HNtNYzWNmcxjYtG8UlnBz/Internal-assets?type=design&node-id=301-5230&mode=design&t=0BBb2vxCOLIj0Bm3-0


Broadcast not getting fired at all.

 Issue by sumeet-chaayos
Friday Feb 22, 2019 at 08:00 GMT
Originally opened as https://github.com/googlesamples/android-credentials/issues/34

@ankitPagalGuy @ithinkihaveacat @keyboardsurfer Sms is not getting broadcasted , when i manually broadcast the intent from adb and its receiving in broadcast receiver. Sms is :

<#> Your Verification Code is 030787. This OTP will s. NKsG/pQa+dw

Need urgent help here.

 Comment by vedasLabs
Tuesday Feb 26, 2019 at 11:18 GMT

Have u founded any solution??
i m also getting the same error


Fix for issue #676: convert integer elements of enum into the correct type

This PR fixes issue #676 .

Thanks for the PR, @ackintosh! I'm sure @gracekarina will review it as soon as she can.
One point though - would it be ok to ask to add support for other types as well? It might as well work for boolean (I've seen it) and number as well.

Thanks for the comment! I'll add support later.


41 arduino car screen

@darot-chen  please fix conflict and also rebase to latest develop

@theacheng  Done with fix conflict code


AttachPrimaryStorageToCluster failure

The error infomation:
"details": "failed to execute shell command: mount 10.X.X.X:/Primary_Storage /opt/zstack/nfsprimarystorage/prim-324b2191508e40e2a404cd17ade861a4\nreturn code: 32\nstdout: \nstderr: mount.nfs: access denied by server while mounting 10.X.X.X:/Primary_Storage\n"
It seems like the problem about authority , but there doesn't have any limit in my NFS server .
drwxrwxrwx    2 root root  4096 Sep 22 11:42 Primary_Storage

Hi,
Did you manually try: mount 10.X.X.X:/Primary_Storage
/opt/zstack/nfsprimarystorage/ in your host?
On Tue, Sep 22, 2015 at 1:33 PM, 251661412<EMAIL_ADDRESS>wrote:

The error infomation:
"details": "failed to execute shell command: mount
10.X.X.X:/Primary_Storage
/opt/zstack/nfsprimarystorage/prim-324b2191508e40e2a404cd17ade861a4\nreturn
code: 32\nstdout: \nstderr: mount.nfs: access denied by server while
mounting 10.X.X.X:/Primary_Storage\n"
It seems like the problem about authority , but there doesn't have any
limit in my NFS server .
drwxrwxrwx 2 root root 4096 Sep 22 11:42 Primary_Storage
—
Reply to this email directly or view it on GitHub
https://github.com/zstackorg/zstack/issues/128.

--
Yongkang You

Thanks, youyk:
I had add the primary storage successfully through zstack ui . Everything is ok when I tried to mount the NFS server manually ,.The problem is when i want to attach a cluster for this primary storage , the error happend.

I found a way that can solve this problem , vi the exports file at the NFS server. Change the client server address from a single IP to the entire network segment.
Example:
Form : /primarystore <IP_ADDRESS>(rw,sync)
Change to  : /primarystore 192.168.1.*(rw,sync)
But if we configure like that , there are some security problems!

Hi,
How about trying nfs server config like:
/primarystore <IP_ADDRESS>(rw,sync,no_root_squash)

Hi,youyk:
I can just use the entire network segment like :  /primarystore 192.168.1.*(rw,sync)
Thanks!


go-lustre is distributed under MIT license whereas liblustreapi is still under GPL

https://jira.hpdd.intel.com/browse/LU-5969 isn't resolved yet

Hi Dennis.
The MIT license is compatible with the GPL. Any distributed binaries which result from combining go-lustre and liblustreapi will be covered by the same version of the GPL as that which covers liblustreapi.
Do you have specific concerns about this issue?

Hi Michael,
I may be probably wrong, however my understanding is that if GPLed code is included in your project, even via linking, you must provide your project under GPL
Again, I just wanted to point it out - feel free to ignore me and close the issue if you are 100% sure

Hi Dennis.
Thanks for the input. Software licensing can be kind of confusing and tricky. The thing about the GPL is that it is a distribution license, so it only comes into effect when binaries are distributed outside of an organization. The original intent of the GPL was to ensure that users of software covered by the GPL would always be able to get the source code for the software.
In our case, the go-lustre code is covered by the MIT license (also known as the Expat License) which is more permissive than the GPL. As such, it is compatible with the GPL when combined into a binary and distributed. If the binary is not distributed, then the GPL doesn't even come into play at all. And if the binary is distributed, then the distributing party is responsible for making the source code available on request (in practice, a link to this github repo).


Rule0033AppManifestRuntimeBehind warning

Hi,
For every project i get the following warning from Lintercop. I tried reinstalling, manually installing. Setting the execution Policy.
Analyzer 'BusinessCentral.LinterCop.Design.Rule0033AppManifestRuntimeBehind' threw an exception of type 'System.ArgumentOutOfRangeException' with message 'System.ArgumentOutOfRangeException: Specified argument was out of the range of valid values. (Parameter 'index') at System.Linq.ThrowHelper.ThrowArgumentOutOfRangeException(ExceptionArgument argument) at System.Linq.Enumerable.ElementAt[TSource](IEnumerable1 source, Int32 index)
at BusinessCentral.LinterCop.Design.Rule0033AppManifestRuntimeBehind.FindValueOfFirstValueLessThan(SortedList2 sortedList, Version version) at BusinessCentral.LinterCop.Design.Rule0033AppManifestRuntimeBehind.CheckAppManifestRuntime(CompilationAnalysisContext ctx) at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.<>c__DisplayClass48_1.b__0() in D:\a_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 618 at Microsoft.Dynamics.Nav.CodeAnalysis.Diagnostics.AnalyzerExecutor.ExecuteAndCatchIfThrows_NoLock(DiagnosticAnalyzer analyzer, Action analyze, Nullable1 info) in D:\a_work\1\s\source\Prod\Microsoft.Dynamics.Nav.CodeAnalysis\DiagnosticAnalyzer\AnalyzerExecutor.cs:line 1102'`

Can you share a small example project which creates this error?

I think I have found a possible repo, could you share your app.json file to confirm my hypothesis?

ALProject1.zip
I attached a small sample project

Found the issue!
"platform": "<IP_ADDRESS>",
"application": "<IP_ADDRESS>",

Setting both of them lower then <IP_ADDRESS> causes this error, because that isn't a valid version; https://learn.microsoft.com/en-us/dynamics365/business-central/dev-itpro/developer/devenv-choosing-runtime#currently-available-runtime-versions.
May I ask why both of them have this value?
Best pratice would be to set both of them to the minimal version of Business Central that the App need to work.

I've resolved this error in the (pre)release version of v0.30.12 of the LinterCop,

The v0.30.13 version of the LinterCop is now released, where I believe this issue is now resolved. If you still encounter issues, feel free to reopen this issue (or create new one).


Security issue: bind <IP_ADDRESS> only for unsecured database

Hello,
I install ClickHouse on the debian server, and play around according the guides. Thanks for the great product! I find ClickHouse server bind to <IP_ADDRESS> by default, and all databases are unsecured,
$ netstat -nao | grep 8123
tcp6       0      0 :::8123                 :::*                    LISTEN      off (0.00/0/0)

Which means everyone can scan 8123 port and see the data using ClickHouse http interface. There are a lot of reports on hijacking unsecured mongoDB.
So I strongly suggest to bind <IP_ADDRESS> IP address only for the unsecured database (like mysql server). Make sense?

Hello.
Yes, that makes sense.
Probably we should bind to <IP_ADDRESS> (::1) by default and highlight it in documentation and tutorials.
Or we could still bind to wildcard address, but don't allow connections without password from everywhere except <IP_ADDRESS>.
(The only difficulty is that users frequently create clustered setup, that use user default without password for internal communications between servers.)
We will choose some solution.

problem:
::1 does not listen <IP_ADDRESS>
need split <listen_host> to allow listen multiple addresses

also all tests with <IP_ADDRESS> <IP_ADDRESS> will fail

done.
https://github.com/yandex/ClickHouse/commit/107fb86a40e39deaaa2a00d32fe2f0af6595348b

@alexey-milovidov, is there a way to have default with password for internal cluster communication?

It is possible to either:

specify password for default user;
use non-default user for inter cluster communication.

In both cases, you just need to specify user and password (near host and port) in definitions of your clusters.


[FLOC-2285] Postgres tests improvements

Already merged into master. Don't actually expect these tests to pass - that will be in follow up PR #1606.

Looks fine and I verified that this helps the postgres acceptance tests pass on Rackspace and AWS in #1606.
Please merge.


Revert the new non_max_suppression_v2 op for the 1.2 release so we can do API review

@martinwicke fyi

Jenkins, test this please

Jenkins, test this please


Reduce rerendering with mobx and add e2e cases

Codecov Report

Merging #14 (73c4ea9) into main (18feff1) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #14   +/-   ##
=======================================
  Coverage   99.64%   99.65%
=======================================
  Files          18       18
  Lines         281      289    +8
  Branches       30       32    +2
=======================================
+ Hits          280      288    +8
  Misses          1        1

Flag
Coverage Δ

root
99.65% <100.00%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

src/mobx/spreadHelpers.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Action for configuring a plist inside an ipa/xcarchive

New Issue Checklist

[x] Updated fastlane to the latest version
[x] I have read the Contribution Guidelines

Issue Description
🔑 I'm doing a blog series on creating a "build-once" deployment pipeline using fastlane. As part of this process, the signing of the IPA happens at deployment time, at the moment the configuration is known, rather than at build.
As part of this, I've written a custom action that might fit well with the main predict. It's code purpose is to set a number of keys in a list that's inside an IPA file. It also supports:

explicit options for the plist version keys
a list of plist buddy commands, rather than the more restrictive (but easier to look at) hash of value
replacing the icon with a .iconset

I'm also thinking of added support for xcarchives in addition to ipas, since people might prefer their single build to work like that.
Would that be of interest and, if so, any thought on name? My current thoughts are:

tempest - because it changes the environment
faceoff - because it's swapping out the configuration
facelift - as above, but slightly less trademark infring-y

Thoughts?

It's not, mr fastlane bot, at least not in any way that's relevant to your linka.

Don't forget there is already a liftoff 👍
Do you plan on providing this as a fastlane plugin?

I know about liftoff, but I'm not sure there's any similarities between it and what I'm proposing.
Yes, that's the plan, unless you can see it being useful in the main product.

@KrauseFx Is there an official way of distributing custom fastlane actions. For example, via gems?

@richardszalay Yes, there is, check out the Plugins guide

Gotcha. Ok, I've run through the documentation and the plugin infrastructure code and I see how it all fits together.
I'll put together the action as a plugin and update this issue when it's ready.

That's awesome, thanks @richardszalay. Let us know if you need any help

Done: https://rubygems.org/gems/fastlane-plugin-facelift
Includes all the trimmings (tests, working example fastfile)

@richardszalay Awesome! Really cool stuff here!

I've finally published my blog series, which might give some context as to why I wanted to do this - https://blog.richardszalay.com/2016/08/22/ios-deploy-pipeline-1-introduction/


Expose UI for target group in Mark Applications

To support brushing and linking (e.g. apply an interval selection to color on a different group in multi-view visualization) we added a targetGroupName field to the MarkApplication definition. This field is analogous to targetMarkName but we currently don't have UI to set it.
Current UI (notice Filter Group 2, Color Group 2, etc):

Current UI when there's multiple marks in the group:

Proposed changes:

add a dropdown for target group analogous to the one for target mark. it should only be shown when there are multiple groups
Remove the extra interaction previews for Color Group 2, etc
Change the Filter Group 2 preview to just Filter and use the target group dropdown to set the preview

Relevant code:

see getTargetMarkOptions in Interaction.tsx to get started

closed by https://github.com/vega/lyra/pull/521


vagrant.config.yml values ignored

Issue Type

Bug Report / Support Request

Your Environment
Vagrant 1.9.4
VirtualBox 5.1.18r114002
ansible <IP_ADDRESS>

Your OS

macOS (version)

Summary
My build process adheres to the Composer method described in Soup-to-nuts:

composer create-project drupal-composer/drupal-project:7.x-dev drupal --stability dev --no-interaction

then into the vendor directory:

composer require --dev geerlingguy/drupal-vm

and finally customizing vagrant.config.yml, e.g.:

vagrant_hostname: local.something.test
vagrant_machine_name: local-something
vagrant_ip: <IP_ADDRESS>
ssh_home: "/var/www/drupal"

The playbook doesn't respect my vagrant.config.yml values. It builds the machine as follows:

Bringing machine 'drupalvm' up with 'virtualbox' provider...
==> drupalvm: Cloning VM...
==> drupalvm: Matching MAC address for NAT networking...
==> drupalvm: Checking if box 'geerlingguy/ubuntu1604' is up to date...
==> drupalvm: Setting the name of the VM: drupalvm.test

Where drupalvm.test should be local.something.test.
Not seeing anything relevant in the commit history. Ideas?

Huh. Jumped the gun. I blew away my Ansible roles and started from scratch and that seems to have fixed it. Leaving this open until I can replicate again in the morning.

As a suggestion:
Use the vagrant.config.yml only if you actually need to override something for vagrant. Generic configuration settings should go to your own config.yml file.


⚠ Bofrost.be has degraded performance

In 54c6c5e, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1164 ms

Resolved: Bofrost.be performance has improved in 61ffbdf.


azure-cli-ml errors with python 3.10

This is autogenerated. Please review and update as needed.
Describe the bug
When running az ml dataset register, get an import error. The same command works successfully under python 3.8.
Command Name
az ml dataset register Extension Name: azure-cli-ml. Version: 1.41.0.
Errors:
cannot import name 'Iterator' from 'collections'<EMAIL_ADDRESS>

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az ml dataset register --skip-validation --workspace {} --resource-group {} --subscription {} --file {} --output-metadata-file {}

Expected Behavior
For the command to run without any errors.
Environment Summary
macOS-12.4-x86_64-i386-64bit, Darwin 21.5.0
Python 3.10.4
Installer: HOMEBREW

azure-cli 2.37.0

Extensions:
azure-cli-ml 1.41.0
azure-devops 0.25.0

Dependencies:
msal 1.18.0b1
azure-mgmt-resource 21.1.0b1

Additional Context

Using this workaround: https://github.com/Azure/azure-cli/issues/21377#issuecomment-1057851461


Segfault

Program received signal SIGSEGV, Segmentation fault. [Switching to Thread 0x7ffff701a700 (LWP 627)] 0x00000000005c5429 in github.com/zhangpeihao/log.(*Logger).ModulePrintf (logger=0x0, module=0x0, level=4, format=..., v=...)     at /home/president/go/src/github.com/zhangpeihao/log/log.go:335 335
Output from gdb, trying to connect.
runtime error: invalid memory address or nil pointer dereference
Output from execution.

You should set the logger before use this lib:
l := log.NewLogger(".", "server", nil, 60, 3600*24, true)
l.SetMainLevel(log.LOG_LEVEL_DEBUG)
rtmp.InitLogger(l)

I added the default logger to fix this bug.  You can update the codes from github.

Alright, thanks.
I'm fairly new to Go so i'm still learning.


Refactor renderTimelineEvents to return HTMLElements instead of a string

It's significantly easier to handle the comment edit UX with named HTML elements instead of strings. This is the first part of the work to have edit/delete support on comments there, just refactoring to use document.createElement

LGTM!


Update and re-organise documentation

This PR updates and re-organises the package documentation.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (70d4834) 98.17% compared to head (c687a07) 98.17%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #61   +/-   ##
=======================================
  Coverage   98.17%   98.17%
=======================================
  Files          61       61
  Lines        3296     3296
=======================================
  Hits         3236     3236
  Misses         60       60

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


support optional annotated field with default value

Currently, it is not possible to specify a SQLModel with an Optional complex type, such as a Decimal, as follows:
from decimal import Decimal
from typing import Annotated
from sqlmodel import Field, SQLModel

class PriceModel(SQLModel, table=True):
    id: int | None = Field(primary_key=True, default=None)
    price: Annotated[Decimal, Field(decimal_places=2, max_digits=9)] | None = None

Doing so results in the following error (see #67, #312) in get_sqlalchemy_type:
TypeError: issubclass() arg 1 must be a class

This PR attempts to fix this issue by adding a check in get_sqlalchemy_type for whether the type of the Field is  a typing_extensions.AnnotatedAlias. If it is, instead of using the class of the AnnotatedAlias for the following comparisons (which results in the above error), it uses AnnotatedAlias.__origin__. Similarly, it infers the metadata from AnnotatedAlias.__metadata__ instead of the AnnotatedAlias itself.
In my testing, this approach seems to work well. I also added a simple test here that checks whether restrictions on an optional annotated field are enforced on the database side. It could probably be improved or extended.

looks like some (many) tests are failing, I'll try to fix them!
Is this feature something that should work with Pydantic v1 too? I'm not too familiar with it.

I've just hit exactly this problem; is there any chance you could update your MR please @proever?


Need standard for denoting fixed fields in settings.json

While working with the water mass transformation POD in cmec-driver, I ran into an issue with generating the file name for "areacello". Since this is a fixed field, it does not have a "frequency" in settings.jsonc. However, in the MDTF file structure all input variable file names, including those for fixed fields, include a frequency.
I found an existing example of one usage of "areacello" (ocean grid cell area) in the MDTF Tropical Pacific Sea Level POD. In this case, the frequency for all variables is "mon", set under the data/frequency keys.
However, the mixed layer depth POD shows another method of recording frequency which is to provide it for each variable separately. In this case, is it still reasonable to use "mon" for areacello even though it is a fixed variable?
It would be helpful to add language to the MDTF/CMEC standards for the "frequency" of fixed fields.

The MDTF Diagnostics packages uses "static" for their fixed field variables. Closing.


Adding validity checks to bin_io

There are some invariants about data structures that should be checked on deserialization.
Off the top of my mind

Well-formedness of proofs, whether the curve points are on the curves/subgroups
The length of the Checkpoints list in the postake state should be at most some constant.

#722 also calls for checking (eg) that Sok_digest.t has length 32