Calculating disposable diapers from kits

I have performed a self-review of my own code,
I have commented my code, particularly in hard-to-understand areas,
I have made corresponding changes to the documentation,
I have added tests that prove my fix is effective or that my feature works,
New and existing unit tests pass locally with my changes ("bundle exec rake")

-->
Resolves #3989
Description
I made modifications to the acquisition report service and the children served report service to take disposable diapers distributed in kits in the annual report. I have added/ modified tests to reflect the changes to the calculations.
Type of change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)

How Has This Been Tested?
Tested in the  spec/services/reports/children_served_report_service_spec.rb and spec/services/reports/acquisition_report_service_spec.rb

Hey @jadekstewart3 -- Just documenting the suggestion I passed on in slack to call the non-kit diapers loose diapers,  to increase the readability of the whole thing.  Maybe use "all" to designate when we are talking about both?

@cielf I think this is ready for review. I know there is a failing check, but the file in question is passing locally. I'm not 100% sure why its not passing completely

@jadekstewart3 We do have some flakey tests. I'm giving @dorner a mention here, as he has been doing some work in that area.

Hey @jadekstewart3 I'm running to the end of my work on this for today -- but is this back in a "ready for review" state?

@cielf yes it is, thank you!

Hrm.  I just did a light manual test on this as follows:
1/  create a distribution with 50 of 2 different items in it that are classed as disposable diapers,  dated last year
2/ choose the 2023 annual report and recalculate it
3/ confirm that there are 100 disposable diapers distributed.
4/  Create a kit that has 50 of each of those 2 items in it.  Allocate 3 kits.
5/ Create a distribution of 2 of those kits, dated last year
6/ choose the 2023 annual report and recalculate it
7/  I expect there to be 300 disposable diapers distributed.  There are 100.

@jadekstewart3 I'll try to kick the tires on this tomorrow -- the suggested sql looks like it  should work to me!   Rubocop is complaining, though -- if you have the cycles to address that in the meantime, that'd be great.

@cielf Hi! I did have to fiddle with the query a bit, but it is returning an integer (YAY!) 😀 It is calculating non disposable items as well, I am having a bit of a time figuring out why. Another set of eyes would be amazing! I pushed up my additional test and the almost working method 👍

@cielf Hi! I did have to fiddle with the query a bit, but it is returning an integer (YAY!) 😀 It is calculating non disposable items as well, I am having a bit of a time figuring out why. Another set of eyes would be amazing! I pushed up my additional test and the almost working method 👍

Is it giving all non disposable items, or just the cloth?

Hrm.  I put together a distribution that had a disposable item and a kit with disposable items for testing,  and the report only counted the disposable items -- not the ones in the kit.     Looking at it in more detail...

@cielf with the updated test this seems to be working! YAY! My test set up was incorrect it seems!

@dorner  -- brakeman thinks there is a SQL injection issue with the query -- can you speak to that?
I'll try to kick the tires on this today or tomorrow.

I think that might be because #{year} is in the query instead of using $1 like the other one.

(Pokes into the code  a bit)  Looks like you might only be using the magic SQL for the children served  service, and not for the aquisition_report_service.   That would explain it.

@cielf I am trying to implement the magic sql in the acquisition report and something is funky. Can you take a look? when I test the distributed_disposable_diapers_from_kits it seems to return 100, but in the full report it is returning 1500?

@jadekstewart3 there seems to be some flakiness here - the order of tests seems to matter. If you run it with --seed 43773, both tests fail with 1500. (In other words, if returns the correct quantity of disposable diapers from kits runs first, it fails, but if it runs after the other test, it passes.)

I think this has to do with it:
      disposable_item = organization.items.disposable.first
      non_disposable_item = organization.items.where.not(id: organization.items.disposable).first

Rather than creating data from scratch, you're relying on items that were created by the factory/seed. To be sure that only the data you care about is being created, you probably should clear out all the items and only create the ones you want to see in the report.

@dorner I am getting a return value of 1500 for the sql query that is calculating distributed disposable diapers from kits. I've gone over my test set up a few times and created the kits and kit items in a similar fashion to the set up in the children_served_report_service_spec, but I cant quite figure out why we are returning 1500 instead of the expected 100.

Hey @jadekstewart3 -- did you look into adding in a clause to in your sql to make sure that the itemizable type for the kit line items is kit?

@cielf I was messing with it yesterday but still having troubles, but I  think I just got it!

@cielf I think this is working! It looks to be failing on a test file that doesnt have anything to do with the reports! 😃

It passes my functional test (adding a mixed diaper and kit-with-diapers distribution) with flying colours!  Hurrah!

Hey @dorner -- Just checking if you have any last comments on this -- you've chimed in, but more on a problem solving than critiquing the code pov.   Thanks!

I should probably take this out for one last spin before we merge it.

Holding off on the merge until the release of April 7 is done.


RN Phone validation doesn't match design

Description
-Fix styles Phone input on the web.

About: #2553

The css was indeed being applied globally affecting the phone input in 'Edit Profile'.
Also found that on native, for this input, error styles (red text & border) were not being applied and spacing was off.
Fixed both issues
Native:

Web:


Support RuboCop v0.85.0 and later

I changed to use Rubocop::Cop::Team#mobilize instead of Rubocop::Cop::Team.new if RuboCop version is 0.85.0 and later because RuboCop changed Team API at v0.85.0.
ref. https://github.com/rubocop-hq/rubocop/pull/8030

Closes #50

Oh man does someone need to merge this in!


fix flaky test cache addresses

Signed-off-by: yeya24<EMAIL_ADDRESS>

[ ] I added CHANGELOG entry for this change.
[X] Change is not relevant to the end user.

Changes
Fix flaky test in #1920

Verification

@simonpasquier  Seems the alerts e2e test failed. Could you please take a look?

Fix for CI: https://github.com/thanos-io/thanos/pull/1926


make a new release

we haven't released the module in a while :(

As stash was renamed to BitBucket, should the module name change yet?

As stash was renamed to BitBucket, should the module name change yet?

As the module attempts to do upgrades from Stash to Bitbucket. I would say no.

we should however, extend our tags

If ready - why not release now? Count my vote for a release... :+1:

you could create a release pr: https://voxpupuli.org/docs/#releasing-a-new-version-of-a-module

I second that vote for a release.
Thanks
On Jun 27, 2016 9:53 AM, "Tobias Brunner"<EMAIL_ADDRESS>wrote:

If ready - why not release now? Count my vote for a release... 👍
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/voxpupuli/puppet-stash/issues/121#issuecomment-228688787,
or mute the thread
https://github.com/notifications/unsubscribe/AA3B_EnacZqQ23FJrjsgFzNnxPgffoDdks5qP4-fgaJpZM4Ij-MQ
.

There have been release since this was opened.


PostgreSQL Standard User

Hello,
I struggle a little bit regarding the user management for Miniflux2.
I created the data base with a superuser as owner. I don't want to use the superuser for the access to the database. So i created the "standard" user with:
createuser -P standard_user
If i try to open Miniflux2, i get the error:
[DEBUG] [Middleware:UserSession] Session not found
[ERROR] [Middleware:Session] unable to fetch session: pq: relation "sessions" does not exist
[DEBUG] [Middleware:Session] Session not found
[ERROR] [Middleware:Session] unable to create session: pq: relation "sessions" does not exist
Only with the existing superuser Miniflux2 runs, but this is not what i want.
Is there a special approach necessary, or which privilegs should have a database user?

Ok,
I found a solution, for everyone who have the same problem.
A standard user is not working. So you need a PostgreSQL user with higher privileges. My try and error approach lead me to a new Superuser. It will be created with:
createuser -P -s miniflux_user
This user should be used also as database owner. Means you should create the new superuser, before you create the database.
The approach to create the Miniflux database and to start the creation of the tables with the existing superuser, combined with a standard database user for the access on database level brings only errors.


[Trigger] Rename csharpscript and align host.json

Port over changes from main and move over trigger stuff. This should fix the issue with the trigger tests failing for CSX as well due to the log level not being at a high enough level.

Merging this in since the one in main was already approved and I want to kick off some runs with my other changes over the weekend...


Custom mjml components - my-image cannot be used inside mj-column, only inside: mj-attributes

Hello, I am trying to build custom components (copy of mj-image and mj-button with some properties added).
Here is a complete working sample: https://github.com/tkafka/mjml-custom-components-sample
The only problem is that I am getting this warning line for every custom component - see https://github.com/tkafka/mjml-custom-components-sample/blob/master/compiled-template.html

Line 10 of /Users/kafkat/Desktop/Dev/#LMC/mjml-components-sample/sample-mjml-template/sample-mjml-template/index.mjml (lmc-image) — lmc-image cannot be used inside mj-column, only inside: mj-attributes
Line 21 of /Users/kafkat/Desktop/Dev/#LMC/mjml-components-sample/sample-mjml-template/sample-mjml-template/index.mjml (lmc-button) — lmc-button cannot be used inside mj-column, only inside: mj-attributes

How can I tell where my custom components can be used? As a code is direct copy of mj-image and mj-button, why does MJML know where mj-image can be used, but doesn't know anything about my component? I noticed there is nothing like 'allowed parent tags' in mj-image.
The components are specified in https://github.com/tkafka/mjml-custom-components-sample/blob/master/.mjmlconfig - is there anything I should add there to tell MJML where they belong?

Aha! Found the components v4 article and added dependencies:
See https://github.com/tkafka/mjml-custom-components-sample/blob/master/mjml-components/lmc-button.js
import { registerDependencies } from 'mjml-validator'

registerDependencies({
  'mj-column': ['lmc-button'],
  'lmc-button': []
});


change editor font?

how can i set the editor default font for all languages?

Hello
File->Settings/Preferences->Editor->Color&Fonts ->Font (you need save scheme as copy - due default schemes are read only)


Duplicate / redundant public key files in sub-stores

Summary
gopass stores redundant public key files in a sub-store .public-keys when
those files can be found in their parent store's .public-keys folder.
These files take about 5K each, this becomes a concern when using
sub-stores to store secrets for multiple devices accessible by multiple
gpg users. In fact, the public keys ends up taking as much space as the
secrets themselves.

Complete example at: nixos-sf-demo-device-secrets.
Duplicate public keys sample:

.public-keys/0x15188BCAC872FFAF
The parent key.

device/demo-virtual-box-vm/.public-keys/0x15188BCAC872FFAF
A first duplicate.

device/another-demo-virtual-box-vm/.public-keys/0x15188BCAC872FFAF
Another duplicate.

Steps to reproduce
I have the following minimal-script-to-reproduce.sh:
#!/usr/bin/env nix-shell
#!nix-shell -p gopass -p gnupg -i bash
printf "Removing whole gopass home (root store + config) and substores.\n"
yes | rm -fr "$PWD/.my-gopass-home" "$PWD/nixos-sf-demo-device-secrets"

export GOPASS_HOMEDIR="$PWD/.my-gopass-home"
export GOPASS_CONFIG="$GOPASS_HOMEDIR/.config/gopass/gopass.yaml"

export GNUPGHOME="$PWD/.gnupg"

printf "\nInit root store:\n"
gopass --yes init<EMAIL_ADDRESS>
printf "\nMounting 1st level substore 'nixos-sf-demo-device-secrets':\n"
gopass mounts add -i<EMAIL_ADDRESS>'nixos-sf-demo-device-secrets' "gpgcli-gitcli-fs+file://$PWD/nixos-sf-demo-device-secrets"

printf "\nMounting 2nd level substore 'nixos-sf-demo-device-secrets/device/demo-virtual-box-vm':\n"
gopass mounts add -i<EMAIL_ADDRESS>'nixos-sf-demo-device-secrets/device/demo-virtual-box-vm' "gpgcli-gitcli-fs+file://$PWD/nixos-sf-demo-device-secrets/device/demo-virtual-box-vm"

printf "\nMounting another 2nd level substore 'nixos-sf-demo-device-secrets/device/another-demo-virtual-box-vm':\n"
gopass mounts add -i<EMAIL_ADDRESS>'nixos-sf-demo-device-secrets/device/another-demo-virtual-box-vm' "gpgcli-gitcli-fs+file://$PWD/nixos-sf-demo-device-secrets/device/another-demo-virtual-box-vm"

printf "\nPrinting recipients list:\n"
gopass recipients

which when I execute yields:
$ ./minimal-script-to-reproduce.sh

Removing whole gopass home (root store + config) and substores.

Init root store:
[init] Initializing a new password store ...
[init] Initializing git repository (gitcli) ...
[init] git initialized at /home/rgauthier/dev/my-nixos-secure-factory/.my-gopass-home/.password-store
[init] git configured at /home/rgauthier/dev/my-nixos-secure-factory/.my-gopass-home/.password-store
[init] Git initialized
[init] Password store /home/rgauthier/dev/my-nixos-secure-factory/.my-gopass-home/.password-store initialized for:
[init]   0x15188BCAC872FFAF - SecureFactory Nixos<EMAIL_ADDRESS>
Mounting 1st level substore 'nixos-sf-demo-device-secrets':
Password store gpgcli-gitcli-fs+file:///home/rgauthier/dev/my-nixos-secure-factory/nixos-sf-demo-device-secrets initialized for:
8CFE2147789A4566A9869DA615188BCAC872FFAF
Mounted nixos-sf-demo-device-secrets as gpgcli-gitcli-fs+file:///home/rgauthier/dev/my-nixos-secure-factory/nixos-sf-demo-device-secrets

Mounting 2nd level substore 'nixos-sf-demo-device-secrets/device/demo-virtual-box-vm':
Password store gpgcli-gitcli-fs+file:///home/rgauthier/dev/my-nixos-secure-factory/nixos-sf-demo-device-secrets/device/demo-virtual-box-vm initialized for:
8CFE2147789A4566A9869DA615188BCAC872FFAF
Mounted nixos-sf-demo-device-secrets/device/demo-virtual-box-vm as gpgcli-gitcli-fs+file:///home/rgauthier/dev/my-nixos-secure-factory/nixos-sf-demo-device-secrets/device/demo-virtual-box-vm

Mounting another 2nd level substore 'nixos-sf-demo-device-secrets/device/another-demo-virtual-box-vm':
Password store gpgcli-gitcli-fs+file:///home/rgauthier/dev/my-nixos-secure-factory/nixos-sf-demo-device-secrets/device/another-demo-virtual-box-vm initialized for:
8CFE2147789A4566A9869DA615188BCAC872FFAF
Mounted nixos-sf-demo-device-secrets/device/another-demo-virtual-box-vm as gpgcli-gitcli-fs+file:///home/rgauthier/dev/my-nixos-secure-factory/nixos-sf-demo-device-secrets/device/another-demo-virtual-box-vm

Printing recipients list:
Hint: run 'gopass sync' to import any missing public keys
gopass
├── nixos-sf-demo-device-secrets (/home/rgauthier/dev/my-nixos-secure-factory/nixos-sf-demo-device-secrets)
│   ├── device
│   │   ├── another-demo-virtual-box-vm (/home/rgauthier/dev/my-nixos-secure-factory/nixos-sf-demo-device-secrets/device/another-demo-virtual-box-vm)
│   │   │   └── 0x15188BCAC872FFAF - SecureFactory Nixos<EMAIL_ADDRESS>│   │   └── demo-virtual-box-vm (/home/rgauthier/dev/my-nixos-secure-factory/nixos-sf-demo-device-secrets/device/demo-virtual-box-vm)
│   │       └── 0x15188BCAC872FFAF - SecureFactory Nixos<EMAIL_ADDRESS>│   └── 0x15188BCAC872FFAF - SecureFactory Nixos<EMAIL_ADDRESS>└── 0x15188BCAC872FFAF - SecureFactory Nixos<EMAIL_ADDRESS>
Now, when I look at the generated files:
$ tree -s -h -a nixos-sf-demo-device-secrets
nixos-sf-demo-device-secrets
├── [4.0K]  device
│   ├── [4.0K]  another-demo-virtual-box-vm
│   │   ├── [  19]  .gpg-id
│   │   └── [4.0K]  .public-keys
│   │       └── [4.6K]  0x15188BCAC872FFAF
│   └── [4.0K]  demo-virtual-box-vm
│       ├── [  19]  .gpg-id
│       └── [4.0K]  .public-keys
│           └── [4.6K]  0x15188BCAC872FFAF
├── [  19]  .gpg-id
└── [4.0K]  .public-keys
    └── [4.6K]  0x15188BCAC872FFAF

6 directories, 6 files

I can see there are 3 identical copies of the 0x15188BCAC872FFAF public key of about 5K each.
That might seems fine for this simple case, but now as I expect to store secrets for many devices (i.e: many sub-stores) and authorize multiple users to each sub-stores, this redundancy becomes quite annoying. Here's the actual listing of files taken from nixos-sf-demo-device-secrets which is closer to our goal:
$ tree -I '.git' -s -h -a nixos-sf-demo-device-secrets
nixos-sf-demo-device-secrets
├── [4.0K]  device
│   ├── [4.0K]  another-demo-virtual-box-vm
│   │   ├── [4.0K]  etc
│   │   │   └── [4.0K]  ssh
│   │   │       ├── [2.2K]  ssh_host_ed25519_key.b64.gpg
│   │   │       ├── [1.8K]  ssh_host_ed25519_key.pub.b64.gpg
│   │   │       ├── [6.1K]  ssh_host_rsa_key.b64.gpg
│   │   │       └── [2.6K]  ssh_host_rsa_key.pub.b64.gpg
│   │   ├── [ 123]  .gpg-id
│   │   ├── [4.0K]  .public-keys
│   │   │   ├── [4.6K]  0x15188BCAC872FFAF
│   │   │   ├── [4.6K]  8CFE2147789A4566A9869DA615188BCAC872FFAF
│   │   │   ├── [4.6K]  BBC8F046A7E47B2C9D325D30EAFCBBEA44A1FADC
│   │   │   └── [4.6K]  CC6A025C9620195AF81141AD0C3EB41A4A4E9D8A
│   │   └── [4.0K]  root
│   │       ├── [4.0K]  _gnupg
│   │       │   ├── [1.8K]  gpg-otrust.b64.gpg
│   │       │   └── [8.6K]  subkeys.gpg-keys.b64.gpg
│   │       └── [4.0K]  _ssh
│   │           ├── [6.1K]  id_rsa.b64.gpg
│   │           └── [2.6K]  id_rsa.pub.b64.gpg
│   └── [4.0K]  demo-virtual-box-vm
│       ├── [4.0K]  etc
│       │   └── [4.0K]  ssh
│       │       ├── [2.2K]  ssh_host_ed25519_key.b64.gpg
│       │       ├── [1.8K]  ssh_host_ed25519_key.pub.b64.gpg
│       │       ├── [6.0K]  ssh_host_rsa_key.b64.gpg
│       │       └── [2.6K]  ssh_host_rsa_key.pub.b64.gpg
│       ├── [ 123]  .gpg-id
│       ├── [4.0K]  .public-keys
│       │   ├── [4.6K]  08E65A21FA20E76B336E2732A2D39F41CCBA5A84
│       │   ├── [4.6K]  0x15188BCAC872FFAF
│       │   ├── [4.6K]  3177342A2FF4AC78D986EA01383296D774073EE9
│       │   └── [4.6K]  8CFE2147789A4566A9869DA615188BCAC872FFAF
│       └── [4.0K]  root
│           ├── [4.0K]  _gnupg
│           │   ├── [1.8K]  gpg-otrust.b64.gpg
│           │   └── [8.6K]  subkeys.gpg-keys.b64.gpg
│           └── [4.0K]  _ssh
│               ├── [6.0K]  id_rsa.b64.gpg
│               └── [2.6K]  id_rsa.pub.b64.gpg
├── [  19]  .gpg-id
├── [ 11K]  LICENSE
└── [4.0K]  .public-keys
    └── [4.6K]  0x15188BCAC872FFAF

16 directories, 29 files

As you can see, the overall size of the public keys in a sub-store take about
the same size as the encrypted secrets themselves. And, in this example, only 2
gpg users are allowed to device sub-stores (the device itself and another user).
Expected behaviour
Would it be possible to prevent such duplication for similar cases either by a
configuration option, a flag or even by default when detected?
Some ideas:

Not storing the public key in the sub-store at all and assuming
the sub-store inherits from the store's public keys.

Using symlinks for sub-store keys already in parent store.
nixos-sf-demo-device-secrets/device/demo-virtual-box-vm/.public-keys/0x15188BCAC872FFAF
-> nixos-sf-demo-device-secrets/.public-keys/0x15188BCAC872FFAF

Thanks for the great tool by the way.
Environment
$ uname -a
Linux rgauthier 4.14.154 #1-NixOS SMP Tue Nov 12 18:19:08 UTC 2019 x86_64 GNU/Linux

$ gopass --version
gopass 1.8.6-git+HEAD go1.12.9 linux amd64

$ gpg --version
gpg (GnuPG) 2.2.17
libgcrypt 1.8.5
Copyright (C) 2019 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <https://gnu.org/licenses/gpl.html>
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Home: /home/rgauthier/.gnupg
Supported algorithms:
Pubkey: RSA, ELG, DSA, ECDH, ECDSA, EDDSA
Cipher: IDEA, 3DES, CAST5, BLOWFISH, AES, AES192, AES256, TWOFISH,
        CAMELLIA128, CAMELLIA192, CAMELLIA256
Hash: SHA1, RIPEMD160, SHA256, SHA384, SHA512, SHA224
Compression: Uncompressed, ZIP, ZLIB, BZIP2

Thank you very much for the execellent bug report. This makes it very easy to reproduce and understand your use case.
The gopass substore support was designed to support a small number of mounted stores with a usually small number of recipients each. We did not anticipate using a very large number of mounted stores with a lot of recipients.
Is this a real issue, e.g. due to limited resources, or more a theoretical concern?
If this is a real problem the best I could offer would be a config option to disable exporting these keys at all (per mount).
Inheriting or linking keys across mounts doesn't seem to be feasible to me, some of my use cases rely on the fact of substores being completely independent.

If this is a real problem the best I could offer would be a config option to disable exporting these keys at all (per mount).

This would be an helpful option. I have a mounted store where the public keys are in an other folder than ".public-keys" and also have other names. I would just like to be able to disable key syncing for this mount.

@dominikschulz :

Is this a real issue, e.g. due to limited resources, or more a theoretical concern?

Theoretical for the time being, however we were afraid it might eventually come to become a real issue as devices were added.

If this is a real problem the best I could offer would be a config option to disable exporting these keys at all (per mount).

This seems like a reasonable option to solve our problem.

Inheriting or linking keys across mounts doesn't seem to be feasible to me, some of my use cases rely on the fact of substores being completely independent.

Indeed such an inheritance scheme might get tricky, much more so across repositories. I understand why you hesitate to add this kind of complexity.
Thanks for the fix.


Move conditionally defined ndb helpers into their own module.

Towards #433.

@nathanielmanistaatgoogle PTAL, I addressed the comments.

You addressed it just fine; a counterargument is a perfectly valid form of addressing a comment. :-)
Looks great otherwise (if you really want to keep the as client_mod, well it's in an internal place so that's fine). Squash, retest, and merge. :-)

Did the rename. Will squash and wait until the sun goes down for Travis


[REVIEW] krshkodes

Twitter profile link
https://twitter.com/krshkun (Please check pinned tweet)

Reviewed on live stream https://www.youtube.com/watch?v=9pXcE3-WnkA


Just Clone and Run gulp build return Error.

Hi.
I just read ur Agile Web Development with Angular.js.
So. I want to test example.
here is my Run env (work on Windows7 64bit)

here is error logs.
$ gulp build
[18:19:15] Using gulpfile D:\96.Dev\Sublime_prj\angularapp-gulp-boilerplate\Gulpfile.js
[18:19:15] Starting 'templates'...
[18:19:15] Finished 'templates' after 10 ms
[18:19:15] Starting 'compress'...
[18:19:16] Finished 'compress' after 160 ms
[18:19:16] Starting 'copy'...
[18:19:16] Finished 'copy' after 1.86 ms
[18:19:16] Starting 'uncss'...
[18:19:16] Finished 'uncss' after 1.08 ms
[18:19:16] Starting 'build'...
[18:19:16] Finished 'build' after 3.32 μs
events.js:87
      throw Error('Uncaught, unspecified "error" event.');
            ^
Error: Uncaught, unspecified "error" event.
    at Error (native)
    at Transform.emit (events.js:87:13)
    at Transform.onerror (D:\96.Dev\Sublime_prj\angularapp-gulp-boilerplate\node_modules\gulp\node_modules\vinyl-fs\node_modules\through2\node_modules\readable-stream\lib\_stream_readable.js:558:12)
    at Transform.emit (events.js:107:17)
    at Transform.<anonymous> (D:\96.Dev\Sublime_prj\angularapp-gulp-boilerplate\node_modules\gulp-useref\index.js:79:42)
    at Array.forEach (native)
    at Transform.<anonymous> (D:\96.Dev\Sublime_prj\angularapp-gulp-boilerplate\node_modules\gulp-useref\index.js:67:35)
    at Array.forEach (native)
    at Transform.<anonymous> (D:\96.Dev\Sublime_prj\angularapp-gulp-boilerplate\node_modules\gulp-useref\index.js:45:36)
    at Array.forEach (native)

I don't have any changes.
How can I fix it?

Same error here.


[7.x] Adds view testing in isolation

This pull request adds the ability of testing views in isolation without the need of using the http layer - and it looks like this:
class WelcomeTest extends TestCase
{
    /**
     * Asserts welcome view.
     *
     * @return void
     */
    public function testWelcomeView()
    {
        $contents = $this->view('welcome');

        $contents->assertSee('Laravel');
    }
}

Of course, just like the view helper, you can pass arguments:
class ButtonTest extends TestCase
{
    /**
     * Tests the button partial
     *
     * @return void
     */
    public function testButton()
    {
        $contents = $this->view('partials.button', [
             'name' => 'Send',
        ]);

        $contents->assertSee('Send');
    }
}

Note: This pull request don't contain tests yet - I will add them if people want this on the framework.

Hmm, I'm not sure I've ever had a use case for this. It seems like you would just test the routes that returns the response.

@taylorotwell You may want to reconsider this buddy - imagine that you have a button partial that accepts multiple arguments. If you want to test in isolation that button behaves correctly with those 3 arguments, and that may be difficult doing it using the http layer.
$this->view('partials.button')->assertSeeLink('to/path');
$this->view('partials.button', ['logged' => false)->assertDontSeeLink('to/path');
$this->view('partials.button', ['link' => 'blabla')->assertDontSeeLink('blabla');

If you still don't see the value on it - its fine, I will do a package with it. 👍

Couldn't you just render the view and do assertions on it?
$this->assertContains('blah', view('partials.button')->render());

May not even need the render function call due to the magic __toString on view.

@garygreen That closes the door for the future. Because ideally we could move things a little further with helpers:
assertSeeLink, assertSeeButton, assertSeeTitle, etc.

Hmm that would be useful, though I would prefer to see those assertions being able to be used in isolation with a generic rendered HTML string.
$someRenderedHtml = new HtmlTest(view('some-view'));
$someRenderedHtml->assertSeeTitle('blah');
$someRenderedHtml->assertSeeLink('http://blah');

And the TestView could be reserved for testing things specific to the view e.g.
$testView = new TestView(view('some-view'));
$testView->assertContainsData(...);
$testView->assertHasSection('title', 'blah');

If you get what I mean? I haven't done much with unit testing for Laravel assertions so not sure if this is already possible.

I think this is very useful.
@taylorotwell here is my usecase - an actual case that a project of mine will employ for a refactoring in the next few months:
We build a PDF out of partial views to be rendered from html using something like DomPDF.
At the moment it's a complete mess for historical reasons and small changes tend to break things.
In this case I would like to be able to test most of the partial views without rendering the complete html or the PDF.

For future reference: https://github.com/nunomaduro/laravel-mojito. I have made package with this idea.

Thanks. Also added raw Blade templating rendering + a few more assertions.


fix: use regexTarget for extend config

Description:
Extend config is super useful, but if you have regexTarget = "line" within [rules.allowlist], extend does not work.
The basic functionality works thanks to https://github.com/gitleaks/gitleaks/pull/1524 but regexTarget was missing.
Beside of this the adding keywords from base and extended rule into global keywords list is required to extend rules.
In addition some documentation to extend a rule has been added.
This should also solve:

https://github.com/gitleaks/gitleaks/issues/999
https://github.com/gitleaks/gitleaks/issues/1027
https://github.com/gitleaks/gitleaks/issues/1384

Checklist:

[x] Does your PR pass tests?
[x] Have you written new tests for your changes?
[x] Have you lint your code locally prior to submission?

Makes sense to me. :+1:
Do you think it'd be beneficial to have clear options for "update this inherited rule" versus "completely replace this inherited rule"? The logic for [extend] isn't fully fleshed out and likely a common source of confusion.

Thanks for the catch. This LGTM 👍🏻. Great work @bufferoverflow


Update ReadPdf.py

Remove duplicated function
Remove the get_pdf_files function() duplications

Related Issue

Description
you have two function called get_pdf_files one is with try and except and the other without so i cleaned the one without try and except
Type

[ ] Bug Fix
[ ] Feature Enhancement
[ ] Documentation Update
[x] Code Refactoring
[ ] Other (please specify):

Proposed Changes

Screenshots / Code Snippets (if applicable)

How to Test

Checklist

[x ] The code compiles successfully without any errors or warnings
[x] The changes have been tested and verified
[x] The documentation has been updated (if applicable)
[x] The changes follow the project's coding guidelines and best practices
[x] The commit messages are descriptive and follow the project's guidelines
[x] All tests (if applicable) pass successfully
[ ] This pull request has been linked to the related issue (if applicable)

Additional Information

Thanks @abdelkareemkobo I will take a look at these changes soon.


[Bug]: Citrix Workspace App not being updated

The problem
Hi,
this issue is not really related to Winet-AutoUpdate, it's more a winget/Citrix issue in my opinion...
Let me explain the problem.
When having devices with older Citrix Workspaces installed, there Id of the package is called "CitrixOnlinePluginPackWeb".
Name				Id		                	Version            Available     Source

----------------------------------------------------------------------
Citrix Workspace 2010		CitrixOnlinePluginPackWeb		<IP_ADDRESS>

The newer versions of Citrix Workspace in the winget source is called "Citrix.Workspace"
when doing a "winget upgrade" the result is that there is no upgrade for Citrix available.
Doing instead an explicit installation of Citrix Workspace "winget install --id Citrix.workspace --silent" it will do a correct upgrade and even keep all the Citrix store conmfiguration.
Citrix can be a real pain in the ass...
do you think you might be able to handle this in the upgrade script as a special type of app and do a lookup for "CitrixOnlinePluginPackWeb" and in case if found update it via the above command or is this out of scope?
What version of WAU has the issue?
1.17.8
What version of Windows are you using (ex. Windows 11 22H2)?
Windows 11 22H2
What version of winget are you using?
v1.6.2771
Log information
No response
Additional information
No response

That ID is internal on your client, it doesn't exist in any source.
You could try from the _AppID-template.ps1 in mods  create a CitrixOnlinePluginPackWeb-preinstall.ps1 (or create your own as that template is an example with deps to _Mods-Functions.ps1) and in that do what you want.
In that template I've provided an:
#Install App from Winget Repo, multiple: "appID1","appID2". Example:
#$WingetIDInst = @("Microsoft.PowerToys")
$WingetIDInst = @("")

And, regarding Citrix.Workspace:
https://github.com/Romanitho/Winget-Install#use-winget-native-parameters

...or, if that doesn't cut it (the ID not existing in any source) you could write your own global solution if that app is installed via:
https://github.com/Romanitho/Winget-AutoUpdate#custom-script-mods-for-wau

Totally agree.
We cannot customize WAU with all specific cases. It would become a mess.
Mods was created for that. You can adapt WAU regarding your context :)

OK thanks for the feedback. I agree, if we can have mods for that there should be no change to WAU!


[SIL] Make sure @objc thunks for 'mutableCopy' are treated as +1

A nice cleanup (part of #14825) turned out to have a bug in it. Revert that and add tests to keep it from happening again.
rdar://problem/40373771

@swift-ci Please smoke test

@stephan-tolksdorf: can't believe neither you nor I caught this at the time. :-( At least we found it!

@jrose-apple Yeah, I should have noticed that! Sorry! :-(


SslStream/NegotiateStream breaking change

Fixes #20865.
Preview link.

It seems like the Remarks sections for each of these overloads may need to be updated. Can you comment on that @wfurt? E.g. for SslStream.BeginAuthenticateAsServer.

Is there specific part of the remarks @gewarren? They generally seem good to me. The Begin* operation does not block and End* must be called to complete the operation.
Calling another Begin before calling End was never allowed and that logic was enforced in the Begin/End functions. With the 5.0 change the Begin/End is implemented on top of underlying Task. So it is possible to call Begin even before End is finished as ling as the underlying Task is done. I think there is marginal window of opportunity when something would fail in the past but may work in 5.0

Is there specific part of the remarks @gewarren? They generally seem good to me. The Begin* operation does not block and End* must be called to complete the operation.
Calling another Begin before calling End was never allowed and that logic was enforced in the Begin/End functions. With the 5.0 change the Begin/End is implemented on top of underlying Task. So it is possible to call Begin even before End is finished as ling as the underlying Task is done. I think there is marginal window of opportunity when something would fail in the past but may work in 5.0

I just thought since the implementation was no longer using APM that the wording may need to be changed, but it sounds like the answer is "no". Thanks for checking!


add welcome messages

I rule and have completed some work on Case Manager that's ready for review!
(brief, plain english overview of your changes here)
This pull request makes the following changes:

(your change here)
(another change here)
(etc)

(If there are changes to the views, please include a screenshot so we know what to look for!)
It relates to the following issue #s:

Fixes #X
Bumps #Y

Codecov Report

Merging #938 into master will decrease coverage by -10.9%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #938      +/-   ##
==========================================
- Coverage    98.1%   87.21%   -10.9%
==========================================
  Files          40       37       -3
  Lines         846      829      -17
==========================================
- Hits          830      723     -107
- Misses         16      106      +90

Impacted Files
Coverage Δ

app/controllers/application_controller.rb
5.26% <0%> (-94.74%)
:x:

app/helpers/last_menstrual_period_helper.rb
36.36% <0%> (-63.64%)
:x:

app/helpers/pledges_helper.rb
42.85% <0%> (-57.15%)
:x:

app/controllers/clinics_controller.rb
50% <0%> (-50%)
:x:

app/controllers/users_controller.rb
51.42% <0%> (-48.58%)
:x:

app/controllers/patients_controller.rb
68.29% <0%> (-31.71%)
:x:

app/helpers/dashboards_helper.rb
62.96% <0%> (-25.93%)
:x:

app/helpers/patients_helper.rb
90.47% <0%> (-9.53%)
:x:

lib/reporting/patient.rb
90.9% <0%> (-9.1%)
:x:

app/models/patient.rb
93.2% <0%> (-3.09%)
:x:

... and 3 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 7a50839...6effe40. Read the comment docs.


Different way to organize reducers, is it right?

const mapActionToReducer = {
    [actions.SET_USERNAME]: (state, action) => {
        console.log('set username action passed to reducer:', action);
        return Object.assign({}, state, {user: Object.assign({}, state.user, {name: action.value})});
    }
}

const rootReducer = (state = initialState, action) => {
    if (action.type in mapActionToReducer) {
         return mapActionToReducer[action.type](state, action);
    } else {
        return state;
    }
};

export default rootReducer;

My question is: how valid is it? Thank you.

Except for using hasOwnProperty instead of in operator, redux docs leads you basically to the same structure: http://redux.js.org/docs/recipes/ReducingBoilerplate.html#generating-reducers

Yep, lookup tables are a very common approach.


Generate one event per metric name

I'd like to use the metrics plugin to generate 1 even per metric name. Clone can be used to manually split apart an event generated by metrics, however it's not always possible to know the name of every metric ahead of time. With 1 event per metric name, a slightly different event format would be useful where the event name is stored in a field instead of in a field-name. See below for examples.
Examples of dynamic metrics: Firewall rule, country, domain (webserver), URL, log event source server, etc.
Example logstash configuration:
filter {
  if "firewall" in [tag] {
    metrics {
      meter => [ "firewall.%{firewall_rule}" ]
      add_tag => "metric"
      rates => [ 1 ]
      flush_interval => 60
      clear_interval => 3600
      split_events => true
    }
  }
}

Example output from metrics plugin
{
@timestamp => "2016-02-21T15:04:28.414Z"
name => "firewall.rule1"
rate_1m => 0.29
count => 8
tag => [ "metric" ]
}
{
@timestamp => "2016-02-21T15:04:28.414Z"
name => "firewall.rule2"
rate_1m => 1
count => 60
tag => [ "metric" ]
}

Example output usage:

output {
  if "metric" in [tag] {

    # Store each metric reading in graphite (or some other time series database)
    graphite {
      metrics => [ "%{name}.count_1m", "%{rate_1m}" ]
    }

    # Store each metric reading in an ES index that's kept for a long period of time (years)
    elasticsearch {
      index => [ "metrics" ]
    }

  } else {
    # Store normal events in an ES database that's purged on a shorter timeframe (1mo/1wk)
    elasticsearch {
      index => "logstash-%{+YYYY.MM.dd}"
    }
  }
}

I have a similar requirement to produce metrics on a per event source basis. I have raised this issue for discussion in the logstash discussion forum and didn't see this particular enhancement discussion.
Trying to Count and Graph Multiple Event Sources
The concise version is that I'm trying to do this:
filter {
  if [message] {
    metrics {
      meter => [ "%{host}" ]
      add_tag => "metric"
    }
  }
}
output {
  # only emit events with the 'metric' tag
  if "metric" in [tags] {
    stdout {
      codec => rubydebug
    }
  }
}

With this valid configuration, the metrics filter does a great job at counting each message based on the event source host and producing the metrics, just not in a usable fashion when it comes to graphing time series.
I end up with metric events that contain an undetermined number of source hosts, I have no way to reliably filter the 1000+ possible server names to produce the individual metric events. If each event source host had it's own metrics event with common metric fields, very similar to the use case of the OP, I could easily produce charts on events per second per source host to a specific logstash instance.
This is particularly useful for determining what server is flooding my Redis queues with a large volume of events. I am currently doing this with statsd output plugin, however, it's pretty unreliable sending increment (1 event) stats and having statsd aggregate them and store in InfluxDB. It works, but it makes a lot more sense to have logstash calculate the rates and to store them in an elasticsearch metrics cluster.
Another slight niggle is that when a host stops sending events, the rate_1m never reaches zero, even after one minute it still producing a value over 0 which indicates to me there is an error with the way that value is being calculated although this is probably a gripe for separate plugin issue.

+1 I need this feature to get a structure I can use 'term split' for graphs later (kibana/grafana)

BTW:
this is not exactly the same as #17. In #17 the data is still structured as
split_%{key} = {  }
here it is flat and the name/key is a value pared with key name.
name = %{key}

Is there any traction on this? I'm running into a similar problem where I can create the metrics for a slew of dynamic names based on a field, but I cannot send them anywhere (in particular statsd) since I cannot reference them.

I have the same problem; wouldn't mind just using an additional script or something that does the cloning and splitting, but at the moment I cannot use the generated metrics events in a usable fashion for e.g. Kibana.

I've built a patch to metrics.rb, that introduces a new boolean option emit_single_metrics.
If set to false - the default - it behaves as before, but if set to true, events are generated in the proposed flat format with one metric per event.
metric.rb.diff.txt

@chbuescher That's awesome, I've built the patch into the latest version of logstash and going to test it shortly, thanks.

You can work around this by using Telegraf https://github.com/influxdata/telegraf
Along with sending the output to Elasticsearch, also send the event to Telegraf, which in turn can handle it like a proper metric event, aggregate and send out to InfluxDB or some other of its outputs.

@elvarb What receiver on Telegraf do you use? I've been actually been using the statsd plugin; and that's fine but it's pretty un-refined way of collecting statistics (spamming UDP packets for each Logstash event).
Can you elaborate on how Telegraf can split the various metric instances in a given event from a standard logstash-filter-metrics event? Thanks :+1:

@berglh I used just the standard tcp input plugin in telegraf, or if I used the statsd plugin. Don't have access to the system I set this up on anymore.
It basically meant that I was doubling the data coming from the logstash machine, one flow went to elasticsearch and the other to telegraf. Another benefit I got from this is that I selected which fields were sent over to telegraf so there I could use influxdb's tag system. So for an event that included  a sourceip field that became a tag set in Influxdb, then I could filter in on only events for a specific sourceip value.

Here's a work-around that I have used for this use-case:
Manipulate the metric events to put metric values in a list using the ruby filter (while recording the metric names in the list), and then use the split filter to generate individual events for each metric name in the list:
filter {
  if "firewall" in [tags] {
    metrics {
      meter => [ "firewall.%{firewall_rule}" ]
      add_tag => "metric"
      rates => [ 1 ]
      flush_interval => 60
      clear_interval => 3600
    }
  }

  if "metric" in [tags] {
    ruby {
      code => "
        metrics = []
        event.to_hash.each_key do |k|
          if k.start_with? 'firewall.'
            values = event.remove(k)
            values['name'] = k
            metrics << values
          end
        end
        event.set('metrics', metrics)
      "
    }

    split {
      field => "metrics"
      target => "metric"
      remove_field => "metrics"
      add_tag => "metric"
    }
  }
}

With this, we get roughly the following output events:
{
@timestamp => "2016-02-21T15:04:28.414Z"
metric => {
  name => "firewall.rule1"
  rate_1m => 0.29
  count => 8
}
tags => [ "metric" ]
}
{
@timestamp => "2016-02-21T15:04:28.414Z"
metric => {
  name => "firewall.rule2"
  rate_1m => 1
  count => 60
}
tags => [ "metric" ]
}

Will this feature be added ?

@viranch, did you face any performance issues with the work around.? We will be having around 50000+ events to split

@1tarak I've been using this fork and it's still installing OK in the most recent logstash versions:
https://github.com/berglh/logstash-filter-metrics
Assuming you are using logstash in the default path:
# Prepare Ruby Build Environmet
export GEM_HOME=/usr/share/logstash/vendor/bundle/jruby/2.3.0
export JRUBY_HOME=/usr/share/logstash/vendor/jruby/bin
export PATH=$(echo $PATH):/usr/share/logstash/vendor/jruby/bin:/usr/share/logstash/vendor/bundle/jruby/2.3.0/bin
cd /usr/share/logstash/plugins
git clone https://github.com/berglh/logstash-filter-metrics.git
cd /usr/share/logstash/plugins/logstash-filter-metrics
gem build logstash-filter-metrics.gemspec
/usr/share/logstash/bin/logstash-plugin remove logstash-filter-metrics
../../bin/logstash-plugin install *.gem
../../bin/logstash-plugin install *.gem

Then you can do stuff like this:
    metrics {
      meter => [ "%{[@metadata][pipeline]}" ]
      rates => 1
      emit_single_metrics => true
      add_field => {
        "[@metadata][metric]" => "true"
        "[@metadata][ls-dev-mode]" => "${LS_DEV_MODE:true}"
        "[metric][description]" => "logstash_processor_pipeline_throughput"
        "[service]" => "logstash"
        "[type]" => "metrics"
        "index" => "uq-metrics-%{+YYYY.MM.dd}"
      }
    }
    metrics {
      meter => [ "service" ]
      rates => 1
      emit_single_metrics => true
      add_field => {
        "[@metadata][metric]" => "true"
        "[@metadata][ls-dev-mode]" => "${LS_DEV_MODE:true}"
        "[service]" => "logstash"
        "[type]" => "metrics"
        "[metric][description]" => "logstash_processor_total_throughput"
        "index" => "uq-metrics-%{+YYYY.MM.dd}"
      }
    }

@viranch, did you face any performance issues with the work around.? We will be having around 50000+ events to split

With that version of the metrics plugin I'm indexing about 4,100,00 metrics a day to a single Elasticsearch node metrics instance.

@berglh, Thank you! i will install and try.

@1tarak As I'm using this in production, if it's not working in Logstash 7 let me know and I'll merge in the latest changes from the upstream plugin so it should build properly there as well as I'll need to do this anyway after the logstash-filter-dns bug is fixed :)

I am still using LS 6.2.3 in our prod. So we are good for now. Thank you!

@berglh Can you please rebase so it bumps to version 4.0.6? I'm using Logstash 6.8.1 and would like to try your patch. Thanks!

@berglh I'm using your patch based on 4.0.5 on 6.8.1 and it's working fine, thanks!

Is this working with Logstash 7?

@voiprodrigo I was using 7.9.1 and it was working fine, I'm just building 7.16.1 right now, I don't think this should be a problem though.

Thanks for the quick update! I'm running without issue on 7.16.1 as well.


Don't store tap config when value is unknown.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Closes https://github.com/Homebrew/brew/pull/16825.
This only happens if HOMEBREW_NO_GITHUB_API is set. Storing the config in that case has no benefit anyways since the whole point of it is to avoid API calls.

Thanks for fixing this!


How can I adjust the width of the date picker?

How can I adjust the width of the datepicker? So if I have a basic datepicker like in this sandbox . How can I adjust the width so it's not got lots of whitespace around the actual date? It's almost twice as wide as it needs to be.

Hi, you can scale down the whole calendar by specifying a font-size for the calendar using calendarClassName prop:
.scaled-down-calendar {
  /* by setting font-size, all the elements will correspond */
  font-size: 8px !important; /* default to 10px */
}

Ah, ok, is there not a way to scale down the datepicker independent of the font size? For example if I wanted the edges of the datepicker box to be close to the edges of the date text?

I was thinking of reducing the width like the box on the right here:

@AshleySetter Sorry for the latency in response. Do you want to reduce the space between the days and the calendar's left/right boundaries?

No worries, yes, that was what I had in mind.

There are a few styles that you need to override. I made a demo on CodeSandbox for that. Here's how the code looks like there:
First off, we give the calendar a custom class:
<Calendar
  value={value}
  onChange={setValue}
  calendarClassName="myCustomCalendar" {/* Giving a custom class to the calendar */}
/>

Then we override the styles based on that:
.myCustomCalendar {
  width: 29em !important; /* previously was 33em */
}

.myCustomCalendar .Calendar__section.-hiddenNext {
  transform: translateX(100%); /* previously was 90% */
}

.myCustomCalendar .Calendar__section.-hiddenPrevious {
  transform: translateX(-100%); /* previously was -90% */
}

.myCustomCalendar .Calendar__section {
  padding: 0 1em; /* previously was 0 3.2em */
}

.myCustomCalendar .Calendar__weekDays {
  padding: 0 0.8em; /* previously was 0 2.6em */
}

I really apologize for the latency in response 🙏🏻 I hope this is helpful.


Clearign an alarm causes an EOL error

Describe the bug
If clear an alarm according to the docs I receive a tedge/error.
To Reproduce

Clear the alarm:

mosquitto_pub -r -q 2 -t 'tedge/alarms/critical/temperature_alarm' -m ''

Watch the MQTT bus:

tedge/alarms/critical/temperature_alarm (null)
c8y/s/us 306,temperature_alarm
c8y-internal/alarms/critical/temperature_alarm (null)
tedge/errors EOF while parsing a value at line 1 column 0

Expected behavior
I would expect no error.
Screenshots

Environment (please complete the following information):

thin-edge.io version 0.12.0

Additional context

This is due to the converter recently introduced to convert messages from the old topics to the new ones.
This converter wrongly try to insert a severity in the empty message.
@PradeepKiruvale I think the simpler to fix that issue (and to prevent other similar issues) is to simply copy the alarm messages unchanged from tedge/# to te/# when the severity cannot be inserted.

This is due to the converter recently introduced to convert messages from the old topics to the new ones.
This converter wrongly try to insert a severity in the empty message.
@PradeepKiruvale I think the simpler to fix that issue (and to prevent other similar issues) is to simply copy the alarm messages unchanged from tedge/# to te/# when the severity cannot be inserted.

What happens when there is an issue with the topic itself? For example, if the alarm/event type is missing/empty then one must throw an error or not.

QA has thoroughly checked the feature and here are the results:

[x] Test for ticket exists in the test suite.
/tests/RobotFramework/tests/tedge_to_te_converter/convert_tedge_topics_to_te_topics.robot
[x] QA has tested the feature and it meets the required specifications.


Changed EC number in other databases

Description of the issue:
Protoporphyrin ferrochelatase enzyme (FECH) was classified as EC <IP_ADDRESS> but now it's classified as <IP_ADDRESS>. This change is already updated in several databases including the Enzyme database, KEGG, MetaCyc.
I came across this one accidentally, but I am wondering if there is any automatic way to do these checkings and update the model once in a while?
Expected feature/value/output:
We can update this one manually, for now. If you @mihai-sysbio will update it on the Metabolic atlas, I think we should keep the old EC number as well.
Current feature/value/output:
Reproducing these results:
I hereby confirm that I have:

[ ] Tested my code on my own computer for running the model
[ ] Done this analysis in the main branch of the repository
[ ] Checked that a similar issue does not exist already

I came across this one accidentally, but I am wondering if there is any automatic way to do these checkings and update the model once in a while?

That's a very good question - I vaguely remember discussions about it, but I could not find any implementation of this functionality. Perhaps @JonathanRob, @haowang-bioinfo or even @pecholley remember something.

If you @mihai-sysbio will update it on the Metabolic atlas

Metabolic Atlas strives to be a perfect mirror of the model content, so if a change on the website is expected, it should come from a change in the model.

We can update this one manually, for now.

Please go ahead @FaribaRoshanzamir 👍🏻

No, I'm not really aware of any automatic functionality that could be used to update this, unfortunately.

EC number changes but not frequently. @FaribaRoshanzamir please go ahead to update this if provided with evidence.

GECKO 3 contains a function to getECfromDatabases via KEGG mapping. Perhaps this is a good opportunity to bulk update the EC codes, and also reduce the missing values by about ~1900 entries.


Add shindig extension to white list allowed URLs

Proposed changes.

With these changes, a list of whitelisted URLs is used to filter internal calls done by the shindig application.
Hostname configured in the carbon.xml files is added by default, tailed by /portal endpoint.
If more URL paths need to be whitelisted, the following property can be added to repository/deployment/server/webapps/shindig/WEB-INF/classes/shindig.properties file.
wso2.shindig.proxy.whitelist=https://localhost:9443/oauth,https://www.google.com
If the provided URI contains a port, exact URI has to be matched. If not, a regex is used to match with the URI pattern allowing any port number.

@vihanga-liyanage , could you please create a doc issue and above content so it's easy to track


Added 4eye principle for approving PRs

Also added maintainers and contributors.
According to https://opensource.zalando.com/docs/releasing/index/#be-compliant

Coverage remained the same at 92.38% when pulling 383eaaef490d0fdefe194b59bccaf3382327840a on daryadedik:master into b5b231f2f19d72acc6e26459c8cb210cf8c3e403 on zalando:master.

👍

👍

👍

👍


Lame encoding

Hello
My file contains LAME3.93 encoding
How can I get metadata through your library, without hieroglyphs
I get the following data: "Ñòî ÷àñîâ" and "Þðèé Ëîçà"

Could you share a file that exhibits the problem?

Юрий Лоза-Сто часов.mp3.zip

Thank you for the test case. Was the ID3 tag in the file generated using LAME also?
The problem seems to be one of text encoding. While ID3v1 tags use the 8859-1 charset (although sometimes the machine's local encoding is used, such as Windows-1251 which appears to be the correct encoding for this particular ID3v1 tag), ID3v2 uses UTF-8. It seems the ID3v2 tag in this file is not encoded using UTF-8 but rather a different character set, most likely Windows-1251, the same as the ID3v1 tag.
Take the "word" Þðèé (should be Юрий) from the TPE frame. The octets in UTF-8 and Windows-1251 have the following hex values:

UTF-8
Hex value
Windows-1251

Þ
0xDE
Ю

ð
0xF0
р

è
0xE8
и

é
0xE9
й

The octet values interpreted using UTF-8 give Þðèé while using Windows-1251 give Юрий. So it seems that the text in both the ID3v1 and ID3v2 tags in this file is incorrectly encoded.

It's strange when Apple's native player easily recognizes text, just like Google Translate
I'll try to search for similar files and let you know the result

That is interesting. I will take a closer look at the file's tag to make sure it is being handled correctly. I've heard of charset detection for ID3v1 tags but for ID3v2 I don't think there should be any guessing involved.

Apple Music says it's version 3

It is an ID3v2.3 tag. The TPE1 frame for example contains the following bytes:

Field
Hex Bytes
Meaning

Frame ID
54 50 45 31
TPE1

Size
00 00 00 0A
10

Flags
00 00

Text Encoding
00
ISO 8859-1

Information
DE F0 E8 E9 20 CB EE E7 E0

It's possible that Music runs text reported as ISO 8859-1 through a character detection library. Based on the ID3v2 tag itself, TagLib (the metadata library used by SFBAudioEngine) is interpreting the data correctly.
It shouldn't be terribly hard to wrap uchardet to add the option for character set detection for ID3v1 or ID3v2 tags using ISO 8559-1 but I haven't investigated what it would entail.

I have the following question.
How can I understand from the text what encoding it has and show the user the appropriate text?

Algorithms for character set detection are something I know little to nothing about. Perhaps an educated guess is made based on a frequency analysis of octets in the input?
For the file that you shared it should be possible to feed the C strings from the metadata to uchardet or a similar library and see what it comes back with, and then use iconv to UTF-8.

This problem is not only with this file, I found a large number of such files, in the native application Files and Music the data is displayed as expected, but using the standard API does not lead to the desired result :(


std.experimental.allocator.make!T must either call GC.addRange or be marked @system

sludwig (@s-ludwig) reported this on 2016-12-18T18:19:08Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=16982
CC List

ag0aep6g
greeenify
johannespfau (@jpf91)

Description
When creating objects that contain references, make() doesn't currently call GC.addRange(), which may lead to dangling references in case the GC frees such an unseen reference.

To fix this, make() must either be marked as @system as soon as an object with references is created, or preferably, it should call GC.addRange(). A vanilla version ("makeNoGC"?) could be added to be able to opt-out of this, but it should be very clear from its name that it is unsafe in a GC scenario, because this is a pitfall that is very easy to hit.

The same reasoning applies to makeArray().

sludwig (@s-ludwig) commented on 2016-12-18T19:55:16Z
Yeah, I used to simply `static if (std.traits.hasIndirections!T) GC.addRange(...);` in vibe.d's allocator module. Using annotations instead of controlling this at the call site is an interesting idea, although I can see applications for both approaches.

Using @nogc sounds good, too, although if that approach is taken, I'd propose to define @nogc as a UDA in object.d instead of adding a new special trait for detecting its presence.

greeenify commented on 2016-12-27T11:36:10Z
*** Issue 15790 has been marked as a duplicate of this issue. ***

greeenify commented on 2016-12-27T11:37:46Z
(I just closed https://issues.dlang.org/show_bug.cgi?id=15790 as a duplicate, but it contains a good example that illustrates this issue)

b2.temp commented on 2016-12-18T18:58:40Z
We could instead imagine a system based on introspection (discover data that should trigger addRange()) and annotations (trust data for for which addRange() should be called but wont). Depending on the result of the introspection, GC.addRange() would be called automatically.

See also:

-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>-<EMAIL_ADDRESS>


blank UI after backup has been running for 12+h ?

Do you have the same issue after you restart the container ?

upping the cpu usage via crashplan on the web and giving it another gig of ram seems to have helped
backup set is HUUUGE on this one.
will keep everyone posted.


feat(packages): add create-scizor

Current dependencies on/for this PR:

main

PR #2   👈

This stack of pull requests is managed by Graphite.


mgmt/mcumgr/lib: doc: Minor fixes to zcbor_bulk documentation

Little polishing here ant there.
Signed-off-by: Dominik Ermel<EMAIL_ADDRESS>

Typo in commit *and


Added scheduled backups to admin page

Added the ability to enable backups with a set frequency (every x hours)
Added the ability for backups to be rolling or auto-delete if they are x hours old
Fixed admin.php page $user variable so faces now appear
Re-added userProfile.php but removed the ability to update any items. (Before it was just a broken link)

Let me know if this is worth bringing into the main repo. Also if I need to make any tweaks / code style changes. I'm currently running this on my own server and the backups appear to be running properly via cron.

This is awesome! I'll take a look at it and if it all looks good, I'll merge it.

This looks good, I'll probably remove the user profile page again and just add it's functionality to the dashboard, but this works for now.

Thanks! Also in the future I'll be sure to leave out all those tabs in my newlines.
My IDE likes to add those for some reason.

I don't mind the whitespace, I just have my editor set to remove trailing whitespace from lines and add a newline at the end of all files, since the extra whitespace is unnecessary. The code isn't consistent enough in this repo for me to care about little things like that :)


fix: ⚡ merge n lists (both allow and restrict)

Removes HEN list fetching

currently, both, the restricted-list and the permitted-list, are coming from the teia repo. does it make sense to keep these lists separated? because otherwise, this could be one single list with only the restricted accounts. the permitted accounts could just be removed from the restricted-list (since we are in control of that list).

currently, both, the restricted-list and the permitted-list, are coming from the teia repo. does it make sense to keep these lists separated? because otherwise, this could be one single list with only the restricted accounts. the permitted accounts could just be removed from the restricted-list (since we are in control of that list).

Imo it's more modular, if we want to optin others list for instance.

makes sense 👍


Problem in the CL configuration in IBM Languages

Hi,
when I make this command:

after the /*FIRST all become comment.
Thanks.

@edoardomessa Sorry for the long wait on this. Released in https://github.com/barrettotte/vscode-ibmi-languages/releases/tag/v0.6.16


App closed after open

I'm Using Samsng A15, with Android 15 version
When try open app closed, i try give app all permention but same
Thank You Developer :)

I tried it on a google A6 and it works, that's about the same resolution as yours. Did you build the app or download from the link, oasis-pos-87.apk?

Yes, I download it from link in Read Me section.
I don't build anything i just downaload apk then install it and select install anyway for Google Play Secerty

Note: I see splash screen then app closed, if there are log file tell me where can i found it to share log if that help
Thank You :)

If you install the android sdk and/or android studio then you can get all the log info.
So do you get to where you input the initial registration and download where you enter user and password?
Are you on wifi or  cell data?


data files of chapter 7

Hello!
Where could I find data files of chapter 7?
Thanks,
mukoedo1993

Hi,
Please take them from the chapter 4.
Best regards,
Kirill Kolodiazhnyi.
12 мая 2021, 13:02:04, от mukoedo1993 @.***>:
Hello!
Where could I find data files of chapter 7?
Thanks,
mukoedo1993
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Thank you.
But, when I used data of chapter 4 to run chapter 7's SharkML program, I got segmentation fault.
Could you tell me how to do so?
Thanks.

Am I understand right that other programs from chapter 7 work correctly with data files from chapter 4? If so please check that you use exact same version of SharkML library as book requires(built from source). Also a stack trace can help a lot.

I have changed my code to follow your CMakeLists.txt file strictly. Now, all problem have been solved. Thanks.


[Request] Local repository

ITNOA
Hi, I think if vcpkg having a mirroring procedures and capability, then we can use all packages in vcpkg in large company with LAN speed, and without needed to internet.
I think it is cool features.
I think vcpkg must be provide local server (LAN access) for satisfy this requirement. and vcpkg command must be discover local server and install from that if available. like nuget and etc...
Thx

@PhoebeHui I think it is vcpkg-feature not port feature
thanks :)

I think it is very important feature for production use and for large scale software development compony
@PhoebeHui  any plan for this feature?
thanks :)

@soroshsabz , yes, it's a vcpkg feature. thanks for pointing out this!
Currently we suggest a workaround to user, it should work.

Download the source code with a web browser from: .
Save it inside <path_to_vcpkg>/downloads/.
Then try to install the port again.

We didn't have a final plan for this now, hopefully we could address it in future.

@PhoebeHui Did you think Package Federation point in vcpkg roadmap could resolve this requirement?

@soroshsabz, we have a good start for it now, see PR https://github.com/microsoft/vcpkg/pull/12532


safety button pairing cmd master update (move from fmu -> safety_button)

The merge makes it difficult to see what's going on.
Here's the overall diff which should preserve https://github.com/PX4/Firmware/pull/12794. https://github.com/PX4/Firmware/compare/master...dagar:pr-pairing-cmd?expand=1

Squashed into https://github.com/PX4/Firmware/pull/12794.


Hotspot.create always throws error

Tried running the example code for Hotspot.create but I get error every time. Is this functionality depended on the hardware, or should it work on any device?

What's your device OS Version?

Hotspot.create didn't work using Android 6.0.
Also tested on an Android 9.0, in which not even Hotspot.enable worked.

You can't change wifi hotspot without enabling it first. Also, check the modify system settings permission is set to on.

Hi, I have the same problem, i can't enable the hotspot even with all permissions set on.
any suggestion?

Till now the library doesn't support a workaround for android 8++, I will solve this in next updates

When this update will come out?


Error building basic AOT library

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Debian GNU/Linux rodete
TensorFlow installed from (source or binary): source
TensorFlow version (use command below): v2.2.0-rc4-8-g2b96f3662b 2.2.0
Python version: Python3
Bazel version (if compiling from source): 3.1.0

Background
I'm following this tutorial. I can't build the sample library. I've already read this issue, which seems similar to mine, but it didn't help.
Steps followed

Clone tensorflow and go to aot's directory

git clone https://github.com/tensorflow/tensorflow.git
cd tensorflow/tensorflow/compiler/aot

Run scipt to generate protobuf

python3 tests/make_test_graphs.py --out_dir=.

Run test_graph_tfmatmul rule.

bazel build :test_graph_tfmatmul
Describe the current behavior
Step 3 errors out with the following message:
ERROR: /usr/local/google/home/adanlopez/tfcompile/tensorflow/tensorflow/python/BUILD:873:1: undeclared inclusion(s) in rule '//tensorflow/python:_pywrap_checkpoint_reader.so':
this rule is missing dependency declarations for the following files included by 'tensorflow/python/util/py_checkpoint_reader_wrapper.cc':
'bazel-out/host/bin/external/local_config_python/python_include/numpy/arrayobject.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/ndarrayobject.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/ndarraytypes.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/npy_common.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/numpyconfig.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/_numpyconfig.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/npy_endian.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/npy_cpu.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/utils.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/_neighborhood_iterator_imp.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/__multiarray_api.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/npy_interrupt.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/ufuncobject.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/npy_math.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/npy_common.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/arrayobject.h'
'bazel-out/host/bin/external/local_config_python/python_include/numpy/__ufunc_api.h'
Target //tensorflow/compiler/aot/tests:test_graph_tfmatmul failed to build
Use --verbose_failures to see the command lines of failed build steps.
INFO: Elapsed time: 135.764s, Critical Path: 47.52s
INFO: 336 processes: 336 local.
FAILED: Build did NOT complete successfully
Describe the expected behavior
Step 3 should not error out.

This happens when you do not have numpy installed for the python distribution you are using to build TF.
Please make sure to read the full guide here before building from sources:
https://www.tensorflow.org/install/source


Rename to awesome?

https://github.com/sindresorhus/awesome

awesome-FLOSSaaS-Providers ?
Do you have other ideas?

I'd suppose this could be awesome-libre-ISPs or something?

I was more on the Hosting Service Providers
But we could be broader and say Internet Service Provider.
Let's stay classic and use: awesome-hosting-provider. What do you think?

awesome-provider-coops?

@pierreozoux awesome-hosting, awesome-isps ?
Actually it could read awesome-indiehosters, because that's what they are, then the label being a commons.

I have done so for the @ecobytes fork at https://github.com/ecobytes/awesome-indiehosters and unsubscribe from this issue.

Sorry, I was a bit out of time lately...
I have troubles to think about IndieHosters.
I see 3 levels:

chatons: offering free software as a service with ethic: http://chatons.org/
migration network: it is a chatons that agreed to accept migrating users from other chatons
IndieHosters is somebody that is using the same infrastructure.

Why not awesome-librehosting ?
Actually this is a list of chatons :)
Here is a list (forked a bit from here: https://framagit.org/framasoft/CHATONS/wikis/structures-list)

It is definitely more than hosting what we do. There is, at least for @ecobytes, a lot of training and process controlling involved in our work.
The term chaton is completely not understandable for non-french speakers.
At least it seems they forked this list here (look at in Berlin), which could be useful for #7.
Yet I agree libre is a beautiful prefix. awesome-librehosters?

Do you provide training to end users of ecobytes?
Do you have a link to such offer (just for my personal reference).
Ok, awesome-librehosters it will be :)
(And yes, they forked from here plus added their info)
On 07-06-2016 16:47, jon r wrote:

It is definitely more than hosting what we do. There is, at least for
@ecobytes https://github.com/ecobytes, a lot of training and process
controlling involved in our work.
The term |chaton| is completely not understandable for non-french speakers.
At least it seems they forked this list here (look at in Berlin),
which could be useful for #7
https://github.com/indiehosters/nice-hosting-providers/issues/7.
Yet I agree |libre| is a beautiful prefix. |awesome-librehosters|?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/indiehosters/nice-hosting-providers/issues/2#issuecomment-224323551,
or mute the thread
https://github.com/notifications/unsubscribe/ABxvHWMPWx2npMbPi2uivBsa60i0MZ6Pks5qJZKIgaJpZM4HpTfx.

--
I use PGP to protect our privacy, if you want to know more, you can
follow this
https://emailselfdefense.fsf.org/en/
If you have further questions, please do not hesitate to ask.
You can verify my public key here: https://keybase.io/pierreozoux

I have updated #9 to reflect the choice above by adding https://github.com/indiehosters/nice-hosting-providers/pull/9/commits/b15af9caa5e65a990c94c1c4753d955bbb070585.


Feature/port2r4

Test out whether we can run project on r 4.x
to do this we had to update the renv stuff and then modify the travis.yml to check whether runs thru with no erorr

closes #2

closes #2


Set a Bus maxConcurrentCalls for a specific azure function

Hi !
I wanted to know if it is possible to have a service bus triggers to run one at a time while others to run in parallel with a a different value of maxConcurentCalls, within a same Azure Function App.
As I understand from the documentation, when setting servicebus->maxConcurrentCalls in the host.json, that applies for all the service bus trigger functions within the same FunctionApp.
Is there a way to apply those settings to only a specific Function within the FunctionApp ?
Thanks !

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

ID: 0440205f-adcc-0fc5-3adb-db4dbe1d1292
Version Independent ID: 14ad1300-590c-8102-6887-366078e1882a
Content: Azure Service Bus bindings for Azure Functions
Content Source: articles/azure-functions/functions-bindings-service-bus.md
Service: functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

Thanks for the question! We are currently investigating and will update you shortly.

@luislinietsky Since the host.json metadata file contains global configuration options that affect all functions for a function app, maxConcurrentCalls setting will get applied to all the service bus trigger functions within the same FunctionApp.
The effect of maxConcurrentCalls will be independent for each function. For example if you have "maxConcurrentCalls":10, each function will have up to 10 messages processed concurrently.
Note: maxConcurrentCalls applies per instance. If you have multiple instances running, the max concurrency increases proportionally.
If this is a feature you would be interested in, I would recommend going to Azure functions feedback and share your feedback, so that others can also vote on it and product team can prioritize this.

@luislinietsky Any update?

@Karishma-Tiwari-MSFT Hello guys,
Do you plan to support this feature in near future ?
Our team really need this functionality. We need to set one specific azure function to have maxConcurrentCalls value = 1, but for all other functions in the function app we need to use the default value ( I believe it is 16 ).
Thanks!
Dido

That feature is available today in newer versions?
I need maxconcurrentcalls with different values per functions

That feature is available today in newer versions?
I need maxconcurrentcalls with different values per functions


FAUST_DATA_with_GIH

I could not understand how to get the data that FAUST_DATA_with_GIH requires.
Specifically, it searches for obj files:  pose_mesh_path = self.path+'tr_reg_'+str(pose_mesh_idx).zfill(3)+'.obj'. However, the FAUST dataset supplies ply files with this prefix (e.g. tr_scan_030.ply).
Furthermore,   gt_path = self.path_GIH_gt+'tr_reg_'+str(identity_mesh_idx).zfill(3)+'.pkl' expects pkl file which I can't understand how to produce.  The LIMP repository produces folders (not pkl files) with this prefix e.g. tr_reg_099_2000. Each folder contains two files: 1. mesh.triv, 2. mesh.vert.
Can you please explain how to get the pkl and obj files?
Alternatively, I read in another issue that you planned to upload the preprocessed data to the cloud 3 months ago. Would it be possible for you to upload the data?
Thanks,
Yoni

Hi, thanks for being interested in our work.
Since I need to run my code on the server, some libraries to load the .ply file like pyrender is not compilable, I had to convert all the files into .obj  format for trimesh to process (way faster).
I generated the GIH myself and save it as .pkl file (again, for fast processing), instead of using the LIMP one. Also, LIMP downsampled the meshes into 2100 vertices while I calculated 6890 vertices, the GIH sizes are different.
Please check the script I newly attached (in the GIH calculating folder) to see how to generate the GIH maps.
Preprocessed data is uploaded and you can download it now, sorry that it takes longer than expected due to my procrastination.

Thank you very much!

Thank you very much!

Welcome! Looking forward to having further research cooperation!


[Bug]: Some Beam playground code snippets fail to load.

What happened?
For instance,
https://beam.apache.org/documentation/transforms/python/aggregation/approximateunique/ fails with Cannot load example: SDK_PYTHON_ApproximateUnique  ;
https://beam.apache.org/documentation/transforms/python/aggregation/groupbykey/ shows an empty snippet with a numbered line 26.
cc: @volatilemolotov @damccorm in case you are aware of recent changes.
Issue Priority
Priority: 2 (default / most bugs should be filed as P2)
Issue Components

[ ] Component: Python SDK
[ ] Component: Java SDK
[ ] Component: Go SDK
[ ] Component: Typescript SDK
[ ] Component: IO connector
[ ] Component: Beam YAML
[ ] Component: Beam examples
[ ] Component: Beam playground
[ ] Component: Beam katas
[ ] Component: Website
[ ] Component: Spark Runner
[ ] Component: Flink Runner
[ ] Component: Samza Runner
[ ] Component: Twister2 Runner
[ ] Component: Hazelcast Jet Runner
[ ] Component: Google Cloud Dataflow Runner

.take-issue

Looks like there are a couple more broken examples: https://lists.apache.org/thread/ccg2cfooq2sljd507o5y9sclmgjdqsgv
https://beam.apache.org/documentation/transforms/python/elementwise/flatmap/#example-3-flatmap-without-a-function
https://beam.apache.org/documentation/transforms/python/elementwise/map/
@Amar3tto  - would you have time to take a look?  Thank you so much!

Appreciate the quick response here!


Support insecure TLS and only CA cert in Elasticsearch scripts

Related to https://github.com/jaegertracing/jaeger-operator/issues/471#issuecomment-552842278

Merging without fuzzing to pass. This PR does not modify anything fuzzer tests.


Bug: Application error: The transaction was aborted, so the request cannot be fulfilled. AbortError: The transaction was aborted, so the request cannot be fulfilled.

Error message: The transaction was aborted, so the request cannot be fulfilled.
AbortError: The transaction was aborted, so the request cannot be fulfilled.
Error stack:
AbortError: The transaction was aborted, so the request cannot be fulfilled.
 AbortError: The transaction was aborted, so the request cannot be fulfilled.
    at Qu.rejectionMapper (altair://-/main-es2018.js:1:835167)
    at Ku (altair://-/main-es2018.js:1:773631)
    at IDBRequest.<anonymous> (altair://-/main-es2018.js:1:802934)
    at IDBRequest.__zone_symbol__ON_PROPERTYerror (altair://-/main-es2018.js:1:775475)
    at IDBRequest.O (altair://-/polyfills-es2018.js:1:31025)
    at e.invokeTask (altair://-/polyfills-es2018.js:1:26210)
    at Object.onInvokeTask (altair://-/main-es2018.js:1:106998)
    at e.invokeTask (altair://-/polyfills-es2018.js:1:26131)

When did this error occur? And does it re occur, or it happened only once?
Please provide some helpful screenshots of the bug

On what platform does this happen?

Hi,
it was Windows 10 - client app.
Petr
Am Mi., 3. Nov. 2021 um 09:27 Uhr schrieb Samuel @.***>:

On what platform does this happen?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/altair-graphql/altair/issues/1734#issuecomment-958738374,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEM3VRJM4DDUAHNUW375STUKDW6PANCNFSM5G7LAQXA
.

--
S pozdravem
Petr Hašlar

When does this error occur? And does it re occur, or it happened only once?

Hi, this error didn't occur lately, so I can't give you any more information :( maybe can close this issue.
Thank you


Where-Object returns abnormally when only 1 object matched

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
I am experiencing a Problem when using  Where-Object to filter objects.
1.When only 1 object matched, Where-Object returns abnormally.
FinaList.Count is 2 instead of 1. And FinaList[0] is ('xxx') instead of ('xxx','bbb').
Code：
$OutPutList = [System.Collections.ArrayList]::new()
$OutPutList.Add(@('xxx','bbb')) | Out-Null
$OutPutList.Add(@('mmm','bbb')) | Out-Null
$OutPutList.Add(@('myy','bbb')) | Out-Null
$FinaList = $OutPutList | Where-Object {$_[0] -like "x*"}
$FinaList.Count
$FinaList[0]
Output：

2.When 2 objects matched, Where-Object returns correctly.
FinaList.Count is 2. And FinaList[0] is ('mmm','bbb').
Code：
$OutPutList = [System.Collections.ArrayList]::new()
$OutPutList.Add(@('xxx','bbb')) | Out-Null
$OutPutList.Add(@('mmm','bbb')) | Out-Null
$OutPutList.Add(@('myy','bbb')) | Out-Null
$FinaList = $OutPutList | Where-Object {$_[0] -like "m*"}
$FinaList.Count
$FinaList[0]
Output：

Expected behavior
$OutPutList = [System.Collections.ArrayList]::new()
$OutPutList.Add(@('xxx','bbb')) | Out-Null
$OutPutList.Add(@('mmm','bbb')) | Out-Null
$OutPutList.Add(@('myy','bbb')) | Out-Null
$FinaList = $OutPutList | Where-Object {$_[0] -like "x*"}
$FinaList.Count
$FinaList[0]
Output：
1
xxx
bbb

Actual behavior
$OutPutList = [System.Collections.ArrayList]::new()
$OutPutList.Add(@('xxx','bbb')) | Out-Null
$OutPutList.Add(@('mmm','bbb')) | Out-Null
$OutPutList.Add(@('myy','bbb')) | Out-Null
$FinaList = $OutPutList | Where-Object {$_[0] -like "x*"}
$FinaList.Count
$FinaList[0]
Output：
2
xxx

Error details
No response
Environment data
$PSVersionTable
Name                           Value
----                           -----
PSVersion                      7.2.2
PSEdition                      Core
GitCommitId                    7.2.2
OS                             Microsoft Windows 10.0.22000
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

If I understand it correctly this is because Where-Object is outputting objects that match. When there is only one object you get that actual object back rather than a single array of 1 object. If you get multiple matches you get multiple objects back which is encapsulated inside an array. To always ensure you have an array back, whether it be empty or a single entry, wrap your pipeline inside @(), e.g.
$obj = @(1..3 | Where-Object { $_ -eq 2 })

By putting it inside the array subexpression it will ensure that the output is always an array. Even if you have multiple matches it will be those matches as array elements rather than an array in an array.

thanks @jborean93 Seem to have correct output after adding @ ()

$OutPutList = [System.Collections.ArrayList]::new()
$OutPutList.Add(@('xxx','bbb')) | Out-Null
$OutPutList.Add(@('mmm','bbb')) | Out-Null
$OutPutList.Add(@('myy','bbb')) | Out-Null
$FinaList = @($OutPutList | Where-Object {$_[0] -like "x*"})
$FinaList.Count
1
$FinaList[0]
xxx
bbb

If I understand it correctly this is because Where-Object is outputting objects that match. When there is only one object you get that actual object back rather than a single array of 1 object. If you get multiple matches you get multiple objects back which is encapsulated inside an array.

Yes. When  Where-Object was asked to select from arrays, the OP either got one array in $finalList - meaning count told him the number of members in that single array,  or $finallist was an array of arrays so count told him how many arrays inside it held.

By putting it inside the array subexpression it will ensure that the output is always an array. Even if you have multiple matches it will be those matches as array elements rather than an array in an array.

Yes another is to add the result to an empty array.


Ability to combine Hystrix and Feign

Currently there is no way to wrap a FeignClient with Hystrix but to have an extra class/bean. In newer versions (e.g. 8.11.) of feign there is Hystrix support (HystrixFeign). An alternative might be to support @HystrixCommand on Feign clients.

Just seen, this is fixed with the upgrade of Feign to 8.11


Environment not scaling down off-peak, any missing steps?

I have a 4x host deployment to try and prove the concept of this solution. Scaling up when peak hours begins seems to work fine, but upon scaling down - the process only manages the first VM in the pool then it seems to time out/fail. This is the output from my automation job where it will just hang until the next job starts - then they both fail within a few minutes.
Starting WVD tenant hosts scale optimization: Current Date Time is: 03/18/2020 15:45:56
It is Off-peak hours
Starting to scale down WVD session hosts ...
Processing hostpool FullDesktop
Checking session host: wvd-1.domain.local
of sessions:0 and status:Available
Checking session host: wvd-2.domain.local
of sessions:0 and status:Available
Checking session host: wvd-3.domain.local
of sessions:0 and status:Available
Checking session host: wvd-4.domain.local
of sessions:0 and status:Available
Stopping Azure VM: wvd-1 and waiting for it to complete ...
Azure VM has been stopped: wvd-1 ...
When I check the portal, wvd-1 has succesfully de-allocated but the other three are still running. I've deployed the Win10 1909 w/ Office Pro Plus image from the gallery and have performed no further config (I've deployed the host pool from the gallery which configures the hosts via DSC for use in an WVD Pool). Are there any other configurations that I need to carry out on the hosts for this to work?

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

ID: 24a6c4e2-702f-2b17-3e2d-c86f33b13beb
Version Independent ID: 1e7811d8-668a-db98-34ca-eaf4d7ee8c1f
Content: Scale session hosts Azure Automation - Azure
Content Source: articles/virtual-desktop/set-up-scaling-script.md
Service: virtual-desktop
GitHub Login: @Heidilohr
Microsoft Alias: helohr

There are no errors or warnings logged against the failed automation task, but this is listed as an exception:
The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: ActivityId: Powershell commands to diagnose the failure: Get-RdsDiagnosticActivities -ActivityId

Experiencing the exact same issue.

I will add that I was able to change the errorActionPreference under the workbook to "continue" and was able to generate the following error.
Get-RdsSessionHost : ActivityId: Powershell commands to diagnose the failure: Get-RdsDiagnosticActivities -ActivityId At line:679 char:27 + ... nHostInfo = Get-RdsSessionHost -TenantName $TenantName -HostPoolName ... + ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ + CategoryInfo : FromStdErr: (Microsoft.RDInf...tRdsSessionHost:GetRdsSessionHost) [Get-RdsSessionHost], RdsPowerShellException + FullyQualifiedErrorId : DefaultNoRdsError,Microsoft.RDInfra.RDPowershell.SessionHost.GetRdsSessionHost

@townendk Thank you for the detailed feedback. We are actively investigating and will get back to you soon.

@townendk Please find similar issue here, you can also find few suggestions/troubleshooting steps from product team.

Thanks All.
VikasPullagura-MSFT - this is not the same issue, I've read the one described in your post but in that scenario the session hosts do not heartbeat. Mine seem to have an appropriate heartbeat. Thank you though!

@viswanadham-k can you please check and add your comments on this.

I've done some further testing this morning.
1- Overnight all VMs in the pool were powered off (manually) and de-allocated in the portal. The tasks overnight all ran succesfully but they didn't power anything on. My "MinimumNumberOfRDSH" variable is set to '1' so I expected it to boot one of them back up for me but I'm perhaps misunderstanding the behaviour in off-peak time(?)
2- Upon peak time starting, the script correctly booted up the first VM in my pool.
3- I logged in two concurrent sessions to this (my current threshold is for one user per vCPU, and the VM had 2x vCPU)
4- The next script run successfully identified that my VM was at capacity with two users on it, so the second VM in the pool was booted up.
5- I logged off one of my users, but both VMs stayed up and running during peak time (I believe this is by design, scaling down only happens out of hours(?))
6- Peak time ended, and the job fired and correctly identified that my second VM host should be powered off. The VM has de-allocated in the portal but the script again is hanging at this stage:
Starting WVD tenant hosts scale optimization: Current Date Time is: 03/19/2020 09:34:44
It is Off-peak hours
Starting to scale down WVD session hosts ...
Processing hostpool FullDesktop
Checking session host: wvd-2.domain.local
of sessions:0 and status:Available
Checking session host: wvd-1.domain.local
of sessions:1 and status:Available
Stopping Azure VM: wvd-2 and waiting for it to complete ...
Azure VM has been stopped: wvd-2 ...
The task seems to hang at this point until the next scheduled occurrence is fired. Whether that's 15mins as per default, or 2 hours - the result is the same. The script hangs, then as soon as the next job fires they both immediately fail within a few seconds of each other.

@ChristianMontoya Can you check and add your comments.

I am experiencing, the exact same output as above as well.

Adding @RoopChevuri as well for comment.

Just wondering if there is an update on this.  Thank you.

Hi @townendk & @Jimbos10
Please let me know your availability time today. I will send a meeting invitation then will connect and assist you to resolve the error. Let me know your mailing addresses or drop a test mail to<EMAIL_ADDRESS>for to send meeting inviation.
Thanks

I have time now if you like, up until 3PM est.

I'm a bit tied up with WVD deployments today/tomorrow - I'll send a meeting invite later in the week. If @Jimbos10 fixes it beforehand, feel free to share the joy :)

Did you guys manage to talk about this? I'm tied up this week (with WVD deployments funnily enough) but I should hopefully have time for a meeting later in the week

Not yet, we did work on it, but I am sure that he is very busy given the current situation.

Hi, I worked with Microsoft and appears to be an issue with the automation account.  My automation account was created in East US, which doesn't appear to be working.  When we created the resource group and the automation account in West US2 it is now working.  They are going to continue to work on the issue with the EUS DC.

I wanted to add that I worked with Viswanadham, who was super knowledgable and very easy to work with.

Hi @Jimbos10, thank you for the update.
@Viswanadham-k - I've been experiencing the same in West Europe, so not sure if that helps the troubleshooting.

Just a quick followup Viswanadham found another issue with a variable that was reporting different values in different datacenter zones.  The updated code should now be available according to Viswanadham.  Thanks again for your help Viswanadham!

@viswanadham-k is there a plan for this code update to be reflected on the github repository referenced in this document?

Hi @thefonz3h
We already updated latest code to github repository. If you have already deployed WVD Scaling tool if you want latest code changes please run this script "createazureautomationaccount.ps1" it will reflect latest code.
Thanks

Hi @viswanadham-k I did a fresh deployment today getting the latest version of the script and it still failed with the same issue. I edited the script and could still see a variable set to WestUS. I tried switching this to WestEurope but it had no impact.
Do you have a direct download link for the latest version of this script so I can check it against my version?
Many thanks

@viswanadham-k Can you please check on this.
@RoopChevuri Please add your comments.

Hi @thefonz3h
We have fixed the issues which you have listed here.
Please go through below steps to update latest changes in your environment.
Access below link and copy the basicScale.ps1 code
https://github.com/Azure/RDS-Templates/blob/ptg-wvdautoscaling-automation/wvd-templates/wvd-scaling-script/basicScale.ps1
Access your automation account resource in Azure portal.
Click on Runbooks
a. Select the "WVDAutoScaleRunbook".
b. Paste into azure automation account runbook.
c. Click on save button and
d. click on publish button.
If you are still facing issue please let me know.
Thanks,

Hi @viswanadham-k thank you for the update. I have updated my automation account script and now can see that the scale down job no longer fails.
This may warrant a separate thread but now that this seems to be working OK, could I please get clarification on what the script SHOULD be doing? I've found it does not scale down hosts that have got users logged into them. I was expecting for these users to be ejected via a grace period but the scaling script just seems to ignore hosts with active user sessions.
Many thanks for the support so far!

Hi @townendk
Thank you for giving feedback.
Can you please close this issue.
Thanks

Hi,
I was experiencing the same issue. Apparently the Hosts Status when they are deallocated has changed from NoHeartBeat to unavailable. I changed to the new script suggested https://github.com/Azure/RDS-Templates/blob/ptg-wvdautoscaling-automation/wvd-templates/wvd-scaling-script/basicScale.ps1. Now some of the hosts were shutdown but not all that should have. For the hosts that were left online the script identified that it was off peak hours and started the "scale down" process but it kept trying to shut down the wrong host. Also, apparently the hosts that were on during Off peak hours and that were supposed to be down were left with the "No Allow New Connections" setting off so no users were able to connect in the morning.
See  sample output below.
It is Off-peak hours
Starting to scale down WVD session hosts ...
Processing hostpool AZC-WVD-Prod
Checking session host: AZCWVD-9.hl.local
of sessions: 0 and status: Available
Checking session host: AZCWVD-2.hl.local
of sessions: 1 and status: Available
Checking session host: AZCWVD-7.hl.local
of sessions: 1 and status: Available
Stopping Azure VM: AZCWVD-0 and waiting for it to complete ...
Azure VM has been stopped: AZCWVD-0 ...
HostpoolName: AZC-WVD-Prod, TotalRunningCores: 8 NumberOfRunningHosts: 2


KAFKA-18094: Remove deprecated TopicListing(String, boolean)

Remove deprecated TopicListing(String, boolean)
Add description about the removal to upgrade.html

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

the failed test is traced by https://issues.apache.org/jira/browse/KAFKA-18297


[FLINK-21463][sql-client] Unify SQL Client Parser and SQL parser

What is the purpose of the change
This is the final step towards unification between SQL parser and SQL Client parser. In this PR,

Remove the SQL Client parser and use the parser from the TableEnvironment to parse Operation.
Remove ResultStore and let the CliView to close the TableResult by themselves.

The reason why we decide to remove ResultStore as follows:

The CliView has the ability to capture the cancel from the user side and don't need to rely on the Executor to stop the job
After remove the ResultStore, the Executor is just a manager of the SessionContext and doesn't need to care about the cancel of the query. Therefore, we can merge the executeSql and executeQuery into one.

Brief change log

Remove the useless SqlClient parser and ResultStore
Replace executeSql and executeQuery with executeOperation in the Executor
Replace getParser with parseStatement in the Executor

Verifying this change
Actually we don't introduce any new feature but just refactor.  The origin tests are enough.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

@wuchong , could you please help @fsk119 to review this pull request. I would like to update the #15282 based on this pull request. Thanks.

Thanks for Jark's help. It looks good to me.


Add FindImagesByDigest to the pyxis client

Add a method to the Pyxis client that accepts a slice of digests,
and returns a slice of CertImages.
So far, it only fills in the ID and Certified fields of the image.

This will be used in the upcoming CsvImagesCertified check.
Signed-off-by: Brad P. Crochet<EMAIL_ADDRESS>

Coverage increased (+0.4%) to 60.623% when pulling da4fee5c4302e60565c297c00a95f0ca680d865a on bcrochet:csv-check into 1d6894d4c738b293cd463ee69b264c33bae77f5c on redhat-openshift-ecosystem:main.