feat: PropagateTagsToVolumeOnCreation: true

Ensures tags that may be used for cost management are propagated to EC2 volumes. At present only the instance is tagged, EBS volumes aren't.
https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/aws-properties-ec2-instance.html#cfn-ec2-instance-propagatetagstovolumeoncreation
Issue #, if available:
Description of changes:
Checklist:

[x] Have you successfully deployed to an AWS account with your changes?
[ ] Have you written new tests for your core changes, as applicable?
[x] Have you successfully tested with your changes locally?
[ ] If new dependencies have been added, have they been pinned to specific versions?
[ ] Is this change also required on the AWS Solution version?
[ ] Have you updated openapi.yaml if you made updates to API definition (including add, delete or update parameter and request data schema)?
[ ] If you had to run manual tests, have you considered automating those tests by adding them to end-to-end tests?
[ ] If you are updating the changelog and vending out a new release, have you updated versionNumber and versionDate in .defaults.yml

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

Hey @manics! Thanks for the PR. We're a little swamped right now but will get these reviewed soon!


feat(cadl): support plainDate and plainTime types

change emtter to:

add InputTypeKind.Date type in emitter
for CADL plainDate type, change the InputTypeKind from DateTime to Date, so that it can be serialized/deserialized as date only in generated codes

add tests

resolve #2473
Description
Add your description here!
Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

What csharp type for Date? in the TypeFactory.cs line 51, it will change to DateTimeOffset, is it correct?
InputTypeKind.Date => new CSharpType(typeof(DateTimeOffset), inputType.IsNullable),

@chunyu3 That's the current behavior: using DataTimeOffset to represent date only type. Only yyyy-mm-dd part will be handled, see: https://github.com/Azure/autorest.csharp/blob/8e02d32fdd84f17e54b5309e091bf36316cb25af/src/assets/Generator.Shared/TypeFormatters.cs#L27

@archerzz,  could you confirm this mapping for plainDate & plainTime with @m-nash  before merging the PR?


Fix jadengeller-helium url stanza

https://github.com/JadenGeller/Helium changed the URL format for releases. The version number is no longer prefixed with a "v".
I did not notice this when updating the formula earlier.

Thank you for the contribution. It was merged directly as f90516e5801a8fd02ccd2a470d21d717909f773c to keep commit history cleaner. Your contribution is still credited to you.


git is missing from tools-deps images - regression

Hi,
I opened this issue a while ago and it was fixed but I think it regressed:
https://github.com/Quantisan/docker-clojure/issues/119
I have a deps.edn with some git libraries and it's failing with missing git.
A repro is in this git repo: https://github.com/sandre1/clj-training/tree/main/htmx-demo .
I am aslo trying to build for arm64 on amd64 and I could not do this using clojure:temurin-17-tools-deps-bullseye .
I had to use clojure:temurin-11-tools-deps-bullseye .
I am still investigating this but it might be an issue with the image (some jlink changes from 11-> 17).
I will come back once I know more.

Fix will be live shortly after this is merged: https://github.com/docker-library/official-images/pull/13829


couldn't make an invoice with the given data: Post "XYZ.onion:9735/v1/invoices": unsupported protocol scheme "XYZ.onion"

couldn't make an invoice with the given data: Post "XYZ.onion:9735/v1/invoices": unsupported protocol scheme "XYZ.onion"

Does this mean that the server doesn't have Tor enabled? I tried several of the public servers. All the same.

I screwed up, used the wrong port ... d'oh.


[FEATURE] Discord Notification on Points Awarded

Is your feature request related to a problem? Please describe.
Currently, we have to manually inform our communtiy, if someone was awarded points.
Describe the solution you'd like
Automate it using trigger.dev
Hit up @Chigala if you have any questions on how to do it best :)
Describe alternatives you've considered
No response
Additional context
No response

/assign

/unassign


[Enhancement] Translation Quest B22

I think we need to update B22 quest details. After Abukuma Kai Ni and 3-2 recent changes now you can use CL as flagship too

meanwhile our translation (taken from english translation):
"244": {
"name": "Deploy the Destroyer Division 30 (Gen-1)",
"desc": "Deploy a fleet that consists of Mutsuki (\u7766\u6708), Kisaragi (\u5982\u6708), Yayoi (\u5f25\u751f), Mochizuki (\u671b\u6708) and up to 2 additional DDs to world 3-2 and encounter the boss. C rank or above is required."
},

hmmm. are the two additional destroyers actually requirement of just hint or the wikia writer to remind readers that only DDs can sortie to it?
we'd have to base it on the in-game JP description

From Wikiwiki, it is already updated...
"「第三十駆逐隊(一次)」の4隻（「睦月」「如月」「弥生」「望月」）+他に軽巡洋艦（旗艦）・駆逐艦がいてもOK"
"Having four ships of the Destroyer Division 30 (Gen-1) (Mutsuki, Kisaragi, Yayoi, Mochizuki), plus other one (flagship) light cruiser and one destroyer are OK"

imho we should still follow in-game description rather than base it on community advice. We do offer on-screen translation and its best to translate raw.
though really, I'm haven't really done any translation to know what is acceptable mainstream, so I might be in the wrong there.

I personally agree with you, @dragonjet, too. But if the mainstream (especially the foreign ttks) is say "yes" to change it, then we need to do.

gonna close this


Add a "Small Pdf" in Home

What feature?
I have an idea  Small pdf is an online platform that offers a range of PDF tools, including a PDF converter. It uses AI algorithms to convert PDFs to different formats while preserving the formatting and layout of the original document.
#gssoc23
Add screenshots
Add screenshots
Code of Conduct

[x] I agree to follow this project's Code of Conduct

Hey, I would like to work on this issue. Can you please assign it to me under GSSOC'23

I have previously worked on such tools to convert and change the format of pdf . Also , i think it would be great if we allow the user to edit pdf without providing users the unease to actually convert it into other format first . So , i think giving an editor tool without having the need to change the file format will provide immense comfort to the user and therefore enhancing the user experience . So i will be more than happy to work on this issue under GSSOC'23 as per your permission . Thankyou!

@kedar1998 is suggesting adding the small PDF to the database, am I right? If that's the case, as you asked first, you will be assigned to this task.

Hey, I would like to work on this issue. Can you please assign it to me under GSSOC'23

As @rayarka26 created the issue, he will be assigned to it. If he fails to create a pull request, then you will be assigned to the issue because it works on a first-come, first-serve basis.
@kavya-gaur, I apologize, but I didn't understand what you wanted to say. Could you please clarify?

@JasonDsouza212 yes I will add this too it will be beneficial

@JasonDsouza212 yes I will add this tool it will be beneficial


Support OpenBSD

Track merge requests in dependencies/crates and update documentation accordingly.
All patches required to run psst on OpenBSD/amd64 have been submitted upstream:

https://github.com/ExPixel/miniaudio-rs/pull/75
https://github.com/linebender/druid/pull/1993
https://github.com/linebender/piet/pull/466

Dependencies have been carefully patched to merely integrate those patches
required for OpenBSD support, i.e. no new features or unrelated changes are
pulled in.
This makes cargo build produce a working ./target/debug/psst-gui.
This is already enough to create an OpenBSD port (from my fork) and provide
working binary packages to users.
Input on how to best incorporate all this would be welcome, hence this draft.

Hi @klemensn, thanks a lot for the PR and for being interested! I'm sorry for being a bit late here.
Lately I've been trying to switch away from miniaudio-rs crate, and there's a fresh #167 that explores this. Do you think we could wait until it gets merged? I reckon we should build on OpenBSD without any patches with it.

piet released 0.5.0 containing my commit.
druid master contains my fix and an internal update to piet 0.5.0.
I tried bumping both dependencies in psst but my rust/cargo foo is not strong enough;
it keeps building piet-0.5.0-pre1 which obviously fails.
Can someone help me out?

Hi @klemensn, I've updated the Druid dep in master!

Thanks a bunch.  Would you mind telling me how to do that?
I've modified the Cargo.{lock,toml} files and/or run cargo update -p piet -p piet-common ... but somehow it always went back to the -pre1 version.

Oh, sorry, it's locked in Cargo.toml to my psst branch of Druid, I've rebased it on top of master.

Once https://github.com/Sinono3/souvlaki/pull/19 is merged, another overridden dependency can go.

[ ] crank souvlaki after OpenBSD build fix

Playback works with cpal but has remaining issues

[ ] rebase https://github.com/RustAudio/cpal/pull/493 ontop of latest RustAudio/cpal and retest

Alright, cpal seems to be the last blocker for now.
I'll see what I can do over there to get the sndio backend working properly (and land it in master).

Rebasing this branch onto latest master breaks playback:
$ RUST_BACKTRACE=full ./target/release/psst-gui
[2021-11-30T00:10:07Z INFO  psst_gui::data::config] loading config: "/home/kn/.config/Psst/config.json"
[2021-11-30T00:10:07Z INFO  psst_core::audio::output] using audio device: "sndio default device"
[2021-11-30T00:10:07Z INFO  psst_core::cache] using cache: "/home/kn/.cache/Psst"
[2021-11-30T00:10:07Z INFO  psst_core::audio::output] opening output stream: StreamConfig { channels: 1, sample_rate: SampleRate(48000), buffer_size: Default }
[2021-11-30T00:10:07Z INFO  psst_gui::data::config] saved config: "/home/kn/.config/Psst/config.json"
[2021-11-30T00:10:07Z INFO  psst_core::session::access_token] access token expired, requesting
[2021-11-30T00:10:09Z INFO  symphonia_format_ogg::demuxer] starting new physical stream
[2021-11-30T00:10:09Z INFO  symphonia_format_ogg::demuxer] selected vorbis mapper for stream with serial=0x0
[2021-11-30T00:10:09Z INFO  psst_core::player] starting playback
[2021-11-30T00:10:09Z INFO  psst_gui::controller::playback] playing
thread '<unnamed>' panicked at 'index out of bounds: the len is 1 but the index is 1', psst-core/src/audio/source.rs:65:13
stack backtrace:
   0:      0x33b342e3bbc - <unknown>
   1:      0x33b3431ca6b - <unknown>
   2:      0x33b342d32e3 - <unknown>
   3:      0x33b342d686b - <unknown>
   4:      0x33b342d63f7 - <unknown>
   5:      0x33b342d6f56 - <unknown>
   6:      0x33b342e43a5 - <unknown>
   7:      0x33b342e3d07 - <unknown>
   8:      0x33b342d6a26 - <unknown>
   9:      0x33b34311150 - <unknown>
  10:      0x33b34311117 - <unknown>
  11:      0x33b340700ce - <unknown>
  12:      0x33b34066cc8 - <unknown>
  13:      0x33b3406e41a - <unknown>
  14:      0x33b341e6be0 - <unknown>
  15:      0x33b340a63da - <unknown>
  16:      0x33b3403339a - <unknown>
  17:      0x33b342de179 - <unknown>
  18:      0x33d46e136a1 - <unknown>
  19:      0x33e0cc4808a - <unknown>

[x] druid ship another release

So that just happened!
I haven't used psst ever since, basically because hacking and testing it on a still semi-supported OS was very time consuming.
I'll give this another shot to see how things improved.

Having ported spotifyd and spotify-qt to OpenBSD, which now ship as packages and work great, I won't persue this effort of finishing psst and its dependencies.


metanorma/metanorma-build-scripts#65 Update GHA configs

As title.
Generated by Cimas.

I have checked tests and looks like behavior changed because the new nokogiri version, I have update tests to work with both versions of nokogiri
@opoudjis @ronaldtse could you please have a look, because I'm not sure that I did it in the right way?

I am going to have to understand what Nokogiri 1.11 is doing with >, so I'll check this, thank you.

Oh, I ended up dealing with this myself, so closing this.

Oops, no, I just need to overwrite your spec changes.


Add Cargo.lock file

Please add the Cargo.lock file so that when building the nix derivation for the rust implementation of binwalk, it will not needed to define each dependency and it's hash, because the cargoHash will be used.
https://nixos.org/manual/nixpkgs/stable/#rust
It'd be great if you could also add a nix derivation for a fully reproducible build of binwalk, since what is from the Dockerfile for sure isn't, since the answer is no to the question: will this result in the same build inputs in half a year, or maybe 2 years.
https://github.com/ReFirmLabs/binwalk/blob/d0c89d855bee7cb8e3bfe7424db2e5c59ab88ebd/Dockerfile#L6

Added Cargo.lock to the git repo. I like the nix idea, I'll leave this issue open until that is implemented. Thanks!

https://github.com/NixOS/nixpkgs/issues/350300

Do I not understand this correctly?
https://github.com/ReFirmLabs/binwalk/blob/e0a4434159f22ff926c1baa3872d93a5f8dbf87e/dependencies/README.md?plain=1#L3
https://github.com/ReFirmLabs/binwalk/blob/e0a4434159f22ff926c1baa3872d93a5f8dbf87e/dependencies/pip.sh#L9
https://github.com/ReFirmLabs/binwalk/blob/e0a4434159f22ff926c1baa3872d93a5f8dbf87e/dependencies/src.sh#L5
Are those not needed for runtime?
Like... where are the dependencies in the buildInputs...
Just because V3 builds, doesn't mean it also has all the necessary dependencies it needs to work in every scenario.
Some dependencies don't even exist in nixpkgs.
I'm trying to get them merged, but it will take some more time until everything is done.
https://github.com/NixOS/nixpkgs/pull/349055
https://github.com/NixOS/nixpkgs/pull/348768

https://github.com/NixOS/nixpkgs/pull/357991


PytorchHub for OpenPCDet

I want to know whether our openpcdet can be loaded from pytorchhub?

No, currently we still do not support it.


Implement Assembly Load Context

It seems like there's a conflict with the AzTable module if that is loaded first. An Assembly Load Context should be implemented.

Now implemented.


CIS: Stop recursively setting permissions on logs files

This suffers from a time-of-check to time-of-use race condition and in general recursively setting file permissions seems to be a bad idea.

@jovial you might want to update your local stackhpc/2023.1 checkout


pkg_resources:DistributionNotFound: Pokedex==0.1

I'm trying to set up the Pokedex database on my server using SQL, but I'm getting this error.
/veekundata/pokedex/bin# python pokedex load -e mysql://                                     myusername:mypassword@localhost/data_veekunpokedex
Traceback (most recent call last):
File "pokedex", line 5, in
from pkg_resources import load_entry_point
File "build/bdist.linux-x86_64/egg/pkg_resources/init.py", line 3020, in <                                     module>
File "build/bdist.linux-x86_64/egg/pkg_resources/init.py", line 614, in _b                                     uild_master
File "build/bdist.linux-x86_64/egg/pkg_resources/init.py", line 920, in re                                     quire
File "build/bdist.linux-x86_64/egg/pkg_resources/init.py", line 807, in re                                     solve
pkg_resources.DistributionNotFound: Pokedex==0.1
Any ideas?

Hi. Did you ever figure this out?

Closing as the reporter has likely either solved their problem or given up by now.
@immewnity If you still need help setting up the pokedex, please leave another comment here or join us on irc.


peer to peer code review

Great work , your are doing well 👍
.
Great work ,

[x] you use es6✔
[x] your code is clean✔
[x] keep it up

Cheers and happy coding

Thank you @techmoves


Official VPC module integration

https://registry.terraform.io/modules/terraform-aws-modules/vpc/aws/1.0.4
The module can create the default database subnet group. How can I use it in RDS module?

You basically have two main options:

Use database subnet group created by VPC module and use module db_instance of terraform-aws-modules/rds/aws.
Use this module which will create RDS instance and database subnet group, but disable creation of database subnet group in VPC module (create_database_subnet_group = false)

For me, it seems that option 2 is better.

Hm... But why I can't use the already created subnet group? Even if it's manually created?

You certainly can. This is the same as the first option. Use module db_instance of terraform-aws-modules/rds/aws.


Add permissions internally to Lib

[ ] Modify / add documentation
[ ] Add logic to request permissions where needed

Closed by commit 40ce82ce7c9c00e8b6ef31a08cdba4d7b1c87cc9


VB.net? Can you add just an example or two for VB.net

Would be hugely helpful to get started. Thanks!

I actually have never written VB.net :). If you have any though I would be happy to merge a change to showing this


test: TimelineStepsFeature.AddAppointmentToTimeline

The test Capgemini.PowerApps.SpecFlowBindings.UiTests.TimelineStepsFeature.AddAppointmentToTimeline is failing.
Test method Capgemini.PowerApps.SpecFlowBindings.UiTests.TimelineStepsFeature.AddAppointmentToTimeline threw exception:
OpenQA.Selenium.NoSuchElementException: Field with name description does not exist.

   at Microsoft.Dynamics365.UIAutomation.Browser.BrowserCommand`1.Execute[T1,T2,T3,T4,T5,T6,T7,T8,T9](IWebDriver driver, T1 p1, T2 p2, T3 p3, T4 p4, T5 p5, T6 p6, T7 p7, T8 p8, T9 p9)
   at Microsoft.Dynamics365.UIAutomation.Browser.BrowserCommand`1.Execute(IWebDriver driver)
   at Microsoft.Dynamics365.UIAutomation.Browser.BrowserPage.Execute[TResult](BrowserCommandOptions options, Func`2 delegate)
   at Microsoft.Dynamics365.UIAutomation.Api.UCI.Timeline.AddAppointment(String subject, String location, String duration, String description)
   at Capgemini.PowerApps.SpecFlowBindings.Steps.TimelineSteps.WhenIAddAnAppointmentToTheTimeline(String subject, String description, String duration, String location) in \bindings\src\Capgemini.PowerApps.SpecFlowBindings\Steps\TimelineSteps.cs:line 27
   at TechTalk.SpecFlow.Bindings.BindingInvoker.InvokeBinding(IBinding binding, IContextManager contextManager, Object[] arguments, ITestTracer testTracer, TimeSpan& duration)
   at TechTalk.SpecFlow.Infrastructure.TestExecutionEngine.ExecuteStepMatch(BindingMatch match, Object[] arguments)
   at TechTalk.SpecFlow.Infrastructure.TestExecutionEngine.ExecuteStep(IContextManager contextManager, StepInstance stepInstance)
   at TechTalk.SpecFlow.Infrastructure.TestExecutionEngine.OnAfterLastStep()

https://dev.azure.com/capgeminiuk/GitHub Support/_build/results?buildId=8397&view=ms.vss-test-web.build-test-results-tab&runId=1015498&resultId=100161&paneView=debug

WebDriverWait wait = new WebDriverWait(driver, TimeSpan.FromSeconds(10));
IWebElement descriptionField = wait.Until(ExpectedConditions.ElementIsVisible(By.Name("description")));

If the element is still not found, double-check the page's HTML to confirm that the description field exists and that its name attribute is correctly specified.


chore: release main

:rocket: Automated Release
dev-dependencies-client: 7.0.0
7.0.0 (2024-12-03)
⚠ BREAKING CHANGES

Vite was bumped from 5.x to 6.x

Bug Fixes

bump some breaking dependencies to latest (#473) (fe35c5d)

Dependencies

The following workspace dependencies were updated

dependencies

@versini/dev-dependencies-common bumped to 5.0.0

dev-dependencies-common: 5.0.0
5.0.0 (2024-12-03)
⚠ BREAKING CHANGES

Vite was bumped from 5.x to 6.x

Bug Fixes

bump some breaking dependencies to latest (#473) (fe35c5d)

dev-dependencies-eslint: 2.0.0
2.0.0 (2024-12-03)
⚠ BREAKING CHANGES

Vite was bumped from 5.x to 6.x

Bug Fixes

bump some breaking dependencies to latest (#473) (fe35c5d)

dev-dependencies-server: 6.0.0
6.0.0 (2024-12-03)
⚠ BREAKING CHANGES

Vite was bumped from 5.x to 6.x

Bug Fixes

bump some breaking dependencies to latest (#473) (fe35c5d)

Dependencies

The following workspace dependencies were updated

dependencies

@versini/dev-dependencies-common bumped to 5.0.0

dev-dependencies-types: 2.0.0
2.0.0 (2024-12-03)
⚠ BREAKING CHANGES

Vite was bumped from 5.x to 6.x

Bug Fixes

bump some breaking dependencies to latest (#473) (fe35c5d)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

dev-dependencies-common-v5.0.0
dev-dependencies-client-v7.0.0
dev-dependencies-server-v6.0.0
dev-dependencies-types-v2.0.0
dev-dependencies-eslint-v2.0.0
:sunflower:


This is for lbaas v1, v2 is different

Might want to make that clear in your readme.
See https://github.com/openstack/neutron-lbaas/tree/master/devstack for more details on lbaas v2. Note that the default provider is Octivia which spins up a nested vm (amphora) which can be very slow on qemu systems.

Thanks @kramvan1 I will see the v2 details and will post for v2 as well


new firmware 1.8.46

MANIFEST_DEFINITIONS appear to match other recent firmware versions, tested on my TomTom Runner

Hi,
this correction in ./ttwatch/manifest.txt I've made for my watch (1.8.46):
treadmill/metrics/left,29,e,1,0=clock,1=duration,2=distance,4=pace,5=avg_pace,6=speed,7=avg_speed,20=calories,21=heart,25=hr_zone
treadmill/metrics/right,30,e,1,0=clock,1=duration,2=distance,4=pace,5=avg_pace,6=speed,7=avg_speed,20=calories,21=heart,25=hr_zone
running/metrics/left,33,e,1,0=clock,1=duration,2=distance,4=pace,5=avg_pace,6=speed,7=avg_speed,20=calories,21=heart,25=hr_zone
running/metrics/right,34,e,1,0=clock,1=duration,2=distance,4=pace,5=avg_pace,6=speed,7=avg_speed,20=calories,21=heart,25=hr_zone
This here does not work for buzz, only for click (0,1) options/click,84,e,1,0=off,1=click+bz,2=click,3=buzz
I think, were is an other option for buzz ??
Kind regards, Dirk

@DirkBri, you should post this as a diff rather than simply the new lines, since it's unclear what you're showing has changed here.
I don't have an HRM so I didn't test any of the HR-related changes, but options/click does not seem to have changed on my watch.

... OK, it's really quiet simple!

@DirkBri I'll check against my watch when I get some time. Probably tomorrow. Did you check to see if the equivalent settings for cycling and freestyle have changed also?
@dlenski FYI, you don't actually need a HRM to test this; I don't have one either. If you enable the heart rate sensor in options, all the heart-rate related settings appear, regardless of whether you actually have one, presumably because the watch doesn't necessarily expect you to be wearing it until you start the activity.

No, I didn't check cycling and freestyle, because my watch supports only treadmill and running.
In case of options/click: On my watch, ttwatch only shows if "click" is enabled or not. It's independent from buzz. (I tried 85 - without success.)

Sorry, I missed that you had a Runner, rather than a Multisport. The cycling, swimming and freestyle entries have changed as well. This is good. Previous versions had the same entries with different numerical values for different activities, which seemed odd to me (eg. calories was 7 for most things, but 8 for swimming). Now they're all the same between different activities.
Re click/buzz, you're right, it's been split into two entries. One for click, one for buzz. The buzz entry has been is entry 163, click stays at entry 84. Both are either 0 or 1 for off or on.
Thanks for letting me know. I have pushed changes to the code.


Elapsed filter with load balanced logstash in Filebeat

Hello.
I have Filebeat with 4 logstash servers configured with load balancing on
  logstash:
    # The Logstash hosts
    hosts: ["logstash1", "logstash2", "logstash3", "logstash4"]

    # Optional load balance the events between the Logstash hosts
    loadbalance: true

And I have logstash-filter-elapsed configured on my logstash servers.
The problem is when "start" event comes to one logstash instance, and "end" event comes to another logstash server I'm getting elapsed_end_without_start with no elapsed_time calculated (obviously).
I didn't find any notes about dealing with this filter in loadbalanced logstash env.
Are there any recommendations how this can be configured?
Or this plug in shouldn't be used in loadbalanced env? Then it makes sense to add this info into documentation.
Thanks.

We are also facing same issue? did you get solution for this, as for us we decided to append the Logstash hostname to unique_id_field so it is unique for each LS

I didn't find the solution, was going to disregard of Elapsed filter in favour to custom solution.
But how does "append LS hostname to unique_id_field" help you? Can you please explain.
Cause how I understand this:

Filebeat send start_event log to LS1
Filebeat send end_event log to LS2

So anyway my end_event will have elapsed_end_without_start. How modifying unique_id_field can help to find start_event log on another LS1 server?

Sorry for confusion, in our case we are using only start_event and timeout where we want to reset count to 0 on that logstash and finally clone event and send it to ES so other LS can read from there to get current count. we are also using Kafka as message broker and hence we have no control over which event goes where but as we using unique field as combination with hostname the elapsed filter calculates time correctly on that LS.


Update dependencies

Updated dependencies for compatibility with new versions of dependent packages.

Update dependencies during migration on null safety (dart sdk >=2.12.0) - #11


Integration testing

Integration tests that deploy the uniswap v3 contracts and use them to create accounting tests for all of the cellar functionality. Ideally these tests are also modular so that consuming repos (such as peggyjv/sommelier) can reuse components of the test suite to test systems integrating volumefi/cellars. There is some prior art in the uni v3 staking contracts https://github.com/omarish/uniswap-v3-staker/tree/master/test that should be taken into account.

Tests are complete. Please check suite and re-open if we still have issues?


Change config tokens from opaque structs to strings.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #14   +/-   ##
=======================================
  Coverage   71.79%   71.79%
=======================================
  Files           9        9
  Lines         670      670
=======================================
  Hits          481      481
  Misses        171      171
  Partials       18       18

Impacted Files
Coverage Δ

logging.go
31.57% <ø> (ø)
: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 d66f0e3...b4a5a2c. Read the comment docs.


Fix for the InvalidParemeter OU error

Look for the LegalEntity ID inside the AccountClaim CR instead of the Account CR.
Ticket

LGTM, @ArielLima can you please address the typo so we can merge?

@rogbas Done

@rogbas Please merge
@fahlmant please look

/hold


added scc file and recipe card and jsx css file

added scc file and recipe card and jsx css file

good


Add ComputePlugins class

In order to get some useful information about checkstyle (parameters, rule-set...) I need to compute maven's plugins.
I thought it might be useful to implement it as a step in case some else wants to re-use it.
I have also added the test cases.

This PR looks good to me.

Perfect, @btellstrom can you merge it ?

Looks good to me as well.


NGINX Mon Pattern Traffic Routing and NGINX resource issue

Describe the bug
NGINX Mon Pattern Traffic Routing and NGINX resource issue
To Reproduce
Steps to reproduce the behavior:
While working on the NGINX Mon Pattern after step 5 Deploy an example Nginx application Ingress resource fails after using the below Ingress resource it works, documentation needs to be updated.
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: ingress-nginx-demo
  namespace: {{namespace}}
spec:
  ingressClassName: nginx
  rules:
  - host: '<Complete URL of NGINX LB SERVICE>'
    http:
      paths:
        - path: /apple
          pathType: Prefix
          backend:
            service:
              name: apple-service
              port:
                number: 5678
        - path: /banana
          pathType: Prefix
          backend:
            service:
              name: banana-service
              port:
                number: 5678

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

Device: [e.g. iPhone6]
OS: [e.g. iOS8.1]
Browser [e.g. stock browser, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

@joozero Is working on this issue.


Peer feedback

Hi Group 12!
This is Rahul from group 11, please find below my feedback regarding the dashboard you created.
Things that I liked:-

The overall dashboard is quite informative.
The UI is easy to use and self-explanatory.

Thing that I felt could be improved:

When I toggle between “Confirmed”,”Death” and “recovered” , the highlight seems to stay on the “confirmed” button even though the map changed, which confused me a little. I think this issue is there with all the tabs.

The Total Cases and Recovered counts for individual countries is using scientific notation for large numbers (for example, when India is selected) which is a little difficult to read.

Perhaps you could use different color schemes for ‘confirmed’ , ‘deaths’ and ‘recovered’ such as green for recovered, red for deaths and blue for confirmed.

Other than this, I think its a good dashboard. Please let me know if you need any clarifications regarding the above points. All the best!

Hi Rahul,
Thank you very much for your feedback, those make sense, we will try our best to implement those in Milestone #4.
Regarding point #1 in the improvement list: there is a square wrapped around the selected button, but I guess it might be too subtle to notice, we will try something else.
All the best,
Mai - on behalf of Group 12


VsCode Cannot be worked after upgrading

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: version 1.91
OS Version: macOS Catalina 10.15.7 ;intel Core i5

Steps to Reproduce:

download intel VsCode version 1.91
Visual Studio Code.app  dont open

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.91.0. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

I also encountered the same problem.  How to solve it?

after updating vscode, it started crashing on launch.
i promptly uninstalled.
then reinstalled from the vscode website - from the update tab (June 2024 release v1.91)
still not opening as of today.
my computer: Catalina
version 10.15.7
MacBook Pro (Retina, 13-inch, Early 2015)
3.1 GHz Dual-Core Intel Core i7
there was info in a closed issue i came across today that in some days there should be a new stable release up:
(https://github.com/microsoft/vscode/issues/219893#issuecomment-2216468779)
the last comment on the closed issue gave a link to track progress of v1.91.1:
(https://github.com/microsoft/vscode/issues/219893#issuecomment-2219375653)

/duplicate

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Adding the same field twice result in internal server error

 Issue by marktani
Wednesday Aug 30, 2017 at 13:35 GMT
Originally opened as https://github.com/graphcool/api-bugs/issues/248

What is the current behavior?
Adding the same field twice at the same time result in internal server error.
Please share the relevant part of your GraphQL schema and all functions, permissions or other project settings for easier reproduction
Add the same field twice to an existing type:
type Item {
  # ...
  # new:
  test: String
  test: String
}

Apply changes. You'll receive an internal server error, and no test field has been added.
What is the expected behavior?
Either an error message, or the field test added once.

 Comment by kbrandwijk
Wednesday Aug 30, 2017 at 13:46 GMT

I'd vote for an error message. Either the schema is 100% correct, and changes will be applied, or it isn't. I would consider it a bad thing if the system starts 'thinking' for itself about how to correct my mistakes :smile:

@do4gr Did we solve this already?

It now throws an error: {Type}: The type {Type} has a duplicate fieldName. So seems solved!


Error: The non-abstract class 'ChipsInputState' is missing implementations for these members:  - TextInputClient.performSelector

Environment
Package version:

Flutter doctor

C:\src\flutter\bin\flutter.bat doctor --verbose

[√] Flutter (Channel master, 3.1.0-0.0.pre.2166, on Microsoft Windows
[Version 10.0.14393], locale en-IN)

• Flutter version 3.1.0-0.0.pre.2166 on channel master at C:\src\flutter

• Upstream repository https://github.com/flutter/flutter.git

• Framework revision 87f1684476 (30 hours ago), 2022-08-08 23:00:11 -0700

• Engine revision b6dd604d90

• Dart version 2.19.0 (build 2.19.0-76.0.dev)

• DevTools version 2.16.0

[√] Android toolchain - develop for Android devices (Android SDK version
32.1.0-rc1)

• Android SDK at C:\Users\Snow\AppData\Local\Android\sdk

• Platform android-32, build-tools 32.1.0-rc1

• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java

• Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)

• All Android licenses accepted.

[√] Chrome - develop for the web

• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[√] Visual Studio - develop for Windows (Visual Studio Community 2019
16.11.14)

• Visual Studio at C:\Program Files (x86)\Microsoft Visual
Studio\2019\Community

• Visual Studio Community 2019 version 16.11.32428.217

• Windows 10 SDK version 10.0.16299.0

[√] Android Studio (version 2021.1)

• Android Studio at C:\Program Files\Android\Android Studio

• Flutter plugin can be installed from:

https://plugins.jetbrains.com/plugin/9212-flutter

• Dart plugin can be installed from:

https://plugins.jetbrains.com/plugin/6351-dart

• Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)

[√] VS Code (version 1.70.0)

• VS Code at C:\Users\Snow\AppData\Local\Programs\Microsoft VS Code

• Flutter extension version 3.44.0

[√] Connected device (2 available)

• Windows (desktop) • windows • windows-x64 • Microsoft Windows [Version
10.0.14393]

• Chrome (web) • chrome • web-javascript • Google Chrome 104.0.5112.81

[√] HTTP Host Availability

• All required HTTP hosts are available

• No issues found!

Process finished with exit code 0

Code sample

import 'package:flutter/material.dart';
import 'package:form_builder_extra_fields/form_builder_extra_fields.dart';

void main() {
  runApp(const MyApp());
  }

  class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  [@OverRide](https://github.com/OverRide)
  Widget build(BuildContext context) {
    return MaterialApp(
    title: 'Flutter Demo',
      home: FormBuilderTouchSpin(
      name: 'Volume',
      ),
    );
  }
}

Description
Expected behavior:
Current behavior:
Steps to reproduce

This
Than that
Then

Images
Stacktrace/Logcat
Launching lib\main.dart on moto g31 in debug mode...
Running Gradle task 'assembleDebug'...
../../AppData/Local/Pub/Cache/hosted/
pub.dartlang.org/flutter_chips_input-2.0.0/lib/src/chips_input.dart:87:7:
Error: The non-abstract class 'ChipsInputState' is missing implementations
for these members:

TextInputClient.performSelector

Try to either

provide an implementation,

inherit an implementation from a superclass or mixin,

mark the class as abstract, or

provide a 'noSuchMethod' implementation.

class ChipsInputState extends State<ChipsInput>
^^^^^^^^^^^^^^^
/C:/src/flutter/packages/flutter/lib/src/services/text_input.dart:1172:8:
Context: 'TextInputClient.performSelector' is defined here.
void performSelector(String selectorName) {}
^^^^^^^^^^^^^^^
FAILURE: Build failed with an exception.

Where:

Script 'C:\src\flutter\packages\flutter_tools\gradle\flutter.gradle' line:
1159

What went wrong:

Execution failed for task ':app:compileFlutterBuildDebug'.
Process 'command 'C:\src\flutter\bin\flutter.bat'' finished with non-zero
exit value 1

Try:
Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.
Get more help at https://help.gradle.org/

BUILD FAILED in 8s

when I add form_builder_extra_fields package I get this error
Can someone help me how to fix it?

Hi Matias,
Thanks for the quick response.
But my problem is whenever I try to use the 'form_builder_extra_fields'
package I get this error.
Actually I want to use the "form builder touch spin" from the
'form_builder_extra_fields' package.

On Tue, Aug 9, 2022 at 5:46 PM Matias de Andrea @.***>
wrote:

Hi @Sakthivel-bpigt https://github.com/Sakthivel-bpigt
Try use FormBuilderFieldState instead ChipsInputState
—
Reply to this email directly, view it on GitHub
https://github.com/flutter-form-builder-ecosystem/form_builder_extra_fields/issues/26#issuecomment-1209305828,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGFRU5L7JWBNKMBBARBG7ZLVYJD3DANCNFSM56AQIJRA
.
You are receiving this because you were mentioned.Message ID:
<flutter-form-builder-ecosystem/form_builder_extra_fields/issues/26/1209305828
@github.com>

Ok.
Please fill this template to know the exactly behaviour:
Environment
Package version:

Flutter doctor

Code sample

Description
Expected behavior:
Current behavior:
Steps to reproduce

This
Than that
Then

Images
Stacktrace/Logcat

Here's the document which has the details you asked for.

Message ID:
<flutter-form-builder-ecosystem/form_builder_extra_fields/issues/26/1209318673
@github.com>

Where?

I attached the document with the previous email, but I am putting the
details directly in this mail
Environment
OS: Widows 10
IDE : Android studio bumblebee 2021.1.1 patch 1
Package version:
Flutter doctor
C:\src\flutter\bin\flutter.bat doctor --verbose
[√] Flutter (Channel master, 3.1.0-0.0.pre.2166, on Microsoft Windows
[Version 10.0.14393], locale en-IN)
• Flutter version 3.1.0-0.0.pre.2166 on channel master at C:\src\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision 87f1684476 (30 hours ago), 2022-08-08 23:00:11 -0700
• Engine revision b6dd604d90
• Dart version 2.19.0 (build 2.19.0-76.0.dev)
• DevTools version 2.16.0
[√] Android toolchain - develop for Android devices (Android SDK version
32.1.0-rc1)
• Android SDK at C:\Users\Snow\AppData\Local\Android\sdk
• Platform android-32, build-tools 32.1.0-rc1
• Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
• Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[√] Visual Studio - develop for Windows (Visual Studio Community 2019
16.11.14)
• Visual Studio at C:\Program Files (x86)\Microsoft Visual
Studio\2019\Community
• Visual Studio Community 2019 version 16.11.32428.217
• Windows 10 SDK version 10.0.16299.0
[√] Android Studio (version 2021.1)
• Android Studio at C:\Program Files\Android\Android Studio
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 11.0.11+9-b60-7590822)
[√] VS Code (version 1.70.0)
• VS Code at C:\Users\Snow\AppData\Local\Programs\Microsoft VS Code
• Flutter extension version 3.44.0
[√] Connected device (2 available)
• Windows (desktop) • windows • windows-x64 • Microsoft Windows [Version
10.0.14393]
• Chrome (web) • chrome • web-javascript • Google Chrome 104.0.5112.81
[√] HTTP Host Availability
• All required HTTP hosts are available
• No issues found!
Process finished with exit code 0
Code
import **'package:flutter/material.dart';
*import *
'package:form_builder_extra_fields/form_builder_extra_fields.dart';
*void *main() {
runApp(*const *MyApp());
}
*class *MyApp *extends *StatelessWidget {
*const *MyApp({super.key});
// This widget is the root of your application.
@override
Widget build(BuildContext context) {
*return *MaterialApp(
title: 'Flutter Demo',
home: FormBuilderTouchSpin(
name: 'Volume',
),
);
}
}
Console output
Launching lib\main.dart on moto g31 in debug mode...
Running Gradle task 'assembleDebug'...
../../AppData/Local/Pub/Cache/hosted/
pub.dartlang.org/flutter_chips_input-2.0.0/lib/src/chips_input.dart:87:7:
Error: The non-abstract class 'ChipsInputState' is missing implementations
for these members:

TextInputClient.performSelector

Try to either

provide an implementation,

inherit an implementation from a superclass or mixin,

mark the class as abstract, or

provide a 'noSuchMethod' implementation.

class ChipsInputState extends State<ChipsInput>
^^^^^^^^^^^^^^^
/C:/src/flutter/packages/flutter/lib/src/services/text_input.dart:1172:8:
Context: 'TextInputClient.performSelector' is defined here.
void performSelector(String selectorName) {}
^^^^^^^^^^^^^^^
FAILURE: Build failed with an exception.

Where:

Script 'C:\src\flutter\packages\flutter_tools\gradle\flutter.gradle' line:
1159

What went wrong:

Execution failed for task ':app:compileFlutterBuildDebug'.

Process 'command 'C:\src\flutter\bin\flutter.bat'' finished with non-zero
exit value 1

Try:

Run with --stacktrace option to get the stack trace.

Run with --info or --debug option to get more log output.

Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 8s
Please let me know if you need more details
On Wed, Aug 10, 2022 at 6:16 PM Matias de Andrea @.***>
wrote:

Where?
—
Reply to this email directly, view it on GitHub
https://github.com/flutter-form-builder-ecosystem/form_builder_extra_fields/issues/26#issuecomment-1210625540,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGFRU5OVQKEWMCYVFN4EEDLVYOQD7ANCNFSM56AQIJRA
.
You are receiving this because you were mentioned.Message ID:
<flutter-form-builder-ecosystem/form_builder_extra_fields/issues/26/1210625540
@github.com>

We don't test or get support our packages with Flutter master branch.
I think that this error is because in this line of external package, don't implement TextInputClient.performSelector that is required on Flutter master version

@danvick Do you want maintain this packages or can transfer to this organization and verified publisher?

https://pub.dev/packages/flutter_touch_spin
https://pub.dev/packages/flutter_chips_input

Thanks

I am also facing this dependency issue. Any workaround would be appreciated, as I cannot get snapcraft to package.
I tried to specify older versions of form_builder_extra_fields, flutter_form_builder, and flutter_chips_input but there are all kinds of dependency issues and I don't know which combination of the older versions will yield a successful compile.

Flutter stable is now requiring :

TextInputClient.didChangeInputControl
TextInputClient.performSelector

How about supporting this, and maybe deploy all the changes to pub.dev ?

same here. after updating to 3.7 i got this error
flutter_chips_input-2.0.0/lib/src/chips_input.dart:87:7: Error: The non-abstract class 'ChipsInputState' is missing implementations for these members:
TextInputClient.didChangeInputControl
TextInputClient.performSelector
Try to either
provide an implementation,
inherit an implementation from a superclass or mixin,
mark the class as abstract, or
provide a 'noSuchMethod' implementation.

same here. after updating to 3.7 i got this error
flutter_chips_input-2.0.0/lib/src/chips_input.dart:87:7: Error: The non-abstract class 'ChipsInputState' is missing implementations for these members:
TextInputClient.didChangeInputControl TextInputClient.performSelector Try to either provide an implementation, inherit an implementation from a superclass or mixin, mark the class as abstract, or provide a 'noSuchMethod' implementation.

Same issue

i have some issue . and fixed it by add this code to file chips_input.dart , add this to class ChipsInputState
@override
void didChangeInputControl(s,h) {}
@override
void performSelector(String selectorName) {}


Remove sign-based floating-point comparison

If there are two finite values such that the absolute difference is less than zero but their sign is different, I think it should compare approximately true. I.e. areApproximatelyEqual(0.0, -0.00001) should evaluate to true.

Let's combine this with #1291 to address them altogether.


Count result may be wrong after a lot of insert and delete

Bug Report
Please answer these questions before submitting your issue. Thanks!
1. Minimal reproduce step (Required)
Repeated do a lot of insert rows and delete alls.
Keep select count after after every operation.

2. What did you expect to see? (Required)
Count result is right.
3. What did you see instead (Required)
Count result is wrong.
4. What is your TiFlash version? (Required)

/cc @lilinghai We need to add a test case for these kind of workload

/cc @lilinghai We need to add a test case for this kind of workload

ok. have discussed it with dezhu and I will add a test case.


⚠ ClamAV has degraded performance

In fe8959c, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 53 ms

Resolved: ClamAV performance has improved in bacf288.


[ISSUE] Rectangle overlap tutorial is either wrong or the problem statement is wrong

Article and Module Links
Tutorial: https://www.baeldung.com/java-check-if-two-rectangles-overlap
Describe the Issue
The tutorial does not define whether "touching" rectangles are considered to be overlapping or not. I would argue that in order for there to be overlap, at least one point needs to be inside the other rectangle.
To Reproduce
Failing testcase:
    @Test
    public void givenTwoTouchingRectangles_whenisOverlappingCalled_shouldReturnFalse() {
        Rectangle rectangle1 = new Rectangle(new Point(5, 0), new Point(17, 14));
        Rectangle rectangle2 = new Rectangle(new Point(0, 0), new Point(5, 14));
        assertFalse(rectangle1.isOverlapping(rectangle2));
    }

Expected Behavior
Either the article should be updated to explain that touching rectangles are considered to be overlapping (which I think is wrong), or the algorithm should be updated to take touching into account (which I'd prefer). This is pretty simple: the bounds checks should simply be made inclusive:
    public boolean isOverlapping(Rectangle other) {
        // one rectangle is to the top of the other
        if (this.topRight.getY() <= other.bottomLeft.getY() || this.bottomLeft.getY() >= other.topRight.getY()) {
            return false;
        }
        // one rectangle is to the left of the other
        if (this.topRight.getX() <= other.bottomLeft.getX() || this.bottomLeft.getX() >= other.topRight.getX()) {
            return false;
        }
        return true;
    }

Source: https://github.com/eugenp/tutorials/blob/master/core-java-modules/core-java-lang-math-2/src/main/java/com/baeldung/algorithms/rectanglesoverlap/Rectangle.java
Screenshots

Additional Context
I realise that not many people are likely to care about this. I only came across this issue because I'm tutoring a student who had trouble with a similar assignment and ended up copying the Baeldung solution.

Hey, @DamnedElric, thanks for the feedback.
This issue will remain open while we look into this.

Well, it really depends on the definition here imho and the point of view. But It's a good point to update the article in order to explicit this.
also
In physics, I guess you cannot state that two objects that are in the same exact spot, are not overlapping.
While practically speaking you can imagine two objects touching but not overlapping, you might imagine that this never really occurs.
but yeah, again, I think we should explicit better in the article

Hi @DamnedElric - we adjusted the code and the article to reflect both solutions, considering two rectangles touch if their borders overlap or not.


on MSVC dump() will hard crash for larger json

What is the issue you have?
I'm using json in a cross platform application to serialize a tree. On all platforms except MSVC I can create a large tree and serialize it with dump(4). However on Windows x64 MSVC Visual Studio 2017.9 when calling dump() I get exception

SEH exception with code 0xc00000fd

Please describe the steps to reproduce the issue. Can you provide a small but working code example?

Create a json tree of ~300MB RAM size
Serialize with dump()
Works on Linux and MacOSX, but not on MSVC x64

What is the expected behavior?
The serialized json should be returned as a std::string of ~24MB size. And the program should not hard crash. SEH exceptions can not easily be handled by the application.

And what is the actual behavior instead?

SEH exception with code 0xc00000fd

Which compiler and operating system are you using? Is it a supported compiler?
Windows x64 MSVC Visual Studio 2017.9 (current latest with all updates)

Did you use a released version of the library or the version from the develop branch?
Release version 3.6.1

Ok it turns out this issue is most likely invalid. We only get this issue if the depth of the json is in the range of 1024 levels or more. This is most likely a hard-coded limit somewhere in the library? Interestingly, the number of items is not the problem, so 1024 arrays inside each other will trigger the issue.

It could be due to a stack overflow during dump. Have you tried increasing the stack size to see if it has any effect. Here is an example of how it could be done by build flags:

Indeed the dump function is recursive, so a limited stack size can be a problem there.

I will close the issue as a know limitation of recursion depth that can most likely be worked around in compiler settings (though I have not tried).

Let's assume your program accepts JSON from untrusted sources. Meaning you have an basically arbitrary recursion depth. No matter which compiler settings you choose, at some point your stack will be too small.
I would like to see a compile time constant to limit the recursion depth of parse/dump and gracefully reject json documents not matching that. The default can be unlimited, while still allowing to have a stable program.


Filled method of the request returns true for empty array

Laravel Version: 9.26.1
PHP Version: 8.1.9
Database Driver & Version:

Description:
In the documentation we have:
If you would like to determine if a value is present on the request and is not empty, you may use the filled method:
Please pay attention to: ...and is not empty..
Steps To Reproduce:
$request = Request::create('/', 'GET', ['foo' => []]);
$this->assertTrue($request->filled('foo'));

Is it correct?

Yeah, this happened to me too before. I think is a bug or a misconception.
I had a query condition like this one:
...
 ->when($request->filled('array_with_ids'),
                fn($query) => $query->whereIn('id', $request->input('array_with_ids'))
  )
...

And passing an empty array, caused the query to fail.
I had to do this:
...
 ->when($request->filled('array_with_ids') && count($request->input('array_with_ids')),
                fn($query) => $query->whereIn('id', $request->input('array_with_ids'))
  )
...

This method is for strings specifically. I've sent in a PR to the docs to clarify this.

@driesvints do you plan to extend this for array also?

No, sorry. You can always attempt a PR if you want.

@driesvints got it, thank you for response.
Maybe clarify comment of the method also?
https://github.com/laravel/framework/blob/9.x/src/Illuminate/Http/Concerns/InteractsWithInput.php#L137


My plugin get group -> pex get uuid -> big delay 4-6k ms

Hi, i have a problem. PEX has a big delay. Check my MassiveLag log:
http://dl2.joxi.net/drive/2016/06/26/0002/2579/178707/07/fa5469bac9.png
http://dl2.joxi.net/drive/2016/06/26/0002/2579/178707/07/003d37db13.png
http://dl2.joxi.net/drive/2016/06/26/0002/2579/178707/07/8ea5f5ab9b.png
Delays also sometimes occur when entering '/pex user player' and other PEX commands.
I have about ~6000 users with different groups. (in permissions.yml, no mysql)
Latest pex, latest 1.8.8 spigot.

Yeah, this is pretty much a limitation of Craftbukkit. You'll also want to make sure that DardDonate looks up players in PEX by UUID rather than username since that can take a network request to resolve.


Windows as Entities

This is going to be both large and controversial, so I would very much appreciate feedback and input on this initial direction so that this turns out the best it can be.
I will update the PR description with more details as I progress.
Current Status: Check todo list at the bottom.
Objective

Make windows be Entities as that would allow users to operate on windows the same as they would already be doing with commands, systems and components
Fixes #4530

Solution
High Level Overview
Windows in Bevy has now become entities with components. The user can query these components to read information about the windows. The user can use change detection to check if various components has changed. These components are 'readony'-ish for the user.
To mutate the windows, the user will now use WindowCommands and apply Commands to the Entity with the window. The window backend is responsible handling these commands and updating the data on the components accordingly.
bevy_window changes
Window_id

WindowId has been replaced by Entity throughout the entire project

Window

Window has been split into multiple components:

Window (marker)
WindowCursor
WindowCursorPosition
WindowHandle
WindowPresentation
WindowModeComponent
WindowPosition
WindowResolution
WindowTitle
WindowCanvas
WindowDecoraded  (marker)
WindowCurrentlyFocused (marker)
WindowResizable (marker)
WindowTransparent (marker)
WindowMinimized (marker)
WindowMaximized (marker)

It is the responsibility of the window backend (bevy_winit) to create, update and remove these components
All fields on these components are private
The user should get data from the fields using the various getter methods exposed
All setter methods are named update_x_from_backend() or similar and should only be used by the window backend.

Windows

Windows has been removed completely
Getting all windows should now be done by querying for entities with the Window marker component

Query<Entity, With<Window>>

WindowCommandsExtension

Created WindowCommandsExtensions so that we can commands.window(entity) to get WindowCommands

Similar to how ÈntityCommands work

WindowCommand

Is no longer an enum
Has been split into structs that all impl Command
These commands will send themselves as events on command.write() for the window backend to handle

PrimaryWindow

PrimaryWindow is a new resource that the bevy_window plugin is responsible for creating when setting up the initial window.
It contains the Entity of the window that is considered the primary window
TODO: Consider if this should be a marker instead of a resource.

bevy_winit changes

TODO

bevy_render changes

TODO

bevy_text

Internal changes: Res -> Query

bevy_ui
Internal changes:

Res<Windows> -> Query
window.

Changelog

TODO: User facing changes to Window

Creating windows
Reading from windows
Writing / making changes to windows

TODO: User facing changes to Winit (This probably does not affect most users)
TODO: User facing changes to bevy_render

Migration Guide

WindowId -> Entity
window: Res<Window> -> windows: Query<&RelevantWindowComponent, With<Window>>
windows: Res<Windows> -> windows: Query<&RelevantWindowComponent, With<Window>>

Remaining Tasks

[x] Changes compiles :)
[ ] Fix timing issues between bevy_window, bevy_winit and bevy_render when it comes to window handle creation.
[ ] Check for system order ambiguities in Winit systems
[ ] Double check that we aren't regressing any features
[ ] Clear out remaining TODOs
[ ] Write tests that covers these changes / systems
[ ] Add documentation
[ ] Write changelog
[ ] Write migration guide

Status: Work has been picked up by @Aceeri over at #5589 as I have shifted focus onto other UI priorities

@Weibye I think you can just close this, since you can reopen it with just a click if needed.

@Weibye I think you can just close this, since you can reopen it with just a click if needed.

Fair point!


Allow fails in Remove large packages

Added || true at the end of large file removal, so that if one step of it fails, the clean-up job would continue.

Thanks for the replay @jlumbroso! 😃
Regarding your first question, I can think of some scenarios where this action could be used as a test, so a failure here could be important, but I think that the general use case only benefits from not stopping a CI due to failed deletion, which hinders development.
Well, we use this github action in (at least) one of our repos and got some CI fails (example here) due to a false return code in ubuntu. I think that the users can tolerate a difference of a few mb's extra, but have a CI fail a few times due to uninstallation error may discourage users to use this tool. Also, I recall talking to a colleague and postulating that this issue may had something to do with the new ubuntu release and additional packages still being adapted to it (could not be verified, though), so this PR (which is already implemented in our software for a while using my fork) gave us the opportunity to continue developing even in a (somewhat) unstable environment associated with open source code (which we absolutely love!).

Solution of @kfir4444 looks good can we approve this PR please many user facing issues

Thanks @pratik-techholding!
I do not think we should hate the code review process, if you want, you can use my clone until this PR is merge. I can ping you when it's merged 😄

Maybe this could fix #9.

Hi, when do we expect to get the fix merged in?

Hi @jlumbroso - do you have any subsequent concerns and/or blockers to getting this PR merged?
If you need extra help maintaining this repository, I'd be happy to volunteer to assist (I have experience with authoring & maintaining several GHAs); just let me know.

@jlumbroso as @ChrisCarini, I can also help with maintenance :smile:

For everyone else struggling with issues, you can use my fork for now (this is what we do in our repos) until this chage becomes official. I can tag you here when this PR is merged so you can return to the original.
@AurevoirXavier @yinweisu @ChrisCarini

Thanks for the replay @jlumbroso! 😃 Regarding your first question, I can think of some scenarios where this action could be used as a test, so a failure here could be important, but I think that the general use case only benefits from not stopping a CI due to failed deletion, which hinders development.

Don't you get for free the error check if you set "continue-on-error: true" in the action?

This would also fix #4. I assume @jlumbroso has just not had a chance to look at this.

Sorry for the delay. @ChrisCarini Thank you for addressing my concerns with your change.

Sorry for the delay.

No worries!

@ChrisCarini Thank you for addressing my concerns with your change.

Of course! FWIW, my offer to help you maintain this repo (ref: https://github.com/jlumbroso/free-disk-space/pull/8#issuecomment-1685200176 ) (approving/merging PRs, releasing new versions, etc) still stands. :) Just let me know! Happy to help.


re-enable numba

after switchting on cupy, we had to disable numba. we should aim at re-enabling it to maybe lower the overhead produced by the core routines.

Wont fix. Closing.


Styling doesn't support bottom placement

This is pretty minor.
I prefer placing the devtools panel at the bottom right of the browser window, because I find that it can get in the way of the web app behind it, when the panel is in the top right corner. It's easy to position the panel by defining bottom and right, but the result in top notch, because the panel doesn't have a border and rounded corners at the top of it.
<DevTools position={{ bottom: 0, right: 20 }}/>

One solution would be to have the border and border radius on all four corners of the panel, and then position the panel a couple of pixels off screen. Somewhat like this: https://github.com/janaagaard75/mobx-react-devtools/commit/6a90dd46246b49c52fcda664c1cc23045625dbe8

PR would be welcome


Remove System.CommandLine dependency

System.CommandLine package is no longer needed for Crossgen2.

Apparently this repo still consumes a release/5.0 build of Crossgen2. The dependency is removed in 6.0 builds.
  <New>
    <Usage Id="Microsoft.CSharp" Version="4.4.1" File="artifacts/src/runtime.cf258a14b70ad9069470a108f13765e0e5988f51/artifacts/obj/coreclr/ILVerify/project.assets.json" />
    <Usage Id="Microsoft.CSharp" Version="4.4.1" File="artifacts/src/runtime.cf258a14b70ad9069470a108f13765e0e5988f51/artifacts/obj/coreclr/crossgen2/project.assets.json" />
    <Usage Id="System.CommandLine" Version="2.0.0-beta1.20253.1" File="artifacts/src/runtime.cf258a14b70ad9069470a108f13765e0e5988f51/artifacts/obj/coreclr/ILVerify/project.assets.json" IsDirectDependency="true" />
    <Usage Id="System.CommandLine" Version="2.0.0-beta1.20253.1" File="artifacts/src/runtime.cf258a14b70ad9069470a108f13765e0e5988f51/artifacts/obj/coreclr/crossgen2/project.assets.json" IsDirectDependency="true" />
  </New>

@crummel to follow up

We are discussing whether want to use System.CommmandLine in other Runtime packages. If yes, we will add a reference there.

https://github.com/dotnet/core/issues/5853#issuecomment-770124124 says that the SDK uses System.CommandLine. If that's true, I dont think we can drop it from source-build.

@omajid My understanding is that System.CommandLine should be listed in SDK's eng/Version.Details.xml then.

Triage: @crummel to follow up on what actually needs this and file upstream PR to fix Version.Details.xml.

This looks like it includes updates for 6.0.  We should pick those changes up with ArPow.  This should not be needed in source-build master.
Thanks for bringing this to our attention.


MRG: Better filtering

Closes #4045.
WIP because it needs to:

[x] respect different trans_bandwidths, e.g. for band-pass, band-stop, and notch filters (by changing length in the building function).
[x] add a bit more narrative (preferably with refs?) to the tutorial
[x] actually test band-stop, band-pass, and notch behaviors
[x] force length='auto' in firwin mode (otherwise the transitions won't work properly)
[x] update lots of tests to not throw warnings
[x] update whats_new.rst

This also fixes a bug where ref_meg channels were not being filtered by default in raw.filter.

@mmagnuski can you see if fir_design='firwin' gives you better performance?

This one should be ready for review/merge

Updated tutorial:
https://4245-1301584-gh.circle-artifacts.com/0/home/ubuntu/mne-python/doc/_build/html/auto_tutorials/plot_background_filtering.html#defaults-in-mne-python

Thanks, Eric! Not much time today unfortunatelly, but I'll test that tomorrow!

That's fine, let's take our time and get it right this time :)

appveyor is not happy for good reasons

CIs are happy, and I have verified that our filter should now match EEGLAB. But it would still be good if @mmagnuski could double-check

Codecov Report

Merging #4113 into master will decrease coverage by 0.03%.
The diff coverage is 98.19%.

@@            Coverage Diff             @@
##           master    #4113      +/-   ##
==========================================
- Coverage   86.18%   86.15%   -0.04%
==========================================
  Files         354      354
  Lines       63748    63889     +141
  Branches     9711     9730      +19
==========================================
+ Hits        54940    55042     +102
- Misses       6130     6157      +27
- Partials     2678     2690      +12

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 20cfc04...45d451e. Read the comment docs.

Yes, sorry @Eric89GXL, this time it really is tomorrow (ie within 24 hours). But I am sure Eric did a gret job, so you could merge and I'll confirm that all is good anyway. :)

@Eric89GXL
I hereby confirm that it works lovely now. :)

don't wait for me to merge if you guys are happy

@jaeilepp docstring updated, can you see if it's good enough?

Looks good, thanks @Eric89GXL


Update Swisscom provider

Add Geneva as a supported location.

Hi @mialdrid - Would you make this content update in the private repo? We'll close this pull request. Thanks!

Hi @mialdrid - Would you make this content update in the private repo? We'll close this pull request. Thanks!


[docs] Improve packages description

A continuation of https://github.com/mui-org/material-ui/pull/24224#commitcomment-45599092
Current

https://www.npmjs.com/search?q=material-ui
Proposal

it's telling that you think we have accessible components but make your PR description inaccessible by using images for text only changes.

@eps1lon The PR description isn't following Material Design, should we remove mention of it from the packages name 😄?

it's telling that you think we have accessible components but make your PR description inaccessible by using images for text only changes.

@eps1lon The PR description isn't following Material Design, should we remove mention of it from the packages name 😄?

more accessible

👍

more accessible

👍

Completely unstyled, fully accessible UI components

https://headlessui.dev/ 🙃

Completely unstyled, fully accessible UI components

https://headlessui.dev/ 🙃

Completely unstyled, fully accessible UI components

headlessui.dev

Not sure what you're getting at here? That might be true, might be the same problem as I already described.

Completely unstyled, fully accessible UI components

headlessui.dev

Not sure what you're getting at here? That might be true, might be the same problem as I already described.

the same problem as I already described

@eps1lon I've only seen comments about how the components are "not accessible, despite best intentions", but nothing more specific. If there are problems that you're aware of you could either open an issue, or submit a PR.

the same problem as I already described

@eps1lon I've only seen comments about how the components are "not accessible, despite best intentions", but nothing more specific. If there are problems that you're aware of you could either open an issue, or submit a PR.

If there are problems that you're aware of you could either open an issue, or submit a PR.

What are you talking about? I'm working constantly on the a11y aspect. Why are you trying to imply I'm not actually working on these issues?

If there are problems that you're aware of you could either open an issue, or submit a PR.

What are you talking about? I'm working constantly on the a11y aspect. Why are you trying to imply I'm not actually working on these issues?


Add missing *Url fields to Instance struct

I asked on the Eureka mailing list about the missing *URL fields in the REST documentation and they updated them (the XSD doc at the end). This PR adds them to the Instance struct.

:+1: Apologies again for being slow to get these tested.


inapplicable document.

[Try to find in example "contoso4" for the user name!!!
Note
The SFTP username is storage_account_name.username. In the example above the storage_account_name is "contoso4" and the username is "contosouser." The combined username becomes contoso4.contosouser for the SFTP command.]

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

ID: e7d93704-cc4c-04a0-beb4-84983853cbb1
Version Independent ID: c999a603-b02d-a56d-1bef-0cd4916b6a2b
Content: Connect to Azure Blob Storage using SFTP - Azure Storage
Content Source: articles/storage/blobs/secure-file-transfer-protocol-support-how-to.md
Service: storage
Sub-service: blobs
GitHub Login: @normesta
Microsoft Alias: normesta

@SFGTA
Thanks for your feedback! We will investigate and update as appropriate.

Hi @SFGTA
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.
@normesta
Can you please check and add your comments on this doc update request as applicable.

HI @SFGTA in the example, "contoso4" is the account name (not username). The SFTP username is the combination of the account name along with a "." followed by a username. I believe that the example featured in the screenshot shows this. Thank you for having us take a second look though!
#please-close

#please-close

in your document user cannot figure out what to do. Don't you think you
should update description or screenshot? Azure blob is not user friendly at
all and you don't make it easier.
Thank you.
вт, 7 февр. 2023 г., 22:27 Norm Estabrook @.***>:

HI @SFGTA https://github.com/SFGTA in the example, "contoso4" is the
account name (not username). The SFTP username is the combination of the
account name along with a "." followed by a username. I believe that the
example featured in the screenshot shows this. Thank you for having us take
a second look though!
#please-close
—
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/104795#issuecomment-1421399745,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVU7534VH2L4CCDGZG6SQ3DWWKVZ3ANCNFSM6AAAAAAURIZJWU
.
You are receiving this because you were mentioned.Message ID:
@.***>

@SFGTA
Thanks for your feedback. Please add your ideas/feedback in below link, so our production team can review it and update the same.
Ideas/feedback · Community (azure.com)


Autobump every hour never works

Description
On the desktop, it works pretty consistently.  But on the laptop, it never works.  They aren't synced, btw.
Console Errors
("." + hostname1).replace(...).match(...) is null 2 EventHandlers.js:35
AbortError: Actor 'Conduits' destroyed before query 'RuntimeMessage' was resolved ConduitsParent.jsm:362
Element.releaseCapture() is deprecated. Use Element.releasePointerCapture() instead. For more help https://developer.mozilla.org/docs/Web/API/Element/releasePointerCapture menupopup.js:169:13
sendRemoveListener on closed conduit {71de700c-ca62-4e31-9de6-93e3c30633d6}.13606456393730 ConduitsChild.jsm:112
sendRemoveListener on closed conduit {71de700c-ca62-4e31-9de6-93e3c30633d6}.13606456393749 ConduitsChild.jsm:112
this.window.gBrowserInit is undefined ext-browser.js:1106
sendRemoveListener on closed conduit {71de700c-ca62-4e31-9de6-93e3c30633d6}.13606456393766 ConduitsChild.jsm:112
System (please complete the following information):

ESGST Version: Extension v8.9.0
Browser + Version: Firefox v100.0.2 (64-bit)

well, this is strange.  it started working perfectly today.


Support for hateoas style

Hi,
would you be able to suggest how to write a rule that accepts hateoas fields, like "_link" and "_embedded", but at the same time validate the casing of other properties in a schema?
For instance, I'd like to enforce a camel casing for this entity, but also accept words starting with "_".
components:
  schemas:
    Application:
      type: object
      properties:
        applicationId:
          type: string
        _links:
          type: object
          properties:
            self:
              $ref: '#/components/schemas/Link'
    ApplicationResponse:
      properties:
        application:
          $ref: '#/components/schemas/Application'
    Link:
      type: object
      properties:
        href:
          type: string

Thanks

Hi @michelecurioni, I think you have two options here. Depending on the version of the validator you're using, we use Spectral's casing function to perform the validations. You may be able to customize the rule to provide greater specificity about the convention used.
If that doesn't work, a less elegant but still effective solution would be to use Spectral overrides. The tool supports this feature - you can ignore properties on a case-by-case basis.

I created this custom rule that seems to do the trick:
'property-case-convention': {
      description: 'Property names must follow camel case',
      message: '{{error}}',
      resolved: true,
      given: schemas,
      severity: 'error',
      then: {
        function: propertyCaseConvention,
        functionOptions: {
          type: 'camel',
          separator: {
              char: '_',
              allowLeading: true
          }
        }
      }
    }


XYPlot fails to infer x domain from VerticalBarSeries when data is un-ordered

Hi there, thanks for your work on this library.
I am populating a stacked bar chart with un-ordered data. All x values in the data set range between 2000 and 2100, but the x-axis is being rendered as from 1960 to 2120. Specifying the xDomain does not change this. The calculated width of the bars also seems to vary unpredictably depending on the order of the data.
Replicated with fake data here: https://codesandbox.io/s/kxz1wx28v3 - refresh the page multiple times to see the x axis and bar widths vary as the data is shuffled.
Ordering the data fixed the issue for me, but it took me a while to work out what was going on!

I'm seeing this too, found this issue after creating an even simpler example spending considerable time trying to figure out what was happening... https://codesandbox.io/s/dank-grass-ls4o7


Issue with cancel on Samsung S10

While using custom fingerprint screen with Samsung S10, user is unable to cancel the screen, because S10 has custom on screen layout. It also counts as a try and in our case we limit fingerprint tries to 3 times, so after 3 cancel attempts, screen closes. Attached a video in email.
expected behaviour: Cancel on S10 fingerprint overlay also  closes custom modal.
or expected behaviour 2: Cancel on S10 fingerprint does not count as a try.

Same on Huawei p30 pro. Could you resolve it?, @narusevic ?

Haven’t found anything useful regarding this


Need good explanation

Hello,
I have read you docs, looked at sample apps, and I still can not understand how to use it, sorry.
Could someone write short and step by step example on how to add it to empty HTML page and use this gantt?
(what js and css files should be added, how to start gantt on page)
Thank you

Just imagine, I have opened the notepad.exe and started writing html and javascript using this angular-gantt - so what should I add and reference for (scripts, css)?
Thank you

ok, just in case someone will need it:

a few things I was not able to figure out so I just commented them:
(the /yourInitializationScript.js file content)
(function () {
var app = angular.module('AppGant', [
'gantt', // angular-gantt.
'gantt.sortable',
'gantt.movable',
'gantt.drawtask',
'gantt.tooltips',
'gantt.bounds',
'gantt.progress',
'gantt.table',
//'gantt.tree',
'gantt.groups',
'gantt.dependencies',
'gantt.overlap',
'gantt.resizeSensor',
//'ngAnimate',
//'mgcrea.ngStrap'
]);
app.controller(....).service(...);
})();
finally, I decided to not use this Gantt framework (for now at least) as it is very hard to understand and to support it, also there is no good documentation.
Thank you.

Yes don't use it, thank you. There's docs, but it seems your are not equiped to find them or read them.


Add a sample of In-Graph

We need to know how in-graph application runs. If our current interfaces fit

in-graph and between-graph roughly relation from our sample code:
Common: both needs to build ClusterSpec
Differences:

in-graph needs to have a list of train operation object while between graph only have one
in-graph needs a worker host as master while between graph doesn't need this

between-graph also needs a master which is passed in asFLAG.target and actually a worker host. I will update the between-graph example to align with in-graph

yeah. thanks for the clarification. Does that means there are multiple master when running between-graph while only one for in-graph?

clients are free to submit graph to one or multiple masters in both between-graph and in-graph replication.  Master is for communication and distributing computation and should not be bound to the application type.

I see. and do you think that we should make our current running job process optional? or just remove it?

I'd rather remove it. The current job.py is not correct since it launches two client scripts sequentially for between-graph replication. Users will want to run them concurrently. Another thing is if we interrupt the client we also kill the cluster while users may want to submit another application.

fixed in https://github.com/Intel-bigdata/TensorFlowOnYARN/commit/6f651060fcf21b0474efcbe6abf434020ad1ddf7. Discussions on launching TensorFlow client can continue at #17. Closing this.


amis: enable setting ami boot mode on registration

Motivation for this change
This is important since legacy bios mode is still the default for Intel and AMD based instances on AWS. That is, even if your image is setup to use UEFI on the OS level, the AMI will still use BIOS unless the boot mode is explicitly set during registration.
Things done
Built and uploaded both kinds of AMIs, regular and ZFS using ec2.efi = true and ensured they both booted fine. Also tested systemctl kexec as a bonus.

Looks good to me, and the bit about terraform is right. I already have
the files to do it, and do it for my own use case today. Upstreaming it
in place of a script is hard though :).
On November 11, 2021, GitHub @.***> wrote:

@jonringer approved this pull request.
diff LGTM
cc @grahamc https://github.com/grahamc

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<https://github.com/NixOS/nixpkgs/pull/145442#pullrequestreview-
803490549>, or unsubscribe
<https://github.com/notifications/unsubscribe-
auth/AAASXLHB4DT22AZDJPSTBSDULNXMHANCNFSM5HZHUM4A>.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-
email&mt=8&pt=524675> or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-
email%26utm_medium%3Demail%26utm_source%3Dgithub>.


There is a bug in com.google.cloud.speech.v1p1beta1 setEnableSpeakerDiarization feature

Environment details

Speech to Text API
OS type and version: Win10
Java version: JDK8
google-cloud-java version(s):

<dependency>
	<groupId>com.google.cloud</groupId>
	<artifactId>google-cloud-speech</artifactId>
	<version>1.6.0</version>
</dependency>

Steps to reproduce

I run the sample code of SpeakerDiarization

import com.google.api.gax.longrunning.OperationFuture;
import com.google.cloud.speech.v1p1beta1.LongRunningRecognizeMetadata;
import com.google.cloud.speech.v1p1beta1.LongRunningRecognizeResponse;
import com.google.cloud.speech.v1p1beta1.RecognitionAudio;
import com.google.cloud.speech.v1p1beta1.RecognitionConfig;
import com.google.cloud.speech.v1p1beta1.RecognitionConfig.AudioEncoding;
import com.google.cloud.speech.v1p1beta1.SpeechClient;
import com.google.cloud.speech.v1p1beta1.SpeechRecognitionAlternative;
import com.google.cloud.speech.v1p1beta1.SpeechRecognitionResult;
import com.google.cloud.speech.v1p1beta1.WordInfo;

public static void main(String args[]) {
	try (SpeechClient speech = SpeechClient.create()) {
		RecognitionConfig config = RecognitionConfig.newBuilder()
					.setEncoding(AudioEncoding.LINEAR16)
				    .setLanguageCode(languageCode)
					.setSampleRateHertz(16000)
		            .setEnableWordTimeOffsets(true)
		            .setEnableAutomaticPunctuation(true)
				    .setEnableSpeakerDiarization(true)
		            .setDiarizationSpeakerCount(2)
		            .build();
		RecognitionAudio audio = RecognitionAudio.newBuilder().setUri(gcsUriEN).build();

		OperationFuture<LongRunningRecognizeResponse, LongRunningRecognizeMetadata> response = speech
				.longRunningRecognizeAsync(config, audio);
		while (!response.isDone()) {
			System.out.println("Waiting for response...");
			Thread.sleep(10000);
		}

		for (SpeechRecognitionResult result :  response.get().getResultsList()) {
                            SpeechRecognitionAlternative alternative = result.getAlternativesList().get(0);
			System.out.format("Transcription: %s\n", alternative.getTranscript());
			for (WordInfo wordInfo : alternative.getWordsList()) {
				System.out.format("%s\t%s.%s sec - %s.%s sec\n", wordInfo.getWord(),
						wordInfo.getStartTime().getSeconds(),
						wordInfo.getStartTime().getNanos()/100000000,
						wordInfo.getEndTime().getSeconds(),
						wordInfo.getEndTime().getNanos()/100000000);
			}
		}
	}
}

I found different SpeechRecognitionResult result has the same words in respective wordslist alternative.getWordsList())

For example, the first result has 10 words, the second result has 20 words, console log shows that the second wordslist contains the first one.

Log is as follows:

Speaker Tag 0: we are here to discuss blar blar omit here...
we	0.5 sec - 1.0 sec
are	1.0 sec - 1.2 sec
here	1.2 sec - 1.4 sec
to	1.4 sec - 1.5 sec
......

Speaker Tag 1: where I mean the circumcenter blar blar omit here...
we	0.5 sec - 1.0 sec
are	1.0 sec - 1.2 sec
here	1.2 sec - 1.4 sec
to	1.4 sec - 1.5 sec
discuss	1.5 sec - 1.9 sec
......
where	33.2 sec - 33.7 sec
I	33.7 sec - 33.8 sec
mean	33.8 sec - 33.9 sec
the	33.9 sec - 34.1 sec
......

This bug only occurs under I configurate setEnableSpeakerDiarization feature, using com.google.cloud.speech.v1p1beta1 package.
After I comment the code, all wordslist recover normal.
.setEnableSpeakerDiarization(true)
.setDiarizationSpeakerCount(2)

Thanks!

@lizekui this seems like a bug in the Speech API itself, and not the Java client.  You would be better served to ask the Speech team's support about this.  https://cloud.google.com/support/docs/issue-trackers has a way to create bugs for the Speech team directly.  Other forms of support can be found here: https://cloud.google.com/speech-to-text/docs/support.

Thanks dear @sduskis ! You are so technical professional , I would propose an issue on your wonderful advice website, just let me find a new world hah  :-)
btw, may I close this issue, or keep it open?

Please star at the issue created for this: https://issuetracker.google.com/136691551 to get further updates from the Speech team.


Assignment to global regex match data is not allowed

Closes #12711

Might want to check multiassign_left_hand too, to disallow multiple assignment


Error About: FIRMessagingDelegate's-messaging:didReceiveMessage: not implemented

Hello plugin works on android but on IOS I am getting message as below and notification does not appear
Time	Device Name	Type	PID	Tag	Message
Feb 14 18:15:30	Kerboo-iPhoneu	Notice	333	MyAppName.iOS(GoogleUtilities)	6.5.0 - [Firebase/Messaging][I-FCM002019] FIRMessaging received data-message, but FIRMessagingDelegate's-messaging:didReceiveMessage: not implemented

Thank you in advance.
Bug Information
Version Number of Plugin: 3.1.6
Device Tested On: IPhone 6 Plus IOS 12.4.5
Simulator Tested On: Yes
Version of VS: 2019
Version of Xamarin: <IP_ADDRESS>1640
Versions of other things you are using:
Xamarin Forms Maps <IP_ADDRESS>1640
Code Snippets
AppDelegete.cs
[Register("AppDelegate")]
    public partial class AppDelegate : global::Xamarin.Forms.Platform.iOS.FormsApplicationDelegate
    {
        //
        // This method is invoked when the application has loaded and is ready to run. In this
        // method you should instantiate the window, load the UI into it and then make the window
        // visible.
        //
        // You have 17 seconds to return from this method, or iOS will terminate your application.
        //
        public override bool FinishedLaunching(UIApplication app, NSDictionary options)
        {

            Rg.Plugins.Popup.Popup.Init();

            global::Xamarin.Forms.Forms.Init();
            Xamarin.FormsMaps.Init();

            FFImageLoading.Forms.Platform.CachedImageRenderer.Init();
            FFImageLoading.Forms.Platform.CachedImageRenderer.InitImageSourceHandler();

            LoadApplication(new App(new iOSInitializer()));

            #region //> Plugin: Firebase Push Notification

            FirebasePushNotificationManager.Initialize(options, new NotificationUserCategory[]
            {
                new NotificationUserCategory("message",new List<NotificationUserAction> {
                    new NotificationUserAction("Reply","Reply",NotificationActionType.Foreground)
                }),
                new NotificationUserCategory("request",new List<NotificationUserAction> {
                    new NotificationUserAction("Accept","Accept"),
                    new NotificationUserAction("Reject","Reject",NotificationActionType.Destructive)
                })

            });

            #endregion

            return base.FinishedLaunching(app, options);

        }

        #region //> Plugin: Firebase Push Notification

        public override void RegisteredForRemoteNotifications(UIApplication application, NSData deviceToken)
        {
            FirebasePushNotificationManager.DidRegisterRemoteNotifications(deviceToken);

        }

        public override void FailedToRegisterForRemoteNotifications(UIApplication application, NSError error)
        {
            FirebasePushNotificationManager.RemoteNotificationRegistrationFailed(error);
        }

        // To receive notifications in foregroung on iOS 9 and below.
        // To receive notifications in background in any iOS version
        [Export("messaging:didReceiveMessage:")]
        public override void DidReceiveRemoteNotification(UIApplication application, NSDictionary userInfo, Action<UIBackgroundFetchResult> completionHandler)
        {
            // If you are receiving a notification message while your app is in the background,
            // this callback will not be fired 'till the user taps on the notification launching the application.

            // If you disable method swizzling, you'll need to call this method.
            // This lets FCM track message delivery and analytics, which is performed
            // automatically with method swizzling enabled.
            FirebasePushNotificationManager.DidReceiveMessage(userInfo);
            // Do your magic to handle the notification data
            System.Console.WriteLine(userInfo);

            completionHandler(UIBackgroundFetchResult.NewData);
        }

        #endregion

    }

    public class iOSInitializer : IPlatformInitializer
    {
        public void RegisterTypes(IContainerRegistry containerRegistry)
        {
            // Register any platform specific implementations
        }
    }
}

info.plist
<key>CFBundleShortVersionString</key>
    <string>1</string>
    <key>CFBundleVersion</key>
    <string>1</string>
    <key>UILaunchStoryboardName</key>
    <string>LaunchScreen</string>
    <key>UIStatusBarStyle</key>
    <string>UIStatusBarStyleLightContent</string>
    <key>UIViewControllerBasedStatusBarAppearance</key>
    <false/>
    <key>UIBackgroundModes</key>
    <array>
      <string>remote-notification</string>
    </array>
    <key>FirebaseAppDelegateProxyEnabled</key>
    <false/>
    <key>CFBundleURLTypes</key>
    <array>
      <dict>
        <key>CFBundleTypeRole</key>
        <string>Editor</string>
        <key>CFBundleURLSchemes</key>
        <array>
          <string>com.google.fcm</string>
        </array>
      </dict>
    </array>

Paylod
{
    "data": {
        "message" : "my_custom_value",
        "other_key" : true,
        "body":"test"
     },
     "notification": {
       "body" : "hello",
       "title": "firebase",
       "sound": "default",
        "content_available" : true
     },
     "priority": "high",
     "to" : "dmAiSSkV-L8:APA91bHkoIFANWWBFegTFR9DpgIMESJnTmd8A06lb14202Y2dBHGpXJP5AyLP4GLLx2jsmJMnOzv_NK5EFPX9tWxviRLiqaFeY0BSzWTW35OxwQENMLh6zR0XXgh8VIp83gBN7OXlmKn"
}

If you want the notification to show up payload should be:
{
    "data": {
        "message" : "my_custom_value",
        "other_key" : true,
        "body":"test"
     },
     "notification": {
       "body" : "hello",
       "title": "firebase",
       "sound": "default"
     },
     "priority": "high",
     "to" : "dmAiSSkV-L8:APA91bHkoIFANWWBFegTFR9DpgIMESJnTmd8A06lb14202Y2dBHGpXJP5AyLP4GLLx2jsmJMnOzv_NK5EFPX9tWxviRLiqaFeY0BSzWTW35OxwQENMLh6zR0XXgh8VIp83gBN7OXlmKn"
}

content_available is for silent notifications

I have tested paylod which you wrote but result is same :(

I am getting error from real iphone device console as below
6.5.0 - [Firebase/Messaging][I-FCM002019] FIRMessaging received data-message, but FIRMessagingDelegate's-messaging:didReceiveMessage: not implemented

I am not using code as below in info.plist
It can be a matter?
<key>CFBundleIconFiles</key>
	<array>
		<string>Icon-60@2x</string>
		<string>Icon-60@3x</string>
		<string>Icon-76</string>
		<string>Icon-76@2x</string>
		<string>Default</string>
		<string>Default@2x</string>
		<string>Default-568h@2x</string>
		<string>Default-Portrait</string>
		<string>Default-Portrait@2x</string>
		<string>Icon-Small-40</string>
		<string>Icon-Small-40@2x</string>
		<string>Icon-Small-40@3x</string>
		<string>Icon-Small</string>
		<string>Icon-Small@2x</string>
		<string>Icon-Small@3x</string>
	</array>


[release-4.8] bootstrap/manifests: set CVO annotations

Make sure all manifests have necessary annotations. Cherry-pick of #176 on release-4.8

/test e2e-aws-upgrade

/test e2e-aws-upgrade

/test e2e-aws-upgrade

/test e2e-aws-upgrade

/retest

/retest

/retest


Dynamic Linking Error 193 with x64 Node

Working on some common code and I am getting the following error calling a small dll for licensing where as my colleagues are not.
library  ./resources/extras/licensing/licensing.dll
Error: Dynamic Linking Error: Win32 error 193
    at new DynamicLibrary (C:\Users\DummyUser\AppPath\app\node_modules\ffi-napi\lib\/dynamic_library.js:74:11)
    at Library (C:\Users\DummyUser\AppPath\app\node_modules\ffi-napi\lib\/library.js:47:14)
    at App.createWindow (C:\Users\DummyUser\AppPath\app\/main.dev.ts:131:20)
    at App.emit (events.js:215:7)

The calling code is the following
import { screen, app, BrowserWindow, ipcMain } from 'electron';
import { autoUpdater } from 'electron-updater';
import { Library } from 'ffi-napi';

const libPath = `./resources/extras/licensing/licensing.dll`;
const libWeb = Library(libPath, {
  validate: ['int32', ['int32']]
});

The dll should be a 32 for reasons related to the licensing code, but still I've tried building it as x64, and running it but I get the same issue.  This appears to be similar to the unresolved issue #425 .
From googling it appears similar to the following link, but the fix was recommended to upgrade the dll to x64, which doesn't work
https://stackoverflow.com/questions/53051284/error-dynamic-linking-error-win32-error-193-while-binding-to-eztwain3-dll-libr
Because it is working on other's machines we are thinking it may have something to do with the build environment.  I have tried re-installing npm & windows build tools.
Versions of tools I'm using are below, appreciate any experience or insight someone may have.

Windows 10 (64)
node version 6.14.8
ffi-napi version 3.0.1

There appeared to be an error in my Windows environment and the issue was fixed by a full windows reset.  Nuclear option, but effective.


Fixed OOM issues when evaluating/predicting

Fixes #693
Goals :soccer:

This PR fixes the issues with OOM when using Trainer.evaluate() or Trainer.predict() on large datasets

Implementation Details :construction:

Trainer.evaluate() or Trainer.predict()  methods both call the Trainer.evaluation_loop() method. Besides computing metrics, it also keeps accumulating the predictions (batch size, item cardinality) and labels (batch size) tensors as batches are processed.
When the item cardinality is high, the accumulated predictions become too large very quickly and leads to OOM on CUDA. If T4RecTrainingArguments.eval_accumulation_steps is set, each N steps the accumulated predictions/labels will be moved from the GPU memory to CPU larger memory. But then you might get OOM on CPU too after some steps. For example, for a batch size of 1024 and item cardinality of 300000, each batch would require 1.14 GB of memory, and be accumulated until OOM occurs.
In order to avoid the OOM issues, the T4RecTrainingArguments.predict_top_k argument had been created since the former versions of the library. It limits predictions to accumulate only the topk predictions in memory or in the resulting tensor when you use trainer.predict(). But it was set to None by default, which means that the user would have to be aware of it and set it manually to avoid these issues when evaluating and predicting on large datasets. Note: The predict_top_k does not affect the metrics calculation, but does affects the memory consumption of trainer.evaluate().
This PR does the following changes:

Sets T4RecTrainingArguments.predict_top_k = 100 as a default value that will avoid OOM issues in most cases, and still returns a reasonable number of top-k predictions in model.predict().
IMPORTANT: This changes the default output of trainer.predict() API, that returns a PredictionOutput object with a predictions property. Before this change, when the predict_top_k option was not set (default) the predictions property was as 2D tensor (batch size, item cardinality) with the scores for all the items. As now we set   T4RecTrainingArguments.predict_top_k by default, the predictions property returns a tuple with (top-100 predicted item ids, top-100 prediction scores) .
Changes Trainer.evaluation_loop() to fix and clarify the interplay between the T4RecTrainingArguments.predict_top_k option and the model.top_k property. The model.top_k was created recently to allow for the model to return top-k prediction scores/item ids instead of all items, in order to serve T4Rec models more efficiently in Triton.

The model.top_k only returns the top-k items in inference mode (when not training or evaluating), which is the case for both Triton inference and when trainer.predict(). So, setting model.top_k limits to that amount the number of predictions we can get in trainer.predict(). For that reason, we raise now an exception if T4RecTrainingArguments.predict_top_k > model.top_k

Testing Details :mag:

Added  test_trainer_predict_top_k_x_top_k to check for all possible combinations between T4RecTrainingArguments.predict_top_k and model.top_k values
Added test_trainer_predict_topk to test if the number of top-k preds matches the T4RecTrainingArguments.predict_top_k and also to ensure the test breaks if the default value changes from 100

@gabrielspmoreira the test_stochastic_swap_noise_with_tabular_features is failing. this test is floppy. can we increase the delta abs in the assert of the test to avoid any fails?


Add missing french translation

Same thing here for the quotes and escaping.

Same thing here, it needs to be created against the MASTER REPO, not the sub-modules.


Update IAM Group Terraform configuration file

to: @airbnb/binaryalert-maintainers
size: small
Background
This notes the available environment variables in the IAM Group Terraform file.
Changes

Minor documentation update pulled from the Terraform AWS Provider docs

Testing
This was tested by copying and pasting the Terraform file from the docs and running it locally.

Coverage remained the same at 92.2% when pulling ff4b93746d3612a66f78ba230a63943db21f2b1d on capsule8:iam-group-tf-update into 1e06b943d2b0e3133a593cc7e4f9398f3882ce74 on airbnb:master.

@mmwtsn I could be missing something, but this appears to only update docs.. not actual TF code?

@ryandeivert correct. This file is only in the docs as far as I've seen. Creating the IAM Group with this file the first step you see when you start the Getting Started guide. I think this should be rolled into the CLI so users can get started without copying and pasting this file from the docs but I figured until then this was useful. If you're on board with that I can look into making the CLI update after this is reviewed.

Updated address @austinbyers's feedback in https://github.com/airbnb/binaryalert/pull/142#pullrequestreview-212583097.

Assuming the RST syntax in here is correct this should be ready.


Remove demo pipeline scripts.

These scripts were created to demo the basic matching pipeline. Now that the pipeline has been implemented in Prefect in #8 we can delete these scripts.

Thanks to CodeSee, I noticed that my work on #45 was leaving dangling references in the old demo scripts, so I decided to remove those first so that my other change would be cleaner and not leave any breakages, even in unused code:


接口调用智能合约失败event service creation failed:

{"code":500,"msg":"失败","data":"event service creation failed: could not get chConfig cache reference: QueryBlockConfig failed: QueryBlockConfig failed: queryChaincode failed: Multiple errors occurred: - Transaction processing for endorser [<IP_ADDRESS>:37051]: Endorser Client Status Code: (2) CONNECTION_FAILED. Description: dialing connection on target [<IP_ADDRESS>:37051]: connection is in TRANSIENT_FAILURE - Transaction processing for endorser [<IP_ADDRESS>:27051]: Endorser Client Status Code: (2) CONNECTION_FAILED. Description: dialing connection on target [<IP_ADDRESS>:27051]: connection is in TRANSIENT_FAILURE"}

这是什么问题可以告诉我排查思路吗，链码容器内部可以调用成功的

好像ip改了就好了