Not able to deploy sample app via Render

Bug report
"No render.yaml found on main branch." Error message on Render.com deploy page
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:
Followed "Deploy to a shareable web link" here: https://docs.stripe.com/baas/start-integration/sample-app?use-or-copy-sample-app=copy-sample-app&copy-sample-app-tabs=no-code-deployment
Screenshots

Hi @kirtan. Thank you for reporting this bug and sorry about the trouble. I've merged a fix that appears to be working for me after I tested it. Do you mind giving it another shot?

Works now, thanks!


Added support for Debian

Just some small changes to work around Debian's non-support for PPAs

Thanks, looks good!


Client Encryption: Adds fix to allow partition key path and id to be part of client encryption policy.

Description

Allow support to include partition key path and id during encryption policy creation (only deterministic encryption type) and bumps up the format version.

If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

@aayush3011 can you please also link the corresponding .Net PR in this PR description.

@aayush3011 What is the status on this PR?

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-cosmos


Introspection v2

Thanks for all the feedback on the previous PR. This is a second pass at introspection that is much simpler. Instead of having interfaces + wrapping types, we just have introspection interfaces with <<GraphQL\InterfaceType(...)>> attributes. The generated classes then implement these interfaces so when we're resolving the introspection query we read the fields from the classes themselves.
The codegen changes were pretty minimal since we can do most of the implementation of the interfaces within the base classes which is really nice. I haven't implemented all the introspection features yet, just enough to validate this direction.
If this looks good, I'll implement the remaining introspection features in another PR.

This looks really good and reasonable to me. Curious what @jjergus thinks!

Updated based on the reviews

Merging this so we can start iterating with validation etc.


Release 1.9.1

Announcements

Ver. 1.9.0 and 1.9.1 have the security fixes #1674 and #1680, respectively.
Please apply the fixes if you want to use old versions.

Security Fixes

Fix CVE-2018-1000656 (#1680)

Bug Fixes

Fixed an issue where lidar_naive_l_shape_detect can not be built (#1681).

released


[Request] Option-based Configs

I would like to have some of the config-options currently set via the config file switched over to permission-options for greater customization. I propose the following configs be switched over to options:

initial-claim-blocks - Allow you to change a players base claim block amount by option
max-accrued-blocks - Allow you to increase a players max claim blocks by option
player-all-claim-expiration-days - Allow you to configure the expiry time of a player's claim by option

Fixed in commit https://github.com/MinecraftPortCentral/GriefPrevention/commit/7548a7796ef9f8d543a4b5d7782648f3cb8a83c4


Reduction not found

Hi, here is a MWE of an unexpected error:
using LoopVectorization

function test(a, b)
    #s = 0.0
    @turbo for i in axes(a,1)
        ai = 0.0
        for j in axes(b,2)
            bij = b[i, j]
            ai += bij^2
        end
        ai = exp(ai)
        a[i] = ai
    end
    return nothing
end

a = rand(256)
b = rand(256, 256)
test(a, b)

LoadError: LoadError: "Reduction not found."
Stacktrace:
[1] reduction_zero
@ $HOME.julia\packages\LoopVectorization\7gWfp\src\modeling\costs.jl:649 [inlined]
[2] add_reduction_update_parent!(vparents::Vector{LoopVectorization.Operation}, deps::Vector{Symbol}, reduceddeps::Vector{Symbol}, ls::LoopVectorization.LoopSet, parent::LoopVectorization.Operation, instr::LoopVectorization.Instruction, reduction_ind::Int64, elementbytes::Int64)
@ LoopVectorization $HOME.julia\packages\LoopVectorization\7gWfp\src\parse\add_compute.jl:255
If I use a[i] = exp(ai) instead of the last two lines, it works. It also works if the last two lines are ai = exp(ai/2); a[i] = ai. I feel the issue is related to your explanation in #498, but honestly I don't fully understand that. My general feeling there is that the error it gives is unnecessary and can be ignored if it can run... Is that correct?

Here is a variant born out of this MWE with a different goal to produce another unexpected error.
function test(a, b)
    ai = 0.0
    @tturbo for i in axes(b,2)
        ai = 0.0
        for j in axes(b,1)
            bij = b[j, i]
            ai += bij^2
        end
        aii = a[i]
        aii *= exp(-ai)
        a[i] *= aii
    end
    return nothing
end

a = rand(256)
b = rand(256, 256)
test(a, b)

gives
LoadError: AssertionError: More than one parent is a reduction over the vectorized variable.
Stacktrace:
[1] storeinstr_preprend(op::LoopVectorization.Operation, vloopsym::Symbol)
@ LoopVectorization $HOME.julia\packages\LoopVectorization\7gWfp\src\codegen\lower_store.jl:19

A more general question is, I feel that sometimes writing code in a mathematically equivalent way will makes a working code no longer works if I use @turbo. This makes me feel worried whether my working code actually has some underlying bug / is not doing what it is supposed to do. In general, do you think that the errors LV give are because of subtleties of making everything SIMD (like LV now cannot transform codes if they are written in an unnecessarily complicated way), or because of essential problem in the original code?


Creating on-boarding module for xkops

This specialization module aims to equip participants with in-depth knowledge and hands-on experience in using Kubecost, Pixie, and Robusta for monitoring, cost management, and automation in Kubernetes environments. By the end of this module, participants should be able to effectively utilize these tools to optimize and troubleshoot Kubernetes clusters.

Deliverable: XkOps Specialization Module

@alikhanxgrid for your review please


Unable to play two DRM-protected medias at the same time

Version
Media3 1.4.1
More version details
I've also tried with ExoPlayer 2.19.1, and Media3 main branch.
Devices that reproduce the issue

Huawei P40 Lite (JNY-LX1), Android 10, EMUI 12.0.0

Devices that do not reproduce the issue

Google Pixel 7 Pro, Android 15 QPR1 Beta

Reproducible in the demo app?
Yes
Reproduction steps
See PR #1878
Expected result
Two DRM-protected media can be played at the same time.
Actual result
Only the audio of each media is being played. A black screen is displayed instead of each video.
Media
See PR #1878
Bug Report

[x] You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

I suspect this device doesn't support multiple secure decoders - in which case there's not much ExoPlayer can do to play multiple encrypted streams with hardware-secure decryption.
Depending on the license constraints, you may be able to use L3 software decryption instead - you could try forcing your playbacks to use L3 as shown in this comment: https://github.com/google/ExoPlayer/issues/7790#issuecomment-678473131
See also https://developer.android.com/media/media3/exoplayer/drm#using-a-custom-drmsessionmanager

You will email the zip file produced by adb bugreport to<EMAIL_ADDRESS>after filing this issue.

I can't see the either the AndroidXMedia3 or ExoPlayerLib log lines that are logged when a player instance is prepared. I didn't spot any useful logs in this BR.
Your demo also doesn't attach an EventLogger - which would tell you lots about the playback, including whether ExoPlayer thinks both encrypted video tracks are supported or not.

Hi @icbaker,
Switching to L3 seems to be working. We'll investigate if this is a viable option for us, or if we will disable that feature in our app on unsupported devices.
FYI: I've added the EventLogger to my branch just in case


Codecov #80

add some unit tests

Codecov Report

Merging #87 (294882c) into main (34f499c) will increase coverage by 2.58%.
The diff coverage is 51.85%.

@@            Coverage Diff             @@
##             main      #87      +/-   ##
==========================================
+ Coverage   30.07%   32.66%   +2.58%
==========================================
  Files          97       98       +1
  Lines        4186     4237      +51
  Branches      660      671      +11
==========================================
+ Hits         1259     1384     +125
+ Misses       2682     2593      -89
- Partials      245      260      +15

Impacted Files
Coverage Δ

.../apache/rocketmq/mqtt/common/util/SpringUtils.java
0.00% <0.00%> (ø)

...a/org/apache/rocketmq/mqtt/cs/starter/Startup.java
0.00% <0.00%> (ø)

...rocketmq/mqtt/cs/session/loop/SessionLoopImpl.java
44.85% <50.00%> (+0.37%)
:arrow_up:

...ocketmq/mqtt/cs/channel/DefaultChannelManager.java
73.77% <66.66%> (+11.27%)
:arrow_up:

...tt/ds/upstream/processor/UnSubscribeProcessor.java
76.47% <66.66%> (+76.47%)
:arrow_up:

...mqtt/ds/upstream/processor/SubscribeProcessor.java
84.21% <70.00%> (+84.21%)
:arrow_up:

...a/org/apache/rocketmq/mqtt/cs/session/Session.java
39.84% <0.00%> (+2.68%)
:arrow_up:

...g/apache/rocketmq/mqtt/cs/channel/ChannelInfo.java
89.89% <0.00%> (+3.03%)
:arrow_up:

...pache/rocketmq/mqtt/ds/meta/FirstTopicManager.java
46.57% <0.00%> (+8.21%)
:arrow_up:

... and 5 more

Continue to review full report at Codecov.

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


Reference doc for kubeadm (release-1.11)

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
For 1.11 Features: set Milestone to 1.11 and Base Branch to release-1.11
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
NOTE: After opening the PR, please un-check and re-check the "Allow edits from maintainers" box so that maintainers can work on your patch and speed up the review process. This is a temporary workaround to address a known issue with GitHub.>
Please delete this note before submitting the pull request.

I don't understand why there's no preview here:
https://deploy-preview-9044--kubernetes-io-vnext-staging.netlify.com/

/assign @mistyhacks

/cc @Bradamant3

/retest

To get the Netlify preview working, I cherry picked the shortcode fix by @Bradamant3.

Previews

kubeadm init
kubeadm-join
kubeadm
kubeadm upgrade
kubeadm config
kubeadm reset
kubeadm token
kubeadm version
kubeadm alpha

It would be nice to get rid of the "What's next" bullet at the top of each ref page. But I would rather save that for another time.

Holding for approval by @mistyhacks.
/hold
/lgtm
/cc @Bradamant3


Array.CreateInstance

Today a chunk needs to allocate an array of array of struct. This is achieved using the API Array.CreateInstance which creates an opaque managed array of objects[].
I see some downside:

the compiler cannot predict the bytes size in memory, this is done during the runtime. So we could have a waste of resources here
accessing to a component might be slower when using the Unsafe.As API for the reason above (things get done runtime side)
the app cannot use the nativeaot optimizations (disable reflection) if the users wants (need to try out tbh)

I was wondering if it's worth to replace the managed Array.CreateInstance with an opaque unmanaged pointers pool.

the compiler cannot predict the bytes size in memory, this is done during the runtime. So we could have a waste of resources here

Using the generic overloads actually registers components during "compile time" ( or atleast the first time they are being invoked ). Thats why i added them : world.Create<Position, Velocity,...>(...);. This is already as efficient as it can get, since the compile time static Component<T> is being used :)
Using the type based approach however will determine during runtime, but theres no way around it... since types are used.

accessing to a component might be slower when using the Unsafe.As API for the reason above (things get done runtime side)

It depends, receiving an array from a Chunk is actually a O(1) operation. A internal lookup array is being used (chunk.ComponentIdToArrayIndex) to get the index of the component array which is the most efficient way... Only a direct field acess is faster.

the app cannot use the nativeaot optimizations (disable reflection) if the users wants (need to try out tbh)
As far as i know, Array.CreateInstance is not being stripped out. So it should actually work, i think i also already ran it once in nativeaot. However we should probably retry this to see if it still works.

I was wondering if it's worth to replace the managed Array.CreateInstance with an opaque unmanaged pointers pool.

The problem with umanaged memory is that its way more restrictive. With unmanaged memory, we could not use managed structs as components anymore public struct Texture{ Texture2D... } and therefore its usage with monogame, godot or unity would be a very very bad experience. I actually had a branch where i used a combination of unmanaged memory and managed arrays to simulate this. However according to benchmarks it was not faster, it performed the same.

the compiler cannot predict the bytes size in memory, this is done during the runtime. So we could have a waste of resources here

Using the generic overloads actually registers components during "compile time" ( or atleast the first time they are being invoked ). Thats why i added API like : world.Create<Position, Velocity,...>(...);. This is already as efficient as it can get, since the compile time static Component<T> is being used :)
Using the type based approach however will determine during runtime, but theres no way around it... since types are used.

accessing to a component might be slower when using the Unsafe.As API for the reason above (things get done runtime side)

It depends, receiving an array from a Chunk is actually a O(1) operation. A internal lookup array is being used (chunk.ComponentIdToArrayIndex) to get the index of the component array which is the most efficient way... Only a direct field acess is faster.

the app cannot use the nativeaot optimizations (disable reflection) if the users wants (need to try out tbh)
As far as i know, Array.CreateInstance is not being stripped out. So it should actually work, i think i also already ran it once in nativeaot. However we should probably retry this to see if it still works.

I was wondering if it's worth to replace the managed Array.CreateInstance with an opaque unmanaged pointers pool.

The problem with umanaged memory is that its way more restrictive. With unmanaged memory, we could not use managed structs as components anymore public struct Texture{ Texture2D... } and therefore its usage with monogame, godot or unity would be a very very bad experience. I actually had a branch where i used a combination of unmanaged memory and managed arrays to simulate this. However according to benchmarks it was not faster, it performed the same.

Just ported the Sample to FNA which is more NativeAOT friendly and it runs fine with some tweaks [removed the managed Texture2D field from the Sprite struct

Just ported the Sample to FNA which is more NativeAOT friendly and it runs fine with some tweaks [removed the managed Texture2D field from the Sprite struct

Thats great to hear :D


Mergeable summaries

There's some good stuff in here that's very relevant:
http://www.cse.ust.hk/~huangzf/mergeable.pdf
In particular, it describes how to merge SpaceSaver, which is the best-performing heavy-hitters estimator.

Hi, I found this looking for code for mergeable summaries. Looks like this project is much more than that.  Does it contain an implementation of it that I can use as a starting point?
Thanks.

We have several mergeable summaries implemented, which are kept in the core project:
https://github.com/twitter/algebird/tree/develop/algebird-core/src/main/scala/com/twitter/algebird
I'm not sure what this issue was originally intended to track, but it seems we should be able to close it?


feat: add upper and lower case string

Converts string, as space separated words, to upper case or lower case.

I'd like to hear arguments for why this should be in Ramda.
It seems to me that besides your UPPER CASE and lower case we would also want to consider Title Case, kebab-case, snake-case, camelCase and UpperCamelCase.  Do you think Ramda should be in the business of converting between any of these formats.  Do we need kebabToCamel and upperCamelToTitle, etc.?
My first thought is that these belong in user-land, or one of the adjunct libraries.

I'd like to hear arguments for why this should be in Ramda.
It seems to me that besides your UPPER CASE and lower case we would also want to consider Title Case, kebab-case, snake-case, camelCase and UpperCamelCase. Do you think Ramda should be in the business of converting between any of these formats. Do we need kebabToCamel and upperCamelToTitle, etc.?
My first thought is that these belong in user-land, or one of the adjunct libraries.

I believe these features can be implemented in other libraries. However, since Ramda already includes toUpper and toLower, I think it would be beneficial to add variations for different types of uppercase and lowercase transformations.

My first thought is that these belong in user-land, or one of the adjunct libraries.

Agreed, ty for the PRs @nhannt201 however these functions are imho too specific for ramda and there are already other libraries that do this


Add Python-slim Docker image

Builds a Python3.9-slim Docker image for master, each tag, and each PR and publishes the image to the GitHub container registry.
This image does NOT install the girder dependencies.
This image is ~2.2 GB when built locally (so not all that slim). Considering this size, I have added a separate slim-geospatial image that only has the sources/utilities relevant for geospatial formats which is ~1.15 GB under the -geo tag suffix

From my perspective, this is ready to merge and the versioning issue I was noticing earlier is now resolved.
Any changes to the Python version or adding addition tags should come in follow up PRs as needed


perf: parallelize imports processing in import analysis plugin

Description
Implement @sapphi-red's idea
I'm not seeing a perf improvement on Sapphi's test, but I think we should be able to find examples where this makes a difference.

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[x] Other

I'm not seeing a perf improvement on Sapphi's test, but I think we should be able to find examples where this makes a difference.

The maximum number of import statements in each file of sapphi's repo is 3-4, I think that's why. Double it, maybe it will make a difference.


Self-hosted kubelet not working correctly

I added the --self-host-kubelet=true flag to hack/multi-node/bootkube-up and launched a cluster.
The kubelet pivot seems to succeed but then on-host kubelet keep restarting and failing binding a socket, and not exiting gracefully when seeing the lock is acquired by the self-hosted kubelet:
Dec 28 18:24:00 w1 systemd[1]: Started kubelet.service.
Dec 28 18:24:00 w1 kubelet-wrapper[11841]: image: using image from file /usr/lib64/rkt/stage1-images/stage1-fly.aci
Dec 28 18:24:00 w1 kubelet-wrapper[11841]: image: using image from local store for image name quay.io/coreos/hyperkube:v1.5.1_coreos.0
Dec 28 18:24:15 w1 kubelet-wrapper[11841]: I1228 18:24:15.896894   11841 server.go:312] acquiring file lock on "/var/run/lock/kubelet.lock"
Dec 28 18:24:15 w1 kubelet-wrapper[11841]: I1228 18:24:15.897636   11841 server.go:317] watching for inotify events for: /var/run/lock/kubelet.lock
Dec 28 18:24:15 w1 kubelet-wrapper[11841]: I1228 18:24:15.900488   11841 feature_gate.go:181] feature gates: map[]
Dec 28 18:24:15 w1 kubelet-wrapper[11841]: I1228 18:24:15.932580   11841 docker.go:356] Connecting to docker on unix:///var/run/docker.sock
Dec 28 18:24:16 w1 kubelet-wrapper[11841]: I1228 18:24:15.933192   11841 docker.go:376] Start docker client with request timeout=2m0s
Dec 28 18:24:16 w1 kubelet-wrapper[11841]: I1228 18:24:16.363777   11841 manager.go:143] cAdvisor running in container: "/system.slice/kubelet.service"
Dec 28 18:24:16 w1 kubelet-wrapper[11841]: W1228 18:24:16.396474   11841 manager.go:151] unable to connect to Rkt api service: rkt: cannot tcp Dial rkt api service: dial tcp [::1]:15441: getsockopt: connection refused
Dec 28 18:24:16 w1 kubelet-wrapper[11841]: I1228 18:24:16.430887   11841 fs.go:117] Filesystem partitions: map[/dev/sda9:{mountpoint:/var/lib/docker major:8 minor:9 fsType:ext4 blockSize:0} /dev/sda3:{mountpoint:/etc/ssl/certs major:8 minor:3 fsType:ext4 blockSize:0}]
Dec 28 18:24:16 w1 kubelet-wrapper[11841]: I1228 18:24:16.432930   11841 manager.go:198] Machine: {NumCores:1 CpuFrequency:2730824 MemoryCapacity:1045471232 MachineID:cf742c5b2ef64b18a93d3785f5444d22 SystemUUID:7DA693DA-2CC1-4EEE-8BA1-0157BB55C2B1 BootID:2d6d518a-dbe8-469d-b2c2-99fc27eaa68a Files
Dec 28 18:24:16 w1 kubelet-wrapper[11841]: I1228 18:24:16.433546   11841 manager.go:204] Version: {KernelVersion:4.7.3-coreos-r3 ContainerOsVersion:CoreOS 1185.5.0 (MoreOS) DockerVersion:1.11.2 CadvisorVersion: CadvisorRevision:}
Dec 28 18:24:16 w1 kubelet-wrapper[11841]: I1228 18:24:16.446561   11841 cadvisor_linux.go:152] Failed to register cAdvisor on port 4194, retrying.
Dec 28 18:24:16 w1 systemd[1]: kubelet.service: Failed with result 'exit-code'.
Dec 28 18:24:21 w1 systemd[1]: kubelet.service: Service hold-off time over, scheduling restart.
Dec 28 18:24:21 w1 systemd[1]: Stopped kubelet.service.
Dec 28 18:24:21 w1 systemd[1]: Starting kubelet.service...
Dec 28 18:24:21 w1 systemd[1]: Started kubelet.service.

Its possible that this is due to /var/run not being bind mounted correctly as noted by @aaronlevy

Manually tested this and saw the same thing. Did a quick test adding mount for /var/run to $RKT_OPTS in the kubelet.service unit -- but didn't resolve the issue.
Not going to be able to take a deeper look at this immediately, but just wanted to note that I'm seeing the same behavior.

@aaronlevy can you help me understand why we nsenter --target=1 --mount --wd=. is needed when launching the self-hosted kubelet. Could this be interfering?

My initial thought is that it shouldn't be interfering. Basically it just means the self-hosted kubelet will be in the host mount namespace - and for the kubelet process specifically - it will see the container FS as its root filesystem.
It is needed so that mount tools operate in the hosts mount namespace. Otherwise mounts like secrets/configmaps/volumes would be within the kubelet container's filesystem/namespace and if it was restarted, they would be recreated in the next kubelet container (whereas the pods themselves would still be running referencing the old mounts).
However, I was planning on dropping that workaround soon, and using the --containerized flag -- which tells the kubelet that it should use a special path /rootfs when creating mounts.
See:
https://github.com/kubernetes-incubator/bootkube/pull/247#discussion_r94185933
https://github.com/kubernetes-incubator/bootkube/pull/162
Also, just to confirm, it does look like we're still properly mounting /run in both on-host and self-hosted kubelet:
https://github.com/coreos/coreos-overlay/blob/master/app-admin/kubelet-wrapper/files/kubelet-wrapper#L71
https://github.com/kubernetes-incubator/bootkube/blob/master/pkg/asset/internal/templates.go#L64-L65
And this definitely worked at some point - so we introduced a regression at some point (I've been a little more lax on testing self-hosted kubelet since defaulting it to off ...)

I'm not sure what exactly is going on yet, but, just going to post some of my testing updates here. I can reproduce this issue with:
k8s: 1.4.7, 1.5.1
CoreOS: 1235.0.0
bootkube at master branch
The host kubelet first exits not because it detects lock contention but because:
systemd[1]: kubelet.service: Service hold-off time over, scheduling restart.
After that restart the self-hosted kubelet grabs the listening address and the host kubelet crashloops.
Something odd is going on with the lock coordination.
$ lsof /var/lock/kubelet.lock
Output information may be incomplete.
lsof: WARNING: can't stat() cgroup file system /var/lib/rkt/pods/run/cc9711a2-d30c-42f5-9f19-14f4e3b8c3dc/stage1/rootfs/opt/stage2/hyperkube/rootfs/sys/fs/cgroup/pids
      Output information may be incomplete.
lsof: WARNING: can't stat() pstore file system /var/lib/rkt/pods/run/cc9711a2-d30c-42f5-9f19-14f4e3b8c3dc/stage1/rootfs/opt/stage2/hyperkube/rootfs/sys/fs/pstore
      Output information may be incomplete.
lsof: WARNING: can't stat() selinuxfs file system /var/lib/rkt/pods/run/cc9711a2-d30c-42f5-9f19-14f4e3b8c3dc/stage1/rootfs/opt/stage2/hyperkube/rootfs/sys/fs/selinux
<SNIP ... this goes on awhile>

$ flock -x /var/lock/kubelet.lock
flock: bad file descriptor: '/var/lock/kubelet.lock'

fwiw, I think the "bad file descriptor" message is because no command was provided:
sudo flock -x /var/lock/kubelet.lock echo foo should work

--self-hosted-kubelet is marked deprecated and will be removed in a future release.


Trait accessors of graph do not include Matches nodes of traits

duplicate of https://github.com/renesca/renesca-magic/issues/22


Propose changes to better support external notifications

While it makes sense to send notifications to users (or broadcast) in many scenarios, its possible that external integrations via NotificationProcessor may not have a 1:1 relationship with users.
For example, while the current interface is adequate for sending Slack DMs to users or teams (via the team expansion code in the notifications-backend) it doesn't facilitate a way to choose a different destination such as a Slack channel or Email mailing list.
We could add another recipient type of external that would accept some arbitrary value that processors can choose to support. For example a value for externalRef may be something like slack:C03SABTG6RX or slack:#abc or<EMAIL_ADDRESS>This would also change the interface for recipients to accept an array so that multiple types of recipients could be specified.
This would still allow for direct user notifications if desired, as well as separate external notifications sent for the same SendNotificationRequest.
One challenge with this approach might be if you want a notification to go to individual users using the WebProcessor and only 1 notification via the SlackProcessor as there is not a way for the processor to know whether it should act on all notifications it receives or not.
Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

IIUC, the suggestion is to use the Backstage as a kind of integration hub.
Maybe I miss something in this proposal, but the notifications are meant for Backstage users only.
A pluggable processor will receive every notification and it is its responsibility to decide whether the notification should be handled and how, including mapping the Backstage user to an external identity.
There's work in progress on the Settings which will provide configuration to these processors (#23716).
In addition, processors can be configured via app-config.yaml, which can be suitable for user's mapping.

For some additional context see https://github.com/kunickiaj/backstage/blob/ajk--slack-notification-processor/plugins/notifications-backend-module-slack/src/lib/SlackNotificationProcessor.ts
The existing interface works well if you want to send a user a notification as a DM as shown in the above example. However, in our org it's common for teams to designate a channel for receiving certain notifications that may be of interest to the entire team.
The notifications plugin currently supports sending to a Group entity which is expanded to N User notifications. In this use case we may want to not expand it and simply send it to a shared resource such as a Slack channel instead of the same DM to N users.
With the proposed change this would be possible, as well as sending individual notifications to the web processor.


Fix all the stuff you broke last night...

In particular the pi_revision thing in PiGPIOPin, all the stuff @lurch
picked up in pins/data.py (thank goodness someone's watching!), and
make all those links pointing to "Notes" point somewhere useful like
"Pin Numbering"...

Current coverage is 70.78% (diff: 9.09%)

Merging #415 into master will decrease coverage by 0.04%

@@             master       #415   diff @@
==========================================
  Files            29         29
  Lines          6240       6244     +4
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           4420       4420
- Misses         1820       1824     +4
  Partials          0          0

Powered by Codecov. Last update bb8ea52...5419bef

Should the PWM stuff have been in a separate PR?

Ahh, I was considering it, but I'm too squeezed for time right now so this is basically "fix everything so you've got a clean PR which only includes Servo et al" - as I'm anticipating that's going to generate a bit of discussion.
Incidentally, there's a Raspbian freeze coming up fairly shortly (@bennuttall tomorrow?) so in a vague effort to get GPIO Zero (and some other stuff) into the next Raspbian image I'm going to try and get a 1.3 release out rather shortly. As a result, the Servo stuff is going to be pared down to the bare minimum that I think is definitely fine (on the assumption that adding stuff is always easier than removing stuff as the latter is backwards incompatible) - I'm also contemplating merging ButtonBoard this evening if you're happy for that to go in as it is?

Yeah, feel free to merge in whatever you think's suitable - afraid I don't have much time for working on gpiozero ATM :-/

I know the feeling - still, very much appreciate you taking the time to pick up on all my bugs :)


Text resize shouldn't change visible file section

Increasing/decreasing font size currently effectively scrolls you through a file. I imagine it's because the vertical scroll/offset is fixed, while the height of the rendered page is changing under it. It would be super fancy and helpful if Atom would scale text and automatically jump you back to wherever you were in the file.

Thanks for the suggestion! This looks like a duplicate of https://github.com/atom/atom/issues/4084 – please subscribe there for updates.


langchain-milvus  install error

Checked other resources

[X] I added a very descriptive title to this issue.
[X] I searched the LangChain documentation with the integrated search.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I am sure that this is a bug in LangChain rather than my code.
[X] The bug is not resolved by updating to the latest stable version of LangChain (or the specific integration package).

Example Code
pip install --upgrade --quiet pymilvus[model] langchain-milvus
Error Message and Stack Trace (if applicable)

Description
can not install langchain-milvus
System Info
System Information

OS:  Windows
OS Version:  10.0.22631
Python Version:  3.10.7 (tags/v3.10.7:6cc6b13, Sep  5 2022, 14:08:36) [MSC v.1933 64 bit (AMD64)]

Package Information

langchain_core: 0.2.5
langchain: 0.2.3
langchain_community: 0.2.4
langsmith: 0.1.77
langchain_cli: 0.0.23
langchain_google_cloud_sql_mysql: 0.2.2
langchain_google_vertexai: 1.0.5
langchain_openai: 0.1.6
langchain_text_splitters: 0.2.1
langchainhub: 0.1.17
langserve: 0.2.1

Packages not installed (Not Necessarily a Problem)
The following packages were not found:

langgraph

Use pip install --upgrade langchain_milvus


⚠ Thumbnails API Endpoint has degraded performance

In 43207b2, Thumbnails API Endpoint (https://thumbnails.roblox.com/v1/assets?assetIds=82738847&format=Png&isCircular=false&size=30x30) experienced degraded performance:

HTTP code: 200
Response time: 1489 ms

Resolved: Thumbnails API Endpoint performance has improved in f0c392d after 7 minutes.


Release 0.6.2

Numbers Data Structures
Improve the usability of these data structures by adding simple methods.
Ranges

Created Contains method for determining if a value is in the range

Pairs

Implemented Equals methods (Fixed and Mutable Pairs can be equal if same primitive type)
Created methods of easy conversion ToFixed and ToMutable

Improved Documentation

[x] Add docs on new Pair method

Dependencies Updated

Dokka 1.8.10

Publishing
Simplify Publishing Workflow by collecting all Artifact versions into one file.

[x] Add Artifacts file to CodeOwner

Quality Improvements

[ ] Each Java test file should have all test methods either public or package-scoped (prefer package for simplicity)
[ ] Increase Code Coverage Limits slightly where convenient


av1: double quote with command inside

Compare with bash
what expected:
"ls -l --color"
what happend:
removed doubel quote and command running

solved in PR #237
https://github.com/tonnytg/42_minishell/pull/237


docs: update trim messages notebook

Update trim messages notebook to include common use cases and explain what the desired behavior is

I'll merge as is, but we can update afterwards -- the role of the highlighting is to prevent users from copy pasta without adjusting the parameters appropriately


Linter mistakenly thinks a random function is a JSX class.

The reason it's getting flagged is that the display-name rule treats functions that return JSX as stateless react components (link), since they sort of technically could be. You might be able to get around it by turning on the acceptTranspilerName option. This is yet another case where it might be good to have an ignoreStatelessFunctionComponents option, but I'm not sure what @yannickcr's thoughts on that are.
Tangential question: what font is that? I love the === and => rendering.

It's tonsky/FiraCode :)

About stateless components:
I see the value in having an ignore option, though there are certain rules that simply make no sense with stateless components (like display-name)

display-name is still useful if you want to prevent "anonymous" components like:
export default function(props) {
  return
Hello {props.name}

}

This is yet another case where it might be good to have an ignoreStatelessFunctionComponents option, but I'm not sure what @yannickcr's thoughts on that are.

I do not really see the advantage over just turning on acceptTranspilerName.
Also, in the next major version the acceptTranspilerName option will be on by default.

Still, that's not the only rule where you might want to have ignoreStatelessFunctionComponents.
In the example above, the intent is to write a react-native Navigator.NavigationBar route mapper. eslint-plugin-react thinks those are stateless components even though they aren't.
Since it's a bit hard to differentiate stateless components from just functions returning JSX, what would be the preferred way of disabling other checks like react/no-multi-comp in these situations?

no-multi-comp has an ignoreStateless option since v3.8.0.
I'm agree it's very difficult to detect stateless components, that's why sometimes we can get some false positives. But I am not against adding an ignoreStateless option to the rules that could have some problems (even display-name if you think acceptTranspilerName do not solve all the case ;) ).


Get Self Test Command verification

https://github.com/openbmc/openbmc/issues/436

Looks like development as of now implemented as  currently "it is not supported ". But it will be revisited down the line seems.

Have tested manually - looks like currently development is supporting the bmc selftest command but the message coming as "Not implemented". Down the line, development is going to revisit and will have the real value there. So will create the base automation for the same.

Based on discussion/update from @williamspatrick , it seems development not going to pursue this. Closing the corresponding test story.


docs(circleci): Add CircleCI to troubleshooting docs

This came from personal difficulties in running Puppeteer tests on CircleCI. I tried to keep the note as brief as possible, while being helpful for an entire CI platform.

@googlebot I signed it!


Error in MD5 checksums

while installing traccar-macos-3.10 in mac it shows an error called Error in MD5 checksums: d41d8cd98f00b204e9800998ecf8427e is different from bf7a8ddec10dc91618276bbeb1532592

Where does it show the error? Have you tried downloading the installer again?

i am all so having the same error while installing traccar-macos 3.10 in mac 10.11.5 os..

It showed in my terminal window and I have also tried newly downloaded file

Can I see the full output. Ideally in text format, not a screenshot.

Susee-Mac-mini:traccar-macos-3.10 apple$ sudo chmod +x traccar.run
Password:
Susee-Mac-mini:traccar-macos-3.10 apple$ sudo ./traccar.run
Verifying archive integrity...Unknown option: n
Usage: head [-options] ...
-m    use method for the request (default is 'HEAD')
-f            make request even if head believes method is illegal
-b      Use the specified URL as base
-t   Set timeout value
-i      Set the If-Modified-Since header on the request
-c  use this content-type for POST, PUT, CHECKIN
-a            Use text mode for content I/O
-p  use this as a proxy
-P            don't load proxy settings from environment
-H    send this HTTP header (you can specify several)
-u            Display method and URL before any response
-U            Display request headers (implies -u)
-s            Display response status code
-S            Display response status chain
-e            Display response headers
-d            Do not display content
-o <format>   Process HTML content in various ways

-v            Show program version
-h            Print this message

-x            Extra debugging output

Error in MD5 checksums: d41d8cd98f00b204e9800998ecf8427e is different from bf7a8ddec10dc91618276bbeb1532592

Me too having the same error which is mentioned above

Just tried on Mac 10.11.6. Works fine for me.

What's the MD5 of the package itself? Mine is:
$ md5 traccar-macos-3.10.zip
MD5 (traccar-macos-3.10.zip) = ca1b307ee53d54d13f3ad593ff00209d

I'm also working on Mac 10.11.6 and my MD5 (traccar-macos-3.10.zip) = d41d8cd98f00b204e9800998ecf8427e

Just tested on another Mac and the same hash as before. Where did you download the package?

from here https://www.traccar.org/download/

Just checked third time. Same correct hash sum as first time. I'm not sure how you get a different one. Possibly some network issue or some kind of malware on your computer. Have you tried re-downloading it? Try on a different computer.

ok I'll try....

I'm tried on another Mac too but it tells the same error

On the same network? In that case the only explanation I can think of is that it gets modified somewhere on the way to your computer.

ok... I will try....

i tried in different mac system.But i got the same error above mentioned.i have no idea to solve this problem

Please try to download a version from SourceForce and see if it has the same problem:
https://sourceforge.net/projects/traccar/files/

ok fine..... Successfully installed...

But in my tracer client(android) not updating the location while my GPS is working good in google maps and I have turn on my data pack

That's a different story. Please create a separate issue and provide as much information as possible. Also check following page:
https://www.traccar.org/troubleshooting

It tells only the send failed message and my device is le 2 and asus zenfone 2 laser

Related forum topic with some more details:
https://www.traccar.org/forums/topic/md5-checksum-error-when-downloading-3-11-macos-server/


CI: build and test contracts

It would be good to have all the contract builds & tests running in CI for near-plugins. This is especially mandatory as this library will be used in other 3rd party contracts.

There is no contract builds, just tests for now.

Solved in #15


update 0.12.1

ydata-profiling 4.1.1 Update
Jira Ticket
Upstream
Dependencies
Changes

Updated version number and sha256
Updated pinnings on dependencies
Added docs to source_files
Added pyproject-metadata to test: requires
skip unnecessary tests that cause build failures

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_d7u7dxp8c7/clone/recipe/meta.yaml:17: patch_unnecessary: patch should not be in build when source/patches is not set
ERROR: /tmp/abs_d7u7dxp8c7/clone/recipe/meta.yaml:24: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_d7u7dxp8c7/clone/recipe/meta.yaml:25: build_tools_must_be_in_build: The build tool ninja is not in the build section.
ERROR: /tmp/abs_d7u7dxp8c7/clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_d7u7dxp8c7/clone/recipe/meta.yaml:26: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_d7u7dxp8c7/clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_d7u7dxp8c7/clone/recipe/meta.yaml:36: build_tools_must_be_in_build: The build tool ninja is not in the build section.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_89ia832yzi/clone/recipe/meta.yaml:17: patch_unnecessary: patch should not be in build when source/patches is not set
ERROR: /tmp/abs_89ia832yzi/clone/recipe/meta.yaml:24: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_89ia832yzi/clone/recipe/meta.yaml:25: build_tools_must_be_in_build: The build tool ninja is not in the build section.
ERROR: /tmp/abs_89ia832yzi/clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_89ia832yzi/clone/recipe/meta.yaml:26: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_89ia832yzi/clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_89ia832yzi/clone/recipe/meta.yaml:36: build_tools_must_be_in_build: The build tool ninja is not in the build section.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_2fce4tfczy/clone/recipe/meta.yaml:17: patch_unnecessary: patch should not be in build when source/patches is not set
ERROR: /tmp/abs_2fce4tfczy/clone/recipe/meta.yaml:24: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_2fce4tfczy/clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_2fce4tfczy/clone/recipe/meta.yaml:25: build_tools_must_be_in_build: The build tool ninja is not in the build section.
ERROR: /tmp/abs_2fce4tfczy/clone/recipe/meta.yaml:26: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_2fce4tfczy/clone/recipe/meta.yaml:31: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_2fce4tfczy/clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_2fce4tfczy/clone/recipe/meta.yaml:36: build_tools_must_be_in_build: The build tool ninja is not in the build section.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_1fwbtfdgni/clone/recipe/meta.yaml:17: patch_unnecessary: patch should not be in build when source/patches is not set
ERROR: /tmp/abs_1fwbtfdgni/clone/recipe/meta.yaml:24: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_1fwbtfdgni/clone/recipe/meta.yaml:25: build_tools_must_be_in_build: The build tool ninja is not in the build section.
ERROR: /tmp/abs_1fwbtfdgni/clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_1fwbtfdgni/clone/recipe/meta.yaml:26: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_1fwbtfdgni/clone/recipe/meta.yaml:31: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_1fwbtfdgni/clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_1fwbtfdgni/clone/recipe/meta.yaml:36: build_tools_must_be_in_build: The build tool ninja is not in the build section.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_18a8zmjxn2/clone/recipe/meta.yaml:17: patch_unnecessary: patch should not be in build when source/patches is not set
ERROR: /tmp/abs_18a8zmjxn2/clone/recipe/meta.yaml:24: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_18a8zmjxn2/clone/recipe/meta.yaml:25: build_tools_must_be_in_build: The build tool ninja is not in the build section.
ERROR: /tmp/abs_18a8zmjxn2/clone/recipe/meta.yaml:25: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_18a8zmjxn2/clone/recipe/meta.yaml:26: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_18a8zmjxn2/clone/recipe/meta.yaml:31: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_18a8zmjxn2/clone/recipe/meta.yaml:32: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_18a8zmjxn2/clone/recipe/meta.yaml:36: build_tools_must_be_in_build: The build tool ninja is not in the build section.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_1a87iaq4p3/clone/recipe/meta.yaml:15: patch_unnecessary: patch should not be in build when source/patches is not set
ERROR: /tmp/abs_1a87iaq4p3/clone/recipe/meta.yaml:22: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_1a87iaq4p3/clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_1a87iaq4p3/clone/recipe/meta.yaml:23: build_tools_must_be_in_build: The build tool ninja is not in the build section.
ERROR: /tmp/abs_1a87iaq4p3/clone/recipe/meta.yaml:24: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_1a87iaq4p3/clone/recipe/meta.yaml:29: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_1a87iaq4p3/clone/recipe/meta.yaml:30: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_1a87iaq4p3/clone/recipe/meta.yaml:34: build_tools_must_be_in_build: The build tool ninja is not in the build section.
Errors were found

As a heads up, the scipy 1.10.1 task I'm working on will require this to comply with the current linter requirements.
So kudos for doing this work, it will get used immediately!

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_3070ti3fmc/clone/recipe/meta.yaml:15: patch_unnecessary: patch should not be in build when source/patches is not set
ERROR: /tmp/abs_3070ti3fmc/clone/recipe/meta.yaml:22: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_3070ti3fmc/clone/recipe/meta.yaml:23: build_tools_must_be_in_build: The build tool ninja is not in the build section.
ERROR: /tmp/abs_3070ti3fmc/clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_3070ti3fmc/clone/recipe/meta.yaml:28: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_3070ti3fmc/clone/recipe/meta.yaml:29: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_3070ti3fmc/clone/recipe/meta.yaml:30: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_3070ti3fmc/clone/recipe/meta.yaml:34: build_tools_must_be_in_build: The build tool ninja is not in the build section.
Errors were found

As a heads up, the scipy 1.10.1 task I'm working on will require this to comply with the current linter requirements. So kudos for doing this work, it will get used immediately!

Thanks Schuyler! Glad I can help!

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_136xxh5u3b/clone/recipe/meta.yaml:15: patch_unnecessary: patch should not be in build when source/patches is not set
ERROR: /tmp/abs_136xxh5u3b/clone/recipe/meta.yaml:22: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_136xxh5u3b/clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_136xxh5u3b/clone/recipe/meta.yaml:23: build_tools_must_be_in_build: The build tool ninja is not in the build section.
ERROR: /tmp/abs_136xxh5u3b/clone/recipe/meta.yaml:28: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_136xxh5u3b/clone/recipe/meta.yaml:29: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_136xxh5u3b/clone/recipe/meta.yaml:30: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_136xxh5u3b/clone/recipe/meta.yaml:34: build_tools_must_be_in_build: The build tool ninja is not in the build section.
Errors were found

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_7bq5pd_3nz/clone/recipe/meta.yaml:15: patch_unnecessary: patch should not be in build when source/patches is not set
ERROR: /tmp/abs_7bq5pd_3nz/clone/recipe/meta.yaml:22: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_7bq5pd_3nz/clone/recipe/meta.yaml:23: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_7bq5pd_3nz/clone/recipe/meta.yaml:23: build_tools_must_be_in_build: The build tool ninja is not in the build section.
ERROR: /tmp/abs_7bq5pd_3nz/clone/recipe/meta.yaml:28: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_7bq5pd_3nz/clone/recipe/meta.yaml:29: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
ERROR: /tmp/abs_7bq5pd_3nz/clone/recipe/meta.yaml:30: host_section_needs_exact_pinnings: Packages in host must have exact version pinnings, except python build tools.
WARNING: /tmp/abs_7bq5pd_3nz/clone/recipe/meta.yaml:34: build_tools_must_be_in_build: The build tool ninja is not in the build section.
Errors were found


Fix the readiness probe using a sidecar container

Hi,
This MR aims to fix the readiness check regarding the helm chart.
The original solution caused the StatefulSet to be not upgradable by standard k8s methods and further problems using GitOps tools(ArgoCD and its health checks).
Additionally, readiness alerts and further things cause problems with the current solution.
(https://community.ibm.com/community/user/integration/discussion/ibm-mq-nativeha-on-kubernetes-with-ibm-messagingmq-helm-chart)
Original Solution:
Only one pod has a readiness check that runs successfully -> service will only serve the leader instance
This Solution:
Patch the label in a sidecar container -> service will only serve the leader instance (via selector)
Thanks for reading this far. I will happily welcome any feedback.
Best Regards,
Lorenz

Hi @callumpjackson, are there any interests in adjustments/improvements of the helm chart ?

Hi @lorenzboguhn - thanks for raising this pull request. This is an interesting approach to the challenge and an area we have considered improving in the past. Until now we have taken the approach of providing sample scripts to assist users with the challenge. On reviewing the approach I can see that this introduces a requirement for access to the Kubernetes APIs to update the labels. In some cases this would not be ideal as many customers run in a multi-tenant environment where they wouldn't want to give these permissions. I would like to hold this Pull Request out at the moment and understand if there are others who would like to see this enhancement and if this solution would address their concerns or not.

Hi @callumpjackson, i can understand that the access to the kubernetes api is not optimal for multi tenant environments. Nevertheless, this access is namespaced provided via the role + rolebinding. And if you are able to install a helm chart, you might as well are able to access the get list and patch APIs for pods.
However, the current solution for updates and changes is based on scripts which are also not(!) ideal for multi tenant environments. They can also easily be executed for the wrong instance, or adapted wrong or else broken.

+1 to this, even if with some templating conditionals to make it an opt-in configuration

Hi @callumpjackson,
I have proceeded as @htaunay has proposed: this is now an opt In feature. It is disabled by default and only enabled by overriding the default values.
Additionally, I have limited the role to just be able to access the mq pods via the resourceNames in the k8s role.
I really would appreciate your feedback.
Best regards,
Lorenz


Request access for DAIR-V2X

Dear Researchers,
I hope this email finds you well. I am writing to ask about the availability of a specific dataset I am interested in using for my research project. After a thorough search, I discovered that the dataset is publicly available for direct website downloading within mainland China. However, since I am located outside mainland China, I understand I need to contact you to request access. The dataset I am referring to is DAIR-V2X.
From my initial assessment, it appears to be a valuable resource for my research. I believe it would significantly contribute to advancing my work in cooperative perception by using infrastructure sensors. I would be grateful if you could provide me access to the dataset. To facilitate this process, please let me know the preferred method of data transfer or any specific steps I need to follow. If any associated costs or fees or usage policies, licensing terms, or restrictions are associated with the dataset, kindly provide the relevant details so I can make appropriate arrangements. Thank you very much for considering my request. I am genuinely excited about the prospect of working with this dataset and believe it will significantly enhance the quality and impact of my research.
I look forward to hearing back from you. If you have any questions or require any additional information, please do not hesitate to let me know.
Best Regards,
Melih Yazgan

Thank you for your interest in our dataset. Please note that, at present, the dataset is officially available for download and use exclusively within Mainland China.
To help researchers become acquainted with our data, we have provided a Google Drive link for direct download of an example dataset.

DAIR-V2X-C-Example: google_drive_link
V2X-Seq-SPD-Example: google_drive_link
V2X-Seq-TFD-Example: google_drive_link

If you wish to utilize our complete dataset, we encourage collaboration with researchers based in Mainland China.


Unit Testing on boarding readme

Unit Testing on boarding readme
Description
Steps to follow to onboard unit testing with cmake, cmock and unity
Checklist:

[ ] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/bot run checks


Offset for menus and tooltips should not be set on the sides

Offsets of positionable components describe the distance to the host component. However, the same value is used for the distance to the browser border and that is not good.
If the offset is set to a value greater than the default value (which looks good out of the box), then the distance to the host component changes as expected. But at the same time the distance to the screen border also increases by the same value.

This should be solved with #112 and if any other side-effects occur where this could cause issues, I would put this under "user fault".


Pagination component accessibility

Currently the pagination component uses  tags without href and is not accessible to keyboard users and screenreaders. Is there any way to add the aria role or other attributes to it to make it accessible?
Buefy version: 0.5
Vuejs version: 2.4
OS/Browser: MacOS Safari/ Chrome

Close it with https://github.com/rafaelpimpa/buefy/commit/8f4fbff766e437e83f5328c70e90d957283daf2d and https://github.com/rafaelpimpa/buefy/commit/30c8027640ad0003c166079515241adfdd609467


BUG: Fix degenerate nans

If the mean of a particular landmark point was the origin for some iteration, the result from then on would become all nans due to a divide by zero.
Updating to just not divide by zero and skip using the landmark with the mean of the origin for that iteration.
The mean will likely not be the origin the next iteration because the rotation will be different.
cc @vicory

LGTM


[clean up] support ddp only

Stack from ghstack (oldest at bottom):

#32
-> #31

@hudeven has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.

@hudeven has imported this pull request.  If you are a Facebook employee, you can view this diff on Phabricator.


module descriptor fails if one jar of many has issues

jarviz module descriptor --classpath=`jbang info classpath env@jbangdev`
java.lang.module.FindException: Unable to derive module descriptor for /Users/manderse/.m2/repository/de/vandermeer/char-translation/0.0.2/char-translation-0.0.2.jar
Unable to derive module descriptor for /Users/manderse/.m2/repository/de/vandermeer/char-translation/0.0.2/char-translation-0.0.2.jar

I would have expected it would list char-translation-0.0.2.jar to not have module descriptor together with rest of the classpath entries.

The current rule is to let the JVM derive the automatic module name is it would normally do. I suppose the exception could be caught when running in batch mode, after all reports already have an entry (named reason) for when such errors may occur.

Released in v0.3.0 -> https://github.com/kordamp/jarviz/releases/tag/v0.3.0


Move rule parsing out of execRule

Move rule parsing out of execRule for rules that run on every lar.
Produces roughly a 25% performance improvement when running a large file locally on the harness.

Coverage increased (+0.07%) to 95.76% when pulling cde531d4651eb4acaa63c990eec81e1afebd17c4 on stevejameskent:code-motion into 140bf51af3d1eb883b04c43cfb572fbf19401126 on cfpb:milestone8.


Update akka-actor, ... to 2.6.15

Updates

com.typesafe.akka:akka-actor
com.typesafe.akka:akka-actor-testkit-typed
com.typesafe.akka:akka-actor-typed
com.typesafe.akka:akka-protobuf
com.typesafe.akka:akka-slf4j
com.typesafe.akka:akka-stream
com.typesafe.akka:akka-stream-testkit

from 2.6.11 to 2.6.15.
GitHub Release Notes - Version Diff
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!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe.akka" } ]

labels: library-update, semver-patch

Superseded by #256.


[Feature]: Support Kubernetes deployment for AMS

Description
Support deploying AMS on Kubernetes.
Use case/motivation
Kubernetes is one of the most popular and widely used container orchestration tools, supporting Kubernetes deployment can facilitate the use of AMS in cloud-native environments.
Describe the solution
No response
Subtasks
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Hi, I want to take this ticket and join this plan in RoadMap 0.5.0

Hi, I want to take this ticket and join this plan in RoadMap 0.5.0

Welcome to the Arctic community!  I assign this issue to you.


Add --snapshot-id option to update in disco_snapshots.py

Adding the option --snapshot-id to update autoscaling group when we already have the snapshot_id in hand.

[x] code
[ ] test with a hostclass

Other than nitpick and John's comment about old comments -> LGTM


Correlated Features Implementation Suggestion

Work in progress

add the test to valied_tests
run it (result.csv gets updated)
check the diff between the output of different explainers

this explainer is failing this test
CorrelatedFeatures TreeShapApproximation
Traceback (most recent call last):
  File "C:\Inn\GitHub\Compare-xAI\reset_experiment.py", line 115, in run_experiment
    _explainer.explain(dataset_to_explain=test.dataset_to_explain, truth_to_explain=test.truth_to_explain)
  File "C:\Inn\GitHub\Compare-xAI\explainers\shap_explainer.py", line 125, in explain
    attribution: shap_lib._explanation.Explanation = self.explainer(dataset_to_explain)
  File "C:\Users\kbelaid\Anaconda3\lib\site-packages\shap\explainers\_tree.py", line 217, in __call__
    v = self.shap_values(X, y=y, from_call=True, check_additivity=check_additivity, approximate=self.approximate)
  File "C:\Users\kbelaid\Anaconda3\lib\site-packages\shap\explainers\_tree.py", line 410, in shap_values
    self.assert_additivity(out, self.model.predict(X))
  File "C:\Users\kbelaid\Anaconda3\lib\site-packages\shap\explainers\_tree.py", line 544, in assert_additivity
    check_sum(self.expected_value + phi.sum(-1), model_output)
  File "C:\Users\kbelaid\Anaconda3\lib\site-packages\shap\explainers\_tree.py", line 538, in check_sum
    raise ExplainerError(err_msg)
shap.utils._exceptions.ExplainerError: Additivity check failed in TreeExplainer! Please ensure the data matrix you passed to the explainer is the same shape that the model was trained on. If your data shape is correct then please report this on GitHub. This check failed because for one of the samples the sum of the SHAP values was 13.217113, while the model output was 2.999100. If this difference is acceptable you can set check_additivity=False to disable this check.

Could you please have a look

plz add it to valid_test
valid_tests = (       ...     CorrelatedFeatures,

plz comment the print in the class to reduce the content of the console

Traceback (most recent call last):
  File "C:\Users\kbelaid\Anaconda3\lib\site-packages\IPython\core\interactiveshell.py", line 3343, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-2-579f3230ff69>", line 1, in <module>
    runfile('C:\\Inn\\GitHub\\Compare-xAI\\src\\aggregate_data.py', wdir='C:\\Inn\\GitHub\\Compare-xAI\\src')
  File "C:\Users\kbelaid\AppData\Local\JetBrains\PyCharm 2022.3\plugins\python\helpers\pydev\_pydev_bundle\pydev_umd.py", line 198, in runfile
    pydev_imports.execfile(filename, global_vars, local_vars)  # execute the script
  File "C:\Users\kbelaid\AppData\Local\JetBrains\PyCharm 2022.3\plugins\python\helpers\pydev\_pydev_imps\_pydev_execfile.py", line 18, in execfile
    exec(compile(contents+"\n", file, 'exec'), glob, loc)
  File "C:\Inn\GitHub\Compare-xAI\src\aggregate_data.py", line 159, in <module>
    export_to_sql()
  File "C:\Inn\GitHub\Compare-xAI\src\export_sql.py", line 14, in export_to_sql
    test = pd.read_csv('../data/01_raw/test.csv')
  File "C:\Users\kbelaid\Anaconda3\lib\site-packages\pandas\util\_decorators.py", line 311, in wrapper
    return func(*args, **kwargs)
  File "C:\Users\kbelaid\Anaconda3\lib\site-packages\pandas\io\parsers\readers.py", line 680, in read_csv
    return _read(filepath_or_buffer, kwds)
  File "C:\Users\kbelaid\Anaconda3\lib\site-packages\pandas\io\parsers\readers.py", line 581, in _read
    return parser.read(nrows)
  File "C:\Users\kbelaid\Anaconda3\lib\site-packages\pandas\io\parsers\readers.py", line 1255, in read
    index, columns, col_dict = self._engine.read(nrows)
  File "C:\Users\kbelaid\Anaconda3\lib\site-packages\pandas\io\parsers\c_parser_wrapper.py", line 225, in read
    chunks = self._reader.read_low_memory(nrows)
  File "pandas\_libs\parsers.pyx", line 805, in pandas._libs.parsers.TextReader.read_low_memory
  File "pandas\_libs\parsers.pyx", line 861, in pandas._libs.parsers.TextReader._read_rows
  File "pandas\_libs\parsers.pyx", line 847, in pandas._libs.parsers.TextReader._tokenize_rows
  File "pandas\_libs\parsers.pyx", line 1960, in pandas._libs.parsers.raise_parser_error
pandas.errors.ParserError: Error tokenizing data. C error: Expected 21 fields in line 46, saw 23


Bug in is_within_weak_subjectivity_period

is_within_weak_subjectivity_period
# Clients may choose to validate the input state against the input Weak Subjectivity Checkpoint
assert ws_state.latest_block_header.state_root == ws_checkpoint.root

This check is currently comparing a state root against a checkpoint (block) root.

open discussion on a PR here: https://github.com/ethereum/eth2.0-specs/pull/2304


Add photographers & graphic designers

Hey, hey,
I really like your style and couple of my friends interior designers recommended your stuff to me.
I would be super happy if you would post some of your products or projects on my app www.zweidesign.co
I just started, developed it all by myself so my wish is to create some Pinterestish gallery of professionals for home improvement projects. But really cool ones. Idea is to work on connecting different architect, renovators etc. with local investors, contractors and with sellers of furniture and decor.
I hope you like it and please, publish something.
Thanks 100x in advance! I am following you:)


PDS 4K

Is there a way of getting which preset is live on from a PDS4k?

currently not available as far as I know

With Barco switchers there is no preset that is "live" but you could track which preset was last recalled in Companion (probably using variables or some other method). But if something else changes an output of the PDS4k then the preset last recalled isn't what is on screen.

On Sep 9, 2024, at 7:43 AM, hadphild @.***> wrote:
Is there a way of getting which preset is live on from a PDS4k?
—
Reply to this email directly, view it on GitHub https://github.com/bitfocus/companion-module-barco-eventmaster/issues/75, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACW2KFQJPZBUYDN2OMTN64LZVWXYDAVCNFSM6AAAAABN4WY6B6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGUYTIMJWHE2TOMI.
You are receiving this because you are subscribed to this thread.


Any parameter / switch to change languages ?

When I programme in PHP, i use
<script src="//www.google.com/recaptcha/api.js?hl=EN" async defer></script>
to load the language for the recaptcha, is there any option / switch for angular-recptcha to change languages ?

It looks like you have to specify the language in the script src, so you'd have to do something like:

Maybe this problem is solved with https://github.com/VividCortex/angular-recaptcha/issues/161

It looks like the issue was resolved. Please reopen otherwise


multiple classes

Currently, if a file contains multiple class definitions, and a class is specified, it will return methods for that class and also all subsequent class. We should make sure it only returns the selected class and no others.

Hi, thank you for the this. I am having trouble when I try to use myopts on a python file that has a class. Especifically, the init  method of a class. Do you perhaps know a solution? It appears that the parser does not recognize it. Maybe it has to do with the formating of the docstring for the init?


XR_ULTRALEAP_hand_tracking_forearm

Monado's hand tracking is getting forearms, and I want to use StereoKit to make sure it works right!
I'll hopefully be poking at the code for real this weekend, but I'm hoping that somebody can point me to examples and where the original XR_EXT_hand_tracking code lives in StereoKit before then. Will update this ticket as I go

Oh heck yeah! The current hand extension is primarily located over here, the xr functions for that are registered in this macro swamp, and it's possible there could be some capabilities stuff that might need to happen closer to initialization.
I was kinda hoping to wrap elbows into a Body class eventually, but I think that's a bit of a bigger task! I'd be happy to see the minimum change necessary in the meantime. I might still like to see an elbow estimation for when XR_ULTRALEAP_hand_tracking_forearm isn't present tho :)

Okay, I've got https://github.com/StereoKit/StereoKit/pull/582 up! This does work but I think we need some more stuff to make sure it still works with runtimes without XR_ULTRALEAP_hand_tracking_forearm. I'll probably poke this again in a few days when I have actual forearm tracking implemented, not just a fake test

Just FYI, I do have some shoulder sim code in SK's core that's used for calculating the hand rays. This is basically some of the early parts of the ReadyPlayerMe stepper's elbow estimation code! Should be helpful for the elbowless platforms :)


Add my name

Thanks for creating this awesome project to help the beginner to learn how to make a request.
(As a native Chinese speaker, I also modify some words in the documents) :)

@WeiChienHsu, we've made a lot of changes to the tutorial recently. if you compare the Chinese translation with the English one, you can see it's outdated. Can you help me out to fix this? I'll create an issue for this and share it with you in this thread.
I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?
We’ve added a tweet feature to our web app recently. Can you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.


Partial with {{> @partial-block }} should not be rendered when processed as stand-alone templates

I am using Pattern Lab Node<EMAIL_ADDRESS>with<EMAIL_ADDRESS>Expected Behavior
Let's suppose we have the following two templates:
Template foo.hbs:
{#> bar }}
  Foo
{{/bar}}

Template bar.hbs:
{{> @partial-block }}
Bar

I would expect this to render as:
Foo
Bar

Actual Behavior
Instead I get the following error message:
[14:01:52] Error: The partial @partial-block could not be found
    at Object.invokePartial (/Users/[...]/test-case/node_modules/handlebars/dist/cjs/handlebars/runtime.js:266:11)
    at Object.invokePartialWrapper [as invokePartial] (/Users/[...]/test-case/node_modules/handlebars/dist/cjs/handlebars/runtime.js:68:39)
    at Object.eval (eval at createFunctionContext (/Users/[...]/test-case/node_modules/handlebars/dist/cjs/handlebars/compiler/javascript-compiler.js:254:23), <anonymous>:6:28)
    at main (/Users/[...]/test-case/node_modules/handlebars/dist/cjs/handlebars/runtime.js:173:32)
    at ret (/Users/[...]/test-case/node_modules/handlebars/dist/cjs/handlebars/runtime.js:176:12)
    at ret (/Users/[...]/test-case/node_modules/handlebars/dist/cjs/handlebars/compiler/compiler.js:525:21)
    at Object.renderPattern (/Users/[...]/test-case/node_modules/patternengine-node-handlebars/lib/engine_handlebars.js:44:12)
    at Object.render (/Users/[...]/test-case/node_modules/patternlab-node/core/lib/object_factory.js:76:26)
    at renderPattern (/Users/[...]/test-case/node_modules/patternlab-node/core/lib/pattern_assembler.js:143:22)
    at Object.renderPattern (/Users/[...]/test-case/node_modules/patternlab-node/core/lib/pattern_assembler.js:536:14)

Steps to Reproduce

Thanks for the report, @teehemkay!

What I think is happening is that Handlebars is being fed a template A that includes a {{> @partial-block }} expression as a "stand-alone" template: meaning not as a template being included as a partial from another template B. And in such a context Handlebars runtime seems to interpret the {{> @partial-block }} expression as a normal include of a partial named "@partial-block" which it can't find of course.
In "normal" Handlebars use case such a situation would typically be an error indeed since template A doesn't make much sense as a "stand-alone" template.
But PL, tries to render every template. And there in lies the problem.
So my current understanding is that {{> @partial-block }} hbs expression should probably NOT be processed when their enclosing template is invoked as a "stand-alone" template.
Also I've looked at the code of the hbs engine and it seems to be doing the right thing by not recognizing @partial-block as the name of a partial.

@geoffp On Gitter,  @bmuenzenmeyer was kind enough to provide some pointers the main modules involved in how the engines are hooked up to the core PL.
This has enabled me to create modify the hbs engine as follows to correct this issue in basically 3 steps:
First add a findAtPartialBlockRE to engine_handlebars with the following value:
findAtPartialBlockRE: /{{#?>\s*@partial-block\s*}}/g`

Second  add an escapeAtPartialBlock  property to engine_handlebars defined as follows:
function(partialString) {
  var partial = partialString.replace(this.findAtPartialBlockRE, '{{> @partial-block }}')
  return partial;
}

And third add the following line to  in renderPattern before compiling the template:
pattern.extendedTemplate = this.escapeAtPartialBlock(pattern.extendedTemplate);

This solved the issue for me.
I'm not sure this is the appropriate way to do it (e.g. may this is internal machinery that shouldn't be exposed in the engine API?) but I hope this helps anyway.
@geoffp  Please let me know if you want me to create a pull request

@teehemkay excellent work. I would love a pull request, with three requests:

Put escapeAtPartialBlock() out in the module scope, since you're right, it doesn't need to exposed as API. While you're at it, move those regexes out to the module scope, and declare  them there as const. We don't need to expose those as API, either!
Try calling escapeAtPartialBlock() in registerPartial() instead. That should get the escaping done earlier in the build process, and seems a bit safer.
(bonus) For mega bonus points, if you want to get your hands dirty, write a test case for this in the handlebars tests in the main patternlab-node repo that fails until you've got your fix applied to the handlebars engine. If you're not familiar with npm link, I highly recommend it for this kind of thing.

I'll be out of town for a week, so no rush.

Also, I think this is my old, personal repo for patternengine-node-handlebars -- I think we should be doing this at https://github.com/pattern-lab/patternengine-node-handlebars instead.

@geoffp Cool. I'll create the PRs later this week.

@geoffp re: point 2. above:  it doesn't work to call escapeAtPartialBlock() in registerPartial().
The escaping must only occur when a partial with a {{> @partial-block }} expression is rendered as a standalone template. Escaping too early such as in registerPartial() would prevent the inclusion mechanism from working when it's needed...

@teehemkay -- good find. Thanks for trying it. I'll check out the PR.

@teehemkay @geoffp
Let me know if / when https://github.com/pattern-lab/patternlab-node/pull/548 can be merged. Sounds like this is still baking?

@geoffp my PR @ https://github.com/pattern-lab/patternlab-node/pull/548 contains tests for this PR. One of them will keep failing as long as this PR as not been accepted...

Cool. I'll review and hopefully merge ASAP.


DNS record being removed

The DNS recorded created by this module is now being removed because it's comparing a bool type var.enabled to the string "true".
I've created PR https://github.com/cloudposse/terraform-aws-cloudfront-s3-cdn/pull/47 to fix this.

PR #47 merged.


adds hf accelerate tracker

Fixes #731
TODO:

[x] Tests
[ ] Docs

Codecov Report
Attention: 9 lines in your changes are missing coverage. Please review.

Files
Coverage Δ

src/dvclive/huggingface.py
83.33% <60.86%> (-16.67%)
:arrow_down:

:loudspeaker: Thoughts on this report? Let us know!.

There


LowestCommonAncestor of two nodes in BinarySearchTree

📚 Documentation
(A clear and concise description of what the issue is.)
Find Lowest Common Ancestor of the given two nodes in a Binary Search Tree given the Binary Search Tree and two nodes.
Have you read the Contributing Guidelines on Pull Requests?
Yes

/assign

/assign

/assign


Search: Show external id in search results

Will make it easier to distinguish where search results have the same

Tried it with extremely long names / extIds and it looks fine:

Looks good, we should probably give serious consideration to entityReference becoming a base class/immutable for all entities.  Lots of overlap.....

Looks good, we should probably give serious consideration to entityReference becoming a base class/immutable for all entities. Lots of overlap.....

Would take one step out of the conversion.

...or perhaps a dedicated EntitySearchResult object which would be more flexible and prevents growth of EntityReference.


SignalDebugger | Add ability to detect signals that are connected at runtime

The SignalDebugger only tracks signals that are connected at build time through the editor. Any signals that are dynamically connected via code are not known to the SignalDebugger.
This enhancement would allow the SignalDebugger to refresh its signal list while the game is running.

This functionality has been added in Release v1.7.0.


Completely incapable of getting the installation requirements working

After following the steps for Ubuntu 16, verbatim, I consistently get this:
from caffe2.python import core
WARNING:root:This caffe2 python run does not have GPU support. Will run in CPU only mode.
CRITICAL:root:Cannot load caffe2.python. Error: No module named caffe2_pybind11_state

I would like some input.
Edit: Basically, ignore these instructions if you have trouble. Your source should be:
https://caffe2.ai/docs/getting-started.html?platform=ubuntu&configuration=compile

I had similar problems that's why I packages up the installation into a self-contained and reproducible Docker environment; see this pull request https://github.com/facebookresearch/VMZ/pull/94

Hah, @daniel-j-h that's good to hear. I didn't see that PR. I too have a docker container now, but it might be beneficial to jump over to yours. Amazing how messed up the instructions are.
A few questions: downloading the data sets to train -- have you had any success? They link to to the ActivityNet from here, but running the scripts from the ActivityNet repo fail because the wonky youtube-dl program is failing to download stuff from YouTube. For example, following the instructions [here](ata/activity_net.v1-3.min.json
root@APOLLO-Linux:~/Downloads/ActivityNet/Cra) result in nothing more than a complete list of "not found" from youtube-dl being spat out into the download_report.json file.
It'd be nice to at least get the thing operational and training something, even a test data set.


utf8

Prepare for utf-8 offsets
reduce code duplication in tests
Make utf8 default, implement utf16 in terms of it
Make it easy to add additional context for offset conversion
Implement utf8 offsets

closes #7453

Would be good to check this against some client!

bors r+

bors r+

gpf

bors r+


no URL configured for submodule "WareHouse/ChatDev"

When cloning main branch from URL https://github.com/OpenBMB/ChatDev.git, I get a message which reads
"no URL configured for submodule "WareHouse/ChatDev"
Running Windows 11.
Configuration bug?

It seems like someone introduced some wrong folders in the software contribution. I have cleaned it. Please try again


title is not optional in dataset schema

https://github.com/chanzuckerberg/cell-census/blob/7152b68c0b3e0debd1fc2b5200f1131bf26a00a0/tools/cell_census_builder/manifest.py#LL63C11-L63C86
"dataset_title": dataset.get("title", ""),  # title is optional in schema
The comment is incorrect and the default value unnecessary. title is required in uns because it is included in Curators MUST annotate the following keys and values in uns.

Done


Converted File is unplayable in iTunes

I can't see any errors in either the Sab or the SB log files, so I have little to go on...
The file that ends up in iTunes shows a total bit rate of 0 kbps and dimensions of 0x0.  The 'Time' value is unavailable and iTunes won't play the file.  One clue I have is that the Apple TV and QuickTime on the computer will play the file, but with no audio or audio tracks available.  VLC will play the file with the expected audio.
I've tried changing the 'output_format' from mp4 to mov. No difference.
I've tried the output_extension as mp4 and m4v.  No difference.
I've changed the audio-codec to 'aac'.  No difference.
I've tried a manual.py conversion, but there are no errors or indicators of a problem.
VLC Codec details indicate the same details as the original:
Code: H264 - MPEG-4 AVC (Part 10) (avc1)
Stream 1:
Type Audio, Codec: MPEG AAC Audio (mp4a)
SickBeard 5.07
Fresh install of sickbeard_mp4_automator, and fresh install of ffmpeg.  The whole process works to get the file where it needs to be, including metatags.
Windows 7 (32bit)
I'm not sure what else to try?  Any assistance or clues as to the problem would be much appreciated.
autoProcess.ini.txt
Log Excerpts.txt

Further to this: Ran ffmpeg manually against the original mkv file in this case and it reported no errors.  Also ran ffmpeg against the converted mp4, and ffmpeg didn't indicate any sign of error.
Overnight, a 'larger file' from a different show came in automatically and the resulting mp4 works properly in iTunes.
A second 'smaller' file that also came in overnight was not working as per the description above.

Can't say I have any ideas, iTunes can be picky for some reason and without logging is hard to pinpoint why it fails
Not sure what build of FFMPEG you're using but I would roll back to the latest stable build and also try a few of the previous builds. FFMPEG tends to change and update way faster than iTunes can support

Thank you! Rolled FFMPEG back to 3.1.4 yesterday and so far with the few files that ran through the process last night, none of the converted files are showing the previously observed symptoms.

Just for additional context, I also had/have this issue. I'm on a Mac and use Homebrew for ffmpeg installation. The problem cropped up with the Homebrew upgrade to ffmpeg 3.2. It seems that 3.2 removes support for faac and instead relies on native support or fdk-aac. I wasn't able to get either to work and instead also rolled back to 3.1.5 to resolve. I didn't try getting ffmpeg to directly encode it correctly to see if it was ffmpeg or sickbeard_mp4_automator at issue but will when I get some time.


net: buf: Make NET_BUF_POOL_USAGE independent of NET_BUF_LOG again

This is a small regression from dd09cbc1c, where additional net buf
pool metadata collection was made dependent on net buf logging. They
aren't related, e.g. 3rd-party apps may want to collect/show such
metadata without relying on Zephyr's logging. (The issue found by
building MicroPython Zephyr port, which offers a function to query
the number of free/total no. of buffers.)
Fixes: #6127
Signed-off-by: Paul Sokolovsky<EMAIL_ADDRESS>

Dup of #6129.

Codecov Report

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

@@           Coverage Diff           @@
##           master    #6130   +/-   ##
=======================================
  Coverage   52.88%   52.88%
=======================================
  Files         412      412
  Lines       40274    40274
  Branches     7801     7801
=======================================
  Hits        21297    21297
  Misses      15761    15761
  Partials     3216     3216

Continue to review full report at Codecov.

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


support for external IPs

ExternalIPs are put as load balancer keys on the default gateway. The default gateway is the one which has the port <IP_ADDRESS> (we used to slap an external id earlier, but not anymore).
ovnkube on the default gateway must be invoked with --nodeport to let the load balancers be installed on the default gateway, or external IPs will be ignored.
To create a service with external IP, see this example.
Tested it to the point where load balancer entries are made properly.
TODO: Test with a configured environment where a packet is sent to the default gw node with the external IP as destination.
@shettyg PTAL. Thanks.
Signed-off-by: Rajat Chopra<EMAIL_ADDRESS>

You will likely need to "sync" this in syncServices(). You will also likely need code in deleteService().
make lint is failing.

/cc @squeed @dcbw

@shettyg PTAL

@rajatchopra
My understanding was that externalIPs are provided like this:
apiVersion: v1
kind: Service
metadata:
  labels:
    name: apacheexterna-ip
    role: service
  name: apacheexternal-ip
spec:
  ports:
    - port: 8800
      targetPort: 80
      protocol: TCP
      name: tcp
    - port: 8900
      targetPort: 90
      protocol: UDP
      name: udp
  externalIPs:
    - <IP_ADDRESS>
    - <IP_ADDRESS>
  selector:
    name: apache

So service.Spec.Type is kapi.ServiceTypeClusterIP. But I see that you have a new check in syncSercvices() for service.Spec.Type != kapi.ServiceTypeLoadBalancer. Can you explain?

LGTM


Test on MS COCO dataset based on trained checkpoint

After I download the trained checkpoint on COCO dataset, after test, the AP are all 0, the command is
python test.py --run_name=BlitzeNet300_x4_coco_det --dateset=coco --split=val2017 --x4 --image_size=300 --detect --ckpt=1
so I don't know what is the problem, looking forward for your help~~~

I can't tell you exactly why you have 0 ap since there is not enough details. However, one thing I  noticed is that you use a split 'val2017' that wasn't in a list of allowed splits initially. Check that you adapted everything correctly.

Thank you for you reply~
I use the coco 2017 val set to evaluate the detection performance and I have modified the corresponding codes to test the model. However, when I use the released model with the command above, all the results are 0. I only modify the code related to the pre-preparation of the dataset and any other codes are the same with the original. So I confuse that if there are configurations or anything else I don't set well. There is another interesting thing is that when I evaluate the release model on Pascal voc 2007, the results are corret~~
so i am confused now and need for your help~~~~

As I said already, 'val2017' is not in the list of supported splits since we have used splits of 2014 year. The corresponding split to 'val2017' is 'minival2014'. So in your command put "--split=minival2014' instead.
Be sure you have the splits downloaded and all the paths (to the dataset and splits) set up correctly.


Create pipes by curve

https://forum.dynamobim.com/t/2023-c3d-pipe-node-test/76580

Not going to have time to implement this.


feat: serialise an AggregateError's sub-errors

ah, AggregateError is an ES2021 feature, and you're targeting ES2020. it's supported in Node since v15, can we bump your target to ES2021?


Multiple recipients

Hello! Thank you for your work on this library. I am trying to fix an issue that I have found where if a message is encrypted with multiple recipients, if the first recipient is not "you" then the message fails to decrypt. I am a bit lost trying to fix this as my fs2 skills are quite a bit weaker than those on display here.
I will continue to work on this, but any advice would be appreciated! So far I've added a unit test that captures the failure I'm experiencing.

Thanks @CJSmith-0141! I pulled the unit test into a new PR that I think fixes the issue; see #170.


Correct package name in package.json

This PR should partially close #253.

@joeferner, could you merge also this one?


ARIA controls. Use with buttons on dynamically created lists

Summary
In PR #401 stories from a (local) database are shown, users have the option to click "Read Full Story" to expand collapsed stories.
Included in the code to expand the stories are ARIA controls which (I believe) could be used to help identify which story is being expanded. I am currently using the title text as the flag as I think that is the most accessible choice but would it be useful upon moderation to have a "short" title specified (and hence expand the PublicExperience class) and use that instead
Deadline
Ongoing
What needs to be done?
Discussion around...

[ ] If aria controls are wanted
[ ] Best way to implement them on pages such as the Shared Stories page

Who can help?

Updates

I'm not super familiar with how ARIA works, but the example that Mozilla gives for the aria-controls part seems to just use the id of the div rather than an actual textual description for making that linkage. Maybe the solution is to just use the experience ID for the id of the expanding panel (as already implemented here)?

I'm happy to do that - I am not at all familiar with the implementation of ARIA on devices, I will look into it. I will flag that the experience ID is the file UUID so it is a bit unwieldy if it needs to be read to a user

Thanks! I suspect that the ID is just used for properly linking it internally for accessibility devices and that they aren't being read out/displayed to users, according to this article:

When a User Agent (UA) supports aria-controls, it makes it possible for focus to be moved from the current element directly to the element it controls.

So it's mostly to automatically redirect the user to the place where a change happened post-press of a button/expand label but not for the "frontend" for the user.

@helendduncan. Closed with https://github.com/alan-turing-institute/AutSPACEs/pull/401/commits/d77c82057e6596d61c98119d62663b916e83c540?

@helendduncan. Closed with d77c820?

Yes


Install operators first in kuttl_db_prep, kuttl_common_prep

Operators install well in parallel, so the pattern of installing one operator then deploying one service suggests there is potential for optimizing kuttl run times.
The following scenarios were tried, with measured time savings (sample size 2):

kuttl_db_prep install mariadb and infra operators in parallel
kuttl_common_prep install rabbitmq and keystone operators in parallel
65 seconds quicker than baseline

kuttl_common_prep inline all of kuttl_db_prep and install 4 operators
in parallel
94 seconds faster than baseline

Based on this, 2) is proposed in for kuttl_common_prep, and kuttl_db_prep also gets a small speed improvement.

Wondering if we want to split kuttl_db_prep to kuttl_db  and kuttl_db_deploy and kuttl_common_prep to kuttl_common and kuttl_common_deploy, so that we can list all deploy targets after all operator targets, to pursue further concurrency.

Wondering if we want to split kuttl_db_prep to kuttl_db and kuttl_db_deploy and kuttl_common_prep to kuttl_common and kuttl_common_deploy, so that we can list all deploy targets after all operator targets, to pursue further concurrency.

Good idea, maybe as a follow-up after this merges?

Wondering if we want to split kuttl_db_prep to kuttl_db and kuttl_db_deploy and kuttl_common_prep to kuttl_common and kuttl_common_deploy, so that we can list all deploy targets after all operator targets, to pursue further concurrency.

Good idea, maybe as a follow-up after this merges?

Yes it can be a follow-up.

/retest


add integration tests for cpu count in virtualbox driver

adds integration test for custom virtualbox CPU count flag (--virtualbox-cpu-count) that was introduced in PR #879
Signed-off-by: Ken Pepple<EMAIL_ADDRESS>

Didn't know about this. Thanks!

LGTM


fixes spelling errors, first pass

here's a first pass at fixing spelling errors.

Manually merged. Thank you very much @sturobson. :)


Annotation override of default ignored for ID field

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior 😯

When defining a schema that overrides the ID field to make it a UUID, using an Annotation to override the default does not work:
type Entity struct {
	ent.Schema
}

func (Entity) Fields() []ent.Field {
	return []ent.Field{
		field.UUID("id", uuid.UUID{}).Default(uuid.New).Annotations(&entsql.Annotation{
			Default: "gen_random_uuid()",
		}),
		field.UUID("not_id", uuid.UUID{}).Default(uuid.New).Annotations(&entsql.Annotation{
			Default: "gen_random_uuid()",
		}),
	}
}

Generated SQL has the correct default for not_id but not for id:
BEGIN;
CREATE TABLE IF NOT EXISTS "entities"("id" uuid NOT NULL, "not_id" uuid NOT NULL DEFAULT gen_random_uuid(), PRIMARY KEY("id"));
COMMIT;

Expected Behavior 🤔

I would expect the generated SQL to include the DEFAULT gen_random_uuid() for the id field.
Steps to Reproduce 🕹

Steps:

Define schema as above
Examine SQL generated with client.Schema.WriteTo

Your Environment 🌎

Tech
Version

Go
1.16.6

Ent
0.8.0

Database
Postgres

Driver
github.com/lib/pq

Hey @captainbeardo, the issue was fixed with https://github.com/ent/ent/pull/1492. Updating to the latest master using go get entgo.io/ent@master will solve the problem.
Closing, but please reopen if you have any issues with it. You're also welcome to join our Slack channel :shipit:

Thanks for the quick response @a8m.


Features needed for hardhat-tracer

Must have

[ ] get list of opcodes that are activated for current hardfork (previously I took that from vm.common)
[ ] add isStaticCall to MinimalMessage
[ ] add more details to MinimalEVMResult like self destruct beneficiary, exception error details, return data or revert data
[ ] add memory on the step
[ ] add more stack elements - currently at most i check upto 6 top stack elements to decode LOG4.

Nice to have

[ ] add isDelegateCall to MinimalMessage

get list of opcodes that are activated for current hardfork (previously I took that from vm.common)

We can add a function that receives the hardfork and returns the list of opcodes. The hardfork can be obtained from hre.config.networks.hardhat.hardfork.

add isStaticCall to MinimalMessage

Will do

add more details to MinimalEVMResult like self destruct beneficiary, exception error details, return data or revert data

Return/revert data seems an obvious thing to have here, and it should be easy. I'm not sure how hard it will be to add a selfdestruct beneficiary; we'll look into it. And can you expand on what do you mean by "exception error details"?

add memory on the step
add more stack elements - currently at most i check upto 6 top stack elements to decode LOG4.

These two I'm not sure about. We are returning the whole tracer from EDR to Hardhat, and both of these would make it much bigger (and it's already huge in some cases, which can have a significant performance impact). Maybe we can have a different "mode" that you explicitly enable and gives you extra data?

add isDelegateCall to MinimalMessage

This we can do, but I think you should also be able to infer this by comparing the recipient and the codeAddress; if they are different, then it's a delegate call.

And can you expand on what do you mean by "exception error details"?

Things like "out of gas" or "stack overflow", as displaying it in the trace is more useful instead of just showing the call reverted.

Maybe we can have a different "mode" that you explicitly enable and gives you extra data?

Yes this makes sense, only activated by plugin when it expects to provide trace details to user.

Closing in favor of https://github.com/NomicFoundation/edr/issues/299. Please let me know if we are missing something there.