FFmpeg has native aptX codec support

FFMpeg devel added native aptX support https://patchwork.ffmpeg.org/patch/5879/. Do you think it is possible to add support for it in bluez-alsa?

Wow, indeed it looks like aptX. It's a pity, that they've added it to the ffmpeg core... Anyway, yes, I think it is possible to integrate ffmpeg with bluez-alsa.

It's already in master: https://github.com/FFmpeg/FFmpeg/blob/master/libavcodec/aptx.c.

This permits to enable Aptx support using FFmpeg
https://github.com/Samt43/BluetoothAPTXForLinux

@Arkq @Samt43
ffmpeg also includes an sbc codec so I would be nice if we could use ffmpeg instead of separate libraries in the end.

@diizzyy as an optional configuration why not, but as a default I'm not sure.
$ qsize ffmpeg
media-video/ffmpeg-3.3.6: 235 files, 24 non-files, 21612.193 KiB
$ qsize sbc
media-libs/sbc-1.3: 12 files, 9 non-files, 221.750 KiB
$ qsize fdk-aac
media-libs/fdk-aac-0.1.5: 10 files, 8 non-files, 906.194 KiB

Bare in mind, that my ffmpeg configuration is pretty "light" :) Furthermore, (not sure if still valid, though) the ABI of ffmpeg libraries is a nightmare, so practically ffmpeg upgrade forces depending applications to be rebuilt.

Hmm but aren't systems going to have ffmpeg anyway (if you are worried about size)?

Allow me to pose a naive question: how much work would it be to rip out the aptx en/de-coder from ffmpeg and create a standalone aptX encoder/decoder library useful for bluez-alsa?
Also, where exactly does the SBC codec live? In code form? Is it bundled with bluez?
Does the code in this repository only provide the glue between bluez and ALSA? Sorry, but I have never been able to overcome the confusion w.r.t. bluetooth audio and dependencies. Being able to eliminate as many dependencies as possible (including PA....) may prove helpful.

Ok. I was just curious. From this, I understand that even SBC will be provided through ffmpeg? How does the code in this repo provide support for SBC today?

@dagbdagb
It links with the sbc library from bluez (stand alone lib).

OK, I admit, my previous comment might have a little bit of naive thinking :/
So, to be fair with you guys. I've got mostly one concern (and in general two concerns) when creating software for fun (and bluez-alsa is my spare time project, which to my surprise gained same attention). Firstly, it has to run fast (PA seems to fail in this area), and I'm not saying about compiler optimizations, but design which will allow fast computing (e.g. in bluez-alsa I'm not using mutexes for threads synchronization). Secondly, it should have small memory footprint. These points determine what I'm willing to do, and what I'm not :)
So, back to this issue. As far as I know, there is (was) a quarrel in the ffmpeg community about compatibility and new functions (I might be wrong, though). So the libav emerged which supposed to be a remedy for API breaking. This case alone is a big "warning" sign for me. But OK, let say, that right now the project is properly managed - there is no more "happy" development in the ffmpeg. ffmpeg library contains a lot of unnecessary codecs (from the Bluetooth point of view), which I've been trying to express in my previous comment (maybe not in the healthiest way) :D HOWEVER, if codecs (SBC, APTX, AAC, etc.) from ffmpeg produce better overall performance I will switch to ffmpeg - performance is more important for me.
Right now, I've got no time for tinkering with codecs, because I'm trying to make the msbc branch working. However, with my limited testing resources, it seems to be more complicated than I thougt.... :D

So I had a brief look at this and ffmpeg's sbc codec is based on the one found at https://www.kernel.org/pub/linux/bluetooth/ which is what bluez-alsa uses so performance should be equal or better. However I'm don't understand how it actually handles different settings such as subbands, bitpool, joint stereo(?) and blocks and they're not exposed as far as I can tell and there's no documentation available. This might be tied to the sbc format itself but I'm not sure. The author is on GitHub but let's give it a try @aurelj .
This is what ffmpeg 4.0 exposes:
ffmpeg -h encoder=sbc
Encoder sbc [SBC (low-complexity subband codec)]:
    General capabilities: small
    Threading capabilities: none
    Supported sample rates: 16000 32000 44100 48000
    Supported sample formats: s16
    Supported channel layouts: mono stereo

@diizzyy
In my initial SBC patch submit to ffmpeg I did include options such as subbands, bitpool... But it was suggested to me that this was not a nice interface and that it didn't match how other codecs are used. You can follow this thread for the details: http://ffmpeg.org/pipermail/ffmpeg-devel/2018-February/thread.html#225644
So now, all those SBC settings are calculated automatically for best quality. The main option you can use to constrain those settings is the bitrate (eg. -b:a 320k). If you want to reduce encoding delay you can set a delay constraint (eg. -max-delay 5m for 5 milliseconds), which will also affect the number of subbands and blocks (possibly reducing quality).
Of course you can also choose the number of channel (1 or 2, eg. -ac 1) and the samplerate (eg. -ar 16000), which will also affect the settings computed to match the target bitrate.
If you want to dive into the details, have a look at the sbc_encode_init() function: https://github.com/FFmpeg/FFmpeg/blob/master/libavcodec/sbcenc.c#L195
Also if it reveals to be absolutely necessary to be able to set subbands, bitpools and other settings manually to fit in the A2DP protocol, let me know. In this case, it would certainly be possible to add direct control of those settings in ffmpeg.

@aurelj
Ahh, I see. Many thanks for replying.

This case alone is a big "warning" sign for me. But OK, let say, that right now the project is properly managed

It should be indeed. Libav is pretty much on life support nowadays AFAIK. See also https://github.com/HandBrake/HandBrake/issues/974

@pali : nice! Is aptX low-latency (LL) supported aswell, or is that just a matter of configuring the right parameters?

No, aptX Low Latency is not there.

Correction about aptX low-latency: Stream produced by aptX and aptX-low-latency is same, so libopenaptx already supports aptX-low-latency. The difference between aptX and aptX-low-latency is on lower layer part bluetooth part (size of bluetooth packets, codec selection) which is irrelevant for libopenaptx.

@pali Thanks for your sharing your insight. Can you explain this in more detail? Especially how LL can be actually used.

@joerg-krause I have aptX-LL patches for pulseaudio and I will publish them after all my other pulseaudio a2dp patches are merged as aptX-LL is based on them.

That's good news! What does it mean for bluez-alsa? Are there any patches required to enable aptX-LL, too?

Of course, it is needed to implement A2DP handshake part.

@joerg-krause in bluez-alsa you can decrease aptX latency by feeding encoded data into the bluez socket more frequently. This loop should iterate over smaller number of samples: https://github.com/Arkq/bluez-alsa/blob/a30207dd173111a9b3618adbebe7469a60501576/src/a2dp.c#L1557-L1562 Beyond that, you will might use bluealsa D-Bus API directly (omit ALSA layer which introduces extra latency).

aptX-LL supports also optional voice backchannel, so you can use it for parallel music streaming and microphone. And when aptX-LL is used on headset (instead of aptX) then it it more cases provides lower latency at headset side.

I'm closing this issue, because I will not switch to FFmpeg for apt-X encoder (and/or decoder). In the openaptx repository, I've posted benchmark results of various publicly available encoders, and the performance of the FFmpeg based encoder is almost two times worse than that of the original Qualcomm library (in some part due to FFmpeg generic API).

I'm closing this issue, because I will not switch to FFmpeg for apt-X encoder (and/or decoder). In the openaptx repository, I've posted benchmark results of various publicly available encoders, and the performance of the FFmpeg based encoder is almost two times worse than that of the original Qualcomm library (in some part due to FFmpeg generic API).


Fix for compatibility with the chef client 12.

Path attribute was deprecated in resource execute in  chef client 12. I fixed cpan_module definition for compatibility with chef client 12.

@jtimberman, ping.

Thanks!


Add cookie jar

Codecov Report

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

@@           Coverage Diff           @@
##           master       #4   +/-   ##
=======================================
  Coverage   48.78%   48.78%
=======================================
  Files           1        1
  Lines          41       41
=======================================
  Hits           20       20
  Misses         16       16
  Partials        5        5

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 8d5adbc...82a4175. Read the comment docs.


Toolbar color not correct when on task management mode

Changed "Primary Color"

Changed "Primary Color"


esphome: 2021.9.1 -> 2021.9.2

Motivation for this change
https://github.com/esphome/esphome/releases/tag/2021.9.2
https://github.com/esphome/esphome/security/advisories/GHSA-48mj-p7x2-5jfm
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Security issue tracked as CVE-2021-41104.


"The given data was not valid JSON."

I am receiving
dataCorrupted(Swift.DecodingError.Context(codingPath: [], debugDescription: "The given data was not valid JSON.", underlyingError: Optional(Error Domain=NSCocoaErrorDomain Code=3840 "Invalid value around line 1, column 0." UserInfo={NSDebugDescription=Invalid value around line 1, column 0., NSJSONSerializationErrorIndex=0})))
when running this code and debugging. On the phone, I am receiving the error "Something went wrong." I've tried to look into it but I haven't quite got to dig into swift enough to know how to fix written code. This happens when attempting to connect to the pwnagotchi.

Found when randomly browsing silsha's repositories, there is a pwnagotchi-state-api that has to be cloned and moved within the pwnagotchi.

Glad you figured it out for yourself. Please also see the Setup Guide.

currently going through this, as the guide is in yml format and pwnagotchi now uses TOML

Ah, then they changed it since I last used my pwnagotchi.
I guess there is no way to send pull requests for the wiki, but let me know if you think I should change anything.


os/user package can not cross compile.

os/user package need cgo support, buf if enable cgo, we can not cross compile.
So, i use "github.com/mitchellh/go-homedir", replace the os/user package
in file spawn_process.go, function SetBaseDirectory(dir string) error {...}
Thanks.

Please create a pull request

We need to replace os/user with go-homedir


Rules are not working

The rules are not working, (at least) in the current version of the VSCode extension on both windows and linux. Formatting or schemas are not applied.
By changing versions of the extension I have found it stopped working with version v0.17.0.

I also confirm that the bug exists

I don't think it is fixed. Could you check it works for you? If so, I will provide example repo.

Thank you

Sorry about the delay and the silence, I am a bit all over the place.

Still not working in v0.19.0


Gpu cpu dep

Summary
Provide a concise summary of the changes introduced by this pull request. Detail the purpose and scope of the changes, referencing any relevant issues or discussions. Explain how these changes address the problem or improve the project.
Test Plan
In this section, describe the testing you have performed to verify the changes. Include:

A clear description of the testing environment.
The steps you followed to test the new features or bug fixes.
Any specific commands used during testing, along with their outputs.
A description of the results and observations from your testing.
This information is crucial for reviewers to understand how the changes have been validated.

Additional Notes
Include any other notes or comments about the pull request here. This can include challenges faced, future considerations, or context that reviewers might find helpful.

This PR looks good to me.
This PR includes and enhances previous PR ( https://github.com/mlcommons/chakra/pull/57).
Once this PR gets merged, I will close the older one.


⚠ Orpheus has degraded performance

In 59b47b0, Orpheus ($OPS) experienced degraded performance:

HTTP code: 502
Response time: 1213 ms

Resolved: Orpheus performance has improved in febf313.


Atari wrapper kwargs missing

Question
I wanted to try your project (which by the way is a great project) to train ppo on the breakout game. Only I realized that he was able to progress well on BreakoutNoFrameskip  (about 400pt) but on Breakout-v0 he was blocked at 15pt. After some research, I found out that the Atari wrapper only kept one frame out of 4 in order to increase the training speed.
https://github.com/DLR-RM/stable-baselines3/blob/ddbe0e93f9fe55152f2354afd058b28e6ccc3345/stable_baselines3/common/atari_wrappers.py#L233
So in the end on the breakout-v0 we keep only one frame out of 16 which prevents it to progress.
A solution would be to add the wrapper_kwargs argument in the exp_manager.py file in order to put a custom frame_skip in the hyperparameters.
https://github.com/DLR-RM/rl-baselines3-zoo/blob/master/utils/exp_manager.py#L516
Currently it is not there, is it an oversight?
Checklist

[ x] I have read the documentation (required)
[ x] I have checked that there is no similar issue in the repo (required)

Hello,
what you are describing is actually already possible : https://github.com/DLR-RM/rl-baselines3-zoo/blob/master/hyperparams/sac.yml#L75
See readme also: https://github.com/DLR-RM/rl-baselines3-zoo#env-wrappers

Hello @araffin ,
I've already seen this possibility however we must use an another wrapper than the atari one...
I don't know if there is a huge difference between both but I guess that atari wrapper is a better choice for atari games.

what i was showing is only the syntax, you should of course use Atari wrapper for atari games https://github.com/DLR-RM/rl-baselines3-zoo/blob/master/hyperparams/ppo.yml#L2

okay understood, thank for you reply !


uv cache prune --ci prunes everything in the cache

I've just upgrade to uv==0.3.0 and was happy to see a tip about caching in Github actions in the documentation.
Although, by applying what is described in the docs, and running uv cache prune --ci, my entire cache is pruned and I'm left with  a 240KB cache, which is far too small for the dependencies that I have.
Am I misunderstanding how this command should work?

For some use cases, we will prune nearly the entire cache. The details are described in the docs.

Yeah, it's strange, but it turns out that it's actually (a lot) faster to just re-download from the registry than it is to upload and then re-download from GitHub Actions (which is roughly what happens when you cache the wheels that came from the registry). So counterintuitively, our uv cache prune --ci, removes any wheels that were downloaded from a registry, but keeps any source distributions that you built locally (since those tend to be costly).


chore(deps): update module github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper to v1.42.8

This PR contains the following updates:

Package
Type
Update
Change

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.42.7 -> v1.42.8

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.42.8
Compare Source
Bug Fixes

deps: update module github.com/go-git/go-git/v5 to v5.13.0 (#908) (94d589b)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

ℹ Artifact update notice
File name: tests/go.mod
In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

6 additional dependencies were updated

Details:

Package
Change

github.com/ProtonMail/go-crypto
v1.0.0 -> v1.1.3

github.com/cyphar/filepath-securejoin
v0.2.4 -> v0.2.5

github.com/go-git/go-billy/v5
v5.5.0 -> v5.6.0

github.com/go-git/go-git/v5
v5.12.0 -> v5.13.0

github.com/skeema/knownhosts
v1.2.2 -> v1.3.0

golang.org/x/net
v0.31.0 -> v0.33.0

/run pipeline


[BCP] "Leave site" is displayed when closing BCP

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Instead of just closing the Tab. BCP displays the follwoing popup:

Steps to reproduce

open eat.bananacakepop.com
open a document
type "foo"
close the browser tab

Relevant log output
No response
Additional Context?
No response
Product
Banana Cake Pop
Version
1.0.4

@PascalSenn The message is misleading, because might be that there is nothing to save. The message should state something like "Your are going to close the App.". Even this behaviour could be behind a user flag.

@PascalSenn cannot reproduce

@artola  I guess there should be no message at all, as this is really annoying when you work on you server and every single time you have to press "leave" when you close a tab.
@rstaib happens in chrome based browsers. you have to have a unsaved change though. my bad

@PascalSenn that's not a bug it's a feature

@rstaib but you do not loose state when you close the tab right?

@PascalSenn, of course, you can lose state as soon as a browser decides whether a certain indexeddb has to be cleaned up.

Not at my PC at the moment, but it was happening every time that I built the project which resulted in the tab being reloaded and being stuck with this message, sometimes duplicated.
I think it would be better to just auto-persist to local storage or something.

@glen-84 we're persisting tabs to the indexeddb, but as I pointed out it's not guaranteed to be safe. that's why we nowadays sync to the cloud. As soon as you save a document and sync with the cloud your documents are safe 👍🏻

@rstaib  i mean this does not happen often. Personally i have never had this cleanup thing of the browser.
I think this popup is more in the way than helpful.

I find very helpful this feature, especially when the using try to navigate with the "go back" as is use in SPA, and just would leave the application, even losing changes.
But, I would suggest to improve the wording of the message and add a flag in the user settings to disable it.

I'm not signed in though. It should just auto-persist locally if you're not signed in. 🙂
I don't really expect the persistence to be 100% safe, these are just queries.
@artola With auto-persistence (debounced typing or whatever), 99.9% of the time the prompt will be unnecessary.
It completely breaks the development flow.

@glen-84 From my point of view, even if there nothing to sync, I would like to have this warning before leaving the app.
For example, I do the same with Chrome and VSCode, just pressing Cmd+Q should ask before close.
About these are just queries I have some colleagues that spent hundred of hours building their queries, they can not just lose them, therefore sync to the cloud is due.

That makes no sense, if there's nothing to sync (no unsaved changes), then why do you need to be warned about closing something? It's a UX anti-pattern.
Also, VSC doesn't prompt me when I close it. Even with unsaved changes (they are still persisted to UI state).
I'm not sure why your colleagues are writing so many queries in the browser (reporting queries?), but that seems like an uncommon use case. Regardless, I'm not against prompting if there are unsynced changes – the changes should just be synched more often, especially locally (which can then cloud-sync if/when the browser is reopened).
I really appreciate the work that you guys do, but this feature is broken (at least when not signed in).

@glen-84 we'll show this message with the next update only when someone is signed in and has some unsaved changes in at least one tab. Still, be aware that the browser can clean up any local storage anytime. Meaning that your documents are not safe if not synced with the cloud. Apart from that, thanks for your feedback.

The warning will be disabled for now in version 2.0.0, which will be released in the mid of next week.


Switching between app and workload graph views

The graph currently has two options for how its displayed: an app view and a workload view.
If all the workloads have the app label applied, then we show the app graph. A user in this situation may want to display the workload graph directly as well.
If any of the workloads is missing the app label, then we display the workload graph. A user in this situation would not be able to view the app graph.
We need to be able to visualise to the user what graph type they are currently using. And if they can display both provide an option to switch graph types.

@kiali/ux-reviewers any feedback would be helpful

In addition, the "app" graph has two "sub-types" - versioned and versionless. For example, a versioned graph shows all the versions for a particular app, like this:

whereas a versionless graph aggregates all versions into a single versionless app node, like this (this is the same app graph as above, only it is versionless):

@mwringe is an app generally a superset of one or more workloads?

@mwringe is an app generally a superset of one or more workloads?

Yes. That is exactly right. An "app" can consist of several workloads.

Note that you can have versioned apps (that is, two labels - "app" and "version" - can be on a workload). It is currently an open question whether a "versioned app" refers to one and only one workload or whether a versioned app can refer to multiple workloads. See: https://groups.google.com/forum/#!topic/istio-dev/Ll9AxJjZYaY

@jmazzitelli version and versionless is another concept that should be discussed separately.

@cshinn @serenamarie125 can this also be added to P1? it's a selector between 2 types of views, probably quite a standard thing ;)

https://issues.jboss.org/browse/KIALI-1229

It might be between 3 things right now: the app, versioned app, and workload.
The app and versioned app might get merged into one thing, but that is still being investigated.

I'd suggest we have a "View by" dropdown which has Application & Workload as 2 options

@mwringe add a screenshot of what this looks like right now ... and then we can provide quick feedback

@cshinn @serenamarie125  Its just a simple dropdown right now with the label "Graph type" and options "App", "Versioned app" and "Workload"

@mwringe I think this works well at the moment. I think it's heavily related to #41 as well as the location of the graph layout toggle, so I'll include suggestions with any designs for those

Closing this issue as I think its been resolved.


Allow PVC of completed PipelineRun to be deleted

Feature request
For PVCs specified as part of a PipelineRun with volumeClaimTemplate:

Once a PipelineRun has completed, remove the pvc-protection finalizer from the PVC to allow it to be deleted by the user (similar to https://github.com/argoproj/argo-workflows/issues/6629)
Provide an option to automatically delete the PVC on completion of PipelineRun

Use case
If a PVC is specified as part of a PipelineRun with volumeClaimTemplate, the PVC lifecycle is the same as the PipelineRun. In the case where PipelineRuns are kept for longer periods this is wasteful of storage resources and some users may want to delete the associated PVC before the PipelineRun, perhaps to keep logs, or because the PipelineRun is managed by a gitops controller.
See #1784 for when this was first noticed after PVC protection had been implemented in Kubernetes

Hey @jlpettersson - a completed pod no longer has the PVC mounted - easily demonstrated as it doesn't prevent a RWO PVC from being mounted by another node. However the PVC protection feature prevents PVCs from being deleted if referenced by pods, even if in completed state.
The linked Argo workflows PR removes the PVC protection finalizer when it knows all the pods are completed and it's safe to do so

/remove-lifecycle stale Still an issue

/remove-lifecycle stale

Just came across this issue this morning where I didn't expect the PVC to be seen after the pipelineRun was completed.
I am trying to understand... what is the use of such PVCs? Aren't they unnecessary reserving resources? Shouldn't this be a bug rather than a feature?
I think, the pvc created with volumeClaimTemplate should be deleted once the pipelineRun is completed.
/cc @skaegi

@pritidesai An option to clean up PVCs per PipelineRun would be awesome! I suppose in some cases people might want the option to keep them around for debugging?
For various reasons I have to keep completed PipelineRuns around, and if I were using volumeClaimTemplates each would consume it's own PVC which would be wasteful and expensive (I'm building 50+ apps). As a workaround I have to use a shared RWX PVC as the workspace with a subPath :vomiting_face:

Related: https://github.com/tektoncd/pipeline/issues/5412

Hey @jimmyjones2 I have created a PoC here - https://github.com/tektoncd/pipeline/pull/6635 Please let me know if this works for you. Thanks! Once we finalize on the design, I will update the PR to include tests. The changes in the PR are tested locally.

@QuanZhang-William Just tried v0.51.0 with coschedule: workspaces and was expecting the PVC to be deleted as per #6940 - however I was left with a PVC in Terminating state as the kubernetes.io/pvc-protection wasn't removed. Am I doing something wrong?

Hi @jimmyjones2,  https://github.com/tektoncd/pipeline/pull/6940 only supports purging PVC with coschedule:pipelineruns mode. The change is not applied to coschedule:workspaces mode due to backward compatibility concerns.
You can see more detail in this discussion

Sorry @QuanZhang-William I wrote that incorrectly - when I tried coschedule: pipelineruns I sometimes ended up with terminating PVCs. With cochedule: workspaces I got the previous behavior of the PVC remaining after completion.

@jimmyjones2 . Thanks for reporting the bug! With some investigation, the flake is caused by the wrong order of purging finalizer and deleting PVCs.
It seems like finalizers cannot be patched/purged when the PVC is not in terminating status, so we need to delete the PVC first and then purge the finalizer.
I'm working on a PR to fix it.

/reopen
/remove-lifecycle rotten


typeahead_setting: Fix the show up of user group with 0 active user

Description:
Fixed the issue that when the user group has 0 current subscriber due to deactivated user or leaving of group, then the user group is now not showing up in the typeahead.
Fixes: #30890

Before
After

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

The HTML template is not the right place to do this sort of exclusion; that should happen in JS. Closing this.


Edit pass of "DevTools overview"

Rendered page for review:
https://review.docs.microsoft.com/en-us/microsoft-edge/devtools-guide-chromium/?branch=pr-en-us-1481
Reviewers:

Any ideas for improving the purpose/focus of this article?
The list of article links at top -- it's a nav section, and detracts from the article.  Should we remove or move that list of article links?  Should we keep the list, but remove the hyperlinks, and create a See also list at bottom of article?  This is excess use of hypertext, not enough separation-of-concerns between "present & read this article" vs. "navigate to a different article".
An earlier PR fixed many problems in this article.

Docs Build status updates of commit d9e27b6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit bb6b9b3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 9b792d3:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit aab73af:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/device-mode/geolocation.md
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/device-mode/index.md
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit 0a5e3c7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/device-mode/geolocation.md
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/device-mode/index.md
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/index.md
:white_check_mark:Succeeded
View

microsoft-edge/devtools-guide-chromium/shortcuts/index.md
:white_check_mark:Succeeded
View

microsoft-edge/toc.yml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Femto Mega's output topic timestamp is wrong

We are using Femto Mega with the latest main branch
OrbbecSDK_ROS1, but the timestamp of the output ros topic was incorrect.
$ roslaunch orbbec_camera femto_mega.launch color_fps:=15 depth_fps:=15 enable_colored_point_cloud:=true depth_registration:=true camera_name:=femto_meg
$ rostopic echo /femto_mega/color/image_raw --noarr -n1
header:
  seq: 0
  stamp:
    secs:<PHONE_NUMBER>
    nsecs:  12999936
  frame_id: "femto_mega_color_optical_frame"
height: 720
width: 1280
encoding: "rgb8"
is_bigendian: 0
step: 3840
data: "<array type: uint8, length: 2764800>"
---

(1623747315 = Tue Jun 15 2021 08:55:15 GMT+0000)
I think the reason for this is that the rosparam of the default use_hardware_time is set to true in the following part.
https://github.com/orbbec/OrbbecSDK_ROS1/blob/b49127fa8abed4200bb0e68791e2e77e3a55defb/src/ob_camera_node.cpp#L149
If we change femto_mega.launch and set the rosparam of use_hardware_time to false, the system timestamp will be used and the desired behavior will be achieved.
$ rostopic echo /femto_mega/color/image_raw --noarr -n1
header:
  seq: 11
  stamp:
    secs:<PHONE_NUMBER>
    nsecs: 391756032
  frame_id: "femto_mega_color_optical_frame"
height: 720
width: 1280
encoding: "rgb8"
is_bigendian: 0
step: 3840
data: "<array type: uint8, length: 2764800>"
---

(1732250614 = Fri Nov 22 2024 04:43:34 GMT+0000)
It seems that the arg for use_hardware_time is provided for the launch of some cameras that use hardware time, such as femto bolt, so it would be better to either change the default for use_hardware_time rosparam to false, or add the arg for use_hardware_time to the launch of cameras such as femto_mega (like https://github.com/Kanazawanaoaki/OrbbecSDK_ROS1/tree/add-use_hardware_time-to-femto-mega ).

Thanks for your suggestion. We will check and revise it.


No responding after some days

Hello,
I installed Homebridge on a rasperrry Pi 3B and everything is working fine.
But after some days when I go in the Home App and at all items I get the Information no update.
Them I restart the Homebridge and everything I working fine again.
Here is you can see my system log file.

Oct  7 06:47:01 raspberrypi CRON[16128]: (root) CMD (test -x /usr/sbin/anacron || ( cd / && run-parts --report /etc/cron.weekly ))
Oct  7 07:17:01 raspberrypi CRON[16758]: (root) CMD (   cd / && run-parts --report /etc/cron.hourly)
Oct  7 08:17:01 raspberrypi CRON[18010]: (root) CMD (   cd / && run-parts --report /etc/cron.hourly)
Oct  7 08:52:00 raspberrypi systemd[1]: Starting Daily apt download activities...
Oct  7 08:52:02 raspberrypi systemd[1]: Started Daily apt download activities.
Oct  7 08:52:02 raspberrypi systemd[1]: apt-daily.timer: Adding 11h 48min 22.279353s random time.
Oct  7 08:52:02 raspberrypi systemd[1]: apt-daily.timer: Adding 8h 3min 35.173561s random time.
Oct  7 09:17:01 raspberrypi CRON[19426]: (root) CMD (   cd / && run-parts --report /etc/cron.hourly)
Oct  7 10:17:01 raspberrypi CRON[20743]: (root) CMD (   cd / && run-parts --report /etc/cron.hourly)
Oct  7 11:17:01 raspberrypi CRON[22046]: (root) CMD (   cd / && run-parts --report /etc/cron.hourly)
Oct  7 12:17:01 raspberrypi CRON[23350]: (root) CMD (   cd / && run-parts --report /etc/cron.hourly)
Oct  7 13:17:01 raspberrypi CRON[24785]: (root) CMD (   cd / && run-parts --report /etc/cron.hourly)
Oct  7 14:17:01 raspberrypi CRON[26074]: (root) CMD (   cd / && run-parts --report /etc/cron.hourly)
Oct  7 15:17:01 raspberrypi CRON[27391]: (root) CMD (   cd / && run-parts --report /etc/cron.hourly)
Oct  7 16:17:01 raspberrypi CRON[28673]: (root) CMD (   cd / && run-parts --report /etc/cron.hourly)
Oct  7 16:19:31 raspberrypi systemd[1]: Starting Cleanup of Temporary Directories...
Oct  7 16:19:31 raspberrypi systemd[1]: Started Cleanup of Temporary Directories.
Oct  7 17:17:01 raspberrypi CRON[30021]: (root) CMD (   cd / && run-parts --report /etc/cron.hourly)
Oct  7 18:17:01 raspberrypi CRON[31339]: (root) CMD (   cd / && run-parts --report /etc/cron.hourly)
Oct  7 19:02:39 raspberrypi systemd[1]: Started Session c2 of user pi.
Oct  7 19:04:52 raspberrypi dbus-daemon[796]: Activating service name='com.teamviewer.TeamViewer.Desktop'
Oct  7 19:04:52 raspberrypi dbus-daemon[796]: Successfully activated service 'com.teamviewer.TeamViewer.Desktop'
Oct  7 19:06:24 raspberrypi dbus-daemon[796]: Activating service name='com.teamviewer.TeamViewer.Desktop'
Oct  7 19:06:24 raspberrypi dbus-daemon[796]: Successfully activated service 'com.teamviewer.TeamViewer.Desktop'
Oct  7 19:10:56 raspberrypi systemd[1]: Started Session c3 of user pi.
Oct  7 19:10:59 raspberrypi systemd[1]: Stopping LSB: Start daemon at boot time...
Oct  7 19:10:59 raspberrypi homebridge[32734]: Not running
Oct  7 19:10:59 raspberrypi systemd[1]: Stopped LSB: Start daemon at boot time.
Oct  7 19:10:59 raspberrypi systemd[1]: Starting LSB: Start daemon at boot time...
Oct  7 19:10:59 raspberrypi homebridge[32742]: Starting homebridge
Oct  7 19:10:59 raspberrypi systemd[1]: Started LSB: Start daemon at boot time.

Thx for help
Stefan

Today I had the Problem again.

homebridge.log
Here is also the hombridge.log

Hi,
Could you fix it? I have the same problem.

      Hi,

Could you fix it? I have the same problem.

No, I don`t know what the problem is.
I'm not an expert. That is the reason why I asking for help.
Do you find somthing out?

Can nobody help us?

@vplustute can you share which router are you using right now? Make sure the router you use does not filter the mDNS packets.

So, what can I do next?

At my Fritzbox Setting I found something like DNS-Rebind-Schutz. I don't know what it is but can I fix here something?
Shout I try to use the pi-hole to fix the Problem? What does pi-hole do?

At my Fritzbox Setting I found something like DNS-Rebind-Schutz. I don't know what it is but can I fix here something?
https://avm.de/service/fritzbox/fritzbox-7390/wissensdatenbank/publication/show/663_DNS-Aufloesung-privater-IP-Adressen-nicht-moeglich/
https://en.wikipedia.org/wiki/DNS_rebinding

Shout I try to use the pi-hole to fix the Problem? What does pi-hole do?
Pi-Hole filters DNS request certain ad-driven Websites, so ads can't be displayed if you are surfing www.spiegel.de . If you haven't installed it yet, I think it wouldn't fix your problem.

But the DNS-Rebind-Schutz could be a possible solution for my homebridge+pi-hole problem. Maybe it should allow .local or something like that https://jonathanmumm.com/tech-it/mdns-bonjour-bible-common-service-strings-for-various-vendors/

Sorry I misunderstood, you install pi-hole.
I'm no expert so I have to read all the things what you gave me and have to check out what I have to type in, at the empty box for DNS-Rebind-Schutz

Did you solve your problem? I have similar problem, but not after some days it is more hours. The plain does not react. Seems other plugin still work. :-( Rebooting heimbringe and the plugin react again. Do You have any idea?

I have absolutely no idea. I think somehow there is still no solution to fix my problem.
I like the homebridge but at this way I can't use it. At the moment I disabled the raspi. I will  set up a new homebridge at the raspi where my loxberry is running. Maybe a new Installation will fix the problem

For me I solved my problem. I have pi-hole installed von my raspberry pi. In my faulty configuration setup I entered the pi-hole IP in my FritzBox router. So EVERY DNS request was forwared to the pi-hole. And I am afraid also raspi DNS requests where also forwared to the pi-hole, so to itself.
So I changed it at follows: My router has again normal internet DNS Server configured. On my clients, which I use for surfing, I manually added the address of the pi-hole. Now everything works fine
PS: I also added a command on the raspy on the startup to disable power management for the wifi


https://developers.google.com/web-components not accessible

This link redirects to 404.
When I tried to click on learn more inside web components section on  https://developers.google.com/web-components on this page.

Hi, the link should be https://developers.google.com/web/fundamentals/web-components, On which page did you click the link?

Hey,
In this section https://developers.google.com/web/fundamentals#whats-hot If you click on learn more inside Web Components, you will be redirected to https://developers.google.com/web-components which eventually throws 404.

🎉  Fixed & has been pushed live to https://developers.google.com/web/


Data mismatch when running full link testing

When sweeping full eth links with direct send test, seeing a mismatch non-deterministically when sending to a PCI device. It looks like the test's clearing of result buffer is happening after test is finished.
Repro (most reproducible on dirtbox 1):

git checkout aliu/load-base-erisc
TT_METAL_SLOW_DISPATCH_MODE=1 ./build/test/tt_metal/unit_tests --gtest_filter="DeviceFixture.EthKernelsDirectSendAllConnectedChips"

completed


Support sync flatten for an array of generic lazy futures

This adds support for syncFlatten on an array of generic lazy futures.
Issue #82 on vapor-community/async has additional details and discussion around implementation options.
I've submitted a related PR to https://github.com/vapor-community/async/pull/83. As mentioned on that PR, it looks like there is a transition under way between core, async, and nio-kit repos for these future helpers.
I'd like to target Vapor 3 with this change. Any feedback on code / where this PR should be open is appreciated.
Cheers,
Will

Out of the listed repos, vapor-community/async is actually dead and obsolete. I didn't even notice the issue and our discussion was there. This is the correct repo to target 3.x.

Hey @wlisac, you just merged a pull request, have a coin!
You now have 1 coins.


Backport #12140 to chef-17

Backport #12140 to chef-17

@jasonwbarnett  can you do one more rebase and push it again? I corrected the problem with the Mac builds. I'll merge it after that.

@johnmccrae The last commit on the chef-17 branch was nine days ago. There is nothing to rebase. In other words, this branch is fully rebased off of chef-17.


Adding prefix to key attribute. Fixes #148

The prefix is the mounted column name, so that users can upload multiple keys/files using the same model.

@Javierchik Could you rebase against current master, please?

@Javierchik Could you also add a deprecation warning when calling the old key and key= methods?

@fabn @p8 : guys here the changes.

@Javierchik Thanks!


Update HPO demo with plot API & importance

Update HPO demo with plot API & importance

Demo is now updated with Plot APIs & importance


Dragon's Crown [PCSE00019]

App summary

App name: Dragon's Crown
App serial: PCSE-00019]
App version: v1.00

Vita3K summary

Version: v0.1.6
Build number: 2968-0df79bb8
Commit hash: https://github.com/Vita3K/Vita3K/commit/7c8db9eab686dbb34c3582266f102a6a660d5258
CPU backend:  Dynamic
GPU backend: OpenGL

Test environment summary

Tested by: CoffeeBrewer64
OS: Kali Linux 2022.4 Debian 11 Bullseye-based. Kernel version 7.1 (I think)
CPU: Intel
GPU: Intel Mesa iGPU
RAM: 8gb
Device: Lenovo Thinkpad T450, with some changes

Issues
The audio is often slow or choppy.
Screenshots

Log
vita3k.log
Recommended labels

Playable
Audio bug

Duplicate of #401


Double "Consider:"

Seems to be an error here ("Consider: Consider:"):
https://github.com/rzach/forallx-yyc/blob/40ecc8a6e1030fc068a4ba9866773c4b0e8bf970/forallx-yyc-tfl.tex#L234

Right you are; thanks!


Add waiting for results method

Add something like this to the client:
task_id = task.get("id")
task = requests.get(
  f"{url}{api_path}/task/{task_id}",
  headers=headers
)
while not task.get("complete"):
	task = requests.get(
      f"{url}{api_path}/task/{task_id}",
  	  headers=headers
    )
  sleep(1)

Maybe we can also add support for the WS channel rather than the polling mechanism.
Also lets add a cool progressbar rather than the endless wainting for results print.
import itertools
import threading
import time
import sys

animation =  itertools.cycle(['|', '/', '-', '\\'])
t = time.time()
for i in range(10):
    time.sleep(1)
    frame = next(animation)
    sys.stdout.write('\rWaiting (' + frame + f') {int(time.time()-t)}s')
    sys.stdout.flush()
sys.stdout.write('\rDone!                  ')

Related: PR #41


After auth0 login, setRoot doesn't seem to call this.content.resize() (?) so top and bottom margins are never set

Ionic version:  (check one with "x")
[ ] 1.x
[x ] 2.x
[ ] 3.x
I'm submitting a ...  (check one with "x")
[x ] bug report
[ ] feature request
[ ] support request => Please do not submit support requests here, use one of these channels: https://forum.ionicframework.com/ or http://ionicworldwide.herokuapp.com/
Current behavior:

I set the rootPage to my login page (auth0)
Once the login is successful, I change the rootPage to my TabsPage (this.navCtrl.setRoot(TabsPage,{});)
It seems like this.content.resize() doesn't run and my scroll-content doesn't have the top and bottom margins so it is partially hidden by the header and tab footer.

However, once I click a tab and then return to the home page tab (default view of TabsPage), the margins appear.
Expected behavior:

I set the rootPage to my login page (auth0)
Once the login is successful, I change the rootPage to my TabsPage (this.navCtrl.setRoot(TabsPage,{});)
Default tab page appears with correct top and bottom margins

Related code:
app.component.ts:
...
export class MyApp {

  // the root nav is a child of the root app component
  // @ViewChild(Nav) gets a reference to the app's root nav
  @ViewChild(Nav) nav: Nav;

  rootPage = LoginPage;
...

login.ts:
...
events.subscribe('user:authenticated', () => {
        this.navCtrl.setRoot(TabsPage,{});
...

Default home page on initial load:

...

Default home page after clicking a different tab then returning to default home page:

...

Ionic info: (run ionic info from a terminal/cmd prompt and paste output below):
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0
Ionic CLI Version: 2.2.1
Ionic App Lib Version: 2.2.0
Ionic App Scripts Version: 1.0.0
ios-deploy version: Not installed
ios-sim version: Not installed
OS: Windows 10
Node Version: v6.9.4
Xcode version: Not installed

Hello, thanks for opening an issue with us. Could you post a minimal repo we could use to reproduce this issue?

Hello! As it seems it has been a while since there was any activity on this issue i will be closing it for now. Feel free to comment if you are still running into this issue. Thanks for using Ionic!


feat: register @vocab context definition

What this PR changes/adds
Registers @vocab context definition (with the https://w3id.org/edc/v0.0.1/ns/ value) at the global scope
Why it does that
To have consistent output jsonld responses
Further notes

this way, every json-ld produced by the edc won't have the "edc:" prefix in the edc namespaced properties
as this could be considered as a breaking change (but technically it is not because the compacted json-ld responses will expand in the same way as the old ones) I added a edc.jsonld.vocab.disable that could be used to switch to the old behavior.

Linked Issue(s)
Closes #3525
Closes #3522
Please be sure to take a look at the contributing guidelines and our etiquette for pull requests.

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

Comparison is base (2bfcdd7) 72.21% compared to head (b5452de) 72.21%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3526      +/-   ##
==========================================
- Coverage   72.21%   72.21%   -0.01%
==========================================
  Files         853      853
  Lines       17162    17164       +2
  Branches      965      966       +1
==========================================
+ Hits        12394    12395       +1
  Misses       4359     4359
- Partials      409      410       +1

Files
Coverage Δ

.../java/org/eclipse/edc/api/model/ApiCoreSchema.java
0.00% <ø> (ø)

.../management/configuration/ManagementApiSchema.java
0.00% <ø> (ø)

...dc/connector/api/management/asset/v3/AssetApi.java
0.00% <ø> (ø)

...c/connector/api/management/catalog/CatalogApi.java
0.00% <ø> (ø)

...ment/contractdefinition/ContractDefinitionApi.java
0.00% <ø> (ø)

...nt/contractnegotiation/ContractNegotiationApi.java
0.00% <ø> (ø)

...tor/api/management/policy/PolicyDefinitionApi.java
0.00% <ø> (ø)

...management/transferprocess/TransferProcessApi.java
0.00% <ø> (ø)

...lector/api/v2/schemas/DataPlaneInstanceSchema.java
0.00% <ø> (ø)

...elector/api/v2/schemas/SelectionRequestSchema.java
0.00% <ø> (ø)

... and 1 more

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

This is about adding the @vocab on "egress" (i.e. compaction), right? because on ingress (i.e. on expansion) we already have that...

@paullatzelsperger yes, in the compaction side of things


How to verify if checks are executed on host machine and not docker container?

I would like to verify/test that all checks are running on host machine and not within the docker container where Sensu client is running. Could you please help out with some hints? My docker-compose file looks like this:
version: '2'
services:
  client:
    build: .
    command: client
    container_name: docker-sensu-client-prod
    image: adop/sensu-client
    environment:
      CLIENT_NAME: aca-jira01
      CLIENT_SUBSCRIPTIONS: basic
      CLIENT_ADDRESS: <IP_ADDRESS>
      REDIS_HOST: <IP_ADDRESS>
      REDIS_PORT: 6379
      REDIS_AUTO_RECONNECT: "true"
      REDIS_RECONNECT_ON_ERROR: "true"
      HOST_DEV_DIR: /host_dev
      HOST_PROC_DIR: /host_proc
      HOST_SYS_DIR: /host_sys
    ports:
      - '3030:3030'
    volumes:
      - /dev:/host_dev/:ro
      - /proc:/host_proc/:ro
      - /sys:/host_sys/:ro

The configuration looks correct.  Setting HOST_ and mounting the values into the container should cause the checks to execute against the hosts /dev, /proc, /sys.  The checks are still running inside of the container, but should be using data from the host.

Thanks @sstarcher! But I still want to verify that it is the host Im checking, and not the docker container it self. Any tip on how to do it?

@ismarslomic run a check in the container for cpu and cause an increase CPU load on the server?  Verify it alerts.


Added TARGET_PARAMETER for PHP8 Attributes

As discussed in the RFC of constructor property promotion this would allow the #[Field] attribute to be used in function parameters but would automatically be picked up by graphqlite because it would find a Field Attribute with ReflectionProperty.
Would resolve #354

@Lappihuan Feel free to commit to the branch you have, allowing Github to run the tests here in this PR.

i'd need to check this again, but from what i found recently.
This should be supported out of the box.
If thats the case we can close this PR

@Lappihuan I haven't tried defining the attribute on a constructor parameter/property yet.  I'm not entirely sure if that's considered a property with property promotion or if it's still considered a parameter.  At any rate, we need tests before this can be merged.


OCSORT in other Language

May I ask if there is any plan for the author to write a C++ implementation of OCSORT code?

so I implemented the OC-SORT w/BYTE in C++, checkout out this

Thanks for the contribution!

code merged. See deployment readme


WSI and bbox json

Hi,
I am trying to run hovernet_lite on a WSI, do you have an example json file for the bbox?
Thank you

Hi,
I am trying to run hovernet_lite on a WSI, do you have an example json file for the bbox?
Thank you

Hi, apologies due to the delayed responses. Here goes a sample JSON file.
Each JSON file corresponds to the tile location of one WSI, in the format of List[List[Tuple[left, top, right, bottom]]]. Tuple[left, top, right, bottom] is the 4-tuple coordinate of a single bounding box (bbox).  Tile bbox within the same connected tissue regions are grouped into a list, and multiple connected tissue regions are further grouped into another list. Therefore here goes a nested list of 4-tuple coordinates.
Note that the coordinates are half-open: open at right and bottom. Meaning the actual pixel interval are within [left, right) and [top, bottom), consistent with the convention of other packages like PIL and openslide.
You may feel free to either modify the corresponding code here if you have another format (https://github.com/CielAl/hovernet_lite/blob/main/hovernet_lite/infer/wsi_bbox.py#L71) or you may use HistoQC's TileExtractionModule (https://github.com/choosehappy/HistoQC) to obtain such bounding boxes.
sample.svs_bbox.zip


[BUG] ODBC driver doesn't use SSL

Describe the bug
ODBC driver doesn't use SSL.
To Reproduce
Steps to reproduce the behavior:

Launch an OS cluster, enable SSL
Install the driver
Test connection

Expected behavior
The driver should be able to use SSL according to the user settings. Option to trust self-signed certificates should be also available.
Version tested: 1.2.0

See also #330. #330 refers to incorrect SSL usage, this one - to ignored option to trust self-signed certs.


cluster_name field not added to journald and file logs

What happened:
In splunk, the cluster_name metadata field is not being set for non container logs.
When I search for kube:docker logs for example, if i add a cluster_name field to the search, logs don't show up.
Sample search:
index="myindex" sourcetype="kube:docker" cluster_name="mycluster"
This does not return results ^
What you expected to happen:
I would like to to see all logs from a cluster irrespective of source when just search for cluster_name and index.
How to reproduce it (as minimally and precisely as possible):
Just deploy splunk-connect and search for cluster_name field for logs with sourcetype="kube:docker"
Environment:

Kubernetes version (use kubectl version):
clientVersion:
buildDate: "2019-03-26T00:05:06Z"
compiler: gc
gitCommit: 641856db18352033a0d96dbc99153fa3b27298e5
gitTreeState: clean
gitVersion: v1.14.0
goVersion: go1.12.1
major: "1"
minor: "14"
platform: darwin/amd64
serverVersion:
buildDate: "2019-03-25T06:30:48Z"
compiler: gc
gitCommit: 16236ce91790d4c75b79f6ce96841db1c843e7d2
gitTreeState: clean
gitVersion: v1.11.9
goVersion: go1.10.8
major: "1"
minor: "11"
platform: linux/amd64

Ruby version (use ruby --version):
Ruby version (use ruby --version):
ruby 2.3.7p456 (2018-03-28 revision 63024) [universal.x86_64-darwin17]

OS (e.g: cat /etc/os-release):
macOS High Sierra
10.13.6 (17G6030)

Splunk version:
7.2.6
Build:
c0bf0f679ce9

I’ve managed to solve this problem by putting the cluster level fields in a separate filter on top of the existing default ones. Try using a <filter **>your fields.

I believe this is fixed with https://github.com/splunk/splunk-connect-for-kubernetes/pull/195
Closing this issue.


Reading and writing the response.

I am trying to provide a json file from project directory as the body.
How can i do that? Documentation does not address it.

@verodhi Thank you for the questions.
Most of the Azure DevOps Web APIs return JSON.
For instance, calling:
GET https://feeds.dev.azure.com/{organization}/{project}/_apis/packaging/Feeds/{feedId}/packages?api-version=5.1-preview.1

...returns a JSON set of your project's packages.
What exact information do you need to send as JSON?

@msebolt
My apologies, I think I was not clear:
I am trying to execute an agentless task in DevOps; Namely: Invoke REST API Task
It allows passing body as parameter i.e:
- task: InvokeRESTAPI@1   inputs:     connectionType: 'connectedServiceName'     serviceConnection: 'exampleconnection'     method: 'POST'     headers: |       {       "Content-Type":"application/json"       }     body: '{"json":"body"}'     waitForCompletion: 'false'
In the body field above I want to pass a json that is sitting in my project root directory as a file, i.e. appsettings.json
I was wondering how can I do that.
Thank you

@verodhi Apologies for the delay.
This would require a custom solution based on either using the Azure DevOps API or something from the Marketplace:

file to variable
json to variable

You could then use the InvokeRESTAPI task with the loaded file variable.

Closing now, please comment below if there are further issues.


smol fix

What kind of change does this PR introduce?
fix, you should also probably update your doc to use "records" but also, you can just change to use "vectors"
https://supabase.com/docs/guides/ai/vecs-python-client
What is the current behavior?
ERROR

What is the new behavior?
NO ERROR
Additional context
Add any other context or screenshots.

thanks!


fix(event_sources): add test for Function URL AuthZ

Issue number: #1416
Summary
Changes

Please provide a summary of what's being changed

Function URL latest introduction is missing a test without authorization fields. This PR ensures code paths work when a function URL is configured as NONE for AuthType.
Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] Meet tenets criteria
[x] I have performed a self-review of this change
[x] Changes have been tested
[ ] Changes are documented
[x] PR title follows conventional commit semantics

Is this a breaking change?
RFC issue number:
Checklist:

[ ] Migration process documented
[ ] Implement warnings (if it can live side by side)

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

thank you so much @rubenfonseca! merging


Containers_running_as_root query faild to evaluate

The query Containers_runs_as_root failed to produce an output

for this Yaml file

---
apiVersion: v1
kind: Pod
metadata:
  name: security-context-demo-2
spec:
  securityContext:
    runAsUser: 10
    runAsNonRoot: false
  containers:
  - name: sec-ctx-demo-100
    image: gcr.io/google-samples/node-hello:1.0
    securityContext:
      runAsUser: 0
      runAsNonRoot: false
  - name: sec-ctx-demo-200
    image: gcr.io/google-samples/node-hedwfwllo:1.0
    securityContext:
      runAsUser: 0
      runAsNonRoot: false

Specifications

Version: 1.2.3
Platform: window

Hi, @mukeshpilaniya.
Thank you for identifying this problem.
The bug was reproduced and it affects all versions, including the most recent one (1.3.1)
We will check it as soon as possible.
Below a small technical report on the data collected from the problem:
Query: Container Running As Root
ID: cf34805e-3872-4c08-bf92-6ff7bb0cfadb
Error: failed to evaluate query: containers_running_as_root:100 :
eval_conflict_error: functions must not produce
multiple outputs for same inputs
Query: Container Running With Low UID
ID: 02323c00-cdc3-4fdc-a310-4f2b3e7a1660
Error: failed to evaluate query: containers_run_with_low_uid:104:
eval_conflict_error: functions must not produce
multiple outputs for same inputs
A possible solution to the problem can be seen at: https://www.openpolicyagent.org/docs/latest/policy-language/
in the section: eval_conflict_error: functions must not produce multiple outputs for the same inputs
Note: The error does not happen if there is only one container in the test sample.
apiVersion: v1
kind: Pod
metadata:
  name: security-context-demo-2
spec:
  securityContext:
    runAsUser: 10
    runAsNonRoot: false
  containers:
  - name: sec-ctx-demo-100
    image: gcr.io/google-samples/node-hello:1.0
    securityContext:
      runAsUser: 0
      runAsNonRoot: false


Add isAllocator method for ids

cc @mclarsen

LGTM

LGTM

LGTM

Overall, this looks great. I only had a question about whether additional release notes should be considered for the next release cycle.

I updated the CHANGELOG so it notes that change, and we should put it in the release notes too


QuickStart: Why publish all RethinkDB ports?

It's nice that http://shipyard-project.com/docs/quickstart/ warns me that following the guide leaves all my rethinkDB ports publicly accessible. But why does the quickstart or Shipyard Deploy leave these ports publicly accessible at all? Wouldn't it be much safer to run e.g.:
docker run -it -d --name shipyard-rethinkdb --volumes-from shipyard-rethinkdb-data -p <IP_ADDRESS>:49153:8080 -p <IP_ADDRESS>:49154:28015 -p <IP_ADDRESS>:29015:29015 shipyard/rethinkdb

so that Shipyard can locally access RethinkDB, but public access is restricted?

This only works if shipyard and rethinkdb are colocated.  This breaks if
you want to add a node to the cluster or another shipyard instance for HA.
On Sat, Mar 7, 2015 at 10:29 AM, Romke van der Meulen <
<EMAIL_ADDRESS>wrote:

It's nice that http://shipyard-project.com/docs/quickstart/ warns me that
following the guide leaves all my rethinkDB ports publicly accessible. But
why does the quickstart or Shipyard Deploy leave these ports publicly
accessible at all? Wouldn't it be much safer to run e.g.:
docker run -it -d --name shipyard-rethinkdb --volumes-from shipyard-rethinkdb-data -p <IP_ADDRESS>:49153:8080 -p <IP_ADDRESS>:49154:28015 -p <IP_ADDRESS>:29015:29015 shipyard/rethinkdb
so that Shipyard can locally access RethinkDB, but public access is
restricted?
—
Reply to this email directly or view it on GitHub
https://github.com/shipyard/shipyard/issues/431.

True. I wonder though if deploying Shipyard on a cluster is a typical use-case, especially for somebody reading the quickstart. Wouldn't it make more sense to explain the simple, secure setup in the quickstart, and describe how to deploy a secure, distributed Shipyard network elsewhere?
I'm simply worried that even with this warning attached, a lot of people are going to leave unsecured, open access to their RethinkDB instance running on production servers. And if the DB data leaks, anybody can read your entire Docker setup. That would be a huge security risk, wouldn't it?

That is a valid point.  I think setting this to localhost by default would be good.  Since we use the data volume you could always start a new RethinkDB instance with the ports exposed to grow.  I will do some testing to make sure it doesn't break anything.  Thanks for the feedback!

No problem!

I'm going to re-open so I can remember to update this :)  Thanks!

Yes please! :+1:

Actually, since the containers are linked, what is the point of exposing the RethinkDB ports at all? Maybe the Quickstart should just contain:
docker run -it -d --name shipyard-rethinkdb --volumes-from shipyard-rethinkdb-data shipyard/rethinkdb

And add a note like "If you want to expose the RethinkDB ports locally, you can expose them by adding the -p [...] parameters".

Yeah that's probably a good idea.  Since we use the data volume, I don't
think there is any reason to expose it.  Thanks!
On Wed, Mar 25, 2015 at 7:58 AM, Jan Keromnes<EMAIL_ADDRESS>wrote:

Actually, since the containers are linked, what is the point of exposing
the RethinkDB ports at all? Maybe the Quickstart should just contain:
docker run -it -d --name shipyard-rethinkdb --volumes-from shipyard-rethinkdb-data shipyard/rethinkdb
And add a note like "If you want to expose the RethinkDB ports locally,
you can expose them by adding the -p [...] parameters".
—
Reply to this email directly or view it on GitHub
https://github.com/shipyard/shipyard/issues/431#issuecomment-86068626.

@ehazlett I've updated shipyard/shipyard, shipyard/shipyard-project.com and shipyard/shipyard-deploy

There is a new deployment method (https://shipyard-project.com/docs/deploy) that no longer exposes rethinkdb.  Thanks!


Make cursor paramstyle configurable

Currently connection.cursor() does not seem to allow specifying the paramstyle for the returned cursor – it would be great if this functionality could be exposed. I assume that setting cursor.paramstyle to the desired value should work as expected, but this does not seem to be documented at the moment.

Hey @GergelyKalmar ,
The team is currently in the process of improving documentation for redshift_connector, I'll forward along your feedback so we can improve transparency of paramstyle.
In the meantime, I've quickly jotted down some high-level examples of each of the paramstyle types.

Valid values for paramstyle are qmark, numeric, named, format, and pyformat
The default value of paramstyle is format
Setting cursor.paramstyle to one of the valid values mentioned above will allow you to use it as expected.

Examples of each paramstyle
# qmark
redshift_connector.paramstyle = 'qmark'
sql = 'insert into foo(bar, jar) VALUES(?, ?)'
cursor.execute(sql, (1, "hello world"))

# numeric
redshift_connector.paramstyle = 'numeric'
sql = 'insert into foo(bar, jar) VALUES(:1, :2)'
cursor.execute(sql, (1, "hello world"))

# named
redshift_connector.paramstyle = 'numeric'
sql = 'insert into foo(bar, jar) VALUES(:p1, :p2)'
cursor.execute(sql, p1=1, p2="hello world")

# format
redshift_connector.paramstyle = 'format'
sql = 'insert into foo(bar, jar) VALUES(%s, %s)'
cursor.execute(sql, (1, "hello world"))

# pyformat
redshift_connector.paramstyle = 'pyformat'
sql = 'insert into foo(bar, jar) VALUES(%(bar)s, %(jar)s)'
cursor.execute(sql, {"bar": 1, "jar": "hello world"})

Wonderful, thank you very much!


Replace hashie mash with hash

Building on comments in #1514 #1279. Defaults to HashWithIndifferentAccess as @request.params needs to be indifferent for the validation code which mutates it.
It largely achieves the API suggested in #1514 but at the expense of Law of Demeter, I'm open to suggestions. Also happy to make Hash the default params object, but didn't do so as that requires an additional transformation of the params.

First, thanks. I closed https://github.com/ruby-grape/grape/pull/1594 in favor of this PR.
Do you think we can reduce the internals of Grape to use a plain Hash and make that the default instead of the activesupport one?
I suspect there's a non-trivial performance penalty in making a Hashie::Mash on top of some other hash in transform_params. I think you have to make an interface where we construct the params object of the right type.
Does Grape::API.extend Grape::Extensions::Hashie::Mash work globally?
I think this is close if we can address the above. Cleanup please, fix the build, document this in README and UPGRADING.

@dblock I think we can initiate with either a Mash or AR::HashWithIndifferentAccess in Grape::Request by wrapping it with a shim to tell it what class to use to create the params with;
      module Mash
        def new_request(env)
           Grape::Request.new(env, params_class: Hashie::Mash)
        end
      end

That would mean the interface doesn't change and would work globally.
I started out with the plain hash which breaks some endpoint specs that test for indifferent access, and therefor presents a breaking change to the Endpoint DSL I also had to enforce a consistent style of key from request, through validation, coercion and the DSL (hence the deep symbolizing module). For those reasons I reverted back to HashWithIndifferentAccess as the default as it's not a breaking change to existing apps and still allows the dropping of Hashie.
I've got a limited window for this, I can give the Hash from Request thru Response a shot, but would still suggest it's better as an option rather than default.

I think specs that rely on mash behavior should be rewritten in terms of Hash and possibly duplicated into a Mash version.
I feel pretty strongly not forcing people into another Hash-like class and letting them choose. Either way it should be configurable, so I would write the code in terms of Hash, then potentially switch the default to the activesupport or mash version.

Ok, those specs I can move over to Hash behaviour, I can make Hash the default, and update the README. The one cost is going to be in building the Hash from params as they come in with string keys and I'm pretty sure the entire stack uses symbols so it's going to require the keys symboliziing. I don't see any way round that. The deep symbolizer I've included is pretty fast, but there is that cost.

Thanks for keeping at it! You should squash these commits. I can look once the build is green.

@dblock The default is now Hash. Hashie and HashWithIndifferentAccess are both options. I ended up adding a build_with option to Parameters.rb as it was much cleaner than extending Grape::API. The commits are all squashed and it should make a lot more sense.
I don't know what's up with Danger failing the build but the tests are all passing. I've gone as far as I can with this and am out of time.

Danger is complaining about the missing period at the end of your changelog line.

We still need to write clear UPGRADING for this and possibly README changes. Thanks for your help @james2m. It's a sensitive change that is going to affect a lot of people, so I am going to wait till myself or someone else is committed to seeing it through and at least somewhat actively supporting users who're getting problems with it during the next release.

For the sake of reducing this PR we probably want to rubocop -a gemfiles as well after appraisals were generated.

@dblock I can do the appraisals, I couldn't get Danger to run locally or verbosely so had no feedback from it so thanks for that.
I avoided writing the UPGRADING and README in case there were further implementation changes as I figured it's going to need someone to kick the tyres. My 'out of time' doesn't mean now way never, just got to get on with the project this is going into.

I personally vote for this implementation, but I may not be grasping the whole scope. We have to finish the code and the README/UPGRADING, tell the grape mailing list to try this out and ask for comments.

If it makes sense, now that everything is Hash-based, and to avoid massive downstream regressions we should probably default the user-facing behavior to the active support version (or any a hash with invariant and method access).
I'm then looking for a 1-liner in README that makes this a plain Hash or a Hashie::Mash (after adding hashie to Gemfile).

I'd definitely be inclined to go the ActiveSupport route for it's first outing as that's going to be the least pain for existing users. It'll take me a few days to get the extra tests in and docs updated. I'll chip away at it next week.

@dblock Before I make the final changes. We are defaulting to HashWithIndifferentAccess? If so I'll update the README and UPGRADING to reflect. Then the PR is done.

Yes, I think that's right. Also cc: @namusyaka

You should squash these commits so we can look at a nice single change. I can do it when merging, but lets make this PR discoverable and easy to read for anyone who wants to look at what changed for something so important.

I'll squash again when we're done with review comments.

I was late to the party. I agree that HashWithIndifferentAccess is default.
@james2m Thank you so much for your great work!

Replacing this with https://github.com/ruby-grape/grape/pull/1610, which includes a way to use ParamBuilder globally for an entire API or a namespace. And a bunch of README, UPGRADING, spec and require cleanup.
Thanks @james2m, lets finish this!


image11

Suggesting to replace the existing tutorial image 11 with the suggested image

Hi @Swathi266 thanks for your PRs! If you could use the same filename for the image as was used in the tutorial previously, it will automatically be updated in the tutorial. (but let me know if you would like some help doing that)

I am not sure how and where to change the file name

@Swathi266 ok no worries, I can take care of it! :+1:

Ok. Thank you @shiltemann

@Swathi266 I have updated the image name to replace the old one. Do you know why the numbers are so different now? Perhaps @AnnaSyme can have a look to see if it makes sense to update this image?

@shiltemann Maybe its because it represents a unique identifier for each specific change?

thanks @shiltemann @Swathi266 I will take a look

could you tell me what tutorial this image is in? thanks! @shiltemann

@AnnaSyme Image after running velvet in the tutorial - https://training.galaxyproject.org/training-material/topics/assembly/tutorials/general-introduction/tutorial.html
Stats file

thanks @Swathi266 @shiltemann I think it's ok that the numbers are different, as it's just showing the top of the file, and overall the assembly is similar (or the same). If you wish to replace this image could I suggest also replacing the preceding image of the contigs file to match? as the top three contigs in that file are then represented as the top three lines in the stats file. Thanks!

@AnnaSyme Yes please.
https://github.com/galaxyproject/training-material/pull/5439

Thanks @Swathi266! And thanks for the review @AnnaSyme! :+1:

Thank you @shiltemann @AnnaSyme


Food Search - Directions to the food

The map feature is great!  However, it would be great to be able to calculate the best way to get there either by walking, public transit, or car.  Clicking the pin points on the map shows the image used to market the food establishment, this could be used to open a mapping function for users to figure out how to visit the business.

If a listing has an address you can click on the address for directions.
Some listings are mobile/food trucks and do not have a stable address and cannot be navigated to. Their location changes and/or they go to you.


How to specify picker theme?

Hi! Great work with the plugin.
I want to open my snippets in ivy themed picker. How do I go about doing that? I have tried the following:
				require("telescope").setup({
					theme = "ivy",
					pickers = {
						frecency = {
							theme = "ivy",
						},
						find_files = {
							theme = "ivy",
						},
						live_grep = {
							theme = "ivy",
						},
						LuaSnip = {
							theme = "ivy",
						},
					},

yet it opens up in the default, non-ivy theme.

You can always do:
require('telescope').extensions.luasnip.luasnip(require('telescope.themes').get_ivy {})

It doesn't seem to be reading from pickers' configuration.


can not pull image from china: docker pull k8s.gcr.io/conformance:v1.22.1

Hi teams:
I am trying to use sonobuoy to test the k8s cluster built with kubeadm.
During the test startup process, I got stuck in the image pull.
I tried to pull the mirror separately and reported the following error.
Is there a good way to download this mirror image in China?
code:
# ./sonobuoy images
INFO[0000] e2e image to be used: k8s.gcr.io/conformance:v1.22.1
INFO[0000] Pulling image: k8s.gcr.io/conformance:v1.22.1 ...
ERRO[0031] failed with following error after 1 retries:
ERRO[0031] Error response from daemon: Get "http://k8s.gcr.io/v2/": net/http: request canceled while waiting for connection (Client.Timeout exceeded while awaiting headers)

How can list all images that need for run test .
I try this :

set image conf

cat <<EOF > conformance-image-config.yaml

dockerLibraryRegistry: my.harb.com/private/k8s.gcr.io
EOF

run test

./sonobuoy run --mode=certified-conformance --sonobuoy-image my.harb.com/private/sonobuoy/sonobuoy:v0.55.1 --e2e-repo-config conformance-image-config.yaml

and find this in log

Back-off pulling image "k8s.gcr.io/e2e-test-images/busybox:1.29-1"

It seems that the mirror is not downloaded from a private source.
my question:

How to get a list of all the mirrors to be downloaded?

How to set sonobuoy to automatically use our private mirror for testing?

Hope someone can provide some help, I have spent 4 days on starting the test...

Sorry for the frustration; airgap (and airgap-like) e2e testing is a thorn in everyones side.
Most likely, that image is referencing a different named registry and not dockerLibraryRegistry.

Here is the current registry list you can fill out: https://github.com/kubernetes/kubernetes/blob/ea0764452222146c47ec826977f49d7001b0ea8c/test/utils/image/manifest.go#L32-L44
You can see that busybox used the promoterE2eRegistry, not the dockerLibraryRegistry
I havent used this yet but it looks like other folks got tired of the multiple-registry overwriting too and made this issue https://github.com/kubernetes/kubernetes/pull/93510 which seems like you can set --plugin.env=e2e.KUBE_TEST_REPO=<your registry> and it will just assume that registry for every image.
Hope these bits help. Feel free to reopen if you have more questions.


Upgraded expound

Codecov Report

Merging #302 (6e87e57) into develop (153e7f0) will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff             @@
##            develop       #302   +/-   ##
===========================================
  Coverage   100.000%   100.000%
===========================================
  Files            36         36
  Lines          3069       3069
===========================================
  Hits           3069       3069


More TypeScript. Missing types or wrong usage?

Description:
I've some more questions on typescript. Either there are some missing types/signatures or I may be using it wrong:
Ractive.transitions seems to be missing entirely in the types
Ractive.WELCOME_MESSAGE seems to be missing in types or is there another way to silence the welcome message`?
Types for EventPlugin state, that the fire callback takes one argument of type Event but Examples at https://ractive.js.org/plugins/#writing_4  it's used like this:  fire({ node: node, original: event }) which does not match the signature. Is the matching signature missing in the types or is it supposed to be used in another way?
I've used Ractive.defaults to add some global helpers to all ractive instances, e.g. to use them in the templates like @.helper()
This used to work back in good old JS days:
Ractive.defaults.helper = function () {
   // do stuff
};

But typescript says that Ractive.defaults does not accept custom properties. Is this missing in the TS types or are there new ways to add methods to all instances, primarily to use them in the templates.
Versions affected:
1.3.14

These are likely issues with the .d.ts file for ractive. I took a file from @typings, adjusted it, filled in a bunch of missing stuff, and added it to the package, but there are some use-cases I don't typically hit. We've had a number of typings contributions to fill in some of the gaps, and it looks like these just haven't been encountered yet. Looking at each:

Yep, transitions is just missing from the Static ractive definition. Ractive is a little harder to model in typescript.
WELCOME_MESSAGE looks like it was just overlooked too.
Event plugins did change a little, so I'll have to poke at the code a little to verify.
I haven't used shared defaults for helpers before, but if it does work that way, the defaults definition will just need to have an index property added.

I think I've got all of these resolved on edge once travis finishes doing its thing.


Is top of the tree supposed to work on IIe?

The analog/ top repo seems to pull 3ebdc40 as a firmware, but when I build that the unit doesn't work. If I revert the last last binary release, it works...
I'm not sure if it's my environment, or something else?
Also, where are you @dkgrizzly ? we miss you :-)

Hey sorry for the hiatus, Life & Work always get in the way.  I'll look into updating the top level project this afternoon.  I may look into refactoring the top-level project with a post-checkout procedure that updates all the sub projects to their latest versions.

Good to see you back David ;-) thanks for all the work !


cpb.swf doesn't have annotations or captions working, and can't even turn on annotations on late 2009-mid 2010 players (nothing happens when clicked)

For some reason, the playlists player (cpb.swf) doesn't really have the ability to use captions or annotations (at the moment at least...).
Like this:
https://github.com/user-attachments/assets/543a2fa5-f803-40c0-8d31-3df22abea2da
(http://<IP_ADDRESS>:80/alt-swf/cpb.swf?player_id=1Aetq1LjgtQ&datatype=playlist&data=PLIl2EzNYri0crmY28ZNs-bsgzGOms2nAn&BASE_YT_URL=http://<IP_ADDRESS>:80/)
Even with forced subtitle_module and iv_module, nothing seems to work.
And regarding that, for late2009-mid2010 players, the annotations button does nothing.:
https://github.com/user-attachments/assets/911a0f92-c905-4836-aed9-d6ce75218255
When will you fix it? Do you have something to make it work on cpb and fix the annotations button? Thank you.

you're the one mismatching all the random players and modules and expecting me to fix that. basically a duplicate of #122. wait for the next update that will add specifying custom as2 modules for watch.

you're the one mismatching all the random players and modules and expecting me to fix that. basically a duplicate of #122. wait for the next update that will add specifying custom as2 modules for watch.

I seriously apologize


fix: harmonize npm name usage in logs

This PR modifies the action source file index.js for consistency with

the npm documentation About npm which uses lower case npm and with
the Cypress documentation Module API which does not include NPM in its name

Changes

Text and comments now refer to npm consistently in lower case.
NPM is removed from the reference to the Cypress Module API

This PR is left is draft state pending merge of PR https://github.com/cypress-io/github-action/pull/887

It is the last PR in a series concerned with adding pnpm and Yarn to the list of supported package manager lockfiles supported for installation. This small change triggered a whole lot of other changes making sure that npm was consistently referred to.
When this PR is merged it will trigger an npm release and that will also update the README on https://www.npmjs.com/package/@cypress/github-action, so before that happens all the other documentation changes should have completed.
For some strange reason the README on https://github.com/marketplace/actions/cypress-io is updated even without a release, except that the released version is only updated with a release to the GitHub Marketplace!

Please review. Thanks.

@mschile
I need to rebase this first due to the Yarn pnp merge


The D435 camera takes photos intermittently.

The project requires a d435 camera to take occasional photos, and I couldn't find a similar project on the official website. I am borrowing from video related projects, but it seems that I cannot avoid timeout issues

Hi @12581djjd  One method of taking occasional photos would be to perform a sleep operation between captures so that the program does not do anything until the sleep period has elapsed (a number of seconds equal to the minutes that you want to wait before the next capture).
If you are having difficulties with timeouts then another method that has been used with other RealSense projects that take occasional captures is to have the camera wait for a trigger pulse signal to be received before initiating capture.
An example of this is a project at https://github.com/IntelRealSense/realsense-ros/issues/1354 involving plants on a conveyor belt that took a capture every 15 minutes.
Another conveyor-based approach described at https://github.com/IntelRealSense/librealsense/issues/2179 is to have capture initiated when an object passes through a photoelectric switch.

Thank you very much for your help，

Hi @12581djjd  Do you require further assistance with this case, please?  Thanks!

Case closed due to no further comments received.


abapgit export object:bopf failed

expert,
i am using abapgit to export package, error occurred when object type bopf and wda.
please help and let me know how to install plugins:bopf and wda.
ps: online mode is not possible, only offline.

thanks larshp,
i installed the plugins your provided and installed all objects mentioned in https://github.com/larshp/abapGit.
now it is working fine.


Uploads report UI success when failure occurs

Link to reproduction
No response
Payload Version
3.0.0-beta.47
Node Version
v20.12.2
Next.js Version
15.0.0-rc.0
Describe the Bug
When experiencing the behavior in #6811, the UI reports upload success although the upload failed.
Reproduction Steps

Reproduce steps in #6811
See the UI success message, but see the S3 logs report failure.

Adapters and Plugins
@payloadcms/storage-s3

Hey @aus - going to close this issue as it's a duplicate of this discussion here.


Purge data in explorer based on a parameter

Signed-off-by: Abhay Kishore<EMAIL_ADDRESS>What this PR does / why we need it:
It will purge the old data present in explorer db based on the parameter blockCount.blockCount property can be defined in config.json.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation, usage docs, etc.:

Duplicate of https://github.com/hyperledger-labs/blockchain-explorer/pull/362


Implement KafkaSchemaTransformWriteConfiguration

This PR addresses #21414 with a KafkaSchemaTransformWriteConfiguration implementation.  It's design goals are to work with a KafkaSchemaTransformWriteProvider that extends a TypedSchemaTransformProvider.

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

[x] Choose reviewer(s) and mention them in a comment (R: @username).

[x] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.

~Update CHANGES.md with noteworthy changes.~

~If this contribution is large, please file an Apache Individual Contributor License Agreement.~

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

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

R: @pabloem

@pabloem the only check that failed was Python_PVR_Flink ¯\_(ツ)_/¯

sorry Damon that I had been slow with this. Just one comment:

Should users be able to specify whether they want to serialize their data into Avro / JSON / something else?
We will have a PCollection as input, right? - how will users decide which is the key for the messages produced?

Should users be able to specify whether they want to serialize their data into Avro / JSON / something else?

Would the getValueSerializer inform the SchemaTransform whether it is Avro / Json or something else?

We will have a PCollection as input, right? - how will users decide which is the key for the messages produced?

Would the getKeySerializer inform the SchemaTransform how to write the Key data?
If both are true, then I wonder if we need to remove the @Nullable annotation from getValueSerializer and getKeySerializer.


Sequence mismatch of map tokens in map2agent attention

Hi, in map2agent attention, you create batch_pl by repeat_interleave operation:
https://github.com/rainmaker22/SMART/blob/69c7a419d5004a762ffd3305ebda4e055bd7a0db/smart/modules/agent_decoder.py#L396-L405
but when conduct attention, you create pos_pl by repeat operation:
https://github.com/rainmaker22/SMART/blob/69c7a419d5004a762ffd3305ebda4e055bd7a0db/smart/modules/agent_decoder.py#L272-L277
which will cause mismatching when creating the edges index. Please let me know if I was wrong.
Thanks!

Sorry I was wrong ...


Account (and possible domain) configure limits reset value on click away

ISSUE TYPE

Bug Report

COMPONENT NAME

UI

CLOUDSTACK VERSION

4.17

CONFIGURATION

OS / ENVIRONMENT

SUMMARY

When editing the Configure limits any mouse click on any other field will reset the changed value of the previous field to what it was before.
STEPS TO REPRODUCE

Go to any Account and try to change values in Configure Limits tab.
Click away from the input box you changed

EXPECTED RESULTS

Change fields and save them by clicking Submit.

ACTUAL RESULTS

Field values are reset. You can only save one by one if you don't click outside the input box

Fixed on #6578


error G76A9B1F6: The method 'launchUrl' isn't defined for the class 'GoTrueClient'

We are Getting
error G76A9B1F6: The method 'launchUrl' isn't defined for the class 'GoTrueClient'
error when building our app
the error comes from:
supabase_auth.dart from line 170
Solotion:
We have fixed this error by:
Changing this line: 👇
final result = await launchUrl(url, webOnlyWindowName: '_self');
to: 👇
final result = await launch(url.toString(), webOnlyWindowName: '_self');
Please Fix the error by fixing this error on the repo so we can have clean builds on our server-side too

@iampopal Sorry to hear that you are experiencing this error. For a temporal fix, could you delete your pubspec.lock file and rerun flutter pub get?

did try this and still getting this error

will be great if you use latest version of url_launcher package

or remove the number of version from url_launcher so it will use url_launcher from the repo

The min version of url_launcher is now ^6.1.2, which is after ^6.1.0, where launchUrl got introduced. So all apps using this package will automatically resolve a version with launchUrl support.


[Bug]: Intermittent Timeout Error when Scraping an Array of URLs

Bug expectation
I created a web scraper that scrapes an array of URLs. The reliability of this script (before it throws a waitForXPath Timeout) is tied to the amount of links. For example, it can scrape 4 links very reliably, but the reliability decreases as the link quantity increases. It has scraped up to 11, but never more than that, before it starts reporting waitForXPath timeout errors. It seems to have nothing to do with the actual substance of the links, but just with the quantities of link.
Bug behavior

[X] Flaky
[ ] PDF

Minimal, reproducible example
const puppeteer = require('puppeteer');
const URLs = [

    'https://landa.app/m/property/Lakeview-Road-9439-Union-City-GA-30291/US_9439LV-UNC',
    'https://landa.app/m/property/Parliament-Place-8796-Jonesboro-GA-30238/US_8796PP-JNB',
    'https://landa.app/m/property/Churchill-Place-8780-Jonesboro-GA-30238/US_8780CP-JNB',
    'https://landa.app/m/property/Ashley-Way-8683-Douglasville-GA-30134/US_8683AW-DGL',
    'https://landa.app/m/property/Ashley-Way-8679-Douglasville-GA-30134/US_8679AW-DGL',
    'https://landa.app/m/property/Ashley-Way-8678-Douglasville-GA-30134/US_8678AW-DGL',
    'https://landa.app/m/property/Ashley-Way-8677-Douglasville-GA-30134/US_8677AW-DGL',
    'https://landa.app/m/property/Ashley-Way-8675-Douglasville-GA-30134/US_8675AW-DGL',
    'https://landa.app/m/property/Ashley-Way-8674-Douglasville-GA-30134/US_8674AW-DGL',
    'https://landa.app/m/property/Ashley-Way-8670-Douglasville-GA-30134/US_8670AW-DGL',
    'https://landa.app/m/property/Ashley-Way-8668-Douglasville-GA-30134/US_8668AW-DGL',
    'https://landa.app/m/property/Ashley-Way-8662-Douglasville-GA-30134/US_8662AW-DGL',
    //'https://landa.app/m/property/Ashley-Way-8659-Douglasville-GA-30134/US_8659AW-DGL',
    //'https://landa.app/m/property/Ashley-Way-8655-Douglasville-GA-30134/US_8655AW-DGL'

];

class property{
    constructor(address, dividend, share, ROI){
        this.address = address;
        this.dividend = dividend;
        this.share = share;
        this.ROI = ROI;
    }
}

async function scrapeProperties(URLs) {
    const browser = await puppeteer.launch({headless: 'false'});
    const page = await browser.newPage();
    var properties = [];

    for (const URL of URLs) {
        await Promise.all([
            page.waitForNavigation(),
            page.goto(URL, {waitUntil: 'load', timeout: 0}),
        ]);

        await page.waitForXPath('/html/body/div/div/div/main/div[2]/div[3]/div[2]/ul/div[1]/div[1]/div[1]/div');
        const [el] = await page.$x('/html/body/div/div/div/main/div[2]/div[3]/div[2]/ul/div[1]/div[1]/div[1]/div');
        const txt = await el.getProperty('textContent');
        var street = await txt.jsonValue();

        await page.waitForXPath('/html/body/div/div/div/main/div[2]/div[3]/div[2]/ul/div[4]/div[2]/div[2]/text()[1]', );
        const [el2] = await page.$x('/html/body/div/div/div/main/div[2]/div[3]/div[2]/ul/div[4]/div[2]/div[2]/text()[1]');
        const txt2 = await el2.getProperty('textContent');
        var dividend = await txt2.jsonValue();
        dividend = dividend.replace(/^\D+|\D+$/g, '');

        await page.waitForXPath('/html/body/div/div/div/main/div[2]/div[3]/div[2]/ul/div[2]/div[1]');
        const [el3] = await page.$x('/html/body/div/div/div/main/div[2]/div[3]/div[2]/ul/div[2]/div[1]');
        const txt3 = await el3.getProperty('textContent');
        var share = await txt3.jsonValue();
        share = share.replace(/^\D+|\D+$/g, '');

        var ROI = dividend * 12 / share;
        ROI = ROI * 100
        ROI = ROI.toFixed(2) + ' %';

        let investment = new property(street, dividend, share, ROI);
        properties.push(investment);

    }
    properties.sort(function(a, b) {
        return parseFloat(b.ROI) - parseFloat(a.ROI);
    });
    console.log(properties);

    await browser.close();
}

scrapeProperties(URLs);

Error string
TimeoutError: Waiting for selector /html/body/div/div/div/main/div[2]/div[3]/div[2]/ul/div[4]/div[2]/div[2]/text()[1] failed: Waiting failed: 30000ms exceeded     at Timeout.
Puppeteer configuration
No response
Puppeteer version
19.7.3
Node version
18.15.0
Package manager
npm
Package manager version
18.15.0
Operating system
Windows

You are probably hitting the resource limits. I would try to find the concurrency level that works for your hardware and the site you are automating. The site might also render different content/fail under load so the expected elements won't be there.


Rename package to pysubstrait

Testing:

Ran pytests
Regenerated proto classes
Built, imported, and ran pysubstrait in console


Missing blueprintIdentifier PBXObjectReference

In 5.0 XCScheme.BuildableReference.blueprintIdentifier isn't a PBXObjectReference, which means there's no way to set this to (for example) a PBXNativeTarget.reference

This change was reverted in 6.0


HowTo: Controller to Controller communication

As it seems to be a very diverse discussion:
How to achieve controller to controller communication in conjunction with services as model?
Solutions are in controller:

$rootScope.$watch

*$rootScope.$broadcast("mynamingpattern" ...

$rootScope.$emit(" ...

$scope.$watch( function(myservice

eventHandling: myService.listenToDataChangedHandler(function(){ ...

promises and notification event

or an angular independant messaging library

story continues how code in the service should look like.
How do you think?

What kind of communication do you need? If it's just one-way like in a controller that's used inside the html of another controller, you can just use data-attributes on your element and get them in your controller.

Hi Dieter,
data-attributes is the jQuery way, but I assume definitely not the angular way.
As described in this styleguide, it is recommanded to share state between services, which is a good idea !

@raDiesle Here are my thoughts on cross component communication: http://www.slideshare.net/rasheedwaraich/angular-js-recommended-practices-mini
You can go through slide # 16 to slide # 21.

Thanks @rasheedamir . Similar fundamental thoughts I have,
but it requires more clarification: code ! , situation to use, more pro-con,
I think it is too common summarized to put into a styleguide, but a great start for explanation.
How do you think? Something which should go into styleguide ?

@raDiesle , I don't think so. When you already have your item, and the subcontroller is used for a part of that item or a subitem of it, it's far better to just use an attribute to 'give' that to your controller. How else are you gonna set the item? You need to have some sort of identifier or the object itself. Unless you hold global states, which is bad, you can't use anything else here....

It depends. Controller-->  sub-Controller doesn't require anything. Directives use isolate scopes (or not).
It really comes down to 2 controllers without a defined relationship. Yes you can use messaging (broadcast/emit), bur I prefer not to spam my app and its fire and forget. If I want the message to get there, I use a service/factory.
I'm not sure this is a style. I see this more as a tip. And its not one way, there are other ways too.
Closing

@raDiesle
you might like this: http://www.aurorasolutions.io/angularjs-cross-component-communication/
one of my team members posted article on the topic "AngularJS Cross Component Communication"


Hide raw text if output is an ipywidget

Codecov Report

:exclamation: No coverage uploaded for pull request base (ds/ipyWidgets@6141edb). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff                @@
##             ds/ipyWidgets   #10620   +/-   ##
================================================
  Coverage                 ?   60.61%
================================================
  Files                    ?      579
  Lines                    ?    31517
  Branches                 ?     4485
================================================
  Hits                     ?    19103
  Misses                   ?    11441
  Partials                 ?      973

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 6141edb...41b1fc4. Read the comment docs.