Add ability to provide custom client configuration

:loudspeaker: Type of change

[ ] Bugfix
[ ] New feature
[x] Enhancement
[ ] Refactoring

:scroll: Description

One common request is to add simple way to provide custom ClientConfiguration for different integrations. This change enables users to define a bean of name com.amazonaws.ClientConfiguration.BEAN_NAME that would be by default set on all AWS clients and in addition to that, define ClientConfiguration per client.
Client-specific beans must have specific name, for example - SQS ClientConfiguration bean must be named - sqsClientConfiguration.
:bulb: Motivation and Context

Common request:

https://github.com/spring-cloud/spring-cloud-aws/issues/412
https://github.com/spring-cloud/spring-cloud-aws/issues/63

:green_heart: How did you test it?
Integration tests.
:pencil: Checklist

[x] I reviewed submitted code
[x] I added tests to verify changes
[ ] I updated reference documentation to reflect the change
[x] All tests passing
[x] No breaking changes

:crystal_ball: Next steps

[x] Add support for modules once PRs are merged:

Parameter Store (https://github.com/spring-cloud/spring-cloud-aws/pull/723)
Secrets Manager (https://github.com/spring-cloud/spring-cloud-aws/pull/721)
SES (https://github.com/spring-cloud/spring-cloud-aws/pull/669)

[x] Update reference docs

Currently, we have our own SpringCloudClientConfiguration which add a suffix to the user agent. By doing an injection of ClientConfiguration we will lose this. I think we can do something like RestTemplateBuilder and RestTemplateCustomizer and not lose this.

@eddumelendez suffix is still added: https://github.com/spring-cloud/spring-cloud-aws/pull/725/files#diff-06d076e20794983d503af739eef415c367243d9e21195c978fc8730d70209722L97

@maciejwalkowiakis is there a plan to also add this to aws-paramstore as currently there isnt a way to customize this (specially in the case w need to pass custom proxy ) . Also even marking beans as primary still uses beans from the default configuration AwsParamStoreBootstrapConfiguration we have observed for ssmClient .


txn: allow mixing latch with concurrency manager

What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
What is changed and how it works?
Proposal: xxx
What's Changed:
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note

/run-all-tests

/bench +sysbench +tpcc

/bench +sysbench +tpcc

/run-all-tests

Benchmark Report

Run TPC-C Performance Test on VMs

@@                               Benchmark Diff                               @@
================================================================================
tidb: f5c6e59f0daf2c297b4ee8cd4fc08e1423257438
--- tikv: 41560b21793843602b2ed363c8c58789fe1b347d
+++ tikv: 6f71b588c48df678f93f844fe2d09dac4dd70bb7
pd: 082217c47f164f0f471051bafa7991064e614356
================================================================================
Measured tpmC (NewOrders): 4334.59 ± 32.68% (std=996.63), delta: -19.14% (p=0.268)

The performance gets a lot worse...


[Tech] Save theme settings to avoid reseting them on each application start

I don't have any clear idea on how to do it but I think a proto Datastore would be a good start like in the NIA app 🤔
We may consider also adding a VM to the MainActivity and branch the repo to it.

@elisa-hery there's no specs for this but can you confirm that when you change a parameter from the theme configurator it does get saved when you leave and come back in the app?

@soulcramer yayyy it's working 🥳
I'm moving the ticket to done


ARROW-12198: [R] bindings for strptime

@kszucs - could you you enable @AlenkaF to build on CI please?

@kszucs - could you you enable @AlenkaF to build on CI please?

Enabled.

Thanks, I guess I need help (approval for CI build) once more =)

@jonkeane @nealrichardson - in failed test expect_equivalent() fails due to time zone being stored in lubridate timestamp but should ignore attributes if I understand correctly. I am running out of ideas how to solve this one :(
Thank you for the help!

Oh, this was a fun one dig through and figure out what was going on. As I'm sure you've seen, the failure is only in the devel build, and it turns out that all.equal.POSIXt() has changed recently: https://bugs.r-project.org/bugzilla/show_bug.cgi?id=17277 is the bug report that adds timezone checking.
Interestingly, comment 4 seems to indicate check.attributes should be respected, thought it is not (currently) being respected. I've sent an email to the r-devel list and asking if check.attributes is supposed to be ignored in this case (but you can see a replication of this in the code below).
For now, I think if you add check.tzone = FALSE to the expect_equivalent() calls that will fix this (and not cause problems in other versions).
> all.equal(
  list(lubridate::ymd_hms("2018-10-07 19:04:05", tz = NULL)),
  list(lubridate::ymd_hms("2018-10-07 19:04:05")),
  check.attributes = FALSE
)
[1] "Component 1: 'tzone' attributes are inconsistent ('' and 'UTC')"
> all.equal(
  list(lubridate::ymd_hms("2018-10-07 19:04:05", tz = NULL)),
  list(lubridate::ymd_hms("2018-10-07 19:04:05")),
  check.tzone = FALSE
)
[1] TRUE
>

Thank you @jonkeane for your feedback.
I corrected the code and would like to ask you for a review, if I may.

@jonkeane @nealrichardson I need approval for the check and the code is ready for another review round. Thank you!


First React Native project post on labs

@persand @vasilakisfil Post about the React Native project, please review and let me know any suggestions you may have, thanks!

Looks good to me! :+1:


Add publish workflow, update version and dependencies

Add CARGO_REGISTRY_TOKEN to repo secrets before release.

Do you know why the CI hardhat test is trying to connect to Infura even if you seem to have changed it to viem's default mainnet endpoint which is cloudflare-eth.com?

Do you know why the CI hardhat test is trying to connect to Infura even if you seem to have changed it to viem's default mainnet endpoint which is cloudflare-eth.com?

I didn't change the RPC for hardhat tests. Please merge it since I don't have write access.


groups: various first-run fixes

Removes GroupSummary from the group home
Renames group home to "Activity" and uses bell icon:

Cleans up overlapping GroupReference layout
Sets the default app route to Find Groups
De-emphasizes the empty activity notices
Adds suggested groups in Find Groups:

A couple updates to this PR:

Correctly sets the feedback asterisk to blue:

Adds a "no groups" empty state:


Add extsources to bzip2

please disable wasm platform

on_install("linux", "macosx", "windows", "android", "iphoneos", "cross", "bsd", "mingw")


Files left 'dirty' after format-on-save

With this extension installed, the files get formatted as expected whenever they are saved. The unformatted file gets saved, then the formatting is applied, but the reformatted file is left 'dirty' (not saved). So the net result is not much different to just running a ReSharper Code Cleanup.
If the file is saved again, then the Code Cleanup is re-run, and sometimes (just sometimes) the save will happen correctly. Mostly though, the file will never save correctly.
I'm using VS2017 and I don't recall this being a problem in VS2015.

Duplicate issue. See #24.


Update glob for node 6 compatibility

Older version of glob uses a version of graceful-fs, which is deprecated and incompatible with node v6.

This also removes a graceful-fs deprecation warning for those with OCD that are don't like that type of thing.

Thanks! It'll have to wait – cannot bump in mocha v2 for the same reason we cannot bump jade: https://github.com/mochajs/mocha/issues/2200#issuecomment-216017864.

I admire that you strictly respect semver and still support node 0.8.x users, even though it's not even maintained by the node team.
However, mocha is also very important on the cutting edge part of the node ecosystem.
To help adoption of Node 6, and also reduce potential for breakage for those upgrading to Node 6, could you release just that small change as mocha 3.0.0 (bumping version just to satisfy semver for node 0.8 users), and rename the currently planned v3.0.0 to v4.0.0?

@pornel I was actually thinking of doing something like that, or maybe v3.0.0-alpha or the sorts – thanks for your voice! Making Mocha run on Node v6 is important imo, and you can tell other users think so as well.
What do you peps think, @mochajs/mocha?

Try npm install mochajs/mocha to use the latest master from GitHub and see if that still has the issue? It looks like #2201 should have eliminated the graceful-fs dependency by updating glob to 3.2.11, it just hasn't made it into a release yet. Of course, one would want to use an actual release of Mocha (at least in production); if I'm not mistaken the changes from #2201 will be able to go into a 2.4.6 or 2.5.0 release, without breaking compatibility, once #2079/#2231 is done.
That said, as a user I'm all for having an early copy of the next "major" version though (which should be able to resolve the Jade warning among other things; updating to the latest version of all dependencies if possible might be a good idea at that point), if it isn't too much trouble to maintain both that and the current 2.x.y release line until the Mocha team is ready to switch over fully.

@ScottFreeCode Seems to be correct that Node.js v6.x is not broken when installing from master, but is from the currently published v2.4.5 (though I have nothing to confirm this except the warning when installing).
Installing from master against Node.js v0.8 yields no errors or warnings.
This leads me to believe we could release v2.4.6 with an upgraded graceful-fs.  I am extremely hesitant to do this until we get automated browser tests in place.  I thought using the "CLI-only" chalk module would not affect browser users; I was incorrect, which is the reason for my hesitation.  Nothing can be so easily assumed when bundling for the browser.
If I can confirm graceful-fs causes a Mocha installation to break--not just yield an install warning--I'll investigate doing a patch release later today; at minimum, we'll want to do some manual browser tests, and update our CI matrix.
cc @mochajs/mocha

Going to close this PR if that is the case; if that is the case, the problem has already been solved.

Can anyone confirm a case where Mocha breaks under Node.js v6?  Our test suite passes with Node.js 6.
I'm having trouble understanding the trail of issues, discussion and code surrounding the deprecation of old graceful-fs versions.  Does it actually break, or will it actually break in Node.js v7?
See isaacs/node-graceful-fs#58 prompting the deprecation warning, and the linked issue in nodejs/node#5213.
Hopefully one of @isaacs @chalker @jasnell or @thealphanerd can help distill the conversation around those issues...?  Thank you, if you can.

@boneskull The deprecation warning upon installation in graceful-fs isn't very precise — it still works with Node.js 6 due to a temporary fix that was landed on Node.js side just for that, see https://github.com/nodejs/node/pull/5102.
That temporary work-around is going to be reverted in Node.js 7.0 (refs: https://github.com/nodejs/node/issues/5213, https://github.com/nodejs/node/pull/6413).
Note that the fix also prints a warning in runtime, from Node.js side, and that warning is printed only in those situations that will throw in 7.0.

I opened https://github.com/isaacs/node-graceful-fs/issues/64 in regards to the deprecation message.
The initial graceful-fs deprecation message was fine, but it was mistakenly changed in https://github.com/isaacs/node-graceful-fs/issues/61.

@ChALkeR Thanks for the explanation.  I can confirm the warning is now fixed.

To summarize for interested parties:

<EMAIL_ADDRESS>which current<EMAIL_ADDRESS>uses, does not actually break on Node.js v6, as you will see from the updated deprecation message of its dependency, graceful-fs.
<EMAIL_ADDRESS>should break under Node.js v7 in certain (?) circumstances.
We updated to<EMAIL_ADDRESS>which should not break under Node.js v7, but have not released this change.
Since nothing is actually breaking right now (there is no Node.js v7 yet), I'm not in a hurry to release anything.
So, I'm going to close this issue, and we'll release a new patch version with some recent fixes (including the upgraded<EMAIL_ADDRESS>once necessary changes for #2231 land.

(warnings can be hidden during npm install by using --loglevel=error or configured permanently in .npmrc, if it's bothering you too much)

Related:
#2201 & #2262


Go through all the Brightspace settings and config to make sure it is setup for the class

I've never used Brightspace to any extent, so I should just run through every option and see what to turn on and off.

I think I've cleared all the old stuff out and disabled most things we will not use. Only thing left to do is the #27.


Some Strange Behavior after initial setup

Following along with instructions when the app runs I see:

Adding event causes 2 events to appear for half a second then it disappears
Commenting causes 2 comments to appear (only 1 is actually created in DynamoDB), and then after 2 seconds or so the entire event screen clears out and becomes a single "Back to Events" button.
Clicking on the button takes me back to the events listing but no event is displayed
Clicking the browser's Refresh Button makes the event appear again.
Attempting to add another comment cause the error below to be thrown:

Uncaught TypeError: Cannot read property 'comments' of undefined
at Object.props (EventComments.js:63)
If it helps I'm running npm start in node v8.9.1 and my browser is "Version 64.0.3282.167 (Official Build) Built on Ubuntu , running on LinuxMint 18.2 (64-bit)"
Thanks

Hi @allain
Thanks for trying this sample! Could you please check if you get the same behavior after deleting your node_modules and re install using yarn?

Thanks for responding.
Using yarn 1.5.1 on first run I'm getting:
./node_modules/react-apollo/react-apollo.browser.umd.js
Module not found: Can't resolve 'apollo-client' in '/home/allain/fq/aws-mobile-appsync-events-starter-react/node_modules/react-apollo'

Unfortunately after yarn add apollo-client it runs but the problems in the issue still persist.

I am also experiencing this problem. Would be great, if it would be fixed quickly.

Hey, can you guys try again with the recently added yarn.lock ?
Feel free to re-open if needed

I just tried the new yarn file and some sources point to localhost. Shouldn't be right?
error An unexpected error occurred: "http://localhost:4873/aws-appsync/-/aws-appsync-1.0.12.tgz: connect ECONNREFUSED <IP_ADDRESS>:4873".

same error here

Hi @allain and @vilius-rugaitis I have updated both lock files.
Please let me know how it goes

I'm having the same issue with latest code from master (duplicates events and comments).
I had the Uncaught TypeError: Cannot read property 'comments' of undefined at Object.props (EventComments.js:63) error as well but can't reproduce at the moment.
I'm pretty new to apollo, GraphQL and the optimistic response mechanism, but shouldn't setting an id on the client (using uuid), updating the cache based on that id, then updating again based on the response from the server (which auto generates a different id) might cause the duplication?
Regarding the Uncaught TypeError: it seems like a timing issue where the data is set to an empty object for a brief moment.
If react happens to perform a render cycle while the data object is empty you get that error.
I haven't been able to figure out the reason for that yet.

Hi @erezrokah did you use yarn or npm install?

@elorzafe I tried both (removed node_modules before each time).
Currently using yarn install.
Also cleared the browser cache.
It is more evident once you have more than 3 events or so.

Same bug here after precisely following all the instructions

@elorzafe can we get an update regarding this issue?

@manueliglesias has this been fixed?

@manueliglesias  has this been fixed?


DownloadBuildArtifact: FileContainer

before coming out of preview we should add UNC support (robocopy).  Perhaps Eric should add that in since he's familiar with it on the publish side ...
@ericsciple


docs: miscellaneous README changes

Add @freitagbr to the bottom of the README
Remove docs about shjs (it's not well supported)
Remove docs about coffeescript (we don't do anything special for
coffeescript anyway)
Modify the section about shelljs/global, since we strongly recommend
avoiding this now
Rewrite the code example to use require('shelljs') instead of
shelljs/global
Mention ESLint next to JSHint
Reformat sections with long lines to be 80 columns

@freitagbr PTAL. I changed NodeJS ➡ Node.js in two places where it appeared.

One of the "NodeJS" occurrences is in the src/exec.js docs, so it needs to be fixed there too.

@freitagbr PTAL. Thanks for catching this

LGTM :+1:


Add hotkey to clear all counters/notifications

It would be nice to have a keyboard shortcut to clear all notifications/counters/favicon.
CTRL+L is a commonly used "clear" shortcut.

CTRL+L is used in my client for moving the marker [b]l[/b]ine, so I would find that particular choice confusing.
But clearing all unread badges would be a nice thing to have, since I use The Lounge as a secondary client and often don't check it for a few days.

CTRL+SHIFT+L maybe then ?

I don't think a shortcut for that would be best. I think it would be more appropriate as a UI element in #791. Then we can think configurable hotkeys as a whole as this is very specific IMO.

I don't think this is very specific, usually software that comes with notifications also comes with an easily accessible way to clear it.
But I would be fine with a configurable hotkey.

Slack and Discord both provide Shift+Esc as shortcut to clear all notification counters. Given the variety of opinions I believe making this shortcut configurable is the best approach.

I would suggest having a context menu item on channels and networks, something like "Mark all messages as read"


This community can close. 这个社区可以洗洗睡了，Meta要介入政治却拒绝社区讨论政治

Title translate: This community can close. Meta insist to involve politics but refuse to discuss politics.
To  gaearon:
Before you close or delete this issue,  please see below 2 links and rethink if it's too early to involve politics:
https://vimeo.com/178871166
https://www.youtube.com/watch?v=6Oh-IE2zmJc

People in Ukraine are in the middle of a severe humanitarian crisis. According to UN, a million of people have become refugees this week. It is widely reported that many people, including kids, are sheltering in basements, traveling in freezing temperatures, and are short on supplies. This is why we would like to encourage everyone using React to help those in need.

To support humanitarian, the standard words should be "Provide Humanitarian Aid to Ukraine", not "Support Ukraine & Provide Humanitarian Aid to Ukraine".  See, I'm a Chinese but I have better English understanding than you. You are not standing on humanitarian, but on politics.

However, this is not a zero-sum activity. First, we are only a single project, and there are other projects which may choose different causes (or not support any causes at all). Second, getting the message out about one crisis does not hurt people affected by a different crisis. This is why the “whataboutism” is mostly a distraction, and we will not spend time seriously entertaining it.

Yes it's not zero-sum activity, if you stand on "Support Ukraine" politic attitude, will hurt people who died in Donbass in last 8 years.

You are free to suggest other causes to support in the future. However, spamming the repo with duplicate issues and comments is disruptive. This violates the GitHub the terms of service, and GitHub will be removing this type of content. There is nothing more we would like to say on this at this point, so we will be locking this issue.

1)It's not spamming, it's HUUUUUUUUUUUUUUUUUUUUUGE political conflict. community has decided to keep neutral, but Meta decide to involve politics, I don't think community can keep calm and continue work on this project until find a solution, in Chinese there is a idiom "道不同不相为谋" ，means if people have conflict politic views they can not work together.
2)If Meta insist  involve politics, and community insist to keep neural, could you quit this community and stop pollute this community with your politic issues? you only represent yourselves politic views , not community's.
3)Violates the GitHub the terms of service? -- Think who first invoke this topic? Meta decide to introduce politic and put politic slogan in source code, why you want forbid community to discuss politics? it's not "political right", and it's double standard.
4)As I said before, if Meta insist involve politics, better to revise the project license to "Politicized MIT license", this is more clear to tell users to kick out "Conflict Political Views Software" from business projects.

We have already said everything we wanted to say on this topic at this moment in https://github.com/reactjs/reactjs.org/issues/4433#issuecomment-1060759889. There's nothing more we want to add at this point.


Symbol search doesn't find items generated by macros

macro_rules! id {
    ($($i:tt)*) => {
        $($i)*
    };
}

id! {
    struct Struct;
}

fn main() {
    let _ = Struct;
    //  ^   ^ go to type definition or go to definition works here
}

I was working on chalk and was searching for some types in the symbol search and could not find them. Confused, I observed that all the affected items are generated by macro_rules macros. To reproduce it with the example above, attempt to use the symbol search to find Struct, which will not be listed. Go to definition and all other analysis seems to work properly, however.
Version: 8cd171cd949f6c82a9ce50645f6ca68b1249c4c8

This is https://github.com/rust-analyzer/rust-analyzer/issues/4842


some docs for Sources

also,

added the tileSize parameter for RasterSource
remember*Source return typed *

I just documented the status quo, you may want to check if you think the type of exceptions thrown is what you want.


add stats and stats_host command for mod_admin_extra

Hello, thank you for your contribution. I noticed that the checks in CircleCI failed: https://circleci.com/gh/dirkmoors/pyejabberd/167
Could you have a look?


Sum of Time and Currency in cell

Hello,
I have 2 separate columns for Currency($) and Time(HH:MM) and I need a sum of columns individually after the end of rows.
Please let me know how can i calculate time and currency

Here is the solution for that
https://github.com/ladaniavadh/num-format-solution


host -v Locked Collateral value is higher than total SC existed on the wallet

./siac host -v
displays Locked Collateral value higher than existed on the wallet.
I believe the place that should be checked is hosting while wallet is locked.

+1 I've seen this problem on my host too.

I believe this issue is adequately tracked between #2262 and #2144.
Recommendation to devs: Close


Requesting less glideins when using multiple credentials

Describe the bug
When using multiple credentials or tokens+GSI the Frontend is requesting fewer Glideins (Idle and Max) than it should.
There are 2 underlying problems:

the splitting expression in the plug-ins is using regular division where it should use integer division
GSI is added to tokens submission to support legacy systems, it should not be counted as an additional credential to split glideins

To Reproduce
Configure a frontend to use SciToken and also send a GSI proxy (current default)
Expected behavior
Both issues should be fixed:

integer division should be in the expression to provide the same results as in python2
the x509 proxy added to scitokens to support legacy Gratia probes should not be counted when partitioning glideins to credentials

Screenshots
NA
Desktop (please complete the following information):

GlideinWMS version: 3.9.5 (and earlier 3.9.x)
Python version: 3.6
OS version: rhel7
HTCondor version: 9.0.11 (any)

Additional context
NA

This was addressed with PR #178.


Set requires_grad to False by default

For #385

Thanks! So I think we need a helper function to do this for the models, maybe if you add the params to an optimizer they get updated to require_grad in the constructor?

Merged #387, was simpler with the tristate. The issue with doing it explicitly is that you will forget a requires_grad on one of your model params, your model won't optimize well, and you won't know why.
But thanks for actually forcing me to do this. Move fast can break things!


Still rearranging the folder

building the final structure before splitting the files into multiple subcategories.

We have a "util" folder and an empty "utilities" folder. Should we get rid of one of them?


Any plans for supporting a Multi-Master environment?

Subject says it all. I have recently been put in charge of supporting a Multi-Master DNS environment.

@majoros One major feature that has been under discussion for some time is zone management.  It requires us to get "closer" to the DNS backends (likely having providers for different DNS software).
If you have any thoughts about specific features you would like to see wrt multi-master please let us know.

VinylDNS currently offers the flexibility to define DNS backends (https://www.vinyldns.io/operator/config-api#default-zone-connections) for zone connectivity which would allow you to define connection information at startup which would be linked to specific masters. Otherwise, the default zone connection would be used when connecting to a zone.

As mitruly pointed out ..
You should be able to utilize what's currently available. Just need to ensure you're updating the backend hidden auth servers you want. You should only need to update one and the other will be updated due to how federation (MM) works. If one goes down, you just change config to use the active backend. We run federated servers in our environment and don't have any issues.

I feel that the DNS infrastructure should handle propagation across primary auth servers. If there's some specific need outside of synchronization that multi-master presents, we can open a specific issue.


Does this program supports tensorboard?

Does this program supports tensorboard? Could not find any logs of tensorbard.

@Chevolier, can you please clarify the program you are referring to? It would be helpful to share what you are running and the expected output. Thanks!

@Chevolier, can you please clarify the program you are referring to? It would be helpful to share what you are running and the expected output. Thanks!

I mean the the 3rd step, applications/DeepSpeed-Chat/training)/step3_rlhf_finetuning/. In particular, I run the multi-node/run_66b.sh using 2 nodes with 16 GPUs in total, and the model is bloomz-7b1, I can see the reward score in the standard output. But can I see the training process using tensorboard?

Hi @Chevolier,
DeepSpeed has monitoring functionality built in and the monitor can be selected by specifying the corresponding configuration (TensorBoard, WandB, csv).
The documentation can be found here:
https://www.deepspeed.ai/docs/config-json/#monitoring-module-tensorboard-wandb-csv
For TensorBoard, an example configuration may look like this:

"tensorboard": {
    "enabled": true,
    "output_path": "output/ds_logs/",
    "job_name": "train_bert"
}

The configuration can be added to the get_train_ds_config utility function found here:
https://github.com/microsoft/DeepSpeedExamples/blob/dafeb2b3be3a085214faa2f59a8979c051424938/applications/DeepSpeed-Chat/training/utils/ds_utils.py#L32
Which will allow models that are initialized to have a monitor specified. Please let me know if you run into any problems with this method.
Thanks,
Lev

Hi @Chevolier,
Just wanted to update you that we have a PR to add various instrumentation across all the DS Chat steps, including tensorboard logging (GH-624)
Feel free to give it a try to see if it works on your end.
Thanks,
Lev


Fix for repeat header table display Issue 111

Attempt to fix issue described here - https://github.com/vandeseer/easytable/issues/111

Thanks for your PR!
I will have a look as soon as I find some time.

Thanks for your PR!
I will have a look as soon as I find some time.

Thanks a bunch for your work @grasshopper7!
There are some very minor stylistic things I will change (like omitted brackets or indentation) – hence I will first merge to the develop branch and soon (maybe in a week or so since I am currently running short on time) will release a new minor/bugfix version from master (obviously after having merged develop into master).
Thanks again.
Best,
Stefan

Thanks a bunch for your work @grasshopper7!
There are some very minor stylistic things I will change (like omitted brackets or indentation) – hence I will first merge to the develop branch and soon (maybe in a week or so since I am currently running short on time) will release a new minor/bugfix version from master (obviously after having merged develop into master).
Thanks again.
Best,
Stefan


How to set Text Alignment?

Hi,
Can someone tell me how to set text alignment on this control. I found it on line# 251 and line# 262 inn NYAlertView.m but want to do it from my ViewController because on some ViewControllers I want Left text alignment and in some ViewController I want center.

It is done now 👍
Needed to create another setter and getter for UITextView in NYAlertViewController
add in NYAlertViewController.h
@property (nonatomic) NSTextAlignment textAlignment;
and add in NYAlertViewController.m
- (void)setTextAlignment:(NSTextAlignment)textAlignment
{
    self.view.messageTextView.textAlignment = textAlignment;
}

Above stuff will start displaying **alertViewController.textAlignment =** into your code.


fix(version): update fabric8-stack-analysis-ui to 0.1.7

There were the following issues with your Pull Request

Commit: 03c2ff29def61fff0aaa0adb6409286904599a61
Your subject line is longer than 50 characters

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com

@fabric8cd snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-723.int.rdu2c.fabric8.io

:tada: This issue has been resolved in version 4.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


add groups to xarray

Codecov Report
Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (164178c) 59.16% compared to head (58ec516) 59.06%.

:exclamation: Current head 58ec516 differs from pull request most recent head ad277a7. Consider uploading reports for the commit ad277a7 to get more accurate results

Files
Patch %
Lines

src/python/mod.rs
0.00%
3 Missing :warning:

src/idx/index.rs
96.77%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
- Coverage   59.16%   59.06%   -0.10%
==========================================
  Files          22       22
  Lines        1200     1202       +2
==========================================
  Hits          710      710
- Misses        490      492       +2

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


chore: include directory for monorepo packages

npm allows to include a directory property for monorepo package
that some tools use to identify package location
https://docs.npmjs.com/files/package.json#repository

a meme

Mhh I don't know about this one as mono repo isn't published in NPM ?

👍 thanks, forget my previous comment


Centralize logic for injection of JSCompiler library code and polyfills.

Currently:

Transpilation passes are responsible for requesting injection of library JS code they need.
polyfills for native JS features (e.g. Promise) are injected by rewritePolyfills based on the language
level detected during parsing.
All of the above happens before type checking, so all of the injected code will be in the AST during type checking.

This has some problems.

Since we're planning to move transpilation passes after type checking a new plan is needed.
The logic around rewritePolyfills and removeUnusedPolyfills is more complicated than it needs to be and also fails to add / remove some uses because it's trying to avoid having polyfills keeping each other alive by using each other.
There are issues where e.g. Map polyfill may not get injected because the code didn't happen
to use any ES6 syntax features (i.e. features detected by the parser), leaving the rewritePolyfills pass untriggered.

In broad strokes the plan is to:

Rework/rename rewritePolyfills so it injects all of the polyfills and library code that could be needed given the requested input and output language levels regardless of what the parser says was in the input.
Transpilation passes will no longer be responsible for injecting library code and may assume it is there.
Update RemoveUnusedCode to handle removal of polyfills.

Make it understand that $jscomp.polyfill('Symbol', ...) is defining Symbol and avoid traversing
the call until it sees an actual use of Symbol.
This should be more robust than the 'guarded use' idea that rewritePolyfills and removeUnusedPolyfills

I believe this work is all done. I just forgot to close the issue.


Who can view reports

I am in  a Global Reader role and I can review the reports but this role is not listed. Is there a problem in my tenant or is this documentation wrong, if so, please correct

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

ID: 545a1ae6-3a3e-8c58-a40e-0a0ae08f0017
Version Independent ID: ec806746-ff82-ccbf-1414-c4e1ccc5fd7b
Content: Administrator analytics and reports for Microsoft Power Automate - Power Platform
Content Source: power-platform/admin/analytics-flow.md
Service: power-platform
Sub-service: admin
GitHub Login: @MSFTMan
Microsoft Alias: deonhe

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


Doc needs to make clear if css prop will be recreated on every render pass

The doc says Define Styled Components outside of the render method which makes sense but does it mean that CSS props in components trigger a recreation with every render as well? If yes we need to add this to css prop.

No. CSS props get transformed at build-time to static styled components, e.g.
const Button = (props) => <button css={`
  color: ${props.primary ? 'papayawhip' : 'palevioletred'};
  background: ${props.primary ? 'palevioletred' : 'papayawhip'};
`} />

// conceptually this gets turned into
const StyledButton = styled.button`
  color: ${props => props.primary ? 'papayawhip' : 'palevioletred'};
  background: ${props => props.primary ? 'palevioletred' : 'papayawhip'};
`

const Button = (props) => <StyledButton primary={props.primary} />

I hope that helps!


Changed formulation of JSSP

Removed the redundant ordering list variable at the price of adding O(N^2) disjunctive constraints between all pairs of job on each machine.

I converted this to a draft to avoid merging. I'm checking one thing I am not sure about..


Created basic web service

Created a basic Calamari web service. The service, which can be run via
./gradlew web:run, has one endpoint, POST /prawn. The endpoint
accepts useSBM and userLinFits as optional URL parameters, but
requires a multipart form with a prawnFile file provided as the
payload. It then responds with a ZIP of reports if the request is
successful.

While there's no UI yet, you can try this out from a Unix command line using curl -X POST -F<EMAIL_ADDRESS>http://<IP_ADDRESS>:9998/prawn > reports.zip—substituting your own local Prawn file—after starting the server.

Excellent.  van Delden says the Academic Resource Network is up and running so I have requested a server ... should be interesting.


0729 test run

test config
redis persistence is disabled (save "")
export NUM_OF_SI=6
export RKV_ROOT_DISK_VOLUME=100
export SI_ROOT_DISK_VOLUME=100
export SI_INSTANCE_TYPE=t2.xlarge
export RKV_INSTANCE_TYPE=t2.2xlarge
export JAEGER_INSTANCE_TYPE=t2.2xlarge
export JAEGER_ROOT_DISK_VOLUME=200
export YCSB_INSTANCE_TYPE=t2.2xlarge
export YCSB_ROOT_DISK_VOLUME=40

records: 5M
k-v payload: 5KB value
test procedure
load test only
workloada setting
threadcount=4
fieldlength=160   #intended 500; due to the encoding, 160 length would yield about 500 payload
recordcount=5000000
operationcount=5000000

test result
ycsb log:
INSERT - Takes(s): 10840.0, Count: 3188140, OPS: 294.1, Avg(us): 13548, Min(us): 4082, Max(us): 1046015, 99th(us): 22239, 99.9th(us): 29471, 99.99th(us): 223231
INSERT - Takes(s): 10850.0, Count: 3191224, OPS: 294.1, Avg(us): 13548, Min(us): 4082, Max(us): 1046015, 99th(us): 22239, 99.9th(us): 29471, 99.99th(us): 223231
INSERT - Takes(s): 10860.0, Count: 3194339, OPS: 294.1, Avg(us): 13547, Min(us): 4082, Max(us): 1046015, 99th(us): 22239, 99.9th(us): 29471, 99.99th(us): 223231
INSERT - Takes(s): 10870.0, Count: 3197327, OPS: 294.1, Avg(us): 13547, Min(us): 4082, Max(us): 1046015, 99th(us): 22223, 99.9th(us): 29471, 99.99th(us): 223231
INSERT - Takes(s): 10880.0, Count: 3200144, OPS: 294.1, Avg(us): 13547, Min(us): 4082, Max(us): 1046015, 99th(us): 22223, 99.9th(us): 29455, 99.99th(us): 223231
... // approaching to 4M records (where redis is almost exhausting its memory), noticing OPS is actually very low (<1 ops at this moment)
INSERT - Takes(s): 13820.0, Count: 3931533, OPS: 284.5, Avg(us): 13969, Min(us): 4082, Max(us): 19005439, 99th(us): 22079, 99.9th(us): 29439, 99.99th(us): 1022463
INSERT - Takes(s): 13830.0, Count: 3931539, OPS: 284.3, Avg(us): 13986, Min(us): 4082, Max(us): 22298623, 99th(us): 22079, 99.9th(us): 29455, 99.99th(us): 1022463
INSERT - Takes(s): 13840.0, Count: 3931541, OPS: 284.1, Avg(us): 13993, Min(us): 4082, Max(us): 22298623, 99th(us): 22079, 99.9th(us): 29455, 99.99th(us): 1022975
INSERT - Takes(s): 13850.0, Count: 3931542, OPS: 283.9, Avg(us): 13999, Min(us): 4082, Max(us): 22790143, 99th(us): 22079, 99.9th(us): 29455, 99.99th(us): 1022975
... // even no ops sometimes
INSERT - Takes(s): 14550.0, Count: 3932125, OPS: 270.2, Avg(us): 14673, Min(us): 4082, Max(us): 33439743, 99th(us): 22111, 99.9th(us): 29855, 99.99th(us): 5308415
INSERT - Takes(s): 14560.0, Count: 3932125, OPS: 270.1, Avg(us): 14673, Min(us): 4082, Max(us): 33439743, 99th(us): 22111, 99.9th(us): 29855, 99.99th(us): 5308415
INSERT - Takes(s): 14570.0, Count: 3932125, OPS: 269.9, Avg(us): 14673, Min(us): 4082, Max(us): 33439743, 99th(us): 22111, 99.9th(us): 29855, 99.99th(us): 5308415
INSERT - Takes(s): 14580.0, Count: 3932125, OPS: 269.7, Avg(us): 14673, Min(us): 4082, Max(us): 33439743, 99th(us): 22111, 99.9th(us): 29855, 99.99th(us): 5308415
...
INSERT - Takes(s): 14640.0, Count: 3932125, OPS: 268.6, Avg(us): 14673, Min(us): 4082, Max(us): 33439743, 99th(us): 22111, 99.9th(us): 29855, 99.99th(us): 5308415
INSERT - Takes(s): 14650.0, Count: 3932126, OPS: 268.4, Avg(us): 14682, Min(us): 4082, Max(us): 33980415, 99th(us): 22111, 99.9th(us): 29855, 99.99th(us): 5320703
INSERT - Takes(s): 14660.0, Count: 3932129, OPS: 268.2, Avg(us): 14708, Min(us): 4082, Max(us): 34570239, 99th(us): 22111, 99.9th(us): 29871, 99.99th(us): 5345279
INSERT - Takes(s): 14670.0, Count: 3932133, OPS: 268.0, Avg(us): 14722, Min(us): 4082, Max(us): 34570239, 99th(us): 22111, 99.9th(us): 29871, 99.99th(us): 5402623
INSERT - Takes(s): 14680.0, Count: 3932136, OPS: 267.9, Avg(us): 14729, Min(us): 4082, Max(us): 34570239, 99th(us): 22111, 99.9th(us): 29871, 99.99th(us): 5423103
...
INSERT - Takes(s): 17050.0, Count: 3933443, OPS: 230.7, Avg(us): 17128, Min(us): 4082, Max(us): 34570239, 99th(us): 22191, 99.9th(us): 31215, 99.99th(us): 12156927
INSERT - Takes(s): 17060.0, Count: 3933451, OPS: 230.6, Avg(us): 17144, Min(us): 4082, Max(us): 34570239, 99th(us): 22191, 99.9th(us): 31231, 99.99th(us): 12230655
INSERT - Takes(s): 17070.0, Count: 3933454, OPS: 230.4, Avg(us): 17155, Min(us): 4082, Max(us): 34570239, 99th(us): 22191, 99.9th(us): 31231, 99.99th(us): 12230655

rkv memory usage when system is about 4M records
              total        used        free      shared  buff/cache   available
Mem:            31G        1.7G         28G        844K        1.5G         29G
Swap:            0B          0B          0B

jaeger cpu and disk usage when system is about 4M recods
               total        used        free      shared  buff/cache   available
Mem:            31Gi       7.8Gi        22Gi       1.0Mi       926Mi        23Gi
Swap:             0B          0B          0B

Filesystem      Size  Used Avail Use% Mounted on
/dev/root       194G  2.4G  192G   2% /

ycsb client cpu and disk usage when system is about 4M recods
               total        used        free      shared  buff/cache   available
Mem:            31Gi       334Mi        30Gi       0.0Ki       788Mi        30Gi
Swap:             0B          0B          0B

Filesystem      Size  Used Avail Use% Mounted on
/dev/root        39G  4.3G   35G  11% /

observations

redis memory usage need to estimate with sufficient buffer beyond 5KB per record. Suggest to use x2 or a bit more.
rkv not exit when redis memory has almost exhausted (though ops is very low should that happens) - disabling redis save should be the help to avoid crash of rkv in that case

closed; milestone has paased.


Multisignature wallet added, and extended with non-native token and proxy functionality

I have added a draft of a multisignature wallet which ought to be able to interact with the proxy contract.
The wallet contract is entirely untested, so use at your own discretion. Also, I have probably made some assumptions along the way, so you should perform extensive testing before using the wallet.

Thanks @jjcnn


Historic R0 estimates plot dates

Time periods in the historic R0 estimates are becoming too crowded.

@damonbayer, a gentle reminder about the dates...


InfluxDB integration with multiple load agents

Let's say I am running K6, the very same script, on multiple load agents at the same time. Is there a way to collect the influxDB measurements of all agents, aggregate it to one so that in Grafana I am able to see the real number of VUs for example?

I guess the problem you are seeing (as you haven't said whether  you tried it or not) is that some metrics (Gauge) are getting only the values of one of the load generators.
This is as k6 doesn't automatically set additional tags (as it doesn't know what htey should be) so that samples from different laod generators can be identified.
You can add --tag generator=2 with different values (and tag name for that matter) so you that influxdb doesn't aggregate them and you can query them all. You likely will need to change some of the dashboards as well.
Hope this helps you!

i did not try it out. I was curious if the aggregation would happen all by itself. I though i i have 3 paralell jobs sending data, they would override each other instead of summing up.....like 10 users on each parallel, I thought I would only see 10 users in Grafana, not 30.
Hope I understood you right. In that case thank you :)


*_url constructors can also throw std::length_error

This needs to be documented.
https://github.com/CPPAlliance/url/issues/203#issuecomment-1158852601

I might have already done this, switched it to std::exception because now the throw comes from result::value().


if the init fails

https://github.com/AmpelProject/Ampel-nuclear/blob/cef592dff6f090628518a1d207a95d7dd6215c82/run_tde_test.py#L88
In my first attempt at
./run_tde_test.py -i
failed because I was using my Ampel API token (and not the archive token). This line the first one that breaks, but the error thrown here make clear it's simply and authorization issue. Shall I add a check for unauthorized in the output from response = requests.post()?

hi Sjoert, yeah, that would be great!


Create ToolsforDevSITAM.md

just a help document

Can we add this as part of the tech guide instead of a separated file?


⚠ ClamAV has degraded performance

In 8507ef1, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 198 ms

Resolved: ClamAV performance has improved in bedd4b9 after 6 minutes.


Standardize file locations for supporting files

Coming from https://github.com/opensearch-project/opensearch-plugins/issues/127.
We should standardize the file location across plugins by implementing following conventions:

All supporting files for plugins should stay within the plugin's own folder: /plugins/my-plugin
Within plugin's folder, we recommend following directory structure:

./config: config files
./bin: executables
./lib: for plugins that use native libraries (e.g. libKNNIndexV2_0_11.so)
./install: scripts called to bootstrap the plugin before first use.
./docs: document files
./certs: certificates for the security plugin
./data: test/demo data

All scripts should come in *nix and .bat flavors, should use OpenSearch's defaults for environment variables like $JAVA_HOME, so users can override these but don't need to specify them by default. Scripts should have 0755 permissions by default.

IM plugin don't have any specified category files in the plugin folder at the moment. Closing the issue


Infinite slide does not work if there only 2 items

This can be reproduced on the demo Codesandbox Playground as well, if the items are reduced to 2, the slide does not go back to the first item once all are rotated.

This is correct. If there are only 2 slides then infinite={true} does not work.

Also "autoplay={2000}" does not work with 2 slides!


follow pep517

Closes https://github.com/Unidata/cftime/issues/148
Don't merge this just yet. I want to refine it a little bit.
Actually any further changes would be better in another PR to avoid confusion. This is good to go.

@jswhit at the moment setup.py covers both options cythonize = False/True. We can limit that to always True if we want with this PR b/c cython will be guaranteed to be present at install time.

Looks good - thanks @ocefpaf !

Great, that was super fast, thanks!


Driver: use a proper error rather than fatalError

fatalError is more akin to an assertion rather than a "condition
failed, please exit".  The improper use here would, upon a typo on
Windows, induce the Doctor to try to categorise the fault and then
emit a minidump.  This is both frustrating and unnecessary.  Add a
new error case and use it to emit the message and exit gracefully.

@swift-ci please test


getopt flag syntax?

Thanks for the awesome library, I've been using it for years.
Just wondering if you're interested in supporting standard GNU getopt() syntax?  This is the one thing I feel this project could benefit from.  At the moment it's just an idea, though I'd be happy to do the work and send a PR if you agree.  A quick google for "golang getopt" shows a number of existing implementations, so it should just be a matter of picking one of those and hooking it in appropriately?
FYI, the end-user impact of this would be as follows.  Say you have the following flag definitions:
cli.IntFlag{
    Name:  "n",
    Usage: "max number of things",
},
cli.BoolFlag{
    Name:  "quiet, q",
    Usage: "Reduces the text output",
},
cli.BoolFlag{
    Name:  "all, a",
    Usage: "show deleted stuff too",
},

then currently the CLI user has to type:
-a -q -n 1

but getopt() allows you to type:
-aqn1

Hey Dan @meatballhat, I opened this issue really just to get some initial feedback.  As I said, I'd be happy to submit a PR if you like the idea.

Hi @boyvinall I'd be happy to review a PR for this, if you're still interested!

Hi @lynncyrin, will see if I get some time but things are fairly busy.  I've thought about this a bit more since opening it though .. I realised that the long flag name form is currently accepted with either a single or double dash (--flag or -flag) which would conflict with this.  So if this issue is resolved then I guess, for backwards compatibility, it would need to be conditionally enabled.  Your thoughts on that?

Isn't this solved on master?

So if this issue is resolved then I guess, for backwards compatibility, it would need to be conditionally enabled. Your thoughts on that?

I believe this is what the UseShortOptionHandling option does? Please correct me if that's wrong!

This has been sitting idle for a while, so I'll circle back and close it soon - unless someone wants to keep it open!

Sorry for going quiet. Yeah, I hadn't noticed UseShortOptionHandling, that looks like the thing, thanks.


[KYUUBI #5165][K8S][SPARK] Build Spark Driver/Executor Pod Name(Prefix) in process

Why are the changes needed?

Print those pod name/prefix in kyuubi server log
Make sure driver pod name in length limit

close #5165
How was this patch tested?

[x] Add some test cases that check the changes thoroughly including negative and positive cases if possible

[ ] Add screenshots for manual tests if appropriate

[ ] Run test locally before make a pull request

Codecov Report

Merging #5168 (290a468) into master (14d0dab) will not change coverage.
Report is 6 commits behind head on master.
The diff coverage is 0.00%.

@@          Coverage Diff           @@
##           master   #5168   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         566     566
  Lines       31654   31699   +45
  Branches     4124    4134   +10
======================================
- Misses      31654   31699   +45

Files Changed
Coverage Δ

...kyuubi/engine/spark/SparkBatchProcessBuilder.scala
0.00% <0.00%> (ø)

...ache/kyuubi/engine/spark/SparkProcessBuilder.scala
0.00% <0.00%> (ø)

...scala/org/apache/kyuubi/util/KubernetesUtils.scala
0.00% <0.00%> (ø)

... and 4 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Provide a Timmy::VERSION constant

So that users can anticipate library's evolution and handle breaking changes smoothly.

Good idea! Added in https://github.com/mindkomm/timmy/releases/tag/0.14.9 and https://github.com/mindkomm/timmy/releases/tag/1.0.0-beta.2.


[10.x] Add Number::clamp

This PR extends the Number class to introduce a clamp method, that clamps a given number between two numbers.
Originally from https://github.com/jbrooksuk/php-clamp, but with the Number class, it makes sense to bring it in.

Was just chatting about this today, as we added a Macro to the number class for converting a dollar value to a number of cents (1.00 => 100).
But from this original tweet, it seems like the facade was designed to be used for numeric -> string.
So the question was "are we misusing the facade if we take in a number and return a number?" but if this gets accepted, the conversation is moot. 😆

But from this original tweet, it seems like the facade was designed to be used for numeric -> string.

I'd read the tweet that "with" is not exclusive, but just happens to be number -> string at the moment. It's a simple method that does have practical uses though, so it's up to Taylor 😄

But from this original tweet, it seems like the facade was designed to be used for numeric -> string.

I'd read the tweet that "with" is not exclusive, but just happens to be number -> string at the moment. It's a simple method that does have practical uses though, so it's up to Taylor 😄

I'm happy either way, just thought it was topical considering the discussion happened this AM.

Should we throw an exception in case the given min is greater than the max value like I did here?
https://www.amitmerchant.com/a-simple-clamp-function-in-php/


decoding Error while loading up the answer_bot.py

im pretty new to python...and all this
so how to fix this??? im getting this error..
ive already googled the error and didnt find any working solution
C:\Users\CIE\Desktop\BOT BUILD\HQ_Bot-master>python3 answer_bot.py
Traceback (most recent call last):
File "answer_bot.py", line 284, in
load_json()
File "answer_bot.py", line 64, in load_json
sample_questions = json.loads(open("Data/questions.json").read())
File "C:\Users\CIE\AppData\Local\Programs\Python\Python36\lib\encodings\cp1252.py", line 23, in decode
return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d in position 93: character maps to

Got the fix on previous issues.. thanks


Pixart - Jordan Brauner's Solution

Comfort: 4
Completeness: 5

Great job with this assignment Jordan. Sounds like you've got a good grasp on all of this. Let me know if you have any questions on anything covered here.


fix kindlingcfg file(#574)

Description

Related Issue

Motivation and Context

How Has This Been Tested?

Thank you for your first contribution!


Update README.md for v2.5.0

Changes:

Update Quick Start.
Remove "Disclaimers" section.
Update "Acknowledgments" section.
OpenSSF Scorecard badge link goes to README section on fuzzing.

@x448 please take a look when you have a moment. :pray:

@x448

Do you want to update Status or doc.go?

Good point.

Updated "Status" section.

Maybe not enough time to update doc.go today.

I want to release tag v2.5.0 today (Sunday, Aug 13).
Can you update benchmarks in the "Status" section today?  If not, I can do that.  This is not a blocker because it needs click to view.


Leaderboard animation

As a player I want to have animated leaderboards so that I can clearly see the changes that occur in the leadersboards.
Acceptance criteria:
1. When a player's ranking changes, its clearly shown by either zoom-in/zoom-out or color change.
2. When swiping, its clearly shown by either zoom-in/zoom-out or color change.
As a player I want to see the result at the end of the race so that I know my ranking and the time of the race.
Acceptance criteria:

The ranking, name and the time is shown at the end of the race with some basic animations.

changed the user story

This animation will be displayed once the player finishes the game


Sharding stops working post node restart

Sharding stops working  post non seed node restart.Only way to recover the shard is to restart seed node. Any specific setting which handles such scenario?

Thanks for contributing to the project :) I will try to reproduce this by killing the non seed node and will get back to you with the answer.

@mohanpriyadarshi I tried to reproduce the problem but have not succeeded. What I am doing is:

start seed node on port 2551
start non seed node on port 2552
send several requests for different junctions
kill non seed node
start new non seed node on port 2552 and wait for them to reconnect and stabilise
send several requests for different junctions and this works exactly like in 3..

Can you elaborate and write the exact scenario? Thanks a lot!


RAM Usage, it seems each call is not closed, multiple TCP connections open

Hi, I have found out there is some issue with unifi-request node (after updating UDMPro to latest version), which causes high RAM usage after some time (12hrs?).
I have this simple flow where which can be seen on screenshot below. Basicaly it GETs, JSON data from cameras (3x), every 2 seconds (motion detection). I did a bit of research and found out that it seems each call of this node is maybe not being closed - meaning there is a huge amout of opened TCP connections after some time. Please see NETSAT output to get an idea. I think this is root cause.
In the past before UDM update it was allright, but now I had to disable it because it lagged whole NR after some time (high RAM, high CPU).
System info:
RPi4 2GB
UDM Pro v3.1.15
Node-red 2.2.2.
node-red-contrib-unifi-os 0.9.0
Please could you look into it? Thanks & keep up the good work 🥇

@marcus-j-davies have you seen this one? Are you bored this weekend?


Inquiry Regarding Automation of Mesh Creation with pyfluent

Dear pyfluent Development Team,
My name is Jay Son, and I reside in South Korea.
I am writing to inquire about PyFluent.
I believe one of the primary benefits of using pyfluent is its potential for automation.
Specifically, I am interested in automating the process of mesh creation, which is currently a repetitive task.
From what I have seen, the examples provided in pyfluent mainly illustrate how to work with already created meshes.
However, my goal is to create .msh or .cas files using Python alone, including pymapdl.
Could you please let me know if this is possible?
If so, it would be greatly appreciated if you could add a simple example demonstrating this to the pyfluent documentation.
Thank you for your assistance.
Best regards, Jay Son

@strustar, Please refer to this example and let us know if you have any further queries:
https://fluent.docs.pyansys.com/version/stable/examples/00-fluent/external_compressible_flow.html#sphx-glr-examples-00-fluent-external-compressible-flow-py


[Feature][Api] Refactor org.apache.dolphinscheduler.api.controller.ResourcesController

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
#10257 Suggest refactor the backend api
Refactor  org.apache.dolphinscheduler.api.controller.ResourcesController
Use case

rename ResourcesController to ResourcesV2Controller ,requestmap to /v2/xxxxx
replace The input and output parameters if map or json to POJO
define explicit error info or message
supplement Api doc if have enumeration ,sample data is required for each parameter

Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

@SbloodyS  I see some   TODO  feature like
//todo verify the directory name

Do we already have some request param verification frameworks like  hibernate validator。Or we need to develop a Specialized  for dolphinscheduler ？

Currently, there is no framework. And it is judged in the service layer. @liguotian

I see that there are many functions inside the taskinstancecontroller,If I participate in the TaskInstanceController task, do I only need to modify the queryTaskListPaging forceTaskSuccess functions in TaskInstanceV2Controller?


Full party list

The bars don't seem to be lining up correctly I tried making the padding more doesn't seem to work. the padding only seems to space the first one and second one apart. While the 2nd through 6 are all stuck to one another.

it also does it with the pet

Try adjusting the padding setting in layout.  If you're not on the latest version, please update to that.


Update install guide to reflect TLS cert host expectations

Services (i.e. dataservice) expect their TLS cert host name be the name of the service separated by a - (e.g. data-service). This will likely change in the future, but for now, this is not documented in the INSTALL.md guide and should be.

Similarly to what I just noted on another issue, we want to prioritize removing roadblocks and snags like this for external community members to get caught on.


Add __init__.py to generated resources folder

Thanks for this library! Just creating this issue, because I'm having some issues with type-checker warnings importing from the generated resources directory. The lack of an __init__.py causes some type-checkers not to recognize it as properly stubbed.

Hi, the __init__.py file as been added as part of the release of the new models for k8s v1.31 lightkube-models==<IP_ADDRESS>

Hi, the __init__.py file as been added as part of the release of the new models for k8s v1.31 lightkube-models==<IP_ADDRESS>

Thanks!


Weird behaviour when long-pressing undo button.

If the user press the undo button until the dialog is dismissed and then releases it first onHide is called because of the automatic dismiss and then onUndo is called because the user released the button (that no longer exists).

Hey jenzz thanks for the amazing demo but the undo bar is not hiding automatically or manually its just stick on my screen


Goon: Key press macros for new text windows don't work

You should be able to press enter to send a message from the dialogue brought up by T. You should also be able to close the window by pressing escape.
It seems like the macros are not being applied?

Right now only the macro set on the main window applies


Add ESLint plugins for Jest, Jest DOM, and Testing Library

Rationale
I was excited to see Testing Library get added to the default template (cra-template), but I noticed the officially recommended ESLint plugins were missing from eslint-config-react-app. They both help to make matcher usage more consistent and fix some bugs where improper usage can causing failing or false positive test results. I also added the Jest plugin which has similar benefits for ordinary Jest usage (with or without testing library packages).
Changes
Added recommend ESLint configs for Jest, Jest DOM, and React Testing Library.
Note on react-scripts
The webpack config used in react-scripts currently only runs ESLint via eslint-loader from the app's entry point. Unfortunately this means that test files will be excluded from the built-in ESLint runner. To make these config additions more useful in react-scripts I attempted to set up jest-runner-eslint, but I'm getting the following error when adding the multiple projects configuration:
Error: Jest: Cannot use configuration as an object without a file path.
Source: https://github.com/nickmccurdy/create-react-app/blob/92075f408baddacde5863fcf9bff6ca9ded1299e/packages/react-scripts/scripts/utils/createJestConfig.js#L68-L80
Help with this error would be appreciated, as it would be a lot more useful to have these ESLint plugins working in react-scripts. Currently they're still useful when using ESLint outside react-scripts, such as through an ESLint extension or the eslint CLI outside of a CRA app.
Testing

Ran eslint manually (outside of react-scripts) and confirmed that linter rules are enabled.
Created an example app with yarn create-react-app my-app (note: this doesn't support linting test files yet, it's just a smoke test to make sure the ESLint config isn't broken).

Thanks for the PR. Can you clarify the thinking with adding this even though we don't run ESLint on the test files? I'm not sure what value this brings to CRA without that.

I have code in progress to do that, but it throws this error when I follow the jest-runner-eslint usage: Error: Jest: Cannot use configuration as an object without a file path.
Source: https://github.com/nickmccurdy/create-react-app/commit/92075f408baddacde5863fcf9bff6ca9ded1299e
I removed this commit from this PR to fix the broken Jest config so that CRA will still work without the test linters, and eslint-config-react-app will work outside CRA with the test linters.

Still should be valid as far as I’m aware, but I’d be happy to take another look if someone can help with this error

I almost open the same PR for setting up some testing-library eslint plugins, glad to have found this issue! I hope it gets merged eventually.

I've updated the dependencies for this pull request. I'm still having the Jest config error with the latest version of jest-runner-eslint, so I'd appreciate help figuring that out (unless we decide to merge before it's fixed).

I'll try to help you, but I'm afraid I'm not an expert in this matter.

@nickmccurdy Sorry I didn't have spare time to check this, but @thomlom (the other amazing collaborator of eslint-plugin-testing-library) mentioned recently this in a chat:

If you use jest-runner-eslint  to lint your files, you have to add the files to testMatch  too

Just sharing here in case it helps you in some way.

I appreciate the effort, but I'm already using testMatch and it's still erroring unfortunately

I've rebased both branches to fix some formatting issues when merging in recent changes. They're now up to date, but the testing-library-linting-integration branch still has the Jest error mentioned above.

Thanks for the PR. Can you clarify the thinking with adding this even though we don't run ESLint on the test files? I'm not sure what value this brings to CRA without that.

@ianschmitz I'd say one big benefit is that VS Code (and any other IDE) would provide linting in your IDE.

@kentcdodds recommends these, and I've been using them at work. That being said, they're also very easy for users to add in.

They are pretty easy for users to add, but the people who benefit from them most are the people who don't know they exist.

I think we should get this one in personally, as there is value for beginners, and advanced users can opt out if they wish (by replacing the ESLint config).
It looks like this is related: #8963.

I think we should get this one in personally

Agreed.

Merge conflicts fixed, ready to merge

Closing in favor of #8963 which also supports ESLint 7


How to apply access policies to class based views

I would like to apply the class-based views, not to ViewSets or function-based views.
For example I would like to use the access policies to Views derived from rest_framework.generics, such as RetrieveAPIView.
I managed to apply the access policy when using "action": ["*"] for the RetrieveAPIView view, but if I use "action": ["list", "retrieve", "get"] it does not work, so I don't know how to use finely-tuned access policies in this context.
Thanks in advance for your help :)

I am also wondering why it doesn't works CBV


CB-3254 Adding GCP support for dataengineering cluster creation

We have not fixed the minimum version for GCP. So just added 7.2.2 and 7.2.3 templates for testing.
Tested via dp cli.

@cegganesh84 similar like https://github.com/hortonworks/cloudbreak/pull/9063

Thanks, @doktoric, I will discard this in favor of #9063


⚠ Bofrost.be has degraded performance

In e11437b, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1072 ms

Resolved: Bofrost.be performance has improved in 15b83aa after 14 minutes.


Bug in filterable ComboBox: "undefined" instead of custom label

The filterable ComboBox using shouldFilterItem doesn't fully support the custom label set by itemToString when displaying the selected item in the input field and displays undefined instead:

I traced it down to lines like the following (there are multiple occurrences):
https://github.com/carbon-design-system/carbon-components-svelte/blob/f8e400f2edd971f41dacfbc21bfff4ee059e5163/src/ComboBox/ComboBox.svelte#L297
In my opinion, this line should make use of the itemToString, e.g., value = itemToString(filteredItems[highlightedIndex]); similar to this line:
https://github.com/carbon-design-system/carbon-components-svelte/blob/f8e400f2edd971f41dacfbc21bfff4ee059e5163/src/ComboBox/ComboBox.svelte#L394
Should I create a pull request for this?

Thanks for flagging – PRs welcome.

Hi @metonym! I created a PR and linked it to this issue. Please let me know if this works for you or if I can be of any more assistance.

Fixed in v0.67.2.


possible typo

The header says C2372.
Just before the example code it says "this will generate C2326."
Then in the code it goes back to C2372.

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

ID: f52cebe4-7649-c7d6-6deb-935ccff75b5f
Version Independent ID: 2dca4016-c72f-1283-f8c4-a1a0cc64c226
Content: Compiler Error C2372
Content Source: docs/error-messages/compiler-errors-1/compiler-error-c2372.md
Product: visual-cpp
Technology: cpp-diagnostics
GitHub Login: @corob-msft
Microsoft Alias: corob

@LaurenENye Fixed, thanks for the report.


[Bug - next-pwa]: Apk opens in browser mode

Provide environment information
"next": "^14.2.0",
"@ducanh2912/next-pwa": "^10.2.6",

Link to reproduction - Issues with a link to complete (but minimal) reproduction code help us address them faster
https://github.com/OrelsProjects/PinkyPartner
To reproduce
Build the APK and try to install it @ https://www.pinkypartner.com
Describe the bug
When I create an apk (also uploaded to Play Store) via pwabuilder and install it on my phone, it opens with the browser navigation bar.
Expected behavior
The app should open without the navigation bar
Screenshots (if relevant)
No response
Additional information (if relevant)
Manifest:
{
  "name": "PinkyPartner",
  "id": "com.orelsprojects.pinkypartner",
  "short_name": "PinkyPartner",
  "theme_color": "#ffffff",
  "background_color": "#FFFFFF",
  "display": "standalone",
  "display_override": [
    "standalone",
    "window-controls-overlay"
  ],
  "description": "PinkyPartner is a habits building app that let's you build them with a partner.",
  "orientation": "portrait",
  "start_url": "https://www.pinkypartner.com/home",
  "dir": "auto",
  "scope": "/",
  "icons": [
    {
      "src": "windows11/SmallTile.scale-100.png",
      "sizes": "71x71"
    },
    {
      "src": "windows11/SmallTile.scale-125.png",
      "sizes": "89x89"
    },
 ....
  ],
  "lang": "en",
  "categories": [
    "entertainment",
    "productivity"
  ]
}

Next config:
// @ts-check
import * as PWA from "@ducanh2912/next-pwa";

/** @type {import('next').NextConfig} */
const nextConfig = {
  headers: async () => [
    {
      source: "/sw.js",
      headers: [
        {
          key: "Service-Worker-Allowed",
          value: "/",
        },
      ],
    },
  ],
  images: {
    unoptimized: true,
    remotePatterns: [
      {
        protocol: "https",
        hostname: "storage.googleapis.com",
      },
      {
        protocol: "https",
        hostname: "lh3.googleusercontent.com",
      },
    ],
  },
};

const withPWA = PWA.default({
  dest: "public",
  cacheOnFrontEndNav: true,
  aggressiveFrontEndNavCaching: true,
});

export default withPWA(nextConfig);

Thanks in advance!

This is actually out of the package's scope and is more related to your web app manifest.
Nevertheless, here is what PWABuilder has to say about the matter. Hope this helps!


How can I change the priority in the Update get request?

Hello
When i run the automatic update, its telling me, that i have already the latest version. But thats not true, because when i run the update via the browser itself, it would update.
So i compared the two different requests in the UpdateLog and i noticed the different priorities:
ManualUpdate (has "Priority":10
[Send][url=https://msedge.api.cdp.microsoft.com/api/v1.1/contents/Browser/namespaces/Default/names/msedge-stable-win-x64/versions/latest?action=select][request={"targetingAttributes":{"AppAp":"stable-arch_x64","AppBrandCode":"GCEU","AppCohort":"rrf@0.80","AppCohortHint":"","AppCohortName":"","AppLang":"","AppMajorVersion":"87","AppRollout":0.8,"AppTargetVersionPrefix":"","AppVersion":"87.0.664.60","HardwareDiskType":2,"IsInternalUser":false,"IsMachine":true,"OemProductManufacturer":"LENOVO","OemProductName":"20JJS3JP00","OsArch":"x64","OsPlatform":"win","OsVersion":"10.0.17763.1637","Priority":10,"Updater":"MicrosoftEdgeUpdate","UpdaterVersion":"<IP_ADDRESS>"}}][filename=]
AutoUpdate (has "Priority":0)
[Send][url=https://msedge.api.cdp.microsoft.com/api/v1.1/contents/Browser/namespaces/Default/names/msedge-stable-win-x64/versions/latest?action=select][request={"targetingAttributes":{"AppAp":"stable-arch_x64","AppBrandCode":"GCEU","AppCohort":"rrf@0.80","AppCohortHint":"","AppCohortName":"","AppLang":"","AppMajorVersion":"87","AppRollout":0.8,"AppTargetVersionPrefix":"","AppVersion":"87.0.664.60","HardwareDiskType":2,"IsInternalUser":false,"IsMachine":true,"OemProductManufacturer":"LENOVO","OemProductName":"20JJS3JP00","OsArch":"x64","OsPlatform":"win","OsVersion":"10.0.17763.1637","Priority":0,"Updater":"MicrosoftEdgeUpdate","UpdaterVersion":"<IP_ADDRESS>"}}][filename=]
So now im wondering, how can i call the updater with different priority?
Thanks

@elinside Hello!  Would you be able to submit this question and diagnostic data through our in-browser feedback tool? It's under "..." menu > Help and feedback > Send feedback.
Also, if you have further questions regarding Microsoft Edge please reach out on the Microsoft Edge Tech Community Forum or for Enterprise specific questions please visit the Enterprise Enterprise section of the Tech Community. Thanks!

@elinside Hello!  Would you be able to submit this question and diagnostic data through our in-browser feedback tool? It's under "..." menu > Help and feedback > Send feedback.
Also, if you have further questions regarding Microsoft Edge please reach out on the Microsoft Edge Tech Community Forum or for Enterprise specific questions please visit the Enterprise Enterprise section of the Tech Community. Thanks!


Multiple wake words possible?

Would it be possible to have multiple wake words on the same device?

Yes, it is possible with two different approaches:

The micro_wake_word component in ESPHome runs two models simultaneously. I have implemented this, but the code needs some clean up. I'm hoping to submit a PR to ESPHome in the next week. You should be able to run two of the current models at the same time, but then the ESP32-S3 is basically at max capacity in terms of CPU usage.
Train a single model with multiple wake word probability outputs. I also have this roughly working, but I haven't achieved results that are comparable to single word models (which is to be expected). I want to explore this more in the future with a different model architecture, but it requires some significant changes to the current training process. Currently, all the metrics and related calculations assume a single output indicating the wake word or not.

Approach 1 would be sufficient for my current needs, although as you say it is not ideal.
I assume that with approach 2 it would be possible to know which wakeword was triggered?

I've made a draft PR for the ESPHome component that supports running two models at once, so I'm closing this issue.


How can I change the background color of markdown code blocks?

Is it possible to change the appearance of the background color of markdown code blocks? I would like to be able to change the appearance of the parsed markdown.

OK I just embarrassed myself, there was an issue discussing link colors that tipped off the answer for me, here it is for anyone else.
markdownParser.code.textBackgroundColor


[GUI-182] Prometheus' address doesn't get changed.

Start command/request
Start Darzee's previously launched docker container and launch Mongoose.
Expected behaviour
Expect to see a correct Mongoose launch.
Observed behaviour
New Mongoose run doesn't get loaded. This is probably because of incorrect loading from browser's local storage.
Related JIRA's issue: https://mongoose-issues.atlassian.net/projects/GUI/issues/GUI-182

The use-case was:

Launch Darzee (via Docker / Kubernetes / in development mode);
Set new Prometheus address. Wait until it validates;
-- here, the page successfully reloads with records constructed from the Prometheus' data --
Reload the page.
Get an error that Prometheus is not available on its original address, not the updated one.


% Done is not included in the list

{"issue":{"subject":"Test Taskzdmct5ab.g1k71557968-980e-4e1a-b4e6-2ddf02d7b21d","description":null,"notes":null,"is_private":false,"project_id":1387,"priority_id":2,"status_id":1,"tracker_id":1,"estimated_hours":"","parent_issue_id":null,"start_date":"","due_date":"","updated_on":"","done_ratio":""}}
done_ratio is empty? but should be '0'

Solved.

On 26 Oct 2015, at 12:42, Antony Novitsky<EMAIL_ADDRESS>wrote:
{"issue":{"subject":"Test Taskzdmct5ab.g1k71557968-980e-4e1a-b4e6-2ddf02d7b21d","description":null,"notes":null,"is_private":false,"project_id":1387,"priority_id":2,"status_id":1,"tracker_id":1,"estimated_hours":"","parent_issue_id":null,"start_date":"","due_date":"","updated_on":"","done_ratio":""}}
done_ratio is empty? but should be '0'
—
Reply to this email directly or view it on GitHub https://github.com/zapadi/redmine-net-api/issues/126.


Write first component of github-profiles-automator charm

Context
This task includes writing the first actual part of the github-profiles-automator charm. This follows https://github.com/canonical/bundle-kubeflow/issues/1150, as it is the implementation on the spec's information. The charm will utilize chisme helper to ease the development process.
This will probably be done in a development branch.
What needs to get done

Use the spec to develop the first compoment.

Definition of Done

There is a working part of the final charm on a development branch, with the necessary tests

Added https://github.com/canonical/charmed-kubeflow-chisme/pull/130 that adds additional events to the PebbleServiceComponent


How can I share local storage, IndexedDb, Session storage between BrowserWindows ?

Describe the bug
I haven't been able to figure out how to share the data in the various storage mechanisms between the main window and new browser windows I create from clicking on tray menu options. I have tried using "parent", "session" and "partition" on the BrowserWindow Objects I create. For example:
const trayWindow: BrowserWindow = new BrowserWindow({
            backgroundColor: '#FFF',
            width: 300,
            height: 150,
            show: false,
            frame: false,
            fullscreenable: false,
            resizable: true,
            //parent: this.mainWindow,
            webPreferences: {
                //partition: 'persist:myapp',
                session: mySession,  // tried with/without
                nativeWindowOpen: true, //tried both true/false
                devTools: isDevMode,
                enableRemoteModule: true,
                nodeIntegration: true, //tried both true/false
                preload: path.join(app.getAppPath(), "preloader.js"),

             }
        });

Where mySession is an object set as the session object in the main window's config or retrieved through
this.mainWindow.webContents.session. Neither way seemed to help.
Using partition: 'persist:myapp',  gives this error, but because setting session doesn't help I don't think that would help anyway:
(node:74976) UnhandledPromiseRejectionWarning: Error: ERR_FAILED (-2) loading 'capacitor-electron://-'     at rejectAndCleanup (electron/js2c/browser_init.js:205:1493)     at Object.stopLoadingListener (electron/js2c/browser_init.js:205:1868)     at Object.emit (events.js:315:20) (node:74976) 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:74976) [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.
To Reproduce
Steps to reproduce the behavior:
Create an electron-capacitor app and main window:
  const mySession: Session = session.fromPartition("persist:myapp");
const config: CapacitorElectronConfig = this.createCapacitorElectronConfig(mySession);
       this.capacitorApp = createCapacitorElectronApp(config);
       this.capacitorApp.init();

    private createCapacitorElectronConfig(mySession: Session): CapacitorElectronConfig {

        const menuItems: MenuItem[] = this.makeMenuItems();

        //https://capacitor-community.github.io/electron/#/./config-options/index
        const config: CapacitorElectronConfig = {
            trayMenu: {
                useTrayMenu: true,
                trayIconPath: TrayGenerator.makeIconPath(),
               trayContextMenu: menuItems
            },

            mainWindow: {
                windowOptions: {
                    height: 920,
                    width: 1600,
                    show: true,
                    webPreferences: {
                        session: mySession,
                        //partition: 'persist:myapp',
                        nativeWindowOpen: true,
                        enableRemoteModule: true,
                        nodeIntegration: true,
                        },
                }
            }
        };
        return config;
    }

I tried making the Tray through the above config as well as making the tray object separately.
Create another window as above "trayWindow" is created and load a url :
const loadPromise: Promise<void> = trayWindow.loadURL(trayUrl);
Expected behavior
The main window's local  storage, session storage and indexedDb storage is shared with other windows created in the app.
Screenshots
Notice the main window uses the  capacitor-electron:// schema in the following screenshot:

However the other browser windows created use the file:// schema:

The IndexedDb is different regardless.
Desktop (please complete the following information):

OS: MacOs High Sierra 10.13.6
Browser: Electron
Version

"electron": "^11.0.1",
   "@capacitor-community/electron": "^1.3.1"

Additional context
I am not writing to IndexedDb directly, but am rather using AngularFire and AngularFireAuth in the renderer process, so I can't grab the data in the main process and pass it to the various windows without significant alterations. I don't want to login again in a new browser window when I open a window from the tray if I have already logged in to the main window. The login info is stored in IndexedDB, as show in the screenshots.
This seems to be related to this stackoverflow question :  sharing indexdDb
However because of the different schemas ("capacitor-electron://" vs "file://" )in my use case I am posting it here as well.

Do you need this second window to be loaded with the file scheme or could it be done with the capacitor-electron scheme?
If the latter is true, sounds like I need to add a CreateWindow type method for people to use.

Do you need this second window to be loaded with the file scheme or could it be done with the capacitor-electron scheme?
If the latter is true, sounds like I need to add a CreateWindow type method for people to use.

I don't have a preference for the scheme, I was just pointing out that one window uses one file and another uses capacitor-electron.

I don't have a preference for the scheme, I was just pointing out that one window uses one file and another uses capacitor-electron.

Not sure if a new method is needed. I think using "partition" or "session" in the webPreferences section should work to share session info, but it doesn't seem to when the config is passed to createCapacitorElectronApp .

Not sure if a new method is needed. I think using "partition" or "session" in the webPreferences section should work to share session info, but it doesn't seem to when the config is passed to createCapacitorElectronApp .


MC estimators for ground truth integrals of models in quadrature tests now use same setup as tests

Issue #, if available:
Description of changes:
The 2 outputs of the integrate method of the warped BQ models in the quadrature package are tested by comparing to MC estimators. The MC estimators are pre-computed to speed up tests. So far, the two implementations (tests and MC estimators) were completely separate which could lead to bugs as different models could be used. Now the script computing the MC estimators uses the same setup as the tests, hence when it is re-run it should produce valid estimators and confidence intervals. It's still not super pretty but better than before.
Additionally this PR parameterizes those tests which makes it easier to add new models to the tests.
Test themselves are not changed.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

I'll merge this again @apaleyes . There are no functional changes in this PR. Just structuring the tests more robustly and more readable.


Replace PushBullet with Web Push Notifications

It makes sense to remove the PushBullet notification system as web push notifications make more sense now as we now have a web UI, they can be configured by just notifications: 'on' / 'off' in profile.json and we don't really have the requirement for cross device notifications at this point.

@tanay1337 -- Can I tackle this issue? May need some assistance, but I'd love to tackle this.

@KyleRConway Sounds good, this is all yours.

@tanay1337 -- I've been reading through and believe I understand what needs to come out. I was having trouble installing on Fedora, so I worked on setting up in Ubuntu. I have the application running, but voice commands aren't working. Just want to make sure I have a working version going so I know if I've broken something.
Is my profile.json lacking needed credentials? I've given the browser access to my microphone and can see the input on the OS level is working. While reading the wiki I noticed that I don't have a "data/snowboy_resources/Melissa.pmdl" folder or Melissa.pmdl file. Am I missing a step?
Any assistance appreciated! I'll probably update the wiki where appropriate based on your responses.
Thanks!

@KyleRConway Many thanks for tackling this. Are you opening the web UI in Chrome?

@tanay1337 Update -- it turns out this works fine on Chrome, but not Chromium or Firefox (which I was using previously). Is Chrome required?
Thanks!

@KyleRConway Yeah, Chrome is required as we are using Chrome's WebSpeech API and it doesn't work that well on other browsers yet.


Mouse scroll when quickedit disabled

Windows Terminal version (or Windows build number)
1.9.1445.0
Other Software
Yori (but only as a means to disable quickedit and call ReadConsoleInput, you might have something better)
Steps to reproduce

Ensure quickedit is disabled, so mouse events are processed by the application.
Attempt to scroll the viewport with the mouse wheel.

Expected Behavior
In conhost, using the mouse wheel when quickedit is disabled still moves the viewport transparently to the application.
This means that any TUI application wanting to implement mouse wheel logic itself is essentially forced to use CreateConsoleScreenBuffer/SetConsoleActiveScreenBuffer to ensure that the viewport has nothing to scroll.
Actual Behavior
In Terminal, using the mouse wheel when quickedit is disabled notifies the program and does not move the viewport.
This means that any CLI application is forced to handle mouse wheel notifications and call SetConsoleWindowInfo as needed.  I don't even know if this works on Terminal (the documentation suggests it might not.)

As I understand it, this is how conhost should work too. ENABLE_MOUSE_INPUT is a default mode; disabling quick edit without disabling mouse mode should result in mouse events being inserted into the input queue.
At the end of the day, we had to settle on a heuristic for determining whether an application actually wanted mouse input or not.
The things you have found with regards to terminal passing mouse input versus handling it locally (i.e. scrolling the viewport) are likely bugs, not the result of a stance the console team is taking. We are certainly not asking a TUI to implement scrolling on its own, unless it has a proclivity for doing so.
Just to confirm: Yori disables quick edit but leaves mouse input on? And the traditional console still scrolls? Highly unexpected. Whenever are we supposed to generate MOUSE_WHEELED events?

Yori disables quick edit but leaves mouse input on?

Yes, intentionally.  It's trying to handle other forms of mouse input (not wheel events specifically.)

And the traditional console still scrolls? Highly unexpected.

Yes.

Whenever are we supposed to generate MOUSE_WHEELED events?

Those are generated too AFAIK.  That's why I'm saying that a TUI application has to use CreateConsoleScreenBuffer/SetConsoleActiveScreenBuffer to ensure that it only gets MOUSE_WHEELED events without also triggering conhost scrolling.
I understand that the traditional conhost behavior is a little counterintuitive.  This really feels like a case of "no good answer", because depending on the class of application it either has to explicitly suppress viewport scrolling or explicitly facilitate it.  It's almost like ReadConsoleInput needs a DefWindowProc ;)

And the traditional console still scrolls? Highly unexpected.
Yes.

Whenever are we supposed to generate MOUSE_WHEELED events?
Those are generated too AFAIK.

You know, I actually can't recapitulate this behavior. I have a program that prints an infinite list of mouse events as they're generated, and when I wheel the viewport definitely doesn't move. I may be missing something...

Ah, perhaps this is the "snap to bottom on new line" behavior confounding.

There's no x11 mouse mode that matches this behavior, so we're going to have to get very creative somewhat quickly.

I don't understand this comment.  Do you mean there's no VT description? x11 should have no idea about the command line program, so it sends the events to the terminal, and the terminal decides how to update its own display and/or what to send to the command line program, no?
Is there anything I can do here that would be helpful?
Note that I didn't mean to file this to say "this behavior is horribly broken"; I meant to say "I'm trying out this mouse support and noticed that it diverges from conhost here, please confirm if this is intentional."  I doubt many users will be affected by this, because mouse driven command line programs are most likely TUI programs that really don't want the viewport to scroll.

I wrote up a whole reply explaining the guiding principles behind conpty before I re-read your question, so I've collapsed that reply below.
I said "x11" instead of "xterm", as xterm is the terminal that specified the mouse modes in VT :smile:

collapsed reply about COnPTY
Sorry, what I mean is:
Terminal must only communicate with conhost using VT, as it uses ConPTY. Other third-party terminals must do the same. This is unequivocal. This is also why there's no backdoor from WT to the client application such that we could know its console modes and act differently¹². :smile:
Therefore, we have to translate a request for ENABLE_MOUSE_MODE on the input handle into a request from a standards-conforming terminal emulator that it enable VT mouse mode.
Now, mouse mode is specified in VT (by xterm) with a number of different modes (i.e. what sorts of events are requested) and encodings. That set of modes doesn't include one that matches this behavior "the terminal sends scrolling events and also scrolls its viewport" :smile:
¹If we communicate that state across ConPTY to Terminal, we're also asking 3rd party developers who are working on terminals for Windows to answer our secret handshake.
²It is also the root cause of us steadfastly not supporting a scrollback buffer on the console app side.

Okay, I think I'm full of it.  It's not that conhost sends mouse events to the app and implemented wheel itself; I wrote code that scrolled the viewport and completely forgot that it existed.
So now I think the issue is that SetConsoleWindowInfo() does scroll the viewport on conhost but not on WT.  That's probably intentional but it's more concrete and less bonkers than my earlier description.

I've tracked down multiple Clink problems to GetConsoleScreenBufferInfo() reporting inaccurate size information.
With Microsoft Terminal, GetConsoleScreenBufferInfo() essentially tells the console app that there is no scrollback buffer, because srWindow == dwSize.  That is very different from normal conhost behavior.
This makes it seemingly impossible for a console program to interact with the scrollback history.
Clink is unable to scroll the window, so none of its scrolling commands or hotkeys work.
Clink is unable to retrieve text or colors from the scrollback history, so none of its "retrieve text/colors" commands work.
I have scripts bound to F7 and F8 in Clink, and they normally scroll through compiler errors in the scrollback buffer.  But they don't work in Microsoft Terminal, because it hides the scrollback buffer from console applications.

csbiInfo

_CONSOLE_SCREEN_BUFFER_INFO

dwSize

_COORD

X
0x0078
short

Y
0x0032
short

dwCursorPosition

_COORD

X
0x0000
short

Y
0x0031
short

wAttributes

0x0007
unsigned short

srWindow

_SMALL_RECT

Left
0x0000
short

Top
0x0000
short

Right
0x0077
short

Bottom
0x0031
short

dwMaximumWindowSize

_COORD

X
0x0078
short

Y
0x0032
short

There have been some related discussions elsewhere on the repo:

#9018
#9622


Flatpak instructions incorrect on Ubuntu 18.04

I don't have a lot of experience with flatpak, and have done no real configuration to it, so this may be a locally scoped problem.
After running flatpak remote-add --if-not-exists flathub https://flathub.org/repo/flathub.flatpakrepo ,
flatpak install org.shadered.SHADERed
Fails with:
"error: REMOTE and REF must be specified"
while
flatpak install flathub org.shadered.SHADERed
installs SHADERed successfully.

Thanks for reporting. I am not very experienced with the flatpak myself but this has been fixed in one of the pull requests.


fix/rehydrate error

perf: ini ncu upgrade
chore: update

Codecov Report

Merging #97 (63ffb83) into main (8a2839a) will increase coverage by 0.92%.
The diff coverage is 0.00%.

:exclamation: Current head 63ffb83 differs from pull request most recent head 035ec61. Consider uploading reports for the commit 035ec61 to get more accurate results

@@           Coverage Diff            @@
##            main     #97      +/-   ##
========================================
+ Coverage   3.47%   4.39%   +0.92%
========================================
  Files        604     653      +49
  Lines       5041    5461     +420
  Branches    1077    1131      +54
========================================
+ Hits         175     240      +65
- Misses      4699    5045     +346
- Partials     167     176       +9

Impacted Files
Coverage Δ

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

apps/app/src/components/AssetDetailsOAT/index.ts
0.00% <0.00%> (ø)

.../AssetDetailsOATLoading/AssetDetailsOATLoading.tsx
0.00% <0.00%> (ø)

...app/src/components/AssetDetailsOATLoading/index.ts
0.00% <0.00%> (ø)

apps/app/src/components/AssetOAT/AssetOAT.tsx
0.00% <0.00%> (ø)

apps/app/src/components/AssetOAT/index.ts
0.00% <0.00%> (ø)

apps/app/src/components/BaseOAT/BaseOAT.tsx
0.00% <0.00%> (ø)

...onents/ProfileBoardItemOAT/ProfileBoardItemOAT.tsx
0.00% <ø> (ø)

.../ProfileBoardSectionOAT/ProfileBoardSectionOAT.tsx
0.00% <ø> (ø)

apps/app/src/config/SwrKeys.ts
66.66% <ø> (ø)

... and 46 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Implement MIDI Control Change

This resloves #21
Hi @Engil, this is what I have come up with

LGTM, merging! Thanks a lot!


[5.3] Maillable - Access to Swift Object

Sometimes, we need access to Swift Object to set some under-hood features, like
a particular Header or set Return Path.
On old style, we can do that, but with mailable, I can't , or I cannot find how to do that.
Mail::plain('myview', [] , function(\Illuminate\Mail\Message $message) {
$message->returnPath('bounce@mail.com');
$message->getSwiftMessage()->getHeaders()->addTextHeader('X-CUSTOM-HEADER,'123);
});

Forget..
$this->withSwiftMessage(function(\Swift_Message $e){
$e->setReturnPath();....
});


Add status CANCELED

Fixes #547.

Description

This pull request: adds status CANCELED, which is displayed in the web UI when the user cancels a build.

Pull request checklist

[x] Did you test this change locally?
[ ] Did you update the documentation (if required)?
[x] Did you add/update relevant tests for this change (if required)?

Additional information

How to test

Run via docker
Cancel a build via the admin UI by clicking on the cancel button
The status of the build should say "CANCELED".

@gabalafou I have not verified if this is working as expected, could you check please?


Added OpenShift deploy option

Don't merge. I directly committed the changes to the master branch.


Publish on npm

First of all, thank you for this great and useful library!
Obviously pure-form is not published yet on npm: https://www.npmjs.com/package/pure-form
@john-doherty could you publish this package on npm in order to make it easier to manage as a dependency? I believe it is better if the actual developer is also the maintainer of the npm package. Most npm packages are not up-to-date when this is not the case.

Thanks @brrd, it's now on npm :)


Setup runtime.slice for containerd/kubelet

This PR adds the runtime.slice and makes use of it as recommended in the Kubernetes documentation.

/retest

/lgtm