MySQL 5.7 compatibility

https://github.com/yiisoft/yii2/blob/f9fd4da8de6c0272cd8956e94da1656b41f1416c/framework/db/mysql/QueryBuilder.php#L41
What steps will reproduce the problem?
Run tests on MySQL 5.7
What is the expected result?
Passed tests
What do you get instead?
1) yiiunit\framework\db\mysql\QueryBuilderTest::testCreateTableColumnTypes
yii\db\Exception: SQLSTATE[42000]: Syntax error or access violation: 1067 Invalid default value for 'col59'
The SQL being executed was: CREATE TABLE `column_type_table` (
...
	`col59` timestamp,
...
PDOException: SQLSTATE[42000]: Syntax error or access violation: 1067 Invalid default value for 'col59'

Additional info
https://stackoverflow.com/questions/9192027/invalid-default-value-for-create-date-timestamp-field
@sergeymakinen

We can set ALLOW_INVALID_DATES for MySQL in test environment.
Developers will resolve this issue on their own.

I would rather replace these tests and use ->timestamp()->null() instead just ->timestamp(), because this is the way how people should use it.
I'm also for calling null() implicitly for timestamp() in 2.1. Right now you need to always use ->timestamp()->null() in your migrations because default value for timestamp may differ depending on DBMS you're using - ->timestamp() will work like ->timestamp()->notNull() for MySQL, but in PostgreSQL it will work like ->timestamp()->null(). Using null() by default improve consistency of ColumnSchemaBuilder and allow to avoid weird mistakes.

Issue was also spotted here https://github.com/yiisoft/yii2/issues/15248#issuecomment-348014962

For the record, this is no issue with:

MariaDB 5.5 and 10.0
Percona 5.7

But we need to check if this is related:

MariaDB 10.3

There was 1 failure:

1) yiiunit\framework\db\mysql\SchemaTest::testColumnSchema
defaultValue of column ts_default is expected to be an object but it is not.
Failed asserting that 'current_timestamp()' is of type "object".

/project/tests/framework/db/SchemaTest.php:485
/project/vendor/phpunit/phpunit/phpunit:52

It's interesting that travis builds work now thanks to the mode NO_ZERO_IN_DATE
I draw your attention that modes NO_ZERO_IN_DATE and NO_ZERO_DATE are different.
Travis sets it here: https://travis-ci.org/yiisoft/yii2/jobs/358090431#L961
Check out the results of my local tests:
MySQL: 5.7.21
Yii2 branch: master
1) First test:
Commands:

sudo service mysql restart
sudo mysql -uroot -proot <<< "SET GLOBAL sql_mode = 'NO_ZERO_IN_DATE';"
php vendor/bin/phpunit --group=mysql

Result:
OK, but incomplete, skipped, or risky tests!
Tests: 706, Assertions: 3251, Skipped: 7.

2) Second test:
Commands:

sudo service mysql restart
php vendor/bin/phpunit --group=mysql

Result:
There was 1 error:

1) yiiunit\framework\db\mysql\QueryBuilderTest::testCreateTableColumnTypes
yii\db\Exception: SQLSTATE[42000]: Syntax error or access violation: 1067 Invalid default value for 'col62'
The SQL being executed was: CREATE TABLE `column_type_table` (
	`col1` bigint(20),
	`col2` bigint(20) NOT NULL,
	`col3` bigint(20) CHECK (`col3` > 5),
	`col4` bigint(8),
	`col5` bigint(8) CHECK (`col5` > 5),
	`col6` blob,
	`col7` tinyint(1) NOT NULL DEFAULT 1,
	`col8` tinyint(1),
	`col9` char(1) CHECK (`col9` LIKE "test%"),
	`col10` char(1) NOT NULL,
	`col11` char(6) CHECK (`col11` LIKE "test%"),
	`col12` char(6),
	`col13` char(1),
	`col14` date NOT NULL,
	`col15` date,
	`col16` datetime NOT NULL,
	`col17` datetime,
	`col18` decimal(10,0) CHECK (`col18` > 5.6),
	`col19` decimal(10,0) NOT NULL,
	`col20` decimal(12,4) CHECK (`col20` > 5.6),
	`col21` decimal(12,4),
	`col22` decimal(10,0),
	`col23` double CHECK (`col23` > 5.6),
	`col24` double NOT NULL,
	`col25` double CHECK (`col25` > 5.6),
	`col26` double,
	`col27` double,
	`col28` float CHECK (`col28` > 5.6),
	`col29` float NOT NULL,
	`col30` float CHECK (`col30` > 5.6),
	`col31` float,
	`col32` float,
	`col33` int(11) CHECK (`col33` > 5),
	`col34` int(11) NOT NULL,
	`col35` int(8) CHECK (`col35` > 5),
	`col36` int(8),
	`col37` int(11),
	`col38` decimal(19,4) CHECK (`col38` > 0.0),
	`col39` decimal(19,4) NOT NULL,
	`col40` decimal(16,2) CHECK (`col40` > 0.0),
	`col41` decimal(16,2),
	`col42` decimal(19,4),
	`col43` tinyint(2),
	`col44` tinyint(3) UNSIGNED,
	`col45` tinyint(3),
	`col46` smallint(8),
	`col47` smallint(6),
	`col48` varchar(255) CHECK (`col48` LIKE "test%"),
	`col49` varchar(255) NOT NULL,
	`col50` varchar(32) CHECK (`col50` LIKE "test%"),
	`col51` varchar(32),
	`col52` varchar(255),
	`col53` text CHECK (`col53` LIKE "test%"),
	`col54` text NOT NULL,
	`col55` text CHECK (`col55` LIKE "test%"),
	`col56` text NOT NULL,
	`col57` text,
	`col58` text,
	`col59` time NOT NULL,
	`col60` time,
	`col61` timestamp NOT NULL,
	`col62` timestamp,
	`col63` timestamp NULL DEFAULT NULL,
	`col64` int(11) COMMENT 'test comment',
	`col65` json
)

/yii2/framework/db/Schema.php:664
/yii2/framework/db/Command.php:1263
/yii2/framework/db/Command.php:1075
/yii2/tests/framework/db/QueryBuilderTest.php:1080
/yii2/vendor/phpunit/phpunit/phpunit:52

Caused by
PDOException: SQLSTATE[42000]: Syntax error or access violation: 1067 Invalid default value for 'col62'

/yii2/framework/db/Command.php:1258
/yii2/framework/db/Command.php:1075
/yii2/tests/framework/db/QueryBuilderTest.php:1080
/yii2/vendor/phpunit/phpunit/phpunit:52

FAILURES!
Tests: 706, Assertions: 3248, Errors: 1, Skipped: 7.

And I want to ask why we use the NO_ZERO_IN_DATE mode in the travis config?

We've tried to use strictest mode possible to eliminate possible errors.

I just created PR that will allow to ignore 'TIMESTAMP' column definition test when NO_ZERO_DATE enabled (it's default for MySQL 5.7).

It strange decision to use strictest mode for MySQL 5.6 test environment because strict mode reduces testing capabilities (for example, it isn't possible to test this 'TIMESTAMP' column definition case), and MySQL <= 5.6 doesn't use strict mode by default. It's good decision for MySQL 5.7 because it uses the strict mode by default: https://www.percona.com/blog/2016/10/18/upgrading-to-mysql-5-7-beware-of-the-new-strict-mode/

However, if you want to use strictest mode you should enable NO_ZERO_DATE and NO_ZERO_IN_DATE in travis configuration together. I've never seen them turned on separately for MySQL instances.

@rob006 you can't use TIMESTAMP NULL column definition without DEFAULT value (for MySQL). So if you want to call null() implicitly for timestamp() you also must call defaultValue(null) for it...

I think we forgot it by mistake.


Improve Test Coverage

More Unit Tests need to be added to take coverage to 90%. 💻

Hi @live-wire @RishabhJain2018 I am up for this issue. Please assign it to me. Also, any suggestions on how to get started with this issue?

@RishabhJain2018 can two people work on this? I have worked with selenium and pupeteer before, doing some karma tutorials now.

Hi @Abhi58 @wadleo,
You both can work on this issue it'd be nice if you will mention here on which part are you working on.

Okay @RishabhJain2018, thanks


Remove Buffer

Buffer no longer supports the registration of new developer applications.

¯_(ツ)_/¯


Few questions

do you plan to support M1/M2 mac?
can you make this jar modularized?

do you plan to support M1/M2 mac?

Current status:
https://github.com/kawamuray/wasmtime-java/issues/36

can you make this jar modularized?

You mean Java module (module-info.java)? I think there's no issue supporting modules in this project, we don't even have to giveup java8 support according to https://stackoverflow.com/questions/54115591/can-i-use-module-info-java-if-target-level-is-8 .

I see, all right.
Regarding to the module: yes, I meant Java module. It's a bit cumbersome to use this in a java project which is modularised. For eg. JavaFX projects generated by IntelliJ are modularised by default.
For the second, one can do a workaround, but the first one is a showstopper atm :( I'll try the manual build what you described in the other issue though.


Support Ids in MultiTermVectors API

This commit adds support for providing a set of Ids to MultiTermVectors API to be used in
conjunction with index and type provided in the URI.
Index() and Type() methods added to MultiTermVectorOperation to allow the default typeof(T)
values to be overidden.
Closes #3219

build seed:58181 test "multitermvectorsids fails with:
   Tests.Document.Multiple.MultiTermVectors.MultiTermVectorsIdsApiTests.SerializesInitializer [FAIL]
      Expected True because RoundTrip: 0 itteration
      Deserialization has result: False
      RoundTripResult`1 success: False
      This is the first time I am serializing
        {
          "ids": [
      -     {},
      -     {}
      +     1001,
      +     1002
          ]
        }

       C# approximation of actual ------
       new {
        ids = new [] {
          1001,
          1002
        }
      };
       C# approximation of expected ------
       new {
        ids = new [] {
          {},
          {}
        }
      };, but found False.
      Stack Trace:
           at FluentAssertions.Execution.XUnit2TestFramework.Throw(String message)
           at FluentAssertions.Execution.AssertionScope.FailWith(String message, Object[] args)
           at FluentAssertions.Primitives.BooleanAssertions.BeTrue(String because, Object[] becauseArgs)
        C:\projects\elasticsearch-net\src\Tests\Tests.Core\Extensions\SerializationTesterAssertionExtensions.cs(9,0): at Tests.Core.Extensions.SerializationTesterAssertionExtensions.ShouldBeValid(SerializationResult result, String message)
        C:\projects\elasticsearch-net\src\Tests\Tests.Core\Extensions\SerializationTesterAssertionExtensions.cs(33,0): at Tests.Core.Extensions.SerializationTesterAssertionExtensions.AssertSerialize[T](SerializationTester tester, T object, Object expectedJson, String message, Boolean preserveNullInExpected)
        C:\projects\elasticsearch-net\src\Tests\Tests.Core\Serialization\ExpectJsonTestBase.cs(24,0): at Tests.Core.Serialization.ExpectJsonTestBase.RoundTripsOrSerializes[T](T object)
        C:\projects\elasticsearch-net\src\Tests\Tests\Framework\EndpointTests\ApiTestBase.cs(37,0): at Tests.Framework.ApiTestBase`5.SerializesInitializer()
    Tests.Document.Multiple.MultiTermVectors.MultiTermVectorsIdsApiTests.SerializesFluent [FAIL]
      Expected True because RoundTrip: 0 itteration
      Deserialization has result: False
      RoundTripResult`1 success: False
      This is the first time I am serializing
        {
          "ids": [
      -     {},
      -     {}
      +     1001,
      +     1002
          ]
        }

       C# approximation of actual ------
       new {
        ids = new [] {
          1001,
          1002
        }
      };
       C# approximation of expected ------
       new {
        ids = new [] {
          {},
          {}
        }
      };, but found False.


PG (Papua New Guinea) shows Invalid Time Zone

TimeZoneNames.TimeZoneNames.GetTimeZonesForCountry("PG", "en-US") threw an Invalid Time Zone error.  Should be coming up with Pacific/Port_Moresby?  All the other countries I'm using resolved properly from what I've seen so far.  Great utility by the way!

Thanks!  I'll investigate.

Traced it back to the CLDR.  It was fixed in the latest 27.1 release.  diff here

Fixed in TimeZoneNames release 1.2.1 and pushed to NuGet.
Thanks!

Looks good, that was quick!

Fortunately, it was already corrected in the CLDR.  So it just required running the existing data builder.

FYI, I'm trying to use it to convert the somewhat ambiguous abbreviation codes to the IANA Region/City code by providing a country code with the abbreviation.  For example:
EEST, RO => Europe/Bucharest.
It is a more challenging task than I expected.  It mostly works ... but not in every case.

TimeZoneNames.TimeZoneNames.GetAbbreviationsForTimeZone("Asia/Karachi", "en-US");
for example doesn't give me any abbreviations to work with.

Yeah, there would be issues with that kind of operation.  If you like, we can chat about it on gitter

gitter is blocked at my work, as are similar on-line communication venues (data security protocols).  Surprised that github isn't entirely blocked.  I've not gotten deep into the CLDR source data, it just looks to be not complete enough to perform this type of backward translation for the abbreviations.  It is close though, so close... I'll probably kick myself later, but I'm just going to supplement the dozen or so missing abbreviation items that my project needs as config settings and go with it.

In general, abbreviations are intended for display purposes, and don't always map backward in a clean manner.   With the country code added, I think you will have fewer ambiguities, but there will probably still be a few that are unresolvable.
And yes - CLDR is missing abbreviations for many zones.  However, while IANA has abbreviations for all zones, they only have English abbreviations, and in many cases they are invented to fill the knowledge gap.   There's some discussion going on right now on the TZ mailing list about MSK being used for Minsk, where Minsk traditionally wouldn't use an abbreviation at all, and MSK is confused with Moscow - which becomes a political issue for Minsk.
Also, they don't always follow common sense, such as HAST/HADT being used for the Aleutian Islands traditionally, but switched recently to HST/HDT such that Hawaii's common HST applies in Adak, Alaska.  :confused:
Good look with the mapping, but it may prove to be a futile exercise to capture the whole world.  Focus on the areas you care about in your particular application.


[now-client] Add debug logs

This PR adds extensive debug logging to now-client and enables it in CLI based on the --debug flag
Debug logging works in either of the following two conditions:

debug: true is provided in the options object of createDeployment/createLegacyDeployment
process.env.DEBUG environment variable is set

This looks like a good starting point. One part that kind of bothers me is that in general I don't think JS libraries should directly output debugging information, but instead provide a mechanism for the top-level app to consume the libraries debugging information.
I was thinking that now-client could yield objects of type: "debug" when the debugging mode is enabled. Generator functions and yield * could be a "neat" way of accomplishing this while staying DRY:
function createDebug(enabled = false) {
  return async function* debug(log) {
    if (enabled) {
      yield { type: 'debug', payload: log };
    }
  }
}

async function* deploy(opts) {
  const debug = createDebug(opts.debug);

  yield { type: 'initializing' };
  yield* debug('uploading files');
  yield { type: 'ready' };
}

async function main() {
  console.log('deploying without debug');
  for await (const event of deploy({ debug: false })) {
    console.log({ event });
  }
  console.log('done deploying without debug');

  console.log();

  console.log('deploying with debug');
  for await (const event of deploy({ debug: true })) {
    console.log({ event });
  }
  console.log('done deploying with debug');
}

main().catch(console.error);

$ node t
deploying without debug
{ event: { type: 'initializing' } }
{ event: { type: 'ready' } }
done deploying without debug

deploying with debug
{ event: { type: 'initializing' } }
{ event: { type: 'debug', payload: 'uploading files' } }
{ event: { type: 'ready' } }
done deploying with debug

Just an idea.

@TooTallNate I was thinking about it but wasn't sure how to neatly accomplish it. TIL about yield*
Will do it the events way 🙏

Codecov Report

Merging #2997 into canary will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           canary    #2997   +/-   ##
=======================================
  Coverage   13.47%   13.47%
=======================================
  Files         267      267
  Lines       10288    10288
  Branches     1211     1211
=======================================
  Hits         1386     1386
  Misses       8840     8840
  Partials       62       62

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e8e95b8...5016615. Read the comment docs.


disable_trimming

In the plotDrGene function, can you add a trim=FALSE option? Sometimes it would be useful to see how strong the markers express without trimming.

Hi, please set the trim parameter to NULL to disable the trimming.

@XingyiShi please see @ykoga07 's comment above. Closing issue.


ported to work with mongo driver 2.X

working with mongo driver 2.x

I don't know why the Travis check is failing on a file I didn't even modify:
plugins/rabbitmq/check-rabbitmq-messages.rb:115:61: C: Do not use semicolons to terminate expressions.
(warn_queues["#{queue['name']}"] = queue['messages']; next) if queue['messages'] >= config[:warn].to_i # rubocop: disable Style/Semicolon
So not sure what's going on there.  But the file changes I did are fine.

@himyouten
Can you also PR these against the new repo.
sensu-plugins-mongodb
Thanks!


React-parallax Prompt HTMLDocument is not defined

Execute the command: npm run dev
The following error:
C:\Users\quzhenfei\desktop\Fedup-PC\node_modules\react-parallax\dist\index.js:1
(function (exports, require, module, __filename, __dirname) { !function(e,t){"object"==typeof exports&&"object"==typeof module?module.exports=t(require("react"),require("react-dom")):"function"==typeof define&&define.amd?define(["react","react-dom"],t):"object"==typeof exports?exports["react-parallax"]=t(require("react"),require("react-dom")):e["react-parallax"]=t(e.React,e.ReactDOM)}(this,function(e,t){return function(e){function t(i){if(n[i])return n[i].exports;var o=n[i]={exports:{},id:i,loaded:!1};return e[i].call(o.exports,o,o.exports,t),o.loaded=!0,o.exports}var n={};return t.m=e,t.c=n,t.p="",t(0)}([function(e,t,n){"use strict";function i(e){return e&&e.__esModule?e:{default:e}}Object.defineProperty(t,"__esModule",{value:!0}),t.Background=t.Parallax=void 0;var o=n(4),r=i(o),s=n(3),a=i(s);t.Parallax=r.default,t.Background=a.default},function(t,n){t.exports=e},function(e,t){"use strict";function n(e,t){if(!t)return!1;var n=e
ReferenceError: HTMLDocument is not defined

Ok, I will have a look. Think it's related to #25.

When I change the index.js the file,He will be able to work. How to better solve the problem Rather than to modify the file after each installation @RRutsche

Hey guys, got the same error, I placed the parallax in a component, when I'm calling it like 2 times it works, 3 times doesn't work, it really random.

Then you need to repair，Site use of this component now

@Q-Angelo @guillaumeko The bug is fixed in version 1.2.8.
Please check if it also works for you.

@RRutsche  Good!

It seems to work great for me! Thank you very much :)

top. thanks. than I can close the issue


Feature/prime sdk paymaster support

Updated @etherspot/prime-sdk to version 1.2.11
Added paymaster prop to <EtherspotBatches /> which is used for setting custom paymaster sent via etherspot-prime provider

@IAmKio thanks, we do have errorMessage on onSent and onEstimated, but we can adjust.


docs: Add documentation

Adds documentation to the docs director. This is not a GUIDE! It's documentation for every function, type, interface, class and method in the lib directory!

Synced up with the new version


Module migration fails when code upload access is set to OnlyAddress

Steps to reproduce

Compile a binary from a 0.3x version  and start chain ( tested with tag v.0.33.0)
Set code upload access with OnlyAddress
Propose upgrade to 0.4x, compile a 0.4x binary and upgrade (tested with v0.45.0)
Upgrade panics with below message :

INF migrating module upgrade from version 1 to version 2 module=server
INF migrating module wasm from version 2 to version 3 module=server
panic: upload access: type: empty

goroutine 7 [running]:
github.com/cosmos/cosmos-sdk/x/upgrade/keeper.Keeper.ApplyUpgrade({{_, _}, _, {_, _}, {_, _}, _, {_, _}, ...}, ...)
	github.com/cosmos/cosmos-sdk@v0.47.5/x/upgrade/keeper/keeper.go:361 +0x398
github.com/cosmos/cosmos-sdk/x/upgrade.BeginBlocker(_, {{0x10531c9d0, 0x14000368080}, {0x105330010, 0x14001400e80}, {{0xb, 0x0}, {0x14001248a08, 0x4}, 0x9b, ...}, ...}, ...)

This seems to be an error originating from the ValidateBasic method

I did notice that it was deprecated and the solution works well. However, when looking over at the migration code for v3->v4, specifically here, I thought it was expected for the module to handle this migration automatically.
Seemed a little counter-intuitive, so decided to open this issue.

This is clearly a 🐛 on our side. Removing old code seems to be a bigger challenge. I appreciate the issue and will work on a fix. But this can take some time to be released


OSSM-5439 Remove the 2.2 SMCP option from the 2.5 SM drop-down in UI

https://issues.redhat.com/browse/OSSM-5439

@luksa Ran make gen before pushing. Only thing that changed was the dates in the normal two files. Then ran make clean gen and got the same output

@mayleighnmyers try running make gen in the $GOROOT/src/github.com/maistra/istio-operator dir instead of where you're currently running it (I think you're hitting the issue where make gen updates the files in the wrong dir again).

@mayleighnmyers try running make gen in the $GOROOT/src/github.com/maistra/istio-operator dir instead of where you're currently running it (I think you're hitting the issue where make gen updates the files in the wrong dir again).

MY dir I am using is /home/mayleighmyers/go/src/github.com/maistra/istio-operator, I have not moved my istio-operator dir since we fixed this issue last time


[Docenhancement][Together mode][1827725]

Updated name change in teams together mode, toc, whats new, teams developer portal and teams apps in meetings.

Docs Build status updates of commit 2bd8e03:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/concepts/build-and-test/teams-developer-portal.md
:warning:Warning
View
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/teams-apps-in-meetings.md
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/teams-together-mode.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/build-and-test/teams-developer-portal.md

Line 84, Column 28: [Warning-file-not-found] Invalid file link: '(~/apps-in-teams-meetings/teams-together-mode.md)'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 92d7145:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/teams-apps-in-meetings.md
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/teams-together-mode.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/build-and-test/teams-developer-portal.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 2cf15a9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/teams-apps-in-meetings.md
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/teams-together-mode.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/build-and-test/teams-developer-portal.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit b24c5d1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/teams-apps-in-meetings.md
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/teams-together-mode.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/build-and-test/teams-developer-portal.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 8200b92:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/teams-apps-in-meetings.md
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/teams-together-mode.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/build-and-test/teams-developer-portal.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 9f9e6d1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/teams-apps-in-meetings.md
:white_check_mark:Succeeded
View

msteams-platform/apps-in-teams-meetings/teams-together-mode.md
:white_check_mark:Succeeded
View

msteams-platform/concepts/build-and-test/teams-developer-portal.md
:white_check_mark:Succeeded
View

msteams-platform/whats-new.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Add support for private repositories in git-sync

This adds support for http authentication, shallow clone using depth flag and one time execution (exit 0 after initial clone)
ref kubernetes/kubernetes/issues/16247

@kubernetes/sig-storage review?

looks like comments are addressed, anyone LGTM?

ping

LGTM


Env markers

Use environment markers instead of a separate requirements file for py2.

Coverage remained the same at 84.222% when pulling 876bea4ddd6c95363126349020deea4c66066a44 on env-markers into 34f47da033c4f3beb55b464515007a98b9a22e69 on master.

@llllllllll What do you think of this replacement for requirements_py2?

just had one comment about the translation to a conda selector, but otherwise lgtm. I wonder if we should make a package like versioneer that has our cython stuff and now our conda translation. I feel like this is a problem other people also have.

Coverage increased (+0.04%) to 84.264% when pulling 962f0a5407a09e606b734bec522c12bc3ffd1265 on env-markers into 34f47da033c4f3beb55b464515007a98b9a22e69 on master.

Coverage decreased (-0.04%) to 84.229% when pulling c9a7b936ed2b5110a55547ddad6942eb44615328 on env-markers into 6bf4fe15e27015a7318b641f58071530eec9c9c2 on master.


Corona Warn App crashes without error shortly after startup

Avoid duplicates

[ x ] Bug is not mentioned in the FAQ
[ x ] Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[ x ] Bug is not already reported in another issue

Describe the bug

My CWA on Android crashes every time I try to launch it. The splash screen with the CWA C-Logo appears for about 5 seconds before the app closes without presenting any error/feedback. During these 5 seconds, the phone becomes unresponsive (does not react to pressing of home/back button or even the hardware buttons.
Expected behaviour
The app doesn't close after 5 seconds without error.

Steps to reproduce the issue
Open the app and see it crash.

Im unsure on how to collect crash logs here.

Technical details

Mobile device: One Plus 6
Android version: Android 10
Oxygen OS version: 10.3.6

Possible Fix

Additional context
What I tried: Reinstalling the app, deleting all local data.

@Jonas164
You didn't mention the version of the app. Is it 1.5.1?
Did reinstalling the app fix the issue or is the app still crashing? I assume it is still crashing, but wanted to be certain.

@MikeMcC399
I have the latest app version 1.5.1.
Yes, app is still crashing. My tries did not fix the problem

Hello @Jonas164
Thanks for the feedback. Did the crashes appear after updating to version 1.5 or did they already happen before that?
Your bug might also be related to https://github.com/corona-warn-app/cwa-app-android/issues/1053 for which a workaround (https://github.com/corona-warn-app/cwa-app-android/issues/1053#issuecomment-690615473) exists. However, https://github.com/corona-warn-app/cwa-app-android/issues/1053 was mostly an issue on Huawei devices and not on OnePlus.
Regards,
CH

Corona-Warn-App Open Source Team

@heinezen I'm unsure if the crash happened before the update or after, I think it is like this for about 4 weeks now.
I might have figured out what causes this behaviour thanks to your link. When I clear my cache and local files, the app will start up just fine. When I lock my phone afterwards and let it idle for some minutes, cwa will once again start crashing on startup.
This must be an issue with my phones energy saving plan, even though I have it fully disabled for CWA. I will try to disable all the energy saving options I can find and hope for the best.
Wouldn't some error handling be a good idea in my case? It seems like just exiting the app when the database cant be decrypted isn't very intuitive. Maybe a warning to the user with advice on how to fix the problem?

@Jonas164 If your issue is indeed the same as in https://github.com/corona-warn-app/cwa-app-android/issues/1053, then it might be fixed in the release 1.6. The issue was originally addressed in 1.5 and 1.5.1 (see https://github.com/corona-warn-app/cwa-app-android/issues/1053#issuecomment-714345887), but we are still receiving reports. However, we are still working on solution for this.

Corona-Warn-App Open Source Team

All right, I would close this then. Thanks alot for your work and stay healthy :)


How Payment module be remove complete?

How can the "payment" module be removed completely?
Thank you

How can the "payment" module be removed completely?
Thank you

Excellent question!
You will need to select the "Regular (Free)" membership as the default one.

And removing any "Paid" memberships through the Billing module


deprecation indicator and deprecation notes standardization

By deprecation notes standardization I have meant following changes:

using current NPM package name
using similar phrasing
positioning deprecation note always on the top of the page

The rest of changes are basically a concept, which you might not like.
I was aiming to clarify deprecation and discourage developers to use those parts of RN so I have added  🚧emoji to the title of pages marked as deprecated Component or API. (Of course I'm open to change selected emoji, this could be anything else.)

I can work further on this PR or on any other solution that help in the case mentioned above.

@rachelnabors I can extract deprecation standardization changes to separate PR if it help merge them sooner than indicator proposal.

@hramos I can't request PR review but maybe you will have any feedback for me?

Hi hi! Thanks for working on this. I love it! Can you do ProgressViewIOS and ProgressBarAndroid as well?

I have updated both pages. Hope I have used the right community repos for those components.


fix: specialist lesson schema [LESQ-704]

Description
Music year: 2011

make developmentstage optional in the schema

Issue(s)
Fixes #
Therapy lessons not loading
How to test

Go to https://deploy-preview-2319--oak-web-application.netlify.thenational.academy
should be the same

Screenshots
How it used to look (delete if n/a):

How it should now look:

:tada: This PR is included in version 1.386.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Use minutely data emission by default

Rework of https://github.com/zipline-live/zipline/pull/37

Coverage decreased (-0.006%) to 86.028% when pulling 7c158b4fcc65a8da0964a75a92599222736284f4 on fix/minutely_emission into bed2a29759badf8afab429ef9955a5b79dfaa1e3 on master.

@bartosh  / @pbharrin : pingg :)


doc: suggest disabling GPG signed commits to allow build to pass

Add instructions to disable signed commits in the rewrite repository if necessary because JGit does not currently have support for SSH format GPG signatures.

What's changed?
Added a suggestion to the "building from source" page to disable GPG signed commits in the repo, in case you are using SSH format GPG signed commits globally, which JGit does not currently support.
What's your motivation?
Make it easier for people to build OpenRewrite from source without running into mysterious test errors related to their git configuration.
Errors during ./gradlew build look like this when SSH GPG commit signing is configured:
GitProvenanceTest > detachedHeadJenkinsNoLocalBranch(Path) FAILED
    java.lang.IllegalArgumentException: Invalid value: gpg.format=ssh
        at org.eclipse.jgit.lib.DefaultTypedConfigGetter.getEnum(DefaultTypedConfigGetter.java:105)
        at org.eclipse.jgit.lib.Config.getEnum(Config.java:403)
        at org.eclipse.jgit.lib.GpgConfig.<init>(GpgConfig.java:86)
        at org.eclipse.jgit.api.CommitCommand.processOptions(CommitCommand.java:620)
        at org.eclipse.jgit.api.CommitCommand.call(CommitCommand.java:180)
        at org.openrewrite.marker.GitProvenanceTest.initGitWithOneCommit(GitProvenanceTest.java:143)
        at org.openrewrite.marker.GitProvenanceTest.detachedHeadJenkinsNoLocalBranch(GitProvenanceTest.java:127)

GitProvenanceTest > detachedHeadBehindBranchHead(Path) FAILED
    java.lang.IllegalArgumentException: Invalid value: gpg.format=ssh
        at org.eclipse.jgit.lib.DefaultTypedConfigGetter.getEnum(DefaultTypedConfigGetter.java:105)
        at org.eclipse.jgit.lib.Config.getEnum(Config.java:403)
        at org.eclipse.jgit.lib.GpgConfig.<init>(GpgConfig.java:86)
        at org.eclipse.jgit.api.CommitCommand.processOptions(CommitCommand.java:620)
        at org.eclipse.jgit.api.CommitCommand.call(CommitCommand.java:180)
        at org.openrewrite.marker.GitProvenanceTest.detachedHeadBehindBranchHead(GitProvenanceTest.java:160)

Anything in particular you'd like reviewers to focus on?
Not really, this is a single-line documentation update.
Anyone you would like to review specifically?
No.
Have you considered any alternatives or workarounds?
Yes, disable GPG signed commits globally, but this may not be an option or the most convenient option for most people because they would like to sign their commits elsewhere.
I looked into upgrading JGit but not even master supports the SSH format. See GpgConfig.java for more information, or this bug.
Any additional context

Checklist

[ ] ~I've added unit tests to cover both positive and negative cases~ N/A
[ ] ~I've added the license header to any new files through ./gradlew licenseFormat~ N/A, no new files
[ ] ~I've used the IntelliJ IDEA auto-formatter on affected files~ N/A
[x] I've updated the documentation (if applicable)

Sorry to hear you had issues getting the tests to work, and thanks for the detailed analysis and update to the docs here! I've tagged Mike for a final review before we merge; this is very helpful to get sorted thanks!

You're both very welcome! I'd like to start contributing more to the OpenRewrite codebase itself and the docs felt like a good place to start. Great to have my PR merged so quickly 🚀

Glad to hear you'd like to start contributing more! Feel free to tag me in any issue you'd like to take on, and I can assign the issue to you and provide guidance as needed. We have a few items tagged as good-first-issue, but you're welcome to work on anything. We also have a public Slack if you have any questions. Hope that helps!


[FEATURE] Reload/request reload after data import

Is your feature request related to a problem? If so, Please describe.
I just tried the import feature in the annotator window and it's great. However, I assumed it was broken because nothing in the viewer changed until I refreshed the page.
Describe the solution you'd like
I think an auto-refresh would be nice.
Describe alternatives you've considered
Ask if the user wants to reload, in case some state would be deleted.

The intention is that it should do an automatic reload when you upload, there was a bug I found last night that may be preventing this with larger annotation files.
Could you provide two pieces of information on this?

What was the approximate size of the annotation files you were uploading in MB?
Additionally could you try this again and open the chrome-dev tools (CTRL+SHIFT+I) and see if there is an error in the console?

My accounts shut down in 25 minutes so I probably won't be getting to it today...maybe as an external collaborator. The files were added on this merge request  to the DVC repo.

The 2007 collect didn't load at all for me, maybe it was too big.

I believe steps to reproduce:

Clone https://viame.kitware.com/#/viewer/605befa98ba6bae828a05f5b into your own public folder.
In the slack chat search for: "sealions_2007_v9_deepmac.kwcoco.json" and download the kwcoco file.
Load up your cloned version and then import the kwcoco file.  It may take a few moments because it is a ~37MB file.  So the importing icon may be spinning for a while.
After complete the system should autokick itself into reloading the data and then you should be able to see the poloygons on the sealions.

I think the auto-reload after the upload should satisfy this issue to close it out.

Confirmed that this is fixed.


Add more social icons?

More social icon SVGs, such as Instagram, LinkedIn and email, can be added.

We could definitely support more social icons natively. I've seen examples in the wild of others who have expanded upon what this project includes (github/twitter).
cc @chuxinh - maybe you have some input on how you accomplished this?

@jakewies oh I thought you were using some ready-made icon packs for the existing GitHub/Twitter icons. I guess I'll use some free "outline social icon packs" for my fork then.

@kentnek The current theme is a bit tedious in terms of adding social icons. My site is achieved by the following:

Have a partial of social-icons.html
define social icons in the partial like below:

To render those icons properly, you need to modify or add in the css. I saved all svgs in a file under static/icons
.social-icons__icon--twitter {
  background-image: url("/icons/twitter.svg");
  margin-right: 1rem;
}

And then you can use those partials anywhere needed. Hope that helps!

@chuxinh oh my problem is actually finding the SVGs. May I know where you obtain the SVGs for other icons?

@kentnek there's a good site for you looking for SVGs: https://www.flaticon.com/
Can use those free one for both personal and commercial  use.

@kentnek i believe the icons I used were from https://feathericons.com/

@jakewies oh awesome this is exactly what I need. I'll add in some more icons after finishing the SCSS PR.

We could open a PR to add some svg icons from feather that cover "most" use cases, and they can be optional based on configuration setup that we decide on.

^ Yep, I'm also thinking that way.

Closing with https://github.com/jakewies/hugo-theme-codex/pull/40


Add  option ExpansionFulfillmentStyle.none

to allow fulfill action without changing state, for example when it is needed to delete item in performBatchUpdate via changing model only (do not delete item via swipeController)

Without this option I need to do some ugly methods
let action = SwipeAction(style: .default, title: button.title) { [weak self] action, _ in
                button.action()
                if button.isDestructive {
                    UIView.animate(
                        withDuration: Constants.animationDuration,
                        animations: {
                            self?.alpha = 0
                        },
                        completion: { _ in
                            self?.prepareForReuse() // to call cell.reset()
                        }
                    )
                }
            }

Why would you need this "hack"? prepareForReuse will/should be called prior new cells. I am trying to understand what is the underlying issue. Could you not call fulfill and everything works fine?

I need to have a way to call SwipeController.reset()
without it, after deleting of cell via swipe in collectionView, swipe of next cell is not working...
If it is necessary, I can try to reproduce it in some test project

I need to have a way to call SwipeController.reset()
without it, after deleting of cell via swipe in collectionView, swipe of next cell is not working...
If it is necessary, I can try to reproduce it in some test project

Test project would be nice. My initial feeling is that something isn't being reset when prepareForReuse is called on the new visible cells.

Please provide a sample project that demonstrates the underlying issue. prepareForReuse should be doing what you need.


What does --toggle-sort do?

Category

[ ] fzf binary
[ ] fzf-tmux script
[x] Key bindings
[ ] Completion
[ ] Vim
[ ] Neovim
[ ] Etc.

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Windows Subsystem for Linux
[ ] Etc.

Shell

[x] bash
[ ] zsh
[ ] fish

Hi, --toggle-sort is being used in fzf_history but fzf --help doesn't mention what it does.
In fzf_history, why have +s followed by --toggle-sort?
Also, can you please explain what -n does with an example?

Have you looked at man fzf? More information can be found in the man page. --toggle-sort is kind of deprecated (no plan to remove it though) as it's equivalent to --bind KEYNAME:toggle-sort. We disable sorting with +s (--no-sort) by default to display the commands in chronological order. But sorting can be dynamically enabled by the key specified in --toggle-sort.
-n 2.. (ignoring index number) allows you to perform prefix match on the command such as ^git.

Have you looked at man fzf?

No, I didn't even know a man page existed. Thanks for letting me know. That helps but just raises more questions for eg. this bit here: -n2..,... The man page says -n2.. will restrict the search scope to the 2nd to the last field but then there's also ,.. after that increasing the search scope to all fields. Either I'm not following what it does exactly or that seems redundant.
Also, the only thing in there is this line toggle-sort is equivalent to --toggle-sort. It doesn't mention how sorting works. I tried using C-r but that doesn't seem to do anything. Here's an example of the before and after pressing it:
Before:

After:

As you can see the order of the command history hasn't changed.

Each comma-separated search scope is tried in order. So fzf first checks if the query matches in 2.. scope, if not, then it searches the whole scope ...

^sudo won't match without 2..
2su won't match without ..
^20 is also valid

See the point?

As you can see the order of the command history hasn't changed.

Try it after typing in some query string. Regarding the ranking algorithm, see https://github.com/junegunn/fzf/blob/0.15.9/src/algo/algo.go#L39

Ah, I see. Thanks a lot for answering all my questions.


Is it goroutine-safe？

i want use GlobalObject，then use the goja.new()，it is  safe？

Not sure what you mean, please provide more details.

i now

i now


Execution of JS in data files

I'm not sure if it is a bug or a feature.
If you pass some url of a javascript file it will get executed in the context of the website.
E.g. if you save the file data.tsv:
alert(window.location);

You'll get the alert box on import from url http://urlto/data.tsv. This may be a security problem if you embed raw in a bigger context, i.e. an intranet.

not able to reproduce it

Thanks for looking into this.
I can still reproduce it.
Run on server:
$ cd rawgraphics
$ git rev-parse HEAD
29272a58d2497548364508b4aa6e1da392e8530c
$ echo "alert(window.location);" > data.tsv
$ python -m SimpleHTTPServer 9999 # assume it's myserver
In Browser:

Point Browser (FF 61.0.1 (64-bit) or Chrome 67.0.3396.99 (Official Build) (64-bit) both on windows) to myserver:9999.
Click "From URL"
Copy and paste "http://myserver:9999/data.tsv" into URL field. The leading http:// is important. Without it you get "Whoops! Something wrong with the URL you provided."
Notice the alert popup.

What happens for you?


(maint) add jessie to build defaults

This adds jessie to our build defaults.

@melissa any subtleties around this?

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://jenkins.puppetlabs.com/job/platform_puppetdb_intn-sys_pr/1785/


COVID Moonshot analysis improvements

The analysis of Folding@home simulation data is time-consuming and must be run on a user's local machine in an ad-hoc fashion.
This issue tracks potential improvements we've identified to address performance bottlenecks in the analysis. Issue #710 tracks efforts to automate the analysis to move pieces onto the F@h work server or cloud resources.

[X] Cache results of BAR calculation so results won't be unnecessarily recomputed when re-running the analysis on a superset of the original data. Implemented in https://github.com/choderalab/perses/pull/701.
[ ] Parallelize BAR calculation at run/phase/gen level

I'd like to add collection of conformations to something that is slow, and should be straight forward to be sped up!
The cache was helping me lots, especially at the end of last week when many GENS were already done, but I wasn't sure if I should commit the cache to github or not?


Links defined in reltables not taken into account when deciding to duplicate topic file names in key scope contexts

Expected Behavior
There are multiple topics referenced in two key scopes. The topics do not have key references inside them but there are relationship tables which define relationships between them based on key scopes. The related links inside the topics should point to the proper topic instances.
Actual Behavior
The related links inside the topics point to the first instance of the topic.
Possible Solution
Add the related links with keyref inside each topic.
Steps to Reproduce

Publish the attached branching.ditamap to HTML5.
Open the "chapter1-1.html". According to the reltable it should have a related link to a "chapter5-1.html" but instead it refers to "chapter5.html". There is no "chapter5-1.html" because the "chapter5.xml" does not contain any keyref inside it.

reltable-keyscopes.zip
Environment

DITA-OT version: 4.1.2
Operating system and version:
(macOS)
How did you run DITA-OT?
(oXygen)
Transformation type:
(HTML5)

I think the decision to create a "chapter5-1.html" should be taken by looking also at reltable links instead of looking if the "chapter5.xml" has keyrefs inside it or not.


Fix copy when adding stablecoin liquidity

en.json

add localization for helper text

existing PR but changed base branch

Are you planning on adding for other languages?

Are you planning on adding for other languages?

I can if you'd like. Are there languages that are prioritized or should I add a Google Translated version for all localizations that currently exist?

We should try to always update them as best we can with google translate, then consult others/natives later on the correctness.

We should try to always update them as best we can with google translate, then consult others/natives later on the correctness.

Ok, will update PR with other localizations.

Think you could bundle this issue into your PR?
Issue #1160

Think you could bundle this issue into your PR?
Issue #1160

Yep, will make this change in the Polish localization too!

Will review zh-tw when it's ready

@PikachuEXE LMK what you think of zh-tw

What should I do to make stablecoinTypes.collateralized appear...
I know it's rendered by https://github.com/osmosis-labs/osmosis-frontend/blob/stage/packages/web/components/complex/add-liquidity.tsx#L194 but reading the code doesn't make it easier to know how to test...

What should I do to make stablecoinTypes.collateralized appear... I know it's rendered by https://github.com/osmosis-labs/osmosis-frontend/blob/stage/packages/web/components/complex/add-liquidity.tsx#L194 but reading the code doesn't make it easier to know how to test...

You can find it by going to Pools > USDC/OSMO Pool (or any pool with a stablecoin on one side) > Add Liquidity, and you should see it as helper text beneath the stablecoin.

@jonator Could we make a type and use it in 'use-current-language' hook when importing JSONS so if they're out of sync with each other it'll give a more explicit error?

IDK if that would fix our problem at runtime? That would require typescript to read the JSON at build time and type check it against our new type, which I don't think is possible.
Alternatively, we could write a node script to find missing members compared to en.json. It could additionally find all uses of t() in components and make sure there's no extraneous members in the json files (something diff-ing doesn't help with, and is otherwise a highly manual process).

IDK if that would fix our problem at runtime? That would require typescript to read the JSON at build time and type check it against our new type, which I don't think is possible.
Alternatively, we could write a node script to find missing members compared to en.json. It could additionally find all uses of t() in components and make sure there's no extraneous members in the json files (something diff-ing doesn't help with, and is otherwise a highly manual process).
There is this, but I don't think it works in our case since we're using dynamic imports to reduce bundle size.

You're right, I see now we're dynamically importing it, which makes type safety a bit harder. Manual checking seems fine though.

Can we have zh-hk updated (just copy from zh-tw) too?
Since #1051 is already merged

@jonator Since #1051 was merged into main, I don't have that commit since I branched off of stage. Should I just copy and paste zh-hk for now?

Added!

@sehyunc Please address comments from former review. Additionally:

zh-hk.json missing pools.myLiquidity, missing last 4 keys in swap, remove pool.unbonded, missing menu.buyTokens,

Sorry, misunderstood your previous comment, I thought you had made those changes on this branch. Updating this now.

Hey @PikachuEXE really appreciate your initiative to jump in and check this stuff! Email me and we can get you an OSMO grant for your work.


Added device to jnp.arange, jnp.eye and tests

Description:

Added device to jnp.arange, jnp.eye and tests

Related to https://github.com/google/jax/issues/21088
Here is how looks the documentation for these methods:

You can fix the test failures by adding the device parameter to the list here: https://github.com/google/jax/blob/2b728d55b6054bba8ae26b3523722e80d660e771/tests/lax_numpy_test.py#L5974

@jakevdp I'll add this note to CHANGELOG.md as well:
* Changes
  * ...
  * {func}`jax.numpy.arange` and {func}`jax.numpy.eye` now have `device`
    argument to put the output array on provided device.

Can this be enough and is it a change of "New Functionality"?

I would probably leave this out of the changelog. We try to strike a balance between making the changelog useful without making it just a description of every commit that goes into the release... It's a gray area, but this seems like a change we don't necessarily need to announce.

I think we can now remove the functions from here: https://github.com/google/jax/blob/b458e3cea1b5a64e28fd085a9d9b228939561179/jax/experimental/array_api/_creation_functions.py#L20
and import the jax.numpy APIs directly here: https://github.com/google/jax/blob/b458e3cea1b5a64e28fd085a9d9b228939561179/jax/experimental/array_api/init.py#L182
We should do this as part of this PR to cover the new code paths via the array API tests.

Array API arange and eye functions signatures are a bit different from jnp ones:
# Array API: step = 1
def arange(start, /, stop=None, step=1, *, dtype=None, device=None)
# vs jnp.arange, step is None
def arange(start: DimSize, stop: DimSize | None = None,
           step: DimSize | None = None, dtype: DTypeLike | None = None,
           *, device: xc.Device | Sharding | None = None) -> Array:

# Array API: arg names n_rows, n_cols
def eye(n_rows, n_cols=None, /, *, k=0, dtype=None, device=None):
# vs jnp.eye: N. M
def eye(N: DimSize, M: DimSize | None = None,
        k: int | ArrayLike = 0,
        dtype: DTypeLike | None = None,
        *, device: xc.Device | Sharding | None = None) -> Array:

array-api-tests do not fail with jnp.arange/eye, but still would like to confirm with you

That looks fine. We can be less strict than the Array API, but not more strict.


Create and Run an ABAP Application

Tutorial URL: https://developers.sap.com/tutorials/abap-create-basic-app.html
Please specify the step you are referring to Step 1: Create new ABAP program
Hello, GE,
It is not posible to create abap program object, (Could it be an account related issue). Look into this below please:

Let us know anything please.
Thanks and regards!
Victor

Hi Victor,
This tutorial is part of a mission: https://developers.sap.com/mission.abap-dev-get-started.html .
Did you follow the whole mission? If not, please do so.
If so, were you able to create a package in the system?
If not, please tell me what system you are working on, whether it is a licensed version, whether it is a developer or trial system etc.
Best wishes,
Julie.

Hi, GA,
Yes, i can create package in the system:

Im using a trial account.
Regards!
Victor

Hi Victor,
When you say Trial, do you mean  : SAP Cloud Platform, ABAP Environment ("Steampunk") Trial, or Download, e.g. 7.52 SP04?
Thanks Julie.

Hi Julie, GE,
Yes, it's the free trial SAP Cloud platform cockpit, then, how can i get a list of available tutorials and missions for this accout type?
Let me know anything please.
Regards!
Victor

Hi Victor,
Yes, you filter by "ABAP Development", then filter by "SAP Cloud Platform, ABAP Environment", then - IMPORTANT - choose a mission, group, or tutorial WITHOUT a key.
Tutorial Navigator: Steampunk
Best place to start:
https://developers.sap.com/mission.cp-starter-extensions-abap.html
, then : Create Core Data Services with ABAP Environment
then 2 tutorials:
Create a Database Table
followed by:
Create an ABAP Table Type.
HTH.
If this is helpful, then can you please close the ticket? Thanks
Julie.

Thank you Julie,
Regards!
Victor


Breadcrumb

🚀 Feature Proposal
Breadcrumbs help determine the user's current location in the application and also assists in navigation.
Motivation
Ease the use in navigation and knowing the context of the current location in the application.
Example
Home/Patients/View Patient

https://react-bootstrap.github.io/components/breadcrumb/.
react-bootstrap has a built in breadcrumb library that we will probably wrap. However, we will have to figure out a way to seamlessly integrate with whatever routing library that we use (I assume react-router).
react-bootstrap defines links as an href attribute which would reload the application if clicked.

@jackcmeyer i would like to  take on this issue


broh, both of the online ide try option is broken

broh, both of the online ide try option is broken

CODIDO seems ok, you just need to find it with CTRL+F. I just fixed the link for Google Colab. Thanks for letting me know!


Village Sync

Village interactions need to be synchronized via RPC and value persistence. Look into what other sync might be needed and implement.
See https://github.com/Bannerlord-Coop-Team/BannerlordCoop/blob/development/source/Coop/Mod/Persistence/Party/EventPartyMoveTo.cs. We use Railgun for RPC, https://github.com/araex/RailgunNet/blob/master/RailgunNet/Logic/RailEvent.cs

OnSettlementOwnerChanged
CalculateLoyaltyChangeInternal
Town.DailyTick
Garrison
Parties
Heros (people in tavern)
InSeige
Owner Changed

Town.ChangeClanInternal
Upgrades?
BuildingCampaignBehavior.DecideProject
Town.TickCurrentBuilding
Villagers
Caravans
Loyalty
Inventory
Town.OnInventoryUpdated
MarketData

Resolved by #727


vm_image_util: add support for VMDK images for EC2 AMIs

Lets us avoid larger data transfers when importing images.

LGTM


Ball Shooting Game

The game is ready to be merged within 2 hours of the approval of this issue
I feel this is more sort of a pull request rather than an issue :) .Thank you
https://github.com/ssitvit/Games-and-Go/assets/118872531/0cbd16af-c9f5-4072-a508-c445a4844b64

Assigned this issue to you. Kindly complete it within a week and go through readme and contributing.md for successful merge. All the best to your work Aryan... !!!

@raj03kumar I have created a pr. Kindly review it and tell me if any necessary changes are required
Thank you


cleaner: rewrite pkg-config file Cellar paths and keg-only

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Concept idea based on manual work in Homebrew/core, e.g.

https://github.com/Homebrew/homebrew-core/blob/e0bb7c4a4c30da9f78d27d5e6035bbe46346b2ca/Formula/m/mpv.rb#L101-L109
https://github.com/Homebrew/homebrew-core/blob/e0bb7c4a4c30da9f78d27d5e6035bbe46346b2ca/Formula/y/yaz.rb#L66-L69

Would be nice to get this in in some form at some point.


03 IDTExtensibility2 Word Sample, the messagebox is shown many times, why?

register many times. deleting reg iterms resolves it.


SDK doesn't expose all API functionality

Hello!

Vote on this issue by adding a 👍 reaction
If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details
Thanks for your work on this provider, I fully intend to manage my Pulumi with Pulumi. 😄
Related to https://github.com/pulumi/pulumi-cloud-requests/issues/459 I am trying to write a C# Pulumi program to automatically coalesce permissions on stacks and teams to a known state.
A few things I've noticed about this provider:

as far as I can tell, the Get functionality is undocumented (it doesn't appear here: https://www.pulumi.com/registry/packages/pulumiservice/api-docs/)
The patterns of this provider don't seem to match others - eg Team.Get / Stack.Get vs GetTeam.Invoke/InvokeAsync, GetStack.Invoke/InvokeAsync similar to how the ec2 provider(s) works - I am wondering, generally speaking, if there is a shape or pattern to be followed for providers (we are thinking about writing our own)?

The reason I am opening this issue: There is no interface to the list functionality provided by the Pulumi API (such as list users, list teams, list stacks) In order to maintain permissions on stacks it would be very helpful to be able to have an interface to list them. As it is now, I have to use RestSharp to fetch these things separately instead, which is not great.
My expectation as a user is that the SDK exposes interfaces to all of the routes documented here. Not having an interface for all of the API leads to a really bad experience because you end up having to store the Pulumi API token separately from the stack itself so it can be used by whatever library you're using to talk to the API. I believe there is no way to retrieve a secret as a raw string so you either store it separately or are forced to do really gross things with Apply().
Affected area/feature
Data retrieval from Pulumi Cloud / interface to API

Good afternoon, thank you for opening the issue!

I see what you mean, PSP's resources like Team don't have docs on Get, like this. These docs are auto-generated, so likely there's something wrong with generation settings, or provider SDK is outdated, I will take a look.
The SDK is auto-generated as well, so likely the issue is the same as with docs - PSP needs to be migrated to newer pulumi-go-provider, which has been part of the plan for a while, but hasn't happened yet. I hope to get it done in the next couple sprints, but I sadly can't guarantee it.

For list logic, we would need to bring Invokes into PSP, which is another change that was deferred for a long time. Both should be addressed as part of the PSPV1 plan, which I am putting together now to be worked on in the next few sprints.
For the last part, I'm probably misunderstanding, but couldn't you just use a code snippet like below to grab your PAT, stored as a secret?
var plaintextPAT: string
config.requireSecret("pat").apply(secret => plaintext = secret)


LLM streaming formatting is buggy

The width calculation seems to be wrong, and code blocks sometimes get lost.

closed by 82d5a5abff0fd2f44a25e648b005d87e37e6f12c


Format with black and markdownlint

Changes

Formatted .py files with black
Formatted README.md with markdownlint

Would you mind adding this to the CI?

Should the CI format and commit the changes or fail and leave it to the user to fix it?

Definitely not commit, just fail.
Could you also split it into different jobs? Both the Markdown and Python formatting.

Thank you! This was very useful and well done.


tsm 24 fix hooks

Remove warning message from Jest
Bumped all dependencies, removed PhantomJS.
Fixes byCallIndex error messages
Added anything() to readme
Added spec for case when last function has been called but method was not called at all
Fixed readme with wrong information about atMost validator
objectContaining matcher to output object representation when a test is failing
Added better verification message feature from @johanblumenbergs fork
Rename @cspotcode org to @ts-mockito
merged changes from nagrock-master
upgraded ts to latest + upgraded karma
test(issue15): added the failing tests of issue 15
switch to typestrong npm scope
test(issue15): runtime fix
feat(mocker): removed rxjs
feat(*): run actions per PR
feat(*): run actions per push
feat(*): run actions per push
2.6.1
2.6.2
2.6.3
fix(*): fixed vulns
fix(*): updated versions and removed redundancies
fix(publish): restructure the publish back to original structure
fix(publish): fixed lock file
fix(publish): wired publish hook
fix(publish): fixing publish hook

Sorry I didn't mean to press the "delete branch" button.  I'm using the GitHub mobile app and tapped the wrong thing.


WebAPI should notify when trace is received that matches filter

When the WebAPI receives a trace, it receives in the form ExportTraceServiceRequest. This request is a list of ResourceSpans, which contain a list of ScopeSpans, which contain a list of Spans.
To keep searching and storage simple, I propose we flatten the data structure with our own Span implementation that includes the parent scope span and resource span data. The structure of this object should be:
classDiagram
    class FlattenedSpan{
        // TODO - flatten and document
    }

Upon receipt of an ExportTraceServiceRequest, the API should extract and add each trace (including the resource and scope information flattened) to the test harness.
The test harness will write new FlattenedSpans to IMemoryCache, where the key is span-{spanId}, to guarantee uniqueness across all spans and distinguish between the various signals. In addition, the test harness will maintain a list of Channel<FlattenedSpan>, and it will write the FlattenedSpan to each channel.
The above functionality requires the ability to add a Channel to the list of channels that the trace test harness is maintaining.
Upon receipt of a new grpc request for a trace, a channel should be created and added to the list of channels. Upon adding the channel, the test harness should also grab every value in IMemoryCache that is prefixed with span and write it to the list.
The grpc request should be SpanQueryService.First that takes a SpanQueryRequest object. The SpanQueryRequest is defined as:
classDiagram
    class SpanQueryRequest{
        // TODO - document
    }

Upon completion (pass|fail) of the request, the test harness should remove the channel from the list of channels it is maintaining and dispose.
The response to .First should be a SpanQueryResponse that includes a list of spans with a single item.
classDiagram
    class SpanQueryResponse{
        // TODO - document
    }

This also requires creating .proto files for the service and the request/response. Upon generation of the proto files, the base service classes will be generated, at which point you can extend them with the SpanQueryService functionality.

I'll start creating the FlattenedSpan for now.
classDiagram
    class FlattenedSpan{
        +string ServiceName
        +string ServiceInstanceId
        +string InstrumentationScopeName
        // TODO - keep adding
    }

I'll start creating the FlattenedSpan for now.
classDiagram
    class FlattenedSpan{
        +string ServiceName
        +string ServiceInstanceId
        +string InstrumentationScopeName
        // TODO - keep adding
    }

I think rather than put things like ServiceName at the root, we should have a resource object attached to the span instead. Let's try this out:
classDiagram
    class FlattenedSpan {
        +Resource Resource
    }

    class Resource {
        +Service Service
    }

    class Service {
        +string Name
    }

    FlattenedSpan --* Resource
    Resource --* Service

And then you can Spans.First(span => span.Resource.Service.Name == "service1");

This link shows what are required vs recommended attributes. Should be useful in determining what we can parse out of the dictionary. https://opentelemetry.io/docs/specs/semconv/resource/

What do you think about just using the ExportTraceServiceRequest for querying traces?
I put together this basic POC to ONLY show how we can just use the ExportTraceServiceRequest for querying traces. This simplifies the need to add any new classes for flattening the span. It also gives the user the ability to also get back the entire exported trace and check properties on it.
The channel and ReadAllAsync implementation in the POC is not the recommended way and is purely used to show one approach for how the FilterDelegate can be used to filter out the ExportTraceServiceRequest.
Our Request & Response objects used by the Service could also include the ExportTraceServiceRequest as a property. That way we're using an already established proto that users are familiar with.

I see where you're going with that, but I don't think we want to do that. ExportTraceServiceRequest (and all the other proto classes) are intended to be over-the-wire. I would advocate we think of them like DTOs, and we should avoid using them in the core library. Check out IsInitialized, MergeFrom, WriteTo, etc. Those are all methods on the Request that are inherited from IMessage in the Google Protobuf library. I don't think we want that floating around for our internal representation of the traces, especially because we may need to select a different data structure for eg. performance.

Here's what I propose (for now) for the span structure:
classDiagram
  class Span {
    +Scope Scope
    +string Name
    +Dictionary~string, object~ Attributes
    +List~SpanEvent~ Events
    +unit Flags
    +SpanKind Kind
    +List~SpanLink~ Links
    +SpanStatus Status
    +byte[] SpanId
    +byte[] TraceId
    +string TraceState
    +byte[] ParentSpanId
    +ulong EndTimeUnixNano
    +ulong StartTimeUnixNano
  }

  class SpanEvent {
    +string Name
    +Dictionary~string, object~ Attributes
    +uint TimeUnixNano
  }

  class SpanKind {
    0 Unspecified
    1 Internal
    2 Server
    3 Client
    4 Producer
    5 Consumer
  }

  class SpanLink {
    +byte[] SpanId
    +Dictionary~string, object~ Attributes
    +uint Flag
    +byte[] TraceId
    +string TraceState
  }

  class SpanStatus {
    +SpanStatusCode Code
    +string Message
  }

  class SpanStatusCode {
    0 Unset
    1 OK
    2 Error
  }

  class Scope {
    +string Name
    +Dictionary~string, object~ Attributes
    +string Version
    +string SchemaUrl
    +Resource Resource
  }

  class Resource {
    +Dictionary~string, object~ Attributes
    +string SchemaUrl
  }

Span --* Scope
Span "1" --* "*" SpanEvent
Span --* SpanKind
Span "1" --* "*" SpanLink
Span --* SpanStatus
SpanStatus --* SpanStatusCode
Scope --* Resource

As I'm thinking about it,  it's more of an "inverted" span rather than a "flattened" span. Maybe we just call it Span for now? And rather than ScopeSpan it's just Scope, and ResourceSpan is just Resource.
We may eventually want to group all spans associated with a trace under some sort of trace tree, but I advocate we see how this performs first.

IMemoryCache doesn't have a way to iterate through each item, so we're going to have to take a different approach. I'm thinking we use a ConcurrentBag<Span> for the source-of-truth Span list, and each operation against that collection should remove any items in the collection that are older than the configured timeout.
For the list of Channels, let's also use a ConcurrentBag<Channel> to hold the collection of channels that need to be notified of Spans. System.Threading.Concurrent because Channels are going to be added to the collection at the same time as the test harness is attempting to iterate the channels to provide updates, so that needs to be synchronized.

classDiagram
  class ISignalList~TSignal~ {
    +void Add(TSignal signal)
  }

  class SpanSignalList {
    -IChannelManager~Span~ _channels
    +void Add(Span span)
    +List~Span~ Query(SpanQueryRequest request)
  }

  class IChannelManager~TSignal~ {
    +void Notify(TSignal signal)
  }

  class SpanChannelManager {
    +void Notify(Span span)
  }

  class TraceService {
    -ISignalList~Span~ _spans
  }

  class SpanQueryService {
    -ISignalList~Span~ _spans
    -IChannelManager~Span~ _channels
    +SpanQueryResponse Query(SpanQueryRequest request, ServiceContext context)
  }

  SpanSignalList --|> ISignalList
  TraceService --* ISignalList : ISignalList~Span~
  SpanChannelManager --|> IChannelManager : IChannelManager~Span~
  SpanSignalList ..> IChannelManager : IChannelManager~Span~
  SpanQueryService --* ISignalList : ISignalList~Span~
  SpanQueryService --* IChannelManager : IChannelManager~Span~

Created the new Span class. Made these classes vs records because one…
… instance can be created and updated in different areas of the code.
I tried to set the properties as required, nullable or with a default based on the proto definitions and OpenTelemetry definitions online.

Hmm, what does "one instance can be created and updated in different areas of the code" mean?


Add minimap to text_edit

Adds an optional text minimap to text_edit similar to that of sublime text.
It supports wrapping, code folding, and scrolling. In addition it uses full syntax highlighting, current line and viewport outline. However does not highlight beyond that eg searches.  It currently draws the characters as blocks.
Due to the extra processing, there was a significant performance hit around syntax highlighting, to combat this text_edit will now cache syntax highlighting colours similar to that of colour regions.
Preview:

closes #4881

It currently draws the characters as blocks.

You can make blocks look closer to actual characters by toning down their opacity slightly (try 70% opacity).

I think the width should be exposed to the settings, and it's default should be the equivalent of the default line guideline (80).

I have attempted something similar before, so here are my findings:

You can make the minimap look better by drawing textured rectangles instead of full blocks (the texture being just 1x2 with the top pixel half-transparent), and draw them without gap to fit more lines, like so:

I did some benchmarking for my C++ version and quickly found that even when caching syntax coloring into an O(1) container (right next to characters, really), the drawing operation was the slowest part, just enough to almost halve framerate compared to not drawing the minimap (to the point I ditched Godot for that project and use SDL instead, which was twice as fast). The Godot editor is already quite crowded, a minimap can add up a lot. Did you check draw time and framerate?

You are drawing every character individually, which may be even slower than what I attempted because it multiplies draw calls. I found batching similarly-coloured blocks together brings improvements.

@Zylann I guess this feature should be disabled by default until the performance bottlenecks are solved. To me, it seems better to have a smooth editor without a minimap compared to a sluggish editor with a minimap.

@Zylann Just did a single run benchmark using OS::get_singleton()->get_ticks_usec(); at the start and end of text_edits draw notification.
Here's the mean results:

No Highlighter Cache, Minimap Diabaled (As it is currently): 8326.3560277537
No Highlighter Cache, Minimap Enabled: 14210.949481865
Highlighter Cache, Minimap Disabled: 6660.104175635
Highlighter Cache, Minimap Enabled: 9555.1651455889

Added a setting for the minimap width and it now slightly fades out the font colours. preview:

By the way, I think the options should be exposed to scripting.

Fixed syntax highlighter cache not clearing properly.
Fixed minimap syntax highlighting when wrapping.
Fixed minimap initial indent when wrapping.
The minimap will now indent more for tabs then spaces.
Exposed to scripting

The minimap now draws blocks of the same colours rather then one char at a time, resulting is a small performance boost  from from 9555 to around 9000 mean.

Thanks!

Shouldn't we enable the minimap by default? Otherwise most users won't even notice that the feature exists.


4 trivial edits to Concepts/Storage

I found multiple trivial errors in the Concepts/Storage section of the documentation, which I fixed.

/retitle Make 3 edits within Storage concepts