Updating feed

There's no change between 2.7.2100 and 2.7.2184 - https://github.com/Azure/azure-functions-core-tools/compare/2.7.2100...2.7.2184
This is because the update was in a release pipeline and we need to create new version to notify users.
I am not going through the usual E2E tests suite given there's no delta.

Actually this PR didn't update. Closing and manually creating the PR


[graylog] Update chart to support Graylog 3

What this PR does / why we need it:

Upgrade default Graylog image to version 3.0.1
Add new graylog.journal.deleteBeforeStart option for delete Journal files before start. This will help to resolve the problem when Graylog journal files corrupted.
Update some error in documentation on README and NOTES.txt
Move startup script to configmap. This will make Pod looks clean

This Chart is backward compatibility with Graylog. 2.x
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/ok-to-test

/lgtm

/assign maver1ck

/assign cpanato

/ok-to-test

/lgtm


feat: doc improvements

This PR attempts to add some improvements to the docs:

Use more modern syntax/features (short functions in 7.4 and throw as an expression in 8) to keep the examples concise
Remove <?php
Always use function imports
Always use camelCase for variables
Standardize the presentation of the example results
Fix an example that would yield a parse error
Fix several spelling and grammar errors

Terrific, thank you!


MDQ broken with MySQL

Hi @smangelsdorf
I've just merged the latest develop to my saml service DEV copy and I can no longer access the MDQ URL.
When I try:
wget -O tuakiri-dev-entities.xml x --header 'Accept: application/samlmetadata+xml' https://saml.dev.tuakiri.ac.nz/mdq/dev-tuakiri-ac-nz/entities

I get a 500 Internal Server error.  The Ruby error page says:
Mysql2::Error: Expression #1 of ORDER BY clause is not in SELECT list, references column 'service_names.value' which is not in SELECT list; this is incompatible with DISTINCT

After merging the changes, I did run
gem install bundler
bundler install
rake db:migrate

So I'm assuming I have the right table structure.
I don't know if this is a bug in the new code or misconfiguration at my end - but reporting it in case it's helpful.
Cheers,
Vlad

The relevant bits from what I see in the logs are:
  Sequel::Mysql2::Database (0.8ms)  SELECT `endpoints`.`id`, `endpoints`.`binding`, `endpoints`.`location`, `endpoints`.`response_location`, `endpoints`.`created_at`, `endpoints`.`updated_at`, `indexed_endpoints`.`is_default`, `indexed_endpoints`.`index`, `assertion_consumer_services`.`sp_sso_descriptor_id` FROM `endpoints` INNER JOIN `indexed_endpoints` ON (`indexed_endpoints`.`id` = `endpoints`.`id`) INNER JOIN `assertion_consumer_services` ON (`assertion_consumer_services`.`id` = `indexed_endpoints`.`id`) WHERE (`assertion_consumer_services`.`sp_sso_descriptor_id` = 68) ORDER BY `index`, `location`

==> log/unicorn-stderr.log <==
Exception `Mysql2::Error' at /home/samlsvc/.rbenv/versions/2.2.7/lib/ruby/gems/2.2.0/gems/mysql2-0.4.6/lib/mysql2/client.rb:120 - Expression #1 of ORDER BY clause is not in SELECT list, references column 'service_names.value' which is not in SELECT list; this is incompatible with DISTINCT
Exception `Mysql2::Error' at /home/samlsvc/.rbenv/versions/2.2.7/lib/ruby/gems/2.2.0/gems/activesupport-5.1.1/lib/active_support/notifications/instrumenter.rb:25 - Expression #1 of ORDER BY clause is not in SELECT list, references column 'service_names.value' which is not in SELECT list; this is incompatible with DISTINCT

==> log/development.log <==
  Sequel::Mysql2::Database (1.4ms)  SELECT DISTINCT `attribute_consuming_services`.* FROM `attribute_consuming_services` INNER JOIN (SELECT `localized_names`.`id`, `localized_names`.`value`, `localized_names`.`lang`, `localized_names`.`created_at`, `localized_names`.`updated_at`, `service_names`.`attribute_consuming_service_id` FROM `localized_names` INNER JOIN `service_names` ON (`service_names`.`id` = `localized_names`.`id`)) AS `service_names` ON (`service_names`.`attribute_consuming_service_id` = `attribute_consuming_services`.`id`) WHERE (`attribute_consuming_services`.`sp_sso_descriptor_id` = 68) ORDER BY `service_names`.`value`

==> log/unicorn-stderr.log <==
Exception `Mysql2::Error' at /home/samlsvc/.rbenv/versions/2.2.7/lib/ruby/gems/2.2.0/gems/sequel-rails-0.9.15/lib/sequel_rails/sequel/database/active_support_notification.rb:21 - Expression #1 of ORDER BY clause is not in SELECT list, references column 'service_names.value' which is not in SELECT list; this is incompatible with DISTINCT
Exception `Sequel::DatabaseError' at /home/samlsvc/.rbenv/versions/2.2.7/lib/ruby/gems/2.2.0/gems/sequel-4.47.0/lib/sequel/database/misc.rb:421 - Mysql2::Error: Expression #1 of ORDER BY clause is not in SELECT list, references column 'service_names.value' which is not in SELECT list; this is incompatible with DISTINCT

==> log/development.log <==
  Sequel::Mysql2::Database (0.4ms)  ROLLBACK

==> log/unicorn-stderr.log <==
Exception `Sequel::DatabaseError' at /home/samlsvc/.rbenv/versions/2.2.7/lib/ruby/gems/2.2.0/gems/sequel-4.47.0/lib/sequel/database/misc.rb:423 - Mysql2::Error: Expression #1 of ORDER BY clause is not in SELECT list, references column 'service_names.value' which is not in SELECT list; this is incompatible with DISTINCT
Exception `Sequel::DatabaseError' at /home/samlsvc/.rbenv/versions/2.2.7/lib/ruby/gems/2.2.0/gems/actionpack-5.1.1/lib/action_controller/metal/rescue.rb:23 - Mysql2::Error: Expression #1 of ORDER BY clause is not in SELECT list, references column 'service_names.value' which is not in SELECT list; this is incompatible with DISTINCT
Exception `Sequel::DatabaseError' at /home/samlsvc/.rbenv/versions/2.2.7/lib/ruby/gems/2.2.0/gems/activesupport-5.1.1/lib/active_support/notifications/instrumenter.rb:25 - Mysql2::Error: Expression #1 of ORDER BY clause is not in SELECT list, references column 'service_names.value' which is not in SELECT list; this is incompatible with DISTINCT

==> log/development.log <==
Completed 500 Internal Server Error in 1270ms (Models: 597.3ms)

==> log/unicorn-stderr.log <==
Exception `Sequel::DatabaseError' at /home/samlsvc/.rbenv/versions/2.2.7/lib/ruby/gems/2.2.0/gems/actionpack-5.1.1/lib/action_dispatch/middleware/callbacks.rb:30 - Mysql2::Error: Expression #1 of ORDER BY clause is not in SELECT list, references column 'service_names.value' which is not in SELECT list; this is incompatible with DISTINCT

==> log/development.log <==

Sequel::DatabaseError (Mysql2::Error: Expression #1 of ORDER BY clause is not in SELECT list, references column 'service_names.value' which is not in SELECT list; this is incompatible with DISTINCT):

lib/metadata/saml.rb:486:in `block in sp_sso_descriptor'
lib/metadata/saml.rb:480:in `sp_sso_descriptor'
lib/metadata/saml.rb:265:in `block (2 levels) in entity_descriptor'
lib/metadata/saml.rb:264:in `each'
lib/metadata/saml.rb:264:in `block in entity_descriptor'
lib/metadata/saml.rb:256:in `entity_descriptor'
lib/metadata/saml.rb:71:in `block in render_known_entities'
lib/metadata/saml.rb:70:in `each'
lib/metadata/saml.rb:70:in `render_known_entities'
lib/metadata/saml.rb:64:in `block (2 levels) in entities_descriptor'
lib/metadata/saml.rb:60:in `block in entities_descriptor'
lib/metadata/saml.rb:55:in `entities_descriptor'
app/controllers/concerns/metadata_query_caching.rb:59:in `block in cache_known_entities_response'
app/controllers/concerns/metadata_query_caching.rb:58:in `cache_known_entities_response'
app/controllers/metadata_query_controller.rb:122:in `create_known_entities_response'
app/controllers/metadata_query_controller.rb:56:in `block in handle_entities_request'
app/controllers/metadata_query_controller.rb:47:in `handle_entities_request'
app/controllers/metadata_query_controller.rb:21:in `all_entities'

And adding the Rails error page in case it helps...
err.zip

Which MySQL version is this? I've not seen this behaviour.

Hi @bradleybeddoes , sorry, this question slipped in my inbox.
The DB used was MySQL 5.7.22-0ubuntu<IP_ADDRESS> onto Ubuntu 16.04.
Cheers,
Vlad

@bradleybeddoes , btw I've just checked with latest develop and still getting the same:
Sequel::DatabaseError in MetadataQueryController#all_entities
Mysql2::Error: Expression #1 of

lib/metadata/saml.rb:503:in `block in sp_sso_descriptor'
lib/metadata/saml.rb:497:in `sp_sso_descriptor'
lib/metadata/saml.rb:256:in `block (2 levels) in entity_descriptor'
lib/metadata/saml.rb:255:in `each'
lib/metadata/saml.rb:255:in `block in entity_descriptor'
lib/metadata/saml.rb:247:in `entity_descriptor'
lib/metadata/saml.rb:71:in `block in render_known_entities'
lib/metadata/saml.rb:70:in `each'
lib/metadata/saml.rb:70:in `render_known_entities'
lib/metadata/saml.rb:64:in `block (2 levels) in entities_descriptor'
lib/metadata/saml.rb:60:in `block in entities_descriptor'
lib/metadata/saml.rb:55:in `entities_descriptor'
app/controllers/concerns/metadata_query_caching.rb:59:in `block in cache_known_entities_response'
app/controllers/concerns/metadata_query_caching.rb:58:in `cache_known_entities_response'
app/controllers/metadata_query_controller.rb:122:in `create_known_entities_response'
app/controllers/metadata_query_controller.rb:56:in `block in handle_entities_request'
app/controllers/metadata_query_controller.rb:47:in `handle_entities_request'
app/controllers/metadata_query_controller.rb:21:in `all_entities'

Can't say I can trigger it on this side, looks like something with MySQL 5.7, we're on 5.6.
$ wget -O entities.xml --header 'Accept: application/samlmetadata+xml' https://saml.test.aaf.edu.au/mdq/aaf/entities
--2018-05-08 13:33:22--  https://saml.test.aaf.edu.au/mdq/aaf/entities
Resolving saml.test.aaf.edu.au (saml.test.aaf.edu.au)... <IP_ADDRESS>
Connecting to saml.test.aaf.edu.au (saml.test.aaf.edu.au)|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: unspecified [application/samlmetadata+xml]
Saving to: ‘entities.xml’

entities.xml                                    [  <=>                                                                                ]   3.54M  15.9MB/s    in 0.2s

2018-05-08 13:34:14 (15.9 MB/s) - ‘entities.xml’ saved [3717494]

$ head entities.xml
<?xml version="1.0" encoding="UTF-8"?>
<EntitiesDescriptor xmlns="urn:oasis:names:tc:SAML:2.0:metadata" xmlns:saml="urn:oasis:names:tc:SAML:2.0:assertion" xmlns:idpdisc="urn:oasis:names:tc:SAML:profiles:SSO:idp-discovery-protocol" xmlns:mdrpi="urn:oasis:names:tc:SAML:metadata:rpi" xmlns:mdui="urn:oasis:names:tc:SAML:metadata:ui" xmlns:mdattr="urn:oasis:names:tc:SAML:metadata:attribute" xmlns:shibmd="urn:mace:shibboleth:metadata:1.0" xmlns:ds="http://www.w3.org/2000/09/xmldsig#" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:fed="http://docs.oasis-open.org/wsfed/federation/200706" xmlns:privacy="http://docs.oasis-open.org/wsfed/privacy/200706" xmlns:remd="http://refeds.org/metadata" ID="aaf20180508033322" Name="https://md.test.aaf.edu.au/aaf-test-metadata.xml" validUntil="2018-05-15T03:33:22Z">
  <ds:Signature>
    <ds:SignedInfo>
      <ds:CanonicalizationMethod Algorithm="http://www.w3.org/2001/10/xml-exc-c14n#"/>
      <ds:SignatureMethod Algorithm="http://www.w3.org/2001/04/xmldsig-more#rsa-sha256"/>
      <ds:Reference URI="#aaf20180508033322">
        <ds:Transforms>
          <ds:Transform Algorithm="http://www.w3.org/2000/09/xmldsig#enveloped-signature"/>
          <ds:Transform Algorithm="http://www.w3.org/2001/10/xml-exc-c14n#"/>

...

Hi @bradleybeddoes
With a thinking cap on, I've managed to narrow this down to one particular EntityDescriptor, for an SP that does not request any attributes in FR and so does not have an AttributeConsumingService element at all - and so also no ServiceName.
Same error when coming via
https://saml.dev.tuakiri.ac.nz/mdq/dev-tuakiri-ac-nz/entities/%7Bsha1%7D6128a70c13f11b63363350a8df5e17e926e32a52

Does this help?
Cheers,
Vlad

Attaching:

metadata of the troublesome EntityDescriptor
error page rendered when accessing the single entity

err-single-entity.zip

Hi @bradleybeddoes ,
Apologies, I got confused on this and I need to correct my position:

I can correctly render metadata for entities that do NOT contain an SPSSODescriptor.
I however get the failure for entities that contain an SPSSODescriptor, regardless of whether they contain an AttributeConsumingService or not.
(Apologies, I got confused about this by running a test against a wrong entityID - the IdP side of the VHO instead of the SP side of the VHO).

I'm now looking again at the MySQL code that trips the error:
SELECT DISTINCT `attribute_consuming_services`.* FROM `attribute_consuming_services` INNER JOIN (SELECT `localized_names`.`id`, `localized_names`.`value`, `localized_names`.`lang`, `localized_names`.`created_at`, `localized_names`.`updated_at`, `service_names`.`attribute_consuming_service_id` FROM `localized_names` INNER JOIN `service_names` ON (`service_names`.`id` = `localized_names`.`id`)) AS `service_names` ON (`service_names`.`attribute_consuming_service_id` = `attribute_consuming_services`.`id`) WHERE (`attribute_consuming_services`.`sp_sso_descriptor_id` = 150) ORDER BY `service_names`.`value`

Sequel::DatabaseError (Mysql2::Error: Expression #1 of ORDER BY clause is not in SELECT list, references column 'service_names.value' which is not in SELECT list; this is incompatible with DISTINCT):

The SQL is a bit tricky to navigate through (double nested INNER JOIN), but, at the top level, it starts with SELECT ... attribute_consuming_services.* FROM...
It does NOT list the columns from the joined tables to be selected - and MySQL is complaining about that - or, specifically, about using one of these columns for ORDER BY.
If I change that SQL to also explicitly include the service_names.value field, it works:
SELECT DISTINCT `attribute_consuming_services`.*, service_names.value FROM `attribute_consuming_services` INNER JOIN (SELECT `localized_names`.`id`, `localized_names`.`value`, `localized_names`.`lang`, `localized_names`.`created_at`, `localized_names`.`updated_at`, `service_names`.`attribute_consuming_service_id` FROM `localized_names` INNER JOIN `service_names` ON (`service_names`.`id` = `localized_names`.`id`)) AS `service_names` ON (`service_names`.`attribute_consuming_service_id` = `attribute_consuming_services`.`id`) WHERE (`attribute_consuming_services`.`sp_sso_descriptor_id` = 150) ORDER BY `service_names`.`value`;

I have no idea how this relates to the way Rails constructs the SQL code (and what would be the way to fix it) - but at the SQL level, this is what makes the difference at my end....
Hope this helps.
Cheers,
Vlad

Thanks - will leave this open for if/when we consider a move to MySQL 5.7, in the interim if you have a pressing need it will be the version of Sequel that we're using that will need to be investigated and tweaked here.

Thanks @bradleybeddoes .
I've just tried upgrading sequel from 4.49.0 to 5.8.0 and ended up getting:
Sequel::Error (class_table_inheritance with duplicate column names (other than the primary key column) is not supported, make sure tables have unique column names (duplicate columns: [:entity_descriptor_id]). If this is desired, specify these columns in the :ignore_subclass_columns option when initializing the plugin):

So, the upgrade won't be as easy ... I will now park this at my end too, and if we get to actually roll out SAML service, I'll dig into with the above information...
Thanks for all your help with this!
Cheers,
Vlad

Fixed by  f99eaf4 in #192


Support APISIX plugin - opa

https://apisix.apache.org/docs/apisix/plugins/opa/
In implementation, however, we need not replicate APISIX exactly, but instead should focus on the actual user needs behind the plugins, combined with the specific circumstances of Higress, to re-abstract configuration fields and corresponding capabilities. We can draw lessons from others, but we should not copy them exactly.

/sig


undefined method named "of" of class "GuzzleHttp\Promise\Coroutine"

PHP version: 7.4.10
Description
After updating to guzzle 1.4.0 there occurs an error that the mehtod Coroutine::of() method cannot be found.
Attempted to call an undefined method named "of" of class "GuzzleHttp\Promise\Coroutine".

How to reproduce
Seems like an issue when using the coroutine() function. See stacktrace below.
Additional context
This issue occured in combination with using flysystem & the aws s3 adapter of flysystem.
The error occured when Filesystem::deleteDir() was called.
Full stacktrace:
Exception trace:
  at /var/www/html/vendor/guzzlehttp/promises/src/functions.php:362
 GuzzleHttp\Promise\coroutine() at /var/www/html/vendor/aws/aws-sdk-php/src/S3/BatchDelete.php:109
 Aws\S3\BatchDelete::Aws\S3\{closure}() at n/a:n/a
 call_user_func() at /var/www/html/vendor/aws/aws-sdk-php/src/S3/BatchDelete.php:217
 Aws\S3\BatchDelete->createPromise() at /var/www/html/vendor/aws/aws-sdk-php/src/S3/BatchDelete.php:118
 Aws\S3\BatchDelete->promise() at /var/www/html/vendor/aws/aws-sdk-php/src/S3/S3ClientTrait.php:139
 Aws\S3\S3Client->deleteMatchingObjectsAsync() at /var/www/html/vendor/aws/aws-sdk-php/src/S3/S3ClientTrait.php:110
 Aws\S3\S3Client->deleteMatchingObjects() at /var/www/html/vendor/league/flysystem-aws-s3-v3/src/AwsS3Adapter.php:202
 League\Flysystem\AwsS3v3\AwsS3Adapter->deleteDir() at /var/www/html/vendor/league/flysystem/src/Filesystem.php:251
 League\Flysystem\Filesystem->deleteDir() at /var/www/html/vendor/shopware/storefront/Theme/ThemeCompiler.php:110

Looks like you need to restart php-fpm because you have a stale cache of the Coroutine.php file. Either that, or you need to delete you vendor folder and composer cache, and run composer update.

We have immutable infrastructure and build the whole container anew for each update, including the composer dependencies.
Meaning restarting php-fpm or running composer update should not have any affect as we do a clean composer install each time.
The logs from composer update clearly state that it installed v1.4.0:
  - Downloading guzzlehttp/psr7 (1.7.0)
  - Downloading guzzlehttp/promises (1.4.0)
  - Downloading guzzlehttp/guzzle (6.5.2)

The thing that confuses me most is that functions.php has to be up2date as it tries to make the "new" call to the Coroutine::of() method, but the loaded Coroutine class, that should be loaded from the same folder/namespace seems to be an old version that does not have the of() method. :thinking:

That indicates there is not a bug here, just that your vendor folder didn't get updated properly. Delete you composer cache directory as well as the vendor folder. The 1.4.0 zip definitely has the correct files in it: https://github.com/guzzle/promises/archive/1.4.0.zip.

After quite some debugging we found the source for this problems. The Coroutine class was autoloaded from another vendor folder than the functions file.
But thank you for your fast replies & your support.


Added Varvara

I must be missing something, but where can I download and play Varvara?

Depends which platform you want to play varvara on, but on NDS it would be here, for desktop platforms it would be here, for playdate it would be here, for 6502 it wou.. well, there's a whole list of emulators to download here
Alternatively, you could just run it in your browser

added. thanks!


WMS ID : 9501 - Kubernetes Solution Installation

WMS ID : 9501
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[x] All filenames are lower case (including folders, images, files, etc.)
[x] Filenames are descriptive
[x] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[x] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[x] Image references in markdown contain an alternate text

Hi @samasahu , as I mentioned to @vikrredd - please un-bold the Tasks in labs 1 & 2. For example, your tasks look like ## Task 1: Navigating to Management Agent Service, you can take out the * to solve the issue ## Task 1: Navigating to Management Agent Service. Only one of you needs to submit this change

Hi @KillianLynch , @vikrredd has incorporated the changes for the lab 1 in his PR.


CumulusTV Community

I've been getting a number of emails and messages with common problems. I should create a community to forward people to so that users can work out issues in a central forum.

Or a subreddit haha

What about xda? I think this is the place for app to become more popular!

My initial thought is to use Google+ as that is a common place for Android users. A subreddit may not be a bad idea either, although I don't know how much control I get as a mod (especially for notification settings).
I don't know if XDA will allow me to have my own subforum just for this app.

@Fleker As a mod of a G+ community with almost 2,000 users. DON'T! As much as I love G+, moderating is near impossible.

What would you recommend?

@Fleker Well a few options I would consider:

Reddit: big userbase but "meh" moderation under reddit's rules
MightyBell: more Social network like, full moderation control, freemium, posts can have full formatting with a Google Docs style editor
NodeBB: self-hosted with complete control over everything and open source/free, major projects use it (like Jekyll, Popcorn Time (before shutdown), Notepad++, etc), expandable with themes and plugins, and much more. It can be freely hosted using https://oasis.sandstorm.io and people can sign in with their Sandstorm account (which is GitHub or Google sign in).

@Fleker well and considering this project was born from a Reddit idea, I think it works.

https://www.reddit.com/r/cumulustv/


Add mentorship form to new site

Now that we have a mentorship program in place (at least in beta 😅 ) we should be more proactive about letting people know! I'd like to update the website to include the link to the mentorship form and a quick blurb about what the program is.

If someone has a blurb for me, I can add it in.

bumping this ...

I've got a prototype Airtable base set up to help manage this. @davidgs and I will be going over it next week. If it seems reasonable, imho we could launch in May.

Doing some cleanup, and figured I would see if we still wanted to try to do a Mentorship form, or if it's something we'll come back to at a later date? (@phrawzty / @mary-grace )

I know I'm not on the admin-team any longer, but I'd like to take a stab at making this a reality if that's ok.

Thanks for that @davidgs - we need to identify how we'll proceed with doing this and then we'll get back to you!


Dart libraries are initialized before the first dart

On Dartium, the Dart libraries appear to be initialized before the first dart script tag is processed in the html entrypoint. For example, if the first script tag in my html entrypoint replaces XMLHttpRequest, this is not reflected by HttpRequest in the Dart runtime. This is different than how browsers appear to normally work with scripts in html, where they are processed in the order they are encountered. This causes problems using JavaScript libraries that rely on being able to replace objects in the browser (e.g. sinonjs, pretenderjs).

Dartium's definition of the DOM APIs is separate from the JS definitions today (they go directly to the native APIs within chrome). I understand that @terrylucas and @alan-knight are changing this to start using js-interop instead. I'm not sure though if your use case would be addressed at that point.

That change is live on bleeding-edge right now, so you can see if it works or not.

Version of Dartium on bleeding-edge should work for your use case if Dart2Js works for your use case.

I created tests for BrowserClient and HttpRequest and ran them in Chrome. Both failed.
00:15 +5 -1: basic tests should work with BrowserClient
  type 'UnknownJavaScriptObject' is not a subtype of type 'HttpRequest'
  dart:_internal                            _LinkedHashSet.add
  package:http/browser_client.dart 50:7     BrowserClient.send.<fn>
  ===== asynchronous gap ===========================
  dart:async                                _Future.then
  package:http/browser_client.dart 48:12    BrowserClient.send
  ===== asynchronous gap ===========================
  dart:_internal                            asyncHelper
  package:http/src/base_client.dart 158:18  BaseClient._sendUnstreamed
  package:http/src/base_client.dart 158:18  BaseClient._sendUnstreamed
  ===== asynchronous gap ===========================
  dart:_internal                            asyncHelper
  package:http/src/base_client.dart 34:33   main.<fn>.<fn>
  package:stack_trace                       StackZoneSpecification.registerUnaryCallback.<fn>.<fn>

00:15 +5 -2: basic tests should work with HttpRequest
  NoSuchMethodError: method not found: 'addEventListener$3' (J.getInterceptor$x(...).addEventListener$3 is not a function)
  package:http_parser/src/utils.dart 23:1        J.x
  dart:html                                      HttpRequest.getString
  package:brproxy/brproxy.dart 11:34             $call
  package:stack_trace                            StackZoneSpecification.registerUnaryCallback.<fn>
  ===== asynchronous gap ===========================
  dart:_internal                                 asyncHelper
  package:matcher/src/core_matchers.dart 109:48  $call
  ===== asynchronous gap ===========================
  dart:_internal                                 asyncHelper
  package:matcher/src/core_matchers.dart 109:48  main.<fn>.<fn>
  package:stack_trace                            StackZoneSpecification.registerUnaryCallback.<fn>.<fn>

It looks like dart2js is picking up my version of XMLHttpRequest, but failing for different reasons in each case?


thumbor for image resize

[ ] Background image
[ ] Rearrange components

will use css instead of resizing images


Anchor links for documentation headings

Summary
Add anchor links to the headings of the Clarity documentation so you can reference a specific section of the docs.
Use case
You can reference a specific section of the docs.
Examples
Example from the angular docs:

Which links to: https://angular.io/guide/npm-packages#packagejson

Yes, this is something on my radar to address but in a larger architectural update.

No longer an issue or resolved in the upcoming website work. Closing as resolved.


No module named yaml

Provision fails on TASK [Gather Cluster facts]
n exception occurred during task execution. To see the full traceback, use -vvv. The error was: ImportError: No module named yaml
fatal: [master1]: FAILED! => {"changed": false, "module_stderr": "Traceback (most recent call last):\n  File \"/tmp/ansible_QFM22F/ansible_module_openshift_facts.py\", line 15, in <module>\n    import yaml\nImportError: No module named yaml\n", "module_stdout": "", "msg": "MODULE FAILURE", "rc": 1}

This one is a show stopper. Yaml is the key component of Ansible. Google shows no workarounds for this one.

I've updated it to Openshift 3.9, it should work now.


improve autogenerated jsdocs

Related #8632

Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

It adds some jsdoc comments to support devs when using the client-preset:

In case the generated types are out of sync it provides a hint to regenerate them:

It explains why the babel-plugin should be used for production:

How Has This Been Tested?
Unfortunately I couldn't update the snapshots on my machine.
Jest showed the following error:
Jest: Couldn't locate all inline snapshots.
Checklist:

[x] I have followed the CONTRIBUTING doc and the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas

Hi @jantimon,
That's a great idea, thank you!
Could you run the test suite locally with the following cli flags to update the snapshots?
yarn test --updateSnashot

Also, you will need to run the E2E tests and commit the result:
yarn generate:examples

Note: both commands must be run at the project root

Unfortunately yarn test --updateSnashot does not work there is any inline snapshots mismatch 🤷♂
I managed to fix all tests manually :)
Pipeline is now green


⚠ Bofrost.fr has degraded performance

In 290f771, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 965 ms

Resolved: Bofrost.fr performance has improved in 18630d1 after 14 minutes.


Define contribution process across dotcom

Lisa-Lurie created the following on Apr 24:
As a contributor of code, I need to clearly know the process of adding new code or fixing a bug so that my work can be prioritized and accepted
This part of the contribution guidelines is so crucial for contributors not on our team. This document should provide our contributors with clear expectations from our team, and step by step instructions of how they can go from zero to hero.

Resources for reference

GitHub Help: Setting guidelines for repository contributors

Acceptance criteria

[ ] CONTRIBUTING.md file exists for v18 within .github folder.
[ ] CONTRIBUTING.md file exists for v18-starter-kit within .github folder.
[ ] CONTRIBUTING.md file exists for v19a within .github folder.
[ ] CONTRIBUTING.md file exists for patterns within .github folder.
[ ] CONTRIBUTING.md file exists for digital-design within .github folder.
[ ] CONTRIBUTING.md file exists for carbon-integration-poc within .github folder.
[ ] CONTRIBUTING.md file exists for northstar-external-website within .github folder.
[x] CONTRIBUTING.md file exists for IBM/carbon-addons-dotcom within .github folder.
[ ] [TBD: #726] CONTRIBUTING.md file exists for IBM/dotcom-services within .github folder.
[ ] [TBD: #743] CONTRIBUTING.md file exists for IBM/dotcom-bundles within .github folder.
[ ] [TBD: #695] CONTRIBUTING.md file exists for IBM/dotcom-website within .github folder.

Original issue: https://github.ibm.com/webstandards/digital-design/issues/745

On Apr 24, james-dow commented:
~@Lisa-Lurie is this issue different from #472?~ Talked to @Lisa-Lurie via Slack, and she differentiated them by saying this lives in the codebase, and #472 lives on the website itself. I'll update the tickets to try and clearly communicate this.

On Apr 26, james-dow commented:
Carbon gave us a great CONTRIBUTION.md file to be used on carbon-addons-dotcom. I doubt we can just copy and paste across the board, but it can serve as a good starting point and outline.

Already created.  Closing


Update spacing for responsiveness in events widget

Issue #1184 :
https://github.com/bcgov/met-public/issues/1184
-Make the event widget more responsiveness at different sizes

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #1298 (d10eba4) into main (44fd9c0) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1298      +/-   ##
==========================================
+ Coverage   72.68%   72.71%   +0.02%
==========================================
  Files         303      303
  Lines        8505     8505
  Branches      599      599
==========================================
+ Hits         6182     6184       +2
+ Misses       2230     2228       -2
  Partials       93       93

Flag
Coverage Δ

metweb
66.65% <ø> (ø)

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

Impacted Files
Coverage Δ

...s/engagement/view/widgets/Events/InPersonEvent.tsx
75.00% <ø> (ø)

.../engagement/view/widgets/Events/VirtualSession.tsx
75.00% <ø> (ø)

...-web/src/components/engagement/view/EmailPanel.tsx
15.15% <0.00%> (ø)

met-api/src/met_api/schemas/engagement.py
92.45% <0.00%> (+3.77%)
:arrow_up:


Use rule collection for BaseShardingEngine to instead of sharding rule only

For #3691.

Pull Request Test Coverage Report for Build 10612

24 of 48 (50.0%)  changed or added relevant lines in 13 files are covered.
18 unchanged lines in 3 files lost coverage.
Overall coverage decreased (-0.1%) to 57.785%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/communication/jdbc/wrapper/PreparedStatementExecutorWrapper.java
0
3
0.0%

sharding-proxy/sharding-proxy-backend/src/main/java/org/apache/shardingsphere/shardingproxy/backend/communication/jdbc/wrapper/StatementExecutorWrapper.java
0
3
0.0%

sharding-core/sharding-core-common/src/main/java/org/apache/shardingsphere/core/rule/ShardingRule.java
0
6
0.0%

sharding-core/sharding-core-entry/src/main/java/org/apache/shardingsphere/core/shard/BaseShardingEngine.java
15
21
71.43%

shardingsphere-underlying/shardingsphere-rewrite/shardingsphere-rewrite-engine/src/main/java/org/apache/shardingsphere/underlying/rewrite/SQLRewriteEntry.java
0
6
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-orchestration/sharding-orchestration-core/sharding-orchestration-core-registrycenter/src/main/java/org/apache/shardingsphere/orchestration/core/registrycenter/util/IpUtils.java
3
76.0%

sharding-scaling/sharding-scaling-core/src/main/java/org/apache/shardingsphere/shardingscaling/core/synctask/history/HistoryDataSyncTask.java
7
76.47%

sharding-scaling/sharding-scaling-core/src/main/java/org/apache/shardingsphere/shardingscaling/core/execute/executor/channel/MemoryChannel.java
8
51.85%

Totals

Change from base Build 1178:
-0.1%

Covered Lines:
11854

Relevant Lines:
20514

💛  - Coveralls


[Android] Getting package io.sentry does not exist error when building for debug

I followed this instructions https://docs.sentry.io/clients/react-native/
The errors:

:app:compileDebugJavaWithJavac
D:\Atom\react-native-ui-kitten\examples\ExplorerApp\android\app\src\main\java\com\explorerapp\MainApplication.java:7: error: package io.sentry does not exist
import io.sentry.RNSentryPackage;
^
D:\Atom\react-native-ui-kitten\examples\ExplorerApp\android\app\src\main\java\com\explorerapp\MainApplication.java:31: error: cannot find symbol
new RNSentryPackage(MainApplication.this),
^
symbol: class RNSentryPackage
2 errors
:app:compileDebugJavaWithJavac FAILED
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':app:compileDebugJavaWithJavac'.
Compilation failed; see the compiler error output for details.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

react-native link was successful. apply from: "../../node_modules/react-native-sentry/sentry.gradle" can be found in app\build.gradle.
ExplorerApp\node_modules\react-native-sentry\ is there
(I'm trying to test install sentry into the example app of react-native-ui-kitten)

the link command didn't adjust my app/build.gradle. I manually add compile project(':react-native-sentry') and it builds success

I am closing all old issues, please if this is still a problem feel free to revive it.
Also, consider moving to our new SDK @sentry/react-native.


Terminal backend question

Do you remember why you chose termion backend for Linux instead of crossterm? I know this was a while back so it might not be that relevant today, but just curious is all.

I tried both executables but didn't see a noticeable difference between crossterm on windows and termion on WSL when comparing skipped pixels.
Also wasn't able to get it to work completely when compiling btw, you didn't commit the lock file which you should here since it's a binary crate. When changing cursive_buffered_backend version or fixing the errors it will run but some things don't show up in the drawing screen. Not sure why since there weren't any errors.
Lastly there's a typo on the opening screen langauges -> languages (I didn't want to make a pr just for this when I can't even get it to work right)

I see, forgive my rookie mistakes there! I will have a look and correct this in the coming days.

Done! You were absolutely right that crossterm and termion aren't different, so I changed to crossterm across the board. Thanks again!


angular6-multiselect-dropdown dynamiccally bind

This is working
this.selectedItems = [
{"id":2,"itemName":"Singapore"},
{"id":3,"itemName":"Australia"}
];
How can I add dynamically in loop?
Like
for (let a of this.serviceTypeSelectedItems) {
this.selectedItems = [
{ "serviceTypeId": a.serviceTypeId, "serviceTypeName": a.serviceTypeName }
]
}
It selecting last one, but can I selectable all in loop
Can you help here how can I select dynamically?

this.selectedItems = [];

for (let a of this.serviceTypeSelectedItems) {
    this.selectedItems.push(
        { "id": a.serviceTypeId, "itemName": a.serviceTypeName }
    )
}

It seems you are not appending the values into list

No its not working like that.
Its resolved with this
var list = [];
for (let a of this.getSelectedItemServiceTypes) {
this.selectedItems = [
{ "id": a.serviceTypeId, "itemName": a.serviceTypeName }
]
//every time its come only one item in foreach
this.selectedItems.forEach((item: any) => {
list.push(item);
});
}
Array.prototype.push.apply(this.selectedItems, list);


Regent: inlining task gives incorrect result

The following
import "regent"

__demand(__inline)
task foo()

	var sum_delx_sqr : double = 0
    	var sum_dely_sqr : double = 0
    	var sum_delx_dely : double = 0

	var nx = 1.0
	var ny = 1.0
	var tx = 1.0
	var ty = 1.0

	var delx = 1.0
	var dely = 1.0

	var dels = delx*tx + dely*ty
	var deln = delx*nx + dely*ny
	var dels_weights = dels
	var deln_weights = deln

	sum_delx_sqr = sum_delx_sqr + dels*dels_weights
	sum_dely_sqr = sum_dely_sqr + deln*deln_weights

	regentlib.c.printf("Sums %0.13lf %0.13lf\n", sum_delx_sqr, sum_dely_sqr)
end

task main()
	foo()
end
regentlib.start(main)

outputs
Sums 0.0000000000000 0.0000000000000

On commenting out the inline request at the top, we get
Sums 4.0000000000000 4.0000000000000

as output. Surprisingly, changing the first 3 declarations to use 0.0 instead of 0 (for example var sum_delx_sqr : double = 0.0) gives the correct result even with the inline request.

This should be fixed by 522329e5. @rupanshusoi Can you pull and try again?

Yes, looks like it's fixed. Just curious, do you have any idea why changing 0 in the initialisations to 0.0 was masking the error?

There were two thing that were contributing to the issue. 1) the task inliner was "forgetting" the type of variables sum_delx_sqr and sum_dely_sqr, which later pick up the type of their literals, int32. 2) unlike in C, printf in Regent (more precisely in Terra) is more strict about argument types. your code with the inliner bug was putting a 32-bit integer variable in a place where the printf was expecting a 64-bit float, which made it read off from a wrong address and end up printing 0.

@rupanshusoi Can you close the issue if you confirmed the fix?


Feature request: Option to show average cell voltage

Please add an equivalent of BetaFlight's average cell voltage
If you implemented #67 , then this would always be accurate-ish
ie, if (Once #67 is implemented) I have set the cell count set to 4S, and the current voltage of the pack is 12v, I want to see 3v in the OSD for the voltage

vote for this!
+Add Voltage in overlay menü of all inputs (HDZero, Analog). So that its possible to react early on a race and change battery during heats, not only if warning comes up. (in my opinion voltage is more important info than fan speed of top fan)

i think both this and #67 are absolutely necessary features and should be implemented at the next juncture.
There is some "plumbing" work i would like to take care off first however.

vote for this! +Add Voltage in overlay menü of all inputs (HDZero, Analog). So that its possible to react early on a race and change battery during heats, not only if warning comes up. (in my opinion voltage is more important info than fan speed of top fan)

I agree

Just to be clear, if we are talking about displaying the "goggle" battery voltage, either total or cell - I dont mind, I think it is essential, and would very much welcome it.

Is this supposed to show the voltage in the flight OSD, or in the menu.
I cant see it in the flight OSD.  Should I raise another issue for this ?

PR supplies the battery logic, but does not modify where it is displayed, as the orginial feature request stated.
So if you would like to see it be displayed somewhere else please raise a new request for that.

PR supplies the battery logic, but does not modify where it is displayed, as the orginial feature request stated. So if you would like to see it be displayed somewhere else please raise a new request for that.

Thanks


Header hygiene: scope http_parser under httpxx

Also dos2unix in one file.

Will fix travis.

ping.
BTW - our http server implementation is done. The httpxx part was smooth - Thanks for this lib.

Thanks for contributing this :-). Glad it was useful!


added support to detect content-type in lakectl

Closes #6584
Change Description
Background
Used mime pkg from golang sdk.
Bug Fix
If this PR is a bug fix, please let us know about:

Problem - lakectl fs upload pdf file can not be previewed
Root cause - lakectl local doesn't support upload with content type
Solution - bug was fixed by using mime pkg to detect content type based on filepath

Testing Details
How were the changes tested?
Using cmd line.
Contact Details
<EMAIL_ADDRESS>

Thanks @0x-chaitu for this awesome contribution!
I think there's some confusion though. #6584 was opened for lakectl local and this PR fixes lakectl fs upload. They both provide functionality to upload objects, but the first doesn't have any support for content-types. lakectl local upload objects here.
I also agree with @arielshaqed about the automation - lakectl local has the ability to upload multiple files in a single command. I can't think of a good way to pass content-types for many at once, so your suggestion to add auto-detection is on spot. However, to avoid mistakes, I think it should not identify types by default and allow advanced users to enable it with some flag, like --include-content-types.


Fixes #1693 (Charmap wide runes) by upgrading to Nstack.core 0.17

Fixes #1693 (Charmap wide runes) by upgrading to Nstack.core 0.17.
(Draft until @migueldeicaza has a chance to release a new NStack.Core (assuming v 0.17) to Nuget).

It isn't on Nuget yet.


Slider component

Moves the original POC into this PR, with quite many changes based on comments in the issue #275, including:

reducing the number of components. While I also tried putting the handle in the slider, this created some further technical complexity, so I left that one as a component.
replaced the mapping functions for tick and tooltip content with slots
created a number of scss vars - the actual values still need work
moved the scss into its own files per the repo's style

I need to spend some more time testing various prop combinations and checking jsdoc types which have been refactored enough times there are probably some errors, so this is not quite ready to merge. Docs are missing too obviously.

The build is failing due to our mistake, you may fix that Stylelint error or wait until #289 is merged and then merge main into your branch.
Thanks for such an extensive contribution! The docs are the most convenient place to preview components, so I might drop in your branch and throw together a bare docs page just to see how the component looks in reality, so far I've just looked over the code. After that I'll get back with more reasonable comments :)

I added some docs so you could see it, although not yet complete (didn't put e.g. scss vars yet). Also was a bit mystified how to put a string union in the props table since it uses the "|".

Could you also add the Slider to attractions/index.js? The docs are failing to build because of this.
As for the | in the docs, yeah, it's a tricky one. See here for a workaround

For clarity would it possible to create a task list in one place of outstanding, blocking issues? I have quite many changes but the some of the comments, I am not sure if must react or not -- or even if I am prepared.


Update Fate_Core sheet based on user feedback.

My previous changes (#4810) didn't go over that well with some players, specifically those who used the features I chose to hide.
This revision of the sheet attempts to address the concerns raised in that thread. It includes:

Formatting improvements/fixes and better browser compatibility.
A "high-level skills" checkbox that shows the high-level skills, rather than having to manually edit attributes.
Roll modifiers, with an option to turn them off.
Better auto-calculation of stress boxes.
Stress tracks are now repeating, and you don't have to manually edit anything because they're always visible. I don't attempt to auto-calculate the stress boxes, because I realized if someone is using a custom track, they may also be using custom rules for how many stress boxes they need.
Found some more lost data (OpenStress1, OpenStress2, and OpenStress3) and wrote a script to copy it over.

Possible future changes:

Leave the "roll modifiers" box unchecked by default.
Add some way to show/hide boxes in the repeating stress tracks, either by picking a number from a dropdown or by typing a skill into a separate textbox.
Add some way to control the default height of stress and extras textboxes, so you don't have to manually resize them every time.

If you feel some less used features are in the way of your gameplay, you could consider making some parts of the sheet optional with a checkbox or something.
I've personally strived to only increased functionality on the sheets I've been working on, and create options for things that are more up to personal taste than strict functionality.

That's what I ended up doing. Roll modifiers are back, but now there's a checkbox for them.

And now there are even more options, cleanly separated into an options menu.

You can hide the advantages and extras sections if you aren't using them. This frees up room for aspects and stunts, respectively.
In the old version of the sheet, the stunts and extras sections had one big textbox rather than repeating small boxes. Now there's now an option to go back to that.
Instead of a single checkbox for the top three rows of the pyramid, there's a dropdown that lets you pick the exact number of rows to display.
Since you almost never need to add or remove aspects, stress tracks, or consequences, I moved that functionality into the options menu, freeing up space in the main sheet.
You can change the governing skill for the physical and mental stress tracks, because one forum user pointed out that these skills vary depending on which flavor of Fate you're playing.
I made the options as intuitive as I could, but I also included title text for all of them, just in case.

Barring bug fixes and rewording, I think that's everything.

I split the options menu into two sections to (hopefully) help people find what they're looking for. I mean, eleven options shouldn't be totally overwhelming, but it isn't dead simple either. I'm not 100% happy with the section names, but I can't think of anything better.
I removed the parts of the text that describe how Fate Core "usually" does things. It just felt too prescriptive. (This only applies to text in the options menu. The skill section still contains Fate Core's list of skills.)
You now have full control over custom skill tracks, meaning you can set governing skills for them and the sheet will track the number of boxes.
Instead of a governing skill, you can type in a number from 2 to 4 and it'll display that many boxes. (Applies to normal and custom skill tracks.)


KIKIMR-21623: Acceptance tests for Split/Merge CS

TBD

Preliminary plan:

Canonize clickbench

Simple scenario at clickbench db:
select
split
select
merge
select
Check that tablets count matched
Check that no blob is shared by the end of test

Stressed scenario
We have 3 kinds of  operations:

Clickbench canonical queries
Upsert some data
Split-merge with checking tablets count
Test runs these concurrently in different combinations.


Add jib-core changelog

Fixes #1161

I guess we should be accustomed to update three CHANGELOG files in many cases. Hope we don't change the public-facing API much.


[CYPRESS] Add e2e test for Klarna (pay later) payment method for Adyen Connector

The Hyperswitch SDK is a versatile tool built to streamline payment integration for developers. It simplifies the process of adding multiple payment methods, allowing you to quickly implement support for various gateways without the need to manually integrate each one. With Hyperswitch, you can efficiently manage payments across platforms, reducing complexity and boosting development speed.
Pre-requisites:
Please follow the README for guidelines on file structure, formatting and instructions on how to set up and run Cypress.
Description:
This task involves adding a cypress e2e test for Klarna (pay later) payment method for Adyen Connector.
🔨 Possible Implementation
Locate the element for this payment method, perform the click action on it and assert whether redirection happens success.
Contribution Guidelines:

Fork the repository and create a new branch for your work.
Ensure the WebSDK follows best practices for API integration and field rendering.
Write clean, well-documented code with clear commit messages.
Make sure to follow our coding standards and contribution guidelines.

Helpful Resources:

Link to WebSDK documentation: WebSDK Docs
Link to the API endpoint specifications: API Docs

Submission Process:

Ask the maintainers for assignment of the issue, you can request for assignment by commenting on the issue itself.
Once assigned, submit a pull request (PR).
Maintainers will review and provide feedback, if any.
Maintainers can unassign issues due to inactivity, read more here.
For this issue, please submit a PR on @juspay/hyperswitch-web repo, and link it to the issue.

Refer here for Terms and conditions for the contest.
If you have any questions or need help getting started, feel free to ask in the comments!
🔖 Note:
Please drop a mail for required credentials at<EMAIL_ADDRESS>

Hey @sccalabr there is another issue assigned to you as well, Kindly let us know, which of these issue you are working on currently.
Thanks!

Cool @sccalabr, thanks for your interest.

please assign this issue to me.

Sure @Rishi-0007 Assigning the issue to you, thanks for your interest!

Hey @gorakhnathy7,
I’ve successfully set up the project locally, and everything seems to be working smoothly. Here are the details:

The app at http://localhost:9060/ runs perfectly, showing a T-shirt and cap added to the cart, but it currently only displays the Card Payment option.
The Cypress test specs are showing up at http://localhost:9050/__/#/specs.

Could you please guide me on how to proceed with solving the issue? Specifically, I need help adding the Klarna (Pay Later) payment method via the Adyen connector and writing the corresponding Cypress E2E test.
Thanks in advance for your help!

@Rishi-0007 Please use the appropriate credentials i.e. publishableKey and secretKey. Also make sure to use the correct profile_id in createPaymentBody as shared to you over mail.

Hi, if noone is working on this can I try it?


New Sample based on Radio Button from UI5 Demo Kit

Hi Team,
I’m excited to share that I’ve submitted PR #239 featuring a new demo for the Radio Button control. This example builds on the Radio Button sample from the UI5 Demo Kit.
The new sample can be found here:

Here’s a preview:

I look forward to your feedback and suggestions for improvements!
Best regards,
Francisco Milan

looks greak francisco !!

Nice! Thank you 🙏 PR merged.


Allow to remove event

Have you consider adding off (detaching events) method to iScroll5? It could be looks something like this:
  off: function (type, fn) {
    if ( !this._events[type] ) {
      return;
    }

    var index = this._events[type].indexOf(fn);

    if (index > -1) {
      this._events[type].splice(index, 1);
    }
  }

I could make pull request with this, but currently I'm to lazy to fight with build system and Contributor License Agreements ;)

It doesn't look like probeType option can be updated once the IScroll instance has been created. If I use off to stop listening to scroll events, I'd like to be able to switch to a less demanding probeType (from 3 to 1) as well. What do you think?


window raise / menubar focus at conflict with each other

🐛 Bug
So turns out #721 which "raised" the main window as actually a reversion of #553 which had deleted that line to fix a menubar focus issue.
In #553 I made the following statement

I'm not quite sure why the raise_ call was in there or if there are any negative issues of dropping it on other platforms, but I can confirm that on a mac everything still functions as expected, including usage from the notebook.

Turns out raise_ was needed to bring the window to the front which is quite nice.
Now I'm not sure how to resolve both #553 and #721 at the same time!!!
@tlambert03 do you have any ideas?

oops, sorry!  (i'm assuming you meant #553?).  Yeah I know the issue you're referring to... and I've long since "grown used to it" with Qt (i've had the same thing for ages with LLSpy, and now I almost instinctively cmd-tab back and forth after starting).  I can say that it depends a bit on how the program is started... (but I can't remember at the moment exactly how/why).  For instance, it didn't affect the pyinstaller app, and I even think when I started from a cli interface it was sometimes better?  For what it's worth, I've even noticed this sort of window/menu stealing at times when importing matplotlib with qt backend... so I think it's rather deep in Qt.
It's a real conundrum, and definitely something I'd love to get to the bottom of!
For now, I wouldn't mind if you wanted to revert #721 and leave this issue open to remember to think about it.  your call.

We don't need to revert right away, i might play around with it a bit more now that I know why raise_ is called, but we might want to change before the next release.
Personally I find it less confusing if I can't see the window from the notebook than if the drop downs don't work first time, but hopefully we don't have to make that trade off!!

I just spent some timing working on this, but no progress!! I can only get either / or to work not both!!

Tried again, still no luck - if any Qt / Mac experts out there have any ideas on how to resolve this it would be much appreciated, I'm still making now progress

gonna try to see what I can figure out here...  One quick observation that might be a clue:
with self._qt_window.raise_() in the Window.show() method (as it currently is in master), I only get the inactive menubar problem if I launch the viewer within the gui_qt() context manager.  If I launch it as __main__ using python -m napari, then the menubar is immediately active.
If I launch ipython with a Qt event loop (ipython --gui=qt) then calling napari.view_image also yields a viewer with an active menubar.  However, when launching ipython like that, my terminal actually loses focus and the "current app" in the osx menubar is "python", forcing me to click on the terminal window to actually continue with launching the viewer.  I think it's that momentary loss of focus that "takes care" of the problem.
so it seems to be only when starting the qt event loop and then immediately raising the main_window when the menubar is inactive?  @sofroniewn, can you confirm that you're using with napari.gui_qt() to start your viewer when you have the inactive menus problem?

Yes that is the case, which means a potential solution might just be to set up some logic / extra keyword params to avoid the ‘raise_’ call when launching with our ‘gui_qt’ event loop and have it called by default from IPython and Jupyter - where it is really needed

yeah, I'm working on something like that now.  I noticed that the raise_ call is only necessary on the second call to gui_qt() (otherwise the viewer is already on top), and during that second call to gui_qt(), Qapplication.instance() returns a valid instance, whereas in the first call it returns None.  So I'm going to see if I can leverage that somehow


Code sharing with VS Code Jupyter Notebook implementation

We are getting closer and closer to finishing the native Jupyter Notebook support in VS Code for Julia (see here). When I started, I copied a few functions from IJulia.jl over, but as things progressed I ended up copying a fair bit of code and now there is probably too much code duplication.
Should we try to arrange this is a way that instead of the VS Code extension copying code we move the shared code into its own package, say JupyterCore.jl, that both IJulia.jl and the Julia extension for VS Code can use? Probably mostly a question for @stevengj, whether he would be on board with that. This is not super urgent, I think we could start out by shipping the extension with all the code duplication and then also clean this up down the road.
For background, the notebook implementation in VS Code will not use the kernelspec configuration that IJulia.jl installs nor the jupyter protocol. Instead, it will be a "no configuration" solution, i.e. if a user has the Jupyter and Julia extension for VS Code installed, everything will work without any further things. The whole execution aspect of things is handled by the native notebook execution API that VS Code started shipping recently. In the VS Code Julia extension we can't take a full dependency on IJulia.jl because we in general cannot take dependencies on packages that pull in anything binary (so in this case MbedTLS.jl and maybe Conda.jl), that wouldn't work with the way we deploy Julia packages in the extension.

So their notebook API doesn't use ZMQ, but uses the same JSON message protocol over some other transport layer?
I'm fine with an IJuliaCore.jl package if that makes sense…

VS Code now has a notebook API that is an in-process JavaScript API, like all the other APIs that VS Code provides. VS Code extensions can just directly interface with that, and the pending PR on the Julia extension does that. At that level we (the Julia extension) essentially tell VS Code that we can execute Julia code in notebooks and provide a callback that does so.
Inside the Julia extension we handle kernel startup and the communication between the Julia extension (written in TypeScript) and the Julia kernel. We are using a custom, internal JSON-RPC based protocoll for that that is based on the existing implementation we use for communication between the Julia REPL in VS Code and the Julia extension. One benefit of that design is that we can re-use all the features we've created for code execution in the REPL: the tree variable explorer, progress reporting, debugger etc. Another nice feature of this is that we can ship everything we need for this to work as part of the extension itself, and there is also no config state anywhere, so hopefully this will be a very robust solution that just always works.
MS is shipping a Jupyter extension for VS Code. That extension knows how to read and write Jupyter notebook files. It also knows how to interface with traditional Jupyter kernels via ZMQ via the normal kernel spec registration stuff. The design now is that the Jupyter extension will use traditional Jupyter ZMQ kernels when there is no language extension installed that provides a richer, more native execution experience. But if say the Julia extension is installed, then execution of code will be handled directly by the Julia extension, and all the additional features that such an extension might have (variable explorer, debugger, in our case progress reporting etc) will light up.
The kind of code that I would move into IJuliaCore.jl is stuff like the MIME type list, the whole console capture code, some of the display bundle creation code etc. Essentially the plumbing where we would want the same behavior of IJulia and the VS Code extension. But anything protocoll specific, setup specific etc. would stay in IJulia.jl because we don't need that in the extension, plus that seems the place where the binary dependencies come into play.
One big question at this point is whether a Julia kernel in VS Code should export an IJulia module... And if it does, we would probably have to try to make it API-compatible with what IJulia does. I'm not sure right now what the best thing to do there is, I think I need to take a more careful look next at what kind of things packages or notebooks do with the IJulia module.
I'll start with an IJuliaCore.jl package for now.


what about giving an element less than 1rem size?

Hi!
Thankfully we have the Spacing system so we can use margins and paddings.
But want if we want to give an element less than 1rem (e.g, 0.5rem) ?
giving pv0.5 doesn't work.

I think you'll like #22 :-)
Basically v2.1.0 adds the ability to do this:
@include new-spacing-helper('0-5', 0.5);

which will output responsive spacing helpers like this:
.pt0-5 {
     padding-top: 0.5rem !important;
}


Error 500 Internal Server Error When Sending a New Update

Description:
I would like to express my gratitude for your contributions to this project.
I've successfully tested the development Docker environment, and it functioned perfectly. However, upon deploying the production Docker on our server, I encountered an error. The web interface operates as expected; however, when attempting to send a new update to the server using the script your provided, I receive the following error message after curl command:

<title>500 Internal Server Error</title>

500 Internal Server Error

I have shared this issue with our DevOps team, but thus far, they have been unable to identify the root cause. They mention that there are no relevant logs on the server side.
Request for Assistance:
I am seeking insights into the potential reasons behind this error message. Any assistance or guidance you can provide would be greatly appreciated as we work to resolve this issue. Thank you in advance for your help.

Hello,
sorry but I completely missed this issue!
Have you managed to resolve it?
Otherwise you can also drop me an email at "umberto" then the at and then "ghio.io"
Closing this issue for now

Hi.
Yeah I resolved this issue. I don't know what was the exact cause of that since I don't know Docker very well. But it is fixed.
Thanks for your support :)


Where is obsdet_msgs in the code?

https://drive.google.com/file/d/1ztLk9Slm656CV-WJieUpBJPlz-Iw14Bk/view?usp=share_link

I'm very sorry to bother you again! But the message type you provided does not match the code.
/home/common/ransac_l_fitting_shape/src/RANSAC-L-shape-fitting-3D-LiDAR-ROS-main/src/ransac_Lshape_fitting.cpp: In member function ‘void LShapeFilter::MainLoop(const CloudClusterArray&)’:
/home/common/ransac_l_fitting_shape/src/RANSAC-L-shape-fitting-3D-LiDAR-ROS-main/src/ransac_Lshape_fitting.cpp:600:37: error: ‘const struct obsdet_msgs::CloudCluster_<std::allocator >’ has no member named ‘orientation’
double theta_kitti = in_cluster.orientation;
^~~~~~~~~~~
/home/common/ransac_l_fitting_shape/src/RANSAC-L-shape-fitting-3D-LiDAR-ROS-main/src/ransac_Lshape_fitting.cpp:602:58: error: ‘const struct obsdet_msgs::CloudCluster_<std::allocator >’ has no member named ‘index’
eval_performance(theta_kitti, theta_star, in_cluster.index, in_cluster.index_seq);
^~~~~
/home/common/ransac_l_fitting_shape/src/RANSAC-L-shape-fitting-3D-LiDAR-ROS-main/src/ransac_Lshape_fitting.cpp:602:76: error: ‘const struct obsdet_msgs::CloudCluster_<std::allocator >’ has no member named ‘index_seq’
l_performance(theta_kitti, theta_star, in_cluster.index, in_cluster.index_seq);


Publish useLazyload as npm package

I really like your updated version of lozad!
Right now, importing it from the plugin folder seems a bit verbose:
import { useLazyload } from '../site/plugins/kirby-blurry-placeholder/src/useLazyload'

Would be great to just have an npm package. Or is there a shorter way to import it?

You may setup an alias in your config, but that would be just another work-around. Probably a good idea to push the library to npm. Will keep that in mind.

Hey @arnoson, it took me some time, but I have managed to finally rewrite the lazy loading hook in TypeScript: https://github.com/johannschopplich/loadeer
I will keep the old hook in the project for compatibility reasons. But you can now just use the 🦌 loadeer npm package!
Head over to the improved readme of the Kirby plugin to see a list of new features, like embedding the library without a build step.
 Thank you very much (:


Possible enhancements

Here's some ideas about possible enhancements to pinnwand:

[ ] Add Burn after reading, in case we want to keep the post longer but to be deleted right after the person read it. This could be a checkbox like the Use a longer URI one.
[ ] Add an option to upload files, such as images/videos, with the correspondent option to limit file size. Useful to show a screenrecord of a weird behaviour along with the actual config, etc.
[ ] Add an option like expiries_default = 3600 or expiries_default = ["expiries.1day"] so we can select what the default expiry is and still have a selectable range of expiries, pretty much like the preferred_lexers option.

Going through them.

Can't be guaranteed due to caching but might be nice-ish.
Other programs and websites exist for this, it's a difficult battle with illegal and copyrighted content which most public instances of pastebins already deal with on just text.
Agreed. Let's add that.


Port IndexPath to FoundationEssentials

Bring IndexPath to FoundationEssentials.

@swift-ci test

The DateRelativeFormatStyleTests.swift test failures are known


added change for 'fn_id' to propagate through to logs

LGTM after fixing @skinowski 's note, that was the only line i saw any api breakage. thanks


Re-route forbidden requests to Borderpatrol

Forbidden requests to Borderpatrol to be routed to forbidden page instead of login page.

[Current coverage][cc-pull] is 97.08%

Merging [#170][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.20%

@@             master       #170   diff @@
==========================================
  Files            36         36
  Lines           957        959     +2
  Methods         943        941     -2
  Messages          0          0
  Branches         14         18     +4
==========================================
  Hits            931        931
- Misses           26         28     +2
  Partials          0          0

Powered by Codecov. Last updated by [aa5565b...ec785dc][cc-compare]
[cc-base-branch]: https://codecov.io/gh/lookout/borderpatrol/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/lookout/borderpatrol/compare/aa5565ba33a040e9d33645104f1214e2cb310af2...ec785dcbf020a13eb500563a810a9bc5ff7b6319?src=pr
[cc-pull]: https://codecov.io/gh/lookout/borderpatrol/pull/170?src=pr


Because I could

Could you explain this a little bit more? I don't doubt its awesomeness, just wondering what it does & how it works.

If you could, it'd also be useful to me to have an explanation of dglsseus's code, as I didn't read it, and I'm just not quite sure what it does. I'd like, if possible, to stay on top of understanding all the code in this module.

@dgelessus code is a super slick use of set theory.  Sets are very useful in logic, math and computer science for implementing DRY to datasets so having sets built into Python is great.  However sets are too often underutilized because folks struggle with the concepts of unions, intersections, two kinds of differences, two kinds of subsets, and two kinds of supersets but it in not so difficult and is surprisingly powerful.
So @dgelessus code is creating three sets and is taking the Set Difference between the second and third and then taking the Union of that result and the first.
print(sorted( {'a', 'b', 'c'} | {'c', 'd', 'e'}))                # --> ['a', 'b', 'c', 'd', 'e']
print(sorted( {'a', 'b', 'c'} | {'c', 'd', 'e'}  - {'b', 'd'}))  # --> ['a', 'b', 'c', 'e']
print(sorted(({'a', 'b', 'c'} | {'c', 'd', 'e'}) - {'b', 'd'}))  # --> ['a', 'c', 'e']  Note: `-` has higher operator precedence than `|`.

My code just uses the parlor trick of combining multiline strings with str.split() to squeeze @dgelessus code with a somewhat startling loss of clarity which is why I called this pull request "Because I can" and I will not be at all surprised or offended if you reject it ;-)

Great, thanks. I get it now. I've only really used sets for subtraction
before, I had no idea they were so powerful.
On Sat, Feb 13, 2016 at 2:15 AM cclauss<EMAIL_ADDRESS>wrote:

@dgelessus https://github.com/dgelessus code is a super slick use of
set theory. Sets are very useful in logic, math and computer science for
implementing DRY https://en.m.wikipedia.org/wiki/Don't_repeat_yourself
to datasets so having sets built into Python is great. However sets are too
often underutilized because folks struggle with the concepts of unions,
intersections, two kinds of differences, two kinds of subsets, and two
kinds of supersets but it in not so difficult
https://github.com/cclauss/Ten-lines-or-less/blob/master/set_theory.py
and is surprisingly powerful.
So @dgelessus https://github.com/dgelessus code is creating three sets
and is taking the Set Difference between the second and third and then
taking the Union of that result and the first.
print(sorted( {'a', 'b', 'c'} | {'c', 'd', 'e'}))                # --> ['a', 'b', 'c', 'd', 'e']print(sorted( {'a', 'b', 'c'} | {'c', 'd', 'e'}  - {'b', 'd'}))  # --> ['a', 'b', 'c', 'e']print(sorted(({'a', 'b', 'c'} | {'c', 'd', 'e'}) - {'b', 'd'}))  # --> ['a', 'c', 'e']  Note: - has higher operator precedence than |.
My code just uses the parlor trick of combining multiline strings with
str.split() to squeeze @dgelessus https://github.com/dgelessus code
with a somewhat startling loss of clarity which is why I called this pull
request "Because I can" and I will not be at all surprised or offended if
you reject it ;-)
—
Reply to this email directly or view it on GitHub
https://github.com/The-Penultimate-Defenestrator/Pythonista-Tweaks/pull/9#issuecomment-183613944
.

@cclauss To be honest I had no idea which operator (- or |) takes precedence ;) It doesn't matter in this case because the first and third set have no items in common. You can subtract set 3 from set 2 and union that with set 1, or you can do the union first and the subtraction second, in the end the result is the same.
@The-Penultimate-Defenestrator If you have any questions, just ask :) That PR changed a bunch of things, if there's anything you don't understand or disagree with, please say so. (The messed up diff because of the file moves didn't really help.)

Great. Thanks for the clearing-up. I'll take some time to read the code,
and I'll let you know if I have any more questions.
On Sat, Feb 13, 2016 at 12:15 PM dgelessus<EMAIL_ADDRESS>wrote:

@cclauss https://github.com/cclauss To be honest I had no idea which
operator (- or |) takes precedence ;) It doesn't matter in this case
because the first and third set have no items in common. You can subtract
set 3 from set 2 and union that with set 1, or you can do the union first
and the subtraction second, in the end the result is the same.
@The-Penultimate-Defenestrator
https://github.com/The-Penultimate-Defenestrator If you have any
questions, just ask :) That PR changed a bunch of things, if there's
anything you don't understand or disagree with, please say so. (The messed
up diff because of the file moves didn't really help.)
—
Reply to this email directly or view it on GitHub
https://github.com/The-Penultimate-Defenestrator/Pythonista-Tweaks/pull/9#issuecomment-183705041
.


Fix FBX texture path resolving

Fix resolving texture paths using the FBX importer.
Previously, the importer would call localize_path() on relative paths, which resulted in relative_texture.png turning into res://relative_texture.png. This was then concatenated to another res:// path resulting in something like res://models/res://relative_texture.png.
Also it turns out that the fallback texture lookup logic was used unconditionally, and the importer did not attempt to check if the original texture path exists. This PR fixes that issue as well.

Thanks!


ng update --next should deduplicate angular packages and just show core

Versions
Angular CLI: 6.0.0-rc.2
Node: 9.2.0
OS: linux x64
Angular: 6.0.0-rc.3
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router

@angular/cli: 6.0.0-rc.2
@angular-devkit/architect: 0.5.4
@ngtools/json-schema: 1.2.0
@schematics/update: 0.5.4
typescript: 2.7.2
webpack: error

Repro steps
ng update --next
Observed behavior
ng update --next
    We analyzed your package.json, there are some packages to update:

      Name                               Version                    Command to update
     ---------------------------------------------------------------------------------------------------------
      @angular/animations                5.2.9 -> 6.0.0-rc.3        ng update @angular/animations
      @angular/common                    5.2.9 -> 6.0.0-rc.3        ng update @angular/common
      @angular/compiler                  5.2.9 -> 6.0.0-rc.3        ng update @angular/compiler
      @angular/compiler-cli              5.2.9 -> 6.0.0-rc.3        ng update @angular/compiler-cli
      @angular/core                      5.2.9 -> 6.0.0-rc.3        ng update @angular/core
      @angular/forms                     5.2.9 -> 6.0.0-rc.3        ng update @angular/forms
      @angular/http                      5.2.9 -> 6.0.0-rc.3        ng update @angular/http
      @angular/language-service          5.2.9 -> 6.0.0-rc.3        ng update @angular/language-service
      @angular/platform-browser          5.2.9 -> 6.0.0-rc.3        ng update @angular/platform-browser
      @angular/platform-browser-dynamic  5.2.9 -> 6.0.0-rc.3        ng update @angular/platform-browser-dynamic
      @angular/router                    5.2.9 -> 6.0.0-rc.3        ng update @angular/router
      typescript                         2.5.3 -> 2.9.0-dev.20180409  npm install typescript

    There might be additional packages that are outdated.
    Or run ng update --all to try to update all at the same time.

Nothing to be done.

Desired behavior
ng update should tell me what to do next, and take into account its own context.
if ng update @angular/core updates all of these synced packages, then all should be grouped. We should be treating them all like one thing unless there's an exception and one is out of date. We want to help people think about them as a single unit.

We need to show only the first package name of the group.


Fixed bug with null arrays when EmitBehaviors.WhenModified is used.

Closes #337

:white_check_mark: Build ExtendedXmlSerializer <IP_ADDRESS>-bpypvmae completed (commit https://github.com/ExtendedXmlSerializer/home/commit/fc8ee92902 by @Mike-EEE)
Install it by running the following command in Package Manager Console:
Install-Package ExtendedXmlSerializer -Version <IP_ADDRESS>-bpypvmae -Source https://ci.appveyor.com/nuget/extendedxmlserializer-preview

:white_check_mark: Build ExtendedXmlSerializer <IP_ADDRESS>-upwohjoe completed (commit https://github.com/ExtendedXmlSerializer/home/commit/381fe47088 by @Mike-EEE)
Install it by running the following command in Package Manager Console:
Install-Package ExtendedXmlSerializer -Version <IP_ADDRESS>-upwohjoe -Source https://ci.appveyor.com/nuget/extendedxmlserializer-preview


fix(link-preview): ensure spinner is not shown indefinetely when no preview results are available

Jira: TEAMS-12649
Related Core PR: https://github.com/StackEng/StackOverflow/pull/19095
Release Candidate<EMAIL_ADDRESS>

Will hold off from merging this PR until this combo is resolved.


Cancel a scheduled timer?

See mbostock/d3#2189.
Probably timer(…) should return a handle so that a timer can be cancelled, like setTimeout & clearTimeout.

Having a timer handle might also present a cleaner API to timerReplace, rather than having it only work on the current timer.

Related mbostock/d3#2468, in that starting and stopping the force layout’s timer is easier when you can cancel the timer externally.

If timer(…) returns a handle and that handle can be used to stop a timer, should we still allow the callback to return true for canceling a timer?


TestClassNameRule

Trying to implement #608

Let's wait until somebody comments on https://www.drupal.org/project/drupal/issues/3268489#comment-15252653 about the  abstract-ness of *TestBase classes.

Verdict is in:

The base classes should always be abstract yes, not sure whether it's documented but it's what should happen.
catch in https://www.drupal.org/project/drupal/issues/3268489#comment-15252749

For me the rule works as expected and is ready for review.

@Boegie I'll defer to your judgement if the baseline failures look OK. When you say it's good, I'll merge

Ohh, the pressure... ;)
Thanks @mglaman, I have something cooking locally that has three rules for test classes, one for the class name (this one), one for the namespace and one for the file location.
Since they both use a lot of the same code it makes sense to me to create a base class for this, thus trying to create one PR to rule them all.
Shouldn't be long...
#FamousLastWords


Contributing link not working

When I click on the "Contributing" link I am send to a page with the link https://www.symbiote.com.au/contributing-open-source but that link does not work anymore. Therefor it is unclear how to contribute to this project.

Hi @StevenLantinga. Can you please give that link another try? The past couple of months we have been transitioning to a new website, so unfortunately some of the links may have disappeared while that was happening, but I can confirm it's working correctly for me at the moment.

@nglasl Thanks for your reply. The link is working again!

If you have any other questions about contributing, please let me know :smile:


How to make iOS 13 modals with RNRF?

I didn't find anything about that,
Is there a way to make an iOS 13 modal with RNRF?
Like this:

Thanks!

any solutions to have these modals?

 I'm trying react-native 0.6.3 but doesn't work


tests: mutex: add some error case testing

Add some error case testing such as invoking mutex with null parameter or using it interrupt context. This is for checking if API robust enough in error condition handling.
There are 6 error cases adding in this PR:

test_mutex_lock_in_isr : Invoke k_mutex_lock in interrupt context, it is expected to trigger assert fail.
test_mutex_unlock_in_isr : Invoke k_mutex_unlock in interrupt context, it is expected to trigger assert fail.
test_mutex_init_null : Invoke k_mutex_init with null parameter, it is expected to cause fatal error.
test_mutex_lock_null : Invoke k_mutex_lock with null parameter, it is expected to cause fatal error.
test_mutex_unlock_null : Invoke k_mutex_lock with null parameter, it is expected to cause fatal error.
test_mutex_unlock_count_unmet : Unlock a mutex which count is zero, it is expected to trigger assert fail.

Signed-off-by: Enjia Mai<EMAIL_ADDRESS>

@LeiW000  I have addressed all the comments your mentioned, could you please help to review it again, thanks!

@mengxianglinx , please review it.


Remove Rule.lint(Path) overload function from :detekt-test

This function was used in the begining of detekt to tests the rules. But we find out that have the code that we are testing on another file wasn't good so we moved all those test to the new Rule.lint(String) (and friends). Now we can remove this. If someone out there is using this it's easy for them to replicate or a good excuse to stop using that bad pattern.

Warnings

:warning:
It looks like this PR contains functional changes without a corresponding test.

  Generated by :no_entry_sign: dangerJS against 18ab5ebf3f30df162d58c742d95b2cc87c3fb96a


[Snyk] Upgrade zod from 3.22.1 to 3.22.2

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade zod from 3.22.1 to 3.22.2.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released 22 days ago, on 2023-08-19.

Release notes

Package name: zod

3.22.2 - 2023-08-19Commits:

13d9e6b Fix lint
0d49f10 docs: add typeschema to ecosystem (#2626)
8e4af7b X to Zod: add app.quicktype.io (#2668)
792b3ef Fix superrefine types

3.22.1 - 2023-08-15Commits:
Fix handing of this in ZodFunction schemas. The parse logic for function schemas now requires the Reflect API.
const methodObject = z.object({
  property: z.number(),
  method: z.function().args(z.string()).returns(z.number()),
});
const methodInstance = {
  property: 3,
  method: function (s: string) {
    return s.length + this.property;
  },
};
const parsed = methodObject.parse(methodInstance);
parsed.method("length=8"); // => 11 (8 length + 3 property)

932cc47 Initial prototype fix for issue #2651 (#2652)
0a055e7 3.22.1

    from zod GitHub release notes

Commit messages

Package name: zod

792b3ef Fix superrefine types
8e4af7b X to Zod: add app.quicktype.io (#2668)
0d49f10 docs: add typeschema to ecosystem (#2626)
13d9e6b Fix lint

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Codecov Report
Patch and project coverage have no change.

Comparison is base (3d1a6c4) 81.66% compared to head (f288cdf) 81.66%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##           master      #65   +/-   ##
=======================================
  Coverage   81.66%   81.66%
=======================================
  Files           4        4
  Lines          60       60
  Branches       11       11
=======================================
  Hits           49       49
  Misses         11       11

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


Unexpected behavior: env_file doesn't (seem to) populate the environmental variables for the compose environment

Describe the bug
I created a test that passes in a test-specific .env file, and the variables are not reflected in the docker compose environment.
Is it because I misunderstand what it's supposed to do?
To Reproduce
A gist has all the files for an example.
# In tests/testing.env
EXAMPLE='hello'

from pathlib import Path

from testcontainers.compose import DockerCompose

def test__compose_environment__has_env_vars_from_file():
    compose_folder = Path(__file__).parents[1]
    compose = DockerCompose(
        compose_folder,
        env_file='tests/testing.env',
        build=True,
    )
    with compose:
        result = compose.exec_in_container(['env'])
        assert 'EXAMPLE' in result

============================================================== short test summary info ==============================================================
FAILED tests/test_env.py::test__compose_environment__has_env_vars_from_file - AssertionError: assert 'EXAMPLE' in 'PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin\nHOSTNAME=3d6f19b460f9\nHOME=/root\n'
================================================================= 1 failed in 1.24s =================================================================

Runtime environment
Provide a summary of your runtime environment. Which operating system, python version, and docker version are you using? What is the version of testcontainers-python you are using? You can run the following commands to get the relevant information.
# Get the operating system information (on a unix os).
$ Darwin WS-J26MXKQ4KP.tld 23.6.0 Darwin Kernel Version 23.6.0: Mon Jul 29 21:14:30 PDT 2024; root:xnu-10063.141.2~1/RELEASE_ARM64_T6030 arm64

# Get the python version.
$ Python 3.11.9

# Get the docker version and other docker information.
 I now understand that the docker compose --env-file argument does a different thing than the docker --env-file.
The docker compose one loads the env vars only for the scope of the docker compose file, not for the images that would be spun up.
To fix this issue, I added changed my docker-compose.yaml file:
services:
  alpine:
    image: alpine:latest
    init: true
    environment:
      - EXAMPLE='${EXAMPLE:-""}'

This will pull the value out of the env file and stick it into the EXAMPLE env var of the container, with a default value of an empty string.


Active rel subclass fix

Fixes #1133

I'm a little confused, were changes merged into Neo4j-core to allow this or does just take advantage of existing behavior?

Yeah, sorry, I'm working my way up the learning curve here...  This fix should take advantage of existing behavior.  Basically passing the labels in as an array already worked in neo4j-core, so the referenced subclass bug is fixed with this.
There was also a bug in neo4j-core when passing in parameter names with colon characters in them - my neo4j-core pull request is fixing that problem (but which is no longer triggered once this fix is merged here).
I also left in some explicit tests in that neo4j-core pull request for the existing "array of labels" behavior this fix uses that you guys may or may not want to keep.  My thought was to be explicit as well as doc the behavior in tests for future contributors.

This is great. Gonna merge, add a test, cherry-pick it back to 6.1.x, update changelog, and release a patch.

Just merged the PR which should help to support this


lerc_encode() may return a smaller size than lerc_computeCompressedSize(...)

@tmaurer3
The documentation says that lerc_computeCompressedSize is optional. But the size returned but lerc_encode may be less that what is needed for successful decoding, no way to know what that value is without calling lerc_computeCompressedSize.
I found out the hard way.
Can we have lerc_encode() return the actual number of bytes needed by decoder, aka the return from lerc_computeCompressedSize?

Or maybe the decoder could be changed to avoid addressing bytes outside of the written size?

@lucianpls  Can you provide a repro case? Those sizes should be the same, otherwise it's a bug.

sample.zip
See attached sample. It fails to decode because the size is short one byte. I am not sure how it was created, it's one of the Esri regression samples, see https://devtopia.esri.com/ArcGISPro/RasterXI/issues/10904
I assumed it was due to the refactoring of LERC in MRF, which uses the lerc_encode() size return, as described above. The old code was using the internal API has been in use for many years previously without ever seeing this issue.

Well, sounds like the data in question was generated in 2016, not sure why it is failing to decode now, after the switch to the public API.
The old code (with the internal API) is here, if you want to take a look.
https://github.com/OSGeo/gdal/blame/d9b489670e6373d4931e91515897a6021a46fa4b/frmts/mrf/LERC_band.cpp

The test in libtiff is also suggesting that this issue does exist, see assert in line 876:
https://github.com/OSGeo/gdal/blob/master/frmts/gtiff/libtiff/tif_lerc.c

@lucianpls  See if this branch works for you
https://github.com/Esri/lerc/tree/tmaurer3/fix_v22_decode_short_blob

@tmaurer3
Yes, it seems to, compiles and passes the reading test.
Can you check that I applied the patch correctly to libLERC source in GDAL, which is older? I had to merge it by hand.
https://github.com/lucianpls/gdal/commit/cfdc58c9ee948bb570876fba7c675a6ca5e616bf
Thanks,
Lucian

@lucianpls  Yes, looks good. Only you have this check 2x now, if (numBytes != numBytesLL). I would remove the 1st one.


Reccomendation: Add TZ to environment

Would recommend adding GRAYLOG_ROOT_TIMEZONE: "whatever" and TZ: "ZoneName" to the environment variables in the docker-compose files.
Since most people may not know how to edit the graylog.conf to modify the admin time (since it cannot be edited from the web console) as well as the container's time defaults to UTC.

I'm using
  environment:       GRAYLOG_NODE_ID_FILE: "/usr/share/graylog/data/config/node-id"       GRAYLOG_PASSWORD_SECRET: "${GRAYLOG_PASSWORD_SECRET:?Please configure GRAYLOG_PASSWORD_SECRET in the .env file}"       GRAYLOG_ROOT_PASSWORD_SHA2: "${GRAYLOG_ROOT_PASSWORD_SHA2:?Please configure GRAYLOG_ROOT_PASSWORD_SHA2 in the .env file}"       GRAYLOG_HTTP_BIND_ADDRESS: "<IP_ADDRESS>:9000"       GRAYLOG_HTTP_EXTERNAL_URI: "http://localhost:9000/"       GRAYLOG_ELASTICSEARCH_HOSTS: "http://opensearch:9200"       GRAYLOG_MONGODB_URI: "mongodb://mongodb:27017/graylog"       GRAYLOG_TZ: America/Sao_Paulo       GRAYLOG_TIMEZONE: America/Sao_Paulo       GRAYLOG_ROOT_TIMEZONE: America/Sao_Paulo


Publish master version

We have received an issue on gulp.spritesmith about being unable to commit node_modules to git due to a bad dependency in url.
https://github.com/twolfson/gulp.spritesmith/issues/34
This seems to have been patched via #5 but hasn't been published to npm. Can we publish the version on master?

Tests fail in Phantom on master. See https://github.com/defunctzombie/node-url/pull/16

Cool, I will subscribe to the PR. Thanks!

The PR seems to have landed, can we update to it and publish master?

Any progress on this?

Published 1.0.4 from master. Apologies for the delay. Would be glad to add
collaborators with npm and git accounts.
On Mon, Mar 30, 2015 at 12:54 AM, Todd Wolfson<EMAIL_ADDRESS>wrote:

Any progress on this?
—
Reply to this email directly or view it on GitHub
https://github.com/kriskowal/url2/issues/6#issuecomment-87582943.

No worries. I am happy to become a contributor. twolfson on GitHub and twolfson on npm. I am going to close this issue as closed =)


Description
This is one of the first building block matchers needed. Coordinates are the basis of all objects, and we need a way to reliably test that they have been formatted appropriately.

A GeoJSON Geometry object of any type other than "GeometryCollection" has a member with the name "coordinates". The value of the "coordinates" member is an array.  The structure of the elements in this array is determined by the type of geometry.  GeoJSON processors MAY interpret Geometry objects with empty "coordinates" arrays as null objects.
~ https://datatracker.ietf.org/doc/html/rfc7946#section-3.1

This matcher should limit its scope by only taking an argument of an individual array value (i.e. not the entire property).

A position is an array of numbers.  There MUST be two or more elements.  The first two elements are longitude and latitude, or easting and northing, precisely in that order and using decimal numbers.  Altitude or elevation MAY be included as an optional third element.
Implementations SHOULD NOT extend positions beyond three elements because the semantics of extra elements are unspecified and ambiguous.  Historically, some implementations have used a fourth element to carry a linear referencing measure (sometimes denoted as "M") or a numerical timestamp, but in most situations a parser will not be able to properly interpret these values.  The interpretation and meaning of additional elements is beyond the scope of this specification, and additional elements MAY be ignored by parsers.
~ https://datatracker.ietf.org/doc/html/rfc7946#section-3.1.1

An OPTIONAL third-position element SHALL be the height in meters above or below the WGS 84 reference ellipsoid.  In the absence of elevation values, applications sensitive to height or depth SHOULD interpret positions as being at local ground or sea level.
~https://datatracker.ietf.org/doc/html/rfc7946#section-4

This matcher will enforce three-member arrays. Separate matchers will be used for 2D arrays and for validating that geometries and features have a correct coordinates key.

WGS84 Bounds: -180.0000, -90.0000, 180.0000, 90.0000
Projected Bounds: -180.0000, -90.0000, 180.0000, 90.0000
~ https://spatialreference.org/ref/epsg/wgs-84/
Also, see https://datatracker.ietf.org/doc/html/rfc7946#section-4

Any values outside of these WGS-84 standards should fail.
The third array member must be a number from -Infinity to Infinity. The spec does not mandate a particular unit for this value (i.e. meters, feet, etc.)
Valid GeoJSON Examples
A three-element array of points:
{
   "type": "Feature",
   "geometry": {
       "type": "Point",
       "coordinates": [102.0, 0.5, 20]
   },
   "properties": {
       "prop0": "value0"
   }
}

An array of coordinates:
{
   "type": "Feature",
   "geometry": {
       "type": "LineString",
       "coordinates": [
           [102.0, 0.0, 0],
           [103.0, 1.0, 100],
           [104.0, 0.0, -20],
           [105.0, 1.0, 45]
       ]
   }
}

{
   "type": "Polygon", // Or LineString
   "coordinates": [
       [
           [100.0, 0.0, 0],
           [101.0, 0.0, 10],
           [101.0, 1.0, -10],
           [100.0, 1.0, 35],
           [100.0, 0.0, 0]
       ]
   ]
}

An array of arrays of coordinates:
{
   "type": "MultiLineString",
   "coordinates": [
       [
           [170.0, 45.0, 2], [180.0, 45.0, 5]
       ], [
           [-180.0, 45.0, 5], [-170.0, 45.0, 100]
       ]
   ]
}

Example Matcher Usage

expect([22, -34.549, 0]).isValid3DCoordinate()
expect([22, -34.549]).not.isValid3DCoordinate()
expect({ coordinates: [22, -34.549, 22, 0] }).not.isValid3DCoordinate()

Passing Tests
Values in Range

Each hemisphere

[0, 0, 0], [102.0, 0.5, 1000], [172.0, -15, -1000], [-10.9, 77, 5000], [-152.0, -33.33333, -5000]

At the boundaries

[180, 0, Infinity], [-180, 0, Infinity], [0, 90, Infinity], [0,-90, Infinity], [180, 90, -Infinity], [180,-90, -Infinity], [-180, 90, -Infinity], [-180,-90, -Infinity]

Failing Tests
Coordinates input not an array

undefined
null
Booleans: true, false
Numbers: 0, 200, -200, Infinity, -Infinity, NaN
Objects: { coordinates: [0, 0, 0] }
Strings: 'Random Coordinate', '[0, 0, 0]', '[[0, 0, 0], [0, 0, 0]]'

Incorrect number of array elements

[ ], [ 20 ], [20, 30], [20, 30, 0, 20, 30, 0, 20, 30, 0]

Coordinates out of range

Latitude: [0, 90.0000001, 0], [0, -90.0000001, 0], [0, 900000, 0], [0, -900000, 0]
Longitude: [180.0000001, 0, 0], [-180.0000001, 0, 0], [1800000, 0, 0], [-1800000, 0, 0]
Both: [181, 91, 0],  [181, -91, 0],  [-181, 91, 0],  [-181, -91, 0]
Each of the values in "Coordinates input not an array" paired with a valid coordinate:

[<value>, 0, 0]
[0, <value>, 0]
[<value>, <value>, 0]

Good Lon/Lat paired with non-number for third element: [0, 0, <value>]

undefined
null
Booleans: true, false
Numbers: NaN
Objects: { coordinates: [0, 0, 0] }
Strings: 'Random Coordinate', '[0, 0, 0]', '[[0, 0, 0], [0, 0, 0]]'

Array nested too deply

[ [ [1, 1, 0], [0, 0, 0] ] ]
[ [ [10, 20, 0], [2, 59, 0] ] ]
[ [ [10, 20, 0], [2, 90, 0], [95, 5, 0] ] ]
[ [ [ [1, 1, 0], [0, 0, 0] ] ] ], [ [ [ [ [2, 2, 0], [3, 3, 0] ] ] ] ]

:tada: This issue has been resolved in version 1.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


OnePlus font issue

Because OnePlus is using custom font, some texts in the profile overview page are cut off by several digits.
To resolve this issue, Roboto & SFUI fonts need to be bundled to the app. In addition, we need to force the app to display the texts using Robot/SFUI.

Fixed in #89


Automatic double quotes are added when importing data to the name of the new table

System information:

Windows 10, DBeaver 21.0.5 CE, Derby Server (reproduces in Postgres)

Steps to reproduce, if exist:

And when you disactivate table name field there double quotes are added:

The DDL:

And then it looks ugly in sql console, because you must do it in that way:

It occurs even if new table nave is in uppercase

It depends on the database you're transferring to. It may be upper-case database, so the table name in lower-case is quoted to keep the case intact. Also, quoting may happen when you use keywords as an identifier.


Create useful layout

There are no styles now, let's add some:

there should probably be three columns:| beginnings |    outcome    | endings |

changing to two columns on small screens?| beginnings | endings |
|        outcome       |

switch from inputs with spaces to textarea with new lines

Closed with 42cc856f00ef526e66114b9eac810ddf5d56f728


ActionSheet with Icons

https://github.com/Friend-LGA/LGAlertView/issues/33 -> does this solve your issue?

it does, thanks @mythodeia


docs: rewrite guide to build on ash_phoenix guide

Thanks for the hard work. @chrishop wrote the current guide, so I'll ask him for his feedback.


Apply missing permissions to topics.

Additional permissions required to find topics by their nitric name, these include listing and reading tags.

Codecov Report

:exclamation: No coverage uploaded for pull request base (develop@8455abe). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop      #95   +/-   ##
==========================================
  Coverage           ?   88.80%
==========================================
  Files              ?       81
  Lines              ?     1260
  Branches           ?      120
==========================================
  Hits               ?     1119
  Misses             ?      141
  Partials           ?        0

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 8455abe...e7444ea. Read the comment docs.


Fix #assertSaid working with strings instead of components

Description
Fixes MessageTarget#assertSaid working mainly with Strings instead of Components possibly leading to unexpected behavior.
Closes #520.
Checklist
The following items should be checked before the pull request can be merged.

[ ] Unit tests added.
[x] Code follows existing code format.

Should probabbly add a unit test around this no?
PlayerMock#SendMessage("&7My &7Message");
PlayerMock#AssertSaid("&7My &7Message");

Well thats not really testing our code as it is adventures. I'll add a text to ensure components are being used properly though