Backport cluster evaluation permissions #1885 to 2.0

Description
Backport of #1885 to 2.0
Issues Resolved
Backport PR #1885
Check List

[x] New functionality includes testing
[x] New functionality has been documented
[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@peternied thanks a lot, I'm working on fixing the issues.

@rutuja-amazon It looks like the build is passing now, but there are some other CI failures that need to be addressed, please make updates for those and then we can see about getting this approved and merged, thanks!

@peternied I have fixed the CI issues and all other checks had been passing as well, we can merge post the all the checks pass for the new changes.


CSS Tweaks section is outdated

CSS Tweaks are now called snippets
There's no obsidian.css anymore. Instead user may create CSS files in vault/.obsidian/snippets/

I suggest adding download button for every snippet

Also, there's a separate repository for many CSS Snippets. Wouldn't it be a better option to link mentioned repo?

Thanks for the suggestion! As you can see, I left some thoughts on this in the issue above ^

Closing in favor of #29


StrongBox keys sometimes have KM_TAG_CREATION_DATETIME set to an enormous value - could also be an issue in the libraries parsing the certificates (BouncyCastle) but that seems less likely

04-26 06:20:01.999  5658  5754 I Attestation: Parsing tag: [701], value: [18446744073709546693]

% factor<PHONE_NUMBER>3709546693
18446744073709546693: 7<PHONE_NUMBER>387078099

So far, I've seen this on GrapheneOS occasionally, and I'm working on trying to reproduce it with the stock OS. It seems like a rare problem and I don't know how to encourage it to happen yet.

I've reproduced this on both GrapheneOS and the stock OS.

A different one this time:<PHONE_NUMBER>3709031998.
% factor<PHONE_NUMBER>3709031998
18446744073709031998: 2 59 46639<PHONE_NUMBER>699

It does seem to be counting like a timer:
>>> 2**64
18446744073709551616
>>> 2**64 -<PHONE_NUMBER>3709031998
519618
>>> 2**64 -<PHONE_NUMBER>3709203078
348538
>>> 2**64 -<PHONE_NUMBER>3709223973
327643
>>> 2**64 -<PHONE_NUMBER>3709332801
218815
>>> 2**64 -<PHONE_NUMBER>3709388049
163567
>>> 2**64 -<PHONE_NUMBER>3709438256
113360

>>> 2**64 -<PHONE_NUMBER>3709535336
16280

It wrapped around and started working again:
04-26 07:23:25.564 19155 22122 I Attestation: Parsing tag: [701], value: [103581]
04-26 07:24:05.990 19155 22904 I Attestation: Parsing tag: [701], value: [145820]

This explains why the failure happens in cycles for long periods of time.

This is worked around by 8137ab5375a66c27b3c5b88c74bd89557ae0a950.


Last include fix

This turns "crypto/foo.h" and "internal/foo.h" #include statements into "<crypto/foo.h>" and "<internal/foo.h>"
It also updates the docs.
Another trivial but tedious PR to review.
Checklist
Thank you for your contribution. Please answer the following:

[X] I acknowledge that I am authorized to submit this code under
the terms of the Apache License
[ ] All new public APIs are documented
[ ] All new public functions have tests
[ ] All new public functions have fuzzing tests

It's not the last, the test header files still need to move.

I used this script to do things in pieces:
#! /bin/sh
DIR=$1

case "$DIR" in
apps|crypto|internal|openssl|providers|test) ;;
*) echo Unknown type $DIR ; exit 1 ;;
esac

for FILE in $(ls include/$DIR ) ; do
    F=$( echo $DIR/$FILE | sed -e 's/.h.in$/.h/' )
    git grep -l "include \"$F\"" > /tmp/x$$
    echo $RF $( wc -l </tmp/x$$ )
    xargs perl -pi -e "s@include \"$F\"@include <$F>@" </tmp/x$$
    rm /tmp/x$$
done


Stateful Storage Pallet Documentation Improvement

Goal
The goal of this PR is to improve the documentation of the Pallets and make that documentation be able to be used on docs.frequency.xyz.
Part of https://github.com/frequency-chain/docs/issues/59
Discussion

Stateful Storage Pallet Readme

Screenshot

Looks good!  Ship it.


Add API to trigger data disk move only

Add a new D-Bus API which allows to trigger the data disk move only. This allows to handle the selection and partitioning from Supervisor directly (via UDisks2 D-Bus API). Handling partitioning directly in Supervisor should improve device detection/selection and allows for better error handling.
Before calling this API, make sure to prepare a data disk using the GPT partition layout and with the partition label (not file system label) of hassos-data-external. Then call the API without argument and reboot the system.
# busctl --verbose call io.hass.os /io/hass/os/DataDisk io.hass.os.DataDisk MarkDataMove

Right, I was somehow under the impression that D-Bus (or the Go library we use for D-Bus) requires a return value, but just tested, that seems not to be the case.
I'll adjust the new API.


Software

Make software in behavior stimulation class accept a list.
Consider a controlled vocabulary for software?

@saskiad do you have any further thoughts on this controlled vocab?

this ticket is already close


Left-align text for specific page or all pages

Hi, thanks for a great theme. New to Hugo, so I apologize if this is a more general question. Could you suggest how I could align-left text for either a specific page or for all pages?
For example, the references to the tools on this page appear strange center-aligned.

Now that you mention it, at least for my preference is for left-alignment over the current justify everywhere. It's an easy change, and I can make a PR to enable it as an option. @fuegowolf thoughts?

I ended up going ahead and making the PR rather than waiting. If merged, the enable site-wide content left-align, you just need to set align_left = true in the site config file (eg. config.toml).

This worked great. Wow, thanks so much!
On Sun, Aug 20, 2017, 11:26 AM Michael Noronha<EMAIL_ADDRESS>wrote:

I ended up going ahead and making the PR rather than waiting. If merged,
the enable site-wide content left-align, you just need to set align_left
= true in the site config file (eg. config.toml).
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_fuegowolf_cocoa-2Deh-2Dhugo-2Dtheme_issues_50-23issuecomment-2D323591972&d=DwMFaQ&c=nE__W8dFE-shTxStwXtp0A&r=K686QAY7eaZdPEFBr8PPgs7NJEd4IX1trEl5UE4sKWQ&m=4toQeaGJX-BLfwPM3BB0lTSken2vu8GsBRzV1S9TSRE&s=KHXLrnImjoeeUFaheyuGgvpFcSHxqos11BCXFf6SDxs&e=,
or mute the thread
https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_AEYh9lM-5Fy4zasMSMpRFKFVBjfgqwl02kks5saFAjgaJpZM4O8n1A&d=DwMFaQ&c=nE__W8dFE-shTxStwXtp0A&r=K686QAY7eaZdPEFBr8PPgs7NJEd4IX1trEl5UE4sKWQ&m=4toQeaGJX-BLfwPM3BB0lTSken2vu8GsBRzV1S9TSRE&s=7-x3qjPiu3rLs-v6qutTNFoCb7bbcdbSFVGtsMPNitk&e=
.

--
Joshua Rosenberg, Ph.D. Candidate
Educational Psychology & Educational Technology
Michigan State University
https://urldefense.proofpoint.com/v2/url?u=http-3A__jmichaelrosenberg.com&d=DwIFaQ&c=nE__W8dFE-shTxStwXtp0A&r=lDfZ8uOz4uHZquAkehbmd8vKaNyOJ4NpD8LJIsNEszGG_Q4g2i8Vvf1oTVVAQsdr_ox1Fnn6TelISIngNmPb2JxAng_dkZNCwdptFwTO-A0KAaPK_8i_ecWUXKPzQivQ7nSbiCJ_eqrzx1SoAinBeA&m=D3xnoTODXpCIo3gfb9Ir8xrRGkNSnY0dIbAQauXCLu8&s=BWupJGLpGlDfSH1_oftKsu1Icird0AD6NoUiOB0Gvfc&e=

Thanks @mtn for the PR ;)


metatensor-learn: next steps

Post (almost merged) Dataset and DataLoader, let's define next steps for metatensor-learn:

[ ] A DataLoader that wraps DataLoaders, for learning on multiple targets and/or Datasets. See: https://github.com/lab-cosmo/metatensor-models/pull/27 in metatensor-models - there is probably a nice way of absorbing this generically into learn.
[ ] Implement ModuleMap
[ ] Loss functions - start with a simple module that calculates the L2 loss between 2 TensorMaps (I have one in rho_learn that we could use as a starting point: https://github.com/jwa7/rho_learn/blob/main/rholearn/loss.py). Plenty of API considerations here: reduction methods, returning global loss, loss per sample, handling sparsity, etc etc

metatensor-operations required for solvers in linear and kernel models

[ ] solve_cholesky (this should be the same doing Cholesky decomposition + triangular solve as done in the kernel)
https://docs.scipy.org/doc/scipy/reference/generated/scipy.linalg.cho_solve.html
https://pytorch.org/docs/stable/generated/torch.cholesky_solve.html
[ ] eigh
https://numpy.org/doc/stable/reference/generated/numpy.linalg.eigh.html
https://pytorch.org/docs/stable/generated/torch.linalg.eigh.html

Follow up Issues from https://github.com/lab-cosmo/metatensor/pull/427:

[ ] merge the construction of test tensor maps https://github.com/lab-cosmo/metatensor/issues/451 to then add tests with multiple blocks
[ ] Figure out to add forward function into doc that is TorchScript compatible: TorchScript allows not # added to appear in doc, :inherited-members: is not compatible with torch
But also does not allow to use super in forward https://github.com/pytorch/pytorch/issues/42885
[ ] add weight and bias  property to Linear (will be probably List[nn.Parameter])
[ ] Consider to also accept single Labels for out_properties for ModuleMap.from_module and Linear default constructor. It is also a bit tricky to the get the list check done in a way that does not break in metatensor-torch. I was not sure if this is useful in the first place, so I skipped this.
[ ] add examples for usage of ModuleMap as mentioned in https://github.com/lab-cosmo/metatensor/pull/427#issuecomment-1849001299

Originally posted by @agoscinski in https://github.com/lab-cosmo/metatensor/issues/427#issuecomment-1911211939

Figure out to add forward function into doc that is TorchScript compatible: TorchScript allows not # added to appear in doc, :inherited-members: is not compatible with torch
But also does not allow to use super in forward https://github.com/pytorch/pytorch/issues/42885

Note that in the recent PR #513 this was solved by moving to compositional modules as opposed to inherited ones


if minDate and maxDate are same Date plugin fails

if minDate and maxDate are same Date plugin fails only take minDate. maxDate not taking in effect

@vaibhav-khot  is there any solutions regarding this?


chore: bump<EMAIL_ADDRESS>

Fixes https://github.com/microsoft/vscode/issues/113883

Curious why the lockfile contains 6 different chokidar versions.

Curious why the lockfile contains 6 different chokidar versions.

@paulmillr likely from dev dependencies that depend on older versions?

@paulmillr likely from dev dependencies that depend on older versions?


[remove_collections] introduce gRPC endpoint

Story

Parent story: https://github.com/MystenLabs/narwhal/issues/70
Description
As part of this task a new gRPC endpoint should be introduced to remove_collections. The description of the endpoint can be found on the document here: https://www.notion.so/mystenlabs/Narwhal-Reference-Mempool-API-6e6268556f1249bc958d32f4bd1d5b77#889a21a374b74a9d8ee38fc40f7489b2
The endpoint should call the block_remover to achieve that.

closed in #224


feat: Prove Measure.add_right_inj for sigma-finite measures

Generalizes a lemma from #7713.

Thanks!
bors r+


Switch Interaction Model client error reporting to just CHIP_ERROR.

Now that we can put a StatusIB inside a CHIP_ERROR, we can stop
passing both, or passing just EmberAfStatus, and pass a CHIP_ERROR
that either contains a StatusIB or contains the actual client-side
error we ran into (e.g. failure to decode).
Problem
We are trying to force a StatusIB through an EmberAfStatus, it's not clear to consumers how the CHIP_ERROR and the StatusIB (or EmberAfStatus or InteractionModel::Status) interact when both are passed, etc.
Change overview
Switch all the IM failure/error callbacks to take CHIP_ERROR.
Testing
Should be no behavior changes for now.  The consumers that could really use this (in CHIPDeviceController, where we are ending up treating a general status as a cluster-specific status) are not using the new APIs yet.

/rebase


Create UpgradeHook for automatic minor update admin-ack

Context
OpenShift requires a manual admin ack when upgrading minor versions. See Providing the administrator acknowledgment.
If we set a release channel in project syn we implicitly acknowledge that a minor upgrade can be done.
We have had cases where we forgot to do the admin ack after a channel upgrade and this did lead to mainenance/on-call alerts.
We should automate the admin ack.
Alternatives
Do manual work and get on-call alerts.

Ack CM from lpg-2
apiVersion: v1
data:
  ack-4.11-kube-1.25-api-removals-in-4.12: "true"
  ack-4.12-kube-1.26-api-removals-in-4.13: "true"
  ack-4.13-kube-1.27-api-removals-in-4.14: "true"
kind: ConfigMap
metadata:
  annotations:
    include.release.openshift.io/ibm-cloud-managed: "true"
    include.release.openshift.io/self-managed-high-availability: "true"
    include.release.openshift.io/single-node-developer: "true"
    release.openshift.io/create-only: "true"
  name: admin-acks
  namespace: openshift-config

Sample cluster version object with {{AdminAckRequired}} condition.
apiVersion: config.openshift.io/v1
kind: ClusterVersion
metadata:
  name: version
spec:
  channel: stable-4.13
status:
  availableUpdates:
    - channels:
        - ...
        - stable-4.13
        - stable-4.14
  conditions:
    - lastTransitionTime: '2023-09-19T20:58:06Z'
      message: >-
        Kubernetes 1.27 and therefore OpenShift 4.14 remove several APIs which
        require admin consideration. Please see the knowledge article
        https://access.redhat.com/articles/6958395 for details and instructions.
      reason: AdminAckRequired
      status: 'False'
      type: Upgradeable

Fixed by #56


Improve connection JSON validation

Add more and clear information rather than just throwing the Pydantic error

Let's divide it into two steps:
If the current code does not support multiple dbs, then let's add the database as required to be consistent
Then, we can work on providing support to have a single service with all the dbs

same as https://github.com/open-metadata/OpenMetadata/issues/4540


Update to nom 6

This PR updates devd-rs to use use Nom 6 instead of 5 to stay inline with the rest of the Rust ecosystem.
If not inconvenient, a patch release would be appreciate of 0.3.2. The only non-transparent change for end users is that the MSRV of Nom went to Rust 1.44, but I don't see a MSRV for this crate.

Huh, a nom update not breaking API? Nice :)


Kernel Assertion Error

Describe the bug
Unable to find kernel credentials in kernel memory
To Reproduce
Steps to reproduce the behavior:
JB
Screenshots

Device (please complete the following information):

iOS Version: 12.4
iOS Device: iPhone 6S
unc0ver Version: 3.5

Place an "x" between the brackets if true:

[ ] this is a bug others will be able to reproduce
[x] this issue is present with all tweaks uninstalled(except for default packages) or disabled
[x] this issue is present after a rootfs restore
[x] this issue is present on the latest version of unc0ver

Logs
If applicable, add logs or error messages here.

Its just some "reboot and it magically works" bug, closing


On MS Edge, clicking "Next Challenge" doesn't do anything

I know this is old, but while testing on the latest MS Edge (vers. 38.xxxx), I was not able to replicate this bug. There's no knowing what went wrong if you don't provide the version of the browser.

Edge may simply not support many js functions.


Vagrant box changed from chef/ubuntu-14.04 to bento/ubuntu-14.04

Today I have tried to run vagrant up in a new project and the box 'chef/ubuntu-14.04' was not found.
If you check the link https://atlas.hashicorp.com/chef/  it says:
"If you are looking for Bento boxes they now have their own organization https://atlas.hashicorp.com/bento/"
This pull request changes the box to 'bento/ubuntu-14.04'.

:+1:


Use field constraint when generating pydantic class

This ensure the generated file passes type checking

Thanks for catching this!


mailto doesn't work

I need help getting Mailrise to send emails through Gmail. I can get it to work for Apprise. I'm able to get Mailrise to work for other services like ntfy. I am using the Gmail app password like Apprise specifies.
configs:

  "*@*":
    urls:
      -<EMAIL_ADDRESS>
tls:
  mode: off

Try the -vv flag to turn on verbose logging. Does anything interesting show up?

Same problem here :
I have set basic authentication without tls
My config
configs:   ntfy:     urls:       - ntfy://<IP_ADDRESS>:9999/fdsjlfd<EMAIL_ADDRESS>    urls:       - ntfy://<IP_ADDRESS>:8888/dfdfkdf smtp:   auth:     basic:       dfdxx: zzzz       testpwd: xxxx tls:   mode: off
but I have now this error

Attaching to mailrise-server
mailrise-server  | /home/mailrise/.local/lib/python3.12/site-packages/aiosmtpd/smtp.py:372: UserWarning: Requiring AUTH while not requiring TLS can lead to security vulnerabilities!
mailrise-server  |   warn("Requiring AUTH while not requiring TLS "
mailrise-server  | [2024-01-06 14:44:53] WARNING:mail.log:auth_required == True but auth_require_tls == False
mailrise-server  | /home/mailrise/.local/lib/python3.12/site-packages/aiosmtpd/smtp.py:372: UserWarning: Requiring AUTH while not requiring TLS can lead to security vulnerabilities!
mailrise-server  |   warn("Requiring AUTH while not requiring TLS "
mailrise-server  | [2024-01-06 14:47:33] WARNING:mail.log:auth_required == True but auth_require_tls == False
mailrise-server  | /home/mailrise/.local/lib/python3.12/site-packages/aiosmtpd/smtp.py:372: UserWarning: Requiring AUTH while not requiring TLS can lead to security vulnerabilities!
mailrise-server  |   warn("Requiring AUTH while not requiring TLS "
mailrise-server  | [2024-01-06 14:49:36] WARNING:mail.log:auth_required == True but auth_require_tls == False
mailrise-server  | /home/mailrise/.local/lib/python3.12/site-packages/aiosmtpd/smtp.py:372: UserWarning: Requiring AUTH while not requiring TLS can lead to security vulnerabilities!
mailrise-server  |   warn("Requiring AUTH while not requiring TLS "
mailrise-server  | [2024-01-06 14:49:44] WARNING:mail.log:auth_required == True but auth_require_tls == False
mailrise-server  | /home/mailrise/.local/lib/python3.12/site-packages/aiosmtpd/smtp.py:372: UserWarning: Requiring AUTH while not requiring TLS can lead to security vulnerabilities!
mailrise-server  |   warn("Requiring AUTH while not requiring TLS "
mailrise-server  | [2024-01-06 14:50:25] WARNING:mail.log:auth_required == True but auth_require_tls == False
mailrise-server  | /home/mailrise/.local/lib/python3.12/site-packages/aiosmtpd/smtp.py:372: UserWarning: Requiring AUTH while not requiring TLS can lead to security vulnerabilities!
mailrise-server  |   warn("Requiring AUTH while not requiring TLS "
mailrise-server  | [2024-01-06 14:50:26] WARNING:mail.log:auth_required == True but auth_require_tls == False

it come with the latest update

Thanks for the report @djonko—I’ve reverted the commit that broke SMTP authentication. It was the fix for #98.
We already test configuration parsing, but I really should write some integration tests, so that we can avoid situations like this in the future.


Create MOSIP profile for the build process

We need a MOSIP profile for the build process. The same needs to be updated in Inji's README file as well.

@kyanthony @jeremi for the MOSIP Profile is npm run build:android:mosip the right one to name it?

Ok, let's go with this.


Logout doesn't redirect to home in Firefox

Description
When using Firefox the logout flow does not redirect the user to the home page as it does with other browsers, so users are presented with a 404 if they were on any page that requires an active session.
Screenshots
Not a visual issue.

Gitea Version
Noticed on 1.16.9 but still happens on current.
Can you reproduce the bug on the Gitea demo site?
Yes
Operating System
No response
Browser Version
Firefox 103.0.2

same here with Firefox 104.0.2

Can not reproduce this on my Firefox.

Since recent changes about logout, I think this has been resolved.


Alignment of z

It is hard to examine if archetype slopes $\beta_k$ are well estimated because of misalignment in $\mathbf{z}_j$, I think...

Is this related to the ordering of columns in the latent variable part of a GLLVM?
Although the order of archetype is not very meaningful to real data (because archetype is nominal, not ordinal), but it matters when we are testing our model on simulated data, and it also seems to cause chain flipping in MCMC

How do / should we "align" $\mathbf{z}_j$?

See https://mc-stan.org/docs/stan-users-guide/mixture-inference.html

See also Nick Golding's post https://forum.greta-stats.org/t/gmm-in-greta-is-non-identifiability-a-show-stopper-in-greta/71/15?u=hrlai

Francis Hui provided a more useful link https://mc-stan.org/docs/2_20/stan-users-guide/label-switching-problematic-section.html


projectexplorer redesign

The fate of the project explorer is going to change. Whereas now, we have the inspector sitting atop the project explorer, instead I am going to take the approach of combining the directories into the inspector. This will culminate in simply using file cells instead of the current substitution cells that are inside of the work menu. This will be a marked improvement, and still quite applicable to the context of granting users permissions.

nice


Minio base support

MinIO standalone server deployment
minio package w/ base Save/Load functions
CreateBucketIfNotExists()

Overall. I'm not sure if we should merge these changes into the dev branch until we've made a decision about minio. Maybe we should keep these changes in feature branch for now? Thoughts?


Cannot Browse Network Shares in Thunar

Issue description
I have samba, gvfs, fuse, and fusesmb enabled/loaded/installed. However, Thunar does not show anything on the network.
Steps to reproduce
Go into Thunar, click network. Mouse busy (background) cursor will appear, but nothing will happen.
Technical details

System: NixOS: 16.09
Nix version: 1.11.6
Nixpkgs version: 16.09.1512.6b28bd0

This is apparently intentionally because "samba is a rather heavy dependency", see here in the code.
Workaround using packageOverrides which fixes the problem for me:
nixpkgs.config.packageOverrides = pkgs: {
  xfce = pkgs.xfce // {
    gvfs = pkgs.gvfs;
  };
};

/cc @vcunat

Well, I can't see anything actionable about this.  Can you?

@vcunat I would propose to not turn off SMB by default "just because Xfce is supposed to be lightweight", especially with no mention of this in the manual and no instruction how to get it back. Why can other desktops have it by default but not Xfce? What do you think about a configuration option in the xfce module options to enable/disable smb?

When I look at gvfs closures themselves, the difference is ~180 MB; with a whole system it might be much less, I don't know.  I'm certainly not against such options.
BTW, Thunar won't even get rebuilt, as this is all through GIO plugins specified by the nixos module in an environment variable.  Perhaps we should factor this out into a specific "pseudo-service" where you can specify what gvfs plugins you want (and DEs will imply some defaults).

OK, I will try to implement an option, SMB can still be off by default.

@ambrop72 the workaround don't work for me. I can't even mount it manually! (works with other Linux)
[davidak@X230:~]$ gvfs-mount smb://<IP_ADDRESS>/Archiv </home/davidak/.smbcreds
Error mounting location: Location is not mountable

I'm still having this issue as of 20.03. Did any of you found a fix?

It may be reasonable to define samba support in gvfs based on the value of config.services.samba.enable in the module config.services.gvfs.

Still an issue.

This issue can be solved now.


⚠ ClamAV has degraded performance

In 51d89f1, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 198 ms

Resolved: ClamAV performance has improved in 196aa83.


Update documentation

I have updated the README.md and point the docs to separate files in the docs-folder.
However, Setup.md is not ready yet, and it's Christmas now, so I'll be away a few days before I can finish it.

Yes!! Marry Christmas (advance)..... Take your time...😊😊😊


[ng] clean up dropdown focus handler to not use setTimeout

DropdownFocusHandler currently uses setTimeout to setup its focus management logic that gets executed in the constructor. This ticket is to revisit to see if we can find a solution without using it.

Closing this as a refactoring we aren't likely to change in Angular and Core is likely to address this differently.


Fix expected error page in ODS-572 plus add 1.19 compatibility in ODS-293

Similar changes applied in previous PR #650 .
I've verified that the new error page happens only in 1.20, so I'm adding a version check in ODS-293 for retro-compatibility

Jenkins Job 1434 RUNNING


Adding Upper Merion Area High School

Official website URL: https://www.umasd.org/site/Default.aspx?PageID=5276
IT related course: https://www.umasd.org/Page/5440

@andrewzirkel First stage of verification passed, official domain is umasd.org as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@andrewzirkel Pull request merged. Thank you.


Fixed broked links in footer

#1031
The language parameter broke the footer-links.
And the blog-link was pointing at an empty page.

Thanks!


Update Dockerfile to use Chrome for Testing instead of regular Chrome

Sorry, since the purpose is auto-updating, I merged my PR which changes CI properly


Is Conditional Expressions for queries properly setup

I tried the following request for querying with a conditional expression:
{"ExpressionAttributeNames":{"#ATTR1":"QueryTerm","#ATTR5":"Attendance","#ATTR3":"PostInformation","#ATTR6":"TTL"},"ExpressionAttributeValues":{":val0":{"S":"9q8yy"},":val1":{"N":"1572368826812"}},"FilterExpression":"( ( ( size ( #ATTR5 )) < #ATTR3.Maximum ) AND ( #ATTR6 > :val1 ) )","KeyConditionExpression":"( #ATTR1 = :val0 )","TableName":"ExampleTableName"}```

And received the following response from the server:
{Amazon.DynamoDBv2.AmazonDynamoDBException: Invalid FilterExpression: Syntax error; Expected ")", "AND", or "OR" but "<" found. ---> Amazon.Runtime.Internal.HttpErrorResponseException

The following expression works fine on DynamoDB local offered by Amazon.

Hi @afshawnlotfi – does this expression work on DynamoDB in production?

Haven't tested production but I didn't generate this the FSharp.AWS.DynamoDB library did and I assume they did everything right. The only time I have this exception is when I want to run a Filter expression when updating

I have noticed that if I do compound conditional expressions for example ```
<@ fun (entry: BroadcastModel) ->
entry.UserId <> userId && entry.UsersAttending.Count < entry.EventInfo.MaxPeople @>)
then this causes a parsing error, however, when I only do a single conditional it works. So maybe there is a bug parsing compound conditionals in ```conditionParser.pegjs```

I have tested on production now and my query I used above works just fine. There is something wrong with this codebase for Conditional Expressions on Queries and a I assume scans

Figured out the issue – it was the extra parens around the size function that caused the issue in ( size ( #ATTR5 )) – I didn't have any tests for that.
Fixed in v3.1.1 – thanks for reporting!

No problem thanks!


Action should be performed if it is shown as successful

Expected behavior
Action should be performed if it is shown as successful
Actual behavior
The REPL shows that the action is successful, but the element is not clicked
Steps to replicate

Run the script with taiko

const { openBrowser, goto, below, image, click } = require('taiko');

(async () => {
   try {
       await openBrowser();
       await goto("amazon.in");
       await click(image(below("Today's Deals")));
       await click("Deal of the Day");
   } catch (e) {
       console.error(e);
   } finally {
       closeBrowser();
   }
})();

The last action is shown as successful, even though it is not performed.

Version
commit - ffddb5823fa8f966bea64377065f5649bf626f96
Fri Jul 27 09:35:17 2018 +0530

Looks like the amazon website has changed and I don't see Deal of the day link.
Could you please recheck this?

Please find another script to replicate the issue.

In the script below, Book Tickets is commented. This has to be manually clicked.

const { browser, openBrowser, goto, click,link,image } = require('taiko');

(async () => {
    try {
        await openBrowser({headless:false});
        await goto("https://in.bookmyshow.com/bengaluru");

        await click("Bengaluru");
        await write('NCR', into(await inputField({"placeholder":"Search for your city"})));
        await click("NCR");
        await click("View All");
        await click(link("English"))
        await click(image(below(link("Now Showing"))))
        //await click(link("Book Tickets"))
        await waitFor("5000")

        await click(link({"class":"__showtime-link"}))
        await click("Accept")
        await click(link(toRightOf("D")))
        await click("Select Seats")
        await waitFor("5000")
    } catch (e) {
        console.error(e);
    } finally {
        await closeBrowser();
    }
})();

After step click(link(toRightOf("D"))) is shown as successful, even though there is no such element to click.

await click(link(toRightOf("D"))) is shown successful because the page below the pop has a link to the right of D. So the element actually exists in the above case.
To actually view the action try await highlight(link(toRightOf("B"))); in this case the highlited link can be seen.
But the question is, whats the expected behaviour?

To actually view the action try await highlight(link(toRightOf("B"))); in this case the highlited link can be seen.

But the question is, whats the expected behaviour?

In many cases when pop up is displayed it masks elements behind the page. Visually there's no way to interact with elements. How does the pop up look on this page? (screenshot helps!)

It looks like this.

Ok. This makes sense.
Ideally or by default it should limit it to the popup area. Is there a way to identify the background masking?

Not sure. Will have to see if CRI gives any such masking.

In the page above, the modal popup is rendered as follows

The display: block display a full height div hiding the other elements. So inspect the DOM, to check if the element is visible before clicking it.

We do filter visible elements using offsetParent, seems it does not cover all the edge cases. Might have to try something like this https://stackoverflow.com/a/33456469/3336401

Fix should be available in fd86932.

In the following case, click on Sign in is shown as successful. However this action is not performed
const { openBrowser, goto, scrollTo, click, button, link } = require('taiko');

(async () => {
    try {
        await openBrowser({headless:false});
        await goto("http://demos.jquerymobile.com/1.4.5/popup/");
        await scrollTo("sign in");
        await click("Sign in");
        await click(button("Sign in"));
        await click(link("Sign in"));
    } catch (e) {
        console.error(e);
    } finally {
        closeBrowser();
    }
})();

Happening due to https://github.com/getgauge/taiko/issues/131

Should be fixed as part of 888638d.
Please verify.


Authentication provider for 'Active Directory Interactive' should maintain access token cache globally

Is your feature request related to a problem? Please describe.
For now we have used Azure CLI and Azure.Identity to use AAD authentication when working locally. We used 'az login' and we could then start and restart several of our services without logging in again and again.
With this solution we have had to gain an access token for a SqlConnection manually.
With the new Authentication provider 'Active Directory Interactive' we don't need to take care of access token handling anymore.
Unfortunately we now have to login every time we start a service. So it seems like that the access token is cached within the current application instance.
We can't use 'Active Directory Integrated', because of MFA.
Describe the solution you'd like
It would be nice when 'Active Directory Interactive' would work like Azure CLI (az login) where we just have to login once for a while.
e.g. When I run 2 or more applications at once or I restart my application(s), I only want to login once.

Hi @Rookian
This has been considered with issue #616 where we will bring in direct support of Azure.Identity and a new option would be provided to fetch "DefaultAzureCredential" from client environment.
In the mean time, you can follow this approach: https://github.com/dotnet/SqlClient/pull/730#issuecomment-705307179 with latest Microsoft.Data.SqlClient driver.
Since we're tracking this change with #616, is there anything else you'd need help with?

Hello @Rookian , did this global cache fix for 'Active Directory Interactive' work for you? For me it does not seem to be cached and I have to login every time I start the application like you originally described.
Currently 'Active Directory Default' is not feasible for me either, as it does not include the interactive mode enabled as last fallback.
Thank you for your time. 🙂

@Basssiiie The built-in "cache" for Active Directory Interactive is only in the scope of the process. So if you restart your application, the driver's cache is gone. There is another feature request to provide an API that will allow applications to define their own cache that can persist across processes. #1895

@David-Engel Thank you for your reply. Would it be possible to configure a global cache with #1895 purely through the connection string or do we need custom code in the application for that as well?
My use case is that I would like an easy to use connection string that uses their AD account (so we can disable SQL accounts) and that would work for all our developers, regardless of whether they use VS, VSCode, Rider or another tool. We also have a microservice project, and currently the AD Interactive gets triggered for every microservice the developer starts every startup. 🙂

@Basssiiie You would need custom code. But it sounds like Active Directory Default would be the closest thing to what you are asking for. It uses Azure.Identity's DefaultAzureCredential under the covers, which traverses a chain of credentials, trying various known, cached credentials (excluding Interactive as MDS disables that to avoid UI issues in headless apps):
https://learn.microsoft.com/en-us/dotnet/api/azure.identity.defaultazurecredential?view=azure-dotnet

@David-Engel Yeah Azure Directory Default would work best at the moment, though it is too bad the interactive mode cannot be enabled as a fallback here if all other options fail.


Sharing vs-chromium search filter

If I have two branch checkouts of the same folder structure in directory root (root/branch1, root/branch2) it would be nice if I could put a single vs-chromium-project.txt at root but have it apply only to the branch I'm in at the time.

It may be worth exploring if visual studio macro expansion is accessible when parsing the vs-chromium-project.txt file so one can use $(SolutionDir)

Added RootDepth feature for this

Added RootDepth feature for this


Ace fork/bugfix/642/unique names conflict

Superseding https://github.com/dynatrace-oss/dynatrace-monitoring-as-code/pull/654 as I am not allowed to add changes to that fork and @tobigremmer-dt is on holiday.
This adds a small change simplifying the error collection in deploy.go with https://github.com/dynatrace-oss/dynatrace-monitoring-as-code/commit/13c3ef973f5734da01523e1ec99062422cd16a84 (see commit message for details)

Merging 'without approval' as original changes by @tobigremmer-dt are reviewed and approved by me (https://github.com/dynatrace-oss/dynatrace-monitoring-as-code/pull/654). This PR opened to add a small change myself rather than keeping this further open due to tobi's holiday.


remove some unwrap calls

Issue #, if available:
#9
Description of changes:
Removed some unwrap calls.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thanks for the pull request! I'll review this and get back to you soon.


Conflicting instructions (I think)

In section 2. Remap records for IP based SSL, it says this step is needed only for IP based SSL, and that for an SNI SSL binding one should go directly to section 3. But then the second bullet says that for an SNI SSL binding the CNAME DNS record should be remapped to sni..azurewebsites.net (by adding the sni prefix).
So which one is it?

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

ID: b2906753-7656-ef4d-0de9-7e238181e3e1
Version Independent ID: 66dcb903-38dd-858b-dfb4-8a8dafaa0930
Content: Secure a custom DNS with a TLS/SSL binding - Azure App Service
Content Source: articles/app-service/configure-ssl-bindings.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

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

We've added this to our backlog to review and update as necessary. #please-close


htmlFor on label breaks Checkbox, Radio and Switch

https://github.com/react-toolbox/react-toolbox/pull/1180 introduced htmlFor attribute on labels that wrap input tags in Chekbox, Radio and Switch.
However, per HTML spec, the for attribute refers to the ID of the input element, not the name. If you in fact pass a name prop to them, these components stop working because input tags lack ids and the click event is not carried over to the htmlFor element because there is no such element.
In addition, the name of radio controls is not unique (all radios in a group are supposed to share the same name), so it cannot be used as an id and htmlFor, so htmlFor should just be removed from all three components.

Thanks for reporting!


Initial implementation

This PR accounts for the initial design and implementation of the opaque auth client.
Provided interfaces are:

login
currentSession
logout

Read more about them in the README.md.

Changes Unknown when pulling a4780352f5521e508356df64ce645113e4620853 on dan-f/develop into ** on master**.

Changes Unknown when pulling 3d959bb637546357e777770979f9f00f5cc268c2 on dan-f/develop into ** on master**.

Changes Unknown when pulling ec622f762a4dbee08efac53f780f9d85aab46762 on dan-f/develop into ** on master**.

Changes Unknown when pulling<PHONE_NUMBER>d53065cf2d96641ae46041c946fea9 on dan-f/develop into ** on master**.

@dmitrizagidulin here's the initial implementation

Changes Unknown when pulling 70c2840d25ee3094b548915b5287d04c20f80ed4 on dan-f/develop into ** on master**.

Looks ok.
One minor suggestion - you probably want to clear the tokens from oidc authorization callback, from the current url hash fragment after rp.validateResponse() gets called, using something like window.history.replaceState() (so that the credentials get cleared from browser url history, etc).

Thanks, yeah I will do that!

Changes Unknown when pulling ff0c2445ac0d12d43fe5b4e6650d71a6615a89dd on dan-f/develop into ** on master**.


changed Minibatch class so that it can change the shape of data freely

This fix was needed for me to change the amount of data that Minibatch holds.

Please, provide tests for your particular case.

Added test test_change_variable_size() in test_dataholders.py.

@awav can you confirm that this doesn't break anything you had in mind for autoflow and merge or close accordingly?


RHEL support on Azure

It seems lower vm sizes on Azure allow nested virtualization also on Azure there are skus for RHEL https://learn.microsoft.com/es-es/azure/virtual-machines/workloads/redhat/redhat-images so we should try to test if we can use it.

Check sku with az vm image list -p RedHat --all -o table


Dependency update

fix to VERSION workflow to use proper poetry version

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 49.6%

Totals

Change from base Build<PHONE_NUMBER>6:
0.0%

Covered Lines:
805

Relevant Lines:
1623

💛  - Coveralls

merge w/o review because only change was to release workflow.


We are moving to GitLab

Dear visitor,
Twing will be moving to GitLab in the close future. We feel that GitLab set of features is now superior to what GitHub has to offer and that Twing would benefit greatly from them, especially in regard to automated pipelines.
Starting on May the 1st 2021, issues should be opened in Twing GitLab repository:
https://gitlab.com/nightlycommit/twing
New issues opened on GitHub won't be considered - only issues prior to May the 1st will be supported.
We are eager to meet you there. :)
Eric.

This seems like a mistake. What does GitLib CI have over GitHub Actions that's so compelling?


[5.5][Runtime] Fix crash in protocol conformance checks on KVO artificial subclasses.

tryGetCompleteMetadataNonblocking crashes on artificial subclasses due to the NULL type descriptor. Explicitly check for artificial subclasses in getSuperclassForMaybeIncompleteMetadata and immediately return their Superclass field. Artificial subclasses are always fully initialized so we don't need to do anything special for them.
rdar://72583931
(cherry picked from commit c3aa44368a4f0469efd12f19c8b430e8ed65005e)

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - d41584de60ef73f7a01a64f2443d7ac33b6876c5

Build failed
Swift Test OS X Platform
Git Sha - d41584de60ef73f7a01a64f2443d7ac33b6876c5

Build failed
Swift Test OS X Platform
Git Sha - d41584de60ef73f7a01a64f2443d7ac33b6876c5

Build failed
Swift Test OS X Platform
Git Sha - d41584de60ef73f7a01a64f2443d7ac33b6876c5

Build failed
Swift Test OS X Platform
Git Sha - d41584de60ef73f7a01a64f2443d7ac33b6876c5


explicitly depend upon pygments.rb

This is in response to #235
To reproduce, change the Gemfile from:
    gem "github-linguist",  "~> 2.10",  :require => false

to
    gem "github-linguist",  "> 2.10",  :require => false

and bundle update
I tried not requiring 'pygments.rb', and using defined?, but it looked like nothing worked without including the requirement.
I updated the README as well.

:+1:

@JuanitoFatas Thanks - any other thoughts?

@kbrock @jch do we still need github-linguist? the newest version totally removed its need (lexer is gone). In one of my project I overrode SyntaxHighlightFilter to only use Pygments:
class HTML::Pipeline
  # Patch SyntaxHighlightFilter (@todo provide bugfix)
  class SyntaxHighlightFilter < Filter
    def lexer_for(lang)
      Pygments::Lexer[lang]
    end
  end
end

This works like a charm ;)

Think this is no longer needed


 : Removing foreach since it conflicts with the std libr…

…ary.
 : fix
Checklist

[ ] New Extension
[ ] New Test
[ ] Changed more than one extension, but all changes are related
[x] Trivial change (doesn't require changelog)

          1 Message

:book:
Executed 180 tests, with 0 failures (0 unexpected) in 8.036 (8.405) seconds

  Generated by :no_entry_sign: Danger

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@928d825). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #392   +/-   ##
=========================================
  Coverage          ?   39.52%
=========================================
  Files             ?       47
  Lines             ?     2168
  Branches          ?        0
=========================================
  Hits              ?      857
  Misses            ?     1311
  Partials          ?        0

Impacted Files
Coverage Δ

Sources/ArrayExtensions.swift
82.25% <ø> (ø)

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 928d825...497f374. Read the comment docs.

Done.

Hi,
I believe that removal of https://github.com/goktugyil/EZSwiftExtensions/blob/928d825dcc5002f468d8cfc0b94b98a5910f160c/Sources/ArrayExtensions.swift#L135 is required for correctly implementing this PR

That method is not deprecated, and we need to think this through if we do want to deprecate it.

@Khalian: thanks for this info. I'll go ahead and fork. Probably for the best :) I'd be glad to start making the minor changes for protocol based extensions on concrete types there.

@lfarah @goktugyil Can we deprecate the method that @DanielAsher mentioned : https://github.com/goktugyil/EZSwiftExtensions/blob/928d825dcc5002f468d8cfc0b94b98a5910f160c/Sources/ArrayExtensions.swift#L135 . If so I can put out a minor release adding this to something like 1.9.1.

@DanielAsher As for unblocking you, I will quickly discuss with the other maintainers and maybe put out a minor version. Sorry for the inconvenience.

@Khalian thanks so much for your attention to this. maintaining a well-used library has many complexities - best of luck and wishes for the future!

@Khalian Why are we deprecating it again?


Import error when cancelAnimationFrame is not defined

RxJS version:
15.0.3 (also encountered in 5.0.0-rc.1).
Code to reproduce:
I encountered this while writing jest tests for a react native app. I wrote a test that looks like this:
import 'react-native';
import rx from 'rxjs';

// ... the rest doesn't matter

That series of imports raised this error:
TypeError: Cannot read property 'bind' of undefined

at new RequestAnimationFrameDefinition (node_modules/rxjs/util/AnimationFrame.js:6:66)
at Object.<anonymous> (node_modules/rxjs/util/AnimationFrame.js:33:26)
at Object.<anonymous> (node_modules/rxjs/scheduler/AnimationFrameAction.js:8:24)
at Object.<anonymous> (node_modules/rxjs/scheduler/animationFrame.js:2:30)

Which led me to src/util/AnimationFrame:8:
function RequestAnimationFrameDefinition(root) {
    if (root.requestAnimationFrame) {
        this.cancelAnimationFrame = root.cancelAnimationFrame.bind(root); // <-- the failing line
        this.requestAnimationFrame = root.requestAnimationFrame.bind(root);
    }
// ...

It seems that under those circumstances, requestAnimationFrame is defined but cancelAnimationFrame is not, so it enters the if block and then throws. However, if rxjs is imported before react-native, everything is fine, presumably because react-native is what adds requestAnimationFrame to global.
I created an issue in the Jest repo with more details, and I've also created an example repo. If you clone that you should be able to see the error by running npm test.
Expected behavior:
Ideally, this should not attempt to bind cancelAnimationFrame unless it exists.
Actual behavior:
The line in question evaluates to undefined.bind and throws an error.
Additional information:
I realize this is a pretty specific situation, just thought I'd point out that in some instances it is possible for requestAnimationFrame to be defined and cancelAnimationFrame not to be.

Until now I was under assumption of requestAnimationFrame and cancelAnimationFrame available as pair, one can't be undefined without others.
So for this cases let's say if cancelAnimationFrame is not available, how does given scheduled one being cancelled via requestAnimationFrame? is there other pair supports those, or cancellation is just not supported in those environment?

As far as I can tell the jest/react-native situation was caused by some odd behavior in jests packager. In that situation I think cancellation may not be supported (and requestAnimationFrame may also just be stubbed out.)
I'm not sure that there is a "normal" situation where this could come up, I just thought I'd point out that in some edge cases this check can cause the rxjs import to fail. I don't necessarily think it's this library's responsibility to account for every possible environment, though it does seem like it would be a fairly straightforward fix to add a cancelAnimationFrame check before attempting to use it.
I guess it just comes down to philosophy. Either way, it's unlikely to affect many users.

@liamfd
At this moment, I wouldn't think add a cancelAnimationFrame check as an immediate fix for two reason - first, if any real environment those have those cases (which has request but no cancel) then RxJS's scheduler implementation will not able to cancel anything by adding those checks, should consider legit strategy to bring cancellation support. Secondly, if it's certain test-framework environment mutation-specific (i.e, jest-runtime) it's up to consumer / or library's call to support those correctly.
By your explanation, I have an initial feeling this might be similar cases to those since jest had similar cases before with RxJS when it does some mocking internally - would you able to confirm if jest does preserve its runtime environment correctly?

@kwonoj still looking into it, but it seems like it's only happening because of the interaction between the RN and Jest. When running jest without importing RN, requestAnimationFrame is not defined so this issue doesn't come up. When just running an RN app, both are defined, so there's no issue there either.
Didn't realize how complex it would be to try to implement the cancelAnimationFrame check in rxjs. Seems fair to close this then!

@liamfd just clarifying bit more - I'm not saying let's not fix RxJS :), would like to understand bit more 1. if this happens by jest 2. if it isn't, how to deal with canceling if given environment.
I'll leave this opened bit more, feel freely ping us if there's additional detail needed in Rx side codebases. It's bit hurry to close at this moment since it's not clear where this comes from.

If cancelAnimationFrame isn't available, I'd imagine we should fall back to setTimeout

@kwonoj sounds good, I just opened an issue in react-native so we'll see where that gets us. Thanks for the response thus far!

I'm going to close issue this for now, as most environment should have pair of scheduling function. We can have fallback to setTimeout as separate PR if needed.

If you are running into this in jest using Create React App, add
global.cancelAnimationFrame = function(callback) {
  setTimeout(callback, 0);
};

to your src/setupTests.js alongside your shim you're probably already running for requestAnimationFrame.

@JHKennedy4 that worked for me! Thanks.


Property access to internal variables

should all functions return pointers to the internal arrays or always return copies? Should there be an toggle for this behavior? Currently might be inconsistent

Fixed in 2b62a13bdf0591e3f0baf3273a914909229fe5dd
only return copies


Wall correction in corridor with door

There are 2 corridors that have a door in the middle leading to another corridor (6, 10)
The wall collision correction is stuttering on the first 7 wall tiles in 6, and the first 8 tiles in 10

Re-wrote CollisionSystem:detectCorridorDoorways function to only check for central doorways, not all doors in the corridor as collisions are ignored for doors at each end


use group feature for dependabot

Use the dependent bot group feature to get the dependency updated every week in a group https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#groups

Fixed by https://github.com/ceph/ceph-csi/pull/4087


Containerize the application

We should add a Dockerfile that containerizes the application to make it easier to run.
In particular this Dockerfile should include installation of gofer and all required configuration.

Closed by:
https://github.com/ultrasoundmoney/oracle-client/pull/12


Performance issue calling another API?

Hi @ndortega,
Congratulations for this great package, I'm using it a lot even into a prod environments
Recently I'm using it with AWS.jl to reach a SageMaker endpoint but I notice a performance issue (maybe due to my ignorance). I use invoke_endpoint that it is just a POST request.
I tried to recreate a minimal example:

When I run this API (with a 150 ms sleep to simulate the time taken by the SageMaker endpoint):

using Oxygen

@get "/model" function()
    sleep(0.15)
    return """{ok}"""
end

serve(port = 8001)

I reach about 560 RPS (with concurrency = 100)
ab -n1000 -c100 'http://localhost:8001/model'
Server Software:
Server Hostname:        localhost
Server Port:            8001

Document Path:          /model
Document Length:        4 bytes

Concurrency Level:      100
Time taken for tests:   1.756 seconds
Complete requests:      1000
Failed requests:        0
Total transferred:      83000 bytes
HTML transferred:       4000 bytes
Requests per second:    569.62 [#/sec] (mean)
Time per request:       175.556 [ms] (mean)
Time per request:       1.756 [ms] (mean, across all concurrent requests)
Transfer rate:          46.17 [Kbytes/sec] received

When I simulate an Oxygen API consuming the SageMaker endpoint

using Oxygen
using HTTP

@get "/test" function()
    response = HTTP.get("http://localhost:8001/model")
    return String(response.body)
end

serve(port = 8002)

I reach only around 100 RPS (ab -n1000 -c100 'http://localhost:8002/test') even using 4 threads with serveparallel. I would expect to see a number of RPS closer to 560
Server Software:
Server Hostname:        localhost
Server Port:            8002

Document Path:          /test
Document Length:        4 bytes

Concurrency Level:      100
Time taken for tests:   9.815 seconds
Complete requests:      1000
Failed requests:        0
Total transferred:      83000 bytes
HTML transferred:       4000 bytes
Requests per second:    101.89 [#/sec] (mean)
Time per request:       981.455 [ms] (mean)
Time per request:       9.815 [ms] (mean, across all concurrent requests)
Transfer rate:          8.26 [Kbytes/sec] received

If I do the same with FastAPI
import requests
from fastapi import FastAPI

app = FastAPI()

@app.get("/test")
def test():
    response = requests.get("http://localhost:8001/model")
    return response.text

With one worker (uvicorn api_test_nested:app --host <IP_ADDRESS> --port 8003 --workers 1) I get (ab -n1000 -c100 'http://localhost:8003/test') around 220 RPS
Server Software:        uvicorn
Server Hostname:        localhost
Server Port:            8003

Document Path:          /test
Document Length:        6 bytes

Concurrency Level:      100
Time taken for tests:   4.436 seconds
Complete requests:      1000
Failed requests:        0
Total transferred:      149000 bytes
HTML transferred:       6000 bytes
Requests per second:    225.42 [#/sec] (mean)
Time per request:       443.618 [ms] (mean)
Time per request:       4.436 [ms] (mean, across all concurrent requests)
Transfer rate:          32.80 [Kbytes/sec] received

But using 4 workers I get almost the same RPS from the endpoint simulating SageMaker (500 vs 560)
Server Software:        uvicorn
Server Hostname:        localhost
Server Port:            8003

Document Path:          /test
Document Length:        6 bytes

Concurrency Level:      100
Time taken for tests:   2.013 seconds
Complete requests:      1000
Failed requests:        0
Total transferred:      149000 bytes
HTML transferred:       6000 bytes
Requests per second:    496.78 [#/sec] (mean)
Time per request:       201.296 [ms] (mean)
Time per request:       2.013 [ms] (mean, across all concurrent requests)
Transfer rate:          72.29 [Kbytes/sec] received

I think it could be a HTTP.jl issue of maybe there is a better way to make the requests but I'm new using Julia so if you have a hint, please let me know!
Thanks in advance, and again thanks for your work in this package
Abraham

Hi @AbrJA,
It might come down to the json serialization of the response from AWS.  In your example you're returning the json body as a string from the request handler. When a string is returned from a request handler in Oxygen it will automatically try to detect the content type, but this does come with a decent cost in performance. If you know the content type of a request handler, I'd heavily recommend using one of the render functions to bypass this process.
In the first example below, it can be a little confusing since we have a json function doing two different things. The first call loads the json from a Response into a dictionary and the second call converts this data into a properly formatted Response
# Use the json() helper functions
@get "/test" function()
    response = HTTP.get("http://localhost:8001/model")
    # convert the body of the response to a dictionary
    data = json(response)
    # convert object to HTTP.Response object with a json payload
    json(data)
end

# Or you could just directly pass the response data through (with appropriate headers set) This is what those helper functions do under the hood
@get "/test2" function()
    response = HTTP.get("http://localhost:8001/model")

    headers = [
        "Content-Type" => "application/json; charset=utf-8",
        "Content-Length" => string(sizeof(response.body)),
    ]

    HTTP.Response(response.status, headers, response.body)
end

Let me know if you see any improvements or have any other questions

Hi @ndortega,
Thank you for your quick answer and your suggestions, I tested them but unfortunately I got the same RPS.
I think the issue is not in the serialization part but in the HTTP request (I don't fully understand why). To prove that, I changed the API response to be equal to the /model API, the unique difference between both APIs is the HTTP.request instead of sleep(0.15).
using Oxygen
using HTTP

@get "/test" function()
    response = HTTP.request("GET", "http://localhost:8001/model") # Similar to sleep(0.15) - Just this line is different
    return """{ok}"""
end

serve(port = 8002)

Doing the benchmark, again I got just 100 RPS (even with serveparallel) a low number compared to the 560 in /model
Server Software:
Server Hostname:        localhost
Server Port:            8002

Document Path:          /test
Document Length:        4 bytes

Concurrency Level:      100
Time taken for tests:   9.827 seconds
Complete requests:      1000
Failed requests:        0
Total transferred:      83000 bytes
HTML transferred:       4000 bytes
Requests per second:    101.76 [#/sec] (mean)
Time per request:       982.679 [ms] (mean)
Time per request:       9.827 [ms] (mean, across all concurrent requests)
Transfer rate:          8.25 [Kbytes/sec] received

I have been trying some ideas but I have not been able to solve it. If you have other clues, please let me know!
Again, thank you for your help :)

@AbrJA
Hmmm, sorry that didn't help.

Could you be getting rate-limited by AWS?  I know you can set rate limits on AWS personalize to keep your costs in check
Have you tried making the HTTP request asynchronously?

Hi @ndortega, I hope you are doing great!
I think I didn't explain myself correctly, I found this issue while using AWS with invoke_endpoint but in my example I don't use it anymore. This issue is not only with AWS but with any HTTP request within an API.
The example I am sharing with you runs locally. I start an API (/model - port 8001) and then I build another API (/test - port 8002) that makes a request to /model.
I tried with asynchronous requests but I didn't get an improvement. However, I changed the /test API to use curl instead of HTTP.jl and I got a huge improvement so I'm almost sure there is something strange with HTTP.jl
using Oxygen

@get "/test" function()
    response = run(`curl http://localhost:8001/model`) # instead of HTTP.request("GET", "http://localhost:8001/model")
    return json("ok")
end

serve(port = 8002)

With this small change I go from 100 RPS to about 440 RPS, much closer to 560 RPS I get from /model and twice the RPS from FastAPI with one worker.
I'm going to put my case in the HTTP.jl repository, maybe there someone has an idea why this is happening.
Thank you very much for your help

Hi @AbrJA ,
That makes sense, it may not be a Oxygen issue at all. But just to clarify - are you using the new @oxidise macro to create multiple instances within a single julia program? If so, you can try using the internalrequest() function which directly passes a http request to one of your request handlers.
module A
    using Oxygen; @oxidise

    @get "/" function()
        text("server A")
    end
end

module B
    using HTTP
    using Oxygen; @oxidise
    using ..A

    @get "/" function()
        text("server B")
    end

    # directly call Sever A from Server B
    @get "/a" function()
        A.internalrequest(HTTP.Request("GET", "/"))
    end
end

try
    A.serve(port=8001, async=true)
    B.serve(port=8002, async=false)
finally
    A.terminate()
    B.terminate()
end

Hi @ndortega,
Nope, I'm running the APIs independently (one API by script) to better simulate the request to an external endpoint. But it's good to know that new functionality, thanks!
I found this recent issue in the HTTP.jl repository JuliaWeb/HTTP.jl/1153. I have a feeling it's related to this but I'm not sure.
Greetings!

Hi @AbrJA,
It seems like you were able to resolve this issue on your own, so I'm going to close it for now. If this issue persists or comes back, feel free to open up another issue


⚠ ClamAV has degraded performance

In 5c24972, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 226 ms

Resolved: ClamAV performance has improved in 418c705.


Add sqlserver entry

Codecov Report

Merging #53 (df68e39) into master (80909ec) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   85.45%   85.45%
=======================================
  Files           1        1
  Lines          55       55
=======================================
  Hits           47       47
  Misses          5        5
  Partials        3        3

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 80909ec...df68e39. Read the comment docs.


chore: avoid uselessly patching the ContinuousArchivingSuccess condition

The archive_command callback is keeping updated the ContinuousArchivingSuccess condition using the result of barman-cloud-wal-archive or the corresponding CNPG-i plugin.
This change makes the instance manager deterministically hit the API server when the condition changes instead of using reflect.DeepEqual, avoiding false positives.
See also: #5366

E2e: https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/11346340514

/ok-to-merge Test are failing for upgrades due to https://github.com/cloudnative-pg/cloudnative-pg/pull/5503, not for this PR


SCRUM-3712 Load AGM Phenotype Annotation data from FMS

@adamgibs can you look at the UI side of this?

@markquintontulloch UI part looks good.


Fix breadcrumb last element position

Typography in breadcrumbs is not aligned.

hey @TSlashDreamy , i am willing to work on this issue/bug.
Plz assign this to me


Top and base descriptions for Intervals

Especially (only?) base: conformable, erosive, downlapping, etc.

Added the property as i.style — which can be any string.
We don't do anything with this info however. Main next thing would be to add a new type of feature to the legend, surfaces. This will require rethinking the legend structure completely though... Will start a new issue.


Level which is similar to those ads where you slide the appropriate door to release the water/lava to save a character

needs #58


Usefull, but...

This is extremely useful. I will be including it in all of my future python projects.
There is only one problem, it seems that you are missing a few integers. From what I've seen it ranges from 10001 to Infinity.
(I would like to note that I can't quite confirm this yet, as my tests are still running.)
Otherwise star++ 👍

I notice your concern, this is a bug our development team is working on fixing and we should deploy a fix in the coming days. We are glad you find our product useful.

Bug is fixed, added the recursive feature to cover all ranges of integers.


Support for playback of already started stream

Hello.
Would it be possible to support playback of already started h264 live stream?
Currently the playback works only when a h264 stream is played from the very begining.
Maybe encapsulation to MPEG-TS or MP4 could help to proper demuxing.
Explanation:
I have developed a simple websocket server which handles various live streams from webcams and recompress them to mpeg1/mpeg-ts streams (using ffmpeg pipe) and sends them to clients via websocket. I use JSMpeg player on client side, it reconnects well to already started mpeg-ts streams, unfortunatelly mpeg1 doesn't provide required quality with low bitrate.
Thus I search for the h264 alternative. Your player looks promissing, but it cannot handle playback from random start point in the middle of a h264 stream.
Thanks.

The player supports playing a already started stream, but because how h264 works it will have to wait for a pps (key frame) packet to start (usually few seconds, depending on the encoder settings) . The default example (one with listen on 5000 for stream) does just that, you can start a dream and connect the client at any point.
Regarding encapsulation, this project aim was to get rid of as much delay as possible over websocket. Encapsulation would either add additional delay or impact the client cpu usage even more. Websocket already provides most of the things anyway (packet order, packet encapsulation,...).
If latency is not a priority you could always use the html native video element for streaming. It supports various codes and encapsulations, there are a couple of projects doing it like this, but delays between 5-10sec.

My priority was minimal latency. I'll try to set key frame interval of h264 as less as possible, hope it helps. I was using default settings of ffmpeg h264 encoder before, that could be the reason why playback didn't start. When there is a support of playing already started stream, there must be a problem on my side. Thanks.

@matijagaspar
I investigated what is wrong with my encoder. The answer is simple. I supposed that ws-avc-player can play raw h264 stream sent via websocket (like jsmpeg player). That's a mistake. H264 stream is processed by server.js and split by NAL separator into messages with some json headers. My server is written in python, I have to imitate this functionality as well. Just for clarification for others.
A simple description of this message format would be helpfull.

Yes, that's right. I did NAL splitting on server to lessen the burden on client, since it already does the decoding on the cpu.
I will put this note to the readme


Resolve relative filePath URLs to absolute.

References
When users wish to load a CLP IR from the same origin of the Log Viewer host, to avoid repeating the log viewer's host url in the filePath URL parameter, we want to support the use case of supplying a relative path in filePath.
Description

Resolve relative filePath URLs to absolute.

Validation performed

Placed test file test-0.0.1.clp.zst under /dist
Loaded http://localhost:3010/?filePath=/test-0.0.1.clp.zst in a browser.
Observed the file gets loaded successfully and the URL in the browser address bar got updated to http://localhost:3010/?filePath=http://localhost:3010/test-0.0.1.clp.zst#logEventIdx=375558 .

@Henry8192 Please help triage this review.


Start HLS buffering when clicking on the play button

Hi , I use HLS inside a Plyr player and I would like to only start buffering the video when the user clicks on the play button ( I have more than 10 players on a page ).
I'm a beginner and I've seen in the documentation that there is a property "playing" that returns a boolean so I tried something but this is obviously not working.
Can someone help me with my code ? It would be much appreciated :)
if(Hls.isSupported()) {
var hls = new Hls({ autoStartLoad: false });
hls.loadSource('https://content.jwplatform.com/manifests/vM7nH0Kl.m3u8');
hls.attachMedia(video);
hls.on(Hls.Events.MANIFEST_PARSED,function() {
IF BUTTON PLAY  >> ( how should I write that ? )
(hls.startLoad(startPosition=-1))
});
}

Thanks a lot for your help :D
I'm gonna work on what you said and wait for your answer.

This should work (also falls back on native hls if hls.js isn't supported and regular video fallbackUrl if neither are):
var hlsUrl = 'https://content.jwplatform.com/manifests/vM7nH0Kl.m3u8';
var video = document.querySelector('video');
if (Hls.isSupported()) {
	var hls = new Hls({autoStartLoad: false});
	hls.loadSource(hlsUrl);
	hls.attachMedia(video);
} else {
	var nativeHLS = video.canPlayType('application/vnd.apple.mpegurl');
	video.src = nativeHLS ? hlsUrl : fallbackUrl;
}

video.addEventListener('play', function() {
	hls.startLoad();
}, {once: true});

new Plyr(video);

This is working ! 👍 I really appreciate the fact that you took the time to help me , thanks :)
I could not have found that by myself.

You're welcome :) This is less than obvious imo. Had some trouble with it myself.

Is it normal that this is working on Chrome and not working on Safari ? I mean this is working on Safari but the video will only loads and not play :/
The code :
Could you tell me if this is working on your browser ? (https://remsfilm.com/test2.html)

In case it helps in Chrome I got :

logger.js:39

[error] > internal error happened while processing hlsManifestLoaded:Cannot read property 'addEventListener' of null
func.apply(self.console,

args);

I let you try it out later and if you can't managed to get it to work you could just send me a copy paste of your elaborate code that you use. 👍
Thanks again for your help,

Just tested, and it works great for me testing in Safari and no errors in Chrome with exactly that code.
https://jsfiddle.net/Larer8ad/
If you need help debugging your code this is not the place.

Oh that's great.
Sorry , I thought it was the addeventlistener fault because it said that in the console.

I'm sorry but I really think that there is an issue somewhere and it's not because of my code.
I resolved the javascript error in the console and I realized that the HLS stream will only start playing when I put it directly on the page and that it will only load and not play when it's in a Jquery lightbox.
If you want I can pay you to try to resolve my problem because I don't want to bother you.
Check this page on Chrome (https://remsfilm.com/test2.html) and it will work as expected.
Check it on Safari and the video will only load and not play. (https://remsfilm.com/test2.html)

I appreciate your attitude, but it's ultimately very time consuming to debug someone elses website, so maybe you can fork https://jsfiddle.net/Larer8ad/ and replicate your issue on jsfiddle. If it turns out to be a Plyr issue I'll reopen this.

And of course if you try to remove the new Plyr() part and the problem persists it's not a Plyr issue.

It may seems dumb but I didn't try this.
I just removed the

new Plyr()

from my code and the problem is still there so this is definitely not a Plyr issue and you can keep this issue closed.
Thanks :)

Good luck!


Improvements to linearization regression testing

This ready to merge.
Feature or improvement description
Add two updates to the linearization regression testing method:

add tables for comparison of frequencies and damping. Export these to files frequencies_ref.txt and frequencies_new.txt
add an optional low pass filter to the frequency comparisons.  There had been some cases where extra very low frequencies might appear in certain linearization checks (this was causing test Fake5MW_AeroLin_B1_UA4_DBEMT3 to fail when a frequency of ~0.00159 Hz - see below).  The low pass filter is set as an optional argument in the CTestList.cmake file (i.e. of_regression_linear("Fake5MW_AeroLin_B1_UA4_DBEMT3"  "-lowpass=0.05"   "openfast;linear;elastodyn;aerodyn"))

Comparison tables
The new table format for the frequency comparisons appears as:
43:  Case: Ideal_Beam_Fixed_Free_Linear:1.lin: :
43:               Frequency (Hz)                        Damping (%)
43:            ----------------------------         ----------------------------
43:             Ref              New                 Ref              New
43:            0.570963420      0.570963420         0.000000000      0.000000000
43:            0.570963426      0.570963426        -0.000000000     -0.000000000
43:            3.546155606      3.546155606        -0.000000000     -0.000000000
43:            3.546155608      3.546155608         0.000000000      0.000000000
43:            9.763888291      9.763888291         0.000000000      0.000000000
43:            9.763888291      9.763888291        -0.000000000     -0.000000000
43:           18.694578736     18.694578736        -0.000000000     -0.000000000
43:           18.694578736     18.694578736        -0.000000000     -0.000000000
43:           33.978106303     33.978106303         0.000000000      0.000000000
43:           33.978106303     33.978106303         0.000000000      0.000000000
43:
43:  Case: Ideal_Beam_Fixed_Free_Linear:BD1.lin: :
43:               Frequency (Hz)                        Damping (%)
43:            ----------------------------         ----------------------------
43:             Ref              New                 Ref              New
43:            0.570963420      0.570963420         0.000000000      0.000000000
43:            0.570963426      0.570963426        -0.000000000     -0.000000000
43:            3.546155606      3.546155606        -0.000000000     -0.000000000
43:            3.546155608      3.546155608         0.000000000      0.000000000
43:            9.763888291      9.763888291         0.000000000      0.000000000
43:            9.763888291      9.763888291        -0.000000000     -0.000000000
43:           18.694578736     18.694578736        -0.000000000     -0.000000000
43:           18.694578736     18.694578736        -0.000000000     -0.000000000
43:           33.978106303     33.978106303         0.000000000      0.000000000
43:           33.978106303     33.978106303         0.000000000      0.000000000

Low frequency cutoff
Before adding the low frequency cutoff, the case Fake5MW_AeroLin_B1_UA4_DBEMT3 would fail with an extra low frequency value sometimes appearing at 0.001591549 Hz.  For this case, this frequency doesn't make much sense.  The low frequency cutoff allows us to ignore it.
40:  Case: Fake5MW_AeroLin_B1_UA4_DBEMT3:1.lin: :
40:               Frequency (Hz)                        Damping (%)
40:            ----------------------------         ----------------------------
40:             Ref              New                 Ref              New
40:            0.618153481      0.001591549         0.077632352      1.000000000
40:            2.542444969      0.618023953         0.960695174      0.077577325
40:                             2.542505004                          0.960671433

Related issue, if one exists
The Fake5MW_AeroLin_B1_UA4_DBEMT3 regression test was added at commit 202bfe7d, then removed 2 months later with commit 8e7e0b94.  Between these commits, no changes were made to anything related to linearization.
This might be a candidate for inclusion in a v3.5.3 release.
NOTE: there is a bug in the AD15 linearization that is currently being fixed by PR #2014.  So the Fake5MW_AeroLin_B* cases are currently disabled even though they are the reason for this PR.
Impacted areas of the software
Linearization.
Additional supporting information
Test results, if applicable
Test results do not change, but the comparison for linearization cases is slightly more robust.

@jjonkman, that's a pretty idiotic mistake on my part. I was thinking lowcut filter, then started typing lowpass without thinking about it again.  I'll change it to highpass, as it should be.
I was suspicious there was something like that occurring with the low frequency modes.  In the initial addition of the Fake5MW_AeroLin_B1_UA4_DBEMT3 case, the low frequency mode doesn't appear.  Somewhere a few commits later, this case started failing due to the appearance of the low frequency mode (100% damped), which perhaps caused by small numerical changes with compiler updates.  In more recent work, that low frequency disappeared again.  So it's good to know this makes reasonable sense to you that we need this high-pass filtering.


Improve Security: Allow authentication via RAW JSON credentials rather than json file path

We would like to be able to authenticate our function that uses nodejs via a service account. However, from our investigation, the nodejs library for bigquery only supports specifying a credentials json file path, not the actual json content. This means that we would need to include the json credentials IN the source code - not a great idea for security given that these credentials give access to our entire database. Is there any way to authenticate the nodejs bigquery library without having a physical json file? There are comments about this functionality being lost and I've dug into the code and can't see a way to get past this? It seems like a big security hole.
Ideally, what would be useful is instead of using an environment variable to specify the credentials path, if we could instead pass the actual json for the credentials in here (or pass via the constructor as an alternative), that will allow us to use tools such as AWS secrets to keep the credentials secure. This appears to be an option in the Google Auth library, but it just hasn't been used in the BigQuery client library.

The credentials object! As far as I know, that still works. It's been a critical option we've supported since day 1.
new BigQuery({credentials: require('./key.json')})

This appears to be an option in the Google Auth library, but it just hasn't been used in the BigQuery client library.

The auth library receives the input from the BigQuery constructor and determines the correct auth mechanism to use. 'credentials' always wins.
Could you give this a try and report back? Sorry for the trouble.

Hi Stephen,
Works perfectly, thank you! Can I suggest adding this to the documentation? I searched everywhere and even google cloud support didn't have the answer. As an example, I used this successfully:
new BigQuery({projectId: "yourprojectid",credentials: {yourjsonobject}});


Add useable bootstrap parameters.

Is your feature request related to a problem?
"type=dhcp-client" doesnt bootstrap the firewall.
Describe the solution you'd like
use the license plugin or basic bootstrap and provide a useable example.
provide commented links in the config
Add a the following lines:
#License plugin
#https://docs.paloaltonetworks.com/vm-series/10-2/vm-series-deployment/license-the-vm-series-firewall/use-panorama-based-software-firewall-license-management
#Install device certificate
#https://docs.paloaltonetworks.com/vm-series/10-2/vm-series-deployment/license-the-vm-series-firewall/vm-series-models/install-a-device-certificate-on-the-vm-series-firewall
insert a real bootstrap example
https://github.com/PaloAltoNetworks/terraform-azurerm-swfw-modules/blob/6cc5eb460955bc03caab10f625088e263e7f3886/examples/common_vmseries/example.tfvars#L234
bootstrap_options = "type=dhcp-client;plugin-op-commands=panorama-licensing-mode-on;auth-key=<YOUR-AUTH-KEY-HERE>;tplname=<YOUR-TEMPLATE-STACK-NAME-HERE>;dgname=<YOUR-DEVICE-GROUP-NAME-HERE>;vm-series-auto-registration-pin-id=<YOUR-PIN-ID-HERE>;vm-series-auto-registration-pin-value=<YOUR-PIN-VALUE-HERE>"

Describe alternatives you've considered.
No response
Additional context
What is published for bootstrapping is crap. The goal is for the customer to be successful. Give us a pathway to success.

This issue duplicates #50. We're working on that.


param '-P'  dont work

wf-recorder -a -c libx264 -r 25 -x yuv444p -f test.mp4 -F scale=1280:-1  -C aac --audio-codec-param b=384k

wf-recorder -a -c libx264 -r 25 -x yuv444p -f test.mp4 -F scale=1280:-1  -C aac -P b=384k

AAC audio bitrate always lock to 128k
it doesn't even output like:
Setting codec option: .........

#268
#135
#79

Does it work if you try -Pb=384k  or --audio-codec-param='b=384k'?

Does it work if you try -Pb=384k or --audio-codec-param='b=384k'?

not work

Hm, I see that load_audio_codec_options() is defined but never called anywhere it seems. Maybe you have found a bug.

Does this patch help?

same, dont work

Can you elaborate about what you've tried that isn't working after applying the patch? Do you still not get the message stating the option set?

Here, is I run $ wf-recorder -o DP-3 -a -Pb=384k -f /tmp/wf-recorder.mp4, I at least see the output Setting codec option: b=384k.

Do you still not get the message stating the option set?
yes

./wf-recorder  -a -c libx264 -r 25 -x yuv444p -f test.mp4 -F scale=1280:-1  -C aac --audio-codec-param b=384k
./wf-recorder  -a -c libx264 -r 25 -x yuv444p -f test.mp4 -F scale=1280:-1  -C aac -P b=384k
./wf-recorder  -a -c libx264 -r 25 -x yuv444p -f test.mp4 -F scale=1280:-1  -C aac -P b=192k
./wf-recorder  -a -c libx264 -r 25 -x yuv444p -f test.mp4 -F scale=1280:-1  -C aac -P b=192
./wf-recorder  -a -c libx264 -r 25 -x yuv444p -f test.mp4 -F scale=1280:-1  -C aac  -Pb=384k
./wf-recorder  -a -c libx264 -r 25 -x yuv444p -f test.mp4 -F scale=1280:-1  -C aac  --audio-codec-param='b=384k'
./wf-recorder  -a -c libx264 -r 25 -x yuv444p -f test.mp4 -F scale=1280:-1   --audio-codec-param='b=384k'
./wf-recorder  -a -c libx264 -r 25 -x yuv444p -f test.mp4 -F scale=1280:-1  -C aac  -Pb=384k
./wf-recorder  -a -c libx264 -r 25 -x yuv444p -f test.mkv  -C aac  -Pb=384k
./wf-recorder  -a -c libx264 -r 25 -x yuv444p -f test.mkv  -C aac  -P b=384k

Can you show the complete output with -Pb=384k set? Maybe the patch isn't applied correctly or something.

It's right there.. Setting codec option: b=384k..

It's right there.. Setting codec option: b=384k..

sorry, I didn't see

but
Stream #0:1: Audio: aac (LC), 48000 Hz, stereo, fltp, 128 kb/s

I think the only thing that can be done here is to decrease the audio bitrate, not increase it. This can be accomplished by setting the audio sample rate with -R. Here, the only way I was able to make a difference was with -R8000 which gave Stream #0:1: Audio: aac (LC), 8000 Hz, stereo, fltp, 96 kb/s instead of 128k.

Actually, with -Clibfdk_aac I can  wf-recorder -o DP-3 -Clibfdk_aac -a -R96000 -Pb=256k -f /tmp/wf-recorder.mp4 and get Stream #0:1: Audio: aac, 96000 Hz, stereo, s16, 279 kb/s. So I think you have found a bug in general, but it will require experimentation to do whatever it is you're trying to do.

Honestly, I'm not sure if this is a problem with wf-recoder.
Maybe it's impossible to adjust the bitrate without change the sampe rate?

You are right about -P/--audio-codec-param not doing anything, this is what the PR fixes, specifically. But so far as changing the audio bitrate, I do think it is directly related to the audio sample rate.


Dim response times slow

Hi.
Alexa response times as follows:
On = Fast
Off = Fast
Dim = Slowwwwww! Anywhere from 5-15 seconds.
Also if I issue a turn off all lights, some lights can stay on for up to 10 seconds longer than the first one to turn off.
Strange. Any ideas?

Have you tested the dim command from another source to see the response time from the device?

Have you tested the dim command from another source to see the response time from the device?

I should have specified but forgot sorry.
If I use Alexa it suffers delay on dimming, has done now for a month or so. Using bridge test dim button results in expected response times, good in other words.
I'm just at a loss as to why on and off would work fast but not dim.

I don't have a good answer on why Alexa would be slower as it would call the same interface as the ha-bridge test dim does.

Well I never got to the bottom of this but it wasn't ha-bridge, it wasn't Alexa devices as they were all reset. But my zwave network crashed and thanks to homeseer development team not giving two shits about backing up controller interfaces for EU customers, my whole network is currently bring rebuilt. In the process the fault dimming the devices has disappeared. Who knows but I'm back with using the Aeotec Gen5 Stick instead of zwave.me UZB so perhaps I had a hardware fault.

Also, I have had issues with my Alexa being slow to respond to my voice commands. I have to reboot the Alexa every so often and then works great.

@smartypantsuk I've experienced a slow Alexa with my Hubitat and one Z-Wave device in particular, read a thread where Alexa is delayed with generic device names, my device was named "Fan".  Renamed it to "Bedroom Fan" and the delay from Alexa is gone.  Are any of the devices with the slow response named generically?

@smartypantsuk I've experienced a slow Alexa with my Hubitat and one Z-Wave device in particular, read a thread where Alexa is delayed with generic device names, my device was named "Fan". Renamed it to "Bedroom Fan" and the delay from Alexa is gone. Are any of the devices with the slow response named generically?

Hi.
Generally not with the exception of 'heating' and 'Printer' everything else is bedroom light or living room light or hallway light etc.
I might try clearing out all the devices from Alexa and adding one back at a time to see if I can identify an issue with names or a particular device.
Thanks for replying.


Load and GetBytes array from signature component (android)

On Android Project :
Add GetBytes & LoadBytes

GetBytes -> Give byte array of current signature
LoadBytes -> Load byte array in component signature

ClearingImageView :

Trap unexepted exception in clear signature method when clearing signature component after LoadBytes.

This has been implemented slightly differently in v2.0:
Task<Stream> GetImageStreamAsync(...)

This method takes all the same values as the GetImage method, but returns a Stream that can be either encoded as a .jpg or .png.
If bytes are needed, then they can be obtained from the stream.


Bug: Submission Permissions

Issue:
Able to submit with a user who only has the prepare role and no relationship to the facility
Steps to Recreate:

Prepare a record for submission
Go to the submit queue and select the record for submission
When the authentication dialog appears authenticate with a valida CDX user who only has the prepare role and follow the rest of the certification process

Issue has been addressed
Private Zenhub Image


Environment variable to control anonymous access

https://github.com/artemiscloud/activemq-artemis-broker-image/blob/7f4a887adaa720e61281a143cf3839227cc758e0/modules/activemq-artemis-install/added/launch.sh#L16
Instead of the hard coded "--allow-anonymous", I would expect the launch.sh script to decide between that and "--require-login", depending on some environment variable (e.g. ALLOW_ANONYMOUS=true/false).

I am closing this issue based on @brusdev reply above.


non_nullable override and some fixes

Seems good. Thanks

@homu r+

:pushpin: Commit 20a3cf1 has been approved by gkoz

:hourglass: Testing commit 20a3cf1 with merge 1320093...

:sunny: Test successful - status


Issues Fitting "Same" Data into Light GBM Classifier

I have an issue fitting my data to a LightGBM Classifier that hopefully I can get some help with.

I have 1 notebook that works fine so I cloned it in order to try to optimize the flow.
The cloned notebook fails using nearly the same code on the same data. The difference between the codes is found in how I 'filter' out test and training data.
The row count is exactly the same when I try to fit() the model.
The notebook fails after I place a filter prior to the StringVectorization and OneHotEncoding steps instead of filtering out the data  post steps. The actual data that is Vectorized and Encoded is the same columns between notebooks.
Below is the data sample 1 row of the data that goes into the 'working' classifier.

Below is a data sample of the same row of data that goes into the 'non-working' classifier.

NOTES:
A. The noticeable difference in the data is the numbers assigned to the column values from the indexer. I don't know why that would affect the fit() function but it may.
B. I recently updated to the new library in MAVEN, but I haven't been able to remove the Azure:mmlspark0.17 library. Would that matter?

@dantetrick Sorry, we do not maintain MMLSpark here. Please copy the content of your issue in the appropriate repository: https://github.com/Azure/mmlspark/issues.


UI: Implement editor for in-package PackageVariant resource

/area ui

/assign @chander12shekhar

Completed with https://github.com/nephio-project/kpt-backstage-plugins/pull/8


Error in load.py for Binford Ecological DBase env_vals

WARNING:Unable to find a Society object with ext_id Ii11 and source Murdock et al. (1999), skipping...
WARNING:Unable to find a Society object with ext_id Na18 and source Murdock et al. (1999), skipping...

Those two societies (Na18, Ii11) have been deleted from the cultural dataset (they were doubles of other societies). Nf1 has also been deleted. I have just updated the ecological data so that there are no longer rows for these three societies. New Environmental data sent to @stefelisabeth.


[1.11] Add disable toggle to freerunners.

Adds feature #3808.
Also changed the default armor switch key binding from F to G as F is currently a default key binding for switching held item with offhand.

Wow! Impressive, you followed the jetpack/gas tank implementation and added full client-server sync for other users. This would make it easy to add some kind of graphic change to the free runners in the future when the mode is changed. Thanks!


[HUDI-2098]support hdfs file lock

Tips

Thank you very much for contributing to Apache Hudi.
Please review https://hudi.apache.org/contributing.html before opening a pull request.

What is the purpose of the pull request
support hdfs file lock
Brief change log
(for example:)

Modify AnnotationLocation checkstyle rule in checkstyle.xml

Verify this pull request
new UT added
Committer checklist

[ ] Has a corresponding JIRA in PR title & commit

[ ] Commit message is descriptive of the change

[ ] CI is green

[ ] Necessary doc changes done or have another open PR

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Codecov Report

Merging #3189 (e8ac5d3) into master (5564c7e) will decrease coverage by 29.78%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master    #3189       +/-   ##
=============================================
- Coverage     46.18%   16.39%   -29.79%
+ Complexity     5384      478     -4906
=============================================
  Files           921      280      -641
  Lines         40040    10888    -29152
  Branches       4294     1109     -3185
=============================================
- Hits          18492     1785    -16707
+ Misses        19664     8944    -10720
+ Partials       1884      159     -1725

Flag
Coverage Δ

hudicli
?

hudiclient
0.00% <ø> (-30.46%)
:arrow_down:

hudicommon
?

hudiflink
?

hudihadoopmr
?

hudisparkdatasource
?

hudisync
5.38% <ø> (-48.67%)
:arrow_down:

huditimelineservice
?

hudiutilities
58.59% <ø> (+0.18%)
:arrow_up:

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

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


Module not found D:\Users\nrabbitt\WebstormProjects\search copy\node_modules\react/lib/ReactDOM

I updated to 2.6.0 and am using react 15.4.1.  When webpack tries to bundle I am getting the following error.  Can anyone please help me with this?  If you need more information let me know. Thank you in advance.
ERROR in ./~/react-bootstrap-table/~/react-toastr/~/react-dom/index.js
Module not found: Error: Cannot resolve 'file' or 'directory' D:\Users\nrabbitt\WebstormProjects\search copy\node_modules\react/lib/ReactDOM in D:\Users\nrabbitt\WebstormProjects\search copy\node_
modules\react-bootstrap-table\node_modules\react-toastr\node_modules\react-dom
@ ./~/react-bootstrap-table/~/react-toastr/~/react-dom/index.js 3:17-46

hi @UX-Unicorn, could you remove node_module and install again?  it may related with #838

That worked thanks


reinit.sh fixed to work the same way with bsd sed that ships with osx

Coverage remained the same at 100.0% when pulling 6954a9362586735f1b79c0e4ff444337a8a48dce on xludx:master into 8bc640247dbf67c48a4384db26ab73625fa173c0 on sramam:master.

Coverage remained the same at 100.0% when pulling 6954a9362586735f1b79c0e4ff444337a8a48dce on xludx:master into 8bc640247dbf67c48a4384db26ab73625fa173c0 on sramam:master.


Blitz.Blitz requests upgrades indefinitely

Brief description of your issue
Blitz has an available update (always the same) whenever you run winget upgrade blitz.
Steps to reproduce

Install Blitz
Run winget upgrade blitz, get the upgrade
Repeat ^

Expected behavior
No new update should be installed until blitz actually releases one.
Actual behavior
The same update is reinstalled whenever i run winget upgrade blitz
Environment
[winget --info]
Windows Package Manager v1.0.11451
Windows: Windows.Desktop v10.0.19041.928
Paket: Microsoft.DesktopAppInstaller v1.11.11451.0

I reckon this is due to the full version number being filed in this repo, while winget only sees the 3-digit version number locally:
PS > winget upgrade
Name                 ID                   Version  Verfügbar   Quelle
---------------------------------------------------------------------
Blitz                Blitz.Blitz          1.14.17  <IP_ADDRESS>1 winget

German: Verfügbar = Available; Quelle = Source

This comes from the dynamic link and wingetbot automaticly updating it and taking the File Version from the installer.

We can fix it with a Static URL https://blitz-stable.blitz.gg/Blitz-1.14.17.exe

@OfficialEsco do we have a bug or feature on the winget-pkgs backlog to fix this, or is this the best the bot can do given the installer and the metadata?

I don’t actually know what the bot does or how it gets the Version, because it SHOULD have the same metadata, so this is probably a feature bug which looks at File Version instead of Product Version

It's been a while since we had the discussion, but I believe we look for Product Version, and if we don't find that we look at File Version. I could be wrong though. I'll double check with the team.

@OfficialEsco We check both but if we find both and major and minor are the same we return the file version.


request error

Hi,
I am a new in mining,
I am getting below error when i run node init.js
[Pool]	[XXXX (Thread 1) Could not start pool, error with init batch RPC call: {"type":"request error","message":"connect ETIMEDOUT"}
help please

Failed RPC Connection
Symptoms
An error message related to daemon RPC with {"type":"offline","message":"connect ECONNREFUSED"}

Solution
Review your daemon coin.conf settings for RPC and make sure they match your pool_configs/coin.json settings for RPC


Upgrade to nom v7

With respect to issue https://github.com/elrnv/vtkio/issues/26 I upgraded the legacy VTK parser to use nom v7 to make the crate more future proof.
For now, I mostly followed your old code directly and just "translated" it. I think for more enhancements, e.g. possibly better error handling it would be more appropriate to make new PRs.
All tests pass, however I cannot guarantee that it behaves 100% consistently compared to the old version outside of the tests (e.g. with respect to white spaces).

Wow, this is amazing! Thank you so much @w1th0utnam3! I should have some time to review on Friday. Looking forward to it, I remember this being quite a lot of effort. If I remember correctly, I think this will also make vtkio quite a lot faster with legacy files, since nom has made many improvements since v3.

Sounds good!
I will do some quick comparisons with larger files I have lying around.


Should Argb::as_hex have a leading pound sign?

Should Argb::as_hex output #abcdef or abcdef?
Since the rest of the crate has switched to lowercase abcdef in usage, should as_hex conform?
I ran into it when writing:
src/scheme/mod.rs tests
let c = Argb::from_str("0000ff")?;
let light = SchemeFromPalette::light(c);

assert_eq!(Argb::as_hex(&light.primary), "#343dff");  // pound sign required

During the day, I thought about several ways to solve this problem, and here are a few of them:

Just remove the pound in Argb::as_hex.
Add a boolean argument based on which it would be decided whether to add a pound or not.
Put it in a separate function like Argb::as_pound_hex.
Or put it in fmt::Display trait (which automatically implements Argb::to_string).

a little bit late, but closed in 808ed5b


Make use of PC stac collection to pull CMIP6 paths

Closes #36
This is a draft PR. Comments welcome on the idea with this approach @joemoorhouse @sharkinsspatial @geohacker (not sure who best to tag)
This PR:

Creates a small function to do a stac search based on a catalog url and search parameters provided as inputs. Returns a collection of items.
In the nasa cmip6 hazard class, swaps out the AWS S3 path logic in the path function for a call to the above STAC utility to retrieve the paths from the STAC metadata in the planetary computer. The paths this metadata points to are I think azure http URLs ; for now I have to change the root of these paths to point to S3. At some point we should have somewhere (in CEDA?) a STAC collection with S3 paths.

This code is not yet tested nor linted. Not familiar with pipenv, haven't tested installing the modified deps (I think we're planning on switching to pdm anyways).

Hi @emileten,
Thanks for this. I've just made some changes to make hazard a healthier environment to develop in :) (and align us with the Linux Foundation standards):

move to pdm and pyproject.toml
restructure; in particular test -> tests and moves to the standard's location
add checks (mypy, isort, black, flake8)
add tox.ini too run checks
add GitHub action to run tox as check on PR

The code changes (a lot of reformatting!) are pushed and I'm just making sure the CI/CD 'tox' step runs OK on GitHub for Python 3.8, then same for Python 3.10. Once that all works, I think you should be good to add your tests and lints (without risk of having to redo!).
Thanks,
Joe

@joemoorhouse I just realized that workflows from my fork wait from approval from a maintainer. Can we change this so that my workflows run automatically to speed up the process? Maybe changing my status to maintainer. Thanks!