Darknet Component - No section: 'net' / armv7

Hello there,
first time viserion user here - I seem to have an issue with the darknet component. It looks like I hit #476 - I am using viserion currently on my QNAP NAS 431P3 which is using armv7. If I understand what you are describing in the linked issue - image / object detection will be horribly slow and therefore you left out the necessary files from that image?
[s6-init] making user provided files available at /var/run/s6/etc...exited 0.
[s6-init] ensuring user provided files have correct perms...exited 0.
[fix-attrs.d] applying ownership & permissions fixes...
[fix-attrs.d] done.
[cont-init.d] executing container initialization scripts...
[cont-init.d] 10-adduser: executing...
************************ UID/GID *************************
User uid:    0
User gid:    0
************************** Done **************************
[cont-init.d] 10-adduser: exited 0.
[cont-init.d] 20-gid-video-device: executing...
[cont-init.d] 20-gid-video-device: exited 0.
[cont-init.d] 30-edgetpu-permission: executing...
************** Setting EdgeTPU permissions ***************
Coral Vendor IDs:
"1a6e"
"18d1"
No EdgeTPU USB device was found
************************** Done **************************
[cont-init.d] 30-edgetpu-permission: exited 0.
[cont-init.d] 40-set-env-vars: executing...
****** Checking for hardware acceleration platforms ******
OpenCL cannot be used
VA-API cannot be used
CUDA cannot be used
*********************** Done *****************************
[cont-init.d] 40-set-env-vars: exited 0.
[cont-init.d] 50-check-if-rpi: executing...
********** Checking if we are running on an RPi **********
Not running on any supported RPi
*********************** Done *****************************
[cont-init.d] 50-check-if-rpi: exited 0.
[cont-init.d] 55-check-if-jetson: executing...
****** Checking if we are running on a Jetson Board ******
Not running on any supported Jetson board
*********************** Done *****************************
[cont-init.d] 55-check-if-jetson: exited 0.
[cont-init.d] 60-ffmpeg-path: executing...
****************** Getting FFmpeg path *******************
FFmpeg path: /home/abc/bin/ffmpeg
*********************** Done *****************************
[cont-init.d] 60-ffmpeg-path: exited 0.
[cont-init.d] 70-gstreamer-path: executing...
***************** Getting GStreamer path *****************
GStreamer path: /usr/bin/gst-launch-1.0
*********************** Done *****************************
[cont-init.d] 70-gstreamer-path: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[2023-05-03 00:24:51] [INFO    ] [viseron.core] - -------------------------------------------
[2023-05-03 00:24:51] [INFO    ] [viseron.core] - Initializing Viseron
[2023-05-03 00:24:52] [INFO    ] [viseron.components] - Setting up component data_stream
[2023-05-03 00:24:52] [INFO    ] [viseron.components] - Setup of component data_stream took 0.0 seconds
[2023-05-03 00:24:52] [INFO    ] [viseron.components] - Setting up component webserver
[2023-05-03 00:24:52] [INFO    ] [viseron.components] - Setup of component webserver took 0.0 seconds
[2023-05-03 00:24:52] [INFO    ] [viseron.components] - Setting up component nvr
[2023-05-03 00:24:52] [INFO    ] [viseron.components] - Setting up component darknet
[2023-05-03 00:24:52] [INFO    ] [viseron.components] - Setting up component mog2
[2023-05-03 00:24:52] [INFO    ] [viseron.components] - Setting up component ffmpeg
[2023-05-03 00:24:52] [INFO    ] [viseron.components] - Setup of component nvr took 0.0 seconds
[2023-05-03 00:24:52] [INFO    ] [viseron.components] - Setup of component ffmpeg took 0.0 seconds
[2023-05-03 00:24:52] [INFO    ] [viseron.components] - Setup of component mog2 took 0.0 seconds
Traceback (most recent call last):
  File "/src/viseron/components/__init__.py", line 152, in setup_component
    result = component_module.setup(self._vis, config)
  File "/src/viseron/components/darknet/__init__.py", line 125, in setup
    vis.data[COMPONENT] = DarknetDNN(vis, config[CONFIG_OBJECT_DETECTOR])
  File "/src/viseron/components/darknet/__init__.py", line 219, in __init__
    super().__init__(vis, config)
  File "/src/viseron/components/darknet/__init__.py", line 164, in __init__
    self._model_width = int(model_config.get("net", "width"))
  File "/usr/lib/python3.8/configparser.py", line 781, in get
    d = self._unify_values(section, vars)
  File "/usr/lib/python3.8/configparser.py", line 1149, in _unify_values
    raise NoSectionError(section) from None
configparser.NoSectionError: No section: 'net'
[2023-05-03 00:24:52] [ERROR   ] [viseron.components] - Uncaught exception setting up component darknet: No section: 'net'
None
[2023-05-03 00:24:52] [ERROR   ] [viseron.components] - Setup of component darknet failed
[2023-05-03 00:24:52] [ERROR   ] [viseron.components] - Failed setup of component darknet
[2023-05-03 00:24:52] [INFO    ] [viseron.components] - Setting up domain camera for component ffmpeg with identifier camera_1

Yes I left them out for that reason, also not to bloat the image with extra models.
However I realize now that it's a bad design choice and not very user friendly.
I will add the tiny models which are quite small. Then every user can make their own decisions on whether it's fast enough

The yolov7-tiny model is now included in the dev tag.
It should be picked up automatically as the default model.


Please invite me to the community

Name
Juan David Sanchez Rodriguez
Additional Context
I am a strong communicator and team player.
I enjoy working in a collaborative environment where I can bounce ideas.

Invitation sent to join the GitHub Organization. Welcome to Infraform 🎉Please check your mail to accept the invitation sent by GitHub! 🤖About usInfraform is a remarkable open-source solution tailored for serverless platforms, aimed at empowering creators to craft their infrastructure solutions. 💻 We're thrilled to welcome you into our vibrant community! ✨ We eagerly anticipate your valuable contributions! 🤓 Don't forget to explore our community on Github and connect with us on LinkedIn 💜 Feel free to spread the word about our community to like-minded individuals interested in development, DevOps, technology, and collaborative contributions. 👉💜👈


file download example

is it possible to download multiple files in parallel using httpie?
an example would be good

Yes that's possible, since 4.0 and no stream you can use raw mode to get a Buffer. Basicaly you just need to manage Promise yourself (nothing crazy here).
const { data } = await httpie.get<Buffer>("url_to_file", { mode: "raw" });
console.log(data) // Buffer

Also possible with stream :)
function downloadOne(url, writable) {
  const cursor = httpie.stream("GET", url);

  return cursor(() => writable);
}

But how do i save the content to a file?
And how do i download multiple files in parallel?
I have like 10k files i need to download

@higoka Use a writable stream as writable (fs.createWriteStream for example). Use Promises to manage multiple files (with Promise.all or Promise.allSettled).
You probably need to study a bit more about Node and JavaScript.

import fs from "node:fs";
import path from "node:path";

import * as httpie from "@myunisoft/httpie";

const apiAgent = new httpie.Agent({
  // Configure maximum of Clients to use for the undici pool
  connections: 10
});

// CONSTANTS
const kDownloadPath = path.join(__dirname, "directory");

function downloadOne(url) {
  const cursor = httpie.stream("GET", url, {
    agent: apiAgent
  });

  return cursor(() => fs.createWriteStream(path.join(kDownloadPath, "filename.xxx")));
}

const filesURL = [
  "https://github.com/NodeSecure/i18n/archive/main.tar.gz",
  "https://github.com/NodeSecure/i18n/archive/main.tar.gz",
  "https://github.com/NodeSecure/i18n/archive/main.tar.gz"
];

await Promise.allSettled(
  filesURL.map((url) => downloadOne(url))
);

However you still need to:

Think how to properly manage errors
Detect or extract the filename (probably depend on the API you'r using)

Most of the time the filename is sent in headers (exploitable in cursor callback)
function getFileName(headers) {
  const [, filename] = /filename[^;=\n]*=((['"]).*?\2|[^;\n]*)/.exec(
    headers["content-disposition"]
  );

  return filename;
}

return cursor(({ headers }) => fs.createWriteStream(path.join(kDownloadPath, getFileName(headers))));


file-size-threshold generates wrong size files

i tried to use odbc2parquet with following options :
odbc2parquet query  --batch-size-memory 100Mb --column-compression-default zstd --file-size-threshold 1GiB --no-empty-file --connection-string "some connection string" "/code/data/processed/test_file_from_server.par" "some sql query"
If i use this query without file-size limitation i get 1.8Gb result file.
If i use limitation, first file is generated in right way ~ 1Gb, i expected to gain  next one with size near 800 Mb, but i got lots of files  with 10-15kb size. (total sum - near 800 Mb). 10-15Kb - the size of one row-group

Hello @Rickitik ,
thank you for reporting the bug. odbc2parquet 3.0.0 has been released which contains a fix.
Best, Markus


Bypass AssertionErrors on nested Hyperlinked fields, ref #119

Coverage remained the same at 96.044% when pulling 5f749de1bc2810b43dd9f860c7eb6381b3251330 on bashu:patch-1 into 624ad01afbc86fa15b4e652406f3bdcd01f36e00 on Tivix:master.

Hi,
as a user of rest-auth, thanks for the contribution!
This repo is not maintained anymore, so the development moved to dj-rest-auth. (reference: https://github.com/Tivix/django-rest-auth/issues/568)
It may be best, if you move this PR there. (and upgrade to using dj_rest_auth)
new repo link: https://github.com/jazzband/dj-rest-auth (I'm not the upkeeper of that repo, it just makes sense for me to help you merge your PR)
Many Thanks,
Barney


Add Rows wrapper and safetyhooks

Work in progress of safetyhooks, which panics when sql.Rows are not closed, this is useful to detect leaks.
Based on sharp-edge finalizers https://crawshaw.io/blog/sharp-edged-finalizers.

Coverage decreased (-0.5%) to 75.746% when pulling 2c7ef1a5fca763a8e0033ce04f214760f44603d2 on safetyhooks into 1b9e7183e69f37a50efe0ec7eccea5990ff9dab4 on master.

Coverage decreased (-0.5%) to 75.746% when pulling 2c7ef1a5fca763a8e0033ce04f214760f44603d2 on safetyhooks into 1b9e7183e69f37a50efe0ec7eccea5990ff9dab4 on master.


feat: at_cli_commons: Add standardAtClientStoragePath and standardAtClientStorageDir to utils.dart

- What I did
feat: at_cli_commons: Add standardAtClientStoragePath and standardAtClientStorageDir to utils.dart
refactor: use standardAtClientStoragePath in CLIBase constructor
fix: use platform path separators in CLIBase constructor

@sitaram-kalluri @srieteja After this has been merged I will take care of the merge from trunk into the other PRs which are in progress here ( https://github.com/atsign-foundation/at_libraries/pull/682 and https://github.com/atsign-foundation/at_libraries/pull/670 )


Expose OAuth headers

Hi, can you elaborate a bit more on what you mean?

I mean that HTTP headers contains useful information i.e. about oauth scopes and should be available for different requests.

Example
X-OAuth-Scopes: admin:repo_hook, repo:status, repo, user:email, read:org
X-Accepted-OAuth-Scopes: admin:repo_hook, public_repo, read:repo_hook, repo, write:repo_hook


Add "presets" to Datatable component

In Datatable component we have filters. It is really useful to save presets of this filters in the local storage so that they can be applied with one click. For instance in the traces, filter on the certain span paths can be quite handy preset.

Hey @skull8888888, can you provide more detail on it?
I would love to work on this.
@dinmukhamedm Can I work on this?

@lucky29-git go for it!
So, to provide more context, currently the datatable filters (used in traces, sessions, and spans views) are stored as URL params under the key filter. It is a stringified array of json filters, where each has keys column, operator and value. We would like to add an ability for users to store a certain filter combination as a preset. Each preset must have a name. The presets must be saved in the localStorage and can be retrieved using something like a Select or a ComboBox component.
Example user flow:

Go to spans view in the traces page
Filter spans by path = my_agent.anthropic.chat and latency > 1.
Click save as preset. In the appearing dialog/pop-up, name the preset something like anthropic slower than 1s.
Go elsewhere in the platform
Come back to the spans view
Select anthropic slower than 1s from the available presets
Laminar fills in the URL param filters with the saved value and shows the filtered data

To have some data for (2) you might need to observe some code and send it to localhost at grpc_port = 8001. Read more in our docs

Thanks @dinmukhamedm I will onto it, will reach out to you if needed any help.

hi @lucky29-git Are you working on this? If not Please let me know.

Hi @sarthakvijayvargiya, I'll work on this
Just needed some time I'm in the middle of my exams right now.
Will let you know if needed any help


🫒 Auto optimization

Describe your changes
Reopen the auto optimization PR with pass flows templates.

Previous PR: https://github.com/microsoft/Olive/pull/610
Checklist before requesting a review

[ ] Add unit tests for this change.
[ ] Make sure all tests can pass.
[ ] Update documents if necessary.
[ ] Lint and apply fixes to your code by running lintrunner -a
[ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes.

(Optional) Issue link

@devang-ml For your comments:

we may not need regulate feature.

With the pass flows templates, we define the possible pass combinations for given accelerators/EP/precisions. It is OK to not regulate the pass_flows.
But the pass config is empty. For example for gpu_cuda_fp32 and gpu_cuda_fp16, there are both OrtTransformersOptimization, but we need assign different config to enable fp16 or not.

do we need update MANIFEST.in to include these files?

Refers to: olive/auto_optimizer/config_template/opt_level_passes.yaml:15 in 3430b4e. [](commit_id = 3430b4e9dbb0d025eb2fb1e95f3ce232ed993418, deletion_comment = False)

Do we have any plan to support multi EP with auto optimizer?

Do we have any plan to support multi EP with auto optimizer?

Thanks! We really have that in planning. From my understanding, the multi-eps should work if we enhance the pass capability properly unless there are potential bugs for mult-eps.  But I still have a few questions for how to distribute multi-eps in different target device and collect the results back.
Let us do that in next step.


Move 'Using the Worker API' into user manual

Content taken from guides.gradle.org/using-the-worker-api
The content is mostly kept in place with minor updates:

Samples are integrated in snippet structure and updated to be idiomatic
Now refers to itself as 'section' and not as 'guide'

Furthermore, all "task development" sections are now grouped in one
chapter - Developing Gradle Tasks - in the "Extending Gradle" area of
the user manual.

@bot-gradle test this

OK, I've already triggered ReadyForMerge build for you.


Allow for other shells

Make sure we're only using posix shell scripts

We could switch to /bin/sh

Why be prescriptive? Just use the users shell as this will allow them to do things like inline for loops in their preferred style:
https://github.com/andrewjanke/qbatch/blob/master/qbatch#L104
and
https://github.com/andrewjanke/qbatch/blob/master/qbatch#L124

Good thought, we have to be careful that our qbatch magics support other shells.
I recall there being some "interesting" interactions between cluster software and the shell types, including what's "allowed" as a shell.

This would also fix the "I need to use a login shell" problem we may be experiencing for some users on scinet.

Method:
Expose shebang as QBATCH_SHEBANG
Expose as command-line option
Use in header of jobfiles
Use as spawner for single-job commands.


feat: show account username suffices

Resolve https://github.com/hirosystems/stacks-wallet-web/issues/1409
Before

After

@kyranjamie any idea why the typecheck is failing here re: parameters?
This function is used elsewhere already with just one parameter. E.g. https://github.com/hirosystems/stacks-wallet-web/blob/4925bad0400daf0d8735ad2ab9ae5df330d6e28c/src/app/features/current-account/current-account-avatar.tsx#L20

I just realized @kyranjamie is out. Redirecting to @fbwoolf 🙏

I just realized @kyranjamie is out. Redirecting to @fbwoolf 🙏

It looks like you deleted the useMemo dependency array. I can fix this for you?


Fix directory selection double-selecting

open_entry gets called twice when double-tapping (which is necessary to open/go into a directory with dirselect=True). Remove one of them.
Also remove a line which doesn't do anything (in the dirselect=False case) which prevented this bug from being more obvious, since it would only be hit in the dirselect=True codepath.
Linked to this issue (reported by me) - https://github.com/kivy/kivy/issues/3708

Am surprised this wasn't merged for 1.9.1, is there something else I need to do?

@ngoonee, thanks a lot.


[wip] ci: 👷update CI to populate with secrets

Doing a bit of testing in production to see if there is an easy solution to #47.

Huh! I don't even have the option to force CI. Seems there are no checks!

Worst thing this can do is break CI workflow - which ATM already doesn't work for contributors, let's try it and see what happens after I make a new PR.


Request of Deploy on ZetaChain Athens 3 Testnet

Chain Name
ZetaChain Athens 3 Testnet
Chain ID
7001
RPC URL
https://rpc.ankr.com/zetachain_evm_athens_testnet
Block Explorer URL
https://explorer.zetachain.com/
Deployment Funds
Yes

Done, thanks!


Wrap text doesn't work in "Xenakios/SWS: Rename takes..."(Only in Windows)

Hi !
I'm using ReaperBlog video scripts, and this feature is useful to rename midi items with text wrap to the next line, to create titles. It works on MacOs, but somehow does not work on Windows. Can you help with it ?
Greetings from Ukraine !
Example:
https://drive.google.com/file/d/1rCv3QNJ9-DYS1exNUlfPMT4znlzLMruB/view?usp=sharing

That text field isn't meant to receive multiline input. Unfortunately, the behavior on Windows and Linux is the correct one.
(SWELL doesn't obey ~ES_MULTILINE: every text box in REAPER on macOS is multiline.)


add support for batch size in metrics service

Currently Metrics Service sends all metrics in a single batch and that causes server side gRPC Message Size (typical 4MB) to trip causing metrics not to be sent. Even if we increase the size, we end up with a catching game. The proposal is to add batch_size defaults to the current behaviour.

cc @jmarantz as metrics_service codeowner.


doc: document Boring Crypto release schedule

This is more of a question than an issue, though it would be helpful to document the release schedule for Go + Boring Crypto (which I'll call goboring) somewhere. Sorry if this documentation exists but I missed it! pointers to existing documentation would be much appreciated. If there's no documentation and a doc PR would be welcome, I'd be happy to open one given some guidance!
Go 1.17 was released last week (Aug 16th), but it appears the corresponding build of goboring is not available yet (checking the Google Cloud Storage bucket as well as Docker Hub). Are these releases typically expected to track the main Go release closely, or do they lag for some period of time?
Thanks so much for producing goboring :heart:
cc @katiehockman @FiloSottile who I gather usually maintain this build

It's also possible that there are no guarantees, and that's why the boringcrypto release process is not documented.
Go1.16 was released on Feb 16th, but the change adding the corresponding boringcrypto release was merged on March 10th. So it appears that the boringcrypto releases can lag behind.

For anyone needing a new version, we found this image tag to be helpful: us-docker.pkg.dev/google.com/api-project-999119582588/go-boringcrypto/golang:1.17.2b7 - I have no idea how well-supported this is, though

boringcrypto is now behind GOEXPERIMENT, and I believe that it won't see any further work with #69536 being the future direction.


clarification issue

Explain the installation procedure according to the OS for each operating system chosen : explain the required commands specific to each os to be able to install the ilke distribution

Fixed by #21


Investigate options for 'Daemonized' sessions

I would like to investigate options for 'daemonizing' kakoune sessions through kampliment in order to improve the startup time of kamp edit.
My current proposal for this functionality would be to add the following in kampliment

[ ] An option to behave similar to the functionality of the useGitDirSessions implementation in kks, this would also include a boolean environment variable KAMP_USE_GITDIR_SESSIONS that works in the same way.

[ ] An option to behave similar to the functionality of the defaultSessions implementation in kks, this would also include a boolean environment variable KAMP_DEFAULT_SESSION that works in the same way.

[ ] Alternatively a kampliment.toml etc might be useful if we intend to add more configuration options to kampliment in the future.

Additionally I think KAMP_USE_GITDIR_SESSIONS and KAMP_DEFAULT_SESSION would help to daemonize other tools like kak-lsp. For example we can use these sessions with kak-lsp --daemonize. I'll see if I can learn some rust and submit a PR.

I think following script fulfills KAMP_USE_GITDIR_SESSIONS scenario:
#!/bin/sh

if [ $# -eq 0 ]; then
    echo "some arg is required"
    exit
fi

directory="$(dirname $1)"

git_dir=$(git -C "$directory" rev-parse --show-toplevel 2>/dev/null)

if [ $? -eq 128 ]; then
    echo "not a git dir"
    exit
fi

session=$(echo "${git_dir##*/}" | sed "s/[^[:alnum:]]//g")

kak -clear
nohup kak -d -s "$session" 2>/dev/null &
kamp -s "$session" edit "$@"

I would like to investigate options for 'daemonizing' kakoune sessions through kampliment in order to improve the startup time of kamp edit.

I'm not sure if 'daemonizing' kakoune sessions will improve the startup time at all. However I decided to benchmark kamp edit with kks edit. To my surprise kks was faster. After some digging I found out that kamp is trying to auto focus corresponding client while kks is not. So I've added optional edit --focus switch in order not to focus by default.
Here is final benchmark invoking edit rc/lsp.kak. This file is about 2500K lines.
❯ hyperfine --warmup 3 -N 'kks edit rc/lsp.kak' 'kamp edit rc/lsp.kak'
Benchmark 1: kks edit rc/lsp.kak
  Time (mean ± σ):       1.6 ms ±   0.4 ms    [User: 1.5 ms, System: 0.2 ms]
  Range (min … max):     1.4 ms …  11.5 ms    797 runs

  Warning: The first benchmarking run for this command was significantly slower than the rest (3.8 ms). This could be caused by
 (filesystem) caches that were not filled until after the first run. You should consider using the '--warmup' option to fill th
ose caches before the actual benchmark. Alternatively, use the '--prepare' option to clear the caches before each timing run.

Benchmark 2: kamp edit rc/lsp.kak
  Time (mean ± σ):       1.5 ms ±   0.1 ms    [User: 1.3 ms, System: 0.2 ms]
  Range (min … max):     1.2 ms …   2.6 ms    2041 runs

  Warning: Statistical outliers were detected. Consider re-running this benchmark on a quiet PC without any interferences from
other programs. It might help to use the '--warmup' or '--prepare' options.

Summary
  'kamp edit rc/lsp.kak' ran
    1.11 ± 0.27 times faster than 'kks edit rc/lsp.kak'
❯ hyperfine --warmup 3 -N 'kamp edit rc/lsp.kak' 'kks edit rc/lsp.kak'
Benchmark 1: kamp edit rc/lsp.kak
  Time (mean ± σ):       1.5 ms ±   0.1 ms    [User: 1.3 ms, System: 0.2 ms]
  Range (min … max):     1.2 ms …   5.5 ms    1416 runs

  Warning: Statistical outliers were detected. Consider re-running this benchmark on a quiet PC without any interferences from
other programs. It might help to use the '--warmup' or '--prepare' options.

Benchmark 2: kks edit rc/lsp.kak
  Time (mean ± σ):       1.6 ms ±   0.1 ms    [User: 1.4 ms, System: 0.3 ms]
  Range (min … max):     1.3 ms …   3.9 ms    1808 runs

  Warning: Statistical outliers were detected. Consider re-running this benchmark on a quiet PC without any interferences from
other programs. It might help to use the '--warmup' or '--prepare' options.

Summary
  'kamp edit rc/lsp.kak' ran
    1.08 ± 0.13 times faster than 'kks edit rc/lsp.kak'

~/.local/share/kak/cork/plugins/kak-lsp/repo master *1                                                                       5s


Attachment - When opening the last one, the first added Public Image is opened

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.72-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Yes, reproducible on both
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5309350
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Open HybridApp
Log in to the HT account
Go to any conversation
Send the following attachments:
A. Image 1 B. Image 2 C. ![off](https://images.unsplash.com/photo-1709983723739-d72ea88434dd?q=80&w=2940&auto=format&fit=crop&ixlib=rb-4.0.3&ixid=M3wxMjA3fDB8MHxwaG90by1wYWdlfHx8fGVufDB8fHx8fA%3D%3D) D. Image 3 E. Image 4 F. ![off](https://images.unsplash.com/photo-1709983723739-d72ea88434dd?q=80&w=2940&auto=format&fit=crop&ixlib=rb-4.0.3&ixid=M3wxMjA3fDB8MHxwaG90by1wYWdlfHx8fGVufDB8fHx8fA%3D%3D)
Open the image from the list under the letter F
Click on the right arrow
Note that the first public image has opened up

Expected Result:
When opening the last added Public Image, it is the last one that must be opened
Actual Result:
When opening the last added Public Image, the first added Public Image is opened instead.
Workaround:
Unknown
Platforms:

[x] Android: Standalone
[x] Android: HybridApp
[x] Android: mWeb Chrome
[x] iOS: Standalone
[x] iOS: HybridApp
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/f10caf94-a9c4-4dd0-afa3-e1e502bf6b22
View all open jobs on GitHub

Proposal
Please re-state the problem that we are trying to solve in this issue.
Attachment - When opening the last one, the first added Public Image is opened
What is the root cause of that problem?
When building the attachments list in a report here
https://github.com/Expensify/App/blob/48b80f65e80d304965cbeb3573d7d6678fb9bd3a/src/components/Attachments/AttachmentCarousel/extractAttachments.ts#L44-L46
we are including unique list of sources so if we have two same mardown images the source is duplicate so the second one in the order will not be included in attachments so when we open it the page index will be the first one's index.
BTW it is a limitation that they have postponed it to be done later when implementing image markdown, here is the discussion
What changes do you think we should make in order to solve the problem?

We need to first start uniquely identify the attachments via their reportActionID.
Add reportActionID in the attachment route
https://github.com/Expensify/App/blob/48b80f65e80d304965cbeb3573d7d6678fb9bd3a/src/ROUTES.ts#L324-L326
 const reportActionParam = reportActionID ? `&reportActionID=${reportActionID}` : '';

            return `attachment?source=${encodeURIComponent(
                url,
            )}&type=${type}${reportParam}${reportActionParam}${accountParam}${authTokenParam}${fileNameParam}${attachmentLinkParam}` as const;
        },

pass the reportActionID argument everywhere we use this getRoute
In ReportAttachments we will get the reportActionID from the route and pass it down to AttachmentModal > Attachment Carousel
Remove the set as we don't need to save unique list of sources here
https://github.com/Expensify/App/blob/48b80f65e80d304965cbeb3573d7d6678fb9bd3a/src/components/Attachments/AttachmentCarousel/extractAttachments.ts#L33
Change the activeSource state to activeReportActionID
https://github.com/Expensify/App/blob/48b80f65e80d304965cbeb3573d7d6678fb9bd3a/src/components/Attachments/AttachmentCarousel/index.tsx#L75
    const [activeReportActionID, setActiveReportActionID] = useState<string | null>(reportActionID);

https://github.com/Expensify/App/blob/48b80f65e80d304965cbeb3573d7d6678fb9bd3a/src/components/Attachments/AttachmentCarousel/index.tsx#L86
const compareImage = useCallback(
        (attachment: Attachment) => attachment.reportActionID === reportActionID && (!attachmentLink || attachment.attachmentLink === attachmentLink),
        [attachmentLink, reportActionID],
    );

https://github.com/Expensify/App/blob/48b80f65e80d304965cbeb3573d7d6678fb9bd3a/src/components/Attachments/AttachmentCarousel/index.tsx#L178
                setActiveReportActionID(item.reportActionID);

https://github.com/Expensify/App/blob/48b80f65e80d304965cbeb3573d7d6678fb9bd3a/src/components/Attachments/AttachmentCarousel/index.tsx#L207-L210
    const extractItemKey = useCallback((item: Attachment) => `reportActionID-${item.reportActionID}`, []);

https://github.com/Expensify/App/blob/48b80f65e80d304965cbeb3573d7d6678fb9bd3a/src/components/Attachments/AttachmentCarousel/index.tsx#L242
                    isFocused={activeReportActionID === item.reportActionID}

and do similar changes in the index.native file too

Result
https://github.com/user-attachments/assets/9f7cbe7c-03d8-4248-9d86-8c9375160269

### What specific scenarios should we cover in automated tests to prevent reintroducing this issue in the future?
What alternative solutions did you explore? (Optional)

Similarly, this isn't a convert/retain/migrate bug, so filing in quality.

Hi, Checking now

BTW it is a limitation that they have postponed it to be done later when implementing image markdown, here is the discussion

Thank you for pointing it out and the RCA seems good.
@roryabraham / @kidroca Since you where working on original implementation and found the issue previously as well. Could you kindly let us know if we should fix the issue as part of this? Also let us know if you have some insights on the solution suggested or problems we could face if you know.

BTW it is a limitation that they have postponed it to be done later when implementing image markdown, here is the discussion

Thank you for pointing it out and the RCA seems good. Hi @roryabraham / @kidroca Since you where working on original implementation and found the issue previously as well. Could you kindly let us know if we should fix the issue as part of this? Also let us know if you have some insights on the solution suggested or problems we could face if you know.

Gentle ping

BTW it is a limitation that they have postponed it to be done later when implementing image markdown, here is the discussion

Thank you for pointing it out and the RCA seems good. Hi @roryabraham / @kidroca Since you where working on original implementation and found the issue previously as well. Could you kindly let us know if we should fix the issue as part of this? Also let us know if you have some insights on the solution suggested or problems we could face if you know.

Gentle ping

I think people are out on holidays


Problem logging into HuggingFace, and changing output dimensions doesn't work?

Hey thankyou for this amazing script. I had a problem using 'huggingface-cli login' which wouldn't let me paste my token, but I got around it by changing line 154 of generate_images.py to add my token.
Everything works great but I can't seem to change the output dimensions, it either continues doing 512,512 or has an error about mismatch? (sorry I'm very new to Python scripting, but your video was really helpful)

Ok the mismatch was because width and height variables weren't being passed into the pipe, which needs to be modified to (GENERATE_DIVERSE)
with autocast(device):                 image = pipe(  # Diffuse magic.                     prompt,                     height=height,                     width=width,                     num_inference_steps=num_inference_steps,                     latents=init_latent,                     guidance_scale=guidance_scale                 )["sample"][0]

I was also not able to paste my token, what did you change on line 154?

I was also not able to paste my token, what did you change on line 154?

Hey, on line 154 mine reads:
use_auth_token="randomtokenstringhere"


dolos daemon shutsdown after awhile

After running dolos daemon for awhile it seems to stop at somepoint. (Maybe after 1-2 hours or more syncing Preview)
here is the logs when its about to shutdown.
2024-01-29T04:07:01.147101Z  INFO stage{stage="roll"}: gasket::runtime: switching stage phase prev_phase=Working next_phase=Teardown
2024-01-29T04:07:01.147037Z  INFO stage{stage="pull"}: gasket::runtime: switching stage phase prev_phase=Working next_phase=Teardown
2024-01-29T04:07:01.147403Z  INFO stage{stage="pull"}: gasket::runtime: stage teardown ok
2024-01-29T04:07:01.147423Z  INFO stage{stage="pull"}: gasket::runtime: switching stage phase prev_phase=Teardown next_phase=Ended
2024-01-29T04:07:01.147371Z  INFO stage{stage="roll"}: gasket::runtime: stage teardown ok
2024-01-29T04:07:01.147458Z  INFO stage{stage="roll"}: gasket::runtime: switching stage phase prev_phase=Teardown next_phase=Ended
2024-01-29T04:07:01.147653Z  INFO stage{stage="ledger"}: gasket::runtime: stage dismissed
2024-01-29T04:07:01.147681Z  INFO stage{stage="ledger"}: gasket::runtime: switching stage phase prev_phase=Working next_phase=Teardown
2024-01-29T04:07:01.147690Z  INFO stage{stage="ledger"}: gasket::runtime: stage teardown ok
2024-01-29T04:07:01.147693Z  INFO stage{stage="ledger"}: gasket::runtime: switching stage phase prev_phase=Teardown next_phase=Ended
2024-01-29T04:07:01.149065Z ERROR stage{stage="chain"}:schedule: gasket::framework: x=error receiving work unit through input port
2024-01-29T04:07:01.149094Z ERROR stage{stage="chain"}: gasket::runtime: stage should stop retry=Retry(None)
2024-01-29T04:07:01.149110Z  INFO stage{stage="chain"}: gasket::runtime: switching stage phase prev_phase=Working next_phase=Teardown
2024-01-29T04:07:01.149117Z  INFO stage{stage="chain"}: gasket::runtime: stage teardown ok
2024-01-29T04:07:01.149130Z  INFO stage{stage="chain"}: gasket::runtime: switching stage phase prev_phase=Teardown next_phase=Ended

I haven't seen this behavior.

Which version are you running?
these logs show that the process has already started to shutdown, but don't show the root cause. Do you have the previous logs?
can you reproduce consistently?

not reproducible in latest versions.


migrate to travis containers for faster build start times

As per:
https://github.com/uProxy/uproxy-lib/pull/264

Reviewed 1 of 1 files at r1.
Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from the review on Reviewable.io

:thumbsup: (I think)

Review status: :shipit: all files reviewed at latest revision, all discussions resolved, all commit checks successful.

Comments from the review on Reviewable.io


Use jackson annotations to serialize and deserialize Account and Container

Use jackson annotation to serialize and deserialize Account and Container, so we don't have to write toJson and fromJson method each time we have a new field added to account or container.
Still keep fromJson and toJson method to make sure this PR won't change anything.
Will remove it after it's tested with requests.

Codecov Report

Merging #1904 (8e802a8) into master (6f21d94) will decrease coverage by 58.57%.
The diff coverage is 4.37%.

@@              Coverage Diff              @@
##             master    #1904       +/-   ##
=============================================
- Coverage     74.04%   15.47%   -58.58%
+ Complexity     9656     2278     -7378
=============================================
  Files           690      690
  Lines         52984    53067       +83
  Branches       6593     6598        +5
=============================================
- Hits          39232     8210    -31022
- Misses        11702    44380    +32678
+ Partials       2050      477     -1573

Impacted Files
Coverage Δ

.../java/com/github/ambry/account/AccountInfoMap.java
0.00% <0.00%> (-91.00%)
:arrow_down:

...com/github/ambry/account/AccountMetadataStore.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...va/com/github/ambry/account/BackupFileManager.java
0.00% <0.00%> (-82.03%)
:arrow_down:

.../com/github/ambry/account/HelixAccountService.java
0.00% <0.00%> (-83.34%)
:arrow_down:

.../com/github/ambry/account/LegacyMetadataStore.java
0.00% <0.00%> (-88.64%)
:arrow_down:

.../com/github/ambry/account/MySqlAccountService.java
0.00% <0.00%> (-81.49%)
:arrow_down:

...ain/java/com/github/ambry/account/RouterStore.java
0.00% <0.00%> (-74.32%)
:arrow_down:

...ava/com/github/ambry/account/mysql/AccountDao.java
0.00% <0.00%> (-91.39%)
:arrow_down:

.../java/com/github/ambry/account/AccountBuilder.java
0.00% <0.00%> (-94.74%)
:arrow_down:

...m/github/ambry/account/AccountCollectionSerde.java
0.00% <0.00%> (-88.47%)
:arrow_down:

... and 559 more

Continue to review full report at Codecov.

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


Workflow_Reporter Script

Created a script that outputs all of the executions of a project into a dataframe. Working on doing the same for pipelines, let me know if there should be more metrics added.

https://github.com/sparkflows/fire-tools/commit/50357d0458d507238dc7e2ab5b0d88eda4a25fe4

Added some more functionality to the script and moved it to a new directory


Remove account setting and use metaData in account instead

Keep UI and allow to read/write Beancount file?

move accountName to ImportedTransaction to keep allowing detecting which posting the user should not edit


Unable to log in JSON structured format

Hi, I am trying to follow: JSON logging option in the console
But the following code: console.log({ data: 12.3, flag: false });
Produces a single string instead of key-value pairs:
{
    "time": "2024-03-01T19:55:28.045Z",
    "requestId": "bf340645-6ed9-40a3-ae15-2a82dc83ec5d",
    "level": "INFO",
    "message": "{ data: 12.3, flag: false }"
}

Hi @nocquidant! Thanks for the report. There seems to be a bit of inconsistent behavior with how Node.js outputs JSON log format in Lambda. I see now that if only 1 argument was passed to console.log it gets expanded and converted into json, if multiple are added, it's consistent with LLRT. I'm raising this with the service team.
Question: Ignoring how it works today, what would in your opinion be the best output for something like this?
console.log({ data: 12.3, flag: false },"example",{obj2: 1});

Fixed by: https://github.com/awslabs/llrt/pull/264


fix alignment in topology-spread-constraints.md

When syncing with this file:
content/en/docs/concepts/scheduling-eviction/topology-spread-constraints.md

find an alignment is incorrect.

Preview: https://deploy-preview-37373--kubernetes-io-main-staging.netlify.app/docs/concepts/scheduling-eviction/topology-spread-constraints/

Thanks.
/lgtm
/approve


Add .env.example file

In readme.md there is mention of a command that copies the .env.example into .env file but there is no file present in repo.
so I would like to add the file in repo

also there is missing command for proper installing the requirements ( running in wrong directory) I would like to update these issues as soon as possible

need to update file structure and packages file , I have suggestions for those if may I?


adding grammar to the head of a class

@JimmyRojas @marcocdlv @ziur  Please check my code

I added some comments, and please add documentation for class member, methods, so on

@JimmyRojas @marcocdlv check , please my corrections

I added more comments, and please add documentation for class member, methods, so on (2)

@JimmyRojas @marcocdlv please, review my new changes

to be merged


Use TryFrom for trying to convert bytearray to Address or Hash

We can also update the test_env to use these generic types rather than Vec<u8> in storage.
https://github.com/Robbepop/pdsl/pull/54#discussion_r275410304
https://github.com/Robbepop/pdsl/pull/54#discussion_r275409958

This commit solves this:
https://github.com/Robbepop/pdsl/pull/54/commits/27635f837576e95dfe1828e6432cab39c6fd790c


Just FYI :)

Not 100% sure if this will cause issues but since I have both mods in my pack thought i would let you know.
[05:21:50] [main/ERROR] [FML]: Unable to parse dependency for mod 'wolfarmor' with dependency string 'after: SophisticatedWolves'. The modId 'SophisticatedWolves' must be all lowercase.

whoops
nice catch, thanks!


Field does not synchronize iteration on synchronized list

In Field.java:908, the synchronized list dependencyList is iterated in an unsynchronized manner,
but according to the Oracle Java 7 API specification,
this is not thread-safe and can lead to non-deterministic behavior.
This pull request adds a fix by synchronizing the iteration.

Created placeholder ticket https://issues.apache.org/jira/browse/VALIDATOR-434
Thanks for your pull request.

Merged. GitHub automation should kick in soon and close this pull request. Thanks @emopers !


Update thumbnails after import + Safer way to download thumbnails

it's ready to merge

@sissbruecker fixed all your comments, could you please check again

Thanks again!

Also added this: https://github.com/sissbruecker/linkding/pull/725


How to force player to use only native caption on iOS iPhone?

Have you read the Tutorials?
yes
Have you read the FAQ and checked for duplicate open issues?
yes
What version of Shaka Player are you using?
4.10.26
What browser and OS are you using?
Safari on iOS 17.5.1 (iPhone 12)
Please ask your question
I noticed that Shaka Player in fullscreen mode on iPhone displays captions in two modes simultaneously - natively and rendered by Shaka's text displayer. Just before the native caption is displayed, I can see the shaka-rendered subtitle flash for a moment.
More over, shaka-rendered subtitles are displayed only when I exit the fullscreen mode (native are hidden then).
Turning off shaka controls with
controls.setEnabledShakaControls(false);
does not help, the subtitles are rendered twice anyway.
How can I force the player to use only native captions rendering, or only rendering by Shaka TextDisplayer (not both in the same time)?
I use UITextDisplayer (configuration below), however with default SimpleTextDisplayer problem is the same.
player.configure({
        textDisplayFactory: function() {
            return new shaka.text.UITextDisplayer (video, document.getElementById('shaka-content')):
         }
});

If the text is rendered natively, we might have fed text tracks to the platform somehow.  UITextDisplayer should only be active when we are in control of text tracks at the player level, and they are not fed to the platform (through src= or TextTrack/VTTCue APIs).

I have the same issue, when in fullscreen mode on my iOS 17.5.1, there are two(three, sometimes) duplicated subtitles rendered. When leaving fullscreen mode, there is only one subtitle rendered as expected.
Here is my code:
    const tag = document.querySelector('video');
    shaka.polyfill.installAll();
    const shakaPlayer = new shaka.Player();
    const shakaConfig = {
      drm: {/** some FairPlay Setting **/}
    };
    // some network filters are skipped
    shakaPlayer.configure(shakaConfig);
    shakaPlayer.attach(tag);
    shakaPlayer.load('https://contents.pallycon.com/test-contents/tearsofsteel/cmaf/master.m3u8');

Could you please provide some advice about this?
(shaka version: v4.12.4)

@shaka-bot reopen


build(deps): Removed react-select and web-vitals as dependencies

Hi,
this PR removes obsolet dependencies, that were added but not used.
In particular the following dependencies are removed:

react-select
web-vitals

Feedback is welcomed.
Cheers,

FYI @jkowalski


platform: add Radxa ROCK 3C platform support

Documentation is missing.

@tingleby can you take a look at this?

@tingleby Can you take a look at this pr? I have tested it and it works!

Thanks for the contribution!


Expose more command line options via built-in ${OPTIONS} variable

The ${OPTIONS} variable was added in RF 5.0 (#4229) and in RF 7.1 we added console_width into it (#5200). Exposing more command line options via it is typically very easy, and I open this issue as a place where to propose what options to add. If there's a command line option you'd need, please add a comment and explain the use case. I promise to add all proposed options that have a reasonable use case unless there are bigger technical difficulties.

The command line options that are currently exposed via ${OPTIONS} are:

--include (${OPTIONS.include})
--exclude (${OPTIONS.exclude})
--skip (${OPTIONS.skip})
--skip-on-failure (${OPTIONS.skip_on_failure})
--console-width (${OPTIONS.console_width}, new in RF 7.1)

When we start robot framework, we do so via a shell script which comfortably helps us preselect a bunch of input variable values and argument files.
These values can vary by chosen tests directory, chosen test environment, chosen tags to include or exclude, etcetera.
With a diverse setup, this means it can become a little confusing whether the right tests were selected to be run. (Yes, the report is green, but are the right tests even run on the right test object on the right test environment?) Or when a test failed; what input variables did you use to get to this result?
Besides being able to search through shell command history or pipelines, I am rather interested to be able to have the test report contain which input variables values were passed to robot framework (which resulted in this, exactly this, test report). For instance by performing  'Log  ${OPTIONS}'.
For my usecase, I would like the OPTIONS variable to also contain:

the actual full command used to start robot framework
rpa or no-rpa
argumentfiles
variable and variablefiles
listeners
chosen test directory (path/to/tests.robot)

Any option of the above that can be added will be very helpful.

@pekkaklarck Please also expose --removekeywords and --flattenkeywords command line options, it's useful for listeners to skip reporting certain keywords.

Exposing --removekeywords and --flattenkeywords as ${OPTIONS.remove_keywords} and ${OPTIONS.flatten_keywords} sounds good to me.
Should we expose the exact values passed from the CLI, do some kind of normalization, or convert values to some kind of a match objects? Especially pattern matching when using something like --removekeywords 'name:* Keyword' would be easier if the value wasn't just a string name:* Keyword.


Not working inside of a Animated.Text

Thanks for the library, I love it.
My issue is, when I wrap the TextTicker component with an Animated.Text, it doesn't animate as we expected
<Animated.Text  numberOfLines={1} style={[styles.headerTitle, { opacity: nameOpacity }]}  >
   <TextTicker
      style={{ fontSize: 24, color: 'white' }}
      duration={3000}
      loop
      bounce
      repeatSpacer={50}
      marqueeDelay={1000}
    >
        Super long piece of text is long. The quick brown fox jumps over the lazy dog.
    </TextTicker>
</Animated.Text>

I think I have messed up before. It works fine. Closing so

I think I have messed up before. It works fine. Closing so


Elemental Adept benefiting incorrect spells

Latest version of UB1

Can I have a save? Saves (no pun intended) a lot of time on our end.

Multiplayer, 3 people, I'm the host (Xa).
Settings: Settings.zip
Map: Shadows over Brightreach.zip
Save: EA_bug.zip

it's not rerolling but there is this UI glitch I'll try to fix.

fixed in DEV. Thanks.


crashes when empty input

when you input empty name (you call just root url) it crashes. I would fix it but I dont know where it should best place to fix this one

Hi @Dominoo, we've fixed this in our fork: tripviss/image-resizer#1
Do check it out. We plan to continue maintaining it. :smile:


issue #518 #2218 is fixed

Issue is fixed overlaping, Creator - Web design #518 #2218
I have added the following file and resources
->modules\overlapblock\overlaping_518_2218_AbrarFaizMohammed.html
->modules\overlapblock\images_and_icons\overlaping_518_2218_AbrarFaizMohammed_payImg.png
->modules\overlapblock\images_and_icons\overlaping_518_2218_AbrarFaizMohammed_signal_Img.png
->modules\overlapblock\images_and_icons\overlaping_518_2218_AbrarFaizMohammed_trend_Img.png

Heartfelt Thanks for Your Contribution to Our Open Source Project!
Dear @AbrarFaizMohammed (Abrar Faiz Mohammed),
I trust this message finds you in good health and high spirits. I am reaching out with a heart full of gratitude for your recent contribution to our open source project. Your dedication has not only elevated the functionality of our project but has also exemplified the collaborative spirit that lies at the core of thriving open source communities.
A heartfelt congratulations are in order for your exceptional work! Your commitment and expertise have played a pivotal role in the success of our project, and we consider ourselves truly fortunate to count you among our community members.
I must extend my sincere apologies for the delay in responding to your pull request. Your patience and understanding during this time are immensely appreciated. As our project continues to grow, we acknowledge the imperative need to enhance the quality and response time for contributions. To address this, we've kickstarted a call for maintainers and reviewers who can help us uphold the high standards of our project and ensure swift responses to pull requests.
Your involvement and insights would be invaluable in this initiative. If you, or someone you know, is interested in taking on a more active role in the project, please don't hesitate to reach out. We firmly believe that with a robust team of maintainers and reviewers, we can elevate both the quality of the project and the speed of our responses.
On an exciting note, I am thrilled to share that we have activated the sponsorship button for our project. This development allows supporters like yourself to contribute to the sustainability and growth of the project. Your ongoing support, whether through code contributions or sponsorship, is genuinely appreciated.
Once again, congratulations on your outstanding contribution, and thank you for being an indispensable part of our open source community. We eagerly anticipate your continued collaboration and shared success.
Best regards,
Ntem Kenyor K.
Lead
Kwickerhub


Support trn_token in sign in journeys

We want to support 'magic links' that we've emailed to people inviting them to AYTQ. This adds support for the trn_token parameter, as with the current ID solution.
For now we're reaching directly into the ID database; normally we'd create APIs but this is temporary until we move all the tokens over from ID (at which point we'll likely need direct DB access any way). For testing this uses the EF in memory provider (setting up real DBs for testing this isn't worth it).
Much of this PR is refactoring to allow the Debug page to share some code in SignInJourneyHelper; TryApplyTrnToken is the meat of the changes.

 Support magic links in Teacher Auth


Make sure to call _super from init.

Without this, this.project is undefined because that is set in the supers init...

Looks like #118 took care of this.


Update to latest node-function-buildpack

This uses the latest release: https://github.com/heroku/node-function-buildpack/releases/tag/v0.4.3

This was superseded by #21.


Stopped working...

Hi,
from the site: https://for-bitcoin.com/calculator/
I get:
Warning: array_keys() expects parameter 1 to be array, null given in /var/www/for-bitcoin.com/calculator/index.php on line 117

Warning: file_get_contents(https://api.coindesk.com/v1/bpi/historical/close.json?start=&end=2019-07-24): failed to open stream: HTTP request failed! HTTP/1.1 404 Not Found in /var/www/for-bitcoin.com/calculator/index.php on line 120

Warning: array_merge(): Argument #1 is not an array in /var/www/for-bitcoin.com/calculator/index.php on line 124

Warning: Division by zero in /var/www/for-bitcoin.com/calculator/index.php on line 183

Warning: Division by zero in /var/www/for-bitcoin.com/calculator/index.php on line 183

I suppose, they changed the coindesk APIs.

Thanks for letting me know

Thanks for the awesome tool. I convinced many friends to join the bitcoin revolution simply showing how easy is to get valuable gains with little effort and capital.
I forked but I hadn't time to look into it (also, I'm not good with php)

Fixed the data file, became empty for some reason.


Can I export the task to the format I want?

Hello!
When I export coco json, I want to change the order a little bit or modify it...

@ikjoooon , unfortunately the description is very poor. You can export your tasks in formats which CVAT supports. You can add your own formats if it is necessary. I will close the issue as invalid. Feel free to reopen it with more details and better explanation.


WIP: Add icon

https://github.com/zulip/zulip/issues/19803
Testing plan:
Not tested
GIFs or screenshots:
Needs to be done

This is a small commit. If it is ok, other icons can be added and the commits squashed.

Closing as I'm happy with how the icons work on this page.


CI of azps.7.5.0

Docs Build status updates of commit 2675177:
:warning: Validation status: warnings

File
Status
Preview URL
Details

azps-7.5.0/Az.CognitiveServices/Get-AzCognitiveServicesAccountModel.md
:warning:Warning
View (azps-7.5.0)
Details

azps-7.5.0/toc.yml
:warning:Warning
View (azps-7.5.0)
Details

azps-7.5.0/Az.ADDomainServices/Get-AzADDomainService.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.ADDomainServices/Remove-AzADDomainService.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.ADDomainServices/Update-AzADDomainService.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.Aks/Get-AzAksNodePoolUpgradeProfile.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.Aks/Get-AzAksUpgradeProfile.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.Aks/Start-AzAksCluster.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.Aks/Stop-AzAksCluster.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.AppConfiguration/Get-AzAppConfigurationStore.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.AppConfiguration/New-AzAppConfigurationStoreKey.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.AppConfiguration/Remove-AzAppConfigurationStore.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.AppConfiguration/Update-AzAppConfigurationStore.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.ApplicationInsights/Get-AzApplicationInsightsWebTest.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.ApplicationInsights/Remove-AzApplicationInsightsWebTest.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.ApplicationInsights/Update-AzApplicationInsightsWebTestTag.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.Automation/Start-AzAutomationDscCompilationJob.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.BareMetal/Update-AzBareMetal.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.BotService/Get-AzBotService.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.BotService/Remove-AzBotService.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.BotService/Update-AzBotService.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.CloudService/Get-AzCloudService.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.CloudService/Get-AzCloudServiceNetworkInterface.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.CloudService/Get-AzCloudServiceOSFamily.md
:bulb:Suggestion
View (azps-7.5.0)
Details

azps-7.5.0/Az.CloudService/Get-AzCloudServiceOSVersion.md
:bulb:Suggestion
View (azps-7.5.0)
Details

This comment lists only the first 25 files in the pull request.

azps-7.5.0/Az.CognitiveServices/Get-AzCognitiveServicesAccountModel.md

Line 0, Column 0: [Warning: file-not-found - See documentation] Invalid file link: './New-AzCognitiveServicesAccountDeployment'.

azps-7.5.0/toc.yml

Line 4330, Column 11: [Warning: bookmark-not-found] Cannot find bookmark '#import/export' in 'azps-7.5.0/Az.ImportExport/index.yml', did you mean '#import-export'?

azps-7.5.0/Az.ADDomainServices/Get-AzADDomainService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAdDomainServicesIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.ADDomainServices/Remove-AzADDomainService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAdDomainServicesIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.ADDomainServices/Update-AzADDomainService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAdDomainServicesIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.Aks/Get-AzAksNodePoolUpgradeProfile.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAksIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.Aks/Get-AzAksUpgradeProfile.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAksIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.Aks/Start-AzAksCluster.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAksIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.Aks/Stop-AzAksCluster.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAksIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.AppConfiguration/Get-AzAppConfigurationStore.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAppConfigurationIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.AppConfiguration/New-AzAppConfigurationStoreKey.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAppConfigurationIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.AppConfiguration/Remove-AzAppConfigurationStore.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAppConfigurationIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.AppConfiguration/Update-AzAppConfigurationStore.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IAppConfigurationIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.ApplicationInsights/Get-AzApplicationInsightsWebTest.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IApplicationInsightsIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.ApplicationInsights/Remove-AzApplicationInsightsWebTest.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IApplicationInsightsIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.ApplicationInsights/Update-AzApplicationInsightsWebTestTag.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IApplicationInsightsIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.Automation/Start-AzAutomationDscCompilationJob.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'NodeName' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.BareMetal/Update-AzBareMetal.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IBareMetalIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.BotService/Get-AzBotService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IBotServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.BotService/Remove-AzBotService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IBotServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.BotService/Update-AzBotService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'IBotServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.CloudService/Get-AzCloudService.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.CloudService/Get-AzCloudServiceNetworkInterface.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.CloudService/Get-AzCloudServiceOSFamily.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

azps-7.5.0/Az.CloudService/Get-AzCloudServiceOSVersion.md

Line 0, Column 0: [Suggestion: disallowed-html-tag - See documentation] HTML tag 'ICloudServiceIdentity' isn't allowed.  Replace it with approved Markdown or escape the brackets if the content is a placeholder.

This comment lists only the first 25 errors (including error/warning/suggestion) in the pull request.
For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Tldraw embed crashes in incognito mode

I wanted to try the tldraw embed in incognito mode, but it gave the following error:

This bug only appears in incognito mode.

Thanks for the report! That's a good find. I'm seeing this in firefox 👍

This has been fixed internally and should be released next week! If you want to try it out, you could visit https://staging.tldraw.com and try refreshing a private firefox window.

I'm still seeing issues with this - we're taking another look at it!

Hey so part of the issue with private browsing in firefox is that refreshing the page doesn't allow the state of the canvas to be persisted, which was causing some issues - and could lead to lost work. So we're notifying the user about that now. Perhaps that is something that can be improved further down the line :)


R119 openmp

Refer to this link for build results (access rights to CI server needed):
/job/libnd4j_PR/38/


Can't send too little BB, like 1BB or 1KBB

When I tried to send 1BB or 1KBB to someone else, the client shows the following error message:
Could not send payment: all paying addresses are unfunded in asset
When I tried to send 1MBB, it works!
So first issue is that is 1BB or 1KBB allowed to be sent, or they must be more? If it is allowed, it should be a bug. Otherwise the error message should be adjusted to give more friendly information!

Most likely, your balance was still unconfirmed when you were making your first attempts to send blackbytes.  Please make sure your entire balance is confirmed, then try again.


ci(.github/workflows/): install all mise tools to share cache

GitHub Actions reaches the 10GB limit quickly, so install all tools for all steps to share the mise cache.


Don't duplicate input names in outputs

Pushed the test first so you can see the failure for yourself.


How to scale LXD daemon ?

Seems LXD run single process daemon. Is it possible run LXD in multi-thread or multi-process ?

LXD is written in Go and heavily uses go routines for parallelism so is very heavily multi-threaded.

@stgraber so, once use systemctrl to assign the dedicated CPU numbers or range to LXD, then LXD daemon will scale to multi-threading automatically  ?

Does "zpool import -d /var/lib/lxd" (directory where the .img file is in) work any better?


Feature/content regression tests

:tada: This PR is included in version 1.3.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Implement members section

This PR implements the members section

📝Changed routes:

/

(view full size)

Commit 8013840bb78b012cd01a658c1b05de1db664695c (https://floripamais-git-implement-members-section.laurabeatriserafim.vercel.app).


Can't export type

It's possible to export a class using CommonJS notation, but not possible to export a type:
class C {}
type t : {x : number, y : number};

exports.C = C;
exports.t = t;  // <--- Error here

type referenced from value position

Not sure if by design.

Note: I didn't successfully setup browserify with ES6 style modules. That's why I'm using CommonJS style modules.

Well, classes exist at runtime, types don't. You need to use export type for this

Yeah, @vkurchatkin is exactly right. If you want to export a type declared in module A for use in module B, you can do something like this:
a.js
export type T = string

b.js
import type { T } from "./a"
var x: T = 0; // error, number incompatible with string

Read more in the docs: https://flowtype.org/docs/modules.html#type-imports--exports
Reply here if this doesn't resolve your issue.


UpdateGatewayDatasource

Script created to update data source credential details

Code is incorrect, closing this one to submit a new one with the correct code


FEAT: Revoke Refresh Tokens on Session-Destroy

Closes (https://github.com/mazipan/tanyaaja/issues/94)
Description
Tested on Logout ✅
HTTP Requests
-------------

GET    /_next/static/media/TanyaAja.15bf8495.svg                      200 OK
GET    /_next/static/chunks/app/login/page.js                         200 OK
GET    /_next/static/webpack/webpack.22acd9070cb303c3.hot-update.js   200 OK
GET    /_next/static/webpack/22acd9070cb303c3.webpack.hot-update.json 200 OK
GET    /login                                                         200 OK
GET    /login                                                         200 OK
DELETE /api/private/user/session-destroy                              200 OK
GET    /api/private/user/by-uuid/***                                  200 OK

📝 I haven't checked the Delete User flow, but I guess we need to Revoke the Refresh token there as well.
References:

Revoke refresh tokens
Delete a user

I don't think the destroy session is necessary in the current flow.
Can you check, how if use previous token after doing logout?
Since we already doing logout from Firebase client side, I assume that the old token can not be verified anymore.
If it's safe, then we can remove the destroy session function completely.

I will help to check from my local.

@mazipan I checked it again and I guess both the master and this branch still has the Security issue.
Scenario:

User login successfully
User get JWT token
Now copy the JWT token
Use it to access private endpoints
All should be good
Now LOGOUT the user
Use the JWT token from step 3 to access private endpoints
Ideally, the request should Fail

Turns out it's bigger topic than I expected. To make it ideal, need to introduce two things:

Firebase Realtime Database
Firebase Security Rules

The approach is like documented here: Revoke refresh tokens

Rules

Revocation sample

Now when we retest common flow above, it will be in ideal state.
{{BASE_URL}}/api/private/question/by-uid/pagination/*** FirebaseAuthError: The Firebase ID token has been revoked.
    at /tanyaaja/node_modules/.pnpm/firebase-admin@11.11.0_encoding@0.1.13/node_modules/firebase-admin/lib/auth/base-auth.js:973:27
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async GET (webpack-internal:///(rsc)/./src/app/api/private/question/by-uid/pagination/[uid]/route.ts:22:34)
    at async /tanyaaja/node_modules/.pnpm/next@13.5.4_react-dom@18.2.0_react@18.2.0/node_modules/next/dist/compiled/next-server/app-route.runtime.dev.js:6:62361 {
  errorInfo: {
    code: 'auth/id-token-revoked',
    message: 'The Firebase ID token has been revoked.'
  },
  codePrefix: 'auth'
}

Fixed the issue 💪🏻
HTTP Requests
-------------

GET    /api/private/question/by-uid/pagination/***   500 Internal Server Error
DELETE /api/private/user/session-destroy.            200 OK
GET    /api/private/user/by-uuid/***                 200 OK
GET    /api/private/question/by-uid/pagination/***   200 OK

📝 Ideally, the response should be 401 Unauthorized instead of 500 Internal Server Error.
  } catch (error: any) {
    console.error(request.url, error)
    if (error.code === 'auth/id-token-revoked') {
      // Token has been revoked. Inform the user to reauthenticate or signOut() the user.
      return NextResponse.json(
        { message: 'Session has expired. Please log in again to continue.' },
        { status: 401 },
      )
    }

    return NextResponse.json(
      { message: 'Error while get question by uid' },
      { status: 500 },
    )
  }

But this will be a huge refactor, let's:


Add Chocolatey 0.10.15

Microsoft Reviewers: Open in CodeFlow

Sorry for the delay.  I appreciate the work to make this installer.  But our goal is to ensure we are installing from the source.  I would like the installer to come from Chocolatey.  Would you be willing to share this installer wrapper with Chocolatey so they can own it?
Feel free to reach out to me so we can discuss<EMAIL_ADDRESS>


Allow WYSWYG while editing the markdown in the notebooks

When editing the markdown sections in the notebooks in VS, there is no way to visualize the markdown while editing it. Will be nice to add this feature as the one exists today in Visual Studio Code.

@tarekgh  are you able to  add the screenshot of the feature in Vs code ?

Closing as support for notebooks in VS has been discontinued


Figure out optimal Dockerfile

Current setup uses a volume to transfer code over into the container
So when we run npm install on build, the modules get installed. But the volume then overrides the node_modules folder with nothing.
The temp solution for now is to run npm install locally.
Ideally we should not require this step.

 Add Node Modules as a volume in Dockerfile


api: enableProxyProtocol in ClientTrafficPolicy

Relates to https://github.com/envoyproxy/gateway/issues/1328

Will there be other fields in the future? if so, prefer to something like:
proxyProtocol:
  enable: true
  xxxx:
  yyy:

@zirain good question, the only thing I can think of is TLV, but unsure how common it is for end users to set it.
Also prefer implicit enable rather than explicit enable with a field

Should ctp reuse the same proxyprotocol like btp https://github.com/envoyproxy/gateway/commit/99b1642b3c1ff20ed8f82ffb0ea8b88108dbdf96 ?

@zirain good question, the only thing I can think of is TLV, but unsure how common it is for end users to set it. Also prefer implicit enable rather than explicit enable with a field

We can make the EnableProxyProtocol as an empty structure for better future-proofing.

hey @Xunzhuo we can't use confirm from btp which picks a specific version (v1 or v2), this is server side, which can interpret both

emissary seems to have a boolean knob too
https://www.getambassador.io/docs/emissary/latest/topics/running/ambassador-with-aws#recommended-configuration
so does contour
https://projectcontour.io/guides/proxy-proto/
I would prefer to start with bool before we introduce a struct because there is no good use case yet


Can't install nuget package

error:
The "MSBuild.Sdk.Extras 1.6.41" package has the "MSBuildSdk" package type, which is not supported by the "osu.Framework" project.
All versions above 1.5.4 gives this error when installing with package manager. But when i hardcoded it to csproj and global.json then all worked properly.
VS community: 15.8.2
Win 10 ver 1803

Same here, need to manually update global.json AND all project files...

Not sure why that particular error is happening, but it is the better to use the Sdk attribute with the global.json to control the verison. Unfortunately, NuGet doesn't support that yet.

same issue


Show full tracebacks when error message is empty

We suppress tracebacks when exceptions arise from within modal code by default. Typically this is because the exception represents some form of input validation or maybe a server-side exception where the details of the calling stack aren't relevant to the user. But occasionally, we raise exceptions with an empty message, which leaves the user with no actionable information at all. This PR simply activates the "verbose traceback" mode in those cases.
While the errors are probably still not user-debuggable, the user experience should be less bewildering, and we'll likely be able to resolve the issue faster when the user reaches out for support.

@prbot approve


Remove obsolete suppressions for Ubuntu 16.04 (Xenial)

Codecov Report

Merging #559 (157c774) into main (7b71aa5) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #559      +/-   ##
==========================================
- Coverage   25.08%   25.07%   -0.01%
==========================================
  Files         116      116
  Lines       28856    28856
  Branches     4686     4686
==========================================
- Hits         7238     7237       -1
  Misses      20650    20650
- Partials      968      969       +1

Flag
Coverage Δ

unittests
25.07% <ø> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/router_core/address_watch.c
73.86% <0.00%> (-1.14%)
:arrow_down:

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 7b71aa5...157c774. Read the comment docs.


Forwarding of motions didn't work

Problem: If a user (superadmin, admin or normal user) wants to forward a motion, nothing happens. Only an error messages appears in the console.
Images:

Reproduction:

Create two committees and grant both committees the permission to forward and receive motions fromt the other one
Add your account as a participant of each meeting (and/or admin).
open first meeting
go to motions and create a motion with simple workflow
go back to motions. open the three-dot-menu, select workflows and open simple workflow: check which status is needed to forward motion. to avoid mistakes select forwarding in each status
head back to motions menu and open the created motion
on the left side should be a button to forward your motion. click it and select in the appaering menu the other meeting. finalize by clicking forward
a success message appears at the bottom of the page, but nothing happens
open console and see the error messages listed above

What should happen:
Forwarding motions should be possible for every kind of user who has the permission to do it. By forwarding a motion, the motion should be forwarded to the desired meeting and it should appear a small information in the metadata of the motion when this happens.

see https://github.com/OpenSlides/openslides-backend/issues/1441  - fix will be done in backend.


Clusternator server should securely store DockerHub/GitHub tokens per project

as a developer I would like my private credentials stored securely on a server

Needs verification


Refactors test_routes as a separate module

This refactors the test_routes file as a module. Addresses Issue #246
Let me know what you think about the file names, whether a test should be moved, or if we should further break down files.
The test_resource_crud.py file (holds all CRUD tests rather than GET routes) is 550 lines long but doesn't have an unreasonable amount of tests. test_resource_retrieval is just under 200 and I think that is the second biggest file now.
.... I just googled CRUD to realize that the R is for Read, not Remove... So suggestions for a new name are appreciated too

I documented the remaining problems in a new issue. I'm not going to touch that issue. If someone comes along and wants to mess with them, great. If not, that issue serves more as documentation so if someone comes along they can have some background as to why these things remain as problems. I think we're good to merge this. Please open additional PRs or issues for remaining problems that were not addressed in this PR.


feat: add validator and handler for idp initiated flow

Which Problems Are Solved
Add support for idp initiated flow by redirecting to application endpoint when relayState is given with idp and domain to trigger sp-initiated flow using zitadel reserved scopes
RelayState is a query containing following fields -

redirect_uri: denoting where to redirect (this will be application endpoint which will trigger sp-initiated workflow)
idp_id: denoting which idp to trigger, used as urn: zitadel:iam:org:idp:id :{idp_id}
org_id: this is required to get domain be passed in as urn: zitadel:iam:org:domain:primary:{domain} when custom login policy is defined
client_id: this will be clientID of an OIDC application which is to be used for checking if redirect_uri is valid or not

How the Problems Are Solved

Validates if POST request is idp-initiated by checking relayState and its query parameters, query parameter validation is failed on following -

if any of the parameters is missing (all parameters are required)
if redirect_uri is not valid url
if idp_id does not belong to any idp
if client_id does not belong to valid OIDC application
if redirect_uri is not in application's RedirectURI
if  there is no primary domain for org org_id

if  validation failed then continue existing ExternalLogin flow

If validated, redirect to redirect_uri with idpID and domain for sp-initiated flow

Additional Changes
Additional Context
Tested with cloudflare zerotrust idp-initiated flow

Closes #8520
https://discord.com/channels/927474939156643850/1153218749260910622
https://discord.com/channels/927474939156643850/1279330055771918476

thanks for your contribution, we will have a look at it

Hey @abhirajranjan I checked your PR. Thanks for the effort, but I have some questions and think we should first settle on what to implement before we continue.
The SAMLResponse is not handled at all and it just uses the relayState to check for some specific encoded parameters and uses them to redirect the user agent.

The SAMLResponse is not handled at all and it just uses the relayState to check for some specific encoded parameters and uses them to redirect the user agent.

The main idea is to redirect user to initiate sp-initated flow, for that we need idpID and orgID. In SAMLResponse we get user email which can be used to get orgID, but there is no function that provide this, so we have to add orgID as well in parameters.
Is there any other approach which we can take to solve this problem ?

The SAMLResponse is not handled at all and it just uses the relayState to check for some specific encoded parameters and uses them to redirect the user agent.

The main idea is to redirect user to initiate sp-initated flow, for that we need idpID and orgID which is used as reserved scope in zitadel. In SAMLResponse we get user email which can be used to get orgID, but there is no function that provide this, so we have to add orgID as well in parameters.
This allows the regular sp-initiated flows while allowing for a more secure IdP-Initiated flow. Is there any other approach which we can take to solve this problem ?
PS: https://auth0.com/docs/authenticate/protocols/saml/saml-sso-integrations/configure-idp-initiated-saml-sign-on-to-oidc-apps is the motivation for this flow

Why should the IdP-initiated flow be more secure?
If the IdP-initiated flow would be properly handled (check the SAMLResponse and authenticate the user at ZITADEL) and only then redirect to the app, it would not need the orgID and idpID.
It could either be given an SAMLResponse for SAML apps as well or an id_token for OIDC apps (as Auth0) does.
In case of SAML the app could handle the IdP-initiated flow itself or OIDC apps can use the id_token to already extract information about the user or just use it as id_token_hint and start a regular code flow with it.
Regardless of the flow then executed, the user would already be authenticated at ZITADEL and does not even need the SSO session of the (external) SAML IdP

Hi @abhirajranjan
Have you seen the last comment from @livio-a ?
Do you still intend to have a look at this?


Add "issuer" and "id" fields to SNS notifications

These fields can help consumers of SNS notifications understand how to process each notification.

Coverage increased (+0.003%) to 60.753% when pulling b2afb2b38afa4350ea7431b5469dc3665f4b105f on jtschladen:add-issuer-id-sns-notification into 4459bbef650edfd32b6aa7bf2836ff0d1e751207 on Netflix:master.


How to get a selector without sharing

What is the best way to have one selector implementation and be able to choose to have a with sharing and a without sharing selector, with the minimal amount of code duplication.
I currently have something like this, but how do I implement this in AT4DX:
public interface IContactsSelector extend fflib_ISObjectSelector
{
    List<Contact> selectById(Set<Id> ids);
 }

pubic virtual inherited sharing class ContactsSelector extends fflib_SObjectSelector implements IContactsSelector
{
   public static IContactsSelector newInstance()
   {
       return (IContactsSelector) Application.Selector.newInstance(Schema.Contact.SObjectType);
   }
   public static IContactsSelector newWithoutSharingInstance()
   {
       return (IContactsSelector) Application.WithoutSharingSelector.newInstance(Schema.Contact.SObjectType);
   }
   public static IContactsSelector newWithSharingInstance()
   {
       return (IContactsSelector) Application.WithSharingSelector.newInstance(Schema.Contact.SObjectType);
   }
   public ContactsSelector()
   {
       super();
   }
   public ContactsSelector(Boolean includeFieldSetFields, Boolean enforceCRUD, Boolean enforceFLS)
   {
       super(includeFieldSetFields, enforceCRUD, enforceFLS);
   }
   public List<Contact> selectById(Set<Id> ids);
   {
      ...
   }

   public with sharing class WithSharing extends ContactsSelector
   {
       public WithSharing()
       {
           super(true, true, true);
       }
       public override List<Contact> selectById(Set<Id> idSet)
       {
           return super.selectById(idSet);
       }
   }
   public with sharing class WithoutSharing extends ContactsSelector
   {
       public WithSharing()
       {
           super(true, false, false);
       }
       public override List<Contact> selectById(Set<Id> idSet)
       {
           return super.selectById(idSet);
       }
   }
}

Typically, selectors should default to with sharing and have an inner class that allows elevation to without sharing.  This forces devs to call the elevated query explicitly instead of depending on the caller class sharing.  I prefer to start with the lowest level of access and only elevate if necessary.
I don't think Application.SelectorFactory should be updated to include With, Without, or Inherited Sharing variants.  These can be handled by exposing additional methods like the one below.
If you are set on using different instances of the selector for each variant, I suggest extending Application and implementing them yourself.
public class with sharing ContactSelector
    extends ApplicationSObjectSelector
    implements IContactSelector
{

  public Contact[] selectById(Set<Id> ids) {
      return (Contact[])selectSObjectById(ids);
  }

  public Contact[] selectByIdWithoutSharing(Set<Id> ids) {
      return new WithoutSharing().selectById(this, ids);
  }

  public class without sharing WithoutSharing {

      private WithoutSharing() {}

      public Contact[] selectById(IContactSelector selector, Set<Id> ids) {
          return selector.selectById(ids);
      }
  }

}

Hi @chazwatkins
Thank you for clarifying.
This code has been thoroughly vetted with my favorite text editor, so take it for what it's worth.  😇  However I think, based on my understanding, this is the first step that I'd take down the path that's been charted.
I've kept the two selector separated -- S in SOLID -- but one could put a static factory method in ContactSelector if they felt strongly about it.  With this, the consuming business logic would be free of extraneous code, as typical, and the unit test classes would still simply provide a mock selector to the Application, as typical when needed.
Please give this a whirl in a project, where you probably have some robust testing setup.  I'm genuinely interested if this is going in the right direction and not broken in some way.
Regards!
public interface IContactSelector {
	Contact[] selectById(Set<Id> ids);
}

public virtual with sharing ContactSelector
	extends ApplicationSObjectSelector
	implements IContactSelector
{
	public static IContactSelector newInstance() {
		return (IContactSelector) Application.Selector.newInstance(Contact.SObjectType);
	}

	public virtual Contact[] selectById(Set<Id> ids) {
		return (Contact[]) Database.query(...);
	}
}

public without sharing ContactSelectorWithoutSharing
	extends ContactSelector
{
	public static IContactSelector newInstance() {
		return new ContactSelectorWithoutSharing();
	}

	private IContactSelector baseSelector = Application.Selector.newInstance(Contact.SObjectType);

	public override Contact[] selectById(Set<Id> ids) {
		return baseSelector.selectById(ids);
	}
}

Hi @stohn777 ,
Thanks for the response.  I agree that the inner-class approach is not the cleanest solution.  I believe most folks, including myself, discover this approach in the blog post Apex Sharing and applying to Apex Enterprise Patterns.
The only problem with the approach you outlined is that you're forced to call ContactSelector.newInstance() or ContactSelectorWithoutSharing.newInstance().  The calling class must know about the selector implementation class instead of using the application selector factory to return the implementation.  Which defeats the purpose of having the selector factory.
I believe this is why @wimvelzeboer went the way of extending the selector factory to include sharing variants.
Another approach could be to allow the caller to specify the sharing rule within which the query would be executed.

The selector query methods would no longer directly call Database.query(...) or Database.getQueryLocator(...).  Rather, they would pass in a query factory to something like:

ApplicationSObjectSelector.select(fflib_QueryFactory qf)
ApplicationSObjectSelector.getQueryLocator(fflib_QueryFactory qf)

These methods would look at the current sharing rule and call the queries within the corresponding class:

ApplicationSObjectSelector.WithSharing
ApplicationSObjectSelector.WithoutSharing
ApplicationSObjectSelector.InheritedSharing

Doing so will allow the same query method can be reused for all the sharing rules, so duplication of method definitions is no longer necessary.
I'm happy to implement it if the team is interested in adding this.  fflib-apex-common might be a better home for setting the selector sharing rule since ApplicationSObjectSelector inherits from fflib_SObjectSelector.
What do you think?
BTW - You left me in suspense.  What is your favorite text editor?

This code has been thoroughly vetted with my favorite text editor, so take it for what it's worth. 😇 ...


Jobs with status MaintenanceWindowExceeded not shown in Log Analytics

Hi,
I've noted that I am not able to find jobs with "MaintenanceWindowExceeded", even though I see them in this way in the Azure Portal, when I do distinct on the UpdateRunProgress by the InstallationStaatus over the last 7 days:
UpdateRunProgress
| distinct InstallationStatus
all I got is:
NotStarted
NotIncluded
Succeeded
Download Failed
Install Failed
However I got some jobs visible in the portal as "MaintenanceWindowExceeded" from 16.01 and 18.01- is there something I am doing wrong in the KQL?

[Enter feedback here]

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

ID: 478e860a-46f6-3e08-4138-8f6d8c8de57c
Version Independent ID: 69e9f9ea-614d-7822-c9c8-430bf1d7c83a
Content: Query Azure Automation Update Management logs
Content Source: articles/automation/update-management/query-logs.md
Service: automation
Sub-service: update-management
GitHub Login: @SGSneha
Microsoft Alias: v-ssudhir

@zaicnupagadi Thanks for your comment! We will review and provide an update as appropriate.

@zaicnupagadi This needs deeper investigation so please reach Azure support for quick resolution. In that regards, here are the steps to raise a support ticket with Azure technical support team if you already have a support plan. But if you don't have a support plan, then please send mail to<EMAIL_ADDRESS>with your subscription ID and a link of this question for context along with subject being "Attn:Krishna" and we will assist you with engaging Azure technical support in a quick way.
Also, just FYI, this azure-docs repository deals about feedback related to particular Azure document page (like correcting doc bugs, doc enhancements, product issues related to wrong doc instructions, etc.) so going forward, if the issue is intended to update / correct / enhance anything in an Azure document then please share that kind of feedback here else if the issue is related to Azure product and not the document then try to leverage Microsoft Q&A forum.


Label and screen shot have changed

The manage multiple workspaces  is now called "View Incidents" - the screen shot is out of date.
Regards,
MG

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

ID: 294ac28e-49b0-0ba7-d894-8e3d8b20a7ba
Version Independent ID: dc4cb3a3-d712-824f-f49c-02a79ceb9eee
Content: Work with Azure Sentinel incidents in many workspaces at once
Content Source: articles/sentinel/multiple-workspace-view.md
Service: azure-sentinel
Sub-service: azure-sentinel
GitHub Login: @yelevin
Microsoft Alias: yelevin

Hello @MGodfrey50 , thanks for taking time to share your valuable feedback! We are taking a look into this and will get back to you soon.

#please-close


GUI - create pipeline need to change the concurrent Pipelines to object

HKube micro-service
To which micro-service the bug related to, Dashboard ,Monitor-Server etc.
Describe the bug
the concurrentPipeLines was change from number to object need to change the UI to support it
 "options": {
    "ttl": 3600,
    "batchTolerance": 80,
    "concurrentPipelines": {
      "amount": 1,
      "rejectOnFailure": true
    },
    "progressVerbosityLevel": "info"
  },

Expected behavior
A clear and concise description of what you expected to happen.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Fixed on v1.2.41

tested on 1.2.41


libvirt is too old for qemu in unstable

Currently, qemu version is 4.2.0 and libvirt is 5.4.0. As I've already mentioned here there are some incompatibilities between qemu >= 4.1 and libvirt < 5.6.0. In some cases it may significantly affect performace (e.g. typical windows vm with vfio).
The latest libvirt version is 6.0.0. So, I think it's time to upgrade.
Unfortunately, it seems that something has changed in libvirt build process and just changing version/hash in existing derivation is not enough.
/cc @fpletz @globin (libvirt maintainers)

Turns out Nix was adding --disable-dependency-tracking flag which caused weird build failures.
After adding dontAddDisableDepTrack = true and some other small tweaks, it builds.
I'll try to make a PR myself this week, but I won't be able to test it on Darwin.
(apparently autotools will be replaced with meson soon in libvirt)

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/nixos-20-03-beta/5935/22


Use native ARM builders for container builds

What does this PR do?
 Unsure if this'll work but I figure it's worth a shot.

nice! Do we need any changes to the metadata tagging to ensure that the manifest gets tagged with the version number during release?

I don't think so but let me reconfirm and make a PR if that's broke.


[BUG][WEBJS] - container not start

Now I can't start the container, running until some hours ago.
docker run -it -p 4000:3000/tcp devlikeapro/waha-plus
yarn run v1.22.19
$ node dist/main
NODE - Catching unhandled exceptions enabled
WAHA (WhatsApp HTTP API) - Running PLUS version...
Segmentation fault (core dumped)
error Command failed with exit code 139.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
Version
latest

Hi! Looking at this, releseard 2024.6.5 exactly few hours ago.
meanwhile, you can keep running old version - use devlikeapro/waha-plus:latest-2024.6.4 image
What OS do you use?

Is it the full command, right? Just wanna test it in the whole context

thank you for the feedback!


Switch postgres operator deployment to helm

Description
This PR syncs cluster documentation with actual cluster status relatively to the deployment of the PostgreSQL operator.

/merge


Drafting: Citizen science

A page linked from the menu describing Citizen science in GBIF.
Blocked by #82

Mine. Will highlight present understanding of #citsci contributions to GBIF-mediated data and (more importantly) direct individuals interested in sharing data to possible solutions (iNat/iNat projects, national platforms, etc.)

First draft live. Needs review and refinement, from y.t. and others…


Register v0.2.2

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/1048
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if Julia TagBot is installed, or can be done manually through the github interface, or via:
git tag -a v0.2.2 -m "<description of version>" 4d5ba3a84a5278901673b90611504449d3a2b63d
git push origin v0.2.2

Done


Is PassFault compatible with java 1.6

Hi Team,
we want to use this library in our application so could dou please let us know Is is it compatible with java 1.6.

HI Team,
Could you please update on this issue?

No.  About two years back I refactored to use Java 8 language features, so down compiling won't be possible. And we've added enough useful features since then that I wouldn't recommend going back to that commit and building from there.


Membership page content

Summary

We're missing the content for the Membership page under Community.

This doesn't seem relevant any more, although there is content to refactor regarding the 'Team' page.


Carry the compiled datatype through to python

Needs a theory_dtype and a mocks_dtype in Corrfunc/__init__.py and corresponding sed within the Makefiles in the two python_bindings directories. Similar to the ones I have already in place for installing the headers.

Then the appropriate datatype for the compiled libraries can always be detected and arrays converted appropriately.

Does not need to be handled anymore. The python extensions will automatically convert to the correct type for the underlying API (see commit 9b73a5a). However, a little more coding will be required to handle both float32 and float64 simultaneously and dispatching to the appropriate function. That will be resolved with issue #64, presumably with release of v2.0.


Macro: The full screen button is in a place that doesn't match the design

Steps to Reproduce

Switch to Macromolecule mode
Look at right bottom of application

Actual behavior
The full screen button is in a place that doesn't match the design

Expected behavior
The full screen button is in a place that match the design

Desktop (please complete the following information):

OS: Windows 10
Browser Chrome
Version 112.0.5615.138 (Official Build) (64-bit)

Ketcher version
Version 2.15.0-rc.4

Tested.


JWT issuance with ssh-agent

#158 adds ssh-agent integration for issuing verifiable credentials and verifiable presentations with proof objects, using ssh-agent for signing. It should be possible to also use ssh-agent signing to create JWT VCs/VPs. This will depend on ssi exposing support for external JWT signing: https://github.com/spruceid/ssi/issues/221.

And ZCaps?


Give a more helpful error on no-op updates

Prior to this commit, an update with no changes would generate invalid
SQL which would result in a database error. Even if the resulting error
were handled, the transaction would be aborted at that point, which may
not be desired. With this change, we provide a more useful error message
and don't risk aborting the current transaction.

Review? @diesel-rs/contributors

CI failure is due to what appears to be a bug in the latest nightly