Implement number of moves counter

Hello,
I have implemented the feature in subject. I am still learning React and Styled Componets as well, so feel free to review my changes and suggest a re-work if you think the changes can be done "cleaner". I will be happy with any kind of feedback.

LGTM

Thanks, @DusanSacha 🍰

Since you're a member of the org now you can create branches directly in the repos (no need to fork), so the CircleCI build will also work and you can know if your unit-tests are OK.


Connection issues never seem to go away(how to deploy prisma live reliably) [MySQL, TypeGraphQL, EBS]

Bug description
Hello,
Ever since our migration from Prisma 1 to Prisma 2, and connection issues never got solved for us. We wanted to deploy our same Nextjs website(after query changes) using the new Prisma 2 backend hosted on lambda using serverless framework and TypeGraphQL.
We tried RDS proxy when it was first suggested by Prisma months ago, but then were surprised in the updated docs that it would be useless when used with Prisma.
That left us stuck, not knowing how to proceed from that. So we decided to step away from lambda and head to long running servers with elastic beanstalk. While preparing for that with the docker image, running the container locally using apollo server(not apollo server lambda), we still faced the same connection issues.
We referred back to the docs for the suggestions regarding deploying using the long running server model, we tried increasing the connection_limit from the default(9) to big numbers like 200 and 300. Connection issues were occurring just the same. We noticed that RDS connections never really go up to the number we specified while Prisma reports that it's actually using the number, while same issues were happening.
Since we're going the long-running server model, we want to guarantee that we won't face these issues once we go live.
How to reproduce
What we were testing the most was the homepage. In getServerSideProps(Nextjs), we prepare 5 graphql requests to get page content, and send them concurrently using Promise.all to the Prisma 2 backend.
Expected behavior
The whole migration we did from Prisma 1 was in hopes of better database schema(Prisma 1 schema in the db level was a mess, many unnecessary relations, etc), better db access, reduced costs(no need for a separate server). But so far our experience has been horrible(deployment and testing stage), we kind of regretted doing the migration. We were better off with Prisma 1(despite Prisma 2 benefits).
Prisma information
const 1stPortion = services.getItems(
    user?.id,
    {
        ignoredBy: {
            none: {
                id: {
                    equals: user?.id ?? '',
                },
            },
        },
        status: {
            in: ['ACTIVE', 'ACTIVEUPDATED'],
        },
    },
    { score: 'desc' },
    4
);
const 2ndPortion = services.getItems(
    user?.id,
    {
        category: {
            equals: 'FARM',
        },
        type: {
            equals: 'RENT',
        },
        ignoredBy: {
            none: {
                id: {
                    equals: user?.id,
                },
            },
        },
        status: {
            in: ['ACTIVE', 'ACTIVEUPDATED'],
        },
    },
    { createdAt: 'desc' },
    4
);
const 3rdPortion = services.getItems(
    user?.id,
    {
        category: {
            equals: 'APARTMENT',
        },
        type: {
            equals: 'SALE',
        },
        ignoredBy: {
            none: {
                id: {
                    equals: user?.id,
                },
            },
        },
        status: {
            in: ['ACTIVE', 'ACTIVEUPDATED'],
        },
    },
    { createdAt: 'desc' },
    4
);
const 4thPortion = services.getItems(
    user?.id,
    {
        category: {
            equals: 'APARTMENT',
        },
        type: {
            equals: 'RENT',
        },
        ignoredBy: {
            none: {
                id: {
                    equals: user?.id,
                },
            },
        },
        status: {
            in: ['ACTIVE', 'ACTIVEUPDATED'],
        },
    },
    { createdAt: 'desc' },
    4
);
const 5thPortion = services.getItems(
    user?.id,
    {
        category: {
            equals: 'LAND',
        },
        type: {
            equals: 'SALE',
        },
        ignoredBy: {
            none: {
                id: {
                    equals: user?.id,
                },
            },
        },
        status: {
            in: ['ACTIVE', 'ACTIVEUPDATED'],
        },
    },
    { createdAt: 'desc' },
    4
);

const pageData = await Promise.all([
    1stPortion,
    2ndPortion,
    3rdPortion,
    4thPortion,
    5thPortion,
]);

Environment & setup

OS: Ubuntu 18.04
Database: MySQL 5.7
Node.js version: v14.15.1
Prisma version: 2.19

Prisma Version
prisma               : 2.19.0
@prisma/client       : 2.18.0
Current platform     : debian-openssl-1.1.x
Query Engine         : query-engine c1455d0b443d66b0d9db9bcb1bb9ee0d5bbc511d (at<EMAIL_ADDRESS>Migration Engine     : migration-engine-cli c1455d0b443d66b0d9db9bcb1bb9ee0d5bbc511d (at<EMAIL_ADDRESS>Introspection Engine : introspection-core c1455d0b443d66b0d9db9bcb1bb9ee0d5bbc511d (at<EMAIL_ADDRESS>Format Binary        : prisma-fmt c1455d0b443d66b0d9db9bcb1bb9ee0d5bbc511d (at<EMAIL_ADDRESS>Studio               : 0.358.0

What kind of connection issues are you referring to? Maybe we have the same issue.
I get the following errors after running my application for around a day:
Error: Timed out fetching a new connection from the pool. Please consider reducing the number of requests or increasing the `connection_limit` parameter (https://www.prisma.io/docs/concepts/components/prisma-client/connection-management#connection-pool). Current limit: 10.

The issue does not appear during high load and prisma never recovers from this error. It just keeps spamming and an application restart is needed.
I raised the connection limit from 2 to 10 without success.
I checked database connections (MySQL 8.0) when the error occurs using SHOW PROCESSLIST and noticed that not a single connection is actually used.

@P4sca1 Yeah, same issue. Changing connection limit doesn't really help.

@omar-dulaimi Thanks for the writeup here.
What is missing is the information how you get from the codesnippet you posted to an error message. With the information I have, I would assume sending a request that executes that code runs 5 queries in parallel, so will use 5 connections in parallel, and then returns that data to the request and user - easy.
But that might be a wrong assumption if services.getItems does more than to simply send a simple query. Can you share more here?
Also, when are you getting these connection errors - just by clicking around your app manually by hand, 1 page at a time, or while load testing with hundreds or thousands of virtual users? How is your database configured?
Besides these open questions:

prisma               : 2.19.0
@prisma/client       : 2.18.0

You should always use identical versions for CLI and Client. Please also try to upgrade to a more recent verison of both, 2.23.0 was released last week.
I am happy to debug this together with you. I will just need some more information from you.

@P4sca1 I think we talked about this in Slack already, but to make it explicit: Please open a new issue and describe your situation. This error message can have a million reasons really, and I doubt you have the same problem as Omar. Thanks.

Many issues have been fixed in more recent prisma releases (>2.20 at least), see https://github.com/prisma/prisma/issues/6329#issuecomment-811097812.

Hello @omar-dulaimi and others,
we are working on a Prisma Data Proxy which will help with connection management in serverless backens. If you are interested in our upcoming Early Acess, you can sign up here: https://pris.ly/prisma-data-proxy

Hello @albertoperdomo and thank you for letting me know. Unfortunately, we have moved away from serverless as our main compute environments.

This is fixed. The full information can be found here https://github.com/importcjj/mobc/issues/63
This will be out in the next release. You can test it now with prisma@dev


Output err msg

Changes

Output err message when unmarshal message failed.
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[] Has Docs included if any changes are user facing
[] Has Tests included if any functionality added or changed
[] Follows the commit message standard
[] Meets the Tekton contributor standards (including
functionality, content, code)
[ ] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[ ] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

/kind clean-up

/ok-to-test

/kind misc

/retest

thanks @cleverhu for this PR!
Converting @bendory's into lgtm!
/approve
/lgtm


Focus removed when popup dismisses

I have a use case where I show a floater style popup whilst a SwiftUI textfield is focuses (keyboard shown). I noticed when the toast auto hides (only case I've tested) it calls some code that dismisses the keyboard, resulting in focus being lost to my textfield.
This seems like it isn't intended behaviour and if so not sure why. I found this PR introducing the change but couldn't find context as to why?
https://github.com/exyte/PopupView/blob/4e64eb67d4510c96f82efd9e34b8fa5b9b397791/Source/FullscreenPopup.swift#L240

Hey @benjamin-es-hall, I remember this PR was meant as a way of fixing some animation glitch, but I guess it wasn't the best way to fix it. Anyway I removed that code and released version 2.8.4, please try and see if it works ok. Have a great day!

Oh wow thanks for the super quick update! I'll close this issue as now works as expected :) Thanks again!


Add support for all CS functions as of 7.53

Hiya @heydona
Hope all is well and sunny! There seems to be more to this than just adding support for 7.53 functions, no?

It is bringing syntax highlighting to all CS functions/keywords just extending the words.
I've not changed any of the regexes (yet)

Ah, I think that my confusion comes from the mismatch of the .YAML-tmlangage files and the .tmlanguage files, I'll have a look whenever I get a chance


ENH : add new notebook about how to interpret IQMs

With this notebook, we are contributing an educational resource on how to interpret and select the most important image quality metrics (IQMs) generated by MRIQC.

Okay, can you rebase on main? I just fixed the github action that was failing

Okay, can you rebase on main? I just fixed the github action that was failing

Done!


Tabs covered by '+v' buttons

When the app opened the first time ("floating" rather than maximized etc.) and I opened a second tab, the right tab and its 'x' close button were covered by the '+v' buttons to add tabs.  I had to expand the window to make it wide enough that the tab was no longer covered.

It is in #1375, the megathread on all issues related to the design, at the end of the post, so they are aware of this issue :-)

Yep that's correct. #1948 will make this better, but we're still waiting on some features to be added to the TabView in the upstream microsoft-ui-xaml project.
Closing as a dupe for now :)

Thanks, folks!  I figured it'd been caught, but my cursory search showed no results and many issues.  Thanks for following up!


docs: clarify caching behavior

PR rebased on alpha as requested in comment.
The example contains at least one inacurate statement,

It will then cache the data using 'todos' and fetchTodos as the unique identifiers for that cache.

and could benefit from more precise language.

:tada: This PR is included in version 4.0.0-alpha.9 :tada:
The release is available on:

npm package (@alpha dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 4.0.0-beta.1 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Create pull_request_template.md

Adds some guidance on how to edit/create intel modules

@ramonpetgrave64 may you can add :

edit cli.py
edit config.py
add module description on READEME.md

May be you can refer to @achantavy Google Doc or add some precisions about auto cleanup (only Node with sub-resource link will be deleted)
May be adding a link to analysis jobs can be helpful too.
A bit out of scope but a detailed exemple of transform with n to n cardinality will be useful.
This template will be helpful.

edit cli.py
edit config.py
add module description on READEME.md
May be you can refer to @achantavy Google Doc or add some precisions about auto cleanup (only Node with sub-resource link will be deleted)
May be adding a link to analysis jobs can be helpful too.
A bit out of scope but a detailed exemple of transform with n to n cardinality will be useful.

This is very good documentation feedback, thank you.
Regarding the pull request template, I think a long list like this would do better in the how to write an intel module guide instead of this checklist.
I think we should keep a pull request template to roughly 5 items max. This is meant to remind a contributor of things they might have missed, not show them how to do everything.

@achantavy @resilience-jychp Thanks. I've shortened it

@ramonpetgrave64 may be you can add a reminder for "extra-index" in your checklist

Create or update schema docs under docs/root/modules/.


Support for Vercel AI SDK v3 / Generative UI

Hi there, wondering if this package would be able to upgrade to SDK v3 to support generative UI with react native?

Fyi, since this is about upgrading, I'll mention that this may or may not be compatible with Expo SDK 50.
I get ReferenceError: Property 'document' doesn't exist when I:

add the metro config config.resolver.unstable_enablePackageExports = true; and then npx expo prebuild --clean + npx expo ios
add "browser": {  "ai/react": "./node_modules/ai/react" }, to package.json
replace ai/react with react-native-vercel-ai

Reverting these changes resolves the error. But my setup is so convoluted, I'll refrain from filing a bug. Just wanted to post this here since you're updating the library.
versions

"ai": "^2.2.26",
"react-native-vercel-ai": "^0.1.2",
"expo": "^50.0.17",

(Thanks for this library!)

Any update ?


First optgroup label is shifted/broken when data-actions-box = true

I'd like to continue using the data-actions-box but I also need some group labels because this select list is going to be very long. Has anyone else noticed this issue? Or is it currently on the list to be addressed?
I searched the existing and previous issues and didn't find anything documented about it or a fix.
This is my code:
<select id='select-customer' class='selectpicker form-control' data-actions-box="true" data-size="15" multiple data-selected-text-format="count>2">
    <optgroup label="CustomerGroup">
        <option>Customer 1.1</option>
        <option>Customer 1.2</option>
	<option>Customer 1.3</option>
    </optgroup>
    <optgroup label="CustomerGroup2">
        <option>Customer 2.1</option>
	<option>Customer 2.2</option>
	<option>Customer 2.3</option>
    </optgroup>
</select>

What browser/OS are you using? Are you sure you don't have some interfering CSS on the page? Can you create a reduced test case and link me to it?

Thanks for checking in with me. It must have had something to do with CSS interference because when I just went back to test again to reply to this it is in the place it is supposed to be now.
Thanks anyway!


How to handle embedding layernorm

some model may do layernorm after embedding, then send it to Attention layers, when face this type model, do i need to add embedding layernorm to eagle or any other trick which i need do to make eagle output right tokens.
I don't know why need -2 when generate train data in llama, and how to change the -2 in myown ge_data script to other model, from now on, i try not -2 at generrate data, and add embedding layernorm or not for training eagle, both don't make good result in parallel decoding, i'm confused, the Model is BluLM-7B-Chat, thanks for helping me!

The hidden state input to the draft model is after the norm layer, so we did not use a norm layer before the attention in the draft model.

What do you mean by -2?

The hidden state input to the draft model is after the norm layer, so we did not use a norm layer before the attention in the draft model.

I'm not mean the hidden state, i mean the embedding of input tokens. The main model generate first token, then send it to eagle, eagle will embedding the input tokens, then if eagle need to do layernorm to the embedding befor concat embedding with hidden state?

What do you mean by -2?

In data generate python script, eagle code comment shows below:

I don't know if i need to change the number to minus when i try to implement eagle in any other Model?

I'm not mean the hidden state, i mean the embedding of input tokens. The main model generate first token, then send it to eagle, eagle will embedding the input tokens, then if eagle need to do layernorm to the embedding befor concat embedding with hidden state?

Due to computational resource constraints, we have not conducted experiments on adding an additional norm layer.

In data generate python script, eagle code comment shows below:

I don't know if i need to change the number to minus when i try to implement eagle in any other Model?

This is to ensure the correct position of the loss mask. You can check tokenizer.decode(input_ids[loss_mask_pos]), which should correspond to the human instruction part offset by one token.

I'm not mean the hidden state, i mean the embedding of input tokens. The main model generate first token, then send it to eagle, eagle will embedding the input tokens, then if eagle need to do layernorm to the embedding befor concat embedding with hidden state?

Due to computational resource constraints, we have not conducted experiments on adding an additional norm layer.

I try to add embedding layernorm into Eagle to make structrue of Eagle is similar to the Original Model, i find after add embedding layernorm, Eagles work good, if i remove embedding layernorm, Eagle works bad. I don't know why, just report the appearance to you~

In data generate python script, eagle code comment shows below:

I don't know if i need to change the number to minus when i try to implement eagle in any other Model?

This is to ensure the correct position of the loss mask. You can check tokenizer.decode(input_ids[loss_mask_pos]), which should correspond to the human instruction part offset by one token.

3Q for you reply, now i know how to estimate this value! It's very helpful~

I find Eagle has to many detail in the code implemention, so if it's convenient, i wish more comment or write a doc about the code design, i already add some in my fork to help me understand Eagle's complex logic, even some details which aren't mentioned in the article.

I'm not mean the hidden state, i mean the embedding of input tokens. The main model generate first token, then send it to eagle, eagle will embedding the input tokens, then if eagle need to do layernorm to the embedding befor concat embedding with hidden state?

Due to computational resource constraints, we have not conducted experiments on adding an additional norm layer.

I try to add embedding layernorm into Eagle to make structrue of Eagle is similar to the Original Model, i find after add embedding layernorm, Eagles work good, if i remove embedding layernorm, Eagle works bad. I don't know why, just report the appearance to you~
Hello, I would like to ask how much performance improvement can be achieved by adding this norm layer? Is it added during training, and then Eagle is retrained with it, or is it only added during inference?


[TASK] download and rename auto-created journeys post transactionalEmail creation

when a transactionalEmail record is created it auto-creates a matching journey.
currently we recommend downloading it but instead we should auto-download it and then change its name to no longer have a datestamp in it

as it is recommended to go via publishing the journey instead, this task seems irrelevant now.


Introduce ResolveDecodeEntity

I've been playing with this idea of supporting an API migration from param("foo").as[Int] to param[Int]("foo"). What makes it complicated is that we have to maintain a support for just param("foo") and have it resolved into Endpont[String]. See #786.
This PR introduces a new, temporary type-class ResolveDecodeEntity that maps a given type parameter (Nothing in the case of param("foo") onto a DecodeEntity instance). Here is how it works:
scala> import io.finch._
import io.finch._

scala> param("foo")
res0: io.finch.Endpoint[io.finch.internal.ResolveDecodeEntity.nothing.Out] = param(foo)

scala> param[Int]("foo")
res1: io.finch.Endpoint[Int] = param(foo)

scala> param("foo").as[Int]
res2: io.finch.Endpoint[Int] = param(foo)

scala> param("foo"): Endpoint[String]
res3: io.finch.Endpoint[String] = param(foo)

Note: res0 type looks rather confusing but it's perfectly convertible into String (as shown in res3). I don't believe this should be a problem for any application's code.
I'd appreciate any feedback people may have about this.

Codecov Report

Merging #873 into master will decrease coverage by 0.76%.
The diff coverage is 69.23%.

@@            Coverage Diff             @@
##           master     #873      +/-   ##
==========================================
- Coverage    81.9%   81.14%   -0.77%
==========================================
  Files          51       52       +1
  Lines         746      753       +7
  Branches       34       31       -3
==========================================
  Hits          611      611
- Misses        135      142       +7

Impacted Files
Coverage Δ

core/src/main/scala/io/finch/internal/Rs.scala
88.23% <0%> (-5.52%)
:arrow_down:

.../scala/io/finch/internal/ResolveDecodeEntity.scala
100% <100%> (ø)

...c/src/main/scala/io/finch/generic/FromParams.scala
50% <50%> (ø)
:arrow_up:

core/src/main/scala/io/finch/endpoint/param.scala
71.42% <66.66%> (-1.65%)
:arrow_down:

core/src/main/scala/io/finch/Input.scala
87.5% <0%> (-12.5%)
:arrow_down:

Continue to review full report at Codecov.

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

Oh, I also did some work last night on this issue. Such a coincidence.
Anyway, I think this new type class is not required since simple prioritization of implicits should solve the problem with default type.

Abandoning this in favor of #875.


Cache

We need a simple yet effective strategy to make several invocations of ncc build or ncc run in a row re-use as much work as possible.

[ ] FS cache for the CLI

Uses the most appropriate location for each OS (like yarn)
Flag to ignore it for both run and build (--no-cache)
Subcommands to find it and clean it (ncc cache dir and ncc cache clean)

[ ] In-memory interface for the programmatic API (ncc())

Of the time spent in the build, over 50% is now spent on minification, with only a very small fraction of time spent in the relocation loader.
Unfortunately Webpack internals don't seem like they offer much in the way of persistent caching right now (correct me if I'm wrong, but got this from - https://github.com/webpack/webpack/issues/6527).
So I think the initial focus needs to be on minification (and TypeScript support when merged).
So the question then is how to properly cache minification in a way that can be shared between builds.
One option here could be running minification as a loader, having it apply per-input-source before Webpack does its tracing and concatentation. This would rely on caching all the other loaders and their emissions as well (including the relocation loader) and then having the previously generated source used when the input matches.
I'm not sure that per-source minification is a proven approach (please let me know if I'm wrong), so this would be yet to be seen what wins can be made here.


x/tools/gopls: panic in ast.Walk

Getting this panic using gopls golang/tools@a1fbc781c2e5ebb63742eb7d6927a638ab448514 and Go tip:
runtime.gopanic:59
runtime.panicmem:?261
go/ast.Walk:184
go/ast.Inspect:?397
golang.org/x/tools/gopls/internal/golang.highlightPath:29
golang.org/x/tools/gopls/internal/golang.Highlight:31
golang.org/x/tools/gopls/internal/server.(*server).DocumentHighlight:14
golang.org/x/tools/gopls/internal/protocol.serverDispatch:280
golang.org/x/tools/gopls/internal/lsprpc.(*streamServer).ServeStream.ServerHandler.func3:5
golang.org/x/tools/gopls/internal/lsprpc.(*streamServer).ServeStream.handshaker.func4:52
golang.org/x/tools/gopls/internal/protocol.Handlers.MustReplyHandler.func1:2
golang.org/x/tools/gopls/internal/protocol.Handlers.AsyncHandler.func2.2:3
runtime.goexit:0

Noticed since gopls was exiting, sourced this stack from my local telemetry data. I assume it will be uploaded at some point but wanted to bring it up.
Maybe related to golang/tools@85146f5b48ef9cb082397e54dab98b7861424bdc / #60589?

Indeed, this is almost certainly a new crash introduced by my CL. Thanks!

Congratulations: you discovered the the first auto-reported crash to pass through the new machinery that landed on Friday!


Can't run on Mac OS Big Sur

This is actually a problem with PYQT5 . We either need to wait for an update, or use another GUI technology.
Many people have the same issue with other Python projects, check Google.

Rolling back to PyQt5==5.13 solves the problem.
pip install PyQt5==5.13


Azure AD Identity and Access (Independent Publisher)

When submitting a connector, please check the following conditions for your PR.

[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a logo.

@sriyen-msft when will this one be checked? 🙂

Can I get an update on this? This is taking very - very long this way.

@natalie-pienkowska changed the connector name as promised. Good to go now.

@sriyen-msft Committed the changes. So all good to go now.

@sriyen-msft It does help when I push the changes as well ;-) Now it should be OK...

@Laskewitz We will move forward with preparing your connector for the next deployment schedule. Once your connector is onboards the next deployment schedule, it will deploy your connector in our production environments which would typically take between 7-10 business days. Please note that the documentation for your connector will not be generated until the deployment process is in the final steps. Thanks

[[certify-connector]]

@Laskewitz  We are prepping your connector production deployment; we need to set up an OAuth app and add its client id and secret values to our Production KeyVault. We are unable to set up an OAuth app using our test account as it doesn’t have the required permissions to Azure AD.
Since you are an internal partner, can you please set up the OAuth app and share the client id and secret to us in a secure channel, so that we can upload them to our Prod KV to proceed with production deployment.
@natalie-pienkowska and @mamurshe  - FYI

@Laskewitz Deployment freeze notice: Due to Thanksgiving week in the United States, we have temporarily halted the deployments of all the connectors till Tuesday, 30th November 2021.
We will begin the deployments on Wednesday, 1st December 2021. If your connector is queued up for Wednesday (1st December 2021) deployment schedule, the production deployment of your connector in all regions will be expected to be completed by Thursday, 9th December 2021. Thanks

Our team is currently working on setting up OAuth for this connector. We will provide updates and comments along the way.

Any update on this?

Hi @Laskewitz this connector is on the deployment train. It should be fully deployed by February 7th :)

Unfortunately, we had some issues with the deployment. @mamurshe is on top of this and will do everything he can to deploy these connectors.

@Laskewitz Apologies for the delay in response, We are actively working on preparing and deploying your connector to publish environments. As you know that we are deploying the connector as a Standard connector, For that, we need to go thru a few guidelines and approvals from the concerned teams (ex. Power BI API team) which is a bit lengthy process.  Thank you so much for your patience.

Dear partner,
We apologize for the delay you might be experiencing while we are working on a high volume of connector requests. Rest assured that we are doing everything we can to address this volume. We are taking measures like improving our processes and hiring additional team members to guarantee resolving your connector requests in a timely manner.
Please let us know if you have any questions in the meantime.

Hello @Laskewitz,
I hope you are doing well.
Your connector is in production deployment and queued up for the next deployment schedule. This process typically takes 3-4 weeks. Please note that the documentation for your connector will not be generated until the deployment process is in the final steps.
Please accept our apologies for the delay in the process.
Please let us know if you have any questions.
Thank you very much for working with us.

@Laskewitz Apologies for the delay in response.
Congratulations! The deployment of your connector and documentation page is now complete, We have a short survey on the connector certification process that we would like for you to complete. Your feedback is appreciated and will help us improve the program, Thanks.


when setslot set cluster version just bigger than last version

Search before asking

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

Motivation
It is easier for controller, the controller can only use one increase bigint sequence for one kvrocks cluster.
Solution
Change cluster version step+1 check to just bigger than check.
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

For CLUSTERX  setnodes command, it's right that we don't require to check if the version is continuous since it will fully update the cluster topology. But for the CLUSTERX setslots, it's a partial update behavior, so we need to check if the version is +1 to prevent missing some updates.

@segment11 It's fine to keep this issue since we can allow the CLUSTERX SETNODES just check if it's bigger than before.


feat: improve apisix-base build

[x] Multi-architecture
[x] Multi-stage build to reduce image size

A demo of the results https://hub.docker.com/r/bzp2010/apisix-base/tags

This PR requires careful review, the code it executes cannot be verified by CI, and is only built and pushed when a new tag is created, so we cannot just build test without pushing the image.

Can we release an updated version of <IP_ADDRESS>.1 for this? (<IP_ADDRESS>.0 => <IP_ADDRESS>.1)

I'm testing this build in my own fork repository here.
https://github.com/bzp2010/apisix-build-tools/runs/4785042310?check_suite_focus=true


Misleading error when unconfigured authClasses are present in Cluster

Preflight checklist

[X] I agree to follow this project's Code of Conduct.
[X] I have read and am following this repository's Contribution Guidelines."
[X] I could not find a solution in the existing issues, docs, nor discussions.

Describe the bug
When one has two Heimdall deployments which are both watching separate authClasses, both instances will produce an error message about an authClass mismatch, even though that is expected behavior. This may mislead users and alerting tools.
How can the bug be reproduced

Deploy two Heimdall instances, each watching a different authClass
Create RuleSets for each authClass
Look for errors referencing RuleSets of the other authClass in the instance logs

Relevant log output
{"_level_name":"INFO","version":"1.1","host":"unknown","_provider_type":"kubernetes","timestamp":1687947309,"level":6,"short_message":"Ignoring ruleset due to authClassName mismatch (namespace=foo-dev, name=foo-dev-bar-rules, uid=8b69fde6-dcb1-4a23-ba65-df676cf96256)"}
{"_level_name":"ERROR","version":"1.1","host":"unknown","_provider_type":"kubernetes","timestamp":1687947309,"level":3,"short_message":"k8s.io/client-go@v0.27.3/tools/cache/reflector.go:231: Failed to watch *v1alpha2.RuleSet: unable to sync list result: couldn't enqueue object: bad authClass in a RuleSet\n"}

Relevant configuration
# Instance A
rules:
  providers:
    kubernetes:
      auth_class: foo

# Instance B
rules:
  providers:
    kubernetes:
      auth_class: bar

Version
2dc87bef6b10b5bbfa6b9f01cb6e65950f4a8bd4
On which operating system are you observing this issue?
Linux
In which environment are you deploying?
Kubernetes with Helm
Additional Context
No response

Actually this is a bit weirder, it seems like all the errors are about the alphabetically first rule and the rest never gets checked, meaning the actually matching rules also never get loaded.
Looking closer into this.

Yeah for some reason the controller gets stuck once it hits a RuleSet with an unsupported authclass.


swing/pull toggle

kind of like in magrope+ u can swing / pull toggle with a button

no. Next time make a big issue with all your ideas please, dont spam them. Or just post them on our Discord in #suggestions channel (using my bot for that)


Virtual Machine screen sorting

currently the Virtual Machine screen is sorted by Machines then VMs I would love to see a toggle for that and sort by just VMs like you do in VMware. If you have not used Vsphere there are 4 area or toggle that you can display VMs based on and thats Node, Vms, Storage and networks.
as a bonus it would also be nice to have a universal search box at the top where you can start typing the object then you get the list of VMs, networks, storage...

I'll figure something out here, I already have some ideas on my mind.
Thanks for suggesting this improvement.


Add check for non-integer value in 'get_battery_info()'

Coverage remained the same at 57.179% when pulling 9a4747667777131d5eafe6c9f72d783f35626e1f on pwt:get_battery_info-extra_check into a75b8bb15b04af40bfc30ad5828f568b6ee5834a on SoCo:master.


[Bug] Jimple2cpg stack trace during spring web jar import

Describe the bug
Importing this jar Spring-Web 5.2.2 is causing a stack trace:
Picked up _JAVA_OPTIONS: -Dawt.useSystemAAFontSettings=on -Dswing.aatext=true
2023-05-06 19:40:52.146 WARN AstCreationPass: Unexpected runtime exception while parsing method body! Will stub the method 'org.springframework.http.client.Netty4ClientHttpRequest.executeInternal:org.springframework.util.concurrent.ListenableFuture(org.springframework.http.HttpHeaders)''
java.lang.RuntimeException: Failed to apply jb to <org.springframework.http.client.Netty4ClientHttpRequest: org.springframework.util.concurrent.ListenableFuture executeInternal(org.springframework.http.HttpHeaders)>
	at soot.asm.AsmMethodSource.getBody(AsmMethodSource.java:2281) ~[org.soot-oss.soot-4.4.1.jar:?]

In the stack trace, it is suggested to do:
Scene.v().addBasicClass(io.netty.channel.ChannelFutureListener,HIERARCHY);
Otherwise, try whole-program mode (-w).

Neither of them fix the issue.
I've had this issue before, but I've never been able to find a fix for this.

@evilpan Have you ever managed to fix this kind of issue?

Sorry that I missed the metion. For this case whole program mode (-W) should work, but somehow it throws another exception:
$ java -cp soot-4.4.1-jar-with-dependencies.jar soot.Main -W -process-path spring-web-5.2.2.RELEASE.jar -f J -d out -allow-phantom-refs
SLF4J: No SLF4J providers were found.
SLF4J: Defaulting to no-operation (NOP) logger implementation
SLF4J: See https://www.slf4j.org/codes.html#noProviders for further details.
Soot started on Thu Jun 01 10:50:08 CST 2023
java.util.ConcurrentModificationException
        at soot.util.HashChain$LinkIterator.hasNext(HashChain.java:582)
        at soot.jimple.toolkits.invoke.StaticInliner.computeAverageMethodSizeAndSaveOriginalSizes(StaticInliner.java:149)
        at soot.jimple.toolkits.invoke.StaticInliner.internalTransform(StaticInliner.java:74)
        at soot.SceneTransformer.transform(SceneTransformer.java:36)
        at soot.Transform.apply(Transform.java:105)
        at soot.ScenePack.internalApply(ScenePack.java:37)
        at soot.Pack.apply(Pack.java:118)
        at soot.PackManager.runWholeProgramPacks(PackManager.java:621)
        at soot.PackManager.runPacksNormally(PackManager.java:500)
        at soot.PackManager.runPacks(PackManager.java:425)
        at soot.Main.run(Main.java:266)
        at soot.Main.main(Main.java:142)

And this was tagged as bug in soot https://github.com/soot-oss/soot/issues/1206 without a fix yet. Maybe I can create some tests with sootup to see if it's solved in their successor.

Hi @DavidBakerEffendi
According to my experiment, the exception in jimple2cpg is caused by these two lines  https://github.com/joernio/joern/blob/master/joern-cli/frontends/jimple2cpg/src/main/scala/io/joern/jimple2cpg/Jimple2Cpg.scala#L178-L180
// output jimple
Options.v().set_output_format(Options.output_format_jimple)
Options.v().set_output_dir(ProgramHandlingUtil.getUnpackingDir.toString)

After commenting out these line, parsing should not throw any exceptions. Most of time people don't need to generate jimple files through jimple2cpg and we don't have any options to toggle this feature, so I think comment out these output option is a good idea for now.

Thanks, @evilpan! I'll follow up with this soon, there are some other issues with jimple2cpg I need to look at, but that would fix a bunch of other legacy issues.

@evilpan seems like I still get the stacktrace above when the above code is commented out. But perhaps this PR sheds some light on the issue https://github.com/joernio/joern/pull/2839

That's weird, I just pulled the latest code and find out that jimple2cpg can successfully generate cpg of spring-web.jar without any error. Here's what I did:
$ git pull && sbt stage
$ ./joern-cli/frontends/jimple2cpg/jimple2cpg spring-web-5.2.2.RELEASE.jar -o spring.cpg
$ ls -l spring.cpg
-rw-r--r-- 1 pan staff 17235968 Jun 11 16:50 spring.cpg
$ ./joern
Compiling /Users/pan/Tools/joern/(console)

     ██╗ ██████╗ ███████╗██████╗ ███╗   ██╗
     ██║██╔═══██╗██╔════╝██╔══██╗████╗  ██║
     ██║██║   ██║█████╗  ██████╔╝██╔██╗ ██║
██   ██║██║   ██║██╔══╝  ██╔══██╗██║╚██╗██║
╚█████╔╝╚██████╔╝███████╗██║  ██║██║ ╚████║
 ╚════╝  ╚═════╝ ╚══════╝╚═╝  ╚═╝╚═╝  ╚═══╝
Version: 1.2.12
Type `help` or `browse(help)` to begin

joern> importCpg("spring.cpg")
Creating project `spring.cpg` for CPG at `spring.cpg`
Creating working copy of CPG to be safe
Loading base CPG from: /Users/pan/Tools/joern/workspace/spring.cpg/cpg.bin.tmp
Adding default overlays to base CPG
The graph has been modified. You may want to use the `save` command to persist changes to disk.  All changes will also be saved collectively on exit
The graph has been modified. You may want to use the `save` command to persist changes to disk.  All changes will also be saved collectively on exit
res0: Option[Cpg] = Some(value = Cpg (Graph [298779 nodes]))

joern> cpg.typeDecl.size
res1: Int = 2248

I've tried on both MacOS and Ubuntu(22.04.1 LTS) and got similar results.

@evilpan It will be successful, but the error/warning describes when a method body will be empty - it will continue generating the CPG though. You can get that by using importCode on the jar directly.

Hi @DavidBakerEffendi, After some digging, I may found the root cause of this issue.
In method Netty4ClientHttpRequest.executeInternal, there's a implicit conversion from lambda expression to functional interface:
	@Override
	protected ListenableFuture<ClientHttpResponse> executeInternal(final HttpHeaders headers) throws IOException {
		final SettableListenableFuture<ClientHttpResponse> responseFuture = new SettableListenableFuture<>();

		ChannelFutureListener connectionListener = future -> {
			if (future.isSuccess()) {
				Channel channel = future.channel();
				channel.pipeline().addLast(new RequestExecuteHandler(responseFuture));
				FullHttpRequest nettyRequest = createFullHttpRequest(headers);
				channel.writeAndFlush(nettyRequest);
			}
			else {
				responseFuture.setException(future.cause());
			}
		};

However the io.netty.channel.ChannelFutureListener interface is not contained in spring-web.jar so it's resolving level is DANGLING. That's why we still get this excption even when whole program mode is enabled.
I've also fire an issue on Soot https://github.com/soot-oss/soot/issues/1975 and hope they can deal with this convertion.

I definitely think the error message from Soot should be much more helpful, but it may require the classes to be dynamically loaded, as is now possible via this config: https://github.com/joernio/joern/pull/2839


Release/azure communication common/1.1.2

Description
Patch release for the azure-communication-common package - version 1.1.2.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:

[x] The pull request does not introduce [breaking changes]
[x] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[x] Pull request includes test coverage for the included changes.

API change check for com.azure:azure-communication-common
API changes are not detected in this pull request for com.azure:azure-communication-common


Prevent uninstalling in-use dependencies

If you try to uninstall a formula that's currently a runtime depends_on (i.e. not :build) for another installed formula it should refuse to uninstall. This should still be possible but require the use of uninstall --force.

It probably makes sense to block this on https://github.com/Homebrew/brew/issues/930 as using that will result in a far more performant solution than Formula#recursive_dependencies on everything installed.


Profile Picture size inconsitency

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
As we didn't define the particular dimension for the user profile, the picture should look like it is producing inconsistency while uploading different dimensions of the image.
 <%= image_tag user_profile_picture(@profile.profile_picture), alt: "project.name", class: "center-row search-usersearch-image" %>
https://github.com/CircuitVerse/CircuitVerse/assets/112646623/1deff9c7-cd0c-4519-9539-53b43a3b0bbe
Steps To Reproduce
->login
->upload the image
->again upload image(with different dimensions)
->see error Upload
Expected Behavior
It would be more ordered and clean if we define the fixed dimensions Irrespective of the size of the image the user uploading
Screenshots
No response
Device Information [optional]
- OS:
- Browser:
- version:

Additional context
No response
Are you working on this issue?
No

will work on this if approved @Malavi1

will work on this if approved @Malavi1

You can work on this

will work on this if approved @Malavi1

You can work on this


Fixed lost thin pool devicemapper docs

- What I did
Fixes #22817
- How I did it
Re-wrote docs missing and applied additional changes made since the docs were lost
- How to verify it
Read the docs :)
- Description for the changelog
First commit includes just what should be cherry picked into the docs branch; combination should go into master
Signed-off-by: Matt Bentley<EMAIL_ADDRESS>

While we're add it, could you download the image used at line 378
https://github.com/docker/docker/pull/22968/files#diff-52525eec69924c13d098f5d19b2d79adR378
![](http://farm1.staticflickr.com/703/22116692899_0471e5e160_b.jpg)

And add it to the https://github.com/docker/docker/tree/master/docs/userguide/storagedriver/images directory? (giving it a sensible name)

Thanks so much for doing this @mbentley! Couple of nits, but looking good
ping @vdemeester  - for reference; I asked Matt to use two separate commits, and keep the docker daemon -> dockerd rename separate, so that we can cherry-pick into the docs branch

Thanks for the nits @thaJeztah.  Updated the commits.  Let me know if I missed anything.

LGTM 🐼
Ok for the commits :angel:

awesome, thanks @mbentley!


[Question]: how dose llama_index work with Chinese text?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
how dose llama_index work with Chinese text?
in documentation, code is as follows:

as for my data is Chinese, i translate the entities into Chinese, however, it doesn't work. I can't get any output. If i use the default settings without changing the entities, there is an output graph.

so, does llama_index work with Chinese text well?

the Dosu assistant did provide some useful information!
Anything on the roadmap that would better support Chinese and/or other East Asian languages?


[SPARK-27355][SS] Make query execution more sensitive to epoch message late or lost

What changes were proposed in this pull request?
In SPARK-23503, we enforce sequencing of committed epochs for Continuous Execution. In case a message for epoch n is lost and epoch (n + 1) is ready for commit before epoch n is, epoch (n + 1) will wait for epoch n to be committed first. With extreme condition, we will wait for epochBacklogQueueSize (10000 in default) epochs and then failed. There is no need to wait for such a long time before query fail if there maybe some message LATE/LOST. In this PR, we make the condition more sensitive.
How was this patch tested?
update existing unit tests.

cc @jose-torres

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9584/
Test PASSed.

Test build #104248 has started for PR 24283 at commit 8c71b2f.

Test build #104248 has finished for PR 24283 at commit 8c71b2f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104248/
Test FAILed.

pending on fixing unit test failure

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9628/
Test PASSed.

Test build #104290 has started for PR 24283 at commit b2281df.

Test build #104290 has finished for PR 24283 at commit b2281df.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104290/
Test PASSed.

As Attila pointed out it contains unnecessary changes which maybe distracted me but at the first glance I don't see the main value.

There is no need to wait for such a long time before query fail if there maybe some message LATE/LOST.

Not sure if I understand the main reasoning. If one Kafka partition have some problem because that specific server is slower than other after 10 epoch just kill the query? If it's an intermittent problem not sure it's the right thing to do.

@gaborgsomogyi Thanks for your reply.  #23156 introduced a maximum queue threshold before stop the stream with a error. In #23156 , we used the same threshold for different queue, i.e. partitionCommits, partitionOffsets and epochsWaitingToBeCommitted. Generally, the size of partitionCommits and partitionOffsets grow much faster than epochsWaitingToBeCommitted. The stream may fail with 10 epochs if partition number is 100. However, we may wait for 10000 epochs before failure if partition number is 1 (if i understand correctly). Well, this is a harsh boundary condition. The main concern of PR is to split there two thresholds to make query execution more sensitive to epoch message late or lost. If you feel like 10 epoch is too sensitive in some  intermittent problem, we can relax this condition to 100 or other.

retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/9706/
Test PASSed.

Test build #104373 has started for PR 24283 at commit ba4d665.

I see the main intention now. I agree that the different queues are filled up with different speed and considered this when the configuration added.
I think the threshold is configurable which makes the actual implementation flexible enough to handle this situation. From my point of view additional fine tuning doesn't really help but makes this code more complex which has to be maintained.

@gaborgsomogyi Maybe we can merge these two configs, and I prefer to keep the epoch threshold config. Then there is no need to check partitionCommits and partitionOffsets again. At one level, this makes sense. These queues would not be unbounded and consume up all the memory easily.

Test build #104373 has finished for PR 24283 at commit ba4d665.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104373/
Test PASSed.

@gaborgsomogyi Could you please take a second glance?

cc @zsxwing

ping @gaborgsomogyi @zsxwing

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10134/
Test PASSed.

Test build #104826 has started for PR 24283 at commit d2c4296.

@gaborgsomogyi I have unified the two configs. We will only check late epochs, but not check epochBacklogQueueSize again. As said above, checking late epochs is enough to avoid using too much memory and avoid waiting for long time before query fail if there maybe some messages LATE/LOST.

Test build #104826 has finished for PR 24283 at commit d2c4296.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104826/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10136/
Test PASSed.

Test build #104828 has started for PR 24283 at commit 7cfd180.

Test build #104828 has finished for PR 24283 at commit 7cfd180.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/104828/
Test PASSed.

ping @gaborgsomogyi

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10480/
Test PASSed.

Test build #105197 has started for PR 24283 at commit a489327.

Test build #105197 has finished for PR 24283 at commit a489327.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105197/
Test FAILed.

retest this please

Test build #105201 has started for PR 24283 at commit a489327.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10487/
Test PASSed.

Test build #105201 has finished for PR 24283 at commit a489327.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105201/
Test FAILed.

retest this please

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/10524/
Test PASSed.

Test build #105243 has started for PR 24283 at commit a489327.

Test build #105243 has finished for PR 24283 at commit a489327.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/105243/
Test PASSed.

Build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/15705/
Test PASSed.


[customer-account] - add extension metadata to standard api

Background
This PR adds typings for adding extension metadata to the standard api of customer account ui extensions as part of
https://github.com/Shopify/core-issues/issues/49460
Implementation: https://github.com/Shopify/customer-account-web/pull/1567
Divergences from checkout's specification:

No editor property
Blocking removed from list of capabilities.
Version is undefined for local dev extensions, but returns draft version (0.0.0) when installed on development stores.

This PR also restructures the folder structure, and collocates some types (i18n, locale for example).
Checklist

[x] I have :tophat:'d these changes
[x] I have updated relevant documentation

/shipit

/shipit


[BUG] colorMapPicker cannot be shown in smartphone

Bug Description
In smartphone, colorMapPicker is not shown
Bug Reproduction
Open colormap picker in smartphone
Bug Impact
No response
Bug Screenshots

Bug References
No response
Bug System Information

OS: iOS

Even this DataCardInfo popup cannot be shown properly.


ART-1434: shst matching robustness improvements

Ticket
I was seeing some failed matches, even when I'd expect a good match just from looking at the trace, and by running OSRM matching directly. There were quite strict constraints on post-processing the OSRM output, and so I have loosened those off a little bit to give us more robustness in the output.
I will comment on the specific individual changes.
For testing, you can use this trace and run the matching on this branch vs the main branch to see the difference:
docker run -it --rm -v ${PWD}:/work --env SHST_TILE_URL=https://rr-binaries.s3.eu-west-2.amazonaws.com/tiles/ shst-image match /work/test.geojson --tile-source=osm/planet-240104 --search-radius=25 --follow-line-direction

{
  "type": "FeatureCollection",
  "features": [
    {
      "type": "Feature",
      "properties": {},
      "geometry": {
        "type": "LineString",
        "coordinates": [
          [
            -0.2094627999999993,
            52.5977402
          ],
          [
            -0.20944320000000027,
            52.59764479999999
          ],
          [
            -0.20948560000000072,
            52.598114
          ],
          [
            -0.21030939999999987,
            52.598419199999995
          ],
          [
            -0.21161750000000076,
            52.59839629999999
          ],
          [
            -0.22327850000000107,
            52.6007805
          ],
          [
            -0.22510950000000032,
            52.60230259999999
          ],
          [
            -0.22987429999999948,
            52.60523220000001
          ],
          [
            -0.23804459999999988,
            52.60895539999999
          ],
          [
            -0.24569849999999904,
            52.61297230000001
          ],
          [
            -0.24825579999999914,
            52.614566800000006
          ],
          [
            -0.24752409999999983,
            52.614574399999995
          ],
          [
            -0.24688100000000002,
            52.61516189999999
          ],
          [
            -0.24682960000000137,
            52.6155472
          ],
          [
            -0.24776610000000016,
            52.6158524
          ],
          [
            -0.24778540000000043,
            52.61575319999999
          ]
        ]
      }
    }
  ]
}

Thanks @edmabon for clear comments!


Develop

Fixed last pull request to be aligned with the current hierarchy of packages and mvn url.

Merci John


[publishing] Add support for the new maven central publisher API

Expected Behavior
Sonatype is changing how artifacts are uploaded with new accounts. See https://central.sonatype.org/register/central-portal/ and https://central.sonatype.org/publish/publish-portal-api/
The new API uploads a single zip file containing a full publication in one go and as a result the current method doesn't work anymore. Would be nice to have built-in support for the new API in the maven-publish plugin.
Current Behavior (optional)
No response
Context
https://github.com/vanniktech/gradle-maven-publish-plugin/issues/722
KotlinLang slack discussion

This is something that we are discussing internally and probably will discuss with Sonatype.
An official plugin from Sonatype would be the best option since they own the protocol which is no longer based on Maven deploy.
If an official Gradle plugin adds that support, it would also make sense to have it outside of the Gradle core plugins, since Sonatype could change their protocol anytime.

Not sure if this is still on the table for Sonatype but keeping a similar API would be quite handy:

no need to change every library using Gradle/Maven out there
could parallelize the uploads and/or upload from different machines (for KMP for an example)

I get that the current implicit staging repos and the split repos are an issue but sounds like this could be fixed transparently for end users by wrapping the file uploads in creation/close API calls?
While I understand the rationale for a new plugin, adding yet another publishing plugin is going to add more complexity to an already complex landscape.
Anyways, I understand this is ultimately not in Gradle hands but these were my 2 cents! Thanks for reading!


/livewire/update 405 when save some html tag

Package
filament/filament
Package Version
v3.2.0
Laravel Version
v10.0.0
Livewire Version
v3.0.0
PHP Version
8.1
Problem description
route /livewire/update 405 when save some html tag, in production mode, how can i fix this
Expected behavior
/livewire/update return 200
Steps to reproduce
fill input with some html code like
Reproduction repository (issue will be closed if this is not valid)
https://github.com/bofoiii
Relevant log output
No response

<title>405 Method Not Allowed</title>

405 Method Not Allowed

hws

No details, no reproduction

how to fix that ?


add OpenAI Classifier

Issue number: #85
Summary
Changes
Added an OpenAI classifier as an inbuilt feature. Updated the relevant documentation to reflect the new functionality. This change eliminates the need for users to create custom classifiers by enabling seamless integration of OpenAI's classifier.
User experience
Before: Users were required to write custom classifiers to integrate OpenAI's functionality, adding additional development effort and complexity.
After: The OpenAI classifier is now available as a built-in feature, improving DX.
Checklist

[x] I have performed a self-review of this change
[x] Changes have been tested
[x] Changes are documented

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Hi @Rajaniraiyn
Thanks a lot for your contribution 🙌
We will review the code, test it and come back with feedback, if any.

@leonardo2204 , feel free to contribute and add function call for OpenAi agent (both TS and python :) )

@brnaba-aws https://github.com/awslabs/multi-agent-orchestrator/pull/96

Hi @brnaba-aws
Thank you for your response earlier. I just wanted to follow up and check if there’s any update regarding the review. Please let me know if you need any additional information or clarification from my side to assist with the process.

@Rajaniraiyn Thanks a lot for your work.
I'll merge the pull request now.
Please don't hesitate to share the contribution on you have made to the framework on social media.
It is making it even better 🚀 ❤

@Rajaniraiyn Do you think you can contribute to the same classifier for python? :)

@brnaba-aws https://github.com/awslabs/multi-agent-orchestrator/pull/101


[BULK] - DocuTune remediation - Sensitive terms with GUIDs (part 47)

Applying sensitive terms with GUID changes as part of Content SFI and outlined in Overview - Writing content securely - Platform Manual. Changes are part of the Microsoft-wide SFI effort. Point of contact: @CelesteDG
DocuTune v<IP_ADDRESS>
CorrelationId: ac15aa43-4e2b-437f-ab1c-fdd7e79cd4db
#docutune

#docutune-review

LGTM


Limit the sdk.Msgs that can be included in a governance proposal

To limit the functionality of the gov module, we can add an additional check that only allows specific sdk.Msgs to be attached to a proposal.
so far, it seems that we need:

[ ] community spends
[ ] IBC channel unfreezing
[ ] software upgrades
[ ] param changes

another alternative would be simply to disable the new gov module, and use the legacy one, but I'm unsure how long the legacy one will be supported by the sdk.

can close since instead of limiting to the above, we are relying on the existing limitations in the cosmos-sdk, and have removed place text proposals or empty proposals


Move the transparency log feature out of experimental

Some things I want to figure out first:
How should this be configured? Opt in vs. opt out?
Proposal: Upload send by default for public images (where we can tell), but opt-in for private images.
How should verification be configured?
Is it a failure to verify if the signature is not present?
Should we check by default?
Proposal: Check by default for a public image., but warn if the signature is not present, but still pass. Opt-in check for private images. If the opt-in flag is passed for public or private, fail if the signature is not present.

I think we're close here with offline bundling. We'll still want to be careful about adding entries to the log, but verification can happen if there's a bundle, whether or not we're in experimental.

@dlorenc is this issue still valid or should we close it out? It was added to the GA plan (unclear as to why), so I wanted to verify before taking it off the plan of record.

So this basically means that we will remove the "experimental" env var requirement from cosign once Rekor/Fulcio are stable. Think it makes sense as a GA requirement since GA assumes the services are reliable not experimental.

I feel like this is covered between https://github.com/sigstore/cosign/pull/2387 and https://github.com/sigstore/cosign/issues/2376 🤷

sgtm, will close!


bau: add COOKIE_NOT_FOUND_PAGE page source

https://github.com/alphagov/verify-frontend/blob/7379b463065c0d9f52413daa012c3fbc2e1ea2f8/spec/features/user_visits_start_page_spec.rb#L38
shows that there is a COOKIE_NOT_FOUND_PAGE page source. This commit adds it to
the feedback_source_mapper.rb. It doesn't map to any url as no cookies means no
session, so the 'Your session has timed out' text should show after the
submission of feedback. This is reflected in the new test added in
user_visits_feedback_page_spec.rb.
@oswaldquek

What's the status of this PR?

Sorry this should have been closed. It's already been deployed.


feat: implement dropdown menu for status bar help button

What does this PR do?
Adds dropdown menu for status bar help icon
Screenshot / video of UI

What issues does this PR fix or reference?

Fix #9407.
How to test this PR?

[ ] Tests are covering the bug fix or the new feature

Tests have not implemented yet.

@Firewall Updated the PR and screenshot.

@Firewall the issue was not telling to remove the other icons from the status bar

Thanks @dgolovin, this is a lot clearer in my opinion.
Seeing the content this way... Do you think we should move the Create Github Discussion and Share your feedback to under the Report a Bug instead? The way it is now they look like sub-items of Learning and that is not the intention.

IMHO I would separate the removal of other icons from this PR. Scope is too big.
First: add things/rework only help and get feedback, second: remove things, not do too many things at once.
The title of the issue or the PR is to move help page to a dropdown, not removing bulb and share feedback icons, so as a reviewer, it's not doing what is said, so I'm requesting changes.
About the content, it looks like the original intent was to reduce the number of clicks while here I would do extra clicks.(if bulb and report feedback direct action icons are moved)
But my point is that to click on useful links, I would have to move the cursor far way from the first click it looks like it'll do the opposite of what it was describing (looks like import links are on the top while clicking on a menu from the bottom I expect the opposite)

Ok, I created a sperate ticket for removing them: https://github.com/podman-desktop/podman-desktop/issues/9900
@benoitf I'll respond to your comments on that issue too.

looks like import links are on the top while clicking on a menu from the bottom I expect the opposite)

Now that I saw the new submenu in action. I agree. @dgolovin can we try to reverse the list and see if that helps gets the most important actions to stand out more?
The list also feels very "tall" for not that many options. Maybe we can reduce the height of each item a bit?

suggestion (non-blocking): I don't think we need the padding on the left for the dropdown items since we have a clear separation, IMO it feel a bit odd to have a padding

The list also feels very "tall" for not that many options. Maybe we can reduce the height of each item a bit?

@Firewall trying to reduce the size might lead to inconsistency with other dropdowns, maybe we should see with @ekidneyrh on the UX call today ?

@dgolovin I brought this as a topic to the UX call and we got a bunch of feedback from the team.

The options for Matrix and Libera can be removed, they are not used.
For Discord and Slack we we use the icons for the respective services? It was also suggested we can remove the words Join and channel. So it would just be :discord-icon: #podman-dekstopand:slack-icon: #general
The learning links feel out of place and are duplicate of the content on the learning center on the dashboard. As a result we can remove them.
There was a feeling of having too many options to choose the same thing. Report a bug, Create Github Discussion and Share your feedback. It was suggested we delete the Create Github discussion option for now as this will be an option in the Share your feedback dialog soon.
It was also voted for now to not remove the icons in the statusbar so I will close the other ticket as wont-do

Let me know if you have any questions.

@Firewall After removing everything you asked it turned to image below.
Couple notes:

I had to remove Learning, because it would have only feedback item
Removed indent for Community, as @axel7083 suggested
Cannot use icons for discord and matrix due to lack of support custom icons for DropdownMenuItem, so I just left names for discord and matrix.

Cannot use icons for discord and matrix due to lack of support custom icons for DropdownMenuItem, so I just left names for discord and matrix.

AFAIK we can use custom icons using fonts but as we can use font awesome icons easily, discord icon is part of
https://www.npmjs.com/package/@fortawesome/free-brands-svg-icons
https://fontawesome.com/icons/discord?f=brands&s=solid

Cannot use icons for discord and matrix due to lack of support custom icons for DropdownMenuItem, so I just left names for discord and matrix.

AFAIK we can use custom icons using fonts but as we can use font awesome icons easily, discord icon is part of https://www.npmjs.com/package/@fortawesome/free-brands-svg-icons https://fontawesome.com/icons/discord?f=brands&s=solid

@benoitf Would follow up issue work here?

yes for icon as follow-up, not for unit tests as follow-up

Comment from @Firewall (https://github.com/podman-desktop/podman-desktop/pull/9867#issuecomment-2476450314)

The options for Matrix and Libera can be removed, they are not used.

Tested the lasted version of the PR, I am getting the following

Discord and Matrix ? I thought we would keep the slack and discord ?

Good catch @axel7083, will fix that. Thank!

@Firewall Final version

@dgolovin Looks great! So close :D but I think for Discord the channel is also called #podman-desktop and not #general
Fixed.

I added the tests to verify if all the items are showed up in menu.

Nice improvement in GUI . Thanks @dgolovin

Re-based on top of origin/main.


[camera][android]Occasionally crashes while recording

Fatal Exception: java.lang.NullPointerException: Attempt to invoke virtual method 'android.hardware.camera2.CaptureRequest android.hardware.camera2.impl.CameraDeviceImpl$CaptureCallbackHolder.getRequest(int)' on a null object reference
       at android.hardware.camera2.impl.CameraDeviceImpl$CameraDeviceCallbacks.onResultReceived(CameraDeviceImpl.java:1919)
       at android.hardware.camera2.legacy.CameraDeviceUserShim$CameraCallbackThread$CallbackHandler.handleMessage(CameraDeviceUserShim.java:323)
       at android.os.Handler.dispatchMessage(Handler.java:106)
       at android.os.Looper.loop(Looper.java:197)
       at android.os.HandlerThread.run(HandlerThread.java:65)

@suli1
Please provide flutter doctor -v and a minimal complete reproducible code sample that shows the issue along with devices on which you are seeing this crash.
Thanks.

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.

@suli1 did you figure out what the problem was?

@suli1 did you figure out what the problem was?


Update the troubleshooting guides

Addresses #3889
I have added content to the troubleshooting guides based on my self-study as well as in-person discussions over the last few months. The docs need to be edited extensively, but wanted to get initial feedback to ensure I am on the right track.
Update: Ready for peer review

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/b8b0b7f047b032596fddfd55d370784135b530fc/

v2.2/cluster-setup-troubleshooting.md, line 244 at r4 (raw file):
Previously, Amruta-Ranade (Amruta Ranade) wrote…
@BramGruneir Thoughts?

? Just means that there is no info sent yet.  If a node was just added, it takes a min before we collect enough latency data for populate it.
If it is there permanently, it means there is most likely a network partition.

v2.2/cluster-setup-troubleshooting.md, line 326 at r4 (raw file):
Previously, Amruta-Ranade (Amruta Ranade) wrote…
@piyush-singh Need help filling in the "??" values in the tables.

I'm not sure we would be able to calculate the utilization rate of disk bandwidth. We'd need to know the denominator: what is the max throughput? This varies a great deal based on storage type, manufacturer, interface type, etc. We can highlight unusually high values using the metrics "Disk Read Bytes" and "Disk Write Bytes".

v2.2/cluster-setup-troubleshooting.md, line 244 at r4 (raw file):
Previously, BramGruneir (Bram Gruneir) wrote…
? Just means that there is no info sent yet.  If a node was just added, it takes a min before we collect enough latency data for populate it.
If it is there permanently, it means there is most likely a network partition.

A network partition would be indicated by a 'X', right?

v2.2/cluster-setup-troubleshooting.md, line 244 at r4 (raw file):
Previously, BramGruneir (Bram Gruneir) wrote…
Yes, you're right.  It normally is an X.  So ? just means not enough time has passed to gather enough data.

Excellent. Thanks Bram!

v2.2/cluster-setup-troubleshooting.md, line 327 at r4 (raw file):
Previously, bdarnell (Ben Darnell) wrote…
"IOPS in progress" is a count, not a duration. From our current metrics, you want to watch the "disk read bytes" and "disk write bytes", but we don't know what the max would be for your deployment.

Okay..so should I remove this as well?

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/e07ddaad45f7501fc89a0511548f97f1e089501c/

This is great, just wanted to leave some notes for future updates.
We need troubleshooting guides for:

Upgrading - Why did the node get a new node number when I restarted it?  What happened to all of the replicas on it?  Etc.
Adding new nodes - How many can I add at once?  What's the best practice around this?
Removing nodes - what's the process, what do I do if something goes wrong?
Replication Factors - Why choose 5x vs 3x.  What's the advantage/disadvantage
Rebalancing - How does rebalancing work? What is a lease, what does it do?  etc.
Certs - This probably deserves its own page, so many ways to mess up
Common Error and Warning Messages in Logs - This could just be a growing list to explain the most common messages

Of course, these should all be living documents!

@sploiselle Thanks for the review! I agree with you about the need to break up the cluster troubleshooting content into smaller pages. In fact, @jseldess and I discussed it just yesterday. I also need to add the content Bram pointed out, as well as some more issues I discovered (like secondary indexes for partitioned tables).
However, the overall feedback is to merge this content and make it available to our users, and then open a follow-up PR to address the requested changes. Would love your help in structuring the content in the follow-up PR :)

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/63cab031d9a7634fbb227c6fe3c8eb2573852728/

@jseldess Ready for your review :)

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/63cab031d9a7634fbb227c6fe3c8eb2573852728/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/66edb15f3134961ecfbfec7697e0909e97986ec6/


Support locale parameter on recognize receipt methods

locale == language?
Optional -> seems like it is to improve the accuracy of the results. (like in TA with language parameter)
Open questions:

How does locale affects the returned results? Is there a locale to fields mapping spec?
What is the standard called that locale is using?

Service team is asking if we could use CultureInfo or an extensible enum for the locale values

Going with strings for now, following what we did in TA.
Having an extensible enum means every time there is a new locale, library needs to update and ship.
Culture info not supported by other languages and not right fit here


Properly respect platform-specific connectors

Motivation
Until https://github.com/DataDog/integrations-core/pull/2322, sqlserver dependency environment markers were not in sync and therefore adodbapi was being install everywhere for tests even though it is Windows-only. Now that it isn't (matching the Agent), an import error occurs when it should not as pyodbc is an equally valid connector.
Additional Notes
As we don't yet offer official support for linux, no changelog entry is needed since this only fixes our ci.

Codecov Report

Merging #2323 into master will decrease coverage by 28.26%.
The diff coverage is 44.44%.

@@             Coverage Diff             @@
##           master    #2323       +/-   ##
===========================================
- Coverage   85.89%   57.63%   -28.27%
===========================================
  Files         196        4      -192
  Lines       13520      642    -12878
  Branches     1454      106     -1348
===========================================
- Hits        11613      370    -11243
+ Misses       1489      228     -1261
+ Partials      418       44      -374


these files were missing copyright headers

the needless noise, in the ./gradlew build output, I could take it no longer

NetflixOSS » SimianArmy » SimianArmy-pull-requests #20 SUCCESS
This pull request looks good

SimianArmy-pull-requests #192 SUCCESS
This pull request looks good

Fantastic!


Fix shell extensions

There's a way to show file extensions within 'Finder' assuming that was the problem.
https://support.apple.com/kb/PH19072?locale=en_US

thanks


Provide secure storage and private keys

Task Description
Secure storage of keys is essential in real-world products, we do not want private keys to be accessible and it is common requirement to have secure storage for important bits.
A way to do it is to use Linux's Trusted Platform Module (TPM) which will use hardware whenever available. Another is to use Intel's gSSO (https://01.org/gsso) which abstracts that and bit more.
We should investigate which approach to use. gSSO looks easier to use, but brings more dependencies. We may check with @ereshetova and @jlaako which know them very well to see how to proceed.
What we should keep in mind for Soletta:

how to make it work with posix main loop
how to make it work on micro-linux, that is, single process, without a daemon. Maybe would need different implementation (ie: sol-...-impl-gsso.c)?
should we expose this as a different C API (ie: sol-sso.h or sol-securestorage.h...)?
should we expose this as a set of different node types? Or have existing node types to support them, like oauth and persistence.

Relevant contacts:

@ceolin (did oauth)
@edersondisouza (did persistence)
@jlaako (gSSO)
@ereshetova (gSSO)

Please take into account that we are not only talking about storing private keys, but there maybe other credential types that needs to be stored, so the solution should be flexible enough to take that into account.
Also, a good SSO solution is not just a credential storage (this is easy to make), but should provide mechanisms to perform operations on credentials it securely stores without revealing the credential itself to an application. It should support a way to define what operation on what credential a certain app can do. In that case, if a certain app is found out to be malicious, its access to operations on credential can be revoke without the credential itself being compromised (and therefore re-provisioned). This a requirement that greatly enhances security.

Great addition @ereshetova.
My only doubt is about the micro-linux case. Ideally in this mode we have a single process userspace (a /init that is the app, this is made easy with Soletta). In this case we try to provide the same API and a feature subset that is useful. Then what to do with gSSO in that case?
We can have multiple implementations, so we could require the TPM in the kernel and use parts of that interface to provide the minimum feature subset.
From the IoT products I've investigated, the most common operations are secure storage of small tokens (ie: some services that do not use modern authentication and requires user to type actual user/pass), verifying signatures (ie: to verify update is correct), HMAC and signing (basically oauth and OIC). Then yes, it matches what you say about not only storing the credentials (first case), but also doing the OAuth or OIC pairing processes without retrieving the keys to the application.

I think when it comes to size, let's ask @jlaako if he has ideas on how to scale down gSSO for that case.

ok, figures so you know: Soletta with FBP and OIC is around 400kb total, statically linked with musl-libc and stripped. A BLE heart-monitor using FBP is around the same size. That is the userspace size, single binary (/init).
Ideally we shouldn't increase much that size if all we need to do is oauth  or secure storage of some key.

gSSO depends, as the name says, on glib, gobject and gio (gdbus) that's where most of the footprint comes from. For some of the plugins also libgcrypt+libsoup(gnutls) are needed...

how cumbersome would be to change gobject/gio/libsoup to libsystemd? It provides easy to use sd-bus.h  and is already required for systemd itself.
Glib is already required for other daemons, most of our comms. But none of them require gobject/gio/libsoup.
I'd not say this is a blocker, but something to consider for the future.

Just almost complete rewrite, so at least a man year effort.
I've been implementing this framework already twice, once for Qt and second time for glib. Not really keen doing third reimplementation.
libsystemd is anyway controversial and Linux-specific. gSSO is not Linux-specific but works also FreeBSD, etc.
Does libsystemd also prove a HTTP stack with proper proxy traversal and TLS/SSL support? This is necessary for OAuth.

I agree that implementing it yet-another time will be painful. As I said, Glib itself is not an issue since connman, ofono, bluez all needs it. But they only need the basic Glib. Actually these use a Glib-DBus layer that is much, much easier to work with than GDBus, worth checking with @holtmann or see the code.
As for http stack, AFAIU they don't, people rather use libcurl. Which we need in Ostro due swupd and soletta anyway.

One reason we use GDBus is that it nicely  supports P2P D-Bus which is essential requirement for gSSO. We prefer not to go through dbus-daemon for security reasons, instead applications talk straight to gsignond using the client library. Another reason is nice integration for GLib's complex data structures over D-Bus since we have lot of things like nested variant maps - "a{sv}" inside "a{sv}" inside "a{sv}".
Which brings me to another topic that would double the reimplementation effort. So far we got the libgsignon-glib client library fork from the original (Qt-based) SSO project and we are still C API/ABI compatible with the original (which is why for example Elementary OS is using gSSO). Moving away from GDBus and GObject would mean rewriting also the client library.
Overall, gSSO footprint is small when running and we anyway already have gobject and gio installed on Ostro images.
If you want something small for single-process systems, gSSO is anyway wrong choice and you should go with something else an use hardware based security implementation like TPM straight.
Canonical is maintaining the original SSO implementation and we are maintaining the gSSO. Maintaining yet another implementation would also increase the effort considerably. So certainly I don't want to even think about such.
If someone else wants to take and reimplement the entire thing, I'm fine with that, but I want to stay away from such.

Well, P2P D-Bus is basically just a matter of opening the right path. AFAIU all libraries can do that.
The complex data structure is also used a lot, mostly all standard ObjectManager or Properties interfaces uses them, then they are well supported in all libraries I've seen.
For the client library, at least in Soletta we'll have to do it ourselves as we use sol-bus.h and internally it is sd_bus.h. So doesn't matter much to us.
To wrap up the summary, Soletta should:

use gSSO server as is
offer a SSO "-impl" that talks via DBus using sol-bus.h
offer an "-impl" that talks to TPM in micro-linux case (low priority)

@ereshetova is checking if kernel's crypto API could use a digest key right from TPM. If this is possible, then by using our sol_message_digest and kernel crypto we could at least do oauth with our current code without exposing the key to userspace. Then it would work nice on micro-linux.


Out of memory on EC2

I realize this is very similar to Issue #248 but I am running out of memory on the lesson1-pets notebook on a fresh install of p2.xlarge on EC2 with everything updated and the most recent commit of course-v3. I was able to run this notebook without any issues on a p2.xlarge EC2 instance a few weeks ago but something has changed in the meantime.
I have since tried using different versions for bs even dialing it all the way down to bs=4 but am still getting an out of memory error. In the lesson1-pets notebook I can run through the first set of examples using resnet34 but after that I am not even able to download the resnet50 model from pytorch before getting the error.

I think I was able to figure this out and it might be due to OS type on EC2. I used several different sizes and setups of ubuntu machines and kept running into the same out of memory error. I switched to Amazon's AMI for p2.xlarge and did not have this issue again.


how to prevent Connection was broken?

I have a code I want to run to wait for connection:
const { SRTReadStream } = require('@eyevinn/srt');
const srt = new SRTReadStream('<IP_ADDRESS>', 1235);
srt.listen(readStream => {
  console.log("Client connected");
  readStream.on("data", (e) => {
    console.log(e)
  })
});
srt.on('end', () => {
  console.log("Client disconnected");
});

console.log("Waiting for client to connect");

I use ffmpeg to send:
ffmpeg -async 1 -vsync 1 -re -i '/home/iam/Videos/1.mp4' -c copy -t 00:00:05 -f mpegts srt://<IP_ADDRESS>:1235?pkt_size=1316
But even with the treatment 'srt.on('end'', when disconnecting the FFMPEG, this error appears:
//home/iam/Documents/pipetest/node_modules/@eyevinn/srt/src/srt-stream-readable.js:136
      const buffer = this.srt.read(this.fd, bytes);
                              ^

Error: srt_recvmsg: Connection was broken
    at SRTReadStream._readSocketAndPush (/home/iam/Documents/pipetest/node_modules/@eyevinn/srt/src/srt-stream-readable.js:136:31)
    at SRTReadStream._read (/home/iam/Documents/pipetest/node_modules/@eyevinn/srt/src/srt-stream-readable.js:171:10)
    at SRTReadStream.Readable.read (node:internal/streams/readable:483:10)
    at flow (node:internal/streams/readable:1005:34)
    at resume_ (node:internal/streams/readable:986:3)
    at processTicksAndRejections (node:internal/process/task_queues:83:21)

I want the connection to be keep wait infinite,  that I can connect and stop ffmpeg at any time,That when disconnecting it wait for new socket connect and wait for a new entry to continue.
Can you help me?

Did you get this resolved ?

Did you get this resolved ?
Unfortunately not!

Did you get this resolved ?

Unfortunately not!

I migrated to srtgo (Go language project)

problem persists


FIX-#2850: use modin.pandas.Series instead of pandas.Series for where func

What do these changes do?

[x] first commit message and PR title follow format outlined here

NOTE:  If you edit the PR title to match this format, you need to add another commit (even if it's empty) or amend your last commit for the CI job that checks the PR title to pick up the new PR title.

[x] passes flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] passes black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
[x] signed commit with git commit -s
[x] Resolves #2850
[ ] tests added and passing
[x] module layout described at docs/development/architecture.rst is up-to-date

@dchigarev let's merge


Changing Font Size

Can't change font size on an iPhone.

I saw this was mentionned mentionned in the "readme". But it is a real problem on IOS, any forecast on the development of this feature?


Example code is not working

When I try to run the example code, this is what I get:
on this line:
 displayItemFn: (dynamic item) => Text(
        item['name'] ?? '',

Exception has occurred.
NoSuchMethodError (NoSuchMethodError: The method '[]' was called on null.
Receiver: null
Tried calling: )

Strangely now it is working.


Removed ycmd-request[-deferred]

Now that we've got properly packaged and up-to-date request and request-deferred, we can get rid of our own copy.

Cool, thanks. I proposed a PR at MELPA to remove the third-party folder from recipe https://github.com/melpa/melpa/pull/3808
Will merge this when they merged the PR


Match the documentation for port_forward args

Hi there 👋
I've been using this framework recently as a dive into making our micro-service development easier. Love what you all have build!
Just addressing a small change in the documentation. It seems "link_text" doesn't work and the "name" does. Just want to see if this was a problem. For example:
k8s_resource('cool-service',
  port_forwards=[
    port_forward(8080, container_port=8080, name='rest-api'),
    port_forward(8081, container_port=5858, name='node-debugger')
  ],
  resource_deps=['cool-service-db']
)

Thanks for all the hard work!

lol, i think this feature hasn't officially launched yet, so i'm glad you found it :joy:
@maiamcc do you know what the expected api here is?


Ceph dashboard osd usage

Not entirely happy with the way Grafana shows the histogram so if anyone has any suggestions feel free to let me know.
There is also a bug with the threshold colouring. It doesn't show up until you edit the panel and change something appearance related like opacity.

Updated the graphs to look like this:

Unfortunately it seems that the colouring is broken in grafana. Additionally it appears that in the histograms you can no longer label the y-axis

@technowhizz are you happy to merge this?


Implement dplyr::rowwise() functionality using DataFrames and ByRow()

So in dplyr, I can group data into individual rows using rowwise() and functions will compute results for each row.  Here is an example:
library(dplyr)
library(purrr)

generate_random_strings_vector <- function() {
  colors <- c("red", "blue", "green", "yellow", "orange")
  sample(colors, sample(1:5, 1), replace = TRUE)
}

df <- tibble(
  id = 1:500
) |>
  mutate(colors = map(1:n(), ~generate_random_strings_vector()))

df |>
  rowwise() |>
  mutate(number_of_colors = length(colors))

If I want to do the same in Julia using DataFrames.jl, I'd do it like this:
using DataFrames, Random

function generate_random_strings_vector()
    colors = ["red", "blue", "green", "yellow", "orange"]
    return rand(colors, rand(1:5))
end

df = DataFrame(id = 1:500)
df[!, :colors] = [generate_random_strings_vector() for _ in 1:nrow(df)]

transform(df, :colors => ByRow(length) => :num_colors)

Does it make sense to have something like that in TidierData.jl?
@chain df @rowwise() @mutate(num_colors = length(colors))

Thanks for bringing this up, @christophscheuch. This use case has definitely been on my mind.
There are a couple of things that make this tricky (but Fei definitely not impossible) to implement.
One of them is that we need to use metadata fields within the data frame to keep track of whether a data frame is currently in rowwise mode. This is somewhat similar to @group_by except that grouping is already tracked within DataFrames.jl whereas rowwise is not. Once this ability is implemented, we need to make all other macros aware of it and make sure that it plays nicely with grouping and ungrouping.
I'm saving this for a later update but agree we should add it.


Add --listen option and filter announcements based on address family

The new --listen (ipv4|ipv6) option will enable a BGP listener on the --ipv(4|6)-local-address.
The new --limit-announce-to-afi option (not sure about the name here, feel free to change) will configure an appropriate family for the neighbor, preventing announcement of IPv4 prefixes to IPv6 neighbors, and vice versa.
The patch also includes a fix for the announcement receiver to correctly parse a special case of eor update.

Thanks for the PR! :)
I would have liked this to be multiple PRs. There are multiple, IMHO distinct, changes in here and they are better handled individually. There's also a sneak upgrade of ExaBGP, nothing wrong with that but I like it explicit :)
For the listening part, I think this should be made default as I don't see a reason not to. ExaBGP documentation is a mess but from what I can make out of https://github.com/Exa-Networks/exabgp/blob/3f99e80985d330ccf24f210de72f5847fa6eeb8c/lib/exabgp/reactor/loop.py#L148 it looks like there is a per neighbor listen parameter, so instead of injecting env vars, just add some variables to https://github.com/plajjan/vrnetlab/blob/master/vr-bgp/exabgp.conf.tpl and have our ipv[46] address sent in.
I think --limit-announce-to-afi should be default. I'm not even sure there should be an option to disable it.... personally I dislike carrying AFI over "wrong" transport. vr-bgp is still in early life and I think it's easier if we just lock it down a bit to how we use it. Makes the whole behaviour easier to explain, understand and work with. If someone wants wrong-AFI-transport in the future I'd rather have a separate change to enable that.

OK, I'll split to multiple PRs :)
Re: listening: indeed, didn't know about the config parameter. Will change to default and remove option.
Re: AFI announce: makes sense. Will change default and add new option --allow-mixed-afi-transport to override.


Adding minimized v1.3 AWS docs to 1.4 Kubeflow site branch

Adding v1.3 AWS docs to 1.4 Kubeflow site branch for most up-to-date content. This content has already been merged into master, and there is a PR (#3174) for merging it into the v1.3 branch.

/approve
/lgtm
approving based on #3169


Publish a step that allows the user to re-install the bundler gem based on a Gemfile.lock

This step basically checks if the current machine bundler gem is able to operate in the project. If the check fails, it looks for a Gemfile.lock to extract the Bundled version, and install that version in the machine.
This was created to fix a problem when running projects bundled with bundle version 2.x on macOS machines.
New Pull Request Checklist
Please mark the points which you did / accept.

[x] I will not move an already shared step version's tag to another commit
[x] I read the Step Development Guideline
[x] I have a test for my Step, which can be run with bitrise run test (in the step's repository)
[x] I did run bitrise run audit-this-step (in the step's repository - note, if you don't have this workflow in your bitrise.yml, you can copy it from the step template.)
[x] I read and accept the Abandoned Step policy

This is the fix for the PR #1899

Thank you for the new step @igorcferreira !
LGTM, Happy Building! 🚀


Training problem

@ConnorJL Thanks for the great work.
Unfortunately, I found out my training using OpenWebTextCorpus is too slow even for 117M model.  The cross entropy loss function decreases rapidly before 10k steps using a batch size of 64. After that it stayed around 3.0. Is this a known phenomenon or is it a dataset problem? I found the loss function in model_fns is not shifted. It should be loss_batch = tf.nn.sparse_softmax_cross_entropy_with_logits(logits=output["logits"][:, :-1],labels=features[:, 1:])  , am I right?

Unfortunately, this is a known phenomena, and I haven't been able to fix it. I perform the shifting of the labels in the input function (it's done in an ugly way, I'd do it differently now, but the effect should be the same). If I didn't shift, the model should converge to 0 loss very rapidly since it's just copying the input. I'm very open to any other ideas of what may be causing this problem. Maybe it is the dataset after all?


[BUG] Global variable detection raised error message

Describe the bug
A global variable create with a Set Test Variable wasn't detected and this variable wasn't see as defined
Steps To Reproduce
*** Keywords ***
Operator Triggers A Backup
    ${backup}       Post   ${URL}
    Set Test Variable       ${BACKUP_NAME_TEST_VAR}     ${backup}

The Backup Is Successfully Created
    [Arguments]     ${backup}=${BACKUP_NAME_TEST_VAR}
    ${backup_list}      Get   ${URL_LIST}
    Should Contain      ${backup_list}      backup/${backup[0]}

The error message is raised
Variable '${BACKUP_NAME_TEST_VAR}' not found.robotcode.namespace(VariableNotFound)
Expected behavior
The error message should not be raised as the variable is "globaly" defined
Screenshots/ Videos

Logs
Variable '${BACKUP_NAME_TEST_VAR}' not found.robotcode.namespace(VariableNotFound)
Desktop (please complete the following information):
Version: 1.93.1
Commit: 38c31bc77e0dd6ae88a4e9cc93428cc27a56ba40
Date: 2024-09-11T17:20:05.685Z
Electron: 30.4.0
ElectronBuildId: 10073054
Chromium: 124.0.6367.243
Node.js: 20.15.1
V8: <IP_ADDRESS>-electron.0
OS: Linux x64 6.8.0-45-generic snap

This is not a bug, it works as it should. This is because RobotCode performs a static code analysis and not a call analysis. It is not guaranteed that the keyword “Operator Triggers A Backup” will be called anywhere, anytime. And also not that exactly this variable ${BACKUP_NAME_TEST_VAR} is meant, maybe it is also set in some other test/keyword and it means something completely different.
What you should always do, also to find all references to this variable, is to define a default value, then you can change the value at runtime with Set Test/Suite/Global Variable, but you are always sure that this variable is really meant.