Remove deprecated rustc-serialize crate

The use of the deprecated rustc-serialize crate is preventing users from using various compile targets, specifically wasm32-unknown-unknown which is crucial for users looking to compile for browsers / nodejs.

@BelfordZ does #107 fix your issue? rustc-serialize is still present as a dependency of rust-crypto. If that's still a problem perhaps we should prioritize moving away from rust-crypto.

You still depend on rustc-serialize via rust-crypto, so probably the best approach will be to follow #98.

Defs getting there! there is the transitive dependency still, though since we are only using this crate for the bip39.

I think moving away from rust-crypto is going to slip for 0.14.0. I'd like to have a new release soon but fixing the crypto deps is pretty invasive -- there's a lot of inefficient stuff related to hashing (like the Transaction impl of BitcoinHash serializing the whole tx into a new Vec before hashing) that should be cleaned up at the same time.
I have code that does this locally, but it needs to be cleaned up and there's a bunch of supporting work to get in first.

I'm also looking for support for wasm as a compile target. If I understand this discussion correctly the decision has been made to solve this by moving to the bitcoin_hashes library.
Is there a branch where I can follow the progress of this or make a contribution to speed this up? @apoelstra

The existing set of maintainers needs to finish the conference circuit that they are all on and approve the open PRs. There isn't anything that other contributors can do to speed it up, sorry.

Addressed by #215


Unable to use with authenticated proxy

Hello there,
I'm trying to make my requests using an authenticated proxy but I'm unable to make it work.
The formats that I've tried:
http://username:password@ip:port
http://ip:port:username:password
username:password@ip:port
ip:port:username:password

Here's the output that I'm getting (regardless of the format used):
my_server_ip vs http://username:password@ip:port
Proxy not working properly.
Failed to load the search engine: load_search_engine()

Any ideas on how I can use an authenticated proxy with se-scraper?
Thanks,
Nicolas

will fix this tomorrow

any update on that? I tried to set prop proxy in the config object and I am getting the same result

It seems this is currently not possible to use socks5 proxy auth with chromium. Here is a detailed explanation.
As  a workaround you can use a local forwarding proxy, or you can just use a proxy server without authentication (and authenticated by IP filtering as I am doing it).
This is how you pass the flag:

const puppeteer = require('puppeteer');

(async () => {
    const browser = await puppeteer.launch({
        args: [
            // SET PROXY HERE
            '--proxy-server=socks5://IP:PORT',
            '--disable-infobars',
            '--window-position=0,0',
            '--ignore-certifcate-errors',
            '--ignore-certifcate-errors-spki-list',
            '--disable-setuid-sandbox',
            '--disable-dev-shm-usage',
            '--disable-accelerated-2d-canvas',
            '--disable-gpu',
            '--window-size=1920x1080',
            '--hide-scrollbars',
            '--disable-notifications',
            '--no-sandbox',
            '--user-agent=Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Ubuntu Chromium/71.0.3578.98 Chrome/71.0.3578.98 Safari/537.36'
        ],
        headless: true
    });
    var page = await browser.newPage();
    await page.setViewport({width: 1920, height: 926});
    await page.goto('http://ipinfo.io/json');
    console.log(await page.content());
    await browser.close();
})();

@NikolaiT are you able to help us how to set up local proxy? I have auth based proxy that I need to use but having trouble setting it up


Fix #1369 - setup-remote.md instructions

Fixes #1369 - "setup-remote.md instructions around network config file are confusing"
Now the instrs are 1:1 with draft blog post

The CI jobs keep failing, despite trivial changes. I reported it in #1374. Re ran 2 times, both also failed.
@calina-c suggestions on how to proceed?

The failures are due to multichain Provider being default now on barge, but we haven't yet merged #1265. We had a successful CI there, but we are now re-running just to make sure everything is all right. After merging it, I'll rebase the changes into this branch.

@calina-c thanks for merging from trunk!
Now tests pass :)
Could you review please?


Elegantly handle when a user enters the /r/ in a Reddit feed

It's commonplace to think of a subreddit as /r/ItsName or r/ItsName. If a user enters that in the Add Reddit feed dialog, we choke on it and throw an error. I know we show it in the assistive text below the input, but it would be much more elegant to just roll with it and eliminate the need for the low-contrast text entirely.

If I need to write something like this in the docs, it's a sign we're handing the experience poorly:

To subscribe to a single subreddit, choose Subreddit, then enter its name. Don’t include the /r/ prefix.

And that's only going to help the people who actually get around to reading the docs. :-)


Add Angular Paginator Component

Angular's Paginator Component (https://material.angular.io/components/paginator/overview) is can be used separately from the table component.
So we should probably create the component separate since it I can definitely see use cases for a paginator that's not part of a table.
In addition the paginator can be probably declared separate of the mesop table component. The paginator event could update the the dataframe for pagination, which would then be passed to the table component.
This would require that the table component have a parameter to know what slice of the data table to show.
Alternatively, we may want to embed the paginator within the table component as well. I think this option only makes sense if the API for specifying the dataframe slice is not intuitive and easy to understand.

I think it would be good to have some examples of the API for using paginator w/ the table, if it's a separate component. I'm a little concerned it would add some boilerplate for common use cases.
For example, it's quite common to have large dataframes (e.g. thousands of rows) where pagination is almost always desirable. Colab has a dataframe table component which provides pagination and is quite handy. IMO, it would be nice where something like me.table(df) would automatically do pagination (e.g. 25 rows/per page and the user can select different options for page sizes, e.g. 25, 50, 100).
Also, we could support having both paginator embedded within table (for this default/common use case) and having a separate paginator component.

That's a good point about it being more user friendly if the paginator was embedded in the table for the common use case.
On a side note I'm planning to write up a quick design doc for the detailing the rest of the table API since I think to implement the sticky columns and column styling stuff will require some design decisions that may affect other feature.

Hello, I’d like to work on this feature.

Hi Kamron. Sure thing. I've assigned you to the task.
To get started here are some helpful links:

https://google.github.io/mesop/internal/contributing/
https://google.github.io/mesop/internal/development/

We also have a Github Codespaces set up that you can try. I'm still working on the instructions for that.
But mainly it involves clicking on the create codespaces button seen on this screenshot.

Then you'll need to wait for the set up to complete, which takes a long time (~15 minutes). That step below needs to complete before you can run mesop for development.

Forgot to mention, we also have a tool that generates the Scaffolding for components, which you can see here: https://google.github.io/mesop/internal/development/


DPL-nnn Cardinal - Pooling samples within a plate [WIP]

User story
As a Cardinal pipeline user, I need to pool together samples on the LCA Chromium Connect PCR XP plate, into a tube. The wells to be pooled, and the volumes coming from each source well will be set based on the information I upload in a file.
Who are the primary contacts for this story
Stef L
Acceptance criteria

Output labware is a tube called "LCA Custom Pool"
Input file format is a csv with headers "Source Well", "Volume to add to pool", and "Dest. well", as in Duplex-seq pipeline

Dependencies
This story is blocked by the following dependencies:

#<issue_no.>
sanger/#<issue_no.>

References
This story has a non-blocking relationship with:

#<issue_no.>
sanger/#<issue_no.>

Additional context
Add any other context or screenshots about the feature request here.

Existing custom pooling functionality in Limber should work for this, although not sure about volume to transfer.

research label added for same reason as on https://github.com/sanger/limber/issues/769


Bulk upload cycle count

What is the motivation for adding/enhancing this feature?

Add the option to bulk upload cycle counts.
https://docs.google.com/document/d/13qZAwvm0-3ioKD55t4UA2Df9zXRwwYpuPh1UdyeaSjo/edit?usp=sharing

Verified in release v2.1.0. Issues found are separately raised. Hence, closing this ticket.


delete button not working

can somebody help to add a search button share button meanwhile delete button not working

@charles935 you can refer here : delete post problem #3
https://github.com/rozdoum/social-app-android/issues/3

thanks bro that worked 👍
may god bless you :*)


Add ABAP Language Version to Repo Settings

Enhance UI to select ABAP Language Version (only visible if ALAV experimental feature is enabled)
Repo Settings:

Ref #6298

good to merge
however, suggest adding a link to some documentation describing how it exactly works/what is the impact when changing these settings, seen from a user perspective

+1 for docs, I didn't quite following all the discussions on this and would be curious if you can "mix" the ABAP Language Version (ALV? ALaV?) with subpackages. I. e. a repo that works onpremise and in the cloud but has onpremise or cloud specific subpackages that only get pulled where supported. Or if that requires separate repos / some kind of branching strategy.

but has onpremise or cloud specific subpackages that only get pulled where supported

I'd recommend against such a setup. Git does always try to sync local and remote, and if having such a package setup, it will by design never be in sync.
do a big package that is compatible with the target versions, and some smaller separate packages if something is version specific

We will certainly document it. I will change the "Undefined" option to "Any (Object-specific ABAP Language Version)". This matches the current abapGit: Each object serializes/deserializes its own ABAP Language Version. We can continue to build "big packages" as Lars suggested.
The other settings come into play when a project is running in a specific environment, only. For example, when you develop something for BTP (ABAP Cloud) using released classes that don't exist on-prem. Or the other way, you use classic features (Standard ABAP) that will never run in ABAP Cloud. Making this explicit in the metadata will also allow searching for/filtering of projects and avoid "try pulling it in a system to see if it will work" scenarios.

That also means from an end user / developer perspective 3 repos for an application targeting both onprem and cloud? 3 repos to update, keep in sync, manage cross repo PRs etc.?

Main repo, needs to work in all environments -> language version ABAP for Cloud Development (?) without usage of cloud specific APIs
OnPremise "addon" repo -> language version Standard
Cloud "addon" repo -> language version ABAP for Cloud Development

I'd be interested to know if there are better / more convenient strategies, though this is not the right place to discuss this :)

its not the right place :) lemme continue
for a setup like https://blogs.sap.com/2023/09/11/proof-of-concept-migrating-segw-project-to-abap-cloud/, there would only be "1. Main Repo" and no 2 and 3, they would be manual scaffolding like https://github.com/abap2UI5/abap2UI5#installation
I'd recommend having as much code as possible in 1, and very small 2 and 3's if needed. This does require compromises, but the other alternative is double maintenance which is worse than a lot of things


Feature/scraper tests

Soup_test.py contains code to pull information from glossary to be used by API to store in eventual database. Alter to store strings in database after established.

Update: bug found https://app.clickup.com/t/86877vvmz


restore smart wallets page missing from crowdin

I can't find these text messages for translation in Crowdin
(see next comment)

Fixed via https://github.com/Chia-Network/chia-blockchain-gui/pull/88


opening/not opening flow triggers git change

Current Behavior
(Not) opening a in combination with another changes triggers a group change. Even if you've never opened the group. See the gif below for more detail:
I think this is related with issue: https://github.com/node-red/node-red/issues/3306

Expected Behavior
I expect that there are no group changes since I did not touch them
Steps To Reproduce

create a flow with a group in it.
create another flow with at least one node.
deploy and commit everything.
refresh the page on flow 2.
make some changes in flow 2 without opening flow 1.
deploy
In the local git changes you will see that the group object in flow 1 is changed even though we did not touch it.

Example flow
[
    {
        "id": "374449399dcc02ac",
        "type": "tab",
        "label": "Flow 1",
        "disabled": false,
        "info": "",
        "env": []
    },
    {
        "id": "1948f31bdffb40c1",
        "type": "tab",
        "label": "Flow 2",
        "disabled": false,
        "info": "",
        "env": []
    },
    {
        "id": "c26b1cf5f69d12f4",
        "type": "group",
        "z": "374449399dcc02ac",
        "name": "test",
        "style": {
            "label": true
        },
        "nodes": [
            "e8f5f0a819345a2d",
            "ddb867691c46787a"
        ],
        "x": 34,
        "y": 59
    },
    {
        "id": "e8f5f0a819345a2d",
        "type": "inject",
        "z": "374449399dcc02ac",
        "g": "c26b1cf5f69d12f4",
        "name": "",
        "props": [
            {
                "p": "payload"
            },
            {
                "p": "topic",
                "vt": "str"
            }
        ],
        "repeat": "",
        "crontab": "",
        "once": false,
        "onceDelay": 0.1,
        "topic": "",
        "payload": "",
        "payloadType": "date",
        "x": 140,
        "y": 100,
        "wires": [
            [
                "ddb867691c46787a"
            ]
        ]
    },
    {
        "id": "ddb867691c46787a",
        "type": "debug",
        "z": "374449399dcc02ac",
        "g": "c26b1cf5f69d12f4",
        "name": "",
        "active": true,
        "tosidebar": true,
        "console": false,
        "tostatus": false,
        "complete": "false",
        "statusVal": "",
        "statusType": "auto",
        "x": 350,
        "y": 100,
        "wires": []
    },
    {
        "id": "1532b66c984cbd0f",
        "type": "inject",
        "z": "1948f31bdffb40c1",
        "name": "",
        "props": [
            {
                "p": "payload"
            },
            {
                "p": "topic",
                "vt": "str"
            }
        ],
        "repeat": "",
        "crontab": "",
        "once": false,
        "onceDelay": 0.1,
        "topic": "",
        "payload": "",
        "payloadType": "date",
        "x": 220,
        "y": 100,
        "wires": [
            []
        ]
    }
]```

### Environment

- Node-RED version: 3.0.2
- Node.js version: v18.18.0
- npm version: 9.8.1
- Platform/OS: Windows & Linux ubuntu
- Browser: chrome

Hi, yes this was an issue with 3.0.2. I don't recall exactly which release fixed it, but the latest is 3.1.6.


strings.xml files on Windows are saved with Windows 1252 Encoding instead of UTF-8

Example Output from file saved with Windows:
<resources xmlns:xliff="urn:oasis:names:tc:xliff:document:1.2">
    <string name="BUTTON_TITLE">Hallo</string>
    <!-- General -->
    <string name="BTN_CONTINUE">Weiter</string>
    <string name="BTN_DONE">Fertig</string>
    <string name="BTN_SKIP">Ãœberspringen</string>

Same Output when file is opened manually with UTF-8 encoding:
<resources xmlns:xliff="urn:oasis:names:tc:xliff:document:1.2">
    <string name="BUTTON_TITLE">Hallo</string>
    <!-- General -->
    <string name="BTN_CONTINUE">Weiter</string>
    <string name="BTN_DONE">Fertig</string>
    <string name="BTN_SKIP">Überspringen</string>

This causes multiple issues:

Git merges can break
Editor tools can't open the file properly because the encoding in the XML header is set as UTF-8
Strings can get misinterpreted in the apps

To fix this, the encoding probably needs to be explicitly set to UTF-8 when writing to the file, in order to prevent platform differences from causing issues. It is not clear at this point whether the issue is caused by simply running the tool on Windows, or subsequently, for example by opening the file in some editor.

Should have been solved with #4


Update Composer dependencies (2022-02-06-00-03)

Visual regression test passed!
View the visual regression test report


Amend identify needs (Additional information)

What does this pull request do?
Allow PP to modify the additional information (Identify needs)
What is the intent behind these changes?
Allows a PP to modify details of a referral which has been submitted, preventing them having to close and make a new referral. This change is for modifying the additional information

cypress/downloads/downloads.html is added, do we really need this?

cypress/downloads/downloads.html is added, do we really need this?

No idea where that came from, removed thanks!


Datascience with python : Correlation coefficients

Description 📜
Detailed Explanation of Correlation coefficients with examples and jupyter notebook
Added README.md
Added Requirements.txt
Added Datasets
Fixes #5846

Type of change 📝

[ ] Audio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Video (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[x] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[ ] Interview #Prep
[ ] Java_#Domain
[ ] JavaScript_#Domain
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

kindly review the PR.
@prathimacode-hub

I have done the required Changes and added  purpose, why correlation coefficients are used, data visualizations, advantages,and  applications in README.md.
kindly review the PR.
@prathimacode-hub


FreeBSD CI does not work as expected

Recently, FreeBSD CI was fixed, but there are some rough edges.
I first got a flaky (I think) test, but all I saw in the CI output was that it failed. Only after scrolling through all the tests was I able to find the actual error, so we need better error reporting:
 ---- test_tail::test_follow_name_truncate4 stdout ----
current_directory_resolved:
write(append): /tmp/.tmpSLjjyw/file
run: /Users/runner/work/coreutils/coreutils/target/debug/coreutils tail -s.1 --max-unchanged-stats=1 -F file
write(truncate): /tmp/.tmpSLjjyw/file
thread 'test_tail::test_follow_name_truncate4' panicked at 'assertion failed: buf_stderr.is_empty()', tests/by-util/test_tail.rs:1985:9

https://github.com/uutils/coreutils/actions/runs/3503280450
Then I reran the test and got a lot of "file has vanished" and at the end
rsync warning: some files vanished before they could be transferred (code 24) at /AppleInternal/Library/BuildRoots/a0876c02-1788-11ed-b9c4-96898e02b808/Library/Caches/com.apple.xbs/Sources/rsync/rsync/main.c(1404) [generator=2.6.9]

https://github.com/uutils/coreutils/actions/runs/3503280450/jobs/5868960733

Check the https://github.com/uutils/coreutils/actions/runs/3503280450/jobs/5868960733 raw log.
I saw the ssh_dispatch_run_fatal: Connection to <IP_ADDRESS> port 2222: message authentication code incorrect which indicates the SSH is disconnecting while the test is still running.
And the file has vanished: /root/work/coreutils/coreutils/target/* is because the files were cleaned by cargo clean while the rsync was copying files.
The issue might be related to the broken SSH connection.

Not my area of expertise to be honest. If you think it would work we can give it a shot!

Would you like to add this commit to your PR?
I have checked that the FreeBSD CI passed. (https://github.com/miles170/coreutils/actions/runs/3506882167/jobs/5874237565)
Check similar issue here: https://github.com/vmactions/freebsd-vm/issues/28

I'd prefer to fix this in a separate PR. Could you open one?


Any plans for gesture support?

I don't set any due date for it.
If I do,

js facade gen from https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/hammerjs/hammerjs.d.ts
Port gesture code.
Integration testing.

AngularDart actually has the code supporting HammerGestures but AFAIK it is not promoted. I'm curious if anyone is using Hammer with AngualrDart in production.


Grace period preview times don't match due date

When creating a new assignment:

Set a due date.
Choose the grace period submission rule
Enter any number of hours, and then change focus (e.g., click "add another period")

The previewed grace period times don't match the due date. I have a feeling this is a time zone issue.

It does in fact seem to be a time zone issue, since it's 5 hours off (GMT vs. EST). I thought I squashed these issues but I guess not. :(

Just tested this and it seem to work now.


Dynamic difficulty

We need to determine how difficulty is adjusted over time. A possible implementation would keep a rolling 1-day window of the number of blocks produced in one day and use that to adjust the difficulty.
Deliverable: Formula for adjust difficulty over time

In the process of implementing the pow contract, I came up with the following algorithm:
block_window_time = ( block_window_time * 8639 ) / 8640; // Decay one interval
block_window_time += current_block_time - last_block_time; // Add current interval
average_block_interval_ms = block_window_time * 1000 / 8640; // Get block interval average in ms
block_time_diff = average_block_interval_ms - 10000; // Get difference between average and desired block interval

if ( current_block_time / 600 > last_block_time / 600 ) // Update block interval once per minute
{
   current_difficulty += ( current_difficulty * block_time_diff ) / 10000;
}

The last two calculations are the most important. If the block interval is too long, block_diff_time is positive. If it is too short, block_time_diff is negative. Once per minute, the difficulty is updated according to how far off the average block interval is. If blocks are being produced twice as fast, then the difficulty target will be halved. Likewise, if the block interval is half as much, the difficulty target with be doubled.


EPIC: Fruity home page

This ticket is for reference purposes, gathering together all of the work related to the Fruity program page.

[ ] #547 Visual design refinement
[ ] Scaffold
[ ] #544 Site map and country filter
[ ] #542 Queries for country-level M&E metrics
[ ] #463 Queries for program-level metrics
[ ] #551 Program blob
[ ] #552 Program empty state/s
[ ] #549 Chart to visually represent % of program reporting period completed to date
[ ] Help pop-up: How are these metrics calculated?
[ ] Pull program status from GAIT
[ ] Filter programs by GAIT status

What is the "program blob?"

On the home page, the entire cluster of program stuff that repeats for
every program. Thought it maybe useful to create a ticket to track every
piece of it.  Maybe not?
On Thu, Sep 6, 2018 at 5:04 PM PJ Souders<EMAIL_ADDRESS>wrote:

What is the "program blob?"
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mercycorps/TolaActivity/issues/180#issuecomment-419150006,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AgHHC4tPgcFfB1siNAYpuIS74PpCh1h8ks5uYUeKgaJpZM4TJJ5P
.

--
JENNY MARX
User Experience Lead - TolaData
MERCY CORPS
tel<PHONE_NUMBER> | cell<PHONE_NUMBER> | skype jennyhavah
45 SW Ankeny St | Portland OR 97204 | USA
https://www.mercycorps.org/email-signature-creator#
Powered by the belief that a better world is possible.
Join us at mercycorps.org https://www.mercycorps.org/
Facebook https://facebook.com/mercycorps | Twitter
https://twitter.com/mercycorps

@jennyhavah I'm not sure what the two GAIT-related acceptance criteria are or how to address them.

Pull program status from GAIT
Filter programs by GAIT status


Add verify command for /cmd_vel topic to README

This PR resolves #17

I've edited the PR description above so that it would automatically close the linked issue when this PR is merged.


Recovery setup re-design

https://github.com/nearprotocol/near-wallet/issues/355

@corwinharrell Noticed the submit button is below the fold on my 13 inch Mac. @Patrick1904 And I were thinking to cut the h2, and reduce the margin of the h1.

Also considering adding a hoverable "?" next to "Basic Security" and "Advanced Security" with additional details around the recovery process and the tradeoffs

@kcole16

Noticed the submit button is below the fold on my 13 inch Mac.

that is also happening on like all the phones and not solvable with just changes to page heading. I think ultimately it's a non issue as users nowadays are more used to scrolling.

@kcole16 also let's make sure to test everything on mobile.
I'll also get us licenses of https://sizzy.app to make it easy to check multiple form factors at once

that is also happening on like all the phones and not solvable with just changes to page heading. I think ultimately it's a non issue as users nowadays are more used to scrolling.

The mobile experience is much better than the desktop one currently, as autofocus makes the submit button clearly visible. Scrolling is fine on mobile, less so on desktop.
Good call on sizzy.app!

@kcole16 I'm in general agreement with @vgrichina about content below the fold. Whenever this topic comes up I usually share this: https://www.hugeinc.com/articles/everybody-scrolls. Others might disagree but I think that we are at a point where people operate digital products with an expectation to scroll and that its not something thats going to break an experience.
In our scenario, the main thing that results in such large content height is our typographic scale and use of white space. Our typography on desktop is quite large for a utilitarian UI. It's more appropriate for marketing sites like our own, where the same styles are used I believe.
For a UI, especially one which plays a role of displaying large amounts of activity and financial / blockchain related data, the typography needs to be a bit tighter and contain less variation. A UI is likely to contain a larger number of interactive elements, components, and labels than just a static page with text. The result of having a scale with such large variations between different font-sizes is that everything overall just ends up being bigger and taking up more space than it needs to. This is something I've noted as a possible thing to explorer in a redesign.
Ultimately I think in the context of a UI like this you should be able to display more than just 3 inputs and a button on a page when viewing it on desktop, so just wanted to keep that in for future changes.

There's part of me that actually feels like we may have stripped out too much content that may have provided some good context, but I may need to look back at it to be sure.

@corwinharrell @vgrichina Fair points. I'm open to continuing with the current version (with some copy tweaks) for now, until we make broader font and spacing changes.
We can monitor for user behavior changes/user testing feedback on this screen in the meantime.
@Patrick1904 Thoughts?

@Patrick1904 please resolve conflicts

@corwinharrell @vgrichina Fair points. I'm open to continuing with the current version (with some copy tweaks) for now, until we make broader font and spacing changes.
We can monitor for user behavior changes/user testing feedback on this screen in the meantime.
@Patrick1904 Thoughts?

I don't have a strong opinion on this as I see pro's and con's with both approaches. Let's go with current and get a feel for it and hopefully some user feedback and we can adjust accordingly.


Device change error?

Hi -
We've implemented the diagnostics app into our Twilio video app - and it mostly works great!
But, some people that are trying to use diagnostics are running into messages I haven't been able to reproduce so far - error has appeared on both Android and iOS.  We were thinking it might be a conflict with the video app and the diagnostic app and have tried opening diagnostics in the same tab, but that seems to make the diagnostics app not function at all.  Any thoughts on how we can try to fix this problem?
Thanks

Hi @mloeff thanks for the question and for testing out our app!
There is currently no mobile web support for this app, but I am working on it as we speak, and it should be available soon 🎉
Interestingly, I was working on mobile support for the <CameraTest /> component yesterday and I was also getting an error on iOS Safari immediately after I granted the browser permission to access my camera. I was able to fix the issue by adding playsInline autoPlay to the <video> element here. Seems to be a requirement for Safari. I haven't been able to test on Android, so I am not sure if this will completely solve your issue (or at all). Your suspicions may be right, and it might have something to do with using the video app at the same time. I'll have to look into this a little more.
Do you happen to know when your users are receiving this Device change error? Are they receiving it after they give their browser permission to access their camera? Or are they seeing this error as soon as they reach the "Check your Video" screen? Also if you could find out which version of iOS Safari and which Android browsers your users are experiencing this issue with, that would be helpful as well.
As I mentioned, I am currently working on this myself, and I'll let you know if I run into the same issue that you are having. Thank you!

Hi @olipyskoty - thanks so much for your response.  I'm sorry it took so long for me to get back to you - I was waiting on some follow-up from the client.
The client says it's happening on the very first screen after "Let's get started".  Details I could get from the client (they stated that iOS/Safari had a problem as well, but I'm still trying to get more information from them).
Chrome application version 97.0.4692.98
Operating System
Android 12; SM-G998U Build/SP1A.210812.016

Hi @mloeff, I am so sorry for the delay!
Mobile support is finally available for this application. Please pull in the changes from the main branch to test out these new changes.
Also, as we were building out the mobile support, I realized that the "Device change: unable to connect" error is an edge case that seems to be related to device permissions. That error occurs when the following happens:

User is prompted by the browser to give it permission to access the user's camera and microphone
User grants permission to the browser
At some point, user revokes permission for the browser to access their camera and/or microphone (can be one or both)
"Device change: unable to connect" error is thrown

To make sure this does not occur, users should grant permission to their browser so that their camera and microphone can be accessed throughout the duration of the application. Note, some browsers need permission to access every single device that is to be used. That means, if a user is on the Camera or Microphone test, and they are switching between devices, their browser may ask for permission each time they switch. The user will need to grant permission each time. Can be kind of frustrating, but unfortunately this is browser specific 😢 .
Hopefully that clears things up, but please let me know if you run into anything else! Thank you!

Hi @olipyskoty - that's great news!  I'll have to try it out - we had made a few updates to the code to allow people to package it up and email it to us.  Many users found it a step too far to manual copy and paste, so we thought if they could do it easily, they were more likely to submit the results report.
Thanks again!
-Michael

Hey @mloeff I'll be closing this issue for now, but please let us know if you run into any more trouble or issues. Thank you!

Hi @olipyskoty ,
First of all kudos for making this amazing app, it's been liked by a lot of people from my team.
And so we recently decided to integrate it into our own Twilio based video app, however, I'm facing this same issue when I do audio playback for audio output test - "Device Change Error" - in Android devices(I have tried with multiple devices). I haven't checked in IOS devices yet.
First I thought it's an issue in my integration so I double checked it and everything was fine.
Furthermore, I found your hosted app - https://video-diagnostics.twilio.com/ - and I'm facing same issue in that as well in Android devices(I have tried with multiple devices). I haven't checked in IOS devices yet.
Is there any fix or is there something I am missing?

@olipyskoty Please check my above comment.


Adding length & is_not validation rule to docs

Is your feature request related to a problem? Please describe.
Currently, the contents at the top of the Rules guide mentions length & is_not but the doc doesn't have any info about these rules

Thanks for reporting this, it was fixed.


Change executionRootDirectory to File type

Update executionRootDirectory in generation context from String type to File type

CI build passed:


[BrokenLinksH2] Fix path in link

Line 77 removed link is not working, (https://channel9.msdn.com/events/teched/northamerica/2010/wcl315). Reaching out author if the linking to right content is possible.

@DhurataJ here is the link for the same video that you can add there: https://youtu.be/PYHqrwQIoxc
Let me know if you have any other questions.

Docs Build status updates of commit 35b7e1e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sysinternals/downloads/process-explorer.md
:bulb:Suggestion
View
Details

sysinternals/downloads/process-explorer.md

Line 66, Column 45: [Suggestion: docs-link-absolute - See documentation] Absolute link 'https://docs.microsoft.com/answers/topics/windows-sysinternals-procexp.html' will be broken in isolated environments. Replace with a relative link.

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

#sign-off


api(IB): Add span-based constuctors to ImageBuf

The flavor of ImageBuf that wraps caller-owned memory is constructed or reset by merely passing a raw pointer and it assumes that the range of memory implied by the strides is entirely safe to read from or write to. And even if correct, there is not much within ImageBuf internals to prevent bugs from buffer overruns.
This PR introduces a new constructor and reset() method to ImageBuf where instead of taking a raw pointer, it takes a span describing the extent of the safely addressible memory (and also an optional "pointer to first pixel" within the buffer, which is necessary if you're using negative strides or for some other reason are describing safe memory that starts before the location of pixel 0). That span is stored in the ImageBuf, and also it remembers if it was a cspan (which should indicate a read-only IB) or a non-const span (which should wrap writable memory).
At this moment, I'm just adding this API and will start using and documenting it as the perferred method, with the raw pointer versions being regarded as unsafe for now (and maybe some day will be deprecated?).  But we're not yet using the span internally for safety checks.
In the future, my intent is to gradually add safeguards to do bounds checking against the span for IB operations that touch the wrapped memory. Because we'll most likely bounds check only in debug mode, it shouldn't ever impact performance, but it will make debugging, sanitizer, and fuzz tests more robust to catch and zero in on buffer overruns. I also like how it forces the caller, at the call site, to think hard about exactly what range of memory the IB is wrapping, and not merely pass a single pointer and hope it works out.
I also changed all ImageBuf ctrs and resets within the OIIO codebase to use the new span-based versions.
This is part of an overall trend to migrate from raw pointer based API calls to span-based API design as much as possible.

Any objections or concerns?

I will merge this tomorrow if nobody has comments by then.


Add composer support

Composer makes dependency management easier and gives us a free autoloader.
If you merge this PR I suggest you submit your code to https://packagist.org/

Thank you! I've been meaning to add a composer.json for awhile. As suggested, I also submitted to packagist.org.


Closurizations of the same named constructor are not equal

According to specification (3-rd edition, June 2015) "16.18.8 Named Constructor Closurization"
"Except that iff identical(T1 , T2 ) then new T1#m == new T2 #m."
But in fact:
class C {
  C.m() {
  }
}

main() {
  print(identical(C, C)); // prints true
  print(new C#m == new C#m); // prints false
}

Tested on Dart VM version: 1.13.0-dev.6.0 (Wed Oct  7 08:09:59 2015) on "linux_x64"

The same issue exists for anonymous constructors too.
class C {
  C() {
  }
}

main() {
  print(new C# == new C#); // prints false
}

Do you have a corresponding co19 test?

I've added them recently. Please see
https://github.com/dart-lang/co19/blob/master/Language/Expressions/Property_Extraction/Named_Constructor_Closurization/identical_t01.dart
https://github.com/dart-lang/co19/blob/master/Language/Expressions/Property_Extraction/Anonymous_Constructor_Closurization/identical_t01.dart

This is a known limitation of the current implementation.

Generalized tear-offs are being removed (#27518) so I don't think we have to worry about this.

Bugs that fix themselves are my favorite ones.

Mine too! :D


Added json files of orders

Please make sure to update all the json

Sir, I've updated all the json files. Please review.

Sanjana resolve the comments, please review once.


Unit testing mssql

Added Unit Testing to the solution. Mainly for the MSSQL project. The other project benefited though. In the future, need to create Unit Tests for the main common projects.
Closes #11
Closes #12
Closes #13
Closes #14
Closes #15
Closes #16
Closes #17
Closes #18
Closes #19
Closes #20

Done merging. Keeping the branch around because more than likely I will add more to MSSQL Unit tests in the future.


Very slow

Issue Type: Bug
Some time is working and some time not working.
Extension version: 1.4.0
VS Code version: Code 1.36.1 (2213894ea0415ee8c85c5eea0d0ff81ecc191529, 2019-07-08T22:59:35.033Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i3-2120 CPU @ 3.30GHz (4 x 3293)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwareoop_rasterization: unavailable_offprotected_video_decode: unavailable_offrasterization: unavailable_offskia_deferred_display_list: disabled_offskia_renderer: disabled_offsurface_synchronization: enabled_onvideo_decode: enabledviz_display_compositor: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.90GB (0.50GB free)

Process Argv

Screen Reader
no

VM
0%

Thanks for your issue. The snippet selection sometimes works and sometimes doesn't work in VSCode? I haven't experienced it yet. Do you have other snippet extension installed? If so do they have the same behavior as the Ionic snippets?


Wrong code for "Upload and tag images" part

[Enter feedback here]

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

ID: 4f6c3f20-25a4-bc19-0cb6-377eff01b00e
Version Independent ID: 02d76c7b-cffa-c94f-2298-ba7e506d4f79
Content: Quickstart: Image classification with Custom Vision client library - Azure Cognitive Services
Content Source: articles/cognitive-services/Custom-Vision-Service/quickstarts/image-classification.md
Service: cognitive-services
Sub-service: custom-vision
GitHub Login: @PatrickFarley
Microsoft Alias: pafarley

@anastasia-nesterenko
Thanks for your feedback! We will investigate and update as appropriate.

@anastasia-nesterenko - thanks for your feedback. This issue is a duplicate of https://github.com/MicrosoftDocs/azure-docs/issues/66714 so we are closing this one and will address the issue in the other one.
#please-close


Update shift.xml

Fix item name typo
Shift_R to Shift Delete - use KeyOverlaidModifierWithRepeat
as delete key repeats by default

PR was updated to include 1 more tweak

Thanks!


Added option to skip dap and lsp setup

rust-tools is deeply integrated with setting up the lsp, so I don't think we need to give an option to disable it as it hinders a lot of functionality. Might change it later with lspconfig hooks but yeah not right now


Add error for multiple If-Match headers.

After discussions in the standup we will not add this error


schema validation

We should add a schema validation thing to find issues like:

schema definition missing in schema
to be continued...

Also when validating a Query there should be a meaningful message in case the validation errs due to missing schema declaration. A possible solution might be to initialize the operation definition values with "-1".

I think we should respect the spec here and add default behaviour as described in the spec:

Default Root Operation Type Names

While any type can be the root operation type for a GraphQL operation, the type system definition language can omit the schema definition when the query, mutation, and subscription root types are named Query, Mutation, and Subscription respectively.
Likewise, when representing a GraphQL schema using the type system definition language, a schema definition should be omitted if it only uses the default root operation type names.
This example describes a valid complete GraphQL schema, despite not explicitly including a schema definition. The Query type is presumed to be the query root operation type of the schema.

Should support extensions, e.g. in case of custom directives. So extensions should be able to define their own rules/hooks to extend the validation rules manually.


[pcre2] Convert JIT to feature

Convert JIT to feature

[x] Changes comply with the maintainer guide
[x] SHA512s are updated for each updated download
[x] The "supports" clause reflects platforms that may be fixed by this new version
[x] Any fixed CI baseline entries are removed from that file.
[x] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

It is currently not failing?

It is currently not failing?

vcpkg ci doesn't check arm64-osx triplet.

I have tested it locally

I have tested it locally

I tested it on osx on the gitgub action ci. and it fail without this fix.

Can you provide the error logs?

Can you provide the error logs?

I will.

Ah I found it:
/Users/leanderSchulten/git_projekte/vcpkg2/buildtrees/pcre2/src/cre2-10.40-42fc341438.clean/src/sljit/sljitExecAllocator.c:156:2: error: "Must target Big Sur or newer"
#error "Must target Big Sur or newer"
 ^
1 error generated.

But then I suggest to make it a feature instead of turning it off completely because it only works on current macs.

But then I suggest to make it a feature instead of turning it off completely because it only works on current macs.

I will try to convert it to a feature.

@autoantwort Can you test my port and check I didn't make errors?

@autoantwort What about android failing?

I think it is a baseline regression not caused by this PR.

It think that until https://github.com/microsoft/vcpkg-tool/pull/813 is merged, default-features should not contain features that have supports expression.
Workaround is to create platform-default-features feature that has platform expression (https://github.com/microsoft/vcpkg/pull/31291/files#diff-fa5764c3245e6fdc2a5bd9ad48405b4e6f2555a4bff9079d365bb4990e3b002a)

/azp run

@Cheney-W
There already fix that approved
https://github.com/microsoft/vcpkg/pull/29767
I will merge the master after you will merge it.

It think that until microsoft/vcpkg-tool#813 is merged, default-features should not contain features that have supports expression. Workaround is to create platform-default-features feature that has platform expression (https://github.com/microsoft/vcpkg/pull/31291/files#diff-fa5764c3245e6fdc2a5bd9ad48405b4e6f2555a4bff9079d365bb4990e3b002a)

This still must be done.

I have converted your PR to draft status. When you respond to Victor's suggestion, please revert to "ready for review". That way, I can be aware that you've responded since you can't modify the tags.

@Cheney-W @vicroms
Ready for review.


Added validation for ISRC strings

ISRC codes are being used in music related projects to identify a specific track (e.g. Led Zeppelin's "Stairway to Heaven") on a specific release (e.g. "Led Zeppelin IV").

Cool, thanks.


Enhancement

Partially Fixed tab spacing in EventController
At the time of teacher login array of events ID created by him are put to session and from it middleware EventOwner will check that whether the event he is trying to access belongs to him
Added on-delete to foreign key migration
Added favicon and chrome address bar colour
Partially completed work of displaying event info when a person is not login
Displayed number of question for every event in a student panel

The reason I told you to check for the existence of subject id because you are not handling the query insertion, so foreign key violation. Now, make separate PR with the changes.


Adding github workflows

The code needs to be tested (and probably deployed as well), so github workflows are a must for the future integration.

Github workflows are implemented for unit testing. Other workflows need to be implemented


css can't be used to have a dynamic selector

When I try to do the following it fails:
static my_selector = Target.the('my_selector')
     .located(by.css('#registry_validation_dropdown_idType li:nth-of-type("{0}")'));

but it works with xpath

 static my_selector = Target.the('my_selector')
      .located(by.xpath('//p-dropdown[@id='my_selector']//li["{0}"]'));

This looks like a duplicate of #93? Please feel free to reopen if that's not the case.

@jan-molak its exactly the same, but for me , it didnt work anything when I used css. can you make sure it works for a dynamic css selector for you?. I saw the commit you did but its only Xpath


Draw a Box - Rolling Pop-ups

@rsmithlal @y2749zha
After transcribing an entire section successfully and you have drawn the annotation box, selecting the 'Mark annotated area and don't show me again' option on the tutorial pop-up has a bug that causes rolling pop-ups to appear. The tutorial pop-up does not disappear, and it will cause the blue coloured annotation popups to appear in multiples, blocking the screen and slowing down the transcription process. If you have completed 6 sections and drawn six annotation boxes, you will have 6 popups for example (see image).
A work around for the moment is refreshing the page between transcribing tabs, or by closing and re-opening the transcriber panel after each row of data, but it has been observed that this does not always help resolve the issue.
Suggested fix: disabling the pop-up feature for the time being to see if this helps.

@rsmithlal Can we disable the box drawing for the moment?

You mean disable the message?

Yes, I believe so. Vicky mentioned it in the meeting tonight.

Several users have told me they stopped transcribing because they were confused by the multiple pop-up messages - along the lines of “but I just drew the box, why do I need to draw another one?”, or were seriously slowed down by having to reload the page or wait for the multiple boxes to disappear once they understood it was a bug. And these are just the ones I know about through personal contact.
Part of this goes back to people not reading the help text/ bug reports/ etc, but it’s causing us to lose users due to confusion or slowing down the process.

On Jun 16, 2020, at 7:36 PM, raakal<EMAIL_ADDRESS>wrote:
Yes, I believe so. Vicky mentioned it in the meeting tonight.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/open-data-rescue/climate-data-rescue/issues/230#issuecomment-645064093, or unsubscribe https://github.com/notifications/unsubscribe-auth/AENOOAJV5OU2Y33K7GN4CNLRW76ZHANCNFSM4NNOGLHA.

Ok, that's good to know. I will try to get this done ASAP and deploy a new version to the server

That would be great, thanks!
Vicky

On Jun 16, 2020, at 9:29 PM, Robert Smith<EMAIL_ADDRESS>wrote:
Ok, that's good to know. I will try to get this done ASAP and deploy a new version to the server
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

Where the message is triggered https://github.com/open-data-rescue/climate-data-rescue/blob/aa00a34c7ebbf2a401ccef6d4bbfb821a7b5b16f/app/views/shared/_transcriber_app.html.erb#L111

The plugin http://bootstrap-notify.remabledesigns.com/

The problematic callback https://github.com/open-data-rescue/climate-data-rescue/blob/aa00a34c7ebbf2a401ccef6d4bbfb821a7b5b16f/app/views/shared/_transcriber_app.html.erb#L780

Blue message trigger https://github.com/open-data-rescue/climate-data-rescue/blob/aa00a34c7ebbf2a401ccef6d4bbfb821a7b5b16f/app/views/shared/_transcriber_app.html.erb#L1097


docs(prs): add pull request template

How does this look? Trying to make sure we remember any necessary changes while not impeding contribution.

@smothers I think I resolved your issues. Please re-review when you get a chance.


MCM Usage to take holds prevents progression

when you do the method mentioned in bug 4 legate rikke does not move to Eastmarch camp imperial camp and instead she just sits in castle dour waiting for you to come (she does appear in the final battle with tulius.)
using the mcm to take a hold also prevents progression

@LABWAZHERE Can you try this again? I made a bunch of fixes to the MCM so handling running campaigns and changing holds should work now. New version is in the Release section.

yes testing them all now

          > >

Can you send me a screenshot of the debug? Where are you and what are you trying to do via the MCM?

campaign phase thing, still won't work, the mcm menu for 6 quests before hold battle that's what i meant
Originally posted by @LABWAZHERE in https://github.com/schofida/CWORR/issues/12#issuecomment-1492791461

Sorry I still need a little more information. What did you tick in the MCM and what were you expecting to happen?

this

other than that hold progression works fine

Ah thank you. I forgot about that. It might be an MCM bug. I just pushed release v1.0.3a to display a different way. I did not get a chance to test at all unfortunately.

I got it working finally on 1.0.5a


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 916

Questions ?
Please refer to the Backport tool documentation


error in download the library

the library was working fine until today

Hey @karimsaad687 , is this still a problem for you? I just tested with the example app in this repo and it seems to be working fine. I remember recently there was an outage in some of the Maven repositories.


[fix] this might be a typo.

@coreyauger I'm reading the node stateless code and found a typo I think (tservice vs. service) on the way and attached a fix for it.
https://github.com/cloudstateio/cloudstate/blob/a70dece2cd2a90397db09fc31fd4ff7f879fd61f/node-support/src/stateless-support.js#L135

JS O_o


MySQL 8 returns upper case column names results in KeyError in dictionary

In function sql_to_dict SELECT result which was returned from sql-server is transformed to dictionary. Then some particular keys are supposed to be in this dict, dependent on SELECT, for example, "column_name" . But MySQL 8 returns column names in upper case, so instead "column_name" there will be "COLUMN_NAME".
Possible solution is to add .lower() to name[0] in sql_to_dict()

If you want to make a PR for this I think the easiest way is to override the sql_to_diff method in the MySQLDiff class to support this.
I am not familiar with MySQL would we need to do this only for MySQL 8 or also for other mysql versions or variants (MariaDB et al) ?


Add  handling similar to  switchToIframe function

Feature request
Expected behavior
I would like to access  elements with something like switchToObject which would allow me to interact with elements within that object.

The  tag defines an embedded object within an HTML document. It's used to
embed multimedia (like audio, video, Java applets, ActiveX, PDF, and Flash).
We do not have access to  contents because it's not a DOM structure. So we cannot execute actions and perform assertions with it. That's why we do not support testing of  contents.

I reopened this issue because the <object> element can represent an external resource that could be nested browser context.


Add Snowman Dress up Game

This project is a game in which you have to dress up the snowman with the things which are there on the screen by just dragging the things and drooping them on the snowman figure.
Its working video,
https://user-images.githubusercontent.com/75881393/117947122-84551200-b32d-11eb-9dbf-1b12bfb905de.mp4

Please @ridsuteri @991rajat @urvashi-code1255 review this issue.


Feature/viper3400/contain in any order

This PR is the result of this Gitter discussion: https://gitter.im/serenity-js/Lobby?at=607982bbd795391861601abd
At the moment there seems to be no need to implement https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/localeCompare. It's doesn't matter in wich exact order they are sorteda as long as both lists match at the end. Both lists, the expected list and the actual list as well, are sorted at the same time, so they should be the same.
@jan-molak : As suggested I replaced the JSON.stringify with equal.
Please review this changes and consider to merge.
Is there is still something that I missed, please provide feedback.
Thanks, Jan

Hi @viper3400 and thanks for submitting the PR, nice work!
As I was reviewing the changes, I thought it might be a good opportunity to look at collection-specific assertions in general and see where we could take them.
At the moment we have:

contain<Item>(expected: Answerable<Item>)
containAtLeastOneItemThat<Actual>(expectation: Expectation<any, Actual>)
containItemsWhereEachItem<Actual>(expectation: Expectation<any, Actual>)

Given how the above are implemented,

[ ] Each is generic, so it would be great to make containInAnyOrder work with types other than string (a'la contain). This way we could use them to assert on API responses, for example. To do that, you might need to abandon sort and implement some sort of array intersection check.

[ ] I am also thinking of renaming containAtLeastOneItemThat to containAtLeastOneThat and containItemsWhereEachItem to containItemsWhereEach to drop the slightly redundant word Item. So maybe we could call the expectation you propose containInAnyOrder rather than containItemsInAnyOrder?

Some more thoughts for context;
At the moment Serenity/JS has two flavours of collection-specific expectations.
One that receives an explicit value of the expected item, for example:
const animals = [ 'cat', 'dog', 'bird' ]

Ensure.that(animals, contain('dog'))

and another one that accepts expectations, so:
Ensure.that(animals, containAtLeastOneThat(startsWith('d')))

I'm considering whether we should continue with this pattern, or use Hamcrest-style overloaded methods.
With the latter, a collection-specific expectation could accept
var-arg with explicit values:
Ensure.that(animals, containInAnyOrder('bird', 'cat')

var-arg with expectations:
Ensure.that(animals, containInAnyOrder(equals('bird'), equals('cat'))

array with explicit values:
Ensure.that(animals, containInAnyOrder([ 'bird', 'cat' ])

array with expectations:
Ensure.that(animals, containInAnyOrder([ equals('bird'), equals('cat') ])

This second part is outside the scope of this PR, but I wanted to write it down to clarify my thinking :-)
Thoughts and feedback welcome!

Hi @jan-molak , I renamed it to containInAnyOrder.
Concerning the generic approach: I have to rethink this again. The sort of the string arrays came with no costs. When talking about API responses and given their large possible varieties ... have to look if I could find a quite moment for this one ...

Hi @jan-molak
I thought about this over the week.
First,
I've the "simple" string array sort function in production at our site and it works as expected. Therefore my urgent problem is solved.
Second,
I'm not sure any more if containInAnyOrder is the correct name at all.
So let's break it down a bit more from my current use case:
I have an actual array of let's say 10 strings. I know all the 10 strings. I also know, that there are doubles within.
If I'd like to ensure, that just one special string is in the array, I would use Ensure.that(actualArrayOfStrings, contains(specialString)).
If I would like to ensure, that all expected strings are in the array - and given, I would also know the order of the strings as well - I would just use Ensure.that(actualArrayOfStrings, equals(expectedArrayOfStrings)).
So, now I don't know the order. With a function called containInAnyOrder one could also assume, that you have an array of 20 strings, where you'd ensure, that an array of just 10 expected strings is just contained in this 20 strings (in any order) ...
I expect all strings to be equal (in any order).
Should it then not be called equalsInAnyOrder?
Third,
you'd like to see a generic approach.
For arrays of string, number or boolean the normal sort will do.
I don't want to implement a fancy deep object comparison or something like that. In case we are dealing with an array of object we can enhanced the sort with a compare function
When comparing generic objects, the framework itself won't know the generic content. Test developer has to pass a property which is present in all of the expected items. This will be a precondition to use the function. You won't be able to use it with an API response like this one:
[
 {
	"id": "1",
	"name: "eggs"
 },
 {
     "price": "1"
 }
]

If you don't pass a property in combination with an object I throw an AssertionError. How can I assert this Error with Chai, didn't get a test for this running?
Finally,
I pushed my current changes to the branch. I also enhanced the tests to make some of my explanations above a bit more clear.
As my current requirements are well covered with the string array, I regret I cannot spent very much more time in research at the moment.
If you have another idea how to reach a better solution here, which I could implement in a reasonable time, I'm still open to give it a try. Maybe I'm just missing something obvious?

Hi @viper3400 and thanks for putting so much thought into it! I've been thinking about it as well, so apologies I haven't replied sooner; you've raised several good points so I wanted to address them.
I've been looking for inspiration in other assertion libraries, and quite like how Hamcrest and AssertJ have gone about their implementations.
If we look at the AssertJ implementation, its naming is quite elegant and most of it feels applicable to @serenity-js/assertions:

contains(ELEMENT... values)
Verifies that the actual group contains the given values, in any order.
containsAll(Iterable<? extends ELEMENT> iterable)
Verifies that the actual group contains all the elements of given Iterable, in any order.
containsAnyElementsOf(Iterable<ELEMENT> iterable)
Verifies that the Iterable under test contains at least one of the given Iterable elements.
containsAnyOf(ELEMENT... values)
Verifies that the actual Iterable contains at least one of the given values.
containsExactly(ELEMENT... values)
Verifies that the actual group contains exactly the given values and nothing else, in order.
This assertion should only be used with groups that have a consistent iteration order (i.e.
containsExactlyElementsOf(Iterable<? extends ELEMENT> iterable)
Same as ObjectEnumerableAssert.containsExactly(Object...) but handle the Iterable to array conversion : verifies that actual contains exactly the elements of the given iterable and nothing else in the same order.
containsExactlyInAnyOrder(ELEMENT... values)
Verifies that the actual group contains exactly the given values and nothing else, in any order.
containsExactlyInAnyOrderElementsOf(Iterable<? extends ELEMENT> values)
Verifies that the actual group contains exactly the given values and nothing else, in any order.
containsOnly(ELEMENT... values)
Verifies that the actual group contains only the given values and nothing else, in any order and ignoring duplicates (i.e.
containsOnlyElementsOf(Iterable<? extends ELEMENT> iterable)
Same semantic as ObjectEnumerableAssert.containsOnly(Object[]) : verifies that actual contains all the elements of the given iterable and nothing else, in any order and ignoring duplicates (i.e.
containsOnlyOnce(ELEMENT... values)
Verifies that the actual group contains the given values only once.
containsSequence(ELEMENT... sequence)
Verifies that the actual group contains the given sequence in the correct order and without extra values between the sequence values.
containsSequence(Iterable<? extends ELEMENT> sequence)
Verifies that the actual group contains the given sequence in the correct order and without extra values between the sequence values.
containsSubsequence(ELEMENT... subsequence)
Verifies that the actual group contains the given subsequence in the correct order (possibly with other values between them).
containsSubsequence(Iterable<? extends ELEMENT> subsequence)
Verifies that the actual group contains the given subsequence in the correct order (possibly with other values between them).

In particular, I quite like the distinction between contains, containsExactly, containsSequence and containsSubsequence, so perhaps that's a pattern we should consider going with as well?
Regarding equality checks - I think this could be solved by using equal from tiny-types, which is what @serenity-js/assertions equals uses under the hood.
What do you think?

@jan-molak I will give you detailed feedback soon.

Hi @jan-molak
To your questions and thoughts:
For the naming: I think both, AssertJ and Hamcrest as well, are good. With a slight preference for the AssertJ naming. It feels a bit more intuitive fore me. But as English is not my native language I don't know what to prefer from a language point of view.
Talking about the flexibility of Hamcrest: Here we would add not just naming, but a different functional approach as well. For me it seems less readable and it adds complexity not only to the implementation but also to the test itself:
Given a shopping list with:

coffee
milk & coffee

What would you expect to be the result of:
Ensure.that(shoppingListItems, containInRelativeOrder(equals('coffee'), startsWith('milk'), equals('milk & coffee')))

I really don't know what to expect here. True, because the last two expecations are met? False, because I would expect a third item called just milk in the shopping list?
On the other site, your are back in just comparing strings in this example. Should this also work with a generic approach, with an Json API response, for instance?
Looking at the assertions package, most of the assertions support just a special type. isBefore just works with Date, isGreaterThan just with numbers. Expecting a string array with exact values, but in different order, was just the use case I needed this time.
Finally, trying to implement a generic approach failed for now, though I think there a just some details missing.
Having said that: Nevertheless, I would like to suggest to postpone or even to reject this PR, for now.
As I mentioned before, I'm afraid that I don't have enough time at the moment to go further with this PR. Discussion show, that we should define some use cases / test cases / small concept what would be expected of such a function.
From my side, I also won't you to spent too much more of your time in a topic, which would be a good enhancement for sure, but is solved for me at the moment as well.
I would prefer to foster progress in https://github.com/serenity-js/serenity-js/issues/805. There is a initiative moving from "pure" Protractor towards cypress.io in my business. I'd like to put Sereninty/JS into consideration as well, but without having #805 in place and something to show, I'd also need to explain, why I'd go further with a deprecated "Protractor" tool.
As I still like to use and to contribute to this beautiful project, I hope you understand, that I would prefer to put all effort on #805 and come back to this topic later on.
What do you think?
Regards, Jan


pacman streaming demo: update instructions to include AWS CLI assume cmd

Users should be able to use pretty much the same instructions with the "assume" cmd from the AWS CLI. Need to verify.

stale-- year old


Updated recursion-string in lessons

Fixes #
Short description of what this resolves:

What Changes proposed in this pull request:
Added content to the recursion-string.md in lessons

Checklist

[yes ] Added description of change
[yes ] Added file name matches File name guidelines
[yes ] Added code examples, test must pass
[yes ] Added documentation so that the program is self-explanatory and educational.
[yes ] Relevant documentation/comments is changed or added
[yes ] PR title follows semantic commit guidelines
[ yes] Search previous suggestions before making a new one, as yours may be a duplicate.
[ yes] I acknowledge that all my contributions will be made under the project's license.

Notes:
I have added content with regards to string recursion in the recursion-string.md file

@Utkarsh1504  Sir, please do review this PR and tell me if I need to make any changes

@Utkarsh1504 Sir, I have added what was requested in the syllabus, please check and review it


The node.js not in the system PATH env

I had another attendee where they installed node.js via the devkit, however it would never resolve inside the VS Code integrated terminal (even after closing and restarting VS code).  I had her manually install node from the web site, making sure to include Node in the path, and that fixed it.

Try to repo this issue first

We tried the following scenarios, but hard to repo this issue.

If uninstall nodejs, and make sure the variable is removed from the Path env, then run the installation script, the nodejs is installed with variable added to the Path env

Just remove the variable from the Path env, then run the installation script, the installation is failed with "Error occurred when running yarn install" error.

Rare case and hard to repo, and since we are working on the new VS Code extension to replace the installation script, so close this issue.


az : ERROR: Failed to parse JSON: ./AdAppManifest.json

Hello,
Just downloaded a fresh copy of all source code, and when running deployment script nothings gets created on AZ . See error message below,
az : ERROR: Failed to parse JSON: ./AdAppManifest.json
At C:\Users**\Desktop\Teams App Templates\FAQPlus\microsoft-teams-apps-faqplus-master\Deployment\deploy.ps1:293 char:17
...             az ad app create --display-name $appName --end-date '2299 ...
CategoryInfo          : NotSpecified: (ERROR: Failed t...ppManifest.json:String) [], RemoteException
FullyQualifiedErrorId : NativeCommandError
Error detail: Expecting value: line 1 column 1 (char 0)
The JSON may have been parsed by the shell. See https://docs.microsoft.com/cli/azure/use-cli-effectively#quoting-issues
....
Although it seems to succeed by the last message (not shown here but on my PS ISE console), nothing is provisioned on Azure.
The only file I modified is the parameters.json per documentation.
UPDATE: There is no adappmanifestr.json file to be found in your code base as it seems there should be one, and no logs.zip are created once the PS script is executed. The Deployment Guide seems to indicate these two files should be under deployment folder, see image

Any updates on this?

Hi @magafaterr,
Thank you for raising the query, we're looking into this issue, will get back to you as early as possible.

I spent many hours today trying to setup this and I can't get his installed.  I now found this issue and am hitting into the same.  Is there any workaround?  I need to get this up and running quickly.

Using the old classic deployment guide worked for me, but not with the automated way ie script ps.
On Feb 1, 2022, at 3:29 PM, Tom-K50 @.***> wrote:

I spent many hours today trying to setup this and I can't get his installed. I now found this issue and am hitting into the same. Is there any workaround? I need to get this up and running quickly.
—
Reply to this email directly, view it on GitHubhttps://github.com/OfficeDev/microsoft-teams-apps-faqplus/issues/155#issuecomment-1027258622, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHRSLCK7CNUJ3II5QEUMS4LUZA7DVANCNFSM5MP6DLQA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you were mentioned.Message ID: @.***>

Hi @Tom-K50 @magafaterr
While we work on identifying the root cause of the issue you're facing with PS deployment, can you please try with the manual deployment option as mentioned in below link and let us know if it works fine.
https://github.com/OfficeDev/microsoft-teams-apps-faqplus/wiki/Deployment-Guide-manual

@magafaterr PowerShell script deployment issue is fixed and it works fine now, you can perform the deployment using PS and let us know if you face any issues.

Closing this as the issue is resolved, please open new one in case of any other issues.


opt: fix StorageInputOutput16 trimming.

While integrating this pass into DXC, I found a lot of missing cases. This PR fixes a few issues centered around this capability while laying out fondations for more fixes.

The grammar can define extensions in operand & opcode tables.

opcode can rely on common capabilities, but require a new extension.
opcode can also rely on a capability which requires an extension. Sometimes, the extension is listed twice, in the opcode, and capability. But this redundancy is not guaranteed.

minVersion check. The condition was flipped: we added the extension when the minVersion was less than current. Didn't noticed the issue as I only tests on the default env.

Capability/Extension instructions were not ignored.

OpCapability Foo will require the Foo capability.
it doesn't mean the module requires the Foo capability. Same for extensions.

ShaderC failure seems unrelated:
Glsl/CompileVulkanToSpirvTest.FromFile/spv_coopmatKHR_comp
Might be fixed with https://github.com/KhronosGroup/Vulkan-ValidationLayers/pull/6231


Error: Got an error from AGPU.

Hello,
The installation of woden2 seems successful, but when I start a demo (WTFSSampleApplicationCube new openInOSWindow), I have got an error from AGPU :
WTPipelineStateCache >> buildShaderSignature:
signatureBuilder addBindingBankElement: bindingType bindingPointCount: bankElement bindingPoints.

AGPUShaderSignatureBuilder >> addBindingBankElement: bindingPointCount:
resultValue_ := AGPUCBindings uniqueInstance
addBindingBankElement_shader_signature_builder: (self validHandle) type: type

resultValue_ return me -2 as error code.
Computer with Ubuntu 16.10 64 bits, and Radeon R7 360.
Thanks !

Can you tell me whether libAgpuVulkan.so (or libAgpuVulkan_d.so) is found
under native-libraries-32/AgpuIcd? There is an obsolete OpenGL backend for
fallback which is untested. I am wondering if all of the dependencies
required for building the Vulkan backend were satisfied or not.
Can you try running the following with an open Transcript?
WTEngine tracingEnabled: true.
WTEngine new inspect
Best regards,
Ronie
2017-03-03 5:08 GMT-03:00 nolwennfournier<EMAIL_ADDRESS>
Hello,
The installation of woden2 seems successful, but when I start a demo
(WTFSSampleApplicationCube new openInOSWindow), I have got an error from
AGPU :
WTPipelineStateCache >> buildShaderSignature:
signatureBuilder addBindingBankElement: bindingType bindingPointCount: bankElement bindingPoints.
AGPUShaderSignatureBuilder >> addBindingBankElement: bindingPointCount:
resultValue_ := AGPUCBindings uniqueInstance
addBindingBankElement_shader_signature_builder: (self validHandle) type: type
resultValue_ return me -2 as error code.
Computer with Ubuntu 16.10 64 bits, and Radeon R7 360.
Thanks !
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ronsaldo/woden2/issues/1, or mute the thread
https://github.com/notifications/unsubscribe-auth/ABepPwTCJMyFwtTh2K-O32GEzr9bV9sWks5rh8pygaJpZM4MR-OF
.

Answer of locate libAgpuVulkan :
woden2/build-32/abstract-gpu/dist/AgpuIcd/libAgpuVulkan_d.so
woden2/native-libraries-32/AgpuIcd/libAgpuVulkan_d.so

I'm sorry, I forget to tell that during the compilation, there were warning with OpenGL not found.
I found surprising that the compilation ends.
In my terminal, I have the following error :
Required extentions are missing.
Error: Got an error from AGPU.

In my transcript, I have just:
...finished baselineGot an error from AGPU: -2

Required extentions are missing.
const char *requiredExtensionNames[] = {
VK_KHR_SURFACE_EXTENSION_NAME,
#if defined(_WIN32)
VK_KHR_WIN32_SURFACE_EXTENSION_NAME,
#elif defined(unix)
VK_KHR_XCB_SURFACE_EXTENSION_NAME
#else
#error unsupported platform
#endif
};

Are you using MIR or Wayland? What is the output of vulkaninfo?
2017-03-03 7:38 GMT-03:00 nolwennfournier<EMAIL_ADDRESS>
Answer of locate libAgpuVulkan :
woden2/build-32/abstract-gpu/dist/AgpuIcd/libAgpuVulkan_d.so
woden2/native-libraries-32/AgpuIcd/libAgpuVulkan_d.so
I'm sorry, I forget to tell that during the compilation, there were
warning with OpenGL not found.
I found surprising that the compilation ends.
In my terminal, I have the following error :
Required extentions are missing.
Error: Got an error from AGPU.
In my transcript, I have just:
...finished baselineGot an error from AGPU: -2
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ronsaldo/woden2/issues/1#issuecomment-283922019, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABepP735i6Nw4ikE9O6HTC2t4NQTqQrUks5rh-22gaJpZM4MR-OF
.

I have done a standard installation of Ubuntu 16.10 for Woden2. I don't know which one is the default display server.
Vulkaninfo :
Cannot create Vulkan instance.
/build/vulkan-ijAUlz/vulkan-<IP_ADDRESS>+dfsg1/demos/vulkaninfo.c:675: failed with VK_ERROR_INCOMPATIBLE_DRIVER

So, I have tried to install mesa-vulkan-drivers mesa-vulkan-drivers:i386 and restart the installation.
No difference in woden2.
But, in Vulkaninfo :
/build/vulkan-ijAUlz/vulkan-<IP_ADDRESS>+dfsg1/demos/vulkaninfo.c:1415: failed with VK_ERROR_INITIALIZATION_FAILED

mesa-vulkan-drivers mesa-vulkan-drivers

The version in the official repositories only  have vulkan drivers for the
Intel graphics cards.
For AMD, they are two vulkan drivers, the open source RADV (and
incomplete), and the proprietary AMDGPU Pro driver. For both of these
drivers, I believe that It can be bit tricky installing the vulkan support
because they require enabling experimental support for older graphics cards
(Southern Island, and Sea Island) in the amdgpu kernel driver. There are
instructions about how to install these drivers somewhere on the internet.
The minimal required support is being able to run vulkaninfo. I manage to
install the RADV driver in a old HD 7770, however Woden 2 is hanging the
machine when running the Cube demo, so I do not know whether it is going to
work for you.
2017-03-03 8:18 GMT-03:00 nolwennfournier<EMAIL_ADDRESS>
I have done a standard installation of Ubuntu 16.10 for Woden2. I don't
know which one is the default display server.
Vulkaninfo :
Cannot create Vulkan instance.
/build/vulkan-ijAUlz/vulkan-<IP_ADDRESS>+dfsg1/demos/vulkaninfo.c:675: failed with VK_ERROR_INCOMPATIBLE_DRIVER
So, I have tried to install mesa-vulkan-drivers mesa-vulkan-drivers:i386
and restart the installation.
No difference in woden2.
But, in Vulkaninfo :
/build/vulkan-ijAUlz/vulkan-<IP_ADDRESS>+dfsg1/demos/vulkaninfo.c:1415: failed with VK_ERROR_INITIALIZATION_FAILED
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ronsaldo/woden2/issues/1#issuecomment-283929572, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABepP4WoYeinJ9NMKViQ0yIer-GaNT05ks5rh_b5gaJpZM4MR-OF
.


Update requirements and environment.yml

Hi, thanks for the code!
I was unable to use the environment.yml file as the error came from the clip installation via pip. I fixed this by changing it to clip-by-openai, and then everything was installed correctly, assuming this is indeed the required clip. However, the Requirements are in conflict with the environment.yml file (the versions of Python and PyTorch), so I assume the latter is the correct one. Any clarification here will be greatly appreciated.

The clip package installed by pip is this one: https://pypi.org/project/clip/
It is indeed not related to OpenAI's project.

That being said, CLIP is already in the repository:
https://github.com/autonomousvision/stylegan_xl/tree/main/feature_networks/clip


Drop down editable

fix: #36
Defect Fixes
When submitting a PR, please also create an issue documenting the error and manually link to an issue or mention it in the description using #<issue_id>.
Feature Requests
Due to company policy, we are unable to accept feature request PRs with significant changes as such cases has to be implemented by our team following our own processes.
Smaller scaled feature implementations such as adding a property to a component will be considered for merging.

temp


[Dataflow Streaming] Use separate heartbeat streams based on job settings

The logic is guarded behind an experiment streaming_engine_use_job_settings_for_heartbeat_pool. The experiment will be removed once backend changes are rollback safe.

R: @scwhittle

Run Java Precommit

precommit is unrelated pubsub auth issue.


:sparkles: Allow ES6 classes

This stays ES5 compatible and allow usage of ES6 classes. See this link for the explanation.
Basically, this allow this smoother syntax:
const util = require('util')
const Scout = require('zetta-scout')

// "Old" syntax

const TestScout = function () {
  Scout.call(this)
}
util.inherits(TestScout, Scout)

TestScout.prototype.init = function (next) {
  next()
}

// "New" syntax

class TestScoutEs2015 extends Scout {
  init (next) {
    next()
  }
}

Both syntaxes still work with this PR.

@AdamMagaluk This looks good to me.  Any objection to merging?  (Asking permission, because well... you know... it's complicated.  😉)

LGTM. Sorry for the delay @kevinswiber and @marvinroger been busy with other projects this week. I hope to give all the other PRs a review in the next few days.

@AdamMagaluk Your "delay" was 12 hours.  Go easy on yourself.  I've committed far more atrocious open source crimes.  😺

Don't worry, moreover we don't live in the same TZ (Europe/Paris for me) so there will be delay anyway. 😉

Thanks! Can you make a release?

Doing it now.

v0.8.0 Released.


improve perf of browseDefinitionsAction (fixes #528)

we were dispatching an update for every definition we got back from the query.
There was also some unused code paths confusing things

so fast!

Anything blocking this smokin' hot performance being unleashed on the public?

The performance improvement is due to the removal of a portion of dispatching a single action for each definition in the list. But actually that action creates a different part of the store that is used to have a direct reference to the definition in the list.
That is, with this change we are not able to make some operation to specific definitions , like reverting.
Also, i'm in doubt if the removal button has some kind of utility in the browse page?

Update: looks like the changes in this PR has nothing to do with the revert function.
I'm looking at it and i will open an issue together with a separate PR.


Question(s) about kubernetes auth for hashicorp vault within the plugin

Hello guys,
I am currently searching for a tool which could help us for GitOps secret management, which also lead me to this awesome project.
During my read-through of the documentation I've noted that the ArgoCD vault plugin currently supports four different authentication modes when it comes to secret management with Hashicorp vault:

token
approle
github
kubernetes

As the kubernetes authentication would eliminate the "secret zero" issue, where you need to set up one initial secret in order to let the whole setup work (by shifting it over to kubernetes service accounts and their tokens with respective information of a token in vault) I am quite interested in this one.
However, from my current point of view (related to the information that I could obtain from the documentation, precisely from here => https://argocd-vault-plugin.readthedocs.io/en/stable/backends/) the only configuration with kubernetes auth is by mentioning and assigning permissions to the kubernetes service account, which runs as part of the argocd-repo-server deployment.
Meaning, that I need to permit access to all the vault resources that are required for all my ArgoCD applications in a single vault role (in the example of the doc this role is called "argocd"). However what I am looking for is an option where I can mention a vault role with respective permissions per ArgoCD application in order to achieve a more "least priviledged" approach for reading secrets from vault.
Basically, if a compromise of a specific kubernetes service account should occur, I do not want to rotate all my secrets for all my ArgoCD apps but limit the blast radius of such a compromise to only the secrets of the ArgoCD application where they belong to.
In general, for the three other auth methods (token, approle & github), it appears to me that the authentication can scoped to an ArgoCD application level, as mentioned in https://argocd-vault-plugin.readthedocs.io/en/stable/config/#multitenancy. Could it be the case that just the kubernetes auth method does not support it, as it relies on a k8s service account which is not in place when you just use ArgoCD, requiring the usage of the argocd-repo-server deployments service account? Or can I create additional service accounts at the kubernetes level, include them in my argoapp and reference the sa, its namespace and the related vault role within the argocd vault plugin config? If that is the case, why is there no parameter to reference a service account, but just a kubernetes role?
From https://argocd-vault-plugin.readthedocs.io/en/stable/backends/ => HashiCorp Vault => Kubernetes Authentication:

VAULT_ADDR: Your HashiCorp Vault Address
AVP_TYPE: vault
AVP_AUTH_TYPE: k8s
AVP_K8S_MOUNT_PATH: Mount Path of your kubernetes Auth (optional)
AVP_K8S_ROLE: Your Kuberetes Auth Role
AVP_K8S_TOKEN_PATH: Path to JWT (optional)

A respective parameter is also not mentioned in https://argocd-vault-plugin.readthedocs.io/en/stable/config/.
(The mentioned full documentation of the k8s auth at the second step of the chapter "Kubernetes Authentication" within the docs seems to be not existing anymore: https://argocd-vault-plugin.readthedocs.io/en/stable/backends/vaultproject.io/docs/auth/kubernetes#configuration Maybe this page would answer my questions.)
Maybe I have missed the related information from the documentation; I would be very happy if you could point me to them if thats the case.
Generally thank you very much in advance for any hint or answer to my questions. :)
Regards,
Maik

Could it be the case that just the kubernetes auth method does not support it, as it relies on a k8s service account which is not in place when you just use ArgoCD, requiring the usage of the argocd-repo-server deployments service account? Or can I create additional service accounts at the kubernetes level, include them in my argoapp and reference the sa, its namespace and the related vault role within the argocd vault plugin config?

As it stands, AVP will run in the context of the argocd-repo-server pod which is given a ServiceAccount. You might be able to do something like this:

Create X ServiceAccounts where each will get access to some specific Vault path/namespace/whatever via a specialized Vault role
Mount each of these ServiceAccount's Kubernetes JWT into different paths into the argocd-repo-server pod
Specify AVP_K8S_TOKEN_PATH in each Argo Application manifest so that a different SA token is used for the Kubernetes auth

I'm not sure if that's a really good idea though. If this doesn't work, your best bet is to have several Argo CD instances and separate the SA permissions that way

Thanks for the reply and input! I guess that it would be worth a try to create multiple SAs and mount their tokens into the argo-cd-repo server. However it'd be great if there was some kind of possibility to specify the k8s sa within an argoapp (of course with the requirement that this sa is part of the argocd application) in order to have a more granular approach without the need to modify the argocd-repo-server deployment for each additional argoapp. However I am not sure if this is possible from a technical perspective. In such a case the argocd vault plugin would require an extension which enables the modification of the argocd application custom resource definition in order to allow additional fields and also some kind of logic that makes sure, that a given sa (mentioned in the argo application resource) exists as a resource manifest within the app to then create the sa and verify its token via the repo servers sa. So for example...
apiVersion: argoproj.io/v1alpha1
kind: Application
metadata:
  name: demo-app
  namespace: argocd
spec:
  destination:
    namespace: demo
    server: https://kubernetes.default.svc/
  project: default
  source:
    path: apps/todo
    repoURL: https://demo-git.com/demo-argo/demo
    targetRevision: demo
  syncPolicy:
    automated:
      prune: true
      selfHeal: false
    syncOptions:
    - CreateNamespace=true
    serviceAccount: demo-sa

On top of that the templates folder of the demo-app would require to include a sa manifest that creates the k8s "demo-sa" in the "demo" namespace. On the vault side it could then be possible to assign this sa a role and the argocd-repo-server should be in the position to verify if this sa has the privilege to use the role.

Being able to specify the templating service account would be great, especially since that's something we could have a linter verify.