mdapi deploy report timeout error for in progress deployments

Summary
Since the sfdx v7.150.0, the sfdx force:mdapi:deploy:report --wait 1 command are failing with an error message when I use to query status of an in progress deployment.
'MetadataTransfer:Metadata API request failed: The client has timed out.'
The sfdx force:mdapi:legacy:deploy:report command is still working but it is deprecated and will be removed.
Steps To Reproduce:

Start a deploy which is running long time (we have 2000 tests, running ~3 hours):
$deployJob = sfdx force:mdapi:deploy --deploydir ./deploydir --ignorewarnings -w 0 --testlevel RunLocalTests -u $deployUser --json | ConvertFrom-Json
Query the status of the deployment:
$report = sfdx force:mdapi:deploy:report -i $deployJobId --wait 1 -u $deployUser --json 2>&1 | ConvertFrom-Json
(We are using this from Azure pipeline)

Expected result
status in json format
Actual result
MetadataTransfer:Metadata API request failed: The client has timed out.
System Information
PS > sfdx version --verbose --json
{
"cliVersion": "sfdx-cli/7.158.1",
"architecture": "win32-x64",
"nodeVersion": "node-v16.15.1",
"pluginVersions": [
"@oclif/plugin-autocomplete 0.3.0 (core)",
"@oclif/plugin-commands 1.3.0 (core)",
"@oclif/plugin-help 3.3.1 (core)",
"@oclif/plugin-not-found 1.2.6 (core)",
"@oclif/plugin-plugins 1.10.11 (core)",
"@oclif/plugin-update 1.5.0 (core)",
"@oclif/plugin-warn-if-update-available 1.7.3 (core)",
"@oclif/plugin-which 1.0.4 (core)",
"@salesforce/sfdx-plugin-lwc-test 1.0.0 (core)",
"alias 2.1.0 (core)",
"apex 0.13.0 (core)",
"auth 2.2.2 (core)",
"community 2.0.0 (core)",
"config 1.4.12 (core)",
"custom-metadata 2.0.0 (core)",
"data 2.0.4 (core)",
"generator 2.0.2 (core)",
"info 2.0.1 (core)",
"limits 2.0.1 (core)",
"org 2.0.1 (core)",
"salesforce-alm 54.6.2 (core)",
"schema 2.1.1 (core)",
"sfdx-cli 7.158.1 (core)",
"sfdx-git-delta 5.3.0",
"signups 1.2.0 (core)",
"source 2.0.3 (core)",
"telemetry 2.0.0 (core)",
"templates 55.0.0 (core)",
"trust 2.0.1 (core)",
"user 2.1.0 (core)"
],
"osVersion": "Windows_NT 10.0.19044"
}
Additional information
But it does not matter from where you start the deploy, when you use the 'sfdx force:mdapi:deploy:report -i $deployJobId --wait 1 -u $deployUser --json' on a deploy in progress, it returns 'MetadataTransfer:Metadata API request failed: The client has timed out.' error.
When you use this on a deployment is already finished or cancelled, it is working as expected.
Before the sfdx version 7.150.0 it was working, and the force:mdapi:legacy:deploy:report command still working.
Please fix it before removing the force:mdapi:legacy:deploy:report command.

Hello, is this happen frequently, can it be attributed to network issues ?

Hi, no, it is happening always.. and the force:mdapi:legacy:deploy:report is working always

has the deployment completed when you run sfdx force:mdapi:deploy:report --wait 1 ?  Or is it still pending/in progress? (you can check Deployment Status in an org's UI to see).
If yes, what happens if you put it to a larger number than 1, like (--wait 10)?  Does it still fail?

one more question: does the same happen if you use force:source:deploy:report with the same jobid on an in progress job ? If so, I'll make sure that gets fixed there, too.

I don't know, we don't use that command.

it's good, we'll fix them both to handle this timeout thing

@t-zakar - note that by providing -w 1 to the report command you are polling for the deploy status for a minute.  If you want an immediate response you could either leave off the wait param or specify -w 0 (which is the default when omitted).  Not sure if this fits with what your script does but thought I'd mention it in case it does.
Regardless, it doesn't match the behavior of the legacy command so it'll be fixed.

I am also facing the same issue and i am using the version sfdx-cli/7.160.0
Waiting for the upcoming fix

Hi,
Is this be fixed in the upcoming sfdx release?
Regards
Rohit
Get Outlook for iOShttps://aka.ms/o0ukef

From: Shane McLaughlin @.>
Sent: Thursday, July 28, 2022 5:57:39 PM
To: forcedotcom/cli @.>
Cc: RohitGagan16 @.>; Comment @.>
Subject: Re: [forcedotcom/cli] mdapi deploy report timeout error for in progress deployments (Issue #1612)
Closed #1612https://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fforcedotcom%2Fcli%2Fissues%2F1612&data=05|01||1787ee835fc740f6216808da70e430cf|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637946422631063607|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=50NdQVbOJAiDEuB2WD1cRq5%2FvEjvZBKW3XJCz84Swko%3D&reserved=0 as completed.
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fforcedotcom%2Fcli%2Fissues%2F1612%23event-7085302688&data=05|01||1787ee835fc740f6216808da70e430cf|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637946422631063607|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=5nwYrA7npi0PJAiSbcCYxU605hekrRJuKzhauYbINyQ%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAUX7WUPL75F7FGIPJR3IIO3VWL65HANCNFSM524PAPHA&data=05|01||1787ee835fc740f6216808da70e430cf|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637946422631063607|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=P8NMXoVWs6%2F4KQVFLbihIg3UG5jZA%2FtzVH9ooHBxBfY%3D&reserved=0.
You are receiving this because you commented.Message ID: @.***>

@mshanemc Could you please confirm if this is fixed in July 28 th rc release

@RohitGagan16 it is not is the 7/28 release.

With which release this be available?
Regards
Rohit
Get Outlook for iOShttps://aka.ms/o0ukef

From: peternhale @.>
Sent: Thursday, July 28, 2022 6:04:00 PM
To: forcedotcom/cli @.>
Cc: RohitGagan16 @.>; Mention @.>
Subject: Re: [forcedotcom/cli] mdapi deploy report timeout error for in progress deployments (Issue #1612)
@RohitGagan16https://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FRohitGagan16&data=05|01||e09f21c2f9d449eab97408da70e513b1|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637946426431983027|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=Gc1FbA1rjF1u6uOgOudcdKQIDg3qzNPJIUmh%2BZwVEuU%3D&reserved=0 it is not is the 7/28 release.
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fforcedotcom%2Fcli%2Fissues%2F1612%23issuecomment-1198675793&data=05|01||e09f21c2f9d449eab97408da70e513b1|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637946426431983027|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=JvPnVP65QluIPSpVGbIh1b%2Fn0ENRr7yeNj1mhm4AfIs%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAUX7WUINORJ6Q4FRD4ZUUU3VWL7VBANCNFSM524PAPHA&data=05|01||e09f21c2f9d449eab97408da70e513b1|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|637946426431983027|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=fGBedpu9c2Ip%2B1Xh4wCwESn1Qp4lWTp5WRlu4aZDhrg%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>

I have just tried the version 7.162.0, it is working fine, thanks.


Run missing gazelle update

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

Nice catch!
/approve


[bug] gui abnormal with optifine's fast render

Try turning bufferIngredientRenders off in the configs.

works. could pin this


[WIP] Lock commands

This pull-request is an incremental step towards the new lock related commands within the Git LFS client.
As an added benefit, these three new commands make extensive use of the new API client, so they serve as an example of its use.
Commands
git lfs locks [-i id] [-p path] [-c cursor] [-l limit]
The git lfs locks allows the client to list locks active on the server corresponding to the given filters, along with using the optional cursor and limit properties to paginate through a longer list of results.
git lfs lock <path>
The lock command allows the client to create a lock against a particular file by making the appropriate API call to the server.
git lfs unlock <path>
The unlock command allows a client to break a lock created by any user against any particular file in accordance with the original specification.
TODO
Still todo, are the following:

[ ] Implement the shell tests for these three new commands

[ ] Implement the LFS locking API within the test server

[ ] Make the command that take a <path> argument resolve to a path relative to the root of a repository, such that they are consistent on the server.

This looks great. A couple suggestions for the cli api:

Add optional --id param to unlock a lock according to its ID, and not the path.
Add optional --force param to force an unlock.
Add optional --remote param to all the lock commands to specify a different remote.

I'd recommend using the naming convention from git log to limit & skip in the results list:
-<number>, -n <number>, --max-count=<number>
       Limit the number of commits to output.
--skip=<number>
       Skip number commits before starting to show the commit output.

I just think it's better to use git conventions wherever possible, plus cursor is kind of an ambiguous term.

@sinbad:

to limit & skip in the results list:

Good call. @technoweenie and I were discussing this, and it would definitely be silly to make every user write their own script to traverse through multiple pages of locks. As a first pass, we thought it may be good to have the client perform this traversal, and then temporarily recommend that you call git lfs locks --limit=1000 | less. --skip would be challenging to implement in this way, since we would either have to teach the server how to skip results, or have the client load all locks and cut off the first n skipped. We may not need this feature, but I will think more about this nonetheless.

As a first pass, we thought it may be good to have the client perform this traversal, and then temporarily recommend that you call git lfs locks --limit=1000 | less. --skip would be challenging to implement in this way, since we would either have to teach the server how to skip results, or have the client load all locks and cut off the first n skipped.

Just a reminder that on windows a user might not have a shell that allows for the type of piping and filtering that you can do on a unix shell (sad, but true).
Is there a template already for the output that these commands will produce?

Hey @shana, sorry for ignoring your comment. Definitely should've replied before merging this :/

Is there a template already for the output that these commands will produce?

There's no "official" template other than printf calls like this. Are you asking because you're interested in integrating git lfs with some native git tool? We haven't thought about that much, and I'm personally open to whatever integrators think is best.
Would you prefer:

We add a --format flag with custom %arg values that integrators can use to customize output. This would protect you from changes to the command output.
How about we teach the command to optionally spit JSON out? Then a native tool could parse that, instead of relying on any text output.


⚠ Bofrost.es has degraded performance

In 309979a, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1593 ms

Resolved: Bofrost.es performance has improved in f4677be after 8 minutes.


Add imphash to meta_pe

This pull request adds the imphash value of a PE file to the PE metadata if the latest version of pefile is installed.

Thanks for your contribution and sorry for the delay in getting this merged.
As noted in the CONTRIBUTORS file, if you would like to attribute your work in the future please add a copyright notice to the file you're working in.


Bugfix/issue 1498 fix parsing issues

This fixes #1498. It only changes distributions.tex and make/manual so you can verify that it fixes the issues @bob-carpenter reported just by doing
make manual

rather than all the tests. It should go in for 2.7.0.

Looks good.


Flo by Moen Unresponsive in HomeKit

Describe The Bug:
 This plug is installed, and is viewable in HomeKit. However, nothing happens when device control is attempted.  For example, when I attempt to close the valve, it just says "stopping" and nothing happens.  This used to work, and now it doesn't
To Reproduce:
Attempt to use the functionality in HomeKit.  Fails every time, so easy to reproduce.
Expected behavior:
When I attempt to close the valve it should close.  it does not.  Also, the other control has Home, Away, and Off settings.  When I select any of those, it should do as requested.  It does not
Logs:
NINGservice restart command received
6/19/2024, 3:20:36 PMFlobymoen Bridge stopping
11/20/2024, 3:17:31 PMhub is running on port 80
11/20/2024, 3:17:32 PMFlobymoen Bridge starting
11/20/2024, 3:17:41 PMFlobymoen BridgeLoaded plugin 'homebridge-flobymoen'
11/20/2024, 3:17:41 PMFlobymoen BridgeLoading 1 platforms...
11/20/2024, 3:17:41 PMFlobymoen BridgeFlo-by-MoenStarting communication with Flo portal
11/20/2024, 3:17:41 PMFlobymoen BridgeBridge is running on port 51826.
11/20/2024, 3:17:41 PMFlobymoen BridgeFlo-by-MoenFlo Info: Using local cache Flo token.
11/20/2024, 3:17:41 PMFlobymoen BridgeFlo-by-MoenFlo Status: Refreshing Token...
11/20/2024, 3:17:43 PMFlobymoen BridgeFlo-by-MoenERRORFlo Login Error: Request failed with status code 400
11/20/2024, 3:17:43 PMFlobymoen BridgeFlo-by-MoenERRORPlug-in configuration error: Flo authentication error, please review your authentication information.'
11/20/2024, 3:31:35 PMFlobymoen BridgeConfiguration change
11/20/2024, 3:31:35 PMflobymoenbridge stopping
11/20/2024, 3:31:41 PMFlobymoen Bridge starting
11/20/2024, 3:31:51 PMFlobymoen BridgeLoaded plugin 'homebridge-flobymoen'
11/20/2024, 3:31:51 PMFlobymoen BridgeLoading 1 platforms...
11/20/2024, 3:31:51 PMFlobymoen BridgeFlo-by-MoenStarting communication with Flo portal
11/20/2024, 3:31:51 PMFlobymoen BridgeBridge is running on port 51826.
11/20/2024, 3:31:51 PMFlobymoen BridgeFlo-by-MoenFlo Info: Using local cache Flo token.
11/20/2024, 3:31:51 PMFlobymoen BridgeFlo-by-MoenFlo Status: Refreshing Token...
11/20/2024, 3:31:53 PMFlobymoen BridgeFlo-by-MoenERRORFlo login error: Request failed with status code 400
11/20/2024, 3:31:53 PMFlobymoen BridgeFlo-by-MoenERRORPlug-in configuration error: Flo authentication error, please review your authentication information.
Show the Homebridge logs here, remove any sensitive information.

**Plugin Config:**
{
    "platform": "Flo-by-Moen",
    "name": "Flo-by-Moen",
    "deviceRefresh": 15,
    "showTemperatureAndHumidity": true,
    "enableValveControl": true,
    "showHealthTestSwitch": false,
    "auth": {
        "username"<EMAIL_ADDRESS>        "password":
    }
}
```json
Show your Homebridge config.json here, remove any sensitive information.

Screenshots:

Environment:

Plugin Version:1.0.17
Homebridge Version: 1.7
Node.js Version: 16.20.2
NPM Version:  Cannot find this.  Version says 4.3.1.  Node version is 16.20.2
Operating System: MacOS

The log shows you have a login error.
11/20/2024, 3:31:53 PMFlobymoen BridgeFlo-by-MoenERRORFlo login error: Request failed with status code 400
11/20/2024, 3:31:53 PMFlobymoen BridgeFlo-by-MoenERRORPlug-in configuration error: Flo authentication error, please review your authentication information.
If you're able to log in to Moen Portal with your username and password and this was working previously.  It may be an OS and node issue.
If you using  MacOS 15.0 / Sequoia -- MacOS Sequoia introduces a new function to control what applications have access to the network. If you have problems with connections, make sure 'node' has access to the local network in macOS Settings > Privacy & Security > Local network (if there is no switch for 'node' try reinstalling node).

Thanks for the quick reply
I can confirm that I am able to log into the Moen portal with my user name and password.  Also, Node has access to the local network.  I'm running 15.1.1

Interesting...
Nothing has changed in the plug-in and both my implementations are functional on Linux or Docker even after multiple restarts.  My only other suggestion is to turn on debug and post output, maybe the detail logs can indicate what causing a communication error (400 indicates client error).

Debug is on.  Do you mean this output?
11/21/2024, 9:37:15 PMflobymoenbridge stopping
11/21/2024, 9:37:20 PMFlobymoen Bridge starting
11/21/2024, 9:37:34 PMFlobymoen BridgeLoaded plugin 'homebridge-flobymoen'
11/21/2024, 9:37:34 PMFlobymoen BridgeLoading 1 platforms...
11/21/2024, 9:37:34 PMFlobymoen BridgeFlo-by-MoenStarting communication with Flo portal
11/21/2024, 9:37:34 PMFlobymoen BridgeBridge is running on port 51826.
11/21/2024, 9:37:34 PMFlobymoen BridgeFlo-by-MoenFlo Info: Using local cache Flo token.
11/21/2024, 9:37:34 PMFlobymoen BridgeFlo-by-MoenFlo Status: Refreshing Token...
11/21/2024, 9:37:36 PMFlobymoen BridgeFlo-by-MoenERRORFlo login error: Request failed with status code 400
11/21/2024, 9:37:36 PMFlobymoen BridgeFlo-by-MoenERRORPlug-in configuration error: Flo authentication error, please review your authentication information.'

As you can see, there is no further info

That is unfortunate, I don't see anything that has changed and can't response your error. If indeed this was functional prior, I would review your configuration for anything that would have changed. I will be releasing a new version to handle a bug with Lux sensor and Health function but will not be touching code in regards to login. The only item that I will do is upgrade to new version of node library that handles communication (maybe) the new version of underlying library will help resolve the problem but I wouldn't put high hope there... since most correction were security fixes.

Thanks for trying.  It was for sure previously functional.  Truly unfortunate, since this was the sole reason I bought the Hoobs product.  How do I update the node library?

I have found instructions on how to update Node library.

Problem solved.  Your new logging had a new error message.  Invalid username/password.  So I checked, and found out that the Flo Password had been set to my Hoobs password.  Don't know how or when that happened.  Sorry for the hassle and all the help.

Awesome.


Use new OAV parameter definition

Follows changes in the OAVParameters in dodal.
Link to dodal PR (if required): #239
To test:

Check that the changes haven't affected the plans
Run tests

Yes, they are. It's what I was trying to fix yesterday with the update_on_zoom not being called - still working on it.
Sorry, forgot to set this back as draft in the meantime ...


Mention rules for using Godot classes in the readme

I've seen a lot of new users of the C++ bindings get stuck on unhelpful/non-obvious errors caused by using Godot classes like normal C++ classes. There are certain "rules" for how they should be used that aren't clearly mentioned in the readme or official docs. The examples follow them, of course, but it's not obvious from there that the "normal" C++ ways don't work.
I would suggest adding this section to the bottom of the readme:
Using Godot classes

Create instances that inherit Object (including your own registered classes) using _new(), not C++'s new operator.

Sprite *sprite = Sprite::_new();

Destroy nodes using queue_free(), not C++'s delete operator.

some_old_node->queue_free();

Wrap instances that inherit Reference in Ref instead of passing around raw pointers. They are ref-counted and do not need to be freed manually.

Ref<Texture> texture = resource_loader->load("res://icon.png");

Pass the core types that do not inherit Object by value. The containers (Array, Dictionary, PoolArrays) manage their own memory and do not need to be explicitly initialized.

Array ints;
ints.append(123);
return ints;

Initialize your classes in their _init() method, not their constructor. The constructor does not yet have access to the base class's methods.
Cast objects using Object::cast_to, not unsafe C-style casts or static_cast.

MeshInstance *m = Object::cast_to<MeshInstance>(get_node("ChildNode"));
// m will be null if it's not a MeshInstance
if (m) { ... }

Let me know if there are more non-obvious rules missing from the list, or if I can improve/condense any of the phrasing.

As a user who has struggled with GDNative, (SheepAndShepard has been a big help with using it), I wholeheartedly support having a list of "common pitfalls", including examples on the right way to go about things.
I myself don't have much C++ experience, so I can't really suggest much, but it would be nice if future users interested in GDNative C++ don't need to come on the Discord, and ask on there about every little thing, which is what I've had to do due to how lacking documentation is.

I'll add one more before I make a PR for this:

Never use Godot value types in static or global variables. These will crash because the Godot API is not loaded until after their constructors are called.

String s; // crashes
class SomeClass {
    static Dictionary d; // crashes
    static Node *singleton; // fine, pointer doesn't call a constructor
};


Allow zIndex on portal screen element

Version of https://github.com/herrethan/react-confetti-explosion/pull/14 as started by mikowl.

Ah I see, awesome, thanks!


Remove config directory

Is your feature request related to a problem? Please describe.
The config directory adds complexity despite the likelyhood of changes being slim.
Describe the solution you'd like
Remove the config directory. Required configurations can be made in the functions file with $app->config() like in lumen.
Additional context
Hard code the view and app configurations and remove the Auth driver for WordPress...

As the framework gets larger and caters for more "flavours", the configs will be more important. I agree that things such as the view path are unlikely to change between projects, but for example, the mail settings will need to be configurable between projects.


Clean up log output

Remove special characters from log traces or add the right characters to the font

This has been fixed.


[MNT] Added docker-pre-commit pre-commit hook

Closes #165

Checklist

[ ] PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF], [MODEL], [DOC]) (see https://neurobagel.org/contributing/pull_requests for more info)
[ ] PR links to GitHub issue with mention Closes #XXXX
[ ] Tests pass
[ ] Checks pass

For new features:

[ ] Tests have been added

For bug fixes:

[ ] There is at least one test that would fail under the original bug conditions.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
368

Relevant Lines:
368

💛  - Coveralls


⚠ Bofrost.de has degraded performance

In 6f0ce00, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1694 ms

Resolved: Bofrost.de performance has improved in 781ae93 after 6 minutes.


missing dst_crs options

In https://github.com/cogeotiff/rio-tiler/blob/82caa86c0739cbb87f56f5dfca8f12a3611397b1/rio_tiler/io/cogeo.py#L294-L302
we are not specifying the output CRS which will then be the internal CRS 🤦 instead we want it to be WebMercator!

done in #199


src/tpm2-provider-store-object.c seems to lack support for TPM2_HT_PCR

As I'm figuring out #5, I found myself in a situation that seems to need some separate attention.
This is my last log from running the test test/rsa_createak_sign_object.sh:
Starting /home/levitte/gitwrk/github.com/tpm2-software/tpm2-openssl/test/rsa_createak_sign_object.sh
+ echo -n abcde12345abcde12345
+ tpm2_createek -G rsa -c ek_rsa.ctx
+ tpm2_createak -C ek_rsa.ctx -G rsa -g sha256 -s rsassa -c ak_rsa.ctx
loaded-key:
  name: 000b34f913a0c15d656722906e4ca5670b93a1f76ea7683c3fb3177ab220955adb47
  qualified name: 000beb4a55c05455008997e8dbf51f1d89b04eb4744af41fa60af80e2da1c511aa2b
++ tpm2_evictcontrol -c ak_rsa.ctx -o ak_rsa.obj
++ cut -d ' ' -f 2
++ head -n 1
+ HANDLE=0x81000003
+ openssl pkeyutl -provider tpm2 -provider base -inkey object:ak_rsa.obj -sign -rawin -in testdata -out testdata.sig
Could not read private key from object:ak_rsa.obj
pkeyutl: Error initializing context
FAIL test/rsa_createak_sign_object.sh (exit status: 1)

Running the last command with gdb got me this far:
Breakpoint 1, tpm2_object_load (ctx=0x5555556b0140,
    object_cb=0x7ffff7d07cae <ossl_store_handle_load_result>,
    object_cbarg=0x7fffffffc3f0,
    pw_cb=0x7ffff7c77ffb <ossl_pw_passphrase_callback_dec>,
    pw_cbarg=0x5555556b0298)
    at ../tpm2-openssl/src/tpm2-provider-store-object.c:256
256	    TPM2_OBJECT_CTX *sctx = ctx;
(gdb) n
259	    int ret = 0;
(gdb)
262	    if (sctx->bio) {
(gdb)
266	        if ((buffer_size = read_until_eof(sctx->bio, &buffer)) < 0)
(gdb)
269	        r = Esys_TR_Deserialize(sctx->esys_ctx, buffer, buffer_size, &object);
(gdb) p buffer_size
$6 = 326
(gdb) p buffer
$7 = (uint8_t *) 0x5555556b02d0 "\201"
(gdb) n
270	        OPENSSL_free(buffer);
(gdb) p r
$8 = 0
(gdb) n
278	    TPM2_CHECK_RC(sctx->core, r, TPM2_ERR_CANNOT_LOAD_KEY, return 0);
(gdb) n
280	    if (sctx->has_pass) {
(gdb)
295	    UINT8 tag = (sctx->handle & TPM2_HR_RANGE_MASK) >> TPM2_HR_SHIFT;
(gdb)
296	    switch (tag) {
(gdb) p tag
$9 = 0 '\000'
(gdb) n
309	    return ret;
(gdb)

The switch that's skipped over is this:
https://github.com/tpm2-software/tpm2-openssl/blob/600308b126591b8b8b499333f295e7ad42fdaa47/src/tpm2-provider-store-object.c#L296-L307
I looked up what a zero tag could mean in /usr/include/tss2/tss2_tpm2_types.h, and here's what I found:
#define TPM2_HT_PCR            ((TPM2_HT) 0x00) /* PCR  consecutive numbers starting at 0 that reference the PCR registers. A platform-specific specification will set the minimum number of PCR and an implementation may have more. */
#define TPM2_HT_NV_INDEX       ((TPM2_HT) 0x01) /* NV Index  assigned by the caller */
...
#define TPM2_HT_TRANSIENT      ((TPM2_HT) 0x80) /* Transient Objects  assigned by the TPM when an object is loaded into transient object memory or when a persistent object is converted to a transient object */
#define TPM2_HT_PERSISTENT     ((TPM2_HT) 0x81) /* Persistent Objects  assigned by the TPM when a loaded transient object is made persistent */

Oops. That was a bug. Sorry for that. It's fixed now.


[Suggestion] - Allow Confluence Synchrony small CPU requests

Suggestion
My synchrony instances (3 pods) in my test environment are hardly working yet they still had a request of 1 cpu core.
When I try to set the cpu request to less than 1 core (for example 500m), The pods will not start since the container crashes immediately.
It seems like the 'entrypoint' configmap uses 'ActiveProcessorCount' with a floored value of my cpu request, and the default is 1 core.
I am unsure if this more of a Java issue with ActiveProcessorCount so please let me know if that is the case.
Thank you!
Product
Confluence
Code of Conduct

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

hi @Roia5 it does look like a Java issue. Do you have logs from the failed synchrony pod? In the meantime I'll experiment with cpu requests in my lab.

Released in 1.15.2


BC break in 2.1.1 - Attempted to load class "Key" from namespace "Firebase\JWT

Hi,
https://github.com/TheNetworg/oauth2-azure/pull/163 breaks compatibility with firebase/php-jwt versions earlier than 5.5.0. Firebase\JWT\Key class exists only in 5.5.0 and above and it conflicts with following composer dependencies:
"firebase/php-jwt": "~3.0||~4.0||~5.0||~6.0"
Thanks,
Kamil

Do you think it would be okay to drop support for 3.0, 4.0. and <5.5 version of php-jwt package?

I think it should be fine. Maybe it's worth to revert #163 and release it as 2.1.2 and then, create a tag 2.2.0 with "firebase/php-jwt": "~5.5||~6.0"?

It's not only #163
This commit (in 2.1.0) added JWK https://github.com/TheNetworg/oauth2-azure/commit/e4f597ef1933fa7d033939386c1077319a4518b1#diff-e71d1d223a40906fdb0c32e375c67398c6f658f4c95a71dad16b855004d91fa7R359
But such class was added in v5.2.0 https://github.com/firebase/php-jwt/commit/b0def5fca80851717920a3816b5c670e6182bc2f
I think such issues could be prevented by having a build that would check for highest and lowest dependencies and then run simple PHPStan with very low level that will resolve all the classes and make sure they exist and are compatible.

So 2.0.1 is the latest version supporting v4 of JWT.

I'll try to contribute Github Actions build that would catch this if I find the time.

I had to require firebase separately


fixing broken links that were caught in Simone's google SEO audit

Description
Simone send a list of broken links on dcos.io so that I could let her know where to redirect stuff. Instead, I'm just fixing the path links.
Decided to add relative links instead of static ones in case we ever move the docs site again. I think we had some discussion about this before, but I think this is the right way to go.
Urgency

[x] Blocker
[ ] High
[ ] Medium

Requirements

See the contribution guidelines.
Build content locally and test for formatting/links.

Just for the record, this is a great point. But there are a lot of /docs/ links on the website and I'd like to take care of them all at once with a separate PR.


Get the codebase up to latest rubocop formatting

https://jira.dev.bbc.co.uk/browse/CONNPOL-3644

Mix of automatic rubocop changes, and some manual interventions to standardize the code formatting.

👍


What you think to add touch control?

Yes, I am planning to add interactive to the gauge control. Just like the radial gauge in windows community toolkit.


Replace 'ERR_EXPLAIN' with 'ERR_FAIL_*_MSG' in "main" and "servers" directories

Part of #31207.

Could you add dots at the end of error messages, where missing? See https://github.com/godotengine/godot/issues/31207#issuecomment-519816715

Done and done!

Thanks!


Section duplicated with right prompt

Is your feature request related to a problem? Please describe.
When adding a section to right prompt, it will also show in the left prompt.
Found this issue in the fish version: https://github.com/matchai/spacefish/issues/129
Details could be found in the issue.
Is your feature request related to new section on prompt? Then consider the following questions:
no, should be related to all sections.

Will it clutter the prompt? yes.
Is it worth to be aware of it? yes.
Will it slow down the prompt? yes. some section would be rendered twice.

Describe the solution you'd like
Ignore the duplicated section in the left prompt.
Describe alternatives you've considered
update left prompt manually.
Documentation, adoption
no.
Maybe a screenshot?

And this is my config:
set -x SPACEFISH_TIME_SHOW true
# set or section order in right prompt
set -x SPACEFISH_RPROMPT_ORDER time

@salmanulfarzy why did you assigned proposal label? From my point of view, everything works as expected.
@Fity you enabled time section, which is disabled by default. Then you've added time section to the right prompt (but haven't removed from main (left) prompt). That's how you got time section duplicated.
Am I wrong?

Am I wrong?

No,This is definitely because of user configuration.
But to have time section on right prompt, User will have to specify complete SPACESHIP_PROMPT_ORDER with all the other sections to avoid duplicate section, because of the way it was disabled on left prompt.
May we could check whether section is specified on on right prompt before rendering left prompt or require user to specify both left and right prompt. AFAIK this affects only time section, right ? Just wanted to know if there anything I missed.

AFAIK this affects only time section, right ?

I would imagine that this would extend to any sections that are present by default. If you decide to have the package section on the right prompt, it should similarly not need to be in the left prompt anymore.

Bit of clarification,

AFAIK this affects only time section by default, right ?

With the current method, to have some section in right prompt we need user to have both right prompt and left prompt configured to avoid duplicates. I think that's an acceptable trade-off for customization, since it was users choice.


Add  with_graceful_shutdown() like axum::Server

Not really much to add to here.

Forgot to close this earlier, I can see axum-server has its own way of handling graceful shutdowns, so the feature is already there (just not as ergonomic maybe as axum).


All screenshots are black

Hi there,
I'm currently trying to set this up, and it seems to be working apart from the fact that all the screenshots I take are black. I'm wondering if this is something that you have come across before?
I'm running fedora 25.

I assume you're on GNOME, Wayland
Wayland is not supported due to it's permission setups which disallow screenshotting. Until wayland  is not a clustered mess I will not be able to do anything about it
Sorry :/

I assume you're on GNOME, Wayland

Aha, that explains it. I have switched to Xorg and it works.
Thanks for this. Maybe worth adding a "Doesn't work with wayland" somewhere in the readme.
Thanks for your work, this is a great product.

Thank you for reporting and feedback!


Proctored MCQ Week2 : Quiz3 and Create Task review

Grader: Nadira Haddach
Score: /4.5
GH Indi Page, Review Ticket, Crossover graded, Study Plan
Create Task and Create Task Crossover Review comments (completed by Thu)
Quiz3 MCQ results and test corrections
3 points, use bullets as indicated above, 1 point per sub-bullet
1 point, Participation: .20 for classroom presence and participation each day
.5 point, Quality of work: grader should be looking at effort toward learning and understanding vs perfunctory bullet chasing (0, .25, .5)
The last part of grading is providing point total, high points in crossover grading session, positives in work,  and suggestions for future.

Good work, all complete.
44/50 on exam score. Corrections complete. +1
Create task complete and submitted. +1
Study plan followed. +1
In class all week and participating. +1
Good quality of work, demonstrates an attempt at learning and understanding. +.5
4.5/4.5


Fail in VS2017 : TRK0005: Failed to locate: "CL.exe"

`Building the projects in this solution one at a time. To enable parallel build, please add the "/m" switch.
TRACKER : error TRK0005: Failed to locate: "CL.exe". The system cannot find the file specified. [C:\Users\himad\Documen
ts\Visual Studio 2017\Projects\BlankNode\BlankNode\node_modules\sharp\build\libvips-cpp.vcxproj]
gyp ERR! build error
gyp ERR! stack Error: C:\Program Files (x86)\MSBuild\14.0\bin\msbuild.exe failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\lib\build.js:285:23)
gyp ERR! stack     at emitTwo (events.js:106:13)
gyp ERR! stack     at ChildProcess.emit (events.js:194:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:215:12)
gyp ERR! System Windows_NT 10.0.15063
gyp ERR! command "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\himad\Documents\Visual Studio 2017\Projects\BlankNode\BlankNode\node_modules\sharp
gyp ERR! node -v v7.9.0
gyp ERR! node-gyp -v v3.6.0
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script 'node-gyp rebuild'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the sharp package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs sharp
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls sharp
npm ERR! There is likely additional logging output above.
npm ERR!      C:\Users\himad\AppData\Roaming\npm-cache_logs\2017-04-16T15_03_11_790Z-debug.log`

Hello, did you see #341

@lovell two different machine running in same configration

It looks like you're using node-gyp v3.6.0, which added VS2017 support, so that's a good start.
KeyError: '2017' suggests node-gyp can't detect the version of Visual Studio. Perhaps try forcing it using:
npm config set msvs_version 2017

@lovell i did every thing because i run the npm config set msvs_version 2017 command thats why is showing KeyError: '2017' if i remove it it will show another error
MSBUILD : error MSB3428: Could not load the Visual C++ component "VCBuild.exe". To fix this, 1) install the .NET Framew ork 2.0 SDK, 2) install Microsoft Visual Studio 2005 or 3) add the location of the component to the system path if it i s installed elsewhere.  [C:\Users\himad\Desktop\testProject\node_modules\sharp\build\binding.sln]
and if i run it with npm config set msvs_version 2015 if will show a another error
error MSB4019: The imported proj ect "C:\Microsoft.Cpp.Default.props" was not found. Confirm that the path in the <Import> declaration is correct, and t hat the file exists on disk.
i install all the component needed to run like here it says
https://github.com/nodejs/node-gyp/issues/1144
but still not working in this pc

Thanks for the update. Given it works on one machine but not another, this is unlikely to be a problem with sharp. I can see you're commenting on the linked node-gyp issue so I'll close this.

Bug is in<EMAIL_ADDRESS>incompatible with super new SDKs 10.0.15063.Desktop
See edit in https://github.com/nodejs/node-gyp/issues/1144#issuecomment-290253041

npm i node-gyp --global
Fixed the issue for me by installing the latest node-gyp version (5.0.4).


It looks like Flame'd logo is giving me the finger

:smile:

I actually like both, @erickzanardo . A nice small square image as logo has some good usages, for example, our favicon. Is it easy to randomize the flame a little bit to make it not look like a hand? (if it even does, I can see the point but it does require some imagination haha).

I agree that it does require a bit of imagination but it's the first thing that comes into my mind when I look at the logo and I can't unsee it. The same thing happened with Resident Evil 6's logo :smile:

gosh now it's in my head too lol

I am gonna close this, it does require a good amount of imagination to see that, and this issue open could influence people the wrong way. We will nevertheless try to change the logo a little bit.

Updated the logo.


Swift 2.3 support

I've made a branch to support Swift 2.3.  The changes were pretty minor.  You may want to take a look at the UIImage+Haneke changes, as they involve some force unwrapping, and I wasn't sure how you thought this should be handled (might require changing the method signature).
I'm guess that you will not want to master, but if you would like to create a swift-2.3 branch, I can close this and submit a new PR pointing there.
This should address #331.

Targeting swift 3 now

That's great, but I think it's a mistake to skip 2.3. A lot of large swift apps (several of ours, for example) have not been able to update all the way to 3.0 yet, and the changes for swift 2.3 are very manageable. IMHO that's something the library needs to support.

+1 for Swift2.3 support.
Simply add a separate branch at 2.3 that contains everything up to the last 2.2 commit and don't add any more features to 2.3. It'll incentivize many people to move to Swift 3 for new features. As @chadmoone said, bigger companies such as ours, cannot move to Swift 3, but we depend on OSS libraries such as this one to get the job done.
Will you reconsider?

For reference on an alternative approach, Alamofire used a minor version to address 2.3 support, while using the major version to address 3.0 support: https://github.com/Alamofire/Alamofire/releases/tag/3.5.0

+1 for Swift 2.3. This is the last library we rely on that's not 2.3 compatible. I'm going to move us to Swift 3.0 in the near-ish future, but not being able to use Xcode 8 right now is pretty annoying.
That said, a fork with any of these 4 2.3 PRs isn't the worst thing until we're ready to go to 3.0.

+1 for Swift2.3 support.
If you are using Alamofire as a network library and still have to support iOS 8, you can't just hop onto the Swift 3.0 train. Therefor it is very important to support both flavours of Swift 2.3 & 3.0. It is really sad to see that this fact gets neglected by so many people.

https://github.com/Haneke/HanekeSwift/pull/350


[DB-DESIGN] Database design needs to be more flexible to add new sites support and multiple user handles support

Current Database design has two blockers for site extensibility.

Every "new site support addition" needs new columns to be added to USER database table (NEWSITE_handle and NEWSITE_lr) for adding user handle and last retrieval time.
Many a times a user ends up having more than one handle on online sites (e.g. During college someone created an id using college roll number as user handle). But after graduating, user created another handle.   As a user if someone wants his progress from two accounts to be merged, he needs to create two separate stopstalk accounts.  A re-design can address both scenarios to make site more elegant.

CURRENT DESIGN
user
codeforces_handle
codeforces_lr
codechec_handle
codeforces_lr
spoj_handle
spoj_lr
......
PROPOSED DESIGN
Add a new table
coding_profiles
user_id : (reference field to user table)
site_name: (string: one of -- CODEFORCES, CODECHEF, SPOJ, ...)
site_handle:  user's handle of site
last_retrieved: DateTime of last retrieval
This will allow adding new sites support easy and elegant and allows a user to add his multiple handles into his single account.
Lastly,  this also allows allows LOSE COUPLING between crawlers and user table or sites. e.g. a Crawler can be written to just fetch one site specific handles and crawl just for that rather than crawling for all user handles of a single user at once.

That's a great idea @sandywadhwa :) Would be the biggest change in terms of code to get this up and running, not sure if can be picked right away.

This can be done in backward compatible way with low cost of code churn or codebreak (by having redundant data for some time).

A new table for coding profiles can be added.
A db script can be written to create entries for every existing user profile (from user table to to coding profiles table).
User registration code and Update Details code paths can be enhanced to make updates/entries into new table along with old table.
APIs to send user profile details to front end can be enhanced to read from new table and sent to front end.
Once changes are thoroughly tested a switch can be made from old code table and apis to new table and apis.

This should reduce cost of changes and any regressions.  While providing flexibility to anytime switch back to old code paths.

This can be done in backward compatible way with low cost of code churn or codebreak (by having redundant data for some time).

A new table for coding profiles can be added.
A db script can be written to create entries for every existing user profile (from user table to to coding profiles table).
User registration code and Update Details code paths can be enhanced to make updates/entries into new table along with old table.
APIs to send user profile details to front end can be enhanced to read from new table and sent to front end.
Once changes are thoroughly tested a switch can be made from old code table and apis to new table and apis.

This should reduce cost of changes and any regressions.  While providing flexibility to anytime switch back to old code paths.

Yep, that's the release plan if going with it - the issue is urgency/bandwidth not feasibility.
Feel free to raise a PR :)


Wifi disconnect on wake from sleep?

Intermittently my HP chromebook 14 (2015 model -- celeron n2940) will not reconnect to wifi networks on wake from sleep and must be rebooted.  I've filed a [fairly detailed bug] (https://code.google.com/p/chromium/issues/detail?id=572592) with the Chromium people, who think this may be a hardware-related PCIe problem.  But I have this nagging worry that this happens only when I'm in a crouton chroot.  Of course I'm often in a chroot -- not always -- so the correlation might be spurious.
Has anyone else observed this?  Is there any reason that crouton could mess with the hardware on wake from sleep?

@free-variation,
I'm not sure whether it related to the kernel-space driver of wifi chip or not, at least as of my ASUS C200 (BayTrail-M, AC7260), sometimes wireless networking was failed on resume after updating ChromeOS system/kernel...
In my case, the 'iwlwifi' module might have an issue of power-saving feature in some kernel release cycles, so I had to disable power-saving feature in that time from chroot:
iw dev wlan0 set power_save off
This command might be able to avoid the power cut in suspend/resume cycles of ChromeOS... But surely this causes bad power-consumption at idle as side effect.
Regards.


Support Functions client development with Storage Emulator

This came up on stackoverflow.com/questions/44049210/support-for-azure-functions-offline. We did some early work to make the SDK mostly work with the emulator, but it's not clear if that fully translates to using it with our Functions client tooling experience (VS, CLI).
First step would be to try it and see if it blows up.

https://github.com/Azure/azure-functions-cli/issues/6 touches on the topic a bit, but mostly as a side thing.


duplicate entry: chat/rocket/android/widget/helper/AvatarHelper.class

Hello Devs.
Compilation currently fails with:
FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':rocket-chat-android-widgets:transformClassesAndResourcesWithPrepareIntermediateJarsForDebug'.
> java.lang.RuntimeException: java.util.zip.ZipException: duplicate entry: chat/rocket/android/widget/helper/AvatarHelper.class

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.

* Get more help at https://help.gradle.org

BUILD FAILED in 7s
94 actionable tasks: 5 executed, 89 up-to-date

Maybe a known issue?

After an environment reset everything went well. Sorry for the noise.


Add configuration settings for Chicken Breeder and Roost speeds

I found myself wanting to adjust the speed of Chicken Breeders for a modpack I'm working on, so created a configuration file for adjusting it.  (And also Roost speed, because it was easy.)  Both settings are floats which multiply the speed of the block (i.e. divide the time until next drop) and they can also be adjusted in the Mod Options GUI.

Looks great, thanks!
I merged into https://github.com/timrwood/roost/tree/dev/1.3.0 and renamed speedRoost to roostSpeed so that it will group with other roost config values if they are added in the future.

Hi, just did a build of the latest 1.3.0 branch and it fails when starting a server with this build. The configuration is causing the server to fail. The client seems to work fine with this version of the mod.
I built an earlier version of the 1.3.0 branch where you marked it as version bump, and it was working fine for both server and client.

@Bletch1971, I made some changes to the way the config is generated using config annotations in https://github.com/timrwood/roost/commit/67b3c98363660e9c03acbe162b4fcb27a88a009e, can you see if the server is failing with those changes? I made that change a week or so ago, but forgot to push the changes until now.

@timrwood, thank you for the update. I have downloaded the latest code from the 1.3.0 branch and I can confirm the configuration changes you have made are working on both my server and client.
I have also tested increasing the rates of the roost and breeder and they are working as expected.
NOTE: I changed the config values on the server from 1.0 to 10.0 for both the roost and breeder. BUT I left the client config values as the defaults (1.0). When I logged into the server, the higher rates were in effect (10.0).
I was not sure if the client would take the server or the client settings, but was pleased that I did not have to set the client config as well. And that the server config does override the client config.
Thank you for all the hard work. I look forward to the official 1.3.0 release and not using my janky 1.3.0 version :)

I added an alpha release file to CurseForge, if everything looks good, I'll promote it to the official 1.3.0 release.
https://minecraft.curseforge.com/projects/roost/files/2635449

@timrwood, you added a feature to show when the breeder is out of seeds.
Would it be possible to add a feature to show if the roost or breeder is full?


Page navigation timeout is not parametrized

Hello there :wave:
thank you for this cool package in advance, i'm using it to access information from sites which do not offer APIs and it is a very helpful tool.
On of the things i require to be able is to set the timeout when calling \HeadlessChromium\Page::navigate.
Currently there is no parameter, therefor \HeadlessChromium\Communication\Connection::$sendSyncDefaultTimeout will be used which can't be set and is hardcoded to 3 seconds. When downloading files from some pages the download might take longer than 3 seconds (because the file is big/needs to be generated by the server first) and $page->navigate($downloadUrl); will throw an exception.
I will provide a non-breaking pull request shortly and i'd be glad if you could merge it into your awesome lib :slightly_smiling_face:

Hi @gsouf
i'm passing the timeout to
Session::sendMessageSync which passes it to SessionResponseReader::waitForResponse and is used instead of the default timout defined in Connection::$sendSyncDefaultTimeout
Please see https://github.com/chrome-php/headless-chromium-php/blob/7c227270f76d8605b585e212f4954e5c7e428afd/src/Communication/Session.php#L78
for the exact location where the provided timout will override the default value.
Prior to the change there is no possibility to define a timout for $page->navigate(), because it is neither parametrized nor configurable in the session.
I am currently running the app with my changes and it solves the problem :slightly_smiling_face:

Hi @vertexvaar
Ok, so if it fix I will provide a different way to fix it. The reason is that it makes it complexe to have to deal with timeout in both of navigate and waitForNavigation methods.

As of version 0.2.0 the timeout passed in $page->navigate($url)->waitForResponse($timeout) is global.
@vertexvaar feel free to give a feedback in case it does not fix your issue.


CLI typo

Describe the bug
Found a little typo! "Deplyment" --> "Deployment"
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

fixed.


Explore general-purpose use cases

In offline discussion with @dgasmith, it sounds like a large portion of this code is fairly general and abstracts over several task-worker frameworks (e.g. Dask, Parsl, Fireworks, etc.).
Would it make sense to explore making portions of this package more general-purpose than quantum chemistry? I have use cases in mind like performing small analysis tasks on many trajectory files with freud. The granularity of these small tasks would be well-suited to this package's task execution interface, I think.
If this makes sense, perhaps the task layer could be branded as Fractal and the quantum chemistry applications built on Fractal could be branded as QCFractal.
I currently use the signac data management framework for my research, and it may make sense to utilize the task-worker components as an execution engine that signac-flow could use for processing its workflows.

I think this would be great to do as well. Can you help direct anyone else who has similar questions or thoughts to this thread so that we can properly assess the parts that we should move out, the requirements of this operation, and gauge the overall interest if we did pull out the central queue?

I would love to see this moved out into its own project. I am interested in developing tools that, ideally, would be scalable efficiently on arbitrary resources, from single machines to multi-cluster.
I am also interested in building a central repository for MD data that will eventually have a machine learning application. It would be immensely useful to me if the software I'm building were to automatically deposit relevant information on a central database as it was running, so that after (hopefully) many different users have applied it to their own use-cases, we would begin to build up a standardized library of diverse training data. Is QCFractal the appropriate tool with which to implement this sort of thing?

I'm tagging a few folks that would benefit from this:

@SimonBoothroyd has built a distributed property estimation framework for assessing and optimizing molecular mechanics force fields, and could benefit from the distributed execution capabilities of the QCFractal engine across different sites with GPUs
@jaimergp and @dominicrufa are working on distributed computing alchemical free energy calculations, and could potentially benefit from distributing those calculations across GPU compute resources at different sites

Excellent! @SimonBoothroyd @jaimergp can you both answer a few questions:

For the control and generation of new tasks (e.g., python functions) what workflow tools do you use?
How much data needs to be moved on and off compute clusters per task (Ranges like 1e3 - 5 kB are useful).
Approximately how long is each task and on approximately how much resources?
Do you require storage of all task input and output?


Update spanner docs to show correct 'Batch.delete' usage.

Closes #3773.

Docs build fixed / merged locally in 19e9579b11d128aaab95c6e04f1abf7fb5d6d581.


core: enable serial executeCommandinHostOS

enable tests to use serial as a comms channel for executeCommandInHostOS() instead of ssh.
Refer to this PR for motivations - and notes on the worker side implementation: https://github.com/balena-os/leviathan-worker/pull/63
Change-type: patch

waiting for worker to be bumped here:
https://github.com/balena-os/leviathan-worker/pull/63


npm warning

Ref:
https://docs.microsoft.com/en-us/aspnet/core/signalr/get-started?view=aspnetcore-2.1&tabs=visual-studio
I understand the author is updating the article so might like to address the following WARN when running npm
PM> npm install @aspnet/signalr<EMAIL_ADDRESS>C:\Users\Will\Source\Repos\Spikes\SignalRChat\SignalRChat --<EMAIL_ADDRESS>npm : npm WARN<EMAIL_ADDRESS>No description
At line:1 char:1

npm install @aspnet/signalr

  + CategoryInfo          : NotSpecified: (npm WARN Signal... No description:String) [], RemoteException
  + FullyQualifiedErrorId : NativeCommandError

npm
WARN<EMAIL_ADDRESS>No repository field.`

@ImageQC @rachelappel After running npm init -y, setting the private property to true in the package.json file will eliminate the warnings. Most projects aren't intended for publication in the npm registry, so this property is probably a good thing to add.

@scottaddie @ImageQC  npm is outside the scope of the SignalR docset so the instructions belong in npm docs, not these docs.
The property can go into the sample though.


Cleaned phantomjs references

PR for #65

Coverage remained the same at 79.277% when pulling 57fe055bce86b1c41bc2e3d5532f03ca29a25382 on 65 into 9c446fa238b8d8f967f20ab6e0f4c2a0e8be4970 on master.


Migrated ruby-runtime plugin to Documentation as code

Migrated all docs to github
Links
https://github.com/jenkinsci/ruby-runtime-plugin/pull/7
Screenshots
Please add screenshots if relevant.
If they are already linked from the pull request, no need to duplicate

Thanks for your contribution! I believe this work has started before the hackfest timeframe was over, so I will integrate this issue. Will try to help to get it landed


Kured logs which indicate that node is rebooted

I am new to kured and was just curious that which kured log exactly say that particular node is successfully rebooted.
Following are logs from kured:
time="2020-06-21T03:00:09Z" level=info msg="Kubernetes Reboot Daemon: 1.3.0" time="2020-06-21T03:00:09Z" level=info msg="Node ID: aks-node-188893-vmss1" time="2020-06-21T03:00:09Z" level=info msg="Lock Annotation: kured/kured:weave.works/kured-node-lock" time="2020-06-21T03:00:09Z" level=info msg="Reboot Sentinel: /var/run/reboot-required every 1h0m0s" time="2020-06-21T03:00:09Z" level=info msg="Blocking Pod Selectors: []" time="2020-06-21T03:00:09Z" level=info msg="Reboot on: Sun------------------ between 02:00 and 08:00 UTC" time="2020-06-21T03:00:09Z" level=info msg="Holding lock" time="2020-06-21T03:00:09Z" level=info msg="Uncordoning node aks-node-188893-vmss1" time="2020-06-21T03:00:10Z" level=info msg="node/aks-node-188893-vmss1 uncordoned" cmd=/usr/bin/kubectl std=out time="2020-06-21T03:00:10Z" level=info msg="Releasing lock" time="2020-06-21T03:31:03Z" level=info msg="Reboot not required"
Does these logs means kured has rebooted nodes, or there must be another logs such as msg="rebooted completed"?
Any suggestion would be greatly appreciated.

No reboot was done in that log.
If a reboot was triggered, you would have more details in your logs, like Commanding reboot for node:

Do you think we need more documentation?

When I tested a reboot for #210 I did not get the "Commanding reboot" line fed through to my log store.
I haven't investigated, but it does seem a bit chancy whether something will make it to disk in the context of a reboot.
Maybe kured should wait a short time between logging and rebooting to improve its chances?

That's weird, because the reboot command should try to sync all writes before effectively rebooting. I checked on my environment, and I indeed see the proper lines.
time="2020-10-29T12:47:32Z" level=info msg="Reboot not required"
time="2020-10-29T12:48:32Z" level=info msg="Reboot required"
time="2020-10-29T12:48:32Z" level=info msg="Acquired reboot lock"
time="2020-10-29T12:48:32Z" level=info msg="Draining node kind-control-plane2"
WARNING: ignoring DaemonSet-managed Pods: kube-system/kindnet-p45nd, kube-system/kube-proxy-shphl, kube-system/kured-tfj9d
time="2020-10-29T12:48:32Z" level=info msg="Commanding reboot for node: kind-control-plane2"

It's maybe a good idea to introduce a timer, but I don't think it's necessary.

It might make it to disk, but when the container gets deleted on shutdown and the pod restarted after reboot I'm not enormously surprised one line can get lost.  Maybe I should have a better scheme to collect the logs (currently fluentd reading the files).

It might make it to disk, but when the container gets deleted on shutdown and the pod restarted after reboot I'm not enormously surprised one line can get lost.

Hehe,  me neither :)


Clients wants to enter exact decimal value (exp 1.011) in vouchers entry

Clients want decimal points entry in vouchers.(CA Amit Jain)

Not Required


feat: (IAC-895) Add Support to Toggle the Google Cloud Managed Service for Prometheus

Changes
After March 15, 2023 all newly created Google Kubernetes Engine standard clusters will have Managed Service for Prometheus in-cluster components deployed by default.
This default enabled feature from Google does add a small amount of cost. See https://cloud.google.com/stackdriver/pricing#mgd-prometheus-pricing-summary
In order to support users who do want this service in their cluster by default we are adding the enable_managed_prometheus to control the deployment of the  "Managed Service for Prometheus" in their clusters. By default we are setting this value to false.
Note: Google has still not enabled this feature by default, but adding the flag as a feature in case users want to enable this themselves.
Tests

Scenario
Provider
Kubernetes Version
Deployment Method
enable_managed_prometheus
Notes

1
GCP
1.26 (v1.26.4-gke.500)
Docker
true
GMP was present in the cluster

2
GCP
1.26 (v1.26.4-gke.500
Docker
unset (default false)
GMP was NOT present in the cluster

Added documentation for the previously undocumented gke_monitoring_* flags and added ther now required gke_monitoring_enabled_components which I am working on testing. I'll requests reviews after testing is complete.


Update K8s helm chart documentation

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6206/
Test PASSed.

Merged build finished. Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6207/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6208/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6209/
Test FAILed.

jenkins, test this please.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6210/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6211/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6212/
Test PASSed.

alluxio-bot, merge this please.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/6213/
Test PASSed.

Merged build finished. Test PASSed.


Suggestion:  Generate a "class" rather than "module + functions" to allow overriding functionality

If the generated code created a base "class" then users would be able to extend and override the code easily.
One of my main pain points with emscripten generated JS was that while it had lots of options that you could tweak and the ability to pre/post append JS, there were always parts of the wrapper that you couldn't get at.
Also, all those options ended up bloating the generated code.
IMO the class abstraction is ideal for this scenario...

@GordonSmith can you clarify what sorts of extensions you are looking to make here? It would help to understand what kinds of interceptions might be useful, above and beyond the current --instantiation output mode.

Disclaimer 1 - I haven't attempted to integrate any of the (new to me) component, wit, wasi-sdk toolchains into a "product", so my comments are just observations on the "hello world" type generated code.  Also as I highlighted above, they are more about certain frustrations with the emscripten generated "module" JS, that I would not like to see repeated here!
Disclaimer 2 - I am not interested in any class v functional debates, IMO they are just different abstractions with their own pros and cons, if anything my mantra is always "good encapsulation" which in this context is the ability to replace well encapsulated "bits" of the generated code.
So with that in mind when I look at the generated code my initial observations are:

The "loader" code (the bit which fetches the wasm), is catering for two separate environments (NodeJS and Browsers) and is also making assumptions about the location of the wasm file.  This has historically been a pain for me (as a JS library writer) as I won't be able to bundle that code cleanly - I like to create a single JS file which "just works" in NodeJS, Browsers (inline or as web workers) AND that my customers can bundle in their web applications without having to think about it.  FWIW I solved this issue by compressing the wasm using zstd and encoding it as Base91 so it can be stored in a JS file as a string, with a helper function that does the decoding and decompressing - so I would need the ability to exclude your loader code and be able to pass in a "pre-loaded" wasm file (and have no reference to any NodeJS/Deno libraries which confuse my bundlers and my customers bundlers).
The helper functions are going to get duplicated when working with more than one wasm component
The use of TextEncode/TextDecoder might be problematic in some environments (does Deno support them?  Does wasmr + embedded JS support them?), While I suspect the answer might be yes in those examples I already have my own encoder and decoder that I might prefer to use.
The shared memory is not accessible (in my toy example anyway), I will want to be able to allocate memory on demand and be able to read / write to it from the JS side (image blobs, or parquet blobs etc.)

The code being generated is super clean and I would hate to see it getting a lot of if (opt.nodejs === true) or if (opt.utf8encode !== undefined) type statements.
Classes could solve these issues, so could well typed functions in separate files...


Fix typing error on the home page

Fixing typo and testing review rule

Admin self-approval by me! :)


Repo headline & gem description claim to test query parameters (it doesn't)

This repo bills itself as:

Tests your rails API against its Swagger description of end-points, models, and query parameters.

But in the README, it says:

Parameters in the query string are not validated or processed by Apivore in any way.

which is actually the behavior.
I would find it very useful to verify the query parameters, but in the meantime, the repo headline shouldn't claim it does something it doesn't.

Swinging by to say: yep, I read the description and was very confused to then read the disclaimer.
This looks like a great tool, thanks for releasing it, but its use is diminished greatly by not caring about query string parameters. I'm mostly using JSON Schema as a basis for my OpenAPI docs (yeah i know they're a bit different, long story) and I can use thoughtbot/json_matchers to confirm the responses confirm.
I was hoping Apivore would be a ruby-based in-RSpec version of Dredd, which takes example values from the query string params listed in the spec. This is amazing as it confirms that your documented query string param values are actually valid, (or just proves that your API accepts whatever you documented without erring at least 😓).
If Apivore is ignoring query string parameters then it's just a replacement for JSON Matchers, which is ok, but means you should probably change the description a bit.


How to use custom queries

PGSync version:   2.1.1
Postgres version:  12
Elasticsearch version: elasticsearch-7.13.2
Redis version:
Python version: 3.7
Problem Description:
                         **How sync data through custom conditional queries like using where clause?**

Error Message (if any):

This seems like a very reasonable and typical use case and I considered this for some time.
I couldn't think of an easy way for specifying generic queries in a
JSON config without re-implementing what Elasticsearch supports in some ways
or a simplified/intuitive language for declaring this in the schema config.
This would also increase the complexity from a usability point of view.
I think you can move this type of functionality into Elasticsearch itself.

PGSync has support for pipelines and this might be the best option.
You can also use PGSync's support for plugins to perform more advanced transformation operations to achieve this

closing as resolved.
WHERE queries are best implemented as plugins for better generalization


Update netlify.toml

Netlify Preview URL: Preview the deployment here
If the preview doesn't load correctly, please check the build logs for more information.


Add project avatars

It could be nice to display some visuals for projects in the application, rather than only data.
It would look like more livery, more "human".
We could display the owner avatar from Github:

for big projects like React or Bootstrap, it would be the organisation avatar
for smaller projects, it would be the main contributor avatar

First iteration
Implementation

Grab project owner id from Github API
Display the avatar inside project cards using the following component

const Avatar = ({ project, size = 100 }) => (
  <img
    src={`https://avatars.githubusercontent.com/u/${project.owner_id}?v=3&s=${size}`}
    width={size} height={size}
    alt={project.name}
  />
)

Top page

Tag page

For some projects, the owner avatar is not really appropriate, a real logo would be better.
For example:
React, Redux and Awesome have their own logo, that is different from the repository owner avatar.
We could use SVG logos from http://instantlogosearch.com/

For some popular projects (React, Redux, You don't know JS), a custom svg file, stored in the database is used instead of the repository owner avatar.


Document Using Descheudler As a Remedy System

In addition to using draino as a remedy system the k8s descheduler can
also be used as a remedy system.

/lgtm
/approval

/approve


Challenge http://www.freecodecamp.com/challenges/waypoint-build-web-apps-with-express.js has an issue

The steps to complete are:
Complete "Hello World"
Complete "Juggling Async"
Complete "Time Server"
Complete "HTTP Collect"
Complete "Juggling Async"
Complete "Time Server"
I think these are wrong.  Shouldn't it be

Thanks, commit 268f58505075225d4d2953ccd1ebaf132ddd99c8 closes this.


Component search issue fix.

Issue:-If palette search textBox is empty while doing search then palette drawers are displayed in expanded state.
Fixes:-
Set PaletteDrawer.INITIAL_STATE_OPEN when search textBox is not empty so palette drawers should display in expanded state.
Set PaletteDrawer.INITIAL_STATE_CLOSED when textBox is empty so palette drawers should display in closed state.


cue/scanner: add raw string support

Swift raw strings seem to be the nicest design for raw strings out there.
Benefits:

just a generalization of normal strings, including multi-line
allow string interpolation within raw strings
frees up back quotes for, for instance, identifier literals (with non-standard characters).
extends to both string and byte literals
can still use escape sequences within the raw strings
but typically does't require escaping any.

#"This is \ a raw string"#
#"Raw string with interpolation \#(interpolated)."#
##"Raw string without interpolation \#(interpolated)."##

etc.

implemented.

This issue has been migrated to https://github.com/cue-lang/cue/issues/1.
For more details about CUE's migration to a new home, please see https://github.com/cue-lang/cue/issues/1078.


How I can add python-social-auth?

I tried: 'social.apps.django_app.default', 'social', 'social.apps.django.app', 'default'.
It doesn't work...

Have you figured this out? Ive been ridiculously busy .. last few weeks. I'll actually have to install python-social-auth on a project to figure out how to add it.
This package modifies context using a middleware .. may be you could use django-debug-toolbar to have a look at template context for django admin index page?


Reduce ethers

Reduce and consolidate the usage of ethers to the bare minimum. Only thing blocking us from removing it entirely is our usage of @snapshot-labs/snapshot.js, which requires an instance of ethers.Providers.Web3Provider into its vote function.

@Da-Colon please re-review.


JetSki needs to wait after sending mutating commands to iDRAC

After JetSki sends a mutating command to iDRAC it needs to wait for the command to succeed.
In particular, --clear-jobs -f (at least on a Dell 740xd) results in the following if one attempts to access it immediately:
mgmt-host.example.com: - ERROR    - internal error,
- INFO     - None
- ERROR    - ComputerSystem's Members array is either empty or missing
- ERROR    - There was something wrong executing Badfish.

My manual workaround is to run --check-boot every 30 seconds or so until it succeeds.
Similarly, after sending a -t foreman or -t director command, it's necessary to wait until that command "takes" (which will be after the node reboots).

so during the last few days, I did all kinds of testing and from what I have seen BadFish API calls returns as a success but in reality, nothing really happened, not sure if this issue only applicable for dell servers.
I took a different approach to this, and instead of using RedFish/Badfish I used racadm to speak directly to the BMC, and the deployment completed successfully with JetSki.

@bbenshab that hasn't been the experience for me in Scale lab. We need to figure out a way to make JetSki work for ALIAS OOTB. Between what @RobertKrawitz commented and what you found, would one of you be willing to submit a PR?
Also, just for the record, the boot order should have been set when the allocation is handed down to you from lab admins/automation.
@grafuls quick question, is badfish being used in the ALIAS lab?  Multiple people are reporting issues with using badfish in ALIAS (doesn't apply to scale lab).

Badfish does work in the alias lab, just with the caveats I mentioned above.  But if racadm is more reliable, gopherit.
Don't rely on the order provided by lab administration.  One might want to use JetSki to reinstall a cluster, in which case the boot order will be different.

@abondvt89 told me that scale lab does not have Dell 740xd machines so could this be specific to the 740xd machines in alias - can someone check if they are running the latest BIOS firmware released by Dell?

@ashishkamra This should be harmless even when running against other types of machines. It's simply a wait..

Sure, worth trying it out.

@grafuls quick question, is badfish being used in the ALIAS lab? Multiple people are reporting issues with using badfish in ALIAS (doesn't apply to scale lab).

It is being used in ALIAS and has been adapted to fit the requirements there. What are the issues?

The most serious issue is that after calling --clear-jobs -f, iDRAC doesn't respond successfully to any further commands for several minutes, with the error message as displayed on the OP.  However, the command returns a status of 0, so any automation that relies on the return value will incorrectly think that the command in question (e. g. setting the boot order, powering on/off the node, etc) has succeeded when in fact it has not.
I've also seen cases where sending the same command multiple times results in iDRAC's not responding "correctly" to the commands other than the first.  For example, repeated attempts to set director boot order result in that not happening.  I don't have a clear record of this, however, and it may simply be due to not having waited long enough for the command to succeed.  Changing the boot order requires a reboot, and that can take 5 minutes on a 740xd.

@grafuls quick question, is badfish being used in the ALIAS lab? Multiple people are reporting issues with using badfish in ALIAS (doesn't apply to scale lab).

It is being used in ALIAS and has been adapted to fit the requirements there. What are the issues?

@grafuls , Does badfish return a non-zero error code when an operation fails? In our case it returns success even if --check-boot fails.


supports uses-allocator construction

@Naios
uses-allocator construction
Seems all it needed is specializing std::uses_allocator<T, Alloc>.

Hi, yes seems like it. Could you provide the specialization through a PR?
A specialization for the class function in fu2::detail should be sufficient:
template <typename Config, bool IsThrowing, bool HasStrongExceptGuarantee,
          typename... Args>
class function<Config, property<IsThrowing, HasStrongExceptGuarantee, Args...>>

Fixed in https://github.com/Naios/function2/pull/62


refactor(egen): template fixes, updates clean up steps

This PR consists of the following updates:
1. Refactors the notebook according to the notebook template (removes boilerplate elements, renames REGION with LOCATION and re-structures the sections).
2. Updates deletion steps in clean up section.

REQUIRED: Fill out the below checklists or remove if irrelevant

If you are opening a PR for Official Notebooks under the notebooks/official folder, follow this mandatory checklist:

[x] Use the notebook template as a starting point.
[x] Follow the style and grammar rules outlined in the above notebook template.
[x] Verify the notebook runs successfully in Colab since the automated tests cannot guarantee this even when it passes.
[x] Passes all the required automated checks. You can locally test for formatting and linting with these instructions.
[ ] You have consulted with a tech writer to see if tech writer review is necessary. If so, the notebook has been reviewed by a tech writer, and they have approved it.
[ ] This notebook has been added to the CODEOWNERS file under the Official Notebooks section, pointing to the author or the author's team.
[x] The Jupyter notebook cleans up any artifacts it has created (datasets, ML models, endpoints, etc) so as not to eat up unnecessary resources.

/gcbrun

/gcbrun


change: Update changelog to use correct PR number for Upcoming Features entry

Description 📝

Updates changelog to link to correct PR (#10797) for entry in Upcoming Features section

Target release date 🗓
9/3/2024
How to test 🧪

Confirm PR for 'Add bucket management Properties Tab for Object Storage Gen2' is #10797, not #10795

ty!


Docs rebuild - improve accessibility of inline links and blockquote links

This PR addresses #3283

Looks good!  Appreciate the a11y tweaks, and if you see any other improvements that can be made, please go ahead and file PRs.  Thanks!

Happy to improve the docs experience, @markerikson! I'll be sure to make more PRs on some of the other a11y enhancements that are needed.

Looking forward to it!
I'd also like to apply these changes to https://github.com/reduxjs/react-redux and https://github.com/reduxjs/redux-starter-kit as well.  Doesn't have to be right away.  You can submit PRs yourself, or we can copy over the changes as appropriate.


Close WSGI response

Fixes #1723 (see also explanation for this PR there)
Unsure about where tests go.

Coverage remained the same at 72.806% when pulling 64b0e557b408bd2bee21600c2a25ad2d1ef8fff2 on untitaker:fix/wsgi-close-response into 7730144dec01b4cdbf0205b936a200e54a287ebb on Miserlou:master.

Any one on the Zappa team watching this?  Any idea when it can be merged in?  It seems like a pretty big issue.

@untitaker that's likely true as Lambda function timeout after 5 minutes anyhow.  However, because of this it seems that the sentry SDK is not logging the proper url or transaction.
You can see my comments here: https://github.com/getsentry/sentry-python/issues/190

@iteratelance responded there, just didn't get around to it yet. I don't think it's something Zappa is supposed to fix.


Added $promptError as an additional parameter to execute()

As proposed in issue #18 and finally making the change today, I've added the $promtError parameter to the execute method.
This allows $promptError to be evaluated dynamically (per execute call) rather than statically (only on connect) as is done currently.
I've added a unit test method. I've run this manually using phpunit (instead of make test).

Travis build fails on HHVM build jobs.
Looking into it further, it seems composer no longer supports HHVM:
https://github.com/composer/composer/issues/8127

Thanks!


Test PR

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_ea8cm8wwyu/clone/recipe/meta.yaml:0: missing_tests: No tests were found.
ERROR: /tmp/abs_ea8cm8wwyu/clone/recipe/meta.yaml:0: missing_section: The requirements section is missing.
ERROR: /tmp/abs_ea8cm8wwyu/clone/recipe/meta.yaml:15: missing_dev_url: The recipe is missing a dev_url
ERROR: /tmp/abs_ea8cm8wwyu/clone/recipe/meta.yaml:15: missing_documentation: The recipe is missing a doc_url or doc_source_url
WARNING: /tmp/abs_ea8cm8wwyu/clone/recipe/meta.yaml:15: missing_description: The recipe is missing a description
ERROR: /tmp/abs_ea8cm8wwyu/clone/recipe/meta.yaml:15: missing_summary: The recipe is missing a summary
Errors were found


OData connection SuccessFactors - Exception reading in Metadata (Annotations)

Getting an exception when trying to make an OData connection to SAP's SuccessFactors.
"The metadata document could not be read from the message content.\r\nUnexpectedXmlElement : The schema element 'Annotation' was not expected in the given context. : (1, 522)\r\nTextNotAllowed : The current schema element does not support text 'Address 1'. : (1, 554)\r\n"
Here's a snipped of their metadata xml being returned.
I don't mind paying you for your timing debugging this.
Thanks

Address 1

Hi Richard,
The snippet didn't make it, I can't see the metadata XML. Can you try again or just send metadata to<EMAIL_ADDRESS>Thanks!

The metadata schema is invalid because of two unsupported items but they can be removed at interception stage using AfterResponse handler. Here's an implementation of AfterReponse handler that I quickly wrote:
    private static void AfterResponse(HttpResponseMessage responseMessage)
    {
        if (responseMessage.RequestMessage.RequestUri.LocalPath.EndsWith("$metadata"))
        {
            var metadata = responseMessage.Content.ReadAsStringAsync().Result;
            var doc = new XmlDocument();
            doc.LoadXml(metadata);

            var manager = new XmlNamespaceManager(doc.NameTable);
            manager.AddNamespace("ns", "http://schemas.microsoft.com/ado/2008/09/edm");
            var nodes = doc.SelectNodes("//ns:Annotation", manager);
            for (var i = nodes.Count - 1; i >= 0; i--)
            {
                nodes[i].ParentNode.RemoveChild(nodes[i]);
            }

            nodes = doc.SelectNodes("//ns:Property[@CollectionKind]", manager);
            for (var i = 0; i < nodes.Count; i++)
            {
                nodes[i].Attributes.Remove(nodes[i].Attributes["CollectionKind"]);
            }

            using (var stringWriter = new StringWriter())
            using (var xmlTextWriter = XmlWriter.Create(stringWriter))
            {
                doc.WriteTo(xmlTextWriter);
                xmlTextWriter.Flush();
                metadata = stringWriter.GetStringBuilder().ToString();
            }

            var byteArray = Encoding.UTF8.GetBytes(metadata);
            var stream = new MemoryStream(byteArray);
            stream.Position = 0;
            var content = new StreamContent(stream);
            foreach (var header in responseMessage.Content.Headers)
            {
                if (header.Key != "Content-Length")
                    content.Headers.Add(header.Key, header.Value);
            }
            content.Headers.Add("Content-Length", metadata.Length.ToString());
            responseMessage.Content = content;
        }
    }


Final Exam

I propose to send out the final exam on Wednesday January 2rd 2020. But I am willing to delay it. When will the school end?

My final exam:Computational Physics ends at January 13th 2020, before that it will take me a lot of time to prepare for the exams. Will the subscription day be later than that so I can have plenty of time prepare for my exams? Thanks very much.

So I will set the deadline of submitting the final exam at January 14th 2020. I guess this is the best I can do.

The final exam has been posted.


Feature Request: Include elevation

This is a superb library for filtering raw location data, thanks for providing it.
For activity tracking not only latitude and longitude are relevant, but also the current elevation. It would be great if you'd filter and return that aswell.

Hi,
Glad to hear that you like it. We are going to add new features soon. Will have elevation data filtering on mind. This was used for an internal project that we did and we had no use of elevation data.
Best Regards,
Hypercube


fix(plugins/k8saudit): solve k8saudit issues and improve error reporting

What type of PR is this?
/kind bug
/kind feature
Any specific area of the project related to this PR?
/area plugins
What this PR does / why we need it:
This PR solves some issues with the k8saudit plugin, mostly related to error reporting. At the same time, this experiments using the newest fixes in the Go SDK.
Which issue(s) this PR fixes:
Special notes for your reviewer:

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: c442de009eeaac8254a6d1cf73d3701492e54d0b


Cancel edit on right click

Allow cancellation of DM editor operation using mouse right click instead of pressing Esc - it's a pain on my keyboard :)

close #349


Cucumber Exhaustive Combination: GC overhead limit errors thrown for small requests

Bug Report
Exhaustive combination tests across three fields, each with a set of five data options, were run. This was expected to generate 124 rows of data, not expected to be a large request, but failed with a memory error.
Test Example 2.txt
A second test expecting 25 roes from three fields (one set of 1 data option and two sets of five data options) also failed with the same memory error.
Test Example 1.txt
Steps to Reproduce:

Run tests from the feature file ExhaustiveCombination.feature
See that there is a failure for the test Scenario: Running an exhaustive combination strategy across three fields with five data options each should be successful
Note that the test is long running and then fails with an out of memory error

Expected Result:
These tests should generate the expected 124 / 25 rows of data based on the exhaustive combination strategy.
Actual Result:
The tests fail with a memory error:

java.lang.OutOfMemoryError: GC overhead limit exceeded
at com.scottlogic.deg.generator.generation.FieldSpecFulfiller.lambda$generate$0(FieldSpecFulfiller.java:35)
at com.scottlogic.deg.generator.generation.FieldSpecFulfiller$$Lambda$133/726181440.apply(Unknown Source)
at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
at java.util.Iterator.forEachRemaining(Iterator.java:116)
at java.util.Spliterators$IteratorSpliterator.forEachRemaining(Spliterators.java:1801)
at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
at java.util.stream.StreamSpliterators$WrappingSpliterator.forEachRemaining(StreamSpliterators.java:312)
at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
at java.util.stream.ReduceOps$ReduceTask.doLeaf(ReduceOps.java:747)
at java.util.stream.ReduceOps$ReduceTask.doLeaf(ReduceOps.java:721)
at java.util.stream.AbstractTask.compute(AbstractTask.java:316)
at java.util.concurrent.CountedCompleter.exec(CountedCompleter.java:731)
at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
at java.util.concurrent.ForkJoinTask.doInvoke(ForkJoinTask.java:401)
at java.util.concurrent.ForkJoinTask.invoke(ForkJoinTask.java:734)
at java.util.stream.ReduceOps$ReduceOp.evaluateParallel(ReduceOps.java:714)
at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:233)
at java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:499)
at com.scottlogic.deg.generator.generation.combination_strategies.ExhaustiveCombinationStrategy.lambda$permute$0(ExhaustiveCombinationStrategy.java:18)
at com.scottlogic.deg.generator.generation.combination_strategies.ExhaustiveCombinationStrategy$$Lambda$87/737945227.apply(Unknown Source)
at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
at java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1382)
at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
at java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:708)
at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234)
at java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:499)
at com.scottlogic.deg.generator.generation.combination_strategies.ExhaustiveCombinationStrategy.permute(ExhaustiveCombinationStrategy.java:19)
at com.scottlogic.deg.generator.generation.databags.MultiplexingDataBagSource.generate(MultiplexingDataBagSource.java:17)
at ✽.the following data should be generated:(C:/DataGenerator/data-engineering-generator/generator/src/test/java/com/scottlogic/deg/generator/cucumber/ExhaustiveCombination.feature:379)
java.lang.OutOfMemoryError: Java heap space
at java.util.HashMap.resize(HashMap.java:704)
at java.util.HashMap.putVal(HashMap.java:629)
at java.util.HashMap.put(HashMap.java:612)
at com.scottlogic.deg.generator.generation.databags.DataBag$DataBagBuilder.set(DataBag.java:76)
at com.scottlogic.deg.generator.generation.FieldSpecFulfiller.lambda$generate$0(FieldSpecFulfiller.java:42)
at com.scottlogic.deg.generator.generation.FieldSpecFulfiller$$Lambda$133/726181440.apply(Unknown Source)
at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
at java.util.Iterator.forEachRemaining(Iterator.java:116)
at java.util.Spliterators$IteratorSpliterator.forEachRemaining(Spliterators.java:1801)
at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
at java.util.stream.StreamSpliterators$WrappingSpliterator.forEachRemaining(StreamSpliterators.java:312)
at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
at java.util.stream.ReduceOps$ReduceTask.doLeaf(ReduceOps.java:747)
at java.util.stream.ReduceOps$ReduceTask.doLeaf(ReduceOps.java:721)
at java.util.stream.AbstractTask.compute(AbstractTask.java:316)
at java.util.concurrent.CountedCompleter.exec(CountedCompleter.java:731)
at java.util.concurrent.ForkJoinTask.doExec(ForkJoinTask.java:289)
at java.util.concurrent.ForkJoinTask.doInvoke(ForkJoinTask.java:401)
at java.util.concurrent.ForkJoinTask.invoke(ForkJoinTask.java:734)
at java.util.stream.ReduceOps$ReduceOp.evaluateParallel(ReduceOps.java:714)
at java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:233)
at java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:499)
at com.scottlogic.deg.generator.generation.combination_strategies.ExhaustiveCombinationStrategy.lambda$permute$0(ExhaustiveCombinationStrategy.java:18)
at com.scottlogic.deg.generator.generation.combination_strategies.ExhaustiveCombinationStrategy$$Lambda$87/737945227.apply(Unknown Source)
at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
at java.util.stream.ReferencePipeline$3$1.accept(ReferencePipeline.java:193)
at java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1382)
at java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:481)
at java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:471)
at java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:708)
at ✽.the following data should be generated:(C:/DataGenerator/data-engineering-generator/generator/src/test/java/com/scottlogic/deg/generator/cucumber/ExhaustiveCombination.feature:528)

Environment:
ExhaustiveCombination branch running in full generation strategy & exhaustive combination strategy on IntelliJ.
Additional Context:
Related to existing performance issues.

Priority: High to investigate at least

Hi @cakehurstryan,
Has the ExhaustiveCombination branch not been pushed? Can't find it to repro this issue. Can you have a look please? Thanks!

Hi @cakehurstryan, I was able to repro with the attached files by copying them into a feature file. Ignore previous comment. :)

It looks like the tests in this issue were failing with OOM due to having 2 unconstraint fields (foo4 and foo5) and running exhaustive strategy over them. Found the tests in ExhaustiveCombination.feature and enabled them (they didn't actually have the foo4 and foo5 there) and they pass.
Created this PR to enable them on master: https://github.com/ScottLogic/data-engineering-generator/pull/361

Closed as resolved in #361


Animate SVG gradient color-stop

Hello. First of all, congratulations on your work, I've been using anime.js and it's amazing!
What I'm trying to do
I'm trying to animate an SVG radial gradient, changing the color values for the 'color-stop' attribute on the <stop> element. I want to use anime.js, but so far no luck.
What I've got so far
I've had success using <animate> inside <stop>, and I have a working codepen here.
However, I'd really like to do the same thing using anime.js because the solution with  poses some difficulties for me, like synchronising it with other anime.js animations using events like "complete".
I'm using anime.js to morph svg polygons with some randomness, and I'd like to add random hsla colors to the animation. I have a function that returns a random hsla color, but I can't get it to work with anime.js. The problem, however, lies with targeting the "stop-color" attribute for animation. Here's what I have:
//randHSLAColor() returns a string like 'hsla(50,60%,80%,0.4)'
function animGradient() {
        let rc1 = randHSLAColor();
        let rc2 = randHSLAColor();
        let gra1 = anime({
            targets: 'grad-stop-01',
            stopColor: rc1,
            easing: 'linear',
            duration: 1000
        });
        let gra2 = anime({
            targets: 'grad-stop-02',
            stopColor: rc2,
            easing: 'linear',
            duration: 1000
        });
    }

Other things I tried
If I use "stop-color" instead of "stopColor" on animGradient() I get an error:
Uncaught SyntaxError: Unexpected token '-'
I've also tried to replace rc1 and rc2 with hardcoded values like 'hsla(80,70%,50%,1)', so I know the error is not coming from there.
Any thoughts on this? I'm not sure if what I'm trying to do is possible with animejs, maybe 'stop-color' is not animatable, or maybe I'm just not writing the function correctly.
Thank you!

Define an object calls animationParameters, put your variables into this object, update stop-color manually inside onChange lifecycle, this may work.
another thing, have you tried syntax like this?
{
  'stop-color': rc2,
}

add a quote will fix your syntax error.

Thank you so much @Losses, using quotes did the trick! I'll also keep in mind your first suggestion in case I come across something that's not solved by the quotes.