Sync AK trunk -> CCS master, 2020/07/10

Pull AK trunk as of 2020/07/10.
Merge applied cleanly.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Test failures in first build:
kafka.admin.TopicCommandWithAdminClientTest.testDescribeUnderReplicatedPartitionsWhenReassignmentIsInProgress
kafka.api.TransactionsTest.testBumpTransactionalEpoch
kafka.network.SocketServerTest.testClientDisconnectionUpdatesRequestMetrics

Second build has one core Kafka failure:
kafka.api.PlaintextAdminIntegrationTest > testConsumerGroups

and some streams test failures:
Exception: java.lang.AssertionError thrown from the UncaughtExceptionHandler in thread "appDir2-StreamThread-1"
Exception: java.lang.AssertionError thrown from the UncaughtExceptionHandler in thread "appDir1-StreamThread-1"
Exception: java.lang.AssertionError thrown from the UncaughtExceptionHandler in thread "appDir2-StreamThread-1"


Use Scanner option.encoding if set

I'm still not sure if this change helps. The problem could be at the default encoding in utils.js Only that I'm not familiar at all with coffee-script-development...

In the second step I call toString() for not yet encoded data and set utf8 as default for decoding in utils.coffee I'm not sure if this is something you intended but it works for me

do u feel like submitting the discussed corrections?

ok. But tomorrow :)

Thanks for taking the time

You're welcome. Can you please publish it on npm?


Got socket error trying to find package image_picker

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
when I try to add image_picker: ^0.8.9 to my Flutter app it takes too much time and gives me this error
Got socket error trying to find package image_picker_platform_interface at https://pub.dev.
Got socket error trying to find package image_picker_ios at https://pub.dev.
Actual results
I tried different versions of image_picker but give me the same error could anyone help, please?
Logs

Logs

<!-- Paste your logs here -->

Flutter Doctor output

Doctor output

<!-- Paste your output here -->

I tried pub get the package again now it worked fine without any problem, maybe was a connection problem, Thank you 👍


vsphere node template resource lookup fails if there are multiple cloud credentials configured

What kind of request is this (question/bug/enhancement/feature request): BUG
Steps to reproduce (least amount of steps as possible):

Setup AWS cloud credentials in Rancher
Setup VMware cloud credentials in Rancher
Try and add a node template for VMware
Look at the network trace, the first call to VMware endpoint is made using the AWS cloud provider ( because that is the first one chosen in the nodeTemplate UI)
This call fails with response code 422.
Then rancher tries the same call with Vmware credentials, the vcenter returns the response
However the UI  no longer processes the response
If you have any other cloud credential configured, then select that node driver before you select Vsphere.
Rancher UI makes calls to vsphere UI using the last selected cloud providers credentials

Result:

The node template functionality may appear broken

Other details that may be helpful:

Work around is to select a cloud provider for which cloud credentials are not configured before you select Vsphere provider.
Now vsphere loads the resources just fine.

Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.3.3
Installation option (single install/HA): HA

Cluster information

Cluster type (Hosted/Infrastructure Provider/Custom/Imported): Hosted
Machine type (cloud/VM/metal) and specifications (CPU/memory): VM
Kubernetes version (use kubectl version): 1.16

(paste the output here)

Docker version (use docker version):

(paste the output here)

@deniseschannon do we want to backport this issue?

@codyrancher yes please!

rancher/rancher:master-head 7578b3c76
rancher/rancher:v2.3.4-rc7

Reproduced in Rancher v2.3.3

Create vSphere cloud credentials with valid credentials and create a node template
Edit the cloud credentials to have an invalid password
Create a new node template
Create a new cloud credential with valid credentials from within the node template creation screen
UI remains locked in some fields even after changing cloud credentials

Follow same steps above in v2.3.4-rc7, and master-head

After selecting the correct cloud credential, all fields are functional in the UI.


#107 Added valueAsDate getter support

@chemerisuk could you please take a look at issue with CI ?

@abaranovskii I like the idea to polyfill this property, but your PR should be improved. Later I'll provide my comment inline.

@chemerisuk done

👍


created a yes/no pop-up window when exiting application

Created a yes/no pop-up window when exiting the application

worked on the above issue

@shashwat-2077 please always mention the issue number while making any pull request.

@shashwat-2077 I have reviewed your changes, it's fine. Nice work.
But I suggest you to close this pull request and make another pull request mentioning the issue number https://github.com/QAZIMAAZARSHAD/Income-Tax-Calcultor/issues/8.

Yes sir, i have closed the previous pull request and made another pull request by mentioning the issue number #8.


WorkerFactoryOptions getDefaultInstance() does not return the default instance

Expected Behavior
WorkerFactoryOptions getDefaultInstance() should return the default instance where the max cache size, max thread count etc. are set to the default values
Actual Behavior
WorkerFactoryOptions getDefaultInstance() does not return the instance with default values
Steps to Reproduce the Problem

call WorkerFactoryOptions.getDefaultInstance() and look at the fields

Specifications

Version: 1.17.0
Platform:

The instance returned by WorkerFactoryOptions#getDefaultInstance will lead to the default configuration if it's used to create WorkerFactory.
We don't publish fields that were not overriden to support merging of properties.

If you want to read the default values for whatever purposes, you may use WorkerFactoryOptions.newBuilder().validateAndBuildWithDefaults()


Add outbound call example.

Do we really need the copyright in the header of example code?

Well technically if you don't provide the copyright, then you're not giving people permission to use it - default copyright is pretty restrictive!
(But the main reason it's in there is because it's in my IDE's Java template. Happy to take it out if you think it's intimidating!)

@judy2k to summarise - how about we...

move to using PRIVATE_KEY and use the String version (great that it supports that 👍)
remove the copyright from each file and ensure we have a LICENSE file in the repo
provide instructions to execute a command to run this example in the README. There will be multiple examples so I'm guessing this won't be a gradle command

WDYT?


Tutorial 2 - Adding a new Command

Add Remark Command for tutorial 2

Codecov Report

Merging #2 (e6e2aa7) into master (44bd9d2) will decrease coverage by 0.25%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master       #2      +/-   ##
============================================
- Coverage     72.15%   71.90%   -0.26%
  Complexity      399      399
============================================
  Files            70       71       +1
  Lines          1232     1235       +3
  Branches        125      125
============================================
- Hits            889      888       -1
- Misses          311      314       +3
- Partials         32       33       +1

Impacted Files
Coverage Δ

...va/seedu/address/logic/commands/RemarkCommand.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
88.88% <0.00%> (-11.12%)
:arrow_down:

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 44bd9d2...e6e2aa7. Read the comment docs.


[Event Request] Transfer header Table No. series is not extensible

we don't have this trigger in transfer header table(OnBeforeTestNoSeries), similar to sales/purchase header.
Request to provide the same with handled pattern.
we need signature same as it is in sales header table.(please look into sales header table for this trigger)
OnBeforeTestNoSeries(Rec, IsHandled);
if not IsHandled then

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


provide granular ACLs for zookeeper

our zk code writes everything as Data.ACL.AnyoneAllUnsafe. consider providing more granular control, both for the dcos bootstrapping configuration as well as the dtab storage code.
motivated by: https://github.com/BuoyantIO/linkerd/pull/271/files#r59498272

Does this specifically pertain to the DCOS bootstrap?  The regular ZkDtabStore supports configurable ACLs.

Yeah, ZkDtabStore got configurability after this ticket was written:
https://github.com/BuoyantIO/linkerd/commit/cb502a0f645135e265d670b94ba60dd680e2c1df
I've updated the description accordingly.
I think it's still worth considering for DcosBootstrap. I think we could make it a DCOS package config option on install. Would require changes in this repo, as well as in https://github.com/mesosphere/universe/blob/version-2.x/repo/packages/N/namerd/0/config.json.


pagespeedonline:v4 not available

I've went to the API Discovery service to check if a service is on the list:
https://developers.google.com/discovery/v1/reference/apis/list
It's not. But when I call getRest method:
https://developers.google.com/discovery/v1/reference/apis/getRest
and specify api=pagespeedonline, version=v4 it returns the new version.
Not sure what can be done about it.

As a workaround one may manually download a JSON specification of API and generate it from command line.

How are you able to actually generate the clients? I've been running the make file (with and without sudo permissions) which finds the correct discovery document but no .go file is generated.

sudo is necessary. Yes, I've generated a .go file. I didn't call make, I've run a generator binary:
https://github.com/google/google-api-go-client/tree/master/google-api-go-generator
I would guess the makefile generates a binary not an api. You have to run this binary.

just as a test, i tried something like sudo -E ./google-api-go-generator -api=youtube:v3 (i added the -E because without it, it complains about NO GOPATH SET), after deleting the youtube .go file, yet no code actually generates. I've also tried sudo -E ./google-api-go-generator -api_json_file=../youtube/v3/youtube-api.json. Do you mind showing me the command you ran?

The problem is: some API are not available through list endpoint, but you still can download a API spec if you directly fetch the version by calling getRest. One must first download not visible api through getRest and later use such json to generate API code.

@orian Would you be willing to publish your version of this library with pagespeedonline v4 support?  That way people could use it in the meantime.

If v4 gets published to the discovery doc it will be automatically added to this library.
It looks like it is just one GET endpoint authenticated with an API key. So this library might almost be too much.
… But to wrap everything up:
go get google.golang.org/api/...
curl 'https://www.googleapis.com/discovery/v1/apis/pagespeedonline/v4/rest' -H 'Accept: application/json' --compressed > jason
google-api-go-generator -api_json_file jason
rm jason
cd $GOPATH/src/google.golang.org/api/pagespeedonline/v4
#profit


Fix grammar issue

@yudomsft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Correction confirmed. Thanks.


Error 500 when url with embedded credentials

Environment
Latest version of Nuxt
Reproduction
Build the site and try accessing it using Firefox browser as<EMAIL_ADDRESS>Describe the bug
Error:
500
[GET] "/_nuxt/builds/meta/649e1437-0874-4340-97bb-b78055f1859a.json": <no response> Window.fetch: /_nuxt/builds/meta/649e1437-0874-4340-97bb-b78055f1859a.json is an url with embedded credentials.

Additional context
The problem does not appear on other browsers
Logs
No response

@huang-julien It doesn't need reproduction, this is exactly happens everywhere, even on nuxt.com:

Open firefox
Go to<EMAIL_ADDRESS>Error
but in chrome everything works good

Does it work when disabling the app manifest?

@manniL With appManifest: false its working properly, what effect does appManifest? I don't know if I can just turn off this option without consequences for the rest of the site

Seems like the problem is the $fetch call to the manifest then (source).
I wonder how important the support of it is as user credentials shouldn't be passed via URL anyway. It works in Chrome because they disabled it for more than 5 years :D

@manniL I only use this for automatic preview for PRs, ngnix creates a page that requires a login and the automatic bot creates a url that you just click on without having to enter any data, it's not some critical but developers using firefox have a bit of a degraded experience

After discussing this internally, we don't think we can avoid that behavior from a technical POV.


[full-ci] Bump Reva

Bumps reva
Fixes https://github.com/owncloud/ocis/issues/9676

There is another Reva bump ongoing. #4411
It fixes some of the test expectations.


Not working anymore

The views are showed as sended but no one is true

Yes ... We already know that


LSL stream with MuseWrapper?

Could the LSL stream be added to the Python library implementation?
P.s. Thanks a lot for this wonderful work!!

Hello,
Done, I added it to the both branches (dongle and master). Haven't got the time to try it out yet. Please let me know if there is an issue with it.
P.S. sorry about the delay. It has been a busy year so far.


Data Collection in Azure CLI License Terms: More info please

Hello,
The Azure CLI License Terms states:
**3. DATA.** The
    software may collect information about you and your use of the
    software, and send that to Microsoft. Microsoft may use this
    information to provide services and improve our products and
    services.You may opt-out of many of
    these scenarios, but not all, as described in the product
    documentation.

I'd like a link to ...the <presumably Azure CLI> product documentation mentioned in the last sentence shown above.
Thank you

@yugangw-msft @achandmsft

You can opt into or out of client-side telemetry by running az configure.

Thank you @tjprescott . For others wishing to opt-out of "telemetry" when using azure cli:
user5@system5:~$ az configure

Welcome to the Azure CLI! This command will guide you through logging in and setting some default values.

Your settings can be found at /home/user5/.azure/config
Your current configuration is as follows:

[cloud]
name = AzureCloud

Do you wish to change your settings? (y/N): y

What default output format would you like?
 [1] json - JSON formatted output that most closely matches API responses.
 [2] jsonc - Colored JSON formatted output that most closely matches API responses.
 [3] table - Human-readable output format.
 [4] tsv - Tab- and Newline-delimited. Great for GREP, AWK, etc.
 [5] yaml - YAML formatted output. An alternative to JSON. Great for configuration files.
 [6] none - No output, except for errors and warnings.
Please enter a choice [1]: 3

Would you like to enable logging to file? (y/N): y

Microsoft would like to collect anonymous Azure CLI usage data to improve our CLI.  Participation is voluntary and when you choose to participate, your device automatically sends information to Microsoft about how you use Azure CLI.  **To update your choice, run "az configure" again.**
**Select y to enable data collection. (Y/n): n**

Then run az configure again to see:
user5@system5:~$ az configure

Welcome to the Azure CLI! This command will guide you through logging in and setting some default values.

Your settings can be found at /home/user5/.azure/config
Your current configuration is as follows:

[cloud]
name = AzureCloud

[core]
output = table
**collect_telemetry = no**

[logging]
enable_log_file = yes


Could not execute entrypoint stage 'main' due to errors, provided by 'medievalweapons'!

Error: crash-2022-03-08_20.49.38-server.txt
Logs: latest.log

Pls update medievalweapons and betternether to the latest version


Cytron PS2 Controller Library

Task Priority : HIGH
** Task Description**
Library for a PS2 Controller to read joystick values and get desired angle and speed for bot motion.
Person assigned: @akshatha-k
Original Cytron PS2 Shield library reference: here
Must Haves

[x] Initialise software serial
[x] Function to calculate angle and speed of the desired motion from analog joystick values.
[x] Function to map the values from a circular space to a square.
[x] Compatibility with buttons to expand its function to toggle modes/start/stop etc.
[x] Have a function that neglects readings from a circular inner region.
[ ] Compatibility with Other Hardware Serials.

Tried to solve the equations to map a square to a circle as the values not lying on the axis were reported to be higher , using the approach followed for a unit square to circle mapping here
But the equations for the transformed co-ordinates were solutions of quadratic and bi-quadratic equations respectively. Hence, the code is mapped to modify all values greater than 127 to 127.

Compatibility issues

Debugger implementation is out of date, please use the DebuggerSerial library from the dev branch (link here)
Documentation fixes:

README has no mathematical proof or proper help for code.
README points to the DATA folder (for image links) that no longer exists on the master or the dev branch. Replace it with .DATA/Images (the folder is now hidden).


[SAI P4] Start a library of common table entries for testing.

[SAI P4] Start a library of common table entries for testing.
Build and UT:
rkavitha@9e478a70a357:/sonic/src/sonic-p4rt/sonic-pins$ bazel build  $BAZEL_BUILD_OPTS ...
INFO: Analyzed 280 targets (0 packages loaded, 0 targets configured).
INFO: Found 280 targets...
INFO: Elapsed time: 0.346s, Critical Path: 0.00s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
rkavitha@9e478a70a357:/sonic/src/sonic-p4rt/sonic-pins$ bazel test  $BAZEL_BUILD_OPTS ...
INFO: Analyzed 280 targets (0 packages loaded, 0 targets configured).
INFO: Found 177 targets and 103 test targets...
INFO: Elapsed time: 0.317s, Critical Path: 0.01s
INFO: 1 process: 1 internal.
INFO: Build completed successfully, 1 total action
//gutil:collections_test                                        (cached) PASSED in 0.2s
//gutil:io_test                                                 (cached) PASSED in 0.2s
//gutil:proto_matchers_test                                     (cached) PASSED in 0.3s
//gutil:proto_test                                              (cached) PASSED in 0.2s
//gutil:status_matchers_test                                    (cached) PASSED in 0.2s
//gutil:table_entry_key_test                                    (cached) PASSED in 0.0s
//gutil:testing_test                                            (cached) PASSED in 0.2s
//gutil:version_test                                            (cached) PASSED in 0.2s
//p4_fuzzer:constraints_util_integration_test                   (cached) PASSED in 0.3s
//p4_fuzzer:fuzz_util_test                                      (cached) PASSED in 0.4s
//p4_fuzzer:fuzzer_showcase_test                                (cached) PASSED in 0.5s
//p4_fuzzer:switch_state_test                                   (cached) PASSED in 0.4s
//p4_fuzzer:table_entry_key_test                                (cached) PASSED in 0.1s
//p4_pdpi:ir_tools_test                                         (cached) PASSED in 0.4s
//p4_pdpi:p4_runtime_matchers_test                              (cached) PASSED in 0.3s
//p4_pdpi:sequencing_util_test                                  (cached) PASSED in 0.3s
//p4_pdpi/netaddr:ipv4_address_and_network_address_test         (cached) PASSED in 0.2s
//p4_pdpi/netaddr:ipv6_address_test                             (cached) PASSED in 0.2s
//p4_pdpi/netaddr:mac_address_test                              (cached) PASSED in 0.2s
//p4_pdpi/packetlib:packetlib_fuzzer_test                       (cached) PASSED in 15.4s
//p4_pdpi/packetlib:packetlib_matchers_test                     (cached) PASSED in 0.2s
//p4_pdpi/packetlib:packetlib_test                              (cached) PASSED in 0.1s
//p4_pdpi/packetlib:packetlib_test_runner                       (cached) PASSED in 0.0s
//p4_pdpi/packetlib:packetlib_unit_test                         (cached) PASSED in 0.9s
//p4_pdpi/string_encodings:bit_string_test                      (cached) PASSED in 0.2s
//p4_pdpi/string_encodings:byte_string_test                     (cached) PASSED in 0.2s
//p4_pdpi/string_encodings:decimal_string_test                  (cached) PASSED in 0.1s
//p4_pdpi/string_encodings:decimal_string_test_runner           (cached) PASSED in 0.0s
//p4_pdpi/string_encodings:hex_string_test                      (cached) PASSED in 0.1s
//p4_pdpi/string_encodings:hex_string_test_runner               (cached) PASSED in 0.0s
//p4_pdpi/string_encodings:readable_byte_string_test            (cached) PASSED in 0.2s
//p4_pdpi/testing:helper_function_test                          (cached) PASSED in 0.4s
//p4_pdpi/testing:info_test                                     (cached) PASSED in 0.1s
//p4_pdpi/testing:info_test_runner                              (cached) PASSED in 0.1s
//p4_pdpi/testing:main_pd_test                                  (cached) PASSED in 0.0s
//p4_pdpi/testing:mock_p4_runtime_server_test                   (cached) PASSED in 0.3s
//p4_pdpi/testing:packet_io_test                                (cached) PASSED in 0.1s
//p4_pdpi/testing:packet_io_test_runner                         (cached) PASSED in 0.0s
//p4_pdpi/testing:rpc_test                                      (cached) PASSED in 0.1s
//p4_pdpi/testing:rpc_test_runner                               (cached) PASSED in 0.1s
//p4_pdpi/testing:sequencing_test                               (cached) PASSED in 0.1s
//p4_pdpi/testing:sequencing_test_runner                        (cached) PASSED in 0.1s
//p4_pdpi/testing:sequencing_util_test                          (cached) PASSED in 0.0s
//p4_pdpi/testing:sequencing_util_test_runner                   (cached) PASSED in 0.0s
//p4_pdpi/testing:table_entry_gunit_test                        (cached) PASSED in 0.3s
//p4_pdpi/testing:table_entry_test                              (cached) PASSED in 0.1s
//p4_pdpi/testing:table_entry_test_runner                       (cached) PASSED in 0.1s
//p4_pdpi/utils:annotation_parser_test                          (cached) PASSED in 0.2s
//p4_pdpi/utils:ir_test                                         (cached) PASSED in 0.3s
//p4rt_app/event_monitoring:app_state_db_port_table_event_test  (cached) PASSED in 0.6s
//p4rt_app/event_monitoring:app_state_db_send_to_ingress_port_table_event_test (cached) PASSED in 0.5s
//p4rt_app/event_monitoring:config_db_node_cfg_table_event_test (cached) PASSED in 0.5s
//p4rt_app/event_monitoring:config_db_port_table_event_test     (cached) PASSED in 0.5s
//p4rt_app/event_monitoring:debug_data_dump_events_test         (cached) PASSED in 0.5s
//p4rt_app/event_monitoring:state_verification_events_test      (cached) PASSED in 0.5s
//p4rt_app/p4runtime:ir_translation_test                        (cached) PASSED in 0.4s
//p4rt_app/p4runtime:p4info_verification_schema_test            (cached) PASSED in 0.4s
//p4rt_app/p4runtime:p4info_verification_test                   (cached) PASSED in 0.4s
//p4rt_app/p4runtime:packetio_helpers_test                      (cached) PASSED in 0.5s
//p4rt_app/p4runtime:resource_utilization_test                  (cached) PASSED in 0.7s
//p4rt_app/sonic:app_db_acl_def_table_manager_test              (cached) PASSED in 0.4s
//p4rt_app/sonic:app_db_manager_test                            (cached) PASSED in 0.5s
//p4rt_app/sonic:app_db_to_pdpi_ir_translator_test              (cached) PASSED in 0.5s
//p4rt_app/sonic:hashing_test                                   (cached) PASSED in 0.4s
//p4rt_app/sonic:packetio_impl_test                             (cached) PASSED in 0.3s
//p4rt_app/sonic:packetio_port_test                             (cached) PASSED in 0.3s
//p4rt_app/sonic:response_handler_test                          (cached) PASSED in 0.4s
//p4rt_app/sonic:state_verification_test                        (cached) PASSED in 0.2s
//p4rt_app/sonic:vrf_entry_translation_test                     (cached) PASSED in 0.4s
//p4rt_app/sonic/adapters:fake_sonic_db_table_test              (cached) PASSED in 0.0s
//p4rt_app/tests:acl_table_test                                 (cached) PASSED in 0.8s
//p4rt_app/tests:action_set_test                                (cached) PASSED in 0.6s
//p4rt_app/tests:api_access_test                                (cached) PASSED in 0.5s
//p4rt_app/tests:arbitration_test                               (cached) PASSED in 0.6s
//p4rt_app/tests:debug_data_dump_test                           (cached) PASSED in 0.5s
//p4rt_app/tests:fixed_l3_tables_test                           (cached) PASSED in 1.6s
//p4rt_app/tests:forwarding_pipeline_config_test                (cached) PASSED in 2.4s
//p4rt_app/tests:grpc_behavior_test                             (cached) PASSED in 5.0s
//p4rt_app/tests:p4_constraints_test                            (cached) PASSED in 0.1s
//p4rt_app/tests:p4_constraints_test_runner                     (cached) PASSED in 0.1s
//p4rt_app/tests:p4_programs_test                               (cached) PASSED in 0.9s
//p4rt_app/tests:packetio_test                                  (cached) PASSED in 2.9s
//p4rt_app/tests:port_name_and_id_test                          (cached) PASSED in 1.4s
//p4rt_app/tests:resource_limits_test                           (cached) PASSED in 1.4s
//p4rt_app/tests:response_path_test                             (cached) PASSED in 1.7s
//p4rt_app/tests:role_test                                      (cached) PASSED in 0.7s
//p4rt_app/tests:state_verification_test                        (cached) PASSED in 0.9s
//p4rt_app/tests:vrf_table_test                                 (cached) PASSED in 0.9s
//p4rt_app/tests/lib:app_db_entry_builder_test                  (cached) PASSED in 0.0s
//p4rt_app/utils:event_data_tracker_test                        (cached) PASSED in 0.0s
//p4rt_app/utils:table_utility_test                             (cached) PASSED in 0.5s
//sai_p4/instantiations/google:clos_stage_test                  (cached) PASSED in 0.2s
//sai_p4/instantiations/google:fabric_border_router_p4info_up_to_date_test (cached) PASSED in 0.0s
//sai_p4/instantiations/google:middleblock_p4info_up_to_date_test (cached) PASSED in 0.0s
//sai_p4/instantiations/google:sai_p4info_fetcher_test          (cached) PASSED in 0.3s
//sai_p4/instantiations/google:sai_p4info_test                  (cached) PASSED in 0.5s
//sai_p4/instantiations/google:sai_pd_proto_test                (cached) PASSED in 0.0s
//sai_p4/instantiations/google:sai_pd_util_test                 (cached) PASSED in 0.3s
//sai_p4/instantiations/google:tor_p4info_up_to_date_test       (cached) PASSED in 0.0s
//sai_p4/instantiations/google:union_p4info_up_to_date_test     (cached) PASSED in 0.3s
//sai_p4/instantiations/google:wbb_p4info_up_to_date_test       (cached) PASSED in 0.0s
//sai_p4/instantiations/google/test_tools:test_entries_test     (cached) PASSED in 0.4s
//sai_p4/tools:p4info_tools_test                                (cached) PASSED in 0.3s
Executed 0 out of 103 tests: 103 tests pass.
There were tests whose specified size is too big. Use the --test_verbose_timeoutINFO: Build completed successfully, 1 total action
Keyword Check:
rkavitha@eonms3vm12:~/Google/Jun-05/sonic-buildimage/src/sonic-p4rt/sonic-pins$ ~/Google/tools/keyword_checks.sh .
Keyword check Passed.

@kishanps, Could you please review.
Thanks.


feat: expose L1 constructs (#1126)

Backport
This will backport the following commits from k8s-24/main to k8s-23/main:

feat: expose L1 constructs (#1126)

Questions ?
Please refer to the Backport tool documentation

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1136

Questions ?
Please refer to the Backport tool documentation


.travis.yml does not cache dependencies

Travis CI can cache content that does not often change, to speed up your build process.

Removed travis


Non-interactive markers

Is it possible to set markers to not be interactive and be just like an object "painted" onto the map so that they don't register any click events and it is as if you're interacting with the map if you try click on them?

@DVassilev in a way I'd like to do this. What I'd like is to overwrite the default on click function of the markers. Right now it animates to the marker in the center, and I'd like to animate the marker to the bottom of screen. I'm having some problems trying to do that smoothly since it seems I can't turn off the default action.

+1 would also like to override.

I found a way to make the marker not respond to any events by doing the following:
<Marker pointerEvents="none" ...otherProps>


Update composer.json

Laravel doesn't follow semver. So having ^5.x, 5.* or >=5.x will mean that composer will install a future version even though that version breaks the package.

You are right, sir, thank you!


Metadata errors with new RawFileReader MSConvert

MSConvert Version (x64):  3.0.18243
We just ran a test with the new MSConvert that uses RawFileReader, and noticed a few errors comparing older results vs. new results when converting to mzML. The peak data is the same, as is most of the metadata, but I'm going to add replies to this with the issues we have found.
To start off:

MS1 spectra have precursors displayed in the mzML, with default values for the items required under the precursorList. (This is on a data file from a QExactive HF with Xcalibur <IP_ADDRESS>06, and doesn't happen with data from an LTQ Orbitrap Elite with Xcalibur 2.7.0 SP1)
HCD is being interpreted as ECD+CID. It appears to be caused by the pwiz Thermo ActivationType Enum using flag values and directly casting from the RawFileReader ActivationType enum, which uses sequential values (RawFileReader: HCD=5, RawFileTypes.h: CID=1, ECD=4)

For reference, the enum values for ActivationType  in ThermoFisher.CommonCore.Data.FilterEnums are:
0: CollisionInducedDissociation
1: MultiPhotonDissociation
2: ElectronCaptureDissociation
3: PQD
4: ElectronTransferDissociation
5: HigherEnergyCollisionalDissociation (hcd)
6: Any
7: SAactivation
8: ProtonTransferReaction
9: NegativeElectronTransferDissociation
10: NegativeProtonTransferReaction
11: UltraVioletPhotoDissociation

This enum is returned by the call to GetActivation as demonstrated by this code snippet
var scanFilter = mXRawFile.GetFilterForScanNumber(scan);
var reactions = scanFilter.MassCount;

var index = reactions - 1;
if (index > 0 && scanFilter.GetIsMultipleActivation(index))
{
    // The last activation is part of a ETciD/EThcD pair
    index--;
}

var activationTypeCode = scanFilter.GetActivation(index);

This snippet is from
https://github.com/PNNL-Comp-Mass-Spec/Thermo-Raw-File-Reader/blob/ThermoNuGetDLL/XRawFileIO.cs#L1130

Thanks for the report. I can definitely see your #2 happening because it happened with a few of the other enums when I started trying to cast directly (with MSFR all those are parsed from strings). It slipped by because apparently none of the pwiz or Skyline tests use HCD! Maybe I can figure out a way to adapt the ScanFilterTest tests to also test RawFileReader.
I cannot reproduce your #1. Can you provide a test file?

You may be able to create a scan filter from a string with RawFileReader.
I think ThermoFisher.CommonCore.RawFileReader.FilterParserFactory.CreateFilterParser().GetFilterFromString(string) would do it.

I also noticed that the "GetAverageMassList" method is now commented out, I'm assuming because it isn't used, but earlier this week I figured out how to get similar results from RawFileReader. I say "similar" because the method in MSFileReader is susceptible to the improper centroiding of FTMS data that would also occur with "GetMassList"

Also, we just ran a test with a Lumos file with EThcD spectra; it crashed. The main error was "precursor count does not match ms level[SpectrumWorkerThreads::work] error in thread". I'm assuming that, if the creating a IScanFilter object using the GetFilterFromString works, this error will probably be fixed in the process of getting the ScanFilterTests to all pass.

Also, if you call IScanFilter.ToString(), you get the scan filter string you would expect to see from MSFileReader.

That was a pretty severe bug. No idea how widespread that was. Fixed in #194

The activation type issue should be fixed in #204.

(if someone finds other RFR issues it can be reopened)


cliconfig: credentials: set default for unix

depends on https://github.com/docker/docker-credential-helpers/pull/7
Signed-off-by: Antonio Murdaca<EMAIL_ADDRESS>

@thaJeztah I changed https://github.com/docker/docker/pull/20975/files#diff-c033ce901f31496d94f0e354c81a9cdbR54 to not refer to GNOME anymore

Thanks LGTM :-)

LGTM 🐭

I think we should rename cliconfig/credentials/default_store_unix.go to cliconfig/credentials/default_store_linux.go and have an _unsupported.go. Keep in mind that the client also works on FreeBSD :wink:

@calavera fixed, PTAL

LGTM, but janky is so unhappy :pensive:

Fixed compilation errors but seems CI isn't so good at all :confounded:

Yup, looks like Janky is not happy at all at the moment :cry: https://botbot.me/freenode/docker-maintainers/2016-03-08/?msg=61730131&page=1

Win2Lin is weird; failing test-results coming in after build was canceled; https://jenkins.dockerproject.org/job/Docker-PRs-Win2Lin/22886/console
15:42:10 Now starting POST-BUILD steps
15:42:10 ---
15:42:10
15:42:10 INFO: Pointing to tcp://<IP_ADDRESS>:2376
15:42:10 INFO: Removing containers...
15:42:11
15:42:11 ----------------------------------------------------------------------
15:42:11 FAIL: docker_cli_build_test.go:3165: DockerSuite.TestBuildInheritance
15:42:11
15:42:11 docker_cli_build_test.go:3183:
15:42:11     c.Fatal(err)
15:42:11 ... Error: failed to inspect testbuildinheritance:
15:42:11
15:42:11
15:42:11
15:42:11
15:42:11 ----------------------------------------------------------------------
15:42:11 FAIL: docker_cli_build_test.go:4438: DockerSuite.TestBuildInvalidTag
15:42:11
15:42:11 docker_cli_build_test.go:4444:
15:42:11     c.Fatalf("failed to stop before building. Error: %s, Output: %s", err, out)
15:42:11 ... Error: failed to stop before building. Error: failed to build the image: , Output:
15:42:11
15:42:11
15:42:11
15:42:12 f2269656e3e0
15:42:13
15:42:13 ----------------------------------------------------------------------
15:42:13 FAIL: docker_cli_build_test.go:869: DockerSuite.TestBuildJSONAddMultipleFilesToFile
15:42:13
15:42:13 docker_cli_build_test.go:886:
15:42:13     c.Fatalf("Wrong error: (should contain %q) got:\n%v", expected, err)
15:42:13 ... Error: Wrong error: (should contain "When using ADD with more than one source file, the destination must be a directory and end with a /") got:
15:42:13 failed to build the image:
15:42:13
15:42:13
15:42:13
15:42:14 INFO: Removing images...
15:42:16 Failed to remove image (256adf7015ca): Error response from daemon: conflict: unable to delete 256adf7015ca (cannot be forced) - image has dependent child images
15:42:16
15:42:16 ----------------------------------------------------------------------
15:42:16 FAIL: docker_cli_build_test.go:913: DockerSuite.TestBuildJSONAddMultipleFilesToFileWild
15:42:16
15:42:16 docker_cli_build_test.go:930:
15:42:16     c.Fatalf("Wrong error: (should contain %q) got:\n%v", expected, err)
15:42:16 ... Error: Wrong error: (should contain "When using ADD with more than one source file, the destination must be a directory and end with a /") got:
15:42:16 failed to build the image:
15:42:16


Integrate Swagger UI

I think there has been some challenge with Spring Boot 1.5.22 for thi s feature, but the 1.5 will use the newer Spring Boot 2.5 #94

[ ] properly indicate the security schemes (authenticated or anonymous)
[ ] make it available under /apidocs

Spring Boot 2.6.8 has been merged! We can look into this feature now :)

With #36, we need to specify the authentication mechanism because it is not any longer documented at the endpoint-level.


#ASK Apache ECharts Panel 5.2.0

Hi VolkovLabs Team,
I saw Apache EChart has been upgraded to 5.2.0. Is there any documentation or YouTube short videos for this version?
Especially on the feature "Add a visual editor to work with data sources (#211)" how does it work ?
Thank you

@itsnull12 We are working on a blog and Shorts. Meanwhile, you can check examples at https://echarts.volkovlabs.io/d/pL5iH7FVk/home?orgId=1&editPanel=23

@mikhail-vl  sounds good! thank you mikhail.

Will be resolved in https://github.com/VolkovLabs/volkovlabs.io/issues/581 for the Business Charts 6.0.0.


Type inferring issues when a union of multiple tuple types is present

Description:

const x = "x";
const y = "y";
const z = "z";

type someType [x, int];
type anotherType [y, string];

type someOtherType z | someType | anotherType;

someOtherType st = [x, 1];

error:
error: .::tuple-type-infer-test.bal:10:20: incompatible types: expected '(z|[x,int]|[y,string])', found 'any[]'

const x = "x";
const y = "y";

type someType [x, int];
type anotherType [y, string];

type someOtherType someType | anotherType;

someOtherType st = [x, 1];

error: .::tuple-type-infer-test.bal:9:20: incompatible types: expected '([x,int]|[y,string])', found 'other'

According to the spec, when contextually expected type is a union we should use number of members in each type to pin down the target type, if we can't do (here both someType and anotherType are two member tuples) it's a compile time error. So IMO current behavior is not a bug.
I think we need to come up with a better error message than what we have now.


New Category: Social Sign-in

@AliasIO  - I am proposing that we add a category to identify technologies like 'Sign-in with Apple' / 'Sign-in with LinkedIn' / Sign-in with GitHub' etc.
Do you have a preference for category name and priority ?
We will have to look for patterns like

Sign-in Provider
Website
Pattern
Example site

Google Sign-in
?
<meta name="google-signin-client_id" / Presence of script https://apis.google.com/js/platform.js / OR presence of 'https://accounts.google.com/gsi/client'
www.wayfair.co.uk

Apple Sign-in
https://developer.apple.com/sign-in-with-apple/
<meta name="appleid-signin-client-id" / Presence of script appleid.auth.js
https://www.farfetch.com/uk/login.aspx

SAP Customer Data Cloud
https://www.sap.com/uk/acquired-brands/what-is-gigya.html
Presence of cdns.gigya.com/JS/gigya.js
Go to www.asos.com and check on sign-in page

Cover following to start with

@AliasIO - Are you happy with this proposal? If yes, please confirm name and priority.

Yes I am, let's go with 'Social login'.

@adityapandey1998 - Another one to add

Provider Name - Okta
Example site - https://console.perimeterx.com/user/login
Pattern - Look for presence of okta-sign-in.min.js


Import Data from SILVIPC to FORSEC

Add utility class to load data exported from SILVIPC to FORSEC.
Use the existing knex/objection data model code to preserve data integrity and consistency.

Description

Types of changes

New feature (non-breaking change which adds functionality)

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have checked that unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have added necessary documentation (if appropriate)

Further comments

SonarQube Analysis: https://sonarqube-vmvfjv-tools.pathfinder.gov.bc.ca/dashboard?id=common-forms-toolkit-pr-115

SonarQube Analysis: https://sonarqube-vmvfjv-tools.pathfinder.gov.bc.ca/dashboard?id=common-forms-toolkit-pr-115

SonarQube Analysis: https://sonarqube-vmvfjv-tools.pathfinder.gov.bc.ca/dashboard?id=common-forms-toolkit-pr-115

SonarQube Analysis: https://sonarqube-vmvfjv-tools.pathfinder.gov.bc.ca/dashboard?id=common-forms-toolkit-pr-115


Using packaging.version for version number parsing, beeing PEP-440 compliant

This includes:

version_windows is parsing the version names with packaging.version.Version
highest_version is parsing the version names with packaging.version.Version
The sort_version_aware templatetag is also parsing the version names with packaging.version.Version
Along the way I moved the logic for parsing and comparing versions into the new module projects.version_handling. That also replaces betterversion.

This might be the potential fix for a number of tickets including #1017 and #1284

Big :+1:

The main thing I think this needs to also address is prerelease and postrelease packages. Currently we just treat everything the same. Do you know if this will return the previous release when a new beta is published?

Also this needs to at least be documented as using PEP 440 in our docs around stable versions.

@ericholscher Good catch! I've added tests and logic for pre-releases. Also the docs are there now.
The stable version will now be the tag with the highest non-pre-release version number.
I think the ticket is again ready for review.


Map reloading entirely upon changing data

Hi it's me again :D
This issue is closely related to this one as I believe it originates from streamlit-plotly-events.
The same kind of issue can be recreated with streamlit-plotly-mapbox-events :
import streamlit as st
import numpy as np
from streamlit_plotly_mapbox_events import plotly_mapbox_events
import plotly.express as px

np.random.seed(1312)

# Initialize zoom and center variables in session state
if "zoom" not in st.session_state:
    st.session_state.zoom = 3
if "center" not in st.session_state:
    st.session_state.center = {"lat": 0, "lon": 0}

x = np.linspace(-5, 5, 50) + np.random.normal(loc=0, scale=0.1, size=50)
y = np.linspace(-5, 5, 50) + np.random.normal(loc=0, scale=0.1, size=50)

a = st.number_input("Left crop", value=0)
b = st.number_input("Right crop", value=len(x))

fig = px.scatter_mapbox(lat=x[a:b], lon=y[a:b], mapbox_style="open-street-map", zoom=st.session_state.zoom, center=st.session_state.center)

# Plotly express version
st.plotly_chart(fig)

# Plotly events version
plotly_mapbox_events(fig)

# Update zoom and center values in session state
st.session_state.zoom = fig.layout.mapbox.zoom
st.session_state.center = fig.layout.mapbox.center

I don't know if I can realistically expect a smooth update of the map as with native plotly, but I'm hoping you can make it happen 🙏
Thank you once again!

Hoi Hugo,
Currently I cant take a look at this.


Run the algorithm locally

How does the algorithm run locally on 3K data sets? I don't know how to organize data sets, could you provide an example of file organization structure? I couldn't download it because the data set was too big.

What do you mean about the 3K data sets? Doing the inference on 3K resolution images? If you want the inference, I recommend trying the inference single image colab demo mentioned in README and modifying it into a local version, which should be fewer than 3 lines of codes in your correct venv.
file organization looks like this:
~/datasets/
    - dis
        - [DIS5K/General/...]
            - DIS-TR
                - im
                    - *.jpg/*.png/...
                - gt
                    - *.jpg/*.png/...
            - DIS-TE1
                - im
                    - *.jpg/*.png/...
                - gt
                    - *.jpg/*.png/...
            - TR-COD10K
                - im
                    - *.jpg/*.png/...
                - gt
                    - *.jpg/*.png/...
            ...

Check the config.py for a better look at the organization of these datasets.

Yes, I mean doing the inference on 3K resolution images. I can run inference single image and get the result. But I change this line to keep the original resolution:

Encouter this error：

Is there any way to solve this error？

Hi, that's a common problem for most deep visual models, which have downsampling methods like pooling or convolution. For example, if there are three pooling2d(2), you have to feed images with resolution as integer times of 2 ** 3; otherwise, resolutions in the encoder and decoder might differ from each.
In our case here, due to the downsampling operations in the backbone, the size of the input tensor must be integral times of 2 ** 5 = 32, e.g., 32x32, 160x160, 768x768, etc.

Thank you. I get orginal resolution mask using resize function.


Adding markers (@mark.asyncio) make discovery fails

Type: Bug
Hello,
I have a strange issue and test discovery. While making new test asynchronus, the test discovery began to broke.
With the exact same test case, adding only async markers and async test methods, the discovery fails with asyncs on, and pass correctly wihout them.
I looked at the output of the discovery command, both are exactly the same expect for the line number due to markers being added.
Test run correctly directly from command line.

Behaviour
Expected vs. Actual
Expected
Discovery works
Actual
Discovery fails without any error message
Discovery outputs
Ok: https://drive.google.com/file/d/1Zqsxk8pW_hcttCnAFydbOqyNX_KU1-ul/view?usp=share_link
Ko: https://drive.google.com/file/d/18aocozebcQ1Rfwsz_g9_CVtiTs40Zxzf/view?usp=share_link
Steps to reproduce:
I do not know how to reproduce on a simpler code base. Already added some async markers without issue so it feel strange.
I will try to reproduce on simpler project

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.10.7
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Venv
Value of the python.languageServer setting: Pylance

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

 ~/venv/bin/python ~/.vscode-server/extensions/ms-python.python-2022.18.2/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir . -s --cache-clear
cwd: .
[ERROR 2022-10-29 19:43:10.76]: Error discovering pytest tests:
 [n [Error]: Waiting up to 5 seconds.
Sent all pending logs.

	at ChildProcess.<anonymous> (/home/vscode/.vscode-server/extensions/ms-python.python-2022.18.2/out/client/extension.js:2:242843)
	at Object.onceWrapper (node:events:646:26)
	at ChildProcess.emit (node:events:526:28)
	at maybeClose (node:internal/child_process:1092:16)
	at Process.ChildProcess._handle.onexit (node:internal/child_process:302:5)]

User Settings

languageServer: "Pylance"

linting
• flake8Enabled: true
• banditArgs: "<placeholder>"
• banditEnabled: true
• mypyEnabled: true

formatting
• provider: "black"
• blackArgs: "<placeholder>"

testing
• cwd: "<placeholder>"
• pytestEnabled: true

Extension version: 2022.18.2
VS Code version: Code 1.73.1<PHONE_NUMBER>f055b99068d3688932416f2346dd3b, 2022-11-09T03:54:53.913Z)
OS version: Linux x64 6.0.6-76060006-generic
Modes:
Sandboxed: No
Remote OS version: Linux x64 6.0.6-76060006-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz (12 x 4299)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: disabled_softwarevideo_encode: disabled_softwarevulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
1, 1, 1

Memory (System)
31.27GB (23.03GB free)

Process Argv
--unity-launch --crash-reporter-id 9193f995-a3c7-4d94-9dd7-1fdc933506b2

Screen Reader
no

VM
0%

DESKTOP_SESSION
pop

XDG_CURRENT_DESKTOP
Unity

XDG_SESSION_DESKTOP
pop

XDG_SESSION_TYPE
x11

Item
Value

Remote
Dev Container: Bet On You API

OS
Linux x64 6.0.6-76060006-generic

CPUs
Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz (12 x 4299)

Memory (System)
31.27GB (23.03GB free)

VM
0%

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30443607
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscorecescf:30445987
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vscaac:30438847
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
azure-dev_surveyonecf:30548226
pyindex848:30577860
nodejswelcome1:30587005
3biah626:30602489
gswce1:30612156
iaj6b796:30613358
dbltrim-noruby:30604474
f6dab269:30613381

:cactus: Found issue. (Sorry, already looked for hours and did not found previously). An import in test code missed env configuration at discovery time and made it failed without any indication.
Still strange cause pytest --collect-only worked fined (usually failed when discovery failed), manual command run was ok, and discovery only failed from plugin directly without logs to help :/

Hello @titouanfreville! Thank you for all your careful investigation, this is very useful to us! We are sorry that this issue occurred. To clarify, does this issue still exist for you or were you able to reconfigure to have the import get the env configuration at discovery time? Could you send over your env configuration that caused the error? I will take a closer look at this and appreciate you submitted this issue to our repo!

Hello, @eleanorjboyd.
The issue still exists but is not linked to what I though (and it makes more sense now). It is direcly linked to the already called issue: https://github.com/microsoft/vscode-python/issues/18431
While testing various think, I added a direct import of the fastAPI server in my tests witch requires some envs to be sat to run. Then it probably raises indirectly an error on discovery. Removing the import solve the issue. I already had this issue when creating the tests cases initially and used a local import in method to solve it.
The strange thing is that there are no line related to those issue when running the command manually and it provides a correct result :dizzy:
An easy reproduction would be to have a file that depends on some env being loaded and import it without setuping env in the test files.

Hello @titouanfreville! Sorry for the delay in circling back. Yes you are correct about it relating to #18431 and therefore the fix for it is the feature request, https://github.com/microsoft/vscode-python/issues/17171. Given we already have an issue tracking this I am going to close this issue in favor of the feature request. Thank you again for all your work debugging and I am sorry you encountered this lack of functionality with environment variables.


fixed Variable json deserialization

in case value is missing, previous behaviour was trying to deserialize the default value from json
added a convenience setter:  Variable.set(key, value)
fixes https://github.com/airbnb/airflow/issues/701
2 new UTs

![img] (http://replygif.net/i/1420.gif)


netopeer2 build getting failed

Hi,
i'm using devel branch of libyang,sysrepo,libnetconf2 and netopeer2.
while building netopeer2,observed the error as below:
#make install
[ 44%] Built target serverobj
[ 61%] Built target netopeer2-server
[100%] Built target netopeer2-cli
Install the project...
-- Install configuration: "Debug"
-- Up-to-date: /usr/local/share/yang/modules/netopeer2
-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date<EMAIL_ADDRESS>-- Up-to-date: /usr/local/share/netopeer2/scripts
-- Up-to-date: /usr/local/share/netopeer2/scripts/merge_config.sh
-- Up-to-date: /usr/local/share/netopeer2/scripts/remove.sh
-- Up-to-date: /usr/local/share/netopeer2/scripts/setup.sh
-- Up-to-date: /usr/local/share/netopeer2/scripts/common.sh
-- Up-to-date: /usr/local/share/netopeer2/scripts/merge_hostkey.sh
-- Up-to-date: /usr/local/sbin/netopeer2-server
-- Up-to-date: /usr/local/share/man/man8/netopeer2-server.8
-- Up-to-date: /etc/pam.d/netopeer2.conf
-- Installing missing sysrepo modules...
-- Generating a new RSA host key "genkey" if not already added...
CMake Error at cmake_install.cmake:129 (message):
scripts/merge_hostkey.sh failed: 127
make: *** [Makefile:100: install] Error 1
Could suggest me a solution for this?

There is no error, unfortunately. Can you maybe run the script manually for it to print something?

hi Michal,
As suggested i have added print and run the script manually. But it didn't get print.

Okay, that complicates it. What Linux are you on, what shell are you using?

Hi michal,
Build issue got resolved now. I have tried ssh callhome with example_configuration  provided with devel branch.
After authentication, timeout occured and observed few errors too in server.
CLI:

listen
Waiting 60s for an SSH Call Home connection on port 4334...
The authenticity of the host '::ffff:<IP_ADDRESS>' cannot be established.
ssh-rsa key fingerprint is 19:20:34:99:50:4f:84:b7:ce:eb:57:dd:9d:29:06:d4:77:e6:c1:aa.
Are you sure you want to continue connecting (yes/no)? yes
Keyboard-Interactive Authentication
Please enter your authentication token
root's password:
status
Current NETCONF session:
ID          : 33
Host        : ::ffff:<IP_ADDRESS>
Port        : 59680
Transport   : SSH
Capabilities:
urn:ietf:params:netconf:base:1.0
urn:ietf:params:netconf:base:1.1
urn:ietf:params:netconf:capability:writable-running:1.0
urn:ietf:params:netconf:capability:candidate:1.0
urn:ietf:params:netconf:capability:confirmed-commit:1.1
urn:ietf:params:netconf:capability:rollback-on-error:1.0
urn:ietf:params:netconf:capability:validate:1.1
urn:ietf:params:netconf:capability:startup:1.0
urn:ietf:params:netconf:capability:xpath:1.0
urn:ietf:params:netconf:capability:with-defaults:1.0?basic-mode=explicit&also-supported=report-all,report-all-tagged,trim,explicit
urn:ietf:params:netconf:capability:notification:1.0
urn:ietf:params:netconf:capability:interleave:1.0
urn:ietf:params:netconf:capability:url:1.0?scheme=http,https,ftp,ftps,file
urn:ietf:params:xml:ns:yang:ietf-yang-metadata?module=ietf-yang-metadata&revision=2016-08-05
urn:ietf:params:xml:ns:yang:ietf-inet-types?module=ietf-inet-types&revision=2013-07-15
urn:ietf:params:xml:ns:yang:ietf-yang-types?module=ietf-yang-types&revision=2013-07-15
urn:ietf:params:xml:ns:yang:ietf-netconf-acm?module=ietf-netconf-acm&revision=2018-02-14
urn:ietf:params:netconf:capability:yang-library:1.1?revision=2019-01-04&content-id=600881550
urn:sysrepo:plugind?module=sysrepo-plugind&revision=2022-08-26
urn:ietf:params:xml:ns:netconf:base:1.0?module=ietf-netconf&revision=2013-09-29&features=writable-running,candidate,confirmed-commit,rollback-on-error,validate,startup,url,xpath
urn:ietf:params:xml:ns:yang:ietf-netconf-with-defaults?module=ietf-netconf-with-defaults&revision=2011-06-01
urn:ietf:params:xml:ns:yang:ietf-netconf-notifications?module=ietf-netconf-notifications&revision=2012-02-06
urn:ietf:params:xml:ns:netconf:notification:1.0?module=notifications&revision=2008-07-14
urn:ietf:params:xml:ns:netmod:notification?module=nc-notifications&revision=2008-07-14
urn:ietf:params:xml:ns:yang:ietf-netconf-monitoring?module=ietf-netconf-monitoring&revision=2010-10-04
urn:ietf:params:xml:ns:yang:ietf-x509-cert-to-name?module=ietf-x509-cert-to-name&revision=2014-12-10
urn:ietf:params:xml:ns:yang:iana-crypt-hash?module=iana-crypt-hash&revision=2014-04-04&features=crypt-hash-md5,crypt-hash-sha-256,crypt-hash-sha-512

SERVER:
[INF]: LN: Call Home client "default-client" timeout of 5 seconds expired, reconnecting.
[INF]: LN: Trying to connect via IPv4 to <IP_ADDRESS>:4334.
[INF]: LN: getsockopt() error (Connection refused).
[INF]: LN: Call Home client "default-client" timeout of 5 seconds expired, reconnecting.
[INF]: LN: Trying to connect via IPv4 to <IP_ADDRESS>:4334.
[INF]: LN: Successfully connected to localhost:4334 over IPv4.
[ERR]: LN: SSH key exchange timeout.
[INF]: SR: EV LISTEN: "/ietf-netconf-monitoring:get-schema" "rpc" ID 1 priority 0 processing (remaining 1 subscribers).
[INF]: NP: Module "ietf-datastores@" was requested.
[INF]: SR: EV LISTEN: "/ietf-netconf-monitoring:get-schema" "rpc" ID 1 priority 0 success (remaining 0 subscribers).
[INF]: SR: EV LISTEN: "/ietf-netconf:get" "rpc" ID 1 priority 0 processing (remaining 1 subscribers).
[ERR]: NP: np2srv_rpc_get_cb: Failed to find NETCONF session with NC ID 33.
[INF]: SR: EV LISTEN: "/ietf-netconf:get" "rpc" ID 1 priority 0 fail (remaining 1 subscribers).
[INF]: SR: EV LISTEN: "/ietf-netconf:get" "rpc" ID 2 priority 0 processing (remaining 1 subscribers).
[ERR]: NP: np2srv_rpc_get_cb: Failed to find NETCONF session with NC ID 33.
[INF]: SR: EV LISTEN: "/ietf-netconf:get" "rpc" ID 2 priority 0 fail (remaining 1 subscribers).
[INF]: LN: Call Home client "default-client" timeout of 5 seconds expired, reconnecting.
[INF]: LN: Call Home client "default-client" endpoint "default-ssh" failed connection attempt limit 3 reached.
[INF]: LN: Call Home client "default-client" endpoint "default-ssh" connecting...
[INF]: LN: Trying to connect via IPv4 to <IP_ADDRESS>:4334.
[INF]: LN: getsockopt() error (Connection refused).
Could you please help on this.

What libssh version are you using?

using libssh 0.9.5

Build issue got resolved now.

Actually, could you mention briefly what the problem was? Maybe there is something to improve.
As for the SSH error, it is quite strange so I am asking again, what platform are you on, are you cross-compiling? What crypto backend library is your libssh using? Ideally, it should be OpenSSL, there are some problems with grypt, for example.

what platform are you on, are you cross-compiling?

Linux 5.13.0-30-generic . No we are not cross-compiling

Ideally, it should be OpenSSL, there are some problems with grypt, for example.

Yes, openssl only, its version is 3.0.0

Then I suggest you turn on debug messages and maybe see something relevant there, for server its -c SSH. However, this is a libssh issue so I will likely not be able to help and you should ask them.

Not really, starting from 0.9.5 it should work, but we are always using (and testing with) recent versions, currently 0.10.5.

Hi Michal,
We are facing issue  frequently during netopeer2 build. Please find the error log as below:
**_-- Installing missing sysrepo modules...
-- Generating a new RSA host key "genkey" if not already added...
CMake Error at cmake_install.cmake:141 (message):
scripts/merge_hostkey.sh failed: 127
Makefile:99: recipe for target 'install' failed
make: *** [install] Error 1_**
Could you please share your comments on this ?

You said you fixed this previously but have not replied how exactly. I cannot comment anything as there is no error message.

just rebuild done for the specific module at that time it got fixed. No other particular fixes we done.

What specific module?

sysrepo and netopeer2

I do not understand but it does not matter, without more information I am not able to help.

Hi Michal,
We are facing issue frequently during netopeer2 build. Please find the error log as below:
**_-- Installing missing sysrepo modules... -- Generating a new RSA host key "genkey" if not already added... CMake Error at cmake_install.cmake:141 (message): scripts/merge_hostkey.sh failed: 127
Makefile:99: recipe for target 'install' failed make: *** [install] Error 1_**
Could you please share your comments on this ?

Issue got fixed after copying libs to expected path.


Is it supports android?

This gem doesn't support Android. But you can use Ruboto to access OpenGL-ES from Ruby on Android like so: https://github.com/ruboto/ruboto-irb/blob/master/assets/demo-scripts/demo-opengl.rb


More data sources

http://www.ga.gov.au/scientific-topics/energy/resources/petroleum-resources
http://www.australianminesatlas.gov.au/mapping/downloads.html
https://github.com/NICTA/nationalmap/commit/bd608fb004919b1e2391ea9269d19f4297b793e7

These and other data sources have been used for the latest version.


[zh] Sync changes to feature-gates

Also fixed some nits in the upstream.

@tanjunchen @howieyuen Please help review.

/lgtm

/lgtm


[BUG] - ES modules import "./some-lib.css" seems not to end up in the shadow-root

Describe the bug
When using a library inside as child of your CE component that uses the ES modules import "./mylib.css" (in the <script> tag!) the css does not end up in the shadow dom but in the regular dom.
Reproduction
https://stackblitz.com/edit/vitejs-vite-nqrj24 (should work now)
System Info
No response
Used Package Manager
npm
Validations

[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a unplugin-vue-ce issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs instead.
[X] The provided reproduction is a minimal reproducible example of the bug.

I need more information to reproduce the problem

Are you referring to a third-party component library (such as vuetify)?

Are you referring to a third-party component library (such as vuetify)?

For example. Anything that is imported with a import "style.css" in the head. This style will not end up in the shadow dom even if the import was done in a sub component. The example I provided seems to be the original one, sorry for that I had it working but might have lost the link somehow...

This should work now: https://stackblitz.com/edit/vitejs-vite-nqrj24

Cool, thanks for the fast fix :)


Update to knife cookbook site list

Description
This updates knife cookbook site list to add order by and limit number functionality. It changes the default behavior from listing all cookbooks to 100. The refactor speeds up the response.
Additionally, code is removed from knife cookbook show list that isn't used.
Check List

[ ] New functionality includes tests
[ ] All tests pass
[ ] RELEASE_NOTES.md, has been updated if required (not required for bugfixes, required for API changes)
[ ] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

@robbkidd

@iennae please fix the chefstyle violations in travis

👍  These were the changes @sigje and I paired on.
NOTE: default behavior changed. With this change, knife cookbook site list won't list all cookbooks at the given Supermarket URL. Defaults to listing the first 100 of a given sort order. If changing the default behavior isn't acceptable, something could be done with some new option switches and two API trips: one to get the total and another to retrieve that number items.

Hard to say whether this behavior change is the right one without know how people use that command.  Some thoughts:

By removing the pagination, it means that if someone sets -n 5000 to ensure they can list all of the cookbooks on the community site, then we make a single request asking for 5000 items.  Do we think that would be problematic for public or private supermarket sites?

An alternative might be to keep the pagination (increasing the page size to 100 or more) and then also adding a configurable overall limit.

I don't think we can change default behaviour in a minor version.

By removing the pagination, it means that if someone sets -n 5000 to ensure they can list all of the cookbooks on the community site, then we make a single request asking for 5000 items. Do we think that would be problematic for public or private supermarket sites?

The Supermarket v1 API currently allows someone to make a request with items = 5000. At the current number of cookbooks on public Supermarket, this hasn't been a problem.

An alternative might be to keep the pagination (increasing the page size to 100 or more) and then also adding a configurable overall limit.

Another alternative is experimenting with chunked transfer encoding. I don't know offhand how the rest client inside chef/knife handles that, though.


Remove non-existent properties from gistValidationAnnotations

These two properties were added to but then removed from gist12.1.0:
gist:symbolUnicode
	gist:nonConformingLabel "true"^^xsd:boolean ;
	.

gist:unitSymbolUnicode
	gist:nonConformingLabel "true"^^xsd:boolean ;
	.

Closing - handled as part of #947: see https://github.com/semanticarts/gist/issues/947#issuecomment-1784213783


Remove version: IrfanSkiljan.IrfanView version 4.67

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://files02.tchspt.com/down/iview467_setup.exe
OK
(Automated message - build 923)

Close with reason: Duplicate;


Roll engine 3c51a7bfff77..d5f15d1cdc83 (7 commits)

https://github.com/flutter/engine/compare/3c51a7bfff77...d5f15d1cdc83
git log 3c51a7bfff776e9e3011384f4ff09c13a697c6b6..d5f15d1cdc83f90b4a78be441725624d34348155 --no-merges --oneline
d5f15d1cd Roll src/third_party/skia de6758683abd..e192c4ce5a33 (2 commits) (flutter/engine#9703)
e46532986 Roll src/third_party/skia 2f62edf1ac5d..de6758683abd (1 commits) (flutter/engine#9700)
05966cd14 Roll src/third_party/skia 1d105080c423..2f62edf1ac5d (1 commits) (flutter/engine#9699)
b132baa44 Ensure that platform messages without response handles can be dispatched. (flutter/engine#9698)
50d622857 fix Picture.toImage return type check and api conform test. (flutter/engine#9685)
001bb541c Roll src/third_party/skia 8063f6cca025..1d105080c423 (3 commits) (flutter/engine#9692)
db4d6836a Roll src/third_party/skia 857c9f955edb..8063f6cca025 (3 commits) (flutter/engine#9679)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jsimmons@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5137981030006784 , https://cirrus-ci.com/task/5700930983428096 , https://cirrus-ci.com/task/5419456006717440 , https://cirrus-ci.com/task/6545355913560064 , https://cirrus-ci.com/task/5982405960138752 , https://cirrus-ci.com/task/4575031076585472 , https://cirrus-ci.com/task/4856506053296128 , https://cirrus-ci.com/task/5910387948519424

Commit queue failed; closing this roll.


Fix the config parser being too strict.

Please read the notes in the readme about pull requests. Additionally, please re-phrase your commit message to read positively instead of negatively. There was nothing wrong with the previous behavior, and it was not broken. Additionally, provide more context in the commit message to explain the change. For tips on writing a good commit message, see http://chris.beams.io/posts/git-commit/

Apparently since I pushed the restored commit history, this pull request has no comment parent and was automatically closed. However, it should be pretty easy to re-open (or rather, open a new one) by fetching my repo, checking it out to a new branch, then cherry-picking your commit on top of the new branch. The code at HEAD is identical to what it was before.


Add link-checking test retries

[x] Fixes #287
[x] Update changelog

Codecov Report

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

@@           Coverage Diff           @@
##           master     #400   +/-   ##
=======================================
  Coverage   77.59%   77.59%
=======================================
  Files          19       19
  Lines        5467     5467
=======================================
  Hits         4242     4242
  Misses       1225     1225

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 8dc8fd5...f940852. Read the comment docs.


Do a deep search when checking for traits

Consider the following.
I want all my models to have the $perEnvironment property set to true. To avoid redundancy, I create my own trait that uses the Algolia trait.
<?php

namespace App\Services;

use AlgoliaSearch\Laravel\AlgoliaEloquentTrait;

trait SearchIndexerTrait
{
	use AlgoliaEloquentTrait;

    public static $perEnvironment = true;
}

And so now I can just use my custom trait in all my models.
<?php

namespace App;

use Illuminate\Database\Eloquent\Model;
use App\Services\SearchIndexerTrait;

class Product extends Model
{
    use SearchIndexerTrait;

    public $algoliaSettings = [
        'attributesToIndex' => [
            'name',
            'text',
        ],
        'attributesToRetrieve' => [
            'name',
            'text',
        ],
        'customRanking' => [
            'asc(name)',
        ],
    ];
}

However, my indices will no longer sync when I update or delete a model.
This is because AlgoliaSearch\Laravel\EloquentSubscriber uses the php class_uses() helper method on the currently updated Model to check if it uses the AlgoliaEloquentTrait in order to determine whether or not the index should be updated. class_uses() only checks the "top level" traits that are used. It does not check the parents traits, or traits inside traits.
To fix the problem, I used this handy fucntion.

@Nikita240 Nice job on that, could you write some test for this ?

+1 Nice, this pull request would really help us to keep our model clean. Thanks

A bit busy rn. Will make tests tomorrow.

Done

@Nikita240 Thanks for contributing.

Can we get a version tag for this PR? I don't want to have to require dev-master.

@Nikita240 Done, I thought I did it but no :smile:

@maxiloc thanks!


Alphabetize Content.

Alphabetize section titles (and tool titles) to match ToC order.
Change "References" section title to "Addendum".

@christianboyle Now everything is in order 👌
thanks!


docs: correct plugin example filename

The file name inside the my-plugin folder is index.js .

@mxhdx Please remember to sign the CLA. It's required for us to merge. PRs without CLA will be closed in two weeks.

@mxhdx can you sign the CLA please?

@mxhdx can you sign the CLA please?

Done.


Moved the sabre/* repos

To: @DominikTo @untitaker @netgusto @jakobsack @staabm @Hywan @strokyl @BastiaanOlij @dennisdegreef @DeepDiver1975 @PVince81 @armin-hackmann @petrkotek @felixfbecker
Hey everyone!
I don't have a way to easily email you all, so I figured I'm just gonna open an issue and include all of you. I moved all the sabre/* repositories to its own organization. What this means is that it's much easier to control access to all repos, instead of me having to manually maintain lists of contributors.
So if you were part of the "To:" list above, it means you got an invite to be a member of this organization, and should have access to all repositories. Not interested? feel free to ignore this!
Anyway, as you might know from my previous blog posst, I no longer have a lot of time to work on sabre/* projects. Some day I might pick up some tasks again, but until then I want to help facilitate people to move forward with these projects.
Reference: https://evertpot.com/sabredav-maintenance-update/
You're in the list because I trust you and I've seen (bits) of your work and work ethic over multiple years. If you are now a member, feel free to:

Do releases
Close issues
Merge PR's
Invite other members

Let me know if you don't have access to something, or if you're unsure how to perform a certain task. I see my role today as a facilitator, but no longer a project runner. The future is yours!

Thanks a lot for your trust @evert - I'm damn sure we will do our best to take care about sabre.

I'll set up travis ....

hmmm ... just lost admin rights ...

My fault. Fixed.

Cool. I just subscribed to Baikal again, but I don't use it at all.

I'll set up travis ....

all integration still seem to be inplace - lets keep that for now and see how far we get ....

I think it would make sense to rename the repos to drop the sabre- prefix then - i.e. the composer package sabre/event is under github.com/sabre/event, not github.com/sabre/sabre-event

@felixfbecker I tried to get the sabre organization name several times over the past few years, but the owner didn't really want to let it go (hence sabre-io, it's the best we could come up with). I agree that it might make sense to drop the second sabre/* regardless though!

I think it would make sense to rename the repos to drop the sabre- prefix

There used to be a problem with email notifications being only tagged with [dav], but since it's [sabre/dav] nowadays, :+1:

Looks like travis is not triggered when pushing to this repo - see https://github.com/sabre-io/sabre-dav/pull/994
@evert can you please have a look where the issue might be - I have no access to the travis account. THX

Taking a look @DeepDiver1975 , and making you an admin there if I can

@DeepDiver1975 I think travis picked up the organization name change. I also manually restarted those builds. Let me know if there's still issues. Can you access this url now?
https://travis-ci.org/sabre-io/sabre-dav

All repository names are now updated to their short names (dav instead of sabre-dav)

Can you by chance grant rights to re-run travis jobs?

Yea I'm just not sure how @DeepDiver1975 ! There doesn't seem real privilege management in travis, so I thought that you just get the privilege by virtue of being a member of the sabre-io github org. Thoughts?

Well ... looks like all just need to be a bit more patient ... sabre-io orga just popped up in travis. 👍

great move @evert. hopefully with this team we can maintain this great packages for the next years. <3

Transition is done - I guess we can close this

Any information about the release process ?


extend AboutWindow

This PR implement previously missing bindings for the following fields in AboutWindow (Adwaita):

applicationIcon
releaseNotes
comments
debugInfo
developers
designers
artists
documenters
licenseType
legalSections
credits
acknowledgements
links

I also added an example for the Adwaita AboutWindow with a screenshot and everything. :slightly_smiling_face:
Originally, I planned to implement all the missing fields.
But since I lack experience with both nim and GTK development, I didn't feel comfortable touching the translators field because it appears to ties into app localization which I was not sure if it was even supported in owlkettle yet.
Some other things I wasn't sure about when implementing the bindings:

legalSections: I defined a custom type to be easier able to fill the list. However I am not sure if thats the way to go here, or if it could be made more concise with standard dictionary/tuple types (which I didn't manage to do).

optional section name in credits: I didn't make it optional in this MR, as I wasn't sure what's the best way to deal with optional values is in nim/owlkettle (see above).

Regarding the legalSections field: Using a custom type a completely valid approach imo.
Regarding optional values: I would say that if possible, using the empty string value as the NULL value. If the widget has different behavior when passing an empty string vs. passing NULL, use std/options.

I appreciate you taking the time to look into the PR! :)
I added a couple commits addressing all your comments.

Regarding optional values: I would say that if possible, use the empty string value as the NULL value. If the widget has different behavior when passing an empty string vs. passing NULL, use std/options.

Thanks! As it turns it in the case of the credits section ( adw_about_window_add_credit_section), using an empty string ("") for the title, already works as intended.

Looks good, thanks for contributing!
I fixed some typos before merging. I also changed the example values for the license field from LicenseArtistic to LicenseMIT_X11, as I am not familiar with the Artistic license and was not sure if it is compatible with MIT (Owlkettle) / LGPL (GTK) (it might very well be, I just don't know). So just to be sure and not accidentally imply a compatibility to potential users, I changed it to LicenseMIT_X11.


No error messages when quota or subscription prevents creation of nodes

In what area(s)?
/area job-scheduling
Expected Behavior
When nodes are not available due to quota or subscription issues, it would be nice to have an indication of this in cyclecloud or ondemand.
Actual Behavior
Cyclecloud refuses to boot the required number of nodes but gives no error why. Ondemand indicates the job is queued and nothing changes.
Steps to Reproduce the Problem
Set quota to 120 vcpus. Schedule an mpi job on slurm requiring two hb120-16rsv2 nodes. Job refuses to run because required quota is 240vcpus (even though you only requested 32...)

This is not a behavior we have control on, you have to understand your quota before configuring your cluster. As an admin you can check in the CycleUI how much quota you have by manually adding a node, which will display how many nodes can be added.


What happened?
Hello,
I looked in the documentation to confirm a bank transfer payment (https://stripe.com/docs/js/payment_intents/payment_method) but the method seems to be missing.
I am using stripe.confirmPaymentIntent (https://stripe.com/docs/js/deprecated/confirm_payment_intent) for now.
Thanks in advance,
Thomas
Environment
No response
Reproduction
No response

This is now accessible; thanks for flagging!


MaskPstnNumbersType parameter and its Description missing

In the cmdlet document page for Set-CsOnlineDialInConferencingTenantSettings, MaskPstnNumbersType parameter and its values description is missing.

Many thanks for noticing and your feedback @Baasky01


JMP3

👷 Deploy Preview for polite-dolphin-8b4710 processing.

Name
Link

🔨 Latest commit
3f1e488746f36acff82e0f5363fb6d2fc3a8922c

🔍 Latest deploy log
https://app.netlify.com/sites/polite-dolphin-8b4710/deploys/635df059935c4e0008aaf058


Install missing prerequisites for Ubuntu package on release

This is a simple oversight on my part; I forgot to replicate this stanza from the test to the release job when making #3955.

Was thinking about asking that on #3955 ... there will certtainly be a few more dependencies on (dynamically-loaded) C libraries required to execute any Haskell program, as well as specific other ones we might need for the crypto hook support.
Let's have a try with the current minimal explicit dependencies, though, as it all depends on the actual use case for the created package.


Race condition when calling a side-effect of an AutoDisposeAsyncNotifierProvider

Discussed in https://github.com/rrousselGit/riverpod/discussions/2502
Describe the bug
void main() async {
  final container = ProviderContainer();
  await container.read(sampleProvider.notifier).fetch();
}

@riverpod
class Sample extends _$Sample {
  @override
  FutureOr<int> build() => 0;

  Future<void> fetch() async {
    state = const AsyncLoading();
    await Future.delayed(const Duration(seconds: 1));
    state = const AsyncData(0);
  }
}

Running this will fail with an exception: Bad state: Future already completed
What happens is:
1- The provider is initialized synchronously with 0
2- state = const AsyncLoading(); get executed and initialize _futureCompleter
3- The provider get disposed after that (no listeners): dispose() method check and find that _futureCompleter has value but at the same time _lastFuture is null (as the provider is synchronously initialized), so it completes _futureCompleter with an error.
4- state = const AsyncData(0); executes after than and find _futureCompleter is not null so it completes it (again). Maybe _futureCompleter should be set to null after completing it in the dispose method to fix this?
I'm curious where's the other part of completing _futureCompleter when _lastFuture is not null (commented here)
Expected behavior
I except the code to run with no exception as the current behavior is suppressing the setter of a disposed provider without throwing an exception.

Your provider got disposed, so it shouldn't be calling state= anymore.
The error isn't the most user-friendly one, your state = const AsyncData(0); isn't valid in your scenario.
That's the same problem as the "don't use build_context across async gap".
You should handle the fact that the provider was canceled within the operation and act accordingly. To begin with, the pattern you've implemented in your notifier (build doing nothing and the fetching being done in a side-effecf) is discouraged. Have build do the fetching instead. If you don't want to fetch immediately, don't read the provider immediately.

Your provider got disposed, so it shouldn't be calling state= anymore.

I agree on that. but the error is just confusing as it's not related to using state setter. I was thinking this should work fine as the current behavior using state = after a provider get disposed doesn't throw.

I was thinking this should work fine as the current behavior of using state = after a provider get disposed doesn't throw.

If you found a case where it doesn't throw, then that's a bug. It should.
The error should be changed to something more reasonable, but you should definitely get an error.

I think currently using state = doesn't throw at all:
https://github.com/rrousselGit/riverpod/blob/8bcf25f7bbee550afb93af4430415e30efd1a03c/packages/riverpod/lib/src/async_notifier/base.dart#L203
It's just ignored as we're checking for mounted before updating the state

Ah well, my bad ha
Anyway it should throw

Removing this line from my sample above will let it work:
state = const AsyncLoading();

Adding it causes the issue because it initializes _futureCompleter again which will then be updated to be completed by dispose method (without setting it to null). Using a setter after that will throw as it checks only if _futureCompleter is not null but not completed:
if (completer != null) {
      completer.complete(value.value);
      _futureCompleter = null;
    }

Ah well, my bad ha
Anyway it should throw

Alright! no problem. but that mean we should be able to check for mounted like with BuildContext?

Check ref.onDispose

I have the same issue. the provider is disposed while  state = await AsyncValue.guard is running.
so Unhandled Exception: Bad state: Future already completed was thrown.
Any practical solution for this?

I have the same issue. the provider is disposed while  state = await AsyncValue.guard is running.
so Unhandled Exception: Bad state: Future already completed was thrown.
Any practical solution for this?

As Remi said you should check if the provider is disposed before setting the state. something like checking mounted for BuildContext.
You might want to check https://github.com/rrousselGit/riverpod/issues/1879 for ideas on how to do that.

@AhmedLSayed9  Thank you, it's a bit cumbersome but it works for me.

state= throws now if called on a disposed notifier. So closing


Update doco

updated Manual Steps guide.
The steps are in a different order, which streamlines the manual process.
Can and possibly should also have PowerShell calls as an option to perform the configuration.

Hi @camiloborges, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by Microsoft and real humans are currently evaluating your PR.
TTYL, MSBOT;

thx Camilo, we will process this in upcoming weeks due resourcing challenges, but we'll absolutely have closer look on this, since feedback is valid and current instructions could be challenging to follow.

Thanks Camilo,
The PR has been merged.

What an honour! Thank you.
Please let me know if I could have made it any easier since I noticed your other PR mentioning conflicts.

No worries Camillo, I had to handle some conflicts between your updates and someone else's updates, but it is a common issue. You did it right. Thanks again.


[Bug] PointerLeave event does not include InputModifiers

Writing a drawing app and I need to process events based on what keys are down while the mouse is moving. If a pointer leaves the canvas a PointerLeave event is fired rather than a PointerMove event meaning the distances between the last move & the current position is not drawn. I have no issues writing code to process the event during the PointerLeave event; However, it does not have the input modifiers set on it like in the Move, Pressed, and Released events. InputModifiers value is always None. I would expect all PointerEvents to pass the Alt, Shift, and Button InputModifiers in the PointerEventArg class.
System: Windows x64

Thanks grokys :)


No intellisense on Express after npm install with a new project

Repro steps:

file > new project > express 4 template
wait for packages to install and analysis to complete
hovering over app yields unknown type

When this issue occurs, of the top-level packages installed (the ones that appear in package.json and were installed with ), some appear to have been analyzed and some don't:

:x: body-parser: unknown type
:x: cookie-parser: unknown type
:x: debug: unknown type
:x: express: unknown type:
:heavy_check_mark: jade: completions available
:heavy_check_mark: morgan: completions available
:heavy_check_mark: serve-favicon: completions available
:heavy_check_mark: stylus: completions available

An interesting coincidence about this list is that the packages yielding unknown type are those packages sorted towards the beginning of the alphabet. This is consistently the case, and this issue appears to be unrelated to the sort order in package.json.
The express 3 template yields similar results:

:x: express: unknown type
:x: jade: unknown type
:heavy_check_mark: stylus: completions available

Workaround: if you install any other package, say azure, completions are suddenly available for all of the packages (including those towards the beginning of the alphabet.)

This issue also affects all dependencies of a package. For instance...
// completions for all express dependencies not available
var connect = require('./node_modules/express/node_modules/connect');

// completions for all stylus dependencies available
var cssParse = require('./node_modules/stylus/node_modules/css-parse');

This problem is currently affecting me, and I tried running:
npm install azure --save
In my project's directory but Express still does not show any Intellisense options. Installing another package (jquery) also did not help. Perhaps I am misunderstanding your workaround.


add entity labels card

Hey, I just made a Pull Request!
add entity labels card that looks like this.
    <EntitySwitch>
      <EntitySwitch.Case if={hasLabels}>
        <Grid item md={4} xs={12}>
          <EntityLabelsCard />
        </Grid>
      </EntitySwitch.Case>
    </EntitySwitch>

If the entity does not have labels it looks like this

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[x] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

As a backstage user, I would not add this card because the "no label" state of the card is a little scary. People may think they have done something wrong as labels are missing.
Maybe there should be a hasLabel function so people could choose if they want to not display the component if there is no labels?

@jcarres-mdsol it doesn't look like it's being added to new instances created using npx @backstage/create-app@latest and existing installs would need to add it manually. Finally this is the same text used on various card, for example Links:

This was taken from there demo site here: https://demo.backstage.io/catalog/default/component/playback-order

I guess in my particular case I know (due to a processor) that we will always have links so never worried about this while I know labels may or may not be present.
I still think in general would be nice if the empty case in any of these components could be either "point to what's missing" or "do not render if not data available". I think that would be more flexible but probably this is outside the scope of this particular component.

I can add a boolean function like entityHasLabels if it helps. But you can also choose to just not add the card.

@jcarres-mdsol Ive added a hasLabels entity switch function. Hope that helps.

To once again compare this to the "Links" component, what do you think to dropping the word Entity from the title of the card, so it is just "Labels"?

I dont think the failure here is related to my change.


Errors and warnings in test CI annotation

There are 20 errors and 2 warnings in annotations in our GHA, e.g. runs/7558932353, but the test themselves are passing fine.
It is probably a good idea to dig into this to get the root cause of this and make it go away.

Related: https://github.com/golangci/golangci-lint-action/issues/135


The first working implementation

The first working implementation.
Example added.(examples/ppgen.cpp)
But I can`t add the Boost detection to the CMakeLists.txt. I did the same as described here[1], but unsuccessfully. Can you please do it by yourself?
Ok to merge?
[1] https://stackoverflow.com/questions/6646405/how-do-you-add-boost-libraries-in-cmakelists-txt

I took a quick look at the code. From what I can gather, it looks quite clean and good. I'll try to merge and take care of the Boost detection within the next few days.
I'll let you know as soon as possible, of course.

Ok.

find_package(Boost)
if (Boost_FOUND)
MESSAGE("Boost Found")
else()
MESSAGE("Boost not found")
endif()
Seems to work fine. It does not check for the preprocessor, but it should be OK for now.
Need to sleep now :-)

Which version of boost do you use? I have 1.54 on my machine, and get this:
$ make
-- Using /usr/bin/c++ (compiler id: GNU)
-- Boost version: 1.54.0
/usr/include
-- Configuring done
-- Generating done
-- Build files have been written to: /home/rbock/temp/nixman/sqlpp11
[ 16%] Built target UnionTest
[ 33%] Built target Sqlpp11Exampleinsert
[ 50%] Building CXX object examples/CMakeFiles/Sqlpp11Exampleppgen.dir/ppgen.cpp.o
In file included from /home/rbock/temp/nixman/sqlpp11/examples/ppgen.cpp:38:0:
/home/rbock/temp/nixman/sqlpp11/include/sqlpp11/ppgen.h:74:50: fatal error: boost/preprocessor/tuple/pop_front.hpp: No such file or directory
#include <boost/preprocessor/tuple/pop_front.hpp>
^
compilation terminated.
make[2]: *** [examples/CMakeFiles/Sqlpp11Exampleppgen.dir/ppgen.cpp.o] Error 1
make[1]: *** [examples/CMakeFiles/Sqlpp11Exampleppgen.dir/all] Error 2
make: *** [all] Error 2
$ ls /usr/include/boost/preprocessor/tuple
eat.hpp  elem.hpp  enum.hpp  rem.hpp  reverse.hpp  size.hpp  to_array.hpp  to_list.hpp  to_seq.hpp
pop_front.hpp is available in array of seq though.

Which version of boost do you use?

1.57.
BOOST_PP_TUPLE_POP_FRONT() is available in 1.56 or greater.

I have very little knowledge of boost preprocessing. Would it be hard to make your code work with slightly older versions?
If yes, then we can stick with 1.56 and greater, of course.

I have to add the implementation of BOOST_PP_TUPLE_POP_FRONT() into ppgen...

OK, lets wait for feedback first. I'll install a more recent version on my machine.

OK, I have boost 1.57 now, works fine :-)
And the code in the example is really quite nice!
How did you produce the preprocessor output you showed me earlier? Is there something like boost_stringify for the generated table code?

How did you produce the preprocessor output you showed me earlier?

Like this: g++ -std=c++11 -E main.cpp > out.hpp && clang-format-3.5 -i out.hpp

I have to try to add the implementation of BOOST_PP_TUPLE_POP_FRONT() into ppgen?

I think, for the time being, I'll write that boost-1.56 is required to use ppgen. I just merged and will add the boost-lookup to the CMakeLists tomorrow.
Thank you very much for implementing the preprocessor table generator!

I think, for the time being, I'll write that boost-1.56 is required to use ppgen. I just merged and will add the boost-lookup to the CMakeLists tomorrow.

Ok.

Thank you very much for implementing the preprocessor table generator!
My pleasure ;)


Create quant2.md

[x] Data Prep
[x] Part 1
[ ] Part 2
[x] Extra Credit
[ ] Write up
[ ] Memo
[ ] Revise & revisit
[ ] Submit

Data prep complete, Extra credit complete

part 1 section 1 finish, part 1 section2 in progress

part one officially finsih

PR Code Suggestions ✨
No code suggestions found for the PR.

@CodiumAI-Agent /update_changelog
--pr_update_changelog.push_changelog_changes=true


Define opts with default empty keyword list performing bulk

Hi there! 👋
All the options have defaults, so I think that we could safely define a default empty list for the options. Also, reading the @doc looks like it was the intention no? 😄

Thanks!


Added workspace "num" attribute to Con object.

This is useful when handling workspace change events.

:+1: Thanks for your contribution.