deps: h3-0.0.6 (w/ tracing feature as h3-quinn)

Resolved issues:
This change simply supports h3 v0.0.6 that updates its trait definitions and supports tracing feature.
Description of changes:
This PR changes the APIs of s2n-quic-h3 according to the update of h3's trait definition. This also adds an optional feature tracing as  the updates of the official crate h3-quinn  implementing h3 traits.
Call-outs:
I believe the change does not need to be updated at this point since it just follows the h3's API changes.
Testing:
Existing tests (but there is no test in s2n-quic-h3 as well as h3-quinn)
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Can you run cargo fmt and push to your branch? We should be able to merge it after that. Thanks!

Hi @camshaft
Yes, I've just done it :-) Thanks!


Reliability Issues with Azure Functions Backend

I have been running an Azure Static Web App that includes both a web application and a backend for an IoT project. Both the API endpoints for the front-end and for the IoT devices are managed through the SWA.
There are a couple of issues that have made the Azure Functions implementation difficult to rely on.
1. Backend API Reliability
A Raspberry Pi sends data to a function that is managed through the SWA project every 10 seconds, and I keep logs on the success of these requests.
There seem to be intermittent periods of reduced reliability that range from a single call to many calls over a period of hours.
Here is a sanitized version of my logs:
https://gist.github.com/rconard/86cad4ca05caf3c392411392eaec2d1f
The errors include:

404: Not Found
500: Internal Server Error
502: Bad Gateway
503: Service Unavailable

2. Long Response Times
Starting a few weeks ago, I am frequently getting API response times of 4 to 10 seconds. This is fine for a machine-2-machine communications, but it is uncomfortably long for an SPA that relies on an API response to load the interface. Application Insights is showing these longer response times.
These slow responses do seem to be connected to cold starts as they frequently occur when the SWA is first pulled up in the browser. They occur more frequently when the Raspberry Pi sending data is offline, but they still occur many times a day while the Raspberry Pi is sending a request every 10 seconds from the same IP address. Prior to a few weeks ago, responses seemed to be consistently fast as long as the Raspberry Pi was sending data.
Note: I have shared additional information on accessing my application's telemetry with @anthonychu .

Hi, could you please provide us with an app name and we can investigate what is going on.

Thanks @mkarmark added you to an internal thread with more info.

Closing as issue should be mitigated.


ENH: special: translate sph_bessel to C++, refactor cyl_bessel

This PR translates and refactors various Bessel functions, both spherical and cylindrical. Currently a work in progress.

@steppi This is done, ready for review.

OK, there's a lot going on here @izaid . I'm going to give you a pass, because I know this is the last week you'll be able to devote to SciPy for a while, so I think it's fine to err on the side of getting as much as possible done. This was tough to review though due to a combination of many unrelated changes, and difficulty determining what was essential. The only change I've seen that is needed is ensuring Python.h is the first included header wherever it appears.
Let me know if the following list of changes is exhaustive.
1. A large number of formatting changes.

2. Passing references instead of pointers for ufuncs with multiple return values.

3. _spherical_bessel.pxd translated into C++.

4. Cylindrical bessel functions moved out of `amos` and into `bessel.h`.
   Can you give an account of what other changes, if any, were made besides moving
   these to a new file and changing some function names?

5. sf_error.cc and sf_error.c combined to a single file, sf_error.c.

6. New header `scipy/special/special.h` for functions where SciPy behavior deviate from behavior in C++ library (returning multiple values, where is from an independent function, runtime warning for noninteger parameters, returning `NAN` when result should be 0.

7. New header for exponential integrals, translation of scaled_exp1 into C++ header.

Did I miss anything?

OK, there's a lot going on here @izaid . I'm going to give you a pass, because I know this is the last week you'll be able to devote to SciPy for a while, so I think it's fine to err on the side of getting as much as possible done. This was tough to review though due to a combination of many unrelated changes, and difficulty determining what was essential. The only change I've seen that is needed is ensuring Python.h is the first included header wherever it appears.
Let me know if the following list of changes is exhaustive.
1. A large number of formatting changes.

2. Passing references instead of pointers for ufuncs with multiple return values.

3. _spherical_bessel.pxd translated into C++.

4. Cylindrical bessel functions moved out of `amos` and into `bessel.h`.
   Can you give an account of what other changes, if any, were made besides moving
   these to a new file and changing some function names?

5. sf_error.cc and sf_error.c combined to a single file, sf_error.c.

6. New header `scipy/special/special.h` for functions where SciPy behavior deviate from behavior in C++ library (returning multiple values, where is from an independent function, runtime warning for noninteger parameters, returning `NAN` when result should be 0.

7. New header for exponential integrals, translation of scaled_exp1 into C++ header.

Did I miss anything?

That's pretty comprehensive! A lot of refactoring and things to make life easier, plus translation of the spherical Bessel functions. I've made the changes requested now too.


Dependabot opening PRs for ignored versions

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
Maven
Package manager version
3.9.3
Language version
Spring Boot 3.0.7, kotlin 1.8.21, java 17.
Manifest location and content before the Dependabot update
No response
dependabot.yml content
version: 2

# Allow Dependabot to use private registry
registries:
  artifact-registry:
    type: maven-repository
    url: https://europe-west1-maven.pkg.dev/vcm-shared/vcm-maven-artifacts
    username: _json_key
    password: ${{secrets.DEPENDABOT_SA_JSON}}
updates:
  - package-ecosystem: maven
    directory: "/"
    registries:
      - artifact-registry # Allow version updates for dependencies in this registry
    schedule:
      interval: daily
    ignore:
      - dependency-name: "sh.vcm.backend-commons-*"
        update-types: ["version-update:semver-major"]
      - dependency-name: "sh.vcm.graphql-*"
        update-types: [ "version-update:semver-major" ]
      - dependency-name: "sh.vcm.feature-toggle-client"
        update-types: [ "version-update:semver-major" ]
    open-pull-requests-limit: 50
    labels:
      - "kotlin"
      - "microservice"
      - "backend"
      - "dependencies"

Updated dependency
graphql-utils-auth
What you expected to see, versus what you actually saw
What I expected to see:

No PR opened for a major update

What I actually saw:

Bump graphql-utils-auth from 0.0.49 to 1.0.10

Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
No response

Since we ignored major updates (at least that is how we interpreted the config) for the specified libraries we expect no PR:s for major version updates (such as Bump graphql-utils-auth from 0.0.49 to 1.0.10).

As @jurre pointed out, the specific library that got an update does not have any ignore conditions configured, right? I'm confused too!

Doesnt
ignore:
- dependency-name: "sh.vcm.graphql-*"
update-types: [ "version-update:semver-major" ]
mean that we want to ignore version-update:semver-major for sh.vcm.graphql-*?
How else would you configure ignore? This is the doc we used

Shouldn't you ignore graphql-* instead? I don't know which role the sh.vcm. prefix plays here but graphql-utils-auth does not match sh.vcm.graphql-*.

Spotted the problem now. We had a . instead of : for as a delimiter between groupId and artifactId. From doc: "For Java dependencies, the format of the dependency-name attribute is: groupId:artifactId".
Thanks for your help. We can close this now


Add to docs: how is this different from RN built in RAM bundles?

Is your feature request related to a problem? Please describe.
React Native lays out how to implement RAM bundles: https://reactnative.dev/docs/ram-bundles-inline-requires
Describe the solution you'd like
What does this package do differently? What's the same? Why use or not use this project?

Hi @JulianKingman
The explanation is below:
This package is built on top of RAM bundles. It allows you to:
- avoid boilerplate code (you don't need to write code as per react-native docs - you can simply wrap your component into `register` function);
- use preload API (when you can load screens in advance and the transition to them will be as if the entire bundle was loaded at once);
- and has web support.

Where you would like to see it on a website? On which page?

I would put it in the getting started section of the documentation:
https://kirillzyusko.github.io/react-native-bundle-splitter/docs/getting-started

What does this package do?
This package is built on top of RAM bundles. In addition to the base functionality provided by React Native, it allows you to:

Avoid boilerplate... etc

I would also love to know how this package relates to Hermes (if you know), since this package conflicts with it (RAM bundles conflicts with it), and solves related problems (Hermes chunks byte code into separate bundles).

@JulianKingman yes, hermes is using similar mechanism. If you are using Hermes, you don't need to enable RAM bundles :)


Mechanism for external configuration

Any interest in enabling mission developers to use a callback to set configuration parameters instead of having to modify the GCICAP.lua file directly?

Sorry for the late reply. Yeah this is a good idea. Function calls are camel case in this script afair.


[NETBEANS-946] PHP 7.3 support

PHP 7.3 RFC
https://wiki.php.net/rfc#php_73
We need to provide support for the following features:

Allow a trailing comma in function calls : this PR
list() Reference Assignment : this PR
Flexible Heredoc and Nowdoc Syntaxes : Not Yet

I've implemented No.1 and 2 in this PR. Main changes are php/php.editor/tools/ASTPHP5Parser.cup and php/php.editor/src/org/netbeans/modules/php/editor/verification/PHP73UnhandledError.java. Other than those, I added unit tests.
However, No.3 is not easy because of BC break.
Probably, we should create a lexer(and parser, too?) for PHP 7.3, but I suppose unit tests will be complicated. (We are using the common lexer and parser for all PHP versions, so far.)

@junichi11

However, No.3 is not easy because of BC break.
Probably, we should create a lexer(and parser, too?) for PHP 7.3, but I suppose unit tests will be complicated. (We are using the common lexer and parser for all PHP versions, so far.)

Why exactly? Could you explain, please? Thanks.

@tmysik
I'm not working on anything about it yet. So, just my estimate... somehow I just thought like that. I'll try it once :) So, just ignore my above comment. Thanks!

@junichi11
You can be right, of course; I just had a feeling that it could be backward compatible (and also I am not aware of many BC breaks on the PHP side for a long time). We will see.
Thanks.

@tmysik
In the below case, unit tests fail if we implement the new syntax. We cannot implement both syntaxes at the same time. However, a user should be able to choose another marker instead. In this case, e.g. choose HEREDOCMARKER instead of ENDOFHEREDOC
https://github.com/apache/incubator-netbeans/blob/6a382b93ebc5a99ee99306b8c758126aba9de59c/php/php.editor/test/unit/data/testfiles/parser/heredoc_009.php#L8-L24
My idea: Just don't provide support for the above case(the old syntax) because I think that that case is almost never used and we can use another marker instead.
That way, we need not create a new lexer for PHP 7.3, maybe. What do you think?
Thanks.

Not sure if I fully understand. I would remove the unit test, basically, this is a nonsense and IMHO worked earlier in PHP just by a mistake. So, NetBeans would "fail" on such a code now - for how many users is this a problem? Three? Maybe less? :)
I would really try to avoid to have more lexers - it will be hell to maintain them all (unless we are able to create them somehow "smart" so they can share their code). What do you think, does it make sense to support mainly the new syntax and the old syntax as much as possible? And if someone really reports a bug, she should fix her code, in fact (that should be trivial).
Am I missing something? Please let me know. Thanks.

Exactly! I think so. Let's go with it :)
We provide support for the old syntax no longer. If someone reported about it, I would say "Please use another text as a marker" ;)
Thanks!

Yes, the code will stop working with PHP 7.3 so it will need to be fixed anyway...


"Our paths" section links are broken

Description
Right now pressing anay of those buttons will link you back to the main page :

Would be nice if the discord button would work for exmaple since the community is active on discord
Screenshots
No response
Additional information
No response

Thank you for mentioning the duplications

@Tamir198 If you can I would love a review of the PR I submittede to address the issue.

Sure, i will aee to it later tosay. Link to the pr?

Sure, i will see to it later today. Link to the pr?
#144


Title false information

Previous author isn't complacent
False information

I think I need more details to understand what's the issue here.


Ansi art

Create ansi art and asciinema players

Coverage increased (+0.02%) to 97.178% when pulling ac609a33d7ef7e7f3c89c86170c0f2facf8a4ba0 on ansi_art into 1fca1417f46abce910f15b1aa6c5721b268139b5 on master.


feat: more integrability statements for iterated derivatives of Schwartz functions

bors r+
Thanks for the review!


Receiving error using autoload/classmap - AssertFileEqualsSpecializations::assertFileEqualsCanonicalizing

This issue occured when I wanted to make run Wordpress plugin Integrationtests. I used following tutorials:
https://make.wordpress.org/cli/handbook/misc/plugin-unit-tests/ and https://sarathlal.com/unit-testing-wordpress-plugin-using-phpunit/
Following those tutorials, running phpunit requires me to install yoast/phpunit-polyfills.
Doing so gives me biiig warnings during installing phase. Afterwards when I want to run phpunit ./bin/vendor/phpunit -c phpunit.xml.dist it gives me some weird errors:
[05-Feb-2024 21:48:16 UTC] PHP Fatal error:  Declaration of Yoast\PHPUnitPolyfills\Polyfills\AssertFileEqualsSpecializations::assertFileEqualsCanonicalizing($expected, $actual, $message = '') must be compatible with PHPUnit\Framework\Assert::assertFileEqualsCanonicalizing(string $expected, string $actual, string $message = ''): void in /var/www/html/wp-content/plugins/voting/vendor/yoast/phpunit-polyfills/src/TestCases/TestCasePHPUnitGte8.php on line 31

Fatal error: Declaration of Yoast\PHPUnitPolyfills\Polyfills\AssertFileEqualsSpecializations::assertFileEqualsCanonicalizing($expected, $actual, $message = '') must be compatible with PHPUnit\Framework\Assert::assertFileEqualsCanonicalizing(string $expected, string $actual, string $message = ''): void in /var/www/html/wp-content/plugins/voting/vendor/yoast/phpunit-polyfills/src/TestCases/TestCasePHPUnitGte8.php on line 31

It turns out, I had to exclude the yoast dependency from the classmapping of composer. Then everything works fine. Like this:
{
    "autoload": {
        "classmap": [
            "vendor/",
            "src/"
        ],
        "exclude-from-classmap": ["vendor/yoast"]
    },
    "require-dev": {
        "phpunit/phpunit": "^8",
        "10up/wp_mock": "*",
      "yoast/phpunit-polyfills": "^2.0"
    }
}

Doing composer update it does not show significant warnings and finally ./bin/vendor/phpunit -c phpunit.xml.dist does work.
Maybe you can address this issue by the next release. Right now it works with the exclusion. I hope this helps somebody.

@mszkb Sorry you are experiencing problems, but this is not an issue with the PHPUnit Polyfills. but user error.
You should never include vendor in your classmap and most definitely not in an autoload directive.
Each dependency defines their own autoload requirements and Composer puts those all together resulting in a vendor/autoload.php file (with sub-files) which handles it all.
This file will differ depending on whether the install was run with --no-dev or without.
The autoload directive in your composer.json should only include your own files.
Additionally, anything non-runtime related (like tests), should be handled via a separate autoload-dev directive.
As side-notes:

For WP integration tests you should not be using the ^2.0 version as the WP Core test suite is not compatible w with PHPUnit 10/PHPUnit Polyfills 2.0. You need to use ^1.0.
Also when using the PHPUnit Polyfills, it is better to not declare a dependency on PHPUnit. The Polyfills have that dependency and your own declaration could conflict with the dependency declaration from the polyfills, so it is better to let the Polyfills package manage the supported PHPUnit versions.

All in all, I don't know those tutorials, but you may want to educate yourself about the use of Composer a little more.
I'd suggest starting with the Composer documentation: https://getcomposer.org/doc/


Antlr Tool fails on Ada2005Parser.g4

grammar: Ada2005Lexer.g4 Ada2005Parser.g4
antlr-ng version: https://github.com/mike-lischke/antlr-ng/commit/e9423bbd9c83a8f2581be1e84282075c32a59483
Output:
ken@DESKTOP-DL44R7B:~/antlr-ng$ node --no-warnings --loader ts-node/esm cli/runner.ts -Dlanguage=TypeScript   $w/Ada2005Parser.g4
file:///home/ken/antlr-ng/src/codegen/model/Action.ts:62
            __runInitializers(this, _chunks_extraInitializers);
            ^

ReferenceError: Must call super constructor in derived class before accessing 'this' or returning from derived constructor
    at new Action (file:///home/ken/antlr-ng/src/codegen/model/Action.ts:62:13)
    at ParserFactory.action (file:///home/ken/antlr-ng/src/codegen/ParserFactory.ts:72:17)
    at OutputModelController.action (file:///home/ken/antlr-ng/src/codegen/OutputModelController.ts:350:33)
    at SourceGenTriggers.element (file:///home/ken/antlr-ng/src/tree-walkers/SourceGenTriggers.ts:501:48)
    at SourceGenTriggers.alt (file:///home/ken/antlr-ng/src/tree-walkers/SourceGenTriggers.ts:300:57)
    at SourceGenTriggers.alternative (file:///home/ken/antlr-ng/src/tree-walkers/SourceGenTriggers.ts:221:26)
    at SourceGenTriggers.block (file:///home/ken/antlr-ng/src/tree-walkers/SourceGenTriggers.ts:168:53)
    at SourceGenTriggers.subrule (file:///home/ken/antlr-ng/src/tree-walkers/SourceGenTriggers.ts:863:40)
    at SourceGenTriggers.element (file:///home/ken/antlr-ng/src/tree-walkers/SourceGenTriggers.ts:492:41)
    at SourceGenTriggers.alt (file:///home/ken/antlr-ng/src/tree-walkers/SourceGenTriggers.ts:300:57)

Node.js v23.5.0
ken@DESKTOP-DL44R7B:~/antlr-ng$

The tool works fine generating the lexer for the lexer grammar, which is done first. It fails for the second (Parser) grammar.

I cannot reproduce this problem with the latest code. Both C++ and TypeScript parsers are generated as expected.

Ah, I know the reason for this failure: ts-node. When you use tsx instead it should run fine.

Works fine with latest.


Add dtls multiaddr

This PR adds a dtls multiaddr to resolve #75. I've not foreseen a value part for now. 0x20 also seems free in multiformats/multicodec. Let me know if I should also open a PR on that repo.

So, regardless of whether or not go-libp2p can currently use dtls, we can still add the protocol code. However,

We should first PR against https://github.com/multiformats/multicodec as that's the complete list of codecs.
We should probably pick a two-byte multicodec as we have very few single-byte multicodecs available.


community: Update get_table_info() for better explainability

I propose this to have both custom information, info from the data def lang and example rows.
I have noticed that when you provide the dictionary with custom information about your tables you do not get the data definition language (the schema with create table ...) and the rows examples. I modified just a couple rows to accommodate for CUSTOM DESCRIPTIONS + DEFAULT DESCRIPTIONS
@RechManuel my X account

Could you provide a before and after from this function? Otherwise it's not clear whether default behavior should be changed for all users

If we decide to incorporate change, we'd need to:
2) Add a unit test for this functionality
3) Potentially provide users with opt-in control

LC
The last part of the function get_table_info() was
for table in meta_tables:
            if self._custom_table_info and table.name in self._custom_table_info:
                tables.append(self._custom_table_info[table.name])
                continue

            # Ignore JSON datatyped columns
            for k, v in table.columns.items():
                if type(v.type) is NullType:
                    table._columns.remove(v)

            # add create table command
            create_table = str(CreateTable(table).compile(self._engine))
            table_info = f"{create_table.rstrip()}"
            has_extra_info = (
                self._indexes_in_table_info or self._sample_rows_in_table_info
            )
            if has_extra_info:
                table_info += "\n\n/*"
            if self._indexes_in_table_info:
                table_info += f"\n{self._get_table_indexes(table)}\n"
            if self._sample_rows_in_table_info:
                table_info += f"\n{self._get_sample_rows(table)}\n"
            if has_extra_info:
                table_info += "*/"
            tables.append(table_info)
        tables.sort()
        final_str = "\n\n".join(tables)
        return final_str

and I changed into
for table in meta_tables:
            if self._custom_table_info and table.name in self._custom_table_info:
                if self._custom_and_default_info:
                    table_info = f"{table} description: {self._custom_table_info[table.name]}\n\n"
                else:
                    tables.append(self._custom_table_info[table.name])
                    continue

            # Ignore JSON datatyped columns
            for k, v in table.columns.items():
                if type(v.type) is NullType:
                    table._columns.remove(v)

            # add create table command
            create_table = str(CreateTable(table).compile(self._engine))
            table_info =  table_info + f"{table} DDL:{create_table.rstrip()}" if self._custom_and_default_info else f"{table} DDL:{create_table.rstrip()}"
            has_extra_info = (
                self._indexes_in_table_info or self._sample_rows_in_table_info
            )
            if has_extra_info:
                table_info += "\n\n/*"
            if self._indexes_in_table_info:
                table_info += f"\n{self._get_table_indexes(table)}\n"
            if self._sample_rows_in_table_info:
                table_info += f"\n{self._get_sample_rows(table)}\n"
            if has_extra_info:
                table_info += "*/"
            tables.append(table_info)
        tables.sort()
        final_str = "\n\n".join(tables)
        return final_str

I added the opt-in control as you suggested me but I am sorry I do not know how to create proper unit test, I am not very experienced, I just added this feature locally and I thought of sharing it!

@manuelrech thanks! I meant the before and after on the return value of the function -- since It's difficult to tell what the change is doing. the diff on the function is visible on the PR itself, it's a bit hard to tell how the change affects downstream code etc

Okay I see. So supposing you instantiate you SQLDatabase like this
db = SQLDatabase(
    engine=engine,
    ignore_tables=['team_data_short', 'player_data_short', 'kpi_definitions_short', 'affiliations'],
    custom_table_info={
        "MatchStats": "Stores match setups and outcomes, featuring dual entries per match ID to represent both competing teams. Each record uniquely identified by a composite primary key consisting of match_id and team_id.",
        "PlayerStats": "Details player-specific performance metrics, akin to MatchStats' structure. Unique entries identified by a composite primary key (match_id, player_id), with match_id linking to MatchStats and team_id used for additional relational context but not as a part of the primary key.",
        "TeamStats": "Captures team performance statistics, structured similarly to MatchStats. Each entry is uniquely identified by a composite primary key (match_id, team_id), which also serves as a foreign key referencing MatchStats"
        }
    )

for example for the SQLDatabaseToolkit() to be used in the react agent for querying a db.
Given a question, the agent will look at the table names and in order to generate the sql query(s). To answer it need to look at the schemas of some of the tables he thinks are relevant. The problem comes here. In the for loop shown above over the meta_tables, the code will get a custom description (as the one I have provided) and it will leave the current iteration of the meta_tables on the continue keyword, not adding the DDL as in create_table = str(CreateTable(table).compile(self._engine)) and the sample rows as in table_info += f"\n{self._get_sample_rows(table)}\n", which are both fundamental for generating a good query.
As an example, before I would get
Tool call: sql_db_schema
    Tool arguments: {'table_names': 'PlayerStats'}
----------------------------------------------------------------------------------------------------
Details player-specific performance metrics, akin to MatchStats' structure. Unique entries identified by a composite primary key (match_id, player_id), with match_id linking to MatchStats and team_id used for additional relational context but not as a part of the primary key.

and now I get
Tool call: sql_db_schema
    Tool arguments: {'table_names': 'PlayerStats'}
----------------------------------------------------------------------------------------------------
Tool response: PlayerStats description: Details player-specific performance metrics, akin to MatchStats' structure. Unique entries identified by a composite primary key (match_id, player_id), with match_id linking to MatchStats and team_id used for additional relational context but not as a part of the primary key.

PlayerStats DDL:
CREATE TABLE `PlayerStats` (
	match_id INTEGER NOT NULL,
	player_id INTEGER NOT NULL,
	team_id INTEGER,
	`role` TEXT COMMENT 'can be Goalkeeper, Defender, Midfielder, Forward',
	start11 TINYINT COMMENT '1 if the player was in the starting11',
	minutes DOUBLE COMMENT 'minutes played by the player per game',
	`teamActions` INTEGER COMMENT 'number of team actions while player was on field',
	`oppTeamActions` INTEGER COMMENT 'number of oppenent''s team actions while player was on field',
	`timeIn` INTEGER COMMENT 'second in which the player entered the game, 0 if he/she was on the starting 11',
	`timeOut` INTEGER COMMENT 'second in which the player left the game, 0 for the substituted in and max value for those who played the entire game',
	`normalizedGame` DOUBLE COMMENT 'normalization constant representing how much of the total gametime was played',
	`gamePlayed` TINYINT COMMENT '1 if the player played in the game',
	`substitutedOff` TINYINT COMMENT '1 if the player was substituted off',
	`substitutedOn` TINYINT COMMENT '1 if the player was substituted on',
	passes INTEGER COMMENT 'number of player passes',
	`successfulPasses` INTEGER COMMENT 'number of player''s successful passes',
	`receivedPasses` INTEGER COMMENT 'number of player''s received passes',
	`forwardPasses` INTEGER COMMENT 'number of player''s passes to forwards',
	crosses INTEGER COMMENT 'number of player''s crosses ',
	`successfulCrosses` INTEGER COMMENT 'number of successful crosses made',
	`keyPasses` INTEGER COMMENT 'number of player''s key passes',
	assists INTEGER COMMENT 'number of player''s assists',
	`targetShots` INTEGER COMMENT 'number of player''s shots on target',
	`postShots` INTEGER COMMENT 'number of player''s shots on post',
	`boxGoals` INTEGER COMMENT 'number of player''s goals from the box',
	`outBoxGoals` INTEGER COMMENT 'number of player''s goals from outside the box',
	shots INTEGER COMMENT 'amount of player''s shots taken',
	`boxShots` INTEGER COMMENT 'number of player''s shots from inside the box.',
	`outBoxShots` INTEGER COMMENT 'number of player''s shots from outside the box',
	goals INTEGER COMMENT 'number of goals made',
	PRIMARY KEY (match_id, player_id),
	CONSTRAINT fk_playerstats_match FOREIGN KEY(match_id, team_id) REFERENCES `MatchStats` (match_id, team_id)
)DEFAULT CHARSET=utf8mb4 ENGINE=InnoDB COLLATE utf8mb4_0900_ai_ci

[/](https://file+.vscode-resource.vscode-cdn.net/)*
3 rows from PlayerStats table:
match_id	player_id	team_id	role	start11	minutes	teamActions	oppTeamActions	timeIn	timeOut	normalizedGame	gamePlayed	substitutedOff	substitutedOn	passes	successfulPasses	receivedPasses	forwardPasses	crosses	successfulCrosses	keyPasses	assists	targetShots	postShots	boxGoals	outBoxGoals	shots	boxShots	outBoxShots	goals
CANNOT DISCLOSE THE CONTENTS FOR PRIVACY REASONS BUT THEY WOULD BE THERE*/

Also, I think this should be a default behaviour because most likely users will provide a string description and not the DDL and sample rows, so the default flag could be set as True (now is False)

Feel free to re-open if you can address comments


Implement testing within the react framework

Is this completed with our unit and functional tests?

Yes, I think so.


Refactor images patterns and components

Tested in staging ✅
Tested locally🚫
Folder structure isn't the same between local environment and staging environment but similar between staging and production environment.
Will investigate a JEKYLL_ENV strategy to make local environment simulate staging and production environments.


cmd/vet: Panics with " already declared" on 1.7/master

What version of Go are you using (go version)?
go version devel +499cd33 Tue May 3 00:49:46 2016 +0000 darwin/amd64
What operating system and processor architecture are you using (go env)?
darwin/amd64
What did you do?
Running go vet on the code base as part of the build, as usual. Works fine in 1.6. With current master, go vet panics on some parts of the code. Only happens when the package has actually been built.
$ cd $GOPATH/src/github.com/syncthing/syncthing
$ git describe --always
v0.13.0-beta.3-24-g29fa05a
$ go vet ./cmd/stindex     # this is fine
$ go install ./cmd/stindex
$ go vet ./cmd/stindex     # this is no longer fine
panic: Open already declared [recovered]
	panic: Open already declared

goroutine 1 [running]:
panic(0x2416a0, 0xc8201e5510)
	/usr/local/go-exp/src/runtime/panic.go:500 +0x18c
go/types.(*Checker).handleBailout(0xc8200ba1c0, 0xc82047f908)
	/usr/local/go-exp/src/go/types/check.go:213 +0x95
panic(0x2416a0, 0xc8201e5510)
	/usr/local/go-exp/src/runtime/panic.go:458 +0x226
go/internal/gcimporter.(*importer).declare(0xc820090000, 0x3b4e60, 0xc8201989b0)
	/usr/local/go-exp/src/go/internal/gcimporter/bimport.go:165 +0x156
go/internal/gcimporter.(*importer).obj(0xc820090000, 0xfffffffffffffffb)
	/usr/local/go-exp/src/go/internal/gcimporter/bimport.go:193 +0x477
go/internal/gcimporter.BImportData(0xc820015050, 0xc8201fa000, 0xd9120, 0xffe00, 0xc8201572f1, 0x23, 0x0, 0x49, 0x283a40, 0x1)
	/usr/local/go-exp/src/go/internal/gcimporter/bimport.go:83 +0x306
go/internal/gcimporter.Import(0xc820015050, 0xc8201572f1, 0x23, 0x7fff5fbff75c, 0xb, 0xc8201f5d10, 0x0, 0x0)
	/usr/local/go-exp/src/go/internal/gcimporter/gcimporter.go:166 +0x527
go/importer.gcimports.ImportFrom(0xc820015050, 0xc8201572f1, 0x23, 0x7fff5fbff75c, 0xb, 0x0, 0xc8201c5ea0, 0x0, 0x0)
	/usr/local/go-exp/src/go/importer/importer.go:70 +0x5a
go/types.(*Checker).collectObjects(0xc8200ba1c0)
	/usr/local/go-exp/src/go/types/resolver.go:191 +0x7c9
go/types.(*Checker).checkFiles(0xc8200ba1c0, 0xc820173ca0, 0x4, 0x4, 0x0, 0x0)
	/usr/local/go-exp/src/go/types/check.go:225 +0xa0
go/types.(*Checker).Files(0xc8200ba1c0, 0xc820173ca0, 0x4, 0x4, 0xc8201715f0, 0x18)
	/usr/local/go-exp/src/go/types/check.go:218 +0x3f
go/types.(*Config).Check(0xc820175280, 0xc820011898, 0x4, 0xc820013500, 0xc820173ca0, 0x4, 0x4, 0xc82016eff0, 0x18, 0xc820176b60, ...)
	/usr/local/go-exp/src/go/types/api.go:344 +0x184
main.(*Package).check(0xc82006f1a0, 0xc820013500, 0xc820173ca0, 0x4, 0x4, 0x0, 0xc8200a5400)
	/usr/local/go-exp/src/cmd/vet/types.go:76 +0x330
main.doPackage(0x298dbc, 0x1, 0xc82000a1f0, 0x4, 0x4, 0x0)
	/usr/local/go-exp/src/cmd/vet/main.go:333 +0x951
main.main()
	/usr/local/go-exp/src/cmd/vet/main.go:241 +0x317
exit status 2

Interestingly, the following doesn't panic:
go vet ./cmd/stindex
go install -gcflags="-newexport=0" ./cmd/stindex
go vet ./cmd/stindex

@griesemer will this be backported to /x/tools/go/gcimporter15 or is one not supposed to use that package anymore?

@dominikh This should have been back-ported already (if not, please file an issue and send it to me). /x/tools/go/gcimporter15 will stick around for the time being until go/importer provides some functionality it is currently missing.


Convert all type comments to type annotations with com2ann

Proposed on dev@: https://lists.apache.org/thread/qn2pmk389zclyncjzt9fcol8jko6lnh8
WIP PR: https://github.com/apache/beam/pull/17109
Requires from __future__ import annotations, so blocked on dropping Python 3.6 support.
Imported from Jira BEAM-14180. Original Jira may contain additional context.
Reported by: bhulette.

Unable to assign user @theneuralbit. If able, self-assign, otherwise tag @damccorm so that he can assign you. Because of GitHub's spam prevention system, your activity is required to enable assignment in this repo.


Update readme to reflect state/settings refactoring

Much has changed since original README.md
Review and update.

Updated!


Option tag with v-for causing DOM nodes memory leak

Version
2.6.10
Reproduction link
https://jsfiddle.net/rcmo2Lwx/1/
Steps to reproduce
Press the toggle button.
What is expected?

Not all nodes are cleared

How to see the garbage collection
Open the fiddle in  https://fiddle.jshell.net/rcmo2Lwx/1/show/light/ (single view of the jsFiddle, so you can open devtools)
Open dev tools -> performance tab->record
Press toggle a bunch of times (or hold enter to keep activating it)
Stop recording
Keyed version
https://fiddle.jshell.net/rcmo2Lwx/2/
https://fiddle.jshell.net/rcmo2Lwx/2/show/light/
The same issue is still happening

Seems to happen with vanilla JS too in latest chrome. I will check a bit more

As I understand browsers will schedule GC at appropriate time and we don't have control over that.

But shouldn't we be able to still force the garbage collection? I opened https://bugs.chromium.org/p/chromium/issues/detail?id=949587 for the moment

Forgot to close this. It's a bug in Chrome


feat(pagination): change pagination display size (sm/lg)

Adding input 'size' to pagination component to support bootstrap 'pagination-lg' and 'pagination-sm' styles
Closes #314

LGTM part from the remark about tests.

Thanks for the explanation, amended

LGTM. Will merge as soon as TravisCI is green.


flutter_gallery's frame build time is super slow on iPhone6S (and possibly other newer iPhones too)

Running flutter drive -t test_driver/transitions_perf.dart inside flutter_gallery gets the following build time benchmarks:
  "average_frame_build_time_millis": 22.62587272727273,
  "90th_percentile_frame_build_time_millis": 31.072,
  "99th_percentile_frame_build_time_millis": 226.192,
  "worst_frame_build_time_millis": 367.374,
  "missed_frame_build_budget_count": 53,
  "average_frame_rasterizer_time_millis": 4.706748858447491,
  "90th_percentile_frame_rasterizer_time_millis": 12.663,
  "99th_percentile_frame_rasterizer_time_millis": 13.571,
  "worst_frame_rasterizer_time_millis": 15.494,
  "missed_frame_rasterizer_budget_count": 0,

which is much slower than the iPhone4S and iPhone6 in our device lab. For example:
                            | iPhone4S  | iPhone6   | iPhone6S
average frame build time    | 5.76ms    | 2.6ms     | 22.6ms

The slow build time seems to only happen during the first animation (which is what our driver test is measuring).
This issue was found because of a customer complaint in https://github.com/flutter/flutter/issues/28113#issuecomment-476530608 . According to that, this seems to also happen on newer iPhones such as iPhone X.
Actions to be taken:

Test this on other newer iPhones such as iPhoneX to see how wide this issue is.
Add iPhone6S or newer iPhones to our device lab. (Maybe create a separate issue to track this.)
Add more tracings to our UI thread frame build process to figure out why it's slow.

CC @cbracken @dnfield @digiter for adding iPhone6S or newer iPhones.

Nevermind, I forgot to add --profile to flutter drive so it's in debug mode... Close this issue.

Well don't leave us hanging - what are the times in profile mode? 😃

@dnfield : I just found another issue: the driver test isn't showing any image resources (https://github.com/flutter/flutter/issues/30641). That's probably why our driver test is so fast while in practice the app is much slower on first animation build. BTW, here's the profile-mode driver test's benchmark (without showing any images).
  "average_frame_build_time_millis": 2.204256756756756,
  "90th_percentile_frame_build_time_millis": 2.981,
  "99th_percentile_frame_build_time_millis": 12.02,
  "worst_frame_build_time_millis": 23.595,


Global Attribute Cluster Revision needs to be fixed for Thermostat User Interface Configuration Cluster

Global Attribute Cluster Revision needs to be fixed for Thermostat User Interface Configuration Cluster
Spec snippet :

Value needs to be 2

General question: how do we want to handle this?  Right now cluster revision defaults to 1 in the XML but then you can set the right thing in ZAP.  Should we be adding overrides of the default value to our cluster XML definitions?  Seems like it would be a good idea....
@vivien-apple @tecimovic

The cluster revision is a quality of the schema definition, not the product-configuration that is encapsulated in the ZAP file.
It should exist in the schema definition. I've recommended to Terrance (who's working on en-mass, automatically inferring a default value for Cluster Revision if one doesn't exist in the attribute store) to automatically stuff a value of 1 if one doesn't exist.
Which means we can have the rule that you either specify the cluster revision in the XML (which can be set to a specific value) or you don't, in which case, it would default to 1.

That seems pretty sensible, yes.

XML already has a per-cluster default value for a global attribute. Is this the case here?

Yes, I know we can set defaults via <globalAttribute> in the cluster XML.  The question was whether that's what we want here.  Sounds like it is, and we should do this for all our clusters...
@tecimovic I assume after that we will still need to manually update .zap files that now have the wrong values?

The values inside .zap files will take precedence, as they are treated as "user typed this in!" which overrides the defaults, wherever they come from.
You would have to either:
1.) Manually update them.
2.) Use the zap scripting to update them automatically via a script.
When I refer to zap scripting I refer to the "post-load" functionality.
When you run zap, you can specify a ".js" file via a --postImportScript command line arg. They point to a file that looks like this:
// Example file for the post load functionality.

async function postLoad(api, context) {
  let endpoints = await api.endpoints(context)
  let ep = endpoints[0]
  // Here we disable cluster with code 2
  await api.disableClientCluster(context, ep, 2)

  // Here we turn off attribute 4 on cluster 0, the manufacturer name
  await api.disableServerAttribute(context, ep, 0, 4)

  await api.enableOutgoingCommand(context, ep, 0x00, 0x00, api.client)
  await api.disableOutgoingCommand(context, ep, 0x00, 0x00, api.client)
}

exports.postLoad = postLoad

(Example takes from some of the unit tests in the test/resource/ dir...)
Essentially, we have this little API that you can use from the .js file, that allows you to muck around with the data right after the ZAP file is loaded into the tables.
So you can then use the:
zap convert --out FILE.ZAP --in FILE.ZAP --postImportScript SCRIPT.JS
command, and that will convert the ZAP file with whatever the script tells it to do, plus any automatic upgrades that zap does to the ZAP file along the way anyway.

A 'default' value for a ClusterRevision doesn't make any sense in an XML definition - unlike other attributes where the value can change, this attribute should have a fixed value that is a function of what is defined in the XML. It should not be over-ridable in a ZAP file, or tweakable later.

I'm almost inclined to say that 'ClusterRevision' not be specified as an attribute in the XML definition but rather, as a separate bespoke tag/value pair to wean it off the association to the attribute, which is just a way to surface it in the Matter IM, and in this representation, couples in baggage and implicit assumptions about the behavior of attributes (which don't apply to this quality/metadata).

SDK Spec Review: At this point we believe that the version is correct to be spec compliant.

SDK Spec Review: Given no cert tests are failing here (cert blocker), we believe this is likely resolved.


[Enhancement] Upgrade AWS S3 SDK to 1.10.36

The reason that needs to upgrade aws-sdk-cpp is already described in https://github.com/StarRocks/starrocks/issues/18912
Consider China's cloud storage like KS3, OBS, if the request path contains '=', will face 403 problems. So we need to upgrade s3 sdk, to enable compliantRfc3986Encoding.
And we are using the same version as ClickHouse, it will be more stable.
Previous pr about this problem: #18912 #19000 #19229 #19228
We need to change BE's  CMakeFiles also, otherwise ci will be failed.
IMPORTANT
Introduce a new parameter aws_sdk_enable_compliant_rfc3986_encoding in be.conf, if you are using OBS/KS3 cloud storage, you should set aws_sdk_enable_compliant_rfc3986_encoding=true.

can't backport to branch-3.0 directly for now.
If absolutely needed, please contact @kevincai for how to proceed.


Replacing common element ids

Hello, I hope you can also add an option to add a prefix for element ids like for .row, .button and etc.
Thanks

Hi, I did not get this question. You can currently assign any custom class names to any view via descriptions. Just add class: "yourClassName" to description of desired view. You can also use Dynamic values, so you can assign it a with a prefix eg. class: ".row-{_yourClassName}"


Update README.md

Related discussions:

Confirming this netpol is for cluster DNS in general not the k8s-dns-node-cache-app https://gigantic.slack.com/archives/C02GLN41UP2/p1711016179149589
Confirming it works for cilium in ENI mode: https://gigantic.slack.com/archives/C02HLSDH3DZ/p1711019026080759

PTALA


custom style not working

If i add custom style as follows
import * as customStyle from 'react-tabtab/lib/themes/bootstrap';
This works fine.
However, if i create my own style  following instructions and put into themes/index.js. Import as follows
import * as customStyle from '../themes';
Then it does not work. Not work meaning the styles are not picked up from my index.js, there are no errors either.
My index.js is exact copy of https://github.com/ctxhou/react-tabtab/tree/master/src/themes/bootstrap

Hi @prodigylabs
Could you paste your copy code to let me know more?
Do you change this line to import {styled} from 'react-tabtab'?


SDK column in the info panel supports displaying visionOS

Xcode 15 beta 2 now supports visionOS SDK.

Should be good to merge it?

This would resolve issue https://github.com/XcodesOrg/XcodesApp/issues/432

Xcode 15.2 came with support of visionOS 1.0, should we release a new version to support this feature?


Added form buttons component

Closes #1095 . Added a form component. Screenshot attached.

Made separate files for each one


Upgrade lnd tor to version v1.1.0

The goal: update lnd/tor to version v1.1.0 so that we can use aperture in taro.

Need to bump the Golang version in CI, the one being used now is pretty ancient.

@guggero @ellemouton Thanks for the reviewing!
I don't have write access to this repository. Can either of you merge please?


Datastore client not getting ProjectID from env

It looks like Datastore client is not getting the projectID from the env automatically.
var pubsub = gcloud.pubsub();

This works fine, but:
var dataset = gcloud.datastore.dataset();

returns:
Error: Sorry, we cannot connect to Google Cloud Services without a project ID. You may specify one with an environment variable named "GCLOUD_PROJECT". See https://googlecloudplatform.github.io/gcloud-node/#/authentication for a detailed guide on creating an authenticated connection.

This fixes the problem
var dataset = gcloud.datastore.dataset({projectId: process.env.GCLOUD_PROJECT});

Good catch, thanks for letting us know. PR incoming!

im not sure if other clients also have this problem. I though they all extend the same base class?

I checked, all of the other APIs do it.
@callmehiphop can you think of a way we can move the normalizing to the Service constructor?

@stephenplusplus we should be able to just normalize them in the Service constructor, no?

I think there might be an issue with double-instantion?
On Thu, Jan 28, 2016, 11:01 AM Dave Gramlich<EMAIL_ADDRESS>wrote:

@stephenplusplus https://github.com/stephenplusplus we should be able
to just normalize them in the Service constructor, no?
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/gcloud-node/issues/1092#issuecomment-176248133
.

Service.call(...) only gets called once IIRC - after a service class as been instantiated.. right?

Using this as an example: https://github.com/GoogleCloudPlatform/gcloud-node/blob/30817fcc10195da0136baefeb38b225986fb1c7e/lib/storage/index.js#L87-L90
if (!(this instanceof Storage)) {
  options = util.normalizeArguments(this, options);
  return new Storage(options);
}

Why do we only normalize if it's not an instance of Storage?

Related: https://github.com/GoogleCloudPlatform/gcloud-node/pull/845#discussion_r38926943

Why do we only normalize if it's not an instance of Storage?

If the current context is not an instance of a service (e.g. Storage), that implies that the context is probably gcloud, which is where we store global config.

So I'm wondering, since we only normalize under that condition, how do we move that logic to the Service constructor?

So I'm wondering, since we only normalize under that condition, how do we move that logic to the Service constructor?

Ahh right, not sure if there's a way around that.. we might have to re-think things a little bit if we want to try and refactor that logic into the Service class.

Should we just leave it?

Might be a good idea for now?


feat: implement stream.onAbort

ref: #1770
Author should do the followings, if applicable

[x] Add tests
[ ] Run tests
[x] yarn denoify to generate files for Deno

Hi, @yusukebe
Here is the POC as of now for now.
I tried to implement it with as few destructive changes as possible, but it looks like I need to change the StreamAPI a bit anyway.
The fact that the test is failing shows that there is a lot of difference in runtime.

@sor4chi
Thanks! I'll check this later!

Hi @sor4chi
This is a great work! It works fine on Node.js and Deno.
One thing you need to consider is that this behavior is changed from the current release version.
The code:
app.get('/stream', (c) => {
  return c.streamText(async (stream) => {
    for (;;) {
      await stream.writeln('Hello')
      console.log('Hello')
      await stream.sleep(1000)
    }
  })
})

The current version:
https://github.com/honojs/hono/assets/10682/60fc7e86-13c3-438a-a41a-587a076ad2e5
This PR:
https://github.com/honojs/hono/assets/10682/a9269705-03ab-4891-91c0-a4779adf41d9
Could you take a look?

@yusukebe
Wow, there was such a bug. I was not aware of that, thank you.
I will certainly have to fix this, but it will take some time.

@sor4chi @yusukebe any updates?

@HeyITGuyFixIt
No, I'm Sorry.
I'm too busy and don't have time to work.

waiting for #1846

Hi, @yusukebe
I have investigated this and I believe that the behavior up to now was a bug in the first place.
Do you have any concerns about this change to a specification that interrupts the stream in the middle?

@sor4chi

Do you have any concerns about this change to a specification that interrupts the stream in the middle?

I do not know. I can look into it myself, but I would prefer to leave it to you.

As for the change that writing to the readable stream is terminated in the middle and no further processing is performed, I would like to proceed as is because the original implementation was poor and I think that changing this behavior in the first place will have little effect on users.

@sor4chi
Okay! Let's go with this spec, which is implemented in this PR.

@sor4chi
Since this referenced the old code, can you create a new PR based on the "next" branch?

@yusukebe
oh, sorry. Okay!

@yusukebe
this is a new pull request. #1871


Add scale keyword argument to mat_decompose

Closes #81

[x] Add test exemplifying problematic roundtrip
[x] Add scale keyword argument to mat_decompose

Would it be sufficient to only provide the signs of the scaling?

Would it be sufficient to only provide the signs of the scaling?

You are right. I've expanded the unit test to cover all possible enumerations of signs, and adjusted the function to only use the signs from the extra keyword argument.

I wonder how we'd deal with cases where e.g. a pygfx both the object and parent have negative scaling signs. To create the world matrix, their matrices are multiplied. Should we separately multiply the scale_signs?
A related question, specific to the code here: should the function check whether the number of flipped axii matches the determinant? And what should it do if it does not match?

I wonder how we'd deal with cases where e.g. a pygfx both the object and parent have negative scaling signs. To create the world matrix, their matrices are multiplied. Should we separately multiply the scale_signs?

I think we're only calling compose and decompose to construct the local matrix using the local SRT components, not on the world matrix. So that shouldn't be an issue.

A related question, specific to the code here: should the function check whether the number of flipped axii matches the determinant? And what should it do if it does not match?

I thought about it but I don't think that's something you can glean off the value of the determinant, actually.

I thought about it but I don't think that's something you can glean off the value of the determinant, actually.

Would it not be something like, if the det is smaller than zero, that the number of negative axii should be uneven?

I've added the validation as far as I think we can provide it, and an accompanying unit test.


Replace use of deprecated Liquibase#update(Contexts) method in H2LiquibaseExtension

Liquibase has deprecated a bunch of the update methods (as well as many rollback methods). We need to find a replacement.
Note that this indirectly affects PostgresLiquibaseTestExtension, because it uses the PreparedDbExtension from the Zonky embedded-postgres library, which uses the same deprecated Liquibase#update method in the LiquibasePreparer class.
I tried using a few of the non-deprecated update methods and they didn't work, because it seems they did not actually perform the database update! e.g. I tried update(Contexts contexts, Writer output) and update(Contexts contexts, LabelExpression labelExpression, Writer output). The documentation is non-existent on the Liquibase#update methods that are not deprecated, i.e. they have no javadocs. And, the deprecated methods don't tell you anything about what the replacement is, so this is going to take some work.

A few references I found:

4.21.1 is out, deprecates Liquibase.update()


02-first-java

Closes # 65
📑 Description
Made a PR because the assignment problems in 02-first-java of DSA-Java Bootcamp doesn't matches with kunal kushwaha's DSA-Java Bootcamp.

[x] Completed: Added current files that matches Kunal kushwaha's DSA-Java Bootcamp.

✅ Checks

[x] My pull request adheres to the code style of this project
[x] My code requires changes to the documentation
[x] I have updated the documentation as required
[x] All the tests have passed

ℹ Additional Information

Am given Question name as the file name.
Happy to Contribute. Hope this will help the folks.
Thank You!

Thanks for merging my PR. Looking forward to contribute a lot.
Thank you1


Container.remove

Let's implement this

The job #117 assigned to @amihaiemil/z, here is why; the budget is 30 minutes, see §4; please, read §8 and §9; if the task is not clear, read this and this

Job gh:amihaiemil/docker-java-api#117 already assigned to @amihaiemil, can't assign to @amihaiemil

Bug was reported, see §29: +15 point(s) just awarded to @amihaiemil/z

Manual assignment of issues is discouraged, see §19: -5 point(s) just awarded to @amihaiemil/z

It is strongly discouraged to assign jobs to their creators, see §19: -15 point(s) just awarded to @amihaiemil/z

@amihaiemil/z this job was assigned to you 5days ago. It will be taken away from you soon, unless you close it, see §8. Read this and this, please.

There is an unrecoverable failure on my side. Please, submit it here:
PID: 4@7a3df257-d8f7-40c4-a574-2a8490a16366, thread: pool-468-thread-1
com.amazonaws.http.AmazonHttpClient$RequestExecutor[1113] com.amazonaws.SdkClientException: Unable to execute HTTP request: null
sun.nio.ch.FileChannelImpl[110] java.nio.channels.ClosedChannelException: null

1.0-SNAPSHOT: Issue: amihaiemil/docker-java-api#117, Action: closed

Order was finished: +30 point(s) just awarded to @amihaiemil/z

The job #117 is now out of scope


[Request External] codeunit 386 "Reverse Payment Rec. Journal" - ErrorIfEntryIsNotReversable

We're creating a custom copy of the Customer / Vendor Ledger entries pages, including all actions. However, we can't copy the 'Reverse Transaction' action, due to restricted access to codeunit 386 "Reverse Payment Rec. Journal"
 Access = Internal;
Can you please make the codeunit public, or at least expose ErrorIfEntryIsNotReversable?
Sample code from page 29 "Vendor Ledger Entries"
action(ReverseTransaction)
                {
                    ApplicationArea = Basic, Suite;
                    Caption = 'Reverse Transaction';
                    Ellipsis = true;
                    Image = ReverseRegister;
                    Scope = Repeater;
                    ToolTip = 'Reverse an erroneous vendor ledger entry.';

                    trigger OnAction()
                    var
                        ReversalEntry: Record "Reversal Entry";
                        ReversePaymentRec: Codeunit "Reverse Payment Rec. Journal";
                    begin
                        ReversePaymentRec.ErrorIfEntryIsNotReversable(Rec);
                        ReversalEntry.ReverseTransaction("Transaction No.");
                    end;
                }

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


Preserve line breaks in TextArea on Summary and Pdf

Description
DSB wants to display text in TextArea without removing line breaks, on Summary page and PDF pages. Currently, line breaks are being removed, and text lines are "merged" on Summary and PDF pages.
Additional Information
No response

Fixed by https://github.com/Altinn/app-frontend-react/issues/2174


Fitting & clustering significant features color_exp

Hello world!
I've starting applying the scFates algorithm to my analysis.
I would like to know if it is possibile to change the color of cells in the function of Fitting & clustering significant features (scf.pl.single_trend) according to the Leiden cluster or based on the pseudotime color (so not just a single color).
In the function there is the colo_rexp option but it does not allow me to assigne to the cells a color code of an observation in adata.obs.
Is it possible to do so?
Thanks so much!!

Sorry for the quite late answer.
There are no easy way to modify the code to allow that. However, here is one trick that can be used to display any color from obs:
ax1,ax2=scf.pl.single_trend(adata,'gene',alpha_expr=0,show=False)
sc.pl.scatter(adata,x='t',y='gene',color='clusters',ax=ax2,show=False,size=10,
              alpha=0.3,legend_loc='none')

Here we set the alpha of the initial scatter plot to 0, and plot on top of it using sc.pl.scatter


Broken issue link (typo)

When I log in, I am presented with a link to blockstack/ux#570 but it fails with a 404. The problem is the target contains /issue/ when it should be should be /issues/

https://github.com/blockstack/ux/issue/570

The issue was solved and the message has been removed. Thanks for trying speed-spend. I am happy to send you some hodl tokens.


updated dependencies, for angular its bugged > ~1.5

PR: https://github.com/byteball/byteball/pull/198
Status
READY
What does this PR accomplish?
updated dependencies, for angular its bugged > ~1.5
so angular and angular-touch in bower is limited to ~1.5
Where should the reviewer start?
execute command:
yarn upd
How should this be manually tested?
yarn upd
yarn sqfix #fix sqlite folder dependency name
yarn  grunt
yarn nw
Any background context you want to provide?
should have latest dependency
Questions:
why?
uses latest dependency; but still dependent to bower

Please do not delete .gitignore and check the files you are adding.


3.2.4 PR + template name change

Issue #, if available:
Description of changes:
Changes to the template name, the license naming and support for 3.2.4.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@bkukoyi - Thanks for the PR. We typically keep the .template.yaml filename extension as that makes it easier for some IDEs to recognize these as a CloudFormation template as opposed to a generic YAML file. Is there a reason you want to rename the file to just .yaml instead of .template.yaml. We can keep just .yaml as well, but just wanted to make you aware of why they had the .template.yaml extension.
Additionally, you'll also need to change this line in .taskcat.yml to point to the renamed template for the tests to work.
Lastly, could you attach output of a successful test run? Many thanks!

I see. I'll update the names and let our PM know. Thanks for pointing that out.

Will run the taskcat and attach the output shortly.

@vsnyc This is ready for review

@vsnyc Just checking in again/ Would you mid going over the PR now?

Thanks @bkukoyi - looks great!


Redshift Multi-AZ cannot use STL scan

Describe the bug
Redshift Multi-AZ cannot use STL scan
version: v0.13.2
To Reproduce
Steps to reproduce the behavior:

Go to Redshift Query in query editor v2
Execute this command

SELECT * FROM stl_insert LIMIT 10;

And you will get

ERROR: permission denied for relation stl_insert [ErrorId: 1-67402a42-124aa79d44733fe95b73d753]

Expected behavior
It should be queried successfully.
Or could you please help to guide how to modify it in the source code? EX: Using other SYS_* views to modify?
Screenshots

Desktop (please complete the following information):
Chrome browser
Additional context
This issue will impact the Redshift ingestion feature.
You can find the AWS official document here https://docs.aws.amazon.com/redshift/latest/mgmt/overview-multi-az.html and which mentioned You can't use STL, SVCS, SVL, SVV, STV views with Multi-AZ deployments as they only support system monitoring views (SYS_* views). Change your monitoring queries to use system monitoring views (SYS_* views).

The version is v0.13.2

@RyanHolstien do you have any other ideas for adjusting it in advance? Due to we are currently using it and we need the data lineage for Redshift.

@treff7es Thanks for your solution. is_serverless can fix this issue. 👍

It works with the is_serverless solution


How to Embedding Dashboard  Apache Superset to our React app?

Please make sure you are familiar with the SIP process documented
(here)[https://github.com/apache/superset/issues/5602]. The SIP will be numbered by a committer upon acceptance.
##How to Embedding Dashboard  Apache Superset to our React app?
need to Implement embedding dashboard apache superset to external applications.
Description of the problem to be solved.

Hello, this is not a bug report and certainly not a SIP.  I will close this accordingly.
I saw you posted in Slack, that's a good place to ask questions like this although this is so broad that your best bet is to search the web for guides on how to do this.  When I type "embed superset dashboard" into Google the first page of search results look promising.  Good luck and feel free to post in Slack with more specific questions, there is a channel just for embedded Superset.


CategoricalDistribution sampling returns incorrect type

If your issue is a request for support in using Optuna, please post it on Stack Overflow.
Support is also available on #general channel of chainer slack and chainer-jp slack.
On registering an issue, write precise explanations on how you want Optuna to be.
Bug reports must include necessary and sufficient conditions to reproduce the bugs.
More specifically, it is helpful for us if you include the following information:
Conditions

Optuna version: 0.13.0
Python version: 3.6.8
OS: Ubuntu
Machine Learning library to be optimized: rl-baselines-zoo

Code to reproduce
# python code
import optuna
study = optuna.create_study(sampler=optuna.samplers.random)
possible_vals = optuna.distributions.CategoricalDistribution(choices=('auto', 1., 2., 3., 4.))
sampled_val = study.sampler.RandomSampler().sample_independent(study, 'my_trial', 'my_param', possible_vals)

print(sampled_val, type(sampled_val)) # returns numpy.str_ even when a float could be sampled

Error messages, stack traces, or logs
# stack traces

Problem code
optuna.samplers.random.py::sample_independent() has the offending line
        elif isinstance(param_distribution, distributions.CategoricalDistribution):
            choices = param_distribution.choices
            return self.rng.choice(choices) # should be choices[self.rng.choice(len(choices))]

Thank you for your time!

Thank you for reporting this problem.
I think this is the same problem with #448 and has been resolved by #449 .
The next release of Optuna will include the patch (the next version will be released within a few weeks).

OK I see that. I'll close this issue.


subscribe_to(...).on(...) is misleading

From the documentation:

subscribes
Assert that a resource subscribes to another in the Chef run
resource = chef_run.service('apache2')
expect(resource).to subscribe_to('template[/etc/foo]').on(:create).delayed

In this example, the wording makes it seem that we are testing that service[apache2] is being notified when template[/etc/foo] executes action :create.  However, what this example actually does is test that a notification is triggering service[apache2] to execute the action :create when the resource template[/etc/foo] is updated.
The error output from Chefspec makes this clear.  Here is an example from my own Chefspec run, where I am attempting to verify that the execute[apache status] resource is run after an apache reload:
     Failure/Error: expect(resource).to subscribe_to('service[apache2]').on(:reload).immediately

       expected "service[apache2]" to notify "execute[apache status]" with action :reload immediately, but did not.

       Other notifications were:

           "[service[apache2]]" notifies "execute[apache status]" to :run, :immediately

I think the example in the documentation should be updated to be a little more clear. For instance:

subscribes
Assert that a resource subscribes to another in the Chef run
resource = chef_run.service('apache2')
expect(resource).to subscribe_to('template[/etc/foo]').on(:restart).delayed

This would make more sense, since it's a little strange to "create" an apache2 service after a template is updated.  It's more likely that recipe authors would restart or reload the service instead.
Even better, perhaps it would be a good idea to add a new method as an alias to "on" in order to clarify the meaning of the method, for instance to (to align with the notify matcher) or with_action (which would be more accurate):

resource = chef_run.service('apache2')
expect(resource).to subscribe_to('template[/etc/foo]').with_action(:restart).delayed

Hi @Poohblah
Notifications are typically confusing because of their inverse nature. For this reason, I would recommend using notifications instead of subscriptions.


Fix translatable

Resolves #2152 while keeping behavior described in #1709

I'm sorry, I was targeting master first and it required 7.2, now I check 2.4.x branch and it looks like 5.3 is minimum dependency. While change in class itself is no problem, tests are obviously not 5.3 friendly. Will fix it

Great, now works :). I can also provide separate PR for master branch if you find this valuable

Much obliged. 🥇

No worries on the separate PR, I'll get it merged upstream. The tests in general need a ton of updating, so no sense in having one perfect one. 😂


WARNING - Invalid identifier:GORULE:0000027: 1298204 does not match any id_syntax patterns for MGI in dbxrefs--PomBase SPBC530.12c pdf1 enables GO:0008474 PMID:15075260 ISO MGI:MGI:1298204 F palmitoyl protein thioesterase/ dolichol pyrophosphate phosphatase fusion protein Pdf1 protein taxon:4896 20040414 PomBase
WARNING - Invalid identifier:GORULE:0000027: 1316717 does not match any id_syntax patterns for MGI in dbxrefs--PomBase SPBC20F10.03 SPBC20F10.03 is_active_in GO:0005634 GO_REF:0000024 ISS MGI:MGI:1316717 C armadillo-type fold protein, human IFRD1 ortholog, implicated in transcription or signaling protein taxon:4896 20170830 PomBase
WARNING - Invalid identifier:GORULE:0000027: 1346084 does not match any id_syntax patterns for MGI in dbxrefs--PomBase SPAC6C3.09 rpp40 part_of GO:0005655 GO_REF:0000024 ISS MGI:MGI:1346084 C RNase P and RNase MRP subunit Rpp40 protein taxon:4896 20061017 PomBase
WARNING - Invalid identifier:GORULE:0000027: 1919005 does not match any id_syntax patterns for MGI in dbxrefs--PomBase SPAC513.06c dhd1 involved_in GO:0042843 GO_REF:0000024 ISS MGI:MGI:1919005 P D-xylose 1-dehydrogenase (NADP+) protein taxon:4896 20150502 PomBase
WARNING - Invalid identifier:GORULE:0000027: 1919005 does not match any id_syntax patterns for MGI in dbxrefs--PomBase SPAC513.06c dhd1 enables GO:0047837 GO_REF:0000024 ISS MGI:MGI:1919005 F D-xylose 1-dehydrogenase (NADP+) protein taxon:4896 20150502 PomBase
but its a bit weird because the display and the URL are MGI:1298204 but on the pop-up it says MGI:1298204
could you have a dig and see if the syntax has been resolved to remove the first MGI: or something?

I can't see anything that's changed.  We get the MGI IDs from the GOA GAF and they look like "MGI:MGI:1919005".
It's a bit weird.  We have "MGI:MGI:1919005" in the GAF file but the error is:
1919005 does not match any id_syntax patterns for MGI

It's like the "MGI:" prefix is being removed twice.
The id_syntax in the db-xrefs file is "MGI:[0-9]{5,}" and that has been the same for a few years.
I think this might be a problem with the GO check.  The issue for the check is still in progress:

https://github.com/geneontology/go-site/issues/2210

@pgaudet   @kltm is this a GO checks problem?
v

Looking at https://github.com/geneontology/go-site/blob/master/metadata/rules/gorule-0000027.md . Okay, "soft" warning, so no data filtering.
The moment of failure is likely here:
https://github.com/biolink/ontobio/blob/master/ontobio/io/assocparser.py#L835
Special casing for MGI  leading into it is:
https://github.com/biolink/ontobio/blob/master/ontobio/io/assocparser.py#L802-L806
So, it looks like MGI:MGI:1919005 would be clipped to MGI and 1919005, the latter of which would fail when checking against the regexp. The options here would be:

change the dbxrefs regexp to reflect our behind-the-scenes fix of MGI (I'm not sure what the knock-on effect would be)
remove the ontobio "fix" (I'm not sure what the knock-on effect would be)
change the MGI full id to MGI:MGI:MGI:1919005 (I know what the knock-on effect would be: hilarity)

Either way, @pgaudet , this is probably best approached as a GO QC bug for the moment (although a "light" one as no fix or filtering is done) and added to the QC worklist.

@pgaudet @kltm  I'm closign this on the PomBAse tracker. It should be in the GO tracker if it's still an issue?


Unbounded memory usage (over 16GB) on a crafted image, bypassing allocation tracking

Decoding this image causes a steadily increasing memory usage: oom-97a147dae72d761a921774a317f89d1d799654ff.jxl.gz
I have killed it the process once it consumed 16GB.
This was found by cargo fuzz using a fuzzing harness identical to the in-tree one, so it sets .alloc_tracker(AllocTracker::with_limit(128 * 1024 * 1024)) // 128 MiB which appears to be ineffective.
Tested on commit b1115f987e2ca269ca903e6726e04a74db703438

Maybe some missing checks are causing a loop somewhere? Will take a look with #163 later today.


Continuous Pulse animation

Hi,
Is it possible to add the functionality that the last value animation is showing continuously the pulse animation?

I guess you need to set up lastPriceAnimation to 1 or LastPriceAnimationMode.Continuous.
https://tradingview.github.io/lightweight-charts/docs/api/enums/LastPriceAnimationMode#continuous


Use createLazyFile to mount a read only view of remote sqlite databases

This is more POC/WIP/curiosity than anything serious. Just wanted to make a draft PR for posterity.

For context, I'm trying to put a 32GB FTS5 sqlite database on the internet to query. I plan to host it on Cloudflare so I do not care about BW costs.
The functionality here is a bit like https://github.com/phiresky/sql.js-httpvfs, but far lazier and inefficient. I originally thought I would have to make my own GUI, but I kind of like the GUI I saw in datasette and wondered if I could reuse it. Unfortunately, current datasette-lite seems to pull everything into memory.
It does work actually, but I don't know how viable this is. Emscripten is by default reads 1MB chunks but it seems to have some way to recompile it to not:
https://github.com/emscripten-core/emscripten/blob/18bc868cb5242e6816a4b3bde74b1e1dcd6fd818/src/library_fs.js#L1713
https://github.com/azavea/loam/issues/75#issuecomment-888515090
Some FTS5 queries take 1GB to query while others may take only 66MB. As I've said, I don't care about BW but the great inefficiency harms UX.
Also, this chunked inefficiency means that I have to hack the URL to not load tables of a database as it seems to try to load the whole database when I click on a database.
I think for my goal, I might have to try to recompile pyodide with a small XHR build of emscripten. 😬
As for any dependencies from the remote URLs, they'll need to have the proper CORS headers set, including "expose" headers. If it can't read expose headers or the remote file is GZIPed, it just downloads the whole file, so it does degrade gracefully. However, it does cause a nasty error message in the console to popup saying something denied access to some headers to Emscripten.
Anyway, this is just for fun.

I also built a fork of some static web file server to also expose the necessary headers too:
https://github.com/joseluisq/static-web-server/pull/144

Wow, I did not think this would be possible without completely replacing the Python sqlite3 module!

The experimental tuned extracted thing explodes on the example databases. Not sure why. Well, that's why this is a draft after all :D.

Also, this chunked inefficiency means that I have to hack the URL to not load tables of a database as it seems to try to load the whole database when I click on a database.

I bet that's because Datasette tries to show a count of all of the rows in each table when idt shows the list on that page, which triggers a full table scan.
Would be great to have a setting that turns that feature off, which could then be exposed as a query string option for Datasette Lite.

https://github.com/phiresky/sql.js-httpvfs/blob/master/src/lazyFile.ts
I think it should be possible to adapt this theoretically more efficient dynamic chunk size version to this approach as well.

Oh, I should probably explain what I'm trying to query or do. I like looking at CA unclaimed property records with SQLite since it's a much more powerful and faster full text search than what's offered on the government site:
https://www.sco.ca.gov/upd_download_property_records.html
It's about 35GB when imported into a FTS5 table, optimized, and vacuumed. I was thinking of trying to expose a similar service with something like https://github.com/wilsonzlin/edgesearch but it was a little too foreign, a little way too tied to one vendor's offering and architecture, and not SQLite.

https://datasette-lite.mindflakes.com/index.html?url=https://datasette-lite.mindflakes.com/odb.sqlite#/odb?sql=SELECT+*+FROM+records+WHERE+records+MATCH+"donald+trump"+ORDER+BY+CAST(CURRENT_CASH_BALANCE+AS+FLOAT)++DESC%3B
The performance kind of leaves a lot to be desired still. 50 seconds for that query. Still, it's pretty neat!

Oh wow I didn't realize you were already aware of this stuff:
https://twitter.com/simonw/status/1421497663732604928
I think we could just use "sql.js-httpvfs" implementation's of createLazyFile as-is and we should be good to go actually on using it with datasette-lite.

https://datasette-lite.mindflakes.com/index.html?url=https://datasette-lite.mindflakes.com/odb.sqlite#/odb?sql=SELECT+*+FROM+records+WHERE+records.owner_name+MATCH+"donald+trump"+ORDER+BY+CAST(CURRENT_CASH_BALANCE+AS+FLOAT)+DESC%3B
This now takes about 30 seconds down from 50 seconds using https://github.com/segfall/static-wiki/tree/master/scripts for database optimization and ideas/thoughts and https://github.com/phiresky/sql.js-httpvfs/blob/master/src/lazyFile.ts for the optimized adjacent pages read lazyfile implementation.

So this is all still research and a POC PR that might not be merged in the end but I have some ideas to keep following up on when time permits that might be interesting for other's future proper PRs/RFCs:

It doesn't seem possible at the moment to have a manifest. I would love to point users at a fatter README or site to describe what's going on in the data set.
What if we pointed datasette-lite at a manifest file of some sort as an alternative to  a sqlite db url in it instead of a DB directly.
What if the sqlite db url in it could also be instead a manifest of split chunks? The underlying rangemapper option of sql.js's lazyfile could be used to handle chunks. An example on generating the necessary split DBs files is in https://github.com/phiresky/sql.js-httpvfs/blob/master/create_db.sh . I think I still want to customize it a bit though, the sql.js-httpvfs's output chunk's filename scheme would not be cached with cloudflare's default caching rules: https://developers.cloudflare.com/cache/about/default-cache-behavior/#default-cached-file-extensions . It's possible to work around the caching choice defaults with page rules but they are very limited in amounts on free Cloudflare plans. I'm super interested in chunking as the current status quo of always missing at the moment incurs a ~300ms cost for any SQLite page read.

Threw on a page rule in CF to see what happens if the sqlite database is forced to be cached.
https://github.com/nelsonjchen/ca_unclaimed_property_db_generator_toolkit/issues/1#issuecomment-1258479652
Unfortunately, I ran into some odd bug where a range request tried to return a 200. All of it, 28GB. At least it tried to before I killed it.
https://community.cloudflare.com/t/bug-partial-requests-for-cached-files-occasionally-results-in-a-200-response-with-the-full-contents/376516
The post is both agonizing and tantalizing since the OP posted a timeline. If Cloudflare gets their stuff together, we could speed up repeated queries and/or cache few levels of popular indirection by as much as ~10x since the hits become 30ms vs 300ms for the current status quo of always missing. It's also agonizing since CF claims the issue was fixed.
Thinking of making a periodic and continuous GitHub repo/actions setup with Playwright to test and validate the bug and running it up to CF engineering.

https://kevincox.ca/2021/06/04/http-range-caching/
Found this great post on caching behavior of ranges in common CDNs.

Made a chunked version as POC test. It is indeed faster!
https://datasette-lite-lab.mindflakes.com/index.html?url=https://datasette-lite-lab.mindflakes.com/sdb/2022-10-02_93eff57de3573985_ca_unclaimed_property.sqlite#/2022-10-02_93eff57de3573985_ca_unclaimed_property?sql=SELECT+*+FROM+records+WHERE+records.owner_name+MATCH+"Elon+Musk"+ORDER+BY+CAST(CURRENT_CASH_BALANCE+AS+FLOAT)++DESC%3B
https://github.com/nelsonjchen/datasette-lite/commit/8eaaf59a4e9f1fa655b5fd45a28df47ffbfbbac2
Something like this definitely needs to have a manifest-like thing though. Upfront, a file size is needed. And I've already expressed my desire for metadata.
4096KB pages, 10MB chunks, ~30ms hits, ~300-500ms misses. Not bad! Cloudflare does seem to expire the cache rather a lot though. At least a page refresh is fast.
On that note. I think I'm going to wind down the experimentation a bit.  Hopefully someone else can use these learnings.
And I have looked you up @simonw . There is something, not a lot and not pocket change, but there is something. Hopefully the CA state controller won't give you too much trouble. You the real MVP!

Note: https://github.com/phiresky/sql.js-httpvfs/issues/40
There's an upstream bug in the lazyFile implementation which I've fixed in my test hack of a lab.

So, with the split database and all that, the cost per month to host and expose this 28GB database with Datasette:
https://developers.cloudflare.com/r2/platform/pricing/
$0.42
The cost can increase if there are a lot of queries but this is extremely negligible.


Component: Event Page Block (design exists)

We need to define, design & scope the types of news post content that will be supported on the DCC cannabis site.

news post, press-release (assumed)
news post, event block  (confirm)
news post, announcement block  (confirm)
any others?

When complete, we need to create issues to scope, design, build out these components.

Let's bring Michael into this discussion since he's keyed into incoming content.

For completeness, sharing the notes from my conversation last week with Christina. I believe these answer these questions, but let me know if I'm missing something that will help keep this moving forward.
(Caveat: I don't know what a "block" in the context of a "news post" is. I assume that these are types of things that can be created within a news section. Let me know if I'm wrong!)

news post, press-release: yes. Christina said that right now press releases will be a major aspect of their communications.
news post, event block: yes. While events are not frequent (at most, once a month), they will need to post them and I see the requirements as different from a news release.
news post, announcement block: yes. This would be really similar to the press release, but just have different headers and footers. This Slack thread includes an example of what they consider a press release and what consider an announcement.
any others? Not for July 1, but if we're putting together a post-July 1 roadmap I can ask Christina. The Google Doc has some

I will confirm these answers with Christina in my end-of-day email to her. She's been pretty good about answering questions like these promptly.

I've repurposed this issue to separate the different concerns. This is now specifically for devs to scope event blocks because we have design for this. I've created a separate issue for design to think through the header/footer differences for press releases vs announcements.

Some updates on this issue:

Needed more definition for the overall publishing system & data handling (well underway)

Also needed to clarify this scope in to phases. This is much more complex, even greatly simplified, to make this work correctly.

We have a simple v1 which is just visual appearance, and I've got a set of data integrations for phase two, which makes the events searchable and filterable for headless. We are very limited in how we can accomplish this and are also deciding patterns for any other content elements that need to operate as entities on the Interwebs, and as sensible data models that can migrate and grow over time.

The UI integrations all depend on and vary greatly with the CMS editor.

Updated the CSS

Finalizing a release to use for the intermediary period.

More updates on this special component:

It's part of a pattern.

Awaiting launch (backed up by headless publishing)

Headless version being vetted

A couple of boards for scoping out the underlying data model

We have a new framework for talking about the publishing system

Have a team talk planned about content models and the design system (for data patterns like this that need general communication & documentation. Will try same publishing system but with an Airtable Google Calendar to make a super simple event flow & be able to use multiple CMS's but still publish same results.

Need tests.

Headless version has other small issues, but shouldn't block just closing this initial request out.

Shipped. Closing this, it's a duplicate of other events issues. - we need a better system to manage "Patterns"  or doing sprints.
We did ship v1 of events.


[SuperEditor][Android] Show expanded handles when expanding the selection (Resolves #1937)

[SuperEditor][Android] Show expanded handles when expanding the selection. Resolves #1937
When the selection changes from collapsed to expanded, the collapsed handle is visible at the beginning of the selection, and the expanded handles aren't visible:
collapsed.webm
This PR fixes the issue by changing the visibility of the handles when needed.

@angelosilvestre please post a video of the solution

expanded.webm

@angelosilvestre this PR now has some conflicts after another PR was merged.
Also, can you take a look at the failing website deployment actions? They seem to be failing in a number of PRs but I'm not sure why.

@matthew-carroll I resolved the conflicts.
The website build failure seems to be due to the LinkAttribution constructor change, where now we expect a String. I already solved this in my website PR, so for new PR's this should fail again.


[tool_crash] UnsupportedError: Unsupported operation: Illegal character in path: assetsF:\PROJECTS\ham-hub\assets\font

Command
flutter --quiet assemble --no-version-check --depfile F:\PROJECTS\ham-hub\build\app\intermediates\flutter\debug/flutter_build.d --output F:\PROJECTS\ham-hub\build\app\intermediates\flutter\debug -dTargetFile=F:\PROJECTS\ham-hub\lib\main.dart -dTargetPlatform=android -dBuildMode=debug -dTrackWidgetCreation=true --DartDefines=Zmx1dHRlci5pbnNwZWN0b3Iuc3RydWN0dXJlZEVycm9ycz10cnVl,RkxVVFRFUl9XRUJfQVVUT19ERVRFQ1Q9dHJ1ZQ== debug_android_application

Steps to Reproduce

...
...
...

Logs
UnsupportedError: Unsupported operation: Illegal character in path: assetsF:\PROJECTS\ham-hub\assets\font
#0      _Uri._checkWindowsPathReservedCharacters (dart:core/uri.dart:1829:11)
#1      _Uri._toWindowsFilePath (dart:core/uri.dart:2863:7)
#2      _Uri.toFilePath (dart:core/uri.dart:2835:38)
#3      ManifestAssetBundle._parseAssetsFromFolder (package:flutter_tools/src/asset.dart:744:29)
#4      ManifestAssetBundle._parseAssets (package:flutter_tools/src/asset.dart:685:9)
#5      ManifestAssetBundle.build (package:flutter_tools/src/asset.dart:245:54)
<asynchronous suspension>
#6      copyAssets (package:flutter_tools/src/build_system/targets/assets.dart:50:26)
<asynchronous suspension>
#7      AndroidAssetBundle.build (package:flutter_tools/src/build_system/targets/android.dart:64:34)
<asynchronous suspension>
#8      _BuildInstance._invokeInternal (package:flutter_tools/src/build_system/build_system.dart:839:9)
<asynchronous suspension>
#9      FlutterBuildSystem.build (package:flutter_tools/src/build_system/build_system.dart:606:16)
<asynchronous suspension>
#10     AssembleCommand.runCommand (package:flutter_tools/src/commands/assemble.dart:320:32)
<asynchronous suspension>
#11     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1183:27)
<asynchronous suspension>
#12     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#13     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#14     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
<asynchronous suspension>

[✓] Flutter (Channel stable, 3.0.1, on Microsoft Windows [Version 10.0.25120.1010], locale en-US)
    • Flutter version 3.0.1 at C:\flutter\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision fb57da5f94 (3 weeks ago), 2022-05-19 15:50:29 -0700
    • Engine revision caaafc5604
    • Dart version 2.17.1
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0)
    • Android SDK at C:\Users\mhdss\AppData\Local\Android\sdk
    • Platform android-32, build-tools 32.0.0
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[✗] Visual Studio - develop for Windows
    ✗ Visual Studio not installed; this is necessary for Windows development.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[✓] Android Studio (version 2021.2)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+7-b1504.28-7817840)

[✓] VS Code (version 1.67.2)
    • VS Code at C:\Users\mhdss\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension can be installed from:
      🔨 https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter

[✓] Connected device (4 available)
    • Redmi 9T Note 9 4G 9 Power (mobile) • 20f65b500321 • android-arm64  • Android 11 (API 30)
    • Windows (desktop)                   • windows      • windows-x64    • Microsoft Windows [Version 10.0.25120.1010]
    • Chrome (web)                        • chrome       • web-javascript • Google Chrome 102.0.5005.63
    • Edge (web)                          • edge         • web-javascript • Microsoft Edge 101.0.1210.53

[✓] HTTP Host Availability
    • All required HTTP hosts are available

! Doctor found issues in 1 category.

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: fb57da5f945d02ef4f98dfd9409a72b7cce74268
Plugins
file_picker-4.6.1
firebase_auth-3.3.19
firebase_auth_web-3.3.16
firebase_core-1.17.1
firebase_core_web-1.6.4
firebase_database-9.0.15
firebase_database_web-0.2.0+14
flutter_plugin_android_lifecycle-2.0.6
fluttertoast-8.0.9
google_sign_in-5.3.2
google_sign_in_android-5.2.8
google_sign_in_ios-5.3.1
google_sign_in_web-0.10.1+2
image_picker-0.8.5+3
image_picker_android-0.8.5
image_picker_for_web-2.1.8
image_picker_ios-0.8.5+5

Hi @vectorcrop
As I see your project name ham-hub is an invalid name which could be a reason you are getting the error. Try to rename it and see whether the error is fixed.
Closing this for now. If it's not fixed, write in comments and I'll reopen it.


Ethan: Blacklist 2023[\W_]NCAA[\W_]Tournament

Ethan requests the blacklist of the keyword 2023[\W_]NCAA[\W_]Tournament. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

2023[\W_]NCAA[\W_]Tournament has been seen in 9 true positives, 0 false positives, and 0 NAAs.

Approved by Spevacus in Charcoal HQ


debug complexity: is also -> is not

Nice essay :+1:
I might be off on this one, but I think maybe you meant "not" here?

Thanks for fixing that missing not, @fuzetsu !


SAN init_path using

Which project are you using?
SAN
Issue description
I'm trying to evaluate your models through san_eval.py file and I'm converting evaluate function to kind of API endpoint to be able to use it as a client.
I wonder why do you use sys.path patching instead of simply importing modules in from lib import models fashion?
For me this is an issue cause it affects my runtime if I use your code.
Could you (or me in PR) fix imports to preserve sys.path untouched?
By the way I could provide PR for an API I've wrote if you are interested in it.

Hi, it is because from lib import models require a python file in the same dir as lib.
Without sys.path it is fine in SAN. While my other projects use sys.path, so I'm used to using it.
You are welcome to create a PR for API. Thanks a lot for your effort.


Glob regex is inefficient

glob.match('**/bower.json',...) results in:
/^(?:[\/\\]|[^\/\\]+[\/\\]|[\/\\][^\/\\]+)*?bower\.json$/  which looks rather complicated.
It seems to work but runs in a timeout on a long content (I tried with the default content of http://regexr.com/)
So its a ('separator' or 'everything but a separator followed by a separator' or a 'separator followed by a everything but a separator') repeated and optional. A very complicated way of saying  /bower.json$/

@aeschli good catch Martin, my regex ended up to do Catastrophic Backtracking (http://www.regular-expressions.info/catastrophic.html).
I pushed a change to simplify it. A "**" is now anything that starts with a path or ends with a path.

This caused a severe performance regression. Have to really add performance tests for the pattern we use to not get into this again...

This is the cause for  #5539

@aeschli pushed a simplification for the regex. we still need to test for more conditions than simply testing for .*. For example **/.html must not match on foo/test.html, but it would if ** would simply translate to .*

"**/project.json" -> "(?:[/\].*?|.+?[/\]|)project.json"  This matches /rrproject.json...
I created #6936

Reopening because the change makes the glob pattern match too much.

VSCode can hang and crash on this glob regex when it tries to select a language (languageSelector.ts) on not-so-extreme conditions:
if I have a file at C:\aaaa\aaaaa\aaaaaaa\aaaaa\aaaaaaaa\aaaaaa\aaaaaaaaaaa\aaa\aaaaaaaaaaaaa\aaaaaa\aaaaaaaaaaa and I open it by pasting the following in the Ctrl+P palette:
C:\\aaaa\\aaaaa\\aaaaaaa\\aaaaa\\aaaaaaaa\\aaaaaa\\aaaaaaaaaaa\\aaa\\aaaaaaaaaaaaa\\aaaaaa\\aaaaaaaaaaa
(same path but with double backslashes)
This happened to me by accident, so this issue affects real world use cases and should be fixed.

@NotWearingPants Can you attach a profile so we know it's this or another issue? Please use this guide to provide more information: https://github.com/Microsoft/vscode/wiki/Performance-Issues. Thanks.

@jrieken
I have debugged vscodethe devtools and saw it was stuck in the regex.test(filepath) inside the glob functionality called by languageSelector.ts.
I was also able to reproduce this in node.js using just regex.test(filepath) with the same regex and filepath.
I don't see a reason to attach a profile if I honed in on the problem. I might upload one later, not sure.


Add Sphinx Wagtail Theme

This PR adds Sphinx Wagtail Theme.
Preview: https://wagtail--6930.org.readthedocs.build/en/6930/index.html

The theme is pinned in setup.py AND docs/requirements.txt (RTD doesn't read the setup.py)
The title reads 'Wagtail Documentation' instead of 'Wagtail'
Copyright is 'YYYY, Torchbox and contributors' instead of 'YYYY, Torchbox' (like the licence)
Searchbox template is updated to reflect the theme (Bootstrap) markup
Sphinx disables the TOC on pages that have a single heading. We added :hidetoc: 1 to disable the TOC on pages with multiple headings, but where a TOC is still redundant.

Most work is done on the theme and according to the design prototype
The new theme:

is based on Bootstrap
is responsive
takes 100% of the viewport
has a table of contents
does not reload the page when open/close main menu items

I expect the commits to be squashed on merge.

Do the tests still pass? (https://docs.wagtail.io/en/latest/contributing/developing.html#testing) YES
Does the code comply with the style guide? (Run make lint from the Wagtail root) YES
For Python changes: Have you added tests to cover the new/fixed behaviour? N/A
For front-end changes: Did you test on all of Wagtail’s supported browsers? Please list the exact versions you tested.

Edge 89
IE 11 (search doesn't work) Note the latest docs also doesn't work on IE11.
Chrome  90.0.4430.72 OSX
FireFox 86 OSX
FireFox 87.0 OSX
IPad Pro 12.9

For new features: Has the documentation been updated accordingly? N/A

https://wagtail--6930.org.readthedocs.build/en/6930/index.html

[ ] Fix override search
[ ] Drop banner.js

The version picker at the bottom appears broken. Maybe some of the removed theme css was injected into it?

[ ] Update project name in conf.py

One small nitpick, but apart from that this looks very complete to me! 😄
Nitpick: I think we should make it more clear that the version in the footer is the version of sphinx_wagtail_theme. Not the version of Wagtail itself.

Hm, I feed like a reference to the theme in the docs would be odd. I think we are better off just removing that and only showing the project name or nothing in the footer.

@tbrlpld I think a reference to the version of the documentation is useful for us (wagtail developers) in case we need to publish a new version of the theme.

I changed the line Wagtail Documentation 5.0.0 into Sphinx Wagtail Theme 5.0.1 and set display none.
Updated this pr.

Re-reviewing this I can confirm there is something funny going on with the search. It looks like our custom Algolia search isn’t completely overriding the built-in one, which causes both search issues I reported above. I think this should be fixed before we merge. Other than that, the "Edit on GitHub" link should be easy to change but I can’t figure out where to set it (cc @allcaps).
Theme default we want to override:
github_url = https://github.com/wagtail/sphinx_wagtail_theme/blob/main/docs/

Looks like all feedback has been implemented. I'm going to squash and merge this now.


some chars in autogenerated passwords not accepted by login page

Some characters in the autogenerated password aren't accepted by the datalake login page, meaning new users can't log in.  So far, at least the '?' char.

@jgc234 I was able to login with a password containing the special character '?'.  There are other restrictions on password which also might have caused the failure. Can you post a sample password you tried that did not work?

Hi @jgc234, please let us know if you still face the issue. If so can you please provide the special characters causing the issue. Also are you using any Federated logins through ADFS or any other IDP provider.


Python 3 fix and support for saving object data to a stream.

This patch fixes one bug and adds one new feature.
Bug: The output file should be opened in 'wb' mode when using Python 3 or else you'll get the following on the out.write() call:
TypeError: must be str, not bytes
Feature: It would be useful to be able to specify a stream instead of a simple path, something like the requests library 'data' parameter.

Thank you for contributing to openstack/python-openstacksdk!
openstack/python-openstacksdk uses Gerrit for code review.
Please visit
http://docs.openstack.org/infra/manual/developers.html#development-workflow
and follow the instructions there to upload your change to Gerrit.


Doesn't work with Firefox Electrolysis (Nightly)

I don't know any more than that yet.

Also seeing this a lot:

The following error message is printed repeatedly when using electrolysis and debugging addons:
unsafe/forbidden CPOW usage                                     RemoteAddonsParent.jsm:872:2
unsafe/forbidden CPOW usage                                      markdown-here.js:80:4
unsafe/forbidden CPOW usage                                      markdown-here.js:79:9
unsafe/forbidden CPOW usage                                      markdown-here.js:86

See: https://developer.mozilla.org/en-US/Firefox/Multiprocess_Firefox/Cross_Process_Object_Wrappers

It's finally time to close this. A WebExtensions MDH has been released, which is compatible with Electrolysis.
https://addons.mozilla.org/en-US/firefox/addon/markdown-here/
See #433 for code changes.


Check voice over for copy code block

After clicking on the code copy button, voice over reads everything. I need to find a way to prevent that.

Checked it on brave with 1.10.0. Seems to have been only temporary and does not exist with latest versions of the theme.


Move interval column to a non-reserved word: interval_json

Deploy was successful so no action needed there.

/lgtm