Support installation as a node module

I would appreciate if this could work as a regular node module with ES6 imports, allowing greater flexibility and interoperability with other libraries as well as own code.
I tried to make this work in a webpack environment to no avail – mostly due to missing dependencies I'm not willing to install explicitly as they have nothing to do with my project, just this library.
I'm looking forward to giving this a shot in the future; especially the snapping features look great.

Hi @pavelhamrik, thank you for the comment.
Could you explain about details?

I got confused by the installation instructions which point me in the direction of linking a minified script in my html, a practice I don't want to turn to in e.g. a React project.
However, I realized I can use script-loader for webpack and the library is indeed to be found in the npm repository, so I can list it as a dependency. Not Ideal, but workable.
Thank you for your quick response.

Ok. :smile:
So, could you close this issue because it was already solved?

If you are not being able to install the library with NPM due to missing dependencies, run the following command:
npm install --save anim-event cssprefix m-class-list pointer-event
It solved the problem for me.

Hi @ubirajaramneto, thank you for your report.
You have to install the devDependencies only when you will develop something. The dev in the devDependencies means that.

Hello Anseki, thanks for the clarification, but what I meant here, and maybe I am posting in the wrong issue is that, when I tried installing your library using NPM, it would error out and give me and error telling me that it could not find these dependencies.
After issuing the above command, everything started working normally. I understand that the dev dependencies are only to be used when requiring dependencies that are only needed at development, but not in production. In my case I will be using this library in production.
Maybe I did not get what you meant, if that is the case I am sorry. Was just trying to help out anyone having issues using this library as a NPM module.

NPM never throw that error because the library has no dependency.
Maybe you mistook NPM for a development tool (e.g. Webpack, Rollup, etc. (that try to bundle modules)) that try to find the link modules. For example, that development tool may throw an error when link modules are not found.
As you said, your case is unrelated to this issue, but thank you for your report. :smile:


Feature/Analytics

Create the Version class
Load the current version from the csproj data through the GetType>Assembly>GetName>Version (I don’t really know if this could have some potential of not working in some cases, let me know that).

The Github Action for some reason stopped working, I fixed the version of the .net in order to fix the problem.

Related to https://github.com/meilisearch/integration-guides/issues/150

bors try

bors merge


False positives

Hi, I think enola is returning false positives.
When I check my username with sherlock, I get 18 results. However when I use enola, I get 49.

don't they use the same data?

the database is almost the same.
I will check, But let me know if you're aware of any specific false positive items.

well, there are at least 31 sites which returned different results.
Among them are mixcloud, twitch and freecodecamp (to name a few).
If you want the full list, just compare the results from sherlock and enola with my username (jon4hz)

Yeah just came here to let you know about the false positives. It seems like sites that have automatic 404 user not found.

Yeah, just downloaded enola for the first time and while it's absolutely gorgeous visually, I'm getting three pages of positive hits, but every link that I've clicked on (aside from the one I know to be real) either goes to a 404 page or if the page does load, there simply isn't an actual user with that username on the site. So as things stand this doesn't seem to be working at all.

dccdd


Invalid character spacing on a specific textPath

What happened?

lib
pic

svg

ink

resvg

What should have happened?

letter distance seems to be strange at "Rio of Fox" (left edge)

Copyright
source https://commons.wikimedia.org/wiki/File:Hidrografía_Galicia_Cunca_Mariña_Oriental_en.svg
author Iago Casabiell González
license https://creativecommons.org/licenses/by-sa/4.0/deed.de
Version and Operating System:

Inkscape Version: 1.1dev
resvg version: 0.9.0
rsvg Version: 2.40.21
Operating System: Ubuntu 18.04

Yes, this is a textPath rendering bug.

same with
INkscape

resvg

vdiff

Hi, I'm running into this bug with a simple textPath (an ellipse). I'm trying to fix this special case temporarily. Could you kindly point out the relevant code? I read usvg and found https://github.com/RazrFalcon/resvg/blob/7b26adbcc9698dcca687214c84d216794f60a5be/usvg/src/convert/text/shaper.rs#L438 suspicious.

https://github.com/RazrFalcon/resvg/blob/7b26adbcc9698dcca687214c84d216794f60a5be/usvg/src/convert/text/shaper.rs#L666
For my problem, the root cause seems to be this line. Because arc length doesn't generally vary linearly with the curve parameter.

@chubei-oppen I haven't looked into this bug, so I don't know the reason. You're on the right path, it's somewhere in the shaper.rs.

@RazrFalcon I came up with a somewhat ugly solution in #326. Some code is copied from kurbo because that computation can be reused.
Whenver we need to map arc length to curve parameter, we subdivide the bezier curve until the start point or the end point of the subdivided curve is within accuracy of the target arc length.
It takes O(log(L / a)) time for a single OutlinedCluster, where L is the arc length, a is our desired accuracy.
Subdivided curves are cached in a segment tree.
It works well enough for me as my use case is not very time sensitive.
If you see this as a plausible algorithm I can add proper documentation, tests and fix code styles (may need your guide here because I'm new to Rust), otherwise I'll keep it in my fork.

@chubei-oppen This is a very complicated algorithm for such a trivial task. I'm not saying that it's wrong, but maybe it should be implemented on the kurbo side?
Also, allocating a tree is fairly expensive, so maybe kurbo authors will came up with a better solution.

@RazrFalcon kurbo has inv_arclen, which uses binary search. I'm not an expert on Bezier curves and don't know if its performace is acceptable. Do you want to see a what a pull request using inv_arclen looks like?

If it produces the expected result - sure.

@RazrFalcon inv_arclen has its own bug. I guess I'll wait until that's fixed.


where is getSum method?

TypeError: Cart.query(...).where(...).getSum is not a function
Model.getSum() Docs
Is the getSum method gone?
I can not find @ adonis4 and knex packages.
My code like this:
const Cart = use('App/Models/Cart');
...
await Cart.query().where(function() {
  this.where('uid', request.input('uid'));

  if (member) {
    this.orWhere('member_id', member_id);
  }
}).getSum('qty');

// also fails `getSum is not a function`
await Cart.query().getSum('qty');

// `sum` method succeed
await Cart.query().where(function() {
  this.where('uid', request.input('uid'));

  if (member) {
    this.orWhere('member_id', member_id);
  }
}).sum('qty as qty');

Hey @brendaniel! 👋
The PR containing this method hasn't been merged yet.
Ref: https://github.com/adonisjs/adonis-lucid/pull/205

Closing since PR has been merged a while back


Add "exists" checkbox feature from Advanced Search to current filters

Currently, the users can filter for specific infectiousAgent, species, healthCondition, etc. and the majority of these will be Not specified as they are missing.
It would be useful to filter for ones that have a value, regardless of what the value is. You can do this in the advanced query by selecting the field and checking the exists box. The NIAID team would like to have this option for each of the already available filters.
Related WBS tasks:
https://github.com/NIAID-Data-Ecosystem/nde-roadmap/issues/52

Ready for review in the staging

excellent
From: Candice @.>
Sent: Friday, December 1, 2023 12:53 PM
To: NIAID-Data-Ecosystem/nde-portal @.>
Cc: Ginger Tsueng @.>; Author @.>
Subject: Re: [NIAID-Data-Ecosystem/nde-portal] Add "exists" checkbox feature from Advanced Search to current filters (Issue #199)
Ready for review in the staginghttps://data-staging.niaid.nih.gov/search?q="HIV"+OR+"AIDS"
—
Reply to this email directly, view it on GitHubhttps://github.com/NIAID-Data-Ecosystem/nde-portal/issues/199#issuecomment-1836760967, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACGGECTWJBUBUBAHL3TASILYHI7SFAVCNFSM6AAAAAA526HZXOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZWG43DAOJWG4.
You are receiving this because you authored the thread.Message ID: @.***>

Viewable in the production  instance

No comment from NIAID team on this after a week in production -- we're good to close it


Bug: Incorrect value for INVALID_SOCKET

Which crate is this about?
windows-sys
Crate version
0.34.0
Summary
windows-sys appears to have an incorrect value for INVALID_SOCKET.
Toolchain version/configuration
installed toolchains
stable-x86_64-pc-windows-msvc (default)
installed targets for active toolchain
x86_64-pc-windows-msvc
active toolchain
stable-x86_64-pc-windows-msvc (default)
rustc 1.59.0 (9d1b2106e 2022-02-23)
Reproducible example
fn main() {
    dbg!(winapi::um::winsock2::INVALID_SOCKET);
    dbg!(windows_sys::Win32::Networking::WinSock::INVALID_SOCKET);
}

Crate manifest
[package]
name = "invalid"
version = "0.1.0"
edition = "2021"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
winapi = { version = "0.3.9", features = ["winsock2"] }
windows-sys = { version = "0.34.0", features = ["Win32_Networking_WinSock"] }

Expected behavior
[src\main.rs:2] winapi::um::winsock2::INVALID_SOCKET =<PHONE_NUMBER>3709551615
[src\main.rs:3] windows_sys::Win32::Networking::WinSock::INVALID_SOCKET =<PHONE_NUMBER>3709551615
Actual behavior
[src\main.rs:2] winapi::um::winsock2::INVALID_SOCKET =<PHONE_NUMBER>3709551615
[src\main.rs:3] windows_sys::Win32::Networking::WinSock::INVALID_SOCKET =<PHONE_NUMBER>
Additional comments
No response

Good catch, transferring to win32metadata. Looks like this was incorrectly set to uint.MaxValue when it's really a UINT_PTR.


Include an icon set as part of the library.

@melmdesign has been actively doing things here. We should provide icons with Circuit.

Closed in favor of https://github.com/sumup/currents


versions newer than 2.21 have trouble with some deploys hanging at "waiting for deploy to go live"

- Do you want to request a feature or report a bug?
bug
- What is the current behavior?
versions >2.21 can fail to complete deploys, "waiting for deploy to go live".  Two customers saw it and the fix worked for both, as described in this thread:   https://community.netlify.com/t/formerly-working-cli-deploys-now-hanging-on-waiting-for-deploy-to-go-live/6519
- If the current behavior is a bug, please provide the steps to reproduce.
I've asked the affected customers to post more details about their situations in comments.
- What is the expected behavior?
Deploys would either go live, or timeout usefully.
- Local Environment Information
hopefully the affected customers will post their info in the comments below.

Same issue here when trying to deploy: https://github.com/jumpalottahigh/blog.georgi-yanev.com via netlify cli. Can confirm that rolling back to version 2.21.0 deploys successfully.

Same issue happening to me too

Same thing happened to me today. Timed out after 20 mins.

same thing happend for me ,worked on 2.21 👍

same issue

looks like this was previously reported too - https://github.com/netlify/cli/issues/634

same here https://github.com/netlify/cli/issues/659

Looks like this will be fixed soon thanks to https://github.com/netlify/cli/pull/674

@mrlubos I haven't tested it for this bug, but #674 has already been released to 2.26.0 if you want to try it out :)

@apuyou I can confirm this issue has now been fixed. Thank you for the fix!

Looks like<EMAIL_ADDRESS>broke it again :/

Can confirm<EMAIL_ADDRESS>hangs as described above. 👍 If you are encountering the same issue.

We are still getting more reports in community of this with 2.31.0:  https://community.netlify.com/t/deploy-is-stuck-from-cli-web/8702

(Reopening this bug, since it appears to be affecting current version)

Running 'netlify-cli/2.32.0 linux-x64 node-v13.7.0' and I can confirm that it hangs at 'waiting for deploy to go live...' after prompting the command netlify deploy.

Same issue here when trying to deploy: https://github.com/jumpalottahigh/blog.georgi-yanev.com via netlify cli. Can confirm that rolling back to version 2.21.0 deploys successfully.

Could you let me know how can I roll back to version `2.21.0'?, Thanks

netlify-cli/2.32.0 darwin-x64 node-v13.8.0 same here. Will continue to use the web app for now.

netlify-cli/2.32.0 darwin-x64 node-v12.14.0 having issues as well. Stucked in Waiting for deploy to go live.

I have this same issue with netlify-cli/2.35.0 here.
Can you guys help me how to roll back to a previous version?

I have this same issue with netlify-cli/2.35.0 here.
Can you guys help me how to roll back to a previous version?

npm i -g<EMAIL_ADDRESS>would do it.

I am also having this issue (netlify-cli/2.35.0 darwin-x64 node-v13.8.0), so I've rolled back to 2.21.0 and the CLI seems to be successful on netlify deploy --prod once more, but hopefully this can get resolved as it's been a recurring break. 🤞

Thank you all for reporting this issue and hanging in there. We've identified the problem and confirmed a fix (https://github.com/netlify/cli/pull/726). The changes will be released soon. Thanks

A few for this has been published recently. Please update to latest version of netlify-cli and let us know if it is still a problem.


Use kubectl rollout restart for zero downtime update

To minimize CoreDNS downtime, kubectl rollout restart is used instead of kubectl delete pod, which improve the overall AKS stability.

This PR does not change the original scope.
repairman@vm-hub:~$ kubectl get pod --namespace kube-system -l k8s-app=kube-dns
NAME                       READY   STATUS    RESTARTS   AGE
coredns-7cfb675cf4-c8fd2   1/1     Running   0          14m
coredns-7cfb675cf4-h99js   1/1     Running   0          14m

repairman@vm-hub:~$ kubectl get deployment --namespace kube-system -l k8s-app=kube-dns
NAME      READY   UP-TO-DATE   AVAILABLE   AGE
coredns   2/2     2            2           13h

repairman@vm-hub:~$ kubectl rollout restart deployment --namespace kube-system coredns
deployment.apps/coredns restarted

repairman@vm-hub:~$ kubectl get pod --namespace kube-system -l k8s-app=kube-dns
NAME                       READY   STATUS        RESTARTS   AGE
coredns-54c65f6954-j8ntj   1/1     Terminating   0          13s
coredns-54c65f6954-zms6r   1/1     Terminating   0          13s
coredns-8498647548-kb77q   1/1     Running       0          6s
coredns-8498647548-mjvkl   1/1     Running       0          6s

repairman@vm-hub:~$ kubectl get pod --namespace kube-system -l k8s-app=kube-dns
NAME                       READY   STATUS    RESTARTS   AGE
coredns-8498647548-kb77q   1/1     Running   0          9s
coredns-8498647548-mjvkl   1/1     Running   0          9s

@asudbring
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@pichuang
Can you confirm that the changes that are made have been tested by running through the article to ensure that the outcome is the same and there aren't any errors?
Thanks!

Currently reproducing all the steps for the chapter without any errors. The following steps and outputs can be referenced.
Rewrite DNS
repairman@vm-hub:~/custom-coredns$ touch corednsms.yaml
repairman@vm-hub:~/custom-coredns$ vim corednsms.yaml
repairman@vm-hub:~/custom-coredns$ kubectl apply -f corednsms.yaml
configmap/coredns-custom configured
repairman@vm-hub:~/custom-coredns$ kubectl get configmaps --namespace=kube-system coredns-custom -o yaml
apiVersion: v1
data:
  test.server: |
    test.com:53 {
    log
    errors
    rewrite stop {
      name regex (.*)\.test.com {1}.default.svc.cluster.local
      answer name (.*)\.default\.svc\.cluster\.local {1}.test.com
    }
    forward . /etc/resolv.conf # you can redirect this to a specific DNS server such as <IP_ADDRESS>, but that server must be able to resolve the rewritten domain name
    }
kind: ConfigMap
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"v1","data":{"test.server":"test.com:53 {\nlog\nerrors\nrewrite stop {\n  name regex (.*)\\.test.com {1}.default.svc.cluster.local\n  answer name (.*)\\.default\\.svc\\.cluster\\.local {1}.test.com\n}\nforward . /etc/resolv.conf # you can redirect this to a specific DNS server such as <IP_ADDRESS>, but that server must be able to resolve the rewritten domain name\n}\n"},"kind":"ConfigMap","metadata":{"annotations":{},"name":"coredns-custom","namespace":"kube-system"}}
  creationTimestamp: "2023-07-12T14:46:39Z"
  labels:
    addonmanager.kubernetes.io/mode: EnsureExists
    k8s-app: kube-dns
    kubernetes.io/cluster-service: "true"
  name: coredns-custom
  namespace: kube-system
  resourceVersion: "548299"
  uid: 9694176d-e759-4a47-9bb0-d1cd63450bd1
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system rollout restart deployment coredns
deployment.apps/coredns restarted
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system get pods -l k8s-app=kube-dns
NAME                       READY   STATUS    RESTARTS   AGE
coredns-59dd88d576-8rnhl   1/1     Running   0          62s
coredns-59dd88d576-md75h   1/1     Running   0          62s

Custom forward server
repairman@vm-hub:~/custom-coredns$ vim corednsms.yaml
repairman@vm-hub:~/custom-coredns$ kubectl apply -f corednsms.yaml
configmap/coredns-custom configured
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system rollout restart deployment coredns
deployment.apps/coredns restarted
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system get pods -l k8s-app=kube-dns
NAME                       READY   STATUS        RESTARTS   AGE
coredns-59dd88d576-md75h   1/1     Terminating   0          3m
coredns-5c9fdbd7f7-5255n   1/1     Running       0          8s
coredns-5c9fdbd7f7-gdgnb   1/1     Running       0          8s
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system get pods -l k8s-app=kube-dns
NAME                       READY   STATUS    RESTARTS   AGE
coredns-5c9fdbd7f7-5255n   1/1     Running   0          11s
coredns-5c9fdbd7f7-gdgnb   1/1     Running   0          11s

Use custom domains
repairman@vm-hub:~/custom-coredns$ vim corednsms.yaml
repairman@vm-hub:~/custom-coredns$ kubectl apply -f corednsms.yaml
configmap/coredns-custom configured
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system rollout restart deployment coredns
deployment.apps/coredns restarted
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system get pods -l k8s-app=kube-dns
NAME                       READY   STATUS              RESTARTS   AGE
coredns-5c9fdbd7f7-5255n   1/1     Terminating         0          93s
coredns-5c9fdbd7f7-gdgnb   1/1     Running             0          93s
coredns-7df5cbd7f-cllgm    0/1     ContainerCreating   0          4s
coredns-7df5cbd7f-r6p6n    0/1     ContainerCreating   0          4s
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system get pods -l k8s-app=kube-dns
NAME                       READY   STATUS              RESTARTS   AGE
coredns-5c9fdbd7f7-5255n   1/1     Terminating         0          97s
coredns-5c9fdbd7f7-gdgnb   1/1     Running             0          97s
coredns-7df5cbd7f-cllgm    0/1     ContainerCreating   0          8s
coredns-7df5cbd7f-r6p6n    0/1     ContainerCreating   0          8s
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system get pods -l k8s-app=kube-dns
NAME                       READY   STATUS              RESTARTS   AGE
coredns-5c9fdbd7f7-gdgnb   1/1     Running             0          99s
coredns-7df5cbd7f-cllgm    0/1     ContainerCreating   0          10s
coredns-7df5cbd7f-r6p6n    0/1     ContainerCreating   0          10s
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system get pods -l k8s-app=kube-dns
NAME                       READY   STATUS              RESTARTS   AGE
coredns-5c9fdbd7f7-gdgnb   1/1     Running             0          100s
coredns-7df5cbd7f-cllgm    0/1     ContainerCreating   0          11s
coredns-7df5cbd7f-r6p6n    0/1     ContainerCreating   0          11s
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system get pods -l k8s-app=kube-dns
NAME                       READY   STATUS    RESTARTS   AGE
coredns-5c9fdbd7f7-gdgnb   1/1     Running   0          101s
coredns-7df5cbd7f-cllgm    0/1     Running   0          12s
coredns-7df5cbd7f-r6p6n    0/1     Running   0          12s
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system get pods -l k8s-app=kube-dns
NAME                       READY   STATUS        RESTARTS   AGE
coredns-5c9fdbd7f7-gdgnb   1/1     Terminating   0          104s
coredns-7df5cbd7f-cllgm    1/1     Running       0          15s
coredns-7df5cbd7f-r6p6n    1/1     Running       0          15s
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system get pods -l k8s-app=kube-dns
NAME                      READY   STATUS    RESTARTS   AGE
coredns-7df5cbd7f-cllgm   1/1     Running   0          40s
coredns-7df5cbd7f-r6p6n   1/1     Running   0          40s

Stub domains
repairman@vm-hub:~/custom-coredns$ vim corednsms.yaml
repairman@vm-hub:~/custom-coredns$ kubectl apply -f corednsms.yaml
configmap/coredns-custom configured
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system rollout restart deployment coredns
deployment.apps/coredns restarted
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system get pods -l k8s-app=kube-dns
NAME                       READY   STATUS        RESTARTS   AGE
coredns-7df5cbd7f-cllgm    1/1     Terminating   0          2m16s
coredns-7df5cbd7f-r6p6n    1/1     Terminating   0          2m16s
coredns-86565c67c4-2qbhn   1/1     Running       0          6s
coredns-86565c67c4-5ctqs   1/1     Running       0          6s
repairman@vm-hub:~/custom-coredns$ kubectl -n kube-system get pods -l k8s-app=kube-dns
NAME                       READY   STATUS    RESTARTS   AGE
coredns-86565c67c4-2qbhn   1/1     Running   0          27s
coredns-86565c67c4-5ctqs   1/1     Running   0          27s

#sign-off


lintKotlin task triggers deprecation warning with Gradle 6.0

After upgrading to gradle 6.0, the use of lintKotlin task triggers a deprecation warning about the use of @PathSensitive with the @OutputFiles annotations.
> Task :lintKotlinIntTest
Property 'reports' is annotated with @PathSensitive that is not allowed for @OutputFiles properties. This behaviour has been deprecated and is scheduled to be removed in Gradle 7.0.

https://github.com/jeremymailen/kotlinter-gradle/blob/master/src/main/kotlin/org/jmailen/gradle/kotlinter/tasks/LintTask.kt#L32-L34

Thank you for the report and fix!

@jmeekhof @jeremymailen was there a suggested replacement?
I observe that currently task is not cacheable on CI since output files are back to ABSOLUTE path sensitivity.

Any examples?

Sure. Is there any projects integrating plugin you check yourself for regression?
If not I will try to create minimal sample to reproduce the issue.

Thanks for highlighting the issue. Would like to have a solution that is not depreciated and caches correctly. Not sure the right way without doing some research, but very open to suggestions and pull requests.

As soon as I exacted minimal project with single kotlin file the issue is gone and cache behaves like it should. Of course 😅
I will keep an eye on it but as of now I don't have steps-to-reproduce anymore (outside of my working project which I wouldn't share). Thank you for taking a time to reply 👍


Juno.render no longer works from the editor

If I type Juno.render([1,2,3]) in an editor and eval with ctrl+enter, nothing happens. Previously it would render [1,2,3] nicely in the console.
Perhaps this feature died in the change to the new console, but it's still in the docs.
Julia 0.6.2, Juno 0.4.0, Atom 0.6.9
Atom 1.23.3, julia-client 0.6.8

Right, that's a docs issue.

This should still work. I'm guessing it's still hooked up to the old console? We can just remove that and let it fall back to the repl.

Well, one argument render uses Console() as the default display IIRC. So we could either pipe those calls back to the base display system or make Editor() the default display (which would be pretty useless).

We explicitly tell render to use the console as the generic fallback here, so it should just be a case of removing that. Just doing Media.unsetdisplay(Juno.Editor(), Any) fixes this for me.

Fixed: https://github.com/JunoLab/Atom.jl/commit/0697e594daacb15e99dfe438b1ce89278b4f1b09.


Vitis AI 2.5: API_0 yolov3_adas_pruned_0_9 VART/adas_detection slower than API_1 yolov3_adas_pruned_0_9 Vitis-AI-Library/yolov3

Despite using the same xmodel file, these two API_0: VART-adas_detection and API_1: Vitis-AI-Library-yolov3 show massive difference in FPS and speed during inference. I'm currently trying to implement my own model using VART, but it also showed that API_0 VART is much slower compared to API_1 and 2. Is this intended or there have been some bad configuration on my end?
API_0: VART - yolov3_adas_pruned_0_9

API_1: Vitis-AI-Library - yolov3_adas_pruned_0_9

DPU: DPUCZDX8G_ISA1_B4096
BOARD: ZCU102
Vitis-AI version: 2.5.0

Hi, @omgitsmj24
Thank you for your feedback.
I tested 2 samples from vart and Vitis-AI-Library.
I preliminarily found the pre-processing and post-processing speed of the vart to be slower than the Vitis-Ai-Library. The main reason is that we only optimized for Vitis-AI-Library. I recommend using Vitis-AI-Library's interface for deployment.
Regards

Closing since no activity for more than 1 month, please reopen if you still have any questions, thanks


ECS Approach

With the power of Kotlin, it's time once again to try an Entity/Component/System approach.
Remember the decisions made and the lessons learned and try to make it as easy as possible to do it "right". Remember to fail early and fail often when something is wrong, such as trying to get a component that is not attached to an entity.
Especially remember the following:

View, what is public, what is private, what is invisible
It should be quite easy to make AIs
Actions are important and can be performed at different times, this is probably where the systems come in, to decide which action is possible when, but the whole "Systems" aspect needs a lot more thought here. How is different from "game flow" and "yield action" and other previous variants of deciding which actions (and options, and recursive options) are allowed?

companion objects in Kotlin can have a name and a type, which makes them a lot nicer to refer to. See how Kotlin has done with coroutine context elements, for example: public companion object Key : CoroutineContext.Key<Job>

Simple Tic-Tac-Toe created. Will try and re-use lots to create similar games: TTTUltimate, Connect4...
Creating AIs in ECS style might be a bit more tricky, especially scoring AIs. Since the way I've done it in TTT encourages one-action-per-entity. So instead of one actionType with 9 possible actions, there is now 9 actionTypes with one action each. The actionTypes are named for example /board/1,2/play


Supports passing arguments to test framework

For #222
I thought I'd open this PR up so there would be a spot to get some feedback. I'm pretty new to bazel and Scala and am definitely willing to listen to feedback.
As it, it still needs to at least be documented and something in SubprocessRunner. If there is a spot some tests could be added, I'd also be happy to do that.
This adds a --option parameter to the TestRunner. --option can be given multiple times. Example below from me running tests using it in another project of mine.
bazel test --test_arg=--option=-oDF --test_arg=--option='-l org.scalatest.tags.Slow' //websocket-api-server:tests
One alternative approach would be to support this options in the file pointed to by the java property scalaAnnex.test.args. That would let the flags persist across runs.
Another alternative, if persisting flags across runs is desired, would be to bubble these up to the scala_test rule (or I missing that you can already specify arguments here?).
I think some persistence would be nice so you can permanently and share between developers some common configuration (such as formatting flags).

One alternative approach would be to support this options in the file pointed to by the java property scalaAnnex.test.args. That would let the flags persist across runs.

I think my lack of Bazel knowledge helped me interpret that area of the code wrong. I've been poking around more now and am realizing that I misunderstood what was going on here. Back to reading and poking

I think I've figured out a better way of doing this. Going to close and reopen a different PR.


Images not appearing on any browser. See information below.

Request type

[] Please close this issue, I accidentally submitted it without adding any details
[x ] New documentation
[ ] Correction or update

Details
So, when the page is in Spanish you can't see the images attached. I am a beginner learning studying web development and I did a code inspection myself, it seems that the images are not charged or I don't know, its just a guessing. Since the site was translate by the community, maybe something went wrong during the process.
I have some photos of what I saw when I inspected the code:

https://developer.mozilla.org/es/docs/Learn/HTML/Introduction_to_HTML/The_head_metadata_in_HTML

Hi @Nikei89! This repository (mdn/sprints) isn't really used to track MDN issues any longer. If this is still an issue, please scroll down to the "Found a content problem with this page?" section and use the "Report the content issue" link to report the issue on the appropriate repository (mdn/translated-content) so that the appropriate team can resolve your issue quickly. Thank you in advance!


How do I use function onStepHit

Describe your problem here.
I’m trying to add an asset mid song using onStepHit but can’t get it to work
code:
local curstep
function onStepHit
if curstep == 2927 then
makeanimatedLuaSprite(‘image’, ‘file location’, 100, 100)
addAnimationByPrefix(‘image’, ‘dance’, ‘ XML LINE’, 24, true
playAnim(‘image’, ‘dance’, false(
addLuaSprite(‘image’, true)
end
end
Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No

Your Need to replace the function onStepHit to function onStepHit()

You can now close this issue

Case sensitivity


Incorrect compile.conf example

While building latest master (1bd8f98217417045756b10b0cc9af7fdf88b6c03) I've found that the example compile.conf file in Developer Guilde is misleading/incorrect.  Instead of:
hadoop_version = 2.3.0                                      // The Hadoop version to compile with
spark_version = 1.4.0                                       // The Spark version to compile with
play_opts="-Dsbt.repository.config=app-conf/resolver.conf"  // Other play/sbt options

it should be:
hadoop_version=2.3.0                                        # The Hadoop version to compile with
spark_version=1.4.0                                         # The Spark version to compile with
play_opts="-Dsbt.repository.config=app-conf/resolver.conf"  # Other play/sbt options

Otherwise reading incorrect configuration fails silently, like so:
Reading from config file...
hadoop_version=2.6.0                                      # The Hadoop version to compile with
compile.conf: line 1: hadoop_version: command not found
compile.conf: line 2: spark_version: command not found

and keeps working with the defaults:
Hadoop Version : 2.3.0
Spark Version  : 1.4.0
Other opts set :
+ trap exit SIGINT SIGTERM
+++ dirname ./compile.sh
++ cd .
++ pwd
+ project_root=/Users/ljank/sandbox/dr-elephant
+ cd /Users/ljank/sandbox/dr-elephant
+ start_script=/Users/ljank/sandbox/dr-elephant/scripts/start.sh
+ stop_script=/Users/ljank/sandbox/dr-elephant/scripts/stop.sh
+ rm -rf /Users/ljank/sandbox/dr-elephant/dist
+ mkdir dist
+ play_command -Dhadoopversion=2.3.0 -Dsparkversion=1.4.0 clean test compile dist
+ type activator
+ play -Dhadoopversion=2.3.0 -Dsparkversion=1.4.0 clean test compile dist

I assumed people would edit the compile.conf file already present under app-conf. Anyways, good to have a correct and non-misleading document. Thanks for pointing it out.
Wiki page updated!


[MP] Fix orphan StringName on close.

Use a static StringName for the registered default interface name.
Fixes #64591

Thanks!


PR: Support all real number dtypes in dataframe editor

Description of Changes

[x] Wrote at least one-line docstrings (for any new functions)
[x] Added unit test(s) covering the changes (if testable)

[x] Included a screenshot or animation (if affecting the UI, see Licecap)

This is a small PR which uses the new function pandas.api.types.is_any_real_numeric_dtype() to decide whether a Pandas dtype is a real number and thus gets the background colouring of numbers in the Dataframe editor. This allows us to supports the nullable integer types and also numeric types that were previously not supported like numpy.int16. A test is also included.
This PR imports pandas inside a function. As far as I can see, no part of Spyder explicitly imports pandas but I think that pandas is always imported when a Dataframe editor is opened, so this should not be a problem.
Screenshot after:

Screenshot before:

These are produced using the following code, from issue #21022:
import pandas as pd

vals = [1, 2, 3, 4, 5]
vals_na = [1, 2, 3, None, 5]

df = pd.DataFrame({
    'old': pd.Series(vals),
    'old_na': pd.Series(vals_na),
    'new': pd.Series(vals, dtype='Int64'),
    'new_na': pd.Series(vals_na, dtype='Int64')
})

Issue(s) Resolved
Fixes #21222
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct: Jitse Niesen

A couple of small suggestions fo you, the rest looks good to me.

Thanks, I applied all the suggestions.


Allow users to create usernames for themselves (Block Profanity)

Allow all users to add a name for themselves. Use Google Profanity words to prevent users from adding profanity to their name. Google Profanity Words-Github

[implementation notes]
use npm package https://www.npmjs.com/package/google-profanity-words.
The words defined are the same.


dataflow library, point to a directory of notebooks

Right now dataflow only runs a single .ojs file, like dataflow run notebook.ojs. this is fairly limiting, if you're working on multiple notebooks at a time, you'd either have to keep killing/restarting dataflow to point to a different file, or run them separately on different ports (which isn't fun at all).
dataflow library (different name?) could instead point to a directory of .ojs files. The dev server would then 1) allow you to view any of the .ojs files in that directory, and 2) offer a UI for navigating through all the notebooks.
This would be similar to jupyterlab, which has a navigation UI for a given directory (and even displays non-notebook files).

Most of the dev server logic assumes the same notebookPath, so will need to probably rewrite most of it
Detect when notebooks are added/removed
What should the navigation UI look like, and how to distinguish it from notebook contents

imho the cleanest option would be to still start from a file, but serve an entire directory and simply allow navigating to /other/notebook.ojs from links rendered from markdown or otherwise in a regular cell. This way we can experiment with navigation live :) and avoid baking something into the UI


[Bug]: Jaeger prometheus metrics are missing "collector" prefix?

What happened?
The jaeger docs here as well as sample Grafana dashboard here reference a metric jaeger_collector_queue_length. However, when running the latest jaeger image jaegertracing/all-in-one:1.52, the "collector" prefix seems to be missing - it only exports a jaeger_queue_length metric. So it seems like either the docs/grafana dashboard are wrong, or there's a bug where the "collector" prefix is missing on the exported metric? Or maybe these are actually different metrics and the "jaeger_collector_*" metrics are just missing?
Here are all the jaeger_collector_* metrics that are referenced from the Grafana dashboard:
jaeger_collector_in_queue_latency_bucket
jaeger_collector_queue_length
jaeger_collector_spans_dropped_total
jaeger_collector_spans_received_total

Unless these are actually different metrics, it looks like the actual metric names that are exported are:
jaeger_in_queue_latency_bucket
jaeger_queue_length
jaeger_spans_dropped_total
jaeger_spans_received_total

Steps to reproduce
Run the latest all-in-one image:
$ docker run --rm -it --net=host jaegertracing/all-in-one:1.52

Check which metrics are exported:
$ curl -fsSL http://localhost:14269/metrics | grep queue_length
jaeger_queue_length{host="..."}

Observe that the metric is called "jaeger_queue_length", not "jaeger_collector_queue_length" as referenced in both docs and sample Grafana dashboards.
Expected behavior
The metric should either be called jaeger_collector_queue_length or the docs + sample Grafana dashboard should reference the actual metric value jaeger_queue_length, depending on which is the "correct" / intended value.
Relevant log output
No response
Screenshot
No response
Additional context
No response
Jaeger backend version
1.52
SDK
No response
Pipeline
No response
Stogage backend
No response
Operating system
No response
Deployment model
No response
Deployment configs
No response

Yeah, looks like a bug. When you run the collector the metric name is correct, but in all-in-one it loses the collector segment:

Thank you for fixing this!


Add more robust validation to the deploy URL user input on config step

Currently the only validation is if the URL is not an empty string. The scope and types of validation are up for discussion. Validation ideas:
-URL validation
-minimum length validation
-http validation?(maybe some people won't want this)

TODO: Needs a spec before community task

Reviewing with Yoni. We're aligned to close this because doesn't seem necessary


build: improve _InternalSwiftSyntaxParser installation

Use the TARGETS form of install to avoid having to spell out the
location and file name to install.  Additionally, setup the installation
rules to work better for Windows by listing install locations for the
archive (static library) form (which includes the import library on
Windows), library form (which is the Unix library), and the runtime
component (the .dll on Windows).

Replace this paragraph with a description of your changes and rationale. Provide links to external references/discussions if appropriate.

Resolves SR-NNNN.

@swift-ci please test

@swift-ci please smoke test macOS platform

@swift-ci please smoke test macOS platform

@swift-ci please smoke test macOS platform

@swift-ci please smoke test macOS platform


Additions?

@lfarah  hey buddy, I bookmarked these a while back to add here.. hope it's ok that I create this issue

[x] https://github.com/algolia/algoliasearch-client-swift
[x] https://github.com/makemoji/MakemojiSDK
[x] https://github.com/DeveloperLx/LxDBAnything
[x] https://github.com/Instapaper/LinkPreviewCustomAction
[x] https://github.com/linkedin/Selene
[x] https://github.com/bignerdranch/CoreDataStack
[x] https://github.com/czechboy0/Buildasaur
[x] https://github.com/conradev/nes-ios
[x] https://github.com/TimOliver/TOActionSheet
[x] https://github.com/NewAmsterdamLabs/ZOZolaZoomTransition
[x] https://github.com/Raizlabs/RZTransitions
[x] https://github.com/xareelee/XLTestLog
[x] https://github.com/overboming/ZCAnimatedLabel
[x] https://github.com/IFTTT/FastttCamera
[x] https://github.com/IFTTT/IFTTTLaunchImage
[x] https://github.com/Yalantis/FoldingTabBar.iOS
[x] https://github.com/Raizlabs/BonMot
[x] https://github.com/northernspark/AHDirectionsObject
[x] https://github.com/macoscope/NotificationController
[x] https://github.com/uber/signals-ios
[x] https://github.com/nevyn/MeshPipe
[x] https://github.com/swisspol/XLFacility
[x] https://github.com/eklipse2k8/FilterKit
[x] https://github.com/hectr/MRWorldMapView
[x] https://github.com/Automatic/Automatic-iOS-SDK
[x] https://github.com/samdmarshall/debugDescription
[x] https://github.com/IFTTT/jot
[x] https://github.com/johnno1962/Xtrace
[x] https://github.com/toco/TOCAssetCatalogBackground
[x] https://github.com/0xced/ABGetMe
[x] https://github.com/andreamazz/AMWaveTransition
[x] https://github.com/andreamazz/UITextField-Shake
[x] https://github.com/FlexMonkey/ShinpuruImage/blob/master/README.md
[x] https://github.com/omichde/BOMFeedback
[x] https://github.com/robotmedia/RMStore
[x] https://github.com/honkmaster/TTOpenInAppActivity
[x] https://github.com/wtmoose/TLLayoutTransitioning
[x] https://github.com/facebook/KVOController/
[x] https://github.com/calebd/CMDQueryStringSerialization
[x] https://github.com/LoganWright/Genome
[x] https://github.com/LoganWright/Polymer
[x] https://github.com/Inferis/IIAutoNSCoding
[x] https://github.com/square/Aardvark
[x] https://github.com/mergesort/UIViewController-Sharing
[x] https://github.com/sarahscott/AREasyBadge
[x] https://github.com/oarrabi/Download-Manager
[x] https://github.com/oarrabi/OAStackView
[x] https://github.com/AriX/MagicKit
[x] https://github.com/qzs21/SSWaveView
[x] https://github.com/edwinbosire/DropMenu
[x] https://github.com/Cocoanetics/DTLoupe
[x] https://github.com/kevinhartman/LiquidKit
[x] https://github.com/gekitz/GKSlidingText
[x] https://github.com/sxyx2008/awesome-ios-animation
[x] https://github.com/patr1ck/Provocateur
[x] https://github.com/sferrini/SFJumpToLine
[x] https://github.com/gemr/sheriff
[x] https://github.com/DavdRoman/Bohr
[x] https://github.com/Draveness/DKNightVersion
[x] https://github.com/Skyscanner/Dixie
[x] https://github.com/freerunnering/HomeKitBridge
[x] https://github.com/DanielTomlinson/DANPromise
[x] https://github.com/antiguab/BAFluidView
[x] https://github.com/MichaelBuckley/MAIKit
[x] https://github.com/Legoless/Alpha
[x] https://github.com/akane/EventListener
[x] https://github.com/Produkt/PDKTModelBuilder
[x] https://github.com/card-io/card.io-iOS-SDK
[x] https://github.com/jumartin/Calendar
[x] https://github.com/pinterest/PINCache
[x] https://github.com/pinterest/ios-pdk
[x] https://github.com/bumaociyuan/zxIpaServer

Not a problem at all... there are way too many to create issues.
Do you want me to add them?

I also have my list of libs....

yeah you or anyone else can add em, if they are good :)

@dkhamsing I checked all of them because I curated and added to my list of libs. Should we close this?

Yes thx


Show offer to buy on collection page

For a collection page, if an inscription in the collection has been listed in the order book show that it is available to buy similar to what is shown in the listings page.
eg. this inscription is listed for sale https://openordex.org/inscription?number=59edb28a4c0c6a9fb319ae6f4e8d5f979b44e786fa127ad012d1eeda83cd944ei0
but it's not reflected on the collections page  https://openordex.org/collection?slug=timechain
I would guess this one might be tricky to implement, as I'm not sure if there is any way to query the nostr relay for only offers which belong to a particular collection?

Done


review C# samples for consistent and correct passing of CancellationToken

review C# samples for consistent and correct passing of CancellationToken

the plan is to make global changes to samples in dotnet will be done after we target 3.1 and botbuilder 4.8


ToC updates twice on mount

afterNavigate(() => {
  update();
  highlight();
});

already calls update(); and highlight(); on mount. No need to do it again in onMount.
Not sure I get it. Does the rest of the onMount function accomplish anything?
onMount(() => {
  // wait for fonts to load...
  const timeouts = [setTimeout(onresize, 1000), setTimeout(onscroll, 5000)];
  return () => {
    window.removeEventListener('scroll', onscroll, true);
    window.removeEventListener('resize', onresize, true);
    timeouts.forEach((timeout) => clearTimeout(timeout));
  };
});

good catch, thanks — the rest of the onMount does serve a purpose, but is currently referencing non-existent functions. fixed in #3878


D_model

thanks for your good work, the D_model would be released for continue train? Another question: Copy T and R images to trainA1 and trainA2? meaning ? how to prepare, it's for initial？

Thank you for your questions. I am a little confused. Can you describe them more clearly？

Thank you for your questions. I am a little confused. Can you describe them more clearly？

Q1: In your readme file, I am confused with the describes, "Prepare some images for synthesis. Copy T and R images to trainA1 and trainA2 separately."
My understanding is the R images are used for initial in train phase, how to prepare the R images for training?
Q2: I want to resume train from your model, but the final_net_D.pth is not provided. The D model would be released? thanks for your reply!

The answer to Q1: Because we don't have enough real data (I and T) to train for SIRR. Before our training, we need to get enough synthetic images I by combining images T and R.  This step is necessary for all research in SIRR.
The answer to Q2: Of course, we will release it soon. Thank you for your reminder.

The answer to Q1: Because we don't have enough real data (I and T) to train for SIRR. Before our training, we need to get enough synthetic images I by combining images T and R. This step is necessary for all research in SIRR.
The answer to Q2: Of course, we will release it soon. Thank you for your reminder.


Add missing env vars

Adds in the env vars that are in the job environment but aren't in the docs. Also added a section for deprecated vars.
I've had a go at descriptions for these new ones, but some of them I couldn't find any info on so they're missing descriptions for now. If either of you could help me fill these in, that would be super! I've also had a guess at some default/examples for them. Let me know if any are inaccurate!

I've added a new section of 'bootstrap configuration variables', and finished off adding in the last few missing vars from the other sections.
If all the descriptions are accurate and the examples are alright, I'd be keen to merge 🚀 In another PR I'll start the addition of the subsets section.

I had a look around at what other companies are doing for their env vars docs, and it's a pretty mixed bag. Some people do a big list in dot points, some people have tables, some people have the vars as headings and the explanations in the body.
I think that dot points work reasonably well, as long as they have nice padding around them (cramped formatting vs spacious formatting).
I like how these two are split up into a page full of headings with more in depth info/examples, and a page with lots of words about setting up and using the vars.
However, I think that purely from a visual-parsing perspective, the table format is the easiest to use. I'm not sure how people are actually using our page though - are they coming here to scan for something they already know exists, or are they reading the descriptions and names to find out what they can do? The answer to this should dictate how we lay out the page.
If people are using the page as a reference, a nicely formatted list of dot points or a table would be great. If people are wanting info about each var and how to use it, having the vars as headings (and not having a toc at the top) would make more sense.
What do you think makes more sense for our audience @toolmantim @lox?

Awesome research!
We might need to have a play with the different formats to see what feels right, but I think I'm leaning towards https://www.terraform.io/docs/configuration/environment-variables.html style environment variable names as subheadings, for what we need. Because similar to ours, the terraform ones can actually change behaviour and have subtle effects that you need to know about.
I'd also guess that if you're trying to achieve a task, you're only going to be using a few of the environment variables, and you'd probably prefer to know as much about them as possible, including multiple examples (which is important if you're writing regular expressions based on expected/possible values). So more details on each is probably better… even if half of them only have a one sentence description and one or two examples.

@toolmantim  I had a go at laying the variables out in the headings/body style, and it looks alright. I think maybe we'd need to figure out some different kind of styling for the variable headings to make the page a little more readable?
Here's what the top of the page looks like now, which is alright:

✨🌱🇦🇺 (emojis to separate these very same-y sections) But then the different sections blur together a bit and it's a bit confusing further down:

I think that works well as headers with a paragraph!
What did you think of removing the separate sections though, and just having one list? And then for each one covering the stuff I suggested earlier:

What's it called? (name) - done!
What is it? (description) - done!
Can I change it? (setting) And if so, where? Places you can change env vars include build env (what you'd provide in the "New Build" dialog), pipeline steps, hooks.

If we're going for just one big list, I'd like to add another bit in that says where that var comes from i.e. from the agent or the bootstrap or wherever it's set. I think that would be useful info, what do you think?

I had a go at merging some of @lox's info from his env vars post into each of the descriptions and I think it's worked reasonably well!
Here's an example of one of the vars that used to be in the Agent section:

I haven't put any extra bits in with the original 'job' variables, maybe that could say something about being able to set it in pipeline env etc.

I like that @harrietgrace, could we have standard thing of whether variables are "read only" or not and a link to what that means so we don't have to duplicate that info?
I think I like "read only" vs "protected".

@toolmantim I got quite confused with "The value cannot be modified" in your example, it took me ages to figure out how you meant it 😅I thought it sounded like the value was set by Buildkite and can't be changed by the user at all (even after saying that it come from the agent config option - hence my confusion).
I've spent like an hour trying to come up with alternatives, but all I've got so far is:
"The value cannot be modified as an environment variable" - giving a little more context and making it clearer that it can be modified in the config file.
"This variable is read only" - trying to explain that it's modifiable as a config option, but not when it's an env var.
I'm not happy with either of these, but I worry that other people will have the same confusion as I did with the simpler "the value cannot be modified".

Yeah, I struggled with it too! “The value cannot be modified as an environment variable” is probably good enough? I figured the other agent configuration document would give them more confidence than this paragraph, even if there was doubt.

It looks like there's only a few different arrangements of content in the variable descriptions:

Has a description, can't be modified

Has a description, can be set in hooks

Links to a description, can't be modified

Is the opposite of the linked description, can't be modified

Questions:
Do we want to differentiate the ones that come from the bootstrap environment? Here's what I've got for those ones so far:

Do we want to mention anything about security in this secret key one? Also these ones have a bit about how you can set them in the pipeline env too.

I think the ability to go deep on each one is really great! They're looking ace.
The agent access token one perhaps should mention…

The variable is read by the agent’s artifact and meta-data commands.

(you need to make sure you don't filter them out… say, with a Docker container, and still hope the commands work)

Do we want to mention anything about security in this secret key one? Also these ones have a bit about how you can set them in the pipeline env too.

We definitely don't want to encourage people to be setting secrets in the pipeline env. I don't think we even want to mention it.
Maybe we don't need to differentiate the bootstrap ones with a generic sentence, but actually explain what the specific situation is with each one? For example, those two are both related to using private s3 buckets too, but we don't mention it.
What about something like:

The secret access key for your S3 IAM user, for use with private S3 buckets. The variable is read by the buildkite-agent artifact upload command, and during the artifact upload phase of command steps. The access key must be set by exporting the environment variable in the environment, pre-checkout or pre-command hooks.

Maybe we don't need to differentiate the bootstrap ones with a generic sentence, but actually explain what the specific situation is with each one?

That sounds cool! I'll definitely need some help with those bits though, everything I know about these vars is already in the doc 😅These are the ones I've listed as being "read from the bootstrap": https://gist.github.com/lox/98f9b16b2ec6a7092f746556de50e72f#bootstrap-run-time-configuration
@toolmantim I've added in your words for those two specific ones in your previous comment. @sj26 I've added your GitHub deployment vars in too 😊

Happy to help get all those 9 bootstrappy options documented. If you wanna go and take a first stab, and then @ @lox and I on which ones need looking at / documenting, we'll get those done. And then we might be close to 🚀?? 🎉🙌🏼

I've just noticed that there's a slew of 'opposite' style env vars that I haven't added, I'm guessing we want to include all these? We've already got BUILDKITE_NO_LOCAL_HOOKS.
BUILDKITE_NO_SSH_KEYSCAN
BUILDKITE_NO_COMMAND_EVAL
BUILDKITE_NO_PLUGINS
BUILDKITE_NO_PLUGIN_VALIDATION
BUILDKITE_NO_GIT_SUBMODULES,
BUILDKITE_NO_HTTP2
There's a bunch of other ones from the buildkite-agent start config section of @lox's document that I haven't added either 😬

In the next Buildkite Agent release, none of the configuration environment variables will show up in builds anymore. Maybe we shouldn't include them in the doc now either?

The next point release or major release? If it's happening imminently, then totally happy to leave them out 👍🏻

It's in 3.9.0, which was just released @harrietgrace

Question: Why the backslashes before the underscores in the headers?
e.g ### BUILDKITE\_AGENT\_ACCESS_TOKEN

Rather than "The value cannot be modified.", perhaps "Set by the agent"? Technically, it CAN be modified, it just doesn't do anything.

@lox It's parsed as markdown now that they're not in an html table, so if we don't escape the underscores it ends up partially italicised 😅They don't all need escaping, we just can't have two either side of a word.

Rather than "The value cannot be modified.", perhaps "Set by the agent"? Technically, it CAN be modified, it just doesn't do anything.

But won't any hook modification the env be reverted? I think for all intents and purposes it can't be modified in a way that gives it any affect.

But won't any hook modification of the env be reverted/ignored? I think for all intents and purposes it can't be modified in a way that gives it any affect.

Yup, correct! The agent will reset this env every step and changing the value won't cause any agent config changes. I just find the statement "it can't be modified" confusing and ambiguous, but that could just be me. "Changes to this environment variable don't have any effect"? Up to you, it just didn't quite ring right to me, but I'm pretty deep in that world.

@lox I had trouble with the phrasing too, it took me ages of brainstorming other ideas to come around to @toolmantim's "the value cannot be modified".
Another idea I had over the weekend was: "Changes will not be persisted" or something to that effect? Or even mention that we're the ones who set it, in a 'you can't set this' kind of way - "Variable set by Buildkite, changes will not be persisted".

Let's video pair on this one! 🎉


Size attribute on object causes problems

An object that I'm trying to serialize has an attribute called size. This is causing the error undefined method 'each'.
I'm assuming this is related to https://github.com/Netflix/fast_jsonapi/pull/234 which is checking for the existence of a method called size to determine whether the object is a collection.

Hey @blim8183,
Your analysis is correct. It is definitely due to the code in #234.
One quick way for you to solve this is to force is_collection to be false, so like,
options = {}
options[:is_collection] = false
hash = YourSerializer.new(your_record, options).serializable_hash

This should work for you. Your case is really an edge case, and the ability to force is_collection was added especially to counter such kind of cases. Let me know if it works! :)

@blim8183 can you close this if the above response solved your problem?

@manojmj92 What do you think about adding a note about this in the readme? I've run into this a few times now (I work with a lot of file meta data). I feel like size, though not super common, is common enough to warrant a footnote at least.


ScatterJS.identity is null without auto-filled when logged in and then refreshing page

hi, maybe it's solved issue before, but i didn't find a solution for me.
I tried to use Scatter Desktop with my website. I used latest scatter-js, eosjs@16
I noticed - "After a successful login, the "Identity" will be available at ScatterJS.identity. If a user refreshes the page and has already logged in, the ScatterJS.identity property will be auto-filled.". However, I found that it works when I used Chrome Extension plug-in, if I disabled Extension plug-in, and used Scatter Desktop, it can login/logout via Scatter Desktop correctly without refresh the page, but if logged in and refreshes the page, ScatterJS.identity will be null again (without auto-filled)
Does the ScatterJS.identity auto-filled for Scatter Desktop, needs local connection between browser and scatter desktop?
I think this might just be a timing issue. Can you login, and then refresh the page, and then set a timeout for checking the identity after doing ScatterJS.connect(). Let's see if delaying the console.log shows the identity loaded.

It definitely does use a local websocket for connections, but if those were all the errors you got then it looks like it found ports 50005 and 50006.
I think this might just be a timing issue. Can you login, and then refresh the page, and then set a timeout for checking the identity after doing ScatterJS.connect(). Let's see if delaying the console.log shows the identity loaded.

@nsjames, ya, correct, it's about timing. thanks for the answer.

Same issue, but i can't get identity after delay. @fire-xyz Can you share sample code ?

I use the latest scatter-js and<EMAIL_ADDRESS>

Same issue, but i can't get identity after delay. @fire-xyz Can you share sample code ?

for my case, before I get value of scatter.identity, i need to await the connection, for example:
await checkConnect()
return ScatterJS.identity
async checkConnect(){
    this.connected = await ScatterJS.connect('scatter', {Network:this.network})
    return this.connected
}

@fire-xyz Thanks you.
I can't get identity with this way, too. maybe it's because I use the scatter-js2 and<EMAIL_ADDRESS>

@fire-xyz Thanks you.
I can't get identity with this way, too. maybe it's because I use the scatter-js2 and<EMAIL_ADDRESS>
I also updated to<EMAIL_ADDRESS>it works fine
Maybe you can post

@fire-xyz Thanks you.
I can't get identity with this way, too. maybe it's because I use the scatter-js2 and<EMAIL_ADDRESS>
I also updated to<EMAIL_ADDRESS>it works fine. I should use the same libs as yours. maybe you can post your codes

I use the code below, when the page init use the pageInit function to connected to scatter and get login identity state. when user click login button then will use scatterLogin function to login.
It's wried that i can't identity after refresh the page.
import ScatterJS from '@scatterjs/core'
import ScatterEOS from '@scatterjs/eosjs2'
import { Api, JsonRpc } from 'eosjs'
import { BehaviorSubject, from, Observable, of, throwError } from 'rxjs';
import { map, switchMap } from 'rxjs/operators';

ScatterJS.plugins(new ScatterEOS())

const network = ScatterJS.Network.fromJson({
  blockchain: 'eos',
  chainId: 'aca376f206b8fc25a6ed44dbdc66547c36c6c33e3a119ffbeaef943642f0e906',
  host: 'nodes.get-scatter.com',
  port: 443,
  protocol: 'https'
})

class EosProxyService {
  private rpc = new JsonRpc(network.fullhost())
  private eosAccount$: BehaviorSubject<any> = new BehaviorSubject(undefined)
  private eos: any

  pageInit() {
    ScatterJS.connect(
      'MyApp',
      { network }
    )
      .then(connected => {
        if (connected) {
          setTimeout(() => {
            if (ScatterJS.identity) {
              // update eos account
              this.eosAccount$.next(ScatterJS.account('eos'))
              // get eos handle
              this.eos = ScatterJS.scatter.eos(
                network,
                Api,
                {
                  rpc: this.rpc,
                  beta3: true
                }
              )
              ;(window as any).ScatterJS = undefined
            }
          }, 2000)
        }
      })
      .catch(() => {})
  }

  scatterLogin (): Observable<boolean> {
    return from(
      ScatterJS.connect(
        'MyApp',
        { network }
      )
    ).pipe(
      switchMap(connected => {
        if (!connected) {
          return throwError({
            type: 'connected-fail',
            message: 'Connect to Scatter Failed!'
          })
        }
        return (!ScatterJS.identity
          ? from(ScatterJS.login())
          : of(ScatterJS.identity)
        ).pipe(
          map(() => {
            // update eos account
            this.eosAccount$.next(ScatterJS.account('eos'))
            // get eos handle
            this.eos = ScatterJS.scatter.eos(
              network,
              Api,
              {
                rpc: this.rpc,
                beta3: true
              }
            )
            ;(window as any).ScatterJS = undefined
            return true
          })
        )
      })
    )
  }
}

It's wried that when i change my page to http protocol then I can get the identity after refresh the page.


IBM Verse Mac App: unable to open Calendar link

Description
I'm using the IBM Verse Mac App and trying to open my Calandar using the Calendar link (as well as the calendar icon).  When I do, I received the following error message:
Sorry, we cannot find the page you requested.
A link may be broken, or the Web address may be incorrect.
Return to Home
1404-404
All other links work as expected.
Steps to reproduce issue

Open IBM Verse Mac App.
Click on the Calendar link at the top of the UI.  or click on the Calendar icon on the lower right of the UI.

Details

Are you nativefying a public website?
Feature request? Have you looked at nativefier --help to see if an existing option could fit your needs?
Full nativefier command used to build your app: <command here>
Version of Nativefier (run nativefier --version): v0.0.0
Version of node.js (run node --version): v0.0.0
OS: <OS here>
Error message / stack trace (if any):

<Error message / Stack trace here>

@mdking117 hi. You are posting here an issue about Nativefier, not the IBM Verse app. I infer IBM is using Nativefier to package their website, but we Nativefier maintainers have nothing to do with it. Please get in touch with the IBM Verse team. Good luck 🙂.

Thanks for the info.  On the app itself, they have a Report an Issue menu item that leads to this github.


[ngxtension] No @angular/core detected

Hi,
I have an Angular 17.1 running inside an Electron 24. To minimize the package build size, most deps are installed as dev deps, including all angular packages, because they are only required to build the app.
I'm assuming that ngxtension is looking for "@angular/core" under "dependencies" and not "devDependencies".
Is this the case?
Thanks!

Does ngxtension not install? Do you have any errors?

@eneajaho Sorry for the confusion, yes, it installs fine. This happens when I run a generator. When I move the package dependency from "devDependencies" to "dependencies", it works as expected.
I would expect it would check both lists to find whatever it needs to run instead of giving this (technically) false positive.

FYI
If you run the following, it gets installed properly:
npm i ngxtension -D
ng add ngxtension

You'll get a message from ng that it's already installed, but it will configure tsconfig.json and set skipLibCheck as expected.

I'm getting the asme as the user above, even after your proposed solution:
`➜  ct-sbip-v4 git:(develop) ✗ ng add ngxtension
Skipping installation: Package already installed
Initializing ngxtension...
Turning on skipLibCheck...
UPDATE tsconfig.json (980 bytes)
➜  ct-sbip-v4 git:(develop) ✗ ng g ngxtension:convert-signal-inputs
[ngxtension] No @angular/core detected
➜  ct-sbip-v4 git:(develop) ✗ `
i'm running it on angular 17.3.6, I can see the tsconfig and the package.json changes, still no luck running the scripts


How can we get info about current video that is being processed

How can we get info about current video that is being processed
Suppose we have 3 videos, how can we check currently which video is being processed.
Is there a listener for this.

(Pasted to all my answers today: it's been a long time since my last issues review. I am sorry about the delay and I know you likely have moved on by now. Still, I'm going to answer where I can)
No, but it could be added.


[8.x] HTTP client: only allow a single User-Agent header

Currently, using this method appends the specified UserAgent string to the headers list. Calling this method twice will send a request with two User-Agent header lines. This is not accepted as valid by a lot of services.
See also.
This PR overrides the User-Agent header instead of appending it.

Heya, as discussed in the other PR, we're wondering if it even makes sense to append any header like that. Maybe we should patch the withHeaders itself? In any case, wait a bit until @taylorotwell has replied to this.

According to this SO answer it should technically be possible.
But I think it only makes sense for a limited set of headers (like Cache-Control).


Consul V2 Catalog Service Mesh support for host-based and subset level splitting

Consul 1.17 introduces v2 API that will allow host based splitting, where-as previously only subset level splitting was available before.
With Argo Rollouts we need both blue green and canary deployment strategies supported for Consul Service Mesh. In addition we should allow the following granularity within those strategies:

Host level splitting
Subset level splitting
TCP route splitting

We should implement those changes in the ArgoCD project so that we can use native Argo Rollout CRDs to deploy those strategies. An example is shown below for a Canary deployment.
apiVersion: argoproj.io/v1alpha1
kind: Rollout
metadata:
  name: rollout-example
spec:
  ...
  strategy:
    canary:
      canaryService: canary-svc  # required
      stableService: stable-svc  # required
      trafficRouting:
        consul:
          service:
            name: rollout-consul-svc   # required
            routes:
            - primary            # optional if there is a single route in VirtualService, required otherwise
      steps:
      - setWeight: 5
      - pause:
          duration: 10m

Closing for now.


Increase default health check delay values: InitialDelaySeconds for readinessProbe and livenessProbe

By default, the values of InitialDelaySeconds for readinessProbe and livenessProbe were set to 0. periodSeconds was set to 10.

This means that the health checks are done as soon as the container starts running. If adding this health check is best practice, it is also good to let some time for the application to start running properly before doing the health check, since a bad response can be read by kubernetess cluster as a failure and it will restart the container.
A period of 10 seconds between every health check seems too short for most use-cases. A check every minute seems more appropriate as a default.

These values are of course modifiable on the values.yaml file, but having more appropriate values seem better for most use-cases and for users who are not tuning these parameters.

Thanks @curquiza
Bors merge


iOS New Architecture Crashing

Bug description:
Our react native project is on version 0.72.6 and uses the new architecture. We're rendering a webview that returns a mapbox map with injectable HTML and javascript. It's a fairly complex and likely atypical use of webview.
I'm attempting to upgrade from react-native-webview version 13.6.3 to 13.8.1.
The reason for this is that in 13.6.3 in iOS there is a bug that occasionally occurs when we dismiss the webview that causes all screen elements rendered prior to the webview to become un-touchable. The app appears frozen except for elements rendered after the webview, which work as expected.
This bug is hard to reproduce and not the subject of this bug report, it's just the reason for the upgrade.
The real issue is that when I upgrade the webview, it crashes immediately with the following code:
W0315 10:45:06.626423<PHONE_NUMBER> UIManagerBinding.cpp:124] instanceHandle is null, event will be dropped
*** Terminating app due to uncaught exception 'NSInvalidArgumentException', reason: '-[RNCWebViewImpl destroyWebView]: unrecognized selector sent to instance 0x1289fea00'
*** First throw call stack:
(0x18d336678 0x1855c7c80 0x18d3c6fdc 0x1902a9828 0x1031f5980 0x18d27bb34 0x18d3bc950 0x1031ec5b8 0x102afa300 0x102afa114 0x102b0c7b8 0x102d8cb70 0x102b0ba10 0x102b0b964 0x1952486a8 0x19524a300 0x195258998 0x1952585b0 0x18d280f9c 0x18d27dca8 0x18d27d3f8 0x1d08334f8 0x18f6a38a0 0x18f6a2edc 0x102835380 0x1affe6dcc)
libc++abi: terminating due to uncaught exception of type NSException

To Reproduce:
I'm pretty confident it's this exact same bug: https://github.com/react-native-webview/react-native-webview/issues/3043
Expected behavior:
The app probably shouldn't crash.
Screenshots/Videos:
N/A
Environment:

OS: iOS
OS version: 17.3.1
react-native version: 0.72.6
react-native-webview version: 16.8.1

Hi @connorpmullins, this problem is related to React Native, a fix was release with RN 0.73.5 but it has not yet backported in RN 0.72.
Check here if you can


NMSSH Framework works on simulator but not on real iPhone

Hello,
I could use NMSSH and it works for iPhone simulator but when I try an actual real iPhone, I get the following error:
(I tried setting the valid architectures to all architectures e.g., arm64, i386, x86_64, etc ) and (also tried to set Build Active Architecture Only to NO) for both my project and when I want to build NMSSH.Framework but still get the same following error when I use a real iPhone:
ld: warning: ignoring file ###/Desktop/XcodeProjects/MYAPPNAME/NMSSH.framework/NMSSH, file was built for x86_64 which is not the architecture being linked (arm64): #/Desktop/XcodeProjects/MYAPPNAME/NMSSH.framework/NMSSH
Undefined symbols for architecture arm64:
"OBJC_CLASS$_NMSFTPFile", referenced from:
objc-class-ref in ViewController.o
"OBJC_CLASS$_NMSFTP", referenced from:
objc-class-ref in ViewController.o
"OBJC_CLASS$_NMSSHSession", referenced from:
objc-class-ref in ViewController.o
ld: symbol(s) not found for architecture arm64
clang: error: linker command failed with exit code 1 (use -v to see invocation)
Any suggestions for this would be appreciated.
Thanks.

Anyone any suggestion on this?
Still it works for simulator but returns the above error for real phone.

I answered to the same question here: https://github.com/NMSSH/NMSSH/issues/253


Prometheus support for metric collection

I want to check if there are there any plans for supporting Prometheus metrics export? If no one is working on it I can also try and raise a PR for the same.

https://github.com/afex/hystrix-go/pull/91
+1. This existing PR seems to cover the common use-cases.


Remove JDK requirement from quickstarts

We're using docker so you don't need to have Java installed to run the quickstarts

https://github.com/stargate/docs/pull/29


🪲 BUG REMOVE: Download Button

Hello @aniketsen123
Features added

Congratulation, your download Button bug is removed now. And now you can update APK file from time to time from google drive.
Add the New images of the application

@aniketsen123
Why do you label this as a medium?
In issue #11  you labeled this as hard, and I resolved the bug properly.

@Suryansh1720001 sorry, my bad

@aniketsen123
Thank You


Upgrade to Inventory 0.1.0.Final

@jmazzitelli the last PR for dev/inventory-0.1.0.Alpha3. Feel free to merge dev/inventory-0.1.0.Alpha3 to master and release.


Interactive Measurements (and type-erased component types)

Closes #80.
I kept this as a fairly conservative change. Most of the code is in separate modules (any and interactive). Aside from a few trait bounds, the existing types and logic stay the same for non-interactive Measurements and Transformations.
If/when we're happy with this, we can migrate the FFI wrappers to use the type-erased (AnyXXX) approach in #88.

Created type-erased versions of Measurement/Transformation component types.
Added type-erasing transformers to Measurement/Transformation.
Defined generic Queryable abstraction.
Implemented make_adaptive_composition() constructor.
Added unit tests.

I realized there are some issues with domains in adaptive composition, putting this back to draft.

I separated the concepts of type erasure (AnyXXX) and polymorphism. There's still more cloning than I'd like, but the logic is cleaner anyway.

Thanks Andy! The type erasure in particular is really impressive. Looking forward to dropping Ffi-specific types in favor of the type-erased versions of the originals.


Publish on F-Droid

What is F-Droid?
F-Droid is an installable catalogue of FOSS (Free and Open Source Software) applications for the Android platform. The client makes it easy to browse, install, and keep track of updates on your device.

https://f-droid.org/

Tindroid uses Crashlytics and it's a problem for F-Droid:
https://f-droid.org/en/docs/Inclusion_Policy/
I find Crashlytics useful for catching bugs in the wild. I would rather not remove it from the Playstore release. But maintaining different code base for different stores takes an effort. If you are willing to help then let's discuss. Otherwise it will have to wait for a volunteer.

Tindroid uses Crashlytics and it's a problem for F-Droid:
https://f-droid.org/en/docs/Inclusion_Policy/
I find Crashlytics useful for catching bugs in the wild. I would rather not remove it from the Playstore release. But maintaining different code base for different stores takes an effort. If you are willing to help then let's discuss. Otherwise it will have to wait for a volunteer.


python311Packages.dash: 2.14.2 -> 2.15.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.dash is: Python framework for building analytical web applications
meta.homepage for python311Packages.dash is: https://dash.plot.ly/
meta.changelog for python311Packages.dash is: https://github.com/plotly/dash/blob/refs/tags/v2.15.0/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.15.0 with grep in /nix/store/vbzkihz94z1z7azihs37rvhpyhwkm5w1-python3.11-dash-2.15.0
found 2.15.0 in filename of file in /nix/store/vbzkihz94z1z7azihs37rvhpyhwkm5w1-python3.11-dash-2.15.0

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

4 package rebuild(s)

First fifty rebuilds by attrpath
das
picosnitch
python311Packages.dash
python312Packages.dash

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/vbzkihz94z1z7azihs37rvhpyhwkm5w1-python3.11-dash-2.15.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python311Packages.dash https://github.com/r-ryantm/nixpkgs/archive/5e0c79009351e7645463e9cec717e2d2d7321d83.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/5e0c79009351e7645463e9cec717e2d2d7321d83#python311Packages.dash

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/vbzkihz94z1z7azihs37rvhpyhwkm5w1-python3.11-dash-2.15.0
ls -la /nix/store/vbzkihz94z1z7azihs37rvhpyhwkm5w1-python3.11-dash-2.15.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages marked as broken and skipped:

python312Packages.dash
python312Packages.dash.dist

2 packages failed to build:

das
das.dist

4 packages built:

picosnitch
picosnitch.dist
python311Packages.dash
python311Packages.dash.dist

Maintainer pings
cc @antoinerg @TomaSajt for testing.

ansi2html should be removed from inputs, see https://github.com/plotly/dash/pull/2721

Result of nixpkgs-review pr 285344 run on aarch64-darwin 1

2 packages marked as broken and skipped:

python312Packages.dash
python312Packages.dash.dist

4 packages built:

das
das.dist
python311Packages.dash
python311Packages.dash.dist


Contracting isolation

This change is

Pull Request Test Coverage Report for Build 2744

256 of 314 (81.53%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 72.753%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/openprocurement/api/validation.py
1
3
33.33%

src/openprocurement/api/utils.py
4
17
23.53%

src/openprocurement/api/models/schema.py
239
282
84.75%

Totals

Change from base Build 2743:
0.1%

Covered Lines:
1797

Relevant Lines:
2470

💛  - Coveralls


Support .yaml extension for base16 themes

Most base16 themes use .yaml instead of .yml for the file extension.
Before, the condition to check a color option for a yaml file only
matched on .yml but not on .yaml. Now it matches on both, just in
case.

Updated regex
Added spec and test file with yaml extension

This is great, thank you! 🙌

Released in v3.1.1 👍


[fix](invert index) ensure that the pred result sign of the inlist is…

… in order
Proposed changes

If the pred_result_sign generated by inlist with inverted index is out of order, the result will be wrong

run buildall

run buildall

run buildall

run buildall


[Improvement] replace the unmaintained markdown-builder package

Describe the improvement
The markdown-builder package is no longer maintained since 2019.
It introduces a redundant legacy version of husky in its dependencies, which pollutes the dependency tree:
https://github.com/callstack/reassure/blob/04825689671482beca6f90010436c44c878076e3/yarn.lock#L6318-L6323
Although it could be resolved by the "resolutions" feature of some package manager, I think it could be better to replace it totally from the @callstack/reassure-compare package.
I found there is a fork mr_markdown_builder and maybe we can make use of it directly?

I've did a quick check where do we use markdown builder and it's pretty minimal. We just use headers.h1, headers.h3 emphasis.b and emphasis.i. Replacements for this should be easy to add to reassure-compare directly. @meteorlxy would you like to contribute it?


Need more info on behaviour and usage of stateful containers

It's a bit unclear how a container which requires persistent state (e.g. a sql container) will behave - will the data be stored in a permanent fashion anywhere? How would one go about backing up the data from an instance like this?

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

ID: d0615dff-cbdd-0583-739e-72eadbb629bf
Version Independent ID: c927ed72-dad6-4265-758a-cb6f5d0f4256
Content: Deploy custom image, multi-container, or built-in image - Azure App Service
Content Source: articles/app-service/containers/choose-deployment-type.md
Service: app-service
GitHub Login: @msangapu
Microsoft Alias: msangapu

@mcintyre321 Thank you for your feedback . We will investigate and update this thread.

@mcintyre321 , We recommend to use Azure SQL/Azure DB for MySQL to customers.
There are a lot of articles explaining the problems you might face (for example https://vsupalov.com/database-in-docker ) and why you should not run PROD DBs in a container (specially if you use a Container Orchestrator that might move your containers around)

In your samples there are application with mysql and redis containers. Are
these samples there to demonstrate what can be done, rather than what
should be done?
On Mon, 25 Mar 2019, 12:23 DashleenBhandari-MSFT<EMAIL_ADDRESS>wrote:

@mcintyre321 https://github.com/mcintyre321 , We recommend to use Azure
SQL/Azure DB for MySQL to customers.
There are a lot of articles explaining the problems you might face (for
example https://vsupalov.com/database-in-docker ) and why you should not
run PROD DBs in a container (specially if you use a Container Orchestrator
that might move your containers around)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/27591#issuecomment-476173120,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAQ0-kJRx0fBFH8ggUaIVYclC3bcTuWLks5vaL_MgaJpZM4b-3VE
.

Hi @mcintyre321 Thank you for your feedback. Can you share the URL to the sample(s) you're referring to?

Hi, this is a screenshot from the Azure portal. As you can see, the sample container app includes an mssql container.

Hi @mcintyre321. This area is to discuss feedback on this article and it's not the correct place for portal feedback. You should provide your feedback through the portal/support to ensure it gets to the correct team.
For persistent storage, see the Application Setting here: https://docs.microsoft.com/en-us/azure/app-service/containers/app-service-linux-faq#custom-containers .
If you'd like to create a production-ready multi-container app, you can follow the steps here: https://docs.microsoft.com/en-us/azure/app-service/containers/tutorial-multi-container-app
The above tutorial is for WordPress and MySQL, but will give you an idea of the workflow to configure MSSQL for production in a multi-container scenario. I suggest using the Docker Compose configuration.
Let me know if you have additional concerns.

Now I'm confused. Your colleague @DashleenBhandari-MSFT says "you should not run PROD DBs in a container", but you suggest the mysql container is prod ready?
How is the mysql container data persisted? It looks like tutorial says how to back the wp container is backed by disk, but not the mysql instance (or am i missing something?)

Hi @mcintyre321 The tutorial uses Azure DB for MySQL. Please take some time to review it properly.

Thanks - it would be good to have some explicit text describing the lifecycle of the containers, so that we can make judgements about which containers it is sensible to run, and which should be replaced by PAAS, or backed by disk.
Also some information about how the containers will interact (or not) in multi-instance app services. E.g. Redis is less useful than in a distributed setup if you have a single, non-persistent instance per web server.

@mcintyre321 Thank you for the feedback. We will consider it when we update this article.

#please-close


[Feature request] callbacks on recognitions of elements

Might I suggest implementing some callback mechanism for successful element recognition. A simple function or a visitor-pattern reporting type and name etc.
A visitor-pattern comes to mind. Simple, effective, flexible.
https://sourcemaking.com/design_patterns/visitor
interface JsonSchemaVisitor
{
    public checkedInt( $value, $name, $path );
    public checkedDouble( $value, $name, $path );
   ....
}
$visitor = new JsonSchemaVisitor();
$validator->check($data, $schema, $visitor);

blabla etc. etc. :)

@patricksavalle PRs are always welcome.


alt+m not opening MarkdownLivePreview

alt+m doesn't open a preview. I can use the Command Palette to open it, but it's much slower. I've reinstalled MarkdownLivePreview with the same results. I'm on a Windows 10 machine. Sublime Text Ver. 3.2.2, Build 3211

I released a new version, and I think this shortcut annoyed a few people. So I have decided to remove it, and let people choose for themselves. Though I forgot to put that in the README... If you liked this shortcut, all you have to do is add this to your keybinding file:
{
  "keys": ["alt+m"],
  "command": "open_markdown_preview"
}

Thanks!

Thanks！


Multi-GPU support in GPUPredictor.

GPUPredictor is multi-GPU
removed DeviceMatrix, as it has been made obsolete by using HostDeviceVector in DMatrix

Closes #3756

@canonizer Can you add a test for multi-GPU prediction? I am about to add a multi-GPU slave worker to the Jenkins CI server. The multi-GPU tests will run as a separate task than single-GPU tests.

Any updates on multi-GPU tests?

Added a multi-GPU test for GPUPredictor and addressed reviewers' comments.

Okay, I fixed a bug that's caught by my profiling script.  But I can't reproduce the one on Jenkins.

@hcho3 Thanks!  I ran cuda-memcheck and Sanitizer with no luck so far.  The test requires very small amount of memory.

I compiled and ran this pull request on my p2.8xlarge instance and got the same error. I will run it through cuda-gdb to see if it helps.

@trivialfis @canonizer I got this backtrace by running testxgboost through gdb:
#0  __memmove_avx_unaligned () at ../sysdeps/x86_64/multiarch/memcpy-avx-unaligned.S:136
#1  0x00007ffff5958bdf in ?? () from /usr/lib/x86_64-linux-gnu/libcuda.so.1
#2  0x00007ffff5a123ee in ?? () from /usr/lib/x86_64-linux-gnu/libcuda.so.1
#3  0x00007ffff5a126ed in ?? () from /usr/lib/x86_64-linux-gnu/libcuda.so.1
#4  0x00007ffff5a139b6 in ?? () from /usr/lib/x86_64-linux-gnu/libcuda.so.1
#5  0x00007ffff59256be in ?? () from /usr/lib/x86_64-linux-gnu/libcuda.so.1
#6  0x00007ffff59259d8 in ?? () from /usr/lib/x86_64-linux-gnu/libcuda.so.1
#7  0x00007ffff5a761d5 in cuMemcpy () from /usr/lib/x86_64-linux-gnu/libcuda.so.1
#8  0x000000000088f022 in cudart::driverHelper::memcpyDispatch(void*, void const*, unsigned long, cudaMemcpyKind, bool) ()
#9  0x000000000086f896 in cudart::cudaApiMemcpy(void*, void const*, unsigned long, cudaMemcpyKind) ()
#10 0x0000000000891ed8 in cudaMemcpy ()
#11 0x0000000000779b52 in xgboost::predictor::GPUPredictor::DeviceOffsets () at /home/ubuntu/xgboost/src/predictor/gpu_predictor.cu:234
#12 0x00007ffff712d43e in ?? () from /usr/lib/x86_64-linux-gnu/libgomp.so.1
#13 0x00007ffff6cf26ba in start_thread (arg=0x7ffff25d7700) at pthread_create.c:333
#14 0x00007ffff6a2841d in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:109

Maybe this line is problematic?
https://github.com/dmlc/xgboost/blob/10bc710098f75298c3c3ca55b568c623cd5a508f/src/predictor/gpu_predictor.cu#L233-L236

I added some diagnostic printout before the cudaMemcpy line:
[08:44:06] /home/ubuntu/xgboost/src/predictor/gpu_predictor.cu:233: cudaMemcpy(&offsets[0 + 1], 0x7c136c0200 + 2 - 1, sizeof(size_t), cudaMemcpyDefault));
[08:44:06] /home/ubuntu/xgboost/src/predictor/gpu_predictor.cu:233: cudaMemcpy(&offsets[1 + 1], 0x7c13ec0200 + 2 - 1, sizeof(size_t), cudaMemcpyDefault));
[08:44:06] /home/ubuntu/xgboost/src/predictor/gpu_predictor.cu:233: cudaMemcpy(&offsets[2 + 1], 0x7c13ac0200 + 2 - 1, sizeof(size_t), cudaMemcpyDefault));
[08:44:06] /home/ubuntu/xgboost/src/predictor/gpu_predictor.cu:233: cudaMemcpy(&offsets[3 + 1], 0x7c132c0200 + 2 - 1, sizeof(size_t), cudaMemcpyDefault));
[08:44:06] /home/ubuntu/xgboost/src/predictor/gpu_predictor.cu:233: cudaMemcpy(&offsets[4 + 1], 0x7c142c0200 + 2 - 1, sizeof(size_t), cudaMemcpyDefault));
[08:44:06] /home/ubuntu/xgboost/src/predictor/gpu_predictor.cu:233: cudaMemcpy(&offsets[5 + 1], 0x7c18380000 + 1 - 1, sizeof(size_t), cudaMemcpyDefault));
[08:44:07] /home/ubuntu/xgboost/src/predictor/gpu_predictor.cu:233: cudaMemcpy(&offsets[6 + 1], 0 + 0 - 1, sizeof(size_t), cudaMemcpyDefault));

The last cudaMemcpy call fails because the device pointer for shard 6 is null.

@hcho3 This again comes down to changing parameters.  We need to handle the situation when n_gpus is limited hence changed by the n_rows of input data.

@canonizer , @hcho3 , @RAMitchell  I tried to overcome it with a check to see if GPUSet is changed, if so all DeviceShards are re-built.  It might not be a nice solution, suggestions are welcomed.

@hcho3 Hi, could you give another look before I merge it?

Thanks everyone!


fix: reenable structure eta during tc search

Fixes #2074  Structure eta is for TC instances is required to see that subobject projections are defeq with constructor applications.  These diamonds occur with fine-grained instance hierarchies like the algebraic hierarchy in mathlib, and they need to be defeq during TC search if the instances occur as a parameter to another type class.
Structure eta was disabled as a quick fix for the issue reported at https://leanprover.zulipchat.com/#narrow/stream/270676-lean4/topic/.60constructor.60.20and.20.60Applicative.60/near/279984801
However the example in that issue had two non-defeq Functor instances.  We do not support non-defeq instance diamonds in general and lots of other modules will break with them.  For example simp lemmas won't fire (because the instances don't match), the elaborator will error out, etc.  Therefore I have decided to change the example from that issue to remove the non-defeq diamond, and reenabled structure eta during TC search.

!bench

Thanks Gabriel!


Update rds, sts to 2.26.7 in scala-steward-dependencies

About this PR
📦 Updates

software.amazon.awssdk:rds
software.amazon.awssdk:sts

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

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

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

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

Superseded by #866.


Is the Captcha in all languages?

The captcha appears in Spanish even though the Service Catalogue language is English. Does it depend on the IP or it is always in Spanish? If it is the second one, it would be better to have it in English.

I guess it is shown the language you have configured the browser. We will try to make it match with the language from the selected by the user at the moment in the platform.

The cpatcha language cannot be changed dynamically due to problems on the way it works.
https://github.com/VividCortex/angular-recaptcha/issues/66


Logic for executing the queries

Given a list/slice/map of queries to be ran, I want to execute those against a live cluster.

Write logic for querying kubernetes for the requested resources to be examined.
Examine resources for specific knowledge
Capture fail/pass state

This logic was implemented in: https://github.com/defenseunicorns/compliance-auditor/pull/18 through the use of the kyverno applyHelperCommand function.


Make it possible to reference environment variables from environmentVariables section.

Make it possible to reference environment variables from environmentVariables section.
That is, what we want to do is the following, but the SCRIPTS_DIR environment variable cannot be defined based on other environment variables.
---
"$extends":
  - base/normal.json
when:
  environmentVariables:
    - SCRIPTS_DIR: "${COMMANDUNIT_HOSTFSROOT_MOUNTPOINT}/${COMMANDUNIT_PWD}/src/main/scripts"
  source:
    - ${COMMANDUNIT_DEPENDENCIES_ROOT}/bud/lib/core.rc
    - ${COMMANDUNIT_HOSTFSROOT_MOUNTPOINT}/${COMMANDUNIT_PWD}/src/main/scripts/target_lib.rc
  cmd: cat
  args:
    - ${SCRIPTS_DIR}/hello.txt
then:
  exitCode:
    - EQUAL
    - 0
  stdout:
    present:
      - REGEX:Hello world!
  stderr:
    absent:
      - REGEX:.+
      - ```

This is just a format error, the definition should be like following:
when:
  environmentVariables:
    SCRIPTS_DIR: "${COMMANDUNIT_HOSTFSROOT_MOUNTPOINT}/${COMMANDUNIT_PWD}/src/main/scripts"


Pin pytest version

It looks like an incompatibility between flaky and the changes to pytest leads to failures in the test matrix.

Proof that this fixes latest/rc/latest: https://github.com/PennyLaneAI/catalyst/actions/runs/8141505724/job/22249017557


loop over output_map

in the following loop in conv2d layer:
for (int i=0; i<h*w; i++) {
for(int j=0; j<ch_out; j++) {
for(int r=0; r<REPS; r++) {
out1.data()[r*ch_out + j] -= static_cast(output_map.data()[i*ch_out + j]) * r_left.data()[r*h*w + i];
}
}
}
how the loop is over h*w since for some layers the output height and width are different, declared as h_out and w_out.

I think there is a bug within this line
} else if (kernel_.dimension(0) == 1 && kernel_.dimension(0) == 1)
first  I think the second condition should be  kernel_.dimension(1) == 1
second this should grantee that input and output h and w be equal but there are some cases of convolution with different input output size that get into this 'if' condition.
When I check eq for these cases it is: "eq: 0" meaning that integrity check failed.
I do not know how when filter size is 1x1 input-output size are not equal if this is the meaning of the condition.

This is for pointwise convolutions where the kernel is of size 1x1. You're right that there seems to be a typo in the check.
I only ever checked convolutions where the kernel is squared, so there might be some hidden bugs for the general case. In particular, you're right that if the kernel is of size 1xk something probably breaks. But these kernels are never used in practice as far as I know.

Yes, that's right. I corrected the condition kernel_.dimension(0) == 1 && kernel_.dimension(1) ==1 to make sure it gets into the if when the kernel is 1x1 but when I run it with resnet50 and kernel size is 1x1 I print the h,w h_out, and w_out and they are not identical and as I said eq is also 0. So there is another problem here besides 1xk kernel size.

Maybe your convolution is using strides larger than 1. I think I only thought about the stride=1 case when implementing this, and it that case you would have h=h_out and w=w_out.

yes, it can be that.  I will check, thanks.

I checked It is the stride issue. In this case, should it be redirected to the next condition block that using convolution with im2col?


update LegacyServiceAccountTokenTracking to beta

PR: https://github.com/kubernetes/kubernetes/pull/114523
Issue: https://github.com/kubernetes/enhancements/issues/2799

Hi @zshihang :wave: please take a look at Documenting for a release - PR Ready for Review to get your PR ready for review as soon as possible. 01:00 UTC Wednesday 22nd March 2023 / 17:00 PDT Tuesday 21st March 2023 is the official deadline.
This PR will need a doc review by Tuesday 4th April 2023 to get this into the release. Please reach out to required SIGs to get their review. Thank you!

/lgtm
/sig auth

/approve

/lgtm
(reapplied)


Refactor monkey.js

The monkey.js file needs some refactoring, that allows adding tests to validate functionality.

fixed


Usage with global providers and preact-router

Hi, I am really excited about replacing react+recompose+react-router with much lighter alternatives, but I am having some issues fitting it all together. I am coming from react/recompose, so I'm sure I am doing something just a bit wrong, but I can't seem to figure it out. Any ideas would be much appreciated.
I have a demo that illustrates what I am trying to accomplish. I want global provider for modal & user state. Here is a code sandbox.
Basically, I can't seem to work out subscribing to a global state and updating the app.
To reproduce, click "login" and login with any email/password. This should trigger 3 provider state changes:

show modal with login form
update user
hide modal with login form

which in turn should change these component-state things:

change menu items in Header
show/hide modal in Header
change routes for / from PageHome to PageDashboard (for logged in users)

I did this, which seems like a terrible hack:
// when global state changes, render app
if (typeof window !== 'undefined') {
  providers.subscribe(props => {
    console.log('PROVIDERS', props)
    render(<App />, document.body)
  })
}

It works a little better at refreshing when it should show the modal, but only about half the time (show modal works, but not hide.)
Initial user-state seem to work ok (derive user from JWT works fine) so you can login, then refresh page and it looks how it should. You can click LOGOUT, and refresh, and it kills the token and user-state, and goes back to anonymous view.

I also added a Footer to show current in-component state indicators, and they work same as other things (on refresh, it shows user: true if logged in, but doesn't update otherwise.)

I made this sandbox as another way to have providers update the app, that I can think of. It also doesn't update, so I'm not sure what I am doing wrong.