Bulk mail

We are considering using Postal for one of our services. The service is sending a lot of e-mails, sometimes more than 100k at once.
I can't find any API for sending the same message to multiple recipients, so if we where to send 100k message we need to do 100k request to Postal and we will get 100k webhooks back? Is there some kind of SMTP-api or something which could handle bulking?
Is this something you find possible for Postal to handle or would you recommend us to use something else?

https://github.com/postalhq/postal/wiki/FAQs
In your scenario, bulk sending should be handled by your application with 100k requests to Postal as you described.
Postal can handle multiple email addresses on a single email (i.e. CC/BCC) but I presume that wouldn't be suitable or appropriate for your needs.

Thank you for your reply!
I understand, no problem for our application to batch/bulk. But there will be a lot of requests for Postal to handle. Will postal have a hard time dealing will all of these requests or is it designed to handle many simultaneous requests and handling all of those webhooks...
Yes we could add multiple addresses to the same message but there are a limit of 50 addresses per request (?). Adding multiple recipients to the same message will also be problematic since Postal doesn't support "recipient variables". If there are something differing between the messages (e.g recipient name) we need to make a new request?
I'm not complaining, just trying to figure out our options.
Thanks,
Anton

No problem at all, Postal should be able to handle the requests but it would be worth making sure you have a well-specced server and Postal configured to take advantage of it.
There are a few issues discussing this topic, #697 and #324 would be a good place to start in this regard.

Thank you for your input! Would you recommend us to use SMTP or the built in HTTP API?

If you want to make use of webhooks, I believe you have to make use of the HTTP API in order to match individual messages you have sent with the received webhook as the output from using the HTTP API will be required for this.

Okay! If using SMTP, we were thinking of using the message-id header (with our own custom ID) to map the message in incoming webhook but that might be a bad idea? :)

I'm not sure that the information returned by the webhooks would include your message ID. If you're hoping to use webhooks, making use of the HTTP API would be a more complete solution.


CLI - Make sure input ligand file exists

Description
This makes sure input ligand file exists when using the CLI. Avoiding freezes.
Motivation and context

Resolves #836
How has this been tested?
Tested manually, will wait for CI jobs to succeed.
Change log

Avoid freeze when ligand file is not found when using CLI.

My bad, was based in the wrong parent.


[Enhancement] [UT] Add --module option to enable run  specific binary test for specific ut in run-ut.sh

…ut in run-ut.sh
What type of PR is this：

[ ] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #

Add --module option to enable run specific binary test for specific ut in run-ut.sh to decrease run ut time;
Add a GTEST_OPTIONS  to support more gtest options.

Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[ ] I have checked the version labels which the pr will be auto backported to target branch

run starrocks_admit_test

run starrocks_admit_test


[Android] fix nightly publish failure

Description
Fix the empty input issue while publishing

Codecov Report

Merging #2001 (df673bd) into master (bb5073f) will decrease coverage by 2.18%.
The diff coverage is 68.82%.

@@             Coverage Diff              @@
##             master    #2001      +/-   ##
============================================
- Coverage     72.08%   69.90%   -2.19%
- Complexity     5126     5916     +790
============================================
  Files           473      586     +113
  Lines         21970    26216    +4246
  Branches       2351     2846     +495
============================================
+ Hits          15838    18325    +2487
- Misses         4925     6511    +1586
- Partials       1207     1380     +173

Impacted Files
Coverage Δ

api/src/main/java/ai/djl/modality/cv/Image.java
69.23% <ø> (-4.11%)
:arrow_down:

...rc/main/java/ai/djl/modality/cv/MultiBoxPrior.java
76.00% <ø> (ø)

...rc/main/java/ai/djl/modality/cv/output/Joints.java
71.42% <ø> (ø)

.../main/java/ai/djl/modality/cv/output/Landmark.java
100.00% <ø> (ø)

...main/java/ai/djl/modality/cv/output/Rectangle.java
72.41% <0.00%> (ø)

...i/djl/modality/cv/translator/BigGANTranslator.java
21.42% <0.00%> (-5.24%)
:arrow_down:

...odality/cv/translator/BigGANTranslatorFactory.java
33.33% <0.00%> (+8.33%)
:arrow_up:

.../cv/translator/InstanceSegmentationTranslator.java
0.00% <0.00%> (-86.59%)
:arrow_down:

...nslator/InstanceSegmentationTranslatorFactory.java
7.14% <0.00%> (-11.04%)
:arrow_down:

.../cv/translator/SemanticSegmentationTranslator.java
0.00% <0.00%> (ø)

... and 499 more

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


Internalize codegen

Related to #62
Description of changes: This removes the need to separately install the codegen component. make build-codegen or make rebuild will handle preparing things. make aws-codegen (and callers) will then use the local binary.
It will also format the bindata code if it changes to preserve formatting.
Installation of go-bindata is still required. It would be nice to run this from a vendored source for consistency, maybe that can be a future effort.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

We include the changes from #136 in this PR

@christopherhein Can you help me understand why this was closed?

Sorry about thank @joerocklin thanks for calling that out. I was mistaken that this was the same as #136 but at second look you are right.
Any chance you can update the /Makefile target for rebuild to reference to call install-aws-codegen unless you feel there is a good reason to move this from an installed binary into a local package…


Arpeggiator does not handle added notes correctly

The arpeggio currently does not handle added notes correctly. When adding a lower note to a held chord the last played note is played again before playing the next note. When adding a note that is higher than all the currently held notes the bevahiour is correct.

This should be fixed by https://github.com/datomusic/duo-imxrt/pull/114


[FEATURE] Google authentication frontend

Description
This matter pertains to the ongoing advancement of issue #711, which involved Google authentication. The backend portion has been previously developed, and now it is imperative to finalize this feature by implementing the frontend component. Kindly assign this task to me.
In this issue we will add a button and validate the user to complete the verification process
Screenshots
🥦 Browser
Google Chrome
Checklist ✅

[X] I checked and didn't find similar issue
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (blank for no)

Sounds good, you can start working on this !
Please make sure that you have read our documentation on Commiting guidelines and Pull request guidelines, without proper format the PR might be closed.


Is there a way to detect when user removed a tag?

I'm trying to notify some app class when a tag was deleted by user. I didn't find any event triggered when this happens so don't see you can detect it.
I could do the change and create a pull request in case it's not implemented and you think it could help other people.
Thanks

No, currently there is no way to tell. PRs are welcome. Did you implement a method in the delegate?

I haven't implemented anything yet, but adding a method in the delegate would be the best. Keep you posted.
Thanks

I may be crazy but I remember being able to delete tags clicking on them. I tested this some days ago but I'm quite sure I saw that. I'm no longer able to do so. Am I crazy? :)

Nope, you're right, I changed the sample to test another issue. You can however remove the last tag by clicking the right navbar button

I've finally used AMTagViewNotification to detect user tap over tag and delete it. Haven't changed anything on the code. Feel free to close this.

Cool :+1:

Hi, I am facing same problem. how to detect tag index ?? please tell me

Let me add a delegate method for this purpose, it'll be more clear.
I'll do it in the next few days.

And by next few days I mean now. Check out the latest commit and let me know if that works for you.

Released 1.4.0


Fix storybook

In order to make Sanity v3 work, we had to set the type of our app as "module", and this is causing problems to start/run storybook

Like magic, adding an empty package.json file to the storybook folder fixed it. The idea of doing it is that it would not use the package.json doc from the parent folder, and instead use its own. Surprisingly, it worked.

Works now.


Add condition field to ClusterQueueStatus

Signed-off-by: tenzen-y<EMAIL_ADDRESS>
What type of PR is this?
/kind feature
/kind api-change
What this PR does / why we need it:
I added condition field to ClusterQueue.
Which issue(s) this PR fixes:

Fixes #250
Special notes for your reviewer:
API design

This PR is ready to review.
Please take a look.
/assign @alculquicondor @ahg-g @kerthcet

Let me know when this PR is ready for another pass :)

Let me know when this PR is ready for another pass :)

Sure.

It looks like you are still working on this.
Please let me know when it's ready. I'll ignore it for now, as I have a lot of review burden at the moment.

Sure.

@alculquicondor @denkensk I have addressed your suggestions. Please take another look.

Still waiting for an update here

Still waiting for an update here

Thanks. I will get back to you when this PR is ready for review again.

/retitle WIP Add condition field to ClusterQueueStatus
Please remove WIP from the title when it's ready for another pass.

/retitle Add condition field to ClusterQueueStatus

@alculquicondor I have completed implementing a custom watcher for ResourceFlavors to the clusterqueue-controller.
PTAL

rebased.

/retest

Before retrying a test suite, please make sure if there is an issue filed for the flaky test.

Before retrying a test suite, please make sure if there is an issue filed for the flaky test.

@alculquicondor I checked failed CI job, it didn't seem to be related to this PR. Maybe, related to the WaitForPodsReady feature.

Summarizing 1 Failure:
[FAIL] SchedulerWithWaitForPodsReady when Scheduling workloads on clusterQueues [It] Should block admission of one new workload if two are considered in the same scheduling cycle
/home/prow/go/src/sigs.k8s.io/kueue/test/integration/scheduler/podsready/scheduler_test.go:147
Ran 3 of 3 Specs in 15.462 seconds
FAIL! -- 2 Passed | 1 Failed | 0 Pending | 0 Skipped
--- FAIL: TestSchedulerWithWaitForPodsReady (15.47s)
FAIL

https://prow.k8s.io/view/gs/kubernetes-jenkins/pr-logs/pull/kubernetes-sigs_kueue/415/pull-kueue-test-integration-main/1605630317664669696

Please do rebase instead of merge.

Please do rebase instead of merge.

Does that mean using git pull --rebase instead of git rebase?

You can do git pull on main (it should fast-forward) and then a git rebase on your branch.
Maybe this is what you did? I just see a lot of commits in this PR that don't seem related, but please correct me if I'm wrong.

You can do git pull on main (it should fast-forward) and then a git rebase on your branch.
Maybe this is what you did? I just see a lot of commits in this PR that don't seem related, but please correct me if I'm wrong.

Yes, I did git pull on main and then git rebase on my branch.
Although, all commits seem to be created by me.
$ git log --oneline
4318a54 (HEAD -> add-condition-to-clusterqueue, origin/add-condition-to-clusterqueue) Implement a custom watcher for ResourceFlavors to the cq-controller
0ffb49b Use testing utils to create kueue reated custom resources in unit tests for cq-controller
acd8199 Create ResourceFlavors before CQ admits workloads
d1f1895 Add test cases for ClusterQueueStatus
ac548d1 Remove unnecessary flavors from workload-controller integration tests
07037fd Remove unnecessary helper functions in integration testing
e3c1419 Ignore ClusterQueueStatus.Conditions in integration tests for the workload-controller
deea85e Merge UpdateCqStatusIfChanged func and UpdateCqStatus func into updateCqStatusIfChagned func
81dccf8 Add tests to verify the bool value returned by UpdateCqStatus func
23448cd Fix a goimports error
eaa4874 Add the Copyright to the unit test for clusterqueue_controller.go
da2b9c1 Add suite tests for cq status conditions
834ec27 Remove the logic to update cq condition just before cq-controller removes finalizers
b904d13 Add a unit test for the UpdateCqStatus function
64282b1 Add a status message when CQ is terminating
687e947 Fix error messages
f8a7ec3 Replace the kueue.ClusterQueueCondition with the metav1.Condition
f49858c Update apis/kueue/v1alpha2/clusterqueue_types.go
29483df Addd condition field to ClusterQueueStatus
9f27bc3 (origin/main, origin/HEAD) Merge pull request #473 from mimowo/block-admission-until-podsready
424aa0a Block admission until PodsReady for all admitted workloads if WaitForPodsReady is enabled
7b8316f Merge pull request #475 from alculquicondor/flavor-modes
ac88466 Merge pull request #466 from cmssczy/use_markers_validate

blocked by #489

Rebased to take #489

Our E2E seems to be so heavy... Maybe, the kube-apiserver is hung up...
{Timed out after 30.001s.

 We have a timeout of 30s. We can easily lift this to 1 min or more.

/retest
Uhm.... it looks like kueue is particularly slow to start in 1.23. We have a timeout of 30s. We can easily lift this to 1 min or more.

I created an issue to track this:
#495

@alculquicondor, I addressed your suggestions. Thanks for the many reviews.

This turned out not trivial.
Kudos for all your effort.
/lgtm
/approve


Incorrect Icon Graphics

I have a strange problem where icons display incorrect images. Most notably, the back-button is an ice-cream cone. For example:

This appears in both the browser preview (localhost:8100), and when built and rendered on Android device. The icons appear slightly differently when the browser emulates a mobile device:

Two of the icons are actually correct, which makes this problem probably more confusing rather than less.
Anyone else have this problem? Any idea how to fix this?

This issue is not related to ionic-cloud in any way.
If you still have this issue, I suggest opening a topic on the Ionic Forum at https://forum.ionicframework.com/. Thanks.
(You can also close this issue here)


Docker tagger uses event API

This is a pull request into the new-report branch. It just brings the docker mapper event API changes into the new docker tagger. We don't necessarily have to merge it this way, it's just so you can see the diff.

I included this in #178

Cool, then I close this one in deference to that one.


Support show both value and percentage on pie chart

Is your feature request related to a problem? Please describe.
We want to show category, value and percentage on the pie chart at the same time.
Now we have to edit the meta, change the chart options and switch between value and percentage

https://github.com/apache-superset/superset-ui/pull/699


Feat/registration

Add more registration fields (alergy, phonenumber)
Add forgot password
Add verify email
Add a redirect to /register if the user is not registered, and foes to a page with getUser shouldRedirect is set to true
Add OmegaId (important milestone for #10)
Added a way to register soelles to members (Fixes #10)
Added Qr code libraries ( #139 #41)
Add UI to admin studyProgrammes
Automaticly add users to OmegaMembershipsGroups

Caution! Merge the feat/notifications #240 branch and the feat/auth folder #228 first!

Thanks for the feedback @Paulijuz

Are you supposed to be able to enter any domain name you want into a mail alias. Would it be better to simply enter the name that should be before the @?

Some users should be able to add different domains than the current domain. Maybe we want the alias<EMAIL_ADDRESS>for example. But I agree that it's unnecessary in 99% of cases not to add this automatically and prevent users from typing the wrong domain.

Should the admissions be a database table? This doesn't seem like something that is likely to change in a loooong time so I don't see the benefit of allowing it to be dynamic. Maybe a simple enum would suffice? Then you could reduce the complexity quite a bit.

I had an idea to use this more generally, like admissions to DGR. But after some thought I don't think this will be used, and then it's just extra unnecessary complexity.

I'm not sure if this is the PR that did it, but the create new user form is broken.

I have some thoughts about this. I think the user form should take in the fields username, firstname, lastname and email. Then when you create a user, they should get an email with a link that leads to the registrations page. And there they can create a password add a mobile number etc. Do you have any thoughts on this as well @JohanHjelsethStorstad ?

I think turning on a lot of notification channels is very tedious. Especially since you have to press the submit button for each channel.

I totally agree the UI should be a lot better. I was not sure how the profile page design would be. But the best is probably to put the settings on a different page.

Some users should be able to add different domains than the current domain. Maybe we want the alias<EMAIL_ADDRESS>for example. But I agree that 99% of cases we just use omega.ntnu.no.

Then I understand your design choice. Ideally, I still think there should be something in the UI to indicate which domains are valid tho. Currently you either have to know which domains are valid or look through the list of used domains. Maybe you could have a drop down of all the valid domains? Though this is not really important, this page will be used by very few people (only admins) so I think how it is now is good enough. We can always improve later on.

I agree that we should have a fixed number of domains allowed

As for the createNewUser form it was meant as an admin-form. So it should probably  take as little info as possible then make the new user give the rest

I have fixed most of the feedback you gave. I have also created a JWT folder, with all the JWT functionality.  The things I haven't fixed yet are:

mobile mixin on all pages
Includer for long queries (I don't feel like the queries should be in config since they are not config)
The user form is still broken, but it would be nice to discuss how this should function. (And I is fixed in a later PR)

The latest comments from the review from @Paulijuz  should have been resolved by now. Hovever I have not bee enable to reproduce the bug:

If I enable all the channels with the top check button I sometimes get the error message "The methods must a subset of the available methods". I'm guessing this is because the top check button checks the disabled check buttons somehow.

Therefore it's probably not fixed, but I have changed how the page works and that can be why I'm unable to reproduce the bug.
@JohanHjelsethStorstad do you want to give me another review?

@JohanHjelsethStorstad fixed your comments.
I'll refactor a few things in this PR while working on push notifications.


Add instructions on how to activate traces in mbed OS 5

In mbed_app.json add:
    "target_overrides": {
        "*": {
            "mbed-trace.enable": 1
        }
    }

And in main.cpp add:
#include "mbed-trace/mbed_trace.h"

void trace_printer(const char* str) {
    printf("%s\r\n", str);
}

int main() {
    mbed_trace_init();
    mbed_trace_print_function_set(trace_printer);

Fixed in #49

Yep, this should definitely be covered in the readme. Thanks for pointing it out and making the PR.

ARM Internal Ref: IOTSYST-1252

Done in https://github.com/ARMmbed/mbed-trace/pull/50


changed position of semicolon

fixed handling of unary

Codecov Report

Merging #113 into master will decrease coverage by 0.36%.
The diff coverage is 78.57%.

@@            Coverage Diff             @@
##           master     #113      +/-   ##
==========================================
- Coverage   72.92%   72.56%   -0.37%
==========================================
  Files          39       39
  Lines        3210     3240      +30
  Branches       51       51
==========================================
+ Hits         2341     2351      +10
- Misses        835      855      +20
  Partials       34       34

Impacted Files
Coverage Δ

src/tokentree/walk/WalkIf.hx
65.85% <0%> (-31.29%)
:arrow_down:

src/tokentree/walk/WalkBlock.hx
94.23% <100%> (+4.75%)
:arrow_up:

src/tokentree/walk/WalkStatement.hx
90.37% <100%> (+0.74%)
:arrow_up:

src/tokentree/utils/TokenTreeCheckUtils.hx
50.72% <70%> (-0.54%)
:arrow_down:

src/tokentree/walk/WalkClass.hx
79.34% <0%> (-2.18%)
:arrow_down:

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 4c3c1ee...e8cedd2. Read the comment docs.


Code styling break Wordpress

The set "margin-bottom" is making the text break apart when I use the the 'code' tag in a WordPress blog post.
code {
  display: block;
  clear: both;
  overflow: auto;
  padding: 1.6em;
  margin-bottom: 1.6em;
  white-space: pre;
  background-color: #f0f0f0;
  font: 14px "Andale Mono",Courier,"Courier New",monospace;
}

Looks okay now, was it fixed? View first paragraph of this page: https://cd.foundation/blog/2020/07/07/towards-more-trusted-open-source/


Fix various spelling mistakes in JSON files

Caught a few uses of British English spelling of behaviour, and "ize" words.

This is good thanks a lot @tonybaloney


Allow passing addresses as string

Feature Request
Most capi runes expect addresses as Uint8Array, I want to pass them as string instead or as well.
Motivation
In my experience most js application interacting with substrate get addresses of contracts and accounts as human readable hex encoded strings.
These strings however, can not be passed to any function/rune of capi directly. Capi expects addresses as Uint8Array. This requires decoding the hex addresses with i.e. scale-ts and encoding the result of the call again to be represented as hex for displaying results in the known format of the user.
This is very cumbersome and inflates each interaction with capi with a decoding and encoding step of multiple values. It would be a nicer dx to just work with addresses as string for most interactions.
Use Cases
A simple use case as reading getting the multisig address is getting this ugly:
    const signatories = members.map((member) => {
      const [_, addr] = ss58.decode(member?.address!)
      return addr
    })

    const multisig: MultisigRune<Westend, never> = MultisigRune.from(westend, {
      signatories,
      threshold,
    })

    const multisigAddress = ss58.encode(
      await westend.addressPrefix().run(),
      await multisig.accountId.run(),
    )

MultiAddress.Id is used a lot throughout capi and can only be constructed with Uint8Arrays as well. Would be convenient to just construct this object from string addresses as well.

Example of current UI flow:
https://github.com/paritytech/capi-multisig-app/blob/7737a7848fcbbc47a46737f0ee24c9650a00278a/www/src/components/wizards/multisig/Members.tsx#L83-L186

Hopefully #1043 is a good first step towards alleviating the bloat. We could potentially allow MultisigRune to be constructed with ss58 addresses as well, but this could complicate the typings.
export type Multisig = {
  signatoryAccountIds: Uint8Array[]
  signatorySs58Addresses?: never
  threshold?: number
} | {
  signatoryAccountIds?: never
  signatorySs58Addresses: string[]
  threshold?: number
}

We don't do overloads / implicit conversion. That being said #1043 has been merged and will hopefully alleviate some of the burden.


Support Win8.1 + Phone 8.1 Universal Apps (WNS), drop support for WP8.0 (MPNS)

Hi,
I found this plugin when searching for a successor of the PushPlugin (https://github.com/phonegap-build/PushPlugin) which seems not to be developed further.
The current readme says that support for "WP8" is under development. However it does not say about the Win 8.1/WP 8.1 (Universal Apps) platform.
As this seems to be a new plugin, what about dropping support for WP 8.0 and instead only supporting Win 8/8.1/WP 8.1 "Universal Apps"? Note that these platforms will use WNS (Windows Notification Service) instead of MPNS (Microsoft Push Notification Service) that was used in WP 8.0 (Note that the Cordova Device Plugin reports an Universal App on WP 8.1 as "windows".
Note that the old PushPlugin (https://github.com/phonegap-build/PushPlugin) does not work on Win 8.1/WP 8.1 in the current Cordova version shipped with VS 2015 RC as it specifies a wrong include file, but I found a fork that fixes this issue: https://github.com/kenwu27/PushPlugin (it worked for me on a Windows Phone 8.1 emulator included in Visual Studio 2015 RC, using a Cordova App).
So you could implement support for Push Notifications on Win 8/8.1/WP 8.1 using WNS based on that implementation.
Thanks!

@T18970237136 thanks for the detailed issue. I'll let @rakatyal comment on Win 8.1 support.

:+1: for the new approach

@T18970237136 : Thanks for the issue. That is exactly what I am working on. I am adding the support for Win 8/8.1/WP 8.1 using WNS. The first version is ready and should be merged soon.

Cool, Thanks!

@T18970237136: This has been merged now.


Prepare Workshop for Security Champion

Description
We need to have a workshop
Additional Information
No response
Tasks
No response
Acceptance Criterias

Identify risks

@TheTechArch - jeg lukket denne. Workshop er gjennomført.


GPL-nnn Pass or fail a sample

User story
As a user of the PacBio section of Traction, I would like to be able to fail a sample based on its QC results, and for this to be visible in Traction, so that I don't accidentally take samples further down the pipeline when they should be stopped.
Who are the primary contacts for this story
Iraad, Craig, Emma B
Acceptance criteria
To be considered successful the solution must allow:

[ ] I can fail a sample using the Traction interface
[ ] I can see which samples have been failed
[ ] This information is propagated to the warehouse (?)
[ ] Failing a sample doesn't stop me from taking it through Traction a second time (as this will be a different aliquot and therefore not contaminated, for instance)

Additional context
This will probably also need a traction-service story.
Viewing the QC results is covered by https://github.com/sanger/traction-ui/issues/579

Superceded by #613


Update temporary.yaml

make aliases for https://lbry.com/support and redirected to https://lbry.com/faq/support

Fixed up and merged, thanks Chris!


[Traefik Plugin Catalog] Plugin Analyzer has detected a problem.

The plugin was not imported into Traefik Plugin Catalog.
Cause:
failed to run the plugin with Yaegi: the load of the plugin takes too much time(10s), or an error, inside the plugin, occurs during the load: 1:21: import "github.com/killer-djon/traefik-http2amqp" error: /tmp/traefik-plugin-gop2157921314/src/github.com/killer-djon/traefik-http2amqp/http2amqp.go:6:2: import "github.com/rabbitmq/amqp091-go" error: /tmp/traefik-plugin-gop2157921314/src/github.com/killer-djon/traefik-http2amqp/vendor/github.com/rabbitmq/amqp091-go/channel.go:425:18: cannot use type func(amqp091.frame) as type func(*struct {},amqp091.frame)

Traefik Plugin Analyzer will restart when you will close this issue.
If you believe there is a problem with the Analyzer or this issue is the result of a false positive, please contact us.

Get forked library to fix this error


Reapply Settings not working in particular case

So the reapply settings button is a godsend for scenes I had made prior to getting Token Mold, but I'm noticing it doesn't work if in the selection of tokens there happens to be any linked actors.
If I select a scene's worth of unlinked actors then it works perfectly but if there are any linked ones among them then none of them will be updated.

I'm not seeing this work in any situation.

@etriebe thats because you're propably on 0.6.0
issue is fixed for next patch


Adding Whatsapp share

Hey mxstbr, thanks for sharing a nice solution. It would be nice if you add Whatsapp too.

Does WhatsApp have a sharing link? Do you have any idea what their structure is?

Well, ofc it works on devices which have whatsapp. Their FAQ about this

Oooh amazing!

Done by the amazing @OGPoyraz in #42! 🎉


List the translation when a document is translated

With a big "Download" button and a:
Attach=concept_mapper_es_ES.xml

header.

Done


Check collinearity of three points in Python

💥 Proposal
Check collinearity of three points in Python
Have you read the Contributing Guidelines on Pull Requests?
YES

/assign


Feature/inventory e2e tests

Resolves #3834

I'm not sure what's going on with the checks.. sometimes GitHub gets confused. The pending Travis build completed successfully (you can click the link to verify). The pending GCB check should not be running, not sure why it's trying to be run. The completed/successful GCB check is the integration tests. Hopefully I can get to migrate more of these tests to pytest and the amount of time required to run the tests will decrease over time.


Healthcheck endpoint returns 401 when BASIC Authentication is enabled

Describe the bug
When Basic Authentication is enabled, the endpoint /ui/actuator/health returns a 401 code. Health check should be unauthenticated and return 200 on success, and 4XX on failure.
The health check endpoint is currently whitelisted when OAUTH authentication is enabled, but not for BASIC Authentication:
Set up
Currently using 0.2.1 in ECS Fargate as a docker image. Fronted by an ALB with health check pointing to /ui/actuator/health
Steps to Reproduce
Steps to reproduce the behavior:

Enable Authentication via environment variable
"AUTH_ENABLED" : "true"
"SPRING_SECURITY_USER_NAME" : "admin"
"SPRING_SECURITY_USER_PASSWORD" : "mysecretpassword"
Hit endpoint /ui/actuator/health
Receive 401 code

Expected behavior
Expect a 200 success when hitting /ui/actuator/health
Screenshots

Additional context
Current work around is to allow 401 on the ALB health check, however it would be better if 200 was provided on success.

Hey, thanks for raising the issue. This will be fixed within #1173


The auto page counter stops counting with many pages.

Browser Type/Version: Google Chrome 50.0
Operating System: Windows 7 SP1
Issue Description: When having more then 3 pages and using the auto counter, the count becomes messed up on all pages beyond the 3rd one. Right now all pages beyond 3rd one show 3 as a page number.

It looks like the counter is only messed up in Web interface when accessing the brew via edit or share link. When previewing the PDF when exporting the counter seems to work fine.

Looks like it's an issue with partial page rendering. Fixed with v2.2.3


Llama3.2-vision image processing not implemented for /generate

What is the issue?
Reported by @oderwat:
https://github.com/ollama/ollama/issues/6972#issuecomment-2437586368
OS
No response
GPU
No response
CPU
No response
Ollama version
0.4.0

Hi @jessegross
Could I confirm that currently only /api/chat is working properly, while /api/generate is not functioning as expected?
Thank you so much.

@chiehpower that's correct. You can try PR #7384 to test out the /api/generate handler.

Got it. Thank you so much!

This was merged into "main", and when compiling main this does not work for me when using 'llama-3.2-vision' neither with chat nor with generate.
key clip.has_text_encoder not found in file
terminate called after throwing an instance of 'std::runtime_error'
  what():  Missing required key: clip.has_text_encoder

I believe that this needs to be merged with v0.4.0-rc5 and get a v0.4.0-r6 to be useable or is the plan to rebase the v0.4.0 code and then create the new release candidate for v0.4.0?
Or did I understand something wrong?

You need to build according to the instructions here in order to support llama3.2-vision:
https://github.com/ollama/ollama/blob/main/docs/development.md#transition-to-go-runner

@jessegross thank you!
For reference: I am on WSL2 with RTX 3090 TI, and it builds (and uses CUDA) with this build script:
#!/bin/bash
echo "Setting limit higher for mlock to work"
limit=8413752832
sudo prlimit --memlock=$limit:$limit --pid $$

git fetch
git checkout main
export LIBRARY_PATH=/usr/lib/wsl/lib
make -C llama -j 4
go build .
# run it for unlimited use in my private network
OLLAMA_HOST=<IP_ADDRESS>:11434 OLLAMA_ORIGINS="*" ./ollama serve

Using this I could update my capollama to use the generate API  👍


_perfectScrollbar2.default is not a constructor

Hi. I was trying to use the perfect scrollbar in my React project, however,  I am getting console error trying to initialize new PerfectScrollbar. Here is my code
const item = document.querySelector('item');
const ps = new PerfectScrollbar(item);

I get this error
 Uncaught (in promise) TypeError: _perfectScrollbar2.default is not a constructor

I did some research but couldn't find anything.
Any help would be appreciated.

You may have installed perfect-scrollbar from npm? The current README is one of the next branch. Please check out master’s doc for.

This is documentation for the next branch. To read documentation for released versions, please visit master.

I have the same issue, there is a problem in the typings, it should not export default, the export should rather be export = PerfectScrollbar


Feature: Ability to open links in Firefox Focus

It's brilliant that opening Links in Firefox is already supported. As I use Firefox Klar as my primary browser on iOS (called "Firefox Focus" in non-German speaking countries), it would be nice to be able to choose this browser for opening links in Slide. Currently only Safari and Firefox can be chosen under "Settings - Link handling".
It looks like the "Focus iOS open in client"-reference is empty though: https://github.com/mozilla-mobile/focus-ios-open-in-client

Firefox Focus is actually already supported :)

Then I'll probably have to create a new issue because it's not showing for me. Maybe it's because I have "Firefox Klar" installed: https://support.mozilla.org/en-US/kb/difference-between-firefox-focus-and-firefox-klar


Update Proof to check for Underflow / Overflow

Some comments

(And) It would be good to automatically generate the various instantiations of And, such as AndU160, etc.

Current status:

allowance (0 errors)
approve (1 error)
balanceOf (0 errors)
decimals (1 errors)
deposit (0 errors)
name (27 errors)
symbol (27 errors)
totalSupply (0 errors)
transfer (0 errors)
util (17 errors)

So, an interesting and important problem raised in approve:

Writing to memory MStore when the address is unknown
This causes verification failures because of the general requirement to track the free memory pointer.

I think another take away is that code which manipulates the free memory pointer is not well handled.

A very strange result I got when playing with fixed point math, multipling 10^18 * 10^18, i.e. wad * wad, I got a 10^34 ...  In lean4 I get a good 10^36 result.
include "number.dfy"
include "maps.dfy"
include "tx.dfy"

// Fixed point arithmetic math
// https://github.com/dapphub/ds-math/blob/master/src/math.sol
module Fixed {

import opened Number
import opened Maps
import opened Tx

    const WAD: u256 :=<PHONE_NUMBER>000000000
    const RAY: u256 :=<PHONE_NUMBER>000000000000000000

    function Add(x: u256, y: u256) : u256
    requires (x as nat) + (y as nat) <= MAX_U256 as nat{
        x + y
    }

    function Sub(x: u256, y: u256) : u256
    requires (x as nat) - (y as nat) >= 0 as nat {
        x - y
    }

    function Mul(x: u256, y: u256) : u256
    requires (x as nat) * (y as nat) <= MAX_U256 as nat {
        x * y
    }

    function Wmul(x: u256, y: u256) : u256
    requires y == 0 || (x as nat) * (y as nat) <= MAX_U256 as nat
    requires Mul(x,y) as nat <= MAX_U256 {
        var m: u256 := Mul(x,y);
        var w: u256 := WAD / 2;
        assume {:axiom} (m as nat) + (w as nat) <= MAX_U256 as nat;
        (Add(m , w))/WAD
    }

}

// --- TESTS

import opened Fixed
import opened Number
import opened Maps
import opened Tx

method {:test} testFindMax() {
    var j :=<PHONE_NUMBER>0000000000 as u256;
    var i :=<PHONE_NUMBER>0000000000 as u256;
    var r :=<PHONE_NUMBER>0000000000 as u256;
    assert Wmul(j,i) == r; // does not pass!

    var x : nat :=<PHONE_NUMBER>00000000;
    // 10^18 * 10^18 = 10^36
    print(x*x); // it has 34 zeros ! help
}

Seems that the issue is on the print function. Anyway, I am getting a precision error when using the Wmul function, see that I added a zero on the r, but doing only with nat it just works:
method {:test} testFindMax() {
    var j :=<PHONE_NUMBER>0000000000 as u256;
    var i :=<PHONE_NUMBER>0000000000 as u256;
    var r :=<PHONE_NUMBER>00000000000 as u256;
    assert Wmul(j,i) == r; // pass only adding a '0'

    var x : nat :=<PHONE_NUMBER>00000000;
    // 10^18 * 10^18 = 10^36
    //print(x*x); // it has 34 zeros ! help

    var x1 : nat :=<PHONE_NUMBER>000000000;
    var x2 : nat :=<PHONE_NUMBER>000000000;
    var r1 : nat :=<PHONE_NUMBER>000000000000000000000000000;
    var r2 : nat := (r1 + (x1/2)) / x1;
    assert x1*x2 == r1;
    print(x1*x2);
    assert r2 == x1;
}


[Bug] AttributeError: '...EnvParams' object has no attribute 'max_steps_in_episode'

Some part of the code (evaluate.py, line 88), assume that the env params struct has a field max_steps_in_episode.

Hi @axelbr!
The field max_steps_in_episode is defined in the base class gymnax.environments.environment.EnvParams, and should therefore be defined in all Gymnax environments. I have therefore considered it part of the standard Gymnax interface.
If this leads to a bug with your code (e.g. you have defined EnvParams that don't inherit from this base class), a simple workaround would be to add the field to your struct.

Thanks for the fast reply.
Hm, this isn't documented in Gymnax anywhere, but you are right. I did not notice the type bounds for the generic.


Can't access account that had been previously tied to my google account

Has the google account access been disabled? I cannot sign in with my gmail and I have lost a lot of content that I was in the process of editing. Has this been moved to a new username? Or were those removed? Any help is appreciated.
Additional Details
Share Link :
or
Brew code to reproduce : Click to expand
PASTE BREW CODE HERE

Absolutely nothing has changed on the website. There have been no code
updates. I logged in myself using my google account just now and it worked
fine.
Does your username need to have have<EMAIL_ADDRESS>at the end or not? That's
been a common error for some people depending which way they typed it in
when the account was first made.
On Mon, Nov 20, 2017, 1:30 AM Ritchieluk<EMAIL_ADDRESS>wrote:

Has the google account access been disabled? I cannot sign in with my
gmail and I have lost a lot of content that I was in the process of
editing. Has this been moved to a new username? Or were those removed? Any
help is appreciated.
Additional Details
Share Link :
or
Brew code to reproduce :
Click to expand
PASTE BREW CODE HERE
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/stolksdorf/homebrewery/issues/555, or mute the thread
https://github.com/notifications/unsubscribe-auth/AFmzBh3Ce6oi-U3-chzxNwgPQoZ4y0usks5s4RySgaJpZM4Qjy_d
.

I was able to access it from my home laptop, just because my account is automatically logged in. I think it was just an issue from the other laptop I was trying to reach it from. I didn't have the typical drop down that offered google or facebook login when pressing the login button in top right. It took me to a new page entirely that just said "Natural Crit" and then had two entry queries, one for username and one for password. I had never seen that before and my google username/email and password didn't work.
Either way, my problem is solved for now, I just was very confused by the alternate login that I ran into. I am still unsure what caused that.


feat: code-based instance configuration support

Motivation:
Code-based instance configurations serve as an alternative to our standard JSON-based instance configuration. JSON-based configs will work in most use cases but they are limited when a more advanced setup is required. Code-based configs use the power of llama scripts to enable features such as creating strategies or accounts with multiple logic contracts and creating permissions based on dynamic conditions (e.g. deploying a guard and then using the deployed address as the target address).
Modifications:

Added a ConfigureAdvancedLlamaInstance.s.sol script that can be run for an instance deployed using an advanced configuration (instant execution strategy on setRolePermission with a single policyholder).
This script authorizes and executes LlamaInstanceConfigScript.sol which can be edited to describe the instance configuration using code. It also includes the post configuration cleanup that removes any trace of the config bot.
Added tests
Added a justfile command and documentation

Result:
Instances can be deployed using a standard JSON-based configuration or an advanced code-based configuration.

https://github.com/foundry-rs/foundry/issues/5776 is something we should be tracking so that we can simplify our advanced script. Ideally it should do all the following:

Deploy the LlamaInstanceConfig script and get its address
Deploy the llama instance with permissions for the LlamaInstanceConfig script pre-assigned (i.e setScriptAuthorization and LlamaInstanceConfig.execute). The only reason we're not doing it currently is because we don't know the target address yet (i.e the LlamaInstanceConfig script). But in this approach we will know what it is and can just use DeployUtils to set up the initial deploy.
Create an action to setScriptAuthorization and execute it.
Create an action to call LlamaInstanceConfig.execute and execute it.

foundry-rs/foundry#5776 is something we should be tracking so that we can simplify our advanced script. Ideally it should do all the following:

Deploy the LlamaInstanceConfig script and get its address
Deploy the llama instance with permissions for the LlamaInstanceConfig script pre-assigned (i.e setScriptAuthorization and LlamaInstanceConfig.execute). The only reason we're not doing it currently is because we don't know the target address yet (i.e the LlamaInstanceConfig script). But in this approach we will know what it is and can just use DeployUtils to set up the initial deploy.
Create an action to setScriptAuthorization and execute it.
Create an action to call LlamaInstanceConfig.execute and execute it.

Yeah I agree. If you look at past commits this was my initial approach. That being said I think this works fine for now and doesn't have any downside besides needing to run two scripts instead of one.


Docs about ECS integration

Is your request related to a new offering from AWS?
Nope
Is your request related to a problem? Please describe.
Kinda. I'm trying to make a state machine using ecs-fargate, and I'm getting IAM errors that I see that are related to #11. But I can't figure out what I need to do in service_integrations to connect ecs to the step function.
Apparently #19 fixed the integration, but the README wasn't updated :cry:
Describe the solution you'd like.
Add to the docs or the examples a snippet on how to use it with ECS.
Describe alternatives you've considered.

Additional context

cc @antonbabenko

Hi @lays147 !
I don't have the capacity to look into this issue in the near future.
Please make sure that you check the code in the examples/complete folder and provide the required details when you create an issue.


adfgvx cipher has fixed key

Note:

Get key along with input
create substitute table based on the key

This should be re checked does it work correctly. Maybe make it similar to this https://cryptii.com/pipes/adfgvx-cipher


Add Guid to Token authentication?

Any particular reason why user guid's are not integrated with Authenication.Token, as they are with Authentication.Forms? If not, I would consider this a good addition :+1:

Because nancy doesn't force you what kind of identifier to use to define your users. In case of Authentication.Forms they use guid to identify user, because to use username is too risky, but at the same time it doesn't mean this guid has to be your primary guid for users, you can use it as reference authentication guid. In case of Authenication.Token there is no need to use any guid, so they don't.
Sidenote.
https://github.com/NancyFx/Nancy/wiki/Token-Authentication
Docs say
It is recommended that you configure the Tokenizer to use an additional piece of information that can uniquely identify the client device.
So, actually they welcome you to put your user id there.


Start a private network tutorial passes secret seed on the command line

https://substrate.dev/docs/en/next/tutorials/start-a-private-network/customchain#option-2-using-cli recommends users to pass their mnemonic phrase on the commandline. This is a security vulnerability because the seed will be retained in eg bash_history. Probably best to remove that section and recommend users use Apps instead.

cc @demimarie-parity

Should we suggest an alternate way of accessing this capability (e.g. using a REST or WebSocket client)? It seems like the RPC API exposes this capability, so we should document it, but we should just do so in a way that encourages people to use it the way it was meant to be used. If the feature itself is fundamentally insecure, perhaps it should be removed altogether.

Option 1 which is already present recommends using Apps which uses the websocket endpoint

Another option could be with subkey. The only question that remains is, if subkey will live for long.
I think that subkey will become part of the main client and we could use that if we need to update it in the future.

Apps is a better alternative, but I am nervous about trusting a website with such sensitive information.

Is this an opportunity to cross-document Parity Signer? It's a very cool product. Is there a way we can make it easily available for people by way of some kind of VM or something?

Created the above PR, which solves the simple problem of removing the "bad" section from the tutorial. I did not set the PR to auto-close this Issue, though, since it sounds like there may be a bit more for us to discuss :+1:

Is this an opportunity to cross-document Parity Signer

Probably. A KB article might fit nicely in the integrate section.

The best option is to pass the secret seed as the contents of a file or as standard input.

I like the suggestion that @DemiMarie-parity is making, but I need to wait until the above issue is resolved before making the changes to this tutorial.

Based on the comment @bkchr made on paritytech/substrate#5817 I think we can use subkey as a safer alternative option to a web UI or something that pollutes bash_history. @DemiMarie-parity, do you agree that this capability of subkey can be used to address your concern?
$ subkey insert --help
subkey-insert
Insert a key to the keystore of a node

USAGE:
    subkey insert <suri> <key-type> [node-url]

FLAGS:
    -h, --help       Prints help information
    -V, --version    Prints version information

ARGS:
    <suri>        The secret key URI. If the value is a file, the file content is used as URI. If not given, you
                  will be prompted for the URI.
    <key-type>    Key type, examples: "gran", or "imon"
    <node-url>    Node JSON-RPC endpoint, default "http:://localhost:9933"

@danforbes If we switch from TCP to IPC, yes.

@DemiMarie-parity so you are saying me that you are afraid of an attack on your loopback device? When you run a tutorial?

@bkchr I am not, but if someone were to use Substrate in an educational setting on lab machines (which are usually shared multiuser systems), this would be a very legitimate risk.

Furthermore, a UXSS vulnerability in a web browser is sufficient for any site to connect to the loopback interface.  Such attacks are often easier than remote code execution.

We are talking here about a tutorial, not on how to deploy a production network.

I want to try to take a best-of-both-worlds approach to this. The tutorial's "happy path" should be an easy, straightforward approach that allows participants to focus on the real intent of the tutorial. However, if a tutorial is doing something that isn't really secure in production, I think it needs to be called out. Whatever is really secure in production also needs to be documented in a straightforward way; if it's straightforward enough, we can include it as an optional part of this tutorial, but if it becomes too complicated we can always just link to a dedicated doc.

I am okay with this if we add the document the attack vector clearly.  Multi-user systems are widely used in education, and in this context localhost attacks are a legitimate threat.

So, you think that someone reads the tutorial and directly after starts a live network from an education PC?
What is the problem when you run the tutorial and someone is able to see your local network authority keys?


bad proxy will raise exception: TypeError: options.request.emit is not a function

When I use an agent created by tunnel with proxy, if the proxy is not reachable, wrong exception raised.
Simple codes:
let a = tunnel.httpOverHttp({proxy: {host: 'nosuch.host.com', port: 8080}});
    return got(anurl, {agent: a})
    .then( res => {
        if (res.statusCode === 200) return res.body;
        else throw new Error('http status ' + res.statusCode + ', ' + res.body);
    });

exception raised:
TypeError: options.request.emit is not a function
    at ClientRequest.onError (/home/xxx/nodejs/proxylib/node_modules/tunnel/lib/tunnel.js:183:21)
    at Object.onceWrapper (events.js:300:26)
    at ClientRequest.emit (events.js:210:5)
    at Socket.socketErrorListener (_http_client.js:406:9)
    at Socket.emit (events.js:210:5)
    at emitErrorNT (internal/streams/destroy.js:92:8)
    at emitErrorAndCloseNT (internal/streams/destroy.js:60:3)
    at processTicksAndRejections (internal/process/task_queues.js:80:21)

But EHOSTNOTFOUND or ENETWORK are expected error.

I am getting this as well

I had the same issue, it appeared that I have used the wrong config

if authentication parameters are bad or any proxy parameter is bad, exceptions are thrown.. Is there a way to catch these exceptions and show to the user what went wrong?


make bridge mtu configurable and default to 1450 instead of 1500

to allow for zer-touch cloud based installations of containerlab the proposal is to enforce mtu for clab bridge to be 1450

the subshelling branch contains a struct and functionality for setting the MTU. This would then just need to be further exposed towards the input file.

@steiler I think your PR introduces ability to set the MTU on the non-management links. The links that we create either for p2p connectivity, or point-to-bridge connection.
Here I am talking about the management network mtu, that docker creates to attach management interfaces of the containters


Add a reader contribution that supports ome-zarr

This adds a reader contribution that vendors the reader from napari-ome-zarr and makes some modifications to it so that the extra-napari metadata support this plugin enables (e.g. axes) can be read from the ome-zarr directory. It also fixes some potential bugs in napari-ome-zarr I found while writing tests.
I also fixed some integration bugs found while doing an edit-write-read loop. I'm sure there are more to find and fix, but this gets a few of them.

Codecov Report

Merging #19 (b3e9298) into main (65cc0ee) will decrease coverage by 2.35%.
The diff coverage is 85.54%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main      #19      +/-   ##
==========================================
- Coverage   93.64%   91.30%   -2.35%
==========================================
  Files          13       15       +2
  Lines         834     1150     +316
==========================================
+ Hits          781     1050     +269
- Misses         53      100      +47

Impacted Files
Coverage Δ

src/napari_metadata/_reader.py
65.87% <65.87%> (ø)

src/napari_metadata/_model.py
87.50% <85.00%> (+0.98%)
:arrow_up:

src/napari_metadata/_axis_type.py
94.11% <85.71%> (-5.89%)
:arrow_down:

src/napari_metadata/_spatial_units_combo_box.py
92.10% <85.71%> (-1.65%)
:arrow_down:

src/napari_metadata/_widget.py
91.17% <85.71%> (-1.14%)
:arrow_down:

src/napari_metadata/_axes_name_type_widget.py
97.93% <100.00%> (+1.73%)
:arrow_up:

src/napari_metadata/_axes_spacing_widget.py
97.61% <100.00%> (ø)

src/napari_metadata/_tests/test_reader.py
100.00% <100.00%> (ø)

src/napari_metadata/_tests/test_widget.py
99.58% <100.00%> (+0.07%)
:arrow_up:

src/napari_metadata/_tests/test_writer.py
99.11% <100.00%> (+0.02%)
:arrow_up:

... and 3 more

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


fix: clicking enter on emoji selector for status results in leaving status pane.

Details

Fixed Issues
$ https://github.com/Expensify/App/issues/47272
PROPOSAL: https://github.com/Expensify/App/issues/47272#issuecomment-2285038374
Tests

Head to status
Click on emoji selector
Search for any emoji
When the emoji you want is auto-focused, hit enter
Verify the emoji is selected and input is focused

[x] Verify that no errors appear in the JS console

Offline tests

Head to status
Click on emoji selector
Search for any emoji
When the emoji you want is auto-focused, hit enter
Verify the emoji is selected and input is focused

QA Steps

Head to status
Click on emoji selector
Search for any emoji
When the emoji you want is auto-focused, hit enter
Verify the emoji is selected and input is focused

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/user-attachments/assets/8f8a2512-f483-49c9-a539-be1a3f496232

Android: mWeb Chrome
https://github.com/user-attachments/assets/1beadd90-6973-4ce0-802f-bb99ddb2aebe

iOS: Native
https://github.com/user-attachments/assets/15014596-3e13-4df4-944a-18364cfc28d3

iOS: mWeb Safari
https://github.com/user-attachments/assets/749473ed-ff39-4259-ba9c-f2f5256c88ea

MacOS: Chrome / Safari
https://github.com/user-attachments/assets/ad980cc1-cbd9-4ebb-b4c4-577164e1ee65

MacOS: Desktop
https://github.com/user-attachments/assets/6bc68b36-c7f9-436b-8090-60758b40ab9f

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/carlosmiceli in version: 9.0.26-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 9.0.26-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


[SPARK-46913][SS] Add support for processing/event time based timers with transformWithState operator

What changes were proposed in this pull request?
Add support for processing/event time based timers with transformWithState operator
Why are the changes needed?
Changes are required to add event-driven timer based support for stateful streaming applications based on arbitrary state  API with the transformWithState operator
As part of this change - we introduce a bunch of functions that users can use within the StatefulProcessor logic. Using the StatefulProcessorHandle, users can do the following:

register timer at a given timestamp
delete timer at a given timestamp
list timers

Note that all the above operations are tied to the implicit grouping key.
In terms of the implementation, we make use of additional column families to support the operations mentioned above. For registered timers, we maintain a primary index (as a col family) that keeps the mapping between the grouping key and expiry timestamp. This col family is used to add and delete timers with direct access to the key and also for listing registered timers for a given grouping key using prefix scan. We also maintain a secondary index that inverts the ordering of the timestamp and grouping key and maintains the list of (expiry) timestamps in sorted order (using BIG_ENDIAN encoding) within RocksDB. This is because RocksDB uses bytewise comparison using the default comparator to determine sorted order of keys. This is used to read expired timers at any given processing time/event time timestamp threshold by performing a range scan.
Few additional constraints:

only registered timers are tracked and occupy storage (locally and remotely)
col families starting with _ are reserved and cannot be used as state variables
timers are checkpointed as before
users have to provide a timeoutMode to the operator. Currently, they can choose to not register timeouts or register timeouts that are processing-time based or event-time based. However, this mode has to be declared upfront within the operator arguments.

Does this PR introduce any user-facing change?
Yes
How was this patch tested?
Added unit tests as well as pseudo-integration tests
StatefulProcessorHandleSuite
13:58:42.463 WARN org.apache.spark.sql.execution.streaming.state.StatefulProcessorHandleSuite:

===== POSSIBLE THREAD LEAK IN SUITE o.a.s.sql.execution.streaming.state.StatefulProcessorHandleSuite, threads: rpc-boss-3-1 (daemon=true), shuffle-boss-6-1 (daemon=true) =====
[info] Run completed in 4 seconds, 559 milliseconds.
[info] Total number of tests run: 8
[info] Suites: completed 1, aborted 0
[info] Tests: succeeded 8, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.

TransformWithStateSuite
13:48:41.858 WARN org.apache.spark.sql.streaming.TransformWithStateSuite:

===== POSSIBLE THREAD LEAK IN SUITE o.a.s.sql.streaming.TransformWithStateSuite, threads: QueryStageCreator-0 (daemon=true), state-store-maintenance-thread-0 (daemon=true), ForkJoinPool.commonPool-worker-4 (daemon=true), state-store-maintenance-thread-1 (daemon=true), QueryStageCreator-1 (daemon=true), rpc-boss-3-1 (daemon=true), F
orkJoinPool.commonPool-worker-3 (daemon=true), QueryStageCreator-2 (daemon=true), QueryStageCreator-3 (daemon=true), state-store-maintenance-task (daemon=true), ForkJoinPool.com...
[info] Run completed in 1 minute, 32 seconds.
[info] Total number of tests run: 20
[info] Suites: completed 1, aborted 0
[info] Tests: succeeded 20, failed 0, canceled 0, ignored 0, pending 0
[info] All tests passed.

Was this patch authored or co-authored using generative AI tooling?
No

@HeartSaVioR - PTAL, thx !

Thanks! Merging to master.


Empty result

Hi!
When starting python3 aplot table -p /path/to/atop_yyyymmdd CPL.avg5 I receive messages "empty result".
What could be the problem?

I using atop-2.4

Thank you for reporting this. The project is not actively maintained as stated in the readme. I’m not sure when I will have time to have a look into it.
Are you interested in submitting a pull request?


Default background dependent on dark mode settings

I am currently using Firefox 82.0.2 (64-bit) on Windows 10 1909 and 2004.
When Firefox is in Dark Mode (following the system settings), the page still flashes white when a new tab is opened.
It can be found in the Inspect Elements that the default body background is still white, even if I set #000000 for the custom background, and #FFFFFF for the foreground:
body {
    margin: 0;
    position: relative;
    background: #fff;
    font-family: "Open Sans",sans-serif;
    font-size: 1.6rem;
    color: #000;
    overflow: hidden;
}

If the extension can follow Firefox's dark mode settings, and use the dark background as the default one, it would drastically improve the overall experience!
And it would be great if users are allowed to choose which mode (light/dark/depends-on-system-settings) the extension uses!
Thank you for developing such a great extension!

I unfortunately do not have the ability to fix this. I can only affect things after the extension has loaded, I have no access to what happens before loading and therefore am unable to change this. It is actually a failure of all the browsers that they still default to a white background even for dark mode before the extension has time to load.
Thank you for your support!

Whoops I'm sorry, bit tired right now when I looked at this and was thinking of it wrong. I re-read your info and you might have something there, reopening and marking as a bug 👍

Fixed in version 3.1.0

Fixed in version 3.1.0

Sorry to comment on a closed issue, however the fix does not seem to work for me. I still see bright white flashes although my Firefox theme is dark. It might be related to the macOS theme since I actually get black flashes when using a bright carettab background. Is it maybe not only picking up on the firefox theme but the OS theme as well?
I could not however get the bright flash to disappear with any combination of bright, dark and auto/default theme in macOS and Firefox. The media queries look fine though, for all I can see, I am no expert on that matter.
@media (prefers-color-scheme: dark) {
  body {
    background: #000;
    color: #bbb;
  }
}

@media (prefers-color-scheme: light) {
  body {
    background: #fff;
    color: #555;
  }
}

#splashScreen {
  opacity: 1;
  transition: opacity .3s ease-in-out;
  position: fixed;
  height: 100vh;
  width: 100vw;
  top: 0;
  left: 0;
  z-index: 99999;
}
#splashScreen.loaded {
  opacity: 0;
}

Firefox 85.0, macOS 10.15.7
Is there anything else I can try or take a look at, with no custom tab the background is dark so I suspect some interaction with the extension, or should I open a new issue?
Apart from that, awesome little extension though, I enjoy opening tabs even more :)

Sorry to comment on a closed issue, however the fix does not seem to work for me. I still see bright white flashes although my Firefox theme is dark. It might be related to the macOS theme since I actually get black flashes when using a bright carettab background. Is it maybe not only picking up on the firefox theme but the OS theme as well?
I could not however get the bright flash to disappear with any combination of bright, dark and auto/default theme in macOS and Firefox. The media queries look fine though, for all I can see, I am no expert on that matter.
@media (prefers-color-scheme: dark) {
  body {
    background: #000;
    color: #bbb;
  }
}

@media (prefers-color-scheme: light) {
  body {
    background: #fff;
    color: #555;
  }
}

#splashScreen {
  opacity: 1;
  transition: opacity .3s ease-in-out;
  position: fixed;
  height: 100vh;
  width: 100vw;
  top: 0;
  left: 0;
  z-index: 99999;
}
#splashScreen.loaded {
  opacity: 0;
}

Firefox 85.0, macOS 10.15.7
Is there anything else I can try or take a look at, with no custom tab the background is dark so I suspect some interaction with the extension, or should I open a new issue?
Apart from that, awesome little extension though, I enjoy opening tabs even more :)


Function win_plus_char incomplete

The function win_plus_char in the .ino file is suppose to emulate the pressing of win(or its equivalent)+<some char> on a keyboard. Complete it.
Bounty Points: 30

if the enter issue is correct can you please assign me this.


Error when restocking

After this last update, restocking scanned professions doesn't work as they used to.

I scan the AH with Auctionator
I do a recipe scan, in this case Enchanting

Go to "Craft Queue" and click on "Restock from Recipe Scan"

From what I've been testing, the "Sale Rate Threshold" is not working properly. I normally have it at 0.1 and it used to work fine, but no matter what value I add, it only adds items to the crafting queue if the value is 0

this was a tsm issue and should now work again


Replace 'startsWith' with checking individual characters

Pre-Submission Checklist

[x] Branches much start with either fix/ or feature/
[x] You have only one commit (if not, [squash](https://github.com/FreeCodeCamp/         FreeCodeCamp/wiki/git-rebase#squashing-multiple-commits-into-one) them into one commit).
[x] All new and existing tests pass npm run test-challenges.

Description

Replaces !value.startsWith('//') with !(value.charAt(0) === '/' && value.charAt(1) === '/') to ensure Internet Explorer support.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue: Closes #15558

When will this be merged?

I don't have the ability to merge, so I don't know, sorry. 🙂

Thank you for providing a patch for this, please see my comment :+1:

I'll change that. Thanks for the suggestion

I switched to regex.

@Bouncey Done.

Great work @CodingPower472 thank you
LGTM 👍


Support stackprof object mode

First off, apologies for not opening and issue to discuss this ahead of time as recommended in the contributing guide -- I kind of worked on this on a whim this morning.
This PR attempts to support stackprof's object mode which tracks the number of allocated objects. This differs from the other modes (cpu and wall) by taking samples every time a Ruby object is allocated using Ruby's NEWOBJ tracepoint.
When importing an object mode profile into speedscope today it still works but what you see is a profile using time units. The profile will only have samples for when an object was allocated which means even if time is reported, the profile is not really meaningful when looking at time.
To address this I've done three things when mode is object:

adjusted the total size of the StackListProfileBuilder to use the number of samples (since each sample is one allocation)
adjusted the weight of each sample to be nSamples (which I believe is always 1 but I'm not positive)
do not set the value formatter to a time formatter

Here's what it looks like before and after my changes (note the units and weight of samples):

wall (before)
object (before)
object (after)

Test code
require 'stackprof'
require 'json'

def do_test
  5.times do
    make_a_word
  end
end

def make_a_word
  ('a'..'z').to_a.shuffle.map(&:upcase).join
end

StackProf.start(mode: :object, interval: 1, raw: true)
do_test
StackProf.stop

File.write('tmp/object_profile.json', JSON.generate(StackProf.results))

StackProf.start(mode: :wall, interval: 1, raw: true)
do_test
StackProf.stop

File.write('tmp/wall_profile.json', JSON.generate(StackProf.results))

Finally, I am not super familiar with TypeScript and even less familiar with this codebase so I'm not sure what tests, if any, are needed for a change like this. I'm happy to add whatever else is needed. 🙇

Hi @alexcoco! Thanks for adding this -- this seems totally reasonable to me.
Before this can land, this needs tests to cover the new codepath. You can see how the other test for stackprof works by looking here: https://github.com/jlfwong/speedscope/blob/main/src/import/stackprof.test.ts
That test file was generated, I believe, using the following test file: https://github.com/jlfwong/speedscope/blob/main/sample/programs/ruby/simple.rb
Please include the ruby file you used to profile, and a new profile. Both the program and the resulting profile should be as short as possible while still exercising the core functionality.
Let me know if you need any help!
Speedscope is badly in need of a new release. I'm planning on doing that sometime in the next month.

Coverage increased (+0.06%) to 41.14% when pulling 9e856e354567dcb0cb7a2db2461def43d76bb8f4 on alexcoco:stackprof-object-mode into e37f6fa7c38c110205e22081560b99cb89ce885e on jlfwong:main.


[BUG] Monitors notifications channel not working With RBAC setting

Bug
While Creating Monitors in that Triggers actions notification channel is not working. I have created the notification channel as custom webhook. When we hit send test message over there it works .But when do send test message while adding monitor it fails .
How can one reproduce the bug?
Steps to reproduce the behavior:

enable backend role filtering
PUT _cluster/settings
{
"transient": {
"plugins.alerting.filter_by_backend_roles": "true"
}
}
Then Create monitor
Add the trigger with action as notification channel as webhook
Try to send the test message will throw an error.and also check the opensearch.log file
Error screenshots and logs are attach below please check.

Problem is it fails to read the backend roles for the user.when filter_by_backend_roles is enabled
host/environment

OS: CentOS 7.9
Version  7.9
Plugins Alert Plugin
Opensearch 2.3

screenshots?

OpenSearch logs
[2022-12-09T18:59:50,893][INFO ][o.o.n.i.ConfigIndexingActions] [test2-server] notifications:NotificationConfig-get org.opensearch.commons.notifications.action.GetNotificationConfigRequest@43c2eec6
[2022-12-09T18:59:50,893][WARN ][o.o.n.a.PluginBaseAction ] [test2-server] notifications:OpenSearchStatusException:
org.opensearch.OpenSearchStatusException: User doesn't have backend roles configured. Contact administrator.
at org.opensearch.notifications.security.UserAccessManager.validateUser(UserAccessManager.kt:24) ~[opensearch-notifications-<IP_ADDRESS>.jar:<IP_ADDRESS>]
at org.opensearch.notifications.index.ConfigIndexingActions.get(ConfigIndexingActions.kt:251) ~[opensearch-notifications-<IP_ADDRESS>.jar:<IP_ADDRESS>]
at org.opensearch.notifications.action.GetNotificationConfigAction.executeRequest(GetNotificationConfigAction.kt:60) ~[opensearch-notifications-<IP_ADDRESS>.jar:<IP_ADDRESS>]
at org.opensearch.notifications.action.GetNotificationConfigAction.executeRequest(GetNotificationConfigAction.kt:26) ~[opensearch-notifications-<IP_ADDRESS>.jar:<IP_ADDRESS>]
at org.opensearch.notifications.action.PluginBaseAction$doExecute$1.invokeSuspend(PluginBaseAction.kt:63) [opensearch-notifications-<IP_ADDRESS>.jar:<IP_ADDRESS>]
at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33) [kotlin-stdlib-1.6.10.jar:1.6.10-release-923(1.6.10)]
at kotlinx.coroutines.DispatchedTask.run(DispatchedTask.kt:106) [kotlinx-coroutines-core-jvm-1.4.3.jar:?]
at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:571) [kotlinx-coroutines-core-jvm-1.4.3.jar:?]
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.executeTask(CoroutineScheduler.kt:750) [kotlinx-coroutines-core-jvm-1.4.3.jar:?]
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.runWorker(CoroutineScheduler.kt:678) [kotlinx-coroutines-core-jvm-1.4.3.jar:?]
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:665) [kotlinx-coroutines-core-jvm-1.4.3.jar:?]
[2022-12-09T18:59:50,894][INFO ][o.o.a.m.MonitorRunResult ] [test2-server] Internal error: Unable to find a Notification Channel or Destination config with id [nsgT94QBikbo0yuk80z2]. See the opensearch.log for details
java.lang.IllegalStateException: Unable to find a Notification Channel or Destination config with id [nsgT94QBikbo0yuk80z2]
at org.opensearch.alerting.MonitorRunner.getConfigAndSendNotification(MonitorRunner.kt:93) ~[opensearch-alerting-<IP_ADDRESS>.jar:<IP_ADDRESS>]
at org.opensearch.alerting.MonitorRunner$getConfigAndSendNotification$1.invokeSuspend(MonitorRunner.kt) ~[opensearch-alerting-<IP_ADDRESS>.jar:<IP_ADDRESS>]
at kotlin.coroutines.jvm.internal.BaseContinuationImpl.resumeWith(ContinuationImpl.kt:33) [kotlin-stdlib-1.6.10.jar:1.6.10-release-923(1.6.10)]
at kotlinx.coroutines.DispatchedTask.run(Dispatched.kt:233) [kotlinx-coroutines-core-1.1.1.jar:?]
at kotlinx.coroutines.scheduling.CoroutineScheduler.runSafely(CoroutineScheduler.kt:594) [kotlinx-coroutines-core-1.1.1.jar:?]
at kotlinx.coroutines.scheduling.CoroutineScheduler.access$runSafely(CoroutineScheduler.kt:60) [kotlinx-coroutines-core-1.1.1.jar:?]
at kotlinx.coroutines.scheduling.CoroutineScheduler$Worker.run(CoroutineScheduler.kt:742) [kotlinx-coroutines-core-1.1.1.jar:?]

@techrna, can you please paste the results for the getUser API for the user that is making the call?

@techrna, can you run the following: GET .opendistro-alerting-config/_doc/<monitor-id>?
The reason to do this is to get the user information stored on the monitor and to see if the backend role is correctly added there as it should be passed to the Notification plugin from there.
Additionally, can you do the [getRoleMapping](GET .opendistro-alerting-config/_doc/c3CwFYQBxyWm8H1EhccS) API for the role, dgvzdf8xx2toawthlte? Just want to make sure that role is associated to the user you shared.
One last question, if you create the monitor with that notification channel associated to the trigger, does the monitor correctly send out notifications when alerts are generated?

@lezzago I tried
Monitor id : HbWuYYUBb9hzdJ2RpaJ5
monitor_type : query_level_monitor
GET .opendistro-alerting-config/_doc/HbWuYYUBb9hzdJ2RpaJ5
{
  "_index" : ".opendistro-alerting-config",
  "_id" : "HbWuYYUBb9hzdJ2RpaJ5",
  "found" : false
}

Monitor Json
GET _plugins/_alerting/monitors/HbWuYYUBb9hzdJ2RpaJ5
{
  "_id" : "HbWuYYUBb9hzdJ2RpaJ5",
  "_version" : 2,
  "_seq_no" : 833,
  "_primary_term" : 35,
  "monitor" : {
    "type" : "monitor",
    "schema_version" : 5,
    "name" : "linux_multiple_logon_failures [workstation]",
    "monitor_type" : "query_level_monitor",
    "enabled" : true,
    "enabled_time" :<PHONE_NUMBER>677,
    "schedule" : {
      "period" : {
        "interval" : 1,
        "unit" : "MINUTES"
      }
    },
    "inputs" : [
      {
        "search" : {
          "indices" : [
            "rushab-dgvzdf8zx2toawthltg-workstation-archives-*"
          ],
          "query" : {
            "size" : 100,
            "query" : {
              "bool" : {
                "filter" : [
                  {
                    "range" : {
                      "@timestamp" : {
                        "from" : "{{period_end}}||-1h",
                        "to" : "{{period_end}}",
                        "include_lower" : true,
                        "include_upper" : true,
                        "format" : "epoch_millis",
                        "boost" : 1.0
                      }
                    }
                  },
                  {
                    "terms" : {
                      "rule.id" : [
                        "5763",
                        "5712"
                      ],
                      "boost" : 1.0
                    }
                  }
                ],
                "adjust_pure_negative" : true,
                "boost" : 1.0
              }
            },
            "aggregations" : { }
          }
        }
      }
    ],
    "triggers" : [
      {
        "query_level_trigger" : {
          "id" : "Mel7D4UBXvIZ4pdFqgDQ",
          "name" : "Test Alert trigger",
          "severity" : "2",
          "condition" : {
            "script" : {
              "source" : "ctx.results[0].hits.total.value > 0",
              "lang" : "painless"
            }
          },
          "actions" : [
            {
              "id" : "Mul7D4UBXvIZ4pdFqgDQ1",
              "name" : "dgvzdf8zx2toawthltg_channel_webhook",
              "destination_id" : "dgvzdf8zx2toawthltg_channel_webhook",
              "message_template" : {
                "source" : """
Trigger name: {{ctx.trigger.name}}
""",
                "lang" : "mustache"
              },
              "throttle_enabled" : true,
              "subject_template" : {
                "source" : "",
                "lang" : "mustache"
              },
              "throttle" : {
                "value" : 25,
                "unit" : "MINUTES"
              }
            }
          ]
        }
      }
    ],
    "last_update_time" :<PHONE_NUMBER>218
  }
}

@lezzago i have added rbac_role while creating but its not reflecting in it.
and also tried updating the monitor. same result its not reflecting added/updated rbac_roles.
User and roles mapping

GET _plugins/_security/api/rolesmapping/dgvzdf8zx2toawthltg
{
  "dgvzdf8zx2toawthltg" : {
    "hosts" : [ ],
    "users" : [
<EMAIL_ADDRESS>    ],
    "reserved" : false,
    "hidden" : false,
    "backend_roles" : [ ],
    "and_backend_roles" : [ ]
  }
}

GET<EMAIL_ADDRESS>{
<EMAIL_ADDRESS>: {
    "hash" : "",
    "reserved" : false,
    "hidden" : false,
    "backend_roles" : [
      "dgvzdf8zx2toawthltg"
    ],
    "attributes" : { },
    "opendistro_security_roles" : [ ],
    "static" : false
  }
}

@techrna, sorry, I forgot to share the correct method to query .opendistro-alerting-config. It is a system index, so to query it, follow this guide.
The command to query it should look like curl -k --cert ./kirk.pem --key ./kirk-key.pem -XGET 'https://localhost:9200/.opendistro-alerting-config/_doc/HbWuYYUBb9hzdJ2RpaJ5'
Also the reason you do not see the rbac roles associated to the monitor is prevent leaking security credential information. If you run the command above, you should get user information tied to the monitor and that will include the rbac_roles. So when you get that, feel free to also obfuscate any important security credentials.
Also can you share the output of GET<EMAIL_ADDRESS>since<EMAIL_ADDRESS>made the initial call?

@lezzago<EMAIL_ADDRESS>its new user created with same settings
GET<EMAIL_ADDRESS>{
<EMAIL_ADDRESS>: {
    "hash" : "",
    "reserved" : false,
    "hidden" : false,
    "backend_roles" : [
      "dgvzdf8zx2toawthltg"
    ],
    "attributes" : { },
    "opendistro_security_roles" : [ ],
    "static" : false
  }
}

GET<EMAIL_ADDRESS>{
<EMAIL_ADDRESS>: {
    "hash" : "",
    "reserved" : false,
    "hidden" : false,
    "backend_roles" : [
      "dgvzdf8xx2toawthlte"
    ],
    "attributes" : { },
    "opendistro_security_roles" : [ ],
    "static" : false
  }
}

curl -k --cert ./kirk.pem --key ./kirk-key.pem -XGET 'https://localhost:9200/.opendistro-alerting-config/_doc/HbWuYYUBb9hzdJ2RpaJ5
{
  "_index": ".opendistro-alerting-config",
  "_id": "HbWuYYUBb9hzdJ2RpaJ5",
  "_version": 3,
  "_seq_no": 847,
  "_primary_term": 35,
  "found": true,
  "_source": {
    "monitor": {
      "type": "monitor",
      "schema_version": 5,
      "name": "linux_multiple_logon_failures [workstation]",
      "monitor_type": "query_level_monitor",
      "user": {
        "name"<EMAIL_ADDRESS>        "backend_roles": [
          "dgvzdf8zx2toawthltg"
        ],
        "roles": [
          "dgvzdf8zx2toawthltg",
          "own_index"
        ],
        "custom_attribute_names": [

        ],
        "user_requested_tenant": null
      },
      "enabled": true,
      "enabled_time":<PHONE_NUMBER>677,
      "schedule": {
        "period": {
          "interval": 1,
          "unit": "MINUTES"
        }
      },
      "inputs": [
        {
          "search": {
            "indices": [
              "rushab-dgvzdf8zx2toawthltg-workstation-archives-*"
            ],
            "query": {
              "size": 100,
              "query": {
                "bool": {
                  "filter": [
                    {
                      "range": {
                        "@timestamp": {
                          "from": "{{period_end}}||-1h",
                          "to": "{{period_end}}",
                          "include_lower": true,
                          "include_upper": true,
                          "format": "epoch_millis",
                          "boost": 1.0
                        }
                      }
                    },
                    {
                      "terms": {
                        "rule.id": [
                          "5763",
                          "5712"
                        ],
                        "boost": 1.0
                      }
                    }
                  ],
                  "adjust_pure_negative": true,
                  "boost": 1.0
                }
              },
              "aggregations": {

              }
            }
          }
        }
      ],
      "triggers": [
        {
          "query_level_trigger": {
            "id": "Mel7D4UBXvIZ4pdFqgDQ",
            "name": "Test Alert trigger",
            "severity": "2",
            "condition": {
              "script": {
                "source": "ctx.results[0].hits.total.value > 0",
                "lang": "painless"
              }
            },
            "actions": [
              {
                "id": "Mul7D4UBXvIZ4pdFqgDQ1",
                "name": "dgvzdf8zx2toawthltg_channel_webhook",
                "destination_id": "dgvzdf8zx2toawthltg_channel_webhook",
                "message_template": {
                  "source": ".................",
                  "lang": "mustache"
                },
                "throttle_enabled": false,
                "subject_template": {
                  "source": "",
                  "lang": "mustache"
                }
              }
            ]
          }
        }
      ],
      "last_update_time":<PHONE_NUMBER>262,
      "ui_metadata": {
        "schedule": {
          "cronExpression": "0 */1 * * *",
          "period": {
            "unit": "MINUTES",
            "interval": 1
          },
          "timezone": null,
          "daily": 0,
          "monthly": {
            "type": "day",
            "day": 1
          },
          "weekly": {
            "tue": false,
            "wed": false,
            "thur": false,
            "sat": false,
            "fri": false,
            "mon": false,
            "sun": false
          },
          "frequency": "interval"
        },
        "search": {
          "searchType": "query",
          "bucketValue": 1,
          "timeField": "",
          "bucketUnitOfTime": "h",
          "where": {
            "fieldName": [

            ],
            "fieldRangeEnd": 0,
            "fieldRangeStart": 0,
            "fieldValue": "",
            "operator": "is"
          },
          "groupBy": [

          ],
          "aggregations": [

          ]
        },
        "triggers": {
          "Brute Force": {
            "value": 10000,
            "enum": "ABOVE"
          }
        },
        "monitor_type": "query_level_monitor"
      }
    }
  }
}

Can you send the test message as the<EMAIL_ADDRESS>user? The monitor has the same backend roles as<EMAIL_ADDRESS>and not<EMAIL_ADDRESS>so that can be part of the issue.

Hi @techrna @lezzago
Are we able to solve this issue as I am also facing the same problem, and this seems a big blocker in my project delivery as if I am disabling the configuration of filtering_by_backend_role its working but than my all tenants was able to see each other's data like their own created monitors and if I am enabling, I am not able to trigger any notification
Please help

Looks like there is an issue in passing in the backend roles to the notification plugin when sending the notification.
The current workaround should be:
PUT _cluster/settings
{
  "persistent": {
    "opensearch.notifications.general.filter_by_backend_roles": "false"
  }
}

This will only disable RBAC for notification and still allow you to have it enabled on Alerting resources.

Thanks, @lezzago
Yes, I was able to send the notification by using the cluster setting as mentioned by you, but this change introduced a big security flaw for me as my product tenants segregated based on their roles was able to see everybody else notification channels
I am able to fix the same by changing the code inside notification-plugin repo,
I commented the method which was throwing this role error inside kotlin file and built the new jar out of it, that's the workaround I did so that with the role-based isolation, notification will also work, but will upgrade with the release once you will give the fix of passing those backend roles to notification plugin.
Please do update here once you fix the issue and give the release with that fix

Closing this issue as the PR has been merged and will be part of the 2.7 release


Unable to zoom back in after zooming out with an OrthoPerspectiveCamera

Describe the bug 📝
If I zoom out to a far distance using an OrthoPerspectiveCamera and then when I attempt to zoom back in the camera does not zoom back in correctly and instead remains far away from the object in view.
This can also be seen in the documentation tutorial https://docs.thatopen.com/Tutorials/OrthoPerspectiveCamera
Reproduction ▶
https://docs.thatopen.com/Tutorials/OrthoPerspectiveCamera
Steps to reproduce 🔢
No response
System Info 💻
System:
    OS: Linux 5.15 Ubuntu 22.04.3 LTS 22.04.3 LTS (Jammy Jellyfish)
    CPU: (20) x64 12th Gen Intel(R) Core(TM) i9-12900HK
    Memory: 27.22 GB / 31.19 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 18.16.1 - ~/.nvm/versions/node/v18.16.1/bin/node
    npm: 9.5.1 - ~/.nvm/versions/node/v18.16.1/bin/npm
  npmPackages:
    openbim-components: ^1.3.4 => 1.3.4

Browsers: Chrome

Used Package Manager 📦
npm
Error Trace/Logs 📃
No response
Validations ✅

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a repository issue and not a framework-specific issue. For example, if it's a THREE.js related bug, it should likely be reported to mrdoob/threejs instead.
[X] Check that this is a concrete bug. For Q&A join our Community.
[X] The provided reproduction is a minimal reproducible example of the bug.

I have a similar problem too, if i zoom out to a far distance then i can't zoom out further, but if i zooming in is still possible, it seems that the further you zoom out, the less sensitive

Hey, for the camera we just use yomotsu's camera controls. They are available in the controls property of all our cameras. We just use those controls without doing anything else, so there's not much we can do 🤔 maybe you can report that error in their library? Alternatively, you can provide a feature to zoom back to the origin, like shown in this tutorial. If there's something else we can do, please let us know! Closing this issue for now, as it seems something related to yomotsu's library. Cheers!


Resolving a circular single reference causes a stack overflow

Description
I had a situation where I'm referring to a singleton class in a recursive manor.  I thought this should work since the child class would just receive a pointer to the parent class during resolution.  For some reason it's not so I think this may be a bug and I'm looking for a workaround.
To Reproduce
Here is a simple example that demonstrates the issue.
        [TestMethod]
        public void TestUnitySingletonRecursion()
        {
            var container = new UnityContainer();
            container.RegisterSingleton<ISingletonTestInterface, SingletonTestClass>();
            container.RegisterSingleton<IChildInterface, ChildClass1>("type1");
            container.RegisterSingleton<IChildInterface, ChildClass2>("type2");

            container.Resolve<ISingletonTestInterface>();

        }

    public interface ISingletonTestInterface
    {
        void Interface1();
    }

    public interface IChildInterface
    {
        void ChildInterface();
    }

    public class SingletonTestClass : ISingletonTestInterface
    {
        private readonly IChildInterface[] _Children;

        public SingletonTestClass(IChildInterface[] children)
        {
            _Children = children;
        }

        public void Interface1()
        {
            _Children.ForEach(x=>x.ChildInterface());
        }
    }

    public class ChildClass1 : IChildInterface
    {

        public void ChildInterface()
        {
            Console.WriteLine("Class1");
        }
    }

    public class ChildClass2 : IChildInterface
    {
        // This simulates what I'm trying to.  This child will ultimately need to make
       // a call to the SingletonClass, but I can't even get it to resolve.  If it remove the
       // dependency attribute it will resolve.
        [Dependency]
        public ISingletonTestInterface SingletonClass {get; set; }
        public void ChildInterface()
        {
            Console.WriteLine("Class2");
        }
    }

Additional context
I tried changing both children to be registered as type PerResolveLifetimeManager and that led to another error I can't explain.
Exception is: InvalidOperationException - Value of ContainerControlledLifetimeManager can only be set once
I'm using version <IP_ADDRESS> of Unity.Container

Duplicate #225


Recently used apps picker

A basic one is in place already. Still to do:

[x] Make sure scrolling is sensible
[ ] Keyboard shortcuts for each app.
[ ] Artwork for apps

Should the version of the app be displayed, too? Started different versions of SpotBugs but was then unable to distinguish between them.

When there are multiple 'pinned' versions in the history list, then yes. But mostly I hope users won't do that.

BTW I made some visual improvements to the app history picker.

Going to split this into separate issues.