L1-regularized PPR on directed graphs?

Hello --
I'm interested in trying to apply the L1-regularized PPR on a directed graph ... do you know if there's an implementation of that somewhere?  Or - if not - is there a modification to the undirected algorithm that I could try to implement?  I tried playing around w/ the math a bit myself ... but didn't make much progress.
Thanks!

Just drop direction is the easiest...
If you want to prefer something with direction you need to articulate what
that is… see local higher order graph clustering for some examples
Easy way: just pick the most meaningful directed triangle and use that as
your motif.
This is short for a much longer discussion… e-mail if you want to chat
about it.
On Wed, Aug 3, 2022 at 10:18 Ben Johnson @.***> wrote:

Hello --
I'm interested in trying to apply the L1-regularized PPR on a directed
graph ... do you know if there's an implementation of that somewhere? Or -
if not - is there a modification to the undirected algorithm that I could
try to implement? I tried playing around w/ the math a bit myself ... but
didn't make much progress.
Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/kfoynt/LocalGraphClustering/issues/127, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAIJS7IMEQ6RK4XTHXQU2DVXKLVJANCNFSM55POLM4A
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I am adding a bit of info to that too.
It’s easy to use higher order method for undirected graphs.
Shenghao (CC’ed) has an implementation here: https://github.com/opallab/Local-Hyper-flow-Diffusion
You can follow up with him on how to use it if you want.
On Aug 3, 2022, at 12:45 PM, David Gleich<EMAIL_ADDRESS>wrote:
Just drop direction is the easiest...
If you want to prefer something with direction you need to articulate what
that is… see local higher order graph clustering for some examples
Easy way: just pick the most meaningful directed triangle and use that as
your motif.
This is short for a much longer discussion… e-mail if you want to chat
about it.
On Wed, Aug 3, 2022 at 10:18 Ben Johnson @.***> wrote:

Hello --
I'm interested in trying to apply the L1-regularized PPR on a directed
graph ... do you know if there's an implementation of that somewhere? Or -
if not - is there a modification to the undirected algorithm that I could
try to implement? I tried playing around w/ the math a bit myself ... but
didn't make much progress.
Thanks!
—
Reply to this email directly, view it on GitHub
https://github.com/kfoynt/LocalGraphClustering/issues/127, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAIJS7IMEQ6RK4XTHXQU2DVXKLVJANCNFSM55POLM4A
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHubhttps://github.com/kfoynt/LocalGraphClustering/issues/127#issuecomment-1204215505, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACZN7WIE2MO7HSLAJYHYZQTVXKO2DANCNFSM55POLM4A.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


Fix/electron fix

Codecov Report

Merging #311 into develop will increase coverage by 0.01%.
The diff coverage is 40%.

@@             Coverage Diff             @@
##           develop     #311      +/-   ##
===========================================
+ Coverage    29.37%   29.38%   +0.01%
===========================================
  Files           54       54
  Lines         5808     5805       -3
  Branches      1289     1290       +1
===========================================
  Hits          1706     1706
+ Misses        2963     2959       -4
- Partials      1139     1140       +1

Flag
Coverage Δ

#rnvunittests
29.38% <40%> (+0.01%)
:arrow_up:

Impacted Files
Coverage Δ

packages/rnv/src/platformTools/electron/index.js
19.35% <40%> (+0.45%)
:arrow_up:

Continue to review full report at Codecov.

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


Please update with new B2B info

Please update this doc to reflect that when using B2B, users can create a self-service portal that does not require invitation: https://docs.microsoft.com/en-us/azure/active-directory/b2b/self-service-portal

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

ID: 2a42ef49-22e4-25f4-97a8-1e392d1679ac
Version Independent ID: 62dad908-fa82-ed71-e486-ff90e67e546d
Content: How to get AppSource certified for Azure Active Directory
Content Source: articles/active-directory/develop/howto-get-appsource-certified.md
Service: active-directory
GitHub Login: @CelesteDG
Microsoft Alias: celested

Thanks for your feedback! We will investigate and update as appropriate.

Thank you for the feedback, @BarbaraSelden - Please review the changes in this PR https://github.com/MicrosoftDocs/azure-docs-pr/pull/49523 and let me know if you approve. The changes are staged here: https://review.docs.microsoft.com/en-us/azure/active-directory/develop/howto-get-appsource-certified?branch=pr-en-us-49523. Thank you.

Looks good. Thanks for the fast turn around
Thanks,
Barbara
Barbara<EMAIL_ADDRESS>Senior Program Manager – Azure Identity
Developer Guidance and Knowledge Management
O.<PHONE_NUMBER> | M.<PHONE_NUMBER>
From: Celeste de Guzman<EMAIL_ADDRESS>Sent: Monday, August 20, 2018 9:30 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Barbara Selden<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] Please update with new B2B info (#13710)
Thank you for the feedback, @BarbaraSeldenhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FBarbaraSelden&data=02|01|Barbara.Selden%40microsoft.com|bd8b6919f4d84949179808d6071ec506|72f988bf86f141af91ab2d7cd011db47|1|0|636704226074300590&sdata=Q7CUxZMmf4giQM8T%2F%2ByHm%2FBtJH4kCr1MQbP6CNBAB98%3D&reserved=0 - Please review the changes in this PR MicrosoftDocs/azure-docs-pr#49523https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs-pr%2Fpull%2F49523&data=02|01|Barbara.Selden%40microsoft.com|bd8b6919f4d84949179808d6071ec506|72f988bf86f141af91ab2d7cd011db47|1|0|636704226074300590&sdata=HVOYx2vfLuSURvQO68fXeTJwdmMDmlNiWm5kWKQYhFU%3D&reserved=0 and let me know if you approve. The changes are staged here: https://review.docs.microsoft.com/en-us/azure/active-directory/develop/howto-get-appsource-certified?branch=pr-en-us-49523https://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Freview.docs.microsoft.com%2Fen-us%2Fazure%2Factive-directory%2Fdevelop%2Fhowto-get-appsource-certified%3Fbranch%3Dpr-en-us-49523&data=02|01|Barbara.Selden%40microsoft.com|bd8b6919f4d84949179808d6071ec506|72f988bf86f141af91ab2d7cd011db47|1|0|636704226074300590&sdata=dceb7RcDI%2FafxGUUS7VP19KCQYyvb1ljaUqzMn08UVg%3D&reserved=0. Thank you.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F13710%23issuecomment-414546370&data=02|01|Barbara.Selden%40microsoft.com|bd8b6919f4d84949179808d6071ec506|72f988bf86f141af91ab2d7cd011db47|1|0|636704226074300590&sdata=63tHD56Vmj8eNvmIfSwRKLQiLPnRjfp%2BYMaXWPfTFxs%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAblqUqC5gpzzVQVdXCvH0II3D4wzSe5rks5uS4zMgaJpZM4WEyUu&data=02|01|Barbara.Selden%40microsoft.com|bd8b6919f4d84949179808d6071ec506|72f988bf86f141af91ab2d7cd011db47|1|0|636704226074300590&sdata=FAfwTleT%2F5dgcYuyvu8CLdlKp35lt1l49Cz7a16SxyA%3D&reserved=0.

Thank you very much for reviewing, @BarbaraSelden - I just updated the date for the article to indicate today's date. This update will go out in this afternoon's doc update (~3:30). Thank you.

PR has been merged and should go live by 3:30 PM today. #please-close Thank you


Update aws-java-sdk-dynamodb to 1.11.974

Updates com.amazonaws:aws-java-sdk-dynamodb from 1.11.969 to 1.11.974.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.amazonaws", artifactId = "aws-java-sdk-dynamodb" } ]

labels: library-update, semver-patch

Superseded by #106.


Presentation: SUSTO

https://github.com/BBVA/susto
https://cloud-native.slack.com/archives/C023TTU27KN/p1654521048533089

@JonZeolla to reach out and try to get a presentation.  Last attempt via CNCF Slack message on 2022-07-26

Reached out separately on 2022-08-08

Reached out on twitter

Contact made; plans to sync up next week

Planned for 2022-08-23

Slides @ https://github.com/BBVA/susto/blob/master/docs/docs/OSS-Open Security Control Testing at Scale.pdf


Propagate trace from incoming HTTP and to outgoing HTTP

Feature or Problem

Propagate trace from incoming HTTP and to outgoing HTTP
Related Issues

#3705
Release Information

Consumer Impact

Testing

Unit Test(s)

Acceptance or Integration

Manual Verification

warning: unused import: `http::HeaderMap`
  --> crates/provider-http-server/src/lib.rs:37:5
   |
37 | use http::HeaderMap;
   |     ^^^^^^^^^^^^^^^
   |
   = note: `#[warn(unused_imports)]` on by default

error[E0308]: mismatched types
   --> crates/provider-http-server/src/lib.rs:154:30
    |
154 |             &HeaderExtractor(req.headers()),
    |              --------------- ^^^^^^^^^^^^^ expected `http::header::map::HeaderMap`, found `http::HeaderMap`
    |              |
    |              arguments to this struct are incorrect
    |
    = note: `http::HeaderMap` and `http::header::map::HeaderMap` have similar names, but are actually distinct types

I use the old version of opentelemetry-http, but  opentelemetry-http dependent the old http version. Maybe we should update all opentelemetry sdk to latest, the opentelemetry-http dependent the newer opentelemetry sdk which is incompatible.

I choose not to use the opentelemetry-http public library.

I don't personally have an issue with the approach taken here and not depending on the upstream library. Perhaps we could use that in the future, but I understand the many different versions of the otel libraries creates difficulties.


Gracefully fail if CopilotProvider is omitted

Describe the bug
Virtually every CopilotKit functionality depends on a CopilotContext provided by the CopilotProvider.
e.g. CopilotTextarea autocompletions, chatbot, etc.
However when a CopilotProvider does not wrap the component, functionality fails silently.
To Reproduce
Steps to reproduce the behavior:

Omit <CopilotProvider>...</CopilotProvider>
trigger useMakeCopilotReadable, useMakeCopilotActionable, CopilotTextarea, CopilotSidebarUIProvider
See how functionality does not work, but no error is emitted

Expected behavior
An error is emitted, with clear description of the likely core issue and how to resolve it (namely, wrap the app in a CopilotProvider). Point to docs.

I would like to take a look at this if its still available.


src props of Avatar doesn't work as expected

Current Behavior
When passing url through the src prop, the image is always blank.
Expected Behavior
Image should load
Tamagui Version
1.90.12

Platform (Web, iOS, Android)
iOS, Android

Reproduction
import { Avatar } from 'tamagui'

export default () => (
  <Avatar circular size="$6">
    <Avatar.Image src="http://placekitten.com/200/300" />
    <Avatar.Fallback bc="red" />
  </Avatar>
)

System Info
No response

I'm not able to re-produce

@TheEhsanSarshar Seems like it's an issue with the Image component in this library. There's also an issue when I try to load an image with file protocol:
<Image flex={1} source={{ uri: fileUrl }} />
This will show nothing. While changing the Image component to the native one fix the problem

emmm, just tested and it seems working fine

emmm, just tested and it seems working fine

@TheEhsanSarshar Hmm interesting i'll investigate further and get back to you. I'm pretty sure all these issues happen after I upgrade tamagui version tho

I face the same issue in the monorepo. I did not check native, but in nextjs it is showing up in the html file but not rendering the picture.
I read a picture with https://... With the normal  Tag it works fine.
<Avatar circular size="$5">
                        <Avatar.Image
                            src={profile?.avatar_url}
                        />
                        <Avatar.Fallback backgroundColor="$blue10" delayMs={600} />
</Avatar>

@TheEhsanSarshar I tried the example with https and it still doesn't work. Using the source prop with width and height set works(e.g. <Image source={{width: 50, heigh:50, uri}}/>). But both width and height must be set to a non-zero value and it's bit awkward as we don't know the image demission in most of the time. The issue can be reproduced on both Android and ios so I doubt it's related to http/https.

I don´t know if the information helps. But when I remove the class in the  tag the picture shows up.
Otherwise it is not working for me.
`

`

Are there any news on this?
I meet this issue in web nextjs version.
I have tried to reinstall tamagui. I added the package for Avatars to my packages.

When I remove the class of the  Tag it works...


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-2.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 317

Questions ?
Please refer to the Backport tool documentation


Saved opt.pth is very large and seems overlap with head.pth and model.pth?

While saving state with model_only option False, I found the size of opt.pth is exactly the sum of head.pth and model.pth? Is it normal?

Opt is for the optimizer’s weight


add:Amitmasram

Description
I have added my profile  details in the devFind

@Amitmasram Thank you so much for your contribution! Your efforts are greatly appreciated and will go a long way in improving our project. Please feel free to share it with others and help us grow even further!


Create helloworld.html

Added hello world in HTML.

Looks Good 👍🏻

thnks🤩

Looks Good 👍🏻

thnks🤩

can i create another file for another pull request


Generate description sentences for Recipes and use them as suggestions

in the browser demo, the suggestions that i currently generate are:
"Display Product List and Choose from Recommended products"
and
"Display Claire's Wishlist"
It can clearly be further improved, but a reasonable starting point, i think.

\o/


Adapt UI tests to match Rancher 2.7.0

Some changes are brought by Rancher 2.7.0 and we need to adapt our Cypress code to match those changes.

And the CI is green! Nightly tests passed without any issue!


Update npm version

Since https://github.com/mimmi20/browscap-js/pull/4 is merged, it would be awesome to use it!
Can you update version and publish package to npm?

the npm package is updated.


Joint Position Controller Bug with multiple entities.

Environment

OS Version: Ubuntu 22.04.2 LTS

Description

Included the Joint Position Controller plugin in our example.
Found out that while clicking on multiple entities the list gets appended and doesn't get clear for the previous entity.

Actual behavior:

Clicked on L_0, we see pan_joint tilt_joint and roll_joint

Clicked on L_1, we see additional pan_joint tilt_joint and roll_joint are added.

They are duplicates. And sliders are not working properly.

If we clicked on all the entities, the list gets appending and sliders doesn't work properly.

Expected behavior:

Expectation is, the previous entity should be cleared out.
Clicked on L_0, we see pan_joint tilt_joint and roll_joint for L_0 only.

Clicked on L_2, we see pan_joint tilt_joint and roll_joint for L_2 only.

Steps to reproduce

We have created the minimal testcase which has 4 Logical Cameras (L_0, L_1, L_2 and L_3) with Joint Position Controller.

Untar the zip file.
$ unzip JPC_bug_testcase.zip

Archive:  JPC_bug_testcase.zip
   creating: LogCamera/
  inflating: LogCamera/model.config
  inflating: LogCamera/model.sdf
  inflating: minimal_JPC_bug.sdf

Source the gazebo path.
$ source /opt/gazebo/install/setup.bash
Export the model path.
$ export GZ_SIM_RESOURCE_PATH="$PWD:/opt/gazebo/models"
Run.
$ gz sim ./minimal_JPC_bug.sdf

Proposal

From steps to reproduce, we reproduced the actual behavior.
However, we have made the fix by JPC_bug_fix.patch
From above patch we can see the expected behavior.

Let us know your valuable inputs on the same.

thanks for the patch. I can confirm that it fixes the bug. I've created https://github.com/gazebosim/gz-sim/pull/2101 based on the patch

Thank you for the support @iche033


Warning: Future data leak

pandas-ta has 2 indicators to be aware of:
Potential Data Leaks: dpo and ichimoku. See indicator list below for details. Set lookahead=False to disable.
https://twopirllc.github.io/pandas-ta/

Thanks for reporting this.  I went ahead and commented them out of the default configuration as tuneta doesn't currently have the ability to set parameters for a specific indicator.  They can be added back manually but probably not a good idea.

Perfect.


Package:discover no longer working on 6.8.0

Laravel Version: 6.8.0
PHP Version: 7.3.7
Database Driver & Version: N/A

Steps To Reproduce:
Running php artisan package:discover --ansi will exit with the following error:
An option with shortcut "s" already exists.

Update: running any command (php artisan) results in An option with shortcut "s" already exists.

I can run artisan just fine on 6.8.0. Can you first please try one of the support channels below? If you can actually identify this as a bug, feel free to report back and I'll gladly help you out and re-open this issue.

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks!

I've just downgrade to Laravel 6.7 and it's working correctly, then upgraded and i'm getting the issue again.

The new option to generate seeders when generating models uses the "s" option now. Do you have a custom command with an "s" option?

I think that one of the packages I use may implement the -s option: babenkoivan/scout-elasticsearch-drive.
Does this classify as a BC or should I create an issue on the repo?

Extending the public api on a minor release isn't a BC break, no. It's best that you open up an issue on their repo.


react-scripts build failed with error code ELIFECYCLE, errno 1 while adding 'node-thermal-printer' in src/index.js

Is this a bug report?
Yes

Can you reproduce the problem with npm 4.x?

Yes
Have you read the Troubleshooting section and searched the User Guide?

ELIFECYCLE, react-scripts build
Environment

node -v: v8.0.0
npm -v: 5.0.0
yarn --version (if you use Yarn): NA
npm ls react-scripts (if you haven’t ejected<EMAIL_ADDRESS>
Then, specify:

Operating system: Ubuntu 16, Zorin OS 12.1
Browser and version (if relevant): NA

Steps to Reproduce

create a fresh app using create-react-app and $ npm install --save node-thermal-printer
add the line import 'node-thermal-printer' in src/index.js
$ npm run build

Expected Behavior

Successful Build.
Actual Behavior

Failed to compile with error code ELIFECYCLE, errno 1
Terminal

$ npm run build
<EMAIL_ADDRESS>build /home/andro/smartd.workspace/pos-space/sample-printer
react-scripts build
Creating an optimized production build...
Failed to compile.
static/js/main.47fb6eaa.js from UglifyJs
Unexpected token: name (i) [./~/node-thermal-printer/node-thermal-printer.js:403,0][static/js/main.47fb6eaa.js:20938,12]
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build: react-scripts build
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!  The printer code works fine in the places where react-scripts build doesn't involve like

static .js (public folder) and linked to index.html (renderer process)
electron's main process (main.js)

But the build fails, when I add the same code in src/index.js where react-scripts build failed with error code ELIFECYCLE, errno 1

have you tried with node 6 or 7?

This is the relevant line:

Unexpected token: name (i) [./~/node-thermal-printer/node-thermal-printer.js:403,0][static/js/main.47fb6eaa.js:20938,12]

It says on line 403 there's syntax Uglify doesn't understand.
Indeed, it's ES6:
let i = "QRCODE_CELLSIZE_".concat(settings.cellSize.toString())

If you compile the package down to ES5 before publishing, the issue will be fixed.
You can use Babel for this.
Hope it helps, and thanks for a great issue report!

I just installed and tried with the versions v6.11.0 and v7.10.0. Still facing the same error.

@gaearon Thanks for your answer.
As of I understood, the npm package 'node-thermal-printer' needs to be compiled down to ES5 before publishing for the reason Uglify doesn't understand a snippet in it. Am I correct?
Will it make sense, if i raise this as issue in 'node-thermal-printer' repository?

Yes, this is correct! For some reason I assumed at first that you published that package—sorry for misunderstanding.


keep the CountDown running even if button pressed

Hi,
is it possible to keep the CountDown running even if the back button pressed and component unmounted?

@h4m3d66 it's not the best practice to do this, if you keep the state of the component even when it's unmounted this will cause a memory leak in the app, also you will encounter performance issues with this, the best way to do it, you should render the component again and pass it the new until value


[FEATURE] [Charts] Combination chart

🚀 Feature request
Is your feature request related to a problem?
Sometimes for comparing data it's required to use a combination chart. For example, a combination of line and bar chart.
Describe the solution you'd like
Allow setting series type (line, bar, scatter, etc)
Describe alternatives you've considered
Some simple way to draw different chart types one on top of the other.
Additional context

This can be achieved by absolutely positioning:
https://stackblitz.com/edit/taiga-combine-charts


Welcoming contributions on Vehicle Security resources!

Please make PR's for resources you want to add about vehicle security and car hacking.

Removing this since I don't think it's necessary. We ask for contributions in README.


Finish homepage copy

write copy for download, manual, and it'd be wise to do another pass on the features' copy

Gonna close this - reopen if edits are needed!


ResourceWriteSuccess.ResourceUri reports a temporary value first and proper value after - not documented

Has a temporary value
/subscriptions/abcdef000-0000-1111-1234-ccf5bbf89daa/resourceGroups/EventGrid-RG/deployments/Microsoft.StorageAccount-20180721223827
rather than actual and no obvious way to correlate it to the resource that is deleted later as ResourceUri is not matching.

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

ID: 278a69ef-7134-37a6-ccc2-9535f7617043
Version Independent ID: 7831770e-c94a-eb88-d53a-a5860a2364e6
Content: Azure Event Grid resource group event schema
Content Source: articles/event-grid/event-schema-resource-groups.md
Service: event-grid
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

I think I've figured this. Wish documentation would be clear about it 😉
There are two ResourceWriteSuccess events. First one is raised with a temporary name used for resource (ResourceUri). Second one has a proper name. CorrelationId allows to connect the two.

@SeanFeldman Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@SeanFeldman - the resource Uri you referenced is actually for a new entry in the deployment history. You'll see the operationName is "Microsoft.Resources/deployments/write"

@tfitzmac not quite following.
My issue was seeing two distinct events with different resource URI and no way to consolidate that.
Could you please elaborate on your response?

Depending on what you are trying to do, I think you could ignore the events that are related to writing to the deployment history.

So I'd need to know how to differentiate normal events from deployment history events. In that case, documentation should help with understaning how it works. This very issue with my question shows that it's not clear from reading the article 🙂


data: pritamrajput

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[ ] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] This PR does not contain plagiarized content.
[ ] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

JSON check is not getting passed. How to make changes and How should I proceed Now ?

How can I access my updated linkfree link ?

How can I access my updated linkfree link ?

Just go to your linkfree URL which should be https://linkfree.eddiehub.io/pritamrajput, changes will be displayed once deployed!


WIP: Add Redux as state container

The rest of the build errors (apart the one I commented on inline) can be fixed by running npm install and npm run lint:fix.

Every source file (.js, .css, .java) must have a license header (I think most of configuration files don't have to, so for example .editorconfig doesn't need it).
Action item for me: set up an automated check for license headers that will fail CI build if it's missing.

Reconnection now works, thanks!


Study Spinnaker and integration requirements

As a developer, I'd like to investigate the integration relationship between Spinnaker and SCDF.
Acceptance:

POC and design review
Install Spinnaker on Cloud foundry and deploy v1 of an app followed by v2.

waiting on Spinnaker PR [1] to continue some additional tests
[1] https://github.com/spinnaker/orca/pull/753


Additional info for question 6

@sujit510 Thanks for PR. But it looks like an opinionated answer. Isn't it/

@sujit510 Thanks for PR. But it looks like an opinionated answer. Isn't it/

@sudheerj I was going through the use cases of when to prefer Class components over Functional components, and came across a use case, so thought of adding it here.. the existing answer indirectly indicates that since React 16.8, Function components has everything that Class component already has, so thought of adding some light there.
What do you think?

@sujit510  Thanks for PR 👍


Feature Request: Quality Adjustment Option for Single .m3u8 Format Videos

Hello,
I believe it would be highly beneficial if there was an option to integrate a feature that enables users to adjust the quality of .m3u8 file format videos directly from the player interface.
Presently, I have noticed examples where different file formats are used to provide varying quality options. However, my suggestion relates specifically to scenarios where only one .m3u8 URL is used. I envision the video player being able to detect this URL and subsequently display a button for changing the video quality, similar to the functionality found on platforms like YouTube.
Thank you for your attention to this matter. Looking forward to your consideration.

You can use the m3u8 example, why I am leaving it out of the logic,
To avoid implementation problems due to my logic since it doesn't cover all cases, so I want it to be easier to edit the logic for all people
If you want me to improve the example let me know

Thanks for your explanation , I  checked the .m3u8 example its good but  I think all of that could be implemented to the plugin then with a true false value that cloud be on and off ,

Will check if it will not conflict with any logics, but most probably it will not be implemented in package, the most I can do is create a handler to it

The .m3u8 example works fine on android , but it dose not work on ios
inside the video player it say "Failed to load video: The operation couldn't be completed. ( CoreMediaErrorDomain error - 12865.)

Did you make sure iOS application has the right permissions? Storage and internet?
Since it does create temp files for each quality, and ofcourse needs internet access to play any of them

I think ios do not need permissions for storage like saving temp file , and i use same exact code for android and ios , also i'm using real device not emulator , i do not get any error from flutter or ios just this :
[flutter_meedu_videoplayer] PlatformException(VideoError, Failed to load video: The operation couldn’t be completed. (CoreMediaErrorDomain error -12865.), null, null)

Seems like a general problem with video_player
https://stackoverflow.com/questions/71154896/video-not-play-in-flutter-ios-getting-error
Check if any of those fixes it , if all fails try to enabling media_kit for iOS

I tried with media_kit but not working , for now i do not know what is the problem

I tried with media_kit but not working , for now i do not know what is the problem

What is the error in media_kit?


Set a timer for scan responses.

This sets a timer for scan responses such that if they are not received within 1 second, the scan result callback will be called and the device deleted from the vector when max_results = 0.

Not confident with this fix, too much involved to implement without errors.


Unable to get local issuer certificate

Facing issue during widget generator. Filled in all required details to build a Mendix widget but faced an error.
Attaching a screenshot for better understanding. Kindly tell me what I am missing.

Dist folder is not created by the Mendix generator which contains the widget mpk.

@JelteMX
I need help sir


Add support for fsspec urlpath

fsspec support needs quite a refactor.
Closes #15

Codecov Report

Merging #44 (99d8260) into main (084f002) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #44      +/-   ##
==========================================
+ Coverage   93.18%   93.19%   +0.01%
==========================================
  Files           7        7
  Lines         543      544       +1
  Branches       62       62
==========================================
+ Hits          506      507       +1
  Misses         20       20
  Partials       17       17

Impacted Files
Coverage Δ

tests/test_esa_safe.py
100.00% <100.00%> (ø)

xarray_sentinel/esa_safe.py
82.14% <100.00%> (-0.38%)
:arrow_down:

xarray_sentinel/sentinel1.py
94.92% <100.00%> (+0.10%)
:arrow_up:

Continue to review full report at Codecov.

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


Bump dependencies

Bumped dependencies again. Here is the list of remaining dependencies that are yet to be bumped at this moment as they require a closer look:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4287

Relevant Lines:
11888

💛  - Coveralls


Caddy log format - client_ip should be used instead of remote_ip

Currently goaccess uses remote_ip in Caddy json format, but according to docs client_ip should be used since it's the one being set with proper ips when using proxies.
https://caddyserver.com/docs/caddyfile/matchers#client-ip

Hey, thanks for letting me know! Do you mind sharing a few lines from your access log so I can check out the latest changes?

Direct access without loadbalancer / proxy:
{"level":"info","ts":1703415994.3220897,"logger":"http.log.access.log1","msg":"handled request","request":{"remote_ip":"X.X.X.X","remote_port":"49570","client_ip":"X.X.X.X","proto":"HTTP/1.1","method":"GET","host":"Y.Y,Y,Y:8080","uri":"/index.php","headers":{"Upgrade-Insecure-Requests":["1"],"User-Agent":["Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"],"Accept":["text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,*/*;q=0.8,application/signed-exchange;v=b3;q=0.7"],"Accept-Encoding":["gzip, deflate"],"Accept-Language":["pl-PL,pl;q=0.9,en-US;q=0.8,en;q=0.7,fr;q=0.6,es;q=0.5,it;q=0.4,ar;q=0.3,zh-CN;q=0.2,zh;q=0.1,da;q=0.1,sv;q=0.1,nl;q=0.1,fi;q=0.1,de;q=0.1,el;q=0.1,id;q=0.1,he;q=0.1,ja;q=0.1,ms;q=0.1,ru;q=0.1,pt;q=0.1"],"Connection":["keep-alive"],"Dnt":["1"]}},"bytes_read":0,"user_id":"","duration":0.000010499,"size":0,"status":0,"resp_headers":{"Server":["Caddy"]}}

client_ip = remote_ip = my public IP
Using loadbalancer / proxy:
{"level":"info","ts":1703416013.6445792,"logger":"http.log.access.log0","msg":"handled request","request":{"remote_ip":"<IP_ADDRESS>","remote_port":"48104","client_ip":"X.X.X.X","proto":"HTTP/2.0","method":"GET","host":"xxx","uri":"/abc","headers":{"X-Forwarded-Proto":["https"],"Sec-Ch-Ua":["\"Not_A Brand\";v=\"8\", \"Chromium\";v=\"120\", \"Google Chrome\";v=\"120\""],"Dnt":["1"],"Referer":["https://xxx/"],"Cookie":[],"Cf-Connecting-Ip":["X.X.X.X"],"Cdn-Loop":["cloudflare"],"If-Modified-Since":["Wed, 18 Jan 2023 11:16:33 GMT"],"If-None-Match":["\"roohzl6az\""],"Sec-Ch-Ua-Platform":["\"Windows\""],"Accept":["*/*"],"Accept-Language":["pl-PL,pl;q=0.9,en-US;q=0.8,en;q=0.7,fr;q=0.6,es;q=0.5,it;q=0.4,ar;q=0.3,zh-CN;q=0.2,zh;q=0.1,da;q=0.1,sv;q=0.1,nl;q=0.1,fi;q=0.1,de;q=0.1,el;q=0.1,id;q=0.1,he;q=0.1,ja;q=0.1,ms;q=0.1,ru;q=0.1,pt;q=0.1"],"Cf-Ipcountry":["PL"],"X-Forwarded-For":["X.X.X.X"],"Cf-Ray":["83a860a2bba70008-IAD"],"User-Agent":["Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36"],"Sec-Fetch-Site":["same-origin"],"Accept-Encoding":["gzip"],"Cf-Visitor":["{\"scheme\":\"https\"}"],"Sec-Ch-Ua-Mobile":["?0"],"Sec-Fetch-Mode":["no-cors"],"Sec-Fetch-Dest":["script"]},"tls":{"resumed":false,"version":772,"cipher_suite":4865,"proto":"h2","server_name":"xxx"}},"bytes_read":0,"user_id":"","duration":0.000209214,"size":0,"status":304,"resp_headers":{"Server":["Caddy"],"Alt-Svc":["h3=\":443\"; ma=2592000"],"Access-Control-Allow-Origin":["*"],"Etag":["\"roohzl6az\""]}}

remote_ip=<IP_ADDRESS> is cloudflare IP here, while client_ip=X.X.X.X is my own public IP.

I've edited my goaccess.conf file, usually found in /etc/goaccess/goaccess.conf
time-format %s
date-format %s
...
# CADDY JSON Structured
-#log-format {"ts":"%x.%^","request":{"remote_ip":"%h","proto":"%H","method":"%m","host":"%v","uri":"%U","headers":{"User-Agent":["%u","%^"]},"tls":{"cipher_suite":"%k","proto": "%K"}},"duration": "%T","size": "%b","status": "%s","resp_headers":{"Content-Type":["%M;%^"]}}
+log-format {"ts":"%x.%^","request":{"client_ip":"%h","proto":"%H","method":"%m","host":"%v","uri":"%U","headers":{"User-Agent":["%u"],"Referer":["%R"]},"tls":{"cipher_suite":"%k","proto": "%K"}},"duration": "%T","size": "%b","status": "%s","resp_headers":{"Content-Type":["%M"]}}

Take note that more had to be changed than just remote_ip to client_ip, it seems like the Caddy JSON preset found here handles the headers differently than the comment in the config file.
Then I can happily view my logs with visitor IPs logged from my trusted proxy's headers.
goaccess /var/log/caddy/access.log -o /var/www/html/goaccess/report.html --real-time-html

The default config file's comment for Caddy JSON should probably be updated to handle headers the same way as the preset -- and then it would be trivial for someone to uncomment it and swap client_ip if they were using a trusted proxy.

Fixed on #2629


Show resource name under tab icon

Resolves #267

Codecov Report

Merging #268 into dev will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev     #268   +/-   ##
=========================================
  Coverage     79.06%   79.06%
  Complexity      223      223
=========================================
  Files            81       81
  Lines           731      731
=========================================
  Hits            578      578
  Misses          153      153

Continue to review full report at Codecov.

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


nim: 1.4.0 -> 1.4.2

Things done

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

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

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

Result of nixpkgs-review pr 106298 1

7 packages built:
- mosdepth
- nim
- nim-unwrapped
- nimble-unwrapped
- nimmm
- nrpl
- vimPlugins.fruzzy


No source when property value is empty

When a default value for property is non-empty and the value passed by the client is empty, the default value wins.
[Required]
public string Test { get; set; } = "!";

<input asp-for="Test" >

Hi @yecril71pl. This particular section describes model-binding errors, so I don't see the connection between your example and generating a model-state error. The suggested change describes a form of precedence, but doesn't alter how model-binding works, AFAICT. Would you like to provide more information around the motivation for this change?

After reading the documentation, I expected the empty value sent by the browser to replace the default value, which should be preserved only if the parameter in question is not sent at all.  This does not happen.

This isn't the correct place for this change. Consider the following URL:
https://localhost:5001/Test?someValue=

Here, the value for someValue is empty. If I have a model with [Required] applied to its SomeValue property, a model-binding error is generated using the preceding URL. The change in this PR suggests that an error isn't generated, but it is.

This isn't the correct place for this change. Consider the following URL:
https://localhost:5001/Test?someValue=

Here, the value for someValue is empty. If I have a model with [Required] applied to its SomeValue property, a model-binding error is generated using the preceding URL. The change in this PR suggests that an error isn't generated, but it is.

If someValue has a default value of "!" and an empty value is passed as a parameter (I experimented with POST parameters), the empty value is ignored, the property gets the default value, which is non-null, therefore no error is generated.

Ok, perhaps there's a difference between GET and POST, or between URL mapping and forms-handling. Regardless of where the difference is, the current coverage of this suggests that the default value will be used in all situations, but I think we can agree that this isn't true.

Ok, perhaps there's a difference between GET and POST, or between URL mapping and forms-handling. Regardless of where the difference is, the current coverage of this suggests that the default value will be used in all situations, but I think we can agree that this isn't true.

I do not know about all situations, I only wrote what I saw: that the default value overrides an empty value explicitly sent by the client.  Which is unexpected.

Hi @yecril71pl. I'm closing this PR for reasons I've explained above. I agree that the behaviour you describe exists, but this is neither the correct place for the remark nor exhaustive enough in explaining the exact scenario where said behaviour occurs. If you feel strongly about adding an explanation, please do open a new PR with a more appropriate location and detailed explanation.

This variety and inconsistency suggests it is a bug that needs to be fixed rather than a feature that needs to be documented.


[Idea] Podfile.local

Take this pattern:
# Allows per-dev overrides
local_podfile = "Podfile.local"
eval(File.open(local_podfile).read) if File.exist? local_podfile

And make it systematic.

that'll currently only work for the "main" target, I think, because multiple target blocks for the same target go :boom:

legit, and that's a bad pattern IMO


How to get asset_id, when we call mintasset method

It's only known afterwards in the logmint action

curl -H 'TRON-PRO-API-KEY:your_api_key' 'https://apilist.tronscanapi.com/api/block'


[Event Hubs] Tune processor ownership docs

Summary
The focus of these changes is to add guidance for choosing the number of processor instances to the configuration sample and reduce the frequency that the "owns too many partitions" warning is logged.  It will now occur only when the owned partition count changes, rather than on each load balancing cycle.

API change check
API changes are not detected in this pull request.


usage

Hi,
Could you provide some steps on how to use this in traefik running on a k8s cluster?
thx

Updated


2.1.1 - unable to copy token when creating API token

Steps to reproduce:

Install influxdb 2.1.1
quick start
create API token

Expected behavior:
After creating the token a modal box appears with the token in, to copy to clipboard
Actual behavior:
No modal appears, but the token is added to the token list
A message box saying "API token was created successfully" appears top right
Environment info:

InfluxDB version: 2.1.1

This error means I am unable to do almost anything from the CLI tool, as I cannot attach a token to a request

Hi,
same here, Docker image influxdb:2.1-alpine

click on "Generate API Token"
select "All Access API Token"
add a description
click "Save"
Token is not shown

There is a workaround:

open the developer console (F12)
select the network tab
create a token
you will find your token in the response:

the issue is still there, v2.3.0

Just got bit by this after finally deciding to upgrade from 1.8
Cant believe it still exists, thx @sgramsch for the work around

The existence of this bug is a truly ridiculous waste of time.

This issue is still present in the most recent version (2.6.1).

Closing this issue as resolved.
On versions prior to 2.5, after creating a token, the user can click on the token name to see the token and copy from there.
On versions 2.5+, the token will be show one time upon creation.
These are the expected behaviors.


Time Input keeps slowly scrolling up/down after the user stops doing so.

Describe the bug
When using the time input selector and scrolling to select your value the column will keep slowly scrolling after the user has stopped doing so, even once they leave the column. It doesn't seem to start this behavior right off the bat but rather after scrolling up and down a little bit first? I made a clip of it here: https://gfycat.com/plainwildargentineruddyduck
To Reproduce
Steps to reproduce the behavior:

Fresh 'npm i' with the latest version (currently "react-datepicker": "^4.3.0",)

<DatePicker
showTimeInput
//Other stuff removed so we can prevent the cause being something else
/>
3. My video showing this behavior had the other fields included because they were going to be part of the design, but I stripped out everything except showTimeInput being passed in, tested it, and the behavior still remained.
Expected behavior
Scrolling should stop when the user stops.
Screenshots
Needs a video: https://gfycat.com/plainwildargentineruddyduck
Desktop (please complete the following information):

OS: macOS Big Sur version 11.0.1
Browser Chrome Version 95.0.4638.54 (Official Build) (x86_64)

Smartphone (please complete the following information):
N/A
Additional context
The minute column is the one mostly affected by this. The hour column is also affected by this behavior but much less so, usually only scrolling one or two more ticks.

I also experienced this same issue. Can anyone attend to this?

I also have the same issue. I'll post if i find a workaround

i found a naughty workaround:
just use a different package lmao
import DatePicker from 'react-native-date-picker'
import { ScrollView, Text } from 'react-native';

export default function EntryPoint() {
  const [date, setDate] = useState(new Date())
  const [open, setOpen] = useState(false)

  return (
    <ScrollView scrollEnabled={!open}>
      <Button title="Open" onPress={() => setOpen(true)} />
      <DatePicker
        modal
        open={open}
        date={date}
        onConfirm={(date) => {
          setOpen(false)
          setDate(date)
        }}
        onCancel={() => {
          setOpen(false)
        }}
      />
    </ScrollView>
  )
}


Update meta.yaml to 1.0.3

Updated meta.yaml to version 1.0.3 with SHA value of f66d2303be57f18cb4134cabb5c95b5cf3fbcf33fc959fd8b447e01b8e47ae03

This was for testing purposes. Cloisng it.


[FR] Option to turn off swap

FEATURE REQUEST
Please add an option to turn off swap if activated instead of stopping the playbook.
Something like https://github.com/kubernetes/contrib/blob/master/ansible/roles/node/tasks/swapoff.yml

That was quick! Cool :)


Improved messaging when requested channel slug not allowed by license

What this PR does / why we need it:
This improves the messaging when a requested channel slug is not allowed by a license, and explicitly inform's when we fall back to the "stable" channel. Theres up to 3 new log entries emitted:

No channel specified in upstream URI, falling back to channel "stable". to explicitly signal we used the fallback
Verifying if channel slug "beta" allowed by license to explicitly signal what slug we're looking for
To install/pull an allowed channel, use one of the following:  appSlug/channelSlug to indicate which would be allowed if there supplied channel slug was not allowed

Which issue(s) this PR fixes:

Fixes https://app.shortcut.com/replicated/story/110998/improved-messaging-when-requested-channel-slug-not-allowed-by-license
Special notes for your reviewer:
Sample usage without a channel slug:
$ kots install e2app --license-file ~/Downloads/testcustomer.yaml -n e2eapp
  • No channel specified in upstream URI, falling back to channel "stable".
  • Checking for license update ✓
  • Verifying if channel slug "stable" is allowed by the license ✗
    • To install an allowed channel, use one of the following: e2eapp/beta, e2eapp/automated-1, e2eapp/unstable, e2eapp/alternate
Error: failed to verify and update license: channel slug "stable" is not allowed by latest license

$ kots install e2app -n e2eapp
  • No channel specified in upstream URI, falling back to channel "stable".
  • Deploying Admin Console
    • Creating namespace ✓
    • Waiting for datastore to be ready ✓
Enter a new password for the admin console (6+ characters):

Sample usage with a channel slug thats not allowed:
$ kots install e2app/notreal --license-file ~/Downloads/testcustomer.yaml -n e2eapp
  • Checking for license update ✓
  • Verifying if channel slug "notreal" is allowed by the license ✗
    • To install an allowed channel, use one of the following: e2eapp/beta, e2eapp/automated-1, e2eapp/unstable, e2eapp/alternate
Error: failed to verify and update license: channel slug "notreal" is not allowed by latest license

Sample usage with a channel slug that IS allowed:
$ kots install e2app/beta --license-file ~/Downloads/testcustomer.yaml -n e2eapp
  • Checking for license update ✓
  • Verifying if channel slug "beta" is allowed by the license ✓
  • Deploying Admin Console
    • Creating namespace ✓
    • Waiting for datastore to be ready ✓

Steps to reproduce

Does this PR introduce a user-facing change?

- Improved messaging when requested channel slug not allowed by license

Does this PR require documentation?

@sgalsaleh updated output:
# airgap & vanilla failure
➜ bin/kots install e2app --license-file ~/Downloads/testcustomer.yaml --shared-password password -n e2eapp --airgap
  • No channel specified in upstream URI, falling back to channel slug 'stable'.
  • Channel slug "stable" is not allowed by license. Please use one of the following: e2eapp/beta, e2eapp/automated-1, e2eapp/unstable, e2eapp/alternate
Error: failed to verify and update license: channel slug "stable" is not allowed by license
➜ bin/kots install e2app --license-file ~/Downloads/testcustomer.yaml --shared-password password -n e2eapp
  • No channel specified in upstream URI, falling back to channel slug 'stable'.
  • Checking for license update ✓
  • Channel slug "stable" is not allowed by license. Please use one of the following: e2eapp/beta, e2eapp/automated-1, e2eapp/unstable, e2eapp/alternate
Error: failed to verify and update license: channel slug "stable" is not allowed by latest license

# airgap and vanilla success
➜ bin/kots install e2app/unstable --license-file ~/Downloads/testcustomer.yaml --shared-password password -n e2eapp --airgap=true
  • Deploying Admin Console
    • Creating namespace ✓
    • Waiting for datastore to be ready ⠸^C
  • Cleaning up
➜ bin/kots install e2app/unstable --license-file ~/Downloads/testcustomer.yaml --shared-password password -n e2eapp
  • Checking for license update ✓
  • Deploying Admin Console
    • Creating namespace ✓
    • Waiting for datastore to be ready ⠇^C
  • Cleaning up


Can't create new Image in ImageTexture

Godot version:

3.2.3
OS/device including version:

Zorin OS 15.3 Nvidia GTX 650 450 proprietary driver
Issue description:

I can't seem to create a new image in the editor for a ImageTexture specificity on the TextureRect node
Steps to reproduce:

Create a new TextureRect node
Assign a ImageTexture in the texture slot
Expand it and try to add a new Image in the image slot
See that it fails

 drivers/gles2/rasterizer_storage_gles2.cpp:814 - Condition "texture->data_size == 0 && !texture->render_target" is true. Returned: Ref<Image>()

**Minimal reproduction project:**
<!-- A small Godot project which reproduces the issue. Drag and drop a zip archive to upload it. -->
[TextureRectImageTextureImageBug.zip](https://github.com/godotengine/godot/files/5594594/TextureRectImageTextureImageBug.zip)

Duplicate of https://github.com/godotengine/godot/issues/38112


Weird behavior when autocompleting

Examples

Why do I have those highlighted zones when I don't fill an autocomplete field with autocomplete? Is there a way to disable that because it constantly causes me to jump around in the buffer for no reason.

Try disabling snippet support with snippet_support = False in your kak-lsp.toml.
a0cc198fbd1c49c75b658e27ec76ff5bb8983d94 already does that on latest master.

Assuming this is answered but please speak up otherwise

Duplicate of #282

Left it open to see if the behavior itself would be corrected or improved. Disabling it was just a momentary workaround from my pov.

Yeah I should have linked the main snippet/completion issue in the first place


Output to Amazon CloudWatch

Adding a new output module for Amazon CloudWatch. This allows jmxtrans
to output values into Amazon CloudWatch 'custom metrics'. See
http://aws.amazon.com/cloudwatch/ and
http://docs.aws.amazon.com/AmazonCloudWatch/latest/DeveloperGuide/publis
hingMetrics.html
Original code by Sascha Moellering, https://github.com/saschamoellering

[ ]   General Comment
 jmxtrans project » jmxtrans #157 FAILURE
Looks like there's a problem with this pull request
(what's this?)
 I do not use AWS personally, so not much motivation to rebase this PR. But if someone has some time to fix the merge and write some tests, I'll be happy to review and merge it.
 Hi, I think, I can help :) The original code is written by me and there are dependencies missing I guess. Should I send an additional pull request?
 Sure ! That would be really nice of you !
 Hi, I fixed the problems in PR 243
https://github.com/jmxtrans/jmxtrans/pull/243
How can I update the documentation? If everything is correct, I'll do the same for jmxtrans agent.
[ ]   File: pom.xml:L400-411
 Pom shoudl be sorted according to Maven recommendations. You can use mvn sortpom:sort to sort it automatically (https://code.google.com/p/sortpom/).
 Should I sort it and open an additional pull request?
 If you push the changes to the same branch (as you did) it will update this PR. No need for an additional PR.
[ ]   File: test/com/googlecode/jmxtrans/model/output/CloudWatchWriterTests.java:L1-22
 This test should probably be named validationFailsWithoutSettings.
[ ]   File: test/com/googlecode/jmxtrans/model/output/CloudWatchWriterTests.java:L1-22
 Test coverage seems to be a bit light. I'll try to add some tests this evening.
 It's hard to add tests here without having a proper AWS account.
 Testing with an AWS account would be integration testing. I care much more about unit testing. You have quite a bit of logic that could be tested without accessing AWS. For example, the convertToDouble() and most of the getRegion() methods can be tested without accessing AWS.
 Oh I see, I can add a few tests if you want.
 I'll write additional tests. Could you just make sure the build passes?
 Sure, NP
[ ]   File: src/com/googlecode/jmxtrans/model/output/CloudWatchWriter.java:L1-155
 Indentation should be done with tabs, not spaces to be coherent with other sources in this project.

Hi, I think, I can help :) The original code is written by me and there are dependencies missing I guess. Should I send an additional pull request?

Sure ! That would be really nice of you !

Hi, I fixed the problems in PR 243
https://github.com/jmxtrans/jmxtrans/pull/243
How can I update the documentation? If everything is correct, I'll do the same for jmxtrans agent.

It seems that your branch is based on a fairly old version of jmxtrans:master. Could you rebase it against HEAD ?

This is weird, I did a rebase: no changes. Additionally I forked the project today.

Problem might be between chair and keyboard on my side ... I had a look at https://github.com/mhanisch/jmxtrans/commits/master and it indicates latest commit on "Sep 15, 2014".

Replaced by #243


Publish Docker Images

We should define Docker images for various MLTE components and distribute them on Docker Hub. These should include:

MLTE UI
MLTE Store

These builds should execute on any merge to the master branch.

This is currently not advised, as these images will only work locally due to the API link handled by the frontend, which is always "localhost". This can be avoided when building an image by configuring a specific IP with the new .env.frontend functionality, but this has to be done when building an image, as it will include the specific config inside the image.  All of this is due us currently converting the frontend to a static set of webpages, and thus the API IP can't be configured in there.


feat: added FailedMount event reason to get the failure in pod analyzer

📑 Description
The FailedMount failure is a common Kubernetes issue when a engineer wants to attach volume to the pod. I was not be able to capture this pod event when I run analyzer with filter Pod. Therefore, I added the event reason FailedMount to be included in failure messages
Before:
./k8sgpt analyze --filter Pod --namespace  my-namespace
AI Provider: openai

No problems detected

After:
./k8sgpt analyze --filter Pod --namespace my-namespace
AI Provider: AI not used; --explain not set

0  my-namespace/efs-csi-app(efs-csi-app)
- Error: Unable to attach or mount volumes: unmounted volumes=[efs-storage], unattached volumes=[efs-storage kube-api-access-n44mc]: timed out waiting for the condition

✅ Checks

[x] My pull request adheres to the code style of this project
[ ] My code requires changes to the documentation
[ ] I have updated the documentation as required
[x] All the tests have passed

ℹ Additional Information

@AlexsJones Hi Alex, thanks for reviewing the PR, could you please trigger the workflow as well? I resolved the conflict and the pipeline got re-triggered, thanks

@AlexsJones Hi Alex, looks like the linter failed with some other files
/home/runner/work/_temp/reviewdog-Lgzh1S/golangci-lint-1.55.2-linux-amd64/golangci-lint run --out-format line-number --timeout=120s
  pkg/analyzer/gateway_test.go:95:29: Error return value is not checked (errcheck)
  	apiextensionsv1.AddToScheme(scheme)
  	                           ^
  pkg/analyzer/gateway_test.go:127:16: Error return value is not checked (errcheck)
  	gtwapi.Install(scheme)
  	              ^
  pkg/analyzer/gateway_test.go:128:29: Error return value is not checked (errcheck)
  	apiextensionsv1.AddToScheme(scheme)
  	                           ^
  pkg/server/server_test.go:36:25: Error return value of `server_config.Shutdown` is not checked (errcheck)
  		server_config.Shutdown()
  		                      ^
  cmd/serve/serve.go:123:20: Error return value of `logger.Sync` is not checked (errcheck)
  		defer logger.Sync()
  		                 ^
  pkg/server/integration.go:40:6: ineffectual assignment to err (ineffassign)
  	var err error = status.Error(codes.OK, "")
  	    ^
  pkg/server/integration.go:127:8: ineffectual assignment to err (ineffassign)
  			in, err := integrationProvider.Get(i)
  			    ^
  pkg/server/server_test.go:31:3: SA2000: should call wg.Add(1) before starting the goroutine to avoid a race (staticcheck)
  		wg.Add(1)
  		^
  pkg/integration/prometheus/config_analyzer.go:1[95](https://github.com/k8sgpt-ai/k8sgpt/actions/runs/7629324462/job/20811494870?pr=883#step:3:100):34: SA1024: cutset contains duplicate characters (staticcheck)
  				path = strings.TrimLeft(arg, prometheusConfigFlag)
  				                             ^
  pkg/integration/prometheus/config_analyzer.go:1[98](https://github.com/k8sgpt-ai/k8sgpt/actions/runs/7629324462/job/20811494870?pr=883#step:3:103):34: SA[102](https://github.com/k8sgpt-ai/k8sgpt/actions/runs/7629324462/job/20811494870?pr=883#step:3:107)4: cutset contains duplicate characters (staticcheck)
  				path = strings.TrimLeft(arg, configReloaderConfigFlag)
  				                             ^
  pkg/analyzer/mutating_webhook.go:64:5: SA4006: this value of `failures` is never used (staticcheck)
  				failures = append(failures, common.Failure{
  				^
  pkg/analyzer/validating_webhook.go:62:5: SA4006: this value of `failures` is never used (staticcheck)
  				failures = append(failures, common.Failure{
  				^
  level=error msg="Timeout exceeded: try increasing it by passing --timeout option"

Not related to my changes, what's the recommendation steps here? thanks


@cols_to_join_on requires that @schema is also set

When @cols_to_join_on is specified, an error is set specifying that a valid @cols_to_join_on has not been set.  When explicitly specifying the schema, it works.
It appears that at line 438 the filter is matching c.TABLE_SCHEMA to @schema, requiring the @schema parameter be defined, while on line 470 the filter is matching c.TABLE_SCHEMA to COALLESCE(@schema, SCHEMA_NAME()), allowing the current schema to be used if not specified.

Good catch, this should now be fixed. Let me know if you still encounter problems with it though.


Fitting 2 folds for each of 6 candidates, totalling 12 fits

after training Rasa nlu model, it is showing following msg "Fitting 2 folds for each of 6 candidates, totalling 12 fits". what exactly does it mean
Rasa NLU version:0.13.7
Operating system (windows, osx, ...):windows 10
Content of model configuration file:

Issue:

This is a message from sklearn i believe. In future, please ask these kind of questions in the forum


Issues with a custom frameProcessor - undefined is not a function, js engine: hermes ❓

Question
Following the guide, I successfully created a custom frameProcessor to perform OCR recognition for my Android app.
I am using it locally in my project I didn't distribute it.
It works! But randomly, after several rebuilds and small code changes, the frameProcessor stop to works and it throws this error (see screenshot belowe):
...
ERROR  TypeError: undefined is not a function, js engine: hermes
...
ERROR  TypeError: Cannot read property 'scanOCR' of undefined
...

I created a file called frameProcessorWrapper.ts with this content:
/* eslint-disable no-undef */
import type { Frame } from 'react-native-vision-camera';

/**
 * Scans OCR.
 */

export function scanOCR(frame: Frame): any {
  'worklet';
  // @ts-ignore
  return __scanOCR(frame);
}

And I use it this way:
import { scanOCR } from './frameProcessorWrapper';
....
  const frameProcessor = useFrameProcessor((frame) => {
    'worklet'
    const fpOcrResult = scanOCR(frame);
  }, []);

I was able to resolve the issue by performing a clean and resetting the pc, but the problem returned after some new rebuilds. I think It is an issue related to how this function is imported from android native to js.
I attach the error screenshot:

What I tried
No response
VisionCamera Version
^2.12.0
Additional information

[ ] I am using Expo
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar questions in the issues page as well as in the discussions page and found none.

Solved.
I put import 'react-native-reanimated' at the top of my Component file, insted of at the top of App.js


Order of axis members won't update when doing DnD

place product family and store state on the rows. execute the query. now invert the two (whichever was first, place it second). the result won't change and still shows the previous order in which they were dragged.
this can be reproduced in 2.2-SNAPSHOT as of September 23rd

fixed


question about 3.5-dev

I found that pyenv install -l have 3.5-dev in it.
it get latest code from python Mercurial repository(https://bitbucket.org/mirror/cpython).
How can I keep it be the latest code everyday after I install it?
pyenv install 3.5-dev everyday?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/22945596-question-about-3-5-dev?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282009&utm_medium=issues&utm_source=github).

At first, you should create cache directory by mkdir -p "$(pyenv root)/cache" to create repository cache. Then, run pyenv uninstall -f 3.5-dev && pyenv install 3.5-dev everyday.

got it. thank you very much :100:


Allow force /warp-ing a target player to work fully across the proxy network

Include /force warp player command to be performed on entire network.
Currently, you need to be in the console of the same server the target player is playing, this way, you will be able to force a player visit a warp, despite if you are in the same server console, or not. This is usefull to perform certain situations server-wide IE: Jails!

Added in v3


Crash when saving picture

Hello,
When I try to save a photo with the action in the right item button, the application crash because I don't have gallery right.
I test it on an iPhoneX and 7, both on iOS11 and i got the crash.
I test it on an iPhone6 on iOS10.3 and no problem.
Is it a know iOS11 issue ? Will it be fixed soon ?
Thanks you so much for your help

Please paste the crashlog or example to reproduce it, or tell me how to do it

Hello,
I found my problem, I was missing this key in my Info.plist :
NSPhotoLibraryAddUsageDescription
And iOS11 absolutely need it or it will crash.
Hope it can help people with the same problem as me.


Words counting issue for mixed languages

It is very common that we could have content of mixed languages such as a paragraph mixed of English and Chinese. A big difference of counting English and CJK words is that CJK does not separate words with spaces (actually "word" and "character" are the same concept in CJK) but they are just adjacent.
For example
The quick brown fox jumps over the lazy dog will be counted as 9 words.
The Chinese translation of that sentence is 敏捷的棕毛狐狸从懒狗身上跃过 it should be counted as 14 words, but the actual result is 1.
This results the issue like https://github.com/TryGhost/Ghost/issues/2656 when writing a blog post of mixed languages.

see how they did this https://github.com/AAlakkad/jQuery-smsHelper/blob/master/jquery-smshelper.js
any jQuery support ?


docs: Add a Python hello-world example to docs

What do you think about adding a subsection in the section "Run the application" that explains how to run the Python application?


emutls: Add helper shared library for TLS test subjects

Fix for #743
Add LLVM sources for __emutls_get_address and build libmull-tls.dylib which can be used to avoid missing symbol errors.
To do:

[ ] Add documentation on how to invoke mull-cxx -ld-preload for subjects using thread_local variables

I'm wondering if it's a good idea to just supply something like libmull-tls.so/dylib (or even libmull-rt.so/dylib) and add an option to the driver (something like -dylibs) and ask the user to specify this option manually?
Then it would be something like:
mull-cxx ... -dylibs=/usr/lib/libmull-tls.so

and the -dylibs will just add whatever is passed to the dynamicLbraries.
Automatic resolution (as implemented in this draft) is nice, but the users would have to specify the library search paths. On one hand, if they are providing these options already then it's very convenient since no changes needed. On the other hand, if they have to specify the search paths, then it's just easier to specify the full path to the file.
We can probably think of several options, e.g.:

Add an extra flag for dynamic libraries.
Automatic lookup in the library search paths (maybe hidden behind -enable-tls flag).
Implement both 1 and 2? :)

Does anything of this make sense to you @gebart? :)

@AlexDenisov thanks for the feedback. I agree that using a command line switch is cleaner.
I like your proposals, how about this implementation? (1+2 above)

Add a command line arg, -ld-preload=foo.so (for similarity with LD_PRELOAD env var)
If a file with the given path is found, load that library, otherwise use -ld-search-path to resolve the library file path.

The name libmull-tls is a much better description than libemutls, I will update this PR with that.

@gebart I cannot stress enough how much I appreciate your work on this problem. In the end, we decided to not use JIT anymore, so this fix is no longer needed. I'm closing this PR hoping that the new release solves this issue and you have a more smooth integration into your project.
I'm looking forward for your further feedback.

@gebart I cannot stress enough how much I appreciate your work on this problem. In the end, we decided to not use JIT anymore, so this fix is no longer needed. I'm closing this PR hoping that the new release solves this issue and you have a more smooth integration into your project.
I'm looking forward for your further feedback.


fix(SearchBar): Restore explore all link

Changes

Restores the Explore all link to the search bar
Adds 2 new Event types, there was one missing for the search results page and one new one added for the SearchBar
Added a test

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Minor request: Can we add a screenshot of the new view into the PR?


Non-UTF-8 code starting with '\x8d¡

when I try to "hack" an ip I get this error, I don't know how to solve it, the complete error is:
SyntaxError: Non-UTF-8 code starting with '\x8d' in file /home/kali/IpHack/IpHack.py on line 5, but no encoding declared; see https://python.org/dev/peps/pep-0263/ for details
I get the error in kali linux and in termux

Not supported Kali linux


The primary group is not updated in the database

Description
After setting the group permission using API to a user, the primary group is not updated in the database.
Reproduction steps

Add a group to the user using the API
Look in the database

Expected behaviour
The database must contain the correct primary group
Environment details

BungeeCord/1.16-R0.5-SNAPSHOT (build 1559)
LuckPerms version: v5.3.33

https://gist.github.com/Sceri/978eca438086011245d3f2a18ded6d4f (used code)
Before giving:

After giving

Database row after giving:

Apparently, the primary group is not updated in internal PrimaryGroupHolder when using the API. Therefore, a wrong value is written to the database when saving. But at the same time everything is fine in the cache.

See https://luckperms.net/wiki/Configuration#primary-group-calculation
(although the wiki is actually a bit out of date, the default is parents-by-weight everywhere now)

Just tested this and the primary group only updates in the metadata cache when using the API, and not in the underlying PrimaryGroupHolder. Using the parent set|add commands actually work as expected.
User#setPrimaryGroup(String) returns FAIL_ALREADY_HAS because it checks against the cached metadata before applying it, even though it differs.

Fixed by the above commit, thanks


unicode characters in large files cause extreme slowdown

Sublime Text 3143
Windows 10 x64

I first noticed this when attempting to select all text with Ctrl + A in the __init__.py file of the keyboard module. Sublime Text would become entirely unresponsive for about 5-10 seconds, and then finally select all of the text in the file.
I managed to narrow down the cause to this line, which contains a ç character.
Tests:

Disabled all plugins, issue disappeared. Had a hunch that it might be WordHighlight since it deals with selecting text, and my suspicion was confirmed when re-enabling all other plugins.
Created a large scratch file (~1000 lines)

tested select all while WordHighlight was disabled - no issue
enabled WordHighlight - still no issue
added the ç character to the end of the file - issue arises
disabled WordHighlight - no issue

I had the same problem after I upgraded my sublime 3 to 3143...when I 'select all' or lots lines in a Chinese text, the sublime will freeze...(special select all in thousands lines text)
A temp solution is set my user setting "highlight_non_word_characters": true -> false
It seems some special codes used in CJK character caused this issue....

i also have this problem and it also appeared only after updating to build 3143. it was terribly difficult to use sublime text 3.
temporary solution: "highlight_non_word_characters": true -> false did not help me (i already had false).
unresponsive occurs if there is cyrillic chars in the text. with ascii characters everything is fine.

I have the same issue when i use "Ctrl+A" to select whole file or mouse to select only some multiline part of the text.
I use a lot of characters in Cyrillic (Windows 1251) encoding.
Is there any way to fix it and still use "highlight_non_word_characters": true?


How to make radar chart with empty center like in doughnut?

As in the topic
How to make radar chart with empty center like in doughnut?
Or how to make padding from center to start point 0?

Please search the closed issues. There are many solutions posted there

I've been looking all over the docs and closed issues and can't find any solution for this problem, is there somebody that does know where to find it?

set minimum value below 0.

When setting the minimum value below 0 the anglelines will start from that point which prevents the 'doughnut' effect
example

@benmccann You mentioned there are many solutions posted here. I was not able to find any radar charts with empty centers. Could you help to link to a solution?

For anyone interested, I have posted a somewhat working solution in Stackoverflow: https://stackoverflow.com/a/70029928/560318

Sorry to bring this thread back to life... but are there any other solutions available at this point? I'm wanting something similar but in my case I need the chart to be responsive. I'm also unsure how the values in that example came to be?
Any help would be much appreciated!


High memory usage in the ingress gateway

Is this the right place to submit this?

[X] This is not a security vulnerability or a crashing bug
[X] This is not a question about how to use Istio

Bug Description
We use Istio for both service mesh and as an ingress gateway.
We have a load test, using gatling, in open model. The load test sends a requests towards one of the services trough an ingress gateway:
load test --> ingress gateway --> upstream service

During the test, the upstream service fails, and ingress gateway starts to return 503, which is a correct behavior. What isn't correct, though, is the fact that ingress gateway memory starts to increase when the upstream service is down.
I would like to know what is the root cause of this issue, and why the memory starts to increase. This doesn't seem to be a memory leak, because when the test finishes, the memory is lowered significantly.
Below you can find graphs:

the first one is number of requests that ended up with either 200 or 503 code, this is based on istio_requests_total
the second one is memory reported by pod, based on container_memory_working_set_bytes
the third one is a p95 latency, measured by the ingress gateway pod, using istio_request_duration_milliseconds_bucket metric. The latency is measured by response code.

And lastly, a heap dump from the gateway, took during the time when memory usage was very high:
File: envoy
Type: inuse_space
Showing nodes accounting for 1432273.49kB, 97.00% of 1476618.51kB total
Dropped 660 nodes (cum <= 7383.09kB)
      flat  flat%   sum%        cum   cum%
 1157888kB 78.41% 78.41% 1168576.19kB 79.14%  Envoy::Network::ListenerFilterBufferImpl::ListenerFilterBufferImpl
148193.22kB 10.04% 88.45% 148193.22kB 10.04%  OPENSSL_malloc
43156.56kB  2.92% 91.37% 101338.88kB  6.86%  std::__1::make_unique
21332.78kB  1.44% 92.82% 169701.08kB 11.49%  <unknown>
14603.37kB  0.99% 93.81% 18612.02kB  1.26%  Envoy::Server::ActiveTcpSocket::ActiveTcpSocket
10901.97kB  0.74% 94.55% 10901.97kB  0.74%  OPENSSL_realloc
 7567.22kB  0.51% 95.06% 20746.36kB  1.40%  Envoy::Event::DispatcherImpl::createFileEvent
 6796.94kB  0.46% 95.52% 1454476.76kB 98.50%  std::__1::__function::__func::operator()
 6175.88kB  0.42% 95.94% 58708.02kB  3.98%  Envoy::Event::DispatcherImpl::createServerConnection
 5947.56kB   0.4% 96.34%  8650.22kB  0.59%  Envoy::Event::LibeventScheduler::createSchedulableCallback
 2803.44kB  0.19% 96.53% 1171379.62kB 79.33%  Envoy::Server::ActiveTcpSocket::createListenerFilterBuffer
 2407.06kB  0.16% 96.69% 1279723.11kB 86.67%  Envoy::Network::TcpListenerImpl::onSocketEvent
    2261kB  0.15% 96.85%  1267517kB 85.84%  Envoy::Server::ActiveTcpListener::onAcceptWorker
 2194.38kB  0.15% 96.99% 12588.70kB  0.85%  Envoy::Network::FilterManagerImpl::addReadFilter
   44.03kB 0.003% 97.00%  9990.62kB  0.68%  Envoy::Http::ConnectionManagerImpl::initializeReadFilterCallbacks
    0.08kB 5.3e-06% 97.00% 22016.08kB  1.49%  std::__1::__tree::__emplace_unique_key_args
         0     0% 97.00% 10901.66kB  0.74%  BUF_MEM_append
         0     0% 97.00%  9660.41kB  0.65%  EVP_parse_public_key
         0     0% 97.00% 41347.50kB  2.80%  Envoy::Buffer::WatermarkBufferFactory::createBuffer
         0     0% 97.00%  8650.22kB  0.59%  Envoy::Event::DispatcherImpl::createSchedulableCallback
         0     0% 97.00% 13179.14kB  0.89%  Envoy::Event::FileEventImpl::FileEventImpl
         0     0% 97.00% 1454410.04kB 98.50%  Envoy::Event::FileEventImpl::assignEvents()::$_1::__invoke
         0     0% 97.00% 73219.06kB  4.96%  Envoy::Extensions::ListenerFilters::TlsInspector::Filter::Filter
         0     0% 97.00% 51274.20kB  3.47%  Envoy::Extensions::TransportSockets::Tls::ContextImpl::newSsl
         0     0% 97.00% 59987.92kB  4.06%  Envoy::Extensions::TransportSockets::Tls::ServerSslSocketFactory::createDownstreamTransportSocket
         0     0% 97.00% 33432.48kB  2.26%  Envoy::Extensions::TransportSockets::Tls::SslHandshakerImpl::doHandshake
         0     0% 97.00% 58182.08kB  3.94%  Envoy::Extensions::TransportSockets::Tls::SslSocket::SslSocket
         0     0% 97.00% 33432.48kB  2.26%  Envoy::Extensions::TransportSockets::Tls::SslSocket::doWrite
         0     0% 97.00% 52544.65kB  3.56%  Envoy::Network::ConnectionImpl::ConnectionImpl
         0     0% 97.00% 36072.94kB  2.44%  Envoy::Network::ConnectionImpl::onFileEvent
         0     0% 97.00% 33480.48kB  2.27%  Envoy::Network::ConnectionImpl::onWriteReady
         0     0% 97.00% 20745.81kB  1.40%  Envoy::Network::IoSocketHandleImpl::initializeFileEvent
         0     0% 97.00% 138676.42kB  9.39%  Envoy::Network::ListenerFilterBufferImpl::onFileEvent
         0     0% 97.00% 52532.15kB  3.56%  Envoy::Network::ServerConnectionImpl::ServerConnectionImpl
         0     0% 97.00% 138955.81kB  9.41%  Envoy::Server::ActiveStreamListenerBase::newConnection
         0     0% 97.00% 1246645.39kB 84.43%  Envoy::Server::ActiveStreamListenerBase::onSocketAccepted
         0     0% 97.00% 1310334.91kB 88.74%  Envoy::Server::ActiveTcpSocket::continueFilterChain
         0     0% 97.00% 138955.29kB  9.41%  Envoy::Server::ActiveTcpSocket::newConnection
         0     0% 97.00% 93104.89kB  6.31%  Envoy::Server::Configuration::FilterChainUtility::buildFilterChain
         0     0% 97.00% 74211.46kB  5.03%  Envoy::Server::ListenerImpl::createListenerFilterChain
         0     0% 97.00% 1452654.44kB 98.38%  Envoy::Server::WorkerImpl::threadRoutine
         0     0% 97.00% 1452647.28kB 98.38%  Envoy::Thread::ThreadImplPosix::ThreadImplPosix()::{lambda(void*)#1}::__invoke
         0     0% 97.00% 1118756.32kB 75.76%  MallocHook::InvokeNewHookSlow
         0     0% 97.00% 1108645.34kB 75.08%  NewHook
         0     0% 97.00%  8466.34kB  0.57%  ProfilerFree
         0     0% 97.00%  9660.16kB  0.65%  RSA_parse_public_key
         0     0% 97.00% 34206.51kB  2.32%  SSL_do_handshake
         0     0% 97.00% 123069.77kB  8.33%  SSL_new
         0     0% 97.00% 1453960.66kB 98.47%  [libc.so.6]
         0     0% 97.00% 22616.88kB  1.53%  bssl::SSLAEADContext::CreateNullCipher
         0     0% 97.00%  7632.47kB  0.52%  bssl::ssl_cert_dup
         0     0% 97.00%  7483.81kB  0.51%  bssl::ssl_get_new_session
         0     0% 97.00% 55694.03kB  3.77%  bssl::ssl_handshake_new
         0     0% 97.00%  9659.91kB  0.65%  bssl::ssl_on_certificate_selected
         0     0% 97.00% 11416.88kB  0.77%  bssl::ssl_open_handshake
         0     0% 97.00% 34206.51kB  2.32%  bssl::ssl_run_handshake
         0     0% 97.00% 22787.55kB  1.54%  bssl::ssl_server_handshake
         0     0% 97.00% 11095.49kB  0.75%  bssl::tls13_server_handshake
         0     0% 97.00% 98860.59kB  6.70%  bssl::tls_new
         0     0% 97.00% 11416.88kB  0.77%  bssl::tls_open_handshake
         0     0% 97.00% 1454402.94kB 98.50%  event_base_loop
         0     0% 97.00% 1454419.80kB 98.50%  event_process_active_single_queue
         0     0% 97.00%  9660.16kB  0.65%  rsa_pub_decode
         0     0% 97.00%  7434.80kB   0.5%  tcmalloc::allocate_full_cpp_throw_oom
         0     0% 97.00% 12572.04kB  0.85%  virtual thunk to Envoy::Network::ConnectionImpl::addReadFilter(std::__1::shared_ptr)

There are two questions that I would like to answer:

first one is why we see increased memory? Is it due to a growing number of connections?
why latency is so high for calls that end up with 503

Version
We are running Istio `1.19.7`, installed in a Kubernetes cluster managed by AWS (EKS). The Kubernetes version is `1.27`.

Additional Information
No response

I think this is likely fixed by the work going on in https://github.com/istio/istio/issues/51910 (?)

@howardjohn
Yeah, that's true, we can close it. However, I would appreciate if you could take a look at this discussion as it is related to this issue. Thanks! 🙂


!! example not working!!

Error LOG:
E/flutter ( 9549): [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: MissingPluginException(No implementation found for method getProperties on channel flutter_user_agent)
E/flutter ( 9549): #0      MethodChannel.invokeMethod (package:flutter/src/services/platform_channel.dart:319:7)
E/flutter ( 9549): <asynchronous suspension>
E/flutter ( 9549): #1      FlutterUserAgent.init (package:flutter_user_agent/flutter_user_agent.dart:19:43)
E/flutter ( 9549): #2      FlutterUserAgent.getPropertyAsync (package:flutter_user_agent/flutter_user_agent.dart:46:11)

Flutter version:
[✓] Flutter (Channel stable, v1.12.13+hotfix.5, on Mac OS X 10.15.2 19C57,
    locale zh-Hans-CN)
    • Flutter version 1.12.13+hotfix.5 at /Users/min/flutter
    • Framework revision 27321ebbad (3 days ago), 2019-12-10 18:15:01 -0800
    • Engine revision 2994f7e1e6
    • Dart version 2.7.0

I can't replicate this issue. You should try cleaning your project and rebuilding from scratch.
Or trying to debug the crash yourself, isolating what is causing it and potentially proposing a fix.


tdbg diagnose command for stuck workflow

We need a tdbg command that could help dump all relevant information and do some basic analysis.
tdbg workflow diagnose
Information includes:

Mutable state
Workflow History
Shard info (ack level, read level for all queues)
Task queue info (including all partitions, backlog size etc)
Namespace info

Diagnose part:
Check if there should have workflow task scheduled or started, but didn't happen.
Check if there is any timer that should already fired but did not happen.
Check if there is worker polling the right task queue.

Thank you for your feature request! We understand the need for a comprehensive diagnostic tool within Temporal.
Currently, there isn't a single tdbg command that fulfills all these requirements. However, you can combine existing tools and techniques to achieve similar results.
To gather the information you need:

Mutable state: Use temporal-cli describe-workflow command.
Workflow History: Use temporal-cli describe-workflow command.
Shard info: Use temporal-cli list-shards and temporal-cli describe-shard commands.
Task queue info: Use temporal-cli describe-task-queue command.
Namespace info: Use temporal-cli describe-namespace command.

For the "diagnose" part, you can use the following:

Check for unscheduled tasks: Use the temporal-cli list-open-workflows command to see if the workflow has any pending tasks.
Check for unfired timers:  Inspect the workflow history using the temporal-cli describe-workflow command. You can find the timer events.
Check for worker polling:  Use the temporal-cli list-workers command to confirm the worker is polling the correct task queue.

We encourage you to join the Temporal community and submit a feature request for a dedicated tdbg workflow diagnose command.  This will help us prioritize development efforts to improve the debugging experience.
Let me know if you have any other questions!

Thank you for this feature request! It sounds like you are looking for a more comprehensive debugging tool within Temporal. Currently, Temporal doesn't have a single command like tdbg workflow diagnose to perform all the actions you've outlined.
However, you can achieve most of this functionality by combining existing tools and approaches:

Mutable State:  Use the temporalctl workflow describe command to inspect a workflow's current state. Temporal CLI Reference
Workflow History: The temporalctl workflow describe command also includes the workflow history. You can also use the temporalctl history describe command for a more detailed view of the workflow history. Temporal CLI Reference
Shard info, Task queue info, and Namespace info: You can use the temporalctl cluster describe command to obtain information about shards, task queues, and namespaces.  Temporal CLI Reference
Diagnosis: Temporal's built-in monitoring and logging tools are valuable for debugging workflow issues. You can use the Temporal UI, the temporalctl CLI, or your custom logging solutions to analyze workflow behavior, identify potential issues, and perform additional troubleshooting. Temporal UI, Temporal CLI Reference

We appreciate your feedback and will consider it for future enhancements to Temporal's debugging capabilities.

Thank you for the feature request!
Currently, Temporal does not have a tdbg workflow diagnose command. However, you can use the following commands to achieve similar results:

Workflow History:

temporal describe-workflow-execution --workflow-id <workflow-id>
temporal history --workflow-id <workflow-id>

Shard Info:

temporal describe-namespace --namespace <namespace> (Includes information about the namespace, which can be helpful for troubleshooting shards)

Task Queue Info:

temporal describe-task-queue --task-queue <task-queue>

Mutable State:

This information is included in the temporal history output.

Diagnose:

You can use the temporal describe-workflow-execution command to determine if a workflow task should have been scheduled or started, but was not.
You can also use the temporal describe-timers --namespace <namespace> command to see if there are any timers that should have fired but did not.
To check if a worker is polling the correct task queue, you can review the worker logs.

For more information on using these commands, please refer to the Temporal CLI documentation: https://docs.temporal.io/docs/temporal-cli/

Thank you for the feature request!  We appreciate you taking the time to share this.
While we don't currently have a tdbg workflow diagnose command, you can get the information you need by using a combination of existing tools:

Mutable State: Use temporalctl workflow describe to view the mutable state of a workflow.
Workflow History: Use temporalctl workflow describe to view the workflow history.
Shard Info:  Use temporalctl cluster describe to view shard info.
Task Queue Info: Use temporalctl taskqueue describe to view the task queue info.
Namespace Info: Use temporalctl namespace describe to view the namespace info.

For the diagnose part of your request, we recommend:

Check if there should have workflow task scheduled or started, but didn't happen: Use temporalctl workflow describe to review the history of the workflow. This should show if any tasks have been scheduled or not.
Check if there is any timer that should already fired but did not happen: Review the workflow history using temporalctl workflow describe.
Check if there is worker polling the right task queue: Use temporalctl taskqueue describe to confirm the task queue the worker is polling.

We will consider adding a tdbg workflow diagnose command in the future.

Thank you for this feature request.
While we don't have a tdbg workflow diagnose command at this time, you can achieve some of the same functionality using existing tools:

Mutable state: The temporalctl workflow describe command can be used to view the current mutable state of a workflow. https://docs.temporal.io/docs/temporalctl/commands/workflow
Workflow History:  You can access the workflow history with the temporalctl workflow history command.  https://docs.temporal.io/docs/temporalctl/commands/workflow
Shard info, Task Queue info, and Namespace info: Use the temporalctl cluster describe command with the --shard, --taskQueue and --namespace options to get detailed information about those entities. https://docs.temporal.io/docs/temporalctl/commands/cluster

To check for potential issues with task scheduling, timers, and worker polling, you can use the following:

Task Scheduling: Review the workflow history to ensure that the expected tasks are being scheduled.
Timers: You can use the temporalctl namespace describe command and look for any active timers to verify they are working correctly.  https://docs.temporal.io/docs/temporalctl/commands/namespace
Worker polling: Use the temporalctl worker describe command to check that workers are polling the correct task queues. https://docs.temporal.io/docs/temporalctl/commands/worker

We appreciate your feedback, and we'll consider adding a tdbg workflow diagnose command in the future.


⚠ Jenkins Plugins Artifactory has degraded performance

In a118f63, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1324 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 1a198e8 after 12 minutes.


Correctly remove boundary references when worker leaves

We were not removing all routes to a boundary to a worker
that left the cluster. This led to problems in in-flight
message acking, since we would try to request acks to
boundaries that were no longer connected.
Closes #2074

@SeanTAllen I only saw this causing a problem with the in-flight message barrier work, so I'm not sure what test we would use here.  I think the fact that it passes our current tests indicates that it's not introducing a new problem.

I think this is only waiting on your review @SeanTAllen

Code looks reasonable. I don't know how to test.

@SeanTAllen As I say, I can't think of a test since this only created a problem with my in-flight message work.  It was doing the incorrect thing before, it just didn't happen to matter.  I think the fact that it's passing our existing tests means it's ok to merge.


Python3 support

Hey guys,
I cleaned up @bakhtiary's PR a bit, and fixed the remaining python3 incompatibilities. Here's the output of nosetests on OSX, using the latest master branch of https://github.com/embotech/ecos-python.
$ python --version
Python 3.4.2 :: Continuum Analytics, Inc.
rmcgibbo@MacBook-Pro  ~ (3.4)
$ nosetests cvxpy
..................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................................
----------------------------------------------------------------------
Ran 1298 tests in 13.151s

OK

Can someone please update me on python3 compatibility?   The comment on the file README.md says "working on passing tests with python3", but python3 is not mentioned in that file.   With the latest version (installed with "pip install --upgrade cvxpy"), I get:
kbriggs:~> python3
Python 3.4.0 (default, Apr 11 2014, 13:05:11)
[GCC 4.8.2] on linux
Type "help", "copyright", "credits" or "license" for more information.

import cvxpy
Traceback (most recent call last):
File "", line 1, in
File "/usr/local/lib/python3.4/dist-packages/cvxpy-0.2.12-py3.4.egg/cvxpy/init.py", line 20, in
from atoms import *
ImportError: No module named 'atoms'

In my opinion, if cvxpy wants to maintain python2+python3 compatibility it should add
from __future__ import division, print_function, absolute_import

at the top of each source file.

I think the travis yml is possibly wrong for python 3, so I think that the continuous integration is not checking python 3 compatibility.

Also, despite the "pip install --upgrade cvxpy", it seems I only got version 0.2.12, not the latest 0.2.16.   Is there some option I should give to pip to make it install into the python3 site-packages directory?

Is there some option I should give to pip to make it install into the python3 site-packages directory?

I think you can use pip3 or pip3.4 or whatever, if you have these installed.

I tried pip3, and now it works!  The cvxpy install docs really ought mention pip3 - I'd never heard of it.
kbriggs:~> python3
Python 3.4.0 (default, Apr 11 2014, 13:05:11)
[GCC 4.8.2] on linux
Type "help", "copyright", "credits" or "license" for more information.

import cvxpy

$ pip install --upgrade cvxpy

By the way, if system-wide installation is not necessary I always try to use the --user flag to install things.
$ pip3 install --upgrade --user cvxpy
Also if you are using the most recently released pip, weird things may happen if you try to upgrade packages that do not use a pep440-compliant version naming scheme, especially if you are trying to upgrade to unreleased development versions.  This doesn't seem to affect released versions of cvxpy.

I tried pip3, and now it works!

Have you tried running the cvxpy tests?  I'm not sure how to run the tests using Python 3.  I suspect $ nosetests cvxpy would use Python 2.

kbriggs:~> nosetests-3.4 cvxpy
...
Ran 1294 tests in 7.229s
OK
kbriggs:~> nosetests-2.7 cvxpy
...
Ran 1294 tests in 5.931s
OK


Display Error without url

I would like to show error (Display) but to remove or change an url, because it contains sensitive info (API key). Is there any easy way to do this?

Hi!
I'm having the same problem.
For now I'm using this dirty trick:
let url = String::from("...");
let res = reqwest::get(url.as_str())
    .and_then(|r| r.error_for_status())
    .map_err(|e| display_http_error(e, url))

fn display_http_error(e: reqwest::Error, url: String) -> String {
    e.to_string().replace(format!("{}: ", url).as_str(), "")
}

It works because I still have access to the URL string, but it is not a nice solution...

Seems like a fair thing to want to do. What seems like a good way to do this? Things that come to mind:

Add some without_url method that returns some impl Display reference, kind of like how the time crate provides formatters.
This would help directly when wanting to format, and could be done with references.

Add without_url that consumes the error and drops the URL field completely.
This would be useful to remove the URL once and prevent forgetting about it in other places. It couldn't be done with a reference, though.

There's probably other ways too, if you have an idea.

fn without_url(&self) -> String should have similar implementation as Display for reqwest::Error has.
What about fn without_url(&self) -> Cow<Self>? A method of Error which removes url.

What about fn without_url(&self) -> Cow? A method of Error which removes url.

I'm assuming the method would return Cow::Owned with a clone of the error if the URL was removed? If it is indeed possible to clone reqwest::Error, I think it's better to implement the Clone trait for Error and make without_url take self by value.
As far as I can tell, these are all viable options (I added the last one).
/// Remove URL information from error and return modified error
pub fn without_url(self) -> Self;
/// Generate a printable object which shows the error without URL information
pub fn without_url(&self) -> impl Display;
/// Remove URL information from this error
pub fn delete_url(&mut self);

Hi all!
I'm coming up against this same issue (with an API key in the query). @seanmonstar /@kangalioo, is there any plan to merge #1299? Otherwise, I'd propose a PR with the following API which probably covers the gamut of the above:

/// Mutative methods

/// Get a mutable reference to the url, if present (for example, if you wanted to modify the url to remove the sensitive information
pub fn url_mut(&mut self) -> Option<&mut Url>;
/// Remove the url entirely
pub fn clear_url(&mut self) -> &mut Self;
/// Strip the url and return self (similar to `clear_url`, but possibly more ergonomic in some cases)
pub fn without_url(mut self) -> Self;

/// Format Structs (where each return wrapper implements Display)

/// Return a wrapper struct (borrowing self) that masks the url in the display impl (but still shows that it exists
pub fn as_url_masked(&self) -> UrlMaskedError<'_>;
/// Return a wrapper struct (borrowing self) that ignores the url field
pub fn as_url_hidden(&self) -> UrlHiddenError<'_>;
/// Return a wrapper struct (borrowing self) that applies a custom formatting function to the url on display.
pub fn with_url_formatter<F: Fn(&Url, f: &mut std::fmt::Formatter) -> std::fmt::Result>(&self, f: F) -> UrlFormattedError<'_, F>;

And, since we're allowing mutation of the url field, it may make sense to make Error::with_url, which already exists, public.


Created 1.31 foundation with updated tags and patches for k8s v1.31.0-beta.0

Issue https://github.com/aws/eks-distro/issues/3214, if available:
Description of changes:

Created the 1.31 foundation with all the git tags and public patches from

K8s Patches https://gitlab.aws.dev/eks-dataplane/eks-kubernetes-patches/-/tree/mainline/patches/1.31/0-public?ref_type=heads
K8's version in 1.31 latest manifest(https://gitlab.aws.dev/mpvarma/aws-wesley-manifests/-/commit/18dec55b22658d02ce80b4f84e7a51541e91b59b)

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

We should also add it to .github/labeler.yml to auto label 1.31 changes.

IF you see the issue, i tagged. I have already added that label: Issue: https://github.com/aws/eks-distro/issues/3214 & PR: https://github.com/aws/eks-distro/pull/3215

/lgtm

/approve

/unhold