Show mcp client synced status in configz page

Sub-task of https://github.com/istio/istio/issues/12298 to show the mcp clients synced status
Signed-off-by: Chun Lin Yang<EMAIL_ADDRESS>

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: clyang82
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: ozevren
If they are not already assigned, you can assign the PR to them by writing /assign @ozevren in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/mcp/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ozevren Please review. I will close the issue #12298 once this PR is merged. Let me know if you have further comments.

@clyang82: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
371a9b69415862aaf00e69844f74b5a592864b29
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@clyang82: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@clyang82: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
371a9b69415862aaf00e69844f74b5a592864b29
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test istio-unit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@clyang82: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
371a9b69415862aaf00e69844f74b5a592864b29
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test istio-unit-tests-master

prow/integ-framework-local-presubmit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test integ-framework-local-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@clyang82: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
371a9b69415862aaf00e69844f74b5a592864b29
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test istio-unit-tests-master

prow/integ-framework-local-presubmit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test integ-framework-local-presubmit-tests-master

prow/integ-galley-local-presubmit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test integ-galley-local-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@clyang82: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
371a9b69415862aaf00e69844f74b5a592864b29
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test istio-unit-tests-master

prow/integ-framework-local-presubmit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test integ-framework-local-presubmit-tests-master

prow/integ-galley-local-presubmit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test integ-galley-local-presubmit-tests-master

prow/integ-security-local-presubmit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test integ-security-local-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@clyang82: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
371a9b69415862aaf00e69844f74b5a592864b29
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test istio-unit-tests-master

prow/integ-framework-local-presubmit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test integ-framework-local-presubmit-tests-master

prow/integ-galley-local-presubmit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test integ-galley-local-presubmit-tests-master

prow/integ-security-local-presubmit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test integ-security-local-presubmit-tests-master

prow/istio-presubmit.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test istio-presubmit-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@clyang82: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
371a9b69415862aaf00e69844f74b5a592864b29
link
/test istio-integ-k8s-tests

prow/istio-unit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test istio-unit-tests-master

prow/integ-framework-local-presubmit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test integ-framework-local-presubmit-tests-master

prow/integ-galley-local-presubmit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test integ-galley-local-presubmit-tests-master

prow/integ-security-local-presubmit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test integ-security-local-presubmit-tests-master

prow/istio-presubmit.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test istio-presubmit-master

prow/integ-pilot-local-presubmit-tests.sh
29bd46c96193e54f33a0d8ef3b0cd9a9b007388a
link
/test integ-pilot-local-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/lgtm (but there is a conflict)

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: clyang82, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/mcp/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@clyang82: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
371a9b69415862aaf00e69844f74b5a592864b29
link
/test istio-integ-k8s-tests

prow/integ-security-k8s-presubmit-tests.sh
a367d342606e8ad2328d004da559d8cf45717d8e
link
/test integ-security-k8s-presubmit-tests-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren I don't know why it still needs lgtm label.

/lgtm

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: clyang82, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

pkg/mcp/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@clyang82: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
371a9b69415862aaf00e69844f74b5a592864b29
link
/test istio-integ-k8s-tests

prow/e2e-bookInfoTests-v1alpha3.sh
aba998cceb8b05519b72c068c8957e31545427e2
link
/test e2e-bookInfoTests-envoyv2-v1alpha3-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@clyang82: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
371a9b69415862aaf00e69844f74b5a592864b29
link
/test istio-integ-k8s-tests

prow/e2e-bookInfoTests-v1alpha3.sh
aba998cceb8b05519b72c068c8957e31545427e2
link
/test e2e-bookInfoTests-envoyv2-v1alpha3-master

prow/istio-pilot-multicluster-e2e.sh
aba998cceb8b05519b72c068c8957e31545427e2
link
/test istio-pilot-multicluster-e2e-master

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-bookInfoTests-envoyv2-v1alpha3-master


Convert U+2212 (MINUS-SIGN) to U+002D (HYPHEN-MINUS).

Closes #12.

Would you mind adding a test?

Yep will do.

Tests added.

Thanks :)!


update akka-http-json

update akka-http-json to 1.14.0

Codecov Report

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

@@           Coverage Diff           @@
##           master      #24   +/-   ##
=======================================
  Coverage   86.77%   86.77%
=======================================
  Files          18       18
  Lines         431      431
  Branches        2        2
=======================================
  Hits          374      374
  Misses         57       57

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 67fd505...9406772. Read the comment docs.


Dependency updates

There are two breaks if you install this with two node modules. This fixes them.

Closing stale PR .. package deps have been subsequently updated.


Add operationoutcomes for errors during bulkdata export

We need to aligned with the updated FHIR bulkdata export spec to provide operationoutcomes for errors during the export just like what we did for bulkdata import.
The javabatch job needs to generate ndjson file(s) for the errors and push to COS and bulkdata operation need to add them to the "error" part of the response.

array of error file items following the same structure as the output array.
*Errors that occurred during the export should only be included here (not in output). If no errors occurred, the server SHOULD return an empty array. Only the OperationOutcome resource type is currently supported, so a server SHALL generate files in the same format as bulk data output files that contain OperationOutcome resources.

{
"transactionTime": "[instant]",
"request" : "[base]/Patient/$export?_type=Patient,Observation",
"requiresAccessToken" : true,
"output" : [{
"type" : "Patient",
"url" : "http://serverpath2/patient_file_1.ndjson"
},{
"type" : "Patient",
"url" : "http://serverpath2/patient_file_2.ndjson"
},{
"type" : "Observation",
"url" : "http://serverpath2/observation_file_1.ndjson"
}],
"error" : [{
"type" : "OperationOutcome",
"url" : "http://serverpath2/err_file_1.ndjson"
}],
"extension":{"http://myserver.example.org/extra-property": true}
}

Team: please confirm that this is implemented and/if not work it.


Don't reuse identifier with registerAction2

I have noticed that some Action2 instances are registered that reuse an id. That's not OK because Action2 is a singleton model object, e.g it keybinding or menu registrations refer to it by id. This should be cleaned up and from then on it should be an error to reuse an id

[ ] command '~remote.forwardedPorts.toggleVisibility'
[ ] command '~remote.forwardedPorts.removeView'
[ ] command 'workbench.panel.repl.view.toggleVisibility'
[ ] command 'workbench.panel.repl.view.removeView'
[ ] command 'workbench.explorer.fileView.toggleVisibility'
[ ] command 'workbench.explorer.fileView.removeView'
[ ] command 'workbench.explorer.openEditorsView.toggleVisibility'
[ ] command 'workbench.explorer.openEditorsView.removeView'
[ ] command 'timeline.toggleVisibility'
[ ] command 'timeline.removeView'

@sandy081 I believe this is all the same issue somewhere in views land?

May I know what is the suggested approach?

Don't register them twice

@sandy081 https://github.com/microsoft/vscode/pull/202000 fixes duplication in view land by first disposing old action and only then creating new ones

@Tyriar @lszomoru you are new to this issue, there are ids that are reused for Action2 and those need to be fixed. Today, this prints a warning but it this become an error soon

Adding @meganrogge for @Tyriar


The scala project under examples would cause NoClassDefFoundError

I followed the steps that is illustrated in the page of
https://github.com/rapidsai/spark-examples/blob/master/getting-started-guides/building-sample-apps/scala.md and built the scala project.
Then I used spark-submit to submit the application to the cluster and I got an exception called
'java.lang.NoClassDefFoundError: scala/Product$class'. It seems that the jar produced by the mvn command does not contain scala library. Please see the attached file for detail.

I note the assembly-no-scala.xml file and find the scala library is excluded, but why?

Hi cfangplus,
What Spark version were you using? The examples here are not compatible with the latest Spark-3.0.0-preview yet. If you run into the same issue on Spark 2.x, please provide your environment details and more logs so I could try reproducing this issue.
As for the Scala library, I think it's already provided by Spark runtime.

yea, I noticed that.
I used spark-submit to submit the application to a spark3.0 cluster and the scala library conflicted, do you know how to fix or how to enable this scala project compatible wih spark3.0?

Spark 3.0 support is currently in development. My suggestion is using Spark 2.x before the 3.0 compatible release.
There is no easy way to fix the compatible issue. Both the xgboost project and the examples project must be updated and re-built. It's complex.

I know this project is developed since June 2019 and at that time the cuda version is 10.1.
Now I have a gpu environment with cuda10.2, how could I get the cudf-0.9.2-cuda10-2.jar and libxgboost4j.so files?

The team is planing to support CUDA 10.2.
As for now, you could install both CUDA 10.1 & 10.2 on your server and run these examples with CUDA 10.1: https://stackoverflow.com/questions/41330798/install-multiple-versions-of-cuda-and-cudnn

That's great.
Now I have another question.  Why does this spark-example project be proposed?
As we konw that, NV Rapids + Dask could provide distributed data processing, machine learning and graph computing, so Apache Spark seems does not been needed, right?

I think Joshua answered your question here: https://github.com/rapidsai/cudf/issues/3643

@chuanlihao Thank you for your reply.
Recently I runed this program with cuda10.1 and the result is good.
As we know, the kernel module of xgboost is writen by c/c++ and provide shared library to python, JVM, R and other languages API. So does cuML has the capacity that could be used by Spark via JNI ? Do you have some similar idea or plan?

Currently cuML only have Python binding. Technically we could apply a similar approach as in XGboost with cuML. Please let us know your use cases for cuML on Spark.

@anfeng Thanks，I mean we want to accelerate our Spark ML/Grapgh applications with gpu.


DO NOT MERGE: revert: fix($animateCss): remove animation end event listeners on close

This reverts commit c98e08fd871fa5df39c66e9700345c2ae0f21bc0.
This commit was identified as incompatible with ng-material at Google
and is causing broken builds there. Proper fix to be investigated once
the immediate regression is addressed.

do not merge anything into 1.4.x branch including this PR.

I'll merge this in when we are ready.
@petebacondarwin, @Narretz, @gkalpak please cherry pick the test fix to master and verify why the lint check didn't catch ddescribe during CI.

Looking into the ddescribe issue.

(Indeed, it was merged without ddecribe in master (6858caf), but with ddescribe in v1.4.x (85e392f)).
For some reason, I can't seem to find any travis jobs forthe commits between 20604e7 and 8f0b482 (on master) and between 33cc75e and 1329e0f (on v1.4.x). If I am not missing something, this means that there was no "CI" for the following commits:
master: 6858caf, bc41ad8, 6428ed5
v1.4.x: 7caf913, 0c1b54f, 85e392f, c98e08f (these include the commit that introduced the ddescribe)
--
The 2 builds on v1.4.x after the missed/skipped commits should have caught the ddescribe error, but they didn't get to run the ci-checks task, because they errored one step before that (on the test:unit task) because of timing out issues.
Maybe it is related to this Travis issue: https://www.traviscistatus.com/incidents/b3xhcrr7g46d

@IgorMinar, by "please cherry pick the test fix to master" I assume you only meant the ddescribe test fix, not the reverted $animateCss tests. If you meant something else, just lmk ;)

Good work @gkalpak .

Thanks @gkalpak. If jquery also removes all events when argument is '' then there's no use in fixing it in jqlite. I'll check.

Yes, we needs to verify what jQuery does (but I would be very surprised if they did it as jqLiteOff).

The results are in ... jquery doesn't remove any events if .off is called with empty string. So we need to fix jqlite, I'm gonna do that now. We can of course also make the code in animateCss more explicit. Wdyt?

@Narretz thanks

Scratch that, actually, jQuery also removes all events when called with '', so the fix must go directly into animateCss

ok. this is good to be merged now, but there are some commits in the v1.4.x branch that might need to be reverted before we land this PR. commits to revert (in this order): 194ade2, 1329e0f.
I talked to @petebacondarwin about this and either him or I will do this cleanup next week. We are not under time pressure to do this as long as v1.4.x remains frozen.

@IgorMinar I have a fix for the animation issue, so it doesn't necessarily have to be reverted. Or is there another reason why we can't just fix it?

@IgorMinar, are you sure 194ade2 is the one that needs to be reverted ?

yes - the two commits listed are in v1.4.x branch but not in google3. g3
now contains an emergency rollback (revert) of the animation commit in this
PR.
Do not touch v1.4.x - pete or me will do the cleanup next week.
On Fri, Dec 11, 2015 at 10:45 AM Georgios Kalpakas<EMAIL_ADDRESS>wrote:

@IgorMinar https://github.com/IgorMinar, are you sure 194ade2
https://github.com/angular/angular.js/commit/194ade29fd7c7751bff52f11764e3854cfe6dd54
is the one that needs to be reverted ?
—
Reply to this email directly or view it on GitHub
https://github.com/angular/angular.js/pull/13504#issuecomment-164014785.

landed as 194ade2...bab3069 (I had to force push to get g3 and github in sync)

Is v1.4.x still "frozen" ?


Options and body=null passed to get/delete request

damm bad testing. typeof null is object. so needs undefined for get/delete

gulp prepare-release please :)

Was out of time. Will do so later (patch). was just quick to let you know
On Thu, Dec 24, 2015 at 10:02 AM -0800, "Wesley Overdijk"<EMAIL_ADDRESS>wrote:
gulp prepare-release please :)

Reply to this email directly or view it on GitHub:
https://github.com/SpoonX/aurelia-api/pull/10#issuecomment-167142580

No worries, it's christmas eve. Enjoy! :)


Review usage of workspaceContains activationEvent

Hi, I'm reaching out to let you know that we are fixing a bug with the workspaceContains VS Code extension activationEvent that could change the activation behavior for your extension.
Here's the issue: This event is supposed to start a search in the user's workspace with the given glob pattern. A pattern like *.txt should activate the extension whenever the root of the workspace has a file matching *.txt, but it should not search the entire workspace. A pattern like **/*.txt should recursively search the entire workspace. We discovered that VS Code is incorrectly applying the ** pattern to every workspaceContains pattern that includes a *.
Since this may impact the startup performance of VS Code, we need to scope these searches correctly. Once we fix this, a pattern like *.txt will only match a .txt file in the root of the workspace.
The change will be in VS Code Insiders in early January. It will be released with 1.53 in early February. Please see https://github.com/microsoft/vscode/issues/110510 for more details and let me know if you have any questions.

@roblourens Thank you for your kind notifications
I will check the behavior after 1.53 is released.
https://github.com/Ikuyadeu/vscode-R/blob/b5be1c3d5235ef44bec85ce554aa189984c43b46/package.json#L36

I don't believe we ran into an issue with workspaceContains. Closing for now!


[DOCS] : Set up a CODE_OF_CONDUCT file

Code of conduct file needs to add to this project.

As I'm setting up the Contributing file, I thought that this project also doesn't have a Code of conduct file that also needs to be added.  So I will like to take this issue.

I would appreciate if both are added in separate PRs.
For the COC, you could use the - Contributor Covenant template from github


Detect best face angle in video stream

How to detect best face angle in video stream for face recognition? Is any recipe for it in dlib or OpenCV?

No.  I would use the landmarks to try and pick out front looking faces though.

Is any example for this ( preferably on a python)? What is best for it - HOG or CNN detector?
We detect face, we track it, our need one foto with this face to put it in database and recognize it.

I don't have any code examples for this.

Ok :) May be some ideas how to get best angle of face based on parameters of 5 or 68 dots landmarks?

You could go crazy with something like PNP, but really, just thresholding
the aspect ratio of the box that fits the 5 face points is probably more
than sufficient to identify front looking faces.

:)) yes, and is it difficult to know look human to camera with open eyes or not?

Can dlib to know - open eyes or closed? Look human to camera or not?

Yes, but you need to write come code to do this.

Do you tell me more information about?

You could run the 68 point model and look at how far apart the eyelids are.  There is a lot of information on the net about this.  However, github issues is not a code writing service, so if you are expecting someone to write it for you that's probably not going to happen.

Oh, yes! :) Thank a lot!


[PoC] compatibility layer between stable datasets and prototype transforms

This is the proof of concept implementation for #6662. Let's keep the discussion there unless there is need for discussion on a technical issue of the proposal.

dbfac05fa75496937ad82ba7341c12a0e4dac565 is a complete of the internals. Here are the main points:

Change the main entry point from the classmethod VisionDatasetDatapointWrapper.from_torchvision_dataset to a standalone function wrap_dataset_for_transforms_v2 (naming TBD). With this we can make the class private and thus users should have no touching points with the internals. On top of that it simplified the implementation quite a bit.
Drop support for additional parameters of the wrapper to specify the dtypes of the wrapped objects, the bounding box format, and whether we want to keep images as PIL. See

https://github.com/pytorch/vision/pull/6663#discussion_r1087146167
https://github.com/pytorch/vision/pull/6663#discussion_r1087153746
https://github.com/pytorch/vision/pull/6663#discussion_r1087151023

Simplify the internal machinery. https://github.com/pytorch/vision/issues/6662#issuecomment-1275821215 established that there is no easy way to provide standalone wrappers for the individual datapoints of a sample. Thus, there is no need to keep the bits that try to do so. This addresses https://github.com/pytorch/vision/pull/6663#discussion_r1086923542 although not as proposed with a standalone class for a task like classification. @NicolasHug PTAL and LMK if my new implementation is simple enough.


[DO NOT MERGE] Async Support

For #52
This is still a work in progress, just to prove that async support wasn't too difficult to get in. Tests are all passing locally.
Perf needs to be measured though.

Is there an async implementation at this time?


New ChannelMod

closes https://github.com/TsubakiBotPad/misc-cogs/issues/12 and https://github.com/TsubakiBotPad/misc-cogs/issues/49


A quick question, does it support react or angular ?

A quick question, does it support react or angular ?
I tried to work it with angular 8, but comes with errors

ERROR in node_modules/@antv/l7-core/es/services/asset/IIconService.d.ts(1,8): error TS1192: Module '"/Users/xisizhe/Documents/projects/game/client/node_modules/@antv/l7-core/node_modules/eventemitter3/index"' has no default export.
node_modules/@antv/l7-core/es/services/camera/ICameraService.d.ts(23,41): error TS2304: Cannot find name 'Omit'.
node_modules/@antv/l7-core/es/services/config/IConfigService.d.ts(1,8): error TS1192: Module '"/Users/xisizhe/Documents/projects/game/client/node_modules/@antv/l7-core/node_modules/ajv/lib/ajv"' has no default export.
node_modules/@antv/l7-core/es/services/layer/IStyleAttributeService.d.ts(71,53): error TS2304: Cannot find name 'Omit'.
node_modules/@antv/l7-maps/es/amap/Wrapper.d.ts(1,23): error TS2688: Cannot find type definition file for 'amap-js-api'.
node_modules/@antv/l7-maps/es/amap/Wrapper.d.ts(5,57): error TS2503: Cannot find namespace 'AMap'.
node_modules/@antv/l7-maps/es/amap/index.d.ts(1,23): error TS2688: Cannot find type definition file for 'amap-js-api'.
node_modules/@antv/l7-maps/es/amap/index.d.ts(10,57): error TS2503: Cannot find namespace 'AMap'.
node_modules/@antv/l7-maps/es/amap/index.d.ts(14,10): error TS2503: Cannot find namespace 'AMap'.
node_modules/@antv/l7-maps/typings/index.d.ts(1,23): error TS2688: Cannot find type definition file for 'amap-js-api'.
node_modules/@antv/l7-maps/typings/index.d.ts(2,22): error TS6053: File '/Users/xisizhe/Documents/projects/game/client/node_modules/node_modules/eventemitter3/index.d.ts' not found.
node_modules/@antv/l7-scene/es/index.d.ts(28,9): error TS1086: An accessor cannot be declared in an ambient context.

have you migrate it with angualr?
best rgds

may me you install  node_modules not successed, you  can reinstall it.

It is installed successfully with +<EMAIL_ADDRESS>but errors keeping coming as showed above

I reinstalled a library, seems some libraries are not installed by npm.
Now there are still some mistakes.

ERROR in node_modules/@antv/l7-core/es/services/asset/IIconService.d.ts(1,8): error TS1192: Module '"node_modules/@antv/l7-core/node_modules/eventemitter3/index"' has no default export.

node_modules/@antv/l7-core/es/services/camera/ICameraService.d.ts(23,41): error TS2304: Cannot find name 'Omit'.

node_modules/@antv/l7-core/es/services/config/IConfigService.d.ts(1,8): error TS1192: Module '"node_modules/@antv/l7-core/node_modules/ajv/lib/ajv"' has no default export.

node_modules/@antv/l7-maps/typings/index.d.ts(2,22): error TS6053: File '/node_modules/eventemitter3/index.d.ts' not found.

node_modules/@antv/l7-scene/es/index.d.ts(28,9): error TS1086: An accessor cannot be declared in an ambient context.

can you help me to spot out which libraries are needed to install?

L7 builds on "typescript": "^3.7.0-beta",
Your local version should be typescript 2.x.

thanks, @ParryQiu, I will try this

thanks, @ParryQiu, I will try this

Latest angular version is 9.00-rc9 but it is still working with typescript 3.6.4, when running with L7 with typescript 3.7.0-beta ,it gives out an error saying
ERROR in The Angular Compiler requires TypeScript >=3.6.4 and <3.7.0 but 3.7.0-beta was found instead.

sorry, there is  something wrong for angular ,we will fixed.
you can use  compiled version  like this
import * as L7  from '@antv/l7/dist/l7

thanks, @lzxue , the solution works

in angular 8, via you approach, import * as L7 from '@antv/l7/dist/l7, and then
ngOnInit() {
const scene = L7.Scene({
id: 'map',
map: L7.Mapbox({
style: 'light',
pitch: 0,
center: [107.054293, 35.246265],
zoom: 4.056,
}),
});
}
but the error showed the following:
RROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'core-js/modules/es6.array.fill' in '/Users/xss/Documents/workspace/ng-uikit-pro-standard-6.2.1/node_modules/@antv/l7/dist'
ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'core-js/modules/es6.array.find' in '/Users/xss/Documents/workspace/ng-uikit-pro-standard-6.2.1/node_modules/@antv/l7/dist'
ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'core-js/modules/es6.array.find-index' in '/Users/xss/Documents/workspace/ng-uikit-pro-standard-6.2.1/node_modules/@antv/l7/dist'
ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'core-js/modules/es6.array.from' in '/Users/xss/Documents/workspace/ng-uikit-pro-standard-6.2.1/node_modules/@antv/l7/dist'
ERROR in<EMAIL_ADDRESS>Module not found: Error: Can't resolve 'core-js/modules/es6.array.iterator' in '/Users/xss/Documents/workspace/ng-uikit-pro-standard-6.2.1/node_modules/@antv/l7/dist'


[Feature request] Pipeline remove download file after process and extract single language

hi thank you for releasing tool!
since cc dump very size/disk demanding
can we have optional pipeline step like this:

immediately process(pipeline step) for each file in download command step instead to waiting all file download complete
remove the file immediately after being process (this will save disk ussage)

also
can we make optional pipeline that we can choose which language to process instead of process all of them
maybe something like
ungoliant pipeline download/ src/ -lang id
thank you before

Hello and thanks for the question, that's a great suggestion!

immediately process(pipeline step) for each file in download command step instead to waiting all file download complete

IIRC there is some code in Ungoliant that could enable this type of processing. However, I don't think the actual pipeline code is ready for this, and I don't know how much work it would take. There are two approaches here:

Download shards, and trigger their deletion once the shard has been processed (quite easy I think)
Treat shards as streams of data that will never touch the disc, and do the processing on-line.

I'm currently busy with other things so it might take some time to see this implemented.
There's also a third option that could be re-implemented for the latest pipeline, and that is not documented (because not extensively tested): We can provide "rebuild" files (one per language), that basically tells Ungoliant which shards to use and how to rebuild the corpus. These files contains metadata for each document.

can we make optional pipeline that we can choose which language to process instead of process all of them

While this could be done, you'd still need to do the whole process (identify all corpus, process all shards etc.). I guess that we could provide the option, but it will only help with disk size.

thanks for reply and the workaround!

Download shards, and trigger their deletion once the shard has been processed (quite easy I think)

can you pinpoint line of code/files that need adjust to make this work?
maybe i can test/try it in my fork(not expert on rust though)

There's also a third option that could be re-implemented for the latest pipeline, and that is not documented (because not extensively tested): We can provide "rebuild" files (one per language), that basically tells Ungoliant which shards to use and how to rebuild the corpus. These files contains metadata for each document.

yes i check this subcommand,,but dont undestrand how to use it ,especially this arg

shard == file with *.txt.gz ? or processed  *.txt.gz
righ with my "dirty" workaround to tackle size problem is by spliting wet.path into multiple files,each files have 2000 line/links
for each files i do this step:

run download command
run pipeline
get my specific json lang (in my case id_meta.jsonl),move it somewhere
remove all files inside download and pipeline folder and back to step 1

thanks again for reply

Adding something like remove_file at the end of the process_shard function should do it https://github.com/oscar-corpus/ungoliant/blob/master/src/pipelines/oscardoc/pipeline.rs#L221.
Feel free to try to implement it! However, to be merged, I think that it would be better to integrate this in the CLI (like ungoliant pipeline src/ dst/ --del_src or smth like that)
For rebuild files, we have not distributed them (yet), and I haven't ported the source code for the latest pipeline. I might do it if you're interested, but it's not top prio right now :(


Display a message whenever somebody hasn't turned on the MO

Fixes: #1922

See https://github.com/quintel/etmodel/issues/1922#issuecomment-173876967.

Ready to merge! :+1:

It looks like the new texts are in place? @grdw are you happy for me to merge this?

are you happy for me to merge this?

Yes :+1: :shipit:


Thumbnail Aspect Ratio on About Page

Just noting that we should probably update the thumbnail aspect ratio on the about page to match the challenges page.
About page:

Also, @fdoflorenzano should the titles on the challenges page be bolded like they are on the about page? I think that might look nice but defer to you all!

I agree the aspect ratio should match. I had a similar thought about it.
About the bolded titles, I'm not sure. There's too things happening there, the titles are bold and black, which brings a lot of contrast. This all also relates to color contrast concerns that I have, so I would probably focus on that aspect separately.

About the bolded titles, I'm not sure. There's too things happening there, the titles are bold and black, which brings a lot of contrast. This all also relates to color contrast concerns that I have, so I would probably focus on that aspect separately.

Got it! I don't have a preference and trust you all on your design decisions! I just had noticed it was different so pointed it out, the aspect ratio is the more important note here!


Crayons buffed to 45 uses

About the PR

I changed the 15 to a 45 in crayon charges.
Why / Balance

It was changed to make drawing easier on space station 14 artists.
15 uses per crayon is disgustingly low to legitimately draw in the game. you end up having to unreasonably demand thousands of dollars of crayons from cargo usually already tight wallet for a item that seems to have counter intuitive qualities.
Each box additionally takes up a 3x3 space in your backpack, making holding more then about 2 without stuffing every nook and cranny with crayons require the use of a toolbox or something similar.
Technical details

only change was changing the number that determines the charges for regular crayons from 15 to 45
Media

Issue visible here is that all of these were BARELY possible with 3 crayon boxes, as seen by the lines in most of them being... well- just a bunch of dots really. exception is the last one where a infinite clown crayon was used.
/Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[x] I have added media to this PR or it does not require an ingame showcase.

Changelog
:cl: Alice Liddel

add: Increased the stations drawing capabilities

Hey thank you for your first PR :)
I will have to ask the other maintainers if this may cause any performance issues and the game admins for potential griefing problems. According to maintainer policy the minimum internal discussion time is 3 days, so I will get back to you then.

Also be a little more specific in the changelog please, so players will know what actually has been changed.

done, dont know if i did it right but ive done the changes you asked... i think..

Looks good! Thank you for your contribution and your first PR!
I will merge as soon as the tests are through.


Only grabs default user profile for Chrome

Most Chrome users create an online profile for Chrome so they can sync all their stuff.
These new profiles get named Profile 1, Profile 2, Profile 3, etc.
This is why it wouldn't didn't pull anything for my Chrome.
Can you have it include ever Profile folder in User Data?
Also, can you grab Extensions, Extensions Data, Bookmarks & History?

Hi,
First, thanks for reported me this problem,
I can do a system where you can choose profile, and in your code for example
ChromeGrabber grabber = ChromeGrabber("Profile 1");

It's this you want?

No, I need it to iterate and grab the files for each profile that's found, because they all may have something different and useful.

Yes it's what i'm telling you, example
ChromeGrabber grabber = ChromeGrabber("Profile 1");

So after you can do
string[] profiles;
foreach (var item in Directory.GetDirectories("path of google"))
{
       if (item.Contains("Profile")
       {
              profiles.add(item);
       }
}
// so
ChromeGrabber grabber = new ChromeGrabber(item[0]); // for get Profile 1 for example

In this case, if i'm good, you can change the password path / cookie path:
                CockyGrabber.ChromeGrabber yourgrabber = new CockyGrabber.ChromeGrabber();
                yourgrabber.ChromeCookiePath = "path of the chrome profile cookie where you want to decrypt...";
                // and your code


Standarize json output keys format

/businesses uses underscores _, inspections uses camelCase, counties don't use any (countyname)
ugh Open Source :(

@marcoslhc this sounds interesting huh? ^ :/

We can use
(defn transform-keys
  "Recursively transforms all map keys in coll with the transform-key fn."
  [transform-key coll]
  (letfn [(transform [x] (if (map? x)
                           (into {} (map (fn [[k v]] [(transform-key k) v]) x))
                           x))]
    (walk/postwalk transform coll)))

and a transform-key function that uses regex to replace some_thing to some-thing for all endpoints.

LOL. I guess we should keep a standars page in the wiki and start refactoring

We'll use underscores for now. Just merged a PR for this. This is what the db/model uses and is simpler. We can reopen if we want to reformat all to camelCase.


Grid component’s tag attribute “could” cause issues

When using gcds-grid, the tag property can accept any HTML element. This can cause issues if a developer uses an element that deosn't make sense to render as a grid container.
To do:

[x] Limit allowed values for tag property
[ ] Add disclaimer about tag use to documentation site

PR for limiting the allowed values:
https://github.com/cds-snc/gcds-components/pull/470


Support indicating that a command is an install command

How would this feature be useful?
nox supports --no-install and -R options for not running installation commands, usually for use when reusing existing environments, and to speed up development where the install process is often the long pole.
But there doesn't seem to be a good way to indicate when something run via session.run is an install command -- specifically, if you're installing something not via pip or conda etc, which you may want to skip in the same scenarios as the above.
Concretely -- given a Javascript project, one might run (p)npm install to install all the project dependencies, followed by some test runner, e.g. jest, all under a noxenv called ui-tests.
In this scenario it would be nice if nox -s ui-tests -R was able to not run the install command, but the existing -R of course has no way to know which command to skip.
Even doing this externally (in the noxfile) seems nontrivial, as it seems the way session.install does this is by accessing private internals, e.g.:
https://github.com/wntrblm/nox/blob/5c82dc553bd04ee017784fc16193da0b35a44ab6/nox/sessions.py#L629-L630
Describe the solution you'd like
Add a session.run_install(...) command which runs a subprocess but assumes that subprocess is an installation command which will be skipped when using -R / --no-install.
Refactoring session.install and session.conda_install to call this to do their no_install check seems also like a nice minor internal refactor.
Describe alternatives you've considered
I personally don't like boolean flags (as a rule), but session.run(..., is_install=True) is an alternative for those who are less averse.
A further alternative is to make more things public API (e.g. whether no_install is active) so that someone can do this themselves in their noxfile, but given that nox itself has 2 places where it itself wants this, it seems like something generally useful to me.
Anything else?
A concrete noxfile where I wanted this functionality is here.

This already exists, but for historic reasons it's called session.run_always. It keeps tripping up people. IMO we should alias this to session.run_install, keeping the old name for compatibility.

Huh, I swear I saw run_always and for some reason thought it still wasn't the right thing, but looking at it again it does indeed look like you're right!
I understand the history I think, there was only a --install-only original right, and so this was "run in that case too" hence the "always" -- but yeah the name seems really unfortunate now.

Yup. On main of my repo all works as expected using it, though I'm going to likely alias it directly in my noxfile to remove the confusion.


Grid Component Set Maximum Height

How can I set the maximum height of the Grid? Please note that I set AllowPaging="false". I'm using Grid on my Dashboard together with other items. However, it is taking the full height of the screen leaving no space for other items. How can I set the maximum height of the Grid without paging i.e. AllowPaging="false"?

@joselitogatchalianalonzo In the future version, we will add Height paramter to the Blazor Grid component

Thanks @gvreddy04. I will try your sample code.


Can't do git fetch upstream release

While doing the following instructions:
git fetch upstream release
git checkout -B release upstream/release
The output is:
fatal: couldn't find remote ref release

Hi @mas-oliveira - I think the branch you're looking for is in another repo: https://github.com/MicrosoftDocs/mslearn-tailspin-spacegame-web-deploy (note the "-deploy" part at the end.) For the DevOps content, we pair GitHub repository per learning path.


add Camera action to the schema

Schema is here - https://github.com/EnsembleUI/ensemble/blob/main/assets/schema/ensemble_schema.json
To test your schema changes -

Use Visual Studio Code
use these instructions for adding custom schemas - https://dev.to/brpaz/how-to-create-your-own-auto-completion-for-json-and-yaml-files-on-vs-code-with-the-help-of-json-schema-k1i
point to the ensemble schema on your disk to test

@snehmehta please close this if no longer applies.


Is there temporal information in the datasets?

In each batch, samples are stored in tables and each table array partition contains IDs for all sampels.
For example, table0:[ids of sample 0, ids of sample 1, ... , ids of sample n ] ... table k:[ids of sample0, ...  , ids of sample n]
Is there temporal information in the array above? Is the trace generated in time series, sample 1 after sample0, and so on?

Hi @oleotiger,
There is no temporal information in the dataset. You may assume that the samples in the dataset are shuffled.

Hi @oleotiger,
There is no temporal information in the dataset. You may assume that the samples in the dataset are shuffled.

The paper "Understanding Data Storage and Ingestion for Large-Scale Deep Recommendation Model Training" from ISCA'22 describes the dataflow of RM training systems at Meta. It provides details on how training samples are generated and fed into models. Based on the description, it seems that the trace is generated as described in DSI, and the samples are generated one by one in chronological order, unless there is an explicit shuffle before trace publishing.
Is there any evidence to prove that there is no temporal information in the dataset?

Hi @oleotiger,
The purpose of this synthetic dataset is to support benchmarking embedding lookups. Therefore, the statistical properties that get preserved in the data generation process are only the ones involving the distributions of indices. While the data used in RM training may contain chronological information depending on if it is shuffled, it is not preserved in this dataset because there is some explicit randomization we employed while synthesizing this data that effectively shuffle the data. I hope this helps.


JBPM-6203 - Remove view mode preference

@mbiarnes @psiroky do you guys have an idea of why we are getting this error on branch 1.3.x of appformer? 12:49:01 Unexpected error while executing the UpstreamReposBuilder! Unknown branch '1.3.x'! Make sure the plugin is aware of the specified branch.
12:49:01 java.lang.IllegalArgumentException: Unknown branch '1.3.x'! Make sure the plugin is aware of the specified branch.

@ederign org.uberfire.ext.metadata.io.IOServiceIndexedGitImplTest seems to be failing randomly, could you take a look?

Jenkins please execute full downstream build.

@cristianonicolai sure. I'll take a look.

Jenkins please execute full downstream build.


Track URL ?

Hi, I see in the Content > Tracks that we can fill an URL of the Track. It says "Download links for tracks will be shown in Content Manager's server information screen.". So I have filled this field but I don't see anything in Content Manager when I select my server. Where is this displayed ? It would be very helpful to resolve checksum issues.
Thank you.

@tmeedend see if this info helps. You probably haven't enabled the right option / opened the port on your server firewall.
https://github.com/JustaPenguin/assetto-server-manager/wiki/Sharing-Content-With-Content-Manager-Wrapper

Thank you I understand better that's perfect it works ! I did no't know this.
One last thing, is it possible to make real links in the webapp ? It seems to be done with javascript and we cannot open links in new tabs. I woulk like to open all my tracks in new tabs to update their URL but I cannot, I have to manually open each one in the same window, and it takes time to load. A middle click to open on new tab would help a lot.
Thank you :)

I agree that would be cool, but I'd suggest opening a feature request as a new issue

Hi, ok I will do it.
One last thing about this url field, sometimes in the ui_track.json file of a track (and possibly of cars too), there is already an 'url' field with the url of the track (near the version). But I can see that it's not read by server manager ? Would it be possible ? What do you think ? For the moment, I see that if I fill the URL, it's written in another file (meta_data.json) with the notes. Why is this done in another file ?
And one last las idea, what do you think of the possibility to generate an archive of tracks / cars deployed so they can be downloaded ? This way, we would not need an external url at all. Is this a problem with permissions to redistribute the track ?
Thanks !


[less] Add namespace support

Maybe I'm doing something incorrectly (definitely possible!), but I receive errors whenever I try to reference a namespace. For example:
.vertical-center() {
	position: relative;
	top: 50%;
	#mixins > .translate-accelerate(0, -50%);
}

For the code above, VSCode will display an error stating } expected. Is this a known issue/is there a way around it?

Fixed by Microsoft/vscode-css-languageservice@551b86dc7a6b06828650f4d7696278646a5021f1


Added PinMAME functions script & helper files

Allows to control ROM volume & DMD of PinMAME conveniently from PinballY

Showpindmd (0 / 1) shows or hides the DMD controlled by PinMAME. Disabling the DMD is useful for older ROM based tables with alphanumeric displays, where PinMAME is showing basic numbers only on the DMD.
Volume (0 to -32) attenuates the volume of the ROM sound from PinMAME. This allows to balance out the volume coming from the backbox speakers vs. the volume of the playfield sound. This is for older ROM based tables, which do not have a DMD menu with volume control. 0 is the loudest (default). I have never needed values below -16. So this is the range being shown in the menu. It is usually sufficient to go in steps of 2, as I have never needed the steps to be finer.

The only change that needs to be done to the script is the scriptpath itself. It defaults to "C:\PinballY\Scripts" which should work out of the box for most installations.
The registry key for the ROM is retrieved from PinballYs metadata. In my experience, this works quite well for 95% of the tables that I have used it for so far.
The changes to the Windows registry are being done by a small helper application, built with AutoIt3. You can use the provided EXE or build it yourself from source.

@mjrgh
Not sure if you noticed this PR. You might want to have a look at it.


request revokable token using client with too many scopes

getting a revokable token through oauth/token fails with 500 error and prints sql query when a client has scopes with total comma-separated length over 1000 chars. Caused by psql exception when storing revokable token throwing a DataIntegrityViolationException.
Steps to reproduce:

create a client with a LOT of scopes/authorities
post to oauth/token with client_credentials using client, without specifying scopes and with token_format revokable or opaque.

uncaught exception specifically being thrown in JdbcRevokableTokenProvisioning.create(), the returned 500 is as follows:
{
"error": "server_error",
"error_description": "PreparedStatementCallback; SQL [INSERT INTO revocable_tokens (token_id,client_id,user_id,format,response_type,issued_at,expires_at,scope,data,identity_zone_id) VALUES (?,?,?,?,?,?,?,?,?,?)]; data exception: string data, right truncation;  table: REVOCABLE_TOKENS column: SCOPE; nested exception is java.sql.SQLDataException: data exception: string data, right truncation;  table: REVOCABLE_TOKENS column: SCOPE"
}

Fixed via
https://github.com/cloudfoundry/uaa/commit/5b3613b3d62b4904ef6c6ba560e1cadc33c967f9


FileNotFoundError: [Errno 2] No such file or directory: './workdirs/MonoDTR/output/validation/imdb.pkl'

where is this imdb.pkl file?

@M0E313 You need to generate it as described in DATA.md


Initial version of Google Cloud Pub/Sub Spring Integration channel adapters

This is the very early stage of the inbound and outbound adapters.
There is still a lot of work required here, I'm just looking for some early validation and eyes on the code. There are also some hacks that I added just so the project would build and run.
The prototype does work on basic use cases, though its usage is still a bit cumbersome on the user side (e.g., no credentials support, clunky project ID injection, etc.)

@joaoandremartins Please take a look at the very simple PubSubMessageTemplate. That's an example, not a final class.
Things that needs to be take care of:

Converters : I'd vote we use a type of CompositeMessageConverter with a list of pre defined message converters. We have converters for String payload such as JSON and XML that do not need the payload to be String, they can be Pojos that Jackson can marshall into json for us. We will need to add header support for content type inference as well. For binary types we can just add support for java serialization or kryo (both supported ATM by integration), those usually yield byte[] as the result of the conversion, and we can use that as an input to ByteString. I recall using that and Avro successfully in the past.

The send method needs better exception handling, the way it's there now it was meant just to compile

With this approach, we now have  MessageTemplate that can send to any topic and we share the executor. Another easy optimization is adding something like String[] topics; as a member variable and on afterPropertiesSet we can pre-populate the Publishers map. Would be useful for the outbound gateways that can set that since they will mostly just attach to a fixed size of topics.

Let me know if you have questions with this approach, we can move to how to handle conversion next.

Well, I think the best sample of the template pattern impl to talk about separation of concerns and more granual API is KafkaTemplate: https://github.com/spring-projects/spring-kafka/blob/master/spring-kafka/src/main/java/org/springframework/kafka/core/KafkaTemplate.java. Also see its converters infrastructure: https://github.com/spring-projects/spring-kafka/tree/master/spring-kafka/src/main/java/org/springframework/kafka/support/converter.
With that in hands it's not so hard to guess that the SI MessageHandler implementation on the matter is pretty simple: https://github.com/spring-projects/spring-integration-kafka/blob/master/src/main/java/org/springframework/integration/kafka/outbound/KafkaProducerMessageHandler.java
@viniciusccarvalho , please, don't confuse people. We don't talk here about MessageTemplate. Let's call things with their arranged names - PubSubTemplate!
The other concern and we had some discussion in the design document is about the for the class which represents Outbound Channel Adapter. Well, that concept exists only for the SI XML config for conveninece. When we come to Java there is only Consumer Endpoint to get data from the MessageChannel and MessageHandler to accept that data. There is no such a real solution like OutboundChannelAdapter class. If you meet some of the MessageHandler implementations with such a name, that is wrong. Because the Outbound Channel Adapter principle implementation with Java can be reached only artificially or, if you wish, by convention. No one stops you to return something from your @ServiceActivator and send it into some channel.
What I mean with that, that we talk about some MessageHandler implementation which just void and send to the PubSub and forget. There is no need to confuse with the outbound gateway. That is my point.

@artembilan I forget about the KaftaTemplate, indeed a very clean implementation. We really should follow, even the sendDefault would work nicely as well as having a PublisherFactory, thanks for advising

Regarding merging, someone with admin on the repo can make "Squash & Merge" the default. We do it on our other repos as well.

Well, "Squash & Merge" is not enough in most cases. We also do rebase before merging to support linear commits history in the master:

Regarding the rebase usually the person approving the PR will take care of it.
Also @joaoandremartins I noticed your classes are missing the ASL header, please look at some of the classes int the spring-cloud-gcp-core module for examples.
Also please add the @author in the class javadoc. We usually follow firstname lastname conventions, examples on the module as well

I might be missing something here, but my understanding is that "Squash & Merge" adds the squashed commit on top of the last commit in master, maintaining the linear commit history. You might be required to rebase against master or merge master into your branch before using "Squash & Merge", but that's enforced by GitHub.

We do rebasing, I just call it squash and merge, but in this case we would do git rebase -i HEAD~X and then squash all commits until the latest one, and finally push this to master.
@artembilan Gary mentioned how you guys work together, I have never yet worked with a direct branch, and I do have a commit in the logs now. I honestly don't mind having it squashed along everything Joao is submitting, but how do you guys solve that?

I see, you do it manually. We're talking about the GitHub PR feature that does it for you.

@meltsufin Someone once explained to me why we don't use that feature, but I forgot and I just continue using it manually 😄

One more time: that "Squash & Merge" does not do rebase.
To avoid all those conflicts we rebase and squash manually or using IDE features and only after that merge into master: https://github.com/spring-projects/spring-integration/wiki/Administrator-Guidelines

Apologies, I still don't get it. If the branch isn't in sync with master, then you can't merge the PR to begin with. You must perform a manual merge or rebase, whichever flavour suits you the best, push and merge.
But the squashing part can be done automatically by GitHub. I'd prefer using this automated tool rather than doing the interactive rebase manually, it feels like a retrocess.

Well, our experience shows that it is much better to pull your PR locally and make a proper review and testing there. And since your branch copy is in my local repository it isn't so hard to perform a couple Git commands to have it merged to master properly.
During local review I may make some refactoring and small improvements. For example those missed Copyrights and @author in your new classes. Mostly we don't see reason to make a new commits for that and just do polishing during merge process. No reason to jump to GH for squashing and back to local version if you can do everything locally and have a proper commit in the master. Another polishing I usually do is final commit message. There is an epic article from my former colleague on the matter: https://chris.beams.io/posts/git-commit/. You can't do that from GH.

Regarding the copyright, I noticed that it contains Copyright 2017 original author or authors..
How do you feel about adding Google Inc. either here, or on a new line?

I don't think we have that as a practice - Adding the company name anywhere in the license.
Which reminds me, have you signed our CLA?

This is Pivotal's project, so you should follow with our License agreement.

I haven't signed the CLA, yet. Where can I do it?
We're contributing to this project as part of Google Inc. As I understand, the copyrights will always be with Google, regardless of whether the notice says that or not. It might be OK to omit Google Inc. from the notice.

Oops, that's a big red flag to us. @joaoandremartins @meltsufin and any others that would like to make commits into the project, we need you guys to sign the CLA: https://github.com/spring-cloud/spring-cloud-dataflow/blob/master/spring-cloud-dataflow-docs/src/main/asciidoc/appendix-contributing.adoc
From your comments Joao, I'd say check with your legal team, what's Google policy on allowing employees making contributions to open source projects, we don't want to get in any sort of problems later.

@viniciusccarvalho ,
we have to add Webhook to the Pivotal CLA for this project, so no one contributor will miss CLA signing in the future.
Not sure how so...

@joaoandremartins Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@joaoandremartins Thank you for signing the Contributor License Agreement!

@joaoandremartins Somehow my project is not compiling anymore, it states that subscriber.stopAsync(); is not a valid method. We were based on 0.17.2 right have we moved to 0.18.0?
You don't need to change anything, this is something I can modify on the PR review and rebase as well as formatting of the code.
I just noticed that there are tons of duplicated libraries such as guava 18 and 19. I'll see what I can do before reaching out to you guys.

Close via dd44846bcf29663c28e6e7e0a494fd96ce2e6b50 and 2d28b5748cd285b5be6de373675f107ab6f2ff13

Are you using IJ? I got the same issue there and I think it's an IJ bug, since mvn clean install builds successfully.
https://youtrack.jetbrains.com/oauth?state=%2Fissue%2FIDEA-173421

It looks more like an issue on google-cloud-java, reverting to 0.17.2-alpha fixes the problem on my end. I'm simplifying the dependencies for now, but we need to discuss further is quite a messy web of libraries in google-cloud-java

There is an effort to improve the dependency hell on the client lib side. If you find any specific issues, please file them on google-cloud-java.


Add form "Info"

Add a modal form "Info" to display

version
copyright notice
link to github

Released with version <IP_ADDRESS>


Router Update

Implementation of react-router is very basic in this project, so can upgrade to new version with no issues.
Easier/better for people to refer to the latest documentation when trying to implement more advanced features.

Cheers Paul!


Update bundle, cloudwatch, sdk-core, ses, sns, ... to 2.25.27

About this PR
📦 Updates

software.amazon.awssdk:bundle
software.amazon.awssdk:cloudwatch
software.amazon.awssdk:sdk-core
software.amazon.awssdk:ses
software.amazon.awssdk:sns
software.amazon.awssdk:sqs
software.amazon.awssdk:ssm

from 2.25.15 to 2.25.27
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "software.amazon.awssdk" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "software.amazon.awssdk" }
}]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #3162.


hb_mc_init_device() and device_launch() should accept memory buffer

https://github.com/bespoke-silicon-group/bsg_f1/blob/b15423845a81f05858d7e4061d47b4707237ff33/cl_manycore/libraries/bsg_manycore_driver.cpp#L432
https://github.com/bespoke-silicon-group/bsg_f1/blob/b15423845a81f05858d7e4061d47b4707237ff33/cl_manycore/libraries/bsg_manycore_driver.cpp#L842
These two functions take a string of a file location and load the binary from the file system.
But in some use cases, the binary will be in the memory already and we just want to pass the memory pointer to the function.
I think it will be better if this function can take memory pointer as input.
For the cases the binary is on the file system, maybe user just write some simple function to read it to the memory and pass it to the function.
Reference: cuLaunchKernel

@cylinbao Is this resolved in Borna's new API?

Hi thanks for addressing this.
This isn't resolved directly yet.
As here
https://github.com/bespoke-silicon-group/bsg_f1/blob/e48613ddb11c7b8d622653974e98211414927928/cl_manycore/libraries/bsg_manycore_cuda.cpp#L596
It still get file_path as input.
But it is solved in a lower level api with this
https://github.com/bespoke-silicon-group/bsg_f1/blob/e48613ddb11c7b8d622653974e98211414927928/cl_manycore/libraries/bsg_manycore_cuda.cpp#L638
Maybe @bornaehsani can add a program init function that calls hb_mc_loader_load() more straightly.

Hi Chien-Yu! Thanks a lot for pointing this out. Yes this is still not implemented, which is why I have left it open for now. But I will implement it very soon.

This issue is now addressed in PR #265 .


Code comments about what example fold_virt_text_handler is doing

Feature description
I am just starting out with ufo, and I’m having trouble understanding what exactly the example fold_virt_text_handler does, and how it is doing it. From the screenshot and the code, it looks like it’s adding virtual text with the number of lines in the fold?
Describe the solution you'd like
Some description of what the example customization is doing, and code comments about what some of the inputs / changes are.
Additional context
No response

Yes, feel free to update the docs :)

That helps, thank you!


Add HasParam func to Context

This func will allow developers check if a request has a specific param. This is useful when using URI parameters.

#33 resolved this issue. Closing issue.


cypress: worked on flaky ui cypress

Describe your changes:

activityFeed
addAndRemoveTierAndOwner

Type of change:

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

Failing cypress is covered in https://github.com/open-metadata/OpenMetadata/pull/12772


Unfortunately myApp has stopped

Hi I am developing Hybrid Mobile Application. Once i create app and installed .apk file in Mobile it is working fine . But the same .apk file installed in samsung galaxy tab3 it is not working once i scaned barcode automatically app will closed and showing an error alert "Unfortunately myApp has stopped." I am getting this Error repeately . So please help me any to resolve this error on samsung galaxy tab3. Android Version 4.1.2

Hi I am just learning how tp basic program plus im am trying to build my
own wide area network to live in because the usa stole my children and
killed my boyfriend. If I knew how to help you I would.
On Thursday, December 3, 2015, Majppl<EMAIL_ADDRESS>wrote:

Hi I am developing Hybrid Mobile Application. Once i create app and
installed .apk file in Mobile it is working fine . But the same .apk file
installed in samsung galaxy tab3 it is not working once i scaned barcode
automatically app will closed and showing an error alert "Unfortunately
myApp has stopped." I am getting this Error repeately . So please help me
any to resolve this error on samsung galaxy tab3. Android Version 4.1.2
—
Reply to this email directly or view it on GitHub
https://github.com/phonegap/phonegap-cli/issues/562.

Hi @Majppl,
Your issues sound as though it's related to Android and the Barcode Scanner plugin.
I'd recommend that you ask your question on the PhoneGap Google Groups:
https://groups.google.com/forum/#!topic/phonegap/
Good luck!
Michael


deadlock when listening tcp

I was hit by massive deadlocks in unbound listening for tcp connections. Sometimes it arrives to state when number of idle tcp connections is maxed at incoming-num-tcp. New connections are not accepted, listening socket is excluded in select().
But idle unused connections never die. So, listening stops forever.
Not sure, but it looks like I found the reason. When incoming-num-tcp is about to be full the last added sockets are added
with c->tcp_timeout_msec==0. The first time in setup_tcp_handler timeout is forced to be TCP_QUERY_TIMEOUT_MINIMUM.
But if the socket did some io, the further comm_point_start_listening() are done with real 0(!) and timer is not started anymore. After several expirations of old sockets with non-zero timeout all the sockets remain with c->tcp_timeout_msec==0 and unbound stucks forever.
It is just a theory. I added capping c->tcp_timeout_msec to TCP_QUERY_TIMEOUT_MINIMUM in setup_tcp_handler().
Waiting for results. But I think more experienced developer could figure out the problem w/o waiting.

Actually, I think the approach with reducing idle timeout when we are congested is deemed to fail. I have some experience with this in linux kernel. I used similar solution there around 2000th, it was cheap and looked nice but soon it became obvious it will never sustain even minor overloads. In single thread unbound the simplest and rock solid solution would be lru list per listening socket, when we remove socket from list when start to do something and put it to the tail when socket enters idle state. Then, when we !c->tcp_free we just close head of lru and proceed. It is not even so chumbersome, is going to be nice and clear. And we do not need  redundant tcp-idle-timeout parameter anymore, established connections are closed when it is necessary.

Hi Thanks for the detailed reasoning about the deadlock and TCP streams.  I fixed it by applying the minimum so the no timeout zero value is not used, but the minimum value.  This should no longer deadlock the TCP streams.


H2 DB Upgrade

This PR will:

modify the test queries needed to upgrade H2 db version from 1.4.199 to 2.1.210.
remove the H2 console from the IS product.

NOTE: Needs to bump the carbon-kernel, carbon-identity-framework, identity-inbound-auth-oauth, carbon-consent-management and carbon-business-process versions after these PRs are merged.
Related to: https://github.com/wso2/product-is/issues/13162

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/1934093256

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/1934093256
Status: cancelled

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/1934238810

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/1934238810
Status: failure


Avoid division by zero in rechunk

cc @pitrou does this look ok to you?

Fixed
On Tue, Jan 10, 2017 at 9:42 AM, Jim Crist<EMAIL_ADDRESS>wrote:

@jcrist commented on this pull request.
In dask/array/tests/test_rechunk.py
https://github.com/dask/dask/pull/1902#pullrequestreview-15971268:

@@ -361,3 +361,13 @@ def test_plan_rechunk_heterogenous():
#  * cf -> fc also fits the bill (graph size and block size neutral)
steps = _plan((cc, ff, cf), (ff, cf, cc), block_size_limit=100)
_assert_steps(steps, [(cc, ff, cc), (ff, cf, cc)])

+def test_rechunk_warning(capsys):

N = 20
x = da.random.normal(size=(N, N, 100), chunks=(1, N, 100))
x = x.rechunk((N, 1, 100))

Perhaps:
with warnings.catch_warnings(record=True) as w:
x = x.rechunk((N, 1, 100))assert not w
https://docs.python.org/2/library/warnings.html#testing-warnings
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask/pull/1902#pullrequestreview-15971268, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AASszD7T9EPnuWGpWKLQw6cqWFEjNhqeks5rQ8LogaJpZM4LfsLe
.


avoid rhn re-registration on every ursula run.

This PR is to address below rhn subscription optimizations:

Register system only when it is not subscribed to rhn
dislablae/enabled rhel repos only for new subscription or refresh repo is true

ok to test

Build started sha1 is merged.

ok to test

ok to test

retest

ok to test


System freeze after Firmware  "update"

Hello all,
I have re-flashed the mksV2.1 board  FW (1.11.5) because i swapped the Axes of the Digital level.
Yesterday system works fine.
After this wenn turn on OAT the Display und system works, Powerled is on.
Moving RA/Dec axis to home, that  works fine.
Digital leveling works fine.
because Indoor GPS is not found, press 2x the right button to go on.
then after this the display shows RA DEc and 01h etc
A few sec later the system freeze , no Displaybutton respond, and the display shows no h.
On the MKS Board the blue led next to powerled and red led next to mosfet turn on
HE1 goes from  0V to 12V
I used the firmware configurator and flashed it several times
result is always the same.
OpenAstrotech Discord :
https://discord.com/channels/693401540744183859/1027676652718792714

Each Time i made a "clean all" bevor build and upload.
I tested several combinations of add-ons.
All Add-ons alone work without problems.
If there are two, there are no problems, no matter what combination.
Only if I use all three (gps, leveling, AutoPA) together
then the system break after skipping the "GPS-Sat search" (indoors) with the R Button when it reaches the "main menu".

Sorry, was this resolved @stephanp66 ? There hasn't been any major FW changes since you had this problem?

Sorry, was this resolved ?
Yes and No. :)
I used the OAT after this without the Digital Leveling, Ohnly GPS and AutoPA. And it worked fine.
After a Week or so my Dec Motor got hot and noisy (it was a Cheap from China) so i changed it to a Quality Motor.
Then i thougt to try again with all Addons and activated the Digital leveling in the Configuration.local
build and uploadet it and it worked without errors.


Add exit dumper

:clipboard: Add associated issues, tickets, docs URL here.
Overview
Describe what your Pull Request is about in a few sentences.
This PR is for the migration.
Since L2ToL1MessagePasser.sol is updated in the v3 and it's behind the proxy, we need to dump the exit message and write them to new storage to avoid the double exit in the v3.
Changes
Describe your changes and implementation choices. More details make PRs easier to review.

Add dumper struct and code to write data to a file

Testing
Describe how to test your new feature/bug fix and if possible, a step by step guide on how to demo this.
The unit tests are added

I bumped up go version to v1.18 so that it supports os.CreateTemp and os.ReadAll.


Host thundergolfer/bazel-mypy-integration

I've been working with @thundergolfer and a client to make Python typechecking work well under Bazel.
Eventually it would be nice for rules_python to provide typechecking, but for now this repo works and might get wider adoption.
It meets the criteria https://bazel-contrib.github.io/SIG-rules-authors/hosting-policy.html#adding-criteria:

License is MIT
Typechecking is a widely needed capability for python users
Point of contact is myself
Already in use by some companies, and well-documented. Has examples and testing on GH actions
Will agree to reply to issues/PRs in 2-3 weeks
At least myself and @thundergolfer will review/approve PRs
Has been publishing semver releases.
Works with Bazel 5

Commenting to record my agreement with the above. 👍

Our hope with this repo is that it becomes more mature and can eventually graduate into rules_python, or perhaps Google will starlarkify the python rules and then it's more trivial to introduce type-checking.

This was approved in the SIG meeting today.

Nice. So I can't make you an admin of bazel-mypy-integration because it's a personal repo, and I can't currently transfer the repo because I don't have permissions. I need to get past this error:

You don't have permission to create public repositories on bazel-contrib

Which must involve me gaining powers in bazel-contrib.

Try now?

Same issue:

You don’t have the permission to create public repositories on bazel-contrib


[FEATURE REQUEST] Add HANA SQL Endpoint to metadata_log.json

Which area is mainly impacted
Other
Requested Feature
I would like to have the HANA SQL Endpoint as part of the metadata_log.json
Use Case/Scenario
As we deploy our application to the created kyma cluster we would need to have the SQL Endpoint available during deployment. Currently the value is given by the user during execution of the script. If it would be available in the mtadata_log.json we could do the deployment without user interaction.
Would you like to support us?

[X] Yes, I would like to support you

Anything else?
No response

Where do you normally get the SQL endpoint from? Is there an API or is it something that the CF or BTP CLI can provide?

I've just created a HANA instance on my trial account and executed the CF CLI to see the metadata it provides.
Is the API endpoint one of the outputs?
`3644e40c4fac:~$ cf service "hana_cloud_instance"
Showing info of service hana_cloud_instance in org cloudfoundry-btp-setup-automator-us10-87b7b1a7-1a78-4910-ab3 / space development as<EMAIL_ADDRESS>name:             hana_cloud_instance
service:          hana-cloud-trial
tags:
plan:             hana
description:      Leverage the in-memory data processing capabilities of SAP HANA in the cloud as one simple gateway to all data.
documentation:    https://help.sap.com/viewer/p/HANA_CLOUD
dashboard:        https://hana-cockpit.cfapps.us10.hana.ondemand.com/start?host=24b1a049-d694-4c35-86a2-153549a4dae2.hana.trial-us10.hanacloud.ondemand.com&component=cockpit&port=443
service broker:   sm-hana-cloud-trial-1ce8c293-a0f4-40b1-a52e-a70f7ac4a749
Showing status of last operation from service hana_cloud_instance...
status:    create succeeded
message:   Backup not ready
started:   2022-04-03T10:31:48Z
updated:   2022-04-03T10:43:51Z
There are no bound apps for this service.
Upgrades are not supported by this broker.`

Addressed with new commit https://github.com/SAP-samples/btp-setup-automator/commit/b08720f123751995eb7b392d64b6deeb8e044ec4 . Please use the collected metadata in the metadata_log.josn file to fetch the SQL endpoint.


Split plugin.go into several files

This is just a nit, but to be consistent with other "objects" 🐮.
Unit testing will come later 👼.
/cc @tiborvass @thaJeztah @runcom @calavera
🐸
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>

LGTM

LGTM, should we wait for @tiborvass?


x/build: leave farmer page for successful runs

CL 7360 started trybot run http://farmer.golang.org/try?commit=26677cd2. That farmer page was present for a while but now (30 minutes later) says "TryBot result not found (already done, invalid, or not yet discovered from Gerrit). Check Gerrit for results.". I presume that this is because the trybots were happy(?). I'd rather leave up a page that says that.

The coordinator is stateless. It keeps some state in memory, but only a rolling window for the past N builds as a debug/curiosity mechanism. If it crashes or is restarted, it regains its state from Gerrit and build.golang.org. I'm not sure how such a feature would be implemented.
The /try?commit=... URL is for people bored and interactively watching their try runs. The state gets stored back in Gerrit as a comment and -1/+1 vote as the TryBot-Success label.
I suppose I could make unknown things just redirect back to the issue on Gerrit?
Suggestions?

interactively watching their try runs.

This was me.

I suppose I could make unknown things just redirect back to the issue on Gerrit?

SGTM

Nowadays we have all the history uploaded to datastore so on that URL (http://farmer.golang.org/try?commit=26677cd2) we could look up 26677cd2 and at least say "That passed, here's the Gerrit link."
/cc @toothrot

It looks like there's agreement to do even better and keep entire logs accessible in issue #34119. I'll close this 2015 issue in favor of that one. (@toothrot has already linked it there, so we can find it easily.)


R geoprocessing tool failing on ArcGIS Server

I have an R script which I successfully tested on Desktop and published as a geoprocessing tool but when I run the service I get the following error message:
Failed to initialize R interpreter. Please install recent version R for Windows Failed to execute (SearchString). Failed to execute (SearchString).
Do I need to have R installed in the same environment of ArcGIS Server?

@notoriusjack Yes, you'll need to install both R and the bridge on the server hosting the Geoprocessing service. You can install a normal R installation on the server, and install the package directly from within R:
https://user-images.githubusercontent.com/12039804/36178881-39dd48e0-10cf-11e8-9357-300a6751c39c.png
The archive containing the package is available here.
If that doesn't work, report back here and we can dig deeper.

Thank you for replying. I have tried that but I got the following error message:

library(arcgisbinding)
*** Please call arc.check_product() to define a desktop license.
arc.check_product()
Error: Could not bind to a valid ArcGIS Pro installation.

Do I really need to also install ArcGIS Pron on the server?

IT made a mistake and installed R on the server controlling Portal. Apologies for the confusion.
However after installing R on the servers controlling ArcGIS Server I am still encountering a problem with the installation.
If I try to install arcgisbinding on the 64 bit version of R I get the following message:
Error: Could not bind to a valid ArcGIS Pro installation.
Found ArcGIS Desktop (32-bit) Version 10.5.6491.
You'll need to use the 32-bit version of R.
But if I try to install it on the 32 bit version of R  it says it cannot find an ArcGIS Desktop installation with the message:
Could not bind to a valid ArcGIS Desktop installation.
I'm puzzled

Resolved!
Even if you get that error message that it cannot find the Desktop installation the script will run correctly anyway from a published geoprocessing service.

@notoriusjack Great! Glad you were able to get up and running. That's correct, currently the standalone R interpreter from R doesn't work directly from Server, but it works fine in a Geoprocessing Service context. We are working on updated documentation for working with R and Server, and will be sure to include this information.
Cheers,
Shaun


Possible bug found

https://github.com/netlify/netlify-plugin-edge-handlers/blob/175a1e306f611830f80414bd2d279206d5fcc9ec/src/upload.js#L56
There might be a silent bug discovered in the function. I've added a description below
of the issue followed up by a suggestion on improvement.
// @returns {Promise} Whether the bundle was newly uploaded (and did not already exist)
{
  await fetch(uploadUrl, {
      method: 'PUT',
      body: buf,
      headers: {
        'Content-Type': CONTENT_TYPE,
      },
    })

    return true
}

Description

POST: request error does not accurately reflect the cause in the function context.
PUT: slightly confusing here contradicting the function return type description.

Suggestion;
Perhaps one way to improve on readability by chaining promises with error handlers in function scope.
Additionally, showing the dependency sequence on success of requesting a valid link, followed by the upload request described below;
PUT here will return success statuses for completing the upload request as;
a) a new resource was created (201)
b) and either resource replaced (200) or nothing to replace (204).
See docs here
Combined with the error thrown in line 39 below; may cause a misunderstanding of an actual error.
 const { error, exists, upload_url: uploadUrl } = await resp.json()
  if (error) {
    throw new Error(`Failed to upload: ${error}`)
  }

Hopefully, the issue and suggestion does not cause confusion otherwise I can clarify them and let me know your thoughts. Thanks.

Hi @daxeh, thank you for reporting this issue.
I'm having a hard time understand the bug.
Can you provide a step by step reproduction of the bug and also describe the actual result vs the expected result?

2. contradicting the function return type description.

In which cases the function doesn't return a Promise<boolean>? Looks like we always return a boolean or throw an error.

PUT here will return success statuses for completing the upload request as;
a) a new resource was created (201)
b) and either resource replaced (200) or nothing to replace (204).

We check for an existing resource here:
https://github.com/netlify/netlify-plugin-edge-handlers/blob/175a1e306f611830f80414bd2d279206d5fcc9ec/src/upload.js#L41

may cause a misunderstanding of an actual error

Can you elaborate on this use case? What are the steps to receive the confusing error, and what error would you expect to receive?


runtime error when importing identitystore

i keep getting this runtime error when importing identitystore. the command used was time terraformer import aws --profile=dev --regions=us-east-1 --resources="config"
2024/01/04 08:37:25 aws importing... identitystore
panic: runtime error: index out of range [0] with length 0

goroutine 1 [running]:
github.com/GoogleCloudPlatform/terraformer/providers/aws.(*IdentityStoreGenerator).GetIdentityStoreId(0x14001849098?)
	github.com/GoogleCloudPlatform/terraformer/providers/aws/identitystore.go:44 +0x198
github.com/GoogleCloudPlatform/terraformer/providers/aws.(*IdentityStoreGenerator).InitResources(0x0?)
	github.com/GoogleCloudPlatform/terraformer/providers/aws/identitystore.go:169 +0x20
github.com/GoogleCloudPlatform/terraformer/providers/aws.(*AwsFacade).InitResources(0x14003e17cc0?)
	github.com/GoogleCloudPlatform/terraformer/providers/aws/aws_facade.go:61 +0x28
github.com/GoogleCloudPlatform/terraformer/cmd.initServiceResources({_, _}, {_, _}, {{0x140007cd800, 0x56, 0x80}, {0x111d956f0, 0x0, 0x0}, ...}, ...)
	github.com/GoogleCloudPlatform/terraformer/cmd/import.go:209 +0x1a4
github.com/GoogleCloudPlatform/terraformer/cmd.initAllServicesResources(_, {{0x140007cd800, 0x56, 0x80}, {0x111d956f0, 0x0, 0x0}, {0x1084860ee, 0x1e}, {0x1083dd607, ...}, ...}, ...)
	github.com/GoogleCloudPlatform/terraformer/cmd/import.go:166 +0x134
github.com/GoogleCloudPlatform/terraformer/cmd.Import({_, _}, {{0x140007cd800, 0x56, 0x80}, {0x111d956f0, 0x0, 0x0}, {0x1084860ee, 0x1e}, ...}, ...)
	github.com/GoogleCloudPlatform/terraformer/cmd/import.go:98 +0x1a0
github.com/GoogleCloudPlatform/terraformer/cmd.importRegionResources({{0x140008d7740, 0x1, 0x1}, {0x111d956f0, 0x0, 0x0}, {0x1084860ee, 0x1e}, {0x1083dd607, 0x9}, ...}, ...)
	github.com/GoogleCloudPlatform/terraformer/cmd/provider_cmd_aws.go:121 +0x200
github.com/GoogleCloudPlatform/terraformer/cmd.newCmdAwsImporter.func1(0x14000cfe580?, {0x1083cebea?, 0x3?, 0x3?})
	github.com/GoogleCloudPlatform/terraformer/cmd/provider_cmd_aws.go:58 +0x804
github.com/spf13/cobra.(*Command).execute(0x14000cfe580, {0x14001682690, 0x3, 0x3})
	github.com/spf13/cobra@v1.1.1/command.go:850 +0x4c4
github.com/spf13/cobra.(*Command).ExecuteC(0x14000ed0580)
	github.com/spf13/cobra@v1.1.1/command.go:958 +0x34c
github.com/spf13/cobra.(*Command).Execute(...)
	github.com/spf13/cobra@v1.1.1/command.go:895
github.com/GoogleCloudPlatform/terraformer/cmd.Execute()
	github.com/GoogleCloudPlatform/terraformer/cmd/root.go:36 +0x20
main.main()
	github.com/GoogleCloudPlatform/terraformer/main.go:39 +0x54

Same issue here. Not sure if it is relevant, but I am using version 5.33.0 of the AWS provider for Windows (AMD64).
Used command:
terraformer plan aws --output hcl --resources="*" --verbose

Same issue with version = "~> 5.47"

Probably introduced in #1685  ?

Still an issue

Fixed in #1808 . However no new release since the fix made it into the codebase


but Defender for Endpoint is set to Allow, should be updated in order to clarify the comment.

[Enter feedback here]
Dear Sir or Madam,
We need clarification in the comment below...
Change from:
/*
In situations when Microsoft Defender Antivirus is set to Block, but Defender for Endpoint is set to Allow, the policy will default to Allow.
/
to:
/
In situations when Microsoft Defender Antivirus is set to Block, but Defender for Endpoint - Indicators - File hash or Certificate is set to Allow, the policy will default to Allow.
*/
Thanks,
Yong

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 1ef58bda-c0ee-ad76-20e8-b226af0cadb6
Version Independent ID: c8580dd2-6ff9-550c-0b0e-6be69523b5f2
Content: Create indicators for files
Content Source: microsoft-365/security/defender-endpoint/indicator-file.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @mjcaparas
Microsoft Alias: macapara

Fixed.


Unable to install - No matching version found for<EMAIL_ADDRESS>

When I try to install css-blocks by npm i -D css-blocks installation fails with this error
npm ERR! code ETARGET
npm ERR! notarget No matching version found for<EMAIL_ADDRESS>...
npm ERR! notarget It was specified as a dependency of 'css-blocks'
Seems like wrong dependency entry was published.
I'm running node 10.1.0 and npm 6.1.0 in WSL.
Install fails also with npm 5.6.0 and node 10.1.0 in windows.

Well that would be because<EMAIL_ADDRESS>doesn't exist :) Only 0.3.0 has ever been published, and because the package is pre-1.0 the ^ means NPM won't automatically grab the latest minor: https://www.npmjs.com/package/@opticss/template-api
However, thats not the actual problem. Integrating with css-blocks isn't quite as easy as simply installing it. You'll install both a build integration (like @css-blocks/webpack, or @css-blocks/broccoli) and a template integration (like @css-blocks/jsx or @css-blocks/glimmer) and consume each as their READMEs direct. This should be explained in the project documentation, please let us know if there are ways we can make it clearer!
The css-blocks package should actually contain nothing – we're squatting on the package name so someone else doesn't take it confuse people. However, it seems like some ancient version of CSS Blocks was accidentally published to it! I believe we have a plan to publish an empty project there with a postinstall hook that throws a nice, helpful "You probably didn't mean to install that package! Heres a link to the integration documentation" error to point users in the right direction. Sounds like we should get on that sooner rather than later!

Ah, thanks for the explanation, I realized I will need @css-blocks/jsx , webpack etc, but thought this is a core package i will need in any case. Silly me.


Add method to combine columns of (concat horizontally) two Tables

Rationale for this change
 Add additional functionality into the concat_tables function
What changes are included in this PR?
 concat_tables in python/pyarrow/table.pxi
Are these changes tested?
 I have tested it as far as I could, I may have missed edge cases though
Are there any user-facing changes?

Would you mind create a github issue for this?

Hello,
Sorry, I didn’t see the first email. I’ll link it to the issue I had seen a
little while ago. Thank you for reaching out and apologies for the delayed
response
On Fri, Apr 12, 2024 at 00:47 mwish @.***> wrote:

Would you mind create a github issue for this?
—
Reply to this email directly, view it on GitHub
https://github.com/apache/arrow/pull/41161#issuecomment-2050966527, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AWDTKVB7P34V7DJOWDGJA53Y45RNBAVCNFSM6AAAAABGDGSC7SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJQHE3DMNJSG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


[Feature] Quantization aware training

Prerequisites
Please answer the following question for yourself before submitting an issue.

[x] I checked to make sure that this feature has not been requested already.

1. The entire URL of the file you are using
https://github.com/tensorflow/models/tree/master/official/vision/beta
2. Describe the feature you request
Add workflow for quantization aware training.
Can reference from some available research implementations, but make sure it is based on
https://www.tensorflow.org/model_optimization/guide/quantization/training_comprehensive_guide#define_quantization_aware_model
3. Additional context
May get major speedups (up to 4x) and allow running on NNAPI or hexagon delegate.
https://www.tensorflow.org/lite/performance/delegates#delegates_by_model_type
Tracking from whizzscooters/data-engine#114.
4. Are you willing to contribute it? (Yes or No)

Current difficulty: little support for subclassed models, have to apply quantization annotation recursively manually.

Think this is better done when tensorflow supports quantization aware training better. Moving to not urgent, since thermal issues is not that bad now.


Implement sodium-ion DFN

Description
Implemented DFN for sodium-ion (same as lithium-ion but in a separate file to avoid conflicts). Implemented parameter value from Chayambuka et al (2022) (actual values and data taken from this COMSOL example).
When adding the CITATION I also modified many citation keys to be consistent with the format (FirstAuthorYear). Hence the many files changed.
Fixes #4397 (not an issue but a discussion)
Type of change
Please add a line in the relevant section of CHANGELOG.md to document the change (include PR #) - note reverse order of PR #s. If necessary, also add to the list of breaking changes.

[x] New feature (non-breaking change which adds functionality)
[ ] Optimization (back-end change that speeds up the code)
[ ] Bug fix (non-breaking change which fixes an issue)

Key checklist:

[x] No style issues: $ pre-commit run (or $ nox -s pre-commit) (see CONTRIBUTING.md for how to set this up to run automatically when committing locally, in just two lines of code)
[x] All tests pass: $ python run-tests.py --all (or $ nox -s tests)
[x] The documentation builds: $ python run-tests.py --doctest (or $ nox -s doctests)

You can run integration tests, unit tests, and doctests together at once, using $ python run-tests.py --quick (or $ nox -s quick).
Further checks:

[x] Code is commented, particularly in hard-to-understand areas
[x] Tests added that prove fix is effective or that feature works

Doctests are failing but I don't know why. @agriyakhetarpal can you help with this?

Amazing work! The next step will be to implement the full DFN (with submodels) for sodium-ion, but this is a welcome and necessary first step.

All you need to do is edit CHANGELOG and this should be ready to merge.

Docs all failed, so I will check on that too

Docs failures are unreleated. I am looking into it, but will fix it in a different PR


Add a warning for name collisions

Follow up of #195

We should document the methods to be used to avoid collision in both the join alias and bound parameter name, when #242 is merged.
On that note, I should probably send a companion PR to api-platform/doc. Somehow missed out on updating docs (weird...)

I think this is outdated ? @theofidry


DipAsm

Dear author
On the server, I installed DipAsm through the docker. I found out it needed three software 3D-DNA, hapsCut this process,However, I found that there is a software that has been unable to install, that is, Deepvariant,(I installed it through docker) it error is as follows

What should I do to solve the problem
Looking forward to your reply

Please see #16

Please see #16


Datasource and resource implementation for Next Available Subnet in an Address Block

Issue/Feature description

Datasource and resource implementation for Next Available Subnet in an Address Block
Fix acceptance test issues for DNS record, DNS Auth zone, DHCP Host(https://github.com/infobloxopen/terraform-provider-b1ddi/issues/3)
Added example usage for the implementation

Tests

Added acceptance tests accordingly
Tested in a local cluster

Reviewers

Thank you for making the changes @venkat-iblox. LGTM


ignore cache

This PR adds .cache directory to .gitignore.
It seems to be created by pytest.

Weird, so it is. Thanks! :cake: :sparkles: :cake:


pins: fix adding and removing pinned DMs & Groups

Moves group pins from channels agent to groups
Defends against accidentally resetting pins since we don't update them individually on the backend
Fixes groups pinning hooks that were unintentionally updated to dm hooks as part of the frontend state rewrite

Fixes LAND-1267

Confused why we're removing the ability to pin channels

Should be ready to merge after @Fang- resolves the talk desk issues


discountOnTotalPrice property not returned in GET /orders response

version<EMAIL_ADDRESS>Order has got discountOnTotalPrice applied and its value is returned in the API response. When using the SDK, discountOnTotalPrice is undefined for the Order.

@lejlabe
Please can you share code snippets on what you are doing and how you also made the direct api call.
Thanks

hi @ajimae
I've tried this morning and could not reproduce the issue.
This is the code snippet
  async fetchOrders(orderIds: string[], commerceConfig: ICommerceConfig): Promise<Order[]> {
    const orderIdsQuery: string = _.map(orderIds, id => `"${id}"`).join(', ');
    try {
      return (
        await this.apiRoot
          .withProjectKey({ projectKey: commerceConfig.commerceProjectName })
          .orders()
          .get({
            queryArgs: {
              where: `id in (${orderIdsQuery})`,
              expand: [
                'customerGroup',
                'lineItems[*].price.discounted.discount',
                'paymentInfo.payments[*]',
                'shippingInfo.shippingMethod',
                'state',
              ]
            },
          })
          .execute()
      ).body.results;
    } catch (error) {
      throw new Error(`Fetch orders failed with ${error.statusCode}: ${error.body}`);
    }
  }

I will close the issue for now, and will reopen it in case it happens again. Thank you.

Hi @ajimae! Unfortunately I am able to reproduce the issue again.
With<EMAIL_ADDRESS>this is a code snipped used to fetch orders
  async fetchOrders(orderIds: string[], commerceConfig: ICommerceConfig): Promise<Order[]> {
    const orderIdsQuery: string = _.map(orderIds, id => `"${id}"`).join(', ');
    try {
      return (
        await this.apiRoot
          .withProjectKey({ projectKey: commerceConfig.commerceProjectName })
          .orders()
          .get({
            queryArgs: {
              where: `id in (${orderIdsQuery})`,
              expand: [
                'customerGroup',
                'lineItems[*].price.discounted.discount',
                'paymentInfo.payments[*]',
                'shippingInfo.shippingMethod',
                'state',
              ]
            },
          })
          .execute()
      ).body.results;
    } catch (error) {
      throw new Error(`Fetch orders failed with ${error.statusCode}: ${error.body}`);
    }
  }

In the response, there is no discountOnTotalPrice property for the order.
With direct API call, it is returned as expected:

The SDK doesn't has any influence on the presence of the field. The field itself is optional, so it's not guaranteed to be existing and you have to check for it.
From the documentation it says that it's only present if there was a total price cart discount applied.

Yes it is. But did you ensured that you try to access this particular resource? I can see that you did a query for multiple orders. Do all of them have the field set?