Question: How to test multiple plugins / run multiple suites at once?

Is your feature request related to a problem? Please describe.
We are using Semantic Kernel, which allows us to define multiple plugins. Each plugin has one prompt template and one config object, which includes OpenAI parameters like max_tokens and temperature. We have multiple plugins, and as such, would like to test all plugins with one command. Currently, we have to run promptfoo evaluate on each plugin, as each has its own provider and set of tests.
Describe the solution you'd like
Ideally, we'd be able to run promptfoo evaluate on an array of EvaluateTestSuite objects and get one clean report and pass/fail result at the end of the command.
After running, we'd love to see an output.json that's an array of results, and a promptfoo view screen that's an array of tables with merged metrics (total cost, etc.) at the top.
Describe alternatives you've considered
Currently we are working on a custom script that calls promptfoo evaluate multiple times, and we are forgoing the (very nice!) database view provided by promptfoo view in favor of a single command that gives us a clear pass/fail result on all our plugins at once.
Additional context
To clarify, we have multiple providers because we provide different OpenAI parameters to each one. Each provider-promptTemplate-testBatch triplet is unique: we don't want provider 1 to run anything from testBatch 2, nor do we want testBatch 2 to be ran against a prompt from promptTemplate 3, if that makes sense.
For example, imagine:
Suite 1:

provider: StoryWriter, max_tokens: 10_000
promptTemplate: "Write me a story about {{ input }}"
tests: Assert that input is present in the output as least 3 times, and the output is at least 5_000 tokens

Suite 2:

provider: Brainstormer, max_tokens: 1_000
promptTemplate: "Give me some ideas for {{ input }}"
tests: Assert that the output is a set of 3 bullet points

Clearly, we don't want to mix these, but we would like to be able to test both in one command! Apologies if I'm missing something!

Hi @mwiemer-microsoft,  you might want to check out this example for running several multi-turn conversations in one evaluation: Multiple-Turn Conversation Example.
I’d need a bit more context about your configuration to make specific recommendations. If you could include a minimal example to reproduce the issue, that would be super helpful. Alternatively, feel free to email me at<EMAIL_ADDRESS>or book a slot on my calendar: https://cal.com/michael-dangelo/15min


Minimal Code Error

The end code snippet the DateTime is missing the 'e' to the right of the '='

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

ID: 20f37a0b-3b5d-225f-a349-ee108ff4ec60
Version Independent ID: 00909a12-d24a-f3f5-d246-8a5cb2431438
Content: Compiler Error CS0234
Content Source: docs/csharp/language-reference/compiler-messages/cs0234.md
Product: dotnet-csharp
Technology: csharp-diagnostics
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Hi @edg3
That typo is intentional. That's why the first code snippet produces the error. (the type System.DateTim doesn't exist).
I'll close this issue.

Ah, right, it didn't click for me that it was intended.


Battle damaged draws for weapons that are identically named

If you any combination of identically named weapons, and you set battle damaged to any of them, it draws the battle damaged icon for all the weapons.

Could make it so no weapon can be of the same name of another weapon? Would probably be the easiest way

Fixed with latest push


Fix for XSS bug

There is stored xss present in issue-name and issue description .
Apply this fix to filter input and prevent xss
https://huntr.dev/bounties/1625393956071-Volmarg/personal-management-system/
https://huntr.dev/bounties/1625394054448-Volmarg/personal-management-system/
If you want to see the report details then check your mail<EMAIL_ADDRESS>and you already have received the report magick  link

Hello,
Will try to take a look on it near weekend.

Accepted. You can send these to<EMAIL_ADDRESS>I will be using this email for this project related communication.

@zidingz - just (cc) you here.

Gotcha<EMAIL_ADDRESS>will receive an email within the hour. Thanks @Volmarg

Hi, can you plz update the report status to valid .
First-one already validated but second-one not validated yet .
Thanks


pencil: v2.0.10 -> v2.0.11

@prikhi does this package builds for you? travis has some problems building it https://travis-ci.org/NixOS/nixpkgs/builds/64539672#L4560

On Fri, May 29, 2015 at 01:58:37AM -0700, Rok Garbas wrote:

@prikhi does this package builds for you? travis has some problems
building it https://travis-ci.org/NixOS/nixpkgs/builds/64539672#L4560
Yup, both xulrunner 38.0.1 and pencil build fine for me...

rebased & pushed. Still working fine for me locally

builds for me locally as well ... merged here


I'm still seeing Bug #18026 exhibited on the InfluxDB Cloud 2.0 interface

This is identical to the bug
https://github.com/influxdata/influxdb/issues/18026
I won't describe it again.
Just to confirm it still exists on InfluxDB 2.0 Cloud interface.  I'm using both Chrome and Firefox.
I do note that a workaround is to refresh the editing page in the browser.
e.g. the one like https://xxx.aws.cloud2.influxdata.com/orgs/xxxxxx/dashboards/xxxxxx/cells/xxxxxx/edit
However this issue prevents the use of the editor for editing which is fundamental
Can we re-open the previous issue?
Or confirm when it's coming through on Cloud 2.0?

Steps to reproduce:
List the minimal actions needed to reproduce the behavior.

...
...
...

Expected behavior:
Describe what you expected to happen.
Actual behavior:
Describe What actually happened.
Environment info:

System info: Run uname -srm and copy the output here
InfluxDB version: Run influxd version and copy the output here
Other relevant environment details: Container runtime, disk info, etc

Config:
Copy any non-default config values here or attach the full config as a gist or file.

Logs:
Include snippet of errors in log.
Performance:
Generate profiles with the following commands for bugs related to performance, locking, out of memory (OOM), etc.
# Commands should be run when the bug is actively happening.
# Note: This command will run for at least 30 seconds.
curl -o profiles.tar.gz "http://localhost:8086/debug/pprof/all?cpu=true"
curl -o vars.txt "http://localhost:8086/debug/vars"
iostat -xd 1 30 > iostat.txt
# Attach the `profiles.tar.gz`, `vars.txt`, and `iostat.txt` output files.

@gefaila which OS are you using?

Windows 10

Just to confirm it still exists on InfluxDB 2.0 Cloud interface. I'm using both Chrome and Firefox.

@gefailia, are you still experiencing this issue?

Having the same issue in the latest docker image


There is no documentation in the readme to launch the server with posts with a future date

When you make a post and put a future publication date on it, you have to put the option --future at the server launch to see it appear.

I'm on it.


Reverse Java-11 specific workaround in gradle.properties

See if we can reverse this now ...
https://github.com/se-edu/addressbook-level3/commit/09837d98cc0443a75e7291e6aceb76764e18ca04

I have tested reverting the TLS workaround, and encountered no issues with GH Actions. both Java CI and Codecov are working as expected.
https://github.com/aureliony/AB3-J17/commit/549e670609793a8ef62b7a7cb6f0cfc11309c4ff

Thanks for checking @aureliony
Go ahead and send a PR for this.


AttributeError: module 'cv2.cv2' has no attribute 'TrackerMOSSE_create'

My cv2 version  is

Why can't I run the tracker?

The latest version of  OpenCV now has the "TrackerMOSSE_create" under the legacy class, instead of
tracker = cv2.TrackerMOSSE_create()
use
tracker = cv2.legacy.TrackerMOSSE_create()

The tracker is implemented in Contrib repository and was moved to legacy namespace at 2020-10-28. MauryaRitesh is right and you need to use new namespace to create object.

tracker = cv2.legacy.TrackerMOSSE_create()
This command is not still working for me.

Seems to be working fine:
pip install opencv-contrib-python
Python 3.11.1 (tags/v3.11.1:a7a450f, Dec  6 2022, 19:58:39) [MSC v.1934 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> import cv2
>>> cv2.legacy.TrackerMOSSE_create()
<cv2.legacy.TrackerMOSSE 0000022C41AEBCD0>

Make sure you have installed the correct package that contains the extra modules in addition to the main modules: opencv-contrib-python.

Before you install opencv-contrib-python, remove opencv-python with pip uninstall opencv-python.
See the README for more detailed installation instructions.

In the future, check out the comprehensive OpenCV documentation for more info on which APIs are included in the different packages: https://docs.opencv.org/4.6.0/index.html

Still its showing the same error :(

Make sure that you have the correct package installed:
import cv2
print(cv2.getBuildInformation())

Good sir thanks you share your knowledge

Thanks!This method successfully helps me solve the problem.

还是用不了

还是用不了

scipy  ==1.10.1
opencv-contrib-python   ==  <IP_ADDRESS>


Parser cannot handle enum variants whose name contains spaces, within larger structs

#[derive(parse_display::FromStr)]
enum Instruction {
    #[display("turn off")]
    TurnOff,
    #[display("toggle")]
    Toggle,
}

//succeeds
#[test]
fn parses_toggle() {
    "toggle".parse::<Instruction>().unwrap();
}

//succeeds
#[test]
fn parses_turn_off() {
    "turn off".parse::<Instruction>().unwrap();
}

#[derive(parse_display::FromStr)]
#[display("{instruction} {value}")]
struct ContainsInstruction {
    instruction: Instruction,
    value: u64,
}

//succeeds
#[test]
fn parses_contains_toggle() {
    "toggle 123".parse::<ContainsInstruction>().unwrap();
}

//fails
#[test]
fn parses_contains_turn_on() {
    "turn off 123".parse::<ContainsInstruction>().unwrap();
}

This is by design.
Since the format is specified as #[display("{instruction} {value}")], ContainsInstruction::parse() works as follows.

Separate the input string by the first space.
Pass the string before the space to Instruction::parse().
Pass the string after the space to u64::parse().

Because of this behavior, it will not work properly if the string of fields contains characters that separate the fields.
If there is only one field that may contain a space, it is possible to separate them by the last space by adding #[from_str(regex = ".*")] as follows.
#[derive(parse_display::FromStr)]
#[display("{instruction} {value}")]
struct ContainsInstruction {
    #[from_str(regex = ".*")]
    instruction: Instruction,
    value: u64,
}

For more complex cases, I think it is better to use a function that converts the beginning of the string to a value, such as a funtion that return IResult in nom crate, instead of a function that converts the entire string to a value, such as FromStr::parse.

Instead of specifying a field that may contain spaces, you can also specify a field that is guaranteed not to contain spaces.
#[derive(parse_display::FromStr)]
#[display("{instruction} {value}")]
struct ContainsInstruction {
    instruction: Instruction,
    #[from_str(regex = r"[^\s]+")]
    value: u64,
}

Thank you for the quick response. I'm not familiar with the internals of this library: would it be practical to re-implement ContainsInstruction::parse with the following algorithm?

defer to Instruction::parse

match the literal "turn off" or the literal toggle, returning the rest of the string via an internal API

match the space literal
defer to u64::parse
err if characters remain in the input string

In other words, construct a and use internally a parser which converts the beginning of a string and returns the unused parts, but present the whole-string parsing interface to the end user.
Obviously that's a pretty big feature request, but it would solve this issue. AFAICT it should be possible for regex-based sub-parsers to work properly under this design as well.
In the meantime, would you be willing to add documentation that this is a known limitation of the library?

Certainly, I think the above algorithm would be able to parse the string more accurately.
If I implement it, I would like to use a standard trait rather than the internal API to be able to use this algorithm even if struct contains fields that do not use parse-display. (e.g. std::u8)

a parser which converts the beginning of a string and returns the unused parts

Do you have any idea of existing standard trait that can be used for such purposes?

Adding the following features is also an option.

Adding #[from_str(regex = auto)] to instruction field will have the same effect as if you wrote #[from_str(regex = "turn off|toggle")].

Once Rust's specialization feature is stabilized, it may be able to omit #[from_str(regex = auto)].
This method has the following pros and cons.
Pros

Fewer modifications are needed
This feature can decide on char range to consume considering the backward string.

Cons

Use the same regular expression twice in the outer structure and the inner structure, and thus the performance may be worse.
Difficult to work with other parser combinators(e.g. nom), unlike https://github.com/frozenlib/parse-display/issues/9#issuecomment-740795611

Do you have any idea of existing standard trait that can be used for such purposes?

I don't know of any, unfortunately. AFAIK the general form would be something like
// returns `(parse result, unused portion of input)`
parse<'a, T: FromStr>(s: &'a str) -> (Result<T, <T as FromStr>::Err>, &'a str);

Adding #[from_str(regex = auto)] to instruction field

This is an interesting idea! Does this feature currently exist? I didn't see it documented.
It seems to me that it would be even more useful if this annotation could be applied to enum Instruction instead of the field instruction, so that we can decorate the declaration, not every usage. This feature, assuming it causes the test cases above to pass, plus appropriate mentions in the documentation, would be sufficient in my mind to close out this issue.

Adding #[from_str(regex = auto)] to instruction field

This is an interesting idea! Does this feature currently exist? I didn't see it documented.

This feature is not yet implemented.

It seems to me that it would be even more useful if this annotation could be applied to enum Instruction instead of the field instruction, so that we can decorate the declaration, not every usage.

I think so too, but I can't think of a way to allow omit #[from_str(regex = auto)].
This is the same for the algorithm described in https://github.com/frozenlib/parse-display/issues/9#issuecomment-740795611.
Since proc-macro cannot determine whether the field type implements the trait or not, you need to annotate whether use FromStr or the different trait described in https://github.com/frozenlib/parse-display/issues/9#issuecomment-741748867.


Uber Authentication failed "HTTP Status 401: Unauthorized, Response: {"error": "invalid_client"}"

i am using this library for Uber Authetication
https://developer.uber.com/v1/auth/
I have done like this
func doOAuthUber(){
    let oauthswift = OAuth2Swift(
        consumerKey:    "fXfXXXXXXXUo9vtKzobXXXXXUDO",
        consumerSecret: "e5XXXXXXXq2w63qz9szEx7uXXXXXXo03W",
        authorizeUrl:   "https://login.uber.com/oauth/authorize",
        accessTokenUrl: "https://login.uber.com/oauth/token",
        responseType:   "code"
    )

    var originalString = "jamesappv2://oauth/callback"
    var encodedCallBackUrl = originalString.stringByAddingPercentEncodingWithAllowedCharacters(.URLHostAllowedCharacterSet())

    println("encodedCallBackUrl: \(encodedCallBackUrl)")

    let state: String = ""
    oauthswift.authorizeWithCallbackURL( NSURL(string: encodedCallBackUrl!)!, scope: "request%20history", state: state, success: {
        credential, response in

        println(credential.oauth_token)
        self.personalDriverLoader.stopAnimating()

        }, failure: {(error:NSError!) -> Void in

            self.personalDriverLoader.stopAnimating()
            println(error.localizedDescription)
    })

}

but getting this response
HTTP Status 401: Unauthorized, Response: {"error": "invalid_client"}
I have triple checked that my client_id (consumerKey) and secret (consumerSecret)  are correct.
What I have done wrong here
Please help

Hi @alikazim I have the exact same issue. I think that encoding is somehow mixed and that the redirect_uri sent to Uber is not correct. I have found that the code which generates queryString is returning following value for redirect_uri:
redirect_uri=oauth-swift%3A%2F%2Foauth-callback%2Fuber

Which is encoded value and it does not correspond with registered value on Uber Developers Portal. I don't believe that this library is working with OAuth2 flow, or I haven't found solution in all these parameters to enhance encoding. If someone knows how to fix this, I'm here and listen.

Did you figure any of this out? I'm looking for how to authorize users in swift / ios. I can't figure out what the redirect_uri is supposed to be, I have my app name set in url types and identity and schemes, etc, but myapp:// isn't working and i'm not sure what comes after the // or what I tell uber to set my redirect uri to.

I'm to this point in the code. I have figured out how to get it to try and authorize my app using OAuth2Swift but now it's giving me the same error it's giving you: Invalid Request Parameters

@sephethus I contacted Uber support regarding this issue and pointed out this thread. I will try to resolve it with them.

Thanks I've posted this in a new, separate issue as well, here, since the description on this issue is for a different error. Mine has more details I think. https://github.com/dongri/OAuthSwift/issues/88

Any progress with this @kittrCZ

They're telling me the redirect is wrong. My app name is set in the plist URL schema and should use the format: MyAppName://whatever but they're saying it has to redirect to https:// which is not going to work with iOS. WTF?

I think that there is problem at Uber. This is what I got from their support:

Hi Tomas, Thank you for the report. Currently, the redirect_url only supports the https scheme. I opened a feature request for us to be more lenient on this. In the meantime, are you able to use an https URI? Thanks, Alec

For https, you can also provide to oauthswift an authentification handler with webview and (navigation) delegate. The delegate could call directly the handlexXX method of oauthswift2, no need to go out from app, to re-enter then, with the application url scheme

That sounds like yet more stuff I gotta learn how to do. Ugh. Have you tried the embedded webview? Does that work?

I found this tutorial but it's saying that there's some kind of webview property in OAuth2Swift but I don't see it and it's complaining that it isn't there: http://www.raywenderlich.com/99431/oauth-2-with-swift-tutorial#comments

@sephethus this tutorial is outdated
don't follow tutorial to learn, read and eat codes
(You loose time, I understand how work OAuthSwift in 30 minutes, just with debug mode)
If you don't want to read the framework codes, read the example, the demo provided at least...
see WebViewController file in demo, function webView....shouldStartLoadWithRequest.... to handle the url
and call  OAuth2Swift.handleOpenURL(url) in case of your url
webViewController become authorize_url_handler (after my own PR merged) because we could handle url with other way (webview and segue from storyboard)

I'm probably not as good as you at "reading and eating" code, but what in the world would I use as the redirect_uri when launching within a webview?

https://yourAppNameOrWebsiteORfakewebsite/callback
then check url host equal to yourAppNameOrWebsiteORfakewebsite

So that doesn't actually go anywhere but you can just capture what it's trying to call?

In the demos, the only ones not using URL schemas are pointing to an http heroku site, which I have no idea what that site is doing or handling or what is set up on that end. Even if I change my redirect_uri to https://whatever it still gets "Invalid Request Parameters".

Further examination shows that the uber site itself is redirecting me to this address once I log in and it gives me the Invalid Request Parameters error:
https://login.uber.com/oauth/authorize?client_id=**********hidden*****&redirect_uri=https:%2F%2Foauthswift.herokuapp.com%2Fcallback%2FUGetMe&response_type=code&scope=profile history

Another update, you know what? I still get invalid request parameters if I just paste the url into a browser with the required client_id and redirect_uri. Something is wrong on their end or this is not how it's supposed to be done. Maybe these parameters need to be in the header.

Experimenting further: just with the authorize url in a browser, outside of swift, I discovered that it is requiring the optional response_code and scope. Once I include that it works, but if I also include the redirect_uri, it fails again. It doesn't want the redirect_uri, it wants the response_code, scope, and client_id. Why doesn't it take the redirect_uri?
https://login.uber.com/oauth/authorize?client_id=fOjVtVqKhu-zUQclUfY2joB14VEAEN9V&response_type=code&scope=history

I think I have solved the problem with Uber, this will require modification of OAuth2Swift.swift, just comment out the line that says:
urlString += "&redirect_uri=\(callbackURL.absoluteString!)"

Therefore:
var urlString = String()
        urlString += self.authorize_url
        urlString += (self.authorize_url.has("?") ? "&" : "?") + "client_id=\(self.consumer_key)"
        //urlString += "&redirect_uri=\(callbackURL.absoluteString!)"
        urlString += "&response_type=\(self.response_type)"

        if (scope != "") {
          urlString += "&scope=\(scope)"
        }

        if (state != "") {
            urlString += "&state=\(state)"
        }

        for param in params {
            urlString += "&\(param.0)=\(param.1)"
        }

        if let q = urlString.stringByAddingPercentEscapesUsingEncoding(NSUTF8StringEncoding) {

            if let queryURL = NSURL(string: q) {
                self.authorize_url_handler.handle(queryURL)
            }
        }

redirect_uri is needed for access_token_url (last step), not really for authorize_url
uber is just too strict and don't want extra parameters
I don't know if other api need redirect_uri for access_token_url and why it is added to authorize_url

Now I'm getting the error this thread is about: HTTP Status 401: Unauthorized, Response: {"error": "invalid_client"}
My client_id and client_secret are correct.  I don't know where it's trying to post them though. I am trying to follow through the entire process of what happens during the postOAuthAccessTokenWithRequestTokenByCode method and I keep getting lost through it. I think the client_id or secret or both are getting dropped somewhere, not sure.

Here's the headers in terms of key/value:
Key: Authorization Value: OAuth oauth_consumer_key="myclientkeyisvalidhere", oauth_nonce="03A0CB93", oauth_signature="f2Lb2CGrS2TX4AE2Xe3U0JLx0X0%3D", oauth_signature_method="HMAC-SHA1", oauth_timestamp="1439168007", oauth_version="1.0"

Maybe it's here that things go wrong?

I finally figured the whole thing out, the solution is nothing to do with OAuthSwift. You have to have a privacy page url on a website somewhere, and you have to check off the scope you're going to use. This is all on the manage app page of the developer site.

I'm getting the { "error": Invalid_Client } again and this time it's not because of any setting on my Uber developer page. I'm requesting only "profile" in the scope. The key is getting all the way to the NSURLConnection request made in the OAuthSwiftHTTPRequest.swift file. It makes no sense why I'm getting this error now.

Hi @sephethus @phimage @kittrCZ @alikazim
I Resolved Uber OAuth problem
https://github.com/dongri/OAuthSwift/pull/96/files
Thanks!

Uber Wiki
https://github.com/dongri/OAuthSwift/wiki/Uber

@dongri  this is awesome! Thank you for your contribution. I will test it.

Object {message: "No authentication provided.", code: "unauthorized"}
code
:
"unauthorized"
message
:
"No authentication provided."
When i trying to hit the uber api i'll error mention above. Please help me out here.

@vishu3258 please open a new issue with details
I will help and comment in it

Getting error 401 while hitting the  uber api in AngularJS. Please help me out here.
Object {message: "No authentication provided.", code: "unauthorized"}
code
:
"unauthorized"
message
:
"No authentication provided."
When i trying to hit the uber api i'll error mention above. Please help me out here.


Screenshots dont match the actual experience on the portal

The screenshots that show the Availability zones listed on the portal by instance size doesnt show up in reality.

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

ID: 9b71318f-48c9-4491-f393-8a69588599a3
Version Independent ID: cf870587-ac9b-327e-9f13-e8f765d907c5
Content: Create a zoned Windows VM with the Azure portal - Azure Windows Virtual Machines
Content Source: articles/virtual-machines/windows/create-portal-availability-zone.md
Service: virtual-machines-windows
GitHub Login: @cynthn
Microsoft Alias: cynthn

#reassign @mimckitt Looks like the screenshots are old? Assign it back to me if you want me to do the refresh :)


Use the saved model for inference on a new set of terms

Hi,
i builded everything from scratch for Italian language, ran the train_RL.py using the italian version of SemEval-2016 datasets for training, validation and test and saved the model. I can't figure out how to use the saved model for taxonomy induction on my own vocabulary. Thanks.

Can you provide more details? What are the errors that you are seeing?

Thank you for the quick reply! There are no errors, i managed to create and save the model using the italian version of your dataset. I just don't know how to integrate my own set of terms(different from the terms contained in the dataset) in the process. So my question is: once i have the model, how can i pass to the model a set of terms in input for taxonomy induction out of these terms?

I am a bit confused. If you managed to train a model using the Italian version of the dataset, can you not replace its test set with your new set of terms? (I assumed that you went through the whole pipeline of feature generation)

Yes, you are right, i went through the whole pipeline of feature generation. I will follow your advice and replace the test set with my set of terms. Thanks.


Nodes-Talking Admin Does not Allow login

With a fresh install of the Nodes-Admin server it is not possible to login.    Need to add info to the Mongo database that will be part of the install and provide a default user.

@rsiewert and I identified that a while back, I believe he had a solution for it, by ensuring the exchange on publish each time.


Potential performance issue when enqueuing a large amount of operations

There seems to be a bottleneck when adding a large amount of operations.
For example, running the following experiment on my machine, enqueuing about 1000 operations is just fine. However, if I increase the operations to a higher number to like 30_000, the code seems to wait for all the pendingTasks to be accumulated instead of trying to also run the already enqueued tasks concurrently? Also observe that the memory goes way up (to the amount of GBs) and seemingly takes a long time for the pending tasks to even start running.
Removing the @MainActor from the experiment function seems to help a tiny bit, but it's still significantly slower than some other serial queuing implementations that makes use of AsyncStream.
import Foundation

@main
struct Test {
    actor Counter {
        var count: Int = 0
        func increment() -> Int {
            count += 1
            print("Incremented: \(count)")
            return count
        }
    }

    @MainActor
    func experiment() {
        let queue = AsyncQueue()
        let s = Counter()
        for i in 1 ... 100_000 {
            queue.addOperation { [i] in
                let result = await s.increment()
                if i != result {
                    print(i, "does not match", result)
                }
            }
        }
    }

    static func main () async {
        print ("test starting")
        Test().experiment()
        print ("test started")
        try! await Task.sleep(for: .seconds(5))
        print ("test finished")
    }
}

I think that 818219f458ff51ebf7651f1d1617fa4e3694956c addresses this. The performance is now much better and peak memory usage seems to have a much more reasonable ceiling.
Still every interested in an AsyncSequence-based solution though!

Thanks for looking into it! I verified that it's indeed way faster with 818219f
The other solution that's based on AsyncStream can be found here. That solution doesn't use any lock and simply relies on the stream/continuation for thread-safe buffering. I think the enqueueAndWait method can be used as a barrier variant.
Some relevant discussion can be found in the Swift forum.

Great! Thanks again for reporting this, I really appreciate it. And also for the pointer to that other library. It looks really interesting - I'm going to put a link to it in the readme just in case.


Custom module development

Hi!
I need to implement Change Password functionality and I thought develop it as a module for authboss is good idea. But I need response class available  for it. Unfortunately this class is internal within authboss. What is the best way for my task? Did you think about user custom modules for authboss?

I don't think there's support for custom modules. The only extension points are the storer interfaces and events.
There is an UpdatePassword function in authboss already. I did need to implement my own change password because I wanted the user to verify the current password first, and also to check minimum complexity on the new password. However, I just did it as my own controller action, not part of authboss.

Hey @shade33. Extra modules are supported. Moreso in version 2. Though as @jack-chung mentioned, this is functionality that exists today. What kind of functionality were you looking at providing?

Hi @aarondl, first of all thanks for authboss. 👍
I was wondering how to get an IP Whitelist with authboss.
How is the more proper way to achieve this feature? Can I write a custom module or should I work with callbacks only?

@thiagocaiubi Typically you should send this kind of request to the mailing list. Are you talking about an IP Whitelist for user's authentication? If so you should create a module of your own, add storage requirements to have the IP Whitelist table/array available somehow, and then do a hook BeforeAuth I think it's called.

Going to close this issue as it's mostly questions.


Fix get_8 returning 0 for last byte of compressed files

Fix #21991.  FileAccessCompressed::get_8 was returning 0 on the last byte of the file.

Thanks!

Cherry-picked for 3.0.7.


Fix Sphinx Design Grids in Latex

Sphinx design grids look very nice in HTML but translate very poorly to Latex. Example shown below. In its current state it is not readable by a user.
HTML

Latex

Uday to create an issue with Sphinx Grids


fix(sentry): Tackle Sentry bugs

fix(classify): add a check for when stories is empty. In response to ValueError Found array with 0 sample(s) (shape=(0, 645)) while a minimum of 1 is required

From the comment on the method I'd expect this clause to return [None, None, []] (note the last item as an empty List). Am I right or does returning model_scores=None make more sense based on where it is called?

No, I think you're right, it doesn't make any more sense returning None. Following the classification task though, there might be a need to also add a check in _add_confidence_to_stories or maybe we can handle the no stories case directly in the classification task, classify_and_post_worker?


introduction text reset

"...start typing whenever you are ready" text should be without opacity when user restarts test. Same for bottom instructions.

Added lines of code in restartTest function:
     document.getElementById("test-title").style.opacity = 0.8;
    document.getElementById("instruction-container").style.opacity = 0.8;


[Fabric 1.20.1] Mythic Metals breaks merling origin swim speed due to shipping newer Additional Entity Attributes

Summary
When using the latest version of Mythic Metals (0.19.8) with Origins, the merling origin's increased swim speed does not work. Issue is not present in earlier versions.
Version Details
Minecraft 1.20.1
Fabric 0.15.11
Mythic Metals 0.19.8
Origins 1.10.0
Expected Behavior
When using merling origin, underwater movement speed will be increased to 4.9 m/s while walking and 9.8 m/s while sprinting.
Observed Behavior
Merling underwater movement speed is unchanged from default (1.96 m/s walking, 3.92 m/s sprinting).

This is not directly caused by Mythic Metals. It is caused by the bundled version of Additional Entity Attributes. Currently I ship 1.7.4, while Origins is still built against 1.3.0.
I don't think there is a ton I could do to fix this, as the best fix would be for Origins themselves to update that dependency.

Ah that makes sense. Thanks!


Having a ember-cli-sass installed without boostrap-sass breaks build.

There is no way to not use bootstrapp-sass (or bootstrap-less (via importBootstrapTheme and importBootstrapCSS) if you have a supported preprocessor installed.
Therefore, since I have ember-cli-sass installed (for other things), but am including the boostrap css elsewhere, I get an error when trying to build (index.js:91):

Npm package "bootstrap-sass" is missing, but required for SASS support. Please run ember generate ember-bootstrap to install the missing dependencies!

I tried running ember generate ember-bootstrap --preprocessor=none but all that accomplished was it uninstalling my ember-cli-sass, which wasn't precisely the solution I was looking for.
(For now, I'm working around this by installing ember-boostrap but removing the import from app.scss)

Output from ember bootstrap:info
Npm packages:
ember-bootstrap: ^1.0.0-alpha.12 -> 1.0.0-alpha.12
ember-cli: 2.12.1 -> 2.12.1
bootstrap: ^3.3.7 -> 3.3.7
bootstrap-sass: n/a
ember-cli-sass: ^5.6.0 -> 5.6.0
ember-cli-less: n/a

Bower packages:
bootstrap: n/a
bootstrap-sass: n/a

ember-bootstrap configuration:
importBootstrapTheme: false
importBootstrapCSS: false
bootstrapVersion: 3
importBootstrapFont: true

It sounds like we should weaken the message to a warning or at least not make it a failing condition.

@srvance Warning probably makes sense!


Add 21.08 spack.yaml

@shahzebsiddiqui

@eugeneswalker you define the compiler stanza for<EMAIL_ADDRESS>I think its fine to tailor the e4s release that is site specific with details for UO system. It helps new users that want to pick up this spack.yaml to have the complete info regarding the spack configuration.

@eugeneswalker you define the compiler stanza for<EMAIL_ADDRESS>I think its fine to tailor the e4s release that is site specific with details for UO system.

Are you saying to embed the full compiler definition for gcc 9.3.0 in this spack.yaml?

Thanks @eugeneswalker i see the compiler stanza this looks good.

I am working on adding the spack find and spack arch output now.


config.bat fails on appveyor for Caribu

config .bat seems to fail when run from openalinea-incubator (see eg https://ci.appveyor.com/project/pradal/caribu)

oups sorry wrong place to post !


how does cubemap projection work ?

How does cubemap projection work ?
Is there a function in the source code that converts 6 cubemap images to 1 equirectangular image ?
Because the cubemap projection seems to be very fluid, i mean there is no border.
Actually, i'm looking for a way to convet 6 cubemap images to 1 equirectangular image in JS.

Pannellum is a viewer, not image processing software. I you want to learn about cube maps, you can read Wikipedia.


Polyon rendering glitch

I noticed in the demo instance that @ebrelsford put up at https://mltdemo.stamen.com that there is some glitch art in Antarctic:

@ebrelsford how recently where the MLTs used in the demo encoded? Can you make sure to re-encode them with a java encoder version that includes https://github.com/maplibre/maplibre-tile-spec/pull/193? Is the glitch still present?

I encoded these this morning so they should include the changes from #193 but I can try again.

On main I rebuilt the cli (./gradlew cli) and am running encode.jar on the input MVTs, seeing the same glitches.

Okay, thanks for checking.


Consider publishing on luarocks

https://github.com/nvim-neorocks/sample-luarocks-plugin

Tried this: https://luarocks.org/modules/andrewferrier/debugprint.nvim. Didn't really work because the trigger doesn't seem to be compatible with use of release-please, but also it doesn't seem to be being adopted, so not a major issue, abandoning.


DOM-diff

Someone https://github.com/marcelklehr quoted DOM-diff that is no longer within your repos (am I wrong?). I am trying to build a support for history  in Geogebra  (to give some evidence whenever you use GGB in activities with auto grading in Moodle). GGB stores its internal status in a large XML and snapshotting it at every student's step  to get some evidence, for all the students in all classes for the whole school year, will sink Moodle MySQL DB. At every step few things changes so storing DOM diffs will be a game changer. When some evidence/explanation  is required for a bad note teacher will show the  history  created applying deltas and showing the student's process. Surprisingly Stackoverflow do not contain any hint for diff-ing XMLs. Obviously I can utter a  bunch of recursive js lines that can do the job but I do not like to reinvent the wheel. To fit in GGB Javascript it must be a single file js. Can you help?

I think that you mistook me for someone else. I would like to help you, but I never did anything similar.


Fix wrong listnav links under ems_network and tests

Fix wrong listnav links under ems_network and tests. Also testing
link for summary in every other NetworkManager listnav.

@martinpovolny fixes the bug you found


Oops, an unexpected error occurred! unreachable

Hi Prisma Team! The following command just crashed.
Command
migrate dev --schema [redacted]
Versions

Name
Version

Platform
windows

Node
v18.16.1

Prisma CLI
4.10.0

Engine
ca7fcef713137fa11029d519a9780db130cca91d

Error
Error: unreachable

Duplicate of https://github.com/prisma/prisma/issues/20668


Make TransactionContext generic

This does 2 things:

Make the TransactionContext not be tied to ZeroMQ, which is useful for testing the TransactionHandler without using the network.
Make the Context trait use &self instead of &mut self methods

Closing in favor of #12


Feat/map from

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (caf6043) 100.00% compared to head (a6b823b) 100.00%.

Additional details and impacted files
@@             Coverage Diff             @@
##                main       #58   +/-   ##
===========================================
  Coverage     100.00%   100.00%
- Complexity        53        62    +9
===========================================
  Files              6         7    +1
  Lines            121       146   +25
===========================================
+ Hits             121       146   +25

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


Add getEnvironment helper

Often useful to know which environment (e.g. local, development, staging, production) the site has been generated in so that templates may switch things in response.

This has been implemented by #249


Contributing

If we agree on it, we could either completely rely on pandas or give some rules, where we use our own conventions (in the wiki or in the docs).

Most of the stuff I wrote was inspired by pandas (maybe I should have made that more clear), but I felt that we should have something more than just "go look at pandas" since new contributors might not yet understand the scientific python ecosystem. I'm open to changing this, though.

Ok, I think this is a good approach. However we could provide the link to pandas, if we agree on all of the thinks that are stated there.

I added a handful of links to the pandas pages. Feel free to edit anything.

Ok, thanks. I'll close this issue.

The new 0.16.1 pandas release has a new Contributing to pandas page. It's really good. I still think we should have a small wiki, but I'll add and emphasize links to this.

I renamed the Development information wiki to Contributing to pvlib python and rewrote some of the text. I hope we can get some other people to add-to/clarify the wiki in the future, so I'm going to leave this open for 0.3. I think @jforbess's perspective would be helpful here. I found it hard to strike a balance between high-level overview with links as opposed to detailed instructions.


feat: SC-8122 new bx columns icon

new bx columns icon

for posterity:

for posterity:

What do you mean by that? Is that color change something internal frontend do?

I just wanted to see what the new icon was without having to run Storybook. Now it's immortalized in this PR. Grabbed the image from your ticket and pasted it in here.


[BugFix] Fix all Role is MASTER when showing frontends

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #8121
Problem Summary(Required) ：

Check the Role type according to the fe's host and master's host.

[FE PR Coverage check]
:disappointed: fail : 0 / 5 (00.00%)
file detail

path
covered line
new line
coverage

:large_blue_circle:
com/starrocks/common/proc/FrontendsProcNode.java
0
5
00.00%

We need to change mater to leader

We need to change mater to leader

I will change the related code in another PR


Set serverless-http and @aws-sdk/client-s3 as peerDependencies.

These two packages are not specifically required to run open-next.
However, they are required in order for the lambda functions to be bundled correctly. Therefore, it makes sense to require them to be installed as dependencies of the Next.js application we are trying to bundle and not as dependencies of open-next itself.
This enables open-next to be used with stricter package managers such as pnpm.

Hey @Shiroy, I make some progress on this over the holidays. I didn't want to have the users install packages "OpenNext" uses. I will share what I have later today, just have to iron out some kinks.

My point is that package managers such as pnpm won't install open next dependencies as the application dependencies.
This prevents the lambda bundler from finding the package required by your lambda functions.
You don't have this issue with npm as everything is merged together.
Let me know if I can help you.
Best,

Got makes sense!
OpenNext used to move the adapters to user's app .open-next/.build and perform the build there, which break pnpm b/c the two dependencies won't be found as u describe.
I just published v0.4.2. It will do the build inside the OpenNext folder, and write to .open-next/.build. This way, it will find the dependencies.
Can you see if v0.4.2 works for ur setup?
^ Here's the pr #20

@fwang it works on my setup !
I haven't tried the result on Lambda yet, but now functions are bundled ant built correctly.


Use popToTop to clean up route stack

There's no guarantee that the route just rendered by _onRouteDidFocus is still on the top of the stack by the time the transition is complete, so the current implementation will chop off any routes that have been pushed on in the interim without them rendering.  This is particularly a problem after cold boot when the navigator takes a while to warm up and Redux might be rerouting several times in the interim as the store is initialized.
This PR addresses this by using popToTop, which resets the stack to be only the top-most route, whether that's the route that's just finished rendering or not.

Nice!


Removed Subfolders in the redhat/overlays directory

Removed the subfolders in the redhat/overlays directories.

/cherry-pick 1.5.2

/cherry-pick midstream-v1.5.2


Upgrading to Gatsby 3

This PR upgrades our site to Gatsby 3. Notable changes:

replaced gatsby-remark-sourcename with some code in gatsby-node.js
switched from deprecated gatsby-image to gatsby-plugin-image. This introduced a number of changes in queries and components. N.B. I removed breakpoints from image queries as they didn't seem to work as expected even though gatsby-plugin-image supports them. I didn't notice any difference in the result, but @kirstenkeister may want to double check once this PR is merged.

Possible blocker:
On build, this error pops up:
(node:32703) [DEP0066] DeprecationWarning: OutgoingMessage.prototype._headers is deprecated

I think it's happening in a dependency, but I can't trace which one. Ultimately it doesn't affect the build so I figure we can monitor it and try to address it after merge, but I welcome thoughts/feedback on this

@raffazizzi The breakpoints do actually something: they limit the number of images that are generated to only those that are indicated (and necessary), which in turn means it doesn't take as long to build the site. I'd prefer to have the breakpoints in there unless they don't work the same with the upgrade to gatsby 3?

Addressed @trevormunoz comment and integrated the latest changes from @kirstenkeister that just came in. For breakpoints, let's complete the switch and re-assess what we can do with the new API going forward

I think part of @trevormunoz's motivation for the the gatsby3 work on umd-mith/airwaves was to get some build speedups. Maybe we'll see some here?


Hybrid Plugin System

What
Add a plugin system that combines multiple plugin systemss and dispatches based on advertised capability, or priority order if multiple factories advertise a capability.
ACs

Uses a shared identifier to locate the conceptual same manager across each plugin system.
Make sure this is generic so it composes the output of N plugin systems.
Uses hasCapability to route.
Picks the first if present in both.

Implementation notes

Ideally have a internally RoutingManagerInterface(vector<ManagerInterfacePtr> interfaces) as this allows eg: Cpp, GPRC Python, in-process Python.

https://lucascheller.github.io/VFX-UsdAssetResolver/resolvers/production.html
https://forum.aousd.org/t/batching-ar2-queries/818/2?u=foundrytom

One potentially controversial change of behaviour worth calling out, required to reduce the amount of work to implement the hybrid plugin system, is to no longer require managers to reject (with an exception) unrecognized settings passed to initialialize - see


Update text-fields.md

[x] I have followed (at least) the PR section of the contributing guide.

@jeffshek On a side note, I have noticed that you are using ag-grid in your application:

Are you using the MIT or enterprise package?

@jeffshek On a side note, I have noticed that you are using ag-grid in your application:

Are you using the MIT or enterprise package?

Enterprise, I highly recommend it!

@jeffshek We are working on a similar advanced data grid component under https://github.com/mui-org/material-ui-x with @dtassone. If you would be interested in beta-testing it out, you can send us an email at<EMAIL_ADDRESS>so we can share more details.


feat: implement debug mode

Self-explanatory. See the proposal https://github.com/deepset-ai/haystack/blob/main/proposals/text/4284-drop-basecomponent.md#debug-mode

Partially fixed by https://github.com/deepset-ai/canals/pull/11


feat(Hangzhou): Add new micheline primitives introduced with Hangzhou proposal

Adds new micheline primitives introduced with Hangzhou proposal

Thank you for the contribution


[e2e]: generic schedule main multi-uses slsa3

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3005290650
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Trigger: schedule
Branch: main
Date: Wed Sep  7 05:47:22 UTC 2022

Repo: https://github.com/slsa-framework/example-package/tree/main
Run: https://github.com/slsa-framework/example-package/actions/runs/3012662980
Workflow file: https://github.com/slsa-framework/example-package/tree/main/.github/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Workflow runs: https://github.com/slsa-framework/example-package/actions/workflows/e2e.generic.schedule.main.multi-uses.slsa3.yml
Trigger: schedule
Branch: main
Date: Thu Sep  8 05:31:43 UTC 2022
Tests are passing now. Closing this issue.


Panic on rim system without any additional args

$ rim system
thread 'main' panicked at 'Usage: rim system [SUBCOMMAND], see help', src/main.rs:56:14
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

I was expecting it to actually print out the help like what happens when I just type in rim

Yeah, I agree. IDK if clap, the tool that generates the UI can do that, but I'll check.


[ISSUE #242]  New consumer may consume all message

What is the purpose of the change
assume one MQ Broker cluster,the broker commitLog min offset is zero , a new consumer start ,
the consumer will starting consume queue's offset zero messages，it's a problem。
Brief changelog
org.apache.rocketmq.broker.processor.ConsumerManageProcessor#queryConsumerOffset
Verifying this change
org.apache.rocketmq.broker.processor.ConsumerManageProcessor#queryConsumerOffset
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
[x] Run mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test  to make sure integration-test pass.
[x] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Hi, we must ensure the behavior that if we expand the queue numbers. That means the consumer has already connected the cluster and but the expanded queue is new and doesn't have the offset record.

@fuyou001 Can you describe the scenario that is problematic? What is unexpected here?


Download PDF is not genrating content

When I try to download the PDF on https://docs.microsoft.com/en-us/dynamics365/customer-engagement/, the generated document is only listing the table of contents.

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

ID: 370dc3b2-f205-da4f-99d0-497be7ec9e18
Version Independent ID: 17bfd2a7-b06b-2f46-dc1a-9b00fa08a714
Content: Dynamics 365 for Customer Engagement
Content Source: ce/index.md
Service: dynamics-365-crossapp
GitHub Login: @JimDaly
Microsoft Alias: jdaly

@interfacemirror Thanks for creating this issue; we are investigating this and will share updates soon.

@interfacemirror This has been fixed; please try downloading the docs PDF again.
Closing this issue now.


Updated Proofpoint State of the Phish 2024 and Readme

Updated Proofpoint State of the Phish 2024 and Readme.

Hash matches and format matches
https://www.hybrid-analysis.com/sample/242f0608765da5a9d254f88f489ee8607cb5826964aae2f30b01e40b7184f671


Zuul - SimpleHostRoutingFilter forwards inbound request paths as unescaped

This issue exhibits itself much like #347, but the fix in bd5d00c addressed it only in RibbonRoutingFilter.
Example of issue:
inbound request path: /thirst%20mutilator
outbound request path: /thirst mutilator
The cause is similar, the routing filter will use the "requestURI" that the PreDecoratingFilter (or really any other filter) puts into the RequestContext. PreDecoratingFilter happens to put an unescaped path into requestURI.
This problem really exists because the PreDecoratingFilter unescapes the uri before putting it into the context. Is having an unescaped path in context.requestURI a requirement?
I have forked locally to fix this in the SimpleHostRoutingFilter, definitely wouldn't mind submitting a PR.

The reason this was not caught is that there was a single unit test which only ran requests through the ribbon path.

PR more than welcome thanks.


[Bug] Decorations visible in non-jumpy mode

Jumpy: 0.2.1
VSCode: 1.6.0
Replicate

Create two editors in single window
Activate Jumpy
Switch to the other editor
Press enter or any decoration shortcut
The decorations are showing outside of Jumpy mode

Thanks I'll look into this along with the jumping across editors feature request.


[enhancement]: .heic image file support

Is there an existing issue for this?

[X] I have searched the existing issues

Contact Details
No response
What should this feature add?
I'd like Invoke to natively open .heic files.
While most images coming from elsewhere are jpg or png files, most of my own photos are from iOS and are thus, .heic files.
Alternatives
Rather than requiring an additional step of converting any photos I want to use, I'd rather Invoke open .heic directly.
Additional Content
No response

The image-file library we use has been reluctant to add HEIF support due to the lack of a clear patent grant, but it looks like we should be able to find a suitable extension: https://github.com/python-pillow/Pillow/issues/2806#issuecomment-370985911
Some highlights from that six-year-old thread:

libheif is in Debian with an LGPL license. (LGPL doesn't include a patent grant, but if Debian's decided it's okay to include, that's a promising sign that it's safe to rely on.)
https://github.com/bigcat88/pillow_heif offers a Python interface to libheif and includes a Pillow extension. 👍

they also publish pi-heif "a light version of Pillow-Heif with more permissive license for binary wheels." -- but I'm having some trouble finding what exactly the difference in license and binaries is.

they also publish pi-heif "a light version of Pillow-Heif with more permissive license for binary wheels." -- but I'm having some trouble finding what exactly the difference in license and binaries is.

It does not include x265 library which is GPL 2.0.
But this can be changed this year, as currently work on libheif's plugin system is almost finished: https://github.com/bigcat88/pillow_heif/issues/154
and x265 is a good candidate to leave binary wheels and be distributed on its own as a separate plugin, when this https://github.com/strukturag/libheif/issues/894 will be finished and the additional libheif plugins for QuickSync and AMD arrive.


layout validation: vec3 float should be able to fit in 16 bytes

The layout validation is insisting that nothing can go in to offset 12-15 after a vec3.  That's wrong.
Example GLSL vertex shader:
#version 450
layout (set=0,binding=0) uniform S { vec3 v; float f;} B;
void main() { }

Produces
               OpCapability Shader
          %1 = OpExtInstImport "GLSL.std.450"
               OpMemoryModel Logical GLSL450
               OpEntryPoint Vertex %main "main"
               OpSource GLSL 450
               OpName %main "main"
               OpName %S "S"
               OpMemberName %S 0 "v"
               OpMemberName %S 1 "f"
               OpName %B "B"
               OpMemberDecorate %S 0 Offset 0
               OpMemberDecorate %S 1 Offset 12
               OpDecorate %S Block
               OpDecorate %B DescriptorSet 0
               OpDecorate %B Binding 0
       %void = OpTypeVoid
          %3 = OpTypeFunction %void
      %float = OpTypeFloat 32
    %v3float = OpTypeVector %float 3
          %S = OpTypeStruct %v3float %float
%_ptr_Uniform_S = OpTypePointer Uniform %S
          %B = OpVariable %_ptr_Uniform_S Uniform
       %main = OpFunction %void None %3
          %5 = OpLabel
               OpReturn
               OpFunctionEnd

Validation says:
error: line 26: Structure id 8 decorated as Block for variable in Uniform storage class must follow standard uniform buffer layout rules: member 1 at offset 12 overlaps previous member ending at offset 15

I may have messed this up with my reworking of the padding calculation.  In any case we didn't have tests to cover this.
This affects Vulkan CTS.

Cc: @alegal-arm   Another one.  :-(


Add leichi algorithm to HEU

[Feature] Add algorithm(leichi_paillier) which uses leichi to speed up Paillier

I have read the CLA Document and I hereby sign the CLA


mavproxy: 1.8.27 -> 1.8.29

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for mavproxy is: "MAVLink proxy and command line ground station"
meta.homepage for mavproxy is: "https://github.com/ArduPilot/MAVProxy"
meta.changelog for mavproxy is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

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

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

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

found 1.8.29 with grep in /nix/store/c6g5yfkxlp9858iy25ila377zyvvh9d0-MAVProxy-1.8.29

directory tree listing: https://gist.github.com/f82549b7705e909c0ed89304d7195bb3

du listing: https://gist.github.com/931a8e55b6277db64116249fae9ac842

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

4 total rebuild path(s)

1 package rebuild(s)

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

First fifty rebuilds by attrpath
mavproxy

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/c6g5yfkxlp9858iy25ila377zyvvh9d0-MAVProxy-1.8.29 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/c6g5yfkxlp9858iy25ila377zyvvh9d0-MAVProxy-1.8.29
ls -la /nix/store/c6g5yfkxlp9858iy25ila377zyvvh9d0-MAVProxy-1.8.29/bin

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

1 package built:

mavproxy

Maintainer pings
cc @lopsided98 for testing.

Result of nixpkgs-review pr 105795 run on x86_64-darwin 1

1 package marked as broken and skipped:

mavproxy


Log warnings despite strictSchema: false

https://ajv.js.org/options.html#strict-mode-options says strictSchema: 'log' displays log warnings and strictSchema: false does not. But even though I am using strictSchema: false, I still get log warnings.

What exactly is the warning? strict: false should suppress all of them, strictSchema suppressed a particular subset - the details are in the doc on strict mode: https://ajv.js.org/strict-mode.html

Does that mean I have to disable something else?

You need to define them as noop one by one, there is no way to indiscriminately ignore all unknown formats - it would make you miss typos.

It’s an unrelated issue btw, it’s all covered in the docs.

I studied the docs but I couldn't find a way to disable this behavior. I can only turn off all format checking but that's not what I want. I need logging and format checking but I want to ignore all unknown formats. Unfortunately, I can't whitelist because I dont't know what formats people will submit.

This is currently not supported. The premise here is that a hard failure is better than undefined behaviour - even for user defined schemas, where the users would probably expect formats do something rather than be quietly ignored.

Unfortunately, I can't whitelist because I dont't know what formats people will submit.

it’s not that difficult actually - you can iterate the schema, e.g. with json-schema-traverse that ajv uses internally, and extract all used formats from the schema


chore: add dev dependencies label

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

v1.13.2
GitHub release

Your semantic-release bot :package::rocket:


Visualization of Receptor Fingerprint

The statistics must be added to the y-axis when displaying the receptor graph.
Receptor density (fmol/mg protein +- SD) or STD (this is written in the tabular).

transfering to siibra-python @AhmetNSimsek


dev server logging prevents console.debug calls from showing in browser

Environment
[12:28:21 PM] Nuxt project info:

Operating System: Linux
Node Version:     v18.16.0
Nuxt Version:     3.11.1
CLI Version:      3.11.0
Nitro Version:    2.9.4
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      typescript, modules, runtimeConfig, alias, vite, app, routeRules, nitro, build, hooks, prometheus
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>~/lib/core/nuxt-modules/apollo/module.ts<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

[12:28:21 PM] 👉 Report an issue: https://github.com/nuxt/nuxt/issues/new
👉 Suggest an improvement: https://github.com/nuxt/nuxt/discussions/new
👉 Read documentation: https://nuxt.com
Reproduction
Issue is obvious from code
Describe the bug
The new server log streaming to client feature (https://github.com/nuxt/nuxt/pull/25936/files) has essentially made it so that all console.* methods are proxied through Consola on the frontend. But there's no way to configure the Consola instance meaning its using its defaults, including the default log level.
What this means in practice is that all console.debug calls are essentially skipped, because the debug log level is too low for the default log level.
Additional context
Introduced by this feature in nuxt 3.11: https://github.com/nuxt/nuxt/pull/25936/files
Logs
No response

Btw, the workaround for now is to just disable this feature entirely w/ devLogs: false in the nuxt config

This is fixed in v3.11.1 - just upgrade 🙏

@danielroe I am on v3.11.1 already 🤔

Oops - my bad!

This should only be true during the initial hydration, right?

@danielroe Repro: https://github.com/fabis94/nuxi-consola-breakage-repro
See the console.debug invocation in app.vue

This has been resolved in https://github.com/nuxt/nuxt/pull/27309 in the nightly release, or in the next release.
Let me know if not and I'll reopen.


[HEVCe] Update IR stripes width calculation

Implemented balancing of IR stripes width (they can have now different
width based on caps.IntraRefreshBlockUnitSize) to meet target CycleSize
more accurately. Stripes remainder is in the tail frames.

@vilichev , what about new HEVCe design? Is there change already there? Please confirm and post a link to code.

@dmitryermilov corresponding changes are already here. https://github.com/Intel-Media-SDK/MediaSDK/blob/master/_studio/mfx_lib/encode_hw/hevc/agnostic/g9/hevcehw_g9_legacy.cpp#L2009


context-aware/flexible few-shot example rendering

Sometimes you want to inject as many few-shot examples that fit into model context. This idea PR calculates token constraints and renders the maximum number of examples to fit under the model limit.

@craterink I had a similar idea a few days ago and sketched out a stub ticket here: https://github.com/piercefreeman/gpt-json/issues/23. In some ways this case is a subclass of that case: calculating allowable length of the message and then truncating the example payload to the amount of the examples that can fit fully.
The one additional component here might be allowing users to truncate a list of objects (like examples) to only include full list values. Want to append that ticket with this use-case?

@piercefreeman second this suggestion. Thanks @craterink keep me in the loop

closing bc #28 is simpler version of this


Add support for writing episode gfx-replay files during eval

Motivation and Context
Similar to how we generate a video per episode, we also now generate a gfx-replay JSON file per episode. Coming soon, these files can be imported into Blender for high-quality visualization (see https://github.com/facebookresearch/habitat-sim/pull/1759).
This feature requires some Hab-sim gfx-replay fixes, which are here: https://github.com/facebookresearch/habitat-sim/pull/1758
How Has This Been Tested
Local testing with Hab 2.0 rearrange eval.
Types of changes

New feature (non-breaking change which adds functionality)

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have completed my CLA (see CONTRIBUTING)
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

All of the functionality in this PR is incorporated into: https://github.com/facebookresearch/habitat-lab/pull/883 (with the refactors Vince mentioned). Can we close this PR?

Closing this because the functionality is included in another PR, #883. See Andrews' comment above.


Update generated patch

Regenerate patches/001.

Not sure why there's so much noise in the new generated patch, with extra whitespace and everything but this is what I get after running scripts/full-initialize-repo.sh.

git -c diff.suppressBlankEmpty=true diff might help?


[ACS] az acs maintenanceconfiguration: Add Operations for Maintenance Configuration

Related command
az aks maintenanceconfiguration
Description

This PR will add in the following commands to the ACS command modules:

az aks maintenanceconfiguration show
aks maintenanceconfiguration delete
aks maintenanceconfiguration list
aks maintenanceconfiguration add
aks maintenanceconfiguration update

The above commands will be used for Planned Maintenance of AKS clusters.
Testing Guide

History Notes

[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change
[Component Name 2] az command b: Add some customer-facing feature

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

[x] I adhere to the Error Handling Guidelines.

draft

Latest live test failed.

E           azure.core.exceptions.HttpResponseError: (UnmarshalError) UnmarshalEntity encountered error: unexpected end of JSON input.
E           Code: UnmarshalError
E           Message: UnmarshalEntity encountered error: unexpected end of JSON input.

Checked log, not sending any content in the request body.

Please refer to the guideline to write the description of those PR changes in the History Notes section of PR description. Please note that this content will be synced to the release notes of CLI. Such as #26261

Please note that we are launching the release for this sprint this week. Please resolve all comments by tomorrow, otherwise the release of this PR will have to be postponed to the next sprint (on 09-05)

Queued live test pipeline, test passed.


Update test browsers

Use latest available chrome & firefox, add android 5.1

Thanks for updating! Seems like we can specify latest, latest-1, and beta. Can you use this style?
https://wiki.saucelabs.com/display/DOCS/Test+Configuration+Options
https://saucelabs.com/platforms

@falsandtru I'm not sure about that.  It might cause passing tests to start failing without any changes on our end, which would start to block releases.  I see the other side too so if @JakeChampion feels strongly about it maybe we can have a chat internally.
Also you might be interested in http://frontend.connpass.com/event/42323/ which we're running next week with the TAG at the Nikkei building. 75% of the talks are in Japanese, and there's translation for the English one.  I think polyfills will come up.

OK, CI is working, but a full compatgen is failing.  I need to do some more work here before merging.

I see. TravisCI's allow_failures option is helpful in this case, but CircleCI doesn't seem to have that option.
BTW, I applied for your event, thanks 👍

It seems that saucelabs have a specific version of edge 14 which breaks on one of our defineproperty tests. This test passes on my local version and the version on browserstack.

Running a full grunt compatgen now with latest code from master merged into this branch.


pubDate is inconsistent with spec

We are currently printing "pubDate": "2015-03-12T10:11:59.018523Z",
Spec has this: "pubDate":"Fri Jan  3 15:50:40 MST 2014",
Please look into this @Roshack or @Dylanlan

Abram mentioned something in class about this. Who was paying attention?


Use parameters from another .yaml file

Hi,
Is it possible to use the parameters defined in an external file?
I have 2 files A.yaml and B.yaml. Both the files are in the same directory.
B.yaml has the parameter defined as below:

openapi: 3.0.2
servers:
- url: https//sample
parameters:
paramA:
name: param A
in: header
description: >
Language locale accepted by client (used for locale specific fields in resource representation and in error responses).
required: true
schema:
type: string
enum: [en-US, es-ES, fr-FR]

When trying to use the parameter defined above in A.yaml, the parameter is not being picked:

openapi: 3.0.2
servers:
- url: https//sample/system
paths:
/path1:
post:
summary: Sample summary.
operationId: sampleSummary
description: |
Sample
parameters:
- $ref: 'B.yaml#/parameters/paramA'

Is the above usage correct?

You can only use $ref in specific places in OpenAPI definitions, and your example is not one of those places. You can use $ref to reference the schema of individual parameters, but you cannot use it for the entire list of parameters.
If you want to break up your OAS definition into multiple files but $ref doesn't meet your needs, check out yamlinc. It lets you place $include statements anywhere you want to merge in another YAML file, and then you run yamlinc when you are ready to merge the files into a single file. You can also set it to watch for changes in a folder and automatically merge and create the single file when files are saved. You can use yamlinc along with $ref.

the .yaml you provided lost its indentation, therefore it is invalid.
but just so you know, RapiDoc usage of $ref in the spec. and as @StephenNay mentioned, it is valid  only in certain valid sections  per the OpenApi spec.
I am closing this issue, in-case you think any of your spec with $ref isnt working, feel free to reopen it and provide a properly formatted spec for me to re-create the issue

I have attached the yaml files (Have appended the files with .log as I was not able to upload .yaml files).
In file Login.yaml the parameter 'accept' that is referred from 'Common.yaml' is not loaded (image AcceptHeaderMissing.png attached):
- $ref: './Common.yaml#/parameters/accept'
Is this the correct way of using $ref parameter from another file?

Common.yaml.log
Login.yaml.log


⚠ GitLab Server has degraded performance

In cde6648, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 9428 ms

Resolved: GitLab Server performance has improved in 1b83a40.


Beginner question: replacing int/bool member variables with Observables

I hope this is the right place for such a question. Please let me know if not.
I'm trying to integrate RxCpp into an existing C++ app. I want to replace certain member variables (in this case int and bool types) of different classes with observables. I then want to use those in a subscribe method, in order to update a UI element.
I'm struggling to work out how to adapt my code. I was hoping it would be a simple case of replacing an int x with rxcpp::observable<int> x, and have the observable emit its value each time it is updated, but it seems that I need to assign it differently. Is it possible to assign it in one class, and then subscribe to changes in a different class?
I've looked at a few examples, but still not clear on how to get started with my use case. Any tips much appreciated.

Hi!
I would need more details about how the value is updated and how the value is used to answer completely.
please look at rxcpp::behaviour_subject and samples of usage of BehaviourSubject in other languages to see if that answers you question.


Swipe with TabBar doesn't pre-render next View/Scene (based on ScrollViewExample)

Expected behaviour
When swiping any direction I expect the content of the next tab to be pre-rendered just like in the ScrollViewExample
Actual behaviour
The next content is not rendered until the transition ends (see gif below)
Code sample, screenshots

I've pretty much put the ScrollViewExample code into a component to which I pass the routes + items to render as props, and I've adjusted some colours. The code is below for completeness sake. Pretty much the only difference is in the _renderScene where I return a standard ScrollView instead of BasicListView as in the original example code.
View that includes the TabBar
...
    render() {
        const items = [
            [
                { title: 'product 1', price: 2.5 },
                { title: 'product 2', price: 3.5 },
                { title: 'product 3', price: 4.5 }
            ],
            [
                { title: 'product 1a', price: 22.5 },
                { title: 'product 2a', price: 32.5 },
                { title: 'product 3a', price: 42.5 }
            ]
        ]

        return (
            <View style={styles.fill}>
                <ItemTabs routes={this.state.routes} items={items} />
            </View>
        )
    }
...

class ItemTabs extends React.Component {
    constructor(props) {
        super(props)

        this.state = {
            index: 0,
            routes: this.props.routes
            //routes: [{ key: '1', title: 'Category 1' }, { key: '2', title: 'Category 2' }]
        }
    }

    _handleChangeTab = index => this.setState({ index })

    _renderLabel = props => ({ route, index }) => {
        const inputRange = props.navigationState.routes.map((x, i) => i)
        const outputRange = inputRange.map(inputIndex => (inputIndex === index ? '#1ABC9C' : '#222'))

        const color = props.position.interpolate({
            inputRange,
            outputRange
        })

        return (
            <Animated.Text style={[styles.label, { color }]}>
                {route.title}
            </Animated.Text>
        )
    }

    _renderHeader = props =>
        <TabBar
            {...props}
            renderLabel={this._renderLabel(props)}
            pressColor="#1ABC9C"
            indicatorStyle={styles.indicator}
            tabStyle={styles.tab}
            style={styles.tabbar}
        />

    _renderScrollViewContent() {
        return (
            <View style={styles.scrollViewContent}>
                {this.props.items[this.state.index].map(item => {
                    return <ItemRow title={item.title} price={item.price} />
                })}
            </View>
        )
    }

    _renderScene = ({ route }) => {
        switch (route.key) {
            case '1':
                return (
                    <ScrollView>
                        {this._renderScrollViewContent()}
                    </ScrollView>
                )
            case '2':
                return (
                    <ScrollView>
                        {this._renderScrollViewContent()}
                    </ScrollView>
                )
            default:
                return null
        }
    }

    render() {
        return (
            <TabViewAnimated
                style={styles.container}
                navigationState={this.state}
                renderScene={this._renderScene}
                renderHeader={this._renderHeader}
                onRequestChangeTab={this._handleChangeTab}
            />
        )
    }
}
const styles = StyleSheet.create({
    container: {
        flex: 1
    },
    indicator: {
        backgroundColor: '#1ABC9C'
    },
    tabbar: {
        backgroundColor: '#fff'
    },
    tab: {
        opacity: 1,
        width: 120
    },
    label: {
        fontSize: 14,
        fontWeight: 'bold',
        margin: 8
    }
})
export default ItemTabs

What have you tried
I've tried actually using the _renderScene implementation that uses 'BasicListView' from the original example. That does give the right transition.

I just can't see what's different that would have this effect other than a ListView vs ScrollView. However, even replacing the ListView in the render() of BasicListView with something like this:
        const data = Array.from({ length: 50 })
        return (
            <ScrollView>
                {data.map((_, i) =>
                    <Text key={i}>
                        {i}
                    </Text>
                )}
            </ScrollView>
        )

makes the ScrollViewExample  still work as expected. So it appears that using a ScrollView instead of a ListView is also not the problem.
Any hints/tips would be much appreciated

I found the culprit.
this_renderScene calls _renderScrollViewContent(). The latter decides based on this.state.index what content to put in the scene and this.state.index is not updated until the transition finishes (at least that's how I understand it now). I changed the indexing to rely on the route.key instead and now it works as expected.
from
                {this.props.items[this.state.index].map(item => {
                    return <ItemRow title={item.title} price={item.price} />
                })}

to
    _renderScene = ({ route }) => {
        const idx = parseInt(route.key) - 1
        return (
            <ScrollView>
                {this.props.items[idx].map((item, i) => {
                    return (
                        <ItemRow
                            key={i}
                            title={item.title}
                            price={item.price}
                        />
                    )
                })}
            </ScrollView>
        )
    }


[BUG] MAX/MIN functions does not work properly with array values

What is the bug?
If MAX or MIN are applied to a field with array values, then the MAX or MIN of any element in any of the array values is returned.
Consider an index with the following data:
{"x": 1, "y": [1, 2]}
{"x": 2, "y": [3, 4]}
{"x": 3, "y": [1, 5]}
{"x": 4, "y": [1, 2]}
{"x": 5, "y": [2, 3]}

and this query:
SELECT MAX(y) FROM test3;
5

or this query:
SELECT MIN(y) FROM test3;
1

For MAX, the expectd result is [3, 4] and for MIN, the expected result is [1, 2]. The comparison should be performed element by element and preserve the entire array value.
How can one reproduce the bug?
Steps to reproduce the behavior:

Create a new index and load the data above
Run the query above

What is the expected behavior?
Use array comparisons to compare the array values and return the MAX or MIN array.
What is your host/environment?

OS: Mac OS X
Version: 3.0 code base
Plugins: SQL plugin

Do you have any screenshots?
N/A
Do you have any additional context?
Issue #1300 had a change recently merged in that allows array values to be used in query evaluation and in the result set.

This appears to be due to how the OpenSearch engine calculates the min and max for arrays values.
POST test3/_search
{
  "size": 0,
  "aggs": {
    "y_min": {
      "min": {
        "field": "y"
      }
    }
  }
}

{
  "took": 6,
  "timed_out": false,
  "_shards": {
    "total": 1,
    "successful": 1,
    "skipped": 0,
    "failed": 0
  },
  "hits": {
    "total": {
      "value": 5,
      "relation": "eq"
    },
    "max_score": null,
    "hits": []
  },
  "aggregations": {
    "y_min": {
      "value": 1
    }
  }
}

POST test3/_search
{
  "size": 0,
  "aggs": {
    "y_max": {
      "max": {
        "field": "y"
      }
    }
  }
}

{
  "took": 7,
  "timed_out": false,
  "_shards": {
    "total": 1,
    "successful": 1,
    "skipped": 0,
    "failed": 0
  },
  "hits": {
    "total": {
      "value": 5,
      "relation": "eq"
    },
    "max_score": null,
    "hits": []
  },
  "aggregations": {
    "y_max": {
      "value": 5
    }
  }
}

For reference, this is how PostgreSQL behaves:
SELECT * FROM with_array;
 x |   y
---+-------
 1 | {1,2}
 2 | {3,4}
 3 | {1,5}
 4 | {1,2}
(4 rows)

SELECT MIN(y) FROM with_array;
  min
-------
 {1,2}
(1 row)

SELECT MAX(y) FROM with_array;
  max
-------
 {3,4}
(1 row)

Thanks for rasing issue. OpenSearch/Lucene does not support ARRAY data type. Lucene allows adding multiple values for the same field name.  for example. when calcualte min/max of field y, OpenSeach read all the value of y, and calcuate metrics, the result is min(y)=1, max(y)=5.
field,value
y,1
y,2
y,3
y,4
y,1
y,5
y,1
y,2
y,2
y,3

Related issue in opensearch-core. https://github.com/opensearch-project/OpenSearch/issues/16420

One solution is if array_field is used in aggregation, we should do post-processing, instead of rewrite as DSL aggregation query.

@penghuo I'm not sure the SQL plugin can easily know that the field is an array field. By default, the mapping will only contain information about the element types.
GET my_index/mapping
{
  "properties": {
    "id": {
      "type": "long"
    },
    "array_field": {
      "type": "long"
    }
  }
}

If the user adds something to the mapping, we could know that it is an array.
https://trino.io/docs/current/connector/elasticsearch.html#array-types
Is there anything similar currently in OpenSearch?

@normanj-bitquill  agree, https://github.com/opensearch-project/OpenSearch/issues/16420 discuss similar approach.

[Catch All Triage - 1, 2, 3, 4, 5]

@normanj-bitquill @acarbonetto I've seen the approach suggested here and IMO we can use the mapping._meta#field.name.type:array location for the user to explicitly hint the specific mapping field is expected to be treated and an array.
it will not cause an error if (during ingestion) this field only contains a single value but the query engine should treat this value as a single list.
IMO the first iteration would expect the plugin should be aware of the mapping._meta#field.name.type:array and actually do the aggregation.


[BUG]: Collection persistent segment dir is not removed when the collection is removed

Refs: #1009
Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

Fixed an issue where the collection's persistent segment dir was not removed; thus, many dirs with segment data were left on the device.

Test plan

[x] Tests pass locally with pytest

Documentation Changes
No doc updates are needed.

Thank you!
On Sun, Aug 20, 2023 at 9:55 AM Trayan Azarov @.***>
wrote:

@.**** commented on this pull request.
In chromadb/segment/impl/manager/local.py
https://github.com/chroma-core/chroma/pull/1010#discussion_r1299399063:

@@ -112,6 +112,9 @@ def delete_segments(self, collection_id: UUID) -> Sequence[UUID]:
segments = self._sysdb.get_segments(collection=collection_id)
for segment in segments:
if segment["id"] in self._instances:

           if segment["type"] == SegmentType.HNSW_LOCAL_PERSISTED.value:

               instance = self.get_segment(collection_id, VectorReader)

               instance.reset_state()

Agreed; this makes sense. Let me update the interface to allow for
delete() method.
—
Reply to this email directly, view it on GitHub
https://github.com/chroma-core/chroma/pull/1010#discussion_r1299399063,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABKW32LDY5W6NYWIKQSHFTDXWIXQTANCNFSM6AAAAAA3XITTCU
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Hammad Bashir
EECS | Cal

Nit: add a simple unit test to make sure files are cleaned up

@HammadB this is ready. Waiting for local tests to complete and we're good to go. Unit tests + skip removing dir if allow_reset=False for reset_state() (or should we raise an exception?)

Done

Issue under Windows:

          assert not os.path.exists(segment._get_storage_folder())

I think dir removal fails. I'll do some win testing

Running Windows tests on this now. Someone reported today that the dir was locked, so that is why it failed on Windows. @HammadB , any clue what might be locking the vector dir?

Tested on Win, found the issue (had to call close_persistent_index() before removing dir). Rebased with #1054 and tested on both Windows and Nix.

@HammadB rebased with latest changes from main. Ready for tests

@HammadB new PR #1080


test: add env_vars pytest.ini option

use env_vars key in pytest.ini to set environment variables that will be set before tests run.

sweet, thanks, @christophertubbs!


Autogenstudio merge with postgres-studio

Merge from Postgres-Studio branch
PR 1429
@victordibia
Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

This PR seems to be merging into the autogenstudio branch, but was the work started by branching off of main? There seem to be changes from main trying to get merged into the autogenstudio branch. If an update from main is in order that should be done in a separate PR since the work being contributed is lost in all the changes

@olgavrou,
This PR was part of a discussion from #1429.
I think we will proceed with that PR, merge into main and then merge that into autogenstudio feature branch.


python: use nested dicts for entity params metadata

Previously lists of dictionaries were used.
Now, metadata for entity parameters can now be accessed as a dict:
md = appleseed.Light.get_input_metadata()
md['directional_light']['irradiance']['help']

Cool!


Add subdomains for HRRR-AK

Added tiles for subdomains in HRRR-AK.  Arguments for --tiles should be 'full AKRange Anchorage Juneau' and not 'all'.

Yes, I think we'd like to preserve the 'all' flag just for CONUS subdomains.


Allow setting a custom AuthenticationFailureHandler

With the following configuration ...
    @Bean
    public JWTSecurityConfig securityConfig() {
        return JWTSecurityConfig.builder() //
               ...
                .addAnonymousPaths("auth-failure")//
                .setValueAuthenticationFailureHandler(new ForwardAuthenticationFailureHandler(
                        "auth-failure"))//
               ...
                .build();
    }

... you can e.g. forward to a custom Jersey-Resource, fetch the spring-security AuthenticationException ...
@Path("auth-failure")
public class AuthFailureResource implements JerseyResource {

    @Context
    private HttpServletRequest request;

    @GET
    @Produces(MediaType.APPLICATION_JSON)
    public void getRoot() {
        final AuthenticationException ae = (AuthenticationException) request.getAttribute(
                WebAttributes.AUTHENTICATION_EXCEPTION);
        throw ae;
    }

}

... and have all exception handling and Response creation in the Jersey layer without having to provide a custom Spring-MVC ErrorController.

Coverage decreased (-0.3%) to 92.015% when pulling 9f06e9007553cba3d5875bbd9f61cccea634523a on custom-auth-failure-handler into 3a165cfd85fc08e0095b79d18f04ebf1ee3ba454 on master.


Improving Test Coverage for getShardedRules call

What this PR does:
Which issue(s) this PR fixes:
Fixes #
Checklist

[ ] Tests updated
[ ] Documentation added
[ ] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

We are experiencing the same problem.
Looking at the code, seems that when we are updating a rule group manager for the first time after a restart we try to restore the rules which ends up on this line:
https://github.com/prometheus/prometheus/blob/ffe8c7e26884148207f5f0df06c0430e2d3b7bca/rules/manager.go#L749-L750


Running with --prod causing RangeError: Maximum call stack size exceeded

Node v8.9.1 (installed on MacOS
10.13.3)
package.json dep lists:
{   "dependencies": {     "axios": "^0.17.1",     "babel-core": "^6.24.1",     "babel-loader": "^7.0.0",     "babel-plugin-react-transform": "^3.0.0",     "babel-plugin-transform-object-rest-spread": "^6.26.0",     "babel-plugin-transform-runtime": "^6.23.0",     "babel-preset-env": "^1.6.1",     "babel-preset-react": "^6.24.1",     "babel-preset-stage-2": "^6.5.0",     "body-parser": "^1.18.2",     "browser-sync": "2.13.0",     "chalk": "^2.3.0",     "concept": "^0.1.3",     "cookie-parser": "^1.4.3",     "cookie-session": "^2.0.0-beta.3",     "cors": "^2.8.4",     "del": "^2.2.2",     "directory-tree": "^2.0.0",     "express": "^4.16.2",     "express-http-proxy": "^1.1.0",     "globby": "^7.1.1",     "gsap": "^1.20.4",     "gulp": "^3.9.1",     "gulp-autoprefixer": "^3.1.1",     "gulp-concat": "^2.6.1",     "gulp-csso": "^2.0.0",     "gulp-if": "^2.0.2",     "gulp-install": "^0.6.0",     "gulp-less": "^3.3.2",     "gulp-nodemon": "^2.2.1",     "gulp-sass": "^3.1.0",     "gulp-sourcemaps": "^2.4.1",     "gulp-util": "^3.0.8",     "handlebars": "^4.0.11",     "http-auth": "^3.2.3",     "jshint": "^2.9.4",     "jshint-loader": "^0.8.4",     "moment": "^2.20.1",     "morgan": "^1.9.0",     "nodemon": "^1.14.11",     "npm-run-all": "^4.1.2",     "parse": "^1.11.0",     "passport": "^0.4.0",     "query-string": "^5.1.0",     "react": "^16.2.0",     "react-alice-carousel": "^1.9.0",     "react-dom": "^16.2.0",     "react-helmet": "^5.2.0",     "react-jsx-parser": "^1.2.5",     "react-redux": "^5.0.6",     "react-router-config": "^1.0.0-beta.4",     "react-router-dom": "^4.2.2",     "readdir-recursive": "0.0.4",     "redux": "^3.7.2",     "redux-devtools": "^3.4.1",     "redux-devtools-dock-monitor": "^1.1.2",     "redux-devtools-log-monitor": "^1.4.0",     "redux-localstorage-simple": "^1.3.1",     "redux-super-thunk": "0.0.6",     "run-sequence": "^1.2.2",     "serialize-javascript": "^1.4.0",     "slug": "^0.9.1",     "striptags": "^3.1.1",     "uglifyjs-webpack-plugin": "^0.4.3",     "underscore": "*",     "vinyl-source-stream": "^1.1.0",     "virtual-module-webpack-plugin": "^0.3.0",     "webpack": "^3.9.1",     "webpack-node-externals": "^1.6.0",     "webpack-stream": "^4.0.1",     "window-or-global": "^1.0.1",     "yargs": "^10.0.3"   },   "devDependencies": {     "enzyme": "^3.3.0",     "enzyme-adapter-react-16": "^1.1.1",     "eslint": "^4.17.0",     "eslint-plugin-mocha": "^4.11.0",     "eslint-plugin-react": "^7.6.1",     "mocha": "^5.0.0",     "react-test-renderer": "^16.2.0"   } }
Starting with --stack-size will eventually seg fault:
e.g. node --stack_size=2500 $(which auditjs) --prod -l error

Thanks for the information. I will look into the issue.

Odd. Initial tests with node v8.2.1 on Mac OS 10.13.3 work fine. I also tried dropping the stack size down to 250 and it still worked.
It also seems to work with node v8.10.0 (which I just upgraded to).
How did you install node? Is there any other information that you think may be helpful?

Works at the default stack size for me as well. What version of audit.js are you running?
auditjs --version

@ken-duck I installed node with nvm. Running auditjs 2.4.2.

I ran into the same issue. Running in the official node:8 docker image with create-react-app.
Node version: 8.11.3
auditjs version: 2.4.3
Command: yarn run auditjs --production
Output:
node@25dda98d925d:/app$ yarn run auditjs --production
yarn run v1.6.0
$ /app/node_modules/.bin/auditjs --production
Unhandled rejection RangeError: Maximum call stack size exceeded
    at String.replace (<anonymous>)
    at lookupSpecMatch (/app/node_modules/auditjs/audit.js:550:24)
    at getDepsFromDataObject (/app/node_modules/auditjs/audit.js:588:25)
    at getDependencyList (/app/node_modules/auditjs/audit.js:513:24)
    at getDependencyList (/app/node_modules/auditjs/audit.js:515:22)
    at getDependencyList (/app/node_modules/auditjs/audit.js:515:22)
    at getDependencyList (/app/node_modules/auditjs/audit.js:515:22)
    at getDependencyList (/app/node_modules/auditjs/audit.js:515:22)
    at getDependencyList (/app/node_modules/auditjs/audit.js:515:22)
    at getDependencyList (/app/node_modules/auditjs/audit.js:515:22)
    at getDependencyList (/app/node_modules/auditjs/audit.js:515:22)
    at getDependencyList (/app/node_modules/auditjs/audit.js:515:22)
    at getDependencyList (/app/node_modules/auditjs/audit.js:515:22)
    at getDependencyList (/app/node_modules/auditjs/audit.js:515:22)
    at getDependencyList (/app/node_modules/auditjs/audit.js:515:22)
    at getDependencyList (/app/node_modules/auditjs/audit.js:515:22)

Audited dependencies: 0, Vulnerabilities: 0, Ignored: 0
Done in 22.13s.

Heavy. Thanks for the information. I will dig into this further.

Also getting this error when running auditjs-win --prod. Version 2.4.4.
Any fixes?

Not yet, as I have been away for a couple of weeks. I am back as of yesterday. I will attempt to resolve the issue today since this obviously affects several people.
Ken

On Jul 10, 2018, at 4:00 PM, Heino Sass Hallik<EMAIL_ADDRESS>wrote:
Also getting this error when running auditjs-win --prod. Version 2.4.4.
Any fixes?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/OSSIndex/auditjs/issues/41#issuecomment-403930956, or mute the thread https://github.com/notifications/unsubscribe-auth/Ac8irbgPy9KUCATwUnjYdVs8LY7FsVAzks5uFPnCgaJpZM4ShFvU.

OK, I might have a fix pushed up now. I have not reproduced the problem itself, but I figured out where the code was wrong and likely allowing infinite recursion to happen.
When you get a chance can you try the fix and let me know if it worked?
Possible fix in version 2.4.5

The problem resolved after updating to 2.4.5. Thank you!

Great to hear. Thanks for the help.
So you know, 3.0.0 (beta) will be released shortly. You will have to explicitly ask for it till it is out of beta, but it uses the new OSS Index 3.0 API which provides access to many more vulnerabilities, and is the database that will continue to grow while 2.x will be deprecated. There are still a few minor features required (username/token for higher rate limit) but it is usable. For many (most) situations an account will likely not be required (somewhere around 2000+ packages for unregistered users; it is a bit fuzzy due to how the rate limiting is implemented).

Closing the issue unless the problem still exists for some users...

The 3.0.0 beta is now available.
https://www.npmjs.com/package/auditjs/v/3.0.0
I have not updated the README yet. You will find most things work the same as they used to. There are several new features which need to be documented.
One thing to note is that the API is rate limited. It is unlikely to be a problem in most cases, but you can increase the rate limit by registering for an account at https://ossindex.sonatype.org ; you can use your username (email address) and security token (on your settings page). Instead of putting this info on the command line you can use a config file, but I still need to document that.
I am interested in any feedback. I have not fully tested all options, I suspect, but most situations should hopefully work fine.


Language changes back to english on refresh

The user's language is respected when the timeline renders first time but then on explicitly refreshing it renders in english

Fixed in https://github.com/anandbn/timeline-lwc/commit/6e6a2006fc06a33b8f79587ed97505195cd2fb5d


Update AlertDialog import to use SupportLibrary.

Motivation
Fix the button not fully shown issue on Alert. This issue happens on Samsung S5 with API 21 and Samsung Note 3 with API 21.
Before:

Test Plan
https://github.com/yihanseattle/ReactNativeAlertButtonText
This app can produce the issue.

What versions of Android have you tested this on? Can you post a screenshot of the "after"?

@ericvicenti it's on API 21. Tested on two real devices(Samsung S5 and Samsung Not 3) and an emulator. They are all on API 21. You are welcome to try the sample app on an emulator with API 21. The AlertDialog failed for me. Yes, will post a screenshot of "after".

Hi @ericvicenti, just added a screenshot of "after". and also added a native Android sample app that you can directly test the two AlertDialog. When I use the Support Library, the text is not cutting anymore. It only happened on a Samsung S5 with API 21. Then I verified that it happens on all devices with API 21. Let me know if you see the same.... thanks! :)


Candidates Processor should output a list of dataframes instead of a single dataframe

I think this should wait until imsub branch has been merged into main