What  Claim Name in openid tokens is used for representing groups?

We are trying to connect directly to the API with for instance Grafana (but it can be other clients for that matter). When working directly with a user it works by setting the “sub”: value in the token to the name of the user. During authorization the API then maps the username of the logged in user to an entry in the rbac. We use Forward OAuth Identity in Grafana and it works fine.
We do however not know how to make it work with groups. What field in the token should be used for groups (or roles?)? We have tried the most obvious “groups”, for instance: "groups": ["/grouptenant1"]. We have also tried to remove the path for example: "groups": ["grouptenant1"]
As a last resort we tried to add the group membership to the sub field. Since it then becomes an array and the code expects a string it of course is a failure.
So in short: How do we present group membership in the token so that the API can use it for authorization?”
We use keycloak to generate the token.

Hi @trippleoone,
The administrator must specify the claim to use for the groups in the tenant configuration, e.g.:
tenants:
- name: test
  id: uuid
  oidc:
    clientID: id
    clientSecret: secret
    issuerURL: url
    redirectURL: redirect_url
    usernameClaim: email
    groupClaim: group

Hi and thank you for the  quick answer. I did a quick test and it works like a charm!

:+1: I'll close this issue then :)


samarth-nagar fix: lazy help tag on line 931

fixing issue #1152
not an one to one solution but help-tag lazy.nvim-🔌-plugin-spec is the closest to
supposed :help lazy.nvim-lazy.nvim-structuring-your-plugins that is nowhere to be found

Please check the commit as it's accidentally introducing unnecessary whitespace in some places irrelevant to the issue, and is also failing the stylua check.

on it

@rivenirvana I have fixed the accidental white spaces


Can not infer where a column is from but then the error message says that column is available?

What happened?
I'm not sure if this is a real bug or just a strange error message?
PRQL input
from data
join side:left other (other.id== data.id)
into A

from codes
into S

from A
select {
A.val
}

SQL output
Error:
   ╭─[:7:3]
   │
 7 │   A.val
   │   ──┬──
   │     ╰──── Cannot infer where `A`.val is from. It could be any of [(119, {}), (116, {})]
   │
   │ Help: available columns: `A`.val
───╯

Expected SQL output
Any sql.

MVCE confirmation

[X] Minimal example
[X] New issue

Anything else?
No response

I did some experimentation with this example.
The from codes \n into S section isn't necessary to reproduce the problem, this produces the same output:
from data
join side:left other (==id)
into A

from A
select {A.val}

Error:
   ╭─[:6:9]
   │
 6 │ select {A.val}
   │         ──┬──
   │           ╰──── Cannot infer where `A`.val is from. It could be any of [(119, {}), (116, {})]
   │
   │ Help: available columns: `A`.val
───╯

The error goes from being confusing to clear when you remove the into A \n from A sequence and remove the A reference in the select:
from data
join side:left other (==id)
select {val}

Error:
   ╭─[:3:9]
   │
 3 │ select {val}
   │         ─┬─
   │          ╰─── Ambiguous name
   │
   │ Help: could be any of: data.val, other.val
───╯

So the root cause of the error message is that A.val is ambiguous in that it could come from either data or other. Unfortunately, the into A \n from A break makes it impossible to directly reference columns from the source relations:
from data
join side:left other (==id)
into A

from A
select {data.val}

Error:
   ╭─[:6:9]
   │
 6 │ select {data.val}
   │         ────┬───
   │             ╰───── Unknown name `data.val`
───╯

The most obvious solution to me is to recommend explicitly defining columns using a select prior to the into A. This resolves the error from the user perspective.
from data
join side:left other (==id)
select {data.val, other.col}
into A

from A
select {A.val}  # no error now

This might actually be the only true solution to this problem, since there doesn't seem to be any other way for the resolver to  disambiguate a column reference when there are multiple source relations with wildcard columns.

Ah, yes I was just thinking in general about explicit col defines via select and didn't think to try it in these exampels.
I wonder if this is perhaps the ONE COMPROMISE that PRQL should really push hard on it's users. The more I get into the weeds the more I understand that very little can be done without knowing the original column names.


Modifier -> SwiftUI -> Scroll views: scrollBounceBehavior(_:axes:) [BETA]

Doc: https://developer.apple.com/documentation/swiftui/view/scrollbouncebehavior(_:axes:)

[ ] Swift implementation
[ ] Elixir implementation

Contribution needed

Blocked on #868


GH pages TLS certificate expired

Something broke the automatic refresh of our Github-issued Let's Encrypt TLS certificate for https://apalache.informal.systems

Looks like the refresh task was stuck somewhere.
I managed to kick it off by going to Settings > Pages > Custom domain, and changing the domain there to a dummy value, and then back to apalache.informal.systems.
This needed a bit of time (few minutes) to propagate through DNS TTL.


PMM-1710 Add support for Supervisord

Do not merge yet.
@delgod Makes sense? Should something be added/changed?

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Display the current speed limit [0.25 days]

Write utility function to display the current speed limit.

The application should always be displaying the current speed limit in the corner of the window. This is extremely useful because sometimes the chaos of the race causes us to miss or forget the speed limit signs. This data can get pulled from the Steps read in from the CSV. However, not every Step is listed with a speed limit so it would require finding the most recent step that did have a speed limit and displaying that.


Asset Generation issues...

I'm experiencing issues when building for dev/dist that are annoyingly happening only 90% of the time.  When asset generation is happening, quite often the compiler falls over with 100s of
"ERROR in ./src/states/Instructions.ts
(25,86): error TS2339: Property 'BitmapFonts' does not exist on type 'typeof "C:/Users/XX/XX/XX/XX/src/assets"'.
It feels like the asset generation has not finished before it tries to compile the JS - meaning I get loads of errors of non-existent assets.
I did try and add in the few lines from the latest merges, but it made no difference.
Any help would be great :)

make sure to add two files with the same name but one .fnt ext and another with .png ext in assets/fonts
to create bitmapFonts
and then run $npm run assets then go and check the assets.ts File, and see if  namespace BitmapFonts with yourFontName class is created or not if this is not your current problem take a screenshot of errors and post them

#39 has been merged in with some asset generation fixes; please try the latest version out. Thanks!

That did not fix the issue.
The issue stems from: https://github.com/1337programming/webpack-shell-plugin/issues/37
You can fix it by replacing
new WebpackShellPlugin({
  onBuildStart: ['npm run assets']
}),

with
new WebpackSynchronizableShellPlugin({
  onBuildStart: {
    scripts: ['npm run assets'],
    blocking: true,
    parallel: false
  }
}),

in your webpack config (npm isntall webpack-synchronizable-shell-plugin --save-dev of course)

@Nepoxx Huh, interesting, I've never encountered that one. Thanks so much for doing the research and providing the PR! Much appreciated 👍

@rroylance I've never encountered the issue myself someone was not able to build our project due to this. Since it's a race condition it needs specific timings to occur, I think. In any case using WebpackSynchronizableShellPlugin should fix the issue :)

Merged in. Thanks again @Nepoxx !


Create OSS Repository for Prometheus to Azure Monitor exporter service

AC: Create OSS Repo into Microsoft org. Ask original authors if we can wipe commit history before publishing.

@mtarng what's the status on this? It might be happen to add a task list to show what has been done


Add avro on the top of Kafka for decoupling message schema

References:

https://www.confluent.io/blog/decoupling-systems-with-apache-kafka-schema-registry-and-avro
https://github.com/confluentinc/confluent-kafka-dotnet/issues/423 => discussion with using Microsoft.Hadoop.Avro.AvroSerializer to seri/deserializer message
https://dzone.com/articles/kafka-avro-serialization-and-the-schema-registry
https://github.com/confluentinc/confluent-kafka-dotnet/blob/master/examples/AvroBlogExamples/Program.cs
https://github.com/jakobz/schema-registry-dotnet

Pros and Cons of various formating types at<EMAIL_ADDRESS>

Now we used Protobuf for communication between publisher and consumer.


[bug] Several Multi Meta Variable captures hangs CLI indefinitely

Given the example below:
use std::cmp::{Eq, Ordering, PartialEq};

fn main() {}

Causes the --replaces below to hang indefinitely:
$ sg run -p 'use std::cmp::{$$$A, Ordering, $$$B};' -r 'use std::cmp::{$$$A, $$$B, Ordering};' -l rs ./src/lib.rs

$ sg run -p 'use std::cmp::{$$$A, Ordering, $$$B};' -r 'use std::cmp::{$$$A};' -l rs ./src/lib.rs

Thanks for reporting! This is fixed and released in 0.5.7!


Simplify (and correct) examples for exporting Service Principal secrets

The existing documentation supplies this snippet for converting a service principal's secret from a SecureString to a human readible string:
$BSTR = [System.Runtime.InteropServices.Marshal]::SecureStringToBSTR($sp.Secret)
$UnsecureSecret = [System.Runtime.InteropServices.Marshal]::PtrToStringAuto($BSTR)

This PR changes that to a simplified example using the native PowerShell cmdlet, ConvertFrom-SecureString:
$UnsecureSecret = ConvertFrom-SecureString $sp.Secret

This is much shorter and easier to follow, but also a colleague was trying to use the current example today and was getting incorrect results. This change did work for them.

Docs Build status updates of commit 109fd85:
:clock10: Full build: the average full build time is 33 min(s) 39 sec(s), this is based on the last 525 build(s) for this repository.

Don't merge this yet, testing to make sure this is 100% right. Have converted to a draft PR in the meantime.

Ha, nope. My bad, nothing to see here 😅

Docs Build status updates of commit 109fd85:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-conceptual/azps-0.10.0/create-azure-service-principal-azureps.md
:white_check_mark:Succeeded
View (azps-0.10.0)

docs-conceptual/azps-1.8.0/create-azure-service-principal-azureps.md
:white_check_mark:Succeeded
View (azps-1.8.0)

docs-conceptual/azps-2.8.0/create-azure-service-principal-azureps.md
:white_check_mark:Succeeded
View (azps-2.8.0)

docs-conceptual/azps-3.8.0/create-azure-service-principal-azureps.md
:white_check_mark:Succeeded
View (azps-3.8.0)

docs-conceptual/azps-4.5.0/create-azure-service-principal-azureps.md
:white_check_mark:Succeeded
View (azps-4.5.0)

docs-conceptual/azps-4.6.0/create-azure-service-principal-azureps.md
:white_check_mark:Succeeded
View (azps-4.6.0)

docs-conceptual/azps-4.6.1/create-azure-service-principal-azureps.md
:white_check_mark:Succeeded
View (azps-4.6.1)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


sort with orderBy lists

Hi.
I need to filter posts based on multiple sorting criteria, so I do the following:
ctx.photon.posts.findMany({
     where: filters,
      orderBy:{
          createdAt:"desc",
          title:"asc",
          content: "asc"
      }
})

I want it to be sorted first according to the creation time, then sorted by title and content.
But I got an error:
Argument orderBy of type PostOrderByInput needs exactly one argument, but you provided `createdAt` and `title` and `content`.

I seem to be able to choose only one, but not multiple.
So I want to know if photon implements multiple conditional sorting. I saw its usage in the specs.

I came across this as a surprise since the official docs say it should work with multiple arguments.
https://www.prisma.io/docs/reference/tools-and-interfaces/prisma-client/crud#reference-1
Eg:
export type UserOrderByInput = {   id?: OrderByArg | null;   name?: OrderByArg | null;   email?: OrderByArg | null;   role?: OrderByArg | null;   profileViews?: OrderByArg | null; };

It says Lets you order the returned list by any model property.. I don't know how you interpret that has saying multiple properties?

Hmm, it is the same description for UserWhereInput where we can specify multiple properties right?
Wraps all fields of a model so that the list can be filtered by any model property.

I think we can change the wording there to make it more clear for now, maybe you can make a PR here for that: https://github.com/prisma/prisma2-docs

Hello @sessionboy and everyone.
We are tracking this as a feature request in https://github.com/prisma/prisma-client-js/issues/702.


make sure we use our resource to get interpreter details

For #11469

[x] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[x] Title summarizes what is changing.
[x] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[x] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #11474 into master will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master   #11474      +/-   ##
==========================================
- Coverage   60.61%   60.60%   -0.02%
==========================================
  Files         612      612
  Lines       33472    33472
  Branches     4719     4719
==========================================
- Hits        20290    20285       -5
- Misses      12731    12735       +4
- Partials      451      452       +1

Impacted Files
Coverage Δ

...client/datascience/kernel-launcher/kernelFinder.ts
61.26% <ø> (ø)

...rc/client/common/process/pythonExecutionFactory.ts
87.25% <100.00%> (+0.98%)
:arrow_up:

src/datascience-ui/react-common/arePathsSame.ts
75.00% <0.00%> (-12.50%)
:arrow_down:

src/client/common/utils/platform.ts
64.70% <0.00%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

src/client/common/process/proc.ts
14.49% <0.00%> (-0.73%)
: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 6218433...8e57daf. Read the comment docs.


Add meta data: site description and preview image

Adds a preview image and a description. The OwnPath English logo is the preview image.
Known issue:
The description and image will be English, even if linking from the Spanish URLs. It'd be a more significant engineering effort to  get that to work, which we could plan for the future if desired.

@jeffmaher I wasn't able to preview the image

@amdarnell7 : Thanks for testing! So that I can reproduce what you're seeing what site did you share on to see if the description/image came up?

@jeffmaher I do see the logo and the summary but I guess I was expecting an image to pull in a post-style format so that if someone were to post to facebook, etc, it would pull in a larger image but I may have misunderstood the ask.

@amdarnell7 : Got it! Right now, just using the logo as the preview image, but we could do a different one. Do you have thoughts on what an alternate preview image should be?
@admoorgit : Do you have thoughts on preview image?

@admoorgit is proposing the following image:


can not connect to the websocket from Lte

D/NetworkSecurityConfig: No Network Security Config specified, using platform default
D/TcpOptimizer: TcpOptimizer-ON
onStateChanged
onThreadStarted
onThreadCreated
D/MainActivity: log: onError: com.neovisionaries.ws.client.WebSocketException: Failed to connect to 'wssddd.ddd.ddd:443': connect timed out

A rude issue which just pasted logs.


refactor: remove condition which is always true

Coverage remained the same at 100.0% when pulling 16b61d421f7d9e62ff8b91c0e175e997a8d1365b on clarkdo:master into 4bc9af44321e5e4c9fccf373d7e189fda8b7a635 on developit:master.

Thanks @clarkdo! However this looks identical to #890.


The example is not correct

The example which uses EventData[] is not correct. The new isolated process doesn't work with EventData and gives empty. Instead string [] works.
The document is not clear.
Eg. If I am upgrading/ moving/ create new function app
There is no clear way how i can fetch the event only after specific date so that the duplication is avoided. Since new app will not have any checkpoint and string [] is not given any date.
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: a109c38e-b241-c9a2-7ef8-2e828659191c
Version Independent ID: 69842b0f-607f-5cf5-6e1a-4d312c98100c
Content: Azure Event Hubs trigger for Azure Functions
Content Source: articles/azure-functions/functions-bindings-event-hubs-trigger.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

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


[BUG]: The web show error code status code 500 when change time to notify.

Describe the bug
When click to change time to notify in the assignment. The web show error code status code 500.
Step To Reproduce the bug:

Go to the home page and login
Click on the assignment
Click on the edit button
Click on Notification Settings
Change time to notice before the due date.
Click on save
See Error code status code 500

Expected behavior
The Web should not show error code status code 500.
Screenshots

Hardware Spec (please complete the following information):

Device: [Computer
OS: Window11
Browser Microsoft Edge
Version 96.0.1054.29 (Official Build) (64-bit)

Additional context
Add any other context about the problem here.

Thank you for reporting this to us. I just looked into the server log and I believe that this issue is related to our backend. So there is not much we can do here since this repo is only for the frontend.
I'll transfer this issue to the GarnBarn/garnbarn-backend and we'll continue to discuss this in that repo.


Handle combatants that have no token.

Modifies the functions that retrieve a token from a combatant to account
for no token to be returned. In the case of the TurnMarker,
MoveToCombatant is changed to clear the marker in the case of an empty
token. The updateCombat hook is modified to use optional chaining,
ensuring that no errors are thrown if there's no token.
This improves compatibility with lancer-initiative, which uses a hidden
combatant at the beginning of the tracker to represent no turn
occurring.

thanks for the contribution


GUVNOR-2415: Boundary events lose it’s documentation after save and r…

…eopen Business Process + fixed missing/wrong named resources

+1 Thanks!


Manual Camera port?

Can i manually set a port for the camera to run on? It seems to switch ephemeral ports on every homebridge restart.
I need this for firewalling.

Any news on this?

Can this be re-opened? I think it is still relevant.

you can set this in config.json
"ports": {
        "start": 52100,
        "end": 52150
    },


[RFC] Radio - create group options and add event handlers

Feature Request
Problem description
A clear and concise description of what the problem is. Ex. I have an issue when [...]
Proposed solution
A clear and concise description of what you want to happen. Add any considered drawbacks.
MVP

fixed in https://github.com/stardust-ui/react/pull/229


Short solution needed: "How to get current date" (nodejs)

Please help us write most modern and shortest code solution for this issue:
How to get current date (technology: nodejs)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to get current date


Use multiple instances of OpenIDConnect middleware in one app

Long story short: for Azure AD B2C, we need to use multiple instances of the OIDC middleware in a single app.  Each one needs its own AuthenticationScheme so that I can invoke the proper middleware via challenges.
I've set up the instances like so:
// The CreateOptionsFromPolicy sets the policy id as the AuthenticationScheme
app.UseOpenIdConnectAuthentication(CreateOptionsFromPolicy(SignUpPolicyId));
app.UseOpenIdConnectAuthentication(CreateOptionsFromPolicy(SignInPolicyId));

Let's say I execute a challenge with AuthenticationScheme=SignInPolicyId.  The requests gets sent to Azure AD, the user logs in, and the response comes back to the app with a valid OIDC message & id_token.
AspNetCore first tries to validate the incoming message using the middleware instance with AuthenticationScheme=SignUpPolicyId.  Problem is (I think) that the state in the OIDC request/response was encrypted/protected with the authentication scheme.  So the "sign up" middleware instance fails to decrypt the state parameter, and fails the authentication.  It never tries to validate the message using the other middleware instance.
In Katana, when this same failure occurred, the sign in request did not fail - instead the next instance of the middleware was attempted (and succeeded).  I was expecting the same behavior here.
Let me know if this makes sense.  Happy to provide sample code if necessary.  And if you wouldn't mind, please tag (@) me in any responses you want me to see so I can keep track.

One possibility for this would be to use app.Map(...) to "fork" the middleware pipeline given different request roots. Then you'll never have a case where both auth middlesware run on the same request.
But we'll need to wait for @Tratcher to return to figure this out.

I think you will need 2 different CallbackPath set so they don't step on each other.

This worked with Katana because it would read all requests and fall through if it didn't recognize the message. You can use OpenIdConnectOptions.SkipUnrecognizedRequests = true to get this behavior, but we don't recomend it. Instead you should assign each instance a unique CallbackPath as @brockallen suggested.

Alright, I'll give it a shot. Thanks guys.

Reopening to ask a related question, not really a new issue.
In this scenario (see original message), the user might be logged into the app with one instance of the OIDC middleware.  Then the developer invokes a different instance of the middleware using ChallengeAsync(AuthenticationSchemeUsedForDifferentInstance, AuthenticationProperties).  I expected this call to trigger an auth request to the IDP.
Instead it hit this code path (https://github.com/aspnet/Security/blob/dev/src/Microsoft.AspNetCore.Authentication/AuthenticationHandler.cs#L346) and redirected the user to the CookieMiddleware's AccessDeniedPath.
To workaround, I had to change the call to ChallengeAsync(AuthenticationSchemeUsedForDifferentInstance, AuthenticationProperties, Microsoft.AspNetCore.Http.Features.Authentication.ChallengeBehavior.Unauthorized).
My question is mostly about the comment in the line referenced above.  If a UserPrincipal already exists, why do we invoke the "Forbidden" handler path (of which I'm sure the OpenIDConnectMiddleware has none)?

@dstrockis this was recently fixed here: https://github.com/aspnet/Security/pull/980
The Forbidden call is to avoid infinite auth challenge loops. If you already had a valid user for this provider then there's no reason to believe another login will grant the same user different permissions.

@Tratcher awesome, thanks.
One important scenario we have is what we call "incremental consent".  This is where a single application signs the user in initially using a basic set of scopes, like ?scope=openid email profile.  Then, when the user tries to perform some action, the app asks for additional permissions, like ?scope=openid profile email https://graph.microsoft.com/mail.send.
In my mind this would have worked with a single provider.  The way I implemented this in Katana was to pass the required scopes in the AuthenticationProperties, then extract them in RedirectToIdentityProvider and append them to the outbound auth request.
I don't think it's the end of the world if in this scenario the developer needs to pass ChallengeBehavior.Unauthorized.  This comes to mind as a valid case where another login with grant the same user different permissions.  Food for thought.


Failed to test the solution and submission shows undefined

🐛 Bug Report
When I try to test code about question 3[无重复字符的最长子串], it shows "failed to test the solution, please open the output pannel for details". And when I try to submission, it is shown in vscode UI is "submission"
To Reproduce
I am not sure, I just wtrite my code and then try to test it in vscode as usual.
Expected behavior
I wander why this bug appears and how to fix it.
Extension Output
ailed to test the solution. Please open the output channel for details.
Guidance: Press Ctrl+Shift+U, and toggle the channel to LeetCode.
Your Environment

os:Windows_NT x64 10.0.22621

extension settings:"leetcode.workspaceFolder": "C:\Users\11239\.leetcode",
// "leetcode.outputFolder": ""${difficulty}",
"leetcode.defaultLanguage": "python3",
"leetcode.hint.configWebviewMarkdown": false,
"leetcode.hint.commentDescription": false,
"leetcode.hint.commandShortcut": false,
"leetcode.filePath": {
"default": {
"folder": "",
"filename": "${id}.${kebab-case-name}.${ext}"
}
},

nodejs version:16.17.1

vscode version:1.78.2 (user setup)

extension version:v0.18.1

I am experiencing the same issue. When trying to run the "Test" I get [ERROR] http error [code=499] and when trying to  "Submit" I also receive [ERROR] http error [code=499].


Track current position across pages

E.g., if I am at chr20:29828000-29830000 on the /overall page, I should view chr20:29828000-29830000 when I switch to /freq page. Currently, we "reset" to the start of the chromosome.

Closed by #9


Issue #11: Shuttle Bus navigation

Issue #11: Shuttle Bus navigation

Added a new Fragment and ViewModel for shuttle.
The fragment can be accessed from the side nav menu
The shuttle view model uses the navigation repository to get the shuttle times
Fixed an issue with the shuttle times not being retrieved at all from the database.
Separating the Shuttle Fragment into 2 Tabs (SGW TO LOY, and LOY TO SGW) using ViewPage2
The data for the departure times is shown in a table
Generating directions between campuses

I saw the changes on the shuttle fragment, everything looks good to merge now.


Fix with Guix

Just a draft Pull Request, as I was interested if using chromium/chromedriver from Guix could make the tests pass within the docker-compose environment.
Back in 2016, the govuk-guix project started as a environment within which to run the Publishing E2E tests: https://github.com/alphagov/govuk-guix/pull/1
A lot has changed since then, but the rigour that Guix brings can help, even to sure up the Dockerfile approach that has been recently contributing to problems with running the Publishing E2E tests.

Closing now, as this is just a proof of concept.


[minor] add update contract request endpoint

https://github.com/edenlabllc/ehealth.api/issues/2190

Coverage increased (+0.03%) to 91.538% when pulling 5a027bfdee98e707c7d802f125f772e13e3f4b37 on update_contract_request#2190 into f1b0b00c1d5da37accb67b9d66aa72bc30330d19 on contracts#2176.


Handle inversion operator

Expected Behavior
I am trying to retrieve all stories not in completed state. Rather than whitelisting all states I want, it would be much better to blacklist the one I don't want.
There is an available inversion operator

Do we know if it can be handled in some way?

The club search command, when called with a string, passes that string directly to the same search functionality in clubhouse (https://help.clubhouse.io/hc/en-us/articles/360000046646-Search-Operators), so yes - this should already work.


How to use cvat-sdk with token

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Is your feature request related to a problem? Please describe.
When creating a client in cvat-sdk you need to enter the username and password but now I have multiple users and I don't know their passwords but I can get their tokens, can I use this token directly to use the cvat-sdk client?
Describe the solution you'd like
I us token directly to use the cvat-sdk client
Describe alternatives you've considered
No response
Additional context
No response

Hi, there is no "official way", but please check this answer and this draft PR.

Hi, there is no "official way", but please check this answer and this draft PR.

Are sessionid and csrftoken necessary? Because my scenario is that I deployed a privatized cvat myself and integrated him into my system, and I want to automate the import of the dataset in one of my system's features, so I shouldn't be able to get the cross-domain cookie information in the browser

If you're working as a local admin, you should be able to download annotations from other users. Consider using this approach instead. Would a special service account for these purposes work for you?

I only want to upload dataset to project,now I think  I can use normal user create project and use admin user upload datset,because I have admin user password,but question is high level sdk have not only import dataset api,it only have create project from datset api. so what can i do?

If you're working as a local admin, you should be able to download annotations from other users. Consider using this approach instead. Would a special service account for these purposes work for you?

oh,sory,I read this manual: https://opencv.github.io/cvat/docs/api_sdk/sdk/highlevel-api/，That's very helpful.,
it worked:
project = client.projects.retrieve(project_id)

I'm having a little problem again.
My code:
with make_client(
        host="my local server url",
        credentials=('admin', 'admin password')

) as client:
    try:
        project = client.projects.retrieve(project_id)
    except Exception as e:
        print('throw exception：', e)
    project.import_dataset(format_name='CVAT 1.1', filename='dataset file path', pbar=pbar)

it throw exception:
HTTP response body: b'{"detail":"CSRF Failed: CSRF token missing or incorrect."}'
I've noticed that in some issues uploading on the page may encounter this problem, just try logging out and logging in, now in the sdk I'm still encountering this problem, how can I fix it? Thank you very much for your reply!

I solved it! It looks like it was a code issue, there was no X-Csrftoken set for the header anywhere in the code for importing the dataset, so it was reporting an error：https://stackoverflow.com/questions/26639169/csrf-failed-csrf-token-missing-or-incorrect
,I'm now changing it this way and it doesn't report an error:
        pairs = client.api_client.get_common_headers()['Cookie'].split('; ')
        dictionary = {pair.split('=')[0]: pair.split('=')[1] for pair in pairs}
        client.api_client.set_default_header("X-Csrftoken", dictionary['csrftoken'])
        client.api_client.get_common_headers()

The core is to get the csrftoken from the cookie and set the X-CSRFToken to the header.
but But there is a new error reported:
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/cvat_sdk/core/proxies/projects.py", line 57, in import_dataset
    DatasetUploader(self._client).upload_file_and_wait(
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/site-packages/cvat_sdk/core/uploading.py", line 321, in upload_file_and_wait
    rq_id = json.loads(response.data).get("rq_id")
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/Library/Frameworks/Python.framework/Versions/3.10/lib/python3.10/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

code is:rq_id = json.loads(response.data).get("rq_id")
But response.data doesn't look like a json string but a byte array

I solved it again,But the sdk api does have problems too, hopefully it will be fixed soon!
project = client.projects.retrieve(project_id)
filename = Path(dataset_path)
params = {"format": 'CVAT 1.1', "filename": filename.name}
url = client.api_map.make_endpoint_url(project.api.create_dataset_endpoint.path, kwsub={"id": project_id})
response = DatasetUploader(client).upload_file(
                url, filename, pbar=pbar, query_params=params, meta={"filename": params["filename"]})

Thank you for reporting the issues.

But response.data doesn't look like a json string but a byte array

Probably, you're using an older version of CVAT, before the change was introduced. Please make sure SDK and server versions match. The change was introduced in https://github.com/opencv/cvat/pull/5909.

I am looking forward to contributing to solving this issue through GSOC 24 and I have some questions regarding the scope of this project.

Scope Clarification:

According to the documentation, there is already a way of Authentication using tokens. Does this mean that the scope of this problem is to make the process of creating these tokens easier with the UI?

Integration with auth_api:

Do I need to add a new method to auth_api for token generation, or utilize the existing login method since it returns an access token?

Display of Access Tokens:

Should the access token be directly visible in the account settings, allowing users to copy and manually include it in the request headers using api_client.set_default_header("Authorization", "Token " + {generated token})?
Alternatively, should the UI indicate successful addition of the access token and automate its inclusion in environment variables for seamless integration into request headers?

I am looking forward to contributing to solving this issue through GSOC 24 and I have some questions regarding the scope of this project.

Scope Clarification:

According to the documentation, there is already a way of Authentication using tokens. Does this mean that the scope of this problem is to make the process of creating these tokens easier with the UI?

Integration with auth_api:

Do I need to add a new method to auth_api for token generation, or utilize the existing login method since it returns an access token?

Display of Access Tokens:

Should the access token be directly visible in the account settings, allowing users to copy and manually include it in the request headers using api_client.set_default_header("Authorization", "Token " + {generated token})?
Alternatively, should the UI indicate successful addition of the access token and automate its inclusion in environment variables for seamless integration into request headers?

I would greatly appreciate your guidance in clarifying this. Your expertise would be incredibly valuable. I truly appreciate your support.
@zhiltsov-max @SpecLad @azhavoro

@Abo-Omar-74,
Hi, thank you for reaching us out about this topic. Overall, the scope for GSoC can vary, depending on your skills and desire. This is a complex task, that can be split into several elements:

SDK / CLI update to support persistent login. Can be done with any type of keys. Basically, it means integrating this code snippet into our current CLI:

This code allows to record user credentials in the user profile directory. Credentials for each visited CVAT host are recorded, to allow future visits without explicit authentication from the user.
from __future__ import annotations

import json
import os
from argparse import ArgumentParser
from functools import partial
from pathlib import Path
from types import SimpleNamespace
from typing import Optional, Tuple

import attrs
import cvat_sdk

API_KEY_VAR = "CVAT_API_KEY"
API_SESSIONID_VAR = "CVAT_API_SESSIONID"
API_CSRFTOKEN_VAR = "CVAT_API_CSRFTOKEN"

def add_cli_parser_args(parser: ArgumentParser) -> ArgumentParser:
    parser.add_argument("--org")
    parser.add_argument("--host", default="https://app.cvat.ai")
    parser.add_argument("--port")
    parser.add_argument(
        "--login",
        help=f"A 'login:password' pair. "
        f"Default: use {API_KEY_VAR}, {API_SESSIONID_VAR}, {API_CSRFTOKEN_VAR} env vars",
    )
    parser.add_argument("--profile-dir", default=None, type=Path, help="User profile dir")

    return parser

DEFAULT_PROFILE_FILENAME = "profile.json"
DEFAULT_PROFILE_DIR = "~/cvat/"

@attrs.define
class UserCredentials:
    token: str
    sessionid: str
    csrftoken: str

@attrs.define
class UserProfile:
    _PROFILE_UMASK = 0o600

    @classmethod
    def get_default_path(cls) -> Path:
        return Path(DEFAULT_PROFILE_DIR).expanduser() / DEFAULT_PROFILE_FILENAME

    path: Optional[Path] = attrs.field(factory=partial(get_default_path.__func__, __build_class__))

    credentials: dict[str, UserCredentials] = attrs.field(factory=dict)
    extra: dict[str, str] = attrs.field(factory=dict)

    @classmethod
    def parse(cls, path: Path) -> UserProfile:
        data = json.loads(path.read_text())

        return UserProfile(
            credentials={k: UserCredentials(**v) for k, v in data["credentials"].items()},
            extra=data.get("extra"),
        )

    @classmethod
    def load(cls, path: Optional[Path] = None) -> UserProfile:
        path = path or cls.get_default_path()

        if path.is_file():
            if (mode := path.stat().st_mode) & cls._PROFILE_UMASK != cls._PROFILE_UMASK:
                raise Exception(f"Invalid profile mode. Expected 600 (rw--), got {oct(mode)}")

            profile = cls.parse(path)
        else:
            profile = UserProfile(path=path)
            profile.save(path)

        return profile

    def save(self, path: Optional[Path] = None):
        path = path or self.path or self.get_default_path()

        data = json.dumps(
            attrs.asdict(self, filter=lambda a, v: a.name != "path", recurse=True), indent=2
        )

        if path.absolute() == self.get_default_path().absolute() and not path.parent.is_dir():
            path.parent.mkdir(mode=0o700, parents=True, exist_ok=True)

        with path.open("w", encoding="utf-8") as f:
            f.write(data)

        path.chmod(self._PROFILE_UMASK)

    def update_credentials(self, host: str, *, token: str, sessionid: str, csrftoken: str):
        self.credentials[self._make_host_key(host)] = UserCredentials(
            token=token, sessionid=sessionid, csrftoken=csrftoken
        )

    def has_credentials(self, host: str) -> bool:
        return self._make_host_key(host) in self.credentials

    def get_credentials(self, host: str) -> UserCredentials:
        return self.credentials[self._make_host_key(host)]

    def _make_host_key(self, host: str) -> str:
        return host.split("://", maxsplit=1)[-1]

class UserClient(cvat_sdk.Client):
    def __init__(self, *args, profile: Optional[UserProfile] = None, **kwargs) -> None:
        super().__init__(*args, **kwargs)

        self.profile = profile or UserProfile.load()

    def login(self, credentials: Tuple[str, str]) -> None:
        super().login(credentials)
        self.save_current_credentials()

    def load_credentials(self):
        credentials = self.profile.get_credentials(self.api_map.host)

        self.api_client.set_default_header("Authorization", f"Token {credentials.token}")
        self.api_client.cookies["sessionid"] = credentials.sessionid
        self.api_client.cookies["csrftoken"] = credentials.csrftoken
        self.api_client.set_default_header("X-Csrftoken", credentials.csrftoken)

    def save_current_credentials(self):
        self.profile.update_credentials(
            host=self.api_map.host,
            token=self.api_client.default_headers["Authorization"].split("Token ")[-1],
            sessionid=self.api_client.cookies["sessionid"].value,
            csrftoken=self.api_client.cookies["csrftoken"].value,
        )
        self.profile.save()

def make_client_from_cli(parsed_args: SimpleNamespace) -> UserClient:
    profile = UserProfile.load(parsed_args.profile_dir)

    host = parsed_args.host
    port = parsed_args.port
    url = host.rstrip("/")
    if port:
        url = f"{url}:{port}"

    with UserClient(url, profile=profile) as client:
        if parsed_args.org:
            client.organization_slug = parsed_args.org

        if parsed_args.login:
            client.login(parsed_args.login.split(":", maxsplit=1))
        elif api_key := os.getenv(API_KEY_VAR):
            client.api_client.set_default_header("Authorization", f"Token {api_key}")
            client.api_client.cookies["sessionid"] = os.getenv(API_SESSIONID_VAR)
            client.api_client.cookies["csrftoken"] = os.getenv(API_CSRFTOKEN_VAR)
            client.api_client.set_default_header("X-Csrftoken", os.getenv(API_CSRFTOKEN_VAR))
            client.save_current_credentials()
        elif profile.has_credentials(client.api_map.host):
            client.load_credentials()

        return client

I can see it can be extended with an auth CLI command, that allows just to login on a host and store tokens locally for further use. It can also be extended with a command to remove any of the recorded tokens from the local profile.

Server updates to support manageable API token generation for a user
UI updates to support API token management options in the personal account page (basically, requires to add such page first, as there is no such section in CVAT yet). Probably, can work similarly to what's on GitHub.
SDK / CLI updates to support API tokens for auth - similar to the existing login/password pair

Basically, I'd propose to start with the first task from this list. It already has a PoC implementation, doesn't require too many changes, and just needs to be productized - i.e. with tests and convenient user interface.

Hi, I'm interested in contributing to this project. My understanding is that the goal is to implement a system within user profiles that allows for the generation of API access tokens. Users should be able to store these tokens locally for persistent authorization, similar to how GitHub handles personal access tokens. Can you confirm if this understanding is correct?
@zhiltsov-max @nmanovic

@ritikraj26, yes, your understanding is correct.

Currently, the user is obtaining the auth token using the username and password. We need to just replace the current method with the API access tokens generated by the user.
@zhiltsov-max

@zhiltsov-max
Where can I connect with you? Would you be open to some discussion? I am keenly interested in contributing to this project.

@ritikraj26, I think it's best to be discussed here for others to see and participate, unless you're going to send your GSoC proposal. Proposals need to be sent via the GSoC site.

The rest of the flow remains the same. Only the method to obtain the auth token is updated?

Basically - yes, but "The rest of the flow" needs clarification. If you meant the point 1 from the comment above, then I think it's quite close to what's expected from the token use point of view. But tokens can also be managed in CVAT as discussed in the comment.

Hi, I recently switched from a ubuntu to mac, I am having some trouble setting up.

Some help please. I am looking forward to complete this project.

Hi, I recently switched from a ubuntu to mac, I am having some trouble setting up.
Some help please. I am looking forward to complete this project.

https://stackoverflow.com/questions/69818376/localhost5000-unavailable-in-macos-v12-monterey

Hi, I recently switched from a ubuntu to mac, I am having some trouble setting up.
Some help please. I am looking forward to complete this project.

https://stackoverflow.com/questions/69818376/localhost5000-unavailable-in-macos-v12-monterey

This issue is resolved. However, I am still not able to see an requests on the debug server.

@ritikraj26, please try the following:

Check the server logs after you tried to open the CVAT page in the browser. If there are no requests listed in the log, try to replace localhost with <IP_ADDRESS> in cvat-ui/package.json:8 ("start": "webpack serve --env API_URL=http://<IP_ADDRESS>:7000 --config ./webpack.config.js --mode=development",)
Then you also might need to start OPA and several other containers, as shown in the development environment setup guide (docker compose -f docker-compose.yml -f docker-compose.dev.yml up -d --build \ cvat_opa cvat_db cvat_redis_inmem cvat_redis_ondisk cvat_server)

@ritikraj26, please try the following:

Check the server logs after you tried to open the CVAT page in the browser. If there are no requests listed in the log, try to replace localhost with <IP_ADDRESS> in cvat-ui/package.json:8 ("start": "webpack serve --env API_URL=http://<IP_ADDRESS>:7000 --config ./webpack.config.js --mode=development",)
Then you also might need to start OPA and several other containers, as shown in the development environment setup guide (docker compose -f docker-compose.yml -f docker-compose.dev.yml up -d --build cvat_opa cvat_db cvat_redis_inmem cvat_redis_ondisk cvat_server)

Changing localhost to <IP_ADDRESS> worked. Thanks.


feat: initial refactor for overhauling old database integration

feat: initial refactor for overhauling old database integration
feat: insert block rows during proposal step
feat: initial transaction inclusion for blocks
style: cargo fmt compliance
feat: support transaction invalidation and output storage

#52  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @L8D and the rest of your teammates on  Graphite


Dart non-nullable experiment not working on Windows

On Windows, I'm unable to successfully trigger the non-nullable path.

This same code throws no issue when loaded on my Mac.

Source is here: https://github.com/timsneath/win32/tree/nnbd

Not sure what is up, but the repro steps don't work for me as in - example\console.dart runs fine:
PS C:\src\win32> C:\src\f\flutter\bin\dart.bat --version
Dart SDK version: 2.11.0-207.0.dev (dev) (Fri Oct 9 09:16:09 2020 -0700) on "windows_x64"
PS C:\src\win32> git log -1
commit 0340bed2e1547fee9fa491f776cccf30044f1cbc (HEAD -> master, origin/master, origin/HEAD)                           Merge: df554985 36837416
Author: Tim Sneath<EMAIL_ADDRESS>Date:   Fri Oct 2 09:58:57 2020 -0700                                                                                Merge pull request #114 from bryancusatis:master                                                                                Added SetCursorPos
PS C:\src\win32> C:\src\f\flutter\bin\dart.bat --enable-experiment=non-nullable .\example\console.dart                 Some console metrics:
  Window dimensions LTRB: (0, 0, 118, 36)
  Cursor position X/Y: (0, 26)
  Window size X/Y: (119, 9999)
  Maximum window size X/Y: (119, 55)

Type a line and press Enter, or q to quit: q
Type any key, or q to quit: q
PS C:\src\win32>

I'm sorry -- I filed lots of bugs yesterday and forgot to note the branch I'm using on this one. Check out the nnbd branch:
[C:\src\win32] git checkout nnbd
Already on 'nnbd'
Your branch is up to date with 'origin/nnbd'.

[C:\src\win32] c:\flutter\bin\dart.bat --version
Dart SDK version: 2.11.0-207.0.dev (dev) (Fri Oct 9 09:16:09 2020 -0700) on "windows_x64"

[C:\src\win32] git log -1
commit ab57a5619a9d535c9b2a47903c35470f3687f359 (HEAD -> nnbd, origin/nnbd)
Author: Tim Sneath<EMAIL_ADDRESS>Date:   Fri Oct 9 16:51:23 2020 -0700

    Manual updates to applied migration

[C:\src\win32] c:\flutter\bin\dart.bat --enable-experiment=non-nullable .\example\console.dart
example/console.dart:25:6: Error: Expected ';' after this.
late CONSOLE_SCREEN_BUFFER_INFO bufferInfo;
     ^^^^^^^^^^^^^^^^^^^^^^^^^^
example/console.dart:25:33: Error: Variables must be declared using the keywords 'const', 'final', 'var' or a type name.
Try adding the name of the type of the variable or the keyword 'var'.
late CONSOLE_SCREEN_BUFFER_INFO bufferInfo;
                                ^^^^^^^^^^
example/console.dart:25:1: Error: Type 'late' not found.
late CONSOLE_SCREEN_BUFFER_INFO bufferInfo;
^^^^
example/console.dart:25:1: Error: 'late' isn't a type.
late CONSOLE_SCREEN_BUFFER_INFO bufferInfo;
^^^^

Got you.
On nnbd branch I'm seeing these errors, but I see them on both win and mac.
The problem seems to be relaxed constraint to sdk in pubspec.yaml which has to be tight to allow nnbd experiment. The following patch fixes that, but example/console.dart still fails to run with other nnbd-related problems:
PS C:\src\win32> git diff
diff --git a/pubspec.yaml b/pubspec.yaml
index f0216189..0827a491 100644
--- a/pubspec.yaml
+++ b/pubspec.yaml
@@ -7,7 +7,8 @@ issue_tracker: https://github.com/timsneath/win32/issues
 documentation: https://timsneath.github.io/win32/

 environment:
-  sdk: '>=2.9.0 <2.12.0'
+  # This must remain a tight constraint until nnbd is stable
+  sdk: '>=2.10.0-0 <2.11.0'

 dependencies:
   ffi:
PS C:\src\win32>

PS C:\src\win32> C:\src\f\flutter\bin\dart.bat --enable-experiment=non-nullable .\example\console.dart
lib/src/structs.dart:36:7: Error: Field 'style' should be initialized because its type 'int' doesn't allow null.
  int style;
      ^^^^^
lib/src/structs.dart:38:27: Error: Field 'lpfnWndProc' should be initialized because its type 'Pointer<NativeFunction<Function>>' doesn't allow null.
 - 'Pointer' is from 'dart:ffi'.
 - 'NativeFunction' is from 'dart:ffi'.
 - 'Function' is from 'dart:core'.

 - 'Pointer' is from 'dart:ffi'.
 - 'Utf16' is from 'package:ffi/src/utf16.dart' ('/C:/Users/aam/AppData/Roaming/Pub/Cache/git/ffi-82605e4f24a7a71a30462901e62a1a433f451906/lib/src/utf16.dart').
  Pointer<Utf16> lpszMenuName;
                 ^^^^^^^^^^^^
lib/src/structs.dart:59:18: Error: Field 'lpszClassName' should be initialized because its type 'Pointer<Utf16>' doesn't allow null.
 - 'Pointer' is from 'dart:ffi'.
 - 'Utf16' is from 'package:ffi/src/utf16.dart' ('/C:/Users/aam/AppData/Roaming/Pub/Cache/git/ffi-82605e4f24a7a71a30462901e62a1a433f451906/lib/src/utf16.dart').
  Pointer<Utf16> lpszClassName;
                 ^^^^^^^^^^^^^
PS C:\src\win32>

You can confirm language version of win32(it has to be 2.10 at present to allow for nnbd) in .dart_tool/package_config.json:
    {
      "name": "win32",
      "rootUri": "../",
      "packageUri": "lib/",
      "languageVersion": "2.10"
    }

OK, got it. Thanks -- we need to do a good job of documenting how the version constraints are applied, but I now understand this not to be a bug in Dart itself. Appreciate the help!


Python logs out to Docker container

Hey there, firstly thanks for the great container!
I'm having some troubles with getting useful output from the python script out to my calls to docker logs.
In my output I get the logs from the entrypoint file and then an error which I believe comes from the xfce4-terminal call?
Failed to connect to session manager: Failed to connect to the session manager: SESSION_MANAGER environment variable not defined

After this I'm  not able to see any of the python output, is this because its running inside of the separate terminal? Do you know of anyway to pipe the output from python out to stdout on the container?

Github, the perfect rubber duck.
Managed to figure this out, for anyone else that comes across, I got the python logging to output to a file rather than just to stdout inside the other process.
e.g. something like
logging.basicConfig(
    filename=os.path.join(BASE_PATH, "output.log"),
    filemode="w",
    format='%(asctime)s %(levelname)s %(message)s', level=logging.INFO
)

Then in entrypoint.sh I added ln -sf /proc/1/fd/1 "$HOME"/output.log it turns out that because we're running a background process we can't just symlink /dev/stdout as some sources say ( Loads more detail here )

@kylebyrne thank you! :+1:


show output on docker unpause

Problem description

Problem location
https://docs.docker.com/engine/reference/commandline/unpause/

I saw a problem on the following URL:

I couldn't find the information I wanted. I expected to find it near the following URL:

Other:

Project version(s) affected

Suggestions for a fix
as $ docker unpause my_container is written as command, we should show its output also

@londoncalling @johndmulhausen thoughts, hackathon :)

Hey @surajnarwade -- quick note, only bugs that we have labeled "hackathon" count towards the hackathon. You are free to file bugs and PRs, though. :)

@johndmulhausen , no  worries :)

Issue moved to docker/cli #589 via ZenHub


Pinned action by SHA and included the URL for verification

This is instead of this PR https://github.com/facebook/docusaurus/pull/6984
It was hard to rebase so opening a new one.
Specifically addressed this comment in this PR to include links to GitHub tags/commits in the comments to validate and test it.
https://github.com/facebook/docusaurus/pull/6984#discussion_r835053598

Motivation

Pinned actions by SHA https://github.com/ossf/scorecard/blob/main/docs/checks.md#pinned-dependencies
Included permissions for the action. https://github.com/ossf/scorecard/blob/main/docs/checks.md#token-permissions

Pin actions to a full length commit SHA

Pinning an action to a full length commit SHA is currently the only way to use an action as an immutable release. Pinning to a particular SHA helps mitigate the risk of a bad actor adding a backdoor to the action's repository, as they would need to generate a SHA-1 collision for a valid Git object payload.

https://docs.github.com/en/actions/security-guides/security-hardening-for-github-actions#using-third-party-actions
Also, dependabot supports upgrading based on SHA.
Signed-off-by: naveensrinivasan<EMAIL_ADDRESS>
Have you read the Contributing Guidelines on pull requests?

Test Plan

Related PRs

Closing because of https://github.com/facebook/docusaurus/pull/6984#issuecomment-1079841421


[stable/rabbitmq-ha] add image.pullSecrets

What this PR does / why we need it:
There are values to define custom docker image and tag. But there is no pullSecrets value. It is needed if private repository is used.
@etiennetremel

/lgtm

/lgtm


Update README.md

After this commit the following command throws an error.
php artisan vendor:publish --provider="OptimistDigital\NovaLocaleManager\ToolServiceProvider" --tag="migrations"

Unable to locate publishable resources.

So, this step is not required anymore, since migrations are automatically loaded from ./vendor/optimistdigital/nova-locale-manager/database/migrations directory.

Ah, yes, forgot to keep the docs up to date as usual. Thanks a ton for pointing this out and fixing it!


Fixed Missing Code and cleanup of Console Output

Added support for GpioPinDriveMode.Input,
Removed unwanted
Console.WriteLine calls,
Added Support for Closing (unexport) a pin  safely close access to the pin.

Just a Note on this i made these changes as i was building an Application using this and noticed it was missing bit's like the Close Pin and Support for Input from the pins so i have added these. Also removed all the unnecessary Console.WriteLines

Brilliant - thank you @barkermn01 !


JBEHAVE-1377 Use provided configuration objects while creating default story parser

The changes have been added to master branch: 65b5dde5610f830d1bf89e5f087587398aeec5dc


Zauber halcyon 5

This has a fix for a persistence that was uncovered, which I had inadvertently caused with my changes for checking if an object was scripted.  Also included are implementations of the four following script functions:
list iwListRemoveElements(list src, list elements, integer count);
list iwListRemoveDuplicates(list src);
iwStartLinkAnimation(integer link, string anim);
iwStopLinkAnimation(integer link, string anim);

Code reviewed. Looks good to go


some sentences are too long

Hi, there.
I found timestamp would be better in version 2.0.4, but some sentence would be too long and lack of punctuation。
The audio speak a little fast, does this have something to do with it？
audio file
code:
model = stable_whisper.load_model('large-v2')
result = model.transcribe('audio.mp3')
result.to_dict()["segments"] // use segments to generate srt

compare:
stable-ts                2.0.4
openai-whisper           20230314

5
00:00:31,460 --> 00:00:41,940
绷不住了咱就是说这是逼boss嘛我要真属驴就好了驴不会感染病毒别忘了咱俩的身份咱现在可都是阳过这个家能随便回的吗

6
00:00:42,100 --> 00:00:50,360
春晚小品不是有句话吗宁可相信世上有鬼也不能相信男人破嘴我看你就是渣男不想对我负责

7
00:00:50,540 --> 00:00:57,900
不愿意带我见家长家人们我真是纯纯大冤种啊你没看见我现在群里聊天都往手机上喷酒精

8
00:00:58,120 --> 00:01:18,500
好家伙病毒还通过网络传播呀这叫警惕新冠病毒打赛博朋克牌现在政策放开了咱俩病也好了也该回家了吧我还是担心害怕呀那你更得回家了有个大小者曾经说过一句话什么话战胜恐惧最好的办法就是面对它加油oligane

9
00:01:18,840 --> 00:01:20,680
那小者是研究东勇的吧

10
00:01:20,900 --> 00:01:22,880
丁总爸爸我们回来了

no stable-ts

2
00:00:14,480 --> 00:00:16,320
养了得有半月多

3
00:00:16,320 --> 00:00:18,040
踏上回家的列车

4
00:00:18,040 --> 00:00:19,760
门口徘徊不进屋

5
00:00:19,760 --> 00:00:21,960
到底为啥你说一说

6
00:00:21,960 --> 00:00:23,160
和你回来之前

thanks for your job.

Have you tried 2.1 yet? It will be far less likely to regroup into such long segments. There's also the option to use the original segments grouping with regroup=False.


Automated mortar

@ammoore00 commented on Sep 21
As per title.

@Hirachosan commented on Sep 21
I propose (regular?)dispensers facing a block that is mortarable can inject mortar into that block. this would make it possible to fully automate a process using block dispensers and regular dispenser and some form of a timer

https://github.com/BTW-Community/BTW-source/issues/85

Done for v1.1.0

Curious as to what you chose here! Care to elaborate?

I just did exactly what you proposed :) A couple other people have also suggested the same thing and I thought it was a good way to go.

so the regular dispenser and facing directly towards the loose block? :)

Yep :)


Set maui-linux as default branch

Description
You declared maui-linux as the branch you accept pull request for. This should be the default branch as in the GitHub Repository settings then.
Steps to Reproduce
just go to https://github.com/jsuarezruiz
Version with bug
Unknown/Other (please specify)
Last version that worked well
Unknown/Other
Affected platforms
I was not able test on other platforms
Affected platform versions
n/a
Did you find any workaround?
No response
Relevant log output
No response

Good suggestion, changed.

And what's the reason for using a different branch than master? Is it to make merges with upstream MAUI easier?


Discard features that result into "cheating"

Going briefly over the SHAP results showed that features unrelated to cryptography are actually the most influential on the detection models. Namely, we talk about anything related to file size of the sample. While we cannot at the moment fully normalize all features (as we don't have LoC numbers for all samples, see #9) for the already processed dataset, we should do so in the Avast dataset.
We should for now drop the n_classes feature. Some of the information will still leak into the other features, but we can't easily fix that now.
By the way, @dmacko232, what exactly does file_sqrt_count feature does? Should we drop it as well?
Possibly, in the paper we'll use the Androzoo dataset merely to describe usage of crypto API by malware authors and we'll leave the new Avast dataset for malware classifiers. We can correctly normalize the data on the new dataset.
Still, it is an interesting observation that while malicious samples are in general much smaller in size, they contain much more crypto. We should explore the Avast dataset. My current hypothesis now is that: Despite malware samples contain fewer lines of code, they still contain more cryptography.

I already dropped the n_classes feature.
file_sqrt_count (the name is not really descriptive) represents square root count of classes (files) that use crypto API (so it's kinda related to crypto - but maybe the file size can leak through this feature). Square root was used because the distribution of the feature was skewed.
Overall, I think we should go over the features once again and decide if any of them need to be dropped.


build: Generate TypeScript types from OpenAPI definition

Requires:

[x] https://github.com/bunqCommunity/bunqJSClient/pull/58 for the tests to pass

This PR will add a script to automatically generate TypeScript types from bunq's OpenAPI definition.
It looks like not all required properties are marked as such in the OpenAPI definition, so there is a way to add these via the script.
To prevent types from being flattened to just primitives such as string, there is the possibility to extend the definition as well.
Generating a new definition file can be done with yarn build:types. Definitions end up in the file src/Types/ApiTypes.ts
A backward compatible layer has been added.

Can this be merged? Would very much like to use this 😄


Virtual Slides issue with swipe/touch in FreeMode

This is a (multiple allowed):

[x] bug

[ ] enhancement

[ ] feature-discussion (RFC)

Swiper Version: 4.3.5

Platform/Target and Browser Versions: Android Chrome and/or probably any touchscreen device

Codepen with issue: https://codepen.io/michiel-huiskens/pen/OEpVNb?editors=0100

What you did
Using Virtual Slides with “FreeMode:true” works perfectly smooth for mouse-wheel/touchpad scroll and scrollbar interactions, but when swiping (or “scrolling” on touch devices) the “soon to be out of view” slides are removed while still in view, immediately on release.
Expected Behavior
Expecting the same behaviour on touchscreen devices as when using mouse-wheel/touchpad.
Actual Behavior
When swiping (or “scrolling” on touch devices) the “soon to be out of view” slides are removed while still in view, immediately on release.

Well it wasn’t intended to be used with free mode and I didn’t know it works with it) The problem here is that during mouse scroll or scroll bar scroll we have an event with every tick where Virtual Slides recalculated, but with long touch swipe we have just initial and final states so we can’t recalculate it during transition.

I may introduce new parameter that can add control of how many Virtual slides must be pretendered

@nolimits4web I believe you can get the events in every tick by listening to touchstart + touchmove (or pointerdown + pointermove with a Pointer Events polyfill) on the same element you listen for the scroll event. Just beware that if you listen to pointermove you'll also get mouse movements unlike touchmove. Do you think it would be possible to use any of this to make virtual slides work with freeMode: true?

@nolimits4web I ask because I'm facing a use case in which I would need virtual slides with freeMode: true to solve performance issues in a long list. If this is not a possibility, can you please point to any other good library that provides virtualization of lists in a similar way?


Site support for Ekeeda.com

Checklist

[x ] I'm reporting a new site support request
[x ] I've verified that I'm running youtube-dl version 2020.03.08
[x ] I've checked that all provided URLs are alive and playable in a browser
[x ] I've checked that none of provided URLs violate any copyrights
[x ] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Description

Please add site support for www.Ekeeda.com

Why is this closed?


update debugid to reflect TYPES of fluff & name tokens

Auto
Manual (from sheet)
Inferred (from the tree from the sheet)

we still don't have "inferred (from tree from the sheet)" anywhere

Example: Roche getting the fact that her name is Roche from her equip is not shown anywhere


Network handoff to/from a NAT'd IPv6 network

This is an enhancement we are making to our network handoff capabilities in preparation for our first General Availability release.
Problem Description
A Client Alice connects to a Room from a NAT'd IPv6 network. The connection succeeds and at a later time Alice switches to another network. The handoff to the new network fails, and eventually a room:didDisconnect callback is raised.
The same issue occurs when Alice connects over an IPv4 network, and then attempts to transition to a NAT'd IPv6 network. The NAT mapping needs to be re-established in order to open a connection on the new network.

We tried our best to deliver this fix in time for our 1.0.0 release. We are very close, but it looks like this enhancement will not land until 1.0.1.

1.0.1 of Twilio Video has just been released which addresses this issue.


Node.js with Azure portal does not work

I'm deploying a node app as explained in the doc but it doesn't work. URL loads forever.
I'm using the Azure portal option.

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

ID: 298b87f4-b6eb-5642-7ab5-cd6df709390d
Version Independent ID: 0518cce4-7677-6d63-979d-e0a9e532efc0
Content: Quickstart: Create a Node.js web app - Azure App Service
Content Source: articles/app-service/quickstart-nodejs.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@kengaderdus,  Thanks for the question! We are taking a look into this and will get back to you soon.


WSL2 seems not support traffic control by tc qdisc

Is your feature request related to a problem? Please describe.
When I run tc qdisc add dev eth0 root netem delay 200ms, there would comes error Error: Specified qdisc not found.
Noticed someone else also have the same problem.
Describe the solution you'd like
If I'm not wrong, the reason is due to related kernel config is not set. A recommended config for this feature would be:
CONFIG_NET_SCHED=y
CONFIG_NET_SCH_CBQ=m
CONFIG_NET_SCH_HTB=m
CONFIG_NET_SCH_CSZ=m
CONFIG_NET_SCH_PRIO=m
CONFIG_NET_SCH_RED=m
CONFIG_NET_SCH_SFQ=m
CONFIG_NET_SCH_TEQL=m
CONFIG_NET_SCH_TBF=m
CONFIG_NET_SCH_GRED=m
CONFIG_NET_SCH_DSMARK=m
CONFIG_NET_SCH_INGRESS=m
CONFIG_NET_QOS=y
CONFIG_NET_ESTIMATOR=y
CONFIG_NET_CLS=y
CONFIG_NET_CLS_TCINDEX=m
CONFIG_NET_CLS_ROUTE4=m
CONFIG_NET_CLS_ROUTE=y
CONFIG_NET_CLS_FW=m
CONFIG_NET_CLS_U32=m
CONFIG_NET_CLS_RSVP=m
CONFIG_NET_CLS_RSVP6=m
CONFIG_NET_CLS_POLICE=y

If I want to contribute to this feature, do I modify wsl-config directly? It seems it has been stale for a long time.

Following https://github.com/microsoft/WSL2-Linux-Kernel/issues/182#issuecomment-691286846, it seems I could change it locally. Do I still need to leave this issue here?

I have compiled a kernel with these configs. If it's needed, I'm willing to submit a PR for this issue.

Do I still need to leave this issue here?

Short answer yes. At least until how kernel modules is handled in WSL more generally, they're being done one =y request at a time.

We have a similar issue.
Any eta on this?

We have a similar issue.
Any eta on this?

You could try to compile the kernel yourself, with several config enabled

also stumbled upon this.
any reason there isn't a PR for this, yet?

I also face this problem. Please help...

I built a cusom kernel that enable tc to add latecy, but got error when using tbf. I enable tbf in my wsl-config file (CONFIG_NET_SCH_TBF=y). Does anyone know why?
After typing this command.
tc qdisc add dev eth0 root tbf rate 1mbit burst 32kbit latency 400ms
I got the following error...
Error: Specified qdisc not found.

3 years passed, why there's no support for this by default?

This feature has been enabled by default on any kernel I've used in the past. Can this please be enabled for WSL as well. It's incredibly useful to add arbitrary latency to connections, so that you can test programs in those situations without actually needing to have bad internet.

wsl2 can't use linux tc. It's really unconvenient.

Building your own kernel like @Yiyiyimu suggested indeed works around the problem btw. But that's quite annoying to have to do.

bump

bump, the other issue has a broken link.

bump

I did enable the netem module as built-in with make menuconfig  ..... but the tc command still didn't work .


Asserting New Error Format?

With the pre 2.0 version of Ecto, I had the following code:
assert {:username, "can't be blank"} in errors_on(%User{}, %{username: nil})

where errors_on looked like this:
def errors_on(model, data) do
  model.__struct__.changeset(model, data).errors
end

but now, with the new format of errors, I get the following error from my assertion:
Assertion with in failed
     code: [:username, "can't be blank"] in errors_on(%User{}, %{username: nil})
     lhs:  [:username, "can't be blank"]
     rhs:  [username: {"can't be blank", []}]

(note the rhs is what is now returned from the changeset errors in Ecto 2.0).
How can I assert that my changeset contains {:username, "can't be blank"} now? I've tried various configurations of maps and keyword lists. I'm brand new to Elixir, so be gentle. :)

@jamescmartinez you can manually extract the errors in your errors_on function, perhaps something like this:
errors |> Enum.map(fn {field, {message, _opts}} -> {field, message} end)

You can also look into Ecto.Changeset.traverse_errors/1 which allows you to conveniently pluck out error messages from a changeset with a function.
P.S. please follow the issue template and not use the tracker for support requests. Instead, ask questions on SO/ML so that any answers given will be available to broader audience and the Core team can focus on real issues.

Thanks @wojtekmach, I'll close this issue and use SO or the ML. Sorry!


Modulus operator

I think it would be nice if xtensor supported operator% for xexpressions.  This requires a slightly more complicated implementation than the other arithmetic operators, since one can't use % with non-integer operands.
My first attempt at implementing this PR failed.  I had attempted to selectively disable the modulus function in xsemantic.hpp (and elsewhere) for non-integer types via enable_if, but it turns out that doesn't work.  The xexpression_type is "incomplete" (since the xsemantic hierarchy employs the CRTP), so enable_if can't be used at that level.  (Unless I'm missing something... I'm not an expert.)
Instead, this PR uses a simpler solution.  The modulus operation exists for all xexpression (even non-integer ones), but the implementation for non-integer types throws an exception at runtime.
(Except for the code in xoperation.hpp, the rest of this PR is merely copied from the other arithmetic operators.)

BTW, I realize the above-mentioned complexity is probably why you didn't bother implementing operator% in the first place.  In case you're wondering, my motivation for adding it is that % is often useful when performing arithmetic on array shapes.

@stuarteberg thanks for the pull request! I don't think that there was a particular reason why operator% would not be in xtensor. It is probably that it had not come up before!
Regarding the disabling of operator% for non-integral types, I am not sure that it is necessary to do the disabling at all. After all, if x % y should not compile because the scalar version does not exist, then we may let it not compile. Disabling is important when we want to remove a specialization from the overload set.

I am not sure that it is necessary to do the disabling at all.

Whoops, you're right.  Thanks.  Now I've changed this PR to simply mimic the other operators.  (I guess I was confused by a bug in my test code...)

Thanks!

It seems I did not validate my previous comment (since I cannot see it here), so here it is again:
Nice PR, thanks for this feature!


Develop PyTorch interface

Figuring out the best way to include PyTorch models in autoemulate.

Each model should be a scikit-learn estimator so that it's compatible with the wider ecosystem

For PyTorch, an easy way to do this is to use skorch. But in the current implementation in neural_net_torch.py it fails the tests in test_estimators. I think this is because it's currently wrapping the skorch wrapper, as we need the class to have a few other methods such as get_grid_params and _more_tags which defines which tests to run, see again here

Closing this as resolved by @bryanlimy's PR #107


Generates Mgmt Storage: missing ApiVersion parameter

Generating:
https://github.com/Azure/autorest.python/blob/add_multiapi_client/test/services/azure-mgmt-storage/README.md
All operations are missing api-version as query parameters

Refactoring API version handling entirely -- see # 173

Working now, don't ask me why....


Update Composer dependencies (2020-10-24-00-09)

Visual regression test passed!
View the visual regression test report


Retry introduced significant delays in VCN operations

After addition of retries VCN operations became much slower.
Commit before retry changes:
$ go get github.com/vchain-us/ledger-compliance-go@dadda9f4b5514c85f2fed4f78eaa332b58ac89a1
...
$ make
...
$ time ./vcn a vcn

Warning: 008990c65fa433ed7b0696990da7306a3553a538653c34a4761d1e8b5e65794d was not notarized

real	0m0.186s
user	0m0.159s
sys	0m0.023s

Last commit with retry changes:
$ go get github.com/vchain-us/ledger-compliance-go@309f3c068f0050c37556bf2e423b40752c8e0063
...
$ make
...
$ time ./vcn a vcn

Warning: 1fc99bf465f0c8cf560b9f6a7cc4a44d0a721d09aaaf436e6175c8d4c8c2556c was not notarized

real	0m5.009s
user	0m0.144s
sys	0m0.063s

In dataservice I see that VCNGetArtifacts is called 3 times instead of 1 (as it was before the change). Everything (immudb, dataservice, vcn) is running locally so there is no reasons for retries. This problem occurs every single I run vcn a or vcn n.

https://github.com/codenotary/immudb/issues/1255

@qrdl , let's disable retry and by default have 0 for now in vcn. We discussed and this will need to be handled in immudb most probably (cc @byo , @ldej , @raducrisan1 , @joe-mann )
https://github.com/codenotary/vcn-enterprise/blob/5169af3bd38b9fab9741e03544bcc0c69d29f580/pkg/cmd/cmd.go#L96

@qrdl , let's disable retry and by default have 0 for now in vcn. We discussed and this will need to be handled in immudb most probably (cc @byo , @ldej , @raducrisan1 , @joe-mann )

The branch I'm working on was branched off develop before retries were introduced, so there is no retry-attempts param at all, but it still retries 3 times because it is a default set in ledger-compliance-go: https://github.com/vchain-us/ledger-compliance-go/blob/master/grpcclient/lc_client.go#L164
Until I merge my changes I cannot change this parameter.

@qrdl , let's disable retry and by default have 0 for now in vcn. We discussed and this will need to be handled in immudb most probably (cc @byo , @ldej , @raducrisan1 , @joe-mann )

The branch I'm working on was branched off develop before retries were introduced, so there is no retry-attempts param at all, but it still retries 3 times because it is a default set in ledger-compliance-go: https://github.com/vchain-us/ledger-compliance-go/blob/master/grpcclient/lc_client.go#L164 Until I merge my changes I cannot change this parameter.

Ah ok, so we need to change default in ledger complaiance.

@raducrisan1 after merging https://github.com/vchain-us/ledger-compliance-go/pull/61, please do not forget to update dependency to ledger compliance go in vcn-enterprise develop branch.

A pull request has been created on vcn-enterprise, checks passed, approved and merged to develop. Link to PR: https://github.com/codenotary/vcn-enterprise/pull/452

Issue has been fixed


Added .gitignore and untrack node_modules

I have made the necessary changes to remove node modules and stopped git from tracking it further  . Issue #7

Hi @AKSHATH99,
Thank you for your PR and for addressing the issue of adding a .gitignore file to untrack node_modules. However, it seems that the PR removes a large number of files (around 5000+), which could indicate that the node_modules folder was mistakenly included in the commit history.
To resolve this:
Please make sure the node_modules folder is excluded properly using the .gitignore file.
If the folder was accidentally tracked, you can remove it from the repository history by running the following command before committing again:
git rm -r --cached node_modules
Once done, recommit and push the changes. This should reduce the file changes to only the .gitignore file.

Hey , i actually ran that command and  git is no more tracking those .
The reason its showing 5000+ files is that i accidentally deleted node modules first , then ran the command , so it tracked the deletion . I should have just ran the command .
It no more tracks node_modules   i think

Hey , i actually ran that command and git is no more tracking those .
The reason its showing 5000+ files is that i accidentally deleted node modules first , then ran the command , so it tracked the deletion . I should have just ran the command .
It no more tracks node_modules i think

Please make sure that node_modules/ is included in the .gitignore file. This will prevent any future changes to this directory from being tracked by Git.

Its in there . I think now you can merge it

Hi @AKSHATH99
Could you please test it locally after setting up the project on your machine? I have another ticket related to this for Hacktoberfest, and there are some deployment issues with the website. It would be really helpful if you could verify everything. Would you be able to assist with testing?

Yeah , what exactly is the issue ? I could look up on it . Is the node module issue PR merged ?

Yeah , what exactly is the issue ? I could look up on it . Is the node module issue PR merged ?

Please note that the node_modules directory cannot be ignored. Ensure that it's included, then commit the changes along with the required updates.

I didnt get what you meant . Technically node_modules should be ignored by git


Payload still buffered between each message

I just found out that the payload doesn't "flush" between each message being received when you're subscribed.
Explanation:
If you're subscribed to something, and you receive this payload:
ZZZZZZZZZZ
The callback function is going to receive a length of 10, and the described payload. But, if you send another message after that one, like this:
A
The callback function is going to receive a length of 1, but the payload is going to be this:
AZZZZZZZZZ
(Is going to replace the first character, but leaving the remaining characters from the previous message).
I don't know if that's a bug, but I just found out that :)
Using v2.6.0

Hi,
you are right that the internal buffer is not cleared between operations.
This is why you must pay attention to the length parameter passed in the callback - it tells you the length of the payload array for this message. If you access payload beyond length then you're accessing data unrelated to the message.


Verify Config file parameters

It has been found that some values of parameters within SimEng config files cause stalling or memory leaks to SimEng, for example Vector-Length causes the program to hang if it is set higher than the Load or Store bandwidth of LSQ-L1-Interface. Fetch-Block-Size also causes a memory leak at values greater than 128.
Different parameter sets need further investigating, with underlying issues patched or bounds on input ranges tightened.

Issue fixed in config-bug-fixes PR as part of 0.9.6


Document browser support

The documentation does not say anything about browser support. As such, I initially dismissed marked as an option for my project.
However, apparently it does have browser support:

https://github.com/chjj/marked/issues/62
https://github.com/chjj/marked/blob/master/lib/marked.js#L783-791

but this is useless unless this is advertised.
Additionally, minified versions of the library should be available for use in the browser.

Seems to have been resolved by merge. Closing.


Extend the debugging points of Authentication flow

Currently, there are only a limited amount of debuggable points in the authentication flow (Start and end of the protocol). Adding more debuggable locations such as processes inside the authentication framework would give a developer much more understanding of the flow.

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Avoid needing explicit call to resize on computer

Just check if resolution has changed and resize textures if so.

I tried to do this and it was surprisingly tricky so I gave up.


Crash with Sinytra Connector

I know that the Connector and your mod aren't tied at all and potentially should not conflict, however, with such a simple combination, Minecraft crashes at the download stage even without a crash report (although Forge often does so):

Forge 1.20.1, the latest versions used.
Here are very short logs that appeared before the crash (though it doesn't seem to me to mean anything):
[27Mar.2024 23:12:17.975] [main/INFO] [cpw.mods.modlauncher.Launcher/MODLAUNCHER]: ModLauncher running: args [--username, Korben_Dream, --version, Forge 1.20.1, --gameDir, C:\Users\User\AppData\Roaming\.minecraft, --assetsDir, C:\Users\User\AppData\Roaming\.minecraft\assets, --assetIndex, 5, --uuid, bb98d0f3ee384e998f8180c26ef5636f, --accessToken, ❄❄❄❄❄❄❄❄, --clientId, , --xuid, , --userType, mojang, --versionType, modified, --width, 960, --height, 540, --launchTarget, forgeclient, --fml.forgeVersion, 47.2.21, --fml.mcVersion, 1.20.1, --fml.forgeGroup, net.minecraftforge, --fml.mcpVersion, 20230612.114412]
[27Mar.2024 23:12:17.979] [main/INFO] [cpw.mods.modlauncher.Launcher/MODLAUNCHER]: ModLauncher 10.0.9+10.0.9+main.dcd20f30 starting: java version 17.0.8 by Microsoft; OS Windows 10 arch amd64 version 10.0
[27Mar.2024 23:12:18.167] [main/INFO] [net.minecraftforge.fml.loading.ImmediateWindowHandler/]: Loading ImmediateWindowProvider fmlearlywindow
[27Mar.2024 23:12:18.250] [main/INFO] [EARLYDISPLAY/]: Trying GL version 4.6
[27Mar.2024 23:12:18.433] [main/INFO] [EARLYDISPLAY/]: Requested GL version 4.6 got version 4.6
[27Mar.2024 23:12:18.484] [main/INFO] [mixin-transmog/]: Mixin Transmogrifier is definitely up to no good...
[27Mar.2024 23:12:18.513] [main/INFO] [mixin-transmog/]: crimes against java were committed
[27Mar.2024 23:12:18.541] [main/INFO] [mixin-transmog/]: Original mixin transformation service successfully crobbed by mixin-transmogrifier!
[27Mar.2024 23:12:18.594] [main/INFO] [mixin/]: SpongePowered MIXIN Subsystem Version=0.8.5 Source=union:/C:/Users/User/AppData/Roaming/.minecraft/mods/Connector-1.0.0-beta.40+1.20.1.jar%23159%23162!/ Service=ModLauncher Env=CLIENT
[27Mar.2024 23:12:18.629] [pool-2-thread-1/INFO] [EARLYDISPLAY/]: GL info: NVIDIA GeForce GTX 1060 6GB/PCIe/SSE2 GL version 4.6.0 NVIDIA 551.86, NVIDIA Corporation
[27Mar.2024 23:12:19.189] [main/WARN] [net.minecraftforge.fml.loading.moddiscovery.ModFileParser/LOADING]: Mod file C:\Users\User\AppData\Roaming\.minecraft\libraries\net\minecraftforge\fmlcore\1.20.1-47.2.21\fmlcore-1.20.1-47.2.21.jar is missing mods.toml file
[27Mar.2024 23:12:19.196] [main/WARN] [net.minecraftforge.fml.loading.moddiscovery.ModFileParser/LOADING]: Mod file C:\Users\User\AppData\Roaming\.minecraft\libraries\net\minecraftforge\javafmllanguage\1.20.1-47.2.21\javafmllanguage-1.20.1-47.2.21.jar is missing mods.toml file
[27Mar.2024 23:12:19.207] [main/WARN] [net.minecraftforge.fml.loading.moddiscovery.ModFileParser/LOADING]: Mod file C:\Users\User\AppData\Roaming\.minecraft\libraries\net\minecraftforge\lowcodelanguage\1.20.1-47.2.21\lowcodelanguage-1.20.1-47.2.21.jar is missing mods.toml file
[27Mar.2024 23:12:19.218] [main/WARN] [net.minecraftforge.fml.loading.moddiscovery.ModFileParser/LOADING]: Mod file C:\Users\User\AppData\Roaming\.minecraft\libraries\net\minecraftforge\mclanguage\1.20.1-47.2.21\mclanguage-1.20.1-47.2.21.jar is missing mods.toml file
[27Mar.2024 23:12:19.857] [main/WARN] [net.minecraftforge.jarjar.selection.JarSelector/]: Attempted to select two dependency jars from JarJar which have the same identification: Mod File:  and Mod File: . Using Mod File:
[27Mar.2024 23:12:19.862] [main/INFO] [net.minecraftforge.fml.loading.moddiscovery.JarInJarDependencyLocator/]: Found 48 dependencies adding them to mods collection
[27Mar.2024 23:12:20.100] [main/INFO] [dev.su5ed.sinytra.connector.locator.DependencyResolver/]: Dependency resolution found 0 candidates to load
[27Mar.2024 23:12:21.726] [main/INFO] [dev.su5ed.sinytra.connector.service.hacks.ModuleLayerMigrator/]: Successfully made module authlib transformable
[27Mar.2024 23:12:23.341] [main/INFO] [mixin/]: Compatibility level set to JAVA_17
[27Mar.2024 23:12:23.466] [main/INFO] [cpw.mods.modlauncher.LaunchServiceHandler/MODLAUNCHER]: Launching target 'forgeclient' with arguments [--version, Forge 1.20.1, --gameDir, C:\Users\User\AppData\Roaming\.minecraft, --assetsDir, C:\Users\User\AppData\Roaming\.minecraft\assets, --uuid, bb98d0f3ee384e998f8180c26ef5636f, --username, Korben_Dream, --assetIndex, 5, --accessToken, ❄❄❄❄❄❄❄❄, --clientId, , --xuid, , --userType, mojang, --versionType, modified, --width, 960, --height, 540]
[27Mar.2024 23:12:26.630] [main/INFO] [MixinExtras|Service/]: Initializing MixinExtras via com.llamalad7.mixinextras.service.MixinExtrasServiceImpl(version=0.3.5).
[27Mar.2024 23:12:32.238] [Datafixer Bootstrap/INFO] [com.mojang.datafixers.DataFixerBuilder/]: 188 Datafixer optimizations took 337 milliseconds

Issue in the Connector repository:
https://github.com/Sinytra/Connector/issues/944

The current version of Neruina forge does not work with forge versions over 47.2.19
This is fixed in a future update.

It's sad for 1.20.1 as the most common version for mods among 1.20.X

The next version of neruina will maintain support with every version back to 1.18.2


Support changing event type to listen to

I have a use case where I'd like to listen to outside mouse releases, not presses. There's many ways to do it, but the least intrusive is to just allow a prop that sets the evt type to listen on. Thoughts?
Tested and works fine on desktop and mobile.

while a small change, that does change the mixin into a generic "onAnythingOutside", so might actually be worth just spinning off as its own, more generic, mixin.

I could live with it, but I'd rather not make a new repo that is 99% similar to this one. Are you a fan of this solution?

no, I don't want to have a project that's called "onclickoutside" lie about what it does. However, what we can do is start react-onoutsideevent, which I just have. If you want to refile your PR against that then we can continue work there, and eventually retire this project instead.

Fair enough, moved to https://github.com/Pomax/react-onoutsideevent/pull/1 .


Flaky visual corruption in Milestone-Jefferson

I've had a play with the Milestone-Jefferson now. However, on Weston (Intel G45 graphics) there is some mild image corruption evident, seemingly related to the titlebar. I haven't seen anything like it before, either in earlier releases that I've tried, or with xwayland clients.

The corruption appears to be related to the tab animations, although it has appeared elsewhere. I'm not sure what causes it.
I'd be happy to test out changes, although it takes a while for my machine to rebuild everything.

can u try out latest master and see if you can see this issue ?

Will do.
There will be a lag of a few days; I'm going to try to reproduce the libsecret issue first, then update the PKGBUILD.

I might also have a play around with less recent versions of the kernel/mesa/chromium-wayland to see whether that changes anything, although I haven't seen it with xwayland or wayland-terminal.

This is still an issue with not-quite-latest (yesterday's) master.
I haven't had a chance to play around with less recent versions of the kernel/mesa/chromium wayland, though, so it might not be a chromium-wayland issue.

This is still an issue, although the same caveat as above applies... hopefully I'll get around to this at some point.

Mesa regression between 10.3 and 10.6; I haven't investigated further.
Sorry for the noise!

As a note, there's this error in the logs for both my Milestone-Jefferson and Milestone-Trask builds:
[2940:2940:0825/161104:ERROR:texture_manager.cc(1980)] [GroupMarkerNotSet(crbug.com/242999)!:D092BDA995130000]GL ERROR :GL_INVALID_VALUE : glTexImage2D: <- error from previous GL command

It might be related...

Hmm, ya. So, you are using Mesa 10.6. I can give it a try and see if I can replicate this. Let me know, mesa wayland and weston versions.

Mesa: 10.6.5
Wayland: 1.8.1
Weston: 1.8.0


Ticket 24944

squash all the commits in a single commit according to guideline

Can also add tests for this?

Hi,
I am not sure how to add tests for this feature. Can you guide me in how to
go about doing it?
On Mon, Aug 3, 2015 at 10:14 PM, Simon Charette<EMAIL_ADDRESS>wrote:

Can also add tests for this?
—
Reply to this email directly or view it on GitHub
https://github.com/django/django/pull/5094#issuecomment-127326957.

Take a look at tests/auth_tests/test_views.py for how tests for the other parameters are done.

Hi Tim,
In the tests for auth views, there are no tests for parameters like
extra_context.
Why is that?
Also, if extra_context cannot be tested, how can extra_email_context be
tested? As they are very similar.
On Mon, Aug 3, 2015 at 11:13 PM, Tim Graham<EMAIL_ADDRESS>wrote:

Take a look at tests/auth_tests/test_views.py for how tests for the other
parameters are done.
—
Reply to this email directly or view it on GitHub
https://github.com/django/django/pull/5094#issuecomment-127344224.

There is no good reason why extra_context is not tested. Pretend you are using the feature. There is a test in admin_views for a similar extra_context parameter there.

The test isn't testing this feature. If I revert your changes in django/contrib/auth/views.py I'm pretty sure the test will still pass.

@timgraham Should I send the extra_email_context to send_mail function too?

It should be merged with the context dictionary: https://github.com/django/django/blob/7efdd40407838dc3c922e0e62e9b4eefabe7de01/django/contrib/auth/forms.py#L246-L254

Hi Tim,
I've done it...Any other changes?
On Tue, Aug 25, 2015 at 9:07 PM, Tim Graham<EMAIL_ADDRESS>wrote:

It should be merged with the context dictionary:
https://github.com/django/django/blob/7efdd40407838dc3c922e0e62e9b4eefabe7de01/django/contrib/auth/forms.py#L246-L254
—
Reply to this email directly or view it on GitHub
https://github.com/django/django/pull/5094#issuecomment-134632100.


Feature rename keys

This feature allow rename all keys defined by user. Reference issue #30.

Thank you so much :)


fix: update undici

🚨 IMPORTANT: Please do not create a Pull Request without creating an issue first.
Any change needs to be discussed before proceeding. Failure to do so may result in the rejection of the pull request.
Description
Reason for pinning was fixed in Undici 5.7.0, and 5.8.0 has security fixes.
Related #52

Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Screenshots/Sandbox (if appropriate/relevant):
N/A
How Has This Been Tested?
N/A

[ ] Test A
[ ] Test B

Test Environment:
N/A

OS:
package-name:
NodeJS:

Checklist:

[ ] I have followed the CONTRIBUTING doc and the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests and linter rules pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Undici has been updated in the latest release.


Updated website

This PR fixes #29, and #39 by adding a live demo to the website. In order to do this I made the Jekyll template Bootstrap 3 compatible. I migrated all dependencies over to CDN except for jQuery.hotkeys. Since jQuery.hotkeys does not have a CDN I added it using bower so that it does not have to be managed manually.
Finally, I have updated the website so that the basic usage now contains all the information on the Mindmup website. This way all the information is contained on one page and doesn't have to point to the Mindmup website other than to show that this repo was originally made by them.

This is good to go. I've merged it in. Thanks for the great work on this @RandomlyKnighted


Add a skeleton of the pytest plugin

Currently broken because there are no tests.

From this list it looks like we should name the project pytest-selfie to follow the naming convention.

How I did this (roughly):

use this project to setup the template

https://github.com/pytest-dev/cookiecutter-pytest-plugin

convert it to poetry
this blog post was helpful: https://blog.pecar.me/pytest-plugin
docs on writing plugins: https://docs.pytest.org/en/7.1.x/how-to/writing_plugins.html
docs on writing hook functions: https://docs.pytest.org/en/7.1.x/how-to/writing_hook_functions.html

In the end we have:

pytest-selfie/pytest_selfie/plugin.py

pytest_sessionstart - called before all tests
pytest_sessionfinish - called after all tests
pytest_pyfunc_call - called before test, yield, after the test

example-pytest-selfie - playground where we can use pytest-selfie like an end user