r2.12 cherry-pick: 46cdedf47c1 "Include TSL proto files in the pip package"

Refer to the original commit: https://github.com/tensorflow/tensorflow/commit/46cdedf47c1f547d3c4ff9e3f63c12b7e7ec5a22

This cherry-pick resolves https://github.com/tensorflow/tensorflow/issues/59514: TF 2.12 is missing TF Core .proto files that were moved to TSL in between 2.11 and 2.12. Without these files, external plugins such as the DirectML plugin cannot be compiled, which is a pretty notable regression.
There are no side effects to this change except that missing static .proto files are once again included inside the TF Python wheel. They do not change any behavior as far as I know.


discord join link is expired

not an issue with the theme, but i cant join your discord since all of the links i found on your repos are all expired.

thank you <3 fixed


Bus 'ide.0' not found when creating a new virtualized VM (v3.1.3 preview)

Describe the issue
When creating a new "virtualized" Linux VM through UTM using the provided defaults (Virtualize->Linux->Boot ISO Image->4096MB, default CPU cores, no GPU accel->64GB->no shared dir), two things happen:

ARM64 (aarch64) is chosen, but no system is chosen, causing the error "Use -machine help to list supported machines" upon launch.
When you choose "virt" as the system (alias of virt-6.2), an error occurs when launching the VM: 'qemu-aarch64-softmmu: -device ide-cd,bus=ide.0,drive=cdrom0,bootindex=0: Bus 'ide.0' not found'

This worked in earlier revisions of UTM.
Configuration

UTM Version: v3.1.3 (51) preview
OS Version: 12.2.1
Intel or Apple Silicon? Apple Silicon

Crash log
None provided
Debug log
debug.log
Upload VM
config.plist.zip

Duplicate of #3722

Will be fixed next update.


Stale link in README

The README points to the #tock IRC channel on freenode, but the channel looks completely dead.
I'm happy to hang out on #tock at libera.chat (as I do now), or set up some Matrix channel, if anyone wants to change the link. Otherwise I would suggest dropping the link altogether.

The Tock slack is where people mostly hang out now: https://join.slack.com/t/tockos/shared_invite/enQtNDE5ODQyNDU4NTE1LWVjNTgzMTMwYzA1NDI1MjExZjljMjFmOTMxMGIwOGJlMjk0ZTI4YzY0NTYzNWM0ZmJmZGFjYmY5MTJiMDBlOTk

+1 for a Matrix channel :)


Assorted fixes

Assorted fixes after the completion of the internal hackathon.

@usrinivasan Can you take a look at this? You'll have to modify https://github.com/xeno-by/scalameta/blob/f837409b670e8cfcc1a4f923d9b18c9ea63fb4f1/scalameta/semanticdb-scalac/src/main/scala/scala/meta/internal/SemanticdbPipeline.scala#L52-L65.

Any chance to get this PR in before we release 2.0.2? cc/ @ShaneDelmore

Closing this PR.  I will make another from my branch with the requested changes.


feature request: block scoped macros

why
Yuepack which I developed in response to #173 puts all imported file in do blocks, as such macros in them sadly don't work.
example code
do
  macro test = -> "123"
  print $test

current behavior
compilation error
2: can not define macro outside the root block
  macro test = -> "123"

expected behavior
a block scoped macro
interesting workaround
macro imports do work in nested blocks
-- main.yue
do
  import "macros" as $
  print $test

-- macros.yue
export macro test = -> "123"

although they arent bound to the block and leak upwards
do
  import "macros" as $

print $test -- still works

Implementing the macro system in the root block in a module was just for simpler coding, but now the block-scope-aware macro system was done in the newer commit. So that you can now write a "local macro", and importing "macros" in the block scope.

Thanks! seems to be working perfectly!


Cache captured_inputs

Benchmarks over 3 batches of 10 trials each
Minimum
defun_matmul_2_by_2_CPU: 156 -> 155 us
defun_matmul_2_by_2_CPU_async: 84 -> 82.5 us
defun_matmul_2_by_2_with_signature_CPU: 206.5 -> 201 us
Mean
defun_matmul_2_by_2_CPU: 177 -> 173 us
defun_matmul_2_by_2_CPU_async: 86.5 -> 85 us
defun_matmul_2_by_2_with_signature_CPU: 223 -> 233 us
Median
defun_matmul_2_by_2_CPU: 176 -> 173 us
defun_matmul_2_by_2_CPU_async: 85.5 -> 83.5 us
defun_matmul_2_by_2_with_signature_CPU: 224 -> 224 us

Please take a look at these Ubuntu CPU CI fails and see if you can reproduce https://source.cloud.google.com/results/invocations/f185b429-a41f-471c-9f59-9deb5a016144/targets/%2F%2Ftensorflow%2Fpython%2Feager:function_test/tests;group=main.MultiDeviceTest;test=testDeferredCaptureTypeError;row=1

Closing this PR because updates to the return values of the function calls x() cannot be appropriately cached. testDeferredCapture is an example


Certificate instead of clientSecret

Hello,
i want to know if it's possible to use a certifacte instead of a clientSecret?
Regards
.nowrap

I think it should be possible: https://docs.microsoft.com/en-us/azure/active-directory/develop/v2-oauth2-client-creds-grant-flow#second-case-access-token-request-with-a-certificate
It would be cool to actually create a helper method to generate the JWT for client_credentials....
Where would you have your certificate stored? Are we talking a file? Or pulling it from the certificate stores?

Hello hajekj,
thx for your answer.
I guess a certificate as a file or a path would be the easiest approach.
Certificate stores i only know from java or mac os. On linux i have seen almost only files (e.g. let's encrypt).
Regards
.nowrap

Is there any specific use-case where would you prefer to use the certificate rather than regular secret? I mean, I would consider a secret in an environment variable about as secure as a file on a filesystem, but I may be wrong. I can understand that it's more secure in certificate store or I could most definitely see a very nice use-case with Azure Key Vault, so that the certificate is never exposed to the server itself.

The use case are company rules only allowing access via certificates.
And we are running on own hardware so the Azure Key Vault is not an option, if i am right?

Yup, you are right. So I think the best way to start would be to put a code together to create the JWT payload and header as per Microsoft's docs and getting it signed via the certificate (https://docs.microsoft.com/en-us/azure/active-directory/develop/active-directory-certificate-credentials). I would probably expose it as a method - createAssertionWithCertificate($pathToCertificate) which would return the JWT and then passing the client_assertion_type with value urn:ietf:params:oauth:client-assertion-type:jwt-bearer and client_assertion with value of the generated JWT.
If you ommit the client secret, I believe the library should work, depends on what AAD would do, since even if not specified, the client_secret will be added to the URL even as ...&client_secret=&... as per getAccessToken which might need to get overrided. Unfortunately, I am busy throughout the week to play with it, but I might have some spare time during the weekend to test it out.

Thx alot!
I started some research for alternative solutions based on linux. The main diffcultiy via PHP is the needed read access to everything to use. So somewhere must be a starting point, like passphrase, databases access, env var, ...
I found this so far:
https://www.codementor.io/@ccornutt/keeping-credentials-secure-in-php-kvcbrk55z

I am interest in this, too, because secrets are only valid for a maximum of 2 years on Azure. A certificate doesn't seem to have a time limit.

@hajekj could you check the MR? That worked great in my app.
I will create a fork for https://github.com/knpuniversity/oauth2-client-bundle, too, to get those new options in.

Hello, sorry for late reply, I have reviewed the PR and merged it! Awesome work, thank you so much for it! @uncaught

@hajekj thank you for the merge!
If you have time, could you publish a new release? Then I can reference the required version of this library in knpuniversity/oauth2-client-bundle for certificates to work :)
And you can close this issue, too, with the release :)

Hi, with master branch and certificate authentication, i still have AADSTS700023: Client assertion audience claim does not match Realm issuer..
I've fixed my tenant id , double check certificate fingerprint, JWT header and payload, everything looks good and match documentation, but error persist :(
payload
[
'aud' => 'https://login.microsoftonline.com/3d81913c-0473-498f-8829-61a0f999932d/oauth2/v2.0/token',
'exp' => (int)<PHONE_NUMBER>,
'iat' => (int)<PHONE_NUMBER>,
'iss' => '8b5779b7-7443-4938-8ced-f6b062dd2f11',
'jti' => 'e0e48d69b94cc6f0b627d2eff2253637af11fdaa',
'nbf' => (int)<PHONE_NUMBER>,
'sub' => '8b5779b7-7443-4938-8ced-f6b062dd2f11',
]

Header
[
'x5t' => 'ViGYIlTgwCCy4vg9uKawmNytYb8=',
]

@Erwane the tenant must match your app settings, see these options you have when creating an app on Azure:

For each of those options above, you have to set these as tenant:

Option 1 (organization only), you have to use your tenant id (3d81913c-0473-498f-8829-61a0f999932d I guess)
Option 2 (multitenant), you have to set organizations as tenant
Option 3 (multitenant + personal), you have to set common
Option 4 (personal only), you have to set consumers

You can find these values here: https://learn.microsoft.com/en-us/azure/active-directory/develop/v2-oauth2-auth-code-flow#request-an-authorization-code

My app is created with "Single tenant" option and i set my tenant id exactly how doc say.
This app work well with the "secret key" authentication, not with the certificate.

Hmm, strange. We've been using this implementation for more than half a year now with no problems. Both with a single tenant and multi tenant.
We are using a few commits before HEAD though - although from the changes that happened since then, I can't see a reason why it would break. But if you like, you could try with "thenetworg/oauth2-azure": "dev-master#dc095e5a6ae485be8a0c8b88a0d07616c18d484b".
Are you using knpuniversity/oauth2-client-bundle, too? I see we configured the end point version there:
knpu_oauth2_client:
  clients:
    azure:
      default_end_point_version: '2.0'
      type: azure


Cannot request a ticket until a user is authenticated

Is your feature request related to a problem? Please describe.
Kerberos fails with a message that is hard to understand.
Describe the solution you'd like
I would like to understand the error message.
Describe alternatives you've considered
Reading the code. This will be the next step if this fails.
Additional context
---> OracleInternal.Network.NetworkException (0x80004005): NA  Kerberos5: Authentication handshake failure at stage: %s
---> System.InvalidOperationException: Cannot request a service ticket until a user is authenticated
at Kerberos.NET.Client.KerberosClient.CopyTicket(String spn) in D:\a\1\s\Kerberos.NET\Client\KerberosClient.cs:line 1092
at Kerberos.NET.Client.KerberosClient.GetServiceTicket(RequestServiceTicket rst, CancellationToken cancellation) in D:\a\1\s\Kerberos.NET\Client\KerberosClient.cs:line 591
at OracleInternal.Network.AnoKerberos.Authenticate(String KRB5Conf, String KRB5CCName, String SN, String Host, Byte[]& kkey)
at OracleInternal.Network.Ano.KerberosHandshake(AuthenticationService AS, String KRB5Conf, String KRB5CCName)
at OracleInternal.Network.OracleCommunication.DoConnect(String tnsDescriptor)
at OracleInternal.Network.OracleCommunication.Connect(String tnsDescriptor, Boolean doNAHandshake, String IName, ConnectionOption CO)
at OracleInternal.ServiceObjects.OracleConnectionImpl.Connect(ConnectionString cs, Boolean bOpenEndUserSession, OracleConnection connRefForCriteria, String instanceName)
at OracleInternal.ConnectionPool.PoolManager3.Get(ConnectionString csWithDiffOrNewPwd, Boolean bGetForApp, OracleConnection connRefForCriteria, String affinityInstanceName, Boolean bForceMatch)    at OracleInternal.ConnectionPool.OraclePoolManager.Get(ConnectionString csWithNewPassword, Boolean bGetForApp, OracleConnection connRefForCriteria, String affinityInstanceName, Boolean bForceMatch)    at OracleInternal.ConnectionPool.OracleConnectionDispenser3.Get(ConnectionString cs, PM conPM, ConnectionString pmCS, SecureString securedPassword, SecureString securedProxyPassword, OracleConnection connRefForCriteria)
2023-07-19 15:58:41.862121 TID:9   (PRI) (EXT) OracleConnectionDispenser.Get()
The user is authenticated so what is the problem?
KR
Henrik

The error is accurate. You're at the stage of requesting a ticket and to request a ticket you need a TGT. There is no TGT in the cache available, and the only way you get a TGT is by authenticating. Therefore the user is not authenticated.

From: hwjensen @.>
Sent: Wednesday, July 19, 2023 7:17:04 AM
To: dotnet/Kerberos.NET @.>
Cc: Subscribed @.***>
Subject: [dotnet/Kerberos.NET] Cannot request a ticket until a user is authenticated (Issue #349)
Is your feature request related to a problem? Please describe.
Kerberos fails with a message that is hard to understand.
Describe the solution you'd like
I would like to understand the error message.
Describe alternatives you've considered
Reading the code. This will be the next step if this fails.
Additional context
---> OracleInternal.Network.NetworkException (0x80004005): NA Kerberos5: Authentication handshake failure at stage: %s
---> System.InvalidOperationException: Cannot request a service ticket until a user is authenticated
at Kerberos.NET.Client.KerberosClient.CopyTicket(String spn) in D:\a\1\s\Kerberos.NET\Client\KerberosClient.cs:line 1092
at Kerberos.NET.Client.KerberosClient.GetServiceTicket(RequestServiceTicket rst, CancellationToken cancellation) in D:\a\1\s\Kerberos.NET\Client\KerberosClient.cs:line 591
at OracleInternal.Network.AnoKerberos.Authenticate(String KRB5Conf, String KRB5CCName, String SN, String Host, Byte[]& kkey)
at OracleInternal.Network.Ano.KerberosHandshake(AuthenticationService AS, String KRB5Conf, String KRB5CCName)
at OracleInternal.Network.OracleCommunication.DoConnect(String tnsDescriptor)
at OracleInternal.Network.OracleCommunication.Connect(String tnsDescriptor, Boolean doNAHandshake, String IName, ConnectionOption CO)
at OracleInternal.ServiceObjects.OracleConnectionImpl.Connect(ConnectionString cs, Boolean bOpenEndUserSession, OracleConnection connRefForCriteria, String instanceName)
at OracleInternal.ConnectionPool.PoolManager3.Get(ConnectionString csWithDiffOrNewPwd, Boolean bGetForApp, OracleConnection connRefForCriteria, String affinityInstanceName, Boolean bForceMatch) at OracleInternal.ConnectionPool.OraclePoolManager.Get(ConnectionString csWithNewPassword, Boolean bGetForApp, OracleConnection connRefForCriteria, String affinityInstanceName, Boolean bForceMatch) at OracleInternal.ConnectionPool.OracleConnectionDispenser3.Get(ConnectionString cs, PM conPM, ConnectionString pmCS, SecureString securedPassword, SecureString securedProxyPassword, OracleConnection connRefForCriteria)
2023-07-19 15:58:41.862121 TID:9 (PRI) (EXT) OracleConnectionDispenser.Get()
The user is authenticated so what is the problem?
KR
Henrik
—
Reply to this email directly, view it on GitHubhttps://github.com/dotnet/Kerberos.NET/issues/349 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AAJHTYLRR62DHCZXB5YBDUDXQ7T6FBFKMF2HI4TJMJ2XIZLTSSBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJLJONZXKZNENZQW2ZNLORUHEZLBMRPXI6LQMWBKK5TBNR2WLKJVGY2DENZSGM2TPJDOMFWWLKLIMFZV63DBMJSWZAVFOZQWY5LFUR2HE5LFURXGC3LFVZ3WC5DDNBPWCY3UNF3GS5DZVRZXKYTKMVRXIX3UPFYGLJKJONZXKZNGORXXA2LDOOJYFJDUPFYGLKTSMVYG643JORXXE6NFOZQWY5LFVA4DKNBYHEYTGOECUR2HS4DFUVUXG43VMWSXMYLMOVS2UMJYGEZDANJWGAZDTAVEOR4XAZNFNRQWEZLMUV3GC3DVMWUTKNRUGI3TEMZVG6TXI4TJM5TWK4VGMNZGKYLUMU.
You are receiving this email because you are subscribed to this thread.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

OK, Thanks, I am guessing that what is going on is that we are trying to use kerberos in a way that was not intended.
We are already using kerberos without problems in an earlier version of Oracle client.  It appears that in this new implementation we shall have to implement code that retrieves the TGT from the DC and loads the cache with this before a logon can be attempted.  Do you know of example code that can get us started? We would like to work with others on this
this code will probably be needed by a lot of people who wants to use Kerberos.Net in an AD context. I am not sure this is the right forum for this though, do you know of a better place?
TIA
Henrik

It turns out that if the cache is initialized before the authorization process the ticket exchange actually works. The Bruce/kinit failed for some reason, but an old okinit from 12c database we had, did initialize the cache and that did the trick. So now there is a new problem, but this is probably with the 19c database.
Thank you for your help!
KR
Henrik


'pageunload' does not work on iOS Safari

Recently we have tried to switch to using batching for our events using Segment analytics.js however we have found that we have lost approx 15% of our events being fired, upon investigation we found this was isolated to the last events being fired on iOS Safari.
We did some investigation and found that the pageunload event is not supported by Safari on iOS. There are a few issues raised on the internet that support this.

https://stackoverflow.com/questions/4127621/is-there-any-way-to-use-window-onbeforeunload-on-mobile-safari-for-ios-devices
https://github.com/GoogleChromeLabs/page-lifecycle/issues/11#issuecomment-770031911

The official documentation from Apple https://developer.apple.com/library/archive/documentation/AppleApplications/Reference/SafariWebContent/HandlingEvents/HandlingEvents.html shows that they deprecated pageunload as well which supports the hypothesis that this won't be fixed.
Interestingly the pageunload event does work in Desktop Safari
The suggestion is that instead of using pageunload we switch the library to use pagehide which has good browser support with Chrome, Edge, Firefox, Opera and Safari having good long term support https://developer.mozilla.org/en-US/docs/Web/API/Window/pagehide_event
I debugged this with @agouil who is going to raise a PR for this change

Thank you for bringing our attention to this! It would be great to replace beforeunload, but when it comes to simply replacing it with pagehide I left a comment here https://github.com/segmentio/analytics-next/pull/608#issuecomment-1267404204


fix: Link to root of app

Fixes https://github.com/halkeye/jenkins-io-components/issues/15
cc @halkeye

can you update your commit message to fix(navbar):

i'll look into how to fix the app to allow semantic pr titles later

rawr, okay, that tests thingie doesn't work on PR, i'll override.


Update quill-jdbc, quill-jdbc-zio to 3.16.0

Updates

io.getquill:quill-jdbc
io.getquill:quill-jdbc-zio

from 3.10.0 to 3.16.0.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.getquill" } ]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #273.


Updated UI & new broadcast features

This PR introduces a new user-friendly interface for hypervision. It is built with choo v4.
There are also a number of useful features added on the broadcast screen, including a preview window, bitrate selection, and input toggling between your webcam and screen sharing.
I also added a little summary and screenshot to README.md.

woooot


Update card UI

close #125
Hi, @jayk-gupta, I've updated the UI of card. you can check the screenshot If you think It's looking fine then please merge my pull request in the main branch, and If any changes required please let me know.

There are merge confilcts please fix that
Also make a PR to dev branch!

Close this PR!

ok @jayk-gupta, I'll Fix the conflicts and make a pull request to dev branch, Thank you


KanColleViewer v3.8.2r560 did not start

Your previous realease (v3.8.2r559) work fine on my laptop. But v3.8.2r560 did not start, when i run it, nothing happen, i check task manager but no KCV icon here. When i try to delete it, OS tell me that KCV is open in system.

I just close my KCV, unzip the kcv onto the original kcv, file, and then run the file and things work normally.

I just close my KCV, unzip the kcv onto the original kcv, file, and then run the file and things work normally.

I haven't changed anything related to startup, so it should work identical to r559 (you can also have multiple instances running, so it should start even if you didn't close another KCV).
What you can try: check if there is anything in processes in task manager (not just applications), reboot if anything, remove or backup settings in %APPDATA%\grabacr.net\KanColleViewer, have it extracted to a folder with good permissions (e.g. not Program Files, but somewhere under your home directory), run it from there (it can give a security warning on a first run). Basically run it from a clean state, then it should start (and write error.log on critical errors).

I copy KCV folder to my C:\ (my OS installed here) and run with administrator privileges, after 1-2 minutes, KCV work, then I close it and run again without administrator privileges, it starts quickly. Then I come back to the KCV folder in D:\ and run, still not work. Maybe i have to put it in C:\ from now (run the KCV in D:\ with administrator privileges  didn't work)

Maybe this is cold startup, depending on the system it can take some time (20s for me).

After serveral time starting KCV in C:\ , i try to run KCV in D:\ and it work fine, maybe this problem is caused by cold startup as you say.


[Bounties] Display Bounty reward in "Bounty Done" proposal card

Display reward amount in "Bounty Done" proposals the same way its displayed for Bounty proposal

These details are not available in bounty done proposal kind. We have no way to show it.


libgit2: 1.7.2 -> 1.8.1; python312Packages.pygit2: 1.14.1 -> 1.15.0

Description of changes

1.7.2 -> 1.8.1

This also replaces the dependency on the abandoned http-parser with llhttp.

fix incorrect recursive structure in tests (tracking issue: #312345)

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

please test the reverse deps enough to not cause to many breakges

unfortunately there are lots of breakages, I'm trying to go through them when I get a chance, hence left the PR in draft for now

I meant the key reverse deps listed in passthru 😅

Bisect claims that 602760ce42b83bf9071f52d97adcbebc046f5fe1 libgit2: fix incorrect recursive structure in tests broke the eval of libgit2.tests in staging as:
$ nix build --no-link -f. libgit2.tests

error:
       … while calling a functor (an attribute set with a '__functor' attribute)
         at pkgs/development/libraries/libgit2/default.nix:73:40:
           72|
           73|   passthru.tests = lib.mapAttrs (_: v: v.override { libgit2 = finalAttrs.finalPackage; }) {
             |                                        ^
           74|     inherit libgit2-glib;

       … while calling a functor (an attribute set with a '__functor' attribute)
         at lib/customisation.nix:144:43:
          143|       # Re-call the function but with different arguments
          144|       overrideArgs = mirrorArgs (newArgs: makeOverridable f (overrideWith newArgs));
             |                                           ^
          145|       # Change the result of the function call by applying g to it

       (stack trace truncated; use '--show-trace' to show the full, detailed trace)

       error: function 'anonymous lambda' called with unexpected argument 'libgit2'
       at pkgs/applications/version-management/gitstatus/default.nix:1:1:
            1| { callPackage, lib, stdenv, fetchFromGitHub, git, zsh }:
             | ^
            2|

I think it says that gitstatus does not have an immediate libgit2 input.

Thanks for catching.
And yikes, how would we even override that? it's coming in via buildInputs = [ (callPackage ./romkatv_libgit2.nix { }) ]; which in turn has a libgit2 input...

I try to take a look at this later today


WIP filter validation checks to modified chains

h/t to @vdamle and @bitwiseguy

In favour of #453


Various improvements for long file and commit names

Fixes: https://github.com/go-gitea/gitea/issues/29438
This contains numerous enhancements for how large commit messages and large filenames render. Another notable change is that the file path is no longer cut off by backend at 30 chars, but rendered in full with wrapping.

Another nice improvement for the commits table in https://github.com/go-gitea/gitea/pull/30374/commits/cfdfda51a70feef48932f00f21ddf3cf78152997. Now the timestamp column takes up much less width, makeing more room for the commit message.
Interestingly, there was already a #repo-files-table td.age {width: 120px} rule but it was ineffective because of table-layout: fixed. The new width calculation of that th is definitely advanced, but works in all viewport widths well in my testing.


[Feautre] Add custom redact functionality to SDK

QA

see agenta/agenta-cli/tests/redact/...

@jp-agenta Thanks for the PR, it would be great if you could update the SDK howto observability to add a short section on how to redact information with a short example

@mmabrouk -- added docs.


Make debugger and storage stateless

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Tests pass?
✔

This is the alternative to https://github.com/yiisoft/yii-debug/pull/39

 Debug

Thanks!


[Feature Proposal] Stack utilities

It is common to want a border between each of the elements in a list. The Tailwind documentation recommends adding borders to each element but the first or last, using the :first-child variant:

    {{ item }}

By using the "lobotomized owl" selector, as introduced here and elaborated on here, it is possible to write HTML that looks like this:

    {{ item }}

The border class is moved from the component elements to the surrounding context. This reduces the number of classes necessary. To me, it also conveys the intention of the design better: "this is a vertical stack of elements with a border between each of them."
This is applicable to Tailwind UI. Obviously, it applies to bordered lists. In the case of panels/cards, it is especially useful in procedurally generated HTML, when a panel will sometimes have multiple sections, but other times not.
Margin and padding are very similar—often you want the same amount of space between a series of n elements. This is most easily configured on the context surrounding the elements, rather than on each element but the first or last. Otherwise, you can accidentally end up with different amounts of space between elements, or with spacing bugs when the order of elements is switched.
To support these use cases, I've been adding the following stack-* plugins to my projects. I propose they could be promoted to core plugins.
{
    plugins: [
        plugin(function ({ addUtilities, e, theme, variants }) {
            const utilities = _.flatMap(theme('padding'), (size, modifier) => ({
                [`.${e(`stack-my-${modifier}`)} > * + *`]: { marginTop: `${size}` },
                [`.${e(`stack-mx-${modifier}`)} > * + *`]: { marginLeft: `${size}` },
                [`.${e(`stack-py-${modifier}`)} > * + *`]: { paddingTop: `${size}` },
                [`.${e(`stack-px-${modifier}`)} > * + *`]: { paddingLeft: `${size}` },
            }));
            addUtilities(utilities, variants('padding'));
        }),
        plugin(function ({ addUtilities, e, theme, variants }) {
            const generator = (value, modifier) => ({
                [`.${e(`stack-border-y${modifier}`)} > * + *`]: { borderTopWidth: `${value}` },
                [`.${e(`stack-border-x${modifier}`)} > * + *`]: { borderLeftWidth: `${value}` },
            });
            const utilities = _.flatMap(theme('borderWidth'), (value, modifier) => {
                return generator(value, modifier === 'default' ? '' : `-${modifier}`);
            });
            addUtilities(utilities, variants('borderWidth'));
        }),
    ],
}

If you'd like, I'd be willing to put together a PR.

This is something I think about a lot, and I use similar custom spacing utilities in my own projects (we actually use them on the Tailwind UI website, called space-x-{n} and space-y-{n}).
I've been wanting to add this to Tailwind for a while but the border problem has made me hesitate, mainly because I'm not sure where to specify the border color and what the class should be called.
I don't personally love the stack-border-y naming convention but also don't have any really great alternatives
One alternate approach I considered was a between: variant that used the owl selector, so you could do stuff like:

...or:

I feel like I thought of a deal-breaking flaw in that solution at some point but can't remember it off-hand. It feels a little more "Tailwind-y" in some sense that it's not trying to be as cute of an abstraction and is instead more direct, but I'm not really convinced one way or another.
For borders we could even reserve a new namespace, like dividers-y or dividers-x, and then dividers-gray-500 for the color. If we did something like that and space-y-4 and space-x-4 it would feel fairly consistent at least in the sense that both have new names.

As I just mentioned on Twitter, imho this should be introduced as a variant. I’ve been using an owl variant plugin in all of my projects for a while and it does wonders (thank you Heydon for the awesome name).
To give a little context, it started as a fork of this other plugin. Reached out to the author suggesting changing it to a variant to close all of the issues opened at that time (haven’t checked now if there’s any new one). He said he’d think about it but never got back to me.
Then one day I brought it up on Discord and @benface decided to rename it and add to his tailwindcss-children variants plugin. Here’s a link to the relevant issue: https://github.com/benface/tailwindcss-children/issues/5
——
Should be a variant because it can be and will let everyone use it in other contexts we’re probably not thinking of right now. Why should that require writing more plugins and maybe even adding more stuff in core?
Regarding the border color question what I do is add that to the children or use Ben’s children variant to keep it on the parent element. By adding it to the children you can do cool things like rainbow/gradient border scale (just a silly example that someone might actually need to implement one day). Maybe in this case core should also add children?
The deal-breaking flaw might have been related to variant stacking (ie responsive + another one) or something like prefix or important. I haven’t really tested against all of core’s features but didn’t encountered any issues with the things I tried.
——
Name wise I still think owl is great because it’s short and describes what’s going to happen.
between works well for margin but not for other styles because you’re only skipping the first element.
children:not-first makes a lot of sense in the context of Ben’s plugin but I feel like it’s too long and verbose (imagine a website with 5 breakpoints and changing the space between items for each of them).

[A variant] feels a little more "Tailwind-y" in some sense that it's not trying to be as cute of an abstraction and is instead more direct

Agreed, one of the things I like about Tailwind is the lack of cuteness. A variant would be more direct. That said, I shied away from a variant in this case because you would have to know the implementation to use it correctly. For example, a variant would generate between:border-b, but using that would (almost always) be a mistake. The right way to use it would be between:border-t. That's why I chose to use the x and y mnemonics, which strays from the variant path.
To me, a variant also seems like overkill. Besides spacing and borders, I can't imagine what else you would put it on. There are a few other "directional" utilities like .rounded and .inset, but I haven't yet found the need to adjust the border-radius between elements.

I'm not sure where to specify the border color

I've wrestled with that question too, and not found a good answer.
Up until now, I've been putting the border color on the children. That's been good enough for me because I rarely have a list whose borders are sometimes gray and sometimes green. This is a question of whether the children are truly context free. I guess I care that the border's presence is context free, but not its color.
@hacknug, thanks for the pointer to benface/tailwindcss-children. If the need ever arises, I'll pull that in to use children:border-gray-500.

what the class should be called

I'm not crazy about stack either. I like divide, especially the idea of combining border position and color as divide-x-{size}, divide-y-{size} and divide-{color}. For margins, space-x-{size} is OK, though it would preclude padding helpers.
If it's definitely going to be a variant, I don't love between:. As @hacknug points out, there's no such thing as the font size "between" two children.

This is something I think about a lot, and I use similar custom spacing utilities in my own projects (we actually use them on the Tailwind UI website, called space-x-{n} and space-y-{n}).
I've been wanting to add this to Tailwind for a while but the border problem has made me hesitate, mainly because I'm not sure where to specify the border color and what the class should be called.
I don't personally love the stack-border-y naming convention but also don't have any really great alternatives
One alternate approach I considered was a between: variant that used the owl selector, so you could do stuff like:

...or:

I feel like I thought of a deal-breaking flaw in that solution at some point but can't remember it off-hand. It feels a little more "Tailwind-y" in some sense that it's not trying to be as cute of an abstraction and is instead more direct, but I'm not really convinced one way or another.
For borders we could even reserve a new namespace, like divide-y or divide-x, and then divide-gray-500 for the color. If we did something like that and space-y-4 and space-x-4 it would feel fairly consistent at least in the sense that both have new names.

I implemented this in my personal sass Library, PetricorCSS, which is inspired by Tailwind css.
I use the same (more o less) naming convention and I implemented de lobotomized owl as a variant.
Border width stack variant

Html class
Css class
Property

stack:border-t{-size?}
.stack\:border-t{-size?} > * + *
border-top: $rem

stack:border-l{-size?}
.stack\:border-l{-size?} > * + *
border-left: $rem

Margin stack variant

Html class
Css class
Property

stack:mt-{size}
.stack\:mt-{size} > * + *
margin-top: $rem

stack:ml-{size}
.stack\:ml-{size} > * + *
margin-left: $rem

The stack variant is applied in  a series of elements, be it vertical or horizontal. For this reason I only apply this to the top or bottom properties when working with borders or margins. This is what I do, but it could be applied to all sides.
Note: Notice this is applied to the parent element instead of children.
This is how implement this variant in my SASS library
Usage

Item 1

Item 2

Item 3

Item 4

I've only implemented this for border and margin utilities but I think this could be a global variant as it can be useful when designing tables.

It looks pretty complicated and creates one more thing to learn.
Isn't it more obvious to use not-first/not-last as in #1441?

Implemented in #1584 👍

Isn't it more obvious to use not-first/not-last as in #1441?

@thanosisalive I definitely considered this but for situations where you are not generating things in a loop, it feels sort of gross:

  Home
  Projects
  Team
  Settings

I agree it's simpler from a technical point of view and closer to how the rest of Tailwind has been designed up until this point, but this just feels so much nicer to me that I think it's worth it:

  Home
  Projects
  Team
  Settings

🚀 thanks @adamwathan! Looking forward to using the new tools.

It looks pretty complicated and creates one more thing to learn.
Isn't it more obvious to use not-first/not-last as in #1441?

When we talk about space between elements, applying those in a component basis makes no sense. Margins have a side effect, they affect components around them. When applying margins in components we make them less reusable. Margins work better applied in context, from parent to children. That's why the grid formatting context has gap and it's applied in the parent, not children.
Let's work with a real example. A list of social links with icons. They can be set horizontally or vertically.
/* All element but first */
.stack\:mt-4 > * + * {
  margin-top: 1rem;
}

Per component basis style

Facebook

Twitter

Youtube

Instagram

With parent context

Facebook

Twitter

Youtube

Instagram

In the second example you can add more items to the list without modifying your code. As the “lobotomized owl” technique applies margin to all elements but first —you can make the other way around, all elements but last.

When we talk about space between elements, applying those in a component basis makes no sense. Margins have a side effect, they affect components around them. When applying margins in components we make them less reusable. Margins work better applied in context, from parent to children. That's why the grid formatting context has gap and it's applied in the parent, not children.

I agree, positioning and spacing should not be applied to the components. I usually solve this problem with a component wrapper that's responsible for layout.
As in the list example, the
 itself is a layout,
's are layout items, and their contents can be any component and these components don't know anything about their whereabouts.

If you don't use
's as wrappers, then at least something like

would be not an ideal solution, but an obvious one.


fix double embed on ;handle get/set

Thanks for the fix.
I am going to change it a little. I keep sending profile embeds in set and identify, and just removing it from _set, similar to the cheran-senthil master branch.


Order by name first and by id by default

@hbagdi should we consider this a breaking change?

This is borderline but I feel this is NOT a breaking change:

The schema of the state file is part of the compat promise. The ordering inside is not.
This is a positive change that most users will welcome. I'm sure there are more people who want their git diffs of kong's configuration to look better.
I can't think of a case where a change in ordering will break someone's workflow. The ordering was nondeterministic in the first place. I mean the ordering was based on ID but someone depending on UUID-based ordering sounds really odd.

Is that fair enough?

Codecov Report

Merging #337 (b16927c) into main (f38e18d) will increase coverage by 1.07%.
The diff coverage is 71.42%.

@@            Coverage Diff             @@
##             main     #337      +/-   ##
==========================================
+ Coverage   52.58%   53.66%   +1.07%
==========================================
  Files          60       60
  Lines        4874     4877       +3
==========================================
+ Hits         2563     2617      +54
+ Misses       2013     1962      -51
  Partials      298      298

Impacted Files
Coverage Δ

file/writer.go
15.28% <6.66%> (+0.26%)
:arrow_up:

file/types.go
69.50% <100.00%> (+26.06%)
:arrow_up:

dump/dump.go
1.56% <0.00%> (-0.01%)
: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 f38e18d...b16927c. Read the comment docs.

This looks good from a product perspective. Let's merge when technical review is complete.


Jupyter notebooks of all examples are broken

The jupyter notebooks are using the old API and are not working anymore. The only working code seems to be 1D_Schroedinger.py which is not well documented.
This makes it quite hard to actually use the nice new API :(

Hey Nico thanks for mention that issue. I will update that as soon as possible.

Hey Nico thanks for mention that issue. I will update that as soon as possible.


fix: use srcDir to retrieve static folder

It looks like the staticFiles variable is not matching any static assets if the srcDir is set in nuxt.config.ts.
This PR takes the srcDir configuration from the nuxt config & resolves the static assets from its path.

Looks really good. Thank you @Gomah.

@Gomah Apologies for the delay in merging - have now had time to test - and looks great.


Applying same ext_proc filter checker rule for immediate response.

Currently, the ext_proc filter Immediate response mutation checker is very permissive. It is allow changing of anything including host/path headers, also can let any headers in without any check.  These may cause security issues.
The solution is to applying same header mutation check rule for immediate response, same as any other header/trailer response.

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@yanjunxiang-google looks good. Can you add a test, please?

/wait-any

@yanjunxiang-google looks good. Can you add a test, please?

Sure, added two integration tests to just verify immediate response now honors the filter mutation check configuration same as other response messages.  The comprehensive tests for mutation checker are already in place.

Member

This ImmediateMutationChecker https://github.com/envoyproxy/envoy/blob/7c571ac7c61768d72929feda7ef9edeb5046b56f/source/extensions/filters/http/ext_proc/ext_proc.cc#L44
basically has two rules, allow_all_routing(true) and allow_envoy(true).
This basically let the ext_proc server can send back header mutation to add system headers, like :scheme/:authority, or Envoy internal headers "x-envoy-***".  And such headers will be in the local reply messages sent back to client.  This probably is not what Envoy want. Is there a good reason to keep that door open?

Member

This ImmediateMutationChecker
https://github.com/envoyproxy/envoy/blob/7c571ac7c61768d72929feda7ef9edeb5046b56f/source/extensions/filters/http/ext_proc/ext_proc.cc#L44
basically has two rules, allow_all_routing(true) and allow_envoy(true).
This basically let the ext_proc server can send back header mutation to add system headers, like :scheme/:authority, or Envoy internal headers "x-envoy-***". And such headers will be in the local reply messages sent back to client. This probably is not what Envoy want. Is there a good reason to keep that door open?

I think essentially this comes down to one question: Will your PR break other users? From a quick scan, it seems so. For example, what if other users don't have mutation rule set but want it in immediate response OR what if they want to have different mutation rule in immediate response. They could have their own usage we don't know.
But I will let others make the call here.

We've decided to allow braking changes for now and transition API to stable after the fix for UTF-8 problem.

@tyxia  To satisfy the original goal to have a separated, more liberal set of mutation rules for ImmediateResponse, instead of having it fixed in the Envoy code,  an option is to adding a separate header mutation rule for ImmediateResponse in filter config.

We decided to have ext_proc filter to use the same header mutation rule in the filter config to check the header mutations in the immediate response from the ext_proc server.  But added a runtime flag to revert to old behavior.

LGTM with small suggestion for the release note.

/retest


Get more tests working

lists
\n\n =>

hr
code blocks
block quotes

@ShynRou ready for review

@hipstersmoothie Thanks for the PR. I'm sorry i copied the commented tests from another markdown project, but I do not approve of some of the cases. Like the behavior for . Sorry should have made that clear.
I review your changes tommorow.

okay! I'm down for adding more test cases to cover whatever behavior you have in mind

So since GitHub banished my file comments 😠 I write it as a summary here:
\n\n !=

micro-down handles a clear line as a start of a new paragraph and that should stay that way.
* list and * * * =>

as you might have noticed the * is also used for inline styles and i kept it out of the list for that reason, but I see the use of it as list item.
But your implementation clashes with the hr syntax.
I would prefer it if you remove the spaced variant of the hr syntax and only allow ***, that would remove the need for convoluted regexs for list and hr.
multiline inline
Can you explain why inline shouldn't be allowed to work in multi line text?
blockquote
The block parsing is fine with me, but why
 ?
Multiple PRs would propably been easier for both of us 😉
Best Regards,
Dustin

I will split it up into multiple PRs! I went a little too crazy lol

Looking forward for more targeted contributions.


Let 'init' put convention plugins into "build-logic" dir while migrating maven projects with incubating.

Signed-off-by: Yanshun Li<EMAIL_ADDRESS>
Fixes #23084
Context

While running 'init' command to migrate maven projects to gradle with incubating,  convention plugins are still put in buildSrc directory, which is not the same as the release note.

When generating builds with the init task and opting in to incubating features, Gradle now places convention plugins under the build-logic directory instead of in buildSrc.

Contributor Checklist

[x] Review Contribution Guidelines
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team
[x] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective
[ ] Provide unit tests (under <subproject>/src/test) to verify logic
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes
[x] Ensure that tests pass sanity check: ./gradlew sanityCheck
[x] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest

Gradle Core Team Checklist

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation

@bot-gradle test this

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

I've triggered the following builds for you. Click here to see all build failures.

PullRequestFeedback build

@bot-gradle test and merge

I've triggered a build for you. Click here to see all failures if there's any.

Thank you @Chaoba !
This will be in Gradle 8.4


RedLock Java ambiguous implementation

Hi,
we should implement a distributed lock system for our clustered Spring Boot application.
We have a Redis "cluster" (no cluster mode enabled) with Sentinel.
Now, reading the documentation it seems that RedissonRedLock should be used as locks wrapper class to use lock/unlock api.
It seems that RedissonRedLock should be used with different RedissonInstance e multiple RLock's.

RedLock should only be used in Redis running in cluster mode, as per specification.

Ok. What about configuration with cluster mode enabled?
What are the differences between RedissonRedLock and RLock?


Add Coding Game seo for google search

Enhancement
Add schema to help CodeCombat get found on google under "Coding Games". Schema only added to home page.

@codeluggage Thanks for the feedback. Will add it to the task and handle in a separate PR.


Fix config bugs

There are a couple parameters missing from the config which leads to train.py erroring. It's a pretty easy fix, we just need to edit config.py line 234 to
logger_params = OmegaConf.to_container(self.cfg.logging, resolve=True)

and then edit base.yaml to add the following:
logging:
      logger_type: null

view_batch:
    enable: false
    no_train: false

Fixed


Options aren't optional

When calling getAllItems() on Android with no parameters supplied, it will error out with:
com.facebook.react.bridge.NativeArgumentsParseException: RNSensitiveInfo.getAllItems got 2 arguments, expected 3

If I pass in an empty object it will throw the exception error:
com.facebook.react.bridge.NoSuchKeyException: couldn't find key sharedPreferencesName in dynamic object

I am basically forced to specify an object with a sharedPreferencesName property. This applies to all the methods on Android that take an options parameter.

Same on iOS, the options are not optional. Maybe, for the beginning update the readme file.


Fix transaction ordering for ETH txns

This resolves Issue https://github.com/OpenBazaar/openbazaar-go/issues/1893.

Coverage remained the same at 40.998% when pulling 5203cf911586a94c924e527b915f7abb4a14ac5c on fix_transaction_ordering into c110900e13ec8d621707e9e12d7c03a73da86018 on ethereum-master.


UIRefreshControl support

This PR should address #141
I've updated the demo project to see it working, including a simple scrollview with a UILabel with a lot of bottom padding.
Thoughts?

I've added proper unit testing for UIRefreshControl

This look totally good all test passing will be making a release with pods semantic version updates


Avro datum factory fixup

Addressing post-merge comments for https://github.com/apache/beam/pull/26320

Remove DecimalConversion from default logical-type conversions
Restore AvroCoder cloud object compatibility by checking presence of previous fields.

Run Java Examples_Direct

@RustedBones Thanks for a quick fix, beam_PostCommit_Java_Examples_Direct_PR is passed.
Could you fix Spotless check, please?

Thanks for the quick fix!

@aromanenko-dev Sorry, learned. I meant to approve. Will explicitly do so.
I am trying to close active test failures before the 2.49.0 release cut


chore: add projectshut open source project

I want to add projectshut open source project.
What is ProjectsHut?

Projectshut is an open-source web app that enables users to freely publish their projects and create user profiles within the platform.

Repository URL
https://github.com/priyankarpal/ProjectsHut

i dont understand your issue

I mean I want to add my repository for good-first-issue


fix bug in body-fixed XYZ Euler angles continued

obsoletes #36

Has anyone tested this in actual use?  I don't understand what is going on here... you just rotated the vehicle and gimbal 90 deg., but not the rest of the world?  How is this backwards compatible with anything currently working?
-Max

@mpaperno This fixed an incompatibility with PX4/Firmware. I'm not aware of other users of PX4/jMAVSim; DrTon/jMAVSim is unchanged

This fixed an incompatibility with PX4/Firmware.

Didn't anyone use jMAVSim with PX4 before this fix?  Or was it always broken?

I'm not aware of other users of PX4/jMAVSim; DrTon/jMAVSim is unchanged

@LorenzMeier indicated earlier this is the new master for jMAVSim in general.  DrTon's version is no longer maintained/updated/gets any response, otherwise I would have submitted all my previous updates to him instead.   There is clearly code in there now to support different autopilots.  Which I took the time to do instead of making everything specific for the AP I use.  This is done to keep the versions compatible and any improvements easier to carry over/merge as-is.
And you didn't answer my first question.
-Max

@mpaperno This swaps roll & pitch axis for the attitude. I consider this a bug fix, since the previous implementation did not comply with NED (It's not documented within jMAVSim, but that's the standard we are using). I tested this within PX4 and found nothing wrong, so could you please explain what exactly breaks for you?
Yes this is the new official version, so external projects should be able to use this, and I do care about their compatibility.

@mpaperno Sorry, thoroughly tested by me at least. Please let me know if your testing indicates any actual issues with other systems.

@mpaperno And yes, I've been a user of and contributor to PX4 and jMAVSim for several years.

OK, thanks for clarifying that Beat.  I see now that attitude is something I added and it is (or was) only used for the gimbal.  I agree pitch & roll were swapped -- IIRC this was done to get the gimbal working properly but w/out anyone needing to swap their gimbal roll/pitch channels.  Perhaps it's better this way in the long run, and the attitude is now shown properly in the reporting output.
I do see attitude is now being used in the HIL_STATE_QUATERNION message... another way to get the quats would be right from the rotation matrix (same thing as attitude is derived from):
Quat4f quaternion = new Quat4f();
quaternion.set(vehicle.rotation);

-Max


Removing Windows Documentation

He's just removing the document, not the main program, but saadrahim didn't say anything regard this action, though

why ???? we waited for windows release

yeah... i dont know why he would remove it.


Updating to new options pattern

https://github.com/aspnet/Options/issues/105

This middleware currently is only configured by options added in DI from ConfigureServices. I'm assuming there is no need to add an overload to take an instance of SessionOptions.

Add the UseSession extension that takes new SessionOptions.

:up::date:

:shipit:


Settings editor entries jump up when interacted with

open setting UI
type "inlay"
scroll so that a setting is half revealed
select/unselect it
:bug: the setting jumps up, it does not always select/unselect the item

(sometimes needs a few tries/scrolls to reproduce...)
https://user-images.githubusercontent.com/1794099/135087770-8a4f6a67-5323-439c-967b-353596989d61.mov

Something to do with these inlay settings and their dumb async rendered descriptions.

The dynamic heights of the rows with this markdown are not handled correctly, we have to rethink this. For example, these rows will start overlapping if you resize the editor.

For me, this happens not sometimes, but all the time, to the point that it's effectively impossible to edit settings via the editor, forcing me to edit JSON directly.

I second that.
Some recent update causes the right pane in the UI to jump, "jump scroll" upwards in roughly 40px increments and on occasion see the cursor/focus hopping fields as well as between the two panels:  the outline changes on the active field/pane.
It randomly works but for the most times the UI became unusable.

I can add to this conversation that this issue pops up if I have the following setting in my settings.json file.
"terminal.integrated.profiles.windows": {
    "PowerShell": {
      "source": "PowerShell",
      "icon": "terminal-powershell"
    },
    "Command Prompt": {
      "path": [
        "${env:windir}\\Sysnative\\cmd.exe",
        "${env:windir}\\System32\\cmd.exe"
      ],
      "args": [],
      "icon": "terminal-cmd"
    },
    "Git": {
      "path": "C:/Git/bin/bash.exe",
      "args": ["-i", "-l"]
    }
  },
"terminal.integrated.defaultProfile.windows": "Git"

In my case I have GIT installed in a non standard location. This issue goes away if default profile is not set.
The following is my vscode version.
Version: 1.62.3 (user setup)
Commit: ccbaa2d27e38e5afa3e5c21c1c7bef4657064247
Date: 2021-11-17T08:11:14.551Z
Electron: 13.5.2
Chrome: 91.0.4472.164
Node.js: 14.16.0
V8: 9.1.269.39-electron.0
OS: Windows_NT x64 10.0.19042

I'd like to confirm @abelgladstone's findings, but in my case it doesn't have to do with Git.
Here's the relevant part of my settings.json:
"terminal.integrated.profiles.windows": {
        "PowerShell": {
            "args": [
                "-nol"
            ],
            "path": "C:/Windows/System32/WindowsPowerShell/v1.0/powershell.exe"
        },
        "PowerShell Dev": {
            "args": [
                "-nol",
                "-noe",
                "-c",
                "&{Import-Module 'C:/Program Files (x86)/Microsoft Visual Studio/2019/BuildTools/Common7/Tools/Microsoft.VisualStudio.DevShell.dll'; Enter-VsDevShell 2a7a9ed6 -DevCmdArguments '-arch=x64'}"
            ],
            "path": "C:/Windows/System32/WindowsPowerShell/v1.0/powershell.exe"
        }
    },
"terminal.integrated.defaultProfile.windows": "PowerShell Dev",

For me, it's enough to comment the terminal.integrated.defaultProfile.windows line while leaving terminal.integrated.profiles.windows entry alone, and it fixes the buggy behavior of settings editor immediately.

For the Settings editor issue related to the terminal profile settings, see https://github.com/microsoft/vscode/issues/136509.
@jrieken does your original issue also go away when commenting out the terminal profile setting(s)?

This problem seems to be fixed for me in version 1.63.2

1.63.2 fixes this for me too, yay!


Don't set CMAKE_SYSTEM_NAME when using the Visual Studio generator

Setting CMAKE_SYSTEM_NAME and CMAKE_SYSTEM_PROCESSOR is often enough for CMake to handle cross-compilation even without a toolchain file, but it gets in the way of generators that handle cross-compilation on their own (in Visual Studio's case, using the -T "toolset" option).
Narrow fix for the issue I brought up in https://github.com/rust-lang/cmake-rs/pull/158#issuecomment-1544695163. Possible fix for #171 as well, if it does turn out to be the same thing.

(Test failures are also happening on upstream.)


Making allele_frequency nullable

Old: population_allele_frequency->allele_frequency (float)
New: population_allele_frequency->allele_frequency (float or null)

This is no longer required


Error at DeepDream demo [prototxtParser is not defined]

Got this error, while tried to run DeepDream thing in two browsers (Opera & Chrome)..
Uncaught (in promise) ReferenceError: prototxtParser is not defined
    at PrototxtParser.parseString (caffe.js:6029)
    at caffe.js:2054

thanks for reporting this error! I fixed it and it should work again :)
once it is runing, it should look like this:


Release

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

GitHub release
v2.0.0
v2.0.0

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.0.0-dev.3 :tada:
The release is available on:

GitHub release
v2.0.0-dev.3
v2.0.0-dev.3

Your semantic-release bot :package::rocket:


fix: stop making classmethod generic at runtime

This fixes a problem where in class_utils.py, classmethod is made generic at runtime when you don't have to.

Ah, that's a great solution, thanks! Let me quickly remove a line from pyproject.toml so we no longer ignore the now-removed classmethod override, and I'll get this merged.


Add support for stopping sound expressions.

Alternative to #7 to address #6.
See https://github.com/lancaster-university/microbit-v2-samples/pull/5 for a test that lets you explore the behaviour.

Thanks @microbit-matt-hillsdon. Looks grand!


Resources - alphabetical order

It would be nice if the list was alphabetical as it would be easier for the eye to scroll down: http://shescoding-wamuq6fved.elasticbeanstalk.com/resources
Career Advancement
Diversity
Education
Gaming
Hiring
Kids
Leadership
Local
Mentorship
Negotiations
Networking
Online Classes
Podcasts/YouTube Channels
Research/White Papers
Sexism
Statistics
Workshops

Fixed in #67.


Failed to download VPN device configuration template

@nnamedieh commented on Wed Aug 22 2018
Hi all,
unfortunately I ran into an error when I tried to download any VPN device configuration template from the "Connection". I copied and pasted all the powershell commands mentioned in this documentation to create all necessary services, but when I want to download the VPN device configuration it shows the following error:
"File download error: Failed to dynamically fetch target download uri."
Looking into the activity log the error is
"Error code IpsecPolicyNotSupportedOnVpnDevice"
But I actually don't configure any ipsec policy on the VPN gateway, just copied the instructions mentioned in this doc.
I figured out that when I disable BGP in the Connection, I am able to download any vpn device configuration template.
Can anybody help me to figure out what's the problem in this case and why I am only able to download the vpn device configuration template when it's disabled in the connection?
Thank you very much,
Daniel

Dokumentdetails
⚠ Bearbeiten Sie diesen Abschnitt nicht. Er ist für die Verknüpfung von docs.microsoft.com zum GitHub-Artikel erforderlich.

ID: 27b300f3-7b57-0772-9a36-6448a56b3d23
Version Independent ID: 959e9abd-3033-df6c-cb30-8277b0acbf0d
Content: Konfigurieren von Aktiv/Aktiv-S2S-VPN-Verbindungen für VPN Gateways: Azure Resource Manager: PowerShell
Content Source: articles/vpn-gateway/vpn-gateway-activeactive-rm-powershell.md
Service: vpn-gateway
GitHub Login: @yushwang
Microsoft Alias: yushwang, cherylmc

Hi team,
Can you check on this issue?
Thanks!

@LisandroSu are you trying to set up a VPN between 2 VNETs or for on-prem to Azure? also, what is the SKU of the gateway that you are using?

Hi @TravisCragg-MSFT , thank you for your answer. I wanted to build a vpn connection between Azure and on-prem with a Azure VPN gateway SKU VpnGw1.
And - just for understanding purposes - it works! But when I enable BGP in "Connection" and want to download the vpn configuration for on-prem vpn gateway, an error occur (take a look at initial post).
I build this conncetion exactly the same like in this post (Part 1 & 2), but I'm still not able to download the vpn device config with BGP disabled in connection -->
https://docs.microsoft.com/en-us/azure/vpn-gateway/vpn-gateway-activeactive-rm-powershell
Thank you, Daniel

@nnamedieh what VPN device are you using on-premise?

Hi @TravisCragg-MSFT ,
thanks for your reply! Unfortunately it doesn't matter which configuration I choose - with bgp enabled in connection I'm not able to download any configuration for on-prem template. Did you rebuild the environment like in the Azure Doc I linked in my last post? Maybe anyone else has the same problem?
Thank you!

@nnamedieh We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

@LisandroSu did you ever fix this? We have just run into the exact same issue. Raising a support ticket directly, but would be good to know what the problem was in the end?

Hi @ksix ,
I think I solved the problem (at least in my case). The problem was that I had a ReadOnly resource lock configured on the VPN Gateway. When you want to download the vpn config template, it seems that Azure needs to do a write operation to be able to provide the template. Since I deleted the resource lock I was able to download the config template.
Hope that helps!
Daniel

This can occur if the service is newly created. While it may be available ensure that Azure is 100% done with the deployment.

We had the same issue, an the comment from dani3lheidemann helped. For us it was not due to a lock but about the write operation
We gave Contributor rights and it works.

I can also confirm that the fix for this for us was NOT a resource lock but rather insufficient permissions. I assigned my account an owner level permissions using the RBAC IAM at the Azure Subscription level the signed out and then signed back in again and now was able to download the configuration for the Azure VPN client. It is simply a zip file with two XML configs. There is no exe/msi/msix/appx in there.

I had the same issue but I was able to download it in the end. I just set up this VPN profile so Azure was probably not done with deployment. They should have an appropriate error message for this though. I would recommend waiting at least 10-15 minutes after setting up a P2S


[WIP] fromJSON does not work for range within object

@Zalastax would you have any clue why this test fails? From debugging I found that it fails at line 332:
__traits(compiles, mixin ("t." ~ name ~ "= fromJSON!("
                ~ (typeof(__traits(getMember, t, name))).stringof
                ~ ")(jsonAA[\"aFieldName\"])"))

As far as I can tell that should translate into:
t.routes= fromJSON!(Route[])(jsonAA["aFieldName"])

and should compile.

I checked what it compiles to (by printing the mixin):
t.la= fromJSON!(string)(jsonAA["aFieldName"])
t.routes= fromJSON!(Route[])(jsonAA["aFieldName"])
I will investigate further.

I found out why it doesn't compile. I added a static if that would only be true for the case I was searching for. Inside the static if I placed the code I wanted to get the compile errors for.
Route isn't available in the scope.
source\painlessjson\painlessjson.d(358): Error: undefined identifier Route

So what do you need to do to solve this? For the library user the only available solution right now is to move Route to a public place.
Is there any possibility for the library to do anything different? If we can't support the use case at least add a better error message? If we can't produce a better error message we need to document it clearly.

Thanks @Zalastax for finding that out. I did just try moving the test and moving JourneyPlan to a separate source file and even that did not work. Seems like you would have to move it to a separate library, which seems a big limitation.
About better error: I guess it should be possible to check whether a type is known at compile time.
About fixing it. Not really sure how we could do that. I guess the problem is that the new types are not known yet at compile time. Could we force them to be loaded first? I guess going the mixin approach of jsonizer would work around it. Wonder if cerealed has solved this problem?

I think this fix is quite good. The user only have to make their symbols accessible by import.
Could you try to fix the errors on older compilers?

Thanks @Zalastax. I guess you still need to define the unittest in a different module, but this is a definitive improvement.
Not sure though if this is fixable on older compilers though. Error looks like a bug in the std library of the older compilers.
Guess could make test etc compiler version specific

Ok, I think this feature is important enough that it warrants dropping support for the older compilers. @Zalastax do you agree? If so I will merge this and bump the version.

Yeah. There seems to be substantial bugfixes in the latest released compiler. Before merging you could probably replace https://github.com/BlackEdder/painlessjson/blob/master/source/painlessjson/painlessjson.d#L391-L404 since that bug is fixed in the latest compiler version.


Ionic 5: Tapping to stop scrolling a list on an Android device evokes a (click) event

If you start scrolling the list with a flick, you can tap the screen before the scrolling finished to make the scrolling stop early.
The problem is in Ionic 5, on a physical Android device, the tap to stop the scrolling also calls the (click) of any element underneath where you tapped.
The expected behavior would be for that tap to only stop the scrolling and not carry through to elements. It worked this way in Ionic 4. In Ionic 5 it still works that way on iOS but it's carrying the click through to other elements on Android.
Bug Report
Ionic version:
[x] 5.x
Current behavior:
The (click) event is firing on Android when you tap to stop the scroll
Expected behavior:
Stopping the scroll with a tap shouldn't open the item. This works correctly on iOS
Steps to reproduce:

Open the sample app attached on Android device
Flick the list to start scrolling and tap to stop the scroll. The (click) event will fire on Android

Related code:
https://github.com/jswilliams/ionicListScrollBug

Thanks for the issue. What kind of Android device are you testing this on?

Thanks for the quick reply! We're testing on a Samsung Note 8 (SM-N950U), Android 9.0

Update: I can duplicate this in the Android emulator using the Pixel 2 API 29 as well.

If you set [scrollEvents]="true" (for angular apps) on ion-content does that fix the issue?

@liamdebeasi Indeed this will fix this issue. But the solution is not very intuitive. This also will lead to performance regression on older devices.

Oddly enough scrollEvents were on for all Ionic 4 apps. If your apps worked fine on v4 it should be safe to re-enable scrollEvents for now.
This may just be a quirk with the Chromium webview, but I am not 100% sure yet.


creating

[ ] my name is pallavi

created a task

[ ] Turn on GitHub Pages
[ ] Outline my portfolio
[ ] Introduce myself to the world


Look at old notebooks on ANN

[ ] check out the notebooks
[ ] Check out tutorials online  (Statquest videos, etc. )

completed


Add Brazil (17 cities)

Hello. Thanks for great work.
Would it be possible to add Brazil to the website?
I've identified 17 cities that together can cover the entire territory of Brazil (I just skipped an area where basically the entire population is indigenous).
If that helps in any way:
"São Sebastião do Oeste" 500km
"Jacutinga, PR" 500km
"Encruzilhada do Sul" 500km
"Geraizinho, MG" 500km
"Tupanaci, PE" 500km
"Cabeceira Velha, BA" 500km
"Jaupaci, Goias" 500km
"Tangará da Serra" 500km
"Goianésia do Pará" 500km
"Azauri, PA" 500km
"Rorainópolis" 500km
"Japurá (Amazonas)" 500km
"Ponta Alegre, AM" 500km
"São João, RO" 500km
"São Antonio, AC" 500km
"Três Bocas, AM" 500km
"Aquidauana" 250km
Thank you.

Thanks for the help, ended up adjusting to use 18 tabs for fuller coverage. Pushing it to the site.


[FeatureRequest] I have fully translate to Greek language.

I have fully translate (via weblate) the Greek language for Amarok.
If you see any issues, just tell me.
Thanks for this app.
Keep the good work.

Some strings is read only.
Like "Night has fallen. Be careful."
In Greek they have nice translations for this and the other read only.

Thanks for your contribution!

Thanks for this great app. 🙏
I noticed, some strings is read only.
Like "Night has fallen. Be careful."
In Greek they have nice translations for this and the other read only.


Add usage example to the README

Would be nice to have an example of how to use primer-primatives IRL 🎉

Fixed in https://github.com/primer/primer-primitives/pull/6


Update to new look for Octicons (v10.0.0)

Hello 👋 I'm not sure if this project is still maintained, but I've been using it for some time now.
You may have noticed that GitHub's octicons have been updated to a new look (v10.0.0). I'm wondering if you will be updating this package to use these new octicons?
Thanks!

Hi @sashaafm. I am planning to add the new icons styles this weekend.


[I am started following the roadmap]

I am gonna finish learning this roadmap and I will share my experiences through this repository !

Welcome @Yourpabi2002


Should t.same use assert.deepStrictEqual?

Hi
The t.same method uses assert.deepEqual. This is also true when primitives do not have the same type. When using assert.deepStrictEqual, this is not the case.
assert.deepEqual({foo: 5}, {foo: '5'});
// true

assert.deepStrictEqual({foo: 5}, {foo: '5'});
// false

Same for t.notSame.
If for some reason, the implementation should be kept this way, new assertions can be added? t.strictSame and t.notStrictSame or something like that.

Good point, I think we should switch. @sindresorhus what do you think?

Not sure when the strict variants where introduced though. They do not exist in 0.12 so maybe we should use a ponyfill for older versions?


GrudGenerator: Allow extending name attribute list

When doing single-language (but not english) applications, one may have localized names in database columns. Therefore, one may have 'nombre' or 'nom', instead of 'name'.
This change allows setting in the module configuration the extra names so that the CRUD generator will pick them up.

I think this should better be done by extending the generator than having it in the framework.

Agree. Name attribute detecting could go more complex than simple string matching. I prefer we keep the core code simple so that it is easier to be extended.


crl_auto_renew not working RedHat versions due to alias

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 7.20.0
Distribution: CentOS Stream 8
Module version: v10.2.1

How to reproduce (e.g Puppet code you use)
crl_auto_renew: true

from hiera for openvpn::serevers:
What are you seeing
The certificates don't auto renew all though they used to, maybe before we switched to Centos Stream, can't remember the details of exactly when it went wrong, I tought that the fix in the changelog for v10.2.1 would sort it but it didn't for us https://github.com/voxpupuli/puppet-openvpn/pull/437
What behaviour did you expect instead
Certificates are renewed automatically
Any additional information you'd like to impart
I have been digging into it deeper and I believe the issue is because RedHat sets up an automatic alias for the cp command which turns it into interactive mode
$ alias
alias cp='cp -i'

I had noticed that when we run it manually to fix then it always asks for confirmation.

Shell aliases don't exist in the shell environment of puppet.

Damn thought I was on to something, guess I need to keep looking.
@kenyon do you have any pointers where I can look?

even if the provider is 'shell'?

Yes, even if the provider is shell.
Also "crl" is certificate revocation list, which has nothing to do with certificate renewals. I don't see any certificate renewal functionality in this module.

In manifests/server.pp
## Renewal of crl.pem
      if ($crl_auto_renew) {
        schedule { "renew crl.pem schedule on ${name}":
          range  => '1 - 4',
          period => $crl_renew_schedule_period,
          repeat => $crl_renew_schedule_repeat,
        }
        case $openvpn::easyrsa_version {
          '2.0': {
            exec { "renew crl.pem on ${name}":
              command  => ". ./vars && KEY_CN='' KEY_OU='' KEY_NAME='' KEY_ALTNAMES='' openssl ca -gencrl -out ${server_directory}/${name}/crl.pem -config ${server_directory}/${name}/easy-rsa/openssl.cnf",
              cwd      => "${server_directory}/${name}/easy-rsa",
              provider => 'shell',
              schedule => "renew crl.pem schedule on ${name}",
            }
          }
          '3.0': {
            exec { "renew crl.pem on ${name}":
              command  => "./easyrsa gen-crl && cp ./keys/crl.pem ${server_directory}/${name}/crl.pem",
              cwd      => "${server_directory}/${name}/easy-rsa",
              provider => 'shell',
              schedule => "renew crl.pem schedule on ${name}",
            }
            ~> exec { "copy renewed crl.pem to ${name} keys directory":
              command     => "cp ${server_directory}/${name}/easy-rsa/keys/crl.pem ${server_directory}/${name}/crl.pem",
              refreshonly => true,
              provider    => 'shell',
            }
          }
          default: {
            fail("unexepected value for EasyRSA version, got '${openvpn::easyrsa_version}', expect 2.0 or 3.0.")
          }
        }
      }

we use easyrsa version 3, just for extra info

I added these values to our config
crl_renew_schedule_period: 'daily'
crl_renew_schedule_repeat: 8

and it did actually renew certificate, so I have no idea why it was failing and requiring me to do it manually after the certificate expired every month.
We have 3 VPN server configured by puppet so I have changed them to different settings and will keep an eye on if it regenerates them and will see if we get the same issue next month

There is no certificate being renewed, it is the CRL that's being renewed.

Okay sorry my wording is wrong, whatever it is it's not been working (although use to with this module) the error is TLS error which made me think of certificates.
Whatever it is it still brings down the VPN


ruby: bump | claim | update buildfile

Existing package

[x] I am the new maintainer of this package.

#864

The sed hack probably isn't required now with busybox 1.35.0, i think you forgot to remove the sed command along with the comments

The sed hack probably isn't required now with busybox 1.35.0, i think you forgot to remove the sed command along with the comments

Cleaned it up


SOTA of Face Alignment

Just a kind reminder!
3.41 may not be SOTA for  AFLW2000-3D. To my knowledge, there already exists lower NME (2.81) result, please check Uncertainty-Aware Mesh Decoder for High Fidelity 3D Face Reconstruction for more details.

Thank you for the remind.
There are two annotation versions of AFLW2000-3D. Original and Reannotation
from LSW-3D. We reported the number separately. More details can be checked
in our paper and supplementary. Our NME of 3.41 is reported on the original
version and based on current SOTA 3DDFA-V2. For using the better quality
annotation, our NME is also reported in the supp.
On Tuesday, April 20, 2021, Xscape @.***> wrote:

Just a kind reminder!
3.41 may not be SOTA for AFLW2000-3D. To my knowledge, there already
exists lower NME (2.81) result, please check Uncertainty-Aware Mesh
Decoder for High Fidelity 3D Face Reconstruction
https://urldefense.com/v3/__https://openaccess.thecvf.com/content_CVPR_2020/papers/Lee_Uncertainty-Aware_Mesh_Decoder_for_High_Fidelity_3D_Face_Reconstruction_CVPR_2020_paper.pdf__;!!LIr3w8kk_Xxm!5IWABK_ZMQwqlYg-0xp5rLLdtseoQbXortAvgIiedao9xnXbCCTiuABsxYJqds5x$
for more details.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://urldefense.com/v3/__https://github.com/choyingw/M3-LRN/issues/1__;!!LIr3w8kk_Xxm!5IWABK_ZMQwqlYg-0xp5rLLdtseoQbXortAvgIiedao9xnXbCCTiuABsxeniDeng$,
or unsubscribe
https://urldefense.com/v3/__https://github.com/notifications/unsubscribe-auth/AKMLAJOCYMU4JMBMEKV7DU3TJZO5HANCNFSM43JTRPFA__;!!LIr3w8kk_Xxm!5IWABK_ZMQwqlYg-0xp5rLLdtseoQbXortAvgIiedao9xnXbCCTiuABsxTqwNs5E$
.

--
Ph.D. candidate of Computer Science Department
University of Southern California

Since the results of PRNet is 3.62 reported in Uncertainty-Aware Mesh Decoder for High Fidelity 3D Face Reconstruction (CVPR2020). So I believe their result is based on the original annotation, which is 2.81.

Thank you for the interest in the SOTA comparison. The annotation quality of AFLW2000-3D of the original version raises issues as mentioned in PRNet. Much lower NMEs on the original version seem to be tricky. NME of 2.8 to 3.3 are reasonable on the reannotated version using for the recent methods with their publicly released codes.
The paper you mentioned did not release codes after the publication, and thus the performance cannot be validated. Besides, ECCV 2020 is a venue that came after CVPR 2020, and 3DDFA-V2 has released their codes (with 1.5k stars) to verify their performance. Thus, we believe 3DDFA-V2 is the current SOTA on the task.
I hereby clarify the issue you raised. I hope this helps.
Thank you,
Cho-Ying Wu


Updated code to make it work with Swift 3.0 PREVIEW 6 on Ubuntu

These updates make SwiftCLI "compileable" with Swift 3.0 PREVIEW 6 on my Ubuntu installation resolving #18.
Just followed compilation errors and Foundation library docs to remove issues.
BTW You need to check class function checkForPipedData() in Input.swift as I wasn't sure how to solve the issue, so I just commented out problematic lines. Can be Swift guys corrected behavior you were trying to solve with this function and now it works OK.
Feel free to comment.
BTW Thanks a lot for SwiftCLI, it's making creating my app much more easier.

Your changes broke compilation on Mac unfortunately -- it seems as though these issues are caused by discrepancies between Swift Foundation and Objective-C Foundation. Commits 810821bd9960f8037781dc1abb179727c777e2d0 and  b391d9f2c2921d1475f55ef51cd7f94381a77c51 should fix these issues for now, but by the time Swift 3 is released, these changes should no longer be necessary

That breakage was unintentional. I don't have Mac so I couldn't test it. Thanks for code updates.


Branch input defaults to undefined string

When the branch input is not specified it defaults to "undefined" instead of "master" or something more sane.
/usr/bin/git clone https://github.com/odin-lang/Odin /home/runner/odin --branch undefined --depth=1 --single-branch --no-tags

nvm, looking at the inputs it seems that it actually defaults to master... where did the undefined come from then
Run laytan/setup-odin@v2
  with:
    release: false
    llvm-version: 17
    cache: true
    branch: master
    build-type: debug
    repository: https://github.com/odin-lang/Odin
Warning: Invalid access token, falling back to git based install.
Main cache key: linux-https://github.com/odin-lang/Odin-undefined-debug-llvm_17
LLVM 17 comes pre-installed on this runner
Main cache MISS
/usr/bin/git clone https://github.com/odin-lang/Odin /home/runner/odin --branch undefined --depth=1 --single-branch --no-tags
Cloning into '/home/runner/odin'...
warning: Could not find remote branch undefined to clone.
fatal: Remote branch undefined not found in upstream origin
Error: The process '/usr/bin/git' failed with exit code 128

Thanks! Should be fixed now.
Also, I recommend testing this out with cache disabled, in my experience it is actually slower to use the cache than just fetching every run.