Termination and spaces

In the present version, colouring of the different fields (name, loglevel, hostname, etc.) seems to be dependent on placement of white spaces. I.e. if no white spaces are placed between fields, consecutive fields will have the colour of the first in string. Is that intended behaviour? It can be seen using the following code:
import coloredlogs, logging
from logging import debug, info, error, warning as warn
coloredlogs.install(level='DEBUG', fmt=formatstring, datefmt='%d.%m.%y %H:%M' )
debug("debug")
info("info")
warn("warn")
error("error")

where two different format strings are used (note the spaces around levelanme, name and process string formatters):
formatstring='[ %(levelname)-8s ] (%(asctime)s %(name)s @ %(hostname)s [%(process)d]'   # with spaces
formatstring='[%(levelname)-8s] (%(asctime)s %(name)s@%(hostname)s[%(process)d]'       # without spaces

The result is something like this (note the color of the levelname and the blurred hostname field):

Yes this is intended behavior. The current approach is documented under the colorize_format() method. Basically I decided to implement it like this (the "simple" approach) until someone came up with a use case not covered by the simple approach. And here you are! :-)
So consider this a documented implementation detail. If we can find a way to improve this that doesn't break backwards compatibility (at least not in a surprising way) I'm open to changing the implementation.

An advantage of the current simple approach (from my perspective) is that it's possible to highlight things like %(name)s[%(process)d] as a single logical unit with a single color that encompasses the [ and ] characters. It also helps that coloredlogs doesn't need to fully understand and (properly) parse the format of logging format strings.
Making the current behavior more flexible introduces more decisions to make. For example if the format string contains %(name)s[%(process)d] and styling has been specified for both name and process, what should the color of the [ and ] tokens be? Just leave them at the default color? That looks kind of weird, but maybe that's just me :-).
The following approach might work:

We split the format string on whitespace resulting in a list of whitespace separated tokens.
For each whitespace separated token the following decision process is followed:

If the token contains only one %(...) field with a color/style defined then the complete token is highlighted in that color/style (this provides compatibility with the current behavior).
If the token contains multiple %(...) fields with a color/style defined then each %(...) field is highlighted according to the configured color/style but any text in between is left as is.

This change in behavior requires real/proper parsing of logging format strings. This isn't impossible to do of course, but it might be a bit tricky to get it right :-). So one the one hand this complicates the library but on the other hand it can guarantee correct behavior (provided that the format string parsing and rewriting is correct of course). In contrast the current approach is kind of dumb but works fine most of the time (it's possible to break it though, I guess).

Sounds good. Another approach would be to introduce a reset string, that ends the current colouring, or colour parenthesis strings respectively.

Hi again @con-f-use and sorry for the long silence here. I've just released coloredlogs 10.0 which should fix the issue you reported: coloredlogs is now finally able to properly distinguish formatting directives from their surrounding text, dropping the dependency on whitespace delimiters.

I'm still seeing this issue with coloredlogs 15.0.1. With fmt="{asctime} [{levelname[0]}] {name}: {message}", the trailing colon after the name field is colored as well. If I change it to fmt="{asctime} [{levelname[0]}] {name} : {message}", the colon is not colored.


feat(modelgen): add timestamp fields createdAt & updatedAt for @model

Issue #, if available:
fix #112
Description of changes:
Add timestamps fields createdAt and updatedAt as read-only fields in codegen output for graphql types with @model. This is achieved by adding a new metadata isReadOnly: boolean in CodeGenField. This change is applied to all platforms except for flutter.
The feature will be released behind a feature flag of CLI. Refer https://github.com/aws-amplify/amplify-cli/pull/7273
export type CodeGenField = TypeInfo & {
  name: string;
  directives: CodeGenDirectives;
  connectionInfo?: CodeGenFieldConnection;
  isReadOnly?: boolean // ---> new metadata
};

Related PR from libraries:
Android : https://github.com/aws-amplify/amplify-android/pull/1249
iOS: https://github.com/aws-amplify/amplify-ios/pull/1133
Implicit schema
Since they are read-only and auto-generated, we suggest to use implicit schema. The timestamp fields are automatically added to modelgen output as an optional type.

Simple @model

type Post @model {
  id: ID!
}

Use timestamps field in @model for implicit schema
You can also use timestamps field in @model to change the default timestamp field name. This will add the desired name as optional AWSDateTime fields.

type Post @model(timestamps:{createdAt: "createdOn", updatedAt:"updatedOn"} {
  id: ID!
}

Explicit schema (read-only disabled)
You can also include createdAt, updatedAt  in the schema field explicitly. However, the timestamp field will be no longer read-only.
type Post @model {
  id: ID!
  createdAt: AWSDateTime
  updatedAt: AWSDateTime
}

This behavior conforms to the fact that graphql transformer will include timestamp fields in input variables in explict schema.
How are these changes tested:
yarn test
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

overall this LGTM my only concern is that these fields shouldn't be in the constructor? my understanding is that when you use this directive, app sync creates and updates these fields for you and we just want them to be generated on the front end to read from, not set. @undefobj or @renebrandel  can you provide some color here possibly?

overall this LGTM my only concern is that these fields shouldn't be in the constructor? my understanding is that when you use this directive, app sync creates and updates these fields for you and we just want them to be generated on the front end to read from, not set. @undefobj or @renebrandel  can you provide some color here possibly?

Correct they're set in the GraphQL resolver code not in the client mutations code

@AaronZyLee per the above confirmation, please remove the generation of these fields in the constructors for each platform, they should be props on the structs or class but not in the constructor.

@kneekey23 @undefobj I agree partially on removing the timestamp fields in constructor. IMO it is dependent on the schema definition. I read the official doc and previous PR https://github.com/aws-amplify/amplify-cli/pull/4149 and find out that there is a back door for users to manipulate the timestamp value if they include them explicitly in the schema. I would suggest that to remove the fields in implicit cases while keep them in explicit cases.

I agree with that @AaronZyLee , I think that is what I meant actually. In the case of it being implicit, it should not be in the constructor.

Codecov Report

Merging #114 (7a2549e) into master (b878956) will increase coverage by 0.04%.
The diff coverage is 97.87%.

@@            Coverage Diff             @@
##           master     #114      +/-   ##
==========================================
+ Coverage   84.41%   84.46%   +0.04%
==========================================
  Files         139      139
  Lines        5410     5433      +23
  Branches     1122     1132      +10
==========================================
+ Hits         4567     4589      +22
- Misses        726      727       +1
  Partials      117      117

Impacted Files
Coverage Δ

...en-plugin/src/languages/swift-declaration-block.ts
85.71% <ø> (ø)

...odelgen-plugin/src/scalars/supported-directives.ts
100.00% <ø> (ø)

...nc-modelgen-plugin/src/visitors/appsync-visitor.ts
95.53% <92.85%> (-0.26%)
:arrow_down:

...delgen-plugin/src/visitors/appsync-java-visitor.ts
92.41% <100.00%> (ø)

...elgen-plugin/src/visitors/appsync-swift-visitor.ts
97.66% <100.00%> (+0.14%)
:arrow_up:

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 b878956...7a2549e. Read the comment docs.

Any reason this wasn't done with flutter? Currently need access to createdAt


[Event Request] Page 55 "Purch. Invoice Subform" - Field "Description" - trigger OnValidate()

                field(Description; Description)
                {
                    ApplicationArea = Basic, Suite;
                    Caption = 'Description/Comment';
                    ShowMandatory = Type <> Type::" ";
                    ToolTip = 'Specifies a description of the entry of the product to be purchased. To add a non-transactional text line, fill in the Description field only.';

                    trigger OnValidate()
                    begin

----------------------- ADD EVENT ----------------------
                IsHandled := false;
                OnBeforeOnValidateFieldDescription(Rec, IsHandled);
                if IsHandled then
                    exit;
----------------------------------------------------------

                        UpdateEditableOnRow();

                        if "No." = xRec."No." then
                            exit;

                        ShowShortcutDimCode(ShortcutDimCode);
                        NoOnAfterValidate();

                        UpdateTypeText();
                        DeltaUpdateTotals();
                    end;
                }

...............

[IntegrationEvent(false, false)]
local procedure OnBeforeOnValidateFieldDescription(var PurchaseLine: Record "Purchase Line"; var IsHandled : Boolean)
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


feat: replace preHandler hook with onRequest to define ip property

👋
I modified the hook so that the IP is set in the first phase of the request transformation. This follows the PR, and I am proposing this change because I have a case where I need to log the IP in an onRequest hook (mb, i'm sorry😓).

👌thanks


Additional localisation work

This is some leftovers from #1344:

[ ] PR #1445

Translate text of error messages (some will need simplification)
As there are planned changes to de/serialization this is paused for now

[ ] PR #1441

Lots of overlap with other despawn group (ext vehicle types)
The UI is confusing - when despawn is disabled, only then can you choose what is allowed to despawn heh
We should consider something less confusing

[ ] PR #1284

Mostly done: https://github.com/CitiesSkylinesMods/TMPE/issues/1288
Remaining: Localise lozenges New, Updated, Fixed, Mod, Removed etc

when despawn is disabled, only then can you choose what is allowed to despawn heh

It makes no sense to change anything if the feature is inactive - would be even more confusing 😂

Yeha, I know. It just seems really weird the way it currently works. Like, it makes sense, but only after you work out what it's doing and even then it still feels a bit weird. I've not thought of anything better for now, but it's nagging me.

Will you be adding support for new langs (Vietnamese?) I didn't notice that in PR (maybe missed 🤷♂ )?

Will you be adding support for new langs (Vietnamese?) I didn't notice that in PR (maybe missed 🤷♂ )?

I can do - what needs editing?

Probably dropdown only (new option)
Shit.. I forgot to update scheme files... Will do later today so it will export all langs and they will be available to use in game


Modify SetColumnRemarks support for mysql. The original code does not support mysql modification notes

It turns out that the sql statement generated by setColumnRemarks of the mysql database is ALTER TABLE cat.person COMMENT = 'A String';
Now modify it to ALTER TABLE cat.person MODIFY COLUMN id int COMMENT 'A String';
┆Issue is synchronized with this Jira Bug by Unito
┆Fix Versions: Community 4.x

Thanks for the pull request. Can you please edit it so that all the comments are in English?
We will need to review this because of the change to the xsd and decide when it can be included.

Thanks for the update!

Hi @zyzyzy523 ,
Thanks for your PR submission.  Its been awhile since we heard from you.  Did you need any help or further guidance around the changes @nvoxland requested?
Thanks!
Ronak

@zyzyzy523 I will move this into Conditioning but size it RiskMed.  Eng time will be needed to make the change requested by @nvoxland.  Thanks for including tests and translations!

What the "modification notes" that the old version did not support?
The other problem we'll have is that the new code requires dataType, which will cause everyone with existing changeSets on mysql that didn't need to specify the dataType before to fail.
We'll need to either use one SQL or the other depending on if dataType is set, or find a way to support the modification notes without requiring dataType.

I've just faced it on 3.8.9
Is there plans to backport the fix to 3.x ?

@php-coder, we are concentrating on 3.10.1 and 4 currently, and operational patch fixes to the existing branches is being hashed out. Hang tight and we'll update this PR shortly.

I merged this into master after moving the new xsd attributes to 4.3.xsd.
I did take out the new defaultValue setting. I know setRemarks is going to cause that to be lost, but lots of other things get lots too (primary keys, constraints, etc.). It's a general problem with mysql we need to find a better solution to. But the defaultValue attribute is just a part-there answer so at least trying to be consistent in our handling of it.
I also cherry picked the changes to 4.3.x for the 4.3.0 release.

I merged this into master after moving the new xsd attributes to 4.3.xsd.
I did take out the new defaultValue setting. I know setRemarks is going to cause that to be lost, but lots of other things get lots too (primary keys, constraints, etc.). It's a general problem with mysql we need to find a better solution to. But the defaultValue attribute is just a part-there answer so at least trying to be consistent in our handling of it.
I also cherry picked the changes to 4.3.x for the 4.3.0 release.


ansible:latest - 404 Error

Expected behavior
Expected to successfully download the ansible:latest image
Actual behavior
(HTTP code 404) unexpected - {"message":"manifest for ansible/ansible:latest not found"}
Information about the Issue
Unable to download the ansible image
Steps to reproduce the behavior

... Download the image from Docker Hub via Kitematic

Please see: https://github.com/docker/kitematic/issues/2626#issuecomment-302021541


Request for documentation - how backup and mobile/laptop workflow

I am reading the features page and so far I love gopass. thank you for working on such an important project!
There are two parts of documentation that I believe are missing:

How to backup things (data store, gpg keys, or anything else that is needed)
How to setup a workflow for both a laptop and a phone. I understand that only android is supported at the moment and it's being done using this android app and also with github repo (or maybe git is enough) but it is not clear what are the steps to do that

Can someone share the two points above? having this written down somewhere would be very valuable and will increase the adoption of gopass especially among less technical users.

Pass for IOS works perfectly with a single store/repo.


mv: observation multiple PIs produce dupes

rick added condition on role, to avoid dupes in full extract.  here we may want to propagate fix to incremental extract

addressed with #219


[BUG]: Dialog body padding

Issue description
Dialog body doesn't have stable padding at bottom.
Media & Screenshots

Baklava Version
2.1.0-beta.6
Operating system
No response
Priority this issue should have
Low (slightly annoying)
Please review the checkboxes that are applicable.

[ ] This issue is directly related to React

Actually might be top too, but the paddings doesn't seem to work.

We have an issue like #624 that maybe covers your concern.


[FIX-frontend] Change Loading Animation after Onboarding to a Consistent Style

Description:
After a new user completes the onboarding process, a loading animation appears before they are redirected to the homepage. This animation is inconsistent with the other loading animations used throughout the platform. We need to replace this loading animation to maintain visual consistency across the app.
Acceptance Criteria:

Replace the loading animation that appears after onboarding with the same loading animation used elsewhere in the platform.
The animation should match the style, duration, and branding of other loading animations.
The user should be redirected to the homepage once the animation finishes.

Steps to Reproduce:

Sign up as a new user.
Complete the onboarding process.
Observe the loading animation before being redirected to the homepage.

Proposed Solution:

Use the existing consistent animation already implemented elsewhere in the platform. Ensure that the transition after the onboarding and the timing of the animation are smooth.

Files Involved:
Possibly in Onboarding folder or layout file
Difficulty: Beginner
Do refer README & CONTRIBUTING guides.

Hi, I would love to work on this issue... can you assign it to me?

Hi, I would love to work on this issue... can you assign it to me?

Cool Beans 👍, go ahead.

i also want to work on this issue please assign me

i also want to work on this issue please assign me

Cool assigned it to you. Go ahead.
Go through Contributing.md and read setup
use mock account keys given in readme.md (as it is a frontend issue)
Let us know if you get stuck.

Just a heads up the start command would be bun start:all not bun start if someone can raise a issue and fix the readme it would be great.


Maintainer(s) wanted

Dear Gotenberg Community,
I wanted to take a moment to share some news with you about the direction of Gotenberg. Lately, you may have noticed that the pace of updates and development has slowed down a bit. This is due to the fact that my available time to devote to Gotenberg has decreased.
However, this does not spell the end for Gotenberg. Rather, I see it as an opportunity for the project to evolve and welcome fresh input. Therefore, I am inviting anyone interested, or groups, who are willing to assist in the ongoing maintenance and growth of Gotenberg.
Your role would involve contributing new ideas, developing additional features, and ensuring the smooth running of Gotenberg, keeping this tool reliable and user-friendly for all of us.
Please understand, this isn't a goodbye, but a shift towards a new chapter. I'm hopeful this change will allow Gotenberg to continue to serve its users well, and I'm excited to see how it develops with new stewardship.
Thank you for your understanding and continued support of Gotenberg.
Best regards,
Julien Neuhart
PS: I want to assure you that I'll continue to maintain the project in the meantime.

Closing this in favor of #710.


[FLINK-27438][table-planner] Fix the constructing a map array. Backport 1.15

What is the purpose of the change
This is the backport of https://github.com/apache/flink/pull/19648 without any additional changes.
This pull request fixes the ValidationException appears while trying to construct the array of maps. The bug appeared after adding explicit casting into SqlArrayConstructor in https://github.com/apache/flink/commit/0f51995bc156435eb9e97d2ac15e0225117877e2.
Brief change log

add cast to map sql data type functionality into SqlValidatorUtils
add tests for plan verification inside batch and stream test sets

Verifying this change

Extended unit tests for Calc exec plan verification

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no

@flinkbot run azure


Remove Mockito

This PR removes Mockito from test suites of the project.
Also, the Mockito dependency is no longer exposed as an API by the testlib. The Mockito-based utility  Spy is removed.
Spine version advances to 1.1.1-SNAPSHOT+1.

@armiol PTAL.

@armiol PTAL again.


Handle interrupts preserve exception API

This is just one possible approach to allow @CRogers's "Cancel requests when thread is interrupted" PR (#587) to maintain http-remoting's current API.

I'm gonna merge this into @crogers' branch so that we have one place to iterate and can address all comments there!


Use old version of cocoapods-downloader where necessary

Motivation:
Ubuntu 16 ruby is too old to support latest cocoapods gem.
Modifications:
Try installing latest cocoapods-downloader first,
if that fails install and old one.
Result:
If the latest version fails, we'll continue with an old version.

I think this is going to affect all our CI (I know it affects some others already, I've seen it hit). So we do need a solution.
Updating Ruby is not going to work because we are already installing latest. We could instead flip over to using rvm, I suppose: that may be the better choice.


Refactor 1.x content (or remove it)

While working on #9268, I noticed some problems with the 1.x versioning.
The versioning that appears for 1.x points to a 2.0 sample, so it's incorrect in the topic. See this 1.1 version ...
https://docs.microsoft.com/aspnet/core/tutorials/getting-started-with-swashbuckle?view=aspnetcore-1.1&tabs=visual-studio%2Cvisual-studio-xml#xml-comments
... scroll down a bit, and you'll see a CreateDefaultBuilder in there.
It also looks like the 1.x links to the sample provide a 2.0 project file.
cc: @scottaddie

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

ID: 14fce65f-1337-58cb-2193-8d8e0e62792d
Version Independent ID: 59222460-a971-e76e-4fe9-23d4d7d768ee
Content: Get started with Swashbuckle and ASP.NET Core
Content Source: aspnetcore/tutorials/getting-started-with-swashbuckle.md
Product: aspnet-core
GitHub Login: @zuckerthoben
Microsoft Alias: scaddie

@guardrex You're correct that the Program.cs sample displayed for 1.1 is incorrect.

I would've fixed it on the spot, but I think the project file snippet situation needs work, too. Best to work it separately from the other issue, which has priority bug fixes that should go in quickly-ish.


How to use a library after installation

After using vcpkg install libxxx successfully, the method of how to use will be output under the CMD, but later I forget that I want to find out how to use libxxx 。
such as  When I install a leveldb successfully，After success, he will say
 The package leveldb:x64-windows provides CMake targets:
 find_package(leveldb CONFIG REQUIRED)
 target_link_libraries(main PRIVATE leveldb::leveldb)

I'll use the installed package later, but I forget about it

See documentation.

ok thanks


0.4.0 Release

This list is a work in progress

[ ] Refactor List component
[ ] Implement GridView component
[ ] implement Floating Action Button (FAB)
[ ] Swipe Views with Tabs (spec). Create a wrapper around react-native-scrollable-tab-view - maybe?
[x] Animated IconToggle badge changes.

Anything more planned for 0.4.0? I guess you are handling the List Refactor.

The GridComponent? Also I need to debug a Ripple bug where the state persists. Any other ideas I'm open to!

What about a FAB?

Ah yeah. How should the position of it be done? Simply render the FAB and
let the user decide where to place it?
On 5 Jan 2016 9:00 p.m., "David Gil"<EMAIL_ADDRESS>wrote:

What about a FAB?
—
Reply to this email directly or view it on GitHub
https://github.com/react-native-material-design/react-native-material-design/issues/12#issuecomment-169132655
.

Hmm not really sure. Specs about fabs say how they should be positioned
with relation to other objects, and they seem to be always bottom-right ...
Maybe we can style them lime that by default and let the user override?
El mar, 5 de enero de 2016 22:02, Elliot Hesp<EMAIL_ADDRESS>escribió:

Ah yeah. How should the position of it be done? Simply render the FAB and
let the user decide where to place it?
On 5 Jan 2016 9:00 p.m., "David Gil"<EMAIL_ADDRESS>wrote:

What about a FAB?
—
Reply to this email directly or view it on GitHub
<
https://github.com/react-native-material-design/react-native-material-design/issues/12#issuecomment-169132655
.

—
Reply to this email directly or view it on GitHub
https://github.com/react-native-material-design/react-native-material-design/issues/12#issuecomment-169133138
.

What about a Bottom Sheet?

Yes. The FAB has to be at the bottom-right according to the spec.
Though FAB comes in handy, but, personally I somewhat like this article - Why the Floating Action Button is bad UX design
@Ehesp I can have a look at FAB, Gridview.

With respect to Grid View, I will be implementing this first (from the spec)-

@pewh problem with a bottom sheet is that currently there's no way to force a component to the top of everything else (I believe).

@pewh @Ehesp There is a way to do that. But, not sure its available on Android yet. We can tweak concepts and code from the Modal and implement it I believe. I have seen such an implementation somewhere. Not sure how good or feasible it is.

@bozzmob Okay cool - Shall look into it.
What does everyone think of 'Swipe Views with Tabs'? I've got it working on another project using the project mentioned in the list, but had to create a custom tab bar to make it feel native...?

Badge animations done!

@Ehesp Yeah! There is a ViewPager component which exists officially. We can make use of react-native-scrollable-tab-view as you have suggested. But, some css changes are needed so that the look and feel is similar to the existing components.

Nice work @Salakar, really cool.
@bozzmob I'll do that then as I've already got it on another project. react-native-scrollable-tab-view doesn't allow a single tab too, but I've got a way around that.

@Ehesp How about we use react-native-action-button for FAB?

That looks good! Will look if there's any benefit of pulling it in and making a wrapper around it. If not, it might just be best to suggest this for a FAB.

A dropdown menu component could be awesome!

@zoontek I agree, however part of the problem we have (Bottom Sheets has this issue too), is that React Native does not yet support an "on the top of everything" API. So the dropdown menu might get stuck behind a component which is nested deeper down the chain.

I am working on the Gridview component now. Have made some progress. Still a lot to do.

Screenshots of Gridview-
Currently, its working for a fixed height, width.
I am working on making it a generic component. Currently most values like margin, padding are being hard-coded. So, thought of making it responsive.

@bozzmob any updates on the grid view?

@Salakar Yes. I am working on it. It should be available in a couple of days.

The dropdown is available in React-Native 0.19-rc: https://github.com/facebook/react-native/commit/18437093f2664beedd3239bb525d7daa3f2bcdbd :+1:

Awesome :D
I just wish there was some "z-index" type style so we could push whatever we wanted to the top of the stack.

@Ehesp Yes, or a custom component "Above"
<Above
  backgroundVisible={true}
  backgroundColor="#000"
  backgroundOpacity={0.5}
>
  <View>
    <Text>Could be really usefull!</Text>
  </View>
</Above>

Is the project dead? :(

Nope, was using it in a internal work project however that's not currently being developed so focus has shifted. Plan on picking it up in the next few weeks hopefully.

Great! I will check if I can do the Swipe Views with Tabs!

@Ehesp Any chance to integrate with https://github.com/xinthink/react-native-material-kit?

I would like to have a proper dialog (in future releases), example:
https://github.com/aakashns/react-native-dialogs


Implement e2e dualStack control plane tests

This PR includes the necessary support

to deploy dualStack/IPv6 KIND clusters
to deploy the supported Istio versions in a dualStack mode

and verifies that Istiod is properly configured to support dualStack use-cases.
To run e2e dualStack tests, use the following command.
$ export IP_FAMILY=dual
$ make test.e2e.kind
Related to: https://github.com/istio-ecosystem/sail-operator/issues/372
Signed-off-by: Sridhar Gaddam<EMAIL_ADDRESS>

@sridhargaddam: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_sail-operator_main
50789da08bc6c889ee8e29eb01d275e676e74c5e
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@sridhargaddam: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_sail-operator_main
50789da08bc6c889ee8e29eb01d275e676e74c5e
link
true
/test lint

e2e-kind-multicluster_sail-operator_main
50789da08bc6c889ee8e29eb01d275e676e74c5e
link
true
/test e2e-kind-multicluster

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@sridhargaddam: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_sail-operator_main
50789da08bc6c889ee8e29eb01d275e676e74c5e
link
true
/test lint

e2e-kind-multicluster_sail-operator_main
50789da08bc6c889ee8e29eb01d275e676e74c5e
link
true
/test e2e-kind-multicluster

e2e-kind_sail-operator_main
50789da08bc6c889ee8e29eb01d275e676e74c5e
link
true
/test e2e-kind

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@sridhargaddam: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_sail-operator_main
50789da08bc6c889ee8e29eb01d275e676e74c5e
link
true
/test lint

e2e-kind-multicluster_sail-operator_main
50789da08bc6c889ee8e29eb01d275e676e74c5e
link
true
/test e2e-kind-multicluster

e2e-kind_sail-operator_main
50789da08bc6c889ee8e29eb01d275e676e74c5e
link
true
/test e2e-kind

e2e-kind-olm_sail-operator_main
50789da08bc6c889ee8e29eb01d275e676e74c5e
link
true
/test e2e-kind-olm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

I put some comments. But checking deeply I'm think if worth to have a complete separated test for dualstack? Checking the entire test run the steps are the same one than the regular control plane installation besides two or three differents steps, so, maybe worth to add those steps inside the current test and skip those steps when dualstack is not being set? wdyt?

Thanks @fjglira for reviewing. Yes, currently this looks very similar to the regular control_plane tests (with some minor differences), but I'm also working on adding data-path tests to this test-suite where we try to validate datapath use-cases like a client is able to connect to IPv4/IPv6/dualStack services. So, having a separate dualStack testsuite would be readable and will allow adding any dualStack specific tests without cluttering the existing test suite. Also, as I mentioned in my comment, we want the existing control_plane suite to be validated (i.e., Istio deployed with defaults) along with dualStack suite (where Istio will be deployed in dualStack mode).

I put some comments. But checking deeply I'm think if worth to have a complete separated test for dualstack? Checking the entire test run the steps are the same one than the regular control plane installation besides two or three differents steps, so, maybe worth to add those steps inside the current test and skip those steps when dualstack is not being set? wdyt?

Thanks @fjglira for reviewing. Yes, currently this looks very similar to the regular control_plane tests (with some minor differences), but I'm also working on adding data-path tests to this test-suite where we try to validate datapath use-cases like a client is able to connect to IPv4/IPv6/dualStack services. So, having a separate dualStack testsuite would be readable and will allow adding any dualStack specific tests without cluttering the existing test suite. Also, as I mentioned in my comment, we want the existing control_plane suite to be validated (i.e., Istio deployed with defaults) along with dualStack suite (where Istio will be deployed in dualStack mode).

Ok, this make a lot of sense also. So, ok for me then


add retry logic and tests to experian fetch token endpoint

BACKEND PULL REQUEST
Related Issue
Fixes #3893
Changes Proposed

Add retry logic in LiveExperianService.fetchToken to retry twice on 500 errors

Additional Information

Because we're using the DemoExperianService in lower envs, we don't have a good way of smoke testing this. What @emmastephenson and I decided on was to temporarily stick some log statements into Azure to grab the shape of the 500 error when it occurs and then use that in a future PR to catch on a more informed error object than the current catch-all 500 check.

Testing

See the tests added in LiveExperianServiceTest and validate the mocking setup covers the necessary bases.

Checklist for Primary Reviewer

[x] Changes comply with the SimpleReport Style Guide

Thanks for the callout @mehansen. Made a ticket here


java.lang.Double cannot be cast to com.facebook.react.bridge.ReadableMap

Describe the bug
When I try to use any of the Charts available in the package I get the following issue. Tested on multiple devices, also with a newly init App. However, weirdly when I copied the code for the charts from source it works without any errors.
To Reproduce
Steps to reproduce the behavior:

Copy Example App
See error

Screenshots

Device tested on: [OnePlus Nord 2, ROG Phone 5, Samsung Galaxy M52]
OS: [Android 12]
Version of the library being used [1.2.41]

@Vynex I got this issue. After fixing the following packages with the specific versions and reinstalling all the packages worked for me.
"react-native-linear-gradient": "2.5.6",
"react-native-svg": "12.1.0",

I can confirm that @zameerm99x fix works with 2.6.2:
"react-native-linear-gradient": "2.6.2",
"react-native-svg": "12.1.0",

@FranDepascuali
@zameerm99x
I had the same error and changed the versions as you guys said but it doesnt work...
In my case I tried in ios and android expo go application. In ios works perfectly but in android I still have the error "
java.lang.Double cannot be cast to com.facebook.react.bridge.ReadableMap".
also tried to change it to the newest version which is "react-native-svg": "^13.6.0" but did not work either...
Anybody with other solutions?

just to mentioned that the error content changed to "Invariant Violation: requireNativeComponent: "RNSVGSvgViewAndroid" was not found in the UIManager.". I can tell it is still related with the svg version of react-native-svg I guess... Hoping somebody run into the same error in the past...
below the versions of my files
"react-native-linear-gradient": "2.6.2",
"react-native-svg": "^12.1.0",

@FranDepascuali @zameerm99x I had the same error and changed the versions as you guys said but it doesnt work... In my case I tried in ios and android expo go application. In ios works perfectly but in android I still have the error " java.lang.Double cannot be cast to com.facebook.react.bridge.ReadableMap". also tried to change it to the newest version which is "react-native-svg": "^13.6.0" but did not work either... Anybody with other solutions?

Same for me

I had the same issue due to an empty quote "" on the fill attribute like this:
. After I had removed it, it worked fine.

so is this problem solved?

I have the same problem
"react-native-svg": "13.4.0",
"styled-components": "^5.3.6",
"react-native-linear-gradient": "^2.6.2",
"react-native-fontawesome-pro": "^2.1.0",

Update:I can confirm these versions work:
"react-native-linear-gradient": "2.6.2",
"react-native-svg": "12.4.4",

Downgrading to Expo v45 worked for me.
I used expo-cli upgrade 45, it helps u change the Expo version, change dependencies version to a compatible one, remove package-lock.json & node_modules and install everything back again, works like magic.

I had the exact same err on android. Downgrading to Expo v45 worked for me. I was on Expo v47.
I used expo-cli upgrade 45, it helps u change the Expo version, change dependencies version to a compatible one, remove package-lock.json & node_modules and install everything back again, works like magic.
"react-native-svg": "13.4.0" was also automatically downgraded to "react-native-svg": "12.3.0"

How do you install everything again?

Can you go more in depth with this? What file is this?
Is gifted charts working fine for you now? Or anyone else?
I think it would be helpful if someone just posted the whole package json at this point...

I had the same issue due to an empty quote "" on the fill attribute like this: . After I had removed it, it worked fine.

Same for me, had an svg with the following logic <Pause fill={timerRunning && "#0B6DFF"}/>
Changed it to <Pause fill={timerRunning? "#0B6DFF" : "#000000"}/> and works perfectly

What file? How did you know this was the issue?

in my case i am using react-native-iconly and the same issue occurs, no clue on what to do as i'm only a fresher

Hey guys👋
The dependencies have been upgraded in version 1.3.0🎉
Please use the latest version and feel free to reopen if the issue still persists.

Hi @Abhinandan-Kushwaha
I have tried with these versions
"react-native-svg": "^13.6.0", (also 13.4.0)
"react-native-linear-gradient": "^2.6.2",
"react-native-gifted-charts": "^1.3.0",
Now fails with (requireNativeComponent: "RNSVGSvgViewAndroid" was not found in the UIManager)

Hi @Abhinandan-Kushwaha
I have tried with these versions "react-native-svg": "^13.6.0", (also 13.4.0) "react-native-linear-gradient": "^2.6.2", "react-native-gifted-charts": "^1.3.0",
Now fails with (requireNativeComponent: "RNSVGSvgViewAndroid" was not found in the UIManager)

I have this error too. Expo Dev Client version 47.
What is the correct setup?
Also, which versions of Expo even support svg 13.9?

Has anyone found a solution to this with Expo Dev Client?

Are you guys still facing this issue? A new version of react-native-gifted-charts is recently released which has react-native-svg listed as a peer dependency instead of a direct dependency. Please try the latest version of the library where this issue is hopefully resolved.


Datastore runQuery doesn't throw exceptions

I can't seem to catch exceptions from datastore's runQuery(). I believe this is a library issue caused by exceptions not correctly bubbling up. I can correctly catch exceptions for other methods like insert()
Here's my code:
try {
    $query = $db->query()
        ->kind("Post")
        ->limit(3)
        ->order('timestamp', Query::ORDER_DESCENDING)
        // This is intentionally a malformed cursor to force an exception.
        ->start("CjgKEwoJdGltZXN0YW1wEgYIhPWOyQUSHWoIY2NhbGJsb2dyJ1sSBFBvc3QYgICAgICAwAsMGAAgAQ==");
    $posts = $db->runQuery($query);
} catch (Google\Cloud\Core\Exception\ServerException $e) {
    die("I am never called.");
} catch (Exception $e) {
    die("I am never called.");
}

The above results in the following exception but it is never bubbled up to the runQuery() method

Fatal error: Uncaught exception 'Google\Cloud\Core\Exception\ServerException' with message '{"error":{"status":"UNKNOWN","code":500}}' in /Users/calsmith/Projects/ccalblog/ccalblog/vendor/google/cloud/src/Core/RequestWrapper.php:241 Stack trace: #0 /Users/calsmith/Projects/ccalblog/ccalblog/vendor/google/cloud/src/Core/RequestWrapper.php(150): Google\Cloud\Core\RequestWrapper->convertToGoogleException(Object(GuzzleHttp\Exception\ServerException)) #1 /Users/calsmith/Projects/ccalblog/ccalblog/vendor/google/cloud/src/Core/RestTrait.php(81): Google\Cloud\Core\RequestWrapper->send(Object(GuzzleHttp\Psr7\Request), Array) #2 /Users/calsmith/Projects/ccalblog/ccalblog/vendor/google/cloud/src/Datastore/Connection/Rest.php(105): Google\Cloud\Datastore\Connection\Rest->send('projects', 'runQuery', Array) #3 /Users/calsmith/Projects/ccalblog/ccalblog/vendor/google/cloud/src/Core/Iterator/PageIteratorTrait.php(230): Google\Cloud\Datastore\Connection\Rest->runQuery(Array) #4 /Users/calsmith/Projects/ccalblog/ccalblog/vendor/google/clo in /Users/calsmith/Projects/ccalblog/ccalblog/vendor/google/cloud/src/Core/RequestWrapper.php on line 241

What's going on here?

That seems quite odd. One thing to try might be:
catch (\Exception $e)
instead of
catch (Exception $e)
This could be the issue if you do not have a use line for Exception.

Hey @calsmith, the runQuery method returns an iterator - and in this case no network requests are initiated until you start to iterate through the results.
If you add the following, I expect you should see your exception bubble up:
foreach ($posts as $post) {
    // Handle $post
}

Thanks @dwsupplee the foreach() bubbled up the exception correctly 🥇


[type:feature] support openGauss database

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #4856 (7a5f651) into master (ddda44b) will decrease coverage by 0.04%.
The diff coverage is 90.54%.

@@             Coverage Diff              @@
##             master    #4856      +/-   ##
============================================
- Coverage     64.60%   64.56%   -0.04%
- Complexity     8201     8228      +27
============================================
  Files          1153     1159       +6
  Lines         34019    34081      +62
  Branches       3047     3045       -2
============================================
+ Hits          21979    22006      +27
- Misses        10302    10326      +24
- Partials       1738     1749      +11

Impacted Files
Coverage Δ

.../mybatis/og/handler/OpenGaussSQLStringHandler.java
66.66% <66.66%> (ø)

...mybatis/og/handler/OpenGaussSQLBooleanHandler.java
83.33% <83.33%> (ø)

...apache/shenyu/admin/mybatis/og/enums/NullEnum.java
88.88% <88.88%> (ø)

.../og/interceptor/OpenGaussSqlUpdateInterceptor.java
94.44% <94.44%> (ø)

...che/shenyu/admin/config/DataBaseConfiguration.java
100.00% <100.00%> (ø)

...a/org/apache/shenyu/admin/config/MapperConfig.java
91.66% <100.00%> (-8.34%)
:arrow_down:

...og/interceptor/OpenGaussSQLPrepareInterceptor.java
100.00% <100.00%> (ø)

...s/og/interceptor/OpenGaussSQLQueryInterceptor.java
100.00% <100.00%> (ø)

... and 23 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

#4857


fix: expose types for esm/api

I was using ReturnType<typeof register> to get the type - which I wasn't able to get the overloads with namespaced one. I think it would be nice to expose them.

Sounds good. I think you'll want to re-export them here too:
https://github.com/privatenumber/tsx/blob/master/src/esm/api/index.ts

:tada: This issue has been resolved in v4.10.2
If you appreciate this project, please consider supporting this project by sponsoring :heart: :pray:


Create postman.yml

https://github.com/phildini/stayinghomeclub/pull/876/files
Adds or updates the following companies, events, or universities:

Checklist
Company updates

[x] I have put the most recent relevant date in the "Last Update" column
[ ] I have linked to the article about the change, not the company's homepage (Please put N/A if there is no public post)

Event updates

[ ] I have linked to the article about the change, not the event's homepage

University updates

[ ] I have linked to the article about the change, not the university's homepage

Thanks!


on_demand compiler pool doesn't seem to scale up properly

Run edb server --compiler-pool-mode=on_demand --compiler-pool-size=16
Run edb cli migration create on a big schema that will take a while; I used the one from https://github.com/edgedb/edgedb/issues/4677
Try running select 1 another cli.

It will hang for a long time. (It might sneak in somewhere in the middle of the migration, which is multiple commands.)
on_demand is the default for dev mode, so I was confused for a while. This reproduces without any compiler flags to edb server but I wanted to be explicit

The on_demand compiler pool only scales up when the number of queued requests goes up in a 3-seconds window. That means, the first select 1 will only start the 3-seconds window (with 1 queued request) and you'll need a second select 1+2 within 3 seconds to trigger the scale-up. This was designed to avoid scaling up for a busy-balanced scenario when the queue size is stable, but we can always change it (like count the starting queue size by 1 less so that we can adapt to the scenario mentioned in this issue).


Accents sending double letters

When typing a letter that is composed by first typing some kind of modification (´, `, ^,...) and then the letter to modify (mostly e, a, o and u) to get é, è, à and so on, the result I get when I type this is eè, eé, aà and so on - so there is one letter too much.
I am using Version 1.16 on Windows 10 (1909) and Sony G8441 (Android 9)

Use scrcpy --prefer-text.
See c916af0984f72a60301d13fa8ef9a85112f54202 and #650

Thanks a lot! I was searching for one like this but did not find it.


GLES 2 screen jank

Godot version:
Godot 3.2 beta 2
OS/device including version:
OSX 10.14.6
Macbook Pro 13" 2017. Intel Iris 640
Issue description:
When using GLES2 on a simple project, I get a serious jank/stutter even on a simple project when running from the editor.
https://www.dropbox.com/s/c5fcxoh65ub65c2/godot_jank_es2.mov?dl=0
If I switch this project to GLES3 it runs butter smooth, as I would expect.
Now, if I export the project as an osx app and run, it is also butter smooth, which makes me think it is something related to running from the editor.
In fact if I still keep the editor as GLES3, but I force GLES2 rendering via project settings, there is no jank/jitter.

Unfortunately when I reopen the project this switches the editor automatically to GLES2 and the jitter persists.
Anyone else had this problem?
I've tried to disable all debug/stout options with no luck
Steps to reproduce:
Just build the project in GLES2 on OSX
Minimal reproduction project:
simple_test.zip

I have the same issues with my current (GLES2) project -- I can hardly run the thing in Godot for macOS. (I'm generally seeing very poor editor performance on macOS with the integrated Intel Graphics, documented in #17584, which this issue here may or may not be related to.)
One thing I noticed was that there's specifically noticeable jank when the game creates instances of preloaded scenes. I've tried if the profiler surfaced something, but it didn't.

@gianmichelemariani @hmans Can you still reproduce this on Godot 3.3.2 or any later release?

The last few releases have been butter smooth in GLES2 as well for me!


Galaxy url fail

Hi @itcraftsmanpl ,
Your original galaxy url is fail, maybe you can change to new one.

Original: https://galaxy.ansible.com/detail#/role/6364
New: https://galaxy.ansible.com/itcraftsmanpl/php7/

Thank you for your roles, again.
@chusiang

Thanks. Link updated.


Symbol for [esc] key not universal

Hi Keycastr team - following up on an earlier comment regarding the symbol for the escape key (https://github.com/keycastr/keycastr/issues/195)
While the esc symbol used in the keycastr app makes sense to many mac users, it may not be universally understood by non-mac users. When creating content that's intended for cross-platform audiences (windows + linux), it would be helpful if users can customize the symbol for the esc key, or have a "cross platform" mode - just a suggestion
thanks!

Hey @daniel-packard thank you for opening an issue about this. While KeyCastr is most definitely a mac app, it makes sense that screencast creators may want the ability for more customization in order to target different audiences.
There does seem to be a need allow/deny/override lists for more advanced use cases. We have other issues for implementing allow/deny lists (see #152) so I will edit this to reflect the desire to override at least certain keystrokes. In addition to the escape key use case, I could see this being desirable for a few other special keys (fn vs 🌐) , as well as upcoming enhancements to the mouse click visualizer.

The control key would be another candidate.


🚀 [New Contribution] : Maven basics explained

Maven is the most commonly used build tool. Aiming to write an article to publish basics of maven

Hi @atapas,
Will this not be counted towards hacktoberfest?

Yes, it should be. What happen?

Here it's showing excluded

Here it's showing excluded

Ok, I see. We will try to seek more information on what happened. There could be some issues with the registration process. Give us a couple of days, we will update you here.

Sure


Aesthetic Enhancement - Visual Basic Syntax Highlights

Within the editor .vbs files have the appropriate syntax highlights but it seems it doesn't appear in the file viewer.

Here is my gist so you can use it for testing if it helps.
https://gist.github.com/DaemonPhoenix42/cd775d69c3542ee0de4679a3b1f46553
Just adds a toggle for hidden files to the context menu for easier access because I disabled the Explorer ribbon.
The first file is a .reg, I believe gist doesn't have support for registry files so I sent a request for it (idk if it will go through, I'm unaware of any reasons i.e. security implications perhaps for lack of support). I changed it to .txt because otherwise it gets marked as "#Other" in the language tab.

Thanks @meilinz !


fix: incorrect moment of change outside action listener state

:tada: This PR is included in version 3.0.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fix conformance jobs

This PR fixes the conformance job failure, which is caused by unable to build kubetest

thanks
/lgtm


Add option to TVMode for channel +  / channel -  command

Hi, how do you expect them to be added? there is no way from what I see in TV remote widget. Or do you expect them to be bound to up / down arrow ?

It's possible to change channels with Apple Remote App (IOS 12.2b4) so this request is irrelevant.

Hi, thanks for feedback . How do you change channel in b4 ?
Sent with GitHawk

Hi
By just selecting the up down arrows
Le 5 mars 2019 à 11:55:38, Nicolas Dujardin<EMAIL_ADDRESS>a
écrit:

Hi, thanks for feedback . How do you change channel in b4 ?
Sent with GitHawk http://githawk.com
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/nicoduj/homebridge-harmony/issues/80#issuecomment-469636207,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AtIvqXKRc5SBjWwGrS3AyRUxBzQl3PmEks5vTk0qgaJpZM4bc-0p
.

Ok, so it is probably because your activity and up/ down commands are mapped to this (there is no command explicitly for channel up / down in HomeKit)
Sent with GitHawk


feature: add collapse/expand all button to ui target page for react ui

This was added for the old UI but not the react UI, but now that the react ui is default this is relevant again, see following issue:
https://github.com/prometheus/prometheus/issues/6127

The ui target page can contain a lot of different scrape jobs which can be collapsed to give a good overview.
What is missing is a collapse and expand all targets button to help navigate to targets on is interested in quickly.
In our case we have prometheus configurations with many jobs, some of them have thousands of scrape targets. The default on first visit is to have all targets are expanded and one has to scroll many pages to get to a target one wants to check. After the first visit the collapsed jobs are stored in the browser which makes it ok, but if you have many different prometheus instances you have to do a of clicks each time you visit a new one.

I can work on this.


Fix: fmt Format String is not parsed and normalized

Resolved by https://github.com/exercism/nim-representer/pull/6?

Yes. I commented that it closes it here. I guess it needed have been the first comment or in the commit messages