MudPopover does not work with isolated css

Things to check

[X] I have searched the existing issues for this bug
[X] To rule out a caching problem I made sure the bug also happens in an incognito tab

Bug type
Component
Component name
MudPopover
What happened?
When trying to create a razor component with css isolation that uses the MudPopover component it is not possible to style the mud popover component using only the styling on the component. This appears to be due to the fact that the popover is rendered in the popover provider which means that the isolated css no longer has access to this component.
Is this expected behavior or am I missing something.
Also sorry in advance for the reproduction link as it does not allow me to add isolated css
Expected behavior
Popover components can be used on razor components with isolated css and the isolated css will be applied.
Reproduction link
https://try.mudblazor.com/snippet/mEGSvkvyopUQmnYd
Reproduction steps
Please find some example code showing off the behavior I am experiencing.  In this case the style from the class bob is not applied to the popover.
TestComponent.razor

TestComponent

    <MudPopover Open="true" Class="bob">

            <MudText>Content of the popover can be anything.</MudText>

    </MudPopover>

and then the isolated css
TestComponent.razor.css
::deep .bob {
    background-color: red;
}

Which can then be used as follows
<TestComponent />
Relevant log output
No response
Version (bug)
7.12.0
Version (working)
No response
What browsers are you seeing the problem on?
Firefox, Chrome, Edge, Safari
On which operating systems are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Is this expected behavior or am I missing something.

Expected.
The popovers are rendered in MudPopoverProvider in your MainLayout. Your scope it outside of their scope.
Currently, the architecture doesn't allow to fix it. There is in plan to completely rewrite popovers and use Floating UI, but there is no ETA, could be in few years only.


$__interval_ms seems to crash the plugin?

What happened:
select '$__interval' correctly returns eg '2s'. select '$__interval_ms' hangs for a long time and then sometimes gets an error from the server
Environment:

Grafana version: 9.5.1
Plugin version: 2.9.2
OS Grafana is installed on: linux/docker
User OS & Browser: chrome/linux
Others:

Thanks for the report @mzealey, just curious when it hangs, does it not have an error? I was able to reproduce it, but it never hangs for me. It always returns an error.
It's coming from https://github.com/grafana/grafana-aws-sdk-react/blob/07bd2a2b9bde2321ad8b85ff40d18544b2bda3c0/src/sql/utils/utils.ts#L29 because $__interval_ms is a number instead of a string.
Also for your queries, is there a reason you need to use $__interval_ms over $__interval?

I couldn't dig into it too much - there was some weird behaviour and it seemed more than just returning an error but it may have been due to something else.I'm not an expert at Athena SQL/grafana macros but I was planning to use an interval as integer to normalize counts over range for graphing, so the obvious way that came to mind was to use the ms interval, but perhaps there is some other way to do this more easily as with some other datasource plugins?On 10 May 2023 19:53, Kevin Yu @.***> wrote:
Thanks for the report @mzealey, just curious when it hangs, does it not have an error? I was able to reproduce it, but it never hangs for me. It always returns an error.
It's coming from https://github.com/grafana/grafana-aws-sdk-react/blob/07bd2a2b9bde2321ad8b85ff40d18544b2bda3c0/src/sql/utils/utils.ts#L29 because $__interval_ms is a number instead of a string.
Also for your queries, is there a reason you need to use $__interval_ms over $__interval?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

I was planning to use an interval as integer to normalize counts over range for graphing, so the obvious way that came to mind was to use the ms interval

Can you provide an example of the query you're trying? I'm not entirely sure what you mean by "use an interval as integer to normalize counts over range".

so for example:
select
  $__timeGroup(timestamp, $__interval) AS t
  , count(*) as total
...
group by t

will produce a nice graph, but when changing date range because the interval changes with zoom the y-axis numbers change.
So typically (in other sql sources) I would do something like:
select
  $__timeGroup(timestamp, $__interval) AS t
  , count(*) / $__interval_ms * 1000 as qps
...
group by t

so it's in a scale which does not change with date range level

Thanks very much!


support stats compression

Please provide a description of this PR:
Alternative for https://github.com/istio/istio/pull/47934
xref: https://github.com/istio/istio/issues/30987

@zirain: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
002dd6f5b6db556530ea06cf46d94bcee8ab7777
link
true
/test release-notes

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.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

release-notes_istio
002dd6f5b6db556530ea06cf46d94bcee8ab7777
link
true
/test release-notes

gencheck_istio
002dd6f5b6db556530ea06cf46d94bcee8ab7777
link
true
/test gencheck

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.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
002dd6f5b6db556530ea06cf46d94bcee8ab7777
link
true
/test gencheck

release-notes_istio
699c311e9f7b0acdcf6840cd5f64c544f7c2a9bc
link
true
/test release-notes

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.

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
002dd6f5b6db556530ea06cf46d94bcee8ab7777
link
true
/test gencheck

release-notes_istio
699c311e9f7b0acdcf6840cd5f64c544f7c2a9bc
link
true
/test release-notes

unit-tests_istio
699c311e9f7b0acdcf6840cd5f64c544f7c2a9bc
link
true
/test unit-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.

Can you add the commits from https://github.com/istio/istio/pull/47934 and/or make @kaiburjack a co-author on the commits?

Can you add the commits from https://github.com/istio/istio/pull/47934 and/or make @kaiburjack a co-author on the commits?

I don't really need/want to take credit for this in any way, since I wasn't the one coming up with the original solution anyways. If at all, it should be @svenwltr via https://github.com/istio/istio/issues/30987#issuecomment-822517456

Isn't it negotiated by the client (Prometheus)? Envoy can just accept any
of them and client can pick
On Wed, Nov 22, 2023, 4:02 PM zirain @.***> wrote:

@.**** commented on this pull request.
In pkg/bootstrap/config.go
https://github.com/istio/istio/pull/47997#discussion_r1402811910:

@@ -274,6 +274,12 @@ func getStatsOptions(meta *model.BootstrapNodeMetadata) []option.Instance {
}
}

var compression string
// TODO: move annotation to api repo
if statsCompression, ok := meta.Annotations["sidecar.istio.io/statsCompression"]; ok {

as @kyessenov https://github.com/kyessenov suggest, we can exend to
support zst or brotli, so maybe it can be a permanent API.
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/47997#discussion_r1402811910, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXOTIQZCM7ULKLGLVYTYF2HCZAVCNFSM6AAAAAA7WFS2VSVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTONBVGU2DKOJSGI
.
You are receiving this because your review was requested.Message ID:
@.***>

@zirain: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

unit-tests_istio
699c311e9f7b0acdcf6840cd5f64c544f7c2a9bc
link
true
/test unit-tests

lint_istio
7808f4394490f3c796ebe5435b378b2f26a9f630
link
true
/test lint

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.

FYI: I've done some testing with a simple Go HTTP proxy sidecar doing gzip compression for the istio-proxy sidecar's 15090 /stats/prometheus and declaring a new port 14090 to compress the response.
The results are amazing! Without compression (istio-proxy default behaviour) one scrape for one of my pods is 3,979,285 bytes (so ~4 MB) and with gzip compression with everything working and Prometheus happily using the gzip endpoint: 48,554 bytes (so down to just ~48KB!!). That is an improvement of ~81x and will definitely reduce our Inter-Zone Egress Costs by a lot.

It seems for agent metrics or for those using merged metrics via the agent at 15020 we are still at the mercy of the agent server supporting/not supporting compression.
While kaiburjack's solution of defining a new listener (e.g. <IP_ADDRESS>:14020) on an adjacent port and routing to the corresponding cluster (<IP_ADDRESS>:15020) will work as an intermediate solution perhaps we can get sidecar.istio.io/statsCompression to support merged metrics via the agent as well?

Now that stats compression has been implemented (it seems), can someone please explain to me, how I can enable this? In the source code there is a mention about some sidecar.istio.io/statsCompression annotation. Is this supposed to be an annotation on the pod that has the istio proxy sidecar enabled? And would a possible concrete value for this annotation be gzip?

It seems for agent metrics or for those using merged metrics via the agent at 15020 we are still at the mercy of the agent server supporting/not supporting compression.
While kaiburjack's solution of defining a new listener (e.g. <IP_ADDRESS>:14020) on an adjacent port and routing to the corresponding cluster (<IP_ADDRESS>:15020) will work as an intermediate solution perhaps we can get sidecar.istio.io/statsCompression to support merged metrics via the agent as well?

can you open another ticket?

can you open another ticket?

Opened: #49987


Add module to calculate forces and torques applied over the boundary by the particles

Why do so many tests fail now?

@shahabgol I'll let you review and I will review afterwards

@shahabgol I'll let you review and I will review afterwards

ok, I'll review the PR today.

I think we can merge this PR now.

Ok, each comment has been processed. If no CI problems, should be able to merge :)


🐛 Bug Report: lack of permissions to control access of scaffolder endpoints

📜 Description
Many of the scaffolder endpoints are exposed and not protected by the permission framework, which allows users without the scaffolder permissions to access and modify scaffolder resources.
The existing permissions only prevent users from triggering actions and/or view template parameters/steps without the correct permissions. This however, is insufficient for preventing an actual task from being created and written into the database if user does not have access to the actions or template parameter/steps.
Additionally more permissions need to be introduced to provide more granular permissions, such as task read permissions, task cancellation permissions and task creation permissions.
Currently any user with a valid authentication token (does not need to contain any scaffolder related permissions), can access endpoints like the GET /v2/tasks/:taskId, GET /v2/tasks/events, and GET /v2/tasks/eventstream endpoints to view task-related information. They can also access the POST /v2/tasks/:taskId/cancel endpoint to cancel a running task, which is not ideal.
👍 Expected behavior

Users without the proper permissions for the scaffolder should not be able to create tasks (even if they're mostly empty) into the database.
They should not be able to access scaffolder resources like task information, task events, etc. (task read related permissions don't exist yet)
They should definitely not be able to cancel running tasks without permission (delete/cancel related permissions don't exist at the moment)
Trying to access these resources without permission should throw a 403 error instead of granting the resource or creating an empty task in the case.

👎 Actual Behavior with Screenshots
The existing permissions prevent users from triggering actions and viewing template parameters/steps without the correct permissions. However, a user with read access to catalog entities would be able to trigger scaffolder tasks (albeit empty ones) via the POST /v2/tasks endpoint, and write entries into the task table of the backstage_plugin_scaffolder database which is not ideal.
Example, using a user who does not have access to the scaffolder permissions and only the catalog.entity.read permission:
curl -X POST http://localhost:7007/api/scaffolder/v2/tasks -H "Content-Type: application/json" --data '{"templateRef":"template:default/test-template","values": {"random-value": "does not need to be valid at all"}}' -H "Authorization: Bearer eyJhbGciOiJFUzI1NiIsImtpZCI6IjEzNzNjNWU5LTVhYTktNGRkZi1hNzMxLTg1ZDQ1NmIzOGE5YSJ9.eyJpc3MiOiJodHRwOi8vbG9jYWxob3N0OjcwMDcvYXBpL2F1dGgiLCJzdWIiOiJ1c2VyOmRlZmF1bHQvemFwZXJleCIsImVudCI6WyJ1c2VyOmRlZmF1bHQvemFwZXJleCJdLCJhdWQiOiJiYWNrc3RhZ2UiLCJpYXQiOjE3MTMzNjE4NzcsImV4cCI6MTcxMzM2NTQ3N30.362Lrv91uLT9IzSMS6S66sWoUmbJMnEhUV2cUz_YA4Icx30kXL6xw3AQk06aSuKKWYzPLPFKKg25VZJvQRMeFw" | jq

Response:
{
  "id": "f0d6828f-f865-4671-826a-ad58c39293ad"
}

Then using this same user, we can access the information regarding the task (and all other tasks if we want to):
curl http://localhost:7007/api/scaffolder/v2/tasks/f0d6828f-f865-4671-826a-ad58c39293ad -H "Authorization: Bearer eyJhbGciOiJFUzI1NiIsImtpZCI6IjEzNzNjNWU5LTVhYTktNGRkZi1hNzMxLTg1ZDQ1NmIzOGE5YSJ9.eyJpc3MiOiJodHRwOi8vbG9jYWxob3N0OjcwMDcvYXBpL2F1dGgiLCJzdWIiOiJ1c2VyOmRlZmF1bHQvemFwZXJleCIsImVudCI6WyJ1c2VyOmRlZmF1bHQvemFwZXJleCJdLCJhdWQiOiJiYWNrc3RhZ2UiLCJpYXQiOjE3MTMzNjE4NzcsImV4cCI6MTcxMzM2NTQ3N30.362Lrv91uLT9IzSMS6S66sWoUmbJMnEhUV2cUz_YA4Icx30kXL6xw3AQk06aSuKKWYzPLPFKKg25VZJvQRMeFw" | jq

Response:
{
  "id": "f0d6828f-f865-4671-826a-ad58c39293ad",
  "spec": {
    "apiVersion": "scaffolder.backstage.io/v1beta3",
    "steps": [],
    "output": {},
    "parameters": {
      "random-value": "does not need to be valid at all"
    },
    "user": {
      "ref": "user:default/zaperex"
    },
    "templateInfo": {
      "entityRef": "template:default/test-template",
      "baseUrl": "file:///home/frkong/coding/janus-idp/backstage-showcase/catalog-info.local.yaml",
      "entity": {
        "metadata": {
          "namespace": "default",
          "annotations": {
            "backstage.io/managed-by-location": "file:/home/frkong/coding/janus-idp/backstage-showcase/catalog-info.local.yaml",
            "backstage.io/managed-by-origin-location": "file:/home/frkong/coding/janus-idp/backstage-showcase/catalog-info.local.yaml"
          },
          "name": "test-template",
          "title": "Test Template",
          "description": "A Test Template for Backstage",
          "tags": [
            "test"
          ],
          "uid": "de1e85e0-7c46-41e6-b699-a4bfc3b2957b",
          "etag": "2408dcc88eb72249307f932bcf04f7db241056bc"
        }
      }
    }
  },
  "status": "completed",
  "lastHeartbeatAt": "2024-04-17T14:06:04.000Z",
  "createdAt": "2024-04-17T14:06:04.000Z",
  "createdBy": "user:default/zaperex"
}

If we have a user with scaffolder permissions create an actual task (ideally a long running one), the user without permissions can cancel the task.
curl -X POST http://localhost:7007/api/scaffolder/v2/tasks/f5a19ac4-8f90-4ad7-9de3-32b7a8b6d1e0/cancel -H "Authorization: Bearer eyJhbGciOiJFUzI1NiIsImtpZCI6IjU1YmZmODE5LWMyMzgtNDg3OC05N2IxLTAwM2Y2MWZkZmZjNSJ9.eyJpc3MiOiJodHRwOi8vbG9jYWxob3N0OjcwMDcvYXBpL2F1dGgiLCJzdWIiOiJ1c2VyOmRlZmF1bHQvZGF0c2tydWIiLCJlbnQiOlsidXNlcjpkZWZhdWx0L2RhdHNrcnViIl0sImF1ZCI6ImJhY2tzdGFnZSIsImlhdCI6MTcxMzM2NDkyNiwiZXhwIjoxNzEzMzY4NTI2fQ.hk8m4i0NX7BQvobN0ZAjWk2ZATd8Fl7u5rSe-7z0AVU-Bh7RvUPMmh8lbj6HLYGqVnmJVJS5697WMvLRmykvXg" | jq
{
  "status": "cancelled"
}

👟 Reproduction steps

Setup the permissions framework in the backend and enable it
Grant one user all the scaffolder permissions and catalog.entity.read permissions to be able to properly run software templates (call this user A)
Grant one user only catalog.entity.read permissions (call this user B)
Grab the authentication token for both users (Go to the frontend, press F12 and obtain token in the network tab from requests to any backend endpoint)
Create a task with both user A and B and observe that both are added into the database
Observe that both users can query the /v2/tasks and v2/tasks/events endpoints to get task information
Create a task with user A (ideally make it a relatively long running task, ex: add a debug:wait 20s action in it). Grab the taskId of this task and use the auth token of user B to request the cancellation of that task with /v2/tasks/:taskId/cancel and observe that task was cancelled.

📃 Provide the context for the Bug.
This is not ideal from a security standpoint as it allows almost all users to be able to access the scaffolder resources even with existing permissions in place.
🖥 Your Environment
OS:   Linux 6.8.5-201.fc39.x86_64 - linux/x64
node: v18.18.2
yarn: 1.22.19
cli:  0.26.2 (installed)
backstage:  1.25.2

Dependencies:
  @backstage/app-defaults                                          1.5.3
  @backstage/backend-app-api                                       0.6.2, 0.5.14
  @backstage/backend-common                                        0.21.6, 0.20.2
  @backstage/backend-defaults                                      0.2.16
  @backstage/backend-dev-utils                                     0.1.4
  @backstage/backend-dynamic-feature-service                       0.2.0, 0.2.3, 0.2.8
  @backstage/backend-openapi-utils                                 0.1.9
  @backstage/backend-plugin-api                                    0.6.10, 0.6.16
  @backstage/backend-tasks                                         0.5.21
  @backstage/backend-test-utils                                    0.3.6
  @backstage/catalog-client                                        1.6.3
  @backstage/catalog-model                                         1.4.5
  @backstage/cli-common                                            0.1.13
  @backstage/cli-node                                              0.2.4
  @backstage/cli                                                   0.26.2
  @backstage/config-loader                                         1.7.0
  @backstage/config                                                1.2.0
  @backstage/core-app-api                                          1.12.3
  @backstage/core-compat-api                                       0.2.3
  @backstage/core-components                                       0.14.3, 0.13.10
  @backstage/core-plugin-api                                       1.9.1
  @backstage/dev-utils                                             1.0.30
  @backstage/errors                                                1.2.4
  @backstage/eslint-plugin                                         0.1.6
  @backstage/frontend-plugin-api                                   0.6.3
  @backstage/integration-aws-node                                  0.1.12
  @backstage/integration-react                                     1.1.25
  @backstage/integration                                           1.9.1
  @backstage/plugin-api-docs                                       0.11.3
  @backstage/plugin-app-backend                                    0.3.64
  @backstage/plugin-app-node                                       0.1.16
  @backstage/plugin-auth-backend-module-atlassian-provider         0.1.8
  @backstage/plugin-auth-backend-module-aws-alb-provider           0.1.8
  @backstage/plugin-auth-backend-module-gcp-iap-provider           0.2.11
  @backstage/plugin-auth-backend-module-github-provider            0.1.13
  @backstage/plugin-auth-backend-module-gitlab-provider            0.1.13
  @backstage/plugin-auth-backend-module-google-provider            0.1.13
  @backstage/plugin-auth-backend-module-microsoft-provider         0.1.11
  @backstage/plugin-auth-backend-module-oauth2-provider            0.1.13
  @backstage/plugin-auth-backend-module-oauth2-proxy-provider      0.1.9
  @backstage/plugin-auth-backend-module-oidc-provider              0.1.7
  @backstage/plugin-auth-backend-module-okta-provider              0.0.9
  @backstage/plugin-auth-backend                                   0.22.3
  @backstage/plugin-auth-node                                      0.4.11
  @backstage/plugin-auth-react                                     0.0.3
  @backstage/plugin-azure-devops-backend                           0.6.3
  @backstage/plugin-azure-devops-common                            0.4.0
  @backstage/plugin-azure-devops                                   0.4.2
  @backstage/plugin-bitbucket-cloud-common                         0.2.17
  @backstage/plugin-catalog-backend-module-bitbucket-cloud         0.2.3
  @backstage/plugin-catalog-backend-module-bitbucket-server        0.1.30
  @backstage/plugin-catalog-backend-module-github-org              0.1.11
  @backstage/plugin-catalog-backend-module-github                  0.5.7
  @backstage/plugin-catalog-backend-module-gitlab                  0.3.14
  @backstage/plugin-catalog-backend-module-openapi                 0.1.34
  @backstage/plugin-catalog-backend-module-scaffolder-entity-model 0.1.14
  @backstage/plugin-catalog-backend                                1.21.0
  @backstage/plugin-catalog-common                                 1.0.22
  @backstage/plugin-catalog-graph                                  0.4.3
  @backstage/plugin-catalog-import                                 0.10.9
  @backstage/plugin-catalog-node                                   1.11.0
  @backstage/plugin-catalog-react                                  1.11.2
  @backstage/plugin-catalog                                        1.18.2
  @backstage/plugin-dynatrace                                      10.0.2
  @backstage/plugin-events-backend                                 0.3.3, 0.2.22
  @backstage/plugin-events-node                                    0.3.2, 0.2.22
  @backstage/plugin-github-actions                                 0.6.14
  @backstage/plugin-github-issues                                  0.4.0
  @backstage/plugin-home-react                                     0.1.11
  @backstage/plugin-home                                           0.7.2
  @backstage/plugin-jenkins-backend                                0.4.3
  @backstage/plugin-jenkins-common                                 0.1.25
  @backstage/plugin-jenkins                                        0.9.8
  @backstage/plugin-kubernetes-backend                             0.16.4
  @backstage/plugin-kubernetes-common                              0.7.5
  @backstage/plugin-kubernetes-node                                0.1.10
  @backstage/plugin-kubernetes-react                               0.3.3
  @backstage/plugin-kubernetes                                     0.11.8
  @backstage/plugin-lighthouse-common                              0.1.5
  @backstage/plugin-lighthouse                                     0.4.18
  @backstage/plugin-org                                            0.6.23
  @backstage/plugin-permission-backend                             0.5.40
  @backstage/plugin-permission-common                              0.7.13
  @backstage/plugin-permission-node                                0.7.27
  @backstage/plugin-permission-react                               0.4.21
  @backstage/plugin-proxy-backend                                  0.4.14
  @backstage/plugin-scaffolder-backend-module-azure                0.1.8
  @backstage/plugin-scaffolder-backend-module-bitbucket-cloud      0.1.6
  @backstage/plugin-scaffolder-backend-module-bitbucket-server     0.1.6
  @backstage/plugin-scaffolder-backend-module-bitbucket            0.2.6
  @backstage/plugin-scaffolder-backend-module-gerrit               0.1.8
  @backstage/plugin-scaffolder-backend-module-gitea                0.1.6
  @backstage/plugin-scaffolder-backend-module-github               0.2.6
  @backstage/plugin-scaffolder-backend-module-gitlab               0.3.2
  @backstage/plugin-scaffolder-backend                             1.22.3
  @backstage/plugin-scaffolder-common                              1.5.1
  @backstage/plugin-scaffolder-node                                0.3.3, 0.4.2
  @backstage/plugin-scaffolder-react                               1.8.3
  @backstage/plugin-scaffolder                                     1.19.2
  @backstage/plugin-search-backend-module-catalog                  0.1.21
  @backstage/plugin-search-backend-module-pg                       0.5.25
  @backstage/plugin-search-backend-module-techdocs                 0.1.21
  @backstage/plugin-search-backend-node                            1.2.20
  @backstage/plugin-search-backend                                 1.5.6
  @backstage/plugin-search-common                                  1.2.11
  @backstage/plugin-search-react                                   1.7.9
  @backstage/plugin-search                                         1.4.9
  @backstage/plugin-sonarqube-backend                              0.2.18
  @backstage/plugin-sonarqube-react                                0.1.14
  @backstage/plugin-sonarqube                                      0.7.15
  @backstage/plugin-tech-radar                                     0.7.2
  @backstage/plugin-techdocs-backend                               1.10.3
  @backstage/plugin-techdocs-module-addons-contrib                 1.1.8
  @backstage/plugin-techdocs-node                                  1.12.2
  @backstage/plugin-techdocs-react                                 1.2.2
  @backstage/plugin-techdocs                                       1.10.3
  @backstage/plugin-user-settings                                  0.8.4
  @backstage/release-manifests                                     0.0.11
  @backstage/test-utils                                            1.5.3
  @backstage/theme                                                 0.5.2, 0.4.4
  @backstage/types                                                 1.1.1
  @backstage/version-bridge                                        1.0.7

To simply the permission assigning process, I also used the RBAC plugin to assign the permissions to the users:
"@janus-idp/backstage-plugin-rbac-backend": "2.6.2",
"@janus-idp/backstage-plugin-rbac-node": "1.0.6",

👀 Have you spent some time to check if this bug has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
Yes I am willing to submit a PR!

Hmm, yeah I think that the first pass of permissions implementations didn't think of a broader scope for this #11403. I think that this makes sense, @vinzscam do you have any ideas on how best to implement this?

Thanks @vinzscam @benjdlambert , I'd like to take a work at this, can I be assigned this issue?

Thanks @vinzscam @benjdlambert , I'd like to take a look and work on this, can I be assigned this issue?

done!


Detect when the loader gets finish the required time.

Hi,
I'm using the RMDownloadIndicator and want one solution that how can we detect the loader fills the whole circle means the time when it completed the download bytes.

RMDownloadIndicator is a view, so it must not controll progress, it gets progress value and just update itself. So detection of the ending of your task is a responsibility of an entity which sends progress values to your RMDownloadIndicator instance.

Thanks Morozkin:)


How to interrupt program without Ctrl-C?

Your Windows build number

Microsoft Windows [Version 10.0.17134.765]

What you're doing and what's happening:

I often use Ctrl-C to interrupt and restart a program however with the copy/paste functions now using the control key I'm not seeing a way to do this?

If no text is selected, typing Ctrl+C in a cooked read should generate a CTRL_C_EVENT in all processes that are attached to the console. If text is selected, Ctrl+C copies to the clipboard and clears the selection. Press Ctrl+C a second time to generate the event.
It's up to a program how it should handle this control event. It can ignore it or exit gracefully. By default a program started in a new console process group ignores Ctrl+C (e.g. CMD's start /B creates a new process group).  Try Ctrl+Break if Ctrl+C seems to be ignored. Ctrl+Break is rarely ignored, except by shells such as CMD. Usually Ctrl+Break is left to the default handler (i.e. exit immediately with STATUS_CONTROL_C_EXIT) or at least is handled by exiting gracefully.

As @eryksun says, this is up to the shell you're running, not to the Windows Terminal program.

See also #813

@eryksun I am using CMD and I can't seem to be able to do Ctrl-Break due to a lack of a Break key on my keyboard. Do you know of any alternatives?
I appreciate the help.

You may have an fn key that shifts another key to pause/break. Otherwise use the on-screen keyboard.

using powershell 7.2.5 to run a ping -t command is not interruptible. Ctrl+c does not work. Ctrl+Break from the on-screen keyboard does not work.
Any other ideas? Kinda annoying.


triggering deploy: unable to create-deployment for app: exit status 1

Hi, for seemingly no reason it started to crash without any help to troubleshoot.
My job config is this:
  deploy:
    timeout-minutes: 10
    runs-on: ubuntu-latest
    steps:
    - name: DigitalOcean App Platform deployment
      uses<EMAIL_ADDRESS>      with:
        app_name: zbx-receivers-app
        token: ${{ secrets.DIGITALOCEAN_ACCESS_TOKEN }}

Here's the GH Action job log:
job-logs.txt

Same error for me..
Run<EMAIL_ADDRESS>/usr/bin/docker run --name c3483a7367adeddf44713a427329[5](https://github.com/ciderpop/musetheory/actions/runs/8808177659/job/24176745912#step:3:6)18dc497e_819e84 --label 2c3483 --workdir /github/workspace --rm -e "INPUT_APP_NAME" -e "INPUT_TOKEN" -e "INPUT_IMAGES" -e "HOME" -e "GITHUB_JOB" -e "GITHUB_REF" -e "GITHUB_SHA" -e "GITHUB_REPOSITORY" -e "GITHUB_REPOSITORY_OWNER" -e "GITHUB_REPOSITORY_OWNER_ID" -e "GITHUB_RUN_ID" -e "GITHUB_RUN_NUMBER" -e "GITHUB_RETENTION_DAYS" -e "GITHUB_RUN_ATTEMPT" -e "GITHUB_REPOSITORY_ID" -e "GITHUB_ACTOR_ID" -e "GITHUB_ACTOR" -e "GITHUB_TRIGGERING_ACTOR" -e "GITHUB_WORKFLOW" -e "GITHUB_HEAD_REF" -e "GITHUB_BASE_REF" -e "GITHUB_EVENT_NAME" -e "GITHUB_SERVER_URL" -e "GITHUB_API_URL" -e "GITHUB_GRAPHQL_URL" -e "GITHUB_REF_NAME" -e "GITHUB_REF_PROTECTED" -e "GITHUB_REF_TYPE" -e "GITHUB_WORKFLOW_REF" -e "GITHUB_WORKFLOW_SHA" -e "GITHUB_WORKSPACE" -e "GITHUB_ACTION" -e "GITHUB_EVENT_PATH" -e "GITHUB_ACTION_REPOSITORY" -e "GITHUB_ACTION_REF" -e "GITHUB_PATH" -e "GITHUB_ENV" -e "GITHUB_STEP_SUMMARY" -e "GITHUB_STATE" -e "GITHUB_OUTPUT" -e "RUNNER_OS" -e "RUNNER_ARCH" -e "RUNNER_NAME" -e "RUNNER_ENVIRONMENT" -e "RUNNER_TOOL_CACHE" -e "RUNNER_TEMP" -e "RUNNER_WORKSPACE" -e "ACTIONS_RUNTIME_URL" -e "ACTIONS_RUNTIME_TOKEN" -e "ACTIONS_CACHE_URL" -e "ACTIONS_RESULTS_URL" -e GITHUB_ACTIONS=true -e CI=true -v "/var/run/docker.sock":"/var/run/docker.sock" -v "/home/runner/work/_temp/_github_home":"/github/home" -v "/home/runner/work/_temp/_github_workflow":"/github/workflow" -v "/home/runner/work/_temp/_runner_file_commands":"/github/file_commands" -v "/home/runner/work/musetheory/musetheory":"/github/workspace" 2c3483:a7367adeddf44713a427329518dc497e  "" "mt_server" "***" 2024/04/23 22:44:11 triggering deploy: unable to create-deployment for app: exit status 1

@z4y4ts was your action working normally before yesterday with that same config?
I just setup actions pipeline for my app platform project yesterday for the first time and ran into this on first attempt.
Wasn't sure if it was something on my end or a problem on Digital Ocean's side. Talked to DO support and they had me verify some access token permissions however the issue persists.

As I can see from the workflow logs, the problem appeared 3 days ago - Apr
21st.
Can't recall touching the access token anywhere around that time.
Also in the job logs, I see GitHub does pass it. Can't check the value
though, since it's masked.
ср, 24 квіт. 2024 р. о 15:53 ciderpop @.***> пише:

@z4y4ts https://github.com/z4y4ts was your action working normally
before yesterday with that same config?
I just setup actions pipeline for my app platform project yesterday for
the first time and ran into this on first attempt.
Wasn't sure if it was something on my end or a problem on Digital Ocean's
side. Talked to DO support and they had me verify some access token
permissions however the issue persists.
—
Reply to this email directly, view it on GitHub
https://github.com/digitalocean/app_action/issues/112#issuecomment-2074879447,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADOTN33LJWE7D3VJ6MYOLTY66TMTAVCNFSM6AAAAABGUQVHE6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZUHA3TSNBUG4
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
If anybody asks you what Industrial music sounds like, tell them its
like a bunch of angry Belgians having a fight with a washing machine.

ah okay I see. yeah my token seems to be setup properly as well and I even tried a full access token instead of custom scopes and same issue, so doesn't seem permissions related.
still waiting to see if their support will advise on this, will report back here if I get any more info from them.

@ciderpop exactly the same thing with the token. Now also waiting for more info from the support.

Same error here using v1.1.6 :/

yeah, for me v1.1.5 & v1.1.4 are also failing

Potentially related issues? #56 #70
I'm just going to double check my app names to make sure because it sounds like that could be related, but glad it's not just me... making me feel like it might be just an outage or something

Mine was user error actually... my credentials on DO for pulling my GitHub Container Registry Docker image had expired :sweat_smile: I was able to tell because I couldn't manually deploy from inside DigitalOcean either. Sometimes gotta re-check the basics lol

@arimgibson ah yes realized I made a similar mistake! had accidentally had a _ instead of a - for one of the app names so it broke the actions. everything is working properly for me now.
oof very easy rookie mistake I think I need to get my eyes checked now 🤓

We've just released v2.0.0 of the app which surfaces errors much more clearly and is going to help debugging issues like this in the future. Please check it out: https://github.com/digitalocean/app_action/discussions/122.


Replace expandable with accordion

https://pub.dev/packages/accordion seems to be actively developed
https://pub.dev/packages/expandable hasn't been updated for a while.
Plus I wonder if it'd solve #220

Not sure if I'd spend time on this.


Carmelo Data Update

Are there any plans to update the Carmelo projections for players?

I've been looking for the same... essentially the player by player data set

These are published each year with updated projections.


"No such file or directory" error

Here's my code:
#![allow(unused_variables)]
use cifar_ten::*;

fn main() {
    let (train_data, train_labels, test_data, test_labels) = Cifar10::default()
        .show_images(true)
        .download_and_extract(true)
        .build_u8()
        .expect("Failed to build CIFAR-10 data");
}

With cargo run I get this:
% cargo run
    Finished dev [unoptimized + debuginfo] target(s) in 0.02s
     Running `target/debug/uni`
thread 'main' panicked at 'Failed to build CIFAR-10 data: Os { code: 2, kind: NotFound, message: "No such file or directory" }', src/main.rs:9:10
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Here's how I have the dependency:
[dependencies]
cifar-ten = {git = "https://github.com/quietlychris/cifar-ten", branch = "master"}

@seisvelas Sorry for the delay; just saw this issue now. As it looks like you've seen from the changed issue title, the fix for this should be to enable the download option, or alternatively, already have the extracted data located in the directory specified by the base_path configuration. This library requires one or the other; you must either supply the data or enable the option to download it.
You'll also probably run into an print statement saying that you don't have the display feature enabled. This shouldn't interfere with anything after the actual crate, unless you're somehow piping stdout into another progoram (it's just a print error, not a panic).
Is there a change to the crate that you were interested in have added, or more wanted to find a fix to an issue you were having?


Update?

If i'm correct, current version of dsmr is 5.1
However latest is already 5.9
Any chance to get an update?
I'm asking this cause i would love to have the peak sensors... So i can see capacitytariff in Belgium :)

Hi @skank01, that's not correct as the addon is on the latest DSMR Reader version for a while. Can you force the update check? Another option is to reboot your HA instance.

Hi thats strange, my current version (addon) says its 1.2.0
My dsmr says under support: dsmr reader v 5.1
If i click in check for update under dsmr reader it says theres an update
If i click check for udpates in HA: it says theres no addon update

Any idea? @sanderdw

I really don't know, as you can see here the addon version should be 1.9.0.
Can you post a screenshot?

[url=https://ibb.co/Fbpt2K8][img]https://i.ibb.co/PZk8KN1/shot1.jpg[/img][/url]
[url=https://ibb.co/ZTq7f6H][img]https://i.ibb.co/tCVFHMZ/shot2.jpg[/img][/url]

Could it be that you using the custom_component variant? So a manual copy?

Could it be that you using the custom_component variant? So a manual copy?

No, i installed it using the addon store.
Before my version now, i had no problems with updating (it autopop automatically)
Now what?
Is there a way to install it without losing settings and history?

Yes just remember the settings and remove the addon. After that reboot your system and install the addon again. No worries about the data, this is persisted in the database you configured.

ok i updated to latest version without loss of data


Server side

Does this mod work on server? if not could you do it in the future? it would be great if someone doesnt have access to their router to turn on port forwarding/their router didnt had the option

currently not supported, no plans for future support


Textareas have scroll bars

I think this happened along with the upgrade to the new clockface, but I'm not certain.


Bullet engine: Rigidbody Flickering when inherits parent position

**Windows 10 64bits, Godot 3.0 Beta 1
Issue description:
When a Rigidbody inherits its parent's position, the Rigidbody position start flickering until its sleeps.
Steps to reproduce:
Create a Area, Position3D, spatial(Any node with 3D coordinates). Add a rigidbody with gravity scale set to 0. Try to move the parent and the rigidbody positions start to flick. With GodotPhysics it works as expected.
Link to minimal example project:
Godot3Bug Bullet Engine.zip

CC @AndreaCatania

Confirmed in the current master branch.

This works with Godot physics engine but it is not intended to work. The rigid body should be always moved by setting Forces and Torques.


Do not show 'open article action button' for vouchers

1. Why is this change necessary?
The button 'open article' in the order positions list (in the backend) leads to random articles when the position is a redeemed voucher.
This is because the field 'articleId' of the underlying record is actually the id of the redeemed voucher (code) when the position is a voucher. Then the click on the button opens the article that accidentally shares the same id with the redeemed voucher (code).
2. What does this change do, exactly?
Hide the 'open article' button for redeemed vouchers.
3. Describe each step to reproduce the issue or behaviour.
Create an order that has a redeemed voucher code. Open the order in the backend. Click 'open article' button in the order positions list.
6. Checklist

[ ] I have written tests and verified that they fail without my change
[ ] I have squashed any insignificant commits
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[ ] I have read the contribution requirements and fulfil them.

Hi,
thx, for your PR.
But it seems that you missed out this icon in the order list

Thank you, i will add this :)

Done.

Hey @windaishi,
please change the validation to mode != 0.

I applied the requested changes.

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-20485
Please use this issue to track the state of your pull request.

Hi,
thx. Your commit has been mergend with this commit: https://github.com/shopware/shopware/commit/b9ee675e1ca470b72dbfc4c274f3b900ecf0b089
Greetings,
Marcel


Emileber/address/reset cache

Description
To avoid side-effects that the cache can have in different situation, like testing, I changed the cache so that it can be both read publicly and reset as well.
Type of change

[x] @shopify/address Minor: New feature (non-breaking change which adds functionality)

Checklist

[ ] I have added a changelog entry, prefixed by the type of change noted above (Documentation fix and Test update does not need a changelog as we do not publish new version)

Though it is not required, I'd like to merge https://github.com/Shopify/quilt/pull/2178 first.

While using the new cache reset in this package tests, I discovered that some tests now fail, so I'll investigate to make sure they're not passing because of any side-effects of the cache.

While using the new cache reset in this package tests, I discovered that some tests now fail, so I'll investigate to make sure they're not passing because of any side-effects of the cache.


use absolute touchex for microsoft whiteboard

can I use absolute touchex for microsoft whiteboard? if yes, please help.
My laptop is Asus X513EA which has Asus precision touchpard as the touchpad driver.

try injecting into the executable of ms whiteboard?


GitHub Sponsors

Add a sponsor URL or sign up for GitHub Sponsors.
https://github.com/sponsors
https://help.github.com/en/github/administering-a-repository/displaying-a-sponsor-button-in-your-repository
I would like to support your project.

No thanks, just enjoy the script, that is payment enough.


Training module for Intents

AS A Chatbot Trainer
I WANT TO create new and modify existing intents and relating functionalities
SO THAT the chatbot can be sufficiently trained

Views: Intents view
Files: rasa-ilmateenused/data/nlu/ each file ending with "nlu.yml" is a separate intent file

Acceptance Criteria

[ ] #44
[ ] #11
[ ] Limit example length
[ ] Intent release ban if the minimum amount of examples is not exceeded
[ ] Calculate the average example amount within released intents
[ ] Ban adding duplicate intents and examples
[ ] #42
- [ ] Display intent statistics based on the latest trained model (https://github.com/buerokratt/Training-Module/issues/3) -- @vmugra - obsolete!
[ ] #50
[ ] #46 - obsolete!
[ ] #47 - Lookup tables is obsolete!
[ ] Display and add new entities (list of entities in the model) - from domain.yml file
[ ] #49
[ ] Common intent view for common topics by using endpoints from https://github.com/buerokratt/Training-Module/issues/50
[ ] Add new or delete common examples (deleted examples will be commented out) based on #44
[ ] Two separate views for common intents and local intents, where common intents could not be added but can be modified (examples can be added/modified/deleted as well as switched off from the model) and local intents can be added and modified.
[ ] #25
[ ]  #24

For local intents if user is adding a new intent, they must choose if the local data is private/sensitive or public data (no sensitive information)

GUI

[ ] #53
[x] #54
[ ] #55
[x] #56

DSL

[ ] #74
[ ] #76
[ ] Get intents
[ ] Disable intent from dataset
[ ] Get examples for intents
[ ] Add, modify and delete examples for each intent
[ ] Count examples in each intent
[ ] Last changed tables (intents or modifying examples)
[ ] Search and sort intents
[ ] #75
[ ] #25

All back-end requests can only be consumed via Ruuter

Ruuter

[ ] Limit example length as a hardcoded value in DSL
[ ] Intent release ban if the minimum amount of examples is not exceeded is set as a hardcoded value in DSL

Resql

[ ] Endpoint samples/average/intents
[ ] #43
[ ] Use intents endpoint with a predefined title parameter to validate for duplicate intents

Nodejs

[ ] Apply changes on Rasa Synonyms YAML files based on provided input as POST body

@turnerrainer
GUI endpoints:

Add endpoint for requesting all intents
Add endpoint for requesting intent examples
Add endpoint for creating a new intent
Add endpoint for deleting an existing intent
Add endpoint for updating an intent
Add endpoint for updating an example
Add endpoint for deleting an example
Add endpoint for adding an intent to model
Add endpoint for removing an intent from a model

PS! All these endpoints should apply to both common and local intents separately

Prototype:

@vmugra GUI AC-s are in-review. This task can be moved forward aswell.

Done in subtasks

PR: https://github.com/buerokratt/Training-Module/pull/270
This pull request finalizes the add, update, and delete intents functionality in the training module.
On any one of those processes, the request sent from the front-end remains relatively simple while Ruuter will have to work through a relatively long DSL that makes several different calls between the file system, OpenSearch, and Database, all of which need to remain in the same state to avoid errors.
To accomplish this, most processes are also checked and validated before happening, which adds to the complexity of the request. This also means that, depending on the server load, requests will usually take between 1 to 5 seconds and front-end is not updated instantly until they are completed, which is why some changes were made in the GUI, particularly the optimization of the page as well as changing the order of things happening on the screen between the start, load, and completion of requests. Additionally, added a small new component, LoadingDialogue, a simpler derivative of the regular Dialogue (no new imports used), which can be used anywhere in the application.
In addition to that, fixed some console inconsistencies and bugs regarding file deletion without changing the processes already working elsewhere in the module.

Simplified and optimized front-end
Added a separate delete-intent script
Added a LoadingDialogue component
Added OpenSearch checks & calls in DSL files
Added Database checks & calls in DSL files
Fixed a bug with empty Intent (without examples)
Fixed several deletion and file check bugs


Merge 2016.09

Merges latest Snabb release (v2016.09) onto lwaftr branch.

It makes sense, I guess. :-)


GH-43572: [JS] Move most dependencies to the devDependencies

Rationale for this change
Adding apache-arrow as a dependency causes its dependencies to interact with the projects own dependencies. It's also bloating up JS build sizes.
Moving these dependencies to devDependencies will resolve this issue.
What changes are included in this PR?
Moves most dependencies to devDependencies in the package.json.
Are these changes tested?
n/a
Are there any user-facing changes?
This does not change user-facing APIs.

GitHub Issue: #43572

This probably breaks the cli, no? I also think it's common to have typescript typings as dependencies.
Iirc there was a similar pr recently that didn't work.

@domoritz Thanks for the review! I updated the PR to restore some of the imports used in the bin commands.
It looks like the only js/bin being exported is the arrow2csv. But I restored the deps being imported directly in the js/bin to be safe.

I think we need tslib and the helpers, though, no?

Here is the pull request I was tinking of with discussion of why we need the dependencies: https://github.com/apache/arrow/pull/43215. We could make the package lighter by removing the CLI tool from the public API. Would you like to make a pull request for that?


Refactor firebase lookup endpoint

WHAT DOES THIS PR DO?
Refactors lookup endpoint to;

return exists as a boolean value.
return "user does not exist" message when user does not exist and Phone number or email address is invalid.

Jira cards

MOB-146

GitHub issues

Closes #

HOW DO I TEST OUT THIS PR?

Checkout this branch and navigate to src/auth-service
Run npm install and npm run dev-mac.
Test out using this firebase lookup endpoint.
WHICH ENDPOINTS SHOULD BE READY FOR TESTING?:
Firebase Lookup
IS THERE ANY JENKINS CONSOLE LINK FOR CODE COVERAGE AND BUILD INFO?

SCREENSHOTS

ARE THERE ANY RELATED PRs?

Thanks @BenjaminSsempala , I have just left some small comment.
Also, please shed more light about the changes? Could you use the PR description or review comments to share more context here?
Thanks!

Thank you @Baalmart , I've updated te description and also added some review comments.


Tagging for API Request

Your architecture was so good. But i have question. I need to upload 10 photos  and this 10 photos must be in arrange. So, i added a numbering in request. My question is, How to retrieve that numbering so i can get the exact arrangement in my photos?

NO Api needed. I need only for displaying.

Hi, @chonex143

this 10 photos must be in arrange.

Wha do you mean by in arrange.?
I also a little bit confused in general, are you asking how you can implement this with RxSwift?


[SPARK-19915][SQL] Exclude cartesian product candidates to reduce the search space

What changes were proposed in this pull request?
We have some concerns about removing size in the cost model in the previous pr. It's a tradeoff between code structure and algorithm completeness. I tend to keep the size and thus create this new pr without changing cost model.
What this pr does:

Exclude cartesian products in the "memo". This significantly reduces the search space and memory overhead of memo. Otherwise every combination of items will exist in the memo. We can find those unjoinable items after reordering is finished and put them at the end.
This pr also includes a bug fix: if a leaf item is a project(_, child), current solution will miss the project.

How was this patch tested?
Add a test case.

Test build #74500 has started for PR 17286 at commit f472c74.

please review @hvanhovell @cloud-fan @nsyca

Test build #74500 has finished for PR 17286 at commit f472c74.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
  case class JoinPlan(
case class Cost(rows: BigInt, size: BigInt)

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74500/
Test PASSed.

Merged build finished. Test PASSed.

Test build #74527 has started for PR 17286 at commit 9aee29a.

Test build #74528 has started for PR 17286 at commit dfc34fc.

Test build #74527 has finished for PR 17286 at commit 9aee29a.

This patch fails SparkR unit tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74527/
Test FAILed.

Test build #74528 has finished for PR 17286 at commit dfc34fc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74528/
Test PASSed.

Co-existing cross joins (join without a condition) and inner joins make the reordering procedure cumbersome. On one hand, putting cross join candidates into memo is not good in terms of search performance and memory consumption. On the other hand, adding cross joins after inner join exploration also has problems with multiple unjoinable groups, just putting them at the end of the plan is not right.
So I decide to do reordering only for consecutive inner joins, which are separated by other plans (including cross joins). For bushy trees, this means we will reorder joins within each inner joinable "groups". For left deep trees, the ReorderJoin rule puts cross joins to the end, so all the previous inner joins will be reordered. Since join reorder is a complicated problem, we may need several levels of optimization, we can consider ReorderJoin as a heuristic algorithm.
For common cases when there's no cross join, this algorithm behaves just like before.

Test build #74656 has started for PR 17286 at commit 237ad9d.

Test build #74656 has finished for PR 17286 at commit 237ad9d.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74656/
Test PASSed.

LGTM except some minor comments

Test build #74737 has started for PR 17286 at commit aa75b6e.

Test build #74737 has finished for PR 17286 at commit aa75b6e.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74737/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

Test build #74741 has started for PR 17286 at commit aa75b6e.

Test build #74741 has finished for PR 17286 at commit aa75b6e.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74741/
Test PASSed.

Merged build finished. Test PASSed.

@wzhfy Some thoughts on how to solve the Cartesian problem as part of the join enumeration algorithm is to apply a similar strategy to the one that we discuss for star-plans. You keep track of "connected" tables and "unconnected" tables. During join enumeration, mixed combinations are pruned until the plan for the “connected” set of tables was built. Then, we add tables from the "unconnected" set - maybe only as left-deep trees (i.e. the size of the inner is one). Also, knowing that a set of tables are connected through join conditions, will allow further plan pruning based on the presence of join predicates. Integration with star-join, and probably other heuristics, would require to introduce some filtering/pruning strategies on top of the search engine. Just some thoughts…

thanks, merging to master!

@ioana-delaney How can we get disconnected parts before reordering? They can not only be leaf tables (this case is easier to deal with), but also subplans (tables can be joined internally but among subplans there's no join condition). We need to consider both cases and they may co-exist. Also, some disconnection can be eliminated by reordering.

@wzhfy Given a set of input plans (either base table access or plans over derived/complex plans), one can build a graph based on the join conditions among the plans. I think join enumeration should be driven by the presence of join conditions between two plans. Following this approach, I think we can detect items that are not connected and plan them towards the end.

Given {A, B, C} with the join conditions {A.a = B.b, B.b > C.c}. {A, C} should be pruned.

@gatorsmile An equality join in most cases has a better filtering than an inequality join. This can be used heuristically. However, this is not always true. An equality join can be a lookup join from a fact table to a dimension table that it does not reduce anything but an inequality join such as a range join over time period.

My example is not related to inequality join or equi join.

@gatorsmile Your example is correct.
Given A J1 B J2 C:
•	level 0: (A), (B), (C)
•	level 1: {A, B}, ~{A, C}~, {B, C}
•	level 3: {A, B, C}
Given A J1 B J2 C cross join D
•	level 0: (A), (B), (C), (D)
•	level 1: {A, B}, ~{A, C}~, ~{A, D}~, {B, C}, ~{B, D}~
•	level 3: {A, B, C}, ~{A, B, D}~, ~{A, C, D}~, ~{B, C, D}~
•	level 4: {A, B, C, D}
The presence of join conditions between the outer-inner sets are good heuristics to be used during join enumeration.

Right. I misread it. if there is no join predicate between a table and any cluster of tables, we should not consider that table in the join enumeration at all. We can simply push that table to be the last join.

@ioana-delaney Thanks for the replies.

Given A J1 B J2 C:   - case 1
•	level 0: (A), (B), (C)
•	level 1: {A, B}, {A, C}, {B, C}
•	level 3: {A, B, C}
Given A J1 B J2 C cross join D  - case 2
•	level 0: (A), (B), (C), (D)
•	level 1: {A, B}, {A, C}, {A, D}, {B, C}, {B, D}
•	level 3: {A, B, C}, {A, B, D}, {A, C, D}, {B, C, D}
•	level 4: {A, B, C, D}

Currently the algorithm can prune cartesian products in case 1. For case 2, now it is based on the output of ReorderJoin, cartesian products are put at the end. So we have the same effect on the final plan.
In the future, we can merge these rules together.


Kernel task hogging the computer when Atom is open

Prerequisites

[x] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: https://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: https://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
After some minutes using Atom kernel_task takes a disproportioned amount of CPU and the computer becomes unusable, meaning clicking on any button in firefox or any other application reacts after seconds (sometimes 10 or more).

The only way to stop this is closing Atom.
This is a typical pattern for the CPU Load opening and then closing Atom.

I tried:

removing .atom/storage
disabling most of the addons I use, one by one, nothing worked
atom --safe

This seems to be happening since when I upgraded to Catalina but I'm not 100% sure.
Steps to Reproduce
Open and use Atom normally.
Expected behavior:
Everything should run smoothly for the whole day.
Actual behavior:
The computer becomes unusable after a while, sometimes minutes, sometimes an hour or so.
Reproduces how often:
Every time I use Atom.
Versions
$ atom --version
Atom    : 1.42.0
Electron: 4.2.7
Chrome  : 69.0.3497.128
Node    : 10.11.0

$ apm --version
apm  2.4.3
npm  6.2.0
node 10.2.1 x64
atom 1.42.0
python 2.7.17
git 2.24.1

$ node -v
v13.5.0

$ npm -v
6.13.4

$ git --version
git version 2.24.1

$ sw_vers
ProductName:	Mac OS X
ProductVersion:	10.15.2
BuildVersion:	19C57

Additional Information
None

@mindrones - sorry to hear the trouble you experience.
Please could you test something based on a guess:
There are a couple of 'core' packages that are not snoozed when using atom --safe which may be causing this behaviour. Deactivate all packages that has anything to do with git. Then cleanly restart Atom in safe mode. Any change?

@WilSnoop thanks!
Yesterday I've disabled all of the git-related packages and for an hour or so all has been fine with atom --safe.
I'll test more today by using Atom not in safe mode, but with git-related packages disabled, will report back!

So I have not experienced this the whole morning while working with Atom not in safe-mode and with these git-related packages disabled:

Core:

https://atom.io/packages/git-diff
https://atom.io/packages/github
https://atom.io/packages/language-git
https://atom.io/packages/open-on-github

Community:

https://atom.io/packages/git-menu

@WilSnoop I'll be testing with these on and off one by one, but it could save some time if you have an idea of which one of these could be the offender.
Is there a way to track a specific package activity maybe?
Thanks!

Some users have reported problems where some git package was scanning the home directory (did I say that correctly?). I am not a Mac user, so I cannot be more specific; it was not important to me to memorize the details.
Try finding the answer on the discuss.atom.io forum, if a web search does not deliver a positive result. Use the search facility provided there.
Else post a question on the forum. Your Github identity works there too.  There is a knowledgeable gentleman who visits the forum regularly who is able to help - if he wants to.

Thanks!


Light Theme: White Terminal Color unreadable

When using the light theme, the terminal's white color should be rendered as grey/black. In its current form, certain outputs are unreadable. I attached some screenshots to show what I mean:

Ayu Light

Light (Visual Studio)

Fixed it in https://github.com/teabyii/vscode-ayu/pull/69

Hi @ccaspers, could you check please if the latest fixes the issue for you. If not, let's see if we can push your PR :)
P.S. Sorry for the delay in response — I accidentally turned notifications off for the repo :/

@dempfi no problem, glad you saw it anyway :) I updated my pull request. The issue remained after the latest update as brightWhite for the light variant still was at #FFFFFF. I chose #D1D1D1 rather arbitrarily. If the original ayu colors provide a grey with a similar brightness, we should probably choose that instead.

Merged. @ccaspers Thank you very much for your contribution 🙏


[RuntimeError]: RuntimeError: CUDA error: out of memory CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect. For debugging consider passing CUDA_LAUNCH_BLOCKING=1. Compile with TORCH_USE_CUDA_DSA to enable device-side assertions.

Description / 描述
i was trying to use Tesla V100-SXM2-16GB to process finetune.py, and come with an error.
there was another issue closed which is similar to mine: https://github.com/OpenBMB/MiniCPM/issues/78
however i had tried their solution and still get the same error...orz
Case Explaination / 案例解释
i was trying to process finetune.py with the command bellow:
deepspeed finetune.py \     --model_name_or_path ~/.cache/modelscope/hub/OpenBMB/MiniCPM-1B-sft-bf16/ \     --output_dir output/AdvertiseGenLoRA/$formatted_time/ \     --train_data_path data/AdvertiseGenChatML/train.json \     --eval_data_path data/AdvertiseGenChatML/dev.json \     --learning_rate 5e-5 --per_device_train_batch_size 1 \      --per_device_eval_batch_size 1 --fp16 --use_lora \     --gradient_accumulation_steps 1 --warmup_steps 100 \     --max_steps 3000 --weight_decay 0.01 \     --evaluation_strategy steps --eval_steps 500 \     --save_strategy steps --save_steps 500 --seed 42 \     --log_level info --logging_strategy steps --logging_steps 10 \     --deepspeed configs/ds_config_zero2.json
but there is an error:

modify the finetune.py bellow:

delete the token 95338, which means an empty string.

We post an PR in https://github.com/OpenBMB/MiniCPM/pull/134, please check~


IE9 XHR .open Issue

This is a known issue with IE9 and [has a few workarounds] (http://stackoverflow.com/questions/5793831/script5-access-is-denied-in-ie9-on-xmlhttprequest).
We're forking mpulse.js now and I'll contribute a fix if I get a clean one working.

Thanks Mark!  To confirm, you see this in IE9 only?  We should fall back to XDomainRequest for IE9 (which is where I've heard of this error appearing before)

Yes sir, and thank you for your prompt response! A QA initially found the issue on a machine natively running IE9, and I'm able to reproduce it on [the "IE9 on Win7" VM from Microsoft] (https://developer.microsoft.com/en-us/microsoft-edge/tools/vms/).

Should be fixed by https://github.com/SOASTA/mpulse.js/issues/20


Installing iree-jax on M1 Mac errors

I am reading and following https://github.com/iree-org/iree-jax#development -- not sure if this is the right way to build and install iree-jax -- it failed with the following errors.
16:48 $ python -m pip install -e .[test,xla,cpu] -f https://iree-org.github.io/iree/pip-release-links.html
Looking in indexes: https://pypi.apple.com/simple
Looking in links: https://iree-org.github.io/iree/pip-release-links.html
Obtaining file:///Users/y/w/iree-jax
  Installing build dependencies ... done
  Checking if build backend supports build_editable ... done
  Getting requirements to build editable ... done
  Preparing editable metadata (pyproject.toml) ... done
WARNING: iree-jax 0.1.dev1 does not provide the extra 'cpu'
Requirement already satisfied: numpy in /Users/y/miniforge3/lib/python3.9/site-packages (from iree-jax==0.1.dev1) (1.24.0)
Collecting iree-compiler>=20221121.334
  Downloading https://github.com/iree-org/iree/releases/download/candidate-20230121.404/iree_compiler-20230121.404-cp39-cp39-macosx_11_0_universal2.whl (75.0 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 75.0/75.0 MB 7.6 MB/s eta 0:00:00
Collecting jax>=0.4.1
  Downloading https://pypi.apple.com/packages/packages/e5/aa/3adbd63fe645ee4fbc0daeba9423b21970929145bc63cbf2d813b9d150f5/jax-0.4.1.tar.gz (1.2 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.2/1.2 MB 1.6 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... done
Collecting jaxlib>=0.4.1
  Downloading https://pypi.apple.com/packages/packages/81/e5/e5a912dfd84695bbae9d61dae66339fc9956e4a8926032e5c0b88d2ff7c7/jaxlib-0.4.1-cp39-cp39-macosx_11_0_arm64.whl (49.7 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 49.7/49.7 MB 3.0 MB/s eta 0:00:00
Collecting iree-runtime>=20221121.334
  Downloading https://github.com/iree-org/iree/releases/download/candidate-20230121.404/iree_runtime-20230121.404-cp39-cp39-macosx_11_0_universal2.whl (3.3 MB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 3.3/3.3 MB 3.8 MB/s eta 0:00:00
Collecting lit
  Downloading https://pypi.apple.com/packages/packages/2f/5e/8b27ba3715becd8ee26f1f0c2ef9d18fa425a9f6bbf535573b90c08171b4/lit-15.0.7.tar.gz (132 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 132.3/132.3 kB 1.7 MB/s eta 0:00:00
  Preparing metadata (setup.py) ... done
ERROR: Could not find a version that satisfies the requirement iree-tools-xla>=20221121.334; extra == "xla" (from iree-jax[cpu,test,xla]) (from versions: none)
ERROR: No matching distribution found for iree-tools-xla>=20221121.334; extra == "xla"

The command worked after I removed xla from pip -e.
$ python -m pip install -e .[cpu] -f https://iree-org.github.io/iree/pip-release-links.html
....
Successfully installed iree-compiler-20230121.404 iree-jax-0.1.dev1 iree-runtime-20230121.404 jax-0.4.1 jaxlib-0.4.1 opt_einsum-3.3.0 scipy-1.10.0

Thank you - the iree-tools-xla package used to be required and must never have been removed from the requirements and old versions of it rolled off of the repository.

Got it. Thank you, @stellaraccident !


Enchance outputs oct24

Now sets are sorted, so no randomization in saved optimization model (order of terms in constraints or order of constraints themselves).
Apart from that some refactoring changes and some additions to the data dict.

Solves issue #11


Support **kwargs for PyTorch models.

Some models in Pytorch instantiating from torch.nn.Module, has extra parameters in the forward function call. These extra parameters can be passed as Dict or as positional arguments.
Example of PyTorch models supported by Hugging Face -> https://huggingface.co/bert-base-uncased
Some torch models on Hugging face
Eg: https://huggingface.co/docs/transformers/model_doc/bert#transformers.BertModel

inputs = {
     input_ids: Tensor1,
     attention_mask: Tensor2,
     token_type_ids: Tensor3,
}

model = BertModel.from_pretrained("bert-base-uncased") # which is a
# subclass of torch.nn.Module

outputs
= model(**inputs) # model forward method should be expecting the keys in the inputs as the positional
arguments.

Transformers integrated in Pytorch is supported by Hugging Face as well.

Imported from Jira BEAM-14337. Original Jira may contain additional context.
Reported by: Anand Inguva.
Subtask of issue #21435

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

One of the unfortunate downsides of moving to issues is that non-committers can't self-assign/triage issues that they didn't create by default. I think that's a workflow we'll need to support going forward though, I'll look into it. For the moment, I'd recommend commenting/declaring intent to work on any issues.
I have access through https://github.com/apache/beam/blob/4dce7b8857f37608321253073745fe7611a48af9/.asf.yaml#L39 which gives triage permissions, unfortunately that's limited to 20 people though

Once https://github.com/apache/beam/pull/21719 is in, you should be able to do this with chat-op commands

Got it, thanks!

we decided to move forward with a separate param:  https://github.com/apache/beam/pull/21806


[FP]: Chronicle-Wire showing up as FP when building Chronicle-Queue

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:wire:wire:2.24.ea3:::::::*
CVE
CVE-2020-27853
ODC Integration
{"label"=>"Maven Plugin"}
ODC Version
7.3.0
Description
If you build this branch of Chronicle-Queue you will see it reports a false positive for Chronicle-Wire.
The CVEs it refers to relate to another product or library called "Wire"

approved


Pylint with config

List of related issues or pull requests
Related: #168
Describe the changes made in this pull request

Adds pylintrc configuration
Updated githook
Also fixes the violations

Instructions to review the pull request
Generated the standard config using pylint --generate-rcfile, then edited a bit. Here is the diff (default left, this PR right):
					      >	        duplicate-code,
					      >	        empty-docstring,
					      >	        missing-module-docstring,
					      >	        unsubscriptable-object,
           j,				      <
           k,				      <
check-quote-consistency=no		      |	check-quote-consistency=yes
check-str-concat-over-line-jumps=no	      |	check-str-concat-over-line-jumps=yes
init-import=no				      |	init-import=yes
max-line-length=100			      |	max-line-length=120

I've configured sonarcloud to complain only when the 'after merge' metrics fall below the thresholds of standard Sonar Way

Glad you like it! And thanks for reviewing


Add TOVA to kvpress

Add TOVA implementation
Hi, I think the following code will be the kvpress version of TOVA (https://arxiv.org/abs/2401.06104):
class TOVA(ObservedAttentionPress):
    """TOVA implementation"""

    compression_ratio: float = 0.0
    output_attentions: bool = False

    def score(
        self,
        module: nn.Module,
        hidden_states: torch.Tensor,
        keys: torch.Tensor,
        values: torch.Tensor,
        attentions: torch.Tensor,
        kwargs,
    ) -> torch.Tensor:
        assert attentions is not None, 'Set output_attentions=True and attn_implementation="eager" to use this hook'
        scores = attentions[:, :, -1] # take the last token attention
        mean_scores = scores.mean(1, keepdim=True) # average across heads
        bsz, num_key_value_heads, n_tokens, _ = keys.shape
        scores = mean_scores.repeat(1, num_key_value_heads, 1)
        return scores

Here is a notebook link that I used for debugging: link.

Hi @hassidm,
Thanks for your interest in the repository. If I understand correctly, TOVA defines the score of a key-value pair as "the average attention weight of the last token across all heads" ?
If yes I think you could easily make your code compatible with flash attention by looking at what we've done with the SnapKVPress. Feel free to open a PR when you're ready so that we can integrate TOVA in our next release

@hassidm I just opened a PR for TOVA press to make it compatible with flash attention, could you review the code ? I explicitly mentioned your name in the docstring.
  def score(...)

      if attentions is not None:
          attn_weights = attentions[..., -1:, :-1]
      else:
          attn_weights = self.compute_window_attention(module, hidden_states, keys)

      # Average across heads
      scores = attn_weights.mean(1)
      scores = scores.repeat(1, keys.shape[1], 1)

      # Keep the last token. Very slight difference from TOVA but last attention weight is usually very high
      scores = F.pad(scores, (0, 1), value=scores.max().item())

      return scores

As mentioned earlier I reused the SnapKVPress. Hence there is one very slight difference with TOVA: the last token of the prompt will always be kept. I don't think it's an issue because the last token usually receive a very high attention weight so it would be kept anyways.


Pull sidebar out of plugin-app

These are small changes just to organize.

@knowuh, is this PR ready for review? It seems that it's more changes than the title suggests (I can see authoring page too), plus some failing tests, so I'm wondering if it's in the right state.

@pjanik -- Thanks for asking,  there is still a lot of churn.  And you are right, the title is out of date at this point.
@DALove1025 and I can work on getting cleaner PRs put in Today.


Update the Zookeeper tutorial to point to the generated kubectl docs

Since we now generate kubectl doc for each release, it is no longer necessary to use the old redirects. I plan to clean up the references in a few reasonably batched PRs, and finally delete the redirect.

Closing and combining into https://github.com/kubernetes/website/pull/8208.


ROCm build doesn't detect GPU

System information

OS Platform and Distribution  Ubuntu 18.10
TensorFlow installed from source
TensorFlow version tensorflow-1.13.0rc0-cp36-cp36m-linux_x86_64.whl
Python version: 3.6
Installed using pip with wheel built from source:
Bazel version 0.21:
GCC/Compiler version 8.2:
CUDA/cuDNN version N/A:
GPU model and memory: AMD Vega RX 64

Describe the problem
Selecting ROCm in "./configure" doesn't find GPU in "tf.session()"

@Dekken Were you able to successfully install tf_gpu? I have followed the following resource and installed tf_gpu successfully. Could you try it. You might need to modify few things like CUDA version and drivers etc. Please let me know how it progresses. Thanks!

I can install the ROCM deb file and it's fine, but I think that link is sure for Nvidia GPUs no?

logic for ROCm in upstream TensorFlow is not complete yet, to the very least we need #25596 in.
For now please use downstream fork from AMD at: https://github.com/ROCmSoftwarePlatform/tensorflow-upstream

fair enough, just seems a bit odd to have the config option IMO


build successfully but it doesn't print # Hello World

System information (Please provide as much relevant information as possible)

OS Platform and Distribution (e.g. Linux Ubuntu 16.04, Android 11, iOS 14.4): macOS 13.1

$ export GLOG_logtostderr=1
$ bazel run --define MEDIAPIPE_DISABLE_GPU=1
mediapipe/examples/desktop/hello_world:hello_world
…
…
INFO: Build completed successfully, 1 total action
INFO: Running command line: bazel-bin/mediapipe/examples/desktop/hello_world/helINFO: Build completed successfully, 1 total action
dyld[3859]: symbol not found in flat namespace '__ZN6google8protobuf8internal14ArenaStringPtr3SetEONSt3__112basic_stringIcNS3_11char_traitsIcEENS3_9allocatorIcEEEEPNS0_5ArenaE'
zsh: abort      bazel run --define MEDIAPIPE_DISABLE_GPU=1

Hi @Hyun-Jegal,
As from the description I see that build has completed successfully but as mentioned here  it does not printing the Hello World! right?

@Hyun-Jegal,
Thanks for reporting out this bug! We are forwarding this internally and will let us know if there will be any specific update about the bug.

@Hyun-Jegal,
This issue has been fixed in the latest release of the mediapipe version v0.9.1. I could successfully reproduce the issue in Xcode 14.2, macOS Ventura 13.2 with no errors. Please try with this latest release and feel free to close this as issue has been resolved. Thank you!


Accessing the PyObject

I am a new julia user and I am having difficulty understanding how to access the contents of  PyObject that is returned from a python function call.
julia> using PyCall
julia> @pyimport networkx as nx
julia> G = nx.erdos_renyi_graph(10,0.1,directed=true)
PyObject <networkx.classes.digraph.DiGraph object at 0x7f24b6a5f450>
julia> scc = nx.strongly_connected_components(G)
PyObject <generator object strongly_connected_components at 0x7f24b2aeeeb0>
Now, how do I get the connected components. The documentation lacks examples and it would be great if some complicated examples are added. Also, any help with the above issue is appreciated.

The Python
scc.some_method(arg1, arg2)

gets translated into the following Julia code
scc[:some_method](arg1, arg2)

scc is a generator expression in python. I have to iterate over it to get the lists. In python to get the components I would do
for i in scc:
print i
This would give me the lists. If I do an equivalent for loop in julia, it does not print anything.

This works for me on the latest PyCall master branch (Pkg.checkout("PyCall", "master")).  You might need up update your version of PyCall?
julia> scc = nx.strongly_connected_components(G)
PyObject <generator object strongly_connected_components at 0x31fcf8b88>

julia> for s in scc
       @show s
       end
s = Any[5]
s = Any[2,6,4]
s = Any[7]
s = Any[0]
s = Any[1]
s = Any[3]
s = Any[8]
s = Any[9]

How can you cast them into julia native data type to do further manipulations?

Cast what? As you see the variable s is an Julia Any Array (conversion type from a Python list).

Ideally, we should implement conversion of generators into Julia iterators; converting to an explicit Array is problematic for generators that produce huge (possibly unbounded) sequences.

Wow, I totally forgot that I implemented this.   Yes, Julia iteration over PyObjects corresponding to Python generator objects is already implemented and has been for some time.  So, you can loop over a generator with for, convert it into an array with collect, and in general do whatever you might want to do with a Julia iterator.


Complete chunk

this would be the basic structure of complete::chunk
its not even close to being finished. But i wanted to check if you thought about it like this.
any thoughts on the structure?
maby i misunderstood something about y_range() becouse if i try getting a block from the lowest y it always fails.
this is true for complete::chunk but also for CurrentJavaChunk

maby i misunderstood something about y_range() becouse if i try getting a block from the lowest y it always fails.
this is true for complete::chunk but also for CurrentJavaChunk

-64 should be the lowest y value that is valid for a Minecraft chunk nowadays correct? Is that what's failing?

normaly -64 is the lowest, yes.
it is also the lowest y a block can be read from.
but why does y_range() return -80 then?

ok... i kinda figured it out.
it seems like on of my mods (only perfomance optimizations) does something weird with the world data.

Not sure how far you want to take this PR. The basic structure is here and allows iterating over blocks in a read only way. I'm happy for that to be this PR.
I might be interested in working on top of this to make it serializable back into a region file, if you're not already interested in doing so.

just wanted to add the most basic pre18 | pre13 support.
btw...
thanks for your always quick answers, and for letting me work on this

Sounds great, and no problem. Thank you for contributing.
Let me know when the PR is ready and I'll have a look again.


how to make it release the incremental part to medium

Hello,
Can you provide a bit more details of what are you trying to do?
In general action allows you to release a markdown file to Medium.

I want to use this action to publish the markdown file to medium every time I commit.
But now my understanding of this action, the configuration item markdown_file is a fixed value?
btw, how to get app_id.

When i click on that link，I got:

To get app_id you need to:

Login on Medium
Visit page https://medium.com/me/applications
Create new application

So, this action does minimal thing - it publishes whatever we tell it to publish.
Here is a hack how you can determine changed files and detect markdown files from it:
    - id: files
      uses: jitterbit/get-changed-files@v1
    - id: posts
      name: Detecting posts from the changes
      run: |
        i=0
        for changed_file in ${{ steps.files.outputs.added_modified }}; do
          echo "Do something with ${changed_file}."
          if [[ "${changed_file}" == "content/posts"* ]];
          then
            echo "File ${changed_file} matched post."
            echo "::set-output name=post${i}::${changed_file}"
            ((i=i+1))
          fi
        done
    - if: steps.posts.outputs.post0
      name: Publish to medium
      uses<EMAIL_ADDRESS>      with:
        app_id: ${{ secrets.MEDIUM_APP_ID }}
        app_secret: ${{ secrets.MEDIUM_APP_SECRET }}
        access_token: ${{ secrets.MEDIUM_ACCESS_TOKEN }}
        markdown_file: ${{ steps.posts.outputs.post0 }}
        base_url: https://myblog.com

I will extend documentation with this example

It's released in here -> https://github.com/InfraWay/post-medium-action/releases/tag/v1.3.0

To get app_id you need to:

Login on Medium
Visit page https://medium.com/me/applications
Create new application

yes, I have already login on Medium, but when I navigate to " https://medium.com/me/applications", and then I entered a page that says I don't have permission
OK, fine. I will explore this problem myself

So, this action does minimal thing - it publishes whatever we tell it to publish.
Here is a hack how you can determine changed files and detect markdown files from it:
    - id: files
      uses: jitterbit/get-changed-files@v1
    - id: posts
      name: Detecting posts from the changes
      run: |
        i=0
        for changed_file in ${{ steps.files.outputs.added_modified }}; do
          echo "Do something with ${changed_file}."
          if [[ "${changed_file}" == "content/posts"* ]];
          then
            echo "File ${changed_file} matched post."
            echo "::set-output name=post${i}::${changed_file}"
            ((i=i+1))
          fi
        done
    - if: steps.posts.outputs.post0
      name: Publish to medium
      uses<EMAIL_ADDRESS>      with:
        app_id: ${{ secrets.MEDIUM_APP_ID }}
        app_secret: ${{ secrets.MEDIUM_APP_SECRET }}
        access_token: ${{ secrets.MEDIUM_ACCESS_TOKEN }}
        markdown_file: ${{ steps.posts.outputs.post0 }}
        base_url: https://myblog.com

This part filters files from directory where all post are.
"${changed_file}" == "content/posts"*
I will extend documentation with this example

I get it , thank you every much


remove unused log4j dependency

https://github.com/apache/rocketmq/issues/3819
Make sure set the target branch to develop
What is the purpose of the change
Trying to remove the dependency on log4jv1
Brief changelog
pom change
Verifying this change
CI build
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

log4jv1 is still in use in some subprojects
[ERROR]   The project org.apache.rocketmq:rocketmq-logappender:4.9.3-SNAPSHOT (/home/travis/build/apache/rocketmq/logappender/pom.xml) has 1 error
[ERROR]     'dependencies.dependency.version' for log4j:log4j:jar is missing. @ org.apache.rocketmq:rocketmq-logappender:[unknown-version], /home/travis/build/apache/rocketmq/logappender/pom.xml, line 35, column 21
[ERROR]
[ERROR]   The project org.apache.rocketmq:rocketmq-test:4.9.3-SNAPSHOT (/home/travis/build/apache/rocketmq/test/pom.xml) has 1 error
[ERROR]     'dependencies.dependency.version' for log4j:log4j:jar is missing. @ org.apache.rocketmq:rocketmq-test:[unknown-version], /home/travis/build/apache/rocketmq/test/pom.xml, line 31, column 21


Use mongodb， can't  new MongoClient

Environment
node: v16.14.0
nitropack: lastest
Reproduction
import { MongoClient } from 'mongodb'
export default cachedEventHandler(async (event)=>{
const { page, pageSize = 10 } = getQuery(event)
// Connection URL
const mongoUrl = 'mongodb://localhost:27017';
console.log('aaaa')
const client = new MongoClient(mongoUrl);
console.log('bbbb')
await client.connect();
const db = client.db('mydb');
const collection = `db.collection('list');
......
Describe the bug
[nitro] [request error] [unhandled] mongodb_connection_string_url_1.default is not a constructor
Additional context
No response
Logs
No response

Possibly duplicate #949 #1024


Fix LeafVersion serde

The default implementation maps to visit_u64. The current implementation
does not roundtrip with many deserializers, including serde_json. See
the failing test in the second commit

This still fails on secp256k1::Parity type because of the same reason. I think we should also fix it there to use visit_u64 so that we users can use serde_json. Even though we are testing the structures with serde_test, I think it should be considered a bug if the given serde_json test fails.
Anyways, input from people using serde is appreciated. My intention is to fix #1327.

Heh, I'm surprised we're using serde_test anywhere because IME it's completely useless.

Having said this, okay, I'm fine deserializing u64s like this but IMO it's a bug in serde(_json) that you cannot actually roundtrip u8s by using serialize_u8 and deserialize_u8.

This looks like robustness principle on the surface but actually isn't so concept ACK.

IMO it's a bug in serde(_json)

Yeah, looks like one.

Looks like the CI failure is an actual code breakage of some sort.

Heh, I'm surprised we're using serde_test anywhere because IME it's completely useless.

You meant "IMO", right? Can you expand on this? I ask because we use it in a few places and I added tests using it thinking it was useful, I'm interested to know if I was wrong?

lol, as my kids are fond of telling me "I'm such a boomer" I had to look up the definition of IME (In My Experience) :)
We have a serde_json dev dependency already so should it be on my todo-at-some-stage list to remove the serde_test usage and favour roundtrip tests with serde_json?


Fix makefiles (align slides & exercises, add copy target)

should close PR 1169

 Clean makefiles & update contribution guidelines


Consider repurposing this doc

Would it be possible to scrub this doc of the references to the performance testing, and instead just have it reflect how to create a multi-step web test?
As of the time of writing, the performance test feature is already gone (as of March 2020) but this page continues to be relevant since Azure Application Insights has the ability to run availability tests with these multi-step tests.
There's a link to this page from this one on the Azure side:
"For guidance on creating Visual Studio web tests consult the official Visual Studio 2019 documentation."
https://docs.microsoft.com/en-us/azure/azure-monitor/app/availability-multistep
And here are a couple of previous doc threads that have run into the same point of confusion.
https://github.com/MicrosoftDocs/azure-devops-docs/issues/3793#issuecomment-494181005
https://github.com/MicrosoftDocs/azure-docs/issues/44978#issuecomment-567751463
If we could at least include a clearer note on this page to say that the process has been deprecated ONLY for performance tests, but is still supported through the lifecycle of VS 2019 for availability test purposes, that might help some users.

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

ID: cff4fb7d-66a8-29bc-ff88-dca4f11074b0
Version Independent ID: 02d11237-7352-d699-852c-b7f4117dd628
Content: Create a Web Service Test - Visual Studio
Content Source: docs/test/how-to-create-a-web-service-test.md
Product: visual-studio-windows
Technology: vs-ide-test
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@dcbrown16 Unfortunately I think the confusion lies in the naming choices made by Visual Studio and App Insights which at this point aren't going to be changed. To create an Application Insights  multi-step web test in Visual Studio you need to create what Visual Studio calls a "Web Performance and Load Test Project".
What we call the Performance Testing feature in App Insights is basically the load testing feature from Visual Studio Online which also happens to rely on web performance and load test project type. But it is really just the load testing portion that is completely deprecated per the previous announcement.  To create a multi-step web test in Visual Studio you are still using a  Visual Studio "Web Performance and Load Test Project" which while VS 2019 is the last version that will contain this project type it is still fully supported in Visual Studio 2019 and remains the only way to create an Application Insights multi-step web test.

@mrbullwinkle -- thank you for answering.
I am closing this issue now.  You are welcome to @ mention me for any followup.
We hope to hear from you again.

And if it is something that we need clarity within the App Insights docs rather than simply linking to the VS doc, definitely let me know and I am happy to try making some modifications on the App Insights docs side


update hugo version

https://github.com/kubeflow/website/issues/3070
/assign @Bobgy

Checked locally & here https://deploy-preview-3072--competent-brattain-de2d6d.netlify.app/
/assign @shannonbradshaw

@Arhell This PR is probably not needed because of https://github.com/kubeflow/website/pull/3078


I got 9 emeralds at just the start

How can I reduce that?

Please check the Lootr description on CurseForge. Specifically, this is answered by the following sections: Q. How does Lootr generate loot? and Q. How do I change what loot is generated with Lootr? How do I include loot from other mods in Lootr containers?.


chore: add code generator for plugin default config

Signed-off-by: xuwu<EMAIL_ADDRESS>Pre-Checklist
Note: please complete ALL items in the following checklist.

[x] I have read through the CONTRIBUTING.md documentation.
[x] My code has the necessary comments and documentation (if needed).
[x] I have added relevant tests

Description

Generate embed_gen.go automatically which is used to embed YAML files
Add go generate in Makefile

Related Issues

#727
New Behavior (screenshots if needed)

N/A

What a beautiful optimization!!!


[DSL] Support Common Workflow Language

This is a low priority enhancement. If you are interested in this feature, please express your interest in the comments.
Describe the solution you'd like
Support translation from Common Workflow Language to Camel.
To do this, implement the StepParserService for Common Workflow Language.
Describe alternatives you've considered
If there is interest, we could also implement the DeploymentGeneratorService to generate CWL source code.
Additional context
This is a generic workflow language created several years ago used by some applications. It could be nice to be interoperable with it.

Really interesting but I would like to see the workflow serverless language in camel generally. Please check this link: https://blog.kie.org/2021/12/implementing-the-filter-eip-using-camel-k-and-kogito.html


New material system

Link between lights and materials
Lights and materials are tightly bound. A material can be seen as a function of light. Because we can’t generalize light, a material has to implement all kinds of lights.
So, what’s a GPULight? Well, it’s basically a function that will update itself into a GPUMaterial. Do we have to batch? I don’t think so. The GPUMaterial will have several interfaces (i.e. one per light kind) we’ll be able to use. The first one is of course the ambient slot interface. Then, when we commit stuff, we can simply send it via the slot so that it’s get updated.

 :D
2015-01-23 11:02 GMT+01:00 Dimitri Sabadie<EMAIL_ADDRESS>
Lighting
Lighting is, at current revision
https://github.com/phaazon/photon/tree/1cf78bc93042704fc22ab82c1e88deeddcde913f,
implemented only in terms of Omni lighting. This is an issue because
we’ll want several kinds of light be included among:

ambient ligthing (see #26
https://github.com/phaazon/photon/issues/26)
omnidirectional lighting, already implemented
spot lighting (see #47 https://github.com/phaazon/photon/issues/47)
etc.

The idea of this change is to refactor the current lighting system to make
it more about to include a new kind of light.
Light, GPULight, Lighting, Shadowing and Lit Light
This is a mandatory core type used to represent lights. There’re a few
functions to create lights.
GPULight
This type is less intuitive, yet very simple. It represents a Light on
GPU, which is a piece of code plugged onto a render to make it lit. To
understand that type, we have to understand what a Shaded object and
Rendered objects are.
A Rendered object represents a rendered scene. If you got a Rendered
object, you know that it gathers at least one mesh – but it’ll almost
always contain the whole scene – rendered.
A Shaded object is a Rendered object with material information attached.
A Shaded object has two basic primitives:

runShaded, which simply executes the Shaded code
runShadedNoMaterial, which executes the Shaded code discarding
materials

runShadedNoMaterial can be useful when a higher level of abstraction
needs to run a Shaded object but it doesn’t need materials (e.g. shadow
mapping).
Then, what does a GPULight actually do? Well, it runs a Shaded object
with extra information around. For instance, a GPULight obtained via an
Ambient light will simply add color and power information onto the Shaded
object. How is this performed? A GPULight has access to a few other
objects – which are generated by the renderer – so that it can does its
job. A GPULight is just a function of Lighting, Shadowing and Accumulation
and outputs a LitBatch. A LitBatch is a simple object used to batch
lights so that we lighten the scene with all omnidirectional lights once,
all ambient lights once, etc.
Lighting
This type is pretty convenient because it contains anything a light might
want. It has shader programs, it has uniforms, it has shadowing
information. It’s a helper bridge to the renderer lights can use to
implement various kinds of effects.
Shadowing
This type is included in Lighting and expose shadowing concepts. For
instance, it has programs and uniforms for omnidirectional lights and spot
lights.
Lit
This type is the higher level of abstraction of lights. It’s an end-user
type which is created by the lighten function.
Future
What happens if I suddenly spawn at the corner of a room and shouts:
— Holy shit! @xtrium https://github.com/xtrium added ultra cool area
lights! I need the same thing!
How new lights integration works?
—
Reply to this email directly or view it on GitHub
https://github.com/phaazon/photon/issues/45.

--
Charles 'xtrium http://xtrium.net/' Bulckaen
:: www.frequency.fr
:: www.lnx-demo.org

@xtrium sowhat?  I just got it!

C'est bien, c'est explicite comme commit message, ça. :p

Quoi donc ?

Done as I’ve removed explicit materials. They’re now fully polymorphic, and so are shaders. \o/


Fix role ARN comparison for user ID strict check (#669)

What this PR does / why we need it:
Cherry-pick https://github.com/kubernetes-sigs/aws-iam-authenticator/pull/669
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #

/ok-to-test

/lgtm
/approve


[Slider] Resolve slider value to respect max and min

We encountered a case when current value has exceeded maximum value.
Consider following example of <Slider min={10000} max={37000} step={10000} /> In some cases the value of Slider was 40000.
The problem was caused by this line. We extracted all validation of value in min-max range to one function resolveValue(value).
Screenshot:

I need to try out the demos of the documentation to see if they are still working. Still, the source code looks good.

@janmarsicek Thanks!

@oliviertassinari Happy I could contribute! 👍

@oliviertassinari Hey, can I ask when this fix will be available on NPM in new patch version? Thanks

By the end of the week-end.

Super awesome 🎉  thank you!