Feature: Badged Managers

This is being postponed in favor of further research into N-Badge before deploying it inside Badger. While the functioning system could be implemented in today's version, the active unlock is minimal, and changes are likely.
We will come back to this as it is significant. However, there are more valuable things to focus on at this time.

We have landed on a different approach.
With module-level managers, NBadge Registry can be made a manager to the power of permissions.


dockerTools: stdenv-linux/setup: line 1305: jq: command not found

Issue description
I am playing around with the dockerTools and when trying to building the image I get the following error:
# more output not included above...
Unpacking base image...
From-image name wasn't set. Read docker.io/nixos/nix.
From-image tag wasn't set. Read latest.
/nix/store/j6k3spjs9rfd334za09ndk6zpbmaailq-stdenv-linux/setup: line 1305: jq: command not found
[    2.294509] reboot: Power down
builder for '/nix/store/ifr0wlrs7fwzckbxy13dwibd6g7wm0zg-docker-layer-test.drv' failed with exit code 127
cannot build derivation '/nix/store/2msxbarm2yxcpmjjrv86k9519yp7vd5k-docker-image-test.tar.gz.drv': 1 dependencies couldn't be built
error: build of '/nix/store/2msxbarm2yxcpmjjrv86k9519yp7vd5k-docker-image-test.tar.gz.drv' failed

However looking into /nix/store/j6k3spjs9rfd334za09ndk6zpbmaailq-stdenv-linux/setup:1305 there is no jq mentioned there and neither is it anywhere in that file.
Steps to reproduce
I created a minimal example here: https://gist.github.com/markus1189/f006d58e494cce4c3ba570e64ec0baef
Executing nix-build on that triggers the error for me, full output here: https://gist.github.com/markus1189/f006d58e494cce4c3ba570e64ec0baef#file-output-txt
Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the
results.
 - system: `"x86_64-linux"`
 - host os: `Linux 4.19.9, NixOS, 19.03pre162671.44b02b52ea6 (Koi)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.1.3`
 - channels(root): `"nixos-19.03pre162671.44b02b52ea6"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

@nlewo just tested with master and it works :) Good call, closing this


Change xmap signature for better type-inference

xmap[B](ba: B => A)(ab: A => B) ←instead of this
xmap[B](ab: A => B)(ba: B => A) ←this gives better type-inference because A is already known.
Type-inference is good at working out the output of a function but terrible at working of the input of a function (in fact the scalac guys just hardcoded a few hacks here and there cos they found it too hard due to subtyping).

Oh and I renamed map to xmap because it's the common name for this shape change. Usually when changing F[A] to F[B]

map(A => B) = covariant map
c(ontra)map(B => A) = contravariant map
xmap(A => B)(B => B) = exponential map

Oh, didn't know about the type-inference rules. LGTM!

Yeah I really hope they do better in Dotty (Scala 3).


Fix link to point at new issue tracker

Hi,
I just tried the link to Trac and it redirected me to JIRA. Maybe the link should be updated?
Regards,
James

Thanks!
Sorry it took so long to look at this pull request. We have just added a "CONTRIBUTING" file documenting our contributions procedures and, rather than point you at that and force you to go through signing the CLA, etc., I decided to update the README, fixing the URL in the process.
Please don't let this stop you from contributing, though! The more the merrier. Our brand new contribution procedures are here:
https://github.com/glyptodon/guacamole-client/blob/master/CONTRIBUTING
In theory, GitHub will display these automatically when pull requests are opened.


SRDB: fix NAs / NACs in citation.ID

There's a huge number of records with NA/ NAC in citation.ID. I'm not yet sure why.

@ValentineHerr , I'm not sure what's causing this-- something in the code?
An example is ForC record 22249/ SRDB record 691. Site = Fazenda Vida Nova. Study = #937, Author = Fiegl.

Good catch. sorry for that.
There was a problem in the code. citation.ID is populated only if Quality_flag is not "Q02" but the code was not considering the possibilities where Quality_flag is NA...
I fixed it.

Ok, now all records coming from SRDB that have citation.ID imported as "NAC" (418 cases) are that way because their Quality_flag is "Q02" in the original SRDB data base.
@teixeirak , please close this issue if that is what is supposed to be.

For the record, Q02 = data from another study.
This is correct according to my original instructions, but I'm wondering if it would be better to somehow retain the citation in SRDB in association with these records-- at least in source.notes. @bpbond, do you have an opinion on this? Would the citations associated with records with this code typically at least point to the real source? It seems important to preserve those citations.  Maybe list the study in citation.ID and then put a note in source.notes that data may be from other study?

Usually the "Q02" codes aren't that important; it signals that some of the ancillary data, not the primary soil respiration data, come from another study.

Okay, @ValentineHerr , let's include the citation.ID for these studies, but add a note in source.notes on variables other than soil respiration stating that data may come from another study.

Ok, now all records for variables other than  "R_soil_C", "R_het_litter_C", "R_auto_root_C", "R_het_soil_C", "R_auto_root_C" or "R_het_soil_C" get "Non-soil respiration data may come from another study." added to their source.notes field if their quality flag was originally "Q02" in SRDB.
@teixeirak, please close if that is correct.

Good, thanks!


The REFramework fails to initialize.

Describe the bug
Game: Resident Evil 2. The REFramework fails to initialize and is stuck on the Default Menu. The Default Menu Key doesn't work. The REFramework creates two empty folders: "reframework\plugins" and that's it.
Logs and any crash dumps
The log file is attached to this message.
re2_framework_log.txt
Identify the REFramework version
The REFramework version used: 1.5.3 TDB66 (The one without Ray Tracing)
Expected behavior
I expect it to initialize and behave according to the description of what it's supposed to do.
Desktop and VR (please complete the following information):

OS (Windows 10)
GeForce GTX 1080 Ti
AMD Ryzen 5 3600

Verifying the integrity of the game files fixed it.


Fix fresh install registry auth issue

During a new install, the docker inspect sidecar command returns a non-empty string.  Prior conditional statement checked for length of 0 which prevented the Secret Blob prompt to the user.

Closing, new pull request with null check submitted.


[ServiceBus] Message body allows dictionary but can't receive the message

azure-servicebus:
7.0.0b6:
Windows:
3.7.9:

Describe the bug
According to the docs azure.servicebus.Message can accept string or bytes as body but by mistake I sent a dictionary without converting to string or bytes, the message was sent but I was not able to receive message on the other end. I spent a lot of time thinking my code was wrong on the other side cause I see there is a message active in the service bus.
Later I realized the error is because I sent the body without converting to string or bytes. Is there a way we could check for the type when creating the message?
To Reproduce
Following is example code which resulted in message getting stuck in the servicebus.
dict_test = {
"test1": "",
"test2": {
"test3": "",
"test4": "",
"test5":"westus2",
"test6": "2020-09-24"
},
}
message = Message(dict_test)
Expected behavior
Should not allow message body to be other type than string or bytes

Hey @divyag9 ;
Thanks for reaching out!  This is more perfect timing that you probably know, as we were contemplating this failure mode internally, and you've given strong impetus to taking action on it.
I believe were're in agreement as to the semantic goal, that invalid types should fail-fast with a ValueError during Message construction; the subtlety that we've been debating has been how to do this validation in a robust and complete way, as the breakdown of valid types is a little more fine-grained then mentioned.  (e.g. there are some valid AMQP scenarios involving lists, although the behavior is sometimes unintuitive)
Will keep this thread looped in as we settle on a conclusion and PR it in, expect to see something in this release cycle, and don't hesitate to shout here if you have any thoughts on the above or additional color to add.

Great thank you!

Hey @divyag9 , just wanted to close this out with the good news that the 7.0 GA release contains the expected fail-fast behavior in Message construction.
There is ongoing work in accordance to python guidelines to support Valid dict-representations in leiu of strongly typed arguments (see here ) but this should not distract from the behavior when an invalid dict is passed, to immediately raise a TypeError.
Don't hesitate to shout/reopen if any further questions or issues, and thanks as always for reaching out and your patience as we addressed this!


Clashing entries results in "Error: user aborted"

Summary
Attempting to run gopass show a when there are multiple entries starting with "a" results in the error Error: user aborted.
Steps To Reproduce
Suppose your gopass structure is as follows:
gopass
└── Privacy+Security
    ├── alpha
    └── apple

Then if you run gopass show a, it results in:
➜ gopass show a
Entry 'a' not found. Starting search...

Error: user aborted
➜

Expected behavior
A better error message telling me that "a" can match multiple entries.
Environment

OS: Ubuntu 18.04
OS version: Linux labs-319 4.15.0-43-generic #46-Ubuntu SMP Thu Dec 6 14:45:28 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
gopass Version:

gopass 1.8.4 (47d1c43d 2019-01-02 14:30:40) go1.10.7 linux amd64
<root>     -  gpg 2.2.3 - noop 0.0.0 -   fs 0.1.0
Available Crypto Backends: gpgcli, openpgp, plain, xc
Available RCS Backends: gitcli, noop
Available Storage Backends: fs, inmem

Installation method: using docker

Additional context
If I run gopass show x (note that there is no entry starting with "x"), then I get a better error message:
➜ gopass show x
Entry 'x' not found. Starting search...

Error: no results found
➜

It seems something is wrong with your terminal. The curses UI doesn't seem to work.

Just like #1032, I will close this for now; and reopen in future if necessary. Apologies for not having followed this up sooner.


Fix: remove unwanted line that was part of old config option metadata…

Fix remote repo document for maven repos

I have read the CLA Document and I hereby sign the CLA

recheck

I have read the CLA Document and I hereby sign the CLA

I have read the CLA Document and I hereby sign the CLA


Project splash page

Spectacles needs a splash page, which will be hosted on the CORE studio website.  @mnaugle-tt wants to look into some fancy CSS and has some ideas about layout.

This might be also a cool example. Very clean!
http://github.hubspot.com/pace/docs/welcome/

I am looking into something like this for the VR page
http://www.fiftythree.com/
You can scroll down and get the projects to visualize

@mostaphaRoudsari --
Looking at mocking up two different single pages.  One that utilizes bootstraps similar to something like this: http://ironsummitmedia.github.io/startbootstrap-freelancer/
And one full width, single click scroll page similar to modelo.io or this: http://alvarotrigo.com/fullPage/#secondPage
Too torn on which one to go with, so I will try both and we can vote on which we like better!


Fixed various things that Landscape was complaining about

Coverage decreased (-0.38%) to 76.92% when pulling 448f53e2b56cd4e93c01d02be325598c12f3c3b2 on erikbern/landscape into 1939a06c1b205fcb20ba80223537345333f45c6a on master.

88% -> 90% so I guess minor improvement

Coverage increased (+0.01%) to 77.31% when pulling cd3550589b53ba9bbb2a09f811b2577f8aab070b on erikbern/landscape into 1939a06c1b205fcb20ba80223537345333f45c6a on master.

Coverage decreased (-13.51%) to 63.78% when pulling e606e0fcb8ab1d7f562e280b450707d1658807ac on erikbern/landscape into 1939a06c1b205fcb20ba80223537345333f45c6a on master.

Coverage decreased (-0.11%) to 77.18% when pulling 41e413186a8ca761967d61d3adba233ac65c657f on erikbern/landscape into 1939a06c1b205fcb20ba80223537345333f45c6a on master.

can has merge?


feature-wip support the array type in reverse function

Proposed changes

this pr is used to support the array type in reverse function.
Issue Number: #10052

2.we can run this function in sql as follow:
mysql> set enable_array_type=true;
mysql> set enable_vectorized_engine=true;
mysql> select k1, k2, reverse(k2) from array_test;
+------+-----------------------------+-----------------------------+
| k1   | k2                          | reverse(k2)         |
+------+-----------------------------+-----------------------------+
|  1   | [1, 2, 3, 4, 5]             | [5, 4, 3, 2, 1]             |
|  2   | [6, 7, 8]                   | [8, 7, 6]                   |
|  3   | []                          | []                          |
|  4   | NULL                        | NULL                        |
|  5   | [1, 2, 3, 4, 5, 4, 3, 2, 1] | [1, 2, 3, 4, 5, 4, 3, 2, 1] |
|  6   | [1, 2, 3, NULL]             | [NULL, 3, 2, 1]             |
|  7   | [4, 5, 6, NULL, NULL]       | [NULL, NULL, 6, 5, 4]       |
+------+-----------------------------+-----------------------------+
mysql> select k1, k2, reverse(k2) from array_test01;
+------+-----------------------------------+-----------------------------------+
| k1   | k2                                | reverse(k2)               |
+------+-----------------------------------+-----------------------------------+
|  1   | ['a', 'b', 'c', 'd']              | ['d', 'c', 'b', 'a']              |
|  2   | ['e', 'f', 'g', 'h']              | ['h', 'g', 'f', 'e']              |
|  3   | [NULL, 'a', NULL, 'b', NULL, 'c'] | ['c', NULL, 'b', NULL, 'a', NULL] |
|  4   | ['d', 'e', NULL, ' ']             | [' ', NULL, 'e', 'd']             |
|  5   | [' ', NULL, 'f', 'g']             | ['g', 'f', NULL, ' ']             |
+------+-----------------------------------+-----------------------------------+
Problem Summary:
Describe the overview of changes.
Checklist(Required)

Does it affect the original behavior: (No)
Has unit tests been added: (No)
Has document been added or modified: (No)
Does it need to update dependencies: (No)
Are there any changes that cannot be rolled back: (No)

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

please also update reverse docs.

please also update reverse docs.

done

LGTM

Please rebase master

Please rebase master

done


cpe_chrome is broken on chef 16

-->
Description
on Linux and macOS, cpe_chrome always returns with zero resources. This is because the resource names are identical for both windows/non-windows.
Chef 16 looks to not handle that any longer.
Chef Version
Chef 16.1
Platform Version
macOS/Linux
Replication Case
Run chef, notice that the profiles get removed.
Client Output
Not needed
how to fix
change cpe_chrome/recipes/default.rb
if node.windows?
  cpe_chrome_win 'Configure Google Chrome'
else
  cpe_chrome 'Configure Google Chrome'
end

change cpe_chrome/resources/cpe_chrome_win.rb
resource_name :cpe_chrome_win
provides :cpe_chrome, :os => 'windows'
default_action :config

That cant be true. Then none of the bulit in resources would would. EX:
file, package, user
How were you able to re-pro ?
On Mon, May 18, 2020 at 2:27 PM Erik Gomez<EMAIL_ADDRESS>wrote:

-->
Description
on Linux and macOS, cpe_chrome always returns with zero resources. This is
because the resource names are identical for both windows/non-windows.
Chef 16 looks to not handle that any longer.
Chef Version
Chef 16.1
Platform Version
macOS/Linux
Replication Case
Run chef, notice that the profiles get removed.
Client Output
Not needed
how to fix
change cpe_chrome/recipes/default.rb
if node.windows?
cpe_chrome_win 'Configure Google Chrome'
else
cpe_chrome 'Configure Google Chrome'
end
change cpe_chrome/resources/cpe_chrome_win.rb
resource_name :cpe_chrome_win
provides :cpe_chrome, :os => 'windows'
default_action :config
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/facebook/IT-CPE/issues/243, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6YUZK4KYNETNDCXCEHWQDRSGR4NANCNFSM4NEN64HA
.

By running chef lol. Try it out :)
Thanks,
Erik Gomez

From: Mike Dodge<EMAIL_ADDRESS>Sent: Monday, May 18, 2020 9:19:25 PM
To: facebook/IT-CPE<EMAIL_ADDRESS>Cc: Erik Gomez<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [facebook/IT-CPE] cpe_chrome is broken on chef 16 (#243)
That cant be true. Then none of the bulit in resources would would. EX:
file, package, user
How were you able to re-pro ?
On Mon, May 18, 2020 at 2:27 PM Erik Gomez<EMAIL_ADDRESS>wrote:

-->
Description
on Linux and macOS, cpe_chrome always returns with zero resources. This is
because the resource names are identical for both windows/non-windows.
Chef 16 looks to not handle that any longer.
Chef Version
Chef 16.1
Platform Version
macOS/Linux
Replication Case
Run chef, notice that the profiles get removed.
Client Output
Not needed
how to fix
change cpe_chrome/recipes/default.rb
if node.windows?
cpe_chrome_win 'Configure Google Chrome'
else
cpe_chrome 'Configure Google Chrome'
end
change cpe_chrome/resources/cpe_chrome_win.rb
resource_name :cpe_chrome_win
provides :cpe_chrome, :os => 'windows'
default_action :config
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/facebook/IT-CPE/issues/243, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA6YUZK4KYNETNDCXCEHWQDRSGR4NANCNFSM4NEN64HA
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/facebook/IT-CPE/issues/243#issuecomment-630537411, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABLL6GHZWAGTH5WX5IFECRLRSHUC3ANCNFSM4NEN64HA.

Chef 16.0.287, current head of this repo (5e0bce5876e88faeb8a4d272566d2627af4423db), Windows 10. The cpe_chrome cookbook doesn't appear to have any issues for me.
Having a better example of a reproduction would be helpful.
PS C:\Users\btm\Documents\IT-CPE\itchef> cat .\chrome.json
{
        "cpe_chrome": {
                "mp": {
                        "FileContents": {
                                "first_run_tabs": [
                                        "https://www.facebook.com"
                                ]
                        }
                }
        }
}
PS C:\Users\btm\Documents\IT-CPE\itchef> chef-client -z -o cpe_chrome -j chrome.json
[2020-05-19T20:21:52+00:00] WARN: No config file found or specified on command line. Using command line options instead.
Starting Chef Infra Client, version 16.0.287
[2020-05-19T20:22:06+00:00] WARN: Run List override has been provided.
[2020-05-19T20:22:06+00:00] WARN: Overridden Run List: [recipe[cpe_chrome]]
resolving cookbooks for run list: ["cpe_chrome"]
Synchronizing Cookbooks:
  - cpe_profiles (0.1.0)
  - cpe_chrome (0.1.0)
  - cpe_helpers (0.1.0)
  - fb_helpers (0.1.0)
Installing Cookbook Gems:
Compiling Cookbooks...
Converging 1 resources
Recipe: cpe_chrome::default
  * cpe_chrome[Configure Google Chrome] action config[2020-05-19T20:22:07+00:00] WARN: TotalMemoryLimitMb is not a supported setting

    * registry_key[HKLM\Software\Policies\Google\Chrome\AutoplayWhitelist] action delete_key (up to date)
    * registry_key[HKLM\Software\Policies\Google\Chrome\AutoplayWhitelist] action create (skipped due to not_if)
    * registry_key[HKLM\Software\Policies\Google\Chrome\ExtensionInstallForcelist] action delete_key (up to date)
    * registry_key[HKLM\Software\Policies\Google\Chrome\ExtensionInstallForcelist] action create (skipped due to not_if)
    * registry_key[HKLM\Software\Policies\Google\Chrome\ExtensionInstallBlacklist] action delete_key (up to date)
    * registry_key[HKLM\Software\Policies\Google\Chrome\ExtensionInstallBlacklist] action create (skipped due to not_if)
    * registry_key[HKLM\Software\Policies\Google\Chrome\DefaultPluginsSetting] action delete_key
      - delete key HKLM\Software\Policies\Google\Chrome\DefaultPluginsSetting
    * registry_key[HKLM\Software\Policies\Google\Chrome\DefaultPluginsSetting] action create
      - create key HKLM\Software\Policies\Google\Chrome\DefaultPluginsSetting
      - set value {:name=>"DefaultPluginsSetting", :type=>:dword, :data=>nil}
    * registry_key[HKLM\Software\Policies\Google\Chrome\ExtensionInstallSources] action delete_key (up to date)
    * registry_key[HKLM\Software\Policies\Google\Chrome\ExtensionInstallSources] action create (skipped due to not_if)
    * registry_key[HKLM\Software\Policies\Google\Chrome\PluginsAllowedForUrls] action delete_key (up to date)
    * registry_key[HKLM\Software\Policies\Google\Chrome\PluginsAllowedForUrls] action create (skipped due to not_if)
    * registry_key[HKLM\Software\Policies\Google\Chrome\RelaunchNotification] action delete_key
      - delete key HKLM\Software\Policies\Google\Chrome\RelaunchNotification
    * registry_key[HKLM\Software\Policies\Google\Chrome\RelaunchNotification] action create
      - create key HKLM\Software\Policies\Google\Chrome\RelaunchNotification
      - set value {:name=>"RelaunchNotification", :type=>:dword, :data=>nil}
    * registry_key[HKLM\Software\Policies\Google\Chrome\RelaunchNotificationPeriod] action delete_key
      - delete key HKLM\Software\Policies\Google\Chrome\RelaunchNotificationPeriod
    * registry_key[HKLM\Software\Policies\Google\Chrome\RelaunchNotificationPeriod] action create
      - create key HKLM\Software\Policies\Google\Chrome\RelaunchNotificationPeriod
      - set value {:name=>"RelaunchNotificationPeriod", :type=>:dword, :data=>nil}
    * file[delete-c:\Program Files (x86)\Google\Chrome\Application\master_preferences] action delete (skipped due to only_if)
    * directory[C:\Program Files (x86)\Google] action create (up to date)
    * directory[C:\Program Files (x86)\Google\Chrome] action create (up to date)
    * directory[C:\Program Files (x86)\Google\Chrome\Application] action create (up to date)
    * file[create-c:\Program Files (x86)\Google\Chrome\Application\master_preferences] action create (up to date)

[2020-05-19T20:22:07+00:00] WARN: Skipping final node save because override_runlist was given

Running handlers:
Running handlers complete
Chef Infra Client finished, 7/22 resources updated in 09 seconds

You can follow along at https://github.com/chef/chef/issues/9885 for the root cause, but the tl;dr is that Chef would silently determine what resource it should be loading, but when that went away it defaulted to using the Windows resource. As such, the reason why you saw nothing on Linux and MacOS was that the Windows resource was sanely returning since it wasn't running on WIndows :)
I think by simply renaming the resource to cpe_chrome_win the collision is gone, it's correctly determining the resource through provides, and this issue should be solved! Please let me know if you're still having problems and we can look into it, otherwise closing this out.
Thank you so much for bringing this to our attention, helping us get rid of this issue in a few other places too!

Sweet, I'll backport this fix as well. Glad you were able to replicate it.

@steelcowboy as I mentioned over on this commit, I recommend leaving the resources with the same name, but the linux version needs a provides line added to it.
This makes it easier on the user, you don't have to figure out and remember if it is cpe_chrome_win or cpe_chrome_linux or cpe_chrome or win_cpe_chrome or any other combination. You can have multiple resources with the same name (even on Chef 16) but chef needs the provides line to figure out which one to use.

Interesting because even with the provides, it did not work for me on Darwin.
Thanks,
Erik Gomez

From: Bryan McLellan<EMAIL_ADDRESS>Sent: Wednesday, May 20, 2020 7:00:25 AM
To: facebook/IT-CPE<EMAIL_ADDRESS>Cc: Erik Gomez<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [facebook/IT-CPE] cpe_chrome is broken on chef 16 (#243)
@steelcowboyhttps://github.com/steelcowboy as I mentioned over on this commithttps://github.com/facebook/IT-CPE/commit/5218f618d9d6b7ca4e8439dc4e414e8092eafcfc#commitcomment-39293782, I recommend leaving the resources with the same name, but the linux version needs a provides line added to it.
This makes it easier on the user, you don't have to figure out and remember if it is cpe_chrome_win or cpe_chrome_linux or cpe_chrome or win_cpe_chrome or any other combination. You can have multiple resources with the same name (even on Chef 16) but chef needs the provides line to figure out which one to use.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/facebook/IT-CPE/issues/243#issuecomment-631429508, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABLL6GBFS34FZOND23DQ3FLRSPA5TANCNFSM4NEN64HA.

@steelcowboy as I mentioned over on this commit, I recommend leaving the resources with the same name, but the linux version needs a provides line added to it.

@btm what directory structure would you recommend, if the resources have the same names?
Currently we have
cpe_chrome
|- resources
|- cpe_chrome_posix.rb (renamed from cpe_chrome.rb)
|- cpe_chrome_win.rb
curious what your recommendation is. This works, and it's the Facebook convention so people who use Chef won't get confused, so if there's a more canonical way we should consider doing a full refactor to be consistent.

@michel-slm that is fine, but would drop the resource_name lines from the resources.

lamont: if you haven’t set a resource_name yet, chef 16 will wire up a resource_name for the first provides it finds
lamont: (there’s also the fallback default of cookbook_name+resource_file if nothing at all is found)

when you have both a resource_name and provides you're kind of telling chef what the resource is called twice.

@btm oh interesting. Is that safe on all Chef versions? On the production side Facebook is still using Chef 13 (and trying to upgrade); on the client side we're on 15/16 but we want to keep our style guide consistent.


Make sure grape-engine exit when CTRL-C without crash

What do these changes do?

Reset the pointer to prevent double-shutdown which causes crash inside grpc
Terminate dispatcher (and its queues) correctly when closing

Related issue number
Fixes #2749

Codecov Report

Merging #2761 (852bed9) into main (df16284) will increase coverage by 26.04%.
The diff coverage is n/a.

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #2761       +/-   ##
===========================================
+ Coverage   41.39%   67.44%   +26.04%
===========================================
  Files          99       99
  Lines       10381    10381
===========================================
+ Hits         4297     7001     +2704
+ Misses       6084     3380     -2704

see 60 files with indirect coverage changes

Continue to review full report in Codecov by Sentry.

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


Automatically Close Normal Alerts

Hi,
I have a peculiar requirement, where I would like to close alerts automatically when they go back to Normal state. I am quite happy with the rest of the Alert Life Cycle. How should it affect the overall Bosun performance?

Bosun doesn't have that feature. You could do it via the api. I don't really think it would impact performance, but you might get a lot of alert noise.
Perhaps the log alert directive might do what you want? That basically bypasses the entire alert cycle, and just triggers when the condition is true every nth check)

"Log" alerts are not that useful, since I want to handle other Alert events such as Ack.
What I did was tweak some of code, so that whenever alert goes to "Normal" state, close the incident in same way the "unknown" alerts are closed when "ignoreUnknown = true".
That code changes seems to be working for me for now :)
Thanks.

FWIW i added this as a configuration on the alert for a use-case we needed at our company. If the key is set inside the alert config, the issue will automatically close and notify if it returns to the normal state.
https://github.com/knowroozi/bosun/commit/c0e09081d3f45574c237fe278f18a971edbe91d4
I can submit a PR if this is desired to be included to the project

After fishing in alerts for quite a while, we realized it is not such a great idea to automatically close normal alerts. 😆
Thanks.

@knowroozi your patches fail sometimes:
2016/08/30 17:00:53 info: check.go:529: check alert servers_packet_loss start
2016/08/30 17:00:53 info: check.go:553: check alert servers_packet_loss done (25.717244ms): 3 crits, 2 warns, 0 unevaluated, 0 unknown
2016/08/30 17:00:54 info: search.go:207: Backing up last data to redis
2016/08/30 17:01:03 info: alertRunner.go:55: runHistory on servers_packet_loss took 10.13331037s
2016/08/30 17:01:53 info: check.go:529: check alert servers_packet_loss start
2016/08/30 17:01:53 info: check.go:553: check alert servers_packet_loss done (23.790258ms): 3 crits, 1 warns, 0 unevaluated, 0 unknown
2016/08/30 17:02:00 info: check.go:249: auto close servers_packet_loss{instance=[REDACTED]} because returned to normal
2016/08/30 17:02:00 error: check.go:252: cannot close active alert
2016/08/30 17:02:03 info: alertRunner.go:55: runHistory on servers_packet_loss took 10.205952693s

I've not investigated in full yet, but have a feeling this has something to do with notification chain that is still active for that alert.


Unable to npm install

Please only file bugs/feature requests for rollup-plugin-closure-compiler here.

If you have questions about how to use AMP or other general questions about AMP please ask them on Stack Overflow under the AMP HTML tag instead of filing an issue here: http://stackoverflow.com/questions/tagged/amp-html
If you have questions/issues related to Google Search please ask them in Google's AMP forum instead of filing an issue here: https://goo.gl/utQ1KZ

Unable to install this module with corporate Artifactory or public npm registry.
Briefly describe the bug/feature request.
I used the suggested installation method:
yarn add @ampproject/rollup-plugin-closure-compiler --dev

And received:
error An unexpected error occurred: "https://registry.yarnpkg.com/@ampproject%2frollup-plugin-closure-compiler: User not found".

We'll likely need to know:

Your Rollup configuration.

Empty repo with a node_modules folder

Error logs from your console when invoking Rollup with this plugin present.

λ yarn add @ampproject/rollup-plugin-closure-compiler --dev

yarn add v1.7.0
warning package.json: No license field
info No lockfile found.
warning No license field
[1/4] Resolving packages...
error An unexpected error occurred: "https://registry.yarnpkg.com/@ampproject%2frollup-plugin-closure-compiler: User not found".
info If you think this is a bug, please open a bug report with the information provided in "/home/tbranyen/yarn-error.log".
info Visit https://yarnpkg.com/en/docs/cli/add for documentation about this command.

Very strange, am trying to reproduce here and cannot. The only thing I can come up with is a propagation delay for the publication of v 0.5.0.
Would you mind trying again?

Looks good now for me too, thanks! Must have been an npm hiccup.


Open Channel/React - Copy message

Needs to clarify the issue with files.

@james-a-rob @chrischabot This PR is ready for review


Change exception incorrect password

currently returning user not found

https://github.com/fredsonchaves07/foodfy/commit/d3f7615b5241613c6274a2e1e572945f1531750e


Fix issue inserting fulltext fields when all included field values are null

This PR fixes an issue causing invalid insert queries to be constructed for fulltext fields.
In the case of an insert of an entity with a fulltext field, an issue arose when all fields included in the fulltext field are null.  When walking the ast of the QueryFragments in an InsertQuery this led to an invalid query fragment, ()), being constructed  here.

Also, it looks like the formatting isn't up to date with the latest rustfmt.

@Jannis I added a simple test of inserting an entity with null values for each of the fields included in its fulltext index and I updated the formatting using the latest version of rustfmt.


Gavryushev gentx

Thanks for being interested in noble.  Please see this.
https://github.com/strangelove-ventures/noble-networks/tree/main/testnet/grand-1#permissioned-network


telemetry status updates

one telemetry_status publication per mavlink instance
each telemetry_status has an array of unique HEARTBEATS

This is useful to monitor what's going on with other mavlink components that are part of the same system (gimbals, etc). It's also more convenient for review to have a telemetry status per mavlink instance rather than mixing them on the same publication instance.

Note for later, let's review the size over time. It could be possible with mavlink routing to have more than 4 mavlink components (same system) talking to the same mavlink instance. Something like gimbal + VIO node + avoidance node logging + ?.


TACLS Configs - 4. Fix TACLS-Config-RealismOverhaul

Added missing versions v7.0.0, v8.3.0 to v8.4.0
Added a ksp_version_max to restrict installs in 1.2

Rebuild after #1259 and #1260

#rebuild


[BEAM-8805] Remove obsolete worker_threads experiment in tests

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Run Java PreCommit

Thanks for the clean-up.


Recursive show rules

This PR makes two changes to how guards work:

Every element in the output of a show rule is guarded against that rule (not just the input)
An Nth guard also guards against any higher show rules

The first one should be rather uncontroversial, while the second one brings some actual semantic differences with it.
Before, if you ran
#show heading: "a"
#show "a": "b"
= heading

it would give you b.
With this change, it instead gives a.
I think this makes it more intuitive, as the output of show rules is only styled with previous, not subsequent, show rules,
so show rules "bubble up" to the top.
We can also revert the second change if the new behaviour is considered bad, but that carries with it the cost of still allowing infinite mutual recursion to occur, i.e.
#show "a": "b"
#show "b": "a"
b

(which just gives b with the second change)

Personally, the second change doesn’t feel satisfactory to me. My intuition says that
#show heading: "a"
#show "a": "b"
= heading

should indeed show as b, while the following would show as a:
#show "a": "b"
#show heading: "a"
= heading

But I’m open to other people’s opinions on this.

This turned out to be (probably) unfixable.
I have a local prototype of a different approach, which also currently has a major issue, but I already have a fix for that in mind.
With that said, I'll close this for now and open a new PR with the new approach when I have time (probably in a few days).


Error with number field

Hi Norm
I am getting this error on a number field because it doesn't have a format.

KTL-0.28.6.js:1530 Uncaught TypeError: Cannot read properties of undefined (reading 'mark_decimal')
at Object.extractNumericValue (KTL-0.28.6.js:1530:44)
at Object.enforceNumeric (KTL-0.28.6.js:2631:66)
at HTMLDocument. (KTL-0.28.6.js:2343:24)
at HTMLDocument.dispatch (k_035ba839ee79fc1580af1ff0456be90131c144ea.js:2:199750)
at HTMLDocument.s (k_035ba839ee79fc1580af1ff0456be90131c144ea.js:2:195785)
Craig

Actually I think you have already fixed this in dev


Update read-key-value-pairs.mdx

Addresses https://github.com/cloudflare/cloudflare-docs/issues/18226


Sharing RequestPostProcessor across multiple Requests sharing same interface

I am trying to see if this is something that is possible using MediatR and if it fits within the paradigm. I have two different Requests with there own request handlers however they share the same interface. I would also like them to both share a RequestPostProcessor after they have been handled.
Example as shown below:
public class ActivateRental : IUserRequest
{
    public string UserId { get;set; }
}

public class UpdateUserAccess : IUserRequest
{
    public string UserId { get;set; }
}

public interface IUserRequest
{
    string UserId { get;set; }
}

Whilst both may have there own handlers for ActivateRental and UpdateUserAccess I am trying to do the post processing via a RequestPostProcessor that only expects a IUserRequest i.e.
public class UserAccessPostHandler : IRequestPostProcessor<IUserRequest , bool>
{
    public async Task Process(IUserRequest request, bool success)
    {
        // do something here
    }
}

Is this possible? Do I need to do something to hook this all up. Any information would be much appreciated.

What container are you using?
On Thu, Dec 7, 2017 at 8:57 AM Andrew Barker<EMAIL_ADDRESS>wrote:

I am trying to see if this is something that is possible using MediatR and
if it fits within the paradigm. I have two different Requests with there
own request handlers however they share the same interface. I would also
like them to both share a RequestPostProcessor after they have been handled.
Example as shown below:
public class ActivateRental : IUserRequest
{
public string UserId { get;set; }
}
public class UpdateUserAccess : IUserRequest
{
public string UserId { get;set; }
}
public interface IUserRequest
{
string UserId { get;set; }
}
Whilst both may have there own handlers for ActivateRental and
UpdateUserAccess I am trying to do the post processing via a
RequestPostProcessor that only expects a IUserRequest i.e.
public class UserAccessPostHandler : IRequestPostProcessor<IUserRequest , bool>
{
public async Task Process(IProductAccessCommand command, bool success)
{
// do something here
}
}
Is this possible? Do I need to do something to hook this all up. Any
information would be much appreciated.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/jbogard/MediatR/issues/217, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGYMly4alCsDGLhWNhsd7K1GYu8_-jrks5s-BjjgaJpZM4Q53jJ
.

Hello, sorry about ressurect this issue. How can i acomplish the same using MediatR.Extensions.Microsoft.DependencyInjection as container? Thank you.


Properly filter requirements for lambda layer

Please consider this PR. Originally part of #339 which was rejected because noDeploy behavior changed.
This part is still needed to make lambda layer behavior consistent with behavior when not using lambda layer.

Hey @ysgit  - it's been a long time since this PR was proposed. I'm going to close it, if you feel like the issue is valid, please open a new issue or a new PR against the latest main branch. Thanks 🙇


try to auto detect branch for queue build command

to resolve #92
Similar to the way how it is done for pull requests
Note that I am not sure if I could compile it correctly locally.

@bergmeister Thanks for the contribution!  Your change is now in the dev branch.


Create Basic Backend

We need to create a basic backend that our scorekeeper UI can push updates to and our scoreboard UI can read updates from. In terms of sample data, we can start up the app with some sample data (i.e. teamOne = "trash pandas", teamTwo = "mighty ducks", teamOneScore = 1, etc).
Once we can show that we're able to perform basic CRUD operations with this basic API, we can start pulling modifying the code to start from a database.

I'm calling dips on this. I'll take point on initializing the gofiber project. If anyone has thoughts about organization/implementation, please let me know.

I'm calling dibs on this. I'll take point on initializing the gofiber project. If anyone has thoughts about organization/implementation, please let me know.

I think I'm gonna revoke your dibs for now. I'm loving the enthusiasm, but we need to make sure that everyone gets a slice of 'za.

Sounds good to me.

I'm also happy to help on this one


any idea do a python version?

as title.

Please direct future questions to the dedicated forum.
Will never happen from me.


Get new Height and Width of Box on onResizeStop Event

How to get new height and width of Box on onResizeStop on event.
In my current project as soon as resizing is complete on box I want to send the new height and width of box to API of Map to bring the Map of that size.
How to get new height and width of box after resize is complete.

Got the answer I set layout in state of component in  onLayoutChange callback

That is exactly my need.
Would you mind sharing the corresponding code fragments ? I got the idea but wonder how to do it.


Add Croatian language translation

Pull Request Template for Home Assistant / Lovelace Card Repository
Overview
Add Croatian language translation
Type of Change

[ ] Bugfix
[ ] New feature
[x] Enhancement
[x] Documentation update
[ ] Refactoring
[ ] Other (please describe):

Checklist
Please ensure that you have completed the following tasks before submitting your pull request:

[x] I have read the contributing guidelines for this project.
[x] I have tested my changes against the dev branch (or another appropriate branch) and verified that they are working as expected.
[x] I have updated the documentation (if applicable) to reflect my changes.
[x] My changes adhere to the repository's code style guidelines.
[x] My changes do not introduce any breaking changes or unexpected behaviors.

Related Issues / Pull Requests
Please include any related issues or pull requests that this change is associated with, and use the appropriate keywords (Closes, Fixes, Resolves) to automatically close the related issues when this pull request is merged.

Closes #issue_number
Fixes #issue_number
Resolves #issue_number

Additional Details
Please provide any additional information that you feel is relevant to this pull request, such as screenshots, code snippets, or examples.

Done.

Dear @deadly667, just fyi, your Github is still not correctly configured to reference the used mail address.
Anyhow, this won't stop us. Thanks for your contribution!!


Can't change button text?

Hi, love your read-more solution but I can't seem to change the text values within the trigger.
I did the following:
    ReadMore.settings({
        moreLink: "Lees verder",
        lessLink: "Lees minder",
    });

    ReadMore.init();

What am I doing wrong?

This one works but I don't think this is ideal.
    ReadMore.settings = function(){
      return {
        content: document.querySelectorAll('.js-read-more'),
        originalContentArr: [],
        truncatedContentArr: [],
        moreLink: "Read More",
        lessLink: "Show less",
      }
    }
    ReadMore.init()

Took me a sec to get around to improving the api for this lib, but finally did.
Now, with v2 you can now pass in options, which include editing the read more / read less text.
import ReadSmore from 'read-smore'

const readMores = document.querySelectorAll('.js-read-smore')

const options = {
  blockClassName: 'read-more',
  moreText: '"Some custom read more',
  lessText: 'Some custom read less'
}

ReadSmore(readMores, options).init()

I documented this in the Readme and added a Docs/Demo page


Override ListViewItem Style

Hello,
I'm working on Overriding ListViewItem Style.
So I created a copy of ListViewItem Style with Blend.
I Successfully obtain a style.
I Updated the first line as above :
<Style x:Key="ListViewItemStyle1" TargetType="{x:Type ListViewItem}" BasedOn="{StaticResource MetroListViewItem}">
When I Apply this Style to en existing ListViewItem it is applied but with DataBinding Errors especially concerning Mah:ItemHelper
Example :  "(mah:ItemHelper.HoverSelectedBackgroundBrush)' property not found on 'object'"
Do you have an idea about why this Databinding does not occurs on mah;ItemHelper ?
Thank you

@RegisLaine It could be that Blend doesn't work well with custom helper classes on custom styles. But if you want to know the default style of MahApps for ListViewItem then you can take a look at the source here:
https://github.com/MahApps/MahApps.Metro/blob/0dffed2b9a98ed928d3ac0b5c8d5c7708f1aa413/src/MahApps.Metro/Styles/Controls.ListView.xaml#L147-L240


Bug fixes

Fixed directory not found error
Fixed applescript bugs
Fixed other minor bugs

sorry we have close this not becuase the conflicts but becuase we have fixed all that and are almost ready to fix the whole thing!

bye!


Added pack template test

Related to https://github.com/fsprojects/Paket/issues/1414#issuecomment-173169406
The test is creating a Nuget package and as you can see if you unzip it, it doesn't include the output assembly (my test isn't very accurate because it's checking the file size but you get the point).

can yuou please rebase / squash that. so that we loose the history of that packages thingy

I tried something, let me know if the history is better.

nope ;-) as you can see here it's now 2 commits more

https://github.com/ginatrapani/todo.txt-android/wiki/Squash-All-Commits-Related-to-a-Single-Issue-into-a-Single-Commit

I assume you forgot to force push.
If you pull again and push, then you merge the old stuff in

Sorry, my doppelganger took control of my account and kept adding commits. Fortunately I managed to fix things up :P

ha! look much better now :+1:

fixed in https://github.com/fsprojects/Paket/pull/1420
thanks for creating a repro case


project documentation should have description on how to report bugs

It looks like this is described here https://github.com/NLeSC/guide/blob/master/best_practices/documentation.md#how-to-file-a-bug-report