CAMEL-15108: Extract the implementation of the JtaTransactionPolicy from

camel-cdi to camel-jta
https://issues.apache.org/jira/browse/CAMEL-15108
[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] If you're unsure, you can format the pull request title like [CAMEL-XXX] Fixes bug in camel-file component, where you replace CAMEL-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[ ] Run mvn clean install -Psourcecheck in your module with source check enabled to make sure basic checks pass and there are no checkstyle violations. A more thorough check will be performed on your pull request automatically.
Below are the contribution guidelines:
https://github.com/apache/camel/blob/master/CONTRIBUTING.md

@oscerd yeah - I run a full build but the camel-jta is not a "real" component and just have the JtaTransactionPolicy which could be used by camel-cdi and camel-quarkus to support the transacted(). So I think it could not  be includes in endponit DSL or compoent DSL.

@davsclaus do we need to have a look at camel-karaf and camel-spring-boot also for this?

No as it was in camel-cdi and they are not on karaf/SB.
For Karaf then we have historically used TX via camel-spring.


Not able to download the best_model.pth

I tried logging in to https://pan.baidu.com/s/111neutXya4cwYpqUUYQ7ew
but haven't been able to download it after multiple trials.
Could you please attach it here?
我尝试登录https://pan.baidu.com/s/111neutXya4cwYpqUUYQ7ew，但经过多次试用后仍无法下载。
你能把它附在这里吗？

Hi, I found a way to download it. Here is the attachment.
model_best.zip


TypeError: Error in invocation of storage.get(optional [string|array|object] keys, function callback): No matching signature.

Hi,
I'm trying basic usage of Plasmo Storage API and getting this error.  Data is stored in popup session, but when I open pop-up again, storage seems to be reset.  Any ideas?
I've tried both the React Hooks API and the standard base Storage API.
  const [checked, setChecked] = useStorage('checked', true);
Error is triggered by running the above.  I must be missing something very simple.  Any ideas? Running Plasmo v.0.62.2 and  @plasmohq/storage 1.5.0
hook.cjs:1 Uncaught (in promise) TypeError: Error in invocation of storage.get(optional [string|array|object] keys, function callback): No matching signature.     at d.rawGet (hook.cjs:1:2608)     at d.get (hook.cjs:1:4002)     at hook.cjs:1:4500     at commitHookEffectListMount (react-dom.development.js:23150:26)     at commitPassiveMountOnFiber (react-dom.development.js:24926:13)     at commitPassiveMountEffects_complete (react-dom.development.js:24891:9)     at commitPassiveMountEffects_begin (react-dom.development.js:24878:7)     at commitPassiveMountEffects (react-dom.development.js:24866:3)     at flushPassiveEffectsImpl (react-dom.development.js:27039:3)     at flushPassiveEffects (react-dom.development.js:26984:14)

Data is stored in popup session, but when I open pop-up again, storage seems to be reset. Any ideas?

Did you try query those data in chrome.storage? the localStorage view is not the source of truth.
I'd recommend upgrading the framework as well, also which browser are you using?

Yes, using chrome.storage.local works.
Browser is Chrome 113.0.5672.92 (Official Build) (arm64)
Will try upgrading framework later.

Ok upgraded framework to v0.70.1, same issue.
If I downgrade to v1.0.0, the error disappears.

If you do pnpm create plasmo --with-storage and test that example project out, does it work or still error?

manifest_version: 2
version: "@plasmohq/storage": "^1.6.0"
error: Error in invocation of storage.get(optional [string|array|object] keys, function callback): No matching signature.

@louisgv yes, that works, but unfortunately I didn't start plasmo project with pnpm

i got the same problem when i use mv2

I am running into the same issue with --target=chrome-mv2.
In my background page this does not work and triggers the error:
console.log( chrome.storage.local.get(null) )

While this works (callback argument required):
chrome.storage.local.get(null, result => console.log(result))


Add LiveScript in Available  languages section in readme file.

I would like to work on this issue. Can u please assign me this issue.


package(s) :)

great work!
(120 package(s))

Some alternatives:

packages: 120
📦120
credit.packages.length > 1 ? 'packages' :'package'

:+1:

https://github.com/sindresorhus/plur :smile:


fatal error: sweep increased allocation count since 1.2

Hey,
I use inflixdb on cubietruck (armhf) and since 1.2 I get very often this error together with an auto-restart:
fatal error: sweep increased allocation count
Attached two trace dumps, hope it helps.
Is it possible to downgrade to 1.1.2 (whch was the last version before, or ?!) or will this not work?
fatal-influxd2.txt
fatal-influxd.txt

This might be a Go runtime issue.  Some relevant ones logged there:

https://github.com/golang/go/issues/19029
https://github.com/golang/go/issues/16778

You should be able to downgrade to 1.1.2.

@Apollon77 Did you compile the binary yourself or use one of our released versions.  If you compiled it, which version of Go did you use?

@jwilder FWIW, I can tell from the line numbers in the stack traces that it was compiled with at least Go 1.7 and at most Go 1.7.4, not Go 1.7.5.

I used The only provided as package by your PPA (installation as described on your website).
I moved some data into a second instance and now it is more stable, but experienced this error first in 1.2

@Apollon77 @rsc This would be with 1.7.4 then.  Our released binaries are compiled with 1.7.4.

not able to reproduce with current builds ...


Updated ELB remove-tag documentation

ELB remove-tag documentation didn’t match the schema c7n requires. c7n looks for a list of tag keys but the documentation doesn’t show that and also uses the wrong property name

Thanks!


[Bug] FastForward MV3 on Firefox broken after AMO relisting

Body
FastForward does not work at all after updating after FastForward being relisted on AMO
Version
Manifest-v3
What browser(s) are you seeing the problem on?
Firefox browsers
What platform are you seeing the problem on?
Computer (Windows, Linux, MacOS)
Anything else?
I am not joining your Discord. Stop contributing to information voiding.

Can confirm. The new version does nothing.

We are currently investigating this (and another issue with the version not showing up)

We are aware of the issue already, and trying to get a fix for this.
Could you explain "information voiding", please?

http://ascii.textfiles.com/archives/5509

..what? we have documentation on github wiki, and we are working on the fix, but we didnt comment about anything with the public, until this issue, because we thought this was going to be a simple fix, but the way mozilla wants the consent dialog to be implemented make it a hard thing to fix

update:
this is happening because Mozilla asked us to add a consent dialogue in the new version because of new rules they have about processing urls. they said we have to have the user press agree BEFORE running any code. unfortunately, the code that manages the consent page isn't working properly, so the extensions main code (the background script) isn't being run, so nothing is working.
#1134 may fix this, but needs testing. once that is done, we will upload the new fixed version to Firefox and hopefully this issue will be sorted

update:
We've identified possible problematic commits.

another update:
we've identified the problematic code and are testing a fix

So, unless a new release is published to AMO, Firefox users still can't use this addon?

exactly or you can download the latest package zip file in actions (and if using firefox stable or beta, change the contact email in the manifest and upload it to firefox addons as self distributed so they sign it real quick so you can install it)

As of recent update, it's working now.
(Although some Example Links on the homepage can't be bypassed, like the clictune.com one. But I think that belongs to another issue


possible deSolve issue

Ran into an interesting issue during testing - I don't think this should be Bayes specific, but this is the code I ran and the error message I got:
sp <- function(split_dates) { replace(
  specs(mm_name('bayes', err_proc_iid=FALSE),
    n_cores=3, n_chains=3, burnin_steps=300, saved_steps=100, verbose=FALSE),
  'split_dates', split_dates
) }
dat <- data_metab('1', res='30')
mm <- metab(sp(split_dates=FALSE), dat)
mm <- metab(sp(split_dates=TRUE), dat)
dat <- data_metab('3', res='30')
mm <- metab(sp(split_dates=FALSE), dat)
mm <- metab(sp(split_dates=TRUE), dat)
## Error in .Call("call_rkFixed", as.double(y), as.double(times), Func, Initfunc,  :
##   "call_rkFixed" not resolved from current namespace (deSolve)
## Error in .C("unlock_solver") :
##   "unlock_solver" not resolved from current namespace (deSolve)

Sometimes it breaks on the third metab() call, sometimes on the fourth. It occurs whether I've compiled the stan code during this R session or a previous one.
From then on, I get the same error message anytime I try to run any Bayesian model, though an MLE works fine:
mm <- metab(sp(split_dates=FALSE), dat)
## Error in .Call("call_rkFixed", as.double(y), as.double(times), Func, Initfunc,  :
##   "call_rkFixed" not resolved from current namespace (deSolve)
## Error in .C("unlock_solver") :
##   "unlock_solver" not resolved from current namespace (deSolve)

mm <- metab(specs('mle'), dat)
mm
## metab_model of type metab_mle
## [etc.]

mm <- metab(sp(split_dates=FALSE), dat)
## R session aborted

See
http://r.789695.n4.nabble.com/deSolve-unresolved-namespace-error-td4679888.html: The solution may be to call library(deSolve) before loading/running streamMetabolizer. But I don't want to add a CRAN-breaking code line (library()) until I'm sure it's necessary, so I'm waiting and just adding this issue for now.

Haven't seen the issue in a while. Might not even need the requireNamespace call now that deSolve is an official Depends package, but leaving as-is because it isn't currently broken

I just go this excat same error running a student's code on my computer.  Worked for him, not me.
Error in .Call("call_rkFixed", as.double(y), as.double(times), Func, Initfunc,  :
"call_rkFixed" not resolved from current namespace (deSolve)
Error in .C("unlock_solver") :
"unlock_solver" not resolved from current namespace (deSolve)

Interesting. Does it help to call library(desolve); library(streamMetabolizer) as your very first calls in a fresh R session (with no automatically loaded data)?

I did not try calling library(desolve). A fresh R session solved the problem on its own.  If this problem comes back, I will add library(desolve) to the code.


Clarify which limits can be increased

Clarify which limits can be increased.

emailed author

Teams message to author

Emailed author 10/20.

I sent an email to the author today.
@MicrosoftDocs/public-repo-pr-review-team

I sent email to @mrbullwinkle about this PR.

email and Teams msg to author

This content has been moved and updated since the pull request was created, and the proposed update might no longer apply.
If you want to make further changes, please open a new PR. Thanks for your contribution!


Version 1.0.24

💡 Features

New 'loading' prop for Button Component

Could you update the PR description to match previous release notes like https://github.com/lyytioy/lyyti-design-system/releases/tag/v1.0.21?

Could you update the PR description to match previous release notes like https://github.com/lyytioy/lyyti-design-system/releases/tag/v1.0.21?

sure :)


Field in drawer cannot be edited

Link to reproduction
Not for now
To Reproduce
This is just a reminder for a closed issue.: https://github.com/payloadcms/payload/issues/2815. More than one of us are reporting this but we got no replies. I guess the reason is the ticket is closed.
Describe the Bug
https://github.com/payloadcms/payload/issues/2815
Payload Version
1.10.2

Hey @Stupidism this should be fixed with 1.11.0, can you give this a shot and let me know how it goes?


update releases scripts to point to wagtail.org

https://github.com/wagtail/wagtail/issues/7841

Note to self. Rebase after Black

Closing this - not a priority and can be addressed if needed in the future.


JAV-241 use fabirc8 maven plugin gen k8s deploy script

Coverage remained the same at 66.443% when pulling 1c8ae14618226ccd84926f7cbf8810cb65955665 on zhengyangyong:JAV-241_k8s_deployment_script_generation into a5c69133356b3d5fecbd5aaa4e2a3c1d2781fd12 on WillemJiang:master.


windows

=========================
� Check config file...  ✅ Json config found
(node:9292) UnhandledPromiseRejectionWarning: Error: Browser is not downloaded. Run "npm install" or "yarn install"
at ChromeLauncher.launch (C:\Users\nour\Documents\vw\1337nour\node_modules\puppeteer-core\lib\Launcher.js:236:15)
at async C:\Users\nour\Documents\vw\1337nour\app.js:43:15
(node:9292) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 1)
(node:9292) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

First: Thank you for this release its very cool
Tutorial for windows
**-You need to download chromium (extract in your computer)
-Open cmd and navigate to folder bot like a this: cd C:\Users\you-username\Downloads\bot
-type npm install
-type npm start
-put your auth-key
-put commanline for exe chromium like a this:
C:\Users\you-username\Downloads\chrome\chrome.exe
Press enter and done.
FIXES:
**If you get error restart the bot (CTRL+C) and "npm start" again in cmd.

@Nou4r
Please tell me your OS, valorant-watcher version.
Did you download the chromium browser?

Btw for the people who dont know how to access the twitch auth token:
Chrome: F12->Application->Cookies and under twitch.tv there should be an entry for auth token
Obviously you need to do this on the twitch website

thank you for the neat script and help.
i´m using windows 10 and downloaded and installed all the programs.
i used the steps provided by alanferreiravalorant  and program seems to work (see pic).
is it right that only the cmd window shows up (or should it also auto start chromium?)
and is docker really needed then?
thx 4 help :)

@Nou4r Are solved this problem?


Header style issue

When I opened write.html in a web browser it was something like this :

I can change this style to make it look like this :

You can assign me this issue.

Assigned @kom-senapati


Architecture overview

This is not an issue actually. I want to understand how we are getting MDC styles into your app. I believe its done in webpack . But I am not able to get it working.
I tried to add tabs. But I could not figure out how MDC components are added.
When I add tabs I get the view like this :

There is style for button but not styles for classes mdc-tab , what am I missing.

I could find it .
We are doing this in component.ts
const MDC_TOOLBAR_STYLES = require('@material/toolbar/mdc-toolbar.scss');

styles: [String(MDC_TOOLBAR_STYLES)],

I could get it working after adding there .

@pvamshi, LGTM 👍


Only mention end-of-the-road on uturn

Let's only mention end of the road when it makes semantic a lot of change, otherwise keep it short.
We might want to revise this decision, e.g. by moving end of the road to the front of the instruction. For now I'm keeping parity with the state we had before in Mapbox Directions.

I think we should keep them separate for now, since me may still choose to include "end of road" information, like "At the end of the road, turn left ...".


Add Recoverable to repl

Never done this before...
Jest uses these.

https://github.com/facebook/jest/blob/f509856258499bdfa4b1bb43016652b9acd86b81/packages/jest-repl/src/cli/repl.js#L36
https://github.com/facebook/jest/blob/f509856258499bdfa4b1bb43016652b9acd86b81/packages/jest-repl/src/cli/repl.js#L79
https://nodejs.org/api/repl.html

Guessing we're seeing errors because of #4291. /cc @zacharygolba

Looks like I forgot to add the definition for repl.Recoverable!


issues while running tt-buda-demos

ERROR    | pybuda.run.impl:startdeviceprocesses:1180 - Process spawn error:
An attempt has been made to start a new process before the
current process has finished its bootstrapping phase.
    This probably means that you are not using fork to start your
    child processes and you have forgotten to use the proper idiom
    in the main module:

        if _name == '__main':
            freeze_support()
            ...

    The "freeze_support()" line can be omitted if the program
    is not going to be frozen to produce an executable.

smoke.py runs fine but when I try to run bert or resnet it blows up on the same issue, thanks

@jes-rey can you provide some additional details to help debug:

Which device you are on
Which version of pybuda
Which version of driver (tt-kmd) and firmware (tt-firmware)
Which command / script you are trying to execute

check your python environment.

No response for follow-up.
@jes-rey please re-open if issue is still persistent.

Thanks @breuera. I've root caused this issue and opened a FAQ item in our Discord channel: https://discord.com/channels/863154240319258674/1231945279864045680
Working on updating the FAQ in the repo as well: https://github.com/tenstorrent/tt-buda-demos/blob/main/FAQ.md
Can you give the following solution a try?
Solution:
If trying to run pybuda application from a Python file (.py), make sure that the main module can be safely imported by a new Python interpreter without causing unintended side effects (such a starting a new process).
One should protect the “entry point” of the program by using if __name__ == '__main__': as follows:
# test_bert.py

import pybuda
from transformers import BertForMaskedLM, BertTokenizer

if __name__ == '__main__':

    # Load Bert tokenizer and model from HuggingFace
    model_ckpt = "bert-base-uncased"
    tokenizer = BertTokenizer.from_pretrained(model_ckpt)
    model = BertForMaskedLM.from_pretrained(model_ckpt)

    compiler_cfg = pybuda.config._get_global_compiler_config()  # load global compiler config object
    compiler_cfg.default_df_override = pybuda._C.DataFormat.Float16_b

    # Load data sample
    sample_text = "The capital of France is [MASK]."

    # Data preprocessing
    input_tokens = tokenizer(
        sample_text,
        max_length=128,
        padding="max_length",
        truncation=True,
        return_tensors="pt",
    )

    # Run inference on Tenstorrent device
    output_q = pybuda.run_inference(
        pybuda.PyTorchModule("pt_bert_masked_lm", model),
        inputs=[input_tokens],
    )
    output = output_q.get()

    # Data postprocessing
    mask_token_index = (input_tokens["input_ids"] == tokenizer.mask_token_id)[0].nonzero(as_tuple=True)[0]
    predicted_token_id = output[0].value()[0, mask_token_index].argmax(axis=-1)
    answer = tokenizer.decode(predicted_token_id)

    # Answer - "paris"
    print(f"Context: {sample_text}")
    print(f"Answer: {answer}")

Thank you @milank94. I added the if __name__ == '__main__': to both scripts and the compiler now runs in my installations.


Expression always evaluated to 0

As far as I know [anything mod 1 = 0]. Please verify and or reject this pull request.

why i always get this email
发自我的 iPhone

在 2015年9月1日，17:03，Krom Stern<EMAIL_ADDRESS>写道：
As far as I know [anything mod 1 = 0]. Please verify and or reject this pull request.
You can view, comment on, or merge this pull request online at:
https://github.com/memononen/recastnavigation/pull/111
Commit Summary
Expression always evaluated to 0
File Changes
M DetourCrowd/Source/DetourObstacleAvoidance.cpp (4)
Patch Links:
https://github.com/memononen/recastnavigation/pull/111.patch
https://github.com/memononen/recastnavigation/pull/111.diff
—
Reply to this email directly or view it on GitHub.

@wanhconyan: Because you have subscribed to receive these emails from memononen/recastnavigation (Notifications: Watching (be notified of all conversations))

I don't think this is the correct fix. The array has 2 vectors, and making this 0 only ever uses the first. It should probably be %2 instead of %1.
@hymerman @grahamboree Do any of you know the appropriate fix? I have to admit that I do not quite understand the adaptive sampling algorithm (it's outlined in this blog post).

If should be %2 or &1.
The code in question used to call sin/cos for every sample, and every other ring of samples was offset by half the angle between the samples to cover the are better. A bit hard to example, by say, we have 6 samples on a ring, angle between samples is 360/6 = 60, then odd rings would be offset by 30 degrees. There was a refactoring do to this code to avoid calling sin/cos for each sample, which improved the code quite a bit, but the off looking loop.
The adaptive sampling works so that we take a set of samples on each iteration, pick best sample, shrink the sampling radius and do it again. The goo thing about that sampling is that we get pretty good guess by using less samples, the bad thing is that values that are being sampled are very discontinuous, so sometimes you get temporal flickering in the solution.

Thanks Mikko, I will get that checked in. I didn't know if I should mention you on these issues or not - which would you prefer?

I tend to read these as they pop into my inbox and comment as I get the change or have time. If you're stuck, or need my input specifically, please add a mention.

Glad to see Issues being processed now! :-)

Seein as you're still active @Kromster80, can you upgrade your PR with the changes?

Done

Thanks! Can you squash your commits so they are ready for the merge?

How can I do that through github gui and is it really necessary?

It's not necessary, but it helps maintain a clean history in the project. Instead of having these two commits (where the first one is actually completely undone by the second one), we want just the last commit in the master branch.
I don't believe the Github UI can do squashes, but I believe it can open a terminal window for you? If it can, you can use a process like this: http://gitready.com/advanced/2009/02/10/squashing-commits-with-rebase.html

Too bad :-(
Sorry, I don't fancy Git (it is over-complicated for my taste) and I hate commandline, so if there's no GUI version of it I won't be able to do it. Sorry.

Another way would be to simply create a new branch and apply the fix there. If you don't want to spend anymore time on it, I understand. I will just open a new PR for it then. Still, thanks for bringing it to our attention!

Sounds like a plan =)
(not to mention we spent x10 more time on it just commenting)
I'll do the PR now

New PR is #132


Remove extraneous top-level await requirement

This may be a bug, I'm not sure. So I'm submitting as a feature request.
Search Terms
await, top-level,  disable rule
Suggestion
"isolatedModules": true, should not be required to remove top-level await error.
Use Cases
I'm scripting a very simple file I/O script using Deno. The script runs without issue. Full code is in the below example.
Even though it works, VS Code complains. I think the complaint is at the TS level and I can't ignore it as a specific error. I can ts-ignore line-by-line, but that's not satisfactory.
The error is 'await' expressions are only allowed at the top level of a file when that file is a module, but this file has no imports or exports. Consider adding an empty 'export {}' to make this file a module.ts(1375)
Examples
const decoder = new TextDecoder("utf-8");
const data = await Deno.readFile("tsconfig.json"); // <!-- VS Code shouldn't have red squiggle here.
console.log(decoder.decode(data));

Checklist
I have no idea whether this feature meets the items in the checklist.
Related Issues

[merged PR] https://github.com/microsoft/TypeScript/pull/35813
https://github.com/microsoft/TypeScript/issues/19573
https://github.com/microsoft/TypeScript/issues/37402
https://github.com/microsoft/TypeScript/pull/37424
[closed] https://github.com/microsoft/TypeScript/issues/36036

@Vandivier TLA await are invalid in script files.  Deno loads every file as an ESM file, but to VSCode your example doesn't look like a module, so it it is treating it like a script.  The current canonical way to for a file to be a module instead of a script when there are no other imports or exports is to include an export {} in the file.  This will resolve the issue.
There is a proposal for a pragma to force modules, but it is Stage 1 (early stages) in TC39.

@kitsonk We now have a way to know whether files are implicitly to be treated as a module though. type: "module" so can we consider this?

Addressed at https://github.com/microsoft/TypeScript/issues/36036#issuecomment-1211134182


GeoJSON polygon click events don't bubble through to CircleMarker Points

When there is a geojson polygon layer above a circlemarker point, the click event on the polygon layer does not bubble through to the circlemarker, although it bubbles through to a regular marker.
Here is a demo in the playground which highlights this issue:
http://playground-leaflet.rhcloud.com/buyo/edit?html,output
I added a layer control to allow you to toggle different types of layers.  If you toggle the regular markers and toggle the polygon layer above them their popups still show when clicked, but if you do the same for the circlemarker layer with the polygon layer above it then the popups don't work.

The circle markers are below the polygon, while the regular ones are above (that's Leaflet default to put Markers above vectors).
So unless the polygon as interactive: false I think is normal DOM behaviour not to forward events to an element that is covered.


DELETED

Please don't do that. Wasting others' time without a single word of explanation is not cool 😢


[feature] Deployment CI/CD guide

Would be good to include a guide on integrating a CD pipeline based on Rancher.
There are so many questions and issues raised surrounding this functionality.

Hi @andrewmclagan, not sure if you have seen our eBook/posts on this:
Part1
Part2
Part3

Oops...


Can we make the URL encoding of online editor easier to use

Why
const a = 5;

https://www.typescriptlang.org/play?#code/MYewdgzgLgBAhjAvDArAbgLACgg

I have a piece of code that I hope can be directly passed to the online editor in the form of URL and suffix. But now I don't know how to code his string. Can this method be exposed

You can use the old src format:
https://www.typescriptlang.org/play?#src=hello

If you want to create a compressed-style format, see the code:
https://github.com/microsoft/TypeScript-Website/blob/9cf9ad852e9caf89e334a0740fc61f092b2932cf/packages/sandbox/src/compilerOptions.ts#L85


Add validation function name to TestSession report

As a user, I want to see in the TestSession report, the string representation of the validation function that was used in TestComponent , so that I can better understand what has been validated and how.
Currently, the TestSession report only shows the results of the validation (pass/fail) for each TestComponent, but not the actual function that performed the validation. This makes it hard to trace the source of the validation logic and to debug any issues.
I propose to add a new property to the TestComponent class, called ValidationFunction, which will store the string representation of the function that was used to validate the component. This property will be populated when the TestComponent is created or updated, and will be included in the TestSession.
This will improve the readability and transparency of the TestSession report, and will allow users to easily identify the validation function for each TestComponent.
Acceptance Criteria:

[ ] The TestComponent class has a new property called ValidationFunction, which stores the string representation of the validation function.
[ ] The ValidationFunction property is populated when the TestComponent is created or updated, based on the validation function that was used it the component.
[ ] The TestSession includes the ValidationFunction property for each TestComponent, along with the validation result.

Consider adopting an approach similar to the NUnit framework which use classes like It, Has, and Does. This could streamline XPathContentValidator, RegexContentValidator, and StringContentValidator into a single ResponseContentValidator class. The specific matching function would then be selected through corresponding methods, such as Does.Match(RegexExpression) or It.StartsWith.


In the past

Related to issue #227
With this PR, only detected event where the end is superior or egual to the actual date will be passed to the extension.
The icon will be displayed only for those events.

I removed the variable now.

Thanks for this, Jeremy!


Fix the bug that UTCTiming is ignored even when autoCorrectDrift is off

This is the fix for this issue.
https://github.com/google/shaka-player/issues/2411
shaka.dash.DashParser.processManifest is using the presentationTimeline.usingPresentationStartTime() to determine if it parses UTCTiming or not.
The usingPresentationStartTime always returns the false when maxSegmentEndTime_ is not null.
When the manifest has SegmentTimeline, it seems always setting a value to maxSegmentEndTime_.
I added the code not to set maxSegmentEndTime_ when autoCorrectDrift is false.

@googlebot I signed it!

Thanks for your contribution!


TCP Adaptor: TLS: Adaptor tries to perform a write operation after a PN_RAW_CONNECTION_CLOSED_WRITE event has been received

E           RuntimeError: Errors during teardown:
E
E           Process 5485 (name=http2-two-router-tls-A) error: returned error code -6
E           skrouterd -c http2-two-router-tls-A.conf -I /home/runner/work/skupper-router/skupper-router/skupper-router/python
E           /__w/skupper-router/skupper-router/skupper-router/build/tests/system_test.dir/tests/system_tests_http2_tls/Http2TestTlsOverTcpTwoRouter/setUpClass/http2-two-router-tls-A-80.cmd
E           >>>>
E           skrouterd: /home/runner/work/skupper-router/skupper-router/skupper-router/src/adaptors/tcp/tcp_adaptor.c:973: encrypt_outgoing_tls: Assertion `!IS_ATOMIC_FLAG_SET(&conn->raw_closed_write)' failed.
E           Fatal Python error: Aborted
E
E           Thread 0x00007feec24f33c0 (most recent call first):
E             <no Python frame>
E           <<<<

https://github.com/skupperproject/skupper-router/actions/runs/3396484612/jobs/5647602053#step:28:4107

This issue has been fixed with https://github.com/skupperproject/skupper-router/commit/853cd911e0c667f5c787f8c2b8e9c8467d9b5a2b
Will reopen if we see this again.


[Docs Improvement] - Bring consistency to docs examples

The changes to be implemented in the examples  here is gotten from the docs review made by @it09  https://www.notion.so/pl-strflt/Docs-Review-a322426a26f34ad7862f1cc1b73b840e

[x] Add introduction for all examples
[x] Add prerequisite - how to install the Bacalhau client
[x] replace bacalhau → Bacalhau
[x] Review formatting for examples
[x] Review examples that have large output - delete outputs from not-Bacalhau commands
[x] Take out output that can be confusing for readers
[x] Add TD;LR section
[x] For examples with Docker - add a uniform header "Containerize Script using Docker"
[x] Add build and push docker container sub section
[x] Add a H3 section for link on how to build and push docker container
[x] Add link to docker docs on how to do above
[x] Add "checking the state of your job" section in all the examples
[x] Add "viewing outputs" on all examples
[x] Add section "Running a Bacalhau Job"
[x] Add Subsection Structure of commands

The examples is a total of: 36 pages
No of pages reviewed by me in the past week: 18 pages
No of pages left: 18 pages
Blockers: Made some additional adjustments while working on the pages

https://github.com/filecoin-project/bacalhau/issues/1314

Yeah


New Router - Clear up the intended behavior and significance in the order routes are defined.

Please see the following issue as a reference.
https://github.com/angular/angular/issues/10474

An explanation of the route order is already stayed in the router guide.
The order of the routes in the configuration matters and this is by design. The router uses a first-match wins strategy when matching routes, so more specific routes should be placed above less specific routes. In our configuration above, the routes with a static path are listed first, followed by an empty path route, that matches as the default route. The wildcard route is listed last as it's the most generic route and should be matched only if no other routes are matched first.

Closing for reasons stated by Brandon


Move "Show statistics" to "Design area" and rename to "Leaderboard"

Feature is not fully implemented as leaderboard does not show up in preview.

Compare to:


require('ws').Server causes undefined error on client

I'm using this library with React Native via aws-iot-device-sdk. React Native uses browser WebSockets, therefore it is going for the ws-fallback.js instead of the ws package.
https://github.com/maxogden/websocket-stream/blob/master/server.js#L3 is failing since the browser WebSocket object doesn't have a Server property.

I do not think 'ws' can run on react-native. You should check with them.
cc @lpinca.

React Native uses browser WebSockets

Yes, in this case you should use the native WebSocket object which obviously doesn't work as a server.

Thanks for the feedback. I was thinking more between the lines of conditionally include the server code inside the index.js. Something like:
module.exports = require('./stream.js')

if (WebSockets) {
  var Server = require('./server.js')
  module.exports.Server = Server
  module.exports.createServer = Server
}

because of this line this should be done by your bundler. We may need some special directives for react-native., something like: https://github.com/nodejs/readable-stream/blob/master/package.json#L44.
Would you mind firing up a PR that address this issue?

Sure, I'll look into it and send a PR. Thanks for the info.

FYI: opened an issue on React Native:
https://github.com/facebook/react-native/issues/13474


Exclude subpaths when parent path is clicked

Hi @phpbg and thanks for sharing this app as oss. I open this PR for to enable or disable all subpath under a parentPath when clicked in the advanced settings page.

It also would be better to hide those subpath when parent is disabled. Are you OK with this UX change?

Hi,
thanks for your PR.
It looks good, but we have to discuss if this is necessary.
Say you have following folders:

DCIM
DCIM/foo

Excluding DCIM will only exclude files that are direct children of DCIM. DCIM/foo will be synced unless you explicitely disable it too.
Your PR will disable automatically DCIM/foo if you disable DCIM. This may be interesting, but maybe it is not what the user expects.
At least we must not hide subfolders and let user decide.
This behaviour can be seen here: https://github.com/phpbg/easysync/blob/e0109c07102bec7fe53c23a13a7bf863c58ff7a0/app/src/main/java/com/phpbg/easysync/mediastore/MediaStoreService.kt#L71
We do not filter if file path starts with an excluded folder, we filter if file path match exactly with an excluded folder.
What do you think?

Hi @phpbg ,
that is exactly the desired behavior and I think this a better UX than previous one.
In fact, I have  Music Folder in which my audios are classified like this: Music/artist/album/audios.mp3 with about 100 artists and many albums per artist. Previously it was almost impossible to disable syncing Music (more than hundred clicks needed!) now you can easily select which folders can be synced.
Nextcloud client app propose a similar approach that seems appropriate, they also have a sort of hierarchy in the ui to show the file structure and allowing folders handling.
Also hiding sub folders would be handy, when you have a lot of subfolders like me it is a lot clearer. Also here maybe the hierarchy ui could be a win instead of just hiding.

It looks good, I do a few more tests then I'll merge it and make a release.


Clojure test failure: Could not transfer artifact com.fasterxml.jackson.core

Encountered at http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-16597/4/pipeline/ :
Could not transfer artifact com.fasterxml.jackson.core:jackson-core:jar:2.9.0 from/to central (https://repo1.maven.org/maven2/): GET request of: com/fasterxml/jackson/core/jackson-core/2.9.0/jackson-core-2.9.0.jar from central failed
Could not find artifact com.fasterxml.jackson.core:jackson-core:jar:2.9.0 in clojars (https://repo.clojars.org/)
This could be due to a typo in :dependencies, file system permissions, or network issues.
If you are behind a proxy, try setting the 'http_proxy' environment variable.
@gigasquid Is this due to the flakiness of package serving infrastructure?

This is a flaky package serving infrastructure problem getting a well known library from maven central https://mvnrepository.com/artifact/com.fasterxml.jackson.core/jackson-core

This type of error has also happened with other Scala java dependencies that are pulled. I'm not sure what can be changed to alleviate it other than maybe separating out any dependency fetching call and putting retries around it, but the solution should be considered holistically through all the MXNet fetches

@lanking520 Do you have any ideas to make the jar dependency fetching more robust for Scala/ Clojure from maven? Maybe we should make a different issue to track these more globally and close this one?

@gigasquid I guess that will be more of a CI problem? Most of the cases is like the network reset @marcoabreu have you faced the network problem before? Is there any good solution to solve it?

Are there multiple maven repositories with different SLAs? I'm thinking that we might be hitting an endpoint which is not highly available. Or is there a possibility to define mirrors as fallback?

Maven central is now part of the apache infrastructure I think. I don't believe it has an SLA - there are times that it has trouble https://status.maven.org/
There do appear to be mirrors https://stackoverflow.com/questions/5233610/what-are-the-official-mirrors-of-the-maven-central-repository

Mirrors can be set in leiningen (the clojure side) and also in the maven pom.xmls for Scala https://maven.apache.org/guides/mini/guide-mirror-settings.html.
It would be a good chunk of work to research and implement the mirroring. I vote that we see if it happens more often with Clojure/Java/Scala jars before taking action on it.

As far as I understand this wouldn't be in the application pom.xml but rather in the .m2/settings.xml of the Jenkins slave - thus something that we can do on the infrastructure side by adding it to the docker container. It should be a matter of adding the settings.xml in the install scala.sh file.

@marcoabreu If that's true, then it will get "work" for Clojure too, since it's built on the Scala image and it uses .m2/settings.xml

@lanking520 assign @larroy
Pedro please add this to the backlog for CI enhancements to download dependencies in advance and cache

Similar one here for slingshot (clojure jar) for unrelated PR - https://github.com/apache/incubator-mxnet/pull/16894
Could not transfer artifact slingshot:slingshot:pom:0.12.2 from/to clojars (https://repo.clojars.org/): Failed to transfer file: https://repo.clojars.org/slingshot/slingshot/0.12.2/slingshot-0.12.2.pom. Return code is: 503 , ReasonPhrase:Backend unavailable, connection timeout.

This could be due to a typo in :dependencies, file system permissions, or network issues.

If you are behind a proxy, try setting the 'http_proxy' environment variable.

2019-11-25 01:56:51,849 - root - INFO - Waiting for status of container 824684cbe67d for 600 s.

2019-11-25 01:56:52,064 - root - INFO - Container exit status: {'Error': None, 'StatusCode': 1}

2019-11-25 01:56:52,065 - root - ERROR - Container exited with an error 😞

ok

Connection timeout. Unrelated PR : #18206
http://jenkins.mxnet-ci.amazon-ml.com/blue/organizations/jenkins/mxnet-validation%2Funix-cpu/detail/PR-18206/6/pipeline
Could not transfer artifact org.clojure:tools.analyzer:jar:0.6.9 from/to central (https://repo1.maven.org/maven2/): Connect to repo1.maven.org:443 [repo1.maven.org/<IP_ADDRESS>] failed: Connection timed out (Connection timed out)

[2020-05-05T19:08:36.663Z] Could not find artifact org.clojure:tools.analyzer:jar:0.6.9 in clojars (https://repo.clojars.org/)

[2020-05-05T19:08:36.663Z] This could be due to a typo in :dependencies, file system permissions, or network issues.


"Custom Connectors" under Dataverse in the new make.powerapps.com

In the section (https://learn.microsoft.com/en-us/power-platform/guidance/coe/setup-almacceleratorpowerplatform-preview#configure-the-devops-custom-connector) it refers to navigating to Data,

however, in the new make.powerapps.com UI the Custom Connectors are now under Dataverse

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

ID: b760eb27-13a1-a521-2e36-2a791c8b6963
Version Independent ID: b760eb27-13a1-a521-2e36-2a791c8b6963
Content: Set up ALM accelerator for Microsoft Power Platform using Admin App - Power Platform
Content Source: power-platform/guidance/coe/setup-almacceleratorpowerplatform-preview.md
Service: power-platform
Sub-service: guidance
GitHub Login: @jenschristianschroder
Microsoft Alias: jeschro

Thanks for your comment, @davins-msft. This issue has been assigned to the doc author for review.


Refactoring Step 1 and 2 in DePSI to Python

In DePSI, step 1 and two are:

Loading setup parameters
Apply preselection on PSI

With the current sarxarray and stm, this can already be done. We need to first translate this part to Python.
Expected results:

PyDePSI load a basic setup file (yaml or json) for S1 processing
PyDePSI load corregisterd data in a lazy way
PyDePSI output pre-selection into zarr

Since TUD is getting external resources to do the DePSI tranlastion to Python, we will pause this for now and focus on the data libraries (sarxarray and stm)


Shared library build output location ignores CMAKE setting

I was expecting to have .dll at binary root directory when having
set(CMAKE_LIBRARY_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR})
set(CMAKE_RUNTIME_OUTPUT_DIRECTORY ${CMAKE_BINARY_DIR})

options, but looks like slang build ignores this and put output files in binaryRoot/Debug/bin.
It would be good if slang respects the CMAKE settings.

here is a temp work around
set_target_properties(slang
    PROPERTIES
    LIBRARY_OUTPUT_DIRECTORY "${CMAKE_BINARY_DIR}"
    RUNTIME_OUTPUT_DIRECTORY "${CMAKE_BINARY_DIR}"
)


Baking an AI road introduces spurious line segments

Description
Part 1: jaggies bad
The process of adding an AI road segment to the map often adds a big jagged line segment. I reported this a few weeks ago, but I think I failed to describe the problem accurately.
The issue looks like a simple bug of some sort: it literally connects a point near one end of the polyline to a point near the far end, thus introducing a big "jaggie" that has to be excised by hand.
Loss of editing capability
If the angle between the jaggie and the real polyline is acute enough (so that the geometry almost turns into two overlapping lines), the user loses the ability to click on any of the control points in the affected area. When this occurs, a whole lot of extra fiddling is required to fix up the broken geometry.
Jaggies used to not be a thing
This overall behavior is a regression that occurred some time in the past few months. These spurious jaggies used to never happen, whereas now they're fairly common: I'll encounter a number of them in any hour of use of the tool.
Part 2: regression in overlap with prior road
I also want to draw attention to the fact that in the example here, some of the AI geometry sits on top of an existing road segment, and will have to be excised by hand. This has recently been happening much more often than it used to.
In fact, sometimes Rapid will create geometry with the exact same control points as some roads that I added using Rapid months or years ago. Because Rapid is very bad at manipulating roads that share the same control points, any accidental use of an AI road that overlaps with a pre-existing road can be very difficult and time consuming to fix.

AI road segment. Note the overlap at the top with the pre-existing road segment.

After baking the AI segment onto the map. Note the spurious line segment that wasn't previously present.

Version
2.4.3
What browser are you seeing the problem on? What version are you running?
Safari v18.1
The OS you're using
mac
Steps to reproduce
I'm going to leave the offending example in place so that you can verify the problem by visiting the example URL.
The browser URL at the time you encountered the bug
https://rapideditor.org/edit#map=15.00/37.9713/-114.6464&background=Bing&datasets=fbRoads,msBuildings&disable_features=boundaries
The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/605.1.15 (KHTML, like Gecko) Version/18.1 Safari/605.1.15

We're currently investigating some potentially related issues with the conflation service and high error rates. The first thing I want to understand is why the roads and buildings are always conflating properly- I suspect that fixing that issue will also fix the 'spurious segment' issue.
I'm a bit more conversant in this space now that I fixed the roads, so I may be able to make some headway soon.
@bos, can you recall around when you first saw this behavior? Earlier than a few weeks ago, but you first submitted it then? Or did you first encounter it a few weeks ago?

I've a hunch that #1579 is coming into play here, but the problem is made worse when you combine it with the 'data isn't always conflated' bug that seems to be plaguing our conflation service more and more lately.

Oh yeah, it's definitely #1579 and friends - I recognize the artifact where the head and tail of the road are connected by a long straight line.  The server probably gave us pieces of that road drawn in different directions.
Also the "loss of editing control / If the angle between the jaggie and the real polyline is acute enough" part is #1429

I'd been seeing the issue for several months, i.e. significantly earlier than I reported it, but I was silently coping for a while.
The conflation service (thanks for giving me its correct name) is indeed in bad shape. There are lots of roads in this map area where the AI and underlying roads fully overlap.


plasma-applet-volumewin7mixer: 24 -> 25

Semi-automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/plasma-applet-volumewin7mixer/versions.
meta.description for plasma-applet-volumewin7mixer is: "A fork of the default volume plasmoid with a Windows 7 theme (vertical sliders)."
meta.homepage for plasma-applet-volumewin7mixer is: "https://github.com/Zren/plasma-applet-volumewin7mixer"
meta.changelog for plasma-applet-volumewin7mixer is: ""
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 25 with grep in /nix/store/gsmsp2zs05j9j9wg0v3hgwbvy5lhd9ah-plasma-applet-volumewin7mixer-25

directory tree listing: https://gist.github.com/6eb80c9a3d2b99ed8114f7d3728c3f3c

du listing: https://gist.github.com/90edc6c41ef7f431ae5fb70e11808dd6

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

3 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
0 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
plasma-applet-volumewin7mixer

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/gsmsp2zs05j9j9wg0v3hgwbvy5lhd9ah-plasma-applet-volumewin7mixer-25 \
  --option binary-caches 'https://cache.nixos.org/ https://r-ryantm.cachix.org/' \
  --option trusted-public-keys '
  r-ryantm.cachix.org-1:gkUbLkouDAyvBdpBX0JOdIiD2/DP1ldF3Z3Y6Gqcc4c=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gsmsp2zs05j9j9wg0v3hgwbvy5lhd9ah-plasma-applet-volumewin7mixer-25
ls -la /nix/store/gsmsp2zs05j9j9wg0v3hgwbvy5lhd9ah-plasma-applet-volumewin7mixer-25/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review 1

1 package built:
- plasma-applet-volumewin7mixer

Maintainer pings
cc @mdevlamynck for testing.

Looks good to me. I've tested locally, it builds and runs.


Remove redundant rescaling in feature extraction

Hi guys and thank you for your amazing code,
I noticed that you are scaling the feature vector before normalizing it which is equivalent to simply normalize it.

I guess it's even better to avoid normalizing because original vectors are not normalized
Thus, if you picked some distance threshold for the original ones then you are not able to apply it after this multi-scale inference procedure


Add Signal to services

It's a secure/encrypted an alternative to messenger.
https://signal.org/

Yes please... I need Signal too!

This is not possible as they do not provide a web service.
Reasoning can be found here: https://www.reddit.com/r/signal/comments/brr65x/why_signal_has_no_web_based_interface/

Duplicate of #241.
This is not possible as they do not provide a web service.
Reasoning can be found here: https://www.reddit.com/r/signal/comments/brr65x/why_signal_has_no_web_based_interface/

Isn't it possible to implement it from their electron desktop application?

To be clear: you are asking a developer, at no cost to you, to reimplement an extremely complicated application within another application.
The Franz API doesn't allow for this kind of integration and it would destroy the integrity of the security systems signal puts in place.


Fix groups prefix in CIS data.

@akatsoulas quick r?

Coverage remained the same at 88.716% when pulling 03fa50e6854d7ecf0354664228cadc3f2b391265 on johngian:fix-cis-prefix into 770b72f259a45484f8d3030defd08b81e10bf002 on mozilla:master.


Incorrect formatting: Keywords are highlighted as keywords even when they are part of a larger word

Ref. Bodies to map script, line 3

reportTerraformables.

Note: This does not appear to occur for all keywords. It only occurs for the subset of key words that are defined with Predicate statement rules.

LOL thanks.


enhance the jdbc datasource to support multiple table query

Enhance the data reading capabilities of relational databases：

support single-table scanning, and also support multi-table queries.
For nebula-exchange_spark_2.2, since the jdbc datasource only allows reading of a single table, and does not allow querying from the database, we support to query the single table in SparkSQL and still does not support multiple tables query.

for nebula-exchange_spark_2.2, the jdbc config should be:
      url:"jdbc:mysql://<IP_ADDRESS>:3306/test"
      driver: "com.mysql.cj.jdbc.Driver"
      user: "nebula"
      password: "Nebula@123"
      table: "test.people"
      sentence: "select * from  people" # the table name in sentence must just be table, not db.name

the mysql/oracle/postgresql config shoud just config table, do not support sentence.
for nebula-exchange_spark_2.4 or 3.0, the jdbc config should be:
      url:"jdbc:mysql://<IP_ADDRESS>:3306/test"
      driver: "com.mysql.cj.jdbc.Driver"
      user: "nebula"
      password: "Nebula@123"
      table: "test.people"
      # sentence and table cannot config together.
      # sentence: "select * from  people, player, team"

the same for mysql/oracle/postgresql as jdbc.

Codecov Report
Patch coverage: 15.78% and project coverage change: -0.23% :warning:

Comparison is base (1f40692) 56.01% compared to head (4eaca63) 55.78%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff              @@
##             master     #153      +/-   ##
============================================
- Coverage     56.01%   55.78%   -0.23%
  Complexity       83       83
============================================
  Files            18       18
  Lines          1655     1667      +12
  Branches        315      326      +11
============================================
+ Hits            927      930       +3
- Misses          556      559       +3
- Partials        172      178       +6

Files Changed
Coverage Δ

.../vesoft/exchange/common/config/SourceConfigs.scala
49.71% <12.00%> (-4.19%)
:arrow_down:

...la/com/vesoft/exchange/common/config/Configs.scala
63.78% <23.07%> (+0.66%)
:arrow_up:

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


CTest command line accumulates tests

Brief Issue Summary
When a simple C++ folder with two tests is built and the tests are run, the command line reported in the Output tab of the VSCode accumulates executed tests.
main.cpp:
int main()
{
}

CMakeLists.txt:
cmake_minimum_required(VERSION 3.13 FATAL_ERROR)

project(issue VERSION 0.1)

enable_testing()

add_executable(test_one main.cpp)

add_test(test_one test_one)

add_executable(test_two main.cpp)

add_test(test_two test_two)

The full output in question is provided below. Please note here how the "executing command" lists both tests in the second case.
[build] ninja: no work to do.
...
[proc] Executing command: "...\ctest.exe" -j6 -C Debug -T test --output-on-failure -R test_one
...
[ctest] 1/1 Test #1: test_one .........................   Passed    0.27 sec
...
[proc] Executing command: "...\ctest.exe" -j6 -C Debug -T test --output-on-failure -R test_one -R test_two
...
[ctest] 1/1 Test #2: test_two .........................   Passed    0.65 sec
...

Visual Studio 2022 release_amd64 toolkit is used with Ninja generator but I doubt it has anything to with this.
CMake Tools Diagnostics
{
  "os": "win32",
  "vscodeVersion": "1.77.0-insider",
  "cmtVersion": "1.14.20",
  "configurations": [
    {
      "folder": "d:\\src\\_private_\\issue_report",
      "cmakeVersion": "3.25.2",
      "configured": true,
      "generator": "Ninja",
      "usesPresets": false,
      "compilers": {
        "C": "C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.35.32215/bin/Hostx64/x64/cl.exe",
        "CXX": "C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.35.32215/bin/Hostx64/x64/cl.exe"
      }
    }
  ],
  "cpptoolsIntegration": {
    "isReady": true,
    "hasCodeModel": true,
    "activeBuildType": "Debug",
    "buildTypesSeen": [
      "Debug"
    ],
    "requests": [
      "file:///d%3A/src/_private_/issue_report/main.cpp"
    ],
    "responses": [
      {
        "uri": "file:///d%3A/src/_private_/issue_report/main.cpp",
        "configuration": {
          "includePath": [],
          "defines": [],
          "compilerPath": "c:/program files/microsoft visual studio/2022/community/vc/tools/msvc/14.35.32215/bin/hostx64/x64/cl.exe",
          "compilerArgs": [],
          "compilerFragments": [
            "/DWIN32 /D_WINDOWS /W3 /GR /EHsc /MDd /Zi /Ob0 /Od /RTC1"
          ]
        }
      }
    ],
    "partialMatches": [],
    "targetCount": 2,
    "executablesCount": 2,
    "librariesCount": 0,
    "targets": [
      {
        "name": "test_one",
        "type": "EXECUTABLE"
      },
      {
        "name": "test_two",
        "type": "EXECUTABLE"
      }
    ]
  },
  "settings": [
    {
      "communicationMode": "automatic",
      "useCMakePresets": "auto",
      "configureOnOpen": false
    }
  ]
}

Debug Log
No response
Additional Information
This is the full output log for "build" and "run tests" actions.
[main] Building folder: issue_report
[build] Starting build
[proc] Executing command: "C:\Program Files\Microsoft Visual Studio\2022\Preview\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin\cmake.exe" --build d:/src/_private_/issue_report/build --config Debug --target all --
[build] ninja: no work to do.
[driver] Build completed: 00:00:00.076
[build] Build finished with exit code 0
[proc] Executing command: "C:\Program Files\Microsoft Visual Studio\2022\Preview\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin\ctest.exe" -j6 -C Debug -T test --output-on-failure -R test_one
[ctest] Cannot find file: D:/src/_private_/issue_report/build/DartConfiguration.tcl
[ctest]    Site:
[ctest]    Build name: (empty)
[ctest] Cannot find file: D:/src/_private_/issue_report/build/DartConfiguration.tcl
[ctest] Test project D:/src/_private_/issue_report/build
[ctest]     Start 1: test_one
[ctest] 1/1 Test #1: test_one .........................   Passed    0.02 sec
[ctest]
[ctest] 100% tests passed, 0 tests failed out of 1
[ctest]
[ctest] Total Test time (real) =   0.04 sec
[ctest] CTest finished with return code 0
[proc] Executing command: "C:\Program Files\Microsoft Visual Studio\2022\Preview\Common7\IDE\CommonExtensions\Microsoft\CMake\CMake\bin\ctest.exe" -j6 -C Debug -T test --output-on-failure -R test_one -R test_two
[ctest] Cannot find file: D:/src/_private_/issue_report/build/DartConfiguration.tcl
[ctest]    Site:
[ctest]    Build name: (empty)
[ctest] Test project D:/src/_private_/issue_report/build
[ctest] Cannot find file: D:/src/_private_/issue_report/build/DartConfiguration.tcl
[ctest]     Start 2: test_two
[ctest] 1/1 Test #2: test_two .........................   Passed    0.01 sec
[ctest]
[ctest] 100% tests passed, 0 tests failed out of 1
[ctest]
[ctest] Total Test time (real) =   0.02 sec
[ctest] CTest finished with return code 0

@ImaginaryExponent Thanks for the issue post!
@xisui-MSFT Do you have any context for why this is happening? Of if this is expected? Thanks


README: add hit counter badge (daily/total)

I think I'm going to close this unmerged. If that's too disappointing, I'm open to discussion. But organization members can already see the Traffic under Insights, which provides this sort of thing already.


Document contexts

I'm submitting a

[ ] bug fix
[ ] feature
[x] documentation addition

What changed?
Add come documentation for commands. Mostly for the contexts command.
Also removed the beta warning about it in the CLI output.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 27.11%. Comparing base (b574682) to head (420575a).
Report is 32 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #448      +/-   ##
==========================================
- Coverage   28.59%   27.11%   -1.48%
==========================================
  Files          85       83       -2
  Lines        2301     2242      -59
==========================================
- Hits          658      608      -50
+ Misses       1643     1634       -9

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


Update MSRV

toml_edit updated winnow in 0.19.8 which bumps MSRV to 1.64
https://github.com/toml-rs/toml/commit/4bb106680b0e6acc26ca0cb9644e644383a0b726 & https://github.com/toml-rs/toml/commit/9e43da13b04601d9115747610753d73852fbf4c9
could alternatively lock to  0.19.7 and update MSRV in a new patch.

I integrated this into the following pr: https://github.com/bkchr/proc-macro-crate/pull/39


fix(deps): update icr.io/appcafe/open-liberty/samples/getting-started:latest docker digest to 1ac4699

This PR contains the following updates:

Package
Update
Change

icr.io/appcafe/open-liberty/samples/getting-started
digest
3999aa8 -> 1ac4699

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 1.4.43 :tada:
The release is available on:

GitHub release
v1.4.43

Your semantic-release bot :package::rocket:


Create README.md

Signed-off-by: John Bent<EMAIL_ADDRESS>Describe your changes in brief
Changes

[ ] Why is this change required? What problem does it solve?
[ ] If proposing a new change then please raise an issue first

How Has This Been Tested? (Optional)

[ ] Please describe in detail how you tested your changes.
[ ] Include details of your testing environment, and the tests you ran to
[ ] How your change affects other areas of the code, etc.

Screenshots (if appropriate)
Checklist

[ ] tested locally
[ ] added new dependencies
[ ] updated the docs
[ ] added a test

View rendered doc/PDFs/README.md

Looks good. @justinzw could you please approve it.


Fixing graph root padding and truncating long queries in execution plan (#19703)

Fixes issue #18961
PR to main branch #19703

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 41.95%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/contrib/executionPlan/browser/executionPlanViewHeader.ts
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
27931

Relevant Lines:
62116

💛  - Coveralls


Document the scripts in the deployer module

@sgibson91 and @yuvipanda, are you ok with me merging this PR and track updating "sub-scripts" to deployer actions in a different issue + PR?


chore: Create GitHub Issue templates

Had this sitting around in drafts for awhile, but I'm not sure if it needs more work before you might accept it.

Adds the Issue config, so users will have a little nudge to open Issues on the "correct" repo, instead of this one to start
Use the new YAML forms to allow a little guided data collection
Still allows the "blank" issue templates, if people don't like the config

I'm opening this on main instead of dev as the templates need to be on the default branch to work. Could open it against dev, but then they wouldn't show up till the next sync

Interesting ideas here, let me think about this! (Most of my own experience with templates has been more negative than positive.) (I try to never touch main branch, so I would want this in next as you suggest.)

I think I'll merge these ideas into a very simple template based on the original format. (Forms is apparently not official: https://docs.github.com/en/communities/using-templates-to-encourage-useful-issues-and-pull-requests/configuring-issue-templates-for-your-repository#creating-issue-forms) There doesn't seem to be a way to preview issue templates, so I'll iterate in a different repo, then credit you in the commit.

No need for credit :)
You can also use the builder in the repo Settings, and that will create the older Markdown based templates for you. You could pull in the config.yml separately if you like the generic links/descriptions form that file


add winiotsvcs stable api

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[x] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Can one of the admins verify this patch?

This was reviewed and approved in the PR branch: https://github.com/Azure/azure-rest-api-specs-pr/pull/789.

Automation for azure-sdk-for-python

Automation for azure-sdk-for-java

Automation for azure-sdk-for-go

This is the same as https://github.com/Azure/azure-rest-api-specs-pr/pull/789/files. Also this has ARM Approval. Approving and Merging this now....


Formatter returns too many decimals.

I want to use PHPGeo for simplifying lines in geojson files.
PHP has a precision set to 8 (Leaving 6 decimals for the coordinates)
I add points to a new Polyline with addPoint and new Coordinate.
$line = new Polyline;
 foreach ($feature['geometry']['coordinates'] as $coordA){
      $line->addPoint(new Coordinate($coordA[1],$coordA[0]));
}
I.e. the first coordinates from $feature['geometry']['coordinates'] are
Lng: 15.640289
Lat: 60.604169
For debugging I'm outputting to my html with:
$formatter = new GeoJSON;
echo var_dump($line->getPoints()[0]).'
';
echo $line->format(new GeoJSON()).'
';
The var dump on the points shows that the Polyline does indeed only have 6 decimals.
But the echo on the results from the format shows Lng and Lat with this amount of decimals:
Lng: 15.6402892857124840020333067514002323150634765625
Lat: 60.6041694346744890253830817528069019317626953125
Which defeats the whole purpose of my simplification goal. As I save more bandwidth by just removing decimals. I don't want precision on subatomic levels.
Edit: Fixed some typos. Was in a hurry when I wrote it.

Hi, thanks for looking into this. I was able to quick fix it by altering the source code of PHPGeo. In Location/Formatter/PolyLine/GeoJSON.php on line 36 I made sure the $point->getLng() and $point->getLat() gets formatted to 6 decimals with number_format.
$points[] = [$point->getLng(), $point->getLat()];
To
$points[] = [number_format($point->getLng(),6), number_format($point->getLat(),6)];
This is a quick fix and most likely not the optimal fix. (For people coming here with the same problem)
Thanks for a great library otherwise. Does exactly what I want it to do ( except the above bug ). 12mb too 2.8 mb by removing decimals and 2.8 mb to 900 kb by simplifying.

Okay after some fiddling I realized there is nothing wrong with your functions (did and echo on $points->getLng() ). The decimals get added by PHP's json_encode. I mentioned I added and ini_set precision to my code but an serialize_precision ini_set is needed also.
http://php.net/manual/en/function.json-encode.php
So I added the serialize_precision to my code and now it works as intended. Maybe you could update the docs so it mentions that you can control the amount of decimals in outputted GeoJson by using serialize_precision.
ini_set('serialize_precision',8);
8 gives 6 decimals as it counts the whole numbers to.  Works for me but may need altering for other uses.

Hi,
thanks for the update. I think, it's the best if that behaviour stays unchanged and is mentioned clearly in the documentation. I'll update the docs accordingly.

Docs are updated


Add Cloud Compute methods to test API

For more test coverage, tack on the following API methods in the testdata Compute API json:

compute.addresses.update
compute.addresses.patch
compute.addresses.aggregatedList

Resolve ResourceName naming conflicts in configgen, e.g. when one resource name path template is projects/{project}/region/{region}/addresses/{address} and another is projects/{project}/addresses/{address}, create resource names address and projectAddress respectively.

Codecov Report

Merging #1864 into master will decrease coverage by 0.07%.
The diff coverage is 53.77%.

@@             Coverage Diff             @@
##             master   #1864      +/-   ##
===========================================
- Coverage     84.68%   84.6%   -0.08%
- Complexity     4633    4647      +14
===========================================
  Files           438     438
  Lines         18405   18453      +48
  Branches       1953    1966      +13
===========================================
+ Hits          15587   15613      +26
- Misses         2140    2155      +15
- Partials        678     685       +7

Impacted Files
Coverage Δ
Complexity Δ

.../com/google/api/codegen/config/DiscoveryField.java
33.09% <0%> (-1.72%)
26 <0> (ø)

...er/java/JavaDiscoGapicSchemaToViewTransformer.java
95.78% <0%> (-1.06%)
17 <0> (ø)

.../java/com/google/api/codegen/discovery/Schema.java
63.63% <0%> (-21.6%)
20 <0> (+2)

...oogle/api/codegen/config/DiscoveryMethodModel.java
74.75% <100%> (-0.49%)
38 <0> (ø)

...om/google/api/codegen/config/FlatteningConfig.java
66% <100%> (ø)
9 <0> (ø)
:arrow_down:

...va/com/google/api/codegen/config/ProtoTypeRef.java
77.27% <100%> (+2.27%)
38 <4> (+4)
:arrow_up:

...ava/com/google/api/codegen/config/FieldConfig.java
64.15% <100%> (ø)
47 <0> (ø)
:arrow_down:

.../configgen/transformer/DiscoConfigTransformer.java
98.79% <100%> (+2.32%)
16 <4> (+1)
:arrow_up:

...r/java/JavaDiscoGapicRequestToViewTransformer.java
93.82% <100%> (+1.12%)
17 <0> (+1)
:arrow_up:

...oogle/api/codegen/config/DiscoveryRequestType.java
42.85% <50%> (+1.94%)
10 <2> (+2)
:arrow_up:

... and 8 more

Continue to review full report at Codecov.

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

@michaelbausor this PR isn't as lean as it could be, so there is implementation of codegen outside of the configgen domain. The main point of this PR was to expose more of the test API, which did need some codegen changes in order to get around NPEs and other exceptions that cropped up when the generator was first run on the updated JSON API defintion. Thanks!

Baseline changes:

addition of name resource path "projects/{project}/addresses/{address}" in test json requires

changes in DiscoGapicNamer to fully qualify resource names. This generates the ProjectAddress.java file, which is different from the existing Address.java file.
changes in ApiCallableTransformer to name an Address resource object parameter as addressResource rather than address, which would clash with the existing resource name path string address
changes in DiscoConfigTransformer to fully qualify resource names that have the same simple resource names, e.g. simple resource name address and the qualified resource name projectAddress

addition of AddressAggregatedList, which contains new AddressScopedList, in test json requires

the Schema.findChild() method, because AddressAggregatedList must first traverse through AddressScopedList to retrieve the desired List<Address> response object.

The other changes are either for style or they result from bubbling the above diffs throughout the toolkit pipeline.

Added unit test for Schema.findChild() and commented on baseline changes.
PTAL

PTAL

PTAL


Bondstore

Could some private data be kept in flash using bondstore? I tried using the read command but failed to do so(for now). Any suggestions?

Also interested in doing this. I’ve already modified it to store IRK/LTK’s. So I think this would be simple enough to implement.

Pls share an example. Maybe it will help?

Sure, when I get a few mins I'll make a fork and upload the code.

So I made a fork but then I couldn’t find the code with the bond store changes... I must have overwritten it when I was moving files around :(.
I guess I’ll redo it and I might as well  write some code to store arbitrary data as well.
Sorry for the delay.

So I made a fork but then I couldn’t find the code with the bond store changes... I must have overwritten it when I was moving files around :(.
I guess I’ll redo it and I might as well write some code to store arbitrary data as well.
Sorry for the delay.

Hi! Can you share an example for storing that bond data in flash? Or can you give an example of how to write data in flash while the BLE is running?

Wow, so sorry, I completely forgot about this. What I ended up doing is just creating a second instance of bondstore and storing arbitrary data in it by just using bondstore::putData(). I had modified the code previously for other things and made another solution but ended up doing it this way instead.


ASC / UserRightAssignment.ps1 force needed for every setting.

The following code is applied successfully but with not result in a compliant state if additional groups are included in the user right assignment.
src/Parsers/ASC/UserRightsAssignment.ps1
if ([string]::IsNullOrEmpty($Accounts))
{
$policyHash.Force = $true;
}
$policyHash.Policy = $Privilege;
$policyHash.Identity = $Accounts
This will set the user rights as declared
$policyHash.Force = $true;
$policyHash.Policy = $Privilege;
$policyHash.Identity = $Accounts

Just to clarify, The code above does not work as intended, and you think the IsNullOrEmpty check should be removed? That seems to be the only difference in the code.


Use the "warning" color for the "restart required" label

This makes the color match the warning icon placed besides the label (it's now yellow instead of red).

looks great

Thanks!

Cherry-picked for 3.1.1.


Elemental community release

While we have all the functional pieces working, noone from the community can consume the great work the team has done in the last 2-3 weeks.
Let's share our achievement with the community and enable them to use the "new" elemental stack
overhaul of rancher.github.io/elemental (written descending priority)

[ ] #235
[ ] #237
[ ] #236
[ ] #241
[ ] #244
[ ] #242
[ ] #243
[ ] #240
[ ] #239

Goal: Have someone from the community deploy an Elemental cluster (single node k3s is sufficient)

We also need to fix https://github.com/rancher/elemental/issues/193 as the release pipeline is currently disabled.
@kkaempf  What are the requirements for the release? A OCI artifact? ISO? vagrant environment? ISOs are useless currently unless we create some script to inject the cloud config into a final iso, as the iso will boot and do nothing. We have the create-elemental-iso which accepts that but also with defaults wont work out of the box.
What is the acceptable steps that someone from the community need to do to deploy a cluster? Manual elemental install and then manual call to elemental-register with an url? We want everything automated? half of it?

For example, on the elemental-operator we have a target int he makefile that locally installs a kind cluster and installs everything, including the operator.
Would that be acceptable for the community to play with? OR do we want VMs for that? Or we want them to install the operator and play with it in a real k8s cluster?

@kkaempf What are the requirements for the release?

Documentation (esp. a "quick start") to get a machine deployed with Elemental Teal (based on the build service artefacts) and connected to Rancher.

What is the acceptable steps that someone from the community need to do to deploy a cluster? Manual elemental install and then manual call to elemental-register with an url? We want everything automated? half of it?

No automation, no UI, just a handful of CLI commands. This is my usual script for deployments:

"helm install elemental-operator"
kubectl apply -f
wget https:///elemental/registration/....
elemental-iso-build ...
boot the ISO (this should auto-install and reboot)
lookup the machineInventory in the Rancher UI
kubectl label MachineInventory ...
kubectl apply -f machine-selector-template-...
kubectl apply -f create-cluster.yaml

Would that be acceptable for the community to play with? OR do we want VMs for that? Or we want them to install the operator and play with it in a real k8s cluster?

We should assume that people have successfully deployed Rancher Manager before. (Ideally, this is just a docker call :wink:)

Discussions ongoing. Will close this issue now.


feat(searchbar): add border-radius css variable

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[X] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[X] Build (npm run build) was run locally and any changes were pushed
[X] Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[X] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: #17426 & #18247
What is the new behavior?

Added CSS Variable --border-radius to <ion-searchbar>
Old Values are now default Values if --border-radius is not set

Does this introduce a breaking change?

[ ] Yes
[X] No

Merged. Thank you!


fix: skip types in current cloud backend

Description of changes
This PR removes types/ folder from backend -> current cloud backend -> s3 -> current cloud backend -> backend flow.
The types folder is always regenerated, therefore it's not needed to include it in that flow.

Issue #, if available

Description of how you validated changes

added unit tests for unzipper
full e2e run

Checklist

[ ] PR description included
[ ] yarn test passes
[ ] Tests are changed or added
[ ] Relevant documentation is changed or added (and PR referenced)
[ ] New AWS SDK calls or CloudFormation actions have been added to relevant test and service IAM policies
[ ] Pull request labels are added

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Thanks for fixing it!


Add documentation around prefix third-party fork branch names

This add information on how users can prefix third-party fork branch names

@mbelton-buildkite Thank you for the feedback. I accepted your suggestion.


checkup, tests: Update assertConfigMapWriterRoleBindingCreated()

Use checkup.NewServiceAccountSubject() instead of a struct literal.
Signed-off-by: Orel Misan<EMAIL_ADDRESS>This PR depends on PR #135.

Added this change to PR #135.


feat: Add discord as a link on the profile settings.

Pretty straight forward ask. We will need an open-sauced/api update for this.

User Settings

Hey @bdougie, should the discord URL be like https://discord.gg/opensauced or https://discordapp.com/users/{USER_ID}?

Hey @bdougie, should the discord URL be like https://discord.gg/opensauced or https://discordapp.com/users/{USER_ID}?

Should be the second. The user is linking to their personal discord for DMs, friend request, etc.

Okay, one way for users to add their discord link is by copying their ID and then updating their settings

But we don't want to display the full URL in their profile or just the ID.
Displaying the user handle is what we want to do.
We can do that by hitting the discord API (providing an API key) to get the username. But I'm looking for another way to get the username without making a fetch request.
I tried discordapp.com/users/{username} but it doesn't seem to be working.
Any ideas @open-sauced/triage?

Maybe just don't redirect this field anywhere if you want to display the username with the #tag.
This is what discord does with xbox ids (see my profile).
I also don't know of a way to get the username from the id without an API key.

Discord will not use the #tag anymore, they are going to use a normal username so we can ignore that idea now. (I might share the related link and read the full issue later)
Sent from mobile device

From: Divyansh Singh @.>
Sent: Tuesday, July 18, 2023 10:37:43 AM
To: open-sauced/insights @.>
Cc: Abdurrahman Rajab @.>; Team mention @.>
Subject: Re: [open-sauced/insights] feat: Add discord as a link on the profile settings. (Issue #1358)
Maybe just don't redirect this field anywhere if you want to display the username with the #tag.
This is what discord does with xbox ids (see my profile).
I also don't know of a way to get the username from the id without an API key.
—
Reply to this email directly, view it on GitHubhttps://github.com/open-sauced/insights/issues/1358#issuecomment-1639670666, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AELNMKIXEMQR3WTD5SIT5M3XQY4MPANCNFSM6AAAAAA2KUO4EY.
You are receiving this because you are on a team that was mentioned.Message ID: @.***>

More context on this decision. https://discord.com/blog/usernames

This doesn't change the original suggestion though, are we fine with a non redirecting label on the profile is the question here.


freertos_int32_publisher stops publishing after a few ros2 topic echo

int32_publisher or freertos_int32_publisher stops publishing after a few ros2 topic echo

Hardware description:  M5Core2
RTOS: FreeRTOS
Installation type: RTPS
Version or commit hash:  a2bab46d307f8ce19693a100af93c4c4e2553068 (same with older versions)

Steps to reproduce the issue

On the ESP32, I run the publisher. It publishes (writes "publishing: xxx")
On the PC side (docker ros:humble, I use ros2 topic echo --no-daemon /freertos_int32_publisher (same with --no-daemon), let it run for a while, then kill it, then run, etc.

Expected behavior

we should be able to see the published data

Actual behavior

after a few kills, the topic does not exist anymore.

root@talos-dev3:/# ros2 topic list
/freertos_int32_publisher
/parameter_events
/rosout
root@talos-dev3:/# ros2 topic echo --no-daemon /freertos_int32_publisher
data: 35
---
data: 36
---
data: 37
---
data: 38
---
data: 39
---
data: 40
---
data: 41
---
data: 42
---
^Croot@talos-dev3:/# ros2 topic echo --no-daemon /freertos_int32_publisher
data: 41
---
data: 42
---
data: 43
---
data: 44
---
data: 45
---
data: 46
---
data: 47
---
data: 48
---
data: 49
---
data: 50
---
^Croot@talos-dev3:/# ros2 topic echo --no-daemon /freertos_int32_publisher
data: 44
---
data: 46
---
data: 47
---
data: 48
---
data: 49
---
data: 50
---
data: 51
---
data: 53
---
data: 54
---
^Croot@talos-dev3:/# ros2 topic echo --no-daemon /freertos_int32_publisher
data: 53
---
data: 54
---
data: 55
---
data: 56
---
data: 57
---
data: 58
---
data: 59
---
^Croot@talos-dev3:/# ros2 topic echo --no-daemon /freertos_int32_publisher
WARNING: topic [/freertos_int32_publisher] does not appear to be published yet
Could not determine the type for the passed topic

The ESP32 node continues to publish (it continues to write Publishing: xxx every second --- it is not crashed or stuck in a loop).
It works again after a reset of the ESP32.
Additional information
root@talos-dev3:/# ros2 topic info -v   /freertos_int32_publisher
Type: std_msgs/msg/Int32

Publisher count: 1

Node name: _CREATED_BY_BARE_DDS_APP_
Node namespace: _CREATED_BY_BARE_DDS_APP_
Topic type: std_msgs/msg/Int32
Endpoint type: PUBLISHER
GID: df.bf.2a.3f.0d.ce.89.48.ee.2d.<IP_ADDRESS>.<IP_ADDRESS>.<IP_ADDRESS>.00.00
QoS profile:
  Reliability: RELIABLE
  History (Depth): UNKNOWN
  Durability: TRANSIENT_LOCAL
  Lifespan: Infinite
  Deadline: Infinite
  Liveliness: AUTOMATIC
  Liveliness lease duration: Infinite

Subscription count: 0

root@talos-dev3:/# ros2 topic info -v --no-daemon  /freertos_int32_publisher
Unknown topic '/freertos_int32_publisher'

FYI, I do not get the same behavior with the XRCE_DDS middleware (but this requires an agent... so it's much less convenient: we need to dedicate a PC with wifi etc for the agent).

Hello @jbmouret , we are not providing active support on this feature because this middleware is experimentally integrated into micro-ROS. Maybe you can find more information and answers in the official repo of the middleware: https://github.com/embedded-software-laboratory/embeddedRTPS


Added Consistent property to RiakBucketProperties

Signed-off-by: myles-mcdonnell<EMAIL_ADDRESS>

Good start, this will have to go though: SetConsistent.
The consistent and data type variables can only be set at the bucket type level from the command line, so that private property should only be able to be set through the RpbBucketProps version constructor.

Why'd you close these PR's?

I created a new one with the Consistent pro, it's waiting for merge..I managed to delete the branch with the FisherYates strategy (doh!), I'll do it again...


Error: Building package zlib:x86-windows failed with: BUILD_FAILED

-- The C compiler identification is MSVC 19.0.24234.1
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64_x86/cl.exe
-- Check for working C compiler: C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64_x86/cl.exe -- broken
CMake Error at E:/github/vcpkg/downloads/tools/cmake-3.14.0-windows/cmake-3.14.0-win32-x86/share/cmake-3.14/Modules/CMakeTestCCompiler.cmake:60 (message):
The C compiler
"C:/Program Files (x86)/Microsoft Visual Studio 14.0/VC/bin/amd64_x86/cl.exe"

is not able to compile a simple test program.
It fails with the following output:
Change Dir: E:/github/vcpkg/buildtrees/zlib/x86-windows-rel/CMakeFiles/CMakeTmp

Run Build Command(s):E:/github/vcpkg/downloads/tools/ninja/ninja-1.8.2/ninja.exe cmTC_37a29
[1/2] Building C object CMakeFiles\cmTC_37a29.dir\testCCompiler.c.obj
[2/2] Linking C executable cmTC_37a29.exe
FAILED: cmTC_37a29.exe
cmd.exe /C "cd . && E:\github\vcpkg\downloads\tools\cmake-3.14.0-windows\cmake-3.14.0-win32-x86\bin\cmake.exe -E vs_link_exe --intdir=CMakeFiles\cmTC_37a29.dir --rc=rc --mt=CMAKE_MT-NOTFOUND --manifests  -- C:\PROGRA~2\MICROS~1.0\VC\bin\AMD64_~2\link.exe /nologo CMakeFiles\cmTC_37a29.dir\testCCompiler.c.obj  /out:cmTC_37a29.exe /implib:cmTC_37a29.lib /pdb:cmTC_37a29.pdb /version:0.0  /machine:X86  /debug /INCREMENTAL /subsystem:console  kernel32.lib user32.lib gdi32.lib winspool.lib shell32.lib ole32.lib oleaut32.lib uuid.lib comdlg32.lib advapi32.lib && cd ."
RC Pass 1: command "rc /foCMakeFiles\cmTC_37a29.dir/manifest.res CMakeFiles\cmTC_37a29.dir/manifest.rc" failed (exit code 0) with the following output:
系统找不到指定的文件。
ninja: build stopped: subcommand failed.

CMake will not be able to correctly generate this project.
Call Stack (most recent call first):
CMakeLists.txt:4 (project)
-- Configuring incomplete, errors occurred!
See also "E:/github/vcpkg/buildtrees/zlib/x86-windows-rel/CMakeFiles/CMakeOutput.log".
See also "E:/github/vcpkg/buildtrees/zlib/x86-windows-rel/CMakeFiles/CMakeError.log".
ninja: build stopped: subcommand failed.

cmake.exe -E vs_link_exe --intdir=CMakeFiles\cmTC_37a29.dir --rc=rc --mt=CMAKE_MT-NOTFOUND --manifests

mt.exe? Is it a tool packaged with vcpkg?

I found mt.exe in the windows sdk directory, but vcpkg didn't find it.
How should I config it?
C:\Program Files (x86)\Windows Kits\10\bin\10.0.17763.0\x86\mt.exe

vcpkg env --triplet x86-windows --debug
where mt.exe

e:\github\vcpkg>vcpkg env --triplet x86-windows --debug
[DEBUG] Using vcpkg-root: e:\github\vcpkg
[DEBUG] _wpopen(""e:\github\vcpkg\downloads\tools\cmake-3.14.0-windows\cmake-3.14.0-win32-x86\bin\cmake.exe" --version 2>&1")
[DEBUG] _pclose() returned 0 after   279667 us
[DEBUG] _wpopen(""e:\github\vcpkg\downloads\tools\cmake-3.14.0-windows\cmake-3.14.0-win32-x86\bin\cmake.exe" "-DCMAKE_TRIPLET_FILE=e:/github/vcpkg/triplets/x86-windows.cmake" -P "e:/github/vcpkg/scripts/get_triplet_environment.cmake" 2>&1")
[DEBUG] _pclose() returned 0 after    80964 us
[DEBUG] _wpopen(""C:\Program Files (x86)\Microsoft Visual Studio\Installer\vswhere.exe" -all -prerelease -legacy -products * -format xml 2>&1")
[DEBUG] _pclose() returned 0 after    55285 us
Warning: The following VS instances are excluded because the English language pack is unavailable.
C:\Program Files (x86)\Microsoft Visual Studio\2017\Community
Please install the English language pack.
[DEBUG] CreateProcessW(cmd.exe /c ""C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\vcvarsall.bat"  amd64_x86   2>&1 <NUL && cmd")
Microsoft Windows [Version 10.0.14393]

where can I download english language pack for vs 2017 community?

this is not closed? why is it marked as close?

what are you talking about? I do not see any marks at all) although I should close
VCPKG requires the installation of the English locale, because in the parameters forcibly worth 1033

english language pack for Microsoft Visual Studio 14.0 ? ))

I compile using another machine using vcpkg, which is ok. Then I copied it back to my machine and do the rest developing works.
I close this issue for now. And thanks


(1.16.2, Astromine 1.7.0) Crash when viewing Astromine's armor

Whenever I try to look at my inventory or look in third person with Astromine's armor on, I immediately crash. I've been able to recreate this crash by simply wearing some armor and then going into my inventory.
crash-2020-08-18_19.00.33-client.txt

@shedaniel oopsie.


[BugFix] fix right join on empty left table

What type of PR is this：

[x] bugfix
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #10722
Problem Summary(Required) ：

Checklist:
If left table is empty, the _build_match_flag would not be initialized, so right join would not work.

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

@Mergifyio backport branch-2.4

run starrocks_admit_test

run starrocks_admit_test


typo: exress -> express

@nschonni : Thanks for your contribution! The author, @charwen, has been notified to review your proposed change.

@charwen and @nschonni I'm moving these commits to the private repo for validation and staging. I'll add the new PR here when it's created, and will then close this PR.

@nschonni thanks for fixing the typos

https://github.com/MicrosoftDocs/azure-docs-pr/pull/57847 in the private repository replaces this pull request. Closing this PR.


Crash when unexpected key type is received

I'm trying out Pijul and it's sending a sign request to the SSH agent which crashes it.
As far as I can tell, it's sending something other than an ECDSA key and sekey is running off the end of the buffer in EcdsaSha2Nistp256::read and panicking.
From the SSH agent protocol spec, it looks like the agent is expect to just return SSH_AGENT_FAILURE if it doesn't doesn't have a key (regardless of type, etc).
Handler::sign_request() or EcdsaSha2Nistp256::read() could be made a little bit more robust and check the key type.

@fmpomar wrote a new ssh-agent library that we will use for SeKey, I will work on implementing that today and push a new version. one of the main issues with SeKey is on the first version all the error handling was poorly done. With that implemented this issue will be closed.

Great, thanks!

This will be the new agent that we will be using. -> https://github.com/sekey/ssh-agent.rs/tree/dev


Start 1.2.1

No review needed.

Coverage remained the same at 32.582% when pulling 43abc6b7b044db63e6a7e7e355fc4fdbef485a8b on start_1.2.1 into a2980289e3aa6f8b0bd0846abf6caac7dc2cecf1 on stable_1.2.


Unhandled Fernet error leads to crash of shared instance

I think I culled the log down to reasonable limits, but this was the error that took down Between the Borders recently. I think this is one of the errors we've been having trouble tracking down.
[2023-10-28 17:44:03] [Verbose] Validating link request <aa9f22231857f8adbcebdf27cacf4f51>
[2023-10-28 17:44:03] [Extra] Registering link <aa9f22231857f8adbcebdf27cacf4f51>
[2023-10-28 17:44:03] [Verbose] Incoming link request <aa9f22231857f8adbcebdf27cacf4f51> accepted
[2023-10-28 17:44:06] [Error] Decryption failed on link <aa9f22231857f8adbcebdf27cacf4f51>. The contained exception was: Fernet token HMAC was invalid
[2023-10-28 17:44:06] [Error] An interface error occurred, the contained exception was: packed data must be type 'bytes' or 'bytearray'
[2023-10-28 17:44:06] [Error] Tearing down LocalInterface[37428]
[2023-10-28 17:44:06] [Error] The interface LocalInterface[37428] experienced an unrecoverable error and is being torn down. Restart Reticulum to attempt to open this interface again.
[2023-10-28 17:44:06] [Critical] Permanently lost connection to local shared RNS instance. Exiting now.023-10-28 17:43:56] [Debug] Timed out waiting for 2 parts, requesting retry
[
[2023-10-28 17:44:06] [Error] No interfaces could process the outbound packet

Lots of other timeouts as well. I don't think propagation nodes like the bandwidth provided, but I think that's a mostly unrelated event.

Yes, I think it was one of those hard to track ones as well, and now that I went through it, it makes sense. Great you spotted this. Let's see if it stops happening now.


double-beep: Watch decoenligne.org

double-beep requests the watch of the watch_keyword decoenligne\.org. See the MS search here and the Stack Exchange search here.

decoenligne\.org has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by tripleee in Charcoal HQ


website examples added list

ml5 functions

[ ] DCGAN()

[ ] example1
[ ] example2

[ ] featureExtractor()

[x] FeatureExtractor_Image_Classification
[ ] FeatureExtractor_Image_Regression

[ ] CVAE()

[ ] example1
[ ] example2

[ ] KNNClassifier()

[ ] KNNClassification_VideoSound
[x] KNNClassification_Video
[ ] KNNClassification_PoseNet
[ ] KNNClassification_VideoSquare

[ ] pitchDetection()

[ ] PitchDetection_Game
[x] PitchDetection

[ ] styleTransfer()

[ ] StyleTransfer_Video
[x] StyleTransfer_Image

[ ] UNET()

[ ] example1
[ ] example2

[X] word2vec()

[X] Word2Vec

[ ] CharRNN()

[ ] CharRNN_Interactive
[X] CharRNN_Text
[ ] CharRNN_Text_Stateful

[ ] example()

[ ] example1
[ ] example2

[ ] poseNet()

[X] PoseNet example on image with single detection
[X] ePoseNet example using p5.js with webcam

[ ] YOLO()

[X] YOLO image detection on single image
[ ] YOLO_webcam

[ ] pix2pix()

[X] Pix2Pix_callback
[ ] Pix2Pix_promise

[ ] SketchRNN()

[X] SketchRNN_basic
[ ] SketchRNN_interactive

[ ] imageClassifier()

[x] ImageClassification
[x] ImageClassification_Video
[ ] ImageClassification_VideoScavengerHunt
[ ] ImageClassification_VideoSoundTranslate
[ ] ImageClassification_VideoSound
[ ] ImageClassification_MultipleImages

All of our examples in the codebase are now stored in ml5-library and are available on examples.ml5js.org!


There is two CSRF vulnerability that can delete user or usergroup

Software Link : https://github.com/chekun/DiliCMS
After the administrator logged in,open the page
test.html delete user POC:

    <img src="http://<IP_ADDRESS>/DiliCMS/admin/index.php/user/del/1" />

test2.html delete group POC:

    <img src="http://<IP_ADDRESS>/DiliCMS/admin/index.php/role/del/2" />

Please use https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2018-19291 for this vulnerability.


Fix Guest Auth

Fixes #1391 by introducing a new SessionRole called Guest.
Now backend can be configured to enable user system or not. By default, user system is disabled, and the Guest role can be used as a placeholder for endpoints which optionally requires user information, such as api/queryplan/autocomplete.

Looks good.
Since we have added one more conf to enable user dashboard, can you update our wiki to include this config: https://github.com/Texera/texera/wiki/Guide-for-Developers#to-enable-user-dashboard
Also the guest account is available when the user dashboard is not enabled. I'm thinking if there is a way we automatically assign the guest account to non-logged-in user. For example, visit the production system without logging in.

added.


Losses in training have a lot of shock

Dear@JiaRenChang ，When I used PSMNet to train SceneFlow datasets, the loss was quite volatile. Is the data set not clean? And how do I deal with it? Should I delete some unreliable data? Look forward to your reply！

Thank you for your timely reply. I also noticed the restrictions on disparity in the code, and I didn't modify the code. Attached picture is my training loss.

@walkingmelon
Yep, it is fluctuating.
Which pass of Scene Flow do you use ?
We use the cleanpass version.

Yes, I use the same data set. Maybe my data set is contaminated. I will now download the dataset again. I hope I will have good results.


If verbose_eval is int, any verbose_eval would be '1'

In 1.3.0, if verbose_eval is int, any verbose_eval would be '1'.
e.g. If verbose_eval=100 was passed, the evaluation metric on the validation set should be printed at every '100' boosting stage, but actually the evaluation metric would be printed at every '1' boosting stage.
I confirmed this doesn't happen in 1.2.0.

Yup, it's fixed in master branch.  For now,  you can consider getting a nightly build by the link listed in installation doc.