Hybrid Custody contract

Second approach to the Hybrid Custody problem (still a work in progress).
This new approach has basically three resources:

ChildAccount - wraps the AuthAccount capability and has helper methods to share access to the account with parents, revoke parent access, and to change ownership of the account (no owner, or delegate ownership elsewhere)
ProxyAccount - there is one ProxyAccount resource for every published parent. Each ProxyAccount has its own rules applied, but they can be shared across different accounts since only a capability to these rules is needed.
Manager - The resource used by parent accounts to control what child accounts it has redeemed, and what accounts it now owns as well. A manager can borrow a child account and then get capabilities from it

Proposed diagram (currently) looks like the following:

Current TODOs:

[ ] Manager helper methods for owned accounts
[ ] Should we permit child accounts of the same address on a parent?
[ ] CapabilityFilter panic instead of force unwrap
[ ] Helper methods to access the CapabilityProxy on a borrowed Account (public and private)
[ ] Variable renaming, relinquishOwnership, and helper methods to get path identifiers to start
[ ] Implement MetadataViews interfaces and discuss HC-specific views
[ ] Borrow an account by address
[ ] Borrow an owned account by address

Looks pretty good to me! Mostly small comments about docs, error messages, and such. I agree that we should add more events and metadata views stuff

Thanks for taking a look! My hope is that we can form breakout discussions to quickly iterate on metadata views and on events as soon as this is merged. If we are okay with the general approach to access, I think we can let this go pretty soon and then start flagging what's left for anyone else to come in and contribute if they want

Last main piece I think we should tackle here:

How to let the owner manage a child account's CapabilityFilter, and CapabilityProxy
What we need to be able to discover on a child account from the owned account

How to let the owner manage a child account's CapabilityFilter, and CapabilityProxy

Once they're the owner, they would have access to all resources in the child account, giving them the ability to add arbitrary Capabilities to any CapabilityProxy in said account.
Thinking of the case @rrrkren mentioned where wallets like Dapper Wallet have restricted access on a child account on which another self-custodial wallet has owner access, this raises an issue. For instance, DW would not want to have an access path on a FungibleToken Capability without their authorization on that access path.
This leads me to believe that access should either be immutable or require a handshake of sorts to grant new Capabilities. If I'm an owner and I want to give a parent access to an additional capability, I should ask them to receive that Capability.
Put simply, owners should be able to initiate adding capabilities but require parent approval to do so while reserving sole authority to remove capabilities from a proxy or filter.
One solution here might be removing the ability to add Capabilities to proxies and filters altogether. This implies the need to create a new ProxyAccount and replace the corresponding Capability in the parent's Manager for the associated ChildAccount.
Curious to hear others' thoughts here.

What we need to be able to discover on a child account from the owner

Thoughts here are:

All associated parent accounts
Metadata about the account

I would say the contents of the child account, but that can be done via scripts/off-chain systems as is already common practice.

Once they're the owner, they would have access to all resources in the child account, giving them the ability to add arbitrary Capabilities to any CapabilityProxy in said account.
Thinking of the case @rrrkren mentioned where wallets like Dapper Wallet have restricted access on a child account on which another self-custodial wallet has owner access, this raises an issue. For instance, DW would not want to have an access path on a FungibleToken Capability without authorizing their being granted that access.
This leads me to believe that access should either be immutable or require a handshake of sorts to grant new Capabilities. If I'm an owner and I want to give a parent access to an additional capability, I should ask them to receive that Capability.
Put simply, owners should be able to initiate adding capabilities but require parent approval to do so while reserving sole authority to remove capabilities from a proxy or filter.
One solution here might be removing the ability to add Capabilities to proxies and filters altogether. This implies the need to create a new ProxyAccount with new CapabilityFilter & CapabilityProxy and replace the corresponding Capability in the parent's Manager for the associated ChildAccount.

We want this management back and forth to be as light a lift as possible. If we need to put either of these options in place, we should have a discussion about it to be sure that this is necessary since they will make the control flow here much more complex. It could be that Dapper needs a custom manager resource which lets them gate what capabilities they can get regardless of what the underlying child account does

Had some discussion offline in the flow discord, we're going to look into a mechanism that lets the manager gate what types can be returned by the proxy and factory, regardless of what's in the ProxyAccount resource. We'll want confirmation from the Dapper Wallet team that this approach would suffice, but it sounds like a workable approach to ensure parent accounts can follow whatever rules they have to adhere to.
The plan is to tackle that and discoverability questions in separate PRs so we can keep this moving forward. There is certainly much work to do, still, but if we stall too long nothing will happen and we'll be in the same position we are now in a month.
I am planning to merge this PR by EOD today so we can proceed with next steps. Mainly:

Events for indexer discoverability
Metadata Views
Discoverability helper functions. We need to explore what questions you should be able to ask the Child, Parent, or in some cases the Owner.

Each of these I think we can tackle separately but it's possible the MetadataViews piece will be closely tied to the discoverability piece. I'm guessing we will leverage metadata views for discoverability.

Retrieval through the Capability Proxy is now working as a first version, and the manager resource restricting what's returning in private types works now. I wasn't sure if we needed to also restrict public types, the thought behind not applying a manager's CapabilityFilter to public capabilities via the proxy or factory is that it's still accessible to them because it would be accessible to everyone. Hopefully that's the right approach, we will want to confirm.

I think that's it for this version! @sisyphusSmiling feel free to review again if you would like, I will merge this later tonight and we can plan next steps tomorrow 🤝


Switch not working in safari

Switch component is not working in safari.
https://lusaxweb.github.io/vuesax/components/switch.html#state
You can check it in docs.

I encountered the same issue for Safari v12.0.1 (146<IP_ADDRESS>.1).


exterior elements is empty

Hello, I am having a problem running the code you provided.
The code is as:
file = parse("duplex.ifc")
surfaces = create_geometry(file)
voxels = voxelize(surfaces)
external = exterior(voxels)
shell = offset(external)
export_ifc(file, shell, surfaces, "exterior.ifc")
My operating system is Windows.
Due to the lack of corresponding DLLs in the exe provided by conda, I downloaded the exe from here (https://github.com/IfcOpenShell/voxelization_toolkit/releases/tag/v0.3.0).
I test  sample data (duplex.ifc).
The result is that  "exterior.ifc" is empty.

Can you provide other versions of exe that can be used for Windows? Thank you very much. Looking forward to your reply.

Most likely, it's not empty, but it doesn't contain the spatial hierarchy to connect the elements to the project so some viewers will not show anything.
A more versatile approach is to use export_json instead and use ifcopenshell's IfcPatch ExtractElements recipe on the resulting elements from the JSON. ExtractElements will copy over the full hierarchy.
CI/CD builds are being worked on. Soon you can just install this from using python pip.

Hi, aothms.
Thank you for your reply. I have followed your suggestion and obtained a JSON file, but this JSON file does not fully contain some obviously external elements, such as the IfcWallStandardCase with UUID "2O2Fr$t4X7Zf8NOew3FLQD":

Here is the exported JSON file:
[
{"id":274,"guid":"1hOSvn6df7F8_7GcBWlRBU"},
{"id":217,"guid":"0jf0rYHfX3RAB3bSIRjmmy"},
{"id":271,"guid":"1l0GAJtRTFv8$zmKJOH4hv"},
{"id":340,"guid":"0kF45Qs8L9PAM9kmb1lT2N"},
{"id":297,"guid":"1hOSvn6df7F8_7GcBWlS4Q"},
{"id":290,"guid":"1Eo2$BaHX42AEkDvQQDocD"},
{"id":337,"guid":"0kF45Qs8L9PAM9kmb1lT5t"},
{"id":231,"guid":"1hOSvn6df7F8_7GcBWlRLx"},
{"id":275,"guid":"1l0GAJtRTFv8$zmKJOH4qs"},
{"id":256,"guid":"1CZILmCaHETO8tf3SgGEWh"},
{"id":227,"guid":"1l0GAJtRTFv8$zmKJOH4$e"},
{"id":294,"guid":"1l0GAJtRTFv8$zmKJOH4oq"},
{"id":273,"guid":"1hOSvn6df7F8_7GcBWlS_W"},
{"id":292,"guid":"1Eo2$BaHX42AEkDvQQDoy2"},
{"id":296,"guid":"1l0GAJtRTFv8$zmKJOH4u1"},
{"id":251,"guid":"1s1jVhK8z0pgKYcr9jt7AB"},
{"id":230,"guid":"1hOSvn6df7F8_7GcBWlRRL"},
{"id":269,"guid":"1l0GAJtRTFv8$zmKJOH4ZZ"},
{"id":272,"guid":"1hOSvn6df7F8_7GcBWlSXO"},
{"id":268,"guid":"1hOSvn6df7F8_7GcBWlR72"},
{"id":336,"guid":"0kF45Qs8L9PAM9kmb1lT5d"},
{"id":252,"guid":"1hOSvn6df7F8_7GcBWlRGQ"},
{"id":298,"guid":"1hOSvn6df7F8_7GcBWlSnC"},
{"id":228,"guid":"1hOSvn6df7F8_7GcBWlS2V"},
{"id":270,"guid":"1l0GAJtRTFv8$zmKJOH4aS"},
{"id":293,"guid":"1hOSvn6df7F8_7GcBWlSp1"},
{"id":226,"guid":"1l0GAJtRTFv8$zmKJOH4pU"},
{"id":338,"guid":"0kF45Qs8L9PAM9kmb1lT2B"},
{"id":253,"guid":"1hOSvn6df7F8_7GcBWlRH8"},
{"id":291,"guid":"1l0GAJtRTFv8$zmKJOH4kJ"},
{"id":229,"guid":"1hOSvn6df7F8_7GcBWlSga"},
{"id":334,"guid":"0kF45Qs8L9PAM9kmb1lT5l"},
{"id":255,"guid":"1CZILmCaHETO8tf3SgGEXu"},
{"id":299,"guid":"1hOSvn6df7F8_7GcBWlS1M"},
{"id":254,"guid":"1s1jVhK8z0pgKYcr9jt781"},
{"id":295,"guid":"1l0GAJtRTFv8$zmKJOH4gQ"}
]
Also, I have another question, why did others successfully extract external elements and appear to be directly exporting them using "export_ifc"?
For example:
https://github.com/IfcOpenShell/voxelization_toolkit/issues/13
https://github.com/IfcOpenShell/IfcOpenShell/issues/1523
Is the tool and code I am using not matching?
I am looking forward to and appreciating your reply.

but this JSON file does not fully contain some obviously external elements, such as the IfcWallStandardCase with UUID "2O2Fr$t4X7Zf8NOew3FLQD":

Probably the inclusion criteria need to be tweaked. The full list of available arguments to the function is here:
https://github.com/IfcOpenShell/voxelization_toolkit/blob/ae3735da20df3d8e1d560c4b60ba72b451f818ad/voxec.h#L1273
These factors and face counts are an attempt at preventing situations like this where the gray element would be counted as external due to the voxel resolution.

factor concerns the overlap of voxels for the element vs voxels in the input grid to this function. factor=2 means at least 1/2 of the voxels for an individual element need to be present in the input set
individual_faces=0|1 means whether this factor is applied to the full element surface or to individual faces
num_faces in case of the latter =1 this governs how many faces (absolute) value needs to have at least factor overlap.

why did others successfully extract external elements and appear to be directly exporting them using "export_ifc"?

Not all viewers have the requirement to have a fully connected spatial decomposition tree. The model from export_ifc is not really invalid, it's just that the elements are not really part of the project and viewers choose to handle that differently.


CEQ-7373: Support extension: camel-quarkus-xml-jaxp

https://issues.redhat.com/browse/CEQ-6669
https://issues.redhat.com/browse/CEQ-7373

The rest looks good to me!


[Backport baremetal] Fix issues in handling of client reconnecting from the same port

This is the backport of #3132 to the baremetal.
Usually the baremetal branch gets its bug fixes backported via the 2.16 branch by regularly merging back from it. But in this instance, the bug was introduced in 2.19 (as a foward port to development of the record checking feature that was first introduced in baremetal), so we need a direct backport of the fix.
This also improves tests and debug logging, which was also backported to 2.16. Due to differences in this area between baremetal and 2.16 (linked to the new features in baremetal), it's possible that git may have trouble resolving conflits next time you merge 2.16 to baremetal - in that case, keep the version in the baremetal branch, which has everything that the 2.16 backport has, and more.

There are CI failures, including in the newly-added test case, so I think the CI has found something that the reviewers didn't.

The failures are in test_hardcoded_io_callbacks_cmake_clang and test_baremetal which are specific to the baremetal branch, so it makes sense that we seeing those failures here for the first time. I'll investigate and fix.

Should be fixed now.

the CI has found something that the reviewers didn't

As the issues were rather specific to the baremetal branch (especially the first one), it's not surprising the reviewers missed them, but the PR author, having worked on that branch, should have known better. Well, I guess that's one of the reasons we have a CI and systematically add tests for each new config.h option. Apparently it's hard for us humans to remember reliably enough.

Changes from this PR have been incorporated into https://github.com/ARMmbed/mbedtls/pull/3330.

This has been superseded by #3330 which has now been merged, so I'm closing this one.


Fix server stall issue in case some requests get cancelled midway by the client

This fix seems to be leaking metadata strings and am currently investigating..
Before that, I wanted to get some initial feedback on my PR (just to make sure I am not doing something very wrong here).
There are two cases where the core layer was not notifying the c++ layer of incoming RPC calls. This meant that the incoming call (that wasn't notified to the C++ layer) took a request_id from the request matcher but it was never being put back.  (This is because normally the the request_id is put back in the request matcher by core api queue_call_request() which gets called by C++ only after it was notified of an incoming rpc call).
Case 1) The connection was closed by the client somewhere in the middle while the RPC is being handled by the server (more specifically, after grpc_call_start_batch_and_execute() was called but before the batch was actually completed). The connection termination handling logic was not calling post_batch_completion() callback with failure.
(Note: Post_batch_completion() is the one that calls the publish_registered_or_batch() callback which then puts an event on completion queue that the C++ layer is waiting on and hence C++ layer never learns of this RPC)
Case 2) This case only happens if there are any requests in PENDING state queued at the request matcher  (because a request_id wasn't available from request matcher).
In this case, if the connection was terminated by the client AFTER a request was queued in PENDING state, the code at the surface layer (i.e at queue_call_request() does not know about it and proceeds to "ACTIVATE" the request the moment a request id is available) and that starts a batch.  That batch would never complete since the transport is already closed at this point..which means post_batch_completion() never gets called (and as in case 1, this will not notify C++ layer about this request).

/cc @yang-g

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can you check and see if the C++ part of this is still needed?

Right now your fix (i.e https://github.com/grpc/grpc/pull/4453/) works without the need for the C++ part.
However, I think this is by accident :) and we still need the C++ part.. here is why:
In the change (https://github.com/grpc/grpc/pull/4453/files#diff-b0e0a5183c8545d5b061588dd6c4060fR977), in receiving_slice_ready() function, if success is false, we do not set bctl->success to false before calling post_batch_completion().
As a result, publish_registered_or_batch() is called with success == true which then calls grpc_cq_end_op with success == true and eventually the c++ code at (https://github.com/grpc/grpc/blob/master/src/cpp/server/server.cc#L559) thinks the op succeeded - i.e ok == true.
Had we set bctl->success to false  in receiving_slice_ready,  this would have caused ok to be false which then meant that mrd->SetupRequest() at
https://github.com/grpc/grpc/blob/master/src/cpp/server/server.cc#L562 would not have been called and we would have a stall again!

Oh, my previous statement is based on the assumption that we should set bctl->success to false in receiving_slice_ready() function in case of failures :). Let me know if this is a correct assumption. If so, I can make the change (and the corresponding C++ code changes)


Add retina assets for mobile

https://github.com/CartoDB/cartodb.js/commit/d9f054b9b07e90c01c6b98441ab1328847176aa2


how to use fileds

In order to help us troubleshoot issues with this project, you must provide the following details:

ElasticDump 2.4.2
Elasticsearch 2.3.3

./elasticdump --input=http://<IP_ADDRESS>:9200/domain6 --output=query1.json --debug=true --limit=1 --offset=0 --searchBody='{"query":{"range":{"id":{"lte":"80"}}},"fields": ["id"], "_source": true}' --sourceOnly=true
this result:
{"id":31,"host":"4yt.net","cname":null,"ename":null}
but I want only this id ,how to change?

this question had Ok.

"_source": "id" or "_source": "_id", can't remember which.

fields says which fields are queried, not returned. source says which fields are returned.


KIALI-331 graph summary for namespace

This aggregates info about all services reflected in the namespace graph
also:

remove horizontal break in RateTable to conserve space/improve locality
add updateTime to graph state (for future use)

NOTE: This depends on new backend support, setting to do not merge until core PR is merged.

This can be reviewed but is blocked by backend https://github.com/kiali/swscore/pull/126

"Click graph background to see summary information..."
Can we not just run this on the page load so that people don't need to click on the background to see something?

switching views from a namespace that has a service mesh to a namespace that doesnt also doesn't properly update the graph (it updates the name, but not the other details). This might go with my comment above with reloading the information when the page is loaded, or the namespace changes

@mwringe Yes, I think this is related to something else I noticed today. Click any edge, see the side panel load with data, click another edge... most of the side panel data is static (doesn't change). We need to fix something about the click handler.

Can we not just run this on the page load so that people don't need to click on the background to see something?

No, we can't :)   This is what I was whining about yesterday.  At least I haven't figured out a way to do it yet, I'll look more today but let's get this in as is first, so that we at least have something to show.


Persist benchmark results in CI

Benchmark results are now collected to a json file which will be appended to a file in s3.
Another action will generate static HTML from these results and push them to GitHub Pages.

Should the benchmark workflow have a workflow_dispatch event so that it can be run manually as well as on merge?

Will do

The benchmark-results.json file in S3 is currently empty even though this workflow looks like it's been run a few times. Is that expected?

Yep, that's fine. I've been throwing away the test runs on the PR since they'll have an incorrect SHA.


Feature: Create github action to fetch secrets from Azure KeyVault

Create a github action to fetch Azure KeyVault secrets

Will that be a keyvault on our own development Azure subscription? Because it is not possible to fetch information in that direction until we've gotten Github approved and integrated with our official sandbox subscription.


gnome3.gnome-autoar: 0.3.0 -> 0.2.4

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for gnome3.gnome-autoar is: "Library to integrate compressed files management with GNOME"
meta.homepage for gnome3.gnome-autoar is: ""
meta.changelog for gnome3.gnome-autoar is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 0.2.4 with grep in /nix/store/amf70a2gliy82h6cn5mrplw6rf1hg4xk-gnome-autoar-0.2.4

directory tree listing: https://gist.github.com/ec3f6eb81dd9af0d0732861ce61dc36f

du listing: https://gist.github.com/fea54dff512b3bf36a907755cb0a35eb

Rebuild report (if merged into master) (click to expand)

74 total rebuild path(s)

25 package rebuild(s)

25 x86_64-linux rebuild(s)
24 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
23 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
adapta-gtk-theme
chrome-gnome-shell
dropbox-cli
gnome-recipes
gnome3.file-roller
gnome3.gnome-autoar
gnome3.gnome-control-center
gnome3.gnome-session
gnome3.gnome-shell
gnome3.gnome-terminal
gnome3.gnome-tweak-tool
gnome3.gnome-tweaks
gnome3.gnome-user-share
gnome3.gnome_control_center
gnome3.gnome_session
gnome3.gnome_shell
gnome3.gnome_terminal
gnome3.nautilus
gnome3.nautilus-python
gnome3.pomodoro
gnomeExtensions.easyScreenCast
gnomeExtensions.gsconnect
gnomeExtensions.night-theme-switcher
pantheon.elementary-session-settings
pantheon.extra-elementary-contracts

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/amf70a2gliy82h6cn5mrplw6rf1hg4xk-gnome-autoar-0.2.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A gnome3.gnome-autoar https://github.com/r-ryantm/nixpkgs/archive/65ee7147f89d3544e155dd5e18ab24e97a36c062.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/amf70a2gliy82h6cn5mrplw6rf1hg4xk-gnome-autoar-0.2.4
ls -la /nix/store/amf70a2gliy82h6cn5mrplw6rf1hg4xk-gnome-autoar-0.2.4/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

21 packages built:

adapta-gtk-theme
chrome-gnome-shell
dropbox-cli
gnome-recipes
gnome3.file-roller
gnome3.gnome-autoar
gnome3.gnome-control-center
gnome3.gnome-session
gnome3.gnome-shell
gnome3.gnome-terminal
gnome3.gnome-tweak-tool
gnome3.gnome-user-share
gnome3.nautilus
gnome3.nautilus-python
gnome3.pomodoro
gnomeExtensions.easyScreenCast
gnomeExtensions.gsconnect
gnomeExtensions.night-theme-switcher
kgx
pantheon.elementary-session-settings
pantheon.extra-elementary-contracts

Maintainer pings
cc @hedning @jtojnar @worldofpeace @dasj19 for testing.

This is a downgrade.


Add Stremio (http://www.strem.io)

Hi,
I'd like to add Stremio - it's a media center to organize your movies / series / channels and receive notifications about them.
It also allows playback through an Add-ons system - right now it has add-ons for YouTube, Filmon.tv and a third-party add-on for streaming from torrents.
It's a nice tech example because it also uses WebChimera.js, i.e. binds electron with libvlc.
Icon here: http://www.strem.io/images/stremio-logo-small.png
BTW: according to CONTRIBUTE.md, I should open an issue. Should I PR as well?

Feel free to open a PR!

https://github.com/Stremio/electron.atom.io/tree/add-stremio
The branch is ready, testing it and then PR-ing

Added :pizza: :sparkles: :ship:


Update for MySQL RP <IP_ADDRESS>

@xiaofmao : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@xiaofmao: I'm sorry - only the author of this article, @mattbriggs, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

Hi @xiaofmao
This contribution needs to be made through the private repository so the content can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
For more information, see Should I work in a private or a public repo?
If you need help getting set up to work in the private repo, see Set up Git repository locally for documentation.
Once you are working in the private repo, if your update is tied to a specific date and/or time, you'll need to sign off only when the PR is ready to be merged and published. In either repo, we can't hold PRs to merge on a specific date.
Would you create a pull request in the private repo to make these updates? We'll close this PR. Thank you.


Rm preflow payments

This removes the old payment engine from rippled. The changes in RippleCalc should be carefully reviewed. The commits are split as they are so they can be easily reviewed. The first commit removes the engine, but does as little refactoring/reformatting as possible. The other commits clean up the code a bit.

Codecov Report

Merging #3220 into develop will decrease coverage by 0.42%.
The diff coverage is 86.84%.

@@             Coverage Diff             @@
##           develop    #3220      +/-   ##
===========================================
- Coverage    70.78%   70.35%   -0.43%
===========================================
  Files          689      672      -17
  Lines        54296    52311    -1985
===========================================
- Hits         38432    36803    -1629
+ Misses       15864    15508     -356

Impacted Files
Coverage Δ

src/ripple/app/paths/RippleCalc.h
100% <ø> (ø)
:arrow_up:

src/ripple/app/paths/RippleCalc.cpp
86.84% <86.84%> (+4.95%)
:arrow_up:

src/ripple/protocol/impl/Rate2.cpp
50% <0%> (-47.37%)
:arrow_down:

src/ripple/app/paths/impl/FlowDebugInfo.h
0% <0%> (-17.62%)
:arrow_down:

src/ripple/protocol/impl/UintTypes.cpp
95.83% <0%> (-2.09%)
:arrow_down:

src/ripple/protocol/STPathSet.h
96.69% <0%> (-1.66%)
:arrow_down:

src/ripple/core/impl/Workers.cpp
98.93% <0%> (-1.07%)
:arrow_down:

src/ripple/app/paths/Pathfinder.cpp
86.66% <0%> (-0.73%)
:arrow_down:

src/ripple/protocol/impl/STAmount.cpp
90.11% <0%> (-0.17%)
:arrow_down:

src/ripple/ledger/impl/View.cpp
90.21% <0%> (-0.13%)
:arrow_down:

... and 5 more

Continue to review full report at Codecov.

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

Due to the changes, the following comments may no longer apply:
https://github.com/seelabs/rippled/blob/rm-preflow-payments/src/ripple/ledger/View.h#L302
https://github.com/seelabs/rippled/blob/rm-preflow-payments/src/ripple/protocol/jss.h#L180
https://github.com/seelabs/rippled/blob/rm-preflow-payments/src/ripple/protocol/jss.h#L239
https://github.com/seelabs/rippled/blob/rm-preflow-payments/src/ripple/protocol/jss.h#L276
https://github.com/seelabs/rippled/blob/rm-preflow-payments/src/ripple/protocol/jss.h#L535
https://github.com/seelabs/rippled/blob/rm-preflow-payments/src/ripple/protocol/jss.h#L265
https://github.com/seelabs/rippled/blob/rm-preflow-payments/src/ripple/protocol/jss.h#L376

The loop can be reduce to a single offer this part of the test:
https://github.com/seelabs/rippled/blob/rm-preflow-payments/src/test/app/Offer_test.cpp#L273-L275
eg.
        env (pay (gw, alice, USD (100)));
        env (pay (gw, carol, EUR (100)));
        env (offer (carol, USD (1), EUR (2)));```

tecExpect may no longer be needed...
https://github.com/seelabs/rippled/blob/rm-preflow-payments/src/test/app/Offer_test.cpp#L3572-L3573


Icons not showing

redhat 7.3
firefox 59.0b10

Can you look at your Network Monitor view in Firefox?  https://developer.mozilla.org/en-US/docs/Tools/Network_Monitor
See if any of the calls are 404'ing.  The icons are SVG files, so try opening the file directly in Firefox and seeing if your browser supports displaying those.  Ex. https://github.com/duo-labs/cloudmapper/blob/master/web/icons/aws/Compute/Compute_AmazonEC2.svg

I ran into the same problem on a freshly spun up CentOS 7 VM. After restarting the instance, the issue cleared itself up though. Haven't been able to replicate the behavior since.

I ran into the same issue. I tried rebooting my VM... did not worked. So I upgraded python to 3.6, created a new environment and it started working again...

@kpawloski and @rafarost Were you each on Firefox as well?  Did the tool bar load?  I'd be interested in seeing a screenshot if anyone duplicates this, and ideally any log messages from the Python webserver (or whatever webserver you use, as it's just static files being hosted), and a screenshot of the developer tools in the browser showing network access.

I'm using Chrome. Unfortunately I did not took any screenshot at the time. Like I said, it's working for me now. If it happen again, I will post here. Thanks.

I tried to describe a bit about some of the visual elements.
If the data.json file cannot be found (ie. you didn't create it), then the pan and zoom controls will not load and neither will the graph. I suspect this may be the root cause.  I created https://github.com/duo-labs/cloudmapper/issues/39 to add an error message.

In my case the icons, pan and zone and overview in the lower right loaded just fine using the demo data but the actual graphs in the main area wouldn't show at any zoom level. Was using FireFox. Haven't been able to reproduce the issue since initial run.

responding svg data from web server, its mime type should be " image/svg+xml" instead of "application/octet-stream"
My PR may solve this problem.
Please check #46 .

Someone sent me a screenshot privately, so this bug still exists.  The toolbar icons in the top left were visible, but no icons for the different nodes.  So the cytoscape map showed up, showing the connections and VPCs, but just the SVG's for the nodes themselves were not showing. Odd.
In order to find these icons, I just tell cytoscape to use a style.json file: https://github.com/duo-labs/cloudmapper/blob/29abd7f968758631f188dc4b76d4b7bc9307951a/web/js/cloudmap.js#L40
This file then references the .svg's: https://github.com/duo-labs/cloudmapper/blob/29abd7f968758631f188dc4b76d4b7bc9307951a/web/style.json#L92
In the image I saw, the compound nodes had the borders and colorings specified in the style.json file, so the style.json must have been loaded, but just the SVG's did not load.
One possibility is perhaps the webserver was killed, or crashed, after the page was initially loaded, but before the diagram was displayed.

I ended up being on a very slow network today and loaded the demo at https://duo-labs.github.io/cloudmapper/ and experienced this issue.  After waiting a while, the images did eventually load.  So this is one way in which this issue will exhibit itself.

Closing due to staleness. I'm assuming this might have been fixed at some point somehow.


[IA-4048] Add to timeout of failing steps

It looks like this allow the test to wait longer for the list of analyses to show up. Will that actually help if the list analyses request fails because of permissions? Terra UI randomly retries GCS requests for up to 5 seconds.
https://github.com/DataBiosphere/terra-ui/blob/542ec2b3ada246df040f06ce3d166c03cec176ac/src/libs/ajax/ajax-common.ts#L18-L37
If those retries fail, the request won't be retried again, so there's no need to wait longer. It looks like Puppeteer's default timeout is 30s, which is more than enough time for all of Terra UI's retries. Thus, it looks like this would only make the test take longer to fail after a failed GCS request.

It looks like this allow the test to wait longer for the list of analyses to show up. Will that actually help if the list analyses request fails because of permissions? Terra UI randomly retries GCS requests for up to 5 seconds.
https://github.com/DataBiosphere/terra-ui/blob/542ec2b3ada246df040f06ce3d166c03cec176ac/src/libs/ajax/ajax-common.ts#L18-L37
If those retries fail, the request won't be retried again, so there's no need to wait longer. It looks like Puppeteer's default timeout is 30s, which is more than enough time for all of Terra UI's retries. Thus, it looks like this would only make the test take longer to fail after a failed GCS request.

Oh, my bad, I meant to include the log that motivated this change. So the polling eventually succeeds. The POST to create a file eventually succeeds,  and GET on the storage API then returns a few 403s. Eventually it does return 200, but not fast enough for the page to render.
page.http.error GET 403 https://rawls.dsde-staging.broadinstitute.org/api/workspaces/saturn-integration-test-stage/terra-ui-test-workspace-47fcc852-e301-40d2-8354-e31d4e50abb3/checkBucketReadAccess {
  causes: [],
  message: '...'
  source: 'rawls',
  stackTrace: [],
  statusCode: 403
}
page.http GET 200 https://rawls.dsde-staging.broadinstitute.org/api/workspaces/saturn-integration-test-stage/terra-ui-test-workspace-47fcc852-e301-40d2-8354-e31d4e50abb3/checkBucketReadAccess
…
page.http POST 200 https://storage.googleapis.com/upload/storage/v1/b/fc-2dfeae5e-320f-4abf-bf29-3880436edc63/o?uploadType=media&name=notebooks%2Ftest-notebook.ipynb
page.http GET 403 https://storage.googleapis.com/storage/v1/b/fc-2dfeae5e-320f-4abf-bf29-3880436edc63/o?prefix=notebooks%2F
page.console Failed to load resource: the server responded with a status of 403 ()
page.http GET 403 https://storage.googleapis.com/storage/v1/b/fc-2dfeae5e-320f-4abf-bf29-3880436edc63/o?prefix=notebooks%2F
page.console Failed to load resource: the server responded with a status of 403 ()
page.http GET 403 https://storage.googleapis.com/storage/v1/b/fc-2dfeae5e-320f-4abf-bf29-3880436edc63/o?prefix=notebooks%2F
page.console Failed to load resource: the server responded with a status of 403 ()
page.http GET 200 https://storage.googleapis.com/storage/v1/b/fc-2dfeae5e-320f-4abf-bf29-3880436edc63/o?prefix=notebooks%2F

I would have expected the default timeout to be enough to include timeouts. 🤷


DatabaseLog level aware engine (task #5091)

Codecov Report

Merging #418 into master will decrease coverage by 0.14%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##             master    #418      +/-   ##
===========================================
- Coverage     17.24%   17.1%   -0.15%
- Complexity      638     643       +5
===========================================
  Files            54      56       +2
  Lines          2198    2216      +18
===========================================
  Hits            379     379
- Misses         1819    1837      +18

Impacted Files
Coverage Δ
Complexity Δ

src/Shell/DatabaseLogShell.php
0% <0%> (ø)
3 <3> (?)

src/Log/Engine/LevelAwareDatabaseLog.php
0% <0%> (ø)
2 <2> (?)

Continue to review full report at Codecov.

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


Fastlane action verify_build making invalid assertion

Hey All,
I've been playing with the new verify_build action (we were very excited to see it as we have our own custom scripts that do something similar).
I don't believe the following exception is valid:
raise "Inconsistent identifier found" unless plist['Entitlements']['application-identifier'] == "#{values['team_identifier']}.#{values['bundle_identifier']}"

Our app's bundle identifier is not prefixed by our team identifier but is prefixed with another value.  This makes the entire action unusable since the exception is raised regardless of whether or not you passed in 'bundle_identifier' to be verified.

@RobDay Is this still an issue for you?


[Feature Request] Dedicated Horizontal and Vertical Controls for Wheel and Text, separately

Seems like 'padding' sets both the horizonal and vertical at the same time... so its not possible to get the text, without a wheel, correctly placed on a given background, like the example where I want it in the lower FullDMD window but left justified-kinda.

I'll add the additional properties "Margin Top/Left/Right". This way, this should solve the issue.

thank you... this will make things work really nicely!

I've added margins and wheelsize properties and also fixed some calculations.
I assume the combination of these properties should be sufficient for now.
Part of 2.17.0

nice - thank you!


Extract Response factory from Message factory

i only read the discussion on slack now. but i think i agree that if we should split, we should aim for more consistency.
i think a client library is likely to only need to create requests, but never responses. creating the response is the job of the Httplug implementation/adapter. if i could typehint RequestFactory, it would be clear that i only want to create requests, never build responses from something. to me the consistent thing would be RequestFactory, ResponseFactory and a MessageFactory that can do both.
i know that this is inverse to the PSR-7 request and response, where message is the common base. i think for creating them, the common thing needs to be the union rather than a subset. the factories share no common methods as far as i can see.

i think a client library is likely to only need to create requests, but never responses

I don't think this is true. For example in case of SocketClient, the client creates responses from plain data. Any adapter, with an underlying client which is not PSR-7 compliant will return a response which must be transformed into a PSR-7 response...using a message factory.
And one last thought here: if you typehint for RequestFactory, you won't be able to create responses as you can't expect that it is a response factory or not. This is a bit similar to the case with async and not async.

RequestFactory, ResponseFactory and a MessageFactory that can do both

This would imply that you could create RequestFactories as well. Do you really want that?

i know that this is inverse to the PSR-7 request and response

Actually I don't think this is a problem.

sorry, i formulated that unclear. by "client library" i am thinking of a generic library that uses a Httplug implementation and the request factory to send requests. not client in the sense of HTTP.
with libraries that provide some functionality on top of Httplug, like FOSHttpCache for example, we only need to create requests, never responses (except maybe in tests but then we could mock the interface)

Ah, I see. Makes sense.
That still leaves the question open: do we actually want to implement RequestFactory objects separately? Currently I implemented a ResponseFactoryTrait which is not usable on its own, but can be used with Server Message and plain Message factories.

i don't think we should implement them separately, but somebody might
want to.
but more important, i think a library should be able to typehint it
separately, for code readability.

Ok, I think that is not a big deal. The typehinting thing actually makes sense.

@dbu I think this is ready.


[ETFE-2858][WIP] - Added movement tab

Need to look at how to retrieve some of the fields from the movement view response

There is a lot going on in this PR:

Got some reverse engineering from the movement view response from CoD. In CaM we know specifically what the movement type is but this gets abstracted when sent to Core so we have to figure out (using MovementScenario) what the movement type is
Reads in IE818 to figure out if the movement has been receipted, what the status was and when the movement items arrived
Should go in with https://github.com/hmrc/emcs-tfe/pull/110
Test data is not available for all the scenarios so I've had to rely on unit tests
eAD status explanation was not included in the copydeck so I asked UCD who provided a spreadsheet (link in ticket)
Refactored the coverage so that it ignores all the fluency models

James has kindly picked up the changes and a separate PR is going to be raised to address the caveats


Update the documentation to reflect the hooks that have been added and removed as of 0.1.6

The README should be updated as well.

Resolved with #138


Cannot add to homebridge

LG Air Conditioner Model
model RAC_056905_WW
Describe the bug
Can detect AC but error as below.
Symptom check

Does your unit appear in the Home app? No
Is the current on/off state, temperature, and mode being reported? No
Does your unit respond to the controls? No

Relevant logs
[10/24/2020, 6:42:41 AM] [LgThinqAirConditioner] Discover found 1 total devices
[10/24/2020, 6:42:41 AM] [LgThinqAirConditioner] "Living Room AC" (model RAC_056905_WW) uses the thinq1 platform, which is not supported. Please see https://github.com/sman591/homebridge-lg-thinq-ac/issues/4 for updates.

Hi there! Thanks for writing in.
It looks like your device communicates over the legacy ThinQ 1 API, which is not supported yet. Please follow #4 for updates until this is complete


Jira version updated to 8.11.0

Jira version updated to 8.11.0

@pavanp21 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


How can I remap

Did you check existing requests?

[X] I have searched the existing issues

Describe the feature
How can I remap
["<C-s>"] = "actions.select_vsplit",
["<C-h>"] = "actions.select_split",

Provide background
No response
Additional details
No response

You can set them to false to disable the keymap, or you can put anything in the right hand side that you could pass to vim.keymap.set.

If you managed to do it, can you please share how?
I did
["<C-k>"] = "actions.select_vsplit",

but nothing seems to change, I've also tried with a lot of other Keybinds and also tried to make an autocommand like suggested in another issue, but it still doesn't work

This was my experience @sbancuz .  Same exact behavior as you described.   Ultimately, I GAVE UP trying to remap it in my local config and ultimately changed the source located at .local/share/lunarvim/site/pack/lazy/opt/oil.nvim/lua/oil/config.lua
@stevearc suggestion did not work for me. Maybe @stevearc can investigate this further.

Can you provide a minimal repro? I used this file and with nvim -u repro.lua . I found that <C-k> opened a vsplit and <C-s> did nothing, both as expected.
-- save as repro.lua
-- run with nvim -u repro.lua
-- DO NOT change the paths
local root = vim.fn.fnamemodify("./.repro", ":p")

-- set stdpaths to use .repro
for _, name in ipairs({ "config", "data", "state", "runtime", "cache" }) do
  vim.env[("XDG_%s_HOME"):format(name:upper())] = root .. "/" .. name
end

-- bootstrap lazy
local lazypath = root .. "/plugins/lazy.nvim"
if not vim.loop.fs_stat(lazypath) then
  vim.fn.system({
    "git",
    "clone",
    "--filter=blob:none",
    "--single-branch",
    "https://github.com/folke/lazy.nvim.git",
    lazypath,
  })
end
vim.opt.runtimepath:prepend(lazypath)

-- install plugins
local plugins = {
  "folke/tokyonight.nvim",
  {
    "stevearc/oil.nvim",
    config = function()
      require("oil").setup({
        keymaps = {
          ["<C-s>"] = false,
          ["<C-k>"] = "actions.select_vsplit",
        },
      })
    end,
  },
}

require("lazy").setup(plugins, {
  root = root .. "/plugins",
})

vim.cmd.colorscheme("tokyonight")

I can't replicate the bug anymore, maybe I was just configuring it wrong after following some old issue. But what you provided works. Sorry to have bothered you for nothing.

Ah, no worries. Could have been a typo; happens to the best of us


Allowlist for Authorize Request Objects

We would like to implement an allowlist for Authorize Request Objects so we can prevent requests to untrusted sources.
The spec says:

To prevent such attack to succeed, the server should (a) check that
the value of "request_uri" parameter does not point to an unexpected
location."

https://tools.ietf.org/html/draft-ietf-oauth-jwsreq-23#section-10.4
I'm aware we can implement our own IJwtRequestUriHttpClient and implement the behaviour described in 10.4.* and create a list with allowed endpoints that we can request.
We can also set a property on the Client with an allowlist so we can prevent requests to unexpected locations.
How do you feel about this?

We won't add this to the core - but added all the necessary extensibility points we think.
The long term solution to this will be PAR (https://tools.ietf.org/wg/oauth/draft-ietf-oauth-par/) IMO.

Out of curiousity: why don't you consider this as a core feature?
I can confirm that the extensibility points are in place so it's quite easy to prevent requests to untrusted sources.
PAR makes sense to me, this helps mitigate possible SSRF issues.

because
https://tools.ietf.org/html/draft-ietf-oauth-jwsreq-25#section-10.4
And as stated above - the real solution IMO is to provide a "drop location" for jws requests. And that's called PAR. We will add PAR once it is stable.


[Candidate 1v1 Tournament Himalaya V2] Title text is not concise

Issue Description
Title text says: "MISSING: 'Tourn.Himalaya Made By A5SA5SIN Play Fair And HaveFun'"
Expected behavior
Title text says "Tournament Himalaya, by A5SA5SIN"
Screenshots

before i release version 3 do you think this text is ok?

Fixed with https://github.com/xezon/GeneralsRankedMaps/commit/9c3d6405a7e5aee03da0eba37f37615db63c09e7


Packaged .exe

How to  packaged .exe ?
I used  'gulp -- vscode-win32-x64'  command ,but have no .exe
I used electron command. but it can't work.

I want a  installation package ,like  'VSCodeSetup-ia32-1.32.3.exe'.


Using RTree to select features for tiles

This PR

Adds RStar, geo and geo_types as a dependencies
Generates an RTree in webmercator space and inserts objects representing the features
Queries the RTree to get the potential contents of a given tile
Populates the tiles by iterating over the and selecting the contents

I think it basically works, there are a few potential optimizations

Currently we are iterating over all possible tiles in each zoom level. It should be able to exclude ranges of tiles based on the  bbox of the lowest level envelope in the RTree
There is a copy in the conversion of geo_json::Feature to TreeFeature which I think can be removed
We should be able to exclude the child tiles of an empty tile from processing. If a level 2 tile is empty then it's descendants will be too. This might be a marginal speed up though as the RTree should already catch this

This is awesome, thanks! For a real input from Edinburgh, it takes the total time from 35s down to 30s. And after the CachedEnvelope, down to 27s. Happy to merge this and keep iterating on ideas for pruning which tiles to calculate

Are those timings using release?

Are those timings using release?

Yes. time cargo run --release ../examples/edinburgh/output/output.geojson (after building -- this doesn't include compilation). I can send over that GJ input, or you can generate it by cd examples/edinburgh; python3 setup.py; cargo run --release config.json


File outside of Vite serving allow list

Describe the bug
I am building a Svelte component library, I am using Storybook and would like to import a .css file inside my components, more precisely inside the preview.ts file of storybook. My .css file is inside my static folder, when I try to import it I get the error:
The request url "/Users/myapp/static/app.css" is outside of Vite serving allow list.

- /Users/myapp/src/lib
- /Users/myapp/src/routes
- /Users/myapp/.svelte-kit
- /Users/myapp/src
- /Users/myapp/node_modules
- /Users/myapp/.storybook

I read the docs and to fix this issue it seems like the solution would be to put the file inside my vite serving allow list like this:
  server: {
    fs: {
      allow: [
        searchForWorkspaceRoot(process.cwd()),
        "/static/css/bootstrap-italia-custom.min.css",
      ],
    },
  },

But nothing seems to change, I tried putting the full url, I tried by selecting only the parent folder, I tried without the searchForWorkspaceRoot utility, but nothing seems to woek. What am I doing wrong?
Reproduction
Small repo created from npm create svelte@latest with only storybook added as dependency: https://github.com/lungarella-raffaele/vite-issue to recreate the problem
Steps to reproduce
npm run storybook
Vite cannot serve the app.css file even though it is specified in the configs
System Info
node: v21.7.3
MacOs Sequoia

Used Package Manager
npm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

Probably storybook is manipulating server.fs.allow, so you should check with them.
Usually server.fs.allow should be process.cwd() = "/Users/myapp" and that shouldn't cause the issue.
For example, you can enforce this by the plugin for the time being:
export default defineConfig({
  plugins: [
    sveltekit(),
    {
      name: "config",
      enforce: "post",
      config(config) {
        config.server.fs.allow = [process.cwd()];
      },
    }
  ],
})


Sinkhorn iteration is not converging in A100 GPU

I have tried the following code in A100 and other GPU. I found in other GPU, such as T4, V100, A40, this Sinkhorn iteration will converge, but in A100 GPU, the Sinkhorn iteration is stuck in some value until the maximum iteration is reached. Do anyone know what happened here?
To Reproduce in A100 GPU
n           = 50
𝜀           = 1
threshold   = 0.01

threshold   = 1e-2
blob_std    = 1

x, data_mem   = make_blobs(n_samples=n, n_features=4, centers=1, cluster_std=blob_std, random_state=1)
mu                 = np.ones((n,)) / n # size of M
nv                  = np.ones((n,)) / n # size of N
w                    = np.random.normal(size=[4,2])
w_int               = w + 1 * np.random.normal(size=[4,2])
y                      = x.dot(w) + 1 * np.random.normal(size=[n,2])
y_pred            = x.dot(w_int)
y_jx                = jnp.array(y)
pred_jx          = jnp.array(y_pred)

from ott import utils
geom = pointcloud.PointCloud(pred_jx, y_jx, epsilon=𝜀)
prob = linear_problem.LinearProblem(geom, a=mu, b=nv)
solver = sinkhorn.Sinkhorn(
        threshold=threshold, inner_iterations=200, lse_mode=True, max_iterations=1000,
        norm_error=1,use_danskin=False, progress_fn=utils.default_progress_fn()
    )
out = solver(prob)

The screenshot of the output in A100 is

The screenshot of the output in T4 is

Hi @yexf308 , I can reproduce the above on A100 using:

As for the reason, I think there are things at play:

The values of in the cost matrix is really big, it's in around [0.15, 417], which can lead to numerical instability. One way to solve this is to scale the cost matrix, e.g., by the mean as scale_cost='mean'. Other solution is to use 64-bit precision.
XLA seems to generate different code for CPU/GPU/other accelerators, so I assume this is where the CPU/GPU discrepancy comes in; unfortunately this is out of our control

@michalk8 I have checked out.matrix and compared with the output error. The out.matrix is actually converged to desired coupling matrix within the threshold, the error calculating is actually in-correct. That is why I suspect there is some bugs. Thanks for the quick reply!

Here is the screenshot.

This issue is resolved if one uses double precision.


Security error, Error message: Sequence contains no matching element

When trying to document a solution (worth mentioning that this is a Dataverse for Teams one) I always bump into this. The documentation of solution is not complete and cut after name of first Dataverse table.

attached solution in txt
solution.txt

Just saw that I already fixed this separately (https://github.com/modery/PowerDocu.Common/commit/b3e55640af15e10aa277604541a134a40835fd27), so the next release will no longer throw this error. Aiming to publish it sometime next week

I notice that the release is still 1.2.3. When are you releasing the new version with the bug fix?

Apologies for the delay, work and general life took up a lot more time in recent weeks, so barely touched PowerDocu. I'll try to get the next version ready over the upcoming long weekend

1.2.4 has just been released. Let me know if the problem still occurs
https://github.com/modery/PowerDocu/releases/tag/v-1.2.4


Expected to return a value at the end of async arrow function  consistent-return

Why do I get this error?  Can someone please help me?
"Expected to return a value at the end of async arrow function  consistent-return."
I use it on an onChange since I would like the URL put in the input to be used.
const handleChange = async (event) => {
try {
const article = await extract(event.target.value)
return article
} catch (err) {
console.trace(err)
}
}

@niamao it seems your handler only returns value in the case everything work well, and returns nothing when there is error occurs.
Try this:
const handleChange = async (event) => {
  try {
    const article = await extract(event.target.value)
    return article
  } catch (err) {
    console.trace(err)
  }
  return null;
}


Consider using vcpkg

Currently the version 2 is still a bit too unstable and version 1 isn't really maintained.
I'll be looking into this in a month or two when I have finished cleaning everything up and re-added all the features missing that were in v1.


revert parser for json files

This commit resolves #268.
The conflict was not in the prettier settings per se. Instead, when lint-staged is run as the last step before a commit, it is invoked using npx and the remote config is using the json parser on JSON files. The other tooling for the repo, most specifically vscode, uses json-stringify for the JSON parser.
This difference in parsers was causing the locally-built (via build script) and committed JSON task files to be reformatted with arrays on a single line. Using the json-stringify parser will instead keep the arrays as multiline, which aligns with how all of the JSON task files have existed in the library prior to adding prettier to the repo.
To preserve the usage of Shopify's prettier settings, a new prettier.config.js file was created to import and export those settings while adding an override on the JSON parser.
This commit also updates the affected task JSON files that were committed since the prettier addition.

Will this cause all of the updated task to show as Updated in the interface?

Will this cause all of the updated task to show as Updated in the interface?

No, because their actual attributes have not changed. :) Glad you're thinking about that though!


(PUP-1293) Don't ignore status when using upstart provider

As requested by @baurmatt (I believe), here's a PR to backport the fix for PUP-1293 to 3.x.

Awesome, thanks! :)


How to handle nested structures in ent?

For example, I got an json from front-end like this:
{
	"name": null,
	"email"<EMAIL_ADDRESS>        "keys": {
            "publicKey": "balabalbalblabsldbalbsdlbasldblasbdlb",
            "encryptedPrivateKey": "asdasdasdaksjheqwjhekjqwhekjqhwkejhqwkje"
        }
}

I use ent to init an account with
ent init Account

And I want to decode this json to ent.Account, but how to write code about keys in Account's Fields function?
// Fields of the Account.
func (Account) Fields() []ent.Field {
	return []ent.Field{
		field.UUID("id", uuid.UUID{}),
		field.String("name").
			Optional(),
		field.String("email").
			Match(regexp.MustCompile(`(\w[-._\w]*\w@\w[-._\w]*\w\.\w{2,3})`)).
			Unique(),
		// code about keys
	}
}

Hey @404cn,
For cases like this, I would use a JSON type. See docs .

Hey @a8m , thanks for you answer.
I try to write it like this:
type Keys struct { PublicKey, PrivateKey string}

func (Account) Fields() []ent.Field {
	return []ent.Field{
		field.UUID("id", uuid.UUID{}),
		field.String("name").
			Optional(),
		field.String("email").
			Match(regexp.MustCompile(`(\w[-._\w]*\w@\w[-._\w]*\w\.\w{2,3})`)).
			Unique(),
		// code about keys
                field.JSON("keys", &Keys{}),
	}
}

But in generated code the Keys type is *schema.Keys and an error here because the generated code didn't import package scheme

What version of ent do you use?
Can you try to move the Keys object and re-run the codegen? Does it work in this case?
Thanks

version 0.2.5
Yes, if I remove the Keys object it will work.
But this json is defined in the front-end, I cant't change it .
What I want to do is only use ent.Account one struct, decode request.Body into it, and use it to save data to database.
And now the Keys object is a nested struct in struct Account, I had a look at doc but still don't know how to make it work.
Although I can write another Account struct, decode json into it and use CreateAccount to save this Account info into ent.Account,  but you know, it looks a bit redundant.
BTW: Sorry for my pool English.

Yes, if I remove the Keys object it will work.

Can you create a small schema to reproduce this (or in a temporary repo)?

What I want to do is only use ent.Account one struct, decode request.Body into it, and use it to save data to database.

You can decode the request body to this struct, but in order to create it or update an existing one, you must use the relevant builder explicitly as follows:
client.Account.Create().
    SetName(a.Name).
    SetEmail(a.Email).
    // ...
    Save(ctx)

If you want to do it with "one command" (like ScanAndSave(req.Body, ctx)), you can generate a custom function for you using external templates that will call the relevant setters based on your business logic.

Here it is.
I got this json from front-end:
{
	"name": null,
	"email"<EMAIL_ADDRESS>        "keys": {
            "publicKey": "balabalbalblabsldbalbsdlbasldblasbdlb",
            "encryptedPrivateKey": "asdasdasdaksjheqwjhekjqwhekjqhwkejhqwkje"
        }
}

I can write a struct
type Keys struct {
    PrivateKey string
    PublicKey string
}

type Account struct {
    Name string
    Email string
    Keys Keys
}

And decode json into Account
_ := json.NewDecoder(request.Body).Decode(&Account)

Then use sql statement likeinsert into account ... to insert account info to database.

Now I use ent and want write code like this:
Scheme:
// Fields of the Account.
func (Account) Fields() []ent.Field {
	return []ent.Field{
		field.UUID("id", uuid.UUID{}).
			Default(uuid.New),
		field.String("name").
			Optional(),
		field.String("email").
			Match(regexp.MustCompile(`(\w[-._\w]*\w@\w[-._\w]*\w\.\w{2,3})`)).
			Unique(),
                 // assume this will work, this is the part that I don't know how to write
		field.JSON("keys", &struct {
                         PublicKey string
                         PrivateKey string
                 }),
	}
}

And the decode part like this, use ent.Account:
_ := json.NewDecoder(request.Body).Decode(&ent.Account)

If I remove the Keys object from scheme, it will says can not decode, missing Keys
If I don't remove the Keys object, I don't know how to write Account's scheme............😰😰
And a function CreateAccount:
func CreateAccount(ctx context.Context, client *ent.Client, account ent.Account) (*ent.Account, error) {
	a, err := Client.Account.
		Create().
		SetName(account.Name).
		SetEmail(account.Email).
                // assume this will work
		SetKeys(account.Keys).
		Save(h.Ctx)
	if err != nil {
		return nil, fmt.Errorf("failed creating account: %v", err)
	}

	return a, nil
}

Hey @404cn and sorry for the delay.
Let's make it simpler - can you please create a temporary repository for this schema and the stuff that are not working for you and I'll give it a look? You can make it private and send me an invitation.
Thanks

Done
field.JOSN("keys", &struct {PublicKey, PrivateKey string}{}),


Real support for SwiftPackageManager

I tried to integrate the library using SwiftPackageManager and it fails, from just browsing the Sources I notices the library uses storyboards and xib files, only the XCode 12 beta supports packages with assets. How this library can support SPM then?
Thank You for your help in advance, I can't use the Carthage package because it fails to compile with the new XCode 12 beta versions:

Build version 12A6163b

Only logic part of the framework was ever supported by SPM, meaning you'd have tracking abilities and could log leaks etc but not visual aspect of it

Any update on this? We are switching to SPM and loosing LifetimeTracker would be a shame :worried:

I just ran into this as well so I made a PR https://github.com/krzysztofzablocki/LifetimeTracker/pull/74

🙇


Get Shares Link

Can you give an example of getting the shared link please? So far I'm able get most everything working I need to except that.
Thank You,
Jim M

@jmanring have you been able to get the create folder working I am getting a Call to undefined function Box\Models\Request\array_key_exists_and_not_null()
if (array_key_exists_and_not_null($args, self::FOLDER_UPLOAD_EMAIL)) {
$args[self::FOLDER_UPLOAD_EMAIL] = new BoxEmailRequest($args[self::FOLDER_UPLOAD_EMAIL]);
}
any clue?

Came here for answers to the same issue. I am unable to get the shared_link to work, I'm attempting to set access to open with no luck. Was anyone able to get this this to work?


TCPShield v2.5

This PR adds support for Geyser, Fabric and Paper without ProtocolLib and removes the illegal access message in some cases.
Geyser
Geyser support only targets Floodgate 2.0 (still in beta).
Huge, huge thanks to the whole GeyserMC team and specifically Tim203 for their incredible support on this journey.
Native Paper
Native paper support depends on a PR only merged recently so it only supports Paper 1.16 build #503 or higher.
Illegal Access
The illegal access message is removed in cases where it isn't needed. The final field is now queried lazily (when it's needed).
Fabric
see #40

Floodgate can just reference the API module api; using the Spigot dependency is unnecessary here.


Question About Account Filter

hi guys,
one question about account filter in the following section.
nem2-docs/source/concepts/account-filter.rst
what's the difference between left-side circle and right-side circle in the diagram(concepts-account-properties-array.png)?
both circles show that account receive txs from the addresses in "Allow" circle and don't receive txs from other addresses, right?
would be grateful if you would let me know the difference .
thanks!

Hi @ryo-opening-line,
Thanks to your comment, we have noticed that allow and block arrays are mutually exclusive. The next commit will have the image updated image and a more detailed explanation.
The "bison" branch is work in progress. Don't take any information as final until is merged to master branch.


Changing the port json-server runs on?

Is it possible to run the app on a different port than the default 3000?

1
Would be great to have kind of Settings file to configure.
https://github.com/typicode/json-server/issues/185 could be an option in this file as well. Just as an example.

Sure json-server -p 4000 db.json

file package.json (Windows)
"scripts": {
"start": "set PORT=4000 && react-scripts start"
}
file package.json (Ubuntu)
"scripts": {
"start": "export PORT=4000 && react-scripts start"
}

package.json     "start" : "Port = ####  react-scripts start"


Hello

Hello there!
I was just questioning what you are trying to do with my code. You had withdrawn my name from it so I assume you are not trying to develop it. This was code that I got and it could land you in some legal problems if you try and steal it. I am awaiting your response.
Echo

@LearningGamesForSchool would you like for me to pursue legal action?

@3kh0 You do know that because you have your code as open source, people can do this? If you don't want this, close the pr and don't merge.
@LearningGamesForSchool You should have forked this and updated it on your fork, instead of trying to push to the original, that was stupid.
(use mit licence so they have to put your name on it if they want to use, currently your licence does almost nothing :)


Fixed typo in docs: connnection => connection

As already established in #272, test failure is due to intermittently failing tests (requesting network resources). I only changed docs.

This landed apparently: https://github.com/cloudflare/pingora/commit/2d30077683f2c18e22f36b2872bcc7eaca85c505


fix: Spacecat API service responds with 503 when a large import is started

This PR moves the creation of URL records and queuing of URL messages (for the scraper) to the content-import-worker. They API service now simply creates the job and sends a single message with ALL URLs to the content-import-worker in the following form:
{
  "processingType": "import",
  "jobId": "f91afda0-afc8-467e-bfa3-fdbeba3037e8",
  "urls": [
    "https://www.example.com/path/to/resource/1",
    "https://www.example.com/path/to/resource/2",
    "https://www.example.com/path/to/resource/3"
  ]
}

Related Issues

SITES-23834

:tada: This issue has been resolved in version 1.63.5 :tada:
The release is available on:

v1.63.5
GitHub release

Your semantic-release bot :package::rocket:


Styles and themes

Most of this is preparation for further materialization - more to come....
This also changes the colors a bit, I tried to compromise between what we have and the material guidelines here. Screenshot of what it looks like on API 22:

Looks good! Will merge when travis is happy - especially like the coordinates in the screenshot ;-)

credits to the location go to genymotion - made me smile when I first used
it ;)
On Fri, Oct 16, 2015 at 2:19 PM, ligi<EMAIL_ADDRESS>wrote:

Looks good! Will merge when travis is happy - especially like the
coordinates in the screenshot ;-)
—
Reply to this email directly or view it on GitHub
https://github.com/criticalmaps/criticalmaps-android/pull/94#issuecomment-148701949
.

:+1:

nice!


Update ediscovery-troubleshooting-common-issues.md

added another workaround to fix Error/issue: Downloaded export shows no results.
removing old searches fixes this issue.

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/4714

@msbemba PR merged. Thanks for making this fix. Appreciate it!


Best way to send an image/attachment via the browser?

Been digging into this for a few hours now, and I'm not sure what the recommended way to upload a file to smooch + attach it to a message via the browser is at the moment. I'm using 4.3.0, and initing with const api = new SmoochCore({ appToken: smoochAppToken });
I have a coworking who's working with smooch via the iOS SDK, and he's able to upload 7mb images and send them as messages, but I can't seem to do the same thing over the web, so I'm wondering if I'm doing something wrong or if requests from the web just have a filesize restriction.

Older docs recommended using appUsers.uploadImage, but throws request entity size errors for files over ~2mb
POSTing directly to https://api.smooch.io/v1/appusers/${this.userId}/images with a source of the file, a role of 'appUser' and an app-token header, produces the same result — "entity too large" errors. This is the exact route our iOS library is hitting, with the same files, but it doesn't seem to get any errors.
There doesn't seem to be a an attachments object on the api (this is mentioned in the readme, but it's undefined on the SmoochCore instance itself) that I can use.
POSTing to (what looks like) attachments with the following code throws a 401 (unauthorized) error.

    const body = new FormData();
    body.append('source', image);

    axios.post(`https://api.smooch.io/v1/apps/${smoochAppId}/attachments?access=public`,
      body,
      {
        headers: {
          'app-token': smoochAppToken,
        },
      },
    )

I guess, in summary, my questions are:

What is the current, recommended method for sending a file via the API with this lib?
Is there a reason you can think of that an iOS API consumer would be able to send larger files without issue?

What is the current, recommended method for sending a file via the API with this lib?

To send a file via the API, you need to be authenticated with an app JWT, which brings me to the next question...

There doesn't seem to be a an attachments object on the api (this is mentioned in the readme, but it's undefined on the SmoochCore instance itself) that I can use.

When you instantiate SmoochCore with an appToken, only the APIs that are accessible via appToken are exposed on the resulting object. Anything else would result in 401 anyways.

Is there a reason you can think of that an iOS API consumer would be able to send larger files without issue?

The iOS SDK actually resizes the image before sending it to the /images API. That's why it's able to send larger files. You could attempt a similar method before attempting the upload via the browser, but note that you will be limited to uploading images only.
For an example of what this would look like, you can take a look at our implementation in smooch-js.
If you'd like to attach different file types, the best way is to call the attachments API from your backend, using an app scoped JWT, then sending the message with the resulting attachment.
I realize this is a fair amount more work, but it's the only safe way to do this currently.
Hope that helps.

Thanks @jugarrit, this is an extremely helpful response. We have our own file endpoint/storage system, so we'll continue uploading to that and sending those as mediaUrl with our messages. This solution works for us, I just had a request to make it as fast as it is on iOS, so your explanation of why iOS is so fast is exactly what I wanted to hear.
Thanks again!


wback Exception Handling

WebAssembly provides try, catch, catchall, throw, rethrow and delegate instructions for handling exceptions. However the exceptions which can be handled by the catch and catchall blocks are the exceptions which are explicitly thrown using the throw instruction.
In the following program if the second parameter is 0 it is expected to throw a division by zero error. But this error is not caught by the catchall block. It is caught by the runtime.
(module
  (type $i64_=>_none (func (param i64)))
  (type $i64_i64_=>_none (func (param i64 i64)))
  (import "console" "log" (func $println (param i64)))
  (export "testFunc" (func $testFunc))
    (func $testFunc (param $0 i64) (param $1 i64) (local $2 i64)
      (try
        (do
          (local.set $2
            (i64.div_s
              (local.get $0)
              (local.get $1)))
          (call $println
            (local.get $2)))
        (catch_all
          (call $println
            (i64.const -1))))))

Following works because we are explicitly throwing the error when second parameter is zero.
(module
  (type $i64_=>_none (func (param i64)))
  (type $i64_i64_=>_none (func (param i64 i64)))
  (import "console" "log" (func $println (param i64)))
  (tag $a-tag)
  (export "testFunc" (func $testFunc))
  (func $testFunc (param $0 i64) (param $1 i64)
    (try
      (do
        (if
          (i64.eq
            (local.get $1)
            (i64.const 0))
          (throw $a-tag)
          (call $println
            (i64.div_s
            (local.get $0)
            (local.get $1)))))
      (catch $a-tag
        (call $println
          (i64.const -1))))))

Exceptions can be handled using the try catch in the JS file.
Considering the div3-p.bal.
Expected output is,
panic: divide by zero
div3-p.bal:9

This contains a line number in the original bal file relevant to the error instruction.
Actual output is,
panic: divide by zero

In the error object in Javascript. It has two properties. The message and the stack.
The stack contains details where the exception was thrown.  But it is a line in the WASM file. The line of the error output is missing in the actual output.

Range of ints representable in WASM
-9223372036854775808 -<PHONE_NUMBER>854775807

Range of ints representable based on the test cases
-9223372036854775807 -<PHONE_NUMBER>854775807

When outside this range it is expected to throw an Error.
In but in WASM it does not throw an error for this.(Behaves in a circular way:<PHONE_NUMBER>854775807 + 1 = -9223372036854775808).
Therefore should the handling of this Exception be done on the WAT code.
if op1 > 0 && op2 > 0 && result < 0
  throw "overflow"

Debug line info options:
a) Differ this for a later subset.
b) "Bake-in" the lines as part of error creation
c) Use https://yurydelendik.github.io/webassembly-dwarf/
Lets go with a) for now

Wrapping
https://github.com/WebAssembly/design/blob/main/FutureFeatures.md#integer-overflow-detection
I think we'll have to do a per-check, but your proposed per-check is wrong. Please take a look at https://www.cs.utah.edu/~regehr/papers/overflow12.pdf

btw ballerina ints can have the value -9223372036854775808

Following checks for overflow,

For addition:

if (s1 > 0) && (s2 > 0)
  if (s1 > (INT_MAX − s2))
    throw overflow
else if (s1 < 0) && (s2 < 0)
  if s1 < (INT_MIN − s2)
    throw overflow

For substraction:

if (s1 > 0) && (s2 < 0)
  if s1 > (INT_MAX + s2)
    throw overflow
else if (s1 < 0) && (s2 > 0)
  if s1 < (INT_MIN + s2)
    throw overflow

For multiplication:

if ((s1 > 0 && s2 > 0)
  if (s2 > INT_MAX/s1))
    throw overflow
else
  if abs(s2) > abs(INT_MIN/s1)
    throw overflow

For division:

if (s1 == INT_MIN) && (s2 == -1)
  throw overflow

Possible methods,

Creating a seperate WASM Module for checking overflow and importing the functions from that module to do the checks and throw errors.
e.g.: main.wat, overflow.wat, runtime.js
Defining overflow checking functions and importing them to the module.
e.g.: main.wat, runtime.js

Using inline checks for handling overflows


Question: random access to elements

I am trying to write some sort of header where I may have a list of offsets. E.g. 1 offset per record.
How could I implement a schema that is conceptually a vector<int64> or at least blob in such a way, that I can address individual elements without reading the lot?
The reason is, I am trying to write a big data stream that may be, say, 3-4TB and may consist of, say, 10 billion records. For this stream, just the header would be some 80GB. We really need to avoid deserializing the lot.

Bond expects to deserialize entire objects and that is fairly deeply baked in. I think it would be best to build your index header outside of Bond. Each record can be serialized/deserialized individually using Bond, but the index and the collection of records would not be Bond objects themselves. Does that make sense?

Yes, it does. That was my current design, actually. Just wanted to make sure that I am not missing anything. Thanks


indexer-agent: maxAllocationEpochs not being considered when identifiyng expired allocations

Multiple indexers have reported that the Indexer Agent will close and renew allocations one epoch after creation.
This means that the maxAllocationEpochs value provided by the Staking contract is not being considered when identifying expired allocations during the reconciliation loop.
### Requirements
- [x] The Agent must only close allocations when they are duly expired.
- [x] For a better indexer experience, add logs that inform how the allocation expiration epoch was calculated: either by an indexing rule, the `maxAllocationEpoch` value obtained from the Staking contract, or the minimum default of one epoch.

### Success Criteria
- [x] **Manual Test**: Confirm (on-chain) that allocations are closed `N` epochs after creation, where `N` is the expected value for `maxAllocationEpochs`. _(considering governing indexing rules have no value set for `allocationLifetime`)_

After a thorough review of user feedback suggesting a potential bug, we've found evidence indicating that the Agent is functioning as intended.
Logs show that maxAllocationEpochs value is being correctly fetched from the Staking contracts and being considered in the desired allocation lifetime calculation as designed.

Closing this issue. See comment above for explanation.


Add Terraform validate test

Add a test that will scan for all Terraform modules and run init and validate on each.

Closing this out due to inactivity. Re-open if desired.


chore(CPlusPlus): add reverse stack

added algorithm for reversing the stack

@ming-tsai Please change the title to chore(CPlusPlus): add reverse stack

Please resolve the conflicts coming as I have no access to do it

@anirudh09041 any changes??

Warning❗❗
@anirudh09041  if you do not respond in 2 weeks then we will close the PR

Any changes required

Any changes required

yes I have mentioned above  check it


Debugging a single file

Environment

Elixir & Erlang versions (elixir --version): 1.10.4
Operating system: Linux (Debian)
Editor or IDE name (e.g. Emacs/VSCode): VSCode
Editor Plugin/LSP Client name: ElixirLS debugger v0.5.0

Troubleshooting

[x] Restart your editor (which will restart ElixirLS) sometimes fixes issues
[x] Stop your editor, remove the entire .elixir_ls directory, then restart your editor

NOTE: This will cause you to have to re-run the dialyzer build for your project

If you're experiencing high CPU usage, it is most likely Dialyzer building the PLTs; after it's done the CPU usage should go back to normal. You could also disable Dialyzer in the settings.
Description
I am trying to set up a debug launch configuration which only runs tests for the currently open file. In order to ensure the issue is not with VSCode variables, I have used a static file path. I've tried passing the file in as an option to taskArgs as well as changing the requireFiles to only include the file in question. No matter what I do, the debugger runs through all of the test files in the project.
Does ElixirLS support this use case? If so, how could I achieve the desired result?

I can confirm it doesn't work with task arg test/some_test.exs. It does with test/some_test.exs:12.

After more investigation it seems that mix test some_test.exs will not honour some_test.exs task param when there are other *_test.exs files required by requireFiles option in launch config. Please note that the default one requires
"requireFiles": [
    "test/**/test_helper.exs",
    "test/**/*_test.exs"
  ]

The workaround is to only require the file you want to debug or not require *_test.exs at all. The downside is that you won't be able to set breakpoints in not required .exs - as pointed out in the readme.
I'll update the readme with the workaround.
@josevalim If you believe it may be a mix issue please let me know

The require files should only be the test helper, at least is its purpose on Mix. The others are loaded on demand depending on what is being tested. While are all test files being required?

If the files are not required :int will not accept breakpoints with error
** Invalid beam file or no abstract code: 'Elixir.SomeTest'
see
https://github.com/elixir-lsp/elixir-ls/blob/99b6d7800b253f1c06a81bd29455b557a1243505/apps/elixir_ls_debugger/lib/debugger/server.ex#L671

Unfortunately I do not know how the debugger works to have an opinion on that. All I can say though is that, if you require the test files, then they will be required, and it is not the job of mix to filter the list you asked it to load. :)


PWA updates

Connected to #67

added the fallback content and styling to speed up first meaningful paint from 1.8s to 770 ms (both metrics measured with Lighthouse for bundled version serving locally by polymer-cli)
aligned the Service Worker config and manifest.json with the examples by Polymer team
documented basePath-related stuff, added description how the default build config for prpl-server differs from the one required to serve bundled version with polymer-cli

This change is

Reviewed 9 of 9 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


reorganize pi_utils

The "pi_utils" folder contains everything that is useful for the raspberry pi, it is better to separate what is utils to the pi zero from what is util to the main pi. In particular for the updates

I will delete these .env files and create a PR to add in the README steps to launch the system

Nice! There is one unittest failing though
And I suggest adding .env to the gitignore to avoid having people commit their file later on :)


SQL Kernel: A T-SQL variable is not visible / accessible outside of cell context

Issue Type: Bug
Step 1. Create a new cell using Add Code and paste this:
 declare @a int = 100  select @a
Step 2. Create yet another cell using Add Code and paste this:
select @
Step 3. Run Cells
You will get this error which indicates that variables in the SQL kernel are only cell-scope. In the Python kernel they are available in the entire Notebook
Msg 137, Level 15, State 2, Line 1
Must declare the scalar variable "@a".
Azure Data Studio version: azuredatastudio-insiders 1.13.0-insider (34fe2b44cc3960d9ac46cd3655d1516ff050a6d6, 2019-10-24T07:49:24.669Z)
OS version: Windows_NT x64 10.0.18362

Hi @PiJoCoder. For now, this is known behavior.

Hello.  I know this isn't the most exciting feature to implement.  It is quite serviceable and I'm wondering how we might gain some traction for its implementation.  I can see this being valuable in preparing some training material for debugging and troubleshooting stored procedures by breaking down the individual statements.
This behavior does deviate from a Python notebook and is an item I've been eagerly anticipating being resolved.  If there could be given consideration to its placement in the backlog and advancing it forward, it would be appreciated.  Thank you in advance for reviewing my comments.
Respectfully,
Peter

@kburtram @chlafreniere eagerly waiting.......🙄

It is acrucial funtion, should be a priority. That's how notebooks work in other languages. For instance you want to execute 5 queries with the same date range, that changes between workbook executions. Otherwise you are better off with search and replace in notepad.
Workaround: You can use the session context to store and retrieve the variables. See this approach: https://www.ericgharrison.com/?p=418


I'm closing this.


Update README.md

Since Prisma now supports Node v14 and above, this means Top-level await is now ready to be used in ES modules.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


"hasSameElementsAs" compilation error

Summary
The code:
java.util.List<? extends T> list = new ArrayList<>();
java.lang.Iterable<? extends T> iterable = list;
org.assertj.core.api.Assertions.assertThat(list).hasSameElementsAs(iterable);

Causes compilation error in Eclipse:
The method hasSameElementsAs(Iterable<? extends capture#5-of ? extends T>) in the type AbstractIterableAssert<ListAssert<capture#5-of ? extends T>,List<? extends capture#5-of ? extends T>,capture#5-of ?
 extends T,ObjectAssert<capture#5-of ? extends T>> is not applicable for the arguments (Iterable<capture#6-of ? extends T>)

and in Java9:
error: incompatible types: Iterable<CAP#1> cannot be converted to Iterable<? extends CAP#2>
		assertThat(list).hasSameElementsAs(iterable);
		                                   ^
  where CAP#1,CAP#2 are fresh type-variables:
    CAP#1 extends Number from capture of ? extends Number
    CAP#2 extends Number from capture of ? extends Number

Java 8 specific ?
I don't know

This is not specific to AssertJ, you are trying to compare elements that may not be of the same type.
<? extends T> means a T or a subclass of T, if T is a Number you might be comparing a list of Double with an Iterable of Long which is not something Java allows (that's what the cryptic error with capture means).
See http://howtodoinjava.com/core-java/generics/java-generics-what-is-pecs-producer-extends-consumer-super/ for a more in-depth explanation.

So what is the proper use case of assertThat(list).hasSameElementsAs(iterable); ?

this is:
java.util.List<T> list = new ArrayList<>();
java.lang.Iterable<? extends T> iterable = list;
org.assertj.core.api.Assertions.assertThat(list).hasSameElementsAs(iterable);


Raise Error when template contains interpolation string outside of root node.

To replicate:

create a simple component and bind something immediately inside the template tag

<template>
    {{title}}

</template>

I have been tripped up by this a couple of times (vue 2.8.0), where a simple binding outside the root div gives no output to the DOM, but also gives no error, like you would receive if you had two root nodes:
template syntax error Component template should contain exactly one root element:
...
If you are using v-if on multiple elements, use v-else-if to chain them instead.

Is there a valid reason to allow bindings like this in the root? If not it is a very small thing but it might help very new users from scratching their head like I did

Provide minimal reproduction code using JSFiddle/JSBin/Codepen to demonstrate your issue. You could start with this template that already includes the latest version of Vue.
Please make sure to read the Issue Reporting Guidelines before opening new issues. The issue list only accepts bug reports and feature requests.

Thanks for using Vue!
If I have read correct, @simonmurdock is proposing that Vue should allow multiple nodes( e.g. text node + element node) in component root, is this right?
To my best knowledge, I'm afraid this is probably out of Vue2.0's scope. Since the code base has a strong assumption that component root has a single node. Supporting text node outside root node would not only require a total rewrite but also introduce calamitous breaking changes.

Not that it should allow multiple nodes, just warn that the {{binding}} will have no output if placed in the root, outside of a div

@simonmurdock For the record, can you add a reproduction on jsfiddle? Thanks.

I don't think I'm able to replicate on jsfiddle as I think webpack is required to replicate.
But here is how to do it from nothing:
mkdir test
cd test
vue init webpack  //used default settings
sudo npm install

Then I change the generated Hello.vue to have some stuff immediately after the opening <template> tag like this:
<template>
  this does not output to the browser {{ msg }}

{{ msg }}

我觉得这个例子不错


Add ability to download default subs/captions only

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.07.09. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.07.09

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[x] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=XvgquqdyTfA
Single video: https://www.youtube.com/watch?v=37sR5Akt_Q4

For these videos, the auto-generated captions are for their respective language (spanish and korean). I would like an option to choose "default" for the subtitle language part so that it can download whatever the default speaking language is rather than just one language or every language.

Description of your issue, suggested solution and other information
I would like to download subtitles in the default language of the video rather than only one language or every language. If the video is spoken in Chinese, I want to download the chinese subtitles only. And if the video is English, I want to download the english version without having to change my code.
Maybe detecting the default language of the video. When the auto-generated subtitles are of a different language, it usually says so in the video player.

:+1:
Same goes for the title and description.


Include Link To Wiki

Include a link on the Debug console page to the Wiki, allowing for access to more detailed information about possible commands & their usage.
This should solve #312

This works on my linux box and in gitian builds. Relatively trivial otherwise so I'm merging it.


[BUG] Stale Secret Fix

follow up to #46537 to automatically clean stale secrets
With the landing of the initial fix for #44347 we need to consider how we'll clean-up the resulting stale secrets.
Affected customers will likely have large numbers of Secret resources in the cattle-impersonation-system namespace, with only a tiny-fraction of these actually containing anything useful.

Validated with v2.11-07fc426551bfcf412d94d51f1860ed23901024f0-head. The previously reported issue with stale secrets was not observed: https://github.com/rancher/rancher/issues/46894#issuecomment-2347835928


User refactor

Refactor Users endpoint:

Match code to the current library standard.
Improve tests.
Deprecate 'roles' and use 'role' instead, to match the current API.

This PR will be merged into branch 1.11.0, which contains all the updates for v1.11.0

Closing, will open a new one instead.


Cryobeaker buff, potbomb buff

undo the cryo nerf, ive overcome my trauma (they break on first throw and hold 60u again)
buff pot water bombs a bit (nowhere near as much as before but still respectable bam)

heres are its breaching capabilities now using a rapid fire launcher as seen in the dark ages

it is no longer assblast giga destroyer but will still murder people and destroy windows

Where's the CL?


Update circle script to stop comment from being posted if no open PR

This would be the data to add


Fix plugin icon

Summary
Fix plugin icon file name
Ticket Link
None

Codecov Report
Base: 65.85% // Head: 65.85% // No change to project coverage :thumbsup:

Coverage data is based on head (21f5ed1) compared to base (2ea6115).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #155   +/-   ##
=======================================
  Coverage   65.85%   65.85%
=======================================
  Files          16       16
  Lines        1637     1637
=======================================
  Hits         1078     1078
  Misses        520      520
  Partials       39       39

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Export visible markers simply exports all markers

I'm not sure if it's the intended action, but the export all visible markers button simply exports all markers.
It would be nice if I could search to show only a few radius markers, and then only export those ones, so I don't have to copy it to a notepad and remove all other markers if I want to share only a few of them (instead of sharing every single marker I have).

the export all visible markers button simply exports all markers

Is it really not working for you? It only respects the top level visibility button next to the delete marker button, not the sub level visibilities.

It would be nice if I could search to show only a few radius markers, and then only export those ones

This is a good suggestion. I will improve the exporting to respect search terms, filtering and the usual visibility.

Maybe I've been misinterpreting it - I initially thought that export all visible markers would be exporting the ones that I can see near me, or that it would export markers that were visible in the sidebar following searching. If it's just markers that you have enabled, then that makes sense.
Thanks for taking on board the searching export idea!

As of version 1.7 exporting markers now respects search terms and/or filtering.

Filter
Info
Illustration

All
Every marker will be exported

Region
Only markers in the current world region (64 x 64 tile area) will be exported. This is somewhat equivalent to what you are actually able to see in-game

Visible
All markers that are enabled will be exported

Hidden
All disabled markers will be exported

Searching can be combined with the filtering to export very specific markers.


Trying to run will result in a version issue with protobuf-java

$ mill worker.run
[24/47] worker.resolvedIvyDeps | Downloading [6/6] artifacts (~0/0 bytes)
1 targets failed
worker.resolvedIvyDeps
Resolution failed for 1 modules:
--------------------------------------------
  $com.google.protobuf:protobuf-java:
        not found: /Users/work/.ivy2/local/$com.google.protobuf/protobuf-java/ivys/ivy.xml
        not found: https://repo1.maven.org/maven2/$com/google/protobuf/protobuf-java//protobuf-java-.pom
        not found: https://oss.sonatype.org/content/repositories/snapshots/$com/google/protobuf/protobuf-java//protobuf-java-.pom
        not found: https://s01.oss.sonatype.org/content/repositories/snapshots/$com/google/protobuf/protobuf-java//protobuf-java-.pom

Thanks for the report, I'm analyzing this as it was not happening before.

Apparently it's a bug while resolving transitive packages. I just pushed a fix in the latest commit.
Let me know if it fixed for you.

Thank you very much, it works


Text will be in second line if using a small screen device

I may adjust the text size but the solution for the outputed image is really low. Furthermore, I don't think anyone will use the second line. So it's better to force all text in one line.

yes, but my CSS is not good.
I don't know is here any props can do this?


Price is pulled with moved decimal by two digits

@jansenmarc
@PyWaves

the price has been pulled as 1750.39 instead 17.5039
same issue on all pairs.
Are the last commits from March 2019 correct?
setting the manual price in the bot.cfg file doesn't solve the problem, since every time order is filled, bot reads the wrong sell price with moved decimal.
Is there a change to discuss it with anyone here?

Yep, price calculation depends on the version of transactions.
You can try to change all lines:
10 ** (PAIR.asset2.decimals + (PAIR.asset2.decimals - PAIR.asset1.decimals)
into
10 ** 8
since PyWaves is currently using Version 3 of exchange transactions.
If that is successful, please provide a pull request.

it works now by replacing:
10 ** (PAIR.asset2.decimals + (PAIR.asset2.decimals - PAIR.asset1.decimals)
with:
10 ** 6
After changing it to 10 ** 8 decimal got moved two digits too far (0.175039).

After some testing it turns out, that 10 ** 8 works for some pairs and 10 ** 6 works for other pairs.
I'm not familiar with Python so unfortunately I won't be able to fix the code.

Can you please check which verson of PyWaves you are using? If it is not the latest version, please upgrade accordingly. Since version 3 is used for orders, 8 should be working properly.

How to check this? For the WAVES/USDN pair I had to use 10**6.

Show me the output of:
pip list | grep PyWaves
please.

PyWaves                   0.8.41

I checked in the Waves developers group and this is in fact the latest version of PyWaves.

Again, did you check if the following formula works:
10(8 + priceAssetDecimals - amountAssetDecimals)

This formula works, but I tested only on the WAVES/USDN pair.


App requirements are complete #3

App requirements are complete, please review. /cc @codepathreview

:+1: Nice work! The point of this homework was to explore a simple example of a full MVC application with a RESTful API. For each point in the homework feedback checklist, I'll either confirm that you're on the right track with the implementation, or I'll provide some feedback:

You should declare all unnecessarily public properties as private for better encapsulation.
Note that you should resolve all the Auto Layout constraint warnings
Nice work handling the retweet label by dynamically modifying the constraint. This is a situation where its not possible to specify the behavior that you want solely using Interface Builder.
Nice work setting up the controls within the Tweet cells. It can sometimes be hard to have a clean design for embedded controls because it usually involves cumbersome plumbing. This is an instance where I think having the model handle the network action helps greatly simplify the implementation.
You should try implementing the post tweet insertion into the feed. In real MVC apps, the more polished experiences have to do a certain amount of local object management to make a fully seamless experience.

In general, you should challenge yourself to create visually polished applications. That's where you'll find all the rough edges of the iOS framework and earn the next level of mastery.