Louis/node-unit-tests

Purpose
Added unit test templates for every node to get things started. Each node only contains a loading test right now, but figured it would be a good idea to merge these templates now for consistency in case others start adding unit tests.
Added a helper class for providing common unit test functionality. Given that many of the nodes will probably be tested in similar ways, figured it would be a good idea to create a single file where that common functionality can be called from.
Also updated the projects dependencies.
Changes

Added node-red-test-helper dev dependency (0fbb8e1bcf5be8a7dfaff5d815b0213bf3609151).
Updated dependencies (07a44a88d39d39633f42fe6e4a2061e93b57fa6e).
Added unit test helper class (bc371b3fb17d5492fb1d1765df4028d91790759d).
Added test templates for each node (ab2f2fc0e34bb90ae9ad585d6df14de8d10f46dc).

New Requirements

New dev dependency node-red-test-helper has been added.

Nice work, Louis! Just one thing, instead of using .test.js for the file names, I think Node-RED convention is to use _spec.js for the tests so that it works with the node-red-test-helper framework. This is my understanding based on the unit testing guidance here: https://nodered.org/docs/creating-nodes/first-node#testing-your-node-in-node-red

There are two standard conventions for naming test files in JS: *.test.js and *.spec.js. The one used by the node-red-test-helper is just the convention they went with, though using a different one shouldn't stop it from working. The only purpose of the name is so that you can specify a glob for Mocha to look for when searching for test files, other than clarification on the purpose of the file. Personally, I find using the word "test" to be clearer than "spec", as it's more immediately clear what the purpose of the file is. What do you think?

Nice work, Louis! Just one thing, instead of using .test.js for the file names, I think Node-RED convention is to use _spec.js for the tests so that it works with the node-red-test-helper framework. This is my understanding based on the unit testing guidance here: https://nodered.org/docs/creating-nodes/first-node#testing-your-node-in-node-red

There are two standard conventions for naming test files in JS: *.test.js and *.spec.js. The one used by the node-red-test-helper is just the convention they went with, though using a different one shouldn't stop it from working. The only purpose of the name is so that you can specify a glob for Mocha to look for when searching for test files, other than clarification on the purpose of the file. Personally, I find using the word "test" to be clearer than "spec", as it's more immediately clear what the purpose of the file is. What do you think?

Ah ok. While I do agree with you that "test" would be the clearer terminology to use, I'm not sure that we should diverge from the way chosen for Node-RED, since we're making a Node-RED palette. @Theo-Reignier what do you think about the naming of the test files?

Nice work, Louis! Just one thing, instead of using .test.js for the file names, I think Node-RED convention is to use _spec.js for the tests so that it works with the node-red-test-helper framework. This is my understanding based on the unit testing guidance here: https://nodered.org/docs/creating-nodes/first-node#testing-your-node-in-node-red

There are two standard conventions for naming test files in JS: *.test.js and *.spec.js. The one used by the node-red-test-helper is just the convention they went with, though using a different one shouldn't stop it from working. The only purpose of the name is so that you can specify a glob for Mocha to look for when searching for test files, other than clarification on the purpose of the file. Personally, I find using the word "test" to be clearer than "spec", as it's more immediately clear what the purpose of the file is. What do you think?

Ah ok. While I do agree with you that "test" would be the clearer terminology to use, I'm not sure that we should diverge from the way chosen for Node-RED, since we're making a Node-RED palette. @Theo-Reignier what do you think about the naming of the test files?

I do not know how much the testing file naming conventions are important.
However, since there is no need to make the file names user-friendly, I would stick with the Node-RED conventions.

@KhesimReid I've pushed a commit to update the names 👍

@KhesimReid I've pushed a commit to update the names 👍

Thanks Louis!


Invalid params. Invalid parameter "/": the parameter "host" is missing.

Describe the bug
Upon upgrading to the latest versions of Zabbix, Grafana and Grafana-zabbix plugin, I am encountering the following error on all panels whose datasource is zabbix:
Invalid params. Invalid parameter "/": the parameter "host" is missing.
Appreciate your help on this topic as all our dashboards are currently not working.
Expected behavior
Dashboard and exploring zabbix data should be successful
Screenshots
The following error is displayed when a dashboard is loaded and also when the 'Save and Test' button within the zabbix datasource configuration page is selected.

Network data

Software versions

Grafana
Zabbix
Grafana-Zabbix Plugin

10.3.1
6.4.11
4.4.5

hi @clibon , could you describe for us how to reproduce the issue? does this happen in explore too? could you describe a minimal query that triggers this error?

Hi @gabor. The problem also happens within the explore tool and what we noted is that post upgrade the error message is displayed every time a query or test connection is performed against the grafana-zabbix plugin.
Unfortunately I cannot provide the exact query as in the meantime we have reverted back to an older spec since it was affecting production tools. Thanks for your query.

@clibon thanks for the info.. just to clarify, which component(s) did you revert to fix the problem?

grafana?
grafana zabbix plugin?
zabbix?

Hi @gabor. We actually had to revert the complete VM to a snapshot taken before the upgrade started.
As when we attempted to downgrade grafana or the grafana-zabbix plugin, both resulted in the same error.
By reverting back we are now fully operational once again.

i see, thanks for the info 👍 .unfortunately i am not able to reproduce the issue, so we'll have to close the issue. if you find any more info in the future, just reply here, and we can reopen the issue.

Hi,
we came across the same issue, this time using the following component versions:
Zabbix: 6.4.12
Grafana: 10.4.0
Grafana-Zabbix Plugin: 4.4.6
Could you kindly assist?
Thanks.

Upon further investigations, I noticed that following behaviour when selecting the 'Save & test' button on the data source page. All dumps taken from Chrome's developer mode:
This request works fine:
Payload ->

Response ->

From now onwards, exceptions are reported from the Zabbix-API:
Request 1:
Payload ->

Response ->

Request 2:
Payload ->

Response ->

UPDATE: The issue has now been resolved as follows.
The grafana user on zabbix was not part of any group, so we created a new group called API Group and gave read-only access to the hosts.
Issue can be closed and thanks.

i'm glad it worked out at the end. closing issue as requested.


Update generate-configuration.zh.md

fix typro mistake

I'm assuming this is a correct correction :]


Remaining UNSAFE_componentWillUpdate

@orangemug, while looking into https://github.com/maputnik/editor/issues/467, I noticed:
https://github.com/maputnik/editor/blob/cec87765fc49affa4090efb54f37c91825b04424/src/components/modals/AddModal.jsx#L56
Was it omitted intentionally, when working on https://github.com/maputnik/editor/pull/358?

The above mentioned lifecycle hook is the only one left with UNSAFE_ prefix.


Add support for dictConfig

logging.config.dictConfig is an extremely powerful configuration tool, especially when using more complicated logging pipelines such as with StructLog. It would be very useful to have it in picologging.
The implementation is mainly in the DictConfigurator class, logging.config.dictConfig just defers to it.

Thank you very much!
This project looks awesome!

+1 to this feature


Issue with Phone Authentification

Hello,
at the moment im working with the tdlib java Example, which i managed to add to my existing java project using IntelliJ.
Now i ran into the problem of an expired authentication code and it seems to have stuck in this state:

I do know that there is a method i can call to resend the authentication code during the AuthorizationStateWaitCode-State:
            case TdApi.AuthorizationStateWaitCode.CONSTRUCTOR: {
                client.send(new TdApi.ResendAuthenticationCode(), defaultHandler);
                String code = promptString("Please enter authentication code: ");
                client.send(new TdApi.CheckAuthenticationCode(code), new AuthorizationRequestHandler());
                break;
            }

When i execute the programm like it is shown above, i do not revceive another code on my phone.
Anyone can help?

ResendAuthenticationCode works only when there is nextType in authorizationStateWaitCode.codeInfo. If you receive PHONE_CODE_EXPIRED error, you need to restart authorization. You should be able to do this by restarting the example.

Not working... The authorization-process does initialize the connection process by  case TdApi.AuthorizationStateWaitTdlibParameters.CONSTRUCTOR, but jumps from there back to the  case TdApi.AuthorizationStateWaitCode.CONSTRUCTOR: waiting for code.
Is there any other way to receive a new authentication code or restart the authentication process?

Authorization state is reset to AuthorizationStateWaitPhoneNumber in 5 minutes. There are no explicit way to reset authorization state in the example.


[arethetypeswrong] Plugin types are wrong

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

Describe the bug
https://arethetypeswrong.github.io/?p=%40stylistic%2Feslint-plugin%402.4.0

Incorrect default export
The resolved types use export default where the JavaScript file appears to use module.exports =. This will cause TypeScript under the node16 module mode to think an extra .default property access is required, but that will likely fail at runtime. These types should use export = instead of export default.

Reproduction
TS playground

Yeah sadly TS doesn't allow you to have both an export = and an ESM export -- even if the other ESM exports are just types.
One option is to use a namespace to merge the types onto the export ='d name, eg:
import { Linter, ESLint, Rule } from 'eslint';
import { UnprefixedRuleOptions as UnprefixedRuleOptions_ } from './rule-options.js';
import { RuleOptions as RuleOptions_ } from './rule-options.js';
import '@stylistic/eslint-plugin-js';
import '@stylistic/eslint-plugin-jsx';
import '@stylistic/eslint-plugin-ts';
import '@stylistic/eslint-plugin-plus';

declare namespace plugin {
    export type Rules = { ... };
    export interface StylisticCustomizeOptions<Flat extends boolean = true> { ... }
    export type UnprefixedRuleOptions = UnprefixedRuleOptions_;
    export type RuleOptions = UnprefixedRuleOptions_;
}
declare function customize(options: plugin.StylisticCustomizeOptions<false>): Linter.BaseConfig;
declare function customize(options?: plugin.StylisticCustomizeOptions<true>): Linter.Config;

declare const _configs: { ... };
declare const configs: ESLint.Plugin["configs"] & typeof _configs;

declare const plugin: {
    rules: plugin.Rules;
    configs: typeof configs;
};

export = plugin;

This will allow people to access the types via type T = plugin.Rules;, etc.
But this does mean that people can't import the types directly like import { type Rules } from  '@stylistic/eslint-plugin'.

I believe that explicitly setting type: "commonjs" would also fix it without any changes to the types.
For example for typescript-eslint that's what we do and it works as expected.

I wanted to push ESLint ecosystem towards ESM since now we have the flat config. I don't really want to go backward to use CJS conversions

I definitely understand this! But until your tool fully drop support for eslintrc configs you're stuck supporting CJS 😢.
If you want to dual-emit CJS+ESM then there are tricks you can do to support types for that situation but things do get a bit complicated IIRC.

I believe that explicitly setting type: "commonjs"

Do you mean by doing it in the package.json? It doesn't seems to solve it from my local reproduction

I would compare things with our typescript-eslint package which is a CJS packaged authored in .ts files in ESM and built to .js files in CJS.
https://arethetypeswrong.github.io/?p=typescript-eslint%408.1.0
I think the main difference is the use of type: commonjs? Perhaps there's something else I missed.


Fix invalid docs.rs targets syntax

The target field does nothing. There are two options you could have been trying to set:

default-target, the target built for the main page (docs.rs/stm32h7xx-hal)
targets, which controls all targets built.

This changes it to the second since I assume stm32h7xx-hal is the only target you want to build, but I can change it to the first if you want to build all tier one platforms instead.

Thanks! You're correct, this crate is targeting thumbv7em-none-eabihf only.
bors r+


problem in activation of both tensorflow and rdkit in anaconda win

I have a problem in activation of both tensorflow and rdkit(2019.03.4) in anaconda  windows (10)
The activation of one of them leads to deactivation of another!

I expect that there is a conflict with lib boost.  We may need some more information.  Could you send/post the environment file?
conda env export > environment.yml

@fi000 any update on this?


Should integrate ImageStacks, Animator, SemiLandmarks into the Lab module

Still need the ImageStacks and Animator.

I thought the plan was just to drop the modules into the customized version?

They are already present in the customized version. But we need a more manageable way of incorporating changes in external modules. Clone, copy/replace existing module, and repackage is error prone. Having them under the main package will make things a lot more straightforward, and easier for extension users as well. But I am open to suggestions.

While this pull request is still pending and under discussion, we will need to restart the application when switching module versions.  I don't think this is terrible, just inconvenient.


[Lookahead Quality Score] might have a bug

This method was not reduceing the quality score even though my node was connected to 4 peers.
the peer count inside the quality score service was showing just one peer.
        protected void OnStalling(ChainedBlock chainedBlock)
        {
            this.logger.LogTrace("({0}:'{1}')", nameof(chainedBlock), chainedBlock.HashBlock);
            BlockPullerBehavior behavior = null;

            lock (this.lockObject)
            {
                this.assignedBlockTasks.TryGetValue(chainedBlock.HashBlock, out behavior);
            }

            if (behavior != null)
            {
                double penalty = this.peerQuality.CalculateNextBlockTimeoutQualityPenalty();
                this.logger.LogTrace("Block '{0}' assigned to peer {1:x}, penalty is {2}.", chainedBlock.HashBlock, behavior.GetHashCode(), penalty);

                behavior.UpdateQualityScore(penalty);
                if (Math.Abs(behavior.QualityScore - QualityScore.MinScore) < 0.00001)
                {
                    behavior.ReleaseAll(false);
                    this.AssignPendingVectors();
                }
            }
            else
            {
                this.logger.LogTrace("Block '{0}' not assigned to any peer.", chainedBlock.HashBlock);
                this.AssignPendingVectors();
            }

            this.logger.LogTrace("(-)");
        }

When I manually called
 behavior.ReleaseAll(false);

All continued fine.

need to see the logs for this

i believe this has nothing to do with quality score, my guess is there was a problem with block downloading, but can't tell really without further information

I have the node.txt file but I am not sure it will be very helpful
node.txt

Your log does not show anything unusual. I tend to believe the issue you have witnessed is not related to QS function but rather to the problems with syncing we've had in last couple of days.

close it or have we seen this again?


HADOOP-17199. S3A Directory Marker HADOOP-13230 backport to branch-3.1

Branch-3.1 cherrypick of #2210
This backports the listing-side changes of HADOOP-13230.
With this patch in, this branch of Hadoop is compatible with S3A clients
which do not delete directory markers when files are created underneath.
It does not allow this version to disable marker deletion; if the
fs.s3a.marker.retention option is changed to request this, a message
is printed at INFO and the policy remains at "delete"
The s3guard bucket-info command has been extended to support
probing for marker retention, as has the hasPathCapability method on
S3AFileSystem.
Read the documentation!
NOTICE
Please create an issue in ASF JIRA before opening a pull request,
and you need to set the title of the pull request which starts with
the corresponding JIRA issue number. (e.g. HADOOP-XXXXX. Fix a typo in YYY.)
For more details, please see https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute

Test run against London with s3guard. Failures as the SDK/tests aren't ready for 0 provisioned IO Capacity; and something is up with AssumedRole permissions. There's changes in that test for branch-3.2, which I assume "make it go away". This patch doesn't go near s3guard
[ERROR] Failures:
[ERROR]   ITestDynamoDBMetadataStore.testProvisionTable:588->Assert.assertEquals:542->Assert.assertEquals:555->Assert.assertEquals:118->Assert.failNotEquals:743->Assert.fail:88 expected:<640> but was:<320>
[ERROR] Errors:
[ERROR]   ITestAssumeRole.testAssumeRolePoliciesOverrideRolePerms:336 » AWSServiceIO ini...
[ERROR]   ITestAssumeRole.testAssumeRoleRestrictedPolicyFS:309 » AWSServiceIO initTable ...
[ERROR]   ITestAssumeRole.testAssumedInvalidRole:150->lambda$testAssumedInvalidRole$1:152 » IllegalArgument
[ERROR]   ITestAssumeRole.testPartialDelete:727->executePartialDelete:759 » AWSServiceIO
[ERROR]   ITestAssumeRole.testPartialDeleteSingleDelete:735->executePartialDelete:759 » AWSServiceIO
[ERROR]   ITestAssumeRole.testReadOnlyOperations:355 » AWSServiceIO initTable on stevel-...
[ERROR]   ITestAssumeRole.testRestrictedCommitActions:621 » AWSServiceIO initTable on st...
[ERROR]   ITestAssumeRole.testRestrictedRename:435->executeRestrictedRename:469 » AWSServiceIO
[ERROR]   ITestAssumeRole.testRestrictedSingleDeleteRename:444->executeRestrictedRename:469 » AWSServiceIO
[ERROR]   ITestAssumeRole.testRestrictedWriteSubdir:405 » AWSServiceIO initTable on stev...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestAssumedRoleCommitOperations.setup:81 » AWSServiceIO initTable on stevel-l...
[ERROR]   ITestS3GuardToolDynamoDB>AbstractS3GuardToolTestBase.testDestroyNoBucket:231->AbstractS3GuardToolTestBase.run:95 » IllegalArgument
[ERROR]   ITestS3GuardToolDynamoDB.testDynamoDBInitDestroyCycle:224->AbstractS3GuardToolTestBase.exec:286->AbstractS3GuardToolTestBase.exec:308 » AWSServiceIO
[ERROR]   ITestS3GuardToolLocal>AbstractS3GuardToolTestBase.testDestroyNoBucket:231->AbstractS3GuardToolTestBase.run:95 » IllegalArgument

:broken_heart: -1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
12m 14s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  1s
No case conflicting files found.

+0 :ok:
markdownlint
0m  0s
markdownlint was not available.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s
The patch appears to include 10 new or modified test files.

_ branch-3.1 Compile Tests _

+0 :ok:
mvndep
2m 59s
Maven dependency ordering for branch

-1 :x:
mvninstall
2m 21s
root in branch-3.1 failed.

-1 :x:
compile
1m  7s
root in branch-3.1 failed.

-0 :warning:
checkstyle
0m 50s
The patch fails to run checkstyle in root

-1 :x:
mvnsite
0m 39s
hadoop-common in branch-3.1 failed.

-1 :x:
mvnsite
0m 33s
hadoop-aws in branch-3.1 failed.

-1 :x:
shadedclient
2m 57s
branch has errors when building and testing our client artifacts.

-1 :x:
javadoc
0m 24s
hadoop-common in branch-3.1 failed.

-1 :x:
javadoc
0m 11s
hadoop-aws in branch-3.1 failed.

+0 :ok:
spotbugs
4m 25s
Used deprecated FindBugs config; considering switching to SpotBugs.

-1 :x:
findbugs
0m 39s
hadoop-common in branch-3.1 failed.

-1 :x:
findbugs
0m 12s
hadoop-aws in branch-3.1 failed.

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 40s
Maven dependency ordering for patch

-1 :x:
mvninstall
0m 36s
hadoop-common in the patch failed.

-1 :x:
mvninstall
0m 10s
hadoop-aws in the patch failed.

-1 :x:
compile
1m  3s
root in the patch failed.

-1 :x:
javac
1m  3s
root in the patch failed.

-0 :warning:
checkstyle
0m 43s
The patch fails to run checkstyle in root

-1 :x:
mvnsite
0m 38s
hadoop-common in the patch failed.

-1 :x:
mvnsite
0m 12s
hadoop-aws in the patch failed.

-1 :x:
whitespace
0m  0s
The patch has 10 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply

-1 :x:
shadedclient
0m 49s
patch has errors when building and testing our client artifacts.

-1 :x:
javadoc
0m 15s
hadoop-common in the patch failed.

-1 :x:
javadoc
0m 12s
hadoop-aws in the patch failed.

-1 :x:
findbugs
0m 37s
hadoop-common in the patch failed.

-1 :x:
findbugs
0m 11s
hadoop-aws in the patch failed.

_ Other Tests _

-1 :x:
unit
0m 36s
hadoop-common in the patch failed.

-1 :x:
unit
0m 11s
hadoop-aws in the patch failed.

+1 :green_heart:
asflicense
0m 30s
The patch does not generate ASF License warnings.

33m  7s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/2247

Optional Tests
dupname asflicense compile javac javadoc mvninstall mvnsite unit shadedclient findbugs checkstyle markdownlint

uname
Linux ea6d61c7ffc5 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.1 / 0808cb6

Default Java
Oracle Corporation-9-internal+0-2016-04-14-195246.buildd.src

mvninstall
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/branch-mvninstall-root.txt

compile
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/branch-compile-root.txt

checkstyle
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/buildtool-branch-checkstyle-root.txt

mvnsite
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/branch-mvnsite-hadoop-common-project_hadoop-common.txt

mvnsite
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/branch-mvnsite-hadoop-tools_hadoop-aws.txt

javadoc
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/branch-javadoc-hadoop-common-project_hadoop-common.txt

javadoc
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/branch-javadoc-hadoop-tools_hadoop-aws.txt

findbugs
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/branch-findbugs-hadoop-common-project_hadoop-common.txt

findbugs
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/branch-findbugs-hadoop-tools_hadoop-aws.txt

mvninstall
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/patch-mvninstall-hadoop-common-project_hadoop-common.txt

mvninstall
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/patch-mvninstall-hadoop-tools_hadoop-aws.txt

compile
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/patch-compile-root.txt

javac
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/patch-compile-root.txt

checkstyle
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/buildtool-patch-checkstyle-root.txt

mvnsite
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/patch-mvnsite-hadoop-common-project_hadoop-common.txt

mvnsite
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/patch-mvnsite-hadoop-tools_hadoop-aws.txt

whitespace
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/whitespace-eol.txt

javadoc
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/patch-javadoc-hadoop-common-project_hadoop-common.txt

javadoc
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/patch-javadoc-hadoop-tools_hadoop-aws.txt

findbugs
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/patch-findbugs-hadoop-common-project_hadoop-common.txt

findbugs
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/patch-findbugs-hadoop-tools_hadoop-aws.txt

unit
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/patch-unit-hadoop-common-project_hadoop-common.txt

unit
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/artifact/out/patch-unit-hadoop-tools_hadoop-aws.txt

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/testReport/

Max. process+thread count
151 (vs. ulimit of 5500)

modules
C: hadoop-common-project/hadoop-common hadoop-tools/hadoop-aws U: .

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2247/1/console

versions
git=2.7.4 maven=3.3.9

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

failures in hadoop-common, which would seem utterly unrelated
[ERROR] COMPILATION ERROR :
[INFO] -------------------------------------------------------------
[ERROR] /home/jenkins/jenkins-home/workspace/hadoop-multibranch_PR-2247/src/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/io/nativeio/NativeIO.java:[481,16] error: cannot find symbol
[ERROR] /home/jenkins/jenkins-home/workspace/hadoop-multibranch_PR-2247/src/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/crypto/CryptoStreamUtils.java:[40,20] error: cannot find symbol
[INFO] 2 errors

did the local build of a full distro; ran aws tests, all good. so merging


[0.16] Comments at the end of a pattern matching block raising a syntax problem

(With 0.16-alpha2)
Ending a pattern matching block with a comment fails:
type Foo = Bar | Baz -- | Buzz

fooToString : Foo -> String
fooToString x =
  case x of
    Bar -> "bar"
    Baz -> "baz"
    -- Buzz -> "buzz"

-- SYNTAX PROBLEM ------------------------------------------------------
I ran into something unexpected when parsing your code!
I am looking for one of the following things:
    whitespace

This was working fine on the previous alpha.

Oh, nice find! This is because of 6e1a9e1694ad150a8ac37f925d2a3009aaefa3af fixing #1083. I'll take another look at it for a fix.

Interesting, it seems that the amount of whitespace after the comment changes things. If the end of file comes at the end of the line comment, this is the error:
-- SYNTAX PROBLEM ------------------------------------------------------ tmp.elm

I ran into something unexpected when parsing your code!

8│     -- Buzz -> "buzz"
                        ^
I am looking for one of the following things:

    "\n"
    "\r\n"
    end of input
    whitespace

Alright, this should be fixed by #1101, thanks for the report!


chore: change deprecated excludes_analyse to excludePaths in README.md

[ ] Added or updated tests
[ ] Documented user facing changes
[ ] Updated CHANGELOG.md

Changes
I updated the example phpstan.neon snippet in the README.md to use the new excludePaths parameter. Relates to #759.

Thank you, good catch!

Thank you, good catch!


NanoPB utilities for FieldValue migration

This is part of #7904. It will break the feature branch, but I am planning to send out small reviewable chunks with an end goal of a feature branch that passes CI.
This PR adds a couple of new classes and methods that are used in the FieldValue port.
Example usage:

SharedMessage is a shared pointer around Message, which allows a single Protobuf value to be used in multiple classes. This makes it possible for a Query and a Target to contain the same Filter and Bounds. See https://github.com/firebase/firebase-ios-sdk/pull/7904/files?file-filters[]=.cc&file-filters[]=.h&file-filters[]=.mm&file-filters[]=.txt#diff-ca2b3b9278f6b19636908d4f2da32d8657081510e6ffb60b823d5766992bf263R87

SetRepeatedField simplifies initialization of nanopb repeated fields. See https://github.com/firebase/firebase-ios-sdk/pull/7904/files?file-filters[]=.cc&file-filters[]=.h&file-filters[]=.mm&file-filters[]=.txt#diff-68ee4b0277a710dcf518d3f8aba6cd6190ae73dc04840aee102c7a6ba5627944R368

ReleaseFieldArray is used to transform ownership of nanopb objects from the Serializer to ObjectValue (and hence the documents): https://github.com/firebase/firebase-ios-sdk/pull/7904/files?file-filters[]=.cc&file-filters[]=.h&file-filters[]=.mm&file-filters[]=.txt#diff-243c7453f0512236aeb3cb21fdc2ea18b9c280b05bf7f8a105c32b82b823a145R229

ResizeFieldArray is only used in test to allow the tests to build up maps incrementally: https://github.com/firebase/firebase-ios-sdk/pull/7904/files?file-filters[]=.cc&file-filters[]=.h&file-filters[]=.mm&file-filters[]=.txt#diff-f89aa393ca7ce959593fe159bff272ef460b58a99592b3b8563ca73b446083f0R163

I wonder if we need a helper class to wrap Nanopb arrays, similar to Message? (not for this PR)

I am using ArrayValue instead of a vector of values in the implementation which allows me to re-use the existing Message class.


Add Zipkin exporter support

Zipkin exporter is available now in otel-java 0.4.0-SNAPSHOT, and will be available soon in the otel-java 0.4.0 release.

can I work on this issue and to be assigned? cannot do it myself.

awesome! assigned to you

blocked by https://github.com/open-telemetry/opentelemetry-java/issues/1182

This should be unblocked now by the opentelemetry-java 0.4.1 release 👍

great, I will proceed.

zipkin exporter uses HttpURLConnection therefore it's self-traced. @trask is it possible to disable HttpURLConnection traces only for zipkin in the auto-exporter?

Oh that's interesting! We don't normally have this problem since we exclude the exporter class loader from any bytecode instrumentation, but HttpURLConnection is in the bootstrap class loader 🙃.
I think probably the easiest thing is for us to configure Zipkin to use OkHttpSender instead, e.g.
ZipkinExporterConfiguration.builder()
    .setSender(OkHttpSender.create(zipkinEndpoint))
    .setServiceName(serviceName)
    .build()

:+1: to what @trask trask said. The most common http reporter used by ZIpkin users is the OkHttp one, it is the default in spring-cloud-sleuth. The URLConnection one exists mostly in the case of classpath issues. (cc @adriancole in case he has any opinions)
That brings up a second thought for me. Can you create an issue to add configuration that would allow for using the other reporter options (kafka, rabbit, etc.)

@devinsba I added OkHttpSender, and created #417 for other options

done via https://github.com/open-telemetry/opentelemetry-auto-instr-java/pull/411
can be closed now


[UIE-97] Resolve warnings in useAnalysisExportState test

Currently, the test for useAnalysisExportState logs two warnings about:
Warning: An update to TestComponent inside a test was not wrapped in act(...).

Wrapping the call to copyAnalysis in act resolves them.

Replaced by #4013, which updates all tests using renderHook.


Properly set sessionId and other correlating info to logs from test runs

Right now the internal logger for our CI test runs uses one sessionId per node process, and doesn't do any logging to help correlate logs back to the test runs themselves in the pipeline.
This work needs to cover both e2e tests and the stress test.

dupe of #5295

@anthony-murphy - not sure why this is a dupe of that one... did you mean a different issue?

This is distinct from the issue mentioned above.  Some specifics here:

Log the pipeline run id
Consider focusing on LoaderId as primary key rather than Session_Id since its interpretation in the code is clearer
Consider reusing existing hostName or hostScenario fields if applicable to concepts here (e.g. Orchestrator v. Runner, Runner ID, e2e test case name)
Check if runtimeVersion and loaderVersion include the patch/prerelease version numbers in them - that would be great too

Primary goal at first is to be able to go back and forth with ease between logs and pipeline runs for investigation etc.

Assigning primarily to Si for March, and myself in case we break this down into multiple items.

This has been broken down into specific work items, closing.  cc @anthony-murphy and @chensixx  as FYI


Fix remove TEMP column in block after filter

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 47.65 seconds
stream load tsv:          548 seconds loaded<PHONE_NUMBER>9 Bytes, about 130 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          29.6 seconds inserted 10000000 Rows, about 337K ops/s
storage size:<PHONE_NUMBER>7 Bytes

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 45.68 seconds
stream load tsv:          546 seconds loaded<PHONE_NUMBER>9 Bytes, about 130 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.6 seconds inserted 10000000 Rows, about 349K ops/s
storage size:<PHONE_NUMBER>7 Bytes


Profiler call graph display

Would be nice to display the call chain like:
http://pypi.python.org/pypi/linesman

:+1:


Switch tabs with Shortcut skips tabs after cycling around

Shortcut for cycling tabs does not operate correctly.

Expected Behavior

Cycle through each tab one by one, and wrap around to the first/last when moving past the edge.
Current Behavior

On Mac, when using Cmd+(Left | Right) for switching tabs works normally. When Cmd+Right is used on the right most tab, it goes to the second tab, then fourth (of four total tabs). From then on it will cycle through the second and fourth tab when using the shortcut until the application is restarted.
Possible Solution

Cycle through tabs with shortcut.
Context

Annoying, who uses the mouse??? :)
Your Environment

Franz Version used:
Operating System and version:
MacOs

Please complete the issue template by adding the Franz version you are using


A bash guide shouldn't use zsh for examples

Instead of using this to demonstrate export:
$ export
SHELL=/bin/zsh
AWS_HOME=/Users/adnanadnan/.aws
LANG=en_US.UTF-8
LC_CTYPE=en_US.UTF-8
LESS=-R

$ echo $SHELL
/usr/bin/zsh

let's omit zsh, which is an alternative to bash. That's confusing, and at cross-purposes with this tutorial.
I suggest updating that part of the README to this:
$ export
AWS_HOME=/Users/adnanadnan/.aws
LANG=en_US.UTF-8
LC_CTYPE=en_US.UTF-8
LESS=-R

$ echo $AWS_HOME
/Users/adnanadnan/.aws

Thanks @josephgrossberg

TY! Good guide; this is nice. 👍


Version with the todos completed

I'm having an issue not quite understanding some of the todos and wondering if you would provide a branch with the completed todos so we can compare with our solutions
FYI, Love the course, I've watched it twice

If you complete all (or most of) the TODOs on your own repo or fork just add a link in the comments here.


Package name should ideally be Jupyter.jl

This just occurred to me a couple of days ago, that this package would be ideally called Jupyter.jl. Of course, I imagine this may be a hard thing to pull off now that the name is well established - but I thought I would at least mention it here.

The name IJulia is inline with the other naming conventions in the Jupyter community.
Jupyter refers to the broader platform (including the notebook frontend as well as the standards about how communication works) whereas the individual kernels are typically named with the I (part of the history of the Jupyter project splitting out of IPython).
So, the Python kernel is actually ipykernel, for R it's IRKernel, etc. and they all implement the Jupyter kernel communication spec.

Duplicate of #401.


feat(auth): add POC for supporting token authentication

This is only a POC for now.
GetAuthToken was not confirmed to work as I don't have a slack-app handy at the moment.
However, looking at the doc, it should work with properly configured Bot, Users, and App-level token without any additional cookie.
E.g. usage:
package main

import (
	"context"
	"fmt"

	slack "github.com/rneatherway/slackclient"
)

func main() {
	c := slack.NewClient("example")

	// this
	if err := c.WithCookieAuth(); err != nil {
		panic(err)
	}

	// or that
	if err := c.WithTokenAuth("xox..."); err != nil {
		panic(err)
	}

	response, err := c.API(context.Background(),
		"POST",
		"chat.postMessage",
		nil,
		[]byte("{\"channel\":\"nobe4-test-room\",\"text\":\"test\"}"),
	)
	fmt.Println(string(response), err)
}

cc https://github.com/rneatherway/gh-slack/pull/57

I've just tested this with a Slack App I had and it works! I'll merge and we can continue on this basis.


Simplify spacing, fixes vertical alignment

Fixes https://github.com/arrowtype/type-x/issues/47

throws workflow rules out of the window Merging this as I need it


CC Connect improvements

https://cockroachlabs.atlassian.net/browse/CC-4538

This change is

@emily-horing To align the Free and Dedicated Quickstart experiences step for step, creating a sql user and authorizing your network would need to be removed from the Dedicated guide. I just tested a new Dedicated cluster to see if this would be possible, but it looks like there is no default option and these need to be set up to proceed.
I think it might be best to leave these steps as-is, but let me know if you had something else in mind.

Makes sense @gemma-shay , just looking to see that the outcomes (ie sql statements in this case) are more aligned than before. This LGTM!


More fixes for 1290

This should fix GetEntityPos, GetEntityModel1 and GetEntityModel2 (the rest of #742).
Euphoria, however, is broken again (due to more obfuscation...) and requires some work to get it working again.
It's way too late here so I'm not gonna do that now.

The memory patterns in the MemoryAccess constructor work fine for v1.0.1180.1, too 👌


New Component: Dashboard Layout

This function solves the problem (这个功能解决的问题)
Most of the applications need dashboard layout to show graphs or metrics in home page. It would be awesome to provide such component in naive-ui.
Expected API (期望的 API)

What does you mean dashboard layout? It's there more specific reference?

With Dashboard layout I mean, a component that allows you to add one or more widgets into it, re-arrange them, resize them etc.,
https://www.google.com/search?q=dashboard+layout
https://www.syncfusion.com/react-ui-components/react-dashboard-layout#:~:text=React Dashboard Layout is a,programmatically or dynamically at runtime.

What you need is a dashbord teamplate.
Compose a few components of naive-ui, you can get a dashbord layout.
We don't need to provide a component.
If you don't want to do it yourself,base on one of dashbord teamplate in https://github.com/naive-ui/awesome-naive also a choice.

What you need is a dashbord teamplate. Compose a few components of naive-ui, you can get a dashbord layout. We don't need to provide a component. If you don't want to do it yourself,base on one of dashbord teamplate in https://github.com/naive-ui/awesome-naive also a choice.

Yes, I think grid + card is somehow enough. If it's not enough I think it's better to use a library for the specific scene.


updated Contributors.md

First PR

@yashasingh, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


<IP_ADDRESS> Status log

This component concerns the recording of each ingest activity status in order to:

Monitor the ingest job status.
Automatically recover failed ingests.
Record warning and other error messages to enable manual intervention if required, for example if expected data are not received.

Status log will belong in "Core Application Database" (potentially shared with other systems including wis2box)? Not WMO Climate Data Model and not specific to OpenCDMS in our wider "OpenCDMS Application Database"
Application should list issues that need to be resolved


How to ES6 import?

See http://twiddle.joostdvrs.com/1c0da49ac7eab2af3d77
In the console, I get the error message: Could not find module views/view-stack
The module views/view-stack.js exists, what is the correct path by which I can import it?
I have tried:
import ViewStack from "views/view-stack";

import ViewStack from "./views/view-stack";

import ViewStack from "../views/view-stack";

import ViewStack from "app/views/view-stack";

I have to look into this a little closer. I'm working on a next iteration which drastically changes how modules are resolved.

@Gaurav0 currently, your app is called demo-app, so lookups need to happen on that path. I've got an issue open to fix that #41.

@joostdevries Relative imports are working though, so the user doesn't really need to know the app name. http://ember-twiddle.com/04afc25368d924967979


WebSockets3: the Revenge of WebSockets

Rebased #14

I tried running this code and added ws:// endpoint as an upstream endpoint under the exact same path as my http:// endpoint and got
2015/10/15 14:26:36 oauthproxy.go:103: mapping path "/" => upstream "http://ip-172-31-6-32.us-west-2.compute.internal:40000"
2015/10/15 14:26:36 oauthproxy.go:108: mapping web sockets "/" => upstream "ws://ip-172-31-6-32.us-west-2.compute.internal:40000"
panic: http: multiple registrations for /

I changed the config to assume that that the websockets is always under a subpath, in this case I am using jupyter notebook so I think that it's under /api/kernels/ .
After applying this patch and changing the config to have different subpaths, it looks like everything is working. I am unsure though if it makes sense to make people specify a specific path like that in the configuration. I think that my original configuration would have been a lot easier for people to use.

I couldn't get this to work either for exactly the same reason that you describe. Since I wasn't able to specify different subpaths, and I'm only proxying an app that needs websockets, I changed this in line 105 of oauthproxy.go:
serveMux.Handle(path, &WebsocketReverseProxy{u.Host, proxy})

In other words, I'm simply always using the WebsocketReverseProxy instead of the UpstreamProxy that was used for http/https traffic. This seems to work fine.
Another suggestion for a less 'brute force' method would be appreciated.

hmm, you could probably make that work by using the same proxy for both and sniffing the request scheme

here: https://github.com/Gamevy/oauth2_proxy/blob/proxy_websockets/websocket_reverse_proxy.go#L70

I think that is more of what nginx does, they just have an option to allow upgrading

+1  looking forward to seeing the new and improved websocket functionality merged.

I was having websockets issues with ipython terminal (terminado) when using this solution (#145). After reverting it and adding soellman's implementation (https://github.com/bitly/oauth2_proxy/pull/201), it started working.


vscode-extensions.elixir-lsp.vscode-elixir-ls: 0.12.0 -> 0.13.0

Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I wondew why nixpkgs is not using a single source... https://github.com/NixOS/nixpkgs/blob/4f50411cfc6cb793479818dec6b88732474a4afe/pkgs/development/beam-modules/elixir-ls/pin.json I had submitted an update here: https://github.com/NixOS/nixpkgs/pull/211639


Ensure Resource Manager installs by default

Change the extension of the install file to zip so it will install by default.
Resolves #4944

@donker I'll go ahead and edit your description, but take a look at this article for future reference on the specific keywords that will ensure the associated issue will be auto-closed from a PR merge.

This change should ONLY be for clean installations. I believe this will impact both

Our build processes are not designed for this. We can examine if this is possible to change or we can accept that upgraded installations have the new module but only have it replace the old one optionally.

We're already renaming zip to resources for the Telerik packages in the Upgrade zip:
https://github.com/dnnsoftware/Dnn.Platform/blob/29abc2ee481aab4c97ea44e100211eda946c7a02/Build/Tasks/CreateUpgrade.cs#L40-L61

getting the new module installed even on upgrades is not a problem IMO. Admins still need to replace the module on the admin page and uninstall DAM (and Telerik).


InlineEdit autoFocus onConfirmHandler instant call

Describe the bug
OnBlur event wrong logic with autoFocus and dropdown menu
Code example
codesandbox
To Reproduce
Steps to reproduce the behavior:

Click on dropdown menu
Click rename
See that onConfirmHandler called instantly

Thank you for reporting!
The issue was fixed in the latest release
https://github.com/semrush/intergalactic/releases/tag/v15.28.1


Issue with Warp

Good evening,
I'm encountering an issue to use comet through warp proxy.
I've created a docker proxy network and I use Nginx as reverse proxy. I have another Stremio solution that works perfectly.
The .env file is properly filled as well.
Do you have any suggestion please?
Thank you for your help.

what's the issue

Hi, Comet doesn’t stream through the proxy. Real debrid shows my real IP address. The warp container logs show nothing as well.

Hi, Comet doesn’t stream through the proxy. Real debrid shows my real IP address. The warp container logs show nothing as well.

what realdebrid shows is irrelevant, also the stream is not proxied through Warp : warp is only used to generate the download link

Hi, thank you for your answer.
I use one account for multiple users (and then different public IP adresses). My other add-on (Stream-Fusion) is forwarding all the traffic though the proxy, in order to avoid debriders bans

flemme d'expliquer à quelqu'un qui comprend rien

C'est toi qui comprend rien, ça ne suffit pas ne générer le lien depuis le proxy. Il ne faut PAS qu'il y ait de streams simultanés avec deux IPs DIFFERENTES. Donc il faut proxifier également la diffusion.
Go se soulager au bois de Boulogne mec.

c'est bien ce que je dis tu comprends rien :)


Ender io and ic2 compatability crash?

http://pastebin.com/wLB7BZ3Q
I'm not sure what to do here.

see #3055


iOS build breaks

git rev-parse HEAD
70de76e696c21da617fd2e6435cf7fedab220db8
ld: symbol(s) not found for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
make: *** [/Users/Shared/Develop/google/tensorflow/tensorflow/contrib/makefile/gen/host_bin/proto_text] Error 1

'[' 2 -ne 0 ']'
echo 'armv7 compilation failed.'
armv7 compilation failed.
exit 1

The rest of the output is here:
https://gist.github.com/john-difool/67126c4ccf5d8201cd1842ca18acc78b

tensorflow/contrib/makefile/compile_ios_protobuf.sh PASSED
make -f tensorflow/contrib/makefile/Makefile TARGET=IOS IOS_ARCH=ARM64   FAILS

I met the same problem

How did you fix this issue?

encounter the same issue.

I have a pending fix in PR #3253. It was caused by an error in the ordering of header and library paths, so the build would pull from global /usr/local installations of different versions of the protobuf library, which didn't match the results from the locally-built protoc.

can work, cool!


does not connect to server

I tried to do it on a remote machine, and in the local network.

You need to use a client such as Cigar2 and connect through it to be able to play.

LMAO


Update on this library

Hi, I found this library when searching for a reliable way to parse with haskell-src-exts files that use cpp directives.
I noticed the last upload of this package on hackage is quite old, is this project still alive? Were its functionalities merged in haskell-src-exts?
What's the recommended way to parse source files with cpp now?

I'm not working on it anymore, and I don't know whether anyone is.


Incorrect string interpolation for char

Hi,
It seems that string interpolation operator treats 'char' as 'int' resulting in incorrect System.String.format parameter.
Steps To Reproduce
https://deck.net/8203d3976e7b6e35c2a5b43e4aaee3cc
public class Program
{
    public static void Main()
    {
        Console.WriteLine($"expected 'Y' but got: {'Y'}");
    }
}

Expected Result
In C# interactive it returns
expected 'Y' but got: Y

Actual Result
In deck it returns:
expected 'Y' but got: 89

It is Deck problem. Online transpiler uses the following transpiler rule
"rules": {
    "boxing": "Plain"
  }

So, boxing functionality is deactivated. If you create a project and install Bridge package then boxing will be active by default and your sample will work correctly.

Here's an updated sample on Deck with a BoxingRule override.
https://deck.net/a073f276d4146e2ca3c706836a4593cb
using System;

public class Program
{
    [Bridge.Rules(Boxing = Bridge.BoxingRule.Managed)]
    public static void Main()
    {
        Console.WriteLine($"expected 'Y' but got: {'Y'}");
    }
}

Within your bridge.json file, you can revise the rules section by setting "boxing": "Managed" instead of "Plain".
"rules": {
    "boxing": "Managed"
  }

Hope this helps.


Option to alert or block from a given severity threshold

At the moment I believe the scan simply outputs a file, but in line with #3, it would be cool to be able to stipulate that only findings of a given severity or higher (e.g. Medium) are alerted, at least in a PR comment (the full report can still be included as a file if one is interested in all findings). Similarly, another variable to control whether or not the current scan should block the CI from a given severity (e.g. High).

Both look a good idea, but false positives might be a concern here, so this could be an optional flag.
Do you fancy doing a PR covering this and #3? :)

This one is outside my area of expertise, so it will likely be better suited by someone with actual experience handling it I think.

Closing due to inactivity


Add support for watchOS in the podspec

This is for CocoaPods 0.38, I'd suggest we hold of merging until the stable version has been released.

Perfect, thanks @kylef!

Thanks @kylef. I added this change in 5b4a59b4.


OSSM-569 Use default resources for prometheus sidecar

Manual cherry-pick of #836
/hold until #840 is merged

/retest


Kestrel

Once this is merged, I'll rename the repository to kestrel and make it public. Once that is done I can look at publishing to maven central via Sonatype.

Force-pushed to squash down @admackin's formatting suggestions and fix a small typo that was introduced.


Also clean up properly if signal received

Does this or a previous commit close #10?

I think it should 👍


ReactGridLayout children are rendered on top of each other by x coordinates

The problem is that the layout object as:
const gridLayout = [
{ i: '1', x: 0, y: 0, w: 1, h: 2 },
{ i: '2', x: 1, y: 0, w: 1, h: 2 },
{ i: '3', x: 0, y: 1, w: 1, h: 2 },
{ i: '4', x: 1, y: 1, w: 1, h: 2 }
];
is rendering items 1, 2 on top of each other
and 3, 4 as well. But items 1,2 and 3,4 are positioned correctly vertically.
The container with class name "react-grid-layout" has 0 width and correct height.
I can drag items vertically but can't horizontally

I've managed to render the items without overlaying each other, but all the items are placed vertically. Changing the 'x' in the array doesn't affect the position of each child horizontally. The transform property for each child has the same coordinates for 'x' axis: '10px'.

I'm also facing the same issue.

I'm also facing the same issue. Does anyone know how to fix this?

I'm also facing the same issue. Does anyone know how to fix this?


Fixes to make sure the docker container names match what is expected

Fixes #4

You can test this PR on GitPod by clicking on this link: https://gitpod.io/#https://github.com/muze-nl/federated-timesheets-tests/pull/6


Handle empty write ahead logs more gracefully.

For certain error situations a tablet server may get into a retry loop and create lots of empty write ahead logs.  An empty write ahead log may have a header and no data, so its not actually an empty file.  All of these empty write ahead logs can cause slow recovery.  There are at least two possible way to handle this problem.  The tablet server could not add empty write ahead logs to zookeeper OR clean up known empty write ahead logs from zookeeper.  If that is not a workable solution, then maybe empty write ahead logs could be filtered during recovery so that they are not assigned to tablets.

I did some research and found the tablet server code that is causing this.  Every time a write ahead log is created it is immediately added to zookeeper.  When writing to a walog if any error occurs then the WAL is closed.  Sometimes the exceptions that occur are unrelated to writing to the walog and occur before anything was ever written to the walog.  If the error keeps happening then this ends up adding lots of empty walogs to ZK which are used during recovery.
One possible solution is that when a walog is closed if nothing was written to it then mark the walog in ZK in such a way that is GCed.  Could possibly immediately remove the file after marking it in ZK.  If the process dies between ZK and delete, then the Accumulo GC would still clean it up.


feat: Add JSON DB type support in BigQuery

This isn't complete and ready yet.
I am figuring out a whitespace behavior observed while testing.  When I call the ListRows API on the table, whitespace in JSON string between field and value seems to be preserved while they are lost when using the StandardQuery (Select * from {table}).
I will investigate this, document my findings and request the review tomorrow. Thanks.

When I call the ListRows API on the table, whitespace in JSON string between field and value seems to be preserved while they are lost when using the StandardQuery (Select * from {table}).

First thing I'll try to check is whether this is an API behaviour, it does looks like it.

When I call the ListRows API on the table, whitespace in JSON string between field and value seems to be preserved while they are lost when using the StandardQuery (Select * from {table}).

First thing I'll try to check is whether this is an API behaviour, it does looks like it.

Right Amanda, as per docs whitespace characters are not preserved. So, When I save "{"x": 10, "y": "text"}" , it is saved as "{"x":10,"y":"text"}" and same is returned from StandardQuery. However, while using ListRows API, I am getting  "{"x": 10, "y": "text"}" (with whitespace characters). I may be missing options or something else so I will explore further. But I think, I can seek feedback on this PR while I understand this behavior.

I'd like to insist this does look like an API behaviour, whith a quick look at the PR, I don't think the library is going into the fields and parsing the JSON in one way or another, we are just simply converting these to string. The easiest way to confirm that is by trying the tabledata.list operation on the API explorer. If you observe there the same behaviour as with the library, then there's nothing for us to do, except maybe raise it with the API team.
I'll try to review later today, or first thing Monday. But it would be great if you could check with the API explorer and leave a comment with what you see. If the behaviour is caused by the library, then I'd prefer to review once that has been sorted out.

I'd like to insist this does look like an API behaviour, whith a quick look at the PR, I don't think the library is going into the fields and parsing the JSON in one way or another, we are just simply converting these to string. The easiest way to confirm that is by trying the tabledata.list operation on the API explorer. If you observe there the same behaviour as with the library, then there's nothing for us to do, except maybe raise it with the API team.
I'll try to review later today, or first thing Monday. But it would be great if you could check with the API explorer and leave a comment with what you see. If the behaviour is caused by the library, then I'd prefer to review once that has been sorted out.

Sure Amanda. Thanks. I will check the behavior with the API explorer and update this thread.

Thanks Amanda. This is indeed the API behavior.
ListRows API returns the whitespaces (as is) . I tested with no whitespace, single whitespace between key and value and multiple whitespace between key and value and the returned JSON string was exactly the same as the input. This is what I had expected (just that doc says whitespace characters are not preserved)
When standard query i.e. (Select * from table) is executed, whitespace characters are lost in all the three cases above.


Bump cert manager to v1.15.3

Description of changes
Notable changes in v1.14.0:
The startupapicheck job uses a new OCI image called "startupapicheck", instead of the ctl image. If you run in an environment in which images cannot be pulled, be sure to include the new image.
PR: https://github.com/cert-manager/cert-manager/pull/6549
Notable changes in v1.15.0:
cmctl and kubectl cert-manager have been moved to the https://github.com/cert-manager/cmctl repo and will be versioned separately starting with cmctl v2.0.0
PR: https://github.com/cert-manager/cert-manager/pull/6663
Based on the above breaking changes, this PR updates the cert-manager-ctl image repository with cert-manager-startupapicheck image to match the upstream image and binaries. This will also require add the cert-manager-startupapicheck private ECR repository to the build and packages CDK brazil packages. We can also consider updating the corresponding fields in the EKS-A bundle as well here.
Build CDK: https://code.amazon.com/packages/AwsModelRocketBuildInfrastructureCDK/blobs/6bbbc109532484b0d56944aeabcff5807e9bd630/--/lib/ecr-resources.ts#L22
Packages CDK: https://code.amazon.com/packages/AWSModelRocketPackagesInfrastructureCDK/blobs/f8d93ff076f18896f70f592a35e8b60ca3bceb2a/--/lib/ecr-resources.ts#L37
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/hold
/approve

/unhold


Improve dependency of kubelet and updatecacert services

What this PR does / why we need it:
Improve dependency of kubelet and updatecacert services
Now updatecacerts will run before the kubelet.
Which issue(s) this PR fixes:
Fixes #1705
Special notes for your reviewer:
The nodes will not be rolled out with this change.
Release note:

An issue where nodes were failing to join the cluster because of circular dependency between docker, kubelet and updatecacerts services has been fixed.

/LGTM


Added note about firefox bug

CORS is broken for resources which redirect. I don't know every version affected so I added it to my version and all newer versions as the bug is still open.

Thanks!


fix: Add index tracking to notifications page

#602 fixed the bug, but we need the same fix on the notifications page.
Note: This bug seems to have been caused by two notifications that have identical IDs -- in reality we should never encounter this issue as we are the gatekeepers of production notifications.

Contributor License Agreement adherence:

[x] This Contribution is under the terms of Individual Contributor License Agreements (and also Corporate Contributor License Agreements to the extent applicable) appearing in the Apereo CLA roster.

👍


I want to fix tool bar like angular material nav bar. Please help me out how to do

Documentation Feedback
Provide a brief summary of what you would like to see changed in our
documentation at material.angular.io.
Feel free to provide any suggestions of content or examples you’d like us to include.
Affected documentation page: Insert a link to the affected page on material.angular.io

Like We have sticky:true; do we have any class to stick tool bar

The mat-toolbar is a very simple component, it's only used for layout it's children elements. you can style mat-toolbar itself to fix it.
<mat-toolbar class="header"></mat-toolbar>
<style>

.header{
  position:fixed;
  top:0;
left:0;
right:0;
}
</style>

Don't angular material has predefined style for fixing toolbar like in side nav bar

no there is no predefined style for that

Actually i wan't tool bar fixed and scroll as same like that picture(Angular material page)

please tell me that css

style={ position: "fixed", top: "0%", width: "100%", zIndex: "100" }
Try your code with this CSS

Please submit Angular Material and CDK questions here.


Replace TPIE sorting by partioning and merging fsa's

#180 made my think.
For some reason my vision was to replace the sorting code. I realized this might not be necessary, we have everything we need.
I compared the suggestion I gave in #180 on larger data sets:

creating a keyvi file from scratch, utilizing TPIE
creating x small keyvi files (using the "small data compilers" which don't use TPIE sort) and run merger on it

I ran different cases, in summary the merge approach was roughly 20% slower. Note, I did not optimize anything (I used simple python scripts). My merge approach had to copy more data, an improved implementation would avoid that.
The idea is as follows

create an in-memory sorter
if the in-memory sort buffer hits the threshold, sort the data, create an fsa, persist it, free buffers
go to 1.
after all data has been processed, sort, create, persist the final chunk
merge the fsa's and create the final keyvi file

@hendrikmuhs yeah, I really liked this. And this approach was tested on production on large scale datasets, so def worth doing!


Package version of release 2020.04.16 is not updated

Package.json version of release 2020.04.16 is not updated.

Considering forget to update the version string of package.json is a common mistake. It happens quite a few times. For example, 2020.04.16, 2020.03.10a, and 2020.02.29. I would suggest

An automatic approach.
But that breaks the style of using the date string as version number, you may need find some tools like release-it but can specify a version number manually.

thanks for pointing that :) I will fix that asap.


mathext/amos/internal/amos: clean up linter warnings

Please take a look.
This also suggests that there should be consideration about the error status returns here.

That's definitely code it would be nice to have a clean implementation of, if we knew the algorithm


the installation instructions arent clear

where do I add all of this?
here is my file structure:
  nvim/
  | .git/
  | after/
  | | ftplugin/
  | | | netrw.lua
  | | plugin/
  | | | fugitive.lua
  | | | harpoon.lua
  | | | lightline.lua
  | | | lsp.lua
  | | | telescope.lua
  | | | treesitter.lua
  | | | undotree.lua
  | | | winbar.lua.bak
  | lua/
  | | donjikong/
  | | | init.lua
  | | | packer.lua
  | | | python.lua
  | | | remap.lua
  | | | set.lua
  | | | test.py
  | | | win.lua
  | plugin/
  | | packer_compiled.lua
  | init.lua

do I not put it in my packer.lua?

can i use
use 'DylanKendal/nvim-treeclimber'
in my packer file?
if so, you should update the readme to make this clear

I can update the instructions, my apologies if it isn't clear but yes - you can use whatever package manager you like. If you want to use packer you should be able to just packer as normal: use "Dkendal/nvim-treeclimber"

The readme has been updated.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dependencies-2.x" workflow

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

Questions ?
Please refer to the Backport tool documentation


Blazor lazy loading renamed assemblies and DLL ext

Fixes #19628
Fixes #19495

I was just asking about that on the other PR. Artak said to go ahead and merge immediately those earlier Pre8 PRs before Pre8 landed. I was just asking if we're going to keep doing that. If so, we can just merge this now. If someone asks here, I guess I can say ... it will be for RC1.

Yes ... I'll split depending on Artak's response. If he says 'merge now' like he did last time, we're good. I'll just have to tell anyone who writes in 'where is this?' that it will arrive in a few weeks.


Please consider a new release and updating dependencies

Please consider creating (and releasing) a new release.
package.json states 1.0.5
io-package.json states 1.0.3
npm lists 1.02
...
Thats confuding for users and listing at iobroker repositories is outdated because of this.

In addtion please consider updateing dependencies, i.e. you are using adapter-core 1.x.x; current version 3.0.4. This might cause problems in future.
If you need any help or assitance please let uas (me) know.
Thanks
mcm2957

As i mentioned before i don't have a iobroker instance.
So i don't know the current versions.
If you now please make a pull-request and i will merge it.
Thanks

Also there seems to be a problem with the commands as button.
What is the current way to use this?

Also there seems to be a problem with the commands as button.

What command as buttons you refer to?

Play, Pause, Mute, etc

As i mentioned before i don't have a iobroker instance. So i don't know the current versions.
If you now please make a pull-request and i will merge it.
Thanks

OK, I can create a PR

adding current standard tests
adding @alcazones release script

But if you do not have a working development environment you will not be able to create a clean release. Creating a Release is not possible by PR, this must be done at the repository.
If you do not have and do not plan to set up a development environment we should eventually discusse the future. I can offer to migrate the adapter to iobroker community organisation. There multiple developers would have access to code and could create releaeses. Of course you will have normale read/write access to your code too if you want to have.
If you want to setup a dev environemnt at your site I (and others) will try to guide an halp you too. This is NO try to get your code in any way. Especially I cannot really maintain the code itself as I do not have any matching environment. I can only assist in releasemenagement etc.
Please let me / us know what direction you prefer. Its solely your decision.
AND THANKS VERY MUCH for the work you did until now. THANK YOU

Play, Pause, Mute, etc

Ok, thats  a issue different from environment. Must be checked by any dev who has a matching device.
I could ask at developer telegram channel if someone has time to take a look.

Sure we can Transfer it to the community.
It would be great of I had permissions on the repo so I can help you if there are other Bugs in the future.
Where do i need to Transfer it to?

Great idea.

Well
I ll answer from mobile for now. I m on holiday next week thuesday to friday. I can try to perform the transfer until monday but i cannot promise.
YOU must do the following to initialize the transfer
Log in to npm and go to  the adapter package pafe. Open setup and send an INVITE to mcm1957
Log in to github and go to adapter repository.
Open setzp and scroll down to DANGER ZONE
Initialize a transfer of the repository to mcm1957.
I will transfer the repo to iobroker community adapters immidiatly afterwards and add you as maintainer. The transfer using me as intermediate is required as only administrative members of github orga can import a repository.
Plese ask if any more questions.
My direct mail is<EMAIL_ADDRESS>

You will have to delete your repo first.
Otherwise I cant Transfer it to your Account on Github.

Sorry, I've forgotten my fork.
Have it deleted now.
npm invite was successfull
THANKS A LOT

Thanks for transferring.
I already moved it to iobroker-community-adapters. You should have access there. If anything does not work as expected, please let me know.
I will have to do several adaptions (i.e. adapting repo urls). I will do this during the next days.

Transfer was successful.
Issue will be handled in #25


Remove subobject NetGUIDs during RemoveComponent

This is what was happening before dynamic typebindings. Backported the code.

So what is the reason we want to unregister the subobjects when we receive OnRemoveComponent for UnrealMetadata? What's wrong with waiting until we get the OnRemoveEntity and unregistering the subobjects together with the actor?

This was in my original implementation: When removing the UnrealMetadata from an Entity, we can be reasonably sure the entity is going to be deleted soon after, or at least that it will no longer have any useful Unreal mappings => therefore we can unregister its subobjects. Also at this point we still have access to the subobjects through the view (this changes with the C API, as we do not have a default view - part of this PR is reintroducing that for UnrealMetadata, and I think Sahil created a ticket for a complete View class).


nit/noise: Rename hugo.IsMultiHost and hugo.IsMultiLingual

Feel free to close, but both multihost and multilingual are one word without hyphenation, so these seem a better choice:

hugo.IsMultihost
hugo.IsMultilingual

I noticed yesterday that, excluding the methods exposed to the templates, we are inconsistent internally:

IsMultihost (no camel case)
IsMultiLingual (camel case)

I'd like to change the exposed methods and make the internal usage consistent with the exposed methods.
hugo.IsMultiHost was just introduced last week in v0.123.8, so if we (you) want to change this, now would be a good time.

Yea, that makes sense. I'm back from a short vacation now and I'm going to collect some issues into a final v0.123.9 (before moving on to 0.124) release.

IsMultiHost I suggest that we just rename.
IsMultiLingual, probably best to deprecate and create a copy.

IsMultiHost I suggest that we just rename.
IsMultiLingual, probably best to deprecate and create a copy.

I'm assuming the above are reversed.

hugo.IsMultiHost was part of 123.8, so should be deprecated and copied.
hugo.IsMultiLingual has been merged and not released, so we can just rename.

I will submit a PR. Shout if you think I got it wrong.

I'm assuming the above are reversed.

It was in a roundabout way, but you are right.


/ingestFromFile rest call fails to upload segment if user name password is enabled Pinot 0.9.2

Looks like auth token needs to be passed through when calling the controller API
/ingestFromFile
which calls
/segments api
Caused by: org.apache.pinot.common.exception.HttpErrorStatusException: Got error status code: 403 (Forbidden) with reason: "Permission is denied for access type 'CREATE' to the endpoint 'http://somehost:9000/v2/segments' for table 'dimSomeTable_OFFLINE'" while sending request: http://somehost:9000/v2/segments?tableName=dimSomeTable to controller: somehost, version: Unknown
        at org.apache.pinot.common.utils.FileUploadDownloadClient.sendRequest(FileUploadDownloadClient.java:510) ~[pinot-all-0.9.2-jar-with-dependencies.jar:0.9.2-3e5e5aca07ad796d453ad
d80a2b2b1bf1d654cf1]
        at org.apache.pinot.common.utils.FileUploadDownloadClient.uploadSegment(FileUploadDownloadClient.java:791) ~[pinot-all-0.9.2-jar-with-dependencies.jar:0.9.2-3e5e5aca07ad796d453
add80a2b2b1bf1d654cf1]
        at org.apache.pinot.segment.local.utils.SegmentPushUtils.lambda$pushSegments$0(SegmentPushUtils.java:120) ~[pinot-all-0.9.2-jar-with-dependencies.jar:0.9.2-3e5e5aca07ad796d453a
dd80a2b2b1bf1d654cf1]
        at org.apache.pinot.spi.utils.retry.BaseRetryPolicy.attempt(BaseRetryPolicy.java:50) ~[pinot-all-0.9.2-jar-with-dependencies.jar:0.9.2-3e5e5aca07ad796d453add80a2b2b1bf1d654cf1]
        at org.apache.pinot.segment.local.utils.SegmentPushUtils.pushSegments(SegmentPushUtils.java:117) ~[pinot-all-0.9.2-jar-with-dependencies.jar:0.9.2-3e5e5aca07ad796d453add80a2b2b
1bf1d654cf1]
        at org.apache.pinot.segment.local.utils.IngestionUtils.uploadSegment(IngestionUtils.java:204) ~[pinot-all-0.9.2-jar-with-dependencies.jar:0.9.2-3e5e5aca07ad796d453add80a2b2b1bf
1d654cf1]
        ... 28 more
2022/07/06 23:31:00.893 ERROR [PinotIngestionRestletResource] [jersey-server-managed-async-executor-5] Caught exception when ingesting file into table: dimSomeTable_OFFLINE. Caught
exception while uploading segments. Push mode: TAR, segment tars: [[file:/local/data/logs/s3:/blah/upload_
dir/working_dir_dimSomeTable_OFFLINE_1657164660862/segment_tar_dir/dimSomeTable_1657164660863.tar.gz]]
java.lang.RuntimeException: Caught exception while uploading segments. Push mode: TAR, segment tars: [[file://local/data/logs/s3:/blah/upload_
dir/working_dir_dimSomeTable_OFFLINE_1657164660862/segment_tar_dir/dimSomeTable_1657164660863.tar.gz]]
        at org.apache.pinot.segment.local.utils.IngestionUtils.uploadSegment(IngestionUtils.java:207) ~[pinot-all-0.9.2-jar-with-dependencies.jar:0.9.2-3e5e5aca07ad796d453add80a2b2b1bf
1d654cf1]

Yes. This API requires the auth token because it will try to generate and add a segment to the table (access type CREATE)

Issue here is that an api which is provided the token /ingestFromFile, calls another api /segments and does not propagate the token.

I see. I saw a recent PR #8670 that might solve this issue. @apucher Can you please help verify if the PR can fix the issue?

Hi @pjpringle @Jackie-Jiang currently, pinot uses the controller's client credential to perform file upload. This should work once your've configured the segment fetcher credentials (you need those anyways)


libgit2-sys/build.rs checks for insufficient libgit2 version

build.rs checks for libgit2 >= 0.21.0, while git_libgit2_init() and git_libgit2_shutdown() were only introduced in libgit 0.22.0.

Thanks for catching!


Reopening issue changes issue #, invalidating references

Just experienced this one time, so maybe it was a fluke?

Thanks your feedback!
What's your Gogs version and can you reproduce it?

Close by now and open for continuing discussion. :octocat:


For lualatex, if the main math font is TeX Gyre Termes Math, simplify and document getting new characters from STIX Math or STIX Two Math without causing unrelated issues.

(History: http://tex.stackexchange.com/a/721080 .)
Let's set our main math font as TeX Gyre Termes Math. We also wish to get some symbol(s) not existing in TeX Gyre Termes Math from another math font. To this end, we write
\setmathfont{TeX Gyre Termes Math}
\setmathfont[Extension=.otf,range={…}]{STIXTwoMath-Regular}% Specify the Unicode position(s) in the value of range.

However, this leads to a warning in the log (“Script 'Math' not explicitly supported within font 'STIXMath-Regular'”) and, if lualatex is used, superfluous vertical gaps in \substack{…\\…}, \stackrel{…}{…}, \frac{…}{…}, and \underbrace{…}_{…}. To remove these gaps, we append
\setmathfont[range={}]{TeX Gyre Termes Math}% Leave the value of the range parameter empty or put \int there.

Combined:
\documentclass{article}
\pagestyle{empty}
\usepackage{unicode-math}
\setmathfont{TeX Gyre Termes Math}
\setmathfont[Extension=.otf,range={"2040}]{STIXTwoMath-Regular}% Adding some char not present in the default math font.
\setmathfont[range={}]{TeX Gyre Termes Math}% Without this line, you get unwanted vertical space in the output below when compiling with lualatex.
\begin{document}
\(\stackrel{\text{def}}{\Longleftrightarrow} \frac{m}{n}\ \substack{1\\2} \underbrace{\langle\rangle\dotsm \langle\rangle}_{k\ \text{terms}}\)
\end{document}

Running lualatex on this yields:

This seems to work but is (1) undocumented (at least not visibly documented in unicode-math.pdf), (2) a bit bloated, and (3) misleading (the last \setmathfont[range=… doesn't really request a range of new characters; also, from a high-level viewpoint, a user doesn't really wish to mention TeX Gyre Termes Math twice just because of the way things currently work internally).
In this issue report, we ask to simplify and/or document the user's interface to adding new characters when using lualatex without causing unrelated issues (here: vertical gaps).
(Two asides are to be made. First, for xelatex, the issue doesn't exist; you may drop the last \setmathfont… line. Second, I have not tested other pairs of fonts; if you have, please feel free to change this report, including its title.)

Reloading with no range was common practice to work around unicode-math bugs, and I thought (but wouldn't guarantee!) I had fixed those issues. I'll look into this at some point; thanks for the report.


⚠ Bofrost.it has degraded performance

In 4a5b27f, Bofrost.it (https://www.bofrost.it) experienced degraded performance:

HTTP code: 200
Response time: 3405 ms

Resolved: Bofrost.it performance has improved in d2e872f after 23 minutes.


chore: prepare runtime for registration

Adds a new runtime crate pop-runtime with the following changes in comparison to pop-runtime-testnet:

Barrier: allows for version subscriptions and removed unpaid execution.

pub type Barrier = TrailingSetTopicAsId<(
	TakeWeightCredit,
	AllowKnownQueryResponses<PolkadotXcm>,
	WithComputedOrigin<
		(AllowTopLevelPaidExecutionFrom<Everything>, AllowSubscriptionsFrom<Everything>),
		UniversalLocation,
		ConstU32<8>,
	>,
)>;

pallet_collator_selection:

Sets the desired number of candidates to 0 and the maximum number of candidates to 0 too. Effectively allowing only invulnerable collators.

pallet_contracts:

CallFilter is changed so it whitelists Nothing.
Relevant relay chain specific params.

Codecov Report
Attention: Patch coverage is 9.10384% with 639 lines in your changes missing coverage. Please review.

Project coverage is 7.11%. Comparing base (71d81b9) to head (c7e2026).

Files
Patch %
Lines

runtime/live/src/lib.rs
3.89%
370 Missing :warning:

runtime/live/src/extensions.rs
0.00%
114 Missing :warning:

node/src/chain_spec.rs
0.00%
88 Missing :warning:

node/src/command.rs
0.00%
27 Missing :warning:

runtime/live/src/config/proxy.rs
0.00%
19 Missing :warning:

runtime/live/src/config/xcm.rs
0.00%
6 Missing :warning:

runtime/live/src/weights/paritydb_weights.rs
73.33%
4 Missing :warning:

runtime/live/src/weights/rocksdb_weights.rs
73.33%
4 Missing :warning:

runtime/live/src/config/contracts.rs
75.00%
3 Missing :warning:

runtime/live/src/weights/block_weights.rs
81.81%
2 Missing :warning:

... and 1 more

@@           Coverage Diff            @@
##            main    #238      +/-   ##
========================================
+ Coverage   6.40%   7.11%   +0.70%
========================================
  Files         27      36       +9
  Lines       1997    2698     +701
  Branches    1997    2698     +701
========================================
+ Hits         128     192      +64
- Misses      1869    2506     +637

Files
Coverage Δ

runtime/live/src/weights/block_weights.rs
81.81% <81.81%> (ø)

runtime/live/src/weights/extrinsic_weights.rs
81.81% <81.81%> (ø)

runtime/live/src/config/contracts.rs
75.00% <75.00%> (ø)

runtime/live/src/weights/paritydb_weights.rs
73.33% <73.33%> (ø)

runtime/live/src/weights/rocksdb_weights.rs
73.33% <73.33%> (ø)

runtime/live/src/config/xcm.rs
0.00% <0.00%> (ø)

runtime/live/src/config/proxy.rs
0.00% <0.00%> (ø)

node/src/command.rs
0.00% <0.00%> (ø)

node/src/chain_spec.rs
0.00% <0.00%> (ø)

runtime/live/src/extensions.rs
0.00% <0.00%> (ø)

... and 1 more


Add MoEfication & PLE

@ZhengZixiang  Thanks for your PR!
Please add these two papers back to the list and commit it. I notice that you remove them in your PR.

Parameter-Efficient Mixture-of-Experts Architecture for Pre-trained Language Models
Efficient Language Modeling with Sparse all-MLP


Urgent Fix: kwargs

There are some methods like IntervalSet.__init__ that has unused **kwargs. This can result in an unwanted behavior if one, for example, misspell a keyword argument it won't be set and no error will be raised.
We should go through all the defs that includes **kwargs and think if we can remove any and fix the logic.

convolve strip the column names

TsGroup test for time_support if not None

check kwargs list for count


california flattening and agency names

when i build the california output using the code in the repo, the code from #6 correctly flattens contiguous stints into one row. But looking things up on the site, those records appear disaggregated. For example when I look up last name "ourjanian", I see adjacent stints at "Sierra CO Sheriff's Office Office". When I build locally and look at the upload folder, I see those correctly collapsed to one stint, and further the agency is just named as "Sierra CO Sheriff's Office" (without the redundant second "Office").

Fixed in PR #28. Thanks!


[WIP] Fixing issue386

This should fix issue 386
https://github.com/lemire/simdjson/issues/386
It seems to lower the stage 1 performance in some hasty tests.
Needs to be reviewed.

This PR still has many issues. I will fix them.

Arch-specific bitmanipulation.h seems like the right approach!

@jkeiser
I am basically pursuing your approach. I have weird performance issues, so I need to revisit everything... but this should be more maintainable.
I expect to be able to merge this today.

The PR currently has a significant regression during stage 2 on ARM. I am investigating.
(I was not expecting any difference.)

Hopefully I fixed the regression.

Green light!!!


Because of the change to UI, this doesn't work on Nintendo Switch

I loaded this to the Nintendo Switch but I couldn't find the panel.  I believe that this is because of the change to the UI.
Can this be updated to work on Switch?

Fixed


error in install TermUX

my goodness,i just want change my phone to Aiden's one
current directory: /data/data/com.termux/files/usr/lib/ruby/gems/2.6.0/gems/nokogiri-1.10.5/ext/nokogiri    /data/data/com.termux/files/usr/bin/ruby -I /data/data/com.termux/files/usr/lib/ruby/site_ruby/2.6.0 -r ./siteconf20191202-31542-1e27xgm.rb extconf.rb            checking if the C compiler accepts ... *** extconf.rb failed ***                                            Could not create Makefile due to some reason, probably lack of necessary                                    libraries and/or headers.  Check the mkmf.log file for more details.  You may
need configuration options.
Provided configuration options:
--with-opt-dir
--without-opt-dir
--with-opt-include
--without-opt-include=${opt-dir}/include
--with-opt-lib
--without-opt-lib=${opt-dir}/lib
--with-make-prog
--without-make-prog
--srcdir=.
--curdir
--ruby=/data/data/com.termux/files/usr/bin/$(RUBY_BASE_NAME)
--help
--clean
/data/data/com.termux/files/usr/lib/ruby/2.6.0/mkmf.rb:467:in try_do': The compiler failed to generate an executable file. (RuntimeError) You have to install development tools first.         from /data/data/com.termux/files/usr/lib/ruby/2.6.0/mkmf.rb:585:in block in try_compile'
from /data/data/com.termux/files/usr/lib/ruby/2.6.0/mkmf.rb:532:in with_werror'         from /data/data/com.termux/files/usr/lib/ruby/2.6.0/mkmf.rb:585:in try_compile'
from extconf.rb:138:in nokogiri_try_compile'         from extconf.rb:162:in block in add_cflags'          from /data/data/com.termux/files/usr/lib/ruby/2.6.0/mkmf.rb:643:in with_cflags'                            from extconf.rb:161:in add_cflags'                   from extconf.rb:416:in `'                                                                     To see why this extension failed to compile, please ch
eck the mkmf.log which can be found here:
/data/data/com.termux/files/usr/lib/ruby/gems/2.6.0/extensions/aarch64-linux/2.6.0/nokogiri-1.10.5/mkmf.lo
g                                                                                extconf failed, exit code 1                           ~

Hi - termux isn't a supported platform for nokogiri, but I'll do what I can to help.
Have you looked at the previous termux issues or asked this question on a termux forum?

https://github.com/sparklemotion/nokogiri/issues/1939
https://github.com/sparklemotion/nokogiri/issues/1928
https://github.com/sparklemotion/nokogiri/issues/1816
https://github.com/sparklemotion/nokogiri/issues/1884
https://github.com/sparklemotion/nokogiri/issues/1739

yes,but is looks a little different,
My system is try to find something is not C lib(mkmf says lot of things never been define)
also,i found something not exist in gem lib,maybr that's the answer

i give up
i gonna to switch my OS to arch

For me the following sequence worked on a fresh termux:
$ pkg install ruby clang make pkg-config libxslt
$ gem inst nokogiri -- --use-system-libraries

@larskanis thanks, I'll add that to the installation docs on nokogiri.org.

see https://nokogiri.org/tutorials/installing_nokogiri.html#termux


Hiding Variable is slow

I noticed that hiding variables in HFSS through pyaedt is very slow.  Tried 2 techniques
1:
hfss.hidden_variable(var_name, True)
2:
hfss.variable_manager.variables[var_name].hidden = True
Both techniques seem comparable.  It was taking on the order of 5 - 10 seconds to hide 80 variables.
In the HFSS GUI, I could hide all the variables, and it appeared to happen instantaneously.

Hi @adamgregory81 ,
I have implemented a new option in the "hidden_variable" method. Now it allows a list of variables like:
aedtapp.hidden_variable(["my_oo_variable", "$dim", "my_hidden_variable"]).
This will hide all these variables, and it will be as fast as the UI.
It will available in the next pyaedt release.
Thanks

Well done @Samuelopez-ansys . This will change the performance of the scripts execution for the best.

Thanks for the update and the fix

Using the latest version of Pyaedt (v0.6.71), I found that hiding variables is still slow.  I noticed that the code used in the November 14th 2022 commit (#1977) in Design.py no longer appeared to be there.  Referring to lines 1856 - 1878 in that commit.  As a test, I implemented that code myself, and it was still slow.  Debugging a bit, I discovered the culprit was fetching these lists inside of the loop:
self.variable_manager.design_variable_names
self.variable_manager.project_variable_names
When I changed to fetching them a single time outside of the loop, I am able to hide 100+ variables as fast as the HFSS GUI.


Feature request: add timestamps to comments

Hello! This tool is exactly what I needed, thank you for making it available :)
There's only one thing I miss: date/time on comments. Would it be feasible to add it?

This would be good. I don't have time to add new features at the moment but I'll accept pull requests. If I get some time I'll add it.


fix a field typo 'serialzeFeatures' in JSONField to serializeFeatures

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Codecov Report

Merging #1417 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1417      +/-   ##
============================================
- Coverage     85.27%   85.26%   -0.01%
+ Complexity     5852     5851       -1
============================================
  Files           135      135
  Lines         20725    20725
  Branches       4661     4661
============================================
- Hits          17673    17672       -1
  Misses         1967     1967
- Partials       1085     1086       +1

Impacted Files
Coverage Δ
Complexity Δ

...m/alibaba/fastjson/serializer/FieldSerializer.java
86.42% <100%> (ø)
49 <0> (ø)
:arrow_down:

...baba/fastjson/serializer/ASMSerializerFactory.java
96.98% <100%> (ø)
199 <0> (ø)
:arrow_down:

...n/java/com/alibaba/fastjson/util/JavaBeanInfo.java
87.21% <100%> (-0.21%)
164 <0> (-1)

...m/alibaba/fastjson/serializer/SerializeConfig.java
80.32% <100%> (ø)
118 <0> (ø)
:arrow_down:

...main/java/com/alibaba/fastjson/util/TypeUtils.java
85.69% <100%> (ø)
451 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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

谢谢反馈问题，这个改动会导致不兼容，还是不改了哈

呃，好吧，总感觉很别扭

希望能继续参与，提交PR

好的，没问题，我再找找

欢迎微信联系我 wenshaojin


Add Shahul as reviewer

Changes
@shahulsonhal has provided nice contributions recently:

Adding validation for resource name length
Mutating images to add Build-specified annotations and labels
Git source and output image results in the BuildRun

I think he is ready to become a reviewer.
Submitter Checklist

[ ] Includes tests if functionality changed/was added
[ ] Includes docs if changes are user-facing
[x] Set a kind label on this PR
[x] Release notes block has been filled in, or marked NONE

Release Notes
NONE

/verify-owners


On the confirm order page the TH number is displaying a  dummy number.

TH number needs to display correct amount that correlates to the contract

Issue has been fixed


[logger] adds logger to ci

/cc @mitchdenny
It looks like it already has sdk-type: client :)

CI failing, #5606 should fix that.


Add logger aspect to the mixer.

This adds a basic aspect with a Log([]Entry) interface to be
implemented by various adapters. A simple manager is provided for
downscoping to an Entry from the attribute.Bag based on the
configured Input attributes.
Tested with: pkg/aspectsupport/logger$ bazel test ...
Test coverage: 90%.

This change is

PTAL. Addressed date/caps issues.
I don't have a good answer for passing down inputs to adapters in a typed fashion at this point (without adapters being written with full knowledge of the expected type for the inputs ahead of time).


[ALLUXIO-1771] Made correction of TFS to Alluxio FS in AbstractFileSystemTest

Can one of the admins verify this patch?

Jenkins, test this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/9008/
Test PASSed.

@Slyrich
Thanks for fixing the email account. Now it is correctly linked.
Here is another issue: somehow you reverted your previous change.
see https://github.com/Alluxio/alluxio/pull/2907/files where it shows no change to current master branch.
Could you update the PR again?
Thanks

Sorry about the trouble. I would close this pull request and try to set up a new one.
Thanks
Rich


Make determination of output data types more robust

Note to self: this way I can determine the placeholders in a string:
from string import Formatter
fieldnames = [fname for _, fname, _, _ in Formatter().parse(yourstring) if fname]


Broken image/Image doesn't exist anymore

@emilyriederer it looks like your image doesn't exist anymore (here's the original path: https://github.com/emilyriederer/website/blob/master/content/authors/admin/avatar.jpg?raw=true)

Do you have a new one that we can add? 😊 If you have a link, I'm happy to add it quickly 😊
Thank you!

@cosimameyer Thanks for catching this and my apologies! I refactored my website over the holidays, and I didn't consider where else I might have used that link as a dependency
Could you please switch it to this path? https://avatars.githubusercontent.com/u/19798371 . That one should be persistent to any more mischief on my part

Awesome, thanks so much ☺ I'll update it once I'm back at my computer (later this week) 🙌


canvas: add shape snapping

requested by @CoreyCole
https://github.com/lockbook/lockbook/assets/66345861/5951835f-a74d-4b3e-9572-77d93cb0e853

Wow amazing work


[Tabular][FASTAI] Fixing single row prediction failure

Description of changes:
fastai has issues predicting on a single row, duplicating the row as a workaround
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-748-1 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-748/1/index.html


Create Class Diagram for #4

Hey @parvezpatel30 , Let me know if we are good with this and if anything else is needed to implement #11 in Greenfoot!

here's the commit link
https://github.com/appsmehta/CMPE202-CSUnplugged-PeruvianFlipCoin/blob/master/PeruvianClassDiagram.png

Hey @appsmehta , that's awesome. I will let you know if I need anything else.

@parvezpatel30  nope I don't think it's required, since only place I see this is for player objects and all our objects on the canvas would be extending the actor class in greenfoot.

Hey @appsmehta : Shouldn't class captain be a subclass of the class player? As essentially, Captain "is a" Player after all.

@parvezpatel30  Agree, I was not making Captain a subclass of Player just because I wanted Captain to be different from rest of the teammates. Since other teammates can't contact the Rival team captain directly, so any requests have to be routed through their captain.
Now that I think of it, Player Parent and Captain child makes more sense. In case we feel the need for a second child, we can name it "Teammate".
Any thoughts @pavanshah77 @vjzaveri @yugalk14

@appsmehta
"Since other teammates can't contact the Rival team captain directly, so any requests have to be routed through their captain."

Captain class can extend the Player class and then add its own method "contactRival()".

Commit for #10 updated
https://github.com/appsmehta/CMPE202-CSUnplugged-PeruvianFlipCoin/commit/530fb0a3cf4d4cb561c2287c374865df145ef3bc

I think we should make a class named "Team" extending actor, and the other two, captain and teammates will extend Team, doing this we will be properly able to create two teams (making two objects of teams, and then two captains extending their respective teams) as well as inheritance will be proper.

Making captain as a subclass of player class is a better option. We can
inherit all the basic properties of player class into captain class while
keeping the unique properties of captain related to toss, inserting input
to circuit, checking output etc separate from other teammates.
Teammates can be another subclass of player class.
Player Class --> {Captain Class}{Teammate Class}
On Sat, Sep 24, 2016 at 12:04 AM, Vansh Zaveri<EMAIL_ADDRESS>wrote:

I think we should make a class named "Team" extending actor, and the other
two, captain and teammates will extend Team, doing this we will be properly
able to create two teams (making two objects of teams, and then two
captains extending their respective teams) as well as inheritance will be
proper.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/appsmehta/CMPE202-CSUnplugged-PeruvianFlipCoin/issues/10#issuecomment-249349791,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUQ9mbKm3mvY4QT70RvTW2RxwQwRAOmtks5qtMuHgaJpZM4KCWWi
.


[Adjust] Report user’s anonymous ID as a callback parameter.

Closes #2128

Nice one, looks legit to me 👍