[optimize] Replace dubbo-demo-native

What is the purpose of the change?
#14878

Remove dubbo-demo-native
Modify dubbo-demo-springboot to support native functionality
Modify related github-action files

Checklist

[x] Make sure there is a GitHub_issue field for the change.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[x] Make sure gitHub actions can pass. Why the workflow is failing and how to fix it?

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

Project coverage is 2.19%. Comparing base (ef6e8bd) to head (053f43c).
Report is 55 commits behind head on 3.3.

:exclamation: Current head 053f43c differs from pull request most recent head b52a631
Please upload reports for the commit b52a631 to get more accurate results.

Additional details and impacted files
@@              Coverage Diff              @@
##              3.3    #14877        +/-   ##
=============================================
- Coverage   37.53%     2.19%    -35.34%
=============================================
  Files        1813     26083     +24270
  Lines       80737   1389674   +1308937
  Branches    12005    220210    +208205
=============================================
+ Hits        30304     30545       +241
- Misses      46042   1354685   +1308643
- Partials     4391      4444        +53

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

@CrazyHZM PTAL


Bug: Book Not Getting Added to Cart

If we add any book to the cart it isn't getting added to the cart. And after clicking on Purchase we get another Prompt "Item Added to Cart".
Uploading BookTown HomePage - Google Chrome 2023-10-03 11-19-20.mp4…

https://github.com/Karamraj/BookTown/assets/92857243/426393d2-0423-4174-a611-2ebd9a244198

Hey @Karamraj, can you please assign me this under Hacktoberfest tag?

@Yasir761 Assigned.
Join The Discord of All Contributors of this repo - https://discord.gg/3E8QXXps

@Karamraj, I guess it is happening because of CORS policy, because when I ran it locally it's working fine.

Oh Okay.


Add fallback linker options.

Description of the Change: Stablish a linker fallback order to check for available linkers at run time. Place the user preferred linker at position 0 (top priority). Raise error messages and warnings.
Benefits: Increased reliability
[sc-34425]

@josh146 I revived the troubleshooting/debugging tips section in the #33. I do not believe this change should go in the getting started as I don't see it providing any new functionality to the user, just allowing the user to provide a suitable environment if the default is not available in their system. I'll add the documentation as part of that PR, independent of this one.


Example for using google contacts api

Hello,
i want to manage google contacts with an excel sheet. i tried to create a vba-procedure to query contacts from a google account, but it doesn't work:
Sub testContacts()
    Dim Client As New WebClient
    Dim Auth As New GoogleAuthenticator

    EnableLogging = True

    Auth.Setup _
      ClientId:="<myId>", _
      ClientSecret:="<mySecret>"
    Auth.Scopes = Array("https://www.google.com/m8/feeds")
'    Auth.ApiKey = "<optionalMyApiKey>"
    Auth.Login

    Set Client.Authenticator = Auth
    Client.BaseUrl = "https://www.google.com/m8/feeds"

    Dim Request As New WebRequest
    Dim Response As WebResponse

    'Request.Resource = "contacts/{userEmail}/full"
    Request.Resource = "contacts/default/full"

    Request.Method = WebMethod.HttpPut
    Request.RequestFormat = WebFormat.FormUrlEncoded
    Request.ResponseFormat = WebFormat.Json

    Set Response = Client.Execute(Request)

End Sub

Result:
ERROR - GoogleAuthenticator.GetToken: -2147210463 (11041 / 80042b21), An error occurred while retrieving token.
408:
ERROR - WebClient.Execute: -2147210493 (11011 / 80042b03), An error occurred during execute
-2147210492 (80042b04): An error occurred while preparing http request
-2147210463 (80042b21): An error occurred while retrieving token.
408:

User-Agent: VBA-Web v4.0.20 (https://github.com/VBA-tools/VBA-Web)
Content-Type: application/x-www-form-urlencoded;charset=UTF-8
Accept: application/json
Content-Length: 0

<-- Response - 12:29:42
408 Request Timeout

Any idea for my problem?

There are two issues that I can think of that would cause a timeout:

For slow connections, you can raise the timeout milliseconds with the following: Client.TimeoutMs = 30000 ' 30 seconds
It could be a proxy issue, in which case you can use Client.EnableAutoProxy = True. (This is disabled by default currently, but has been proposed to make enabled by default in a future version of VBA-Web)

Give these a try and let me know if it's still not working.

Okay, it is the proxy solution.
Now i get the following error:
PUT https://www.google.com/m8/feeds/contacts/default/full?key=<MyKey>
User-Agent: VBA-Web v4.0.20 (https://github.com/VBA-tools/VBA-Web)
Content-Type: application/x-www-form-urlencoded;charset=UTF-8
Accept: application/json
Content-Length: 0

ERROR - WebHelpers.ParseByFormat: 11000, An error occurred during parsing
10001: Error parsing JSON:
<html><head
^
Expecting '{' or '['
ERROR - WebHelpers.ParseByFormat: 11000, An error occurred during parsing
10001: Error parsing JSON:
<html><head
^
Expecting '{' or '['
<-- Response - 12:48:44
401 Authorization required
Cache-Control: private, max-age=0
Date: Tue, 19 Jan 2016 11:48:44 GMT
Transfer-Encoding: chunked
Content-Type: text/html; charset=utf-8
Expires: Tue, 19 Jan 2016 11:48:44 GMT
Accept-Ranges: none
Server: GSE
Vary: Accept-Encoding
X-Content-Type-Options: nosniff
X-Frame-Options: SAMEORIGIN
X-XSS-Protection: 1; mode=block

<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<title>Error
401
(Client Error)!!1</title>
<style type="text/css">

I get the same error or error page when i execute the url in my browser.

That's occurring because they are sending an html error page that is failing to be parsed as json. It looks like the error code, 401 (seen in the <title> of the response), is an authentication error. I would look at the Response.Content which includes the unconverted response for more details on the error. My guess is that you can't use a PUT request with that resource (at least without including something in the Body).

dumperios,
Try to open at browser http://www.google.com/accounts/DisplayUnlockCaptcha for the first time before sending request.

When i use the google login screen (not the apikey) then i get another result as follows:
GoogleAuthenticator.Login: Login succeeded: 4/2whnhWduXCfmwI2Tw3Tnonn-60wvfG6ucCNDBFqjavU
VBA-Web: Loaded auto-proxy for https://accounts.google.com:443:
Server = ...
WebClient.PrepareHttpRequest: SetProxy: ...
--> Request - 13:17:10
POST https://accounts.google.com/o/oauth2/token
User-Agent: VBA-Web v4.0.20 (https://github.com/VBA-tools/VBA-Web)
Content-Type: application/x-www-form-urlencoded;charset=UTF-8
Accept: application/json
Content-Length: 253

code=4%2F2whnhWduXCfmwI2Tw3Tnonn-60wvfG6ucCNDBFqjavU&client_id= ... &client_secret= ... &redirect_uri=urn%3Aietf%3Awg%3Aoauth%3A2.0%3Aoob&grant_type=authorization_code

<-- Response - 13:17:10
200 OK
Cache-Control: no-cache, no-store, max-age=0, must-revalidate
Date: Thu, 21 Jan 2016 12:17:13 GMT
Pragma: no-cache
Transfer-Encoding: chunked
Content-Type: application/json; charset=utf-8
Expires: Fri, 01 Jan 1990 00:00:00 GMT
Accept-Ranges: none
P3P: CP="This is not a P3P policy! See https://support.google.com/accounts/answer/151657?hl=en for more info."
Server: GSE
Set-Cookie: NID=75=... ;Domain=.google.com;Path=/;Expires=Fri, 22-Jul-2016 12:17:13 GMT;HttpOnly
Vary: Accept-Encoding
X-Content-Type-Options: nosniff
Content-Disposition: attachment; filename="json.txt"; filename*=UTF-8''json.txt
X-Frame-Options: SAMEORIGIN
X-XSS-Protection: 1; mode=block
Alternate-Protocol: 443:quic,p=1
Alt-Svc: quic=":443"; ma=604800; v="30,29,28,27,26,25"
Cookie: NID=75=...
{
  "access_token" : "...",
  "token_type" : "Bearer",
  "expires_in" : 3600,
  "refresh_token" : "..."
}

VBA-Web: Loaded auto-proxy for https://www.google.com:443:
Server = ...
WebClient.PrepareHttpRequest: SetProxy: ...
--> Request - 13:17:10
PUT https://www.google.com/m8/feeds/contacts/<email>/full
Authorization: Bearer ...
User-Agent: VBA-Web v4.0.20 (https://github.com/VBA-tools/VBA-Web)
Content-Type: application/x-www-form-urlencoded;charset=UTF-8
Accept: application/json
Content-Length: 0

ERROR - WebHelpers.ParseByFormat: 11000, An error occurred during parsing
10001: Error parsing JSON:
Not Found
^
Expecting '{' or '['
ERROR - WebHelpers.ParseByFormat: 11000, An error occurred during parsing
10001: Error parsing JSON:
Not Found
^
Expecting '{' or '['
<-- Response - 13:17:11
404 Not Found
Cache-Control: no-cache, no-store, max-age=0, must-revalidate
Date: Thu, 21 Jan 2016 12:17:13 GMT
Pragma: no-cache
Transfer-Encoding: chunked
Content-Type: text/html; charset=UTF-8
Expires: Fri, 01 Jan 1990 00:00:00 GMT
Accept-Ranges: none
Server: GSE
Vary: X-Origin
Vary: Origin,Accept-Encoding
X-Content-Type-Options: nosniff
X-Frame-Options: SAMEORIGIN
X-XSS-Protection: 1; mode=block

Not Found

It seems that the login works but the request now gets an error with code 404 - Not Found. Thats the values in request.content. Another question: is there a working example with vba-tools to read/write google contacts?

Hello dumperios,
Did You manage to access Google contacts with VBA code ?
I'm also interested to try this...

Hi ovironelectronics,
no, iam still trying. i tested to copy the gmail-example in the excel-document which is working. The authorization to google contacts by ie works but the query throws an error. Now i have to step deeper into the vba-code.

--> Request - 13:17:10
PUT https://www.google.com/m8/feeds/contacts//full

There is a double // in the URL - which could be the problem. According to your original code should there be a userid in there?

Closing to track in general examples issue


When scrolling the list page, the page displays empty state if api fails while loading more users

Current behavior

When loading more users on the users list page and if the api fails then the page displays an empty state
Expected behavior

In this case when api fails while loading more users, the previous loaded content should not be removed and empty state should also not get displayed.
Steps to reproduce the issue

Can you handle fixing this bug by yourself?

[x] YES
[ ] NO

Environment details

Browser:
OS:
Code Version:

Additional information

Issue verified and working fine in release v.1.9.0. Hence, closing this ticket.
https://jam.dev/c/f4e4bc7e-b082-4c12-b886-4e04b1a34d32


Flow complains about undefined within an "if" that guards against just that

Flow 0.46 (Windows)
Two issues in one:

The complaint about undefined inside an if statement that guards against just that

That this error only occurs when some other completely unrelated element inside the same string exists (function fn1: error, function fn2: no error)

// @flow
'use strict';

type MyRule = {|
    jsType?: string
|};

const fn1 = function (rule: MyRule): void {
    const jsTypeStrings = ['string', 'boolean', 'number', 'object'];

    if (rule.jsType !== undefined) {
        throw new TypeError(
            `Must be ${jsTypeStrings.toString()} but is ${rule.jsType}`
        );
    }
};

const fn2 = function (rule: MyRule): void {
    if (rule.jsType !== undefined) {
        throw new TypeError(
            `Is ${rule.jsType}`
        );
    }
};

Leads to an error inside fn1 but not inside fn2
$ flow check
src/test.js:13
13: `Must be ${jsTypeStrings.toString()} but is ${rule.jsType}`
                                                  ^^^^^^^^^^^ undefined. This type cannot be coerced to
13: `Must be ${jsTypeStrings.toString()} but is ${rule.jsType}`
    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ string

Found 1 error

Link to demo

EDIT:
When I use a modified fn1 I find that anything that does a string-conversion somewhere else in the string triggers the error, for example
const fn1 = function (rule: MyRule): void {
    if (rule.jsType !== undefined) {
        throw new TypeError(
            `Must be ${String('string')} but is ${rule.jsType}`
        );
    }
};

shows the error too but
const fn1 = function (rule: MyRule): void {
    if (rule.jsType !== undefined) {
        throw new TypeError(
            `Must be ${'string'} but is ${rule.jsType}`
        );
    }
};

does not.

the problem is that type refinements are very brittle, ie. they are invalidated if you call another function (in your case jsTypeStrings.toString()). the workaround is to store the refined value in a const variable:
const fn1 = function (rule: MyRule): void {
    const jsTypeStrings = ['string', 'boolean', 'number', 'object'];

    if (rule.jsType !== undefined) {
      	const type = rule.jsType;
        throw new TypeError(
            `Must be ${jsTypeStrings.toString()} but is ${type}`
        );
    }
};

Type refinements in the docs


Support restart of test when it crashes

As discussed today, there's a use case for restarting a test when it crashes:
09:26:12                 out: :: [ 14:26:12 ] :: [   PASS   ] :: Command 'make all' (Expected 0, got 0)
09:26:12                 out: :: [ 14:26:12 ] :: [  BEGIN   ] :: Running 'echo 1 > /sys/kernel/vkm/write_um_crash'
09:26:12                 out: ./tmt-test-wrapper.sh.default-0: line 1:  6543 Segmentation fault      bash ./write_um.sh
09:26:12                 out: Shared connection to <IP_ADDRESS> closed.
09:26:12         Command returned '139'.

In this case, the user would like to see the test restarted - the test was killed by a kernel oops, and when restarted, it would take care of follow-up steps, like decoding the kernel dump.
After some discussion, the proposal would be:

a test key to indicate the test shall be restarted when it crashes. Might be a list of existing codes, or tmt might define the list of crash-like exit codes, and this key would be a simple flag.
a test key to indicate how many times the test should be restarted. We need to avoid endless loops, and tmt should give up at some point. The default might be a zero, or a reasonably low value - the value would not be used unless the first key is enabled anyway.
a test key to indicate whether to reboot the guest before restarting the test. In this particular case, there should be no guest reboot, the test needs to re-enter the environment as it is.
ew environment variable, similar to TMT_REBOOT_COUNT, but counting test restarts. With reboot disabled, the test might run multiple times while TMT_REBOOT_COUNT remains zero.

https://github.com/teemtee/tmt/pull/2787

@pablmart hello, could you share the test? I'd like to use it as a reproducer when working on the feature.

@pablmart hello, could you share the test? I'd like to use it as a reproducer when working on the feature.

Yes the test is on the same repo linked in the above comment mentioning 'rstrnt-prepare-reboot':
kernel-stack-overflow-udnerflow-scribbling

I encountered a similar problem when testing ftrace= kernel parameter with tmt run.
Test with auto-osbuild-qemu-rhivos9-qa-ostree-aarch64-7874633.e1769674.qcow2.xz by manual
The available tracers are:
$cat /sys/kernel/debug/tracing/available_tracers
timerlat osnoise hwlat blk function_graph wakeup_dl wakeup_rt wakeup function nop

Install a vm with above image

export CMDLINEARGS="ftrace="

rpm-ostree kargs --append-if-missing="${CMDLINEARGS##-}" --import-proc-cmdline

systemctl reboot

Then the host cannot ssh connect again. Only "timerlat" and "osnoise" make host panic.

Kicking off the implementation of the actual test restart in https://github.com/teemtee/tmt/pull/2870. It does have some rough edges, although there is a test that passes.
I plan to run it with the kernel-stack-overflow-udnerflow-scribbling test provided by @pablmart, feel free to experiment too.
One piece we need to address ASAP - naming. I picked some names for new keys, but they are ugly and I don't like them. I can change them easily, but I'm out of ideas - feel free to propose changes here as well, besides the actual bugs and issues :)

A similar case: what if the test does not crash, but triggers a reboot, e.g. through Ansible role, unable to use tmt-reboot? This would manifest as a broken SSH session:
                out: TASK [sap_general_preconfigure : Flush handlers] *******************************
                out:
                out: RUNNING HANDLER [sap_general_preconfigure : Reboot the managed node] ***********
                out: Shared connection to restqe01 closed.
            cmd: rsync --version
            err: ssh: connect to host restqe01 port 22: Connection refused
            cmd: dnf --version
            err: ssh: connect to host restqe01 port 22: Connection refused
            cmd: rpm-ostree --version
            err: ssh: connect to host restqe01 port 22: Connection refused
            cmd: yum install -y rsync
            err: ssh: connect to host restqe01 port 22: Connection refused

The MR 2870 solves the issue with the [kernel-stack-overflow-underflow-scribbling] (https://gitlab.com/pablmart.rh/centos-kernel-tests/-/tree/kernel-stack-overflow-underflow-scribbling/memory/mmra/kernel-stack-overflow-underflow-scribbling) test. Many thanks!

Hi @pablmart,
This is Coiby from the kernel debug sst. I'm considering adopting tmt for https://github.com/rhkdump/kdump-utils tests. In our tests, we need to trigger a kernel crash intentionally and then check if the crash dump can be collected. I want to study your kernel-stack-overflow-underflow-scribbling test to learn to make use #2870 but it's gone now. Can you re-share it with me? Thanks!

Hi @happz,
I wrote a mutihost test which is to dump a kernel crash to a remote NFS server . but unfortunately it failed with an error,
                        kdump: Starting kdump: [OK]
                        :: [ 10:08:20 ] :: [   PASS   ] :: Command 'kdumpctl restart' (Expected 0, got 0)
                        :: [ 10:08:20 ] :: [  BEGIN   ] :: Running 'echo 1 > /proc/sys/kernel/sysrq'
                        :: [ 10:08:20 ] :: [   PASS   ] :: Command 'echo 1 > /proc/sys/kernel/sysrq' (Expected 0, got 0)
                        client_loop: send disconnect: Broken pipe
                    journal.txt: /var/tmp/tmt/run-035/plans/kdump/execute/data/guest/client/client-test/tests/client-4/journal.txt

I expect #2870 to restart the test after a kernel panic
diff --git a/tests/client/main.fmf b/tests/client/main.fmf
index d74446f..261b8bb 100644
--- a/tests/client/main.fmf
+++ b/tests/client/main.fmf
@@ -1,3 +1,5 @@
 summary: Dump kernel crash to an NFS server
 test: ./test.sh
 framework: beakerlib
+restart-on-exit-code: 79
+restart-max-count: 5

But unfortunately it doesn't work. Do I miss anything? Thanks!

Oh, I notice the error is from beakerlib
00:00:28 errr /client-test/tests/client (on client) (beakerlib: State 'started') [1/1]
# or
00:00:28 errr /client-test/tests/client (on client) (beakerlib: State 'imcomplete') [1/1]

And after digging into the documentation and some trial-and-error, eventually I find using tmt-reboot -c "echo c > /proc/sysrq-trigger" to trigger a kernel panic instead of rlRun "echo c > /proc/sysrq-trigger" can lead to green test results.


fix(html/minifier): small merge fix

Description:
Small fix
BREAKING CHANGE:
No
Related issue (if exists):
No

@kdy1 Fixed :+1:


Training error. bg_num_rois = 0 and fg_num_rois = 0, this should not happen!

Hi, I meet some proｂlems when training.
The error message is as follows:
ValueError: bg_num_rois = 0 and fg_num_rois = 0, this should not happen!

And I find before the error, the loss has turned to nan, and I followed some suggestions like climp gradient or reduce lr, none of them worked.
[session 1][epoch  1][iter  300/2164] loss: nan, lr: 1.00e-04
			fg/bg=(128/0), time cost: 29.000118

I checked my annotation files, some xmin is 0, I don't know if it is the problem, because I plus xmin to 1, it's not work.
And I print gt_boxes and I found xmin is more than 64041, appａrently it's not right.
gt_boxes is tensor([[[6.4041e+04, 1.7687e+02, 2.2182e+02, 4.3876e+02, 2.0000e+00],
         [0.0000e+00, 0.0000e+00, 0.0000e+00, 0.0000e+00, 0.0000e+00],
         [0.0000e+00, 0.0000e+00, 0.0000e+00, 0.0000e+00, 0.0000e+00],
         [0.0000e+00, 0.0000e+00, 0.0000e+00, 0.0000e+00, 0.0000e+00],
         [0.0000e+00, 0.0000e+00, 0.0000e+00, 0.0000e+00, 0.0000e+00]]],

So I think there is somewhere wrong about compute the gt_boxes in your code, but it hard to find out, could you give me a clue about how to fix it?
Thank for your kindly reply!

Check #349

Hi, I found the same bug while trying to create my own data with the images from OpenImage for the Kaggle competition.
Check in: https://github.com/jwyang/faster-rcnn.pytorch/blob/358cecacf876717ff13988dc6396de10e265279c/lib/datasets/pascal_voc.py#L234-L237 of your new generated dataset .py file e.g: openimage.py I recommend you copy the pascal_voc.py and work from there. Delete the -1.
Moreover, change in:
https://github.com/jwyang/faster-rcnn.pytorch/blob/358cecacf876717ff13988dc6396de10e265279c/lib/datasets/imdb.py#L121-L122
delete the -1.
There are objects where the bbox are 0,1,0, for example, which makes either the code the crash due to an assertion error or the loss to become nan. If you are using a dataset with some bbox annotations that are either 0 or equal to the image width, apply the changes.
hope it helps! :)

make sure x2 and y2 < width because it will flip image and annotation
        wh = tree.find('size')
        w, h = int(wh.find('width').text), int(wh.find('height').text)
        for ix, obj in enumerate(objs):
            bbox = obj.find('bndbox')
            # Make pixel indexes 0-based
            x1 = float(bbox.find('xmin').text)
            y1 = float(bbox.find('ymin').text)
            x2 = float(bbox.find('xmax').text)
            y2 = float(bbox.find('ymax').text)
            x1 = max(x1, 0)
            y1 = max(y1, 0)
            x2 = min(x2, w)
            y2 = min(y2, h)


Response body is lost when using dapr client

When using dapr client to send an http request. If the server returns a response with a status code which is in a range outside of 200-300 and the response contains a body describing error details, the response content may still be discarded
The function convert_to_error() of class DaprHttpClient assumes that the error response contains keys “message” and “errorCode” in desterilized error_info.
In our case, for example, the error response looks like this
{'requestId': 1, 'status': 'Failed', 'statusDescription': ‘error reason is here’}
DaprInternalError which is raised in this case looks like this:
dapr.clients.exceptions.DaprInternalError: (None, 'UNKNOWN')
And we lost the useful information about error reason.
Suggestion: when wrapping error response into DaprInternalError always include response body into exception

We generally recommend performing service invocation directly via HTTP libraries (like requests) instead at this time, rather than using the Dapr SDK. Our documentation on docs.dapr.io has been updated accordingly.
I opened a PR to add raw_response_bytes to the DaprInteralError class, so that you can get the complete response the way it is being received.

Fixed in 1.12.0


[ISSUE #3394] fix duplicate keys in trace message

Make sure set the target branch to develop
What is the purpose of the change
#3394
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice, it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR.

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test(over 80% coverage) to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Coverage decreased (-0.03%) to 54.475% when pulling 80ca16c52fc142a39a3a12489fe8fb50235a2fa9 on yuz10:develop3 into 94a527c339af0308b5d43fd1d404766889ab04a4 on apache:develop.

Codecov Report

Merging #3395 (80ca16c) into develop (94a527c) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             develop    #3395      +/-   ##
=============================================
+ Coverage      48.21%   48.23%   +0.01%
- Complexity      4602     4605       +3
=============================================
  Files            555      555
  Lines          36755    36756       +1
  Branches        4841     4841
=============================================
+ Hits           17723    17730       +7
+ Misses         16802    16792      -10
- Partials        2230     2234       +4

Impacted Files
Coverage Δ

...apache/rocketmq/client/trace/TraceDataEncoder.java
69.73% <100.00%> (+0.20%)
:arrow_up:

...ain/java/org/apache/rocketmq/test/util/MQWait.java
43.24% <0.00%> (-10.82%)
:arrow_down:

...org/apache/rocketmq/store/ha/WaitNotifyObject.java
66.07% <0.00%> (-5.36%)
:arrow_down:

...a/org/apache/rocketmq/store/StoreStatsService.java
28.85% <0.00%> (-1.97%)
:arrow_down:

...org/apache/rocketmq/common/stats/StatsItemSet.java
41.79% <0.00%> (-1.50%)
:arrow_down:

.../rocketmq/broker/filter/ConsumerFilterManager.java
72.19% <0.00%> (-0.90%)
:arrow_down:

...ocketmq/store/schedule/ScheduleMessageService.java
73.83% <0.00%> (-0.85%)
:arrow_down:

.../apache/rocketmq/logging/inner/LoggingBuilder.java
63.60% <0.00%> (-0.32%)
:arrow_down:

...etmq/client/latency/LatencyFaultToleranceImpl.java
50.00% <0.00%> (ø)

...ent/impl/consumer/DefaultLitePullConsumerImpl.java
69.03% <0.00%> (+0.17%)
:arrow_up:

... and 8 more

Continue to review full report at Codecov.

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


Cannot reset step count

Trying to reset the step counter at a certain time of the day.  Calling ac.stepInit() in the watchface only seems to work when the watch is awake (screen is on).  What is the correct way to call this function when the watch is in light sleep?

I am afraid that you have to wake from light sleep to execute any code. Have a look at the code for the alarm utility. You can probably hack it to have a special alarm that resets the step counter without turning on  the display or sounding the buzzer.


Update README with new link to introjs options code

The old link was broken, so I hope that helps others not to search for it.

Coverage remained the same at 100.0% when pulling 939e4bb611625a0ad4cde1a7bfad6f7329b2438e on AquaGenLive:patch-1 into 951d41b1ab39dc50d3b4ba09c668e6d146b0b397 on HiDeoo:master.

Hi,
Thanks for spotting the issue and fixing it with a pull request 👍🏼


cask/cmd/upgrade: fix missing versions variable

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.
[ ] Have you successfully run brew style with your changes locally?
[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

This reference to the versions variable is unnecessary. It was changed recently as a part of https://github.com/Homebrew/brew/pull/14778.
It is causing the following errors:

https://github.com/Homebrew/homebrew-cask/issues/142050
https://github.com/Homebrew/homebrew-cask/issues/142129

It originally didn't reference a versions variable at all(look at line 124) and no other versions variable exists in the same file.
/u/l/Homebrew (master|✔) [2]$ grep -i versions '/usr/local/Homebrew/Library/Homebrew/cask/cmd/upgrade.rb'
          ofail "Not upgrading #{count} `installer manual` #{::Utils.pluralize("cask", versions.count)}."

A similar line exists in the cask/cmd/uninstall file so that probably was the source of the confusion.

Review period skipped due to critical label.


Update curl examples to include --compressed flag

A very simplistic optimisation to include --compressed argument to generated curl examples. While this may be blocked on some  configurations, it should work for most. Especially when working with large results, this can heavily save data size being transferred over network.

The curl syntax is provided solely to represent the equivalent API call. The actual API request is made using JavaScript's Fetch API, not curl. Since the purpose of the curl syntax is to illustrate the API call, including --compressed  flag in syntax may cause confusion, as it might imply that special handling is required to manage compressed responses


Why do you not parse line information?

It seems fairly critical to me, if someone is making a debugger or anything like that.

I stated this in the README, but you might have missed that:

At the moment, there is no support for C13 line information and the TPI type stream data, because Live++ does not make use of that information yet.

As you said, line information and type symbols are crucial for a debugger, but they aren't implemented (yet) because we didn't have a need for them.
We already know where line information is stored (in the module streams, see [https://llvm.org/docs/PDB/ModiStream.html]), but we don't provide a data structure for parsing that yet.
Can you point us to information how C13 line information is structured? We might add support for it ourselves, depending on how much work it's going to be.

Sorry for missing that part of the read. I've been trying to find out the structure of line information for a while and have found next to nothing. In the microsoft-pdb github repo in file mod.cpp there is a function Mod1::GetC11LinesFromC13(Buffer& bufC11Lines) that seems to be the where they parse line information, however their code is extremely hard to read and I can't even find definitions for some of the methods and classes being used there so I haven't been able to actually find out how the information is structured. Anyway I hope this helps, thanks for your time.

No need to apologize.
It seems that the Rust PDB crate has full support for C13 line information, but by the looks of it, it's another set of complicated data streams in an already complicated format, so it looks like a more time-consuming task.

@Vasil-Kirov Just wanted to let you know that @lukekasz added support for C13 line information to the library, thought you might be interested in that.


smart completion in kubernetes to handle [[name = "something"]] for containers, volumes, env vars etc

I love the pkl intellij plugin and it makes overriding kubernetes yaml structures super easy! There's one thing I'd really love though as a developer UX thing
imagine we are editing this template; its a kubernetes Deployment and the cursor is inside the containers {...} block:
amends "..."

deployment {
  spec {
    template {
      spec {
        containers {

        }
      }
    }
  }
}

what I'd love is the top 3 completions to be

new (its currently quite far down the list)
[...] (for index based override)
[[name = "..."]] which if you select the latter gives:

amends "..."

deployment {
  spec {
    template {
      spec {
        containers {
          [[name == "<cursor here>"]]  {
          }
        }
      }
    }
  }
}

FWIW the [[name == "..."]] completion should only appear if the type (in this case PodSpec.Container) has a name property.
I guess this is very kubernetes specific but its very common to want to override things in a Listing<T> by name in kubernetes.
e.g. customising any Deployment / Job / StatefulSet / Pod you usually want to configure a container / env var / volume and they all need to be done using the [[name == "..."]] syntax. Users should need to decide if they want a new or amend an existing item.
BTW it would be totally awesome if the plugin could grok what name values exist already in the parent template then smart completion could let you complete the string value on the right hand side of the "==" expression - so that you can only pick valid names :). Though thats probably much harder to do

This seems a reasonable (doable, sufficiently general) ask. It's not Kubernetes specific at all for member predicates to be part of suggestions inside Listings and Mappings. I think we can generally improve the use of == by using type information there (which we currently don't). It's finicky, though, because types of the operands of == only need to overlap (not be contained).
Evaluating upstream values is very quickly prohibitive perf-wise. We only do very limited evaluation in the plugin, because there's no telling how much time the plugin will need (and it needs to remain responsive).


Compare the Strings by frequency of the smallest characters

Problem Link
Compare the Strings by frequency of the smallest characters
https://leetcode.com/problems/compare-strings-by-frequency-of-the-smallest-character/
Coding platform
Leetcode
Programming language

[ ] C
[ ] C++
[ ] Java
[ ] Python

Hi @Nivedita967
I would like to solve this problem in Java. PLease assign it to me.

I would like to do this in C++.

I will do it in python


misc visual updates

Please makes sure these boxes are checked before submitting your PR, thank you!

[x] Make sure you follow Element's Contributing Guide.
[x] Rebase before creating a PR to keep commit history clear.
[x] Add some descriptions and refer relative issues for you PR.

冲突？

嗯，先等等，我待会还有个 commit

求合。然后你们就能看到深空灰版首页了。

lgtm


Need a way for capturing information that only plugins understand in the API

Inlining a map doesn't seem to work. http://play.golang.org/p/NnwTh9KI5r
Another (better?) possibility is to evolve APIObject. It should be possible to make an object type that can just store and regurgitate the JSON for an object that isn't understood.

I think @smarterclayton implemented the last bit of this already? am I misremembering?

I believe both parts (unrecognized foreign child objects and extensions to objects) should be working now.

Is there an example so I can try it out?
On Jan 15, 2015 2:25 PM, "Daniel Smith"<EMAIL_ADDRESS>wrote:

Closed #945 https://github.com/GoogleCloudPlatform/kubernetes/issues/945
.
--
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/issues/945#event-219749853
.


[BUG]: Error when editing a record

I opened demo site and I visited following URL:
http://<IP_ADDRESS>/admin/world/city/2823/edit
Now when I click submit, it gives following error:

The problem is with Country field, It is showing country name is Pakistan but value is null. Therefore it is generating foreign key exception.
When I renter Pakistan in this field and hit submit button. Then form gets saved.
Please correct this issue.

Thanks for your support😁， I'll fix this later.

has this issue been resolved???


The distinguished name specified for this replication operation is invalid

A few users have reported that they sometimes see an error with message "The distinguished name specified for this replication operation is invalid" instead of "Access denied" when using the Get-ADReplAccount cmdlet.

Any news about the issue?

This bugfix will be present in the next release of DSInternals. It is actually a workaround, because Microsoft's API itself returns error DS_DRA_BAD_DN instead of DS_DRA_ACCESS_DENIED. So after encountering this error, I send another request to the DC, asking it to transtate this DN into GUID, to check its existence. But it will still not work under some very rare circumstances.


parallel downloads

Currently each episode is downloaded sequentially. It would be nice if the program would pass all episodes to aria2, so that multiple episodes can be downloaded at the same time.
to do this with aria2 would probably be something like:
"echo "http://ep1.mp4 http://ep2.mp4 http://ep3.mp4" | aria2 -i-"
this loop would need to be skipped:
https://github.com/vn-ki/anime-downloader/blob/7e956a6f0c40e5f0a9b256fde95e10bb77a698d0/anime_downloader/cli.py#L125

What about links expiring @Meisterlala ?

Closing due to no response


Update sbt to 1.9.1

About this PR
📦 Updates org.scala-sbt:sbt from 1.8.3 to 1.9.1
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scala-sbt", artifactId = "sbt" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, version-scheme:early-semver, commit-count:1

Codecov Report

Merging #142 (8898fb3) into master (c3e6e0e) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 8898fb3 differs from pull request most recent head a369eb9. Consider uploading reports for the commit a369eb9 to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.


Shortcut to ignore the ingredients screen when crafting

Perhaps pressing shift.

Does bring some dangers since you can then start tasks that might not finish. Which you can now with the force start, but at least you were made aware of it


Move parsing tests from PM also.

We need to add the testing from PM in order to improve code cov

@claytonpbarrows the cdm parser needs some tests
https://codecov.io/gh/NREL/PowerSystems.jl/pull/110/tree/src/parsers


HIVEMALL-22: Update news feed for next Apache release

see: https://issues.apache.org/jira/browse/HIVEMALL-22

Merged https://github.com/apache/incubator-hivemall/commit/556503b1ac4d67bb8b0cb06e20afffdcaa8f5851
Thank you for the contribution. Could you close this issue?

hmm... It seems the contributor is not tracked on github.
When using github squash web UI, it was tracked https://github.com/apache/incubator-hivemall/commit/effe461446f2af84931901c112d542b8d2e69cfd though

@Lewuathe Could you update the project site as well? Do you know the instructions?
You can use bin/build_site.sh to build the site.
git://git.apache.org/incubator-hivemall-site.git (asf branch)

Thanks. I updated the site.
http://hivemall.incubator.apache.org/

Good to hear that you can run a ASF development procedure :-)


feat: ✨ add binary search problems

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@a854556). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master        #3   +/-   ##
==========================================
  Coverage          ?   100.00%
==========================================
  Files             ?         5
  Lines             ?       199
  Branches          ?        34
==========================================
  Hits              ?       199
  Misses            ?         0
  Partials          ?         0

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 a854556...eb21956. Read the comment docs.


Error: 0x8007007e when running Ubuntu for WSL

When installing Ubuntu from the Windows Store and running it for the first time (and thereafter), the following output is seen instead of starting the Ubuntu install process:
> ubuntu
Installing, this may take a few minutes...
Installation Failed!
Error: 0x8007007e
Press any key to continue...

Pressing any key drops back to the Windows shell.
I'm running the Windows 10 Pro Insider Preview Build 16193.rs_prerelease.170507-1300
Judging from the 1 star reviews on the Windows Store a number of other users are experiencing this same issue.
I haven't been able to capture any logging with more descriptive information. If any devs can advise on how I might capture more useful information I'll be glad to do so.

@nlundquist - Have you enabled the Windows Subsystem for Linux optional component?

Yes the "Windows Subsystem for Linux (Beta)" option is enabled under the Windows Features control panel. I've had it enabled since before I updated to the Insider Preview builds. Would you advise uninstalling and reinstalling it?

@nlundquist - you should not need to.  Could you please collect a trace for me?
https://github.com/Microsoft/BashOnWindows/blob/master/CONTRIBUTING.md#8-detailed-logs

I've run those commands prior to running unbuntu and ended up with the following output after stopping:
logman.zip
Didn't see anything of relevance in the output but perhaps I wasn't reading them correctly.

@nlundquist - Thanks for collecting the logs.  Unfortunately there aren't any data points in them which means that things are falling over before it gets to our component...  Could you double check that the optional component is still enabled?  There is a bug that will occasionally disable optional components on build upgrade.

Hmm interesting. I'm certain the WSL feature is enabled (or at least appears to be):

and the Windows bash shell still works as expected:

Yeah this might be related to the Store API's.  Adding some of the folks that worked on the Store API's here to see if they have any idea how to diagnose this further. @bitcrazed @paulcam206 @zadjii-msft.
Can you think of anything "special" about your machine, any settings relevant to store apps that you may have changed?

I've checked for the above binaries and %windir%\System32\wslapi.dll is missing. The rest of the binaries are present. This may be the culprit? No idea how this one .dll could be missing though.
Can't think of anything particularly out of the ordinary about my system configuration. Standard paths for everything. Most unusual thing may be that I have Docker installed.

I spoke with the dev lead for the Windows console and it appears that the appx that was submitted to the store has the wrong minimum version number.  Installing the store application requires build 16203 or later (you are running 16193).
Sorry for the inconvenience, but the good news is as soon as you upgrade to your next insider build things should be good.

Ahh I'll give that a shot and report back.

@nlundquist - Thank you very much for your help!

@nlundquist - Thanks for reporting & for your help diagnosing this snafu. We're working to remedy this with our partners too.
Just goes to show that we're not kidding when we say that we can't make WSL awesome without the help and support of our community 😀

This happened to me as well but I'm on OS build 16237.1001
I found out that restarting after ticking Windows Subsystem for Linux from Windows Features doesn't install anything at first try, well at least to my experience.
It took me two un-tick/tick attempts before it installed anything upon restart and made the bash terminal to work.

I've confirmed this is fixed when I'm on 16232. This issue can be closed imo.

@benhillis @nlundquist I just installed 16232 and had the same issue.  So probably not fixed here.

This happened with me, even on a newer build as mentioned above.
I installed Windows 10 yesterday (no idea build) and immediately upgraded to build 16241. In my case I went to the store and installed the Ubuntu tile, and received the same error. When I went to features WSL wasn't even enabled despite the tile being available. Upon manually enabling it worked as expected.

@spotlightishere yeah the tile will appear even if WSL isn't installed. I don't think that's a bug. WSL currently must be installed manually.
The issue I was experiencing was that even with WSL installed I was seeing this error.

We have reached out to the distribution partners and provided guidance on how to detect that the optional component is not installed.  We also have an ask our to the Windows Store team to filter or allow enabling our optional component from the store UI.

Confirming the issue exists in 16257.rs3_release.170728-1700.

And the same error in a VMware WS Pro 12.5.7 VM running on Debian Sid

@mrfelker have you installed the WSL optional component?
https://msdn.microsoft.com/en-us/commandline/wsl/install_guide

Enable the "Windows Subsystem for Linux" optional feature and reboot.
Open PowerShell as Administrator and run:
Enable-WindowsOptionalFeature -Online -FeatureName Microsoft-Windows-Subsystem-Linux

No, I have not.    I will do so on both the physical and virtual machines and get back to you.   Thanks

No, I have not.    I will do so on both the physical and virtual machines and get back to you.   Thanks

Same error on the server

@mrfelker - You will need to enable the Windows Subsystem for Linux optional component and reboot your machine.  Have you done that?

This is working on VM!.  I will recheck the server (it is possible I didn't type the command correctly.  Will see if I can cut and paste to rememina

And now it works on the server as well.   Thank you very much for pointing this out.  I hope that's all that the other posters needed to do

Just letting you know that I encountered this same issue today after getting the Ubuntu install package from the Windows store.

same issue in 16299.15. I have already enabled the windows subsystem for Linux.

@JackRBlack - Could you open a CMD prompt and type enter Ubuntu.exe (or whichever distro you're using) and paste a screenshot?

@bitcrazed Sure. Please check the following screenshots.

Thanks @JackRBlack
Did you reboot after enabling WSL? I reboot is required because WSL must load v. early in the boot process, before any 3rd party code, in order to ensure chain of trust.

Open PowerShell as Administrator and run:

Enable-WindowsOptionalFeature -Online -FeatureName Microsoft-Windows-Subsystem-Linux
2. Restart your computer when prompted.

Seriously canonical, what is this nonsense.  The "ubuntu" pane in the MS Store looks like a one-click install.  I am trying to get it working on a clean install of windows---one of about 10 things I need to install to get back to my normal workspace---and I'm stuck on this one.
This is the kind of thing that gives Linux a reputation for low usability!  Thanks for everything you do to make it more usable, but REALLY: USERS SHOULD FIND OUT THAT ANOTHER INSTALL IS REQUIRED FOR IT TO WORK BEFORE TRYING TO RUN AND GETTING AN ERROR.
It's this feeling that you think you're off to the races and then some error grabs your ankles that is so frustrating.

I just want to get a linux enviro up, it's not my fault that I'm on a windows machine, and here I am learning about the PowerShell so I can use it to install WSL so that then I can enable WSL in some mysterious windows dialog box.
shaving.  yaks.

@karenerobinson Thanks for sharing your thoughts here. You're right - this experience can and will be improved in the future. @tara-raj & team are looking into improving the store experience in the future, reducing/eliminating points of friction like this.
In the interim, while not entirely optimal:

The WSL installation docs  do describe the step by step installation instructions
Canonical, SUSE etc. do state in their product description that WSL needs to be enabled before the distro can be used,  & provide instructions on how to do so:

.
Thanks for sharing your experience. We look forward to continuing to work with you and the community with us to improve WSL for all our users.

I have the same issue now. Already enabled WSL feature, winver is 16299 and install Ubuntu from Windows Store still get the error
The WSL optional component is not enabled. Please enable it and try again.
See https://aka.ms/wslinstall for details.
Error: 0x8007007e

@hiitiger did you reboot after enabling WSL? The reboot is required before WSL will run.

Unable to enable WSL

@jean-ls - my sincere apologies, but I am terrible with languages: could you confirm the translation of that error message:

The message says:
"Could not read the key of configuration Registry"
I showed WSL to my colleague Jean (@jean-ls): WSL installation completed, asked for reboot, when we tried to install Ubuntu, it gave an error indicating WSL was not installed Then when we returned to Control Panel it was unckeched again.
Tomorrow we will put here his WinVer screenshot.

I have the same issue as @jean-ls and @TonyHenrique
I tried Enable-WindowsOptionalFeature -Online -FeatureName Microsoft-Windows-Subsystem-Linux in an administrator powershell and got the same error message (just in German).
PS C:\WINDOWS\system32> Enable-WindowsOptionalFeature -Online -FeatureName Microsoft-Windows-Subsystem-Linux
Enable-WindowsOptionalFeature : Der Schlüssel der Konfigurationsregistrierung konnte nicht gelesen werden.
In Zeile:1 Zeichen:1
+ Enable-WindowsOptionalFeature -Online -FeatureName Microsoft-Windows- ...
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Enable-WindowsOptionalFeature], COMException
    + FullyQualifiedErrorId : Microsoft.Dism.Commands.EnableWindowsOptionalFeatureCommand

PS C:\WINDOWS\system32>

Im running on a Surface 3 with Windows 10 Pro 1709 Build 16299.125

I have a similar issue as @TonyHenrique . I have tried the powershell command as administrator and tried checking the windows subsystem box and after the reboot it comes back unchecked. i have tried it more than 10 times and still cant run ubuntu.
Windows 10 Pro Version 1709 OS Build 16299.251
Please any help would be much appreciated

Same here would be awesome if this could get fixed

On my notebook, I am using latest Windows preview build and WSL just works.

I might be hitting this issue. The error message is different, but the error code (0x8007007e) is the same. Details here and here.


Incorrect behavior of painting tiles

Godot version
4.0 beta1
System information
Win11  GeForce GTX1060 6GB
Issue description

Steps to reproduce
.
Minimal reproduction project
test2.zip

Can you describe the steps to reproduce? The GIF alone isn't sufficient to understand what's going on.


cozette: 1.22.0 -> 1.22.1

Diff: https://github.com/slavfox/Cozette/compare/v.1.22.0...v.1.22.1
Changelog: https://github.com/slavfox/Cozette/blob/v.1.22.1/CHANGELOG.md
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[x] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

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

1 package built:

cozette


Feature/r2dbc infra

Description (Note: It is DRAFT pull request. Changes are in progress.)
Move mysql plugin to r2dbc driver.

Make changes to POM file to bring in r2dbc jar files.
Change mysql plugin methods to use r2dbc APIs.

Fixes appsmithorg/appsmith #1654
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
In Progress
Checklist:
In Progress

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

Raising new one.


Integrate with travis-ci

On OSX, might be case insensitive folder names? I was getting an extra folder in the build directories on Linux. Removing all build folders fixed the tests (in the PR).

Big thanks @RobLoach!
0.7.1


Initial functionality

Create a subcommand to generate a file like:
https://github.com/Senzing/knowledge-base/blob/master/lists/versions-latest.sh

Pull requests:

https://github.com/Senzing/dockerhub-util/pull/4


Add support for javax.cache:cache-api:1.1.1

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]

Yes, refer to

https://github.com/oracle/graalvm-reachability-metadata/pull/122
https://github.com/oracle/graalvm-reachability-metadata/pull/124

Describe the solution you'd like
A clear and concise description of what you want to happen.

Currently, com.hazelcast:hazelcast:5.2.1 and org.ehcache:ehcache:3.10.8-jakarta are both in the nativeTest process, and a part of javax.cache:cache-api:1.1.1  GraalVM reachability metadata is manually maintained dependent. It should be expected behavior to supplement this API facade library with more specific GraalVM reachability metadata.

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Considering that javax.cache:cache-api:1.1.1 itself is just a set of interfaces, writing tests for it seems like a hassle.

Additional context
Add any other context or screenshots about the feature request here.

Just link to https://github.com/linghengqian/graalvm-trace-metadata-smoketest/issues/1 .

Blocked by https://github.com/oracle/graalvm-reachability-metadata/pull/152#issuecomment-1359537797 .

I'm working on this issue.

I'm working on this issue.


Update code to Python 3.7 compliance

Simply runs pyupgrade with the option --py37-plus
Since Python 3.6 has officially reached end of life from this year. It might be nice to keep our code a bit up to date :)
P.S. My health has recovered. And extremely sorry for all the delays this month.

This is the solution to run recursively on all python files. I am sure I will forget this command and won't find it easily again. Hence pasting it here.
pyupgrade `find . -name "*.py" -type f` --py37-plus

Then I guess our code was already compliant to python 3.7 and we aren't using any special Features introduced in python 3.7.

Although it looks like we could automate this, https://github.com/pytorch/vision/pull/4585#pullrequestreview-775918877 implies that we can't have it as pre-commit hook. This mean, we probably should run pyupgrade regularly to fix stuff like this.

Python 3.7 end of life is in 2023 so probably we don't need to run this soon to upgrade python to 3.8.
But we would need to run it remove code which carries old python syntax.
I guess we can run some cronjob once in 2 months ? Which just reports it such old code exist? Or just leave it as it doesn't affect much.


Replace the GA implementation

The current implementation of the genetic algorithm requires some refactoring. A new version was released as a separate package: https://github.com/krzysztofarendt/modestga.
modestga has a very similar interface to scipy.optimize.minimize, so it should be very easy to add it.
To do:

[ ] Add the interface to modestga based on the existing interface to scipy
[ ] Test the performance against the current implementation of GA
[ ] Delete the current implementation if tests are successful

Note:
Unlike the currently implemented GA, modestga does not support dynamic mutation. However, since modestpy can chain GA with gradient-based methods, the benefit of the dynamic mutation is questionable. Originally, the dynamic mutation was implemented to speed up the search of the local optimum. Currently, it works as follows: when GA cannot decrease the error throughout a number of generations, it increases the mutation rate of a specified share of individuals, but restricts the gene changes to a limited range around the current value.

New optimized and benchmarked version of modestga: https://github.com/krzysztofarendt/modestga

Will work on this in this branch: https://github.com/sdu-cfei/modest-py/tree/issue_57_parallelize

Parallel GA (modestga) test successfully in examples\simple\simple_ga_vs_ga_parallel.py: https://github.com/sdu-cfei/modest-py/blob/issue_57_parallelize/examples/simple/simple_ga_vs_ga_parallel.py


🔧 MAINTAIN: Switch to pyproject.toml

@giovannipizzi I'll merge this so I can continue with the devops update/cleanup.


Master port optional

Currently, an optional port of 6066 is appended to the URL for talking to the Spark service.
There is a use-case in which the port is completely missing, and only the host is needed.
Is this a feature which could be supported? I could add a PR for it then.

@ywilkof Any comment on this?

Hi @avneet15 , this should already be possible, as it was adde in https://github.com/ywilkof/spark-jobs-rest-client/issues/14.


MF-549 - Change metadata format from JSON string to JSON object

What does this do?
It updates metadata format to native JSON object instead of a string. It also migrates users, things and bootstrap service to sqlx. This PR also updates MongoDB reader and writer in order to use new stable version of mongo driver.
Which issue(s) does this PR fix/relate to?
Resolves #549, and resolves #681.
List any changes that modify/break current functionality
Metadata is no longer in string format in HTTP response. It's still in string format when sending it over Redis Stream.
Have you included tests for your changes?
I've updated existing tests.
Did you document any new/modified functionality?
No.

Codecov Report

Merging #706 into master will decrease coverage by 0.82%.
The diff coverage is 82.05%.

@@            Coverage Diff             @@
##           master     #706      +/-   ##
==========================================
- Coverage   86.53%   85.71%   -0.83%
==========================================
  Files          64       64
  Lines        3848     4019     +171
==========================================
+ Hits         3330     3445     +115
- Misses        361      393      +32
- Partials      157      181      +24

Impacted Files
Coverage Δ

things/api/http/responses.go
90.14% <ø> (ø)
:arrow_up:

sdk/go/sdk.go
96.15% <ø> (ø)
:arrow_up:

things/things.go
100% <ø> (ø)
:arrow_up:

things/api/http/requests.go
96.15% <ø> (ø)
:arrow_up:

things/postgres/init.go
91.66% <100%> (ø)
:arrow_up:

users/postgres/init.go
82.6% <100%> (ø)
:arrow_up:

readers/mongodb/messages.go
92% <100%> (+0.69%)
:arrow_up:

bootstrap/postgres/init.go
92.85% <100%> (ø)
:arrow_up:

bootstrap/service.go
86.7% <100%> (+2.28%)
:arrow_up:

things/redis/events.go
88.57% <60%> (-11.43%)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

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


Unify ankaios cli

Issues: #

Definition of Done
The PR shall be merged only if all items mentioned in CONTRIBUTING.md have been followed. In case an item is not applicable as described, please provide a short explanation in the description.

[ ] All steps in CONTRIBUTING.md have been handled

@HorjuRares: I am starting review of this PR today. I think we need resolve again the conflicts with main. Could you that?

@krucod3: Shall @HorjuRares link the utests for add_default_workload_spec_per_update_mask and overwrite_using_field_mask to the swdd swdd~cli-provides-set-desired-state~1 ?


Swashbuckle and comments in DLL

Hello everyone,
I wonder if it is possible to have the comments of the classes found in the DLL in Swashbuckle.
For example: I have a class entity in another project that is guided and which is referenced in the project (DLL), when I go to http://url/swagger I have no comments found in this class.
Thanking you in advance for your help
Mickaël

Assuming you enabled the generation of XML documentation for your DLL, when you build your web app the XML file for that DLL should be in the bin directory. You'll have to include that XML file in SwaggerConfig.cs
For example, if your DLL is named Foo.dll, you should see Foo.xml in the web app's bin directory. Tell Swashbuckle to include this by adding the line
c.IncludeXmlComments($@"{System.AppDomain.CurrentDomain.BaseDirectory}\bin\Foo.xml");

Great It's working.
Thank you very much


Publish using webdeploy options are supported only when using Windows agent even though my publish profile use ZipDeploy

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: AzureRmWebAppDeployment@4
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - Azure Pipelines

If using Azure Pipelines, provide the account name, team project name, build definition name/build number: I can't provide those information.

Agent - Hosted.

Issue Description
My publish profile looks like this but when I use this task it keeps telling me that Publish using webdeploy options are supported only when using Windows agent. I know that I'm using an ubuntu image but I don't use webdeploy and don't want to use webdeploy. I want to use zipDeploy.
<Project>
  <PropertyGroup>
    <WebPublishMethod>ZipDeploy</WebPublishMethod>
    <PublishProvider>AzureWebSite</PublishProvider>
    <LastUsedBuildConfiguration>Release</LastUsedBuildConfiguration>
    <LastUsedPlatform>Any CPU</LastUsedPlatform>
    <SiteUrlToLaunchAfterPublish>xxxxxxxxxxxxx</SiteUrlToLaunchAfterPublish>
    <LaunchSiteAfterPublish>true</LaunchSiteAfterPublish>
    <ProjectGuid>xxxxxxxxxxx</ProjectGuid>
    <PublishUrl>xxxxxxxxxxxx</PublishUrl>
    <UserName>xxxxxx</UserName>
    <_SavePWD>true</_SavePWD>
  </PropertyGroup>
</Project>

Task logs
Starting: Deploy to eri-test-prod1 - Zip Deploy
==============================================================================
Task         : Azure App Service deploy
Description  : Deploy to Azure App Service a web, mobile, or API app using Docker, Java, .NET, .NET Core, Node.js, PHP, Python, or Ruby
Version      : 4.209.0
Author       : Microsoft Corporation
Help         : https://aka.ms/azureappservicetroubleshooting
==============================================================================
##[error]Error: Publish using webdeploy options are supported only when using Windows agent
Finishing: Deploy to eri-test-prod1 - Zip Deploy

Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
Starting: Deploy to eri-test-prod1 - Zip Deploy
==============================================================================
Task         : Azure App Service deploy
Description  : Deploy to Azure App Service a web, mobile, or API app using Docker, Java, .NET, .NET Core, Node.js, PHP, Python, or Ruby
Version      : 4.209.0
Author       : Microsoft Corporation
Help         : https://aka.ms/azureappservicetroubleshooting
==============================================================================
##[error]Error: Publish using webdeploy options are supported only when using Windows agent
Finishing: Deploy to eri-test-prod1 - Zip Deploy

Have you ever solved this issue?


Add DeleteAfter parameter to Quota

This patch add a DeleteAfter parameter, which define number of hours the template instance should be alive. If the time expires the template instance will be automatically removed.

@rawagner PTAL

Should the paramater be in hours or minutes? WDYT?

looks great. Can we use the same parsing logic for time as Hive ? https://github.com/openshift/hive/blob/master/apis/hive/v1/clusterclaim_types.go#L24-L34

looks great. Can we use the same parsing logic for time as Hive ? https://github.com/openshift/hive/blob/master/apis/hive/v1/clusterclaim_types.go#L24-L34

Oh sure, this method looks cool.

/lgtm
can you rebase ?

/lgtm


Cloth Config Dependency Outdated

Hello! The cloth config version that Falling Leaves is using is 9.0.93 which does not work with the 1.19.3 release as is compatible only with 1.19.3 pre release 2. Version 9.0.94 works fine for 1.19.3.
This breaks other mods when your mod is installed, as sometimes the cloth config version that your mod includes takes priority.
See more here
Hope it helps, thanks!

Falling Leaves does not contain Cloth Config. (It once did ages ago in 1.6.0 for Minecraft 1.16.)
What you were seeing is simply the version it was compiled against which does not affect execution in any way.
For the issue you were seeing:
Mods that are contained in other mods (jar-in-jar mods) are sorted by version and can only be overridden by top-level mods, meaning there is no randomness to this and Goober19 had Cloth Config 9.0.93 installed directly.


Invoking Call#hangup multiple times before the  event is received causes an error.

Invalid, this is taken care of.


Strip out unnecessary "panda_user" group

Doh! Not unnecessary, this controls access to the API methods.


/etc/init/ecs.conf gets overwritten when upgrading the ecs-init package

This results in ecs-agent not talking to the weave proxy

I see three ways to proceed:

Send a PR to ecs-init so that the rpm doesn't overwrite /etc/init/ecs.conf if it was modified (assuming that's possible).
Maintain our own ecs-init packages (sounds like a bad idea).
Do not touch /etc/init/ecs.conf and make /var/run/docker.sock point to the weave proxy. Which would also address https://github.com/weaveworks/integrations/issues/3 . This would require:

Changing docker's configuration (to listen on a different unix socket let's say /var/run/docker_original.sock) which could bring us to base 0 if the config is tracked by docker's rpm
Make weaveproxy listen on /var/run/docker/sock (I don't know if that's configurable)
Make sure we don't break the Convox integration (they also mess with docker's config at https://github.com/convox/rack/blob/master/api/dist/kernel.json#L505)

@tomwilkie I would appreciate your input on this one.

PR #6 will mitigate the impact of this issue.
This will be solved once aws/amazon-ecs-init#27 gets merged and released. Will close this issue once that happens.

For the record, this issue was resolved using Upstart override files (which I didn't know existed when opening it).


Upgrade images in .drone.yml to more recent versions

This builds Gitea with go 1.11, and upgrades alpine to 3.7
Note: As this uses golang 1.11 for the make fmt check it does a different check and so developers need to be aware of this when doing the check.

Codecov Report

Merging #4819 into master will increase coverage by 16.52%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           master    #4819       +/-   ##
===========================================
+ Coverage   20.69%   37.21%   +16.52%
===========================================
  Files         167      305      +138
  Lines       32389    45041    +12652
===========================================
+ Hits         6702    16764    +10062
- Misses      24706    25840     +1134
- Partials      981     2437     +1456

Impacted Files
Coverage Δ

modules/templates/helper.go
49.84% <ø> (ø)

modules/markup/html.go
89.59% <100%> (+9.69%)
:arrow_up:

models/migrations/v60.go
0% <0%> (ø)

routers/api/v1/org/hook.go
0% <0%> (ø)

modules/public/dynamic.go
100% <0%> (ø)

modules/httplib/httplib.go
25.44% <0%> (ø)

models/migrations/v65.go
0% <0%> (ø)

routers/admin/admin.go
0% <0%> (ø)

routers/api/v1/org/member.go
0% <0%> (ø)

routers/init.go
75% <0%> (ø)

... and 229 more

Continue to review full report at Codecov.

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


Downloaded tar from golang website, tar -xvf in /opt.
Now
i) GOPATH="/opt/go"
ii) GOROOT="/opt/go"
Then, install any go program
go install github.com/go-acme/lego/v4/cmd/lego@latest
It will be downloaded in /opt/go/pkg cleanly which is desirable but also splits out warning
warning: GOPATH set to GOROOT (/opt/go/) has no effect
but installs go program cleanly.
Now change env to
GOROOT="/opt/go"
GOPATH="/opt/go/pkg"
Then it will unnecessarily create /opt/go/pkg/pkg instead of just using /opt/go/pkg similar to when GOPATH==GOROOT. I am trying to avoid warning of GOPATH==GOROOT but at the same time its unnecessarily creating extra /opt/go/pkg/pkg folder.
What did you see happen?
Extra pkg directory /opt/go/pkg/pkg
What did you expect to see?
No extra pkg directory. Simply install in /opt/go/pkg

Related Issues and Documentation

cmd/go: misleading warning on GOROOT == GOPATH in module mode #47889
cmd/go: emits warning when GOPATH == GOROOT #65656 (closed)
cmd/go: go list should set Package.Root to the module root directory, not GOPATH #46119 (closed)
cmd/go: go get no longer creates $GOPATH/pkg dir in 1.10 #24360 (closed)
cmd/go: /root/go as GOROOT results in "missing $GOPATH" #34488 (closed)
cmd/go: warn if implicit GOPATH is equal to GOROOT #24623 (closed)
cmd/go: multiple GOPATH entries don't work as expected #3652 (closed)
cmd/go: env should populate GOBIN as effective install target directory for main packages #28337 (closed)
cmd/go: error for unset $GOPATH when GOROOT is $HOME/go is unhelpful #34628
cmd/go: go install should warn when installing outside PATH #35912

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

I wanted behaviour of /opt/go/pkg/mod/github.com being added packages when GOROOT="/opt/go" AND GOPATH="/opt/go/pkg" but when i add that it causes /opt/go/pkg/pkg/mod/github.com

I believe the separation is intentional, see also GOMODCACHE.


feat: MultipartFormData ContentHeaderEncoding UTF-8 & iso-8859-1

Try to fix https://github.com/Alamofire/Alamofire/issues/3755

Thanks for the PR! Please follow the PR template to inform why you'd like to see this change and how applicable it is for all Alamofire users.

Thanks for the PR! Please follow the PR template to inform why you'd like to see this change and how applicable it is for all Alamofire users.

Thx for the quick reply, just opened a issues https://github.com/Alamofire/Alamofire/issues/3755 to describe the problem

I guess I don't understand the need here. UTF-8 is already used for the filename encoding we get the from the system, so why would a less capable encoding be necessary? And I'm not sure cString is an appropriate API to use here, as we aren't dealing with C strings in the first place.
Additionally, a change like this requires both updated documentation as well as tests. I can write the documentation if you can't, but tests are required as part of this change.

Hey @jshier thx for the feedback and sorry for the late reply. Currently it look's like we use use iso-8859-1 by default and not UTF-8 for the  MultipartFormData ContentHeaderEncoding. Maybe everywhere else but not for MultipartFormData. 🤔


update readme

include environment variables setting

The comment is also about env. var. for hipcc. Do we really need CPATH and LIBRARY_PATH vars?

I've updated the Makefile and instruction. I'm not using docker, and I need CPATH for miopen.h, and LIBRARY_PATH for linking MIOpen library


Pcie wdt

PR for the watchdog timer and PCIe requirements patches.

Merged

Merged. Thanks.
Regards
Kumar
From: mdchitale @.>
Sent: Friday, July 16, 2021 9:33 AM
To: riscv/riscv-platform-specs @.>
Cc: Subscribed @.***>
Subject: [riscv/riscv-platform-specs] Pcie wdt (#31)
PR for the watchdog timer and PCIe requirements patches.
You can view, comment on, or merge this pull request online at:
https://github.com/riscv/riscv-platform-specs/pull/31
Commit Summary

Server extension: PCIe requirements
System Peripherals - watchdog timer

File Changes

M Makefile
https://github.com/riscv/riscv-platform-specs/pull/31/files#diff-76ed074a9305c04054cdebb9e9aad2d818052b07091de1f20cad0bbac34ffb52
(18)
A pcie-topology.ditaa
https://github.com/riscv/riscv-platform-specs/pull/31/files#diff-ee7b832b3ad8ac10bca8cb4590217bf44d40913ccef70c15bb2bca249cc54eb7
(28)
M riscv-platform-spec.adoc
https://github.com/riscv/riscv-platform-specs/pull/31/files#diff-e2e67320afc3dbcf884d10778f562bfa141048bbccbf7be067df8813a55a66fd
(165)

Patch Links:

https://github.com/riscv/riscv-platform-specs/pull/31.patch
https://github.com/riscv/riscv-platform-specs/pull/31.diff

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/riscv/riscv-platform-specs/pull/31, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABLRGVR2QZRXNWKPF3HW6CLTYBNM5ANCNFSM5AP4QJNA
.


Fix hero.tsx: Removed repetitive text in the hero paragraph

It originally contained the phrase: "The Virtual DOM Replacement for React." twice.
Please describe the changes this PR makes and why it should be merged:
Status

[x] Code changes have been tested against prettier, or there are no code changes
[x] I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:
- [ ] This PR changes the codebase
- [ ] This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
- [ ] This PR changes the internal workings with no modifications to the external API (bug fixes, performance improvements)

[x] This PR only includes non-code changes, like changes to documentation, README, etc.

Thank you @shikot0, but, this was intentional actually.


add a field for funding notes

and also not funding.
The text should go in the email that goes to them

Maybe two fields -- private notes for our own use, and the edited notes we send to the artist

only needs to appear on the discuss/fund page


Don't nest RouterActor under another actor

RouterActors assume the same actor path for their peers that live on other nodes. When nesting the RouterActor under the ActorLifecycleManagerActor, the ActorLifecycleManagerActor was being named with a random actor name, which didn't necessarily line up with peers and lead to dead letters & the inability for the router to talk with its peers.
Essentially for 2 nodes if:
nodeA router path = akka://SurgeActorSystem/user/$A/someRouterActor and
nodeB router path = akka://SurgeActorSystem/user/$B/someRouterActor
the routers for nodes A and B will be unable to communicate properly over remoting.

Codecov Report

Merging #125 (13940b8) into main (43b4c42) will decrease coverage by 1.07%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #125      +/-   ##
==========================================
- Coverage   75.33%   74.26%   -1.08%
==========================================
  Files         147      147
  Lines        2842     2840       -2
  Branches       79       78       -1
==========================================
- Hits         2141     2109      -32
- Misses        701      731      +30

Impacted Files
Coverage Δ

...surge/internal/core/SurgePartitionRouterImpl.scala
55.55% <100.00%> (-30.66%)
:arrow_down:

...ternal/akka/actor/ActorLifecycleManagerActor.scala
76.00% <0.00%> (-12.00%)
:arrow_down:

...in/scala/surge/internal/config/TimeoutConfig.scala
92.30% <0.00%> (-7.70%)
:arrow_down:

...a/surge/internal/domain/SurgeMessagePipeline.scala
88.63% <0.00%> (-6.82%)
:arrow_down:

...afka/streams/AggregateStateStoreKafkaStreams.scala
82.22% <0.00%> (-4.45%)
:arrow_down:

...rnal/health/supervisor/HealthSupervisorActor.scala
74.45% <0.00%> (-4.38%)
:arrow_down:

.../scala/surge/internal/health/HealthSignalBus.scala
71.27% <0.00%> (-4.26%)
:arrow_down:

...alth/matchers/SignalPatternMatcherDefinition.scala
90.32% <0.00%> (-3.23%)
:arrow_down:

...health/windows/actor/HealthSignalWindowActor.scala
91.95% <0.00%> (-1.15%)
:arrow_down:

... and 4 more

Continue to review full report at Codecov.

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


WIP: Qwik store 🎁

With this PR we can integrate the Qwik Store in the documentation 👀 🚀

added a little title and some minor CSS!

Let's solve paypal/shopify confifuration, and we can ship! this is amazing!

we need to test a full checkout process. Now we store the card id inside cookie. If the order is complete, do we need to delete it?

What's missing to merge this PR?

I can't wait to buy something from this shop 🤑


RLocalCachedMap with Broadcast Mode tracking

Can RLocalCachedMap be used with Broadcast mode tracking?
In Redis CLI we would enable this as "CLIENT TRACKING on REDIRECT 10 BCAST PREFIX"

client tracking is not support since it's not supported in Redis cluster and master / slave environments.

client tracking is not support since it's not supported in Redis cluster and master / slave environments.

Thanks - just to clarify - Redis doesn't support Client Tracking in Redis Cluster, or is that a Redisson limitation?

Thanks - just to clarify - Redis doesn't support Client Tracking in Redis Cluster, or is that a Redisson limitation?

it's not a Redisson limitation, RLocalCachedMap works in Redis cluster.

it's not a Redisson limitation, RLocalCachedMap works in Redis cluster.

ok - so client tracking is not supported on clusters, hence not supported by Redisson - got it. Then what is the mechanism used by Redisson to invalidate cache keys on local cache and force a re-fetch from remote cache? ... sorry just digging deeper to fully understand this.

ok - so client tracking is not supported on clusters, hence not supported by Redisson - got it. Then what is the mechanism used by Redisson to invalidate cache keys on local cache and force a re-fetch from remote cache? ... sorry just digging deeper to fully understand this.

it uses redis pubsub channel and two modes are available:
INVALIDATE - Default. Invalidate cache entry across all LocalCachedMap instances on map entry change
UPDATE - update cache entry across all LocalCachedMap instances on map entry change

it uses redis pubsub channel and two modes are available:
INVALIDATE - Default. Invalidate cache entry across all LocalCachedMap instances on map entry change
UPDATE - update cache entry across all LocalCachedMap instances on map entry change

Current implementation of server assisted client side caching doesn't send the field name of Redis hash, but only a key name. This means that invalidation per field isn't possible. Thus "CLIENT TRACKING" doesn't bring any benefits for local cached Redis hash structures.


Implement basic file service

For rogue we used a java based fileservice but since we are not using tomcat/geoserver in vida app, we need a basic service with an api that allows users to 1) upload file 2) download file 3) list files. When a file is uploaded, it should be named based on the sha1 hash of its binary content in order to easily facilitate syc with other stores.

Completed a basic app that meets are needs.


feat(website): Working on website section

@Patticatti The linter test is failing. Looks like you may need to ensure the files mentioned here have consistent (Unix) line endings: https://github.com/firezone/firezone/actions/runs/10277936365/job/28440785308?pr=6178#step:7:30


[Perf] Perf Framework occasionally fails to compute average

One time, the perf framework failed to compute the average throughput:
[02:14:00.445] === Test ===
[02:14:00.445] Current          Total           Average
[02:14:01.448] 8475             8475            NaN
[02:14:02.452] 8860             17335           NaN
[02:14:03.460] 9745             27080           NaN
[02:14:04.468] 9411             36491           NaN
[02:14:05.476] 9532             46023           NaN
[02:14:06.478] 9010             55033           NaN
[02:14:07.484] 9122             64155           NaN
[02:14:08.488] 9514             73669           NaN
[02:14:09.500] 9561             83230           NaN
[02:14:10.502] 9068             92298           NaN
[02:14:10.508] 0                92298           NaN

I have seen this for C++ just or the first row from the print thread.
My guess is that we could be trying to read from one of the results array while its been updated.
How important is this? Wouldn't it be OK if the final result row provides the avg ?

The question is what should the perf framework do if one of the parallel work streams has never completed a single operation.  For context, here's how the perf framework works at a high level.
Say parallel=32.  The perf framework creates 32 parallel work streams (async tasks or threads), and for each work stream it tracks the number of completed operations and the time of the last computed operation.  The time is necessary to correctly compute the average operations/second across all the work streams, since each work stream finishes at a slightly different time.
Now say one of the work streams gets stuck on its first request, so it has completed zero requests and the "last completion time" is undefined.  What is the average operations/second for this work stream?
Strictly speaking it is undefined, since the numerator is zero and the denominator is undefined.  We could choose to keep it this way in the perf framework, since this is usually a sign something is wrong with your test or config (say parallel is too high).  It's best to ensure each work stream has completed one operation.
Alternatively, we could count the average operations/second as 0.  This slightly undercounts since the work stream has likely done some work already, but it has the advantage of ensuring there is always a valid "average throughput" even if one of the work streams got stuck on it's first request.
I prefer the "count as 0" approach since I think it more closely fits the sprit of the perf framework, which is to calculate the performance of the work that was completed, and ignore any pending work that was cancelled when the test completion timer fired.

What is the average operations/second for this work stream?

I though we only provide the entire all threads avg result. Can we show the per-thread avg?  There are 2 places where we need to calculate (make the division) and print the avg. The first place in from the progress reporter, and the other place is once test duration is reached and the test results are printed.
In both cases, the displayed avg is the avg of the all avgs from each thread.  There is a zip() operation to get each thread avg and produce the total avg right?
Then, even if there is one or two threads that got stock and did not complete not even one operation, it would just impact the total avg but, how do I noticed there was a slow or dead thread in there?
I might be missing something, but, what I meant is:  if we just don't consider the results from the array where operations count is zero for the avg results, (update the Zip method with a condition to check operation count > 0), wouldn't that be just enough ?

@vhvb1989: I understand what you are suggesting now, and I updated #18512.  I agree this is a cleaner way to fix this.
I created an issue to add an option to display the per-parallel data:
https://github.com/Azure/azure-sdk-for-net/issues/18614


I want to know what wrong with this?

My Sass isn't compiling
I am using vue+sass to build my project,but when I run npm run dev
the terminal will return this:
ERROR in ENOENT: no such file or directory, scandir '/Users/baidu/Documents/mydata/hfutoj/node_modules/.npminstall/node-sass/3.8.0/node-sass/vendor'
 @ ./~/vue-style-loader!./~/css-loader!./~/vue-loader/lib/style-rewriter.js!./~/.npminstall/sass-loader/4.0.0/sass-loader!./~/vue-loader/lib/selector.js?type=style&index=0!./src/components/footer.vue 4:14-273 13:2-17:4 14:20-279

I add code in my vue components just like:
<style lang="scss">
.footer {
  margin-top: 3em;
  padding-top: 2em;
  border-top: 1px solid #ccc;
  text-align: center;
  font-size: 1em;
  clear: both;
  height: 4em;
</style>

I am a new learner ,so I search this question by google or baidu,but I can not find the way to solve this question.

Ok,I resolve this by installing all node_modules again,I think node-sass install failed,so I turned the version of node-sass to 3.7.0 and used cnpm install to reinstall all node_modules


Unable to export app when 1.5.1 is installed as Rancher App from dev-v2.x

Describe the bug
Rancher:v2.6.9 on k3s v1.24.6+k3s1
Epinio  (100.0.3+up1.5.1) Rancher App installed from dev-v2.6 rancher repository
and
Rancher:v2.7.0 on k3s v1.24.6+k3s1
Epinio  (101.0.2+up1.5.1) Rancher App installed from dev-v2.7 rancher repository
After deploying a sample app over webUI and calling epinio app export sample ./ user is getting only values.yaml file. Chart and Image are not exported.
Missing Image was expected as we don't have skopeo image mirrored due to a image format incompatability but we should get chart in addition which is missing.
epinio app export sample ./

🚢  Export application
Namespace: workspace
Application: sample
Target Directory: ./

Downloading app values to 'values.yaml' 100% |█████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| (357/357 B, 871 kB/s)

❌  error exporting app: server status code: Service Unavailable

In epinio-server logs (ENV TRACE_LEVEL=20 in its deployment)  we see that chart endpoints returns status 503 for some reason.
2022/12/15 11:08:41 EpinioServer: "level"=1 "msg"="/api/v1/namespaces/workspace/applications/sample/part/chart" "status"=503 "method"="GET" "path"="/api/v1/namespaces/workspace/applications/sample/part/chart" "query"="" "ip"="<IP_ADDRESS>" "user-agent"="Go-http-client/1.1" "time"="2022-12-15T11:08:41Z" "latency"="1.33619546s"

The endpoint is called three times from the client, for the three pieces (manifest, chart tarball, and image).
The /api/v1/namespaces/workspace/applications/sample/part/chart is the call asking for the chart tarball. I.e. the application template.
This is also the second call of the .../part endpoint. The first, for the.../part/values has to have succeeded.
The skopeo becomes important in the third call for .../part/image. Which we do not reach here.
Which means that this issue is separate from skopeo.

Relevant code looks to be https://github.com/epinio/epinio/blob/df785232277e302b7e86f52bdc8bb9f2da4de8aa/internal/api/v1/application/part.go#L116-L125
And noticing function chartArchiveURL I am reminded that for Rancher integration the app chart tarball is placed into the server's filesystem, through a helm chart asset. I cannot seeing this playing well with a http.Get().

I've successfully validated the app export command when epinio-1.6.1 is deployed as a Rancher App with the new epinio (server+cli) binary built after merging #2050.
This is how I was testing it:

built latest epinio binary by make command
deployed rancher 2.7.1 (on single node k3s v1.24.6+k3s1) by calling make install-k3s install-helm install-cert-manager install-rancher  in checkouted e2e repo
added epinio app repository into rancher: https://github.com/epinio/charts.git branch: epinio-1.6.1
installed the epino app from that app repository over rancher UI
patched the epinio server binary in my cluster by calling scripts/patch-epinio-deployment.sh (with modified namespace cattle-epinio-system) from epinio repo
deployed sample-php app by epinio push --name sample --path .
exported the app by epinio app export sample ~/export-app/:

🚢  Export application
Namespace: workspace
Application: sample
Target Directory: /home/sles/git/export-app/

Downloading app values to '/home/sles/git/export-app/values.yaml' 100% |███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| (357/357 B, 681 kB/s)
Downloading app chart to '/home/sles/git/export-app/app-chart.tar.gz' 100% |██████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| (3.4/3.4 kB, 8.6 MB/s)
Downloading app image to '/home/sles/git/export-app/app-image.tar' 100% |█████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| (929/929 MB, 120 MB/s)


filter results based on AND operation

How to customize filter results based on AND operation by clicking on  checkbox field (ie if user select crime and drama, it should show the results which is having both crime and drama categories).
But now it is showing the results of the movies which is having either crime or drama.

Hi, I have the same question. Did you find an answer yet? Thanks in advance! J

Have a look at #83 @jiren really needs to push the committed code though, as it's a working feature.


Will EWS to the 365 Third-Party-Data endpoint work after EWS end-of-life

[Hello, Will Microsoft's third-party-data endpoint to the Azure service in Microsoft 365 work after EWS end-of-life?
EWS end of life:  Deprecation of Basic authentication in Exchange Online | Microsoft Docs
The endpoint used to connect to the Azure service in Microsoft 365:
HTTP
https://office365ingestionsvc.gble1.protection.outlook.com/service/ThirdPartyIngestionService.svc
Endpoint:  https://office365ingestionsvc.gble1.protection.outlook.com/service/ThirdPartyIngestionService.svc
Thank you,
Sharon Bacon
17a-4, llc
<EMAIL_ADDRESS>
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 3d168aa8-f335-f879-d12f-d0b198f2a339
Version Independent ID: 049d4291-2fd8-c80b-18a8-31e5ecb5fc88
Content: Work with a partner to archive third-party data - Microsoft Purview (compliance)
Content Source: microsoft-365/compliance/work-with-partner-to-archive-third-party-data.md
Service: o365-seccomp
GitHub Login: @v-tophillips
Microsoft Alias: v-tophillips

@sbacon17a4 Thanks for your feedback! New issues and pull requests for this content are closed while we update this content to support upcoming feature improvements.


Update README.rst

Coverage decreased (-0.3%) to 87.301% when pulling 4f1ab41c4a15963e40776fe4a20a46c1eeda92e3 on SREELAKSHMI-NAYATH:patch-1 into 895b6d747913f104ba04dc27993b58045ef48d1a on quantopian:master.

Coverage decreased (-0.3%) to 87.301% when pulling 4f1ab41c4a15963e40776fe4a20a46c1eeda92e3 on SREELAKSHMI-NAYATH:patch-1 into 895b6d747913f104ba04dc27993b58045ef48d1a on quantopian:master.


Incorrect "reedemed in" for outputs

In TXs with multiple outputs, the "redeemed in" will just loop for the first addresses, then list as "not yet redeemed" for the rest. What this does is making it show that an output has been redeemed in address, which it necessarily haven't.
Expected behavior is that it will only display "redeemed in" for those outputs that have been redeemed, which involves skipping some addresses in the loop.
An example of this can be found in xpybits.com payout:
https://ledger.paycoin.com/transaction/80506ec081d5827d51f017be45b7eaab2bf4888d237021d01da058f74c2eb027

Nice pick up!


Pmag GUI should have functionality for adding column names on the fly

Put add columns into Pmag GUI step 3 (as in MagIC GUI)

This has been implemented along with a bunch of other improvements.  Pmag GUI step 3 now uses a very similar grid frame to MagIC GUI, which is a bit nicer.


AzurePowerShell@5 Az.CostManagement cannot be installed/imported on the ms-hosted AzureDevOps agent

Note
Environment: ubuntu-20.04
Required Information
Question, Bug, or Feature?
Type: Bug
Enter Task Name: AzurePowerShell@5
Environment

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name:
Environment: ubuntu-20.04

Issue Description
When running the task below on the agent, I can't use the module https://docs.microsoft.com/en-us/powershell/module/az.costmanagement/. However, on my local it works. That's not right.
- task: AzurePowerShell@5
                  inputs:
                    azureSubscription: platform-${{ variables.spokeName }}-subscription
                    scriptType: inlineScript
                    scriptArguments: ${{ variables.SubscriptionId }} ${{ variables.regionShort }} ${{ variables.spokeEnv }}
                    errorActionPreference: stop
                    failOnStandardError: false
                    TargetAzurePs: latestVersion
                    inline: |
                      Install-Module -Name Az.CostManagement -force -verbose -Confirm -AllowClobber -SkipPublisherCheck
                      Get-InstalledModule
                      Import-Module -Name Az.CostManagement -force -verbose
                      Remove-AzCostManagementExport -Name kubecostExport -Scope 'subscriptions/$1'
                      New-AzCostManagementExport
                        -Scope "subscriptions/$1"
                        -Name "kubecostExport"
                        -ScheduleStatus "Active"
                        -ScheduleRecurrence "Weekly"
                        -RecurrencePeriodFrom "2022-05-31T00:00:00Z"
                        -RecurrencePeriodTo "2049-05-29T00:00:00Z"
                        -Format "Csv"
                        -DestinationResourceId "/subscriptions/$1/resourceGroups/mcs-platform-$2-$3-sto-rg/providers/Microsoft.Storage/storageAccounts/mcs$2$3kubecost"
                        -DestinationContainer "exports"
                        -DestinationRootFolderPath "ad-hoc"
                        -DefinitionType "ActualCost"
                        -DefinitionTimeframe "WeekToDate"
                        -DatasetGranularity "Weekly"

Task logs
Import-Module: /home/vsts/work/_temp/8d31971f-7d0d-4af3-b37c-8e2b85e8921d.ps1:7
Line |
7 |  Import-Module -Name Az.CostManagement -force -verbose
|  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| The specified module 'Az.CostManagement' was not loaded
| because no valid module file was found in any module directory.
Remove-AzCostManagementExport: /home/vsts/work/_temp/d9c5ba05-1dc6-4741-b2ce-06c0e19878e5.ps1:5
Line |
5 |  Remove-AzCostManagementExport
|  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
| The term 'Remove-AzCostManagementExport' is not recognized as
| a name of a cmdlet, function, script file, or executable
| program. Check the spelling of the name, or if a path was
| included, verify that the path is correct and try again.

Guyz, the 'label' pipeline in this repo gives:
label
Unexpected input(s) 'version', valid inputs are ['always-auth', 'node-version', 'node-version-file', 'architecture', 'check-latest', 'registry-url', 'scope', 'token', 'cache', 'cache-dependency-path']


Android 10 : asking LocationAlways gives RESULTS.BLOCKED

Bug
I try to request/check PERMISSIONS.ANDROID.ACCESS_BACKGROUND_LOCATION but every time I get RESULTS.BLOCKED so I redirect the user to go check the settings of the app, but it's not what I expected. I'd like to have a popup asking for All the time geolocation permission.
Any idea ?
Environment info
System:
    OS: macOS 10.15.3
    CPU: (4) x64 Intel(R) Core(TM) i5-5257U CPU @ 2.70GHz
    Memory: 187.82 MB / 8.00 GB
    Shell: 5.7.1 - /bin/zsh
  Binaries:
    Node: 13.12.0 - /usr/local/bin/node
    Yarn: 1.22.0 - /usr/local/bin/yarn
    npm: 6.14.4 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  SDKs:
    iOS SDK:
      Platforms: iOS 13.4, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
    Android SDK:
      API Levels: 23, 26, 27, 28, 29
      Build Tools: 28.0.3
      System Images: android-29 | Google Play Intel x86 Atom
      Android NDK: 19.0.5232133
  IDEs:
    Android Studio: 3.6 AI-192.7<IP_ADDRESS>00805
    Xcode: 11.4/11E146 - /usr/bin/xcodebuild
  npmPackages:
    @react-native-community/cli: ^4.5.1 => 4.6.3
    react: 16.11.0 => 16.11.0
    react-native: 0.62.0 => 0.62.0
  npmGlobalPackages:
    react-native-cli: 2.0.1
    react-native-git-upgrade: 0.2.7

Library version: x.x.x
Steps To Reproduce

Try to request always geolocation permission on a recent android device with Android 10 (like Pixel a3)
Ask for PERMISSIONS.ANDROID.ACCESS_BACKGROUND_LOCATION
Get RESULTS.BLOCKED
...

Describe what you expected to happen:

Try to request always geolocation permission on a recent android device with Android 10 (like Pixel a3)
Ask for PERMISSIONS.ANDROID.ACCESS_BACKGROUND_LOCATION
Get RESULTS.DENIED or RESULTS.GRANTED

Reproducible sample code
const dummy = async () => {
  try {
    const checkResult = await check(PERMISSIONS.ANDROID.ACCESS_BACKGROUND_LOCATION);
    switch (checkResult) {
      case RESULTS.UNAVAILABLE:
        Alert.alert(i18n.geolocation.notAvailableOnThisDevice);
        return;
      case RESULTS.DENIED:
        // The permission has not been requested / is denied but requestable
       // I ask for it then... (code not shown here)
        break;
      case RESULTS.GRANTED:
        // The permission is granted
        // Then I keep going
        break;
      case RESULTS.BLOCKED:
        // The permission is denied and not requestable anymore -> App settings
        setTimeout(showGoToSettingsAlert, 1000);
        break;
    }
  } catch (error) {
    console.warn(error);
  }
};

You have to request both at the same time. Their API is quite bad.
requestMutiple([
  PERMISSIONS.ANDROID.ACCESS_BACKGROUND_LOCATION,
  PERMISSIONS.ANDROID.ACCESS_FINE_LOCATION // or PERMISSIONS.ANDROID. ACCESS_COARSE_LOCATION
])

Thank you. I confirm, it’s bad