[SPARK-30731] Update deprecated Mkdocs option

Split from #27534.
What changes were proposed in this pull request?
This PR updates a deprecated Mkdocs option to use the new name.
Why are the changes needed?
This change will prevent the docs from failing to build when we update to a version of Mkdocs that no longer supports the deprecated option.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
I built the docs locally and reviewed them in my browser.

Test build #118651 has started for PR 27626 at commit 55b01d3.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/23404/
Test PASSed.

Test build #118651 has finished for PR 27626 at commit 55b01d3.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/118651/
Test PASSed.

Thanks, merged to master, branch-3.0 and branch-2.4.

@nchammas . You should use a new JIRA ID because the scope is completely different.
This patch lands branch-2.4 while SPARK-30731 is Improvement JIRA.

This PR was split from #27534, and the referenced JIRA covers removing deprecated options related to building the docs. So the scope of the JIRA is fine, IMO, and includes this PR.
I'm not sure why this patch was backported to branch-2.4. If you prefer, I can create a new JIRA just for this PR and update the PR title accordingly. Let me know what you want to do.

I think this is OK for a backport, as it is a potential 'fix', is quite minor, and in any event only affects doc gen. Any problem it would cause would be obvious at docs build time.

Yes, @srowen . The individual PR is okay for backporting.
The problem at that time was that @nchammas had three independent PRs had the same JIRA IDs (SPARK-30731) while he split one PR to three PRs.

Since this is merged first for master/3.0/2.4. I switch SPARK-30731 for this PR.
For the other two PRs, I created separate JIRAs and updated the PR titles like [SPARK-30880][DOCS] Delete Sphinx Makefile cruft.

Yes, I couldn't notice this links the same JIRA. Thanks for correcting it @dongjoon-hyun

Also, I tested and verified it works, with Python 2.7, in mkdocs 0.17 (2017-10-19) which is before Spark 2.3 (2018-02-28). This mkdocs dependency was added as of Spark 2.3 (SPARK-21485).
This pages was deprecated in Mkdocs 1.0.0 (2018-08-03). Spark 2.4.0 release was 2018-11-02.
If a release manager uses the same machine with the same environment used in Spark 2.3 for Spark 3.0 or 2.4 release, it will still work, unofficially though.
If a release manager uses the same machine with the same environment used in Spark 2.4 for Spark 3.0 or Spark 2.4.x, it will work officially.
So, I think technically there is virtually no overhead.


Move FLAnimatedImage support to subspec

Adding a mirror subspec for PINRemoteImage/Core instead of PINRemoteImage would be useful imo when GIF support isn't required (which results in one less dependency, FLAnimatedImage).
Or maybe the other way around ? Add FLAnimatedImage in a new subspec. Same result, different approach.
USE_FLANIMATED_IMAGE is the flag defined by PINRemoteImage to detect whether FLAnimatedImage is available or not.

Closing as this next merge uses Pinterest's PINCachedAnimatedImage and FLAnimatedImage will be gone.


Hierarchical Inference paper for January 28, 2020

@makagan Want to propose a good starter paper for our first meeting, 0900 PST on 1/28/2020? Sebastian @swagnercarena has agreed to be our first lead reader :-) Others, feel free to chime in as well.

Hi All, I think we discussed reading:
Hierarchical Implicit Models and Likelihood-Free Variational Inference
by Dustin Tran, Rajesh Ranganath, David M. Blei
https://arxiv.org/abs/1702.08896
Is this reasonable, or would you prefer to start with a paper on more traditional Hierarchical inference?

I think this is a good one - and it has the basic math of hierarchical Bayes in it, I notice. @swagnercarena tells me that he is up for it, so I'll assign this issue to him, and look forward to hearing his discussion of it on Tues Jan 28! Thanks @makagan , thanks Sebastian!


Support querying a data stream

Description
Fixed index expression resolution to include backing indices of a data stream. This will make querying a data stream consistent with querying an index alias.
Issues Resolved

Closes https://github.com/opensearch-project/sql/issues/55

Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.
Signed-off-by: Ketan Verma<EMAIL_ADDRESS>

Thanks for contributing this PR! Could you fix the unit test error in Github action?
Btw, data stream seems a x-pack feature. Are you using x-pack with our SQL plugin together?
2021-05-14T15:44:56.3441278Z com.amazon.opendistroforelasticsearch.sql.legacy.esdomain.mapping.FieldMappingsTest > flatFieldMappingsShouldIncludeFieldsOnAllLevels FAILED
2021-05-14T15:44:56.3444598Z     java.lang.IllegalStateException: Failed to read mapping in cluster state for indices=[field_mappings], types=[]
2021-05-14T15:44:56.3448366Z         at com.amazon.opendistroforelasticsearch.sql.legacy.esdomain.LocalClusterState.getFieldMappings(LocalClusterState.java:218)
2021-05-14T15:44:56.3457993Z         at com.amazon.opendistroforelasticsearch.sql.legacy.esdomain.LocalClusterState.getFieldMappings(LocalClusterState.java:172)
2021-05-14T15:44:56.3465882Z         at com.amazon.opendistroforelasticsearch.sql.legacy.esdomain.mapping.FieldMappingsTest.flatFieldMappingsShouldIncludeFieldsOnAllLevels(FieldMappingsTest.java:68)
2021-05-14T15:44:56.3471283Z
2021-05-14T15:44:56.3471639Z         Caused by:
2021-05-14T15:44:56.3472259Z         java.lang.NullPointerException
2021-05-14T15:44:56.3473122Z             at java.base/java.util.Arrays.sort(Arrays.java:1040)
2021-05-14T15:44:56.3475761Z             at com.amazon.opendistroforelasticsearch.sql.legacy.esdomain.LocalClusterState.sortToList(LocalClusterState.java:251)
2021-05-14T15:44:56.3481108Z             at com.amazon.opendistroforelasticsearch.sql.legacy.esdomain.LocalClusterState.findMappingsInCache(LocalClusterState.java:244)
2021-05-14T15:44:56.3486622Z             at com.amazon.opendistroforelasticsearch.sql.legacy.esdomain.LocalClusterState.getFieldMappings(LocalClusterState.java:206)
2021-05-14T15:44:56.3489248Z             ... 2 more

@dai-chen  Fixed the broken unit-tests.
We are adding data streams' support in OpenSearch which is being tracked here: https://github.com/opensearch-project/OpenSearch/issues/675

Sorry I just realized that my PR renaming all packages to org.opensearch impacts yours. Could you please address the conflicts or rebase your changes?

@dai-chen No worries! I have rebased with the 'develop' branch.


204 No Content on Cloudflare Pages

Hey folks, thanks for the awesome library. My github oauth login is working on localhost:3000 just fine.
But when I deploy to CF Pages, nothing seems to happen when I use window.location.href = '/api/auth/github

Cloudflare seems to be returning 204 no content, it's like I'm never hitting this route. I can see the github token has never been accessed. Any ideas? Seems like infrastructure configuration, but I can't figure it out.
I am not using nuxt generate, and my server API calls are working fine: https://minitinkertown.com/api/hello returns JSON

Did you create the API route using oauth.githubEventHandler?
I recommend you to checkout nuxt-todos-edge as it is deployed on CF Pages too

Yeah, I did. I tried both in server/routes and server/api
export default oauth.githubEventHandler({
  config: {
    emailRequired: true
  },
  async onSuccess(event, { user, tokens }) {
    await setUserSession(event, {
      user
    })
    return sendRedirect(event, '/dashboard')
  },
  // Optional, will return a json error and 401 status code by default
  onError(event, error) {
    console.error('GitHub OAuth error:', error)
    // return sendRedirect(event, '/')
  },
})

I am able to login on localhost. It's only when it's deployed that it doesn't work. I'm looking at the todo package but I cannot see anything obvious that stands out 🤔

I may have gotten tokens mixed up 🤔
Seems to be working! Thank you @Atinux (love nuxt)


Dump when training Text column model on GPUs

Describe the bug
The model dumps with error when training a model on GPU runtime
To Reproduce
Train a model with Free text column on GPU device
Expected behavior
Should not give any error
Versions:

Auto-Tensorflow: 1.0.1
Tensorflow: 2.5.0
Tensorflow-Extended: 0.29.0

Additional context
Add any other context about the problem here.

RCA: Universal STS encoder from TF_Hub has problems on GPU training, to check for alternates from TFHub for text column

Issue has been fixed as part of e60c973 , changes will be deployed in 1.1.0
Fix: Changed position of strategy.scope()


Extend the documentation with changes from #672

Made in #672

Very nice! I think we can include:

The new description property for all data types
The new way of creating relationships with camelCase names instead of author_id()
The new property for relationship:

{
    "relationship": {
        "key": "id",
        "label": "name",
        "page_slug": "admin/users"
    }
}

To display them as links.

Point out that select_multiple can also be configured as a normal select_dropdown (without relationship but options).
And finally, the new format property for date fields, which formats the input with Carbon::formatLocalized()

Thanks @Frondor. I have made changes according to your feedback.
Please let me know if you notice anything else.
However, I am not much into the BREAD relationship, so could you possibly make a documentation note about the new property for relationship?

@marktopper very nice! I just sent a review with a few suggestions regarding to some strings.
Now, about the relationships, I've explained it in that PR, you can use this text on this PR

You can optionally add a new page_slug property to the relationships object in the BREAD details in order to display proper links to relationship records. i.e.:
{
"relationship": {
"key": "id",
"label": "name",
"page_slug": "admin/users"
}
}
if no page_slug is provided, then the real "text" (relationship.label) is still being "resolved", but no anchor link is created.

Thanks @Frondor. Updated the pull request.


Extension causes high cpu load

Issue Type: Performance
Extension Name: html-css-class-completion
Extension Version: 1.18.0
OS Version: Darwin x64 17.5.0
VSCode version: 1.31.1

:warning: Make sure to attach this file from your home-directory: /Users/orionstar/Zignd.html-css-class-completion-unresponsive.cpuprofile.txt :warning:
Find more details here: https://github.com/Microsoft/vscode/wiki/Explain:-extension-causes-high-cpu-load

Closing because it's already reported here https://github.com/zignd/HTML-CSS-Class-Completion/issues/155


New version: Lenovo.LeAppStore version 9.0.1630.704

Created by 🥟 Dumplings in workflow run #6510.

Log
Updated: 9.0.1530.524 → 9.0.1630.704
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

https://github.com/microsoft/winget-pkgs/issues/142854


adding rancher/server host

http://docs.rancher.com/rancher/latest/en/quick-start-guide/#add-hosts fails
root@ranch1:~# docker ps -a
CONTAINER ID        IMAGE                  COMMAND                  CREATED             STATUS                     PORTS                              NAMES
2da250762013        rancher/agent:v1.0.1   "/run.sh http://192.2"   7 minutes ago       Exited (1) 7 minutes ago                                      modest_jennings
604204c207a3        rancher/server         "/usr/bin/s6-svscan /"   24 minutes ago      Up 24 minutes              3306/tcp, <IP_ADDRESS>:8080->8080/tcp   drunk_hypatia
root@ranch1:~# docker logs -f modest_jennings
Error response from daemon: 400 Bad Request: malformed Host header
root@ranch1:~# docker --version
Docker version 1.10.3, build 20f81dd

Host is Digital Ocean 16GB box running Ubuntu 16.04

Useful
Info

Versions
Rancher v1.1.0-dev1 Cattle: v0.160.3 UI: v1.0.7

Access
github admin

Route
hosts.new.index

Is a duplicate of #4500 Sorry.


RSSI

Would it be possible to get rssi for the wifis from the esp device?

No worries, thanks for letting me know


BUG: Meson build creates broken scipy.sparse.linalg._isolve on archlinux w/ system openblas

Describe your issue.
see https://trac.sagemath.org/ticket/34779
Reproducing Code Example
https://github.com/sagemath/sage/blob/develop/build/pkgs/scipy/spkg-install.in

Error message
{{{
 [sagemath_doc_html-none]   [plotting ]   File "/sage/local/var/lib/sage/venv-python3.10/lib/python3.10/site-packages/scipy/sparse/linalg/_isolve/iterative.py", line 9, in <module>
  [sagemath_doc_html-none]   [plotting ]     from . import _iterative
  [sagemath_doc_html-none]   [plotting ] ImportError: /sage/local/var/lib/sage/venv-python3.10/lib/python3.10/site-packages/scipy/sparse/linalg/_isolve/_iterative.cpython-310-x86_64-linux-gnu.so: undefined symbol: slamch_
}}}

SciPy/NumPy/Python version information
1.9.3

Thanks for the report @mkoeppe. That is a very annoying compatibility issue in the way Arch Linux builds OpenBLAS. From https://trac.sagemath.org/ticket/34081#comment:61: "I suppose this is caused by our openblas not including lapack symbols. Our distro package is fine since we haven't switched to meson (yet)".
From the OpenBLAS README: For a general introduction to the BLAS routines, please refer to the extensive documentation of their reference implementation hosted at netlib: https://www.netlib.org/blas. On that site you will likewise find documentation for the reference implementation of the higher-level library LAPACK - the Linear Algebra Package that comes included with OpenBLAS.
Arch Linux builds OpenBLAS with the non-default NO_LAPACK=1: https://github.com/archlinux/svntogit-community/blob/54dec2e1cd8b576e749038c8028b8b66b3c94f2f/trunk/PKGBUILD#L23. That seems like a very unhealthy choice. Also because OpenBLAS doesn't just include a copy of Netlib LAPACK, it replaces routines with more optimized ReLAPACK implementations.
As reported on the Sage issue, it can be worked around by using meson setup .... -Dlapack=lapack to builds against the reference LAPACK instead.
The reason this doesn't show up in the setup.py based builds is that the numpy.distutils BLAS/LAPACK detection does symbol checks (only for OpenBLAS, probably because of this issue) and skips a detected library if it's present but misses those symbols. See _require_symbols in numpy/distutils/system_info.py.
I guess I'll put this on the pile of BLAS/LAPACK detection work. There is a design issue here - if we're asking Meson for dependency('openblas') it caches the result of there. So if we make that call twice, once for blas and once for lapack, we by design get the same result. And what we want here is to get YES for blas and NO for lapack. So that'll require some change in API.
@antonio-rojas any chance this can be changed in Arch Linux? Even once we improve handling this in SciPy, it will continue to be a footgun when explicitly selecting libraries with dependency('openblas'), and it loses performance.

@antonio-rojas any chance this can be changed in Arch Linux? Even once we improve handling this in SciPy, it will continue to be a footgun when explicitly selecting libraries with dependency('openblas'), and it loses performance.

I'd very much like that, since this is not the first time our non-standard openblas packaging is causing issues with other projects, but this is ultimately the package maintainer's decision @felixonmars
We would still keep building our official scipy package against netlib's blas though, as we don't want to tie the package to any specific blas implementation.

We would still keep building our official scipy package against netlib's blas though, as we don't want to tie the package to any specific blas implementation.

That's not an issue - it looks like libblas.so is a symlink, so if one installs openblas one still gets scipy + openblas. This kind of "virtual BLAS package" is what many distros do, to ensure that all packages pick up the same library and that that library is runtime switchable.

This was fixed in Arch Linux, it now builds OpenBLAS in a more standard way a while back (thank you Arch Linux folks!), so closing.


[Event Request] Codeunit 90 OnBeforeCreatePositiveEntry and OnBeforePostRevertedWarehouseEntry

We Need two new Events in codeunit 90:
Codeunit 90:
local procedure RevertWarehouseEntry(var TempWhseJnlLine: Record "Warehouse Journal Line" temporary; JobNo: Code[20]; PostJobConsumptionBeforePurch: Boolean): Boolean
var
IsHandled: Boolean;
Result: Boolean;
begin
IsHandled := false;
OnBeforeRevertWarehouseEntry(TempWhseJnlLine, JobNo, PostJobConsumptionBeforePurch, Result, IsHandled);
if IsHandled then
exit(Result);
    if PostJobConsumptionBeforePurch or (JobNo = '') or PositiveWhseEntrycreated then
        exit(false);

    with TempWhseJnlLine do begin
        "Entry Type" := "Entry Type"::"Negative Adjmt.";
        Quantity := -Quantity;
        "Qty. (Base)" := -"Qty. (Base)";
        "From Bin Code" := "To Bin Code";
        "To Bin Code" := '';
    end;
  +++++++++++++++++++++++++++
  OnBeforePostRevertedWarehouseEntry(VAR TempWhseJnlLine: Record "Warehouse Journal Line" temporary);
  ------------------------------------------
    exit(true);
end;

local procedure CreatePositiveEntry(WhseJnlLine: Record "Warehouse Journal Line"; JobNo: Code[20]; PostJobConsumptionBeforePurch: Boolean)
begin
if PostJobConsumptionBeforePurch or (JobNo <> '') then begin
with WhseJnlLine do begin
Quantity := -Quantity;
"Qty. (Base)" := -"Qty. (Base)";
"Qty. (Absolute)" := -"Qty. (Absolute)";
"To Bin Code" := "From Bin Code";
"From Bin Code" := '';
end;
//++++++++++++++++++++++++++++++
OnBeforeCreatePositiveEntry(VAR WhseJnlLine: Record "Warehouse Journal Line");
//----------------------------------------
WhseJnlPostLine.Run(WhseJnlLine);
PositiveWhseEntrycreated := true;
end;
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


add IORING_OP_{MKDIRAT,SYMLINKAT,LINKAT}?

https://github.com/axboe/liburing/blame/f3dcc5c30df83890d3176f2694acca21cc7f6a2a/src/include/liburing/io_uring.h#L145

Are you willing to open a PR?

It has been added.


Feature Request: Apple Script Support

Automatic refresh bitbar with apple script is a easy way to customise the use of this smart app.
Thanks!

You can now refresh using our URL scheme: https://github.com/matryer/bitbar/blob/master/Docs/#refreshplugin


Rootchain CLI commands do not produce JSON output

[ Rootchain CLI commands do not produce JSON output ]
Description
Adding --json flag to CLI command doesn't have any effect for e.g. polygon-edge rootchain deploy command.
Your environment

OS and version - docker on macOS
The version of the Polygon Edge - 0.9.0
The branch that causes this issue.
Locally or Cloud hosted (which provider) - locally
Please confirm if the validators are running under containerized environment (K8s, Docker, etc.) - it's config issue

Steps to reproduce

just run polygon-edge rootchain deploy command with --json flag, e.g. polygon-edge rootchain deploy --deployer-key 0x123xyz... --genesis ./genesis.json --json-rpc http://rootchain:8545 --json

Expected behavior

Expected output would be of a JSON format so that it's easier to retrieve addresses of the contracts that are required in the next steps
Instead it produces this output:

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = RootERC20
Contract (address) = 0x2A184ce78767AA6855CdCb1e2F5E2D28Fe27539A
Transaction (hash) = 0xf539333d202849af9e6e0d3ce070423444abf8b35a0ac577718852c49eeb404b

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = RootERC721
Contract (address) = 0xd5F3D0ABa9782FfE628E2276e9b5D7b86267eed0
Transaction (hash) = 0xead90d1c0efd4d0411439e9fed4c86f8600014e7d3ae1f5e353c80bb329cf776

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = RootERC1155
Contract (address) = 0xd4f4a113c69e57AB6804D8109aFb9fa844097785
Transaction (hash) = 0x9ebca7aa8f0dd2a909e1fbb98cb9d47d0ab823d07b5346ca0ad51abdf9606fe3

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = StateSender
Contract (address) = 0x4204a98375943BF08C4c443AAFd76008AADddFD7
Transaction (hash) = 0x1d33be906f77cb2d61d732e344aa1ae7640411e9f97c091acf160e77eb39a3b0

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = CheckpointManager
Contract (address) = 0xF560334C0b9e226DbEEc37c223120aD78E66c3e3
Transaction (hash) = 0x1dd8414a777b0caa458179a420892be33df6b93fef1fded42750f8e19be3cbec

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = BLS
Contract (address) = 0x970630FB29e477c622dF2A43ecfA2ec37254B2ce
Transaction (hash) = 0x3aa247eb1053f77647f358f29a01174220b0195affafe003735fbf7300c5de8b

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = BN256G2
Contract (address) = 0xa05Ee2F8E9DF86ba4Df1A47f130212917915f1a3
Transaction (hash) = 0x7ba274705e8a4d6b1f464d75e9453a31e13951efaf88f7a034eedf0ad8bb96ea

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = ExitHelper
Contract (address) = 0xa53B90794D11DC52bc65f46BB636A0969D8642E8
Transaction (hash) = 0xb571ea77b36cd900ba9f4b709ad561fae1f68ffe2fa026d9e29dc808a05c4b82

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = RootERC20Predicate
Contract (address) = 0x7e9dbB9160C176650D0eb3b498047cDb3E4e4eFf
Transaction (hash) = 0x6367cc4ebd2f7ad620e3e4f1ea93e683b181e1514d79757dc68849e544449fbf

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = ERC20Template
Contract (address) = 0x3c2Fa4FCa2cbfCFbD5c491c522E6a23531e8E853
Transaction (hash) = 0xb0fa4756bc27d8721fb9adbb3d83de7e8aa4436b36a591568f443483b7f3ce7a

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = RootERC721Predicate
Contract (address) = 0xe141F10B617e273b4754Ee0A3f701CcA36210128
Transaction (hash) = 0x4d523788eca25c7f1328014f3f85180052ade3e18fcea46ff77fe0fefcdb954d

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = ERC721Template
Contract (address) = 0x82b0A7f4bB12970bF6193a81b6492E2E57b9E513
Transaction (hash) = 0xb5e09384e64acef012b9e58ac0161cd6a12e902f686344179e9fd42cc3eecf4d

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = RootERC1155Predicate
Contract (address) = 0x4E8d02b41a6187e81DdD0D9fDFDfb75C049856d2
Transaction (hash) = 0xe5cedb5d8a010ea6377730ae46738a538593e9df9b221d52c739c2ca37a19166

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = ERC1155Template
Contract (address) = 0x67c93deBC8f542487B9825c351C89a65E358F078
Transaction (hash) = 0x02477f70c503e199247cda839ef327d4e1d74cb9d569a19432f425ca4eaa4b78

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = StakeManager
Contract (address) = 0x31517DeE54Fba6EA22122828e4A937BF1106849B
Transaction (hash) = 0x254a16a84aeb906d5cfe9c4bf58b36440a86a6fbe60e09d4591d37307524c762

[ROOTCHAIN - DEPLOY CONTRACT]
Name               = CustomSupernetManager
Contract (address) = 0x2AB13b1C0B0e20bE4bBC533AbFf9c2aE3CAccEd3
Transaction (hash) = 0x3ba92637d822a4fd54a6182825abdb5590ec79756cb1b6e78aa4973bdf6b4844

[ROOTCHAIN - CONTRACTS DEPLOYMENT] StakeManager contract is initialized

[ROOTCHAIN - CONTRACTS DEPLOYMENT] CustomSupernetManager contract is initialized

[ROOTCHAIN - CONTRACTS DEPLOYMENT] [VALIDATORS]
Address=0x5AaD1C67e76dcA1D3825e75C1f3591101aA4d0F0; Balance=1000000; P2P Multi addr=/dns4/validator_0/tcp/1478/p2p/16Uiu2HAm6kmdbWK5hJwq3wSvLpVS219c37Jt7n8nQUCQEU67gFht; BLS Key=1afbc5bebbe6fcf75554467b9b113085bd0ab13f59bc18ea804c48b8d7f02f2e25381d272ee05bc49397e51a73c8062c3d32b9305e7fb25d25992d59f1f63add2822b4db45d22e6e450aa3019d0f3f0d04004daa99af3dd6ae9981a60261741b2cc35c7b482fa850e4920a5067d890eb331d9e4858a1ac02bbabdf63abb5bc3c;
Address=0x1A2DfdE323455d13e76b9201130fC6286b28bC15; Balance=1000000000000000000000000000000000000; P2P Multi addr=/dns4/validator_1/tcp/1478/p2p/16Uiu2HAm5vJjodbvywxJkgAF3YKgskLnXcUoq8PUYyRBYviJGwne; BLS Key=285a4b680e416ef78a21f366892bfbbea41ed9987d5bb31cc29f053eff6c748320d1ab86b5ab550bf5414441029553b1fceb82980de32a5afb5e28ca40efbd8b2cfed9efbbc982724dad69fa1d1d5c4a5cb0dc87f8a87ec82a6753bd12f9cad82141ecce20daef82bac0d00047c2e1876daa578cce6df753fabc969ecea62308;
Address=0x0A3Adc5B644B50a6E6f63A2FD4611E0e63C51841; Balance=1000000000000000000000000000000000000; P2P Multi addr=/dns4/validator_2/tcp/1478/p2p/16Uiu2HAm7ooWYNA97SRBqwp5azPrvHfMf4xBxLaHDtGhLnsZKyBP; BLS Key=0cc05f1d5c94fc910f84f03d19651d2dcc0fd9775d314f0b3b21592a73ad9c19088c29e39cceeb8bcfcbc46e5bb104b2492320c61de049ec6667568a949fce022a8b9d0a578831a07bab7092b7aa3333ab3a4fc4cfa65695c6f2cefb573d48ab0c3df3add78e9cbc7725c4b5cbdc17a5f52f25ca272884b93af5fd71f40ff4e3;
[ROOTCHAIN - CONTRACTS DEPLOYMENT] Validators hash: 0xdd91a262307539ae3f04c8451bfd87b8764c9ca07a8773439f5fa5a15497d4cb
[ROOTCHAIN - CONTRACTS DEPLOYMENT] CheckpointManager contract is initialized

[ROOTCHAIN - CONTRACTS DEPLOYMENT] ExitHelper contract is initialized

[ROOTCHAIN - CONTRACTS DEPLOYMENT] RootERC20Predicate contract is initialized

[ROOTCHAIN - CONTRACTS DEPLOYMENT] RootERC721Predicate contract is initialized

[ROOTCHAIN - CONTRACTS DEPLOYMENT] RootERC1155Predicate contract is initialized

{"message":"[ROOTCHAIN - CONTRACTS DEPLOYMENT] finished. All contracts are successfully deployed and initialized."}

Logs

n/a

Proposed solution
Rewrite result output logic for this set of commands so that they properly utilize --json flag.

This was implemented through PR 1724


  :)

Thanks. I should have updated the readme earlier. 😄


Notes from getting xhprof running

Drupal VM is great...thank you for the work you put into this project.
Here are few of my notes on setting up xhprof to work with Drupal VM:

the default ini file is /etc/php5/apache2/conf.d/xhprof.ini
output directory is set in there and is xhprof.output_dir="/tmp/xhprof"
virtual machine setup: it does not create the directory above; create it and set permissions to 777
there is no warning that the destination directory doesn't exist or isn't writable (but there are several patches in various states of readiness about this)
download and enable https://www.drupal.org/project/xhprof; turn it on in the configuration page for a bit more UI w/r to the runs.
see the runs at /admin/reports/xhprof; this mimics xhprof.domain.dev
No runs were generated until I disabled xdebug (/etc/php5/apache2/conf.d/xhprof.ini) and restarted apache
I disabled xdebug because it will skew the results...and happily xhprofs runs started showing up

I'm curious about the runs not appearing until I turned off xdebug. I had spent a couple hours trying to get them to show up when I stumbled across this. Is this known behavior?
If the above sounds correct, I'm happy to add to the documentation.

Not sure why xdebug breaks XHProf exactly, but I have heard from others that they can't have both enabled at the same time anyways. Do you think we need to update the Using XHProf with DrupalVM docs page a bit?

If I'm not the first to see this behavior then, yes, I would definitely update the docs. I get that profiling shouldn't be done with xdebug on (hence why I was turning it off) but in my case I was just trying to set up a machine for the first time attempting to get everything working. I think it could save others some frustration wondering why their profile runs aren't showing up.
Is it possible to create the /tmp/xhprof directory as well? That definitely seems missing from the playbook.
Also, those other bits, like where the various .ini's are located, would be handy for others, too, I think.

The issue with the tmp folder not existing is that distros clean out the /tmp folder now and then. So I'm using the following in all my settings.php files.
if (extension_loaded('xhprof') && !is_dir('/tmp/xhprof')) {
  @mkdir('/tmp/xhprof');
}

better yet
if (extension_loaded('xhprof')) {
  if ($xhprof_output_dir = ini_get("xhprof.output_dir") && !is_dir($xhprof_output_dir)) {
    @mkdir($xhprof_output_dir);
  }
}

Maybe we should move the default path out of the /tmp directory though?

@oxyc - I'm game for that... but would want a path that works with CentOS/Ubuntu at least, and wouldn't require special permissions (that was why /tmp was an easy choice when I set it up originally :) ).

I've seen a lot of setups using /var/tmp/ instead, and apparently that one is persistent across reboots.
You already have a task the sets the permission to 777 in the ansible role so that shouldn't be an issue across distros right?
Prefer fixing it upstream or adding the variable to the example config?

Yeah, that makes sense (fixing upstream).

The output directory issue is now fixed upstream in https://github.com/geerlingguy/ansible-role-php-xhprof/commit/b096612a79c9633cb61bd8a2a141f44cc1200d22.
@aangelinsf maybe you can update the docs regarding the xhprof + xdebug issue?

Yep, will move that forward today.

On Jan 26, 2016, at 9:55 AM, Oskar Schöldström<EMAIL_ADDRESS>wrote:
The output directory issue is now fixed upstream in geerlingguy/ansible-role-php-xhprof@b096612 https://github.com/geerlingguy/ansible-role-php-xhprof/commit/b096612a79c9633cb61bd8a2a141f44cc1200d22.
@aangelinsf https://github.com/aangelinsf maybe you can update the docs regarding the xhprof + xdebug issue?
—
Reply to this email directly or view it on GitHub https://github.com/geerlingguy/drupal-vm/issues/381#issuecomment-175142611.

@aangelinsf - I've made some docs updates; let me know if you think any others would be helpful.

Hi y'all -- I've followed these steps and am getting Could not open /var/tmp/xhprof/56ddcc8518d20.Sitename.xhprof errors in the Drupal status log. var/tmp/xhprof exists and has permissions set to 777. Is there some other magic step I'm missing?

I'm also getting the same error as @timstallmann through the Drupal interface. The only success I've had so far is running the drush xhprof-list command which does generate output in the /var/tmp/xhprof directory, but nothing that is helpful.

I'm able to get XHProf working on the latest Drupal VM 2.3.1 — the issues above may be due to an older version of Drupal VM being used.

Also make sure to refer to the latest docs for XHProf here: http://docs.drupalvm.com/en/latest/extras/xhprof/ – things should be working for latest D7 and D8 with Drupal VM 2.4.0+ now.


Add support for +hooks.server.js route files

Describe the problem
The current hooks functionality is really useful, particularly when used with sequence() to create an event pipeline. However, I'm increasingly finding that I want to specialise hooks for a given request paths, which requires pathname checking. Whilst this works, it is brittle - changing the route folder requires changing the pathname check. It would be preferable to colocate specialised hooks with the routes they pertain to.
Describe the proposed solution
I would like to be able to add a +hooks.server.js (or .ts) route file to a route folder that would export server hooks specifically for that route and its children. The request event handler would call the exported handle function after the current hooks.server handle function had been called, effectively appending it to the global hook sequence.
Where a child route folder has parent route folders with hooks, these would be appended prior to any hooks specific to the child.
Alternatives considered
Prior to the introduction of generated types I had used an interceptor pattern to wrap Handle and / or RequestHandler, but this has become too unwieldy to type correctly, and has resulted in unnecessary boilerplate-like code.
Importance
would make my life easier
Additional Information
No response

This might be a good way to handle the problem described in https://github.com/sveltejs/kit/issues/6315

It would be helpful to have some concrete use cases for this. We've discussed it a fair amount but so far determined that it wouldn't be a good addition, because it would add a lot more complexity.
For example, if you have a situation like this...
src/
├ routes/
│ ├ foo/
│ │ ├ bar/
│ │ │ ├ baz/
│ │ │ │ ├ +layout.js
│ │ │ │ │ +page.js
│ │ │ │ └ +page.svelte
│ │ │ └ +layout.js
│ │ └ +layout.js
│ └ +layout.js
└ hooks.server.js

...then there's some stuff you have to understand about what happens when, but it basically makes sense:

handle in src/hooks.server.js starts

load functions in the +layout.js and +page.js files run concurrently
once load functions finish, we render +page.svelte

handle finishes

If we now add a bunch of +hooks.server.js files...
src/
├ routes/
│ ├ foo/
│ │ ├ bar/
│ │ │ ├ baz/
+ │ │ │ ├ +hooks.server.js
│ │ │ │ ├ +layout.js
│ │ │ │ ├ +page.js
│ │ │ │ └ +page.svelte
+ │ │ ├ +hooks.server.js
│ │ │ └ +layout.js
+ │ ├ +hooks.server.js
│ │ └ +layout.js
+ ├ +hooks.server.js
│ └ +layout.js
└ hooks.server.js

...then the order gets somewhat more complex:

handle in src/hooks.server.js starts

handle in src/routes/+hooks.server.js starts

handle in src/routes/foo/+hooks.server.js starts

handle in src/routes/foo/bar/+hooks.server.js starts

handle in src/routes/foo/bar/baz/+hooks.server.js starts

load functions in the +layout.js and +page.js files run concurrently
once load functions finish, we render +page.svelte

handle in src/routes/foo/bar/baz/+hooks.server.js finishes

handle in src/routes/foo/bar/+hooks.server.js finishes

handle in src/routes/foo/+hooks.server.js finishes

handle in src/routes/+hooks.server.js finishes

handle in src/hooks.server.js finishes

I feel like the mixture of sequential and concurrent code would be a real source of confusion for people, particularly the fact that e.g. src/routes/foo/bar/+hooks.server.js runs before src/routes/+layout.js (and also after it, sort of). The waterfall is an opportunity for slowness to creep in. And the flip side of being able to scope handle code with more granularity is that it becomes harder to figure out where a particular thing is happening.
On top of that, while it makes sense to chain handle functions, it probably doesn't make sense to chain handleError. Meanwhile handleFetch could probably go either way. These sorts of ambiguities are best avoided unless they're absolutely necessary.

Thank you for taking time to consider this @Rich-Harris.
Authorisation is my primary use case for adding optional +hooks.server.js to routes. I want to check a user's authorised scopes before handling a request, and sometimes this needs to be nested. I was able to do this with a simple interceptor that wrapped the handler, but it wasn't easily portable to the newly typed interfaces. The checks can become boilerplate-like quite quickly, and so pulling them out to a hooks module would be useful.
I've just read the proposal to add guard functions to layouts #6912 and this is similar to what I am looking for, albeit extending the +layout.server.js module.

Reflecting on your response - and just thinking about handle - if the hooks in your example were necessary, they would reside with hooks.server.js right now, and would be called sequentially. It would be simpler from the framework's point of view, since they would be exported using sequence() or a single handle implementation. From a developer point of view, this would be more complex.
Also, +hooks.server.js proceeds +layout{.server}.js in lexicographical order, and so I don't think it would be too much of a stretch for developers to understand the order in which the modules are called (I don't think we should name future route files with this in mind, but it is useful in this instance.) While the hook will run before the layout and page load functions, the fact is that is can operate before, after, and around (i.e., before and after) the load. This can afford lifecycle functions to individual routes, and goes a long way to providing a middleware solution for SvelteKit.
The benefit of colocating hooks with the routes that they apply to reduces the overhead of guard functions to check if the hook applies to the request or not. I accept that it would introduce complexity for the framework as it would be necessary to create route specific sequences, but the benefit will be for the developer.
A lot could be achieved by in +layout{.server}.js but this is adding business logic to layout, even if the functions were imported from elsewhere. I do think that a route +hooks.server.js resolves this, and is obviously entirely optional for the developer.

I feel like this issue can almost be renamed to "Add support for route-level middleware".
To summarize the two big use cases:

Enabling "route guards" or "protected routes"

Other frameworks have similar features
Closes #6315

Empowering the SvelteKit ecosystem with pluggable route-level middleware

Right now only possible through brittle string matching in hooks.*.js

I'm not personally convinced this is actually that complicated to introduce to developers. The mental model could be as simple as:

"Before" hooks run first, synchronously
Load functions run next, async by default
"After" hooks run last, synchronously

Load functions are introduced earlier in the docs and tutorial, encouraging async by default, with hooks beign introduced under the "advanced" section in both the docs and tutorial. You could really hammer the point home with a warning like:

Use +hooks.client.js and +hooks.server.js sparingly for things like authentication where you intentionally want route-level, blocking waterfalls.

As for the question about handle, handleError, handleFetch, they could be simply answered with "they all work the same as with any other middleware". If a route wants to handle requests, errors, or intercept fetches, let it. If a developer wants to selectively opt-in to anything else, they can do so with their own middleware.
Finally, this all could be made a bit more simple in SvelteKit v2 by deprecating src/hooks.*.js in favor of a top-level src/routes/+hooks.*.js. 🙂

I think if people are going to use SK as their backend with a filesystem based router that router regardless of its description needs to support middleware. If I were using express or koa or whatever I would have middleware groups, which solve this problem entirely.
Whether it's called hooks or not, I don't see how a functional router can exist without supporting middleware. If I made an API router that you had to string match the URI to choose to run middleware it would be DOA, I don't see how SK can be a mature full stack solution without this.

If a hooks.server.js per directory is too complicated, maybe a hooks.server.js per routing group would be a good compromise.
src/routes/
│ (app)/
│ ├ hooks.server.js <----
│ ├ dashboard/
│ ├ item/
│ └ +layout.svelte
│ (marketing)/
│ ├ hooks.server.js <----
│ ├ about/
│ ├ testimonials/
│ └ +layout.svelte
├ admin/
└ +layout.svelte

Personally, I feel like doing it per routing group would make it more complicated.

Hooks can be defined in one top-level hooks file for all routes in /src/hooks.server.js or they may be defined per routing group in /src/routes/. If you want a set of hooks to affect only one directory, create a routing group with just that one directory.

vs. something like

All hooks are defined per route through +hooks.server.js files, similar to +layout.server.js files.

I don't think there's a major difference in complicatedness, and having both would be even better.

having both would be even better.

"Having both" is achieved with the second description I shared which is why I would advocate for it over a special exception for something that only works within layout groups.

Personally, I feel like doing it per directory would make it more complicated.

Hooks can be defined in one top-level hooks file for all routes in /src/hooks.server.js or they may be defined in /src/routes or any directory under /src/routes. If you want a set of hooks to affect only one directory, you won't be able to create hooks.server.js file in any of the parent directories or in the /src folder.

vs. something like

Hooks can be defined either in either /src/hooks.server.js or /src/routes/(routing-group)/hooks.server.js

That quote is not what I'm suggesting - remove /src/hooks.server.js in favor of /src/routes/**/+hooks.server.js files that following a middleware pattern, and the old /src/hooks.server.js is is replaced by a top-level /src/routes/+hooks.server.js.

That quote is not what I'm suggesting - remove /src/hooks.server.js in favor of /src/routes/**/+hooks.server.js files following a middleware pattern, and the old /src/hooks.server.js is is replaced by a top-level /src/routes/+hooks.server.js.

I feel like that just makes sense. Pattern matching the pathname in a large application is a pain, and it sometimes causes performance issues since the same middleware runs for every single request.

Wanted to share RIch's user-land implementation for having per-directory hooks
https://github.com/Rich-Harris/declarative-handlers#option-two--glob-imports


Default fields

Many elements will have lots of fields. It would be nice to have some way to leave them default without specifying to much.

Probably will be done using record update syntax


[Bug - Sparrow Server] Rejected mix UTXO due to unconfirmed status

Description
While waiting for confirmation, Sparrow Server seems to re-register an utxo in the 5.000.000 sats pool postmix although it has an unconfirmed status. Its input therefore gets rejected frequently (the mix progress bar resets the error message but  once it re-registers it gets rejected again).
<DATE> ERROR [Whirlpool-HttpClient-COORDINATOR_WEBSOCKET-85] c.s.w.c.m.MixClient[1672074727137] [null:-1] ERROR: Waiting for first mix confirmation
<DATE> ERROR [Whirlpool-HttpClient-COORDINATOR_WEBSOCKET-85] c.s.w.c.w.WhirlpoolWallet [null:-1]  - [MIX] 0.05btc ⣿ WHIRLPOOL FAILED ⣿ Input rejected ; Waiting for first mix confirmation

Some more background:

It's the only utxo of the wallet in said pool denomination.
The balance increased by an additional 5.000.000 sats as Sparrow shows the old utxo with mixesDone and the unconfirmed one with mixesDone+1
This behavior hasn't been observed with other pool denominations.
Sparrow is connected to a fulcrum server and receives block updates
Restart doesn't stop the behavior.

I'm pretty confident that this isn't a back-end issue...
UPDATE:
After hitting <Refresh> in the Postmix UTXO section the balance decreased by 5.000.000 back to the correct amount. Now there is only the "old" i.e. already confirmed utxo with mixesDone. Yet, the input still gets rejected, so there is still the unconfirmed transaction sitting in the mempool somewhere but apparently Sparrow Server doesn't know about it and keeps trying to re-register...
UPDATE 2:
The unconfirmed (and in the UI not visible utxo) has been confirmed and Sparrow has now been informed correctly over the fact, i.e. the mix count has been increased to mixesDone+1, the balance has the correct amount and the "old" utxo with the error messages has been replaced by the new one and isn't getting rejected anymore.
I see two problems on Sparrow's side:

Somehow Sparrow tries to re-register an unconfirmed remixer UTXO
Hitting <Refresh> drops the formerly shown unconfirmed utxo from the UI while problem no. 1 still exists.

Thanks for the detailed report.
Sounds like a temporary miscommunication between Sparrow and Fulcrum resulting in incorrect wallet history, which was then picked up by the Whirlpool client (which has it's own history cache that updates on a different schedule). Could be a bug in Fulcrum or Sparrow, more likely just a missed notification due to network issues. Could also be reorg, although they are rare so that is less likely.

Thanks for the info. I will keep an eye on it and report back if it happens again.

Going to close this off as it didn't happen again.


AppInstaller Update gets broken on CANCEL.

Please have a look at the Stackoverflow post. Some microsoft guy already did, but it wasn't solved.
https://stackoverflow.com/questions/55651889/auto-updates-with-app-installer-stop-working-if-user-cancels-the-automatic-updat/58188533#58188533

@driesvandamme
Canceling the in progress update probably broke some internal state in the update pipeline. I've filed an internal bug with our dev team to investigate this scenario.
Tanaka MSFT

@driesvandamme
A similar issue was fixed in Windows 1903 ( Build 18362). If you're using an earlier build, updating your OS to Build 18362 or greater may resolve the issue.
Tanaka MSFT

@Huios - Thanks: We did indeed log a ticket with Microsoft to get it resolved. Seems it was indeed a bug, but was already resolved with a higher Windows 10 version. (My customer however isn't on a newer version yet. They just did an upgrade to 1809)


Added Shorts

Created 2 new functions one for styling by element name, and the other to remove any yt shelf by category (for now only shorts category is filtered if checked)
Added a "Remove Shorts" in the Popup UI

Thank you for this contribution.
Looks good @rajeshias, merging it


Support "Interfaces Implementing Interfaces"

Continues work done in https://github.com/graph-gophers/graphql-go/pull/436. Implement https://spec.graphql.org/draft/#sec-Interfaces.Interfaces-Implementing-Interfaces
Key differences:

Avoids changing the core Schema type, sticks to using Schema.Types as a means of tracking interface validation.
Single loop validation of all interface types.
Improved test coverage.

Thank you @suessflorian


Fix cDU to correctly make async request

When a new id prop is passed in, getDerivedStateFromProps sets this.state.externalData to null. If I understand the new API correctly the return value of gDSFP() either becomes the new state or is merged into the prev state? If so, then componentDidUpdate should examine the new state (this.state.externalData), not prevState.externalData. Otherwise the async request will never happen.

Glad to help! Unrelated question: Any suggestions for familiarizing oneself with or attempting to wrap ones head around the React codebase? I'm really interested in understanding the internals and contributing somehow in the future but it's so intimidating in terms of sheer lines of code. Not even sure where to start. Other than taking a core package like react-reconciler and proceeding directly to a line by line deep dive, is there some other way to get a high-level understanding of the major components/structures and their relationships? For example stuff like, what exactly is a fiber, what is their structure and behavior and how do they interact. How does the upcoming async-mode work in terms of prioritizing/scheduling updates from a high level whiteboard point of view. How does reconciliation work. Etc etc. But with a focus on code and internals, not the API doc stuff on reactjs.org. Any tips you might have? Just asking. No offense if you don't have time to answer. Know you guys are all super busy between work and family.

There’s an outdated document here but it’s pretty old and some things changed. https://github.com/acdlite/react-fiber-architecture
There’s some background that’s still relevant here. https://github.com/facebook/react/issues/7942
This might be helpful. https://engineering.hexacta.com/didact-fiber-incremental-reconciliation-b2fe028dcaec
Other than that we don’t currently have a lot of info but I’d encourage you to look at five  files:

ReactFiberBeginWork (“entering” a component)
ReactFiberCompleteWork (“leaving” a component)
ReactFiberCommitWork (flushing changes to DOM)
ReactFiberScheduler (choosing next component to work on)
ReactChildFiber (diffing insertions/deletions of children)

I’m happy to answer questions (perhaps in a separate issue) and it would be great if you could write a doc with your learnings 🙂


Allow OLED script to run even if no drives are mounted

Currently rascsi_oled_monitor.py will error out if no drives are mounted:
Running with the following display:
<Adafruit_SSD1306.SSD1306.SSD1306_128_32 object at 0x766dcd50>

Will update the OLED display every 250ms (approximately)
Traceback (most recent call last):
  File "/usr/local/bin/rascsi_oled_monitor.py", line 151, in <module>
    output = str.strip(fields[1]) + " " + str.strip(fields[3]) + " " + os.path.basename(str.strip(fields[4]))
IndexError: list index out of range

This prohibits any display and also will stop the systemd service from auto-running on boot due to the script error. Demanding the user manually start the service once a drive has been mounted:
● monitor_rascsi.service - Monitor RaSCSI service
Loaded: loaded (/etc/systemd/system/monitor_rascsi.service; enabled; vendor preset: enabled)
Active: failed (Result: exit-code) since Mon 2020-09-14 18:42:41 EDT; 3s ago
Process: 943 ExecStart=/usr/local/bin/rascsi_oled_monitor.py (code=exited, status=1/FAILURE)
Main PID: 943 (code=exited, status=1/FAILURE)

As some users may prefer on the fly mounting and not want any drives mounting automatically, it would be best if the script was able to tell no drives were mounted and instead display just the date and time and perhaps a "No Drives Mounted" message on the OLED display.
I have temporarily resolved this by editing line 148 of rascsi_oled_monitor.py to:
if (len(line) == 0) or line.startswith("+---") or line.startswith("| ID | UN") or line.startswith("No images"):

@thewesker fixed this issue with pull request: https://github.com/akuker/RASCSI/pull/45


bench_results_change Table Getting Wrong Change Values

The bench_results_change table when running the Benchmark snippet is getting wrong values for some entries.  The below query shows it.
from urllib.request import urlopen; import os

root = 'file:///nfs' if os.path.exists('/nfs/deephaven-benchmark') else 'https://storage.googleapis.com'
with urlopen(root + '/deephaven-benchmark/benchmark_tables.dh.py') as r:
    benchmark_storage_uri_arg = root + '/deephaven-benchmark'
    benchmark_category_arg = 'nightly'  # release | nightly
    benchmark_max_runs_arg = 5  # Latest X runs to include
    exec(r.read().decode(), globals(), locals())

bench_bug_change = bench_results_change.where(
    ['benchmark_name=`ParquetWrite- No Codec Multi Col -Static`'
]).sort('timestamp')

Investigated this one, and it doesn't look like a bug, just a misunderstanding of what the Change column is.  It is a rate comparison, not to the previous rate, but to the mean of all previous rates in the set.


Incorrect sql with group by clause

What happened?
prql version: 0.8.1
database: SQLite 3.42.0
comments: generated sql is incorrect in the following examples.
Error message from the database: HAVING clause on a non-aggregate query.

No response

I think this is fixed in 0.9.0 — please can you confirm @YuxiLing ?
from tb1
sort {c1}
aggregate {new_col = min c2}
filter (new_col >= 0)

WITH table_0 AS (
  SELECT
    MIN(c2) AS new_col,
    c1
  FROM
    tb1
  HAVING
    MIN(c2) >= 0
)
SELECT
  new_col
FROM
  table_0

Yes, it is fixed in 0.9.0.


gocolly with for loop

package main

import (
	"fmt"

	"github.com/gocolly/colly"
)

func main() {

	CATETORIES := []string{
		"cate1",
		"cate2",
		"cate3",
	}

	c := colly.NewCollector()

	for _, cate := range CATETORIES {

		c.OnRequest(func(r *colly.Request) {
			fmt.Println("Visiting categories", r.URL)
		})

		c.Visit(cate)
	}
}

will print
Visiting categories http://cate1
Visiting categories http://cate2
Visiting categories http://cate2
Visiting categories http://cate3
Visiting categories http://cate3
Visiting categories http://cate3

@dfang hi, I got the same issues, how did you solve this problem?
Thanks


Crash when launched with Jitterbug

Describe the issue
I just installed the latest version of UTM on my non-jailbroken iPad. When I launch the UTM manually, I get the warning about remote debugging requirement. So I launch it via Jitterbug on my iPhone - UTM launches with no pop-up this time. But the second I try to run any VM, it crashes.
Configuration (required)

UTM Version: 2.1.1
OS Version: 14.6
Device Model: M1 iPad
Is it jailbroken (name jailbreak used)? No
How did you install UTM? Same as always - signed with my dev provisioning profile with iOS App Signer, then installed on the device with Apple Configurator.

Crash log
UTM-2021-06-04-222545.zip
Debug log
-[CSConnection initWithHost:port:]:269
-[CSInput initWithSession:]:293

That’s it, that’s the whole log.
Upload VM
Happens with all VMs (old and new).

The same issue for iPad Pro 2020 iOS 14.5. UTM has been installed through AltServer and launched from the second device via Jitterbug without no pop-up warning. Trying to open settings like System, QEMU or starting virtual machine lead to crash. For the instance - DolphiniOS is working fine through Jitterbug on the same device.

I believe it’s fixed in https://github.com/utmapp/UTM/commit/77d6ca13b42b768b7fc3c30a00ef0a6cc49971ed but that hasn’t been released yet.


[nowide] Upgrade to 11.1.1

Describe the pull request

What does your PR fix? Fixes #16912

Which triplets are supported/not supported? Have you updated the CI baseline?
Same as the current version

Does your PR follow the maintainer guide?
Probably yes

@PhoebeHui
 tatsu@TATSU-DPC-2ND   base  ~\..\..\vcpkg   master ≣  ./vcpkg x-add-version nowide
 tatsu@TATSU-DPC-2ND   base  ~\..\..\vcpkg   master ≣  git status
On branch master
Your branch is up to date with 'origin/master'.

nothing to commit, working tree clean

@PhoebeHui Done.

nowide 11.1.1 is not available any more. That causes pipeline failures, I have created #17209


TypeError: ParamPaCMAP.fit() got an unexpected keyword argument 'y'

Hi,
I encountered a TypeError when using fit_transform():
TypeError: ParamPaCMAP.fit() got an unexpected keyword argument 'y'

this is addressed in my branch/PR: #2

Thank you for your interest in our work! I think the currently merged PR #2 should have fixed most of the issues. Feel free to reopen it if you still encounter this issue!


load_job fix

Fix to load_job for fin_steps arg.

On https://github.com/ROCmSoftwarePlatform/MITuna/blob/c451c85342b2a2743d9c97bb3796f5ce38f8f02a/tuna/miopen/subcmd/load_job.py#L51, should the check be if args.fin_steps == 'not_fin' ?

On
https://github.com/ROCmSoftwarePlatform/MITuna/blob/c451c85342b2a2743d9c97bb3796f5ce38f8f02a/tuna/miopen/subcmd/load_job.py#L51
, should the check be if args.fin_steps == 'not_fin' ?

not_fin is a valid value for the fin_steps argument. This check is correct, it requires the fin_step input not to be an empty string. Though notable not_fin is now the default in the input parsing.


Fix #3792: Work around a 2.13.1 regression in overload resolution.

This commit works around https://github.com/scala/bug/issues/11755. Instead of relying on an arguably sensitive overload, we use an extra argument for explicit disambiguation.
Locally tested with
> ++2.13.1
> testSuite/test
> compiler/test

Irrespective of whether https://github.com/scala/bug/issues/11755 gets fixed upstream and when, we'll need this change before we can enable 2.13.1 on our CI.

This seems to be a pretty severe regression. Have we considered not supporting 2.13.1?

It's too late for that. 0.6.29 was already successfully published for 2.13.1 (it didn't have IdentityHashMap yet).

It's too late for that. 0.6.29 was already successfully published for 2.13.1 (it didn't have IdentityHashMap yet).

Ah... OK.


[New game]: Idle Miner

🎮 Game Request
Basic Description
In an Idle Miner game, players manage a mining operation to extract resources, upgrade equipment, and expand their mining empire. The game progresses even when the player is not actively playing, with automated systems and workers continuing to mine resources. The player invests the resources collected to upgrade various aspects of their mining operation, making it more efficient and profitable over time.
Game logic
Players start with a basic mining setup that automatically extracts a small amount of resources over time.
Players can click to manually mine resources for an extra boost.
Use collected resources to upgrade mining equipment, which increases the rate of resource extraction.
Point down the features
Game points
5 mining areas : Vadapav centre, Kulfi centre, Shawrma centre, Fish centre and Biryani centre
In-game currency : Rupees
upgrades : 5 managers , 5 profits multipliers
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

Hey @manasdeshpande !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

/assign

Hey @manasdeshpande ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Update date

Added several date updates, so "daylastupdated" works properly even when not downloading anything.
Though I think it would be best to remove the updating from the updateLastDownloadedImage (unless I am mistaken the last downloaded image value isn't used anywhere?) function

On a side note, no clue why it includes the old commits and a "conflict" which isn't one, sorry for that

I think you better get the latest commit, merge the changes for this and #758, then only create new pull request?


chore: examples, add subscription examples

Checklist

[ ] run make test
[ ] PR title must follow conventional-commit-standard
[ ] tests and/or benchmarks are included
[ ] documentation is changed or added
[ ] commit message and code follows the Code of conduct

Whats missing here?

Whats missing here?

non failing tests

@YuriBuerov, how can we move forward?


Create /DELETE route for WordSuggestions

There should be a DELETE route at api/v1/wordSuggestions/:id that will delete the wordSuggestion object in the database.
Hide this route behind process.env.NODE_ENV !== production since we don't want the world to be able to use this route.

Closed by #190


Fix detect issue for no code read permission

Please make sure the code is following contribution guidelines in CONTRIBUTING.md

[ ] : This PR has a corresponding issue open in the Repository. #687
[ ] : Approach is signed off on the issue.

@gauravsaralMs Please take a look.


Improve search result table's design

The current search results table is cluttered.
Especially service name badges...
And the code was more complicated than it needed to be.
So I've updated these components from scratch.
Before

After

I have a question.
The current result table has filtering feature.
Is this necessary?

I haven't implemented this yet...

The current result table has filtering feature.
Is this necessary?

Does this only allow filtering by service name? Is there a way to search for traces containing two or more different services? Maybe it is used for that, but if the search can do that maybe it isn't needed.
Unrelated and not critical to address in this pull request as I assume it is existing behavior, but what is the sorting of the service names? It made me wonder if it isn't more helpful to sort them by the highest number of spans if there isn't other meaning to the current order.
Someone who uses Zipkin on a regular basis should probably chime in on whether the service names being visible or not (without mousing over) affects their usual workflow (e.g. ctrl+F search in page for service... though I guess this is what the filter mentioned above is for).

Thank you for your comment!
I added service filter feature and sort service names by span count!

I raised this issue to correct the count so they aren't crazy inflated and also we can know what they mean cc @shakuzen https://github.com/openzipkin/zipkin/issues/3192
The UI presentation won't change, though this will help the numbers make sense.
The only impact this could have wrt presentations is if we want the charts to also show how many downstream/exit spans per service. I think we can wait for that, as it would layer on this if we did it as opposed to invalidate it.

I truly love this <3
tir. 1. sep. 2020, 07:20 skrev tacigar<EMAIL_ADDRESS>
@tacigar commented on this pull request.
In zipkin-lens/src/components/DiscoverPage/DiscoverPageContent.tsx
https://github.com/openzipkin/zipkin/pull/3180#discussion_r480826381:

const handleSettingsButtonClick = useCallback(() => {
setIsOpeningSettings((prev) => !prev);
}, []);

let content: JSX.Element | undefined;

const [filters, setFilters] = useState<string[]>([]);
const filterOptions = useMemo(
() =>

 Object.keys(

   traceSummaries.reduce((acc, cur) => {

     cur.serviceSummaries.forEach((serviceSummary) => {

       acc[serviceSummary.serviceName] = true;

Oops! I was correcting a different code.
I'll use Set here.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/openzipkin/zipkin/pull/3180#discussion_r480826381,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXOYAUUE5JOK3PNCPENQKLSDSAA7ANCNFSM4QFWJBQA
.

@jcchavezs Thank you so much!

Someone who uses Zipkin on a regular basis should probably chime in on whether the service names being visible or not (without mousing over) affects their usual workflow (e.g. ctrl+F search in page for service... though I guess this is what the filter mentioned above is for).

I'm still curious about this, but don't let my theoretical concern stop progress here if others are happy with the change. Compared to before, it is an extra click and mouseover away from seeing what the count is for a given service in a trace and if you are looking for the count of a particular service (theoretical use case: looking for traces where a service was called many times? or no times) you only have the first letter to figure out where to mouseover.

I see... 🤔
I have two ideas..

Add "Collapse All" button
Add small tables under bar chart for showing complete service name and span count.

User scenario

Push "RUN QUERY" button
Filter services using Service Filter
If the user want to narrow down search more, push "COLLAPSE ALL" button.
All service name & span count chart and table will appear.
Then, the user can ctrl-F search (because the table displays complete service names).

What do you think? @shakuzen

PS the original design idea for the letters was that services that shared a same first letter would be grouped. The bar chart would not have X axis ordered by span count, rather by letters like books on a shelf.
When you see a tall letter (high span count) you click into it and see which of the N services that might be there.
This is imperfect idea, as service names are neither always starting with a letter, nor are they distributed across the letters. However, it was an idea to allow you to browse a trace that has an extreme amount of span names in a way besides wrapping 20 times.
The design drifted a little from that which might not be bad, but it lost the impact of the grouping. You can see in the images the same letter repeated as it is now ordered by span count. If we go back to this then we can consider trying labels instead of the first letter or something else.
cc also @bulicekj
PS anyone with a massive amount of (but valid) service name trace would be appreciated, cc @narayaruna @drolando @llinder also would be their feedback.

FWIW I think in practice this is showing that really large traces may have slightly different visual needs than smaller ones. If you have less than 10 service names, I think there's no point grouping at all, or even abbreviating. That said service names that are really long will always have a problem as their labels mess up things.

I think we need to discuss bar charts a bit more, so I decided to make it a separate PR.

I think screen shots are up to date correct?

yes!

EPIC!!!

Hi, I am looking closer now and feel expand/collapse in the index screen is clutter. I don't know why someone would click this also with the ability to filter by service..
Is this mainly to guess some people prefer to always see duration and other people not?

I personally always like less controls


Standardize buttons across multiple platforms/controller types

We need some generic way of identifying buttons in a platform and controller independent way
Also standardize dead zones and intensities of the triggers
Related: https://github.com/flame-engine/gamepads/issues/4

This might be a useful resource.

I imagine this would look like defining some common set of controller elements inside gamepads_platform_interface and then each plugin implementation mapping inputs to those before invoking onGamepadEvent.


Don't automatically add reflect & stdlib to classpath via Gradle

This matches the behaviour of the Kotlin Gradle plugin:
https://github.com/JetBrains/kotlin/blob/v2.0.0/libraries/tools/kotlin-gradle-plugin/src/common/kotlin/org/jetbrains/kotlin/gradle/dsl/KotlinJvmCompilerOptionsHelper.kt#L21-L22
Stacks on #7408
Required for #4465

Warnings

:warning:
It looks like this PR contains functional changes without a corresponding test.

  Generated by :no_entry_sign: dangerJS against 6d6e82bef83a041395b7517754bdc63ede0a8773


cli/tests/e2e-tests.sh is not posix sh compliant and breaks

Running the cli/tests/e2e-tests.sh produces an error in bash:
Creating cluster "tobs" ...
 ✓ Ensuring node image (kindest/node:v1.21.1) 🖼
 ✓ Preparing nodes 📦
 ✓ Writing configuration 📜
 ✓ Starting control-plane 🕹
 ✓ Installing CNI 🔌
 ✓ Installing StorageClass 💾
Set kubectl context to "kind-tobs"
You can now use your cluster with:

kubectl cluster-info --context kind-tobs

Thanks for using kind! 😊
trap: SIGHUP: bad trap

I believe this is due to the following:

In POSIX sh, prefixing signal names with 'SIG' is undefined.

https://github.com/koalaman/shellcheck/wiki/SC3048

Thanks for reporting! We'll take a look.

That file was removed and we are no longer running those tests using a shell script, as such I consider this to be fixed. If you feel otherwise, please reopen.


Create Sample to for using steel toe with K8s

I do not find any samples for using STeelToe with K8s. Is there some code available for the same

These two should run in k8s:

Configuration
Discovery

Kubernetes samples will only be provided for master/3.0, but the default branch was 2.x  so that probably made them harder to find. We expect to be adding more and improving what's there over time, though there aren't specific plans beyond the reference app


Fix KeyDecoder bug and add laws checking

KeyDecoder's handleErrorWith wasn't even trying the original decoder. This is a quick fix and some laws checking.

Codecov Report

Merging #676 into master will increase coverage by 1.21%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master     #676      +/-   ##
==========================================
+ Coverage   85.29%   86.51%   +1.21%
==========================================
  Files          70       70
  Lines        2136     2136
  Branches      137      149      +12
==========================================
+ Hits         1822     1848      +26
+ Misses        314      288      -26

Impacted Files
Coverage Δ

...re/shared/src/main/scala/io/circe/KeyDecoder.scala
100% <100%> (+56.75%)
:arrow_up:

...re/shared/src/main/scala/io/circe/KeyEncoder.scala
100% <0%> (+26.31%)
:arrow_up:

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 38ed69a...6f361a1. Read the comment docs.


fix(gui-client): Ensure Tauri client honors LOG_DIR on Linux

Per https://github.com/firezone/firezone/pull/3810/files#diff-1a04997a5ad650a6038e9bb30f6e136f6c18a7940112933612db15d07e61e046

Yeah there's been some discussion around this. Maybe we don't need to expose env vars for the gui clients?
The others expose their log filter etc from the settings view.
Will need to think more of a long term approach, whether the gui client can also run headless or we require you to use a separate binary to run headless (currently it's the latter)


Doc : Correct injection-scopes/request-scope's example snippet

Informations

Type
Breaking change

Doc
No

Description
Correct injection-scopes/request-scope's example snippet

For this change create "scope-request.ts" file
Usage example
Example to use your feature and to improve the documentation after merging your PR:
import {} from "@tsed/common";

Todos

[ ] Tests
[ ] Coverage
[ ] Example
[ ] Documentation

Thanks  for your PR :D

:tada: This PR is included in version 5.44.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Split long lines

Can one of the admins verify this patch?

Can one of the admins verify this patch?


Add CYBERTEC to the list of CNPG support companies

Thanks for considering.

@df7cb, thanks for your submission! We recently published a clearer and more transparent set of guidelines for listing as a professional support organization or individual, as we noticed this was previously lacking. You can find the updated guidelines in our SUPPORT.md file.
We also noticed that your company website doesn’t mention CloudNativePG with a backlink to our site:

Could you please ensure this is addressed?
Additionally, for future submissions, please remember to comply with DCO requirements and sign your commit.
Thanks again!

Originally this was very urgent (that's why I stepped in to submit the PR), but now the internal machinery here to produce that web page is taking its time :roll_eyes:.

Hi all,
We are currently working on this and looking at what we can do "holistically" to ensure we overachieve on all the requirements.
Thanks for asking!!

Hi Christoph and Jan, thanks for submitting. I checked the page you linked and, while it mentions CloudNativePG, it doesn't include a link back to our website. See the requirement below:

Website Link: Include a link to your homepage or a dedicated landing page that explicitly mentions CloudNativePG support and includes at least one link back to cloudnative-pg.io.

Can you please add the link and update this ticket? Also, please file a PR.

Hi @gbartolini, we updated the web page with the back link.
I'm not sure what you meant by saying we should file a PR? This is already a pull request.
Thanks!

Hi @df7cb,

Hi @gbartolini, we updated the web page with the back link.

Awesome, thanks!

I'm not sure what you meant by saying we should file a PR? This is already a pull request.

My bad (I was looking at the commit).
Ciao,
Gabriele


Uncaught Exception: OverflowError: math range error

    xi, x_std, self.s = self.skater(y=round(yi, 6), s=self.s, k=1)
  File "/usr/local/lib/python3.9/site-packages/timemachines/skaters/smdk/smdkarma.py", line 21, in smdk_p5_d0_q3_n500_aggressive
    return smdk_arma_factory(y=y, n_agents=500, max_p=5, max_q=3, s=s, k=k, a=a, t=t, e=e, r=0.95)
  File "/usr/local/lib/python3.9/site-packages/timemachines/skaters/smdk/smdkarmafactory.py", line 119, in smdk_arma_factory
    x, x_std, s['s_pks'] = precision_weighted_skater(y=y_for_pws, s=s['s_pws'], k=k, a=a, t=t, e=e, r=r)
  File "/usr/local/lib/python3.9/site-packages/timemachines/skatertools/ensembling/precisionweightedskater.py", line 30, in precision_weighted_skater
    w  = normalize( [ 1./math.pow(y_std,expon) for y_std in y_stds ] )
  File "/usr/local/lib/python3.9/site-packages/timemachines/skatertools/ensembling/precisionweightedskater.py", line 30, in <listcomp>
    w  = normalize( [ 1./math.pow(y_std,expon) for y_std in y_stds ] )
=========================================================================

 Uncaught Exception: OverflowError: math range error

Any idea why this happens and how to avoid that?
My code:
 self.skater = local_skater_from_name('smdk_p5_d0_q3_n500_aggressive')
 self.s = {}
 self.x = []

This is inside a loop:
if self.index == 0:
    returns = utils.prices_to_returns(self.candles[:, 2])[1:]
    # warmup
    for yi in returns:
        xi, x_std, self.s = self.skater(y=round(yi, 6), s=self.s, k=1, e=-1)
        self.x.append(xi)
else:
    yi = utils.prices_to_returns(self.candles[-5:, 2])[-1]
    # predict
    xi, x_std, self.s = self.skater(y=round(yi, 6), s=self.s, k=1)
    self.x.append(xi)
self.x = self.x[-240:]

Thank you.

Could it be the s argument getting to big? Is there a way to limit/slice the memory of it?

The s argument was the reason. If I reset it after a certain amount of iterations, this error doesn't happen. Would be great to not have to clear all the history, but being able to keep a certain amount. If I understood it right, the s dict is different for each skater, so not an easy task I think. Still an incredible repository you made. I think adding a warning to the docs, that s might have to be reset in certain cases might be enough.

Hmm.. sorry just seeing this issue now. So you think the s is growing too large?

And yes you are right the responsibility/interpretation of s is known to the skater, not the caller.
The memory should plateau though.
It might be something else. Can you try the same thing but without _aggressive? I just want to test a theory.

I should put bounds on the variances anyway. There's no good reason to risk the error you are seeing.

btw I have better reflexes in the slack

============================== EXCEPTION TRACEBACK:
  File "/usr/local/bin/jesse", line 33, in <module>
    sys.exit(load_entry_point('jesse', 'console_scripts', 'jesse')())
  File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1128, in __call__
    return self.main(*args, **kwargs)
  File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1053, in main
    rv = self.invoke(ctx)
  File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1659, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/usr/local/lib/python3.9/site-packages/click/core.py", line 1395, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/local/lib/python3.9/site-packages/click/core.py", line 754, in invoke
    return __callback(*args, **kwargs)
  File "/home/src/jesse/jesse/__init__.py", line 301, in backtest
    backtest_mode.run(start_date, finish_date, chart=chart, tradingview=tradingview, csv=csv,
  File "/home/src/jesse/jesse/modes/backtest_mode/__init__.py", line 65, in run
    simulator(candles)
  File "/home/src/jesse/jesse/modes/backtest_mode/__init__.py", line 300, in simulator
    r.strategy._execute()
  File "/home/src/jesse/jesse/strategies/Strategy.py", line 813, in _execute
    self.before()
  File "/home/live/strategies/TMTest/__init__.py", line 71, in before
    xi, x_std, self.s = self.skater(y=yi, s=self.s, k=3)
  File "/usr/local/lib/python3.9/site-packages/timemachines/skaters/smdk/smdkarma.py", line 17, in smdk_p5_d0_q3_n500
    return smdk_arma_factory(y=y, n_agents=500, max_p=5, max_q=3, s=s, k=k, a=a, t=t, e=e, r=0.75)
  File "/usr/local/lib/python3.9/site-packages/timemachines/skaters/smdk/smdkarmafactory.py", line 119, in smdk_arma_factory
    x, x_std, s['s_pks'] = precision_weighted_skater(y=y_for_pws, s=s['s_pws'], k=k, a=a, t=t, e=e, r=r)
  File "/usr/local/lib/python3.9/site-packages/timemachines/skatertools/ensembling/precisionweightedskater.py", line 30, in precision_weighted_skater
    w  = normalize( [ 1./math.pow(y_std,expon) for y_std in y_stds ] )
  File "/usr/local/lib/python3.9/site-packages/timemachines/skatertools/ensembling/precisionweightedskater.py", line 30, in <listcomp>
    w  = normalize( [ 1./math.pow(y_std,expon) for y_std in y_stds ] )
=========================================================================

 Uncaught Exception: OverflowError: math range error

Same for smdk_p5_d0_q3_n500

Strange, as tests are passing. I made some minor changes can you try:
  pip install --upgrade git+https://github.com/microprediction/timemachines.git

?

Okay to close this?


Added GMS pre-commit hooks + lwc-jest

cspell
prettier
eslint
string label audit
lockfile-lint

Updated sfdx-lwc-jest
Critical Changes
Changes
Issues Closed
Unaggregated Pull Requests

Updating formatting to meet standards part 1 [PR27]

@davidmreed , I want to add a PR that installs our team's pre-commit hooks, make minimal changes to code so they pass the linting and formatting checks, and upgrade to API Version 50.0.
This will be a massive Pull Request, and I am trying to make reviewing this Pull Request as digestible as possible.   This PR only includes adding our team's pre-commit hooks so far.  Does it look valid?
Child PR #27 will include the code changes + other changes needed to run on API Version 50.0.

@davidmreed , made requested changes + added Github actions from GMS.
Note: Packaged code changes are in the child branch #27

@davidmreed , does our Python / pytest check fail because we don't have any python in the repo?   Should we remove this Github action?

W-8840996

@davidmreed , does our Python / pytest check fail because we don't have any python in the repo? Should we remove this Github action?

Yeah. If you're not planning to use any Python tasks, feel free to remove it.

@davidmreed , does our Python / pytest check fail because we don't have any python in the repo? Should we remove this Github action?

Yeah. If you're not planning to use any Python tasks, feel free to remove it.

@davidmreed , I prefer keeping the Github Action to protect any future python added to the repo.  I added tasks/tests/test_sample.py as a sample test since the Github Action requires at least one python test.   If you don't like this, can you re-review with changes requested?  Thank you!


Return a Response object for all the middlewares.

I really like the simplicity of this framework.
Recently I play around with some third party middlewares like the CORS middleware from oscarotero/psr7-middlewares. It does not work with the harmony framework.
I found it is because harmony do not care whether middlewares return the Response object. But nearly all the other middlewares follow this rule:

It can do whatever is appropriate with these objects. The only hard requirement is that a middleware MUST return an instance of \Psr\Http\Message\ResponseInterface. Each middleware SHOULD invoke the next middleware and pass it Request and Response objects as arguments.

which is quoted from the documentation of slimframework: http://www.slimframework.com/docs/concepts/middleware.html
Is this mechanism by design?
I think it would be nice that harmony framework can work with most of the third party middlewares out there.

That would be awesome indeed if Harmony could work well with other middlewares! So thank you really much for the report as the current mechanism wasn't intentional. :)
Could you tell me other details about your setup? Did you use any official Harmony middleware along with the CORS one? If so, I do hope the only problem is that Harmony's middlewares don't return a response.

I just pushed to dev-master with the fixes! Could you ensure if it works for you?

hi @kocsismate , after this modification, everything works. Below is my setup:
$app
    ->addMiddleware('cors', $corsMiddleware)
    ->addMiddleware("fast_route", new FastRouteMiddleware($router))
    ->addMiddleware("dispatcher", new DispatcherMiddleware($container))
    ->addFinalMiddleware("responder", new DiactorosResponderMiddleware(new SapiEmitter()));

Actually I am also working with your yin-middlewares, I think there are also some modifications needed. For example, in JsonApiDispatcherMiddleware.php an ErrorDocument is returned directly from the middleware:
// JsonApiDispatcherMiddleware.php

...
if ($callable === null) {
    return $this->getDispatchErrorDocument($this->getDispatchError())->getResponse($response);
}
...

Good job with the PR, I'll release Harmony 2.3 shortly with this fix! Your comment on Yin-middlewares is also valid, I'll address it soon.

I've done with the release: Harmony will trigger a deprecated notice if a middleware returns with a Response.

good news!


better ris-live stream format support for OPEN and STATUS messages

OPEN message and STATUS message from ris-live stream need some effort to be better translated and handled.
ExaBGP API reference: https://github.com/Exa-Networks/exabgp/wiki/Controlling-ExaBGP-:-API-for-received-messages
ExaBGP FSM: https://github.com/Exa-Networks/exabgp/blob/master/lib/exabgp/bgp/fsm.py

After checking with ExaBGP source code (mainly referencing peer.py), here is the state mapping:

down -> IDLE: TCP connection lost.
connected -> CONNECT: TCP connection established, ready to send OPEN message
up -> ESTABLISHED: peer BGP connection established.

status message fixed in commit 53300b2

Remaining tasks on this issue:

[ ] add OPEN message parsing to bgpstream_parsebgp_common
[ ]  parse the direction field from ris-live JSON

This issue is resolved by earlier commits. After double checking with the RIS Live manual, the current version has sufficient support for STATUS and OPEN messages.


Passes option to skip Ember.String processing

[x] Add behaviour
[ ] Test behaviour

@locks what the test? use case
I want help you on this

@locks - Looks good, I just released the polyfill with the required changes, can you rebase and update the minimum version?

@villander we need to check that we are actually detecting the @ember/string and passing the appropriate option to the babel plugin.

We are G.T.G.™


Configuration for Barong

I can make Peatio working by Auth0 or Google authentication, but I can't with Barong so far.
So, when you installed Peatio, how did you set nginx configuration for Barong?
There is an example in Peatio installing guide for Peatio and Trading UI.  But not for Barong.
In my case, I have installed Peatio, Trading UI, Barong in an instance.
So can you please give me a hint how to run Barong server and how to set Nginx configuration for Barong?
Another one is: Does Barong work without DNS?
Thank you.

I think workbench would be the best example in installing both
We don't have a better example at the moment.

@Coding-Jewelry please reopen issue if you still have question.


RAINCATCH-451: Update to use new simple store module

What
Added missing id field in create handler to be used with the new simple store module. Added function to check for and remove angular generated variables.
JIRA Ticket
https://issues.jboss.org/browse/RAINCATCH-451

Moved addition of id field to each raincatcatcher modules(message, workflow, workorder, result). Moved removal of angular generated variables to demo-portal.


Missing linux dependency in feedstock?

Howdy,
It looks like a dependency is missing from the wxpython linux feedstock. When running in the conda-forge linux build container (docker run -it condaforge/linux-anvil-comp7 bash), the following commands:
conda create -y -p testenv python=3.6
conda activate `pwd`/testenv
conda install -y -c conda-forge wxpython
python -c "import wx"

produce this error:
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/home/conda/testenv/lib/python3.6/site-packages/wx/__init__.py", line 17, in <module>
    from wx.core import *
  File "/home/conda/testenv/lib/python3.6/site-packages/wx/core.py", line 12, in <module>
    from ._core import *
ImportError: libXxf86vm.so.1: cannot open shared object file: No such file or directory

I think this has not been seen before because libXxf86vm is being implicitly installed due to gtk2-devel being listed in yum_requirements.txt.
I've tried and failed to resolve the issue by running conda install -c conda-forge libXxf86vm-cos6-x86_64. Any ideas?

The docker container does not provide the X libraries. They are expected to be present in the end user system.


I use puppeteer to get chrome transferred protocol, The correct result is QUIC，but i get TLS 1.2

this is my code :
let response = await page.goto('https://caddy.olplan.com/movie/10.mp4');
// console.log(cookies);
console.log(response.securityDetails().protocol());
the result is :TLS 1.2
expect result is: QUIC
Is there something wrong？

@caserffer the link doesn't open for me - server responds with 503.
Do you have another link for me to look into this?


[Bug] [flinkx-core] 不支持-s参数

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened

flinkx 不支持 -s参数    也就是不支持手动checkpoint 恢复
What you expected to happen
希望支持  flinkx    -s参数
How to reproduce
flinkx  -s
Anything else
No response
Version
1.12_release
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Try this.
第二次启动的时候配置checkpoint path在-confProp中
-confProp "{"execution.checkpointing.interval":10000,"execution.savepoint.path":"hdfs://localhost:9000/checkpoint/01c6ca211f3f0b632ef77123eb254134/chk-22/_metadata"}"


Model size and test speed for help

Sorry to disturb you again. As I use the script "run_distributed_on_disk_a6k5_AdamW_Curicullum_Large_assistant_teacher_num_3_aa.sh" offered to train, I get the model size is 3715.30M and the pretrained Bnext_large model size is 1246.96M. Is there something wrong with me, can you help me ?Forthermore, the table of paper say that the BNext-L param is 106.1M, what is the matter.
There is other problem ： how can I test the quant model speed on the cpu. can you give me same advices?
Thank you so much！

Dear Shan Yang,
I will try to answer the second question. You cannot easily test the speed using existing open-source software toolkit. But we are working on it. We plan to support for both CPU and GPU hardware, and please stay tuned.

Hi Shan Yang,
For your first question: you can check the code here (https://github.com/hpi-xnor/BNext/blob/dfcf347a30e3bc08606b8cad2c8d4a329d5a5b28/src/train_assistant_group_amp.py#L648-L662), we save not only the model state_dict, but also optimizer state_dict and the training procedure information, which explain why the checkpoint size is way larger than model size.
For your second question: The existing model is still saved using torch.save() function, which only supports 32-bit representation. In this case, it is impossible to directly get a  106.1M BNext-L using the torch library, even though all weights in HardBinaryConv are represented as +1&-1. We plan to support a BNN-specific torch extension toolkit in the near future, please stay tuned.

Thanks for your answers!

please check the binary layers implemented in bitorch-engine.


Javascript strings not matched in some cases

I'm using this in the context of marked but I don't think that matters in this case.
✅  Works
➡  input
 ```js
 var foo = debug('app:meta');
 ```

⬅  output

var foo = debug('app:meta');

❌  Works not
➡  input
 ```js
 debug('app:meta');
 ```

⬅  output

debug('app:meta');

Not that the string is not properly captured and it erroneously identifies :meta as a "symbol."

Is this still an issue with current builds?  Thanks.  If so a fiddlejs example would be great.
Quick template:
https://jsfiddle.net/ajoshguy/nagkqytv/


fix: Search by author or tag won't trigger

Closes #168

@near-akaia-root Do you have a preview for this fix? Also, previously our search support combination search conditions, e.g. find posts with label abc and by author xyz.near (and also combines search with keyword and sort). If I read right, your refactor dropped this feature

For example:

I still have previous widgets deployed at my account, you can try combine the conditions here: https://near.social/#/bo.near/widget/gigs-board.pages.Feed?nearDevGovGigsContractAccountId=devgovgigs.near

UPD:
Preview
https://near.org/root.akaia.near/widget/gigs-board.pages.Feed?nearDevGovGigsContractAccountId=devgovgigs.near

@ailisp this is how it looks on preview:

@near-akaia-root If I read right, your refactor dropped this feature

It shouldn't

Cool! Just tried out your preview, looks good!


wrapped the functions with JIT compiler, which is faster than the reg…

…ular python functions.

Fixes #98

Linking the other comment here - https://github.com/einsteinpy/einsteinpy/issues/98#issuecomment-1669638247. coordinates is already numba-accelerated.


[INLONG-8116][Sort] Support table api config setting

Prepare a Pull Request

[INLONG-8116][Sort] Support table api config setting

Fixes #8116

Motivation
Support table api config setting
ref to https://nightlies.apache.org/flink/flink-docs-master/docs/dev/table/config/
Modifications
Add two table config setting in sort core

Please rebase from the master to fix the failed UT after https://github.com/apache/inlong/pull/8128 is merged.


Handling junctions in Windows

A similar tool in Rust - dust - handles junctions correctly. Might be worth a look.

Thank you for reporting this.
I'm most probably not going to work on this myself (not a Windows user), but would maybe find the time to review a PR, if someone wants to work on that.

Thank you for reporting this.
I'm most probably not going to work on this myself (not a Windows user), but would maybe find the time to review a PR, if someone wants to work on that.


AttributeError: Can't get attribute 'tokenizer' on <module 'main'>

Hi,
I am trying to test the pickled objects, to verify that I can import the vectorizer and unpickle the classifier.
import re
import os
from vectorizer import vect

clf = pickle.load(open(os.path.join('pkl_objects', 'classifier.pkl'), 'rb'))

However, I get this error:
----> 6 clf = pickle.load(open(os.path.join('pkl_objects', 'classifier.pkl'), 'rb'))
AttributeError: Can't get attribute 'tokenizer' on <module 'main'>
What is going on and how can I fix this error?
Thank you!

Hm, this looks like some namespace issue. Pickle is very sensitive about that. Do you have the tokenizer defined in the vectorizer.py file, like so?
   def tokenizer(text):
       text = re.sub('<[^>]*>', '', text)
       emoticons = re.findall('(?::|;|=)(?:-)?(?:\)|\(|D|P)',
                              text.lower())
       text = re.sub('[\W]+', ' ', text.lower()) \
                     + ' '.join(emoticons).replace('-', '')
       tokenized = [w for w in text.split() if w not in stop]
       return tokenized

   vect = HashingVectorizer(decode_error='ignore',
                            n_features=2**21,
                            preprocessor=None,
                            tokenizer=tokenizer)

Also have this issue. Can confirm vectorizer.py looks like that.

For what it's worth doing:
>>> import pickle
>>> import re
>>> import os
>>> from vectorizer import *
>>> clf = pickle.load(open(os.path.join('pkl_objects', 'classifier.pkl'), 'rb'))
>>> clf

Seemed to work for me. Thanks to digging around here https://stackoverflow.com/questions/40287657/load-pickled-object-in-different-file-attribute-error#comment67835396_40287657

Ah think I know the source of my issue. I was pickling the clf after the logistic regression model. In the textbook it has the pickling after the SGDCClassifier.

Thanks for the comments. Hm, that's weird, the SGDClassifier should behave exactly the same as the LogisticRegression classifier when it comes to pickling. I think in most cases, it's usually a namespace issue. Hope you were able to resolve it.

Also I realized I picked the Pipeline object by pickling the earlier classifier...
https://github.com/rasbt/python-machine-learning-book-2nd-edition/blob/faaacc278924bf3ff0f27b56cc8962a94e90d0f4/code/ch08/ch08.py#L485


Format not avalible (am I being dumb?)

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.06.06
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log

## Verbose log
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['-v', '-f', '[height=720]', 'https://www.youtube.com/watch?v=VD0z_YB6gWc']
[debug] Encodings: locale cp1250, fs utf-8, out utf-8, pref cp1250
[debug] youtube-dl version 2020.06.06
[debug] Python version 3.8.3 (CPython) - Windows-10-10.0.18362-SP0
[debug] exe versions: none
[debug] Proxy map: {}
[youtube] VD0z_YB6gWc: Downloading webpage
[youtube] {18} signature length 109, html5 player 39dd62a0
[youtube] {137} signature length 109, html5 player 39dd62a0
[youtube] {248} signature length 109, html5 player 39dd62a0
[youtube] {136} signature length 109, html5 player 39dd62a0
[youtube] {247} signature length 109, html5 player 39dd62a0
[youtube] {135} signature length 109, html5 player 39dd62a0
[youtube] {244} signature length 105, html5 player 39dd62a0
[youtube] {134} signature length 109, html5 player 39dd62a0
[youtube] {243} signature length 109, html5 player 39dd62a0
[youtube] {133} signature length 109, html5 player 39dd62a0
[youtube] {242} signature length 109, html5 player 39dd62a0
[youtube] {160} signature length 105, html5 player 39dd62a0
[youtube] {278} signature length 109, html5 player 39dd62a0
[youtube] {140} signature length 109, html5 player 39dd62a0
[youtube] {249} signature length 109, html5 player 39dd62a0
[youtube] {250} signature length 109, html5 player 39dd62a0
[youtube] {251} signature length 105, html5 player 39dd62a0
ERROR: requested format not available
Traceback (most recent call last):
  File "c:\users\user1\appdata\local\programs\python\python38\lib\site-packages\youtube_dl\YoutubeDL.py", line 808, in extract_info
    return self.process_ie_result(ie_result, download, extra_info)
  File "c:\users\user1\appdata\local\programs\python\python38\lib\site-packages\youtube_dl\YoutubeDL.py", line 863, in process_ie_result
    return self.process_video_result(ie_result, download=download)
  File "c:\users\user1\appdata\local\programs\python\python38\lib\site-packages\youtube_dl\YoutubeDL.py", line 1635, in process_video_result
    raise ExtractorError('requested format not available',
youtube_dl.utils.ExtractorError: requested format not available

Description

The Video is playable on YT in 720 and 1080, but yt-dl only wants to do 360 for some reason. I hope I'm just doing something wrong. Video only download works fine. What's the matter here? (am I just being dislectic or smthn?)

[height=720] means best[height=720], for what does best mean read FAQ.

maybe you misunderstood. The Downloader doesn't want to download an mp4 formatted file in anything higher then 360p, even though the video is avalible in higher qualities. Parsing nothing results in a 360, parsing the you want something higher in a combined vid-aud results in it being not avalible.

Again: read what does best mean.


faulty "App wants additional access to your Google Account" prompt

Describe the bug
Upon repeated sign in, the SDK shows a prompt that reads "App wants additional access to your Google Account" even though no extra access is being requested. This is highly confusing for users.
Issue is reproducible in the sample app.

screenshot

To Reproduce
Steps to reproduce the behavior:

sign in
sign out
sign in again

See video
https://github.com/user-attachments/assets/3ef805e3-39de-473c-91b2-35afe30ef66b
Expected behavior
I don't expect to see the "App wants additional access to your Google Account" because I'm not asking for extra access.
Screenshots
see above
Environment

Device: iphone 17.4 simulator

Yes, this is a big deal and a regression compared to months ago where it unnecessarily scares users into thinking something funny/risky is going on.  +1 for this being an important fix!

Thanks for the issue. We are investigating. I will update here as I learn more.

Same here

same here

It's clear at this point that there's an issue, and people are looking into it. Let's please not spam the issue and wait for actual updates.
Thank you

Hi @mdmathias thanks for looking into this. It's been over 2 weeks now - do you have any updates to share?

Same here when using SwfitUI. Any updates?

👋  thanks for response, does this link work?

Sadly no: "This private-user-images.githubusercontent.com page can’t be found"

hello, I put it on Y: https://youtube.com/shorts/O88Dw8pfdYU


[Docs] Replace mesheryctl app and onboard commands

Current State
The Meshery Catalog concept page lists an antiquated mesheryctl command:

Onboarding an application. mesheryctl app onboard -f [file-path]

Desired State
The mesheryctl design command should be listed.

Contributor Guides and Handbook

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

@leecalcote These changes align well with this pending PR #10603
Thank you!


fix: make cantract_call costs instantiation cost to each instantiating callee VM

Description
This PR makes cantract_call costs instantiation cost to each instantiating callee VM. This is for https://github.com/line/cosmwasm/issues/257.
This PR is constructed by following 2 parts

follow https://github.com/line/lbm-sdk/pull/835 's changing of the interface of GetContractEnv
use the instantiation cost calculated by the above change to limit the input size of dynamic links

TODO
update dependent cosmwasm after it is updated
Types of changes

[x] Bug fix (changes which fixes an issue)
[ ] New feature (changes which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] ETC (build, ci, docs, perf, refactor, style, test)

Checklist

[x] I followed the contributing guidelines.
[x] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.
[x] The PR title and commits are followed conventional commit form.

LGTM
If the dependencies have been updated and the test has passed, approve it.


Add vendor/ to .gitignore

This allows developers to install project dependencies under vendor/ without it polluting the git status view.

Coverage increased (+0.06%) to 99.821% when pulling 0667f0df5d3bf83be136335ba5ed7819244648a5 on renier:master into 0adc0e4f26251252be7d29234bb2c8758da83504 on softlayer:master.

Coverage increased (+0.06%) to 99.821% when pulling 0667f0df5d3bf83be136335ba5ed7819244648a5 on renier:master into 0adc0e4f26251252be7d29234bb2c8758da83504 on softlayer:master.

Coverage increased (+0.06%) to 99.821% when pulling 0667f0df5d3bf83be136335ba5ed7819244648a5 on renier:master into 0adc0e4f26251252be7d29234bb2c8758da83504 on softlayer:master.

Coverage increased (+0.06%) to 99.821% when pulling 0667f0df5d3bf83be136335ba5ed7819244648a5 on renier:master into 0adc0e4f26251252be7d29234bb2c8758da83504 on softlayer:master.

@SLsthompson Happend here, too...
The checks in this pull request fails only when testing on JRuby19, because of the coveralls->tins dependency (tins requires Ruby 2.0).
So, should we remove jruby19 testing from .travis.yml? Or should we use something other than coveralls for the coverage testing? e.g. simplecov