Multiple Backends

As I mentioned in #3, it would be nice to support multiple backends, including immutable ones. You mentioned it might be clearer if I sketched out exactly what sort of generic interface I had in mind, and I finally found time to do so: https://github.com/Ericson2314/haskell-vcache.
The generic interface is https://github.com/Ericson2314/haskell-vcache/blob/master/generic-store/hsrc_lib/Database/Generic.hs, in a separate package. The *Mut type classes extend the immutable ones. The type of hash-everything content addressable store I am interested in would just implement the immutable classes, and vcache would additionally implement the mutable ones.
A few things on the implementation if you like the basic idea:

My Persistant* typeclasses are more restrictive than VCacheable, because they require that the instance work for all MonadGet/MonadPut. This means any (de)serialization scheme that takes advantage of some backed-specific functionality won't work. For that reason I kept around VCacheable so as to keep non-generic use in order to use more vcache-specific features (in general, great job for a really clean API, but something things like the caching strategy I deem premature to add to the generic interface).
I used some Data.Coerce on VRefs (and, once I'd spec out more, PVars too) to make this work---I think its perfectly sound due to the way VRefs store their parser, but if not, it should be possible to write a functor instance for VRef that would use the functor instance on MonadPut and drop the cached value, which might help.

I didn't actually try using this, but ensured that everything type-checked/built.

It turns out I'm taking a different direction for my Wikilon project, rewriting and specializing haskell-vcache in a C runtime for my bytecode. So I'll probably not get back to maintaining haskell-vcache. If you want to take over, I'd be happy to put you on the maintainer list.

Well, I have one more semester of school coming up, and probably not much time to maintain it then, but I am definitely interesting in this for the long term, so count me in!

Would be nice if the deprecation was added to the README.md.


Extending the HoardSuperblock to support different headers

Hi,
I'd like to be able to add additional fields to the HoardSuperblock header. If the Header were passed as a template parameter, that would be easy.
Thanks,
Irene

Forgot to close (implemented with https://github.com/emeryberger/Hoard/commit/baa62abb7eb873bd40068fbb96f44d7939164a20).


WIP improvements for placement of turn restriction icons

I know this is not the priority, but it's been bugging me 😉
This is a first attempt to "stack" turn restriction icons along the length of the road when there is more than one. (Mentioned in https://github.com/a-b-street/abstreet/issues/1096). For a long road segment, this works reasonably well.
The problem cases are:

Where the turn restriction icons occupy more than half the length of the road. Currently, they are forced into the correct half of the road but can still overlap each other.
Worse, when the road is shorter than it is wide.
If the road segment is curved, then this can cause oddities where the turn type is assessed differently depending on how close the icon is to the intersection.

This is what it looks like in ideal conditions:
https://github.com/a-b-street/abstreet/assets/5346065/1c809181-40d6-49e3-8e9f-bc934b78e77d
Below are the problem cases:

Where the turn restriction icons occupy more than half the length of the road. Currently, they are forced into the correct half of the road but can still overlap each other.

Here, two icons are added that partly overlap. It is possible to tell there is more than one icon, even though it is hard to read them:
https://github.com/a-b-street/abstreet/assets/5346065/a1f3d7ba-8322-4a87-a7d1-7c6e2dd12050

Worse, when the road is shorter than it is wide.

Here, the multiple icons are drawn on top of each other. It is not possible to see that there is more than one icon. This is the worst outcome, but it is the same behaviour that occurred whenever there was more than one icon before:
https://github.com/a-b-street/abstreet/assets/5346065/66d3d5c2-6bb7-4a6a-9f78-e1650c0d4cf5

If the road segment is curved, then this can cause oddities where the turn type is assessed differently depending on how close the icon is to the intersection.

In this example, if the turn restrictions are added in the order (a) no straight ahead (b) no right turn, then the correct icons are added. If the turn restrictions are added in the order (a) no right turn (b) no straight ahead, this results in the "no straight ahead" icon being added to the road at the curve, and the angle of the restriction being incorrectly classified as a second "no right turn" restriction:
https://github.com/a-b-street/abstreet/assets/5346065/ab1a4141-96bc-4535-8cbc-d809fbfa5511

Merging this as is. I'll make any additional improvements on a new branch/PR


Unit detect fix and update

please reopen this from commit ff80e591dff39def0ea550bbe5f1dd171de17e88 and set the version to 0.4.2


account deletion and multilogin

currently, if i socially login with discord, then logout, then login with google, the system will create two different user IDs
as a user, I would like the option to be able to delete one of my accounts if I do this, and the ability to delete both of my accounts if i want. i should get a warning and need to confirm
then as a user i would like access to a profile page where i can associate a google and discord account to one user id

this should include email verification


⏱  [Process] - Add comment for future contributors around solutions guide coordination

Add a comment to google_sql_database_instance to update solutions guide if the tier (machine type) changes.
https://github.com/GoogleCloudPlatform/terraform-dynamic-python-webapp/blob/main/infra/database.tf#L33
Filing this in the backlog so we can use the published URL of that guide as a reference.

Guide no longer references this specified value.


Add DSE and a new AST infrastructure to StencilKernel

This merge represents a massive step towards Devito-3.0.It integrates the DSE with the current StencilKernel and adds a new flexible AST node and tree-visitor infrastructure to represent and manipulate arbitrary loop nests. In particular this merge adds:

Perform symbolic DSE optimisation on raw incoming SymPy expressions before building Iterations
Add a hierarchy of AST nodes that include Expression and Iteration among others
Add base classes for tree traversal visitors:

Visitor for passive tree traversal, eg. finding things in the AST
Transformer for manipulating and re-building a tree

FindSymbol visitor, allowing us to infer the kernel signature more elegantly
FindSection visitor and Timer node, allowing us to automate section-specific profiling in C
SubstituteExpression transformer that performs symbolic substitution on Expression nodes in a tree
ResolveIterationVariable transformer that creates and substitutes explicit iteration variables for Dimensions, including explicit buffer variables (eg. t0, t1, t2) for buffered variables

What can I say, this is a joint PR from me and Michael, so maybe someone else should review this code.
I again skimmed through it. There's a lot of polishing that needs to be done, but the PR is already massive. I therefore think this should land ASAP, since it will be followed by a stream of smaller PRs

Signed off in the chat by @ggorman. So merging now...


Crash on i3 restart

This may very well be an issue with i3 itself but I'd like to hear your input anyways before(if) submitting report for i3.
Debug log when running imgur-screenshot from command line: https://hastebin.com/ocuhetixam.php
Debug log when running imgur-screenshot using keybindings: https://hastebin.com/edusukegap.coffeescript
When I run the command by using keybinding defined in i3config:
bindsym --release Shift+Ctrl+Print exec "imgur-screenshot --debug"
I get the notification with imgur link but for some reason xdg-open doesn't get executed. Also there is some gibberish at the end of debug log. Some times when I restart i3 in place after executing imgur-screenshot with keybindings i3 completely freezes and I have to kill it manually from different TTY.
Here's my config file: https://hastebin.com/iposeqavol.ini
Any ideas?

The "gibberish" is actually ANSI escape codes. After applying them you get this prompt:

Searching for the text suggests this is coming from Links or eLinks. So xdg-open does get executed and runs said program. Unfortunately there's no standard behavior of xdg-open, but my best guess is your implementation uses different programs to open HTTP links depending on the shell/environment it runs in.

Indeed, I check if there is $DISPLAY variable available and depending on that I set my $BROWSER to either chromium or links. Thanks


awake icon no longer in system tray

Microsoft PowerToys version
0.83
Installation method
PowerToys auto-update
Running as admin
Yes
Area(s) with issue?
Awake
Steps to reproduce
awake icon used to be visible in the system tray
It no longer show up
I need to click on powertoys - plus - awake
And then I can change the power mode and it works
The issue is : the icon is not visible
Other than that everything seems OK
Windows version 23H2 22631.4112 French
✔ Expected Behavior
awake icon should be visible
❌ Actual Behavior
No response
Other Software
No response

/dup https://github.com/microsoft/PowerToys/issues/34148


WebApp - Show incorrect account on the Token detail screen when disconnect account injected

Describe the bug
Show incorrect account on the Token detail screen when disconnect account injected
To Reproduce
Steps to reproduce the behavior:

Open the WebApp
Perform connecting extension successfully
Switch to All accounts mode
Go to the token details screen of any token (Substrate token)
Observe the list token displayed
Disconnected any Substrate account on Extension
Back to token details screen on WebApp then observe the list token displayed again

Actual: Still shows account disconnected

Only show address, not show account name

Expect: Do not show disconnected account

@lw-cdm, method BalanceMapImpl.removeBalanceItems will be useful in this case

Version test : https://aa80be5e.subwallet-webapp.pages.dev/
Validation passed

When connecting to extension :

When not connected to the extension :


[ADD] Make sqlalchemy_extractor compatible with sqlalchemy>=1.4

feat: Make sqlalchemy_extractor compatible with sqlalchemy>=1.4
(Moved from  #2209)
Description
These changes to the sql_alchemy_extractor module allow for using the extractors with updated versions of sqlalchemy.
Motivation and Context
This addresses issues such as

#2108 which require the text function to be used to wrap queries in SQLAlchemy>=2.0
Posts on the Slack channel such as this and this which require the change to explicitly map the Row objects to dicts for SQLAlchemy>=1.4

How Has This Been Tested?
Tested using sampple_mssql_metadata.py script for

SQLAlchemy==1.3.16
SQLAlchemy==2.0.23

Documentation
None
CheckList

[X] PR title addresses the issue accurately and concisely
[X] Updates Documentation and Docstrings
[X] Adds tests
[X] Adds instrumentation (logs, or UI events)

@ozandogrultan Anything else needed for this one? Happy to update if needed

Bumping for review


feat: external FO in obs_sequences

e.g. GSI2DART
it would be good to read this in and examine external forward operators

e.g. how many obseversions in an obs_seq file have external forward operators.

Reading external_FO in an writing it in as of  ea68901
Not doing anything with this data.


Distinguish between Slerp and Angle

Vector3 can be slerped (when treated as an offset from origin), or can be rotate to closest angle (when treated as euler angles).
In either case the linear tween makes sense without any modifier, but chaining .Slerp on an euler angles rotation would create unexpected behaviour.
Also support shortest angle lerp for Vector3.
transform.TweenRotation(vec3, 1f).ShortestAngle();

Actually not sure this matters.


adding rwmix parameters to the schema

Changes introduced with this PR
Adds rwmixread and rwmixwrite parameters. Fixes #44

By contributing to this repository, I agree to the contribution guidelines.

Given that "the latter of the two will [...] override the first", it seems that having both is just plain silly. However, I suppose it allows you to specify the split using whichever side you prefer, and you're just propagating fio's silliness, so, fine.

I agree it's silly (of fio) to have both. But it's possible to have an input file with inheritance from a "global"

the latter of the two will be used to override the first

How do I, as an Arcaflow user (or a workflow writer) know which one is "the latter"?  There is no ordering in the interface at this level...right?
On the other hand, I suspect that we can determine at the Arca-FIO interface which of the two will always override the other...and so we really should document that.

the latter of the two will be used to override the first

How do I, as an Arcaflow user (or a workflow writer) know which one is "the latter"? There is no ordering in the interface at this level...right?
On the other hand, I suspect that we as Arcaflow developers can determine at the Arca-FIO interface which of the two will always override the other...and so we really should document that.
(And, it would be great if we had a test which could confirm that that behavior is maintained (e.g., by checking the ordering of the switches on the command line).)

Is the order based on the plugin-generated FIO input file? If so, that would then depend on how your code orders its representation of the config from the schema input. If that's the case, then the plugin defines the order, and we should document it based on the plugin's implementation instead of the generic documentation for FIO.

the latter of the two will be used to override the first

How do I, as an Arcaflow user (or a workflow writer) know which one is "the latter"? There is no ordering in the interface at this level...right?
On the other hand, I suspect that we as Arcaflow developers can determine at the Arca-FIO interface which of the two will always override the other...and so we really should document that.
(And, it would be great if we had a test which could confirm that that behavior is maintained (e.g., by checking the ordering of the switches on the command line).)

Is the order based on the plugin-generated FIO input file? If so, that would then depend on how your code orders its representation of the config from the schema input. If that's the case, then the plugin defines the order, and we should document it based on the plugin's implementation instead of the generic documentation for FIO.

Yeah, the ordering I see is an issue. The order precedence comes from the underlying tool, but it's non-deterministic for us. I'll think about how best to address this.


[BUG] Programmatic access to config validator (via env vars) is out of scope

Environment
Other (Specify below)
System
NixOS Unstable (Nixpkgs /b73c2221a46c13557b1b3be9c2070cc42cf01eb3)
Version
3.1.1+ (Git 0b1af9db483f80323e782e7834da2a337393e111)
Describe the problem
While attempting to package Dashy for NixOS, I tried to use yarn validate-config as part of the build process to validate that the configuration is correct before building the dashboard, which can often take several minutes.
In order to stop the build, however, a command needs to exit with a non-zero exit code - and as it currently stands, yarn validate-config exits with 0 whether or not the config is valid.
Looking into the config validator, there appears to be an environment variable set if the configuration is valid. In practice (by inserting a printenv), the variable seems to either be unset or out of scope after the validator exits, which makes programmatic access to the status harder.
Additional info
Using my (albeit limited) javascript knowledge, I tried to alter the validator script to exit with a non-zero code:
try { // Try to open and parse the YAML file
  config = yaml.load(fs.readFileSync(`./${process.env.USER_DATA_DIR || 'user-data'}/conf.yml`, 'utf8'));
  validate(config);
} catch (e) { // Something went very wrong...
  setIsValidVariable(false);
  logToConsole(bigError());
  printFileReadError(e);
+ process.exit(1);
}
module.exports = config;

...Which still seemed to exit with code 0 while config errors were present.
Please tick the boxes

[X] You have explained the issue clearly, and included all relevant info
[X] You are using a supported version of Dashy
[X] You've checked that this issue hasn't already been raised
[X] You've checked the docs and troubleshooting guide
[X] You agree to the code of conduct

Additionally related to packaging with NixOS (though unrelated to the config validator), I'm getting these errors when fetching the yarn packages to an offline cache:
success Set "yarn-offline-mirror" to "/nix/store/49a9m7wdssvxz91lcxvq8w6206b7cx4f-offline".
Done in 0.02s.
yarn install v1.22.22
[1/4] Resolving packages...
[2/4] Fetching packages...
[3/4] Linking dependencies...
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^5.0.0".
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency "eslint-plugin-react-hooks@^4 || ^3 || ^2.3.0 || ^1.7.0".
warning "eslint-config-airbnb ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>|| ^4.0.0".
warning Workspaces can only be enabled in private projects.
[4/4] Rebuilding all packages...
warning Ignored scripts due to flag.
Done in 11.83s.


Update DefaultAzureCredential, InteractiveBrowserCredential, and DeviceCodeCredential docs

closes #44840

API change check
API changes are not detected in this pull request.

API change check
API changes are not detected in this pull request.


Gitops install should not require kubectl

Short description
As an app developer I want to install gitops into my cluster(s) from a container and I don't want to build kubectl into the container to run
Acceptance criteria

[ ] install successfully creates k8s entitles when run from an environment with or without kubectl installed.
Works with local kind clusters as well as AWS, AKS, GKE

Not required for the New Directory structure work.  It can be addressed independently.


feat: Enhance block import log line

Adds gas, gas used, tx count, base fee, and import elapsed time to the block imported log line.
for bug #5856

sample line
2024-01-16T13:24:16.664029Z  INFO reth::commands::node::events: Block added to canonical chain	txs=222 mgas=14.450 gas_pct=48.17% base_fee_gwei=25.51 elapsed=5.22s number=19018271 hash=0x5f8b453ea9797c810e66e0c63642739ed54f792ac2248f1be502be1201334caf num_connected_peers=93

Current log status line:
024-02-08T20:49:50.431152Z  INFO reth::commands::node::events: Block added to canonical chain number=904063 hash=0x9abdc5fae5bad6434d7e5f3da10ce66e0d9e95fe35afce807482e091a41f2af3 peers=7 txs=28 mgas=8.368 full=27.9% base_fee=1.05gwei blobs=6 excess_blobs=642 elapsed=91.610542ms


Restore Permissions Issues

The 03 Built-in Query Intelligence / pspop lab requires a restore from c:\samples.  People are having permissions issues doing the restore. People have successfully changed their sql service account to local system and it worked fine. Other people struggled due to lock-down policies from their IT department.
We could probably prevent this issue by having people copy the backups to the SQL data directory.

thanks for this. It was the first time I've seen this so I added a comment in the readme files for each exercise to copy to the default data directory should they have a perms issue.


Huge difference between istanbul 1.0-alpha2 and isparta

I'm currently testing istanbul v1.0 alpha2 instead of isparta for a project but I'm getting a huge difference in the report... Same codebase, just different tool:
isparta:
Statements   : 81.06% ( 5548/6844 ), 662 ignored
Branches     : 75.01% ( 2467/3289 ), 800 ignored
Functions    : 72.87% ( 1050/1441 ), 117 ignored
Lines        : 60.44% ( 1948/3223 )

istanbul:
Statements   : 56.22% ( 1817/3232 )
Branches     : 41.98% ( 599/1427 )
Functions    : 47.96% ( 364/759 )
Lines        : 56.9% ( 1772/3114 )

@tleunen it appears as if code coverage for transpiled code is all over the place right now. No two tools produce the same result 😛
This thread is a good read: https://github.com/jmcriffey/babel-istanbul/issues/56
The last comment suggests that this project is accurate, but it's pretty new and might produce accurate results in all cases.
If anyone figures out the best solution for coverage, do let me know. I'm trying to figure out what to use for the Babel boilerplate (https://github.com/babel/generator-babel-boilerplate/issues/330)

Hi @jmeas - I'm currently collaborating with @dtinth on an instrumentation library that uses his ideas in the __coverage__ plugin and preserves the behavior of istanbul's current instrumenter.
That work is going on here and it should be ready "soon".
https://github.com/istanbuljs/istanbul-lib-instrument/issues/4
https://github.com/istanbuljs/istanbul-lib-instrument/pull/5


Parse hexadecimal and binary integers

We only support decimal integers now. However, in certain contexts it is clearer to write 0xff or 0b11 instead. To solve this issue we need to:

[ ] Figure out the syntax:

do 0x10 and 0X10 mean the same? are they both valid?
what happens with the -? is processed as usual and just flips the sign of the number?

[ ] Write two new parsers  hex_numberand bin_number inside parser::literal.
Those parsers should detect and consume the 0b or 0xprefix but discard it. Leaving just the actual digits of the number and then use from_str_radix to convert them to i64s.

[ ] Add some parsing and evaluation tests. Stuff like 2 == 0b10 and so on.

One problem here is that there is no method in std to produce a i64 from an string like 0x42f23a.

What about from_str_radix https://doc.rust-lang.org/std/primitive.i64.html#method.from_str_radix ? We would just need to remove the 0x while parsing.

That sounds great :) I'll update the instructions to reflect that

If we use this method we could invent a integer syntax with arbitrary (2-36) integer base. For example BASE_n_NUMBER

how standardized is the syntax for bases different from 2, 8, 10 and 16?

As far as I know there aren't other syntaxes for other bases because they aren't used/useful.
We can make our own syntax xkcd 😃

I feel personally attacked by that comic.
I think we are OK with decimal, binary and hexadecimal for now.  I'd like to keep syntax as minimal as possible.

I feel personally attacked by that comic.

Sorry for that 🙏
So for the syntax there is as far as I now (just lower case for now)

0xNUMBER for hex
0bNUMBER for binary
0NUMBER for octal

I propose using 0cNUMBER for octal to make parsing easier and equal for all special cases.

Sorry for that 🙏

Heh, no problem. I was joking. Sometimes I forgot that text doesn't have the tone I have when talking :P

I propose using 0cNUMBER for octal to make parsing easier and equal for all special cases.

what about 0o like rust? https://doc.rust-lang.org/std/fmt/trait.Octal.html

what about 0o like rust? https://doc.rust-lang.org/std/fmt/trait.Octal.html

The o is very similar to 0 in some fonts but it is still fine if we decide to just take lower case letters (0O / 0O won't be readable).

I think lowercase is good then

I will try to implement this then if it's ok.
Can you edit the first post based on the discussion?

I updated the issue to reflect the current state


Added bit manipulation practice problems

📥 Pull Request
Description
Added Practice problems of bit manipulation with basic , Advanced Bit Manipulation Problems, Masking and shifting problems, Miscellaneous Bit Manipulation Problems
Fixes #310
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

@ajay-dhangar please review my pr


In danger-gitlab, ability to ignore ssl errors

Report
What did you do?
I'm trying to set up Danger in a weird situation where our Gitlab installation has an invalid SSL certificate. But, since I'm not the owner of it, can not fix the certificate.
What did you expect to happen?
It'd be nice to have an option to ignore SSL Errors... so I can use Danger. :)
What happened instead?
/usr/local/lib/ruby/2.6.0/net/protocol.rb:44:in `connect_nonblock': SSL_connect returned=1 errno=0 state=error: certificate verify failed (unable to get local issuer certificate) (OpenSSL::SSL::SSLError) 	from /usr/local/lib/ruby/2.6.0/net/protocol.rb:44:in `ssl_socket_connect' 	from /usr/local/lib/ruby/2.6.0/net/http.rb:996:in `connect' 	from /usr/local/lib/ruby/2.6.0/net/http.rb:930:in `do_start'
Your Environment

Running Gitlab Enterprise
Latest and greatest Danger
My Dangerfile is just one line that outputs a message.

Sure, as long as it's not a default.

Yeah, most definitely should not be a default.

Is this something that can be added easily? Wondering if I should wait. I'd help but I'm not a Ruby guy. ;)

Did some experimenting with @fishgills, you are able to disable Gitlab's SSL verification by passing in the environment variable GITLAB_API_HTTPARTY_OPTIONS="{verify: false}" as documented in the README.md of the Gitlab gem. I think we can close this one for Gitlab in particular, might be useful for other VSC systems though.

Thanks @georgewilli-am . You sound like a good employee.


Inputs in prepend and append not sizing correctly

Using the form fields, the differents kind of input have not the same size as default, well, this can be fixed using size classes.
But, when I use prepended text to inputs, I cannot have all my fields that have the same length, I can add CSS, but the "default" style size is incorrect.

The only way to do this is to use javascript to detect the size of the add-on and deduct that from the size of the input. That's a bit too much overhead to abstract and encourage others to use, so whatever solution you have now will have to do.


Post job phase fails to remove private key

Description:
If you have multiple setup steps with different settings but same gpg-private-key every time, then post job phase fails with the following message:
Post job cleanup.
removing private key from keychain
Error: failed to remove private key

Task version:
actions/setup-java@v1
Platform:

[X] Ubuntu
[ ] macOS
[ ] Windows

Runner type:

[X] Hosted
[ ] Self-hosted

Repro steps:

Create a workflow with two (or more) setup-java steps, with different settings but same gpg-private-key each time.
Run the workflow.
The first post job phase completes successfully.
All the other post job phases fail with the error "failed to remove private key".

Expected behavior:
Post job phase should try to remove the private key if still present, but not fail if a previous step have already removed it.
Actual behavior:
Post job phase fails trying to remove a private key already removed.

Hello @tbossi. Could you please define the gpg-private-key only for the first call of setup-java.

Oh, I hadn't thought of that...
That way it works, thanks!

Hello @tbossi. Thank you for your response. I'm closing the issue because proposed solution has helped.

@dmitry-shibanov This is a recurring papercut. Is there any way setup-java could be fixed to not require the workaround?


ARTEMIS-1360 Update config-reload documentation on queues

Updated details on max-conumsers and purge-on-no-consumers now these fields  on queue update on reload.

@howardgao here is the few changes mentioned on your pr, could you review please?


replaces  with . Removes one duplicate assignment.

Replaces node['organization'] ? node['organization'] || 'Facebook' with node['organization'] || 'Facebook'. Removes one duplicate assignment.
https://travis-ci.org/facebook/IT-CPE/builds/407917459?utm_source=github_status&utm_medium=notification

I don't think this is a duplicate assignment. This is a guard to ensure there isn't a nil value by accident. While your OR statement will work, it is technically not a guard.

Thanks for feedback!
The duplicate assignment is line 122 and 150 of cpe_chrome/resources/cpe_chrome.rb.
Wouldn't the ternary and the OR statement always evaluate to the same, given the same values for the rightside value (Facebook) and node['organization']? The ternary is just a verbose way of expressing || in this case.
2.5.1 :004 >  x = nil
 => nil
2.5.1 :005 > y = 'Facebook'
 => "Facebook"
2.5.1 :006 > x ? x : y
 => "Facebook"
2.5.1 :007 > x || y
 => "Facebook"
2.5.1 :008 > y = nil
 => nil
2.5.1 :009 > x ? x : y
 => nil
2.5.1 :010 > x || y
 => nil
2.5.1 :011 > x = 'Gusto'
 => "Gusto"
2.5.1 :012 > x ? x : y
 => "Gusto"
2.5.1 :013 > x || y
 => "Gusto"

In this particular case (strings) you are right that this code is functionally the same, however in other cases this will not be the case.
Ternary's evaluate the values and are expressed as a shortened if/else, whereas this is an or. While they are similar, my opinion is that the current methodology is safer and more accurate with what the author's expect the code to do.

@erikng For my own knowledge, mind supplying example code of the case you're referencing?
I'm failing to understand exactly what you're implying (not disputing, just would like to see the code to substantiate, what seems to be ☝ , a contentious statement).

Because node is always defined in the context of a chef run, if you try to access a key that doesn't exist, it will return nil:
[1] pry(main)> node = {}
=> {}
[2] pry(main)> node['doesnotexist']
=> nil

Of course, if node were not defined at all, we'd have more problems than this particular line (chef would not work).
With organization present:
[1] pry(main)> node = {}
=> {}
[2] pry(main)> node['organization'] = 'Blah Co'
=> "Blah Co"
[3] pry(main)> node['organization'] ? node['organization'] : 'Facebook'
=> "Blah Co"
[4] pry(main)> node['organization'] || 'Facebook'
=> "Blah Co"

Without:
[1] pry(main)> node = {}
=> {}
[2] pry(main)> node['organization'] ? node['organization'] : 'Facebook'
=> "Facebook"
[3] pry(main)> node['organization'] || 'Facebook'
=> "Facebook"

This is functionally identical and definitely cleaner.
The duplicate code on the chrome part looks like a copy paste error (as the first declaration is in scope for the second usage).
As for the chrome change, did you confirm that chrome canary still functions as expected? I'd like to see that functionality tested even though it looks fine.


Proxy error events

Together with https://github.com/andris9/buildmail/pull/2, it notifies users if an error event is triggered by the content stream.
Users can then handle this error accordingly.

This is fixed by now, errors from buildmail are passed upstream

Thanks!


GET, PUT and LS commands do not work with esp8266

GET, PUT and LS commands do not work with esp8266 via ampy.
Firmware: Micropython 1.9.4
Command: sudo ampy --port /dev/ttyUSB0 --baud 115200 ls
Traceback (most recent call last):
  File "/usr/local/bin/ampy", line 11, in <module>
    load_entry_point('adafruit-ampy==1.0.3', 'console_scripts', 'ampy')()
  File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 722, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 697, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 1066, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 895, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.6/dist-packages/click/core.py", line 535, in invoke
    return callback(*args, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/ampy/cli.py", line 147, in ls
    for f in board_files.ls(directory):
  File "/usr/local/lib/python3.6/dist-packages/ampy/files.py", line 103, in ls
    raise ex
  File "/usr/local/lib/python3.6/dist-packages/ampy/files.py", line 96, in ls
    out = self._pyboard.exec_(textwrap.dedent(command))
  File "/usr/local/lib/python3.6/dist-packages/ampy/pyboard.py", line 265, in exec_
    raise PyboardError('exception', ret, ret_err)
ampy.pyboard.PyboardError: ('exception', b'', b'Traceback (most recent call last):\r\n  File "<stdin>", line 6, in <module>\r\nAttributeError: no such attribute\r\n')

For me, it works running Ampy on Windows 7, Adafruit ESP8266 Huzzah.  Here is what I see with v1.9.4:
C:\test>ampy --version
ampy, version 1.0.3
C:\test>ampy -pCOM10 -d1 ls
boot.py
main.py
C:\test>ampy -pCOM10 -d1 get boot.py my_boot.py
Adding the "-d1" delay is critical for getting Ampy to work with my ESP8266 devices.

I downloaded micropython 1.9.4 again, erase flash and write. And now ... It works!
I dont now if I did something wrong on first time, but tried everything e nothing worked.
Anyway, thanks for your support!


Docs.MS Release Updates for azure-resourcemanager-cosmos

Update docs metadata and targeting for release of azure-resourcemanager-cosmos

Docs Build status updates of commit 1cf585a:
:clock10: Full build: the average full build time is 279 min(s) 9 sec(s), this is based on the last 923 build(s) for this repository.

Docs Build status updates of commit 1cf585a:
:warning: Validation status: warnings

File
Status
Preview URL
Details

legacy/docs-ref-autogen/overview/EventHubs/Client.yml
:warning:Warning
View (azure-java-legacy)
Details

docs-ref-services/resourcemanager-cosmos-readme.md
:white_check_mark:Succeeded
View (azure-java-stable) View (azure-java-preview) View (azure-java-legacy)

package.json
:white_check_mark:Succeeded

legacy/docs-ref-autogen/overview/EventHubs/Client.yml

[Warning-DuplicateUids] Uid(azure.java.sdk.landingpage.services.eventhub.Client) has already been defined in docs-ref-services/messaging-eventhubs-readme.md.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

merge to https://github.com/Azure/azure-docs-sdk-java/pull/1377


Fix nodeCount constant upgrade in KubernetesClusterNodePool and add azurerm_storage_container_immutability_policy

Description of your changes

Fixes #
I have:

[x] Read and followed Crossplane's contribution process.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Couldn't run the whole make reviewable because it kept crashing my machine but I run make generate and tested all the changes I did here manually in our tenant. Fixes #636 and closes #804


 Since it traverses all indexes without considering presence, this seems to be causing it to terminate iteration early if there is a missing child in the list.

I would like to add unit tests for this change, but it looked like build-script -t wasn't causing the tests in test/SwiftSyntax to build or run. Is there a flag I'm missing?

I forgot, -t goes to --test, right? In that case, it sounds like a bug. @harlanhaskins your thoughts? If it is a bug, please fix in a different PR, then loop back to this one with tests.

Hmm...I checked for those tests when I ran CI when we fixed the build. Are you testing on macOS? Also, thanks for finding this bug!

Sorry about that—false alarm! I must have missed the output when I ran the full test suite, and I'm still getting used to the process to run individual tests documented in Testing.md, so when it says to pass a build-relative path like ${SWIFT_BUILD_DIR}/test-macosx-x86_64/SwiftSyntax/ to lit.py I was expecting there to be actual files there 🙂  There aren't, but it turns out it still executes the tests.
I've pushed a new commit that adds some basic tests for the iterator.

@swift-ci please smoke test

This LGTM. Thanks for this!


[7.x] Allow store resource into postgresql bytea

Allowing store files into postgresql database with bytea column type. You can simple use:
\DB::table('table')->insert([ 'bytea_column' => fopen('filename', 'r') ]);

You can simple storing files from user upload etc...

We'll need much more info on why this is needed.

In Postgresql there is bytea column where you can store variable-length binary string (more information: postgres datatype binary. So you can have input with type file on page, where user uploading files. There are no needs to convert uploaded file to base64 or others formats, because you can store it in native postgresql way as bytea. But when you try store files like pdf etc, it fails, becase PARAM_STR not supporting special chars (I tried this, but because it is not working I went deep into problem and found this solution).
You can argue that files can be stored on a server outside the database, but trust me, this is not solution for me. Using PARAM_LOB is standard way (see the php documentation). This is not hacking or tricky way.
I think this cannot be used in Connection.php, because for example mssql needs aditional params (which is demonstrate here: mssql examples).
//ms sql way
$tsql = "INSERT INTO Production.ProductPhoto (LargePhoto)
 VALUES (?)";
$uploadPic = $conn->prepare($tsql);
$fileStream = fopen($_FILES['file']['tmp_name'], "r");
$uploadPic->bindParam(1,
  $fileStream,
  PDO::PARAM_LOB,
  0,
  PDO::SQLSRV_ENCODING_BINARY);
$uploadPic->execute();

This means that if this patch is not acceptable, I think I can create patch for all databases, but for me, it may be hard to test it (I haven't mssql database). So, please, if you not accept it, tell me, what I can do for acceptable patch.
Which solved this patch for postgresql:
//before patch you can try this, but it should fail (for example txt file can work because there are no mostly special chars).
\DB::table('table')->insert([ 'bytea_column' => stream_get_contents('filename') ]);

Because PARAM_LOB is for resources, I created a condition with is_resource in this commit, so you can simple do which I wrote in first comment and it work like a charm, without any problems.


feat(sqlalchemy): extend SQLAlchemy Plugin to include Alembic

Pull Request Checklist

[ ] New code has 100% test coverage
[ ] (If applicable) The prose documentation has been updated to reflect the changes introduced by this PR
[ ] (If applicable) The reference documentation has been updated to reflect the changes introduced by this PR

Description

Close Issue(s)

I still need to add tests, but I'm going to remove the draft status on this one.  I don't think there's anything additional features to add here.

In commit cffc81b, the addition of _namespace_metadata=op._namespace_metadata to order_columns addresses the AttributeError: 'NoneType' object has no attribute 'schema' issue. This workaround will be necessary until the resolution of sqlalchemy/alembic issue #1193 to ensure that custom column sorting within CREATE TABLE works.

Is there any reason why this was closed and the branch deleted? It appears a significant amount of work was put into this and it was just about done unless I'm misunderstanding something here.

@Faolain - cofin stepped down as maintainer. You can see the discussion in our discord server.
I restored the branch for now.


Add a command

Use a command to install files (instead of automatically installing stuff)... this means that composer plugin is going to be removed.

done with bin/jawira-skeleton


Web(canvaskit) + iOS 15 will not show anything If use Hiragana.

Text that should be displayed when opening the web using iOS 15 is not displayed.
Steps to Reproduce

Run flutter create bug.
Update the lib/main.dart as follows:

import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({Key? key}) : super(key: key);

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        primarySwatch: Colors.blue,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({Key? key, required this.title}) : super(key: key);

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text(widget.title),
      ),
      body: Center(
        child: Text(
          'あ' * 50,
          style: const TextStyle(
            fontSize: 16,
            height: 1.4,
          ),
        ),
      ),
    );
  }
}

Run flutter build web --target=lib/main.dart --web-renderer canvaskit
Open index.html in Safari on iOS 15 (iPhone 13).

Expected results:
Expect to see the Flutter Demo Home Page and あ * 50.
Actual results:
Nothing is displayed.

What we know.

There is no error log in Safari's devtool.
If あ * 20, it will be displayed
This does not happen on Chrome on Mac or Android.
The issue was not reproduced on my iOS 14.7.1 (iPhone 7).

Logs

Logs
 $ flutter build web --target=lib/main.dart --web-renderer canvaskit --verbose
[  +87 ms] executing: sysctl hw.optional.arm64
[  +14 ms] Exit code 1 from: sysctl hw.optional.arm64
[        ] sysctl: unknown oid 'hw.optional.arm64'
[   +5 ms] executing: [/Users/hisaichi5518/flutter/] git -c log.showSignature=false log -n 1 --pretty=format:%H
[  +15 ms] Exit code 0 from: git -c log.showSignature=false log -n 1 --pretty=format:%H
[        ] ffb2ecea5223acdd139a5039be2f9c796962833d
[        ] executing: [/Users/hisaichi5518/flutter/] git tag --points-at ffb2ecea5223acdd139a5039be2f9c796962833d
[  +31 ms] Exit code 0 from: git tag --points-at ffb2ecea5223acdd139a5039be2f9c796962833d
[        ] 2.5.1
[   +9 ms] executing: [/Users/hisaichi5518/flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[  +14 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/stable
[        ] executing: [/Users/hisaichi5518/flutter/] git ls-remote --get-url origin
[  +14 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[ +102 ms] executing: [/Users/hisaichi5518/flutter/] git rev-parse --abbrev-ref HEAD
[  +17 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] stable
[   +5 ms] executing: sw_vers -productName
[  +13 ms] Exit code 0 from: sw_vers -productName
[        ] macOS
[        ] executing: sw_vers -productVersion
[  +13 ms] Exit code 0 from: sw_vers -productVersion
[        ] 11.6
[        ] executing: sw_vers -buildVersion
[  +14 ms] Exit code 0 from: sw_vers -buildVersion
[        ] 20G165
[  +62 ms] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterWebSdk' is not required, skipping update.
[   +4 ms] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'WindowsUwpEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[  +45 ms] Artifact Instance of 'MaterialFonts' is not required, skipping update.
[        ] Artifact Instance of 'GradleWrapper' is not required, skipping update.
[        ] Artifact Instance of 'AndroidGenSnapshotArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'AndroidInternalBuildArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterSdk' is not required, skipping update.
[        ] Artifact Instance of 'WindowsEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'WindowsUwpEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxEngineArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'LinuxFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'MacOSFuchsiaSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerSDKArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FlutterRunnerDebugSymbols' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'IosUsbArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'FontSubsetArtifacts' is not required, skipping update.
[        ] Artifact Instance of 'PubDependencies' is not required, skipping update.
[  +50 ms] Skipping pub get: version match.
[ +109 ms] Generating /Users/hisaichi5518/src/github.com/10xinc/stailer-client/testapp/android/app/src/main/java/io/flutter/plugins/GeneratedPluginRegistrant.java
[  +68 ms] 💪 Building with sound null safety 💪
[  +36 ms] Compiling lib/main.dart for the Web...
[   +7 ms] Initializing file store
[   +4 ms] Done initializing file store
[  +53 ms] Skipping target: web_entrypoint
[   +2 ms] Skipping target: gen_localizations
[ +255 ms] Skipping target: dart2js
[  +48 ms] Skipping target: web_release_bundle
[   +8 ms] Skipping target: web_service_worker
[   +1 ms] Persisting file store
[   +6 ms] Done persisting file store
[  +15 ms] Compiling lib/main.dart for the Web... (completed in 393ms)
[        ] "flutter web" took 758ms.
[ +197 ms] ensureAnalyticsSent: 194ms
[   +1 ms] Running shutdown hooks
[        ] Shutdown hooks complete
[        ] exiting with code 0

 $ flutter analyze
Analyzing testapp...
No issues found! (ran in 1.3s)

 $ flutter doctor -v
[✓] Flutter (Channel stable, 2.5.1, on macOS 11.6 20G165 darwin-x64, locale ja)
    • Flutter version 2.5.1 at /Users/hisaichi5518/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision ffb2ecea52 (8 days ago), 2021-09-17 15:26:33 -0400
    • Engine revision b3af521a05
    • Dart version 2.14.2

[!] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
    • Android SDK at /Users/hisaichi5518/Library/Android/sdk
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for more details.

[✓] Xcode - develop for iOS and macOS
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 13.0, Build version 13A233
    • CocoaPods version 1.11.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Users/hisaichi5518/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7678000/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] Android Studio (version 2020.3)
    • Android Studio at /Users/hisaichi5518/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7678000/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] Android Studio (version 2020.3)
    • Android Studio at /Users/hisaichi5518/Library/Application Support/JetBrains/Toolbox/apps/AndroidStudio/ch-0/203.7621141/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] IntelliJ IDEA Ultimate Edition (version 2021.2.2)
    • IntelliJ at /Users/hisaichi5518/Applications/JetBrains Toolbox/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 60.1.4
    • Dart plugin version 212.5284.31

[✓] IntelliJ IDEA Ultimate Edition (version 2021.2.2)
    • IntelliJ at /Users/hisaichi5518/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/212.5284.40/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] IntelliJ IDEA Ultimate Edition (version 2021.2.1)
    • IntelliJ at /Users/hisaichi5518/Library/Application Support/JetBrains/Toolbox/apps/IDEA-U/ch-0/212.5080.55/IntelliJ IDEA.app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.60.2)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.26.0

[✓] VS Code (version 1.55.2)
    • VS Code at /Users/hisaichi5518/Dropbox/My Mac (MacBook Pro)/Downloads/Visual Studio Code.app/Contents
    • Flutter extension version 3.26.0

[✓] Connected device (1 available)
    • Chrome (web) • chrome • web-javascript • Google Chrome 93.0.4577.82

! Doctor found issues in 1 category.

refs https://github.com/flutter/flutter/issues/90705

If I turn off Safari's experimental feature GPU Process: Canvas Rendering and set web-renderer to html, it works as expected.

refs https://github.com/flutter/flutter/issues/89655
refs https://github.com/flutter/flutter/issues/89655#issuecomment-919685843


Segmentation fault in MPIGetCellListInSearchUsingMesh in EAGLE-XL DMONLY runs

I'm trying to run a 300Mpc DMONLY box using Swift with on-the-fly Velociraptor. I'm using revision 5c22b31faa189beddd639d24a94998fa34e41cd0 of Velociraptor with the sample_swiftdm_3dfof_subhalo.cfg configuration from the repository. I've configured it with
cmake ..

The problem occurs in the function MPIGetCellListInSearchUsingMesh() on this line:
                if (ignorelocalcells && opt.cellnodeids[index]==ThisTask) continue;

The expression opt.cellnodeids[index] evaluates to an invalid address, which causes a segmentation fault. The variable index is zero when it crashes. This first happened on the 300Mpc box running on Irene but I can reproduce it on a 75Mpc box on Cosma.

Hi John, how are the mpi domains being set in swift? Is it using metis? I haven't encountered the same issue but have just used swift with a max of 16 mpi domains in current tests. If there is an issue with opt.cellnodeids it suggests that there is something amiss with how they are set in swift and passed to vr.

I'm using parmetis for these runs. I have it running in ddt now so I'll try to see where this bad value of cellnodeids is coming from.

I think I've found the problem. When I run with VR_NO_MASS the mass element is omitted from Velociraptor's struct swift_vel_part in SwiftParticle.h. But on the Swift side there's no #ifdef and we don't know about VR_NO_MASS anyway so Swift's struct swift_vel_part in swift_velociraptor_part.h still contains a mass entry.

Could we just always leave the mass entry in the struct whether it's used or not? Or should swift know about velociraptors configuration somehow?

Happy either way. We can discuss it with Pascal tomorrow morning.

I'll just comment out the #ifdef for now and see if that gets the 300Mpc box going again.

Looks like there are some fixes needed on the Swift side too. The siminfo struct doesn't match between swift and velociraptor in the latest master branches and I think the particle mass is not set anywhere with VR_NO_MASS=ON because the mass_uniform_box entry in siminfo doesn't exist in swift.

It seems to get further with the following changes:

Remove the #ifdef NOMASS in SwiftParticle.h
Remove numcellsperdim and mass_uniform_box from struct siminfo
Set MassValue in InitVelociraptorHydro
I'll make a pull request with a tidied up version of these fixes.

I have pushed the branch VR_detection in SWIFT to fix the issue.
The latest VR master exposes functions in the libary depending on how it was compiled that the SWIFT configure script uses to decide whether to compile the SWIFT-VR interface in NOMASS mode or not.

For Matthieu's fix to work we need this change too - #68.

This seems to be fixed now.


[3 of 3: Mobile Page Header Navigation] DCOS-12571: Implement mobile page header navigation in Service Form

1 of 3: #1850
2 of 3: #1872
3 of 3: #1873
This PR implements the mobile page header navigation component in the service form.

Diff LGTM.

Please rebase @jfurrow :-)

@MatApple Rebased :)

Still flicts!

@mlunoe I'm going to wait until 1 and 2 of this PR series merge, I have rebased this too many times already.

Can one of the admins verify this patch?

@mlunoe I reassigned this PR to you, I hope you don't mind!

Ping @orlandohohmeier @mlunoe

This works great too! :shipit:

I did notice this issue: .
Could we handle that in a separate issue/PR?


proxy configuration for IoTEdge

Add proxy configuration for IoTEdge and docker runtimes along with the linux environment

for #171


fix(alertList): set correct type

fixes: #201

Type is still generated as "alertList" with an upper case L instead of a lower case like so : "alertlist"
Thanks !

🤦♂


Automate Unrealized P/L Test

We've automated realized P/L tests but not any unrealized ones.

I think I found a P/L bug while automating this test: https://github.com/AugurProject/augur/pull/7670/files#diff-d1006467ba28f2a4debbe4142ebaa922R513
This passes but it shouldn't: B's profit/loss should be unrealized.

Nope not a bug. I misunderstood the test description, plus there was a typo.


Testing - Headless Menus

Menus are being refactored. Adding this issue as a reference on our testing approach. We can pick this up again if we have a managed sprint focused on testing. Closing/archiving.


Update documentation URLs

https://docs.ghost.org/docs page is no longer available so the link leads to 404 error. Other links were updated where appropriate too.
I'm not sure about https://docs.ghost.org/docs/upgrade link which leads now to question about specific version (https://docs.ghost.org/faq/upgrade-to-ghost-2-0) so I left it as it is.

@kirrg001, thanks for your review, the commit has been amended.


[FEATURE] Static Code Analysis

[x] have mypy configured
[x] have mypy run in tox
[x] have mypy run in CI/CT for pull requests & commits to master

Complete in 0.4.0.


error "This argument does not satisfy the let/app invariant"

This is with hermit version 1.0.1 and GHC 7.10.3 on Linux. Program is simple:
x = 2 :: Integer
y = 1 / fromIntegral x :: Double

main = print y

I ran the follow hermit session:
hermit test.hs
[starting HERMIT v1.0.1 on test.hs]
% ghc test.hs -fforce-recomp -O2 -dcore-lint -fsimple-list-literals -fexpose-all-unfoldings -fplugin=HERMIT -fplugin-opt=HERMIT:*:
[1 of 1] Compiling Main             ( test.hs, test.o )
module Main where
  y ∷ Double
  main ∷ IO ()
  main ∷ IO ()
hermit<0> rhs-of 'y
(/) ▲ $fFractionalDouble (D# 1.0) (fromIntegral ▲ ▲ $fIntegralInteger $fNumDouble (__integer 2))
hermit<1> repeat (any-bu specialize >>> bash)
case doubleFromInteger (__integer 2) of wild ▲
  _ →
    case (/##) 1.0 (doubleFromInteger (__integer 2)) of wild2 ▲
      _ → D# wild2
hermit<2>
*** Core Lint errors : in result of Core plugin:  HERMIT0 ***
<no location info>: Warning:
    In the expression: /## 1.0 (doubleFromInteger (__integer 2))
    This argument does not satisfy the let/app invariant:
      doubleFromInteger (__integer 2)
*** Offending Program ***
y :: Double
[LclId, Str=DmdType]
y =
  case doubleFromInteger (__integer 2) of _ [Occ=Dead] { __DEFAULT ->
  case /## 1.0 (doubleFromInteger (__integer 2))
  of wild2_a1q9 { __DEFAULT ->
  D# wild2_a1q9
  }
  }

main :: IO ()
[LclIdX, Str=DmdType]
main = print @ Double $fShowDouble y

main :: IO ()
[LclIdX, Str=DmdType]
main = runMainIO @ () main

*** End of Offense ***

<no location info>:
Compilation had errors

Is it related to issue #139? Does changing GHC version help?

Thanks. We'll look into this.

It turns out if
  D# ((/##) 1.0 (doubleFromInteger (__integer 2)))

is rewritten as:
  case case doubleFromInteger (__integer 2) of w' ▲
         _ → (/##) 1.0 w'
   of z' ▲
    _ → D# z'

Then lint will succeed. But doing so manually is rather tiresome. I tried the following with the above program, and it works (not sure about the smash part in the proof, but whatever works!)
rhs-of 'y
repeat ((any-bu specialize) >>> smash)
application-of 'doubleFromInteger
{ let-intro 'z; let-body; case-intro-seq 'z; case-expr; inline 'z; up; up; let-elim }
up
case-float-arg-lemma A
-- recording obligations as lemmas : A
smash
end-proof -- proven A
{ let-intro 'z; let-body; case-intro-seq 'z; case-expr; inline 'z; up; up; let-elim }
up
case-float-arg-lemma B
-- recording obligations as lemmas : B
smash
end-proof -- proven B
up

Turning the "troubled" arg into case was relatively straightforward, but since there are two levels of nested application (/##) and D#, case-float-arg has to be done twice.
Would appreciate if there is a more automatic way to fix this kind of error, or better yet if we can avoid violating the let/app rule in case elimination in the first place.

Given: https://github.com/ghc/ghc/blob/master/compiler/coreSyn/CoreSyn.hs#L375
My guess is that some rewrite is building the application with an explicit App constructor rather than using the mkCoreApp smart constructor.
Can you narrow down when exactly that expression gets built?

Here is a log after I set-auto-lint True and use bash-debug:
...
[case-reduce (before)]
case D# (doubleFromInteger (__integer 2)) of wild1 ▲
  D# y →
    case (/##) 1.0 y of wild2 ▲
      _ → D# wild2
[case-reduce (after)]
case (/##) 1.0 (doubleFromInteger (__integer 2)) of wild2 ▲
  _ → D# wild2
<1> <no location info>: Warning:
    In the expression: /## 1.0 (doubleFromInteger (__integer 2))
    This argument does not satisfy the let/app invariant:
      doubleFromInteger (__integer 2)
Error in expression: repeat ((any-bu specialize) >>> bash-debug)
Core Lint Failed:
<no location info>: Warning:
    In the expression: /## 1.0 (doubleFromInteger (__integer 2))
    This argument does not satisfy the let/app invariant:
      doubleFromInteger (__integer 2)

The issue seems to be a result of using substExpr in the implementation of caseReduceDataconR. So a valid App f x becomes invalid because x is substituted. Not sure what is the best way to guard this. Maybe rule out the situation when the "substitutee" is of an unboxed type?


Using rows: 2 and on responsive row: 1 doesn't work

I need to have 1 row when going responsive < 680 px and show 2 items per row.
Somehow that doesn't work.
I have made a JSFiddle;
https://jsfiddle.net/b5bqk68j/925/
(refresh when on < 680 px wide)
My Slick call;
$('.slider-4st').slick({
	dots: false,
	arrows: true,
	nextArrow: '
»
',
	prevArrow: '
«
',
	infinite: false,
	speed: 300,
	slidesPerRow: 4,
	rows: 2,
	adaptiveHeight: true,
	//slidesToShow: 4,
	//slidesToScroll: 4,
	responsive: [
		{
			breakpoint: 1024,
			settings: {
				slidesPerRow: 3
			}
		},
		{
			breakpoint: 680,
			settings: {
				rows: 1,
				slidesPerRow: 2
			}
		},
		{
			breakpoint: 480,
			settings: {
				rows: 1,
				slidesPerRow: 1
			}
		}
	]
});

I just want to confirm that @rosivanov 's suggestion to use the un-minified 1.8.1 resolved the issue for me. Thanks.

So with Slick 1.8.1 fixed on the un-minified version still you can't do rows on responsive. One workaround is to have the mobileFirst: true on the initial settings and add the row settings there, then have a separate option on the responsive for 1024 where you default back.
@claudiuTicu 's example is the way to do it.

Still facing this issue, sad to see the plugin isn't maintained anymore.

As suggested from @rosivanov with .js version is ok.
You can minify the .js version and use it.

Had same problem. rows: 1 along with slidesperRow: 2 had strange effect on slides
Just switching to unmified version of slick.js resolved issue

I just want to confirm that @x35a https://github.com/kenwheeler/slick/issues/3277#issuecomment-357706063 reply is working fine. The same fix also apply works for 1.8.0.

I'm very late to the party but I am facing this issue as well. I appreciate some of the hints and tips given above.
There is codepen at https://codepen.io/booom/pen/dyYjREp which I started playing with to see if it was something I was doing locally that was breaking it and stopping rows working in responsive mode.
What I wanted was (mobile first) 1x1 slideshow on mobile, 2rowsx1col @ min-width:576px and 2x2 @ min-width:992px.
With some adjustments to the jQuery on the right, I got exactly it working. Rows do work on responsive.
If you go to the Codepen link, swap in the following code to the right JS panel and wait for the bottom slider to update, drag the window width, to the breakpoints I mentioned and then you will see what I mean. It works.
$('#slick1').slick({ 		mobileFirst: true, 		dots: false, 		arrows: true, 		infinite: true, 		speed: 300, 		slidesToShow: 1, 		slidesToScroll: 1, 		responsive: [ 			{ 				breakpoint: 576, 				settings: { 					rows: 2, 					slidesToShow: 1, 					slidesToScroll: 2 				} 			}, 			{ 				breakpoint: 992, 				settings: { 					rows: 2, 					slidesToShow: 2, 					slidesToScroll: 2 			  } 			} 		] });
So, I have it working in the CodePen but not in my site.. GRRRR!!!!!!
Life goes on.
Peace and propz
muzKore

I'm very late to the party but I am facing this issue as well. I appreciate some of the hints and tips given above.
There is codepen at https://codepen.io/booom/pen/dyYjREp which I started playing with to see if it was something I was doing locally that was breaking it and stopping rows working in responsive mode.
What I wanted was (mobile first) 1x1 slideshow on mobile, 2rowsx1col @ min-width:576px and 2x2 @ min-width:992px.
With some adjustments to the jQuery on the right, I got exactly it working. Rows do work on responsive.
If you go to the Codepen link, swap in the following code to the right JS panel and wait for the bottom slider to update, drag the window width, to the breakpoints I mentioned and then you will see what I mean. It works.
$('#slick1').slick({ 		mobileFirst: true, 		dots: false, 		arrows: true, 		infinite: true, 		speed: 300, 		slidesToShow: 1, 		slidesToScroll: 1, 		responsive: [ 			{ 				breakpoint: 576, 				settings: { 					rows: 2, 					slidesToShow: 1, 					slidesToScroll: 2 				} 			}, 			{ 				breakpoint: 992, 				settings: { 					rows: 2, 					slidesToShow: 2, 					slidesToScroll: 2 			  } 			} 		] });
So, I have it working in the CodePen but not in my site.. GRRRR!!!!!!
Life goes on.
Peace and propz
muzKore

2022 still problem with rows in breakpoints
even swiper doesnt work

In slick.min.js its broken, in slick.js - all good. Just use unminified version.

Unminified does not work for me, still breaks when using rows 1, slidesPerRow: 1 in a responsive setting.
Using mobileFirst: true just completely breaks the HTML and slide when going up sizes then. For some reason it strips out the slide HTML and messed stuff up big time.


Beyond Zork: Map displayed incorrectly

Ezekiel000 opened 2011-11-06:
The map in Beyond Zork and the area description aren't shown correctly at the moment. I think bacause the symbol or font3 font isn't implemented and the area description is incorrect because it relies on fix-width fonts.

@wmvanvliet commented 2012-09-15:

Looks like it demands a certain width of the window. GLK has no support for such a thing I believe.

@ptomato commented 2012-09-16:

We've added Bocfel as a possible interpreter to use. It supports the symbol font, so Beyond Zork will at least work on that.

@ptomato closed 2012-09-16


IRONN-210 allow for change of withdrawal dates

This change allows for an update to an existing withdrawn consent.  Previously, a 404 was generated if the user didn't have a valid matching consent when a withdrawal was requested.
It is required that the requested withdrawal date is:

greater than the last valid consent for the research study
less than 1 day in the future (to allow for any timezone details)

Hello @pbugni! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file portal/views/user.py:

Line 879:80: E501 line too long (85 > 79 characters)
Line 883:80: E501 line too long (95 > 79 characters)

grr, the select for review didn't go through last night, sorry 'bout that.


LPC55S69 NS target refactor

Refactoring LPC55S69 non-secure target by moving partitioning so _ns version can use it.

LGTM, but would like @d3zd3z to take a look at the flash partitions bits.

sorry, i am wrong, MuSca board is using the _nonsecure suffix. the 'ns' is used in nrf5340, nrf9160 (and friends).

So Musca and AN521 boards use _nonsecure, Nordic uses _ns, both are merged, so we should clearly pick one or the other and generate a PR to harmonize naming. Any votes/opinions, then? I don't have any strong feelings myself, but we should use the right one in this PR at least.

So Musca and AN521 boards use _nonsecure, Nordic uses _ns, both are merged, so we should clearly pick one or the other and generate a PR to harmonize naming. Any votes/opinions, then? I don't have any strong feelings myself, but we should use the right one in this PR at least.

I lean toward _ns because _nonsecure is a little long, but I don't have a strong preference.

So Musca and AN521 boards use _nonsecure, Nordic uses _ns, both are merged, so we should clearly pick one or the other and generate a PR to harmonize naming. Any votes/opinions, then? I don't have any strong feelings myself, but we should use the right one in this PR at least.

I lean toward _ns because _nonsecure is a little long, but I don't have a strong preference.

I agree shorter is likely better, and can put a PR together renaming the AN521 board and documentation, and @agansari can revert to the original _ns here (sorry Andrei).

@microbuilder @MaureenHelm @ioannisg i'll wait for a decision regarding naming conventions, dropping the renaming commit is easy.

@agansari Everyone seems to prefer the shorter _ns so lets stick with that and I'll update the musca and AN521 boards accordingly.

@microbuilder done! I've dropped that commit.

@agansari You might want to amend the commit message from boars to boards 🐗

@ioannisg can you have another look at this pull?

not required anymore, will be replaced by #27780


Creating mkdocs website for documentation

Hi there, would you be interested in the realization for a mkdocs page to use as documentation for this project? I'd love to help and make it!

@devfernandoa sure !


💬 [Question] End Meeting on Welcome Table

💬 Question

End Meeting on Welcome Table.

It is possible to end the meeting for all users on the welcome table. This can allow users to disrupt this kind of meeting.
Do we want to make all users attendes except the administrators which could still be moderators?
Downsides?

Decisions:

Users are attendees
clarify if users can draw on the board as attendee

Closing for https://github.com/dreammall-earth/dreammall.earth/issues/2470


[cycle-dom] transposition not working as expected

When using stream transposition in vtree, along with {textContent: '...'}, the last child in array is not re-rendering.
https://jsbin.com/cucecoyome/1/edit?js,console,output
function main(sources) {
  const interval$ = Observable.interval(1000)
    .timeInterval()
    .take(10)
    .map(({value}) => div('value ' + value))
    .share();

  return {
    DOM: Observable.of(1)
      .map(() => {
        return div(
          // commenting out the next line make the intervals$
          // showing properly
          {textContent: 'text'},
          [
            interval$,
            interval$,
            interval$
          ]
        );
      })
  }
}

the result:
text
value 9
value 9
value 0

expected result (it's working like this when {textContent...} is commented out) :
text
value 9
value 9
value 9

Why do you want to set content using {textContent: 'text'} and then fill it with children?

That is probably a wrong use of vDom, so this bug could be very low priority. Anyway, it's interesting that only the last stream is not updating. That could be confusing for some beginners (it's hard to find the detailed documentation of cycle-dom).

Seems to work fine with xstream v4, @cycle/xstream-run and @cycle/dom v10.0.0-rc28
http://www.webpackbin.com/N1rvi2A7W


Update esp-hal packages

Blocking on #296 because... well, examples need a lot of work.

@MabezDev should I close this then? I see you did everything in #296 :D

Yeah #296 was blocked on esp-hal changes so I inadvertently have done the upgrade anyways :D. Closing this.


[feature] store captures on S3 instead of GCP

I'm interested in using this with an S3-compatible storage (@Minio) on bare metal, rather than GCP storage.
Any suggestions on where to start?

To add support for any storage for vhs you need to

Implement the flow.Source and flow.Sink interfaces. (Currently, flow.Sink is just an io.WriteCloser).
Add the new source and sink to the default parser.

It probably makes the most sense to implement these with https://github.com/minio/minio-go since we could use the same source/sink to work with S3 as well.


[RFC] New Design

Replace components with new design

Do you require help with the UI design?

The design is finished, I'll replace the website with new design soon. But PR welcome to improve it.


Change comment

@austin-hoover  Looks like this PR combines the fix about transposed matrix comment and envelope solver.
You also submitted #44 that deals with envelop solver only.
Can you please rs-submit this PR so it has only comment change?
This will make these tow modifications clearly decoupled.

Sorry, didn't mean to do that. Let me re-submit.


misc: logger, mode permission and storages

logger f

a := "world"
logger.Print(a)
logger.Printf("hello %v!", a)
logger.Println(a)
logger.Debug(a)
logger.Debugf("hello %v!", a)
logger.Info(a)
logger.Infof("hello %v!", a)
logger.Warn(a)
logger.Warnf("hello %v!", a)
logger.Error(a)
logger.Errorf("hello %v!", a)
logger.Fatalf("hello %v!", a)

more logs
change cycler config file permission
fixed model.Storages store_with bug
add GOBACKUP_DIR env

Use temp dir as default workdir instead of a fixed /tmp/gobackup
Before
$ sudo ./gobackup perform
2022/12/03 01:30:48 [Modal: demo] WorkDir: /tmp/gobackup/1670059848802701471/demo
2022/12/03 01:30:48 [Database] => database | postgresql : postgresql
2022/12/03 01:30:48 [PostgreSQL] -> Dumping PostgreSQL...
2022/12/03 01:30:49 [PostgreSQL] dump path: /tmp/gobackup/1670059848802701471/demo/postgresql/postgresql/demo.sql
2022/12/03 01:30:49 [Database]
2022/12/03 01:30:49 [Compressor] => Compress | xz
2022/12/03 01:30:49 [Compressor] -> /tmp/gobackup/1670059848802701471/2<IP_ADDRESS>.30.49.tar.xz
2022/12/03 01:30:49 [Storage] => Storage | local
2022/12/03 01:30:49 [Local] Store successed /tmp/demo1
2022/12/03 01:30:49 [Model] Cleanup temp: /tmp/gobackup/1670059848802701471/

$ ./gobackup perform
2022/12/03 01:30:57 [Modal: demo] WorkDir: /tmp/gobackup/1670059857936056064/demo
2022/12/03 01:30:57 [Database] => database | postgresql : postgresql
2022/12/03 01:30:57 [PostgreSQL] -> Dumping PostgreSQL...
2022/12/03 01:30:58 [debug] /usr/bin/pg_dump --host=localhost --port=5432 --username=postgres demo -f /tmp/gobackup/1670059857936056064/demo/postgresql/postgresql/demo.sql
2022/12/03 01:30:58 [Modal: demo] pg_dump: error: could not open output file "/tmp/gobackup/1670059857936056064/demo/postgresql/postgresql/demo.sql": No such file or directory

2022/12/03 01:30:58 [Model] Cleanup temp: /tmp/gobackup/1670059857936056064/

After
$ mkdir /tmp/demo
$ sudo ./gobackup perform
2022/12/03 02:56:23 [Modal: demo] WorkDir: /tmp/gobackup4023818121/1670064983565757453/demo
2022/12/03 02:56:23 [Database] => database | postgresql: postgresql
2022/12/03 02:56:23 [PostgreSQL] -> Dumping PostgreSQL...
2022/12/03 02:56:23 [PostgreSQL] dump path: /tmp/gobackup4023818121/1670064983565757453/demo/postgresql/postgresql/demo.sql
2022/12/03 02:56:23 [Database]
2022/12/03 02:56:23 [Compressor] => Compress | xz
2022/12/03 02:56:23 [Compressor] -> /tmp/gobackup4023818121/1670064983565757453/2<IP_ADDRESS>.56.23.tar.xz
2022/12/03 02:56:23 [Storage] => Storage | local
2022/12/03 02:56:23 [Local] Store succeeded /tmp/demo
2022/12/03 02:56:23 [Model] Cleanup temp: /tmp/gobackup4023818121/

$ ./gobackup perform
2022/12/03 02:56:26 [Modal: demo] WorkDir: /tmp/gobackup2923488212/1670064986188901362/demo
2022/12/03 02:56:26 [Database] => database | postgresql: postgresql
2022/12/03 02:56:26 [PostgreSQL] -> Dumping PostgreSQL...
2022/12/03 02:56:26 [PostgreSQL] dump path: /tmp/gobackup2923488212/1670064986188901362/demo/postgresql/postgresql/demo.sql
2022/12/03 02:56:26 [Database]
2022/12/03 02:56:26 [Compressor] => Compress | xz
2022/12/03 02:56:26 [Compressor] -> /tmp/gobackup2923488212/1670064986188901362/2<IP_ADDRESS>.56.26.tar.xz
2022/12/03 02:56:26 [Storage] => Storage | local
2022/12/03 02:56:26 [Local] Store succeeded /tmp/demo
2022/12/03 02:56:26 [Model] Cleanup temp: /tmp/gobackup2923488212/

Added prestart, poststart, always_poststart configs for database.
Scenario:

SELECT pg_wal_replay_resume(); to pause replay in PostgreSQL replica
pg_dump
SELECT select pg_wal_replay_resume(); to resume replay in PostgreSQL replica no matter pg_dump succeeds or not

2022/12/03 04:09:31 [Model: demo] WorkDir: /tmp/gobackup1781575701/1670069371448053572/demo
2022/12/03 04:09:31 [Database] => database | postgresql: postgresql
2022/12/03 04:09:31 [Database] Run dump prestart command
2022/12/03 04:09:31 [Database] Dump prestart command succeeded
2022/12/03 04:09:31 [PostgreSQL] -> Dumping PostgreSQL...
2022/12/03 04:09:31 [debug]  /usr/bin/pg_dump --host=localhost --port=5432 --username=postgres demo -f /tmp/gobackup1781575701/1670069371448053572/demo/postgresql/postgresql/demo.sql
2022/12/03 04:09:31 [Database] Dump failed
2022/12/03 04:09:31 [Database] always_poststart is true, start to run post start command
2022/12/03 04:09:31 [Database] Run dump poststart command
2022/12/03 04:09:31 [Database] Dump poststart command succeeded
2022/12/03 04:09:31 [Model: demo] pg_dump: error: connection to server at "localhost" (::1), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?
connection to server at "localhost" (<IP_ADDRESS>), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?

2022/12/03 04:09:31 [Model] Cleanup temp: /tmp/gobackup1781575701/

rename hooks

prestart => before
poststart => after
always_poststart => on_exit: always, success, failure

before + after + always on_exit
2022/12/03 18:48:54 [Model: demo] WorkDir: /tmp/gobackup3821489654/1670122134570844525/demo
2022/12/03 18:48:54 [Database] => database | postgresql: postgresql
2022/12/03 18:48:54 [Database] Run dump before hooks
2022/12/03 18:48:54 [Database] Dump before hooks succeeded
2022/12/03 18:48:54 [PostgreSQL] -> Dumping PostgreSQL...
2022/12/03 18:48:54 [debug]  /usr/bin/pg_dump --host=localhost --port=5432 --username=postgres demo -f /tmp/gobackup3821489654/1670122134570844525/demo/postgresql/postgresql/demo.sql
2022/12/03 18:48:54 [Database] Dump failed
2022/12/03 18:48:54 [Database] on_exit is always, start to run after hooks
2022/12/03 18:48:54 [Database] Run dump after hooks
2022/12/03 18:48:54 [Database] Dump after hooks succeeded
2022/12/03 18:48:54 [Model: demo] pg_dump: error: connection to server at "localhost" (::1), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?
connection to server at "localhost" (<IP_ADDRESS>), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?

2022/12/03 18:48:54 [Model] Cleanup temp: /tmp/gobackup3821489654/

before + after + success on_exit
2022/12/03 18:49:21 [Model: demo] WorkDir: /tmp/gobackup4250935195/1670122161319166282/demo
2022/12/03 18:49:21 [Database] => database | postgresql: postgresql
2022/12/03 18:49:21 [Database] Run dump before hooks
2022/12/03 18:49:21 [Database] Dump before hooks succeeded
2022/12/03 18:49:21 [PostgreSQL] -> Dumping PostgreSQL...
2022/12/03 18:49:21 [debug]  /usr/bin/pg_dump --host=localhost --port=5432 --username=postgres demo -f /tmp/gobackup4250935195/1670122161319166282/demo/postgresql/postgresql/demo.sql
2022/12/03 18:49:21 [Database] Dump failed
2022/12/03 18:49:21 [Database] on_exit is success, skip run after hooks
2022/12/03 18:49:21 [Model: demo] pg_dump: error: connection to server at "localhost" (::1), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?
connection to server at "localhost" (<IP_ADDRESS>), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?

2022/12/03 18:49:21 [Model] Cleanup temp: /tmp/gobackup4250935195/

before + after + failure on_exit
2022/12/03 18:49:33 [Model: demo] WorkDir: /tmp/gobackup2759939241/1670122173977769211/demo
2022/12/03 18:49:33 [Database] => database | postgresql: postgresql
2022/12/03 18:49:33 [Database] Run dump before hooks
2022/12/03 18:49:33 [Database] Dump before hooks succeeded
2022/12/03 18:49:33 [PostgreSQL] -> Dumping PostgreSQL...
2022/12/03 18:49:33 [debug]  /usr/bin/pg_dump --host=localhost --port=5432 --username=postgres demo -f /tmp/gobackup2759939241/1670122173977769211/demo/postgresql/postgresql/demo.sql
2022/12/03 18:49:33 [Database] Dump failed
2022/12/03 18:49:33 [Database] on_exit is failure, start to run after hooks
2022/12/03 18:49:33 [Database] Run dump after hooks
2022/12/03 18:49:33 [Database] Dump after hooks succeeded
2022/12/03 18:49:33 [Model: demo] pg_dump: error: connection to server at "localhost" (::1), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?
connection to server at "localhost" (<IP_ADDRESS>), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?

2022/12/03 18:49:33 [Model] Cleanup temp: /tmp/gobackup2759939241/

before + after + no on_exit
2022/12/03 18:49:39 [Model: demo] WorkDir: /tmp/gobackup1635565693/1670122179577122327/demo
2022/12/03 18:49:39 [Database] => database | postgresql: postgresql
2022/12/03 18:49:39 [Database] Run dump before hooks
2022/12/03 18:49:39 [Database] Dump before hooks succeeded
2022/12/03 18:49:39 [PostgreSQL] -> Dumping PostgreSQL...
2022/12/03 18:49:39 [debug]  /usr/bin/pg_dump --host=localhost --port=5432 --username=postgres demo -f /tmp/gobackup1635565693/1670122179577122327/demo/postgresql/postgresql/demo.sql
2022/12/03 18:49:39 [Database] Dump failed
2022/12/03 18:49:39 [Model: demo] pg_dump: error: connection to server at "localhost" (::1), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?
connection to server at "localhost" (<IP_ADDRESS>), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?

2022/12/03 18:49:39 [Model] Cleanup temp: /tmp/gobackup1635565693/

before + no after
2022/12/03 18:49:56 [Model: demo] WorkDir: /tmp/gobackup1413645216/1670122196976374293/demo
2022/12/03 18:49:56 [Database] => database | postgresql: postgresql
2022/12/03 18:49:56 [Database] Run dump before hooks
2022/12/03 18:49:56 [Database] Dump before hooks succeeded
2022/12/03 18:49:56 [PostgreSQL] -> Dumping PostgreSQL...
2022/12/03 18:49:56 [debug]  /usr/bin/pg_dump --host=localhost --port=5432 --username=postgres demo -f /tmp/gobackup1413645216/1670122196976374293/demo/postgresql/postgresql/demo.sql
2022/12/03 18:49:56 [Database] Dump failed
2022/12/03 18:49:56 [Model: demo] pg_dump: error: connection to server at "localhost" (::1), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?
connection to server at "localhost" (<IP_ADDRESS>), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?

2022/12/03 18:49:56 [Model] Cleanup temp: /tmp/gobackup1413645216/

no before + no after
2022/12/03 18:50:04 [Model: demo] WorkDir: /tmp/gobackup3975745782/1670122204116304516/demo
2022/12/03 18:50:04 [Database] => database | postgresql: postgresql
2022/12/03 18:50:04 [PostgreSQL] -> Dumping PostgreSQL...
2022/12/03 18:50:04 [debug]  /usr/bin/pg_dump --host=localhost --port=5432 --username=postgres demo -f /tmp/gobackup3975745782/1670122204116304516/demo/postgresql/postgresql/demo.sql
2022/12/03 18:50:04 [Database] Dump failed
2022/12/03 18:50:04 [Model: demo] pg_dump: error: connection to server at "localhost" (::1), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?
connection to server at "localhost" (<IP_ADDRESS>), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?

2022/12/03 18:50:04 [Model] Cleanup temp: /tmp/gobackup3975745782/

no before + after + always on_exit
2022/12/03 18:55:29 [Model: demo] WorkDir: /tmp/gobackup1435367072/1670122529691722088/demo
2022/12/03 18:55:29 [Database] => database | postgresql: postgresql
2022/12/03 18:55:29 [PostgreSQL] -> Dumping PostgreSQL...
2022/12/03 18:55:29 [debug]  /usr/bin/pg_dump --host=localhost --port=5432 --username=postgres demo -f /tmp/gobackup1435367072/1670122529691722088/demo/postgresql/postgresql/demo.sql
2022/12/03 18:55:29 [Database] Dump failed
2022/12/03 18:55:29 [Database] on_exit is always, start to run after hooks
2022/12/03 18:55:29 [Database] Run dump after hooks
2022/12/03 18:55:29 [Database] Dump after hooks succeeded
2022/12/03 18:55:29 [Model: demo] pg_dump: error: connection to server at "localhost" (::1), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?
connection to server at "localhost" (<IP_ADDRESS>), port 5432 failed: Connection refused
        Is the server running on that host and accepting TCP/IP connections?

2022/12/03 18:55:29 [Model] Cleanup temp: /tmp/gobackup1435367072/

add systemd-style - prefix to ignore hook exit
2022/12/04 19:43:25 [Model: demo] WorkDir: /tmp/gobackup2329085183/1670211805991203562/demo
2022/12/04 19:43:25 [Database] => database | postgresql: postgresql
2022/12/04 19:43:25 [Database] Run dump before_script
2022/12/04 19:43:25 [debug]  /tmp/prestart hello world 1
2022/12/04 19:43:25 [Database] Run dump before_script failed: , ignore it
2022/12/04 19:43:25 [PostgreSQL] -> Dumping PostgreSQL...
2022/12/04 19:43:26 [PostgreSQL] dump path: /tmp/gobackup2329085183/1670211805991203562/demo/postgresql/postgresql/demo.sql
2022/12/04 19:43:26 [Database] Dump succeeded
2022/12/04 19:43:26 [Database] Run dump after_script
2022/12/04 19:43:26 [Model: demo] Run dump after_script failed: echoecho cannot be found
2022/12/04 19:43:26 [Model] Cleanup temp: /tmp/gobackup2329085183/


Doesn't work in WkWebview

Doesn't seem to work at all in a WkWebview. It's just javascript right, so I'm not sure why.

Sorry, my mistake... I was using the WKWebView loadHTMLString and so it wasn't picking up the .js file.


Prepend event listeners

fixes https://github.com/szmarczak/http-timer/issues/7
Seems to work great but unable to write a failing test.

Pull Request Test Coverage Report for Build 63

10 of 10 (100.0%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-2.3%) to 97.74%

Files with Coverage Reduction
New Missed Lines
%

source/index.ts
1
92.73%

Totals

Change from base Build 62:
-2.3%

Covered Lines:
326

Relevant Lines:
328

💛  - Coveralls

:heart: great job


rebrand design feedback

Summary:

What: adding some more replacements of Aspen to CZ Gen Epi
Ticket: sc175310
Env: <rdev link>

Demos:
Notes:
Checklist:

[x] I merged latest happy-trails-aspen
[x] I manually verified the change
[x] I added labels to my PR
[ ] I tested in multiple browsers
[ ] I added relevant unit tests
[ ] I have notified others of changes they need to make locally (migrations, jobs, package updates, etc)

@vincent-czi added a few more replacements since you last reviewed

@Janeece when you get a spare sec can you weigh in on vincent's comment?

@phoenixAja @vincent-czi I think we should use "Chan Zuckerberg GEN EPI" and "CZ GEN EPI" (all caps) -- this is the formatting in other areas that have already been updated/reviewed. Let me know if that doesnt answer the question :)
Good catch!


feat: [firebase_ui_auth] [firebase_ui_oauth] Allow for custom labels to be used for custom OAuth providers

Description
I made a custom OAuth provider so that I could add it to the sign in page. The issue is that the existing code expects there to only ever be the providers that exist in this repo. This was causing a crash when fetching the OAuth button label text.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]).
This will ensure a smooth and quick review process. Updating the pubspec.yaml and changelogs is not required.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[ ] My PR includes unit or integration tests for all changed/updated/fixed behaviors (See Contributor Guide).
[x] All existing and new tests are passing.
[x] I updated/added relevant documentation (doc comments with ///).
[x] The analyzer (melos run analyze) does not report any problems on my PR.
[x] I read and followed the Flutter Style Guide.
[x] I signed the CLA.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require plugin users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change.
[x] No, this is not a breaking change.

Seems like there are analysis issues in other packages unrelated to this PR. Probably not within the scope of this PR to fix them.

@Rexios80 out of curiosity: what provider did you implement? Is this an officially supported provider that is available on firebase console?

https://github.com/IO-Design-Team/firebase_ui_oauth_oidc/blob/master/lib/src/provider.dart

Do you want to own that package? Or is it too custom to be in this repo? I was going to publish it myself if you don't see an issue with it.

@Rexios80 feel free to publish


Allow multiple hyphens for context header separator

Apparently Talon supports multiple hyphens to close the context header? See https://github.com/phillco/talon-axkit/pull/36

This is deprecated, but still supported for the time being, so it makes sense to support it for now.

I think I'd be acceptable to just raise an error -- I think aegis has said that -- is deprecated -- rather than trying to support it.

This was fixed a long time ago.


[Snyk] Upgrade react-dom from 16.6.3 to 16.14.0

Snyk has created this PR to upgrade react-dom from 16.6.3 to 16.14.0.

As this is a private repository, Snyk-bot does not have access. Therefore, this PR has been created automatically, but appears to have been created by a real user.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 24 versions ahead of your current version.
The recommended version was released 5 months ago, on 2020-10-14.

Release notes

Package name: react-dom

16.14.0 - 2020-10-14React

Add support for the new JSX transform. (@ lunaruan in #18299)

16.13.1 - 2020-03-19React DOM

Fix bug in legacy mode Suspense where effect clean-up functions are not fired. This only affects users who use Suspense for data fetching in legacy mode, which is not technically supported. (@ acdlite in #18238)
Revert warning for cross-component updates that happen inside class render lifecycles (componentWillReceiveProps, shouldComponentUpdate, and so on). (@ gaearon in #18330)

Artifacts

react<EMAIL_ADDRESS>react-art<EMAIL_ADDRESS>react-dom<EMAIL_ADDRESS>react-is<EMAIL_ADDRESS>react-test-renderer<EMAIL_ADDRESS>scheduler<EMAIL_ADDRESS>

16.13.0 - 2020-02-26React

Warn when a string ref is used in a manner that's not amenable to a future codemod (@ lunaruan in #17864)
Deprecate React.createFactory() (@ trueadm in #17878)

React DOM

Warn when changes in style may cause an unexpected collision (@ sophiebits in #14181, #18002)
Warn when a function component is updated during another component's render phase (@ acdlite in #17099)
Deprecate unstable_createPortal (@ trueadm in #17880)
Fix onMouseEnter being fired on disabled buttons (@ AlfredoGJ in #17675)
Call shouldComponentUpdate twice when developing in StrictMode (@ bvaughn in #17942)
Add version property to ReactDOM (@ ealush in #15780)
Don't call toString() of dangerouslySetInnerHTML (@ sebmarkbage in #17773)
Show component stacks in more warnings (@ gaearon in #17922, #17586)

Concurrent Mode (Experimental)

Warn for problematic usages of ReactDOM.createRoot() (@ trueadm in #17937)
Remove ReactDOM.createRoot() callback params and added warnings on usage (@ bvaughn in #17916)
Don't group Idle/Offscreen work with other work (@ sebmarkbage in #17456)
Adjust SuspenseList CPU bound heuristic (@ sebmarkbage in #17455)
Add missing event plugin priorities (@ trueadm in #17914)
Fix isPending only being true when transitioning from inside an input event (@ acdlite in #17382)
Fix React.memo components dropping updates when interrupted by a higher priority update (@ acdlite in #18091)
Don't warn when suspending at the wrong priority (@ gaearon in #17971)
Fix a bug with rebasing updates (@ acdlite and @ sebmarkbage in #17560, #17510, #17483, #17480)

Artifacts

react<EMAIL_ADDRESS>react-art<EMAIL_ADDRESS>react-dom<EMAIL_ADDRESS>react-is<EMAIL_ADDRESS>react-test-renderer<EMAIL_ADDRESS>scheduler<EMAIL_ADDRESS>

16.12.0 - 2019-11-14React DOM

Fix passive effects (useEffect) not being fired in a multi-root app. (@ acdlite in #17347)

React Is

Fix lazy and memo types considered elements instead of components (@ bvaughn in #17278)

Artifacts
• react<EMAIL_ADDRESS>• react-art<EMAIL_ADDRESS>• react-dom<EMAIL_ADDRESS>• react-is<EMAIL_ADDRESS>• react-test-renderer<EMAIL_ADDRESS>• scheduler<EMAIL_ADDRESS>

16.11.0 - 2019-10-22

16.10.2 - 2019-10-03

16.10.1 - 2019-09-28

16.10.0 - 2019-09-27

16.9.0 - 2019-08-08

16.9.0-rc.0 - 2019-08-05

16.9.0-alpha.0 - 2019-04-03

16.8.6 - 2019-03-28

16.8.5 - 2019-03-22

16.8.4 - 2019-03-05

16.8.3 - 2019-02-21

16.8.2 - 2019-02-14

16.8.1 - 2019-02-06

16.8.0 - 2019-02-06

16.8.0-alpha.1 - 2019-01-15

16.8.0-alpha.0 - 2019-01-09

16.7.0 - 2018-12-20

16.7.0-alpha.2 - 2018-11-13

16.7.0-alpha.1 - 2018-11-13

16.7.0-alpha.0 - 2018-10-25

16.6.3 - 2018-11-13

    from react-dom GitHub release notes

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

This seems to pass checks but in other apps this bump has been failing checks...I'm curious if this particular app just isn't using the same checks as other apps?
https://github.com/newrelic/nr1-integrations-manager/runs/1698689314?check_suite_focus=true
https://github.com/newrelic/nr1-integrations-manager/pull/45


Support Disabling Source Type Override Parameters

The source type override parameter is automatically overridden on the CodeBuild job to the GITHUB type and current repository. We have a scenario that requires our CodeBuild job to source from a different location, but would still like to drive the execution of the CodeBuild job from a GitHub action.
Can we add a new parameter that allows us to disable the source type override parameters and use the CodeBuild job's default values?

Adding another vote for this capability, we do not want the GitHub repository replicated to the CodeBuild job - we wish to manage this ourselves.

Another vote, we would like to use this action to run already configured code build jobs.

@mattsb42-aws Is there any update on getting PR #64 reviewed?

@robin-aws Our organization has a ton of use-cases for this enhancement (and the workaround is fairly painful). Any chance this gets reviewed anytime soon?

We've simply forked this action and disabled these overrides, so we can use what is specified on the codebuild job.
diff --git a/code-build.js b/code-build.js
index 67d2e14..55be488 100644
--- a/code-build.js
+++ b/code-build.js
@@ -152,7 +152,7 @@ async function waitForBuildEndTime(

 function githubInputs() {
   const projectName = core.getInput("project-name", { required: true });
-  const { owner, repo } = github.context.repo;
+  // const { owner, repo } = github.context.repo;
   const { payload } = github.context;
   // The github.context.sha is evaluated on import.
   // This makes it hard to test.
@@ -177,8 +177,8 @@ function githubInputs() {

   return {
     projectName,
-    owner,
-    repo,
+    // owner,
+    // repo,
     sourceVersion,
     buildspecOverride,
     envPassthrough,
@@ -188,15 +188,15 @@ function githubInputs() {
 function inputs2Parameters(inputs) {
   const {
     projectName,
-    owner,
-    repo,
+    // owner,
+    // repo,
     sourceVersion,
     buildspecOverride,
     envPassthrough = [],
   } = inputs;

-  const sourceTypeOverride = "GITHUB";
-  const sourceLocationOverride = `https://github.com/${owner}/${repo}.git`;
+  // const sourceTypeOverride = "GITHUB";
+  // const sourceLocationOverride = `https://github.com/${owner}/${repo}.git`;

   const environmentVariablesOverride = Object.entries(process.env)
     .filter(
@@ -209,8 +209,8 @@ function inputs2Parameters(inputs) {
   return {
     projectName,
     sourceVersion,
-    sourceTypeOverride,
-    sourceLocationOverride,
+    // sourceTypeOverride,
+    // sourceLocationOverride,
     buildspecOverride,
     environmentVariablesOverride,
   };

@ejohn20 You can try dark-mechanicum/aws-codebuild github action. By default, it does not pass anything but is very flexible in configuration. You literally can override any parameter for the startBuild API call  ;)