More thorough tests

Shallow rendering unit tests only get us so far. It's worth investigating into more heavyweight test utils. Perhaps something like this setup with karma

I've been setting up karma tests in a branch of my react-transform-boilerplate fork and it seems to work nicely (for basic rendering at least).
Considering I plan to rebase on that boilerplate, adding karma tests for react-player should be fairly easy after that.


About these two datasets: Avunue and Shanghai Tech

About these two datasets: Avunue and Shanghai Tech，can we train them directly or do we need to preprocess them first? If it is preprocessing, is the operation the same as the UCSD dataset?
Thanks！

Yes. Training is straightforward on other datasets. Note that training on Shanghai Tech dataset takes more time. I will try to find time to clean the code for preparing the data files for other datasets.

About these two datasets: Avunue and Shanghai Tech，can we train them directly or do we need to preprocess them first? If it is preprocessing, is the operation the same as the UCSD dataset?
Thanks！

Hello,
could you please send me the premodels about avenu and shanghaitech.
I really need them. and my email is<EMAIL_ADDRESS>or<EMAIL_ADDRESS>Thanks!!!

and

About these two datasets: Avunue and Shanghai Tech，can we train them directly or do we need to preprocess them first? If it is preprocessing, is the operation the same as the UCSD dataset?
Thanks！

Hello,
could you please send me the premodels about avenu and shanghaitech.
I really need them. and my email is<EMAIL_ADDRESS>or<EMAIL_ADDRESS>Thanks!!!

hello，do have the premodels about shanghaitech？i do not know deal with shanghaitech dataset by using matlab_script.Thanks

About these two datasets: Avunue and Shanghai Tech，can we train them directly or do we need to preprocess them first? If it is preprocessing, is the operation the same as the UCSD dataset?
Thanks！

Hello,
could you please send me the premodels about avenu and shanghaitech.
I really need them. and my email is<EMAIL_ADDRESS>or<EMAIL_ADDRESS>Thanks!!!

hello，do have the premodels about shanghaitech？i do not know deal with shanghaitech dataset by using matlab_script.Thanks


Add rand48 functions

r? @JohnTitor
(rust-highfive has picked a reviewer for you, use r? to override)

@bors r+

:pushpin: Commit a68d3ed0acf3b77e191b4756146fd58f851e8591 has been approved by JohnTitor
It is now in the queue for this repository.

:hourglass: Testing commit a68d3ed0acf3b77e191b4756146fd58f851e8591 with merge 4493a07473243c4574793db35206ba8e032b6627...

:sunny: Test successful - checks-actions, checks-cirrus-freebsd-12, checks-cirrus-freebsd-13, checks-cirrus-freebsd-14
Approved by: JohnTitor
Pushing 4493a07473243c4574793db35206ba8e032b6627 to master...


CMR-6818 - Change 2 - Now with GitHub instructions

Part of the effort with this ticket was to setup building in github, the last pull request allowed a github action to trigger which created artifacts. A python wheel file was one of those artifacts. After testing we found we could install directly from github with the pip3 command (which we were not sure of before). This pull request is for the documentation updates showing how to install using this process.

You can ignore the link syntax comments

as discussed in slack, I'm removing some quotes that are not needed. Then I will merge.


Middleware/Rack/Rollbar::Middleware::Rails::RollbarMiddleware#call very slow

New Relic says the average call for this method takes over 8 seconds. Any idea?

@jamesfzhang it may be related to delay issues in our API since that middlewares sends a POST request when an error is found. In general you shouldn't have this behavior.
With this, we always recommend use any of the delay handers we support, take a look at https://github.com/rollbar/rollbar-gem#asynchronous-reporting.
If you have any other problem please ping us.
Thanks!

Sorry for necro-ing, but I'm actually experiencing something very similar - and we are using asynchronous reporting via DelayedJob.  However, this call doesn't appear to be async, and appears to hang until it times out.

I'm wondering if in our case it's something to do with the interaction between Rollbar and rack-timeout, because when one of these long calls occurs, it sets off a chain of them which can take down our entire server.

Hi @zakthompson, thanks for the report. It's quite strange having that duration time on our call. Are you sure the time you are wasting is in the middleware itself?
Did you try running the on-live profiler that New Relic has? That will give you accurated stats about the time you are wasting on each method.


alwaysRefresh option for route definition

Hi,
There was an option "alwaysRefresh" on route definitions which forced component to be reloaded when switch between two routes using the same component.
How can I handle this functionality in current router version ?

I think canReuse is what you need.

Thank you, it works!


fix: get autosynth working again

Should get autosynth working again, I noticed a few issues that we will want to address in the upstream generator:

webpack.config.js generates with linting issues.
the system tests generate with the name translations rather than @google-cloud/translate.
the system tests have linting issues:

/usr/local/google/home/bencoe/google/nodejs-translate/system-test/fixtures/sample/src/index.js
  19:29  error  "@google-cloud/translate" is not found                                                            node/no-missing-require
  22:9   error  'translationServiceClient' is assigned a value but never used                                     no-unused-vars
  23:15  error  Replace `"translationServiceClient·was·created!"` with `'translationServiceClient·was·created!'`  prettier/prett

fixes #379

re: system test name - will be taken care of by https://github.com/googleapis/gapic-generator-typescript/issues/134 (we just need to give a hint to the generator about the package name to use).


added support for sbt

I added support for SBT. I tested it and it works.
p.s. I noticed however for scala routes the gutter icon does not appear (completion on uri parameters works). I'll have a look why the gutter icon does not appear.

Also beware of changing the order of the java imports, as the build uses a checkstyle check where it reports errors if the code formatting is not correct.

This can be closed as you already added support for sbt. I leave it up to you to close it.

Ah thanks closing


error[E0277]: the size for values of type ... cannot be known at compilation time

The following errors are issued when trying to build the ERC20 contract with rust nightly.
error[E0277]: the size for values of type `Erc20` cannot be known at compilation time
error[E0277]: the size for values of type `DefaultEnvironment` cannot be known at compilation time
error[E0277]: the size for values of type `Transfer` cannot be known at compilation time
error[E0277]: the size for values of type `__ink_EventBase` cannot be known at compilation time
error[E0277]: the size for values of type `Approval` cannot be known at compilation time
error[E0277]: the size for values of type `__ink_MessageDispatchEnum` cannot be known at compilation time
error[E0277]: the size for values of type `__ink_ConstructorDispatchEnum` cannot be known at compilation time
error[E0277]: the size for values of type `ink_env::AccountId` cannot be known at compilation time

etc.

5   | #[ink::contract]
    | ^^^^^^^^^^^^^^^^ doesn't have a size known at compile-time

17 |         balances: ink_storage::collections::HashMap<AccountId, Balance>,
   |                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ doesn't have a size known at compile-time

Upgrading the contract and the toml file to the latest versions found here made it work.


Orders query address fields performance improvements

I want to merge this change because it improves performance of address fields when querying orders.
Local improvement when querying 2 addresses on 10 orders:
before: "query-count": 34
after: "query-count": 15
Impact

[ ] New migrations
[ ] New/Updated API fields or mutations
[ ] Deprecated API fields or mutations
[ ] Removed API types, fields, or mutations
[ ] Documentation needs to be updated

Pull Request Checklist

[ ] Privileged queries and mutations are guarded by proper permission checks
[ ] Database queries are optimized and the number of queries is constant
[ ] Database migration files are up to date
[X] The changes are tested
[ ] GraphQL schema and type definitions are up to date
[ ] Changes are mentioned in the changelog

Task linked: SALEOR-2732 Optimize address resolvers in Order type


allow hub concept for prometheus init container busybox image

In the current default state-of-affairs we can use global.hub and other chart *.hub values to pull from alternate image locations, which is great. However, the prometheus chart has a hardcoded busybox image in the initContainer. Here's what I used to identify this while running a helm template of the istio chart:
$ helm template --set global.hub="myhub" --set prometheus.hub="myhub" . | grep image:
        image: "myhub/proxy_init:master-latest-daily"
        image: [[ annotation .ObjectMeta `sidecar.istio.io/proxyImage`  "myhub/proxyv2:master-latest-daily"  ]]
          image: "myhub/kubectl:master-latest-daily"
          image: "myhub/kubectl:master-latest-daily"
          image: "myhub/galley:master-latest-daily"
          image: "myhub/proxyv2:master-latest-daily"
        image: "myhub/mixer:master-latest-daily"
        image: "myhub/proxyv2:master-latest-daily"
        image: "myhub/mixer:master-latest-daily"
        image: "myhub/proxyv2:master-latest-daily"
          image: "myhub/pilot:master-latest-daily"
          image: "myhub/proxyv2:master-latest-daily"
        image: "busybox:1.30.1"
          image: "myhub/prometheus:v2.8.0"
          image: "myhub/citadel:master-latest-daily"
          image: "myhub/sidecar_injector:master-latest-daily"

This change allows for dynamically setting that image's location, and would allow us to rely solely on signed images from our known location(s).
The same helm template command looks like the following after making this change:
$ helm template --set global.hub="myhub" --set istio.prometheus.hub="myhub" --set prometheus.hub="myhub" --set prometheus.busybox.hub="myhub" . | grep image:
        image: "myhub/proxy_init:master-latest-daily"
        image: [[ annotation .ObjectMeta `sidecar.istio.io/proxyImage`  "myhub/proxyv2:master-latest-daily"  ]]
          image: "myhub/kubectl:master-latest-daily"
          image: "myhub/kubectl:master-latest-daily"
          image: "myhub/galley:master-latest-daily"
          image: "myhub/proxyv2:master-latest-daily"
        image: "myhub/mixer:master-latest-daily"
        image: "myhub/proxyv2:master-latest-daily"
        image: "myhub/mixer:master-latest-daily"
        image: "myhub/proxyv2:master-latest-daily"
          image: "myhub/pilot:master-latest-daily"
          image: "myhub/proxyv2:master-latest-daily"
        image: "myhub/busybox:1.30.1"
          image: "myhub/prometheus:v2.8.0"
          image: "myhub/citadel:master-latest-daily"
          image: "myhub/sidecar_injector:master-latest-daily"

And just to make sure, what it looks like by default after making these changes:
$ helm template . | grep image:
        image: "gcr.io/istio-release/proxy_init:master-latest-daily"
        image: [[ annotation .ObjectMeta `sidecar.istio.io/proxyImage`  "gcr.io/istio-release/proxyv2:master-latest-daily"  ]]
          image: "gcr.io/istio-release/kubectl:master-latest-daily"
          image: "gcr.io/istio-release/kubectl:master-latest-daily"
          image: "gcr.io/istio-release/galley:master-latest-daily"
          image: "gcr.io/istio-release/proxyv2:master-latest-daily"
        image: "gcr.io/istio-release/mixer:master-latest-daily"
        image: "gcr.io/istio-release/proxyv2:master-latest-daily"
        image: "gcr.io/istio-release/mixer:master-latest-daily"
        image: "gcr.io/istio-release/proxyv2:master-latest-daily"
          image: "gcr.io/istio-release/pilot:master-latest-daily"
          image: "gcr.io/istio-release/proxyv2:master-latest-daily"
        image: "busybox:1.30.1"
          image: "docker.io/prom/prometheus:v2.8.0"
          image: "gcr.io/istio-release/citadel:master-latest-daily"
          image: "gcr.io/istio-release/sidecar_injector:master-latest-daily"

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

Needs approval from an approver in each of these files:

install/OWNERS

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

Hi @rockholla. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

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.

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project (if not, look below for help). Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://cla.developers.google.com/ to sign.
Once you've signed (or fixed any issues), please reply here (e.g. I signed it!) and we'll verify it.
What to do if you already signed the CLA
Individual signers

It's possible we don't have your GitHub username or you're using a different email address on your commit. Check your existing CLA data and verify that your email is set on your git commits.

Corporate signers

Your company has a Point of Contact who decides which employees are authorized to participate. Ask your POC to be added to the group of authorized contributors. If you don't know who your Point of Contact is, direct the Google project maintainer to go/cla#troubleshoot (Public version).
The email used to register you as an authorized contributor must be the email used for the Git commit. Check your existing CLA data and verify that your email is set on your git commits.
The email used to register you as an authorized contributor must also be attached to your GitHub account.

ℹ Googlers: Go here for more info.

I signed it!

/ok-to-test

We do something similar in other components like:
{{- if contains "/" .Values.global.proxy_init.image }}
        image: "{{ .Values.global.proxy_init.image }}"
{{- else }}
        image: "{{ .Values.global.hub }}/{{ .Values.global.proxy_init.image }}:{{ .Values.global.tag }}"
{{- end }}

Although I suppose we don't want to use global hub and global tag as we don't publish our own busybox image (afaik)

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

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
47ed2768b111974256ac9ab8c5c6999785b48fc5
link
/test e2e-simpleTests

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.

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

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
47ed2768b111974256ac9ab8c5c6999785b48fc5
link
/test e2e-simpleTests

prow/e2e_pilotv2_auth_sds.sh
47ed2768b111974256ac9ab8c5c6999785b48fc5
link
/test istio_auth_sds_e2e

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.

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

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
47ed2768b111974256ac9ab8c5c6999785b48fc5
link
/test e2e-simpleTests

prow/e2e_pilotv2_auth_sds.sh
47ed2768b111974256ac9ab8c5c6999785b48fc5
link
/test istio_auth_sds_e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
47ed2768b111974256ac9ab8c5c6999785b48fc5
link
/test istio-pilot-e2e-envoyv2-v1alpha3

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.

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

Test name
Commit
Details
Rerun command

prow/e2e-simpleTests.sh
47ed2768b111974256ac9ab8c5c6999785b48fc5
link
/test e2e-simpleTests

prow/e2e_pilotv2_auth_sds.sh
47ed2768b111974256ac9ab8c5c6999785b48fc5
link
/test istio_auth_sds_e2e

prow/istio-pilot-e2e-envoyv2-v1alpha3.sh
47ed2768b111974256ac9ab8c5c6999785b48fc5
link
/test istio-pilot-e2e-envoyv2-v1alpha3

prow/e2e-mixer-no_auth.sh
47ed2768b111974256ac9ab8c5c6999785b48fc5
link
/test e2e-mixer-no_auth

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.

I've updated to use a helper to address lint failures against helm 2.7.2 with the initial PR code, which I think actually is a bit better, makes me rethink how I've been doing similar stuff in other charts, so hopefully you all feel the same.
I've re-run the manual templating validation tests in the description and all looks good. I'll monitor the rest here to see what other tests or other things I might need to adjust.

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

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
99ff65adda56eb07babde67a049815b5e84ef438
link
/test istio-integ-k8s-tests

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

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

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
408778ca387404998ce0844ac5c46c0850759a53
link
/test istio-integ-k8s-tests

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

closing in favor of #13571


SdkClient lacks of client parameter information

We call listClients to get the client list. Take this as example,

client is a client parameter for ServiceClient, Baz, and BazFoo.
name is a client parameter for Baz, and BazFoo, not for ServiceClient.

So when I call listClients, I should somehow know the client parameters of specific clients, or I should somehow know a specific parameter belongs to which client.

this is handled in SdkClientType, not in listClients. after you adopt getAllOperations, you will get that info.


Feat:Want to Update Canteen Cards

Describe the feature
Hey want to update Canteen cards currently has no fixed width or length which is making site inconsistent
plus canteen card should also have ratings as well so i will add those and some other fields like opening and closing time
Add ScreenShots

I will fix this whole card full ensure responsiveness and dark mode as well
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Ok, Assigning this issue to you.


[Performance] To use the large 2D load size to load the DPAS operands A and B.

To use the dense stride instead of warp stride in tiling the tt.dot operation. So that we can use packed 2D load/store to load the operands A and B.
Here is the difference of dense stride and warp stride in load 2d:
https://github.com/intel/intel-xpu-backend-for-triton/blob/3541627120d23258baac9d633754b02940d3cf57/lib/Conversion/TritonGPUToLLVM/LoadStoreOpToLLVM.cpp#L609

This issue is duplicated with https://github.com/intel/intel-xpu-backend-for-triton/issues/1323


replace torch.nn.functional.tanh (depreciated)

torch.nn.functional.tanh is depreciated. I replaced this line (F.tanh in this case) with the preferred method, torch.tanh
see pytotch documentation on the subject here

Cleaning up my PRs. Closing due to lack of repo activity.


[Z-2591] Crashes on collaboration attempts

Check for existing issues

[X] Completed

Describe the bug / provide steps to reproduce it
Zed itself crashes if I either attempt to start a collaboration session or attempt to accept an invite from someone else. There are errors in the log as well.
Environment
Zed: v0.93.4 (stable)
OS: macOS 11.7.8
Memory: 16 GiB
Architecture: x86_64
If applicable, add mockups / screenshots to help explain present your vision of the feature
No response
If applicable, attach your ~/Library/Logs/Zed/Zed.log file to this issue.
If you only need the most recent lines, you can run the zed: open log command palette action to see the last 1000.
2023-07-10T14:16:50 [INFO] ========== starting zed ==========
2023-07-10T14:16:50 [INFO] Opening main db
2023-07-10T14:16:50 [ERROR] crates/zed/src/main.rs:87: login shell exited with error
2023-07-10T14:16:50 [INFO] Opening main db
2023-07-10T14:16:50 [INFO] set status on client 0: Authenticating
2023-07-10T14:16:50 [INFO] set status on client 0: Connecting
2023-07-10T14:16:50 [INFO] build git repository ".git"
2023-07-10T14:16:51 [INFO] connected to rpc endpoint https://collab.zed.dev/rpc
2023-07-10T14:16:51 [INFO] Opening main db
2023-07-10T14:16:51 [INFO] Opening main db
2023-07-10T14:16:51 [INFO] add connection to peer
2023-07-10T14:16:51 [INFO] waiting for server hello
2023-07-10T14:16:51 [INFO] got server hello
2023-07-10T14:16:51 [INFO] set status to connected (connection id: ConnectionId { owner_id: 0, id: 0 }, peer id: PeerId { owner_id: 188, id: 120000 })
2023-07-10T14:16:51 [INFO] set status on client 0: Connected { peer_id: PeerId { owner_id: 188, id: 120000 }, connection_id: ConnectionId { owner_id: 0, id: 0 } }
2023-07-10T14:16:51 [INFO] starting language server name:typescript-language-server, path:"/Volumes/projects/front-end", id:0
2023-07-10T14:16:51 [INFO] starting language server name:eslint, path:"/Volumes/projects/front-end", id:1
2023-07-10T14:16:51 [INFO] starting language server name:json-language-server, path:"/Volumes/projects/front-end", id:2
2023-07-10T14:16:53 [INFO] unhandled notification client/registerCapability:
{
  "registrations": [
    {
      "id": "db6e1e22-b45f-461c-8369-396c4d7233fa",
      "method": "workspace/didChangeWorkspaceFolders",
      "registerOptions": {}
    }
  ]
}
2023-07-10T14:16:54 [ERROR] crates/project/src/project.rs:2500: No such file or directory (os error 2)
2023-07-10T14:18:13 [INFO] ========== starting zed ==========
2023-07-10T14:18:13 [INFO] Opening main db
2023-07-10T14:18:13 [ERROR] crates/zed/src/main.rs:87: login shell exited with error
2023-07-10T14:18:14 [INFO] Opening main db
2023-07-10T14:18:14 [INFO] set status on client 0: Authenticating
2023-07-10T14:18:14 [INFO] set status on client 0: Connecting
2023-07-10T14:18:14 [INFO] build git repository ".git"
2023-07-10T14:18:14 [INFO] Opening main db
2023-07-10T14:18:14 [INFO] connected to rpc endpoint https://collab.zed.dev/rpc
2023-07-10T14:18:14 [INFO] Opening main db
2023-07-10T14:18:14 [INFO] add connection to peer
2023-07-10T14:18:14 [INFO] waiting for server hello
2023-07-10T14:18:14 [INFO] got server hello
2023-07-10T14:18:14 [INFO] set status to connected (connection id: ConnectionId { owner_id: 0, id: 0 }, peer id: PeerId { owner_id: 188, id: 120031 })
2023-07-10T14:18:14 [INFO] set status on client 0: Connected { peer_id: PeerId { owner_id: 188, id: 120031 }, connection_id: ConnectionId { owner_id: 0, id: 0 } }
2023-07-10T14:18:14 [INFO] starting language server name:typescript-language-server, path:"/Volumes/projects/front-end", id:0
2023-07-10T14:18:14 [INFO] starting language server name:eslint, path:"/Volumes/projects/front-end", id:1
2023-07-10T14:18:14 [INFO] starting language server name:json-language-server, path:"/Volumes/projects/front-end", id:2
2023-07-10T14:18:16 [INFO] unhandled notification client/registerCapability:
{
  "registrations": [
    {
      "id": "d8606c6b-e605-4632-948d-d6e99e130d8f",
      "method": "workspace/didChangeWorkspaceFolders",
      "registerOptions": {}
    }
  ]
}
2023-07-10T14:18:18 [ERROR] crates/project/src/project.rs:2500: No such file or directory (os error 2)
2023-07-10T14:18:18 [INFO] open paths ["/Users/mconrad/.config/zed/settings.json"]
2023-07-10T14:18:18 [INFO] starting language server name:json-language-server, path:"/Users/mconrad/.config/zed/settings.json", id:3
2023-07-10T14:18:18 [INFO] unhandled notification $/typescriptVersion:
{
  "version": "5.1.6",
  "source": "bundled"
}

Z-2591

@MasterPtatoknerrichmckenzie @as-cii for 👀

Ahh, yes, telemetry was turned off. I turned it on and tried collaboration
again: both trying to accept and trying to initiate.
Zed: v0.94.4 (stable)
OS: macOS 11.7.8
Memory: 16 GiB
Architecture: x86_64
Mark Conrad
On Mon, Jul 17, 2023 at 10:31 AM Joseph T. Lyons @.***>
wrote:

I'm querying our panic_events with the info above and nothing exists for
that version of macOS and app version. I wonder if the user has telemetry
disabled.
—
Reply to this email directly, view it on GitHub
https://github.com/zed-industries/community/issues/1747#issuecomment-1638272294,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLFOAARSIBDG2SQIFS4YM3XQVEE7ANCNFSM6AAAAAA2EUB254
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
The information transmitted in this email is confidential and/or
privileged, and is intended only for the person(s) or entity to which it is
addressed. Any review, retransmission, dissemination or other use of, or
taking of any action in reliance upon, this information by persons or
entities other than the intended recipient is prohibited. If you received
this email in error, please contact the sender and permanently delete the
email from any computer. Nothing in this email is intended to bind Lotame,
which only operates under the terms of written agreements signed by an
authorized officer.

Thanks @markaconrad, I'm not sure what might be happening, but I'm still not seeing a panic in our system. Also, I should've mentioned here that you don't need to turn on diagnostic telemetry reporting to get us a panic report. The panics are stored in a text file in this directory ~/Library/Logs/Zed after the crash occurs, and before you reopen Zed again (reopening Zed clears out the panic files - sending them for users who have the diagnostic telemetry setting enabled).
Still not sure why we aren't seeing it on our end though...


Terminus site backup get --element=db #215

This addresses issue #215

@jcandan quick question, why not just normalize the value at the site() function of php/commands/site.php. Just curious. This looks good to me.

not sure what you mean by "normalize the value at the site() function of php/commands/site.php". could you elaborate?

@mikevanwinkle made suggested improvement. let me know your thots. thx!

This also addresses #216.

@jcandan I just meant that if you modify site.php to transform "database" into "db" then all the changes to the Environment file seem to be unnecessary.  I feel like one place of the other should be responsible for the transformation and not both and I was just curious why you picked Environment.php instead of site.php.

@mikevanwinkle I guess I was thinking it'd be best to correct the bug, since db is what the help documentation shows, and is used correctly in other places. I only added the transform for backwards compatibility (good idea btw). however, one plus with this setup is that at some point in the future you can deprecate and remove database as a valid option.

@mikevanwinkle, so, what do you think?
A) should i remove the work done in Environment.php, and we just go with a handshake in site.php to translate db to database?   or
B) do you like the idea of Environment correctly taking db as per documentation, and site.php translates database to db for backward compatibility (as is currently done in my patch)?

@mikevanwinkle so, no go, eh?

This looks good,, @jcandan. Thanks! I'll merge it into master.


Schedule a collection every 5 seconds

This makes the dashboard more responsive and usable.
<--
Contribution Checklist

Name the pull request in the form "[Issue XYZ][component] Title of the pull request", where XYZ should be replaced by the actual issue number.
Skip Issue XYZ if there is no associated github issue for this pull request.
Skip component if you are unsure about which is the best component. E.g. [docs] Fix typo in produce method.

Fill out the template below to describe the changes contributed by the pull request. That will give reviewers the context they need to do the review.

Each pull request should address only one issue, not mix up code from multiple issues.

Each commit in the pull request has a meaningful commit message

Once all items of the checklist are addressed, remove the above text and this checklist, leaving only the filled out template below.

(The sections below can be removed for hotfixes of typos)
-->
(If this PR fixes a github issue, please add Fixes #<xyz>.)
Fixes #
(or if this PR is one task of a github issue, please add Master Issue: #<xyz> to link to the master issue.)
Master Issue: #
Motivation
Explain here the context, and why you're making that change. What is the problem you're trying to solve.
Modifications
Describe the modifications you've done.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If yes was chosen, please highlight the changes

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API: (yes / no)
The schema: (yes / no / don't know)
The default values of configurations: (yes / no)
The wire protocol: (yes / no)
The rest endpoints: (yes / no)
The admin cli options: (yes / no)
Anything that affects deployment: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

In many cases (with many topic) that will cause a huge load on brokers and kill postgres as well.
Also, the rates are already pre-computed in brokers at 1min interval.
The interval is already configurable (although, ehem, it's not documented..)
docker run .... -e COLLECTION_INTERVAL=5 pulsar-dashboard..

I should have seen the COLLECTION_INTERVAL env var. Thanks.


[Bug] Enable that components can be nested inside nested basic type structures too

Description
This is bugfix made on top of https://github.com/mckinsey/vizro/pull/903.
This PR enables that model_manager can find components that are nested arbitrarily deep inside basic type structures (e.g. lists within lists) too, and not just specific attributes like (e.g. "components", "tabs", "controls", "actions", "selector"). So, this PR enables you to define any component structure, For example: my_custom_components_property: list[list[tuple[vm.VizroBaseComponent]]].
This PR does not solve nesting components inside dict type. This is known limitation emphasised with the "TODO" already, and will be solved separately.
See how scratch/app.py works from this, but doesn't work from the main branch. The best way to see the problem is to refresh the page and see if the controls are taken into account when the page loads.
Screenshot
Notice

[x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.
I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorized to submit this contribution on behalf of the original creator(s) or their licensees.
I certify that the use of this contribution as authorized by the Apache 2.0 license does not violate the intellectual property rights of anyone else.
I have not referenced individuals, products or companies in any commits, directly or indirectly.
I have not added data or restricted code in any commits, directly or indirectly.

Can we write some tests for this case ;)

I also had this on my mind, but I've decided not to do so because all these util methods are still private. We don't test private much methods because it can easily happen that their structure or API completely changes. And, model_manager will be changed again (maybe even drastically changed) when we introduce pydantic_v2 (this is also highlighted in the TODO comments).

Can we write some tests for this case ;)

I also had this on my mind, but I've decided not to do so because all these util methods are still private. We don't test private methods much because it can easily happen that their structure or API completely changes. And, model_manager will be changed again (maybe even drastically changed) when we introduce pydantic_v2 (this is also highlighted in the TODO comments).

I agree with @maxschulz-COL that we should include unit tests though. We do test private methods in other models, but for some we have deprioritized it. Ideally, anything that could be unintentionally broken should be tested. We don't need to test every possible level of nesting, but we should cover basic cases, such as the code snippet mentioned in this issue: https://github.com/mckinsey/vizro/issues/82
Is this scenario included in our tests? If not, would you mind adding it? I remember Antony sent out this issue again, but it broke after the action refactoring, and we only noticed when another user requested a similar feature right? I think optimally unit tests should have caught this earlier 😬

Due to prioritisation and capacity for the current sprint, I will write unit tests and finalise this PR next year.


Test Only: Layout for Individual Configurations Tab - Workspace

As an ECMPS user, I want a separate screen for the individual Configurations selected so that I can review and update QA & Cert data for it
Acceptance Criteria:
Given I am an ECMPS user on the Select Configurations screen
When I open a Configuration tab
Then I see that the layout of the tab matches that in the prototype provided below
Workspace screen shall have Import option
Prototype for Reference (Use screens 3, 4 of 14):
https://xd.adobe.com/view/c103c998-65ff-432a-b1f9-c586ebbceafa-9450/

Test Case: 2823 Layout for Individual Configurations Tab

Test Passed


Simplify Immutable

Closed in favour of https://github.com/cefn/lauf/pull/53 which introduces this change and aligns apps with new restrictions.


Rename to toofz Steam

Moving the entity types to toofz Leaderboards Core (Data) left this library with code only relevant to interacting with Steam APIs.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #34   +/-   ##
=======================================
  Coverage   94.03%   94.03%
=======================================
  Files          27       27
  Lines         553      553
  Branches       83       83
=======================================
  Hits          520      520
  Misses         13       13
  Partials       20       20

Impacted Files
Coverage Δ

toofz.Steam/HttpContentExtensions.cs
91.66% <ø> (ø)

toofz.Steam/LoggingHandler.cs
83.33% <ø> (ø)

toofz.Steam/ProgressReporterHttpClient.cs
90% <ø> (ø)

toofz.Steam/HttpRequestMessageExtensions.cs
100% <ø> (ø)

...bApi/ISteamRemoteStorage/UgcFileDetailsEnvelope.cs
100% <ø> (ø)

toofz.Steam/ExponentialBackoff.cs
100% <ø> (ø)

...Steam/WebApi/ISteamRemoteStorage/UgcFileDetails.cs
100% <ø> (ø)

...fz.Steam/CommunityData/SteamCommunityDataClient.cs
87.71% <ø> (ø)

toofz.Steam/ClientApi/SteamClientApiException.cs
100% <ø> (ø)

toofz.Steam/WebApi/ISteamUser/PlayerSummary.cs
100% <ø> (ø)

... and 17 more

Continue to review full report at Codecov.

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


Override System.Text.Json version to work around false positive CVE-2024-30105

Cherrypick of https://github.com/microsoft/WindowsAppSDK/pull/4554

/azp run

/azp run

/azp run


Removed partially custom BlockId implementation

Usage related changes
Development related changes
Implemented as_ref method for wrapper BlockId(starknet-rs::BlockId) to return reference to its inner type.
Removed usage if any of our custom BlockId implementation and left only its custom deserialisation.
Refactored methods to receive &BlockId as an argument, instead of BlockId, to avoid copying of 32 bytes. Copying is done because starknet-rs BlockId implements the Copy trait.
Checklist:

[ ] Applied formatting - ./scripts/format.sh
[ ] No linter errors - ./scripts/clippy_check.sh
[x] No unused dependencies - ./scripts/check_unused_deps.sh
[ ] Performed code self-review
[ ] Rebased to the last commit of the target branch (or merged it into my branch)
[x] Documented the changes
[x] Linked the issues which this PR resolves
[x] Checked the TODO section in README.md if this PR resolves it
[ ] Updated the tests
[ ] All tests are passing - cargo test

Looks good, I assume the remaining checkboxes could be checked. Also, even though the PR title contains the word "partially", we don't plan to make more changes related to this, right?

No, no more changes related to BlockId


ISSUE: can only concatenate str

Describe the bug
can only concatenate str. Error from logs below.
Environment (please complete the following information):

OS: HassOS
Type of device: RPi4 8GB
Home Assistant version: 2021.2.0b0
Hassio
Component version: 0.3.0-b38

Logs
Logger: custom_components.mail_and_packages
Source: custom_components/mail_and_packages/helpers.py:459
Integration: Mail and Packages (documentation, issues)
First occurred: 12:26:44 PM (1 occurrences)
Last logged: 12:26:44 PM

Unexpected error fetching Mail and Packages (imap.gmail.com) data: can only concatenate str (not "NoneType") to str
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 149, in async_refresh
    self.data = await self._async_update_data()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 137, in _async_update_data
    return await self.update_method()
  File "/config/custom_components/mail_and_packages/__init__.py", line 64, in async_update_data
    return await hass.async_add_executor_job(process_emails, hass, config)
  File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/mail_and_packages/helpers.py", line 104, in process_emails
    fetch(hass, config, account, data, sensor)
  File "/config/custom_components/mail_and_packages/helpers.py", line 185, in fetch
    count[sensor] = get_mails(
  File "/config/custom_components/mail_and_packages/helpers.py", line 459, in get_mails
    filecheck = os.path.isfile(image_output_path + image_name)
TypeError: can only concatenate str (not "NoneType") to str

Logger: homeassistant.components.sensor
Source: custom_components/mail_and_packages/sensor.py:129
Integration: Sensor (documentation, issues)
First occurred: 12:26:44 PM (1 occurrences)
Last logged: 12:26:44 PM

Error while setting up mail_and_packages platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 198, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/mail_and_packages/sensor.py", line 27, in async_setup_entry
    sensors.append(ImagePathSensors(hass, entry, variable, coordinator, unique_id))
  File "/config/custom_components/mail_and_packages/sensor.py", line 129, in __init__
    self._image = self.coordinator.data[const.ATTR_IMAGE_NAME]
TypeError: 'NoneType' object is not subscriptable

Incoming PR to fix this.

Incoming PR to fix this.

With b39
Logger: custom_components.mail_and_packages
Source: custom_components/mail_and_packages/helpers.py:175
Integration: Mail and Packages (documentation, issues)
First occurred: 12:54:51 PM (1 occurrences)
Last logged: 12:54:51 PM

Unexpected error fetching Mail and Packages (imap.gmail.com) data: 'image_name'
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 149, in async_refresh
    self.data = await self._async_update_data()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 137, in _async_update_data
    return await self.update_method()
  File "/config/custom_components/mail_and_packages/__init__.py", line 64, in async_update_data
    return await hass.async_add_executor_job(process_emails, hass, config)
  File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/mail_and_packages/helpers.py", line 105, in process_emails
    fetch(hass, config, account, data, sensor)
  File "/config/custom_components/mail_and_packages/helpers.py", line 175, in fetch
    image_name = data[const.ATTR_IMAGE_NAME]
KeyError: 'image_name'

Logger: homeassistant.components.sensor
Source: custom_components/mail_and_packages/sensor.py:129
Integration: Sensor (documentation, issues)
First occurred: 12:54:53 PM (1 occurrences)
Last logged: 12:54:53 PM

Error while setting up mail_and_packages platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 198, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/mail_and_packages/sensor.py", line 27, in async_setup_entry
    sensors.append(ImagePathSensors(hass, entry, variable, coordinator, unique_id))
  File "/config/custom_components/mail_and_packages/sensor.py", line 129, in __init__
    self._image = self.coordinator.data[const.ATTR_IMAGE_NAME]
TypeError: 'NoneType' object is not subscriptable

With b39
Logger: custom_components.mail_and_packages
Source: custom_components/mail_and_packages/helpers.py:175
Integration: Mail and Packages (documentation, issues)
First occurred: 12:54:51 PM (1 occurrences)
Last logged: 12:54:51 PM

Unexpected error fetching Mail and Packages (imap.gmail.com) data: 'image_name'
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 149, in async_refresh
    self.data = await self._async_update_data()
  File "/usr/src/homeassistant/homeassistant/helpers/update_coordinator.py", line 137, in _async_update_data
    return await self.update_method()
  File "/config/custom_components/mail_and_packages/__init__.py", line 64, in async_update_data
    return await hass.async_add_executor_job(process_emails, hass, config)
  File "/usr/local/lib/python3.8/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/mail_and_packages/helpers.py", line 105, in process_emails
    fetch(hass, config, account, data, sensor)
  File "/config/custom_components/mail_and_packages/helpers.py", line 175, in fetch
    image_name = data[const.ATTR_IMAGE_NAME]
KeyError: 'image_name'

Logger: homeassistant.components.sensor
Source: custom_components/mail_and_packages/sensor.py:129
Integration: Sensor (documentation, issues)
First occurred: 12:54:53 PM (1 occurrences)
Last logged: 12:54:53 PM

Error while setting up mail_and_packages platform for sensor
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 198, in _async_setup_platform
    await asyncio.shield(task)
  File "/config/custom_components/mail_and_packages/sensor.py", line 27, in async_setup_entry
    sensors.append(ImagePathSensors(hass, entry, variable, coordinator, unique_id))
  File "/config/custom_components/mail_and_packages/sensor.py", line 129, in __init__
    self._image = self.coordinator.data[const.ATTR_IMAGE_NAME]
TypeError: 'NoneType' object is not subscriptable


change loading behavior of trackers.

[x] allows loading broken users like spf
[x] fix offsets in search input (this overloaded input itself which breaks the alignment)
[x] fix square behind avatars

@keybase/react-hackers this is ready for review


md-tooltip takes considerable time to disappear after mouse leave event,

BUG
What is the expected behavior? : tooltip text must disappear when mouse leave
What is the current behavior? : it takes considerable time to disappear?
Which versions of Angular, Material, OS, browsers are affected? :Material Alpha10

@da45 this is not a bug, it seems to be on purpose. Perhaps there is a plan to customize this timing globally.

The delay (1.5 seconds) actually conforms to the material design spec:

On lift, display the tooltip for 1.5 seconds.

However I agree it seems too long and it would be nice if it was exposed as a configuration option.

Working as intended now per the spec. It will also go away if you interact with anything on the page.
However, we should make this configurable.

I opened a proper issue here.

Closing in favor of #1808


[FR] Translation for documentation.md

@julienXX Please could you review this one? :)

@tchoutri je t'ai laissé quelques suggestions d'améliorations.

@julienXX vu ! je m'en occupe ce soir.

@tchoutri / @julienXX is this ready to go?

I'm currently very sick and mostly unable to leave my bed.
You're going to have to rely on @julienXX for this one, sorry :/
On 04/12/2016 05:18 PM, Sean Callan wrote:

@tchoutri https://github.com/tchoutri / @julienXX
https://github.com/julienXX is this ready to go?

Oh no!  I'm sorry @tchoutri, I hope you start to feel better soon.

That's pretty unlikely, but thanks for your support ;-)
On 04/12/2016 05:23 PM, Sean Callan wrote:

Oh no! I'm sorry @tchoutri https://github.com/tchoutri, I hope you
start to feel better soon.

@tchoutri I hope you feel better.
I noticed that there are some accented words on the Headers pointed by the summary's anchors.
But Jekyll skips accented letters when naming the headers.
So, to an anchor defined like https://github.com/doomspork/elixir-school/pull/244/files#diff-a4526323894c9b3c44ffbfc0dea56a46R14
The header need to have a custom name like
### La documentation linéaire

Or the summary's anchor will not work.

We have implemented a new feature that automatically generates the table of contents: #356.
This may help you avoid copypaste, typos and broken anchors. In order to use it you can replace the manually written TOC with {% include toc.html %}.

@tchoutri can we merge this?

@tchoutri and @julienXX — what should we do with this PR?

@doomspork better close it,  the repository that contained it does no longer exist.
I've had some troubles when I had to use some advanced features of Git for such a simple technical task.

@tchoutri I'd be happy to go over rebasing et al with you if you'd like 😀


Better unknown errors

Two changes that should reduce/eliminate the chances of a http response with an error but no meaningful details.
These changes should fix #19.
@VladVega @go-oleg @davidvgalbraith

updated, can you look again?

+1


Select Same

Found an inaccuracy in the Select > Same menu. minVersion: 26 - but some of the items in this menu are available in earlier versions of Illustrator.
I need time to compare which menu items exist in earlier versions of Illustrator and which were added after CC 2021-2022. For now, I'm opening an issue so that we both don't forget.

I have attached a list of the commands. In short, only Select > Same > Font / Text has been added in v.26, the previous commands are available with CS6, i.e. they do not need minVersion.
Select.csv

Oh, this is great! I have it updated and will push a new version release shortly. Thanks again for your help on this!


Navbar submenu does not work

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

[ *] I am running the latest version
[* ] I checked the documentation and found no answer
[* ] I checked to make sure that this issue has not already been filed
[ *] I'm reporting the issue to the correct repository (for multi-repository projects)

Expected Behavior
Want to have a multi level drop down in the nabar.
Current Behavior
I have used the class submenu as mentioned in bootstrap 4 documentation. But no luck!

Hello @praveenagunasekera
Thank you for using our products, I will ask you to send us your orderID to make some verification.
All the best,
Stefan

Having this issue too.

Hello guys,
Thank you for your interest in working with our products, please let us know about what product are you using because the multi-level dropdown is a PRO feature.
Best,
Stefan

Hi Stefan,
Thankyou for the email. I am developing a website using material-dashboard for the use of inventory management for my personal business. How can I be able to use the multi-level dropdown as I’m still unable to use it.
Regards,
Praveena.
Sent from Mailhttps://go.microsoft.com/fwlink/?LinkId=550986 for Windows 10
From<EMAIL_ADDRESS>Sent: Tuesday, December 17, 2019 10:14 PM
To<EMAIL_ADDRESS>Cc<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [creativetimofficial/material-dashboard] Navbar submenu does not work (#104)
Hello guys,
Thank you for your interest in working with our products, please let us know about what product are you using because the multi-level dropdown is a PRO feature.
Best,
Stefan
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/creativetimofficial/material-dashboard/issues/104?email_source=notifications&email_token=AIRTVCMWKSCWQELJ6JO23ULQZD6WJA5CNFSM4ISVZE5KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHDFOWI#issuecomment-566646617, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AIRTVCIT6USBLTNFV3WIF3LQZD6WJANCNFSM4ISVZE5A.

You will be able to use multi-level dropdown by using this product: https://www.creative-tim.com/product/material-dashboard-pro (the PRO version)
Best,
Stefan


Fix type resolving issues related to constants

Purpose

$title. Related discussion https://github.com/ballerina-platform/ballerina-lang/discussions/35745.

Fixes #33890
Fixes #32804
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

@SandaruJayawardana should we keep this open?

@SandaruJayawardana should we keep this open?

Yeah. I will reopen this PR once I start working on it.

Since we are revamping the SymbolEnter (https://github.com/ballerina-platform/ballerina-lang/issues/36538), I will send a separate PR instead of this.


Jobs should be recovered from mongodb

I think there's a couple things at play here, but basically if I loose my Jenkins master but I bring up a new one and connect DotCi to the MongoDB where it was storing all its info, I think that database has enough info to automatically recreate the jobs. I believe this because currently if you try to add back a job for a private repo that is in the DB then you get an exception complaining about that job already existing. At that point, its reconfigured the org and you can see all your jobs again. However the one that you just added now has multiple deploy keys for the private repo, which causes it to be marked as not private, and thus the job can't pull any code.
Repo steps:

Setup docker build:

docker-compose build
docker-compose run --rm plugin npm run build
docker-compose up plugin

Visit http://localhost:8080/jenkins and configure DotCi with a test GitHub application
Add a job for a private repository (don't really care if it works or not, but a green one would be better)
Stop your docker-compose up plugin command
Remove ./work folder, which should clear out all the saved state about for Jenkins
Start up jenkins again with docker-compose up plugin
Notice that there are no jobs present; if you want to see the DB state you can docker-compose run --rm mongo mongo --host mongo and then

use dotci
db.deploy_keys.find({}, {repo_url: 1})

That will show you there is an existing key for the job
8. Try to add the same job from step 3
9. Notice that you get an exception, which is bad; should detect this and handle gracefully.
10. Go back to root and that repo's organization should be there now and you can navigate to the job; i.e. it was added despite the exception.
11. Go to the job and try a build; it should fail with a public key denied reason.
12. Try step 7 again, you'll notice there are now two keys; if you can figure out which one is right you can delete the other one (or modify the repo_url so its not found) and then the build will work again.
I think this illustrates a few issues:

On step 6/7 DotCi should be able to tell that it has jobs configured and automatically make all those jobs be present again; that would actually obviate all the issues in the following steps, although I think there's a benefit to making them robust with the rest of these items.
On step 8/9 this scenario should be detected and handled gracefully; i.e. look for the job earlier (hopefully before adding a second deploy key) and if it exists do the work to recreate the file structure to see the jobs and then redirect them to the job.
On step 11, this code is kinda the issue: src/main/java/com/groupon/jenkins/github/services/GithubDeployKeyRepository.java although I also think this code src/main/java/com/groupon/jenkins/github/services/GithubRepositoryService.java is either not working or not called correctly too. Eitherway before adding deploy keys we have a handle to the Github API, which not go through the existing keys and see if any work first; if they don't then delete them, if they do then use that one. Basically this area can be more robust.
This is more of a suggestion and its really specific to debugging this kinda, but I think since the keys are encrypted in the DB it'd be nice to calculate the fingerprint and store that so that one could correlate them to what's in Github easily.

hi @ppg, Let me research this over the weekend.


evaljs with jquery causes Splash to crash

I have following innocently looking Lua script using evaljs:
function main(splash)
    splash:go("http://www.ica.se/handla/ica-nara-ovrells-id_01739/")
    splash:set_viewport_full()
    splash:wait(1)
    splash:evaljs("$('#search').val('cola');")
    splash:evaljs("e = $.Event('keypress'); e.which = 13; $('#search').trigger(e)")
    splash:wait(1)
    return splash:jpeg()
end

this crashes Splash every time I run it. I found warning in docs about eval js

Don’t use splash:evaljs when the result is not needed - it is inefficient and could lead to problems; use splash:runjs instead. For example, the following innocent-looking code (using jQuery) may fail A gotcha is that to allow chaining jQuery $ function returns a huge object, splash:evaljs tries to serialize it and convert to Lua. It is a waste of resources, and it could trigger internal protection measures; splash:runjs doesn’t have this problem.

it's nice we warn users about evaljs but I think we should do something about it, it seems bad that evaljs can actually crash your splash server. What internal protection measure do we have now? Is there a way to improve those protection measures to avoid problem like this?

There is protection against converting deep Python objects to Lua (https://github.com/scrapinghub/splash/blob/master/splash/qtutils.py#L190). Previously (in qt4?) it was enough to make Splash raise an error on evaljs instead of crashing.


Theoretical Problems about SO(3) Fourier Transformation

Hi,
Thanks for your great work! Your repo really helps me a lot! However, i am encountering some theoretical problems about the derivative of SO(3) fourier transformation when i tries to write the back propagation formula related to SO(3) fourier transformation. To be more specified, if i have

In your code, you write like this:
class SO3_fft_real(torch.autograd.Function):
    @staticmethod
    def forward(ctx, x, b_out=None):  # pylint: disable=W
        ctx.b_out = b_out
        ctx.b_in = x.size(-1) // 2
        return so3_rfft(x, b_out=ctx.b_out)

    @staticmethod
    def backward(self, grad_output):  # pylint: disable=W
        # ifft of grad_output is not necessarily real, therefore we cannot use rifft
        return so3_ifft(grad_output, for_grad=True, b_out=self.b_in)[..., 0], None

It seems very intuitive that the partial derivative of SO(3) fft is SO(3) ifft, but i still wonders how you can get this. Can you help to illustrate this in details or is there any paper deriving this formula or discussing about this?
Thanks a lot for your help!!

Although it's implemented in a clever way, the FFT is ultimately just a linear transformation, that is furthermore orthogonal (in the real case) / unitary (complex). So you could write it as a matrix multiplication y = F x where x is the spatial signal, y is the spectral signal, and F is the Fourier matrix. The Jacobian dy/dx of a matrix-vector product function is just the transpose / conjugate transpose of F. Since F is orthogonal/unitary, this equals the ifft.

i understand what you mean, thank you so much for your help!!


RP-11 Add authorization

Add authorization to find out the cost of products.

Fixed by https://github.com/zwieback/relef-parser/pull/32


Fixed a typo in drdlat

code was calling drdsph in the wrapper function
also bumped version to 1.1.2.dev0

Coverage remained the same at 97.522% when pulling 07375e7358630a99a40cef38399d10cc3602cc69 on fix_drdlat into 5e066109aac237133d6335322a951de6cad80ae5 on master.


Merge Alice ID -> Secret into event sourcing WIP

This continues the simple renaming in #6767 with changes necessary to keep this unique identifier secret in the event sourcing code.
WalletWasabi.Tests project doesn't build in upstream so only partial attempts at fixing it were made, but the client and backend have been tested and appear to be working.
Input registration still contains the secret, since the round aggregate keeps track of it it in the InputState, but clients will not see those events. On the client, each InputState will have the default Guid as its secret in the property which is otherwise unused.

@molnard @lontivero @bitcoin-eagle


Fix admin bug report labels

Description
Follow up to #454. admin_bug_report.md label list is invalid at the moment and doesn't show up in the issue list either as seen in the below image:

big thanks @EliiseS 🙏


Push Docker image to Docker Hub

As requested by @martindsouza  in issue #1 .

Built image successfully on Docker Hub.


Board local REPL workflow

Currently, npm run dev in seeds/breadboard-web provides a pretty decent developer experience: the board runs in a browser with a simple UI, which gets reloaded on save (thanks Vite!)
Imagine that this was part of breadboard CLI? The flow would be:

Set up a blank repo for a board
Install Breadboard CLI: npm i @google-labs/breadboard-cli
Run it in REPL mode npx breadboard dev
A link is printed for a local server
Local server shows a simple UI like above
Any edits to the board in the repo cause the board to be built and UI restarted

Declaring victory 🚢


Updating to match new releases

Updating to match new releases

@stgeorgi : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @stgeorgi - Will you resubmit this change in the private repo? A small change like this probably will auto-merge in the private repo, which is a valuable efficiency. Thanks.
To edit an article in the private repo from within the documentation, after you select Edit on an article page,  add “-pr” to the repo name in the URL.

Also, if you're interested, SpineEdit is a Chrome extension that opens the edit option in the private repo, if there is one and if you have access to it.

https://github.com/MicrosoftDocs/azure-docs/pull/59883


Extend registry mirror

Currently, registry mirror works just for https://registry-1.docker.io,shall we extend it's range to cover other distribution hub, and extend option--registry-mirror as format mirror hub addr@main hub addr. How about your opinion?

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO


This repository has been disabled. Access to this repository has been disabled by GitHub staff

I am getting this page when I try to access Latest updates page in order to install any app:
This repository has been disabled.
Access to this repository has been disabled by GitHub staff. Contact support to restore access to this repository.

Yeah, it's a pity the guy went MIA...

Who are you referring to? @w0lfschild still seems to be active.
I don't see that error over here btw, perhaps it was temporary? Or do you have a link to it?

app_updates seems to be back.

app_updates seems to be back online.


Delete Optimizable properties that have dofs and state in the name

The above mentioned properties are synonymous to x and felt not needed by other team members. Delete them to provide a simple interface.

Addressed in PR #67


Fix I18n.interpolate when ActiveSupport::SafeBuffer passed as argument

There could be a problem with ActiveSupport::SafeBuffer and I18n interpolate.
I've implemented a part of SafeBuffer that cause the issue in the test case.
I18n.interpolate("%{a}".html_safe, :a => 1)

By some reason $1 and $2 is not set appropriately and we have to use gsub match data instead.

What is the use case of passing a safe buffer here? Would anyone realistically use this in their code?

@radar I did. The thing is - you never know if buffer you are using is safe or not.
The example of that would be:
<%= I18n.interpolate("Get your €%{amount} off".html_safe, amount: 10) %>

In my case, users are editing localizations themselves and basic HTML is allowed (like in markdown).

Ok, that makes sense to me. Thanks @bogdan.


fix: Ensure default network policies are created

The previous behavior was to add some default ingresses and egresses to every NetworkPolicy other than default-deny. The flaw with this behavior is that if no NetworkPolicy matches the pod, the pod will not have these default ingresses and egresses. This instead creates separate NetworkPolicy objects for defaults.

This change is

Depends on #1767


Feature : Add ONNX conversion script

Hi @jozhang97
I made ONNX conversion script for this ResNet50 DETA model
There are two slight modification in original code.

onnx input process
modify boolean type of input to float in cumsum operation

If you have time, please review and merge.
Also if you need further modification feel free to ask.

FYI, to convert this model into tensorRT, it requires different conversion script (e.g. MultiScaleDeformableAttention)

Hi @SangbumChoi , I tried to convert onnx to tensorrt engine. But there is error:
[E] 2: [myelinBuilderUtils.cpp::getMyelinSupportType::1270] Error Code 2: Internal Error (ForeignNode does not support data-dependent shape for now.)
[!] Invalid Engine. Please ensure the engine was built correctly
[E] FAILED | Runtime: 22.843s | Command: /home/osmagic/anaconda3/envs/pytorch/bin/polygraphy run --trt weights/out_nms_0527_sim_san.onnx

Do you know why there's this error and how to solve it?

@demuxin AFAIK, you need some custom kernel (deformable attention) to convert this model in to tensorRT. In my experience I have succeed to convert ONNX but not tensorRT.
I would happy to work and collaborate on converting tensorrt engine. Do you have any experience of writing CUDA programming?

@demuxin AFAIK, you need some custom kernel (deformable attention) to convert this model in to tensorRT. In my experience I have succeed to convert ONNX but not tensorRT.
I would happy to work and collaborate on converting tensorrt engine. Do you have any experience of writing CUDA programming?

Yes, I have accelerated many algorithms using CUDA in C++. However, I am not familiar with models related to transformers.

@demuxin Will you like to share your email and Slack message to discuss about this?

@demuxin Will you like to share your email and Slack message to discuss about this?

Yes, my email address is<EMAIL_ADDRESS>

@SangbumChoi Hi,Swin-L  DETA model can be conver to onnx?  I use the code conver Swin-L  DETA to onnx :
deta.pt2onnx(img_size=(1440,832),weights='/mnt/data1/download_new/DETA-master/exps/public/deta_swin_ft_2024.4.3/best.pt')

def pt2onnx (self,weights,img_size,batch_size=1,device='cuda:0',export_nms=False,simplify=True):
       model=self.model
       print(img_size)
       img = torch.zeros(batch_size, 3, *img_size).to(device)
       try:
           import onnx

           print(f' starting export with onnx {onnx.__version__}...')
           f = weights.replace('.pt', '.onnx')  # filename
           torch.onnx.export(model, img, f, verbose=False, opset_version=11, input_names=['images'], output_names=['output'],
                               dynamic_axes={'images': {0: 'batch', 2: 'height', 3: 'width'},  # size(1,3,640,640)
                                               'output': {0: 'batch', 2: 'y', 3: 'x'}} )
           # Checks
           model_onnx = onnx.load(f)  # load onnx model
           onnx.checker.check_model(model_onnx)  # check onnx model
           # print(onnx.helper.printable_graph(model_onnx.graph))  # print

           # Simplify
           if simplify:
               try:
                   # check_requirements(['onnx-simplifier'])
                   import onnxsim

                   print(f'simplifying with onnx-simplifier {onnxsim.__version__}...')
                   model_onnx, check = onnxsim.simplify(model_onnx,
                                                       input_shapes={'images': list(img.shape)} )
                   assert check, 'assert check failed'
                   onnx.save(model_onnx, f)
               except Exception as e:
                   print(f' simplifier failure: {e}')
           # print(f'{prefix} export success, saved as {f} ({file_size(f):.1f} MB)')
       except Exception as e:
           print(f' export failure: {e}')

It return :
/mnt/data1/download_new/DETA-master/models/deformable_detr.py:243: TracerWarning: Iterating over a tensor might cause the trace to be incorrect. Passing a tensor of different shape won't change the number of iterations executed (and might lead to errors or silently give incorrect results).
  for a, b in zip(outputs_class[:-1], outputs_coord[:-1])]
 export failure: 0INTERNAL ASSERT FAILED at "../torch/csrc/jit/ir/alias_analysis.cpp":607, please report a bug to PyTorch. We don't have an op for aten::fill_ but it isn't a special case.  Argument types: Tensor, bool,

Candidates:
	aten::fill_.Scalar(Tensor(a!) self, Scalar value) -> (Tensor(a!))
	aten::fill_.Tensor(Tensor(a!) self, Tensor value) -> (Tensor(a!))

Do you have any suggestions for this mistake? thank you！！

@xinlin-xiao Since Swin-L can be converted into ONNX I think overall the answer might be yes

Hi, thanks for the great work! I'm also working on onnx export but with Huggingface optimum, but I encountered some issues when I try to export with optimum. As I can see from your PR, it seems like you added NestedTensor function for onnx export. May I know the idea behind?


[Feature] Support MySQL TIME types when using HiveCatalog in CDC

Search before asking

[X] I searched in the issues and found nothing similar.

Motivation
Currently, Hive doesn't support TIME type.  So if MySQL table has TIME field, and set Paimon catalog to HiveCatalog, it will throw error.
Solution
No response
Anything else?
No response
Are you willing to submit a PR?

[X] I'm willing to submit a PR!

#1813


Appler via DOSBox/Retroarch

So I've been able to load Appler under RA/DOSBox-pure, but there are some odd things about arrow key or Joystick d-pad recognition after a DSK file has loaded.
If I go into the Disk Manager, such as with F3, the arrow keys or d-pad directional movements are recognized just fine.  Likewise, going into Keyboard mapping with F4, if I press down on the D-pad, the application recognizes that as a DOWN action, and prompts to re-assign the Down action.  But when I load a DSK object, the actual game rom is somehow not recognizing the kbd/joy events the way the game documentation says it should (arrow keys for example on the keyboard should work during the game, but they don't).
Do you have any idea what might be contributing to this issue, where the arrow keys are recognized via the menus, but not when running a game?   I will try using just normal dosbox under Windows to see if I encounter the same behavior, although I anticipate that it will be a similar experience.  I've not really encountered something similar with other RA+Dosbox apps so far.
Thank you for any additional information you can provide!

OK, I couldn't get the arrow keys to work properly in regular dosbox either (when DSK is loaded).  So even though the arrow keys aren't being recognized, but should be supported by the game according to the manual, there are alternative keys that can be used A, Z, L, semicolon mentioned in the manual, instead of using arrow keys.  I think this will work.

Yeah, just tried to read the joystick with a for loop in basic using PDL(0), it doesn't seem to work.

To workaround, I used the dbox core pad mapper under retroarch, and mapped joystick actions to perform the alternate keyboard keys, but was surprised the keyboard arrow keys aren't being recognized when the game is active with regular or RA dosbox. The arrow keys work in the appler menus though.

I can't find any DSK file that doesn't seem to have the HGR problem with Appler, are you aware of some specific ones where this is not an issue?


failed to connect to mysql database after I upgrade the mariadbclient library inside the msys2

Hi, I have recently update the msys2 to the latest version, and soon I found that soci failed to connect to the mysql library.
Here is the details:
soci+mariadbclient 3.3.8-1 failed to connect to a mysql database, while it works with 3.1.13(replace the libmariadb.dll with old one works) · Issue #19861 · msys2/MINGW-packages
Maybe, this is caused that the mariadbclient 3.3.8 has the different interface with the mariadbclient 3.1.13.
Any idea how to fix this issue? Thanks.

This looks like mariadb installation problem, i.e. I'm pretty sure that just using its C API directly would fail too. IMO it's not a SOCI problem at all, so closing.

Hi, @vadz , with the help of msys2 package devs, this issue is located and now fixed in the msys2, See that issue (soci+mariadbclient 3.3.8-1 failed to connect to a mysql database, while it works with 3.1.13(replace the libmariadb.dll with old one works) · Issue #19861 · msys2/MINGW-packages) for more details.
Sorry about the noise.


fix(azure-swa): azure SWA request handling and generation of function.json

This fixes two issues for Azure Static Web Apps adaptor:

the URL was not taken from the header x-ms-original-url
the function.json was missing the scriptFile entry

Fixes #2867
What is it?

[ ] Feature / enhancement
[x] Bug
[ ] Docs / tests

Description
See above
Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

Thanks for fixing this up. I'd like to add azure to our server e2e testing, which you can see is on the TODO list: https://github.com/BuilderIO/qwik-city-e2e


Added the functions

#196

I made the changes mentioned

Hey, there are some conflicts in the documentation. md file, do take a look into it.

I have tried resolving the conflicts through the last commits. I guess I have done it right.

I also tried resolving the conflicts again

@nikhil25803 can you please review this?

Looks good @GayathriManeksha
Do not forget to

Share your work on socials and tag us and Clueless 🫂
Give this repository a star ✨
Follow Clueless Community on GitHub ✅


Tap element by image doesn't automatically wait for the image to be present

Implemented and respects the standard element identification timeout.


Deploy python-sdk on astro

Description
closes: #913
What is the current behavior?

What is the new behavior?

Does this introduce a breaking change?
Checklist

[ ] Created tests which fail without the change (if possible)
[ ] Extended the README / documentation, if necessary

@kaxil @tatiana @pankajastro is there any security concerns with having astro deployment as a part of CI/CD? Perhaps we should only do this with release branches instead of with  the main branch? WDYT?

@kaxil @tatiana @pankajastro is there any security concerns with having astro deployment as a part of CI/CD? Perhaps we should only do this with release branches instead of with the main branch? WDYT?

Hi @dimberman We will run this only on the main branch as a nightly job and not on each merge. Also, CI/CD will take care of the deployment of our example DAG on astro cloud only and we have a master DAG which will take care of running our example DAG at the scheduled time. We do similar things in the provider. Please do let me know if you think there can be any specific potential security issues.

https://github.com/astronomer/astro-sdk/issues/1737

Let's please try to use https://github.com/astronomer/deploy-action
cc @tatiana

Created a separate ticket for this: https://github.com/astronomer/astro-sdk/issues/1742


cython.final not allowed on class

The doc says:

@cython.final terminates the inheritance chain by preventing a type from being used as a base class, or a method from being overridden in subtypes. This enables certain optimisations such as inlined method calls.

but I get the following error:
@cython.cclass
@cython.final
class WorkerState(object):
^
------------------------------------------------------------

distributed/schedulerstate.py:57:0: The final compiler directive is not allowed in class scope

And then when I want to add @cython.final to a method I get:

    @cython.final
    @cython.ccall
    @cython.returns(cython.Py_ssize_t)
    def get_nbytes(self):
   ^
------------------------------------------------------------

distributed/schedulerstate.py:471:4: Only final types can have final Python (def/cpdef) methods

Thanks, yes, that should be allowed. Can't test right now, but do you get the same error when you apply the decorators in reversed order?

do you get the same error when you apply the decorators in reversed order?

Yes.

Note that I only enabled the case where @cclass is below @final, i.e. you first have to convert the Python class to a cclass, and then mark that as final.


Import/Export window and build issues

Hi, and thanks for this wonderful tool !
I was wondering, is there a way to export or import window ?
The idea would be to import a window from a interface.xdat file (interface A) into another interface.xdat file (interface B).
Also it looks like some of the maven repositoories are not available anymore :

The following repositories used in your gradle projects were not indexed yet: https://raw.githubusercontent.com/acmi/Serializer/mvn-repo https://raw.githubusercontent.com/acmi/L2unreal/mvn-repo https://repo1.maven.org/maven2 https://raw.githubusercontent.com/acmi/L2crypt/mvn-repo https://raw.githubusercontent.com/acmi/L2io/mvn-repo  If you want to use dependency completion for these repositories artifacts, Open Repositories List, select required repositories and press "Update" button

Also, I'm having the following build issues :

C:\Users\GE60\IdeaProjects\xdat_editor>gradlew build -x test
Starting a Gradle Daemon (subsequent builds will be faster)
:commons:io:compileJava FAILED
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':commons:io:compileJava'.

Could not find tools.jar. Please check that C:\Program Files (x86)\Java\jre1.8.0_121 contains a valid JDK installation.

Do we need to have a specific Java version installed  ?
gradlew run gives the same error.
Finally, I tried to run the project anyway (I wasn't expecting much but if the output can help), but this gives me a bunch of errors related to java.io.InputStream. It looks like any InputStream method can't be found. (Missing dependency ?)
Any help would be appreciated.
Regards.

Hello.
Sorry, I have no time to develop new features.

Also it looks like some of the maven repositoories are not available anymore

This is not a regular maven repository, so it's OK that it couldn't be indexed.

Also, I'm having the following build issues

It seems that you have JRE but not JDK in your PATH, so Gradle couldn't build project. You can create gradle.properties file in xdat_editor folder with following content:
org.gradle.java.home=/path_to_jdk_directory

errors related to java.io.InputStream

If you are using Intellij Idea, you can delegate build action to Gradle to resolve this issue.

Thanks for your quick reply ! It works fine now.
If you got a few more time, could you help me with something ?
I'm trying to modify the IOEntity xdat object.
Basically I want to access the windows list, and replace some elements of this list with elements coming from the list of another IOEntity importedXdat
But I can't manage to access the IOEntity  properties (I only saw them in debug mode). This is probably because IOEntity is just an interface, and I need to use the XDAT class of the proper version (ct26 in my case).
But I can't manage to import the XDAT class. I saw the class is in a .groovy file. How can I import it in order to be able to use it in the Controller ?
Or how can I modify the xdat properties somehow if that's not the way to go ?
Regards,


why length can't be a number?

https://github.com/freeall/progress-stream/blob/1beb155de00a5ee77feb7f422e56e3bec0bcd167/index.js#L67

Yeah this is a bug, but I might be able to explain it. I removed this line and suddenly the progress started tracking properly based on Content Length.
I think the length was expected to be a string due to it coming from Content-Length as a response header. Later in the code I spot a lot of parseInt's that are useless with the progress being a number (maybe only now after NodeJS revisions?).


WhenIStartAsClone block

The CreateClone block is almost useless without it. I would love for this block to be implemented.

Thanks.


Parsedown never generates  break tags / empty lines

Hi,
I tried to generate empty lines both in your demo and in a test page on my server with the same result. Parsedown never generates
 tags.
I tried to follow the markdown syntax which says:

When you do want to insert a break tag using Markdown, you end a line with two or more spaces, then type return.

but it seems Parsedown just ignores the spaces at the end of a line.
For example testing this text I get this parsed output in your demo whereas I would expect at least a break tag between the second and the third lines.

Whitespace before and after a paragraph is ignored. But inside of a paragraph you can add a
:
## par 1
This is a line.
And this is after the break.

parsed as

par 1

This is a line.

And this is after the break.

by the parsedown demo.

Yes I know, but that's not what I am looking for.
Basically I want to generate an empty line and it seems there's no way to get that with Parsedown.

well, you can just insert a
 in markdown itself:
this is a line

will result in:

this is a line

Try:
Foo.

Bar.

@cebe yes, that's a possible workaround, but it doesn't work when HTML is escaped.
@tovic that works too, but it's just another workaround.
It would be nice if Parsedown could follow the markdown syntax for breaks.

It does follow the markdown syntax for breaks. breaks are allowed inside of paragraphs. between paragraphs no breaks are allowed in markdown, this is consistent with all markdown parsers: http://johnmacfarlane.net/babelmark2/?normalize=1&text=Line+1++
Line+2++

new+par

between paragraphs no breaks are allowed in markdown

I can't find that statement in the paragraphs and line breaks section of the markdown syntax.

It's not a very big change, so if you want the functionality here's how to do it:
@@ -57,6 +57,15 @@ function setBreaksEnabled($breaksEnabled)

     protected $breaksEnabled;

+    function setLiteralBreaks($literalBreaks)
+    {
+        $this->literalBreaks = $literalBreaks;
+
+        return $this;
+    }
+
+    protected $literalBreaks;
+
     function setMarkupEscaped($markupEscaped)
     {
         $this->markupEscaped = $markupEscaped;
@@ -121,7 +130,7 @@ protected function lines(array $lines)

         foreach ($lines as $line)
         {
-            if (chop($line) === '')
+            if ( ! $this->literalBreaks and chop($line) === '')
             {
                 if (isset($CurrentBlock))
                 {
@@ -184,7 +193,15 @@ protected function lines(array $lines)

             # ~

-            $marker = $text[0];
+            if (isset($text[0]))
+            {
+                $marker = $text[0];
+            }
+            elseif ($this->literalBreaks)
+            {
+                $marker = '\n';
+                $text = '  ';
+            }

             # ~

Enable the behaviour with
$Parsedown->setLiteralBreaks(true);

So the following
## par 1
This is a line.

**par 2**

And this is another one **WOW!**

Will be parsed as:

par 1

This is a line.

par 2

And this is another one WOW!

I think @cebe is correct in this being off syntax for markdown though (it breaks the ability for you to interrupt a paragraph with several new lines).
As a point of reference, the commonmark reference parser is consistent with Parsedown's behaviour for this one.

I still believe the syntax is a bit ambiguous there, but it seems every parser is doing the same, so maybe that is what people expect anyway.
Thank you for the patch @aidantwoods , I'll try it in my project.


Spelling Fixes - Wave 1

As requested @Court72 , i will break all the spelling fixes into multiple PRs to make it easy to review.
I will try to stay within 'Service-Folders' as @JasonWHowell referred to them , unless its less than 10-20 overall, just to not get to 70 PRs floods

@Gargi-Sinha and @kengaderdus
Can you review the proposed changes?
Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team


Attach Gif

Attach the gif which was missing due to an IO error

Github Action passed, merging!

Did squash and merge this time. Yay!


notif: Use Zulip's custom notification sound on Android

(Stacked on top of #981)
(Split from #717)
Fixes: #340

Thanks for the review @PIG208! Addressed them in the new revision.

Thanks for the review @gnprice! I've pushed the revision — PTAL.

Thanks for the review @gnprice! Pushed a new revision, PTAL.


[#24789][prism] V0 Go Direct Runner Replacement, Prism.

Add the final connections for making the runner usable.
Add in the pipeline execution tests, and initial separation harness tests.
Note, SDF splitting isn't yet implemented, although non-splitting expansion is supported.
See #24789. This doesn't complete the the issue as the issue is intended to replace the default Go SDK runner. This PR doesn't yet set prism as default, which would require at minimum instructions for switching back to the legacy direct runner, and for repository infrastructure to test on presubmit.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @johannaojeling
cc: @jrmccluskey
As foretold, this is a big one, but it's the last one in this veritable barrage of PRs. Thank you very much!

Ready for another look!

These may be my personal preferences, but I think the code would be easier to follow if RunPipeline, as the main function in the internal package, was defined in the beginning of execute.go, followed by the functions it uses (runEnvironment, executePipeline, and so on).

Fair! I re-arranged a little before sending this out. That's the reverse of my usual preference (having things defined before they're used). But I agree that ordering is more approachable for reading how the runner works in this case.

It might also make sense to move stage from execute.go to preprocess.go, or to its own file if you would prefer that. As it is now, executePipeline() in execute.go initializes preprocessor and invokes preprocessor.preProcessGraph(), which is defined in preprocess.go. Whereas preprocessor.preProcessGraph() initializes stage, which is defined in execute.go.

This one is trickier. Preprocess.go only uses a single field in stage: the slice of transform IDs, everything else is used around this file, and is more central to execution. So it shouldn't live in preprocess.go IMO.
Preprocess used to live in execute.go but the preprocessor, while simple now, will eventually handle stage fusion, and become more complicated, so it's worth splitting to it's own file for testing. A bunch of stuff should probably moved to the pre-processor too.
But I've ended up moving it to its own file in the end, because it turns out it's most of the content? Some of it will need to become its own "handler", for handling messages to the FnAPI. I had clearly lost scope about how much code was unique to building and managing stages. Ultimately, it'll be a candidate to move to it's own package along with associated unit testing and documentation.

Makes sense and looks good 🙂


Beats Music Web Player

The scrobbler scrobbles immediately when a song starts in the Beats Music web player. This results in large amounts of duplicate, erroneous, scrobbles when skipping through a playlist.

@cdmoye @david-sabata I am working on a update to the beats connector. I have a v2 working, I am testing it before pushing to prod.


feature(client) fix failiing test

[143585203]

Coverage remained the same at 80.0% when pulling c9ed250125d7417e8b7e83c2bb1fd7a4bc0a6d5a on feature/143585203/build-frontend-components-with-effective-UI/UX into 89e91f64be17ab049344c1eaa7b73991a448ceb5 on develop.


repl Ctrl-r keybinding to search history

Most repl's I've used before allow you to quickly get a previous command back by using the Ctrl+r key to search the command history. It makes life so much easier when you're testing stuff out. Any chance this could be implemented for the nodejs repl?

To piggy back on this idea, in bash and zsh I have a bind on Ctrl-r to show my history in fzf, it would be great if there was some kind of mechanism to provide re-maps (prehaps maybe some kind of .nodereplrc.js?)


Update planning-optimization-fit-analysis.md

Updated based on customer input

@ChristianRytt , thanks for this update. I merged it.


Migrate scim howto

[x] https://community.superoffice.com/en/technical/documentation/identity-management/how-to/provision-users-with-scim/
[x] https://community.superoffice.com/en/technical/documentation/identity-management/how-to/provision-users-with-scim/activate-and-deactivate-rules/
[x] https://community.superoffice.com/en/technical/documentation/identity-management/how-to/provision-users-with-scim/add-new-scim-rule/
[x] https://community.superoffice.com/en/technical/documentation/identity-management/how-to/provision-users-with-scim/preview-sync/
[x] https://community.superoffice.com/en/technical/documentation/identity-management/how-to/provision-users-with-scim/restart-synchronization/
[x] https://community.superoffice.com/en/technical/documentation/identity-management/how-to/provision-users-with-scim/sign-in-to-scim/
[x] https://community.superoffice.com/en/technical/documentation/identity-management/how-to/provision-users-with-scim/sync-company/
[x] https://community.superoffice.com/en/technical/documentation/identity-management/how-to/provision-users-with-scim/sync-user/
[x] https://community.superoffice.com/en/technical/documentation/identity-management/how-to/scim-set-up-guide/
[x] https://community.superoffice.com/en/technical/documentation/identity-management/how-to/scim-set-up-guide/create-azure-enterprise-app/
[x] https://community.superoffice.com/en/technical/documentation/identity-management/how-to/scim-set-up-guide/enable-automatic-sync/
[x] https://community.superoffice.com/en/technical/documentation/identity-management/how-to/scim-set-up-guide/get-scim-config/
[x] https://community.superoffice.com/en/technical/documentation/identity-management/how-to/scim-set-up-guide/get-sync-error-messages/

@pavlas77 can you check this section and close issue if ok?


Show suggestions list if customActions are, but not suggestions

At now, if setuped customActions, but not have suggestions by target query - suggestions list will be hidden.
If is some custom actions to show - suggestions list must be opened

not reproduced at v1.3.4 branch
supposedly it was side-fixed by another changes of RC


Question: Can I skip the detection and only go with the recognition?

For my use case, I want to perform inference faster. Can I take the entire resized image as detected area, I suppose if the image contains a lot of text this should have made the inference faster rather than running the recognition model on each detected line? Since on the web for onnxruntime, the inference is too slow!

Hey @DipanshuJuneja, you can fork this project and test the recognition feature by enabling only this line, then review the results. If it works well, I’ll consider implementing it.
One more way to speed it up is by enabling the resize max size to 960 here, which should greatly improve the speed. (Depends on the original image size)


make format yields formatted files when go1.19 is installed

How to categorize this issue?

/area dev-productivity
/kind bug
What happened:
On master, I can observe the following:
$ go version
go version go1.19 darwin/amd64
$ make clean-tools-bin
rm -rf hack/tools/bin/*
$ make format
...
> Format
extensions/pkg/controller/worker/genericactuator/actuator.go
extensions/pkg/terraformer/types.go
pkg/apis/authentication/v1alpha1/register.go
pkg/client/core/clientset/internalversion/fake/register.go
pkg/client/core/clientset/versioned/fake/register.go
pkg/client/core/clientset/versioned/scheme/register.go
pkg/client/kubernetes/chartoptions.go
pkg/client/kubernetes/test/options_matchers.go
pkg/client/seedmanagement/clientset/versioned/fake/register.go
pkg/client/seedmanagement/clientset/versioned/scheme/register.go
pkg/client/settings/clientset/versioned/fake/register.go
pkg/client/settings/clientset/versioned/scheme/register.go
pkg/operation/botanist/component/extensions/operatingsystemconfig/original/components/logrotate/logrotate.go
pkg/operation/botanist/component/kubeapiserver/horizontalpodautoscaler.go
pkg/operation/botanist/component/vpnshoot/monitoring.go
pkg/operation/seed/seed_suite_test.go
pkg/operation/shoot/status_test.go
pkg/utils/errors/errors.go
pkg/utils/kubernetes/kubernetes.go
pkg/utils/test/matchers/cmp.go
pkg/utils/test/test.go
pkg/utils/utils_suite_test.go
pkg/utils/validation/features/featuregates.go
test/framework/utils.go
hack/tools/gomegacheck/pkg/gomegacheck/testdata/src/github.com/onsi/gomega/gomega_dsl.go

Accordingly, this breaks make check for me.
When using go1.18, no files are formatted:
$ export PATH="/usr/local/opt/go@1.17/bin:$PATH"
$ go version
go version go1.17.13 darwin/amd64
$ make clean-tools-bin
rm -rf hack/tools/bin/*
$ make format
...
> Format
$ # no files formatted

What you expected to happen:
make format and friends to work in a backward-compatible manner even if I upgrade my toolchain to go1.19.

Gardener version: 74e1996176d70a102a2d32c857b458c4ecd71569

To fix this issue, I propose to upgrade to go1.19 in this repository.
/assign


Reduce the definition of test stubs for update_sriov_pf_map

The stub for update_sriov_pf_map is defined for many test cases in test_objects.py. Reduced the redefinition of the same

Thanks, this looks good, I appreciate the follow up.


Custom sorting for column does not work

I cannot set custom sorting function for column.
In types, there is defined sortByFn
  export interface HeaderColumn<D> {
    /**
     * This string/function is used to build the data model for your column.
     */
    accessor: IdType<D> | AccessorFn<D>
    Header?: ReactNode | ((props: TableInstance<D>) => ReactNode)
    Filter?: ReactNode | ((props: TableInstance<D>) => ReactNode)
    Cell?: ReactNode | ((cell: Cell<D>) => ReactNode)
    /**
     * This is the unique ID for the column. It is used by reference in things like sorting, grouping, filtering etc.
     */
    id?: IdType<D>
    minWidth?: string | number
    maxWidth?: string | number
    width?: string | number
    disableSorting?: boolean
    canSortBy?: boolean
    sortByFn?: SortByFn
    defaultSortDesc?: boolean
    isAggregated?: any
  }

However, it is not used anywhere by this library. The only way I found to implement custom sorting was adding a new sortTypes and then referencing it through sortType property on column. That is not defined on the HeaderColumn type though.
I'm not sure what's more updated, whether API docs or ts types.

The types are still not up to date with v7. The docs should say that you can pass a custom function as the sortType prop on a column to get the functionality you want. If the docs don't reflect this, please submit a PR to update them? Thanks!

Types for v7 are not yet published. Types for v6 are published on DefinitelyTyped. You can provide a declaration file for your project in the meantime. Otherwise, I'm looking for feedback on #1535

Will types be published on DefinitelyTyped? I noticed there's a trend to keep them with repository so they don't get out of sync so easily.

Types will be published in the package for v7.


Display bug on iPad with EULA on initial startup

No way to check the boxes. iPhone working fine. This is the initial release version of the wallet on an iPad Air2 running 11.3.1.

This version wasn’t intended for iPad
Best,
Zack
On Jun 22, 2018 at 8:32 AM, dbachm123<EMAIL_ADDRESS>wrote:
No way to check the boxes. iPhone working fine. This is the initial release
version of the wallet on an iPad Air2 running 11.3.1.
[image: 63b0107c-1a36-400a-82d3-c52ff8e85e7b]
https://user-images.githubusercontent.com/35655469/41776606-a9717caa-75e5-11e8-95d9-726d18fe2807.jpeg
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/nano-wallet-company/nano-wallet-ios/issues/9, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AA12S55FIVypKxq8KwTjkD3dwqEOsBwMks5t_OPigaJpZM4Uzujc
.

Ok. It says “Compatible with iPhone, iPad and iPod touch.” on the App Store page though.

That should be fixed now. Closing


Text customization, sync from click2vox

Text customization. Sync with click2vox.

Oh, I overlooked the differences with click2vox file, I will be doing the specific changes for this one, sorry for that

I will merge from a clean branch