Once a group has been discovered as "modified" and reported on, should it always be listed as modified? Or, should it be back in "Unmodified" going forward? It will be easier to see future changes if it is reclassified back as “Unmodified since last report” - Thanks

Actually, this may be an issue with our environment, I have made some test changes and see them in the report as expected, but one particular group seems stuck in the modified section with no new  changes and repeated report runs.

The expected behavior is that it will report as modified once, and then next time if it hasn't been modified since the last report it should appear as unmodified.
That comparison relies on successfully writing the latest groups data back to an XML file on the file system. It may be that step is failing. Can you check whether that file has been updated next time the script runs?

It has been updating the XML file. Upon further testing it seems to be related to the group owner or specifically owners. The group that continuously shows up in the “Modified Groups found” section has 2 owners and when we remove one of them it behaves as expected and shows the change on the next report run and then shows as “Unmodified” on subsequent reports.

So if the group has two (or presumably anything more than one) owners, it will repeatedly and incorrectly be listed as Modified?

That is my experience so far, I will be able to test more later today.

Yes, when I remove all but one owner, the group goes to unmodified when you run the report two times after the change.

Thanks. We'll repro that and get a fix added.

Repro'd, working on a fix now.

Hi @Roger-at-GT, this has been fixed. Groups will now report correctly when their manager attribute is either an array of multiple managers or is changed.
If you want to test the update, please try running the script from the development branch.


toMatchText failed due to whitespaces

Hi guys,
I replaced the deprecated method toHaveText with the new one toMatchText and many of the tests failed. The reason for the failure is the presence of whitespace.
For example, this code is going to fail:
<button>

Click here
</button>
------------------------
await expect(page).toMatchText('button', 'Click here');

Is there an option to modify toMatchText to run trim on textContent?

@yotov toMatchText supports regex for partial string matching.  This should work.
await expect(page).toMatchText('button', /Click here/);


Account informations

Hello
Where should i input my account login and password please ?

PatreonDownloader.App.exe --url https://www.patreon.com/****
2021-01-19 13:22:23.6044 INFO Retrieving cookies...
2021-01-19 13:23:10.0969 FATAL Fatal error, application will be closed: System.Exception: Unable to retrieve cookies
at PatreonDownloader.App.Program.RunPatreonDownloader(String url, Boolean headlessBrowser, Uri remoteBrowserAddress, PatreonDownloaderSettings settings) in E:\Sources\BigProjects\PatreonDownloader\PatreonDownloader.App\Program.cs:line 141
at PatreonDownloader.App.Program.Main(String[] args) in E:\Sources\BigProjects\PatreonDownloader\PatreonDownloader.App\Program.cs:line 74

PatreonDownloader.App.exe --url https://www.patreon.com/****
2021-01-19 13:22:23.6044 INFO Retrieving cookies...
2021-01-19 13:23:10.0969 FATAL Fatal error, application will be closed: System.Exception: Unable to retrieve cookies
at PatreonDownloader.App.Program.RunPatreonDownloader(String url, Boolean headlessBrowser, Uri remoteBrowserAddress, PatreonDownloaderSettings settings) in E:\Sources\BigProjects\PatreonDownloader\PatreonDownloader.App\Program.cs:line 141
at PatreonDownloader.App.Program.Main(String[] args) in E:\Sources\BigProjects\PatreonDownloader\PatreonDownloader.App\Program.cs:line 74

Internal browser with patreon login page should show up if login is required.

Internal browser with patreon login page should show up if login is required.


Rabbit MQ integration

After thorough investigation it was discovered that we have fully working projects that are working when correct .env files are provided. With both EmailRenderService and EmailSendingService up and running it is possible to send emails to recipient addresses. .cmd files that are responsible for launching this projects was developed and included in project.


feat: Build grpc API with swift-bridge and package with Swift

Overview
Top-level issue link
This PR goes hand-in-hand with an update to our Swift package repo to hide all of this nonsense behind a single API defined in ApiService.swift in that repo.
Why is this PR so big?
Things to ignore: 1) generated Rust proto code 2) generated Rust-Swift binding code 3) I also deleted corecrypto from the crate cause we didn't need it
What changed in the Rust<>Swift flow?
xmtp-ios should just be consuming a run-of-the-mill Swift package with a super simple Swift-only API.
How we did this was:

Use swift-bridge to generate nice C headers and Swift files from our code
Push the C headers into our include directory to get bundled into the XMTPRustSwift.xcframework
Behind the scenes, push the Swift code into xmtp-rust-swift after adding an "import XMTPRustSwift" line to each file
Finally within xmtp-rust-swift (our Swift package), we use the XMTPRustSwift.xcframework, wrap it with ApiService and deliver a nice clean Swift package interface

Right now the shim strategy is to convert protos to JSON and throw them across the Rust<>Swift boundary. We can optimize this as necessary but number one priority is to get it working.
Next Steps
Need to implement "subscribe" and make sure our Rust gRPC can target TLS.
Test Plan
Bring up a local xmtp-go-node with gRPC port mapped to 15555 (or change port in the test) and run unit tests.

Copying all our protos into this package seems less than ideal. Maybe a project for me next week is to avoid the need for that

Copying all our protos into this package seems less than ideal. Maybe a project for me next week is to avoid the need for that

Oh yeah I should make them a submodule now actually. Let me fix that

I'm hoping to get around the nightmare that is our proto repo by using Buf Schema Registry. Just pushed up the first package.
Submodule is fine for now, but I'm happy to take on buf-ifying this thing on Monday as a test before I buf-ify the mobile SDKs.

@neekolas this commit addresses the base64 issue in a very targeted and brittle way. As long as we only serialize the MessageApi protos to JSON this should be fine, but gotta keep an eye on this for sure.
https://github.com/xmtp/libxmtp/pull/50/commits/263a4d28be490f791feee5bc19c7835d3d6f3ce8
https://github.com/tokio-rs/prost/issues/75

@neekolas this commit addresses the base64 issue in a very targeted and brittle way. As long as we only serialize the MessageApi protos to JSON this should be fine, but gotta keep an eye on this for sure.
263a4d2
tokio-rs/prost#75

For a prototype, that all seems fine. I suspect we're going to want to pass these back and forth as structs to optimize perf at some point.

Gonna add one quick fix to bindings/xmtp_rust_swift, since the bindings crate is in the Cargo.toml all artifacts go into a top-level target directory. I'll change the target-dir in the Makefile.

FYI: I removed bindings/xmtp_rust_crate from the Cargo workspace so its release profile in its own Cargo.toml would be respected. This was why we were getting: warning: profiles for the non root package will be ignored, specify profiles at the workspace root: warnings


An ability to filter apps/deploys by Labels

What would you like to be added:
It would be nice if we could filter applications and deployments by Labels.
Using multiple input like environments fields in the piped registration form would be good:

One thing that I'm considering with this UI is any candidate label won't be suggested. But I would say it's fine if you can share the link to the search result.
Related:

https://github.com/pipe-cd/pipe/issues/2756
https://github.com/pipe-cd/pipe/issues/2757

Why is this needed:

/assign @nakabonne


undefined local variable or method `configuration' after upgrade to 0.2.0

Just upgraded ember-cli-rails to 0.2.0, and now when starting my Rails server and hitting http://localhost:3000, I get this:
undefined local variable or method 'configuration' for #<EmberCLI::App:0x007fcfe18fece8>
Cleared out my tmp directory with no avail.
Unsure what could be happening. I had 0.1.* working before upgrading to 0.2.0 - and I tried on a fresh Rails project -- it worked fine.
For some reason, on this existing project, I can't get it to work (even if I start the process over).

Can you provide a stacktrace?

Yes indeed!
NameError (undefined local variable or method `configuration' for #<EmberCLI::App:0x007f81c4e5db00>):
  ember-cli-rails (0.2.0) lib/ember-cli/app.rb:89:in `method_missing'
  ember-cli-rails (0.2.0) lib/ember-cli/app.rb:117:in `block in build_timeout'
  ember-cli-rails (0.2.0) lib/ember-cli/app.rb:117:in `fetch'
  ember-cli-rails (0.2.0) lib/ember-cli/app.rb:117:in `build_timeout'
  ember-cli-rails (0.2.0) lib/ember-cli/app.rb:56:in `wait'
  ember-cli-rails (0.2.0) lib/ember-cli-rails.rb:94:in `block in each_app'
  ember-cli-rails (0.2.0) lib/ember-cli-rails.rb:94:in `each'
  ember-cli-rails (0.2.0) lib/ember-cli-rails.rb:94:in `each_app'
  ember-cli-rails (0.2.0) lib/ember-cli-rails.rb:70:in `wait!'
  ember-cli-rails (0.2.0) lib/ember-cli/middleware.rb:9:in `call'
  rack (1.6.0) lib/rack/etag.rb:24:in `call'
  rack (1.6.0) lib/rack/conditionalget.rb:25:in `call'
  rack (1.6.0) lib/rack/head.rb:13:in `call'
  actionpack (4.2.1) lib/action_dispatch/middleware/params_parser.rb:27:in `call'
  actionpack (4.2.1) lib/action_dispatch/middleware/flash.rb:260:in `call'
  rack (1.6.0) lib/rack/session/abstract/id.rb:225:in `context'
  rack (1.6.0) lib/rack/session/abstract/id.rb:220:in `call'
  actionpack (4.2.1) lib/action_dispatch/middleware/cookies.rb:560:in `call'
  activerecord (4.2.1) lib/active_record/query_cache.rb:36:in `call'
  activerecord (4.2.1) lib/active_record/connection_adapters/abstract/connection_pool.rb:649:in `call'
  activerecord (4.2.1) lib/active_record/migration.rb:378:in `call'
  actionpack (4.2.1) lib/action_dispatch/middleware/callbacks.rb:29:in `block in call'
  activesupport (4.2.1) lib/active_support/callbacks.rb:88:in `call'
  activesupport (4.2.1) lib/active_support/callbacks.rb:88:in `_run_callbacks'
  activesupport (4.2.1) lib/active_support/callbacks.rb:776:in `_run_call_callbacks'
  activesupport (4.2.1) lib/active_support/callbacks.rb:81:in `run_callbacks'
  actionpack (4.2.1) lib/action_dispatch/middleware/callbacks.rb:27:in `call'
  actionpack (4.2.1) lib/action_dispatch/middleware/reloader.rb:73:in `call'
  actionpack (4.2.1) lib/action_dispatch/middleware/remote_ip.rb:78:in `call'
  actionpack (4.2.1) lib/action_dispatch/middleware/debug_exceptions.rb:17:in `call'
  actionpack (4.2.1) lib/action_dispatch/middleware/show_exceptions.rb:30:in `call'
  railties (4.2.1) lib/rails/rack/logger.rb:38:in `call_app'
  railties (4.2.1) lib/rails/rack/logger.rb:20:in `block in call'
  activesupport (4.2.1) lib/active_support/tagged_logging.rb:68:in `block in tagged'
  activesupport (4.2.1) lib/active_support/tagged_logging.rb:26:in `tagged'
  activesupport (4.2.1) lib/active_support/tagged_logging.rb:68:in `tagged'
  railties (4.2.1) lib/rails/rack/logger.rb:20:in `call'
  actionpack (4.2.1) lib/action_dispatch/middleware/request_id.rb:21:in `call'
  rack (1.6.0) lib/rack/methodoverride.rb:22:in `call'
  rack (1.6.0) lib/rack/runtime.rb:18:in `call'
  activesupport (4.2.1) lib/active_support/cache/strategy/local_cache_middleware.rb:28:in `call'
  rack (1.6.0) lib/rack/lock.rb:17:in `call'
  actionpack (4.2.1) lib/action_dispatch/middleware/static.rb:113:in `call'
  rack (1.6.0) lib/rack/sendfile.rb:113:in `call'
  railties (4.2.1) lib/rails/engine.rb:518:in `call'
  railties (4.2.1) lib/rails/application.rb:164:in `call'
  rack (1.6.0) lib/rack/content_length.rb:15:in `call'
  thin (1.6.3) lib/thin/connection.rb:86:in `block in pre_process'
  thin (1.6.3) lib/thin/connection.rb:84:in `catch'
  thin (1.6.3) lib/thin/connection.rb:84:in `pre_process'
  thin (1.6.3) lib/thin/connection.rb:53:in `process'
  thin (1.6.3) lib/thin/connection.rb:39:in `receive_data'
  eventmachine (1.0.7) lib/eventmachine.rb:187:in `run_machine'
  eventmachine (1.0.7) lib/eventmachine.rb:187:in `run'
  thin (1.6.3) lib/thin/backends/base.rb:73:in `start'
  thin (1.6.3) lib/thin/server.rb:162:in `start'
  rack (1.6.0) lib/rack/handler/thin.rb:19:in `run'
  rack (1.6.0) lib/rack/server.rb:286:in `start'
  railties (4.2.1) lib/rails/commands/server.rb:80:in `start'
  railties (4.2.1) lib/rails/commands/commands_tasks.rb:80:in `block in server'
  railties (4.2.1) lib/rails/commands/commands_tasks.rb:75:in `tap'
  railties (4.2.1) lib/rails/commands/commands_tasks.rb:75:in `server'
  railties (4.2.1) lib/rails/commands/commands_tasks.rb:39:in `run_command!'
  railties (4.2.1) lib/rails/commands.rb:17:in `<top (required)>'
  bin/rails:4:in `require'
  bin/rails:4:in `<main>'

Thanks for the speedy response @rwz!

@rwz -> Looks like it's blowing up due to me not having a timeout:  setting in my app declaration in config/initializers/ember.rb.
Without specifying it, options.fetch is returning configuration.build_timeout
https://github.com/rwz/ember-cli-rails/blob/master/lib/ember-cli/app.rb#L117
Which, configuration doesn't exist in the app instance.

Fixed in 0.2.1 :)

:) Thanks, @rwz !


CocoaPods

Hi are you going to add this repo to CocoaPods?

I will add it, but I have certain doubts regarding usage of JSON as theme file. I have a updated version without JSON, using struct for theme, shall I commit that before submitting to Cocoapods?
btw have to add swifty json as dependency if we are going JSON route

@sai-prasanna
To be honest I've tried to add cocapods support by myself. But then I saw that you're using SwiftyJSON.
In my opinion it's better if your library is not using other third party library that is not your. It's much safer that way. So if you did implement struct for theme then I think it would be a great idea to submit that before you submit to Cocoapods.

@Dudi00  Added pod support


Is snc removing unused images which are actually needed?

This came up on public IRC, setup where crc runs on an internal network, no external access to internet, so an internal registry has been setup
Full log:
16:06 < thopa> hi all,   i have installed crc version 1.5.0, Openshift version 4.2.14 on my linux laptop at work...  network access is restricted and internet is not accessible.. when i try to push an image to
               the internal registry (the push works) but deploying that image, results in an error that says a timeout occurred accessing quay.io...
16:06 < thopa> is external network required for crc to work?
16:12 < teuf> what do you mean exactly by "network access is restricted"?
16:13 < teuf> 1.5 has http proxy support
16:13 < teuf> but I guess what you have is internal network with no access whatsoever to external network?
16:14 < thopa> thats correct
16:16 < thopa> i also tried pulling an image from my org's docker image repository...  for example running this command
16:17 < thopa> oc new-app --docker-image=<my docker image repo>/mongo:3.6 --name=mongo --insecure-registry=true
16:17 < thopa> and the pod events show
16:18 < thopa> "error pinging docker registry quay.io: Get https://quay.io/v2/:dial tcp: lookup quay.io on <IP_ADDRESS>:43: read udp <IP_ADDRESS>:60495 -> 19<IP_ADDRESS>:53: i/o timeout
16:19 < thopa> so if i am trying to run a image from my repo, why is a connection quay.io being made?
16:25 < teuf> maybe 2 registries are configured, and it's looking in quay.io first?
16:26 < teuf> dunno if https://docs.openshift.com/container-platform/4.3/registry/accessing-the-registry.html could help debugging this?
16:27 < thopa> thanks.. will take a look
17:22 < thopa> found out that the quay.io error is happening because of this image "Deployer"
17:22 < thopa> deployer                                      quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:1987114d68a54affa52c4da2c7d9877580a346a9ddf215a0144cfbdbc2fc1483
17:22 < thopa> which is missing in my local repository..
17:22 < thopa> when i try to deploy my image,  openshift needs the deployer and not finding it locally, seems to be reaching out to quay.io to try and download the deployer image..
17:23 < thopa> so i am going to download this deployer image on a laptop where i have internet access, push it to my air-gapped repo and see if that fixes my issue
17:30 < teuf> thopa: ah interesting
17:30 < teuf> thopa: we remove unused images from the cluster to save on disk space/download size
17:30 < teuf> but it could be that we are being too eager in our removal

This is the same image as
$ ~/.crc/bin/oc adm release info quay.io/openshift-release-dev/ocp-release:4.2.14-x86_64  |grep deploy
  deployer                                      sha256:1987114d68a54affa52c4da2c7d9877580a346a9ddf215a0144cfbdbc2fc1483

and it's indeed missing from our crc image.
We should not be removing it if it's needed for deployments in 'disconnected' environments.

@cfergeau deployer image is not pulled by default until there is a deployment and that's the reason it is not part of  CRC, even we don't delete any images. I have a bundle which I created without deleting any images to test some bits and even in that, this image is not part of.
$ oc adm release info 4.3.0 | grep deployer
  deployer                                      sha256:dd06a8ddfda407007ce348b59424b36a1a43c17bf48102219775e902daad7037

# podman images --format json | grep -A5 dd06a8ddfda407007ce348b59424b36a1a43c17bf48102219775e902daad7037
sh-4.4#

$ oc adm release info 4.3.0 | grep prometheus
  prometheus                                    sha256:91b840bbf21b792e0d749e1b29b42d05c4e93c422184defd0cea7d79c308846a

sh-4.4# podman images --format json | grep -A5 91b840bbf21b792e0d749e1b29b42d05c4e93c422184defd0cea7d79c308846a
            "quay.io/openshift-release-dev/ocp-v4.0-art-dev@sha256:91b840bbf21b792e0d749e1b29b42d05c4e93c422184defd0cea7d79c308846a"
        ],
        "digest": "sha256:91b840bbf21b792e0d749e1b29b42d05c4e93c422184defd0cea7d79c308846a",
        "digests": [
            "sha256:91b840bbf21b792e0d749e1b29b42d05c4e93c422184defd0cea7d79c308846a"
        ],
        "created": "2020-01-21T19:17:14.95764354Z",
        "size": 341272268,
        "readonly": false
    },

We need to manually add that image to have that usecase in.

hi @praveenkumar
How can we add the image manually?


[Darwin] MTRDevice should detect device configuration change and inform delegate

As title says. Part of this is to investigate what are all the necessary / sufficient signals for this, and how it should be conveyed.

Some thoughts on what might be relevant here:

PartsList changes.
AcceptedCommandList changes.
AttributeList changes.
DeviceTypeList changes.
ClusterRevision changes.
FeatureMap changes.

Following are the changes that were tracked.
PartsList changes.
AcceptedCommandList changes -> Cluster revision should be updated for this.
AttributeList changes  -> Cluster revision should be updated for this.
DeviceTypeList changes.
ClusterRevision changes.
FeatureMap changes.
ServerList is needed for changes in clusters that are supported per endpoint


Nether sky appears very dark after installing the mod

mod version: 1.5.4 (1.20.4)

Are you sure this is related to this mod? Can you send screenshots of before and after. Just to confirm are you installing the mod on your client or the server (or both)?

Closing, because there is no activity, if you have any information feel free to reopen.


Delete unnecessary directories

Deletes .idea and .gradle directories

@unpluggedsam they are completely useless for IntelliJ users
IntelliJ would auto-generate those files if they aren't present in the directory you're importing, and Gradle does the same for projects missing the .gradle directory.
Typically, a .gitignore file would have directories such as .gradle or .idea in them, but I don't think we have a .gitignore setup.

The .idea directory contains our code style config and .github contains our Github Actions workflows as well as issue templates so why do you want to remove them?!


Err "There is no page with the number 2"

This code throw error for me: https://pastebin.com/Qee8Ai2v
Error: https://pastebin.com/eAQKwGjS
When I call "page.isLast()" it prints "false" and when I call "page.lastPage()" it prints "2". Does this mean that the second page still exists?
Why might this be happening?

Looks like you are using the #rows(Page) method. The error is because you didn't enter anything for this page in #rows(Page) and it can't find this page.
Could I see your builder?

https://pastebin.com/2ddqUiS8

I can hardly do anything with that. You have your page array, what values are in there?
Have you possibly already debugged?

https://pastebin.com/qtMV5ASJ

You have page 1 (0) and page 2 (1). Somehow you are trying to access page 3 (2), which does not exist.

Are you saying that the problem is in my algorithm for creating an array of pages?

No in the nextSlotAlgorithm() method I call the size() method. When it finds no more slot on the page, it increases the page. It seems that it then goes to the next page where it then gets the error because it doesn't exist.

Interesting

What do you say? Are there any successes?

Unfortunately, I have only now found the time to fix the error.

Update v1.5.0 If there are any problems, of course report them.


chore: update deps (#84)

This pull request updates deps and improves project setup without releasing.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
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 99.836%

Totals

Change from base Build<PHONE_NUMBER>1:
0.0%

Covered Lines:
1830

Relevant Lines:
1833

💛  - Coveralls


Replace HTTP 200 status code with 201 for POST

The correct status code for successful POST requests is 201, not 200. Even the accompanying image shows 201 as opposed to 200. The presence of a location header is confirmation that the response code is 201 as 200 does not return that header.

Internal previews

📄 File
🔗 Preview link

aspnetcore/tutorials/min-web-api.md
Tutorial: Create a minimal API with ASP.NET Core

@SuleymanCommits thanks again for the contribution!  See the Contributor License Agreement (CLA) ask for you that is above.

@dotnet-policy-service agree

Fixes #33399


semantic rules are not validated

Syntax errors in rules/foo.sr will not be found in deploy, but fail when container tries to apply the generated config.
It looks like parsing/validation used to happen in RuleConfigDeriver which was removed in
https://github.com/vespa-engine/vespa/pull/5043

Fixed in https://github.com/vespa-engine/vespa/pull/27368, released in 8.175.37


Dashboard error

[espHomeDashboard] Error: Command failed with exit code 1: /home/iobroker/.cache/autopy/python/3.11.6/bin/python3 -m venv /home/iobroker/.cache/autopy/venv/esphome Could not find platform independent libraries <prefix> Could not find platform dependent libraries <exec_prefix> Python path configuration: PYTHONHOME = (not set) PYTHONPATH = (not set) program name = '/home/iobroker/.cache/autopy/python/3.11.6/bin/python3' isolated = 0 environment = 1 user site = 1 safe_path = 0 import site = 1 is in build tree = 0 stdlib dir = '/install/lib/python3.11' sys._base_executable = '/home/iobroker/.cache/autopy/python/3.11.6/bin/python3' sys.base_prefix = '/install' sys.base_exec_prefix = '/install' sys.platlibdir = 'lib' sys.executable = '/home/iobroker/.cache/autopy/python/3.11.6/bin/python3' sys.prefix = '/install' sys.exec_prefix = '/install' sys.path = [ '/install/lib/python311.zip', '/install/lib/python3.11', '/install/lib/python3.11/lib-dynload', ] Fatal Python error: init_fs_encoding: failed to get the Python codec of the filesystem encoding Python runtime state: core initialized ModuleNotFoundError: No module named 'encodings' Current thread 0x0000007fb995b040 (most recent call first): <no Python frame>

Thank you, what kind of system ist this, Linux/docker/windows ?

ioBroker installed on an Raspberry Pi 4

Interesting, which Operation System?
It should work on any Linux based OS

Linux ioBroker1 6.1.0-rpi6-rpi-v8 #1 SMP PREEMPT Debian 1:6.1.58-1+rpt2 (2023-10-27) aarch64 GNU/Linux

Linux ioBroker1 6.1.0-rpi6-rpi-v8 #1 SMP PREEMPT Debian 1:6.1.58-1+rpt2 (2023-10-27) aarch64 GNU/Linux

Would it be possible to try on in Ubuntu setup?
Trying to find out of it's related to Debian 😅

I just installed version 0.5.0-beta.8 on my test environment. Initially, there were errors. I reconfigured the adapter and restarted it several times. Now it seems to be working.

would it be possible to share the errors ?


nodePackages.matrix-appservice-irc: Init

I know I didn't regenerate anything, but that leads to a lot of
conflicts if somebody with direct master access adds a package first.

Motivation for this change
I'm running my own IRC bridge.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Already proposed in https://github.com/NixOS/nixpkgs/pull/62864, which also adds a wrapper and a module for matrix-appservice-irc.

Closing in favor of #62864


View not coming full height in Android

Description
I'm trying to show an ImageBackground as full screen, For that, I have given its parent and  flex:1, so that it will cover the whole screen. It's working in iOS but not in Android. I'm not using SafeAreaView.
React Native version:
"react-native": "0.63.3"
Steps To Reproduce
This is what I'm trying to achieve.

This is what I'm getting.

This is working fine in iOS. FYI, I'm using react-navigation v5.
This is the component code.  https://gist.github.com/praveen-wal/79c495e58f54d5b8b7ace5f93d3bbff9

Able to solve it using this <StatusBar translucent backgroundColor="transparent" />


Increased search limit for deliverables

#2831  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tommylau523 and the rest of your teammates on  Graphite


Enable rules in eslintrc

When working in #11, I disabled many rules to be able to incrementally migrate to TypeScript. Let's re-enable them:

[x] #20
[x] #21
[x] #22
[x] #23
[x] #25
[x] #24

All done! 🎉


Automated cherry pick of #10790: fix: avoid panic

Cherry pick of #10790 on release/3.6.
#10790: fix: avoid panic

Codecov Report

Merging #10792 (d43e645) into release/3.6 (b9d30c0) will increase coverage by 6.06%.
The diff coverage is 0.46%.

@@               Coverage Diff                @@
##           release/3.6   #10792       +/-   ##
================================================
+ Coverage         5.86%   11.93%    +6.06%
================================================
  Files              828      232      -596
  Lines           129133    25991   -103142
================================================
- Hits              7577     3102     -4475
+ Misses          120876    22578    -98298
+ Partials           680      311      -369

Flag
Coverage Δ

aFlag
11.93% <0.46%> (+6.06%)
:arrow_up:

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

Impacted Files
Coverage Δ

pkg/apigateway/handler/auth.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/auth_totp.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/idp.go
1.84% <0.00%> (ø)

pkg/apigateway/handler/misc.go
0.00% <0.00%> (ø)

pkg/apigateway/handler/resource.go
0.00% <0.00%> (ø)

pkg/apis/const.go
0.00% <0.00%> (ø)

pkg/apis/input.go
0.00% <ø> (ø)

pkg/apis/list.go
0.00% <ø> (ø)

pkg/baremetal/cronjobs.go
0.00% <0.00%> (ø)

pkg/baremetal/manager.go
0.50% <0.00%> (-0.01%)
:arrow_down:

... and 580 more

Continue to review full report at Codecov.

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


publish<EMAIL_ADDRESS>

Requested by: @bitsandfoxes
Quick links:

View changes
View check runs

Assign the accepted label to this issue to approve the release.
Targets

[ ] symbol-collector
[ ] upm
[ ] github
[ ] registry

Targets marked with a checkbox have already been executed.  Administrators can manually tick a checkbox to force craft to skip it.

We'll add something else to it and make a new release


Install via Carthage

Can the library be installed via Carthage?
Currently trying to build the product by adding this to my Cartfile:
github "wickwirew/Runtime" == 1.1.0

I get
*** Skipped building Runtime due to the error:
Dependency "Runtime" has no shared framework schemes

Possibly with a little bit of work. I can try to get to it some time this week

Any news here?


Logout callback

Looking at this link https://help.sap.com/viewer/8b8d6fffe113457094a17701f63e3d6a/GIGYA/en-US/4137589670b21014bbc5a10ce4041860.html, in JS sdk there are some parameters to add on Logout, in particular a we can add a callback to check the response. How can I check error case?
How can we do this on flutter?

Hi,
This feature is not supported by the Core native SDKs and therefore cannot be applied in the Flutter plugin.
It is currently not in our backlog.
You can register a feature request with support for this.
Regards,
Tal


External CSS

How would I add an external stylesheet to a notebook?
Thanks!

Are you trying to style an html cell, or are you trying to change the appearance of beaker itself?

html cell.  I have some javascript (yasqe.js) that has some svg that isn’t rendering correctly.

On Mar 14, 2017, at 8:40 AM, Andrew M Olney<EMAIL_ADDRESS>wrote:
Are you trying to style an html cell, or are you trying to change the appearance of beaker itself?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/twosigma/beaker-notebook/issues/5022#issuecomment-286441817, or mute the thread https://github.com/notifications/unsubscribe-auth/ABPKhVYbXUfsmr4Lp0sMsAokHP_8QHlUks5rlqbZgaJpZM4Lze44.

Perhaps you got further than me, but in the dev console I'm getting "YASQE not defined" with the code below. Sorry I'm not a dev here but thought I might be able to help.
<!DOCTYPE html>

  <title>Example</title>
  <meta charset="utf-8">
  <link rel="stylesheet" href="https://cdn.jsdelivr.net/yasqe/2.11.9/yasqe.min.css">


The demo package is using @wdio/sync which isn't compatible with Node v16

Describe the bug
https://github.com/webdriverio-community/wdio-video-reporter/blob/v3.3.0/demo/package.json

The demo package is using @wdio/sync which isn't compatible with Node v16
The package version is 3.0.0 but the tag is 3.3.0.

Environment (please complete the following information):

wdio-video-reporter version: 3.3.0

We have updated the demo to not use sync anymore. Thanks for reporting.


Trigger next "mood"

Is it possible to switch to a next "mood"? Similar to what happen when you click the middle button of Loxone Touch light switch.
My use case is to use non-loxone button (zigbee using zigbee2mqtt) as additional light switch.

You can implement it yourself like I showed herer:
https://github.com/JoDehli/PyLoxone#example-2-with-a-updownanalog

@rnovacek
You can do it with a service call on the uuid from the lightcontroller like this:

If this works for your light controller you can combine this service with a script which you can trigger from the zigbee device.
Can you close the issue if it works?

Awesome, I'll give it a try this week and will report back.

It works! Thank you very much. This automation does the trick for me:

For the reference, all I had to do is to open http://<loxone-ip>:<port>/data/LoxAPP3.json, find the room by name in rooms top level object, use the uuid of that room to find LightControllerV2 with room equal to that uuid and use the uuid of LightControllerV2 in the action. And value: plus means next mood.
Shall I open a PR to add this comment to the readme?

@rnovacek If you want you can add your problem and your solution as a 3rd example to the main page with a PR.


Translations

Hello,
For https://github.com/AntennaPod/AntennaPod we're looking into implementing support for the <podcast:person> tag.
When we get to display persons on podcast and/or episode pages, we would most likely also display the roles, based on your taxonomy. These role names would also have to be translated, in order to fully integrate with the rest of the app.
We could translate this at the level of our project, but for consistency across apps and for efficiency it would probably be helpful to do translations at your level.
We would be happy to ask our translators (all volunteers, just like me and the rest of the AntennaPod team) to translate the role titles, and submit a PR with translations. Descriptions and Groups should probably be translatable as well, but are of lower priority.
I'm wondering, therefore, if a) if you're open to such collaboration and b) if we could agree on the structure of translations files. (I was thinking folder translations, one file per language (without country code if possible) e.g. pt_PT.json and in each file an array with roles, as such: code, positionTitleamd description.
(Side note on groups, because while potentially relevant to other projects for AntennaPod we won't proces or display them: For the efficient translations of groups, if desired, I would change the json to include groupCodes and define the group names in a dedicated array or json file. That way each group name would only be translated once.)
Many thanks for considering!

Hi @keunes! Thanks for your offer to help with translating. This all sounds good, so if you're happy to work on translations you can submit a PR and we can then incorporate the changes into the repo.
Your point on groups is true, though I want to ensure we're not breaking any existing apps by removing functionality. I suggest we keep groupTitle as it is in the existing file, but add groupCode as well, which would refer to the code of the group in a new groups.json or creator-role-groups.json file (trying to consider future uses).

Thanks @ryanstockau for the reply. I'll create a PR to add the group codes and file.
For the actual translations, I would hook up the repo with Weblate, which would automatically create PRs once in a while, as translations drip in. Can you just tell me what folder/file structure you would expect?


Preview Blocks

We new implement the preview concept in block.

Implementing the preview concept in block is a fantastic development! It’s exciting to see how this feature will enhance our user experience by allowing us to see and interact with content before finalizing it.

Thank you for your response I'm looking forward to seeing how it enhances our  user experience by allowing us to interact with  content before finalizing it.


Typing "=" when cursor is before ")" causes it not to be typed and the cursor instead moves to after the ")".

Does not occur when the plugin is disabled so I'm pretty sure it must be the plugin's fault. This might be an intended feature, idk, but how do I turn it off?

hi ! @wobbba
sorry, I was not able to replicate the issue on my machine.
As this plug-in does not have any trigger related to the = sign, I could not see how this could have happened...
If the problem still persist, you might like to check out the other community plugin called "Latex Suite" by artisticat1 which has many similar functionality as this one, as well as many other really neat features~

That is very odd since it also happens with all other plugins disabled (even all core plugins). I can't think of anything else that could be the cause. I'm using plugin version 2.4.0 with Obsidian version 1.0.0. There is also nothing suspicious in my custom shorthand.
Actually, I have now narrowed it down to the "Autoclose round brackets" feature. Disabling that fixes the problem. Having only that enabled, makes the problem appear. It's like it has the two characters mixed up since when I type a ) when the cursor is already in front of a ) then I'd expect the behaviour it shows with the = but instead it adds another ).

Hi! @wobbba
Thank you so much for sharing all the details!
Although.... sorry I still could not replicate this issue...
However, I updated the plugin to v2.5.1 with some changes to the "Autoclose round brackets" and other related functions. Please kindly see if that might happen to resolve your issue?

Thanks but the issue persists. I found another thing: If the Autoclose curly brackets option is enabled, if I try to input a ? between two $$, it instead comes out as ${}$. I don't really know what we could try, maybe it is a problem with keyboard layout? I'm using German layout.
Yeah, I tested English layout and that is probably the problem. When I try to input an = I press Shift+0 which is closing round bracket in US layout. Does that ring a bell for you? Otherwise I'm gonna assume that this is an Obsidian-Problem that arises in the interaction with your code for some reason and create an issue there.
Anyway thank you for being so responsive and for creating this plugin!! :)

I have the same problem and I am using a German layout as well. It did work a few versions back though.
Another issue related to this is that the closing ) occasionally adds a ( as well because the plugin seems to fail at detecting the existing ( in some cases.

Im using german Keyboard Layout, And when I tested it right now, neither the '$(=)$'-problem, nor the '$?$'-problem occured, has this maybe been fixed in newer versions? :)
While testing this a little bit I found another weird behaviour: highlighting ')' and then inserting a '=' , now returns '=)=' ...?
Still an improvement over the bugs described by @wobbba though.


chore: disable internal packages

:tada: This PR is included in version 2.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Multiple Exception

Is it possible to use this extension for multiple exceptions?

I think so?
One of the tests in the library is similar to:
@retry_decorator.retry(ExceptionToCheck=(TestError, AttributeError), tries=2)

If this isn't what you're looking for (or you're unsure on the exact usage), please re-open.


DeepCopy Error When Importing Client-Go

Bug description
When importing client-go compatible types e.g. istionetworking "istio.io/client-go/pkg/apis/networking/v1alpha3" go vet produces errors such as vendor/istio.io/client-go/pkg/apis/networking/v1alpha3/zz_generated.deepcopy.gen.go:16:9: in.Spec.DeepCopyInto undefined (type v1alpha3.DestinationRule has no field or method DeepCopyInto)
Affected product area (please put an X in all that apply)
Extensibility? Not sure if there is an area for this?
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[X] User Experience
[X] Developer Infrastructure
Expected behavior
No errors. Types are available to calling code such as a custom controller.
Steps to reproduce the bug
Create a basic controller. Import "istio.io/client-go/pkg/apis/networking/v1alpha3"
Version (include the output of istioctl version --remote and kubectl version)
branch = "master"
name = "istio.io/client-go"
packages = ["pkg/apis/networking/v1alpha3"]
revision = "97bfafa2744d19a4953eb9a70e2300fd920a8d72"
How was Istio installed?
dep ensure
Environment where bug was observed (cloud vendor, OS, etc)
Development environment. Local build.

@rcernich  @nrjpoddar

The PR addresses only a partial fix, we need to regenerate client code and then close this issue.

I'll regenerate the client once a new tools image is available.

Note that we will need to cherrypick for 1.4 release.

This is fixed in master of istio.io/client-go as I have an example package using the generated code successfully: https://github.com/istio/client-go/pull/16


Uncaught Error: Module did not self-register.

[Enter steps to reproduce:]

...
...

Atom: 1.46.0 x64
Electron: 4.2.7
OS: Unknown macOS version
Thrown From: Atom Core
Stack Trace
Uncaught Error: Module did not self-register.
At ELECTRON_ASAR.js:160

Error: Module did not self-register.
    at process.module.(anonymous function) [as dlopen] (ELECTRON_ASAR.js:160:31)
    at Object.Module._extensions..node (internal/modules/cjs/loader.js:722:18)
    at Object.module.(anonymous function) [as .node] (ELECTRON_ASAR.js:169:18)
    at Module.load (internal/modules/cjs/loader.js:602:32)
    at tryModuleLoad (internal/modules/cjs/loader.js:541:12)
    at Function.Module._load (internal/modules/cjs/loader.js:533:3)
    at Module.require (/app.asar/static/index.js:72:46)
    at require (internal/modules/cjs/helpers.js:20:18)
    at customRequire (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:1:695134)
    at get_KeyboardLayoutManager (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2778162)
    at get_manager (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2778264)
    at Object.getCurrentKeyboardLayout (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:2778390)
    at e.keystrokeForKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:14:1081110)
    at KeymapManager.keystrokeForKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1253631)
    at KeymapManager.handleKeyboardEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:1251616)
    at WindowEventHandler.handleDocumentKeyEvent (/Applications/Atom.app/Contents/Resources/app/static/<embedded>:11:284744)

Commands
     -0:53.6.0 core:select-all (input.hidden-input)
     -0:33 fuzzy-finder:toggle-buffer-finder (input.hidden-input)
  4x -0:11.9.0 core:close (atom-workspace.workspace.scrollbars-visible-when-scrolling.theme-atom-dark-syntax.theme-one-dark-ui)

Non-Core Packages
terminal-plus 0.14.5

Thanks for taking the time to contribute!
We noticed that this is a duplicate of https://github.com/atom/atom/issues/20222. You may want to subscribe there for updates.
Because we treat our issues list as the Atom team's backlog, we close duplicates to focus our work and not have to touch the same chunk of code for the same reason multiple times. This is also why we may mark something as duplicate that isn't an exact duplicate but is closely related.
For information on how to use GitHub's search feature to find out if something is a duplicate before filing, see the How Can I Contribute? section of the Atom CONTRIBUTING guide.


[ResourceManager] Major issues with Lock commands

There are a number of serious usability issues with the az lock commands. As far as I can tell, only the list command works...

[ ] There are no tests for any of the lock commands.
[ ] I could not create a lock on my VNET using the az lock create command. It says: The resource type could not be found in the namespace 'Microsoft.Network' for api version '2015-01-01'. It could be using the API version of the lock API but needing the API version of the virtualNetworks resource. The az resource commands handle this behind the scenes.
[ ] As reported in #2688, I cannot delete a lock with az lock delete. If I specify only the name (which is the only required parameter but logically shouldn't work because the resource is associated with a resource group) I get no output. If I specify name and some of the optional parameters, I get no output. If I specify ALL of the parameters, I get the same output about not finding the resource type for the API version as when I try the lock create command.
[ ] az lock show takes resource group and name as parameters, but I cannot show a lock that I know exists by specifying these. It simply returns nothing.
[ ] --parent-resource-path is listed as optional, but if you omit it, it says "Parameter 'parent_resource_path' can not be None." You have to specify ''...
[ ] The parameter names for specifying the resource are inconsistent with the az resource commands. Specifically: az lock: --parent-resource-path, --resource-provider-namespace are inconsistent with az resource: --namespace, --parent.
[ ] The ids parameter is not supported for either the lock itself or the target resource. It should be supported for both.
[ ] in az resource, --resource-type accepts the type or namespace/type. az lock does not do this.
[ ] No short summaries for any of the commands
[ ] az lock update has parameters with no help text

@tjprescott what was the specific lock command? Creating locks does work, I just tried it:

Also, listing the lock works for me:
az lock show --resource-group bburns-test --name foo

--parent-resource-path is optional depending on the incantation. It is only required to lookup locks on specific objects, so it is optional for many calls, just not all of them.

@brendandburns I was trying to create/delete a lock on a VNet when I got the API error. Listing is the only command that I could get to work.
I agree that --parent-resource-path is optional, but if you omit it, an error is raised. VNet should not require a parent resource path, but I am forced to specify --parent-resource-path '' or I will get an error.

@tjprescott can you try the command that I cut and pasted above? for create/delete, that locks a VNet resource and it worked correctly for me.
Can you clarify, create doesn't require a --parent-resource-path, are you saying that show requires one?
Thanks
--brendan

@brendandburns
The command you use does work, but it does not create a lock on the VNet... it creates a lock on the resource group because you didn't specify --resource-name. That would be a separate issue.
Attempting to create a lock on a VNet:
az lock create --lock-type CanNotDelete -g tjp-lock --resource-provider-namespace Microsoft.Networks --resource-type virtualNetworks --name lock1 --resource-name vnet1

Parameter 'parent_resource_path' can not be None.

Now once I set --parent-resource-path ''
az lock create --lock-type CanNotDelete -g tjp-lock --resource-provider-namespace Microsoft.Network --resource-type virtualNetworks --name lock1 --resource-name vnet1 --parent-resource-path ''

The resource type could not be found in the namespace 'Microsoft.Network' for api version '2015-01-01'.

Ah, thanks I will try to reproduce this.

Please see:
https://github.com/Azure/azure-cli/pull/2742
for the fix for create/delete.
Apologies for the bugs.

@tjprescott
az lock show ...

Is not recursive, that is if you try to show a lock that is at resource scope, but you don't give us the resource, it's not going to work...  I could probably work around this using list, but I'm not sure what the right approach would be.  Thoughts?
--brendan

Assigning to Willie to review and clean up.


Corrected typo in link to examples

Corrected typo in link to examples.
Description
Corrected typo in link to examples
Checklist

[x] The base branch of this PR is dev, rather than main
[x] The relevant docs, if any, have been updated or created

Screenshots
[ "For visual changes, include screenshots." ]
Testing
[ For new or modified features, provide testing instructions. ]

thanks for catching this!


Update openebs helm installation configurable parameters

OpenEBS helm documentation is a bit out of date from the openebs charts from kubernetes/stable in terms of the parameters that can configured.
openebs helm documentation:
https://docs.openebs.io/docs/next/installation.html#helm
kubernetes/stable openebs charts documentation
https://github.com/kubernetes/charts/tree/master/stable/openebs
Also review this document for typos -- like "stable" is spelt as "table"
Documentation can be edited by clicking on the "Edit" button on top of the changes and following the github PR flow.

Hi @kmova,
Even though the PR for this issue has been duly submitted and merged, the openebs helm documentation still seems to have some periods missing and the position of the chart seems to be incorrect when viewed.
How about I make the required changes by editing the changes in the above document?
CC: @NisantaSahoo

sounds good @utkarsh2102, please feel free to go ahead and make the improvements.

Sure. Would submit a PR by EOD :)

@kmova,
A PR has been made for the issue discussed above.
Kindly check the same.


Upgrade scalapb dependency

It seems scalapb 0.7.0 final is out, we should upgrade asap.

I suggest that we wait for Scala Native 0.3.7 before doing the upgrade. Otherwise, we'll have to publish ScalaPB 0.7.0 and (all or some) of its dependencies against our custom fork of Scala Native 0.3.6 (this isn't hard, but it'll take 1-2 hours of our life that we won't be getting back).

Scala Native 0.3.7 has been staged, but it doesn't work with our build yet (see my attempt at https://github.com/xeno-by/scalameta/commit/6d97eccd4dd8112240980b525a3af6ecb75f18ea):
Project loading failed: (r)etry, (q)uit, (l)ast, or (i)gnore? r
[info] Loading project definition from /Users/eburmako/Projects/scalameta/project/project
[info] Loading project definition from /Users/eburmako/Projects/scalameta/project
/Users/eburmako/Projects/scalameta/build.sbt:101: error: not found: value NativePlatform
lazy val semanticdb3 = crossProject(JSPlatform, JVMPlatform, NativePlatform)
                                                             ^
/Users/eburmako/Projects/scalameta/build.sbt:187: error: not found: value NativePlatform
lazy val metap = crossProject(JSPlatform, JVMPlatform, NativePlatform)
                                                       ^
sbt.compiler.EvalException: Type error in expression
[error] sbt.compiler.EvalException: Type error in expression
[error] Use 'last' for the full log.
Project loading failed: (r)etry, (q)uit, (l)ast, or (i)gnore?

Can this happen because we're using org.portable-scala % sbt-crossproject? I vaguely remember running into the same problem when I forked Scala Native 0.3.6. /cc @densh

Fixed in https://github.com/scalameta/scalameta/pull/1464.


HTTP, SMTP, and DNS parser enhancement

HTTP parser can parse all the request and response headers.

SMTP parser can parse all the headers

DNS parser can parse and store authoritative and additional IPs and hostnames

Please add to the tests/config.test.ini the end of the default section and add to pr

parseSMTPHeaderAll=true
parseHTTPHeaderRequestAll=true
parseHTTPHeaderResponseAll=true

Run tests.pl in there, issues I notice

pcap/wireshark-bdat  the white space before the headerValues should be removed

# *  39|          headerValue => [                                                                                *    |                                 |
# *  40|            ' from root by testhost.test.ex with local-bsmtp (Exim x.yz) (envelope-from <sender@dom>) id 1bwpfB-0000Be-Tu for<EMAIL_ADDRESS>Wed, 19 Oct 2016 13:08:05 +0100',  *    |                                 |
# *  41|            ' foo',                                                                                *    |                                 |
# *  42|            '<EMAIL_ADDRESS>                                                                                *    |                                 |
# *  43|            ' sender@dom',                                                                                *    |                                 |
# *  44|            ' Wed, 19 Oct 2016 13:08:05 +0100'                                                                                *    |                                 |
# *  45|          ],                                                                                *    |                                 |

I just want to confirm for http headers you are excluding ones already indexed, such has host/user-agent

Is there really a need for two counts?  responseHeaderValueCnt and responseHeaderFieldCnt should always be the same right?

For DNS it looks like the field name for some results has moved from ip to answer.ip.  Not sure we should be renaming fields.  I need to look at DNS more.

For tests you are happy with run ./tests.pl --make pcap/file.pcap and add to the PR

Q. I just want to confirm for http headers you are excluding ones already indexed, such has host/user-agent
A. Yes.
Q. Is there really a need for two counts? responseHeaderValueCnt and responseHeaderFieldCnt should always be the same right?
A. Yes, the values of these two fields should always be the same. The requestHeaderFieldCnt and responseHeaderFieldCnt  fields can be removed. Please advise
Q. For DNS it looks like the field name for some results has moved from ip to answer.ip. Not sure we should be renaming fields. I need to look at DNS more.
A.  The dns.ip and dns.host contain all the ips and host name. I added separate fields to clearly show what IP and host belong to what resource record (query, answer, additional, and authoritative). I introduced an option in config.ini (parseDNSRecordsAll) to control parsing of DNS metadata. If parseDNSRecordsAll is set to TRUE then fields for query, answer, additional, and authoritative will be populated and shown in the UI. I am showing the IP and host break down in a separate section (DNS Record Details) in the UI. Please review and advise
Q. For tests you are happy with run ./tests.pl --make pcap/file.pcap and add to the PR
A. I made changes in capture test castes according to your suggestion. The capture test-cases are passing. However, some of the viewer test cases are failing. I am working to fix those.

Great, making progress.

Can you make sure there are no trailing spaces in new code "/ $", git will highlight
I'm not sure I want the new hosts/ips added to old dns fields.  Is that just to make queries easier?

Q. Can you make sure there are no trailing spaces in new code "/ $", git will highlight
A. Sure, I will work on it.
Q. I'm not sure I want the new hosts/ips added to old dns fields. Is that just to make queries easier?
A. Yes, one reason is to make the query easier. A user can search for any host and ip using dns.ip and dns.host APIs.
I think this is a design problem. I need your suggestion in this regard. When parseDNSRecordAll is set TRUE, then I populate the IPs  (dns.query.ip, dns.answer.ip, dns.authoritative.ip, dns.additional.ip, dns.update.ip) and hosts ( dns.query.host, dns.answer.host, dns.authoritative.host, dns.additional.host, dns.update.host)  from DNS records to show a break down in the UI.  The dns.host and dns.ip are shown in the DNS section, which means the DNS section shows a unified information. Next, I show the IP and Host break down in the DNS Record Detail section.

I think probably shouldn't populate it twice, instead should maybe make two new fake fields that query all those.  like host.dns.all and ip.dns.all maybe?  And leave host.dns and ip.dns the way it is now.  Or maybe even turn those off writing ip.dns and host.dns fields.  duplicate data in es always makes me sad.  speaking of duplicate data, since you don't duplicate http headers, it looks like you are duplicate smtp headers.  on purpose?

Yes, I am parsing SMTP header twice. I did not notice it. Thanks for pointing it out. I will provide a fix in the next commit. The next commits will include the following changes: a) SMTP header parsing fix; b) DNS header fix c) viewer taste cases fix. Please suggest.

I have completed making the changes. The changes are as follows:
HTTP Parser

If parseHTTPRequestHeaderAll is set to TRUE in the config.ini then all the HTTP-headers are parsed and stored in ES.
If parseHTTPResponseHeaderAll is set to TRUE in the config.ini, then all the HTTP-headers are parsed and stored in ES.
It is ensured that no HTTP header is parsed twice
The values of the HTTP headers are shown in a separate section (HTTP Header Detail) in the session detail GUI
Test cases were updated according to the changes

SMTP Parser

If parseSMTPHeaderAll is set to TRUE in the config.ini, then all the email-headers are parsed and stored in ES. It is ensured that no email header is parsed twice
The values of the Email headers are shown in a separate section (Email Header Detail) in the session detail GUI
SMTP test cases were updated according to the changes

DNS Parser

If parseDNSRecordAll is set to TRUE in the config.ini then results from Authoritative and Additional records are parsed. Name Server and Mail Exchange server information (IPs and Hostnames) are in separate labels in the UI.
DNS test cases were updated according to the changes


Should the panic hook propagate messages to the caller?

At line https://github.com/dfinity/cdk-rs/blob/9809f51d1299d2a757df5313b399881ef13f09fe/src/ic-cdk/src/printer.rs#L20, the message passed to a panic! is sent to api::print. This means that it might (depending on the replica) get logged, but it will not be propagated back to the caller.
This leads to poor messages on the caller's side. I'd recommend calling ic_cdk::api::trap instead.
Maybe the idea was that too much details should NOT go to the caller; why not. But if so, then we need a good API for custom rejects, so that there is a way to send the caller good error messages. Today, I believe the only way to reject is through the guard attribute.

CC @dsarlis


Literal block style multilines don't survive round trip; can it be forced?

When using literal multiline block strings inside my YAML file, converting to JS, and then back,  sometimes the multiline literal block survives, but not always.
Is there a way to force a particular behavior? Having them be literal block strings inside the YAML is, IMHO, definitely more readable and editable by humans.
An example program:
const yaml = require('js-yaml')

const bad = `
abc:
  - name: xxx
    sample: |
      {
          "abc": 2,
          "def" : {
          "efg" : {
      	"ok" : 6666,
      	"nope" : "./yyy",

`.trim() // make sure the whitespace is not the difference

const good = `
abc:
  - name: xxx
    sample: |
      {
          "abc": 2,
          "def" : {
`.trim() // make sure the whitespace is not the difference

const badDoc = yaml.safeLoad(bad)
const goodDoc = yaml.safeLoad(good)

const goodDumped = yaml.safeDump(goodDoc)
if( goodDumped === good ) {
    console.log( "Round trip was good for GOOD file\n" )
}

const badDumped = yaml.safeDump(badDoc).trim()
if( badDumped !== bad ) {
    console.log( "Round trip was bad for BAD file" )
    console.log( "INPUT: ", bad, "\n\n" )
    console.log( "OUTPUT: ", badDumped,"\n\n" )
}
else {
    console.log( badDumped, bad )
}

The output of this is:
Round trip was good for GOOD file

Round trip was bad for BAD file
INPUT:  abc:
  - name: xxx
    sample: |
      {
          "abc": 2,
          "def" : {
          "efg" : {
      	"ok" : 6666,
      	"nope" : "./yyy",

OUTPUT:  abc:
  - name: xxx
    sample: "{\n    \"abc\": 2,\n    \"def\" : {\n    \"efg\" : {\n\t\"ok\" : 6666,\n\t\"nope\" : \"./yyy\",\n"

Note that sample has been turned into a string with newlines encoded, rather than a literal multiline block.
It seems strange that the shorter version (good) does preserve literal block style, but the longer one does not. Am I missing something subtle about using the library?
(IMHO, it would seem more usable to have shorter strings drop the literal multiline block, while longer ones always use the | format, but the library works the opposite way.)
I would really like a way to always have multiline values (any strings with newlines) to be output as literal multiline block strings so that I have the option to edit them by hand easily.
I'm using version "js-yaml": "^3.12.1"

No option to force style. There are only some logic to select the most suitable automaticall. May be, it can be improved.

I'm happy to help with a PR if you give me some guidance on where to look @puzrin . Thanks for looking at the issue.

Somewhere here. This part was done not by me, i don't remember details.

Interesting comment here:
// Also prefer folding a super-long line.
https://github.com/nodeca/js-yaml/blob/master/lib/js-yaml/dumper.js#L301
I wonder if this means the person prefers folding (collapsing) a long line. I prefer the opposite behavior, if I understand what they mean.

Perhaps related: I'm seeing SOME |-style block strings dumped as >, which will ruin the spaces when read back again. But not both.
# input
---
mode: citation
input:
  - id: ITEM-1
    type: book
result: |
  a,d,e,f
csl: |
  <?xml version="1.0" encoding="utf-8"?>
  <style xmlns="http://purl.org/net/xbiblio/csl" class="note" version="1.0.1" default-locale="en-US">
    <info><id>https://cormacrelf.net/citeproc-rs/test-style</id><title>test-style</title></info>
    <macro name="Inner">
      <text value="d" />
      <text value="e" />
    </macro>
    <citation>
      <layout>
        <group delimiter="," >
          <text value="a" />
          <text macro="Inner" />
          <text value="f" />
        </group>
      </layout>
    </citation>
  </style>

# yaml.safeDump(yaml.safeLoad(...input...))
---
mode: citation
input:
  - id: ITEM-1
    type: book
result: |
  a,d,e,f
csl: >
  <?xml version="1.0" encoding="utf-8"?>

  <style xmlns="http://purl.org/net/xbiblio/csl" class="note" version="1.0.1"
  default-locale="en-US">
    <info><id>https://cormacrelf.net/citeproc-rs/test-style</id><title>test-style</title></info>
    <macro name="Inner">
      <text value="d" />
      <text value="e" />
    </macro>
    <citation>
      <layout>
        <group delimiter="," >
          <text value="a" />
          <text macro="Inner" />
          <text value="f" />
        </group>
      </layout>
    </citation>
  </style>

I made a pull request for a change that adds a flag that makes it possible to force literal block style.

// Also prefer folding a super-long line.

This comment does not make functional sense.  Anyone using this library to generate YAML containing multi-line blocks with white space sensitive content that goes over an arbitrary line length will get caught on this (which is increasingly common given the pattern of inlining resources in Kubernetes objects such as ConfigMaps).  I think it makes more sense for the code to attempt to honor the contents of the string, which would essentially always be |.

Seems like is pass lineWidth to be -1 will set block style to be STYLE_LITERAL https://github.com/nodeca/js-yaml/blob/d6983dd4291849b2854e8d26e1beb302edfd4c76/lib/js-yaml/dumper.js#L274

It seems strange that the shorter version (good) does preserve literal block style, but the longer one does not. Am I missing something subtle about using the library?

You are missing something subtle about your document. There are tabs (\t, 0x09) in the longer version, but not in the shorter version. That's the difference.
Whether tabs should or should not be in output document is an interesting question. But adding a separate option just for these feels wrong.

Could it be that js-yaml adds these \t somehow, because I have the same problem of having some strings (that did not change) basically randomly switch between " and |


Add propagation API

This is the first step toward #777. After this, all call-sites will still need to be updated to use the new API.

Codecov Report

Merging #797 into master will decrease coverage by 25.26%.
The diff coverage is 56.25%.

@@             Coverage Diff             @@
##           master     #797       +/-   ##
===========================================
- Coverage   92.56%   67.29%   -25.27%
===========================================
  Files         237       79      -158
  Lines       10435     2003     -8432
  Branches     1013      295      -718
===========================================
- Hits         9659     1348     -8311
+ Misses        776      655      -121

Impacted Files
Coverage Δ

packages/opentelemetry-api/src/api/propagation.ts
56.25% <56.25%> (ø)

.../opentelemetry-plugin-ioredis/test/ioredis.test.ts

...in-dns/test/integrations/dnspromise-lookup.test.ts

...entelemetry-plugin-dns/src/enums/AttributeNames.ts

...ry-plugin-dns/test/functionals/dns-disable.test.ts

...ackages/opentelemetry-exporter-zipkin/src/utils.ts

packages/opentelemetry-plugin-https/src/version.ts

packages/opentelemetry-plugin-dns/src/version.ts

...opentelemetry-base/test/resources/resource.test.ts

...s/opentelemetry-metrics/test/MeterProvider.test.ts

... and 151 more

@open-telemetry/javascript-approvers we need more reviews on this one!

Maybe I'm not understanding this fully but I miss the Composite Propagator. Or is this added by a separate PR?

Maybe I'm not understanding this fully but I miss the Composite Propagator. Or is this added by a separate PR?

Will be a separate PR as the composite propagator is an SDK detail. From the API perspective it is the same as any other propagator.

Will be a separate PR as the composite propagator is an SDK detail. From the API perspective it is the same as any other propagator.

Really? API exposes initGlobalPropagator so it's only possible to set the one and only propagator but nothing to add your propagator into list of the composite.

Will be a separate PR as the composite propagator is an SDK detail. From the API perspective it is the same as any other propagator.

Really? API exposes initGlobalPropagator so it's only possible to set the one and only propagator but nothing to add your propagator into list of the composite.

In your opinion is the composite propagator an API concern or a blocker for this PR? Keep in mind, with this PR we are only adding the propagation API. No code paths are actually using it yet.

Nope, it's not blocking.


Sc-5371: Refactoring certificate submit component

Type of change

[ ] bug fix
[ ] new feature
[ ] documentation
[x] Refactoring

Scope of changes
Sc-5371: Refactoring certificate submit component
Acceptance criteria

Author checklist

[x] I have manually tested the change and/or added automation in the form of unit tests or integration tests
[ ]  I have updated the dependencies list
[ ]  I have recompiled and included new protocol buffers to reflect changes I made
[ ]  I have added new test fixtures as needed to support added tests
[ ]   Check this box if a reviewer can merge this pull request after approval (leave it unchecked if you want to do it yourself)
[x]  I have moved the associated ShortCut or Jira ticket to "Ready for Review"

Reviewer(s) checklist

[ ] Any new user-facing content that has been added for this PR has been QA'ed to ensure correct grammar, spelling, and understandability.

Codecov Report

Merging #531 (5af75d9) into main (e33c1c1) will decrease coverage by 12.35%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main     #531       +/-   ##
===========================================
- Coverage   44.78%   32.42%   -12.36%
===========================================
  Files         470      236      -234
  Lines       13714     1986    -11728
  Branches     1058      457      -601
===========================================
- Hits         6142      644     -5498
+ Misses       6595     1334     -5261
+ Partials      977        8      -969

Impacted Files
Coverage Δ

...omponents/CertificateReview/BasicDetailsReview.tsx
75.00% <ø> (ø)

...rc/components/CertificateReview/ContactsReview.tsx
68.42% <ø> (ø)

...components/CertificateReview/LegalPersonReview.tsx
79.16% <ø> (ø)

...ts/CertificateReview/TrisaImplementationReview.tsx
93.33% <ø> (ø)

...i/src/components/CertificateReview/TrixoReview.tsx
78.37% <ø> (ø)

...eb/gds-user-ui/src/components/Head/LandingHead.tsx
12.50% <ø> (ø)

...ds-user-ui/src/components/Header/LandingHeader.tsx
16.66% <ø> (ø)

.../gds-user-ui/src/components/ReviewSubmit/index.tsx
4.34% <0.00%> (ø)

web/gds-user-ui/src/components/Section/AboutUs.tsx
0.00% <ø> (ø)

web/gds-user-ui/src/components/Section/JoinUs.tsx
25.00% <ø> (ø)

... and 238 more

Continue to review full report at Codecov.

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


feat(context-menu, text-button, button): add readonly prop

To Reviewers

[ ] Skipping review is not a problem (style, chore ONLY)
[ ] Not that difficult

Type of Change

[ ] New feature
[ ] Bug fixes
[x] Feature improvement

[ ] These changes require a usage change

[ ] Others (performance improvement, refactoring, CI/CD, etc.)

Checklist

[x] Updated Storybook documents for component
[ ] Wrote test codes for composable, utils and etc.
[ ] Tested with console(if usages are changed)

Description
SSIA
Button
https://user-images.githubusercontent.com/26986739/210499045-b96bb2fe-42ad-4708-bd1a-9995ad4b479c.mov
Text Button
https://user-images.githubusercontent.com/26986739/210499150-f8713c20-a9f3-4fc8-a5b1-6a821cfbc95d.mov
Context Menu
https://user-images.githubusercontent.com/26986739/210499076-99293a21-8597-4e2a-8f6f-d210022e15fb.mov
Things to Talk About

:tada: This PR is included in version 1.1.0-beta.505 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Badge Type List

Summary:
As a DHO user I would like to see a list of all Role Archetypes in an overview page.
Description:
old URL:
/#/documents/badges
new URL: (?)
/organization/archetype
/archetype
/role-archetype
/activity?type=badge-type
Acceptance Criteria:

[ ] Headline: Badge Types
[ ] Copy: Some paragraphs of explanation copy

List / Card View of active assignments with the following elements:

[ ] Label for document type (Badge Type)
[ ] Icon
[ ] Title
[ ] Date activated

Filter:

[ ] to be done later (currently not needed with less than 15 badges)

Action

[ ] When I click on the card the user goes to the detail view page https://github.com/hypha-dao/dho-web-client/issues/548

Sorting:

[ ] sort by newest activated role archetypes

Ressources:
Designs:
https://xd.adobe.com/view/a31415be-a724-49ad-8070-1a7d67dd6880-f647/screen/d667f4ec-b394-413f-aca5-be9b3e74f1f7?fullscreen
https://xd.adobe.com/view/d802317c-bf4d-4084-a77e-b6c2d046d49c-8d51/

@pugakn we added some buttons on the badge list. Can you estimate how much work it is to add that button? Then I will decide if it is MVP or not:
https://xd.adobe.com/view/2e6dc275-76dc-4722-a922-cc60c5d7d653-7b3e/screen/d667f4ec-b394-413f-aca5-be9b3e74f1f7/
It is the same button as on the badge detail page: https://github.com/hypha-dao/dho-web-client/issues/548


GEODE-9325: Replace old ITs worker spawner

Replaced references to ACE_Process realated to old ITs process
spawners.
Added Boost program_options project, given that was needed to fulfill
above changes.

Regarding the progress of this PR. I am seeing that for Windows whenever the test fails ctest gets stuck until all the processes terminate. I am currently looking into it. I'll publish the PR as soon as I get that working correctly.

Regarding the progress of this PR. I am seeing that for Windows whenever the test fails ctest gets stuck until all the processes terminate. I am currently looking into it. I'll publish the PR as soon as I get that working correctly.

Follow up on this: the issue have been solved in Revision 1


Add build badge for color tool master.

This adds a badge for the master branch line CI.
This will also be an end-to-end test of pull request validation and the master branch CI.

@miniksa,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Overkapping of revies with photos below

we can add a separation between the two.

Hey please allow me to resolve this issue.
thank you


CIF-888 - re-add all content package

Description
Re-add all content package which was removed during the cleanup.
Related Issue
CIF-888
Motivation and Context
Re-add all content package to have one single dependency for customer projects.
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[X] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[X] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes and the overall coverage did not decrease.
[X] All unit tests pass on CircleCi.
[X] I ran all tests locally and they pass.

Codecov Report

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

@@            Coverage Diff            @@
##             master      #55   +/-   ##
=========================================
  Coverage     62.32%   62.32%
  Complexity      166      166
=========================================
  Files            26       26
  Lines           937      937
  Branches         64       64
=========================================
  Hits            584      584
  Misses          324      324
  Partials         29       29

Flag
Coverage Δ
Complexity Δ

#karma
95.16% <ø> (ø)
0 <ø> (ø)
:arrow_down:

#unittests
54.19% <ø> (ø)
166 <ø> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


config yml CPU CUDA

when set to - cuda - gives this error
(venv) D:\liveportrait_talker>python inference.py --config_path config.yaml --source_path "D:\liveportrait_talker\Face\Tarkan1080.jpg" --audio_path "D:\liveportrait_talker\Audio\MyMemory.wav" --save_path "D:\liveportrait_talker\Output"
Config File is loaded succesfully!
Traceback (most recent call last):
File "D:\liveportrait_talker\inference.py", line 78, in
main(args)
File "D:\liveportrait_talker\inference.py", line 23, in main
preprocess = Preprocess(device=cfg.device,
File "D:\liveportrait_talker\src\modules\preprocess.py", line 28, in init
self.sd_prep = SadTalkerPreprocess(device=device,
File "D:\liveportrait_talker\src\utils\preprocess\sadtalker_preprocess.py", line 20, in init
self.detector = init_alignment_model('awing_fan', device=device, model_rootpath=model_path)
File "D:\liveportrait_talker\src\utils\preprocess\helper.py", line 147, in init_alignment_model
model.load_state_dict(torch.load(model_path, map_location=device)['state_dict'], strict=True)
File "D:\liveportrait_talker\venv\lib\site-packages\torch\serialization.py", line 1040, in load
return _legacy_load(opened_file, map_location, pickle_module, **pickle_load_args)
File "D:\liveportrait_talker\venv\lib\site-packages\torch\serialization.py", line 1272, in _legacy_load
result = unpickler.load()
File "D:\liveportrait_talker\venv\lib\site-packages\torch\serialization.py", line 1205, in persistent_load
obj = restore_location(obj, location)
File "D:\liveportrait_talker\venv\lib\site-packages\torch\serialization.py", line 1313, in restore_location
return default_restore_location(storage, map_location)
File "D:\liveportrait_talker\venv\lib\site-packages\torch\serialization.py", line 393, in default_restore_location
raise RuntimeError("don't know how to restore data location of "
RuntimeError: don't know how to restore data location of torch.storage.UntypedStorage (tagged with gpu)
when set to - cpu -  this error
(venv) D:\liveportrait_talker>python inference.py --config_path config.yaml --source_path "D:\liveportrait_talker\Face\Tarkan1080.jpg" --audio_path "D:\liveportrait_talker\Audio\MyMemory.wav" --save_path "D:\liveportrait_talker\Output"
Config File is loaded succesfully!
D:\liveportrait_talker\venv\lib\site-packages\torchvision\models_utils.py:208: UserWarning: The parameter 'pretrained' is deprecated since 0.13 and may be removed in the future, please use 'weights' instead.
warnings.warn(
D:\liveportrait_talker\venv\lib\site-packages\torchvision\models_utils.py:223: UserWarning: Arguments other than a weight enum or None for 'weights' are deprecated since 0.13 and may be removed in the future. The current behavior is equivalent to passing weights=None.
warnings.warn(msg)
Downloading: "https://github.com/xinntao/facexlib/releases/download/v0.1.0/detection_Resnet50_Final.pth" to D:\liveportrait_talker\pretrained_models\sadtalker\detection_Resnet50_Final.pth
100%|████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 104M/104M [01:19<00:00, 1.37MB/s]
Traceback (most recent call last):
File "D:\liveportrait_talker\inference.py", line 78, in
main(args)
File "D:\liveportrait_talker\inference.py", line 23, in main
preprocess = Preprocess(device=cfg.device,
File "D:\liveportrait_talker\src\modules\preprocess.py", line 31, in init
self.__load_3dmm_coeff_model(sadtalker_checkpoint_path=sadtalker_checkpoint_path)
File "D:\liveportrait_talker\src\modules\preprocess.py", line 34, in __load_3dmm_coeff_model
checkpoint = safetensors.torch.load_file(sadtalker_checkpoint_path)
File "D:\liveportrait_talker\venv\lib\site-packages\safetensors\torch.py", line 259, in load_file
with safe_open(filename, framework="pt", device=device) as f:

Did you download the pretrained models?

yes, now it gives this error
(venv) D:\liveportrait_talker>python inference.py --config_path config.yaml --source_path "D:\liveportrait_talker\Face\Tarkan1080.jpg" --audio_path "D:\liveportrait_talker\Audio\MyMemory.wav" --save_path "D:\liveportrait_talker\Output"
Config File is loaded succesfully!
D:\liveportrait_talker\venv\lib\site-packages\torchvision\models_utils.py:208: UserWarning: The parameter 'pretrained' is deprecated since 0.13 and may be removed in the future, please use 'weights' instead.
warnings.warn(
D:\liveportrait_talker\venv\lib\site-packages\torchvision\models_utils.py:223: UserWarning: Arguments other than a weight enum or None for 'weights' are deprecated since 0.13 and may be removed in the future. The current behavior is equivalent to passing weights=None.
warnings.warn(msg)
Traceback (most recent call last):
File "D:\liveportrait_talker\inference.py", line 78, in
main(args)
File "D:\liveportrait_talker\inference.py", line 34, in main
map2lp = Map2LivePortrait(device=cfg.device,
File "D:\liveportrait_talker\src\modules\map2lp.py", line 13, in init
self.mapping.load_state_dict(torch.load(mappingnet_model_path))
File "D:\liveportrait_talker\venv\lib\site-packages\torch\serialization.py", line 1025, in load
return _load(opened_zipfile,
File "D:\liveportrait_talker\venv\lib\site-packages\torch\serialization.py", line 1446, in _load
result = unpickler.load()
File "D:\liveportrait_talker\venv\lib\site-packages\torch\serialization.py", line 1416, in persistent_load
typed_storage = load_tensor(dtype, nbytes, key, _maybe_decode_ascii(location))
File "D:\liveportrait_talker\venv\lib\site-packages\torch\serialization.py", line 1390, in load_tensor
wrap_storage=restore_location(storage, location),
File "D:\liveportrait_talker\venv\lib\site-packages\torch\serialization.py", line 390, in default_restore_location
result = fn(storage, location)
File "D:\liveportrait_talker\venv\lib\site-packages\torch\serialization.py", line 307, in mps_deserialize
return obj.mps()
File "D:\liveportrait_talker\venv\lib\site-packages\torch\storage.py", line 144, in mps
return torch.UntypedStorage(self.size(), device="mps").copy(self, False)
RuntimeError: torch.UntypedStorage(): Storage device not recognized: mps
(venv) D:\liveportrait_talker>

I updated requirements.txt. Now ,it won't delete installed PyTorch or install PyTorch. You can follow the instruction from readme file.

ok l ll try

wheel needs to be installed i guess

pip install wheel - then - pip install requirements

Visual Studio c++ build tools - needed it says

i have it ready, will install now

no errors for now

set config yml - cuda

set device in config yml to - cpu

You are right, I updated the code, could you please try again?

Pytorch should have installed numpy. Can you try to install numpy first then run the requirements.txt command.

Pytorch should have installed the numpy. Can you install numpy, then run the requirements.txr command?

yeah i did, pip install numpy, it sayd requirement already met,
lws==1.2.8    this line makes trouble

What is your python version? Also, can you import numpy?

What is your python version? Also, can you import numpy?

which version of python it is needed?, then i will redo venv with correct version

pip install facexlib==0.3.0

pip install kornia==0.7.3

pip install librosa==0.10.1

pip install lws==1.2.8
this is error message

i did it, installation went without any errors

set device to CUDA in config yml

i guess now it is working

ffmpeg needs to be installed

have to delete previous files - extracted images

I assume that there is not tmp folder, I updated code to check before creating new one, you can pull and use it.


[incubator/solr] Support for Kubernetes 1.16.2

Error: unable to build kubernetes objects from release manifest: unable to recognize "": no matches for kind "StatefulSet" in version "apps/v1beta1"
Version of Helm and Kubernetes:
Helm: version.BuildInfo{Version:"v3.0.0", GitCommit:"e29ce2a54e96cd02ccfce88bee4f58bb6e2a28b6", GitTreeState:"clean", GoVersion:"go1.13.4"}
Kubernetes:  1.16.2

bump


Shorthand for ReturnIfAbrupt

Algorithm steps are littered with ReturnIfAbrupt calls. @wycats proposes a shorthand that could be compiled to ECMASpeak without too much difficulty: https://gist.github.com/wycats/1b829d55d1fdb97d8762.

I would rather just change ECMAspeak! Both the input and the output should contain this shorthand.

I agree, although I suspect changes to ECMASpeak will take a while to happen...

Nah. Just write new specs using them. That's how they happen.

Cf. https://streams.spec.whatwg.org/#conventions

Updating ECMASpeak is in progress: https://github.com/tc39/ecma262/pull/129.


UKMO land/sea file for bucket basis function not publicly available

The file "../countries/country-EUROPE-UKMO-landsea-2023.nc" used in the "bucket basis function" isn't distributed with inversions, and I couldn't find it on Blue Pebble.
We should either include this file with inversions, or modify the code so that the land/sea map can be provided as an argument.

Closed by PR #101


[Performance] Any bench marking data or scripts available for ONNXRuntime ?

This can be used for testing perf of ONNXRuntime: https://github.com/Microsoft/onnxruntime/tree/master/onnxruntime/test/perftest
We are also participating in the MLPerf effort.

@srkreddy1238 Does this address your question? If so, please close the issue. If not, let us know how we can further help. Thanks! :)

@faxu thanks for the info. but perftest need a bit of crafting required for the model and input too.
I used below sample code to keep it common across all my comparisons.
But I see ONNX GPU is quite expensive compared to others.
    onnx_file = "./onnx-models/" + name + ".onnx"
    if not os.path.exists(onnx_file):
        onnx_graph = tf2onnx.tfonnx.process_tf_graph(sess.graph, input_names=[input_node], output_names=[output_node])
        model_proto = onnx_graph.make_model("test")
        with open(onnx_file, "wb") as f:
            f.write(model_proto.SerializeToString())

    from onnx import load
    import onnxruntime.backend as backend
    model = load(onnx_file)
    sess = backend.prepare(model, 'GPU')

    X = input_data

    sess.run(X)
    sess.run(X)

    total = 0;
    for ii in range(600):
        start = time.time()
        sess.run(X)
        done = time.time()
        total += (done - start)
    avg =  round(total/600*1000, 4)
    results[name]['ONNX'] = avg

    print("ONNX Avg:", avg)

Hi @srkreddy1238 ,
Could you try this: https://github.com/mlperf/inference/tree/master/cloud/image_classification

Hi @snnn
Sure, I will try this and let you know the results.

Hi @srkreddy1238 , is your problem solved?

@snnn thanks.
I was a bit occupied these days. I will try it this week some time.
I am closing this case for now.


Show which prayer icon to select when Stay Vigilant (Akkha) is active

I am learning stay vigilant it would be nice to know which prayer to select (by showing a prayer icon in the corner during the akkha fight) and maybe a highlighted melee style icon to show which melee setup i should attack with (so if he is in his ranging phase which is where you attack with melee - above him he will have mage and range prayers activated - maybe show a (for example) blue highlighted strength skill icon indicating to attack him with melee, and a (for example) red ranging icon to show to pray with protection from missiles

This is expressly forbidden by Jagex's Third Party Client Guidelines:

Indicates which prayer to use in any combat situation


feat: Add support to configure parameters form insights-client.conf

Enhancement: Make most of the parameters in /etc/insights-client/insights-client.conf configurable/manageable by this role.
Reason: This enables users to configure the insights-client parameters at scale.
Result: Currently only the parameters autoupdate, ansible_host, display_name, remediation, tags, and rhc_proxy are supported by this role. This PR adds support for the following parameters:

loglevel
auto_config
authmethod
username
password
base_url
obfuscate
obfuscate_hostname
redaction_file
content_redaction_file
tags_file

Issue Tracker Tickets (Jira or BZ if any):
https://issues.redhat.com/browse/RHEL-35245
Additional information:

As I'm not a software developer and this is my first PR for this role, please bear with me if I have missed something or did something wrong
I followed the contribution guide to my best judgement
To add the new features I copied, pasted, and modified existing tasks so my contribution should align well with the existing code
Advice on how to improve are appreciated

With commit e2395bc I implemented the configuration to create file-redaction.yaml and file-content-redaction.yaml.
I guess that's it. I wait for the review by @ptoscano. :-)

How can we test this?

Sorry for the late answer on this.
While I appreciate the efforts on this, I have to raise my concerns on this implementation. The general "style" of the rhc role is to work on use cases & functionalities related to the connection of a system to Red Hat / Satellite, and functionalities related to maintaining such. This means for example ensuring a system is connected & registered, it can get content, and there are certain Insights features available. Because of this, the role does not expose configuration bits of the underlying tools simply because they exist, and only in case they are needed, sometimes with different semantics than the underlying tools (e.g. rhc_insights.tags), or even that do not map at all (e.g. rhc_repositories, rhc_insights.remediation).
What I'm missing here are the actual use cases & stories behind the addition of those options. Sometimes configuring an option may be the right solution, and sometimes a problem can be solved in a different way. Unfortunately, without knowing them, it is hard to determine whether adding the options is the correct possibility, and even testing whether the use cases were actually solved. Also, new options mean additional maintenance cost, which thus needs to be "justified".
In addition to that, I find some options problematic to add:

rhc_insights_auth: the authentication was explicitly designed to be the same as used for registering a system, as a separate authentication for Insights is an extreme corner case; in addition to that, basic authentication for Insight is officially deprecated, and will be disabled at the end of this year (2024) [1]
rhc_insights.autoconfig & rhc_insights.baseurl: these two option are used to manually configure where insights-client connects to, without detecting that from the way the system is registered with subscription-manager; this is yet another corner case, mostly to support insights-client registered using basic authentication (and thus not able to automatically detect from subscription-manager)
rhc_insights.logevel: the default behaviour of insights-client is to even log debug messages to its log file (/var/log/insights-client/insights-client.log); it is not clear to me why this needs to be configurable via the rhc role

Regarding the other two "blocks" of options, i.e. redaction and obfuscation: these big features definitely require use cases and discussions on what is expected, what is needed, and so forth, and what would be the proper API to support them.
Because of what I said, my recommendation for this PR would be to hold it, and move the discussion as internal with us, and determine the various use cases, etc.
[1] https://docs.redhat.com/en/documentation/red_hat_insights/1-latest/html/release_notes/april-2024#remote_host_configuration_rhc_and_the_insights_client


Visible history does not respect time correctly

Describe the bug
Not sure what is the exact logic right now (because it somewhat works), but setting a history in seconds yields unexpected results right now.
To Reproduce
import numpy as np
import rerun as rr
rr.spawn()
rr.set_time_seconds("test", 0)
rr.log_points("point", np.random.rand(5,3))
rr.set_time_seconds("test", 10)
rr.log_points("point", np.random.rand(5,3))

Moving to the last message, and setting the history to a number >0 will show all ten points.
Expected behavior
Should only show 5 points, until history >=10.
Screenshots

Note that the data is 10s apart on the timeline, yet it shows both with a history of 0.1s.
Desktop (please complete the following information):

OS: Ubuntu 20.04

The behavior you're observing is "technically correct" but it's a bit nuanced and I see why it's unexpected without better documentation on the rerun temporal model of persistence. Going to leave this open until we can at least make this more clear.
The Rerun model is generally that "All logged components exist until they are replaced."  Although it's not indicated on the timeline as such, you can think of this as meaning that the first set of 5 points is "valid" over the semi-open interval [0, 10) and the second set of 5 points is valid over the semi-open interval [10, Inf).
As such, the visible history query on the interval: [9.9, 10] intersects with both of these validity intervals and subsequently both versions of the points are displayed by the query.

Makes sense to me. I'll close this for now.


Cherry-picks for the 3.5 branch (future 3.5.2) - 1st batch

@kleonc noted in #66063:

Marking for cherry-pick as it's a bugfix but note it's possible that some users use these only in 3D and thus they could have setup things according to the previous buggy behavior. Meaning if someone uses AtlasTextures with vertical margins (so I'd guess not so common case) and has tweaked some settings (like offset etc.) to make it look visually according to their needs then this PR will break things for them (they would need to change their setup). So it should be clearly pointed out in the change log if cherry-picked.

Thanks!


Fix or remove tca_example

The example exists, but was not added to examples_infos. Hence, it is not tested as part of CI.

Fixed with #702.


feat: add oxbridge table and add pre table

给表格添加了 PreTable 的字段，作为自己的前置表，前置表数据可根据 PreTableDetect 获取某个内容进行比较来确定替换规则。
Signed-off-by: Frankzhaopku<EMAIL_ADDRESS>

Codecov Report

Merging #53 into master will decrease coverage by 0.3%.
The diff coverage is 23.52%.

@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
- Coverage    29.8%   29.49%   -0.31%
==========================================
  Files          17       18       +1
  Lines         406      417      +11
  Branches       19       22       +3
==========================================
+ Hits          121      123       +2
- Misses        285      294       +9

Impacted Files
Coverage Δ

app/schema/table.ts
50% <ø> (ø)
:arrow_up:

app/service/github.ts
26.31% <0%> (-1.47%)
:arrow_down:

app/service/gitee.ts
18.18% <0%> (-4.05%)
:arrow_down:

app/schema/table_oxbridge.ts
100% <100%> (ø)

config/config.default.ts
100% <100%> (ø)
:arrow_up:

app/service/shimo.ts
25.28% <22.5%> (+0.28%)
:arrow_up:

app/schema/table_hospital.ts
33.33% <50%> (-6.67%)
:arrow_down:

Continue to review full report at Codecov.

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


Multipledge models

I rule and have completed some work on Case Manager that's ready for review!
One of the remaining confusing things about this app is that we (I) aren't doing a good job meaningfully distinguishing a DCAF pledge from an NNAF pledge from a BRAAF pledge from a BAF pledge. The former two are common scenarios, the latter are less common. To simplify this distinction, this renames the Pledge object to ExternalPledge, and does some other assorted prep for stringing it into the view.
This pull request makes the following changes:

Pledge is ExternalPledge now
Adds ExternalPledgesController#destroy
Edits some of the ExternalPledge attributes to significantly simplify the object model.

TODO:

[x] test new controller destroy method
[ ] test external_pledge default scope
[x] fix external_pledges controller strong params
[x] enforce uniqueness of pledge -- no more than one active pledge from a given source at a time

It relates to the following issue #s:

Affects #746 , #747

Current coverage is 98.53% (diff: 97.56%)

Merging #755 into master will increase coverage by 0.15%

@@             master       #755   diff @@
==========================================
  Files            34         34
  Lines           682        684     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            671        674     +3
+ Misses           11         10     -1
  Partials          0          0

Powered by Codecov. Last update 171e19d...0a84dc7

🎉


libssl-dev : Depends: libssl1.1 - unable to install

❯ uname -a
Linux raspberrypi 5.10.17-v7l+ #1403 SMP Mon Feb 22 11:33:35 GMT 2021 armv7l GNU/Linux

❯ cat /etc/issue
Raspbian GNU/Linux 10 \n \l

❯ dmesg
[    0.000000] Booting Linux on physical CPU 0x0
[    0.000000] Linux version 5.10.17-v7l+ (dom@buildbot) (arm-linux-gnueabihf-gcc-8 (Ubuntu/Linaro 8.4.0-3ubuntu1) 8.4.0, GNU ld (GNU Binutils for Ubuntu) 2.34) #1403 SMP Mon Feb 22 11:33:35 GMT 2021
[    0.000000] CPU: ARMv7 Processor [410fd083] revision 3 (ARMv7), cr=30c5383d
[    0.000000] CPU: div instructions available: patching division code
[    0.000000] CPU: PIPT / VIPT nonaliasing data cache, PIPT instruction cache
[    0.000000] OF: fdt: Machine model: Raspberry Pi 4 Model B Rev 1.4

Description
Whilst trying to install tinypilot, an error is thrown which stops installation.
What's the behavior that you expect?
Installation completes.
What's happening instead?
The following packages have unmet dependencies:
 libssl-dev : Depends: libssl1.1 (= 1.1.1d-0+deb10u5) but 1.1.1d-0+deb10u5+rpt2 is to be installed
E: Unable to correct problems, you have held broken packages.

What are the steps to reproduce this behavior?

Install latest Raspbian on Raspberry PI 4.
Ensure all is up-to-date by running apt-get update and apt-get upgrade -y
Run:

curl \
  --silent \
  --show-error \
  https://raw.githubusercontent.com/mtlynch/tinypilot/master/quick-install | \
    bash - && \
  sudo reboot

Logs
...
...
Reading package lists... Done
+ sudo apt-get install -y git libffi-dev libssl-dev python3-dev python3-venv
Reading package lists... Done
Building dependency tree
Reading state information... Done
git is already the newest version (1:2.20.1-2+deb10u3).
python3-dev is already the newest version (3.7.3-1).
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:

The following packages have unmet dependencies:
 libssl-dev : Depends: libssl1.1 (= 1.1.1d-0+deb10u5) but 1.1.1d-0+deb10u5+rpt2 is to be installed
E: Unable to correct problems, you have held broken packages.

Thanks for reporting this!
Is this reproducible on a clean RasPi OS install? It looks like there may be something wrong with your package manager on that particular system.

E: Unable to correct problems, you have held broken packages.

It's also possible that some of the apt packages changed and the dependencies need to be updated, but if you could try it on a clean system, that would be helpful.

Hi,
It was a fresh install, brand new from latest image - installed about 30 mins before I attempted to install tinypilot. Admittingly, it's not a completely untainted installation, as I normally bring my system up-to-date with my standard user configuration (using ansible) before I do anything else.
I did install tinypilot on the same machine, last week without issue, from again a fresh base installation (I wiped the card this morning to try something out, so rebuilding things again).
I can attempt again with another fresh install, before I do my user ansible setup.

Hi,
Frest install, no issues detected. I suppose something that I install prior gets in the way. Happy to close this issue as not a fault of tinypilot. :-)

Thanks for letting me know!


Create mechanism which means we don't return NaN on live site when ONS data is delayed

Likely to be picked up as part of #123

@gilesdring I know you've been working on this, and just wanted to mention that us showing a figure on the dashboard landing page, but a N/A on the NEET page itself creates some cognitive dissonance.
See below for pics for LTU
Dashboard landing page

NEET page

Let's discuss and work out if we've resolved it in sprint 8


Navbar login link populates "next"

Now matches the behavior of JS. When pressing "login" on a challenge page, you will be redirected back after logging in!
Closes #102

Great, thanks!