adds rule for detetcing getting information about open app'n windows.

Signed-off-by:ashutoshmore658<EMAIL_ADDRESS>(cloudDefense.ai)
What type of  PR is this?

/kind feature

Any specific area of the project related to this PR?

/area rules

Special notes for your reviewer:
Adds a rule to detect special kind of malicious system activity which tries to gather system information by getting information about open application windows in a system.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

54a9db7 adds rule for detetcing getting information about open application windows

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Welcome @ashutoshmore658! It looks like this is your first PR to falcosecurity/rules 🎉

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: ashutoshmore658
Once this PR has been reviewed and has the lgtm label, please assign jasondellaluce for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

54a9db7 adds rule for detetcing getting information about open application windows
26820e9 adds evet.type in the rule as per the review recieved

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

54a9db7 adds rule for detetcing getting information about open application windows
26820e9 adds evet.type in the rule as per the review recieved
38e6ad5 Merge remote-tracking branch 'origin/add-get-window-info-rule' into add-get-window-info-rule

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Adding label do-not-merge/contains-merge-commits because PR contains merge commits, which are not allowed in this repository.
Use git rebase to reapply your commits on top of the target branch. Detailed instructions for doing so can be found here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Can I  create an another branch and raise PR's freshly ?

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

54a9db7 adds rule for detetcing getting information about open application windows
26820e9 adds evet.type in the rule as per the review recieved
38e6ad5 Merge remote-tracking branch 'origin/add-get-window-info-rule' into add-get-window-info-rule
00ef6da Merge branch 'add-get-window-info-rule' of https://github.com/ashutoshmore658/rules into add-get-window-info-rule

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

54a9db7 adds rule for detetcing getting information about open application windows
26820e9 adds evet.type in the rule as per the review recieved
38e6ad5 Merge remote-tracking branch 'origin/add-get-window-info-rule' into add-get-window-info-rule
00ef6da Merge branch 'add-get-window-info-rule' of https://github.com/ashutoshmore658/rules into add-get-window-info-rule

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closed this PR.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


Attempting to use this as a library throws undefined_symbol

I've just tried including and writing the line Hilbert *h = init_hilbert(100,100.f);
and for some reason the linker isn't finding the implementation of the function.
It's in xcode but I just put together a fresh command line project, compiles otherwise.
any tips would be of great help.

duuuuh.  Added extern "c" to include.  worked fine.


Add alerting for VFS.kext.error messages

There isn't currently an alert for VFS.kext.error messages and one should be added.

@jrbriggs @jeschu1 FYI - I marked this one as pri1

@jeschu1 yep we can close this, I will close it now


Unable to apply migration: [...]the entity type is defined in shadow state[...]

Unfortunately I have no clue what EF is trying to tell me with the exception. This is what I did:

Upgraded my EF7-beta4 project to beta6
Added a simple string  property to an entity
Created a migration via dnx . ef migration add
Manually cleaned up the migrations Up and Down methods, because the automatic scaffolding tried to (re-)create all tables etc.
Tried to apply the new migration via dnx . ef migration apply

The exception I get:
dnx . ef migration apply
Using context 'Db'.
Using database 'Wuzlstats' on server '(localdb)\MSSQLLocalDB'.
System.InvalidOperationException: The navigation property 'LeagueId' cannot be added to entity type 'Wuzlstats.Models.Ga
me' because the entity type is defined in shadow state and navigations properties cannot be added to shadow state.
   at Microsoft.Data.Entity.Metadata.EntityType.AddNavigation(String name, ForeignKey foreignKey, Boolean pointsToPrinci
pal)
   at Microsoft.Data.Entity.Metadata.Internal.InternalEntityTypeBuilder.Navigation(String navigationName, ForeignKey for
eignKey, Boolean pointsToPrincipal, ConfigurationSource configurationSource, Boolean canOverrideSameSource)
   at Microsoft.Data.Entity.Metadata.Internal.InternalEntityTypeBuilder.Navigation(String navigationName, ForeignKey for
eignKey, Boolean pointsToPrincipal, ConfigurationSource configurationSource)
   at Microsoft.Data.Entity.Metadata.Internal.InternalEntityTypeBuilder.Relationship(InternalEntityTypeBuilder principal
EntityTypeBuilder, InternalEntityTypeBuilder dependentEntityTypeBuilder, String navigationToPrincipalName, String naviga
tionToDependentName, IReadOnlyList`1 foreignKeyProperties, IReadOnlyList`1 principalProperties, ConfigurationSource conf
igurationSource, Nullable`1 isUnique, Nullable`1 isRequired, Func`2 onRelationshipAdding)
   at Microsoft.Data.Entity.Metadata.Internal.InternalEntityTypeBuilder.Relationship(InternalEntityTypeBuilder principal
EntityTypeBuilder, InternalEntityTypeBuilder dependentEntityTypeBuilder, String navigationToPrincipalName, String naviga
tionToDependentName, ConfigurationSource configurationSource, Nullable`1 isUnique, Boolean strictPrincipal)
   at Microsoft.Data.Entity.Metadata.Internal.InternalEntityTypeBuilder.Relationship(EntityType principalEntityType, Ent
ityType dependentEntityType, String navigationToPrincipalName, String navigationToDependentName, ConfigurationSource con
figurationSource, Nullable`1 isUnique, Boolean strictPrincipal)
   at Microsoft.Data.Entity.Metadata.Builders.EntityTypeBuilder.ReferenceBuilder(EntityType relatedEntityType, String na
vigationName)
   at Microsoft.Data.Entity.Metadata.Builders.EntityTypeBuilder.Reference(String relatedTypeName, String navigationName)

   at Wuzlstats.Migrations.Initial.<>c.<BuildTargetModel>b__6_4(EntityTypeBuilder b) in C:\Users\saxx\Documents\Wuzlst
ats\src\Wuzlstats\Migrations\20150519184700_Initial.Designer.cs:line 85
   at Microsoft.Data.Entity.ModelBuilder.Entity(String name, Action`1 buildAction)
   at Wuzlstats.Migrations.Initial.BuildTargetModel(ModelBuilder builder) in C:\Users\saxx\Documents\Wuzlstats\src\Wuz
lstats\Migrations\20150519184700_Initial.Designer.cs:line 83
   at Microsoft.Data.Entity.Migrations.Infrastructure.MigrationModelFactory.Create(Action`1 onModelCreating)
   at Microsoft.Data.Entity.Migrations.Migrator.ApplyMigration(Migration migration)
   at Microsoft.Data.Entity.Migrations.Migrator.ApplyMigrations(String targetMigration)
   at Microsoft.Data.Entity.Commands.MigrationTool.ApplyMigration(String migrationName, String contextTypeName, String s
tartupAssemblyName)
   at Microsoft.Data.Entity.Commands.Program.<>c__DisplayClass13_0.<ApplyMigration>b__0()
   at Microsoft.Data.Entity.Commands.Program.Execute(String startupProject, Func`1 invoke)
   at Microsoft.Data.Entity.Commands.Program.ApplyMigration(String migration, String context, String startupProject)
   at Microsoft.Data.Entity.Commands.Program.<>c__DisplayClass10_3.<Main>b__9()
   at Microsoft.Framework.Runtime.Common.CommandLine.CommandLineApplication.Execute(String[] args)
   at Microsoft.Data.Entity.Commands.Program.Main(String[] args)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.Framework.Runtime.Common.EntryPointExecutor.Execute(Assembly assembly, String[] args, IServiceProvider s
erviceProvider)
   at Microsoft.Framework.ApplicationHost.Program.ExecuteMain(DefaultHost host, String applicationName, String[] args)
   at Microsoft.Framework.ApplicationHost.Program.Main(String[] args)
--- End of stack trace from previous location where exception was thrown ---
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at Microsoft.Framework.Runtime.Common.EntryPointExecutor.Execute(Assembly assembly, String[] args, IServiceProvider s
erviceProvider)
   at dnx.host.Bootstrapper.RunAsync(List`1 args, IRuntimeEnvironment env, FrameworkName targetFramework)
   at dnx.host.RuntimeBootstrapper.ExecuteAsync(String[] args, FrameworkName targetFramework)
   at dnx.host.RuntimeBootstrapper.Execute(String[] args, FrameworkName targetFramework)

Everything works fine when I create a new database from the model, only trying to migrate an existing database fails.
The affected code is on https://github.com/saxx/Wuzlstats/, please see this branch for the exact changes I did: https://github.com/saxx/Wuzlstats/compare/master...Timeouts?expand=1
What is a "shadow state", what is this exception trying to tell me, and how can I fix it? Thanks!

Migrations don't upgrade very well between pre-releases. There's been a lot of code churn in that area recently. Try deleting all your migrations, the model snapshot, and your database, then add a new migration and apply.

Thanks! Not really happy with that, because I need to drop my existing database and create a new one (and move all existing data to the new one), but I guess that comes with using a preview :)
Creating a new database works fine, by the way.

In the future if the schema hasn't changed, you could manually insert into the _MigrationHistory table to mark the new migration as already applied.


Multiarch Containerimage

Hi
Thanks a lot for your work! I'm currently rebuilding the container, as I need it in another arch that x86_64. Not completely sure, but I think the only thing you need to do is adding this:
platforms: <PLATTFORMS>

somewhere here:
https://github.com/pryorda/vmware_exporter/blob/b441656a68e8141ea6e3f75c8d246bb493228cb4/.github/workflows/docker-push.yml#L18-L21
I can't test it that way, but if you would like I can create a PR.
Kind regards

Hey @tuxpeople, IF you could get a PR for this, Ill take a look.


Intercept exceptions

My ideal situation is to be able to:

Declare a callback for when an exception is caught
The callback can decide to modify the exception, or respond differently

I did write a hacky solution in a pull request which was understandably denied because it had problems: 0423e8b1e89fdef3bd06d657c0821d0c6c90ab9f

this is now possible, pls see here http://autobahn.ws/python/reference/autobahn.wamp.html#autobahn.wamp.protocol.ApplicationSession.onUserError


How to? Renew letsencrypt plesk certificate via cronjob

Hi everyone,
First of all: Great work! I'm really happy we will get an let's encrypt plesk plugin!
But to get it automated in the most convenient way, I would like to give the renewal task to a cronjob.
I tried to make it work using letsencrypt-plesk plugin directly:
./letsencrypt-auto --renew-by-default --letsencrypt-plesk -d [my_domain]
But it doesnt work, says it needs some sort of secret key?
letsencrypt: error: argument --letsencrypt-plesk:plesk-secret-key: expected one argument
So i tried further on and was able to set up a certificate using webroot-plugin:
$ ./letsencrypt-auto --renew-by-default --webroot  -w /var/www/vhosts/[my_location] -d [my_domain]
Works all fine, and it does create the certificate and will give me a blue promt asking, if I would like to install it to apache or plesk. If i choose plesk, it works.
Is there any way to extend my command so it doesn't show that promt and automatically chooses plesk-installation, so I can use it for a cronjob? Or can someone explain to me, what kind of secret key it needs and how to use it?
Thanks in advance!

Hey @geschke-eu, arguments --authenticator letsencrypt-plesk:plesk --installer letsencrypt-plesk:plesk could be used for automation.
The secret key is an API authentication (doc), but this is optional argument - it is created on demand while executing with root privileges.
BTW, try the extension - it creates the cronjob after the installation.

@xgin The cronjob is an excellent feature, thank you!

@xgin Where can I see this cronjob from the extention in plesk?

@ntimo Server Management > Tools & Settings > Scheduled Tasks

Hey @xgin, thanks for your reply! Your arguments worked like a charm :+1:
Also, it's pretty cool that we have a plesk plugin for UI now. Thanks for all your awesome work!
But I guess, i will use the command line option to go on, because we need some script automation in setting up domain/website instances anyway.
I would close the Issue, but maybe anyone can help me with the second promt, which is asking if I would like to set a redirect, so that everyone has to use HTTPS. How can I pre-set this to "easy" via argument? Found nothing in the help...
Thanks @ all!

@geschke-eu create a separate issue about the redirect enhancement.

Ok, I'll do. Thanks!

I saw that the certificate is renewed every month in cronjob, shouldn't be every 90 days?

@TeHashX there is a note here https://letsencrypt.readthedocs.org/en/latest/using.html#renewal

make it less than every 90 days to avoid problems, say every month

Cronjob is created automatically by the script, I'm not sure if this is correct for every 88 days?

*88 * *


add cognitive AI service module documentation

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


Cannot reproduce the reported scores on YouTube Highlight (domain = parkour)

Hi @EasonXiao-888 ,
Thank you for sharing your great work. I trained UVCOM on YouTube highlight (domain = parkour), but could not reproduce the reported score: 75.7. I used CLIP+Slowfast features. Could you share the trained weights? Do you have any tips to train UVCOM on YouTube Highlight?
# Reproduced score
{"brief": {"mAP": 0.63171}}

# Reported score (from the paper)
75.7

Thanks for your interest and sincerely for the late reply. Given the length of time and the volume of files, we will reorganize the log files and related training scripts soon and will post them here.


Expose our babel preset as next/babel

This will make writing tests or modules that share the babel configuration with next much easier :)

See twitter thread with @lacker for context: https://twitter.com/lacker/status/807706564693852160?lang=en


:book: Update README and drop duplicated content from community

Everything is in  :handshake: community repository now

canceled jobs...


Links do now work anymore

These seems to work instead:
<script src="<EMAIL_ADDRESS>"></script>
<script src="https://rawgit.com/basarat/typescript-script/master/transpiler.js"></script>


Enumerate subsites of the root site returns accessDenied

Using the Microsoft Graph Explorer, when I attempt to enumerate the subsites using

https://graph.microsoft.com/v1.0/sites/root/sites

it returns
{     "error": {         "code": "accessDenied",         "message": "The caller does not have permission to perform the action.",         "innerError": {             "request-id": "44e8cca0-ce71-4353-a165-0daefed7aad0",             "date": "2019-08-06T17:45:27"         }     } }
Even though I am assigned the permissions Sites.Manage.All, Sites.Read.All  and Sites.ReadWrite.All
Calling

https://graph.microsoft.com/v1.0/sites/root

correctly return the details of my root site.
I'm not sure why the subsites are not being returned.

Thanks for your feedback.  Your OAuth token is a Delegated token, which means that the application and user must have permission to enumerate subsites.  While the application has permission to enumerate subsites of the root site, the user does not have permission to enumerate subsites of teh root site  Therefore, the accessDenied failure is expected.


Fix the NullPointer error seen with an AD setup

Fix an NPE in forming the AD query.
https://github.com/rancher/rancher/issues/7824

@alena1108 review/merge please

Unrelated TF: mysql: Failed to get connection to database, will retry for 5 minutes


React Slick breaks on fast scroll

Try scrolling with your mouse really fast and let go as soon as possible. You need to go forwards two times and then back to reproduce the bug
Can be seen on the default demo
https://codesandbox.io/s/ppwkk5l6xx

Oh man. I just ran into this testing on a project working on.
I just spent 2 days trying to get this to work in a flex based layout and then i discovered this issue.

It doesn't seem to be a problem when switching the transition to fade.

#1740

Anyone found a workaround for this issue yet? Experiencing the same issue when swiping backwards in the slider. It only occur while swiping and not when using arrow navigation.


Create .all-contributorsrc

@all-contributors please add @JP1016 for tests and code

@all-contributors


v8.2.0 seeing failed assertions leading to panics in terraform bridge

Describe what happened
After upgrading from v8.1.0 to v8.2.0 we started encountering errors running pulumi preview or pulumi deploy of any gcp related resources:
Diagnostics:
  gcp:serviceaccount:Account (python-service-account):
    error: error reading from server: EOF

  pulumi:pulumi:Stack (my_cool_staging_stack):

    panic: fatal: An assertion has failed: The bridge does not accept secrets, so we should not encounter them here
    goroutine 123 [running]:
    github.com/pulumi/pulumi/sdk/v3/go/common/util/contract.failfast(...)
    	/home/runner/go/pkg/mod/github.com/pulumi/pulumi/sdk/v3@v3.130.0/go/common/util/contract/failfast.go:23
    github.com/pulumi/pulumi/sdk/v3/go/common/util/contract.Assertf(0x60?, {0x57dac69?, 0xc005408140?}, {0x0?, 0x410a05?, 0x8?})
    	/home/runner/go/pkg/mod/github.com/pulumi/pulumi/sdk/v3@v3.130.0/go/common/util/contract/assert.go:35 +0xe8
    github.com/pulumi/pulumi-terraform-bridge/v3/pkg/tfbridge.(*ConfigEncoding).UnfoldProperties(0xc00449c040, 0xc002006b70)
    	/home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/v3@v3.90.1-0.20240918114950-4954882afe70/pkg/tfbridge/config_encoding.go:143 +0x29a
    github.com/pulumi/pulumi-terraform-bridge/pf/internal/configencoding.(*provider[...]).ConfigureWithContext(0xc00526bac0, {0x60e6d38?, 0xc002006990}, 0xc002006b70)
    	/home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/pf@v0.43.1-0.20240918114950-4954882afe70/internal/configencoding/provider.go:110 +0x45
    github.com/pulumi/pulumi-terraform-bridge/pf/internal/plugin.(*provider).Configure(0x55c34e0?, {0x60e6d38?, 0xc002006990?}, {0x0?})
    	/home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/pf@v0.43.1-0.20240918114950-4954882afe70/internal/plugin/provider_context.go:143 +0x25
    github.com/pulumi/pulumi-terraform-bridge/pf/internal/plugin.providerThunk.Configure({{0x610dcf0?, 0xc0023365c0?}}, {0x60e6d38, 0xc002006990}, {0xffffffffffffffff?})
    	/home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/pf@v0.43.1-0.20240918114950-4954882afe70/internal/plugin/provider_server.go:83 +0x76
    github.com/pulumi/pulumi/sdk/v3/go/common/resource/plugin.(*providerServer).Configure(0xc0018a6510, {0x60e6d38, 0xc002006990}, 0xc00526b900)
    	/home/runner/go/pkg/mod/github.com/pulumi/pulumi/sdk/v3@v3.130.0/go/common/resource/plugin/provider_server.go:314 +0x331
    github.com/pulumi/pulumi-terraform-bridge/pf/internal/plugin.(*providerServer).Configure(0x60ace60?, {0x60e6d38?, 0xc002006990?}, 0x1?)
    	/home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/pf@v0.43.1-0.20240918114950-4954882afe70/internal/plugin/provider_server.go:57 +0x25
    github.com/pulumi/pulumi-terraform-bridge/x/muxer.(*muxer).Configure.func1()
    	/home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/x/muxer@v0.0.9-0.20240227144008-2da15b3d6f6e/muxer.go:298 +0x6f
    github.com/pulumi/pulumi-terraform-bridge/x/muxer.asyncJoin[...].func1()
    	/home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/x/muxer@v0.0.9-0.20240227144008-2da15b3d6f6e/muxer.go:595 +0x42
    created by github.com/pulumi/pulumi-terraform-bridge/x/muxer.asyncJoin[...] in goroutine 121
    	/home/runner/go/pkg/mod/github.com/pulumi/pulumi-terraform-bridge/x/muxer@v0.0.9-0.20240227144008-2da15b3d6f6e/muxer.go:594 +0x79

The identical code snippet (below) previews and deploys fine on v8.1.0
Sample program
index.ts;
import * as gcp from '@pulumi/gcp';

export = async () => {

  const pythonServiceAccount = new gcp.serviceaccount.Account(
    'python-service-account',
    {
      accountId: `sa-python`,
    }
  );
};

gcp related vars like project and credentials are defined in a config Pulumi.stack.yaml
Log output
No response
Affected Resource(s)
No response
Output of pulumi about
❯ pulumi about (irrelevant values redacted via ###)
CLI
Version      3.122.0
Go Version   go1.22.7
Go Compiler  gc

Plugins
NAME        VERSION
cloudflare  5.39.0
docker      4.5.5
gcp         8.2.0
nodejs      unknown
postgresql  3.12.0
random      4.16.5

Host
OS       nixos
Version  24.05 (Uakari)
Arch     x86_64

This project is written in nodejs: executable='/etc/profiles/per-user/scott/bin/node' version='v20.15.1'

Current Stack: ###

TYPE                                      URN
pulumi:pulumi:Stack                       urn:pulumi:###
pulumi:providers:gcp                      urn:pulumi:###::pulumi:providers:gcp::default_8_2_0
gcp:serviceaccount/account:Account        urn:pulumi:###::gcp:serviceaccount/account:Account::python-service-account

Found no pending operations associated with ###

Backend
Name           pulumi.com
URL            https://app.pulumi.com/###
User           ###
Organizations  ###
Token type     personal

Dependencies:
NAME                VERSION
@pulumi/cloudflare  5.39.0
@pulumi/random      4.16.5
@types/node-forge   1.3.11
@pulumi/docker      4.5.5
@pulumi/gcp         8.2.0
@pulumi/postgresql  3.12.0
@pulumi/pulumi      3.133.0
@types/node         22.5.5
node-forge          1.3.1

Pulumi locates its logs in /tmp by default

Additional context
No response
Contributing
Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

Hi @scttl, thank you for filing this issue.
I could reproduce this issue with the following steps:

Set project: pulumi config set gcp:project <our-project>
Set credentials: pulumi config set gcp:credentials <path-to-file>--secret <-- this is the issue
Run pulumi up with v 8.2.0.

Additionally I've verified this does not panic on v8.1.0.
Furthermore, this does not panic on v8.2.0 unless the config value is set to --secret. But of course you need to be able to make your credentials value secret.
We're hoping to have a fix for this soon. Thank you again for reporting.

Seeing the same error elsewhere: https://github.com/pulumi/pulumi-hcloud/issues/603

Hi @scttl - we have reverted the bridge change that caused this behavior. This week's new GCP release will incorporate that version. I've verified with a local build that the changes address your issue.
Once pulumi-gcp v8.3.0 is merged and released you should be able to upgrade to that version.
Please let us know if you continue to see trouble - thank you very much for your patience here. 🙏

8.3.0 is released.
Please upgrade to 8.3.0. We apologize for the trouble.

Thanks @guineveresaenger ! v8.3.0 works well for me too.


Can SRWE cause problems with monitor settings???

Hello everyone. It's my first time using this site but I'm hoping somone can help me.
I downloaded simple runtime windows editor and I think that it may have changed my monitor settings. I now want to remove/uninstall it but I can't find it anywhere on my PC.
I have a tripple monitor setup and below are pics/screenshots after I stretch a window across into the 3 screens. Pics were taked on different days.

Any help appreciated.

SRWE changes the aspects of a running application only. So it might stretch a window/resize it, but that's it. As soon as the application exits, what srwe did is no longer active. So what you show here likely is caused by something else. did you cahnge the resolution in the windows settings perhaps?

I don't recall changing the resolution but I will check. Thanks for your heelp.

@Kerryman47 any update on this? If you figured out what the cause was it would be nice to be able to close this issue.


Question on a possible alternative to getting pkg/cli/environment/environment.go/Envsettings values from environment

Output of helm version: N.A
Output of kubectl version: N.A
Cloud Provider/Platform (AKS, GKE, Minikube etc.):  GKE
I see that Envsettings gets almost all of its values using os.Getenv. This doesn't appear thread safe when the client is used in an application which might access this functionality while serving multiple request threads/goroutines. Looking at the code of this repository and the code wtihin New() (Envsettings) function I wonder whether it is possible to allow the user to pass these values via a struct or some other means whose scope is limited to the particular block from where helm install is being performed.
Or is there some specific reason why this is done this way? Please enlighten me.

Do you mean func New() *EnvSettings {  ?
https://github.com/helm/helm/blob/e81f6140ddb22bc99a08f7409522a8dbe5338ee3/pkg/cli/environment.go#L93 ?
The EnvSettings type can be constructed any was as needed. This helper which reads from env vars doesn't needed to be used. This logic is roughy (I presume) from when the Helm SDK was quickly separated from the Helm CLI, but a "clean" separation was not entirely achieved.

This doesn't appear thread safe when the client is used in an application which might access this functionality while serving multiple request threads/goroutines

Env variables are process wide, yes. But typically they are not changed during a processes execution. Certainly updating a processes env vars during execution when additional threads/goroutines may be executing would/could lead to data race conditions (non-thread safe behavior)

Do you mean func New() *EnvSettings { ?
https://github.com/helm/helm/blob/e81f6140ddb22bc99a08f7409522a8dbe5338ee3/pkg/cli/environment.go#L93
?
The EnvSettings type can be constructed any was as needed. This helper which reads from env vars doesn't needed to be used. This logic is roughy (I presume) from when the Helm SDK was quickly separated from the Helm CLI, but a "clean" separation was not entirely achieved.

This is what I too thought before looking deeper into the code. As you can see in , the namespace variable is private.
Similarly here you can see another value being retrieved from env.
There is no way around these unless we maintain a separate fork of this client for our organization. I was wondering whether it will be a good idea to refactor code and raise a PR which addresses this.

This doesn't appear thread safe when the client is used in an application which might access this functionality while serving multiple request threads/goroutines

Env variables are process wide, yes. But typically they are not changed during a processes execution. Certainly updating a processes env vars during execution when additional threads/goroutines may be executing would/could lead to data race conditions (non-thread safe behavior)

Exactly what I was saying. We have a service which handles requests from its many consumers for provisioning and managing kube resources. These actions are initiated by end users through a UI and hence race conditions can of course occur. Since this is such a basic issue, I was wondering if there are other valid reasons for getting these vals from environment or whether I can try working on a fix to change this behaviour (such that it can be set both from env as well as by passing these vals in as arguments).

What I understood from the thread:
There are some inputs that are being pulled in from the environment variables and doesn't seem to be thread safe. This makes it hard to use it in a service that has multiple threads serving multiple requests from clients
Just my 2 cents on this, based on what I have understood:
First of all, it's absurd for me that there's a cli directory in the pkg directory. I would have expected all CLI specific logic to go inside cmd, maybe some pkg inside cmd or something like that
Second, I think the service should not use any of the cmd or cli packages 📦 ideally. Those are for the CLI. The servicce should depend on the raw package 📦  or library and try to do what the CLI packages do, without any dependency on the CLI related packages. Ideally this should be possible. If this is not possible, maybe the project needs some refactoring, hopefully in a non-breaking way, because there would be many people depending on the Helm packages 📦 (SDKs)


Show inventory history for pre-event read

Fix for #4433 .
This will show inventory history regardless of whether the time selected had events active or not. If the time is before events were active, it will use inventory item versions (using paper_trail).

Hey @awwaiid I'll be reviewing this functionally tomorrow (hopefully pre-meeting).  I fully expect it to work.   Could you do a technical review in parallel?

Looks like it works, ish-- the totals are wrong,  and they are wrong for everyone on the production branch as well - i suspect it's just showing the current total inventory and that it has always been wrong but haven't looked into it.   That is out of scope for this issue, I guess, but I would not be at all averse to including a fix for that in.

Fixed for the events_read cases (both within event time and before it).

LGTM.  Waiting for @awwaiid's review.


[CodeCov] UI Components

There are already test cases for classes without javafx-related code ie UiPart and CommandHistory.
As for the rest of the classes under ui, I considered using testfx, but there seems to be some conflicts between CI and testfx test cases (see #363 in forum).
Does anyone has any suggestions on how to test classes involving javafx without using testfx?


mlx90640_thermal_camera: 1.0.0-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository mlx90640_thermal_camera to 1.0.0-1:

upstream repository: https://github.com/vakshit/mlx90640_thermal_camera.git
release repository: https://github.com/vakshit/mlx90640_thermal_camera-release.git
distro file: noetic/distribution.yaml
bloom version: 0.11.2
previous version for package: null

I have updated the package name. and also merged the master branch of the main repo.

So, i need to do the bloom-release part again right?
Also, just to keep things consistent, I need to use _ instead of - in the package name and the debian name would be taken care of by the buildfarm.

So, i need to do the bloom-release part again right? Also, just to keep things consistent, I need to use _ instead of - in the package name and the debian name would be taken care of by the buildfarm.

Yes, exactly right on both counts.

I guess i need to raise a new PR because the name is with - in the current branch name.

Closing this in favor of #38558


Running docker containers as non-root user

What this PR does / why we need it:
I've added a user to the controller and ingress-shim Dockerfiles. The processes don't need root access so I figured I might as well limit it.

My only concern here is around glog attempting to write a log file to disk (which we don't want it to do anyway, but I know it sometimes does bad things, because glog is bad 🙃)
/ok-to-test

That's a good point, I didn't test myself with logfiles enabled so that might fail.

/retest

@munnerz By default, glog logs in /tmp, and that should be an o+rwx directory, so I don't think it should cause issues. Even in a scratch container I think docker mounts a tmpfs with appropriate permissions there.
If a user configures the log-dir away from default, it seems reasonable to also expect them to correctly permission it.

/retest

/retest

What's up with your 1.9 environment?

We were hitting some issues relating to Helm timeouts (see #429)
/lgtm
/approve


Following guidance docs, fails uploading stemcell

I got to this doc: https://github.com/cloudfoundry-incubator/bosh-azure-cpi-release/blob/master/docs/get-started/deploy-bosh-using-arm-templates.md
And deployed my dev box by clicking the button.  I left tenantID as TENANT-ID and clientID as CLIENT-ID, and set clientSecret to some new secret I made up.  The dev box provisioned fine, and then I tried to run ./deploy_bosh.sh.  I did not update ~/bosh.yml because it's not clear what values I need to change, and what they should be changed to.
The error I'm seeing is:
agupta@devbox:~$ ./deploy_bosh.sh
Deployment manifest: '/home/agupta/bosh.yml'
Deployment state: '/home/agupta/bosh-state.json'

Started validating
  Downloading release 'bosh'... Skipped [Found in local cache] (00:00:00)
  Validating release 'bosh'... Finished (00:00:03)
  Downloading release 'bosh-azure-cpi'... Skipped [Found in local cache] (00:00:00)
  Validating release 'bosh-azure-cpi'... Finished (00:00:00)
  Validating cpi release... Finished (00:00:00)
  Validating deployment manifest... Finished (00:00:00)
  Downloading stemcell... Skipped [Found in local cache] (00:00:00)
  Validating stemcell... Finished (00:00:21)
Finished validating (00:00:26)

Started installing CPI
  Compiling package 'ruby_azure_cpi/3db71123fb72f5ec81955710b2e89e2cbbd8aca0'... Finished (00:00:00)
  Compiling package 'bosh_azure_cpi/5985e1e82c78fadb5f2c951f319012403ee04fd8'... Finished (00:00:00)
  Installing packages... Finished (00:00:11)
  Rendering job templates... Finished (00:00:03)
  Installing job 'cpi'... Finished (00:00:00)
Finished installing CPI (00:00:15)

Starting registry... Finished (00:00:00)
Uploading stemcell 'bosh-azure-hyperv-ubuntu-trusty-go_agent/0000'... Failed (00:00:01)
Stopping registry... Finished (00:00:00)
Cleaning up rendered CPI jobs... Finished (00:00:00)

Command 'deploy' failed:
  creating stemcell (bosh-azure-hyperv-ubuntu-trusty-go_agent 0000):
    CPI 'create_stemcell' method responded with error: CmdError{"type":"Bosh::Clouds::CloudError","message":"http_get_response - get_token - http error: 400\n/home/agupta/.bosh_init/installations/2a344a15-6309-449a-5e8c-266083aa014f/packages/bosh_azure_cpi/lib/cloud/azure/azure_client2.rb:842:in `get_token'\n/home/agupta/.bosh_init/installations/2a344a15-6309-449a-5e8c-266083aa014f/packages/bosh_azure_cpi/lib/cloud/azure/azure_client2.rb:861:in `http_get_response'\n/home/agupta/.bosh_init/installations/2a344a15-6309-449a-5e8c-266083aa014f/packages/bosh_azure_cpi/lib/cloud/azure/azure_client2.rb:1013:in `http_post'\n/home/agupta/.bosh_init/installations/2a344a15-6309-449a-5e8c-266083aa014f/packages/bosh_azure_cpi/lib/cloud/azure/azure_client2.rb:792:in `get_storage_account_keys_by_name'\n/home/agupta/.bosh_init/installations/2a344a15-6309-449a-5e8c-266083aa014f/packages/bosh_azure_cpi/lib/cloud/azure/table_manager.rb:13:in `initialize'\n/home/agupta/.bosh_init/installations/2a344a15-6309-449a-5e8c-266083aa014f/packages/bosh_azure_cpi/lib/cloud/azure/cloud.rb:367:in `new'\n/home/agupta/.bosh_init/installations/2a344a15-6309-449a-5e8c-266083aa014f/packages/bosh_azure_cpi/lib/cloud/azure/cloud.rb:367:in `init_azure'\n/home/agupta/.bosh_init/installations/2a344a15-6309-449a-5e8c-266083aa014f/packages/bosh_azure_cpi/lib/cloud/azure/cloud.rb:19:in `initialize'\n/home/agupta/.bosh_init/installations/2a344a15-6309-449a-5e8c-266083aa014f/packages/bosh_azure_cpi/bin/azure_cpi:26:in `new'\n/home/agupta/.bosh_init/installations/2a344a15-6309-449a-5e8c-266083aa014f/packages/bosh_azure_cpi/bin/azure_cpi:26:in `block in \u003cmain\u003e'\n/home/agupta/.bosh_init/installations/2a344a15-6309-449a-5e8c-266083aa014f/packages/bosh_azure_cpi/vendor/bundle/ruby/2.1.0/gems/bosh_cpi-1.3100.0/lib/bosh/cpi/cli.rb:70:in `call'\n/home/agupta/.bosh_init/installations/2a344a15-6309-449a-5e8c-266083aa014f/packages/bosh_azure_cpi/vendor/bundle/ruby/2.1.0/gems/bosh_cpi-1.3100.0/lib/bosh/cpi/cli.rb:70:in `run'\n/home/agupta/.bosh_init/installations/2a344a15-6309-449a-5e8c-266083aa014f/packages/bosh_azure_cpi/bin/azure_cpi:34:in `\u003cmain\u003e'","ok_to_retry":false}

When I look at run.log, I see:
[Cmd Runner] 2015/12/25 01:49:18 DEBUG - Stderr: I, [2015-12-25T01:49:17.892396 #61489]  INFO -- : http_post - trying to post https://management.azure.com//subscriptions/<REDACTED>/resourceGroups/<REDACTED>/providers/Microsoft.Storage/storageAccounts/<REDACTED>/listKeys?api-version=2015-05-01-preview
I, [2015-12-25T01:49:17.892683 #61489]  INFO -- : get_token - trying to get/refresh Azure authentication token
E, [2015-12-25T01:49:18.064396 #61489] ERROR -- : http_get_response - get_token - http error: 400

When I try to curl the URL, I get a 401 error because I'm not setting the Authorization headers, which makes sense.  But the deploy is getting 400, so I assume it's setting the Authorization headers fine, and there's something else malformed about the request, but I don't know what it is, and can't see the request payload in any log output.
I wouldn't be surprised if my bosh.yml manifest needs some values changed, but I don't know what would need to be changed.  The docs say:

If you leave TENANT-ID, CLIENT-ID, CLIENT-SECRET at default values in section 1, you need to update these three properties in ~/bosh.yml.

But update the values to what?

You need to create a service principle by following this [guide] (https://github.com/cloudfoundry-incubator/bosh-azure-cpi-release/blob/master/docs/get-started/create-service-principal.md). Then you need to update those three parameters in bosh.yml. And you also need to do below manual steps by following the [guide] (https://github.com/cloudfoundry-incubator/bosh-azure-cpi-release/blob/master/docs/get-started/deploy-bosh-manually.md).
1.2.3 Create Containers
1.2.4 Create Tables

Thanks, I had done the service principal step already, but the documentation didn't make it clear what values for CLIENT-ID, etc. I was supposed to fill in those forms.  I also never noted down my CLIENT-ID so I had to do azure ad sp list and find the one corresponding to the input I gave to the command when I ran azure ad create app.  Seems to be working now, thanks.

@AbelHu I'm not deploying on BOSH manually, so would I still need to do those manual steps of creating tables/containers?

@Amit-PivotalLabs I have checked the steps. You can skip it because the template has used storage account/key to create containers and tables automatically.


Check keystore field when export Android release apk

Check Keystore section when export Release apk.
Can create keystore file if want
Keystore path, user name and password is filled automatically after creating keystore

Updated to be able to create a keystore file while exporting for android release apk.

Holy Moly! This is a Game Changer.  :+1:   :trophy:
My suggestion is :
a) We need to add -storepass field, some people prefer using different password for both -keypass and storepass. Because it can be a problem if the keystore was generated using different password for keypass and storepass.
b) I would rename "Keystore" section to "Release Keystore" (to differentiate it from Debug Keystore) and also rename A to B to make it more understandable.
[ A ] (before)
Release :
Release User :
Release Password :
[ B ] (after)
Key Path :
Key Alias :
Key Password :
Store Password* :
*: if only -storepass field [point (a)] added

@ScotFlux I think your suggestion is pretty reasonable.
I will update soon.

@ScotFlux I read again your suggestion and noticed it will break backward compatibility.
@akien-mga May I break compatibility since this is 3.0 milestone?
or make 2 commits for 2.1.x and 3.0?

I reconsidered it and @ScotFlux how about leave it as now simple?
I can add more later if needed.

@volzhs It's still great. Though, make commits for 2.1.x and 3.0 could be better as less confuse -> less trouble -> less complain haha lol. But, I could be wrong.


[WIP]Cleaner code for flash-taskbar

The changes don't make the code more readable.

it adds more code and an extra variable with no gain
the duration length variable doesn't store the duration length. Instead it is a "boolean" (2500 means true and 0 means false).
there is an empty {} block


MethodArgumentSpaceFixer - Regression

bug is in the current 2.12 branch, but is not released yet
I think this is regression after https://github.com/FriendsOfPHP/PHP-CS-Fixer/commit/75b55b529e09b32f3d3215b4429939154e0c5fab
causes 3.0 currently to fail as well
ping @keradus

@SpacePossum I tried to rebase your branch on latest 2.15 and I noticed it contains the same test but with the change being expected. I would say the current behavior of 2.15 is correct and this PR can be closed.

thank you @julienfalque !


Add uptime monitoring for lbry.tv

@nikooo777 did you do something basic for this? Can you? This would be for both lbry.tv web server itself and api.lbry.tv.

@sayplastic saw you just merged the status endpoint PR. are you handling this issue? is there anything i can contribute?

I created an endpoint for @nikooo777, I think he will take it from here. Let me know if any improvements are needed.

As mentioned on slack, i can only do string matching from uptimerobot
id appreciate if you could add a top level key that reports the general status:
if everything is "ok" then just have it report:
"general_state": "ok"

Status endpoint is ready: https://api.lbry.tv/internal/status

@andybeletsky should lbry.tv be updated so that when general_state != ok  we should display an alternative/status/issue page? If not that specifically, can you clarify how endpoint could/should be used?

The idea was to integrate this into https://status.lbry.com — @nikooo777 is on it I believe.
There's nothing stopping us from displaying this somewhere in the UI of lbrytv. We can even display it in case of SDK troubles on the instance the current user is on but it will require more work from the app team.

First step is for @seanyesmunt to display indicator (nag? but different color) when performance is degraded. Then assign back to @andybeletsky for further development.


Response type "code id_token" not working

Describe the bug
When AuthConfig.response_type='code id_token', implicit flow is implied wrongly in :

loadDiscoveryDocumentAndTryLogin
tryLogin
initLoginFlow
Even if initCodeFlow and tryLoginCodeFlow is used directly, the PKCE is not sent.

It seems that all of the conditions responseType === 'code' should be replaced by responseType.indexOf('code') != -1 in order to request multiple response type in the authorize request (like code id_token).

The PR 809 address the second point of the bug (PKCE not sent when response type is multi-valued and contains the code response type).

code id_token is Hybrid Flow which is not supported by this library.


Add timeouts to requests in Freshmaker

Add configurable timeout to requests to avoid FM from hanging.

@qixiang , @apaplaus, please review.

LGFM, just address the comment from qwan

LGTM


update dll from  NLPIR commit

https://github.com/NLPIR-team/NLPIR/commit/aaac8732931e0a2cdb2ccdb7942d046e44402a5f
https://github.com/NLPIR-team/NLPIR/commit/6a0d87892fd6a49f126f2321135ac1874a5a0919
https://github.com/NLPIR-team/NLPIR/commit/b66956a89997628c18f452cd77ca143ddff36f30
https://github.com/NLPIR-team/NLPIR/commit/83ae60c617964034efa1e2fa77ede46f7bf44cf4
https://github.com/NLPIR-team/NLPIR/commit/25bece8a512554dba41c36ad4f2733ab42048190

https://github.com/NLPIR-team/NLPIR/commit/25bece8a512554dba41c36ad4f2733ab42048190 change the output json in Detail

But we are not using this , so leave it in another issues

Most of the commits is arm-based and loongson-based, ignore it


StreamDeck not detected

StreamDeck not detected
i tried with official steam deck app not running, and running, both.
It detects well my saitek panel, but not the stream deck
Is it normal ?

It is really hard to say what is the issue. Try and connect it directly to the PC.

It should detect the Streamdeck just as it detects the Saitek panels.


Plans to add multipart requests support?

What do you think about adding multipart/form-data requests that are made with -F option in curl?
It would be awesome for testing endpoints for file uploads that are located on API Gateways with AWS Auth enabled.
Example [aws]curl command:
awscurl \
        -X POST \
        -H "Content-Type: multipart/form-data" \
        -F "field1=value1" \
        -F 'field2={"some":"json"};type=application/json' \
        -F<EMAIL_ADDRESS>\
        https://redacted.execute-api.us-west-2.amazonaws.com/path

Expected request body:
--------------------------d9afcbc8f3fe869a
Content-Disposition: form-data; name="field1"

value1
--------------------------d9afcbc8f3fe869a
Content-Disposition: form-data; name="field2"
Content-Type: application/json

{"some":"json"}
--------------------------d9afcbc8f3fe869a
Content-Disposition: form-data; name="file"; filename="picture.png"
Content-Type: image/png

<binary data>
--------------------------d9afcbc8f3fe869a--

Hi @Envek ,
That looks interesting, but I don't have a good knowledge of this part of curl. I also don't have a platform for testing multipart/form-data.
But the project is always open for contributions, so I would be glad to review the Pull-Request implementing this feature


Revert "MM-37916: close product switcher on item click (#8594)"

Summary
This reverts #8594 since it added a bug to the existing webapp.
Adding UX and QA to check for correct restoration of previous behaviour.
Ticket Link
MM-37916
Tickets reporting this bug:
MM-38019
MM-38056
MM-38057
Related Pull Requests
#8594
Release Note
NONE

@andrewbrown00 @jgilliam17 This would be good to include for this Wednesday's cloud release.
@michelengelen Nevy is off this week so another dev may need to be tagged for this.

Hey, @michelengelen, I'm not sure what I'm supposed to review on this PR. I can probably be removed as I've already approved the actual PR to fix the closing popover on item click https://github.com/mattermost/mattermost-webapp/pull/8678.

@andrewbrown00 it is regarding the other popovers mentioned in the issue list. Although there really might be no need for UX review, you are right. I added to make sure that all the other Popovers are working as expected.

@michelengelen TimedDND feature flag is true, but the test server does not appear to have the option to set custom time. Can you please take a look? One of the tickets is for the time selector on DND. Thanks

it works for me:

oh, that's the custom status ... sry ... i am having a look now.

@jgilliam17 it works on my local instance ... just to be sure I rebased the branch onto the current master ... if that does not fix it we should try it with a CWS server.

Not present on this server, I'll create a CWS one

@michelengelen not working on CWS server either, not sure what the issue could be

Cherry pick is scheduled.

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
From github.com:mattermost/mattermost-webapp
   ed744a9bd..b9080b18d  MM-37859_fix-product-switcher-menu-items -> upstream/MM-37859_fix-product-switcher-menu-items
   825b5b5b3..3b6586488  cloud      -> upstream/cloud
   1b5405a6b..2835176cd  custom-status-class-typo -> upstream/custom-status-class-typo
   346c6edd3..485a679e7  master     -> upstream/master
Fetching origin
Failed to add the RSA host key for IP address '<IP_ADDRESS>' to the list of known hosts (/app/.ssh/known_hosts).
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-mattermost-webapp-#8677-upstream-release-6.0-1629821292
Switched to a new branch 'automated-cherry-pick-of-mattermost-webapp-#8677-upstream-release-6.0-1629821292'
Branch 'automated-cherry-pick-of-mattermost-webapp-#8677-upstream-release-6.0-1629821292' set up to track remote branch 'release-6.0' from 'upstream'.

+++ About to attempt cherry pick of PR #8677 with merge commit 485a679e78456b3145c5b61ab89a0737a0ac03ea.

error: could not apply 485a679e7... Revert "MM-37916: close product switcher on item click (#8594)" (#8677)
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add <paths>' or 'git rm <paths>'
hint: and commit the result with 'git commit'
!!! git cherry-pick failed

+++ Aborting in-progress git cherry-pick.

+++ Returning you to the master branch and cleaning up.

/cherry-pick cloud

Cherry pick is scheduled.


Namespace conflict when running tests

[x] Possible documentation mistake in testing and configuration?

Given this config in composer.json:
"extra": {
        "phel": {
            "loader": {
                "hello-world\\": "src/"
            },
            "loader-dev": {
                "hello-world\\": "tests/"
            },
            "tests": [
                "tests/"
            ]
        }
    },

And these files:
src/boot.phel:
(ns hello-world\boot)

(println "Hello, World!")

tests/boot.phel
(ns hello-world\boot
    (:require phel\test :refer [deftest is]))

(deftest test-variable
    (is (= 4 (+ 2 2))))

When running vendor/bin/phel test, I get:
Hello, World!

Passed: 0
Failed: 0
Error: 0
Total: 0

but when I rename the test file to test.phel (and update the namespace to match) or anything other than what a file is called in src directory it works.
.

Passed: 1
Failed: 0
Error: 0
Total: 1

As far as I understand we are telling people in the documentation to use hello-world namespace for both loader and loader-dev. That is why this is happening. I'm going to send a PR to change loader-dev namespace in the documentation to hello-world\\tests\\.

What would you want to happen in this case? Should there be a error? I think the same happens in PHP if you use the PSR-4 Autoloader. If you have two files with the same namespace in different location only the first one found is loaded.

I think that by changing the documentatiom we have avoided this issue.
Best to leave it alone for now because it is easy to catch the mistake. Maybe investigate a bit how it works in PHP.

I'm with you @mabasic. This PR https://github.com/jenshaase/phel-lang/pull/92 was a great idea :)


Segment Indexing (keys)

Problem:
one observation from testing out segmentation library last week is that segments are hard to reference by key. Different segmentation methods (e.g., create_segment vs generate_segments) inconsistently collected and apply human-readable segment names as segment dict.keys() making some iteration and reference methods challenging to use.
Suggestion:
Allow a segment name stem as an optional, 'str' argument that can be used to create logical, human-readable labels even with the number of segments to be generated is not known.
Alternative is to advocate methods like dictionary[new_key] = dictionary.pop(old_key) in iteration, which can be cumbersome in usage, requiring additional user code to modify both segment_names and dict.keys(), which require iteration/comprehension
Examples:
using distill.create_segment requires passing in a list of segment names as an arg to the method. It results in a dictionary of segments with .keys() that match segment_names. That makes methods for accessing or iterating through instances of segments very easy:
superSegment['toggle1'].num_logs

However, methods like distill.generate_segments, distill.detect_deadspace work differently. These source uid's from segment_names.append(key_list[i]), which means that keys in source log dicts (which may be completely arbitrary) become keys when segments are initially written. See here:
deadSpaceSegments = distill.detect_deadspace(sorted_data_paths, 360, 0, 0) # call detect_deadspace``

for d in deadSpaceSegments.values(): # iterate through segment object instances print values
    print(d.segment_name, d.segment_type)

# returns

840 Segment_Type.DEADSPACE
991 Segment_Type.DEADSPACE
1007 Segment_Type.DEADSPACE
1121 Segment_Type.DEADSPACE
1231 Segment_Type.DEADSPACE
3040 Segment_Type.DEADSPACE

Renaming segments doesn't reassign keys:
for counter, d in enumerate(deadSpaceSegments.values(), start=1):
    d.segment_name = str("deadSpace" + str(counter)) #renaming segment names on the fly
    d.segment_length_sec = (d.start_end_val[1] - d.start_end_val[0])/1000 #adding custom segment-object attributes
    print(d.segment_name, d.segment_type, d.start_end_val, d.segment_length_sec, d.num_logs, d.uids)

# returns

deadSpace1 Segment_Type.DEADSPACE<PHONE_NUMBER>069,<PHONE_NUMBER>897) 68739.828 2 [840, 924]
deadSpace2 Segment_Type.DEADSPACE<PHONE_NUMBER>598,<PHONE_NUMBER>182) 745.584 2 [991, 990]
deadSpace3 Segment_Type.DEADSPACE<PHONE_NUMBER>927,<PHONE_NUMBER>776) 18469.849 3 [1007, 674, 1126]
deadSpace4 Segment_Type.DEADSPACE<PHONE_NUMBER>632,<PHONE_NUMBER>282) 64270.65 6 [782, 981, 1121, 1222, 1227, 1241]
deadSpace5 Segment_Type.DEADSPACE<PHONE_NUMBER>419,<PHONE_NUMBER>250) 18456.831 3 [1231, 1426, 1589]
deadSpace6 Segment_Type.DEADSPACE<PHONE_NUMBER>587,<PHONE_NUMBER>931) 1300376.344 3 [3040, 3049, 3061]

# but:

deadSpaceSegments.keys()

# returns:

dict_keys([840, 991, 1007, 1121, 1231, 3040, 3018, 3582, 5053, 7238, 7262, 7281, 7287, 7747, 8147, 8794, 8682, 8956, 8938, 10647, 11876, 11783, 11838, 12294, 12292, 12302])

@krassmann12 and @Grtnation had a better idea.


[C++] Thrift static libraries are not used with recent patch

Bug introduced in https://github.com/apache/parquet-cpp/pull/285
Reporter: Deepak Majeti / @majetideepak
Assignee: Deepak Majeti / @majetideepak
Note: This issue was originally created as PARQUET-945. Please see the migration documentation for further details.

Wes McKinney / @wesm:
Issue resolved by pull request 290
https://github.com/apache/parquet-cpp/pull/290


fix: add missing .next directory to docker builds where required

Description
Fixes #3992 - missing .next directory from build container.
Additionally, add .dockerignore entries to avoid package version conflicts.

[x] I have read and understand the CONTRIBUTING.md document in this repository.

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

For the the templates that use next, the .next directory is generated by the build process and is required for next based projects. otherwise it fails on start with this issue 3992. Therefore I think it should error rather than fail silently when the .next directory is not generated during the next build process.
For cases where next is no longer used (as I understand this comment alludes to) I think an alternative solution would be to update the eject script to remove the COPY --from=builder /home/node/app/.next ./.next line when next is removed from the template.
(FYI I'm just a (new) member of the community, so it would be better for someone in the dev team to have a say on this).

Therefore I think it should error rather than fail silently when the .next directory is not generated during the next build process.

I agree with this. I think having to create the .next directory in the Dockerfile is masking the larger issue of not having built properly.
@shutupflanders Thanks for the PR, and I'm open to an alternative solution. However, I don't think this is the right one.

@shutupflanders Thanks for the PR. I'd rather not have the .next dir created as that could mask a build issue.
If you update this, I'd be willing to merge it.


Update TSC to current voting members

@KennyPaul please verify
Fixes https://github.com/nephio-project/nephio/issues/358

Confirmed.
Thanks!


Make Buffer:add_utf8 support item_offset and item_length.

Added Buffer:get_length which wraps hb_buffer_get_length.
Added docs.
Version bump.
Fix #20.

@khaledhosny Leaving it here for a few hours, in case you wanna take a look. Will merge after that.


[BUG] setActive only works when link components wrapped with  tags

Here is my code,
I found out that if I'm not wrap the <Link> component with
, I will never get the onSetActive works,
but I'm able to click and scroll to the section I want
const formItemScrollProps: Partial<ReactScrollLinkProps> = {
  containerId: 'form-container',
  duration: 500,
  smooth: true,
  spy: true,
  onSetActive(to: string, element: HTMLElement) {
      console.log(to)
  }
};

const scrollToItem: Record<PartFormSection, ReactScrollLinkProps> = {
    [PartFormSection.PART]: {
        to: PartFormSection.PART,
        ...formItemScrollProps
    },
    ......
};

import {Element, Link as ScrollTo} from "react-scroll";

<ScrollTo
    className={PartFormSection.PART} {...scrollToItem.PART}>{PartFormSection.PART}</ScrollTo>
<ScrollTo
    className={PartFormSection.PRINT_JOB} {...scrollToItem.PRINT_JOB}>{PartFormSection.PRINT_JOB}</ScrollTo>
.........

but this works

    <ScrollTo className={PartFormSection.PART} {...scrollToItem.PART}>{PartFormSection.PART}</ScrollTo>

    <ScrollTo className={PartFormSection.PRINT_JOB} {...scrollToItem.PRINT_JOB}>{PartFormSection.PRINT_JOB}</ScrollTo>

what I want to say is, the document didnt stated about this, I guess this is a BUG
can anyone please explain, thanks

and one more thing, sometimes it works, sometimes not works, the active class will not added even I clicked on the item.


turbo watch doesn't start tasks in order

Verify canary release

[X] I verified that the issue exists in the latest Turborepo canary release.

Link to code that reproduces this issue
https://codesandbox.io/p/devbox/d4gmxh?migrateFrom=fxlctj
What package manager are you using / does the bug impact?
Yarn v2/v3/v4 (node_modules linker only)
What operating system are you using?
Mac
Which canary version will you have in your reproduction?
2.0.11 (there is no canary after)
Describe the Bug
When starting a task, let's say dev, in watch mode that depends on some packages with build tasks and dev is a persistent task, it will start immediately without waiting for its dependencies to build. This is quite annoying since I run a Next.js app in dev and it imports something in the instrumentation hook that depends on a package being built. But since the server starts immediately it will always throw because Next.js doesn't recover from failures in instrumentation.
Expected Behavior
I would expect it to behave like running without watch mode enabled, i.e. that all tasks waits for their dependencies to be fulfilled before starting even though they're persistent.
To Reproduce
In the reproduction example, run the following in the terminal:
yarn dlx turbo watch build --filter=app-a...

Notice how the build task for the app-a is started immediately even though it has dependencies that needs to be built.
Additional context
No response

Duplicate of #8673


error: no bin target named $CRATE

The latest version seems to have broken something. It returns this error message:
ACTUAL OUTPUT:
┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈
error: no bin target named `$CRATE`

        Did you mean `trybuild000`?
┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈┈

This error happens when any of the previous tests fail. I close it.

Okay, this does very strange things. It is random and is taking the previous path. I don't know but it seems like I break everything. But it is a line, it is difficult to break it, I would look at it in detail.


Auto-updates removes old config/plans

Updating through the auto updater in-app removes previously created plans and settings

Yeah, which is why the updates always end with: "Make sure to export your config from the dropdown menu if you don't want to lose your settings!".. Although yeah I do plan on adding persistent configuration asap.

Yeah, which is why the updates always end with: "Make sure to export your config from the dropdown menu if you don't want to lose your settings!".. Although yeah I do plan on adding persistent configuration asap.


LV translation fixes

Q
A

Is bugfix?
❌

New feature?  ❌

Breaks BC?
❌

Tests pass?
-

Fixed issues
#18018

@Deele looks good?

@samdark yes
@uldisn Paldies! Thanks!

Thank you!


fs/mount: move mountpoint inode create after bind

Summary
fs/mount: move mountpoint inode create after bind
This can avoid the issue of invalid inode access when bind is not yet completed.
Impact
Bug fix
Testing
Vela

[Experimental Bot, please feedback here]
This PR appears incomplete based on the provided NuttX requirements. Here's why:

Summary Lacks Details:

What functional part is changed?  Specifying "fs/mount" is a start, but what specifically within that area is modified?
How does the change work? "move mountpoint inode create after bind" is the idea, but a technical explanation of the existing problem and how moving the code solves it is missing.

Impact is Too Brief:

Bug fix: What bug? Provide the issue number or a concise description of the bug being fixed.
Other Impacts:  While this specific change might not affect other areas, it's good practice to go through each and confirm (even if it's "NO"). This demonstrates you've considered potential side-effects.

Testing is Insufficient:

Build Host: Be specific about your development environment.
Target: "Vela" is a board. Specify the architecture and configuration.
Logs:  "your testing logs here" isn't helpful. Provide actual logs demonstrating the problem before the change and the successful outcome after.

To improve this PR:

Expand the Summary:  Provide more context and technical details about the change.
Complete the Impact Assessment:  Address all points, even if it's to say "NO" with a brief justification.
Provide Detailed Testing Information: List your build host setup, the exact target configuration, and relevant logs showing the bug and the fix.

[Experimental Bot, please feedback here]
This PR appears incomplete based on the provided NuttX requirements. Here's why:

Summary Lacks Details:

What functional part is changed?  Specifying "fs/mount" is a start, but what specifically within that area is modified?
How does the change work? "move mountpoint inode create after bind" is the idea, but a technical explanation of the existing problem and how moving the code solves it is missing.

Impact is Too Brief:

Bug fix: What bug? Provide the issue number or a concise description of the bug being fixed.
Other Impacts:  While this specific change might not affect other areas, it's good practice to go through each and confirm (even if it's "NO"). This demonstrates you've considered potential side-effects.

Testing is Insufficient:

Build Host: Be specific about your development environment.
Target: "Vela" is a board. Specify the architecture and configuration.
Logs:  "your testing logs here" isn't helpful. Provide actual logs demonstrating the problem before the change and the successful outcome after.

To improve this PR:

Expand the Summary:  Provide more context and technical details about the change.
Complete the Impact Assessment:  Address all points, even if it's to say "NO" with a brief justification.
Provide Detailed Testing Information: List your build host setup, the exact target configuration, and relevant logs showing the bug and the fix.

Before the cross-core filesystem rpmsgfs successfully binds, the corresponding mount point inode should not be created, because if the mounting process is time-consuming, or if during this period an application happens to access the corresponding inode, the information of this inode will be incorrect. This PR is aimed at addressing this issue. Thank you.


add older atlassian images

jira 6.0+ supported java7
try tu use zulu-build
https://www.azul.com/downloads/zulu-community/?&version=java-7-lts&os=&os=debian&architecture=x86-64-bit&package=jdk

before add older jdk 6 and 7
https://github.com/EpicMorg/docker-scripts/issues/41

+confluence 5.5
+jira 6.3.15


Unix socket URLs don't work with PostgreSQL

Bug Description
The bug seems to be the same as #1202 but it was marked as completed with a hack to bypass the defect instead of an actual fix.
I have a PostgreSQL server running locally with an account for my Unix username. According to the PostgreSQL specification, I should be able to connect to it using postgres:///my_database as a connection URL. But sqlx refuses to do that at compile-time and says error: empty host. The host definitely is empty, but that shouldn't be a problem.
Minimal Reproduction
.env:
DATABASE_URL=postgres:///database

Have a query! invocation in your Rust code, and it won't compile.
Temporary fix
Write the explicit URL-encoded path to the Unix socket as your hostname.
Info

SQLx version: 0.7.1
SQLx features enabled: ["postgres", "runtime-tokio"]
Database server and version: PostgreSQL 14
Operating system: Linux 5.15.0-84-generic
rustc --version: 1.72.0

The bug is likely at this line:  https://github.com/launchbadge/sqlx/blob/main/sqlx-postgres/src/options/parse.rs#L16
Which unconditionally overwrites the default host set in new_without_pgpass(): https://github.com/launchbadge/sqlx/blob/main/sqlx-postgres/src/options/mod.rs#L146
It should only overwrite options.host() if the host part from the URL is nonempty.

The bug is likely at this line: https://github.com/launchbadge/sqlx/blob/main/sqlx-postgres/src/options/parse.rs#L16

That should not be the problem since for the parsed string "postgres:///database" url.host_str() returns None so that line does not get executed.
@edgarogh where does your socket live? The issue may be that the default is not where your socket is. In fact, while Postgres defaults only to /tmp if the host is empty (in linux) sqlx actually looks for a socket in (in order) /var/run/postgresql, /private/tmp and /tmp before defaulting to using a host of localhost. So, using sqlx version 0.7.1, for the following piece of code:
use sqlx::PgPool;

#[tokio::main]
async fn main() -> Result<(), sqlx::Error> {
    let url = std::env::var("DATABASE_URL").unwrap();
    let pool = PgPool::connect(&url).await?;
    let row = sqlx::query!("SELECT 1 as int").fetch_one(&pool).await?;
    assert_eq!(row.int, Some(1));

    Ok(())
}

The first error I get (before creating a psql socket) is a connection error, as my DATABASE_URL (set to postgres:///database) resolves to postgres://localhost/database;
After creating a socket in /var/run/postgresql it complains the database does not exist; and finally after creating it, this code works.
At no point I'm getting a "host empty" error, so would be nice to have more info on that.

When I stop the PostgreSQL systemd service and run psql, it fails with:
psql: error: connection to server on socket "/var/run/postgresql/.s.PGSQL.5432" failed: No such file or directory
	Is the server running locally and accepting connections on that socket?

So I guess that my socket listens on /var/run/postgresql/.s.PGSQL.5432. As you might expect, when the service is up, the aforementioned socket exists and psql with no arguments does log me in with my UNIX user (with peer auth). I'm on Linux Mint (Ubuntu/Debian/APT-based) and have never changed PostgreSQL's config at all.


Update README.md to point to new org

What kind of PR is this?:

/kind chore

Codecov Report

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

:exclamation: Current head 91a7cd4 differs from pull request most recent head bd1a41e. Consider uploading reports for the commit bd1a41e to get more accurate results

@@           Coverage Diff           @@
##             main     #126   +/-   ##
=======================================
  Coverage        ?   97.33%
=======================================
  Files           ?       14
  Lines           ?      488
  Branches        ?      139
=======================================
  Hits            ?      475
  Misses          ?        8
  Partials        ?        5

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 99f1d48...bd1a41e. Read the comment docs.


[1.10.2] RF not transferring through universal pipes

i have an ender io capacitor attached to a uni pipe from mek to a sag mill and the rf is not transferring. the capacitor is set to out put to the appropriate side. Do they still transfer RF? the tool tip says they can i assume yes.

This is due to a lack of Tesla integration by EnderIO.  Hopefully they'll update soon!


Use the same favicon for SNL site as openliberty.io

Currently, our SNL portal uses the round favicon.  Use the same squarish favicon instead to be consistent with openliberty.io.

@gkwan-ibm

Updated on 14th of June 2021.

@yeekangc if you could please verify that would be great.

LGTM.  Thanks, @jakub-pomykala.


Research on jumps to public function in solidity code

Source issue: https://github.com/blockscout/frontend/issues/1016
Research how to extract mapping functionName -> fileLocation from verified contract

https://github.com/sevenzing/sol-abi-jumps

The concept itself seems to be working. For now there are several issues needed to be fixed before the service could be implemented. The main is that we need an evm.bytecode.sourceMap input for all verified contracts. Currently, we neither keep that info, nor compile it at all.

Update verification service to output sourceMap during the compilation, and return it as a VerificationResult
Update eth-bytecode-db service to return sourceMap as a VerificationResult and to store that info inside the internal database
Update blockscout database schema to store the output of the sourceMap
Recompile all verified contracts and store sourceMap for them


README text adjustments

Thanks! Give me a shout when you're done and I'll merge it right away!

Please check if my commented out example with parameters is correct and merge.

Ready to merge?

yep

Merged.


Maestro does not identify containers restarted by docker upon reboot

I've got this in my YAML:
  sonarr:
    image: tuxeh/sonarr:develop
    instances:
      sonarr:
        ship: media
        restart: always
        ports: {shttp: 8989}
        lifecycle:
          running: [{type: tcp, port: shttp}]
        volumes:
          /volumes/media:                      <snip>
          /volumes/config:                      <snip>
          /volumes/completed:               <snip>

I rebooted the server, and Docker happily restarted this container as per the restart settings.
However, now running maestro -f myfile.yaml start fails (as shown below) because the port is already bound and maestro doesn't realize that the running container is the one defined in my yaml. I can manually fix this - just run docker stop on the offending container and then I can run maestro start as expected, but if there's any way to address this edge case inside maestro that would obviously be better.
  5. sonarr               sonarr          media                e6fdb68         failed to start container!
Traceback (most recent call last):
  File "/usr/local/bin/maestro", line 9, in <module>
    load_entry_point('maestro==0.1.7', 'console_scripts', 'maestro')()
  File "/usr/local/lib/python2.7/dist-packages/maestro/__main__.py", line 88, in main
    getattr(c, options.command)(**vars(options))
  File "/usr/local/lib/python2.7/dist-packages/maestro/maestro.py", line 210, in start
    plays.Start(containers, self._registries, refresh_images).run()
  File "/usr/local/lib/python2.7/dist-packages/maestro/plays.py", line 184, in run
    result = self._start_container(o, container)
  File "/usr/local/lib/python2.7/dist-packages/maestro/plays.py", line 318, in _start_container
    privileged=container.privileged)
  File "/usr/local/lib/python2.7/dist-packages/docker/client.py", line 742, in start
    self._raise_for_status(res)
  File "/usr/local/lib/python2.7/dist-packages/docker/client.py", line 115, in _raise_for_status
    raise APIError(e, response, explanation=explanation)
docker.client.APIError: 500 Server Error: Internal Server Error ("Cannot start container e6fdb680e2f18e6a244a4e71964c28f201111347e8546ee9a547dedacf45db50: Bind for <IP_ADDRESS>:8989 failed: port is already allocated")

I believe this is fine now, at least with Docker 1.5.0 (haven't tested earlier versions), containers correctly keep the same name when the Docker daemon is restarted.


Update version to 2.3

Signed-off-by: Sruti Parthiban<EMAIL_ADDRESS>Is your feature request related to a problem? Please provide an existing Issue # , or describe.
Update version to 2.3
Describe the solution you are proposing
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #221 (7e863ed) into main (735bf4a) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##               main     #221   +/-   ##
=========================================
  Coverage     71.51%   71.51%
  Complexity     2985     2985
=========================================
  Files           380      380
  Lines         18952    18952
  Branches       1463     1463
=========================================
  Hits          13553    13553
  Misses         4801     4801
  Partials        598      598

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

./gradlew updateShas task will pass after this PR is merged in.


Testing the FPS in code

Required Info

Camera Model
D435

Firmware Version
<IP_ADDRESS>

Operating System & Version
Win (10)

Platform
PC

SDK Version
2.32

Language
C++

Segment
desktop app

Issue Description
I'm trying to get the number of FPS that I am able to process from the camera.
I used rs-align-advanced.cpp as the template for my own project. But I don't use the color stream and I do other things...
Anyway, I tested the speed of rs-align-advanced.cpp adding the following code to the main procedure:
    int counter = 0;
   auto start_time = std::chrono::steady_clock::now();
   auto current_time = std::chrono::steady_clock::now();
  while (app) // Application still alive?
{
     current_time = std::chrono::steady_clock::now();
    int tot_time = std::chrono::duration_cast<std::chrono::seconds>(current_time - start_time).count();
		if (tot_time>10)
    {
        std::cout << counter;
        return 1;

    }
   ...

    if (!aligned_depth_frame || !other_frame)
    {
        continue;
    }
    counter += 1;
    remove_background(other_frame, aligned_depth_frame, depth_scale, depth_clipping_distance);

I got around 80 frames in 10 seconds. That was with the remove_background function making a pass over the depth/video tables.

Am I  going about this  correctly? If No, disregard 2 and 3 and the rest of the post and point me to the right way if you could.
Is the number of frames that I should expect - 80 in 10 seconds?
I still haven't checked the content of the frames. Can I assume they are all good?

This is the last phase of my project before I use real data(which costs) and I want to make sure I get every depth frame possible
for analysis. The motions captured  can be quick. Right now I on some gestures I get 2 frames to work with. I assume that's something I'm doing and not the camera. I worked with the kinect, using c#, writing my own bitmap readers, and I think I got better FPS.  And they were 30 FPS and I'm using the default settings in rs-align-advanced which may be 30FPS.
Thanks in advance.
.

I want to add that I've been very happy with the camera so far.

do you have a discord?

Hi @arothenberg   There is not a RealSense Discord support account.  Public support for RealSense is provided by the GitHub forums and at Intel Support's Intel RealSense Help Center forum.
https://support.intelrealsense.com/hc/en-us/community/topics
Advice about FPS calculation and frame drops to compare to your own method is provided in the link below.
https://github.com/IntelRealSense/librealsense/issues/7488#issuecomment-704124850
If a "hiccup" occurs in the stream then the SDK will try to go back to the last good frame and continue onward from there, which is why you may see the same timestamp repeated twice in a row sometimes.  So whilst you can assume that the SDK has done its best to collect "good" frames, examining the list of timestamps may provide indications of whether there are problems that might be occurring during streaming.
You can expect a greater burden on processing during align operations.  This is why it can be useful to offload processing from the CPU onto a graphics GPU to accelerate processing.  With computing devices that have an Nvidia GPU, this can be done with CUDA support.  For non-Nvidia GPUs, GLSL processing blocks provide offloading and are 'vendor neutral', meaning that they should work with any GPU brand (though improvement may not be noticable on low-end computing devices).
The link below provides a good discussion of the pros and cons of using GLSL.
https://github.com/IntelRealSense/librealsense/issues/7316

Thanks Marty. I know the align operation takes time so that's why I used it. It is probably comparable to the operation that I
do time wise.
So the other question was whether 8 fps on the rs-align-advanced is a reasonable number.  It's certainly less than the 90FPS that is , maybe, possible. I'll try GSL processing blocks but if its only a nominal improvement over 8 FPS than it work for me.
If I just take the depth frame and I don't align or remove any background at 90fps (on the camera)
and render it using the sdk cpp method I get 45 FPS.
I'll try the GSL blocks and see if that helps.

The discussion in the link below about aligning in a project with low-power computers may provide helpful insights.
https://github.com/IntelRealSense/librealsense/issues/5296

Y'know Marty, I tested my code (not the rs-aligned-advanced - which i've been using as a benchmark) and I might be getting 65FPS which is much more than enough for me.  I suppose the aligned tables take a lot of time. .
I'll close this

Thanks very much for the update @arothenberg - I'm pleased that your code produces results that are saisfactory for your needs.

And thanks for that link to the gsl blocks. that might be important. tyvm.


live lines on map?

hi,
is there a way to get live lines on the map as the object moves? if yes how?
thanks

This is a duplicate of #2008.


Update doctest 2.3.1 in bincrafters

you can find the link of the pull request here: https://github.com/bincrafters/conan-doctest/pull/6

seems that they just merged it and are testing it :)
I try not to dabble with that since I still haven't gotten time to use conan.

I see it is already accepted there so I'm closing this


Initialize and Protect Problem Count

Initializes Problem Count field upon creation.
Protects Problem Count from modification on update endpoint.
resolve #33
resolve #14

Could we hold off on merging this until I merge the last of the problems tests? I did a lot of refactoring of the controllers, regarding updating problemCount, so there will be changes to make

Sure, will wait :p


MSID: 87238 Version: 2 DOI: 2<IP_ADDRESS>7146

[PLACE MANUSCRIPT AND EDITOR DETAILS HERE WHEN AVAILABLE] See step 3
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=87238)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/87238/v2)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>7146)
[ ] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[ ] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/87238/v2/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/87238)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/87238

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/87238

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/87238/v2/87238-v2.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/87238)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Corresponding authors not indicated. Should be first and last authors; not present in latest bioRxiv online version though it is noted in the PDF.
Publisher names missing from book references.
eLocations missing from references.

Urgh - bioRxiv have not included any xref between corresponding authors and the corr note. Probably quicker just to fix ourselves.

@JGilbert-eLife should be fixed now. Please check and update accordingly


PP-6146 Add missing gocardless env vars

The following env vars are being provisioned by the selfservice secret
service but are not in the env-map or manifest.
GOCARDLESS_TEST_OAUTH_BASE_URL
GOCARDLESS_LIVE_OAUTH_BASE_URL
GOCARDLESS_TEST_CLIENT_ID
GOCARDLESS_LIVE_CLIENT_ID
WHAT
Found these whilst cleaning up our staging config.

@mrlumbu said this isn't actually needed by Selfservice so I'll remove the values form tfvars instead...


No output on Intel/AMD iMac with Monterey (macOS 12.7)

Plain llama works (i.e. --base_model=llama) but nothing else does. I've tried chavinlo/alpaca-13b, openlm-research/open_llama_13b, openlm-research/open_llama_7b_v2 and openlm-research/open_llama_7b. It seems to at least try to run the LLM but when I ask it anything it just pauses for two minutes or so before asking for a second prompt, without ever providing a response from the model. During this time the GPU appears to be under load and the rest of the computer is considerably less responsive. I also need to use PYTORCH_MPS_HIGH_WATERMARK_RATIO=0.0 because it gives OOM otherwise, even with options like --low_bit_mode=4 and load_4bit=True, even for the 7b models.
Hardware:

Quad-Core Intel i7-4790K CPU @ 4.00GHz
32 GB 1600 MHz DDR3
AMD Radeon R9 M295X 4 GB

Yes, I am using OpenCore to boot a macOS version later than supported (Big Sur doesn't have MPS after all) but the same thing happened on the CPU with Big Sur, which leads me to believe MPS isn't the issue. The most unusual thing is that it works fine when I set --base_model=llama but I don't want to use llama.

I've determined that my GPU probably just isn't suited for LLM tasks, even though it works just fine with stable diffusion. Testing with llama.cpp, the program just hangs with the GPU working continuously doing seemingly nothing if the VRAM usage significantly exceeds 4GB, and if it's less than 4GB (or only slightly over the limit) it outputs gibberish, even though CPU inference works perfectly well. Given this, I'll try to get h2ogpt to use CPU inference if I can. Since I'm using the CPU, is there any way to run quantized models? I can use llama.cpp to convert a model to gguf and then quantize it, but h2ogpt doesn't seem to work with gguf.

GGUF definitely works, just not OOTB with default instructions.  See: https://github.com/h2oai/h2ogpt/blob/main/docs/FAQ.md#gguf

Closing since no activity in this ticket, feel free to reopen if required!


[flutter_tools] Migrate xcode_backend.sh to flutter assemble

Description
FYI @dnfield
Still a WIP as I test behaviors and update existing tests
Fixes https://github.com/flutter/flutter/issues/32925

Separate PR is probably ideal, should be a quick to review PR