Main Test Workflow

Performance results based on clockticks comparison with main HEAD (higher %25change shows improvement):%0A%0A|arch|engine|phase|%25Change_(1-Head/Patch)|%0A |-|-|-|-|%0A|x86_64|patch|Compilation|0.069821|%0A|x86_64|patch|Instantiation|0.070701|%0A|x86_64|patch|Execution|-0.708193|

/bench_x64

/bench_x64

/bench_x64

Great stuff!

Great stuff2!

Performance results based on clockticks comparison with main HEAD (higher %25change shows improvement):%0A%0A|arch|engine|phase|%25Change_(1-Head/Patch)|%0A |-|-|-|-|%0A|x86_64|patch|Compilation|0.052620|%0A|x86_64|patch|Instantiation|0.063459|%0A|x86_64|patch|Execution|0.000083|

Performance results based on clockticks comparison with main HEAD (higher %25change shows improvement):%0A%0A|arch|engine|phase|%25Change_(1-Head/Patch)|%0A |-|-|-|-|%0A|x86_64|patch|Compilation|0.052620|%0A|x86_64|patch|Instantiation|0.063459|%0A|x86_64|patch|Execution|0.000083|

Performance results based on clockticks comparison with main HEAD (higher %25change shows improvement):%0A%0A|arch|engine|phase|%25Change_(1-Head/Patch)|%0A |-|-|-|-|%0A|x86_64|patch|Compilation|0.052620|%0A|x86_64|patch|Instantiation|0.063459|%0A|x86_64|patch|Execution|0.000083|

Performance results based on clockticks comparison with main HEAD (higher %change shows improvement):

arch
engine
phase
%Change_(1-Head/Patch)

x86_64
patch
Compilation
0.052620

x86_64
patch
Instantiation
0.063459

x86_64
patch
Execution
0.000083

Working?


Update to Manage inactive user accounts in Azure AD

Hello - LastSignInDate is key identifier for inactive accounts (which is covered by this page). Another key identifier is an account that is a member of zero groups. A common example scenario is guest accounts that have been removed from all guest memberships via Access Reviews.  If you could publish a Graphi API example and/or PowerShell example showing how to get a list of users that have zero group membership, that would complement the LastSignInDate property. Customers have expressed a need for this because an account still may have a person or bot trying to signin even after that account has been removed from all groups. Thanks!

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

ID: 3e0a0671-953a-7797-6e0c-6616420f4c3b
Version Independent ID: a2de95d1-67b2-bcc6-402f-f5d23c53d13a
Content: How to manage inactive user accounts in Azure AD
Content Source: articles/active-directory/reports-monitoring/howto-manage-inactive-user-accounts.md
Service: active-directory
Sub-service: report-monitor
GitHub Login: @MarkusVi
Microsoft Alias: markvi

@tfurstms
Thanks for your feedback! We will investigate and update as appropriate.

I have created a backlog item to track this.
#please-close


Support for LISTEN/NOTIFY

Is there support for LISTEN/NOTIFY as per this? I didn't see any examples in the docs and not sure how to go about it.

You can access this using the C API in LibPQ.libpq_c, the function is listed here: https://www.postgresql.org/docs/14/libpq-notify.html
However, receiving notifications is unfortunately tied to consuming input or executing queries. If you're fine waiting for queries, you can call PQnotifies to get the next notification. But if you want to receive notifications asynchronously without executing queries, you might have some challenges working around what is currently in LibPQ.jl. Right now, PQconsumeInput is only called when processing asynchronous queries, and so the code in LibPQ.jl for calling it is highly coupled to the code for fetching asynchronous results. It's also not supported behaviour to operate on a connection from multiple threads (see https://www.postgresql.org/docs/14/libpq-threading.html) so really there needs to be an internal rearchitecture to decouple this functionality.

thanks for the pointers - I'll have a look and will post a mwe here before closing the issue. At the moment, I'm not interested in async or multi-threaded operation.

hi @iamed2 - I managed to identify whenever a notification is sent but I am struggling to get the payload.
My code is the following:
using LibPQ
using ConfigEnv
dotenv()

lpqc = LibPQ.libpq_c

function onenotify()
    conn = lpqc.PQconnectdb(ENV["EXPCONNSTR"])
    res = lpqc.PQexec(conn, "LISTEN virtual")
    lpqc.PQclear(res)

    while true
        notify = lpqc.PQnotifies(conn)
        if notify != C_NULL
            @show notify
            lpqc.PQfreeNotify(notify)
            break
        end
        lpqc.PQfreeNotify(notify)
        lpqc.PQconsumeInput(conn)
        sleep(1)
    end
    lpqc.PQfinish(conn)
end

Calling this function results in:
julia> onenotify()
notify = Ptr{Nothing} @0x0000000004cbed50

julia>

any hints on how to convert this to LibPQ.libpq_c.pgNotify ?
Thank you for your time and apologies for my poor C...

Ah, I noticed that there is a pgNotify and a PGnotify, and the PQnotifies function is set to return the latter, which is an alias to a void pointer. I recommend changing the return type of PQnotifies to Ptr{pgNotify}, and then you should be able to get that data using unsafe_load. Note that the string fields of the struct are still pointers to libpq-allocated memory, so if you want to pass the data back to the user after freeing the memory you need to copy those strings (use unsafe_string for that), otherwise you might see segfaults.
Even without changing the return type of the ccall in PQnotifies, you could get the data out by converting:
tmp = unsafe_load(Ptr{lpqc.pqNotify}(onenotify()))

Thank you - I raised https://github.com/invenia/LibPQ.jl/pull/245 with the fix you suggested.
Closing the issue for now. Once I've played around with this some more, I might raise another PR that allows for the handling of these messages a little more cleanly. I am thinking something similar to conninfo(ci_ptr::Ptr{libpq_c.PQconninfoOption}) but for notifications.


Update README with instructions on getting depdencies and serving

This are the commands I had to run to get the docs building from scratch on an Ubuntu machine. On MacOS as far as I can tell the pre-installed ruby is fine so just the gem install + bundle exec are needed

thanks @driazati, the PR is now merged!


More classic / legendary games parsed in tests

I think it'd be cool to add a few more PGNs of great games to our tests. It'd also be a good to know that more irl PGNs can be parsed :)
For a good reference of how to go about adding a new PGN, checkout the tests directory. I imagine that PGNs themselves would be added to tests/Shared.elm, and tests added to tests/ClassicPgnsTests.elm.
If you're interested but still have questions, let me know!
Also, if you find a PGN that fails to parse, I'd love it if you still made a PR adding it to the test suite so we can give addressing any issues it uncovers in this library a go.

Hi, I'm interested in this one. Could you assign this for me? :)

@carlosfrodrigues that would be fantastic! i've assigned this ticket to ya, let me know if i can help in any way!

Closing now but would always welcome more of these classic games, thanks again @carlosfrodrigues!


feat: Add clap to dcore

Just to make arg parsing easier and allow to measure clap overhead in a small
and controlled environment.

Codecov Report
Attention: Patch coverage is 0% with 126 lines in your changes are missing coverage. Please review.

Project coverage is 80.43%. Comparing base (6c6bd85) to head (7ee002e).

Files
Patch %
Lines

dcore/src/main.rs
0.00%
126 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #631      +/-   ##
==========================================
- Coverage   80.64%   80.43%   -0.21%
==========================================
  Files          97       97
  Lines       23702    23763      +61
==========================================
  Hits        19114    19114
- Misses       4588     4649      +61

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


Tab manager Extension

If any one interested go for it..... build a tab manager Extension.

I would like to contribute. please can you add hacktober fest tag to this.

Assigned go for it


Implement loom data conversion functions

Fixes #24
Ultimately, I think we should implement our own write_zarr function for Loom (and potentially contribute it to the loompy package) and then write new loaders, similar to the anndata zarr loaders, and then read the zarr-based files directly (rather than using the JSON formats)

Have you experimented with https://anndata.readthedocs.io/en/latest/anndata.read_loom.html?

Have you experimented with https://anndata.readthedocs.io/en/latest/anndata.read_loom.html?

No, I had not seen that, I will try it

No, I had not seen that, I will try it

I have removed the LoomWrapper class and just updated the create_cells_json of AnnDataWrapper to support spatial coordinates. AnnData's read_loom works great

I have removed the LoomWrapper class and just updated the create_cells_json of AnnDataWrapper to support spatial coordinates. AnnData's read_loom works great


Typos and error in segs_game_mysql_create.sql

When running dbtool -f create and using a MySQL database there are a couple of typos in Projects/CoX/Data/mysql/segs_game_mysql_create.sql.
Lines 13-15: dropping "emails" table needs to be done before dropping "characters" table because of constraints:
DROP TABLE IF EXISTS `characters`;
DROP TABLE IF EXISTS `accounts`;
DROP TABLE IF EXISTS `emails`;

should be
DROP TABLE IF EXISTS `emails`;
DROP TABLE IF EXISTS `characters`;
DROP TABLE IF EXISTS `accounts`;

Line 77: string contains a semi-colon in date instead of a colon, which causes an error.
(8, 'emails', 0, '2018-09-23 08:00;00');
should be
(8, 'emails', 0, '2018-09-23 08:00:00');
Lines 117-119: First ADD CONSTRAINT statement on line 118 has a semi-colon at the end of the line instead of a comma:
ALTER TABLE `emails`
  ADD CONSTRAINT `emails_ibfk_1` FOREIGN KEY (`sender_id`) REFERENCES `characters` (`id`) ON DELETE CASCADE;
  ADD CONSTRAINT `emails_ibfk_2` FOREIGN KEY (`recipient_id`) REFERENCES `characters` (`id`) ON DELETE CASCADE;

should be
ALTER TABLE `emails`
  ADD CONSTRAINT `emails_ibfk_1` FOREIGN KEY (`sender_id`) REFERENCES `characters` (`id`) ON DELETE CASCADE,
  ADD CONSTRAINT `emails_ibfk_2` FOREIGN KEY (`recipient_id`) REFERENCES `characters` (`id`) ON DELETE CASCADE;

Expected behavior
Creates new databases and tables, or replaces old tables if used with -f switch.
Actual behavior/symptoms
Causes error and fails to create or replace tables correctly.
Steps to reproduce behavior/symptoms

Change settings.cfg to use MySQL.
Run dbtool create (if first time), or dbtool -f create to force replacing databases
Fails to fully create database and tables.

SEGS version
0.5.0
Operating system
Fedora 28
C++ compiler and version
g++ (GCC) 8.1.1 20180712 (Red Hat 8.1.1-5)
Other relevant information

Great find @whitegryphon!!!
If you're up for it, please feel free to submit a PR correcting these issues (and any counterparts that may exist in the postgresql and sqlite databases)

Looking through the files a second time, there doesn't seem to be any problem with neither pgsql nor sqlite files. Great finds @whitegryphon !


Update documentation about PREreview on Sciety

Summary
PREreview is a platform, resource center and convener. We provide ways for feedback to preprints to be done openly, rapidly, constructively, and by a global community of peers.
Our mission is to bring more equity and transparency to scholarly peer review by supporting and empowering communities of researchers, particularly those at early stages of their career (ECRs) and historically excluded, to review preprints in a process that is rewarding to them.
PREreview is a fiscally sponsored project of the non-profit organization Code for Science and Society.
Evaluation model
The PREreview platform is a free and open-source website for the crowdsourcing of preprint reviews (PREreviews).
As a registered user (PREreviewer), you are able to:

Request a PREreview on any preprint with a registered DOI

Fill out a rapid PREreview—a structured form of 12 yes/no/not sure/n.a. questions designed to capture the essence of the preprint

Compose a full PREreview—a text review report that upon submission is assigned a DOI via Zenodo

Invite a co-reviewer, another PREreviewer who can write a full PREreview with you and be recognized as a co-author

Invite a mentor, another PREreviewer who can give you feedback on your full PREreview draft and be recognized as mentor

Comment on others’ full PREreviews

Endorse others’ full PREreviews via Plaudit

Report others’ full PREreviews as violating our Code of Conduct

Join communities of like-minded reviewing groups

Start your own community

As an unregistered user, you are able to:

Search for preprints that have PREreviews or requests for PREreviews on the site

Read and sort through aggregated rapid and full PREreviews alongside the preprint pdf (if available)

Search through PREreviewers and view users’ profiles and contributions to preprint reviews and scholarship

Read more about what you can do our platform and how to join us here.
In the interest of fostering an open and welcoming environment we, as contributors and maintainers, pledge to making participation in our project and our community a harassment-free experience for everyone, regardless of age, body size, disability, ethnicity, gender identity and expression, level of experience, nationality, personal appearance, race, religion, or sexual identity and orientation.
At PREreview we ask all contributors to disclose any competing interest (CI) that may exist between a PREreview author (or affiliated organization) and the author(s) (or affiliated organization) of the reviewed preprint.
Read more about our Code of Conduct here.
People
PREreview is led by a small team of women scientists, all PhDs, all immigrants, all first generation, passionate about making science and scholarship more equitable, transparent, and accessible to all.
Read more about our team here.
Content license
Content is licensed Creative Commons Attribution 4.0 International unless otherwise specified.

Hi @dasaderi here's the markdown file on Sciety that we use to render your group page so feel free to submit a pull request with these changes and the team will review it:
https://github.com/sciety/sciety/blob/main/static/groups/prereview-community--10360d97-bf52-4aef-b2fa-2f60d319edd7.md

Done. I hope I did not mess things up. It's been a while since I did a PR
:D!
https://github.com/sciety/sciety/pull/926
Thanks everyone!
On Thu, Jul 15, 2021 at 3:40 PM Paul Shannon @.***>
wrote:

Hi @dasaderi https://github.com/dasaderi here's the markdown file on
Sciety that we use to render your group page so feel free to submit a pull
request with these changes and the team will review it:
https://github.com/sciety/sciety/blob/main/static/groups/prereview-community--10360d97-bf52-4aef-b2fa-2f60d319edd7.md
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PREreview/prereview/issues/384#issuecomment-880703627,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADI462MW2IRNQGTYQCG5XS3TX3QNPANCNFSM5ANQK3IQ
.

--
Daniela Saderi, Ph.D.
*Co-Founder and Director *
PREreview https://prereview.org
Twitter: @Neurosarda / @PREreview_
Pronouns: She/Her/Hers
Timezone: Pacific Time
*PA: Laura Weall *
@.*** @.**>
Please note that while I might have sent you this email on a weekend or
during a holiday, I do not expect you to reply outside of your work hours.


Cannot see InitCode and Recent Transactions of Contract in Hedera Mirror Node Explorer

Description
After deploying toy smart-contract to Local Network, I can't see initcode and Recent Transactions.
(But smart contract calls work well as intended.)
This is same for TESTNET at Hedera Mirror Node Explorer.
I don't know why.
Any help?
Steps to reproduce

Deploy contract to local network
call smart contract transaction

Additional context

Hedera network
other
Version
latest
Operating system
Linux

Move to hedera-mirror-node-explorer issues
https://github.com/hashgraph/hedera-mirror-node-explorer/issues/603


More fine-grained partial updating for proxy

Describe the problem/challenge you have
When I add unit tests for pkg/agent/proxy, I found that the updating of a field can triggers all flows, routes and ipset updating.
There is no need to updated all flows, routes and ipset when updating a some field. For example, when only updating a NodePort port, it is no need to update flow and route for its corresponding ClusterIP; when only updating port of a ClusterIP, it's no need to update the route for the ClusterIP. As a result, I think we can have a more fine-grained way to update flows, routes and ipset for ClusterIP, NodePort and LoadBalancer.
Describe the solution you'd like
In the following table, I list the every field which should trigger the minimum range updates.

Updated Field
Flows for ClusterIP
Routes for ClusterIP
Flows for NodePort
IPset for NodePort
Flows for LoadBalancer
Routes for LoadBalancer

ClusterIP
Yes
Yes (proxyAll)

Port
Yes

Yes

NodePort

Yes (proxyAll)
Yes (proxyAll)

Protocol
Yes

Yes (proxyAll)
Yes (proxyAll)
Yes

externalTrafficPolicy

Yes (proxyAll)

Yes

internalTrafficPolicy
Yes

Ingress IPs

Yes
Yes (proxyAll)

StickyMaxAgeSeconds
Yes

Yes (proxyAll)

Yes

Could you give some suggestions? @tnqn @jianjuns @antoninbas @wenyingd  Thanks!

When you say "triggers all flows, routes and ipset updating", you mean flows/routes of a single Service right?

When you say "triggers all flows, routes and ipset updating", you mean flows/routes of a single Service right?

Yes, just for a single Service

In that case, updating attributes of a single Service after creation should not be common right? Maybe not worthwhile to optimize?

In that case, updating attributes of a single Service after creation should not be common right? Maybe not worthwhile to optimize?

Yes, you are right, maybe it is not worthwhile. Currently, we have partial updating for LoadBalancer (ingress IPs), and I also was inspired by this. Maybe it is too fine-grained listed in the table, I think we can do like this:

Updating one or more attributes triggers the flows updating for ClusterIP, NodePort and LoadBalancer.
Only updating attribute ClusterIP triggers the route updating for Cluster.
Only updating attribute NodePort or Protocol triggers the ipset updating for NodePort.
Only updating attribute Ingress IPs triggers the route updating for LoadBalancer.

Sure, if you have ideas to optimize without adding too much complexity, it is always good.


[readline-unix] Update, fix cross builds

What does your PR fix?
Updates to 8.2.
Adds mirrors.
Fixes #27909.

Which triplets are supported/not supported?


only generate docs for Mario and Main modules

Looks like I accidentally removed the --codegen flag here: https://github.com/michaelficarra/purescript-demo-mario/commit/66dada670a1e90ff309bacc47595f9625ea0b625#diff-b67911656ef5d18c4ae36cb6741b7965L39

Adding back the --codegen flags didn't fix this. I'm not sure what to do here. I'll try asking on #purescript soon.

This is fixed with PureScript 0.7 rewrite.


Cannot build the software on linux

Hi there i'm just trying out gommerce but having some issue with building the binary...
my operating system is Ubuntu 17.04
my steps are

git clone https://github.com/netlify/gocommerce in my home directory
cd into gocommerce
make deps (got everything downloaded)
make package_linux

my go related env
export GOROOT=/usr/local/go
export GOPATH=$HOME/gocommerce
export PATH=$GOPATH/bin:$GOROOT/bin:$PATH

seems like the test can't run for some reason
If i take out the test from the make file
i get this error
Making gocommerce for linux/amd64
GOOS=linux GOARCH=amd64 go build -ldflags "-X github.com/netlify/gocommerce/cmd.Version=`git rev-parse HEAD`"
main.go:7:2: cannot find package "github.com/netlify/gocommerce/cmd" in any of:
	/usr/local/opt/go/libexec/src/github.com/netlify/gocommerce/cmd (from $GOROOT)
	/Users/andrew/projects/gocommerce/src/github.com/netlify/gocommerce/cmd (from $GOPATH)
make: *** [build] Error 1

Error with test
can't load package: package _/root/go/github.com/netlify: cannot find package "_/root/go/github.com/netlify" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify (from $GOPATH)
can't load package: package _/root/go/github.com/netlify/api: cannot find package "_/root/go/github.com/netlify/api" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify/api (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify/api (from $GOPATH)
can't load package: package _/root/go/github.com/netlify/assetstores: cannot find package "_/root/go/github.com/netlify/assetstores" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify/assetstores (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify/assetstores (from $GOPATH)
can't load package: package _/root/go/github.com/netlify/calculator: cannot find package "_/root/go/github.com/netlify/calculator" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify/calculator (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify/calculator (from $GOPATH)
can't load package: package _/root/go/github.com/netlify/claims: cannot find package "_/root/go/github.com/netlify/claims" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify/claims (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify/claims (from $GOPATH)
can't load package: package _/root/go/github.com/netlify/cmd: cannot find package "_/root/go/github.com/netlify/cmd" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify/cmd (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify/cmd (from $GOPATH)
can't load package: package _/root/go/github.com/netlify/conf: cannot find package "_/root/go/github.com/netlify/conf" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify/conf (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify/conf (from $GOPATH)
can't load package: package _/root/go/github.com/netlify/context: cannot find package "_/root/go/github.com/netlify/context" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify/context (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify/context (from $GOPATH)
can't load package: package _/root/go/github.com/netlify/coupons: cannot find package "_/root/go/github.com/netlify/coupons" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify/coupons (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify/coupons (from $GOPATH)
can't load package: package _/root/go/github.com/netlify/mailer: cannot find package "_/root/go/github.com/netlify/mailer" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify/mailer (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify/mailer (from $GOPATH)
can't load package: package _/root/go/github.com/netlify/models: cannot find package "_/root/go/github.com/netlify/models" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify/models (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify/models (from $GOPATH)
can't load package: package _/root/go/github.com/netlify/payments: cannot find package "_/root/go/github.com/netlify/payments" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify/payments (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify/payments (from $GOPATH)
can't load package: package _/root/go/github.com/netlify/payments/paypal: cannot find package "_/root/go/github.com/netlify/payments/paypal" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify/payments/paypal (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify/payments/paypal (from $GOPATH)
can't load package: package _/root/go/github.com/netlify/payments/stripe: cannot find package "_/root/go/github.com/netlify/payments/stripe" in any of:
	/usr/local/go/src/_/root/go/github.com/netlify/payments/stripe (from $GOROOT)
	/root/go/src/_/root/go/github.com/netlify/payments/stripe (from $GOPATH)

Thanks in advance
Cheers!
A

This sounds like you are running into a quirk of Go. The software needs to be cloned to a location where the path ends with github.com/netlify/gocommerce, otherwise it cannot find the subpackages.
I have GOPATH=$HOME/go and clone repos into ~/go/src/..., so gocommerce would be in ~/go/src/github.com/netlify/gocommerce.
Give that a shot and report back. See https://golang.org/doc/code.html#Workspaces for more details.

thanks @brycekahle that did it. im such a go noob.


Add compatibility with Very Good CLI

Add compatibility with Very Good CLI flavors

You just create configuration file name follow the flavor name that Very Good CLI generated for example (development, staging and production).

icons_launcher-development.yaml
icons_launcher-staging.yaml
icons_launcher-production.yaml

Then just run dart run icons_launcher:create --flavors development,staging,production
Read more for flavor here


sshephalopod needs a configuration mechanism

Right now, a lot of things are hard-coded in the js -- eg the duration of the signature, the ssh options which are applied. It would be better to pull these out into some kind of configuration Thing so that they could be adjusted without having to redeploy the code.

After some conversations, I'm going to start by putting a config object into S3, using the same bucket that we use to persist the CA keypair. The config object will (initially) be a JSON document, which looks like:
{
  "sshOptions": [
    "no-agent-forwarding",
    "no-user-rc"
  ],
  "signatureDuration": 3600
}

We've thought quite a bit about caching the configuration with each Lambda function, and the subsequent cache-busting and cached version consistency challenges, and decided not to do that at all -- we'll just fetch the config every time. After all, right now the code pulls in the IDP metadata, private and public keys every run anyhow, so what's one more?

External config now implemented


up doc

Codecov Report

Merging #351 (a4c5601) into master (e9adc85) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #351   +/-   ##
=======================================
  Coverage   87.06%   87.06%
=======================================
  Files          17       17
  Lines        1933     1933
=======================================
  Hits         1683     1683
  Misses        250      250

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 e9adc85...a4c5601. Read the comment docs.


BETA: Video Capture in v2

There is a video capture implementation in version 2 coming up🎉✨😊
Still in beta, following items need to finish:

[x] Add Rest-Api to capture (like still) -> will not add for now.
[ ] Add tests
[x] finalize routes (start, stop)
[x] add keyboard to trigger video
[x] add keyboard to stop video
[x] add gpio to trigger video
[x] add gpio to stop video
[ ] refine handling of scaled versions in gallery
[ ] option to print video? (also true for gif) or just disable for now?
[x] share video via QR works?
[x] improve countdown in UI
[ ] framerate customizable in admin backend
[ ] boomerang (later v2)

currently out of scope:

audio recording.

Hi,
really looking forward to the final implementation.
I think some parameters like video-lenght and framerate are customizable (at least in the backend-code).
Is there a possibility to add a functionality to export the video in a boomerang-style gif?

Video length default is a parameter in admin backend.
Framerate not yet, but I add it because it's nice.
boomerang is not planned for the first release currently. Also I think mp4/h264 makes more sense than a gif as h264 is widely supported and messenger like whatsapp can convert easily if desired by the user.

The video is captured from the mainbackend ?
If the mainbackend is not support liveview no video capture right ?
The new version is great

after a while the liveview screen flip to a random screen. Is the a switch in the config to deactivate this flip?

Found it "Show Automatic Slideshow Timeout" set to 0

The video is captured from the mainbackend ? If the mainbackend is not support liveview no video capture right ?
The new version is great

As long as no second backend is configured, it tries to get video / preview from main backend. If a second one is configured, that is used for live/video exclusively and main only for stills.

sorry do not get this .
I have a webcam configured   EnumImageBackendsMain (it is realy named like this) with gphoto2 my Nikon D3100 (no liveview possible) and EnumImageBackendsLive with Webcamcv2 and a webcam .
so any chance to use the webcam for video recording without changing the gphoto2 ?

Yes, use main gphoto and live webcamcv for video. Gphoto will capture only stills in this case.
Earlier you referred to use webcamcv2 two times which is just not working that way.

sorry for confusion ( i would prefer to discuss in german ) I'm testing both releases.
The question regards webcamcv2 was related to the actual release. The idea here was to reduce hardware for a smaler Booth.
The question regards Video is for the version 2

beside the 2 Typos.
The video catrue via Webcam is working :-) I try to view the video not on the raspberry attached screen in chrome but on my pc connected to the webserver on my pi with firefox.
In chrome on localhost:8000 it is shown.
Strange but it is an issue with my Firefox setup

Version 2 is released now: https://github.com/photobooth-app/photobooth-app/releases/tag/v2.0.0
The original issue is closed.


Transcript config

Generally looks good. I think dvol config --help which shows which keys can be set would aid discoverability. Otherwise, please address my comments and then merge. Thanks!

@lukemarsden can I get a re-review with the new text please.
Re --help I agree, but I don't know what is yet possible with Cobra. I will bear this in mind when I am developing this.

Oh, yeah, I was happy for you to merge after addressing my comments to your satisfaction. Thanks, merging now.


Python3 and Twisted

Twisted now supports the Python3-style async def functions. This is via a new API called ensureDeferred.
However, the tx side of txaio only uses maybeDeferred in txaio.as_future and maybeDeferred does not support async def. So, we should add some logic and tests to txaio.tx.as_future so that it (maybe) calls ensureDeferred in the correct circumstances.
Either that or convince Twisted that maybeDeferred should do this. Or maybe we just need to call ensureDeferred for all cases?

This fixed by https://github.com/crossbario/txaio/pull/132


Creating sprase image before compressing

In order to get smaller compressed image, place allocated by deleted files shall be zeroed.  This is called “sparse image”.  According to https://ext4.wiki.kernel.org/index.php/Ext4_VM_Images#Good_Tools_for_Making_Sparse_Images:_fstrim.2Fe2fsck.2Fzerofree to create a sparse image one has to call e2fsck -E discard $loopback.
• Call e2fsck -E discard $loopback on the resulting image.

The command e2fsck -E discard $device does not seem to discard the free space. Even e2fsck -f -y -D -E discard $device does not work.
On contrary, the zerofree of dd if=/dev/zero of=zerofile;sync;sync;rm zerofile works just fine.
There is also a possibility to create a sparse file at the end, if you are not compressing it by fallocate -d image

Duplicate of #59


[FFMPEG QSV][Sample decode] use derived data instead of sw copy to improve the performance

Patches to illustrate the issue:
ffmpeg:
https://github.com/fulinjie/ffmpeg/commit/b0c99fb53433145a04d1819a38674c4601cd6eab
msdk:
https://github.com/fulinjie/MediaSDK/commit/c19a4210e1edeaed6e14e92ccbc34088d1e3906b
Test cases:
4K input -> Decode + CSC -> output to null (or /dev/null)
CMDLINE:
ffmpeg -hwaccel qsv -c:v h264_qsv -i Nature.h264 -vf scale_qsv=format=rgb32,hwdownload,format=rgb32 -f null -
./sample_decode h264 -i Nature.h264 -vaapi -rgb4 -o /dev/null
Tested on APL:
sample_decode:
59.98 fps
ffmpeg qsv:
3.4 fps -> 61 fps
After applying these patches in ffmpeg qsv and MSDK core library, performance can be greatly improved(nearly 20x), and meets the performance of sample_decode.
Performance improvement could also be observed on core platform (KBL).
The possible root cause can be that:
Currently, in libmfx core library, sw copy is used to copy derived data
to system memory.
Since the derived data in video memory could be used directly (like sample decode does), sw copy can be redundant under some situations. And if the frame size is large, the performance can be greatly influenced.
Is it possible to add  a by-pass for derive image data directly to DST without copying? (If it has already existed somewhere in the core?)

Hi @fulinjie ,
I'd like to request more info about the change. I don't understand it well. Mostly, I don't understand where the improvement comes from.
MSDK can produce either system or video memory. To get the ball rolling, the first question is in this case above, how MSDK is initialized by ffmpeg ? To produce system or video memory?
Basically, there can be two modes when app needs eventually system memory at output (e.g. to dump data to disk):

app initializes MSDK with produce video memory. Application gets video memory and 2) maps(vaDeriveImage/vaMapBuffer) it to system memory.

app initializes MSDK with produce system memory. MSDK internally decodes to video memory and then internally 2) makes copy from video memory to system memory. It can be done by sw copy("vaDeriveImage->vaMapBuffer->memcpy->vaUnmapBuffer->vaDestroyImage)", or GPUCopy. Application 3) gets system memory.

Which mode are you talking about?
P.S. I'm aware of very slow gtt map access on BXT. As far as I remember it was resolved by "GPUCopy".

There is also vaGetImage/vaPutImage API which might be better in performance...

Yes, I know. But it's not applicable for MSDK as far as I understand.

Hi @fulinjie ,

Hi @dmitryermilov , thanks for the detailed answer for my question.
I'm seeking for performance improvement solution for ffmpeg qsv as a requirement from customer:
HW Decode + CSC (rgb4) and dump to system memory.

I'd like to request more info about the change. I don't understand it well. Mostly, I don't understand where the improvement comes from.
MSDK can produce either system or video memory. To get the ball rolling, the first question is in this case above, how MSDK is initialized by ffmpeg ? To produce system or video memory?

To produce system memory eventually like the two modes you provided.

Basically, there can be two modes when app needs eventually system memory at output (e.g. to dump data to disk):

1 ) app initializes MSDK with produce video memory. Application gets video memory and 2) maps(vaDeriveImage/vaMapBuffer) it to system memory.

This is exactly the way we want to process Decode and CSC in video memory and maps to system memory.

1 ) app initializes MSDK with produce system memory. MSDK internally decodes to video memory and then internally 2) makes copy from video memory to system memory. It can be done by sw copy("vaDeriveImage->vaMapBuffer->memcpy->vaUnmapBuffer->vaDestroyImage)", or GPUCopy. Application 3) gets system memory.

Above cmdline in ffmpeg qsv works in this mode.
What I did with the provided patches seems to force mode 2 (Map/memcpy/Unmap/system memory) to act like mode 1(vaDeriveImage/vaMapbuffer/system memory).

Which mode are you talking about?

Currently works in mode 2, and mode 1 is the expected way to improve the performance without memory copy.
(Since 4K frame is large and additional sw copy (even if gpu copy) for each frame may greatly influence the performance)
I think we'd better include mode 1 in ffmpeg qsv as well.
Any hints for the IOPatterns/flags/Code path in libmfx core library will be very helpful.

P.S. I'm aware of very slow gtt map access on BXT. As far as I remember it was resolved by "GPUCopy".

Yes, we used to enbale gpucopy to dump nv12 data for performance improvement, but in this case,
gpu copy improved from 3.7 fps to 24 fps in APL and didn't meet the bench mark of sample decode.
And that's the reason for a better solution.

I think there is a misunderstanding. Let's resolve it together.
A few points to discuss:

It's still not clear where 20x improvement comes from. It can't be due to system->system copy. At least I'm very skeptical about it(but of course I can be wrong). But it can be due to inefficient "video->system" copy.

Basically, there can be two modes when app needs eventually system memory at output (e.g. to dump data to disk):

1 ) app initializes MSDK with produce video memory. Application gets video memory and 2) maps(vaDeriveImage/vaMapBuffer) it to system memory.

This is exactly the way we want to process Decode and CSC in video memory and maps to system memory.

In this case you won't be able to use GPUCopy because gpu copy functionality in hidden inside msdk.

What I did with the provided patches seems to force mode 2 (Map/memcpy/Unmap/system memory) to act like mode 1(vaDeriveImage/vaMapbuffer/system memory).

What is the difference here? Which calls do you mean under "Map/Unmap"

Any hints for the IOPatterns/flags/Code path in libmfx core library will be very helpful.

Sure! Actually while writing comments above I realized that I didn't pay attention that there is CSC in the pipeline. Probably the perf problem is due to this. For highest performance, for the "decode->CSC->file dump" pipeline memory shared between decoder and VPP should be video memory, memory outputted by VPP should system memory (produces by GPUCopy). Do you agree? It's important to get an agreement here. If so, can you please check that decoder is initialized with IOPattern=MFX_IOPATTERN_OUT_VIDEO_MEMORY, VPP is initialized with IOPattern=MFX_IOPATTERN_IN_VIDEO_MEMORY | MFX_IOPATTERN_OUT_SYSTEM_MEMORY ?

It's still not clear where 20x improvement comes from. It can't be due to system->system copy. At least I'm very skeptical about it(but of course I can be wrong). But it can be due to inefficient "video->system" copy.

Observed in "perf top" log,  copyVideoToSys() is heavily used (highlighted in red). That's the reason for my suspect.
Same issue exits in ffmpeg vaapi as well, and could also be addressed by eliminating the copy from derived data (mapped) to destination data.

Basically, there can be two modes when app needs eventually system memory at output (e.g. to dump data to disk):

1 ) app initializes MSDK with produce video memory. Application gets video memory and 2) maps(vaDeriveImage/vaMapBuffer) it to system memory.

This is exactly the way we want to process Decode and CSC in video memory and maps to system memory.

In this case you won't be able to use GPUCopy because gpu copy functionality in hidden inside msdk.

Yes, there is no memory copy exists in this pipeline if I got it correctly.

What I did with the provided patches seems to force mode 2 (Map/memcpy/Unmap/system memory) to act like mode 1(vaDeriveImage/vaMapbuffer/system memory).

What is the difference here? Which calls do you mean under "Map/Unmap"

Avoid the memory copy in mode 2 could address this performance issue according to my test.

Any hints for the IOPatterns/flags/Code path in libmfx core library will be very helpful.

Sure! Actually while writing comments above I realized that I didn't pay attention that there is CSC in the pipeline. Probably the perf problem is due to this. For highest performance, for the "decode->CSC->file dump" pipeline memory shared between decoder and VPP should be video memory, memory outputted by VPP should system memory (produces by GPUCopy). Do you agree?  It's  important to get an agreement here.

I've tried the this pipeline with:https://github.com/fulinjie/ffmpeg/commit/cefed3e301f8997662d7fe8b49a08023ebcf1d52
pipeline memory shared between decoder and VPP is video memory, and output memory for VPP is system memory (use GPUCopy), the result in APL is 24fps (double confirmed by customer).
I thought the highest performance may depend on the fourcc/format. (not quite sure)
For Tiled format (like NV12), gpu copy may have the best performance, since Tiled format data in video memory would be accessed slowly if we use deriveImage/Mapuffer simply.
For Linear format (like rgb32), derive/map and then access the linear data in video memory may have the best performance, since linear data in video memory has no difference compared with copying them into system memory.

If so, can you please check that decoder is initialized with IOPattern=MFX_IOPATTERN_OUT_VIDEO_MEMORY, VPP is initialized with IOPattern=MFX_IOPATTERN_IN_VIDEO_MEMORY | MFX_IOPATTERN_OUT_SYSTEM_MEMORY ?

I thought the highest performance may depend on the fourcc/format. (not quite sure)

Yes, I agree.

Observed in "perf top" log, copyVideoToSys() is heavily used (highlighted in red). That's the reason for my suspect.

It means that GPUCopy is not involved:) But it's focus on it a bit later.
Well, I think I realized what these two patches do. There is one idea/concern. I need you help to check.
Since you already have an environment, can you please run(with applied patches ):
ffmpeg -hwaccel qsv -c:v h264_qsv -i Nature.h264 -vf scale_qsv=format=rgb32,hwdownload,format=rgb32 -f /dev/null
and share resulted fps? Will it be ~61 or the ~3.4? I'll explain the idea later.
I'm internationally interested into writing to a file (even if it's dummy like /dev/null/).

I thought the highest performance may depend on the fourcc/format. (not quite sure)

Yes, I agree.

Observed in "perf top" log, copyVideoToSys() is heavily used (highlighted in red). That's the reason for my suspect.

It means that GPUCopy is not involved:) But let's focus on it a bit later.
Well, I think I realized what these two patches do. There is one idea/concern. I need your help to check.
Since you already have an environment, can you please run(with applied patches ):
ffmpeg -hwaccel qsv -c:v h264_qsv -i Nature.h264 -vf scale_qsv=format=rgb32,hwdownload,format=rgb32 -f /dev/null
and share resulted fps? Will it be ~61 or the ~3.4? I'll explain the idea later.
I'm internationally interested into writing to a file (even if it's dummy like /dev/null/).

Provided cmdline is not runnable.
IMHO, what you intended to test may be
'ffmpeg -hwaccel qsv -c:v h264_qsv -i Nature.h264 -vf scale_qsv=format=rgb32,hwdownload,format=rgb32 -f rawvideo /dev/null '
For some network reasons, APL is not accessible currently.
Instead, I tested on KBL (50+ fps -> 90+ fps after applying these two patches)
Results:
Pipeline 1. -f null -:  97 fps
Pipeline 2. -f rawvideo /dev/null: 55 fps
Before these patches:  about 50+ fps
And the reason is that there is extra memory copy in raw_encode() for Pipeline 2 to copy data from AVFrame to AVPacket for file dump.
https://github.com/FFmpeg/FFmpeg/blob/47b6ca0b022a413e392707464f2423795aa89bfb/libavcodec/rawenc.c#L52
If result in APL is still needed, will provide next week.

If result in APL is still needed, will provide next week.

Yes, please share them.

On APL,

ffmpeg -hwaccel qsv -c:v h264_qsv -i Nature.h264 -vf scale_qsv=format=rgb32,hwdownload,format=rgb32 -f rawvideo /dev/null

frame=  103 fps=1.4 q=-0.0 Lsize= 3337200kB time=00:00:01.81 bitrate=15033598.0kbits/s speed=0.0239x

ffmpeg -hwaccel qsv -c:v h264_qsv -i Nature.h264 -vf scale_qsv=format=rgb32,hwdownload,format=rgb32 -f null -

frame= 1585 fps= 57 q=-0.0 Lsize=N/A time=00:00:26.54 bitrate=N/A speed=0.956x

Thanks for quick reply!
Well, the figures above prove my concern - yes, we can eliminate MSDK internal mecopy and return pointers obtained from vaMapBuffer to application (ffmpeg is app for MSDK in current case) but access to it 'mapped' memory works extremely slowly! Do you see my point? Access to this memory can be inside MSDK or on app side, but anyway this access will exist somewhere in real usage case.
Running with " -f null -" may say that performance 20 times higher but it's a false impression.
Accessing to this memory from downstream component (whatever it is: file writer, sw encoder, sw screen renderer) will kill this performance.

Well, the figures above prove my concern - yes, we can eliminate MSDK internal mecopy and return pointers obtained from vaMapBuffer to application (ffmpeg is app for MSDK in current case) but access to it 'mapped' memory works extremely slowly! Do you see my point? Access to this memory can be inside MSDK or on app side, but anyway this access will exist somewhere in real usage case.

Got your point.
A concern is that since the "mapped memory" is accessible,  the slowly access should be an issue for APL (atom) and need to be fixed in media driver, like https://github.com/intel/media-driver/issues/620
I think @FurongZhang is working on this.

Running with " -f null -" may say that performance 20 times higher but it's a false impression.
Accessing to this memory from downstream component (whatever it is: file writer, sw encoder, sw screen renderer) will kill this performance.

Is there the same issue(false impression) in sample_decode?
Customers may take sample_decode as the bench mark and require a matched performance for ffmpeg-qsv.

Although I agree that results from KBL is a point for discussion. I mean on systems where gtt map works fast, indeed, an additional memcopy might be a bottleneck.
Actually a few months ago we had several discussions about it. We came to a conclusion that we need to make libva map to system memory provided by app. Currently libva doesn't provide such API. Somehow we abandoned further discussions of libva proposal for a while..

Is there any issue/PR on this filed in libva repo or somewhere else?
We may need to trace this.
From above discussion,

if directly map access is slow, use gpu copy could get the best performance.
if directly map access is quick, use mapped memory and avoid additional memcopy could get the best performance.

So IMHO, a query in libva may also be needed whether data of specific format could be fast accessed on specific platform in directly mapping.
Hi @XinfengZhang , how do you think of these 2 API requirements?

Is there the same issue(false impression) in sample_decode?

In case of output system memory from MSDK, MSDK "touches" this slow gtt mapped memory in copyVideoToSys.

Is there any issue/PR on this filed in libva repo or somewhere else?
We may need to trace this.

Not yet..

From above discussion,

if directly map access is slow, use gpu copy could get the best performance.
if directly map access is quick, use mapped memory and avoid additional memcopy could get the best performance.

So IMHO, a query in libva may also be needed whether data of specific format could be fast accessed on specific platform in directly mapping.
Hi @XinfengZhang , how do you think of these 2 API requirements?

I fully agree with you and ready to take part in this proposal discussion.

I fully agree with you and ready to take part in this proposal discussion. It still would be great why GPUCopy doens't work in the case. @fulinjie , do you have bandwidth to investigate it?

Sure.
GPUCopy takes effect for both ffmpeg and sample_decode to improve the "actually access" performance from 1.4~3.4 fps to 24~30 fps.
There may be some "false impression" exist in previous sample_decode test (59 fps result).
For ./sample_decode h264 -i Nature.h264 -vaapi -rgb4 -o /dev/null, the performance is 4.5 fps on APL actually.

BTW, @fulinjie , JFYI: intel/media-driver#623 . Probably we should check if the same can improve system memory access on BXT.

Thanks a lot, will work on framework side to support better performance.

I fully agree with you and ready to take part in this proposal discussion. It still would be great why GPUCopy doens't work in the case. @fulinjie , do you have bandwidth to investigate it?

Sure.
GPUCopy takes effect for both ffmpeg and sample_decode to improve the "actually access" performance from 1.4~3.4 fps to 24~30 fps.

So it works?! Great! Because originally from the conversation above I got an impression that it doesn't work on BXT.

There may be some "false impression" exist in previous sample_decode test (59 fps result).
For ./sample_decode h264 -i Nature.h264 -vaapi -rgb4 -o /dev/null, the performance is 4.5 fps on APL actually.

Right. It's because output memory from MSDK is video memory. App accesses it externally (not inside msdk) so GPUCopy isn't involved.

@fulinjie can we close this one?

@daleksan, this could be closed at this stage since query in libva is needed at first. And we can use hwmap=mode=direct explicitly to force deriving data instead of sw copy.


Supporting GitLab search API

With large (many repo) projects, I find it common to use the GitLab search (in the browser) to try and find projects of which I know only part of the name for sure. Is there some interest in exposing this part of the API (1) into lab? For example, lab project search?
I see #208 implements the issue side already, probably it could be included in project and in mr also.

@tfurf lab project list has some search and filtering functionality. https://zaquestion.github.io/lab/lab_project_list.html
Does this work for what you're looking for?


Composite key break multi row results

It seems that if using composite key, the result of function is forced to first row
CREATE TABLE t1 (
	u1_id INTEGER,
	u2_id INTEGER,
	name TEXT,
	PRIMARY KEY (u1_id, u2_id)
);

-- get-all-u1.sql
select * from t1 where u1_id = :u1_id limit 100

Thanks for reporting, I'll look into that.
In the meantime, to avoid getting stuck on this error, you can try the following:

Modify the query slightly to ensure TypeSQL generates multiple rows. For example, use SELECT * FROM t1 WHERE u1_id > :u1_id LIMIT 100 (using > instead of =).
Let TypeSQL generate the boilerplate code.
Delete the .sql file or rename it to .txt (for example) to avoid regenerating the code.
Update the generated code with the correct query.

This should be fixed in version v0.14.3.


Legacy definitions in DistrhoUIPrivateData

There remains a use site of unused DISTRHO_PLUGIN_TARGET_VST, following the VST3 update in develop branch.
https://github.com/DISTRHO/DPF/blob/a8f912a7d3debd4b01333dee2780412599c08d30/distrho/src/DistrhoUIPrivateData.hpp#L33

Fixed in 97bb090e3b89a863b6657d485a26c105f2667435 thanks


WebView2 currently does support the .NET 5

This doc (https://docs.microsoft.com/en-us/microsoft-edge/webview2/get-started/winforms) indicated that WebView2 is not supported by designer, but I just managed to drag-n-drop the component on the designer and was able to set a few properties.
What isn't supposed to work, exactly ?

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

ID: 48ba6a76-eb0b-e8f2-dfc9-d2c0df6261a4
Version Independent ID: 8d62fdee-f44a-3d40-6800-04d7bd6193f8
Content: Get started with WebView2 for WinForms apps - Microsoft Edge Development
Content Source: microsoft-edge/webview2/get-started/winforms.md
Product: microsoft-edge
Technology: webview
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel

Apologies about the late reply. It seems like this page was updated in the meantime, and no longer mentions that designer isn't supported. So I will close this issue.


Helpers for reporting controller errors as K8s events

Description
Fixes #321
Review Checklist

[x] Code contains useful comments
[x] (Integration-)Test cases added (or not applicable)
[x] Documentation added (or not applicable)
[x] Changelog updated (or not applicable)

bors r+


CAMEL-12343 : refer to new constructor with defaultCamelContext for deprecated constructor

Migrate from 2.19.2 to 2.20.2 failed.
java.lang.IllegalArgumentException: CamelContext must be specified on: Message[]
	at org.apache.camel.util.ObjectHelper.notNull(ObjectHelper.java:342)

Lets wait for the 2.21.0 release first.

Same error with 2.21.0 from repository
https://repository.apache.org/content/repositories/orgapachecamel-1103/

That's a deprecated method. We can fix this after the release.

By the way, the deprecated constructors can be removed in 2.22.0, since we deprecate them in 2.20

This PR is invalid, the code should be as-is

I understand. Ok to remove deprecated constructor in 2.22.0. So, i will change my code to use the new constructor to avoid exception.

By removing deprecated constructor, i can detect error at compile time instead of runtime.

Conclusion : no Backward compatibility between version 2.19.2 and 2.20.2 because i need to change my code. I really prefer to detect error at compile time instead of runtime...


GPL-nnn Clean up / refactor Pacbio Run

User story
As a developer, I would like to refactor the building of a Pacbio Run as it is starting to become unwieldy.
Who are the primary contacts for this story
Harriet
Steve
Ben
Some possible clean up areas are:

pacbio run/plate/well/library etc requests to GraphQL as Controller is starting to get unwieldy
csv generator look at, starting to get quite big
well factory spec is getting quite big - could pull out nested classes in the factory

BUG: cant remove libraries from a well

BUG: cant remove libraries from a well


Stream Error / Socket Error in Swift 5 & Xcode 11 Beta

I'm building a new app, targeting iOS 13 and using SwiftUI so am using Xcode 11 beta.
After receiving a message, the connection is then unexpectedly terminated:
Connected to MQTT server.
Subscribed to topic 'state'.
Message received:  Hello world
2019-07-14 16:55:49.128126+0100 MyApp[3575:118403] SocketStream read error [0x600001580540]: 1 54
Server disconnected: None
Server disconnected: Stream Error
Server disconnected: Stream Error
Connection failed:  Socket Error

This sounds very much like #39 and #42, however I've tried the Swift 5 and Xcode 10.2 fixes in  in @anatoliykant's and @kervich's forks with no luck. Has anyone tested SwiftMQTT in Xcode 11 beta yet?
Thanks!

@jamesbebbington I didn't try iOS 13 beta yet, however the issue you describe seems to be a different one - at least I see that you can subscribe to a channel and receive a message. I suggest that you first run the tests from SwiftMQTT to see what exactly is going wrong there (you might be getting ECONNRESET due to not responding to ping or something like that)

Hey @kervich many thanks for getting back to me, I shall take a look.


Warning that may result in errors or download from an untrusted repo

Command:
stable-ts test.mp4 --model medium --model_dir D:*******els\ --output testmedium.srt --language Japanese --vad True --vad_threshold 0.35 --demucs True --refine
Produces the following warnings:

******\Lib\site-packages\torchaudio\functional\functional.py:1458: UserWarning: "kaiser_window" resampling method name is being deprecated and replaced by "sinc_interp_kaiser" in the next release. The default behavior remains unchanged.
warnings.warn(
\Lib\site-packages\torch\hub.py:286: UserWarning: You are about to download and run code from an untrusted repository. In a future release, this won't be allowed. To add the repository to your trusted list, change the command to {calling_fn}(..., trust_repo=False) and a command prompt will appear asking for an explicit confirmation of trust, or load(..., trust_repo=True), which will assume that the prompt is to be answered with 'yes'. You can also use load(..., trust_repo='check') which will only prompt for confirmation if the repo is not already trusted. This will eventually be the default behaviour
warnings.warn(

The second warning should not show up after loading VAD for the first time.
For first time loading, both warnings should be not showing up after a6b2b05568e75b1602a6e23891b59c4a9e218f6b.

I have tested the video: https://www.youtube.com/watch?v=1NfFIpZocWs against:

large-v4

medium with command (since dialogues and music is sort of mixed up):
stable-ts test.mp4 --model medium --model_dir D:*******els\ --output testmedium.srt --language Japanese --vad True --vad_threshold 0.35 --demucs True --refine

medium model using just:
stable-ts test.mp4 --model medium --model_dir D:*******els\ --output testmedium.srt --language Japanese

The result is:
A) Large-v4 wins (not only in quality but also has best sync),
B) Second place (strangely command no. 3 outputs better) and
C) Last is option 2 (command taking more time)??


Random question....

Does disabling the Voice Chat button disable hearing other players? Or does it only disable talking?

The "Voice chat" setting toggles voicechat completely, so you will not be able to talk and you will not be able to hear others.

Thanks a lot!


Need to use --volumes-from flag to use share data between containers

Hi
--volumes-from is missing
To use this flag we need to know / set container name - we cannot set container name by marathon API, also container id is not appears in localhost:8080/v2/apps/app_name so it is impossible link between two containers.
My suggestion is
1.to extend existing json structure to :
"volumes": [
{
"containerPath": "",
"hostPath": "",
"mode": "RO"
"from" : app_name
}
]
In case "from" will be filled "containerPath" , "hostPath" & "mode" can be optional
2.provide container id inside localhost:8080/v2/apps/app_name
Thanks,

how can i mount a volume from  ?

any update on how this could be achieved as of current versions ?
Thanks


Fix crash on umd and module merge, allow umds to be accessed when merged with a non-UMD symbol

Fixes #28762

Works for me 😄

@danielrosenwasser since this fixes a crash, do we want to port it to 3.2?

This is an active 3.2 blocker for a few of our repos so I'd very much appreciate it 😄

Yesterday's nightly should've included the fix, so installing typescript@next should work for the interim.

Thanks, works for me 😀


Added encryption scope blob sas

API changes have been detected in this PR. You can review API changes here
API changes
+         public String getEncryptionScope()
+         public AccountSasSignatureValues setEncryptionScope(String encryptionScope)
+         public String getEncryptionScope()

API changes have been detected in this PR. You can review API changes here
API changes
-         V2020_10_02("2020-10-02");
+         V2020_10_02("2020-10-02"),
+         V2020_12_06("2020-12-06");
+         public String getEncryptionScope()
+         public BlobServiceSasSignatureValues setEncryptionScope(String encryptionScope)

/azp run java - storage - tests

API changes have been detected in com.azure:azure-storage-file-datalake. You can review API changes here
API changes
-         V2020_10_02("2020-10-02");
+         V2020_10_02("2020-10-02"),
+         V2020_12_06("2020-12-06");

API changes have been detected in com.azure:azure-storage-queue. You can review API changes here
API changes
-         V2020_10_02("2020-10-02");
+         V2020_10_02("2020-10-02"),
+         V2020_12_06("2020-12-06");

/azp run java - storage - ci

/azp run java - storage - ci

/azp run java - storage - ci

/azp run java - storage - ci

/azp run java - storage - ci

/azp run java - storage - ci


Choose endpoints intelligently

As of https://github.com/coreos/etcd/commit/7d172bb1142eee5ee3bb6b2b4f08e2390217e01f etcd/client.Client round-robins across all given endpoints. This is not ideal. The client should pin itself to a given endpoint until that endpoint becomes unhealthy. The endpoint should be chosen randomly in order to achieve a pseudo-random distribution of all clients across the cluster's members.

Some good advice from @andrewnicols : https://github.com/coreos/etcd/issues/2549#issuecomment-84753442

The client could pin itself to a given endpoint until that endpoint becomes unhealthy or there is cluster change.
The client will randomly connect to a member when reset, but it cannot achieve a pseudo-random distribution of all clients across the cluster's members under all cases, e.g., server restart, add member. We don't see users meet the problem of server overload today, and don't know how to address it best. So I would collect more feedback and tackle the problem in the future.


use annotated description as Field description

ref: https://github.com/tiangolo/fastapi/discussions/9278
History
Since FastAPI 0.95 we cannot have multiple endpoint routes with optional Path parameter (because we cannot mix the Query/Path for a variable:  https://github.com/tiangolo/fastapi/commit/375513f11494bc3499050ad2a0d378fb6e37ca98#r105161665), while it seems this breaking change is assumed, I personally still need this feature.
To solve this, I have to remove the usage of Query/Path class and use simple Annotated form, but this means that I loose the nice OpenAPI kwargs provided by FieldInfo.
Proposed change
With this PR I'm proposing that we use the first metadata annotation as Field's description
from typing import Annotated
from fastapi import FastAPI

app = FastAPI()

@app.get("/test")
async def variable(foo: Annotated[str,  "get a description"] = "bar"):
   return {"foo": foo}

in ☝ "get a description" will be forwarded to the OpenAPI schema as field's description 👇

I originally designed a more complex solution over https://github.com/tiangolo/fastapi/pull/9289, but I think what I'm proposing with this PR is much more simple (thus acceptable ? 😅).

📝 Docs preview for commit 60d163dce555d898c526503914a56c12508195f4 at: https://648898509e9ccd2141608062--fastapi.netlify.app

Is there anything I need to do here? 🙏


Should first start just do build?

When the user gets system running it tells them to hit START but then they get an error about needing to build.
Maybe if initial build hasn't happened just go straight to build?

Moved to https://github.com/Zonama/ZonamaDev/projects/2


Remove Limited GA from SSCA docs

Thanks for contributing to the Harness Developer Hub! Our code owners will review your submission.
Description

Please describe your changes: __________________________________
Jira/GitHub Issue numbers (if any): ______________________________
Preview links/images (Internal contributors only): __________________

PR lifecycle
We aim to merge PRs within one week or less, but delays happen sometimes.
If your PR is open longer than two weeks without any human activity, please tag a code owner in a comment.
PRs must meet these requirements to be merged:

[ ] Successful preview build.
[ ] Code owner review.
[ ] No merge conflicts.
[ ] Release notes/new features docs: Feature/version released to at least one prod environment.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://664100a9c06afef939214eb9--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://664100d4329d2afe87ce1bd1--harness-developer.netlify.app


Separate main js

So we can use require('remotedev-server')(argv) to start remotedev local server.

I think it will be easy to write code directly in a webpack server script: https://github.com/rackt/redux/blob/master/examples/counter/server.js

I like it :+1:
Please share an example when you incorporate it with Webpack.


Modify specific class

Is there a way to modify a specific class when saving?
I need to hide some buttons that doesn't need to be shown in the images :)

You can probably use the option modifyStyle. It takes a function that receives the text of a CSS rule, including the selector and styles, and returns a replacement.

I can't figure out how to use this option modifyStyle and the index does not contain any example :(
Could you provide a simple example of its use? That'd be great!


flutter_browser_app can't run on Flutter 2.0.2

Environment

Technology
Version

Flutter version
2.0.2

App version

Android version

iOS version

Xcode version

Device information:
Description
Expected behavior:
pod --version
1.10.1
flutter --version
Flutter 2.0.2 • channel stable • https://github.com/flutter/flutter.git
Framework • revision 8962f6dc68 (7 days ago) • 2021-03-11 13:22:20 -0800
Engine • revision 5d8bf811b3
Tools • Dart 2.12.1
Current behavior:

So i can't run this app in iOS, please check.
Thanks.

Didn't test it with Flutter 2.0.2 yet.
However, because some dependencies currently used don't support null-safety, you should run it with the --no-sound-null-safety flag enabled.

Didn't test it with Flutter 2.0.2 yet.
However, because some dependencies currently used don't support null-safety, you should run it with the --no-sound-null-safety flag enabled.

which version of Flutter do you contributor use, I just wanna use the same environment as you currently use.

Because of a bug from Flutter itself (see https://github.com/flutter/flutter/issues/77488), the latest flutter version that should be used with Android Hybrid Composition enabled is Flutter 1.27.0-4.0.pre.

sir, can you show output of flutter --version
so, we can set according it
because it is not working in my computer
thanks

Hi All, you can take here https://github.com/AleksandrSokolov/flutter_browser_app Null-safety version that works with flutter 2.0 / dart 2.12.
Tested on
[√] Flutter (Channel stable, 2.0.6, on Microsoft Windows [Version 10.0.19042.985], locale ru-RU)
• Flutter version 2.0.6 at C:\opt\flutter
• Framework revision 1d9032c7e1 (3 weeks ago), 2021-04-29 17:37:58 -0700
• Engine revision 05e680e202
• Dart version 2.12.3

Hi, I have run this project on flutter 3.0, but get such of errors. Should I use a lower version of flutter, or how should I modify it, I would appreciate answers...

@zaunist You can use this #11


Caffe's bvlc_googlenet and Lasange's GoogLeNet do not produce the exact same results

I have experimented with Lasagne's googlenet.py and I have noticed that although it produces results similar to Caffe's bvlc_googlenet they are not exactly the same. (By results I mean the output of the network after the softmax or the probabilities)
Should they produce the same results?

How much do they differ? Is this on CPU or GPU? Do you use cuDNN?

I use GPU and cuDNN for both caffe and lasagne. Also I have just installed Theano and Lasagne from master today.
The l2 norm of their difference for an example image is 0.14590697. (the example image the is cat image used in the caffe tutorial)
Here is a plot of the difference, as I said they are very similar.

Well this was not a good example (since some might think the preprocessing is not the same for both).
For the input np.zeros((1, 3, 224, 224), dtype=np.float32) the l2 norm of the difference is 0.096159138.
Also here is the plot for plt.plot(lasagne_out - caffe_out).

Hi @yassersouri, thanks for your report.
It looks like several of the pooling layers should use ignore_border=False but are not. My guess is that when I wrote this I was using an older version of Lasagne (before https://github.com/Lasagne/Lasagne/pull/374).
With this change I get an l2 norm of the difference for zero input of 9.7e-8, I will push an updated version shortly.

@ebenolson Thanks.

Oh, good catch! Sadly, this also means it won't be able to use cuDNN for pooling any more (it always ignores the border).

Does other models in the model zoo need similar updates?

I don't think so - I seem to remember this being a peculiarity of googlenet (I think this is why I was using both Pool2DLayer and Pool2DLayerDNN).
However, it would be good to verify that, and add some numerical tests to catch any future problems. PR's welcome :smile:


Support CVE allowlist expiry date

Is your feature request related to a problem? Please describe.
Harbor allows setting an expiry date for allowlists. This provider does not.
Describe the solution you'd like
The provider should let us set an expiry date alongside the cve allowlist.
Additional context

I have written a PR that addresses this system wide but not on individual projects. We can probably use the same logic here.
https://github.com/goharbor/terraform-provider-harbor/pull/276
The PR just needs some attention from @Vad1mo.

thanks the PR has beend merged
will be available with next release


migration for clusterServices

LGTM - only comment is aesthetic - and this might be less specifically for this PR and more for the whole k2recon migration effort - a few times node is set as a fact and then not used which raises 2 questions, does it need to be set in the first place and additionally can we just pass the task 'cluster' as a singular variable and not do the whole with_subelements loop? I also might be misunderstanding something - let me know what you think.

If node is not used, then it generally should not be set, because it is then it means we are doing the same thing for each node pool when it really only needs to be done once per cluster. Patrick's provider ticket shows an example of this.

should be good to go


Captioned block numbering

It would be nice to have them optionally numbered.

The numbering part of this is easy. We'd just keep a counter figure embeds and tables, and label them accordingly. The citation part is the trickier issue, since we'd need to update the syntax for embeds and tables to that they have a particular ID that we can cite elsewhere. I suppose we could add that at the end of an embed or table using some special ID syntax, kind of like LaTeX does with label tags.

To align this with the labels, we would:

Optionally parse labels in captioned content
Use the labels to assign IDs to the captioned content
Ensure links can refer to the captioned content's ID
Ensure links scroll to the captioned content correctly

Reduced scope to numbering since the new labeling feature mostly supports the citation part of this. We would just put a :myfancyfigure label in the caption or above the figure and then cite it with [Figure 1|:myfancyfigure]. The only part that wouldn't work is the automatic numbering of figures. We could do some feature where links with no content get automatically labeled.


Buffer data in RP calls exhausting gitlab log limit

While running playwright suites on CICD, the gitlab log limit is exhausted because of buffer request data in calls to Report Portal

Error: Request body larger than maxBodyLength limit
URL: https://reportportal.company.engineering/api/v1/company/log
method: POST
Request data: {"type":"Buffer","data":[45,45,56,51,57,56,53,51,50,53,51........

Gitlab console

Job's log exceeded limit of 10240000 bytes.
Job execution will continue but no more output will be collected.

Hello @chirag-fynd !
The issue was related to client-javascript, fixed in<EMAIL_ADDRESS>Please reinstall the agent-js-playwright to get the latest version of client-javascript.


Trying to uninstall the app crashes package installer

I tried all possible ways to uninstall it but same problem, using a third party apps crashes it too.

I'm on android 13

It's the crash of your ROM's package manager.
try adb uninstall com.dede.android_eggs
There is no other way for you to provide feedback to your device manufacturer

It only happens with this app, how is this our fault? I've installed from F-Droid, and experience the same behaviour.

Update this Apk to uninstall Easter Eggs.
uninstall_easter_eggs_9.9.9_10000-release.apk
Read this document to learn more

Thank you, it is fixed now


[RFC] Add support for optional flow types

Adds support for optional types. Only supports optional values, not optional params.
With this diff
type maybeTypes = {
  a: ?string,
  b?: string,
  c?: ?string
}

will generate
type maybeTypes = Js.t {.. a : Js.null_undefined string, b : string, c : Js.null_undefined string };

but I think it should generate b as Js.undefined string.
It breaks external function declarations. I could use a pointer to how I can distinguish between a type and an external declaration.
Example:
declare function someFunction(a?: string): string

before this diff, would generate something like
external someFunction : a::string? => string

now it generates something like:
external someFunction : a::Js.null_undefined string? => string

PR Checklist

[ ] Corresponding issue: #0000
[X] Formatted code with refmt
[X] Ran tests npm test and updated snapshots

Hi! I'm not Ryan but thought I'd stop by in case we can both learn something. Here's my thought process from skimming the code:
My guess is there's a gap between Flow's optional keys and the Bstype.t type used to represent the Bucklescript side of things. I'm looking here where we define Object as holding (list (string, t)). To me, that says that we don't recognize/remember whether Flow represented any of the object's props as optional, so we wouldn't be able to print the code correctly.
What I'd probably do is to change that type somehow. The most immediate option though less readable is Object (list (string, bool, t)), where bool is whether it's optional or not.
Then, we need to get that info from Flow decls, maybe around here.
Finally, we print code for optional props differently, to get the results you want above.
I'm assuming here that we want to be precise/pedantic and understand that optional keys are technically different from nonoptional keys with optional values (which basically boils down to using Js.undefined for optional keys, AFAICT). I could be wrong and we might not really care about the distinction, but I figure for large-scale automatic conversions it's probably best to be exact :).
Does that help? Does my reasoning seem on-point?

Hey @dwwoelfel! Thanks so much for this PR 😄 Hope everything went smoothly when developing it!
This is a great change, and anything that can get us closer to generating the exact types is a huge step forward. I do think that @bbqbaron is right, we should be generating Js.null_undefined for nullable types, as in the Flow documentation a maybe type can be null, undefined, or the value. An optional field however should be Js.undefined, because with Flow an optional field can only be undefined, null is a valid value.
This introduces the need for another type! Right now we put optional fields and maybe types into BsType.Optional t, but the use cases here point out that we could also have BsType.Nullable t. BsType.Nullable would be for nullable values that can be either undefined, null, or the value. So where we put BsType.Optional now would become BsType.Nullable, and then the optional fields would become BsType.Optional. Would that make sense? I think with the Nullable and Optional split the function output should be fixed as well!
I think @bbqbaron is correct in saying we should be precise 😃 The unofficial stance of ReasonablyTyped is to be precise*, rather than shaky. A passing test should mean the output has the same semantics as the input.

to an acceptable degree within a reasonable amount of time

I'd love to help out on doing this! Feel free to hit me up on Discord to work through any problems!

Closing because ReasonablyTyped is going through a substantial rewrite


dlangspec.pdf: table of contents

before/after:

Nice. I thought the TOC was broken because of some Latex invocation weirdness (you have to run it twice or something)

Thanks!


Consume remained stdout and stderr from buffer

Backend::Exec can't get entire stdout or stderr due to race condition between specinfra and spawned process.
This PR will consume remained data in buffer.
These specs include sleep 1 to raise error surely.
But, these specs will fail without sleep very rarely(below 1/100 on my environment)
IMHO, original code remain data cause following scenario.

child process write stdout and close it
child process write stderr
specinfra read from stdout and catch EOFError
exit thread although some text remain in buffer of stderr

Thanks!

Released as v2.56.1.


[ADD ME TO THE ORGANIZATION]

Name: Leroy
Location: NY
Interest: I like to learn new tech
Discord username(optional):-
Why do you want to join?

I want to join to learn from this community

You can also join our Discord community here
Feel free to check out other cool repositories of the DevsInTech Community here.

@coolidev  Thanks for your interest. Invitations have been sent to you. Kindly accept it and make your membership public.


treat all handle callbacks as asyncio tasks

It is better to consider all confirmation handlers as asynchronous tasks.
Some few tests around performance are needed

It seems like there was an issue with mypy (false positive) during await of the callback.
Should be workarounded as described here:
https://github.com/python/mypy/issues/7587

going to merge per conversation with @DanielePalaia


[Bug]: Error when trying to generate with BingAI

Contact Details
<EMAIL_ADDRESS>What happened?
It shows this error:

Steps to Reproduce

Just chat with Sydney however you want.
After a while it starts to do this no matter how you message it.

What browsers are you seeing the problem on?
Chrome
Relevant log output
None
Screenshots
Attached
Code of Conduct

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

This is not reproducible on my end, as is the case with many Bing issues. However, there are certainly many things you can try.
You can search previous issues for this, as well as look for common solutions posted on our discord. To join the discord server, use this link: https://discord.librechat.ai/
Once you join, @fuegovic has organized these into a discord thread which you can visit here: https://discord.com/channels/1086345563026489514/1143941308684177429
You can also search the issues on discord.
It would help to note what you have tried once visiting possible solutions. I'll implement SOCKS5 proxy support which should help assuming you have an IP outside of the US as well.


Add linux node affinity to dapr_dashboard helm chart (#1846)

Description
Updated dapr-dashboard helm chart
Issue reference
Please reference the issue this PR will close: #1846
Manual Test
willardstanley@MININT-2KH1IKT cli % kubectl get pods
NAME                                     READY   STATUS    RESTARTS   AGE
dapr-dashboard-7b48557b9f-gskjg          1/1     Running   0          15s
dapr-operator-7b57d884cd-4vhj9           0/1     Running   0          15s
dapr-placement-86b76f6545-blb84          1/1     Running   0          15s
dapr-sentry-7c7bf75d8b-c2g9t             1/1     Running   0          15s
dapr-sidecar-injector-7b847db96f-vjcsr   1/1     Running   0          15s

Checklist
Please make sure you've  completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Unit tests passing
[ ] End-to-end tests passing
[ ] Extended the documentation / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Specification has been updated / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]
[ ] Provided sample for the feature / Created issue in the https://github.com/dapr/docs/ repo: dapr/docs#[issue number]

/ok-to-test


merge dev into master

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

Comparison is base (dfca9c4) 52.98% compared to head (30003a4) 53.16%.
Report is 74 commits behind head on master.

Files
Patch %
Lines

...ations_widgets/injured_count_by_severity_widget.py
47.82%
12 Missing :warning:

...ed_and_injured_count_per_age_group_widget_utils.py
68.42%
6 Missing :warning:

...ets/no_location_widgets/vision_zero_bike_widget.py
0.00%
4 Missing :warning:

...s/killed_and_injured_count_per_age_group_widget.py
71.42%
2 Missing :warning:

...dgets/urban_widgets/vision_zero_10_50_90_widget.py
0.00%
2 Missing :warning:

anyway/widgets/widget_utils.py
71.42%
2 Missing :warning:

anyway/flask_app.py
75.00%
1 Missing :warning:

anyway/models.py
0.00%
1 Missing :warning:

...ously_injured_killed_in_bicycles_scooter_widget.py
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2544      +/-   ##
==========================================
+ Coverage   52.98%   53.16%   +0.18%
==========================================
  Files         117      117
  Lines        9645     9670      +25
==========================================
+ Hits         5110     5141      +31
+ Misses       4535     4529       -6

Flag
Coverage Δ

unittests
53.16% <59.74%> (+0.18%)
:arrow_up:

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

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


The "options.agent" property must be one of type Agent-like Object, undefined, or false. Received type function

During a current gatsby build with version 2.1.0 as of today.
prismic-javascript version: 2.1.0
node version: v10.3.0
gatsby-source-prismic-graphql: 3.3.1
Build Error:
The "options.agent" property must be one of type Agent-like Object, undefined, or false. Received type function
Seems related to recent agent updates merged today which is setting agent to a function and some strict type checking.
  fetchOptions.agent = changeAgent;
function changeAgent(parsedUrl: any) {   return (parsedUrl.protocol === 'http:') ? httpAgent : httpsAgent; }

Same here, reverting back to v2.0.3 fixes it.

Also having the same problem! Like @adam404 said, it's a type checking issue with the new http/https handling. Not my wheelhouse, but it's unclear if changeAgent is getting passed parsedUrl, which is why it's providing a function as a value instead of the result of the function.

This is also breaking things over here, cheers https://github.com/birkir/gatsby-source-prismic-graphql/issues/58

We just released a patch 2.1.1 that fixes the issue.
It was related to our conditional agent that was not supported by the version on cross-fetch.
We updated that dependency so you are able to do so but removed the implementation by default so you won't have any limit their.
Sorry for the inconvenience.


python3Packages.aiohttp-remotes: switch to pytestCheckHook

Motivation for this change
Switch to pytestCheckHook
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 145909 run on x86_64-linux 1

1 package marked as broken and skipped:

doh-proxy

4 packages built:

python38Packages.aiohttp-remotes
python38Packages.connexion
python39Packages.aiohttp-remotes
python39Packages.connexion

Result of nixpkgs-review pr 145909 run on x86_64-linux 1

1 package marked as broken and skipped:

doh-proxy

4 packages built:

python38Packages.aiohttp-remotes
python38Packages.connexion
python39Packages.aiohttp-remotes
python39Packages.connexion


Add Extended priorities support from Work Tab mod.

Extended priorities now can be saved/loaded correctly.

Please compile assemblies by yourself and don't forget to update a documentation.

Wow! This is great @TheLonerD ! I'll test this, merge it and then release! Thank you very much!

Sure! If you found some bugs just post a message here so I can fix it.

There is one bug when saving the valuesInner to the save file using Scribe_Collections.
The issue is the that scribe is expecting to write (list of) Values but instead it gets a collection (with lists of values).
File: WorkLink.cs
foreach (KeyValuePair<WorkGiverDef, List<int>> entryInner in settingsInner)
{
           keysInner.Add(entryInner.Key);
           valuesInner.Add(priorityPerHour);
 }
Scribe_Collections.Look(ref settingsInner, "settingsInner", LookMode.Def, LookMode.Value, ref keysInner, ref valuesInner);

results in this in the SaveFile (which does not work):
	<settingsInner>
	<keys>

FightFires

	</keys>
	<values>

System.Collections.Generic.List`1[System.Int32]

	</values>

It shoud instead something like:
	<settingsInner>
	<keys>

FightFires

	</keys>
	<values>

	         <prioritiesPerHour>

3

3

	            ....24 entries...

3

	          </prioritiesPerHour>

	      	    <prioritiesPerHour>

3

3

	              ....24 entries...

3

	          </prioritiesPerHour>
	</values>

I've found a solution based on how Flufly saves the priorities in WorkTab. It requires back and forth translations but it works!

Sounds good! Let me check how Fluffy did that, as I understand we need to create a some kind of translation layer to handle proper serialization<->deserializaton.

@TheLonerD  I just pushed v2.5 with the integration and with the save/load fully working. I'd like to add you as contributer in steam. Can you share your steam_id so I can add you as contributor to the mod in steam?

Yup. My steam profile is https://steamcommunity.com/id/LonerD/
On Sun, Oct 22, 2023, at 2:42 AM, Alexandre wrote:

@TheLonerD https://github.com/TheLonerD I just pushed v2.5 with the integration and with the save/load fully working. I'd like to add you as contributer in steam. Can you share your steam_id so I can add you as contributor to the mod in steam?
—
Reply to this email directly, view it on GitHub https://github.com/voult2/BetterPawnControl/pull/40#issuecomment-1773938659, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABSBWN3AR5ACU32BWDWTGFLYARFWBAVCNFSM6AAAAAA6KAIXFOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZTHEZTQNRVHE.
You are receiving this because you were mentioned.Message ID: @.***>


Sanitycheck device fixes

Some fixes and improvements for sanitycheck's behavior when running on a real device.
CC: @doanac

Codecov Report

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

@@           Coverage Diff           @@
##           master   #13158   +/-   ##
=======================================
  Coverage   48.73%   48.73%
=======================================
  Files         315      315
  Lines       46540    46540
  Branches    10743    10743
=======================================
  Hits        22679    22679
  Misses      19349    19349
  Partials     4512     4512

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 0a7b45d...d3c8e8b. Read the comment docs.

@mbolivar - thanks. This change helped me find the actual error in my environment.


Support @SchemaMapping and @BatchMapping for schema interface types

Instead of requesting to re-open #294, I'm creating a new issue.
First off, thank you for this library! It has come a long way since I first started looking into Java + GraphQL over 2 years ago. There have been a lot of great QoL improvements everywhere! Thank you!
Between issues #294 and #236 the suggestion was to create a base batching function and then write a new batching function for each concrete type citing that DGS and NestJS don't support the functionality requested. I'd like to revisit this.
Here's my use-case:
interface Activity {
    id: ID!
    completedBy: [User!]!  # requires a dataloader
    labels: [Lable!]!  # requires a dataloader
    # other fields here...
}

type FooActivity implements Activity {
    # repeat Activity fields here...
    foo: Foo!
    # other fields related to Foo...
}

type BarActivity implements Activity {
    # repeat Activity fields here...
    bar: Bar!
    # other fields related to Bar...
}

Query {
    activities(...): [Activity!]!
}

I should be able to write a single test that executes activities()  testing all of the Activity fields which should always pass no matter whether I have one or ten concrete implementations of Activity (though, per the GraphQL spec, I must have at least one).
Right now, I only have a handful of Activity types, but I plan to add more in the future. As I do, with the current state of the library, I will have to just remember to make sure to implement dataloaders for each new concrete type or risk a 500 error on a frontend query that is fetching one of the .users or .labels attributes.
I would like to point out that Python's Graphene library supports setting resolvers on an Interface type and therefore dataloaders so that I don't have to implement the same resolvers on all concrete types (Sadly, this isn't clearly documented as an example, otherwise I'd post a link. I've used it in production code and it just works). That being said, I have to usually supply a TypeResolver to help it out a bit.
Here is what I expect to work:
interface Activity {
    // ... fields here
}

public class FooActivity implements Activity {}
public class BarActivity implements Activity {}

@BatchMapping
Mono<Map<Activity, List<User>>> completedBy(List<Activity> activities) {
    // use repositories to fetch data
}

However, here is my workaround:
// just a regular function on the controller
Mono<Map<Activity, List<User>>> activityCompletedBy(List<? extends Activity> activities) {
    // use repositories to fetch data
}

// due to type erasure, I have to name each method explicitly
// and therefore must specify the GraphQL field name explicitly
@BatchMapping(field = "completedBy")
Mono<Map<Activity, List<User>>> fooActivityCompletedBy(List<FooActivity> activities) {
    return activityCompletedBy(activities);
}

@BatchMapping(field = "completedBy")
Mono<Map<Activity, List<User>>> barActivityCompletedBy(List<BarActivity> activities) {
    return activityCompletedBy(activities);
}

// same for the `labels` field, just swapping fields.

This is a noticeable gap when migrating from graphql-kickstart, which was able to identify schema mappings to interface based resolvers.

we instead make registrations for each concrete type conditionally
Just to check my understanding, you will register the same batch function for each concrete type automatically unless a concrete type already has a concrete type-specific batch function registered?

If so, that sounds great!

Yes, that is what I meant. This is now available in 1.3.0-SNAPSHOT and we'll be releasing 1.3.0-RC1 tomorrow. If you'd like to give this a try and provide feedback, that would be much appreciated!

I apologize for taking so long to get back to this and for not being able to take a look at the SNAPSHOT or RC versions, but I was able to try out 1.3.0 today and it works perfectly, just as I expect! Thank you so much for your help here!

I apologize for the long delay and for not being able to test out the SNAPSHOT and RC versions, but I was finally able to try out 1.3.0 today and it works exactly as I'd expect. Thank you!


Make strides signed everywhere

After some discussion and introduction of signed strides in the strided view we've concluded to finally make the strides signed :)
This will need a rebind template metafunction thing to change the shape_type to a strides type with a signed integer type (ptrdiff most likely).

Is this a duplicate of #261? 😏

Not totally, we keep unsigned integers for shapes.

Yeah! And this one is actionable and in the near future :) but I agree, we
can probably close the other one afterwards.
Le 14 juin 2018 6:32 PM, "Johan Mabille"<EMAIL_ADDRESS>a
écrit :

Not totally, we keep unsigned integers for shapes.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/QuantStack/xtensor/issues/923#issuecomment-397358019,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2BPjfZHhFlOTA1sEKbmzhQbrSMfd4Xks5t8pAUgaJpZM4UoNfe
.

It has been done!
@ukoethe we've merged the signed strides finally !


Getting default category for all tokens in POS tagger

Hello,
I followed the exact instructions for the Brill POS Tagger in the README but I'm getting 'N' tags for all the tokens. What am I doing wrong? Do I need to train it?

Do you also get the ENOENT error? Then this might be the same issue: https://github.com/NaturalNode/natural/issues/518

Please post your code, so I can have a look. I did not experience any problems, maybe it has something to do with search path.

This is caused by a version mismatch between NPM and Github. I upgraded the version on NPM.


Add human readable tables for source_ids, experiment_ids and licenses

Add the searchable HTML tables.  Are there any other fields in this repo that we should include?

This will require GH pages to be turned on - following the same format as the CMIP6_CVs precursors:
https://wcrp-cmip.github.io/CMIP6_CVs/docs/CMIP6_experiment_id.html
https://wcrp-cmip.github.io/CMIP6_CVs/docs/CMIP6_institution_id.html
https://wcrp-cmip.github.io/CMIP6_CVs/docs/CMIP6_source_id.html etc
found in https://github.com/WCRP-CMIP/CMIP6_CVs/tree/master/docs, with https://github.com/WCRP-CMIP/CMIP6_CVs/blob/master/src/jsonToHtml.py used to generate these every PR merge
Ref PCMDI/pcmdi.github.io/#534


npm install fails on Windows8.1 x64

We're getting the following error:
C:\Program Files (x86)\MSBuild\Microsoft.Cpp\v4.0\Microsoft.Cpp.InvalidPlatform
.Targets(23,7): error MSB8007: The Platform for project 'contextify.vcxproj' is
invalid.  Platform='x64'. You may be seeing this message because you are tryin
g to build a project without a solution file, and have specified a non-default
Platform that doesn't exist for this project. [C:\Users\yossi\node_modules\juic
e\node_modules\jsdom\node_modules\contextify\build\contextify.vcxproj]

Why did you close the issue @leibale ? Did you get it to work?

i changed the system to linux before long time, i have no way to help

If I remember right, this was an issue that was related to jsdom having a dependency on contextify, which had some issues building during npm install. So it should be resolved based on contextify no longer being a dependency of juice since jsdom was replaced with cheerio.


Refactor Windows Bootstrap Agent and Add auto upgrade support.

Related issue https://github.com/rancher/rancher/issues/11303 and https://github.com/rancher/rancher/issues/10662

Move most of the common logic into rancher-tools.psm1
Fix the add host issue.
No more download binaries in bootstrap agent. Download them when running the startup.ps1 script.
Add Windows Agent crash-loop.ps1 script.

@deniseschannon Pls review the PR. Thanks!


Some problem with phantomjs-proxy.

pyspider version: 0.3.7
Operating system: mac os 10.13
When i read the code ,I don't know what this code means?

# phantomjs-proxy
    if kwargs.get('phantomjs_proxy'):
        pass
    elif os.environ.get('PHANTOMJS_NAME'):
        kwargs['phantomjs_proxy'] = os.environ['PHANTOMJS_PORT_25555_TCP'][len('tcp://'):]

Can you please explain this to me?
Thank you !

For docker, when you link phantomjs with fetcher


VMI spec: Add owner reference

Follow the convention from the other checkups, and have the VMI under test to be owned by the checkup's pod.

Tested against an OpenShift Virtualization cluster v4.15.


Getting cloudflare cookies after request is complete

Is there a way to get the cloudflare cookies after doing a request? Can't seem to find anything, might be me being retarded

@TadaWasRaped You need to first create custom jar, pass it to the request and after it was made, read from the jar:
https://github.com/codemanki/cloudscraper/blob/master/docs/examples/session-persistence.js#L9

Something like this:
let jar = cloudscraper.jar();
cloudscraper = cloudscraper.defaults({ jar});
cloudscraper.get(uri).then(() => console.log(jar));


Parser does not resolve full domain call

I have tried the example app, this is working fine.
After integrating the current version into my project, I recognized that actual domain did not fetched correctly.
Test Domain: 'https://www.facebook.com/slackhq/posts/848094788621462'
Issue
It seems that the current version only resolves the domain up to

https://www.facebook.com/slackhq

output for example

data.description: 'Slack. 70,622 likes · 1,360 talking about this. Slack is where work happens. It connects you with the people and apps you work with every day, no matter where you are or what you do.'

Expected
full domain resolve.
output should be

data.description: 'We're happy you're here! If you need help, you might want to search for an answer on our Help Center (https://get.slack.help/hc/en-us), drop an email to<EMAIL_ADDRESS>or find us on Twitter as...'

debugging hint
I already debugged your application and it seems to be an npm dependency issue somewhere

Hi @Justinemmerich !
Sorry for the delay. I was just trying it and everything seems to work fine for me. What version of react-native are you using ?
Would you have a yarn.lock or package-lock.json by any chance please ?
Thanks in advance


Skeleton doc

Added documentation for Skeleton related properties on Box, Text, etc as well as the Skeleton component itself.

Just need to resolve the yarn.lock conflict before merging.