Add a function to perform case-insensitive search in mapstr.M

What does this PR do?
FindFold (similar to strings.EqualFold) traverses the map and tries to perform a case-insensitive match of each key segment on each map level.

Why is it important?

It's a prerequisite for https://github.com/elastic/beats/pull/41424
Checklist

[x] My code follows the style guidelines of this project
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have added tests that prove my fix is effective or that my feature works

Related issues

Relates https://github.com/elastic/beats/pull/41424
Relates https://github.com/elastic/beats/issues/22254

:green_heart: Build Succeeded

Buildkite Build
Commit: 8448d19b82d883e8b03c2fbfde83118ea975c5d2

cc @rdner

:green_heart: Build Succeeded

Buildkite Build
Commit: 48bc6f532b6152c5f2c09e0a72792c9b049ecbe6

History

:green_heart: Build #443 succeeded 8448d19b82d883e8b03c2fbfde83118ea975c5d2

cc @rdner


Post v0.14 release, point pkg to master.

Proposed Changes

Post v0.14 release.
Point knative.dev/pkg to branch master.

Release Note
NONE

/cc mattmoor

/lgtm
/approve


Workflows UI step links bug

Summary
When executing a workflow that has conditional steps/branches therein, the UI misrepresents the link between steps in certain cases.

Diagnostics

Running Arog 2.11.0

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

Missing reproduction steps. Please add them.


Fix: change scikit-image to v0.17.2 for Python v3.6 compatibility

For your review, please.

LGTM


update dependencies & build resources

Thanks for the PR. I prefer to handle dependencies internally, but I will update as part of the next release.


feat: LSDV-3025-3: Integrate filter component into outliner

PR fulfills these requirements

[x] Commit message(s) and PR title follows the format [fix|feat|ci|chore|doc]: TICKET-ID: Short description of change made ex. fix: DEV-XXXX: Removed inconsistent code usage causing intermittent errors
[ ] Tests for the changes have been added/updated (for bug fixes/features)
[ ] Docs have been added/updated (for bug fixes/features)
[x] Best efforts were made to ensure docs/code are concise and coherent (checked for spelling/grammatical errors, commented out code, debug logs etc.)
[x] Self-reviewed and ran all changes on a local instance (for bug fixes/features)

Change has impacts in these area(s)
(check all that apply)

[x] Product design
[x] Frontend

What is the new behavior?
When filter is applied on the outliner, the filtered result is reflected to the list
Does this change affect performance?
no
Does this change affect security?
no
What alternative approaches were there?
na
What feature flags were used to cover this change?
fflag_feat_front_dev_3873_labeling_ui_improvements_short
Does this PR introduce a breaking change?
(check only one)

[x] Yes, and covered entirely by feature flag(s)
[ ] Yes, and covered partially by feature flag(s)
[ ] No
[ ] Not sure (briefly explain the situation below)

What level of testing was included in the change?
(check all that apply)

[ ] e2e
[ ] integration
[ ] unit

Codecov Report
Patch coverage: 36.36% and project coverage change: -0.04 :warning:

Comparison is base (ca3586f) 64.32% compared to head (b608b62) 64.28%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1310      +/-   ##
==========================================
- Coverage   64.32%   64.28%   -0.04%
==========================================
  Files         428      428
  Lines       26800    26808       +8
  Branches     6943     6945       +2
==========================================
- Hits        17238    17233       -5
- Misses       9562     9575      +13

Impacted Files
Coverage Δ

src/components/Filter/Filter.tsx
3.33% <0.00%> (ø)

...mponents/SidePanels/OutlinerPanel/ViewControls.tsx
88.09% <ø> (ø)

...ponents/SidePanels/OutlinerPanel/OutlinerPanel.tsx
43.75% <25.00%> (-2.68%)
:arrow_down:

src/stores/RegionStore.js
65.23% <50.00%> (-0.21%)
:arrow_down:

... and 2 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


No documentation on UseControlBus

It is only mentioned in two points of the docs [http://masstransit.readthedocs.org/en/latest/configuration/config_api.html?highlight=usecontrolbus](including one that seems to try to link somewhere on the words 'control bus') but has no actual target. In fact the entire concept is confusing. There's an explanation but as a beginner reading the docs and looking at the startbucks sample I have no idea what it does from this.

This is fair -- the control bus will go away as we drop support for MSMQ in MassTransit 3.0. Basically it's there to support the subscription service, which is needed since MSMQ does not have any routing built in.

If you are not using MSMQ, do not use the control bus construct that exists in MT2.


LinqParsing - MetadataToken is insufficient to uniquely identify an expression method.

Appologies, I'm having trouble writing a Marten test to reproduce this as I have no control over what dotnet assignes for MetadataToken, however here are a couple of screengrabs from my project running against the latest (887afe724be9d2ed938f8ab4deb7c44c910df0d9) codebase which is just after 7.3.0...
As you can see the IsDeleted and String compare methods in two different modules, Marten and System.Linq respectively, are getting the same hash code, and therefore LinqParsing.cs is caching the wrong value in the _methodParsers hash map.
Looking at the MS docs MetadataToken is only unique per module, and therefore I think the cache key needs to be changed..
https://learn.microsoft.com/en-us/dotnet/api/system.reflection.memberinfo.metadatatoken?view=net-8.0

Here is an example from local execution of my codebase with Marten integrated, where MaybeDeletedParser is being returned for a string comparison:

public class LinqParsing: IReadOnlyLinqParsing
{
    private static readonly HashSet<string> Encoutered = new HashSet<string>();

    internal IMethodCallParser FindMethodParser(MethodCallExpression expression)
    {
        var key = ToKey(expression.Method);
...

    private static long ToKey(MethodInfo expressionMethod)
    {
        Encoutered.Add(
            $"{expressionMethod.Module.Name}_{expressionMethod.Module.MetadataToken}_{expressionMethod.MetadataToken}_{expressionMethod.DeclaringType?.Name}_{expressionMethod.Name}");

        Console.WriteLine();
        Console.WriteLine();
        Console.WriteLine("---");
        Console.WriteLine();
        foreach (var x in Encoutered)
        {
            Console.WriteLine(x);
        }
...

 I've been bitten by similar issues in the past where you can never trust the Equals() comparison between MethodInfo
But, the mechanism in the PR is a no go on performance. Could we change the look up so that it's a two stage thing? Keep a separate ImHashMap per module, that has an ImHashMap per int for each method within that module? I'd guess that's faster than doing the string concatenation every time.
We had perf issues in StructureMap years ago from recalculating hash codes all the time that shocked me when someone found it and did a PR.
But hey, I appreciate your recent contributions and your willingness to dive into things!

@jeremydmiller - Yeah, this isn't a "done" implementaion, and agree on the performance concern. Just wanted to get the information in with what error I am seeing on my machine.
I'll have a think on a more performant way.. (I did want to use 'Module.MetadataToken' and store that in a long, but that turned out not to be unique either.)
(For completeness, I see the same error from both the 7.3.0 nuget packages and if I build my solution against Marten's source code.)

@JonnyWideFoot I have to profusely apologize. I dropped the ball on this one and didnt' get it in earlier. I took a separate PR for the same issue (arbitrarily took that one). The fix for this is in the forthcoming 7.26.3 release though.


UX: Persistent "let me add another item"

In the create form, when a user chooses to "let me add another item" at the end of the form, don't only redirect him to the create form, also make sure this time the "let me add another item" radio is checked, because most of the time that's what he'll want.
This can be done in two ways:

a cookie (which would also affect other create forms);
a GET parameter;

I tend to think option no 2 is better.
This enhancement, combined with #72 and #71  will greatly improve the create form usability.

Have added a WIP version of this.
Im using the session rather than a query string for the time being, as I couldnt think of a scenario where sharing the data across forms would realistically happen unless they were to perfectly submit both forms perfectly.
What issues can you see with using the session?
https://github.com/Laravel-Backpack/CRUD/pull/145

Yeah, you're right. Session might be better. No issues.
Cheers!

@tabacitu just going to close this as the discussion has furthered https://github.com/Laravel-Backpack/CRUD/pull/145 we can always reopen if you want :)


Tencent Product Detections

9fda127c11df9a3df255a787c6bcc6ffb8cd24cb Improve Tencent CAPTCHA (Tencent Waterproof Wall) Detection
4d16afc81533d36a71c8cdb5dd4c1e47891a468f Add Tencent Analytics Detection

Thanks!


[crypto] Make sure the crypto API does not expose the hash of a message

At the moment signature verification passes a hash as an argument:
https://github.com/MystenLabs/narwhal/blob/36968d41de1680a8d79dbc3f7c301f322ba4ad1c/crypto/src/lib.rs#L210-L230
https://github.com/SalusaSecondus/CryptoGotchas/blob/master/README.md#signatures

An attacker mustn't be allowed to select the actual value being validated in the signature. (The hashing step in all standard signatures defends against that as they can only select the hash pre-image, not the value of the hash.) If they could then they could trivially craft a valid signature for an arbitrary public key by (essentially) generating a random signature and seeing what message would be verified by that signature and then returning that message/signature pair.

The current code & APIs correctly verify signatures using messages


[New Product] Amazon RDS for MariaDB

Amazon RDS MariaDB based on what is already done for other engines

Amazon RDS for MySQL
Amazon RDS for PostgreSQL

@marcwrobel thanks for the review as usual 👍

@marcwrobel sorry, but one additional question.
We have still some engines not documented for RDS

SQL Server
Oracle
Db2
Could be interesting to have them documented also?
For Aurora, it works a bit differently but there is two engines Aurora MySQL and Aurora PostgreSQL (see #3451).

I've just created a PR to extract automatically data for RDS MariaDB endoflife-date/release-data/pull/363.

Could be interesting to have them documented also?

Yes it is.

For Aurora, it works a bit differently but there is two engines Aurora MySQL and Aurora PostgreSQL (see https://github.com/endoflife-date/endoflife.date/issues/3451).

There is also an (rather old) open PR : https://github.com/endoflife-date/endoflife.date/pull/3490, with a pending discussion about which version to document (major or minor).

@marcwrobel sorry, but one additional question.
We have still some engines not documented for RDS

SQL Server
Oracle
Db2

Could be interesting to have them documented also? For Aurora, it works a bit differently but there is two engines Aurora MySQL and Aurora PostgreSQL (see #3451).

Nowadays I can find

https://endoflife.date/oracle-database
https://endoflife.date/mssqlserver

but not DB2  :-(


Set ProjectContext with currencies from CTP

Blocked by https://github.com/sphereio/sphere-jvm-sdk/issues/928

solved by #260


[Feature Request]: Theme Switch

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
Currently, our application lacks a feature of theme switching, limiting user customization options. Add a feature to switch between themes. The PR for this issue should address creating the following:

A theme switching button

White theme

Dark Theme

Use Case
Create and implement a theme toggle button and a dark theme for the website, allowing users to switch between three theme options: light, dark, and auto (device - theme). This toggle should dynamically adjust the color scheme and styling of various elements across the application based on the selected theme.
Benefits
No response
Add ScreenShots
No response
Priority
High
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

This issue is already created @rahul1841


hw/mcu/dialog: SNC

Sensor node controller API. This commit allows users to create
SNC programs using the macros in da1469x_snc.h and run them
either under software control or use the PDC to start them. Note
that PDC control of the SNC is not contained here; the PDC api
already contain all that is needed to have the PDC start the
SNC.

Added in this commit is a test program (hosted) that illustrates
how a program can be created and run under software control. The
code for the test program is in hw/mcu/dialong/da1469x/hosttest.
This directory is also intended for other host tests and not
just the SNC test.

All comments should now be resolved. I forced pushed this latest commit as I had messed up pushing a squashed version of my previous commits on the branch.


Register servicebroker to gorouter

[#151509412]
1 disable tls of servicebroker
2 register servicebroker to gorouter by nets

@qibobo why do we need to register servicebroker with gorouter ?
cloud controller is already able to reach it using consul's service discovery and it is more secure as it's not exposed to anyone else.

@rohitsharma04  Updated for port and system_domain. Please have a review. Thanks.


Json Schema validation is not friendly when used with chart dependencies

Currently, if you use a json schema with Chart.yaml dependencies, you will run into - (root): Additional property global is not allowed.
This can be reproduced with a Chart.yaml like
apiVersion: v2
name: istio-gateway
version: 0.0.1
appVersion: 1.12.0
description: "This Helm chart is an installation of istio-gateway, with additional components."
home: https://istio.io/latest/docs/setup/install/helm/

dependencies:
- name: gateway
  version: 1.12.0
  repository: https://istio-release.storage.googleapis.com/charts

Then we will see:
$ helm template .
Error: values don't meet the specifications of the schema(s) in the following chart(s):
gateway:
- (root): Additional property global is not allowed

This is because the helm chart does not define global in the json schema. This seems reasonable from the chart point of view - when used as-is, global is not a valid.
Its not clear how either a chart author should accommodate for this, or how a a chart consumer should account for this.

I can reproduce this behavior in v3.7.0:
$ helm install issue-10392 . --dry-run

Error: INSTALLATION FAILED: values don't meet the specifications of the schema(s) in the following chart(s):
gateway:
- (root): Additional property global is not allowe

I am facing the same issue.
version.BuildInfo{Version:"v3.7.1", GitCommit:"1d11fcb5d3f3bf00dbe6fe31b8412839a96b3dc4", GitTreeState:"clean", GoVersion:"go1.16.9"}

The same for me
version.BuildInfo{Version:"v3.5.3", GitCommit:"041ce5a2c17a58be0fcd5f5e16fb3e7e95fea622", GitTreeState:"dirty", GoVersion:"go1.16"}

I had to download the chart, unzip it and add the global field inside the values.schema.json file.
Now I include the dependency as a local folder to get around this issue but I would prefer a more clean solution from the chart itself.

I just took a look at Bitnami's way of handling this. Bitnami's Apache chart
They are omitting to set the "additionalProperties" property in the values.schema.json, which defaults to "additionalProperties": true where in the Istio gateway chart it is set as: "additionalProperties": false (Code reference).
I just tested the istio gateway chart as a subchart where it works by omitting "additionalProperties" or setting "additionalProperties": true in values.schema.json in the gateway chart.
This means that it is possible to supply any additional properties (whether it is .global or .gateway-subchart-name.whatever), which I don't know whether if it is a benefit or a disbenefit. If "additionalProperties": true is set, Helm only verifies that any overriding property value set by people is following the definitions defined in values.schema.json (defined by the chart owner). So people won't be able to set an array where the property value should be a string.
I don't know whether a chart owner should be strict or be lenient in regards of supplying undefined properties in a subchart.

+1 on the issue.  Helm version:
version.BuildInfo{Version:"v3.7.1", GitCommit:"1d11fcb5d3f3bf00dbe6fe31b8412839a96b3dc4", GitTreeState:"clean", GoVersion:"go1.17.2"}

This keeps many charts from being referenced as dependency if the comment above is true.

any update on this? It is still reproducible  with
version.BuildInfo{Version:"v3.8.1", GitCommit:"5cb9af4b1b271d11d7a97a71df3ac337dd94ad37", GitTreeState:"clean", GoVersion:"go1.17.8"}

We have run into this issue as well, but I guess it is both expected and surprising depending on how you look at it. It seems that when you reference a subchart, the global section from the parent chart is implicitly injected into the subchart's values.yaml file for validation. After all, the global section is shared among all subcharts.
Furthermore, if you reference a subchart (even a library chart), an implicit entry is made into the parent chart's values.yaml file for the subchart's configuration, such as:
global:
  ...

some: "config"

subchart: # This is implicitly injected
  child: "Config"

So, if you have not taken into account this implicit sections in your values.yaml file, and set "additionalProperties": false in the root of values.schema.yaml, you will run into similar trouble. In this case where we reference a subchart, it is easily handled since we are the author's of the parent chart and can change the schema.
However, in the former case, I guess it is just a matter of explicitly permitting the global keyword in your schema files if you really wish to set "additionalProperties": false. I am not sure that I would say this is necessarily Helm's issue, but rather improper use of values.schema.json files by chart authors being a little too strict. I cannot come up with a case where including a subchart forces any other section than global to be injected, so it seems like a fairly general case to cover. When not permitting the global keyword, your schema file is essentially stating that global values are explicitly not permitted - and I don't know why you would want to enforce that.
You could say this is just a quirk of writing JSON schemas for Helm?

Hi @howardjohn,
While doing canary upgrade from 1.11 to 1.12, for Istio ingress, I observed similar error:
$ helm upgrade istio-ingress istio-1.12.4/manifests/charts/gateway -n istio-system
Error: UPGRADE FAILED: values don't meet the specifications of the schema(s) in the following chart(s):
gateway:

(root): Additional property gateways is not allowed
(root): Additional property global is not allowed
(root): Additional property defaultConfig is not allowed
(root): Additional property meshConfig is not allowed

Is the error resolved yet, if not when can we expect it to be resolved?

@Divyakhatnar that is istio specific, but only global has been fixed/mitigated - the rest are legitimately invalid keys that are not used in the chart.

This is still an issue, would be great to have open

This should still be open, I too am running into this problem.

As @nc-ruth wrote, this is by design. The JSON schema spec. defines how a JSON schema Interpreter - such as the one Helm is using - should handle additionalProperties.
Helm simply sticks to the spec. here.
The problem seems to be that the authors of your chart - in Ops case the Istio chart - have defined the additionalProperties setting in their chart, essentially telling Helm - or really any JSON schema validator - that it's not allowed to have any additional properties defines besides those defined in their JSON schema. You should discuss the reasoning behind that decission with the chart authors.
A temporary workaround could be to disable chart validation when running Helm against a chart you don't control which has that property defined. That said it's arguably better to create a fork of such a chart and remove that  additionalProperties since you probably want to keep chart validation.

Thank you for that answer, @omniproc. That is correct and answer's the OP's question. Since this is not a helm issue, but rather an issue with how the chart validation is defined, I'm going to close this as completed.

I am adding global to my sub chart json validation schema and keeping additionalProperties=false. WDYT?
      "additionalProperties": false,
      "properties": {
       "global": {
            "type": "object",
            "description": "Define global in subchart json schema",
            "default": {}
        },


fix(chip_if): GPIO assign

chip_if has had wrong GPIO assignment. gpio_test.c assumes MIO0 to MIO31 are assigned to GPIO0 to GPIO31 except IoC3, IoB4, IoC4, IoB5.

Even this cannot fix gpio test, this fix is necessary for me to move on chip_sw_sleep_pin_retention test.

I have a proper fix for all GPIO tests in the works, which will be out by tonight.

I have a proper fix for all GPIO tests in the works, which will be out by tonight.

OK. Let me close this. Can you create a draft so that I can cherry pick and move on?


feat(v2): render Markdown on server-side instead

Motivation
Render the Markdown in the loader such that the client doesn't need to do any parsing and we don't need to ship the Markdown renderer into the bundle:
Before: 1.4MB
After: 672KB
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Load doc + blog
Related PRs
NA

Previously,
There is 667 kb associated javascript file

Now it's just this (we'll have to shave the bundle size a bit. All is < 244kb now but the main bundle combined is 266kb


helm test stops at hook failure, will not execute all tests

I would like for all hooks to execute when I issue the helm test command. Currently if there is a failure then the remaining hooks will be skipped completely. I can see why this might make sense for order-dependent resource creation, but for tests I would like to know the result of all tests regardless of whether a given test fails or not.
The problem appears to be here and the fix should be as simple as executing every hook before returning the status.
https://github.com/helm/helm/blob/e63083492b1917b361321ef6951fad08db81d11f/pkg/action/hooks.go#L94
Output of helm version:
% helm version
version.BuildInfo{Version:"v3.9.0", GitCommit:"7ceeda6c585217a19a1131663d8cd1f7d641b2a7", GitTreeState:"clean", GoVersion:"go1.17.5"}

Output of kubectl version:
% kubectl version
WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"24", GitVersion:"v1.24.1", GitCommit:"3ddd0f45aa91e2f30c70734b175631bec5b5825a", GitTreeState:"clean", BuildDate:"2022-05-24T12:26:19Z", GoVersion:"go1.18.2", Compiler:"gc", Platform:"darwin/arm64"}
Kustomize Version: v4.5.4
Server Version: version.Info{Major:"1", Minor:"23", GitVersion:"v1.23.16-gke.200", GitCommit:"822bbca3cfca1ac13d8c759cc72496ffd10b5f59", GitTreeState:"clean", BuildDate:"2023-01-20T09:29:11Z", GoVersion:"go1.19.5 X:boringcrypto", Compiler:"gc", Platform:"linux/amd64"}

Cloud Provider/Platform (AKS, GKE, Minikube etc.):  GKE

After taking a closer look, I see that there is no separation between running hooks for tests and for resource creation, so it seems like the fix may be rather complicated. Perhaps the addition of an annotation like helm.sh/hook-ignore-failures is appropriate?

I also notice that even in the case where multiple hooks ARE executed, if one of the hooks fails then only the result for that hook is returned. I'm sorry I can't provide example output, the tests I have are not generic enough to share currently.

I would like for all hooks to execute when I issue the helm test command.

Can you please elaborate the use case you have in mind? I suspect it is because you want all tests to be run, and not exit on the first failure.

The above doesn't seem unreasonable as a feature request. But I think any changes would need to account for the ordering in which [test] hooks are run: https://helm.sh/docs/topics/charts_hooks/#hooks-and-the-release-lifecycle
In particular, a user today might have ordered their test hooks as to not execute tests which would certainly fail if a previous test failed.
However, OTOH, running those additional tests should just be noise ie. would have no effect except to report the additional failures. But any impacts of running additional tests post-failure would need to be strongly considered.
The reason I mention the above, is the helm project requires maintaining backwards compatibility for all feature changes. Helm is used in a great number of workflows and places. And non-backwards compatible changes will likely break users, which is neither good for helm nor its users.


Use clearer inputs for action

Summary

Contents (#6)
IMPORTANT: There is 1 breaking change.
Enhancements

💥 BREAKING CHANGE: Combine action inputs into one

Operations

Improve workflows
Increase version

Other

Improve README

Upgrade instructions

💥 Combine action inputs into one
Replace the version_source_type and version_source_path inputs with the new path input, giving its value as the path to the file containing your package's version information.

Codecov Report

Merging #6 (1b6c8e7) into main (541e45a) will decrease coverage by 0.41%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main       #6      +/-   ##
==========================================
- Coverage   93.75%   93.33%   -0.42%
==========================================
  Files           1        1
  Lines          48       45       -3
==========================================
- Hits           45       42       -3
  Misses          3        3

Impacted Files
Coverage Δ

check_semantic_version/check_semantic_version.py
93.33% <100.00%> (-0.42%)
:arrow_down:

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


proper size of wmt_ende_tokens_32k-{dev, train}* file?

I got quite low performance compared to the paper.
So, i did some research, and I found the sizes of wmt_ende_tokens_32k-{dev, train}* are too small as follows.
444K    wmt_ende_tokens_32k-dev-00000-of-00001
730M    wmt_ende_tokens_32k-train-00000-of-00001
I ran t2t_datagen again, then i got the following sizes. (with 100 split option)
820K    wmt_ende_tokens_32k-dev-00000-of-00001
14M     wmt_ende_tokens_32k-train-00000-of-00100
....
(total 1400M)
what is the proper size of  wmt_ende_tokens_32k-* file?

The first size looks correct: 444K for dev set (it's only a few thousand sentence pairs, each sentence is ~20 ints, 2204 bytes/int gives ~160 bytes/sentence pair, so ~400kb looks ok). My -train is sharded 100x and I have 7MB in each file (the dataset is 4M pairs, so again, it makes sense).

Since a few people are complaining, could you post the details of your training and results? Did you train on 1 or many GPUs? For how many steps? What is the eval printing out? If it's 1-GPU, you should use the transformer_base_single_gpu hparams config, we should make this clearer in the readme.

[3X titan black, base model]
global_step = 331361, loss = 1.5812, metrics-wmt_ende_tokens_32k/accuracy = 0.663404, metrics-wmt_ende_tokens_32k/accuracy_per_sequence = 0.000824742, metrics-wmt_ende_tokens_32k/accuracy_top5 = 0.84237, metrics-wmt_ende_tokens_32k/bleu_score = 0.325035, metrics-wmt_ende_tokens_32k/neg_log_perplexity = -1.79698, metrics/accuracy = 0.663404, metrics/accuracy_per_sequence = 0.000824742, metrics/accuracy_top5 = 0.84237, metrics/bleu_score = 0.325035, metrics/neg_log_perplexity = -1.79698
=> actual bleu : 21.x on newstest2013
[4x titan x, small model d_model = 256 , d_k = 32, d_v = 32, which is 4th-(C) model in table3 in the paper]
global_step = 167127, loss = 1.48184, metrics-wmt_ende_tokens_32k/accuracy = 0.675814, metrics-wmt_ende_tokens_32k/accuracy_per_sequence = 0.00234962, metrics-wmt_ende_tokens_32k/accuracy_top5 = 0.852813, metrics-wmt_ende_tokens_32k/bleu_score = 0.340511, metrics-wmt_ende_tokens_32k/neg_log_perplexity = -1.6753, metrics/accuracy = 0.675814, metrics/accuracy_per_sequence = 0.00234962, metrics/accuracy_top5 = 0.852813, metrics/bleu_score = 0.340511, metrics/neg_log_perplexity = -1.6753
however, i got "ran out of range" when eval steps are larger than 23.
If i understood correctly, dev dataset has much more pairs and more than 100 eval steps should not results "ran out of range". this is why i suspect dataset for the low performance.

I trained on 8 NVIDIA TITAN-XP with the “transformer_base“ parameters:
@registry.register_hparams
def transformer_base():
"""Set of hyperparameters."""
hparams = common_hparams.basic_params1()
hparams.hidden_size = 512
hparams.batch_size = 4096
hparams.max_length = 256
hparams.dropout = 0.0
hparams.clip_grad_norm = 0.  # i.e. no gradient clipping
hparams.optimizer_adam_epsilon = 1e-9
hparams.learning_rate_decay_scheme = "noam"
hparams.learning_rate = 0.1
hparams.learning_rate_warmup_steps = 4000
hparams.initializer_gain = 1.0
hparams.num_hidden_layers = 6
hparams.initializer = "uniform_unit_scaling"
hparams.weight_decay = 0.0
hparams.optimizer_adam_beta1 = 0.9
hparams.optimizer_adam_beta2 = 0.98
hparams.num_sampled_classes = 0
hparams.label_smoothing = 0.1
hparams.shared_embedding_and_softmax_weights = int(True)
hparams.add_hparam("filter_size", 2048)  # Add new ones like this.
attention-related flags
hparams.add_hparam("num_heads", 8)
hparams.add_hparam("attention_key_channels", 0)
hparams.add_hparam("attention_value_channels", 0)
hparams.add_hparam("ffn_layer", "conv_hidden_relu")
hparams.add_hparam("parameter_attention_key_channels", 0)
hparams.add_hparam("parameter_attention_value_channels", 0)
All hyperparameters ending in "dropout" are automatically set to 0.0
when not in training mode.
hparams.add_hparam("attention_dropout", 0.0)
hparams.add_hparam("relu_dropout", 0.0)
hparams.add_hparam("residual_dropout", 0.1)
hparams.add_hparam("pos", "timing")  # timing, none
hparams.add_hparam("nbr_decoder_problems", 1)
return hparams

The data is split into 100 parts.
The loss is too low and the BLEU on newstest2013 is only  20.37.
Can this configuration achieve the performance in the paper?
Is it enough for 140K steps on 8 GPUs?
Why attention_dropout and relu_dropout are set to 0? Does this hurt BLEU?
INFO:tensorflow:Evaluation [1/20]
INFO:tensorflow:Evaluation [2/20]
INFO:tensorflow:Evaluation [3/20]
INFO:tensorflow:Evaluation [4/20]
INFO:tensorflow:Evaluation [5/20]
INFO:tensorflow:Evaluation [6/20]
INFO:tensorflow:Evaluation [7/20]
INFO:tensorflow:Evaluation [8/20]
INFO:tensorflow:Evaluation [9/20]
INFO:tensorflow:Evaluation [10/20]
INFO:tensorflow:Evaluation [11/20]
INFO:tensorflow:Evaluation [12/20]
INFO:tensorflow:Evaluation [13/20]
INFO:tensorflow:Evaluation [14/20]
INFO:tensorflow:Evaluation [15/20]
INFO:tensorflow:Evaluation [16/20]
INFO:tensorflow:Evaluation [17/20]
INFO:tensorflow:Evaluation [18/20]
INFO:tensorflow:Evaluation [19/20]
INFO:tensorflow:Evaluation [20/20]
INFO:tensorflow:Finished evaluation at 2017-06-28-04:44:23
INFO:tensorflow:Saving dict for global step 145673: global_step = 145673, loss = 0.787518, metrics-wmt_ende_tokens_32k/accuracy = 0.8182, metrics-wmt_ende_tokens_32k/accuracy_per_sequence = 0.00633413, metrics-wmt_ende_tokens_32k/accuracy_top5 = 0.925367, metrics-wmt_ende_tokens_32k/bleu_score = 0.496107, metrics-wmt_ende_tokens_32k/neg_log_perplexity = -0.944883, metrics/accuracy = 0.8182, metrics/accuracy_per_sequence = 0.00633413, metrics/accuracy_top5 = 0.925367, metrics/bleu_score = 0.496107, metrics/neg_log_perplexity = -0.944883

@zxw866 : that looks like a very strong model!
@neverdoubt : when you say "=> actual bleu : 23.85 on newstest2013", how do you measure that exactly? Do you use MOSES scripts, the recent version? Remember that newstest2014 is often 0.5 BLEU or more higher than '13, could you run on that? There is also the hyphenation-split issue which can be around 0.2 difference. We should probably replicate the BLEU calculation we use somewhere too.
Ah, also, we average the last 20 checkpoints with this script:
https://github.com/tensorflow/tensor2tensor/blob/master/tensor2tensor/utils/avg_checkpoints.py
Did you try that? Let's get your results to the same level as ours!
But with your hardware guys, you should try transformer_big too!

I used recent MOSES multi-bleu.perl (https://raw.githubusercontent.com/moses-smt/mosesdecoder/master/scripts/generic/multi-bleu.perl)
actual bleu scores are computed using newstest2013.en file (which is dev set)
i expect 25.8 (as is in table 3) from my base trained model.
Anyway i'll try big model.

Although “metrics-wmt_ende_tokens_32k/bleu_score = 0.496107“ is high, the BLEU on newstest2013 is only 20.37 (multi-bleu.perl).
Can this configuration achieve 25.8 in the paper ?
Is it enough for 140K steps on 8 GPUs ?

@neverdoubt Did you use newstest2013.en without preprocessing or did you postprocess the Tensor2Tensor output before BLEU scoring? I think multi-bleu needs the source and reference to be tokenized...

The result of data generation in Walkthrough is about 1400M, which is double size of BPE training sets. I'm guessing  '_'  is used as an independent token in sentences, which led to the very low loss.
As shown in the data generation process:

I wonder if this is a BUG?

It turns out that the separate "_" was a bug introduced inadvertedly in a recent PR by Villi (see the chat on Gitter). We didn't have it before, so it might be responsible for some of the lower BLEU, but maybe not that much -- we should correct it in any case.
Another point is that all results in the paper are obtained with checkpoint averaging. Use the avg_checkpoints script from utils on the last 20 checkpoints that are saved in your $TRAIN_DIR. It's like a poor-man's version of Polyak averaging, but it's needed to reproduce our results (we're planning to add true Polyak averaging to the trainer at a later point).
And then you need to (1) tokenize the newstest and the (separated) decodes:
perl ~/mosesdecoder/scripts/tokenizer/tokenizer.perl -l de < $decodes > $decodes_file.tok
(2) Split on hyphens to be compatible with BLEU scores from other papers:
Put compounds in ATAT format (comparable to GNMT, ConvS2S)
perl -ple 's{(\S)-(\S)}{$1 ##AT##-##AT## $2}g' < $tok_gold_targets > $tok_gold_targets.atat
perl -ple 's{(\S)-(\S)}{$1 ##AT##-##AT## $2}g' < $decodes_file.target > $decodes_file.atat
(3) Finally run multi-bleu:
perl ~/mosesdecoder/scripts/generic/multi-bleu.perl $tok_gold_targets.atat < $decodes_file.atat
Especially doing the averaging and tokenizing (1) is important, detokenized BLEU is often quite a bit lower than tokenized one.

300k step trained model using  4 titan X
4th row in (C) model  (d_model = 256, d_k=32, d_v=32)
newstest2013.{en,de} bleu 24.2
The paper said 24.5 without averaging. Now we are in the same level.
btw, after 20 checkpoint averaging, i got 24.78

Was this with the newest version of T2T, i.e. 1.08? The one with the separate underscores? Would be nice to get a confirmation that those don't necessarily hurt model performance (and may even make it better ;-) )

@vthorsteinsson I used 1.0.7 for training (which has separate '_' issue), but my training data was created earlier version (maybe 1.0.2 or 1.0.4).

I added utils/get_ende_blue.sh in 1.0.9. This is a script that includes the commands we used to get BLEU in the paper. You might need to fix a path to MOSES and tokenized newstest2013 there.
Please: could you average your checkpoints with utils/avg_checkpoints.py and then run utils/get_ende_blue.sh and report back the results? Just to make sure where your models really stand compared to our results, even despite possible tokenization differences. Thanks!

When using BPE training set, i got 24.64 on newstest2013. It's close to the results in the paper.
Next i will try utils/get_ende_blue.sh. Thanks!

Just remember that wmt_ende_bpe32k is tokenized, so instead of the tokenizer call in the get_ende_bleu.sh script, do this: perl -ple 's{@@ }{}g' > $decodes_file.target. Also, did you average checkpoints? Let us know what numbers you get!

transformer_base hparams.
110k steps using 1 titan Xp, then 140k steps using 8 titan Xp.
I averaged 7 checkpoints.
I removed '@@' using "sed -r 's/(@@ )|(@@ ?$)//g'".
Then I got 24.64 on newstest2013.
I'm guessing the learning rate decay was affected in my experiment.
Next I plan to run the big model.
I really appreciate your help！

That looks reasonable. Did you run get_ende_bleu, I mean esp. the "atat" part? That can be 0.2 or 0.3 BLEU if you forget it.

Hi @lukaszkaiser   I have read the discussion above and found the details for calculation BLEU in https://github.com/tensorflow/tensor2tensor/issues/44 is different. So I want to make sure something:
What the format is needed for the decoding file (--decode_from_file=$DECODE_FILE, such as newstest2013 or newstest 2014), Do I need to do tokenizaton and put compounds in ATAT format before feeding into the decoding process? Do I also need to do bpe? Or just use the raw text sentences?

The input file should be detokenized, pure text. (Except if you do BPE, but I suggest trying without it.)

@lukaszkaiser    When I feed pure test for decoding and checkpoint average, I can get a BLEU score 26 on the base configuration model.

That sounds reasonable. I'm closing this issue for now as it's gotten long and tokenization changed in 1.0.11. I hope things are ok now, but please either re-open or make a new issue if you see the problems again!


  You started the Same fuse I did that can’t go out.

I don't believe you


broken links and working on logic of routes✨

the github link is broken and the navbar needs to be fixed as links are broken there.
when ur in the world clock u cant go back to counter , i can add that in the navbar.
i want to add a digital clock as well.
@param-code can you assign me this task.

the github link is broken and the navbar needs to be fixed as links are broken there. when ur in the world clock u cant go back to counter , i can add that in the navbar. i want to add a digital clock as well.
@param-code can you assign me this task.

Ofcourse.All the Best.

the issue i wanted to fix has been solved.
can i instead add storage feature where u can signup and login and the user details will be saved in the local storage.
@param-code

 the issue i wanted to fix has been solved. can i instead add storage feature where u can signup and login and the user details will be saved in the local storage. @param-code

Okay.All the Best!!


Upgrade to Piwik 3.1.0

Could you upgrade Piwik in this to version 3.1.0?

Build on Docker Hub is in progress, it will be available pretty soon at https://hub.docker.com/r/webhippie/piwik/tags/ as 3.1.0 and latest.

And now also updated to 3.1.1.


FBP-131. Do not draw drag anchor

Fixes #131

Codecov Report

Merging #147 into master will decrease coverage by 0.02%.
The diff coverage is 0.00%.

@@             Coverage Diff              @@
##             master     #147      +/-   ##
============================================
- Coverage     61.49%   61.46%   -0.03%
  Complexity      785      785
============================================
  Files            84       84
  Lines          3459     3459
  Branches        644      644
============================================
- Hits           2127     2126       -1
  Misses         1113     1113
- Partials        219      220       +1

Flag
Coverage Δ
Complexity Δ

#unittests
61.46% <0.00%> (-0.03%)
785.00 <0.00> (ø)

Impacted Files
Coverage Δ
Complexity Δ

...lugin/render/elements/shapes/ShapeRenderElement.kt
76.13% <0.00%> (-0.57%)
46.00 <0.00> (ø)

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 9f5a3bf...5ec7209. Read the comment docs.


Queue:works always fail after the first job

Laravel Version: 5.5
PHP Version: 7.0
I am facing below problem. Please help me to resolve this issue. I am dispatching multiple queues using below code using for loop:
ProcessTask::dispatch($taskID);
I checked queue table and Laravel sets all queues in table but when supervisor executes it, first queue works fine and remaining queues fail. When I manually run using php artisan queue:listen, it is working fine.
Thanks in advance!

Sounds like you've done code changes that the supervisor-hosted queue worker hasn't picked up on. Try restarting it. If it still fails, tell us how it fails, preferably with an exception or explanation.

It shows following error message:
**Message:** App\Jobs\ProcessTask has been attempted too many times or run too long. The job may have previously timed out.
**Code:** 0
**File:** /var/www/html/myapplication/vendor/laravel/framework/src/Illuminate/Queue/Worker.php
**Line:** 394

And Supervisor file is laravel-worker.conf
[program:laravel-worker]
process_name=%(program_name)s_%(process_num)02d
command=php /var/www/html/myapplication/artisan queue:work --timeout=0 --tries=1
autostart=true
autorestart=true
user={{usernamehere}}
numprocs=8
redirect_stderr=true
stdout_logfile=/var/www/html/myapplication/worker.log

Note: When we run queue using php artisan queue:listen, it is working fine. Kindly help me to solve using supervisor.
Thanks in advance !

Does the log files contain any relevant errors?
This repository is for bug reporting. So far this is just general support. If you cannot find a reproducible bug, try asking on the forums or on Slack.


[build-presets] Verify generated SwiftSyntax files during Swift smoke testing

Through #35613, we should have caught PR breakage caused by SwiftSyntax files not being updated. However, https://github.com/apple/swift-syntax/pull/259 slipped past us again. Turns out, I missed the argument to actually verify the generated SwiftSyntax files.

@swift-ci Please smoke test

@swift-ci Please smoke test windows

@swift-ci Please smoke test macOS


not able to pick up internet facing interface

there's seem to be an issue when I input interface connected to the internet, doesn't matter which interface I use. I've installed all needed dependencies while running the script for the first time
[?] Install/Update dependencies? Y/n: 'n'
[?] Please enter the name of your wireless interface (for the AP): 'test'
[?] Please enter the name of your internet connected interface: wlan0
Traceback (most recent call last):
File "mitmAP.py", line 70, in
net_iface = input("[?] Please enter the name of your internet connected interface: ")
File "", line 1, in
NameError: name 'wlan0' is not defined
when I try 'wlan0'
[?] Please enter the name of your internet connected interface: 'wlan0'
[I] Backing up NetworkManager.cfg...
[I] Editing NetworkManager.cfg...
Traceback (most recent call last):
File "mitmAP.py", line 76, in
write_file("/etc/NetworkManager/NetworkManager.conf", network_manager_cfg )
File "mitmAP.py", line 36, in write_file
_run_cmd_write((sudo, tee, path), s)
File "mitmAP.py", line 29, in _run_cmd_write
stdout=subprocess.DEVNULL,
AttributeError: 'module' object has no attribute 'DEVNULL'
Also it seems there's a problem with user input need to use single quotes, see the below:
[?] Install/Update dependencies? Y/n: 'n'

I've run once again with python 3 and all works fine


Geocode photographs as they are submitted

Need to check with Andrew to see if this has been done already.

Duplicate of https://github.com/modilabs/formhub/issues/41


VcsHostTest: Stop using Mercurial repos from Bitbucket

As Bitbucket stopped hosting Mercurial in favor of only Git, see
https://bitbucket.org/blog/sunsetting-mercurial-support-in-bitbucket.
Signed-off-by: Sebastian Schuberth<EMAIL_ADDRESS>

Merging despite the failing VcsHostTest which is unrelated and gets fixed by https://github.com/oss-review-toolkit/ort/pull/2947.

Merging this despite some bogus test failures which cannot be reproduced locally. I'll watch test failures on master / in upcoming PRs closely.


[BUG] NodeInfoRequest fails when connecting to Amazon OpenSearch Service

What is the bug?
When sending a NodeInfoRequest to Amazon OpenSearch Service the request fails with the following error:
Missing required property 'NodeInfo.host'
This issue appears to be caused by  Amazon OpenSearch Service obfuscating/removing some fields from the REST API responses that the client code expects to be not null.
Beside NodeInfo.host the following fields are also missing from the response and are causing a similar error:

NodeInfo.IP
NodeInfo.TransportAddress

How can one reproduce the bug?

Connect the client to Amazon OpenSearch Service with basic auth and only fine-grained access control enabled (IAM is disabled).
Send a NodeInfoRequest:

   NodesInfoRequest nodesInfoRequest = new NodesInfoRequest.Builder().metric(List.of("plugins")).build();
   NodesInfoResponse nodesInfoResponse = openSearchNodesClient.info(nodesInfoRequest);

Do you have any additional context?
I can raise a PR with a fix which consists in removing requireNonNull from these specific fields.

Thanks for opening this @dancristiancecoi. Removing the null constraint is likely the right move. As we are working on generating much of this code from spec in #366, please also check https://github.com/opensearch-project/opensearch-api-specification for correctness?

@dblock I think we can remove these two fields from the required list if my findings are actually correct and its not due to a misconfiguration on my part. Cheers!

@dblock I think we can remove these two fields from the required list if my findings are actually correct and its not due to a misconfiguration on my part. Cheers!

Is the only way to reproduce this behavior to run against AWS serverless? Either way care to contribute the fix into the spec, please?

I have only seen this issue with Amazon OpenSearch Service. Have not tested Serverless but I suspect it will have a similar behavior.
And sure, I will raise an issue & PR.

PR merged

PR merged.

v2.14.0 has been released and includes the fix for this issue


When a new tab is opened, multiple tabs open on the paired session.

This behavior seems to be somewhat inconsistent. Sometimes I get a duplicate tab, and other times its just blank tabs.
I have been able to recreate this on local and remote pairing sessions.

:+1: Also seeing this

The behavior may be related to switching files. I shared a session with a friend remotely, and when the session started there were only the expected tabs on his end. I opened another file, and his machine got flooded with tabs that eventually all became copies of the file I’d opened.

Yup, also I noticed if you click on the same file in your file tree, it opens up another copy of the file on the paired machine.

Same thing happens to me, not totally annoying but would be nice to see fixed soon.

Thanks for raising this! - I'll try to address this in the next release.

Thanks @jpatel531 :+1:


Some translation of the Description core's section

I plan to merge this PR to master and hence please address the pending issue on the PR for the error message "All checks have failed".


Fix a broken example for the "Response Headers" docs

This PR provides a small fix with a broken example in the "Response Headers" section of the docs.
PR Checklist

[x] Have you followed the guidelines in CONTRIBUTING.md?
[ ] Have you got 100% test coverage on new code?
[ ] Have you updated the prose documentation?
[ ] Have you updated the reference documentation?

@all-contributors add @Jarmos-san docs


Bug in some animation models

Some models do not export and animate as expected.
Here are two samples:
https://sketchfab.com/models/710bbaccf8a442ea9b900354361f6d5a
https://sketchfab.com/models/8d06874aac5246c59edb4adbe3606e0e

@kcoley what is interesting about the first model
https://sketchfab.com/models/710bbaccf8a442ea9b900354361f6d5a
Its actually animating correctly in xcode beta 6. I've attached a mov file showing the animation working (unzip to run)
spiderman_animation_xcode10_beta6.zip
Attached is the resulting usdz (non-animating t-pose).
spiderman.usdz.zip
It may be a bug with an Apple usdz displaying in Safari?

@wave-electron that is interesting.  Based on some of the weird distortions on certain frames, this leads me to believe it is because some key frame animation data may be missing.  I'll check the logic there.  Thanks for the reference!

I just had a look at the animation again in xcode. I also noticed some flashing black disortions on the keyframes. Maybe it is that!

@kcoley
Did you notice this comment by Spiff today in the usd-interest group?
I've noticed we use Xform rather than SkelRoot in the bone animations I've been building.
UsdSkel

From a strictly USD perspective, there are no layering problems.  However, (and I know ModelIO doesn't currently care, but we hope it will in the future) there is a problem with your construct.  Valid skeletal animation and geometry must be located under a "SkelRoot" prim.  So, in your file, either /root/node0 or /root/node0/node2 should be a "SkelRoot" prim rather than "Xform".  Also, we try to put the skel:animationSource and skel:skeleton bindings on the SkelRoot, when possible.

Spiff comments were in respect to this 01.usda file inside this CesiumMan.zip..... 01.usda animates in xcode10 beta 6, but only previews with no animation in IOS 12 beta 12.
CesiumMan.zip

@wave-electron thanks for the note on the SkelRoot.  I can look into that.  Also cool that the model at least works on iOS!  Looking at the usda file, I see the problem with the popping animation due to some missing values for some of the joints at certain frames.  I thought I had resolved this, but I must have missed some cases.  I'll work on fixing that.

ahh interesting. I create a SkelRoot with a Skeleton + SkelAnimation inside it in #36. However I have a feeling that my changes to support blend shapes will probably break normal rigging animations. Do you have any models that have blendshapes + animations?

After several hours of trying today. I built up my own bone animation from scratch using a 3dwarehouse model of hand
I built the joint structure & created different poses in Cheetah 3D. Then exported as Collada Model. I imported into Blender in order to use the GLTF 2 exporter. After using the gltf2GLTF command tool , I edited the usda file fixing the Joint name manually to overcome that bug relating to #22. The result is the model in my gallery at fusionar.app. Finally found a way to create these bone animations from scratch!


Locked Collateral is lost when no data uploaded under contract?

https://forum.sia.tech/topic/1576/locked-collateral-is-lost-when-no-data-uploaded-under-contract
I'm using storage obligation api to see what contracts my host have.
https://github.com/NebulousLabs/Sia/compare/master...S-anasol:obligationsapi
With https://s-anasol.github.io/hostdbviewer/#/ which allow me to see my host contracts in beautiful way
With this two tools i'm able to see what contracts my host have.
I see that there are quite big ammount of contracts with locked collateral, but no data uploaded. And i confirm locking of collaterl by this contracts in "wallet transactions"
I was thinking that locked collateral for failed contracts with no data uploaded will be returned when storage proof window ends, but as long as i can see in "wallet transactions" this doesn't happen.
When contract is failed i see
2017/07/11 18:55:43.180727 storageobligations.go:574: Missed storage proof. Revenue would have been<PHONE_NUMBER>79227611912127230.
in logs
And according to wallet transactions when this happens -1.33SC is deducted. And transaction fees of 1.33 is added to failed obligation.
Anyone have some ideas about it? What is going on?
Not sure but maybe it is somehow related with https://github.com/NebulousLabs/Sia/issues/1862 https://github.com/NebulousLabs/Sia/issues/1861

Contract: https://explorer.siahub.info/hash/91c9b542dd1562921f304a3996091e3f310f2a2a7231047ed9e9056dc9a4264b
Locking of collateral deducted from my balance https://explorer.siahub.info/hash/df564aac153028e44dacdfc6591e5ec9ffc4b4db89bdaa77158d982c76c743b2
siac wallet transactions | grep df564aac153028e44dacdfc6591e5ec9ffc4b4db89bdaa77158d982c76c743b2 113659   df564aac153028e44dacdfc6591e5ec9ffc4b4db89bdaa77158d982c76c743b2         -11.58 SC             0 SF
Missed proof output to one of my addresses
https://explorer.siahub.info/hash/737b67ed79dc6ecb8dbd36a4861920b4fe5a0a217b78f2e3e24ffc00ef889cbd
siac wallet addresses | grep  76f700da12f1504dc9eab5dd49eb605bfe80f0e95875da7ec484f11ce6bc033ca6c7fd4ba6c7 76f700da12f1504dc9eab5dd49eb605bfe80f0e95875da7ec484f11ce6bc033ca6c7fd4ba6c7
transaction with output to this address https://explorer.siahub.info/hash/661f821eab3effcc07248357947a495a7730cb98c8c20980b0a911941f43c740
siac wallet transactions | grep 661f821eab3effcc07248357947a495a7730cb98c8c20980b0a911941f43c740       113659   661f821eab3effcc07248357947a495a7730cb98c8c20980b0a911941f43c740           0.00 SC             0 SF
why it is 0.00 SC and not 11.58 SC?

I think they are related #1933

Well Hi there.  You linked to my bug.
My bug was to say that earnings weren't reflected in the wallet activity.  The siac uses a 'soft unlock' in that it just slides them in without showing you.  SC was earned and collateral returned in my cases.  I just thought it was dumb the UI didn't reflect it.

@JamesM424 I think locked collateral is returned, bit it is not reflected.


Added redirect_url property to RefTimeseriesLogicalFile

This is a quick fix for Issue #3511. I just added the redirect_url property found in the GenericLogicalFile to the RefTimeseriesLogicalFile.
Pull Request Checklist:

[ ] Positive Test Case Written by Dev

[ ] Automated Testing

[ ] Sufficient User and Developer Documentation

[ ] Passing Jenkins Build

[ ] Peer Code review and approval

Positive Test Case

From the landing page of a resource with reference time series content, try downloading the refts file using the download button, and then using the file URL.

I know there's some work being done with aggregations that might affect this, but I wonder if this property should be added to the AbstractLogicalFile? The issue seems to affect all single file aggregations, but GenericLogicalFile already had the redirect_url property.


Strange behaviour with truncate_to_unique

I've a strange behaviour with the option POWERLEVEL9K_SHORTEN_STRATEGY="truncate_to_unique"
If I set it that way the prompt returns
prompt_dir:66: no matches found: //.*/
if the folder is a hidden one, or
prompt_dir:66: no matches found: //D*/
if the folder starts with D
But it's pretty random, in some folder it returns that and in other works normally
If i remove that row from the .zshrc it works flawlessly in every folder
gnome-terminal 3.28.1-1ubuntu1mint2
zsh 5.4.2 (x86_64-ubuntu-linux-gnu)
oh-my-zsh and p9k cloned from git, latest

I am having this same issue

@G4Zz0L1 - Thanks so much for the bug report, and thanks for confirming, @nvonbulow!
@onaforeignshore has already put together a fix. Can you check out #915, give that PR a shot, and let us know if it resolves your issue?

It kind of works for me too, but with a strange behavior regarding the beginning of the path printed on the screen:

That ~h/d/ seems wrong to me, it should be ~/ or /h/d/

What is the content of your $HOME variable?

I'm having the same weird path beginning as @diraol with the unwanted ~h. My $HOME is set to /home/nick

I have updated the fix. Can you please pull it again and see if the issue persists?

The issue is still there. It doesn't look like anything changed

It is better now for me.

With the addition of the code simplification commit, everything works for me now.

@nvonbulow That's good news.

Thanks so much, @nvonbulow and @diraol, for testing the fix in #915! And thanks for filing the initial bug report, @G4Zz0L1! Great work, everyone.
The fix #915 has been merged into master =)


add cluster nacos support

Please answer these questions before submitting pull request

Why submit this pull request?
[ ] Bug fix
[ x ] New feature provided
[ ] Improve performance

New feature or improvement

Support for discovering cluster nodes through nacos.

This feature depends on alibaba/nacos (https://github.com/alibaba/nacos).
The LICENSE is APL 2.0 (https://github.com/alibaba/nacos/blob/master/LICENSE).

This feature depends on alibaba/nacos (https://github.com/alibaba/nacos).

Does nacos-client depend on other 3rd-party libs? I assume YES. And what are they?

This feature depends on alibaba/nacos (https://github.com/alibaba/nacos).

Does nacos-client depend on other 3rd-party libs? I assume YES. And what are they?

This is the list of 3rd-party libs depended by nacos-client:
        <dependency>
            <groupId>org.slf4j</groupId>
            <artifactId>slf4j-api</artifactId>
            <optional>true</optional>
        </dependency>

        <dependency>
            <groupId>org.apache.logging.log4j</groupId>
            <artifactId>log4j-core</artifactId>
            <optional>true</optional>
        </dependency>

        <dependency>
            <groupId>org.apache.logging.log4j</groupId>
            <artifactId>log4j-slf4j-impl</artifactId>
            <optional>true</optional>
        </dependency>

        <dependency>
            <groupId>${project.groupId}</groupId>
            <artifactId>nacos-common</artifactId>
        </dependency>

        <dependency>
            <groupId>${project.groupId}</groupId>
            <artifactId>nacos-api</artifactId>
        </dependency>

        <dependency>
            <groupId>ch.qos.logback</groupId>
            <artifactId>logback-classic</artifactId>
            <optional>true</optional>
        </dependency>

        <dependency>
            <groupId>ch.qos.logback</groupId>
            <artifactId>logback-core</artifactId>
            <optional>true</optional>
        </dependency>

        <dependency>
            <groupId>com.google.guava</groupId>
            <artifactId>guava</artifactId>
        </dependency>

        <dependency>
            <artifactId>commons-codec</artifactId>
            <groupId>commons-codec</groupId>
        </dependency>

        <dependency>
            <groupId>org.codehaus.jackson</groupId>
            <artifactId>jackson-mapper-lgpl</artifactId>
        </dependency>
        <dependency>
            <groupId>net.jcip</groupId>
            <artifactId>jcip-annotations</artifactId>
            <optional>true</optional>
        </dependency>

        <dependency>
            <groupId>com.github.spotbugs</groupId>
            <artifactId>spotbugs-annotations</artifactId>
            <optional>true</optional>
        </dependency>

        <dependency>
            <groupId>junit</groupId>
            <artifactId>junit</artifactId>
            <scope>test</scope>
        </dependency>

        <dependency>
            <groupId>io.prometheus</groupId>
            <artifactId>simpleclient</artifactId>
            <version>0.5.0</version>
        </dependency>

The license info:

component
license

slf4j-api
MIT License

junit
EPL 1.0

spotbugs-annotations
LGPL

jackson-mapper-lgpl
LGPL 2.1

logback
EPL 1.0 & LGPL 2.1

The other compotents license is APL 2.0.

The licenses look like good, but what does the optional mean? Does the end user need that or not?
The right list would be the jars added in skywalking dist tar. Could you cross check your new dist with skywalking's master dist?

Wow, It's a cool feature for skywalking cluster.

The licenses look like good, but what does the optional mean? Does the end user need that or not?
The right list would be the jars added in skywalking dist tar. Could you cross check your new dist with skywalking's master dist?

'optional' means it will not be included in the dist jars.

Coverage remained the same at 13.898% when pulling 29c6fae9298932822b7849dc2a44b86c998c4438 on litian33:cluster-nacos-support into 16aca37e70baad8d967c919dab5acc2ae6aca834 on apache:master.

@JaredTan95 Could you test and verify this PR?
@litian33 You need to follow this LICENSE file, https://github.com/apache/incubator-skywalking/blob/master/apm-dist/release-docs/LICENSE#L204

add license descriptions of your new jars(in dist lib)
If LICENSE is not Apache 2.0, you need to copy the license file to license folder
If LICENSE is Apache 2.0, and it includes NOTICE file in same folder, you need to copy the contents into our dist NOTICE

This is the legal way :) So you must follow. Also, after you have done, please provide a list in here to help us recheck these dependencies.

| spotbugs-annotations | LGPL |
| jackson-mapper-lgpl | LGPL 2.1 |

@wu-sheng GNU LGPL 2, 2.1, 3 may NOT be included within Apache products.

GNU LGPL 2, 2.1, 3 may NOT be included within Apache products.

@peng-yongsheng Oh, sorry I miss one. Dependency jackson-mapper-lgpl is not optional. Thank you for pointing this out.
@litian33 We can't accept this PR because it breaks the Apache foundation requirements. You can use them, but in Apache release and source code host, we can't. If you want to contribute this, I will recommend you open source in another repo, and release there, we could provide a document link to there. In that repo, you could do release by following your own wish and provide the document to guide user to use.
If you want to make this in Apache repo, then all GPL/LGPC must be removed, please read catalog-x

OK. I will matain the fork myself.

@litian33 I suggest you do something like https://github.com/SkyAPM/java-plugin-extensions . You deploy the plugin based on our release API, then release by yourself.
People could download from your page, and copy them into skywalking lib folder to active it.
The fork is unsearchable in GitHub. So, a new repo is a better choice from my perspective.

@litian33 After you released, we are welcome to accept your PR to submit documents in cluster management section, which could include a link to your repo or release page.
We are highly appreciated your contributions and extension.

@litian33 Nacos team is responding.  You could follow the issue I just raised.

@litian33 Could you try to use the latest Nacos release to submit a new PR and list the new dependencies there?


[cbuild] doesn't report build status properly via exit code

Using cbuild to build a solution context which has compiler issues still returns with error level 0:
$ cbuild demo.csolution.yml --context demo.debug+target --packs --update-rte
info cbuild: Build Invocation 1.5.0 (C) 2023 Arm Ltd. and Contributors
 :
ninja: build stopped: subcommand failed.
error cbuild: error executing 'cmake' build
error cbuild: error building './demo.debug+target.cprj'

$ echo "cbuild returned with exit code $?"
cbuild returned with exit code 0
                                ^
                                this should reflect the error

Instead, cbuild should report the overall result in its exit code.

I could replicate the issue cbuild 1.5.0.
Seems like this issue is resolved in the cleanup and restructuring. Couldn't replicate in the current binaries.
https://github.com/Open-CMSIS-Pack/cbuild/actions/runs/4365909868

@JonatanAntoni I am closing this now, but please reopen in case you find it is not solved for you.


Issue with k8s.io/docs/getting-started-guides/kops/

This is a...

[ ] Feature Request
[x ] Bug Report

Problem:
The instructions for installing on MacOS begin with a wget command. wget is not available on MacOS by default. It should also be noted that the download link redirects.
Proposed Solution:
Use curl command instead. For example:
curl -OL https://github.com/kubernetes/kops/releases/download/1.8.0/kops-darwin-amd64

The -L option is important for following the redirect.
Page to Update:
https://kubernetes.io/docs/getting-started-guides/kops/

#7613  fixes this

@zacharysarah Can you close this issue? I forgot to use the magic words in my PR :joy:. Thanks!

@pmallory Sure thing! :smile:


Remote SSH fingerprint change promt

Remote SSH is pure 🔥. However, when using different local VMs for different projects SSH fingerprint changes causing broken pipe. I would really appreciate a promt informing that fingerprint has changed and ask for confirmation to connect. The current workaround now is to delete "known_hosts" from .ssh.
Example of current situation
VM1 is brought up and portforward to <IP_ADDRESS>:2222
VM1 is used for remote SSH in VSCode
VM1 is brought down
VM2 is brought up  and portforward to 127.0.01:2222
VM2: pipe is broken, fingerprint has changed

I realize this should be posted on the repo for extension


scope variable for element being active

Hi,
I am trying to use the element.active state as trigger for another directive. Is there a way to access this state parameter in the scope of the element? This could be very useful if it become a standard part of the API.
Best,
Ge

This should be doable using duScrollspy:becameActive events.

Thanks! Ended up doing that.


jumpapp: init at 1.0

Motivation for this change
Packaging a useful utility for window management on X11: https://github.com/mkropat/jumpapp
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

This pull request has been mentioned on Nix community. There might be relevant details there:
https://discourse.nixos.org/t/packaging-shell-scripts/2447/3


Update jdbc.md Change postgres username

change postgres default username mysql_user/mysql_password to pg_user/pg_password.
What type of PR is it?
Documentation

ping @huage1994


Environment variable override

At the moment new values need to be added to the secret store by calling crypta set ....
As secrets are usually handled via environment variables in CI/CT environments, it would be necessary to call crypta set ... for each defined environment variable in order to pre-populate the secret store for later secret retrieval. (Of course, one could also use the environment variables directly where necessary, but this would require different handling of the secrets for CI/CT and local development usecases. Crypta is developed to exactly bridge this gap, and streamline/unify the credential handling for both usecases.)
Therefore, crypta should automatically pre-populate values from environment variables.
The environment variables have the following format: CRYPTA_SET_<KEY>. The value given for <KEY> will be used as the name with which the value of the environment variable will be added to the secret store.
The environment variables are parsed once when the daemon starts. They can then be retrieved by calling crypta get <KEY>
The pre-populated values in the secret store can be overwritten by calling crypta set ..., and therefore only provide initial default values.

Shall the environment variable override directly affect the get command of the client, or shall it set default values in the daemon?

Shall the environment variable override directly affect the get command of the client, or shall it set default values in the daemon?

Nevermind. I think the handling is easier in the daemon, as it will also allow to overwrite the value defined by the environment variables if necessary.

Since the environment variables should mainly handle the CI purpose where the environment variables are always set, I would prefer to not start the daemon at all. It is not required since the values are stored in the environment variables anyway and it should always take those. Otherwise, it is hard to understand if you set the secrets directly but other values will be taken (e.g., because another command sets another value). Therefore, I would return the values directly without starting or interacting with the daemon.

Wouldnt this limit the possible usecases "by-design"?
Don't you think that local usecases with sane default values in Environment variables, but with the possibility to overwrite them via the tool might be useful?
If we were to move some of the "main application logic" into the client I fear that this will limit future usecases "by our architectural design choices".
Also, i would personally prefer if the way to use the tool is always the same. It involves the client & the daemon, regardless of the context in which it runs. If we now start to implement "special usecases in certain scenarios" I fear that the tool will get more complex. Especially when I think about the "auto-start daemon feature" that is also planned. I would prefer if we move most of the logic into a central component, and not distribute it between the client and the server. This would also allow us to test most of the logic on component level (focusing on one component), instead of end2end tests. But I have to admit that this is a personal preference, and I am not sure what the state-of-the-art for this is at the moment.
Let me know what you think

iImplementation wise it makes probably more sense to move it to the daemon so that the client logic is always the same. I'm not sure if there is a hybrid scenario where you will have environment variables and values provided by the user in parallel but you are right, it probably makes sense to not fully limit this special case. Therefore, I would be with you to implement it on daemon side.


eslint complaining about not finding libStorage

Checklist

[x ] I have read Caveats documentation and didn't find a solution for this problem there.

Bug description
With v1.0.0
If I do this
in package.json
    "devDependencies": {
        "aws-sdk-client-mock": "^1.0.0"
    }

in mytest.spec.js
    const { mockClient } = require("aws-sdk-client-mock");
    const { mockLibStorageUpload } = require("aws-sdk-client-mock/libStorage");

eslint will complain about not finding the reference for libStorage, although when I run the tests, they run ok.
So the solution for now is to add a disable for next line, but it is still weird it shows up.
// eslint-disable-next-line import/no-unresolved
Environment

Node version: 14
Testing lib and version: jest 28
Typescript version: js
AWS SDK v3 Client mock version: 1.0.0
AWS JS SDK libs and versions: 3.112

Hey. I plan to remove the mockLibStorageUpload altogether in the next release. It caused far too many problems. See #105.
Please see the updated README on the libStorage mocking: https://github.com/m-radzikowski/aws-sdk-client-mock/blob/26311f15e8/README.md#lib-storage-upload
In short, all you have to do to mock the libStorage is to mock those two Commands:
const s3Mock = mockClient(S3Client);
s3Mock.on(CreateMultipartUploadCommand).resolves({UploadId: '1'});
s3Mock.on(UploadPartCommand).resolves({ETag: '1'});

Note: I've released the 2.0.0-beta.0 version without the mockLibStorageUpload. Please try it, manually mocking CreateMultipartUploadCommand and UploadPartCommand.

thanks for the answer


add option to choose image on interval end

Description
Added a preference that lets users choose their own markdown image link to be displayed on interval completion.
Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Thanks for the update @brodelp 😊 Do you mind adding a changelog entry too?
@KELiON can you check this?


Having unicode-range at the end of a block throws indentation error in next block

Sass Lint version: 1.11.1
Configuration: sass-lint.yml | node script to run it

My code:
@font-face
    font-family: 'balooregular'
    font-style: normal
    font-weight: normal
    src: url('../_fonts/baloo.woff') format('woff')

@font-face
    font-family: 'lato'
    font-style: normal
    font-weight: 400
    src: local('Lato Regular'), local('Lato-Regular'), url('../_fonts/lato-regular.woff') format('woff')
    unicode-range: U+0000-00FF, U+0131, U+0152-0153, U+02C6, U+02DA, U+02DC, U+2000-206F, U+2074, U+20AC, U+2212, U+2215

@font-face
    font-family: 'lato'
    font-style: normal
    font-weight: 700
    src: local('Lato Bold'), local('Lato-Bold'), url('../_fonts/lato-bold.woff') format('woff')
    unicode-range: U+0000-00FF, U+0131, U+0152-0153, U+02C6, U+02DA, U+02DC, U+2000-206F, U+2074, U+20AC, U+2212, U+2215

@font-face
    font-family: 'lato'
    font-style: italic
    font-weight: 400
    src: local('Lato Italic'), local('Lato-Italic'), url('../_fonts/lato-italic.woff') format('woff')
    unicode-range: U+0000-00FF, U+0131, U+0152-0153, U+02C6, U+02DA, U+02DC, U+2000-206F, U+2074, U+20AC, U+2212, U+2215

Result:
_sass/_fonts.sass
  16:5   Expected indentation of 0 space but found 4.     (indentation)
  17:5   Expected indentation of 0 space but found 4.     (indentation)
  18:5   Expected indentation of 0 space but found 4.     (indentation)
  19:5   Expected indentation of 0 space but found 4.     (indentation)
  22:5   Expected indentation of 0 space but found 4.     (indentation)
  23:5   Expected indentation of 0 space but found 4.     (indentation)
  24:5   Expected indentation of 0 space but found 4.     (indentation)
  25:5   Expected indentation of 0 space but found 4.     (indentation)
  26:5   Expected indentation of 0 space but found 4.     (indentation)

If I remove the unicode-range: lines, the error goes away. If I move the unicode-range lines so they are not the last line in a block, then the errors go away (however new errors regarding property-sort-order appear because of my other sass-lint settings).

Note: The unicode-ranges match the same pattern that Google uses:

https://fonts.googleapis.com/css?family=Lato:400,400i,700


refactor(cmd-server-socket,validator): [draft] refactor the directory structure of cmd-server-socket and validators

resolving #1233 (work-in-progress)
status: Draft

Signed-off-by: Takuma TAKEUCHI<EMAIL_ADDRESS>

closed by #719


Upgrade Next.js and Jest to latest versions

Acceptance Criteria

[ ] Upgrade Next.js;
[ ] Upgrade Jest;
[ ] Update unit tests;
[ ] Tested

Assets

Mockup:

NOTES

Since the release of Next.js 12, Next.js now has built-in configuration for Jest. So it would be great to implement it as well https://nextjs.org/docs/testing#setting-up-jest-with-the-rust-compiler;
Sometimes unit tests fails without any reason. That's why we need to reorganize all unit tests to avoid failing

Next.js upgrade is now addressed

Jest upgrade is now addressed

I'm not sure what was meant by "update unit tests", but I added proper CSS snapshotting and would rather link it to this issue.


BTSG airdrop showing up as multiple deposits to wallet, when only one deposit occurred

Hello, and thank you for writing this tool. It's a lifesaver
I recently did an export of my wallet, and when looking at the exported data, it would appear that [Osmosis BTSG Pool 560] reward incentives or airdrop (https://www.mintscan.io/osmosis/txs/57E82EA67EA00C54EC6068F2860E0DA227532C5F5013745F7071808FD4786471) were reported as deposits into my wallet. The following line is repeated in the exported CSV 41 times.
2021-12-31 17:03:02,,,<AMOUNT-HERE>,BTSG,,,,,,,57E82EA67EA00C54EC6068F2860E0DA227532C5F5013745F7071808FD4786471-560

The issue is that I only received the amount just once, and this has resulted in nearly every tax tool thinking that I earned the entire USD value of the airdrop. 😧
I've been able to manually verify and fix the CSV locally, but wanted to bring this to your attention in case someone else runs into it 😅

Oi, thanks for the awesome details and description!  Will investigate asap

Do you mind DM'ing (@staketax or<EMAIL_ADDRESS>me your wallet address or bookmark url? Looks like it's an issue specific to your wallet address for that airdrop.  If not comfortable, no worries.  Just got I'd ask.

oh wait nm, I think I can figure out something.

Just checked: seems like it's currently working for the above implied transaction/address.
I thought I fixed this issue recently-ish, but my memory is fuzzy exactly when (maybe like a week ago).  So I'm wondering if you created the CSV possibly before the fix?
If you created this CSV after my fix, do you mind DM'ing wallet address or bookmark url?  I believe it would likely be an address-specific issue in regards to the airdrop transaction.  If not comfortable, no worries.

Hi, sorry for not responding as quickly. I had filed this bug during my lunch break 😅
Regarding when the export finished, it was actually 30 minutes before I filed this issue so unless you deployed sometime today I don't think it would have been from that.
That said, I completely forgot to bookmark the data export, and while I would normally be fine with sending stuff like this over I kind of need to think it over and process whether I'm comfortable with sharing that or not. Give me some time to think it over? (also I assume by DM you mean on twitter 😅)

Yup, Twitter DM.
Of course, take your time and cautiousness is well understood.
On Thu, Jan 6, 2022 at 5:49 PM Isabella Muerte @.***>
wrote:

Hi, sorry for not responding as quickly. I had filed this bug during my
lunch break 😅
Regarding when the export finished, it was actually 30 minutes before I
filed this issue so unless you deployed sometime today I don't think it
would have been from that.
That said, I completely forgot to bookmark the data export, and while I
would normally be fine with sending stuff like this over I kind of need to
think it over and process whether I'm comfortable with sharing that or not.
Give me some time to think it over? (also I assume by DM you mean on
twitter 😅)
—
Reply to this email directly, view it on GitHub
https://github.com/hodgerpodger/staketaxcsv/issues/18#issuecomment-1007074439,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATDGEKZJ5NYP3BSFCIWS43UUZBCRANCNFSM5LNG7P6A
.
  I found an example and deployed a fix.  Thanks for reporting the issue!

Thank you! I had actually completely spaced DMing you my info on twitter. Glad it worked out in the end 🙂


refactor iavl cache max represented in terms of both bytes and the number of nodes

Closing this in light of further work going in IAVL / key refactoring changing the performance tradeoffs at hand


Refactoring the login screen

Contents
- Login Screen 컨테이너 간격 조정
- 재사용 위젯 확인 후 리팩토링
- Add Skeleton Manage Screen
- Route to Manage Screen
- Different font weights for each weight (NotoSansKR)

Contents
- Login Screen 컨테이너 간격 조정
- 재사용 위젯 확인 후 리팩토링
- Checking the input type is null with snackbar
- Add Skeleton Manage Screen
- Route to Manage Screen
- Different font weights for each weight

Completed

[x] Login screen 컨테이너 간격 조정
[x] Add Skeleton Manage screen
[x] Route to Manage screen
[x] Different font weights for each weight

Not completed

[ ] 재사용 위젯 확인 후 리팩토링
[ ] Checking the input type is null with snackbar
[ ] Add Skeleton Member screen
[ ] Route to Member screen

https://user-images.githubusercontent.com/26790710/210765334-a31d95e9-8a39-4bf5-8598-b5557ecac2e8.mov

Completed

[x] Login screen 컨테이너 간격 조정
[x] Add Skeleton Manage screen
[x] Route to Manage screen
[x] Different font weights for each weight
[x] 재사용 위젯 확인 후 리팩토링
[x] Add Skeleton Member screen
[x] Route to Member screen

Not completed

[ ] Checking the input type is null with snackbar


Pycoral with GRPC error

While utilizing the GRPC python library on a raspberry pi, once the import grpc line of code is run, the load delegate function will fail to connect to the USB Coral accelerator.  Can someone please advise what steps I can take?
Thank you.
python3-pycoral version: 1.0.1
Using load delegate:

Using a detection engine:

@bomerzz I see that you are working with virtual environment. Is the USB Accelerator visible in your environment.
Can you share the output of 'lsusb' ?

Hi @manoj7410  above is the lsusb output

@bomerzz lsusb output looks okay. Are you able to run the demo given at : https://coral.ai/docs/accelerator/get-started/#3-run-a-model-on-the-edge-tpu ?

Yes i am able to however if i add import grpc to the demos it will not run as well

Okay.  Can you paste the detailed error here ?


Request via proxy?

Hi,
thanks for the great effort building this module!
Is there a possibility to fetch data (goto(url)) via a proxy like in puppeteer?
Kind regards

I figured out how to add the proxy-server. Mainly inspired by #176.
const chromeLauncher = require('chrome-launcher');
const Chromeless = require('chromeless').Chromeless;

const run = async () => {
  const chrome = await chromeLauncher.launch({
    port: 9222,
    chromeFlags: [
      '--window-size=1200,800',
      '--disable-gpu',
      '--headless',
      '--proxy-server=http://www.proxy.com:9999'
    ]
  });
  console.log('Chrome debuggable on port: ' + chrome.port);
  const chromeless = new Chromeless({
    launchChrome: false
  });
  const t1 = new Date().getTime();
  return await makeScreenshot(chromeless);
};

const makeScreenshot = async chromeless => {
  return await chromeless
    .goto('https://www.amazon.com')
    .type('harry potter ', 'input[id="twotabsearchtextbox"]')
    .click('.nav-input')
    .wait(2000)
    .screenshot();
};

run();

Did you run this proxied on Lambda by chance?  I'm dying for a solution to get it to run proxied on Lambda is the reason I ask...

@jborden13 that's what I wanted to test next.
Maybe just adding the option remote: true could work, like in this example:
  const chromeless = new Chromeless({
    launchChrome: false,
    remote: true
  });

If this doesn't work with the local version, one could try the approach suggest by @adieuadieu (#176) by uploading the local chromless version to AWS lambda.
Another possibility would be setting up a little express server which handles your requests to the proxy server like .goto('http://www.yourexpress.com?encodedurl'). This works for sure, but that's not an elegant solution :-)

@Raidus thanks for the thoughts.  tried using your chrome-launcher approach, but didn't have success.  When calling Lambda, you have to define the endpoint url and api key in the remote property, so unfortunately setting it to true didn't work.
I'll give the approach @adieuadieu recommends in #176 a shot next.
Not sure I follow what you are recommending in your final paragraph.  Are you're suggesting setting up an express endpoint that calls Lambda?  If so, that's what I already have, but it's still the Lambda IP on every request to the end page.  Please advise if I missed your suggestion.
Also, please please let me know you are ever able to get Lambda proxied.  Following this issue closely and have been banging my head against it for too long.

@jborden13 if you just set it to true, then you need to export your credential before or just set it in javascript
const chromeless = new Chromeless({
  remote: {
    endpointUrl: 'https://XXXXXXXXXX.execute-api.eu-west-1.amazonaws.com/dev',
    apiKey: 'your-api-key-here',
  },
})

Unfortunately remote doesn't work in my case, see #273.


Rework analysis_server to clear flakiness

The main thrust of this change is to stop using analysisServer.analysis.onError, and instead use analysisServer.analysis.getErrors.
Events come through analysisServer.analysis.onError when adding and removing (I think) overlays, as well as events on analysisServer.server.onStatus with isAnalyzing = false, so that it is very tricky to handle across asynchronous code which errors and complete analyses are for a given analysis request.
All of the other cleanup (tons of deleted code, simplified warmups, simplified tests) follows from that one change.

@scheglov would you mind reviewing this? I think you can help indicate if we are using any specifically outdated APIs here.


How to add (Edit &Delete) buttons in datatable?

That is not in the scope of this library. You will need to read the docs over at https://datatables.net/ or probably google for a DataTables delete button.
You probably just need to use the select plugin, then add a button to get the selected row, then fire a DELETE request at your controller with a row ID.


Non-GUID link updates

Addresses Part 3(2) of #2580
Converts Non-GUID MSDN links ...
https?://msdn.microsoft.com(/en-us)?/library/[A-Za-z]{2}[\d]{6}\(v=vs\.[\d]{2,3}\).aspx

All link changes are logged. No bad links were found on this pass.
log-nonGUID.txt (CSV file)

The build failed, but its status is not reported here, and the build report appears to be unavailable. Closing and reopening to start new build.

@guardrex,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@guardrex,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


Implement gridstack min/max item dimension

Hi ! I found a big help in implementing your library.
I propose you to add better implementation of gridstack.js capabilities.
The new options for a widget are :

minWidth : applied with gs-item-min-width attribute
minHeight : applied with gs-item-min-height attribute
maxWidth : applied with gs-item-max-width attribute
maxHeight : applied with gs-item-max-height attribute

I let you generate the dist file :)
btw : it's my first PR so I don't know which information you would need

Hi @Zacaria,
thank you for your PR. 👍  Could you please fix the indentations so that we can see better which lines have changed? See: https://github.com/kdietrich/gridstack-angular/pull/8/files Thanks!

Would you please tell me the name of the linter/formatter you are using ?

@Zacaria I've added jscs. Could you resubmit your PR? Thanks!

Hi, @Zacaria @kdietrich
I think there is no need to watch the "data-gs-min-height", "data-gs-min-width", "data-gs-max-height", "data-gs-max-width" in gridstackitem.directive.js. These four properties can't be changed by user.
PS: It is still very hard to see the changed part because of the indentation...


python311Packages.sievelib: 1.2.1 -> 1.3.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python311Packages.sievelib is: Client-side Sieve and Managesieve library written in Python
meta.homepage for python311Packages.sievelib is: https://github.com/tonioo/sievelib
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.3.0 with grep in /nix/store/kvp63psd5fj7za07vqshsjq88v79wlic-python3.11-sievelib-1.3.0
found 1.3.0 in filename of file in /nix/store/kvp63psd5fj7za07vqshsjq88v79wlic-python3.11-sievelib-1.3.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
python311Packages.sievelib
python312Packages.sievelib

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/kvp63psd5fj7za07vqshsjq88v79wlic-python3.11-sievelib-1.3.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/27789992b620a5afd988f44a9b27797a716998ca#python311Packages.sievelib

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/kvp63psd5fj7za07vqshsjq88v79wlic-python3.11-sievelib-1.3.0
ls -la /nix/store/kvp63psd5fj7za07vqshsjq88v79wlic-python3.11-sievelib-1.3.0/bin

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

4 packages built:

python311Packages.sievelib
python311Packages.sievelib.dist
python312Packages.sievelib
python312Packages.sievelib.dist

Maintainer pings
cc @leenaars for testing.

Result of nixpkgs-review pr 300991 run on x86_64-linux 1

4 packages built:

python311Packages.sievelib
python311Packages.sievelib.dist
python312Packages.sievelib
python312Packages.sievelib.dist


Intro

Name: Ryan Oillataguerre

GitHub: oillataguerre

Twitter: rmoillataguerre

LinkedIn: https://www.linkedin.com/in/ryan-oillataguerre-42924110b?trk=hp-identity-name

Completed Courses

@oillataguerre please add your profile in the issue Students Profiles #6.
Thank you! :smile:


My own notes (ipynb) with a little tweaking.

Unfortunately, the provided changes are made in the notebook and we actually make the changes in the python scripts from the python_scripts folder and generate the notebook from it.
It would be easier to review the changes.
Maybe you can write a summary in this PR regarding the changes that you wanted to provide us?


Get Cache-Time Endpoint

What
Added new GetCacheTime endpoint with 200 response.
How to review
In order to check the endpoint is correctly setup, you will need to:

Have mongodb shell installed (https://www.mongodb.com/docs/mongodb-shell/install/)
Run mongodb in a docker container (use command from readme).
Open a new terminal and enter mongosh - Your terminal should now be prefixed with 'test'
Then insert a document into our datasets collection by running:
db.datasets.insertOne({  _id: “123”,   path: "hello",  etag: "testetag",  collectionId: 1234,  releaseTime: new Date("2023-12-12T00:00:00Z")})
Run the app with make debug
Using postman or similar, make a GET request to localhost:29100/v1/cache-times/123
You should see a 200 status code with the above inserted document in the response.

Who can review
A member of the ONS

Approved on the proviso that distinctions between 500, 404 and 400 response and provided on a future PR.


MM-31243: broadcast channel tweaks

Summary
Keep showing Unknown Channel when a private channel is configured as the broadcast channel in the backstage, but restore showing Select a channel when none is selected at all.
As part of writing the unit tests here, I discovered it was impossible to type a channel name: search was broken. I restored this behaviour with a simple includes scan on id, name and display name.
Ticket Link
Fixes: https://mattermost.atlassian.net/browse/MM-31243

/check-cla


Extending of ampersand with pseudo class doesn't work in new version

I have a piece of code with pseudo class extending, that worked perfectly in v2.0.4, but after update to v2.1.0 it doesn't work anymore:
.site-nav_link {
  ...
  ...
  &:hover {
    color: #cccccc;

   &:before {
      opacity: 1;
    }
    .site-nav_label {
      padding-left: 5px;
    }
  }

 &[data-active="true"] {
    @extend &:hover;    //<<<<<<------- it doesn't work in v2.1.0

    &.site-nav_link--exp:after {
      content: $icon-arrow-top;
    }
  }
}

Message from gulp compiler:
Error in plugin 'gulp-sass'
Message:
    ..\src\main\assets\scss\components\_filename_.scss
Error: Can't extend &:hover: can't extend parent selectors

       Backtrace:
        ../src/main/assets/scss/components/_filename_.scss:144
        on line 144 of ../src/main/assets/scss/components/_filename_.scss
>>     @extend &:hover;
   ------------^

@malcheg how do you solved your problem ?