Push (ipfs daemon needs running):
# ctr image pull ghcr.io/stargz-containers/ubuntu:20.04-org
ghcr.io/stargz-containers/ubuntu:20.04-org:                                       resolved       |++++++++++++++++++++++++++++++++++++++|
index-sha256:adf73ca014822ad8237623d388cedf4d5346aa72c270c5acc01431cc93e18e2d:    done           |++++++++++++++++++++++++++++++++++++++|
manifest-sha256:86ac87f73641c920fb42cc9612d4fb57b5626b56ea2a19b894d0673fd5b4f2e9: done           |++++++++++++++++++++++++++++++++++++++|
layer-sha256:5e9250ddb7d0fa6d13302c7c3e6a0aa40390e42424caed1e5289077ee4054709:    done           |++++++++++++++++++++++++++++++++++++++|
layer-sha256:345e3491a907bb7c6f1bdddcf4a94284b8b6ddd77eb7d93f09432b17b20f2bbe:    done           |++++++++++++++++++++++++++++++++++++++|
layer-sha256:57671312ef6fdbecf340e5fed0fb0863350cd806c92b1fdd7978adbd02afc5c3:    done           |++++++++++++++++++++++++++++++++++++++|
config-sha256:7e0aa2d69a153215c790488ed1fcec162015e973e49962d438e18249d16fa9bd:   done           |++++++++++++++++++++++++++++++++++++++|
elapsed: 3.5 s                                                                    total:  27.2 M (7.8 MiB/s)
unpacking linux/amd64 sha256:adf73ca014822ad8237623d388cedf4d5346aa72c270c5acc01431cc93e18e2d...
done: 886.582309ms
# ctr ipfs push ghcr.io/stargz-containers/ubuntu:20.04-org
bafkreih7o4kebgbzdl5maeq2yxr6nn2abvdztb2lshxvszjzre4s4vsl5a

Pull and run (ipfs daemon needs running):
# ctr ipfs pull bafkreih7o4kebgbzdl5maeq2yxr6nn2abvdztb2lshxvszjzre4s4vsl5a
bafkreih7o4kebgbzdl5maeq2yxr6nn2abvdztb2lshxvszjzre4s4vsl5a:                      resolved       |++++++++++++++++++++++++++++++++++++++|
index-sha256:c3d79ec819e7572dc0f374a88cc704ba167a867302a749c2d1ed0b0088770409:    done           |++++++++++++++++++++++++++++++++++++++|
manifest-sha256:f5a09083ac89d020e91e19e685ad88148bc93810e101837a1854be8c8bc26b80: done           |++++++++++++++++++++++++++++++++++++++|
config-sha256:7e0aa2d69a153215c790488ed1fcec162015e973e49962d438e18249d16fa9bd:   done           |++++++++++++++++++++++++++++++++++++++|
layer-sha256:57671312ef6fdbecf340e5fed0fb0863350cd806c92b1fdd7978adbd02afc5c3:    done           |++++++++++++++++++++++++++++++++++++++|
layer-sha256:345e3491a907bb7c6f1bdddcf4a94284b8b6ddd77eb7d93f09432b17b20f2bbe:    done           |++++++++++++++++++++++++++++++++++++++|
layer-sha256:5e9250ddb7d0fa6d13302c7c3e6a0aa40390e42424caed1e5289077ee4054709:    done           |++++++++++++++++++++++++++++++++++++++|
elapsed: 1.2 s                                                                    total:  27.2 M (22.7 MiB/s)
# ctr run --rm -t bafkreih7o4kebgbzdl5maeq2yxr6nn2abvdztb2lshxvszjzre4s4vsl5a foo echo hello
hello

This example uses overlayfs snapshotter but any snapshotters can be used in theory because IPFS is handled by the IPFS-aware remotes.Resolver not by snapshotters. For example, stargz snapshotter supports directly mounting eStargz from IPFS to containers's rootfs with experimental lazy pulling support. Please see https://github.com/containerd/stargz-snapshotter/blob/main/docs/ipfs.md for details.
IPFS-enabled OCI Image
Each descriptor in an IPFS-enabled OCI image must contain the following IPFS URL in urls field.
ipfs://<CID>

<CID> is the case-insensitive CIDv1 of the blob that the descriptor points to.
An image is represented as a CID pointing to the OCI descriptor of the top-level blob of the image (i.e. image index).

# ipfs cat bafkreih7o4kebgbzdl5maeq2yxr6nn2abvdztb2lshxvszjzre4s4vsl5a | jq
{
  "mediaType": "application/vnd.oci.image.index.v1+json",
  "digest": "sha256:c3d79ec819e7572dc0f374a88cc704ba167a867302a749c2d1ed0b0088770409",
  "size": 313,
  "urls": [
    "ipfs://bafkreigd26pmqgphk4w4b43uvcgmobf2cz5im4ycu5e4fupnbmaiq5yebe"
  ]
}

Code changes to containerd client
This commit adds a package contrib/ipfs which contains containerd client codes for distributing OCI images on IPFS.

ipfs.Push pushes the image with converting to the IPFS-enabled format described above. Each blob is pushed as an UnixFS file for leveraging the chunking support.
The pushed image can be fetched using IPFS-aware ipfs.Resolver. When pulling the image, this resolver sees urls field in each descriptor and fetches the contents from IPFS.

Next steps
We will add the following changes to enable containerd to fully support IPFS.

Deciding the image reference naming convention of IPFS images and fixing references pkg of contianerd. (ipfs://<CID> or /ipfs/<CID>)

Discussed in nerdctl (https://github.com/containerd/nerdctl/issues/465).

Fixing components/tools that use containerd (CRI, nerdctl, etc) to support the IPFS image ref.

Related discussions

https://github.com/containerd/nerdctl/issues/465

discussion thread about the image reference for IPFS

https://github.com/containerd/stargz-snapshotter/pull/496

support of IPFS for containerd with experimental lazy pulling

ipcs: https://github.com/hinshun/ipcs

enables containerd to pull IPFS images with an IPFS-aware content store.
difference between ipcs and this PR is

the image format: ipcs fixes descriptor's digest field to the hash of the blob's root object but this PR leverages urls for keeping OCI compatibility.
the integration approach: ipcs integrates content store to IPFS but this PR (and stargz-snapshotter) change client-side (Converter and Resolver) and optionally snapshotter (if enabling lazy pulling).

Thanks @AkihiroSuda for the discussion.

Is this a feature that could start out in nerdctl? ctr doesn't seem like the right place to introduce it

Could we move #6176 forward at least? That is a generic patch and will be used by nerdctl and stargz-snapshotter.

#6176 has been merged.
Regarding IPFS, I agree with @dmcgowan. While IPFS is really cool, we don't have to have that in containerd itself.

IPFS will come to nerdctl soon https://github.com/containerd/nerdctl/pull/505

@dmcgowan @kzys Thanks for the comments. I'm closing this PR in favor of the work in nerdctl (https://github.com/containerd/nerdctl/pull/505). I'll reopen it when changes will be needed to containerd itself.

@AkihiroSuda I'll complete the PR in nerdctl first. Thank you for your time to review this PR and the one in nerdctl.

i would find this useful in containerd


Make CRDs schemas available

Make CRDs schemas available and find a way to use them in vscode.

I would see that today how the helm template do it


Organize content strategically: style fix

The ODI style guide says numbers above one should be numerals. Organize content strategically had an instance of three instead of 3. This issue is to change to 3.

Merged with April release.


feat: Attaches log group resource policies

Description

Log groups can have resource policies attached to them which are required when an AWS service is writing logs such as ElasticSearch.
Motivation and Context

Allows resource policies to be passed to the log-group module which will create the attachments.
Breaking Changes

None, the variables are new and default to empty thus creating nothing when not passed.
How Has This Been Tested?

[x] I have tested and validated these changes using one or more of the provided examples/* projects

I have created a log-group example and created the resources in our environment.

🤦  These resource policies are actually global, not pertained to a particular log group.  It is better to create them separately.


Function torch.onnx.symbolic_helper._onnx_unsupported()

Getting this error on the step python bark_perform.py:
Stacktrace:
Traceback (most recent call last):
  File "C:\Users\f1am3d\repos\bark\bark_perform.py", line 6, in <module>
    from bark_infinity import config
  File "C:\Users\f1am3d\repos\bark\bark_infinity\__init__.py", line 1, in <module>
    from .api import generate_audio, text_to_semantic, semantic_to_waveform, save_as_prompt
  File "C:\Users\f1am3d\repos\bark\bark_infinity\api.py", line 14, in <module>
    from .generation import get_SUNO_USE_DIRECTML
  File "C:\Users\f1am3d\repos\bark\bark_infinity\generation.py", line 7, in <module>
    from encodec import EncodecModel
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\encodec\__init__.py", line 12, in <module>
    from .model import EncodecModel
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\encodec\model.py", line 14, in <module>
    import torch
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\torch\__init__.py", line 1239, in <module>
    from torch import onnx as onnx
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\torch\onnx\__init__.py", line 12, in <module>
    from . import (  # usort:skip. Keep the order instead of sorting lexicographically
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\torch\onnx\symbolic_caffe2.py", line 4, in <module>
    from torch.onnx import symbolic_helper, symbolic_opset9 as opset9
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\torch\onnx\symbolic_helper.py", line 611, in <module>
    def _onnx_unsupported(op_name: str, value: Optional[_C.Value] = None) -> NoReturn:
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_decor\decorcache.py", line 77, in beartype
    return beartype_object(obj, conf)
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_decor\decorcore.py", line 98, in beartype_object
    _beartype_object_fatal(obj, conf=conf, **kwargs)
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_decor\decorcore.py", line 147, in _beartype_object_fatal
    beartype_nontype(obj, **kwargs)  # type: ignore[return-value]
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_decor\_decormore.py", line 178, in beartype_nontype
    return beartype_func(obj, **kwargs)  # type: ignore[return-value]
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_decor\_decormore.py", line 242, in beartype_func
    func_wrapper_code = generate_code(bear_call)
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_decor\wrap\wrapmain.py", line 187, in generate_code
    code_check_return = _code_check_return(bear_call)
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_decor\wrap\wrapmain.py", line 669, in _code_check_return
    reraise_exception_placeholder(
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_util\error\utilerror.py", line 214, in reraise_exception_placeholder
    raise exception.with_traceback(exception.__traceback__)
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_decor\wrap\wrapmain.py", line 618, in _code_check_return
    ) = make_func_wrapper_code(hint_sane, bear_call.conf, cls_stack)  # type: ignore[assignment]
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_util\cache\utilcachecall.py", line 259, in _callable_cached
    raise exception
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_util\cache\utilcachecall.py", line 251, in _callable_cached
    return_value = args_flat_to_return_value[args_flat] = func(
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_decor\wrap\_wrapcode.py", line 86, in make_func_wrapper_code
    ) = make_check_expr(hint, conf, cls_stack)
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_util\cache\utilcachecall.py", line 259, in _callable_cached
    raise exception
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_util\cache\utilcachecall.py", line 251, in _callable_cached
    return_value = args_flat_to_return_value[args_flat] = func(
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_check\code\codemake.py", line 706, in make_check_expr
    die_if_hint_pep_unsupported(
  File "C:\Users\f1am3d\miniconda3\envs\pydml_torch2\lib\site-packages\beartype\_util\hint\pep\utilpeptest.py", line 205, in die_if_hint_pep_unsupported
    raise BeartypeDecorHintPep484Exception(
beartype.roar.BeartypeDecorHintPep484Exception: Function torch.onnx.symbolic_helper._onnx_unsupported() return PEP 484 type hint "typing.NoReturn" invalid in this type hint context (i.e., "typing.NoReturn" valid only as non-nested return annotation).

OS: Windows 11
GPU: Radeon 7900 XTX

same here😥

How did everyone install Bark?

I ran the INSTALL_bark_infinity_windows.bat file, and the installation appeared to be successful (no errors popped up). However, I encounter this error when I run the launch file. I have also installed various things like Python, PyTorch, and NumPy for other program(with pip in global). I'm wondering if there might be a conflict.

I'll check it out Tuesday afternoon, didn't get a chance tonight. Enough people all having the same problem, must be something changed recently.

How did everyone install Bark?

Following these steps: https://github.com/JonathanFly/bark#-pytorch-20-bark-amd-install-test-pytorch-20-
And after the first unsuccessful attempt, these steps: https://github.com/JonathanFly/bark#-this-amd-pytorch-1131-slower

I ran the 'INSTALL_BARK_INFINITY' (this seems to be an updated name from the docs).

Run COMMAND_LINE_BARK_INFINITY.bat
Then type pip install beartype>=0.16.1 --upgrade
After it finishes, close that window, and open the GUI or command line again and I think it should work.
If not, try also typing pip list and see what version it says, if it's not version 0.16.1.

upgrading beartype to 0.16.1 did the trick! Thanks so much Jonathan!

Ok closing for now. I think we're past the age of beartype troubles.


NIFI-9267 Replaced nifi-standard-web-test-utils with mockwebserver

Description of PR
NIFI-9267 Replaces usage of the custom Jetty-based TestServer in nifi-standard-web-test-utils with the OkHttp MockWebServer.  The OkHttp MockWebServer provides a streamlined approach to sending prepared responses and evaluating requests.  Additional changes include removing several integration tests which referenced the SslContextUtils class in nifi-standard-web-test-utils.  These adjustments allowed for removing all references to nifi-standard-web-test-utils.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[X] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[X] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[X] Has your PR been rebased against the latest commit within the target branch (typically main)?

[X] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[X] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[X] Have you written or updated unit tests to verify your changes?
[X] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Thanks for this update, @exceptionfactory, it looks cleaner to use the MockWebServer rather than the custom TestServer. The tests all pass. +1 for me, once the merge conflicts are resolved!

Thanks for the review @gresockj! I rebased to resolve the conflicts, and also consolidated the three AWS Gateway classes into a single class that contains two additional methods.  This ensures that all three types of credentials get tested without running through every single test method three times.


Add support for 'credentials: "include" when fetching challenge

I implemented the challenges server side by generating a new HMAC key for every session (via PHP). Thus, PHP sends a session ID via cookie to the client. The problem now is that the cookie gets ignored , because the ALTCHA widget does not include a credentials: "include" in its fetch() call for requesting the challenge. Thus, when I submit the form, a new session is created (and with it a new HMAC key), resulting in a signature mismatch. Would it be possible to add this option?

Hi, yes, this will be added soon with custom headers #81

When that feature will be implemented, will it require manual handling of the Cookie header, or will this be handled by the browser or ALTCHA?
Because having access to set a Cookie header is one thing, but automagically handling cookies via credentials: "include" is a whole different thing, as the latter leverages the browser's Cookie mechanism

Released as 1.1.0. See the docs for more details.

Works perfectly for my use case! 👍 Thank you!


fix(runtime): revert alignment of commonjs exports dependency

!canary

@JSerFeng  please write the revert reason


Support webviews without internet

I can't seem to open a webview without a working internet connection.
VS Code is doing request on vscode-webview-test.com, which fail without internet.
I cannot find that endpoint in your documentation, only in source code
Could it be possible to

document why this endpoint is needed
make webviews work without it
allow users to opt-out of this call

Would be wonderful! Thanks a lot

Steps to Reproduce:

Open this project
Disable your internet connection
Run command "Start cat coding session"

Version: 1.44.0-insider
Commit: 13b38d2a494612ace32c4fb878824d6329d4c51e
Date: 2020-03-30T08:08:20.753Z (2 wks ago)
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 4.15.0-91-generic
Reproduces with all extensions disabled

I forgot I had this in my settings "webview.experimental.useExternalEndpoint": true in order to make webviews work.
Apologies.
Related to #83188


[HOTFIX][ZEPPELIN-2178] Prevent from cleaning output in Personalized mode

What is this PR for?
Fixing empty output issue while using personalized mode.
What type of PR is it?
[Hot Fix]
Todos

[x] - Making default user's paragraph while accessing at the first time.

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-2178

How should this be tested?
Use personalized mode
Screenshots (if appropriate)
N/A
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

CI results here: https://travis-ci.org/jongyoul/zeppelin/builds/206477104 For current travis issue, cannot see all the profiles in details, thus Jenkins fails

LGTM!


[FEATURE] create Incident Date search component

FIGMA design spec

Search component for specific incident date.
Requirements

Input fields for specific date (month, day, & year, or whatever the user has)
Link to switch to Date Range
Back and Next buttons, which will move user through the flow

only need one component for specific date + date range


Sentry: enhance logging

@egabancho  is this deployed on CDS?

Yes

I like it! :shipit:
Can you rebase on top of latest legacy and add Reviewed-by: me?

Thanks, merged as:

67cd1bca8ffced7c502119533c93878bd9c65a6f WebStyle: logging level on some IOError
5f7b20d88d9052e654f465ffb6b332eef843dc30 WebStyle: default logging level for SERVER_RETURN
2ae9fdd24c89f677ca47bcaf83c44d8ae631374a errorlib: sentry logging enhancement


[Feature] Module decorator

Informations

Type
Version

Improvement
4.x

Description
As a developer, I would like to create a Module and configure components/controllers exported by the module. The module can be added to the ServerSetttings.
Example
import {Module} from "@tsed/common";

@Module({
     controllers: [],
     components: []
})
export class MyModule {
    $onInit(){ // support hooks
    }
}

Then:
@ServerSettings({
    imports: [
        MyModule
    ]
})

5.x


Add export xlsx button

Close #388
Currently I made it so it just downloads a sheet for the last month e.g. if you click it in November, it gives the excel for October.
I also limited this to admins.
I have checked on random days and the sheet looks correct on those days.
Please check whether this is good.

Pull Request Test Coverage Report for Build 2480

9 of 9 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 96.364%

Totals

Change from base Build 2469:
0.05%

Covered Lines:
583

Relevant Lines:
605

💛  - Coveralls

Looks good to me! Just a minor detail tho, I don't have admin privileges HAHA

If Hilda say can then that means can xD
Okay jk I will try to merge this asap but need to find out what this vulnerability thing D:

Okay jk I will try to merge this asap but need to find out what this vulnerability thing D:

🤣 I'm guessing it's cause of axlsx?

Looks good to me! Just a minor detail tho, I don't have admin privileges HAHA

Eh, I thought ops and logs have admin rights?

Since ops and logs don't have admin privileges, do I just make this button available for everyone?

Since ops and logs don't have admin privileges, do I just make this button available for everyone?

I've gotten admin privileges, can merge now, thanks!


Shinigami ID

Source information
Shinigami 1.4.57
Source language
Indonesia
Source new URL
https://shinigamitoon.app/
Other details
Change domain shinigami ID shinigamitoon.id to shinigamitoon.app
Acknowledgements

[X] I have updated all installed extensions.
[X] I have opened WebView and checked that the source URL has not changed yet.
[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I will fill out all of the requested information in this form.

https://github.com/keiyoushi/extensions-source/issues/1552
Can't you add this feature?

Can't you add this feature?

Just FYI, sometimes when sources switch URL, they might change the websites theme or other portions of stuff, which breaks the extension, so sometimes it's not enough to just switch URLs and call it a day.
I don't know much more about this, and maybe it'll work currently, but if other parts get change, someone has to rewrite the code for the extension. (And looking at the code for Shinigami, looks like it needs some extra lines to get things to work/show up)

Ahhh, I see too bad that can't be done  🥲
but that's okay, thanks for the info 🙌


MySQL. Missed package name for the type "Time"

Probably(not sure) if an attribute name ends with "Time" - package name is missed
see the "PaymentTime" attribute
CREATE TABLE `event_purchase` (
  `id` bigint(20) NOT NULL AUTO_INCREMENT,
  `time_stamp` timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP,
  `payment_date` date NOT NULL,
  `payment_time` time DEFAULT NULL,
  PRIMARY KEY (`id`),
) ENGINE=InnoDB AUTO_INCREMENT=47 DEFAULT CHARSET=latin1

	PaymentDate     *time.Time     `json:"payment_date"`     // payment_date
	PaymentTime     Time           `json:"payment_time"`     // payment_time

MySQL supports a lot of aliases for other types, and unfortunately xo doesn't try to cover all of them at this time. I would just simply suggest changing your schema to use timestamps if possible. Also, I'm more than willing to take a pull request with the added name alias for the Go type.


Travis: only install pypy for pypy job

Use a single pypy job only, with latest pytest.

Codecov Report

Merging #157 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #157   +/-   ##
=======================================
  Coverage   28.77%   28.77%
=======================================
  Files           4        4
  Lines         497      497
=======================================
  Hits          143      143
  Misses        354      354

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 fe1e991...8665fb2. Read the comment docs.

This appears to help with the pypy segfaults.


MAINT: linalg: Use the new fft subpackage in linalg.dft test and docstring.

Also tweaked the dft docstring to show the matrix m.

LGTM too, merged. Thanks @WarrenWeckesser

Thanks Eric, Ralf.
I just noticed that I didn't change the reference to scipy.fftpack.fft in the dft docstring.  A one word PR is forthcoming...


Fix attributes assignment in Metal Diagnostics Options of LaunchAction

I have made a mistake in https://github.com/tuist/XcodeProj/pull/828
Short description 📝
In my previous PR, I mistakenly set the attributes "showGraphicsOverview" and "logGraphicsOverview" to the value of .xmlString. This was incorrect, as these two fields, unlike others, should contain values in the form of "Yes" or "No" instead of the values returned from .xmlString, which are "YES" and "NO".
Currently, I have changed it to:
if showGraphicsOverview {
    attributes["showGraphicsOverview"] = "Yes"
}
if logGraphicsOverview {
    attributes["logGraphicsOverview"] = "Yes"
}

Is this approach acceptable, or should we consider creating a new property, such as .xmlStringCapitalized? Alternatively, do you have a better suggestion?
This PR is related to https://github.com/tuist/tuist/pull/6537

I will release a new patch version with this 🤞


feat(zetaclient): add support for EIP-1559 outbound transactions

Description
This PR introduces support for EIP-1559 gas fees for outbound transactions in zetaclientd.
Closes: https://github.com/zeta-chain/node/issues/1952, https://github.com/zeta-chain/node/issues/1085
The main changes are located in:

zetaclient/chains/evm/observer/observer.go
zetaclient/chains/evm/signer/gas.go
zetaclient/chains/evm/signer/outbound_data.go
zetaclient/chains/evm/signer/signer.go

Changes

[x] Minor refactor of  Orchestrator
[x] Logging improvements. Moving dynamic values into log event
[x] Refactoring of signer.NewOutboundData(...)
[x] Support for DynamicFeeTx
[x] Fetch GasTipCap (evm.SuggestGasTipCap)
[x] Minor refactoring

Type of change

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Include instructions and any relevant details so others can reproduce.

[x] Tested CCTX in localnet
[ ] Tested in development environment
[x] Go unit tests
[x] Go integration tests
[ ] Tested via GitHub Actions

Checklist:

[x] I have added unit tests that prove my fix feature works

@coderabbitai review

This PR is obsolete. I'll create a new one.


feat: make provider interface "stateless"

This PR

State management for providers is now handled internally, eliminating the need for providers to implement or maintain their own state. The associated method has been removed from the interface, simplifying provider implementations.
State updates occur both during provider initialization and dynamically at runtime through various events emitted by the provider implementation.
Related Issues

fixes #258

Great progress! I have 2 concerns: this and this.


How do I disable semicolon? I don't want to see semicolon in my code.

Environment information
CLI:
  Version:                      1.6.4
  Color support:                true

Platform:
  CPU Architecture:             x86_64
  OS:                           windows

Environment:
  BIOME_LOG_DIR:                unset
  NO_COLOR:                     unset
  TERM:                         unset
  JS_RUNTIME_VERSION:           "v21.6.0"
  JS_RUNTIME_NAME:              "node"
  NODE_PACKAGE_MANAGER:         "pnpm/8.15.5"

Biome Configuration:
  Error:                        Expected a property but instead found '}'.
  Error:                        Found an unknown value ``.

		}
	}
}

Playground link
https://biomejs.dev/playground/
Code of Conduct

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

Please check the document: https://biomejs.dev/reference/configuration/#javascriptformattersemicolons


Fix #98 by adding authorize function support

This pull request fixes #98 by adding authorize function support to App and AsyncApp. This is based on #99 branch. The essential change is this commit: https://github.com/slackapi/bolt-python/commit/8e8248ddb0618a37476485913b9fe5d789e9f43c
Category (place an x in each of the [ ])

[x] slack_bolt.App and/or its core components
[x] slack_bolt.async_app.AsyncApp and/or its core components
[ ] Adapters in slack_bolt.adapter
[ ] Document pages under /docs
[ ] Others

Requirements (place an x in each [ ])
Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you are agreeing to those rules.

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've run ./scripts/install_all_and_run_tests.sh after making the changes.

Codecov Report

Merging #100 into main will decrease coverage by 0.91%.
The diff coverage is 68.53%.

@@            Coverage Diff             @@
##             main     #100      +/-   ##
==========================================
- Coverage   89.58%   88.67%   -0.92%
==========================================
  Files         129      133       +4
  Lines        3621     3778     +157
==========================================
+ Hits         3244     3350     +106
- Misses        377      428      +51

Impacted Files
Coverage Δ

slack_bolt/auth/async_authorize.py
46.15% <46.15%> (ø)

slack_bolt/auth/authorize.py
46.15% <46.15%> (ø)

slack_bolt/auth/async_authorize_args.py
66.66% <66.66%> (ø)

slack_bolt/auth/authorize_args.py
66.66% <66.66%> (ø)

...e/authorization/async_multi_teams_authorization.py
70.96% <69.23%> (+12.43%)
:arrow_up:

...dleware/authorization/multi_teams_authorization.py
73.52% <73.33%> (+16.02%)
:arrow_up:

slack_bolt/app/app.py
84.27% <83.87%> (+0.85%)
:arrow_up:

slack_bolt/app/async_app.py
91.30% <87.50%> (+0.03%)
:arrow_up:

slack_bolt/auth/authorize_result.py
95.23% <95.23%> (ø)

slack_bolt/auth/__init__.py
100.00% <100.00%> (ø)

... and 15 more

Continue to review full report at Codecov.

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

Let me merge this and release a new alpha version. Please feel free to write in when you see any rooms for improvements. I will fix them by the first beta release.


[Sema] Diagnose 'init' use as identifier in "case" statements

Provide a diagnostic which points out that 'init' can only
be used in "case" statements when properly escaped.
Resolves: rdar://problem/44751408

@jrose-apple I tried to add this logic into parseCaseStmt but it seems like parser failures aren't propagated all the way, which causes exhaustivity checker to run and produces "not exhaustive diagnostic", so that's why I decided to put it where it is right now.

Also can you test the behavior with both -swift-version 4 and 5.

Is there supposed to be difference between 4 and 5? Sorry I don’t know the details of special names behavior...

@xedin Yes, the special declname for init behaves differently between 4 and 5. See test/Compatibility/special_case_name.swift (4) and test/decl/enum/special_case_name.swift (5).

Okay, good to know! I will move these new tests there too then...

@xedin @jrose-apple I don't think banning case patterns named init is source-compatible. Look at this program:
enum E {
  case foo
  case `init`
  case other
}

func f(_ e: E) {
  switch e {
  case .foo: print("foo")
  case .init: print("init")
  default: print("default")
  }
}

f(.foo)
f(.init)
f(.`init`)
f(.other)

In -swift-version 4, it prints:
foo
init
init
default

In -swift-version 5, I get this:
xcrun swiftc q.swift -swift-version 5
q.swift:10:9: error: pattern cannot match values of type 'E'
  case .init: print("init")
       ~^~~~
q.swift:16:4: error: type 'E' has no member 'init'
f(.init)
   ^~~~

If I remove the default case,
q.swift:8:3: error: switch must be exhaustive
  switch e {
  ^
q.swift:8:3: note: add missing case: '.init'
  switch e {
  ^

So I'm not really happy with this PR as-is. It warrants further investigation.

In -swift-version 4, a member reference named ".init" looks up both the special declname and the member named init. So a "case .init" is probably getting resolved to an expression pattern. We could probably fix this so that it resolves to a case pattern then we don't have to ban anything.

Looks like this is more behaviour correctness problem then, I will let somebody who understands this better to deal with the problem...

In -swift-version 4, a member reference named ".init" looks up both the special declname and the member named init. So a "case .init" is probably getting resolved to an expression pattern. We could probably fix this so that it resolves to a case pattern then we don't have to ban anything.

I thought about that, but I'm not sure I like it, because it means we have different behavior for .init in pattern and expression contexts. It doesn't matter in this case, but it would for enums with payloads:
enum Confusing {
  case `init`(a: Int, b: Int)
  case other

  init(a: Int, b: Int) {
    print("Confusing, eh?")
    self = .other
  }
}

let x = Confusing.init(a: 0, b: 0) // calls initializer
if case .init(a: _, b: _) = x { // ??
  print("matches")
}

@jrose-apple Perhaps just emit a warning for 'case .init', but not an error?


interp: weaken panics to errors and return panicked values

Please take a look.
Updates #413.
Updates #487.

Thank you for your efforts. Sorry to bring my response a bit late in the process.
I have to say that  I'm not convinced that recovering panics inside Eval is a better practice than recovering it outside, i.e. in the host application (if necessary). My rationale is:

as panic can be recovered outside of Eval anyway, it is not to me a showstopper if it is not implemented in yaegi, the user can always achieve its desired behavior at a small cost.
panics may be of different origins, system problem, bug in yaegi,, script action. etc. I'm not sure that transforming all these in a yaegi error is the right thing to do. In fact I don't know the best thing to do (and can't), and therefore the best (to me) is still to let the panic propagate, and let the user decide based on context.
The context in panic is a full stack trace, invaluable for debugging and post-mortem (if no recover), and much richer than a custom error. With the current patch, I loose the ability to debug, and forbid the user to debug as well. Or we could generate stack trace part of error info, but in this case, why not just propagate panics ?

I just went through the PR again, and noticed that you indeed capture the info necessary to access the stack traces, sorry for having missed that. Although it doesn't change the conclusion.

I differ in opinion. What you find is that basically all uses of Eval without this then must be wrapped in a recover in order to avoid bringing down the application.

Note that all the things that a user can do with recovery they user can do with a type assertion on the returned error from Eval and then introspection on the interp.Panic.Value. With some additional work, the user could even have a stack trace of the interpreted code (which would not be possible at all with the propagated panic).

ping

I added a formatted stack buffer (as returned by runtime/debug.Stack()) to preserve the previous tracing behaviour, and still be able to recover automatically. Panic.Callers could indeed be used to dump a call trace at interpreter level in the future.

Thanks


interfaces not findable with search box

In my set of documentation, some interfaces are not findable in the search box. For example, when I type "rigidbody" I see several suggestions involving the class RigidBodyEventHandler, but nothing for RigidBody interface itself.
I will try to get more info about which interfaces or classes are having this problem.  This was previously a problem with another interface (called Edge) but after updating Dossier to latest version that one is now appearing.

Are these interfaces included in the side navigation menu? Everything in that menu should also be included in the search index.
One potential problem is the autocomplete widget. I'm using goog.ui.ac.AutoComplete, and I've noticed before the default matcher doesn't always match on the "closest" string - in your case showing RigidBodyEventHandler instead of RigidBody.

Yes, RigidBody is listed in the side navigation menu.  Looks like everything is listed there.  I found a more extreme example:  I have 12 classes or interfaces that start with the word "Display" but only one of them (DisplayGraph) appears when I type "display" into the search bar.

If you'd like further adjustments, please provide the types.js file generated by dossier so I have concrete data to work with.

Confirmed fixed.  Works well now.  Thanks!
For example, all the classes that start with "Display" are showing up.
That said, there are aspects of the current behavior that I might alter (but not urgent):
When I type "rigidbody" I get some of the classes/interfaces that start with "rigidbody" but not all of them.  I suspect the length of the list is being limited, and in my case I'm seeing some static properties that are less important than the other classes (with longer names) that are more important.
To be specific, I'm seeing:

But I'd rather see classes RigidBodyEventHandler, RigidBodyObserver on the list than static properties like RigidBodySim.W_.
Perhaps I'm asking to prioritize classes and interfaces over properties & methods?  Or to prioritize names that have the word fragment in them.
I've uploaded the types.js file to http://www.myphysicslab.com/rmnp/types.js

The list is currently capped at 10 entries.
Good suggestion for favoring types of properties. Will track that in #86


Example needed for handling auth exceptions

I need to be able to detect and handle specific errors for authorization requests (such as email being invalid or user account already exists). The documentation seems to only cover this for iOS and I don't see any way to detect these specific errors short of...
if(task.Exception.ToString().Contains("The email address is already in use by another account"))
or similar which is pretty unsatisfactory. An example of the proper way to achieve this would be super helpful.

Thanks for reporting this.  We've exposed granular errors for auth in the 4.2.0 release of the SDK  https://firebase.google.com/support/release-notes/unity#4.2.0
See https://firebase.google.com/docs/reference/unity/namespace/firebase/auth#namespace_firebase_1_1_auth_1aef1e65c370d8edb018dc417ae73ec11c for the set of errors and https://github.com/firebase/quickstart-unity/blob/master/auth/testapp/Assets/TestApp/UIHandler.cs#L210 as the example.
Cheers,
Stewart

Above github link not found. Here's is the updated link: https://github.com/firebase/quickstart-unity/blob/master/auth/testapp/Assets/Firebase/Sample/Auth/UIHandler.cs


opening a recipe flashes 404 while loading

The fix for #68 was defective, in that it treats a not-yet-loaded recipe as missing as well, so the 404 is flashed up briefly, instead of the spinner. It works correctly for the deleted case.
Three cases:

opening a recipe which is present in the apollo cache
opening a recipe which is not in the apollo cache, but does exist
opening a recipe which is not in the apollo cache, and doesn't exist in the database

In the first case, the recipe should render directly.
In the second, a spinner should show until the recipe is retrieved, and then it should render. Currently it shows 404 when it should show a spinner.
In the second, a spinner should show until the recipe is confirmed to not exist in the database, then it should show 404.
The second and third cases should work correctly when opening from a library search and by entering a URL directly in the location bar.

I also noticed the GraphQL query doesn't raise an error when a non-existent recipe is requested, it returns null. The equivalent REST endpoint raises NoResultException.

Interestingly, I am unable to delete a recipe and getting a foreign key constraint error in checking the use case. I will file a bug report and continue investigating at a later time:

I think that means the recipe is used as an ingredient in another recipe (e.g., meatballs w/in spaghetti and meatballs). You can just enter any non-existent ID to replicate the "deleted recipe" use case. It didn't have to previously exist.

So, the first thing that I did was to change the fetchPolicy of the useQuery call to network-only so that I could consistently reproduce use case two, where the flash of 404 was occuring instead of the loader. I also confirmed that when a recipe does not exist in the database, we get a null value back from the useQuery method for that recipe.
So the flash of 404 makes sense because in this case, both an existing recipe that has not loaded yet and a deleted recipe both have null results. The fix needs to distinguish between these two states.
In the first use case, I did not notice any defects in loading so that is working as intended.
For both the second and third use cases, there needs to be a way to know that there will never be a return value for a particular recipe. In my opinion, this belongs on the server side and returning a null value for a recipe is incorrect behavior. I am going to try and implement that solution. If a NoResultException is thrown from the server instead of returning a null, then it should be a simple matter of throwing a 404 when an error is returned and a loader while it is in a null state.


Failing to compile

Fails to compile with:
Compiling libracerd v0.3.1 (file:///home/.../nvim/plugged/YouCompleteMe/third_party/ycmd/third_party/racerd)
error: type alias is never used: `ListCompletionsResponse`
   --> /home/.../nvim/plugged/YouCompleteMe/third_party/ycmd/third_party/racerd/target/release/build/libracerd-38cb6b871b76c55b/out/http/completion.rs:281:1
    |
281 | type ListCompletionsResponse = Vec<CompletionResponse>;
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
    |
note: lint level defined here
   --> src/lib.rs:9:9
    |
9   | #![deny(warnings)]
    |         ^^^^^^^^

error: aborting due to previous error

error: Could not compile `libracerd`.

To learn more, run the command again with --verbose.

This is on
$ rustc -V
rustc 1.15.0-nightly (71c06a56a 2016-12-18)

This was fixed on master, just need to land it in YCM.


how to change to horizontal position?

how to change to horizontal position?

add  RotatedBox in Widge Chart
  RotatedBox(
      quarterTurns: 1,
      child: Chart(...),
      )

add rotated box in widget chart,
The problem was only with the numbers "0,1.."

Hey @andre-ab. There is no easy way to do this yet. You would have to make your own decoration with rotated values. You can do that with WidgetDecoration as well. ex:
WidgetDecoration(
  widgetDecorationBuilder: (builder, state, itemWidth, verticalMultiplier) {
    return Stack(
      children: [
        ...List.generate(
          state.data.listSize,
          (index) => Positioned(
            bottom: 0,
            width: itemWidth,
            height: 32,
            left: index * itemWidth,
            child: Center(
              child: RotatedBox(
                quarterTurns: 3,
                child: Text('$index'),
              ),
            ),
          ),
        ),
      ],
    );
  },
  margin: EdgeInsets.only(bottom: 32),
)


fix: check "sendIdleEvents" before calling "stopTimers"

Summary
Avoid calling stopTimers when sendIdleEvents is true, since that's how setSendIdleEvents works.
Changelog
[iOS] [Fixed] - avoid stopTimers when sendIdleEvents is true
Test Plan
None

@aleclarson could you go into more detail on why this change is necessary and spec out the test plan to verify that the change is having the intended effect? It's hard to say why this change is necessary with the little information you provided. Thank you.

Ah, I didn't notice the _sendIdleEvents check in hasPendingTimers method. My bad!


chore: Remove all merge tests except integration

No point in running format/build and unit tests on merge when they're run on all PRs and must be successful to induce a merge

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 79.14%. Comparing base (0bb1f67) to head (c1ea933).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #832   +/-   ##
=======================================
  Coverage   79.14%   79.14%
=======================================
  Files         250      250
  Lines       10675    10675
=======================================
  Hits         8449     8449
  Misses       2226     2226

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


When using cloud firestore on iOS, the app was killed

Steps to Reproduce

I just tried to use the cloud firestore.
However, the following bug was caused while running iOS application.
Logs

[  +34 ms] executing: [/Users/kyeonghwan-kong/flutter/] git rev-parse --abbrev-ref --symbolic @{u}
[  +39 ms] Exit code 0 from: git rev-parse --abbrev-ref --symbolic @{u}
[        ] origin/beta
[        ] executing: [/Users/kyeonghwan-kong/flutter/] git rev-parse --abbrev-ref HEAD
[  +10 ms] Exit code 0 from: git rev-parse --abbrev-ref HEAD
[        ] beta
[        ] executing: [/Users/kyeonghwan-kong/flutter/] git ls-remote --get-url origin
[  +13 ms] Exit code 0 from: git ls-remote --get-url origin
[        ] https://github.com/flutter/flutter.git
[        ] executing: [/Users/kyeonghwan-kong/flutter/] git log -n 1 --pretty=format:%H
[  +23 ms] Exit code 0 from: git log -n 1 --pretty=format:%H
[        ] 5ab9e70727d858def3a586db7fb98ee580352957
[        ] executing: [/Users/kyeonghwan-kong/flutter/] git log -n 1 --pretty=format:%ar
[  +11 ms] Exit code 0 from: git log -n 1 --pretty=format:%ar
[        ] 4 weeks ago
[        ] executing: [/Users/kyeonghwan-kong/flutter/] git describe --match v*.*.* --first-parent --long --tags
[  +15 ms] Exit code 0 from: git describe --match v*.*.* --first-parent --long --tags
[        ] v0.8.2-0-g5ab9e7072
[ +330 ms] executing: /Users/kyeonghwan-kong/Library/Android/sdk/platform-tools/adb devices -l
[   +8 ms] Exit code 0 from: /Users/kyeonghwan-kong/Library/Android/sdk/platform-tools/adb devices -l
[        ] List of devices attached
           ce0417146180ac1702     device usb:338755584X product:dream2lteks model:SM_G955N device:dream2lteks transport_id:1
           emulator-5554          device product:sdk_gphone_x86 model:Android_SDK_built_for_x86 device:generic_x86 transport_id:2
[  +14 ms] executing: idevice_id -h
[  +42 ms] /usr/bin/xcrun simctl list --json devices
[ +203 ms] More than one device connected; please specify a device with the '-d <deviceId>' flag, or use '-d all' to act on all devices.
[   +5 ms] /Users/kyeonghwan-kong/Library/Android/sdk/platform-tools/adb -s ce0417146180ac1702 shell getprop
[ +119 ms] ro.hardware = samsungexynos8895
[   +2 ms] /Users/kyeonghwan-kong/Library/Android/sdk/platform-tools/adb -s emulator-5554 shell getprop
[  +27 ms] ro.hardware = ranchu
[   +3 ms] SM G955N                  • ce0417146180ac1702                   • android-arm64 • Android 8.0.0 (API 26)
[        ] Android SDK built for x86 • emulator-5554                        • android-x86   • Android 8.1.0 (API 27) (emulator)
[        ] iPhone XS Max             • FF00A34F-7D49-4625-BEBA-6B4C97A8CEED • ios           • iOS 12.0 (simulator)
[  +19 ms] "flutter run" took 699ms.

#0      throwToolExit (package:flutter_tools/src/base/common.dart:26:3)
#1      RunCommand.validateCommand (package:flutter_tools/src/commands/run.dart:239:7)
<asynchronous suspension>
#2      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:350:11)
<asynchronous suspension>
#3      FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:306:33)
<asynchronous suspension>
#4      AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#5      _rootRun (dart:async/zone.dart:1124:13)
#6      _CustomZone.run (dart:async/zone.dart:1021:19)
#7      _runZoned (dart:async/zone.dart:1516:10)
#8      runZoned (dart:async/zone.dart:1463:12)
#9      AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#10     FlutterCommand.run (package:flutter_tools/src/runner/flutter_command.dart:297:20)
#11     CommandRunner.runCommand (package:args/command_runner.dart:194:27)
<asynchronous suspension>
#12     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:347:21)
<asynchronous suspension>
#13     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#14     _rootRun (dart:async/zone.dart:1124:13)
#15     _CustomZone.run (dart:async/zone.dart:1021:19)
#16     _runZoned (dart:async/zone.dart:1516:10)
#17     runZoned (dart:async/zone.dart:1463:12)
#18     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#19     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:303:19)
<asynchronous suspension>
#20     CommandRunner.run.<anonymous closure> (package:args/command_runner.dart:109:29)
#21     new Future.sync (dart:async/future.dart:224:31)
#22     CommandRunner.run (package:args/command_runner.dart:109:11)
#23     FlutterCommandRunner.run (package:flutter_tools/src/runner/flutter_command_runner.dart:211:18)
#24     run.<anonymous closure> (package:flutter_tools/runner.dart:59:20)
<asynchronous suspension>
#25     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:142:29)
<asynchronous suspension>
#26     _rootRun (dart:async/zone.dart:1124:13)
#27     _CustomZone.run (dart:async/zone.dart:1021:19)
#28     _runZoned (dart:async/zone.dart:1516:10)
#29     runZoned (dart:async/zone.dart:1463:12)
#30     AppContext.run (package:flutter_tools/src/base/context.dart:141:18)
<asynchronous suspension>
#31     runInContext (package:flutter_tools/src/context_runner.dart:42:24)
<asynchronous suspension>
#32     run (package:flutter_tools/runner.dart:50:10)
#33     main (package:flutter_tools/executable.dart:52:9)
<asynchronous suspension>
#34     main (file:///b/build/slave/Mac_Flutter_Packaging/build/archive/flutter/packages/flutter_tools/bin/flutter_tools.dart:8:3)
#35     _startIsolate.<anonymous closure> (dart:isolate/runtime/libisolate_patch.dart:287:32)
#36     _RawReceivePortImpl._handleMessage (dart:isolate/runtime/libisolate_patch.dart:171:12)

Analyzing sample_flutter_chat_app...

   info • Unused import: 'package:flutter/material.dart' • test/widget_test.dart:7:8 • unused_import
   info • Unused import: 'package:sample_flutter_chat_app/main.dart' • test/widget_test.dart:10:8 • unused_import

2 issues found. (ran in 2.6s)

[✓] Flutter (Channel beta, v0.8.2, on Mac OS X 10.13.6 17G65, locale ko-KR)
    • Flutter version 0.8.2 at /Users/kyeonghwan-kong/flutter
    • Framework revision 5ab9e70727 (4 weeks ago), 2018-09-07 12:33:05 -0700
    • Engine revision 58a1894a1c
    • Dart version 2.1.0-dev.3.1.flutter-760a9690c2

[!] Android toolchain - develop for Android devices (Android SDK 28.0.2)
    • Android SDK at /Users/kyeonghwan-kong/Library/Android/sdk
    • Android NDK at /Users/kyeonghwan-kong/Library/Android/sdk/ndk-bundle
    • Platform android-28, build-tools 28.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)
    ! Some Android licenses not accepted.  To resolve this, run: flutter doctor --android-licenses

[✓] iOS toolchain - develop for iOS devices (Xcode 10.0)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 10.0, Build version 10A255
    • ios-deploy 2.0.0
    • CocoaPods version 1.5.3

[✓] Android Studio (version 3.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin version 29.0.2
    • Dart plugin version 181.5616
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1136-b06)

[!] IntelliJ IDEA Ultimate Edition (version 2018.1.6)
    • IntelliJ at /Applications/IntelliJ IDEA.app
    ✗ Flutter plugin not installed; this adds Flutter specific functionality.
    ✗ Dart plugin not installed; this adds Dart specific functionality.
    • For information about installing plugins, see
      https://flutter.io/intellij-setup/#installing-the-plugins

[✓] Connected devices (3 available)
    • SM G955N                  • ce0417146180ac1702                   • android-arm64 • Android 8.0.0 (API 26)
    • Android SDK built for x86 • emulator-5554                        • android-x86   • Android 8.1.0 (API 27) (emulator)
    • iPhone XS Max             • FF00A34F-7D49-4625-BEBA-6B4C97A8CEED • ios           • iOS 12.0 (simulator)

! Doctor found issues in 2 categories.

😭

SelfAnswer
The bundle ID in Xcode has confirmed that is a capital letter.
When registering an iOS project in Firebase, I replaced all uppercase letters in the bundle ID with all lowercase letters and registered them with firebase.
and solved it.


Add Attenuation Model with not attenuation

The Idea behind this is that sometimes only the direction from which the sound comes from is important in which case it can be nice if the attenuation can be rules out of the equation.
This can be useful to make a sound source that move and where the developer chooses how attenuation works.
For the moment the only way to deal with this is to do code that goes against the already existing one which is quiet a waste of time (performance + developer time)
I'll be very happy to do a pull request but I first want to make sure this is a wanted feature or at least one that will be pulled in.

For the moment the way I have found to deal with this problem without having a custom engine (which I don't want to and is why I am submitting this issue) is to use the following line of codes.
var distance = $AudioStreamPlayer3D.global_transform.origin.distance_to(player.global_transform.origin)
$AudioStreamPlayer3D.unit_db = INITIAL_DB - linear2db(1 / (distance / $AudioStreamPlayer3D.unit_size))

which is exactly the opposite of what https://github.com/godotengine/godot/blob/329904598829dcbed462b012dee39bc37d7d7c83/scene/3d/audio_stream_player_3d.cpp#L217 is doing.
Another reason which I didn't mention before for why I would be interesting to have this is that it would allow to have the sound attenuation be against another object instead of the actual AudioStreamPlayer3D.

#26868 Was submitted to fix this problem.

Fixed by #26868.


Tax adding crazyness

This is more like a feature request or a wish:
Right now adding taxes can be a very time-consuming thing. Like if the shop is sitting inside the EU, the rules are:
for ALL inside EU countries excluding your own: apply VAT based on billing address, allow deactivation of VAT of number included
for your own country inside EU: apply VAT based on billing address, no deactivation possible
for all other countries in the world: apply VAT based on billing address, no deactivation possible
most EU countries have at least 2 tax rates, so this means setting up 2 tax rates for every possible customer country in the world... urghs.
Could there be an easier way? Like a prepared list template or something?

That sounds horrible. Move to a different country perhaps?  :-)
Could you envisage a user interface that would solve this problem, some sort of grid entry perhaps?

First thing i taught about was a db-table that could be imported. But the flat shipping rate interface (CSV-import) would work better for most people.
But of course, a interface-thingy would be great.
So to keep it EU-wise simple:

Checkbox 'Is your shop inside the EU'
Select 'Choose your home country'
Define tax rates / classes

Boooom magic:
Every country in the world outside the EU has magically applied the VAT and no chance for tax exemption - every country inside the EU has applied VAT with the possibility of tax exemption but your own cannot.
rule-wise it looks pretty simple :)

Oh, one more rule missing: There is one special class of products (digital goods) where another rule applies: The tax rate depends on the location of the recipient.
So if I am selling FROM italy (20% tax) TO Austria (22%), and the product is a "digital good" it will be charged with 22% tax.
This is called 'reverse-charge' and is only in law since 2015.

But I'd say - setting up such little 'special rules' rules that apply only to the EU countries is easy to be done by hand. When all the other stuff ('world') is already  defined.

Look:
https://github.com/yireo/Magento_EU_Tax_Rates

not sure if you've seen this https://github.com/concrete5-community-store/community_store/issues/108

There are a lot of libraries dealing with this,  here's one I've used before https://github.com/dannyvankooten/vat.php

Crazy, he read my thoughts before I had them :-)
Thank you, that's exactly the right track!

@cahueya I'm having the same issue..
ecommerce in Italy and sell to Italy and EU.
Did you have integrated the library (https://github.com/ibericode/vat) in some way?
at now, as per CommunityStore works, if some one enter a VAT ID, then the Tax has not been applied in any case but it's wrong.. the TAX should not be applied only if the entered VAT ID is different from the country of ecommerce eg. I'm in Italy:

If I sell to Italian customer, I have to apply the TAX
if I sell to France, Germany, EU State, I don't have to apply TAX
if I sell to state outside EU (Switzerland, GB, ...) I've to apply a different TAX


Add support for RWops versions of functions

Introduces support for using RWops (or, more specifically, cen::file wrapper of this library) for file-related operations in SDL and its libraries. Main usecase is cooperation with physfsrwops wrapper from physfs project (cen::file is used to wrap RWops objects created by it).
All functions of RWops family have a parameter that allows to close file upon completion. I've decided not to set or expose it as cen::file takes care to close handle on destruction anyways.

Feel free to drop the second commit if you think it's not a good idea.

I think the code looks fine, but I'm not sure about passing the files as const&. I assume that the files are observably modified when parsed (is this true?), in which case I don't think the extra brevity is worth evading const correctness.

I think the optimal solution is to rebase this on top of #130 and make two versions of each function: one accepting file& and another accepting file&&, with latter stealing ownership and utilizing auto-closing argument. This will allow to both keep const-correctness and pass temporaries.

Alright, so I've dropped commit that added const and instead used file.release() from #130 (which the second comment requires) to implement rvalue versions of functions, allowing to pass in temporaries (or std::moved files, of course). These versions behave like you would expect from rvalue-using functions - take ownership of file handle (via file.release) and let function they call close it. IMO this is the optimal solution.

There's one thing I'd like to discuss before getting this merged: there are currently assert(file.is_ok()) guards in added functions to mirror checks that string versions do. However, SDL and derivative libraries already do these checks themselves, which set an error - which centurion promptly turns into exception. As such, I'm not sure if those asserts should be there, as otherwise user can handle such error condition more gracefully. Would like to hear your input.


change log level for tests in ci

change log level to debug in tests

Is this still an issue? @almogdepaz

I don't think this is an issue, this can be enabled on a per-test basis or when debugging as necessary. having debug on for all tests all of the time would be extraordinarily verbose.


Bug: Default configuration on install ipk is invalid

Probleme
When I install the last ipk 4.1.3 the default file of configuration have an field with wrong key.
Default configuration is :
[backend]
  enabled="semtech_udp"

  [backend.semtech_udp]
    udp_bind="<IP_ADDRESS>:1700"

and doest work.
But in documentation the exemple is
[backend]
  enabled="semtech_udp"

  [backend.semtech_udp]
    bind="<IP_ADDRESS>:1700"

Fix
Change at line 12 in file chirpstack-mqtt-forwarder.toml udp_bind="<IP_ADDRESS>:1700" to bind="<IP_ADDRESS>:1700"

Yes, you are right, this should have been bind instead of udp_bind:
https://github.com/chirpstack/chirpstack-mqtt-forwarder/blob/master/src/config.rs#L141


add(README.md): Documentation for the env-injector snapcraft extension

At https://github.com/locnnil/env-inject-doc/pull/1/commits/55a87ea47cc8f9db8d7720bb039c4f80b397b44a I've just reordered the tables so the syntax for the snap options and the env file stay together and then the table with the allowed characters show up.
Also, I've added a tiny introduction sentence here because an H2 title followed directly by a H3 title without nothing in the middle seems ugly.


static vhost

Hi, I liked marathon-lb - is it realy cool tool. I used to consul+mesos-consul-bridge+consul-template+lighttpd, but now migrate our cluster to marathon-lb.
I have question: is possible create static vhost - means. vhost which doesn't in marathon (aka marathon.my_cloud -> my_cloud:8080 for example)?
thanks

My idea is on each master run mesos-master, marathon and marathon-lb.
I do template templates/HAPROXY_HTTP_FRONTEND_HEAD:
backend marathon_8080
  balance roundrobin
  mode http
  option forwardfor
  http-request set-header X-Forwarded-Port %[dst_port]
  http-request add-header X-Forwarded-Proto https if { ssl_fc }
  server local localhost:8080

backend mesos_5050
  balance roundrobin
  mode http
  option forwardfor
  http-request set-header X-Forwarded-Port %[dst_port]
  http-request add-header X-Forwarded-Proto https if { ssl_fc }
  server local localhost:5050

frontend marathon_http_in
  bind *:80
  mode http

  acl marathon hdr(host) -i marathon.my_cloud
  use_backend marathon_8080 if marathon
  acl mesos hdr(host) -i mesos.my_cloud
  use_backend mesos_5050 if mesos

and then
docker run --rm -v $PWD/templates:/marathon-lb/templates -e PORTS=9090 --net=host --privileged=true mesosphere/marathon-lb:v1.2.2 sse --marathon http://localhost:8080 --group internal
this is work... exists some better way?
thanks

Yep, you can do that. This isn't really how MLB was meant to be used, but it's possible. As an alternative, I'd suggest you just run a second instance of HAProxy or nginx on Marathon pointing to your backend, and configure that the normal way through MLB.


Add QTD app

Description
This PR adds a apps subdirectory where we can store examples of apps built with superduperdb.
Related Issues

#653
Checklist
N/A
Additional Notes or Comments

Codecov Report
Patch coverage has no change and project coverage change: -0.03% :warning:

Comparison is base (c64259e) 79.65% compared to head (46504de) 79.63%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #686      +/-   ##
==========================================
- Coverage   79.65%   79.63%   -0.03%
==========================================
  Files          79       79
  Lines        4950     4944       -6
==========================================
- Hits         3943     3937       -6
  Misses       1007     1007

see 6 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


New Features

[ ] Support multiple files and show duplication on both
[ ] Show some visualizations on the result


Apache 403 page instead of custom error page

First I want to say thank you for Grav, because it is an amazing CMS! But I am getting the ugly default Apache 403 page if I try to access a forbidden directory (for example /backup, /vendor, ...):

My Apache 000-default.conf file looks like:
<VirtualHost *:80>
	<Directory /var/www/html>
        	Options Indexes FollowSymLinks MultiViews
                AllowOverride All
                Order allow,deny
                allow from all
	</Directory>
	ServerAdmin webmaster@localhost
	DocumentRoot /var/www/html
	ErrorLog ${APACHE_LOG_DIR}/error.log
	CustomLog ${APACHE_LOG_DIR}/access.log combined
</VirtualHost>

I tried several config changes (e.g. ErrorDocument), but only the default Apache error page occures for the 403 error. If I remove the [F] flag from the RewriteRule at the .htaccess file (e.g. line 47 and 57) it works for files (LICENSE.txt, composer.lock, composer.json) but not for folders (cache, bin, logs, backup, webserver-configs) because of an ERR_TOO_MANY_REDIRECTS error. I would like to return the 403 Forbidden header and show the content of the root homepage or error page.
Thanks!

Well, this is because the .htaccess is specifically blocking this, hence the Apache style error message. Maybe this will help?
http://stackoverflow.com/questions/30983968/simple-way-to-redirect-403-and-404-errors-to-the-main-page-htaccess

If I add ErrorDocument 403 / to the .htaccess file I got a ERR_TOO_MANY_REDIRECTS error. The best way would be a redirect to a custom error page (e.g. /error/forbidden). Any ideas how can I achieve this? Thank you.

ErrorDocument 403 http://yoursite.com/error/forbidden
should work

Yeah it does, thanks. But is there a way to do this via relative path?

This maybe possible, but i'm afraid I don't know off hand.  At this point, it's purely an Apache thing, and not a Grav thing. So closing.


How to use it in WEBUI or COMFYUI?

I used it as a normal 1.5LORA with SD1.5 to generate pictures, in COMFYUI，but it didn't have the advertised effect

It's not a LoRa; it requires custom nodes. Try this plugin.

https://github.com/ExponentialML/ComfyUI_ELLA/

It's not a LoRA; it requires custom nodes. Try this plugin.

THX~

https://github.com/ExponentialML/ComfyUI_ELLA/

THx~


THRIFT-4889: Nonblocking server with sasl support

This PR is already merged to the upstream, we just need to backport it.


New Feature: Location-based Wikidata tagging of uploaded photos

The idea would be to use the location data gathered from a photo to suggest nearby Wikidata items that could be tagged, to enable greater reuse of commons uploads.

Thank you for your suggestion. I think similar ideas are also addressed here: https://github.com/commons-app/apps-android-commons/issues/259

Yes, apologies for not finding that one, please mark this as a duplicate.

Don't worry about it. It would be great if you could comment on the existing issue how you would expect the feature to work.

Hi @kalpaitch ,
Thanks for posting here! Sorry, I had forgotten about #259 when I was discussing this feature with you and John.
We could definitely use help with implementing this feature anyhow. :)


During watch one more new service worker is being added each time after save

If was run brunch watch and e.g. index.html is changed 3 times, then service worker will be included 3 times.
....

Solved by #9


Update to AtomsBase 0.5

Requires EmpiricalPotentials to update to AB 0.5

Circular dependency in the docs between DFTK and GeometryOptimzation. So I will merge this and release despite failing tests to make sure I can get the AB0.5-compatible DFTK version released. With the next patch of this the docs can build properly again.


okta-aws passing profile name twice to aws CLI

Describe the bug
On MacOS (and probably other linux), it seems that withokta isn't stripping the profile name from the arg list when it calls withokta using the following alias
function okta-aws {
    withokta "aws --profile $1" "$@"
}

All we get is a generic error from aws CLI complaining about bad arguments.
If the function is changed to
function okta-aws {
     withokta "aws --profile" "$@"
}

it seems to work fine, although I'm not sure that's the correct solution.
To Reproduce
Steps to reproduce the behavior:

Run okta-aws test sts get-caller-identity
See error

Expected behavior
Get logged into the profile and get your caller identity.

Pretty sure this is related to https://github.com/oktadev/okta-aws-cli-assume-role/issues/376, I think the fix is straightforward (see my comment).

@hcourse-nydig yes, it appears to be the same issue (although the fix is slightly different for bash).

If you've got time, could you double check the PR? This seems like a weirdly fundamental issue to still exist so I wonder if we're missing something? Anyway, should do the job!


Info Windows & Map Events

So before I implemented clusters I had my Markers open infowindows and on marker click triggered events with the markerSelect and markerInfoWindowTapped directives of the MapView. (I'm using NativeScript with Angular 2.x)
Anyway, after clustering, neither the info windows nor the MapView directives work/trigger when I click on a marker. For "shits & giggles" I modified the index.android.td adding the following line mapView.notifyMarkerEvent("markerSelect", clusterManager); in the function setupMarkerCluster().
With that the markerSelect directive will trigger, but with no marker to be found... 😩
Any thoughts or insight would be helpful here, thank you!

@MichaelSolati well .. this one wont be simple to handle. the android google sdk recreates the markers dynamically thus the reference link in JS becomes obsolete.  will see...

I think i got the same issue, I'm using Angular too and a marker added with mapView.addMarker(marker); the funcion: (markerSelect)="onMarkerSelect($event)" works fine. but when you click on a marker added by setupMarkerCluster()  the function "onMarkerSelect" is triggered but the value of "marker" is undefined.

yes, I've attempted to reimplement the marker click listener but that didn't work .. will see

I am experiencing the same thing with the demo folder in the repo.  I can click on the Tunis marker with no problem but when I click the other ones that are created I receive an error.

@MichaelSolati @xyz765 @randy-johnson check latest version, should be fixed

You're the real MVP @naderio !!!! I'll have to update my projects over the weekend.


Table component

The first release of the table component takes inspiration from a lot of the community discussion, other notable design systems and previous work last year. We decided for the first release we would ship a vertical table layout e.g:

heading 1
heading 2

row 1, col 1, cell 1
row 1, col 2, cell 2

row 2, col 1, cell 3
row 2, col 2, cell 4

row 3, col 1, cell 5
row 3, col 2, cell 6

Community discussion: https://community.digital.gov.au/t/table/92/20
Community credits / thanks:
@dkeeghan
@TrebBrennan for https://codepen.io/AusDesignSystem/pen/PeGwjr
@steveworley + @tobybellwood for #202
@alex-page
@AndrewArch
@Chris820
@rooby
Related issues:
#393
#202
#8
#552

thanks @maxious have made some updates accordingly. Now the order of the row data can be in any order and have removed the headerContentTypes prop


Turn enforce into an eponymous template + undocument enforceEx

Resubmission of the reverted https://github.com/dlang/phobos/pull/6086.
We need to figure out why this broke the DMD testsuite on SemaphoreCi and CircleCi.
test_results/d_do_test: symbol lookup error: test_results/d_do_test: undefined symbol: _D3std9exception__T7enforceZ__TQmTbZQrFNaNfbLAxaAyamZb
Makefile:209: recipe for target 'test_results/runnable/helloUTF16BE.d.out' failed

See also:

https://github.com/dlang/phobos/pull/6086 (original PR)
https://github.com/dlang/phobos/pull/60130 (revert)

The SemaphoreCI problem has been found and fixed (see https://github.com/dlang/dmd/commit/1e81c60cde88aec09704ebc9f89b774d5bd1df17).
So we can put this back in again :)
@CyberShadow around for a quick merge?

Jenkins didn't send a proper success code, but it passed:

Everything else is passing too:

So I'm merging this manually.


added option to use the cass name lowercased as a salt by specifying …

…_fixed_filename_salt=True in the class declaration. Default is current behaviour

accidental whitespace...


Dictionary creation is successful when distinct count of this column above high.cardinality.threshold

when we config high.cardinality.threshold=10000 in carbon.properties, it doesn't work.
in this pr, i have fixed this issue

LGTM


Ensure artifacts are pushed to https://igniterealtime.org/archiva

Currently, the artifacts are only kept (for a short time) on Gitlab. They should be uploaded to the Ignite Realtime Maven repo.

All of the automation around pushing artifacts into Archiva is depends on Travis. Every other plugin uses a set of customized maven commands. For reference, have a look at the Travis configuration file for the Monitoring plugin; https://github.com/igniterealtime/openfire-monitoring-plugin/blob/master/.travis.yml
Every plugin uses that exact same configuration (with the notable exception of the encrypted credentials being different.
This plugin uses Gradle, which makes re-using this functionality difficult. Although Gradle will have lots of benefits, the occasional "we must update the Travis config for every plugins" already is a cumbersome task even with every plugin using the exact same config. I dread having to support different variants.
The obvious way to prevent issues is to migrate this project to Maven, but maybe there are alternatives?

Understood! I'll experiment (possibly not soon) with how to do it with Gradle unless something changes before then.

I didn't attempt to migrate this plugin to our standardized github workflow for openfire plugins.

This /possibly/ now just needs the secrets adding to the repo. I don't know what they are though :(

The organizational secrets should already be visible to this repo, I am not seeing any errors denoted with the publish step?

The was a difference with this repo for the Openfire Developers group not having admin privs, I have updated that now.

Ah, great. Thanks, I'll see if I can trigger a build and figure out where it's all going wrong :)

That seems to have done it - thanks!
https://igniterealtime.org/archiva/repository/maven/org/igniterealtime/openfire/plugins/PasswordReset/0.0.1-SNAPSHOT/

(it's possible I've got the format wrong, mind you. Things may well have changed since this repo saw any activity!)

Ah, seems to be OK
https://www.igniterealtime.org/projects/openfire/plugin-archive.jsp?plugin=PasswordReset

But the plugin doesn't work with the latest+greatest Openfire :(


Navigation arrows hover state not reverting until click outside

The hover state of the navigation arrows works as expected until you click one of them. The arrow then gets stuck on its hover state until you click somewhere else on the screen. This is barely noticable until you have a design with big arrows. Is there a way to get the hover state to revert when moving the pointer of the arrow even if you have clicked on it?

can you not add the style button.slick-next:focus to the same css as default?

ya @josefulander i think you mean the default arrows (they are buttons) have a :focus effect you don't desire (this is accessibility feature) ... as @saunders1989 says; you can change the CSS to suit your application.
Regards, Simon


Binary 0.5.0 on arm-unknown (RPi 0) appear broken on RPi0W

I just installed the binary https://github.com/mfreeborn/heliocron/releases/download/v0.5.0/heliocron-v0.5.0-arm-unknown-linux-gnueabi.tar.gz onto a Raspberry Pi 0 WiFi running Raspbian 10 (buster).
When I call heliocron, it always estimates sunrise and sunset to be at 00:00:00 local time (and hence the day is pretty short).
So something isn't working in that release.
Here is a sample output from the RPi0w (from README.md and works fine on another Linux box I tested on):
user@rpi0w:~/heliocron-v0.5.0-arm-unknown-linux-gnueabi $ ./heliocron -d "7 May 2065" -f "%e %B %Y" -l 55.9533N -o 3.1883W report
LOCATION
--------
Latitude: 55.9533N
Longitude: 3.1883W

DATE
----
2065-05-07 12:00:00 +01:00

Solar noon is at:         2065-05-07 00:00:00 +01:00
The day length is:        0h 0m 0s

Sunrise is at:            2065-05-07 00:00:00 +01:00
Sunset is at:             2065-05-07 00:00:00 +01:00

Civil dawn is at:         2065-05-07 00:00:00 +01:00
Civil dusk is at:         2065-05-07 00:00:00 +01:00

Nautical dawn is at:      2065-05-07 00:00:00 +01:00
Nautical dusk is at:      2065-05-07 00:00:00 +01:00

Astronomical dawn is at:  2065-05-07 00:00:00 +01:00
Astronomical dusk is at:  2065-05-07 00:00:00 +01:00

I assume something doesn't quite build right on this platform, although I haven't tried to rebuild from source.

Exact same thing on my side. Thanks for the help in advance, @mfreeborn

Thank you both for flagging this issue - I've managed to take a look at this at last. For reasons that I don't entirely understand, it should all be working now that I've recompiled the RPi 0 binary using a the arm-unknown-linux-gnueabihf target triple rather than arm-unknown-linux-gnueabi. I can only seem to find conflicting information regarding which one is correct and which one works at all when compiling for the RPi 0. In this case, both seem to run, but only the hf-variant is correct.
It works on my end, but it would be reassuring if either of you could retry with the new binary on the release page (still v0.5.0).

No further replies received


Use otel collector

Prepare to use the otel collector rather than having each language instrumentation post directly to ingest.  This should reduce latency for customers' apps/functions.
The splunk-otel-lambda side of this work involves modifying the scripts to unset SPLUNK_REALM and friends so they fall back to the otel default behavior of talking OTLP to localhost.  Along the way, I noticed and fixed a problem where we had forked some upstream otel-lambda wrapper scripts rather than simply applying small edits to them during our build process.
NB: based on past experience, this process will likely take a few cycles of CI to iron out tiny script problems.

@johnbley could you give some more explanation of what this change is doing? Are the steps to set  SPLUNK_REALM and SPLUNK_ACCESS_TOKEN in the configuration section of the readme still relevant? It seems like with this change the splunk-default-config script immediately unset's these, so I'm unclear why we'd set those still. How should the token be configured after this change?

@dude0001 they are used by the collector that is included in the layer, instead of by the individual SDKs, which is why its unset in the script.

Thank you @tsloughter for the further clarification. Is there a complete example of how to consume this layer after this change? I'm trying to upgrade to latest to get some other fixes. However, on latest I'm seeing 401 Unuauthorized errors assumingly because the SPLUNK_ACCESS_TOKEN I was setting up in an environment variable is getting unset now. I'm unclear how our Lambda should be authenticating now when ingesting traces directly into Splunk e.g. https://ingest.us1.signalfx.com:443/v2/trace/otlp.

Just noticed my response yesterday didn't send somehow, sorry about that.
@dude0001 what version of the layers are you using?
Are you setting any env variables besides realm and access token?

I had to step away from this for a bit, but looking at it again @tsloughter. I am using arn:aws:lambda:us-east-2:254067382080:layer:splunk-apm:365. I am moving this conversation to an issue and opened #129. We are trying to keep the token secret, and this change broke our mechanism for this. Looking for a suggestion on how to accomplish what we need or to add a mechanism to the Splunk Lambda wrapper extension to inject the token as a secret.


Fix compile errors

This include paths is probably from the author's local machine:
https://github.com/hanickadot/co_curl/blob/375c93aa156b2f51b42500a00394a1bdcade95cc/examples/return.cpp#L1
so this examle fails to compile
Also my version of compiler (gcc 11.4) is not happy with the following warning treated by default as error:
[..]/co_curl/include/co_curl/easy.hpp:130:33: error: declaration of ‘fnc_t& f’ shadows a parameter [-Werror=shadow]
  130 |                         fnc_t & f = *static_cast<fnc_t *>(udata);
[...]
[..]co_curl/include/co_curl/easy.hpp:180:33: error: declaration of ‘fnc_t& f’ shadows a parameter [-Werror=shadow]
  180 |                         fnc_t & f = *static_cast<fnc_t *>(udata);

Thanks for reporting this issue.


⚠ Quirk Club Argentina (API endpoint) has degraded performance

In fbe4657, Quirk Club Argentina (API endpoint) (https://us-central1-quirkclub-dev.cloudfunctions.net/api/check/api) experienced degraded performance:

HTTP code: 200
Response time: 8218 ms

Resolved: Quirk Club Argentina (API endpoint) performance has improved in 593d1dd.


Update pre-cache docs to mention filtering and space checking

/lgtm

/retest

/cherrypick release-4.13


Funky fresh: Additional adjustments to repo setup for fresh machines

I went through the local-setup installation process on a fresh machine, which found a few more gaps I'd missed in our scripts:

A few stray references to truffle instead of npx truffle.
Some unused dependencies.
A reliance on coreutils.

Those changes are reflected here.

Not sure why I just got a lint failure on OldTokenStaking since I didn't touch it :sob:

The lint failure is unrelated, so I'm going to roll this in.


Fix for #516 - Slashes in BkBasic

Added slashes fix + junit test for this check.
Maybe it will be nice to expose header name for future reuse.

@alexey-krylov Many thanks, I will find someone to review it before we merge

@dmzaytsev review this one, please

@alexey-krylov you have checkstyle issues, see AppVeyor's or Travis's log
let's fix it

@alexey-krylov according the quality rules http://at.teamed.io/qa.html

Pull request description explains the solution proposed and contains a link to the original ticket it is related to.

please update the pull request description

Resolving #516.
Code reformatted according to teamed.io rules.

@dmzaytsev please check - codestyle is fixed now. I have to suppress ExcessiveImports on BkBasicTest because it is initially overloaded. Also found that PMD suppression is a usual practice in 'takes' junits.
As an option - move headers check to separate test case.

@alexey-krylov the original ticket mentioned UnknownHostException on getLocalAddress and getRemoteAddress calls
However I hadn't saw these calls in your test
You have to reproduce the issue in a test first
and then fix it

@alexey-krylov

Messages in a ticket always start with a name of a user they are addressed to.
see http://at.teamed.io/qa.html

please fix your comment

alexey-krylov commented 8 hours ago
Resolving #516.
Code reformatted according to teamed.io rules.

@dmzaytsev is it possible to find puzzle invalid format errors before commit to Travis? I run 'mvn clean install -Pqulice' and it works fine for code that i modified.

@alexey-krylov yep, this line checks for correct puzzle
see also https://github.com/teamed/pdd how to install pdd locally

@dmzaytsev all checks have passed :) Please check.
Also i added address-related check.

@alexey-krylov please see a few comments above

@dmzaytsev this spaces added by supposed codestyle from @yegor256 and caused by limited to 80 chars maximum line lenght (because 'for' statement is too long for 80 chars). Also this style did not cause warnings in Qulice.
I dont think that manual code reformatting is a good and productive way to create clean code.
I will be very appreciate to you if your share IDE config with this check.
Regarding to another comments i will fix them.

@alexey-krylov okay let's keep it as is
I use same config as Yegor
and usually the indentation step is 4 spaces

@dmzaytsev test rewritten by your comments. pls check

@dmzaytsev done

@alexey-krylov please see a few cosmetic commantrs above and we are ready to merge

@dmzaytsev done with spaces.

@alexey-krylov thank you!

@rultor merge

@rultor merge

@dmzaytsev Thanks for your request. @yegor256 Please confirm this.

@rultor merge

@rultor merge

@yegor256 OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@yegor256 Done! FYI, the full log is here (took me 16min)

@ypshenychka please, check this issue for QA compliance, as per par.24

@rultor please deploy

@rultor please deploy

@davvd OK, I'll try to deploy now. You can check the progress here

@rultor please deploy

@davvd Done! FYI, the full log is here (took me 11min)

@dmzaytsev

dmzaytsev added a note 2 days ago
and here

According to our QA Rules:

Messages in a ticket always start with a name of a user they are addressed to.

Please correct your code review comment by indicating an addressee in the beginning.

@ypshenychka fixed, thanks

@dmzaytsev Thank you!

@davvd Quality is acceptable.

@davvd Quality is acceptable.

@ypshenychka thanks for the QA review, we'll work better next time

@dmzaytsev just added 10 mins to @ypshenychka (for QA), payment ID is 75304095
Thank you, I have added 44 mins to your account in payment/transaction "75304098", time consumed: 52 hours and 6 mins
review comments (c=29) added as a bonus
added +44 to your rating, now it is equal to +2486


Please clarify the size of the container filesystem.

Like Windows, it would be nice if we could clarify the size of the container filesystem that varies by SKU in Linux.
I think some users wonder how much size of the filesystem they can utilize.
S1 is 29GB
root@275706637de7:/home# echo $WEBSITE_SKU
Standard

root@275706637de7:/home# lscpu | grep PCU
CPU op-mode(s):      32-bit, 64-bit
CPU(s):              1
On-line CPU(s) list: 0
CPU family:          6
Model name:          Intel(R) Xeon(R) Platinum 8272CL CPU @ 2.60GHz
CPU MHz:             2593.906
NUMA node0 CPU(s):   0

root@275706637de7:/home# df -h
Filesystem                                                                            Size  Used Avail Use% Mounted on
none                                                                                29G   22G  5.0G  82% /

P1V2 is 35 GB
root@734bdb1c1b19:/home# echo $WEBSITE_SKU
PremiumV2

root@734bdb1c1b19:/home# lscpu | grep CPU
CPU op-mode(s):      32-bit, 64-bit
CPU(s):              1
On-line CPU(s) list: 0
CPU family:          6
Model name:          Intel(R) Xeon(R) Platinum 8171M CPU @ 2.60GHz
CPU MHz:             2095.078
NUMA node0 CPU(s):   0

root@734bdb1c1b19:/home# df -h
Filesystem                                                                            Size  Used Avail Use% Mounted on
none                                                                                35G   23G   11G  69% /

In Windows, local disk size is described in the following document.
https://docs.microsoft.com/en-us/azure/app-service/operating-system-functionality#types-of-file-access-granted-to-an-app

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 53dd65b1-6f9d-9431-5850-9cee636aeabd
Version Independent ID: 0012e819-bb2b-2955-d87b-d6f145fb6af7
Content: Overview - Azure App Service
Content Source: articles/app-service/overview.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.

#taken-by:btardif

I don't want to add a disclaimer about this until after ANT99 rolls out later this year.
We are working on changes that will affect how the local disc is used by offloading some of the caching that we do to the /temp drive. This should result in freeing up significant space on the local drive as well as giving a stable free space we can guarantee.
I have tagged a few PR that should help mitigate this until then.
@cephalin let me know if this is enough for now.
FYI --> @RyanHill-MSFT

Internal item created: https://dev.azure.com/msft-skilling/Content/_workitems/edit/288559
#please-close


Entity not available: camera.map

I updated Valetudo to Firmware: 2021.02.0 and the lovelace-valetudo-map-card to v2021-02-20.
I have mqtt discovery enabled and the device was created with all the entities, as it should be. also the new entity is present:

When i add a new card according your expample, it says: Entity not available: camera.map

Before i updated both, i was successfully using this card with the former valetudo firmware.
Thanks.

Can you click the map entity in your first screenshot to see what entity name it actually has?

That is weird, it should work. Are you sure there is no browser cache or anything? Home Assistant likes to cache very very aggressively

ok, wow it was a cache issue. the interesting thing: i tried to refresh with Control + Shift + R which didnt help. after your suggestion i hit F12 > Chrome Developer Tools > Application tab > Clear storage in left tree. Now it works. thank you.

Just stopping by to say thank you - cache reset was my issue as well when I migrated to Valetudo 2021.02.0 and the new map card and then saw the same Entity not available: warning on my iOS app. Refreshing the cache sorted me out.
For anyone else on iOS: Open the App Configuration sidebar section, then Reset frontend cache at the very bottom.

Just want to add that I was successful in getting rid of this error by deleting all topics/data on homeassistant/vacuum, homeassistant/camera, valetudo and robot via the "MQTT Explorer" tool and then disabling and re-enabling the "MQTT Connectivity" in the vacuum. I also tried to delete entities in HA but that wasn't possible for all. It's possible that I had old remaining enitites for whatever reasons (maybe due to updating the vacuum). After fixing the correct value for vacuum (in my case: vacuum: valetudo_imaginaryunwrittenworm), it works again.
Hopefully, this helps someone else too. :-)


Remove resolver codegen

Having runtime and codegen together increases complexity of extension. We seen comments that codegen is not that valuable as:

breaks types
requires formatting
do not allow to edit
it gives no value as it has mostly boilerplate
code is hard to manage

Our runtime layer needs to be improved towards performance and querying capabilities after chatting with community we decided to deprecate graphback-codegen-resolvers
What this means?

We will remove template for codegen
We will improve our runtime layer
Users will need to do manual update of their code in new version

CC @craicoverflow

All work available as part of the feature branch:
https://github.com/aerogear/graphback/compare/crudspec-runtime


Generate separate Prettier and ESLint ignore files

Another proposal about #1414

After https://github.com/foray1010/ignore-sync/pull/301, now we can split the files!

@aomarks do we need something else to merge this?

@aomarks what do you think of this proposal? Do we need something else to merge it?

I think we should fix https://github.com/foray1010/ignore-sync/issues/302, because the semantics are currently wrong. But we should be fine in practice for now, so we can merge.


HTTP Proxy support for Pushy

ping @maleck13 @ziccardi

Wouldn't be better to support both socks and HTTP proxy?

LGTM

Verified push and Nagios dashboard on osm3 with @luigizuccarelli

@ziccardi @PhilipGough @maleck13 why was this not submitted to upstream ?

For the record, code that also is relevant for the community, like this one, always has to be merged against upstream.
We will do a rebase afterwards to get the Java fix to here


Include summary in help output for individual commands if we have it

Sometimes the "summary" is all the help that is required for a command and adding more in "Help" would redundant.
Also, sometimes "summary" serves as a useful introduction to a longer  "Help" document.
So when people ask for the detailed "help" for a command, include the "Summary" as well as the full help.
It's the same value as including a title when someone asks for a document.

Nice! Merging this one in. Thanks!