Debugging ResourceExhausted Error in big block sanity test while submitting txs from txsim to celestia validator Instances in knuu

We are running an e2e test using knuu that involves two validators and one txsim instance for each (Link to the test; link to the PR). One txsim runs without issues, but the other consistently produces the following error:
rpc error: code = ResourceExhausted desc = grpc: received message larger than max (20016481 vs. 4194304)

2024-05-03 17:42:20.161
{"level":"info","height":5,"address":"celestia1phk8jqaylvy7znp2n8444jxxph3eaf4hpa42yy","msgs":"/cosmos.bank.v1beta1.MsgSend,/cosmos.bank.v1beta1.MsgSend,/cosmos.bank.v1beta1.MsgSend,/cosmos.bank.v1beta1.MsgSend,/cosmos.bank.v1beta1.MsgSend,/cosmos.bank.v1beta1.MsgSend,/cosmos.bank.v1beta1.MsgSend,/cosmos.bank.v1beta1.MsgSend,/cosmos.bank.v1beta1.MsgSend,/cosmos.bank.v1beta1.MsgSend","time":"2024-05-04T00:42:20Z","message":"tx committed"}
2024-05-03 17:42:16.972
{"level":"info","address":"celestia1phk8jqaylvy7znp2n8444jxxph3eaf4hpa42yy","balance":10000000000000000,"time":"2024-05-04T00:42:16Z","message":"set master account"}

Each txsim generates PFBs each with the total blob size of 1MB (1,000,000) (which are less than the indicated max 4194304 in the error messages).

Multiple theories are being considered:

There may be a need for configuration adjustments to the validators (identifying the specific configurations required).
This issue could stem from limitations on the knuu side, such as resource sharing across instances (from DevOps response it looks like this is not the case and each instance runs in its own pod, hence no resource sharing happens) or the handling of network communication (this still needs to be verified as whether the resources may get shared during network communication between pods).

I'll conduct tests and share further results here to determine the minimal settings in which this error occurs:

Does it occur with any PFB size?
Does the number of sequences impact the error?
Can we direct two txsim instances to the same validator?

I conducted the same tests under various txsim configurations, and the results are detailed below. My goal was to identify the point at which errors begin to manifest. For each configuration, two txsim knuu instances were created. Tests using the configs marked with a two green checkmark concluded without errors, while others experienced failures in the second txsim, as indicated by the provided error messages.

BlobsPerSeq = 1, BlobSequences = 1, BlobSizes = 100,000 ✅ ✅
BlobsPerSeq = 1, BlobSequences = 1, BlobSizes = 200,000 ✅ ✅
BlobsPerSeq = 6, BlobSequences = 1, BlobSizes = 200,000 ✅ ✅
BlobsPerSeq = 6, BlobSequences = 5, BlobSizes = 200,000 ✅  🛑
BlobsPerSeq = 1, BlobSequences = 5, BlobSizes = 200,000 ✅ ✅
BlobsPerSeq = 1, BlobSequences = 30, BlobSizes = 200,000 ✅ 🛑
BlobsPerSeq = 1, BlobSequences = 15, BlobSizes = 200,000 ✅ 🛑

rpc error: code = ResourceExhausted desc = grpc: received message larger than max (6013070 vs. 4194304)

BlobsPerSeq = 1, BlobSequences = 10, BlobSizes = 200,000  ✅ ✅
BlobsPerSeq = 1, BlobSequences = 12, BlobSizes = 200,000  ✅ 🛑

rpc error: code = ResourceExhausted desc = grpc: received message larger than max (4809096 vs. 4194304)
Error: rpc error: code = ResourceExhausted desc = grpc: received message larger than max (4809096 vs. 4194304)

BlobsPerSeq = 1, BlobSequences = 12, BlobSizes = 100,000  ✅ ✅
BlobsPerSeq = 1, BlobSequences = 30, BlobSizes = 100,000  ✅ 🛑

Error: rpc error: code = ResourceExhausted desc = grpc: received message larger than max (6024558 vs. 4194304)

BlobsPerSeq = 1, BlobSequences = 48, BlobSizes = 100,000 ✅  🛑

rpc error: code = ResourceExhausted desc = grpc: received message larger than max (9631632 vs. 4194304)

BlobsPerSeq = 1, BlobSequences = 48, BlobSizes = 200,000 ✅  🛑

rpc error: code = ResourceExhausted desc = grpc: received message larger than max (19231637 vs. 4194304)

No conclusive findings yet; I'm continuing to debug and checking the details of txsim internals. The only notable insight so far is that the initial txsim consistently succeeds. Furthermore, a comparison of the highlighted configurations reveals that merely increasing the sequence count by two causes the second txsim to begin failing.

BlobsPerSeq = 1, BlobSequences = 10, BlobSizes = 200,000  ✅ ✅
BlobsPerSeq = 1, BlobSequences = 12, BlobSizes = 200,000  ✅ 🛑

Theory: Could the error be misleading, with the txsim failing due to resource limitations either on the validator side or the txsim side?
Test: Increasing resources on both the validator side and the txsim side does not resolve the issue. Here are the resources used:
	ValidatorResource: testnet.Resources{
		MemoryRequest: "12Gi",
		MemoryLimit:   "20Gi",
		CPU:           "10",
		Volume:        "20Gi",
	},
	TxClientsResource: testnet.Resources{
		MemoryRequest: "1Gi",
		MemoryLimit:   "4Gi",
		CPU:           "4",
		Volume:        "1Gi",
	}

Theory: The second validator might be problematic. Therefore, directing both  txsims to the first validator could potentially resolve the issue.
Test and Results: However, pointing both txsims to the same validator did not eliminate the failure.

Adding more context as I further my debugging: The line from which the error is emitted is this line from the parser's recvMsg method from grpc library.

Theory: Are the messages/txs being batched on the txsim side?
It does not seem so, each sequence of txsim uses the BroadcastTx rpc API, which is a unary API, and encapsulates encoded version of one single tx, hence no batching is expected.

I did a few tests and was able to reproduce the issue locally:

Run single-node.sh script
Run txsim --blob-sizes 20016469 --key-mnemonic "arch extra share tag great resource family owner shrug nominee crunch bulk cart twenty ripple bunker sugar carbon unable visual fold leave lyrics soda"  --grpc-endpoint localhost:9090  --blob 1

This returns:
{"level":"info","address":"celestia1kkthrhfut4s7weqkzd0g3667r53nrjkc77lxk6","balance":899984999534100,"time":"2024-05-13T01:23:56+04:00","message":"set master account"}
{"level":"info","height":148,"address":"celestia1kkthrhfut4s7weqkzd0g3667r53nrjkc77lxk6","msgs":"/cosmos.bank.v1beta1.MsgSend","time":"2024-05-13T01:23:59+04:00","message":"tx committed"}
{"level":"info","address":"celestia1nuxenvm599vezdsrtxufmrr032xrcrrcyefgp3","balance":1000000000,"account number":10,"time":"2024-05-13T01:23:59+04:00","message":"initialized account"}
{"level":"error","error":"rpc error: code = ResourceExhausted desc = grpc: received message larger than max (20016861 vs. 10485760)","address":"celestia1nuxenvm599vezdsrtxufmrr032xrcrrcyefgp3","blobs":"1","total size of blobs":20016469,"time":"2024-05-13T01:24:00+04:00","message":"tx failed"}
{"level":"error","error":"sequence 0: rpc error: code = ResourceExhausted desc = grpc: received message larger than max (20016861 vs. 10485760)","time":"2024-05-13T01:24:00+04:00","message":"sequence failed"}
Error: sequence 0: rpc error: code = ResourceExhausted desc = grpc: received message larger than max (20016861 vs. 10485760)
Usage:
  txsim [flags]

Examples:
txsim --key-path /path/to/keyring --grpc-endpoint localhost:9090 --seed 1234 --poll-time 1s --blob 5

Flags:
      --blob int               number of blob sequences to run
      --blob-amounts string    range of blobs per PFB specified as a single value or a min-max range (e.g., 10 or 5-10). A single value indicates the exact number of blobs to be created. (default "1")
      --blob-sizes string      range of blob sizes to send (default "100-1000")
      --feegrant               use the feegrant module to pay for fees
      --grpc-endpoint string   grpc endpoint to a running node
  -h, --help                   help for txsim
      --key-mnemonic string    space separated mnemonic for the keyring. The hdpath used is an empty string
      --key-path string        path to the keyring
      --master string          the account name of the master account. Leaving empty will result in using the account with the most funds.
      --poll-time duration     poll time for the transaction client (default 3s)
      --seed int               seed for the random number generator
      --send int               number of send sequences to run
      --send-amount int        amount to send from one account to another (default 1000)
      --send-iterations int    number of send iterations to run per sequence (default 1000)
      --stake int              number of stake sequences to run
      --stake-value int        amount of initial stake per sequence (default 1000)
      --suppressLogs           disable logging

sequence 0: rpc error: code = ResourceExhausted desc = grpc: received message larger than max (20016861 vs. 10485760)%

Then, I change the max-recv-msg-size in app.toml to<PHONE_NUMBER>000, restart the node (in your branch this doesn't work, so I just add this to the single-node.sh: sed -i'.bak' 's#"10485760"#"1048576000000"#g' "${CELESTIA_APP_HOME}"/config/app.toml,
Then I run again the txsim command, and it returns:
{"level":"info","address":"celestia1kkthrhfut4s7weqkzd0g3667r53nrjkc77lxk6","balance":899984999936500,"time":"2024-05-13T01:24:59+04:00","message":"set master account"}
{"level":"error","error":"tx was included but failed with code 11: out of gas in location: WritePerByte; gasWanted: 100000, gasUsed: 100375: out of gas","address":"celestia1kkthrhfut4s7weqkzd0g3667r53nrjkc77lxk6","msgs":"/cosmos.bank.v1beta1.MsgSend","time":"2024-05-13T01:25:02+04:00","message":"tx failed"}
Error: error funding accounts: tx was included but failed with code 11: out of gas in location: WritePerByte; gasWanted: 100000, gasUsed: 100375: out of gas
Usage:
  txsim [flags]

Examples:
txsim --key-path /path/to/keyring --grpc-endpoint localhost:9090 --seed 1234 --poll-time 1s --blob 5

Flags:
      --blob int               number of blob sequences to run
      --blob-amounts string    range of blobs per PFB specified as a single value or a min-max range (e.g., 10 or 5-10). A single value indicates the exact number of blobs to be created. (default "1")
      --blob-sizes string      range of blob sizes to send (default "100-1000")
      --feegrant               use the feegrant module to pay for fees
      --grpc-endpoint string   grpc endpoint to a running node
  -h, --help                   help for txsim
      --key-mnemonic string    space separated mnemonic for the keyring. The hdpath used is an empty string
      --key-path string        path to the keyring
      --master string          the account name of the master account. Leaving empty will result in using the account with the most funds.
      --poll-time duration     poll time for the transaction client (default 3s)
      --seed int               seed for the random number generator
      --send int               number of send sequences to run
      --send-amount int        amount to send from one account to another (default 1000)
      --send-iterations int    number of send iterations to run per sequence (default 1000)
      --stake int              number of stake sequences to run
      --stake-value int        amount of initial stake per sequence (default 1000)
      --suppressLogs           disable logging

error funding accounts: tx was included but failed with code 11: out of gas in location: WritePerByte; gasWanted: 100000, gasUsed: 100375: out of gas%

Which is a gas issue, meaning that the transaction passed in the gRPC channel.
Given the original message in the first message in the issue:
grpc: received message larger than max (20016481 vs. 4194304)

It shows that the gRPC channel received 20mb of data vs 4mb that it's the default (not sure where this value is coming from, maybe some config in the test? I didn't dig into this one as I guess you have more info on this). The 20mb doesn't necessarily mean you sent a PFB of 20mb, but maybe it received 20 PFBs at the same time, each with 1mb.
I wanted to make the changes and test it on knuu, but it's too cumbersome and CI is taking too much time. Also, wanted to discuss this with you in case you already went this path so I don't waste time with CI.
Let me know what you think.
Note: I guess the process of waiting for the CI to build the image is unpractical and slow, we should improve on there.

to update, we can close this issue when we merge https://github.com/celestiaorg/celestia-app/tree/experimentation


Post filter: read response body

How it is possible to read a response body while using Zuul as a proxy in post filter?
I am trying to call the code like this:
@Component
public class PostFilter extends ZuulFilter {

    private static final Logger log = LoggerFactory.getLogger(PostFilter.class);

    @Override
    public String filterType() {
        return "post";
    }

    @Override
    public int filterOrder() {
        return 2000;
    }

    @Override
    public boolean shouldFilter() {
        return true;
    }

    @Override
    public Object run() {
        RequestContext ctx = RequestContext.getCurrentContext();
        ctx.getResponseBody(); // null

        // cant't do this, cause input stream is used later in other filters and I got InputStream Closed exception
        // GZIPInputStream gzipInputStream = new GZIPInputStream(stream);
        return null;
    }

}

You probably have to wrap the response stream in some way (similar to the way the request is handled in ServletInputStreamWrapper). We don't provide any utilities to do that in Spring Cloud, and I don't think it's provided in Zuul either.

Ok thanks!


fix(behavior_path_planner): unsafe mutex operation

Signed-off-by: NorahXiong<EMAIL_ADDRESS>Description

Fix unsafe mutex operation in behavior_path_planner.
https://github.com/autowarefoundation/autoware.universe/blob/main/planning/behavior_path_planner/src/behavior_path_planner_node.cpp#L516
If parking scenario is activated, the function will exit without unlocking the mutexes which belong to the class object. The program will try to lock the mutexes again when reentering the function and something unexpected will occur.
Related links

Tests performed

Tested in planning simulator.
Before: no route is generated when scenario changing from parking to lane-driving because behavior_path_planner is stuck

After: route is generated after scenario changes from parking to lane-driving

Notes for reviewers

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[ ] I've confirmed the contribution guidelines.
[ ] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[x] The PR follows the pull request guidelines.
[x] The PR has been properly tested.
[x] The PR has been reviewed by the code owners.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.
[ ] The PR is ready for merge.

After all checkboxes are checked, anyone who has write access can merge the PR.

@TakaHoribe @yukkysaito Please have a look.

LGTM @NorahXiong Is this ready to merge?

Sorry for the late reply. This is ready to merge.


WICKET-6688 add RFC and replace eval with DOM eval

Addressing https://issues.apache.org/jira/browse/WICKET-6688.
Didn't update any tests yet.
Differences from #378 :

replaced evals in wicket-ajax-jquery with jQuery.globalEval (DOM eval) with optional nonce
introduced a new meta response item which currently only has nonse, which is used make all the old code with appendJavaScript work.

That's it old code can work with strict CSP, without a need to rewrite.
The RFC is still here, because it's a way faster, than eval or DOM eval. see https://github.com/apache/wicket/pull/378#issuecomment-525530622

The script might contain multiple functions, each requesting a separate suspension. So it's safer to count them.

Thanks @svenmeier I reproduced the case and added the count back.
https://github.com/andruhon/wicket-delayed-animation/commit/b5d9408278376676d94ae1e4dc25f855846e7061
@svenmeier @martin-g do you think this is close to what we want to achieve? I'll create a new PR if yes.

Created a new one for globalEval #384

This issue can be closed now, right ?

Closed. Thank you!


FR: Vulnerabilities check fail in GoogleUtilities/AppDelegateSwizzler:6.5.1

We are using below dependencies in our projects
'FirebaseCore', '6.6.4'
'FirebaseMessaging', '4.3.0'
'FirebaseAnalytics','6.3.1'
We ran the Dependency checker to find the vulnerabilities in GoogleUtilities/AppDelegateSwizzler:6.5.1. Is this will be fixed in the future release.

@RameshIDPal Which dependency checker and what vulnerabilities?  Please provide more detail to help us investigate.

We are using OWASP Dependency checker and attached the screenshot below,

Thanks, but we still need the specifics of what vulnerabilities you'd like to see fix. You're also welcome to send a PR with the desired changes.

@paulb777 The vulnerabilities are

These messages don't refer to specific code. We will likely need file and line numbers to be able to progress.

Your dependency checker is mistaken. Those vulnerabilities refer to DeleGate, a network proxy. They were also issued in 2005, years before this software even existed.


[Feature] *webp format support? *

I get the message "some elements could not get processed" when trying to import .webp files which I assume happens because it's not supported? could it be an issue with the hardware?
anyway the reason I ask is because .webp files tend to come out much smaller than gifs, let alone mp4s etc. it would be greatly appreciated.

HI, this seems resonable. But backlog for now.
Thank you for your contribution :)


Enchancement - MDMS Configuration

@krishnaprasadsannidhi what's this task? Is this moving from MDMS v1 to v2?


Support converting '.NET Long Running Task' to the numeric thread ID on .NET 8

After upgrading to .NET 8 from .NET framework 4.7.2, the logs using the property %thread print a generic thread name (.NET Long Running Task) instead of the thread ID.
Reference issue: ".NET TP Worker"

@Audela-220 could you test the issue when we provide a preview version via nuget.org?

Hello @FreeAndNil
Sure, please let me know if you prepare a preview version via nuget.org.
Regards

@Audela-220 please test with https://github.com/apache/logging-log4net/releases/tag/rc%2F3.1.0-preview.1
https://www.nuget.org/packages/log4net/3.1.0-preview.1

@FreeAndNil
I did it. No problem.
Thank you!!


[CH-45]Support count(1)/count(*)

Relate to the issue https://github.com/Kyligence/ClickHouse/issues/45
What changes were proposed in this pull request?
A empty base_schema will be made in the scan node  when run the following queries
select count(1) from t;
select count(*) from t;
select sum(2) from t;
select 1 from t;

The backend source node will return empty data or throw an exception when there is no column to read.
So we fill a attribute to read in FileSourceScanExec.
Another problem is in ProjectExecTransformer. If ProjectExecTransformer's child is an ColumnarInputAdapter and projectList is empty, it would generate a scan node with empty base_schema. So when childCtx is null, we don't return directly.
How was this patch tested?
(Please explain how this patch was tested. E.g. unit tests, integration tests, manual tests)
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

Please add some UTs

In FilterHandler.applyFilterPushdownToScan , it also needs to handle like this.
BTW, please rebase to main

I don't think we need to add this in FilterHandler.applyFilterPushdownToScan. In this case, the columns to read are never empty

wait for https://github.com/Kyligence/ClickHouse/pull/175 be merged


logo standardization

Is there an existing issue for this?

[X] I have looked through the existing issues

Current Behavior
Currently we are using an emoji as a logo: 👀, but it renders differently on different operating systems.
We should come up with a logo or think of something else.
Expected Behavior
No response
Steps To Reproduce
No response
Anything else?
No response

https://github.com/BuidlGuidl/address-vision-port/assets/108868128/1534b1d5-b33b-4284-8831-5b5348ecf065
Perhaps using a kaomoji could be cool? lmk what you think

How about using an SVG rather than text?

I think emojis are copyrighted if you render it anywhere but the client @ceeriil

was just looking around...
I found this: https://github.com/microsoft/fluentui-emoji
which is amazing! The emojis have the MIT license!

Finally closing, now everyone will see the same emoji in address.vision!


Module Requirements Update, Issue #85

Issue link: https://github.com/Yelp/puppet-uchiwa/issues/85
I took a look at the module to make sure there wasn't anything that wouldn't function with the latest version of puppetlabs-apt (5.0.1), and it seems pretty straightforward, nothing that shouldn't work.
I have the module running cleanly on Ubuntu 14.04.5 and 16.04.4 so I added those to the supported OS list as well. I see no reason why this wouldn't run on Debian 8 at the least, and probably 9 as well, but I didn't have any VMs handy to try installing on.
I noticed a bit of redundant logic in the params class and cleaned that up also.

Thanks!


Internal error: Geometry delta is too big

I'm curious about the following error:
Internal error: Geometry delta is too big: -4188947,2528672683
This is coming while tiling about 600 million OSM objects (buildings) with the following flags:
-Pf -Z14 -z14 -d18 -pf -pk -pg -pD -ps -b0
Would changing to z12 -d20 solve this for any reason?
Thanks!

In the old version, the corrupted geometries would still be tiled, and would just be wrong in the tile.
The error is detected quite late in processing so it would be hard to print the original GeoJSON. It would be possible to print the attributes, though, and to optionally leave out the feature.

Makes sense -- Yes, the ability to optionally leave out the feature instead of stopping completely (and maybe report on the number of failed features at the end and/or print the attributes would be very helpful; I'm not sure if it'd be useful for any other users... but it would save my bizarre use cases.


Istio Metric data now showing on Skywalking dashboard on EKS cluster.

Please answer these questions before submitting your issue.

Why do you submit this issue?
[x] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?

Bug

Which version of SkyWalking, OS and JRE?

Which company or project?

What happen?
If possible, provide a way for reproducing the error. e.g. demo application, component version.

Requirement or improvement

Please describe about your requirements or improvement suggestions.

After installed Skywalking on EKS Kubi am not seeing data on Skywaking dashboad

Do you set the mixer adopter or open als?

I have followed this document https://github.com/apache/skywalking-kubernetes/tree/master/archive to install Skywalker on EKS.

I have installed below yaml file tto push metrics into Skywalking.
skywalking-kubernetes/archive/6/6.0.0-beta/istio.
01-metrictemplate.yml
02-skywalingadapter.yml
03-operator_cfg.yml

@wu-sheng ,
Is there any other steps to set mixer adopter ? Can you please help me on this issue.

@wu-sheng ,
As part of istio setup we have installed Prometheus for istio on eks.

Do you run this? https://github.com/apache/skywalking-kubernetes/tree/master/archive#setup-istio-to-send-metric-to-oap-1
Istio mixer adopter should be config. And you could check istio mixer log.

Thanks for replying @wu-sheng ,Yes i have performed below url all steps on EKS/Istio but still i dont see istio metrics on Skywalking dash board.

followed url steps : https://github.com/apache/skywalking-kubernetes/tree/master/archive#setup-istio-to-send-metric-to-oap-1

If you are familiar with istio, you need to learn about istio mixer, check the istio mixer logs, at least, you make sure mixer bypass adopter is working.

I found the issue and now able to see data on skywalking.
issue is not able to communicate on 11800 port for oap service.

Thanks @wu-sheng


remove redundant code

Sorry, this fell off my notifications.
Merged in https://github.com/SanketDG/gitdl/commit/b03ad9d65b83baf1fbaaeaafc8fc28725ae10830
Thanks!


bug: iOS (Safari) scroll bounce jitters when IonButton is target of touch scroll

Bug Report
Ionic version:
[ ] 4.x
[x] 5.x
Current behavior:
In iOS, using @ionic/react, if a user scrolls until triggering the Safari "scroll bounce" while an IonButton is under the user's finger, the bounce will jitter.
This jitter is similar - if not identical - in appearance to that depicted in issue https://github.com/ionic-team/ionic/issues/20486
Presumably the technical explanation is similar to what is described in this comment of the same issue:
https://github.com/ionic-team/ionic/issues/20486#issuecomment-586459674
(Please note, the cause is not the same as in this issue; no custom CSS is involved in the behavior described here.)
Additionally, this behavior can be triggered by an IonItem if it is set as a button (button={true}) and forced to display as "md" (mode="md"), then scrolling in iOS as described above.
Expected behavior:
Expected behavior is a smooth scroll and single bounce, as occurs when other elements are under the finger on a touch scroll.
Steps to reproduce:
This behavior appears in Ionic's Conference demo app:
https://github.com/ionic-team/ionic-conference-app
Load the demo app in iOS, navigate to the "About" page, and drag downward on either of the buttons appearing at the top (IonMenuButton on the left; IonButton with ellipsis icon on the right). The behavior is exaggerated by faster and longer drags.
Related code:
Please see Ionic's Conference demo app, page "About":
https://github.com/ionic-team/ionic-conference-app
I have recreated the behavior in other contexts; it appears to require nothing more than:
 <IonPage>       <IonContent>               <IonButton />       </IonContent> </IonPage>
Other information:
N/A
Ionic info:
Ionic:
Ionic CLI       : 6.10.0
Ionic Framework : @ionic/react 5.0.5
Capacitor:
Capacitor CLI   : 1.3.0
@capacitor/core : 1.3.0
Utility:
cordova-res : not installed
native-run  : not installed
System:
NodeJS : v10.15.2
npm    : 6.13.1
OS     : macOS Mojave

Thanks for the issue. I am going to close this as this is not a bug in Ionic Framework.
I have seen this bug appear a few times before, and it turns out this is a bug in WebKit. I have reported this bug to the WebKit team, so hopefully they can resolve it soon: https://bugs.webkit.org/show_bug.cgi?id=209968.

Thank you Liam for your swift response!


[SPARK-10938] [SQL] remove typeId in columnar cache

This PR remove the typeId in columnar cache, it's not needed anymore, it also remove DATE and TIMESTAMP (use INT/LONG instead).

cc @liancheng @rxin

Merged build triggered.

Merged build started.

Test build #43261 has started for   PR 8989 at commit e74d7fd.

LGTM

Test build #43261 has finished for   PR 8989 at commit e74d7fd.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
public class BufferHolder
public class UnsafeArrayWriter
public class UnsafeRowWriter

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

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #43273 has started for   PR 8989 at commit 55c2e59.

Test build #43273 has finished for   PR 8989 at commit 55c2e59.

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

Merged build finished. Test PASSed.

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


Add Transactional Behavior to Your Core Data Services

Tutorial URL: https://developers.sap.com/tutorials/abap-environment-transactional-enablement.html
Please specify the step you are referring to 'Step 4: Create behavior implementation'
Hi,
I cannot find the 'New Behavior Implementation' after the behavior definition is created in the Ecliplse ADT. Please check more detail in the attachment. Thank you.

Hi @Ouarzazate ,
thank you for your feedback. We will update this as soon as possible. Please do following:
Right-click on your package > select file on your menu > new > other > search for behavior implementation and select it.
Best regards,
Merve

Hi Merve,
Thank you for your reply, I've just tried it, but cannot find 'Behavior Implementation' in the menu, please see in the attachment. Does it mean I miss some role, maybe?
Kind regards,
Hui

Hi @Ouarzazate ,
no, you have the right roles. The UI just changed, we will update this as soon as possible.
Best regards,
Merve

Hi @MichaelCzcz,
we can close this ticket.
Best regards,
Merve


Plugin "react" was conflicted between "./.eslintrc" and "BaseConfig

Describe the bug
eslint-plugin-react conflicts with CRA base eslint config
Steps to reproduce

Create a react-app
Setup Eslint and extend eslint-plugin-react
Removed the default Eslint config in package.json
yarn build

Failed to compile.

Plugin "react" was conflicted between "./.eslintrc" and "BaseConfig » /Users/xxxx/xxx/project/web/node_modules/eslint-config-react-app/base.js".

Did you try recovering your dependencies?
I have tried to reinstall dependency and start a fresh react-app project
Environment
  Binaries:
    Node: 16.13.1 - ~/.nvm/versions/node/v16.13.1/bin/node
    Yarn: 1.19.2 - /usr/local/bin/yarn
    npm: 8.3.0 - ~/.nvm/versions/node/v16.13.1/bin/npm
  npmPackages:
    react: ^17.0.2 => 17.0.2
    react-dom: ^17.0.2 => 17.0.2
    react-scripts: 5.0.0 => 5.0.0
  npmGlobalPackages:
    create-react-app: Not Found

Duplicate of #11825


Return error to the client when exception is thrown from Elasticsearch impl

Add configurable boolean member for enabling throwing exceptions when they occur in ElasticsearchImpl instead of catching them in class loader. The purpose is to return error response to client requests instead of 204 no content in case an error occurred.
Added exception mapper for returning internal server error with 500 status code.
Note: This PR has some indentations due to try-catch additions, for better readability you can add "?w=1" to the URL

Hello @liatiusim , following the merge of another commit this PR now conflicts, could you fix it please ?

Sure, I'll fix that soon

thank you

Done


Removal of addwu/subwu/addiwu instructions

We are considering removal of addwu/subwu/addiwu from the bitmanip extension. They are very cheap to implement, but there are concerns if they add enough to justify even the small implementation cost and/or ISA encoding space cost.
Please post relevant code examples and benchmarks below.
(Note that all *wu and *u.w instructions can only replace an add/sub instruction and a pack instruction. So they never replace more than two instructions, thus only relatively common code patterns are of interest.)

If we remove addiwu then it won't be possible anymore to load a zero-extended 32-bit value into a register in only two instructions with lui+addiwu. (Note that addi+pack can handle the cases that lui+addiwu can't handle.)

There might not be a need to materialize a 32-bit zero-extended constant if it is only consumed by prefix-zeroing instructions including stores.

I would guess there are many cases where they are consumed by logic, arithmetic, or bitmanip instructions. This needs the be benchmarked ofc. But unfortunately that's hard with current https://github.com/riscv/riscv-gcc/tree/riscv-bitmanip. For example:
unsigned long foo() { return 0x00000000fedcba98UL; }
unsigned long bar() { return 0x00000000fffffabcUL; }

Produces (irrelevant lines removed):
$ /opt/riscv64b/bin/riscv64-unknown-elf-gcc -o - -S -Os -march=rv64ifdb demo.c
foo:
	li	a0,1068969984
	slli	a0,a0,2
	addi	a0,a0,-1384
	ret
bar:
	addiwu	a0,zero,-1348
	ret

foo() should be lui+addiwu, but instead it's a three instruction sequence<PHONE_NUMBER> = 0x3fb73000).
bar() can be implemented in a single addiwu and gcc does that correctly.
Looking at this I think benchmarks with current gcc under-count cases where addiwu could be useful to materialize constants. (This should be easy enough to fix tho ofc.)

Was this completed by https://github.com/riscv/riscv-bitmanip/commit/ba8677c2d9b2b35edff38c0f1e78f35c08e873d0?
Because ADDIWU is still in the encoding tables, it took me quite a while to figure out what was going on with that instruction.  The minus sign and the sentence at the very bottom of page 45 were easy to miss.  Can this encoding changes table be put in its own section 2.12.1 or in some other way clearly distinguished from the rest of the tables?

And was https://github.com/riscv/riscv-bitmanip/commit/ba8677c2d9b2b35edff38c0f1e78f35c08e873d0 merged directly to master without a PR?  Because this issue was never closed and there was no PR, people watching this repo didn't get a notification of the change.

Rolled up into the public review release 1.0.0.


Fan control sliders should have text boxes for % values

I have figured out that scrolling moves the sliders by exactly 5%, but this isn't exactly pretty.

I know this isn't exactly a full solution, but if you use arrow keys you can adjust by 1%.

Hey I love your Project. LACT works perfectly fine on openSUSE Tumbleweed and I'm very happy that your next Version will include an openSUSE Tumbleweed.rpm
as @43615 mentioned, I would love a % slider as well, otherwise LACT is perfect!
Thanks for the tip with the Arrow keys.
I'm no programmer, just created this account today.
Thanks for your work. @ilya-zlobintsev

This has been implemented in https://github.com/ilya-zlobintsev/LACT/commit/d1acdde89caa00e2fa3b48fe55e0f7c7d2093529:


Sysmon for Splunk App not displaying any data from Windows Event Collector forwarded logs.

Sysmon for Splunk App not displaying any data from Windows Event Collector (we are using Windows Event Collector as a single forwarder). How might I troubleshoot and reconfigure if necessary in my deployed environment?
Sysmon data is in Splunk.
sourcetype =	WinEventLog:ForwardedEvents
I can see logs in Splunk when I search under  the data feed Host (in my environment is is called): WinEvtCollector
I tried to adjust the macro using the above sourcetype but that did not work. I think I'm missing some fundamental piece of info here.
Maybe because I'm collecting my logs from a single Windows Event Forwarder and not the individual boxes themselves (maybe)?
It looks like the app is based on the idea that a person would be sending their logs from the windows box straight into the Splunk instance and not through a Windows Event Collector.
Can you provide me any help in understanding this so that I can utilize your app? It looks amazing.

Are you sure you are searching in the correct index? If so can you make sure the data is parsed as XML, the fields should be properly extracted in order for the app to work.
You also need to have the Splunk for Sysmon TA installed

It was installed. I was using both the Sysmon TA nad Sysmon for Splunk apps.  I have deleted them both. I'm going to reinstall just the Sysmon TA and see what happens.

So I have fixed it and in doing so I have learned a lot.  For those who would follow in my foot steps.
Their are multiple "Sysmon" apps in the splunk app store.
You must download the Sysmon TA app first. Official name is: Microsoft Sysmon Add-on (Folder Name TA-Microsoft-sysmon)
This will add another data source under your Splunk data sources. In my deployment I am pulling sysmon data from a Windows event collector which is my first data source.
The Sysmon TA app appears to reparse that data. Not positive about this but that is my assumption.
When you look at the data from the Sysmon TA app data source it will be in a raw xml format. Basically all mashed together.
Download a second app called Sysmon App for Splunk.
This will parse the information into already built searches on a nice dashboard. Make sure that you hit submit on the time <Last 24 hours (or your setting)> because by default it does not pull the data.
And it is really that simple.
Thanks for the help olafhartong.

Hi @johnyb0312,
The steps are outlined on the github Readme and on the Splunkbase setup guide. Did you see them?  Are they clear enough on the setup?

So, I posted above because I feel some background about the different apps would help. And since I am new to this I wanted to share my struggle with the anyone else.
I'm working on a dev version of my deployment (and was able to get it completely working).  However, my production will not display the logs from the TA. A guide that really helps understand what the system is doing would assist in troubleshooting.
So, although what was posted is accurate, I would love a little more depth (just because I'm so new to this).
If you happen to know why the data source from the TA is not displaying or where to look that would be most helpful.  I will continue to look.
Thank you for your response.


build: remove unused config line

build: remove unused config line

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


WIP

Signed-off-by: Brett Logan<EMAIL_ADDRESS>

Push common functions out into a sourceable script and import them


Fix exception handler for web requests

Description
Call the settings' defined exception handler for web requests. For this and the tests I had to also include the following changes:

Make futures from requirementst.txt only install for python 2.7.
Add a mock_exception_handler that is reset on tests.test_handler.TestZappa.setUp so it's state is correct for every step.
Added a tearDown method so you can use different settings in each test (currently it's always reusing the same config).
Added tests for checking the work of the exception handler on bot events and web requests.

GitHub Issues
#1353

Coverage increased (+0.2%) to 74.221% when pulling bdeab7d5d865d4799fcb830f98d2fc4d97b1c9ae on jneves:fix-exception-handler-for-web-requests into d59f450cd5d21635638f036664ee6ea4f1911cfd on Miserlou:master.


child record is setting parent record data

If a parent record and child record have the same column names with different values and different ids:
-child record sets parent record values
-changing the child record value updates the parent record value
--changing the parent record value DOES NOT update the child record value which is correct
--if the parent record value is changed first it stops updating the child record
see sample json file

fixed by PR #79


I'm attempting to open a new window form (.net 7) on the MessageReceived method; it initially opened in a non-responsive state and subsequently crashed.

Describe the bug
I have 2 desktop based application one is legacy one (in .net framework 4.7.2) and another is new desktop app in (.net 7), and I want to open new app through legacy app, it means i want to pass some data from legacy desktop app to new app and based on that data want to open a specific or related form, but when i received data and able open new form but the form not responding and it automatically closed after some time. I have adding my both application here.
LaunchPadOld - Legacy Code
LaunchPad - New Code.
Please help me
Steps to reproduce the bug

First run LaunchPadOld application
Run LaunchPad
LaunchPadOld => Click Connect => Send Empty Data (For Now)
You can see LaunchPad open new form but it's not responding mode and it's crashed after some time.
c# winforms code with pipes.zip

Expected behavior
If we open this form without message received then it's working fine
Screenshots

NuGet package version
No response
Platform
Windows Forms
IDE
Visual Studio 2022
Additional context
No response

@HavenDV - I need your help here. Thanks

Very busy work schedule right now, I work 60 hours a week and I’m unlikely to be able to help in the near future
One idea - are you sure you're using the main thread? Because by default events come from the thread pool, which will not work with UI elements

@HavenDV Thanks for your reply, and I am sorry that I bother you. So In my case how I can open any new window form based on message received? This is very critical for me and i need some solutions. Need your guidance. I have tried everything but didn't get any luck. I apricated your hard work. Thanks

Please check my demo code; it will give you more contect about my concern. Thanks

Either call the Invoke method on an existing UI element, or save SynchronizationContext.Current somewhere (in the main thread) and then use it after receiving events
May be this can be useful: https://stackoverflow.com/questions/661561/how-do-i-update-the-gui-from-another-thread


[Improvement] update Menu item (Dropdown item) to accommodate designs for Coaching

Description
Add an option for Menus (dropdowns) that line up with designs for Coaching
Design
Coaching usecase
Applicable Figma reference - does not include this variation
Additional Information
Screenshots

These look nice! Here I see a lot of similarity again with existing components. In case its helpful:

We have a dot effect like this in the recent updates to StatBox with legend_dot. Are these sizes and color options the same? If not, should they be? Should this be a new component we can drop into layouts (kind of like icons?) with some props for adding/enforcing necessary a11y features?
If this has same interactive effects as standard dropdown items, perhaps we can consider a way to open up dropdown items to take a custom inner content (if they don't already?)

Once again it seems that some effort to reconcile this variation on dropdown items with the main library of Sage things would be beneficial before we build it out.

Spoke with @QuintonJason and we will work on constructing a collection of components for this use case, within kajabi-products rather than a custom component or update to an existing component in sage-lib. Closing for now.
Relevant ticket for BUILD: https://kajabi.atlassian.net/browse/BUILD-1623


Cannot load Ticker.h library (conflicts with the default Ticker.h library)

Hi, I am using Arduino IDE 1.8.15 and that has a default library called Ticker.h. I installed your library, but if I try to use it, Arduino IDE refers to the default Ticker.h library and not yours. So it says error such as "error: 'class Ticker' has no member named 'start'", "error: 'class Ticker' has no member named 'update'", "'class Ticker' has no member named 'remaining'". Is there any solution for this issue? Thank you.

Ok, it seems you use an ESP. You must rename the Library items.

Yes I use ESP. Can you please tell me what exactly do I need to rename? I renamed the Ticker.h and Ticker.cpp to TickerESP.h/TickerESP.cpp but arduino IDE cannot recognize it. Thank you.

In the .cpp you must also change the #include "Ticker.h" to #include "TickerESP.h"

Renaming the #include solved the issue. Thanks.

have a look to the TickTwo library


Make the release script tag "latest" on operator releases

I've tested it on a branch in the zookeeper-operator (https://github.com/stackabletech/zookeeper-operator/tree/test-1) and it successfully deployed the updated tag to the repo.


Decide how to initialise new maverick repo

fork of old repo
import?
mass commit

Would prefer not to be a fork - should be a new start, but there are a LOT of commits and issues in the old repo that have been used to document that development of the project.

you can transfer the repo.  but if we do that then we need to remove this maverick repo (or rename it).
https://help.github.com/articles/about-repository-transfers/
up to you @fnoop seems like there is some good info in the issue queue you may want to preserve?

https://github.com/ahadik/git_mover/blob/master/README.md
This tool looks like a possible option to keep the past issue, milestone
and label goodness while moving the repository. The difference with this
and the github option is that the old repo will continue to exist and the
redirects won't be in place.
On Mon., 1 Jan. 2018, 1:00 am Coby<EMAIL_ADDRESS>wrote:

you can transfer the repo. but if we do that then we need to remove this
maverick repo (or rename it).
https://help.github.com/articles/about-repository-transfers/
up to you @fnoop https://github.com/fnoop seems like there is some good
info in the issue queue you may want to preserve?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/goodrobots/maverick/issues/2#issuecomment-354605384,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AH6BeNfdAYom4xtKcfDWQ1D-a0xMrPV8ks5tF5OLgaJpZM4RPyh7
.

Oh cool, the transfer repo looks good.  Keeping all the commit and issue history would be a good thing I think.  I'll probably close out most of the open issues to give it a fresh start.  That OK with everyone?

All good with me! I can see the transfer worked :)

Does that look OK?  The open issues are mostly quite useful so I left them in place for now.    We should use projects going forward, they look great for structuring goals and releases.

Looks good and sounds like a plan!


fix broken documentation markup

(I would have indented the code block one more column, but for some reason doing that causes rustdoc to crash.)

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @huonw (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. The way Github handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see CONTRIBUTING.md for more information.

Thanks!

for some reason doing that causes rustdoc to crash.)

Indenting 4 spaces makes the whole thing considered  code block, including the ```'s, which aren't valid Rust so rustdoc (which tries to tokenise it to syntax highlight, and compile it to run tests) will be unhappy. In any cas, I think the indentation is unnecessary, it's just putting the blank line between the paragraph and the start of the codeblock (the ```rust) that fixes this problem.
Also, according to travis, making it considered code properly is causing it to fail tests (i.e. it wasn't valid code), the fix would be to import the two types from time use time::{PreciseTime, Duration}; and to define the do_some_work function, even just fn do_some_work() { /* ... */ } at the  end of the code block is good enough.

Thanks for the review! I've undone the indentation and added the necessary imports, the tests pass now. Should I squash this into a single commit?

Yes, squashing would be useful if you can.
(Sorry for the delay @fhartwig, if I'm ever so slow again feel free to comment as a reminder.)

Squashed and rebased. No worries about the delay, this wasn't particularly urgent (-:


Update automate-backend-* components to ES 6.8.23

:nut_and_bolt: Description: What code changed, and why?
A customer trialling HA backend notes that while we have updated
standalone A2 to ES 6.8.23 to mitigate Log4J we didn't make the same
changes to automate-backend components for HA, specifically:-

automate-backend-elasticsearch
automate-backend-kibana
automate-backend-journalbeat
automate-backend-metricbeat

For more details on the affected customer discuss with Ram Joshi and @collinmcneese
Signed-off-by: Richard Nixon richard.nixon@btinternet
:chains: Related Resources
See previous PR https://github.com/chef/automate/pull/6609
:+1: Definition of Done
:athletic_shoe: How to Build and Test the Change
No special requirements. Use the standard test/release processes for the listed components
:white_check_mark: Checklist
All PRs must tick these:

[x] I have read the CONTRIBUTING document.
[x] All commits signed-off for the Developer Certification of Origin.

With occasional exceptions, all PRs from Progress employees must tick these:

[x] Is the code clear? (complicated code or lots of comments--subdivide and use well-named methods, meaningful variable names, etc.)
[x] Consistency checked? (user notifications, user prompts, visual patterns, code patterns, variable names)
[ ] Repeated code blocks eliminated? (adapt and reuse existing components, blocks, functions, etc.)
[x] Spelling, grammar, typos checked? (at a minimum use make spell in any component directory)
[x] Code well-formatted? (indents, line breaks, etc. improve rather than hinder readability)

All PRs from Progress employees should tick these if appropriate:

[ ] Tests added/updated? (all new code needs new tests)
[ ] Docs added/updated? (all customer-facing changes)

Please add a note next to any checkbox above if you are NOT ticking it.

Superseded by #6795


[Event Modification] codeunit 5813 "Undo Purchase Receipt Line" OnAfterCopyItemJnlLineFromPurchRcpt

Can you please add the following to an event
        OnAfterCopyItemJnlLineFromPurchRcpt(ItemJnlLine, PurchRcptHeader, PurchRcptLine);

to
OnAfterCopyItemJnlLineFromPurchRcpt(ItemJnlLine, PurchRcptHeader, PurchRcptLine, WhseUndoQty);
WhseUndoQty as var

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Playground snippets do not register clicks

All clicks in the marked area of a Playground snippet are ignored. This results in a playground selection not being performed as requested by the user.

Lepiter replaced Playground and this issue is no longer relevant.


correct errors on Closures.md & Decorators.md

I correct some errors in above files.

Fixed, thanks.


Feat(kclvm-win): Before calling the python version of kclvm in kcl.go check if kclvm is installed.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

issue #357

2. What is the scope of this PR (e.g. component or file name):

kclvm windows.
3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[x] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Other

Before calling the python version of kclvm in kcl.go check if kclvm is installed. If it is not installed, call pip to install it.
4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[ ] Unit test
[ ] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[x] Other

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 89.712%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1369

Relevant Lines:
1526

💛  - Coveralls


Remove npm scripts.

No reason in duplicating these since we have the node make targets.

I don't mind either, but duplication is always bad... For example, we have "test-nc": "node make test-nc" which doesn't exist.
That is why I prefer to have everything in one place, using one way: the way I also use myself...

👍 🚢


Add selinux test from #486

Signed-off-by: TomSweeneyRedHat<EMAIL_ADDRESS>Adds a test to the baseline test suite to make sure an selinux issue is no more.  This is not run by the CI tests systems, only by hand to check on the health of a new kit.

LGTM


Render examples

IT WORKS!

But we should investigate how to drop Babel-standalone and compile the code in the backend with Babel.
@jantimon I ran into an issue here where Babel causes a loop or so. Maybe we can improve this together.

The example doesn't work for me, going to /styleguide renders an empty white page for me.

Going to merge this because I want to continue working, but please fix the examples!


quote image

I wonder if a solution is to reduce the height of the image, cutting off a good chunk of the sky (either by reducing the window frame or cropping the image). Then instead of pinning the text to the bottom, pinning it to the top, just below the skyline?

I removed a chunk of sky from the image, and adjusted the position of the text to be positioned from the top rather than the bottom.

Review small screen behaviour next.

Looks ok to me


Syntax error on last line reporting empty range from checkDSL

One error I'm encountering;

If I make an error here like in the attached screenshot (removing a closing bracket on the last line), it reports an invalid range. I get the following error contents;
{
  severity: 8,
  startColumn: 0,
  endColumn: 0,
  startLineNumber: 21,
  endLineNumber: 21,
  message: "Invalid syntax",
  source: "linter",
}

Right line, but 0s on the start & end cols.

@d-jeffery is this still the case or can we close this issue?

@rhamzeh can't replicate anymore, so we should be good to cose.


Tears of Guthix plugin to track tear streams' guaranteed 9 second minimum lifespans

Around 2 months ago Tears of Guthix received a rather significant buff. That buff was guaranteeing that tear streams will last a minimum of 9 seconds before vanishing

Please create a plugin that tracks tear streams to indicate their remaining guaranteed life span
I would recommend color coded pie timers that would count down from 9 seconds to 0. As for colors you could either green and blue to match the good and bad tear colors. Or maybe Blue and Red to be easier to discern quickly.

I have the code for this thats a month or two old, I guess I'll work on updating it when I get the chance

How would you do it once its past 9secs though? Because as far as i know there is no maximum time

i guess just create some red marker?

No they disappear after 9 seconds

Looks great! Exactly what I had in mind

Alright I guess I'll update it/clean it up and push it

Yep #3475, changed it to use the new ProgressPieComponent we have which made things cleaner and simpler, so thank you to Kamiel


Fix missing prompt configuration in DynamicProjects example

Description
Trivial fix for missing default argument.
Checklist

[x] I've read CONTRIBUTING.md

[x] I've considered how to best test these changes (property, unit,
manually, ...) and concluded: trivial docs fix, no tests required

[ ] I updated the CHANGES.md file (not required)

Thanks!


Update angular to ^5. uses @scope properly

Please, merge this changes to upgrade angular dependencies from ^4 to ^5.
Also, this is on it's way to handle #10 and #12
@a1300 May be I am going to need your help to complete fix for #10 and #12, would you mind?

@mautematico I will have a look :-)

Done. #007350c830a3967c19f702b7b5051085793fc9ff fixes #10, #11 and #12
@techiediaries, please merge/rebase? It is a fast-forward.

Wait. So there's no support for angular 4 anymore? No backward compatibility?...


Make elytra and shulkers renewable

Elytra: add to dragon loot?

Shulkers at the very least are renewable via the Cocoon of Caprice. A better way would be ideal I feel.
As for Elytra, adding it to the Ender Dragon's loot would work. A Datapack could probably work, and there is definitely a mod out there to do this as well.


Doc/add mae benchmark

add mae large benchmark

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.yanhaiqiang.yhq seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[Enhancement] Add related reference contents in the same doc

Description
Some examples on the website that use context.configMap does not provide a sample ConfigMap content as an example. Such as the following link: https://kyverno.io/policies/other/allowed-base-images/allowed-base-images/
There is a config map called baseimages and have a key allowedbaseimages. I searched this name on both of policies and kyverno repos and couldn't find much. So It would be nice to have related reference's content in the same doc.
An end user would expect the content of external sources. (baseimages configmap, in this example)
Slack discussion
No response

@Dentrax in almost all cases, there's really nothing special about the data stored in these ConfigMaps as long the contents are in an array as described here. I have fought with how/if to provide an example of these ConfigMaps in the policy definitions themselves and it's a bit problematic because users often download directly from kyverno/policies the manifest they want and this would give them something other than policies. What you see on the website is only a rendering of the policy manifests stored there and they are programmatically produced, not hand written like documentation in other places.
If you have a better idea of solving this I'm glad to know.


Fix mailto: URL handling in ImprintFragment

android.net.MailTo is broken and not necessary at all in this case
See https://cketti.de/2020/06/22/android-net-mailto-is-broken/
"android.intent.action.SENDTO" is the proper Intent action to send emails
See https://cketti.de/2016/01/08/sending-email-using-intents/
Querying PackageManager is unnecessary and doing so will require more work with Android 11
See https://developer.android.com/preview/privacy/package-visibility

Hi @cketti,
thank you for your contribution. It's awesome! Plus: you are the first community contributor. 🎉😃
As far as I can tell your contribution will be pulled, but it might take some time before we actually do.
I'm looking forward to receive more of those pull requests. Thanks again!


flex:1 is not working?

I want use flex:1 on childNode's height

See #42 , the "flex" style isn't currently supported. Try using flex-grow: 1; instead.

thx

@DaJobat thanks!


Fixed Makefile.release

@gaby mind taking a quick look?

@runabol Looks good to me, I would suggest looking into goreleaser. https://goreleaser.com/ i dont have any experience with it but a lot of projects use it to publish binaries


add small note for spacing without padding

I recently saw a note on slack that "padding doesn't seems to work on line 142. i wanted to shift the text away from the edge, at least 30px".
Just remembered that I also had this weird moment in the beginning, where I forgot about all CSS words when working with elm-ui and totally had the same "hey, why is spacing not respecting the spacing to the border"?
Feel free to disapprove if you think this is too much of a newbie concern or if this place is not the right one to add it.

Thanks


Implement the database tables

Priority: Medium
Description:
Implement the database tables as prepared in the design.
Estimated time:
1h


Add documentation for cli usage & idl-format

As a user I want to get a quick overview in the Readme on how the CLI works & what the djinni-IDL is.
To be defined: Should the in-detail djinni-IDL documentation be in the main Readme like in the old repo, or should it be in a separate document, e.g. in a subfolder /doc?

Hi,
Subfolder will be nice.
Thanks.

Thanks for you feedback. :) We are currently planning to have a single website that pulls the documentation from all repositories related to djinni and displays it on one website: https://github.com/cross-language-cpp/djinni-documentation
More details can be found here: https://github.com/cross-language-cpp/djinni/issues/17
To make this work I will have to put all more detailed documentation in a subfolder.
I'm closing this issue, the decision is already made.
Pls. be patient regarding the documentation, we are working on it (or at least think about it 😋 )

Oops, the issue is about initially adding the docs at all, I need to reopen it. Sry for the confusion 🙈


support adaptive icon

It seems that pkg.Icon(nil) fails if apk uses adaptive icon format

@fisherWang2017 I just found some random apk and got icon error. Is this a same case? File: file.apk.zip

this code fixed my same problem
icon, err := pkg.Icon(&androidbinary.ResTableConfig{
    Density: 720,
})

Adaptive Icon is available from SDK Version 26.
So, adding SDKVersion: 25 may be a workaround.
icon, err := pkg.Icon(&androidbinary.ResTableConfig{
    SDKVersion: 25,
})


Update to latest otel version

Might it be that renovate doesn't grok the gradle.properties approach here?

It seems like it doesn't 😞 https://github.com/renovatebot/renovate/issues/6220
Although it's quite strange because they say they do in their docs. I'll take a deeper look into this this week to see if there's something we can do to make it work, otherwise, we'd probably have to define these versions somewhere else.


[MRESOLVER-272] Update parent POM

Remove version overrides and update bnd plugin
as well.

https://issues.apache.org/jira/browse/MRESOLVER-272

@michael-o take a look at this one as well ,please.... seems some ignores changed, similar to those you discuss on ML?

Yes, @kwin knows how to deal with.

@cstamas Can you enable https://creadur.apache.org/rat/apache-rat-plugin/check-mojo.html#consoleOutput in the root pom.xml's plugin configuration so that it is more easy to see where the failure is? I assume it is related to https://issues.apache.org/jira/browse/RAT-308. We can either add the Apache License header to it or add it to the ignores.

I added .gitattributes to excluded list for RAT in https://github.com/apache/maven/pull/792

Not sure about IP of a gitignore file but to me it seems safer to add an Apache License header to it according to https://www.apache.org/legal/src-headers.html#faq-exceptions.

IMHO, git config files carries no IP...  they are more "environment" or so...

I don't see creativity in .git* files - so IMHO header is not needed


App freezes after scanning more than 2 objects.

Greeting again,
Currently when I testing the app, the app will start freeze when the app detect more than 2 objects.
I just wonder that is it possible to allow the app to stop scanning after 2 objects is detected?
by the way, this is the Unity Version I'm using,
Unity3D 2020.3.16f1
and here are the phone that I'm testing with,
Samsung s10 Plus
Edit: I also using YOLO 3 Tiny to test to scan everyday item.

the same question
Unity 2021.3.1f1
Galaxy fold3,  redmi note 10

Did anyone have any luck solving this issue?

Same issue, any luck anyone?


az webapp up --->error webapp doesn't exist

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az webapp up --resource-group learn-c38dc864-20ad-4d06-8c3b-bb014cd8718a --sku F1 -n educationapp-$RANDOM
Errors:
webapp educationapp-30191 doesn't exist
'NoneType' object has no attribute 'upper'
Traceback (most recent call last):
python3.6/site-packages/knack/cli.py, ln 206, in invoke
cmd_result = self.invocation.execute(args)
cli/core/commands/init.py, ln 603, in execute
raise ex
cli/core/commands/init.py, ln 661, in _run_jobs_serially
results.append(self._run_job(expanded_arg, cmd_copy))
cli/core/commands/init.py, ln 652, in _run_job
cmd_copy.exception_handler(ex)
...
cli/command_modules/appservice/custom.py, ln 2993, in webapp_up
create_app_service_plan(cmd, rg_name, plan, _is_linux, False, sku, 1 if _is_linux else None, location)
cli/command_modules/appservice/custom.py, ln 1390, in create_app_service_plan
sku = _normalize_sku(sku)
cli/command_modules/appservice/utils.py, ln 20, in _normalize_sku
sku = sku.upper()
AttributeError: 'NoneType' object has no attribute 'upper'

## To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

- _Put any pre-requisite steps here..._
- `az webapp up --resource-group {} --sku {} -n {}`

## Expected Behavior

## Environment Summary

Linux-4.15.0-1063-azure-x86_64-with-debian-stretch-sid
Python 3.6.5
Shell: bash
azure-cli 2.0.76
## Additional Context

<!--Please don't remove this:-->
<!--auto-generated-->

This is fixed by #11019

Still getting the same issue.  webapp doesn't exist.

The fix will be released with 2.0.77

Please downgrade to 2.0.75 for this to work. Sorry for the inconvinience
pip install azure-cli==2.0.75

Please update to the latest version of CLI https://pypi.org/project/azure-cli/ version 2.0.77 and re-try. there was a known regression with version 2.0.76

Same problem here.
az --version
azure-cli                         2.11.1 *
core                              2.11.1 *
telemetry                          1.0.5 *
Python location 'C:\Program Files (x86)\Microsoft SDKs\Azure\CLI2\python.exe'
Extensions directory '.azure\cliextensions'
Python (Windows) 3.6.8 (tags/v3.6.8:3c6b436a57, Dec 23 2018, 23:31:17) [MSC v.1916 32 bit (Intel)]
Legal docs and information: aka.ms/AzureCliLegal
You have 3 updates available. Consider updating your CLI installation with 'az upgrade'
Please let us know how we are doing: https://aka.ms/azureclihats
and let us know if you're interested in trying out our newest features: https://aka.ms/CLIUXstudy


Add a Lexer for Numba IR

Add a Lexer for Numba Intermediate Representation (IR).

(I am working with @Matt711)
@jeanas Many thanks for reviewing this - this PR was opened prior to us receiving approval to contribute it upstream to Pygments due to a miscommunication. Thank you for providing the suggestions above - we'll hold off updating the PR until we've received approval to make this contribution, then address the feedback above (I hope this won't delay things too long). Apologies for the slight confusion here.

We received permission / approval to make this contribution upstream, so we can now continue to work on this PR - many thanks for your time and understanding so far @jeanas, and apologies for the delay.


PHP 8.x warnings

PHP Deprecated:  uasort(): Returning bool from comparison function is deprecated, return an integer less than, equal to, or greater than zero in /www/example.com/httpdocs/opcache.php
To fix this :
Find :
uasort($status['scripts'], function ($a, $b) { return $a['hits'] < $b ['hits']; });
Replace :
uasort($status['scripts'], function ($a, $b) { return $b['hits'] <=> $a ['hits']; });
I hope this helps someone.

My fork that is fully PHP 8.x compatible :
https://github.com/qcybb/opcache-dashboard-PHP8


Timeout due to certificate error during "ext install"

VSCode Version: 1.2
OS Version: Windows 7

I realize that this has been brought up tangentially in a few other issues (#69 and #155).  However, the issues resolved in both of those are not my current issue.
At my company, market.visualstudio.com is issued a certificate after the https traffic is intercepted.  It seems that VS Code uses the IP <IP_ADDRESS> when trying to browse/install extensions, which causes a cert problem because the cert was issued to market.visualstudio.com, causing a mismatch.  If I browse to <IP_ADDRESS>, IE and firefox complain, but market.visualstudio.com works just fine.  The computer does not use any proxy settings so the "http.proxyStrictSSL": false setting does not help.  Why is the IP address being used in the first place instead of the hostname?
Any suggestions?

We use the address. Your computer resolves it to that IP address.
Can you show us a screenshot of the error?

The error looks like:

Any browser or utility I use (like ping) does not resolve it to the IP (as I understand it).  If I go to market.visualstudio.com in a browser, all is good.  If I use the IP there is a cert error because my company assigned the cert to "market.visualstudio.com" not the ip.  Maybe I am misunderstanding something.

So... if you run ping marketplace.visualstudio.com what IP do you get?

Ah, I have misunderstood some things.  Sorry about that.  First, you question was on point: I am using a proxy, though it is automatically detected by most applications.  There is no system level proxy settings.  This is not a VS Code problem, but an issue with proxy settings.
I will close this in a bit or you can close it, but I was wondering as an aside: will auto detect proxy settings make it into VS Code at some point?

Yes we absolutely want to get there. Can you show me how your wpad proxy is configured in your system?

Well, I can't link my companies wpad file, but it was located at http://wpad/wpad.dat and contained what appears to be a JavaScript function?  I am no expect in WPAD files, but it looks like JS.
The function:

Calls for the DNS to resolve the hostname
Checks that the URL meets on of many criteria (in network, ftp, etc.) returns "DIRECT"
If not matched with a special case then it returns "PROXY :"

Does that help?  That is about all the info I can give on our WPAD, but I would assume it would be similar for other companies.  Let me know if that helps.


Add support to the ManualEventAgent for Liquid formatting and multiple Events

Fix #1228 and #1234
@ohingardail, would you mind testing this?

Yes, I'll test #1228 and #1234.

Thanks!

#1228 and #1239 look good so far (#1234 is out of scope of this testing). Serial or individual events work OK, as do the few Liquid templates I've tested for both serial or individual events.
Note that although it now appears possible to mix serial and individual event specifications in the Manual Event Generator summary page (the intention of which would be difficult to interpret), the individual event is discarded, which could cause some confusion to users if they misinterpret the instruction "Otherwise, everything entered will be emitted as a single Event."
Example :
{
  "payloads": [
    {
      "A": "1",
      "B": "2"
    },
    {
      "C": "3",
      "D": "4"
    }
  ],
  "E": "5",
  "F": "6"
}

The result output is 2 events, {"A":"1","B":"2"} and {"C":"3","D":"4"}. {"E":"5","F":"6"} is not emitted.

I've noticed another unexpected effect, although I don't know if the problem is with the deduplication agent or the manual event agent. If I pipe a manual event generator into a deduplication agent (where property is null), then a duplicate event within one run of the manual event generator is not recognised as such.
Example. Manual event generator is set thus:
{
  "payloads": [
    {
      "A": "1",
      "B": "2"
    },
    {
      "C": "3",
      "D": "3"
    },
    {
      "A": "1",
      "B": "2"
    }
  ]
}

... which is piped into a deduplication agent, which (incorrectly) outputs all three events :
{"A":"1","B":"2"}
{"C":"3","D":"3"}
{"A":"1","B":"2"}

On the second run of exactly the same arrangement, the deduplication agent (correctly) emits none of the events.
After the first run of the deduplication agent, its agent memory pane (on its details page) includes one value (where I think there should be two) and after the second run it has two values.
When I repeated the test after clearing the deduplication agent memory, I got the same results except on the second run the deduplication agent (incorrectly) emitted {"C":"3","D":"3"}
I'm not sure how to interpret this inconsistent behaviour.

Thanks for testing! Do you think we should allow for your case of emitting a single event as well as multiple ones if payloads is provided?  That seems confusing to me.  Maybe I'll just document that you have to pick one or the other?
Interesting observation about the deduplication agent.  I wonder if we have a bug with it's handling of full payloads (we JSON.dump the payload, so I wonder if we could have JSON ordering issues or something, although I wouldn't expect it).  Does it work if you give it a property?  I also won

[1] One approach would be not to give users a choice (and document this on the agent summary page); they must always set an array of events, even if that's only one event (and dispense with the "payloads" keyword), so these would be valid:
{
 [
    {
      "A": "1",
      "B": "2"
    }
 ]
}

(one event emitted)
{
 [
    {
      "A": "1",
      "B": "2"
    },
    {
      "C": "3",
      "D": "4"
    }
 ]
}

(two events emitted)
But this would no longer be valid :`
 {
      "A": "1",
      "B": "2"
 }

nor would this
{
 [
    {
      "A": "1",
      "B": "2"
    },
    {
      "C": "3",
      "D": "4"
    }
 ],
{
      "E": "5",
      "F": "6"
 }
}

... and invalid configurations would not be accepted quietly as they are now.
Failing that (if the "payloads" keyword must be kept, or the older non-array method of event specs must be maintained), documentation on the agent summary (with examples?) explaining what is valid is needed and it would be really useful if invalid configs weren't quietly accepted. This latter requirement may be questionable for other 'production' agents, where some graceful flexibility may be indicated, but I think agents used only in testing should be uncompromising - the user needs to know what they're testing otherwise its not much of a test.
[2] Re the de-duplication issue. If I set the dedupe agent property to {{A}} and pipe this through it :
{
  "payloads": [
    {
      "A": "1",
      "B": "2"
    },
    {
      "C": "3",
      "D": "3"
    },
    {
      "A": "1",
      "B": "2"
    }
  ]
}

... I correctly only get {"A":"1","B":"2"} (once) and {"C":"3","D":"3"} out, although the dedupe agent memory shows :
{
  "properties": [
    "1",
    ""
  ]
}

... the second (empty) entry seems a little odd.
On the second run of the same set of events, I get nothing out of the dedupe agent, and I expected {"C":"3","D":"3"}
I reset the dedupe agent memory and re-ran from the start, with exactly the same results.

Re: the deduplication agent.  Why did you expect to get events out the second time?  Shouldn't it have emitted nothing because of the dups?

Why did you expect to get events out the second time?
Because I was de-duping on property={{A}}, and event {"C":"3","D":"3"} had no property={{A}} . I'm guessing that the unexpected empty string in the de-dupe agent memory is the null value for {{A}} in the {"C":"3","D":"3"} event. In standard SQL, null doesn't match to null because it isn't a real value (although its often implemented internally as one); it means "I don't know" and matching events where you don't know the value of a specific property with all the other events where you don't know the value of the same property doesn't have a useful meaning. It looks like the de-dupe agent is implementing null properties as empty strings, and matching these empty strings. At the moment, if I set property={{A}}, then pushing event {"C":"3"} through the de-dupe agent means that following event {"D":"4"} is de-duped and not emitted (because both share the same null value for {{A}}) which seems illogical.
This comment thread is moving away from the manual event generator; would you like me to open 2 new issues for the 2 deduplication agent issues I've found?

Oh, I understand.  I'm sure what's happening is that the A key doesn't exist on the second event, so it gets treated like nil or "", and so we de-dup on that.  I agree that's a bit silly.  Yes, please open new issues for these.  I think we'll have to just make it pass through without de-duplication events that are missing the property key.  The real question is what to do when the property key exists, but has a nil value.

I've made it display "If you provide the 'payloads' key, please do not provide any other keys at the top level." if you use payloads plus other keys.  Otherwise good to go?

I think the documentation suggested should do the trick.

Thanks @ohingardail!


link clang tools extra against libclang

Checklist

[X] Used a fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[X] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Requested by @isuruf in https://github.com/conda-forge/clangdev-feedstock/pull/64#issuecomment-538787084

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@isuruf, looks like the patch you pointed to doesn't apply cleanly.  I see that 10.0.0 is in the works now.  Is it worth spending time to get this to apply and work for 9.x or should we just wait for 10.x?

If it’s not already in 9, it seems worthwhile. I thought that was what your original PR did though or am I misreading things?

If it’s not already in 9, it seems worthwhile. I thought that was what your original PR did though or am I misreading things?

#64 added clang-tools-extra to clang-tools.  This PR is a followon that Issuru asked for at the end of #64.  I'll go ahead and cleanup the patch.

Ah ok. I may have missed that. Sorry

Hmm.  Even after getting the patch cleaned up, for some reason the package tests fail.  I'm going to build osx locally and see more detail about how the tests are failing.

@timsnyder, don't spend too much time on this. Update to 10.0.0 is more important.

I'm abandoning this in favor of just updating to 10.0.


SpawnEntityEvent does not fire when ExperienceOrbs Spawn

Hello all, I've been fiddling with detecting when ExperienceOrbs spawn and I've discovered that SpawnEntityEvent does not fire when ExperienceOrbs spawn. Some sample code for my listener
    @Listener
    public void onExpSpawn(SpawnEntityEvent event){
        if(event.getEntities().contains(ExperienceOrb.class)){
            getLogger().info(event.getCause().toString());
        }
    }

This does not fire at all, I have tested exp from killing mobs, bottles 'o enchanting, and mining and smelting ore.

[10:03:17] [Server thread/INFO] [STDOUT]: [org.spongepowered.mod.plugin.SpongeWorldTest:onEntitySpawn:668]: SPAWNED ENTITY EntityXPOrb['Experience Orb'/1112, l='ChunkTest1', x=-395.55, y=65.00, z=150.45]
Definitely does work. Why are you checking for the class? I pass the actual Entity object, not the class. Just do the following check
for (Entity entity : event.getEntities()) {
    if (entity.getType() == EntityTypes.EXPERIENCE_ORB) {
        // do something
    }
}


Initial eRASS census building can be completely skipped without showing a warning or an error

I am making Joe's test dataset, and have found that it is possible to have a fresh xga config (i.e. just an xga.cfg) and have it completely skip making a census (or even an 'erosita' directory in the config directory).
I must have messed up the config file, but it should still show a warning I feel

It's because I had the 'root_erosita_dir' config entry set to a directory that didn't exist - I think that if there is a non-default entry in the config, and we know it is a path, it should be checked for existence. Surprised I didn't do that!


[OpenAPI 3.0.0] Parsing of content type fails in some cases for arrow functions

Language: Javascript/Typescript
Package version<EMAIL_ADDRESS>If an arrow function is defined and assigned to a variable before being used in a router, content types have to be escaped or the parsing will fail due to a syntax error.
Example: ApiRoute1.js on this repository
const getUser = (req, res) => {
    // #swagger.tags = ['User']
    // #swagger.description = 'Endpoint to get a specific user.'
    const users = []
    const data = users.find(e => e.id === req.params.id)

    // If you unescape the / for the content type, swagger-autogen throws a syntax error
    /* #swagger.responses[200] = {
      content: {
          "application\/json": {
              schema: { "$ref": "#/definitions/User" },
          }
      },
      description: "User registered successfully." } */
    res.status(200).json({
        data: [],
        message: 'Successfully found'
    })
};

router.route('/users/:id').get(authorize, getUser)

The generation will work here since "application/json" has the / escaped, however, if we were to do this instead:
const getUser = (req, res) => {
    // #swagger.tags = ['User']
    // #swagger.description = 'Endpoint to get a specific user.'
    const users = []
    const data = users.find(e => e.id === req.params.id)

    // If you unescape the / for the content type, swagger-autogen throws a syntax error
    /* #swagger.responses[200] = {
      content: {
          "application/json": {
              schema: { "$ref": "#/definitions/User" },
          }
      },
      description: "User registered successfully." } */
    res.status(200).json({
        data: [],
        message: 'Successfully found'
    })
};

router.route('/users/:id').get(authorize, getUser)

We then get the following error:
[swagger-autogen]: Syntax error:   content: {  "_¬¬¬__application/json"_¬¬¬__: {  schema: { "$ref": "#/components/schemas/User" },  }  },  description: "User registered successfully."
[swagger-autogen]: '#swagger.responses' out of structure in
'./src/controllers/ApiRoute1.js' ... router.get('/users/:id', ...)

Escaping the content type in that case is a valid workaround, however, this seemed like a bug so I thought opening this issue could be helpful.

Hi @BastienGarcia! Thank you for opening the issue.
I'll fix it as soon as possible.

The version 2.21.0 has fixed this. If the problem persists, let me know.


Http client monitor does not show configuration data

The http client is configured, for instance the three timeout settings. This configuration can be done in multiple ways and in ways where one way overwrites the other, for instance first set all three timeouts and secondly set only one of the timeouts (and the others have the default), which overwrites the first way resulting in incorrect settings.
As a user of the http client monitoring I like to see the actual configuration data like timeouts, so I can check that the configuration is as wanted.

Adds HttpClientRequestConfig mbean and SocketConfig per route in PoolingHttpClientConnectionManager mbean


Failed to generate Signed APK.

Error:Execution failed for task ':OpenEdXMobile:transformClassesWithJarMergingForProdRelease'.

com.android.build.api.transform.TransformException: java.util.zip.ZipException: duplicate entry: com/segment/analytics/Analytics$1.class

This issue is caused due Segment library's old version as compared to the gradle plugin and wrapper version.
You should update to latest master branch or rebase any of your branch to latest master to get this issue fixed.
p.s. This issue was fixed in this PR: https://github.com/edx/edx-app-android/pull/938


[Example] Run Next.js and Toolpad together

New example that explains integrating Next.js and Toolpad

@prakhargupta1 Merging this so I can release. Feel free to review still


Generate test coverage metrics using Coveralls/Codecov

I wonder how much of EPANET actually gets tested when the regression test suite runs?
Now that we have nrtest running under CI, a useful next step would be to configure Coveralls/Codecov and generate test coverage metrics.

I propose we re-purpose the Travis worker to run tests and generate coverage. Let me know if this presents a problem for you so we can work something out. :smile:
Background:
I am working on getting a coverage metric computed for our testing system. A branch dev-coverage has been created that builds the epanet library in debug configuration using gcc linked with lcov to generate coverage data when tests are run. I would like to use the Travis worker to get Coveralls up and running.

Why are you asking if it would present a problem? What potential problems could it pose?

@LRossman I am trying to be polite and communicative. Just giving a shout out in case someone is relying on the Travis build for some reason that I'm not aware of.
I don't anticipate any problems except low test coverage numbers! :smile:

OK, go for it. Regarding the engine itself, the amount of code coverage will directly depend on what's in the test problems it gets fed (duh...). We could probably get the same info as the coverage stats provide by filling in a table with network features (head loss formula, valve type, pump curve type, etc.) as columns and test problems as rows.

@LRossman That's true, but this way the computer does the work and everyone can see the results.

Coverage is running at 60% running only unit tests Once regression tests are running coverage is likely to go up. So we are in pretty good shape!


[Extensibility] Expose variables as protected in page 414 G/L Balance

In page 414 G/L Balance, the variable DebitCreditTotals should become a protected var, in order to be accessible from page extensions. Please consider exposing the other matrix filters too.
It would also be nice to have the variable Emphasize become a protected var as well; although it's not really necessary for our logic, it still helps to maintain uniformity in the UI.
Thanks.

any update on this ?

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

We found the changes in the daily branch but checking BC17.2 they are missing.
When will they be available?


Properly vectorize propagators

At the moment, the "vectorization" step for most propagators happens outside of numba, harming performance. For example:
https://github.com/poliastro/poliastro/blob/e57d8c4330e45aa4d2f7d2bd1131d806dc95a373/src/poliastro/twobody/propagation.py#L146-L151
This is related to #921 and perhaps should be tackled afterwards.
cc @Josvth

Prospective contributors: consider looking at #921 first, then we can work on this.

@andrea-carballo expressed interest in working on this! The steps should be:

Move test_propagate_with_coe to a newly created tests/tests_core/test_propagation.py
Move the functions in core/propagation/__init__.py to core/propagation/vallado.py, core/propagation/mikkola.py... as done with core/propagation/farnocchia.py and adapt tests and imports accordingly
Write farnocchia_coe_tofs, mikkola_coe_tofs, markley_coe_tofs, pimienta_coe_tofs, gooding_coe_tofs, danby_coe_tofs doing this:

@jit
def propagator_tofs(k, ..., tofs, ...):
    nus = np.zeros_like(tofs)
    for tof in tofs:  # pseudocode
        # call corresponding propagator with single tof (time of flight) and store nu
    return nus

This doesn't fully fix the issue, but it's a first step.

Postponing this to the next release.

Blocking this on #921.

I think #921 will need a significant refactor, so I'm postponing this issue. At least will try to tackle #921 for the next release.

Now propagators have a propagate_many method that still is iterating on the Python side rather than on the numba side:
https://github.com/poliastro/poliastro/blob/3abced1b27ee7a9ac9ae51b2bdf4723b1c3239e4/src/poliastro/twobody/propagation/farnocchia.py#L59-L65


Update Dashbaord to latest model implementation

Background
As outlined in #2535, the Dashboard still relies on the v1beta model specification.
Action
Update the dashboard type system to the latest version of the model specification

What is the plan for the existing data in the dashboard database that is currently in the legacy specifications?

There is undoubtedly value in keeping it. We could export it to CSV and persist it somewhere (estuary, s3, r2, w/e) for historical reference. But the effort to keep the dashboard working as it does today (its queries and ingestion) with the legacy data and new formats we are moving towards will burden development as the spec continues to evolve.
How does this sound:

Export all existing data to CSV, and pin it on a storage backend.
Hibernate the dashboard, and possibly truncate it's database (would save $$$)
If we want to query the historical data use bacalhau + duckdb.
Later, awaken the dashboard, and at that point decide if we want to import historical data.

To mediate the loss of metrics we'll face from dashboard hibernation we can move almost all of its (metric related) functionality to Grafana. This will require implementing some new metrics in the Requester node, but that's something we intended to do anyways - so a win-win in my book.


discovered resources: link to discovery resource

include id and link to discovery resource that reported it
Follow the managed resource link format https://docs.inmanta.com/inmanta-service-orchestrator/latest/reference/apilinks.html#api-self-referencing-links

See status on Slack


Reward for killing squids

Description of issue:
Hello, I am using the latest version of the plugin on 1.15.2 server.
I tried to define job that will reward players for killing squids (and tested this issue affects only squids by adding a reward for killing a pig too).
When player on the job kills pig, they receive experience and money. They receive nothing when killing squids. I am attaching relevant part of the job config below. It reports no issues, it just does not work.

CONFIG SECTION:
    Kill:
      SQUID:
        income: 2.0
        points: 2
        experience: 2.0
      PIG:
        income: 2.0
        points: 2
        experience: 2.0

Server Type (Spigot/Paperspigot/etc): Spigot
Server Version (using /ver): CraftBukkit version git-Spigot-ae72bf4-183139d (MC: 1.15.2) (Implementing API version 1.15.2-R0.1-SNAPSHOT)

As of testing, it works as it should. It could be due to squid being spawned from spawner, or player being in a boat while having disabled payments while riding or having allow-pay-for-durability-loss set to false. Other then this, default configuration does work as it should at this point.


Add connected peers to protocol, use in sync provider

Fix #1853

The idea behind this change is that we can use context_data.peers as a thread-local piece of data, that is used extensively by the protocol thread. Then we have connected_peers that is shared with other threads via SyncProvider, and that is updated only when necessary by the protocol thread. This should reduce blocking on awaiting the lock on connected_peers to a minimum.

Comments addressed, and rebased. Sorry, it's out of habit!

Ok, had to reshuffle the rpc tests a bit to be able to use PeerId::random()...

Ok, had to reshuffle the rpc tests a bit to be able to use PeerId::random()...

@tomaka @twittner Your comments have been addressed, and I had to make a few additional changes to the rpc test code.

Nice one @gterzian :+1:


Adding ability to read multiple requests from a single test case

Purpose
To allow for multiple request inputs from a single test case file enabling complex fuzzing scenarios. Currently libdesock reads the file up to the size of the input buffer. Instead, I propose these changes so that this library will read until a delimiter is found indicating the end of one reqeust and resume reading after the delimiter on the next call to read/recv.
New Features

Ability to read/recv from a single testcase multiple times and return different data each time.
Ability to dup2 stdin when a compile option is provided. Note: This allowed for using libdesock while running under gdb

Code Changes

README.md, meson_options.txt, meson.build

Updated to include two new compile time options allow_dup_stdin and multiple_requests

examples/nginx/README.md

Modified to include code modification instructions to fuzz nginx with multiple_requests enabled.

libdesock/src/dup.c

Added compile time flag to allow dup2 on stdin

libdesock/src/hooks.c

Added compile time flag to enable the code which will search for and read up to the delimiter in stdin. It maintains the correct file position between calls.

run-tests.sh, tests/helper.py, tests/test_multi_read.py

Added tests for this new feature.

Testing
I added tests for this multiple_requests feature. I did not want them to affect the coverage data generated by the original tests, so I added an optional flag to run-tests.sh. Invoke these tests by running
./run-tests.sh -multi

Thank you for this awesome PR! This definitely is a useful feature to have. Merging...


Update Readme

Men should be included too in this beautiful (terrible) erotic art.

On one hand, this is pointless political grandstanding
On the other hand, it arouses me when people are told what to do


Failure to import solution with Power Platform Build Tools from DevOps Pipeline using SPN connection

Hi all,
I'm using a SPN connection to import a solution, but it failed with the message "GraphUserDetailNotFound Message: The user 'xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx' was not found in Graph".
I have tried other Tasks in the build pipeline which perform OK with the same SPN connection configured.
Looking at the stack trace it looks like it happens when importing a Canvas Power App:
at Microsoft.Crm.ObjectModel.PowerAppsValidation.PowerAppsServiceResult[T](ResourceResult`1 powerAppsServiceResult)
at Microsoft.Crm.Tools.Core.ImportExportPublish.ImportCanvasAppHandlerInternal.d__25.MoveNext()
Multiple reports have been made on this issue, and it has been identified as a bug.
Created a GitHub issue for visibility and follow-up purposes.

Fixed in 1.0.4


Create @intersystems-community/intersystems-servermanager NPM package

This PR adds an NPM package that other extensions can use when leveraging the API this extension publishes.
It also updates Server Manager to use this package, as a way of validating the package's correctness.

@isc-bsaviano this PR is a draft because it contains the part which defines the new package. Please take a look at the commit and raise any queries with me. Once you are happy with it I will build and publish the package, then add a second commit that makes Server Manager itself use the package. At that point I will set the PR as ready for review.

Looks good to me

@isc-bsaviano this is ready for you, but there's no rush because I have already published the NPM package and started using it in another project.


Being able to provide our own widget for the logo section

Instead of passing a string to display the image, provide a Widget.  So this is opening the doors of using an SVG image as an example.
Just make sure to wrap this widget in a box with constraints.

 You can also place other things (not just images) with the help of this change. Thanks for your beneficial ideas!


Allow service-loader API to locate SAML2 metadata generators

This should allow pac4j to provide alternative implementations for metadata management based on JPA, Mongo, etc.

Three questions:

can you drop a line in the release notes?
I guess you don't want to make that change in the "old" pac4j-saml module which is now deprecated and will be removed next year, right?
we also have a ConfigurationManager in the SAML support: would it make sense to incorporate your change with it?

I updated the docs and the release notes.
Could do it, sure but to me that module is dead and in pure maintenance mode given OpenSAML v4 is more or less EOL. News features should probably stick to the v5 module. If you'd like, I can certainly backport.
If I understand correctly, the ConfigurationManager is entirely separate from here. One could use this SPI trick for ConfigurationManager or  SAML2MetadataGenerator, or both. I updated the docs to note this. I think what you're saying is, we could consider modifying the SAML2MetadataGenerator, so that it would allow one to also configure and define a SAML2MetadataGenerator? We could go that route, but I feel like it might be best to keep them separate. The change here mainly will be used in CAS to provide alternative stores for metadata management of the SP, and if I mix this with the configuration manager, I would also have to account for that implementation as well. Separation is a good thing here.

By the way, do you think pac4j should keep alternative impls of SAML2MetadataGenerator? based on JPA, Mongo, etc as a separate module? I have seen the request a few times on the list. My plan is to ultimately build support for this in CAS, and I would prefer if these alternative impls were offered by Pac4j and only configured in CAS, rather than CAS directly owning them. What do you think?

Thanks for the docs and release notes
No need to backport, let's focus our efforts on none-deprecated modules.
Yes, that was my question: can we merge the ConfigurationManager with what you propose in this PR? But this really was a naive question and I understand you prefer to keep them separated.
If new implementations (based on JPA, Mongo, etc) are created, it makes sense to host them in pac4j rather than in CAS as it could be used by other pac4j users.

Wonderful. I'll follow up on the dev list. Thanks again!


Convert DDF Rules to USDM v3.0

Do the following for each existing DDF rule:

Update the values of the following parameters in the Rule Editor to reflect changes to the rules editor schema:

Standards.Name (from "DDF" to "USDM")
Standards.Version (from "2" to "3.0")
Standards.References.Origin (from "DDF Conformance Rules" to "USDM Conformance Rules"

Note that we agreed not to change the Rule Identifier

Add extra details for each rule definition, including things such as:

Full citations for each rule in the rules editor (e.g., include text/section number for the relevance citation(s) in the user guide).
Standards.References.Criteria

Copy test data across to the USDM v3.0 test data template and rerun rule.

Record details of each rule in the new Harmonized Rule Catalog Spreadsheet

Actions:
Done: Harmonized Rule Catalog Spreadsheet updated and aligned with team.
To do:

Align process with team - see picture below - Q: do we need separate define and create step in CORE rule editor?
Updated process from phase 3a in rule requirement document


fix typos and spaces in overview-mode help

Currently the help box looks like this:

This PR is aimed to fix some missing and superfluous spaces

Thank you @sappelhoff, that's a nice improvement of the text.